refactor/mappings

With this PR i want to indroduce mappings between our Entity and Model (Data Transfer Objects) and vice versa. Additionally some mapping between the models.
Benefit is improved encapsulation and readability. If there are more entities and models comings it is easy to get going and create them (Open Closed Principle).
insertAndUpdate - renamed since it's more precise what is happening inside this method. There is always an updating happening. I'll create an issue in this regard, since database access is expensive.
Please have a look at #56 - The order of merging is relevant. Changes have to be made in mappings, since i expect #56 is going to be merged first. Value 0 is going to be valid in TestResult. Important notes

Thanks for encapsulation hint!
I think exporting this in new classes is kind of oveerloaded - may later MapStruct will be introduced. But i now encapsulated it in the service itself as method first.
https://github.com/corona-warn-app/cwa-testresult-server/pull/59

Thanks for encapsulation hint!
I think exporting this in new classes is kind of oveerloaded - may later MapStruct will be introduced. But i now encapsulated it in the service itself as method first.
#59

Please keep in mind that by doing so, you violate the SRP (Single Responsibility Principle) since service class is now doing the logical mapping too.
I see your point by referring to MapStruct, so it would be good to have the logical mapping encapsulated (in classes) so if we use MapStrcut we can easily migrate our code to it. But since mappings happen in controller layer too, we need to look at the different layers for migration. In this regard I do not share the opinion that it seems overloaded.
But again it's your decision, I was trying to help and improve in the ways I can.


feat: Markdown library comparison: react-markdown

How to test?

yarn dev. try copy and paste here then send a message.

Changes

ticket: []
Additional Notes

Checklist
Before requesting a code review, please check the following:

[ ] [Required] CI has passed all checks.
[ ] [Required] A self-review has been conducted to ensure there are no minor mistakes.
[ ] [Required] Unnecessary comments/debugging code have been removed.
[ ] [Required] All requirements specified in the ticket have been accurately implemented.
[ ] Ensure the ticket has been updated with the sprint, status, and story points.

closing because we will not be using this library.
we chose this instead


Move the udf-examples module from spark-rapids repository to here

[BUG] Dependencies missing of spark-rapids v21.12.0 release jars https://github.com/NVIDIA/spark-rapids/issues/4253
Move the udf-examples module from spark-rapids repository to here
Move the udf-examples to the external repository spark-rapids-examples and the spark-rapids will no longer contain the udf-examples module.
The moved udf-examples depends on the previous release dist jar, can't depends on the Snapshot jar, because of the Snapshot jar is not published to the Maven Central.
e.g.:  udf-examples 22.04 Snapshot depends on dist 22.02.0 version.
Signed-off-by: Chong Gao<EMAIL_ADDRESS>

Can we change this pr target to 22.02, the udf-examples can be found more easily. Since we will update 22.04 as the default branch two months later(the date when rapids 22.04 released)

Re-target to branch 22.02


[gradle][maven] Remove packages:write from call to lowperms delegator

Tracking issue for:

[ ] https://github.com/slsa-framework/slsa-github-generator/security/code-scanning/521

/cc @AdamKorcz

Strange, the scorecard issue scanning alert got marked as fixed, but the packages: write is still there.
https://github.com/slsa-framework/slsa-github-generator/blob/5b75337ded01fe8282ba78ff61d76957524634da/.github/workflows/builder_gradle_slsa3.yml#L58

This also seems to be the case for the Maven builder.

Gradle is being done here: https://github.com/slsa-framework/slsa-github-generator/pull/2458

Maven is being done here: https://github.com/slsa-framework/slsa-github-generator/pull/2459


Move hello-world template to use vite

Part of https://github.com/Shopify/hydrogen-internal/issues/109
Try following changes for skeleton from https://github.com/Shopify/hydrogen/pull/1912/files#diff-a8d9184f80eafa2d357d59aaa222bbaec3290cd69c4a73379d571fa9abb70e15 for Hello-world template

cc @frandiox would you know why type is failing so hard in vite.config.ts. I must be missing something obvious 🤔


Question: I want stop event handling on exception.

I have few WorkHandlers and I want to freeze event handling on time period when some exception was thrown and return event into queue. How I can do this with standard disruptor api.
I know how I can do this standard java api.

Was discussed on the mailing list.


fix: sanitise CQL text query

What's being changed:
When submitting a CQL query to the confluence API, the queries failed if a white space character was included in the query (for example "BBQ Grill" would cause an error like
ERROR:provider.app:Upstream search error: Error during Confluence search: {"statusCode":400,"data":{"authorized":true,"valid":true,"errors":[],"successful":true},"message":"com.atlassian.confluence.api.servic
e.exceptions.BadRequestException: No field exists with the name: 'Grill'"}

looking closer at the code showed that the queries were being passed without quotes (the cause of the issue).
How did you test this change (include any code snippets, API requests, screenshots, or gifs):
The queries are now passed to CQL

adds " quote characters around the query string passed to CQL
using %s string formatting rather than f-string formatting (to avoid injection attacks
uses a regex to strip out exotic characters and pass a simple string

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


registerSystem should check if system conforms to interface

We expect a WorldContextConsumer in registerSystem:
https://github.com/latticexyz/mud/blob/08d7c471f9a2f4d6c237641eea316313d010373c/packages/world/src/modules/core/implementations/WorldRegistrationSystem.sol#L110
But we cast to address everywhere it's referenced in the method.

I assume the more specific WorldContextConsumer is there for our tests and so the compiler can help us identify issues, but should the registerMethod also check system.supportsInterface(WORLD_CONTEXT_CONSUMER_INTERFACE_ID)?
Should this be System instead of WorldContextConsumer?
Would bad things happen if we register a system contract that doesn't conform to these interfaces? If not, should we just make the type address?

That check happens here: https://github.com/latticexyz/mud/blob/08d7c471f9a2f4d6c237641eea316313d010373c/packages/world/src/modules/core/implementations/WorldRegistrationSystem.sol#L117
The System interface is currently just an alias for WorldContextConsumer without additional methods.
Bad things would happen if the contract was not a system: you could make the World call arbitrary contracts (don't have a specific attack vector in mind, calling the World itself is not possible anymore, but still feels dangerous and unintended).

oh awesome, missed that!


🐞 [Bug]: Add validation to compute capacity module

Is there an existing issue for this?

[X] I have searched the existing issues

which package/s did you face the problem with?
grid_client
What happened?
No errors were thrown when testing with invalid min, max, empty values, and decimal values.
Steps To Reproduce
Run the following test: https://github.com/threefoldtech/tfgrid-sdk-ts/pull/2820/files
which network/s did you face the problem on?
Dev
version
Locally
Twin ID/s
No response
Node ID/s
No response
Farm ID/s
No response
Contract ID/s
No response
Relevant screenshots/screen records

Relevant log output
-

We can add a MethodDecorator which should validate object before trigger the targeted function
function TriggerValidate(): MethodDecorator {
  return <T>(
    target: object,
    propertyKey: string | symbol,
    descriptor: TypedPropertyDescriptor<T>,
  ): void | TypedPropertyDescriptor<T> => {
    function value(...args: any[]): any {
      const errors = validateSync(target);

      if (errors.length > 0) {
        console.log({ errors });
        throw new Error("Something went wrong");
      }

      return target[propertyKey]?.(...args);
    }

    return {
      ...descriptor,
      value: value as T,
    };
  };
}

which can be used like
class Zmount extends WorkloadData {
  @TriggerValidate()
  challenge(): string {
    return ''
  }
}

Work completed:
Implemented a way to validate class prop/methods when set/call.

feat: Add a class decorator with options to validate any needed prop/method
feat: Use Validate class decorator in ComputeCapacity class to throw errors when needed
chore: Remove skip from tests and run them using SKIP_PROPS_VALIDATION=true env var
feat: Included SKIP_PROPS_VALIDATION to grid client workflow so tests won't fail

verified compute capacity is now added to the gridclient tests module and ran the test and it is working fine


#1754 - implement strikethrough formatting and client keybinding

@grissly-man thanks for this! Do you want to also implement monospace (#1755)? It should be pretty much the same way.

@xPaw Yes absolutely - I'll get on #1755 right after this

Thanks for adding more tests.

I'll get on #1755 right after this

Do you want to do this as a separate PR or part of this one?

@xPaw Whatever you'd prefer! Doesn't really make a difference to me

@grissly-man Probably simpler if you do it in this PR.

@xPaw Heard that

@xPaw heard that, pushed

These keybinds need to be added to autocompletion help in client/views/windows/help.tpl.

@grissly-man, I took the liberty to squash your commits into 2 meaningful commits (one for strikethrough and one for monospace), and I added the doc to the help section.
Thanks a lot for your help!!


feat(pack) Added Amazon Athena

Summary
Added dasboard for Amazon Athena.
Pre merge checklist

[x] Did you check you NRQL syntax? - Does it work?
[x] Did you check your dashboard image quality? -  Do they look good?
[x] Did you check that your alerts actually work?
[x] Did you include an InstallPlan and Documentation reference?
[x] Did you check your descriptive content for voice, tone, spelling and grammar errors?
[x] Did you attach images of your dashboards to the PR so we can see them working?

Screenshots
Attach images of any visual changes, such as a dashboard here.

I have same question here as in the aws-qldb pack, should I create any alerts on these packs?


Please provide a reference near top of document for Powershell Graph SDK

Hi there, although this documentation was updated last week, apparently it is obsolete. It appears that microsoft is now killing AzureAD module in favor of powershell graph SDK.  It would probably be helpful to modify this article to alert users of this fact as people like me, new to AzureAD module, were looking to do a new script and now I know not to bother with Azure AD module and to go get and learn about yet another new thing.
https://docs.microsoft.com/en-us/powershell/microsoftgraph/migration-steps?view=graph-powershell-beta

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

ID: 55574af9-0105-dcd1-7eb0-77513df4e54f
Version Independent ID: 5fe2849b-d39a-4d75-a5ce-e8f577c9a161
Content: Connect to Microsoft 365 with PowerShell - Microsoft 365 Enterprise
Content Source: microsoft-365/enterprise/connect-to-microsoft-365-powershell.md
Service: o365-administration
GitHub Login: @kelleyvice-msft
Microsoft Alias: kvice

@saraporter Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/8446
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @saraporter Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Why is this marked as updated. There is zero advice that the AzureAD and Msol modules will be depreciated.

I also do not see where in this doc it indicates that the AzureAD module is being replaced by powershell graph SDK - there should be a line about this at the top of the document so new adopters can choose to utilize the newer methods for powershell.  Please update the top of the document- I suggest something like
Note: The AzureAD module is being replaced by Powershell graph SDK.
Get started with the Microsoft Graph PowerShell SDK
You can use the Microsoft Graph PowerShell SDK to access all Microsoft Graph APIs.
For more information, see Get started with the Microsoft Graph PowerShell SDK

@msbemba

@saraporter I have moved the SDK module to the note at the top of the article. This can be tracked via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/8500
Thanks
Sri

@yogkumgit This can be closed.

Thank you!
Sara
From: Sriraman M S @.>
Sent: Thursday, April 28, 2022 10:38 AM
To: MicrosoftDocs/microsoft-365-docs @.>
Cc: Sara Porter @.>; Mention @.>
Subject: [EXTERNAL] Re: [MicrosoftDocs/microsoft-365-docs] Please provide a reference near top of document for Powershell Graph SDK (Issue #8299)
@saraporterhttps://urldefense.com/v3/__https:/github.com/saraporter__;!!NCZxaNi9jForCP_SxBKJCA!TRimtbLp4FROD7dJHvjjexRxpw6yyt40LR8XodXhSOL9oQZ4Vwk19B1wLKZ3DarbQQEjnrMwaUXZBUwsA84TU4NWwg$ Thank you for your feedback. We have made changes to the document via PR##8446https://urldefense.com/v3/__https:/github.com/MicrosoftDocs/microsoft-365-docs/pull/8446__;!!NCZxaNi9jForCP_SxBKJCA!TRimtbLp4FROD7dJHvjjexRxpw6yyt40LR8XodXhSOL9oQZ4Vwk19B1wLKZ3DarbQQEjnrMwaUXZBUwsA87tDgGDfA$
Once the author approves, the changes will be live.
Thanks
Sri
—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/MicrosoftDocs/microsoft-365-docs/issues/8299*issuecomment-1112425903__;Iw!!NCZxaNi9jForCP_SxBKJCA!TRimtbLp4FROD7dJHvjjexRxpw6yyt40LR8XodXhSOL9oQZ4Vwk19B1wLKZ3DarbQQEjnrMwaUXZBUwsA863A4ElTg$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/ASNY32LTNDYADGZMJVWL63DVHK5H7ANCNFSM5TI55TVA__;!!NCZxaNi9jForCP_SxBKJCA!TRimtbLp4FROD7dJHvjjexRxpw6yyt40LR8XodXhSOL9oQZ4Vwk19B1wLKZ3DarbQQEjnrMwaUXZBUwsA84sjG43dw$.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Prepare for PMM 2.24.0 development cycle

PMM 2.23.0 RC will be cut today, new development cycle begins afterwards.

Test name
Status

api-tests
passed


Support balance assertion tolerance syntax (~)

Thanks for addressing all the other issues Jonathan.
Updating and continuing on, I see the precision syntax fail:
2013-09-10 balance Assets:US:Vanguard                                   305.205 ~ 0.002 RGAGX

Thanks for the report, I didn't know that syntax.
I usually refer to this document: https://beancount.github.io/docs/beancount_language_syntax.html and it looks like doesn't mention it.
But of course, I'd like this parser to accept any valid beancount syntax, so the ~ precision syntax should be supported.

Great!
Id like to build something and learn some Rust in the process using your parser.

Local tolerance parsing looks like this in bison:```
amount_tolerance:
number_expr currency
{
BUILDY(DECREF($1, $2),
$$.pyobj1, "amount", "OO", $1, $2);
$$.pyobj2 = Py_None;
Py_INCREF(Py_None);
}
| number_expr TILDE number_expr currency
{
BUILDY(DECREF($1, $4),
$$.pyobj1, "amount", "OO", $1, $4);
$$.pyobj2 = $3;
}

I added some text int he docs:
https://docs.google.com/document/d/1wAMVrKIA2qtRGmoVDSUBJGmYZSygUaR0uOMW1GV3YE0/edit#heading=h.5jx4fqseqk64

This is fixed in version 2.1.0

Thank you!  :-)
I now confirm this parser is able to grok my entire gnarly personal Beancount file.


[RFE] Pass env vars to the odo create command

We can't specify ENV VAR to a component created using the command odo create openjdk18 my-spring-boot --binary ./target/demo-0.0.1-SNAPSHOT.jar
For some scenario, it is needed to pass runtime's ENV VAR to by example for Spring Boot, configure the application's property file to be used, enable the remote debugging, ...
envs:
  - name: SPRING_PROFILES_ACTIVE
    value: openshift-catalog

The existing odo create command should be extended to support to pass such parameters
odo create openjdk18 my-spring-boot --binary ./target/demo-0.0.1-SNAPSHOT.jar --envs SPRING_PROFILES_ACTIVE/openshift-catalog,...

or we could imagine to extend the command odo link to pass such info  or an Component Custom Resource containing such metadata could be used for that purpose
apiVersion: component.k8s.io/v1alpha1
kind: Component
metadata:
  creationTimestamp: null
  name: my-spring-boot
spec:
  deployment: innerloop
  envs:
  - name: SPRING_PROFILES_ACTIVE
    value: openshift-catalog
  exposeService: true
  runtime: springboot
  version: 1.5.16

@cmoulliard Here is the PR for that https://github.com/redhat-developer/odo/pull/626 . About to be merged :)

I will rebuild odo and make a test. Thanks

closing this one, as it is already implemented.
@cmoulliard if you find something missing in the current implementation feel free to open a new issue.


Crash when clicking citation before answer is ready.

Please provide us with the following information:

This issue is for a: (mark with an x)
- [ x ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

Add Custom documents, azd up
Ask a question
Click citation before answer is ready

Any log messages given by the failure

Unexpected Application Error!
Cannot read properties of undefined (reading '1')
TypeError: Cannot read properties of undefined (reading '1')
at Kn (https://randomwebsiteaddress.net/assets/index-9e08f79b.js:2:19543)
at dv (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:38:19518)
at cS (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:3139)
at AL (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:44788)
at yL (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:39750)
at yB (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:39678)
at Jd (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:39532)
at TS (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:40:35900)
at yy (https://randomwebsiteaddress.net)/assets/vendor-db55aa8e.js:40:36703)
at Ko (https://randomwebsiteaddress.net/assets/vendor-db55aa8e.js:38:3274)

Expected/desired behavior

When loading an answer from custom documentation, it would be really nice to have an load indicator that would indicate that the app is still preparing it answer as it tends to crash if citation is clicked before answer is ready or if an new question is prepared before answer is fully done.

OS and Version?

Windows  10

azd version?

1.6.1

Versions

Mention any other details that might be useful
I am trying to create an prototype with this template that retrieves technical information from one our manuals. It works great, but the manuals are very complex, so it takes a while to answer. If citations are clicked before they are ready, i get an crash.

Thanks! We'll be in touch soon.

I have seen that crash as well, and I think we should just fix that crash - an app shouldn't crash if you click at the wrong time. Thanks for filing!

This seems to occur when the citation is clicked before the answer object exists.
I think I have been able to fix pretty easily by the following change in chat.tsx, and a similar change in AnalysisPanel.tsx
{answers.length > 0 && activeAnalysisPanelTab && (
    <AnalysisPanel
        className={styles.chatAnalysisPanel}
        activeCitation={activeCitation}
        onActiveTabChanged={x => onToggleTab(x, selectedAnswer)}
        citationHeight="810px"
        answer={answers[selectedAnswer][1]}
        activeTab={activeAnalysisPanelTab}
    />
)}

into
{answers.length > 0 && activeAnalysisPanelTab && (
    <AnalysisPanel
        className={styles.chatAnalysisPanel}
        activeCitation={activeCitation}
        onActiveTabChanged={x => onToggleTab(x, selectedAnswer)}
        citationHeight="810px"
        answer={answers[selectedAnswer]?.[1]}
        activeTab={activeAnalysisPanelTab}
    />
)}


Clearer instructions please

Can you please give a more detailed and guided set of instructions for this demo?

I'm also looking for a more detailed step by step


Upgrading to Phoenix 1.7

I am in the process of upgrading from Phoenix 1.6 to 1.7 and I can see that pow currently only supports 1.6. When trying to start the server, I'm receiving this error:
== Compilation error in file lib/scorpion_web/router.ex ==
** (UndefinedFunctionError) function Phoenix.Router.Route.exprs/1 is undefined or private. Did you mean:

      * exprs/2

    (phoenix 1.7.0-rc.0) Phoenix.Router.Route.exprs(%Phoenix.Router.Route{verb: :get, line: 254, kind: :match, path: "/session/new", hosts: [], plug: Pow.Phoenix.SessionController, plug_opts: :new, helper: "pow_session", private: %{}, pipe_through: [:browser], assigns: %{}, metadata: %{log: :debug}, trailing_slash?: false, warn_on_verify?: true})
    (pow 1.0.27) lib/pow/phoenix/router.ex:216: Pow.Phoenix.Router.any_matching_routes?/3
    (elixir 1.14.0) lib/enum.ex:4313: Enum.reject_list/2
    (pow 1.0.27) lib/pow/phoenix/router.ex:188: Pow.Phoenix.Router.__filter_resource_actions__/6
    lib/scorpion_web/router.ex:254: (module)

@danschultzer this appears to also be a pow_assent issue as well, tagging you because it looks like you're working on both repos.

I ran into the same issue, however it looks like it was fixed in master with this commit, waiting on a release now.

Yup, fixed in main. The compiler warning in #683 has been fixed as well.
I want to update the template logic as well to handle the change from view to component templating in Phoenix 1.7 before releasing. I might just release this one ASAP so you can use a relased Pow with Phoenix 1.7 although it'll then still depend on phoenix_view.

Any thoughts on when there might be a release? Seems last was Apr 2022. Running into this issue as well.

@danschultzer any plan to release this one? I am also in the process of upgrading from 1.6 to 1.7.

@danschultzer Could you please cut a release with the current changes? In our testing, the latest version has worked with Phoenix 1.7. As you say, it would still depend on phoenix_view, but for us an earlier release without component templating would be the preference. The reason is that our legendary_core hex package is dependent on pow, and we can't publish a Phoenix-1.7-compatible version of the package until our dependencies are Phoenix 1.7 compatible.

@prehnRA Right, I've released 1.0.28!
I'm also working on https://github.com/pow-auth/pow/tree/phoenix-view to get rid of phoenix_view, and default styling with tailwind instead of minimalist. This will be introduced in 1.0.29 if anyone is waiting for switch to component templating.


Multi-asset coin selection additions

Issue Number

ADP-605
Overview

96c077d86c250322c116da06f8a90a203a9292d2
:round_pushpin: add few properties about filtering 'TokenMap'

01969b450c0147d7771ae45118c327c85b41f7fd
:round_pushpin: allows for multiple filters when selecting matching quantities
This will be crucial for Hardware devices and other clients who won't support MA initially. It is a only a best effort solution as this would only prioritize ada-only UTxO but eventually reach out to multi-assets UTxO if there's not enough ada-only utxo to cover for a particular payment. It is also generally good for other purposes to only select Ada utxo if we can to avoid needlessly moving tokens around.

bd2772651f24d192692130f0f148726990d097dd
:round_pushpin: Make 'prop_runSelection_UTxO_muchMoreThanEnough' blind about 'UTxOIndex'
Failures generate unreadable output which are pointless. For this test, it'll be better to rely on counterexample built for
analyzing failures.

4ee5c71b6e7d4eeb98de482f160a1f52716d6ee4
:round_pushpin: allow 'runSelection' to take Ada from an extra source
Although, it only does so if at least one input has already been selected. I've adjusted properties accordingly, generating an extra source when it made sense, and leaving it as 'Nothing' for some others.

c640a393a7970d3aefadce2a6e72393aa4bc2df6
:round_pushpin: revise 'makeChange' to take fees, deposits, withdrawals and minimum ada values into account
The strategy is as follows:

We start by computing the surplus amount coming from the difference
between the total requested outputs and the total incoming value.

From there, we start by creating as many change outputs as there
are outputs, and assign surplus tokens proportionally to the
quantity present in each outputs. That is, to create change outputs
that are following a similar distribution as the requested outputs.

For tokens which are not present in any requested outputs (coming
from selected inputs), we forward them into pre-existing change
outputs, coalescing small values as necessary so that there the
number of change outputs remains equal to the number of requested
outputs.

Then, we assign some coins from the surplus to each change output
so that each can meet the minimum coin value required. At this
point, we may return stop and return 'Nothing' if there's not
enough surplus coins to assign all required minimum to all change
outputs.

Then, we split the remaining coin amount across all change outputs,
proportionally to the size of each corresponding output.

b6c442b8883ae4cbfbafb004d57d18df10d50db0
:round_pushpin: WIP: update 'performSelection' to work with fees, withdrawals, deposits and minimum coin value
This is still WIP for the various properties related to
'performSelection' needs to be adjusted. New ones may also be added as
necessary.
It adds / tweaks quite a lot of the existing function but in essence it
is doing pretty much the same thing, with retries. Let's give some
details:

First of all, I've abstracted away as functions the functions that
will need to calculate the minimum coin value needed for each
output, and the one that would calculate the total cost (fee +
deposits). These are trivially implemented using functions from
the ledger and our current transaction layer.

Then, the main challenge is to construct a selection with change
outputs in such way that the cost and the minimum coin value have
been handled correctly.

As a starter, the algorithm augments every non-ada requested
output with a minimum coin value. We expect users who want to
transfer some asset A to only request a payment of asset A, and
not worry about how much ada is necessary. So we take care of it.
Requested outputs are also fully known, so we can do this step
early on before running the round-robin selection.

Then, we run the round-robin selection which, in case of success,
yields a set of inputs such that the sum of all inputs covers at
least for all requested outputs, possibly more.

Then, need to create change outputs with the excess from the
previous steps. However, we also need to make sure that we leave
enough money out of the change to pay for the transaction cost
(fees + deposits). Yet, the full cost of a selection is only
known after the selection has been made.

We start off by creating a pre-selection, that is, a selection
with change outputs when the cost and minimum coin value are
considered null. This is guaranteed to succeed because we have
already selected enough inputs to cover to all requested output.
The pre-selection gives us a baseline from which we can compute
the final cost.

Then, we repeatedly attempt to construct a final selection, this
time considering the actual cost (calculated from the
pre-selection) and minimum coin value. If it fails, we select
another ada-only input and recurse. The algorithm exits when we
run out of ada-only input or, when a final selection could be
successfully constructed. Note that for this to work, the cost
function must not be affected by a change in the change outputs
quantities. To enforce this, the cost function does not have to
full 'TokenBundle' but only to '[AssetId]' and should be able to
yield an approximate cost from this only. Adding more coins to a
change output should therefore not change the cost of a selection,
and ensure that the algorithm eventually converges.

bors r+

bors r+

bors r+

bors r+


Better experience for multi-module Android projects

As an engineer, I want to be able to run multiple test APKs in the same invocation, so I can save on setup/tear down time and test execution costs.
You can have multiple modules in an Android project, all of which can produce instrumentation app/test APKs. It would be nice to test multiple APKs one flank (and underlying firebase) invocation.
This is currently, partially, possible with the additional-app-test-apks flag. One can specify several app and test APKs and they are started in separate matrices in parallel. The limitation of this approach is that tests can not be sharded across modules, a device needs to be started for each module.
It would be great if Flank could merge test APKs instead of mapping each APK to a separate matrix.

I think the root limitation is that FTL only allows one test apk per matrix.

Yeah I think so. I wonder if there's some way to merge androidTest APKs and upload just the merged one.
This may be a bigger challenge for application modules where there is an app APK and a test APK, but may be easier for library modules where there's just a test APK.

I am afraid that this feature is currently impossible to implement. But maybe will be after successful integration with Corellium, we will see.


Implement split_in_batches

Example:
def split_in_batches(self, itr: Iterable[T], batch_max_size: int = BATCH_SIZE) -> Generator[slist[T], None, None]:
        batch = slist()
        for el in itr:
            batch.append(el)
            if batch.size() == batch_max_size:
                yield batch
                batch = slist()
        if batch.size():
            yield batch

Implemented in stream.batch(). Covered by tests & used in prod code.


All account management functions broken

I'm trying to upload a package using python3 setup.py sdist upload. It fails:
error: Upload failed (400): User 'whitequark' has no verified email addresses, please verify at least one address before registering a new project on PyPI. See https://upload.pypi.org/help/#verified-email for more information.

Sure. The document it links to says:

You can manage your account's email addresses in your Profile. This also allows for sending a new confirmation email for users who signed up in the past, before we began enforcing this policy.

The Profile link is broken and leads to a 404 page.
Sure. Maybe I could reset my password instead. The help page says:

You should be able to reset your password on the site; you'll receive an email with a password reset link.

The reset password link is also broken and leads to a 404 page.
In addition there are no account management functions whatsoever linked from anywhere on upload.pypi.org.

Ah, the link to Profile should be to pypi.org, not upload.pypi.org.

That worked, thanks!

Looks like this is a bug in how we're generating URLs in our error messages. Thanks for the report!

I think the URL in the error is fine? At least, it correctly leads me to the help page. It's the others that return 404.

The help page is actually at https://pypi.org/help/, the upload domain shouldn't be displaying it.

This affected another user: https://github.com/pypa/packaging-problems/issues/137#issuecomment-379971088


Initial commit of sphinx template

@gassmoeller for review.
This is the initial commit for the sphinx documentation template.
Some things to fix if wanted:

Adding a logo on the sidebar. I could not get this to work ...
Directory structure reorganization. I think there is a better way to organize this.  Suggestions welcome.

And of course there are many improvements that can be made in the future by others but this should be a good start.

As discussed this will be redone in a new PR.


According to user location so the country flag must be added and store into back-end in Firebase

Steps to Reproduce

...
...
...

Logs

@manu223 seems you earlier opened similar issue.
Please see comment there for code assistance platforms.
Closing this as non-issue.


ChromiumFXRenderProcess.exe crash problem

Recently, we found our app will show blank screen after running in 4K screens for some time.
We are using large images for 4K screens which may cause more cpu and memory using.
Memory using of ChromiumFXRenderProcess.exe will growing up to about 400M and then the process will go missing which causes blank screen.
And sometimes, ChromiumFXRenderProcess.exe will not be killed after the main application exited.
It seems ChromiumFXRenderProcess.exe is uncontrollable.
I found ChromiumFX has some new release after 3.3440.x, will these updates be useful?
What do you suggest to locate the problem?
Thanks!

You may use Chromium inspector memory tabs to try get more information of the problem
Are you using 64 bits builds this may help with memory problem.
The updates may be useful as Chromium keeps improving its memory performance, I will provide in the short/mid-term.

It will be also nice if you couls list your application here: https://github.com/NeutroniumCore/Neutronium/issues/12

Thanks for replying.
We reduced the size of these large images yesterday and never see blank screens again.
I will go digging this problem after our project launched.

Update:
Using 64 bits builds solve this problem.
Problem I found while using 64 bits builds:
#127


[Network][CDN] fix misleading sample for cmd cdn endpoint purge

Related command

az cdn endpoint purge
Description

The sample given in help menu mislead user.

Testing Guide

az cdn endpoint purge --resource-group edge-cloud-config-platform --profile-name EdgeProdCDN --name edge-consumer-static --content-paths /
will raise the error that failed the command

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

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

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Network


Support for Koa v2

changed koa version in package.json
replaced generator functions with async functions (and this with ctx param)
changed package version in package.json

friendly bump


Add ability to specify audience in auth config passed to auth provider

Is your feature request related to a problem? Please describe.

$subject
It's possible to define multiple audiences for an application in the WSO2 Identity Server/Asgardeo console, but there is no way to specify such audiences in the auth config passed to authprovider component in react SDK.
Describe the solution you would prefer

An option to specify audience/s

Hey @NipuniBhagya assign me this please

@pavinduLakshan it isn't clear to me, so can you explain it more with an explain if can ? I will try to solve it


Shapeless build fails in recent 2.11.x builds

[shapeless] [error] **** Missing dependency: the library org.scalamacros#paradise is not provided (in space "default") by any project in this configuration file.
[shapeless] [error] In order to control which version is used, please add the corresponding project to the build file
[shapeless] [error] (or use "check-missing:false" to ignore (not recommended)).
[shapeless] java.lang.RuntimeException: Required dependency not found

last 2.11.x-jdk6 success: d6a4f56e0783655c48244ec23c4f4c2d98401a72 (Jan 4)

first 2.11.x-jdk6 failure: 7c0a87364c5d990fc50923dfd0af8aaada4683c2 (Jan 14)

last 2.11.x-jdk8 success: same as above

first 2.11.x-jdk8 failure: 2ce5d7dfa1fac107b277bee862708ff0c9ac71ca (Jan 13)

% git hist 2ce5d7dfa1fac107b277bee862708ff0c9ac71ca ^d6a4f56e0783655c48244ec23c4f4c2d98401a72
* 2ce5d7d - Revert "Updated Sonatype realm." (26 hours ago) <Miles Sabin>
*   8075ac4 - Merge pull request #531 from dwijnand/non-fatal-console (26 hours ago) <Miles Sabin>
|\
| * 454c51a - Disable -Xfatal-warnings in console (28 hours ago) <Dale Wijnand>
|/
*   23ace5c - Merge pull request #528 from dwijnand/CompatContext-is-a-runtime.Context (34 hours ago) <Miles Sabin>
|\
| * 07f4aba - Drop c2 in CaseClassMacrosMixin (compat.scala) (2 days ago) <Dale Wijnand>
|/
* 495e275 - Updated Sonatype realm. (2 days ago) <Miles Sabin>
* 0e2a141 - Refreshed Travis env vars. (2 days ago) <Miles Sabin>
* d16892e - Merge pull request #515 from dwijnand/uni (2 days ago) <Miles Sabin>
* e387788 - Add myself (Dale) to compat.scala copyright holders (2 days ago) <Dale Wijnand>
* 93413fc - Switch to contiguous copyright date ranges (2 days ago) <Dale Wijnand>
* aede308 - Add macro-compat to enable Scala 2.10 in master (3 days ago) <Dale Wijnand>
* 81274d7 - Misc cleanups (3 days ago) <Dale Wijnand>

looks like https://github.com/milessabin/shapeless/commit/aede308fc6fb22827a8bab0881f80adf6b4e88b6 is the culprit

note that scalamacros/paradise used to be part of the community build, but was removed in https://github.com/scala/community-builds/commit/ecae16dbe68c024b68704c9658a1b475d674bb13

test run w/ paradise added: https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/190/console

adding paradise wasn't enough, needed to add milessabin/macro-compat as well.
this led to:
[macro-compat] [error] **** Missing dependency: the library org.scalamacros#quasiquotes is not provided (in space "default") by any project in this configuration file.
[macro-compat] [error] In order to control which version is used, please add the corresponding project to the build file
[macro-compat] [error] (or use "check-missing:false" to ignore (not recommended)).

for reasons I don't understand. the logic in https://github.com/milessabin/macro-compat/blob/master/build.sbt for only including the quasiquotes dependency on 2.10 seems correct to me:
> eval CrossVersion.partialVersion("2.11.8-dbuildx1df2747ab4cf758a246c29f4688c6bdee4ad043d")
[info] ans: Option[(Int, Int)] = Some((2,11))

fyi @milessabin

to get the build green again, in #204 I froze Shapeless at d6a4f56e0783655c48244ec23c4f4c2d98401a72 for now

@gkossakowski any insight?

The change in https://github.com/milessabin/shapeless/commit/aede308fc6fb22827a8bab0881f80adf6b4e88b6 is a switch from a multi-branch build (ie. separate branches for 2.10.x and 2.11.x+) to a single branch build using macro-compat. macro-compat is needed as a dependency for 2.11.x+ but that should be incredibly lightweight ... just enough to completely eliminate an @bundle annotation.
We're currently cross-building for 2.12.0-M3 without a problem ... I'm not clear what the issue is here?

Is macro-compat itself part of the community build?

It's a shame macro-compat isn't in the community builds as I guess it would also be broken, and would've broken long before this ticket. And it would've either be broken before adding it, or we would've been able to bisect to what broke it..

@milessabin jinx, no it's not: https://github.com/SethTisue/community-builds/blob/f805766e371e10719a1157b652202e4200d15c1a/common.conf

Probably not useful given dbuild uses the sbt builds instead of the artefacts, but there's no macros paradise in the poms:

https://oss.sonatype.org/content/repositories/snapshots/com/chuusai/shapeless_2.11/2.3.0-SNAPSHOT/shapeless_2.11-2.3.0-SNAPSHOT.pom
https://oss.sonatype.org/content/repositories/snapshots/org/typelevel/macro-compat_2.11/1.1.1-SNAPSHOT/macro-compat_2.11-1.1.1-SNAPSHOT.pom

Any more info on this?

I heard a rumour that you can't have builds in the community build without all its dependencies also in the community build. So perhaps the next step is to add macro-compat?

Is macro-compat itself part of the community build

it wasn't, but I tried adding it as part of my attempt to fix this. I already described this in https://github.com/scala/community-builds/issues/202#issuecomment-171804434, you might want to give that a closer re-read

@SethTisue One suspect, to me at least, is the log line:
[macro-compat] [info] Setting version to 2.10.6

It looks to me like it wants to test building everything against scala version 2.11.8-dbuildx1df2747ab4cf758a246c29f4688c6bdee4ad043d but then for some reason it switches the macro-compat project to 2.10.6.. and then fails to find quasiquotes?
Does that looks suspect to you? Do you know why it might be doing that?

macro-compat has a .sbtrc which contains,
alias boot = ;reload ;project testJVM ;++2.10.6 ;iflast shell

Could that be the issue? It doesn't cause a problem on Travis.

Good find, yeah it could very well be.
For reference that's there for the development convenience of test compiling and running tests under 2.10.6.

Oh and the convenience of test compiling and running tests only for the JVM, not Scala.js, during development.

@SethTisue That's been merged now ... could you give it another try?

new test run: https://scala-ci.typesafe.com/job/scala-2.11.x-integrate-community-build/198/console

That seems to have worked?
[macro-compat] Publishing: testJVM
[macro-compat] --== End Building macro-compat ==--

[shapeless] Publishing: shapeless
[shapeless] --== End Building shapeless ==--

merged #206. thanks, Dale and Miles! nice not just to get the regression fixed, but to get two new projects in.

Cool :)


Add softlayer provider

Introducing IBM SoftLayer provider into ManageIQ.
What's included:

CloudManager

Refresher (VMs, templates, auth key pairs, flavors)
Provisioning from template
VM power operations (start/stop/terminate)

NetworkManager

Refresher (port, router, private network, public network, subnet)

Discovery
UI (add provider, discover)

For more information ping me, @Ladas or @martinpovolny .

@tumido Thanks for creating this PR!
Since this PR is quite large and extensive, the review may take quite a few weeks.  Also, we may ask you to split out parts of this PR into smaller, stand-alone PRs that will be easier to review and merge.
Thanks again!!

@chessbyte I'm aware of that and I also feel a bit sorry for any inconvenience that you might face during the PR review. I know it's huge. I can try to split it, on the other hand all the parts are quite connected and linked together. I'm in touch with @Ladas though who was fine with such a huge pull request.(volunteer for the review? haha) If you want me to adjust anything just let me know. :)

If you guys will want to split, should be possible to separate refresh, discovery, provisioning. Not sure what else, we can't divide it too much and still push working pieces. :-)
@tumido Of course I volunteer. :-)

@tumido : please, start looking into the Rubocop issues.
e.g. ```Do not prefix reader method names with get_.`` is relevant.
We can help you setup Rubocop locally if you want.

@martinpovolny I take a look at that. But I'm not sure especially the mentioned ones are relevant. Unfortunately this is a naming convention used across all providers. We can discuss it tomorrow. :)

Right. So just fix the obvious ones + the failing specs ;-)

@tumido we should add specs, that will cover the added code. :-) Search for refresh specs for other providers, plus controller and listnav specs should be there. Also specs for the whole provisioning process. Get inspiration with the other cloud providers. :-)

@tumido can you look at some of the Rubocop warnings.  Specifically any of the following:

Operator => should be surrounded by a single space.
Use hash rockets syntax.

@tumido can you look at some of the Rubocop warnings. Specifically any of the following:

Actually these errors, "Operator => should be surrounded by a single space." in Haml files would have been removed with my Rubop PR that was merged. As this is a bug with the Haml validations.

@tumido please rebase and push again

Looks like this PR has gone stale for now.  I'll close it and if there's time to get back to it, we can reopen.

Hey all! Is anyone still interested in this? Would it make sense to move this to an "out-of-tree" plugin repo?

Haha, obviously we may start considering this again.
Well, I'm not sure if we capacity to rewrite this and maintain a new provider now... It would need to be rewritten as a plugin + accommodate to all the changes with happened in last 2+ years... :confused:
cc @bronaghs @Ladas @agrare

Well in the light of the recent changes it might be worth reviving ;-)

Hey @tumido any new provider should be built using the provider generator in a new plugin repo.
@martinpovolny I'm not sure what recent changes you mean :wink: but we should not alter our roadmap/priorities for the project

@agrare that was rather a joke, than anything else. :wink:  This PR is over two years old. It comes from the ancient years before ManageIQ was split into more repos. Also I don't have any time myself to rewrite it. But if anyone is interested, feel free to use it...


[ChatGPT] Need GPT-4-32K

Extension
https://www.raycast.com/abielzulio/chatgpt
Description
All the models are available, only GPT-4-32K is missing. I hope it can be provided, thank you.
Who will benefit from this feature?
No response
Anything else?
No response

The latest version already supports.


Can't run latest Flancy as Job anymore

Repro script:
Start-Job -Name 'Flancy' -ScriptBlock {
    Import-Module -Name 'Flancy' -ErrorAction Stop
    New-Flancy -WebSchema @{
        Method = 'Get'
        Path= '/'
        Script = {'Hello, world!'}
    }
}

Wait-Job -Name 'Flancy' | Receive-Job

Output:
PS > .\Flancy-AsJob.ps1

Id     Name            PSJobTypeName   State         HasMoreData     Location             Command
--     ----            -------------   -----         -----------     --------             -------
2      Flancy          BackgroundJob   Running       True            localhost            ...

Exception calling ".ctor" with "1" argument(s): "Unable to resolve type: Nancy.ViewEngines.ViewEngineApplicationStartup"

    + CategoryInfo          : InvalidOperation: (:) [New-Object], MethodInvocationException
    + FullyQualifiedErrorId : ConstructorInvokedThrowException,Microsoft.PowerShell.Commands.NewObjectCommand
    + PSComputerName        : localhost

You cannot call a method on a null-valued expression.

At .\flancy\flancy.psm1:395 char:9
+         $flancy.start()
+         ~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (:) [], RuntimeException
    + FullyQualifiedErrorId : InvokeMethodOnNull

PowerShell version:
Name             : Windows PowerShell ISE Host
Version          : 5.0.10018.0
InstanceId       : 03eac871-dcbd-4e6a-8c7c-b5c018dbcda8
UI               : System.Management.Automation.Internal.Host.InternalHostUserInterface
CurrentCulture   : ru-RU
CurrentUICulture : en-US
PrivateData      : Microsoft.PowerShell.Host.ISE.ISEOptions
DebuggerEnabled  : True
IsRunspacePushed : False
Runspace         : System.Management.Automation.Runspaces.LocalRunspace

This doesn't happen with this version of Flancy.

I think this is the same as issue #32

@h0rnman Agree. Curious, that I have no issues while running Flancy in PS console/ISE, only as job.

Is your ISE environment importing a different build of the module perhaps?  I am seeing the error in both ISE and console.

I've nailed it! It all comes down to the Flancy's Path parameter. When Flancy is run in the interactive PowerShell session and Path parameter is not specified, it will be set to the current PowerShell's directory. But if you run it as a job, $PWD points to the current user's My Documents folder  (C:\Users\Administrator\Documents in my case).
Here is the code in the Flancy.psm1 that sets it:
if (!$path) {
    $path = ''
    if ($MyInvocation.MyCommand.Path) {
        $path = Split-Path $MyInvocation.MyCommand.Path
    } else {
        $path = $pwd -replace '^\S+::',''
    }
}

And that's where weird things start to happen: Nancy tries to access folder C:\Users\Administrator\Documents\My Music which doesn't exist!
System.UnauthorizedAccessException: Access to the path 'C:\Users\Administrator\Documents\My Music' is denied.
    at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
    at System.IO.FileSystemEnumerableIterator`1.AddSearchableDirsToStack(SearchData localSearchData)
    at System.IO.FileSystemEnumerableIterator`1.MoveNext()
    at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
    at System.IO.Directory.InternalGetFileDirectoryNames(String path, String userPathOriginal, String searchPattern, Boolean includeFiles, Boolean includeDirs, SearchOption searchOption, Boolean checkHost)
    at System.IO.Directory.InternalGetFiles(String path, String searchPattern, SearchOption searchOption)
    at Nancy.ViewEngines.DefaultFileSystemReader.GetFilenames(String path, String extension)
    at System.Linq.Enumerable.<SelectManyIterator>d__14`2.MoveNext()
    at System.Linq.Enumerable.<DistinctIterator>d__81`1.MoveNext()
    at System.Linq.Enumerable.WhereSelectEnumerableIterator`2.MoveNext()
    at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
    at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
    at Nancy.ViewEngines.DefaultFileSystemReader.GetViewsWithSupportedExtensions(String path, IEnumerable`1 supportedViewExtensions)
    at Nancy.ViewEngines.FileSystemViewLocationProvider.GetViewsFromPath(String path, IEnumerable`1 supportedViewExtensions)
    at Nancy.ViewEngines.DefaultViewLocator..ctor(IViewLocationProvider viewLocationProvider, IEnumerable`1 viewEngines)
    at lambda_method(Closure , Object[] )
    at Nancy.TinyIoc.TinyIoCContainer.ConstructType(Type requestedType, Type implementationType, ConstructorInfo constructor, NamedParameterOverloads parameters, ResolveOptions options)

I'm also able to reproduce this in the PowerShell session (not job). It looks like Nancy is trying to traverse down the given path, because when I set Path to the root of my drive (E:\), exception is thrown about E:\$RECYCLE.BIN\S-1-5-18.
To get this details I've modified Flancy.psm1 to catch and unwind inner exceptions when creating new Flancy object. This gives you the full list of the exceptions, instead of the outer one.
Before:
$flancy = new-object "flancy.flancy" -argumentlist $url

After:
try
{
    $flancy = new-object "flancy.flancy" -argumentlist $url
}
catch
{
    function Unwind-Exception
    {
        Param($Exception)

        if($Exception.InnerException)
        {
            $Exception.InnerException.PsObject.Properties | Select-Object -Property Name, Value
            Unwind-Exception $Exception.InnerException
        }
    }

    Unwind-Exception $_.Exception.InnerException
}

I will include the exception unwinding in flancy.psm1 in the devel branch.  Doing that now.  If you set your directory to something else in the job, is it working?  I'd like to understand how to make it work.  I'd also like to understand what's causing it.

never mind - I duplicated and was able to get it to work with this:
Start-Job -InitializationScript {import-module c:\flancy\flancy.psd1} -ScriptBlock {cd c:\flancy; new-flancy;while ($true) {sleep 60}}
I'm thinking the best way to fix this would be to create an asjob parameter.  I could then do the following:

load the module in the initialization script from get-module -listavailable flancy |select -expandproperty modulebase
Set the current directory in the job to the current directory
Pass any arguments as a single argument to the scriptblock
Pass the arguments to the scriptblock to new-flancy
Create the loop with a 60 minute sleep

Confirmed, this appears to be a bug in Nancy, not flancy.  You cannot seem to set the root path to c:\users\username\documents.  I attempted to compile it with that root path and it errors.  Actually, I found that any directory (such as root drives due to the recycle bin) where a subfolder gives access denied appears to error.  i'm unsure whether or not I can use the following to validate the rootdir just now.
Get-ChildItem c:\users\tome\documents -Depth 1 -force -Recurse -ea stop
It will work for my docs and for c-drive, but I'm unsure if the depth goes further in Nancy.  For now, I wiil try adding the validation of the path with the above and catch the error to notify people that they cannot use that directory and to use the -path parameter to override the path.
I opened this in Nancy's issue list: https://github.com/NancyFx/Nancy/issues/2146

OK - We should be able to close this pending validation that @beatcracker can implement a workaround with -Path and that he now receives a better error message when not using -Path in a job (or changing directory in the job itself prior to calling new-flancy).  Once confirmed, we'll close this, and then we'll implement asjob per Issue #25

I'm able to run Flancy as job if I specify path without inaccessible directories and I receive a new error message if path is contains such inaccessible directories.
But we have to do something about the validation code, because it's very slow - it tries to enumerate all directories/subdirectories in given path. In my case it takes 3 seconds to start Flancy as job when Path targets an empty directory and 25 seconds when Path is set to the drive root (+ heavy HDD activity).

Well, we could opt for an easier way - I think it might be better anyway:  Let's ensure that we are not in the root of a drive or that we are not in $env:userprofile\documents or $enf:userprofile\my documents.  As long as path doesn't match those 3, chances are the person is in an ok directory.  AND if they are not, they will get the normal access denied errors thanks to your update.
I think that's fair.

Yes, I was thinking along those lines. In fact, I don't even think that we should block user from supplying those paths, just check and Write-Warning if they match.

Just figured out why accessing My Music generates exception in Nancy (of course, it's a permissions issue, but a curious one): https://github.com/NancyFx/Nancy/issues/2146#issuecomment-162368570


[CPO] Update go version to 1.14.4

This PR updates go version to 1.14.4 for all CPO jobs.

@wangxiyuan could you review this?


[SPARK-25470][SQL][Performance] Concat.eval should use pattern matching only once

What changes were proposed in this pull request?
The PR proposes a solution for a problem described here.
How was this patch tested?
Run the existing tests for Concat expression.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Test build #96277 has started for PR 22471 at commit 9051c23.

@ueshin can you review?

Test build #96277 has finished for PR 22471 at commit 9051c23.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96277/
Test PASSed.

@mn-mikke As you pointed out, I think we need to fixReverse, too. Do you have time to do?

LGTM except for the minor comments

If you don't mind, I will include fixes for Reverse and ElementAt to this PR.

Test build #96340 has started for PR 22471 at commit 9e153fd.

LGTM.

@maropu Could you take a look at this again please?

Test build #96340 has finished for PR 22471 at commit 9e153fd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96340/
Test PASSed.

ok

@maropu Do you want to merge this as your first work as a committer?
I think this can be merged into master/2.4 because this is a performance regression fix.

away from keyboard now, so will do when I’m back. Thank!

Thanks! merging to master/2.4.


Customizing the layout

It would be great to be able to customize the layout / html / css. Something like adding a custom component in .storybook.

Could you give us more information? What you are trying to achieve?

I would like to be able to customize the layout, make the sidebar smaller... I tried to inject css in config.js and use global selectors, but it did not work, and seems hackish anyway.
I like to have my code and storybook side by side, and on my screen the current design takes a lot of space. I'm sure there could be many other use cases !

I made a PR in case you find this useful.

If someone is interested, this can already be done using a custom webpack config file :
module.exports = {
  //...
  plugins: [
    new webpack.NormalModuleReplacementPlugin(/^\.\/layout$/, 'custom-layout'),
    new webpack.NormalModuleReplacementPlugin(/^\.\/controls$/, 'custom-controls')
  ],
  resolve: {
    alias: {
      'custom-layout': path.resolve('.storybook/layout.js'),
      'custom-controls': path.resolve('.storybook/controls.js')
    }
  }
}

Great. I added a comment on this PR. https://github.com/kadirahq/react-storybook/pull/67


Presetupdate

Added presets including 12 categories.
Added more description for each action.
Updated Help and Readme.

Tested and works.


Multiple errors reported by Sentry

I have the latest version of monaco-editor installed used with React wrapper, and it's working as expected when we test it.  But when it used in production, we get lots of handled and unhandled errors logged by Sentry:
TypeError: Cannot read property 'parent' of undefined
    at bT (https://cdn.jsdelivr.net/npm/monaco-editor@0.22.3/min/vs/language/typescript/tsWorker.js:21:1520827)

RangeError: Maximum call stack size exceeded
    at u (https://cdn.jsdelivr.net/npm/monaco-editor@0.22.3/min/vs/language/typescript/tsWorker.js:21:770316)

Canceled canceled(npm/monaco-editor@0.22.3/min-maps/vs/editor/vs/editor/file:/Users/alex/src

TypeError: Cannot read property 'kind' of undefined
    at Object.e.isNewExpression (https://cdn.jsdelivr.net/npm/monaco-editor@0.22.3/min/vs/language/typescript/tsWorker.js:21:730631)

I don't have clear ways of reproducing those errors, but we get them reported every day.
OS: WIndows 10
Browser: Chrome 89
monaco-editor version 0.22.3

facing same problem.
for now using workaround mentioned in #2382


cudnnPackages: Remove statically linked .a files.

Motivation for this change
The latest version of CUDNN is really huge.  It is distributed with both statically linked and dynamically linked libraries.
This PR adds an option to remove the statically linked libraries.  This is nice if you don't actually use the statically linked libraries.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Without setting removeStatic to true, the size of cudnn_cudatoolkit_11_2 is about 3G
$ nix-build -A cudnn_cudatoolkit_11_2
/nix/store/chncvxzv80p006i1ajz4mpq9xjkv9pmm-cudatoolkit-11.2-cudnn-8.1.0
$ nix path-info -s -h /nix/store/chncvxzv80p006i1ajz4mpq9xjkv9pmm-cudatoolkit-11.2-cudnn-8.1.0
/nix/store/chncvxzv80p006i1ajz4mpq9xjkv9pmm-cudatoolkit-11.2-cudnn-8.1.0           2.9G

With setting removeStatic, it is about 1.5G:
$ nix-build -E 'with import ./. {}; (cudnn_cudatoolkit_11_2.overrideDerivation (origDrv: { XXX = origDrv; })).XXX.override { removeStatic = true; }'
/nix/store/fsw48xz2r3k42lyygzi0j1ci90xi1y7l-cudatoolkit-11.2-cudnn-8.1.0
$ nix path-info -s -h /nix/store/fsw48xz2r3k42lyygzi0j1ci90xi1y7l-cudatoolkit-11.2-cudnn-8.1.0
/nix/store/fsw48xz2r3k42lyygzi0j1ci90xi1y7l-cudatoolkit-11.2-cudnn-8.1.0           1.4G

This is a pretty nice space saving.

There is something somewhat unfortunate with the interaction of cudnnPackages and callPackages.  You can't directly use .override on packages like cudnn_cudatoolkit_11_2.
cudnnPackages is defined with callPackages here:
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/top-level/all-packages.nix#L3514-L3527
pkgs/development/libraries/science/math/cudnn/default.nix has the following arguments (basically just the various cudatoolkit versions):
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/default.nix#L1
Each of the cudnn_cudatoolkit_* packages is defined with callPackage in the above file:
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/default.nix#L3-L21
The cudnn generic.nix args look like the following:
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/generic.nix#L1-L23

The problem here is that the .override on the packages produced with callPackages gives you the ability to override arguments passed to callPackages.  In the case of cudnn, it gives you the ability to override these:
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/default.nix#L1
I could do something like this:
$ nix-build -E 'with import ./. {}; cudnn_cudatoolkit_11_2.override { cudatoolkit_11_2 = cudatoolkit_11_2; }'
/nix/store/chncvxzv80p006i1ajz4mpq9xjkv9pmm-cudatoolkit-11.2-cudnn-8.1.0

However, .override does not give you the ability to override the arguments to the generic.nix callPackage defined here:
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/generic.nix#L1-L23
So this doesn't work:
$ nix-build -E 'with import ./. {}; cudnn_cudatoolkit_11_2.override { fetchurl = fetchurl; }'
error: anonymous function at pkgs/development/libraries/science/math/cudnn/default.nix:1:1 called with unexpected argument 'fetchurl', at lib/customisation.nix:170:63

This is quite surprising.
You can see how I worked around this in https://github.com/NixOS/nixpkgs/pull/115768#issuecomment-796388639 by using overrideDerivation to get a references to the underlying cudnn_cudatoolkit_11_2 derivation, so I could call the actual .override function.

My suggestion here would be to either restructure
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/default.nix#L1
so it gets all the arguments defined in
https://github.com/NixOS/nixpkgs/blob/eb0952e45e78443248be9b26301acfd269274493/pkgs/development/libraries/science/math/cudnn/generic.nix#L6-L11
so that the arguments are easily overridable.
Or, just stop using callPackages for cudnnPackages.
Although, I don't think problem should affect whether or not this PR gets merged in, since it is sort of orthogonal.

It has been about a week, so I'm going to go ahead and merge this in.
@mdaiter If there is anything you'd like changed in this, please let me know and I'll send a follow-up PR.


Class names reference wrong icon character (sorting issue)

In a project that has been using grunt-webfont for a couple years without issue, we are now seeing an issue where the order in which class names are listed in the generated CSS does not match the order of the font characters in the generated font files.
The issue appears to be specific to the sorting rules for the class names in the CSS file. For example, the files:
circle.svg
circle-1.svg
circle-2.svg

are ordered in the font file in the order shown above but in the CSS (Stylus in this case) in the order:
$icon-circle-1 = "\e634"
.icon-circle-1:before
  content:"\e634"

$icon-circle-2 = "\e635"
.icon-circle-2:before
  content:"\e635"

$icon-circle = "\e636"
.icon-circle:before
  content:"\e636"

So a CSS reference to .icon-circle-1 actually renders the font charactercircle.svg as it is first in the font file.
We are using fontforge and it is not clear if the root of the issue is with a recent release of fontforge. Reverting to an older version (we were not seeing these issues the last time we generated web fonts in July 2017) produced errors that prevented running grunt-webfont at all.
Grunt task looks like:
webfont: {
      icons: {
        src: 'public/svg/icons/*.svg',
        dest: 'public/static/fonts',
        destCss: 'client/css/styleguide',
        options: {
          hashes: false,
          htmlDemo: false,
          embed: 'ttf,woff',
          startCodepoint: 0xE600,
          stylesheet: 'styl',
          relativeFontPath: '/static/fonts',
          template: './tasks/icons.template.css',
          templateOptions: {
            baseClass: 'icon',
            classPrefix: 'icon-',
          },
        },
      },
    },

Posting this issue as I have not found a similar open issue for this project or for fontforge and it appears to be new in the last 6 months or so. There is a closed issue https://github.com/sapegin/grunt-webfont/issues/91 that appears related but it is very old (2014).

FYI @budleigh

Are you using the codepoints option? If not, I can you try it and see if that corrects the issue?

This happens regardless of engine, I switched to node and the same bug happens, this is a hassle to my workflow. I will try the codepoints option @alampros

Hmm, it doesn't seem to change anything


Fix Catalog ACL Creation

Currently the ACL creation is failing because the ermrest api has been deprecated. Instead we should use the new API.

Done as part of this.


tfa.seq2seq: Error in AttentionWrapperState when the Decoder has Stacked RNN cells.

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):  Linux Ubuntu 20
TensorFlow version and how it was installed (source or binary): Source - 2.6.0
TensorFlow-Addons version and how it was installed (source or binary): Source - 0.14.0
Python version: 3.7
Is GPU used? (yes/no): yes

Describe the bug
I want to build a seq2seq model with a Decoder with multiple RNN Cells. I have been following tutorials and they all use only a single layer RNN LSTM. When I try to change just this, it raises an error coming from the AttentionWrapperState specifically:
I changed:
self.decoder_rnn_cell = tf.keras.layers.LSTMCell(self.dec_units)
to:
self.decoder_rnn_cell = tf.keras.layers.StackedRNNCells([tf.keras.layers.LSTMCell(self.dec_units) for _ in range(3)])
and the error obtained is:

TypeError: The two structures don't have the same nested structure.
...
Entire first structure:
AttentionWrapperState(cell_state=([., .], [., .], [., .]), attention=., alignments=., alignment_history=(), attention_state=.)
Entire second structure:
AttentionWrapperState(cell_state=[., .], attention=., alignments=., alignment_history=(), attention_state=.)

To reproduce the error you can check the classic tutorial on the Tensorflow page, in the Decoder model part, line 16: Seq2Seq Translation Tutorial
Thank you in advance and I hope you can help me.

I think the code expects the encoder and decoder to have the same number of RNN layers (and so the same state structure). You should either use the same number of cells in the encoder, or update the code that initializes the decoder state from the encoder state.

Thank you very much, you are right.
For the encoder model:

And for the Decoder states initialization:


Use same prettier config as Kiali and core-ui projects

Hi @ferhoyos , I noticed a lot of prettier warning when I created an initial vscode project for ossmc.  Not sure if it was intentional to have a different prettier config for OSSMC, but if not let's use the same as Kiali and core-ui.  Otherwise we should do a big reformat of the OSSMC repo.

To be consistent, I think we should use same prettier config in all kiali repositories. Even we should use same prettier file format (I don't know why kiali upstream use json format but OSSMC prettier file is defined as yaml).


content error when contain tabview

When my content contain tabview .i could not click on tabitem

Hi,
Can you share your code, Xcode version and SwiftUIMenu version, please? It's working for me:
var body: some View {
    TabView {
        menu.tabItem { Text("Tab 1") }
        Text("Another Tab").tabItem { Text("Tab 2") }
    }
}

var menu: some View {
    NavigationView {
        Menu(...)
    }
}

And the result:

Have in mind that Menu should be used as a tab, not to wrap TabView.

thanks for your reply.in my case.I want to menu contains 2 tabs.click on tab 2 menubutton still exist to open and close menu.any suggest?

Hi @huongtran84 ,
I released version 1.1.1. Try this version and call disableContentTap. It should work like that.

Hi.I face a bug that each close or open menu content view always reload .what is way prevent it?

Hi @huongtran84 ,
Please, open a new issue for that. Have in mind that in SwiftUI, Views get redrawn when a state variable changes. In this case, Menu has several bindings and states that will trigger an update, that's right.
Let me know if this thread is closed as I'd like to close it.
Thanks


duplicity: add missing deps for S3

Motivation for this change
At some point Duplicity added support for the boto3 library, but we didn't add the dependency on the library so it's broken.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)

It adds ~52 megs, which is kind of ridiculous.

[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

There's boto and boto3 in this list, which sounds suspicious.

There are two backends. s3:// URLs use the older boto2 backend and
boto3+s3:// URLs use the newer boto3 backend.
On Sat, Nov 21, 2020 at 11:24 AM Vika<EMAIL_ADDRESS>wrote:

There's boto and boto3 in this list, which sounds suspicious.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/104089#issuecomment-731624764, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPATLAE7LOCPVOACLJA6LSRAHYXANCNFSM4TZCHSBA
.

Is there anything blocking this? How can I help? I have being running with boto3 for a long time without any issues, but tests take so long to run, having duplicity+boto3 cached would be great.

Is there anything blocking this? How can I help? I have being running with boto3 for a long time without any issues, but tests take so long to run, having duplicity+boto3 cached would be great.

AFAIK we're just waiting on somebody with a commit bit.
On Sun, Jan 3, 2021 at 8:09 PM Sebastián Galkin<EMAIL_ADDRESS>wrote:

Is there anything blocking this? How can I help? I have being running with
boto3 for a long time without any issues, but tests take so long to run,
having duplicity+boto3 cached would be great.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/104089#issuecomment-753745959, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPATJ5IKQDE6E42TG4RKTSYE5QHANCNFSM4TZCHSBA
.

AFAIK we're just waiting on somebody with a commit bit.
On Sun, Jan 3, 2021 at 8:09 PM Sebastián Galkin<EMAIL_ADDRESS>wrote:

Is there anything blocking this? How can I help? I have being running with
boto3 for a long time without any issues, but tests take so long to run,
having duplicity+boto3 cached would be great.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/104089#issuecomment-753745959, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAPATJ5IKQDE6E42TG4RKTSYE5QHANCNFSM4TZCHSBA
.


Remove dependency on time from chrono.

It would be good to get rid of the time dependency pulled in by chrono because: a) https://rustsec.org/advisories/RUSTSEC-2020-0071 and b) it's not needed.

@zslayton I figured out that the "clock" feature needs to be enabled.  I wasn't seeing the error on my end because chrono-tz (a dev-dependency) enables that feature, commenting it out I also saw the build failure.  I just pushed the patch so it should build fine now.

Nice, thanks!


Unable to find dependencies

I am using the java-classic working example project. I run the build class on IntelliJ itself. But unable to find the location where the dependency jars are downloaded. It's also not available in the output jars

Hi,
You'll find them in [USER_HOME]/.jeka/cache/repo. I will add a note to the documentation.

Thanks @djeang for your quick response.
But how about defining the custom path for the downloading dependencies!
And also, why we have chosen to store the dependencies in a commonplace?

It is stored in a common place by default to minimize download over the network, this common place acts as a cache. This is ok for most of usage.
Can you tell me, please, what you want to achieve ? May there is simple way to do it without "redirecting" download. For exemple, you can get the download result by using javaPlugin.getProject().getMaker().fetchDependencies(). From the result, you can easily get resolved files to copy them in the place of your choice.


HMDA update

Corresponds to PR 31 on regulations-xml.

:+1:, see discussion on regulations-xml#31.


Use a transparent red tint instead of a solid bright red colour for search box errors.

This resolves #556

👍


cask/audit: update OSDN URL style

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

OSDN (JP) has made some changes last year and now has a cleaned up URL layout. Reflect that in the cask audit checker.
Ref: https://github.com/Homebrew/homebrew-cask/pull/52975#issuecomment-427613612

Leave this as DNM pending the resolution of a few edge case casks in the fonts repo.
https://github.com/Homebrew/homebrew-cask/pull/52975#issuecomment-427616054
https://github.com/Homebrew/homebrew-cask/pull/53029#issuecomment-427616834

Extended to allow /downloads/ subdirs along with /dl/.


Logstash NameError

Hey @blacktop, I really appreciate your maintaining this repository for logstash alpine images - those have the potential for being the leanest images for logstash.
Unfortunately, they do present some challenges as well and that's what I'm opening this issue for. I've already written up a detailed account of what's going on on discuss elastic. Would you mind reviewing that and giving me your opinion on what the issue might be with that particular docker image?

It looks like it works on the official docker image, but not my alpine version?  Is that correct?

Yep - that's exactly right.

I have confirmed that alpine does NOT like that plugin 😄  I am not sure why.  I know that alpine does not use libc it uses musl and that causes a lot of problems.  I will try again this weekend, but using the official docker plugin might be your best bet.

Thanks @blacktop, yeah I did some research in the area as well and found several libraries that might be the culprit, libc being one of them. Could we replace musl in the image?

the entire Alpine OS is based on musl, you can install libc-compact but it doesn't seem to help.
I read that journald came out of systemd, and I doubt systemd is installed in alpine as it's init system.

@blacktop did you get a chance to do some research here?

Alpine Linux uses OpenRC for its init system. We have followed suit and added it as the init system in our Alpine Linux base images over systemd. This change allows the containers to seem more like fully fledged Linux environments.

https://j2h2.com/entry/alpine-linux-and-systemd-containers-round-2

https://forum.alpinelinux.org/forum/general-discussion/alpine-plans-systemd

this might work? https://github.com/balabit/syslog-ng/issues/1005

so alpine doesn't like/want to support systemd which you need for journald, so it might not make sense to have it in alpine.
Is there a reason for it to be alpine?  I feel like once you add ALL the stuff necessary to get it in alpine the image might have grown to a size comparable to what it would have been on a very slimmed down ubuntu:xenial ?

Nice research.
No, no specific reason to use alpine. My general effort is to slim down logstash images as much as possible and I'm open to your suggestion of using another pretty slim OS image to accomplish that goal. I will try using ubuntu:xenial next and let you know how that goes down.

👍


pad: remove terminal edges on success and failure to avoid shorting later routes into shape

Fixes:

on a failed RDL route the graph edges were left behind opening the possibility to shorting routes because the cleanup did not occur. This ensures the edges are removed in either case to prevent routing shorts in RDL.

conflict to resolve

@maliberty resolved.  Thanks


chore(deps): bump docker image for tibuild to v2024.11.4-3-gcec492f

Ref: https://github.com/PingCAP-QE/ee-apps/pull/194

/approve


samples: Add sample application for sensor HTS221

Add sample application for sensor HTS221
with disco_l475_iot1 as default board.
Was: https://gerrit.zephyrproject.org/r/#/c/13187/

This change is

recheck


[WebAPI] DOM general tasks, Fx 73

Trello
https://trello.com/c/aBs1o6Pg/86-dom-general-enhancements-fx-73
Acceptance criteria

[ ] window.innerWidth/innerHeight should return the dimensions of the layout viewport

[ ] Should we make default of "wheel" (and "DOMMouseScroll", "MozPixelMouseScroll") event listeners on root "passive"?

The first one looks pretty easy; half a day tops.
The second one is nightly-only, so would just get a mention on the experimental features page for now. Even if it were shipping in 73 release, it would only need a quick mention in release notes and a BCD update to add a note to the affected events. Quarter day or so tops, if so.

Updating estimate, as I reckon you are the king of fast ;-)

I think https://bugzilla.mozilla.org/show_bug.cgi?id=1602163 should be added to this task.

I think https://bugzilla.mozilla.org/show_bug.cgi?id=1602163 should be added to this task.

Added. Thanks; I missed this one. I've put the estimate back up to 1 too, just in case.


Added biome system

I tried fooling around to create a crude but relatively expandable biome modification system for the simulation that shouldn't have any noticeable impact on performance, functioning in the state that it is, but given you seem to be trying to implement swimming, something might need to be modified for that to work with this.
Otherwise it should work fine, the changes I've tried with the desert were vaguely noticeable.
Also wanted to see if it would be fine to pull or if it should be left out.

Sorry but I gotta add swimming first, me and a friend are practically rewriting the world update function


ApplicationOnlyCredentials

I am trying to create a Filtered Stream using application only credentials and it always hits the stopped event with a null reference exception. If I switch it to use an access token as well then it works fine.
I'm using the below code:
var credentials = Auth.SetApplicationOnlyCredentials("[CONSUMER KEY]", "[CONSUMER SECRET]", true);
//this works fine
//var credentials = new TwitterCredentials("[CONSUMER KEY]", "[CONSUMER SECRET]", "[ACCESS TOKEN]", "[ACCESS TOKEN SECRET]");
var stream = Tweetinvi.Stream.CreateFilteredStream(credentials);
//add tracks
stream.StartStreamMatchingAllConditionsAsync();
Should this functionality be working?

As mentioned in the Twitter doc, you cannot access a stream endpoint with Application Only credentials.
With Application-only authentication you won’t be able to:

Post tweets or other resources;
Connect in Streaming endpoints;
Search for users;
Use any geo endpoint;
Access DMs or account credentials;

I will put that has an improvement so that the error raised informs the developer that this is not possible.

This issue was already fixed in 0.9.10.x.


Translate fr

Translate fr
The entire content has been translated

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


Web | Customer | Help and Support | Wrong Error Message displayed when user enter an Invalid mobile number.

Environment

Build/ Version/ Commit id
Php version : 7.42
Mysql : 5,4
platform : Android/ ios / Web
Browser : Chrome / firefox
Server : Dev /QA /QA1/Staging /preprod /production
Android : 14

Description

When user tries to enter Invalid mobile number then Wrong error message displayed.

Test Steps

Open URL :-   https://www.drishtilearningapp.com/
Navigate to footer and click on Help and Support.
Enter Invalid Mobile number and submit the enquiry.

Test Data

Invalid Mobile no.-   5677894

Expected

Error Message should be displayed : Invalid Mobile Number.

Actual

Error Message displayed : Student Mobile Number : Invalid Mobile Number.

@Sanjeevpaswan  Please appropriate error message for the scenario mentioned as it is not mentioned in story.


⚠ Bofrost.fr has degraded performance

In fc4b799, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1002 ms

Resolved: Bofrost.fr performance has improved in beb9a8f.


Fix updating tags on aws_launch_template

Fixes #1504
Due to discrepancies between TF and Pulumi Bridged provider life cycle, editing tags on a Launch Template resource caused failure in pulumi-aws. This is now fixed by opting in this resource to the PlanResourceChange behavior that tracks TF lifecycle more closely.

/release patch


[Bug] David Marcus is a Sith Lord

David Marcus is a Sith Lord. He uses the Force to further the evil purposes of The Empire.
They call it Libra and pretend that it is a heavenly body designed for freedom. But I will tell you what it is. It is a Death Star. Brothers and sisters of the rebellion, it is time to fight.

Don't waste your energy. This is not a competitor to Bitcoin. It's facebook's answer to Wechat pay and nothing more.


[BUG] Longhorn UI is freezing up when there are many disks

Describe the bug
When there are many disks per node in Longhorn, editing individual disks is almost impossible since the user interface is becoming close to unresponsive due to some operations happening in the background and blocking the UI.
Reproduce

Have nodes with > 32 disks configured in Longhorn
Try to remove a disk in the UI

Chrome Profile recording available on request.

@janeczku Could you please provide the chrome profile recording? Thank you.
cc @smallteeths

Is it taking up too much memory in chrome? I'll try to reproduce the bug later. cc @jenting

I simulated 40 disks of data and then compared the memory snapshots and there were no significant problems. No page freezing up was encountered either. Is it only when saving that the page freezing up is encountered? @janeczku cc @jenting
https://user-images.githubusercontent.com/27733391/147025480-d40db5fc-d540-4667-a27f-91995129322d.mp4
https://user-images.githubusercontent.com/27733391/147025944-f5f8b9e5-2098-4b41-a403-a3f7563f5dc6.mp4

This was related to https://github.com/harvester/harvester/issues/1718 where we had hundreds of disks because of a bug. Since that's not happening for me anymore, I can't reproduce this issue. @janeczku should we just close this issue?


Error:  "Failed to activate the atom-typescript package"

Install atom-typescript (and possibly the stuff below?)
2.Open a typescript file

Atom: 1.38.2 x64
Electron: 2.0.18
OS: Mac OS X 10.14.6
Thrown From: atom-typescript package 13.9.1
Stack Trace
Failed to activate the atom-typescript package
At Invalid regular expression: /(?:(?![\u{10000}-\u{10FFFF}])[\$_\p{Lu}\p{Ll}\p{Lt}\p{Lm}\p{Lo}\p{Nl}])(?:(?![\u{10000}-\u{10FFFF}])[\$_\p{Lu}\p{Ll}\p{Lt}\p{Lm}\p{Lo}\p{Nl}\u200C\u200D\p{Mn}\p{Mc}\p{Nd}\p{Pc}])*$/: Invalid escape

SyntaxError: Invalid regular expression: /(?:(?![\u{10000}-\u{10FFFF}])[\$_\p{Lu}\p{Ll}\p{Lt}\p{Lm}\p{Lo}\p{Nl}])(?:(?![\u{10000}-\u{10FFFF}])[\$_\p{Lu}\p{Ll}\p{Lt}\p{Lm}\p{Lo}\p{Nl}\u200C\u200D\p{Mn}\p{Mc}\p{Nd}\p{Pc}])*$/: Invalid escape
    at /packages/atom-typescript/dist/main/atom/autoCompleteProvider.js:107:25)
    at /packages/atom-typescript/dist/main/atom/autoCompleteProvider.js:217:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:147172)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:150720)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:504:12)
    at Function.Module._load (module.js:496:3)
    at Module.require (/app.asar/static/index.js:60:45)
    at require (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:146462)
    at /packages/atom-typescript/dist/main/pluginManager.js:18:32)
    at /packages/atom-typescript/dist/main/pluginManager.js:314:3)
    at Module.get_Module._compile (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:147172)
    at Object.value [as .js] (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:150720)
    at Module.load (module.js:561:32)
    at tryModuleLoad (module.js:504:12)
    at Function.Module._load (module.js:496:3)
    at Module.require (/app.asar/static/index.js:60:45)
    at require (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:146462)
    at Object.activate (/packages/atom-typescript/dist/main/atomts.js:10:31)
    at Package.activateNow (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:3418998)
    at string.e.trim.length.activationHookSubscriptions.add.packageManager.onDidTriggerActivationHook (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2445)
    at Function.simpleDispatch (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1206583)
    at Emitter.emit (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1208024)
    at PackageManager.triggerDeferredActivationHooks (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:381556)
    at Promise.all.then (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:380354)
    at <anonymous>

Commands

Non-Core Packages
after-dark-syntax 0.1.0
atom-material-syntax-dark 1.0.0
atom-material-syntax-light 0.4.6
atom-material-ui 2.1.3
atom-monokai 0.10.9
atom-monokai-dark 0.4.0
atom-oceanic-next-italic 0.1.1
atom-typescript 13.9.1
autoclose-html 0.23.0
bracket-colorizer 1.4.0
busy-signal 2.0.1
chester-atom-syntax 0.3.2
cursor-history 0.14.0
fonts 3.11.0
goto-definition 1.3.4
gruvbox-plus-syntax 1.3.1
ide-haskell 2.4.1
intentions 1.1.5
isotope-light-ui 2.3.0
jsx-comment 0.2.0
kecleon-ui 0.1.1
language-haml 0.32.1
language-haskell 1.20.0
language-javascript-jsx 0.3.7
language-vue 0.26.0
linter 2.3.1
linter-eslint 8.5.5
linter-rubocop 2.5.7
linter-stylelint 4.5.1
linter-tslint 1.10.43
linter-ui-default 1.8.1
markdown-preview-plus 3.9.2
markdown-scroll-sync 2.1.2
oceanic-next-italic-white 0.1.3
one-dark-italic-syntax 1.1.0
prettier-atom 0.60.1
rainbow-tree 1.0.1
seti-syntax 1.2.0
seti-ui 1.11.0
Sublime-Style-Column-Selection 1.7.5
tester 1.4.0
tester-jest 1.2.4
theme-for-a-summers-day-syntax 0.0.0
theme-switch 0.6.0
tomorrow-night-eighties-syntax 1.1.0
uuidgen 1.4.0
warn-before-quitting 0.1.7

Ouch. Sorry, apparently I forgot to bump the lower Atom version in v13.5.1. Can't really do much about that now. v13.5.0 or earlier should work with your Atom version. I would suggest you upgrade Atom instead of downgrading Atom-TypeScript though.

I'll give that a try, thanks!

All good after updating Atom.  Thank you very much for your quick response, I appreciate it!

Idk if this is the same issue but i still have the issue with atom.1.55.0-nightly only on dev mode.

We don't support nightly builds. But if you want to help, post the complete report in a new issue. To copy the complete error report to clipboard, click on the small clipboard-looking icon when the error pops up:

I have the same error with atom 1.54

This happens on stable only on dev mode

@ThatXliner for the love of God, it's not that hard to open a new issue, is it?
I'll fix the issue shortly.

Aw, fudge. That's a bug in parcel. Hold on.

Try parcel 2 beta-3.1 or beta-2


allows custom format, for params and definition properties

adds spec for default value

Coverage remained the same at 100.0% when pulling dccbfc35cbb4fcfe565e8c622938e263f57a3ca1 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling dccbfc35cbb4fcfe565e8c622938e263f57a3ca1 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling d8197c588e509b177d8af19a15e376a529047ae8 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.

Coverage remained the same at 100.0% when pulling b0da106b60e0b5a35b34d13df126e43eaea8eb29 on LeFnord:master into d78e1adfed04b654be2b8db3f07cc965a238f667 on ruby-grape:master.


Add option to the instantiation of a module.

Is your feature request related to a problem? Please describe.
Not really a problem, but something I'd like to have.
Describe the solution you'd like
I have written code in ph4 to allow the user to specify the number of GPUs for each worker in the parallel code to use (requires sapporo_2).  Currently I pass that number as a parameter:
ph4.parameters.n_gpu = xxx.

I'd like to have the alternative of including it in the instantiation of ph4:
grav = ph4(number_of_workers = 3, mode = "gpu", number_of_gpus = 2)

Describe alternatives you've considered
I can do it using parameters, but the init function for ph4 is opaque to me, so advice on how to modify it would be appreciated.
Additional context
Add any other context or logfiles about the feature request here.
N/A

I think we can add something to the baseclass __init__ that tests keywords arguments against parameter names and sets them if it a parameter..let me think about this though...

Not a huge priority.  I can manage it using parameters, and that may actually be the right way to do it, since the number of GPUs doesn’t have to be known in order to instantiate the worker.
Stephen L. W. McMillan
Department Head and Professor of Physics, Drexel University
+1<PHONE_NUMBER>/2723, FAX +1<PHONE_NUMBER><EMAIL_ADDRESS>
On Apr 21, 2020, at 4:20 PM, Inti Pelupessy<EMAIL_ADDRESS>wrote:
I think we can add something to the baseclass init that tests keywords arguments against parameter names and sets them if it a parameter..let me think about this though...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/amusecode/amuse/issues/618#issuecomment-617393044, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAPFB3XHHFJB5ISQP432XCTRNX5YZANCNFSM4MLAWDVA.

ok..I leave it for now...I may implement this later since there are more use cases where you would want something like this..


updated step to accept 9 decimals

After accepting this PR, PR#154 can be closed (Fixing the same issue the same way) (This one is up to date with master)

Not up to date. The needed decimal is now 0.1


Configured isort to use its "django" profile.

isort 5 added profiles to represent common configurations, including one copying Django’s config ( https://pycqa.github.io/isort/docs/configuration/profiles.html ), so we no longer need to specify the options directly.

@adamchainz Thanks for this patch :+1: However I don't think that we should "import" our configuration from the 3rd-party package. It would create a circular logic dependency and it's confusing. isort's "django" profile is rather for others not for Django itself :shrug: . Let's see what other think.

Also, when we adopt black won't we want to use isort's black profile over its django profile? That'll be amusing… 😄

I have no strong feelings, the "django" profile is probably only relevant till we can drop in "black" (sooooon ;)). I somewhat agree with Mariusz that it at least feels weird to pull our configuration from someone else (but all in all no strong feelings). What if we were to change the config in the future? Then we'd have to ask isort to release a new release for us :D

Also, when we adopt black won't we want to use isort's black profile over its django profile? That'll be amusing… 😄

Yes, absolutely.

What if we were to change the config in the future? Then we'd have to ask isort to release a new release for us :D

We can override it on a per-rule basis. But with so many years without change, plus changing to the Black profile on the horizon, it seems very unlikely to change.

But with so many years without change ...

I still don't see any benefits, grumbling... :older_man: :wink: Sure we can do this, but why? just because we can :thinking: that's not a rationale (my :two: ¢.)

You are getting a grumpy old man 🤗🤗 but I do agree with you that the gains are negligible. I think both approaches are fine, and I am perfectly fine with letting the fellows decide. This will be the last (I hope, unless you manage to trigger me 😜) I will write here -- as well intentioned as the proposed change and discussion are, I do not think there is much to win here aside of a waste of time (please note that I am not suggesting that anyone here is trying to waste anyones time, we are all trying to make Django better). Those are my two €, let's use it for a beer next time we meet.
On Mon, Jul 5, 2021, at 19:55, Mariusz Felisiak wrote:

But with so many years without change ...

I still don't see any benefits, grumbling... 👴 😉 Sure we can do
this, but why? just because we can 🤔 that's not a rationale (my 2⃣ ¢.)
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/django/django/pull/14594#issuecomment-874260130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAT5C52WHBGOXPSMBRXRA3TWHWYDANCNFSM472NEEPA.

I'm happy whether you close or merge. One small improvement is that the shorter config makes it easier for others looking to make packages to adopt the same style. Also, moving the config to pyproject.toml is good long term as the ecosystem moves to configuring all tools through that file.

@adamchainz Thanks again :hugs:
Closing due to lack of consensus.


[Bug] test_vitis_ai_codegen.py frontend tests run even when USE_VITIS_AI OFF

The series of tests under tests/python/contrib/test_vitis_ai/test_vitis_ai_codegen.py are set to run conditional to the presence of pyxir module in the namespace, which is true for many of our CI containers including ci_cpu.
However, in case we have TVM built with USE_VITIS_AI OFF and try to run the same tests, it will fail with:
E           AttributeError: Invalid config option 'relay.ext.vitis_ai.options.target' candidates are: relay.ext.verilator.options
,relay.ext.cmsisnn.options ,tir.usmp.enable ,tir.usmp.algorithm ,relay.ext.ethos-u.options ,tir.detect_global_barrier
,relay.FuseOps.max_depth ,tir.HoistIfThenElse ,relay.fallback_device_type ,tir.LoopPartition ,tir.InjectDoubleBuffer
,relay.backend.use_meta_schedule ,relay.backend.use_auto_scheduler ,tir.UnrollLoop ,tir.noalias ,tir.instrument_bound_checkers
,tir.disable_vectorize ,tir.disable_cse_tir ,tir.is_entry_func ,relay.ext.ethos-n.options ,relay.FuseOps.link_params ,tir.disable_assert
,tir.disable_storage_rewrite ,tir.add_lower_pass ,tir.debug_keep_trivial_loop

The error is expected once TVM was build with USE_VITIS_AI OFF, but I guess the test module could be improved and skipped properly to cover this situation.
I'm facing this issue when trying to progress while we wait for https://github.com/apache/tvm/issues/10696 to be fixed.
cc @jtuyls @anilmartha

Just as a suggestion, I believe ACL uses similar logic to check if TVM was built with the expected configuration to run the tests. The skip functionality in the tests is implemented here: https://github.com/apache/tvm/blob/main/tests/python/contrib/test_arm_compute_lib/infrastructure.py#L159

Ah apologies I see there is already something similar: https://github.com/apache/tvm/blob/main/tests/python/contrib/test_vitis_ai/infrastructure.py#L60

With #10802 merged, we can close this now.