Import releases and tags from docker-machine-kvm

Currently missing the two that were not done on master:

https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.10.0

https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.8.2

https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.7.0
https://github.com/dhiltgen/docker-machine-kvm/commit/a3882af  "Clean up logging a little"

https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.6.0
https://github.com/dhiltgen/docker-machine-kvm/commit/8d63cea "Add disk cach/io tunables"

https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.5.6

https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.0.6

No major differences, but it would be nice to have a full import:
*   b14a80e (tag: v0.8.2) Merge pull request #33 from r2d4/leases
|\
| * e5724f2 Use last entry when parsing DHCP status file
* |   dc9207e Merge pull request #34 from dongsupark/dongsu/migrate-libvirt
|\ \
| * | a7eeb00 import libvirt/libvirt-go instead of alexzorin/libvirt-go
| |/
* |   cd8ae9d Merge pull request #32 from caio1982/cpuhost
|\ \
| |/
|/|
| * c7becbf The current domain XML used for the KVM driver does not specify a CPU mode, which makes things very slow and usually lead to a constant >25% CPU usage for people using your driver with Kubernetes and Minikube. This solves the problem as it simply tells KVM to use everything the host supports and emulate whatever else is needed.
|/
* 37bb4cc Put NewVirConnection inside of PreCreateCheck, added error message for (#18)
* a6ecdc2 Update gitignore based on new build layout
* 7077fd0 Refactor README (#25)
*   9fb6d24 Merge pull request #22 from mikefaille/cmd-path
|\
| * a2f0b19 FIX : Permit `go build .` to output right name
|/
* e302cd8 Add license file
* 5f4c9b1 Add disk cach/io tunables
| * a3882af (tag: v0.7.0) Add disk cach/io tunables
| * 8d63cea (tag: v0.6.0) Clean up logging a little
|/
* 4bc0ec9 Add a little more detail on getting started
*   c8f7e8a Merge pull request #8 from dpw/readme-quick-start
|\
| * 28e100b Add quick start instructions
|/
* f4dee20 (tag: v0.5.6) Merge commit '6096a1a39eb1ec6878c797b45afbe0e48410eda0' into rsampaio_conn_fix

Also because this version (v0.7.0) is currently being used by minishift:
https://docs.okd.io/latest/minishift/getting-started/setting-up-virtualization-environment.html#setting-up-kvm-driver

https://github.com/dhiltgen/docker-machine-kvm/releases/download/v0.7.0/docker-machine-driver-kvm

You have commit rights, so you are able to push these.
On Sun, Sep 9, 2018, 01:09 Anders Björklund<EMAIL_ADDRESS>wrote:

Currently missing the two that were not done on master:

https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.10.0
https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.8.2
https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.7.0
https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.7.0
dhiltgen/docker-machine-kvm@a3882af
https://github.com/dhiltgen/docker-machine-kvm/commit/a3882af "Clean
up logging a little"
https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.6.0
https://github.com/dhiltgen/docker-machine-kvm/releases/tag/v0.6.0
dhiltgen/docker-machine-kvm@8d63cea
https://github.com/dhiltgen/docker-machine-kvm/commit/8d63cea "Add
disk cach/io tunables"
https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.5.6
https://github.com/machine-drivers/docker-machine-driver-libvirt/releases/tag/v0.0.6
No major differences, but it would be nice to have a full import:

b14a80e (tag: v0.8.2) Merge pull request #33 from r2d4/leases
|
| * e5724f2 Use last entry when parsing DHCP status file
|   dc9207e Merge pull request #34 from dongsupark/dongsu/migrate-libvirt
|\
| * | a7eeb00 import libvirt/libvirt-go instead of alexzorin/libvirt-go
| |/
|   cd8ae9d Merge pull request #32 from caio1982/cpuhost
|\
| |/
|/|
| * c7becbf The current domain XML used for the KVM driver does not specify a CPU mode, which makes things very slow and usually lead to a constant >25% CPU usage for people using your driver with Kubernetes and Minikube. This solves the problem as it simply tells KVM to use everything the host supports and emulate whatever else is needed.
|/
37bb4cc Put NewVirConnection inside of PreCreateCheck, added error message for (#18)
a6ecdc2 Update gitignore based on new build layout
7077fd0 Refactor README (#25)
9fb6d24 Merge pull request #22 from mikefaille/cmd-path
|
| * a2f0b19 FIX : Permit go build . to output right name
|/
e302cd8 Add license file
5f4c9b1 Add disk cach/io tunables
| * a3882af (tag: v0.7.0) Add disk cach/io tunables
| * 8d63cea (tag: v0.6.0) Clean up logging a little
|/
4bc0ec9 Add a little more detail on getting started
c8f7e8a Merge pull request #8 from dpw/readme-quick-start
|
| * 28e100b Add quick start instructions
|/
f4dee20 (tag: v0.5.6) Merge commit '6096a1a39eb1ec6878c797b45afbe0e48410eda0' into rsampaio_conn_fix

Also because this version (v0.7.0) is currently being used by minishift:
https://docs.okd.io/latest/minishift/getting-started/setting-up-virtualization-environment.html#setting-up-kvm-driver

https://github.com/dhiltgen/docker-machine-kvm/releases/download/v0.7.0/docker-machine-driver-kvm
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/machine-drivers/docker-machine-driver-libvirt/issues/4,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAHZg4SMXCAqKG3cVF6aMb6gOQs-a6mks5uY_nMgaJpZM4Wf9Cq
.

Right you are! Now it is done.


I failed to run Python 3 on Windows 10

The system prompts:
[Running] /usr/bin/env python3 "c:\Users\hp\Desktop\python\1.py"
ϵͳ�Ҳ���ָ����·����
[Done] exited with code=1 in 0.068 seconds
I want to know why.

Here is my Python source code:
#!/usr/bin/env python3
# -*- coding: utf8 -*-
a = 1
while a < 10:
    a = a + 1
    print(a-1)
print('$')

Unfortunately you are using the Code Runner extension to run your code and not the Python extension (the Play button is white for Code Runner, green for the Python extension). You will have to change the configuration for Code Runner to make this work or disable it so that the Python extension provides the functionality instead.


Feature Request: Override titlepage completely (PDF)

I'd like to just design the title page PDF myself and then add it as the title page. This currently does not seem possible, even via workarounds like setting a titlepage background and trying to make the text transparent (titlepage-text-color will not accept transparent values).

I'm thinking that a titlepage-override-latex parameter would be nice:
$if(titlepage-override-latex)$
  $titlepage-override-latex$
$elseif(titlepage-background)$
{
% The titlepage with a background image has other text spacing and text size
[...]
}
$else$
{
[...]
}
$endif$

@Wandmalfarbe: I'm willing to implement this and the associated tests, if it's ok for you

@ctjlewis: thanks for the idea of using pdfpages ! I wrote a filter that  generates the cover based on an SVG template and the pandoc metadata
https://github.com/daamien/pandoc-cover/
By the way, a couple of remarks about your solution:

AFAIK The \pagebreak command is not necessary
This may not work when the table of content (toc is activated), you insert the cover using the include-before variable

include-before:
   - \includepdf{Cover.pdf}

I'd also like the ability to override the cover page.
I am not familiar with SVG templates, as used in pandoc-cover.
The ability to use any files supported by pandoc as template is most versatile.
We could use pandoc partials in that respect in order to have variables in that temple be expanded (see https://github.com/jgm/pandoc/issues/5931#issuecomment-557847759).
Something along the lines of:
\begin{titlepage}
$cover.tex()$
\end{titlepage}


NullPointerException: Securestoragelibrary.SecurePreferences.contains

Hello,
I'm having a crash. When the app is backgrounded for periods of time, the system kills it. When the app re-initializes and dagger is setup again, I get this crash.
Caused by: java.lang.NullPointerException: Attempt to invoke virtual method 'java.lang.Object java.lang.ref.WeakReference.get()' on a null object reference
        at de.adorsys.android.securestoragelibrary.SecurePreferences.contains(Unknown Source:2)

Any ideas?

Hey, thanks for reporting this. So you use SecureStorage with Dagger? Why? You can access every method statically. No additional setup required.

I'm not injecting SecureStore with dagger. I'm injecting SecureStorage
internal interface SecureStorage {
    val downloadSecretKey: ByteArray
}

internal class SecureStorageImpl : SecureStorage {
    private companion object {
        const val DOWNLOAD_KEY = "download_key"
        const val STRING_LENGTH = 20
        const val ALLOWED_CHARS = "ABCDEFGHIJKLMNOPQRSTUVWXTZabcdefghiklmnopqrstuvwxyz0123456789"
    }

    override val downloadSecretKey: ByteArray
        get() {
            return if (SecurePreferences.contains(DOWNLOAD_KEY)) {
                SecurePreferences.getStringValue(DOWNLOAD_KEY, null)!!.toSecretByteArray
            } else {
                randomString.apply {
                    SecurePreferences.setValue(DOWNLOAD_KEY, this)
                }.toSecretByteArray
            }

        }

    private val randomString: String
        get() {
            return (1..STRING_LENGTH)
                    .map { ALLOWED_CHARS.random() }
                    .joinToString("")
        }

    private val String.toSecretByteArray: ByteArray
        get() {
            val keyBytes = ByteArray(16)
            val md = MessageDigest.getInstance("SHA-256")
            md.update(this.toByteArray())
            System.arraycopy(md.digest(), 0, keyBytes, 0, keyBytes.size)
            return keyBytes
        }
}

We will look into the issue. It's related to the apps context. We use a ContentProvider to get the COntext, which doesn't seem to work in your case...

Hi @NathanSass, we refactored the way we handle context in SecureStorage. This implies you now have to pass the context directly to the methods. This solves your problem. Make sure to update to 1.1.0 as soon as possible

1.1.0 has just been released


(Wiki) Importing audio has new parameters

The 'EAudioFormat' parameter does not exist in newer versions. It has been replaced with the 'ERuntimeAudioFormat' parameter.
RuntimeAudioImporter->ImportAudioFromFile(TEXT("C:/Folder/Example.mp3"), EAudioFormat::Auto);

The correct line should be:
RuntimeAudioImporter->ImportAudioFromFile(TEXT("C:/Folder/Example.mp3"), ERuntimeAudioFormat::Auto);

wiki page
Extras:
If you are using old versions, the 'ETranscodingStatus' parameter has been replaced with the 'ERuntimeImportStatus' parameter.

Thanks for pointing this out, should be fixed!


A question about alpha value

In tineuvox.py at line 324, the alpha is calculated as alpha = nn.Softplus()(density_result+self.act_shift)
But it seems that the alpha should be alpha = 1 - exp(-density_result * dist)
I am confused by this, is this a kind of approximation?

Hi, following MipNeRF and DirectVoxGo, softplus is used for better optimization.

Thanks for replying to me!
I see, but should it be:
density = nn.Softplus()(density_result+self.act_shift)
alpha = 1 - exp(-density * dist)
```?
MipNeRF and DirectVoxGo only changed the activate function from ReLU to Softplus, but they still need the dist to get alpha right?

Sorry to bother you, but I still can't figure out this question...

Hey yeah, dist is omitted, which is yet constant as points are sampled with equal intervals in our implementation; so it leads to negligible effects. If you change to sample points in a different way, better follow the strict formulation to compute alpha.

OK! I understand. Your code has helped me a lot. Thank you very much!


Adjust copy and delete

Summary

Adjusted process-delete module and process-copy module

Removed don't-ask-again option (user-preferences and checkbox)
Removed single-delete component
Implemented Scrollbar component in copy-modal
Removed option to keep multiple editing tabs open in copy-modal (only one open at a time)

Added copy function to process-list's action bar and replaced process-delete module in processes.tsx and process-list.tsx with pop-confirmations --> resulting in error for both delete and copy (todo!)

Adjusted mobile view for screensize <= sm

Hide side-bar
Make PROCEED icon appear in header
Hide process metadata

Popconfirm should be replaced by confirmation-button, for which you have to wait until this pr is merged.

Once @FelipeTrost is happy with his review, I can take care of the merge conflicts with main, since I changed a lot with the switch to Server Components.


Codespace: Pylance diagnostics out of date

Environment data

VS Code version: 1.60-Insiders
Jupyter Extension version (available under the Extensions sidebar): 2021.8.2027072902
OS (Windows | Mac | Linux distro) and version: Mac
Python and/or Anaconda version: 3.8
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): venv
Jupyter server running: Local

This should be fixed with the latest build of the python extension.


Mac crash 7.4.170

"Toggl Desktop doesn't work anymore.
(Quits each time I enter a word in the case "what are you working on")"
Screenshots of crash dump included in case/720269

I was unable to replicate the issue with similar data that user has.
Might be connected to the Mac version. User has Mac OS X 10.9.5.

Potential second case at /case/721837
log files included, will ask if they can update OSX

/case/722770

Tried replicating on macOs 10.10 with no success.

Hi @IndrekV to clarify: the desktop app won't work on macOS 10.9.5 and earlier?

I haven't been able to reproduce the issue with later systems and macOs 10.9.5 is so old that it is impossible for me to get a machine running it. It's almost 5 years old and the latest update for the system was 4 years ago. I think there isn't sadly anything we can do about it. We just need to update the wording on our page that describes what operating systems we support.

I think I actually found a way to test it. But we will need to start dropping support for old systems.

No Update on the solution yet. I've narrowed the issue down to some part of the autocomplete dropdown positioning logic, but have not found a fix yet.

Found the cause. We were using methods that were not supported in the older versions of macos. Will see if I can work around this.

Same issue for me.
I'm running 10.9.5.
I've been using the app since april 2015 with no problem.  Have been running 7.4.170 probably since it was updated.
I confirm that since few weeks I'm experiencing the same issue.
I haven't updated or installed any software on my mac since in the past weeks.

There are still some issue, but I'm hopeful I can fix them and create a new version that works ok on mac 10.9

Thanks for the update.
Still crahsed with the 7.4.194, though. Anytime you type anything in the description or project text boxes.
Crash log attached.

I just created a new development version. Could someone test it out to see if we'e managed to fix this thing. https://github.com/toggl/toggldesktop/releases/download/v7.4.203/TogglDesktop-7_4_203.dmg

Yes!
Autocomplete is back with 7_4_203.
No more crash :)


feat: add new stacking strategies

Brief Information
This pull request is in the type of:

[x] bug fixing
[x] new feature
[ ] others

What does this PR do?
This PR adds a new property stackStrategy to the SeriesStackOptionMixin which defaults to samesign (current behavior). Associated PR on the docs repo: https://github.com/apache/echarts-doc/pull/255
Fixed issues

Closes #9317
Close #9141
Close #13747
Close #15083
Close #14365

Details
Before: What was the problem?
Currently stacking is only applied if the cumulative sum has the same sign as the value to be stacked. This makes it impossible to mix positive and negative values in the same series, something typically needed when creating confidence bands:

After: How is it fixed in this PR?
Here we introduce the following new stacking options:

all: stack all values, irrespective of the sign of the stackable value:

positive: only stack positive values:

negative: only stack negative values:

We also add the strategy samesign which corresponds to the current behavior which we default to to avoid introducing a breaking change. However, I propose defaulting to all in the next major version, as I assume this is the behavior users will expect to see.
Misc

[ ] The API has been changed (apache/echarts-doc#xxx).
[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
New test cases added to the area-stack.html test.
Others
Merging options

[x] Please squash the commits into a single one when merging.

Other information

Docs for the proposed feature: https://github.com/apache/echarts-doc/pull/255

@pissang let me know if you feel this should be implemented some other way so I can change the PR accordingly

@villebro Thanks for the work!  All looks good to me except a tiny code style issue.

This could fix a lot of issues.  I have added in PR description


use node to parse php ,that is goods idea.

can I user define php grammar,  analysis expression.
like:
main.php
<?php

@Controller
class main
{
    @Router("/xxx/xxx", "GET")
    public function __construct()
    {
        echo "main:main";
    }
}

node
let main = fs.readFileSync('./main.php');

let mainWriter = new writer(main);
let mainClass = mainWriter.findClass('main');
let decorators = mainClass.getDecorators();
decorators.forEach(decorator=> dosomething);

Hi @wuchuguang,
I've done something similar here (working on decorators) : https://github.com/ichiriac/pdomap/blob/master/main.js#L71 - it's a work in progress but you can take a look at the documentation in order to see what it can parse...
The trick is for requesting and searching data I'm actually using another library
php-reflection based also on php-parser and doc-parser for reading annotations.
With this library you are indexing only one file, or as many as you need (your whole project), and the all project responds as a graph database, so you can look at inheritance, calls, instanciations ... etc ...

@ichiriac ok , I try it.

@ichiriac thx


Update LegacyPassword with less than core.loginRegistration.passwordMinLength not possible

PHP Version
8.1
Shopware Version
<IP_ADDRESS>
Expected behaviour
Imported customers with LegacyPassword can easily update their passwords and core.loginRegistration.passwordMinLength schould be ignored or the customer should get an prompt to update the password.
Actual behaviour
It's not possible to login as customer with LegacyPassword less than core.loginRegistration.passwordMinLength.
How to reproduce
Add customer with LegacyPassword less than core.loginRegistration.passwordMinLength and try to login via storefront.

Can you open a PR?

Should be done with the #2990


Add pub struct Formatter to defmt_decoder::log.

The purpose of Formatter is to format defmt frames according to the log format. Specifically I will be using this to add custom log format support to probe-rs.
@andresovela Would you have time for a review?

I am a bit short on time and will therefore merge this PR as is. I linked the discussion from the code and will refactor it at a later point. Otherwise I am open for a refactoring PR.


vioinput: Fix catalog file generation

resolves:
BZ#2012756 - [virtio-win][vioinput]Signtool verify viohidkmdf.sys file failed with virtio-win-prewhql-211 on win10+ guests
BZ#2141603 - HCK-CI failed to install virtio-input driver

Signed-off-by: Kostiantyn Kostiuk<EMAIL_ADDRESS>

Hi @kostyanf14 ,
We still have a problem:
`, [2022-11-15T10:47:22.217335 #96784]  WARN -- : Installing driver package on machine CL1 failed
I, [2022-11-15T10:47:32.217799 #96784]  INFO -- : Trying again to install driver package on machine CL1
W, [2022-11-15T10:47:56.851561 #96784]  WARN -- : Tools action failure
-- Running 'pnputil -i -a C:\Users\ADMINI~1\AppData\Local\Temp\007a01e0-dd29-4134-9cc1-4c44a3d72f64\vioinput.inf' failed with exit code -536870325.
--    -- stdout:
-- Microsoft PnP Utility
-- Processing inf :            vioinput.inf
-- Adding the driver package failed : The hash for the file is not present in the specified catalog file. The file is likely corrupt or the victim of tampering.
--
-- Total attempted:              1
-- Number successfully imported: 0
W, [2022-11-15T10:47:56.851829 #96784]  WARN -- : Installing driver package on machine CL1 failed
I, [2022-11-15T10:48:06.852295 #96784]  INFO -- : Trying again to install driver package on machine CL1
W, [2022-11-15T10:48:30.023739 #96784]  WARN -- : Tools action failure
-- Running 'pnputil -i -a C:\Users\ADMINI~1\AppData\Local\Temp\24f56d4d-60cd-4c1e-a234-feb8cc4877b2\vioinput.inf' failed with exit code -536870325.
--    -- stdout:
-- Microsoft PnP Utility
-- Processing inf :            vioinput.inf
-- Adding the driver package failed : The hash for the file is not present in the specified catalog file. The file is likely corrupt or the victim of tampering.
--
-- Total attempted:              1
-- Number successfully imported: 0`
Best regards,
Yan.

Looks like we need a packaging project. When I build SDV for vioinput and hidpassthrough, the cat file corrupted


Examples panic at runtime

Running the example styling results in the follow
The back trace can be found here https://pastebin.com/63ESmAMy.
OS: Windows 10 Pro version 1909 (build 18363.815) 64 bit
Graphics: Intel(R) HD Graphics, driver version: <IP_ADDRESS>59

Do the wgpu examples  work for you?
As far as I know, the DirectX 11 backend for wgpu still has some rough edges. You should try to use DirectX 12 if possible.

So I've tested all of the examples and they all worked except the hello-compute example.

After modifying the line for selecting a backend to wgpu::BackendBit::DX12 the examples successfully run using the PCs virtual GPU!
I forgot to have mentioned in the original post that the PC I'm running it on does not have a dedicated GPU.

We should change that line to use wgpu::BackendBit::PRIMARY, as DX11 and OpenGL are not well supported yet.

Sure! Should the PR target both the master and v0.1 branch?

Just master. We can cherry-pick later.


Add a Makefile to each chart

Add a Makefile in each chart folder to simplify maintenance operations like packaging or publishing the new charts.

Fixed in https://github.com/request-yo-racks/charts/pull/9.


iOS: Native cordova camera  FILE_URI and DATA_URL

I have tried using fileReader and base64 native but it is not working, I have used same in ionic 3 . I think it is cause of webview ionic:// or earlier we used to have file://
https://github.com/indraraj26/camera-test-ios/blob/master/src/app/home/home.page.ts
Repo: https://github.com/indraraj26/camera-test-ios/
When we choose DATA_URL it is crashing timeout timeout.
ionic info:
Ionic:

   Ionic CLI                     : 5.2.3 (/usr/local/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.11.8
   @angular-devkit/build-angular : 0.801.3
   @angular-devkit/schematics    : 8.1.3
   @angular/cli                  : 8.1.3
   @ionic/angular-toolkit        : 2.1.2

Cordova:

   Cordova CLI       : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms : ios 5.1.1
   Cordova Plugins   : cordova-plugin-ionic-keyboard 2.2.0, cordova-plugin-ionic-webview 4.1.3, (and 5 other plugins)

Utility:

   cordova-res : not installed
   native-run  : 0.2.8 (update available: 0.3.0)

System:

   ios-deploy : 1.9.4
   ios-sim    : 8.0.2
   NodeJS     : v12.13.0 (/usr/local/bin/node)
   npm        : 6.12.0
   OS         : macOS High Sierra
   Xcode      : Xcode 10.1 Build version 10B61

It's probably not a good idea to log big base64 strings, that seem to cause the timeout.
Since it only happens when you log it, it's probably a cordova-ios problem, not a plugin problem.
If instead of loging it I display in a img tag it works as expected.


Illuminate\Foundation\Testing\Concerns\MakesHttpRequests::decodeResponseJson() treats false as invalid JSON

As title,
null is the only error returned by json_decode, where false is a valid JSON document.

Please send a PR. :)


feat(solid): auto upload mode for solid. Fix of TW config

Is the drop zone package broken? I don't seem to get any files selected at all...

@juliusmarminge Just checked on my side and all good 🤔


gdb: 9.1 -> 9.2

Motivation for this change
See https://lists.gnu.org/archive/html/info-gnu/2020-05/msg00008.html
for release information
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg build gdb


Requesting events with "since" and "until" not reliably polling

Output of docker version:
Client:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 21:04:48 2016
 OS/Arch:      darwin/amd64
 Experimental: true

Server:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 21:04:48 2016
 OS/Arch:      linux/amd64
 Experimental: true

and
Client:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 18:34:23 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.0
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   4dc5990
 Built:        Wed Apr 13 18:34:23 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 99
 Running: 2
 Paused: 0
 Stopped: 97
Images: 52
Server Version: 1.12.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 250
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host overlay null
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: seccomp
Kernel Version: 4.4.15-moby
Operating System: Alpine Linux v3.4
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 1.954 GiB
Name: moby
ID: A7F5:MEE2:XJ2N:ZQ6Q:HRKO:SRQJ:CTJF:RR3B:6K7F:SQPP:NBJ7:I7MM
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 26
 Goroutines: 41
 System Time: 2016-08-10T18:29:32.322513876Z
 EventsListeners: 1
No Proxy: *.local, 169.254/16
Username: flavin
Registry: https://index.docker.io/v1/
Experimental: true
Insecure Registries:
 <IP_ADDRESS>/8

and
Containers: 7
 Running: 0
 Paused: 0
 Stopped: 7
Images: 1
Server Version: 1.11.0
Storage Driver: aufs
 Root Dir: /var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 15
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge null host
Kernel Version: 4.2.0-27-generic
Operating System: Ubuntu 14.04.4 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.954 GiB
Name: xnat
ID: 3XVY:66LA:O7YA:ZRXD:5O3L:HR56:FBKD:5CG3:HAV3:JDG7:K3MX:44G7
Docker Root Dir: /var/lib/docker
Debug mode (client): false
Debug mode (server): false
Registry: https://index.docker.io/v1/

Additional environment details (AWS, VirtualBox, physical, etc.):
the first machine is my local Docker for Mac installation; the second is an Ubuntu 14.04 VM in VirtualBox.
Steps to reproduce the issue:

Use the remote API to GET events, using the since query param.. For example, here is a simple example that asks for all events since time 0. (I have verified this behavior with other, more realistic time values.)

curl --unix-socket /var/run/docker.sock http://localhost/events?since=0

Describe the results you received:
The events in the specified time range (in the example, from time 0 until now) are sent to me from docker, and the HTTP stream remains open.
Describe the results you expected:
The events in the specified time range (in the example, from time 0 until now) are sent to me from docker, and the stream is closed. This is the documented behavior when using the since or until query params.

Hm, wondering if this is a documentation error; still digging in the history, but this PR; https://github.com/docker/docker/pull/2348 seems to indicate that --since should also stream
Show previously created events and then stream

Never mind, looks like that PR may have the wrong description; the original PR mentions the "polling"; https://github.com/docker/docker/pull/1180

I just tried on docker 1.5.0 and it shows the same behavior, so if the behavior is wrong, it's been like that for a very long time
ping @vieux any idea? is this just a documentation issue ("polling")?

I think the description os wrong, it should indeed be Show all events created since timestamp and then stream
The current behavior is correct in my opinion.

If the documentation about since and until switching from streaming to polling, is there no way to signal that I don't want to stream? Just a stream=false param or something?

how about
--since=0  --until=`date +%s`

We have a  --no-stream (Disable streaming stats and only pull the first result) for docker stats, so perhaps it can be considered; is there a specific use case?
Meanwhile, let me fix the docs for this flag
if @vieux suggestion doesn't solve your use-case, could you open a new issue / feature request for that flag?

That does work. Combining since and until returns the events and closes the stream.
An explicit flag would be nice, but since there is a way to get the behavior I want, I wouldn't consider it necessary. (As long as the documentation makes this behavior clear.)

Can we reopen this issue? I tested it yesterday and it worked (as I commented above). But in retesting it, it does not work.
Both docker events --since <some time> --until <some time a few seconds later> and curl --unix-socket /var/run/docker.sock 'http://localhost/events?since=<some time>&until=<some time a few seconds later>' are leaving the stream open.
$ docker version
Client:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 21:04:48 2016
 OS/Arch:      darwin/amd64
 Experimental: true

Server:
 Version:      1.12.0
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   8eab29e
 Built:        Thu Jul 28 21:04:48 2016
 OS/Arch:      linux/amd64
 Experimental: true

Looks like it doesn't stop if there's no events at the until time, or events "after" until (I guess it keeps waiting until it receives an event for that, which would make sense if --until is a future date)
I'll reopen so that we can check the intended behavior and/or improve (I think it --until is in the past, and there's no more events, then it should disconnect. Possibly, if --until is in future, then wait until events take place)

Your suggestion makes sense to me. If I give an until time that is in the past there is no need to stream. There is only a finite set of events that could possibly match the criteria in the request; once they're all sent the connection should be closed.
Any word on whether this change/improvement will be considered?

Any word on whether this change/improvement will be considered?

Given that this behavior has not changed (it's not a regression), don't expect it to be given a high priority, but I can see this being a nice isolated issue for a contributor to work on

#dibs

I am unable to reproduce this issue (in addition to the version below, I tested on the current dev version). If until is in the future, the connection streams up until the future time is reached (regardless of whether it's combined with since). If until is in the past or present, the connection closes immediately. Here are some examples:
# since and until are both in the past -- terminates immediately as expected

time curl --unix-socket /var/run/docker.sock
    "http://localhost/events?since=$((-5 + $(date +%s)))&until=$((-3 + $(date +%s)))"
real	0m 0.00s

# `since` is in the past with no `until` given -- streams indefinitely as expected

time curl --unix-socket /var/run/docker.sock
    "http://localhost/events?since=$((-500 + $(date +%s)))"
^CCommand terminated by signal 2
real	0m 11.84s

# `since` is in the past and `until` is in the future -- streams until reaching `until`, then terminates

time curl --unix-socket /var/run/docker.sock
    "http://localhost/events?since=$((-5 + $(date +%s)))&until=$((5 + $(date +%s)))"
real	0m 4.31s

# `since` is not given and `until` is in the future -- streams until reaching `until`, then terminates

time curl --unix-socket /var/run/docker.sock
    "http://localhost/events?until=$((3 + $(date +%s)))"
real	0m 2.40s

Am I missing something, or is this working as intended? It also appears that no further docs updates are needed since #25588 addresses this behavior.
/ # docker version
Client:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 17:52:38 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   23cf638
 Built:        Thu Aug 18 17:52:38 2016
 OS/Arch:      linux/amd64

I'm not able to reproduce either
docker events --since 2016-10-10T16:00:00
2016-10-10T16:24:06.041323948+02:00 container die 43e590051f5b270ffb2311df11386a3601d53cb88403827ee824f8d40426822c (exitCode=0, image=debian, name=boring_payne)
2016-10-10T16:24:06.321929200+02:00 network disconnect 8ea6824048558eec134d83f58a801454028d52c6ec48299b0c4ccb820bd1645c (container=43e590051f5b270ffb2311df11386a3601d53cb88403827ee824f8d40426822c, name=bridge, type=bridge)
2016-10-10T16:24:06.506482280+02:00 container destroy 43e590051f5b270ffb2311df11386a3601d53cb88403827ee824f8d40426822c (image=debian, name=boring_payne)

So last event is 16:24:06
docker events --until 2016-10-10T16:30:00

(returns immediately)

docker events --since 2016-10-10T16:25:00 --until 2016-10-10T16:30:00

(returns immediately)

Setting a future date does block (current time is 17:36), which is also how I updated the docs (https://github.com/docker/docker/pull/25588), i.e., stream until --until is reached.
docker events --since 2016-10-10T16:25:00 --until 2016-10-10T20:30:00

@johnflavin I think this is the expected behavior, or did you expect it to stop streaming in all cases?

The polling behavior you described is what I would expect, but I can't reproduce it.

Both since and until in the past (note: I'm using the date -v option that lets you specify time offsets)

$ curl --unix-socket /var/run/docker.sock "http://localhost/events?since=$(date -v -5S +%s)&until=$(date -v -1S +%s)"
<stream remains open>
$ docker events --since=$(date -v -5S +%s) --until=$(date -v -1S +%s)
<stream remains open>

since in the past, until in the future

$ curl --unix-socket /var/run/docker.sock "http://localhost/events?since=$(date -v -5S +%s)&until=$(date -v +1S +%s)"
<stream remains open indefinitely, longer than the 1 second I would expect>
$ docker events --since=$(date -v -5S +%s) --until=$(date -v +1S +%s)
<stream remains open indefinitely, longer than the 1 second I would expect>

My docker version is different now than it was when I opened this ticker, so I'll update it here. I'm on the beta channel of docker for mac.
Client:
 Version:      1.13.0-rc2
 API version:  1.25
 Go version:   go1.7.3
 Git commit:   1f9b3ef
 Built:        Wed Nov 23 17:40:58 2016
 OS/Arch:      darwin/amd64

Server:
 Version:             1.13.0-rc2
 API version:         1.25
 Minimum API version: 1.12
 Go version:          go1.7.3
 Git commit:          1f9b3ef
 Built:               Wed Nov 23 17:40:58 2016
 OS/Arch:             linux/amd64
 Experimental:        true

Note that, if this is actually a regression, it could be recent. I have had a test case for this in my code for a while, and that test only started failing this morning. I'll give it here (sorry for speaking java):
@Test(timeout = 10000)
public void testEventPolling() throws Exception {
    controlApi.pullImage(BUSYBOX_LATEST);

    // Create container, to ensure we have some events to read
    final ContainerConfig config = ContainerConfig.builder()
            .image(BUSYBOX_LATEST)
            .cmd("sh", "-c", "echo Hello world")
            .build();

    final Date start = new Date();
    final ContainerCreation creation = client.createContainer(config);
    client.startContainer(creation.id());

    Thread.sleep(500); // Wait to ensure we get some events
    final Date end = new Date();
    final List<DockerContainerEvent> events = controlApi.getContainerEvents(start, end);

    // The fact that we have a list of events and not a timeout failure is a victory
    assertThat(events, not(empty()));
}

This test makes a container, waits a little while, then reads the events. It uses since and until parameters that are not in the future, so it should never stream, only poll. If it gets any events (i.e. doesn't block forever waiting for the stream to end) it is a success. But right now this test fails by timing out after 10 seconds.

I'm trying to reproduce (also Docker for Mac), but the stream immediately detaches :cry:
curl --unix-socket /var/run/docker.sock "http://localhost/events?since=$(date -v -5S +%s)&until=$(date -v -1S +%s)"

Could it be that the clock of docker for mac's VM is out of sync? Given that the offset is just 1 second, it could make a difference.
docker info | grep "System Time"

Wow, you’re right. The system time is off. It thinks it is yesterday.
$ docker info | grep Time
 System Time: 2016-11-30T23:36:43.20583885Z

How can I fix that?

It should sync periodically, but this thread may contain some pointers / workarounds; https://github.com/docker/for-mac/issues/17

Also, not sure it uses the same timezone, or if it defaults to UTC

Yep, that thread helped. I re-synced the clock with $ docker run --rm --privileged alpine hwclock -s and now the event polling works exactly as expected.

I'm guessing this was the underlying cause of my issue all along, thus this issue is safe to close.

Thanks! Happy we solved that mystery!


Not able to fetch title, description, and images for a tweet (youtube works though)

Describe the bug
Not able to fetch title, description, and images for a tweet
To Reproduce
Steps to reproduce the behavior:

use the librarygetLinkPreview('https://twitter.com/elonmusk/status/1515799688296943636', {proxyUrl: xxx}).then(data => console.log(data))
get result

{
  "url": "https://twitter.com/elonmusk/status/1515799688296943636",
  "title": "",
  "siteName": "Twitter",
  "mediaType": "website",
  "contentType": "text/xml",
  "images": [],
  "videos": [],
  "favicons": [
    "https://abs.twimg.com/favicons/twitter.2.ico",
    "https://abs.twimg.com/responsive-web/client-web-legacy/icon-ios.b1fc7276.png"
  ]
}

add User-agent: Twitterbot to headers give me the same result

However, I am able to get all these info for a Youtube link https://www.youtube.com/watch?v=MejbOFk7H6c.
{
  "url": "https://www.youtube.com/watch?v=MejbOFk7H6c",
  "title": "OK Go - Needing/Getting - Official Video",
  "siteName": "YouTube",
  "description": "Website | http://www.okgo.netInstagram | http://www.instagram.com/okgoTwitter | http://www.twitter.com/okgoFacebook | http://www.facebook.com/okgoStore | htt...",
  "mediaType": "video.other",
  "contentType": "text/xml",
  "images": [
    "https://i.ytimg.com/vi/MejbOFk7H6c/maxresdefault.jpg"
  ],
  "videos": [],
  "favicons": [
    "https://www.youtube.com/s/desktop/b292b5ed/img/favicon_32x32.png",
    "https://www.youtube.com/s/desktop/b292b5ed/img/favicon_48x48.png",
    "https://www.youtube.com/s/desktop/b292b5ed/img/favicon_96x96.png",
    "https://www.youtube.com/s/desktop/b292b5ed/img/favicon_144x144.png",
    "https://www.youtube.com/s/desktop/b292b5ed/img/favicon.ico"
  ]
}

Expected behavior
Should get description and images of the tweet https://twitter.com/elonmusk/status/1515799688296943636
Screenshots
Here's the tweet:

Desktop (please complete the following information):

OS: macOS 10.15.7
Browser: chrome 100.0.4896.60
Version: 2.1.13

Additional context
Thank you very much for the library, and appreciate your help in advance!!!

Read the README

Hi @ospfranco, thanks for the reply! I have read README though I didn't find the answer. Could you please help explain? Thanks a lot!

You cannot do cross-domain requests from a browser

You cannot do cross-domain requests from a browser

Oh, it is through my backend proxy, let me update the issue description. Here's the backend log
INFO:     None:0 - "GET /cors_proxy/https%3A//twitter.com/elonmusk/status/1515799688296943636 HTTP/1.1" 200 OK

Can you help revisit the issue? Thanks!

Then it is not a problem with the library, as you can see you can get the response from youtube, so Twitter is not returning the correct headers/information to extract the info.
It either purposefully does not return this info (maybe due to some user config?) or is not using OpenGraph tags, or changed the response shape so the parsing/extracting is not working.
As stated in the README you will have to debug this type of issue yourself. The code is fairly simple, it just fetches the HTML and looks for the correct OpenGraph tags. If the info is there, you can submit a PR that tries to take care of exceptional cases by parsing other tags. Otherwise, the info is just not there and there is nothing that can be done about it.

Then it is not a problem with the library, as you can see you can get the response from youtube, so Twitter is not returning the correct headers/information to extract the info.
It either purposefully does not return this info (maybe due to some user config?) or is not using OpenGraph tags, or changed the response shape so the parsing/extracting is not working.
As stated in the README you will have to debug this type of issue yourself. The code is fairly simple, it just fetches the HTML and looks for the correct OpenGraph tags. If the info is there, you can submit a PR that tries to take care of exceptional cases by parsing other tags. Otherwise, the info is just not there and there is nothing that can be done about it.

@ospfranco thank you so much for the details, I will try to PR if I can. Thanks a lot, again :)

Since this is a social network it could also be an issue with redirections or facing a login screen redirection. In any case, the library is so simple that 98% of the time it is not a problem with the code, but just how the services respond to requests.


Format your description

Markdown contains code highlighting tools
https://guides.github.com/features/mastering-markdown/
This is for your README.md file.

Completed, description and code have been styled

Do not edit files in the web-interface.
You have to use feature-branch for each issue and link commit with issue via comment.


Login - Magic code is sent to wrong email after editing when using ENTER key

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.24-4
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to www.staging.new.expensify.com
Enter any email and use Enter button to continue to next page
On magic code page select "go back"
Edit email and use "Enter" to continue

Expected Result:
Magic code is sent to new email entered
Actual Result:
Magic code is sent to initial email
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] Windows: Chrome
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/ee3d7836-9468-4a8f-b066-d334cf96f4ce
View all open jobs on GitHub

Not reproducible on main

This PR https://github.com/Expensify/App/pull/34313 solved the issue
The root cause is in here
https://github.com/Expensify/App/pull/34313/files#diff-2a0d77decd98a62a279ac9fb139cad79d6d7a20f4c724de3dc811911bc5dd9b1L142-L151
onPress wasn't added to the dependency. We can CP this PR

Yep thanks, confirmed not reproducible on main:
https://github.com/Expensify/App/assets/7277067/f7f814f6-4151-430e-be47-bcbf2254f2f9
But reproducible on staging branch locally:
https://github.com/Expensify/App/assets/7277067/bf444b41-92df-4191-939b-be9527a1303d
I'll check in about the cherry pick, I'm surprised it hasn't been already

I think @Julesssss is requesting CP https://github.com/Expensify/App/pull/34313

I think he's off for the night, I'm requesting it now

The CP is complete. I retested on staging to verify it's fixed:
https://github.com/Expensify/App/assets/7277067/e2c3e587-6625-4008-9ff8-c8c33d5bb6c8

Oh thanks @amyevans I forgot to CP this


PawnPlus & MySQL

amx_forked(fork_data)
{
CompareKeys = task_new();
amx_forked(.use_data = true)
{
threaded(sync_explicit)
{
thread_sync();
new query[224];
mysql_format(Base, query, sizeof(query), "SELECT u.ID aid, u.Name nm, ui.last_log la_log FROM users u JOIN userinfo ui ON ui.id = u.id WHERE u.kie LIKE '%e%%' GROUP BY ui.aid ORDER BY ui.last_log DESC;", "test");
mysql_tquery(Base, query, "UsersCompareKeys");
}
}
await CompareKeys;
printf("%d", task_get_result(CompareKeys));
task_delete(CompareKeys);
}

Result: [15:01:10] [ERROR] CCallback::Execute - invalid AMX instance
Following the information on how this works, PawnPlus creates a new amx and works with it, because of which the address changes. If you do not use the newly created amx - this loads the processor up to 100%. This is half the code that I showed, I want to use this for significant optimization in future code. Is it possible to add an optional parameter here that will work not only with one amx, but also with the amx that the user enters?
Something like this:
mysql_tquery_ext(MySQL:handle, const query[], const callback[] = "", Amx:source = amx_this(), const format[] = "", {Float,_}:...);

https://pastebin.com/pMnXSMpN


Pub/Sub: more refactoring

Using Preconditions
Adding onSuccess and onFailure to OutstandingPublish

Looks like you need to run the code formatter again.


preventDefault error: [Chrome] Unable to preventDefault inside passive event listener due to target being treated as passive.

While using Google Chrome i got an error:

[Intervention] Unable to preventDefault inside passive event listener due to target being treated as passive. See

I actually fixed this by removing all the uses of preventDefault()  function in selectric plugin.
I don't know why, but it works.

+1

+1

The error occurs when we "scroll" with mouse inside the Drop-down

The error occurs when we "scroll" with mouse inside the Drop-down

do you have any ideas how to fix it?

do you have any ideas how to fix it?

I actually fixed this by removing all the uses of preventDefault() function in selectric plugin. like @Robb110

do you have any ideas how to fix it?

I actually fixed this by removing all the uses of preventDefault() function in selectric plugin. like @Robb110

sounds not cool)

do you have any ideas how to fix it?

I actually fixed this by removing all the uses of preventDefault() function in selectric plugin. like @Robb110

sounds not cool)

but solved the problem, I remove only one "preventDefault" in my source

Is there a fix that keeps the scrolling behavior of preventdefault in tact?


Cr86 retrieve case by case

Motivation and Context
This change adds an endpoint to retrieve a case by CaseId.
It's using our latest understanding of the case service response DTO's, so this is an area for potential changes as the RM team complete their work on the endpoints to fetch cases.
What has changed
Addition of endpoint.
How to test?
Run all test cases.
Links
https://collaborate2.ons.gov.uk/jira/browse/CR-86

Codecov Report

Merging #11 into master will decrease coverage by 1.36%.
The diff coverage is 75.71%.

@@             Coverage Diff              @@
##             master      #11      +/-   ##
============================================
- Coverage      77.3%   75.93%   -1.37%
- Complexity       73       98      +25
============================================
  Files            25       29       +4
  Lines           326      374      +48
  Branches          7        9       +2
============================================
+ Hits            252      284      +32
- Misses           65       80      +15
- Partials          9       10       +1

Impacted Files
Coverage Δ
Complexity Δ

.../client/addressindex/model/ResponseStatusData.java
33.33% <ø> (ø)
1 <0> (?)

...tactcentresvc/service/impl/AddressServiceImpl.java
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

.../addressindex/AddressServiceClientServiceImpl.java
100% <ø> (ø)
4 <0> (?)

...dressindex/model/AddressIndexSearchResultsDTO.java
60% <ø> (ø)
3 <0> (?)

.../addressindex/model/PostcodeSearchResponseDTO.java
33.33% <ø> (ø)
2 <0> (?)

...ent/addressindex/model/AddressIndexAddressDTO.java
85.71% <ø> (ø)
6 <0> (?)

...ion/contactcentresvc/endpoint/AddressEndpoint.java
88.88% <100%> (ø)
5 <1> (ø)
:arrow_down:

...ient/caseservice/CaseServiceClientServiceImpl.java
100% <100%> (ø)
3 <3> (?)

...contactcentresvc/service/impl/CaseServiceImpl.java
100% <100%> (ø)
5 <5> (+4)
:arrow_up:

.../contactcentresvc/ContactCentreSvcApplication.java
91.66% <100%> (+1.19%)
7 <1> (+1)
:arrow_up:

... and 10 more

Continue to review full report at Codecov.

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


Updating to validate JWT tokens from the emulator

Description
When the emulator is configured (via the channelService property in the .bot file) to use the US Government Azure AAD endpoint, this causes a different set of issuers and a different open id metadata URL to be needed to validate the emulator's JWT token.
Testing
Because this involves testing JWT tokens which I cannot check-in, I validated with:

Emulator configured to use Gov talking connecting to a local .NET core bot configured to use Gov running locally
Emulator configured to use Gov talking connecting to an Azure hosted .NET core bot configured to use Gov running locally
DirectLine talking to the same bot (via US Gov)

You can go here to play with the test bot:
https://helloworldgov.azurewebsites.net/

Pull Request Test Coverage Report for Build 43895

0 of 0   changed or added relevant lines in 0 files are covered.
16 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.06%) to 53.087%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Connector/Authentication/EmulatorValidation.cs
16
64.29%

Totals

Change from base Build 43815:
0.06%

Covered Lines:
4979

Relevant Lines:
9379

💛  - Coveralls


backup object seems to be missing actual kv pairs

Trying to use backup operator but it doesn't seem to be getting any data extracted into the backup object.
Here are our bucket contents:

Currently there are quite a lot of objects in etcd:
/ # etcdctl ls -r /dotmesh.io | wc -l
9629

Backup operator logs:
➜  etcd-operator git:(master) ✗ kubectl logs etcd-backup-operator-5758d6f855-m8n2k -f
time="2018-12-05T13:22:57Z" level=info msg="Go Version: go1.10"
time="2018-12-05T13:22:57Z" level=info msg="Go OS/Arch: linux/amd64"
time="2018-12-05T13:22:57Z" level=info msg="etcd-backup-operator Version: 0.9.3+git"
time="2018-12-05T13:22:57Z" level=info msg="Git SHA: b3b2f83"
time="2018-12-05T13:23:14Z" level=info msg="Event(v1.ObjectReference{Kind:\"Endpoints\", Namespace:\"dotmesh\", Name:\"etcd-backup-operator\", UID:\"3a48523f-e439-11e8-94c8-42010a9a0132\", APIVersion:\"v1\", ResourceVersion:\"48584655\", FieldPath:\"\"}): type: 'Normal' reason: 'LeaderElection' etcd-backup-operator-5758d6f855-m8n2k became leader"
time="2018-12-05T13:23:14Z" level=info msg="starting backup controller" pkg=controller
time="2018-12-05T13:24:09Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"
time="2018-12-05T13:30:10Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"
time="2018-12-05T13:52:08Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"
time="2018-12-05T13:57:59Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster:2379 revision (1)"
time="2018-12-05T14:00:09Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"
time="2018-12-05T14:30:11Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"
time="2018-12-05T15:00:11Z" level=info msg="getMaxRev: endpoint http://dotmesh-etcd-cluster-client:2379 revision (1)"

While the backup object is binary, I can see via cat that it only contains peer and client URLs of the members.

Currently there are quite a lot of objects in etcd:
/ # etcdctl ls -r /dotmesh.io | wc -l
9629

etcdctl ls is against the etcd v2 store backup operator is backing up and restoring v3 store.
What is output of
ETCDCTL_API=3 etcdctl get --prefix=true "" /dotmesh.io | wc -l

hmmm, so this is the situation:
/ # etcdctl ls -r /dotmesh.io | wc -l
9629
/ # ETCDCTL_API=3 etcdctl get --prefix=true "" /dotmesh.io | wc -l
Error:  too many arguments, only accept one argument when `--prefix` or `--from-key` is set.
0
/ # ETCDCTL_API=3 etcdctl get --prefix=true /dotmesh.io | wc -l
0
/ # ETCDCTL_API=3 etcdctl get --prefix=true "/dotmesh.io" | wc -l
0
/ # ETCDCTL_API=3 etcdctl get --prefix=true "" | wc -l
0

I suspect that the project uses v2 API then, will need to investigate :|

Right, I am going to close this as the issue seems to be v2 data which is not supported. If you find this is not the case or have additional information please add to this issue and we can reopen if necessary.


how to make the sv formatter change my code with indent space=4 not the indent space=2

my command is “ ./verible-verilog-format  --assignment_statement_alignment=align --named_parameter_alignment=align --class_member_variables_alignment=align --formal_parameters_indentation=wrap tb.sv”

Hi @ezio1996 , now there is a flag --indentation_spaces which solves your request. So in your case, set it to --indentation_spaces=4

Hi @ezio1996 , now there is a flag --indentation_spaces which solves your request. So in your case, set it to --indentation_spaces=4

thanks so much


Remove deprecated command-line arguments

Inside the main Salomon script there still are deprecated command-line arguments remaining kept for backwards compatibility purposes.
Those arguments are:

--config-file (replaced by --color-file)
--highlight (replaced by --highlight-matches)
--no-follow or -n (replaced by --analyze, --action analyze or -a analyze in short)
--no-header (replaced by --no-info)

Due to the fact, that they are very old it would make sense removing them.

Added a message to the deprecated-argument information that the given argument will be removed somewhen.

Removed in 1.18.2.


Add files via upload

Created responsive footer with social media icons.

Created responsive footer with social media icons.


Updated DataSource and DataType according to new template

Fixes #
Added DataSource
Proposed Changes

Hey Shain,
Yes, WireData does not have a DataSource (a.k.a DataConnector), and it is suppose to be empty
From: shainw<EMAIL_ADDRESS>Sent: Saturday, July 6, 2019 7:21 PM
To: Azure/Azure-Sentinel<EMAIL_ADDRESS>Cc: Yaron Fruchtmann<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/Azure-Sentinel] Updated DataSource and DataType according to new template (#199)
@shainw approved this pull request.
Approved, assuming the DataSource field was expected to be empty in the items I commented on.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2FAzure-Sentinel%2Fpull%2F199%3Femail_source%3Dnotifications%26email_token%3DALX3FJ5XEBGCJKPJ3FMTNLLP6DA5VA5CNFSM4H4YZTT2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOB5U5OUY%23pullrequestreview-258594643&data=02|01|Yaron.Fruchtmann%40microsoft.com|2895caeccb394037367e08d7022de4be|72f988bf86f141af91ab2d7cd011db47|1|0|636980268454327747&sdata=Qlj9doYzjrOrelLMDotS%2FYznwigHvCL55ie7rHpLduA%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALX3FJ7OHZ6S4YPJIREJVFTP6DA5VANCNFSM4H4YZTTQ&data=02|01|Yaron.Fruchtmann%40microsoft.com|2895caeccb394037367e08d7022de4be|72f988bf86f141af91ab2d7cd011db47|1|0|636980268454327747&sdata=iChV26vk22%2FDTo7n98K6mr7h7yxnT4phhAKF2UEWBhs%3D&reserved=0.


Fix socket and memory leak on shutdown

When the client is shutting down, its likely that the async_connection_process_loop is polling the stream(socket) and input channel futures, and never wakes so that the stop/shutdown channel can be checked.
As a result the async_connection_process_loop task never exits and while not harmful to subsequent connections, the memory associated with the future is never released, and, since the task owns the stream, the socket itself is leaked as well.
The fix provided by this patch is straightforward, simply poll the stop channel future as well.
An even better fix than the one provided here would be to save the task handle future returned by spawn, poll it somewhere (in case it fails), and join the main task to it on shutdown, after the stop channel has been written to.

Thanks for the contribution. I'd have another release after the clippy fix.
BRs


Extracts only the relevant pieces of the version string

Fixes issue where versions like "rethinkdb 1.16.2+1~0trusty" were not parsed correctly by the previous logic.

:+1:


Renamed Bigchain.federation_nodes as Bigchain.nodes_except_me

Someone new to the BigchainDB code might look at the list named Bigchain.federation_nodes and think that it includes all the federation nodes, but it doesn't. (It doesn't include the public key of the current node; that's stored in Bigchain.me.)
I figured it would be a good idea to rename it so there's less ambiguity. I changed it to Bigchain.nodes_except_me. That name makes a good pair with Bigchain.me.
An alternative would be Bigchain.keyring but that name is also somewhat ambiguous. I suppose BigchainDB developers should know what "keyring" means (i.e. that it excludes the public key of the current node). I prefer to assume as little as possible.

[Current coverage][cc-pull] is 88.36%

Merging [#390][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master       #390   diff @@
==========================================
  Files            18         18
  Lines          1246       1246
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           1101       1101
  Misses          145        145
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [ac603c2...cd70707][cc-compare]
[cc-base-branch]: https://codecov.io/gh/bigchaindb/bigchaindb/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/bigchaindb/bigchaindb/compare/ac603c2744a98fd3591cdaac1473887bbe07ce2a...cd7070722f7803157e466c96cfc72d656a0a3d3b
[cc-pull]: https://codecov.io/gh/bigchaindb/bigchaindb/pull/390?src=pr

:+1:


[FLINK-15114][SQL-CLIENT]Add execute result info for alter/create/drop database in sql-client

What is the purpose of the change
Add execute result info for alter/create/drop database in sql client
Brief change log

Add execute result info for alter/create/drop database in sql client

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@xuefuz @bowenli86 @KurtYoung @danny0405 to have a review.


Error Pushing to Origin

At the end of Homework Part 1, when being asked to push the changes I made, an error message is popping up. It is saying git credentials are not found. I have the repo open and ready, what am I missing?

This most likely means you have somehow signed out.
Can you open preferences and make sure you are still signed into GitHub. You may need to try “Signing Out” Then “Signing in” again.

 I even
singed in and out of my Moodle and adjusted setting in the repository.
On Sep 28, 2017 2:10 PM, "Michael Musick"<EMAIL_ADDRESS>wrote:

This most likely means you have somehow signed out.
Can you open preferences and make sure you are still signed into GitHub.
You may need to try “Signing Out” Then “Signing in” again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Montana-Media-Arts/120_CreativeCoding_Fall2017/issues/34#issuecomment-332950057,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeMhbhVj6O0EOSQGZ--yMKrZ1xEf4dcLks5sm_0ygaJpZM4Pn0Z2
.

First, what we are doing has absolutely no connection to Moodle. This is entirely separate from technologies provided by the University itself.
Second, bummer… Could you upload a screenshot?

@trason-provence  I have not got a clue but I did find this on Github. You may find your solution there.
Github error status 1 Apparently someone else had the same issue looks like a bugger to work around it though. Hope it helps. Probably @michaelmusick  may have to help you filter through this workaround for a solution.

Thank you @JonSwallow - I will check it out and see if I can make it work. If not, is there any chance you might be around your office @michaelmusick?

Hey@ trason-provence, I am around until about 5:30 today. If you had some time to come by, we could definitely look at it.
If you do come by, knock loudly, as I will be in my office recording videos.

Also, @trason-provence. did you try the suggestion in the post?
Go up to the Repository menu and select Open in Command Prompt. It'll launch Command Prompt in the repository. Then run copy and paste del /f .git\hooks\* into Command Prompt and hit Return.

Be there soon.
On Sep 28, 2017 4:07 PM, "Michael Musick"<EMAIL_ADDRESS>wrote:

Also, @trason-provence. did you try the suggestion in the post?
Go up to the Repository menu and select Open in Command Prompt. It'll
launch Command Prompt in the repository. Then run copy and paste del /f .git\hooks\* into Command Prompt and hit Return.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Montana-Media-Arts/120_CreativeCoding_Fall2017/issues/34#issuecomment-332977489,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AeMhbkqus-4DKbPkVUqjOQhgXXlqhfL9ks5snBiygaJpZM4Pn0Z2
.

Huge thank you to @JonSwallow! Those steps with @michaelmusick's help made it all pan out!

@trason-provence  Glad I could help. Now I know it works I am fixin to go bookmark it in case that ever happens to me!! Cheers.


added timer for token expiration

While loop for token checking is replaced with a setInterval object called once a second, which increments a timer variable and exits when a minute is exceeded.

Superseded by #8


Lightclient wallet for ETH + ERC20 balances

Motivation
Lodestar has so far developed a consensus lightclient that tracks the head with minimal resources. This is an important piece towards fully functional lightclients but doesn't have practical use for users. With the upcoming Amsterdam devconnect lightclient event https://lightclients.xyz/ it's very important to show the community the technical capabilities of lightclients today to rally support in this direction.
Browser based lightclients that support the full JSON RPC API require EVM execution with special SLOAD opcode that fetches storage dynamically. This is not an easy task, and it's being actively research by Portal Network participants https://github.com/ethereum/portal-network-specs. That's the long term goal, but meanwhile there are some valid use-cases that don't require EVM execution.
A read-only wallet that shows an account's ETH balance and ERC20 balances of well-known contracts doesn't require EVM execution since it can directly fetch storage slots at deterministic locations. This is what the main goal of this issue, and deploy it in Kintsugi / Klint testnets and then add p2p support for head tracking (upcoming).
Todo
In a web-app, or stand-alone library initially for easier testing:

[ ] Import lodestar's lightclient and sync to the head. Existing implementation in https://github.com/ChainSafe/eth2-light-client-demo
[ ] Request a consensus state proof to latest_execution_payload_header.state_root, with Lodestar proof API
[ ] Pre-compute execution state address of an account balance
[ ] Pre-compute execution state address of an account ERC20 balance and pre-selected contracts by the users
[ ] Request a execution state proof with EIP-1186 eth_getProof. Supported by Geth and EthereumJS. Validate proof against latest_execution_payload_header.state_root

Let's go! I can jump in latter and think of some design to show the info

Is there anything outstanding in here you want to continue tracking @dapplion ? Otherwise, we can create new issues and add to the #3469 light client tracker.

@g11tech did a fantastic job here so we can close it. If we have more ideas to extend it further let's do new issues


General-purpose buttons

Currently, Chakra Templates has no general purpose buttons. Buttons need to be extracted from other templates and components.
This PR contains code towards three (3) general purpose buttons -

Color Mode Toggle
Secondary Options Menu
Button with Tabbed Popover

I have also renamed Social Button subcategory from buttons to socialButtons and made some linting fixes.

@all-contributors please add @danish17 for code


'runtime' is an invalid keyword argument

Hi, when I use jnitrace, it told me below errors
frida version:12.2.6
jnitrace version:3.0.7
android version: 4.4.2(kitkat) arm emulator
jnitrace -l xx.so com.xxx
Traceback (most recent call last):
File "/usr/local/bin/jnitrace", line 8, in
sys.exit(main())
File "/usr/local/lib/python3.6/site-packages/jnitrace/jnitrace.py", line 552, in main
script = session.create_script(jscode, runtime="v8")
File "/usr/local/lib/python3.6/site-packages/frida/core.py", line 148, in create_script
return Script(self._impl.create_script(*args, **kwargs))
TypeError: 'runtime' is an invalid keyword argument for this function

after some dig, i know this may not jnitrace problem but frida core. But for some reasons I can't use higher frida version, so can we do some remediation in jnitrace as its depence meet?

ooooh. doesn't support 12.2.6? frida need >12..

Hi,
Thanks for letting me know about this. I have pushed a new version that requires Frida 12.5.0 to prevent the error you were getting.
Does Frida 12.5.0 not work for you? What kind of issue are you seeing?
If it does not work I can look at putting a patch in place to support older versions of Frida.
Thanks,

@jiych Frida 12.8.0 should work fine on that emulator. If not, could you tell me if Frida master works for you? I'm about to push a new release and would really like to fix regressions such as this.

@oleavr Frida 12.8.20 didn't work for me, I've been opened a new issue here:
https://github.com/frida/frida/issues/1299
I will test Frida master if work on my emulator later.

Closing as issue filed with the Frida project

Closing as issue filed with the Frida project


Fix is_within_weak_subjectivity_period() assertion

Currently, is_within_weak_subjectivity_period() has the following assertion:
assert ws_state.latest_block_header.state_root == ws_checkpoint.root

However, ws_checkpoint.root is a block root, so when asserting that provided ws_state has the matching root, the assertion should be changed to:
assert hash_tree_root(ws_state.latest_block_header) == ws_checkpoint.root

We could do:
assert ws_state.latest_block_header.state_root == store.block_states[ws_checkpoint.root]

If the block with root ws_checkpoint.root has been processed by the client, then store.block_states[ws_checkpoint.root] would have been populated already.
If the client is bootstrapping itself from a trusted ws_state and ws_checkpoint (not the current sync design, but planned in future upgrades), then we can require that store.block_states[ws_checkpoint.root] be populated with ws_state, so that it doesn't break this function or other fork choice logic.


[BUG] Dispatch from editor at runtime

The dispatch button does not trigger the event from code listener. Do I miss something? Or is it a bug?

Seems to be broken yeah. I'll look into it ASAP - thanks for reporting!

Fixed!


v-chgri-commerce-ts-mig_1

Pull request guidance
Thank you for your contribution. The content team is monitoring the pull requests. Once your pull request is ready for review, please type "#sign-off" in a new comment or add a comment such as "PR is ready for review and merge."

Hi @v-chgri, I made minor changes to the draft and added the article to the TOC. The welcome page is also updated to include the article. Can you have a final review?

Hi @v-chgri, this article is public, and the link is https://learn.microsoft.com/en-us/troubleshoot/dynamics-365/commerce/customer-accounts-loyalty/email-not-sent-when-new-customer-created. Feel free to let me know if any questions. As this is the first D365 Commerce article migrated from PG, so we also requested Jo Sawyer, Brian Storie, and Aarti Goyle to have a review in another email thread. For the further migrated articles, I’ll also left a comment for you to have a final review before publishing. You can review the changes and add a “#sign-off” in the PR to give us the greenlight to merge it. Thanks!


Unix sockets support

I want to suggest another path, which is unix sockets.
For example, we are using many docker containers on one host, and I'd like to be able to get the logs on real time from the docker.sock.
Thanks
D.

@panda87  I believe getting logs in real time over unix socket is beyond the scope of what tail should be responsible for, having said that you should be able to tail your log files on your container(s) using this library depending on how/what file system you are using for your containers. If you are still considering to stream your logs over some sort of socket or using docker stream api, you should still be able to do this with a small utility.


Repo sync for protected CLA branch

The pull request is created from master637653349156445024 to master to fix git push error for protected CLA branch

@huypub : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Resolve function arguments using mixin content scope

This should resolve (or help resolve) #449, #446.

Hi,
should this be used in production?
It fixed bootstrap v4.0.0@3 compilation for me but can't figure out if there is more to do?

Hi @axi, we, @moodleHQ, will be using it in production soon.

@FMCorz thanks

Works flawless :)


Dataflow partitioning

Currently, data is not deliberately partitioned in the dataflow. Partitioning based on a unique identifier (systemid + company) can reduce data shuffling between worker nodes and reduce execution time.

This sounds promising, may I ask a question about this?

"..If you plan on using non-equality comparisons in your custom expression, you should utilize the 'Fixed' broadcast setting and specify a minimum of 1 stream to be broadcast. If broadcasting, ensure that your Integration Runtime is sized appropriately.."
Is this a warning we should consider? And if so, what would be the best setting for the Broadcast options?

@Arthurvdv The custom expression in the "Remove Deleted" step does not involve any non-quality comparisons, so I have not paid any mind to this warning.

@HenriSchulte-MS, thank you for sharing. I'll update our pipeline ahead of the merge of this PR.


Access is Denied

I did all the Prerequisite and requirements, But when run the program
get this error: ...\ALL_BUILD access is denied.
Do you know how can I solve this problem !?

This is possibly due to your folder permissions or IDE configuration.

The "3D-Eye-Tracker-master" folder is read-only, and there is no way to remove this read-only access.
Is there any solution for that?

If you're on a Windows machine with Administrator privileges, you should be able to change this. Also make sure that the build folder is not open in some other application or another instance of your IDE. In any event, this is likely a problem with your OS/setup rather than the code.


Impossible to use the virtual column generated by the timestamp index ($ts$DAY) in the startree index

When I try to create my realtime table with a timestamp and a startree index, I get this error :
Invalid table config: my-table-name_REALTIME. Column Name $ts$DAY defined in StarTreeIndex Config must be a valid column defined in the schema
Currently the table config validation requires all the star-tree index to be explicitly configured within the schema.

@Jackie-Jiang I would like to work on this but I will need some guidance for this.
Should we even allow creating indexes on top of generated columns?

We have plans to re-design the timestamp index because currently it is more or less just the syntactic sugar for derived column + expression override. In regular case, derived column should be defined within the schema so that it can be used within the star-tree, but it is not the case for timestamp index as of now.
Add @gortiz to share more ideas


fix running_difference return type is not right

Proposed changes
Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[ ] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[ ] No
[ ] No Need

Has document been added or modified:

[ ] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[ ] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[ ] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

this PR do not add regression test because running_difference's result is not stable

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 35.8 seconds
load time: 508 seconds
storage size:<PHONE_NUMBER>0 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230117184625_clickbench_pr_82546.html


Strictness properties missing in documentation

The haddock of Data.HashMap.Strict has a heading Strictness properties but no properties.
They seem to have been deleted here: https://github.com/tibbe/unordered-containers/pull/186/files#diff-1b71e586508ac38a4157ee51fdc2def7L115

Whoops! Thanks for the heads up.
On Wed, Jul 24, 2019, 7:09 AM Leon Schoorl<EMAIL_ADDRESS>wrote:

The haddock of Data.HashMap.Strict has heading Strictness properties
https://hackage.haskell.org/package/unordered-containers-<IP_ADDRESS>/docs/Data-HashMap-Strict.html#g:1
but no properties.
They seem to have been deleted here:
https://github.com/tibbe/unordered-containers/pull/186/files#diff-1b71e586508ac38a4157ee51fdc2def7L115
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tibbe/unordered-containers/issues/238?email_source=notifications&email_token=AAOOF7JX2B7M57CPMGJDEPTQBAZ4ZA5CNFSM4IGO4GM2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HBFRFPQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOOF7P424NIZZVT6MAIYEDQBAZ4ZANCNFSM4IGO4GMQ
.

Fixed in #255, I believe? Otherwise, please reopen.


Visual Studio Team Foundation Server / Azure DevOps Repos

We need to be able to support integration with Visual Studio Team Foundation Server and Azure DevOps Repos.

Created a public workspace with the OpenAPI and a collection for Azure DevOps - https://api-evangelist.postman.co/workspace/Azure-DevOps~261535e3-3f7c-4373-b04d-96704545a682/overview -- I do not see anything for Team Foundation, but will keep looking for something there.

Azure DevOps Postman Collections - https://github.com/hkamel/azuredevops-postman-collections

Integrate automated test in Azure DevOps using the Postman API - https://medium.com/younited-tech-blog/integrate-automated-test-in-azure-devops-using-the-postman-api-288f5566bf11

Three Ways to Use Postman and Azure DevOps - https://apievangelist.com/2020/01/22/three-ways-to-use-postman-and-azure-devops/

Integration Test with Postman in Azure DevOps Pipeline: Part 1 - https://dev.to/kenakamu/integration-test-with-postman-in-azure-devops-pipeline-part-1-2h5j
Integration Test with Postman in Azure DevOps Pipeline: Part 2 - https://dev.to/kenakamu/integration-test-with-postman-in-azure-devops-pipeline-part-2-nfk

Feature Request: TFS - Sync your Postman Collections #3796- https://github.com/postmanlabs/postman-app-support/issues/3796

Added Action - https://apis.how/actions/integrate-with-azure-devops/


hnswlib.v0.8.0: checkIntegrity fail on tests/cpp/sift_1b.cpp

My branch is at v0.8.0. When building hnsw index in tests/cpp/sift_1b.cpp, I add appr_alg->checkIntegrity(); on line 346 to see if the graph built by the algorithm has in-edge for every node during report_every. However, the assertion fail!
the code fragment I modified is as follows
                if (j1 % report_every == 0) {
                    cout << j1 / (0.01 * vecsize) << " %, "
                         << report_every / (1000.0 * 1e-6 * stopw.getElapsedTimeMicro()) << " kips " << " Mem: "
                         << getCurrentRSS() / 1000000 << " Mb \n";
                    stopw.reset();
                    appr_alg->checkIntegrity(); // add this at line 346
                }

the output of command line like this
~/code/hnswlib/tests/cpp/downloads$ ../../../build/main
Loading GT:
Loading queries:
Building index:
0.05 %, 40.9574 kips  Mem: 40877 Mb
main: ~/code/hnswlib/tests/cpp/../../hnswlib/hnswalg.h:1403: void hnswlib::HierarchicalNSW<dist_t>::checkIntegrity() [with dist_t = int]: Assertion `inbound_connections_num[i] > 0' failed.
Aborted (core dumped)

If some node has no in-edge, then it's totally isolated, which means that search on the graph will never visisted it, even it has out-edge.

Yes, I've encountered the same situation, and it does look as if the hnsw algorithm can't avoid nodes with in-degree 0, even if all points have out-degree.

i think it may be because when the new insertion point creates an edge with its immediate neighbors, the cropping of the neighbor points happens to crop out the new insertion point. Do you know the cause or any other findings?


Coerce item data type

Resolves #1905
Description
Request items were being saved as they come. This could happen through the system itself, or through API calls. This PR adds data coercion before saving to request items' item_id and quantity
Type of change

Bug fix

How Has This Been Tested?
This PR fixes for all input types, the specific bug request that raised this concern is https://github.com/rubyforgood/diaper/issues/1905, and we're still waiting for the user to describe how to reproduce this issue.
Screenshots
See https://github.com/rubyforgood/diaper/issues/1905

This looks good, just a couple rubocop violations.


Rework pinned unit section

Rename to "summary"
Increase width, reducing chat and board width to accommodate
Increase height so it takes 2/3 rather than 1/2 of vertical space

For increasing width, keep the dimensions of the discussion as it is currently. But split it down center so chat is on left, pinned/focused is on right. It would be good if units are the same width in chat or pinned (summary) so they both have importance/attention.

We will also need to consider how big the board should be, especially with the 2D situation.

Eventually make sections adjustable.


fixes for #381 (const functions)

Trying to fix #381. Digging through some Rust source I found out that there can be functions with const keyword also. Racer thinks that this is a constant instead of a function and this messes up the parsing. Maybe this is a hack, but you can reject this if it is :)

Hi @jaxx. This looks great, sorry I didn't get to it earlier. I'll have a quick check through and then merge

Hi @jaxx
Thanks again for the patch. It did occur to me that any function that contains an inner const function won't complete after this patch e.g.
#![feature(const_fn)]
fn blah() {
    const fn foo() {
    }
}

fn main() {
    b     <---- doesn't complete to blah
}

But that's not as big a deal as #381 so have merged. Thanks again!


refactor(*): rustc 1.0.0-beta.4

This allows mopa to work with rust 1.0.0-beta. I've tried to maintain the ability to use #[no_std] but I have not tested this against nightly.
Breaking Change
Users must now use mopa::Any rather than std::any::Any or core::any::Any.

I don't know how to feature gate the no_std examples. For now they are commented out. Looks like this issue could resolve it: rust-lang/cargo#1570

+1 as it would permit  rustless framework to compile with rust beta

I took the basic concepts here and adjusted them as suited me best and made 24f6c2df065ad72f9ad3579551f360d9395be8fa of it; mopa 0.2.0 is now published with this change.
Thanks!

Cool, I think I've learned a little more on exactly what a trait object is and how to interact with them. Cheers!


Confused how to get swagger to call endpoint correctly

export const campaign = router({
  findOne: publicProcedure
    .meta({
      openapi: {
        method: 'GET',
        path: '/campaign/findOne',
        description: 'Will return a individual admanager campaign by id'
      }
    })
    .input(z.object({ id: z.string() }))
    .output(Campaign)
    .query(({ input }) => {
      console.log('input', input);
      return {
        id: '0a1ee26ae7c739472a9ca53810a661d1',
        name: `campaign name: ${input.id}`
      };
    }),

});

The above works fine with the client by using:  trpc.campaign.findOne.useQuery({ id: 'xyz12374574' });
However, I've tried changing the path to /campaign.findOne & /campaign.findOne/{id} & /campaign.findOne?id={id} but no success, seems to not format the url correctly to resolve the response, any idea why?
[\n  {\n    \"code\": \"invalid_type\",\n    \"expected\": \"object\",\n    \"received\": \"undefined\",\n    \"path\": [],\n    \"message\": \"Required\"\n  }\n]

This is the error I'm getting back in swagger

This wasn't an issue, I setup incorrectly

And for people who finds this issue in the future, the solution of such problem is intercept the original request and re-build the request that meets TRPC's requirements.
requestInterceptor={(req) => {
  // Check if there are any query parameters
  if (req.url.includes('?')) {
    const [urlBase, queryParams] = req.url.split('?');

    // Convert key/value parameters to JSON
    const params = new URLSearchParams(queryParams);
    const jsonParams: any = {};
    for (const [key, value] of params.entries()) {
      jsonParams[key] = value;
    }

    // Re-construct the URL with the JSON-encoded parameters
    req.url = `${urlBase}?input=${encodeURIComponent(JSON.stringify(jsonParams))}`;
  }

  return req;
}}

Final pattern reference: https://trpc.io/docs/rpc


How pass data

How pass data, for example: global.json and then use in tpl

Hi @webmarkelov!
There is a loader param named context. You can use it to pass data into a template.
Inside the webpack.config.js
First create the nunjucks querystring:
const sourceDir = path.join(__dirname, '../src/');

const nunjucksConfig = JSON.stringify({
  'searchPaths': [sourceDir],
  'context': require(sourceDir + 'data/index.js')
});

Then use the querystring in the loader as so:
{
    include: /\.(njk|nunjucks)$/,
    // Here we are using the nunjucks config variable created before
    use: ['html-loader', 'nunjucks-html-loader?' + nunjucksConfig]
}

Finally create the index.js file inside the data folder located at sourceDir, with the content as follow:
const site = require('./site.json');

module.exports = {
  site
};

[]'s

Thanks @ginaldoterencio for the explanation.
@webmarkelov I'll close the issue for now. Feel free to reopen it if the directions @ginaldoterencio gave you do not work for your case.


Next in a series of PR's: supporting dynamics in JsonPatch.

This PR contains one small fix on the Add operation (.GetType() must be called on the property value on the existing object, not on the value passed in) & the new Remove method, the last "big one" (plus additional unit tests for dynamic add/remove & unit test fixes in relation to the new error messages from the previous PR/Ryan's improvements).
The new Remove method returns the type that's being removed - that'll be used in the next PR. Having it return the type allows for faster execution of operations that use the remove op (eg: Replace).  In the next PR, the remaining operations (+ cleanup) will be implemented.  They typically reuse parts of Remove/Add, or follow the same logic; so, once Add and Remove are checked, the rest should be a breeze ;-)

:watch: looks pretty good overall. Happy to see so many test adds.

:shipit:

054b46c1cce2bc90eea177a0fecfcc19448ad2b8


Suggestion- On product page there should be a product video section

Suggestion- there should be a product video section so that we can upload the video of that particular product in bagisto like Amazon and eBay.

I suggest youtube/vimeo video as an alternative solution

You can add youtube video in description


angular 1.4 support

it seems like the current state of this project would work in angular 1.4. Would it be possible to update the bower dependency to allow angular 1.4?
Thanks

Added angular 1.4 support in angular-poller v0.3.3. :)

thanks!


(Sentinel) Remove too many submissions check

Description
Since we have a client submitting themselves, this is no more accurate alert
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Deployment

[ ] Should publish npm package
[ ] Should publish Docker image

lgtm!
i wonder if checking our own submission in this logic would take too much time to implement 🤔

I assume it shouldn't be too difficult, let me make an update then

lgtm!
i wonder if checking our own submission in this logic would take too much time to implement 🤔

I assume it shouldn't be too difficult, let me make an update then

looked into the event table, and found out there aren't any msg.sender column, which means it is impossible to parse based on tx sender wallet address. For now, will just remove this check


Issue with the rendering of this Wiki

Hello,
When I go to this page : https://github.com/electro-smith/DaisyWiki/wiki things don't seem to be rendered properly. Thanks for your help!
Romain

This should be fixed!

Yes it is, thanks!