Customized tooltips on scatter plot not working

I have been able to get them to work outside of react-google-charts, see http://jsfiddle.net/4THAe/1/
I have been stumped by this error while using react-google-charts: "All series on a given axis must be of the same data type". Can you confirm that you can get tooltips working on scatter plots with react-google-charts?
Here is my code, it is as basic as possible:
function build_scatter() {
return {rows: [[0.5, 0.5, 'test123']], columns: [{
	    type: 'number',
			role: 'domain',
			label:'Overall Score'
	  },
	  {
	    type: 'number',
			role: 'data',
	    label:'QB'
	  },
		{
	    type: 'string',
			role: 'tooltip',
	    label: null
	  }
	]};
}

and then for my plot:
var ScatterChartData =
{
    circle_scatter:
    {
      rows : circle_scatter_data.rows,
      columns : circle_scatter_data.columns,
      options:
      {
        title: 'Overall Score vs. Position',
        legend: {bottom: 'top', textStyle: {color: 'black', fontSize: 12}},
        colors: ['#0000FF','#FF0000', '#00FF00'],
        pointSize: 2,
        vAxis: {title: 'Overall Score'},
      }
    }
};

Hey,
In the version of the package you were using the role attribute in column was not supported.
It should work now :)
Cheers

awesome thanks!
On Wed, Mar 23, 2016 at 7:02 AM, Rakan Nimer<EMAIL_ADDRESS>wrote:

Hey,
In the version of the package you were using the role attribute in column
was not supported.
It should work now :)
Cheers
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/RakanNimer/react-google-charts/issues/23#issuecomment-200297356

--
-Daniel


JVM is ignoring LD_LIBRARY_PATH

Have you tested with the latest version? yes
Summary
A brief summary of the problem.
The HotSpot JVM is ignoring LD_LIBRARY_PATH and expecting shared libraries to be one level up from where libjvm.so is loaded. This differs from J9 and we believe this to be a bug because the library path is ignored when searching for shared libraries
Steps to reproduce
Run our test program below using the input supplied we have included our tests and results here
#include <jni.h>
int main() {
printf("begin..........\n");
JavaVM jvm; // denotes a Java VM
JNIEnv env;  // pointer to native method interface
JavaVMInitArgs vm_args;  // JDK/JRE 6 VM initialization arguments
JavaVMOption options = new JavaVMOption[1];
char optionString[] = "-Djava.class.path=/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/classic";
options[0].optionString = optionString;
vm_args.version = JNI_VERSION_1_8;
vm_args.nOptions = 1;
vm_args.options = options;
vm_args.ignoreUnrecognized = false;
// load and initialize a Java VM, return a JNI interface pointer in env
int res = JNI_CreateJavaVM(&jvm, (void*)&env, &vm_args);
printf("result=%d\n", res);
delete options;
jvm->DestroyJavaVM();
return 0;
}
Test run summary
Hotspot simply ignores the library path and only works when the library path points to the libjvm.so and one level up contains libjava.so. Below is proof using the test program above.**
Tests with results
The bin/classic directory contains everything that is in the server directory when we point LD_LIBRARY_PATH to the classic directory the JVM does not find libjava.so because it is in amd/64
export LD_LIBRARY_PATH=/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/classic
./javatest
begin..........
Error occurred during initialization of VM
Unable to load native library: /home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/libjava.so: cannot open shared object file: No such file or directory
In the second case test we see the JVM did not find the libjava.so in amd64 even though it is in the Library path and contains libjava.so.
export LD_LIBRARY_PATH=/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/classic:/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/lib/amd64
./javatest
begin..........
Error occurred during initialization of VM
Unable to load native library: /home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/libjava.so: cannot open shared object file: No such file or directory
The third case works because we meet the requirement of the first entry of the library path pointing to the jvm and one level up contains libjava.so
export LD_LIBRARY_PATH=/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/lib/amd64/server
./javatest
begin..........
result=0
Just to prove the ordering doesn't matter this case fails too.
export LD_LIBRARY_PATH=/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/lib/amd64:/home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/classic
./javatest
begin..........
Error occurred during initialization of VM
Unable to load native library: /home/mkarra/sandboxes/hotspotjdk/bin/linux64/bin/jvm/bin/libjava.so: cannot open shared object file: No such file or directory
We have been asking this question in the question section but moved it to here with a simple test program to get a response as to weather you consider this a bug, We have two rather large products that depend on the behavior of the JVM and a great deal of code will need to be changed if you don't see this as a problem. We currently use the J9 approach where we depend on the library path being able to be set to point to shared libraries. The classic directory you see above is part of the J9 directory structure dating back to IBM where the classic directory contains everything in the server directory.
## Triaging info
Java version:
Paste the entire output of java -version.
java -version
openjdk version "1.8.0_312"
OpenJDK Runtime Environment (Temurin)(build 1.8.0_312-b07)
OpenJDK 64-Bit Server VM (Temurin)(build 25.312-b07, mixed mode)
What is your operating system and platform?
Unix version
cat /etc/os-release
NAME="CentOS Linux"
VERSION="7 (Core)"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="7"
PRETTY_NAME="CentOS Linux 7 (Core)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:centos:centos:7"
HOME_URL="https://www.centos.org/"
BUG_REPORT_URL="https://bugs.centos.org/"
CENTOS_MANTISBT_PROJECT="CentOS-7"
CENTOS_MANTISBT_PROJECT_VERSION="7"
REDHAT_SUPPORT_PRODUCT="centos"
REDHAT_SUPPORT_PRODUCT_VERSION="7"
How did you install Java?
We copy the JRE directory to our  JVM directory
Did it work before?
Yes for years using the J9 VM.  We are now moving to HotSpot
Did you test with other Java versions?
We tested with the previous version but when asking the question of this forum we were asked to upgrade so we did .
If available, test with an upstream build:
N/A

When this bug was reported we thought LD_LIBRARY_PATH is being ignored but we later learned that the LD_LIBRARY_PATH is used only in system wide loading. We then learned from the Java API document that during programmatic loading of libraries using System.loadLibrary() the JVM  uses -Djava.library.path. Unfortunately the HotSpot JVM is ignoring the directories we specified in -Djava.library.path.

duplicate of https://github.com/adoptium/adoptium-support/issues/448


Can't analyse files after a fresh installation of LiSa

Hi,
I installed LiSa on a fresh new Ubuntu mate (Ubuntu MATE 18.04.3 LTS (Bionic)) and I submitted three malware samples (same Mirai compiled with different architectures from UrlHaust (r4z0r.arm, r4z0r.mips, r4z0r.x86)) and all three analysis failed.
ae2da66a4435800c63e50de2257b268e  r4z0r.arm
e357a85565f26c505f20fb9c4aa9711e     r4z0r.mips
4a388c6d3dfd5b54e3d74924337eae73   r4z0r.x86
References:
https://ubuntu-mate.org/download/       (to download ubuntu)
https://urlhaus.abuse.ch/browse/          (to get the download url of mirai)
When looking into the nginx webpage failed tab, I am seeing the following error message that is confusing me because it mention no image was found.

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/pexpect/spawnbase.py", line 150, in read_nonblocking
    s = os.read(self.child_fd, size)
OSError: [Errno 5] Input/output error

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/pexpect/expect.py", line 99, in expect_loop
    incoming = spawn.read_nonblocking(spawn.maxread, timeout)
  File "/usr/local/lib/python3.6/site-packages/pexpect/pty_spawn.py", line 465, in read_nonblocking
    return super(spawn, self).read_nonblocking(size)
  File "/usr/local/lib/python3.6/site-packages/pexpect/spawnbase.py", line 155, in read_nonblocking
    raise EOF('End Of File (EOF). Exception style platform.')
pexpect.exceptions.EOF: End Of File (EOF). Exception style platform.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.6/site-packages/celery/app/trace.py", line 385, in trace_task
    R = retval = fun(*args, **kwargs)
  File "/usr/local/lib/python3.6/site-packages/celery/app/trace.py", line 648, in __protected_call__
    return self.run(*args, **kwargs)
  File "/home/lisa/lisa/web_api/tasks.py", line 80, in full_analysis
    master.run()
  File "/home/lisa/lisa/analysis/top_level.py", line 69, in run
    sub_output = analyzer.run_analysis()
  File "/home/lisa/lisa/analysis/dynamic_analysis.py", line 36, in run_analysis
    self._vm.start_vm()
  File "/home/lisa/lisa/core/qemu_guest.py", line 100, in start_vm
    self._proc.expect('login: ')
  File "/usr/local/lib/python3.6/site-packages/pexpect/spawnbase.py", line 321, in expect
    timeout, searchwindowsize, async)
  File "/usr/local/lib/python3.6/site-packages/pexpect/spawnbase.py", line 345, in expect_list
    return exp.expect_loop(timeout)
  File "/usr/local/lib/python3.6/site-packages/pexpect/expect.py", line 105, in expect_loop
    return self.eof(e)
  File "/usr/local/lib/python3.6/site-packages/pexpect/expect.py", line 50, in eof
    raise EOF(msg)
pexpect.exceptions.EOF: End Of File (EOF). Exception style platform.
<pexpect.pty_spawn.spawn object at 0x7f2e80483fd0>
command: /home/lisa/images/arm/run.sh
args: [b'/home/lisa/images/arm/run.sh', b'/home/lisa/data/storage/3aad1d26-b450-4d79-adb7-68d632369bc5/rootfs']
buffer (last 100 chars): ''
before (last 100 chars): '/home/lisa/images/arm/run.sh: 10: /home/lisa/images/arm/run.sh: qemu-system-arm: not found\r\n'
after: <class 'pexpect.exceptions.EOF'>
match: None
match_index: None
exitstatus: 127
flag_eof: True
pid: 59
child_fd: 27
closed: False
timeout: 110
delimiter: <class 'pexpect.exceptions.EOF'>
logfile: <_io.TextIOWrapper name='/home/lisa/data/storage/3aad1d26-b450-4d79-adb7-68d632369bc5/machine.log' mode='w' encoding='utf-8'>
logfile_read: None
logfile_send: None
maxread: 2000
ignorecase: False
searchwindowsize: None
delaybeforesend: 0.05
delayafterclose: 0.1
delayafterterminate: 0.1
searcher: searcher_re:
    0: re.compile("login: ")

The same kind of error message appear for the other architectures. After the installation of the operating system, I did the following command to run LiSa. I didn't do anything else after that, that could had interfere with the installation.
sudo apt-get install git
git clone https://github.com/danieluhricek/LiSa.git
cd LiSa
sudo apt-get install docker docker-compose
sudo docker-compose build
sudo docker-compose up

It weird because it says that I dont have the qemu image for mips. Same thing for the other architecture. When reading your docker image (worker), I can see that it download a TAR.GZ with the images from https://github.com/danieluhricek/linux-images/archive/v1.0.1.tar.gz. I was able to download the file so I am not sure why it is complaining about that.
I am fairly new with Docker and this framework but I don't mind trying to find what going on but If you know something about this issue, please let me know.
Thanks for your help.

I have find the error. In fact, the worker docker file (LiSa/docker/worker/Dockerfile) should contains an additional instruction to install the package "qemu-system" that contains the different architecture packages {qemu-system-mips, qemu-system-x86, qemu-system-arm, etc...}.

@Vince147
Sorry for late response. This problem was discussed in issue #1 :

Would you rather consider (and test if everything works that way on your side) to change your pull request that the Dockerfile installs qemu-system metapackage that installs qemu-system-* for all architectures (or as for now supported architectures by LiSa - packages qemu-system-x86, qemu-system-arm, qemu-system-mips)?

Since original author didn't change the pull request, I forgot about this issue.
Would you mind to fill the pull request with your change? (Otherwise I can fix this issue directly).

Sure thing.

Solved by #4


Refactor and Standardize Route paths

Works Done

Paths changed:

/account/import-ledger        -->> /import/ledger-full-screen
/account/create                     -->> /create-account-full-screen
/account/restore-json           -->> /import/restore-json-full-screen
/account/import-seed           -->> /import/seed-full-screen
/account/import-raw-seed    -->> /import/raw-seed-full-screen
/account/export-all               -->> /export/all

ALLOWED_URL_ON_RESET_PASSWORD updated.

packages/extension-ui/src/Popup/index.tsx routes updated.

Close #1361.

Summary by CodeRabbit

New Features

Introduced full-screen views for account creation, importing, and restoring actions.

Updates

Updated navigation paths across the app for a more consistent user experience, including new paths for creating accounts and importing data.
Adjusted allowed URLs on the reset password page to match the new full-screen paths.

Bug Fixes

Fixed navigation issues by updating outdated paths to the new full-screen URLs.

@AMIRKHANEF
if we are going to have
/import/ledger-full-screen
then we can similarly have changes like
/import/attach-qr ==> /import/qr
/import/add-watch-only ==> /import/watch-only
an so on
right?

@AMIRKHANEF
if we are going to have
/import/ledger-full-screen
then we can similarly have changes like
/import/attach-qr ==> /import/qr /import/add-watch-only ==> /import/watch-only
an so on
right?

good suggestion, I will handle it.


Demonstrate how to use local fonts for styling

Fleshed out the example a bit to show how @font-face src: local API can be used to load local fonts for styling.
Fixes #52

issue #52 is mentioned in #48.


Recipe for target build failed when pulling from master

Today I tried to pull from master and after running make clean && make && make install I get the following output and error:
 Building debug lnd and lncli.
GO111MODULE=on go build -v -tags="dev" -o lnd-debug -ldflags "-X github.com/lightningnetwork/lnd/build.Commit=v0.5.1-beta-40-gfc4fe070100a66ea220a14e74f52f9c9d1550636" github.com/lightningnetwork/lnd
github.com/lightninglabs/neutrino/filterdb
github.com/btcsuite/btcwallet/internal/legacy/keystore
github.com/lightninglabs/neutrino/headerfs
# github.com/lightninglabs/neutrino/filterdb
../../lightninglabs/neutrino/filterdb/db.go:102:47: not enough arguments in call to builder.BuildBasicFilter
        have (*wire.MsgBlock)
        want (*wire.MsgBlock, [][]byte)
../../lightninglabs/neutrino/filterdb/db.go:119:21: undefined: builder.BuildExtFilter
../../lightninglabs/neutrino/filterdb/db.go:211:34: not enough arguments in call to gcs.FromNBytes
        have (number, []byte)
        want (uint8, uint64, []byte)
# github.com/lightninglabs/neutrino/headerfs
../../lightninglabs/neutrino/headerfs/store.go:529:48: not enough arguments in call to builder.BuildBasicFilter
        have (*wire.MsgBlock)
        want (*wire.MsgBlock, [][]byte)
../../lightninglabs/neutrino/headerfs/store.go:545:22: undefined: builder.BuildExtFilter
# github.com/btcsuite/btcwallet/internal/legacy/keystore
../../btcsuite/btcwallet/internal/legacy/keystore/keystore.go:2801:12: cannot use addresses (type []"github.com/btcsuite/btcd/vendor/github.com/btcsuite/btcutil".Address) as type []"github.com/btcsuite/btcutil".Address in field value
../../btcsuite/btcwallet/internal/legacy/keystore/keystore.go:2883:15: cannot use addresses (type []"github.com/btcsuite/btcd/vendor/github.com/btcsuite/btcutil".Address) as type []"github.com/btcsuite/btcutil".Address in assignment
Makefile:124: recipe for target 'build' failed
make: *** [build] Error 2

Your environment
I was on this commit 6b19df162a161079ab794162b45e8f4c7bb8beec of LND and was updating to fc4fe070100a66ea220a14e74f52f9c9d1550636 of LND
OS is Ubuntu 16.04
BTCD commit 3dcf298fed2d5fd65918dc560b3942b2aa0629e8
Steps to reproduce
cd $GOPATH/src/github.com/lightningnetwork/lnd
git pull
make clean && make && make install

Doh! Didn't realize I was on an old box and was using Go 1.10.
Updating to Go 1.11-2 solved it.
Closing.


Support enabling network access when using kpt fn run --image

Since there is no place to declare function behavior in this situation, I would suggest supporting a command-line flag for enabling access.
Maybe we could trust --network ultimately in case --image is already set?

@phanimarupaka I think this may already be supported, would you confirm?

@dflemstr This feature is already supported in kpt. You can find the usage by running "kpt fn run -h" and scroll though flags section.

Thanks for getting back to me!  I saw the flags in that section, but despite that I keep getting errors.
Some examples:
# A simple kpt function that simply pipes objects to kubeval
$ kpt fn run --network --image gcr.io/spotify-declarative-infra/kubeval
ERR  - stdin: Failed initalizing schema https://...: Get https://...: dial tcp: lookup kubernetesjsonschema.dev on <IP_ADDRESS>:53: dial udp <IP_ADDRESS>:53: connect: network is unreachable

However, running the exact same function with this config works:
apiVersion: v1
kind: ConfigMap
metadata:
  name: kpt-fn-2
  annotations:
    config.k8s.io/function: |
      container:
        image: gcr.io/spotify-declarative-infra/kubeval
        network:
          required: true

The image is private, let me know if I should create a test case that can be reproduced publicly.

@phanimarupaka ideas?

The issue is that network is only enabled if the function requests.  This is possible for declarative functions, but not for ones specified on the cli.  There is a TODO for this:
https://github.com/kubernetes-sigs/kustomize/blob/master/cmd/config/internal/commands/run-fns.go#L85
The solution is to add request for network on line 89.

@dflemstr We have merged the changes. Please check out the head and run "make build" to verify if the issue is fixed and let me know.

@dflemstr The issue has been fixed in the new release. Can you please pull the latest kpt version and check if the issue is resolved. Thanks for your patience.

Yes, I can confirm that the issue is now fixed!


£ causing float issues

Hey, awesome scraper tool you got here just trying to work through a couple issues involving UK sites.
Amazon scraper half works but reports a float error due to the currency. I've tried to fix this myself by using getText() and stripping the first character from the string returning only the value, I.E from £17.99 to 17.99. Using the logger I'm able to see a clean output but the script still seems to think the Item is out of stock. Add to cart is finding the correct tag too.
Not sure if you can point me in the general direction of your work flow. Finding it hard to follow.

Turns out Amazon UK uses Add to Basket instead of Add to cart hence the requirements was not met for the scalper.


Changing route metrics

I have a scenario in which I'd like to dynamically change route metrics between two network interfaces (Ethernet and WiFi) depending on tested connectivity through each (tested with the help of some Linux policy based routing rules). I'm seeing a couple of behaviours of dhcpcd which aren't ideal for this scenario:

I've found that if I try to dynamically redo route rules with updated route metric numbers, then dhcpcd detects it and restores the old values. That isn't what I want in my scenario.
I update the /etc/dhcpcd.conf file with new values for metric for each interface. But dhcpcd doesn't get the new values until I restart it or send SIGHUP. But when I send SIGHUP, it also removes the IP address and renegotiates it with the DHCP server. That is understandable for many config changes, but not for this one in which a route metric change is the only thing I'm changing.

What would be a good way to use dhcpcd in this scenario, of dynamically changing route metrics, which avoids the two issues described?

I would like to understand why you need to change the metrics once dhcpcd is running.
However, dhcpcd should not remove the address on SIGHUP, that sounds like a bug and not one I can reproduce.
What dhcpcd version are you running?

I would like to understand why you need to change the metrics once dhcpcd is running.

I have an embedded Linux device with Ethernet WAN and Wi-Fi client both active. Normally, the routing metric is set up so the Ethernet WAN is the preferred route.
But, something may go wrong with the Ethernet WAN route, so packets aren't getting through, even though the interface still appears "up" (eg someone unplugs a cable on an upstream switch/router). I have a monitoring task that, using policy based routing, is able to periodically test connectivity on each of Ethernet WAN and Wi-Fi. If Ethernet WAN is a bad route, then the goal is to dynamically change the route metrics so the Wi-Fi becomes the preferred route. And vice-versa.

However, dhcpcd should not remove the address on SIGHUP, that sounds like a bug and not one I can reproduce.

Looking in the code, I see that SIGHUP results in a call to both reload_config() and reconf_reboot(). Does reconf_reboot() redo all addresses, both IPv6 and IPv4?

What dhcpcd version are you running?

I'm running v8.1.6. Built by Yocto dunfell, for an ARM64 embedded Linux system.

Assuming that we only care about router reachability we can automate this in dhcpcd itself.
Linux has unreachability detection here: https://github.com/NetworkConfiguration/dhcpcd/blob/master/src/if-linux.c#L902
IPv6 ND handling code here: https://github.com/NetworkConfiguration/dhcpcd/blob/master/src/ipv6nd.c#L694
So when we make the route for the prefixes we can bump the route metric if it's unreachable here: https://github.com/NetworkConfiguration/dhcpcd/blob/master/src/ipv6.c#L2185
We could then extend this to DHCP as well. This would work fine for Linux and NetBSD as their ARP implementations work in a similar fashion to IPv6 ND - other OS's have crusty ARP implementations where this would not work, but heh ho.
Would this satisfy your use case?


developing-with-github_20888355

https://github.com/LLW12345/developing-with-github/blob/master/src/pages/LLW12345.md

Nice to meet you too! Sorry for the late merge :P


Lazy stacktrace toString conversion

when disabling atoms debug mode, StackTraceEntries are allocated, but their strings are never used.
this costs much performance in the InfinitiveWave Demo.
create many enemies and make them move. observe the unity profiler

The InfiniteWaves Demo is resetting waves to 1 when I play and I can't find where to set it. It also keeps giving me a NullReferenceException that my Player has 2 Tags, one of which is null, even though I see this:

It makes it hard to test this PR, but if you like it @miikalo I trust it works as advertised.

The InfiniteWaves Demo is resetting waves to 1 when I play and I can't find where to set it. It also keeps giving me a NullReferenceException that my Player has 2 Tags, one of which is null, even though I see this:

It makes it hard to test this PR, but if you like it @miikalo I trust it works as advertised.


Referrals not working for lightning (and slashtags?)

Description
For some reason, isNewUser is always false in the jwt callback on sign up when using lightning and thus referrerId is not set
Steps to Reproduce

Create user and set name to test01 and logout
Go to /r/test01 and see sn_referrer cookie being set with test01 as value
Sign up as new user using lightning

=> isNewUser is false in jwt callback and referrerId was not set for the new user
Expected behavior
referrerId is set

Got another report that an invite link wasn't recorded for one github login and another email login


Provide a description for Timer Definitions

Is your feature request related to a problem? Please describe
Help users to fill in a value for a timer definition.
Describe the solution you'd like
@jakobfreund proposed a hint like this

Link to docs: https://docs.camunda.io/docs/reference/bpmn-processes/timer-events/timer-events#time-duration
Describe alternatives you've considered
Leave as-is
Additional context
Please note: for now, we could use the input description to provide that information. Later this might be replaced by better contextual help as part of the prop panel redesign (not yet in scope).

To discuss: Short description with a direct link to the docs VS more context (sketched above). I'm personally in favor of shorter descriptions.
I'm also wondering why it is called Timer Definition while the field actually refers to the Duration.

The screenshot is a wording proposal. I agree, when we work on that issue we should refine it.

Since I made the proposal: Yes absolutely, was just a quick proposal to sketch out the idea! I just reckoned that even though it's just one click away, giving the examples from the doc right where you have to put in the expression would already go a long way. But the beautiful thing about the Cloud Console Modeler is that we can quickly try out ideas and see how they resonate, so I wouldn't overthink it too much, I will be happy with everything you come up with :)

How should the description text differ for the various timer events?

Timer Start Event: Date vs Cycle
Event-based Subprocess with Timer Start Event: Duration
Timer Intermediate Event: Timer Duration

How should the description text differ for the various timer events?

For each timer definition, there is a short explanation available that we could use or link to: https://docs.camunda.io/docs/reference/bpmn-processes/timer-events/timer-events/#timers

Will include it in https://github.com/bpmn-io/properties-panel-next/issues/59, hence moving to ready


hooks/14_FormValidation/fix typo in path

Still working with your amazing tutorial, guys. Thx a lot. Small typo in path.

Thanks a lot, good catch :)


Ungoogled Thorium?

Is your feature request related to a problem? Please describe.
I've been using Thorium for a few days now, and I'm really annoyed about all the "Google smarts" embedded in the browser. I don't want to be spied on.
Describe the solution you'd like, including relevant patches or source
A version of Thorium (mac pls) based on ungoogled-chromium. It's a drop-in replacement for Chromium, so it might be easy? idk?

#115


Add exemple with private github

Thanks for this project.
I test it now but I have question it's possible to use it with a private github ?

In current implementation plato-as-service downloads zip using public url. But that could be updated to use API V3: https://developer.github.com/v3/repos/contents/#get-archive-link
$ curl -H "Authorization: token ab499f3b..." \
-L https://api.github.com/repos/godzilla/my_priv_repo/zipball > wut.zip

This implementation requires OAuth token from service consumer.
Url schema could be:
/:user/:repo/:branch/?oauth_token=ab499f3b...

@kosssi PR is pending


Docker Image Build Issue with VSTS

Hi,
I am trying to build docker image with VSTS by following the steps on  Scenario C: Deploying from VSTS CD pipeline to an Azure VM using the Docker Tasks in VSTS in "https://github.com/dotnet-architecture/eShopModernizing/wiki/03.-How-to-deploy-your-Windows-Containers-based-app-into-Azure-VMs-(Including-CI-CD)".
I keep getting below error.  does "obj\Docker\publish" suppose to exist?  I am using hosted VS2017 agent to build.  Do i need to create my own private docker build agent?

2018-01-05T19:22:22.8988540Z Step 4/4 : COPY ${source:-obj/Docker/publish} .
2018-01-05T19:22:23.6913778Z COPY failed: GetFileAttributesEx \?\C:\Windows\TEMP\docker-builder239959299\obj\Docker\publish: The system cannot find the path specified.
2018-01-05T19:22:34.4747563Z ##[error]C:\Program Files\Docker\docker.exe failed with return code: 1
Any help is appreciated!
Thanks,
Dennis

@CESARDELATORRE hi, I am having a similar problem trying to build the docker file - every time I try with a docker-compose build command, I get an error like
ERROR: Service 'eshoplegacymvc' failed to build: COPY failed: CreateFile \\?\C:\ProgramData\Docker\tmp\docker-builder407623176\bin\Release\PublishOutput: The system cannot find the path specified.
I know that bin\Release\PublishOutput exists. That is a relative path from where my Dockerfile is.  I can't figure out how to solve.
Thanks

@michaelsrichter I had a similar issue.  I couldn't see the folder structure in hosted agent.  I setup a private build agent to run the docker build in VSTS (very easy to setup), so i can see the folder structure easier.  I also need to manually build the VS 2017 solution with "Release" configuration once, then run the docker build in VSTS using my private build agent and everything worked afterwards.   Hope this helps.
Dennis

I have been struggling with this similar issue for few days. I managed to find a way finally. I use ".Net Core" build and publish tasks to publish service into "obj/Docker/publish" folder and then use "Docker Compose" tasks to build and push the images to private container repository.
Kalai

@abhayongit Unfortunately that doesn't work for me. I'm still stuck with the weird "programData" path :(

@willemodendaal https://github.com/willemodendaal  -  Could you please
share your DockerFile and what you want to achieve ?
On Fri, Oct 19, 2018 at 5:15 AM Willem Odendaal<EMAIL_ADDRESS>wrote:

@abhayongit https://github.com/abhayongit Unfortunately that doesn't
work for me. I'm still stuck with the weird "programData" path :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet-architecture/eShopModernizing/issues/6#issuecomment-431343174,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ap56imAS8jr-GlreMEUo10SnlyLvLAPzks5umcJngaJpZM4RXIWZ
.

Extremely frustrating. Same issue. Azure DevOps with on-premise build agent. Everything is super default. Fresh Server 2016 server, fresh docker-for-windows, fresh ASP.NET Core 2.1 project, simple configuration
FROM microsoft/dotnet:2.1-aspnetcore-runtime AS base
WORKDIR /app
EXPOSE 12073
EXPOSE 44337

FROM microsoft/dotnet:2.1-sdk AS build
WORKDIR /src
COPY ["ACo.API/ACo.API.csproj", "ACo.API/"]
RUN dotnet restore "ACo.API/ACo.API.csproj"
COPY . .
WORKDIR "/src/ACo.API"
RUN dotnet build "ACo.API.csproj" -c Release -o /app

FROM build AS publish
RUN dotnet publish "ACo.API.csproj" -c Release -o /app

FROM base AS final
WORKDIR /app
COPY --from=publish /app .
ENTRYPOINT ["dotnet", "ACo.API.dll"]

But always getting:
COPY failed: CreateFile \?\C:\ProgramData\Docker\tmp\docker-builder597994875\ACo.API\ACo.API.csproj: The system cannot find the path specified.
Pretty new to docker so not sure what to try. It works perfectly okay from Visual Studio.

Go inside ACo.API folder and change the copy command like follwoing
COPY ["./ACo.API.csproj", "."]
On Wed, Oct 24, 2018 at 5:57 AM amivit<EMAIL_ADDRESS>wrote:

Extremely frustrating. Same issue. Azure DevOps with on-premise build
agent. Everything is super default. Fresh Server 2016 server, fresh
docker-for-windows, fresh ASP.NET Core 2.1 project, simple configuration
FROM microsoft/dotnet:2.1-aspnetcore-runtime AS base
WORKDIR /app
EXPOSE 12073
EXPOSE 44337
FROM microsoft/dotnet:2.1-sdk AS build
WORKDIR /src
COPY ["ACo.API/ACo.API.csproj", "ACo.API/"]
RUN dotnet restore "ACo.API/ACo.API.csproj"
COPY . .
WORKDIR "/src/ACo.API"
RUN dotnet build "ACo.API.csproj" -c Release -o /app
FROM build AS publish
RUN dotnet publish "ACo.API.csproj" -c Release -o /app
FROM base AS final
WORKDIR /app
COPY --from=publish /app .
ENTRYPOINT ["dotnet", "ACo.API.dll"]
But always getting:
COPY failed: CreateFile
?\C:\ProgramData\Docker\tmp\docker-builder597994875\ACo.API\ACo.API.csproj:
The system cannot find the path specified.
Pretty new to docker so not sure what to try. It works perfectly okay from
Visual Studio.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dotnet-architecture/eShopModernizing/issues/6#issuecomment-432643931,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Ap56iviq11VKpSL0eEQ7M4kj-d5XKGH3ks5uoGOogaJpZM4RXIWZ
.

Hi i have similar problem , My docker file is as stated below.
I have docker file and this simulator folder in same location and exe file is in simulator --> bin --> x64 --> Debug --> simulator.exe
I am struggling to copy whole source code and then run my exe app. and this app is .Net framework app built on version 4.6.1. And am i using right image ?
Please suggest me better way of writing Docker file and my app is .Net framework app built on version 4.6.1.
#FROM mcr.microsoft.com/windows/servercore:ltsc2016
FROM mcr.microsoft.com/dotnet/framework/sdk:4.8
WORKDIR /app
COPY simulator .
ENTRYPOINT ["\simulator\bin\x64\Debug\simulator.exe"]
#RUN powershell -Command C:\simulator\bin\x64\Debug\simulator.exe


WFRC - Active transport interactive map Estimate

Provide a cost estimate for the set of apps described in the high-level requirements below:

Utah Active Transportation Interactive Map (high level requirements)

Purpose:
Provide the most up-to-date, mapped inventory of the multi-use pathway and on-street bike facilities, and proximate amenities/destinations (i.e. rec centers, school, grocery stores etc) and other context (AADT traffic volume and approximate speed limit -- or Level of Traffic Stress LoTS)
Allow users to explore areas of interest to view bike-friendly transportation links, with full detail about the type of facility: class/subclass for on street bike facilities (e.g. buffered bike lane, sharrow route, etc), paved/non-paved for multi-use trails
Allow users to use the interactive map to contribute geographic-specific edit suggestions to the map -- specifically new, updated, or missing links
Versions:
Desktop/tablet browser design
Complementary native mobile app for iOS and Android (prompt user to obtain/open app?)
Extras:
Ability to zoom to a city, township, or zip code in Utah
Ability to turn on a map layer of planned future bike facilities
Ability to permit current location to be displayed on the map (blue dot)
Trusted editors can make or accept edits (perhaps through AGOL)
Edit history is tracked, perhaps using branched versioning
Possible filtering functionality (show me routes that....)
Exclusions:
Point-to-point routing -- let's let Google continue to do that (IT scaling, liability issues)

Benefit Hypothesis
Provide a quote that will likely lead to more revenue.
Success
A more defined set of requirements with an associated ballpark estimate.

Sprint 2

[ ] come up with questions for Bert

Sprint 4

[ ] Provide estimate ( @stdavis , @nathankota)

Sprint 4 Notes

As far as the atlas and the blue dot go, it was more of a curiosity/suggestion -- I was a little surprised that it would have been taken out without explicit direction to do so. If we move forward on the bike map app and there are synergies between coding the blue dot functionality into that new app and adding it back into atlas, I think that's how we would feel comfortable supporting that mini-project, i.e. indirectly.

estimate send via email today (11/27)


Create a gitignore for ReactNative lib developer

Reasons for making this change:
The React Native lib is one of the most widely used today for hybrid mobile application development and has a very active community.
Links to documentation supporting these rule changes:
React Native 0.61.x
Expo SDK 36

Link to application or project’s homepage: https://facebook.github.io/react-native/

Is this "ReactNative.gitignore" file intended to replace the "react-native.gitignore" file from pull request #3129 ?

Not exactly carycode, this pull request contains a gitignore for React Native + Expo Manager and pull request #3129 is just for pure react native, so to speak.


[pulsar-client] Refactor seek to reuse common logic.

Motivation
Refactor seek to reuse common logic so adding expression based seek later on would be easier.
Modifications
Describe the modifications you've done.
Change seekAsync(long) and seekAsync(MessageId) in ConsumerImpl to share common code base.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:
NA
Documentation
NA

/pulsarbot run-failure-checks

The CI-CPP, Python. Tests / cpp-tests task kept failing for quite some time. Maybe it's something related to docker image upgrade?

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


IMAP and POP3 support

Hi,
Is there any update on the part if IMAP and POP3 will be supported in the future for the new Outlook add-ins platform?
Thanks,

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

ID: f9c9b375-5f5a-7a79-d654-e8ca9bcb23e5
Version Independent ID: ee1bb242-ed4c-5b99-9e85-973f79cd4513
Content: Outlook add-in requirements - Office Add-ins
Content Source: docs/outlook/add-in-requirements.md
Service: outlook
Sub-service: add-ins
GitHub Login: @o365devx
Microsoft Alias: o365devx

Hi @usselite, thanks for submitting! We use GitHub issues to report bugs with the product, and it looks like you are requesting a feature. For feature requests, we recommend submitting via Microsoft 365 Developer Platform Ideas. We use Microsoft 365 Developer Platform Ideas to prioritize community requests for additions to the product. If the community upvotes your request, it's more likely to be added to the add-ins platform.
Since this isn't a bug, I will close this issue. Let me know if you have any questions!
Cheers,
Alison


there are some mistake in data

hi~
dataset_1.csv~dataset_5.csv
drugID matching the wrong smiles
i check by use drugID searching smiles in Drugbank

When the codes run ,In the data frame (Drug_vec), only the first row is separated !!! Why?  that is true for all datasets!!


"publish" action is triggered at each merged

See https://github.com/openzim/phet/actions/workflows/publish.yml
It should be published only when new release tag on master.

@rgaudin Do you understand why https://github.com/openzim/phet/blob/master/.github/workflows/publish.yml ? I'm a bit clueless.

Fixed with https://github.com/openzim/phet/commit/3cddbac98e31c3ba779e41c4d2d0f89b9339d1f7


Ibce preview

Refactors required for In browser content editor to become functional.
Another key change is that mosaic.config.js has been renamed everywhere to mosaic.config.mjs.  This allows us to import workflows into the config file but also is consistent with our approach of trying to have "ESM everywhere".
Hi @origami-z

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 19 (57.89%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 63.246%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/content-editor-plugin/src/api/save.ts
0
1
0.0%

packages/content-editor-plugin/src/components/PersistEditDialog/PersistStatus.tsx
1
2
50.0%

packages/content-editor-plugin/src/components/PersistEditDialog/index.tsx
2
8
25.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1305

Relevant Lines:
1801

💛  - Coveralls


Add class description for DataGenerator

Why are the changes needed?

Add usage description so we can use this tpcds data generator easily.
How was this patch tested?
Not need.

Codecov Report

Merging #700 (b2aea77) into master (43d60bd) will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #700      +/-   ##
==========================================
+ Coverage   80.41%   80.52%   +0.11%
==========================================
  Files         120      122       +2
  Lines        4717     4730      +13
  Branches      573      572       -1
==========================================
+ Hits         3793     3809      +16
+ Misses        601      600       -1
+ Partials      323      321       -2

Impacted Files
Coverage Δ

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
93.87% <0.00%> (-0.28%)
:arrow_down:

...main/scala/org/apache/kyuubi/KyuubiException.scala
100.00% <0.00%> (ø)

...cala/org/apache/kyuubi/config/internal/Tests.scala
100.00% <0.00%> (ø)

...g/apache/kyuubi/engine/spark/udf/KDFRegistry.scala
100.00% <0.00%> (ø)

...yuubi/engine/spark/udf/KyuubiDefinedFunction.scala
83.33% <0.00%> (ø)

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
91.89% <0.00%> (+0.11%)
:arrow_up:

.../engine/spark/session/SparkSQLSessionManager.scala
93.10% <0.00%> (+0.24%)
:arrow_up:

...ache/kyuubi/engine/spark/SparkProcessBuilder.scala
91.02% <0.00%> (+0.66%)
:arrow_up:

...apache/kyuubi/ctl/ServiceControlCliArguments.scala
65.06% <0.00%> (+1.20%)
:arrow_up:

...ala/org/apache/kyuubi/util/KyuubiHadoopUtils.scala
83.33% <0.00%> (+20.83%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks, merged for v1.3.0.

thanks all


Change JMX stats to deliver per node values & fix calculation errors

Previously on each individual access of a property, a query was
triggered to retrieve the values required for the calculation of the
property.
There were several problems with the implementation:

it was very in-efficient and could lead to overloading of the
threadpool
it resulted in cluster based metrics which is not ideal for monitoring
it had bugs in the calculation of the properties

This is a total rewrite of the calculation. It now utilizes a local
in-memory structure to calculate the metric.

Codecov Report

Merging #5605 into master will decrease coverage by 0.03%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #5605      +/-   ##
============================================
- Coverage     82.82%   82.79%   -0.04%
- Complexity    11770    11771       +1
============================================
  Files          1390     1390
  Lines         45238    45238
  Branches       5621     5621
============================================
- Hits          37469    37455      -14
- Misses         5160     5169       +9
- Partials       2609     2614       +5

Impacted Files
Coverage Δ
Complexity Δ

...ava/io/crate/operation/collect/stats/JobsLogs.java
97.95% <ø> (ø)
21 <0> (ø)
:arrow_down:

...operation/collect/sources/SystemCollectSource.java
94.28% <100%> (ø)
26 <4> (ø)
:arrow_down:

...n/java/io/crate/exceptions/JobKilledException.java
0% <0%> (-66.67%)
0% <0%> (-1%)

...crate/executor/transport/TransportShardAction.java
75.67% <0%> (-21.63%)
8% <0%> (-2%)

...rc/main/java/io/crate/analyze/UnboundAnalyzer.java
75% <0%> (-10%)
2% <0%> (ø)

...ration/collect/collectors/OrderedDocCollector.java
90% <0%> (-10%)
5% <0%> (-1%)

...crate/planner/projection/MergeCountProjection.java
63.63% <0%> (-9.1%)
5% <0%> (-1%)

.../java/io/crate/analyze/symbol/ParameterSymbol.java
87.5% <0%> (-6.25%)
6% <0%> (-1%)

...io/crate/planner/projection/ProjectionVisitor.java
35.29% <0%> (-5.89%)
6% <0%> (-1%)

...te/blob/pending_transfer/PutHeadChunkRunnable.java
76.71% <0%> (-4.11%)
12% <0%> (-2%)

... and 22 more

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 26ce951...eefd417. Read the comment docs.

Values match with the values reported by the client (cr8)
selects: ~ 63.26 requests/s
inserts:  ~ 895.17 requests/s


Twitch Chat not connected when first connecting to twitch

When you first oauth with twitch the chat client is never told to connect, so no chat related functionality works until you restart the application or wait for the chat reconnect period to elapse (is 30 mins currently).

This has been fixed as part of #21


Game broken

When attempting to access this at https://githubschool.github.io/github-games-joshg253, I am getting a 404. This could be caused by a couple things:

GitHub pages needs to be enabled on master. You can fix this in the repository settings.
the index.html file is incorrectly named inde.html. We will fix this together in class.

Can you please fix the first bullet @joshg253?

fixed.


Made headers in target definitions retain case sensitivity

Fixes #191.  Based on solution in #124.

Simple tests:
Vegeta
~$ cat targets.txt
GET http://localhost:8080
X-Account-ID: 8675309

GET http://localhost:8080
X-ACCOUNT-ID: 9035768

~ $ ./vegeta attack -duration=1s -rate=20 -targets=targets.txt | vegeta report
Requests      [total, rate]            20, 21.05
Duration      [total, attack, wait]    955.28554ms, 949.999921ms, 5.285619ms
Latencies     [mean, 50, 95, 99, max]  5.211109ms, 5.285619ms, 6.427649ms, 6.427649ms, 6.453087ms
Bytes In      [total, mean]            0, 0.00
Bytes Out     [total, mean]            0, 0.00
Success       [ratio]                  0.00%
Status Codes  [code:count]             0:20
Error Set:

Python simple HTTP server

```
Listening on localhost:8080

----- Request Start ----->

/
Host: localhost:8080
User-Agent: Go-http-client/1.1
X-Account-ID: 8675309
Accept-Encoding: gzip

<----- Request End -----

<IP_ADDRESS> - - [22/Aug/2016 10:10:58] "GET / HTTP/1.1" 200 -

----- Request Start ----->

/
Host: localhost:8080
User-Agent: Go-http-client/1.1
X-ACCOUNT-ID: 9035768
Accept-Encoding: gzip

<----- Request End -----

<IP_ADDRESS> - - [22/Aug/2016 10:10:58] "GET / HTTP/1.1" 200 -

...
```

Thanks!


Have a library to be used in jupyter notebooks

Is your feature request related to a problem? Please describe.
As SparseML user,
I would like to have a way to start training of a model using the library, not only CLI.
Reference: https://github.com/os-climate/aicoe-osc-demo/issues/91
Describe the solution you'd like
Have methods in library that can be imported and used from a jupyter notebook.
Describe alternatives you've considered
Additional context

from sparseml.transformers import question_answering

question_answering(
    output_dir="dense_bert-question_answering_squad",
    model_name_or_path="zoo:nlp/masked_language_modeling/bert-base/pytorch/huggingface/wikipedia_bookcorpus/base-none",
    recipe="zoo:nlp/masked_language_modeling/bert-base/pytorch/huggingface/wikipedia_bookcorpus/base-none?recipe_type=transfer-question_answering",
    dataset_name="squad",
    per_device_train_batch_size=16,
    per_device_eval_batch_size=24,
    preprocessing_num_workers=6,
    do_train=True,
    do_eval=True,
    evaluation_strateg="epoch",
    fp16=True,
    seed=42,
    save_strategy="epoch",
    save_total_limit=1
)

Same for other useful methods, like:

from sparseml.transformers import export_onnx

export_onnx(
    model_path="sparse_quantized_bert-text_classification_squad",
    task='question-answering',
    sequence_length=128
)

hi @pacospace thank you for the great suggestion.

for the export_onnx script, we do have a method in our API that exposes this functionality: from sparseml.transformers import export_transformer_to_onnx
the large number of arguments in the transformers finetuning scripts make it significantly easier to work with as a command line function.  What are your thoughts on executing the script as a shell command in your notebook using the special ! character?

You should be able to launch the script in a notebook as follows:
!sparseml.transformers.question_answering \
  <ARGUMENTS> \
  ...

hi @pacospace thank you for the great suggestion.

Thanks @bfineran for the answers!

for the export_onnx script, we do have a method in our API that exposes this functionality: from sparseml.transformers import export_transformer_to_onnx
the large number of arguments in the transformers finetuning scripts make it significantly easier to work with as a command line function.  What are your thoughts on executing the script as a shell command in your notebook using the special ! character?

You should be able to launch the script in a notebook as follows:
!sparseml.transformers.question_answering \
  <ARGUMENTS> \
  ...

Yes I thought about that option but I thought with a python method, I can also manipulate inputs if I want to schedule different trainings iteratively, can you do that with ! character?

you can interpolate python variables into ! commands using the $ character, you can also loop the ! commands as well:
directories = ["dir_1",  "dir_2"]

for directory in directories:
    !ls $directory

see: https://stackoverflow.com/questions/35497069/passing-ipython-variables-as-arguments-to-bash-commands for more


Readme Plugin: add support to loading images in readme

Describe the feature
At the moment, when the readme.md file has an image, it doesn't load correctly in the backstage plugin, it only loads in GitHub.
This request is about adding images support to the readme backstage plugin.
Added value
There is currently no workaround, unfortunately the image loads as a broken file (screenshot below). We rely heavily on having images and diagrams in the readme of the repos.
Additional context
the code of the md file looks like this:
# Testing

refersh pls

![aiven-logo.png](docs%2Faiven-logo.png)

The plugin is rendered like this:

Hi @wanisfahmyDE! Sorry for the late reply. We welcome all fixes to this, would be great! Has also been bugging me. :100:


Login with trakt and sync history in movies

I've implemented these features:

login with Trakt in web-view and get access token
get movie history by TraktID
Now user can add movies to Trakt history

if you have any feedback, I would be happy to hear it.
#16

Dear @hadi-norouzi, just wanted to say a big thanks for the awesome pull request you dropped in this repo. It's clear you've put a lot of effort into it, and it really stands out. Can't wait to dig into the details!
Currently, I have some urgent commitments to attend to this week, which means I won't be able to get to your PR right away. However, I want to assure you that your PR is a priority for me, and I plan to allocate dedicated time for a review as soon as possible. I expect to be able to do this by the end of next week.
Thanks again for your hard work and dedication to the project. Your contributions are really valuable to me.

Dear @hadi-norouzi, just wanted to say a big thanks for the awesome pull request you dropped in this repo. It's clear you've put a lot of effort into it, and it really stands out. Can't wait to dig into the details!
Currently, I have some urgent commitments to attend to this week, which means I won't be able to get to your PR right away. However, I want to assure you that your PR is a priority for me, and I plan to allocate dedicated time for a review as soon as possible. I expect to be able to do this by the end of next week.
Thanks again for your hard work and dedication to the project. Your contributions are really valuable to me.

Dear @hadi-norouzi, just wanted to say a big thanks for the awesome pull request you dropped in this repo. It's clear you've put a lot of effort into it, and it really stands out. Can't wait to dig into the details!
Currently, I have some urgent commitments to attend to this week, which means I won't be able to get to your PR right away. However, I want to assure you that your PR is a priority for me, and I plan to allocate dedicated time for a review as soon as possible. I expect to be able to do this by the end of next week.
Thanks again for your hard work and dedication to the project. Your contributions are really valuable to me.

It's my pleasure to work with you and I wish to be more valuable to this project.

Dear @hadi-norouzi
I just wanted to say I'm really sorry for getting to your PR so late. I know you put effort into this, and it wasn't cool to keep you waiting. Thanks for your patience.
Currently, you might not be in the mood to revise and update the existing PR, so I kindly request that you simply create issues where I've noted in the comments. This way, we can enhance the code later based on these new issues.
Best, Reza
I've no problem at all. I've just waiting for your youtube videos 😄.

@hadi-norouzi, could you kindly reach out to me via Telegram (@moallemi) or Discord to discuss a few matters?


rename aws plugin

We renamed the plugin today, the older 'version' has been deprecated.

This PR is mainly meant as announcement to pybuilder devs, that we have renamed our plugin.

@esc just out of curiosity - are you handling EC2, IoT, S3 and Container Services from AWS plugin as well? If not, maybe the name should stay?

No, but the plan is to extend it to allow anything we may encounter. We have already renamed everything, so are beyond the point of no return. ;)

Someone to merge this?


In BrokerRequestToQueryContextConverter, remove unused filterExpressionContext

Codecov Report

Merging #8238 (64d7efc) into master (21632da) will decrease coverage by 1.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #8238      +/-   ##
============================================
- Coverage     71.00%   69.93%   -1.07%
  Complexity     4320     4320
============================================
  Files          1629     1629
  Lines         85132    85130       -2
  Branches      12812    12812
============================================
- Hits          60445    59535     -910
- Misses        20526    21463     +937
+ Partials       4161     4132      -29

Flag
Coverage Δ

integration1
28.68% <100.00%> (-0.02%)
:arrow_down:

integration2
?

unittests1
67.37% <100.00%> (-0.01%)
:arrow_down:

unittests2
14.15% <0.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

...xt/utils/BrokerRequestToQueryContextConverter.java
98.38% <100.00%> (-0.03%)
:arrow_down:

...t/core/plan/StreamingInstanceResponsePlanNode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ore/operator/streaming/StreamingResponseUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ager/realtime/PeerSchemeSplitSegmentCommitter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/core/plan/StreamingSelectionPlanNode.java
0.00% <0.00%> (-88.89%)
:arrow_down:

...ator/streaming/StreamingSelectionOnlyOperator.java
0.00% <0.00%> (-87.81%)
:arrow_down:

...re/query/reduce/SelectionOnlyStreamingReducer.java
0.00% <0.00%> (-85.72%)
:arrow_down:

...pache/pinot/common/utils/grpc/GrpcQueryClient.java
0.00% <0.00%> (-80.86%)
:arrow_down:

...ller/api/access/BasicAuthAccessControlFactory.java
0.00% <0.00%> (-80.00%)
:arrow_down:

...roker/requesthandler/GrpcBrokerRequestHandler.java
0.00% <0.00%> (-78.58%)
:arrow_down:

... and 83 more

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 21632da...64d7efc. Read the comment docs.


openload.io extractor not working

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2016.06.14. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2016.06.14

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add -v flag to your command line you run youtube-dl with, copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):

C:\>youtube-dl --get-url --verbose  https://openload.co/embed/kUEfGclsU9o/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['--get-url', '--verbose', 'https://openload.co/embed
/kUEfGclsU9o/']
[debug] Encodings: locale cp1256, fs mbcs, out cp1252, pref cp1256
[debug] youtube-dl version 2016.06.14
[debug] Python version 3.4.4 - Windows-8.1-6.3.9600
[debug] exe versions: none
[debug] Proxy map: {}
ERROR: Unable to extract video URL; please report this issue on https://yt-dl.or
g/bug . Make sure you are using the latest version; type  youtube-dl -U  to upda
te. Be sure to call youtube-dl with the --verbose flag and include its complete
output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpa3ey2lj9\bu
ild\youtube_dl\YoutubeDL.py", line 676, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpa3ey2lj9\bu
ild\youtube_dl\extractor\common.py", line 342, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpa3ey2lj9\bu
ild\youtube_dl\extractor\openload.py", line 112, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpa3ey2lj9\bu
ild\youtube_dl\extractor\common.py", line 645, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract video URL; please report
this issue on https://yt-dl.org/bug . Make sure you are using the latest version
; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose
flag and include its complete output.

Duplicate #9706. Openload has deployed enhanced encryption and there's current no good solution to it.


Chat

Hey, I think this project is what I need to, I would like to chat with you about this, is there some discord server to talk or something similar?
Nice work.

Thank you. There's no discord server now. I'm traveling at the moment and not on a computer at a consistent time of day, so it might not be that helpful. I might try out a chat platform in a couple weeks.

We could try out github discussions now, though.


Documentation for "rust.customBuildConfigurations"

Version of VSCode: 1.8.1
Version of the extension: 0.3.5
OS: Windows 10
Description:
I cannot figure out how to use "rust.customBuildConfigurations".
Reading the code it seems to me that the elements should be strings, I tried "rust.customBuildConfigurations": ["--debug"] (there is no --debug flag for cargo, I was confused) but the command seems to do nothing, no error message.
I expected the command "Cargo: Build using custom configuration" to trigger a debug build. Am I doing it wrong or is it a bug?
Output of the "Rust logging" channel:
No output.

I figured it out :) I had no idea vscode would autocomplete the settings values while typing, very nice feature.
Here is the settings I am using now, working perfectly:
    "rust.buildArgs": [
        "--release"
    ],
    "rust.runArgs": [
        "--release"
    ],
    "rust.customBuildConfigurations":[
        {
            "title": "Debug",
            "args": []
        }
    ]

@anderejd,
Hi.
Thanks for you feedback.
I will add a page on wiki soon.
I will close the issue as soon as I add the page.

@anderejd,
I have added some pages on the wiki.
I am closing the issue.


Reqnroll Extension v2024.3.152 does not work on Visual Studio 17.11.4

Used Visual Studio
Visual Studio 2022
Are the latest Visual Studio updates installed?
Yes
Content of reqnroll.json (if present)
No response
Issue Description
When attempting to load a Reqnroll project on the latest version of Visual Studio 2022 with the latest Reqnroll extension installed, the extension will not load and functionality is not present.
Steps to Reproduce

Have a Reqnroll solution
Install latest Visual Studio 2022 updates
Install latest Reqnroll extension (v2024.3.152)
Extension functionality is not present.

Workaround
We have issued a hotfix, to the simplest workaround is to update the package to v2024.4.154 that should work.
An alternative workaround for this issue would be to downgrade the Visual Studio 2022 extension to the previous version.
The download for the previous version can be found here:
https://marketplace.visualstudio.com/_apis/public/gallery/publishers/Reqnroll/vsextensions/ReqnrollForVisualStudio2022/2024.2.93/vspackage
To downgrade, you'll need to

Uninstall the current Reqnroll for Visual Studio 2022 extension
Disable automatic updates (for now):

Then close Visual Studio 2022 and install the VSIX file downloaded in the URL above.

I can confirm that when downgrading to the previous release (2024.2.93), the extension works as expected on the same Visual Studio 2022 version:

anhtuan266 posted about this in the relevant discussion, which could suggest some kind of dependency issue?

Debugging the Visual Studio Extension locally, it seems to work fine on commit #e22. Might be something to do with updating dependencies @gasparnagy on #41a?

thanks guys for being so quick on this. Meanwhile, is there a way I can get 2024.2.93 version?

amazing. I just tried it. All works absolutely fine now :) Thanks again @UL-ChrisGlew
You guys are doing a fantastic job. Cant tell you how much relieved I'm by seeing this support. I was scared today, when after a couple of years since using Specflow last time, today when I saw it dead, I was like freaked out. Now I'm confident :)

@gasparnagy I'm seeing the below in the activity log when loading a project, could be related:

The issue is not reproducible on my machine (I have just upgraded to Visual Studio 17.11.4), so I decided to apply a hotfix and reverted the commit that is most probably causing the problem. My plan is to release this ASAP and diagnose & fix the original issue without the stress of users being blocked.
I need help:

There is a release candidate in the pipeline. For a few of those who experienced the issue, please download the generated VSIX try to install it (close VS and double click on the vsix file and follow the instructions) and inform if the extension loads correctly.
If I get positive responses, I will release the hotfix.
After that, again for those who can reproduce it, please try to analyze the issue. The mentioned assembly has been upgraded in the reverted commit, so it is surely related, but the details are unknown. It is generally very hard to diagnose such issues, but I have written earlier some notes about it in the CONTRIBUTION.md guide, please check it and hopefully you get closer to it.

@gasparnagy  - Tested that VSIX file, seems to fix the issue.
One thing I did notice with the previous version is that the version of System.IO.Abstractions we're now using has dependencies on other NuGet packages (TestableIO.System.IO.Abstractions and TestableIO.System.IO.Abstractions.Wrappers), which had assemblies missing from the output directory - possible there's more than just these missing?

@UL-ChrisGlew thx for the quick reaction. The hotfix (v2024.4.154) is currently being published, it will be available in the marketplace within a few minutes.
It can be that the package is just simply missing some additional assemblies, and maybe I have some other extension that deployed that and that is why it is not reproducible here.

@gasparnagy  Works for me too

@UL-ChrisGlew thx for the quick reaction. The hotfix (v2024.4.154) is currently being published, it will be available in the marketplace within a few minutes.
It can be that the package is just simply missing some additional assemblies, and maybe I have some other extension that deployed that and that is why it is not reproducible here.

Possibly yes, I did have to test this on another machine with just that Reqnroll extension installed in order to see what's going on with this. Unfortunately the code is a bit of out my wheelhouse so I'm not 100% confident in figuring out what's going on - I'll try and have another look though.

v2024.3.152 also had the same described issue on our side. Can confirm that the hotfix (v2024.4.154) is working as expected.

hotfix (v2024.4.154) works for me

I've managed to 'fix' and replicate a before/after with the PR #39.
Testing done:

Replicated the setup seen by some users in Visual Studio 2022 17.11.4 and Reqnroll.VisualStudio v2024.3.152
Updated the relevant files to add the missing .dll files.
Tested using an experimental environment that correct behaviour is restored.
Closed everything and installed the debug package into Visual Studio 2022.
Validated that correct behaviour is still observed.

It's obviously a bit risky as it's very tricky to get error output/etc from these sort of changes - but given the error I was seeing in the ActvityLog.xml has dissappeared - it's doing something right at least!

@gasparnagy I've had a play around and the extension does seem to work correctly with all updates re-applied and the missing files added back. Have re-raised #39 for review.

There is a CI build now that contains the proper fix by @UL-ChrisGlew, and I would like to release this soon to address the security issues. To avoid the situation we had last time, @orifn1, @jdb0123, @JimCooperFromGmailAccount could you please help trying it out with the VSIX of this CI build downloadable from here.
If it works, you can even keep it installed. It is basically the released version plus the problematic package updates.

it works for me

@UL-ChrisGlew Funny. Try to uninstall completely. Start VS. Wait until uninstall finished. Start VS again (so it updates caches). Close it again and after that try installing it from VSIX again.

I had that with the 152 update. VS gets a wee bit confused from time to time :-)
Uninstall it (it took 2 tries for me) and reinstall and it gets VS back to normal

On 19 Sep 2024, at 11:37, Chris Glew @.***> wrote:
I'm having a very strange issue installing that extension from the VSIX file - the VSIX is installed (I can see the files in the relevant folder), but VS 2022 isn't recognising it as installed.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Looks good for me too

On 19 Sep 2024, at 11:25, Gáspár Nagy @.***> wrote:
There is a CI build now that contains the proper fix by @UL-ChrisGlew, and I would like to release this soon to address the security issues. To avoid the situation we had last time, @orifn1, @jdb0123, @JimCooperFromGmailAccount could you please help trying it out with the VSIX of this CI build downloadable from here.
If it works, you can even keep it installed. It is basically the released version plus the problematic package updates.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.

Yeah, I've tried uninstalling the extension completely, then installing from the Extensions manager - and updating the VSIX from the file provided and no dice - just won't show up in Visual Studio :/

Okay, redownloaded the file and tried again and it worked okay - must have been a duff download 🤷

Thx. Just for the record, sometimes in stange installation problems you can also try devenv /ResetSkipPkgs from a VS console. That basically attempts to reload packages that were skipped earlier due to errors. (Or something like that.)


Config: dependency monkeyPatching

Landed in https://github.com/metarhia/impress/commit/1743a71e5157acb384a15663b45b7b6f68638c0e


UICONSET-169: Consortia manager - Duplicate authorization role

Purpose

UICONSET-169: Consortia manager - Duplicate authorization role

The corresponding PR is here https://github.com/folio-org/stripes-authorization-components/pull/33

Approach

Screenshots

https://github.com/user-attachments/assets/c29619b6-c9c8-42b4-9d63-71fd4cb76d6c

Pre-Merge Checklist
Before merging this PR, please go through the following list and take appropriate actions.

[ ] I've added appropriate record to the CHANGELOG.md
Does this PR meet or exceed the expected quality standards?

[ ] Code coverage on new code is 80% or greater
[ ] Duplications on new code is 3% or less
[ ] There are no major code smells or security issues

Does this introduce breaking changes?

[ ] If any API-related changes - okapi interfaces and permissions are reviewed/changed correspondingly
[ ] There are no breaking changes in this PR.

If there are breaking changes, please STOP and consider the following:

What other modules will these changes impact?
Do JIRAs exist to update the impacted modules?

[ ] If not, please create them
[ ] Do they contain the appropriate level of detail?  Which endpoints/schemas changed, etc.
[ ] Do they have all they appropriate links to blocked/related issues?

Are the JIRAs under active development?

[ ] If not, contact the project's PO and make sure they're aware of the urgency.

Do PRs exist for these changes?

[ ] If so, have they been approved?

Ideally all of the PRs involved in breaking changes would be merged in the same day to avoid breaking the folio-testing environment.  Communication is paramount if that is to be achieved, especially as the number of intermodule and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems, ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.

Please fix eslint issues

fixed


New package: ResearchAccelerator v1.0.0

Registering package: ResearchAccelerator
Repository: https://github.com/ParallelGSReg/ResearchAccelerator.jl
Created by: @adanmauri
Version: v1.0.0
Commit: f3e26ceca0c4d3c5fa53679807501ae0c4a618c2
Reviewed by: @adanmauri
Reference: https://github.com/ParallelGSReg/ResearchAccelerator.jl/commit/f3e26ceca0c4d3c5fa53679807501ae0c4a618c2#commitcomment-122887638
Description: ResearchAccelerator.jl is a Julia Package that provides broad-based support to accelerate applied research using feature selection / dimensionality reduction techniques.

[noblock]
Hi, congrats on the package! Would you perhaps consider a slightly more indicative name? LiteratureReviewAccelerator?


npkg is not working

npkg does not install anything. It just loads and goes back to the terminal.

Npkg is deprecated, don't use it.

Then, how do we install software?

Npm
El 11/7/2016 10:10, "SpaceboyRoss01"<EMAIL_ADDRESS>escribió:

Then, how do we install software?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/NodeOS/NodeOS/issues/274#issuecomment-231668269, or mute
the thread
https://github.com/notifications/unsubscribe/AAgfvjFajmUwQF5CDCWs5E_DqdVkiWdNks5qUfpkgaJpZM4JJBEO
.


AMRHF snap build of lxd is missing lxd-images script

By default with the provided snap, I haven't been able to successfully import any images, as it appears the lxd snap is missing lxd-images script.
(RaspberryPi2)ubuntu@rpi1:~$ lxd-images import ubuntu trusty arhmf --alias ubuntu
-bash: lxd-images: command not found

ah yeah, we probably ought to include lxd-images in there.
I tested it with:
lxc remote add images images.linuxcontainers.org
lxc launch images:ubuntu/trusty/armhf blah

ah excellent. i fetched the lxd-images script and it failed out with unable to find a running lxd service.
I'll re-test with the above suggested work around. Thanks @stgraber

Fixed in 0.15 snap.


Set default branch to 'main' instead of the Nextflow default 'master'

This came up in relation to changing the default branch within GitHub. I don't think that would have made a difference for Nextflow, because the hard coded default branch is 'master'. This commit changes that default for the clean pipeline to the 'main' branch.
See: https://www.nextflow.io/docs/latest/config.html#scope-manifest

Looks good to me!
(I never changed manifest.defaultBranch, e.g., in CovPipe2, which uses main instead of master and never had problems, but it makes sense for sure!)


Add "Get..." getter methods and remove "Must..." getter methods

In types I'd suggest to avoid the "Must..." prefix in getters, they make access to deeply nested attributes too verbose.

I would prefer to keep the current two methods, Attr() and MustAttr(), for getting attributes. And will add panic when the attributes not available.


InfoClickWin: Fire event on click

Often it is needed to listen to the feature click event from other components in Wegue. Therefore, it would be desirable to fire an event once a feature is selected.

Hi @JakobMiksch ,
Is this issue already solved? I was about to take this issue but found out that there is a util/MapInteraction.js that probably does the intended:
https://github.com/wegue-oss/wegue/blob/7f53a55545015d37c5b4fc73c8f9dba334240b7d/src/util/MapInteraction.js#L43-L50

Yes, IMHO this is exactly what is needed by this issue. But maybe @JakobMiksch could also confirm this.
One learning could be that an overview of the fired Wegue events should be listed somewhere. Maybe a markdown file as a simple start for this would work out as a first step.

Agreed, documenting on all the WguEventBus events sounds reasonable, if they are part of the public API we are going to rely on.
One of the things on my list to change is standardising the event signatures (could be done for the next Wgue major version). E.g. as an ad-hoc decision I sometimes coded the "event target" into the event name which is not a good design choice:
WguEventBus.$emit(this.moduleName + '-visibility-change', !this.show)
should rather become something like
WguEventBus.$emit('module-visibility-change', this.moduleName, !this.show)

I see, that seems fairly easy. To kick start this I did a quick search using WguEventBus.$.* and found the following events:

Event
Parameters

app-mounted
-

sidebar-scroll
component

map-selectionchange
lid, selected, deselected

ol-map-mounted
olMap

ol-map-unmounted
olMap

this.overlayId + 'update-overlay'
visible, position, data

this.moduleName + 'visibility-change'
visible

To complement the standardization @fschmenger, I was also wondering if the events:

should use the same tense? (i.e.: app-mounted vs app-mount);
anatomy should take the form of ${subject}-${verb} or any other fixed form? (i.e.: update-overlay would be overlay-update).

Yes, good catch. Another point would be to consistently relay event-targets either as an object or by it's id (only where applicable). So an argument can be made to change the event from above to
WguEventBus.$emit('map-selectionchange', layer, evt.selected, evt.deselected );
to be consistent with e.g. ol-map-mounted.
I propose we close the issue over here - if @JakobMiksch confirms - and move some of the content to a dedicated issue, e.g. "standardise event signatures"

Yes, the original reason for my issue was in a custom application that added the layer later on. Therefore the SelectInteraction was not added automatically.
@fschmenger I agree with @fschmenger 's proposal to close this issue and create a new one with the overview of the events


Adding .readthedocs.yaml file

Fixes #26

Added simple .readthedocs.yaml.

As this adds   fail_on_warning: true it'll need to wait until https://github.com/google/gf180mcu-pdk/pull/35 is merged.

I removed the fail_on_warning: true flag so that I can merge this pull request before we fix all the warnings.


feat!: Implement AllowedMentions object

About
This pull request adds AllowedMentions and AllowedMentionTypes objects to work with allowed_mentions
Checklist

[x] I've ran pre-commit to format and lint the change(s) made.
[x] I've checked to make sure the change(s) work on 3.8.6 and higher.
[ ] This fixes/solves an Issue (If existent):.

resolves #

I've made this pull request for/as: (check all that apply)

[x] Documentation
[x] Breaking change
[x] New feature/enhancement
[ ] Bugfix

I don't want this to be breaking, so please add an _._json if isinstance(_, Class) else _ condition to where allowed mentions is used

Okay. Will do. What about typehinting?

Keep current.
Optional[Union[AllowedMentions, Dict[str, list]]]
Optional[Union[AllowedMentions, Dict[str, List[int]]]]
Optional[Union[AllowedMentions, dict]]

I don't want this to be breaking, so please add an _._json if isinstance(_, Class) else _ condition to where allowed mentions is used

Okay. Will do. What about typehinting?

Keep current.
Optional[Union[AllowedMentions, Dict[str, list]]]
Optional[Union[AllowedMentions, Dict[str, List[Union[str, int]]]]]
Optional[Union[AllowedMentions, dict]]

Late response, sorry, 4 is fine


Change name of class from Queue to Buffer

https://github.com/grandeurtech/js-sdk/blob/6369f568aa086097a89af7870a86f11ab3b94c09/src/handlers/duplex.handler.js#L25

Solved in #23


Add polymer headers to the playground, feature playground on the homepage.

This change is

Reviewed 6 of 6 files at r1.
Review status: all files reviewed at latest revision, 2 unresolved discussions.

demos/playground/index.html, line 20 at r1 (raw file):

  <head>
    <meta charset="UTF-8">
    <!-- This script is stripped during vulcanization, available only during development. It provides

nit: since you are wrapping this comment anyhow,  make it 80 width

demos/playground/playground.html, line 32 at r1 (raw file):

../homepage/assets/support.js

might be good to remove this and re-add <script src="bundle.js> so that we work on phones.

Comments from Reviewable

Review status: 0 of 12 files reviewed at latest revision, 3 unresolved discussions, some commit checks pending.

demos/homepage/assets/style.css, line 234 at r2 (raw file):

.featured-demo div.mdl-card__title:before {
  height: 352px;
  background: none;

is this needed?

demos/homepage/assets/style.css, line 257 at r2 (raw file):

  font-size: 0.6em;
  letter-spacing: 0.5px;
  background-color: rgba(0, 0, 0, .54);

is this needed?

Comments from Reviewable

Review status: 0 of 12 files reviewed at latest revision, 4 unresolved discussions, all commit checks successful.

demos/homepage/assets/style.css, line 234 at r2 (raw file):

.featured-demo div.mdl-card__title:before {

nope!

demos/homepage/assets/style.css, line 257 at r2 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
is this needed?

This is just moved.

demos/playground/index.html, line 20 at r1 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…
nit: since you are wrapping this comment anyhow,  make it 80 width

Done.

demos/playground/playground.html, line 32 at r1 (raw file):
Previously, dsmilkov (Daniel Smilkov) wrote…

../homepage/assets/support.js

might be good to remove this and re-add <script src="bundle.js> so that we work on phones.

Will do this in a follow up.

Comments from Reviewable


Add .formatter.exs to the Hex package

import_deps: [:ex_cli] does not currently work since the .formatter.exs is not included in the Hex package. I’ve added it to the list to fix this issue.

Thank you! I just pushed this to hex.


docs: consistent fullstory_data_center references

Description
Fixes a couple issues with the docs.
Checklist before submitting PR for review

[x] This change requires a doc update, and I've included it
[x] My code follows the style guidelines of this project
[x] I have ensured my code is commented and any new terraform variables have proper descriptions

:tada: This issue has been resolved in version 0.1.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


refactor(src): reorganize directories by content

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.752%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
128

Relevant Lines:
322

💛  - Coveralls


When one pickup is parented below another, toposort the updates

Updating the child before parent results in a bit of jankiness, we should sort the parent to update first (assuming they use the same update phase)

TODO: How to mesh with OnPostLateUpdate?


Add caching for hamilton

The problem
We want to enable caching of functions and their downstream results.
Say we want to alter a function and rerun the entire DAG. The function that we want to alter runs late enough that we'd be redoing a significant amount of computation. While iterating could often be solved by executing individual nodes, its completely reasonable to iterate on the entire DAG.
In our internal use of Hamilton, we actually have a decorator called @cache that runs entirely separate from Hamilton -- this allows us to cache the results of individual functions. This decorator uses (a) the code a function runs and (b) the hash of the parameters. That said, its not foolproof -- changes in external libraries referenced within functions can get ignored, its not DAG-aware (it doesn't care about downstream functions you might also not want to rerun), and it depends on hashability of parameters.
I envision this as useful for:

Rerunning/iterating on a DAG locally
Running expensive DAGs in production ETLs that all use the same cache but change minimal parts

Some options

Automatically cache functions, have a clear_cache or use_cache in the execute function
Use the @cache decorator we have internally
Manage the cache externally -- pass in as an override to the driver's execute function. Then have a method on the driver to manipulate the cache as needed.

I'm partial to (1) although we need to make it visible to the user and easy to override. E.G. to mark things as changed. We could also have decorators that say dont_cache if needed.
Probably a few other things we can do -- welcome feedback! Might want to think about making it pluggable -- saving to disk is nice, but saving it to a backing store could be even nicer. Shouldn't get locked in.

Relying on the hash-ability of parameters can be dealt with in the way that Joblib does it - by using pickle.dumps to turn the parameter into bytes and then hashing the result. With of Pickle Protocol 5 (Python >3.8 or with a backport) this is actually very fast even with large Numpy arrays or Pandas DataFrames.
Regarding hashing, I've personally also found that for large data inputs xxhash.xxh3_128 can be a useful speed up - though it does introduce an additional dependency.
 Nice callout. Sorry for the delay. I think a big question is what we want to cache... If we cache at the configuration side (E.G. all sync nodes to a function), then we can do a downstream trace to determine what to rerun. Similar to the approach spark takes. The nice thing about this is we don't need to worry about hashing -- as we're hashing configuration items and not datasets.
TBD on what the user interface would look like for this -- I think an external caching layer could work nicely, as we expose this capability in function_graph to some extent.

I think caching should be controlled by the driver + adapter -- it's most useful when iterating/building a DAG. Once someone has this running in production, we already don't recompute by design, so not sure how much value it would be in production, other than for "resumeability" of workflows in case of error (e.g. like https://docs.ray.io/en/latest/workflows/concepts.html).
My motivating use case -- I'm iterating locally, and would like the earlier nodes to be cached, since I'm working on downstream ones. Setting that via the driver makes sense to me.

We are moving repositories! Please see the new version of this issue at https://github.com/DAGWorks-Inc/hamilton/issues/11. Also, please give us a star/update any of your internal links.


Eager macro-expansion failure: (error "Unknown rx form ‘anychar’")

Update to 9edd4f2, such an error occurred.
macOS mojave & emacs-plus 26.3

emacs --debug-init gets:
...
DOOM Incrementally loading tree-widget
DOOM [ui/window-select] Loading deferred package winum from doom--after-call-winum-h
DOOM [editor/evil] Initialized evil-collection-outline
Eager macro-expansion failure: (error "Unknown rx form ‘anychar’")
DOOM Incrementally loading recentf
DOOM Incrementally loading savehist
DOOM Incrementally loading yasnippet
DOOM [editor/evil] Initialized evil-collection-elisp-mode
...

Please include your M-x doom/info.
And please do not ignore the issue template in the future.

OK，In the future, if I have any questions, I will provide as much information as possible.
doom/info:

emacs   version    26.3
        features   RSVG DBUS GLIB NOTIFY ACL GNUTLS LIBXML2 ZLIB TOOLKIT_SCROLL_BARS MODULES THREADS LCMS2
        build      May 19, 2020
        buildopts  (--enable-locallisppath=/usr/local/share/emacs/site-lisp --infodir=/usr/local/Cellar/emacs-mac/emacs-26.3-z-mac-7.9/share/info/emacs --prefix=/usr/local/Cellar/emacs-mac/emacs-26.3-z-mac-7.9 --with-mac --enable-mac-app=/usr/local/Cellar/emacs-mac/emacs-26.3-z-mac-7.9 --with-gnutls --with-modules --with-rsvg)
        windowsys  mac
        daemonp    server-running
doom    version    2.0.9
        build      HEAD -> develop, origin/develop, origin/HEAD 0f3bd5710 2020-06-13 03:03:11 -0400
        dir        ~/.doom.d/
system  type       darwin
        config     x86_64-apple-darwin18.7.0
        shell      /bin/zsh
        uname      Darwin 18.7.0 Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT 2019; root:xnu-4903.271.2~2/RELEASE_X86_64 x86_64
        path       (/usr/local/bin /usr/bin /bin /usr/sbin /sbin /Applications/VMware Fusion.app/Contents/Public /Library/TeX/texbin /usr/local/go/bin /usr/local/MacGPG2/bin /opt/X11/bin<EMAIL_ADDRESS>~/bin ~/.pyenv/shims /usr/local/opt/libxml2/bin ~/.cargo/bin /usr/local/bin/gs /Applications/Wireshark.app/Contents/MacOS ~/.rvm/bin /usr/local/anaconda3/bin /usr/local/Cellar/go/1.13/bin /usr/local/opt/tcl-tk/bin /usr/local/Cellar/emacs-mac/emacs-26.3-z-mac-7.9/libexec/emacs/26.3/x86_64-apple-darwin18.7.0)
config  envfile    nil
        elc-files  0
        modules    (:input japanese :completion (company +childframe +auto) (helm +fuzzy) (ivy +fuzzy +icons) :ui workspaces deft doom doom-dashboard modeline doom-quit hl-todo nav-flash neotree treemacs (popup +all +defaults) unicode vc-gutter vi-tilde-fringe (window-select +switch-window +numbers) zen :editor (evil +everywhere) file-templates snippets fold lispy (format +onsave) multiple-cursors rotate-text word-wrap :emacs (dired +icons) electric vc (ibuffer +icons) (undo +tree) :checkers syntax spell grammar :tools eval lsp (lookup +dictionary +docsets) ansible debugger editorconfig macos make (magit +forge) pass pdf rgb terraform tmux upload :lang common-lisp data emacs-lisp ess hy scheme javascript julia (latex +latexmk +cdlatex +fold) ledger lua (markdown +grip +pandoc) (org +present +habit +babel +pomodoro +attach +capture +export +dragndrop +hugo +gnuplot +ipython +pandoc +brain +roam) raku plantuml ruby (sh +zsh) :email (wanderlust +gmail) :term eshell vterm :app calendar irc (rss +org) twitter :private myorg misc mylang :config (default +bindings +snippets +evil-commands))
        packages   (n/a)
        unpin      (n/a)
        elpa       (with-editor transient pcache marshal magit logito ht github-clone git-commit gh dash async)

No matter whether it is started with emacs --debug-init or M-x toggle-debug-on-error after emacs is started, no detailed error information can be traced, just that one.

It turned out to be a problem caused by updating org-roam, the latest commit of org-roam has solved this bug.


Leetcode Question Number 1431

Leetcode Question Number 1431 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Assign me


chore(package): update rollup to version 0.61.1

Closes #2296

Haven't merged greenkeeper PRs yet because the build keeps failing. We'll have to look at it first.

It is failing because cypress ><EMAIL_ADDRESS>><EMAIL_ADDRESS>is using TypeScript 2.8.
Even if we fix cypress version to 3.0.0, it still uses<EMAIL_ADDRESS>><EMAIL_ADDRESS>Working version:

After yarn upgrade:


nrf52840pdk: fix incorrect jtag configuration

resulting in inability to program last half of flash. Closes #1192

Thanks @andrzej-kaczmarek for the solution.


[core, react, vue]: Enhancement - Add Sepolia testnet

Description
Add Sepolia testnet to native support(chain icon and background color - both point to the Eth mainnet values)
Closes #1598
Checklist

[x] Increment the version field in package.json of the package you have made changes in following semantic versioning and using alpha release tagging
[x] Check the box that allows repo maintainers to update this PR
[x] Test locally to make sure this feature/fix works
[x] Run yarn check-all to confirm there are not any associated errors
[x] Confirm this PR passes Circle CI checks
[x] Add or update relevant information in the documentation

@Adamj1232 Thank you so much for the amazing work.
Just one more thing I noticed.
The notification callback in the notify package doesn't receive events for the Sepolia test net. We use this to record payments, analytics among other things
I wasn't sure where to reach out with this.

@lewis-kori We are very happy that you are enjoying web3-onboard!
We currently don't have transaction notifications running on Sepolia but it is on our roadmap and will be prepped as Goerli wind-down


Use existing Services

Hi, is it possible to use an existing Services (e.g. an existing PostgreSQL) with the Open Serivce Broker in K8s?

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

ID: 77905ec1-fd7e-d037-cac5-62a5b8075c1b
Version Independent ID: 239518e8-cb47-3fd3-9cc2-2238ef57cbc6
Content: Integrate with Azure-managed services using Open Service Broker for Azure (OSBA)
Content Source: articles/aks/integrate-azure.md
Service: container-service
GitHub Login: @sozercan
Microsoft Alias: seozerca

Thanks for the question! We are investigating and will update you shortly.

@florianrusch I believe this is the relevant issue: https://github.com/Azure/open-service-broker-azure/issues/585

Good find @sozercan!
@florianrusch I would suggest chiming in on the issue @sozercan mentioned. The team in that GitHub repo would best be able to assist you with your question. I am tracking that issue as well.


End game if team reveals all of their words

The game should end if a team guesses all of the words matching their team's color. That team becomes the winner.

Updated the game svc and bff. Still need a winning message in the game spa. @skinney87, you have time to add that?

Updated the game svc and bff. Still need a winning message in the game spa. @skinney87, you have time to add that?

Sure, I can work on the game_spa today. @deesejohn

Sure, I can work on the game_spa today. @deesejohn

@deesejohn Made a commit but forgot to tag it with this issue. At this point I just have the board replaced by a message once a team wins.

@deesejohn Made a commit but forgot to tag it with this issue. At this point I just have the board replaced by a message once a team wins.


TryAdd, Keyed, and TryAddKeyed Attributes

Add initial support for TryAdd, AddKeyed, and TryAddKeyed DI methods that are discussed in issues #5 and #6.

@patrickklaeren and @ristogod, here is an in-progress draft of what I'm working on to support the two other issues

@ChaseDRedmon Thanks. Looks good.

I'm going to open this PR as ready to review.
I need to write the other unit test cases for testing the new TryRegister, TryRegisterKeyed, and RegisterKeyed methods under the Tests project, but I think we could start reviewing the PR initially @patrickklaeren

Version bump would also need to be incremented to v1.4

Minor change for the parameter name in constructor.
Will just need summary of attributes being added. Readme and Wiki needs to updated once released, I will add that as a chore for me to do.

Summary:
Added 3 new attributes types

TryRegister
TryRegisterKeyed
RegisterKeyed

TryRegisterKeyed and RegisterKeyed methods take a ServiceKey argument where you must pass in a string type key. Failing to pass in a key, or setting an empty or whitespace key results in an argument exception that prevents the source generator from executing.
TryRegisterScoped, Singleton, and Transient attributes generate their corresponding extension methods: ServiceCollection.TryRegisterScoped, ServiceCollection.TryRegisterSingleton, and ServiceCollection.TryRegisterTransient
TryRegisterKeyedScoped, KeyedSingleton, and KeyedTransient attributes generate their corresponding extension methods: ServiceCollection.TryRegisterKeyedScoped("Key"), ServiceCollection.TryRegisterKeyedSingleton("Key"), and ServiceCollection.TryRegisterKeyedTransient("Key")
RegisterKeyedScoped, KeyedSingleton, and KeyedTransient attributes generate their corresponding extension methods: ServiceCollection.RegisterKeyedScoped("Key"), ServiceCollection.RegisterKeyedSingleton("Key"), and ServiceCollection.RegisterKeyedTransient("Key")


Support VAST video support for GAM and MoPub AppBidding

JIRA: DPP-3441

Note that coverage is low because test-utils coverage report is not generated when run from test module. See https://jira.criteois.com/browse/EE-1370 for more details

Just by curiosity : do you know where is the integration guide that explain how to setup the ad srever part ? (because I think that publishers will need to put some conditions there based on the new keyword)

The public doc is here: https://publisherdocs.criteotilt.com/app/android/app-bidding/google-ad-manager/#ad-server-setup
Currently there is nothing on video as the feature is not released yet


mousemask API safety

The pancurses::mousemask accepts a mutable pointer as a second argument and proceeds to pass this to an unsafe FFI.  Part of the point of a library around an FFI is to allow the Rust programmer to work without unsafe features, like pointers.
With this in mind, I was wondering if there was a reason that mousemask didn't just accept an Option<&mut mmask_t> for its second argument.  This solves both requirements of the pointer API, in that you have to be able to pass a "nonexistent" option (null pointer) or a reference to a modifiable mmask_t.  The advantage is that you no longer risk passing a non-null pointer that doesn't reference valid memory.
In fact, if I'm reading this correctly, the Option version of the interface could even be used in the extern function declaration, though that's outside the scope of this library.

Creating a pointer is 100% safe, and rust will auto-coerce a &mut T passed into a function into a *mut T on the function's side of things if necessary.
Using the pointer is the unsafe part, but the C code underneath isn't going to care anyway.

Creating a pointer is 100% safe, regardless of the value of that pointer. This means that I can pass in the address 0x0001, which (probably) doesn’t refer to valid memory. The result is it gets dereferenced in the C library, creating a segfault or worse.
So in reality, this is an unsafe function because it can’t guarantee memory safety.

Oh my! I didn't notice that the function was marked as safe! You're quite right then, this is a clear soundness issue.