Stack based navigation throw error when write to userdefault in reducer and use scenePhase environment key

Description
When include @Environment(\.scenePhase) var scenePhase in the root view and use stack based navigation, I cannot write to UserDefaults in child view reducer.
Errors:
A "forEach" at "userdefault_tca/ContentView.swift:19" received an action for a missing element. …

  Action:
    RootFeature.Path.Action.child(.test)

This is generally considered an application logic error, and can happen for a few reasons:

• A parent reducer removed an element with this ID before this reducer ran. This reducer must run before any other reducer removes an element, which ensures that element reducers can handle their actions while their state is still available.

• An in-flight effect emitted this action when state contained no element at this ID. While it may be perfectly reasonable to ignore this action, consider canceling the associated effect before an element is removed, especially if it is a long-living effect.

• This action was sent to the store while its state contained no element at this ID. To fix this make sure that actions for this reducer can only be sent from a view store when its state contains an element at this id. In SwiftUI applications, use "NavigationStackStore".

Checklist

[ ] I have determined whether this bug is also reproducible in a vanilla SwiftUI project.
[ ] If possible, I've reproduced the issue using the main branch of this package.
[X] This issue hasn't been addressed in an existing GitHub issue or discussion.

Expected behavior
I can write to UserDefaults in reducer
Actual behavior
Xcode throw errors
Steps to reproduce
Please see example: https://github.com/Doraemoe/tca-scenephase-userdefault
The Composable Architecture version information
1.3.0
Destination operating system
iOS 17
Xcode version information
Version 15.0.1 (15A507)
Swift Compiler version information
swift-driver version: 1.87.1 Apple Swift version 5.9 (swiftlang-<IP_ADDRESS>.108 clang-15<IP_ADDRESS>)
Target: x86_64-apple-macosx14.0

@Doraemoe Your root app scene is creating a brand new store in its body, so when it detects a scene change your state is replaced with a brand new store with no path, which puts SwiftUI in a bad state. I believe you'd have the same problem in vanilla SwiftUI using an observable object.
The fix is to hold onto your store in your app so that it is never recreated:
@main
struct userdefault_tcaApp: App {
  @Environment(\.scenePhase) var scenePhase
  let store = Store(initialState: RootFeature.State()) {
    RootFeature()
  }

  var body: some Scene {
    WindowGroup {
      ContentView(store: self.store)
    }
  }
}

In general you should not create new models in the body of scenes or views as SwiftUI's observation can recreate them at any time.
Since this isn't a bug with the library, I'm going to convert it to a discussion instead.


Fixed #30619 -- Made runserver --nothreading use single threaded WSGIServer.

Ticket: #30619
Client: Chrome 75.0.3770.100/Firefox 67.0.4 on macOS 10.14.5.
Server: macOS 10.14.5., Python 3.7.3, Django 2.2.3
Running web application with single-threaded wsgi server may stop responding.
This is because Web browser uses multiple connection, and all of them has Connection: keep-alive header by default.
When the first request is finished, wsgi server continue to read the socket first request used because the connection is keep-alive.
So, the second connection is kept waiting without accepted by wsgi server, until the fist connection is closed. But the first connection will not be closed by browser for very long time.

Ruby on Rails, sorry for unrelated comment

Ruby on Rails, sorry for unrelated comment

Ruby on Rails, sorry for unrelated comment

@carltongibson  Thank you for comments.
I modified the code and added a test. Please let me know further enhancement required.

Thank you.
I fixed flake8/isort errors, but Jenkins reports another error.
I cannot understand why tests. generic_relations_regress failing. It works in my local box.
I'll investigate tomorrow.

@carltongibson Thank you for the quick review!


C source files are not allowed when not using cgo: syscall.c

After running go get github.com/a-palchikov/wmi, I receive this error message:
c:\gopath\src\github.com\a-palchikov\wmi\wmi.go:4:2: C source files not allowed when not using cgo: syscall.c
I'm running Windows 7 x64 and go version go1.4.2 windows/amd64.
[Looking at the Go v1.4 release notes] (https://golang.org/doc/go1.4#gocmd), this seems to be because the com-and-go package include a c source file, [syscall.c] (https://code.google.com/p/com-and-go/source/browse/) and attempts to compile it upon installation.

Unless cgo is being used to build the package, the go command now refuses to compile C source files, since the relevant C compilers (6c etc.) are intended to be removed from the installation in some future release. (They are used today only to build part of the runtime.) It is difficult to use them correctly in any case, so any extant uses are likely incorrect, so we have disabled them.

Do you know how to resolve or work around this error?

@MicksMix Hey, sorry for the late reply.
It seems that the com-and-go project has been exported to gihub and the syscall issue fixed in the last commit: https://github.com/tianlin/com-and-go/commit/a2b35ebfc50741ef03b8546a64d956461215d7e4

Thanks! Seems your example program to query Windows services no longer compiles though. Either that or I have a problem with my local configuration.

Never mind...I figured out I needed to change the import on the example to:
import (
    "github.com/tianlin/com-and-go/v2"
    "github.com/a-palchikov/wmi"
    "fmt"
    "log"
    "unsafe"
)

I left off the /v2 for the updated "github.com/tianlin/com-and-go"


Please add support for GPTQ models

I'd love to run this model with VLLM, but I get an error currently due to quantization!
https://huggingface.co/TheBloke/Llama-2-13B-chat-GPTQ

Will this be added to main VLLM package soon?
I get this error now:
llm = LLM(model="rshrott/description-awq-4bit")
RuntimeError: Cannot find any model weights with rshrott/description-awq-4bit

You have to pass in the kwarg for quantization
llm = LLM(model="rirv938/wizard-vicuna-13b-uncensored-awq-4bit-g128", **{'quantization': 'awq'})

Thanks. How do I compile the PR? Is this not currently available for general usage?

I'm trying to do in google colab:
TypeError                                 Traceback (most recent call last)
 in <cell line: 1>()
----> 1 llm = LLM(model="rshrott/description-awq-4bit", **{'quantization': 'awq'})
/usr/local/lib/python3.10/dist-packages/vllm/entrypoints/llm.py in init(self, model, tokenizer, tokenizer_mode, trust_remote_code, tensor_parallel_size, dtype, seed, **kwargs)
54         if "disable_log_stats" not in kwargs:
55             kwargs["disable_log_stats"] = True
---> 56         engine_args = EngineArgs(
57             model=model,
58             tokenizer=tokenizer,
TypeError: EngineArgs.init() got an unexpected keyword argument 'quantization'

Hi @ryanshrott,

If you are using VLLM via LangChain, so, the correct code is as follows.

from langchain.llms import VLLM
model = VLLM(model=model_path, tensor_parallel_size=1, trust_remote_code=True, vllm_kwargs={"quantization": "awq"})

Other case, you use vllm directly, the command is as given
from vllm import LLM
model = LLM(model=model_path, tensor_parallel_size=1, tokenizer=model_path, trust_remote_code=True, quantization='awq')


Release Notes for 1.71.0

[phrase library..[@/libs/dynamic_bitset/ Dynamic Bitset]:]

Enabled hardware-assisted popcount on MSVC. ([github_pr dynamic_bitset 38])
Added support for boost::hash and std::hash. ([github_pr dynamic_bitset 45])
Support copy-initialization with default constructor ([github_pr dynamic_bitset 48])

https://github.com/boostorg/website/pulls


Server with mod gives the "Error Adding Element" error

I'm trying to make a modded Minecraft server and get "Error Adding Element" error when the server finishes loading. Any way to fix this?
latest.log

same,version 2.20

Yep same. Caused severe lag on my server. Went back to 2.1 and it works good now

Came by to say the same thing. It'll cause lag every 5 minutes or so and makes it annoying to play. I also noticed the same issue in single player although not quite as bad as on a server.

We're looking into it right now. We think it is related to #262. Is everyone playing with Quark installed?

Yes I am playing with Quark installed

I also have Quark installed but I did some further testing and was able to recreate the error with the following set of mods:

Forge 36.1.61
Abnormals Core 3.2.0
Endergetic Expansion 3.0.0
Infernal Expansion 2.2.0
JEI <IP_ADDRESS>
The One Probe 3.1.4
Xaero's Minimap 21.15.1
Xaero's World Map 1.16.1

So I think the conflict lies with Endergetic Expansion.

With Quark installed

I have the same issue, but I did some testing, when I was trying to find what mod was making the errors:
The Endergetic Expansion mod usually gives me only one "[Server thread/ERROR] [minecraft/WorldSettingsImport]: Error adding element: Base value out of range: 256 [-10-128]" error while loading up a world, but while I have Infernal Expansion installed, the error happens 4 times instead, each time the game is saved. The error does not happen at all without Endergetic Expansion on the other hand.
There is also an almost identical issue with another mod, valhelsia structures, where without Infernal Expansion installed an error only happens maybe once, but if both mods are installed, the error happens on each world save. (the error is not the same error as the one that happens with Endergetic Expansion, this one says Error adding element: Value 10 outside of range [0:7], instead)

Adding onto the list: betterdefaultbiomes gives warnings whenever this mod (2.2.0) is introduced, the same lag spike as faeldray mentioned happens every 5mins.
Test environment was just betterdefaultbiomes and infernalexpansion. version 2.1.1 works just fine.

Pretty sure this is the issue: https://github.com/infernalexp/Infernal-Expansion/blob/0df577b872303e508a4dff270a47390584396d52/src/main/java/org/infernalstudios/infernalexp/mixin/common/MixinChunkGenerator.java#L29-L32
The ChunkGenerator is being replaced regardless of the config file settings and this is a massive change. Since this effects everything that uses the NoiseChunkGenerator.
I fixed this issue by just importing org.infernalstudios.infernalexp.config.InfernalExpansionConfig.WorldGeneration and adding an if statement to check if the ((boolean) WorldGeneration.REPLACE_NETHER_BIOME_PROVIDER.get()) is true.
    @Inject(at = @At("HEAD"), method = "<clinit>")
    private static void IE_clinit(CallbackInfo ci) {
        if ((boolean) WorldGeneration.REPLACE_NETHER_BIOME_PROVIDER.get()) {
            NoiseChunkGeneratorUtil.useCustomNetherBiomeProvider();
        }
    }

Mentioned Users

@Unusual-Danger
@Theoretical-Enday
@zoyer2
@faeldray
@jodlodi
@iMahco
@wchen1990
@MrJorman
@The-Noble
@LordBoos

Please let me know if using this version of the mod doesn't fix your issue: infernalstudios.org/etc/infernal-expansion-2.2.0+20210829.1.jar

It did indeed @SwanX1, no more 5min-lag spikes!


NadekoBot.Core.Common.ShmartNumber instead of a number value

In pretty much all command responses nadeko says 'NadekoBot.Core.Common.ShmartNumber' instead of a number amount. I noticed that picking doesn't do this though.
I'm using the latest docker image on debian 9. I re-created the bot following the instructions but this issue still happens.

Means your bot is not updated. @shikhir-arora I think he owns the docker image so he needs to update it if it's not.

@Kwoth I don't own the Docker image xD

Oh my bad, @willysunny does

Well anyway, it's not hard to change if this is all:
FROM microsoft/dotnet:sdk
MAINTAINER Poag<EMAIL_ADDRESS>
WORKDIR /opt/

#Install required software
RUN echo "deb http://www.deb-multimedia.org jessie main non-free" | tee /etc/apt/sources.list.d/debian-backports.list \
	&& apt-get update \
	&& apt-get install -y --force-yes deb-multimedia-keyring \
	&& apt-get update \
	&& apt-get install -y git libopus0 opus-tools libopus-dev libsodium-dev ffmpeg

#Download and install stable version of Nadeko
RUN curl -L https://github.com/Kwoth/NadekoBot-BashScript/raw/1.9/nadeko_installer_latest.sh | sh \
	&& curl -L https://github.com/Kwoth/NadekoBot-BashScript/raw/1.9/NadekoARN.sh > nadeko.sh \
	&& chmod 755 nadeko.sh

VOLUME ["/opt"]

CMD ["/opt/nadeko.sh"]

Why not update the official docker image?

Because I wasn't aware of the update, as there hasn't been really much new announcements in the nadeko help channel, so I didn't check the update as often, anyways, the docker images has been updated to 2.16.11 as of this post, and the issue should be resolved.


teleport: 7.0.3 -> 7.1.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/teleport/versions.
meta.description for teleport is: "A SSH CA management suite"
meta.homepage for teleport is: "https://goteleport.com/"
meta.changelog for teleport is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

/nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0/bin/teleport passed the binary check.

/nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0/bin/tctl passed the binary check.

/nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0/bin/tsh passed the binary check.

/nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0/bin/.tsh-wrapped passed the binary check.

4 of 4 passed binary check by having a zero exit code.

0 of 4 passed binary check by having the new version present in output.

found 7.1.0 with grep in /nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0

found 7.1.0 in filename of file in /nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0

directory tree listing: https://gist.github.com/bbf4d76c6ec867eb6cca15ba43aa8708

du listing: https://gist.github.com/0917a5bb90ae0087f61065d30a770bd2

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
teleport

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A teleport https://github.com/r-ryantm/nixpkgs/archive/a84754e68cebc5c6b787d7ce51fbbb6032503705.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0
ls -la /nix/store/yx6f9z7gcxqw5ii3nx2xigh4dlvqzh0g-teleport-7.1.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

teleport

Maintainer pings
cc @sigma @tomberek @freezeboy @kalbasit for testing.

Result of nixpkgs-review pr 137710 run on x86_64-linux 1

1 package built:

teleport


Redesign the Unified Execution Server UI - kie server controller enha…

…ncements

closing this one as https://github.com/droolsjbpm/droolsjbpm-integration/pull/344 includes all the changes from here


Add ability to create a Change Log fragment file

At @oasisprotocol, we use the towncrier tool to automatically assemble the Change Log (an example for the Oasis Core 20.11 release) from the Change Log Fragments when preparing a new release.
We have a CI check that ensures each pull request has a corresponding Change Log fragment and now we need to manually amend each Dependabot's pull request with a Change Log fragment file.
It would be great if Dependabot supported creating a file at a given location that would include the same line as in git commit message's subject.
For example, for the following git commit message:
ci: bump actions/setup-python from v1 to v2.1.3

Bumps [actions/setup-python](https://github.com/actions/setup-python) from v1 to v2.1.3.
- [Release notes](https://github.com/actions/setup-python/releases)
-<EMAIL_ADDRESS>
Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>
it would create a file at .changelog/<PR-number>.internal.md with the following contents:
ci: bump actions/setup-python from v1 to v2.1.3

where the file's location and name template would be configurable.

This doesn't seem like something we'll likely add to dependabot, but you could certainly write a GitHub Action to do this for you rather than manually doing it each time, we do something similar internally for license information. Here's an example as inspiration:
name: Dependabot
on:
  push:
    branches:
      - "dependabot/**"

jobs:
  ci-image:
    name: Change log fragment
    runs-on: ubuntu-latest
    steps:
      - uses: actions/checkout@v2
        with:
          fetch-depth: 1
          ref: ${{ github.event.pull_request.head.ref }}
      - name: Configure git
        run: |
          git config --local user.email "$(git log --pretty='%ae' -1)"
          git config --local user.name "Dependabot[bot]"
          git checkout ${{ github.event.pull_request.head.ref }}
      - name: Do things
        run: |
          # Parse the dependency
          commit_message=$(git show -s --format=%B ${{ github.event.pull_request.head.sha }} | head -1)
          dependency_name=$(sed -nE 's/Bump (.*) from.*/\1/p' <(echo $commit_message))
          # Grab the version and write it to a file.. I don't have any code handy but should be similar to ^
      - name: Commit
        run: |
          git add .changelog/<PR-number>.internal.md
          lockfile_changed=$(git status | grep 'Gemfile_next.lock\|vendor/licenses/bundler/')
          git commit -m "[dependabot skip] Add fragment"
      - name: Push changes back to branch
        run: |
          git push origin ${{ github.event.pull_request.head.ref }}

Handy tip is to include [dependabot skip] in the commit message, so Dependabot will still rebase the PR and not complain that it has been edited. The action will run again after Dependabot force pushes to the branch.
Hope that helps

@jurre, thanks for your suggestion!
I agree that this is probably too niche to be added to Dependabot directly and automating it via a GitHub Action is a great idea. I'll give it a try.

@jurre what are your thoughts on implementing changelog edits with Github Actions after this change from Github?
I've switched to the 'pull_request_target' event because on the 'push' event git push doesn't work anymore (probably due to the permissions change of last februari). This poses a new problem, because the workflow isn't executed when Dependabot commits to the PR.

@jurre would it be relevant for this ticket?

Closing, as recommended above, a GitHub action is the best way to accomplish this.


feature(add-source-plugin) : Configure maven source plugin to attach …

…sources artifacts with kotlin sources

@dgautier this is not required. just build project with -DperformRelease and source jars will be properly created


[ipsec] Enable healthcheck for service

Related:

https://github.com/rancher/rancher/issues/9855
https://github.com/rancher/rancher/issues/10217

Tested with rancher server version - v1.6.13-rc3
ipsec version 0.2.2 with rancher/net:v0.13.6.
ipsec service is now health check enabled.
Following test cases were executed -
Ipsec_health_Check.xlsx
Issue found - https://github.com/rancher/rancher/issues/10584


It is not Unicode friendly

If have Unicode character, the width is confusion.

It has nothing to do with this. It's because of the fonts


module 'pyro' has no attribute 'plate'

When I run lda.py in the example directory, there is an error.
Traceback (most recent call last):
File "lda.py", line 137, in
main(args)
File "lda.py", line 105, in main
true_topic_weights, true_topic_words, data = model(args=args)
File "lda.py", line 35, in model
with pyro.plate("topics", args.num_topics):
AttributeError: module 'pyro' has no attribute 'plate'
What is the problem?
Thanks.

I think you need to use Pyro dev: pip install git+https://github.com/uber/pyro.

all examples need to be run with the dev version of pyro

Thanks! I will try it.

Does pyro 0.3 needs pytorch 1.0?

Does pyro 0.3 need pytorch 1.0?

That's right. We are waiting for pytorch 1.0 release before releasing pyro 0.3, but you should be able to use the pytorch pre-release version with pyro 0.3.


Question: Fetching Certificates for Device

When we need to fetch Certificates for Device , We are using the helper function from CertificateUtils
CertificateUtils.getAllOwnedCertificates(blockchainProperties)

where in blockchainProperties, we need to set the active user, which is a wallet instance, To get this wallet instance we need to pass the private keys
export interface IBlockchainProperties {
    web3: providers.FallbackProvider | providers.JsonRpcProvider;
    registry: Registry;
    issuer: Issuer;
    activeUser?: Signer;
}

So Is there any other way to fetch the Device certificate, without exposing the device private key?

Sure, the activeUser is an optional parameter (hence the question mark in activeUser?).
You should be able to get details about the certificate even without the activeUser property, but you won't be able to trigger actions such as .transfer(), .claim() etc. because they require sending a transaction to the blockchain.

Hi @JosephBagaric many thanks for your reply ..
Agree that, we can get owned Certificate data without pk of active user.
But  for transfer ,
We do need to the Certificate instane
Const [certificate] = await CertificateUtils.getAllOwnedCertificates(blockchainProperties);
this to get this instance we also need to set activeUser like you have mentioned
later this instance are being used for initiate transfer
await certificate.transfer(deviceOwnerWallet.address, totalVolume)
Now, is it possible if we can get this signing part using HCM and then send the signed transaction instead using raw PK.

activeUser is of type Signer, meaning that we support any Ethers.js compatible Signer module (private key, MetaMask, etc.). Doesn't have to be a raw PK.
There's currently no support to send a signed raw transaction to the Certificate facade to execute .transfer() or .claim().
It's only done through the Ethers.js Signer object.
Can you create a custom Signer module for HSM and use that? That should work.

Do we have any direction for creating a custom Signer module and how that module will be integrated into Origin operations

Sure, probably you can just create a TypeScript implementation of the abstract Signer interface from ethers.js.
https://github.com/ethers-io/ethers.js/tree/master/packages/abstract-signer

Thanks @JosephBagaric for your prompt inputs .. Let me explore on this


what epd pins are pre-defined in the magtag board definition

Don't need to set them again if they are pre-defined.

updated definitions based on working code from another MagTag project


Draft: Fix failover beacon nodes being ignored when using --beacon-node-api-endpoints parameter

PR Description
Issue originally raised in discord. I modified the test RemoteValidatorAcceptanceTest.shouldFailoverWhenPrimaryBeaconNodeGoesDown and it fails when the --beacon-node-api-endpoints is used instead of --beacon-node-api-endpoint. Switching the order in the Validator client @Option makes both scenarios pass.
Fixed Issue(s)

Documentation

[ ] I thought about documentation and added the doc-change-required label to this PR if updates are required.

Changelog

[ ] I thought about adding a changelog entry, and added one if I deemed necessary.

@StefanBratanov looks like my guess was wrong - the order of names in @Options does not matter, as the following tests pass. (this is really testing the picocli rather than Teku, but just wanted to confirm)
package tech.pegasys.teku;

import picocli.CommandLine;

import org.junit.jupiter.api.Test;
import tech.pegasys.teku.cli.options.ValidatorClientOptions;

import static org.junit.jupiter.api.Assertions.assertEquals;

public class ValidatorClientOptionsTest {

    @Test
    public void testArgBeaconNodeApiEndpointSingular() {
        String[] args = {
                "--beacon-node-api-endpoint",
                "http://localhost1,http://localhost2"
        };

        ValidatorClientOptions app = new ValidatorClientOptions();
        new CommandLine(app).parseArgs(args);
        assertEquals(2, app.getBeaconNodeApiEndpoints().size());
    }

    @Test
    public void testArgBeaconNodeApiEndpointsPlural() {
        String[] args = {
                "--beacon-node-api-endpoints",
                "http://localhost1,http://localhost2"
        };

        ValidatorClientOptions app = new ValidatorClientOptions();
        new CommandLine(app).parseArgs(args);
        assertEquals(2, app.getBeaconNodeApiEndpoints().size());
    }
}

will revert the change and keep the test to see if the test fails on CI

Will revert the change and keep the test to see if the test fails on CI.
On my machine the acceptance test continues to run after 2+ minutes, but if I switch the order in ValidatorClientOptions, it passes after 39 seconds.

Hey, thanks for looking into it. The reason the test fails is because in the TekuValidatorNode Config constructor we do the following which does not get overwritten if we use beacon-node-api-endpoints (because of priority order) :
configMap.put("beacon-node-api-endpoint", "http://notvalid.restapi.com");
In reality, this wouldn't be a problem because the user will define one of the two properties.

In reality, this wouldn't be a problem because the user will define one of the two properties.

Ah I see, thank you! I will close this now :)


[Feature]: Execute builtin keymaps without remapping

Similar Issues

[X] Before filing, I have searched for similar issues.

Description
Hi!
Currently builtin keymaps are executed w/ remapping. Therefore, it is not possible to remap them without creating an inconsistency with the description. My use case involves mapping (many) builtins to <nop> so I wouldn't trigger them accidentally. However, I would like them to still work when selected in legendary. Imo executing them w/o remapping would be better in general.
diff --git a/lua/legendary/api/executor.lua b/lua/legendary/api/executor.lua
index 85cf6ea..ba17be1 100644
--- a/lua/legendary/api/executor.lua
+++ b/lua/legendary/api/executor.lua
@@ -74,7 +74,7 @@ function M.exec_item(item, context)
           vim.cmd(cmd)
         end
       elseif Toolbox.is_keymap(item) then
-        util.exec_feedkeys(item.keys)
+        util.exec_feedkeys(item.keys, item.builtin)
       elseif Toolbox.is_autocmd(item) then
         local impl = item.implementation
         if type(impl) == 'function' then
diff --git a/lua/legendary/util.lua b/lua/legendary/util.lua
index 8738266..45abfb7 100644
--- a/lua/legendary/util.lua
+++ b/lua/legendary/util.lua
@@ -60,8 +60,12 @@ end
 ---Execute the given keys via `vim.api.nvim_feedkeys`,
 ---`keys` are escaped using `vim.api.nvim_replace_termcodes`
 ---@param keys string
-function M.exec_feedkeys(keys)
-  vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes(keys, true, false, true), 't', true)
+function M.exec_feedkeys(keys, noremap)
+  local mode = 't'
+  if noremap then
+    mode = mode .. 'n'
+  end
+  vim.api.nvim_feedkeys(vim.api.nvim_replace_termcodes(keys, true, false, true), mode, true)
 end

 ---Run the given function, timing it's duration and logging the performance

Imo executing them w/o remapping would be better in general.

Let's just do that. Can you put up a PR?


AO-20158-Package-Install-Tests

This pull request is the last in a series of Pull Requests to set up GitHub Actions (for this repo and the upstream one).
Overview
Pull Request adds a Verify (verify.yml) workflow that runs after the Release workflow has completed.
It provides confidence that a released package can successfully install and auto instrument apps on a wide variety of node and operatiing system combinations.
Workflow will load the Target Group from the bindings repo and for each image in group:

Install  Appoptics agent (either the released version or the one specified manually).
Check that installation of binary completed successfully (either from prebuilt tarball or from source).
Start an instrumented http server to confirm auto instrumentation against staging is working.
Run a simulation http client to generate data on staging backend.
Start an instrumented http server to confirm auto instrumentation against production is working.

The Verify workflow can also be triggered manually on a specific version.
Commits:
First commit (0f4bdf8) adds a simple utility client server duo to GithHub Actions. The server is a simple, Appoptics instrumented, http server. The client makes paced http requests (GET, POST, PATCH, DELETE) to the server resulting in trace data being sent to the Appoptics collector (staging).
Second commit (33965d0) modified both client and server to accommodate various scenarios when running in GitHub Actions. Server emits error if not instrumented. Client emits error if fails to connect to server after set number of trys. Simple it is no more...
Third commit (f748536) added the Verify workflow.
Results
The workflow hits both staging ad production backends for visual inspection
Staging:

Note - clicking in shows 81 traced requests (initial GET + 20 * GET/POST/PATCH/DELETE) for all.
Production:

Note - there are no traced request, only readyToSample check.
Notes:

The client server duo is a private package with own package.json. It has 2 widely used dependencies plus the appoptics agent itself that always gets version specific install. No real maintenance needed unless (far) future fails happen.
Workflow takes about 3:30 minutes to run and 1:15 hours in commutative run time (billable).
The workflow uses workflow_run trigger. It will run after Release completes successfully, but won't run if it fails.
For simplicity, the server of the client server duo, runs on the runner rather than own container.
When running manually, fails should be expected, depending on what stage in a release process we are at. To illustrate: if workflow is ran on appoptics-apm@latest right now all the node 16 non-builder images (e.g node:16-alpine3.11) will fail while node 16 builder images (e.g. node:16-buster) will pass. This is because the prerelease version of the bindings already supports node 16 with prebuilts but the latest published version of the agent uses a prior version of the bindings that does not.

Also, are you concerned that generating traces against the Production environment would cause issues? It should be OK, so we should enable the tracing part against Production too.

Added test against production as per offline discussion.


[Web]: Korean and custom font is not rendering on some devices when "FLUTTER_WEB_USE_SKIA=true"

My app is web app
I used korean and custom font(NotoSansKR)
but some device text cut-off last character lite this

The correct is '사용 후기 등록하기', but some device '사용 후기 등록하',
I think this problem reproduced only at high resolution(Perhaps over FHD)
I tested this list
MacBook Pro (Retina, 15-inch, Mid 2015) -> fine,
MacBook Pro (15-inch, 2017) -> not fine,
galaxy note 10+ -> not fine,
galaxy note 8 -> fine,
Iphone8+ -> fine,
Iphone6 -fine,

and my flutter -v doctor here
[✓] Flutter (Channel master, 1.25.0-4.0.pre, on Mac OS X 10.15.7 19H2 darwin-x64, locale en-KR)
    • Flutter version 1.25.0-4.0.pre at /Users/taz/Documents/flutter_channels/flutter_mastar
    • Framework revision a7f5fd5360 (3 days ago), 2020-11-30 13:14:13 +0100
    • Engine revision 20caf54969
    • Dart version 2.12.0 (build 2.12.0-76.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /Users/taz/Library/Android/sdk
    • Platform android-30, build-tools 29.0.2
    • ANDROID_HOME = /Users/taz/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (2 available)
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 87.0.4280.88

• No issues found!

@taz1208
Tried the text on latest master on web, which rendered properly:

My machine : MacBook Pro (13-inch, 2020, Four Thunderbolt 3 ports)
Can you provide a complete reproducible code sample that shows the issue ?

flutter doctor -v
[✓] Flutter (Channel master, 1.25.0-5.0.pre.45, on Mac OS X 10.15.4 19E2269
    darwin-x64, locale en-IN)
    • Flutter version 1.25.0-5.0.pre.45 at
      /Users/dhs/documents/Fluttersdk/flutter
    • Framework revision 81e1f7d1ed (6 hours ago), 2020-12-02 17:19:57 -0800
    • Engine revision 20caf54969
    • Dart version 2.12.0 (build 2.12.0-76.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/dhs/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.0.1, Build version 12A7300
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 46.0.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (4 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11
      (API 30) (emulator)
    • macOS (desktop)             • macos         • darwin-x64     • Mac OS X
      10.15.4 19E2269 darwin-x64
    • Web Server (web)            • web-server    • web-javascript • Flutter
      Tools
    • Chrome (web)                • chrome        • web-javascript • Google
      Chrome 87.0.4280.88

• No issues found!

My code snippet here
void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'TEXT BUGGY',
      theme: ThemeData(
        primarySwatch: Colors.blue,
        visualDensity: VisualDensity.adaptivePlatformDensity,
      ),
      home: Scaffold(
        body: LayoutBuilder(
          builder: (context, constraints) => SubmitButton(
            isMobile: constraints.maxWidth > 800,
          ),
        ),
      ),
    );
  }
}

class SubmitButton extends StatelessWidget {
  final bool isMobile;
  const SubmitButton({
    Key key,
    this.isMobile = false,
  }) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      child: Container(
        margin: const EdgeInsets.only(top: 100),
        width: isMobile ? 240 : 337,
        height: 52,
        alignment: Alignment.center,
        decoration: BoxDecoration(
          borderRadius: BorderRadius.all(Radius.circular(5)),
          color: Theme.of(context).accentColor,
        ),
        child: Text(
          '사용 후기 등록하기',
          style: TextStyle(
            fontFamily: 'NotoSansKR',
            fontSize: 14,
            fontWeight: FontWeight.w500,
            color: Color(0xff303030),
          ),
        ),
      ),
    );
  }
}

and pubspect setup here
fonts:
    - family: NotoSansKR
      fonts:
        - asset: fonts/NotoSansKR-Regular.otf

font is download from https://fonts.google.com/specimen/Noto+Sans+KR?query=noto
I found another clue.
The problem only occurs when applying the font.

@taz1208
Tried your code sample with the font, but didn't see the rendering issue. Also tried the same on Pixel 3XL emulator and iPhone SE simulator (14) and saw no issues.

Are you consistently seeing the rendering issue ?
I don't have the said machine MacBook Pro (15-inch, 2017) on which you said the rendering has issue, so can't really replicate.

@darshankawar
Yes, it still  same problems at same devices
My step is

flutter build web
deploy to my webserver(use golang in my case)
And then access to web from all device web browser

I tried same browser (Chrome) on all devices
Did you test in real device?
The problem is not showing in mobile mode in Chroem developer tools.
I think the each devices different only resolution (but maybe it's barking up the wrong tree)

This is working fine on Mac (MacBook Pro (Retina, 15-inch, Mid 2015))

and
this is not fine on Galaxy note 10+

and resolution of Galaxy note 10+

@taz1208
I tried flutter build web and used localhost to verify on Web chrome which worked well and iphone SE simulator which also worked well:

Can you also check by trying flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_SKIA=true and see if you get same issue ?

@darshankawar
It's not working fine form this command flutter run -d chrome on MacBook Pro (15-inch, 2017)
But it working fine when use the command flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_SKIA=true
How can I fix when deploy webserver?

additional
.svg not showing when use the command flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_SKIA=true
pacakge : https://pub.dev/packages/websafe_svg

@taz1208
Regarding .svg issue, you may need to open issue in the said package and follow-up there for updates.

flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_SKIA=true

flutter run -d chrome

Tail icon is .svg

flutter run -d chrome --release --dart-define=FLUTTER_WEB_USE_SKIA=true

flutter run -d chrome

Tail icon is .svg

@taz1208
the svg are not supported on web yet. Check this open issue, https://github.com/dnfield/flutter_svg/issues/173

Keeping this issue open for further analysis, since I am unable to replicate this issue on the devices I tried, per https://github.com/flutter/flutter/issues/71627#issuecomment-738605612 and https://github.com/flutter/flutter/issues/71627#issuecomment-737863568

@darshankawar Thank you for your guidance.
I'm opened issue here : https://github.com/peiffer-innovations/websafe_svg/issues/15#issue-759912120

It seems the original issue (Korean text rendering dropping last character) is not a problem in CanvasKit according to https://github.com/flutter/flutter/issues/71627#issuecomment-740309659
Removing the CanvasKit label.

@taz1208 is there any non-canvaskit issue to be reported here?

@taz1208 is there any non-canvaskit issue to be reported here?

hmm.. It's iffy situation.
same SDK version, same code.
It only the machine is different.
As I mentioned before
MacBook Pro (Retina, 15-inch, Mid 2015) -> fine,
MacBook Pro (15-inch, 2017) -> not fine,
galaxy note 10+ -> not fine,
galaxy note 8 -> fine,
Iphone8+ -> fine,
Iphone6 -fine,

Usually only problem in the latest models.

hmm.. It's iffy situation.
same SDK version, same code.
It only the machine is different.
As I mentioned before
MacBook Pro (Retina, 15-inch, Mid 2015) -> fine,
MacBook Pro (15-inch, 2017) -> not fine,
galaxy note 10+ -> not fine,
galaxy note 8 -> fine,
Iphone8+ -> fine,
Iphone6 -fine,

Usually only problem in the latest models.

It's very strange..:(
It working differently on the same widget structure on the same device
For example.
my widget like this
Column(
   children:[
      Text('안녕하세요'),
      Text('반갑습니다'),
   ]
);

First child works fine like '안녕하세요',
but second child not works fine like '반갑습니' (last character cut-off)

It's very strange..:(
It working differently on the same widget structure on the same device
For example.
my widget like this
Column(
   children:[
      Text('안녕하세요'),
      Text('반갑습니다'),
   ]
);

First child works fine like '안녕하세요',
but second child not works fine like '반갑습니' (last character cut-off)

Can you please try this on the latest master channel? This was fixed recently.

Can you please try this on the latest master channel? This was fixed recently.

I tried this version about 5days ago
 Flutter (Channel master, 1.26.0-2.0.pre.177, on macOS 11.1 20C69 darwin-x64, locale en-KR)
    • Flutter version 1.26.0-2.0.pre.177 at /Users/ylabs/Documents/flutter_channels/flutter_mastar
    • Framework revision 7c26c8fbb1 (3 weeks ago), 2021-01-05 01:19:03 -0500
    • Engine revision b9dc0cf1d8
    • Dart version 2.12.0 (build 2.12.0-179.0.dev)

It still problem

I tried this version about 5days ago
 Flutter (Channel master, 1.26.0-2.0.pre.177, on macOS 11.1 20C69 darwin-x64, locale en-KR)
    • Flutter version 1.26.0-2.0.pre.177 at /Users/ylabs/Documents/flutter_channels/flutter_mastar
    • Framework revision 7c26c8fbb1 (3 weeks ago), 2021-01-05 01:19:03 -0500
    • Engine revision b9dc0cf1d8
    • Dart version 2.12.0 (build 2.12.0-179.0.dev)

It still problem

@taz1208 could you upgrade the master channel? Yours is 3 weeks old.

@taz1208 could you upgrade the master channel? Yours is 3 weeks old.

Awesome, It very cool
When I upgrade SDK, It solved all problem
Thanks every one!

Awesome, It very cool
When I upgrade SDK, It solved all problem
Thanks every one!


minimum centos doesn't have 'tar' command

Then it is needed to install 'tar' package.
I guess I can do simply insert following line to here.
pkgs += %w(tar) if platform?('centos')

When you say minimum CentOS what version do you mean?

@burtlo: I've had this occur today on a minimal version of CentOS 6.7.

I saw this during a Test Kitchen run, using a Docker image of CentOS 6.7. The relevant part of the stack trace is here: https://gist.github.com/sgerrand/b0fa15c432d1c9f1944c

This is what #61 was trying to fix - I'm going to have a working fix for this later today I think.

This is fixed in #126 - I'll close the issue when 0.9.1 is released

This has been fixed in v1.0.0


Feature: Add Project benchmark

Task

Add Data and support file.
Add a benchmark notebook from here.

After we close this issue, we could finish our first project Project Proposal and move on our next project. Just want to notify you @troyxiao @dy11  .


Clean cluster destruction

Originally created by Tom Smyth at 2017-09-30 19:54, migrated from redmine-#7239

Status changed from New to Closed

By Tom Smyth at 2017-10-01 01:50


vsce package error

Following README instructions, I get the below when running vsce package:
➜  lurk-vscode git:(main) vsce package
Executing prepublish script 'npm run vscode:prepublish'...

><EMAIL_ADDRESS>vscode:prepublish
> npm run compile

><EMAIL_ADDRESS>compile
> tsc -p ./

src/extension.ts:3:25 - error TS2307: Cannot find module 'vscode' or its corresponding type declarations.

3 import * as vscode from 'vscode';
                          ~~~~~~~~

src/extension.ts:7:50 - error TS2304: Cannot find name 'setTimeout'.

7 const delay = (ms: number) => new Promise(res => setTimeout(res, ms));
                                                   ~~~~~~~~~~

src/extension.ts:24:13 - error TS2304: Cannot find name 'setTimeout'.

24             setTimeout(queueLoop, waitTime!);
               ~~~~~~~~~~

src/extension.ts:37:9 - error TS2304: Cannot find name 'setTimeout'.

37         setTimeout(queueLoop, 100);
           ~~~~~~~~~~

src/extension.ts:82:13 - error TS2584: Cannot find name 'console'. Do you need to change your target library? Try changing the 'lib' compiler option to include 'dom'.

82             console.log("Sending to lurk REPL: %s", expr);
               ~~~~~~~

Found 5 errors in the same file, starting at: src/extension.ts:3

 ERROR  npm failed with exit code 2

The instructions have been updated and I think it solves this issue. If you still see it, please reopen.


org.jruby/jruby 1.7.0

Type: Incorrect
Summary:
org.jruby/jruby 1.7.0
Details:
Not sure where the tooling is pulling LGPL-3.0.  The LICENSE and NOTICE both reference Apache-2.0 as the license.
Resolution:
Declared = Apache-2.0
Affected definitions:

jruby 1.7.0

@capfei - we also located the source repo, and that says Jruby is tri-licensed - https://github.com/jruby/jruby/blob/1.7.0/COPYING.  It may be that the Apache LICENSE/NOTICE files in the package are actually only applicable to a portion of the package.  What are your thoughts on this one?

@ariel11 I revisited this one after thinking about it some more and realized that I wasn't able to locate this license.txt. Reading through issues, I think that the Apache license was being included in error (seen this kind of issue before).
I think this should be corrected to the tri-license listed in the source repo and in the sources.jar.
CPL-1.0 OR GPL-2.0-or-later OR LGPL-2.1-or-later If you agree, I will correct this one.


Selected cell doesn't always have a superview (IGListCollectionView).

I am working on interactive transition to detail view when user clicks on one of the cells.
I am using didSelectItem(at index: Int) function to check when the user clicks on the cell.
I then get the cell they selected:
guard let cell = self.cellForItem(at: index) as? DesignInspirationCell else {
            return
}

In the details view I need to get original frame of the cell and convert it into new coordinate system.
When I try to get cells superview it sometimes comes back as nil, and other times as IGListCollectionView.
Wondering why this is the case?
Here is the bit of code that calculates the originFrame:
 func calcOriginFrame(_ sender: UIView) -> CGRect {
        if let senderViewOriginalFrameTemp = sender.superview?.convert(sender.frame, to:nil) {
            return senderViewOriginalFrameTemp
        } else if let senderViewOriginalFrameTemp = sender.layer.superlayer?.convert(sender.frame, to: nil) {
            return senderViewOriginalFrameTemp
        } else {
            return .zero
        }
}

Because sometimes the sender.superview is nil and same for the superlayer case the frame ends up being zero, this results in weird animation. Wondering if there is something else I should be doing here.
Any help will be appreciated.

[X ] I have reviewed the README and documentation
[ X] I have searched existing issues and this is not a duplicate

General information

IGListKit version:  2.1.0
iOS version(s): 10.1
CocoaPods/Carthage version:
Xcode version: 8.2.1
Devices/Simulators affected: All
Reproducible in the demo project? (Yes/No): Not Tested

hey @talkaboutdesign, i think this is due to the nature of how collection views store cells for reuse. if you take a look at the implementation of the single section controller you'll notice that it ends up simply calling one of the dequeueReusableCell... methods. depending on how much scrolling has occurred, the cell that you dequeue may not have yet been added to the collection view. this is why you're seeing the behavior where sometimes the cell has a superview and other times it does not.
the method you're using will grab a cell from the reuse pool, potentially creating a new one, but in your case you don't really want a new cell, you just want the frame of an existing cell. one possible solution would be to use the visibleCellsForSectionController method in the selection callback. you could filter this array for the cell with the index you care about, and since it is visible it should be attached to the view hierarchy.
there might be a way to query the underlying layout for the cell's layout attributes directly, but i'm not sure if it is exposed via the list kit interfaces.

@jamieQ is spot on. @talkaboutdesign you're probably dequeuing another cell. Instead you want to do:
guard let cell = self.collectionContext?.cellForItem(at: index, sectionController: self) as? DesignInspirationCell else {
  return
}

This will query the underlying UICollectionView for a cell that is already in the hierarchy, which will have a superview.
If this solves your problem feel free to close this issue!

Here is a gif i recorded with the behavior. Sometimes the animation is correct, others it has no idea where the cell is and the preview comes from the bottom of the screen ignoring the position of the cell.
http://www.giphy.com/gifs/l44QEjtdPMOrBbSgM

@talkaboutdesign have you tried changing the API you use to fetch the cell that we mentioned above?

This is how I get the cell now.
 func didSelect(_ sectionController: IGListSingleSectionController, with object: Any) {
guard let cell = sectionController.cellForItem(at: 0) as? DesignInspirationCell else {
            return
        }
}

@talkaboutdesign that's still using the same API that will dequeue a new cell. You want to use one of the methods that gets a cell directly from the collection view. See:

Our API to get a cell within a section controller
Or the UICollectionView API

Using UICollectionView API I see the same results.
        guard let cell = self.collectionView.cellForItem(at: [0]) as? DesignInspirationCell else {
            return
        }

And your API method returns an error in my code:

😕  its a little tough to tell why you're getting a compiler error w/out the scope of the code. Are you using that from within the section controller's didSelect(...) function?
Your code above looks like its using IGListSingleSectionController, but the compiler error shows SingleSectionViewController, so self would be the view controller, no? You can only access collectionContext from a section controller, not a UIViewController.

 Was debugging wrong ViewController. This fixed it:
guard let cell = self.collectionContext?.cellForItem(at: index, sectionController: self) as? DesignInspirationCell else {
  return
}


Add mp-values

Hi! Here is the PR to add the mp-value computation per perturbation (replicates are merged as described in the original paper). I tried to stay as close as possible to the original definition for now but a lot more can be tweaked if needed. Hopefully this integrates correctly with the other operations but I could actually not reuse so many of the utils methods as the mp-value is not correlation-based but relies on the Mahalanobis distance. Concretely this means similarity_melted_df is not needed for the computation of the mp-value.
Limitations:

This might still need some linting
In cytominer_eval.evaluate it would be possible to skip the computation of the melted data frame when operation == "mp_value"
I do most of my testing in Jupyter notebooks so I hope I didn't add mistakes when migrating the code to the corresponding files.
In the methods section of the paper they state "To find S, the covariance matrices are first calculated separately for the treatment and vehicle subsets of P, and then each resulting covariance matrix is weighted by the fraction of total samples in P that it contains. The weighted matrices are summed to produce S." which I did not do for now. This should not affect the result too much anyway.
The Mahalanobis distance is implemented in cytominer_eval.operations.util.calculate_mahalanobis. This could be implemented as separate operation (either in this PR or in another one). I don't think this would make sense to implement directly as a metric as this is not design for pairwise distances.

Cheers.

@koalive - I literally just saw this now! Somehow this notification got mixed up somewhere. I will review ASAP. Thank you

@koalive - I literally just saw this now! Somehow this notification got mixed up somewhere. I will review ASAP. Thank you

One quick thing as I review the remainder of the files:

This might still need some linting

Can you run black and then add the updated files? This automates the linting process and all other scripts have been adjusted

One quick thing as I review the remainder of the files:

This might still need some linting

Can you run black and then add the updated files? This automates the linting process and all other scripts have been adjusted

Hi Greg,
Happy new year to you too!
No worries, thanks for reviewing my PR. With these new commits, I think I addressed your comments and I'll reply to some of them with more details. Overall, regarding the interpretability of these values in comparison to distances, maybe we could simply use the -log10 of the mp-value?
Let me know what you think!

Hi Greg,
Happy new year to you too!
No worries, thanks for reviewing my PR. With these new commits, I think I addressed your comments and I'll reply to some of them with more details. Overall, regarding the interpretability of these values in comparison to distances, maybe we could simply use the -log10 of the mp-value?
Let me know what you think!

Codecov Report

Merging #26 (3a39362) into master (6f9d350) will decrease coverage by 0.03%.
The diff coverage is 99.15%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   99.35%   99.31%   -0.04%
==========================================
  Files          19       21       +2
  Lines         616      726     +110
==========================================
+ Hits          612      721     +109
- Misses          4        5       +1

Flag
Coverage Δ

unittests
99.31% <99.15%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cytominer_eval/operations/util.py
98.83% <97.29%> (-1.17%)
:arrow_down:

cytominer_eval/evaluate.py
100.00% <100.00%> (ø)

cytominer_eval/operations/__init__.py
100.00% <100.00%> (ø)

cytominer_eval/operations/mp_value.py
100.00% <100.00%> (ø)

cytominer_eval/tests/test_evaluate.py
100.00% <100.00%> (ø)

...ominer_eval/tests/test_operations/test_mp_value.py
100.00% <100.00%> (ø)

cytominer_eval/tests/test_transform/test_util.py
100.00% <100.00%> (ø)

cytominer_eval/transform/util.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #26 (3a39362) into master (6f9d350) will decrease coverage by 0.03%.
The diff coverage is 99.15%.

@@            Coverage Diff             @@
##           master      #26      +/-   ##
==========================================
- Coverage   99.35%   99.31%   -0.04%
==========================================
  Files          19       21       +2
  Lines         616      726     +110
==========================================
+ Hits          612      721     +109
- Misses          4        5       +1

Flag
Coverage Δ

unittests
99.31% <99.15%> (-0.04%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

cytominer_eval/operations/util.py
98.83% <97.29%> (-1.17%)
:arrow_down:

cytominer_eval/evaluate.py
100.00% <100.00%> (ø)

cytominer_eval/operations/__init__.py
100.00% <100.00%> (ø)

cytominer_eval/operations/mp_value.py
100.00% <100.00%> (ø)

cytominer_eval/tests/test_evaluate.py
100.00% <100.00%> (ø)

...ominer_eval/tests/test_operations/test_mp_value.py
100.00% <100.00%> (ø)

cytominer_eval/tests/test_transform/test_util.py
100.00% <100.00%> (ø)

cytominer_eval/transform/util.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Good catch, I added the parametrization of the mp-value computations at the mp-value and evaluate stages as well and tested for incorrect parameter names in all cases. If I'm not missing anything, this should be ready for merging! 👍

Good catch, I added the parametrization of the mp-value computations at the mp-value and evaluate stages as well and tested for incorrect parameter names in all cases. If I'm not missing anything, this should be ready for merging! 👍

I'm going to merge. Thanks again @koalive!! I am eager to collaborate in the future, and will keep you posted about manuscript plans

I'm going to merge. Thanks again @koalive!! I am eager to collaborate in the future, and will keep you posted about manuscript plans


CodePipeline : display different stages & actions of pipelines

display different stages & actions of pipelines. As of now only pipeline status is coming in widget.

We have started a new project to better scale our efforts regarding Backstage plugins. The functionality in this repository is being migrated. If this issue is still relevant please create a new issue in this repository:
https://github.com/awslabs/backstage-plugins-for-aws


table: multi-select

[x] need a design
[ ] what are the requirements

requirements were figured and worked by sweeting and dane. they are in PR. Uses react-table. This card is worthless.


Add support for dollar-quoted strings

Summary of the changes / Why this improves CrateDB
Resolves #11512.
Add support for dollar quoted strings that cannot handle nested dollar quoting yet.
This is not part of SQL standard, no need to update sql_features.
Referenced:  https://github.com/antlr/grammars-v4/tree/master/sql/postgresql
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Question: The current implementation unquotes the dollar quoting in the very beginning while parsing the statement. Are there any scenarios that we need the original form of the string again?

retest this please
unrelated: Could not resolve com.github.jk1:gradle-license-report:2.1

retest this please
unrelated : testTransientErrorsDuringRecoveryAreRetried


Incompatibility with earlier versions of minitest

In earlier version of minitest, Minitest.after_run isn't available. Instead, the method for doing that was Minitest::Unit.after_tests. I will submit a PR soon with a fix for this.

Is there a good way to test this?

Right now I don't have a good idea how to test this across two minitest versions.
In rspec adapter I did similar workaround https://github.com/ArturT/knapsack/blob/master/lib/knapsack/adapters/rspec_adapter.rb#L10,L15
Please create PR then we will figure out something.
Thanks!

Fixed in the PR https://github.com/ArturT/knapsack/pull/26


Py-libp2p broken link.

Hey everyone. I was just starting to look around because I want to contribute to this project. I read that an implementation of py-libp2p should be one of the first step but currently the link found in the roadmap is broken.
As any work been done yet and if so where is it?

Unfortunately you commented during a time when there was nobody with commit rights watching this project, hence the lack of response. Please see the more recent comments on #1 for an idea about the current status of this project.
The link you mentioned likely never worked because there never has been an implementation of libp2p in Python. I decided to simply remove to reflect this. Thanks for reporting this!


Damconuong

Source name
Damconuong
Source link
https://damconuong.org/
Source language
Vietnamese
Other details
18+
Acknowledgements

[X] I have checked that the extension does not already exist by searching the extension listing and verified it does not appear there.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a meaningful title with the source name.
[X] I will fill out all of the requested information in this form.

New URL: https://damconuong.top/


question : can I run a test on one function

Hello,
Im doing now this course : https://github.com/rwobben/structured-data
all the exercises are in src and the test in test.
Now I wonder if I could test only 1 function with midje.
If it can, how do I do this .?
Roelof

You can use the repl tools to test a single function: https://github.com/marick/Midje/wiki/Repl-tools
I personally use in-repl autotest https://github.com/marick/Midje/wiki/Autotest
These questions are probably better asked in the mailing list: https://groups.google.com/forum/#!forum/midje


pulumi package error glibc 2.38 when used with dotnet

Describe the bug
I created a pulumi c# project and ran pulumi import
Diagnostics:
pulumi:pulumi:Stack (kv-pulumi-test-dev):
/home/username/.dotnet/dotnet: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.38' not found (required by /nix/store/bn7pnigb0f8874m6riiw6dngsmdyic1g-gcc-13.3.0-lib/lib/libstdc++.so.6)     /home/username/.dotnet/dotnet: /lib/x86_64-linux-gnu/libc.so.6: version GLIBC_2.36' not found (required by /nix/store/bn7pnigb0f8874m6riiw6dngsmdyic1g-gcc-13.3.0-lib/lib/libstdc++.so.6)
linux mint 21.3
latest nix
used nix-env --install pulumi
pulumi-3.120.0
dotnet 8.302 install using script from Microsoft- not using nix version of dotnet because I need the latest version.
ldd --version
ldd (GNU libc) 2.39

strings /lib/x86_64-linux-gnu/libc.so.6 | grep GLIBC
latest is GLIBC_2.35
Expected behavior
A clear and concise description of what you expected to happen.
when I run pulumi import it works correctly.  When I download pulumi binaries directly, they work correctly.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Notify maintainers

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.

 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.0-113-generic, Linux Mint, 21.3 (Virginia), nobuild`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.23.0`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Add a :+1: reaction to issues you find important.
I assume the issue is that dotnet was not install from nix and therefore uses the operating system libc and not the one installed from nix.  I am trying to switch from brew to nix and this was the only issue I ran into.  I have read many of the other articles but I thought this was a different scenario.

Doesn't seem like a nixpkgs issue if you didn't use the package from nixpkgs. BTW, getting-started questions are best asked on Discourse, for future reference.

@relcodedev Is the environment.memoryAllocator.provider option set to something other than libc? I had a similar issue when I use jupyter.


feat(add rgb color variant): add RGB color variant

Description
Adding RGB color variant to some css variables so we can use them with rgba()

Looks good, could you please only update the commit message to:
style(colours): add new RGB colour variants

Commit message changed


React 18

Closes #34
Per @tom-sherman's instructions, I forked his react-18 branch and added a couple of commits. I attempted to mirror the React 18.1 documentation:

upgraded to react and react-dom 18.1
deprecated ReactDOM.hydrate, ReactDOM.unmountComponentAtNode
added ReactDOM.Client to encapsulate createRoot, hydrateRoot, and Root module

I'm happy to modify as needed and do any additional work to close #34, but I'm pretty new to ReScript (background in Clojure and OCaml) so please advise on the following:

I do not know how to run tests correctly. Running node test/React__test.bs.js did not work
Introducing the Client module is my attempt to mirror React 18.1 but may not be the proper way to do things in ReScript
There is probably a much more efficient way to detect changes between the alpha and release of React 18 than hunting through documentation and javascript code.
If someone with more experience lists the remaining work, I have some time to tackle it.

maybe somebody have any updates about timeline for merging this PR?

I think these changes would be good land as a stacked diff on top of #35 (ie. after it's been merged)
@rickyvetter what do you think?

I marked this ready for review for my last 2 commits. This will eventually be rebased once #35 is merged.

@whitchapman @tom-sherman Thank you for your work and sorry for the very long delay. I am currently trying to clean up the repo and to move the pull requests forward.
Would it be possible to have a single React 18 PR (either this one or #35) rebased on master for review? And the other one to be closed?

@whitchapman @tom-sherman Thank you for your work and sorry for the very long delay. I am currently trying to clean up the repo and to move the pull requests forward.
Would it be possible to have a single React 18 PR (either this one or #35) rebased on master for review? And the other one to be closed?

Absolutely, whatever is easier. I forked @tom-sherman 's repo, so would it be better for me to cherry pick b254874 so that #46 is has all 5 commits? What do you think Tom?

Sounds good!

Ok, I force pushed with all 5 commits -- I think I did it correctly; although, there are conflicts.

@tom-sherman Since my fork is from your fork of the original repo, I think you need to sync your fork first before I can sync mine -- how did this get to be so complicated!?

@whitchapman Couldn't you just rebase on the lastest master (d5d9168) and force push?

My fork doesn't have that commit since Tom's fork doesn't have that commit. I would need to refork directly from rescript-lang and open a new PR -- definitely doable. I'll give it a go.

IMHO it shouldn't matter if your fork has that commit, you should be able to add the rescript-lang/rescript-react repo as a remote in your local project dir and perform a fetch, then you should have that commit locally and be able to rebase on it.

Ok, @cknitt I did what you suggested which resulted in modifying the PR to remove the test changes since the test folder has been removed in master.

I think that's fine if the tests have been removed in the meantime.
Looking at the changes, is still everything there that you would expect (except for the removed tests)?
CI is still failing, you need to make some small adjustments in the V3 compatibility modules to fit the changes.

Yes, my changes are as expected. It has been several months since I last looked at this, and I'm not familiar with the V3 changes. Is there a resource for the V3 changes? I can look at this later today.

Looking at the CI errors, I think it's just about adapting to these changes:

ReactDOM.Experimental.root -> ReactDOM.Client.Root.t
useTransition does not take a transitionConfig anymore

Sorry!!! Looking again at the discussion in the original PR (https://github.com/rescript-lang/rescript-react/pull/35#discussion_r839220770) and also at the TypeScript typedefs for useTransition, I realize my previous comment was wrong and this definition
external useTransition: unit => (bool, (. unit => unit) => unit) = "useTransition"

is actually correct. Let's change it to this in both the React and React_V3 modules.
I think then we are done. 🎉
     * The `useTransition` hook returns two values in an array.
     *
     * The first is a boolean, React’s way of informing us whether we’re waiting for the transition to finish.
     * The second is a function that takes a callback. We can use it to tell React which state we want to defer.

Ok, merging now. In case anything is still missing or incorrect we'll address it in separate PRs.


Added compatibility for Desynchronized

https://steamcommunity.com/sharedfiles/filedetails/?id=2222607126

Sorry I took so long, it escaped my mind.

Sorry I took so long, it escaped my mind.


error when input name is 'nodeName'

http://jsfiddle.net/tb9ooc8q/
There is only a text input named nodeName in a form tag, when click the input box, it triggers a error:

Uncaught TypeError: elem.nodeName.toLowerCase is not a function

it only happens when the input's name is nodeName

that seems to be a problem with jQuery, rather than Bootstrap? /cc @fat

@patrickhlauke http://jsfiddle.net/8q3sL5s9/ without bootstrap it is OK

well, without Bootstrap there's no actual JavaScript code being executed, so jQuery is not called...

"Uncaught TypeError: elem.nodeName.toLowerCase is not a function" jquery-2.1.3.js:1626

Duplicate of https://github.com/jquery/jquery/issues/1721


Support Yokogawa WT310E

Fixes #238 e.g.
All checks passed

INFO    Sucessfully loaded MLPerf log from closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server/accuracy/mlperf_log_detail.txt.
INFO    Detected power logs for closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server
INFO    Sucessfully loaded MLPerf log from closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server/performance/run_1/mlperf_log_detail.txt.
INFO    Sucessfully loaded MLPerf log from closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server/performance/run_1/mlperf_log_detail.txt.
INFO    Sucessfully loaded MLPerf log from closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server/performance/run_1/mlperf_log_detail.txt.
[x] Check client sources checksum

[x] Check server sources checksum

[x] Check PTD commands and replies

[x] Check UUID

[x] Check session name

[x] Check time difference

[x] Check client server messages

[x] Check results checksum

[x] Check errors and warnings from PTD logs

[x] Check PTD configuration

[x] Check debug is disabled on server-side

[x] Check release version

Tested both with the new data:
anton@krai:~$ python3.8 $(ck locate env --tags=mlperf,inference,r1.0)/inference/tools/submission/submission-checker.py \
--more-power-check --submitter Qualcomm --version v1.0 \
--input ~/projects/mlperf/submissions_inference_1_0-audit \
2>&1 | ~/projects/mlperf/krai-mlperf-inference/dump-repo-to-submission/colorize.py
...
INFO    ---
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/offline<PHONE_NUMBER>00 with power_metric = 566.887367
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server 78501.900000 with power_metric = 536.106667
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/ssd-resnet34/offline 1773.960000 with power_metric = 603.937761
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/ssd-resnet34/server 1557.430000 with power_metric = 554.615000
INFO    ---
INFO    ---
INFO    Results=4, NoResults=0
INFO    SUMMARY: submission looks OK

as well as with the v1.0 data:
anton@krai:~$ python3.8 $(ck locate env --tags=mlperf,inference,r1.0)/inference/tools/submission/submission-checker.py \
--more-power-check --submitter Qualcomm --version v1.0 \
--input ~/projects/mlperf/submissions_inference_1_0 \
2>&1 | ~/projects/mlperf/krai-mlperf-inference/dump-repo-to-submission/colorize.py
...
INFO    ---
INFO    Results closed/Qualcomm/results/aedk_15w-qaic-v1.3.14-aic100/resnet50/offline 5034.180000 with power_metric = 27.928277
INFO    Results closed/Qualcomm/results/aedk_15w-qaic-v1.3.14-aic100/resnet50/singlestream 0.854660 with power_metric = 0.018512
INFO    Results closed/Qualcomm/results/aedk_15w-qaic-v1.3.14-aic100/ssd-resnet34/offline 92.993400 with power_metric = 26.626792
INFO    Results closed/Qualcomm/results/aedk_15w-qaic-v1.3.14-aic100/ssd-resnet34/singlestream 31.703549 with power_metric = 0.627218
INFO    Results closed/Qualcomm/results/aedk_20w-qaic-v1.3.14-aic100/resnet50/offline 7806.900000 with power_metric = 36.425503
INFO    Results closed/Qualcomm/results/aedk_20w-qaic-v1.3.14-aic100/resnet50/singlestream 0.888152 with power_metric = 0.020497
INFO    Results closed/Qualcomm/results/aedk_20w-qaic-v1.3.14-aic100/ssd-resnet34/offline 161.908000 with power_metric = 36.290015
INFO    Results closed/Qualcomm/results/aedk_20w-qaic-v1.3.14-aic100/ssd-resnet34/singlestream 30.435498 with power_metric = 0.595354
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/offline<PHONE_NUMBER>00 with power_metric = 561.943939
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/resnet50/server 78501.900000 with power_metric = 533.812000
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/ssd-resnet34/offline 1777.910000 with power_metric = 598.093263
INFO    Results closed/Qualcomm/results/r282_z93_q5-qaic-v1.3.14-aic100/ssd-resnet34/server 1557.430000 with power_metric = 547.826500
INFO    ---
INFO    ---
INFO    Results=12, NoResults=0
INFO    SUMMARY: submission looks OK

(No change for all power submissions from NVIDIA, Dell, Qualcomm and Krai.)

To test on a branch taken from master, I had to disable and then re-enable the version check.

Merged in WG on 6/2/21.


langchain[minor]: Generic chat models

Todo:

[x] tests
[ ] docs

Updated packages which had dependencies on langchain to no longer depend. All instances had langchain as a dep for tests, so no functionality was lost by doing this. In most cases it was for agent tests, so we need to figure out a new way to test integrations with agents.
I am aware langchain latest/lowest deps CI test is failing, will need a slight refactor

A few more nits


how to clear disk cache?

I'm also wondering how to clear the cache.. is this possible?
My use case is - I want to cache videos while a user is at a certain event (facebook like stream of media during the event). But, after they leave that event I'd like to clear the cache since there's no need to keep these videos around.

@jpage4500
try
@Click(R.id.cleanCacheButton)
    void onClearCacheButtonClick() {
        try {
            Utils.cleanDirectory(getExternalCacheDir());
        } catch (IOException e) {
            Log.e(null, "Error cleaning cache", e);
            Toast.makeText(this, "Error cleaning cache", Toast.LENGTH_LONG).show();
        }
    }

@bravekingzhang
I assume this method just deletes all files in the directory? I'm pretty sure I don't see a Utils class in this library
Utils.cleanDirectory(getExternalCacheDir());

I was looking for something a little more targeted to just the video files that were being cached locally. But, ultimately I ended up making a small change to add in custom logic for keeping/removing cached files - it's in a pull request here: https://github.com/danikula/AndroidVideoCache/pull/106

@jpage4500 That's great ！Thanks for you contribution！

@bravekingzhang You can sent cache's directory in HttpProxyCacheServer.Builder and then just clean it.

From com.danikula.videocache.StorageUtils.java, it is clear that the cache director can be internal or external. In my case it is at "/storage/emulated/0/Android/data//cache/video-cache". Since the class is not accessible publicly, I copied it to my local module and fetched the cache director. And once it's know deleting it can be straight forward as indicated above. Not sure what is 'Utils.' class here though.


cannot use Hyper as terminal

when i choose hyper (https://hyper.is/) as terminal for ssh
the app crash.
Is there something i am doing wrong ?
tabby/ windows terminal seems to work

How exactly did you choose hyper as a terminal and what error does it return?
To reason hyper is not supported by xpipe out of the box is that it is missing a CLI. Last time I checked there was no way to start hyper with a specified command to run.

could be ... hosnestly , i just tried the direct way :)
i try with :
settings => terminal => custom => <path_to>\Hyper\Hyper.exe
i then get a "JavaScrip error occurred in the main process"
cannot create process, error code: 267

Yeah that is due to hyper not supporting that. You can try lobbying them to think about supporting passing commands via the CLI but I doubt that that will happen.
So will sadly close this as non-fixable from my end.


Exception is thrown when trying multiple login-attempts

Describe the bug
An exception is thrown when the app tries to remove subjectID from the encrypted storage that does not exist. This occurs when a user tries to repeatedly login (for example with invalid credentials) and also to navigate back to the landing screen between tries.
To Reproduce
Steps to reproduce the behavior:

Go to 'Login Screen'
Login with wrong credentials
Navigate back to "Landing Screen"
See error

Expected behavior
No thrown exception.
Screenshots

Smartphone (please complete the following information):

iPhone Simulator
OS: iOS 14.5

closed with #64


telemetry: fix target compid of two commands

If we send them to any component, then we might get an answer from someone else rather then the autopilot and get confused.
Found by @TSC21.

@julianoes should be target_component_id

It's what is defined on the MAVSDK API.

Oooh that's in the MAVSDK API! I was really confused by looking at the mavlink specs :laughing:. Thanks Nuno!


⚠ Fetchers has degraded performance

In 34419d8, Fetchers ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 364 ms

Resolved: Fetchers performance has improved in 1aec152.


problem with werkzeug and flask in nginx-flask-mysql

There is some sort of version / dependency issue with Flask and Werkzeug:
cd nginx-flask-mysql
docker compose up

Crashes with the following trace, over and over:
nginx-flask-mysql-backend-1  | Traceback (most recent call last):
nginx-flask-mysql-backend-1  |   File "/usr/local/bin/flask", line 5, in <module>
nginx-flask-mysql-backend-1  |     from flask.cli import main
nginx-flask-mysql-backend-1  |   File "/usr/local/lib/python3.10/site-packages/flask/__init__.py", line 7, in <module>
nginx-flask-mysql-backend-1  |     from .app import Flask as Flask
nginx-flask-mysql-backend-1  |   File "/usr/local/lib/python3.10/site-packages/flask/app.py", line 28, in <module>
nginx-flask-mysql-backend-1  |     from . import cli
nginx-flask-mysql-backend-1  |   File "/usr/local/lib/python3.10/site-packages/flask/cli.py", line 18, in <module>
nginx-flask-mysql-backend-1  |     from .helpers import get_debug_flag
nginx-flask-mysql-backend-1  |   File "/usr/local/lib/python3.10/site-packages/flask/helpers.py", line 16, in <module>
nginx-flask-mysql-backend-1  |     from werkzeug.urls import url_quote
nginx-flask-mysql-backend-1  | ImportError: cannot import name 'url_quote' from 'werkzeug.urls' (/usr/local/lib/python3.10/site-packages/werkzeug/urls.py)
nginx-flask-mysql-backend-1 exited with code 1

This is a version problem. I had the same. I haven't looked much into it to understand the real issue, however, I found that those two versions work together. :
Werkzeug==2.3.7
> flask==2.2.2

That did the trick! You might consider making that change to the repo, it would be really helpful. Thanks!

bumping the flask version should work too
Flask==3.0.3
mysql-connector==2.2.9

More info here: https://stackoverflow.com/a/77214086

So, you can use libs without number version, like:
Flask
mysql-connector

In this way, you'd use the latest versionof the libraries.


.build_model() Inconsistencies

.build_model() is supposedly a required step between instantiating a clv model and fitting it. However, models can still be fit if this is skipped, and from what I can tell, the results are identical. The below example is for BetaGeoModel, but it's the same deal with ParetoNBDModel:
import pandas as pd
from pymc_marketing.clv.models import BetaGeoModel

# load data
url_cdnow_rfm = "https://raw.githubusercontent.com/pymc-labs/pymc-marketing/main/datasets/clv_quickstart.csv"
df = pd.read_csv(url_cdnow_rfm)
df['customer_id'] = df.index

# init and fit model
bgnbd = BetaGeoModel(df)
#bgnbd.build_model()
bgnbd.fit()

# this will raise an error if build_model() was not called
repr(bgnbd)

The only difference is the missing __repr__ attributes.
Moving the .build_model() call inside the fit() method seems like an easy fix for this. Even so, how's it possible to fit a model that was never defined to begin with?

fit() calls build model under the hood, and as far as I'm concerned there isn't any suggestion stating that it needs to be called before fitting the model to the data. So calling build_model before fit() makes no difference, as it just executes build twice in a row

fit() calls build model under the hood, and as far as I'm concerned there isn't any suggestion stating that it needs to be called before fitting the model to the data.

Why isn't the repr method working properly if build_model is already being called under the hood?

I'm not familiar with your specific implementation, it might have some differences. All other models implement a separate test for repr being used, so it might be a good idea to implement it in Pareto as well, and worst case scenario we'll get some insight on what's failing

(@ColtAllen) here. There is already a passing repr test for ParetoNBDModel, but this is also happening for BetaGeoModel.  Omitting the call to .build_model() is an untested edge case for all models.

Calling self.build_model() within the subclass __init__ is an easy fix for this, and would make any calls within self.fit() redundant. I'm working on a PR now that does the latter for ParetoNBDModel.

This is an artifact of the ModelBuilder API, where you don't specify the data until you call .fit.


Unable To Retrieve Contact Chat

Is there any other method of firestore or realtime database manual method to display users to chat???


ENH: add pd.test to enable nose test runnning from the imported session, #4327

closes #4327
copied from numpy nosetester.

any comments / objections? anything further I should add to docs?

Maybe I'm just making trouble, but should we consider switching to pytest?

I actually do like pytest better - fixtures IMHO are much nicer
is there an easy / automated way to convert things?

convert is not the right word / more of adjust

https://pytest.org/latest/nose.html

I also prefer pytest (using it for my personal projects), but the question is maybe also how much effort it is to switch such a big test suite.
Matplotlib is switching from nose to pytest: https://github.com/matplotlib/matplotlib/pull/5405/files

and xref: https://github.com/pydata/pandas/issues/10788

FYI: http://pytest.readthedocs.org/en/latest/nose.html specifically
Unsupported idioms / known issues

unittest-style setUp, tearDown, setUpClass, tearDownClass are recognized only on unittest.TestCase classes but not on plain classes. nose supports these methods also on plain classes but pytest deliberately does not. As nose and pytest already both support setup_class, teardown_class, setup_method, teardown_method it doesn’t seem useful to duplicate the unittest-API like nose does. If you however rather think pytest should support the unittest-spelling on plain classes please post to this issue.
nose imports test modules with the same import path (e.g. tests.test_mod) but different file system paths (e.g. tests/test_mode.py and other/tests/test_mode.py) by extending sys.path/import semantics. pytest does not do that but there is discussion in issue268 for adding some support. Note that nose2 choose to avoid this sys.path/import hackery.
nose-style doctests are not collected and executed correctly, also doctest fixtures don’t work.
no nose-configuration is recognized

Just ran them both,
Nose
Ran 5468 tests in 932.244s

FAILED (SKIP=48, errors=4, failures=1)

(Two of those errors are from the branch I'm working on, not sure about the cause of the others)
Pytest
============= 21 failed, 5525 passed, 48 skipped in 958.65 seconds =============
```

Just glancing at the output the majority of the additional failures came from `test_testing.py`

Hi,
I noticed an issue when attempting to use the new pd.test() function, at least on a windows machine.
If I try to run it from anywhere within the pandas repo in particular in the root folder I get the following output:
[pandas_dev] C:\Users\jahfe_000\Documents\GitHub\pandas>python
Python 2.7.11 |Continuum Analytics, Inc.| (default, Jan 29 2016, 14:26:21) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://anaconda.org
>>> import pandas
>>> pandas.test()
Running unit tests for pandas
pandas version 0.17.1+311.g70f79ce
numpy version 1.10.4
pandas is installed in pandas
Python version 2.7.11 |Continuum Analytics, Inc.| (default, Jan 29 2016, 14:26:21) [MSC v.1500 64 bit (AMD64)]
nose version 1.3.7

----------------------------------------------------------------------
Ran 0 tests in 0.000s

OK
<nose.result.TextTestResult run=0 errors=0 failures=0>
>>>

However if I browse out of the pandas folder, it works properly:
[pandas_dev] C:\Users\jahfe_000\Documents\GitHub\pandas>cd ..

[pandas_dev] C:\Users\jahfe_000\Documents\GitHub>python
Python 2.7.11 |Continuum Analytics, Inc.| (default, Jan 29 2016, 14:26:21) [MSC v.1500 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
Anaconda is brought to you by Continuum Analytics.
Please check out: http://continuum.io/thanks and https://anaconda.org
>>> import pandas
>>> pandas.test()
Running unit tests for pandas
pandas version 0.17.1+311.g70f79ce
numpy version 1.10.4
pandas is installed in c:\users\jahfe_000\documents\github\pandas\pandas
Python version 2.7.11 |Continuum Analytics, Inc.| (default, Jan 29 2016, 14:26:21) [MSC v.1500 64 bit (AMD64)]
nose version 1.3.7
C:\Anaconda\envs\pandas_dev\lib\site-packages\matplotlib\backends\qt_compat.py:84: ImportWarning: Not importing directory 'C:\Anaconda\envs\pandas_dev\sip': missing __init__.py
  import sip
....................................S..............S..............S..............S.................F................................................................................................................................................................................................................EE......SS...........S....S.............S.

I think this may be because while in the root folder, pandas.__file__ returns a relative path from the package root. When launched from outside the module path, it returns an absolute path which seems to be processed nicely by NoseTester.
I don't know if this is really important given that people running a develop instance of pandas from the git repo will know how to run the tests separately but I though I would flag this up.
I can replicate this problem on windows after following the steps to setup a dev instance of pandas from the documentation.

@clembou good point.
I think that if this is run in a development version then it should fail loudly (if the extensions are not built), or work. Can you create a separate issue for this (and PR to fix if you can)

@jreback done: #12285.
I'll see if I can find a fix, and attempt to make a PR!

Hmm, is there good reason to prefer pytest to nose?

@charris I really like pytest, but the codebase was originally setup using nose, so sort of stuck with it. I mean it works, but we don't really use any of the power of pytest IOW fixtures, so no real reason to switch at this point.
you guys thinking of chaning in numpy?

@jreback No. I was just surprised that there was a package that others would consider for the role. A quick look does show some interesting features in pytest, but currently nose works well enough for numpy, at least in my opinion ;)


[ML][Pipelines] fix: support implicit internal components enabling

Description
Fix a bug that, exception will be raised when detecting internal component schema while corresponding environment variable is not set to True after implicitly enabling internal components by import azure.ai.ml._internal.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-ml


Add Settings Page

Overview
Allows the user to access their profile, tokens, and embedded map
configuration under tabs in a new settings section. Adds the initial
embedded map configuration controls and preview.
Follow-up Tasks:

Create API to manage Contributor Custom Fields.
Integrate API for Contributor Custom Fields to Embed Config Page.
Allow Contributor Custom Fields in Embedded Map.
Make Contributor Custom Fields Draggable
Allow Custom Color Theming
Allow Custom Font
Make Iframe Preview and Embed Code Responsive

Connects #1287
Demo

Testing Instructions

Navigate to a contributor page and confirm that you can see the profile as before
Login and click the dropdown in the top-right corner. You should see ‘settings’ as an option.
Go to ’settings’. You should see your profile page under a ‘profile’ tab. You should see an embed tab and a tokens tab.
Select the tokens tab. You should be able to create and delete your token as before.
Select the embed tab. The embed controls should match the wireframe.
Confirm that the select all and individual checkboxes for fields behave as expected.
Change the text for a field. A ‘reset’ button should appear. Click the reset button and the filename should be restored to the default value.
Change the theme color. Confirm it is represented in the iframe code.
Select a font. Confirm that the fonts are previewed in the select. The selected font should be reflected in the iframe code.
Change the width and height of the iframe. This should be reflected in the embed code and the preview.
Select the ‘100%’ option. The width input should be disabled and the height input should have helper text and change to %.
You should see an iframe code with a copy button. Click the copy button and confirm the code was added to your clipboard.

Checklist

[ ] fixup! commits have been squashed
[ ] CI passes after rebase
[ ] CHANGELOG.md updated with summary of features or fixes, following Keep a Changelog guidelines

Looking at this now.

Made a number of updates - the largest being adding a feature flag for embedded maps. In order to use the feature flag with the tabs, I needed to move the tabs into a dynamically calculated array and grab the indexes from it, which in terms of future adaptability may be preferable regardless. I have a note out to Katie re: the help text.

Looking at this now.


Miscelaneous bugfixes

9964da8: Solves a missing error check in the middle-end.
23fcdb8: Solves a potential data race in the mock.
24003a6: Solves another potential data race in the mock.

Btw, unsure if you noticed the usage of backend vs b

Yeah, it's in diferent files but should be fixed anyways. One more gardening TODO 😄.


Dockerify2

I've fix the #180.
Now it's perfectly working with bin/benchmarker -r 1
> ./bin/benchmarker -r 1
Last update: 2018-04-17

OS: Darwin (version: 17.3.0, arch: x86_64)
CPU Cores: 8
threads: 9, requests: 1
Benchmark running ...
Done. <- rails
Done. <- sinatra
Done. <- roda
Done. <- rack-routing
Done. <- kemal
Done. <- router.cr
Done. <- raze
Done. <- lucky
Done. <- amber
Done. <- spider-gazelle
Done. <- echo
Done. <- gorilla_mux
Done. <- iris
Done. <- fasthttprouter
Done. <- gin
Done. <- actix
Done. <- iron
Done. <- nickel
Done. <- rocket
Done. <- express
Done. <- polka
Done. <- plug
Done. <- phoenix
Done. <- vapor
Done. <- perfect
Done. <- kitura
Done. <- akkahttp
Done. <- aspnetcore
Done. <- sanic
Done. <- japronto
Done. <- flask
Done. <- django
Done. <- tornado
Done. <- jester
Done. <- mofuw

@waghanza I've pushed my patches at here.
If you are ok, please merge this.
If you have more patches, please push directly on this and request review for me.
Thanks!

I really totally don't understand why you say I lied. Please stop saying like that strong words.
I just try to confirm that the all containers working well or not here.
That is the reason why I put the -r 1 option for the execution.

No no!  I'm not saying you lied at all!  I'm stating that the display from the output is lying about what it is actually doing (it implies it is using 9 threads worth of power to siege the servers, which it is not doing), plus I don't think that was your code anyway.  ^.^

Oh OK, sorry for that. Let me discuss about it at another place. 😄

@waghanza Of course. Please open the channel and put the link onto README.md :+1:
BTW, is this PR OK for @waghanza ?
Your commits in this.

@OvermindDL done it :+1:
I'll PR about this next week

@waghanza What's going on?

I'll merge this after test (somewhere around tomorrow)

use shards to release all binaries (client,benchmarker)

Sorry, what? Do you know that neither make nor neph doesn't build bin/benchmarker now, what is written in README about usage.

@AlexWayfer yeah the README is quiet  outdated
After publishing results, I will re-write the documentation

After publishing results, I will re-write the documentation

OK, thank you!


Jenayah: Watch hostbring.com

Jenayah requests the watch of the watch_keyword hostbring\.com. See the MS search here and the Stack Exchange search here.

hostbring\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


[HOLD for payment 2024-02-26] [$500] Add support for redirecting user to a target URL(via exitTo query param) when they click on the magic link code

Part of https://github.com/Expensify/App/issues/34166
Please go through the description of the above issue carefully before you post any questions.
We aim to enhance user experience by allowing first-time users to directly access our product feature without a need to manually go through the login process. This enhancement involves embedding a magic code in the offline activity email sent to users.
Magic Link Format: https://new.expensify.com/v//?exitTo=valid_target_path_in_newDot
https://new.expensify.com/v/123/000000?exitTo=r/12345
Functional Requirements:

If the user is not logged in, the magic code should trigger the sign-in process automatically and redirect the user to the target URL specified in the exitTo parameter of the link once the sign-in logic is successfully executed.

If the user is already logged in, the magic code sign-in should be skipped, and the user should be directly redirected to the exitTo URL.

Resources:
You will probably need to play with ValidateLoginPage logic
Website file: https://github.com/Expensify/App/blob/main/src/pages/ValidateLoginPage/index.website.tsx
Other platform: https://github.com/Expensify/App/blob/main/src/pages/ValidateLoginPage/index.tsx
Let me know if anything is unclear or if you need help with anything. Thanks.
CC @MitchExpensify
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a6d9657733b55abf
Upwork Job ID:<PHONE_NUMBER>380135936
Last Price Increase: 2024-02-12
Automatic offers:

rayane-djouah | Contributor | 0

I was already playing with changes similar to above, can you please add a branch for testing in your proposal, thanks.

Cc @rayane-djouah @abzokhattab

Great, thanks for the screen recordings. It seems to behave the way we expected. @allroundexperts, can you please review the above proposals? This is a time-sensitive one, thanks. 🙇

Both @rayane-djouah and @abzokhattab proposed the solution correctly. @abzokhattab was the first one to propose, but @rayane-djouah created a draft PR and shared the working screen recordings as well. Normally, the first one to propose gets the issue but since we need to move fast here, and the draft PR is already up, let's go with @rayane-djouah's proposal.
🎀 👀 🎀 C+ reviewed

Switching this to Daily so I'm on top of it for payment

@rayane-djouah Another improvement that we can make here is that the magic code page appears for a moment. QA team shared this video.
https://github.com/Expensify/App/assets/35863227/06803114-31b0-447f-b2d8-c7c015e65675
Can you look into it and see if we can improve this.

Reminder set to pay

@techievivek just checking in on your comment here - do you consider this a regression from the solution used in this issue?

No, we can't classify it as a regression because it doesn't cause any general breakage. It would simply contribute to a smoother transition.

Gotcha, thanks for clarifying.
Payment summary:
C+: $500 to be paid to @allroundexperts through NewDot Manual Requests
Contributor: $500 to be paid to @rayane-djouah via Upwork - complete ✅
@allroundexperts, a reminder to complete the BZ checklist when you get a chance - thanks!

Bump on the BZ checklist @allroundexperts - thank you!

@bfitzexpensify This is a new feature and checklist isn't needed here.

Cool - closing this out. Thanks everyone!

$500 approved for @allroundexperts based on summary.