Scale measurement

@daanpape @DujardinGijs I've released a new version to npm v0.0.3 which includes this PR and the previous one


make lookup operator honor refined return type of "get()"

[@jvasileff] In "6.8.7. Correspondence, subrange, and stream operators", the lookup operator is defined as yielding the result of lhs.get(index) with type Y? for lhs Correspondence<X,Y>. But the actual return type of lhs.get(index) may be a subtype of Y?.
It would be nice if the lookup operator honored the actual return type of get rather than assuming an optional type.
Usage example:
object multimap satisfies Correspondence<String, [Integer*]> {
    shared actual Boolean defines(String key) => nothing;
    shared actual [Integer*] get(String key) => [];
}
[Integer*] result1 = multimap[""]; // error Integer[]? is not assignable to Integer[]
[Integer*] result2 = multimap.get(""); // ok

[Migrated from ceylon/ceylon-spec#1411]

Bumped into this too. :+1: I created a map that automatically creates items if they did not exist in the map before, and thus the return value is never null..

If only I would have had the foresight to define Correspondence and Map/List like this:
shared interface Correspondence<in Key, out Item=Anything>
        given Key satisfies Object {
    shared formal Item get(Key key);

    ...
}

Or to have defined Map like this:
shared interface Map<out Key=Object, out Item=Anything>
        satisfies Collection<Key->Item> &
                  Correspondence<Object,Item?>
        given Key satisfies Object { ... }

shared interface List<out Element=Anything>
        satisfies Collection<Element> &
                  Correspondence<Integer,Element?> &
                  Ranged<Integer,Element,List<Element>> {

I screwed up here, it seems to me :-(

Does the x[y] operator have to be tied to the Correspondence interface? It can't just alias to x.get(y) ?

On the other hand, there's a somewhat strong argument that for a "total" correspondence, the correct type to use is X(Y), not Correspondence<X,Y>. The operations of Correspondence assume that we are dealing with a partial function.

Does the x[y] operator have to be tied to the Correspondence interface? It can't just alias to x.get(y) ?

Well, that's the way all other operators in Ceylon are defined.

We could, in principle, the next time we break BC, make the following change:
shared interface Correspondence<in Key, out Item=Anything, out Absent=Null>
        given Key satisfies Object {
    shared formal Item | Absent get(Key key);

    ...
}

is that better/less breaking than the earlier suggestion you had?

Less breaking.

for a "total" correspondence, the correct type to use is X(Y)

That would require #3950 to be useful.

For the Correspondence interface, I'd keep Null as the type to represent absence and instead restrict Item to be of type Object, i.e. changing the interface to:
shared interface Correspondence<in Key, out Item=Object>
    given Key satisfies Object
    given Item satisfies Object

For the Correspondence interface, I'd keep Null as the type to represent absence and instead restrict Item to be of type Object, i.e. changing the interface to:
shared interface Correspondence<in Key, out Item=Object>
    given Key satisfies Object
    given Item satisfies Object

If one tries to write a generic function for storing arbritary values, Absent could be included in the Item type, too. In that case, the return type may still be ambiguous. What would make more sense is to stick with Item?, but let Item satisfy Object. For example, if Null needs to be stored, one could define Item as Supplier<Anything> to differentiate between the null returned for "no item found" and the null, that is stored as an item.
Alternatively, although I don't know if it's possible, Absent would make sense, if it's possible to tell the compiler something like "given Item not satisfies Absent", i.e. Item may not include the type of Absent in order to rule out ambiguity when calling Correspondence.get.

@phlopsi There's nothing stopping you from doing that today. For example:
alias Maybe<Element> => [Element] | [];
Map<String, Maybe<Integer>> m = map { "a"->[], "b"->[1] };

But I haven't seen that done in Ceylon code, and I'm not sure it would be a good idea to force people to use that pattern.
Taking a step back, we know that optionals can be represented by monads (Maybe<Element>) or union types (Element | Null). They have the following respective downsides:

With maybes, you can wind up with deeply nested types like Maybe<Maybe<Maybe<Element>>> that carry more information that you may want, if all you need is Maybe<Element> (e.g. listOfMaybeStrings.first would return Maybe<Maybe<Element>>)
With union types, Element | Null | Null | Null always collapses to Element | Null, resulting in an ambiguity when you do care about the intermediate steps.

Which one is better varies by use case. (Although, it's worth noting that unions have the additional advantage that Element is assignable to Element | Null.)
Now, your suggestion is have two standard ways to represent optionals in order to eliminate the above tradeoff. But I doubt that would work well in practice:

there would be two standard/common ways to do things like exists, else, and ?. for optionals which would make both reading and writing code more difficult
inevitably you'd run into confusing mixes of optional types at various depths, like Maybe<Maybe<String?>?>

A very "simple" type that I think would be confusing to work with:
Map<String, Maybe<Map<String, Maybe<String>>>>

as opposed to the more Ceylonic:
Map<String, Map<String, String?>?>

So, I think it's better for a language to stick with one approach or the other, which for Ceylon has been to use union types.

Deep-nesting has nothing to do with this. It's exactly 1 wrapper, if and only if the Absent type (currently fixed to Null) would be included in the Item type, if no wrapper is used. Your argument seems overly generic and not tailored to the actual problem.
If the language provides a way to model the inverse of given A satisfies B, that would be a huge improvement, because the strong point of this language is, that it is explicit and wherever possible, uses syntax errors instead of runtime errors, to detect errors early instead of too late. Having a part of the language be ambiguous and prone to semantic errors on purpose looks like a

@jvasileff Deep-nesting has nothing to do with this. It's exactly 1 wrapper, if and only if the Absent type (currently fixed to Null) would be included in the Item type.
If the language would provide a way to model the inverse of given A satisfies B, that would be a huge improvement, because the strong point of this language is, that it is explicit and wherever possible, uses syntax errors instead of runtime errors, to detect errors as early as possible, namely during typechecking/compilation. Having a part of the language be ambiguous by design and prone to semantic errors doesn't look ceylonic to me, at all, if there is an obvious way to model them as syntactic errors, instead.
The only other way to not be ambiguous is to reserve a type, that may not be included in the Item type. Since Null is the only type, that is not an Object and Object is implicitly satisfied by everything except Null, restricting Item to being an Object and reserving Null as a pure Absent type, is the only logical choice I'm seeing, unless I'm missing important information.


Bug with menu in ember-paper

When you use a paper-menu from the ember-paper add-on, the menu doesn’t disappear anymore when that add-on is also installed. The bug seems to be related to Ember Paper’s backdrop component.

Can you reproduce this on the ember-paper docs website?


Remove support for CloudantToolKit

What
Remove support methods and classes for CloudantToolKit.
How
Remove SavedHTTPAttachment class and DocumentRevision#buildRevisionFromMap
method.
Reviewers
reviewer @emlaver
Issues
Fixes #254

LGTM +1


Would this work for CodeMirror 5.2 ?

Hi,
I was looking for a simple UI for CodeMirror very similar to what you developed. But I am currently using latest version (CodeMirror 5.2). Do you think it could work?

I honestly have no idea what it'll do with CM 5.2. It's been quite a while since I've done much with this project. I'd love to hear about how it goes if you give it a try.


Fix typos

Proposed changes
Fixed typos.

Deploy preview for docsdocker ready!
Built with commit 00113883ce790a452337ec775f030ca6b4c0941c
https://deploy-preview-9494--docsdocker.netlify.com


Pass props to keyframes?

How would I go about passing props to keyframes? Like this?
const frames = ({color}) => ({
  '0%': {
    background: color,
    transform: 'scale(1.0)'
  },
  '20%': {
    background: color,
    transform: 'scale(1.3)'
  },
  '35%': {
    background: color,
    transform: 'scale(1.0)'
  },
  '50%': {
    background: color,
    transform: 'scale(1.3)'
  },
  '100%': {
    background: color,
    transform: 'scale(1.0)'
  },
})

Versions:
Fela 4.3.2
Preact-Fela 4.3.2
Preact 8.1.0
Environment: DOM

as per docs this is the way to do it.
Here's a codepen with a short example:
https://codepen.io/queckezz/pen/qmaamY

Guess this one is solved? Reopen if there are still questions :)


header missing in firefox

I found the tag "ag-body" is not assigned the property "padding-top"
only in firefox(my version is 31.0)

in the first time when i opened the page it was all right, but after 15 minutes it appeared
since then it has continued despite the restart of the browser

same as https://github.com/ceolter/angular-grid/issues/124

Yes

it's ok now


Autocomplete widget customization

Would be great if the autocomplete widget could be customized.
Late coders for web based apps wont likely notice, but programmers coming from more conventional IDEs (Visual Studio, MonoDevelop/Xamarin, Rider, Eclipse, etc) find the suggestions widget very ugly looking (no offense).
Could we have tools for customizing the autocomplete rendering? Icons, font family and background color (font size, and line height are already there it seems, but wont affect the signature help box sadly). Also a layout template would be ideal awesome.

This can all be done by now - there is themes and font configs


cert-manager - unknown field "acme" in io.cert-manager.v1alpha2.Certificate.spec

On fresh clone and install CRDs installed, happens right after make helm-install:
Error: unable to build kubernetes objects from release manifest: [unable to recognize "": no matches for kind "Deployment" in version "extensions/v1beta1", error validating "": error validating data: ValidationError(Certificate.spec): unknown field "acme" in io.cert-manager.v1alpha2.Certificate.spec]

Did you try to update dependencies?

Yeps all usual except I run it on Kubernetes 1.17.0 now.

I think this fixes it, but have no idea why magic-namespace was pulled in helm/app once it was removed from requirements.lock, since it has been moved to helm/namespace #34

Merged #34. Please close if no longer valid.


Not able to delete a custom resource

How did you install the Amplify CLI?
npm install -g @aws-amplify/cli
If applicable, what version of Node.js are you using?
14.19.3
Amplify CLI Version
10.5.2
What operating system are you using?
Windows
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No
Describe the bug
Hello,
In my Project i have two Custom Resources.

PSBuildSearchIndex
PSBuildOpenSearch

Each Custom Resource creates a Step Function.
One Step Function (PSBuildSearchIndex) is called by the other one (PSBuildOpenSearch).
So there is a dependency in the backend-config.json

But now, i want to delete the second StepFunction (PSBuildOpenSearch).
I already adjusted my cdk-stack for the first StepFunction in a way, that the second is no longer needed.
But i'm not able to delete the second StepFunction via the amplify command: "amplify custom remove"

i can't remove the dependency via the amplify command "amplify custom update" because it just gives me the possibility to edit the stack-files for custom resources.
i already tried to remove the dependency directly in the backend-config.json. but the error still exits.
=> i assume that the amplify cli considers the backend-config.json from the "#current-cloud-backend" folder?!
I'm not sure if it's a good idea to edit files from the "#current-cloud-backend" folder
if i remove the dependency from the backend-config.json in the amplify folder, and do an "amplify push", then amplify restores the dependency.
Can someone help me to delete the dependency/custom resource?
Kind Regards
Stefan
Expected behavior
Deletion of the custom resource should be possible
Reproduction steps
edit backend-config.json (remove the dependency)
amplify custom remove
=> Error
Project Identifier
Project Identifier: fb73b5648b874fc6d944e3d311bcaaa0
report-1672786058380.zip
Log output

# Put your logs below this line

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @StefanVolkmer-WW :wave: thanks for raising this! After removing the dependency from backend-config.json it is possible this dependency still exists in amplify/backend/amplify-meta.json. Can you try running amplify env checkout <current-env-name> after manually removing the dependency in backend-config.json to see if this alleviates the issue on push?

Hi @josefaidt :
Thanks for your response. i already tried an env checkout after deleting the dependency. but with the env checkout the dependency was added again.
But 2 Minutes ago, I found the Reason.
In the first StepFunction had a
AmplifyHelpers.addResourceDependency
This leads that the dependency is added everytime to the backend-config.json when you do an amplify push or an amplify env checkout.
i removed the coding, then i was able to delete the dependency in the backend-config.json, and do afterwards an env checkout, which doesn't add the dependency again.


Stamp metadata into top level init files on build.

Addresses #242

Adds some code in the setup.py file that stamps the __version__ and __author__ metadata into the  top level init.py files.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #285   +/-   ##
=======================================
  Coverage   90.95%   90.95%
=======================================
  Files          49       49
  Lines        4325     4325
=======================================
  Hits         3934     3934
  Misses        391      391

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 33ad026...3cdbc40. Read the comment docs.


Function with if/else returns null from else block

I noticed that if i declare a function with an if/else statement, the else block doesn't actually return a value. The function just returns "null". I thought maybe the function was continuing execution, so i put a ret -1 at the end of the method, but null is still returned.
[Fact]
public void TestScribanIfElseFunction()
{
    var template = Template.Parse(@"
    func testIfElse
        if $0 < 0
            ret 1
        else
            ret 0
        end
        ret -1
    end
    testIfElse testValue
    ", lexerOptions: new LexerOptions { KeepTrivia = false, Mode = ScriptMode.ScriptOnly });
    var templateContext = new TemplateContext
    {
        LoopLimit = int.MaxValue,
        MemberRenamer = s => s.Name.ToCamelCase()
    };

    templateContext.BuiltinObject.SetValue("testValue", -1, true);
    var result = template.Evaluate(templateContext);
    Assert.Equal(1, result);
    templateContext.BuiltinObject.SetValue("testValue", 1, true);
    result = template.Evaluate(templateContext);
    Assert.Equal(0, result); // returns null
}

Good catch, this should be fixed by commit a75c1a6 and available soon in  2.3.4
It was implicitly fixed in 3.0.0 alpha+


Probes values support

Need it in my environment and rather not store a local copy of the chart.
Nothing fancy just support for liveness and readiness values in values.yaml

Mainly our problem was that the timeout and and failure threshold were too low, initially just saved the chart and deployed with different values but saw this project is still maintained so prefer to just get the chart and change the hardcoded values of the probes

Hi @viliger18
We've released a new version of the chart with your added fields:
https://github.com/sighupio/gatekeeper-policy-manager/releases/tag/gatekeeper-policy-manager-0.13.0
Thank you for the collaboration!


Fix DetectChanges::last_changed returning the wrong tick

Objective
Make last_changed behave as described in its docs.
Solution

Return changed instead of last_change_tick. last_change_tick is the system's previous tick and is just used for comparison.
Update the docs of the similarly named set_last_changed (which does correctly interact with last_change_tick) to clarify that the two functions touch different data. (I advocate for renaming one or the other if anyone has any good suggestions).

It also might make sense to return a cloned Tick instead of u32.

Changelog

Fixed DetectChanges::last_changed returning the wrong value.

Migration Guide

The incorrect value that was being returned was the previous run tick of the system checking for changed values. If you depended on this value, you can get it from

The incorrect value that was being returned was the previous run tick of the system checking for changed values. If you depended on this value, you can get it from

This is an incomplete sentence fragment.


Add multivalue RDN and request/subject overwriting report.

Add multivalue RDN and request/subject overwriting report to the x509 utility.
Useful if a request needs to be tweaked (or if a variant CN or other RDN needs to be added for legal reasons).
Replaces https://github.com/openssl/openssl/pull/10570. Also updates documentation and testcases.

[X] documentation is added or updated
[X] tests are added or updated.

This PR doesn't do what you advertise...  It rather seems like you're making multivalue RDN parsing support optional, the same way we already have with openssl req.  Without this PR, openssl x509 supports multivalue RDN parsing unconditionally.
Small nit: that option is called -multivalue-rdn with openssl req, it would be better if you used the same option name with openssl x509

I do not think this should be merged. This goes against the direction we did in 1a337754a92f8324cfb3180a11039ce503c593a7.

Yes - agreed. I'll swap this and double check the test cases before re-submitting.


alertmanager flag unknown in v2.0-beta

When passing the "-alertmanger.url" flag (v2.0-beta), I'm receiving the following message:
"prometheus: error: unknown long flag '--alertmanager.url'"
I tried it without the double "--" and received the following message:
"prometheus: error: unknown short flag '-a'"

Yeah, the flag (which was previously deprecated) has been removed in 2.x and Alertmanagers can only be configured in the config file now. See https://prometheus.io/blog/2017/06/21/prometheus-20-alpha3-new-rule-format/
That'll be more properly documented soon.


Update the readme in accordance with PR #4617

Reflecting the changes in PR #4617 in the ReadMe.

Since these doco changes should be tied to the alias changes, they could just go in the same PR. Even after you've already made a PR, you can revise it by making additional commits the the branch you based it on, or even doing git commit --amend to change the original commit, and then pushing that branch up to GitHub again.

@apjanke I have found after working in repos with multiple contributors that all pull requests must be made on a per file basis to avoid any issues.
Let me know if you'd like me to revise it.

@apjanke I have update the PR #4617 as per your comment. Kindly merge it.

Thanks.
I'm not one of the repo owners; so I can't merge it myself. That will need to wait for robbyrussell or mcornella to get to it.


fix(List): Cannot read property 'clientHeight' of null

Fixes the following:
    TypeError: Cannot read property 'clientHeight' of null

      at node_modules/rc-virtual-list/lib/List.js:242:56

Usage:
<Select
      labelInValue
      showSearch
      style={{ width: 250 }}
      options={options}
      placeholder="Select Country"
      filterOption={(input, option): boolean =>
        (option?.label as string).toLowerCase().includes(input.toLowerCase())
      }
      onChange={handleChange}
      value={countryValue}
/>

Ant Design version: 4.3.1 (latest)
cc @zombieJ

Thanks for PR. It's caused by List trigger async which not cancel by raf. Patch to clean up instead: https://github.com/react-component/virtual-list/commit/28c069da8766f681eb1779f7626ee540b9f24197#diff-1488bd82d3d6abd9637b03bf2e5dff9dR381

Thanks for PR. It's caused by List trigger async which not cancel by raf. Patch to clean up instead: 28c069d#diff-1488bd82d3d6abd9637b03bf2e5dff9dR381

Awesome, thanks!


🐛 [core] Add isGooglePlayServicesAvailable() wrapper

The com.google.android.gms.common.GoogleApiAvailability package provides a isGooglePlayServicesAvailable() method. It would be nice to expose it in flutterfire.

It is not part of Firebase so it will not be added.


Cantaloupe JP2 support

Cantaloupe was never configured to support JP2, only TIFFs.
There are a couple of settings that need to be fixed in the cantaloupe.properties template, and we should update to the latest 4.1.x version of Cantaloupe but I've got this working so it can be fixed.
PR TK

Grok is being built in the playbook now, but it is installed by setting a shortcut to the OpenJP2 installer. Seems unnecessary, I'll see if Grok can run directly.


Retrieve matplotlib figure instance from plot function

Hi all,
I was wondering whether it could be possible to retrieve the matplotlib figure instance created within a nhood enrichment test, or - even better - pass a matplotlib axes object to sq.gr.nhood_enrichment() to allow some more customized plotting?
Thanks in advance!

Hi @jo-mueller
thanks for your interest in Squidpy
great suggestion, we might able to include it for next release. ofc if you'd like to submit PR also very much appreciated!

Hi @giovp ,
thanks for the offer! I think the easiest way would be to add an ax keyword to the nhood_enrichment() function, which would then be passed on to _heatmap(). Passing an ax keyword would offer the advantage of not meddling with the function returns.
I'll try myself with an implementation.

I think it makes a lot of sense indeed, if you can draft something I'd be happy to help you out, thanks a lot!
You can see contributing infos here: https://github.com/theislab/squidpy/blob/master/CONTRIBUTING.rst
if you have issues with pre-commit and can't solve some errors, no worry just git commit --no-verify and it will skip pre-commits checks that we can fix later.

Alright, I think I have it implemented and tests pass locally (only for the heatmap though - I guess it could make sense for the other plot functions, too?)
One issue I'm seeing though:
Building the documentation fails on my machine; tox -e docs gives me an error which is probably the same as in https://github.com/theislab/scanpy/issues/2108:
Traceback (most recent call last):
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\sphinx\config.py", line 323, in eval_config_file
    exec(code, namespace)
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\docs\source\conf.py", line 25, in <module>
    import squidpy  # noqa: E402
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\squidpy\__init__.py", line 1, in <module>
    from squidpy import gr, im, pl, datasets
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\squidpy\gr\__init__.py", line 2, in <module>
    from squidpy.gr._build import spatial_neighbors
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\squidpy\gr\_build.py", line 8, in <module>
    from scanpy import logging as logg
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\scanpy\__init__.py", line 14, in <module>
    from . import tools as tl
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\scanpy\tools\__init__.py", line 17, in <module>
    from ._sim import sim
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\scanpy\tools\_sim.py", line 23, in <module>
    from .. import _utils, readwrite, logging as logg
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\scanpy\readwrite.py", line 10, in <module>
    import tables
  File "C:\Users\johamuel\Documents\BiAPoL\Projects\squidpy\.tox\docs\lib\site-packages\tables\__init__.py", line 45, in <module>
    from .utilsextension import get_hdf5_version as _get_hdf5_version
ImportError: DLL load failed while importing utilsextension: The specified module could not be found.

When installing squidpy, installing h5py and pytables separately with conda works but I'm not sure how to fix it when installing it through tox. I'm on Python 3.8.12


Add tfexec client

Issue: Closes rancher/qa-tasks#984
Problem
There was no method via the golang test framework to interact with terraform.
Solution
Added a tfexec client which wraps the tfexec library to provide easy-to-use wrapper functions around common operations.
Testing

Did various tests on local machine, not tested in Jenkins.
Adds additional dependencies to go.mod:

github.com/hashicorp/hc-install
github.com/hashicorp/terraform-exec
github.com/hashicorp/terraform-json

Example tfexec config:
tfexecInput:
  workspaceName: "workspaceToApplyModuleIn"
  workingDir: /path/to/a/terraform/module/
  execPath: /path/to/executable/terraform <--- if omitted, there is an InstallTfVersion() function to enable installing a specified terraform version
  varFilePath: /path/to/terraform/module.tfvars
  planFilePath: "/pre/existing/plan/file/to/use"
  planOpts:
    outDir: /path/to/output/planfile/to

Engineering Testing
Manual Testing

Automated Testing
QA Testing Considerations

Regressions Considerations

This will be moved to the new rancher/shepherd repo


Runtime subcomponents compilation not working

Version
3.2.31
Reproduction link
stackblitz.com
Steps to reproduce

Create a vue v3 application with runtime compilation, some root component and a subcomponent with some specific $data
Provide html for that application that uses a subcomponent and text binding inside that subcomponent's html to some of that subcomponent's data
Compile the application and see it on a server (using runtime compiler)

What is expected?
subcomponent's html should be parsed and bindings should show data from that subcomponent's data
What is actually happening?
instead of subcomponent, an empty comment block is shown

After debugging Vue, I see that someone completely forgot about subcomponents and their contents in the runtime compiler, they are not handled properly at all, their render function is not provided by the compiler, but even if it would be provided, the context of the subcomponent compilation is also wrong (compiler doesn't recognize that subcomponents have their own context). This is a big issue

Nothing was forgotten. this is simply not a feature. you can't define a component's template like this - that way is semantically already used to define "slot content".
In Vue 2, that was possible by adding a special "inline-template" prop or something, but that feature was officially dropped in Vue 3.

So to be clear: if you want to define templates for subcomponents, you can't do that from within a parent component's template - you either provide a string template in the sub-component, or define the template in a <template> block somewhere outside of the app.
See: https://vuejs.org/api/options-rendering.html#template

Thank you for the clarification, it confused me because this is how it works for the root component, too bad that there is no section in the documentation concerning peculiarities of runtime compilation. Those subcomponents I mentioned didn't use any slots and didn't have templates so I assumed that the runtime compiler would treat their inner html as the template.

Those subcomponents I mentioned didn't use any slots and didn't have templates so I assumed that the runtime compiler would treat their inner html as the template.

Slot usage happens in the template of a component so "the components don't use any slots" is not something the compiler would know if the components don't have any templates of their own.


pre-template click issue before logging calls

Although I have set data-log-level to "error" but still seeing logger calls in the console. And before the log calls are complete, clicking on the checkout button gives error of pre-template rendering click. Once the pre-template click error has appeared, any subsequent clicks doesn't initiate paypal payment. To avoid the pre-template erorr checkout button needs 3-4 seconds to be ready and it is creating a really bad user experience.
I am loading the checkout js script dynamically when a user selects payapal payment method on the checkout page.

We are also experiencing this.
To reproduce, just:

load a page with the paypal checkout button (e.g. www.kiva.org/lend, checkout, then continue to payment, and on the payment page)
click the paypal checkout button immediately (hover over the spot where it will load and click as soon as you see it)
checkout with paypal button disappears
After this, user is unable to checkout

This shows up in the log as the console message: logger.js:53 ppxo_button_pre_template_click

Is this issue related to https://github.com/paypal/paypal-checkout/issues/416 ? Looks like the button is hidden regardless of whether it's clicked on.

Other issue was resolved so closing this, unless you have any more info. Thanks!

Hi,  this was not related to #416. It is clearly different issue.  Do you need any clarification?

Yes, please -- can you reproduce this anywhere else, like for example on https://developer.paypal.com/demo/checkout/#/pattern/client ?
How do you figure the issue is unrelated, if the button on kiva.org was disappearing as a result of a code change on their side?

@bluepnume I think you may be confounding the two issues. I was tagging on to Rashid's report as we saw a similar symptom. I'll delete my comment to avoid confusion.

@bluepnume Please see the screenshot. I got two events where the popup didn't open. So there is at least one event where the popup doesn't appear and user has to click again to open it.

Can you please let me know if you still see this issue? This should be resolved by now.

@bluepnume Yes I still see the issue

Specifically this comment:

any subsequent clicks doesn't initiate paypal payment

Can you confirm that the button is totally unusable if you click before it's ready? I've been unable to repro that.

any subsequent clicks doesn't initiate paypal payment
this is fixed. after it loads the button works fine.

Only thing remaining is the pre-template-click. This you can still see on the demo page

OK, thanks. This is a known issue, we're working on making the pre-rendered button clickable.

@bluepnume I'm also having this issue now. Is there any temporary solution?

This appears to still be an issue.
It can be reproduced by going to this page and clicking on the PayPal button within a few seconds after it loads:
https://developer.paypal.com/demo/checkout/#/pattern/client

Sometimes it may take up to 10 seconds before the button reacts. If a user clicks on the button before it is fully loaded, the button appears dead and there is no feedback to the user.
Possible solutions:

Show a loading/spinner icon on the button before it is ready to be clicked.
Have the "pre_template_click" trigger a loading/spinner icon on the button, then show the popup when it is ready. This would let the user know that the button has been clicked, and that something is happening.

Solution number 2 would probably be the best user experience.

Still having this issue, and I cant find a way to catch the ppxo_button_pre_template_click event if there is one

@axul -- which environment -- sandbox or production?

@bluepnume both cases

@bluepnume

I also have the same issue. Is there any event that is dispatched when the button is ready?


I2C: Work around STM32 code generation bug

This slightly generalizes the earlier commit 2852e56 from https://github.com/ZigEmbeddedGroup/microzig/pull/125.

I've set this to 'draft' again, until https://github.com/ZigEmbeddedGroup/microzig/pull/153 has been merged to main.

@marnix is this still an issue on 0.13.0?

Closing because it's nearly a year old, the compiler is a newer version, and the stm32 code generation has drastically changed.


I like to collaborate

Hi, I am mohammad, an bioinformatician and AI expert in helsinki University. I like to help in this project.

Hey Mo,
great! Let me hear what you would like to do... I think at the moment it would be interesting to try it with data sets that have not been tried yed and use representations asidies from the ones tried that haven't been tried.
What do you think?
Cheers,
Daniel

Hey,
That should be great. but first of all can you send your email or another way communication to coordinate our works.
Second, What dataset do you suggest, Maybe a large scale dataset suit us more and by another representations you means something beside gzip?


Typescript > JavaScript

Cheers! I performed a gentle-enough refactor to turn this awesome project into a Typescript project, in case that's interesting for anyone.
Awesome work, @mlynch !

Ah, one slip, I created an "Accounts.tsx" page but forgot to exclude it from my commit.

I would really like this too, this is awesome!
Going to leave some comments because I'm sure some others may want these changes too

Thanks for the PR. Looks like there might be some outstanding issues so I will wait for those to resolve before considering this. I wasn't sure when I started this project whether JS or TS would be better. Does anyone know what the breakdown of people using JS vs TS is in the Next.js ecosystem?

Ah, sorry for the delay here, I've been pulled into other work and this PR slipped my mind.
TS is hailed as the far better option in the JS vs TS war as it provides a type-safe dev environment and significantly reduces the number of potential runtime errors in a production environment. I'm not sure about the statistic, JS vs TS users, but I am sure at least that TS is a far more dependable and professionally-viable language.
That said, of course it is excellent to have a JS version available for folks unfamiliar with TS, I remember the frustrations of my pre-TS days, and it is a great relief to have an accessible boilerplate.
“Go not to the Elves for counsel, for they will say both no and yes.”

Update, I've made some updates but noticed more errors, so I'm going to circle back soon :-)
I've included another Route path in the next commit, as it seems to silence the again-persistent isProxied error I'd mentioned above. Let's see if it does the trick!
I also ignored a TS error in /components/pages/Tabs.tsx which offered an error for a seeming non-issue.
And finally, I removed as many unused import statements as I could come across.

honestly please don't ruin this great project with TS Extra non-sense there will be a lot of boilerplate code everywhere!

Hey @amunrarara, we ended up merging support for this in #53 due to the conflicts + age of this branch. Thanks for contributing!


add metrics for customer/company active watch

This patch introduces metrics for active company and customer watches.
Resolves #236

Codecov Report

Merging #333 (2ca76dc) into master (381b8a8) will increase coverage by 0.09%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #333      +/-   ##
==========================================
+ Coverage   68.80%   68.89%   +0.09%
==========================================
  Files          33       33
  Lines        2170     2180      +10
==========================================
+ Hits         1493     1502       +9
- Misses        472      473       +1
  Partials      205      205

Thanks for the PR! This works to count the watches created when Watchman is running, but doesn't include the counts in the database. Shouldn't we include those as well?

can we do that as a separate pr because we don't have metrics for the DB layer?

@tharun208 yea, but can we rename these? Something like created_customer_watches_count?

@tharun208 yea, but can we rename these? Something like created_customer_watches_count?

but, this is for the active customer/company watch created_customer_watches_count seems to be a bit misleading metric name. WDYT?

I'd call "active" to be the count of watches in the database that Watchman cares about. These counts are only "watches created by this instance since it started."
If you're running two Watchman instances (w1, w2) then these metrics would be different between w1 and w2 depending on which ones behind the load balancer received the POST/DELETE requests.


Guessing to add a new feature and change behaivour of existing and return value

First of all, thank you for such amazing work!

Would be nice if you consider adding a watermark option in settings ( IMGPUSH in the right lower corner for example )
Change return value after uploading file from filename to full URL pointing to the file.
example :
220: return jsonify(filename=str(request.url)+str(output_filename)) but this doesn't work with gunicorn + nginx. it's always returning http://<IP_ADDRESS>:5000/filename.jpg

I've solved second issue by forwarding headers in nginx config:
proxy_set_header Host $host:$server_port;

what about watermark feature?


chore: remove guards and EOL requirements.txt deps

Remove guards for >= 3.x that were all or equal to the minimum version
removed six and urllib3 that are no longer imported

This file is now auto-genned; those guards are added by pipenv. If you can figure out how to make it not generate those, I'd be fine with that, but just editting this file will get overwritten next time I update requirements.
(But I also don't think it's important, so don't spend time on it if you don't immediately know how to do this with pipenv.)
I'll get six and urllib3 removed properly, thanks for discovering that.


Add crashdumping through VMCoreInfo

These pathes add driver which exposes Windows crashdump header to QEMU at startup. Then QEMU can use the header to produce full crashdump even if guest OS can't do it.
At the moment, this driver can only be used with 64-bit guest, also memory hotplug isn't supported.

Hello,

Please rename the driver to fwcfg or something similar

Should I replace dummy fwcfg driver with this driver or keep it separate and just rename?
Regards,
Viktor Prutyanov

Hello,
If you are going to add 32bit support soon, than replace now. But if it will take some time, keep it separate.
Thanks,
Yan.

On Apr 11, 2018, at 12:48, Viktor Prutyanov<EMAIL_ADDRESS>wrote:
Hello,
Please rename the driver to fwcfg or something similar
Should I replace dummy fwcfg driver with this driver or keep it separate and just rename?
Regards,
Viktor Prutyanov
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Hello,
All 6 issues have been fixed.
Driver has been renamed to fwcfg64.
Regards,
Viktor Prutyanov


chore: move compiler defaults from nim.cfg to config.nims

At this moment, in some cases setting the default gc/mm setting for the compiler results in a compilation error.
The workaround for this is moving the setting to the nim.cfg file. Having an additional configuration file is not great, so as soon as the fix for the bug lands in stable the setting should be moved back to the config.nims and nim.cfg should be removed.
This is a PR that sould have fixed the issue: https://github.com/nim-lang/Nim/pull/19455. Not sure why we're still hitting it.

Not sure why we're still hitting it.

The compilation error was obtained with Nim 1.6.12 from the post of @vil02 that you linked.
Is it still hit on Nim-devel? If so, please open an issue in the Nim repository and link it here.


How to train a (Linguistic + acoustic + vision) model on the emotion labels ?

I use the command : 'python main.py --model Model_LAV --name mymodel --task emotion --multi_head 4 --ff_size 1024 --hidden_size  512 --layer 4 --batch_size 32 --lr_base 0.0001 --dropout_r 0.1'
Unfortunately, the error is ‘RuntimeError: mat1 dim 1 must match mat2 dim 0’
I would appreciate it if you could help me solve the problem.

Hello.
Because we could manage to improve baseline of LA with LAV, we dont provide the audio features
Now you can still run the code, in the dataset you need to change this line with your video features:
https://github.com/jbdel/MOSEI_UMONS/blob/master/mosei_dataset.py#L28
(here we put audio mels as dummy input)
then you need to sets this args right:
https://github.com/jbdel/MOSEI_UMONS/blob/master/main.py#L28
and it should work.
Best

Hi,
Where can I get the "_r21d.p" files for the video features?
Many thanks.

Hello,
We dont provide the video features as they dont improve our scores.
You can make you own, as long as they are 2D matrices for each video.
JB

Hey, one more question, how did you preprocessed the CMU-MOSEI data? I see that Acoustics and Visuals have 60 x 80 dimension in your files. But the dataset has {variable seq length} x 74 for Acoustics and {variable seq length} x 35 for Visuals.
I appreciate the help!

But the dataset has {variable seq length} x 74 for Acoustics and {variable seq length} x 35 for Visuals.
What are you refering to ?
I myself extracted mels features (audio) from the raw videos
For feature extraction please read the paper: https://www.aclweb.org/anthology/2020.challengehml-1.1/


C# Implementation

I implemented Your algorithm in C# for the Windows users. You can find the implementation here: https://github.com/ColorfulSoft/Demos/tree/master/Colorization/2016. Colorful Image Colorization

I implemented Your algorithm in C# for the Windows users. You can find the implementation here: https://github.com/ColorfulSoft/Demos/tree/master/Colorization/2016. Colorful Image Colorization

Hi can you re up the link? I'm a windows user.

Hi Travsmakoy!
You can find the code here: https://github.com/ColorfulSoft/StyleTransfer-Colorization-SuperResolution/tree/Russian/Colorization/2016. Colorful Image Colorization


tour: rnd

Context: https://tour.golang.org/basics/1
Change the title above to describe your issue and add your feedback here, including code if necessary
output rnd is always 1

rand.Intn(10) always returns 1

from the page you link to:

Note: the environment in which these programs are executed is deterministic, so each time you run the example program rand.Intn will return the same number. (To see a different number, seed the number generator; see rand.Seed.)


Simplify command line client flags

With the transition onto Trillian, several of the previous flags are now obsolete.
This PR cleans up and clarifies command line flags.
This PR also fixes a few things in the client and cleans up docker-compose.
References:
#712
Closes #659

Codecov Report

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

@@           Coverage Diff           @@
##           master     #738   +/-   ##
=======================================
  Coverage   48.66%   48.66%
=======================================
  Files          30       30
  Lines        2503     2503
=======================================
  Hits         1218     1218
  Misses       1088     1088
  Partials      197      197

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 7e01002...6da8ebe. Read the comment docs.

Updated PTAL


Option to switch sorting

I still love this plugin.
How difficult is it to add some option in sorting.
Now the list is sorted by  Due @Myself -> Notebook -> Task
For me (and maybe more users) it would be nice to change it to:
Notebook - Due -> @whom -> task
And no is also on answer ;-)
Keep up the great work.

This wouldn't be too challenging, a new summary type could be created that has the alternative sorting. I won't be adding this, but I would accept a pull request for it.

I will see if i can find someone to help me with this.
Myself i do not have any idea how to do this.


Custom CSS/JS and cache busting

Describe the bug
If the links to the custom CSS/JS are kept after making some changes, the brower will not reload them, will use the cached version.
To Reproduce
Steps to reproduce the behavior:

Make some custom CSS/JS
Add them in larecipe's config
Visit a documentation page.
Make changes to the file made in (1.)
Visit page again
Realize that changes were not affected. (don't be with inspector open and "disable cache" enabled)

Expected behavior
This is expected, as it's an asset, and browser will cache it.
"Similar" to Laravel Mix, we could add a random string as query param.
This means that every request will bust the browser cache, but it's better than having viewers always viewing older versions.
This could be achieved by changing on the default.blade.php
I didn't change the core, i just published views to resources/views/vendor/larecipe so they would be load from there, and made the changes...
I will gladly do PR, just wanted to check in with you If you would like it or not?
<link rel="stylesheet" type="text/css" href="{{ asset($css) . '?id=' .\Illuminate\Support\Str::random(8) }}">

(...)

<script type="text/javascript" src="{{ asset($js) . '?id=' .\Illuminate\Support\Str::random(8) }}"></script>

Screenshots
N/A
Desktop (please complete the following information):
All
Smartphone (please complete the following information):
All
Additional context
N/A

Awesome catch 💯 but we might lose some cache benefit with adding this feature especially if the custom CSS/JS files are huge but TBH I don't think this is the majority since most of the time the custom files are just small modification or enhancement made to suit the need.
Just proceed with it 💃👏
Thank you a lot @ijpatricio 😍

Thanks for feedback! Yeah, there's a caveat in always putting a random string, but also believe that in 99.9% of use case, it's just fine... ;)
No worries, love to contribute, such a great project ❤❤❤
Will proceed to PR then!

What about using filemtime() function to generate ?id? I know this is not perfect, but probably better than random string?


How Can I decrypt my image encrypted by offline_fs_kbs?

Hi! I have used offline_fs_kbs to encrypt an image refer to the document . But I don't know how to decrypt that.
I encrypted the image as follows:
cargo run --release --features offline_fs_kbs -- --keyprovider_sock <IP_ADDRESS>:50000

OCICRYPT_KEYPROVIDER_CONFIG=ocicrypt.conf skopeo copy --encryption-key provider:attestation-agent:/root/encryptimage/keys.json:key_id1 oci:ubssh oci:ubssh_enc

Then I try to decrypt the images as follows:
make KBC=offline_fs_kbc && make install
RUST_LOG=attestation_agent attestation-agent --keyprovider_sock <IP_ADDRESS>:50000
OCICRYPT_KEYPROVIDER_CONFIG=ocicrypt.conf skopeo copy --insecure-policy --decryption-key provider:attestation-agent:offline_fs_kbc::/root/encryptimage/keys.json:key_id1 oci:ubssh_enc oci:ubssh_dec

But it didn't work, LOG information is as follows：
[2022-11-17T09:18:55Z DEBUG attestation_agent] KeyProvider gRPC service listening on: <IP_ADDRESS>:50000
[2022-11-17T09:18:55Z DEBUG attestation_agent] GetResource gRPC service listening on: <IP_ADDRESS>:50001
[2022-11-17T09:19:03Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-11-17T09:19:04Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Failed to load keys: No such file or directory (os error 2)
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-11-17T09:19:04Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Failed to load keys: No such file or directory (os error 2)
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-11-17T09:19:04Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Failed to load keys: No such file or directory (os error 2)
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] The UnWrapKey API is called...
[2022-11-17T09:19:04Z DEBUG attestation_agent::grpc::keyprovider] Call AA-KBC to decrypt...
[2022-11-17T09:19:04Z ERROR attestation_agent::grpc::keyprovider] Call AA-KBC to provide key failed: Failed to load keys: No such file or directory (os error 2)

The ocicrypt.conf content is as follows:
{
    "key-providers": {
        "attestation-agent": {
            "grpc": "<IP_ADDRESS>:50000"
        }
    }
}

AA version: 0.1.0
skopeo version: 1.11.0-dev
How can I decrypt this image correctly?

I think you need to put your keys json file at /etc/aa-offline_fs_kbc-keys.json

Thank you @fitzthum ! It worked! Thanks again! I will close this issue.


Array of JSX is not allowed

The type of component function does not allow to create react component with Array JSX return.
This should get compile, because react library allowes this syntax.

mkCounter :: Component Int
mkCounter = do
  component "Counter" \initialValue -> React.do
    counter /\ setCounter <- useState initialValue

    pure
      $ [ R.button
          { onClick: handler_ do
              setCounter (_ + 1)
          , children:
              [ R.text $ "Increment: " <> show counter ]
          }
          ]

Yeah Component is just a type alias
type Component props = Effect (props -> JSX)

so if you have an Array JSX that you want to return, the recommendation is to use fragment :: Array JSX -> JSX https://pursuit.purescript.org/packages/purescript-react-basic/17.0.0/docs/React.Basic#v:fragment


Improved newline handling in interactive code edits and edit actions

The following is from my personal fork to improve editing (interactively or via actions) of visual selections; maybe it's interesting for main? In any case, thanks for the great plugin :).
Currently, if a visual selection is edited (either interactively or via an edit action), an unwanted newline was inserted. This pull request avoids such newlines.
The changes (a) ensure that the visual selection is yanked/pasted without creating additional newlines and (b) enforce that ChatGPT's answer ends in exactly as many newlines as the visual selection. The latter is necessary since ChatGPT seems to always add a newline at the end of its answer, even if the input did not contain one. Note that this might be considered a downside of the pull request, since requests like add 3 newlines at the end are now ignored.
Notable changes:

Moved the handling of a non-existing current/previous visual selection into Utils.get_visual_lines.
Got rid of Utils.get_lines, Utils.get_visual_start_end, since Utils.get_visual_lines seems now simple enough to handle this directly.
Utils.end_col is no longer needed and Utils.paste are also gone (replaced by a direct call to vim.api.nvim_buf_set_text).
Utils.split_string_by_line: Previously both some text and some text\n resulted in {"some text", ""} (i.e., two lines). Now only the latter does, while the former yields {"some text"}.

Good job! Could you please fix formatting to load the tests

Ah sorry, didn't see the formatting checks. Done.


Flesh out [derive]

Handle deriving classes for inductive types with universe levels and parameters. Add a derive handler for has_reflect.

I'm having problems with indexed families.
The following works:
universes u
inductive foo (α β : Type u) : nat → Type u
| bar : ℕ → foo 0
| baz (n : nat) : foo n → α → foo (nat.succ n)

instance (α β : Type u) (n : nat) [decidable_eq α] [decidable_eq β] : decidable_eq (foo α β n) :=
by tactic.mk_dec_eq_instance

but the following one doesn't
universes u

set_option trace.app_builder true

@[derive decidable_eq]
inductive foo (α β : Type u) : nat → Type u
| bar : ℕ → foo 0
| baz (n : nat) : foo n → α → foo (nat.succ n)

@[derive inhabited] assumes parameters are inhabited. This produces sub-optimal instances.
Example:
universes u

@[derive inhabited]
inductive foo (α β : Type u)
| mk  : foo
| mk1 : α → foo
| mk2 : β → foo

#print foo.inhabited -- assumes that α and β are inhabited

instance better_instance (α β : Type u) : inhabited (foo α β) :=
⟨foo.mk α β⟩

I think this will generate confusion. See the following example
@[derive inhabited]
inductive List (α : Type u)
| nil : List
| cons : α → List → List

#print List.inhabited -- assumes that α is inhabited

@leodemoura I see, so inhabited does not quite follow the same pattern as the other two typeclasses. I think I'll remove it for now since it's not a very important typeclass (I think).

@leodemoura While addressing your first bug, I noticed that mk_has_sizeof_instance itself breaks on indexed families (and mk_has_reflect_instance as well since it's based on the same template):
inductive foo : nat → Type
| baz (n : nat) : foo n → foo (nat.succ n)

lemma foo.size (α β : Type u) (n a : ℕ) : has_sizeof (foo a) :=
by (tactic.mk_has_sizeof_instance)

I have now removed the inhabited handler and explicitly disallowed indexed families for has_reflect. I pushed two unrelated but trivial fixes as well.


Update the secure naming based on the SAN in the actual GCP certificates

Update the secure naming based on the SAN in the actual GCP certificates

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: lei-tang
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @quanjielin

/cc @wattli

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: lei-tang, quanjielin
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: zackbutcher
If they are not already assigned, you can assign the PR to them by writing /assign @zackbutcher in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: lei-tang, quanjielin, wattli
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pilot/OWNERS [wattli]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/hold
Please check in this code in branch collab-gcp-identity instead

This PR is moved to https://github.com/istio/istio/pull/7547 to commit the code into the collab-gcp-identity branch.


move dockerutils under docker

also remove fixtures pkg as it would clutter godoc (there's not need
to have a .go files with fixtures)
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@mtrmac PTAL

I was a bit fond of keeping the fixtures info in the fixtures directory :sob: but I can see the rationale.
Perhaps move fixtures/info.go to fixtures_info_test.go, and keep it separate from the other _test.go files, as they may be added over time? (See how the various _test.go files share fixtures in skopeo/signature.)
(I was also considering just adding this all to skopeo/docker or perhaps naming this skopeo/docker/utils, but neither seems better than what you do here.)

(I was also considering just adding this all to skopeo/docker or perhaps naming this skopeo/docker/utils, but neither seems better than what you do here.)

This is fine for me! much cleaner to have docker/utils

Perhaps move fixtures/info.go to fixtures_info_test.go, and keep it separate from the other _test.go files, as they may be added over time? (See how the various _test.go files share fixtures in skopeo/signature.)

moved to fixtures_info_test.go

(I was also considering just adding this all to skopeo/docker or perhaps naming this skopeo/docker/utils, but neither seems better than what you do here.)

This is fine for me! much cleaner to have docker/utils

The trouble is that then all users would either have to use the vague utils.… naming, or they would end up importing under a dockerutils or similar rename anyway. So I do prefer docker/dockerutils, actually.
Anyway, this is trivial and not worth reworking the code over and over. Use whichever you like.

The trouble is that then all users would either have to use the vague utils

right, anyhow, this is how golang works also and I like docker/utils (which of course will be renamed to dockerutils lol)


Why can i download latest version e.g. v2.8, but download v2.4

The downloaded files from homepage is v2.4, not latest version e.g. v2.8. The same problem in npm and github

We will update later, please pay attention.


distributed DQN using colossalai

Add  a colossalai naive DQN agent example that can use data-parallel for training

Hi  @number1roy Could you cancel the review request for feifeibear? This prevents other reviewers' action.


any plan to support use numpy 1? cuz lots of dependency requires numpy<2 in my old project

So I would like to try ell, but I cant find a solution to deal with it.
Thanks

I think we could make it agnostic to numpy version want to look into this?

I've pull a request to add support for numpy 1 #267 , and I think it will works

Thank you!


About BN in Advprop

I would like to ask whether PGD attacker (calculate gradients) will update the parameters (mean, var) of aux_bn in Resnet. Thank you.

Sorry about forgetting to reply.
No, in this implementation, PGD attacker won't update the parameters of aux_bn during attacking.
In fact, this is a problem about which mode of batch normalization is better to use during attacking. As for our experience on ImageNet, eval mode is better, which freezes the parameters (mean, var) of batch normalization. Similar results are reported on Cifar by Tianyu et al. in "BAG OF TRICKS FOR ADVERSARIAL TRAINING".


Is there another way to see the mediapipe's log file?

For some reason i cant use  viz.mediapipe.dev to analyze the log
Is there another way to see the mediapipe's log file?

Hi @superbento, What platform and solution are you referring. Thanks!

HI, my program is running on Ubuntu 18.04
And I just found the solution, i'll close this issue


Uncaught Error: Atom can only handle files < 2MB for now.

[Enter steps to reproduce below:]

...
...

Atom Version: 0.165.0
System: Mac OS X 10.10.2
Thrown From: Atom Core
Stack Trace
Uncaught Error: Atom can only handle files < 2MB for now.
At /Applications/Atom.app/Contents/Resources/app/src/project.js:462

Error: Atom can only handle files < 2MB for now.
  at Project.module.exports.Project.buildBuffer (/Applications/Atom.app/Contents/Resources/app/src/project.js:462:15)
  at Project.module.exports.Project.bufferForPath (/Applications/Atom.app/Contents/Resources/app/src/project.js:439:63)
  at Project.module.exports.Project.open (/Applications/Atom.app/Contents/Resources/app/src/project.js:393:19)
  at Workspace.module.exports.Workspace.openUriInPane (/Applications/Atom.app/Contents/Resources/app/src/workspace.js:482:29)
  at Workspace.module.exports.Workspace.open (/Applications/Atom.app/Contents/Resources/app/src/workspace.js:409:19)
  at Ipc.<anonymous> (/Applications/Atom.app/Contents/Resources/app/src/window-event-handler.js:45:32)
  at Ipc.emit (events.js:110:17)
  at process.<anonymous> (/Applications/Atom.app/Contents/Resources/atom/renderer/api/lib/ipc.js:22:29)
  at process.emit (events.js:118:17)

Commands

Config
{
  "editor": {
    "fontSize": 12
  }
}

Installed Packages
# User
autocomplete-plus, v0.21.0
color-picker, v1.2.5
emmet, v2.2.1
javascript-snippets, v1.0.0

# Dev
No dev packages

See #4421 and #307.


Delete verified.json

This file is no longer used and we should update verified/flagged spaces on Sidekick from now.

This can me merged here but do not update this change on Snapshot UI yet, I will do myself.


Step on parked goroutine only goes to current PC

Stepping on a parked goroutine should continue to the next line, not just the current PC (making the goroutine active).

Bug only valid after https://github.com/derekparker/delve/pull/563 is merged.

This issue was fixed when we reimplemented step to work like next and continue.


Update wire-compiler to 1.8.0

Signed-off-by: Eddie Ringle<EMAIL_ADDRESS>

Thanks. There's a lot to do here before we publish, but plans are in motion.

@JakeWharton I was just about to submit a PR to add the maven plugin so that, at the very least, JitPack can build this repo. Would that be alright?

The maven plugin is in the main repo so no need.

Oh you mean the maven gradle plugin. That's probably fine...


Bug while parsing a table

I have this table: https://pastebin.com/Wjf8zJ7X
I get the rows in this way:
HtmlNodeCollection rows = doc.DocumentNode.SelectNodes(".//table//tbody//tr");
inside the iteration:
foreach (HtmlNode tr in rows) {      var c = tr.SelectSingleNode(".//td[@class='number statistic appearances available']"); }
I get this exception:

why happen this?

Hello @serdam55 ,
Thank you for reporting, we will try to look at it.
Best Regards,
Jonathan
Best   Regards,       Jonathan       ---       Performance   Libraries context.BulkInsert(list,   options => options.BatchSize = 1000);     Entity   Framework Extensions • Bulk   Operations •    Dapper Plus •  LinqToSql Plus       Runtime   Evaluation Eval.Execute("x   + y", new {x = 1, y = 2}); // return 3 C#   Eval Function • SQL Eval   Function

@JonathanMagnan forgot to say that in some iteration on the same line I get:
startIndex cannot be larger than length of string. Parameter name: startIndex

@JonathanMagnan  are passed 4 days and still no news, there are problems to fix this?

@JonathanMagnan there are problems to fix this?

Hello @serdam55 ,
My developer tried it but could not reproduce the issue.
Here is the code he used:
var doc = new HtmlAgilityPack.HtmlDocument();
doc.LoadHtml(@"

Season	Team	Comp

2017/2018	Skënderbeu Korçë	CUP	50	2	0	2	0	2	0	0	0	0

Total	50	2	0	2	0	2	0	0	0	0

");

HtmlNodeCollection rows = doc.DocumentNode.SelectNodes(".//table//tbody//tr");

foreach (HtmlNode tr in rows)
{
	var c = tr.SelectSingleNode(".//td[@class='number statistic appearances available']");
	var innerHtml = c.InnerHtml;
	var innerText = c.InnerText;
}

Could you provide additional information so he could reproduce it?
Best Regards,
Jonathan

@JonathanMagnan  after many attempts, I can say that this problems come when I use the parallelization, but I need to do more test


Title renders as link, even with with no url attributes

Hi @jerdftw! I tried reproducing the behaviour and it seems, the Contact doesn't appear as link rather simple plain text.


Consider completely removing type equality/typehood judgments

And replacing them with member equality judgments, using the universe.
This would simplify automation quite a bit, I think. What do you think @favonia? Do we lose anything if we do this?

I thought you want level synthesis? Without it, I believe the only thing we cannot internalize is the following judgment:
A and B are equal types at some unknown, finite level.
The Nuprl would ask for or infer the level as an input to the tactic/rule. One example is dfun/eq/lam (DFun.Eq) where we have to make sure the domain is a type.

@favonia

I'm OK with not having level synthesis generally; it's a cool thing, but I don't think it's as important as it sounds.

Ahh, you're right about A = B type with an unknown/unrestricted level! That may be the sticking point that requires us to keep this form of judgment. It was always annoying in Nuprl and JonPRL to have to put in a level by hand there.

And this will be possible again after implementing the plan in #407.

@jonsterling Actually, it seems the equality judgments can already be removed. Would you mind changing the title?

@favonia Which equality judgments? the type equality ones or the member equality ones? (I assume the latter)

Yes, member equality ones.

@favonia I've opened a new ticket for that, and we can leave this ticket open for tracking the question of type equality judgments.

@jonsterling Well, there is already #407 for type equality judgments, right?

Closed as this was split into #407 and #457.


Crash on https://doc.deno.land/https://esm.sh/preact/~/createElement

[error] Error: Not every node of kind "function".

    at assertAll (file:///mnt/starship/Projects/github.com/denoland/docland/components/doc.tsx:37:11)
    at CodeBlock (file:///mnt/starship/Projects/github.com/denoland/docland/components/doc.tsx:129:5)
    at renderFunctionalComponent (https://deno.land/x/nano_jsx@v0.0.21/core.ts:164:18)
    at _render (https://deno.land/x/nano_jsx@v0.0.21/core.ts:138:70)
    at https://deno.land/x/nano_jsx@v0.0.21/core.ts:48:17
    at Array.forEach (<anonymous>)
    at appendChildren (https://deno.land/x/nano_jsx@v0.0.21/core.ts:43:12)
    at h (https://deno.land/x/nano_jsx@v0.0.21/core.ts:255:3)
    at DocPage (file:///mnt/starship/Projects/github.com/denoland/docland/components/doc.tsx:273:25)
    at renderFunctionalComponent (https://deno.land/x/nano_jsx@v0.0.21/core.ts:164:18)

Ok, I know what the issue is, just unexpected.  Thanks for the repo.


Installation issue on Kubuntu

Description
I don't see the TM:PE icon in game.
I am using Kubuntu 18.04, playing on Steam. I have no problem with the (one) other mod I tried.
Reproduction Steps

Subscribe to mod on steam workshop, restart CS

Mod List

TM:PE (I don't know. Looking at steam workshop, 10.20 I suppose)
Mod listing
Sunken Plaza

Output Log
Player.log
No TM:PE log
Screenshots (Optional)
In game mod manager : https://imgur.com/LBRv3Nm
Thank you for your help

Hi, did you enabled it? (Mods tab - search for TM:PE)

If there's no TMPE.log it's likely the mod isn't enabled:
Main menu > Content manager > Mods > TM:PE > Enable

That's it ! Sorry for the bothering... It saw it in the 'steam workshop' tab, didn't look for the mod tab.

Glad you got it working :)


fix: always check x-forwarded-for

fixes #30

checkHeaders didn't have to be removed only the ! had to be.  Latest commit 30378c8 fixes this.
Thanks!


Keeps Closing on me

I try to run it but it keeps closing on me
test.zip
That zip contains a video since I couldnt find a suitable place to upload it

Have you done the changes in the config.ini? On first run, the program closes itself to generate the config.ini and files so you can edit them before trying to run the program again. Maybe if you didn't change any settings, might be a possible reason it's closing itself, like for example no channels to watch.

Already tried removing the config and it still crashes. Even tried using one that was untouched still didnt work but I'm using an older version and it works just fine.

Which version works fine? Are you using Windows 64 Bit?

I'm using Windows 10 64 bit yes and I'm using version 1.7.1 I havent tried any of the newer ones other than the most recent one which is the one that kept closing I also tried a few releases before that and they did the same thing. Is it possible that me running it from my flashdrive could some how cause an issue?

Does it print an error before it crashes?
I don't think running it from your flash drive causes issues.


Allow style and class output to be determined early on in the chain

There's something I really dislike about assigning BSS as style and class (and .toString() TBH) at the trailing end of a multi-line, multi-block chain. Having that stuff visible ahead of time would be good.
What do you think about introducing simple b.class and b.style functions that simply consume an object (or operate as tagged template interfaces like b) and return the corresponding property of the same name?

I very rarely need either .class or .style due to how .valueOf() gives the className, and I don't really need .style because I'm using spread. example
Would you mind making an example of the issue?
Just to be sure, you mean having something like this right? I suppose they could be exported as named exports? (although the name class won't work).

Actually my idea was just b.style = x => x.style. But - I didn't realise only style was enumerated during spread, which is really neat! I need to step away from the code base for a bit to get some distance. Did I really need the class/Name property? Probably not…

Hey @barneycarroll . Do you want to elaborate on this, or is the option of using spread a good enough solution for you?

Yeah I can do this in userland. Unshakeable thought tough - I'm wondering if we could somehow overload BSS with an iterator API to expose class. So m(X, ...b()) yields a class, m(X, {...b()}) yields style? Hmm.

Wow.. that would be some pretty cool overloading 😀
I'm still not sure what m(X, ...b()) brings instead of m(X + b())?
Ahh, I think I might get it now... Is it for components where you want to pass attributes to the inside with the class? Sorry if I'm just repeating what you stated, I just want to be sure I get it right ;)
So m('div' + b()) is all good, but m(component + b()) of course won't work, and m(component, { class: b().class }) seems like it could be better?
Another thing that could be doable is having b.class and b.style return template functions on a raw instance which returned an object with either { class } or { style } so you could do as requested like:
m(component, b.class`
  background red
`)

and
m(component, {
  special: 'wat',
  ...b.style`
     background black
  `
})

Yes, that last thing was my original thought. The class hack is a bit silly, I realise. The point is, when composing many parts of a whole, with conditional classes etc, with interpolations, along with other conditional aspects to a vnode expression… That in these scenarios relying on a dangling property makes the whole expression that significant bit harder to read. Eg
m('whatever' + b`
  some stuff
`
  .$nest`
    some more
  `
    .class // essential to the comprehension of prior X lines
)

I see 😊
Sorry if I'm being pedantic, but in your last example, .class would not be needed?

FFS I need to sleep, my brain is muck!

Maybe it's better to export multiple b that user can select from?
Like patchinko that have P, O for different purpose.
So like below code:
import b, {bc} from 'bss'

// same as below:

Is that possible/resonable?

I think I love that idea @futurist, not sure that's what barney meant, but also very relevant!
What you mention wrt. react, is possible to do now, but I'm not happy with the solution mentioned in the readme about overriding valueOf either.

Wrt. React, something like this could also be done, which I think is pretty cool
<h1 {...b`c red`}>Hello React!

Ok, so taking these thoughts further brings @barneycarroll 's original thought into it again I think..
Bss exports 3 things b(same as default), c for class, s for style..
b works as we know it
c will only have className as enumerable property (allows for jsx example above), and valueOf will return the className without a period (which b does)
s will have every property specified enumerable to use with style / dom element assignment

it's very cool! Then it could possible do below thing in React with classnames:
<div className={{opened:true,  [c`c red`]: true}}>

The concern is to use with user passed classes like opened, only way I can think about is use classnames module like above, but it's totally fine I think.

Or the c can have the below forms? (c.class method)
<div {...c.class('opened nav').`c red`}>

<div {...c.class({opened: true, nav: false}).`c red`}>

If you have time @barneycarroll I'd also love to hear your thoughts on the changes above... Thanks

The spread operator will overwrite any existing className attr, in v2 I think this should be taking into account.

The spread operator will overwrite any existing className attr, in v2 I think this should be taking into account.

@futurist yeah, that's a very good point.. I was actually a victim to it myself just yesterday.
The reason why I'd rather want className than style is that style won't allow pseudo classes to be set, so using style for this kind of composition limits the options.
One way of solving it could be adding whitespace around the returned className.
Then you'd make an extensible component like this:
const Component = {
  view: ({ attrs, children }) =>
    m('div'
    + b... // General styling
    + attrs.className // Component consumer styling
    + b... // Styling not overridden by consumer styling
    ,
      children
    )
}

As discussed/discovered last year on gitter, it appears we can have the best of both world by having valueOf return ' . ' + this.class + ' ' which makes it work with hyperscript text concat and attrs class/className.
I'll be experimenting a bit more with this for v2.

@barneycarroll I'm planning to land the ' . ' + this.class + ' ' solution in v2, do you think there's anything more to add to the concerns in this issue?


Proposal: split out the auth components

Motivation
Users of @solid/react might only be interested in using the authentication-related components (useWebId, <LoggedIn>, etc), and use a library like rdflib to interact with RDF. It's a bit wasteful to also send them LDFlex and Communica.
Proposal
Extract the auth-related components into e.g. a @solid/react-auth package, and making @solid/react depend on that package.

wasteful

It's an ES6 module, so tree shaking should work?

Two thoughts there:

Unfortunately there are many things that can break tree-shaking, I believe. It doesn't tree-shake for me - shall I report a new bug for that?
While 1. is certainly the main problem, there's still a downside to having those two libraries and all their dependencies shipped to the developer's machine even when they're not sent to the client, such as the higher risk of malicious postinstall scripts, and increased disk usage.

But yes, tree-shaking would remove the main pain point. Unfortunately, I've hardly ever seen it working reliably.

1. It doesn't tree-shake for me - shall I report a new bug for that?

Yes please, let's fix tree shaking first. Can you include a reproducible example (or link to a minimal repo)?

Update: tree shaking can indeed not work the way things are at the moment. So no real need for an example yet.

Alright, let me know when you need an example - happy to provide one.

ca154add36c9f7fb06fcc4b68bb9c0a41722ee40 should provide you with the necessary; can you check?

@RubenVerborgh Could you provide some guidance on how to test that? I've currently run:
$ yarn add https://github.com/solid/react-components.git#ca154add36c9f7fb06fcc4b68bb9c0a41722ee40
$ cd node_modules/@solid/react
$ npm install
$ npm run build

But that final step fails with a bunch of errors to the gist of:
ERROR in ./src/components/withWebId.jsx 10:11
Module parse failed: Unexpected token (10:11)
You may need an appropriate loader to handle this file type, currently no loaders are configured to process this file. See https://webpack.js.org/concepts#loaders
|  */
| export default higherOrderComponent('WithWebId', Component =>
>   props => <Component {...props} webId={useWebId()} />);
|
 @ ./src/index.js 9:0-47 34:0-68:2

Not sure what yarn add does; does it include .babelrc and others? Can you perhaps git clone instead?

Confirmed that webpack acts up when installed in a node_modules subfolder (which is an edge case, will check what is wrong there). You can still test with npm run build:module.

I tested it in a new Create React App setup just now using npm link, but am now hitting the following error:

Invalid hook call. Hooks can only be called inside of the body of a function component. This could happen for one of the following reasons:

You might have mismatching versions of React and the renderer (such as React DOM)
You might be breaking the Rules of Hooks
You might have more than one copy of React in the same app
See https://fb.me/react-invalid-hook-call for tips about how to debug and fix this problem.

I only added
import { LoggedOut } from '@solid/react';

and
          <LoggedOut>Test</LoggedOut>

(Btw, this feeds back into our earlier discussion on why npm installing (or yarn adding) from GitHub isn't a replacement for installing from npm ;) )

You are likely hitting You might have more than one copy of React in the same app.

why npm installing (or yarn adding) from GitHub isn't a replacement for installing from npm

Definitely; I have found React to be very troublesome in these cases. It behaves weirdly with npm link.
What I usually do there is remove the node_modules folder from the linked repository, such that i is ignored. It's a bit of fiddling.
Let me know if you can work it out; if not, I'll release a beta.

@RubenVerborgh Thanks, removing node_modules after linking did fix it.
And yes, tree shaking works now, thanks!

Super, thanks for testing. I imagine the gains in size to be quite significant 🙂

Published as v1.8.0.


Update overview.md

Update Overview, address issue #12

Coverage remained the same at 22.162% when pulling 0738c974167b4307061911428fdd60f87edda1f9 on Overview-first-try into 06f5330aa4633c8cef412cacb0a3bcdabad81d3c on master.


(403) Frobidden is back. on GetBytes()

everything is on the title : getting 403 forbidden on all videos.
example : http://www.youtube.com/watch?v=tAGnKpE4NCI

same error for all videos.
Youtube changed something- again.

Trymy last commit (https://github.com/omansak/libvideo)

Seems to be fine now.
Thanks!

Trymy last commit (https://github.com/omansak/libvideo)

Hi, I can't compile libvideo. I get

Field "YouTubeVideo.DFunctionRegex" is never assigned to, and will always have its default value null (File YouTubeVideo.Decrypt.cs, Line 14)
The name "DFunctionRegex_Dynamic" does not exist in the current context.  I get

Field "YouTubeVideo.DFunctionRegex" is never assigned to, and will always have its default value null (File YouTubeVideo.Decrypt.cs, Line 14)
The name "DFunctionRegex_Dynamic" does not exist in the current context. (File YouTubeVideo.cs Line 24 and 26)
I'm not good with Regex and therefore can't fix it myself ^^

ok i fixed, try again

Trymy last commit (https://github.com/omansak/libvideo)

Hi, I can't compile libvideo. I get

Field "YouTubeVideo.DFunctionRegex" is never assigned to, and will always have its default value null (File YouTubeVideo.Decrypt.cs, Line 14)
The name "DFunctionRegex_Dynamic" does not exist in the current context. (File YouTubeVideo.cs Line 24 and 26)
I'm not good with Regex and therefore can't fix it myself ^^

ok i fixed, try again

It works. Thanks!

Fixed. Update 3.0.0


Update Posts “a-post-with-figures-and-video”

Automatically generated by Netlify CMS

👷 Deploy Preview for enchanting-chaja-c384e5 processing.

Name
Link

🔨 Latest commit
c5c85270946ebe3160603b4b369dfc5bb7bfa392

🔍 Latest deploy log
https://app.netlify.com/sites/enchanting-chaja-c384e5/deploys/66d387e428568500088bcb6c


Bring back the function started/function completed log lines

This was removed by mistake in 1.4.0
See https://discord.com/channels/1128210118216007792/1136316066696532069/1301606873035964487

Hello, I've fixed this issue!
This is what I've done:
I have reintroduced the endpoint() function in the public_api.ts file. This function was mistakenly removed in version 1.4.0. The endpoint() function is crucial for creating a new RestateEndpoint, which is used to bind services and start listening on a port. Here's what I did:

Added the following code to packages/restate-sdk/src/public_api.ts:

import { NodeEndpoint } from "./endpoint/node_endpoint.js";

/**
 * Create a new {@link RestateEndpoint}.
 */
export function endpoint(): RestateEndpoint {
  return new NodeEndpoint();
}

This change ensures that users can continue to use the endpoint() function as they did in previous versions, maintaining backwards compatibility and preventing breaking changes in existing code.
I also created pull request: https://github.com/restatedev/sdk-typescript/pull/458

[!CAUTION]
Disclaimer: This fix was created by Latta AI and you should never merge before you check the correctness of generated code!

This bug was fixed for free by Latta AI - https://latta.ai/ourmission


Approxfun support started

Should we merge this into master now?
I'm going to try getting ApproxFun to use Domains.jl again, see if the crash is still there.

Codecov Report

Merging #13 into master will increase coverage by 0.12%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #13      +/-   ##
==========================================
+ Coverage   88.42%   88.54%   +0.12%
==========================================
  Files          24       24
  Lines         743      751       +8
==========================================
+ Hits          657      665       +8
  Misses         86       86

Impacted Files
Coverage Δ

src/domains/interval.jl
95.23% <100%> (+0.23%)
:arrow_up:

Continue to review full report at Codecov.

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

Coverage increased (+0.1%) to 88.549% when pulling e054e23bc3c46122da11aae8ad7b1923cfd1d648 on approxfun-support into ce2c071b415e5b561145f611a10f56424e522721 on master.

Yes, this can be merged.

Great! I had another look at the SEGFAULT but it's still there, definitely at compile time (I forgot that I had reduced it to a precompile call), and I wasn't able to work around it. As the dust on Julia v0.7 starts to settle I'll try again on master: if the SEGFAULT is still there they might be able to fix it.

That is unfortunate! I remember I also had a compile-time issue for which I commented out some code, I will try that again and file a report if needed.

I was thinking of another issue. It is still there, I filed https://github.com/JuliaLang/julia/issues/26487.

I wonder if it's the same issue? Sometimes stack overflows cause SEGFAULTs (I'm not sure why).

That would be nice, because https://github.com/JuliaLang/julia/issues/26487 is probably easier to fix. However, I rewrote the offending lines so as not to trigger this bug, so if it is the same (or a similar) issue it must at least be elsewhere too.


Rancher fails to run jail command on first startup

What kind of request is this (question/bug/enhancement/feature request):
Bug/question
I'm not sure what to classify this as, as I don't know whether this is an error on my end or not.
Steps to reproduce (least amount of steps as possible):

Create 2 kubeadm nodes with openSUSE Kubic
Make one control-plane, etcd with kubeadm init and join the other as worker
Add weave CNI
Install cert-manager
Use helm to install rancher with letsEncrypt as issuer
Grab a rancher pod log

Result:
Rancher pods never deploy, healthz fails and they crash multiple times
https://hastebin.com/raw/icifimasaz
Other details that may be helpful:
Kubic does not use the docker runtime, I believe it uses CRI-O. I'm not sure whether that is the thing that would be causing the issue or not.
Environment information

Rancher version: rancher-latest/rancher

Installation option: 2-node non-HA with kubeadm (1 control-plane, etcd and 1 worker)

/etc/os-release

NAME="openSUSE MicroOS"
# VERSION="20201221"
ID="opensuse-microos"
ID_LIKE="suse opensuse opensuse-tumbleweed"
VERSION_ID="20201221"
PRETTY_NAME="openSUSE MicroOS"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:opensuse:microos:20201221"
BUG_REPORT_URL="https://bugs.opensuse.org"
HOME_URL="https://www.opensuse.org/"
DOCUMENTATION_URL="https://en.opensuse.org/Portal:MicroOS"
LOGO="distributor-logo"

kernel = 5.9.14-1-default

Cluster information

Cluster type: Custom Hetzner kubeadm on openSUSE Kubic
Machine type: KVM, 4vCPU, 8GB ram
Kubernetes version:

Client Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.0", GitCommit:"af46c47ce925f4c4ad5cc8d1fca46c7b77d13b38", GitTreeState:"clean", BuildDate:"2020-12-16T00:00:00Z", GoVersion:"go1.15.6", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.0", GitCommit:"af46c47ce925f4c4ad5cc8d1fca46c7b77d13b38", GitTreeState:"clean", BuildDate:"2020-12-16T00:00:00Z", GoVersion:"go1.15.6", Compiler:"gc", Platform:"linux/amd64"}

Merging into https://github.com/rancher/rancher/issues/27160

@cardoso81priscila

#vamosjuntosparasempre


Add UIO_MAXIOV constant to uclibc

#1880 added UIO_MAXIOV to linux-like libc, but did not add it
to uclibc. This causes build breaks for std against uclibc.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @JohnTitor (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Looks good, but CI didn't agree with it. Let's re-run it to see if it still happens.

Found out that rust-lang/rust#74532 causes the failure on macOS nightly CI (s390x's is spurious and semver check's is unrelated to this). I'd skip the build check for that target if the upstream doesn't get a fix in a few days.

Sounds good! Thanks for looking into that.

Thank you!!


add PackFromReader

#21

Codecov Report

Merging #22 into master will decrease coverage by 0.68%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   55.25%   54.57%   -0.69%
==========================================
  Files           9        9
  Lines        1122     1136      +14
==========================================
  Hits          620      620
- Misses        407      421      +14
  Partials       95       95

Impacted Files
Coverage Δ

packx.go
66.92% <0%> (-3.86%)
: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 59d54fd...b123309. Read the comment docs.


Mean Value Theorems

I want to add mean value theorems inside calculus section. Mean Value Theorems include (i) Rolle's Theorem, and (ii) Lagrange's Mean Value Theorem (LMVT) . Geometrical Interpretation for these two can also be included for more clarity.
If seem relevant please assign this issue to me under gssoc21

Will you create a Calculator for it?

Not the calculator.
Theorems and explanations.


Use dark scrollbars when in darkmode

This will change the scrollbars from OS defaults into the dark variant when the app is in darkmode.

Example from the .log file list:

that's a great catch, did not know about this property!
Thanks @tfevens 🚢  will tag a release later today

v1.2.4 is out - https://github.com/opcodesio/log-viewer/releases/tag/v1.2.4


Temporary Fix for 5915

Related Issue
https://github.com/DoSomething/phoenix/issues/5915
Affected Pages
All campaign pages with additional materials hosted on the site, but hacked on Naughty or Nice, Say it with the Stars, Not Stuck in Traffick, Treat Yo Friends, and #ReelWomen -- both their GL EN and EN translations.
Hack Explanation
Downloads were not displaying despite being uploaded on the back end, so we added them to the stuff you need section like so (last line in Stuff You Need):

Please note if this needs to be undone at some point and why.
This needs to be undone when the fix for 5915 is deployed to Prod otherwise the materials will all show up twice in the Stuff You Need section.

Also documented here https://docs.google.com/a/dosomething.org/spreadsheets/d/1ryBMOw05mMYtK49Gms20QZHfAbnqze5wkykyylvyc_E/edit?usp=sharing

Reverted now that the fix has been deployed to prod. Closing.


Add sortBy to group by opeation.

Codecov Report

Merging #64 into master will increase coverage by 0.75%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   31.89%   32.64%   +0.75%
==========================================
  Files          44       44
  Lines        1207     1219      +12
  Branches       42       44       +2
==========================================
+ Hits          385      398      +13
+ Misses        822      821       -1

Impacted Files
Coverage Δ

...et/scalax/fsn/slick/operation/GroupOperation.scala
89.83% <100%> (+4.11%)
:arrow_up:

...cala/net/scalax/fsn/slick/atomic/GroupSelect.scala
86.95% <100%> (+1.24%)
:arrow_up:

Continue to review full report at Codecov.

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


rustdoc-json: add support for --document-private-items

See https://doc.rust-lang.org/rustdoc/command-line-arguments.html#--document-private-items-show-items-that-are-not-public

Thank you for taking the time to create a PR :pray:
Looks good! Except your code didn't work ;)
error: Found argument '--document-private-items' which wasn't expected, or isn't valid in this context

  If you tried to supply '--document-private-items' as a value rather than a flag, use '-- --document-private-items'

Usage: cargo rustdoc --lib --target-dir <DIRECTORY> --quiet --manifest-path <PATH> [args]...

I fixed that and added a regression test.
May I ask what your use case is? Since your code didn't work, it makes me suspect you don't need it by yourself, but instead you just thought this option could be good to have? I agree with that. I think it is only a matter of time before someone needs it.

Looks good! Except your code didn't work ;)

Ups sorry, thanks for catching that!

May I ask what your use case is?

Sure! I'd like to experiment with the possibility of compile-time introspection by abusing rustdoc 😅
The use case would be improving my rusteval crate.
I'd like to have a build script or proc macro generate a dynamic interface for a chosen struct and use that to drive a REPL.

Very interesting! Good luck on your adventures!
I just released the new function in rustdoc-json 0.7.2


Improve format of command line output

Would you like me to update the CLI output in README.md as well?

@jung-kurt sure, that would be very helpful. Thank you!


Fixes

[x] Make typescript compile it's own workflow: (pnpm/npm)-typescript-compile
[x] Fix pnpm install on typescript compile pnpm
[x] Make a single workflow for both (pnpm/npm)-eslint-typescript, extending to (pnpm/npm)-eslint and  (pnpm/npm)-typescript-compile.

Fixed in b8ddae3225cf62e99afa3b2a2cccbf8f72211f99


Add url dialog for to allow copying for non-github element repos

Currently if an element is coming from a bower url which isn't in the github user/repo format the github links break. This update adds a dialog to allow the user to copy the url if it's not a github link.

I signed it!

I signed it!

If I had to do this a different way I'd make an element-source element since this _githubLink() happens in app/elements/pages/page-element.html as well


AttributeError: type object 'Block' has no attribute 'CreateRequest'

in url:https://docs.steamship.com/packages/cookbook/collecting-and-querying-sentiment.html
def add_document(self, content: str = None) -> str:
file = File.create(self.client,
blocks=[Block.CreateRequest(text=content)]). # bug here!
bug is in version 2.17.11

Hi there,
My apologies, the docs were out of date.
If you remove the .CreateRequest, this should work.
blocks=[Block(text=content)]


update Dockerfile to use ubuntu:18.04

Ubuntu 18.10 is no longer supported so the apt-get commands fail leaving the Dockerfile unbuildable.
Dropping down to the closest LTS version.
It might be worthwhile updating this to 20.04, but will need an actual maintainer to do that because some of the dependencies no longer exist (eg. libffi6).

Thanks for the PR!


Rename creasty/apperrors -> izumin5210/rrr

apperrors is too long...  😭

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   96.18%   96.18%
=======================================
  Files           5        5
  Lines         131      131
=======================================
  Hits          126      126
  Misses          3        3
  Partials        2        2

Impacted Files
Coverage Δ

error.go
100% <ø> (ø)
:arrow_up:

stack.go
84.37% <ø> (ø)
:arrow_up:

pkgerrors.go
100% <ø> (ø)
:arrow_up:

hash.go
100% <ø> (ø)
:arrow_up:

options.go
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


feat: enabling nullifier tree snapshot

In this PR I enable nullifier tree snapshot and I use that in the inclusion proofs test.
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
Metrics with a significant change:

node_database_size_in_bytes (10): 5,836,770 (+49%)
note_trial_decrypting_time_in_ms (8): 19.4 (-57%)
note_trial_decrypting_time_in_ms (32): 44.6 (-56%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 6877ca1c and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
222,996
868,136
3,449,324

l1_rollup_execution_gas
842,083
3,595,244
22,204,693

l2_block_processing_time_in_ms
2,189 (-9%)
8,435 (-3%)
35,277 (-1%)

note_successful_decrypting_time_in_ms
310 (-8%)
919 (-6%)
3,331 (-5%)

note_trial_decrypting_time_in_ms
:warning: 19.4 (-57%)
:warning: 44.6 (-56%)
199

l2_block_building_time_in_ms
20,077 (-1%)
80,102 (-1%)
320,148 (-1%)

l2_block_rollup_simulation_time_in_ms
16,564 (-1%)
66,246 (-1%)
264,588 (-1%)

l2_block_public_tx_process_time_in_ms
3,481 (-2%)
13,784 (-2%)
55,316 (-2%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
24,882
48,091 (+1%)

note_history_successful_decrypting_time_in_ms
2,182 (-1%)
4,207

note_history_trial_decrypting_time_in_ms
163
231 (+1%)

node_database_size_in_bytes
3,630,554
:warning: 5,836,770 (+49%)

pxe_database_size_in_bytes
29,748
59,307

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
199
43,109
20,441

private-kernel-ordering
114 (-1%)
25,833
9,689

base-rollup
2,954
667,692
873

root-rollup
89.5 (+4%)
4,072
881

private-kernel-inner
260 (-1%)
64,516
20,441

public-kernel-private-input
171 (-1%)
25,203
20,441

public-kernel-non-first-iteration
168 (-1%)
25,245
20,441

merge-rollup
10.8 (-2%)
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
10,323
25,938


Allow environment variables substitition in yaml files

It would be useful if environment variables substitution was allowed for yaml files. An usage example would be refering to a different table according to the environment we are:
entities:
  MY_TABLE:
    source_database: BIGQUERY
    table_name: ${GOOGLE_CLOUD_PROJECT}.mydataset.mytable
...

# or

rule_bindings:
  MY_BINDING:
    entity_uri: bigquery://projects/${GOOGLE_CLOUD_PROJECT}/datasets/mydataset/tables/mytable
...

Hi @Luiscri thanks for the suggestion. I think this would be a great feature. I estimate this effort to be quite large as we will need to make a few important decisions on how to allow interpolating environment variables into the YAML configs. I will update this ticket when we are able to prioritize this work.

That would be really helpful for my project actually. There's a plan for this implementation?