Broken HTML report when merging coverage info with nyc report

Link to bug demonstration repository.

https://github.com/bennyn/typescript-jasmine-karma-nyc

Expected Behavior
HTML report where you can click on files to see their code coverage.
Observed Behavior
HTML report which is broken for some of the files:

TypeError: Cannot read property 'locations' of undefined

Scenario

I wrote TypeScript code which behaves differently depending on the environment (Node.js or a HTML5 browser) it runs on.
To test every part of my code, I need to run tests in a Browser environment and in a Node.js environment.
For running tests in Node.js, I am using nyc in combination with jasmine and ts-node/register.
Browser tests are performed by karma with karma-jasmine and karma-chrome-launcher.
When I execute the browser tests in my project (yarn test:browser), I am getting HTML output (coverage/browser/html) and JSON output (coverage/json/coverage-browser.json). The HTML report works fine. I can see my TypeScript files in the report and their coverage.
When I execute the Node.js tests in my project (yarn test:node), I am also getting HTML output (coverage/node/html) and JSON output (coverage/json/coverage-final.json). The HTML report looks fine here too!
The code coverage of my browser tests report that half of my code is covered by tests.
The code coverage of my Node.js tests report that the other half of my code is covered.

I would like to join both code coverage reports now to receive one universal (merged) report that states that my app code is fully covered by tests.
I have read documentation on nyc merge and thought it is a good idea to run nyc report (which implicitly merges coverage information) and point it to my JSON output files:
nyc report --temp-dir coverage/json --report-dir=coverage/merged/html --reporter=html

The nyc report command finishes and I am receiving an HTML report but the report is broken (see "Observed Behavior"). I would like to know what is going wrong there because when I check the individual HTML reports (coverage/browser/html & coverage/node/html) they look fine.
Forensic Information
Operating System: Windows 10 Pro (but I think the problem is unrelated to the OS)
Environment Information:

Node.js v10.9.0
npm v6.4.1

Closing here in favor of istanbuljs/istanbuljs#322.

I do not use karma and have the exact same issue with merging coverage files.
I use cypress and jest and want to merge their coverage outputs.
What happens is exactly what was described by @bennyn.
I use:

jest 24.8.0
cypress 3.4.1
istanbul-lib-coverage 2.0.5
nyc 14.1.1


Upload assets

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

GitHub release
v0.1.0

Your semantic-release bot :package::rocket:


Casey implement drone scanning flowcharts

Task card: https://trello.com/c/qxownQOg/904-implement-drone-scanning-flowcharts
This implements 2 different flowcharts for how the drone scans items, for the designers to figure out which works better. To test both, on the Drone gameobject, at the top of its script, there's a bool "Button Press Scanning". Make it false to test the automatic scanning task, and make it true to test the button press scanning task.

I looked at your pull request and merged it. Good work!


Changed from master to main

name: Pull request
about: ""
title: "[PR]"
labels: "i: pull-request, i: needs triage"
assignees: ""

🚀 🚀 Pull Request
All Submissions:

[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 update/change?
[x] Is your Pull Requests linked to an Issue #1051

good catch

requested @davidbuniat 's review because he set up the release drafting stuff


Circle disable rotation

I have a circle falling on a static rectangle (ground). Somehow it starts rotating whilst falling (gains angular velocity?) which makes it 'roll' on the rectangle's surface.
What would be the best / most performant / cleanest way of disabling the rotation on a circle body?

Try this:
var circle = Bodies.circle(x, y, 10, { inertia: Infinity });


Add tips on tracking the number of open issues

Under Retention, the Metrics guide suggests tracking the number of open issues and pull requests:

If these numbers get too high, you might need help with issue triaging and code reviews.

The GitHub UI doesn't currently offer a way to view the development of these metrics over time (like for contributions, commits, or code frequency).
Many maintainers may be feeling overwhelmed by what feels like a large number of open issues, but without a way to see how many were open last year or two years ago, they're left to rely on memory or gut feelings rather than hard data.
There are many commercial options that claim to offer burndown charts or similar visualizations of historic issue data, but it seems like an "opensource.guide" should point to open source solutions that can be used to keep track of whether the overall issue load is increasing or decreasing.
Can we recommend anything here or provide more information about how to go about that?
— or should we open an issue to add a new Issues chart to the Insights tab of the GitHub UI?

Hi @infotexture! If you have open source solutions that you'd like to add to the Metrics guide, that'd be great. However, this repo isn't the right place for discussing GitHub feature requests.
Closing this out for now, but if you have specific additions to the Metrics guide that you'd like to suggest, feel free to open a PR.


Error: cannot execute INSERT in a read-only transaction

After replacement JpaRepository to BaseJpaRepository
@Service reports: Error: cannot execute INSERT in a read-only transaction during inserts.
methods with inserts are not using @Transactional.
Adding @Transaction(readonly = false) on @Service class level fixes it.
What's the reason and how to use BaseJpaRepository correctly without @Transactional?(if it is possible)
Thank you in advance

The transaction boundaries should be set by the Service Layer, not by the Data Access Layer.
If you don't set the @Transactional annotation on your Service method and you call 3 methods on the Repository, each method will execute in its own transaction and will cause 3 connection acquisitions from the pool.
Anyway, #584 fixes this issue so this problem should be fixed even if you use transactions on the Repository, which you should not.

We din't have any @Transcational annotations before,(not on service, and of course not in repository level). That's the reason of the question, we just switched JpaRepository to BaseJpaRepository , and received the error.
for e.g.
---before
public interface SaleRepository extends JpaRepository<Sale, UUID>
{.}
@Service
someLongRunningMethod(){
...
saleRepository.save(entity);
...}
----after
@Repository
public interface SaleRepository extends BaseJpaRepository<Sale, UUID>
{}
@Service
someLongRunningMethod(){
...
//and this is now not working - cannot execute INSERT in a read-only transaction
saleRepository.persist(entity);
...
}
---->
@Transactional(readOnly = false)
@Service
someLongRunningMethod(){
...
//now it works
saleRepository.persist(entity);
...
}
but now we have to use @Transactional and refactor the code in order not to have too long running database transaction

You can try the 3.3.0 version as it should work without you doing any change.

You can try the 3.3.0 version as it should work without you doing any change.

thank you!

You're welcome


AS & RVPS | Proposal for an attestation applied policy format

Background
During a long time, we use Rego policy to check against parsed claims derived from a TEE evidence. Because of Rego's flexibility we did not talk about the topic so much. Now I raised some limitations of the rego workaround and propose a new format that hopefully could well fit the all typical cases in remote attestation, and also extensible for future profile.
Limitations of Current Rego Policy

Does not support mask rule.

For some cases, the claim value is a bitmap. Different bit of the bitmap shows different things of the TEE. Like td_attributes  in tdx's claims. Intel's profile for IETF RATS' CoRIM (carrier of reference values) defines similar rules for this. However, in rego it is hard to do bitwise operation.

Restrictions upon the writing style of the policy

In current implementation, we assert that the policy would define a bool value named allow to explicit show the result of the policy. In other words, if the policy is not written with an allow value, the enforcement will always fail.

Troubles when working together with RVPS

In current implementation, the reference value of a specified claim can be given in two ways:

Directly in the policy, just like the example rego does
Provide from RVPS. This is a little tricky. We now couple the code and policy to achieve this. In the policy we use data of OPA to carry reference values from RVPS. In code, we will always query every key inside the parsed claims from RVPS to get reference values, and make the results into a data for OPA.

In a more clear design mode, we should not query any claim keys from RVPS, but rely on some hints in the policy.
There is another topic of "Combined Reference Value" discussed here. This hints that the policy should give a way for users to both a)directly specify the reference value against concrete claims  and b) use a TE_id. TE_id here could be linked to a bunch of reference value rules defined in a CoRIM or something else.
Some more information about TE_id. In RATS, we have defined Target Environment to refer to the environment to be attested. Thus some reference value carrier like CoRIM would define a list of values to be matched together against an evidence. it sounds like "hey, if you(the tee) want to match the target environment, you need to match all of the following match rules". The TE_id is the key to index such set of rules.
Proposed Rule Syntax
Based on the observations mentioned before, I want to share a draft of CoCoAS policy design. It support some typical rules.
Numeric Expressions
Syntax
("<claim-key>" <op> <value>)

<claim-key>: the key of the parsed claims defined in the doc. e.g. sgx.body.isv_svn
<op>: could be >, >=, ==, <=, <
<value>: i64 number

Note that once numeric operation is involved, the value of claim-key will be treated as a hex format of an integer number.
For example, ("sgx.body.isv_svn" > 16 means that only if "sgx.body.isv_svn") is greater than 16, this expression will be true.
Set Expressions
We now only define in rule for set.
Syntax
("<claim-key>" <op> <set>)

<claim-key>: the key of the parsed claims defined in the doc. e.g. sgx.body.isv_svn
<op>: could be in
<value>: a list of acceptable values.

For example, ("snp.measurement" in ["aabbcc", "ccbbaa"]) means that only if "snp.measurement" is "aabbcc" or "ccbbaa", the expression will be true.
Naive Assertion Expressions
Syntax
("<claim-key>" is <value>)

<claim-key>: the key of the parsed claims defined in the doc. e.g. sgx.body.isv_svn
is: the keyword
<value>: the expection value for the claim key

For example ("snp.measurement" is "aabbcc") means that only if "snp.measurement" is "aabbcc" the expression will be true.
Mask Expressions
Syntax
("<claim-key>" mask <mask> equ <value>)

<claim-key>: the key of the parsed claims defined in the doc. e.g. tdx.quote.body.td_attributes
mask: the keyword
<mask>: a mask to be applied onto the value of
equ: the keyword
<value>: the expected value after applying the mask

For example ("tdx.quote.body.td_attributes" mask "0x0000f0" equ "0x000010") means that only if "tdx.quote.body.td_attributes"'s value does a bitwise-and with 0x0000f0 and gets 0x000010, the expression will be true.
Logic Expressions
Different expressions could be combined using logic expressions.
Syntax
# And
(Expr1 and Expr2 [..and Exprn])

# Or
(Expr1 or Expr2 [..or Exprn])

# Not
(not Expr1)

For example (("tdx.quote.body.td_attributes" mask "0x0000f0" equ "0x000010") and ("tdx.quote.header.version" > 10))
Reference Value Link Expression
This is a special expression rule. This rule will trigger a query to RVPS for a given TE_id, and apply all the reference values upon the current evidence. If the rules of the TE_id all matches, the expression will be true.
Syntax
(with TE <TE_id>)

<TE_id>: The TE_id of an expected TE.

Full example with different combinations of rules
Here is an example that defines a rule of a combined attestation claim with both GPU and CPU.
It asserts the GPU part should match TE_id gpu-nvidia:123456789 from RVPS. Then, for TDX and SNP it has different rules to check against different fields.
(with TE "gpu-nvidia:123456789")
and
(
  (
    ("tee_type" is "tdx")
    and
    ("tdx.quote.body.mr_td" in ["aa", "bb"])
    and
    ("tdx.quote.body.tcb_svn" > 10)
    and
    ("tdx.quote.body.seam_attributes" mask "0xffffffff" equ "0x00000000")
  )
  or
  (
    ("tee_type" is "snp")
    and
    ("snp.measurement" is "cc")
  )
)

Backup
In RVPS, we could store reference values in such a format too. As reference values are not only simple expected values to directly be matched, but also could involve some rules.
This is still a un-mature draft, please share any suggestions upon this.

One of the goals we are exploring with Regorus is to use it as a runtime for different policy languages. So, I'd be following the evolution of the language in your proposal closely.
FWIW, the equivalent Rego would be something like

rules["gpu-nvidia:123456789"] if {
  input.tee_type == "tdx"
  input.tdx.quote.body.mr_td in ["aa", "bb"]
  input.tdx.body.tcb_svn > 10
  bits.and(input.tdx.body.seam_attributes, 0xffffffff) == 0x00000000
} else {
  input.tee_type = "snp"
  input.snp.measurement == "cc"
}

allow = rules[input.gpu_part] if {
   reference_values = rvps_get_reference_values(input.gpu_part)
   check_reference_values(reference_values)
}

where rvps_get_reference_values is a custom builtin function that is registered via Engine::add_extension

Hey, @fitzthum shares some very good points also I highly agree with.

It is confusing to have a policy for the AS and a policy for the KBS.

Yes. Currently they two both share logic to verify evidence. An ideal form should be

AS has its policy to verify evidence, and get an EAR or some token.
KBS has its policy. The policy should have two functionalities (probably we need to split KBS into two parts and each has a policy) a) rules to map an EAR to a certificate(This is for authorization) b) client identity and YES/NO to access the resource (authentication and access control)

The output of the policy is constrained (as @Xynnn007 mentions, users have to know that the policy should return certain fields).

+1

No support for EAR. Generating a valid token in OPA is cumbersome and if we require the output of the policy to be EAR, we will have a much more extreme case of the above issue, where users will be required to write a complex policy.

Yes. If we ensure the output is an EAR, explicitly the user should at least define "what claims should be related to executables / file-system trustworthiness-vector" and things similar. Hints that once EAR is chosen, the policy should match that. A baby-shape workaround in CoCo, could be

rootfs verification like dm-verity root hash will always be mapped to file-system trustworthiness-vector.
verifier verification (almost all tees' verifier plugin only checks the endorsement of the tee evidence) will be mapped to hardware trustworthiness-vector.
initdata/kernel cmdline will be treated as configuration
guest fw/td-shim, kernel will be mapped to executables

Or, we could add some extra rules upon the proposed to map specific rule set to specific vector.

RVPS interaction is under-defined. We should introduce a level of indirection that allows different reference values to be used in different situations. I'm not sure the best way to do this yet.

Only a personal view. After reading some parts of Intel profile for CoRIM, I found that the reference value defined by IETF RATS is beyond simple "value match". It also could include some complex boolean logic. This enlights me to treat reference value inside the RVPS also a defined "policy". This means, although the inputs into the RVPS is some different provenance (CoRIM, in-toto, ...), but the actual thing stored is a id (TE_id, to refer to a reference value) and its related policy/rule set.
If CoCoAS wants to query reference value from RVPS, the actual thing it gets is a part of policy, which shares the same format as CoCoAS' (this is just the inner storage of RVPS, and transparent to users). So please see the Reference Value Link Expression part in the proposal, it could be understood as an "import" statement, "importing" the reference value/rule set marked TE_id from RVPS.

@anakrish Thanks for the pointers and the form of rego looks pretty dry, TBO.
When I tried to use add_extension of regorus and still does not come up with a good solution for Reference Value Link Expression mentioned in the proposal. The translation has almost the same meaning of the original example, apart from the (with TE "gpu-nvidia:123456789") part. That means similar to include for some other languages. It will include extra rules set indexed by gpu-nvidia:123456789 as a key, and use the original parsed claims as a whole to work as the input.
add_extension supports regorus::Values as inputs, such as String, etc. I tried passing TE_id(a string, s.t. the key to rule set) as a parameter, but found that the content of the input json could not be referenced inside the closure function.
Another problem might be from my poor rego experience. Try to think of such a combined logic A or B or (C and (D or E)). If we use rego, we should write as
allow {
   A
}

allow {
   B
}

allow {
   C
   C'
}

C' {
   D
} else {
   E
}

all are flattened and might not be straright forward/friendly enough for most developers. IMOO, I still want the users could have a straightforward way to write policies -- at least or rules could be seen together without defining intermediate variables.
Some ways

way
Pros
Cons

Using existing rego
directly
a) reference value b) or rule writing

Invent a new one, and implement evaluation logic for the DSL
No need to care about translation part
need to write AST parser

Invent a new one, and implement the translation logic from the DSL to rego, and use regorus as runtime
No need to care about evaluation
a) need to write AST parser b) translation part would make trouble

@Xynnn007

add_extension supports regorus::Values as inputs, such as String, etc. I tried passing TE_id(a string, s.t. the key to rule set) as a parameter, but found that the content of the input json could not be referenced inside the closure function.

I'm not sure if I follow you exactly. One can pass any value including the input to extensions. E.g: rvps_get_reference_values(input).
In case something doesn't work as expected, can you share the code here or create an issue at https://github.com/microsoft/regorus? Since Regorus is being used more and more internally at Microsoft, I'd want to promptly address any issues.
One thing you will not be able to achieve using Regorus extensions is to add rules to the current engine. If we were to allow that, then it will violate Rego semantics - an extension could add an alternative for a rule that has already been evaluated or the added rules could change the outcome of already evaluated rules etc.
However, an extension is free to instantiate another engine instance to evaluate rules.

@Xynnn007 I don't exactly follow your use case, but the following technique of composing policies could be relevant: https://www.styra.com/blog/dynamic-policy-composition-for-opa/

Yeah I am still not sure what to do here. Maybe we should talk about this in the next Trustee developer meeting.


⚠ Bofrost.be has degraded performance

In 89ebc6b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 887 ms

Resolved: Bofrost.be performance has improved in 34de576.


Added Preloader

Closes: #125
Describe the changes you've made
Added a new Preloader feature with animation effect
Give a clear description what modifications you have made
Modified the index.html style.css and app.js file to build a animated preloader
Type of change
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Mention any unusual behaviour of your code (Write NA if not)
Any unusual behaviour of your code
NA
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly whereever it was hard to understand.
[x] My changes generate no new warnings.
[x] Any dependent changes have been merged and published in downstream modules.

Its great @archanagandhi  , but can u match the colors of the background and loader . You can choose some good combination
like #f57b70 and #b1a1ff or any of your choice

I couldn't get you @ankitapuri , Which background and loader? Can u please tell which color should be changed?

see when the pre-loader ends , we immediately get a page whose color changes to so much extreme and it doesn't suit the color of the pre-loader . so I was talking to change some color combination

https://user-images.githubusercontent.com/74424757/115192357-51da3f80-a108-11eb-8984-fb276b814851.mp4
How about this? Tried the logo color combination.

modified.color.mp4
How about this? Tried the logo color combination.

cool

Not sure why, but it doesn't seem to work well, it usually takes a lot of time before the loader appears (in chrome).
although in Safari, it doesn't load at all (there is a black screen).

Not sure why, but it doesn't seem to work well, it usually takes a lot of time before the loader appears (in chrome).
although in Safari, it doesn't load at all (there is a black screen).
can you guys confirm? @ankitapuri @avinal

I don't have idea about Safari , but for me Chrome is fine

Not sure why, but it doesn't seem to work well, it usually takes a lot of time before the loader appears (in chrome).
although in Safari, it doesn't load at all (there is a black screen).
can you guys confirm? @ankitapuri @avinal

It does take some time to appear, except that it is working fine in both chrome and firefox

@rish-singhal @avinal For me it works completely fine in Chrome as well as Firefox. Could you please share a video of what you're facing?

https://user-images.githubusercontent.com/74424757/115330661-26139400-a1b2-11eb-99a3-9c8be03cc13c.mp4
You can have a look at this. As soon as I click enter preloader appears.

Seems to work fine now. @avinal let's merge?

Seems to work fine now. @avinal @ankitapuri let's merge?

Is it now working fine in Safari @rish-singhal

Yes


APPSRE-10782 add optional TGW conection cidrBlocks field

https://issues.redhat.com/browse/APPSRE-10782
This will allow to route multiple cidr blocks towards a TGW

What's the plan here? move all cidrBlock to cidrBlocks and remove cidrBlock in the near future?

What's the plan here? move all cidrBlock to cidrBlocks and remove cidrBlock in the near future?

that would be best but nor fully required. This needs to be synced with all instances where we use TGW attachements (incl fedramp i suppose).

Reading the code I don't think it is feasible to remove cidrBlock as I understand that you need to define the moved block.
I would then add documentation in the schema to state the usage of cidrBlock is deprecated and that cidrBlocks should be used instead for new attachments.

Once the resource has been renamed, the moved block is not needed and can be removed. At that point we can remove safely cidrBlock and move its value within cidrBlocks

Great, then at some point we can remove cidrBlock from the schema and just use cidrBlocks
As for the deprecation message, I'd just add a note in the description field.


[Ready for review] Rename scopes to use underscores, not plus signs

References [#OSF-4574]
Purpose
Some OAuth tools have trouble dealing with proper encoding of scope names. This changes the + character (which has special meaning and requires encoding) to an underscore (which should pass through without problems)
Summary of changes

Rename all scopes (internal and public facing) to separate permissions from scope name using an underscore (instead of a plus sign).
Fix the feature that pretty-prints a list of what views/ core scopes go with what publicly available/composed scope
Updated all unit tests accordingly. Times like this that I wish I'd used named constants....

@abought ... there's no OSF-4574 (?)

@abought ... there's no OSF-4574 (?)

https://openscience.atlassian.net/browse/OSF-4574
I thought I added it to the sprint; not sure what's going on there.

All good! Looks right! ... I was searching under 'APIv2' Thanks!

Assuming @icereval is fine with _ vs . segmentation this looks good. CR 2 :+1:

Discussed with @icereval . We can rename in the future if users adds subcollections (which is when namespacing would become relevant). But we only ever refer to CoreScopes items using named constants (and only from within python), so the exact content of the string doesn't matter... for now anyway.


clients/web: use full bundle for client-side Shiki

Attempt to fix #3622
Not sure to merge it though, I fear the bundle size might explode.

The size increase looks reasonable, probably because Shiki is able to async load languages on demand. Will merge it!


FLA-681 Added SubmissionSheet link and icon

Description
This PR includes the following proposed change(s):
https://justice.gov.bc.ca/jira/browse/FLA-681

Updated PackageReview to include "Print Submission Sheet" link and icon

the icon was obtained from Marie Anik and uploaded to src. @alexjoybc, should we use react-icons/md instead?

Updated PackageReview unit tests and snapshots to include new functionality

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
yarn test -u (to update snapshots)
yarn test
yarn lint
Does the change impact or break the Docker build?

[ ] Yes
[x] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

Yes @KDNeufeld we should use the react icon library

Yes @KDNeufeld we should use the react icon library


[WIP] New blocks

Adding more block types, or at least what is trivially possible thanks to https://github.com/satoshinm/NetCraft/pull/120


Improve eval() in command executor

allow eval to use correct filename, this will produce better stacktrace and allow things like require_relative in eval'd files

Thank you :+1:

Sorry about the delay in merging this, totally slipped my mind


SelenideElement execute(Command) method does't work with native commands

I faced an error when calling the execute method with Click native command:
element.execute(new Click());
Method throws java.lang.ClassCastException: com.codeborne.selenide.commands.Click cannot be cast to com.codeborne.selenide.ClickOptions
Cause many commands are tied to the number of args passed through the execute method and are confident in their reliability:
else if (args.length == 1) {
  ClickOptions clickOptions = firstOf(args);
  click(driver, webElement, clickOptions);
}

After adding the execute(Command, Duration) method there may be even more such errors.
I think that Execute command should not pass proxy args to the inner command:
Command<ReturnType> command = firstOf(args);
try {
  return command.execute(proxy, locator, Command.NO_ARGS);
} catch (IOException e) {
  throw new RuntimeException("Unable to execute custom command", e);
}

And the correct way to call native commands is to use this decorator:
public class CustomArgsCommand<T> implements Command<T> {
    private final Command<? extends T> origin;
    private final Object[] args;

    public CustomArgsCommand(final Command<? extends T> origin,
                             final Object... args) {
        this.origin = origin;
        this.args = args;
    }

    @Override
    public final T execute(final SelenideElement proxy,
                           final WebElementSource locator,
                           final Object[] ignoredArgs) throws IOException {
        return this.origin.execute(proxy, locator, this.args);
    }
}

element.execute(new CustomArgsCommand(new Click()));

element.execute(new CustomArgsCommand(new Click(), ClickOptions.usingJavaScript()));

You need to include this decorator in the library (perhaps as utility method for users convenience) and describe the use case in the documentation.
Another options:

Remove non-command arguments (Command, Duration, Long) when processing in a proxy. But this option is less obvious to users.
Adding new special interface where execute method doesn't has Object[] args argument

@ParametersAreNonnullByDefault
public interface CustomCommand<T> {

    @Nullable
    T execute(SelenideElement proxy, WebElementSource locator) throws IOException;
}

@evpl to be honest I have no clue why do you call internal selenide commands with execute().
Could you elaborate a little bit on that?

@BorisOsipov cause I can :) Method takes Command argument and I see no caveats about using native commands.
And I'm sure that after adding method overload with custom timeout many users will want to use native commands with custom timeout.

After writing the issue, I realized that the option with adding the CustomCommand interface is more concise.

@evpl sure you can do whatever you want. We are interesting in real use cases to make sure we do the best for our users :)

@evpl

I agree with your first suggestion: we shouldn't pass args to custom commands:
return command.execute(proxy, locator, Command.NO_ARGS);
If anyone wants to pass arguments to his custom command, he can just pass these arguments to the custom commands constructor:

element.execute(new CustomClick("arg1", "arg2", "arg3", ClickOptions.usingJavaScript(), "whatever args"));

I am working on it (Execute command doesn't pass args to inner command)


Override theme including ToolTip

I am trying to import the Snow theme to override - which I am able to do
const SnowTheme = Quill.import('themes/snow')
I am however unable to get a handle on the SnowTooltip - is it exported in anyway that an end user can build on it?

I am trying to extend the SnowTooltip as well. Were you able to figure this out?


IIS Express / Windows Authentication

Under "Enable Windows Authentication with IIS"
"Provide a local web.config file that activates Windows Authentication on the server when the app is deployed."
Is it possible to be more specific for where to place web.config? Additionally, is there any affect when using IISExpress?

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

ID: 401ad9a0-9e51-80a2-5846-82e9790d7257
Version Independent ID: fbc36c26-9992-1f4c-66d3-02f898ee7ec4
Content: Configure Windows Authentication in ASP.NET Core
Content Source: aspnetcore/security/authentication/windowsauth.md
Product: aspnet-core
GitHub Login: @scottaddie
Microsoft Alias: riande

Thanks for contacting us. We believe that the question you've raised have been answered. If you still feel a need to continue the discussion, feel free to reopen it and add your comments.


[Feature Request] Auto "Truth of..." skills activation

Would like to have a config to automatically activate the "Truth of..." skills when changing classes.
I've been collecting timed nodes at level 55, and have to have separate lists for Miner and Botanist because when it changes classes, it loses the buffs and can't see the nodes it needs.

You have that with Pandora's box plugin

I personally never noticed this because at level 60 you actually get a trait that auto enables this for you upon class switch just like with Sneak.


place() collapses table/grid inside bulleted list

Description
placeing a grid or a table inside a bulleted list seems to collapse the grid/table when the actual text of the list item is too short.
Repro:
#let t = table(columns: 3, ..range(9).map(str))

#place(t)
#v(2cm)
- typst #place(t)
#v(2cm)
- #place(t)

Reproduction URL
No response
Operating system
Web app
Typst version

[X] I am using the latest version of Typst

As you didn't specify the column sizes (you specified columns: 3 which is equivalent to columns: (auto, auto, auto)), the column sizes will be (approximately) equivalent to the largest width of a cell in each column, but reduced to fit the page (or container) width when needed. Therefore, it appears that, with #place, the available width isn't made very clear, so you can force a particular container width by yourself to tell the table it has enough space to expand as much as it needs.
#let t = table(columns: 3, ..range(9).map(str))

#place(t)
#v(2cm)
- typst #place(block(width: 200pt, t))  // indicate the table has 200pt available to expand (it won't fill it entirely though)
#v(2cm)
- #place(block(width: 200pt, t))

with #place, the available width isn't made very clear

Ah yes, you are right. (I really should not try to report bugs after midnight and go to sleep instead.)
Seems to happen for all containers whenever the parent container size is automatically determined.

I think it's still quirky, so maybe we should leave it open.

I think it's still quirky, so maybe we should leave it open.

I have to admit that, indeed, it could be considered a bit odd for place to behave like this by default... perhaps it should give more available space to whatever is inside it by default (or maybe even unlimited available space), similar to measure?

Unlimited would probably be undesirable. E.g. when placing something at the top of the page, you still want it to respect the page width. But something should probably be changed here.

Right... after investigation, it appears that this is the same problem as https://github.com/typst/typst/issues/529. Consider:
#let t = table(columns: 3, ..range(9).map(str))

#place(t)
#v(2cm)
- typst #place(t) #h(1fr)
#v(2cm)
- #place(t) #h(1fr)

The list items just aren't expanding to full width unless explicitly prompted to do so, thus not giving enough space for the table, which didn't explicitly request any particular amount of space (because place makes it request 0 space).

I think a related problem is that it is impossible to center-align and place content at the same time, because place communicates 0 space.
#place(align(center, {
  //h(1fr)
  rect(fill:blue, width: 1cm, height: 1cm)
  [hi]
  //h(1fr)
})

The h(1fr) seem to do interesting things here, centering either the whole thing or just the rectangle. Make sure to try all 4 combinations of (un)commenting them.

The rect is block-level here, so the whole setup is a bit strange because h is inline.


UpperInvariantLookupNormalizer.Normalize() can throw SEHException on Windows Server 2016 TP5

Under certain conditions, UpperInvariantLookupNormalizer.Normalize() can throw System.Runtime.InteropServices.SEHException on Windows Server 2016 TP5.

Typically, the API works the first time a Windows Container is run, but fails on subsequent runs until Windows is restarted.
Sometimes the API never fails, no matter how many containers are run.
We believe the root cause is a Windows bug, and ASP.NET and CoreFx should not need any changes.

Please keep this issue open for tracking until the Windows bug is fixed.
Workaround
As a temporary workaround, the UpperInvariantLookupNormalizer can be replaced with a similar implemention which skips the call to Normalize().
First, add a nested class to Startup.cs:
private class MyLookupNormalizer : Microsoft.AspNetCore.Identity.ILookupNormalizer
{
    public string Normalize(string key) => key?.ToUpperInvariant();
}

Then add a line to ConfigureServices(IServiceCollection services):
services.AddScoped<Microsoft.AspNetCore.Identity.ILookupNormalizer, MyLookupNormalizer>();

Exception
Unhandled Exception: System.Reflection.TargetInvocationException: Exception has been thrown by the target of an invocation. ---> System.Runtime.InteropServices.SEHException: External component has thrown an exception.
   at Interop.mincore.NormalizeString(Int32 normForm, String source, Int32 sourceLength, Char[] destenation, Int32 destenationLength)
   at System.StringNormalizationExtensions.Normalize(String strInput, NormalizationForm normalizationForm)
   at Microsoft.AspNetCore.Identity.UpperInvariantLookupNormalizer.Normalize(String key)
   at Microsoft.AspNetCore.Identity.UserManager`1.FindByNameAsync(String userName)
   at MusicStore.Models.SampleData.<CreateAdminUser>d__6.MoveNext()
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder.Start[TStateMachine](TStateMachine& stateMachine)
   at MusicStore.Models.SampleData.CreateAdminUser(IServiceProvider serviceProvider)
   at MusicStore.Models.SampleData.<InitializeMusicStoreDatabaseAsync>d__3.MoveNext()
   at System.Runtime.CompilerServices.AsyncTaskMethodBuilder.Start[TStateMachine](TStateMachine& stateMachine)
   at MusicStore.Models.SampleData.InitializeMusicStoreDatabaseAsync(IServiceProvider serviceProvider, Boolean createUsers)
   at MusicStore.Startup.Configure(IApplicationBuilder app)
   --- End of inner exception stack trace ---
   at System.RuntimeMethodHandle.InvokeMethod(Object target, Object[] arguments, Signature sig, Boolean constructor)
   at System.Reflection.RuntimeMethodInfo.UnsafeInvokeInternal(Object obj, Object[] parameters, Object[] arguments)
   at System.Reflection.RuntimeMethodInfo.Invoke(Object obj, BindingFlags invokeAttr, Binder binder, Object[] parameters, CultureInfo culture)
   at Microsoft.AspNetCore.Hosting.Internal.ConfigureBuilder.Invoke(Object instance, IApplicationBuilder builder)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.BuildApplication()
   at Microsoft.AspNetCore.Hosting.WebHostBuilder.Build()
   at MusicStore.Program.Main(String[] args)

Related
https://github.com/dotnet/corefx/issues/9650

closing as linked bug has been closed as well


Update configmap to use ms for iter8 latency

This is for v1.0.0

Will redefine all tutorials later,


Get a peak AC power around 6:00 am in the morning

I run the bellowing code and got a high AC power at 6:00 am.
import pandas as pd
import matplotlib.pyplot as plt
import datetime
import sys
# seaborn makes the plots look nicer
import seaborn as sns; sns.set_color_codes()
from pvlib.forecast import GFS, NAM, NDFD, HRRR, RAP,ForecastModel
from pvlib.pvsystem import PVSystem, retrieve_sam
from pvlib.tracking import SingleAxisTracker
from pvlib.modelchain import ModelChain
from pvlib.location import Location

sandia_modules = retrieve_sam('sandiamod')
cec_modules = retrieve_sam('CECMod')
cec_inverters = retrieve_sam('cecinverter')
module = sandia_modules['Trina_TSM_240PA05__2013_']
inverter = cec_inverters['SolarEdge_Technologies__SE9K_208V__CEC_2013_']
print(module)

latitude, longitude, tz = 53.622537, -113.644237, 'Canada/Mountain'
start = pd.Timestamp('07/30/2017 00:00:00', tz=tz)
end = start + pd.Timedelta(days=5)
fx_model = RAP(set_type ='best')
fx_data = fx_model.get_processed_data(latitude, longitude, start, end)

system = PVSystem(module_parameters=module,\
                  inverter_parameters=inverter,\
                  modules_per_string=22,\
                  strings_per_inverter=2,surface_tilt = 10,surface_azimuth =150 )
mc = ModelChain(system, fx_model.location,name= 'pv')
mc.run_model(fx_data.index, weather=fx_data)

mc.ac.plot()
plt.ylim(0, None)
plt.ylabel('AC Power (W)')

The spectral_modifier (spectral loss) in the bellowing code is high at 6:00 am. Multiplying spectral_modifier to get effective irradiance makes the effective irradiance high at 6:00 am, which ultimately leads to a high AC power. I am not sure if the formula to get the effective_irradiance is correct.
    def effective_irradiance_model(self):
        fd = self.system.module_parameters.get('FD', 1.)
        self.effective_irradiance = self.spectral_modifier * (
            self.total_irrad['poa_direct']*self.aoi_modifier +
            fd*self.total_irrad['poa_diffuse'])
        print('aoi_modifier')
        print(self.aoi_modifier)

        return self

Could you post the following time series: air mass, AOI, spectral modifier.  Thanks.
Sent from my iPhone
On Aug 4, 2017, at 3:47 PM, jielianguo <notifications<EMAIL_ADDRESS>wrote:
I run the bellowing code and got a high AC power at 6:00 am.
import pandas as pd
import matplotlib.pyplot as plt
import datetime
import sys
seaborn makes the plots look nicer
import seaborn as sns; sns.set_color_codes()
from pvlib.forecast import GFS, NAM, NDFD, HRRR, RAP,ForecastModel
from pvlib.pvsystem import PVSystem, retrieve_sam
from pvlib.tracking import SingleAxisTracker
from pvlib.modelchain import ModelChain
from pvlib.location import Location
sandia_modules = retrieve_sam('sandiamod')
cec_modules = retrieve_sam('CECMod')
cec_inverters = retrieve_sam('cecinverter')
module = sandia_modules['Trina_TSM_240PA05__2013_']
inverter = cec_inverters['SolarEdge_Technologies__SE9K_208V__CEC_2013_']
print(module)
latitude, longitude, tz = 53.622537, -113.644237, 'Canada/Mountain'
start = pd.Timestamp('07/30/2017 00:00:00', tz=tz)
end = start + pd.Timedelta(days=5)
fx_model = RAP(set_type ='best')
fx_data = fx_model.get_processed_data(latitude, longitude, start, end)
system = PVSystem(module_parameters=alberco_module_similar,
inverter_parameters=alberco_inverter,
modules_per_string=22,
strings_per_inverter=2,surface_tilt = 10,surface_azimuth =150 )
mc = ModelChain(system, fx_model.location,name= 'pv', losses_model='no_loss')
mc.run_model(fx_data.index, weather=fx_data)
mc.ac.plot()
plt.ylim(0, None)
plt.ylabel('AC Power (W)')
[image]https://user-images.githubusercontent.com/26880408/28986996-01d33122-792f-11e7-92a7-35b4ddf8ae38.png
The spectral_modifier (spectral loss) in the bellowing code is high at 6:00 am. Multiplying spectral_modifier to get effective irradiance makes the effective irradiance high at 6:00 am, which ultimately leads to a high AC power. I am not sure if the formula to get the effective_irradiance is correct.
def effective_irradiance_model(self):
    fd = self.system.module_parameters.get('FD', 1.)
    self.effective_irradiance = self.spectral_modifier * (
        self.total_irrad['poa_direct']*self.aoi_modifier +
        fd*self.total_irrad['poa_diffuse'])
    print('aoi_modifier')
    print(self.aoi_modifier)

    return self

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/pvlib/pvlib-python/issues/358, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AFJNLzxp6ObAYk6IG4XjsTYtfCj1pIZrks5sU5FjgaJpZM4OuOUI.

Sure!
aoi_airmass_spectralModifier.txt

Sure!  Thank you @cwhanse
aoi_airmass_spectralModifier.txt

The spikes in AC power result from the spectral modifier being unconstrained.  I'll have to look into how we've addressed this in Matlab and in other analyses.

Got you! Thank you.

The problem stems from the underlying SAPM model for the spectral modifier, which fits a polynomial (4th order) in air mass to measurements.  The pvlib implementation sets a lower bound of 0 for the modifier, but no upper bound.  For some modules, at high air mass, the function will return extremely large values for the spectral modifier.
We don't have a replacement SAPM spectral model.  Two things you can do:
1 - switch to the FirstSolar contributed model for the spectral modifier, in pvlib/atmosphere/first_solar_spectral_correction.  I don't know how to wire that into the model chain.
2 - edit your copy of pvlib/pvsystem/sapm_spectral_loss to impose a maximum on the spectral loss modifier.  My own opinion, I think 1.1 is appropriate.
@wholmgren @jforbess  what are your thoughts on this? I don't think its a bug per se.  I think opening an issue for an enhancement to prevent sapm_spectral_loss from returning overly large values is appropriate.  Is there a way to bring the first_solar_spectral_correction forward as an option when specifying a modelchain?

@jielianguo if you're using forecast data then you might not need to worry about the marginal accuracy improvements from a spectral loss model. The forecast is likely to be the limiting factor in your model accuracy. Simply set spectral_model='no_loss' and avoid all the trouble.
@cwhanse I'd support a pull request to implement a maximum value in the spectral loss function.
In the mean time, one alternative to editing the sapm_spectral_loss source code is to pass a function into the ModelChain constructor. I've not tested the code below, but it should work:
def sapm_spectral_loss_clipped(mcobj):
    mcobj.spectral_modifier = mcobj.system.sapm_spectral_loss(self.airmass['airmass_absolute'])
    mcobj.spectral_modifier = np.maximum(mcobj.spectral_modifier, 1.1)

mc = ModelChain(..., spectral_model=sapm_spectral_loss_clipped)

I'll make another issue for bringing the first_solar_spectral_correction function up to ModelChain.

I think the main problem is that the polynomial is not a great function form for this phenomenon, but it is often risky to use polynomial functions, especially higher orders, for extrapolation anyway.
Limiting the output certainly fixes the symptoms.  Limiting the input could be another option--one which could reflect the range of measurements that were used to obtain the coefficients.

@adriesse I agree, a polynomial is a poor choice, but it was made years ago and is now baked into the parameter databases. We ought to replace it with a functional form which has constrained asymptotic behavior, but that's a discussion not for pvlib GitHub.


fleetctl: introduce an option --max-print-units

For users who want to limit number of units to be printed, introduce an option "--max-print-units" for these commands: destroy, unload, stop, list-unit-files, and list-units.
Note that the default value of --max-print-units should be 0, which means "all units are printed", so that the default behavior should remain as before.
Fixes https://github.com/coreos/fleet/issues/1593
/cc @kayrus

NACK, why bake this into fleet? This doesn't affect performance or anything, so can't people just use head?


Fix incorrect executable target diagnostic

The determination for executable targets isn't limited to Swift files, it will also find 'main.m' for example. This was originally reflected in the diagnostic, but #3254 updated it incorrectly.

@swift-ci please test


Support whitelisting access for dotted hosts

Is your feature request related to a problem? Please describe.
Legacy DNS installations may have a predominance of dotted hosts that are not cleaned up.  Currently, VinylDNS does not allow the maintenance of dotted host records.
Describe the solution you'd like

Allow create and update of dotted hosts in VinylDNS
Whitelist the domains that are allowed to contain a dotted host via config.  Do not allow a new or updated dotted host that is not in the whitelist
Allow only super users and support users to manage whitelisted dotted hosts by default
Allow ACL rule access to manage whitelisted dotted host records
Add verbiage in a tooltip for dotted hosts when displayed to indicate the issue with dotted hosts

#702 and #703 take care of support for updating and deleting existing dotted hosts.

we also already have a warning sign and tooltip in the portal for dotted host records.


setup via module system

Hi, sorry for my Eng.
When i try use bump.js via CDN like https://cdn.rawgit.com/kittykatattack/bump/2fe38b76/bin/bump.js
all working correct.
But when i try use webpack or another build system from npm_modules i get error: Bump is undefined...
Builded code don't have 'var Bump ='.
This don't work for me:
let bump = new Bump(PIXI);

app.js:2 Uncaught ReferenceError: Bump is not defined

yarn info bump.js
yarn info v1.5.1
warning package.json: No license field
{ name: 'bump.js',
description: 'A 2D Collision library for Pixi',
'dist-tags':
{ latest: '1.0.3' },
versions:
[ '1.0.0',
'1.0.1',
'1.0.2',
'1.0.3' ],
maintainers:
[ { name: 'happierall',
email<EMAIL_ADDRESS>} ],
time:
{ modified: '2016-10-09T12:35:04.165Z',
created: '2016-10-09T12:12:07.356Z',
'1.0.0': '2016-10-09T12:12:07.356Z',
'1.0.1': '2016-10-09T12:16:08.974Z',
'1.0.2': '2016-10-09T12:23:39.232Z',
'1.0.3': '2016-10-09T12:35:04.165Z' },
homepage: 'https://github.com/kittykatattack/bump#readme',
keywords:
[ 'collision',
'2d',
'pixi' ],
repository:
{ type: 'git',
url: 'git+https://github.com/kittykatattack/bump.git' },
author:
{ name: 'kittykatattack' },
bugs:
{ url: 'https://github.com/kittykatattack/bump/issues' },
license: 'MIT',
readmeFilename: 'README.md',
version: '1.0.3',
main: 'bin/bump.js',
scripts: {},
dist:
{ shasum: 'e0bd953d1ac46d60ab24610d7c632f92295cf8d4',
tarball: 'https://registry.npmjs.org/bump.js/-/bump.js-1.0.3.tgz' },
directories: {} }

https://www.npmjs.com/package/pixi-plugin-bump


Allow Shenzhen Crashlytics ENV vars

The Shenzhen Crashlytics plugin supports ENV vars for:

CRASHLYTICS_FRAMEWORK_PATH
CRASHLYTICS_API_TOKEN
CRASHLYTICS_BUILD_SECRET

The fastlane Crashlytics action requires that all of these are sent in via hash parameter, which negates the ability to use ENV vars. The assert_valid_* methods in the fastlane action should be modified to check for ENV vars as part of the validation.
Reference: https://github.com/nomad/shenzhen/blob/master/lib/shenzhen/plugins/crashlytics.rb

I agree, that needs to be updated.


Access to "error", "status" and "meta" fields in responses

Having access to the "error", "status" and "meta" would be really nice.
In particular, the "meta" field can contain pagination information (ex: number of items, number of pages) which are extremely helpful when iterating through paginated lists.
Response payload examples:

error:

{
    "status": "error",
    "error": {
        "field": "ship_id",
        "message": "INVALID_SHIP_ID",
        "code": 407,
        "value": "3315513040a"
    }
}

status + meta:

{
    "status": "ok",
    "meta": {
      "count": 100,
      "page_total": 7,
      "total": 608,
      "limit": 100,
      "page": 1
    },
    "data": {
        "3315513040": {
            "ship_id":<PHONE_NUMBER>,
            "name": "[Zaō]"
        }
    }
}

I will take a look how to achieve that.

Error
You already get access to the whole error field
_, err := client.Wot.AccountList(context.Background(), RealmEu, "Licht", nil)
var respErr *ResponseError
if errors.As(err, &respErr) {
	if respErr.Code == 504 {
		test.Skip("Method seems to be deprecated.")
	}
	if respErr.Code == 404 {
		test.Skip("Method seems to be temporarily unavailable.")
	}
}

wargaming.ResponseError
Status
Is the returned error nil, status was ok. If the returned error  was of type ResponseError it was error.
Meta
You are right, there is no easy way, to get that field currently, will work on that!

@kakwa
You should be now able to get the meta return field. At least for some specific endpoints, more are following soon. I added Wows.EncyclopediaShips especially as you are using it.
You can get the version with the meta return field with
go get github.com/IceflowRE/go-wargaming/v3@5fdc7300c5fdd7acde9d976b0d5aafcf48340af3

thanks, I will take a look.

Finished with 0015da771f7ceb719ce45c358b19ea97c2faba5c.
Introduced a GenericMeta struct which covers all known possible meta values.


Unexpected non-type-safety in findUnique: why does Prisma accept id: undefined in where clause?

Bug description
findUnique expects that I pass in a non-empty where object, or Prisma will throw runtime error "type WhereUniqueInput needs at least one argument". However, the Typescript types for findUnique does not seem to catch this, as I am able to pass in an empty object without Typescript compile-time error
How to reproduce

Create simple model with a unique field

model User {
  id           String   @id @default(uuid())
  username     String
}

Call db.user.findUnique with an empty where object

const userOrNull = await db.user.findUnique({ where: { id: undefined}})

Expected behavior
Not sure if this is intended behavior, but maybe a better type definition can enable Typescript to catch this and error out when the where object is empty.
Prisma information
prisma schema file
generator client {
  provider = "prisma-client-js"
}

datasource db {
  provider     = "mysql"
  url          = env("DATABASE_URL")
  relationMode = "prisma"
}

model User {
  id           String   @id @default(uuid())
  username     String
}

I use Prisma with Remix.run
// db.server.ts
import { PrismaClient } from '@prisma/client';

let db: PrismaClient;

declare global {
	var __db: PrismaClient | undefined;
}

// this is needed because in development we don't want to restart
// the server with every change, but we want to make sure we don't
// create a new connection to the DB with every change either.
if (process.env.NODE_ENV === 'production') {
	db = new PrismaClient();
} else {
	if (!global.__db) {
		global.__db = new PrismaClient();
	}
	db = global.__db;
}

export { db };

---

// Component file:
...
const user = db.user.findUnique({ where: { id: undefined} })
// Or just a plain empty object, still no Typescript error
const user = db.user.findUnique({ where: { } })

Environment & setup

OS: Windows 11
Database: MySQL 8.0
Node.js version: 16.17.1

Prisma Version
prisma                  : 4.10.1
@prisma/client          : 4.10.1
Current platform        : windows
Query Engine (Node-API) : libquery-engine aead147aa326ccb985dcfed5b065b4fdabd44b19 (at node_modules\.pnpm\@prisma+engines<EMAIL_ADDRESS>Migration Engine        : migration-engine-cli aead147aa326ccb985dcfed5b065b4fdabd44b19 (at node_modules\.pnpm\@prisma+engines<EMAIL_ADDRESS>Format Wasm             : @prisma/prisma-fmt-wasm 4.10.1-1.80b351cc7c06d352abe81be19b8a89e9c6b7c110
Default Engines Hash    : aead147aa326ccb985dcfed5b065b4fdabd44b19
Studio                  : 0.481.0

This is still actual. Got the error Argument 'where' of type StationWhereUniqueInput needs at least one of 'id' or 'name' arguments in production. This would be catched in development if Prisma shows an error when undefined is passed to the where in findUnique.


Allow emp run to be used with unofficial docker registries.

Fixes https://github.com/remind101/empire/issues/739.
This extracts the logic that we were using to pull from unofficial registries during deployments into the dockerutil client so it can be re-used for emp run.
Tested this locally with the quay.io/remind/acme-inc image and it works as expected.

:+1: awesome


Return output to promise in verification process

Fixes issue #7.
Essentially, we log all output as before, but we also store it in a string variable that is passed back to the promise chain. This allows clients (such as pact-foundation/pact-js) to provide better messaging to their users.

@mefellows Check why the build is failing and please use the same promise library throughout the project.  We can't have a mismash of libraries trying to do the same thing.

Not sure what you mean with promise libs, I didn't change that (AFAIK). Weirdly the build is failing intermittently - never a good thing.

You added Promise.reject within an integration test, which doesn't resolve to anything:
return expect(verifier.verify().catch(function(err) {
							if (!_.isEmpty(err) && err.indexOf("unexpected token at 'Unauthorized'") > -1) {
								return Promise.reject(new Error("failed"));
							}
						})).to.eventually.be.rejectedWith(Error, "failed");

You should be using q.reject instead. I'm guessing the build is failing intermittently because of an async test that calls done too quickly.

Right. I didn't add a new Promise in this PR though (look at the diff) so
that's a bit weird. I'll look into it, thanks for digging through it.
On Thu, Jun 9, 2016 at 11:05 AM, Michel Boudreau<EMAIL_ADDRESS>wrote:

You added Promise.reject within an integration test, which doesn't resolve
to anything:
return expect(verifier.verify().catch(function(err) {
if (!_.isEmpty(err) && err.indexOf("unexpected token at 'Unauthorized'") > -1) {
return Promise.reject(new Error("failed"));
}
})).to.eventually.be.rejectedWith(Error, "failed");
You should be using q.reject instead. I'm guessing the build is failing
intermittently because of an async test that calls done too quickly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/pact-foundation/pact-node/pull/8#issuecomment-224775840,
or mute the thread
https://github.com/notifications/unsubscribe/AADSjC1Q91s8l5580T8RyuyZJrSXUeUZks5qJ2bKgaJpZM4IxbIH
.

--
Matt Fellows

http://www.onegeek.com.au/


Token Generator

It must generate a singleton dynamic token, used for identity in all API methods for interact with another tokens (each token is an user representation).

The idea:

User send a "secret" when doing "hello"
API insert new user in collection, that generates an ID
Create a hash with derivation from the ID using the "secret" plus the "internal secret" (only the API with the "secret" sent by the user - it is not stored - can resolve to an ID)
Return this hash

In future calls, the user will send the token and secret, the API will resolve the ID with the internal secret.


Feature/atx 17 activity with multiple views

Creating PR early, with the intention of changing the builder_macro to:

Take in required parameters in the generated Builder constructor
Return T instead of Result<T, E>

See discussion at
https://users.rust-lang.org/t/idiomatic-way-to-construct-object-with-some-non-required-fields/8078

Changes Unknown when pulling 0210c9be4ec571366b1bcca7cf16cca6b0f5475d on feature/atx-17-activity-with-multiple-views into ** on master**.


[Question] Code Analysis (FixCop) and StyleCop

Hi !
Have you an idea how to integrate Linter Code Analysis (FixCop) ans StyleCop to OmniSharp Vim and ALE ? By goal is to have lint warnings in the Vim gutter and and the QuickFix, like with tslint for TypeScript.
I've installed the linters FixCop and StyleCop thanks the guide at https://carlos.mendible.com/2017/08/24/net-core-code-analysis-and-stylecop/, and I could see the lint warnings in console when I run dotnet run, but I see nothing in NeoVim.
Thanks in advance ;-).

Hi @JulioJu
This is really a question for OmniSharp-roslyn, OmniSharp-vim is just the vim front-end and doesn't have any control over what the server does or what kind of outputs it gets, so try opening an issue there.
In the meantime, you can use vanilla vim to get this output in a quickfix, by setting your `makeprg' and 'errorformat' appropriately, something like this:
let &l:makeprg = 'dotnet build /nologo /v:q /property:GenerateFullPaths=true /clp:NoSummary .'
let &l:errorformat = ' %#%f(%l\,%c): %m'

Add those lines to a ~/.vim/after/ftplugin/cs.vim file, then try :make (quickfix) or :lmake (location window). This should output your c# errors plus linting output to the quickfix (open it with :copen or :lopen) and you can navigate to errors using standard :cnext/:cprevious etc.

@nickspoons thanks for your answer. New issue open at https://github.com/OmniSharp/omnisharp-roslyn/issues/1341


Questions on Stackoverflow

In order to promote this package, I believe we should not miss questions on Stackoverflow which have the keyword "pip-review" in them. These are accessible by just searching properly (link), however I did not find any tags for the package. Maybe we can suggest one to be created.
There are very good questions that with this package they can be solved. There are questions which make suggestions with which we may be able to improve this package. Also there are questions that need clarification to help users use this package more efficiently. There are so many opportunities there to get ideas to make this package improved.

Good suggestion! I personally don't have time to keep an eye on questions about pip-review on Stack Overflow, but I encourage everyone who wants to contribute to do so and check whether each question has a satisfying answer. If there is no good answer yet and you don't know the answer yourself either, please feel free to draw my attention over here for help.
Also, by all means get inspiration from Stack Overflow and write tickets in this repo to discuss any ideas.


feat: stripfix for target  endpoint

strip  prefix  for the target url

You can configure it more easily through the rewrite middleware.


Might not related but wanna ask: does there can have a c++ version?

Hello, wanna ask 2 questions:

will safetensors provides a c++ version, it looks more convenient then pth or onnx;
does it possible to load safetensors into some forward lib not just pytorch, such as onnxruntime etc?

Thanks, @mfuntowicz actually coded some loader in C++ if I'm not mistaken.

Not sure about creating bindings for Rust here, since the FFI with C++ is always a bit messy and it should be relatively straightforward to implement a loader in any language.
If we did, it would probably be a C export. However, making sure C-bindings are safe is not something I feel confident about doing right now.

Sure, this lib provides tensorflow, numpy, torch, paddle, jax already.
onnxruntime probably cannot be dropped in, since onnx files also contain the graph of the model.
But here's a snippet showing how to export from ONNX to this format: https://github.com/huggingface/safetensors/issues/188
I could look into how to load if you want, but in order to be efficient you would need a way to load the graph and the weights separately, and I'm not sure there's an easy primitive for that.

Provided by @mfuntowicz :¨
Would that work:
https://gist.github.com/Narsil/5d6bf307995158ad2c4994f323967284
?

C bindings should be more than adequate, and I also think they're a bit of a "must" if wider safetensors adoption is a desirable thing. I don't know what your (owners') priorities are, and I'm also unfamiliar with rust, but I see three possible levels of upstream support and I'd appreciate either

[ ] Marking the relevant functions with extern "C". AFAIU, this should already allow users to build .a/.so libraries
[ ] Maintaining the headers upstream
[ ] Ensuring the library can be located by "normal" means (CMake, pkg-config)

Hi @lucasjinreal,
I've been working on adding a C-API binding lately, please don't hesitate to have a look: https://github.com/huggingface/safetensors/pull/259


Restore build

Am interested in using this as a library, setting out to gently refactor until that is a possibility.
This PR is a little too big. It might be worth going through commit-by-commit as I just hacked and slashed until it built and tested clean with new-build:

Fix a few import-related warnings (fixed all the open external imports)
Disable the -fwarn-missing-import-lists warning (it picks up open imports, but also requires explicit constructor imports and doesn't distinguish between local vs external modules, which makes it very awkward to comply)
Move the custom Setup.hs logic into the test suite itself. This restored the build for me (Setup.hs wasn't compiling), tests were passing afterward

Had a library / executable split in here initially, but have taken it out as it would force PVP compliance

This project was in pretty bad shape when I got it code-wise and I was pretty new when I got to it so I don't think I made it much better. I actually tried to do a rewrite of it called seraph but I ran out of steam.
I don't really have much time for this project. I know you're a thoughtful coder and trust your decisions for the project. Feel free to merge any code in as you see fit. If you'd like the project itself transferred to your account or anything else, LMK.

Cheers - let's see if I get anywhere rejuvenating it (will be an infrequent weekend thing) and then consider transferring into a GitHub org or something.
My plan is to keep the angel package intact as a daemon with the current syntax, but to refactor until the internals can be broken out into an angel-core library for process supervision. Build some new things from there.


Laptop to run the presentations

and what cables/connectors are needed, general AV logistics.
I can volunteer a Windows laptop with powerpoint.

Mode slides should be PDF or Google Slides
On Mon, Sep 18, 2017 at 7:45 PM, Leo Suarez<EMAIL_ADDRESS>wrote:

and what cables/connectors are needed, general AV logistics.
I can volunteer a Windows laptop with powerpoint.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/710, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ADgoV_zM18UFcS_GhIP79KSTDv3HBoUCks5sjwCtgaJpZM4PbrEi
.

Nash Hall has an HDMI cable that we need to connect to. They have an HDMI to Lightning dongle also.

FYI - I'm getting the slides together.
Do we have a clicker?
We will also need a volunteer to
a) run the slides for lightning talks
b) be a time keeper

We purchased a clicker a few years ago so it should be in the "bins of
stuff."
On Mon, Sep 25, 2017 at 2:10 PM, Jason Hibbets<EMAIL_ADDRESS>wrote:

FYI - I'm getting the slides together.
Do we have a clicker?
We will also need a volunteer to
a) run the slides for lightning talks
b) be a time keeper
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/codeforraleigh/NCOpenPass/issues/710#issuecomment-331966098,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB3Ie23XHMv1sQUKoZG6smAuPighGUYsks5sl-yzgaJpZM4PbrEi
.

Couldn't locate. new purchased


no train script

There are no train scripts.

There are no train scripts.

Because of the data storage, it is difficult to provide a train script that can run. However, you can refer to another training code of our work (https://github.com/maxin-cn/DiffuSynth), the training of the two is more similar, and I hope it can help you.

Can LaVie model be used as the pretrained model in Latte?

There are no train scripts.

Because of the data storage, it is difficult to provide a train script that can run. However, you can refer to another training code of our work Latte, the training of the two is more similar, and I hope it can help you.

Can LaVie model be used as the pretrained model in Latte?

There are no train scripts.

Because of the data storage, it is difficult to provide a train script that can run. However, you can refer to another training code of our work Latte, the training of the two is more similar, and I hope it can help you.

The LaVie model cannot be used to initialize Latte model since Latte and LaVie are constructed by different network architectures.


Hook interfaces don't pass context

I see there is another issue that talks about an otel plugin. I don't know what the extent of that is but some thoughts after making my own proxy on top of kgo to add tracing.
I also understand the exact changes I'm suggesting would break the api, I'm more just thinking out loud.
type HookProduceRecordBuffered interface {
	OnProduceRecordBuffered(*Record)
}

If we take producing a record for example, the hook is not passed the context the record was published with which means it cannot be used for opentelemetry trace context or baggage propagation. If the hook were passed the Context the text map propagator could be used to serialize the trace context into the headers of the record before publishing.
Consuming is trickier as the api doesn't allow passing back a different context when the record is processed:
func (cl *Client) PollFetches(ctx context.Context) Fetches

But perhaps an api that is callback focused would make it possible and would require new hooks to help drive the callbacks to wrap them in tracing.
func (cl *Client) PollFetches(ctx context.Context, func (ctx context.Context, Fetches))
// But each message could be part of a different trace so really we need to invoke a callback per record
func (cl *Client) PollFetches(ctx context.Context, func (ctx context.Context, record *Record, err FetchError))

Oh I see an open PR that stuffs the context into the Record. That would work ok for producing.
On the consumer side it would still mean manually instrumented tracing to track processing the records but maybe that is ok. I don't have a good sense for how much of that would be heavily application specific.

@brennan-airtime ,  Here's how we propagate tracing context in the kotel plugin:
github.com/yianni/franz-go/blob/kotel-refactor/plugin/kotel/tracing.go

@yianni Ah ok so you're using the Ctx stuffed into the record.
Then I suppose to create the ${r.Topic} process spans I instrument that in my application code by pulling out the record.Ctx that was populated by the fetch hook.  Which honestly feels about right given all the different ways I could choose to process a batch of fetched records.

I'll be merging #201 today (adding ctx to record), but 1.8 is still a bit off pending other features (I'm a bit slow for closing the loop on some of these)

Merged -- agreed with the discussion above, the fetch hook can be used to initialize / populate the record.Context and it's about the best way to do it.
I'm going to close this issue since there isn't anything to do here (I think, though please let me know if it should be reopened). 1.8 should be released soooonish...

Nah I'm all good. This was mostly a "see something say something" issue as I was working on using the library.
It looks like putting the context in the record is the easiest way to not have to change the entire api or bloat the api surface area.
And it wouldn't be too bad in the caller.
fetches := kc.PollFetches(ctx)
fetches.EachRecord(func (record *kgo.Record) {
  ctx, span := tracer.Start(record.Ctx, fmt.Sprintf("%s process", record.Topic))
  defer span.End()
  // ... do work
})

@yianni It may be nice for your kotel package to expose code that can fill in a span with the standard attributes from a record.


Install mailcatcher to both VM and CI server

gem install mailcatcher
apt-get install libsqlite3-dev
gem install eventmachine -v 1.0.3
gem uninstall eventmachine -v 1.0.4

Run mailcatcher
mailcatcher --http-ip SERVER_IP --smtp-ip SERVER_IP --http-port 1080 --smtp-port 1025 --no-quit

php.ini
[mail function]
SMTP = <IP_ADDRESS>
smtp_port = 1025
sendmail_path = /usr/bin/env catchmail --smtp-ip <IP_ADDRESS> --smtp-port 1025

We need pull code from TFALL project. MailCatcher + nginx authentication.

@danylevskyi please provide a link to another tool, based on golang that is more stable than mailcatcher

https://github.com/geerlingguy/ansible-role-mailhog


Prepare for Website Decommission

Adds README text, files, and scripts that are currently on the GitHub pages website in preparation for that site going away:

Summary of the components of our digital infrastructure
Example scripts and RAC bagit profile
Project Electron Values
Licensing Principles and Approaches

Removes some files that are now more appropriately located in internal RAC directories:

rfcs
Bibliography
Transfer milestones

Note that the digital records transfer documentation has been added to the Docs Site, and the Principles for Archival Access will be moved to the DTeamDocs repo (see https://github.com/RockefellerArchiveCenter/DTeamDocs/pull/48)
Do not delete the gh-pages branch until a redirect is in place for projectelectron.rockarch.org website to redirect to this repository.

Just confirming that the things which were entirely removed (rfcs, Bibliography, Transfer milestones) ended up in Sharepoint?

Just confirming that the things which were entirely removed (rfcs, Bibliography, Transfer milestones) ended up in Sharepoint?

Yes, in the "Completed Projects" directory.


Precommix

Add precommix for linting and formatting.

[x] Provide a dev shell
[x] Provide a nix package
[x] Provide a nix flake
[ ] Fix CI
[x] Document installation
[ ] Document hacking
[x] Provide as much architectures as possible

Cool! This is now working! To try it out:
nix shell github:copier-org/copier/precommix

That will load copier installed with nix in your shell. Now go play with it.
Pending: docs, fix CI (all should break right now), provide more arches.

Cool! This is now working! To try it out:
nix shell github:copier-org/copier/precommix

That will load copier installed with nix in your shell. Now go play with it.
Pending: docs, fix CI (all should break right now), provide more arches.

Thank you for packaging copier! Using the exact flake you provided worked at first but I believe the inputs used some GitHub nomenclature related to pull requests so they stopped working (the inputs with # hack comments afterwards). I packaged it using exact git hashes here (https://github.com/nialov/nialov-py-template) and added git into the path of the copier executable as it seemed to error without it. Should work with nix run github:nialov/nialov-py-template#copier.

@yajo FYI there is a project that integrates pre-commit with Nix: https://github.com/cachix/pre-commit-hooks.nix


Clean up when HTTP connection closed.

Load end may never occur its important to clean up when connection is closed.

Could you find the real example when this case happens?

node -e "require('http').createServer().listen(8080)" &
then
cef-pdf --url=http://localhost:8080
this same situation (the connection being left open indefinitely) could happen accidentally on pretty much any web server.

E.g.
./src/Release/cef-pdf.app/Contents/MacOS/cef-pdf --server &
node -e "require('http').createServer().listen(8080)" &
curl -X POST http://localhost:9288/_.pdf --header "Content-Location: http://localhost:8080"

to act as a resilient http service cef-pdf must keep its connections as a client and as a server alive, must clean up all allocated resources on a connection by connection basis as these connections are closed and must not crash.


IONIC_ENV is not available on Ionic Cloud

Note: for support questions, please use one of these channels:
https://forum.ionicframework.com/
http://ionicworldwide.herokuapp.com/
Short description of the problem:
I reported #261 a few days ago, and while this is not the place to report this again, I'm in desperate need of an answer.
It seems IONIC_ENV is not set when the package is build in the Ionic Cloud, which is quite important for apps that depend on that variable.
What behavior are you expecting?
IONIC_ENV should be available during the build.
Steps to reproduce:
1.
2.
3.
insert any relevant code between the above and below backticks

Which @ionic/app-scripts version are you using?
Other information: (e.g. stacktraces, related issues, suggestions how to fix, stackoverflow links, forum links, etc)

Ok, so I now realized that the buildAndroidRelease() function does not build the app in the cloud, it actually just copies the contents of www and then it packages it. It's a little misleading, I thought that it would actually take the app source and build in the cloud.

Ok, so I'm using this npm package to also build before sending to cloud and packaging. I'm using the build() function.
Though I wonder if there is a way to turn off the logging.


create .zenodo.json; closes #43

I've created a list (and order) of authors based on the commit logs. This PR also proposes to use another short description of RK-Opt (which you would nee to update if you like it, @ketch, since I do not have the rights to do so). I think the old description does not reflect the versatility of RK-Opt anymore.

This needs some modifications but I will just merge and then edit.


The SSL certificate for docs.vavr.io is expired

https://docs.vavr.io/
https://www.ssllabs.com/ssltest/analyze.html?d=docs.vavr.io&s=2606%3A50c0%3A8001%3A0%3A0%3A0%3A0%3A153&latest
~ ❯❯❯ openssl s_client -connect docs.vavr.io:443 -showcerts | openssl x509 -text -noout
Connecting to 2606:50c0:8002::153
depth=2 C=US, O=Internet Security Research Group, CN=ISRG Root X1
verify return:1
depth=1 C=US, O=Let's Encrypt, CN=R11
verify return:1
depth=0 CN=docs.vavr.io
verify error:num=10:certificate has expired
notAfter=Nov  4 05:10:06 2024 GMT
verify return:1
depth=0 CN=docs.vavr.io
notAfter=Nov  4 05:10:06 2024 GMT
verify return:1
Certificate:

....

Thanks, this should self-resolve soon:

There are some weird issues on GitHub's site. I need to play with the DNS setup a bit. Site will be inaccessible sometimes

Something is wrong. I reached out to GitHub support.

This is done. For some reason, the previous DNS setup was not ok for GitHub Pages. One small tweak there, and we are fine :)

Thanks for fixing. 👍


Displaying nested arrays of objects?

What version of React-Table are you using?
"react-table": "^6.5.3"
What bug are you experiencing, or what feature are you proposing?
is it possible to display array of objects in this table as a tree view with pivoting ? what I should write in accessor?
{
"name":"foo",
"friends":[
{
"firsName":"John",
"lastName":"Smith",
"age":"34",
},
{
"firsName":"Ann",
"lastName":"Smith",
"age":"32",
},
{
"firsName":"Lily",
"lastName":"Smith",
"age":"4",
}
]
}

This is a question of implementation. We invite you to join the slack channel and ask for help there or on stack overflow. Thanks!


Fix broken development setup

This removes circular dependency in Makefile added in #10278
Unblocks #10052
Fixes #10446
Fixes #10447

@di @ewdurbin please review.

@ewjoachim addressed all comments. PTAL.

@di @ewjoachim @ewdurbin ping?


Update concepts-connectivity-architecture.md - Japan West

@code-sidd
Thanks for approving the PR. For future PRs, please note that a #sign-off comment is the best way to signal to the review team that a PR is ready to merge.

#sign-off


PopmenuButton Mouse tracker issue

The following IndexError was thrown during a scheduler callback:
RangeError (index): Index out of range: index should be less than 2: 2

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 266:49      throw_
dart-sdk/lib/_internal/js_dev_runtime/private/js_array.dart 581:7                 _get]
packages/fl_chart/src/chart/bar_chart/bar_chart_painter.dart 581:36               handleTouch
packages/fl_chart/src/chart/bar_chart/bar_chart_renderer.dart 102:33              getResponseAtLocation
packages/fl_chart/src/chart/base/base_chart/render_base_chart.dart 154:18         [_notifyTouchEvent]
packages/fl_chart/src/chart/base/base_chart/render_base_chart.dart 136:18         <fn>
packages/flutter/src/rendering/mouse_tracker.dart 424:19                          _handleDeviceUpdateMouseEvents
packages/flutter/src/rendering/mouse_tracker.dart 285:5                           [_handleDeviceUpdate]
packages/flutter/src/rendering/mouse_tracker.dart 372:9                           <fn>
packages/flutter/src/rendering/mouse_tracker.dart 210:9                           [_deviceUpdatePhase]
packages/flutter/src/rendering/mouse_tracker.dart 366:5                           updateAllDevices
packages/flutter/src/rendering/binding.dart 398:7                                 <fn>
packages/flutter/src/scheduler/binding.dart 1289:15                               [_invokeFrameCallback]
packages/flutter/src/scheduler/binding.dart 1227:9                                handleDrawFrame
packages/flutter/src/scheduler/binding.dart 1076:5                                [_handleDrawFrame]
lib/_engine/engine/platform_dispatcher.dart 1168:13                               invoke
lib/_engine/engine/platform_dispatcher.dart 218:5                                 invokeOnDrawFrame
lib/_engine/engine/initialization.dart 190:45                                     <fn>
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 367:37  _checkAndCall
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 372:39  dcall

packages/fl_chart/src/chart/bar_chart/bar_chart_painter.dart 581:36               handleTouch
packages/fl_chart/src/chart/bar_chart/bar_chart_renderer.dart 102:33              getResponseAtLocation
packages/fl_chart/src/chart/base/base_chart/render_base_chart.dart 154:18         [_notifyTouchEvent]
packages/fl_chart/src/chart/base/base_chart/render_base_chart.dart 136:18

@RanveerSingh1997
This seems to be coming from third party package fl_chart. If you remove the package code and try again, do you get same error log ? If so, provide minimal reproducible code sample without any third party package code that shows this error log.

provide minimal reproducible code sample without any third party package code that shows this error log.


Make !generateplaylist genre an optional field

Users should be able to choose what filters to provide, for ex: if I wanted !generateplaylist the weeknd, I should get a playlist of just the weeknd's songs, but I should also be able to do !generateplaylist rap, and get a playlist of rap songs.

Marking as resolved, !genplay now has genre as an optional parameter, also added validation to make sure at least one filter does get entered and we notify the user if this is not the case, also will be documentation proper usage of the command should a genre not be entered. Proper usage is as follows: !genplay "" "Drake", where the "" represents an empty string.


Predictions don't close

When selecting a prediction, the prediction text is put into the text field. This might trigger a text change and therefore another prediction search with just 1 result matching the result from the first search.

I resolved this by making sure the text that is on the TextFormField should not be any of the suggested locations
Just edit the google_places_flutter from dependancy level and add the line 100 right before

alPredictions.addAll(subscriptionResponse.predictions!);

subscriptionResponse.predictions!.removeWhere((element)` => element.description==text);

This will solve this problem

The thing is overlay is not removed if latlong fetch is not enabled, so here change is needed:
https://github.com/Shrutimahajan/Google-AutoComplete-TextField-Flutter/blob/4bfdeaa92eb430dbc4b944b0b0992797bcc919dc/lib/google_places_flutter.dart#L148C32-L153C49
if (!widget.isLatLngRequired){
removeOverlay(); // added
 return;
}


Implement Health Status Endpoint

Codecov Report

Merging #18 into main will increase coverage by 0.08%.
The diff coverage is 41.93%.

@@            Coverage Diff             @@
##             main      #18      +/-   ##
==========================================
+ Coverage   50.29%   50.37%   +0.08%
==========================================
  Files          17       17
  Lines        1032     1056      +24
==========================================
+ Hits          519      532      +13
- Misses        438      446       +8
- Partials       75       78       +3

Impacted Files
Coverage Δ

server/service.go
52.53% <7.69%> (-0.56%)
:arrow_down:

server/util.go
68.42% <66.66%> (-1.58%)
:arrow_down:


PayPal popup loads indefinitely after opening

General information

SDK version: 3.75.19
Environment: both
Browser and OS: Chrome 100.0.4896.127 (Official Build) (64-bit) (cohort: Stable)

Issue description
This issue is not reproducible, but happens every now and then, on sandbox, production, during development and for real users.
After the PayPal popup was opened during Checkout, it would show the small loading spinner (the same thing that you see when you open the popup url directly in the browser), but it just keeps spinning and never stops.
This is not the big loading spinner to indicate your account is loading.
If I close the popup in this state, the the call to tokenize() (which opened the popup) will never return and reloading the page seems to never complete. During development, I have to create a new tab, and start the checkout all over to get a usable state.

👋 @ciriousjoker thanks for reaching out. Next time you encounter this issue in sandbox or production, can you contact support referencing this issue and make a note of the following info to help our investigation?

timestamps of when it occurred (UTC preferred but honestly just let us know the timezone if it's not UTC)
browser(s) impacted (I see you mentioned Chrome in this issue, if there's any other browsers where you're seeing this happen please let us know).
link to the site where you saw this behavior or a code sample

As you mentioned, this is fairly difficult to reproduce, but that doesn't mean we shouldn't try and get more info. Can you confirm that you're using the paypal-checkout component and not the paypal component?

You might want to check out https://github.com/braintree/braintree-web/issues/642

Closing this issue since we haven't heard from OP to get more info.


Host Documentation with Github Pages

I really think we need to get all of your great documentation out to the world @davepatterson!

For sure. I have been meaning to set it up but it always slips my mind. It will be up very soon.

I added docs to Github pages in #99.


Dump the in-memory data for crash recovery

We do not necessarily need to use a persistent database for the instance registry, since the persistent data may become invalid in any second the Instance Registry is not actually running. However, it would be useful to be able to recover from an unexpected crash by re-loading the latest data set. There may be an routine that tries to verify the correctness of the data.
This requires the Instance Registry to write its component data to a dump file whenever it changes. It must delete this file on regular shutdown, and must check and load it on startup.
This does depend on the Instance Registry being refactored first (#9).

Closed with #18


Programatic undoable text changes aren't undoable

Poedit 2.0beta4
Windows 10
In the Translation text field, when I press Ctrl+B, the current text is replaced with the source text. For me as a user, this is a single operation, so I should be able to undo that by pressing Ctrl+Z. But when I do that, the Translation field gets empty. I have no chance of getting the original translated text back, not even by pressing Ctrl+Z several times in a row.
This already happened several times to me when I wanted to press Ctrl+V to insert some text.

Confirmed present in 1.8.12 too.

Seems to affect Copy from singular and applying suggestions as well.


when the kind value is transaction and toaddress is new, it will consume more 0.257 tezos，which kind can  avoid the more consumption 0.257

This isn't an issue with Go Tezos. But unfortunately you can't. That is called a burn-cap and it's for originating and revealing wallets.


Can I run an RGBD Dataset through Kimera-VIO?

I have RGBD data from an Intel Realsense camera (in the form of bag files), and would like to run it through VIO and RPGO. However, I am starting to wonder if this would even be possible, since I believe VIO by definition is designed only for stereo cameras.
Any insight would be appreciated! Thank you.

I am curious about this, too! I would like to test Kimera on our TIAGo robot that is equipped with an RGBD camera.


feat(single-mode): recognize condition 切れ者

Description
Add a template and unit test to recognize 切れ者.
Changes

add : support sharp of condition

Tests

Successful unit testing of context_test.py

Others

Related to #287

thanks! 👍


Add private user data to user entity

I have tested this and can confirm it works on production. 👍


Update sbt-scalajs to 0.6.31

Updates org.scala-js:sbt-scalajs from 0.6.29 to 0.6.31.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-js", artifactId = "sbt-scalajs" } ]

labels: semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   77.89%   77.89%
=======================================
  Files           6        6
  Lines          95       95
=======================================
  Hits           74       74
  Misses         21       21

Continue to review full report at Codecov.

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


ffcv loader and batch_size=1

I have a problem related to the ffcv loader. In my code, when I set the batch_size equal to one, the ffcv loader does not work, i.e., when I try to iterate through the dataset, the loader does not return any element. While when I set the batch_size to a different number than one, the loader does work.
      loader = ffcv.loader.Loader(
          str(filename),
          batch_size=1,
          seed=1,
          order=OrderOption.QUASI_RANDOM,
          pipelines=pipelines,
          drop_last=False,
          indices=indices,
          distributed=False,
          num_workers=3,
          os_cache=True,
      )

hi @crocis ! My first guess is that this is an issue with QUASI_RANDOM loading, if you change QUASI_RANDOM to SEQUENTIAL does the issue persist?


⚠ Bofrost.be has degraded performance

In 5e1b3d2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 3639 ms

Resolved: Bofrost.be performance has improved in 3234b91.


Let install proceed before configuring email

This introduces our new, streamlined installer flow, hidden behind an undocumented experimental option.
With this change plus #8166 (for which we have a PR today, #8198), all the pieces will be in place to consolidate steps 2, 3, 4, 5, and 6 of the install instructions into just running the installer, followed by going through a flow in the webapp; it'll just be a matter of writing the new instructions, and flipping a switch here.

Merged, after fixing the one new merge conflict.  Thanks @gnprice!
For EXPRESS_INSTALL, I assume the plan is to flip around the sense of the variable (so one can pass --partial-install or something) once we're quite happy with this?  The use case would I think be cases where one is planning to import data from another database.  Though eventually, I suppose, we could just have an explicit tool for that use case.
Probably after the next round of improvements, we can start getting folks to QA this new experience.

For EXPRESS_INSTALL, I assume the plan is to flip around the sense of the variable (so one can pass --partial-install or something) once we're quite happy with this? The use case would I think be cases where one is planning to import data from another database. Though eventually, I suppose, we could just have an explicit tool for that use case.

Yeah, could do that. My plan was to just unconditionally do this "express install", but it's easy to have an option, and that could be a use case. Might call it something like --no-init-db.

Yeah, --no-init-db makes sense to me.


Adding signup page

PR adds:

Signup page
Reusable code component

Style of code block are not exactly like designs, I can plan another task to add custom styles to code block ad it is rather bigger effort.

i don't think it's worth it to style the code block 100% like the design. the way you have it for now is good enough

there is no submit button for the email

could we make it so that the left and right sides of the page scroll independently?

could we make it so that the left and right sides of the page scroll independently?

I fixes this in a following PR.

there is no submit button for the email

I fixed this also in a following PR


Add nativeM relation

The nativeM relation represents native methods.
Native methods retain their NATIVE Java modifier and are excluded from the list of methods to be processed (since they don't have bodies).  These are added to the relation based on the existence of the modifier.
The exceptions are the native methods which are simulated by Petablox, like arraycopy.  The NATIVE modifier must be removed for Soot to allow them to have a body.  These methods are added manually to the relation.
Knowing which methods are native is important for the information flow analysis I do in my research.

I've realized today that there may be more changes to make.  In RelVT.java, for example, there is the code:
public void visit(SootMethod m) {
    if (m.isAbstract())
        return;
    ....
}

To my knowledge, this prevents you from trying to look through the body of a method that doesn't have one.  So, to support native methods, we would need to add a guard against looking at bodies of native methods:
public void visit(SootMethod m) {
    if (m.isAbstract() || m.isNative())
        return;
    ....
}

(There are many more instances of this that would need to change.)
@akamath32 , @sulekhark : My question is, why is the code not:
public void visit(SootMethod m) {
    if (!m.isConcrete())
        return;
    ....
}

You can see the implementation of isConcrete here.  It would guard against looking at abstract methods and native methods.  But it would also guard against looking at phantom methods.  The existing code does not guard against looking at phantom methods.  Is there a reason for this?  Should the change to the code be the first change I showed above or the second?

Per offline discussion, isConcrete is the way to go.


Reduce the gap between the action buttons in the Datasource detail page

In the Datasource detail page: The current gap between the buttons (16px) is too much and doesn't look proper. Use 12px gap and use a design system token (--ads-v2-spaces-4) for this instead of a direct value.
https://pasteboard.co/9NL2MSQSEZZc.png

The image is not visible.

@asimdelvi Updated the image URL.

Can you assign me this issue?


Masize Snippet Compose Box Issues

These are the current issues with the compose box that need fixing.

[x] The overlay prevents clicking on anything when any compose box is open.

[ ] On resizing, compose boxes get stacked

[ ] Different resolutions need different no. of rows of textarea

Btw, I'm probably late in saying this, did you check npm if something similar to a compose box already exists? Seems like a common UI element.
Googling for "html display gmail like textbox" yields this and this which then tells us that we need to google npm dockmodal which gives interesting hits.

http://preview.codecanyon.net/item/dockmodal-gmail-like-dockable-modal-dialog-plugin/full_screen_preview/6550494?_ga=1.74447826.1950658917.1489035938
https://codepen.io/guruprakash/pen/YpNJed


Roll engine 7f7250f47636..33ad6757e883 (2 commits)

git@github.com:flutter/engine.git/compare/7f7250f47636...33ad6757e883
git log 7f7250f47636..33ad6757e883 --no-merges --oneline
2019-09-19<EMAIL_ADDRESS>[flutter_runner] Port over the tuning advice (flutter/engine#12345)
2019-09-19<EMAIL_ADDRESS>[flutter_runner] Remove usages of shared snapshots from CC sources (flutter/engine#12343)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md

Auto-roller completed checks. About to merge.


Add fast method to special DenseTaggedImage convert cases

https://github.com/JuliaIO/ImageIO.jl/pull/26 enforces the conversion to Array type, this patch makes the conversion not allocating new memory for some array types.

This looks good. I rebased on master and if all the tests pass, I'll merge.
thanks!

Codecov Report

Merging #45 (caaa64d) into master (d640510) will decrease coverage by 1.65%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
- Coverage   89.52%   87.86%   -1.66%
==========================================
  Files          12       12
  Lines         544      544
==========================================
- Hits          487      478       -9
- Misses         57       66       +9

Impacted Files
Coverage Δ

src/types/dense.jl
97.05% <100.00%> (+0.04%)
:arrow_up:

src/TiffImages.jl
0.00% <0.00%> (-100.00%)
:arrow_down:

src/utils.jl
70.37% <0.00%> (-5.56%)
:arrow_down:

src/compression.jl
90.00% <0.00%> (-5.00%)
:arrow_down:

src/layout.jl
86.53% <0.00%> (-1.93%)
:arrow_down:

Continue to review full report at Codecov.

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