How is score value calculated?

Hi, mine is more a question than an actual issue, I'm curious how is the score value calculated in Node.js Toolbox ?
thanks

Hi, the score is calculated as follows:
((Total NPM downloads / Total NPM downloads of most downloaded library) + ((GitHub stars + (GitHub forks * 5)) / (GitHub stars of most starred library + (Github forks of most forked library * 5)))) / 2


UAR-338: Get list of officers for overseas entity

JIRA link
https://companieshouse.atlassian.net/browse/UAR-338
Change description
Add call to API to return list of officers for a specific company
Work checklist

[ ] Tests added where applicable
[ ] UI changes meet accessibility criteria

Merge instructions
We are committed to keeping commit history clean, consistent and linear.  A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

Are there plans to offer the ability to query API to return fields from the DueDiligence array in OE data? https://github.com/companieshouse/overseas-entities-api/blob/main/spec/schema.json#L520

@StephenAbbott This PR is for use of the existing API. Please contact me (slack or teams) about any new API.

@AndreKramer7654 Unfortunately I'm external and not on Companies House's Slack or Teams. I work for Open Ownership and my email is<EMAIL_ADDRESS>Maybe we could connect over email? Or I could post on the CH developer forum?
https://github.com/companieshouse/overseas-entities-web/pull/746#issuecomment-1463990540


[memory related] Can't tensorRT extension continue to train/convert if shared memory is being used?

I want to train/convert a model for a very large resolution, but whenever it starts to use shared memory as well as 24GB vram, it then fails saying there's not enough memory.
Can you please update it to continue training on said higher res? As images looks quite good at even 768x768, but i want to upscale that to 4x, or do 16:9/21:9 low res with 4x upscale, but for that, i need to train the images, as the video memory used even at 768x768 was surprisingly low with tensorRT, so i want to use my time to convert to quite large resolutions even if it uses ram.

For the 4x upscale, what low-res do you intend to use, 768?

For the 4x upscale, what low-res do you intend to use, 768?

768 or 960. some gens do good at 960, but most do great at 768 due to the more details already, the better the upscale.

I'd like to second this, I ran into what was apparently a bug allowing it to overflow into shared memory that I can't reproduce. I was making a 1728x1152 profile and it was using 23gb VRAM and 38.8GB RAM, but I killed it to test static vs dynamic speed difference at a lower res first. I can't replicate whatever was allowing it to overflow, now I just get
"[E] 9: Skipping tactic0x0000000000000000 due to exception [::0] autotuning: User allocator error allocating<PHONE_NUMBER>37-byte buffer
[E] 1: [defaultAllocator.cpp::nvinfer1::internal::DefaultAllocator::allocate::20] Error Code 1: Cuda Runtime (out of memory)
[E] 9: Skipping tactic0x0000000000000000 due to exception [::0] autotuning: User allocator error allocating<PHONE_NUMBER>37"
I have no idea if the engine that run would have resulted in would be usable, but the time savings for img2img upscales would have been worth the time it was taking to generate it.


Unclear outcome

Hi,
I've been trying to use this validator, but I'm not very clear on the outcome, as it just exits the ./validate command after the Installing CPI, Extracting release and Compiling package states. Is this expected behaviour? Thanks in advance.
Regards,
Georgi

Hi Georgi,
It's not the expected output. It looks like the last state test execution wasn't executed.
Could you check the logs folder for errors? The logs folder is in the directory which is used by the validator for the installation. It's printed out at the beginning with Using ... as temporary directory.
May be you can execute the validator again and do a tail -f <temp-folder>/logs/* because we have different logs for the different states.
Thanks,
Beyhan

@georgibarzev are you still having trouble running the validator? Could you please provide some logs as requested? Thanks!

Closing due to lack of activity. Please re-open if the problem persists.


Could not load standard swift library driver bug.

@stevapple
continuing from https://github.com/swift-server/vscode-swift/issues/416

@stevapple sorry to bother, any update on a fix for this?

@compnerd Did you remember in which specific PR you introduced scripting support for Windows?

No, I don't sorry.


Analyzer: Static warnings are reported as errors

Static warnings exist in Dart again, cf. issue #34349, and they should be reported as such and recognized in tests via the outcome expectation static type warning. However, using a fresh SDK checkout (commit fa0d99156f720df84ed48469615d482e81126be5), the following program gives rise to test failures as shown below:
class A {
  int //# 0: static type warning
  operator []=(x, y) {}
  void f([int i = 0]) {}
}

int //# 1: static type warning
set setter(value) {}

int get foo => 42;
set foo(
  String //# 2: static type warning
  s) {}

class B extends A {
  void f([int i = 1]) {} //# 3: static type warning
}

enum E { one, two, three }

void useE() {
  var v = E.one;
  switch (v) {
    case E.one: break;
    case E.two: break;
    /* //# 4: static type warning
    case E.three: break;
    */ //# 4: continued
  }
}

int bar() { return; } //# 5: static type warning

Future<int> barAsync() async { return; } //# 6: static type warning

main() {
  A()[0] = 0;
  setter = null;
  foo;
  foo = null;
  B().f();
  useE();
  bar(); //# 5: continued
  barAsync(); //# 6: continued
}

Here is the output:
tools/testing/dart/main.dart:1: Warning: Interpreting this as package URI, 'package:test_dart/main.dart'.
Test configuration:
    custom configuration(architecture: x64, compiler: dart2analyzer, mode: release, runtime: none, system: linux, vm-options: [], timeout: 960, host-checked, preview-dart-2)
Suites tested: language_2

FAILED: dart2analyzer-none release_x64 language_2/scratch_test/2
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 10.000758s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_2.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|MISMATCHED_GETTER_AND_SETTER_TYPES|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_2.dart|12|1|18|The return type of getter 'foo' is 'int' which isn't assignable to the type 'String' of its setter 'foo'.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/2

[00:11 |  12% | +    0 | -    1]
FAILED: dart2analyzer-none release_x64 language_2/scratch_test/5
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 10.000751s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_5.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|RETURN_WITHOUT_VALUE|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_5.dart|34|13|7|Missing return value after 'return'.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/5

[00:11 |  50% | +    2 | -    2]
FAILED: dart2analyzer-none release_x64 language_2/scratch_test/0
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 11.000286s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_0.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|NON_VOID_RETURN_FOR_OPERATOR|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_0.dart|4|3|3|The return type of the operator []= must be 'void'.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/0

[00:11 |  62% | +    2 | -    3]
FAILED: dart2analyzer-none release_x64 language_2/scratch_test/6
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 11.000320s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_6.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|RETURN_WITHOUT_VALUE|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_6.dart|36|32|7|Missing return value after 'return'.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/6

[00:11 |  75% | +    2 | -    4]
FAILED: dart2analyzer-none release_x64 language_2/scratch_test/1
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 11.000416s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_1.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|NON_VOID_RETURN_FOR_SETTER|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_1.dart|9|1|3|The return type of the setter must be 'void' or absent.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/1

[00:11 |  87% | +    2 | -    5]
FAILED: dart2analyzer-none release_x64 language_2/scratch_test/4
Expected: Pass
Actual: CompileTimeError

--- Command "dart2analyzer" (took 11.000395s):
DART_CONFIGURATION=ReleaseX64 sdk/bin/dartanalyzer_developer --ignore-unrecognized-flags --packages=/usr/local/google/home/eernst/devel/dart/work/sdk/.packages --format=machine --no-hints /usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_4.dart

exit code:
1

stderr:
ERROR|STATIC_WARNING|MISSING_ENUM_CONSTANT_IN_SWITCH|/usr/local/google/home/eernst/devel/dart/work/sdk/out/ReleaseX64/generated_tests/language_2/scratch_test_4.dart|25|3|10|Missing case clause for 'three'.

--- Re-run this test:
python tools/test.py -m release -c dart2analyzer -r none --host-checked -t 960 language_2/scratch_test/4

[00:11 | 100% | +    2 | -    6]

The subtest 3 (about default values) succeeds, reporting 'WARNING|STATIC_WARNING'.
All other subtests fail because the outcome is 'ERROR|STATIC_WARNING', which is not recognized by 'test.dart' as a static type warning. It is not obvious to me whether it's analyzer or 'test.dart' that needs to be adjusted in order to make these subtests succeed.
Note that it's being debated whether the invalid return; statements should be an error or a warning (dart-lang/language#255), which is significant for subtest 5 and 6.

Current status: This topic needs to be clarified, so we'll make it 'area-language' until a decision has been made on the use of warnings in Dart in general.

Closing, the language change decision is handled via https://github.com/dart-lang/language/issues/1012.


please consider adding a pure JavaScript example :)

Hi, Thank you very much for this :D
please consider adding a pure JavaScript example :)

Hi @AVOlight , would you elaborate the scenario that needs a pure JavaScript example?  Is the typescript example not working or you need some specific execution environment?

Hi @neeboo, sorry yes I'm not very experienced with web so pure JavaScript is the easiest way for me to connect through a JavaScript Unity plugin.

Hi @neeboo, sorry yes I'm not very experienced with web so pure JavaScript is the easiest way for me to connect through a JavaScript Unity plugin.

Hey, I will be exposing the client so it can be used outside the supported frameworks yes.

Hi @neeboo, sorry yes I'm not very experienced with web so pure JavaScript is the easiest way for me to connect through a JavaScript Unity plugin.

Hey! Please try the latest version 0.1.0
The client is exposed now. See here: https://connect2ic.github.io/docs/docs/Core/Client
It may not be perfect but hopefully we can fix and improve it according to your needs.

Thank you very much :)

Thank you very much :)


Fix broken deploy_failure.feature tests

It seems that enhancing a task using an after block will not have desired effect if the new task shares the same name (excluding namespace) as the task being enhanced. In other words, after 'deploy:failed', :failed do ... will not work (the block will never be executed). This behavior was recently introduced in 5fcf3da.
To fix the feature test using this code, I renamed the task to be a unique name. The test now works as intended.
Also, I noticed the task defined in fail.rake was sometimes not working as intended, because the shared_path did not exist on the VM's filesystem. This meant that touch was failing. Corrected by issuing mkdir -p first.
With these changes, all feature tests now pass.

If I'm reading this correctly @mattbrictson this doesn't solve the problem for which we have another ticket, but rather works around it by making our tests more robust, and not shadowing another task name?

If I'm reading this correctly @mattbrictson this doesn't solve the problem for which we have another ticket, but rather works around it by making our tests more robust, and not shadowing another task name?

This is correct. I wasn't sure what the resolution of the after issue would be, so I worked around it.


Rylid umil patch 1

why do i keep opening PR's to the main branch on accident

dont delete branch tho pls


Improve the main function BBE

Description:
$title.
With the changes to function parameter order and invocation, the main function BBE could be improved with a better example that demonstrates how combinations of arguments can be passed (for required, defaultable and rest params).
Affected Versions:
1.0-alpha

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/18588.


openblas: add aarch64 platform config

Motivation for this change
let openblas build on aarch64
tests pass, but i haven't checked any further than that
Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

cc @dezgeg
By the way, should we have an aarch64 tag?


Very crude proposal for GraphEdge and MaternHodgeCompositionalKernel

Hi Slava,
It now should fit the new code structure well. If you have time, could you please have a look at the old PR: https://github.com/geometric-kernels/GeometricKernels/pull/139/commits/d7eb878831d90bbf8acd0bcb15b63ef383645eb2

@cookbook-ms, thanks! I plan to take a look within the next week or two.


Get new letters function

What does this implement/fix? Explain your changes.
Implements getNewLetters() function in GameManager. Also includes a potential bug fix.
Does this reference any currently open issues?
No
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. This is simply a reminder of what we are going to look for before merging your code.

[ ] I have linted my code and I have run tests locally that pass
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation

Any other comments?
…

Was getting a 500-some line log for linting, will look more into it tomorrow.

status was set to completed, can this be reviewed now? @GDave50


France map

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 3 committers have signed the CLA.:white_check_mark: mike-koch:white_check_mark: Rendername:x: Mike KochMike Koch seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


UI: fix create vpc private gw by regular users

Description
This PR fixes an regression issue of #6400
Ths issue is, the associated network is always required, even when a network is selected from the drop down, which cause regular users arenot able to create private gateway with associated network.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

UI build: :heavy_check_mark:
Live QA URL: http://qa.cloudstack.cloud:8080/client/pr/6445 (SL-JID-1697)


fix(index): add rootContext (this.rootContext)

Webpack 4 removed this.options from loaders. The replacement for
this.options.context is this.rootContext.

this.options and this.rootContext
webpack 3 already deprecated this.options in the loader context. webpack 4 removes it now.
A lot of people are missing the this.options.context value now. It has been added as this.rootContext.

https://medium.com/webpack/webpack-4-migration-guide-for-plugins-loaders-20a79b927202
Fixes #69

Released in v0.5.6 🎉 Thx


index_page loop not progressing

I have a script for a website with some pages. When I run the script I get up to the page 2 by clicking at the  "play arrow" of the "index_page". When the script reaches:
self.crawl(self.next_page, callback=self.index_page)
(the first time self.crawl is called in the index_page function)
the script is stuck displaying no errors.
However, when I run the same script in debug mode I can see that the links are populated normally.
I'm using pyspider v.038.

Did the following link crawled before? it may be deduplicated.

Nope. It hasn't crawled before. It just doesn't showing pages after the 2nd page regardless of it has crawled before or not.

paste the script.

I've sent you an email.


Added checks for disallowing overlay networks with reserved names.

Certain network names are reserved by Docker. Currently the reserved
names seem to be "host", "bridge", "default" come to mind. Accordingly
we are disallowing creation of networks with these names.

Other than those, LGTM!


no-code-change

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 83.10%. Comparing base (1d96200) to head (1cfce91).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage   83.10%   83.10%
=======================================
  Files           8        8
  Lines         148      148
=======================================
  Hits          123      123
  Misses         25       25

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


foot: Pull patch to fix under Mir

Description of changes
Upstream issue: https://codeberg.org/dnkl/foot/issues/1444, fixes miriway VM tests.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

@ofborg build miriway.passthru.tests


Environment showing as degraded after upgrade from 1.1.2 to 1.2.2

Rancher Versions:
Server: 1.2.2
healthcheck:0.2.0
ipsec:0.8.1
network-services:0.4.0
scheduler:0.5.0
kubernetes (if applicable):N/A
Docker Version:
Client:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:        Thu Aug 18 17:52:38 2016
OS/Arch:      linux/amd64
Server:
Version:      1.12.1
API version:  1.24
Go version:   go1.6.3
Git commit:   23cf638
Built:        Thu Aug 18 17:52:38 2016
OS/Arch:      linux/amd64
OS and where are the hosts located? (cloud, bare metal, etc):
AWS
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single Node rancher with externaldb (RDS)
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle
Steps to Reproduce:
Upgrade Rancher from 1.1.2 to 1.2,2
Results:
-Under Environments / Manage environment our main environment shows up as Degraded
-Stacks are all running without issues, so we are not sure where the "Degrade" state comes from
Expected:
Environment should not show up as degraded, and/or there should be a way to identify the source of that status.

Degraded for environment is the same concept of degraded for stacks.
If any of your stacks have any issues and are not green/active, environment will be degraded.

All our stacks are green and healthy except for two that are marked as Inactive? does this mean that these Inactive stacks cause the environment state to be Degraded ?

Yes, those would cause them to be degraded. Just like how an inactive service causes a stack to appear degraded.

Really ?. A stack that has been explicitly set to inactive causes the whole environment to show as degraded, why ?  Degraded to me suggests that something is not working or exhibiting unexpected behaviour. If I deliberately stop a service for whatever reason, that is the behaviour I expect right ?

I agree, if a service was delierately stopped it should not show the environment as degraded. It's just confusing

@pythianarceri i have updated the title of this ticket to better reflect whats the issue discussed here.
@deniseschannon This issue makes the healthState property of environments useless as to determining whether all the services in those environments are healthy.
Imagine having to monitor 10+ environments. Instead of being able to rely on this aggregated view to get insight into the health of environments

one needs to navigate through each of the environments and check whether any stacks/services are actually unhealthy or just contain one or more deactivated stacks.

I completely agree, if a stack has been purposefully stopped, it should not be marked as unhealthy it should be marked as inactive and it should not degrade the entire environment.

:+1: Stopped stacks should not mark environment as degraded

With the release of Rancher 2.0, development on v1.6 is only limited to critical bug fixes and security patches.


Templates swapped with Polymer

When using certain Polymer elements in Aurelia templates, insertion points such as string interpolation, repeat attributes, and router-view outlets change places. In my particular instance, a paper-item element generated from router navigation routes ends up where the router-view element is and the content of the routed view ends up in the paper-menu containing the paper-item.
So far, the only Polymer elements I've noticed this with are paper-scroll-header-panel and paper-drawer-panel. I checked with paper-header-panel, which also selects child content, and didn't run into the issue.

The
 containing the highlighted <paper-item> contains the router-view, and the original source is

    <router-view></router-view>

The paper-item is generated by
<paper-item repeat.for="route of router.navigation">
    ${route.title}
</paper-item>

I've tried this in both Chrome and Firefox, with Polymer 1.0 and aurelia-templating 0.11.2.
My sample application is here.

I'll take a look at it.

Hi @EisenbergEffect has this issue been sorted, i am working with both libraries in an application we just started.Thanks

Excellent. What was the issue? We recently made some changes to how our anchors worked. Did that solve it or was there something else going on?

Oops. Sorry I misunderstood you and thought you were saying that it had been solved. I haven't looked into it yet. Is it broken with the absolute latest releases?

I've got same issue, here is template code:
<template>

  <paper-drawer-panel force-narrow>
    <paper-header-panel drawer>
      <paper-toolbar></paper-toolbar>

        <paper-menu>
          <paper-item repeat.for="route of router.navigation">
            ${route.title}
          </paper-item>
        </paper-menu>

    </paper-header-panel>
    <paper-scroll-header-panel condenses main id="mainScrollPanel">
      <paper-toolbar class="paper-header" id="mainToolbar">
        <paper-icon-button icon="menu" tabindex="1" paper-drawer-toggle></paper-icon-button>
        ${router.title}
      </paper-toolbar>

          <router-view></router-view>

    </paper-scroll-header-panel>
  </paper-drawer-panel>

</template>

If i remove  content in div.page-host is appeared.
I can share my repo if you need

Got the same issue here

In order to investigate this, I will need a full working sample with instructions. The above link did not build for me, so I wasn't able to investigate further.

I got it working. I'm seeing the odd behavior. I have some theories and will look into it.

I'll also update the sample with the latest versions of everything in case that changes anything

@roguePanda Your sample is very hard to work with because you've got a non-standard build and you've pre-built everything into a bundle, which makes it hard to work with. I have an idea of how to fix this, but instead of it taking me 15 minutes to test, it's going to take me much, much longer. As a result, I can't guarantee it will get fixed in this iteration. If someone wants to send me a minimal reproduction based on top of the skeleton with no custom building and bundling, then I can probably deal with this pretty quickly.

Ok, good news. I've figured out how to fix it.

I've been working on a sample based on skeleton-navigation. Do you still need it?

Polymer's unregistered element bookmarklet flags the nav-bar custom element as unregistered also, but that could be a separate issue since taking it out doesn't seem to change anything.

Here's a brief explanation:
The Aurelia compiler inspects DOM nodes and extracts data about bindings and behaviors..and generally does as much work ahead of time as it can so that view creation is fast. The extracted "instructions" have previously been tied to DOM structure so that a query selector could discover all au-target elements and then we could simultaneously loop over those and the instructions, applying them as we go.
In the case of these special Polymer elements, the DOM gets manipulated by them during the clone of the template. So, at the earliest point that Aurelia can perform a query selector for targets, those Polymer elements have already re-arranged the DOM. The side-effect is that the ordering of extracted instructions no longer matches the ordering of elements returned from the query.
The only way I know to fix this is to tag every au-target with a unique id and then store the extracted instruction by that id, thus making the view instantiation process independent of the DOM structure. This can only happen with Web Components, so this should fix any scenario where a 3rd party web component alters the DOM during the create (clone) phase.

@roguePanda Aurelia doesn't register its custom elements as web components so that bookmarklet would definitely report them. It has no affect on runtime behavior though. We choose not to do that because our implementation is lighter weight, faster and more deeply integrated with the binding system than a standard web component would be. But bottom line: there's no need to worry about what that bookmarklet reports with regard to Aurelia.

To clarify, I no longer need an additional sample. I was able to confirm that my approach to changes to the compiler and view factory fix the behavior in the provided sample.

I've pushed a fix for the issue. It should go out in the next update within a few days. Also, if you've been following the official blog, you may have noticed that we like to feature community members. So, if anyone is interested in writing a blog post on using Polymer with Aurelia, just let me know and we would be happy to feature your article on our official blog.


Fix a couple of IRGen tests on master-next

One is that clang now generates some symbols as internal rather than private, another is a change in label syntax.

@swift-ci please test

(test will fail, there are other issues)

Build failed
Swift Test OS X Platform
Git Sha - 5bbd108283ee8d425ebdef1f1ef74049d7e7778d

Build failed
Swift Test Linux Platform
Git Sha - 5bbd108283ee8d425ebdef1f1ef74049d7e7778d


Adds Http, https, and ftp support to protect (resolves #149)

Failed due to Transgene docker mislabelling. Fixing in #158.

Hey @akul-goyal if you rebase to the latest master the ci test will pass.

Hi Arjun,
I rebased the changes onto master.
Let me know if you have any problems.
Akul
On Tue, Feb 14, 2017 at 7:09 PM, Arjun Arkal Rao<EMAIL_ADDRESS>wrote:

Hey @akul-goyal https://github.com/akul-goyal if you rebase to the
latest master the ci test will pass.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/protect/pull/157#issuecomment-279905144,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUOSxreHafYZqKqWzbS586opW6rb_pttks5rcmx0gaJpZM4MBMI2
.

Hi @akul-goyal , I don't think your rebase worked, the test failed since it is still looking for Transgene:1.0.0 and not 2.0.0. We'll look at it tomorrow.

jenkins, test this please

Hey @akul have you changed the policy on the bucket yet? This will fail as long as the policy isn't set to the new one.

Hi Arjun,
I did change the policy and in the test which I ran on one of the files in
the bucket, it worked.
Akul
On Sat, Feb 18, 2017 at 5:59 PM, Arjun Arkal Rao<EMAIL_ADDRESS>wrote:

Hey @akul https://github.com/akul have you changed the policy on the
bucket yet? This will fail as long as the policy isn't set to the new one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/protect/pull/157#issuecomment-280889710,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUOSxrOesrJsNswBOGGnD9-VgQrPWzZGks5rd6IdgaJpZM4MBMI2
.

This was failing since the bucket was set up as requester pays and anonymous downloads are not permitted from requester-pays buckets. @akul-goyal will add a fix to catch the error that allowed the previous runs to move forward after the failed download and resubmit this PR WITHOUT the https input in the CI test inputs. I will re-run tests with the unit-test make target for jenkins and merge it in pending those tests passing.

Yes!
On Feb 23, 2017 2:53 PM, "akul-goyal"<EMAIL_ADDRESS>wrote:

@akul-goyal commented on this pull request.
In src/protect/test/unit/test_file_downloads.py
https://github.com/BD2KGenomics/protect/pull/157#discussion_r102840471:

@@ -102,5 +103,49 @@ def _download_files(job):
raise

def test_file_downloads_from_https(self):

   """

   Test the functionality of get_file_from_url https

   """

   a = Job.wrapJobFn(self._download_https_files)

   Job.Runner.startToil(a, self.options)

@staticmethod
def _download_https_files(job):

Do you want me to also test for an error ie: put a dummy link?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/protect/pull/157#discussion_r102840471,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALHTOMDdo4MsiASwJcRX6y2SQAzUAV8gks5rfg4CgaJpZM4MBMI2
.

Jenkins, test this please.


Interactive Window settings are hard to identify

All Jupyter extension settings are under the same group, but it's hard to tell what would apply to IW vs notebooks in general

verification steps:

open vscode without activating the jupyter extension (no .ipynb or .py editors visible)
open your settings JSON and add some deprecated settings (should have deprecation message)

"jupyter.interactiveWindowMode": "perFile",
"jupyter.newCellOnRunLast": false,
"jupyter.sendSelectionToInteractiveWindow": true,
"jupyter.enableAutoMoveToNextCell": false

activate the jupyter extension (open a .ipynb file)
check that the settings are eventually migrated

Doesn't work in a non-default profile. When I switch to a non-default profile and then run the "Open Current Profile Settings (JSON)" command, that opens a settings file that isn't the same as the JSON file used for the default profile.

And it's good now, at least on the default profile.


Rename name to qname, closes #76

I honestly don't think that anyone outside Capybara is really using this gem, and definitely not for anything complicated enough to warrant usage of this method, which is rather a fringe feature, since it's usually much easier to specify the tag name on the axis. I think it's probably fine to go straight to 3.0 on this.


Torchvision compatibility

Enable compatibility with most recent torchvision version (>= 0.13.0)

I am thinking about either a dictionary, a named tuple, or a (custom) class that maps the names of the models (usually lower-case) to the names of the model weights (mostly a mix between upper-case and capitalization; see here: https://pytorch.org/vision/stable/models.html). We want to avoid a bunch of if-statements to obtain the corresponding model weights. Possibly appropriate thing to do is something along the lines of model2weights[model_name] = getattr(f'{torchvision.models.weight_name}_DEFAULT).

I thought about that too, but then I saw that using strings is also supported, so if we only care about the default weights, we could also create our models with model(weights='DEFAULT') instead, which might be a bit easier?

I agree that this is the easier thing to do but there is a problem with this approach. Each set of weights has its own image transformations which can only be accessed through the weight objects themselves. We would not be able to access the transformations by using the string approach. See example below.
# Initialize the Weight Transforms
weights = ResNet50_Weights.DEFAULT
preprocess = weights.transforms()

# Apply it to the input image
img_transformed = preprocess(img)

Ah, I see, you're right. Then what about something like this:
import inspect

weights_name = [
    member for member, _ in inspect.getmembers(torchvision.models)
    if (member.lower.startswith(model_name) and 'Weights' in member)
][0]

# do some error handling in case nothing was found

weights = getattr(torchvision.models, f'{weights_name}').DEFAULT

Writing everything into a dict just seems like too much work, but we would have to think about whether something like this could produce any problems ...

Yes, that's actually a good idea! I like it. If member is a string, we could improve it by doing
import re
import inspect

weights_name = [
    member for member, _ in inspect.getmembers(torchvision.models)
    if re.search(f'(?=^{model_name})(?=.*weights$), member.lower())
][0]

# do some error handling in case nothing was found

weights = getattr(torchvision.models, f'{weights_name}').DEFAULT

Enabled. Note that a user has only access to DEFAULT weights with the current approach. We need to make this more flexible by defining extractor classes for each source.


Switch to faster blake

OpenSSL + AVX2 seems to be a faster alternative for blake2 implementation that is used a lot in kagome

Tests performed on Core i7-7500U under Ubuntu 20.04 GCC 9 have demonstrated the absence of any performance boost when a particular optimization hint was specified for the compiler (e.g. -mavx2 or -march=core-avx2). Moreover, narrowing the set of allowed optimizations sometimes gave us worse results than without compiler hints.
As the result, we ended with the conclusion that the reference blake implementation coupled with -march=native (the same as flag absence) produces the most optimized binary code on modern platforms and no further improvement is required for that part of program.


Cascading type parameters don't work with nested generics

Describe the bug
Type inference works with List<T>, but does not work with Func<List<T>> and with other nestings such as List<List<T>>
Steps to reproduce
using Microsoft.AspNetCore.Components;
using System;
using System.Collections.Generic;

namespace Test
{
    [CascadingTypeParameter(nameof(T))]
    public class Grid<T> : ComponentBase
    {
        [Parameter] public Func<List<T>> Data { get; set; } = default!;
    }

    public partial class GridColumn<T> : ComponentBase
    {
    }
}

<Grid Data="@(() => new List<string>())">
    <GridColumn />
</Grid>

Actual Behavior
Error RZ10001: The type of component 'GridColumn' cannot be inferred based on the values provided. Consider specifying the type arguments directly using the following attributes: 'T'

Generated code:
public static void CreateGridColumn_1<T>(global::Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder __builder, int seq)

Expected Behavior
No errors
Generated code:
public static void CreateGridColumn_1<T>(global::Microsoft.AspNetCore.Components.Rendering.RenderTreeBuilder __builder, int seq, global::System.Func<global::System.Collections.Generic.List<T>> __syntheticArg0)

Workarounds
First
I found that if I commented out this check in the Razor source, type inference started to work:
https://github.com/dotnet/razor/blob/7b4b0121a9c6924c94760f1c7d2bfd47a8ae7409/src/Compiler/Microsoft.AspNetCore.Razor.Language/src/Components/ComponentGenericTypePass.cs#L207
// If the parameter value is an expression that includes multiple generic types, we only want
// to use it if we want *all* those generic types. That is, a parameter of type MyType<T0, T1>
// can supply types to a Child<T0, T1>, but not to a Child<T0>.
// This is purely to avoid blowing up the complexity of the implementation here and could be
// overcome in the future if we want. We'd need to figure out which extra types are unwanted,
// and rewrite them to some unique name, and add that to the generic parameters list of the
// inference methods.
if (genericTypeProvider.GenericTypeNames.All(GenericTypeIsUsed))
{

Because genericTypeProvider.GenericTypeNames contains:
System.Collections.Generic.List<T>
T

And componentTypeParameters contains only:
T

Second
Or if I return only IdentifierNameSyntax nodes from ParseCore here:
static ImmutableArray<string> ParseCore(TypeSyntax parsed)
{
    if (TryParseCore(parsed) is { IsDefault: false } list)
    {
        return list;
    }

    // HERE
    if (parsed is IdentifierNameSyntax)
    {
        return ImmutableArray.Create(parsed.ToString());
    }

    return ImmutableArray<string>.Empty;
}

https://github.com/dotnet/razor/blob/7b4b0121a9c6924c94760f1c7d2bfd47a8ae7409/src/Compiler/Microsoft.CodeAnalysis.Razor/src/DefaultTypeNameFeature.cs#L64
.NET Version
8.0.100

@jjonescz Please take a look.

@xiety that's a great analysis you did here, thanks. I think your "second workaround" is a good approach to fix this, I'm going to open a PR.


[crypto] Reorder AES driver checks to avoid race condition.

The AES driver documentation specifies that the dest pointer should accept the output from the previous input block (if any), not the current one. See this comment:
https://github.com/lowRISC/opentitan/blob/62c7b5000a6716a8fd6dd0d54c2bfc6bcd2ecc75/sw/device/lib/crypto/drivers/aes.h#L141-L160
However, the implementation actually passed the next block of input to AES before reading the output, which could lead to a race condition where the AES block overwrites the output data before it is copied to dest, or while the copying is in progress. This has not been triggered by any tests (since the code has always executed faster than the AES block) but it seems better to be on the safe side and fix it preemptively.

super-nit: I think in the quoted comment in aes.h we should change "thus" to "thusly"

super-nit: I think in the quoted comment in aes.h we should change "thus" to "thusly"

Changed to "as follows" since I think that's even clearer :slightly_smiling_face:


Remove STM_spec.AddGC functor

This PR

removes the STM_spec.AddGC functor and
cleans up the old commented out tests

We thereby fix #178
For future reference the removed code is available on the stm-addgc branch: https://github.com/jmid/multicoretests/tree/stm-addgc

CI error summary

two 5.0.0+trunk Ephemeron segfaults
two bytecode, including 3 Thread failure-to-trigger-issues and 1 Ephemeron segfault in 5.0.0+trunk bytecode mode
two bytecode timeouts, including 1 STM Thread failure-to-trigger-issue

I will therefore merge this.


feat(engine): add fit to center

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

npm package (@latest dist-tag)
npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[cli] add refresh_state bool args

to: @jacknagz
size: small
resolves: https://github.com/airbnb/streamalert/issues/194

Coverage remained the same at 94.745% when pulling 0b5ac83a99461d3bb953ad28cccccce11f4bfb8e on ibrahimsharaf:refresh_state into b8e787ba5a0ef92bc3cf2d8206e0214635bab582 on airbnb:master.

@jacknagz what should the unit tests cover?

Coverage remained the same at 94.675% when pulling 6cac15c714b020d95d600560db9a64ed823136e8 on ibrahimsharaf:refresh_state into 3a22d5c1cd8db7f5dda1347813f367f9e6636a6f on airbnb:master.

@ibrahimsharaf they should describe how you verified that this feature is working.  we don't have great unit test coverage right now for this part of the library, but integration testing should suffice

@jacknagz I am not really familiar with the code-base or integration testing methodology, can you give me a hand?

Coverage remained the same at 94.675% when pulling 52b982e1dd51a52cecc9f6990d670b7a0213e69d on ibrahimsharaf:refresh_state into 3a22d5c1cd8db7f5dda1347813f367f9e6636a6f on airbnb:master.

@ibrahimsharaf https://www.streamalert.io/deployment.html to verify things are working.  again, I didn't see any implementation of the feature, so nothing is going to happen

I didn't see any implementation of the feature, so nothing is going to happen

@jacknagz should I modify this line https://github.com/airbnb/streamalert/blob/master/stream_alert_cli/manage_lambda/deploy.py#L160 to    helpers.tf_runner(targets=targets, refresh_state=False)?

Coverage remained the same at 94.675% when pulling 44d9cdd144f0ef68f895f246ca619521077c2726 on ibrahimsharaf:refresh_state into 80a974b66b95924caa2cd5a75f0cda391d13061c on airbnb:master.

Coverage remained the same at 94.675% when pulling 64b272d6fbeda77181167157b91704334cd9c41d on ibrahimsharaf:refresh_state into 80a974b66b95924caa2cd5a75f0cda391d13061c on airbnb:master.

@jacknagz can you review the changes and tell me if this implements the feature? also I don't have an AWS account so I can't setup streamalert and deploy, can you take it from here if so?

Thanks @ibrahimsharaf for the PR! I'm sorry we were never able to merge it, but we definitely referenced it when implementing the fix


[Snyk] Upgrade org.apache.commons:commons-text from 1.10.0 to 1.11.0

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade org.apache.commons:commons-text from 1.10.0 to 1.11.0.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 3 months ago, on 2023-10-24.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@sawood14012 I see this is still not using the 'snyk-bot' user? This duplicates #1069 which I'll try to push through in preference as that is using a correct bot user.

I'm closing this as it was resolved by #897


exists_movie plugin has no effect

I recently built a new flexget database and now when I execute my config, it wants to download every movie on my trakt watchlist. I already have most of these movies. I tried to implement the exist_movies plugin, but it has no effect on the outcome.
Here is my config:
  secrets: secretfile.yml

  tasks:
    get_series:
      priority: 1
      transmission:
        username: '{{ secrets.trans.usr }}'
        password: '{{ secrets.trans.pwd }}'
        main_file_only: yes
        path: '/media/Media/Torrents/Downloads/Shows/{{ series_name }}'
      content_filter:
        reject: '*.rar'
      content_size:
        max: 800
      regexp:
        reject_excluding:
          - ettv|eztv|rartv|PublicHD
      exists_series: '/media/Media/Shows/'
      series:
         - The Walking Dead
         - Game of Thrones
         - Pretty Little Liars
         - Silicon Valley
         - Better Call Saul
         - Mr. Robot
         - Limitless
         - Fear the Walking Dead
         - Heroes Reborn
      discover:
        what:
          - emit_series: yes
        from:
          - search_rss: http://extratorrent.com/rss.xml?cid=8
          - kat:
              category: tv
              verified: yes
          - piratebay:
              category: 208
              sort_by: seeds
          - torrentz: verified
      email:
        from: '{{ secrets.email.usr }}'
        to: '{{ secrets.email.to }}'
        subject: 'Flexget Series'
        smtp_host: smtp.gmail.com
        smtp_port: 587
        smtp_username: '{{ secrets.email.usr }}'
        smtp_password: '{{ secrets.email.pwd }}'
        smtp_tls: yes

    fill_movie_queue:
      priority: 2
      trakt_list:
        username: '{{ secrets.trakt.usr }}'
        account: '{{ secrets.trakt.usr }}'
        list: watchlist
        type: movies
        strip_dates: yes
      exists_movie:
        path: /media/Media/Movies
        type: files
      accept_all: yes
      movie_queue: add

    get_movies:
      priority: 3
      transmission:
        username: '{{ secrets.trans.usr }}'
        password: '{{ secrets.trans.pwd }}'
        main_file_only: yes
        path: /media/Media/Torrents/Downloads/Movies
      torrent_alive: yes
      content_size:
        max: 3000
      quality: 1080p
      discover:
        what:
          - emit_movie_queue: yes
        from:
          - search_rss: http://extratorrent.com/rss.xml?cid=4
          - kat:
              category: movies
              verified: yes
          - piratebay:
              category: 207
              sort_by: seeds
          - torrentz: verified
      movie_queue: accept
      email:
        from: '{{ secrets.email.usr }}'
        to: '{{ secrets.email.to }}'
        subject: 'Flexget Movies'
        smtp_host: smtp.gmail.com
        smtp_port: 587
        smtp_username: '{{ secrets.email.usr }}'
        smtp_password: '{{ secrets.email.pwd }}'
        smtp_tls: yes

    clean:
      priority: 4
      clean_transmission:
        username: '{{ secrets.trans.usr }}'
        password: '{{ secrets.trans.pwd }}'
        finished_for: 7 days

You used the accept all filter in that task as well, which overrides other filter plug-ins like exist.
Also this is more of a forum issue, please pick this up there if needed.

Removing accept all did not help.


Fix the missing comma between required and optional parameters

When using hash with the grpc message object constructor, there was a missing ',' between required and optional parameters.

Current coverage is 80.40% (diff: 100%)

Merging #679 into master will not change coverage

@@             master       #679   diff @@
==========================================
  Files           277        277
  Lines         10547      10547
  Methods           0          0
  Messages          0          0
  Branches       1397       1397
==========================================
  Hits           8480       8480
  Misses         1705       1705
  Partials        362        362

Powered by Codecov. Last update c4e1e09...36157f4

I guess I can try to join required and optional params, but that does not seem to save much.


chore: update for new sn prefix

Looks like they've started using FR:HL, as the unit I just got has that prefix. Add it to the discovery list.

Thanks!

Thanks for merging this so quickly! Installed it, but it's not working. :( Got some weird errors in Home Assistant's log about Bluetooth timeouts from the integration. :-/ Hopefully it figures itself out eventually.


cornerstoneExtensionConfig not working in prod

The following setting in default.js is working in dev, but not in prod.
cornerstoneExtensionConfig: {
    tools: {
      Zoom: {
        configuration: {
          minScale: 0.05,
          maxScale: 2.0,
        },
      },
    },
  },
´´´

@tillmon can you provide a reproduction with a prod build using CodeSandbox?

Sure, I hope this is right:
https://codesandbox.io/s/viewer-script-tag-6pbyt

I tracked the bug down to this line:
https://github.com/OHIF/Viewers/blob/2e215aed4cc283794a4365af15d3203068676e2c/extensions/cornerstone/src/init.js#L155
tool.name is not accessible in prod.

It looks like your codepen is using an old version of @ohif/viewer. When I change this import:
<script src="https://unpkg.com/<EMAIL_ADDRESS>crossorigin></script>

To the latest version:
<script src="https://unpkg.com/<EMAIL_ADDRESS>crossorigin></script>

The configuration works as expected.

Yes this works. Originally I was using the stand-alone Viewer.

@tillmon, does this resolve your issue, or are you still experiencing it but with a slightly different setup?

I tracked the bug down to this line:
https://github.com/OHIF/Viewers/blob/2e215aed4cc283794a4365af15d3203068676e2c/extensions/cornerstone/src/init.js#L155
tool.name is not accessible in prod.

same problem

https://github.com/OHIF/Viewers/blob/2e215aed4cc283794a4365af15d3203068676e2c/extensions/cornerstone/src/init.js#L154
just add the tool before this "tools.forEach" like this:
csTools.addTool(csTools.ZoomTool, {
    configuration: {
      invert: false,
      preventZoomOutsideImage: false,
      minScale: 0.05,
      maxScale: 2.0,
    },
  });

Thank you for your past contributions and for raising this issue related to legacy versions of our OHIF Viewer. Your time and effort have been invaluable in helping us improve the product.
We wanted to update you that our development focus has shifted to OHIF Viewer v3, which is now available for testing on viewer-dev.ohif.org. This site is deployed from our master branch and incorporates new features, optimizations, and bug fixes. If you're still using viewer.ohif.org, it's worth noting that this is deployed from our release branch and may not have the latest updates. You can read more about our branching strategies here.
Given the move to Version 3, we are closing older issues that pertain to Versions 1 and 2, as we are no longer providing support for those versions. If you find that the issue you've raised is still relevant in Version 3, we encourage you to reopen this issue or create a new one for proper triage.
Thank you for your understanding and continued support.


CompilerLifecycleTask is not executed when we use bal run command

Description:
We have developed a in-built compiler plugin for ballerina OpenAPI to generate OpenAPI docs for HTTP services in built time. In this plugin we have used CompilerLifecycleTask provided in ballerina project API to inject resources to generated thin-jar and executable jar.
When we use bal build command, CompilerLifecycleTask is invoked and resource injection will be successful. But if we use bal run command then CompilerLifecycleTask is not invoked and resource injection will be unsuccessful.
Affected Versions:

Ballerina Swan Lake Beta3 RC3

OS, DB, other environment details and versions:
Related Issues (optional):
Related to #1616

I think we should add a new lifecycle method to run after module code has generated. ie ModuleCodeGenerated

With the linked PR, we now support adding sources and resources with code generator compiler plugins and so these will be added to the thin jars and the executable/archive. Refactoring the resource packaging implementation to use code generators will resolve this issue.


Update dependencies

Pull Request Test Coverage Report for Build 2803

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 68.461%

Totals

Change from base Build 2786:
-0.003%

Covered Lines:
7528

Relevant Lines:
10353

💛  - Coveralls


Adding query param support for custom search engines in browsers

Resolves https://github.com/tailwindcomponents/cheatsheet/issues/50#issue-2158983583
Lots of browsers support Custom Search Engines. This change would allow me to type "tw" -> hit tab -> and type in a search term, and then I could open the tailwind cheatsheet to the specific query I'm thinking of. It'd be a huge ergonomic win!
I know that my implementation of this change isn't up to today's standards, but I didn't want to go through upgrading all of the packages to unlock useSearchParams from react-router-dom (I tried and found a few other dependencies on history that made this upgrade a larger swing than I wanted to take on.)

I don't feel strongly about the tool versions or yarn lock changes if you want to toss them

@khatabwedaa any chance you could take a look?

@heysweet, hey man sorry for not response, I'm soo busy last few months but I will check it now.

@heysweet I just text the PR thank you for your awesome work just one think I want you to remove the quarry form URL if it's empty

Thanks for your awesome work @heysweet . 😄  I merged it I will deploy it today so you can check it live.


[SP-6007] - Backport of PDI-19064 - The Append Option in JSON Output will create new files when the number of rows exceed the number of blocs (8.3 Suite)

@ssamora
@smmribeiro
@moraesvc

:white_check_mark: Build finished in 34m 31s
Build command:
mvn clean verify -B -e -Daudit -amd -pl core,engine,plugins/json/core

:ok_hand: All tests passed!
Tests run: 4864, Failures: 0, Skipped: 7    Test Results

:information_source:  This is an automatic message


Add CNI plugin configuration

find out what should be done
make a TO DO plan

Assignee is None

Created by bissquit at 2023-04-27T08:30:11Z


feat(SkewMonoidAlgebra): add SkewMonoidAlgebra with irreducible def

Skew Monoid algebras
This file presents a skewed version of Mathlib.Algebra.MonoidAlgebra.Basic with an irreducible definition.
Context
Hi everyone,
As a part of a bigger project aiming to get Drinfeld Modules into Mathlib, @María Inés de Frutos Fernández  and I have been trying to get a generalization of MonoidAlgebra, SkewMonoidAlgebra, into Mathlib. The first attempt at this was #10541, but progress has been slow as making the definition irreducible caused a huge refactor.
Following @kbuzzard's suggestion, I am opening this PR which contains only the first 160 lines of #10541. In particular, this PR is missing most of what makes these objects skewed (Multiplication was defined only at line 558 as it requires stronger hypotheses and there are many lemmas on the additive structure that precede it.) This means that the purpose of this PR is mainly to get the separation from Finsupp right.
Please consult this thread for more context or in short:
The current separation we have consists of:

Wrapping the definition with a structure.
Making basic operations such as add private and irreducible.
Adapting relevant definitions.
Copying and adapting relevant theorems from the base construct to obtain statements in terms of our new object.

Is this what everyone has in mind? If so, are there parts of the current PR that come short in respect to any of these?
I am happy to integrate suggestions to improve the separation if there are any!
Thanks a lot for your attention and help!
Xavier

This is an irreducible_def not an @[irreducible] def. If there is not kernel performance issues evident, then I much prefer the latter to push maintenance issues to core.

@Ruben-VandeVelde, this has been updated thank you.
@mattrobball, I am a little bit unfamiliar with the [@irreducible] attribute. Where is it defined? In particular, I can't prove instSMulZeroClass.smul_zero when I use it.

It is in core Lean. Current practice is to use unseal foo in before the declaration if you need to access the definition of foo or to use with_unfolding_all in tactic mode for steps that require unfolding

LGTM.

Hey everyone, I have requested reviews from the reviewers of #10541.
In particular, @eric-wieser mentioned diamond problems with the irreducibility of smul here. Is working with @[irreducible] instead of irreducible_def fine? (I assume not?) Should I revert this and only have add as irreducible?

@eric-wieser What is your feeling about irreducibility of add and smul here?

bors r+


Remove SCSS lint in Hound config

Hound was linting our styles with SASSLint which had a default configuration that made inconsistencies in our styles. So I disabled it and leave the linting in the npm run ci passed in Travis and Jenkins.

LGTM but what's the error above with stylelint?

The error was caused by Hound itself when I tried to use the same configuration that we're using in our local linters. Seems like stylelint-config-property-sort-order-smacss is not supported in Hound.

:iago :fuu +1

I've made a couple of changes:

the grunt lint task now runs npm run lint. Before, it ran only eslint for the js. Now, it runs the npm script that pass both js & css. This way we can simplify the CI recipe that has an additional step for css.
I moved lint to the first step of the test grunt task. If lint fails, it exits so we don't have to pass the tests.

Any reviewer for my last changes?


Maximise button

The centre section of the editor has a "maximise" button that hides both sidebars. The bottom panel has an expand button that makes it take over the while middle section. Currently to get the graph editor to take over the whole editor you have to press both of these buttons in a specific order. Would it be possible to add a button to the graph editor with the effect of combining both of these actions?

Added this.


"Failure to set locale" stopping installation

Hey,
I'm running Ubuntu 16.04, and after running sudo ./install to start the process, the ISO is downloaded, I get to NixOS Stage 2, and then I get the following logs:
<<< NixOS Stage 2 >>>

not remounting /
mkdir: cannot create directory '/run/keys': No such file or directory
mount: mount point /run/keys does not exist
chown: cannot access '/run/keys': No such file or directory
chmod: cannot access '/run/keys': No such file or directory
running activation script...
setting up /etc...
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
mount: /dev/pts not mounted or bad option

       In some cases useful info is found in syslog - try
       dmesg | tail or so.
mount: /dev/shm not mounted or bad option

       In some cases useful info is found in syslog - try
       dmesg | tail or so.
mount: /run not mounted or bad option

       In some cases useful info is found in syslog - try
       dmesg | tail or so.
Initializing machine ID from random generator.
unpacking the NixOS/Nixpkgs sources...
created 1 symlinks in user environment
starting /nixos-in-place/stage2...
installing ‘pcre-8.39’
download-using-manifests.pl: perl: warning: Setting locale failed.
download-using-manifests.pl: perl: warning: Please check that your locale settings:
download-using-manifests.pl: 	LANGUAGE = "en_US",
download-using-manifests.pl: 	LC_ALL = (unset),
download-using-manifests.pl: 	LC_PAPER = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_ADDRESS = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_MONETARY = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_NUMERIC = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_TELEPHONE = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_IDENTIFICATION = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_MEASUREMENT = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_CTYPE = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_TIME = "pt_PT.UTF-8",
download-using-manifests.pl: 	LC_NAME = "pt_PT.UTF-8",
download-using-manifests.pl: 	LANG = "en_US.UTF-8"
download-using-manifests.pl:     are supported and installed on your system.
download-using-manifests.pl: perl: warning: Falling back to a fallback locale ("en_US.UTF-8").
download-from-binary-cache.pl: perl: warning: Setting locale failed.
download-from-binary-cache.pl: perl: warning: Please check that your locale settings:
download-from-binary-cache.pl: 	LANGUAGE = "en_US",
download-from-binary-cache.pl: 	LC_ALL = (unset),
download-from-binary-cache.pl: 	LC_PAPER = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_ADDRESS = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_MONETARY = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_NUMERIC = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_TELEPHONE = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_IDENTIFICATION = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_MEASUREMENT = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_CTYPE = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_TIME = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LC_NAME = "pt_PT.UTF-8",
download-from-binary-cache.pl: 	LANG = "en_US.UTF-8"
download-from-binary-cache.pl:     are supported and installed on your system.
download-from-binary-cache.pl: perl: warning: Falling back to a fallback locale ("en_US.UTF-8").
these paths will be fetched (0.38 MiB download, 1.84 MiB unpacked):
  /nix/store/2z89ljhi3mgs7d1fwvlk0751mgzkisa6-pcre-8.39-bin
  /nix/store/dc2ix1i9laafk2r7n8b4anz3494grc09-pcre-8.39-doc
  /nix/store/frz4awqgmyafpgi0nnkvrl2z5rag5rh8-pcre-8.39-man
  /nix/store/zay4pq5zc26bc664mhw1smx0s8jy0wv6-pcre-8.39-dev
fetching path ‘/nix/store/dc2ix1i9laafk2r7n8b4anz3494grc09-pcre-8.39-doc’...
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").

*** Downloading ‘https://cache.nixos.org/nar/0c0jc001ka6cgx4ivp098pi08fkj6qzc1a1yw0w6l1bixaw4whs7.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/dc2ix1i9laafk2r7n8b4anz3494grc09-pcre-8.39-doc’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  309k  100  309k    0     0   476k      0 --:--:-- --:--:-- --:--:--  481k

fetching path ‘/nix/store/2z89ljhi3mgs7d1fwvlk0751mgzkisa6-pcre-8.39-bin’...
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").

*** Downloading ‘https://cache.nixos.org/nar/0ag5z27s80xr1lrwn0lx4j5cd229j8xsyybyshzpy9nzx4897b8x.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/2z89ljhi3mgs7d1fwvlk0751mgzkisa6-pcre-8.39-bin’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 41936  100 41936    0     0   190k      0 --:--:-- --:--:-- --:--:--  194k

fetching path ‘/nix/store/frz4awqgmyafpgi0nnkvrl2z5rag5rh8-pcre-8.39-man’...
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").

*** Downloading ‘https://cache.nixos.org/nar/0s058s15i7bnrv20fk6lh9czd2kv9y63565fycyviybqnrg7jaws.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/frz4awqgmyafpgi0nnkvrl2z5rag5rh8-pcre-8.39-man’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100 27664  100 27664    0     0   153k      0 --:--:-- --:--:-- --:--:--  157k

fetching path ‘/nix/store/zay4pq5zc26bc664mhw1smx0s8jy0wv6-pcre-8.39-dev’...
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").

*** Downloading ‘https://cache.nixos.org/nar/04s36b6sg630j1mvh9vjh02biy5fl9mkl9rvpgdj71y76gayzp57.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/zay4pq5zc26bc664mhw1smx0s8jy0wv6-pcre-8.39-dev’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  7884  100  7884    0     0  44160      0 --:--:-- --:--:-- --:--:-- 44795

building path(s) ‘/nix/store/d3x2yrkm76sd9mlnv4jalj1in2aaqhgg-user-environment’
created 18 symlinks in user environment
perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
	LANGUAGE = "en_US",
	LC_ALL = (unset),
	LC_PAPER = "pt_PT.UTF-8",
	LC_ADDRESS = "pt_PT.UTF-8",
	LC_MONETARY = "pt_PT.UTF-8",
	LC_NUMERIC = "pt_PT.UTF-8",
	LC_TELEPHONE = "pt_PT.UTF-8",
	LC_IDENTIFICATION = "pt_PT.UTF-8",
	LC_MEASUREMENT = "pt_PT.UTF-8",
	LC_CTYPE = "pt_PT.UTF-8",
	LC_TIME = "pt_PT.UTF-8",
	LC_NAME = "pt_PT.UTF-8",
	LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to a fallback locale ("en_US.UTF-8").
writing /nixos/etc/nixos/hardware-configuration.nix...
writing /nixos/etc/nixos/configuration.nix...
pcregrep: Failed to set locale pt_PT.UTF-8 (obtained from LC_CTYPE)

I tried running the process a couple more times, after rebooting as well, all to no avail. (Incidentally, how should I delete the redundant attempted installations?)
Some clarification would be appreciated.
Edit: I should add that these pt_PTs throughout probably stand for Portugal, I don't know why it would matter in this case (probably keyboard or hardware definitions)

Is your locale intentionally pt_PT.UTF-8, or do you want it to be en_US.UTF-8 or similar? I have never seen this error before, but it very much seems to be Perl related, more so than NixOS-related.
Have you taken a look at https://freethegnu.wordpress.com/2007/10/22/how-to-fix-the-locale-settings/ ? It was the first result from a search to DDG with Please check that your locale settings "pt_PT.UTF-8".

(Incidentally, how should I delete the redundant attempted installations?)

When you run the install script, it will output its working directory, which is typically somewhere in /tmp/. You're welcome to rm -rf it if the installation fails.

This is a glibc compatibility issue, see https://github.com/NixOS/nixpkgs/issues/8398

Thanks, @domenkozar! :)


Fluent validation against a group of properties

Is there a way to specify more than one property as part of a rule so that they can all be checked at the same time, probably as part of a custom validator. Something like
RuleFor(f => f.ProductA).AndFor(f => f.ProductB).AndFor(f => ProductC).Must(AtLeastOneProduct).WithMessage("Select at least one product")

private bool AtLeastOneProduct(bool productA, bool productB, bool productC)
{
    return productA || productB || productC;
}

Which would execute once and product a error for each of the properties.

No, that's not something you can do, sorry.
You'd either need to associate the rule with a single property or against
the entire model (with RuleFor(x=>x) ). Either way, you can still access
all 3 properties inside the Must call, but it'll only generate a single
error. If you want an error for each property you'd need to create 3 rules.
On Tuesday, 13 September 2016, srqdev<EMAIL_ADDRESS>wrote:

Is there a way to specify more than one property as part of a rule so that
they can all be checked at the same time, probably as part of a custom
validator. Something like
`RuleFor(f => f.ProductA).AndFor(f => f.ProductB).AndFor(f =>
ProductC).Must(AtLeastOneProduct).WithMessage("Select at least one
product")
private bool AtLeastOneProduct(bool productA, bool productB, bool productC)
{
return productA || productB || productC;
}
`
Which would execute once and product a error for each of the properties.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JeremySkinner/FluentValidation/issues/327, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAFgEn-0S_Xq3FD1uWmYkYgziWNcKYzIks5qpv2wgaJpZM4J8Ebj
.

OK, I've gotten that working. I can see that it's possible to put some extra info in custom state to pass along the the property names from which I would then need to 'raise extra errors but would it be possible to add a new extension, something like RaiseForProperty i.e.
RuleFor(x=>x).Must(AtLeastOneProduct).RaiseForProperty(p => p.ProductA).RaiseForProperty(...
to fluently add other properties to raise the same error for? Then you would not require 3 rules but only one rule. I could see this being of benefit for anywhere that requires complex validation.

That makes sense and now that I've thought more on this I realized how to use the existing extensions for this scenario without needing a custom validation method. It would be advantageous to have an extension method (something like RaiseForProperty(x => x.PropName) ) and be able to chain, to specify another property to raise the error for.

There isn't really a good way to do that at the moment....it's the job of the individual property validators to decide how many ValidationErrors they create, rather than the rule definition doing this. I'll have a think about how best to add an extensibility point that could be used to support this.

I've added a rule against the entire model to handle the user having to have one of two properties populated.
RuleFor(e => e).Must(e => e.PageId.HasValue || !string.IsNullOrEmpty(e.Url)).WithName("Destination").WithMessage("You must select a page or enter a URL for a menu");
How do I run a test against this?
_validator.ShouldHaveValidationErrorFor(e => e, new ContentMenu { Id = 1, Text ="Test", PageId = null, Url = null });
It doesn't like the above.

The ShouldHaveValidationErrorFor method is for testing against properties. As you're using a model level rule you can't use this method- you should execute the Validate method directly and run your assertions on the returne ValidationResult.

Thank you very much, that is exactly what I needed and works a treat :)

@JeremySkinner For model-level rules, how can we add a validation message to the ModelState? When my model-level rule is invoked the ModelState doesn't include anything.

@ciaran036 Should just be a case of adding an entry with a property name of "". If that's not working please create a separate issue that details the problem rather than replying to this issue. Thanks.


Introduce a cop to enforce method calls without parentheses

I work for a company that prefers to avoid the parentheses in method calls. I'd like to have this check as part of our automated style enforcement.
Rubocop already has cops for enforcing the presence or absence of parentheses in method calls without arguments. This cop is trying to work for method calls with arguments. The cops may be suitable for eventual unification, but for now, the new cop is on its own.
The cop is called Style/MethodCallWithoutParentheses, but the name may be up for debate. It has two optional styles:

AllowParenthesesInMultilineCall
AllowParenthesesInChaining

Hopefully, they are self-explanatory. Their names are also up for debate. I may eventually add special-cases for:

method *splat_args to method(*splat_args)
method &pass_block to method(&pass_block)

EDIT:
After a discussion with @bbatsov, I have put this functionality in a new supported style for: Style/MethodCallWithArgsParentheses. The stle is called require_no_parentheses and can be enabled by setting: EnforcedStyle: require_no_parentheses.

Overall you're on the right track, but this should certainly be a configuration style of the existing cop, instead of a new cop.

Should a cop like this apply only to the final method in a method chain? You can technically write something like foo bar .baz, but it's so weird that I don't think even in your company you'd want to do this. I also find multiline calls without parens pretty weird, that at least they are easier to read.

@bbatsov I'm fine with allowing the parens in multiline calls by default. I'll move the code to Style/MethodCallWithoutArgsParentheses. The cop name is a bit ambiguous as it currently implies that it affects only method calls without arguments. The ambiguity is actually helpful here as I'll turn it on for method with or without arguments. 😅

I guess you actually mean https://github.com/rubocop-hq/rubocop/blob/master/lib/rubocop/cop/style/method_call_with_args_parentheses.rb, right?

Nope, I was confused where to put it. 🤦♂ I do get you now, though, and will inline the current cop's functionality as an opposing supported style to the Style/MethodCallWithArgsParentheses cop.

Looks good! Thanks, @gsamokovarov!
P.S. The beers are on your for the extra fast turnaround for family & friends! 😉

Thanks, @bbatsov! Ping me next time you are in Sofia and it all can be arranged! 🙇♂


Rename getChildBySpriteName to getChildByFilename

Let me know if there is anything that needs to be changed

why the rename?

It gets referred to a filename a lot in cocos.

imo old name is fine, and this breaks api and abi so not very nice!

You could've not renamed it but clone it and then rename it. The old one would just be marked as deprecated so it doesn't break the api.


Add windows and cross compile builds

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge/reaper this should be good for review now!

@conda-forge/core pining core team for review as well, since this isn't a very active feedstock

@h-vetinari thanks for the comments, cleaned up the conda forge config, my main issue is that there's not a clear section in the knowledge base for specifically enabling other packages, there's Configuring conda-forge.yml, but it's autogenerated or various blog posts that are no longer relevant but  come up during search, and there's a slew of options that are redundant but documented without a best practices. It might be worthwhile to have a dedicated up-to-date section for different build platforms in the contributing/maintaining packages.

@conda-forge/core would it be possible to merge this in?

Sorry, hadn't seen that you're not a maintainer yet.

Awesome, thanks!


Fix the broken index page of purpureum genome

Its failing because gene cannot be walked up from transcript. So, need to create feature relationships between all genes and transcripts.

Added data patch

Actually there was relationship between gene and transcript.
The issue was that walking up from polypeptide to transcript was going to the wrong gene model which wasn't linked to gene.


[$500] Fix style regressions caused by new Global Nav component

This issue handles all regressions stemmed from the addition if the new Global Nav component in this PR. Any proposals should handle all these regressions. Please refer to the screenshots attached for the new design requirements to solve these issues.
cc: @allroundexperts

Sign in button on Global Nav is cropped (the original issue)

Avatar on Global Nav is cropped

Avatar on Global Nav is very close to the native icons on desktop

Settings Background for profile details Header is changed unintentionally while refactoring LHN

Sign in banner for anonymous user don't have a highlighted green background color

the sidebar has a fixed width which is causing this issue, giving a variable width to the sidebar relative to the button width will solve this issue, even in the future if the button text changes or the font size increases, it won't cause a problem. #simpleCSSHacks

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0135bbe272a370da40

Proposal
Please re-state the problem that we are trying to solve in this issue
Login - Sign in button for anonymous user on New LHN Nav is not fitting
What is the root cause of that problem?
Sign in Button has a width of 82 px while the globalNavigation container has width of 70 px so the sign in button does not fit in container
What changes do you think we should make in order to solve the problem?
We should make the width for sign in button 95% of the container here
width: '95%' https://github.com/Expensify/App/blob/389d7b0c4b96c6f2a2295055c685791d35b65252/src/styles/styles.js#L2296-L2298
What alternative solutions did you explore? (Optional)
NA

🚨 Can you combine this issue with this issue?

melvin, please remove the overdue label.

assigning myself as this is a regression from a wave 8 issue

Design discussion is happening in this App issue

Please see the OP. This issue combined this issue.
Please hold your proposals until the design decision is made ⬆

OP was changed. Please update your proposals to reflect the new requirements 🙇

Proposal
Please re-state the problem that we are trying to solve in this issue.
Fix style regressions caused by new Global Nav component
What is the root cause of that problem?

Sign in button on Global Nav is cropped

Previously we have the Signin button at the top, now we need to replace it with the expensify logo. https://github.com/Expensify/App/blob/0cbee2247e48c17eec428b989afead3dc6f196d4/src/pages/home/sidebar/SignInOrAvatarWithOptionalStatus.js#L40 in this file we need to make changes and remove Signin Button also remove the component as at this only file.

Avatar on Global Nav is cropped

Avatar on glbal nav is only cropped because of the width issue which only happens if we have custom status.

Avatar on Global Nav is very close to the native icons on desktop

Previously we used to have same bg color on the header so it won't be visible. After added the new bg over the global we have different bg colors.
What changes do you think we should make in order to solve the problem?

Update SignInButton https://github.com/Expensify/App/blob/0cbee2247e48c17eec428b989afead3dc6f196d4/src/pages/home/sidebar/SignInOrAvatarWithOptionalStatus.js in this file to use Expensify logo.

Use position property to update the styling of the custom status.

Update https://github.com/Expensify/App/blob/d916effd3ffb2f29adcd3cfad1e314f210a23efa/src/CONST.ts#L885 this line with the height (26) is tested. more on here https://github.com/Expensify/App/issues/29162#issuecomment-1754788318

What alternative solutions did you explore? (Optional)
NA

@shawnborton this is the follow up issue for the PR that added the new Global Nav component

We decided that an engineer from an expert agency is going to work on this issue.
Thank you for all contributors who shared detailed proposals here. We really appreciate the time you took to write these 🙇

@0xmiroslav you don't have to review proposals as an expert agency is going to work on it. We gonna keep you assigned to this issue so that you can review the PR when it's open 🙇

Added this issue to the OP.

@kosmydel @robertKozik I heard you guys are interested in fixing these issues?

@kosmydel @robertKozik could you also take a look at this issue?

omg the style system of App is just too intertwined...
@kosmydel @robertKozik ⬆

@kosmydel @robertKozik could you check if your fix will handle this regression, too?

@hayata-suenaga, I believe we didn't include this regression because we didn't notice it before. However, we can easily add it to the PR to keep all the fixes in the same place.

Should we proceed with the fix inside our PR as in my opinion it would be better to keep all related fixes inside one PR?

Yes let's handle in your PR 👍
We can close this issue if no payment is necessary (@robertKozik is a SWM engineeer)

The PR was closed with implementation
Payment summary
C+ Review payment for @situchan

@situchan could you check the above comment ⬆ ?

Maybe we can pay @ishpaul777 one-time bonus ($50) for all these kinds of reports as they are regressions from one PR, if closed issues also apply bonus.
@zanyrenney will decide.

@zanyrenney could you handle the payment according to this suggestion?

if the payment has been completed, we can close this issue 🙇

@hayata-suenaga can you please remove Reviewing which prevents adding Overdue? This way, @zanyrenney will be bumped every few days

I'm closing this issue. Please comment here if we need payments for this issue.

I noticed that @zanyrenney's slack status indicates OOO until Wed

payment summary - @ishpaul777 $50 for regression reports.
As per this comment, closing.

Paid @situchan through bonus due to problem with hourly contract.

Paid @situchan via a bonus as I messed up the orignal offer and made it hourly. Closing!


Bau improve api test cri calls

Proposed changes
What changed
Extended CRI calling steps in API tests to check that all extra properties sent in the JAR to the CRI are specified in the test.
Why did it change
To ensure that we don't send anything unexpected to the CRIs and to explicitly document the values that we do send.

I think on the whole this is a net positive, but worth noting that this does make our tests a bit more verbose and bit more brittle (in the cases where you aren't too fussed about the 'extra' properties).
E.g. at some point in the future we might start including evidence_requested on all CRI requests, which would add a lot of noise.
Was there a bug you've seen that this would have prevented?

As discussed, this wasn't in response to a particular bug, just to make our interface with CRI's more explicit.
It did highlight the strength = 0 requests to F2F which while not wrong were surprising to me.


Convert existing doc to How to

Converted existing doc to How to format.

Original author, Casey
Can merge at any time

Deploy preview for calico ready!
Built with commit a12ef18ff7a6ce3ad5c9ec51517a534a6a48524b
https://deploy-preview-2907--calico.netlify.com

@bmckercher123 looks like there are conflicts in this PR now and it needs a rebase.


Update dependencies

This is an attempt to consolidate all the PRs that dependabot and snykbot have submitted recently.
If we don't merge the PRs they submit quickly, later PRs end up conflicting with them.
In this PR I'm hoping that our issues with Snyk will also be resolved, which should fix our CI pipeline.

install-example-dependencies phase is failing? do we need to do something about this?

Yep, I'm working on it at the moment. I had issues accessing the CircleCI organisation earlier, but I'm able to access it now so I'll look into this problem.

Ah yea I just messaged you on Reddit, it appears they require you to login, but if you're logged in (from any account) you can view it.

We're still blocked by snyk/snyk#1800 unfortunately.
I'm thinking we might be able to resolve this issue by restructuring the repository away from nested projects and into something closer to a monorepo structure. So the example projects would reside in directories alongside the gatsby-plugin-s3 folder, not contained within it. Any thoughts on this?
The current nested approach is also the reason why we have to run npm ci twice on the main package in CircleCI.

I have a couple weeks off so I'm interested in researching this.
You generally can't accept all that much contribution from my side on this project anymore but I do believe having the CI working is crucial to it's survival, so I'm fine with spending time on that over the next couple of days.

Hello @jariz and @JoshuaWalsh friends, any news on progress about this?
Context

Upon migrating Gatsby from ^3.3.1 to ^4.1.0, we noticed a number of security vulnerabilities in some dependencies, most of them pointing back to gatsby-plugin-s3.
Under the enthusiasm rush, I initially forked this repo to start fixing at the source, but then noticed this PR. ☺
Any ETA on this? Otherwise what's a good way I can contribute to fixing this besides what you already have here?

Big thanks in advance! ✌🏼

Thanks for your interest.
The issue we have at the moment is that Snyk is failing, which prevents our test suite from running. Unfortunately none of us have the time to properly investigate the issue.
I have reported the issue to Snyk, but did not receive any reply.
I suspect the issue might be caused by our nested npm projects. (I also suspect this is the reason why we have to run npm ci 3 times in our CI.) So my next plan was to try restructuring the repo into a more traditional monorepo to see if that fixed the issue. I just haven't had time to do it yet.

@JoshuaWalsh thanks for shedding more light into this!

Unfortunately none of us have the time to properly investigate the issue.

If time is the issue, I'm open to contributing some time.
At this point thought, it seems that even the time required to rebuild context would be rather significant on my end. If either of y'all are willing to break this down a bit, perhaps I can help. Up to you. 🤷🏼♂

At this point thought, it seems that even the time required to rebuild context would be rather significant on my end. If either of y'all are willing to break this down a bit, perhaps I can help. Up to you. 🤷🏼♂

Unfortunately I'm in the same boat. It's been months since I last looked at this.
I've had a quick peek just now and there seem to be multiple issues preventing us from being able to run our automated tests, or even being able to build the example project. #287 #288 #289
I'd love to fix these issues, but at the moment my work and personal life don't leave me with enough time to give them the attention required.
If you're still interested in taking a look, I think the process would look like this:

Install dependencies for gatsby-plugin-s3: npm ci --dev.

This is failing currently because of #287. I have a fix for it, but I'm not able to merge it because I can't test it. You can find my work so far on this branch.

Install dependencies for example project (used by tests): cd examples/with-redirects && npm ci --dev && cd ../..

At the moment this step fails due #288. You can bypass this by removing the "prepare" script from the examples/with-redirects/package.json file.
But then you'll run into #289. I have no idea what's causing this issue.

Install dependencies for gatsby-plugin-s3 again: npm ci --dev (I think this is required because of the nested project structure)

Follow the instructions in TESTING.md to set up and run the e2e tests.

@JoshuaWalsh thanks for the detailed guidance and sorry for not getting back to you sooner.
Decided with my team that our dependency on Gatsby altogether is rather reduced at this point. We are currently evaluating moving away from Gatsby + its dependencies. Therefore this decreased in priority for the moment. If things change, I'll pick it back up from here. Thanks again!

@JoshuaWalsh Is this PR still relevant? #327 also merges a lot of dependency updates? Eitherway I closed all snyk/dependabot PR's because we'll have to consolidate all this in a single PR regardless.


UNC paths are not supported. Defaulting to Windows directory.

Prerequisites

[x] Can you reproduce the problem in safe mode?
[x] Are you running the latest version of Atom?
[x] Did you check the debugging guide?
[x] Did you check the FAQs on Discuss?
[ ] Are you reporting to the correct repository?
[x] Did you perform a cursory search to see if your bug or enhancement is already reported?

For more information on how to write a good bug report or enhancement request, see the CONTRIBUTING guide.
Description
Heyyy, so getting this error an awful lot when working with projects folders and files from a network drive, very annoying :\
Error: '[network samba drive]' CMD.EXE was started with the above path as the current directory. UNC paths are not supported. Defaulting to Windows directory. Error: '[network samba drive]' CMD.EXE was started with the above path as the current directory. UNC paths are not supported.  Defaulting to Windows directory.     at parameters.exit (C:\Users\Nicholas Carter\.atom\packages\linter-jshint\node_modules\atom-linter\lib\helpers.js:47:27)     at triggerExitCallback (C:\Users\Nicholas Carter\AppData\Local\atom\app-1.9.8\resources\app.asar\src\buffered-process.js:215:47)     at ChildProcess.<anonymous> (C:\Users\Nicholas Carter\AppData\Local\atom\app-1.9.8\resources\app.asar\src\buffered-process.js:237:18)     at emitTwo (events.js:100:13)     at ChildProcess.emit (events.js:185:7)     at Process.ChildProcess._handle.onexit (internal/child_process.js:204:12)

I'm seeing this as well. My documents folder is a network drive, but I've mounted it as H: and am actually launching atom from the H: mount. Atom is somehow still resolving the cwd back to the UNC path.

@lorengordon https://github.com/atom/atom/issues/13104 covers what you describe.
@nicholas-c Closing this in favor of https://github.com/atom/atom/issues/13104 or https://github.com/nodejs/node/issues/5160 depending on if you have spaces in the path or not.

@Ben3eeE thanks, i subscribed to that issue to track the resolution or any suggested workarounds.


Train Problem

Dear Authors,
When I train your code, there is a problem. Which choice should I select?

Hi @AmingWu,
To use wandb, you need to:

create a W&B account.
log into your account in the terminal.

If you prefer not to use wandb, you need to set wandb_project to ""
https://github.com/orrzohar/PROB/blob/09d814353933f8bdfe944c8afd4b57b8416ebc8e/main_open_world.py#L144
also, please uncomment lines 169 and 170 in main_open_world.py
I personally always used wandb, but this should work as well.
Best,
Orr

Hi @AmingWu,
I am going to assume that this issue is resolved.
If you are still unable to run training, please re-open this and let me know.
Best,
Orr


Failed to execute sev_component_test.py with some argument. IndexError: string index out of range.

Failed to execute sev_component_test.py with some argument.
The python(sev_component_test.py) can NOT be executed successfully. Error message : IndexError: string index out of range.
Could you help to ask AMD how to fix it? Thank you.
Note. AMD Turin Dense CPU
Test Procedure:

Enabled BIOS settings as below:
SSME: Enabled
SEV Control : Enabled
SEV-ES ASID Space Limit: 100
SEV-SNP Support : Enabled
Iommu: Enabled
Boot into RHEL9.4 and install below packages

yum install qemu-kvm libvirt* git-lfs pyrhon3-pip git
pip3 install cpuid
pip3 install package

Download sev_componet_test file on github website.
GitHub - AMDEPYC/sev_component_test
Unzip the file and save it.
Start to run sev test with sev_componment_test.

python3 ./sev_component_test/sev_component_test/sev_component_test.py –t sev
=> Passed.

Start to run test to only check for enablement

python3 ./sev_component_test/sev_component_test/sev_component_test.py –e
=> Failed. The python sev_component_test can NOT be executed successfully. IndexError: string index out of range.

Start to run automatic encryption test functionality.

python3 ./sev_component_test/sev_component_test/sev_component_test.py –at
=> Failed. The python sev_component_test can NOT be executed successfully. IndexError: string index out of range.

Try running
"sudo modprobe msr"
and then run it again

Hello this was part of a padding issue, if you don't have all of the MSR components enabled, the python bin function shortens the binary number, causing the index out of range issue.
I fixed it with string padding here:
7c8ff92a1759963b9374ca2abb44559cfc0cb7ad


[fix] [broker] [Namespace-level Geo-Replication] Reject a topic creation if there is a confilct topic on the remote side

Motivation
The steps to describe the issue

There is a partitioned topic on the remote cluster, with partitions 1. And the source cluster does not have this topic.
Enable namespace-level Geo-Replication.
Create the topic with 3 partitions on the source cluster.
The broker just printed an error log "[Error] This topic already exists", and users get a successful response.

Modifications
Reject a topic creation if there is a conflict topic on the remote side
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: x

I think we intentionally overlooked the result of creating a topic for the remote cluster.
This PR may break the user's behavior of creating topic.
According to your description, when using the default broker configuration, and the geo-replication is enabled on the namespace level, the remote cluster will create two non-partitioned topics, tenant/namespace/topic-partition-1 and tenant/namespace/topic-partition-2 by the geo producer. Is it right?
If right, I would suggest adding a topic check before starting the replicator to make sure that the topic is the same between local and remote clusters, if they are the same, start the replicator, otherwise throw a log. You can also add this check in the replicator task.

Rebase master

Rebase master