test: fix 'incompatible pointer types' warnings

Refs: https://github.com/libuv/libuv/pull/2686

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/2110/. None of the failures are related to the test changed in this PR.

CI: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/2110/. None of the failures are related to the test changed in this PR.

Landed in https://github.com/libuv/libuv/commit/9c3d692b3941a2a4171629fb52af2e1029c415e8. Thanks for the reviews.

Landed in https://github.com/libuv/libuv/commit/9c3d692b3941a2a4171629fb52af2e1029c415e8. Thanks for the reviews.


Sending ONE from 1Wallet

15000ms fail, transaction won't finalize, transaction successful without ONE being sent to Harmony Wallet. When I go to send and hit max, a zero is entered even with Harmony in the 1wallet. Can't send funds.

Were you hitting daily limit? Also it would help if you post a transaction / explorer link.

I believe that was the issue. Later last night I was able to send and it worked perfectly.

On Jan 10, 2022, at 9:09 AM, Aaron Li @.*** @.***>> wrote:
Were you hitting daily limit? Also it would help if you post a transaction / explorer link.
—
Reply to this email directly, view it on GitHub https://github.com/polymorpher/one-wallet/issues/239#issuecomment-1008908044, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXHECAHVPAN5BL6SK5MYBVDUVLSDDANCNFSM5LSCSXXA.
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 modified the open/close state.

Yeah, I think that would have cleared up the confusion, but I clicked on a few tabs and saw the 1,000/daily limit and figured I just needed to wait. I appreciate getting back to me and love the new wallet. Works so fast and really straight forward.

On Jan 10, 2022, at 9:16 AM, Aaron Li @.***> wrote:
Okay. We are working on something to make daily limit displayed more promptly, and showing proper errors when you hit daily limits
—
Reply to this email directly, view it on GitHub https://github.com/polymorpher/one-wallet/issues/239#issuecomment-1008914361, or unsubscribe https://github.com/notifications/unsubscribe-auth/AXHECAE6WD2UZWZEUHTPCZDUVLS3NANCNFSM5LSCSXXA.
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 modified the open/close state.


Allow other options to be passed into execFile args

It could be quite useful to be able to call things like exportToFile() including more options that  can be passed as args on the ultimate execFile() call. For example, the layout engine.

@all-contributors please add @robross0606  for ideas.


Updated uuid generation and clone repo link (#5)

Description

Deploy url was pointing to the fork. Updated that to point to the main repository.
user id generation was being done using a naive 6 digit random number generator. Changed it to use crypto.randomUUID() library instead.

Best Way to Test

Try the Deploy button and verify the repository is being cloned from the main repo and not the fork.
Run the application and check edgeworker logs. The user id should be uuid instead of a 6 digit integer id.

Type of Change

[x] Bug fix (non-breaking change that fixes an issue)
[  ] New Example
[x] New feature in existing example

New Example Checklist

[x ] 🚀 Link to deployment URL on Vercel
[x] 📱 Is it responsive? Are mobile and tablets considered?
[x] 📚 README.md preview link in PR description (branch)
[x] ⚙ Secrets have instructions for how to set up in the readme
[  ] 🛫 npm run new-example was run to create the example

@lfades I have incorporated the changes you asked for. It will be awesome if you can check and merge this one.


Add Banner to README file

I added the banner to the README file with a title with align to the center.
Also fixed a broken error for the website of Python Canarias.

The original work is also in PNG, but if you want I can add the banner in the original size to the docs repository.

Gracias @cesaralvrz por la colaboración altruista y gracias @jdevera por revisarlo y mergearlo 😄


Variadic GCD and LCM

Tracked by: #275

Implements variadic GCD and LCM. LCM uses the GCD method with accounting for gcd == 0.
TODO:

[ ] Finish documentation
[ ] Test cases
[ ] Account for special cases with Int.min
[ ] Account for 0 cases

In practice, variadic functions for these are going to be less useful than operations on sequences:
@inlinable
public func gcd<T: BinaryInteger>(_ sequence: some Sequence<T>) -> T {
  sequence.reduce(0, gcd(_:_:))
}

@inlinable
public func lcm<T: BinaryInteger>(_ sequence: some Sequence<T>) -> T {
  sequence.reduce(1, lcm(_:_:))
}

Rationale: if you have the sequence version, you can just wrap your values in an array. But if all you have is the variadic version and you have a sequence, you're out of luck.

I did think about that, however I think that the correct place for the gcd/lcm of a Sequence is an extension to match other semantics of "what is the X value in this sequence".
Additionally, that can be easily implemented, as you already do.

That doesn't argue for making them variadics, though. That would argue for making them an extension on Sequence.
I think I disagree: gcd(someValues) reads more naturally to me than someValues.gcd(), even if the latter is more typical for Swift. I'm open to discussing this point, however.

I see their value as variadics and extensions on sequence - Swift has the variadic freestanding for min/max as well as the min on Sequence. I can implement them here as well.

Do we actually want to mirror Swift's structure of the binary and variadic min/max for the freestanding implementations?
ex: have lcm(_:_) and lcm(_:_:_:_:)

The variadic min/max probably don't carry their weight; I don't think that we would add them if we were writing the standard library today, but they date back to the very, very early days of Swift.

I personally do not have a situation where I need to compute the lcm/gcd often enough to make a strong argument for anything. The argument for freestanding variadic could be that I already have my arguments, I don't need to put them into an array first.
An extension on Sequence is only natural to fit the functional paradigm. Frankly, I would see many people making that extension on their own if this package didn't provide it.

Apologies, after my caffeine high I have updated the PR description with the proper reasoning and justifications for this proposal which should have been there to begin with.

What's the reason for not including both variadic and Sequence versions?
Seems like both might be useful (though I'd agree that the free function that takes a Sequence is what I would have reached for).

What's the reason for not including both variadic and Sequence versions?

It's generally desirable for a library to make choices about API, rather than just throwing in everything you think someone might want. You want to have a consistent and familiar way to perform operations when you can. Given the sequence form, it's trivial for users to get the variadic behavior by just making an array at the call site: gcd([a, b, c]), but there's no convenient way to go from the variadic form to an operation on sequences. So the sequence form makes the most sense.
Why provide the binary form, then, since that can also be spelled using the sequence form? Two reasons: first, the overwhelming majority of use is for exactly two values, so it's justifiable to carve out a specific binding for that. Second, you need to have the binary form to implement the sequence API anyway, and simply making it public doesn't add any implementation or maintenance burden.

That makes sense, but isn’t the Variadic form strictly more useful than the binary form? I’d take Variadic over binary as an API consumer.
This is probably a lot of handwringing over a tiny number of use cases, but if the API surface area is a concern Variadic and Sequence-oriented implementations could be public, and the binary form could be a fileprivate implementation detail.

In practice, variadic functions for these are going to be less useful than operations on sequences:

I do agree. A more elaborate algorithm will almost always have to resort to a function taking a sequence rather than a function taking variadic arguments.

I did think about that, however I think that the correct place for the gcd/lcm of a Sequence is an extension to match other semantics of "what is the X value in this sequence".

Making gcd/lcm instance functions via an extension on Sequence (e.g. values.gcd()) reads very unnatural to me. If they should not be part of the global namespace (which is probably a good thing), I would rather prefer type functions, which also matches what has been done with ElementaryFunctions (and others) on Reals (i.e. making the function static and calling them like so: .gcd(values)) – but this diverges from the freestanding binary gcd, so we probably do not want to do this either and just make both freestanding.

This is probably a lot of handwringing over a tiny number of use cases, but if the API surface area is a concern Variadic and Sequence-oriented implementations could be public, and the binary form could be a fileprivate implementation detail.

I can not think of a use case for a variadic function of gcd/lcm that passes less than 2 arguments. If we were to limit the argument count to >= 2, we could merge the binary function and the variadic function like so (or similar):
public func gcd<T: BinaryInteger>(_ a: T, _ b: T, _ n: T...) -> T {
  if n.isEmpty {
    return _gcd(a, b)
  }
  return n.reduce(_gcd(a, b), _gcd(_:_:))
}

This yields us both, a function that takes 2 or any number of arguments. It does not increase the API surface and it's not a source breaking change. Also, it would not be a performance regression for the binary function if the optimiser is able to replace it with the underlying binary function in cases where only 2 arguments are given – although I don't expect it is much of a performance hit otherwise.

In practice, variadic functions for these are going to be less useful than operations on sequences:

I do also agree, however I reiterate that variadic parameters are targeted towards an API implementation allowance where a user may want to use n arguments without making an array/collection. It's nothing more than something nice-to-have as a user and I personally don't see a reason to exclude it just because there can also be a sequence version.
As an API design, I find a variadic form robust in usage and description. In fact, the cpp lcm documentation example uses the binary form to make a variadic form. Now obviously just because something is easy to implement and whether it belongs here has already been said and is an ongoing argument in any API design.

Making gcd/lcm instance functions via an extension on Sequence (e.g. values.gcd()) reads very unnatural to me.

I argue the opposite, but this is very strongly a design choice in functional programming. Swift allows very tacit function composition via type extensions and I am confident in saying that every Swift developer has used the style - either consciously or unknowingly. I could somewhat dive into this into detail, however that's blog post material.

If they should not be part of the global namespace (which is probably a good thing)

These functions are already namespaced by modules. I don't think that it's a good thing to differ from many other language's math/numerics packages that offer these types of functions as freestanding. I don't see namespace crowding as an issue here and it is consistent with other language packages.

I can not think of a use case for a variadic function of gcd/lcm that passes less than 2 arguments.

I am personally fine with this for a developer usage reason and tool-autocomplete reason. If a developer is explicitly using it for one argument, I don't see a reason to disallow it - that's just odd. Additionally, when using autocomplete Xcode (at least) will also add a placeholder for the variadic argument - almost implying that it's required but it could be removed by manual deletion. In instances where auto-complete isn't available, then the developer is very much in a context where they know how many variables they would need to evaluate a function with.
I can also see how my argument is contradictory to avoiding the full-variadic argument. However I'm fine with that contradiction, it wouldn't allow something even more weird and autocomplete tools already guide the developer towards a >= 2 argument context.

At the same time, I'm not arguing for variadic-exclusivity with a sequence argument equivalent. I can actually see uses of all three styles depending on developer context - and there's very little overhead. Obviously we should be conscious that we are making decisions that would impact all future numerical function implementations and I have left some of my thoughts on the PR description.
Specifically for the sequence version, I would lean more towards these functions having named parameters so that the call site reads more easily. I argue that we see this more in standard library functions (not just to avoid conflicts, but also as an API detail) and that this wouldn't diverge from other language implementations (because I wasn't able to find one that implemented an array/collection version).
So in short, I think we should offer all three and I don't think that it introduces much overhead at all to us or users.
lcm(a, b, c)
lcm(of: foo)
bar.lcm()


startPrank does not correctly mock msg.sender when calling a public library from a script

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 (6157d4a 2023-02-03T00:05:03.034611Z)
What command(s) is the bug in?
forge script
Operating System
macOS (Apple Silicon)
Describe the bug
I've created a reproduction here: https://github.com/Kooshaba/forge_bug_repro
It uses an ownable storage contract called by a library from a script. There are two functions in the library: one public, one internal. When using startPrank and calling the storage contract via the public method, the msg.sender is the address of the library. When using the internal method, it correctly mocks the msg.sender to be the pranked address.

bump this is still ruining my life

This is not exclusive to scripts: calling a library's public function will invoke a delegatecall which sets the msg.sender to the test contract's address. I believe this is expected? cc @mattsse

I'm not really familiar with how prank is supposed to behave here
cc @mds1

Expanding on what @DaniPopes said:

The internal method works because the code gets inlined, so it's as if that logic lived directly in your script. As a result, the prank takes place on the _storage.set call
The public method delegatecalls to the function. While this should have the same effect in solidity since it preserves context, I believe this doesn't work with prank because delegatecalls change the depth of execution, so the _storage.set call happens at a different depth, and as a result the prank does not occur in the delegatecall contract.

I'd recommend always using internal library functions unless you have a reason to use public (i.e. deploying a contract to production, using public library methods can shrink code size). But for tests/scripts, you can always use internal

Marking as resolved by https://github.com/foundry-rs/foundry/issues/4266#issuecomment-1500284318 as it is expected behavior


C# 7.x: Unifying treatment of discards and local variables in patterns, deconstruction and out variables

Pattern, deconstruction expressions and out variables all allow for local variables to be declared in places that are not accounted for by existing scope rules. In addition, those locations allow for the use of discards in lieu of existing or fresh variables.
Currently the treatment of these variables is spread out and partially duplicated across #61, #63 and #44. I think there's both an opportunity and a need for unifying this treatment, quite possibly introducing new concepts/terms in the process.
Perhaps the best approach is to treat this as a separate "feature" with its own PR, which the above PRs can then build on. The "feature" introduces new concepts (e.g. discards), scope rules etc. but does not in and of itself surface them in syntax.

As Mads is otherwise occupied for a while, and I have the time, I’ve been working on this issue.
I’m starting with discards, and then will look at local variable declarations.
Thus far

I’ve created (but not yet posted) draft text for a new PR that introduces discards.
I’ve figured out the changes needed for PR #44 (out variables) to point to that new PR, and I’ve found (unrelated) enhancements and corrections for PR 44.

Next, I need to

Reconcile PR #61 (patterns) with the new discard PR, tweaking the discard PR, as necessary.
Reconcile PR #63 (tuples) with the new discard PR, tweaking the discard PR, as necessary.

My thinking is to then

Create the new discard PR and add commits to the 3 existing PRs for any discard-related edits.
Figure out how to unify the declaration of local variables in out variable arguments, patterns, and tuples, with existing local-variable usage.
Add commits to the 3 existing PRs for any local-variable-related edits.

I've created PR 596 to add discards as a separate feature.
I've revised PR 44, as follows:

removed discard spec text
removed handling of discards on lhs of simple assignment
replaced uses of variable_reference with identifier to accommodate discards as well as variable names
simplified and expanded the out-argument examples

I've completed the discard spec part. After looking at the local-variable-creation aspect, I have the following to report:
[The use of not_ indicates "any identifier except _".]
Out Variable
Type is optional
argument_value
   : 'out' local_variable_type? identifier
   ;

local_variable_type
    : type
    | 'var'
    ;

If type/var is present, _ is a typed discard; not_ is a new local variable, provided none by that name already exists (which is an error).
If type/var is absent, _ is an existing local variable if one by that name is in-scope; otherwise, _ is an untyped discard. Either way, not_ must be an existing local variable.
Pattern
Type is required
declaration_pattern
 : type identifier
 ;

var_pattern
: 'var' identifier
;

_ is a typed discard; not_ is a new local variable. identifier is never bound to an existing local variable.
Tuple Deconstruction
Type is optional
destination
 : local_variable_type? identifier
 ;

If type/var is present, _ is a typed discard; not_ is a new local variable, provided none by that name already exists (which is an error).
If type/var is absent, _ is an existing local variable if one by that name is in-scope; otherwise, _ is an untyped discard. Either way, not_ must be an existing local variable.
Commentary
As shown in the descriptions above, the “out variable” and “tuple deconstruction” cases have equivalent syntax and the same semantics, but the “pattern” case is different.
The syntax for local variables is defined in §12.6, “Local variable declarations.” However, that context also allows for the declaration of multiple identifiers as well as initializers, things not allowed in the cases above. And in §12.9.5, “The foreach statement,” we also have custom prose, as only one identifier and no initializer is allowed there. As such, I don’t see any opportunities to leverage further on existing uses of local variable declarations.
@MadsTorgersen, are you able to take ownership of this again?

Thanks so much for the work you've done on this, @RexJaeschke! I've blocked time on my calendar to try to bring it all together before the November meeting.

Fixed in #664


webapp: Cannot deploy by any means to my WebApp, not using FTP, not using az CLI, not in VS

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'NoneType' object has no attribute 'reserved'
Traceback (most recent call last):
python3.8/site-packages/knack/cli.py, ln 215, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 631, in execute
    raise ex
cli/core/commands/__init__.py, ln 695, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 686, in _run_job
    cmd_copy.exception_handler(ex)
...
cli/core/commands/__init__.py, ln 324, in __call__
    return self.handler(*args, **kwargs)
azure/cli/core/__init__.py, ln 574, in default_command_handler
    return op(**command_args)
cli/command_modules/appservice/custom.py, ln 3215, in webapp_up
    current_os = 'Linux' if plan_info.reserved else 'Windows'
AttributeError: 'NoneType' object has no attribute 'reserved'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up -g {} -n {} --html

Expected Behavior
Environment Summary
macOS-10.14.6-x86_64-i386-64bit
Python 3.8.2
Installer: HOMEBREW

azure-cli 2.5.1 *

Additional Context

webapp

So, what is happening with this? Did it go straight to trash can? @AzureAppServiceCLI @antcp

@nordquist as per the title the deployment issue is not specific to CLI - are you still seeing this issue.
Few things to consider

When using CLI please upgrade to the newer version from the current issue - seems like you are using a  very old version
App service currently cannot mix windows and linux apps in the same RG, so if you are trying to use the RG that has windows apps you can use it for linux apps & vice versa.

in general az webapp up should take care of #2, if you don't pass a -g in the command but I definitely recommend upgrading to the latest version of azure-cli which has lots of fixes to az webapp up in general.

#14287

It has nothing to do with the version of the CLI as implicated by the fact that there is no way that works. This is the latest message from VS when trying to deploy an Azure Function (..and yes, I have tried setting WEBSITE_RUN_FROM_PACKAGE to 0 (and why this is not default when it never works?)).

The "detailed" error is:
2020-07-10 11:44:55
System.AggregateException: One or more errors occurred. ---> System.Exception: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.
   --- End of inner exception stack trace ---
---> (Inner Exception #0) System.Exception: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details. <---

System.Exception: Publish has encountered an error. We were unable to determine the cause of the error. Check the output log for more details.

===================

@nordquist given this is CLI I am going to focus on getting this working with CLI -> and if using az webapp up, my recommendation would be to use a more recent version, and run the up command from the context of the DIR where you code to be deployed is. and try creating & deploying a new webapp using webapp up. If this is a function app then this command is not supported for it.
If your question is about run as package deployment then please use this repo to https://github.com/projectkudu/kudu. Hope this helps you.

closing due to lack of response.


incrementLookupHistory is called again when navigating to a list item despite being guarded by !recordedHistory

This causes history to be incremented 2 times
If the user navigates back, then clicks on another list item, it will not be incremented again

Fixed in latest release of Circuit


Job manager implementation

This is the second out of three main PRs. It covers the Job Management and is the first version that is fully usable. The third main PR will handle the resource limits / process isolation.

Minor precision on the design doc around stop job
Broadcaster
All endpoints fully implemented, including logs from the beginning and stop causing a different status
New target make build/debug to compile binaries with the race detector and symbol, and dump goroutines/memory info periodically.
Tests for GetJobStatus with various scenario, happy/sad path.

I seeing both races and flakiness running the tests in a container:
make mtls
docker run -it --rm -v"$PWD:/workdir" -w'/workdir' golang:1.23

# inside container
go test ./... -v -race -shuffle=on -count=10

The non-blocking broadcaster was causing issues in the tests, to fix it, I followed @codingllama initial suggestion and moved the output to the broadcaster.
Tests now 100% stable


Customer emails have no new lines

Customer emails sent have no new lines so everything is on one line.
Displays in Thunderbird as:
2point4 ------------------------------------------------------ Order Number: 1 Detailed Invoice: https://www.somewebsite.com.au/2point4/account_history_info.php?order_id=1 Date Ordered: Tuesday 15 November, 2016 The comments for your order are Your order has been updated to the following status. New status: Processing Please reply to this email if you have any questions.
Raw text of email:
Return-Path<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: from somewebsite.com
by CentOS6-VPS (Dovecot) with LMTP id SgWeMOZkKlgpUwAAhd2+mg
for<EMAIL_ADDRESS>Tue, 15 Nov 2016 12:29:10 +1100
Return-path<EMAIL_ADDRESS>Envelope-to<EMAIL_ADDRESS>Delivery-date: Tue, 15 Nov 2016 12:29:10 +1100
Received: from somewebsite by someotherwebsite.com with local (Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1c6SYg-0005XG-M4
for<EMAIL_ADDRESS>Tue, 15 Nov 2016 12:29:10 +1100
To: "Frank Lastname"<EMAIL_ADDRESS>Subject: Order Update
X-PHP-Script: www.somewebsite.com/2point4/admin/orders.php for <IP_ADDRESS>
X-Mailer: osCommerce
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=_MULTIPART_MIXED_BOUNDARY"
Content-Transfer-Encoding: 7bit
From: "2point4"<EMAIL_ADDRESS>Message-Id<EMAIL_ADDRESS>Date: Tue, 15 Nov 2016 12:29:10 +1100
This is a multi-part message in MIME format.
--=_MULTIPART_MIXED_BOUNDARY
Content-Type: multipart/alternative; boundary="=_MULTIPART_ALTERNATIVE_BOUNDARY";
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4
Order Number: 1
Detailed Invoice: https://www.somewebsite.com/2point4/account_history_info.php?order_id=1
Date Ordered: Tuesday 15 November, 2016
The comments for your order are
Your order has been updated to the following status.
New status: Processing
Please reply to this email if you have any questions.
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: multipart/related; boundary="=_MULTIPART_RELATED_BOUNDARY"
--=_MULTIPART_RELATED_BOUNDARY
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4
Order Number: 1
Detailed Invoice: https://www.somewebsite.com/2point4/account_history_info.php?order_id=1
Date Ordered: Tuesday 15 November, 2016
The comments for your order are
Your order has been updated to the following status.
New status: Processing
Please reply to this email if you have any questions.
--=_MULTIPART_RELATED_BOUNDARY--
--=_MULTIPART_ALTERNATIVE_BOUNDARY--
--=_MULTIPART_MIXED_BOUNDARY--

Thanks!
html part of the Multipart Boundary is not html. There should be HTML coded text (text/html)
Checkout_process generate plain text and no html formatted codes for emails. So there are no  tags only "\n" in line feeds which will never break lines in html context.
Previous versions of oscommerce not folowed the right boundary standards so it was possible to inject into the mail body directly formatted codes.
Fixes need in core before email sendings.

Could you test it?
https://github.com/Gergely/oscommerce2/commit/13a7084005aad5ec1811e628caa0b9b4706075d2

This is the output with your change
Return-Path<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: from somewebsite.com
by someotherwebsite.com.au (Dovecot) with LMTP id yUJ6B7vkLFh5JwAAPuX4MA
for<EMAIL_ADDRESS>Thu, 17 Nov 2016 09:59:07 +1100
Return-path<EMAIL_ADDRESS>Envelope-to<EMAIL_ADDRESS>Delivery-date: Thu, 17 Nov 2016 09:59:07 +1100
Received: from top by somewebsite.com with local (Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1c79AZ-0002jF-0R
for<EMAIL_ADDRESS>Thu, 17 Nov 2016 09:59:07 +1100
To: "Frank L"<EMAIL_ADDRESS>Subject: Order Update
X-PHP-Script: www.somewebsite.com/2point4/admin/orders.php for <IP_ADDRESS>
X-Mailer: osCommerce
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=_MULTIPART_MIXED_BOUNDARY"
Content-Transfer-Encoding: 7bit
From: "2point4"<EMAIL_ADDRESS>Message-Id<EMAIL_ADDRESS>Date: Thu, 17 Nov 2016 09:59:07 +1100
This is a multi-part message in MIME format.
--=_MULTIPART_MIXED_BOUNDARY
Content-Type: multipart/alternative; boundary="=_MULTIPART_ALTERNATIVE_BOUNDARY";
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4------------------------------------------------------Order Number: 3Detailed Invoice: https://www.somewebsite.com/2point4/account_history_info.php?order_id=3Date Ordered: Thursday 17 November, 2016The comments for your order areYour order has been updated to the following status.New status: ProcessingPlease reply to this email if you have any questions.
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: multipart/related; boundary="=_MULTIPART_RELATED_BOUNDARY"
--=_MULTIPART_RELATED_BOUNDARY
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4------------------------------------------------------Order Number: 3Detailed Invoice: https://www.somewebsite.com/2point4/account_history_info.php?order_id=3Date Ordered: Thursday 17 November, 2016The comments for your order areYour order has been updated to the following status.New status: ProcessingPlease reply to this email if you have any questions.
--=_MULTIPART_RELATED_BOUNDARY--
--=_MULTIPART_ALTERNATIVE_BOUNDARY--
--=_MULTIPART_MIXED_BOUNDARY--

Could you copy/paste with "insert code"? (I used html insert code formula) I dont see

Its working for me but as Harald said plan text line breaks "\n" and html breaks tag from language text defs results in duplicated line breaks.
This is a multi-part message in MIME format.

--=_____MULTIPART_MIXED_BOUNDARY____
Content-Type: multipart/alternative; boundary="=_____MULTIPART_ALTERNATIVE_BOUNDARY____";

--=_____MULTIPART_ALTERNATIVE_BOUNDARY____
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

Oscommerce v2.4 project------------------------------------------------------Order Number: 13Detailed Invoice: http://localhost/projects/oscom24_g/catalog/account_history_info.php?order_id=13Date Ordered: Wednesday 16 November, 2016The comments for your order aresdfgsdf
sdfgsdfgs
sdfgsdfg
gYour order has been updated to the following status.New status: PendingPlease reply to this email if you have any questions.

--=_____MULTIPART_ALTERNATIVE_BOUNDARY____
Content-Type: multipart/related; boundary="=_____MULTIPART_RELATED_BOUNDARY____"

--=_____MULTIPART_RELATED_BOUNDARY____
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit

Oscommerce v2.4 project
------------------------------------------------------
Order Number: 13
Detailed Invoice: http://localhost/projects/oscom24_g/catalog/account_history_info.php?order_id=13
Date Ordered: Wednesday 16 November, 2016

The comments for your order are
sdfgsdf

sdfgsdfgs

sdfgsdfg

g

Your order has been updated to the following status.
New status: Pending
Please reply to this email if you have any questions.

--=_____MULTIPART_RELATED_BOUNDARY____--

--=_____MULTIPART_ALTERNATIVE_BOUNDARY____--

--=_____MULTIPART_MIXED_BOUNDARY____--```

Question is what will be the rule for emails: html or text from language files?

Emails will need to have two versions: plain text and html. Until the admin page is ready to manage emails, both plain text and html versions of an email must exist in the language definition file.
Plain text should be sent as is even if it includes html tags (no strip_tags()).
HTML should be sent as is (no nl2br()).
If only a text version is provided, a HTML version can be formed using nl2br().
If only a HTML version is provided, a plain text version can be formed using strip_tags() and a custom br2nl() function.

Thanks @haraldpdl
Something like this may be help without language definitions
    public function isBodyHTML()
    {
      return ($this->body_html == $this->body_plain);
    }

Hope this helps
`Return-Path<EMAIL_ADDRESS>Delivered-To<EMAIL_ADDRESS>Received: from somewebsite.com
by someotherwebsite.com (Dovecot) with LMTP id yUJ6B7vkLFh5JwAAPuX4MA
for<EMAIL_ADDRESS>Thu, 17 Nov 2016 09:59:07 +1100
Return-path<EMAIL_ADDRESS>Envelope-to<EMAIL_ADDRESS>Delivery-date: Thu, 17 Nov 2016 09:59:07 +1100
Received: from somewebsite by someotherwebsite with local (Exim 4.87)
(envelope-from<EMAIL_ADDRESS>id 1c79AZ-0002jF-0R
for<EMAIL_ADDRESS>Thu, 17 Nov 2016 09:59:07 +1100
To: "Frank L"<EMAIL_ADDRESS>Subject: Order Update
X-PHP-Script: www.somewebsite.com/2point4/admin/orders.php for <IP_ADDRESS>
X-Mailer: osCommerce
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=_MULTIPART_MIXED_BOUNDARY"
Content-Transfer-Encoding: 7bit
From: "2point4"<EMAIL_ADDRESS>Message-Id<EMAIL_ADDRESS>Date: Thu, 17 Nov 2016 09:59:07 +1100
This is a multi-part message in MIME format.
--=_MULTIPART_MIXED_BOUNDARY
Content-Type: multipart/alternative; boundary="=_MULTIPART_ALTERNATIVE_BOUNDARY";
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4------------------------------------------------------Order Number: 3Detailed Invoice: https://www.somewebsite.com/2point4/account_history_info.php?order_id=3Date Ordered: Thursday 17 November, 2016The comments for your order areYour order has been updated to the following status.New status: ProcessingPlease reply to this email if you have any questions.
--=_MULTIPART_ALTERNATIVE_BOUNDARY
Content-Type: multipart/related; boundary="=_MULTIPART_RELATED_BOUNDARY"
--=_MULTIPART_RELATED_BOUNDARY
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: 7bit
2point4------------------------------------------------------Order Number: 3Detailed Invoice: https://www.somewebsite.com.au/2point4/account_history_info.php?order_id=3Date Ordered: Thursday 17 November, 2016The comments for your order areYour order has been updated to the following status.New status: ProcessingPlease reply to this email if you have any questions.
--=_MULTIPART_RELATED_BOUNDARY--
--=_MULTIPART_ALTERNATIVE_BOUNDARY--
--=_MULTIPART_MIXED_BOUNDARY--
`

@frankludriks
I have figured out a simple html part for checkout
https://github.com/osCommerce/oscommerce2/compare/master...Gergely:issue_499
and normal html parts for account and pasword forgotten as @haraldpdl mentioned before.
Could you test it? Only checkout_process.php is the required file changes for you.

Thanks!


Verify tenant was set on thread local storage

No problems observed:
[INFO] Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.993 s -- in org.folio.rest.camunda.controller.advice.WorkflowControllerAdviceTest
[INFO]
[INFO] Results:
[INFO]
[INFO] Tests run: 364, Failures: 0, Errors: 0, Skipped: 0
[INFO]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS


[UI]  EKS clusters do not have the option to encrypt the EBS volume for /dev/xvda1

What kind of request is this (question/bug/enhancement/feature request):
enhancement
Original issue here https://github.com/rancher/rancher/issues/21552
Steps to reproduce (least amount of steps as possible):
We have added a new EKS driver boolean option to enable EBS encryption on the worker nodes.
Result:
This should be another advanced option on EKS configuration.
Other details that may be helpful:
TBD: Name in API

Available in master

this is getting set in the backend and after creating the EKS cluster if I go to edit I see the EBS encryption checkbox selected but in the AWS console the EBS volume appears as Not Encrypted
The UI part is meeting the expectation. Closing.


First Working Version

Adds a test script shell file
Adds a test command which calls the script
Adds a test job which wraps the command
Add CI to run the command (which like the GitHub variant, is fairly carbon copy of what we'd see in the real repo)

A better readme would be nice....


PPC building of openjdk could be better optimized for pwr5+

Current openjdk compiles ppc xlc platforms using:
$1_CFLAGS_CPU_JVM="-qarch=ppc64"

https://github.com/openjdk/jdk/blob/3bb6a3d2ab96373baa4e18f4e61173f11225fd4e/make/autoconf/flags-cflags.m4#L720
better optimization could be achieved using -qarch=pwr5

And, I think also adding -qtune=pwr8 (although that is the default behavior on POWER8.
The key item -qarch=XYZ changes is the instruction set availability. -qarch=ppc64 goes back to the very VERY first 64-bit processor (POWER3 iirc).

FYI: POWER Linux LE:

This is what we need to 'fix': eliminate messages such as:
14:23:16  /home/jenkins/workspace/build-scripts/jobs/jdk/jdk-aix-ppc64-openj9/workspace/build/src/build/aix-ppc64-server-release/vm/runtime/codert_vm/pnathelp.s: line 11609: Instruction lxvd2x is not implemented in the current assembly mode PPC64.
14:23:16  /home/jenkins/workspace/build-scripts/jobs/jdk/jdk-aix-ppc64-openj9/workspace/build/src/build/aix-ppc64-server-release/vm/runtime/codert_vm/pnathelp.s: line 11611: Instruction lxvd2x is not implemented in the current assembly mode PPC64.
14:23:16  /home/jenkins/workspace/build-scripts/jobs/jdk/jdk-aix-ppc64-openj9/workspace/build/src/build/aix-ppc64-server-release/vm/runtime/codert_vm/pnathelp.s: line 11613: Instruction lxvd2x is not implemented in the current assembly mode PPC64.

Are those errors new in the JDK17 builds only? I'd be a little wary of changing things in previous releases in cae people are using our builds on older systems and we introduce customer problems. Can we check which series of processor lxvd2x was introduced in? We don't currently list a minimum supported level for AIX systems on our supported platforms page/

@andrew-m-leonard If this has been reported upstream to we have a bug reference?

The assembly language instruction is part of the VSX instructions (a co-processor in the POWER core, iirc).
From: http://cdn.openpowerfoundation.org/wp-content/uploads/resources/openpower-isa-thts-V2-1/content/ch08s01.html
VSX Vector Load Instructions: lxvb16x lxvd2x lxv lxvx lxvh8x lxvw4x

So, the code in runtime/codert_vm/pnathelp.s requires, at a minimum, POWER7.
From the XLC compiler reference (PDF): -qarch description and Usage:
Table 22. Feature support in processor architectures
Architecture Graphics
support
Square root
support
64-bit support Vector
processing
support
Large page
support
pwr4 yes yes yes no yes
pwr5 yes yes yes no yes
pwr5x yes yes yes no yes
ppc yes yes yes no yes
ppc64 yes yes yes no yes
ppc64gr yes yes yes no yes
ppc64grsq yes yes yes no yes
ppc64v yes yes yes VMX yes
ppc970 yes yes yes VMX yes
pwr6 yes yes yes VMX yes
pwr6e yes yes yes VMX yes
pwr7 yes yes yes VMX, VSX yes
pwr8 yes yes yes VMX, VSX yes

As picture (I hope):

@andrew-m-leonard - perhaps the title should be changed to pwr7 rather than pwr5x (for pwr5+).


Angular 2 support

Any considerations for migrating this to Angular 2?

I was also wondering if this will be supported in angular 2.

Definitely planning to do an angular2 version when I get around to it, shouldn't take too long.
It's my understanding that angular2 provides an UpgradeAdapter to run angular 1 and 2 code at the same time, which you can use in the meantime.

On further investigation, angular 2 is really a completely new framework rather than just a new version of angular 1. A proper support in angular 2 style would surely involve a fork that also uses TypeScript, for example. I have therefore decided that I won't do a angular 2 fork, as I barely find time to maintain this version.
I'll keep this issue open though, in case anyone creates a fork for angular 2 (or any other framework) and wants to promote it. I'm also happy to link it from the readme.

+1. Nice work. Would be nice to have this for Angular 2. If it comes to that, I hope to be able to contribute. Anyone interested, let's team up.

DId any one find a Angular 2 (now 6) version of this?


Object.keys with fn gives always undefined as value

Example:
Object.keys({'foo': 'bar'}, function(key, value) {
  console.log(key, value);
});

Result:
> foo undefined

Expected:
> foo bar

Ok that sounds reasonable. Fixed that and pushed it just now.

OK, this is now in v1.2!


Wrong logic excluding event codes with just one event in read_raw_egi

Description of the problem
I believe when creating a RawMFF object, the logic for excluding events with just one event is wrong (line 434), it rejects them if event.sum() <= 1, but event.sum() could be more than 1 since it contains the event id (1 to n_events), a more appropriate to do this would be something like np.count_nonzero(event) <= 1
Also, in the documentation for read_raw_egi, which uses RawMFF, it says

The event channels to be ignored when creating the synthetic trigger. Defaults to None. If None, channels that have more than one event and the sync and TREV channels will be ignored.

which I believe is also wrong.
I can submit a PR once this is confirmed.
Related, I think it might be helpful to keep events that only occur once, they could be important tags that indicate start/end of experiments for example, a suitable place might be in the annotations.
Steps to reproduce
Import raw mff files

Link to data
No response
Expected results
All one-shot events excluded
Actual results
Only the first one-shot event (with ID 0) is excluded
Additional information
Draft PR #11627

I also find the warning "Did not find any event code with more than one event." a bit confusing, I'm not sure why these one-shot events are preserved if none of the event codes have more than one event, maybe someone more familiar with behavior could help me understand? cc @ramonapariciog @agramfort


import React from "react" is required to work

Hello. I'm using the project with the Anchor counter implementation. While it's working for me, I'm curious why I have to add import React from "react"; at the top of my files when I add a new page with a new component, even when using a structure exactly like one of the example components. "I tried to follow the steps described in this video from December 2023 (minute 37), and in the video, it's not required, but in my case, it is.
I also notice that I have to do imports like: import DemoFeature from "../../components/demo/demo-feature" instead of @/components/demo/demo-feature like the example files.
Although my code works, I'm just curious why this might be happening.

Hi there @0xNicko - could you share a reproduction of the issue you are facing? You should not have to add the global React import.

Hi, I'm not sure if this is the best way to reproduce the error. But there you can see that I added a new component. The component has the same code as the 'dashboard-feature.tsx' that comes by default. And if I don't import React, it gives me that error.

Please share a repository with your project that I can download and run locally. This way we can find out if there's something wrong in the app or there's an issue local to your system

Hi. I updated Node and NPM to the latest versions, redownloaded the project from my own repository, and it worked. Honestly, I can't figure out what the problem was, but making those adjustments worked. Sorry for creating the issue. Thank you!.

Glad to hear that fixed it! Good luck with your new app! 🥳


Manual UTM Entry for new UI

[ ] User can manually enter UTM for point geo, consider adding radius option.

form geo edit portion of #3205


Enqueue at a time other than now?

The job table has a process_at column that the workers use to filter out jobs that are not yet ready for work. The core function proletarian.job/enqueue! fixes the value that can be saved here to the current moment:
https://github.com/msolli/proletarian/blob/e74e43d7c3710d4906c0ab474ecb59c807a48a83/src/proletarian/job.clj#L50
Proposal:
Could we add proletarian.job/process-at! (a specific time) &/or proletarian.job/process-in! (some time duration). These would work similarly to the same functions in Sidekiq (perform_at and perform_in).
The minimum version of the ability could be for proletarian.job/enqueue! to accept an optional :process-at key, leaving relative time math to the user. If omitted, then enqueue!'s behaviour is unchanged: default to "now".
Example use cases:

deferring the processing of a job by an arbitrary amount of time (eg. email this user in 10 min, just in case there's more we could gang up to send them by then)
simulating periodic jobs: a job could enqueue another instance of itself in the future with the same arguments.

Thanks for the proposal! I like the idea of having proletarian.job/enqueue! accept an optional :process-at key. I think it could also accept :process-in with a java.time.Duration (but not both at the same time).
What do you think should happen when the application enqueues a job with a :process-at time in the past, or a negative :process-in? The Queue Worker logic for fetching jobs to process is such that it takes the oldest job first. If one enqueues a job for processing in the past, it will picked up before any other jobs have already been enqueued.
So, in effect, this opens up for jobs to "skip the line". It could be a nifty feature, on the one hand. On the other, it could lead to degenerate situations where jobs in a queue are never processed because new jobs with :process-at far in the past are enqueued all the time before the workers get to them. We no longer have a "queue", but some other, more chaotic, concept of job processing order.

Please try out this commit: https://github.com/msolli/proletarian/commit/9d3f1d2ffbcfaf3867b9c9d1b506082d617f66ef.
It adds :process-at and :process-in to the options map. The docstring should explain the usage. I've elected to not allow a job to "skip the line" (see previous comment). This is up for discussion, however.

What do you think should happen when the application enqueues a job with a :process-at time in the past, or a negative :process-in? The Queue Worker logic for fetching jobs to process is such that it takes the oldest job first. If one enqueues a job for processing in the past, it will picked up before any other jobs have already been enqueued.

My first guess at the least surprising behaviour would be a sort by (enqueued-at, process-at), where process-at is earlier than now. Imagine two jobs A = (t0, t1) and B = (t1, t-1). It's now time t2. I'd process them A then B. It still makes an intuitive "first-come-first-served" sense.
I suppose the other options are to throw an exception if process-at < now, or clamp it like you have in ->process-at.
Thanks for the commit; I'll give it a go later today.

Released in https://github.com/msolli/proletarian/releases/tag/v1.0.68-alpha


Docs: Demonstrate integration of Hugging Face Datasets

Similar to the lakeFS PR, this PR adds an example of how to interact with lakeFS repositories from the HF Datasets library.
It also adds a mention of the library in the repository README and docs landing page.

Nice!
On a meta-level: I remember that the polars code example broke out of the blue in some newer version. Should we add those examples to an integration test suite?

Good point. Eventually, I'd very much like that - the only problem is that the code examples in the docs currently assume a certain environment (but I guess we can hardcode that assumption for the time being, still better than no tests at all).


SSL termination for kube-stack

Currently kube-stack provides a TCP classic load balancer to kube-stack. Unfortunately, kubespray does not support terminating TLS/SSL itself for external names, meaning we need to add termination somewhere else.
Possible approaches

Change from tcp to an ssl load balancer, and provide ACM issued certificates
Add a secondary HTTPS load-balancer for the domain

First solution has the advantage that it always terminates TLS traffic, with the disadvantage that you need a certificate to even get started. The second solution has additional costs of one more load balancer. I tend to prefer the first approach.

Alternatively, TLS termination has to be done by the cluster internal load balancer deployed with kubespray, and this has to be documented at usage site, not on this repository.

Change is in this commit here on the issue branch.
I would like it to be tested first before merging into master, though. Once this is done, a PR can be issued.

@aniketd can you review this issue and the related updates to confirm if we have already added support for this into kube-stack, or if this update is still relevant. Thanks!

Checked and confirmed that this is true and relevant. Quoting from https://github.com/kubernetes-sigs/kubespray/blob/master/docs/ha-mode.md

Note: TLS/SSL termination for externally accessed API endpoints' will not be covered by Kubespray for that case (the case of using an external LB for API access). Make sure your external LB provides it.


Remove Ping on gestures & X

The PR removes the ping method which showed X whenever an error occurred due to unreachability to the lights/speakers, due to API call latencies, and spat out 3 lines of ping stats everytime a gesture was received. The X isn't a pleasant UI feature at all.

Test and merge in


Deploy Problem 2

Hi, thanks your great work!
Another question about Pytorch-ONNX-TensorRT conversion.
In the forward function of SetAttention, indexing is used at the beginning to retrieve values. However, when deploying similar operations, TensorRT does not support this because the indices are also a changing tensor, rather than fixed indices like x=torch.tensor([1, 2]), x[0]. How did you solve this in your TensorRT engine? Could you please provide some guidance?
https://github.com/Haiyang-W/DSVT/blob/3b825d11e7695d514d795457771508859a44a08b/pcdet/models/backbones_3d/dsvt.py#L245
Wish your reply.

You need to specify the dynamic shape in Pytorch-ONNX-TensorRT conversion.

You need to specify the dynamic shape in Pytorch-ONNX-TensorRT conversion.

Same as this?
https://github.com/Haiyang-W/DSVT/blob/3b825d11e7695d514d795457771508859a44a08b/tools/deploy.py#L194

Yep. You should to specify it in the trtexec command as well.


Update hello-world.mdx

👷 Deploy Preview for resilient-rabanadas-125ad7 processing.

Name
Link

🔨 Latest commit
34bb7842c6537aab88ed16c659bdef41b248d4dc

🔍 Latest deploy log
https://app.netlify.com/sites/resilient-rabanadas-125ad7/deploys/641d11fe5b330800080a480c


add clamp

@gcanti I think this is ready to go, care to take a look?

return (low: A, hi?: A, x?: A) => {
  if (hi === undefined) {
  }
}

@giogonzo there's a possible corner case when A = undefined

This test is somehow passing already:
const ordUndefined: Ord<undefined> = {
      equals: (a, b) => a === b,
      compare: () => 0
    };
    expect(
      JSONEqual(R.clamp(undefined, undefined, undefined), FR.clamp(ordUndefined)(undefined, undefined, undefined))
    ).toBe(true);

yes, it fails without the JSON.stringify:
expected: undefined
received: [Function]

@gcanti added support for it in the last commit, but it has a perf cost. Probably it can be improved by switching back to function instead of () => and accessing arguments.length

Tracking further perf improvements in #13


[Feature Request] Allow destination with custom domain (GHES)

When I using Github enterprise server with our custom domain, we can simply fetch repo from github.com, but when pushing to internal repo, the error log shows:
fatal: repository 'https://github.com/<org-name>/<repo-name>/' not found

In fact, the behavior is right because we do not have that repo on github.com, but only in our Github enterprise server.
I create this actions file in our internal repo in our Github enterprise server to sync from github.com.
name: Sync from Github.com

on:
    schedule:
    - cron:  "*/15 * * * *"
    workflow_dispatch:

jobs:
    repo-sync:
      runs-on: ubuntu-latest
      steps:
      - uses: actions/checkout@v3
        with:
          persist-credentials: false
      - name: repo-sync
        uses: repo-sync/github-sync@v2
        with:
          source_repo: "${{secrets.SOURCE_REPO}}"
          source_branch: "*"
          destination_branch: "*"
          github_token: ${{ secrets.PAT }}
          sync_tags: "true"

As we using custom domain rather than github.com, I think it is good to have an option to change the prefix or the full repository name before we set the "destination branch", for example, add a destination_repo option in actions.

@repo-sync/reviewers can you assign this to yourself?

We welcome community contributions.


Errors on cloned prepack

When clone from webapp failed (see #31) I've tried to download prepack and start structor from there. unfortunatley there were errors:
Cannot GET /deskpage/
["Cannot read property 'error' of undefined"]

Any hints how to help debug the project?

Package you sent works with Structor in environment: npm 3.3.12, node 5.2.0, mac os x.
Will update node up to 5.3.0 and will try again.

Package you sent works with Structor in environment: npm 3.3.12, node 5.2.0, mac os x.
Will update node up to 5.3.0 and will try again.

Also getting same error as adambro. Running on mac osx, latest mature version of node (v4.2.2).
Also, get error when creating custom component, when selecting "React component with all children". Are you able to create a custom component (say, similar to your video tutorial https://www.youtube.com/watch?v=JLz8B0XJPyk) with the latest Structor version (v0.2.6)?

@radman-x I tried to clone and generate simple component and didn't get error. There is only one place where such error can be thrown, this is when client part of Structor make http request to Express server. In that case client uses isomorphic-fetch lib.

Any hints how to debug it to provide more information or fix it myself?

@adambro please find in source /src/client/src/actions/serverActions.js file. Then go to 162 line - there our problem is (it seems to me).
There is no debug configuration, but you can build structor by yourself. Here is the instruction how to build and run locally structor from the build.

clone this repo
run npm install
run npm run build-client
run npm run build-server
run node structor.js -d <path to some empty folder>

File ./src/client/actions/serverActions.js in bootstrap prepack has got only 60 lines.
I've tried to build from source as you've sugested. Error ["Cannot read property 'error' of undefined"] does not happen, but http://localhost:2222/deskpage/ still returns 404.

@adambro are you trying to open http://localhost:2222/deskpage in browser? structor's URL is http://localhost:2222/structor

As for ./src/client/actions/serverActions.js I meant the file in the source code of structor.

As far as I can see structor tries to open http://localhost:2222/deskpage/  in an iframe - that's the main workspace. In iframe and in separate window I'm getting 404. Also http://localhost:2222/desk/build/bundle.js does not give any HTTP response when I look at network tab in FF inspector.

It seems that you have some issues with dev-middleware of webpack. Let's try to see is it works: in the source code of structor find file <structor source>/src/server/src/MiddlewareCompilerManager.js, go to 109 line and change true to false in noInfo and quit fields accordingly. Try to build and run strucotr again.

This doesnt work for me either.
I started with redux prepack and i got an error saying .structor didn't exist.  then I tried material and got the can't GET /deskpage error

@bradennapier try to download and unpack prepack manually. Read about the second way for running structor here: https://github.com/ipselon/structor#getting-started

:+1:  tried on several environments, ubuntu linux. same cannot get deskpage.

@jamesdwilson did you try to download manually? here is how to do that https://github.com/ipselon/structor#getting-started (the second way)

Yes, tried that way, tried with several node versions. Tried standalone
method 1 and tried method 2 with
https://github.com/ipselon/neal-react-prepack
On Wed, Feb 3, 2016 at 1:19 AM, Alex Pustovalov<EMAIL_ADDRESS>wrote:

@jamesdwilson https://github.com/jamesdwilson did you try to download
manually? here is how to do that
https://github.com/ipselon/structor#getting-started (the second way)
—
Reply to this email directly or view it on GitHub
https://github.com/ipselon/structor/issues/32#issuecomment-179059744.

--
Thank you!

neal-react-prepack is compatible only with 0.2.0 version of structor. If you've installed version 0.2.7 please try bootstrap-prepack or material-ui.

Please mark that in bold on top of it. Thank you for your help and
wonderful products.
On Wed, Feb 3, 2016 at 1:31 AM, Alex Pustovalov<EMAIL_ADDRESS>wrote:

neal-react-prepack is compatible only with 0.2.0 version of structor. If
you've installed version 0.2.7 please try bootstrap-prepack or material-ui.
—
Reply to this email directly or view it on GitHub
https://github.com/ipselon/structor/issues/32#issuecomment-179066063.

--
Thank you!

Please find a label in the top of Readme on each project for Structor's compatibility. Thanks for your kind words.

Please make a feature that compares the template version with the maker.
On Wed, Feb 3, 2016 at 1:34 AM, Alex Pustovalov<EMAIL_ADDRESS>wrote:

Please find a label in the top of Readme on each project for Structor's
compatibility. Thanks for your kind words.
—
Reply to this email directly or view it on GitHub
https://github.com/ipselon/structor/issues/32#issuecomment-179067043.

--
Thank you!

@jamesdwilson could you elaborate on that feature? I'm not sure that I understand what the template and maker you meant.

Need to close issue in view of new version.


Run requirements updater to bump keras, tb and estimator

The requirements.in file was updated in https://github.com/tensorflow/tensorflow/commit/4a43c56418ce2edf6c7f7b61e1f1061a02005e05 but the updater wasn't run.

@mihaimaruseac could you please approve?

Is this a cherrypick from master?


Tests at compile time?

You can do tests in Idris at compile time:
E.g. for the leap exercise:
total is_leap_year : Nat -> Bool
is_leap_year n = assert_total $ (mod n 4 == 0 && mod n 100 /= 0) || mod n 400 == 0

leap_year_2000 : (is_leap_year 2000 = True)
leap_year_2000 = Refl

@srenatus, @yurrriq: Any thoughts on if this is a better choice than the more traditional runtime testing?

I'd prefer to use idris-testing.

I'm not sure but I think this way of "testing" (it's rather the compiler ensuring, with the help of types, that the program is correct) doesn't apply to all problems. For example, in the "hello world" exercise, it's not obvious to me how to do something similar. (Maybe by changing the types?)

I'll have a try with hello-world.
I like it because

dependent types are a key feature of Idris which this highlights
does not need any dependencies.

For hello-world this works ok, but in general I'm opposed to it. If you wanted instead to prove theorems about properties of a given exercise, that would be ok.

@yurrriq I agree. It would make sense to keep hello world as simple as can be -- for getting your feet wet with the exercism CLI, testing of Idris, ... -- and there's nothing keeping you from revisiting hello world (or leap year) to look into advanced features of Idris. This would then be another exercise, though.

Closing this since we're going with the unit testing approach.


Clarify documentation as raised, closes #1003

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1004   +/-   ##
=======================================
  Coverage   63.63%   63.63%
=======================================
  Files         172      172
  Lines        9934     9934
=======================================
  Hits         6321     6321
  Misses       2825     2825
  Partials      788      788

Impacted Files
Coverage Δ

commands.go
87.54% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Error creating ARO cluster QUOTA issue of 36

Hello,
I followed the documentation (tutorial) and while runing the cluster creation command I get following error:
https://docs.microsoft.com/en-us/azure/openshift/tutorial-create-cluster
Command:-
az aro create  --resource-group $RESOURCEGROUP  --name $CLUSTER  --vnet aro-vnet  --master-subnet master-subnet  --worker-subnet worker-subnet --pull-secret @pull-secret.txt
Error
az : ValidationError: Deployment failed. Correlation ID: db1a9ece-bb85-40bf-94c6-36ff747e2ea8. Resource quota of cores exceeded. Maximum allowed: 20, Current in use: 0, Additional requested: 36.
At line:1 char:1
I am not sure what Quota and what needs to be increased from 20 to 36 ...can you please help and I am on MSDN Subscription Azure Tenant. I am stuck and unable to go ahead can you please help?

@ketaanhshah -
Thanks for bringing this to our attention. Since the same issue was raised here . Closing this duplicate issue and our engineers will work on the other issue. Thank you for your understanding. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Function to create a test SCE object

In #14 I wrote a bit of code to create a test SCE object with random counts. since that is a useful thing to have for other tests, we may well want to pull it into a separate function.
While I was writing it, I had the feeling that this was a thing that must exist, but couldn't quite find what I was thinking of. NowI have: DropletUtils::simCounts() provides a fancier function for creating simulated counts, and the extra features it provides may be useful for later testing.
We will still want to wrap it up to return a SingleCellExperiment, and probably use different defaults. For examples, we don't need nempty to be large (or even nonzero?) most of the time. (I also don't like that the gene ids it creates are not padded to the same length, but we can probably deal with that.)

Closed by #13


Java capability not detected on nix sysems

Agent Version and Platform
Version of your agent? 2.136.0
OS of the machine running the agent? Ubuntu Linux
Azure DevOps Type and Version
On prem server
If on-premises, which release? Ubuntu 1604
What's not working?
Java capabilities not recognized by the build agent. The env.sh script only adds the predefined set of env variables to the .env file. One of them is JAVA_HOME. Other JAVA_HOME_* variables are not added (e.g. JAVA_HOME_11_X64). Hence the Capability Provider can't recognize this capability.
Manually adding the env variable to the .env file works.
Ideally, the env.sh could also support adding the variables that match some prefix/regex instead of the direct mach only.

@TingluoHuang JAVA_HOME_11_X64 is the env variable/capability which is needed by e.g. Maven task if you specify the JDK explicitly to JDK 11. Even though the JDK 11 is installed and JAVA_HOME_11_X64 is set, it is not recognized as capability.
I am now a bit further. I'm adding the variables to /etc/enviromnent. It seems that my svc account doesn't pick it up from there. When I export the the variable, then it is recognized properly (by the EnvironmentCapabilitiesProvider I believe).
I guess I analyzed it wrong at first. If this is how it should work, then please close the issue.

I provision the VM almost the same as the hosted Ubuntu. I set the env. variable to /etc/environment during install. The issue was that the /etc/environment was not sourced properly.
Thank you very much for your help.


CausalTestOutcome static methods?

Current code and that of the examples:
causal_test_case = CausalTestCase(control_input_configuration={waypoint_start_dist_from_effector: 0.5},
                                  expected_causal_effect=NoEffect,
                                  outcome_variables={collisions},
                                  intervention=Intervention((waypoint_start_dist_from_effector,), (2 * 0.5,), ), )

This does not work with the code below as NoEffect is not instanced:
test_passes = causal_test_case.expected_causal_effect.apply(causal_test_result)

TypeError: apply() missing 1 required positional argument: 'res'

This fix works but feels rather redundant and is not in line with the covasim example present:
causal_test_case = CausalTestCase(control_input_configuration={waypoint_start_dist_from_effector: 0.5},
                                  expected_causal_effect=NoEffect(),
                                  outcome_variables={collisions},
                                  intervention=Intervention((waypoint_start_dist_from_effector,), (2 * 0.5,), ), )

Is it worth making the apply method of each CausalTestOutcome static to keep in line with current examples? Or should current examples be updated?

I had to read this a couple of times to get what you meant there. I agree that this is one of the many slightly clunky things about the framework. Python's a bit weird with static methods. It did occur to me that it might even be worth making causal test results a method rather than a class since you shouldn't really be doing stuff with them programatically. Possibly worth having a discussion with @cwild-UoS and @bobturneruk  about this to see if they have any suggestions for best practice here.


Upgrade to vault 0.7.1

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

This PR is ready for review.
cc: @conda-forge/vault

Thanks!


Video Conference Mute text "overlay image" is misleading

In Video Conference Mute, there is an option for a "Camera overlay image," which sounds like the user should be able to create a transparent image with things like some text or a company logo that they would like to overlay on their active camera image. However, this image is displayed when the camera is muted. The text should be changed to "Camera replacement image" or "Image displayed when muted" or maybe just add text explaining the behavior. I suppose "overlay" could be the correct word, but it is probably confusing to a general person.

I can do this.

Improved in v0.63


Scroll axis locking

Is it feasible to lock the scrolling onto the axis that it started on, i.e. ignore X axis scrolling when a Y scroll is in progress, and vice versa?
I just installed this, and I love it, but I'm paranoid about inadvertently undoing/redoing a change when cruising through a file.

pretty sure I already do this: https://github.com/mathieudutour/scroll-through-time/blob/master/lib/scroll-through-time.js#L12-L14

I think that when this function is enabled, it should be pressing a hot key to trigger the effect later.


PERFORMANCE ISSUE: BrowseActivity loads very slow

These fetches (latest commit, username fetch) is slowing down the loading.
Should combine threads and maybe add pagination to atleast reduce this issue.
Or maybe create a latest_commit_timestamp field in /project/id/ so we don't need to separately fetch the latest commit.

I've decided to add latest_commit_timestamp


Can't pipe

When used like that:
var fs = require('fs'),
    union = require('union');

union.createServer({
  before: [
    function (req, res, next) {
      var writeStream = fs.createWriteStream('req');
      req.pipe(writeStream);

      writeStream.on('close', function () {
        res.writeHead(200);
        res.end('Wrote to a stream\n');
      });
    }
  ]
}).listen(9000);

file req gets created but no data is found there (empty file) after I POST something to localhost:9000. Can someone confirm it should work so that I can write a test case?
It happens on current master - 3eeeef8e5b9fb39320f7937921b5a286203d85c2.

@mmalecki Is this still a problem on 0.6? I'm seeing all tests passing against<EMAIL_ADDRESS>

@indexzero Test case added in 740204b.


Adding metrics by hour change back to repo

Checklist

[x] manage.py test passes
[x] tests are included
[x] migrations are included
[x] documentation is changed or added

Affected app(s)

metrics
Description of change

Re-adding change for binning the metrics by hour to support https://github.com/ngageoint/scale-ui/issues/322.

I'm not convinced this is adequate yet. I would like to have the dashboard updates nailed down a bit more before committing this change so we know exactly what we need.

@gisjedi Ready to review 👍


Add index.js file which is just an alias to dat-core. fixes #314

I didn't push this directly to beta cause I didn't know if this is the idea you guys had @mafintosh @maxogden

i'd rather not have this now and just wait until later to implement require('dat') for beta when we decide what the high level JS API should be


242 minor paper revisions

Incorporates updates to package in paper
Synthetic data loaders now default to global RNG

Codecov Report

Merging #253 (74b4291) into development (17e4b88) will decrease coverage by 0.40%.
The diff coverage is n/a.

:exclamation: Current head 74b4291 differs from pull request most recent head 81e2f1b. Consider uploading reports for the commit 81e2f1b to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@               Coverage Diff               @@
##           development     #253      +/-   ##
===============================================
- Coverage        83.65%   83.25%   -0.40%
===============================================
  Files               72       71       -1
  Lines             1670     1666       -4
===============================================
- Hits              1397     1387      -10
- Misses             273      279       +6

Impacted Files
Coverage Δ

src/data/Data.jl
100.00% <ø> (ø)

src/data/synthetic/blobs.jl
100.00% <ø> (ø)

src/data/synthetic/circles.jl
100.00% <ø> (ø)

src/data/synthetic/linearly_separable.jl
100.00% <ø> (ø)

src/data/synthetic/moons.jl
100.00% <ø> (ø)

src/data/synthetic/multi_class.jl
100.00% <ø> (ø)

src/data/synthetic/overlapping.jl
100.00% <ø> (ø)

src/evaluation/benchmark.jl
93.24% <ø> (-1.43%)
:arrow_down:

src/evaluation/evaluate.jl
77.35% <ø> (ø)

src/generate_counterfactual.jl
83.33% <ø> (-3.34%)
:arrow_down:

... and 2 more

... and 6 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Use service address for HTTP / Thrift listeners if one is present.

See #22367.

NOTE: This PR is intended to be a discussion point for the reported issue. I don't have the full context on why the HTTP / Thrift listeners don't try to determine the service address in cases the bind is unset.

Hi @dhermes. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

related https://github.com/istio/istio/pull/17169

To elaborate now, rather than leave this with "ill share some info later". I think initially we can just add a sniffing that will make the <IP_ADDRESS>_X listeners logic behve like
if http {
  use routes, which will passthrough unmatched traffic
} else {
  passthrough traffic
}

Which should resolve these cases.
There are a bunch of future, larger, optimization we can do based on that, but that's the core part to get this scenario working. That PR unfortunately got lost in the noise but was recently resurfaced and I am pretty hopeful we can get some form of a fix in for the next release

/ok-to-test

I don't think this is the right fix

Just wanted to spur discussion forward (was comparing how TCP and HTTP listeners got created when I noticed the differences in the code at those points). I have 0 expectation this will get merged.
On the outside change Istio maintainers hadn't realized the len(listenerOpts.bind) == 0 case was being handled differently for TCP listeners vs. HTTP / Thrift listeners, I was hoping this would help out.

we will have N^2 routes, and N listeners

(I'm not familiar with how route creation works.) The TCP case already creates N listeners. E.g. creating
---
apiVersion: v1
kind: Service
metadata:
  name: example
  labels:
    app: example
spec:
  ports:
  - name: tcp1
    port: 10002
    protocol: TCP
  - name: tcp2
    port: 30004
    protocol: TCP
  - name: tcp3
    port: 50006
    protocol: TCP
  type: ClusterIP

leads to
kubectl get service --namespace "${NAMESPACE}" example
# NAME      TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)                         AGE
# example   ClusterIP   <IP_ADDRESS>   <none>        10002/TCP,30004/TCP,50006/TCP   78s

./istio-1.5.0/bin/istioctl proxy-config listeners\
  --namespace "${NAMESPACE}" \
  "${POD_NAME}" \
  --output json \
  > pr-22380.json

/bin/cat pr-22380.json | jq '.[].name' -r
# ...
# virtualInbound
# <IP_ADDRESS>_50006
# <IP_ADDRESS>_10002
# <IP_ADDRESS>_30004
# null

So you're saying you'd expect that to change as well?

compared to N routes and 1 listener today.

ISTM that having 1 listener per service port is a "good thing" (when the IP of the service is known). It allows disparate traffic types to avoid having their filters "fight" over the traffic.

I am pretty hopeful we can get some form of a fix in for the next release

That's great, do you mean 1.5.0? Is there an expected date for that release?
Until then we've got to ban the http-* prefix (and related) in our clusters with an admission webhook. In testing we've already seen postgres-consuming services break due to a <IP_ADDRESS>_5432 listener; this one was caused by the fallback of <IP_ADDRESS> for a missing cluster IP.

ISTM that having 1 listener per service port is a "good thing" (when the IP of the service is known). It allows disparate traffic types to avoid having their filters "fight" over the traffic.

If we just do this as is, it will have major impact due to how Envoy manages the memory - if we have 10 (for example) listeners reference the "http.80" route, rather than just <IP_ADDRESS>_80, then we actually get 10x copies of this in memory. You can see some context https://github.com/istio/istio/issues/16869.
If we went the full route of completely breaking things out to be 1 listener per service, things might be different. I suspect we still have a large cost inside Envoy and in Pilot to handle the additional listeners. We will actually most likely go the other extreme and have a single listener: https://github.com/istio/istio/issues/22217

That's great, do you mean 1.5.0? Is there an expected date for that release?

1.5.0 is already out, so it would likely be 1.6.0. Unless we can make a minimal change and call it a bugfix and put it into a 1.5.x possibly

We will actually most likely go the other extreme and have a single listener: #22217

That is definitely the "other extreme"! But I'm happy some care has been taken about the footprint. Do you think it'd make sense (in the interim) to artificially limit the discoverable services with a Sidecar per-service that was just a whitelist of the services and / or routes that could be used.

1.5.0 is already out

I fat-fingered that, meant 1.5.1 and edited my post since. Sorry.

@dhermes: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
85bbb88222e09b7ee5bcaf26fa0db99c972f7a4e
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@dhermes: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
85bbb88222e09b7ee5bcaf26fa0db99c972f7a4e
link
/test unit-tests_istio

integ-security-k8s-tests_istio
85bbb88222e09b7ee5bcaf26fa0db99c972f7a4e
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

  use routes, which will passthrough unmatched traffic
} else {
  passthrough traffic
}```

Dont we already have this logic when sniffing is enabled and the port's protocol is unknown? I guess in this case, the port's protocol is known but some unknown TCP service gets caught in this port. For that, why can't we do exactly as you suggested? i.e. always have the HTTP sniffer be present even in well known listeners and do the standard fallback logic so as to be able to handle these port conflicts?

We did exactly that in another PR merged yesterday

We did exactly that in another PR merged yesterday

? Which one was that ?

sorry thought it was linked here: https://github.com/istio/istio/pull/17169
On Wed, Mar 25, 2020 at 6:37 AM Shriram Rajagopalan <
<EMAIL_ADDRESS>wrote:

We did exactly that in another PR merged yesterday
? Which one was that ?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/22380#issuecomment-603843907, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXPXIW5Z5STLDSHJWOLRJICLHANCNFSM4LRNSFMQ
.

Ah yes huabing’s PR should have tackled my comment.

@dhermes: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


[AMD] Refactor lowering math.fdiv with f32 inputs

This commit lowered math.fdiv to a truely approximated div operation.

New contributor declaration

[x] I am not making a trivial change, such as fixing a typo in a comment.

[x] I have written a PR description following these
rules.

[x] I have run pre-commit run --from-ref origin/main --to-ref HEAD.

Select one of the following.

[ ] I have added tests.

/test for lit tests
/unittest for C++ tests
/python/test for end-to-end tests

[x] This PR does not need a test because Tests already exist.

Select one of the following.

[x] I have not added any lit tests.
[ ] The lit tests I have added follow these best practices,
including the "tests should be minimal" section. (Usually running Python code
and using the instructions it generates is not minimal.)

It'd be great to invite folks from LLVM team to take a look and confirm the correctness.

is this to improve performance or accuracy?

is this to improve performance or accuracy?

It's to improve performance by using a faster but less accurate div.


feat: track contributors in workspaces

Description
Implements tracking contributors in Workspaces. There are two parts for basic functionality:

Add contributors when creating a workspace under /workspaces/new

This will done in a future PR

[X] Updating contributors in the Settings (individual search to add, table, updating)

[X] TrackedContributorsTable

[x] initial fetch
[X] delete contributor (complete with updateWorkspace())

[X] TrackedContributorsModal (add to Table on complete)
[X] SearchByContributorsStep

add and filter contributors, uses /v2/users/search?username endpoint
other steps to be implemented in future PRs

[x] updateWorkspace() in /workspaces/[id]/settings

[x] Fetching data to display under /workspaces/[id]/contributors

Text only, graphs will be implemented in a future PRs

What type of PR is this? (check all applicable)

[X] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Related Tickets & Documents
Closes #2643
Relates to #2578

Mobile & Desktop Screenshots/Recordings
https://github.com/open-sauced/app/assets/20603494/59712f53-983f-4b03-b248-a8c1fce89aec

Steps to QA

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Redeploying on Netlify completes the build fine, I don't know how to re-run the deploy via Github to clear the Netlify checks.


Conflict with See Through Lava/Water

Mod link: https://www.curseforge.com/minecraft/mc-mods/see-through-lava-water
[22:25:23] [main/WARN]: @Redirect conflict. Skipping origins.mixins.json:BackgroundRendererMixin->@Redirect::redirectFogStart(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;)V with priority 1000, already redirected by seethroughlava.mixins.json:BackgroundRendererMixin->@Redirect::changeFogStartInOther(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;FZ)V with priority 1000
[22:25:23] [main/WARN]: @Redirect conflict. Skipping origins.mixins.json:BackgroundRendererMixin->@Redirect::redirectFogEnd(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;)V with priority 1000, already redirected by seethroughlava.mixins.json:BackgroundRendererMixin->@Redirect::changeFogEndInOther(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;FZ)V with priority 1000
[22:25:23] [main/WARN]: @Redirect conflict. Skipping origins.mixins.json:BackgroundRendererMixin->@Redirect::redirectFogStart(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;)V with priority 1000, already redirected by seethroughlava.mixins.json:BackgroundRendererMixin->@Redirect::changeFogStartInOther(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;FZ)V with priority 1000
[22:25:23] [main/WARN]: @Redirect conflict. Skipping origins.mixins.json:BackgroundRendererMixin->@Redirect::redirectFogEnd(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;)V with priority 1000, already redirected by seethroughlava.mixins.json:BackgroundRendererMixin->@Redirect::changeFogEndInOther(FLnet/minecraft/class_4184;Lnet/minecraft/class_758$class_4596;FZ)V with priority 1000

Fixed in said mod.


SS-7926: adjusted name filtering

https://shapediver.atlassian.net/browse/SS-7926
@snabela please review this new logic!

@snabela do you have any other names to add to the nodeNameBlackList?
const nodeNameBlackList = ["TransformZUpToYUp", "no_transformations"];

@snabela do you have any other names to add to the nodeNameBlackList?
const nodeNameBlackList = ["TransformZUpToYUp", "no_transformations"];

@MajorMeerkatThe3rd sorry I had forgotten to answer your question: I reviewed this and the list is complete.


Bug when change routes (vue router)

When change routes, the error occurs.
Cannot read property 'offsetLeft' of undefined(…)
calculateIndicatorPos @ vue-material.js?ec92:8
boundFn @ vue.common.js?e881:127
(anonymous function) @ vue-material.js?ec92:8

Tks

Oh. That's strange. In the Vue Material documentation there is a vue-router, and it's working fine.
Can you provide a Codepen with an example?

It's really strange. I find this error when I try routing to a page, while the error doesn't happened when I am routing to another page in the same way. I have made some test and it seems that there's no association between the error and the content of the child page.
So far my workaround is:

Using router-link element to produce an <a> instead of  using $router.push in javascript code

Are you sure it's md-button? It seems a bit misleading. CalculateIndicatorPos is a method within mdTabs. Do you have a tabs component on these routes you've been testing perhaps?

HI. Thanks for a such detailed description. This helped me a lot. I've found the problem and I will release a fix soon. Thank you!

Tks @huhk-sysu for reproducing.
Tks @marcosmoura for the help!

`
        <md-button style="color:white" class="md-icon-button">
            <i class="icon avatar-icon"></i>
        </md-button>
        </router-link>`

this code shows the same error


Targets tab > First target period begins > Getting an incomplete error message for unclear reasons.

To reproduce:

Open the indicator modal for an indicator that does not yet have targets.
Pick a periodic target frequency like Annual.
Click in the box next to "First target period begins".
Click "Done" without making any adjustments.

Expected: The field is considered complete.
Observed: I get this.

As soon as I tab out of the field, the error highlighting goes away. But I shouldn't see it in the first place -- this is new behavior, doesn't happen on production.

I briefly see a flash of red outline around the box. Can this be fixed?


Add a script for upgrading rules into the new dynamic format

At the time of the opening of this PR, the script provides the main logic for parsing the rules. This logic is composed of 3 recursive functions that each handle a specific task, and that call into each other. I settled on this approach after trying other things (such as the strategy mentioned in #1674) and failing/getting stuff too complicated.
There remains the following important tasks, but they should be easy to add for the most part:

[x] merge the static and dynamic statements generated at the initial node (rn, it returns them separately)
[ ] add all supported dynamic features into the script
[x] add the logic for saving the generated rules
[x] add static to dynamic scope translation (right now it assumes the "thread" scope for all)
[ ] sort rules by dependency, and exclude match features that depend on out-of-context rules
[ ] optimize and clean the code

Checklist

[ ] No CHANGELOG update needed

[x] No new tests needed

[ ] No documentation update needed

The script's logic is finished I believe, so feel free to review it. All that remains is to add support for filtering-out match rules that point to static-only rules when appropriate, as well as adding the final list of supported dynamic features (perhaps we want to add some characteristics?)
Here's the script's result on a made up complex rule:
original rule:
rule:
  meta:
    name: test rule
    namespace: test
    authors:
      -<EMAIL_ADDRESS>    scope: function
    att&ck:
      - Persistence::Event Triggered Execution::Unix Shell Configuration Modification
        [T1546.004]
    examples:
      - 7351f8a40c5450557b24622417fc478d:0x407D11
  features:
    - and:
      - basic block:
        - or:
          - os: linux
          - match: host-interaction/file-system/write
          - or:
            - mnemonic: call
            - api: someapi_1
          - instruction:
            - mnemonic: write
            - string: somestring_1
      - instruction:
        - api: someapi_2
        - number: 1
      - string: somestring_2

modified rule:
rule:
  meta:
    name: test rule
    namespace: test
    authors:
      -<EMAIL_ADDRESS>    scopes:
      static: function
      dynamic: process
    att&ck:
      - Persistence::Event Triggered Execution::Unix Shell Configuration Modification
        [T1546.004]
    examples:
      - 7351f8a40c5450557b24622417fc478d:0x407D11
  features:
    - and:
      - or:
        - basic block:
          - or:
            - os: linux
            - match: host-interaction/file-system/write
            - or:
              - mnemonic: call
              - api: someapi_1
            - instruction:
              - mnemonic: write
              - string: somestring_1
        - thread:
          - or:
            - os: linux
            - match: host-interaction/file-system/write
            - or:
              - api: someapi_1
      - or:
        - instruction:
          - api: someapi_2
          - number: 1
        - call:
          - and:
            - api: someapi_2
            - number: 1
      - string: somestring_2

[ ] offset features get removed

[ ] comments and quotes get removed

[ ] number exported as decimal

[ ] mnemonic/number removed

See capa\scripts\capafmt.py for potential hints on how to handle the formatting.

[ ] may be a good opportunity to convert all rules to use Unix line endings (\n only)

@mr-tz regarding the removal of mnemonic and offset, isn't that desirable? since instructions aren't extracted by sandbox output (to my understanding)

For the static rule part mnemonic and offset should still be there.

Ah yeah I see. Will work on it...

Currently, getting capa.rules.InvalidRule: invalid rule: subscope must have exactly one child statement on the rule with name: inject pe.

I think this can be closed?


Using the rust/ pytorch dataloader

Hi,
My datasets are at :

Then I try this:
logging.basicConfig(level=logging.INFO)     logger = logging.getLogger(__name__)     logger.info("Starting detection data-loader example ...")     root_dir = Path("/home/data/argoverse/av2/part1")     data_loader = DetectionDataLoader(root_dir, dataset_name="sensor", split_name="train",)     for i, sweep in enumerate(tqdm(data_loader)):         # 4x4 matrix representing the SE(3) transformation to city from ego-vehicle coordinates.         city_SE3_ego_mat4 = sweep.city_SE3_ego.matrix()
But I get an error:

Please let me know what is wrong here. I checked the tutorial and it says: root + {dataset_name} + {split_name} should end up inside one of the part folders train/ val directory. I think I am doing just that.
Best Regards
Sambit

The code did not get formatted properly:

I got it. I did import tqdm instead of from tqtm import tqdm.
Now it works.

Please take a look at the other issue I face now regarding using the old convenience functions with the new Pytorch data loader.
Kind of stuck and following an ugly approach right now - create both DetectionLoader and SensorLoader objects.


preferSimple setting has opposite effect

This issue is in regards to the following code block:
https://github.com/emipa606/PawnsChooseResearch/blob/2e5d49c389ea6215b02a11ab3c4e5b94ef41ecea/Source/PawnsChooseResearch/ResearchPreferences.cs#L33-L38
The num variable here stores the pawn's preference score, and a higher value equates to more of a preference. However, due to the condition in the if statement, the quotient being subtracted from num will always be negative, and since this value is subtracted from num, num will always be increased as a result of entering the if block. Based on how the "prefer simple" / "Avoid Too Advanced Research" setting is described, a pawn would have a lower preference for a more advanced tech, but the opposite is happening.

Ill fix it with a change of
Faction.OfPlayerSilentFail.def.techLevel - researchProject.techLevel
to
researchProject.techLevel - Faction.OfPlayerSilentFail.def.techLevel
to get a positive value instead.


MySQL JNDI fixes

See #831

+1 happy too - very appreciated work!
although some tests indicates that we might be in a semi-supported state (kind of working but not ideal).
still okay by me - have always spent too much time digging into details when looking into JNDI support.

ok, new version that should be lazy...I like it better now


minimumInputLength causing drawing error when dropdown expands upwards

Position your select2 to the bottom of the screen to make it expand upwards.
If you set "minimumInputLength" (to 1 or greater), start typing, the dropdown appears upwards, showing the matching items. Press backspace to clear the search field. The items disappear but the search field will remain where it was.

Can you expand upon the issue you are facing by adding a few more pieces of information to your bug report?

What browser(s) and Operating System have you tested with?
Does the bug happen consistently across all tested browsers?
What version of jQuery are you using? And what version of Select2?
Are you using Select2 with other plugins?

The contributing guide contains a useful section with a few questions which can help us track down the bug and speed up the process of finding a fix for it.  The creation of an isolated test case would also be useful.

The new beta3 fixes this bug, thank you.


Lazy loading & resolveComponentFactory & navCtrl.push problem

Resources:
Before submitting an issue, please consult our troubleshooting guide (http://ionicframework.com/docs/troubleshooting/) and developer resources (http://ionicframework.com/docs/developer-resources/)
Please make sure you are posting an issue pertaining to the Ionic Framework. If you are having an issue with the Ionic Pro services (Ionic View, Ionic Deploy, etc.) please consult the Ionic Pro support portal (http://support.ionicjs.com)
Ionic version:  (check one with "x")
(For Ionic 1.x issues, please use https://github.com/ionic-team/ionic-v1)
[ ] 2.x
[x] 3.x
[ ] 4.x
I'm submitting a ...  (check one with "x")
[ ] bug report
[x] feature request
Current behavior:
I have a leaflet map in my app. I use componentFactory to create custom page content (=unique component) to the leaflet popup.
Now I tried to refactor my code to use lazy loaded pages. Everything works fine, but I can't use the componentFactory with the component's name:
old, working code:
const compFactory = this._resolver.resolveComponentFactory(MapInfoWindow);

lazy loading:
const compFactory = this._resolver.resolveComponentFactory('MapInfoWindow');

For temporarly solution, now I import this page as normal page, and the other ones are used as lazy loaded pages. But I have a problem...
I have a link in the popup window, which navigates to the next tabbed page. When I click to the link, on the first click (and only the first), the nav.push open's the required page, but redirects to the root immediately. When I open the popup again, it works as I expected.
Expected behavior:
Maybe the solution would be the lazy loading of the popup window?
Steps to reproduce:
Sample video
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
@ionic/cli-utils  : 1.12.0
ionic (Ionic CLI) : 3.12.0
cordova (Cordova CLI) : 7.0.1
@ionic/app-scripts : 3.0.0
Cordova Platforms  : android 6.3.0 browser 4.1.0 ios 4.4.0
Ionic Framework    : ionic-angular 3.7.1
ios-deploy : 1.9.2
Node       : v6.11.3
npm        : 3.10.10
OS         : macOS High Sierra
Xcode      : Xcode 9.0 Build version 9A235
backend : pro

Hello! Thank you for opening an issue with us! Would you be able to provide a sample application via GitHub that demonstrates the issue you are having?
general questions:

For temporarly solution, now I import this page as normal page, and the other ones are used as lazy loaded pages

You call this a "temporary" solution. Is there some other solution you are looking at?
Also, you have this marked as a feature request, but it reads more like a bug. What is the feature you are requesting?
Thanks for using Ionic!

Excellent!! I am going to close this then.

@kensodemann thank you, helped me


fix Load profile need to be registered to get real-time profile

Profile need to be registered not only when it is finished, but also when it is running. So that user could get realtime profile of Load command.

run buildall


Issue 1498

Pull request for bug from issue: https://github.com/spring-cloud/spring-cloud-sleuth/issues/1498

@ByJacob Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ByJacob Thank you for signing the Contributor License Agreement!

Great job, thank you for this PR!


Getting response { parts: [], errors: 0 } regardless of what token we use for Bearer

This may not be a bug on your part, just wondering if you've seen this? We've been using the access_token acquired by the usual oauth2 exchange auth code for tokens flow, server side. Is this even the correct token to use here? It seems no matter what we put in for Bearer we get this response. I would expect a 400 or 401 response, not { parts: [], errors: 0 } Thanks for any info you can provide!

What does your bearer token output as? What are you passing into Batchelor?

Hey James!
So we've been using the access_token server receives from the oauth2
process for email user we are sending batch requests on behalf of.  user
has authorized server for gmail
api. (gapi.oauth2Client.credentials.access_token).  I know token is valid
because I can access gmail api for other rest calls. Just having problems
with batch. Is this even the right token to use?
I put a debug console.log in batchelor.js at line 169 to dump responseObj.
There's a bunch of json, showing looks like metadata about the processed
request but in end, not the actual payloads for each request in the batch?
I've attached sample for your review. It also shows the bearer token
received. I looked a bit at the response - its interesting, looks like 5
segments each with associated post request - I think. my inbox has 7
messages and I know all 7 are being added to batch, so thats off.
Also, I seem to get same response regardless of what is set for bearer:
'123456' yields same result.
Thanks in advance for any help you can provide, really appreciate it.
-Brian
On Wed, Mar 4, 2015 at 1:07 AM, James Haley<EMAIL_ADDRESS>wrote:

What does your bearer token output as? What are you passing into Batchelor?
—
Reply to this email directly or view it on GitHub
https://github.com/wapisasa/batchelor/issues/2#issuecomment-77120417.

I didn't see anything attached to your message.
Please add https://www.npmjs.com/package/global-request-logger to the top of your script and paste the output into here, obviously removing any sensitive information, please leave the token structure as is, but change the contents (so that you are not giving away the keys to the kingdom).


MNT: rerender

Hi! This is the friendly automated conda-forge-webservice.
I've rerendered the recipe as instructed in #2.
Here's a checklist to do before merging.

[ ] Bump the build number if needed.

Fixes #2

Hi! This is the friendly automated conda-forge-linting service.
I was trying to look for recipes to lint for you, but it appears we have a merge conflict.
Please try to merge or rebase with the base branch to resolve this conflict.
Please ping the 'conda-forge/core' team (using the @ notation in a comment) if you believe this is a bug.