Experiment with implementing AWQ for BERT models

If we can speed up the BERT model, we will significantly increase the throughput of many cases. Experiment with SentenceTransformers first.

Would love this for image captioning with quantized speedup.
The kosmos-2 model from Microsoft would be another good candidate

hej @casper-hansen I would be curious to implement this for https://github.com/michaelfeil/infinity. Do you see any road-blockers in regards to encoder only architectures? Will the GEMM kernels work for non-causal masked LMs?

@michaelfeil AWQ/GEMM kernels can work for any linear layer. However, there is a challenge in applying it to BERT models because it lacks some scaling methods. For example, we would usually scale from a layernorm to a linear layer.
See more about the scaling of layers here:
https://github.com/casper-hansen/AutoAWQ/blob/main/awq/quantize/scale.py
I also created a PR for better scaling for Mixtral, which may be interesting to you:
https://github.com/casper-hansen/AutoAWQ/pull/301


Default selinux option causes container startup failure on CoreOS 1353.7.0

Using CoreOS 1353.7.0 and kubernetes 1.6.4 on a fresh install, the default weave-kube addon fails to start the container when spc_t is set for selinux options.  Selinux is set to permissive.  Other containers running on the same host work properly.
Using the default weave-kube deployment via:
kubectl apply -f https://git.io/weave-kube-1.6

The weave container goes into a crash loop with the following info from the kubernetes event log:
2017-05-25 09:55:53 -0400 EDT   2017-05-25 09:55:53 -0400 EDT   1         weave-net-xxbsv   Pod                 Warning   FailedSync   kubelet, ip-172-17-115-69.ec2.internal
Error syncing pod, skipping: failed to "CreatePodSandbox" for "weave-net-xxbsv_kube-system(db406ff5-4151-11e7-aa4c-0a5efcdb0a60)" with CreatePodSandboxError:
"CreatePodSandbox for pod \"weave-net-xxbsv_kube-system(db406ff5-4151-11e7-aa4c-0a5efcdb0a60)\" failed: rpc error: code = 2 desc = failed to start sandbox container for pod \"weave-net-xxbsv\":
Error response from daemon: {\"message\":\"invalid header field value \\\"oci runtime error: container_linux.go:247: starting container process caused \\\\\\\"process_linux.go:359: container init caused \\\\\\\\\\\\\\\"write /proc/self/task/5543/attr/exec: invalid argument\\\\\\\\\\\\\\\"\\\\\\\"\\\\n\\\"\"}"

If I manually modify the deployment set and comment out the spc_t option, the container comes up clean:
      securityContext:
        seLinuxOptions:
          #type: spc_t

This error message is coming from runc (part of Docker).
Similar report: kubernetes/kubeadm#269


Image IDs are set incorrectly occasionally

on the igdb branch, occasionally the icon id is set incorrectly and changes the icon for multiple games. not sure if this happens with headers but given they have very similar functions I'd guess it does.

fixed in igdb

merged in #76


[Quetion] Is CP load balance supported for BGP mode?

Hi, from the doc it says

IPVS load balancing is configured for having the VIP in the same subnet as the control plane nodes

but seems doc not updated for quite a while.
Can someone confirm it's supported or not? If not, any plan to support it?
Thanks

Thanks for clarifying.
Yes, but there were downsides for routers LB as routers are designed to be stable instead of reacting fast for failures.


Does a realm need to be manually closed.

I am working on an app and by it's odd nature I need to work with multiple realm. So I'm planning on opening a realm in  function bodies with current config. My question is in this case will the realm object need manual closing in the end. Below is an example code snippet for better understanding.
suspend fun insertFoo(foo: Foo, config: CustomConfig) {
    val realm = Realm.forConfig(config)

    // Inserting with realm

   realm.close() // Is this necessary??
}

Hi, it is not necessary but instantiating a Realm on each function call would be a waste of resources. It would be better to keep a single instance for each Realm, and pass it to the insertFoo function:
suspend fun insertFoo(foo: Foo, realm: Realm) {
    // Inserting with realm
}

In Realm-Kotlin there are no thread confinement restrictions.

Looks ok, you might need to close the old instance before returning the new realm instance.


Add Cordova example

Add an example Cordova project that can be quickly built and tested by users.

Seconded. Thank you.

Mind putting up a quick example? Working on an Ionic project and stumped by no audio output when registering CordovaAudioPlugin.

Sorry I won't be able to get to this right way.
In the meantime, make sure you include the Cordova Media Plugin, account for the change in pathing for hosted files (like android/www/mysoundfilesfolder), and other wise all the code should be the same.

No worries then - interesting, still does not output audio. CPU usage jumps, and no errors are thrown.

I'm assuming if:
I can play a sound with the Cordova Media Plugin from a certain source, that source will work as a source when CordovaMediaPlugin is used?
Example:
var media = new Media('audio/00.wav'); // This will play from source.
createjs.Sound.registerSound('audio/00.wav', '00'); // However, this will not.
createjs.Sound.play('00', {volume: 1 }, 0, 0);

yes that should be correct.
in your play call, all arguments should be either comma separated or in an object.
{volume: 1, delay: 0, offset: 0}

Awesome that you made a plugin for Cordova, I'm going to use it and test it shortly. Hopefully this will eliminate the random sound loss that I get with Web Audio


Jail running (jls) but non-existent (at path) causes "Unable to execute id(1) in jail. Emulation or ABI wrong" error

Host: FreeBSD CURRENT-amd64 11.0-CURRENT r287299: Sun Aug 30 18:18:54 AEST 2015
Jail: FreeBSD snapshot dated 03/09/2015 (-m ftp, AU mirror)
Poudriere: poudriere-devel-<IP_ADDRESS>50903 (3.1.7 reported in  poudriere version)

Attempting to create a jail with the following invocation:
 sudo poudriere jail -c -j 11i386 -a i386 -v 11.0-CURRENT -m ftp

Results in the following error:
Error: Unable to execute id(1) in jail. Emulation or ABI wrong

The issue is/was not reproducible (new jail creation) with CURRENT amd64 jail on CURRENT amd64 host, only i386 on amd64.
Edit: Given the fix (deleting running jail) below in https://github.com/freebsd/poudriere/issues/346#issuecomment-138762484 , the issue root cause is unlikely to be arch specific, but I'll leave the above details intact.
Full poudriere log is:
[00:00:00] ====>> Creating 11i386 fs... done
/storage/poudriere/jails/11i386/fromftp/MANIFE100% of  549  B  960 kBps 00m00s
[00:00:00] ====>> Fetching base.txz for FreeBSD 11.0-CURRENT i386
/storage/poudriere/jails/11i386/fromftp/base.t100% of   68 MB 1484 kBps 00m47s
[00:00:48] ====>> Extracting base.txz... done
[00:00:58] ====>> Fetching src.txz for FreeBSD 11.0-CURRENT i386
/storage/poudriere/jails/11i386/fromftp/src.tx100% of  124 MB 1471 kBps 01m27s
[00:02:25] ====>> Extracting src.txz... done
[00:02:48] ====>> Cleaning up... done
[00:02:48] ====>> Recording filesystem state for clean... done
[00:02:48] ====>> Upgrading using ftp
/etc/resolv.conf -> /storage/poudriere/jails/11i386/etc/resolv.conf
mount: /storage/poudriere/jails/11i386/.p: No such file or directory
jail: "11i386-default" already exists
jail: "11i386-default-n" already exists
[00:02:48] ====>> Error: Unable to execute id(1) in jail. Emulation or ABI wrong.
[00:02:48] ====>> Error while creating jail, cleaning up.
[00:02:48] ====>> Removing 11i386 jail... done

I'm not sure how the system ended up in this state, but jls(8) reports:
   JID  IP Address      Hostname                      Path
   119  <IP_ADDRESS>       11i386-default                /storage/poudriere/jails/11i386
   120  -               11i386-default                /storage/poudriere/jails/11i386

Though there is no such path in the jails directory:
ls -la /storage/poudriere/jails
total 9
drwxr-xr-x   7 root  wheel   7 Sep  9 12:04 ./
drwxr-xr-x   5 root  wheel   5 Mar 29 13:20 ../
drwxr-xr-x  18 root  wheel  22 Sep  9 00:39 102amd64/
drwxr-xr-x  18 root  wheel  22 Sep  9 00:45 102i386/
drwxr-xr-x  18 root  wheel  22 Sep  9 01:05 11amd64/
drwxr-xr-x  18 root  wheel  22 Aug  4 20:29 93amd64/
drwxr-xr-x  18 root  wheel  22 Aug  4 20:33 93i386/

I'm not sure to what extent, or if the jail supposedly existing is related to the Emulation or ABI wrong error, but it may be the case that poudriere needs additional jail check routines to robustly handle this and other cases like "some leftover jail bits may still exist", resulting in a new jail being created as expected.

After deleting the jails listed by jls(8) with jail -r 119 and jail -r 120, the creation of the 11i386 jail succeeded:
 sudo poudriere jail -c -j 11i386 -a i386 -v 11.0-CURRENT -m ftp
[00:00:00] ====>> Creating 11i386 fs... done
/storage/poudriere/jails/11i386/fromftp/MANIFE100% of  549  B  957 kBps 00m00s
[00:00:01] ====>> Fetching base.txz for FreeBSD 11.0-CURRENT i386
/storage/poudriere/jails/11i386/fromftp/base.t100% of   68 MB 1602 kBps 00m43s
[00:00:44] ====>> Extracting base.txz... done
[00:00:52] ====>> Fetching src.txz for FreeBSD 11.0-CURRENT i386
/storage/poudriere/jails/11i386/fromftp/src.tx100% of  124 MB 1628 kBps 01m19s
[00:02:11] ====>> Extracting src.txz... done
[00:02:33] ====>> Cleaning up... done
[00:02:34] ====>> Recording filesystem state for clean... done
[00:02:34] ====>> Upgrading using ftp
/etc/resolv.conf -> /storage/poudriere/jails/11i386/etc/resolv.conf
mount: /storage/poudriere/jails/11i386/.p: No such file or directory
Looking up update.FreeBSD.org mirrors... 5 mirrors found.
Fetching public key from update6.freebsd.org... failed.
Fetching public key from update2.freebsd.org... failed.
Fetching public key from update5.freebsd.org... failed.
Fetching public key from update4.freebsd.org... failed.
Fetching public key from update3.freebsd.org... failed.
No mirrors remaining, giving up.
11.0-CURRENT
[00:02:39] ====>> Recording filesystem state for clean... done
[00:02:39] ====>> Jail 11i386 11.0-CURRENT i386 is ready to be used

This appears to support the theory that additional or more robust check routines are required to account for other possible system states during jail creation, in this case, where:

Path to jail does not or no longer exists
Jail is still running as reported by jls(8) ?

Please modify the title of this issue according to your assessment

mount: /storage/poudriere/jails/11i386/.p: No such file or directory
jail: "11i386-default" already exists
jail: "11i386-default-n" already exists
[00:02:48] ====>> Error: Unable to execute id(1) in jail. Emulation or ABI wrong.

Note the first error. The 'id' claim is a red herring and led you down the wrong path for support.

@bdrewery absolutely :)

My mistake for skimming IRC as well and missing it!

The biggest problem here is that the jail sub-command has very little error checking. It ignores errors that can lead to confusing behavior like this.

I have the same error on an amd64 system trying to create an amd64 jail.
root@poudriere:~ # poudriere jail -c -j 102x64 -v 10.2-RELEASE -a amd64
[00:00:00] ====>> Creating 102x64 fs... done
/usr/jails/poudrierebuilds/jails/102x64/fromft100% of  782  B   17 MBps 00m00s
[00:00:01] ====>> Fetching base.txz for FreeBSD 10.2-RELEASE amd64
/usr/jails/poudrierebuilds/jails/102x64/fromft100% of   64 MB   83 MBps 00m01s
[00:00:02] ====>> Extracting base.txz... done
[00:00:08] ====>> Fetching src.txz for FreeBSD 10.2-RELEASE amd64
/usr/jails/poudrierebuilds/jails/102x64/fromft100% of  118 MB   95 MBps 00m01s
[00:00:10] ====>> Extracting src.txz... done
[00:00:24] ====>> Fetching games.txz for FreeBSD 10.2-RELEASE amd64
/usr/jails/poudrierebuilds/jails/102x64/fromft100% of  865 kB   12 MBps 00m00s
[00:00:25] ====>> Extracting games.txz... done
[00:00:25] ====>> Fetching lib32.txz for FreeBSD 10.2-RELEASE amd64
/usr/jails/poudrierebuilds/jails/102x64/fromft100% of   16 MB   50 MBps 00m00s
[00:00:25] ====>> Extracting lib32.txz... done
[00:00:27] ====>> Cleaning up... done
[00:00:27] ====>> Recording filesystem state for clean... done
[00:00:27] ====>> Upgrading using ftp
/etc/resolv.conf -> /usr/jails/poudrierebuilds/jails/102x64/etc/resolv.conf
jail: jail_set: Operation not permitted
jail: jail_set: Operation not permitted
[00:00:27] ====>> Error: Unable to execute id(1) in jail. Emulation or ABI wrong.
[00:00:27] ====>> Error while creating jail, cleaning up.
[00:00:27] ====>> Removing 102x64 jail...rm: /usr/jails/poudrierebuilds/jails/102x64/dev/fd: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev/led: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev/usb: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev/gpt: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev/gptid: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev/pts: Operation not supported
rm: /usr/jails/poudrierebuilds/jails/102x64/dev: Device busy
rm: /usr/jails/poudrierebuilds/jails/102x64: Device busy

I've tried deleting the jails and the zfs file systems associated with them and recreating the jails, but it makes no difference. I keep getting that error.
In my case, I am running poudriere inside a jail and have mounted the zfs datasets inside the jail.
Poudriere creates the sub datasets without a problem.


Data store

[ ] Document Evolvable::DataStore in README

Probably going to go a different route and just add minimal serializer support to this gem. Something like this could be cool extracted out into another gem after serialization is added.


Router floods network with Child Update Requests after reset

We see a suspicious behavior of a router/leader after reset. If the router had any children then, after reset, it will send a child Update Request to each of them. The issue is that it keeps doing so every second, for about 240 seconds (I guess until all children are removed from the child table). Is that an intended?
Steps to reproduce:

start a device and wait until it promotes to a leader
start another device and wait until it attaches as a child
switch off the second device
reset the first device and observe the issue

The mechanism you are observing is intended to implement the child restoration feature after a reset, as discussed in Thread 1.1 Section 4.7.3.  I'll review the implementation again to see if/how we can improve.

@jwhui Yes, I'm aware of that. The issue is that the current implementation causes quite a lot of traffic (4 frames / child) every second. My understanding of Section 4.7.3 is that it's sufficient to send a single Child Update Request after a reset.

If the implementation is sending 4 frames/child every second, I agree that is a problem.  At the same time, I'm not sure one is that robust since messages can get lost (albeit with low probability).  I think some number between 2 and 4 is sufficient.

From a brief code inspection and info from my colleagues it looks like there is one Child Update Request / per child / per second generated in mle_router. 4 frames per request are due to MAC retransmissions.
I think it should be sufficient to generate a single Child Update Request / per child sometime after reset and leave to MAC to deal with potential frame loss.

Ah, I see what is going on.  I believe the current logic was written with the SED in mind.  For SEDs, the desire was to keep trying to send Child Update Request messages for as long as the SED's timeout.  We felt this was OK given that transmissions are naturally rate limited by the SED's polling.
However, the current logic fails to change that behavior for rx-on-when-idle Children.  I'm fine with adjusting the logic such that it tries only once or twice for rx-on-when-idle Children.  However, I see value in continued attempts with SEDs.
One of the challenges with IEEE 802.15.4 is that received ACKs can lead to false positives (the ACK only includes an 8-bit DSN), especially in dense environments.  That is why retransmissions are not always sufficient.  False positives occur with low probability, but we do see them in practice.


good_web_game::graphics::draw not working correctly on Windows

In my game I am doing something like this in order to draw multiple rectangles using a single Mesh instance (in different destinations of course, but I deleted that code for brevity):
graphics::draw(
                ctx,
                quad_ctx,
                &self.my_mesh,
                graphics::DrawParam::default(),
            )
            .unwrap();
graphics::draw(
                ctx,
                quad_ctx,
                &self.my_mesh,
                graphics::DrawParam::default(),
            )
            .unwrap();

This works fine on Linux (Ubuntu 22.04)

However when running the same code on Windows it looks like this:

I don't know what the issue is, but it seems only one (the last) out of three rectangles got drawn.
Also there were similar issues with drawing text, where text from one draw call got mixed up with another, but I fixed that by using the queue_text function.

Hmm... thanks for sharing.
Does the blend modes example run properly for you? It too draws a single mesh multiple times similarly to what you're doing and last time I checked it ran fine on Windows (but I guess I'm gonna re-check just to be sure).

Yes, the blend modes example is working fine - it is behaving the same way as it is on Linux.
We can take a look at another issue. I added a debug draw mode, where the grid for collisions is drawn (and some other stuff aswell eg. number of objects drawn, player's grid index, guard's rays).
This is correctly drawn on Linux:

However on Windows it looks like this:

The grid is actually flashing at a higher rate, but since I lowered the fps you can't really see that, however it gets the point across.
You can also see the text not being drawn correctly in the top-left, if you compare it to the one being drawn on Linux (I didn't use graphics::queue_text here). It says "player grid" instead of number_of_objects_drawn / total_number_of_objects. Upon further inspection I noticed the text was actually drawn correctly on the first frame of debug draw, but it gets replaced with "player grid" on all the other frames.

Ok, my assumption here is that these bugs are related to https://github.com/ggez/good-web-game/issues/59, or, more precisely, the fix I used for it.
I added this to keep bindings of dynamically allocated meshes alive for one more frame before being dropped, which solved a bug particular to Windows and Nvidia graphics as far as I know: Just dropping the bindings after they were used wasn't really workable there, as their meshes wouldn't get drawn then, probably due to the necessary resources having been released before they were no longer needed.
I realize that this is a very crude twigs-and-duct-tape fix to the problem though.
In your case it might or might not help to simply crank the number of frames (currently 1) up by one or two. If that works I would merge that change into the master branch, in order to hopefully avoid such problems for others (though I realize it's still just twigs and even more duct tape, but hey, at least somewhat of an improvement, maybe).
A more reasonable approach might be to not create, use and then drop the meshes inside of your draw or update cycle, but to instead have them created once and then stored somewhere, until you're sure that you no longer need them.
One advice that I'd definitely give you is to try out the MeshBatch struct for drawing a single mesh multiple times, as it uses instanced drawing, thereby probably improving performance and possibly even solving your problem all on its own (though this is just a wild guess).

You were right, using the (MeshBatch)[https://docs.rs/good-web-game/latest/good_web_game/graphics/struct.MeshBatch.html] solved the issue.
Since this worked, I didn't fork gwg and do what you said, so I unfortunately cannot give any feedback for that.

Glad to hear that :)
I'll still keep this open as it's likely that someone might run into such an issue again.


Support Azure Functions

What is the feature you are proposing?
It would be amazing it Hono supported Azure Functions.
Being able to change back-and-forth between providers is something needed.
For example, from Azure Functions → Cloudflare Workers.

Thank you!


[Security question] Extracting HW-bound keys of Caliptra

Reference commit: 46d8b6e9cfd0020d485012cf261a21278a5e8671
Related SV files: mldsa_ctrl.sv
High-level summary
If I understand correctly, when integrated into Caliptra, one of the security properties ABR wants to preserve is the secrecy of HW-bound keys (i.e. the seed originating from Key Vault IP) against Caliptra SW. In other words, ABR wants to make sure that no logical path is introduced to Caliptra with ABR that SW can exploit to extract HW-bound secrets. That should also include the intermediate secret values appearing in ML-DSA algorithms that give an attacker the signing capabilities. Such example values would be s_1, s_2 vectors, which are also part of the secret key.
I reach to this security goal assumption from the fact that there is a dedicated Key Vault IP in Caliptra and that the ABR spec says "If seed comes from key vault, this register will not contain the private key to avoid exposing secret assets to software." (Section 1.10).
I am thinking about the following scenario, and my question is whether the HW-bound secret can be leaked. In particular, I am thinking about leaking s_1 and s_2 values through SW interactions (or simply the signing key as defined in ML-DSA).
If SW is compromised, I suspect it can do the following to extract the secret signing key (originating from Key Vault IP) from ABR:

SW loads the seed value from Key Vault (by configuring mldsa_kv_rd_seed_ctrl register).
SW starts the KeyGen operation and waits until the operation completes.
In particular, SW does not run zeroize command.
SW reads PRIVKEY_OUT register to extract HW-bound signing key.

Low-level details
In the following line:
https://github.com/chipsalliance/adams-bridge/blob/46d8b6e9cfd0020d485012cf261a21278a5e8671/src/mldsa_top/rtl/mldsa_ctrl.sv#L585
it seems to me that this line disables reading PRIVKEY_OUT register in two cases:

In the middle of the KeyGen operation (which is good),
when KeyGen is completed but the seed is coming from KeyVault. So I interpret this as an attempt to prevent SW from reading these values at the end of the KeyGen operation (mldsa_privkey_lock signal). This part does not seem right to me.

My understanding is that mldsa_privkey_lock goes down few clock cycles after KeyGen is completed. I trace this signal to mldsa_valid_reg in RTL eventually, if I am not mistaken. I think this signal goes down after few clock cycles KeyGen operation is completed:
https://github.com/chipsalliance/adams-bridge/blob/46d8b6e9cfd0020d485012cf261a21278a5e8671/src/mldsa_top/rtl/mldsa_ctrl.sv#L187
Question
I am wondering if SW can extract HW-bound ML-DSA signing key by simply running KeyGen and reading PRIVKEY_OUT register.
@mojtaba-bisheh @ekarabu

Hi Fatih, yeah, we need to wait for zeroize to clear mldsa_privkey_lock.
thank you!

Thanks for the quick response @mojtaba-bisheh.
I had a quick look at ECC IP of Caliptra to see if the same idea applies for EC-DSA keygen and signing. As far as I understand, ECC IP is secure against this attack idea, because when Keygen operation is run, PRIVKEY_OUT register does not latch the signing key. The signing key is handled through another register.

in ecc, we have dest_kv signal that restricts privkey output from fw reads, but here we don't have that signal since privkey will not be stored in KV

For EC-DSA, does Caliptra use deterministic signatures?
Is there anything that prevents SW to pick its own nonce and have a message signed with Key Vault backed seeds? Then SW could compute the private/signing key because it knows the nonce used during signing. Is there also an attempt to prevent SW attackers that push the effort to extract keys a bit higher?

caliptra just supports deterministic ECDSA.
FW nonce is used as an input for HMAC-DRBG along with seed that comes from KeyVault to generate privkey/signing key.
Let's discuss ECDSA topics in Calitpra repo https://github.com/chipsalliance/caliptra-rtl to keep track of conversation.


Chrome Physical Web problem

I have multiple beeks beacons and have programmed them through the beeks app. However, when I go to the chrome physical web widget, I do not see the beacon and/or the URL. It just says the physical web is on and shows nothing else. I am on the latest version of IOS and all of my apps are updated. This is also happening for my coworker, we have the same phone (iPhone 6). Any feedback is appreciated.

I suggest you try these steps from my blogpost on this subject:
http://blog.beeem.co/2016/03/top-4-things-to-watch-out-for-on-chrome-for-android-with-beacons/
Let me know if you still have issues!

We are very close to releasing a developer page that will diagnose problems
like this. However, 99% of the problems are solved by checking the
following things:

Does your website use HTTPS?
Does your website block google.com w/ robots.txt?
Is your website behind a login prompt/firewall?

On Wed, Aug 10, 2016 at 12:25 PM, Ferenc Brachmann<EMAIL_ADDRESS>
wrote:

I suggest you try these steps from my blogpost on this subject:
http://blog.beeem.co/2016/03/top-4-things-to-watch-out-for-
on-chrome-for-android-with-beacons/
Let me know if you still have issues!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/777#issuecomment-238976289,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABAbuqeGfR1BE2rHCyLzR0ytDkE2qDCXks5qeiWwgaJpZM4JhTE3
.

Thanks, I'll try those ideas out!

Hi All,
@scottjenson, unfortunately I think this page could be on 1% :
https://alelonoconarh.com.br/welcome/index.php
it has HTTPS, there is no robots.txt or any other HTTP header blocking the
crawler and it is not behind a firewall or there is no login page.
Do you have any guess what might be happening?
Thanks in advance!
-Jeff
On Wed, Aug 10, 2016 at 4:51 PM, gconno3<EMAIL_ADDRESS>wrote:

Thanks, I'll try those ideas out!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/777#issuecomment-238983493,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgjWVTwgc1ZXRhOqoX4ZesvbPw3ySAoks5qeivcgaJpZM4JhTE3
.

@jeffprestes special characters in the title/description maybe?
alternatively you can try embeding this into a Beeem landing page with the URL tool or iframe-it into an info tool!

When I try that link, I get a 404. That would certainly do it ;-)

@scottjenson It is not working now, but it worked fine 2 hours ago.

I don't believe the issue is necessarily the website, as it works on our lab's iPad and one of my other coworkers iPhones. I guess what I'm trying to say is that it shows up on some of our devices but not others.

It's hard to diagnose a problem when the website comes and goes. The fact that I can't see it is not a good sign. A more objective test would be to plug it into a web page tester, such as https://www.webpagetest.org and see how it performs. If they can't see it, it confirms you have some type of DNS problem.

Hi @scottjensen and all,
We had changed it to static .html for test purposes but as well mentioned
by Ferenc it was working.
But even https://www.alelonoconarh.com.br/welcome/index.html
https://alelonoconarh.com.br/welcome/index.html it doesn't work either.
The short URL is: http://bit.ly/2blqd7i
I had reviewed all docs, tested it against webpagetest.org (see the
results: https://www.webpagetest.org/result/160811_ME_19D8/ ) it worked.
Tested it using Physical Web App Android and iOS and it worked. It worked
at all those places and apps except Chrome.
I can't find a technical reason for this.
I appreciate your assistance and review on this issue.
Thanks,
-Jeff
On Thursday, August 11, 2016, Scott Jenson<EMAIL_ADDRESS>wrote:

It's hard to diagnose a problem when the website comes and goes. The fact
that I can't see it is not a good sign. A more objective test would be to
plug it into a web page tester, such as https://www.webpagetest.org and
see how it performs. If they can't see it, it confirms you have some type
of DNS problem.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/google/physical-web/issues/777#issuecomment-239169712,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgjWfsvz_LGrFE5R11Jne8dVQJSosP-ks5qeytXgaJpZM4JhTE3
.

Sorry for the delay. I've just run this through verify.physical-web.org and everything works, but slowly. This implies that the redirection is taking a long long. If it takes too long, the phone client will not display it (but it will show up on a 'rescan'. I'll close this issue for now but if you still have an issue, please reopen.


Remove Kustomize integration

Fixes #1226.
Fixes #1241.

A fix has been found: https://github.com/projectriff/riff/pull/1247.

Closing in favor of #1247


Added 'set_aspect' and set_aspect_equal function

Added a "set_aspect(Numeric ratio)" and "set_aspect_equal()" function.
Nothing too fancy. Respectively calls :
matplotlib.pyplot.gca().set_aspect(ratio)
and
matplotlib.pyplot.gca().set_aspect('equal')
I like the simplicity of your wrapper. Keep up the good work :)

Thanks for contributing!


Application language sets to Chinese and not possible to change?

My operating system is in French and it seems the app picks Chinese as default language. It should instead pick English as default.
Also it looks like once the language is set it's not possible to change it?

Can you check your Language & Region system prefs. There should be a list of preferred languages. Ensure that English is set higher precedence than Chinese (see screenshot).
If not can you send me a screen shot of      your own settings, it may help me understand the issue.

Can you check your Language & Region system prefs. There should be a list of preferred languages. Ensure that English is set higher precedence than Chinese (see screenshot).
If not can you send me a screen shot of      your own settings, it may help me understand the issue.

(By the way, we now have a French translation which will ship with the next update!)

(By the way, we now have a French translation which will ship with the next update!)

Indeed I had Chinese in the list without realising it, probably because I've tested pinyin input not long ago. So the behaviour actually makes sense, and I'm going to add English to the list to avoid this kind of issue in the future. Thanks for the info!

Indeed I had Chinese in the list without realising it, probably because I've tested pinyin input not long ago. So the behaviour actually makes sense, and I'm going to add English to the list to avoid this kind of issue in the future. Thanks for the info!


Close a resource to avoid resource leak

Please consider closing a resource to avoid resource leak
scanner is not closed in Main::main(.)

Not enough information provided.


Atomic gets that depend on each other

TLDR what I want to do is something like this, but atomic:
k2 = CubDB.get(db, {:index, k1})
v = CubDB.get(db, {:stuff, k2})

In other words, I'm taking the value of one key and using it as part of another key, which doesn't seem to be possible with current APIs. Now for my use case the atomicity isn't essential, so I could just use the double gets. However, some support for this would be nice to have regardless...

Hi @refi64 ,
you are right in that, while it is possible to get multiple "independent" entries atomically, there is currently no API to atomically get two entries, with the second key depending on the first. I will think about this use case, it would be great to enable it.

Hi @refi64 , I know this comes quite a long time after this issue was open, but this pull request (currently in draft) should meet your original need by introducing snapshots: https://github.com/lucaong/cubdb/pull/47

This is now in v2.0.0-rc.1, and will soon be released as part of v2.0.0


Run C++ version on webcam

Hi Daniil-Osokin,
Thank you for your amazing work :-)
I actually generated the OpenVINO .xml human-pose-estimation model, and I would like to test it directly on my webcam. Is it possible to do it, or I can only run it on a video ?
Thank you very much for your answers and have a nice day.
Best regards

When using the following command
sh <SAMPLES_BIN_FOLDER>/human_pose_estimation_demo -m <path_to>/human-pose-estimation.xml -i <path_to_video_file>
Maybe I should replace -i <path_to_video_file> by -i 0. Should it work ?
I cannot test it yet as I am trying to install the human_pose_estimation sample
Best regards

Actually the right answer is just to put 0 instead of the video path.


who can give me a ZJU-Mocap

i send e mail to author but no one answer me! help

They usually respond, so they might be busy.
I have the files but I cannot provide as they have strictly told me not to publicly share it


HDDS-1590 Aggregate bytes write metrics from container to datanode and to SCM

What changes were proposed in this pull request?
#HDDS-1590 Aggregate bytesWrite metrics from container to datanode and all the way to SCM.
(Please fill in changes proposed in this fix)
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-1590
(Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HDDS-XXXX. Fix a typo in YYY.)
Please replace this section with the link to the Apache JIRA)
How was this patch tested?
Tested in env:
TYPE scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_2f0b4887_a6a4_4ab5_ba41_36aa5cfc9056 counter
scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_2f0b4887_a6a4_4ab5_ba41_36aa5cfc9056{hostname="VM_50_210_centos"} 102400
TYPE scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_9b3b4890_b347_4e14_9b84_53d499474473 counter
scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_9b3b4890_b347_4e14_9b84_53d499474473{hostname="VM_50_210_centos"} 307200
TYPE scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_dee39c86_ec69_4edf_bd05_3c5eb16a6ca9 counter
scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_dee39c86_ec69_4edf_bd05_3c5eb16a6ca9{hostname="VM_50_210_centos"} 204800
TYPE scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_398226ba_298d_4b72_9f66_cda4046b037b counter
scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_398226ba_298d_4b72_9f66_cda4046b037b{hostname="VM_50_210_centos"} 11468800
TYPE scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_09c23825_6e8b_4a9d_9d61_603f360c3300 counter
scm_pipeline_metrics_num_pipeline_bytes_write_ratis_three_09c23825_6e8b_4a9d_9d61_603f360c3300{hostname="VM_50_210_centos"} 5
(Please explain how this patch was tested. Ex: unit tests, manual tests)
(If this patch involves UI changes, please attach a screen-shot; otherwise, remove this)

The acceptance details link doesn't work. Wasn't able to see what errors happen. Don't expect this change to affect acceptance test tho.

There is no unit test found. Please add corresponding unit test.

In terms of unit test, this change is aggregating bytesWritten stats from containerData to Ratis server and to SCM thru pipeline report. AFAIK, there is no available test framework who goes all the way. I tried MiniOzoneCluster and it didn't contain containerData in its Ratis server. Need to spend more time investigating it.

Thanks to post this patch @timmylicheng. Overall it looks good, but I am not sure about the motivation.
Why is it required to send metrics from one service to an other one and report from there? The metrics can be published by the datanode itself and can be summarized / organized by the metrics collector.
Is this information required for any of the decision made by the SCM?
I am pretty sure that I missed the context of this patch. Can you please help me to understand the goals?

Thanks to post this patch @timmylicheng. Overall it looks good, but I am not sure about the motivation.
Why is it required to send metrics from one service to an other one and report from there? The metrics can be published by the datanode itself and can be summarized / organized by the metrics collector.
Is this information required for any of the decision made by the SCM?
I am pretty sure that I missed the context of this patch. Can you please help me to understand the goals?

Sure. This patch is to summarize the bytes in for every pipeline, which requires the sum of each container that is allocated to the pipeline. The bytes in stats for each container shall be collected on datanode to be near real-time. So that I collect them up by datanode reports and aggregate by pipeline on SCM.

Thanks to post this patch @timmylicheng. Overall it looks good, but I am not sure about the motivation.
Why is it required to send metrics from one service to an other one and report from there? The metrics can be published by the datanode itself and can be summarized / organized by the metrics collector.
Is this information required for any of the decision made by the SCM?
I am pretty sure that I missed the context of this patch. Can you please help me to understand the goals?

Sure. This patch is to summarize the bytes in for every pipeline, which requires the sum of each container that is allocated to the pipeline. The bytes in stats for each container shall be collected on datanode to be near real-time. So that I collect them up by datanode reports and aggregate by pipeline on SCM.

Thanks to post this patch @timmylicheng. Overall it looks good, but I am not sure about the motivation.
Why is it required to send metrics from one service to an other one and report from there? The metrics can be published by the datanode itself and can be summarized / organized by the metrics collector.
Is this information required for any of the decision made by the SCM?
I am pretty sure that I missed the context of this patch. Can you please help me to understand the goals?

Sure. This patch is to summarize the bytes in for every pipeline, which requires the sum of each container that is allocated to the pipeline. The bytes in stats for each container shall be collected on datanode to be near real-time. So that I collect them up by datanode reports and aggregate by pipeline on SCM.

@elek Wonder if you have further questions. :)

Sure. This patch is to summarize the bytes in for every pipeline, which requires the sum of each container that is allocated to the pipeline. The bytes in stats for each container shall be collected on datanode to be near real-time. So that I collect them up by datanode reports and aggregate by pipeline on SCM.

Thanks to explain it.
I understand that it's good to calculate the number. I don't understand why is it added to the StorageContainerDatanodeProtocol.proto and sent back to the SCM. It seems to be a metric which can be fetched directly from the datanode. We don't need to send it to the SCM.
Unless the SCM needs this data for any decision (to create new pipelines, replica containers, etc..). I might missed it but as I see this is not the case (but please let me now if you have such a plan...).

Sure. This patch is to summarize the bytes in for every pipeline, which requires the sum of each container that is allocated to the pipeline. The bytes in stats for each container shall be collected on datanode to be near real-time. So that I collect them up by datanode reports and aggregate by pipeline on SCM.

Thanks to explain it.
I understand that it's good to calculate the number. I don't understand why is it added to the StorageContainerDatanodeProtocol.proto and sent back to the SCM. It seems to be a metric which can be fetched directly from the datanode. We don't need to send it to the SCM.
Unless the SCM needs this data for any decision (to create new pipelines, replica containers, etc..). I might missed it but as I see this is not the case (but please let me now if you have such a plan...).

Yea I think you already have the picture. We plan to expose pipeline heavy load in terms of bytes written to make pipeline lifecycle more balanced.

There is no unit test found. Please add corresponding unit test.

@ChenSammi Unit test is added. Please take a look.

/retest

+1.  Will commit it shortly.


PublishBuildArtifact creates error on path with spaces

We currently run into problems on the Hosted Agent.
We use a task PublishBuildArtifacts (1.0.24) to make some artifacts. Previously, when there was a space in the PathToPublish there where no problems, but recently we get the following error :
##[error]System.ArgumentException: Illegal characters in path.
Works : PathtoPublish=C:\a\1\s\AGB\10-Vektis.AGB.FEBeheerAGB\Main\Configs\OT
Fails : PathtoPublish="C:\a\1\s\AGB\10 - Vektis.AGB.FEBeheerAGB\Main\Configs\OT
We CAN rework the scripts, but we consider this behaviour as faulty.
Is this by design ?  Will this be resolved ?
Gerwin

This is a bug we discovered.  Ting where we on the fix?
From: GerwinNL<EMAIL_ADDRESS>Sent: Tuesday, January 26, 2016 7:24 AM
To: Microsoft/vso-agent-tasks<EMAIL_ADDRESS>Subject: [vso-agent-tasks] PublishBuildArtifact creates error on path with spaces (#1131)
We currently run into problems on the Hosted Agent
We use a task PublishBuildArtifacts (1024) to make some artifacts Previously, when there was a space in the PathToPublish there where no problems, but recently we get the following error :
##[error]SystemArgumentException: Illegal characters in path
Works : PathtoPublish=C:\a\1\s\AGB\10-VektisAGBFEBeheerAGB\Main\Configs\OT
Fails : PathtoPublish="C:\a\1\s\AGB\10 - VektisAGBFEBeheerAGB\Main\Configs\OT
We CAN rework the scripts, but we consider this behaviour as faulty
Is this by design ? Will this be resolved ?
Gerwin
—
Reply to this email directly or view it on GitHubhttps://github.com/Microsoft/vso-agent-tasks/issues/1131.

We apologize for the issue.  There is a hotfix going out to the service in the next hour that should resolve it.

Chris,
We appreciate your prompt response.
Thanks to Ting as well :)
We will check it out as soon as the fix is available to us.
Gerwin


Creating Azure Databricks with NAT Gateway using the template provided raises error "Cluster terminated. Reason: Azure Vnet Configuration Failure"

I have followed the NAT Template step (2nd bullet point from the bottom).
Replication steps:

Click on Azure Template on the 2nd bullet point from the bottom or use the direct link below. https://github.com/Azure/azure-quickstart-templates/tree/master/quickstarts/microsoft.databricks/databricks-all-in-one-template-for-vnet-injection-with-nat-gateway

Click "Deploy to Azure" and log in to your subscription
Populate "Resource Group" or create new one (e.g. "ResourceGroupPoC") and populate "Workspace Name" (e.g. "DBW01")
Click "Create"
Once deployment completed go to DBW01 in ResourceGroupPoC and click on Launch Workspace
Click on Login using your Azure credencials
Click on Compute tab (left) then Create Cluster (top)
Enter cluster name "ClusterPoC01" and click on "Create Cluster"
Click on Event Log tab the error message reads:
"Cluster terminated. Reason: Azure Vnet Configuration Failure. Encountered error while attempting to create NIC within injected virtual network. myNATgateway cannot be deployed on subnet containing Basic SKU Public IP addresses or Basic SKU Load Balancer".

Can you please advise how to resolve this issue?
Many thanks,
Rob Kostecki

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

ID: 958bd4e0-a052-a55e-7a37-1cbcdbf2dbbe
Version Independent ID: 1b74b0d1-c984-4fd0-899f-72ae6437c286
Content: Secure cluster connectivity (No Public IP / NPIP) - Azure Databricks
Content Source: databricks/security/secure-cluster-connectivity.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Thanks for leveraging Azure docs feedback channel to raise this question. This channel is reserved for ‘Azure docs’ feedback- for sharing doc feedback and suggesting content improvements. We didn't determine any changes for this documentation upon reviewing this feedback.
If your issue is specific to a use case, we request you to please reach out in forums like Microsoft Q&A or Stackoverflow. We have our engineers and other community members who will be happy to assist. Forums have a better suited audience for such queries. Once your issue is posted in forum, please do share the forum thread link here for tracking.
If you have a document which you are following and if you think it needs an update, please let us know about that here.
Thank you

Hi @KranthiPakala-MSFT thanks for replying.
This is a feedback to the document https://docs.microsoft.com/en-us/azure/databricks/security/secure-cluster-connectivity
The specific feedback is: the method prescribed in this document (the second bottom bullet point as copied below) does not work, thus making this documentation incorrect and misleading.
Can you replicate the case, following the steps prescribed in the documentation (perhaps using the detail steps I provided in my previous message), then update the documentation accordingly outlining the method that would allow successfully creating the NAT Gateway that works with Databricks please?
Note please make sure you were able to start the Databricks cluster as part of this investigation.
The fragment of the documentation to follow is:
"For deployments that need some customization, choose an Azure NAT gateway. Configure the gateway on both of the workspace’s subnets to ensure that all outbound traffic to the Azure backbone and public network transits through it. Clusters have a stable egress public IP, and you can modify the configuration for custom egress needs. You can implement this solution using either an Azure template or from the Azure portal."
Many thanks,
Rob

@Robert-Kostecki - Appreciate your detailed feedback and clarification.
We have assigned the issue to the content author to further evaluate this feedback and update the document as appropriate.
@mssaperla - Could you please review above feedback from @Robert-Kostecki and share your thoughts - Thank you

Thanks for your dedication to our documentation. Unfortunately, we have been unable to review your issue and apologize for the delayed response. The requested updates have not yet been made. The timeline for resolution varies based on resourcing. We've created an internal work item (DOC-7145) to incorporate your suggestions. We are closing this issue for now, but feel free to comment here as necessary.
#please-close

#please-close


Replace MockProjectExec by mocking with Spock

MockFor can't mock project.exec due to a bug in Groovy failing to support mocking of interfaces using metaClass programming. The Spock framework however uses JDK Dynamic Proxies which may work instead:
http://spockframework.github.io/spock/docs/1.0/interaction_based_testing.html
Like most Java mocking frameworks, Spock uses JDK dynamic proxies (when mocking interfaces) and CGLIB proxies (when mocking classes) to generate mock implementations at runtime. Compared to implementations based on Groovy meta-programming, this has the advantage that it also works for testing Java code.
Framework needed:
testCompile('org.spockframework:spock-core:1.0-groovy-2.3')

Gradle Forum discussion:
https://discuss.gradle.org/t/support-mocking-project-exec-currently-impossible-due-to-groovy-bug/11975


Music Desync while editing

While editing, when you rewind and playback a specific section the music does not match up with the proper measure. You have to rewind all the way to the beginning of the file.

I have never encountered this issue before. If you have a video of it happening or steps to reproduce it, that would help.

Closing for now, as there have been no other reports and this issue isn't reproducible.

Here's a reproducible example: jashin-no-wacca-zip
To reproduce, try to start playing after the first time signature change and notice the desync happening.
After some testing, it seems that this problem only occurs when the audio is an mp3. If it's wav, flac or ogg, no desync happens.


Release Scorecard - next minor version v4.1.0

In Scorecard bi-weekly meeting it was brought up about the next version of the scorecard which has some bug fixes.
This is a tracking item to discuss this.

@azeemshaikh38 @laurentsimon @justaugustus FYI...

Are folks working on any new features/changes which we should wait for before creating a release?
Also, for some reason the weekly cron jobs haven't been running fine last 2 weeks. I haven't looked into them (might be some GitHub token issue and nothing to do with code itself). But it would be useful to have one successful run and tag that commitSHA as the new release.

Found the reason why cron job was having issues - #1623. Working on a fix for it, would be good to release after this fix.

let's try blocking on ossf/scorecard-action#97, if nobody has objections. wdut?

Do we need to block Scorecard release for this though? This will rely on scorecard-action release correct?

you're correct, no need to block scorecard release. I re-phrased my comment :-)

#1623 is now fixed. I think we are good to release the next minor version. To be safe we can release at 394789cf22887344f084efeffcfe235cc8aa390f which has been tested using the cron job. If folks have any concerns let's discuss before making the release.
@naveensrinivasan could you help with creating the new release?

Yes, getting #1644 in a release would be helpful for Allstar, but we can use hash if you don't want to wait. Thanks

Yes, getting #1644 in a release would be helpful for Allstar, but we can use hash if you don't want to wait. Thanks

Merged!

I'll echo what I mentioned in Slack:
With the logging change in https://github.com/ossf/scorecard/pull/1516, we're going to break consumers, so this should be a major version bump.
Updated the tracking title to reflect v5.0.0!

#1623 is now fixed. I think we are good to release the next minor version. To be safe we can release at 394789cf22887344f084efeffcfe235cc8aa390f which has been tested using the cron job. If folks have any concerns let's discuss before making the release.
@naveensrinivasan could you help with creating the new release?

Scorecard releases are based on commit SHA that has passed cron job. If I am not wrong @azeemshaikh38 is mentioning the SHA https://github.com/ossf/scorecard/tree/394789cf22887344f084efeffcfe235cc8aa390f that has successfully run.
So the #1644 might not make it in the release if it is after the above SHA.

cc @jeffmendoza

With the logging change in #1516, we're going to break consumers, so this should be a major version bump.

Scorecards is a program first and a library as a far distant second. Allstar is the only consumer that we know of, and we can work around code changes in minor versions. In the future, it might make sense to become more formal about what encompasses the "external" API surface and then follow semver when that changes, but I don't think we are there yet.
So far, scorecards major version has reflected major features and milestones in the program's development, so we should consider sticking with that. Thoughts?

Looks like we have consensus here? If so, let's close this after releasing the next minor version. We can consider opening a new issue/PR to discuss the updates we need to make in our README about Scorecard versioning.
@naveensrinivasan can you help do the release? The latest version of cron run is using commitSHA 33f80c93dc79f860d874857c511c4d26d399609d so we can tag this as our next minor version release. Thanks.

Released https://github.com/ossf/scorecard/releases/tag/v4.1.0

Awesome, thanks!


Don't treat go.mod as a "generated" file

While it's true that the go tool does modify go.mod I would like to propose that go.mod be excluded from being treated as a "generated" file since understanding changes to it are a crucial part of the review process.
For example, here's the default files view from a recent PR to a project that groomed some dependencies:

A reviewer must "load diff" on go.mod.  And for PRs that have lots of changes across many files it is easy to gloss over the two collapsed files without actually expanding the contents of go.mod.
From a precedence standpoint dep (Gopkg.toml & Gopkg.lock) has similar properties in that dep modifies Gopkg.toml, but only Gopkg.lock is considered "generated".
Thanks!

I totally missed #4340... Closing in favor of that.


Use explicit column list in SELECT SQL statements

In this PR SQL support is introduced. We would like to use an explicit set of columns in every SELECT statements, since this helps to prevent mistakes

I'd like to work on this, please consider assigning it to me!

Hello, Matthew and I are teammates. We are CMU students who need to contribute to an open-source project as part of our final project. If no other person is working on this issue yet, please consider assigning it to us. Thank you!

@MatthewLeboffe That's awesome, all yours.
Please let us know if your have any questions as you explore. You can also ask questions on the contributors discord https://discord.ockam.io

Hi! I'm working along with Matthew and Yuhe (issues 6947 and 6949) to help contribute to an open source project.
I had one question about this issue: is this more or less a refactoring change where every query of the form "SELECT *" should be changed to specify relevant columns? Would anything else be in scope?

Hi @Sweetsuro, yes that's it. Note that the persistence PR has now been merge on develop.


Vertical Text Alignment

Currently I can't figure out how to set the vertical text alignment (screenshot attached).

I feel like this should be done using: https://developer.mozilla.org/en-US/docs/Web/CSS/vertical-align
left to right for the red box (screenshot):

verticalAlign: top
verticalAlign: middle
verticalAlign: bottom

you can probably achieve this by wrapping your text in a View with flexDirection: row and alignItems: center
vertical alignment for Text primitives aren't supported (in most non-web rendering targets):

http://airbnb.io/react-sketchapp/docs/styling.html#type-styles
https://facebook.github.io/react-native/docs/text-style-props


LSTM-CRF is worse than LSTM

hi, @glample
Recently, I have modified your code in model.py and train.py to adapt to my sequence label mission, punctuation prediction, without modifying your kernel code in nn.py.
The same setup of LSTM and LSTM-CRF is word_dim=100, word_lstm_dim=100,  dropout=0.5, lr_method=adadelta,  char_dim = 0, without using char embedding.
Then I got unexpected results in which LSTM-CRF is worse than LSTM.
I'm wondering whether it is the influence of dropout layer or not? Cause I don't use char embedding, do I need to use dropout layer?
Or maybe LSTM-CRF is not suitable for my specified mission?
Do you have any suggestions?

Did you encounter any errors when setting char_dim=0? when I do so an error will come up in the first place, as described in #18 . I wonder how your settings will work in the first place.

It seems there is a little difference in CRF likelihood loss from in TensorFlow . The unary potentials are in log_sum_exp, while in TF, it is outside.


Small improvements

Hey, sorry for the inconvenience, while trying to revert my commit I somehow managed to delete my branch and also the pull request 😃
I'm glad I could help out, if you want I can still throw out the upper case letters later.
I looked into the code and saw that your comparison returned out of the loop and didnt go all the way thrugh. I adjusted the code a bit and removed the nesting. So maybe it will perform better now.
The whitespaces just got removed due to auto formatting (Ctrl+E, Ctrl+F on VS) I left it out this time :)
Sorry for all the hussle I'm quiet new to git

Hey there, thanks again, but I already changed the GetHashValue Method to something more performant. I just noticed that the code in ByteMemoryEqualityComparer is not used anymore and will be removed. If I remember corretly I moved it closer to its usage and used a constant in the loop so that the compiler actually makes a memory compare instead of looping through the elements.


IOS Deprecations

GoogleMaps.m

'interfaceOrientation' is deprecated: first deprecated in iOS 8.0

GoogleMapsViewController.m

'sizeWithFont:constrainedToSize:' is deprecated: first deprecated in iOS 7.0 (Use -boundingRectWithSize:options:attributes:context:)
'drawInRect:withFont:lineBreakMode:alignment:' is deprecated: first deprecated in iOS 7.0 (Use -drawInRect:withAttributes:)

Map.m

'base64Encoding' is deprecated: first deprecated in iOS 7.0

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/10091519-ios-deprecations?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F637500&utm_medium=issues&utm_source=github).

This plugin need to support even iOS6, because Google Maps SDK for iOS works on iOS6.
I appreciate if you send a pull request with great patch.

And at least base64Encoding is not used anymore.
You might see the old code.
https://github.com/wf9a5m75/phonegap-googlemaps-plugin/search?utf8=✓&q=base64Encoding

Done


Add delegation indexer endpoints

https://github.com/iotaledger/tips/pull/153/commits/7793a69025f091cdbfb5b858ac7fd384e64b2fee

Closed by https://github.com/iotaledger/iota-sdk/pull/1523


fix(readme): s/dist/cargo-dist

lolsob that my readme typo PR is failing clippy somehow

Must have been an update to clippy or something..?


Logging ResponseUrl onSegmentError

Here i can get responseurl onSegmentError successfully.
Consider the below sceneario.
p2pml make a options segment request
receive 200 response
p2pml make a get segment request
receive 302 response redirecting to another url
p2pml make a new options segment request
receive 200 response
p2pml make a new get segment request
receive 403 response
Now, I'm getting a blank responseUrl onSegmentError.
Consider the below scenerio.
p2pml make a options segment request
receive 200 response
p2pml make a get segment request
receive 302 response redirecting to another url
p2pml make a new options segment request
receive 403 response
Thats it. responseurl is blank as p2pml did not make a 2nd get request as 2nd options request returns a 403 error.
How do I capture that 302 url also known as responseurl onSegmentError?

XMLHttpRequest is used. It does all the redirects. If the response URL is empty then most likely it should be so. The code that gets responseURL on error:
private segmentFailure(segment: Segment, error: any, xhr: XMLHttpRequest) {
    segment.responseUrl = xhr.responseURL === null ? undefined : xhr.responseURL;

    this.xhrRequests.delete(segment.id);
    this.failedSegments.set(segment.id, this.now() + this.settings.httpFailedSegmentTimeout);
    this.emit("segment-error", segment, error);
}

I guess the only way is to handle this server side.
Instead of depending responseURL, I need to make do with requestURL onSegmentError log to server.


[stable/redis-ha] Can't connect to Redis from outside the Kubernetes cluster

Describe the bug
When Redis client starts, it calls one of the sentinels to get the IP of the elected master.
It gets the internal IP of the pod and sends requests to this master.
When the client runs outside the K8S cluster, it fails to connect to the master since the IP is only accessible from within the K8S cluster.
Version of Helm and Kubernetes:
Any
Which chart:
redis-ha
What happened:
Connection refused when trying to initiate Redis client
What you expected to happen:
Client should be able to connect.
How to reproduce it (as minimally and precisely as possible):

Install redis-ha chart.
Try to connect to this chart from outside the cluster using sentinel.

@eladtamary I don't think this is a bug as much as a feature. I'm currently thinking of how to best manage this in my environment.
By default, I think this chart was built to support internal comms.
The decision should be made to
a. Not change the chart and recommend a proxy for managing this.
b. Update the chart to deploy a proxy that the service communicates with instead.

One solution to this is to create a loadbalancer service that points to all the master and slave. This can be used for reading.
For writing try creating a custom controller that moves a tag master from one pod to another based on reconfig script invoked by sentinels when there is a failover. By this a new label is attached to a new pod that becomes master. A service name redis-master that attaches itself to a pod having the floating label should always point to the master. Use this for writing.

@AkashKrDutta This is no longer an issue as we have HAProxy as a loadbalancer.

@DandyDeveloper  Thanks for the info! Although the extra running of a deployment can be saved (the ha proxy) if we directly shift the labels using a PATHC to the kuberntes api to just attach the required label. Will be testing both to see which is better!

@DandyDeveloper & @AkashKrDutta  Hi guys,
may I ask to help me in my scenario too, I got the same issue where I can't successfully use Redis on EKS.
Could not connect to Redis at ******.eu-***.elb.amazonaws.com:0: Can't assign requested address
For some reasons Redis master is changing the IP continuously and fails even if I exposed It Externally.
I think I deployed it wrong or I don't understand the full picture...
helm install --name ***-redis --set master.service.type=LoadBalancer --set master.persistence.enabled=true --set master.persistence.size=20Gi --set master.statefulset.updateStrategy=RollingUpdate --set cluster.slaveCount=1 --set password=****  stable/redis
Any clue?


Implement immediateStop in HTTP listener

Description:
Currently, the listener's immediateStop method is not implemented. Need to do a feasibility study on this.

In the implementation, as per the current design, we are using a shared boss and worker EventLoopGroups over all the listeners as well as clients.
As per Netty documentation, in order to shut down a Netty application, you need to shut down the EventLoopGroup. As we have been sharing this eventloop group among other listeners and clients, it is not possible to shutdown it immediately.
It is not possible to create eventloopgroups per listener as it can result in an issue similar to this.

I think what we need to do is instead of sharing the EventLoop group, we should share the thread pool.  It seems we can do that by sharing a thread executor pool.
https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/nio/NioEventLoopGroup.java#L72

I think what we need to do is instead of sharing the EventLoop group, we should share the thread pool. It seems we can do that by sharing a thread executor pool.
https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/nio/NioEventLoopGroup.java#L72

Once the suggestion was implemented(Share  instead of EventLoop group), local build facing Too many open files issue. According to a similar issue[1], this behaviour is seen when multiple event loop groups were created.
[1] https://github.com/netty/netty/issues/639#issuecomment-9167403

I think what we need to do is instead of sharing the EventLoop group, we should share the thread pool. It seems we can do that by sharing a thread executor pool.
https://github.com/netty/netty/blob/4.1/transport/src/main/java/io/netty/channel/nio/NioEventLoopGroup.java#L72

Once the suggestion was implemented(Share instead of EventLoop group), local build facing Too many open files issue. According to a similar issue[1], this behaviour is seen when multiple event loop groups were created.
[1] netty/netty#639 (comment)

As the proper solution, closed the socket pipeline of the server channel initializer after sharing the Eventloop groups again.
Sorted https://github.com/ballerina-platform/ballerina-standard-library/issues/2990


fix: fix time reset

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


feat: highlight active footer navigation

PR Checklist

[x] Addresses an existing open issue: fixes #058
[x] That issue was marked as status: accepting prs
[x] Steps in CONTRIBUTING.md were taken

Overview

Switched all footer navigation links + the back link from standard anchor tags into NavLink components.
Added color: white; to active page-grid-footer-link navigation links.

Wicked.  Thanks!


integration-test update - use KafkaProducer in ITKafkaTest and add retries to task submission

new KafkaProducer seems more resilient to temporary zk timeouts and same for retries. this helps prevent some integration tests false alarms.

👍


fuzz: server config validation fuzz test.

Title: fuzz: server config validation fuzz test
Description: This fuzzes the bootstrap config message on server validation mode.
Risk Level: Low
Testing: Tested under oss-fuzz
Signed-off-by: Anirudh M<EMAIL_ADDRESS>

@mattklein123

@anirudhmurali please ping me when this is ready for another pass.

Running bazel test //test/server/config_validation:config_fuzz_test -config=clang-tsan --runs_per_test=2000 shows like few (3-7) failing tests out of 2000. All with the same message: Didn't find a registered implementation for name: 'tls'.
I tried linking all extensions, and tested, it throws:
[critical][assert] external/envoy/source/common/thread_local/thread_local_impl.cc:19] assert failure: shutdown_. Same reason why tsan failed in the previous commits.
The ASSERT(shutdown_); statement  in ~InstanceImpl() is what is causing this failure. shutdown_ seems to be a atomic boolean object. Which part of the code might have set shutdown_ to false?

@anirudhmurali I suggest grepping (or using other code search tools) to understand how shutdown_ is manipulated. I think that adding back the envoy_all_extensions() was correct; what needs to be done here is figure out why we race in shutdown. We generally expect code committers to take time to understand the framework and fix bugs like this as a gating step before commit.
How often are you seeing the flake with shutdown_?

Used validateConfig() instead of creating ValidationInstance object. Things work now, all tests pass well, even with runs_per_test=2000 set. I guess the server crash was because of some improper initialization of stats_store or access_log_lock.
Should I run more tests to uncover flakes?


Feature: Port CBM-DOS hooks from x16emu

This is really cool and it would be great if Box16 supported it too.

I believe this is implemented as of a0f120f7187168c61b39188c78a89826305e53d7


Switch sprites to triangle draw mode

Resolves #106

Codecov Report

Merging #110 into master will increase coverage by 0.37%.
The diff coverage is 96.15%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
+ Coverage   77.00%   77.37%   +0.37%
==========================================
  Files          74       74
  Lines        2248     2290      +42
  Branches      197      203       +6
==========================================
+ Hits         1731     1772      +41
  Misses        359      359
- Partials      158      159       +1

Flag
Coverage Δ

#unittests
77.37% <96.15%> (+0.37%)
:arrow_up:

Impacted Files
Coverage Δ

src/rendering/draw_mode.ts
85.71% <ø> (-1.79%)
:arrow_down:

src/rendering/image/image_request.ts
37.50% <0.00%> (-2.50%)
:arrow_down:

src/rendering/renderable.ts
100.00% <ø> (ø)

...rd/collision/algorithm/always_collide_algorithm.ts
100.00% <ø> (ø)

src/standard/collision/algorithm/gjk_algorithm.ts
85.71% <ø> (ø)

...ard/collision/algorithm/never_collide_algorithm.ts
100.00% <ø> (ø)

src/standard/collision/collider.ts
85.71% <ø> (ø)

src/standard/collision/collision_info.ts
100.00% <ø> (ø)

src/standard/frustum_culler/frustum_culler.ts
81.81% <ø> (ø)

src/standard/http_image/http_image_system.ts
65.11% <ø> (ø)

... and 14 more


[FEATURE]: German Translation

I´ve made a little translation to german:
de.json

Vielen Dank... :)


New-person and upgrade-person endpoints should support Stripe payments

Our current webshop's back-end Scala code should be ported to JS, and the endpoints POST /api/kansa/people and POST /api/kansa/people/upgrade should be expanded to use the Stripe & SendGrid APIs to effect changes.

This is better tracked in #18, #19 & #20.


Not in sync at Packagist

@electricjones packgist stills on v.1.8.0 :(
https://packagist.org/packages/phoenix/eloquent-meta

Thank you for pointing this out. I apparently had to do some update in composer. It is showing 1.9 for me now. Please confirm and I will close this issue.

Its fine now, thanks. 👍


Issue #1

GIVEN I am taking a code quiz
WHEN I click the start button
THEN a timer starts and I am presented with a question
WHEN I answer a question
THEN I am presented with another question

WHEN I answer a question incorrectly
THEN time is subtracted from the clock
WHEN all questions are answered or the timer reaches 0
THEN the game is over
WHEN the game is over

I also added this. ^^^^^^


A TypeError occurred while running this code in Colab

When I was running code in Google Colab, I encountered the following error message:
File "capsulenet.py", line 244, in <module>routings=args.routings)
 File "capsulenet.py", line 50, in CapsNet     name='digitcaps')(primarycaps)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer.py", line 897, in __call__     self._maybe_build(inputs)
 File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer.py", line 2416, in _maybe_build     self.build(input_shapes)  # pylint:disable=not-callable
 File "/content/drive/My Drive/CapsNet_Keras_master/capsulelayers.py", line 116, in build     name='W')
  File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer.py", line 577, in add_weight     caching_device=caching_device)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/tracking/base.py", line 743, in _add_variable_with_custom_getter     **kwargs_for_getter)
 File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer_utils.py", line 141, in make_variable     shape=variable_shape if variable_shape else None)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variables.py", line 259, in __call__     return cls._variable_v1_call(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variables.py", line 220, in _variable_v1_call     shape=shape)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variables.py", line 198, in <lambda>     previous_getter = lambda **kwargs: default_variable_creator(None, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variable_scope.py", line 2598, in default_variable_creator     shape=shape)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/variables.py", line 263, in __call__     return super(VariableMetaclass, cls).__call__(*args, **kwargs)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/resource_variable_ops.py", line 1434, in __init__     distribute_strategy=distribute_strategy)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/resource_variable_ops.py", line 1567, in _init_from_args     initial_value() if init_from_fn else initial_value,
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/keras/engine/base_layer_utils.py", line 121, in <lambda>     init_val = lambda: initializer(shape, dtype=dtype)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/init_ops_v2.py", line 542, in __call__     fan_in, fan_out = _compute_fans(scale_shape)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/init_ops_v2.py", line 1015, in _compute_fans     receptive_field_size *= dim
TypeError: unsupported operand type(s) for *=: 'float' and 'NoneType'
I'm sure that tensorflow version is 2.2.0, Does anyone have the same problem?

same error

Stuck on the same issue, kindly throw some light on it and share how resolve it. @Bear-Lx @zhangxin9988  @kmader @iretiayo
I am trying to convert the trained model file to a TFLite version where the process throws similar error shown in attached image below. If you need any further details pls let me know.


RenderSwaggerUI Missing From Functions When Deployed In Azure

When I deploy the FA to azure, the "render" endpoints arent shown in the list but they do respond if I goto them via the URL.

Hi, @bakes82 Thanks for the issue! If you run the Azure Functions app in-proc, it's the expected behaviour that you can't see those rendering endpoint on Azure Portal. However, it's working as expected.


[2565] - Configure Geocoder gem to use existing Redis instance

Context
Geocoder has the ability to cache requests. This PR configures it to use the existing Redis instance
Checklist

[x] Make sure all information from the Trello card is in here
[x] Attach to Trello card
[x] Rebased master
[x] Cleaned commit history
[x] Tested by running locally

 Add geocoding setup


Fixes issue #3 : (added save feature)

Please also delete 2 generated files.

@RockingRishav I have updated the wiki PTAL look there and do the required changes.

@rishidyno sir, i have applied the changes please recheck once

@rishidyno sir, Finally implemented the save to gallery feature please review it.

LGTM and Merged.


any way to force SVG icons instead of fonts?

reason
if browsers disabling/not supporting custom fonts, it will look like this:

it doesnt even hide overflow text

i need it too


ResourceCache to use VirtualFileSystem

Instead of tracking file changes the ResourceCache should just use the VirtualFileSystem as much as possible.

I need to revisit these changes and see if we can do better. Then it would be good to merge.


pull request for e2e test at 01/06/2017 08:45:10

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


fix-issue delete space modal doesn't pop down

Type of change

Bugfix

Description

FIx - delete address space space dialogue doesn't pop down

LGTM


How to send large videos using Telegraph?

Hello!
I've been working on a bot that would send users videos (mostly larger than 500mb). I've read the telegram bot api docs. There is a restriction of 50MB, with videos/animations/documents. In the recent changes, Telegram announced now bots can send files upto 2000 MB.
I tried to do it with Telegraph::video, but it didn't work.
There are many bots that are uploading files upto 2GB, So, my question is how to do it?

I'm still seeing a 50mb limit on video size
https://core.telegram.org/bots/api#sendvideo
Could you paste a link to the source of your information?

Hmm, I think we could tune the file size check when the bot server is not the standard one
Will be added in next release

Yes, I would be very grateful for this feature
I'll be waiting :)

fixed in #476


Viewing sqlaudit files

The suggestion that Log Viewer from SSMS can be used to read .sqlaudit files it incorrect, it doesnt do a very good job at all.
1 - you have to locate the log file / browse to it rather than have an option to open it from the audits that are on the active instance in SSMS
2 - once opened the content is broken and includes file read errors and ascii characters.
I'd suggest removing this suggestion from this page or fixing Log Viewer so that it can read and filter sqlaudit files

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

ID: 0b2a1a3a-cffa-ec4b-dedc-c1c23d547abb
Version Independent ID: b0e62617-2dc3-8762-e359-a78af1fe3191
Content: SQL Server Audit (Database Engine) - SQL Server
Content Source: docs/relational-databases/security/auditing/sql-server-audit-database-engine.md
Product: sql
Technology: security
GitHub Login: @VanMSFT
Microsoft Alias: vanto

@fatherjack -- thank you for your documentation feedback.  For Log Viewer changes, you are welcome to add a suggestion here:

SQL Server Feedback

@VanMSFT -- please look into this issue #4209.

Hi @fatherjack! Thanks for the suggestion. It sounds to me like you are experiencing a product issue, and not a documentation issue. Log Viewer is appropriate to view SQL Server audits, and if you are experiencing an issue with the product, please feel free to submit a Product Feedback. Thanks for your time and input on our docs!


MODEXPW-199: Upgrade Users interface to 16.0

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@khandramai
Thank you!  If this is acceptable, can you please merge?  I don't have access to the repo to do so.

@khandramai
Can you please merge?  I don't have access to the repo.  Thanks!


Fix fix test leading random failed

Proposed changes
Problem:
when running pipeline, we get randomly failed of test_leading
Reason:
physical distribute was generated and choosed to be the best plan because we can not get any statistic information of empty table.


updates box setting in dev vagrant file to working box fix #157

attempting to vagrant up on the dev vagrant breaks due to the virtualbox box image not being found. Just updated the Vagrant file to use the bento centos 6.7 for parity and functionality.
URL: ["https://atlas.hashicorp.com/chef/centos-6.5"] Error: The requested URL returned error: 404 Not Found
Tested: did a vagrant up and worked with that image, which is publicly viewable
https://atlas.hashicorp.com/bento/boxes/centos-6.7

Verified that @bojung1 has signed the CLA. Thanks for the pull request!


Address open npm module security issue

https://github.com/lifeomic/cli/network/alert/package.json/open/closed

Closed by #95


Figure some way to make validation data easier to update

I am going to postpone working on the validation step, will probably disable for a bit until I can figure it out better.


Simulator broken in particular size

To Reproduce
Steps to reproduce the behavior:

Reduce the browser size so that MakeCode is in tablet mode
Increase the size slowly.

Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Edge

Same in IE11

Related to https://github.com/Microsoft/pxt-arcade/issues/412


Static Type Hinting: dataprofiler/reports/utils.py

Is your feature request related to a problem? Please describe.
Improve the project in the area of

Make collaboration and code contribution easier
Getting started with the code when a contributor is new to the code base

Describe the outcome you'd like:
Utilizing types from mypy documentation for python3, implement type hinting in dataprofiler/reports/utils.py.
Additional context:

mypy documentation
Why use python types?

I'm working on this issue

PR #640 resolved this issue. Thanks again, @stefanycoimbra!


How to replace NSClient++?

Hi,
i have been watching your repositories for some time and from the commits at monitoring-agent i see that at least the agent is actively developed/maintained.
So i guess you are using it at least in your own company. What about the scripts you are using in order to replace the built-in features of NSClient++ like cpu, memory, services etc? Are you willing to make them public?
Gerhard

Hiya,
Yeah the vast majority of them we're happy to release (and it's script-compatible with NRPE scripts by design).
We've just added windows performance counter support through https://github.com/infraweavers/monitoring-agent-check-nt-replacement which essentially does all the heavy lifting for checking against Windows.
I'll filter through our remote-scripts that we use for windows/Linux and see which ones are generically applicable and which aren't. Any that are general purpose, we're happy to release alongside the plugins themselves.


[SEMANTIC-247] [Bug] Derived metrics with parenthesis in the expression cannot be calculated with metrics.develop macro

Is this a new bug in dbt_metrics?

[X] I believe this is a new bug in dbt_metrics
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
I have defined a derived metric (conversion_rate) which has the following expression:
- expression: number_of_orders/(0.9*sessions)
Where number_of_orders (count_distinct of a column) and sessions (sum of a column) are two simple metrics.
Additionally, I have used metrics.develop macro to calculate the value of conversion_rate using the following macro:
select * from {{metrics.develop(develop_yml=my_metrics_yml, metric_list=['conversion_rate'], grain='month')}}
And I have obtained this error:
syntax error line 89 at position 90 unexpected 'as'
After debugging the dbt-metrics package, I have got to the line 22 of the dbt_metrics/macros/variables/get_metric_definition.sql file and obtained what follows:
{% set metric_expression = metric_definition.expression | replace("metric(","") | replace(")","") | replace("{{","") | replace("}}","")  | replace("'","") | replace('"',"")  %}
It seems that this part of the code is removing the last parenthesis of the expression of the metric conversion_rate and causes the query to fail.
Expected Behavior
I would expect the metric could be calculated without any problem with metrics.develop macro.
Steps To Reproduce
See above.
Relevant log output
No response
Environment
- dbt-adapter & version:v1.4.1
- dbt_metrics version:v1.4.1

Which database adapter are you using with dbt?
snowflake
Additional Context
I have temporarily adjusted the dbt-metrics package code by replacing the code line mentioned above by a more concrete replacement expression:
{% set metric_expression = metric_definition.expression | replace(" ","") | replace("{{metric('","") | replace("')}}","")  | replace("'","") | replace('"',"")  %}
And it has worked without any problem.

@josepfranquetf wow, great find and beautiful that you found the fix as well! Could you submit a PR with your fix? We'll be releasing the 1.5 branch late this week/early next week and I'd love to make sure this is included. Unfortunately I don't have the bandwidth to take it on but I'm happy to review for you!

Hi @callum-mcdata, many thanks for your answer! I have just opened a PR, which you can find here. Tell me in case you need something else!


Make save infallible

Save writes to a vec which is infallible and so the API should reflect that.

Wonderful!


Improving search functionality with a stored cache of keywords

Currently, our search functionality is using a fairly simple method. It grabs the first N words of each article and turns that into the store that's used to lookup keywords in pages. Here's where that happens:
https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/_includes/search/lunr/lunr-store.js#L26
As @psychemedia has noted, this is suboptimal for a number of reasons: it's not guaranteed that the first N words will have the most "important" words, and moreover that cache of words may change over time in unpredictable ways.
It would be fairly straightforward to re-create the functionality of the lunr store js code but using Python at build time. This could do something like:

Any time the book is built, read in each page
Grab the page's content from the notebook
Remove all "common" english words from the content
Remove all repeated words from the content
Sort the words based on their (length / frequency / etc)
Take the top N of those words
Store the result in the same JSON structure that the lunr store code uses.

I think this could be a way to get a more representative sample of words for lunr.js to look through.
Another option is to find a way for the search to scale better so we can search through the full article text instead of just a subset of words.

The lunr.py implements lunr.js in python and may provide handy functions for generating the index in the correct format.
I did a naive pencil sketch some time ago here that indexed a set of notebooks using lunr.py then generated the JS index object required for embedding in the HTML search page.
There is also question about whether to index code or not and how that should be treated.

Oh awesome! I didn't realize there was a python implementation. That should simplify things quite a bit!

I've just started looking around for whether there are any more recent alternatives to lunr.js and elasticlunr looks like it could be interesting.
For example, field based searching would presumably allow searching within markdown, code and code outputs if such fields were separately indexed. (This should be easy enough when building from an .ipynb corpus, but perhaps a bit more challenging when using eg .md, unless it were piped through Jupytext and converted to .ipynb for indexing purposes?)

@psychemedia at this point, we're actually converting all text files into ipynb first with jupytext, so this could still work if it were something that only worked with notebooks.
I think the main question is how lightweight the solution is - we need this to be something that works 99% of the time, that does most of what we want, and that will be fairly straightforward for developers to grok and maintain.

Closing as this should be superceded by beta.jupyterbook.org


[SECURITY] dlang Artifacts are downloaded insecurely

CWE-829: Inclusion of Functionality from Untrusted Control Sphere
CWE-494: Download of Code Without Integrity Check
This project contain files that indicate that this project is resolving dependencies over HTTP instead of HTTPS. Any of these artifacts can be MITMed to maliciously compromise them and infect the build artifacts that were produced.
This vulnerability has a CVSS v3.0 Base Score of 8.1/10
https://nvd.nist.gov/vuln-metrics/cvss/v3-calculator?vector=AV:N/AC:H/PR:N/UI:N/S:U/C:H/I:H/A:H
This isn't just theoretical
POC code has existed since 2014 to maliciously compromise software downloaded inflight.
See:

Want to take over the Java ecosystem? All you need is a MITM!
https://max.computer/blog/how-to-take-over-the-computer-of-any-java-or-clojure-or-scala-developer/
https://github.com/mveytsman/dilettante

MITM Attacks Increasingly Common
See:

https://serverfault.com/a/153065
https://security.stackexchange.com/a/12050
Comcast continues to inject its own code into websites you visit (over HTTP)

Source Locations

https://github.com/mihails-strasuns/setup-dlang/blob/785b25c9c8ab3684326846d3e7d187a35b1a4a20/src/compiler.ts#L74-L77

hm I don't think the D download endpoint supports HTTPS but it has a GnuPG keyring and signatures with each release since 2.067.0.
However on https://help.github.com/en/articles/software-in-virtual-environments-for-github-actions it doesn't list GnuPG as installed so I don't think we can verify it with pre-installed programs. Otherwise I don't know how we could verify the signatures

I'm working on getting this resolved by requiring SHA-256 signatures if you are using HTTP instead of HTTPS.
https://github.com/actions/toolkit/issues/162

sha256 doesn't help much as we don't know full set of possible artifacts that may be downloaded. I'd look into downloading portable gpg binary and using that insted.

I think it would be better trying to include some PGP library and try to parse the keyring they give us somehow and use it in typescript

@WebFreak001 can you maybe also ask D upstream if it would be possible to add https to download.dlang.org? This looks like a simple oversight.

died in the forums<EMAIL_ADDRESS>

You can tell them that using https://letsencrypt.org/ is free.

the download.dlang.org domain is an alias for their AWS bucket where I think were some problems. Directly using their bucket URL via HTTPS is possible but it's possible it would break in the future and adding in a fallback in case it's not valid anymore won't add any security because a MITM could just deny access to the HTTPS encrypted AWS bucket then

Yeah, I intend to stick with gpg solution until github release artifacts are provided - haven't had any time in last few weeks to do it though.

@WebFreak001 just checked and gpg binary is available on Windows CI vms out of the box, so this is a low-hanging fruit. Will submit a patch soon-ish.


Fix legacy_ctrl_to_param() to pay better attention to keytype

The keytype number will only be -1 when control commands are used for
more than one key type.  Sometimes, they share the same underlying
structure, and sometimes not.
Some of the RSA control commands that are using only with the keytype
EVP_PKEY_RSA we misplaced to be handled with the keytype -1.

24 hours has passed since 'approval: done' was set, but this PR has failing CI tests.  Once the tests pass it will get moved to 'approval: ready to merge' automatically, alternatively please review and set the label manually.


internal/build: also log last message from docker

Have any ideas on how to test this well? I think for something like this I care more about getting it in, given that it's only in a case where things are already bad

it's worth pointing out that most buildkit messages are binary-encoded. i think this is safe to try, but there's a risk it might just spew binary gibberish at people in most cases

this should be obsolete now, see discussion in https://github.com/windmilleng/tilt/issues/1967


[Question] Remove picture form database

How to remove a picture from database?

I am not sure what you mean here, sorry.

I read your book and follow the instructions, but right now I want to remove a picture from classifiedad. I could remove it from domain model but it remains in database.

In what database? There are at least three database types used in the book.

Please reopen if the issue still persists.


Question of NetworkCommissioning featureMap implementation

Reproduction steps
I would like to inquire because there seems to be a difference between the SDK implementation and the core specification.
https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/service_device_management/NetworkCommissioningCluster.adoc#4-features
Looking at the Spec above, it is understood that Features can support Wi-Fi, Thread, and Ethernet all.
However, looking at the code implementation details below, it seems that only one can be enabled, so I am curious which one is correct.
https://github.com/project-chip/connectedhomeip/blob/155917e529db5e1e8d7819c6309115fcabc93cff/src/controller/CHIPDeviceController.cpp#L1969
I think one of the two needs to be modified.
Bug prevalence
everytime
GitHub hash of the SDK that was being used
155917e529db5e1e8d7819c6309115fcabc93cff
Platform
core
Platform Version(s)
master
Anything else?
No response

for network commissiong cluster, i remember wifi network cluster is in endpoint0, thread is in endpoint1..different network commissioning cluster lives in different endpoint

I know that when the Commissioner reads, it reads all endpoints.
https://github.com/project-chip/connectedhomeip/blob/155917e529db5e1e8d7819c6309115fcabc93cff/src/controller/CHIPDeviceController.cpp#L2447
However, I could not find a sentence that states that only one network information must be configured for one endpoint. and. The FeatureMap is also divided by bit so that multiple network information can be set.

@joonhaengHeo The "Conformance" column for the features in question in the spec says O.a for all of them.  That means "exactly one of these is allowed".  See https://github.com/CHIP-Specifications/connectedhomeip-spec/blob/master/src/data_model/Data-Model.adoc#313-choice

Also:

I could not find a sentence that states that only one network information must be configured for one endpoint.

This is a general restriction: you can't have two instances of the same cluster ID on the same endpoint, because there is no way to address them, right?

@bzbarsky-apple
What does it mean that a Network Commissioning Cluster must have two instances? Of course, depending on the device implementation, it may have to be divided into two instances, but I understand that supporting more than two features with one instance is not a violation of the specification. In particular, I know that other features will be added after Matter 1.3, but I don't think there should be such restrictions in the controller.

I misunderstand. Thank you for response.

What does it mean that a Network Commissioning Cluster must have two instances?

Only if you support two different network interfaces...

but I understand that supporting more than two features with one instance is not a violation of the specification

It is if the features are defined to be mutually exclusive.


Failed verification results not being published when state is missing on provider side

Dependencies used in the provider tests:
import au.com.dius.pact.provider.junit.Provider;
import au.com.dius.pact.provider.junit.State;
import au.com.dius.pact.provider.junit.loader.PactBroker;
import au.com.dius.pact.provider.junit5.HttpTestTarget;
import au.com.dius.pact.provider.junit5.PactVerificationContext;
import au.com.dius.pact.provider.junit5.PactVerificationInvocationContextProvider;
Test output:
Contract.Provider.ProviderTests > pactVerificationTestTemplate(PactVerificationContext)[3] STANDARD_OUT
Verifying a pact between consumer and provider
  Valid Request
    returns a response which
      has status code 200 (OK)
      includes headers
        "Content-Type" with value "application/json; charset=utf-8" (OK)
      has a matching body (OK)

2019-08-29 14:11:49.009  INFO 19043 --- [    Test worker] a.c.d.p.p.DefaultVerificationReporter    : Published verification result of 'au.com.dius.pact.pactbroker.TestResult$Ok@1d6b0542' for consumer 'Consumer(name=consumer)'

Contract.Provider.ProviderTests > pactVerificationTestTemplate(PactVerificationContext)[4] STANDARD_OUT
2019-08-29 14:11:49.015  INFO 19043 --- [    Test worker] a.c.d.p.p.DefaultTestResultAccumulator   : Not all of the #3 were verified.
Contract.Provider.ProviderTests > pactVerificationTestTemplate(PactVerificationContext)[4] FAILED
au.com.dius.pact.provider.junit.MissingStateChangeMethod: Did not find a test class method annotated with  @ State("Some State")
(...)
Notice that in this last case the failure is not Published which means the contract is seen as being verified when it is not the case.

The test result will now be marked as failed when the state change is missing or fails.

Fix has been released with version 3.6.13


is eslint 4 + babel 6 officially supported?

Seems like babel-eslint 8 is only designed to support eslint 4 + babel 7 beta, but I just wanted to check...

its working well for me. my projects are still babel 6.  running eslint 4.19.0 & babel-eslint 8.2.2
Sent with GitHawk

This has broken my project as it depends on packages that have been pulled from npm. By you, babel. You pulled them.

what was pulled?

9 works with babel 7, we can close this

Thank you for the issue. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


Does B2C support myapps panel ?

In the comparison doc for B2B vs B2C its mentioned that B2C doesn't support SSO to Microsoft based SaaS applications.

Its not clear what exactly does "Microsoft SaaS" here means. Does it support LoB SaaS based application such as Workday in B2C?
https://docs.microsoft.com/en-us/azure/active-directory/b2b/compare-with-b2cSo i have following questions

Does it support SSO to portal kind of experience, something similar to myapps panel in B2B?

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

ID: 1b99765d-5ff4-56bf-fbd1-3a5f281f90da
Version Independent ID: 053fdf19-73bb-4aba-ac18-e3cdae83861f
Content: Compare B2B collaboration and B2C - Azure Active Directory
Content Source: articles/active-directory/b2b/compare-with-b2c.md
Service: active-directory
Sub-service: b2b
GitHub Login: @msmimart
Microsoft Alias: mimart

@prashantabkari
Thanks for your feedback! We will investigate and update as appropriate.

I do not believe that it is supported as B2C has a different app registration experience. In my own b2c tenant it shows up but does not display my applications.

@msmimart and @mmacy to confirm

The product team has confirmed:
Azure AD B2C does not have native integration with the first party services such as O365, and that includes My Apps. It only works if you use the Azure AD functionality of a B2C tenant, such as logging in with an Azure AD account that resides in an AAD B2C tenant. It will not tie into the B2C functionality, such as determining which B2C apps you signed up to.
You would need to build your own front end portal that can list the users applications.

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments or open a thread on MSDN and I will gladly continue the discussion. https://social.msdn.microsoft.com/Forums/en-US/home?forum=WindowsAzureAD


Support APIM

What would you like to be added?
Retrieve JWT and set it to request header
Why is this needed?
To coordinate with APIM, this feature is needed.

From your client code (such as a mobile app or in-browser JavaScript), send an HTTP GET request to /.auth/me (token store must be enabled). The returned JSON has the provider-specific tokens.

https://learn.microsoft.com/en-us/Azure/app-service/configure-authentication-oauth-tokens#retrieve-tokens-in-app-code

fixed by #17

Reopened,  since JWT should be sent from frontend app to APIM