[n to m] range attempt

I'm still learning, so I expect some criticism. The test explains it best:
        statement: "$.foo.bar[1 to 3].location",
        expected: {
          lhs: [
            {
              property: "foo"
            },
            {
              array: "bar",
              element: {
                start: 1,
                end: 3,
              }
            },
            {
              property: "location"
            },
          ]
        }
      },

I suppose 'element' could be changed to 'range'. Carefully review what I did, in any case.
Parsing [last] and [last - 1] would be similar:
element: {
   last: true,
   offset: <ArithmeticExpression>
}

Offset is optional and I assume offset is always negative relative to last (there are no sensible last + 1 or last * 2 operations; though I suppose last/2 is remotely plausible).

A good start, indeed. I did some testing in Postgres and found these statements produce results:
arr[1 to last]
arr[1, 3 to 5]
arr[1, 3 to 0.5 * last]
Perhaps element should be elements so it can hold an array of element declarations?

Have a look in the spec (it's at the root of this repo, called SQL-2016_JSON_spec.pdf) section 6.10.3, page 73. It has a BNF of the element accessor syntax. Might give you some good ideas.

Okay, so it's an arithmetic expression with a fixed variable name.

Or should say: it's an array of arithmetic expressions.

So both start and end are either integers or arithmetic expressions. Also arrays are 1-based so 0 is not a valid index.

Right. So I think there are three elements that go in that array of
indices: wildcard, arithmetic expression (including single digits and last),
and ranges. But I haven't read the spec today (recovering from booster).
On Thu, Jan 20, 2022, 9:00 AM Matt Bishop @.***> wrote:

So both start and end are either integers or arithmetic expressions. Also
arrays are 1-based so 0 is not a valid index.
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1017717414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3NMFJY6EWEDQJO2Z7TUXA5SHANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Booster! I had mine last weekend. Took a couple days to get right again.
—
Matt Bishop

On Jan 20, 2022, at 12:22 PM, Jeff Lowery @.***> wrote:

Right. So I think there are three elements that go in that array of
indices: wildcard, arithmetic expression (including single digits and last),
and ranges. But I haven't read the spec today (recovering from booster).
On Thu, Jan 20, 2022, 9:00 AM Matt Bishop @.***> wrote:

So both start and end are either integers or arithmetic expressions. Also
arrays are 1-based so 0 is not a valid index.
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1017717414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3NMFJY6EWEDQJO2Z7TUXA5SHANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Arrays appear to be 0-based according to the spec. I see 0's all over the
place.
Also, this:

is the “meat” of an SQL/JSON path expression (“wff” stands for
“well-formed formula”).*

but nowhere do they detail what a wff is. Is it an arithmetic expression
(including last, which would be a variable)?
You gave me some examples using Postgres which I cannot find now (typical
for me when using Google Mail) where you had more elaborate mathematical
expressions. Some of those seemed insane.
I added a comment on the issue to better elaborate the requirement.
On Thu, Jan 20, 2022 at 9:00 AM Matt Bishop @.***>
wrote:

So both start and end are either integers or arithmetic expressions. Also
arrays are 1-based so 0 is not a valid index.
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1017717414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3NMFJY6EWEDQJO2Z7TUXA5SHANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Right. I am thinking about Postgres whose arrays are 1-based. Good catch!
—
Matt Bishop

On Jan 21, 2022, at 3:09 PM, Jeff Lowery @.***> wrote:

Arrays appear to be 0-based according to the spec. I see 0's all over the
place.
Also, this:

is the “meat” of an SQL/JSON path expression (“wff” stands for
“well-formed formula”).*

but nowhere do they detail what a wff is. Is it an arithmetic expression
(including last, which would be a variable)?
You gave me some examples using Postgres which I cannot find now (typical
for me when using Google Mail) where you had more elaborate mathematical
expressions. Some of those seemed insane.
I added a comment on the issue to better elaborate the requirement.
On Thu, Jan 20, 2022 at 9:00 AM Matt Bishop @.***>
wrote:

So both start and end are either integers or arithmetic expressions. Also
arrays are 1-based so 0 is not a valid index.
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1017717414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3NMFJY6EWEDQJO2Z7TUXA5SHANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

For examples check the pull request comments. They should be there?
—
Matt Bishop

On Jan 21, 2022, at 3:09 PM, Jeff Lowery @.***> wrote:

Arrays appear to be 0-based according to the spec. I see 0's all over the
place.
Also, this:

is the “meat” of an SQL/JSON path expression (“wff” stands for
“well-formed formula”).*

but nowhere do they detail what a wff is. Is it an arithmetic expression
(including last, which would be a variable)?
You gave me some examples using Postgres which I cannot find now (typical
for me when using Google Mail) where you had more elaborate mathematical
expressions. Some of those seemed insane.
I added a comment on the issue to better elaborate the requirement.
On Thu, Jan 20, 2022 at 9:00 AM Matt Bishop @.***>
wrote:

So both start and end are either integers or arithmetic expressions. Also
arrays are 1-based so 0 is not a valid index.
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1017717414,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3NMFJY6EWEDQJO2Z7TUXA5SHANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.

Okay, I see now that the emails I'm getting about this PR are synced with the comments here. They're easier to follow here in the PR than by Google Mail, so I'll be going here more often.
The simplest explanation of a WFF ("woof") I found:

The image might be too small for you to read, but it's probably not strictly correct anyway.
WFFs are language-dependent. What is the language in this case? Dunno. I don't think it would allow for irrational or imaginary numbers. It may or may not allow power notation. It could allow variables other than last. Maybe negative numbers are allowed (~WFF).
I think the last example you gave, though absolutely insane IMO, is what to shoot for:
arr[1, 3 to (0.5 * last) + 1]
So the parser should consider the following as WFFs:

digits
last
connected WFFs, using +, -, *, /, to as connectors
groups such as (0.5 * last) which contain a WFF

And I that would probably cover just about anything anyone would want to do, plus some.

Okay, so I think I've got the following working (to my satisfaction):

multiple accessors: [1, 3, 5]
last
simple formulas: [1 + 2, last - 1]
grouped formulas: [1 + (last - 1)]

In this approach, element is an array with at least one lhs member. So, foo[1] parsed to something like `{element:[{lhs:1}]. That may seem more complicated, but it is consistent, so the consumer of the CST doesn't need to handle single accessors differently from multiple accessors.
STILL BROKED:
I don't handle the multiple connector case, e.g. foo[1+2+3]

Looks pretty nice! Why do you think it won't handle arr[1 + 2 + 3]?

WFF = WFF connector WFF;  1 + 2 + 3 is WFF connector WFF connector WFF
Not exactly sure how to deal with this:

convert to an array of WFF's and connectors (if that is doable)
recursively deal with WFFs:  WFF connector WFF connector WFF = WFF' connector WFF = WFF''

@JeffML any other changes or are you ready for me to merge this?

I took a look at Chevrolet trains calculator program and I think I figured
out how to do the chain of operations inspirations that I said could be
done so you can hold off on that if you want.
On Thu, Jan 27, 2022, 6:15 AM Matt Bishop @.***> wrote:

@JeffML https://github.com/JeffML any other changes or are you ready
for me to merge this?
—
Reply to this email directly, view it on GitHub
https://github.com/mattbishop/sql-jsonpath-js/pull/13#issuecomment-1023254111,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2VY3PT27UFDHGWNUULC5TUYFHPDANCNFSM5MLIDGAA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Cool, I'll wait then.

rhs is now an array of ops with connector and rhs members.

I'm done now.

Thanks Jeff!


ARTEMIS-4796 simplify SimpleString API

This commit does the following:

deprecate the verbosely named toSimpleString static factory methods
add of static factory methods for all the ctors
replace any uses of the normal ctors with the of counterparts

This makes the code more concise and readable.

Bigger 'SimpleString API simplication': deletion? :)

I wouldn't mind seeing SimpleString gone. Change the packets in Core to send char[] ... and be gone!
But that would require a lot of testing.. especially around the pools on reading?

I don't really like the name "of" but that's just my personal taste probably... (of kind of kicks my OCD.. being a preposition, I feel like of what? :)
but as I said.. that's my personal preference only...
but I don't want to get in the way.. I will get used to it :) and I'm +1 to merge it..  and you should probably merge it soon before you have to rebase it.
+1 from me.

I don't really like the name "of" but that's just my personal taste probably... (of kind of kicks my OCD.. being a preposition, I feel like of what? :)
but as I said.. that's my personal preference only...
but I don't want to get in the way.. I will get used to it :) and I'm +1 to merge it.. and you should probably merge it soon before you have to rebase it.
+1 from me.

I presume it comes from the trend of Map.of(..) etc factory helpers. I dont mind it, though I personally wouldnt have changed it here given I'd rather SimpleString vanish, along with all the realted conversions and the byte doubling etc etc that come with it.

I don't really like the name "of" but that's just my personal taste probably... (of kind of kicks my OCD.. being a preposition, I feel like of what? :)
but as I said.. that's my personal preference only...
but I don't want to get in the way.. I will get used to it :) and I'm +1 to merge it.. and you should probably merge it soon before you have to rebase it.
+1 from me.

I presume it comes from the trend of Map.of(..) etc factory helpers. I dont mind it, though I personally wouldnt have changed it here given I'd rather SimpleString vanish, along with all the realted conversions and the byte doubling etc etc that come with it.

Agreed with Robbie on the name, it matches some Collections APIs etc and isn't hard to grok plus its just less verbose.  But I'd also like to see an effort towards removing the SimpleString to simplify working with the broker internals.

As far as I'm aware, the "of" naming convention was introduced in 2008 in the second edition of Joshua Block's Effective Java. He identified it as a way to draw attention to static factories and adhere to common naming conventions. It is a more concise version of valueOf which is also used prolifically throughout the Java API. It's an industry standard at this point.

That said, if we can remove SimpleString completely I'm good with that too. Of course, this PR is not mutually exclusive with that objective. Consider it a stop-gap on the way.

We can merge it I think.

The change should probably be rebased to current main, which has already added new use of the now-deprecated bits that this misses switching over.
From doing a grep, there also looks to remain a number of other existing uses of toSimpleString that perhaps should be updated also.
Feels like we should be unit testing any otherwise-unused deprecated bits, but it doesnt appear like either of the two SimpleStringTest classes do so.

I rebased and changed as many of the remaining references to deprecated methods that I could. There are a handful of references to deprecated methods in the compatibility tests, but those must remain due to the fact that those tests are actually using older versions that don't have the new methods.

lets merge and review? otherwise you will keep rebasing forever

Same general review comment as last time. There are still several references to toSimpleString (outside compatibility tests). There is now-unused deprecated stuff that isn't tested.


Rename reserved categories variable

categories is a reserved word in Jekyll and our usage breaks in
later versions

Do we want to apply this change to all release branches?

older releases don't use this variable

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/10500/
Test PASSed.

Merged build finished. Test PASSed.

LGTM


Remove <~ from documentation examples

Several examples were still using this now outlawed signal map operator. I replaced those uses by Signal.map, so that the examples actually work with the current version of core.

@evancz, I think this is an important documentation fix. We want example code to actually work.


Project template support

Check out https://github.com/LeanderK/cookiecutter-ml - it's a clean project layout and could be adapted for Guild projects.

Closing for lack of activity.


Crates that are fetched but not added to the index because of a crash will be lost

Secondary problem: apparently crates that are fetched but not added to the index will be lost and not build. Relevant code: https://github.com/rust-lang/docs.rs/blob/master/src/docbuilder/queue.rs#L13
Originally posted by @jyn514 in https://github.com/rust-lang/docs.rs/issues/626#issuecomment-591989434
crates-index-diff just released a change in 5.1.0 that will let us peek without consuming the changes :tada: :tada: . This will give us much better durability in the event of a crash, I just need to implement using the new API.

Mentoring instructions:

In get_new_crates(), change the line from using fetch_changes() to using peek_changes().
After adding all new crates to the queue, call index.last_seen_reference().set_target(changes), as per the instructions in the changelog. See also git2::Reference.
Optionally, change add_crate_to_queue to add multiple crates at once, so the operation will be atomic and we won't add crates that we've already built if we crash halfway through adding crates.

If this is still open, I would like to work on it.

@harrisonbrock go for it! If you have questions, feel free to reach out here or on Discord :)

There was a new API added recently that will make this a little easier: https://docs.rs/crates-index-diff/6.2.0/crates_index_diff/struct.Index.html#method.set_last_seen_reference
Also note that we currently use crates-index-diff 4.0, so it will need to be upgraded to at least 5 (preferably 6) to use this new API.

Thanks. I have been a little sick and have not had a chance to start yet.

Get well soon!

I hope you are doing better, @harrisonbrock!
In case you need additional support related to how to use crates-index-diff, I am happy to help (I have authored crates-index-diff).

https://github.com/rust-lang/docs.rs/pull/746 only logged the failure, it will still not be added to the index.

Ok, this is a slightly different issue than originally - we record the failure internally, it won't be lost, we just don't display it anywhere but in the logs. So it seems from the outside as if the build was never there, but we can still rerun it with UPDATE queue SET attempts = 0;

Closing in favor of https://github.com/rust-lang/docs.rs/issues/903, the original bug this was for has been fixed.


Fix container instantiation timing, IID startup.

The container instantiation could happen at the wrong time, before all
components have been added to the container. This fixes it, and also
moves IID to use the proper instantiation timing instead of relying on
the configureWithApp: call. This is part continuation of #3147, where
I'll be fixing the rest of the SDKs in follow up PRs.
Edit: Further work...
This moves the instantiation of components to after the FirebaseApp is
completely configured and assigned, as it should be. This also exposed
a recursive issue with IID calling the Messaging singleton, which in
turn called IID and instantiated multiple instances.
A change was made to break the cycle: the Messaging isAutoInitEnabled
call was changed to a static method vs an instance method, allowing IID
to call it during initialization without instantiating the Messaging
instance (since it doesn't have a direct dependency on it).

Discussed offline with @paulb777 - we'll look at fixing the IID dependency on Messaging in a follow up PR by making Messaging instantiate first and initialize IID appropriately.


include license file

Many companies require a permissive license to allow use of open-source code (typically MIT or Apache2).  Some projects wish to restrict use to research or personal (often with Creative Commons BY-NC-SA).  For what it's worth, ggml uses the commercial-friendly MIT license.
Would you mind adding a license file that fits with your goals for the project?  That clarity would be greatly appreciated.

Full disclosure: I am with a company that is interested in including your software in our project.  We would obviously prefer a commercial license but will respect your decision.

Hi @humane-barry, thanks for the issue. I am happy to hear that you consider including it in your project, so added the MIT license file. It is the most permissive license I'm aware of.
I'd like to give it a try if you don't mind letting me know wen your product is released. Best of the luck!

Thank you for the license, and also so so much for your contribution to the AI community.


HTML

HTML
User Story
As a bootcamp student
I want the prework notes to be structured on a webpage
so that i can easily find and read information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I visit the website in y browser
THEN I see four boxes titled HTML, CSS, Git, and JavaScript with associated notes listed

I have added the HTML


Fix linear model warning

Changed BaseOptimizer class so that it inherits from sklearn.linear_model.LinearRegression instead of sklearn.linear_model.base.LinearModel. sklearn was throwing a warning that we won't be able to import from sklearn.linear_model.base in the future, i.e.
  /home2/bdesilva/.local/lib/python3.6/site-packages/sklearn/utils/deprecation.py:144: FutureWarning: The sklearn.linear_model.base module is  deprecated in version 0.22 and will be removed in version 0.24. The corresponding classes / functions should instead be imported from sklearn.linear_model. Anything that cannot be imported from sklearn.linear_model is now part of the private API.
    warnings.warn(message, FutureWarning)

I also

Added sklearn's _rescale_data function which we were also previously importing from sklearn.linear_model.base
Removed attribute argument being passed to check_is_fitted as it is being deprecated
Removed extraneous code from sindy.utils.base that was also using LinearRegression
Reformatted sindy.utils.base with black
Updated string formatting in sindy.utils.base

Sorry, this pull request was meant to be submitted elsewhere.


Conform names to updated TemplateFlow, add options conducive to small animal neuroimaging

Names conform to the BIDS standard as implemented in the current version of TemplateFlow.
Use of Laplacian alignment as a criterion during the final (CC-based) registration stage is made optional.
The bspline_fitting_distance is measured in mm. The default setting of 200 can become problematic when images have a smaller FoV, as is often the case in small animal neuroimaging. Thus, setting this value is also made available as an option. (I'm wondering if we shouldn't set this to a reasonable default based on the image affine rather than using 200.)

Good job!


Trying to change contract length has no effect for F1M 24

Pressing +1 and -1 does nothing. In the console, the error is Error: no such column: Accepted

This is due to a change that Accepted column is removed in M24. Would fix it soon

should be fixed now


Gradient not working on my Android 2.3.6

Android 2.3.6 default browser not showing gradient while other browser are fine. I have only tested in Android 2.3.6 version.
.discount-label {
left: -7.1em;
top: 1.4em;
background: url(../img/texturee.png), linear-gradient(left, #de2785 1%,#f954b1 50%,#de2785 100%);}
it's working in all but not android. For me now only option to add vendor-prefixes to whole css. I had made it for a big project.

If you manually add the prefixes instead of using prefixfree, does it work?

Yes if i add this manually
-webkit-gradient(linear, left top, left bottom, color-stop(1%,#de2785),
color-stop(50%,#f954b1), color-stop(100%,#de2785))
then it works. I would be happy if you fix this soon. it will save my time
because I have used lots of gradients in current project.
Jitendra Vyas
Senior Front End Developer
http://skarp.dk

Skype : jitendra.web I Twitter @jitendravyas I Linkedin:
http://linkd.in/jitendra I
Google Plus - gplus.to/jitendra I Read my Blog at http://j.mp/vyasblog
On Mon, Jan 16, 2012 at 10:32 PM, Lea Verou <
<EMAIL_ADDRESS>
wrote:

If you manually add the prefixes instead of using prefixfree, does it work?

Reply to this email directly or view it on GitHub:
https://github.com/LeaVerou/prefixfree/issues/36#issuecomment-3514202

-webkit-gradient() is NOT the same as -webkit-linear-gradient(). It has entirely different syntax. -prefix-free doesn't do any kind of transformation, it just adds the prefix if it's needed. If Android doesn't support -webkit-linear-gradient yet, you will need to add -webkit-gradient() manually.

OK do you know tool where I convert this type code
linear-gradient(left, #de2785 1%,#f954b1 50%,#de2785 100%);
into this -webkit-gradient(linear, left top, left bottom,
color-stop(1%,#de2785), color-stop(50%,#f954b1), color-stop(100%,#de2785))
I have multiple gradients.
Jitendra Vyas
Senior Front End Developer
http://skarp.dk

Skype : jitendra.web I Twitter @jitendravyas I Linkedin:
http://linkd.in/jitendra I
Google Plus - gplus.to/jitendra I Read my Blog at http://j.mp/vyasblog
On Mon, Jan 16, 2012 at 10:46 PM, Lea Verou <
<EMAIL_ADDRESS>
wrote:

-webkit-gradient() is NOT the same as -webkit-linear-gradient(). It has
entirely different syntax. -prefix-free doesn't do any kind of
transformation, it just adds the prefix if it's needed. If Android doesn't
support -webkit-linear-gradient yet, you will need to add
-webkit-gradient() manually.

Reply to this email directly or view it on GitHub:
https://github.com/LeaVerou/prefixfree/issues/36#issuecomment-3514535

http://lea.verou.me/demos/cssgradientsplease/ does that too (make sure not to include the trailing semicolon or it won't work)

Thanks. Nice time/life saver tool :)
Jitendra Vyas
Senior Front End Developer
http://skarp.dk

Skype : jitendra.web I Twitter @jitendravyas I Linkedin:
http://linkd.in/jitendra I
Google Plus - gplus.to/jitendra I Read my Blog at http://j.mp/vyasblog
On Mon, Jan 16, 2012 at 10:54 PM, Lea Verou <
<EMAIL_ADDRESS>
wrote:

http://lea.verou.me/demos/cssgradientsplease/ does that too (make sure
not to include the trailing semicolon or it won't work)

Reply to this email directly or view it on GitHub:
https://github.com/LeaVerou/prefixfree/issues/36#issuecomment-3514786


CAMEL-13795: Fixing the issue of already declared namespaces within the child

CAMEL-13795: Fixing the issue of already declared namespaces within the child
I have also align the tests classes

I have changed the source as recommended in pull request #3071.
I also add a extra test for this fix.
And I have also made some changes for better reading (in my opinion :-))

Thanks, can you update this PR so its similar to what you did for master

Ah you already did, thanks ;)


Fix Brier Score

gold_one_hot needs to follow the dimension of predictions so that it still works when --limit is used and the indexes in gold does not cover all gold indexes.

Thanks!


Handle case where aws thing has multiple certificates attached

Note: the last certificate seems to be the oldest

Does that lager:warning print out with the device or channel id?

They all do


Add autoclosing paranthesis and brackets

Add autoclosing paranthesis and brackets
Fixes #(#54)
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[ ] 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 not work as expected)
[ ] This change requires a documentation update
[ ] Documentation Update

Checklist:

[x] My works and is relatively clean.
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

I dont see any error for this. Can you explain this and attach a screenshot?

I dont see any error for this. Can you explain this and attach a screenshot?

I am not sure how you made this work on your machine, but you need to change the bindings for the textarea. <KeyRelease-{> isn't a valid binding. Here is a list of all the different binding commands you can use: https://www.tcl.tk/man/tcl8.4/TkCmd/keysyms.html.
An example of a valid one would be  or
You also have the function names mixed up. autoclose_brackets autocloses parenthesis and autoclose_parentheses closes brackets in your code. You should change that as well.

I have added the necessary changes

can you tell me the reason for closing this PR?


fix: serialize typing classes assigned to fields

We had a case where a function input used an lru_cache as attribute, not as a decorator, and lru_cache somewhere internally stores the return type of the function... and typing.Sequence for example has an attribute slots but that is None, so the serialization failed every time.
If someone has a better idea how to catch that or what types should be serialized to, I'd be glad to get input :)
Thanks!

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


Software broken: Symbol scaling suddenly wrong, import errors, footprints missing

I just installed the latest version via pip and while its awesome to see that the symbols are stored now in the new KiCad library format, I also noticed some urgent issues, making it currently impossible to use the plugin:

Imported symbols are wayyy smaller then before, making it impossible to connect them to other components

for some components I get an error like this one:

2022-04-08 13:32:35,499 - INFO - created 'JLC2KiCad_lib/footprint/FS-1340.kicad_mod'
2022-04-08 13:32:35,846 - INFO - creating schematic FS-1340_0 in default_lib
2022-04-08 13:32:35,849 - INFO - creating library for component C2979183
2022-04-08 13:32:36,260 - INFO - creating footprint ...
Traceback (most recent call last):
File "/usr/local/bin/JLC2KiCadLib", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/JLC2KiCadLib.py", line 128, in main
add_component(component, args)
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/JLC2KiCadLib.py", line 26, in add_component
footprint_name, datasheet_link = create_footprint(
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint.py", line 27, in create_footprint
footprint_name, datasheet_link, assembly_process = get_footprint_info(component_id)
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint.py", line 136, in get_footprint_info
response["data"]["componentPageInfo"]["list"][0]["componentModelEn"]
IndexError: list index out of range

some footprints (even including the one for the ESP32-WROOM-32E(4MB)) are missing, probably because of that error message?
most footprints however seem correctly imported. With some exceptions, like this SMD nut, which also doesn't have 4 pins / connectors, but just one conducting body

Thank you for reporting this issue.
I increased the default schematics size, i will later add an option to change it.
You can try it by updating the package using pip :
pip install JLC2KiCadLib==1.0.1
or
pip install JLC2KiCadLib --upgrade
The footprint of ESP32-WROOM-32E(4MB) is, indeed broken, but it does not seem to be linked to the error you mentioned. I will work on this.
Do you have the reference of the components the error ?

And I just found out for the ESP32-WROOM-32E(4MB) footprint, this issue came from the parenthesis in the footprint names, which where parsed by KiCad. I will replace these parentheses and the issue should not appear anymore.

awesome, thanks for the fast response and fixing it so quickly!:) Scaling is perfect again now.
Just re-downloaded the parts again. Some parts still get error messages. Any idea what's wrong here?
one example:

2022-04-09 14:35:39,381 - INFO - creating library for component C2879827
2022-04-09 14:35:39,787 - INFO - creating footprint ...
2022-04-09 14:35:40,628 - ERROR - footprint h_TRACK: layer correspondance not found
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint_handlers.py", line 46, in h_TRACK
layer = layer_correspondance[data[1]]
KeyError: '10'
2022-04-09 14:35:40,630 - ERROR - footprint h_TRACK: layer correspondance not found
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint_handlers.py", line 46, in h_TRACK
layer = layer_correspondance[data[1]]
KeyError: '10'
2022-04-09 14:35:40,630 - ERROR - footprint h_TRACK: layer correspondance not found
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint_handlers.py", line 46, in h_TRACK
layer = layer_correspondance[data[1]]
KeyError: '10'
2022-04-09 14:35:40,630 - ERROR - footprint h_TRACK: layer correspondance not found
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint_handlers.py", line 46, in h_TRACK
layer = layer_correspondance[data[1]]
KeyError: '10'
2022-04-09 14:35:40,630 - ERROR - footprint h_TRACK: layer correspondance not found
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint_handlers.py", line 46, in h_TRACK
layer = layer_correspondance[data[1]]
KeyError: '10'
2022-04-09 14:35:40,635 - INFO - creating 3D model ...
2022-04-09 14:35:41,001 - ERROR - request error, no 3D model found
2022-04-09 14:35:41,010 - INFO - created 'JLC2KiCad_lib/footprint/U261-121N-4BS2S.kicad_mod'

and another one

2022-04-09 14:36:04,385 - INFO - creating schematic SMTSO3015CTJ_0 in default_lib
2022-04-09 14:36:04,389 - INFO - creating library for component C2916320
2022-04-09 14:36:04,758 - INFO - creating footprint ...
Traceback (most recent call last):
File "/usr/local/bin/JLC2KiCadLib", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/JLC2KiCadLib.py", line 128, in main
add_component(component, args)
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/JLC2KiCadLib.py", line 26, in add_component
footprint_name, datasheet_link = create_footprint(
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint.py", line 27, in create_footprint
footprint_name, datasheet_link, assembly_process = get_footprint_info(component_id)
File "/usr/local/lib/python3.9/site-packages/JLC2KiCadLib/footprint/footprint.py", line 136, in get_footprint_info
response["data"]["componentPageInfo"]["list"][0]["componentModelEn"]
IndexError: list index out of range

that second one I get multiple times.

The issue was present for components available on EasyEDA but not present on JLCPCB as I fetch some information on their website. I now fetch the datasheet link and the footprint from EasyEDA which solved the issue.
I updated the package,  you can update it with  pip install JLC2KiCadLib --upgrade
I tested it on a few components and I didn't get any issues, let me know if you encounter some.
If it works as expected, you can close this issue.

After more testing, it seems that another error can occur.
The footprints at esayEDA do not have exactly the same descriptions which can cause problems.
I will work on it to solve them.

Issue should be fixed, this component had a line on the 'Edge.Cuts' layer which I had not yet implemented.
I updated the package, you can update it again with pip install JLC2KiCadLib --upgrade
If everything works properly, you can close this issue.

awesome!:) no python errors anymore.
Only those warnings/errors:

2022-04-11 16:01:46,476 - INFO - creating library for component C2979183
2022-04-11 16:01:46,969 - INFO - creating footprint ...
2022-04-11 16:01:47,416 - WARNING - footprint handler, h_TEXT: failed to add text
2022-04-11 16:01:47,416 - WARNING - footprint handler, h_TEXT: failed to add text
2022-04-11 16:01:47,418 - INFO - creating 3D model ...
2022-04-11 16:01:48,014 - ERROR - request error, no 3D model found
2022-04-11 16:01:48,021 - INFO - created 'JLC2KiCad_lib/footprint/BAT-TH_MY-18650-01-2.kicad_mod'
2022-04-11 16:01:48,436 - INFO - creating schematic MY-18650-01_0 in default_lib
2022-04-11 16:01:48,438 - INFO - creating library for component C2979182
2022-04-11 16:01:48,851 - ERROR - failed to get component uuid
2022-04-11 16:01:48,852 - INFO - creating library for component C383071

is there anything one can do about those errors? Or are they caused by an incomplete database from LCSC or JLCPCB?

also, KiCad gives me this error, when opening the footprint editor.

Errors were encountered loading footprints:
Expecting 'at' in '/Users/..../Library/Mobile Documents/com~apple~CloudDocs/KiCad parts/JLC2KiCad_lib/footprint/ESP32-WROOM-32E(4MB).kicad_mod', line 7, offset 34.

Simply delete the file. I replaced the parenthesis in the footprint name (which were the cause of the errors), but your old broken file is this present in your library and KiCad is trying to load it.
The footprint should now be in the ESP32-WROOM-32E_4MB_.kicad_mod file.

awesome!:) no python errors anymore.
Only those warnings/errors:

2022-04-11 16:01:46,476 - INFO - creating library for component C2979183
2022-04-11 16:01:46,969 - INFO - creating footprint ...
2022-04-11 16:01:47,416 - WARNING - footprint handler, h_TEXT: failed to add text
2022-04-11 16:01:47,416 - WARNING - footprint handler, h_TEXT: failed to add text
2022-04-11 16:01:47,418 - INFO - creating 3D model ...
2022-04-11 16:01:48,014 - ERROR - request error, no 3D model found
2022-04-11 16:01:48,021 - INFO - created 'JLC2KiCad_lib/footprint/BAT-TH_MY-18650-01-2.kicad_mod'
2022-04-11 16:01:48,436 - INFO - creating schematic MY-18650-01_0 in default_lib
2022-04-11 16:01:48,438 - INFO - creating library for component C2979182
2022-04-11 16:01:48,851 - ERROR - failed to get component uuid
2022-04-11 16:01:48,852 - INFO - creating library for component C383071

is there anything one can do about those errors? Or are they caused by an incomplete database from LCSC or JLCPCB?

ERROR - request error, no 3D model found : mean that there is no 3D model available on their library. If you go on easyEDA, load the component and try to view the 3D model (view -> 3D view) you will not have the 3D model either.

ERROR - failed to get component uuid : Check the part number you are trying to import. There seems to be no component with the code C2979182  on easyEDA. You might have made a mistake.

WARNING - footprint handler, h_TEXT: failed to add text  : This is caused by an inconsistency in the request, I will try to handle it later. Meanwhile, you can manually add these two text label with the KiCad library editor.

ok, good to know :)
Since those IDs are all from LCSC, I tried to download all the parts which I bought from LCSC. Including C2979182. And yeah, that is one of the parts that doesn't have a footprint or a 3D model.
Just some fine tuning suggestion: it might make sense to replace the "ERROR - failed to get component uuid" message with something like "ERROR - no footprint found for C2979182", think that would be better understandable :)


Problem with Append Queries

Hi,
is it possible that there is a problem with Append Queries in order to combine data for several companies in BC version 14.0 ? In my case he is duplicating only data from the first company multiplied by the number of companies I have (27) if I use the PowerBI Business Central connector. I'm logged in with an organization account.
Regs,
Tobias

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

ID: f562094f-cff2-2c71-c806-e548041718e2
Version Independent ID: 4f1e5021-2ea3-ba4b-8da8-a999589b8962
Content: Setup Reporting for Business Central in Power BI - Business Central
Content Source: business-central/across-how-use-financials-data-source-powerbi.md
Service: dynamics365-business-central
GitHub Login: @edupont04
Microsoft Alias: edupont

Hi toblan,
This field is for submitting feedback about the docs. You might want to ask your question in the Dynmaics 365 Business Central forum here: https://community.dynamics.com/business/
I'm closing the issue as it is a question about product functionality.


CHANGELOG.md is missing

https://github.com/angular/angular-cli/blob/master/CHANGELOG.md
Referenced from the README.md

You now have to use https://github.com/angular/angular-cli/releases
Changelog has been removed on purpose

Maybe we can replace the 404 with it and perhaps much more prominent on the main angular-cli page?

No you can't replace the 404 page with it, you'd have to make a redirect and it would be on Github side.
I saw it on the changelog so I think it was there for few days/weeks and now changelog has been removed.
Agree that it might be written somewhere as devs are (probably) used to take a look into this changelog.

Can be closed due to: https://github.com/angular/angular-cli/pull/6908

"Changelog has been removed on purpose"
What is that purpose? All other angular libraries have the CHANGELOG.md. I find it very useful. Why was it removed? I don't see any discussion or reference to a why it was removed. An explanation would be good.

@ccadieux It's now powered by GitHub Releases. You can find exactly same information there.

As stated above by a few people... the changelog file is no longer being maintained... the same information is available via the github release feature located here.

How about creating a CHANGELOG.md file which tells the reader to go to the GitHub releases page? That way one at least doesn't have to google to find the changelog (which is precisely how I arrived here).


Updated directory structure for k8s manifest files

Closes #190
This PR attempts to reorganize the k8s manifest files used to deploy prombench test infra.
the new dir structure for /manifests looks like this:
├── cluster-wide
│   ├── 1_gcloud_rolebinding.yaml
│   └── 2_ingress-nginx-controller.yaml
├── cluster.yaml
├── dashboard
│   ├── 1a_prometheus_rbac.yaml
│   ├── 1b_prometheus-meta.yaml
│   ├── grafana_dashboard_provision.yaml
│   ├── grafana_dashboard_prow_bench.yaml
│   ├── grafana_dashboard_prow_node.yaml
│   ├── grafana_datasource.yaml
│   ├── grafana_deployment.yaml
│   └── node-exporter.yaml
├── prombench
│   ├── benchmark
│   │   ├── 1a_namespace.yaml
│   │   ├── 1b_serviceaccount.yaml
│   │   ├── 1c_cluster-role-binding.yaml
│   │   ├── 2_fake-webserver.yaml
│   │   ├── 3_prometheus-test.yaml
│   │   ├── 4_node-exporter.yaml
│   │   ├── 5_nginx-ingress-routes.yaml
│   │   └── 6_loadgen.yaml
│   └── nodepools.yaml
└── prow
    ├── components
    │   ├── prow_internals_1.yaml
    │   └── prow_internals_2.yaml
    └── secrets.yaml

6 directories, 23 files

Can we join cluster-wide and dashboard folders?
I think should also remove
https://github.com/prometheus/prombench/pull/201/files#diff-04c6e90faac2675aa89e2176d2eec7d8R36
maybe put it as a commend in the actual deployment, but don't think we need it in the readme file.

I think should also remove
https://github.com/prometheus/prombench/pull/201/files#diff-04c6e90faac2675aa89e2176d2eec7d8R36

this link is not working I think, can you review comment on this one?

I think should also remove

The GCLOUD_SERVICEACCOUNT_CLIENT_EMAIL is used to grant cluster-admin-rights to the service-account. This is needed to create RBAC roles on GKE.

maybe put it as a commend in the actual deployment, but don't think we need it in the readme file.

the rbac manifest file already has a comment, so I just removed the note from the readme.
https://github.com/prometheus/prombench/blob/a0fd6918f681120f9f7f06dbf2a9c122f870d6f4/manifests/cluster-wide/1_gcloud_rolebinding.yaml

not having a domain point to the ingress IP will not break the deployment but all functionalities will not function as expected.

I  just noticed that we have 2 sections that explain about the Meta and Grafana deployment:
https://github.com/prometheus/prombench/blob/a0fd6918f681120f9f7f06dbf2a9c122f870d6f4/README.md#deploy-prometheus-meta--grafana
and
https://github.com/prometheus/prombench/blob/a0fd6918f681120f9f7f06dbf2a9c122f870d6f4/README.md#deploy-prometheus-meta--grafana-1
can you combine these into one and explain how old we get the nginx-ingress-controller IP address to point the domain to.
Can also mention that the domain name can be any domain name.

Also can we join cluster-wide and dashboard folders?

I joined cluster-wide and dashboard directories and I'll added the other changes too. I initially split them because I thought it will enable the user to run the benchmark tests manually without having to setup the dashboards. But having the dashboards will only help.

@krasi-georgiev is there something I should add to this?

all good thanks!


Update pbo-patterns.txt

another case where you can remove unnecessary and a / or a

Thanks Artrag! merged!


dotnet publish-iis keeps adding the app name to web.config

Following https://docs.asp.net/en/latest/publishing/iis.html:
I'm using the Getting started project and have the following project.json:
{
  "version": "1.0.0-*",
  "buildOptions": {
    "debugType": "portable",
    "emitEntryPoint": true
  },
  "dependencies": {},
  "frameworks": {
    "netcoreapp1.0": {
      "dependencies": {
        "Microsoft.NETCore.App": {
          "type": "platform",
          "version": "1.0.0"
        },
        "Microsoft.AspNetCore.Server.Kestrel": "1.0.0",
        "Microsoft.AspNetCore.Server.IISIntegration": "1.0.0"
      },
      "imports": "dnxcore50"
    }
  },
  "tools": {
    "Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-*"
  },
  "scripts": {
    "postpublish": "dotnet publish-iis --publish-folder %publish:OutputPath% --framework %publish:FullTargetFramework%"
  }
}

If I call the following four times:
dotnet publish
The web.config will have the following line (note that the app name keeps getting added):
<aspNetCore processPath="dotnet" arguments=".\aspnetcoreapp.dll .\aspnetcoreapp.dll .\aspnetcoreapp.dll .\aspnetcoreapp.dll" stdoutLogEnabled="false" stdoutLogFile=".\logs\stdout" />
Also, I have a web.config in my app but it's being ignored (that one has stdoutLogEnabled="true").

Try
"Microsoft.AspNetCore.Server.IISIntegration.Tools": "1.0.0-preview2-final"

publish-iis tries to preserve arguments and is replacing only %LAUNCHER_ARGS%. So, if your web.config is already set up correctly you probably should disable publish-iis in the postpublish script.
One thing that I am a bit surprised is that you have many .\aspnetcore.dll. publish-iis works only on the web.config file that was already published so I would expect at most one extraneous .\aspnetcore.dll. Since you have many it looks like they are being changed in the source web.config file - are you publishing to the folder where the source lives thus overwriting the source web.config each time?

Same thing is happening to me.  I don't have a web.config in my project --  publish-iis is generating the web.config on its own.  But each time I run publish-iis it adds another instance of the name of the dll to the arguments attribute of aspNetCore.

I don't think it is possible for publish-iis to append anything if there is no web.config file since it is writing the config from scratch. Please note that publish-iis reads config from the publish folder and not the application folder so even if you did not have web.config in your application and you publish to a target folder it will create one. If you publish again to the same folder, because you did not have a web.config in your app, the one created during the first publish was not overwritten by anything and therefore it is the one that publish-iis uses which results in appending to the path. One of the workarounds would be to delete the web.config file from the publish destination folder before publishing.

@moozzyk Indeed it happens when you publish more than once in a row.  Yes, deleting would be the work-around.  I was hoping that publish-iis would have an option to delete everything in the destination before publishing, but haven't been able to find it.  (Haven't in fact been able to find any docs on the command-line arguments for that command.)

@dkent600 I probably could have (or should have) provided some more detail here: https://docs.microsoft.com/en-us/aspnet/core/publishing/iis#publish-iis-tool
... and I would do so, except that the publish-iis tool is going away with the move to MSBuild/csproj. Soon, that section is going to change over to address how Microsoft.NET.Sdk.Web handles web.config.

We are closing this issue because no further action is planned for this issue. If you still have any issues or questions, please log a new issue with any additional details that you have.


Regarding recommended values of "request.timeout.ms" and "session.timeout.ms".

Could you add recommended values  of "request.timeout.ms" and "session.timeout.ms" to all consumer configuration of this document?
According to the following document, Apache Kafka consumer is mentioned recommended values of "request.timeout.ms" and "session.timeout.ms":
No records received | Troubleshoot issues with Azure Event Hubs for Apache Kafka - Azure Event Hubs | Microsoft Docs
However, Recommended configurations for Apache Kafka clients - Azure Event Hubs | Microsoft Docs document doesn't mention about that. Almost customers don't read the above Troubleshoot document until facing some issue. Therefore, some customers faces client issue. To avoid runing into this issue in advance, we want you to add recommended values of "request.timeout.ms" and "session.timeout.ms" in Recommended configurations for Apache Kafka clients - Azure Event Hubs | Microsoft Docs in all Consumer configuration.
Thanks in advance.

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

ID: 4dee0d90-93d4-b15c-c82b-1877b77d2e0f
Version Independent ID: cc3baaa7-6ec9-723e-1fee-e5bc81437667
Content: Troubleshoot issues with Azure Event Hubs for Apache Kafka - Azure Event Hubs
Content Source: articles/event-hubs/apache-kafka-troubleshooting-guide.md
Service: event-hubs
GitHub Login: @spelluru
Microsoft Alias: spelluru

Thanks for the feedback! We have assigned the issue to the content author to further review this and update the document as appropriate.

Thank you!! Updated the recommended configurations article. The updated article will go live tomorrow morning at 4 AM PST.
#please-close


json based error codes from  nginx

STQC Fix : json error codes will be returned from nginx instead of html page.

json formatted error page - need to specially configure to output as json content or else nginx will serve default html format.
proxy intercept error code - Nginx server will proxy the same content from backend server even if its error.  Intercept setting in nginx should be on, then only can it display the standard error pages defined by us.


Recost LookupJoins to remove special casing for indexes where every column is used in a filter.

I'm pulling this out of https://github.com/dolthub/go-mysql-server/pull/1888 into its own PR so that it can be benchmarked on its own.

#benchmark


Set tracing's release_max_level_debug feature globally

Logs at TRACE level are too noisy and bloat the binaries.
Enabling release_max_level_debug feature will prevent logging at TRACE level and the code is removed from release builds.

FTR this is the source code for STATIC_MAX_LEVEL.
What about adding --features tracing/release_max_level_debug to CARGO_FEATURES in _build-and-test-locally.yml instead? @bayandin
I don't care much in either direction.


Rename Executable retry limit field, getter and setter

This PR resolves #459
These changes are entirely semantic and replace Retry with Attempt, since this more closely captures the intent of this code. We are keeping track of the limit/count of retries, rather than a reference to any specific retry.

Thanks for the MR @Regan-Koopmans , but unfortunately we cannot simply replace [set|get]MaxRetry(), we have to keep both. I should have specified that in the issue; since we have released a major version change we need to deprecate the old methods and add the new methods, while maintaining functionality for both.

@danielakhterov That makes complete sense. I have reintroduced the methods with deprecation annotations.

Fantastic work, thanks again @Regan-Koopmans for the PR.


Incoming calls should put the application in foreground

Currently, the client
-> is ringing
-> triggers an incoming call notification (tapping on it opens the application)

https://github.com/vector-im/vector-android/commit/9dab4a553ba9a32c0ea974d859706dd1d4b9cdeb


Add your school to JetBrains

Adding a new school to Add your school to JetBrains
This is a school from Spain that's not in the Jetbrain system yet. This file was generated from jetbrains site for adding new schools.

@hikager Pull request merged. Thank you!

@hikager Pull request merged. Thank you!


Stop generating events on max retries

'Problem'
When the max retry limit has been reached, the SDK will stop trying to resend events to the backend.
This saves data usage and battery because of saved cpu calculations.
Events however, are still generated and added to the eventbus. These are minor calculations, but it's possible to optimize it in order to reduce overhead and therefore save battery usage.
Solution
Stop tracking events when we know we're not going to send them anyway.
Potential side effect
May the desired function definition for 'max retries' change into 'try it again at a later time', it would have been desirable to have tracked the events anyway.

This is undesirable until further notice. As the potential side effect above explains, there's no good reason to prematurely implement this.


Update TablePagination docs

After 1st of September fix, onChangeRowsPerPage accepts only value, not option

Fixed (based on docs)


Wondering if I can use this to authenticate to Active Directory

This is more of a question than an issue, and discussions aren't set up on this repo or organization, so I guess I'll ask here.
Is there a way to use this to authenticate an LDAP connection to Active Directory?  I'm writing a command line application in Go which interacts with an on-premise AD domain and I'd like to avoid storing secrets if I can help it.  Each user already has a valid Kerberos ticket and is authenticated to AD, I just don't know how to make use of that when I want to talk to LDAP.
Thank you!

I should mention that I am on MacOS.

Hi - thanks for the interest.  Right now the code is not in a state that it should be used in a production context .. while I'm fairly happy with the GSSAPI code, the underlying go-native Kerberos library isn't as well maintained as it needs to be.  I actually started this with the same goal of providing krb auth for LDAP but never managed to spend enough time on it unfortunately.  I think that the native go krb library needs some feeding and watering and/or a well supported C binding needs to exist for GSSAPI on Go to something I would use.  In the mean time I wonder if mTLS is a better option for AD?..

I'll look into that, thank you.


Unable to configure PublicRuntimeConfiguration per readme

Reproduction: https://github.com/tommcclean/nuxtgtmruntime/blob/master/nuxt.config.js#L56

The reproduction is a new vanilla Nuxt site using >2.14, only modifications are installing this module and updating the config.

Expectation

When running the website with either yarn dev or yarn build + yarn start
GTM would request the XHR: https://www.googletagmanager.com/gtm.js?id=GTM-123456 using the publicRuntimeConfig override

Actual

GTM uses the default and requests https://www.googletagmanager.com/gtm.js?id=GTM-XXXXXXX

NB

When I log out the GTM object from runtimeConfig, I can see it is evaluated in runtime; it just doesn't update the module settings.
It also doesn't look like it has been developed yet to support variables; which is in our use case

Hmm, thinking maybe it might not be published actually; the readme in this repo isn't the same version as the one on NPM

^ Thats the badger, hasn't been published since July

Yep, still hasn't been updated.

Yeah I am not sure if this repo is maintained; I tried to reach out to get a publish but couldn't get a reply.
For me I ended up writing my own plugin which initialises GTM, as that gets done at runtime and can support the other variables being dynamic too

Hi. Sorry i kinda lost track of this module a while. Feature released: https://github.com/nuxt-community/gtm-module/releases/tag/v2.4.0


[Google Colab] Format of output video cannot play in some contexts

The current export_to_video function from HuggingFace encodes the output video with MPEG-4 Visual format, which doesn't play in some contexts such as Desktop Discord. (They can play in normal video players and mobile Discord.) Looking at the HuggingFace Space version, they are using imageio to export, which results in a video file with AVC format.
Could the colab be adjusted so the output format is functional in more contexts?

thanks @CarbonatedPastries ❤please try it https://github.com/camenduru/text-to-video-synthesis-colab/commit/3cd036e26cf65c8725aa9a494f3a2e6c7bb1a9ad

Tested and outputs can play in Desktop Discord. Thanks!


Add a new bref deployment command to debug failed deployments

If an error occurs when deploying with SAM we need to look into the Events tab of the stack in the CloudFormation dashboard.
This works but isn't friendly and practical. This PR introduces the following Bref command to list the logs from the latest deployments and understand what went wrong:
vendor/bin/bref deployment <stack-name>

Only events from the last 24 hours are displayed. We could make it configurable in the future (PRs welcome).
Preview of the output:

:heart:


Update terraform equinix provider to v2.5.0

We are currently on v2.4.1. See release notes at https://github.com/equinix/terraform-provider-equinix/releases/tag/v2.5.0

Superceded by https://github.com/rancherlabs/terraform-harvester-equinix/issues/100


Does the code support multi-gpu training?

Hi, thanks for your great work! I use the command python train.py --cfg_path ${config_path} --gpu_id 0,1,2,3 to train the model, however, it seems only the first gpu is working. Does the code support multi-gpu training? Can you share the multi-GPU training command? Thanks a lot.

Hi, the repo doesn't support multiple GPU training.


bug: ion-select shows the labels of selected nullish option

Bug Report
Ionic version:
[x] 5.0.4
Current behavior:
When the underlying value of a ion-select is null or undefined, it shows only the placeholder. However, when an option having such a value is selected manually, its label is shown.
Expected behavior:
The select box should never show the label of null or undefined options, even when they are selected manually. This is also in line with the behavior of mat-select in Angular Material.
Steps to reproduce:

Visit the attached StackBlitz repro.
Use the native select box to select Undefined option or Null option. Note how the Ionic select keeps showing the placeholder.
In Ionic select, selec Undefined option or Null option. In both cases, the label stays visible after selecting.

Related code:
https://stackblitz.com/edit/angular-b4nt6c

This one is really strange, if i change [ngValue] of select native to [value], It select nothing at ion-select.

@indraraj26 AFAICT ngValue is what enables using objects as options. Angular for some reason requires it, while ion-select (and Material) support objects out of the box using value.
Also this bug may be related to #20722 (a problem in the select pop-up). I was unsure about reporting them together so I opened two issues to avoid mixing things.


[Discuss] Add register_fd for epoll

This PR mainly modifies the use of epoll, including 4 parts:

Always set EPOLLONESHOT in epoll_ctl when ONESHOT is set in photon. As is disscussed in https://github.com/alibaba/PhotonLibOS/issues/398. This change is meant to help address part2 issue.

https://github.com/alibaba/PhotonLibOS/pull/388 shows a performance issue when using epoll in a multi-thread program. This PR tries to fix it by allowing user to avoid frequent EPOLL_CTL_ADD:

Offer user a new API to register a fd
When rm_interest is going to delete the fd from epfd, just do nothing instead if the fd is registered (no more events of fd will be reported by epfd, because EPOLLONESHOT is set)
Further add_interest will always use EPOLL_CTL_MOD rather than EPOLL_CTL_ADD
Unregister the fd will EPOLL_CTL_DEL it

entry in _inflight_events will not be modified until the whole add_interest/rm_interest succeeds, preventing dirty data of entry in some failure cases

Combine several rm_interest in wait_for_events, concerned that:

Reduce overhead of multiple calls of epoll_ctl
Consider such unexpected case if no combination: fd is registered as part1 says. EVENT_READ and EVENT_WRITE is reported at the same time by epoll_wait. When rm_interest EVENT_READ, it finds that there is still EVENT_WRITE in entry's interests, so EPOLL_CTL_MOD is used. After that, when rm_interest EVENT_WRITE, nothing will be done because no interest is in entry. Now it is expected that no events of this fd will be reported, but actually EVENT_WRITE is still there.

I don't know if it is necessary to rm_interest event individually (i.e. NO combination). If it is, I need to find another way to address issue 4.2
Finally, the new APIs register_fd and unregister_fd is not that good-designed, as it works only for epoll. Better APIs may be needed. So this PR is not formal and the main goal is looking for more discussion and suggestion.

Thanks for your PR. It seems to me that register_fd() is not necessary, as you know in add_interests() whether an fd has been added in epoll or not, and use proper operation in epoll_ctl().
Correct me if I miss something.

Besides close(fd), you can also delete the event interest(s) from epoll when you wait_for_fd() with an empty event set.

Solved by https://github.com/alibaba/PhotonLibOS/pull/410


fix(url): fix url for gPlusCount and when provided url includes anc…

…hor (#)

Coverage remained the same at 96.726% when pulling b816987503062b38f9d0217eb2a3e1f520da45ef on tinesoft:fix_url into f56264b2874711d7427e50aab50659ba9a30dfcf on MurhafSousli:master.


Demo project is broken (Can't resolve tween.js)

I'm trying to build the demo site, after running npm install && npm start it shows:
ERROR in ./src/utils/map-utils.js
Module not found: Error: Can't resolve 'tween.js' in '/Users/alexb/Projects/deck.gl/demo/src/utils'
 @ ./src/utils/map-utils.js 10:13-32
 @ ./src/components/home.js
 @ ./src/routes.js
 @ ./src/main.js
 @ multi (webpack)-dev-server/client?http://localhost:3000 ./src/main
webpack: Failed to compile.

Tried with npm 4.x and 5.x, failing in both. In 4.x npm identified tween as extraneous.

@ale0xb Confirmed, this works with yarn but not with npm
I'm putting up a PR.


Is it possible to make jishaku hide by default.

You can set the JISHAKU_HIDE environment variable to a bool-like string to achieve this, the possible values are true, t, yes, y, on and 1.
See #4 and the closing commit ca5fa0bdf5a40e7e764050ca7f5234791446b3d1 for the implementation / discussion.

I'm confused..... How would I change the JISHAKU_HIDE var? would I have to make my own .env with it?

It depends on the OS and process manager you use. How are you running your bot? Are you using systemd, init scripts, Windows service manager, running your bot directly on the command line, launchd, or something else?

Oh. I forgot about this. I got it to work :) (I use repl) (which does suck)


Enable injecting middlewares into the Firebase API client

Description
It makes possible adding a Guzzle middleware to the Guzzle HTTP client. This feature was recently introduced in the SDK in version 7.3.0
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have made corresponding changes to the documentation

Thanks! 🙏🏻


Export CSV Page - filtering section

Follow Figma designs for the Export CSV section (follow concept number 2)
implement filtering features (right side of page)

/cib export csv page - filtering section

Branch 28-export-csv-page---filtering-section created!

/cib export csv page - filtering section

Branch already exists


[Connector-V2] Add Email sink connector

Purpose of this pull request

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Hi, please resolve confilt first. Thanks

please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Hi, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Hi, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Hi, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

done;


fix patches list missing most objects

the issue was a default limit of 20 in AssetService.find()

lgtm


Cannot read property 'toast' of undefined

" Cannot read property 'toast' of undefined ".This problem happens in the official demo and my project.
I have used this serval times , but until today , I get this problem.
Any help will be appreciated~

Hi MaxBalance,
Is the plugin added to the project with cordova like in the docs?
Are you testing on a real device or in the browser?

Thanks a lot.I put the plugins with a wrong path...


Configure Parsys Placeholder

Hi,
On the version 2.3.2 there is a feature called touchui-configure-parsys-placeholder that tries to request a configuration from a path like that: /etc/designs/geometrixx/jcr:content/contentpage/par
But as I don't have this path configured in my repository, neither I want to use this feature, I believe that this feature should starts turned off and only when I want to use it I should have a way to turn it on.
When this feature starts and tries to access a node that doesn't exist the author can see this error on the console:
granite.min.js:64 GET http://localhost:4502/etc/designs/default/jcr:content/templateWeb/par.2.json?_=1461268087386 404 (Not Found)
Which is the better way to resolve this issue?

I've also just come across this same issue and hoping for a resolution.

Sorry, perhaps I'm being dense, but why is this an issue? A 404 is a valid response and this doesn't prevent other JS on the page from working.

Hi, i'm facing the same issue. Any fix or work around.
Currently i have installed ACS commons version 2.8.

Facing the same issue on 3.0.2 any resolutions found for this issue?

Has this issue been resolved? I'm using the latest version of AEM Commons (3.7.0) and still seeing the 404 console errors when hovering over parsys.

@adpv-gm can you show the line of code that is initiating this XHR? Like @justinedelson requesting something that doesnt existing isnt an error; it can simply be checking to see if it exists, and if it doesn't it will react accordingly. The 5xx class of responses are the ones that you ought to be worried about.

We had the same issue. All the jsons with .2.json are being blocked. We commented this function below and issue disappear. Please recommend what this function is for and if we have any alternate solution?
`/*        $.ajax( { url: path + ".2.json", async: false } ).done(function(data){
if(_.isEmpty(data) || !data[ACS_COMPONENTS_LIMIT]){
return;
}
        currentLimit = data[ACS_COMPONENTS_LIMIT];

        var limit = parseInt(data[ACS_COMPONENTS_LIMIT]);

        isWithin = children.length <= limit;
    }); */`

@shubhakirani with the next release, you will be able to disable this functionality (it will still be enabled by default for backwards-compatibility reasons).

Fixed via https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/944


Fixed build failure when building with VS 2015

I had to remove AssignTargetPaths from BeforeTargets on the IncludeVsdConfig target or the .vsdconfig isn't generated. Also, sets VSSDK140Install if not already defined.

Thank you!


Use double quotes in output for defaults

Windows doesn't recognize 's for wrapping values. "s should work on all
systems.
Fixes #136

I wonder if it's the case that %q fits here which should be used in function fmt.Sprintf():

%q	a double-quoted string safely escaped with Go syntax


Unmarshaling events and attributes does not cost gas in dynamic link branch

https://github.com/line/wasmvm/blob/9ca6a90d72b4075ab0cdfa5aaf7ee700bfc6a8d8/lib.go#L161
does not cost gas in
https://github.com/line/wasmvm/blob/9ca6a90d72b4075ab0cdfa5aaf7ee700bfc6a8d8/lib.go#L149
But it should be costed.

Merged https://github.com/line/wasmvm/pull/98


Use streams for blobs

Changed to use streams instead of trying to load the entire contents of the blob into memory/array/httpRequest to allow processing of files >2GB. Also added the blob file name to some of the logging messages.

Will resubmit an updated request - there's one additional file that needs to be updated


No way to have urls devoid of language prefix.

I tried removing the multilingual middleware... except this now just causes 404 page not found errors (even if i destroy all pages and create them again)
and urls still have the language prefix on them

Can you provide more detail? What version, what settings, etc? Are there actually multiple languages, or just one?

there is only one language... but i'm not sure why that is relevant... If I simply don't want anything regarding lanuages to show up in the urls then there isn't a way to make that happen.
and when i install djangocms, it's pip install git+git://github.com/divio/django-cms.git

It's relevant because it's not something I've personally ever encountered, using only one language, across any of the released versions.
Would it be at all possible for you to try against a release (2.2, or even 2.1) to determine if it's an issue with develop? I appreciate that is potentially some work for you, but the more information we can get, the better.
Can you also provide any CMS related settings from your Django settings module?

Anecdotal evidence in support of this issue: Spinning up a copy of the demo, which has only the one language ('EN') and adding pages does put the language prefix into the menu URLs on the front-end.
They are addressable with or without the language denominator though, and I've not encountered the prefix issue in any sites I've been involved in.
I don't know enough about the multi-lingual side of the apps to know if it's a configuration mismatch, but I assume it must be. Getting your CMS related settings into this ticket would be a great step to confirming any problem.
Additionally, can you say whether you were following the getting started docs?

That's likely the case - I've just tried that against 4ee24182cc3d6f88986442ae94f576e7abee3c46 (develop HEAD, at the time of writing):
Turning the i18n machinery off, and removing the multilingual middleware gets rid of the URL prefixes; without having to delete/add any pages. Changing those two settings also removes the ability (rightly) to access the page with a prefix. Tested using a copy of the test project.
L10N shouldn't need to be turned off, and probably shouldn't be, as it does localisation configuration for dates etc.
@airtonix can you let us know if the proposed settings alterations fix it for you?

thanks for testing this kezabelle, closed

i've been so busy with work and making websites for other clients with djangocms that i haven't been able to test this.
So i'm doing that now.

ok so when i start a new project and use the four modifications outlined by @beyondwords i get what i expect.
however when i make those same modifications to a site that previously had language codes as part of the url, everything breaks.

I'm still personally unable to replicate it, and short of either your browser caching the redirect, or the menu pool still being cached, I can't understand why I wouldn't be able to, as the process you describe is fairly simple.
For documentation purposes, I did the following:

created 3 pages in the admin (using the test project, with default settings)
marked them all as published and in navigation.
visit front-end, and navigate to the deepest node ('page 3') - the URL is prefixed with /en/
halted the server and flipped i18n and l10n off, and deleted the middleware & context processor.
restarted server, refreshed the page. It now 404s, because /en/page2/page3/ isn't mounted.
visit front-end pages-root (/), and navigate to page 3 again - it is now not prefixed with /en/.

To move this forward and get it re-opened, it would still help to see all your CMS related settings, including any that are in the context or middleware lists. Further, if you can either figure out a test case, or provide some sample data by replicating it against a fresh database and then doing dumpdata, that would help.

https://gist.github.com/1519138

I believe I worked it out, if you look at my settings above, you'll see that even though i have :
MIDDLEWARE_CLASSES += (
...
#	'cms.middleware.multilingual.MultilingualURLMiddleware',
...
)

302 redirects to urls prefixed with the language code ("en" in my case") will still occur If you have :
LANGUAGE_CODE='en/AU'

Also :
$ ./project/manage.py shell
Python 2.7.2+ (default, Oct  4 2011, 20:06:09)
[GCC 4.6.1] on linux2
Type "help", "copyright", "credits" or "license" for more information.
(InteractiveConsole)
>>> import cms
>>> cms.__version__
'2.2'
>>>

just my two cents, LANGUAGE_CODE='en/AU' is NOT a valid language code as defined in RFC 5646. You are probably looking for 'en-AU'. See http://tools.ietf.org/html/rfc5646

As far as I can tell, the effect you're experiencing is actually due to CMS_FALLBACK_LANGUAGE being True, combined with an invalid language code. Without that setting being true, I get an entirely different, far more sensible error (Title matching query does not exist that can be attributed to language code error).

I had a discussion with @ojii on #django-cms about this.
I can't help but feel that :

removing language codes from urls is a popular idea. google will reveal this to you.
The documentation on achieving this is inadequate. Neither of the two related settings properties are mentioned as having affect on this particular operation (or in no certain terms).

The easiest and simplest thing to do ( since modifying these variables causes djangocms to break and not plain django), simply :

provide documentation on how to remove language codes from urls, which involves
mentioning that LANGUAGE_CODE is very important.
ensuring CMS_FALLBACK_LANGUAGE is correct.

Had there been documentation that addressed this operation, I doubt I would have created this ticket in the first place.

I can fully understand that, and I'm never going to disagree about more docs.
I suppose that part of the problem is the more familiar you get, the less easy it is to write, and consider, all the pitfalls that might come up in a way that works for everyone. I don't think that's unique to django CMS, nor even Django. For example, the first docs I can find on the language code barely cover the required format, relying presumably on the comment in the generated settings.py
Do you have any insights or feedback into where it'd be best to insert these extra docs (which should probably link to both of the above, if they don't already), based on where you've been looking?
I'm thinking it should be explained further in the configuration docs, and should probably be in the table of contents for internationalisation; beyond that, not sure.

to be honest, the docs need to be fixed upstream in Django.

I experienced the same problem as outlined in this issue. In LANGUAGES I have 1 language: ('nl', 'Dutch'), but my LANGUAGE_CODE was still at its default value of 'en-gb'. Once I changed LANGUAGE_CODE to 'nl-nl', all was working again.


Feature Request - Ability to specify the position a new column is created in a table

There are times when a new column needs to be added to a table, but there is currently no way to tell Fizz in what order to place that new column.  Mysql allows you to specify a BEFORE or AFTER command followed by the target column.

I started working on this feature here: https://github.com/gobuffalo/pop/pull/6

Covered by #6.


chore: Update website

/rebase

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

v5.5.4
GitHub release

Your semantic-release bot :package::rocket:


Switch from Travis to GitHub Actions

Should be faster and also allows to automate more tasks (i.e. releases, snapshot deployment, changelog generation).
Also travis-ci.org is no longer working:

Since June 15th, 2021, the building on travis-ci.org is ceased. Please use travis-ci.com from now on.

(https://travis-ci.org/github/Adobe-Consulting-Services/acs-aem-commons)

@davidjgonzalez After the merge there are still two issues:

The build image is broken. Should be fixed by https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/2621
The analysis does not work with coveralls, the build fails with Report submission to Coveralls API failed with HTTP status 422: Unprocessable Entity (Couldn't find a repository matching this job.) (e.g. in https://github.com/Adobe-Consulting-Services/acs-aem-commons/runs/2933312168?check_suite_focus=true). This indicates the parametrization for coveralls-maven-plugin is still wrong (https://github.com/lemurheavy/coveralls-public/issues/1435)

Now build is back to green after merging the remaining two PRs.


🔍New Meta-description for home page

Cc: @bradystroud

Pain
The home page doesn't have any logic to display a unique meta description, so it currently uses the placeholder.
Suggested Solution
Add custom logic in to display a unique meta description.
Tasks

[x] Add custom logic in to display a unique meta description
[x] Add a new meta description
[ ] Get the new meta description approve by marketing

@bradystroud
Done - Here's the new meta description


AppImage fails with fSpy

Describe the bug
I tried to run the appimage provided by fSpy. While it works on xubuntu, it fails on Nixos: appimage-run gives an error, and pkgs.appimageTools.wrapType{1,2} fails with a segfault.
To Reproduce
Steps to reproduce the behavior:

Download fSpy here https://github.com/stuffmatic/fSpy/releases/tag/v1.0.3
install appimage-run
Run the following commands:

$ file -k fspy-1.0.3-x86_64.AppImage
fspy-1.0.3-x86_64.AppImage: ELF 64-bit LSB executable, x86-64, version 1 (SYSV)\012-  (Lepton 3.x), scale 232-60668, spot sensor temperature -4.187500, color scheme 15, show scale bar, calibration: offset -0.000000, slope 0.000000\012-  (Lepton 2.x), scale 4111-45000, spot sensor temperature<PHONE_NUMBER>00, color scheme 3, minimum point enabled, calibration: offset -75402534979642766821519867692934234112.000000, slope<PHONE_NUMBER>733706829839455140374904832.000000, dynamically linked, interpreter /lib64/ld-linux-x86-64.so.2, for GNU/Linux 2.6.18, BuildID[sha1]=79dcc4e55a61c293c5e19edbd8d65b202842579f, stripped\012- data

$ appimage-run fspy-1.0.3-x86_64.AppImage
Uncompress fspy-1.0.3-x86_64.AppImage of type 02 @ offset 0
Can't find a SQUASHFS superblock on /home/leo/Documents/Cours/2019-2022_-_These/Work_in_progress/2020_11_-_Asiacrypt_Video/Blender_tools/fspy-1.0.3-x86_64.AppImage
chmod: cannot access '/home/leo/.cache/appimage-run/2feaac484c6e124cd99239d5fc9eabad9dc15ea59077e21fb0de9ade4bd01770': No such file or directory
fspy-1.0.3-x86_64.AppImage is now installed in /home/leo/.cache/appimage-run/2feaac484c6e124cd99239d5fc9eabad9dc15ea59077e21fb0de9ade4bd01770
/usr/bin/appimage-exec.sh: line 74: cd: /home/leo/.cache/appimage-run/2feaac484c6e124cd99239d5fc9eabad9dc15ea59077e21fb0de9ade4bd01770: No such file or directory

$ cat default.nix
{ pkgs ? import <nixpkgs> {} }:
pkgs.appimageTools.wrapType1 { # also tried with wrapType2
  name = "fSpy";
  src = ./fspy-1.0.3-x86_64.AppImage;
  extraPkgs = pkgs: with pkgs; [ ];
}

$ nix-build

$ result/bin/fSpy
/nix/store/3ggh2w2921gbipji8wrxnywabqbns0w6-fSpy-extracted/AppRun : ligne 102 :     3 Segmentation fault      (core dumped)pidof appimaged 2> /dev/null

Notify maintainers

@bignaux @Mic92 @prusnak

appimage-run from unstable works for me, can you please confirm @tobiasBora ?

So on unstable, appimage-run gives me a result close to the one of  wrapType: it segfaults, but continues to run after and gives a few warnings, and then nothing happens but I don't get access to my prompt until I Ctrl-C, "something" is still running in the background I guess.
$ sudo nix-channel --update
$ nix-env -iA nixos-unstable.appimage-run
$ appimage-run fspy-1.0.3-x86_64.AppImage
/home/me/.cache/appimage-run/2feaac484c6e124cd99239d5fc9eabad9dc15ea59077e21fb0de9ade4bd01770/squashfs-root/AppRun : ligne 102 :  8991 Segmentation fault      (core dumped)pidof appimaged 2> /dev/null
Fontconfig warning: "/etc/fonts/2.11/fonts.conf", line 5: unknown element "description"
Fontconfig error: "/etc/fonts/2.11/fonts.conf", line 5: invalid attribute 'domain'
Fontconfig warning: "/etc/fonts/conf.d/10-hinting-slight.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/10-scale-bitmap-fonts.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/20-unhint-small-vera.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/30-metric-aliases.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/40-nonlatin.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/45-generic.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/45-latin.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/49-sansserif.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/50-user.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/51-local.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/60-generic.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/60-latin.conf", line 4: unknown element "description"
Fontconfig warning: "/etc/fonts/conf.d/65-nonlatin.conf", line 4: unknown element "description"
Fontconfig error: Cannot load config file from /etc/fonts/2.11/fonts.conf
^C

Can I give any special command? if it helps, the line 102 of the AppRun file is:
pidof appimaged 2>/dev/null && exit 0

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/cant-run-an-appimage-app/10443/1

This problem seems to be solved with latest unstable... No idea what changed, but great! (I also don't have any many symbol lookup error, so they may actually be linked) So I'll close that issue!


perf: use 2 as first non-qr candidate

1 is a quad residue no matter the field, so we can just skip it when we look for our first non-residue for sqrt. This saves us a call to legendre_symbol, giving a tiny but non-negligible boost.
Type of change

[x] Optimization

Checklist

[x] This change is an Optimization

[x] Benchmarks added/run

Results:
Stark FP operations/sqrt squared
                        time:   [347.54 µs 347.98 µs 348.45 µs]
                        change: [-4.5765% -4.4747% -4.3736%] (p = 0.00 < 0.05)
                        Performance has improved.

Codecov Report

Merging #400 (dbb616c) into main (d1b853c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #400   +/-   ##
=======================================
  Coverage   93.75%   93.75%
=======================================
  Files          61       61
  Lines        7556     7556
=======================================
  Hits         7084     7084
  Misses        472      472

Impacted Files
Coverage Δ

math/src/field/element.rs
94.41% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How do I install Db2 Warehouse Operator Extension

Update readme to include information on how to install Db2 Operator so that you can create a database within CP4D.

For CP4D 3.5 this should be done through the cpdcli instead of an operator.

This has been resolved.


[WIP] feat: chrome dev tools debugger first version

This adds a new folder, extras, that allows some inspection of actions and data that the kernel receives.
It needs a lot of work on the front-end, and for sure could be a little prettier, but for a first functional version I believe it's pretty good.

TODO to merge

[ ] TypeScript
[ ] apply standards
[ ] Any following issues to keep going with this? (GitHub issues, TODO task, instructions to make it work at preview entrypoint)

Closing for the timebeing to avoid clutter in the pull requests. Will reopen later.


Buy/SellNotifications not working on Poloniex

The bot will happily report hourly balances but i could not convince it to msg me on trades on poloniex.
Not sure what logs to provide, here is my config:
  "General": {
    "DatabaseLocation": "database\\cryptogrambot.sqlite"
  },
  "Coinigy": {
    "Enabled": "true",
    "SendHourlyUpdates": "false",
    "Key": "xxx",
    "Secret": "xxx",
    "Endpoint": "https://api.coinigy.com/api/v1/"
  },
  "Telegram": {
    "BotToken": "xxx",
    "ChatId": "xxx"
  },
  "Bittrex": {
    "Enabled": "false",
    "Key": "",
    "Secret": "",
    "SendHourlyUpdates": "false",
    "SellNotifications": "false",
    "BuyNotifications": "false"
  },
  "Poloniex": {
    "Enabled": "true",
    "Key": "xxx",
    "Secret": "xxx",
    "SendHourlyUpdates": "false",
    "SellNotifications": "true",
    "BuyNotifications": "true"
  },
  "BagManagement": {
    "Enabled": "false",
    "PercentageDrop": "30"
  },
  "DustNotification": {
    "Enabled": "true",
    "BtcAmount": "0.0005"
  }
}```

Your config file looks ok except for ``` at the end of the file. The config is dynamically converted into C# objects so any small typo can throw the whole thing.
If you are still not getting any trad enotifications, attach your log file. This is found in the log folder of your installation.
Thanks

Same thing for me. But with normal cfg.
Ubuntu 16.04
CryptoGramBot-20171002.log

Your config file looks ok except for ``` at the end of the file.
This was a markdown mistake here on github, the config is okay.

I have attached my logs here:
CryptoGramBot-20171002.zip
I would like to point your attention to these lines:
2017-10-02 00:27:33.035 +02:00 [Error] Error in getting new orders from poloniex
The remote server returned an error: (422) .

I think this is a known issue with the polo API.  After this it seems the bot does not try again.

I just realized something else: The box my TG-Bot runs from is equipped with Elroys proxybot. Hence all requests for poloniex.com are handel through the proxy. Gunbot or Sjakil's GUI had to be started with NODE_TLS_REJECT_UNAUTHORIZED=0 ./gunthy-gui-linux-x64 to not complain about the SSL Cert error. Maybe this is of relevance here?

I think the 422 error is a red herring. The bot does try again as seem in these lines:
2017-10-02 07:42:31.422 +02:00 [Information] Adding new trades to database
The bug is either that polo is returning no trades or the code thinks that none of the trades are new.

With Version 0.0.139 i am now receiving buy messages from poloniex. Havent had a sale since but @T1aMat reported earlier that its working for him. I assume this issue is solved.

Glad to hear. Please comment on here if it is not.


Allow custom yield- blocks to occur after they're declared

For layouts that place the
{{/yield-scripts}}

Does this not work for you? We do this, and it works fine.


Add unit tests to verify parsing and normalisation of vietnamese names

Adds tests to verify issue #17

Coverage remained the same at 100.0% when pulling 1e791ccb78124c9f313b9be56abf8d0108d03ffc on feature/add-tests-for-vietnamese-names into dbffe730837f98d067900df6e331e11bcd92f982 on master.


Issue when using volume mounts.

When mounting a volume on /etc the container won't launch:
postconf: fatal: open /etc/postfix/main.cf
Shouldn't these be copied into the volume by the scripts on launch?

I'll be suppressed if the container could launch in this case.
The /etc directory contains all the system configuration files, and it's very coupling with the specific system, such as the Alpine Linux which used by SMF.
Why do you want to mount a volume on /etc?

Because I want to keep the data even when recreating the container (for example when I add an alias).
Why is it even registered as a volume in the Dockerfile if I'm not supposed to ever mount it? :)

I understand your requirement of adding an alias.
Could this be done by set SMF_CONFIG? If it could, then you can just restart the container, without any care of the old container /etc data, because SMF_CONFIG will set it for you again.
And sorry for the confusing, I just had a check and found this VOLUME was introduced here:
https://github.com/zixia/docker-simple-mail-forwarder/commit/2c97ba1307c00113fd8f5cdaff5885297f778b34#diff-3254677a7917c6c01f55212f86c57fbfR55
And I totally forgot why I add this. Maybe should get rid of it later.

If you remove it then that's fine with me; the requirement of adding an alias is irrelevant for the volume. It might just have been confusion since I saw it in the Dockerfile.


Status change uses default which does not handle well any new status enums

Adds handling of kQClientStatusClientPendigServerSetup in connect and status change callback.

Typo for "pending" but otherwise looks good