Add mediaQuery helper method to manage media queries

This fixes https://github.com/styled-components/styled-components/issues/791 There's a much longer explanation there.
While my original code was just this:
const mediaQuery = (...query) => (...rules) => css`
  @media ${css(...query)} {
    ${css(...rules)}
  }
`

I had to complicate it a bit to add Flow typing on the params and return types.

Copying over my comment from the issue:

Just a note: Please beware that css(...args) and similar usages of it will break preprocessing. I'm looking out for this kind of code snippets and in the future we won't recommend anything that will break preprocessing in the future 😄

Edit: Btw as a closing note. This is a CSS helper and those do belong into polished or a separate pkg, since the API doesn't contain any helpers like these and shouldn't be bloaty in the sense of delivering anything that is not immediately needed.


Use symbolic shape in tensorflow version of batch_flatten

Yeah, seems reasonable.

Hi,
When I created a non sequential model using the Flatten layer with Keras with tensorflow backend, I got an error whereas the old version seems to work (see below).

pool2_bis
<tf.Tensor 'Reshape_894:0' shape=(?, 2048, 1) dtype=float32>
[-1, prod(shape(pool2_bis)[1:])]
[-1, <tf.Tensor 'Prod_5:0' shape=() dtype=int32>]
[-1, np.prod(pool2_bis.get_shape()[1:].as_list())]
[-1, 2048]

This thread reports the same error https://github.com/fchollet/keras/issues/3450

Hi. The original problem that I was trying to fix was with this code, when using batch_flatten in loss function.
import os
os.environ["KERAS_BACKEND"] = "tensorflow"

from keras import backend as K
from keras.models import Model
from keras.layers import Convolution2D, Input

def my_loss(y_true, y_pred):
    y_true_f = K.batch_flatten(y_true)
    y_pred_f = K.batch_flatten(y_pred)
    return K.mean(y_true_f * y_pred_f)

K.set_image_dim_ordering('tf')
image_shape = (None, 128, 128, 1)
images = Input(shape=image_shape[1:])

output = Convolution2D(1, 3, 3)(images)

model = Model(input=images, output=output)
model.compile(optimizer='sgd', loss=my_loss)

I got the following error:
<ipython-input-1-b426c31b42c5> in my_loss(y_true, y_pred)
     11
     12 def my_loss(y_true, y_pred):
---> 13     y_true_f = K.batch_flatten(y_true)
     14     y_pred_f = K.batch_flatten(y_pred)
     15     return K.mean(y_true_f * y_pred_f)

/Users/stas/.pyenv/versions/anaconda-2.4.0/lib/python2.7/site-packages/keras/backend/tensorflow_backend.pyc in batch_flatten(x)
    776     '''
--> 777     x = tf.reshape(x, [-1, np.prod(x.get_shape()[1:].as_list())])
    778     return x
    779

/Users/stas/.pyenv/versions/anaconda-2.4.0/lib/python2.7/site-packages/numpy/core/fromnumeric.pyc in prod(a, axis, dtype, out, keepdims)
   2490         except AttributeError:
   2491             return _methods._prod(a, axis=axis, dtype=dtype,
-> 2492                                   out=out, keepdims=keepdims)
   2493         return prod(axis=axis, dtype=dtype, out=out)
   2494     else:

/Users/stas/.pyenv/versions/anaconda-2.4.0/lib/python2.7/site-packages/numpy/core/_methods.pyc in _prod(a, axis, dtype, out, keepdims)
     33
     34 def _prod(a, axis=None, dtype=None, out=None, keepdims=False):
---> 35     return umr_prod(a, axis, dtype, out, keepdims)
     36
     37 def _any(a, axis=None, dtype=None, out=None, keepdims=False):

TypeError: unsupported operand type(s) for *: 'NoneType' and 'NoneType'

It seems, that during compilation loss function doesn't know what are the dimensions of its parameters. But after changes in this PR it successfully compiled.
Also I looked at Theano version of batch_flatten that uses dynamic shape (and I didn't have problems while compiling using Theano):
...
x = T.reshape(x, (x.shape[0], T.prod(x.shape) // x.shape[0]))
...

So, I thought it would be reasonable to use dynamic shapes for TF as well.
BTW, there is a good explanation of differences between static and dynamic shape inference in TF.
The TF issue can be seen here:
import numpy as np
import tensorflow as tf
shape = (None, 128, 128, 1)
x = tf.placeholder(tf.float32, shape=image_shape)

tf.reshape(x, [-1, np.prod(x.get_shape()[1:].as_list())]).get_shape()
# returns TensorShape([Dimension(None), Dimension(16384)])

tf.reshape(x, [-1, K.prod(K.shape(x))]).get_shape()
# returns TensorShape([Dimension(None), Dimension(None)])
...

As seen, in the second case TF can't infer the shape.
Actually, now I'm not sure what would be the correct solution to the issue. And whether it is a TF issue or not (because in Theano same shape arithmetics works OK)

This PR broke https://github.com/tensorflow/tensorflow/issues/13568

Maybe PR https://github.com/tensorflow/tensorflow/pull/13561 fixes it. Maybe the problem is/was triggered by -1 in reshape
https://github.com/tensorflow/tensorflow/blob/r1.3/tensorflow/contrib/keras/python/keras/backend.py#L2091


fix wallet schema + version

I was unable to import a JSON wallet exported by neo-cli via Wallet.from_json(). This fixes the version and schema errors
Schema errors

name can be None
path is not required per NEP6 standard
contract validation scheme missing
is_default must match the C# naming convention isDefault
account label can be None

wallet version
Apparently the wallet version changed from 3.0 to 1.0. This was missed because NEO thought it was nice to changed CODE in a PR labelled "Add XML comments" 🤦

Codecov Report

Merging #100 (65e75d6) into feature-wallet (55e078e) will not change coverage.
The diff coverage is 50.00%.

@@               Coverage Diff               @@
##           feature-wallet     #100   +/-   ##
===============================================
  Coverage           84.79%   84.79%
===============================================
  Files                  86       86
  Lines                7536     7536
===============================================
  Hits                 6390     6390
  Misses               1146     1146

Impacted Files
Coverage Δ

neo3/wallet/wallet.py
86.56% <50.00%> (ø)

Continue to review full report at Codecov.

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

Same changes have been merged into the feature-wallet branch via #83 + directs commits on the branch.


Need to catchup doc for IO Connectors.

Describe the bug
some of the connectors are not documented in this link:
http://pulsar.apache.org/docs/en/io-connectors.html#docsNav
such as：
canal
elastic
file
hdfs2/3
mongo
we need catch up the work.

Document of canal has been updated here https://github.com/apache/pulsar/pull/3664.

Thanks @tuteng  for point it out


allow to force scripts invalidation

This PR is aimed to force scripts invalidation via --force|-f command option.

If set to TRUE, the script will be invalidated regardless of whether invalidation is necessary.
http://php.net/manual/en/function.opcache-invalidate.php

OpcacheProxy already has transparent support for this flag, but it was not passed from the command.

removed unused import

Thanks for your contribution, its part of release 3.1.0.


[Metal] Vertex shader buffer bindings need to be incremented?

In Metal, vertex buffers are passed to vertex functions via [MTLRenderEncoder::setVertexBuffer](https://developer.apple.com/documentation/metal/mtlrendercommandencoder/1515829-setvertexbuffer?language=objc).
That same function is used to pass uniform data to a vertex function. Both the vertex buffers and uniform buffers are addressed using the same index. This creates the following problem. Consider the following input:
struct MeshVertIn
{
    float3 pos      : ATTRIB0;
    float3 norm     : ATTRIB1;
    float4 tangent  : ATTRIB2;
    float2 uv       : ATTRIB3;
    float2 uv2      : ATTRIB4;
};

struct MeshVertOut
{
    float4 pos : SV_POSITION;
    float2 uv;
    float2 uv2;
    half3 norm;
    half4 tang;
};

cbuffer Params : register( b0 )
{
    row_major float4x4 objViewProj;
    float meshScale;
};

MeshVertOut SimpleMesh_VS( in MeshVertIn v )
{
    MeshVertOut o;
    o.pos = float4( v.pos * meshScale, 1.0f ) * objViewProj;
    o.norm = half3( v.norm );  //* u.obj33;

    o.tang.xyz = half3( v.tangent.xyz );  //* uniforms.obj33;
    o.tang.w = 1.0f;

    o.uv = v.uv;  // * u.uvMod.xy + u.uvMod.zw;
    o.uv2 = v.uv2;

    return o;
}

half4 MeshNormals_PS( MeshVertOut v ) : SV_TARGET
{
    return half4( 0.5f + v.norm * 0.5f, 1.0f );
}

//T: MeshVisNormals vs:SimpleMesh_VS ps:MeshNormals_PS

Note how the cbuffer uses register b0.
This SPIR-V generated by this input produces the following Metal code:
#include <metal_stdlib>
#include <simd/simd.h>

using namespace metal;

struct MeshUniforms
{
    float4x4 objViewProj;
    float meshScale;
};

constant float4 _204 = {};

struct SimpleMesh_VS_out
{
    float2 _entryPointOutput_uv [[user(locn0)]];
    float2 _entryPointOutput_uv2 [[user(locn1)]];
    float3 _entryPointOutput_norm [[user(locn2)]];
    float4 _entryPointOutput_tang [[user(locn3)]];
    float4 gl_Position [[position]];
};

struct SimpleMesh_VS_in
{
    float3 v_pos [[attribute(0)]];
    float3 v_norm [[attribute(1)]];
    float4 v_tangent [[attribute(2)]];
    float2 v_uv [[attribute(3)]];
    float2 v_uv2 [[attribute(4)]];
};

vertex SimpleMesh_VS_out SimpleMesh_VS(SimpleMesh_VS_in in [[stage_in]], constant MeshUniforms& _27 [[buffer(0)]])
{
    SimpleMesh_VS_out out = {};
    float4 _203 = float4(in.v_tangent.xyz.x, in.v_tangent.xyz.y, in.v_tangent.xyz.z, _204.w);
    _203.w = 1.0;
    out.gl_Position = float4(in.v_pos * _27.meshScale, 1.0) * _27.objViewProj;
    out._entryPointOutput_uv = in.v_uv;
    out._entryPointOutput_uv2 = in.v_uv2;
    out._entryPointOutput_norm = in.v_norm;
    out._entryPointOutput_tang = _203;
    return out;
}

Note that the second parameter of the vertex function received a [[buffer(0)]] annotation, which introduces a conflict: the index 0 refers to the vertex buffer that the shader uses (either that, or I am misunderstanding how passing vertex buffers to vertex functions works on Metal).
The way I have addressed this problem on my side is to go through all uniform buffers used by the vertex stage and add 1 to their binding numbers if the vertex function consumes a vertex buffer. However, I feel like SPIRV-Cross should be able to just do it for me. I'm willing to work on a PR for this, however first I would like to understand if there is anything I'm missing here.

Buffer 0 is not necessarily the vertex buffer. When using [[stage_in]] and a struct with [[attribute(n)]] fields, the mapping from buffers to vertex data is governed by the vertex descriptor of the Metal pipeline. That's set using the vertexDescriptor field of the MTLRenderPipelineDescriptor. It can map from arbitrary buffers to the fields of that struct. So, the same shader code can be used with different buffer configurations and, likewise, a given buffer configuration can be used with different shaders and shader-language vertex data structures. One just has to make sure the field attributes track where you want the data to end up.

Thanks for the clarification, I completely forgot that Metal allows to assign bindings to vertex buffers via attribute descriptors. This makes things much easier; for example one could allocate vertex buffer bindings starting from 30, then 29, 28, etc., to avoid conflicting with "uniform buffer" bindings.

I've tried the above approach and it seems to work, so I'm closing the issue.

Why not set to buffer 0? Now i want to use the tool to translate to MSL in runtime, and i don't know what's suitable buffer index to set to vertex buffer.

i assume there's some abstraction layer between your code and the platform-specific graphics api, you can always do binding translation there (i.e. map "virtual" vertex buffer binding 0 to metal vertex buffer binding 30)

Yep, there is an abstraction layer. But how to know should bind to 30? May be 30 is used in MTL shader. My problem is how to bind to a safe index.

@nicebyte Just for clarification, did you use the VertexAttributeDescriptorArray_bufferIndex starting from a value greater than one? Because this is what I'm doing and newRenderPipelineState is segfaulting for some reason. I'm wondering if this is a Metal bug or expected behavior.

You can see what i did in https://github.com/nicebyte/nicegraf/blob/master/source/ngf-mtl/impl.mm#L1403.
I set the bufferIndex of the vertex attribute descriptor to MAX_BUFFER_BINDINDS - <whatever binding number is specified in the shader>.  Yes, it will have values greater than 1.

Thanks! I found out the root cause. I was indexing the layout wrong. I used the same index for the attribute and layout of the vertex descriptor. The correct way was to use the bufferIndex for the layout index.


Added gradient slider with overlayer + horizontal grid

Nice :)

Hello this is great now your commit is an old format (filetree) I do not know how to change the PR to be in tonel.
I will ask some help. @tesonep
Hi pablo can you help evanpero?
I do not know the consequences if I integrate the PR and get filetree mixed with tonel.
S

I will check.
This is strange because I know that they migrated it.

Ok it is in filetree so I will integrate yoru change and I will check if I can convert the repository.

If you look into the Mooflod package you will see some tests and this is important to have some tests even in limited manner. I will help to maintain the code.

I integrated your changes now

we should not have unclassified methods (I will go over them)
copy pasting the full ICSimpleSlider is not optimal so we should probably refactor the code to avoid this in the future.


Using @eventType does not work

Per documentation @eventType should be either emit or broadcast. It appears that the existing is checking for on

To be exact, it's looking for /^([^\s]*)\s+on\s+([\S\s]*)/
/^([^\s]*)\s+ should probably be something more like /^(\s*), though it's later used as the type which doesn't make sense as @eventType doesn't use a type according to the ngdocs documentation.
The check for on shouldn't be there according to the documentation, but can be resolved with a ? to make it optional for backwards compatibility.
([\S\s]*)/ is going to require the string 'b r o a d c a s t ' if I'm not mistaken and should instead be ([\S]*)/ to grab the last word then ignore everything ever it.  To better enforce the documentation a (broadcast|emit) would work best imo.

Seems to be the correct functionality according to dgeni documentation


"View On GitHub" button not work for organization pages

Cayman uses is_project_page of github-metadata to check the project is github pages or not, but for this case, the checked result is the project is user page (that is is_user_page is true):
https://github.com/rust-open-games/rust-open-games.github.io
I must set is_project_page to true to enable "View On Github" button on pages view.
I don't know, is this an incorrect behavior of github-metadata, or is it cayman, worse case, that I understand it wrong?

I experienced the same behavior:

links showed in local testing of site
initial deployment (without is_project_page) on GitHub pages did not show link
addition of this configuration allowed links to be shown

Many thanks, I think it was resolved now.


Unable to open a realm at path [...] Invalid mnemonic

Recently we noticed an unknown realm error when trying to read schemaVersionAtURL with a wrong key on an encrypted database that was previously copied with writeCopy.
After writeCopy and accessing that Realm either through schemaVersionAtURL or through Realm(config...) we get an error with error code 2 and in the description you read Invalid mnemonic [...].
We currently don't know any other scenario in which this error pops up, but we noticed it kind of luckily because schemaVersionAtURL with a wrong password now throws with an error code 2 instead of code 1.
My questions are:

after writeCopy and connecting to the new copy but with a wrong key what should be the expected error?
can you reproduce the issue yourself and then also with my code snippets below?

How should we check for a wrong password in the future?
Currently for normal unlocking scenarios we do it with
case let e as NSError where e.code == RLMError.fileAccess.rawValue:
which is in fact error code 2.
But up until now we used catch let e as NSError where e.domain == "io.realm" && e.code == 1 when calling schemaVersionAtURL.
What are your suggestions? I am open to adapt our code to be consistent with your API.
Thank you very much in advance for your help!
Goals
With schemaVersionAtURL up until now we checked a DB if the password the user entered was valid. If not we expected to return an error with domain == "io.realm" and code == 1.
In that case we knew, the entered password was wrong.
All other errors are handled differently. We just wanted to know if the entered password was wrong.
Now the error code is 2, which should be fine also, we just have to update our code checking to check for code 2.
The problem is the error description, the message does say that there is a problem, but I cannot relate it to being a 'wrong password' error.
Expected Results
When trying to read the schema version with a wrong encryption key we expect, like it was previously the case, the error domain == "io.realm" and code == 1 (I guess it must have been Error Domain=io.realm Code=1 "Encryption key mismatch").
I looked it up, it came from realm-core
Actual Results
Got the following error:
Error Domain=io.realm Code=2 "Unable to open a realm at path '/var/folders/hx/xhr6l9d91y19cmp2gkd_85200000gn/T/888E677E-444D-451F-B8FC-014237B3E47F-5549-00029BB66432BE4E/DF3E1D3A-F415-4A0B-B3FF-F9FB2DC76A0A/db.realm': Invalid mnemonic. top_ref[0]: 0, top_ref[1]: 0, mnemonic: 0 0 0 0, fmt[0]: 0, fmt[1]: 0, flags: 0." UserInfo={Error Code=2, NSFilePath=/var/folders/hx/xhr6l9d91y19cmp2gkd_85200000gn/T/888E677E-444D-451F-B8FC-014237B3E47F-5549-00029BB66432BE4E/DF3E1D3A-F415-4A0B-B3FF-F9FB2DC76A0A/db.realm, Underlying=Invalid mnemonic. top_ref[0]: 0, top_ref[1]: 0, mnemonic: 0 0 0 0, fmt[0]: 0, fmt[1]: 0, flags: 0, NSLocalizedDescription=Unable to open a realm at path '/var/folders/hx/xhr6l9d91y19cmp2gkd_85200000gn/T/888E677E-444D-451F-B8FC-014237B3E47F-5549-00029BB66432BE4E/DF3E1D3A-F415-4A0B-B3FF-F9FB2DC76A0A/db.realm': Invalid mnemonic. top_ref[0]: 0, top_ref[1]: 0, mnemonic: 0 0 0 0, fmt[0]: 0, fmt[1]: 0, flags: 0.}
This also comes from realm-core.
Steps to Reproduce
Run the code samples below.
Code Sample
do {
    var tempUniqueFolder: URL {
        return URL(fileURLWithPath: NSTemporaryDirectory())
            .appendingPathComponent(ProcessInfo.processInfo.globallyUniqueString, isDirectory: true)
            .appendingPathComponent(UUID().uuidString, isDirectory: true)
    }

    let folderURL = tempUniqueFolder
    try FileManager.default.createDirectory(at: folderURL, withIntermediateDirectories: true, attributes: nil)
    let dbURL = folderURL.appendingPathComponent("db.realm")

    let copyFolderURL = tempUniqueFolder
    try FileManager.default.createDirectory(at: copyFolderURL, withIntermediateDirectories: true, attributes: nil)
    let copyDbURL = copyFolderURL.appendingPathComponent("db.realm")

    // Generate a random encryption key
    var key: Data {
        var data = Data(count: 64)
        _ = data.withUnsafeMutableBytes { bytes in
            SecRandomCopyBytes(kSecRandomDefault, 64, bytes)
        }
        return data
    }

    let correctKey = key
    let wrongKey = key

    let config = Realm.Configuration(fileURL: dbURL, encryptionKey: correctKey)
    let realm = try Realm(configuration: config)

    try realm.writeCopy(toFile: copyDbURL, encryptionKey: correctKey)

    // let's check the schema version with the wrong key on the copied db
    let copyVersion = try schemaVersionAtURL(copyDbURL, encryptionKey: wrongKey)
    precondition(copyVersion == 0, "not the expected version")
} catch {
    fatalError("\(error)")
}

But that not only is true for schemaVersionAtURL! Here is a more generalized version, which also throws this error (delete the created temp folder each time you want to run that code else the first Realm(configuration already throws because next time you run the code the key is different):
do {
    var tempUniqueFolder: URL {
        return URL(fileURLWithPath: NSTemporaryDirectory())
            .appendingPathComponent("realmTest", isDirectory: true)
    }

    let folderURL = tempUniqueFolder
    try FileManager.default.createDirectory(at: folderURL, withIntermediateDirectories: true, attributes: nil)
    let dbURL = folderURL.appendingPathComponent("db.realm")

    try FileManager.default.createDirectory(at: folderURL, withIntermediateDirectories: true, attributes: nil)
    let copyDbURL = folderURL.appendingPathComponent("copydb.realm")

    // Generate a random encryption key
    var key: Data {
        var data = Data(count: 64)
        _ = data.withUnsafeMutableBytes { bytes in
            SecRandomCopyBytes(kSecRandomDefault, 64, bytes)
        }
        return data
    }

    let correctKey = key
    let wrongKey = key

    let config = Realm.Configuration(fileURL: dbURL, encryptionKey: correctKey)
    let realm = try Realm(configuration: config)

    try realm.writeCopy(toFile: copyDbURL, encryptionKey: correctKey)

    // let's check the schema version with the wrong key on the copied db
    _ = try Realm(configuration: Realm.Configuration(fileURL: copyDbURL, encryptionKey: wrongKey))
} catch {
    fatalError("\(error)")
}

Version of Realm and Tooling
Realm framework version: 3.12.0
Realm Object Server version: The one that comes with 3.12.0 (we don't use sync)
Xcode version: 10.1 (10B61)
iOS/OSX version: 10.14.1 (18B75)
Dependency manager + version: Cocoapods  1.5.3

I found the following tickets that could relate to this issue:
realm-cocoa: #5896
realm-core: https://github.com/realm/realm-core/issues/3134
realm-java: https://github.com/realm/realm-java/issues/6257

Could you upgrade to the latest version of Realm and see if this solves your issue? Thanks.

Did you test it with the provided code with the new version?
In production currently we cannot upgrade so I'd need to test it with the code above, too.


git clone via ssh blocked when build image with dind

I use docker:dind in gitlab-ci, and set ssh private key before git clone. Before git clone really start, the console on gitlab shows "Warning: Permanently added the ECDSA host key for IP address '<MY_GIT_IP>' to the list of known hosts.", and stay block until timeout.
There is part of my script:
.gitlab-ci.yml:
build:
  stage: build
  image: docker:git
  variables:
    GIT_STRATEGY: fetch
    DOCKER_HOST: tcp://docker:2375/
  services:
    - docker:dind
  script:
    # the variable SSH_PRIVATE_KEY was set on gitlab>setting>CI/CD>Variables, the value is a key that have access right to my gitlab repositry.
    - docker build --build-arg SSH_PRIVATE_KEY="$SSH_PRIVATE_KEY" -t server:ci .

Dockerfile:
FROM python:3.5-stretch

ARG SSH_PRIVATE_KEY

RUN mkdir /root/.ssh/ \
  && echo "${SSH_PRIVATE_KEY}" > /root/.ssh/id_rsa \
  && chmod -R 600 /root/.ssh/ \
  && touch /root/.ssh/known_hosts \
  && ssh-keyscan <MY_GIT_DOMAIN> >> /root/.ssh/known_hosts \
  && git clone git@<MY_GIT_DOMAIN>:backend/<MY_GIT_PROJECT>.git

In my another project, the dind works perfectly without SSH connection to my gitlab, and also, the Dockerfile above works well on my PC without dind, so I think is there anything I missed, such as limit about network layer?

You have wrong permission with .ssh directory.

FYI:  https://superuser.com/questions/215504/permissions-on-private-key-in-ssh-folder

You have wrong permission with .ssh directory and ~/.ssh/id_rsa file.

I don't agree. The same Dockerfile can be build on my personal compute with the permission 600. sad

Can you reproduce using docker:dind on your local PC?
The best I can figure is that maybe GitLab (or some other intermediary) is blocking or severely throttling connections between your GitLab instance and wherever <MY_GIT_DOMAIN> is hosted.

I can use docker:dind in GitLab CI and it works correctly.
Maybe, you can check if your network is unblocked, as @tianon  said.

After setting verbose print mode with GIT_SSH_COMMAND="ssh -Tvvv" git clone git@<MY_GIT_DOMAIN>:backend/<MY_GIT_PROJECT>.git
the output shows:
...
debug2: callback done
debug2: channel 0: open confirm rwindow 0 rmax 32768
debug2: channel 0: rcvd adjust 2097152
debug3: receive packet: type 99
debug2: channel_input_status_confirm: type 99 id 0
debug2: exec request accepted on channel 0

the last debug2 log is where the git clone blocked at.
I also change the git clone script to git push a new project, and that success:
Initialized empty Git repository in /ssh-test/.git/
[master (root-commit) de1709b] Initial commit
 1 file changed, 0 insertions(+), 0 deletions(-)
 create mode 100644 aaa
To <MY_GIT_DOMAIN>:backend/<NEW_GIT_PROJECT>
 * [new branch]      master -> master
Branch master set up to track remote branch master from origin.
Removing intermediate container 75da3b6a2722
 ---> 8f84ad96a5a3
Successfully built 8f84ad96a5a3
Successfully tagged test-git:latest
Job succeeded

It seems the connection with my gitlab is correct, but I'm not able to clone the source code from it(but clone code from github is OK, and clone my gitlab repo without docker in docker is OK, too).

I test docker in docker on my local PC, and that works perfectly.
Now I can say that problem is the network/proxy of Kubernetes at my company.
Close this issue and many thank for your help.


'TypeError: Invalid value used as weak map key.' error in new testcafe versions

What is your Test Scenario?
Run tests from the attached archive
What is the Current behavior?
'TypeError: Invalid value used as weak map key.' occurs
Steps to Reproduce:

download attached test project Testcafe.zip
Run the following commands

npx<EMAIL_ADDRESS>chrome control.test.ts
npx<EMAIL_ADDRESS>chrome data-inspector.test.ts

PS. works ok in Testcafe v.1.9.4
Your Environment details:

testcafe version:                   1.16.0

fixed in https://github.com/DevExpress/testcafe-hammerhead/pull/2713


your to you're or you are

Grammatical error:
Writing guides typically don't encourage contractions in technical documentation, so probably should stick with 'you are'.
Replace storageaccount.file.core.windows.net with the appropriate FQDN for your storage account.
Note the proper suffix (core.windows.net) depends on the cloud your deployed in.
Resolve-DnsName -Name storageaccount.file.core.windows.net

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

ID: 7f5e21f3-0f89-d4f9-70c6-e2c238292863
Version Independent ID: 4f77c2f2-33e5-5481-8da8-77304704a62a
Content: Configuring DNS forwarding for Azure Files
Content Source: articles/storage/files/storage-files-networking-dns.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

The usage of your is correct here as it is possessive. https://www.kaplaninternational.com/blog/learning-languages/eng/your-vs-youre-grammar-differences#:~:text=Understanding how to use YOUR and YOU'RE%3A Your,the present participle (verb form ending in -ing).

Sorry, I think you must be referring to the first 'your' in the included text and not the second one.  The 'your' in question is in this sentence: "Note the proper suffix (core.windows.net) depends on the cloud your deployed in."  In this case, your is not possessive and is meant to convey "the cloud you are deployed in."  I apologize for the formatting issue in the original submission.

Thanks for clarifying. I have submitted a PR changing you're to you are and it should update shortly.


Add Carta app logo

[x] I have read and followed CONTRIBUTING document.
[x] I only added SVG(s) to the pull request.
URL of the site or app (required, unless it's generic icon): https://carta.com/
Where did you took the BG color(s) (required, unless it's generic icon): https://drive.google.com/drive/folders/1qpJhkwV85MB3RIJptWkdYsnxtiuGDkkJ

Issuer: Carta

Otherwise good, but logo IMO was too big.

Thank you!


[Image Resizer] Support only showing in extended context menu

Feature Request
Can Image Resizer include a setting where it can be configured to only show in extended context menu (i.e. Shift + Right click). It is already supported by Powerrename, and it would be great to also have this setting in Image Resizer.

Great idea!

@niels9001 We have a similar setting for Power Rename, would it be easy to copy it for this?


Local Tracks Disappearing on Refresh

What version of Music Assistant has the issue?
2022.7.0b9
The problem
This problem has been seen before and it appears that https://github.com/music-assistant/music-assistant-lib/pull/429 has not fixed it. When refreshing an artist all local tracks appear and errors appear in the logs. (See the first two log entries)
Havig said that before submitting this is tried a FORCE FULL SYNC which brought everything back but the same behaviour occurs. I went to another artist ALANIS MORRISETTE which was showing only local tracks. On the first REFRESH ITEM the Spotify Tracks appeared in the list along with the local tracks but then I did a refresh again and the local tracks disappeared. I thought I would try deleting the artist and rescanning but when I selected DELETE ITEM FROM DATABASE the third error shown below occurred. I saw this also when I deleted a spurious album entry so I am wondering if it occurs because you delete the album but MA stays on that page so it might be trying to reload info for the page which no longer exists. I think when you delete it should automatically move you up one level in the UI
How to reproduce
Go to ARTIST view and find an artist that has local and Spotify tracks. Select REFRESH ITEM in top right context menu. Observe all local tracks disappear from list. This does not occur with all artists.
Relevant log output
Logger: music_assistant
Source: /usr/local/lib/python3.10/site-packages/music_assistant/mass.py:247
First occurred: 11:36:21 AM (2 occurrences)
Last logged: 11:40:05 AM
Job [Add file://album/2celloscelloverse to database] failed with error Artist is missing provider id(s).
Job [Add file://album/acdc74jailbreak to database] failed with error Artist is missing provider id(s).
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 98, in add
db_item = await self.add_db_item(item, overwrite_existing)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 188, in add_db_item
return await self.update_db_item(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 230, in update_db_item
album_artists = await self._get_album_artists(item, overwrite=True)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 349, in _get_album_artists
album_artists.add(await self._get_artist_mapping(artist, overwrite))
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 361, in _get_artist_mapping
artist = await self.mass.music.artists.add_db_item(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/artists.py", line 203, in add_db_item
assert item.provider_ids, "Artist is missing provider id(s)"
AssertionError: Artist is missing provider id(s)
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/mass/websockets.py:762
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 11:36:21 AM (2 occurrences)
Last logged: 11:40:05 AM
[547040359040] Error handling message: Unknown error (unknown_error)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 27, in _handle_async_response
await func(hass, connection, msg)
File "/config/custom_components/mass/websockets.py", line 114, in async_get_mass_func
await orig_func(hass, connection, msg, mass)
File "/config/custom_components/mass/websockets.py", line 762, in websocket_item
if item := await mass.music.get_item_by_uri(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/init.py", line 225, in get_item_by_uri
return await self.get_item(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/init.py", line 248, in get_item
return await ctrl.get(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 36, in get
if album.artist:
AttributeError: 'NoneType' object has no attribute 'artist'
Logger: homeassistant.components.websocket_api.http.connection
Source: custom_components/mass/websockets.py:867
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 12:18:15 PM (1 occurrences)
Last logged: 12:18:15 PM
[546753123456] Error handling message: Unknown error (unknown_error)
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/components/websocket_api/decorators.py", line 27, in _handle_async_response
await func(hass, connection, msg)
File "/config/custom_components/mass/websockets.py", line 114, in async_get_mass_func
await orig_func(hass, connection, msg, mass)
File "/config/custom_components/mass/websockets.py", line 867, in websocket_delete_db_item
await mass.music.delete_db_item(
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/init.py", line 287, in delete_db_item
await ctrl.delete_db_item(db_item_id, recursive)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/artists.py", line 282, in delete_db_item
await self.mass.music.albums.delete_db_item(db_row["item_id"], recursive)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 279, in delete_db_item
await self.mass.music.albums.delete_db_item(db_row["item_id"], recursive)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 279, in delete_db_item
await self.mass.music.albums.delete_db_item(db_row["item_id"], recursive)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 279, in delete_db_item
await self.mass.music.albums.delete_db_item(db_row["item_id"], recursive)
File "/usr/local/lib/python3.10/site-packages/music_assistant/controllers/music/albums.py", line 282, in delete_db_item
await super().delete_db_item(item_id)
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/media_controller.py", line 419, in delete_db_item
db_item = await self.get_db_item(item_id)
File "/usr/local/lib/python3.10/site-packages/music_assistant/models/media_controller.py", line 317, in get_db_item
raise MediaNotFoundError(f"Album not found in database: {item_id}")
music_assistant.models.errors.MediaNotFoundError: Album not found in database: 4294
Additional information
No response
What version of Home Assistant Core are your running
2022.7.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Raspberry Pi

please confirm if this is fixed in beta 10

fixed


Better virtual console support

In order to showcase Solid's output and capabilities, it's often interesting to log stuff in the console.
One thing that'd be really nice is that the virtual console would be able to show a bit more complex types such as dom elements, proxies, deep object, etc..

This might be an interesting reference (although reactivity within it needs to be reworked) https://github.com/CM-Tech/solid-debugger/tree/main/src/json-tree

Oh I remember that visualization of yours! I would absolutely LOVE to have that somehow integrated in the playground. It would totally be a real plus compared to other more generalized tools such as CSB or StackBlitz.
I see two possibles course of action here.

Since you seem to already have a good part of the JSON parsing done that goes way beyond JSON.stringify, we could first introduce that to the virtual console somehow, which I guess would require some sort of external package on your end, or perhaps try and see what the other use.
Literally integrate some sort of opt-in devtools within the playground. That could also be a good way to dogfeed the devtools before trying to integrate it deeper in the ecosystem (as a chrome plugin or something else).

What are your thoughts on that? @modderme123

So I added a little improvement on that area where I check if the argument is a dom element and log it's outerHTML to the console. Most of the time, when you log, that's the only thing you want to know, you don't necessarily need more. I think we could simplify what @modderme123 has done to just printing basic stuff and checking of each and every case.
Besides Proxys I'm not sure if there are much more specific case that are not handled by JSON.stringify. Maybe Map and Set?


Docs for micro-async-data-loader need improved

Specific examples showing setting default ManifestComparator key for both s3 and couchbase required.

Merged on master


Fix typo in comment

Please. Don't open one pull request per single edit.
Please merge all of your changes into one pr and reopen it.

Ok, I'll try :man_shrugging:. But that takes slightly more time, so probably I won't be fixing as many typos as I could.

I don't know why that takes more time. Just read all the code you want to read, fix typos while doing this and at the end of the day you send one pr. If you send each day a pr, fine. But not 3 prs, over one hour or similar.

It's not that it takes a lot of time more. It's that it takes a little more time. I'm usually reading random source code at the end of my work day, directly in github. I open a file, read, see a typo, hit edit, hit commit, hit pull request. Go on reading. This takes very very little time and brain power.
I don't know if I will find more typos, so I prefer to do it immediately, when I have context over what's going on, and before I forget I had a commit hanging around.
Setting the bar just a little bit higher than that usually makes me run for dinner, drinks, or bed without thinking of anything else. I just shut down all the devices without thinking if I had some fixes in progress. Real life beats typos.
I understand you would prefer to have less pull requests to review. Of course. However, my experience as maintainer has been that if you encourage contributors to help in whatever way fits them best, even on tiny thinks like proof-reading your documentation, they will come back and become every day more engaged.
But I do see your point. And it's your project, the last thing I want to do is to add noise. I will try. I still have lot's of substrate to read :man_facepalming:.
One tip: run codespell in pull requests. It catches most of the things I've fixed.

Thanks for your detailed explanation :)


Platform Specific Insights

Is your feature request related to a problem? Please describe.
How do we collect framework specific accessibility bugs from the telemetry that you are collecting to contribute back to the projects that they are built with?
Describe the desired outcome
I would like to see a list of Drupal, WordPress, GatsbyJS, etc issues, Ideally aligned with modules or plugins that are used. Would be nice to start being able to identify where accessibility bugs are introduced upstream.
Describe alternatives you've considered
I have asked groups like Siteimpove & Tenon to implement things like this.

Hi @mgifford thanks for the feature request!
This request requires more discussion on our side, we will update here when we know more.

It is a huge issue.  That said, it has some really potential benefits.  If you could identify that an accessibility problem arises on 90% of all Drupal 8 websites that use the Bootstrap 4 theme, but doesn't occur on any of the WordPress sites that use Bootstrap 4, it would be possible to highlight post an issue here https://www.drupal.org/project/issues/search/bootstrap that could potentially fix thousands of sites.
Likewise, if it hit all Bootstrap 4 themes maybe you could find the issue here https://github.com/twbs/bootstrap/issues and provide a summary like:
Google Estimates that this accessibility issue affects 23 million people a month around the world. 200,000 people with disabilities face barriers every month because of this bug.

We understand the value you present with this example. The main questions on our side are two:
•	Would Accessibility Insights users be ok with sharing this type of telemetry?
•	And, how do we detect this?
Related to your request, have you seen webAIM report of scanning a million websites? They have some data on the Web Frameworks from the million home pages scanned
Thanks!

I would assume you'd want to have some sort of checkbox that says something like "I approve of information being consolidated so that this product and the web in general can be made more accessible." You'd need to say how the information was going to be used and make the permission an opt-in.  That said, it really could be something that organizations are encouraged to do.
There are lots of ways to detect the framework. Not always 100% but usually there are signs.  There are tools like https://builtwith.com/ that could be leveraged.  There are a few other options with https://www.ostraining.com/blog/drupal/5-ways-drupal/ & https://www.isitwp.com/
The WebAIM report is great. I haven't looked at the data, but amazing that they have it. I look forward to seeing how it compares with reviews done in 2020!

Hi @mgifford, thank you for your interest in Accessibility Insights and this feature request. We have discussed this as a team and concluded that it will not be possible to share this information as we take the privacy of our users very seriously.
As suggested in one of our earlier responses, please review the report from webAIM that analyzes accessibility issues in the top 1 million web pages. You can also scan individual pages in question using Accessibility Insights.
We value your feedback and thank you for being a user of Accessibility Insights!


fix: match npm install semantics by default

Changes behaviour with existing locks to match npm by default (i.e. if other
install flags like freeze aren't set). The only difference is how we handle
secondary locks: if they're in range for their parent package.json, we now
use the lock, and if they're out of range we reset them to latest compatible,
which is what npm does in these cases.

Note that these lock changes also affect non-install operations, which again matches npm's behaviour.


Console error when using TileMap bucket fill tool in editor

Godot version:

v3.3.stable.official
OS/device including version:

Windows 10 / Desktop PC.
Issue description:

When using the TileMap editor bucket fill tool it prints this error in the console:
Bucket Fill
 scene/gui/item_list.cpp:257 - Index p_idx = -1 is out of bounds (items.size() = 0).

I would expect no errors to be shown since nothing appears to be wrong visually. This error also gets printed when you move your mouse around with the bucket fill tool selected.
This "bug" doesn't affect the actual tilemap behavior but I'm reporting it because the console error might be a symptom of a deeper issue. Also I don't recall this error being printed before v3.3 so it might be a regression.
I reported another v3.3 tilemap related issue earlier: #48092. I don't know if it's related but it seems like maybe there were some tilemap related changes in 3.3 that are worth taking a look at.
Steps to reproduce:

Open the attached project
Make sure the "output" console tab is open/visible.
Select the "TileMap" node
Press "B" to activate bucket fill tool
Click in the middle of the yellow tilemap box to fill it.
Notice the error printed in the console.

Minimal reproduction project:

TileMapBugDemo.zip

Caused by #42979.


Set Record.OffsetDelta to avoid broker recompression

This mimicks #1002, but for v3 requests. On Kafka 1.0.0
this prevents recompress on broker to add missing offsets.

Thanks!


Add Gradle toolchains Support

Added Gradle support with reference to the ASDF implementation. #9
Worked fine on Windows and MacOS.

Thanks!


HTML Media Capture: Mark not implemented tests as 'Not applicable'

@anssiko, could you PTAL?

LGTM, with a comment.
Add the following note below the failure analysis paragraph:

Not applicable:

/html-media-capture/capture_audio-manual.html
/html-media-capture/capture_audio_cancel-manual.html
Audio capture is not a normative requirement. The tests are included for completeness and to help future implementations that may want to implement audio capture.

@anssiko, thanks for the review, just addressed your comment.

@Honry, LGTM. Thanks!
@dontcallmedom, can you please merge?

@anssiko, I think I can merge it directly.


Maximized window issues: tasks not visible

Only vertical arrangement of task does not look good when we maximize the window. Tiling the tasks would look better on a maximized or big window

This is small window: (looks ok)

I made tasks wider in 44.5 so it looks better maximized. Tiling gonna add too much complexity, I want to keep application simple as a continuous sheet of paper and not over-complicate things.


code cosmetics

Just a bit code cosmetics

You may also add grunt-contrib-jshint and grunt-contrib-jscs to enforce consistency and prevent future PRs of this kind.

:+1: for merge.

fix all tests !

Tests fixed.
Even tho it doesn't seem to be logical to have the code that way how it is done by digitalBush


New NegativeInt type

Description
We would like to define a new NegativeInt serializable type in the kotools.types.number package for representing negative integers including zero. Here's the minimal API for this type:
public fun NegativeInt(value: Int): NegativeInt {
    TODO("Not yet implemented")
}

public interface NegativeInt {
    override fun equals(other: Any?): Boolean
    override fun hashCode(): Int
    public fun toInt(): Int
    override fun toString(): String
}

This type should be available on all platforms and should be compatible with Kotlin and Java.
Its serialization should behave like an Int.
Checklist

[ ] Implement the type and its minimal API.
[ ] Make the type serializable.
[ ] Check the generated documentation.
[ ] Check and update the public API binaries.
[ ] Update the Unreleased section in changelog.

We will use the existing type instead of defining a new one.

We will work on the existing type instead of reimplementing it from scratch.


Add discovery via akka-dns example + exercise on the new down-all SBR strategy

Adds SBR exercise with down-all strategy
Adds Akka Bootstrap / Discovery using akka-dns

From chat: the interesting example here is exercise_051_, and the question is why cluster bootstrap appears to be doing A lookups instead of SRV lookups.

there seem to be a lot of different things in this PR apart from just the discovery-dns stuff (renames, changes in different exercises). would have been good to split that up more, now it is hard to follow why changes were made.
If I understand correctly the discovery-dns example needs a dnsmasq to run, should its configuration also be included?

Doing that - the README.md file for this exercise was missing.

there seem to be a lot of different things in this PR apart from just the discovery-dns stuff (renames, changes in different exercises). would have been good to split that up more, now it is hard to follow why changes were made.

@raboof Correct... I could have separated the addition of the exercise on the down-all strategy in a specific PR. As this new exercise is inserted between two existing exercises, the exercises after the new one are renumbered leading to an explosion in changed files because the folder names change. Little that can be done about this unfortunately.
In addition to that, I had already started working on exercise 51 (akka-dns), so this got "bundled" with it.


feat(DataStore): DisableRealTimeUpdates flag

Issue #

https://github.com/aws-amplify/amplify-swift/issues/3220
Description

General Checklist

[ ] Added new tests to cover change, if needed
[ ] Build succeeds with all target using Swift Package Manager
[ ] All unit tests pass
[ ] All integration tests pass
[ ] Security oriented best practices and standards are followed (e.g. using input sanitization, principle of least privilege, etc)
[ ] Documentation update for the change if required
[ ] PR title conforms to conventional commit style
[ ] New or updated tests include Given When Then inline code documentation and are named accordingly testThing_condition_expectation()
[ ] If breaking change, documentation/changelog update with migration instructions

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

:exclamation: No coverage uploaded for pull request base (lawmicha.int64@b622589). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@                Coverage Diff                @@
##             lawmicha.int64    #3368   +/-   ##
=================================================
  Coverage                  ?   68.00%
=================================================
  Files                     ?     1078
  Lines                     ?    35992
  Branches                  ?        0
=================================================
  Hits                      ?    24475
  Misses                    ?    11517
  Partials                  ?        0

Flag
Coverage Δ

API_plugin_unit_test
67.07% <0.00%> (?)

AWSPluginsCore
64.45% <0.00%> (?)

Amplify
47.83% <0.00%> (?)

Analytics_plugin_unit_test
81.16% <0.00%> (?)

Auth_plugin_unit_test
79.57% <0.00%> (?)

CoreMLPredictions_plugin_unit_test
59.44% <0.00%> (?)

DataStore_plugin_unit_test
79.87% <0.00%> (?)

Geo_plugin_unit_test
70.75% <0.00%> (?)

Logging_plugin_unit_test
63.09% <0.00%> (?)

Predictions_plugin_unit_test
37.29% <0.00%> (?)

PushNotifications_plugin_unit_test
87.03% <0.00%> (?)

Storage_plugin_unit_test
78.00% <0.00%> (?)

unit_tests
68.00% <0.00%> (?)

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

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


This stops working in version 2024.10.0.

HA core version
2024.4.0
Intergration version
0.3.22
Device type and model
Air Conditioner 00000Q13 (0)
Used App
SmartHome 3.2.2
The description of problem
The midea_ac_lan custom integration is using deprecated climate auxiliary heater
This stops working in version 2024.10.0. Please address before upgrading.
The custom integration midea_ac_lan implements the is_aux_heat property or uses the auxiliary heater methods in a subclass of ClimateEntity.
Please create a bug report at https://github.com/georgezhao2010/midea_ac_lan/issues.
Once an updated version of midea_ac_lan is available, install it and restart Home Assistant to fix this issue.
Warning - 3/4/2024
The logs
No response

Dupe #483

Pull https://github.com/georgezhao2010/midea_ac_lan/pull/420

I've come across the same message... we all get it...


Wow

Radeon HD3650
Intel i5 3470
Ubuntu 14.04
Botmark with ~56k sprites barely ebbing below 60FPS.
That's pretty neat........
Just wanted to say thanks for sharing! I hope this gets some more work, I'll definitely be tinkering with it :3

Thanks! I am currently working on making things better. It always helps to know other people are interested.

Maybe I could throw throw some requests your way?
Abstracted tilemap object with texture atlasing and spritebatching (though seems you got sprite batching down already :D ). Also, support for borderless tilemapping/chunking/faux-infinite tile maps.
I was working on learning OpenGL a bit in order to get the tilemapping stuff done (coming from C#/XNA) and using a key/value map to hold the pointers to "chunks" of the map. ( map[uint64]*Chunk )
Using uint64 so that I can bitwise concatenate two 32bit integers for an x,y coordinate-id of a given chunk (not an individual tile, I thought chunks maybe could have their own coordinate space, and maintain world coordinate space as a double or just 64bit integer), and use that as the key to the pointer containing the chunk.
func Pair(x, y uint32) uint64 {
	pair := uint64(x)
	return pair<<32 | uint64(y)
}

func Unpair(val uint64) (uint32, uint32) {
	x := uint32(val >> 32)
	y := uint32(val & 0xFFFFFFFF)
	return x, y
}

Also, on my big rig:
i7 5930K
DDR4 2800MHz
EVGA GTX980 Superclocked
http://i.imgur.com/bXEx0E0.png


added botanix-testnet

Adding new chain: Botanix Testnet
Using id: botanix-testnet
bun validate returned no errors.

Use https://api.routescan.io/v2/network/testnet/evm/3636/etherscan/api in apiUrls

While you are at it, add https://api.routescan.io/v2/network/testnet/evm/11124/etherscan/api to abstract-testnet API


Improve naming of export zips

From Community Wish List
Append the text 'deployment' or 'archive' to the zips created when you export a project

Hi Fay,
I've added this temporarily to the Decision Tree to let people see if this is what is desired.
However, regarding the XOT template, we have deployment, archive, offline, scorm (1.2 and 2004) and would we also want to add flash / html5 to the name? Or we assume HTML5 and only add Flash since that would be a special case?
So we'd possibly have "Project_Name_deployment_flash.zip" or "Project_Name_archive.zip"

Hmm, I never export so not sure what would be best. I don't think it would need to say html5 on them all. I take it none of this would affect importing something back in?

No it's just the filename that is prompted when you hit one of the export options.
I assume it came up as it's possible to overwrite say and archive one with a scorm one that has been saved for a different option...
I'll wait to see what other people think before implementing the rest but yeah as you say adding _html5 to the filename is probably pointless since it's the norm and I would just add _flash possibly if that option is chosen.

Hi both
I usually manually rename the zip when I export to reflect the type I chose. I guess it's useful to automatically add the format chosen but we need to make sure this doesn't break anything else e.g. export links in the syndication feed, .htaccess rules etc? Not saying that it will but I know for instance syndicate.php include links that end with /export_1123 and /scorm_123

Hi Ron,
The change I made only changes the name that is presented to the user for saving, similar to if the user manually added the suffix in the project name so i'm pretty sure it won't break anything that the user couldn't break by renaming.
I assume that /export_1123 and /scorm_123 simply redirect to the relevant export.php? page with the correct parameters? It would definitely be useful to test this though before deployment.

Hi John
I just quickly checked your commit on a live install and seems to work fine in that the syndication feed link still works so isn't affected by he appending of archive etc.
However seems perhaps there's a separate issue with the syndication feed code displaying links that aren't really relevant. e.g. Create a decision tree project, make it part of the export feed, view the export feed and you get two links e.g.
Download this content from installaddress/website_code/php/scorm/export.php?scorm=false&template_id=123
Download this as a scorm package from installaddress/website_code/php/scorm/export.php?scorm=true&template_id=123
Both links do actually create a zip but obviously there is no scorm export at the moment from decision tree and using either link results in filename_deployment.zip

Ok, i'll take a look at that. I suspect that has always been the case for decision tree as there is a lot less code and error checking in there than in the XOT one.
Just to be sure, what should each of the two urls above give? Or is it that fact that you get one url in the feed that says scorm=true that is the issue? I assume that the two URLs presented (if indeed you should be getting 2 for this template) should still be Archive and Deployment?

Hi John
I'm not sure this is a major issue as you say it's likely always been the case.
I'm not sure how much the syndication feed is used but I've always thought that should show the same export options as is available for any given template if that template has been added to the feed of course. Obviously because SCORM export isn't currently available for decision anyway it just downloads a deployment zip regardless of the link used.
More importantly what I've shown users how to do is to add a download link with in their LO's by adding a project to the syndication feed, enabling for export, copying the link from the syndication feed and then adding that link to a page in the resource. The syndication feed only currently shows links for scorm=false and scorm=true regardless of template used. Yet given that we have offline export in the xot template that might be more likely the link users would want or to add to their LO's for learners to download for offline access. I'm not sure if there's a way to manually construct and add a link to achieve that. If we were to add extra links or remove redundant links to the syndication feed to reflect the export options available for a given template I'm not sure the archive option should be include because that's a bit redundant these days e.g. you can't really edit the rlt file in xerte desktop any longer because of the mismatch in templates and html 5 output.

Haven't yet addressed the syndication feed problems yet since I can't reproduce... All I see when I follow your steps above in Mac/Chrome is:

Hi John
is that the standard rss feed or the syndication feed? If the latter you need to make those projects part of the export feed e.g.

Hi Ron,
Sorry for delay getting back to you. I have these ticked in all projects:

And yet I still get this in Chrome:

And this in Firefox:

Is this a bug? This is a new install checked out of Github develop rather than v3.2 but if there are errors in develop they could make their way into v3.3 when the time comes...

Hi John
sorry this likes too basic a question so perhaps there is a bug with different revisions but just to be sure...
The plain rss feed never shows the export links but for the export links to show in the syndication feed the include in export feed needs to be ticked as yes via RSS properties.
That actually is all that is needed for a self-access export link to work but for most users to get the format of that it's easiest to grab from the syndication feed.
For the projects and their export links to show in the syndication feed the include in syndication feed needs to be ticked in the open content properties:

Now I suspect you already have this so could it be caching? Seems unlikely to be a revision difference but could be I guess?

Ah, thanks Ron,
That was the missing key, I hadn't ticked that box. I'll take a look at that issue now that I can reproduce.

I'm going to close this and open a new one to have the syndicate.php page and export.php pages modified as we have issues with scorm download links for RSS and Decision Tree and you can even download an RSS template as a Scorm package.


Changing example to use AWS-SDK built-in promises

Rather than creating a wrapper, we can use promises directly from the AWS-SDK.

Good catch! It's been on my todo to test the AWS SDK's promise method. We should be able to do this on the front end as well.
I'll test this out and update the code samples as well.

Merged into master. It's live.


Merge request conflicts remaining after manual resolution, counters showing "N/A"

Description
I have a pull request that had a merge conflict, which I resolved manually earlier today, and pushed the changes to Gitea, but Gitea still thinks that there is a merge conflict. Other software I use (Git Extensions and my IDE) show that the merge was successful, and I see the merge commit on Gitea too. The Gitea Commit Grapgh also confirms that the branches were merged in a merge commit.
Other than that, the counter on the "Commits" and "Files Changed" tabs near the top of the merge request page show "N/A" instead of the number of commits and files changed. I think it ended up this way after I uploaded the merge commit.
The tabs are also not clickable, but if I forcibly open them by loading them by URL (e.g. https://gitea.rpi.lan/MPeter/twitch-archiver/pulls/39/commits), then the Commits tab shows "These branches are equal. (master)", and the Files Changed tab shows "Diff Content Not Available".
Opening the Commits tab resulted in an "500 Internal Server Error" being printed to the log but only once, navigating elsewhere and then back to this by URL does not log that error again, and opening the Files Changed tab did not log an error at all.
I don't have logs from when the commit was updated. Forgot to save them before updating Gitea..
Right now I'm running Gitea 1.19.4, but I have recently updated from 1.19.0, which was in use when the merge commit was uploaded.
#24729 may be related, however I didn't even know there should be an option to mark it as manually merged.
Gitea Version
1.19.4
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
https://gist.github.com/mpeter50/ae538e25ae60fa5b3aedeeb110c1b967
Screenshots

Top of PR page
Bottom of PR page

Commits tab
Files Changed tab

Git Version
2.38.5
Operating System
Debian 10
How are you running Gitea?
I have built Gitea from the original sources, using the Docker.rootless Dockerfile in the repo root, at the v1.19.4 git tag, on an x86_64 OS for ARM.
I run Gitea with Docker.
Database
MySQL

#283 looks even more similar: same situation, similar result, even the mentioned counters are the same.
But, in my case Gitea does not seem to want to rerun conflict checking. The screenshots I have taken are all from just a few minutes before sending in the issue.

Oh, it turns out at first they had the same error message as me, but closed and reopened the PR to get that other state.

The same thing happens when Gitea could do the merge automatically, but I forget about it and do it locally, and push the merge commit. The problem may be that Gitea does not realize that the branch has been merged.


Switch reproducer to networking_mapper

Remove most of the IP management from ansible
All of the IP management will mostly rely on the networking_mapper role
and associated tasks.
In order to avoid duplications, we're now ensuring we're just keeping
the minimal parts to get the network interfaces attached to the VMs;
we're also removing the static_ip support and the support for the
cifmw_libvirt_manager_networks parameter.
This patch has been locally tested against:

3-nodes.yml reproducer scenario
validated-architecture-1.yml reproducer scenario

Refactor reproducer role to consume networking_mapper
This patch switches the reproducer role to networking_mapper when it
comes to networking configuration.
It means we have to make a deep refactoring, as well as changing some of
the logic underneath the various tasks.
Mostly, we have to ensure the controller-0 is accessed using the right
user (see previous patch in this same PR), but also ensure controller-0
is ready before we get to the CRC (and OCP) node(s). This is mandatory,
especially for CRC, because we need to get its fixed IP.
This IP is generated via the networking_mapper, so we have to call it
early enough.
Since we'll be consuming some crafted output generated from the
networking_mapper data during the product deployment, we also have to
ensure we have all the needed pieces on the controller-0 - meaning it's
easier to run everything from that node directly, so that the content
gets generated in the right location since the beginning.
There's also a deep cleaning about the delegate_to and related
remote_user, delegate_facts and others since we're consuming a known,
fixed user via the inventory we update dynamically from the
libvirt_manager.
Most of the features should be properly tested from within the molecule
tests running against the reproducer role - but also the ones running
against the libvirt_manager directly.
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1036
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1037
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1032
Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/1043
As a pull request owner and reviewers, I checked that:

[X] Appropriate testing is done and actually running
[X] Appropriate documentation exists and/or is up-to-date:

[X] README in the role
[X] Content of the docs/source is reflecting the changes

/test all

/test all

/test pre-commit

/hold
I want #1043 to get in first.

/unhold

I tested this with the reproducer and then passed the genereated values.yaml to the architecture kustomize.
The output looks good to me so far though the NNCPs get FailedToConfigure because the base_inteface needs to not have the "dot vlan" name. In other words:
[zuul@controller-0 hci]$ grep base_iface values.yaml    #BAD
        base_iface: enp6s0.20
        base_iface: enp6s0.21
        base_iface: enp6s0.22
[zuul@controller-0 hci]$ grep base_iface values-ci-framework.yaml    # GOOD
    base_iface: enp6s0
    base_iface: enp6s0
    base_iface: enp6s0
[zuul@controller-0 hci]$

Otherwise I see all my CRs generated by this:
cd ~/architecture/examples/va/hci
cp ~/ci-framework-data/artifacts/ci_gen_kustomize_values/values.yaml .
kustomize build .

I tested this with the reproducer and then passed the genereated values.yaml to the architecture kustomize.
The output looks good to me so far though the NNCPs get FailedToConfigure because the base_inteface needs to not have the "dot vlan" name. In other words:
[zuul@controller-0 hci]$ grep base_iface values.yaml    #BAD
        base_iface: enp6s0.20
        base_iface: enp6s0.21
        base_iface: enp6s0.22
[zuul@controller-0 hci]$ grep base_iface values-ci-framework.yaml    # GOOD
    base_iface: enp6s0
    base_iface: enp6s0
    base_iface: enp6s0
[zuul@controller-0 hci]$

Otherwise I see all my CRs generated by this:
cd ~/architecture/examples/va/hci
cp ~/ci-framework-data/artifacts/ci_gen_kustomize_values/values.yaml .
kustomize build .

Thanks for confirming, @fultonj ! For the network interface, I nudged a dedicated patch #1046 (should merge shortly). And, also, there's now a patch in this current PR to configure the ctlplane interface on the compute nodes :). Sooooo.... you should be able to actually deploy :smile_cat:

recheck

The install_yamls_makes : Run netconfig_deploy task returned The connection to the server api.crc.testing:6443 was refused - did you specify the right host or port? in BM job - I don't think it's due to the changes, just keeping this here in case it fails again, to compare.

I tested this with the reproducer and then passed the genereated values.yaml to the architecture kustomize.
The output looks good to me so far though the NNCPs get FailedToConfigure because the base_inteface needs to not have the "dot vlan" name. In other words:
[zuul@controller-0 hci]$ grep base_iface values.yaml    #BAD
        base_iface: enp6s0.20
        base_iface: enp6s0.21
        base_iface: enp6s0.22
[zuul@controller-0 hci]$ grep base_iface values-ci-framework.yaml    # GOOD
    base_iface: enp6s0
    base_iface: enp6s0
    base_iface: enp6s0
[zuul@controller-0 hci]$

Otherwise I see all my CRs generated by this:
cd ~/architecture/examples/va/hci
cp ~/ci-framework-data/artifacts/ci_gen_kustomize_values/values.yaml .
kustomize build .

Thanks for confirming, @fultonj ! For the network interface, I nudged a dedicated patch #1046 (should merge shortly). And, also, there's now a patch in this current PR to configure the ctlplane interface on the compute nodes :). Sooooo.... you should be able to actually deploy 😸

Yes, the latest update seems to have resolved my problems. Thanks!

I'll review this PR on Monday's early morning.

/approve


ARROW-13634: [R] Update distro() in nixlibs.R to map from "bookworm" to 12

This updates the script used when installing the R package to correctly infer numerbing with the new Debian unstable version

@nealrichardson This is working now; here's the results from running on Debian Testing.
> distro()
$id
[1] "debian"

$version
NULL

$codename
[1] "Debian GNU/Linux bookworm/sid"

$short_version
[1] "12"

The reason that this didn't work properly last week (the values in /etc/os-release corresponded to the previous Debian version) was because the values in /etc/os-release are dependent on the base-files Debian package, which wasn't updated until 22nd August.  After this date, spinning up a debian:testing docker container, and running apt-get update followed by apt install base-files updated the values in etc/os-release to the correct ones.

@nealrichardson I have now updated this PR to use the code from the latest updates to distro.


Preselect tracked files on open

Preselect files that have been modified and deleted, but new files you have not told Git about are not selected.

Try pulling this. My name might show up. If not go ahead and merge it anyway

Okay, thanks again for the contribution!
:+1:


jenkins: Investigate why thinbackups have rather deep directory paths in them

This, which is in the thinbackup files, looks quite wrong. It is not specific to this particular build job, but I'm using 11/s390x as an example:
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin
17184	./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jobs/jdk11u-linux-s390x-temurin

Not only is it weird, but it is potentially chewing up a lot of space in the backup files unnecessarily. Each one has a different timestamp and is spread over teh course of an hour which gives the impression that it's replicating the contents multiple times over the course of an hour (it's taking roughly 3-4 minutes to get through each one)
15½Gb of the 18Gb in the extracted backup file is under ./FULL-2023-04-11_05-00/jobs/build-scripts/jobs/jobs/jobs/jdk11u/jobs/jobs/jobs/jobs/jobs so I feel there are large savings to be had here.

Note: Only seems to be causing this problem for jdk11u - suggesting a potential rogue symlink on the server for that version.

Rogue symlink in the jobs directory under jdk11u was at fault. Removed.
This should allow the thinbackups to complete successfully (It wasn't including anything after the build jdk11u jobs)


Unable to edit document if field _id is not ObjectId

Option Edit in context menu is inactive when trying to edit document with non ObjectId() value in _id field
Environment:

Ubuntu 18.04.1
Intellij IDEA 2018.2.1
Mongo plugin 0.12.0
MongoDB server 4.0

Steps:

prepare MongoDB server
add testing server to IDEA
prepare data for the test with the following commands:

> use somedb
> db.somecollection.insert({_id: "some-identifier0", foo: "foo", bar: "beer"})
> db.somecollection.insert({foo: "some-foo", bar: "tequila"})

open test DB in IDEA
try to edit first and second documents

Actual result:
document with non-ObjectId() _id field could not be editable
Expected result:
user can edit all documents in DB using context menu and Edit option
Attachments:
Video on Youtube

Hi,
This is an interesting case. What are the current best practices in term of type ? String or ObjectId.

I think, in 90% cases ObjectId is used, but _id may be of any type other than an array. mongodb docs

Any plans to fix/support this feature?


luci-app-unbound: option 'add_extra_dns' and list 'trigger'

This coordinates UCI option 'add_extra_dns' and list 'trigger' with PR https://github.com/openwrt/packages/pull/4361.
HOLD
Please wait for review and comments by requester @poranje

@hnyman because openwrt/packages#4361 is merged, we should merge this as well. Unless somebody has a spectacular idea how to make s1:taboption("resource", Value, "trigger", translate("Trigger Networks:") prettier. Otherwise I will make that better at a later time.

Let's merge it and you can polish it later.


YCS overlays important info about the video

If Chrome on Windows is set to any magnification above 90%, you cannot see the number of views and posting date. Can see less and less of it as you increase magnification. I use 110% at all times, and cannot see the views.
(Sample at 125% zoom)

(Sample at 90% zoom)

depends on your monitor. I can go with 125%

Inspect tool. Select the div, element style: display: block
Now you need to find a way to save the style somehow. "Permanent Inspect Element" didn't work for that case, but maybe you'll get it to work


Paragraph support for mustache ?

Does the paragraph ui object supports rendering mustache templates like Node-RED's built-in template node ?
That would be a great addition to render dynamic content !

Not mustache, but angular templates and expressions.
You can write a template like:
The payload is <b>{{msg.payload}}</b>
or expressions:
1+2={{1+2}}

Thanks I'll have a look at the docs !


Update README.rst

Adding example on how to save a modified entry, as this is no where documented and for starters requires a lot of time to find.

You probably don't want to save for each entry. Suggest moving the second line you added out of the for loop.

Codecov Report

Merging #116 (77dc74a) into master (8eb8e3f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #116   +/-   ##
=======================================
  Coverage   95.28%   95.28%
=======================================
  Files           1        1
  Lines         849      849
=======================================
  Hits          809      809
  Misses         40       40

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 8eb8e3f...77dc74a. Read the comment docs.

@kthy is right, please update the PR to match a real usecase, thanks !

@kthy is right, please update the PR to match a real usecase, thanks !

Indeed, no one would realistically want to set all entries to the same, fixed string. (and surely not save after every entry. Besides, po doesn't exist, the name in that example is pofile)
My suggestion:

Revert the changes to that loop. Reading entries (specially in a loop) and modifying some to add translations are usually distinct steps anyway.
Add a new step to fetch distinct, arbitrary entries (showing how you can do it, using .find(), [i], etc.)
Change the msgstr for those retrieved entries
Save the file

Something like this:
for entry in pofile:
    print(entry.msgid, entry.msgstr)

entry = pofile.find("foo")  # by default it searches by msgid
if entry is not None:
    entry.msgstr = "bar"
    pofile.save('/path/to/pofile.po')   # or simply pofile.save() to overwrite


Panstart event not recognized after .stop()

Hey,
It seems that whenever I stop to recognize events in the current input session, the next panstart is "skipped".
Am I doing something wrong? I am trying to avoid panning if the origin of the pan is not at the edge of the screen. I use Hammer v 2.0.4.
hammer = new Hammer(document.documentElement)

hammer
      .on('panstart', (e) ->
        if e.center.x < 50
          panstart(e)
        else
          hammer.stop()
      )
      .on('panmove', (e) ->
        e.srcEvent.stopPropagation()
        panmove(e)
      )
      .on('panend', panend)

Duplicated this bug: http://codepen.io/anon/pen/pJdBJB
The session should be cleared on the next input start, but is apparently not: https://github.com/hammerjs/hammer.js/blob/master/hammer.js#L443-L451
It seems likely the previous input session is still considered active following .stop(), even when forcibly stopped with .stop(true).
using FORCE_STOP did not let the next INPUT_START begin a new session either.
The session + recognizer logic is here: https://github.com/hammerjs/hammer.js/blob/master/src/manager.js#L71-L119

So i've just now discovered this bug myself - I know it's often hard to predict with these sorts of things - though was wondering if i might trouble you guys for an eta of a potential fix to this issue? We're using hammer for quite alot of touch based web experiments, and this bug is causing some quite annoying interactivity glitches... :P

Early September is currently the target for the first patch release post maintainer changes.  Anything with will-patch will be targeted for that.  We had a lot of issue triage and roadmap considerations to go through to determine out path forward (We're trying to collaborate with other projects to ensure the state of gesture support across all projects improves a lot this year :) ).

Does Merging #824 potentially fix this too?

Any update on this? I'm on 2.0.4 and still seeing the issue.

This may have been a pointers.length related bug.

I add the css style 'backface-visibility: hidden;' to the el, and this fix the issue for me.

^ @runspired


[BEAM-7202] fix inventory jobs

Allow the job_Inventory_* running on new build workers.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.


Use native language detector, with angular-translate detect method, r…

Use native language detector, with angular-translate detect method, returned value was different according used browser, like en-EN on Edge and en on firefox or chrome

@amiceli please resolve conflict This branch has conflicts that must be resolved and I'll merge, thanks.

I manually pushed the commit to master 2742072f4162cc432d996137ef4d8916572778cc

ok thank you


Update protobuf-java to 3.15.7

Updates com.google.protobuf:protobuf-java from 3.15.6 to 3.15.7.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.protobuf", artifactId = "protobuf-java" } ]

labels: library-update, semver-patch

Superseded by #882.


Cannot initialize model

Hello!
I was running the examples found in the repository and I keep finding the same error when trying to initialize a model.
I noticed you recently updated the library to version 0.4.0, but I get no errors for 0.3.5.
For example, in 00_Quick_Start:
model = mogptk.CONV(data, Q=3)

output:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-81-e7b80f8f700a> in <cell line: 1>()
----> 1 model = mogptk.CONV(data, Q=3)

/usr/local/lib/python3.10/dist-packages/mogptk/models/conv.py in __init__(self, dataset, Q, inference, mean, name)
     55             kernel[q].base_variance.assign(np.random.rand(input_dims))
     56
---> 57         super().__init__(dataset, kernel, inference, mean, name)
     58         self.Q = Q
     59

TypeError: Model.__init__() takes from 3 to 5 positional arguments but 6 were given

Thank you!

Please checkout v0.5.0, that should fix your issues! You can re-open or raise a new issue if you encounter other problems, I'm happy to help


Install timezone resources.

Where is the code which reads this data_time_zonespec.csv file ?

See the attached PR.


user's table is detached from the db name and is static

I'm using a db called tester.db, but the table is called experiments, I also can't change it.
At a minimum, we should document this.
We should also allow the user to configure its name/assign the same name as the db (in case it's a one table use case).

this should be opened in sklearn-evaluation


Support for trigger scripts to be passed the rms value as an argument

Added basic support for rms to be passed as argument with trigger script. Example trigger.sh 1234
Added config variable for rms as argument, allowing users to turn it on or off. Default is off to support current uses.
Added README and config info

Thanks for your PR. I'll take a look once I have time.

No sweat. Was just adding some functionality that I really wanted.


Storybook fails to build from InterpolateHtmlPlugin error [alpha.15]

If you are reporting a bug or requesting support, start here:
Bug or support request summary
I tried to upgrade from alpha.12 to alpha.15, and now when I perform a build-storybook, I'm getting the error:
ERR! Failed to build the storybook
ERR!   TypeError: callback is not a function
ERR!
ERR!   - InterpolateHtmlPlugin.js:37 compilation.plugin
ERR!     [web]/[react-dev-utils]/InterpolateHtmlPlugin.js:37:11
.
.
.

Steps to reproduce
Update to alpha.15, perform a build-storybook.  That's all I did, but maybe there are other contributing factors?
Please specify which version of Storybook and optionally any affected addons that you're running
"@storybook/addon-actions": "4.0.0-alpha.15",
"@storybook/addon-info": "4.0.0-alpha.15",
"@storybook/addon-knobs": "4.0.0-alpha.15",
"@storybook/addon-options": "4.0.0-alpha.15",
"@storybook/react": "4.0.0-alpha.15",

Affected platforms
❯ yarn list --pattern react-dev-utils
yarn list v1.9.2
└─<EMAIL_ADDRESS>
It seems like maybe this could be related to https://github.com/storybooks/storybook/pull/3852.

Until the fix is released, I can suggest locking manually a right version of the react-dev-utils (3e165448) in your lock file.

Pinning the version to 3e165448 allows my project to build but then the command line hangs up after completion.
package.json:
{
  "resolutions": {
    "react-dev-utils": "6.0.0-next.3e165448"
  }
}

PS C:\project-path> yarn storybook:build
yarn run v1.9.4
$ build-storybook -c .storybook -s public -o dist/storybook
info @storybook/react v4.0.0-alpha.15
info
info => Loading custom addons config.
WARN => Custom configuration file C:\project-path\.storybook\webpack.config.js is detected
WARN    but impossible to import loader for .js
info => Loading custom webpack config (full-control mode).
info => Copying static files from: public
info Building storybook ...
info Building storybook completed.
Terminate batch job (Y/N)? y

4.0.0-alpha.14 continues to work fine for me.

Do you have a method so I can reproduce?
Either way that plugin is being replaced/removed:
https://github.com/storybooks/storybook/pull/3919

This change might also fix it: https://github.com/storybooks/storybook/pull/3919

@ndelangen No, only #3959 would fix it. WatchMissingNodeModulesPlugin also produces this issue.

Released a fix in 4.0.0-alpha.16


TypeError: Cannot read property 'kind' of undefined     at getErrorSpanForNode

Hello, i have a problem with error after build (start:dev command), where error "TypeError: Cannot read property 'kind' of undefined at getErrorSpanForNode (C:\work\rankseacom\node_modules\typescript\lib\typescript.js:13856:22)
" appears.
Anyone with the same problem?

+1

+1

+1
"next":  "9.4.4",
"ts-node": "8.10.2",
"typescript": "3.9.5",

Same issue for me here,
next: 9.4.4
typescript (tried all of those): 3.9.5, 3.9.4, 3.9.3, 3.9.2
The issue seems to appear after TS f version 3.8.3, I tried using 3.8.3 and it works great.

downgrade typescript to "typescript": "3.1.6"

If anyone is still running into this - upgrading to latest Typescript (4.1.2) also works.


Update translations from Fedora Zanata

Update translations from Fedora Zanata

[x] po-refresh

po-refresh in progress on centosci-tasks-f131p.
Log: https://logs-https-cockpit.apps.ci.centos.org/logs/po-refresh-731-20190716-071616/

po-refresh  done: https://github.com/cockpituous/welder-web/commits/po-refresh-20190716-072020

Codecov Report

Merging #731 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #731      +/-   ##
==========================================
- Coverage   78.64%   78.46%   -0.18%
==========================================
  Files          66       66
  Lines        3343     3343
  Branches      703      703
==========================================
- Hits         2629     2623       -6
- Misses        714      720       +6

Impacted Files
Coverage Δ

components/Form/Password.js
85.71% <0%> (-2.39%)
:arrow_down:

components/Notifications/Notification.js
71.42% <0%> (-1.59%)
:arrow_down:

components/Modal/CreateBlueprint.js
94.68% <0%> (-1.07%)
:arrow_down:

core/sagas/inputs.js
71.42% <0%> (-1.03%)
:arrow_down:

core/sagas/blueprints.js
73.57% <0%> (-0.52%)
:arrow_down:

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 3feefb5...f85fd0c. Read the comment docs.

@larskarlitski updated the string, lets close this and let update zanata once more.


Direct Connect Protocol

port article from wiki https://wiki.altv.mp/wiki/Scripting:Direct_Connect_Protocol to docs

Added https://github.com/altmp/altv-docs/commit/ef6fae89c24919b7c13248c5985ece8e9254e7b1


Fatal error when getting channel detail

Hi, can you please help me with the following errors i m facing right now when trying to fetch channel details.
1- The user is not enabled for live streaming
Also i enabled live streaming option but then i was struck in this error
2- The user is blocked from live streaming.
Regards.

@adeel2188, it not the issue with the package seems youtube has put some restrictions on live streaming.
you can check if you are violating any of these restrictions

https://support.google.com/youtube/answer/2853834?hl=en


Adds extender strategy to case study yaml files

This PR adds the extender strategy and, if available, the sampling method that were used to create/extend each stage of a case study to the case study yaml file format.
If the same stage is extended multiple times with different strategies, the result is mixed as the revisions in the stage cannot be mapped to the different extender strategies any more.
Also, if the same stage is extended by the distribution sampler more than once, the result is always mixed, even if the same distribution was used. This is because the set union of two samples sampled from the same population with the same distribution does in general not result in a valid sample of that distribution.

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- varats/paper/case_study.py  2

See the complete overview on Codacy

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- varats/paper/case_study.py  2

See the complete overview on Codacy