Feat: BE - new endpoint for showing all projects' progress

What's changed:

backend/projects/views/project.py: added new view for listing all projects' progress
backend/projects/urls.py: new endpoint added

The new endpoint is localhost:8000/v1/projects/progress. It will list all projects' progress of current logged in user. The format is below:
{
    "count": 25,
    "results": [
        {
            "project_id": 1,
            "total": 3,
            "remaining": 0,
            "complete": 3
        },
        {
            "project_id": 2,
            "total": 33,
            "remaining": 32,
            "complete": 1
        },
        {
            "project_id": 3,
            "total": 33,
            "remaining": 33,
            "complete": 0
        },
    ]
}

Hello, sorry for the late confirmation,  but I can confirm that this api works well on fe xD


Add settings page and manage activated features via the admin panel

Add settings page to WordPress Admin panel, allow activation and management of plugin parts therein.

Using functions.php module include pattern instead


Possible Issue with --sam-long output

It looks like the SEQ and QUAL fields of the sam file are being concatenated in the nucmer output V4.0.0beta2
Also, is there a description of the delta file format out there?
thank you.

Regarding the first issue, I don't see it. Actually, as it stands, nucmer does not output the QUAL field (just a star).
The format of the delta file is described in the old documentation, in docs/nucmer.README. The doc described version 3, but the format has not changed. This description will be in the MANUAL.


sanjo:jasmine errors in log loading two js files

Using this command:
JASMINE_CLIENT_UNIT=0 JASMINE_CLIENT_INTEGRATION=0 JASMINE_SERVER_UNIT=1 JASMINE_SERVER_INTEGRATION=0 DEBUG=1 JASMINE_DEBUG=1 VELOCITY_DEBUG=1 VELOCITY_DEBUG_MIRROR=1 ROOT_URL=http://localhost:4000 meteor --port=4000 --settings ./tests/settings/local.json
I get two errors while js files are loading:
W20150420-22:09:59.249(-7)? (STDERR) [sanjo:jasmine]: The file "/Users/luis/Jobby-Web/lib/mixins.js" has syntax errors. [TypeError: Property 'memoize' of object # is not a function]
W20150420-22:09:59.265(-7)? (STDERR) [sanjo:jasmine]: The file "/Users/luis/Jobby-Web/server/craigslist.js" has syntax errors. [TypeError: undefined is not a function]
I'm guessing it may have to do with the stubs it autobuilds... the lines it complains about are:
mixins.js:
if (Meteor.isServer) {
// Memoize query on the server
var memoize = WorkpopNpm.require('memoizee');
createMatcher = memoize(createMatcher, {max: 10});

}
craigslist.js: (not positive here... these look suspect)
var webdriver = WorkpopNpm.require('browserstack-webdriver');
var fp = WorkpopNpm.require('firefox-profile');
var profile = new fp();
Cheers,
Luis

Missed this one also in the logs earlier but which I read is ok to ignore on other posts:
W20150420-22:14:15.878(-7)? (STDERR) Could not mock the export WebAppInternals of the package webapp. Will continue anyway.


Add socket address template type

Fixes #101.
Not sure whether the hassle is worth it, Psalm can infer it, PHPStorm doesn't seem to infer it correctly.

I don't think this is worth it. Socket gets used quite a lot and will require a template annotation at every use, since Psalm doesn't support a default template argument 😞
I'm not sure this is entirely accurate either, as connectors don't pass the SocketAddress instance on, the returned socket can actually return a different type of SocketAddress if someone were to implement their own CustomSocketAddress, the returned Socket would still be of Socket<InternetAddress> or Socket<UnixAddress>, not Socket<CustomSocketAddress>.


[ARM] Fix the test of test_resource_policyset

Fix the test of test_resource_policyset
issue: #6008
I ran the test again and found that the problem could not be repeated. So I removed @live_only to add this test case to the CI process.

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

Please merge this PR before this build.


Struct Returned findSplit*() with implicit bool conversion

This enables the following useful syntax in pattern matching:
if (const split = line.findSplit(`-`))
{
    // do something with split
}

instead of current
const split = line.findSplit(`-`)
if (!split[1].empty)
{
}

for the Phobos algorithms findSplit, findSplitBefore and findSplitAfter.
See also discussion at: http://forum.dlang.org/thread/etttlwazcguwpaltyrps@forum.dlang.org#post-tbnvyqawcqmccvociwrr:40forum.dlang.org
Module std.algorithm compiles and tests ok. However compilation of std.traits which in turn uses std.typecons fails as
../std/typecons.d(605,47): Error: reinterpreting cast from inout(string)* to inout(Tuple!(string, string, string))* is not supported in CTFE
traits.d(534,27):        called from here: findSplit(s, "(").asTuple._Tuple_super()
traits.d(536,45):        called from here: adjustIdent("std")
traits.d(524,29): Error: template instance std.traits.fqnSym!(std) error instantiating
traits.d(524,29):        instantiated from here: fqnSym!(traits)
traits.d(446,35):        instantiated from here: fqnSym!(fullyQualifiedName)
traits.d(452,19):        instantiated from here: fullyQualifiedName!(fullyQualifiedName)
traits.d(524,29): Error: template instance std.traits.fqnSym!(QualifiedNameTests) error instantiating
traits.d(707,31):        instantiated from here: fqnSym!(Inner)
traits.d(444,35):        instantiated from here: fqnType!(Inner, false, false, false, false)
traits.d(550,19):        instantiated from here: fqn!(Inner)
traits.d(446,35): Error: template instance std.traits.fqnSym!(func) error instantiating
traits.d(551,19):        instantiated from here: fqn!(func)
traits.d(707,31): Error: template instance std.traits.fqnSym!(Data!int, Data, int) error instantiating
traits.d(444,35):        instantiated from here: fqnType!(Data!int, false, false, false, false)
traits.d(552,19):        instantiated from here: fqn!(Data!int)
traits.d(446,35): Error: template instance std.traits.QualifiedNameTests.Data!int.Data.fqnSym!(x) error instantiating
traits.d(553,19):        instantiated from here: fqn!(x)
traits.d(446,35): Error: template instance std.traits.fqnSym!(tfunc, tfunc, int[]) error instantiating
traits.d(554,19):        instantiated from here: fqn!(tfunc)
../std/typecons.d(605,47): Error: reinterpreting cast from inout(string)* to inout(Tuple!(string, string, string))* is not supported in CTFE
traits.d(534,27):        called from here: findSplit(s, "(").asTuple._Tuple_super()
traits.d(536,45):        called from here: adjustIdent("object")
traits.d(524,29): Error: template instance std.traits.fqnSym!(object) error instantiating
traits.d(707,31):        instantiated from here: fqnSym!(Object)
traits.d(444,35):        instantiated from here: fqnType!(Object, false, false, false, false)
traits.d(510,33):        instantiated from here: fullyQualifiedName!(Object)
traits.d(518,48):        ... (1 instantiations, -v to show) ...
traits.d(446,35):        instantiated from here: fqnSym!(__T4InstTC6ObjectZ, Inst, Object)
traits.d(555,19):        instantiated from here: fqn!(__T4InstTC6ObjectZ)
traits.d(446,35): Error: template instance std.traits.fqnSym!(x) error instantiating

Comint exited abnormally with code 1 at Sat May 16 14:11:40

It seems CTFE-evaluation of findSplit fails with my patch. Is this fixable?

A documented unittest example that illustrates the benefits gained from this patch would be appreciated.

It seems CTFE-evaluation of findSplit fails with my patch. Is this fixable?

Tuple!(T, "foo") subtypes Tuple!T as an lvalue through a reinterpreting cast, which is not compatible with CTFE. It would be easy to support CTFE for subtyping Tuple!T as an rvalue, but it's not possible to conditionally AliasThis depending on CTFE.
It may be possible to support the subtyping as lvalue by redesigning the implementation of Tuple, such as making Tuple!T a field of Tuple!(T, "foo"). I might have a go at it.

Removing the names made make unittest pass on my machine. Two things remain:

Comment structs
Add documenting unittest.

Guidelines for these are welcome. I'm newbie here.

So it seems voldemort + docs - convenience = win.

Should I use
auto pre = longPreExpr;
auto sep = longSepExpr;
auto post = longPostExpr;
return FindSplit!(typeof(pre), typeof(sep), typeof(post))(pre, sep, post);

to reduce code size?

Should I change the names of the structs to Result for compactness now that they are scoped Voldemort types?

It seems a little redundant to define the exact same struct three times. Maybe use a mixin template or make the structs non-Voldemort types?

Yes, Result is fine. A pattern I've used is to call them Voldemort since the word has entered the D vernacular so it's evocative enough. static ensures the struct has no environment pointer. The duplication is minor so I'd go with it instead of factoring a type we then need to worry about.

Ping.

LGTM the duplication is ugly though.

Would any code rely on the return type of findSplit passing std.typecons.isTuple? That seems like it could be a potential point of breakage.

@MetaLang: Do you want me to check for backwards compatibility?

@MetaLang: Do you want me to check for backwards compatibility?

I'd like to see it too.

And please squash all commit into one

Unfortunately this breaks with compatibility of isTuple. We have asTuple though:
import std.typecons : isTuple;
static assert(isTuple!(typeof(r.asTuple)));
static assert(!isTuple!(typeof(r)));

@nordlow how about alias-this Tuple!(T) not just T ?

We currently have
Tuple!(S1, S1, S2) asTuple;
alias asTuple this;

Can you be more precise on what you mean by:
Tuple!(T) not just T
?

Can you be more precise on what you mean by:
Tuple!(T) not just T
?

Ehm I thought it wasn't. NVM

We may be looking into generalizing isTuple still

What about relaxing isTuple?

Should that generalization of isTuple happen before this is merged?

@nordlow there is no hurry since 2.069 is at least 2 months away but I'd much prefer to have a pull  hanging around by the time this pull is merged.

Rebased.

Ok. Tell me when.

Tell me when.

Pick any time from today to ~ October, 3rd  ;)

Ahh, you mean you want me to look into generalizing isTuple, then?
If so, ideas anyone, on how to accomplish this?

Ahh, you mean you want me to look into generalizing isTuple, then?

Yup, I have no plans to extend it for now.
By the look of it you should somehow to trigger implicit conversion:
template isTuple(T)
{
    static if (is(Unqual!T Unused : Tuple!Specs, Specs...))
    {
        enum isTuple = true;
    }
    else
    {
        enum isTuple = false;
    }
}

I'd try __compiles/is(typeof(...)) with a local function that accepts Tuples with the above check, something like:
is(typeof((T t){
 void f(Spec...)(Tuple!Spec tup){ }
  f(t);
 }))

}

I relaxed the definition of isTuple for backwards compatibility.
An alternative approach which I couldn't figure out the syntax for is
enum isTuple_(T) = isImplicitlyConvertible!(T, Tuple!Specs, Specs...);

Things should work now.

@andralex Ping! I believe this is ready now!

@nordlow Try this:
enum isTuple(T, Specs...) = isImplicitlyConvertible(T, Tuple!Specs);

It's pretty impressive that D can work backward from the instantiation of Tuple to infer Specs.

enum isTuple(T, Specs...) = isImplicitlyConvertible(T, Tuple!Specs);

fails as
typecons.d(1384,53): Error: cannot pass type int[2] as a function argument
typecons.d(1384,56): Error: cannot pass type Tuple!() as a function argument
typecons.d(411,49): Error: template instance std.typecons.isTuple!(int[2]) error instantiating
typecons.d(565,13):        instantiated from here: areBuildCompatibleTuples!(Tuple!(int, int), int[2])
typecons.d(1384,53): Error: cannot pass type Tuple!(int, int, int) as a function argument
typecons.d(1384,56): Error: cannot pass type Tuple!() as a function argument
typecons.d(411,49): Error: template instance std.typecons.isTuple!(Tuple!(int, int, int)) error instantiating
typecons.d(565,13):        instantiated from here: areBuildCompatibleTuples!(Tuple!(double, double, double), Tuple!(int, int, int))
typecons.d(579,20): Error: None of the overloads of '__ctor' are callable using argument types (Tuple!(int, int, int)), candidates are:
typecons.d(517,13):        std.typecons.Tuple!(double, double, double).Tuple.this(double, double, double)
typecons.d(1384,53): Error: cannot pass type Tuple!(double, string) as a function argument
typecons.d(1384,56): Error: cannot pass type Tuple!() as a function argument
typecons.d(396,55): Error: template instance std.typecons.isTuple!(Tuple!(double, string)) error instantiating
typecons.d(605,13):        instantiated from here: areCompatibleTuples!(Tuple!(int, string), Tuple!(double, string), "==")
typecons.d(612,13): Error: template instance std.typecons.Tuple!(int, string).areCompatibleTuples!(const(Tuple!(int, string)), Tuple!(double, string), "==") error instantiating
typecons.d(1384,53): Error: cannot pass type Tuple!(int, string) as a function argument
typecons.d(1384,56): Error: cannot pass type Tuple!() as a function argument
typecons.d(396,55): Error: template instance std.typecons.isTuple!(Tuple!(int, string)) error instantiating
typecons.d(605,13):        instantiated from here: areCompatibleTuples!(Tuple!(double, string), Tuple!(int, string), "==")
typecons.d(612,13): Error: template instance std.typecons.Tuple!(double, string).areCompatibleTuples!(const(Tuple!(double, string)), Tuple!(int, string), "==") error instantiating
typecons.d(624,20): Error: template std.typecons.Tuple!(double, string).Tuple.opEquals cannot deduce function from argument types !()(Tuple!(int, string)), candidates are:

Comint exited abnormally with code 1 at Mon Oct  5 22:55:57

Sorry, I missed the ! on isImplicitlyConvertible.

Sorry...
enum isTuple(T, Specs...) = isImplicitlyConvertible!(T, Tuple!Specs);

still fails
typecons.d(579,20): Error: None of the overloads of '__ctor' are callable using argument types (Tuple!(int, int, int)), candidates are:
typecons.d(517,13):        std.typecons.Tuple!(double, double, double).Tuple.this(double, double, double)
typecons.d(624,20): Error: template std.typecons.Tuple!(double, string).Tuple.opEquals cannot deduce function from argument types !()(Tuple!(int, string)), candidates are:
typecons.d(604,14):        std.typecons.Tuple!(double, string).Tuple.opEquals(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "=="))
typecons.d(611,14):        std.typecons.Tuple!(double, string).Tuple.opEquals(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "=="))
typecons.d(678,20): Error: template std.typecons.Tuple!(int, int, int).Tuple.opCmp cannot deduce function from argument types !()(Tuple!(int, int, int)), candidates are:
typecons.d(643,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(657,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(682,20): Error: template std.typecons.Tuple!(int, int, int).Tuple.opCmp cannot deduce function from argument types !()(Tuple!(int, int, int)), candidates are:
typecons.d(643,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(657,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(806,5): Error: template instance std.typecons.Tuple!(int, int) error instantiating
typecons.d(579,20): Error: None of the overloads of '__ctor' are callable using argument types (Tuple!(int, int, int)), candidates are:
typecons.d(517,13):        std.typecons.Tuple!(double, double, double).Tuple.this(double, double, double)
typecons.d(624,20): Error: template std.typecons.Tuple!(double, string).Tuple.opEquals cannot deduce function from argument types !()(Tuple!(int, string)), candidates are:
typecons.d(604,14):        std.typecons.Tuple!(double, string).Tuple.opEquals(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "=="))
typecons.d(611,14):        std.typecons.Tuple!(double, string).Tuple.opEquals(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "=="))
typecons.d(678,20): Error: template std.typecons.Tuple!(int, int, int).Tuple.opCmp cannot deduce function from argument types !()(Tuple!(int, int, int)), candidates are:
typecons.d(643,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(657,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(682,20): Error: template std.typecons.Tuple!(int, int, int).Tuple.opCmp cannot deduce function from argument types !()(Tuple!(int, int, int)), candidates are:
typecons.d(643,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(657,13):        std.typecons.Tuple!(int, int, int).Tuple.opCmp(R)(R rhs) if (areCompatibleTuples!(typeof(this), R, "<"))
typecons.d(489,23): Error: template instance std.typecons.Tuple!(int, string, float) error instantiating
typecons.d(440,28):        instantiated from here: Tuple!(int, "id", string, float)
typecons.d(806,5):        instantiated from here: Tuple!(int, int)
typecons.d(441,13): Error: static assert  (is(Fields.Types == TypeTuple!(int, string, float))) is false
typecons.d(1322,20):        instantiated from here: Tuple!(int, string, double)
typecons.d(468,28):        instantiated from here: tuple!(int, string, double)
typecons.d(806,5):        instantiated from here: Tuple!(int, int)

Comint exited abnormally with code 1 at Tue Oct  6 00:51:32

Ah well, you can't win 'em all. It must be because my custom isTuple is in a different module and thus doesn't conflict with std.typecons' version. I tried messing around with the std.typecons definition but no dice.
It's a bit strange that this doesn't work in the first place; it looks like std.typecons.isTuple actually does check for implicit conversions.
template isTuple(T)
if (is(Unqual!T Unused: Tuple!Specs, Specs...))
{
    //Check if Unqual!T implicitly converts to Tuple!Specs
    static if (is(Unqual!T Unused : Tuple!Specs, Specs...))
    {
        enum isTuple = true;
    }
    else
    {
        enum isTuple = false;
    }
}

Which is exactly what you are checking.

The plot thickens:
enum isTuple(T) = is(Unqual!T: Tuple!Specs, Specs...);

unittest
{
    struct TupleLike(T...)
    {
        Tuple!T payload;

        alias payload this;
    }

    static assert(is(TupleLike!(): Tuple!()));                                                    //pass
    static assert(is(TupleLike!int: Tuple!int));                                                  //pass
    static assert(is(TupleLike!(int, real, string): Tuple!(int, real, string)));                  //pass
    static assert(is(TupleLike!(int, "x", real, "y"): Tuple!(int, "x", real, "y")));              //pass
    static assert(is(TupleLike!(int, TupleLike!real, string): Tuple!(int, Tuple!real, string)));  //fail (due to the inner TupleLike)

    static assert(isTuple!(TupleLike!()));                              //fail
    static assert(isTuple!(TupleLike!(int)));                           //fail
    static assert(isTuple!(TupleLike!(int, real, string)));             //fail
    static assert(isTuple!(TupleLike!(int, "x", real, "y")));           //fail
    static assert(isTuple!(TupleLike!(int, TupleLike!real, string)));   //fail
}

Looks like alias this still needs some work.

So...is this ready, @MetaLang ?

Looks good to me.

Thanks for this work. It's right at the limit of being too clever for its own good, but I did notice that folks are unclear on what to test if the search succeeded.

Auto-merge toggled on

On the contrary, I like this pattern (aside from the need for alias this). It's a good-enough solution to D's lack of flow-based typing, which would be much more complicated to implement.

Could this reach 2.069?

@nordlow It's too late, the first release candidate for 2.069 is already out.

Ok. No worries :)

This broke https://issues.dlang.org/show_bug.cgi?id=15918

IMO, I think it's reasonable to not allow assignment between the two Voldermort types returned from findSplitAfter and findSplitBefore.


Disable a few more failing tests on Windows

https://github.com/swiftlang/swift/pull/77084
@swift-ci Please test


Footer section anchor tag hovering effect with color change.

#315 solved.
#315
Footer section hovering effect.
Type of PR

[ ] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
video -
https://github.com/subhadeeproy3902/BloxAI/assets/136844498/5f372ed9-f2e9-4c62-a6db-000372b188c6
Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
[Include any additional information or context that might be helpful for reviewers.]

@subhadeeproy3902
as per your suggestion I changed only that one effect.

@sakeel-103 Why dont u read Contributing.md its clearly mentioned to git pull

All these changes have been done ... i wont accept it ... cuz or else our convex database might become faulty

@subhadeeproy3902
I don't understand, will you please explain "cuz or else our convex database might become faulty" this line.
and please suggest what I should do cuz I'm very new to Next.js tech.

@subhadeeproy3902 do git pull... u have made changes in convex folder as well check the files u have changed in this PR. generate a new PR if u want to ithout changing other than the issue resolving !


deploy pulsar dev,test,pro cluster at the same time , i hope the pulsar-Operator  just like tidb-Operator

Is your feature request related to a problem? Please describe.
when i deploy pulsar cluster  for dev,test,pro env ,i donot have a  easy way to do it.
Describe the solution you'd like
i hope the pulsar-Operator  just like tidb-Operator futures.

@lookhua Currently if you are looking for solutions to deploy Pulsar in kubernetes, you can use Helm chart - https://github.com/apache/pulsar-helm-chart. We are also developing operators and will open-source them at some time.

Closed as stale and no one seems to work on it. Please create a new issue if it's still relevant to the maintained versions.


Double-clicking app doesn't behave like expected

the current release of prefill v1.9.0 adds detection to show warning if the app is launched from a double click
In my testing, this doesn't behave as expected - the app will open a terminal window for a brief moment and closes.
I have Windows 11 22H2 22621.963

Released in v1.9.1


Icon request: Hosting.de

I have read the document

URL of the site or app (required):
https://www.hosting.de/

Website Logo/ICON: https://pbs.twimg.com/profile_images/996377584731607040/XnYKvV7X_400x400.jpg

Pictures as file (Second picture too):
Hosting.de.zip

Its a Nextcloud Hosting Partner, so would be nice to have it as icon.

Here you go!
https://github.com/aegis-icons/aegis-icons/blob/master/SVG/hosting.de.svg
❓ How to add icon(s) to Aegis?


Serializing pointer segfaults

What is the issue you have?
Serializing a vector/map/.. of pointers segfaults

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

int main() {
    int i = 1;
    std::vector<int*> values{&i};
    nlohmann::json data = values;
}

What is the expected behavior?
Not sure - I guess compiler error ?

And what is the actual behavior instead?
Segfault - Stack overflow from recursive call to

nlohmann::detail::json_ref<nlohmann::basic_json<std::map, std::vector, std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> >, bool, long, unsigned long, double, std::allocator, nlohmann::adl_serializer> >::json_ref<int* const&>

Which compiler and operating system are you using? Is it a supported compiler?
g++ (Ubuntu 8.1.0-5ubuntu1~16.04) 8.1.0

Did you use a released version of the library or the version from the develop branch?
3.3.0

If you experience a compilation error: can you compile and run the unit tests?

Not sure what should be the expected behaviour. But I guess segfault is not the desired behaviour.

@SimonEbner could you please test against the develop branch or update to Version 3.4.0?
I get a compile time error in GCC and MSVC with Develop as well as 3.4.0
GCC says
 error: conversion from 'std::vector<int*>' to non-scalar type 'nlohmann::json' {aka 'nlohmann::basic_json<>'} requested
    nlohmann::json data = values;

While MSVC says
error C2440: 'initializing': cannot convert from 'std::vector<int *,std::allocator<_Ty>>' to 'nlohmann::basic_json<std::map,std::vector,std::string,bool,int64_t,uint64_t,double,std::allocator,nlohmann::adl_serializer>'
          with
          [
              _Ty=int *
          ]
 : note: No user-defined-conversion operator available that can perform this conversion, or the operator cannot be called

Version 3.4.0 fixed a lot of SFINAE-related issues, I strongly recommend upgrading.

Perfect, doesn't compile for me anymore either. Keep up the great work.


(PUP-6924) Improve serialization of parameterized collection types

In this commit:

The element_type attribute was removed from the Collection type and
instead added to the Array type and as value_type to the Hash type.
Serialization of registered Object types now uses the parameter names
and order of the #initialize method to determine in which order the
contained attributes are serialized.

CLA signed by all contributors.


BitBeand not sprouting

Wallet is unlocked, stake=1 in config, blockchain synced, balance confirmed, but
$ ./BitBeand getsproutinginfo
{
    "Enabled" : true,
    "Sprouting" : false,
    "Errors" : "",
    "Current Block Size" : 1000,
    "Current Block Tx" : 0,
    "Pooled Tx" : 0,
    "Difficulty" : 2.98058679,
    "Search Interval" : 1,
    "Weight" : 0,
    "Net Sprout Weight" : 237479203,
    "Expected Time" : -1
}

Add:
sprout=1
If you need further assistance join the community: https://slackkiller.bitbean.org
From: "Rocco Stanzione"<EMAIL_ADDRESS>To: "TeamBitBean/bitbean-core"<EMAIL_ADDRESS>Cc: "Subscribed"<EMAIL_ADDRESS>Sent: Thursday, September 28, 2017 7:34:29 PM
Subject: [TeamBitBean/bitbean-core] BitBeand not sprouting (#5)
Wallet is unlocked, stake=1 in config, blockchain synced, balance confirmed, but
$ ./BitBeand getsproutinginfo
{
"Enabled" : true,
"Sprouting" : false,
"Errors" : "",
"Current Block Size" : 1000,
"Current Block Tx" : 0,
"Pooled Tx" : 0,
"Difficulty" : 2.98058679,
"Search Interval" : 1,
"Weight" : 0,
"Net Sprout Weight" : 237479203,
"Expected Time" : -1
}
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub , or mute the thread .
--
Shawn Kent
Project Manager & Lead Developer
Bean Core, Team Bean, BitBean
"BitBean, Fast..Simple..Secure..More Than Just a Digital Currency!"
www.bitbean.org


libmediainfo: 18.08 -> 18.08.1

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/libmediainfo/versions.
meta.description for libmediainfo is: '"Shared library for mediainfo"'.

Checks done (click to expand)

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 18.08.1 with grep in /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1

directory tree listing: https://gist.github.com/e5dca3682052f8b03fe595a44089bb05

du listing: https://gist.github.com/b9752bf2566cf101064c9a424bfea8b9

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/e204e6c508d3d843f398ac14a4c80296046b045c.tar.gz -p libmediainfo
ls -la /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1

cc @devhell for testing.

Success on aarch64-linux (full log)
Attempted: libmediainfo
Partial log (click to expand)
removed '/nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
'libmediainfo.pc' -> '/nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1
shrinking /nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1/lib/libmediainfo.so.0.0.0
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1/lib
patching script interpreter paths in /nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1
checking for references to /build in /nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1...
/nix/store/wgcmz7ysqy1aim4cfaivmh319dcdq439-libmediainfo-18.08.1

Success on x86_64-darwin (full log)
Attempted: libmediainfo
Partial log (click to expand)
make[2]: Nothing to be done for 'install-data-am'.
make[2]: Leaving directory '/private/tmp/nix-build-libmediainfo-18.08.1.drv-0/MediaInfoLib/Project/GNU/Library/test'
make[1]: Leaving directory '/private/tmp/nix-build-libmediainfo-18.08.1.drv-0/MediaInfoLib/Project/GNU/Library/test'
removed '/nix/store/81dkwa904shgb62maa83zbxqyacyzrv0-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
'libmediainfo.pc' -> '/nix/store/81dkwa904shgb62maa83zbxqyacyzrv0-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
post-installation fixup
strip is /nix/store/df6k4mgdjxciy0f637lryp7c9ln7n1m3-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/81dkwa904shgb62maa83zbxqyacyzrv0-libmediainfo-18.08.1/lib
patching script interpreter paths in /nix/store/81dkwa904shgb62maa83zbxqyacyzrv0-libmediainfo-18.08.1
/nix/store/81dkwa904shgb62maa83zbxqyacyzrv0-libmediainfo-18.08.1

Success on x86_64-linux (full log)
Attempted: libmediainfo
Partial log (click to expand)
removed '/nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
'libmediainfo.pc' -> '/nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1/lib/pkgconfig/libmediainfo.pc'
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1
shrinking /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1/lib/libmediainfo.so.0.0.0
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1/lib
patching script interpreter paths in /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1
checking for references to /build in /nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1...
/nix/store/0k21jyikwlsclvjdk85ip93izpqazn3k-libmediainfo-18.08.1


Updated frontend: fixed search through Drawer panel.

Implemented DebuggerAgent.searchInContent.
Implemented PageAgent.searchInResource.
Added /lib/search.js to provide shared search functions.
Added resource/script source caching.
Fixed highlighting matches bug on minified-file lines.

@3y3 This is not an easy pick pr... I'll push some more commits with syntax and code cleanup for linter... but I know we'll talk about the source code caching in DebuggerAgent implementation... :D

@marcominetti , sorry for pending of your features, but there is a lot of bugs, which I need to fix and release before review.

@3y3, wow I've seen the new issues right now... may I help you? Feel free to assign me something. ;)

Hello, @marcominetti , I target you from Plugin system pr. Also you can look on 0.10.2 milestone, here are bugs and features with hight priority. It is not necessary, what all this bugs needs to be fixed, in some situations we need only to describe solution. For example #566 - you are friendly with new front-end, so you can target man to entry point for this feature (or implement it by yourself).
So look at list, and if you ready to fix something - notify me.

@3y3 I can work on #566 and #580...

Let me know if you have any priority... ;)

Please notify me when you will be ready for review this pr... cause I'm thinking about reworking it after changing the internal behaviour incase of --no-preload param...

I can work on #566 and #580...

Nice! For #580 look also on #628. I think it's a duplication. In #628 posted good example for tests. (For correct installation delete npm-shrinkwrap)

Please notify me when you will be ready for review this pr... cause I'm thinking about reworking it after changing the internal behaviour incase of --no-preload param...

--no-preload was changed somewhere?

--no-preload was changed somewhere?

Not yet, I will elaborate the clue later on reviewing/reworking this pr... btw the ni stack is built to run "locally", so the --no-preload does prevent backend to feed frontend with every available file from disk... (am I right?) I use it to have frontend load quickly and smoothly in large project...
The problem is that the --no-preload flag also avoids frontend feeding the static content provider (at front-end) for actually loaded files... This dramatically impact search, cause at first run, it needs to fetch source code for every loaded file... I will try to override the --no-preload behaviour assuring that script content is fetched automatically for loaded files and stored within the static content provider...

+1 for https://github.com/node-inspector/node-inspector/issues/628 !

Guys please work on this its a really important feature!
Thanks a million in advance :)

hi @a7madgamal @3y3  I've successfully implemented the feature but there is a huge memory consumption cause I need to cache/preempt source files in arrays because search feature generates very huge number of getScriptSourceById/getResourceContent requests to the debugger through DebuggerClient and everything crashes... I could cache source in /tmp/ filesystem (but it's not so clever in cloud environments for the I/O)...
Solutions:

fix debugger client or debugger backend crash on huge requests (best)
implement an ordered queue at getScriptSourceById and getResourceContent (@3y3?)
keep this implementation and/or move cache on disk (worst)

You can try implementation with ljve-inspector module available on npm (I use it actually).

I can't find it, could you provide the npm link?

https://www.npmjs.com/package/ljve-inspector

@marcominetti , I'm not familiar with search API in DevTools, but if I understand right main problem - we receive lot of parallel requests like searchInContent and we try to require sources for each scriptId also in parallel mode. So, we need to review strategy of original DevTools, when there exists a linked local folder:

If in this case DevTools tries to search in all files (not only in loaded to front-end), then we need to completely avoid getScriptSourceById and use fs (this makes some problems for generated scripts, but I prefer to ignore this problems in initial implementation)
Otherwise we can move search logic to app (look at injection example in ConsoleAgent)

In all this cases we need to completely avoid caching.
So, main question on current step - are we need to search in preloaded resources?

@marcominetti I installed your fork and its working greatly but when i search and click on a line it always highlight the NEXT line!
I want you to confirm this and if it turns out to be a bug I think I can help you fix it quickly
HINT:check the function getLine(doc, n) return chunk.lines[n]; in the "codemerror.js" file should be n-1 if not n !==0
@3y3 I think it's ok for the search process to be a little bit slower and works like 10 files at a time then next 10 etc. for a basic fix. After all it's better than nothing!

@a7madgamal , I agree that we must sacrifice performance for stability

@a7madgamal yep the bug of next line is fixed within an unpublished version... I'll publish it after some cleanup...

@3y3, I'm busy but I'll try to comment code just to share the pr/strategy... and I think the best way to improve stability and avoid memory leak is to directly check the filesystem if the file exists and, if not, trying to get through debuggerclient with requests "pagination"... do you agree?

@3y3 hi, i'll have to work on this next weekend... just to confirm, i'll do:

remove caching and

read script contents from fs
eventually pool by X when fs is not available/exist

include fix for wrong-next-line

@3y3 I've rebased against 0.11.1, reverted most of the code and switched to original functions extracted from frontend... It seems to work now. Performance are not exciting but no memory leaks. We can drop my custom implementation with regexp (commented code in SearchOverrides.js and _xxx "private" named functions in search.js). After review I'll fix the jshint stuff...

Hello @marcominetti , sorry for ignoring your previous comment - I'm very busy at this time. I'll review your changes as soon as possible. I'm not sure that this will happen today, but definitely on this week.

Waiting for this =) @3y3

Hi @grsabreu, in the weekend I'll check the status against latest commits to the master branch... are you experiencing specific problems?

@marcominetti , please don't miss time at current weekend. I'll release "Node Inspector Next" at this month (I hope) with new frontend and completely reimplemented server. So it's not reasonable to rebase at current master.
Node Inspector Next breaks compatibility with oldest node versions (compatibility starts with 4.*), but implements lot of new features.
I'm sorry that I pending this pr, but each time then I start to review it, there happens new broken node release.

@3y3 cool! Do you need help on nin?

any updates?


Resolved conflict of #282

Resolved conflict of #282

@leonardobazico thanks but PR #282 was re-synced to the latest.


Locust as library throwing Master-Server connection error in Distributed mode

Describe the bug

When I run the locust as a library with Master-worker mode, I am getting a worker-server connection error.
Expected behavior

The tests should run without any errors.
Actual behavior

Getting the below warning, and the tests are not running.
WARNING:locust.runners:You are running in distributed mode but have no worker servers connected. Please connect workers prior to swarming.
Steps to reproduce

Below is the code
from gen_recommender import RecommenderUser
from locust.env import Environment
import gevent
from locust.stats import stats_printer, stats_history
from locust.runners import STATE_RUNNING
import os

def create_workers():
    for _ in range(20):
        environ = Environment(user_classes=[RecommenderUser])
        environ.create_worker_runner('<IP_ADDRESS>', 5557)

def create_master():
    envi = Environment(user_classes=[RecommenderUser])
    envi.create_master_runner(master_bind_host='*', master_bind_port=5557)
    envi.create_web_ui(web_host, int(web_port))

    gevent.spawn(stats_printer(envi.stats))
    gevent.spawn(stats_history, envi.runner)
    gevent.spawn(checker, envi.runner)

    return envi

def run_tests(environ, users, spawn_rate):
    environ.runner.start(users, spawn_rate=spawn_rate)
    environ.runner.greenlet.join()
    environ.web_ui.stop()

my_env = create_master()
create_workers()
run_tests(my_env, 10000, 100)

Environment

OS: Windows
Python version: 3.7
Locust version: latest
Locust command line that you ran:
Locust file contents (anonymized if necessary):

What you’re trying to do doesnt make a lot of sense. Launching worker runners in the same process will not give better performance than running stand-alone, even if you could make it work. Launch workers as separate processes.

I need to implement this as a service on a platform that simply invokes the stress test once I start it. In this case, I tried to add Master-slave relationship here as my Spawn rate is about 500.
Now if I invoke this as a separate process, I need to deploy the services as equal to the number of workers I need. Is there any alternative to this?
And finally if there are no workarounds, do I need to invoke each worker as a separate process? Or can I at least invoke the Master as a separate process and scale the workers as another consolidated separate process?

Now if I invoke this as a separate process, I need to deploy the services as equal to the number of workers I need. Is there any alternative to this?

Idk what you mean by ”service” in this case? Are you talking about kubernetes?
Maybe locust-swarm can be of use? https://github.com/SvenskaSpel/locust-swarm

Great. Thank you. Could you also let me know if I can use this as a library?

There is no such built in functionality, but you may be able to modify it. Actually it makes some sense to do that.

Essentially you’d start by to modifying this line https://github.com/SvenskaSpel/locust-swarm/blob/6a3250470d102f6be4a0e2306211961ba5d3ae96/bin/swarm#L386 to start the master as-a-library instead.

Sure. I think it"s essential because we can use to write custom Plots w.r.t the obtained results from multiple user classes and so on. I will try to get some progress :)

Closing this as invalid. Feel free to open a new ticket or PR in locust-swarm to discuss making it optionally use as-a-library


Forge create gives deserialization error: expected value at line 1 column 1

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (b1f4684 2024-05-24T00:23:12.392859800Z)
What command(s) is the bug in?
forge create
Operating System
Windows
Describe the bug
Hi,
I have properly installed foundry on windows 10 using git bash by following the installation guide in Foundry Book. I am using Anvil for deploying my contract. Anvil runs without any issues on my terminal and spits out private keys and local rpc server which I have defined in my .env file as below in my vs code project:
Anvil
RPC_URL=http:<IP_ADDRESS>:8545
PRIVATE_KEY=0xac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80
Following is my Test contract:
// SPDX-License-Identifier: MIT
pragma solidity =0.8.19;
contract Test {
constructor() {}
}
I ran below command to deploy contract but get deserialization error as below:
$ forge create Test --rpc-url $RPC_URL --private-key $PRIVATE_KEY
[⠊] Compiling...
[⠰] Compiling 1 files with Solc 0.8.19
[⠔] Solc 0.8.19 finished in 2.48s
Compiler run successful!
Error:
deserialization error: expected value at line 1 column 1
Context:

expected value at line 1 column 1

What am I missing? Appreciate any help. Thank you.

error.log
Attached is the log. Thanks.


:default => false does not save to jsonb column

If you create an object whose jsonb_accessor attribute has a default value of false that attribute will not be saved in the jsonb column.
To reproduce:
class Product < ActiveRecord::Base
  jsonb_accessor :data,
    color: [:string, default: 'blue']
    on_sale: [:boolean, default: false]
end

product = Product.create
product.data
# => { color: 'blue' }

Expected output:
product.data
# => { color: 'blue', on_sale: false }

Released!


Getting Steps or Sleep data as [object], how to decode it ?

I have implemented a library, Authorized is true but when I try to get Sleep data or steps data I'm getting object array and no instructions how to decode it.
"granularity": [[Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object]],
[ LOG  Authorized {"success": true}
 LOG  [{"addedBy": "com.xiaomi.hm.health", "endDate": "2021-04-15T00:26:00.000Z", "granularity": [[Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object]], "startDate": "2021-04-14T18:51:00.000Z"}, {"addedBy": "com.xiaomi.hm.health", "endDate": "2021-04-16T02:10:00.000Z", "granularity": [[Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object], [Object]], "startDate": "2021-04-15T17:34:00.000Z"},]

No idea how to decode that object to get real values

Can you please log any of the index?

https://github.com/StasDoskalenko/react-native-google-fit/blob/fbcf2c70042150bdea86cced641a7e006eb37db5/index.android.d.ts#L307-L320
Check the typescript file

@aboveyunhai @enestatli  seems you didn't understood the question here.
Let me post everything so you guys can understand what is the problem or issue here.
First I have declared a scopes:

  const options = {
      scopes: [
        Scopes.FITNESS_ACTIVITY_READ,
        Scopes.FITNESS_ACTIVITY_WRITE,
        Scopes.FITNESS_BODY_READ,
        Scopes.FITNESS_BODY_WRITE,
        Scopes.FITNESS_SLEEP_READ,

      ],
    }

Then I have created a function to get sleep data

getSleepdata()
  {
    let opt = {
      startDate: "2021-04-01T00:00:17.971Z", // required
      endDate: new Date().toISOString(), // required

    };

    GoogleFit.getSleepSamples(opt).then((res)=> {
      console.log("sleepdata",res)
    });
  }

Now when I call this function I receive this as output log. You will see its not giving me data as required (date, steps etc). It simply showing be [object]

@patriksharma  This is the expected behavior if your log does not handle nested data structures. Object here refers to the type of content instead of the content itself. You need to iterate through the array to check each element by yourself.
It seems like you don't understand some JavaScript stuffs, and heavily rely on log() info. so I highly suggest you search your problem or ask it in StackOverflow. This question technically has nothing to do with this package.
To give you an idea:
https://stackoverflow.com/questions/10729276/how-can-i-get-the-full-object-in-node-jss-console-log-rather-than-object

@aboveyunhai Thanks a lot for sharing that link, It looks helpful. Yes I'm new to react native and worked only on Swift.
thanks for helping out here.

I think we both understand the issue but the point you missing is the logged array holds the all sleep data starting from startDate: "2021-04-01T00:00:17.971Z" until now. So you need to filter thorugh that array and check if data.source === 'com.whatyouneed_steps' and map to the filtered list to get the sleep,steps or whatever you need values.

console.log(JSON.stringify(myObject, null, 4));
yes worked with this


typo fix

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: \small typo
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[X] Successful preview build.
[ ] Code owner review.
[X] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6615b1acdb85303786598a1e--harness-developer.netlify.app


Hello! Does Awesome Theme support...

Hello!
Does this theme support VSCode installations?
It would be great if you could install this theme in VSCode! 👍😜

Apologies, but currently there is no version for VSCode. Themes for VSCode are primarily focused on color schemes. Unfortunately, the color scheme of this  theme is not good and is not well-suited for VSCode. There are no plans to create a version for it at this time.


Validate minimal thresholdfulfillment

will deprecate as weights will be omitted from the original spec, see https://github.com/interledger/rfcs/issues/87


Add production build

How do I make production build from this?

Thanks for your question! The Relay Starter Kit doesn't really have, as an aim, the goal of producing production-ready deployments.
We want to make sure to keep signal strong in the GitHub issue tracker – to make sure that it remains the best place to track issues that affect the development of the Relay Starter Kit.
Questions like yours deserve a purpose-built Q&A forum. Would you like to post this question to Stack Overflow with the tag #relayjs? The community will be happy to answer your questions about going to production with Relay, so long as you ask a specific question. Post a link to your Stack Overflow question here, to so that we don't lose track of it.
https://stackoverflow.com/questions/ask?tags=relayjs


Allow Pylance to detect docker volumes path by default

Currently, in order for Pylance to detect the python packages installed in a docker volume, one needs to go to Pylance settings and manually add the path that points to docker volumes installed on windows. in my case it is:
\wsl$\docker-desktop-data\version-pack-data\community\docker\volumes\backend_python_packages_data
Is it possible to make Pylance support this by default?

Thanks for the report, would you please elaborate on the environment you have set up?
Are you editing a project that is in the docker image? or is it on your local machine and you are trying to include a path that is mapped to a running container?
Where is the interpreter running?
More details will help triage the issue.
thanks

closing stale issue


Add rel="noopener noreferrer" to flyout navigation

1. Why is this change necessary?
Lighthouse report:
Links to cross-origin destinations are unsafe
Add `rel="noopener"` or `rel="noreferrer"` to any external links to improve performance and prevent security vulnerabilities.

Learn more
2. What does this change do, exactly?
It will add rel="noopener noreferrer to the anchors in the flyout navigation to ensure best practices.
3. Describe each step to reproduce the issue or behaviour.

Add a category that will be displayed in the flyout menu as an external link
Run Lighthouse with Best Practices checked

4. Please link to the relevant issues (if any).
5. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-19859
Please use this issue to track the state of your pull request.

@SimonNitzsche Thanks for your Contribution! 💙


Update imx docs

The new framework to build images is buildroot, so the documentation should be updated.

:link: zboto Link

closed via #27


Using MRR (Mean Reciprocal Rank) as a metrics on WikiQA datatset

First of all, Thanks for such a great repo. It's like a goldmine for people interested in deep text matching.
I'm interested in doing an experiment on WikiQA with MRR as an evaluation metrics. Can anyone guide me to illustrate how to use MRR for wikiQA dataset instead of MAP.
Though #306 provides this metric. I'm not able to integrate it with DRMM model.
Please help out. Thanks!

Hi @nipunsadvilkar , our MatchZoo 2.0 is still under development,  this page demonstrate the new API design, and it the MRR is implemented here.
We're still working to ship these features, it will be available in the coming month.

Closed due to inactivity.


onTokenRefresh not being called after getToken(entity, scope)

Android device: Emulator (Nexus 5X API 26 x86)
Android OS version: 8.0 (Google APIs)
Google Play Services version: 10.2.6
Firebase/Play Services SDK version: 10.2.6

I'm trying to reproduce a situation where onTokenRefresh would get called, such as deleting the instance ID (deleteInstanceId()) and then calling getToken().
I can get this to work perfectly with the regular getToken() (which returns the master token for the default Firebase project), but I'm having issues reproducing it if I use getToken(entity, "FCM") (which authorizes an entity to perform an action) instead of using getToken().
Here is the strange part: after deleting the instance ID and calling getToken(entity, "FCM"), if I take the app to the background, close the app, and open it again, then onTokenRefresh() is called on startup. It does not get called at all if I don't previously take the app to the background.
Steps to reproduce:

Generate a token with getToken(entity, "FCM")
Delete the instance ID with deleteInstanceId()
Regenerate a new token with getToken(entity, "FCM")
Notice onTokenRefresh() doesn't get called
Close the app
Open the app: onTokenRefresh() still doesn't get called.
Delete the instance ID with deleteInstanceId()
Regenerate a new token with getToken(entity, "FCM")
Take the app to the background
Close the app
Open the app
onTokenRefresh gets called

Observed Results:

onTokenRefresh doesn't get called reliably. Sometimes it isn't called at all.

Expected Results:

I expected to have onTokenRefresh get called as soon as a new token is generated, with both getToken() and getToken(entity, scope).

Relevant Code:
onTokenRefresh implementation:
public class InstanceIdService extends FirebaseInstanceIdService {

  ...

  public void onTokenRefresh() {
    ...
  }
}

Regenerating the entity authorization token:
new Thread(new Runnable() {
  @Override
  public void run() {
    try {
      FirebaseInstanceId.getInstance().deleteInstanceId();
      FirebaseInstanceId.getInstance().getToken(entity, "FCM");
    } catch (IOException e) {
      e.printStackTrace();
    }
  }
}).start();

To add a little more context: I was trying to do this because we are seeing an increasing number of failed messages, so we believe that we are failing to report tokens back to our server when they are refreshed on the clients.

Hi @fedetrim onTokenRefresh() is only called for the default application. If you are using additional senders (entities) then you need to call getToken(entity, scope) in order to get those tokens.


Allow configuring the encryption parameters

This allows configuring the connection's encryption.
Description
This just adds two extra flags to the connection string.
Motivation and Context
Some people like encryption.
How Has This Been Tested?
I ran the tests. I also wrote a new test that checks that the current connection has encryption.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Hello, @rzane! This is your first Pull Request that will be reviewed by Ebert, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

Coverage decreased (-59.3%) to 27.481% when pulling e751d4831a5e4dba42340fdcfc3afa9a641436d7 on rzane:encryption into f23bd7d5ac07ce220cab9a92517596b4de493188 on findmypast-oss:master.

hey @shdblowers, I'm pretty sure the CI failure isn't because if this PR.

Hi, thanks for the PR. Not quite sure what the problem is here, we were having some issues with our CI previously due to Microsoft keep swapping between versions 13 and 17 of the driver, but it is installing the expected version. Just seems to be installing in an unexpected location

@rzane rebasing/merge master on this branch should make it pass now :smile:

Okay, I just rebased. Would you be able to cut a release after this gets merged? It would really help me out.

1.1.0 is 🚀

👏


Allow specifying source URLs

Is your feature request related to a problem? Please describe.
The extension's UI currently allows the user to only specify regex destination URLs for temporary containers.  I would like to specify regex source URLs.
Describe the solution you'd like
Please allow the user to specify regex source and/or destination URLs for which to open temporary containers.
For example, let's say site example.com has a bunch of links.  Let's also say you want each of those links to open in its open unique temporary container (a unique temporary container for each link).  In this example, example.com is the source URL.
Describe alternatives you've considered
Using the Temporary Containers extension.
Additional context
Thank you!

https://github.com/igorlogius/open-in-temp-container/assets/67047467/777bec43-11cf-4a4e-9a5b-501a13629570


proposal: remove undefined concept from getting-started documentation

On page https://golang.org/doc/tutorial/getting-started
Change "Use the Go package discovery tool to find packages you can use in your own code. " to
"Use pkg.go.dev to find published packages you can use in your own code."
If that's not good enough I propose to define what a "package discovery tool" is, before using this concept in a getting-started document.

This doesn't have to be a proposal.  This can be decided by the documentation maintainers.


Manage multiple windows

Selected tab is no longer managed to avoid a too complicated workflow, see https://github.com/briocheio/localFront/pull/25


Various improvements

Various improvements, see individual commits.

I'll update the nuget package with these changes tomorrow. I need to automate that but I didn't think anyone would actually use this. :)

@Coinio I was just playing around and searched for a .net client. I honestly just picked the first of which I found the source. The package seems simple with no dependencies which I why I picked it.


in html i'm not at all getting coding option in it. kindly please solve the issue. thanl you

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

Closing as incomplete. Please feel free to add more information if you have any and we can re-open the issue.


Fix radio bug

Motivation and context
In the docs tab, the default version of the radio button was not loading.
Before
The default was not loading
After
Everything is loading and I added the ArgsTable for each version
How to test
You can go to the Docs table and check that everything loads correctly
Please have a look as well at the ArgsTable for Radio Button
Thank you

Hey, @Marine-Berthier thank you for the fix.
Keep in mind that if you're only changing the storybook and not actual package behaviour there's no need to trigger a release.


Index should accept CSV/JSON

I was thinking we could add a create (distinct from index) command to load JSON/CSV/FASTA files to initialize an existing database. But I think this could be included in the index command.
If the database does not exist, this creates a new database, otherwise adding stuff to an existing database (which seems to work out of the box). There is no reason either index couldn't accept any of these file types (differentiate by relying on the extension), and this is useful because of #23, and the fact that it is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index field, while other json/fasta files will not. Maybe we should just check for the "index" field and compute the index only if it is not there. In that case the "index" field would be a sort of reserved field that users shouldn't use.

Hi Everyone,
I just want to make sure you guys are comfortable with me moving this repo
over to the NCBI hackathon org. this afternoon.
I'll wait one more hour, since Ive seen a lot of recent commits.
Cheers!
BEn
On Thu, Aug 27, 2015 at 12:38 PM, Mike Panciera<EMAIL_ADDRESS>wrote:

I was thinking we could add a create (distinct from index) command to
load JSON/CSV/FASTA files to initialize an existing database. But I think
this could be included in the index command.
If the database does not exist, this creates a new database, otherwise
adding stuff to an existing database (which seems to work out of the box).
There is no reason either index couldn't accept any of these file types
(differentiate by relying on the extension), and this is useful because of
#23 https://github.com/DCGenomics/seqr/issues/23, and the fact that it
is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index
field, while other json/fasta files will not. Maybe we should just check
for the index field and compute the index only if it is not there. In
that case the index field would be a sort of reserved field that users
shouldn't use.
—
Reply to this email directly or view it on GitHub
https://github.com/DCGenomics/seqr/issues/26.

--
What have you done today to make the world a better place?

Ben,
What's the downside of the move?  Just that we have to check out from another repository?
Best,
Lewis
From: DCGenomics<EMAIL_ADDRESS>Sent: Thursday, August 27, 2015 2:50 PM
To: DCGenomics/seqr<EMAIL_ADDRESS>Subject: Re: [seqr] Index should accept CSV/JSON (#26)
Hi Everyone,
I just want to make sure you guys are comfortable with me moving this repo
over to the NCBI hackathon org. this afternoon.
I'll wait one more hour, since Ive seen a lot of recent commits.
Cheers!
BEn
On Thu, Aug 27, 2015 at 12:38 PM, Mike Panciera<EMAIL_ADDRESS>wrote:

I was thinking we could add a create (distinct from index) command to
load JSON/CSV/FASTA files to initialize an existing database. But I think
this could be included in the index command.
If the database does not exist, this creates a new database, otherwise
adding stuff to an existing database (which seems to work out of the box).
There is no reason either index couldn't accept any of these file types
(differentiate by relying on the extension), and this is useful because of
#23 https://github.com/DCGenomics/seqr/issues/23, and the fact that it
is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index
field, while other json/fasta files will not. Maybe we should just check
for the index field and compute the index only if it is not there. In
that case the index field would be a sort of reserved field that users
shouldn't use.
—
Reply to this email directly or view it on GitHub
https://github.com/DCGenomics/seqr/issues/26.

--
What have you done today to make the world a better place?
—
Reply to this email directly or view it on GitHubhttps://github.com/DCGenomics/seqr/issues/26#issuecomment-135520461.

I'd like to use my github account on Travis-CI for this project. We don't have that now; I'm not sure what permissions I will need in order for that to happen within an organization, but it works for the organizations I'm a part of (I'm admin in that organization though.)

Exactly.  The upside is that we have some expanded admin capabilities.
Cheers!
Ben
On Aug 27, 2015 3:02 PM, "lewisg-ncbi"<EMAIL_ADDRESS>wrote:

Ben,
What's the downside of the move? Just that we have to check out from
another repository?
Best,
Lewis
From: DCGenomics<EMAIL_ADDRESS>Sent: Thursday, August 27, 2015 2:50 PM
To: DCGenomics/seqr<EMAIL_ADDRESS>Subject: Re: [seqr] Index should accept CSV/JSON (#26)
Hi Everyone,
I just want to make sure you guys are comfortable with me moving this repo
over to the NCBI hackathon org. this afternoon.
I'll wait one more hour, since Ive seen a lot of recent commits.
Cheers!
BEn
On Thu, Aug 27, 2015 at 12:38 PM, Mike Panciera<EMAIL_ADDRESS>wrote:

I was thinking we could add a create (distinct from index) command to
load JSON/CSV/FASTA files to initialize an existing database. But I
think
this could be included in the index command.
If the database does not exist, this creates a new database, otherwise
adding stuff to an existing database (which seems to work out of the
box).
There is no reason either index couldn't accept any of these file types
(differentiate by relying on the extension), and this is useful because
of
#23 https://github.com/DCGenomics/seqr/issues/23, and the fact that
it
is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index
field, while other json/fasta files will not. Maybe we should just check
for the index field and compute the index only if it is not there. In
that case the index field would be a sort of reserved field that users
shouldn't use.
—
Reply to this email directly or view it on GitHub
https://github.com/DCGenomics/seqr/issues/26.

--
What have you done today to make the world a better place?
—
Reply to this email directly or view it on GitHub<
https://github.com/DCGenomics/seqr/issues/26#issuecomment-135520461>.
—
Reply to this email directly or view it on GitHub
https://github.com/DCGenomics/seqr/issues/26#issuecomment-135524538.

Sounds good to me, that is have an update and a create command.
Best,
Lewis
From: Mike Panciera<EMAIL_ADDRESS>Sent: Thursday, August 27, 2015 12:39 PM
To: DCGenomics/seqr<EMAIL_ADDRESS>Subject: [seqr] Index should accept CSV/JSON (#26)
I was thinking we could add a create (distinct from index) command to load JSON/CSV/FASTA files to initialize an existing database. But I think this could be included in the index command.
If the database does not exist, this creates a new database, otherwise adding stuff to an existing database (which seems to work out of the box). There is no reason either index couldn't accept any of these file types (differentiate by relying on the extension), and this is useful because of #23https://github.com/DCGenomics/seqr/issues/23, and the fact that it is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index field, while other json/fasta files will not. Maybe we should just check for the index field and compute the index only if it is not there. In that case the index field would be a sort of reserved field that users shouldn't use.
—
Reply to this email directly or view it on GitHubhttps://github.com/DCGenomics/seqr/issues/26.

Ive moved it over.  Let me know if it has the functionality you guys want.
If not, Matt or I will make the appropriate adjustments.
Cheers!
Ben
On Thu, Aug 27, 2015 at 5:33 PM, lewisg-ncbi<EMAIL_ADDRESS>wrote:

Sounds good to me, that is have an update and a create command.
Best,
Lewis
From: Mike Panciera<EMAIL_ADDRESS>Sent: Thursday, August 27, 2015 12:39 PM
To: DCGenomics/seqr<EMAIL_ADDRESS>Subject: [seqr] Index should accept CSV/JSON (#26)
I was thinking we could add a create (distinct from index) command to load
JSON/CSV/FASTA files to initialize an existing database. But I think this
could be included in the index command.
If the database does not exist, this creates a new database, otherwise
adding stuff to an existing database (which seems to work out of the box).
There is no reason either index couldn't accept any of these file types
(differentiate by relying on the extension), and this is useful because of
#23https://github.com/DCGenomics/seqr/issues/23, and the fact that it
is hard to store metadata in FASTA format.
Another factor is that the swissprot data already has a computed index
field, while other json/fasta files will not. Maybe we should just check
for the index field and compute the index only if it is not there. In that
case the index field would be a sort of reserved field that users shouldn't
use.
—
Reply to this email directly or view it on GitHub<
https://github.com/DCGenomics/seqr/issues/26>.
—
Reply to this email directly or view it on GitHub
https://github.com/DCGenomics/seqr/issues/26#issuecomment-135561614.

--
What have you done today to make the world a better place?

The Travis CI permissions required http://docs.travis-ci.com/user/github-oauth-scopes/ are tame, so I think everyone who's a current collaborator should be able to start using TravisCI. Admittedly, I've never used it.
That being said, I just made @lianyi , @lewisg-ncbi , & @averagehat 'admin' for the NCBI-Hackathon/seqr repo, so any of you have extra privileges. You can even add new collaborators!

@nyetsche I think I need to be a member of the organization hosting the repository in order to add it in travis.

I already added it in travis
On Fri, Aug 28, 2015 at 12:44 PM, Mike Panciera<EMAIL_ADDRESS>wrote:

@nyetsche https://github.com/nyetsche I think I need to be a member of
the organization hosting the repository in order to add it in travis.
—
Reply to this email directly or view it on GitHub
https://github.com/NCBI-Hackathons/seqr/issues/26#issuecomment-135829075
.

--
What have you done today to make the world a better place?

My mistake, apparently travis-ci urls are case sensitive. The builds are live here:
https://travis-ci.org/NCBI-Hackathons/seqr
Thanks!

:+1:

@lianyi When inserting documents, is it necessary to use the FindIndex class manually, or is Solr set up to automatically index them? I know the latter is true for searching, but is it true for indexing? Thanks.

The most recent update won't require FindIndex for indexing.  When the sequence provided in the "sequence" field, i.e: {sequence:"AAAAAAA",id:6,...}. It will be automatically tokenized as we used to do in FindIndex.

also probably we can add an option to allow the user to wipe clean all of the indexes before indexing new FASTA/JSONs.  i.e -clean
without this -clean option, it's basically an incremental update mode.


Add save/load plotter configuration

It's painful in the current tool to have to constantly re-setup plotting configurations between sessions. Yukon should allow users to save to a file and load from a file any such configuration.
The interchange format should use a human-readable, declarative format.

I urge you to ditch the mindset of separate GUI components and consider the
SCADA-themed approach I described on the forum.
On Tue, Apr 7, 2020, 21:07 Scott Dixon<EMAIL_ADDRESS>wrote:

It's painful in the current tool to have to constantly re-setup plotting
configurations between sessions. Yukon should allow users to save to a file
and load from a file any such configuration.
The interchange format should use a human-readable, declarative format.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UAVCAN/Yukon/issues/25, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZFIZCSCDO54GHV6RVGQM3RLNTWXANCNFSM4MDKKVOA
.


Null-cancelling Movement script addon

How will we bind to the appropriate keys for movement?

How will we bind to the appropriate keys for movement?

Default wasd are bound in the script, if you want to use custom wasd buttons you can bind them to the universally accepted +mfwd etc. from the null cancelling movement addon.

It seems the null movement script is empty. I feel like if it requires that user specific customization just to set up, it should be left till later, when we have a way to be keybind agnostic.

It seems the null movement script is empty. I feel like if it requires that user specific customization just to set up, it should be left till later, when we have a way to be keybind agnostic.

That's fair, I just assumed that the use case of having different wasd keys was so niche that you could assume they'd be willing to put in the legwork to make it work. If you're planning on having an app to manage keybinds it makes a lot of sense to wait until then. :)

Thank you for your contribution to mastercomfig!


[fix]: Ensure that Iroha crates compile with feature combinations

Description
Ensure that all feature combinations are explicitly checked for compile errors.
Making sure that implicit features are explicitly enabled in the respective Cargo.tomls.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

4 of 10 (40.0%)  changed or added relevant lines in 3 files are covered.
4193 unchanged lines in 82 files lost coverage.
Overall coverage decreased (-1.5%) to 57.939%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

p2p/src/peer.rs
0
1
0.0%

p2p/src/network.rs
0
5
0.0%

Files with Coverage Reduction
New Missed Lines
%

client_cli/src/main.rs
1
0%

cli/src/main.rs
1
0%

cli/src/torii/pagination.rs
1
98.9%

config/base/src/runtime_upgrades.rs
1
51.72%

config/src/torii.rs
1
96.67%

core/src/smartcontracts/isi/block.rs
1
87.5%

core/src/sumeragi/network_topology.rs
1
96.76%

crypto/src/merkle.rs
1
96.23%

data_model/src/account.rs
1
52.21%

data_model/src/domain.rs
1
47.37%

Totals

Change from base Build<PHONE_NUMBER>:
-1.5%

Covered Lines:
19793

Relevant Lines:
34162

💛  - Coveralls


Script for running GPflow tests in sequential and parallel modes.

Hello all,
This script runs tensorflow based tests in distinct processes. It prevents bad memory accumulations which can lead to crashes or slow runs on resource limited hardware.
In #471, I found that adding variables to default graph can cause significant slowness, even after calling graph resetting, tensorflow.reset_default_graph(). Previously, this issue was not present as it was mitigated by creating new graphs, instead of reusing default one.
The script has two modes parallel and sequential. In parallel mode it detects number of cpus and spawns nosetest processes against of each test files.
This script is used at #471. Parallel mode doesn't cut the test time execution significantly, only 1-2mins.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #472   +/-   ##
=======================================
  Coverage   99.83%   99.83%
=======================================
  Files          28       28
  Lines        2375     2375
=======================================
  Hits         2371     2371
  Misses          4        4

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 dc4b56b...1c28de5. Read the comment docs.

Specific reason to have a seperate script? Nose has by default a multiprocess plugin, it allows executation as follows:
nosetests --nologcapture --processes=4 --process-timeout=120
It requires a static directive in the test class to execute tests in parallel which I added in #437. The problem is we saw no actual performance improvement on travis. However locally it makes a big difference.

@javdrher, nosetests distributes load across processes=workers. Each worker runs all tests at same memory scope and thereafter tensorflow will do allocation within it, it caused #471 failing badly. The script runs each test suit (test file) as distinct nosetests process and alleviates memory issues of using default graph.

did you add the _multiprocess_can_split_ directives in the test classes? If not the multiprocessing plugin starts N processes but runs all tests in the same process. Nonetheless I agree it wouldn't solve your issues in case of a sequential build.

Yes. The problem with nosetests is that it distributes tests across N processes, it does make sense to have N being equal to the number of cpus, 2 cpus in case of travis. So, you have N=2 memory layouts. Even in that case tensorflow struggled with deallocation, so that process either got killed or hung out.

@javdrher, for example, you can check this run with $ nosetests --nologcapture --processes=4 --process-timeout=600, _multiprocess_can_split_ is turned on: https://travis-ci.org/awav/GPflow/builds/262027107,
result: _No output has been received in the last 10m0s, this potentially indicates a stalled build or something wrong with the build itself. _

@javdrher any thoughts/objections? :)

@awav this week I'm at a conference next week I'll have a closer look when I'm back. I find it surprising TF is having issues with GC causing extreme loss of performance so I'm definitely going to reproduce it.
We got rid of the multiprocessing plugin because of sudden extreme Travis slowdowns combined with the fact that nose only gives any feedback when all processes have finished for me with python 3.x builds (do you have any idea why it did in the build you posted?). Hence our builds got terminated because after 10 minutes nothing got reported: e.g. this build for instance.
From your Travis log that clearly isn't the issue here. However, it seems that most tests execute quickly (3 minutes) and the problem is caused by the notebook tests: those are not running in parallel in your build because they are launched from a single test function. In GPflowOpt we are splitting this up.

@awav to what extent are the memory issues with the default graph alleviated by using tf.reset_default_graph() ? We had a related problem early on in development which was largely fixed by calling the reset in the unit test constructor. If there is still and issue with the default graph I'd like to establish exactly what it is. If we can't fix it maybe we shouldn't be using the default graph.

@awav @markvdw @jameshensman I had into this issues surrounding this as discussed.
TensorFlow does indeed have its own unit test class that inherits from the class we are currently using:
https://www.tensorflow.org/versions/r1.3/api_docs/python/tf/test/TestCase
https://www.tensorflow.org/versions/r1.3/api_guides/python/test
This first linked page contains the following quote:

This method behaves different than session.Session: for performance reasons test_session will by default (if graph is None) reuse the same session across tests. This means you may want to either call the function reset_default_graph() before tests, or if creating an explicit new graph, pass it here (simply setting it with as_default() won't do it), which will trigger the creation of a new session.

This implies that they deliberately use the same graph across tests. I confess myself slightly confused as to why they want to do that but obviously it has significance for the current discussion.

@alexggmatthews, It does make more sense now! The graph keeps only interaction definition of variables and operations upon them. But session is a real culprit in this case, as it performs memory allocations and computation itself. If you look at https://github.com/tensorflow/tensorflow/blob/r1.3/tensorflow/python/framework/test_util.py#L275, you can see that session is properly closed in the end of the test case and GPflow tests/code doesn't do it at all. It means that GPflow abandons one session and creates another one without release resources, this is kind of a leak. Here is a quote from session documentation:

A session may own resources, such as variables, queues, and readers. It is important to release these resources when they are no longer required. To do this, either invoke the close() method on the session, or use the session as a context manager.

Good detective work. Do you think the right fix is therefore to move over to the TensorFlow unit test class?

@alexggmatthews, Yes, it is worth to try. I'm experimenting in parallel branch 'artemav/session-graph-assign-experiments'. I haven't finished it yet though. I will publish results as soon as possible.

@alexggmatthews, @jameshensman Okay, I did it! https://travis-ci.org/GPflow/GPflow/builds/265325416.
The 'artemav/session-graph-assign-experiments' alters tests using proper session closing via tensorflow.test.TestCase. It helped avoid test crashes in #471. This PR with ./run_script.sh script was created only because of these crashes.
Recap:

Once created session always must be closed to release resources. It can be done via using context manager or explicitly closing session. tensorflow provides nice wrapper for TestCase and test_session() function which turns default session on and closes it in the end of each test.
'artemav/session-graph-assign-experiments' changes only make sense in context of #471, as it has capability to use default graph and default session bypassing tedious hacks.
current tests are flawed and 'broken'

I'm about to merge 'artemav/session-graph-assign-experiments' into #471. @jameshensman and @alexggmatthews could you review this code once more time? :)

fixed by #471


Site-wide Frontend Complete

By "site-wide frontend" we mean the features of the site which will broadly appear on every single page. Broadly speaking this will be:

The navbar
A site-wide footer
The general organisation of each page (columns, rows, etc.)

This should be finished by the first beta release.

Finished for now!


Additional Info in US Stories missing

Additional Info in US Stories missing.
Example: FF (I) 1 (I) and FF (I) 1 (I) (1. Hälfte) are showing the same name.

done.


Add MAS approval step to the release script

Summary
I'm adding an extra step to the release script in which we will generate a "final" release that goes to MAS for approval. If approved, it already has the version number without the rc or develop post-fix and can be directly releases to MAS. If not, it is tagged as vx.x.x-mas.x instead of vx.x.x and we can submit fixes and generate new tags until it is approved. Once approved, we can tag it appropriately.
NONE

The files scripts/release.sh, scripts/release.sh are in the blocklist for external contributors. Hence, these changes are not tested by the CI pipeline active until the build is re-triggered by a core committer or the PR is merged. Please be careful when reviewing it.
/cc @mattermost/core-security @mattermost/core-build-engineers


Remove expired collaborator from Terraform file/s for johngriffin

Hi there
This is the GitHub-Collaborator repository bot.
The collaborator johngriffin review date has expired for the file/s contained in this pull request.
Either approve this pull request, modify it or delete it if it is no longer necessary.

Replaced by PR that removes the module.


cannot install streamlit via conda due to package conflict

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:
hello,
i've problem in installing streamlit v0.86.0 via conda, the solving environment took long time, and failed, after that error found conflict
but the package report is confusing
i used :
conda install -c conda-forge streamlit=0.86.0
after that, i tried :
conda install -c conda-forge streamlit
after that it can install streamlit, but the version is v0.81.1
could anyone help to locate the conflict?
i've just re-installed the miniconda, i deleted all packages, and after that installed spyder, pandas, plotly, altair only, before trying to install streamlit
package_list.txt

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

i added update to this case, i tried to install via pip
pip3 install streamlit
it can install streamlit v0.86.0, i've tested it, and seems working fine
i think the issue could be on conda

Thank you for this issue. Going to close because it seems like things work and this is on a much older version of Streamlit.


Added PHP book in bengali

What does this PR do?
Added a PHP programming textbook in Bengali by Mizanur Rahman.
Description
A Bengali book about PHP programming that could be used to help people become outstanding coders. A user-friendly, free ebook that can be downloaded in the form of a PDF.
Why is this valuable (or not)?
It is valuable because there is no Bengali textbook for PHP programming. The book can be beneficial for beginners who want to learn PHP in Bengali.
How do we know it's really free?
You can click on the provided link and PFA the pdf version of the book downloaded from the same website.
For book lists, is it a book? For course lists, is it a course? etc.
It is a book that can be downloaded in the form of a PDF.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

it's a site linking to another site decorating with hundreds of ads


Contradictory Statements?

The statement in the first paragraph:
All databases on a server are created within the same region as the logical server.
Seems to be contradicted by the second paragraph:
Specifically, the SQL Database service makes no guarantees regarding location of the databases in relation to their logical servers ...
Am I missing something here? What is the situation? Can we pin databases down to the same region as the logical server or not?
Thanks

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

ID: 65d101a0-4111-7c51-119b-3e36461cd732
Version Independent ID: c07cd714-2d23-1ebb-f4eb-b3765ebc0f23
Content: Azure SQL logical servers
Content Source: articles/sql-database/sql-database-logical-servers.md
Service: sql-database
GitHub Login: @CarlRabeler
Microsoft Alias: carlrab

@vmlambe what this is trying to say is that, within a region, there is no guarantee that an Azure SQL database instance will be physically located on the same underlying hardware as the logical server that manages it. Additionally, two databases associated with the same logical server within a region may not be located on the same physical hardware.

@vmlambe  We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


bug fix: remove double-counting of heat corr (StieglitzSnow.F90)

In the rare circumstance when more snow would need to be sublimated than is available in the top layer, the residual energy accounting term was added twice, once here and again ~30 lines further down.
This appears to be a bug.  It was discovered during the review of the land energy budget for M21C.
Fixing this bug is (probably) a non-0-diff change for the restarts.  It should be non-0-diff for at least some model diagnostics.
Further examination is on-going.
cc: @rdkoster

The change is 0-diff for restarts and diagnostics in the full suite of GEOSldas nightly tests run by @gmao-rreichle on 17 Jun 2024.
This means that the condition indeed happens very rarely, if ever.
The next step is to run the GEOSgcm tests once @biljanaorescanin  is back in the office.
cc: @elakkraoui @rdkoster @lcandre2

I am getting zero diff for all GCM testing. 1 day AMIP, REPLAY and INC REPLAY are all zero diff branch vs stock ( develop GEOSgcm_GridComp). The regression test is a success as well.


[BUG] My Dashy Docker container keeps looping and not starting

Environment
Self-Hosted (Docker)
System
Debian Latest with Portainer 20.10.24 I think
Version
V-3.1.1
Describe the problem
When I start the container it never actually starts, I will attach the logs, I think it keeps looping after building, but i'm not sure lol, it worked before, but I had my storage space full for a few months without realizing it, so I think that I was on an old version and now when it finally updated the looping happens, I tried to delete the entire container and remake it with my compose file, but it still happens, sorry for the bad explanation lol
Additional info
_Dashy_logs.txt
Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

Hi
From your log I can read, that your port 8080 is already in use ->
Unable to start Dashy's Node server
 Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080

Please share your docker compose file, if the issues persists after changing the port on the host side, but not on the docker side, (like 8090;8080)
Also have you tried re-pulling the latest image and rebuilding? (Make sure the conf.yml is mapped as bind volume beforehand.)

Hi From your log I can read, that your port 8080 is already in use ->
Unable to start Dashy's Node server
 Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080

Please share your docker compose file, if the issues persists after changing the port on the host side, but not on the docker side, (like 8090;8080)
Also have you tried re-pulling the latest image and rebuilding? (Make sure the conf.yml is mapped as bind volume beforehand.)

Thank you for the reply :D
I do have Qbittorrent on 8080, but I thought I had remapped Dashy correctly to 2121.... I guess not lol, not sure how to fix it as I'm using Gluetun, the weirdest thing if that it worked before I updated... here's my config: (I guess I maybe don't need Dashy on a VPN anyway? I'm pretty bad at this lol sorry)
`version: "3"
services:
gluetun:
image: qmcgaw/gluetun
container_name: gluetun
cap_add:
- NET_ADMIN
volumes:
- /home/cayuga/configies/glueman:/gluetun
environment:
- VPN_SERVICE_PROVIDER=private internet access
- OPENVPN_USER=CENSORED
- OPENVPN_PASSWORD=CENSORED
- SERVER_REGIONS=SE Stockholm
- VPN_PORT_FORWARDING=on
- PORT_FORWARD_ONLY=true
ports:
- 8080:8080
- 6881:6881
- 6881:6881/udp
- 8989:8989
- 7878:7878
- 9696:9696
- 6767:6767
- 9000:9000
- 2121:80
restart: always
qbittorrent:
image: lscr.io/linuxserver/qbittorrent
container_name: qbittorrent
network_mode: "service:gluetun"
environment:
- PUID=0
- PGID=0
- TZ=Etc/UTC
- WEBUI_PORT=8080
volumes:
- /home/cayuga/configies/qbittorrents:/config
- /home/cayuga/teleporter/Downloads:/downloads
restart: always
healthcheck:
test: "curl -sf https://example.com  || exit 1"
interval: 1m
timeout: 10s
retries: 1
sonarr:
image: lscr.io/linuxserver/sonarr:develop
container_name: sonarr
network_mode: "service:gluetun"
environment:
- PUID=0
- PGID=0
- TZ=Etc/UTC
volumes:
- /home/cayuga/configies/sonarr:/config
- /home/cayuga/teleporter/Shows:/tv #optional
- /home/cayuga/teleporter/Downloads:/downloads #optional
restart: always
healthcheck:
test: "curl -sf https://example.com  || exit 1"
interval: 1m
timeout: 10s
retries: 1
radarr:
image: lscr.io/linuxserver/radarr:develop
container_name: radarr
network_mode: "service:gluetun"
environment:
- PUID=0
- PGID=0
- TZ=Etc/UTC
volumes:
- /home/cayuga/configies/radarr:/config
- /home/cayuga/teleporter/Movies:/movies #optional
- /home/cayuga/teleporter/Downloads:/downloads #optional
restart: always
healthcheck:
test: "curl -sf https://example.com  || exit 1"
interval: 1m
timeout: 10s
retries: 1
prowlarr:
image: lscr.io/linuxserver/prowlarr:latest
container_name: prowlarr
network_mode: "service:gluetun"
environment:
- PUID=0
- PGID=0
- TZ=Etc/UTC
volumes:
- /home/cayuga/configies/prowlarr:/config
restart: always
healthcheck:
test: "curl -sf https://example.com  || exit 1"
interval: 1m
timeout: 10s
retries: 1
bazarr:
image: lscr.io/linuxserver/bazarr:latest
container_name: bazarr
network_mode: "service:gluetun"
environment:
- PUID=0
- PGID=0
- TZ=Etc/UTC
volumes:
- /home/cayuga/configies/bazaarr:/config
- /home/cayuga/teleporter/Movies:/movies #optional
- /home/cayuga/teleporter/Shows:/tv #optional
restart: always
healthcheck:
test: "curl -sf https://example.com  || exit 1"
interval: 1m
timeout: 10s
retries: 1
dashy:
container_name: Dashy
network_mode: "service:gluetun"
image: lissy93/dashy

# Pass in your config file below, by specifying the path on your host machine
volumes:
  - /home/cayuga/teleporter/dashy_conf.yml:/app/public/conf.yml
  - /home/cayuga/teleporter/icons:/app/public/item-icons

# Set port that web service will be served on. Keep container port as 80

# Set any environmental variables
environment:
  - NODE_ENV=production
# Specify your user ID and group ID. You can find this by running `id -u` and `id -g`
  - UID=0
  - GID=0
# Specify restart policy
restart: always

whisperasr:
image: onerahmet/openai-whisper-asr-webservice:latest
container_name: whisper
network_mode: "service:gluetun"
environment:
- ASR_MODEL=tiny
- ASR_ENGINE=faster_whisper
restart: always
watchtower:
image: containrrr/watchtower
container_name: watchtower
volumes:
- /var/run/docker.sock:/var/run/docker.sock
restart: always
autoheal:
image: willfarrell/autoheal
container_name: autoheal
volumes:
- '/var/run/docker.sock:/var/run/docker.sock'
environment:
- AUTOHEAL_INTERVAL=10
- CURL_TIMEOUT=30
restart: always`

ports:
- "4000:8080"  # Mapping any port to port 8080

THANK YOU SO MUCH it worked :D
I am still super new to Docker, should I separate all of the containers into their own stacks? I thought the best way to Docker was to just have everything in a single file lol

Awesome! 🎉
To be honest if you are new to Docker, I'd rather suggest something like portainer, but if you feel comfortable with docker compose, then it's definitely a good choice, as it gives much more freedom.
Generally speaking docker containers are used to isolate specific aplications.
So generally most people put things like frontend, a database and possibly a backend into the same stack, and then just open up the frontend, this way the backend stays protected and the frontend is accessible.
So it's not really a good idea to put all into one single docker compose, as if you need to change something, all services will go down, and that file will get messy once more services are added.
But those are only some ideas/suggestions, feel free to do what you like :)

Thank you again :D
I tried moving it back to using Gluetun so my vpn widget would work, but I keep getting this "Unauthorized" screen :/ maybe I did it wrong though lol, heres what the log says
`Unable to start Dashy's Node server
Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080
at Server.setupListenHandle [as _listen2] (node:net:1872:16)

at listenInCluster (node:net:1920:12)

at doListen (node:net:2075:7)

at process.processTicksAndRejections (node:internal/process/task_queues:83:21) {

code: 'EADDRINUSE',
errno: -98,
syscall: 'listen',
address: '<IP_ADDRESS>',
`

Hi
According to your log there is still an issue with the port ->
Error: listen EADDRINUSE: address already in use <IP_ADDRESS>:8080

I really think there's still a problem between your vpn or dashy occupying the port.
For a vpn I could strongly recommend things like tailscale or twingate, if you're new, you basically install the client on your phone/laptop and on the server , auth with your account and they both get an internal ip where you can reach them.
This will then automatically enable the full functionality as if you'd be right at home, without messing with that docker compose that much.

Thank you :) I think it's because qbittorrent and Dashy both have 8080 now... If there's no way to change one of the ports I may just have to skip the VPN widget I have in Dashy, since if I bind both to Gluetun they clash :/
Thank you for the Tailscale suggestion, but I only use Gluetun with Private Internet Access, I only access my server at home anyway :D

Oh awesome 😁