Derivatives (wrt parameters and input values)

Hi,
to be able to use this kernel module in my Gaussian Process regression code - instead of rolling my own covariance kernels, which do basically the same, just not as fancy as your code - I will need the derivatives of kernels, both with respect to the parameters (for optimizing log marginal likelihoods) and with respect to input values (for training from and prediction of derivatives of function values, not just the function values themselves). What are your thoughts (if any) on the interface for that ?
Have a look at https://github.com/st--/juliagp/blob/master/covariances.jl for how I've implemented it so far...
-ST

Hey
Sorry... I've been busy with work and I'm not that familiar with Github, so this totally passed under the radar!
Ahhhhh, calculus! Noooo! I'm meltinggggg!
Kidding, I think this sounds like a great feature.
I'm not currently familiar with Gaussian processes. I'm assuming your covariance functions just positive definite kernel functions? Let's say you have a kernel k(x,y;a) where x and y are vectors (data) and a is a kernel parameter. You'll need something along these lines:

The first derivative of k wrt a (dk/da) which will be a function of x,y and will return a scalar
The first partial derivative of k wrt x or y (dk/dx) which will be a function x and y and will return a vector
The second partial derivative of k wrt x or y (dk^2/dx^2) which will be a function x and y and will return a square matrix
The mixed partial derivative of k wrt x and y which will be a function of x and y and will return a matrix

Do I have that correct for the most part? I played around with your code and it looks like you have the first two defined for vectors and the latter two defined for scalars.
As for implementation, I like what you've started. I'd use x and y rather than strictly relying on positional use of x or x.
kernel_function is kind of verbose. Let's say it's changed to 'kernel'
I'm thinking it would make sense to have a parameters() function for each kernel that will return an ordered list of symbols (for each parameter). Users can specify which parameter to take the derivative with respect to using the index or the parameter.
Something like dkernel_dp(K, :a, x, y) or dkernel_dp(K, 1, x, y)
Then the others (for input values) are like:
dkernel_dx (dkernel_dy)
d2kernel_d2x (d2kernel_d2y)
d2kernel_dxdy
What are your thoughts?

Yes, pretty much spot on! Yes, Gaussian Processes generally need posdef kernels (you need to invert the covariance matrix). Except I don't think I actually need the second non-mixed derivatives.
In my implementation, each covariance kernel only acts on a single piece of data - so a scalar if it's a 1D problem, or a vector of e.g. the position if it's multi-dimensional. Then covmat() iterates over the different points to construct the covariance matrix. The second mixed derivative was for prototyping around on a 1D problem in an IJulia notebook, so it's not yet used in any of the code, which is why it might not be as fleshed out / not working for vectors yet...
Your suggestions for how to construct the interface seem very sensible!
The only thing slightly more complicated are composite kernels - if you have a standard kernel, then the symbolic names for parameters are unambiguous, but what about, e.g., 5GaussianKernel()+3GaussianKernel()? how to access the different scaling factors, and the parameters of the two instances of the Gaussian kernel?
Regarding the implementation, the derivatives very often use what's effectively the original kernel - particularly anything with an exp() in it! And in Gaussian Processes you often want to construct a covariance matrix between two lists of positions (so in data storage terms two two-dimensional arrays), block-diagonal as follows: top-left kernel(x,y), top-right kernel_dx(x,y), bottom-left kernel_dy(x,y) [or vice versa, I never remember], bottom-right kernel_dxdy(x,y)... so it would be nice if we could get the code in such a way that it does the minimum amount of calculations needed!

Thought on composite kernels: Just use dot notation? E.g. for my example the symbols would be :a1, :k1.eta, :a2, :k2.eta. For something more complicated you might end up with :k1.k2.k2.c ... but I don't think there's any way around that.
NB for back-end code it obviously makes it look a lot neater to use Unicode symbols as you did, but anything exposed to the user - as these symbols - shouldn't require anything beyond ASCII...

I'm definitely targeting speed with this package. Do you have a good source for a description on fitting a Gaussian process? If your positions are p dimensional and you have n observations in one set and m in another set, then you're looking at calculating a (p+1)(p+1) dimension matrix for each pair (mn times).
Do you calculate this all at once? Or do you work with one block at a time and move onto the next one? I would imagine the former is memory intensive.
The Unicode characters are pretty gimmicky to be honest. You bring up a good point, I'm going to change all the parameters in the types back to ASCII but leave the Unicode in the printed output such as the descriptions. That way, the symbols used for accessing parameters will be strictly ASCII. I'm in complete support of the dot notation.
My intention with the composite kernels was intended as more of an exploratory, "cool" feature to try out more exotic kernels, so I'm less concerned about their functionality/performance at this point in time. I will definitely try to accommodate them, but for now I think it's best to get something working for the regular kernels.
I'm going to fix up those kernels now and finish making some changes to the kernel matrix calculations.

Related question: right now I have 'kernel_function' and 'kernel_matrix'.
They both accept array arguments. However, 'kernel_function' assumes the arrays are vectors and 'kernel_matrix' assumes they are observation/data matrices and operates on the rows/columns to generate the matrix.
Do you think I should condense this to one function "kernel" and do multiple dispatch? IE if arrays, then return kernel matrix. If vectors, kernel function that returns a scalar. My hesitation would just be that these are used very differently, so I would want to keep them sufficiently separate. Further, if I rely on multiple dispatch, you HAVE to wrap any column or row from a matrix with vec().
IE kernel(vec(A[1,:]), vec(A[2,:])) versus kernel_function(A[1,:], A[2,:])
Is 'kernel_function' too long? Especially if we're going to have derivatives. 'dkernel_function_dx'. Any ideas? 'kf' and 'km'? 'kernel' and 'kernel_matrix'?

The first two chapters of http://www.gaussianprocess.org/gpml/chapters/ are a pretty good explanation into Gaussian Processes for Regression (which is what I use them for). TBH, I've learnt the most by implementing them and playing around with how it works.
A lot of that did involve composite kernels, and on http://www-ai.cs.uni-dortmund.de/weblab/static/api_docs/pyGPs/demoMaunaLoa.html you can find a run-through in a python package of what seems to be the canonical 1D example for GP regression. I'm not too bothered about speed of composite kernels at this point, but I think it's worth making available all the functionality of derivatives for them. (I don't think this is a very complicated issue.)
Generally you can split GP regression into "teaching" on points x (with observed function values and/or derivatives y, y') and "prediction" on points xs; the first involves the covariance matrix C(x,x) and calculating its inverse (or its Cholesky decomposition); the second involves the covariance matrix C(x,xs). As you need the inverse in principle you need the whole at once, and you're right, for many dimensions and/or large data sets that definitely becomes an issue (though there's various approaches to speed that up and reduce memory footprint, perhaps trading off accuracy - such as sparsification - but in the first instance I'd focus on a solid implementation of basic GP regression, because that's what I'm playing with. For the big datasets I'm using our group's fortran code anyway. I'm glad we've got a compute node with 1.5 TB of RAM, but even that can still be limiting...).
I think it'd be better to keep kernel[_function] and kernel_matrix separate: you might want to calculate the point kernel of two vectors, or the kernel matrix of lists of 1D points - in either case it'd be two one-dimensional array inputs, but it's entirely separate things, in the first case returning a scalar, and in the second case a matrix - so I think it's worth making the distinction clear.
I like kernel() and kernel_matrix(). They clearly say what they do without being overly verbose. dkernel_dx, dkernel_dp are also nicely self-explanatory.

Thanks for the resource! I'll definitely be checking that out.
I took a look at your first cut of the kernel derivatives. Abstracting the kernel function away from the dot product has worked out nicely in calculating the derivatives. It should be easy to apply this strategy to the rest of the kernels that are functions of a norm. The scalar product kernels should also be simple. The separable kernel will be a little more interesting...
I didn't have much time today, so I only made some aesthetic changes. I renamed euclidean_distance to norm2 and kernelize_scalar to kernelize to shorten things up a bit. Then I renamed a few of the derivative functions for consistency with the naming scheme mentioned above. Any thoughts/objections?

I had wondered about kernelize[_scalar] and what the name means - still not sure about it, maybe there's a better name for it, but in the end here it doesn't matter because this shouldn't be exposed to users; it's a purely internal method.
Agreed that euclidean_distance is a bit of a mouthful, and dnorm2_dx sure is nicer. However, 'norm2' sounds like it'd be the 2-norm [i.e. norm(x,2) == sqrt(dot(x,x))] rather than the square of the 2-norm [norm(x,2)^2 == dot(x,x), which is what we actually use(*)]. Would you be fine renaming it to sq[uare]norm? Then also the second derivative can be called d2kernelize_dsqnorm2, in line with the mathematical notation: ∂²f/∂x²...
An alternative would be to move to the actual norm rather than epsilon'epsilon for the argument to kernelize() for EuclideanDistanceKernels, but maybe best to leave it as is for now.
(*) To be honest I think this is thinking about premature optimisation; in the end it probably isn't the bottleneck, but all the same I was wondering if it wouldn't be more efficient to do something like (ϵ=x-y;dot(ϵ,ϵ)) instead of norm(x-y)^2 ... on the other hand, Base.generic_vecnorm2() [in linalg/generic.jl] which is what gets called eventually seems to do some scaling to minimize floating point errors or somesuch. Hum.

I am in no way attached to the names/labels of anything. If you have alternatives, please do make a suggestion.
Also, I have not used Leibniz notation in forever... I totally switched up the order in the differential... I thought it was ∂²x for some reason.
I'll just give you a quick overview of how things were intended to work and the story behind kernelize. Scanning through a data matrix and computing the covariance function/kernels gets really slow. However, I saw that the majority of kernels that I had come across were either an element-wise function of the scalar product or squared euclidean distance. I could compute the gramian quickly using BLAS. Similarly, transforming the gramian so that each ij element represented the norm squared of the lag (or error) between two vectors xi and xj was straight forward. This form the gramian I named "lagged_gramian". Separable kernels are similar story where the transformation is element wise on the vectors themselves (not their dot product or euclidean distance).
 If you have a better descriptor for this function, let me know.
As for the dots/norm calculation. I haven't had a chance to look too deeply into them... up until now I was just concerned about calculating the kernel matrix for something else I was working on. Element wise, the gramian returned by BLAS is a squared norm or dot product which was why I initially designed it to take in a dot(ϵ,ϵ) or dot(x,y). I would prefer to continue to use something like ϵ=x-y;dot(ϵ,ϵ).
Long story short, I'd like to stick with squared norm but I'm indifferent on how it's defined/named.
function sqnorm(x,y)
ϵ=x-y;
dot(ϵ,ϵ)
end
Does that sound good?

It does, and thanks for the explanation! The one thing I'm slightly confused by is you saying that the kernelize() function returns a matrix, because that's the original kernelize_scalar, and it takes the epsilon'epsilon scalar and returns a scalar, not a matrix - or what did I get wrong there?
For teaching and predicting function values with a Gaussian Process all I need is the kernelmatrix() as implemented already. For teaching or predicting function derivatives what's also needed is the covariance matrix between function values and derivatives and/or between derivatives and derivatives.
If the input space is d-dimensional, and there are N points, then the total number of input values is Nd. kernelmatrix() returns an N by N matrix. I would also need (a) kernelmatrix_dx() which calculates dkernel_dx() for each combination of x and y, this is a vector of length d, and stitch them together into a Nd by N matrix and (b) kernelmatrix_dxdy() which calculates d2kernel_dxdy, a d by d matrix, and stitching those together results in the final Nd by Nd matrix. Thoughts on how to implement that in a fast way ?
If that helps I can just implement the naïve+slow implementation for reference...

OH, sorry! I wasn't clear.
Mathematically, there's a matrix valued function f that transforms a gramian into a kernel matrix.  Really all it does it apply another function (lets call it h) to each of the elements of G or Gl.
So K = f(G) = [h(g_11) h(g_12); h(g_21) h(g_22)] for a 2d gramian matrix where g_ij is the ith, jth element.
In my implementation:

f == kernel_matrix (or kernel if it's just two vectors)
h == kernelize

The above works for kernels that are a function of the dot product or squared norm. For separable kernels, the kernelize function is applied to the data matrix. Then the gramian of the 'kernelized' data matrix is calculated.
As for the derivatives, I was going to approach it the same way you were. Let's say you those derivative matrices are denoted dKij.
Do you need it in this form: [dK11, dK12, dK13, ... , dKnn]? IE a bunch of matrix blocks like this image
Because if not, I think it would be beneficial to lay these matrices out in contiguous blocks of memory. You'd just iterate through your observations and calculate each matrix.  Since you know the size in advance, you could just reinterpret the block as a certain size. Basically you would have a (dd)(N*N) matrix. Each column would correspond to an entire d2kernel_dxdy matrix.
That's just my first thought, though.
Sorry I haven't been able to add much the last couple days - I'm busy with work stuff.

Hi both, do you think it's possible to get the derivatives that you require using automatic differentiation?
http://www.juliadiff.org/

What is the current status of derivatives? I see that there is a derivatives branch that seems to implement the functionality as discussed above. However I get the following error when I tried to use it:
ERROR: LoadError: LoadError: UndefVarError: FloatingPoint not defined
 in eval_user_input(::Any, ::Base.REPL.REPLBackend) at ./REPL.jl:64
 in macro expansion at ./REPL.jl:95 [inlined]
 in (::Base.REPL.##3#4{Base.REPL.REPLBackend})() at ./event.jl:68
while loading .../.julia/v0.5/MLKernels/src/auxfunctions.jl, in expression starting on line 32
while loading .../.julia/v0.5/MLKernels/src/MLKernels.jl, in expression starting on line 57

The branch is also lagging the main branch by quite a few commits. What needs to be done to make it work?

Closed in favour of #46


feat: go twilio

Sample Go Application with Twilio

@rishavmehra Please sign the DCO

@khareyash05  The DCO has been signed


Keyboard shortcut for toggling "Awake"

Description of the new feature / enhancement
Add an option for a keyboarďshortcut to toggle awake.
Scenario when this would be used?
Awake
Supporting information
No response

/dup #31451


OpenTimestamps client requires virtual environment to run

Noob here. Hello all.
Have got through the guide and have just completed IBD.
Went to install the OpenTimestamps client as per instruction here – https://raspibolt.org/guide/bitcoin/bitcoin-client.html#opentimestamps-client
Python install seemed to go correctly with large output. Let me know if this is necessary for debug.
Installing OpenTimestamp-client results in the following:
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.

    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.

    For more information visit http://rptl.io/venv

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.

Successfully installed it into a virtual environment.
Is this the right way to go about this? Do I then donwload the updated core version and checksum files into the (env) and run them from there? Doesn't seem right but can't work out the way.
On a Raspi 4B with 8GB RAM and 2TB SSD, accessed via Terminal on iMac. Have updated to 25.1.
No LSB modules are available.
Description:	Debian GNU/Linux 12 (bookworm)
Environment

Hardware platform: [e.g. Raspberry Pi 4, Odroid, ... ]
Operating system: [e.g, Raspberry Pi OS 64bit, Debian 11, ... ]
Version: [ output of lsb_release -d ]

Did you use Raspberry Pi OS (bullseye) debian 11 based? Or something different?

Raspibolt use debian 11 bullseye version, it has not been updated yet to the debian 12 bookworm version, you should use the bullseye version


Improve edge labeling

Suggestion
Is a suggestion as I'm not sure if we want to have this kind of default style that overrides cytoscape patterns.
Seeing this amazing example by @pwrose:

That can be tested here, I thought we could improved the default way to show labels.
Currently is hard to read.
How to
We have a style.json file and I believe the changes should happen there.
Whoever take this issue feel free to be creative regarding on how to show the labels :) the general goal is to improve readability.

A quick google search showed me results where labels are not on top of edges. I'm not sure what's the best default behavior tbh.

A quick google search showed me results where labels are not on top of edges. I'm not sure what's the best default behavior tbh.


details about the first milestone

connects #46

Hi @bennage, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (christb). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

This connects to #51


Optimise constant f-strings and constant builtin method calls

I actually started off optimising constant builtin method calls here and ended up doing a lot of additional optimisations for f-strings. Especially the expression deduplication seems a very neat improvement, although there's a tiny risk of diverting from strict Python semantics. It's difficult to run into for users, though.

One nice side-effect (although probably less helpful in practice) is that the test_fstring.py CPython test runs visibly faster because it generates long fstring sequences that now lead to much shorter C code. :-]


gles: Remove outdated recipes and patches

Remove recipes for GLES 1.7, 1.9 and 1.10.
Remove patches used by removed recipes.
Signed-off-by: Ruslan Shymkevych<EMAIL_ADDRESS>

Acked-by<EMAIL_ADDRESS>

Reviewers are added.
PR is tested by build for h3ulcb-4x2g-kf.
@iusyk Could you please merge?


feat(caip): add bnb to thorPoolIdAssetIdSymbolMap and export it

Current dependencies on/for this PR:

main

PR #1154   👈

This comment was auto-generated by Graphite.


Update Checkout configuration with default delivery mode

Similar to enabling the feature itself, provide some way for seller to configure default delivery
For now we can't get or save delivery config in backend.

By default, or if not configured, use free or least expensive delivery option
Configuration options: free option, least expensive non-free option, most expensive option, OR
specify code of delivery option to use by default (however if this doesn't exist, default to our default)

@Xymmer
What is the difference between free and least expensive?
If we have config 'free' and there is no free option should it use the least expensive or do something else?
What if we config 'least expensive' and there will be a free option too?

I see three options:

free
least cost (cheapest price for shipping that still costs money)
most cost

We can allow for set list of preferences, the first match will be chosen.
defaultDeliveryMode: ['code1'],
// or
defaultDeliveryMode: [DeliveryModePreferences.FREE, 'code1', 'code2'],

Should we still support?
defaultDeliveryMode: 'code1',
// or
defaultDeliveryMode: DeliveryModePreferences.FREE,

After discuss with @marlass and @dunqan we decide to not support simplified setting here.
We do not support it anywhere, so if we want to change it, we should update all occurrences of this case.


Respect cargo profiles / enable debug info in release mode

I'm trying to get my workspace to compile with debug info in release mode.
My Cargo.toml has the following:
[profile.release]
debug = 1

however that seems to be ignored when using crate2nix.
Solving
The documentation of buildRustCrate suggests that extraRustcOpts can be used (source code), however transposing it to crate2nix doesn't work:
            buildRustCrateForPkgs = pkgs: pkgs.buildRustCrate.override {
              extraRustcOpts = "-C debuginfo=1";
            };

yields the following error:
       error: function 'anonymous lambda' called with unexpected argument 'extraRustcOpts'
       at /nix/store/zaza7mgggz4m5h6z18kajabf4wciaj47-source/pkgs/build-support/rust/build-rust-crate/default.nix:7:1

That is because extraRustOpts can't be specified at a global level:
https://github.com/NixOS/nixpkgs/blob/30d044f8235dbd03675bef6e2b59e1c8a94e2c51/pkgs/build-support/rust/build-rust-crate/default.nix#L7
It can only be specified at a crate level:
https://github.com/NixOS/nixpkgs/blob/30d044f8235dbd03675bef6e2b59e1c8a94e2c51/pkgs/build-support/rust/build-rust-crate/default.nix#L180
=> It looks like in order to be able to respect cargo profiles, or at least give an interface that allows users to easily override rust flags at the buildRustCrateForPkgs level, one would need to PR something into nixpkgs so that one can propagate arguments from there.
It looks like as a workaround, one may override the crates one by one through the defaultCrateOverrides mechanism, but that requires re-mentioning every crate in there.
It seems that this may also be a first step towards ~fixing #338.

Current workaround:
{ pkgs, lib, cargoNix }:

let
  defaultOrOverrides = pkgs.defaultCrateOverrides;

  # Enable debuginfo on all known crates. Unfortunately this has to be overridden on a per-crate basis,
  # and we can only do this by looking up crates list from cargoNix.internal.
  # The fact this is so complex is tracked by https://github.com/nix-community/crate2nix/issues/345
  withDebugInfo = defaultOrOverrides // (
    builtins.listToAttrs
      (builtins.map
        (crate:
          let
            crateName = crate.crateName;
            defaultOverride = defaultOrOverrides.${crateName} or (crate: { });
          in
          {
            name = crateName;
            value = crate: (
              let defaultOverrideApplied = defaultOverride crate;
              in
              defaultOverrideApplied // {
                extraRustcOpts = (defaultOverrideApplied.extraRustcOpts or [ ]) ++ [ "-C debuginfo=1" ];
                extraRustcOptsForBuildRs = (defaultOverrideApplied.extraRustcOptsForBuildRs or [ ]) ++ [ "-C debuginfo=1" ];
              }
            );
          }
        )
        (builtins.attrValues cargoNix.internal.crates))
  );
in
withDebugInfo


How do you generate the Figure 3 in the paper?

In the paper, Figure 3 provides intuitive differences between PointCLIP V1 and V2.
I wonder how do you generate and visualize this figure. In my impression, the paired depth map and text description are both vectors, how do you use them to make this figure?
Thanks.

Waiting for a reply...


EFCore Bulkextension's BulkInsertAsync not setting the id (identity field)

EFCore Bulkextension's BulkInsertAsync not setting the id (identity field).
we are using EFCore.BulkExtensions version 6.3.3 and as per this  link this issue is fixed in v5.3.7. Is it really fixed or am I missing anything here ? We are getting negative id in result variable the below code:
public async Task InsertAsync(List users)
{
BulkConfig bulkCOnfig = new BulkConfig()
{
SetOutputIdentity = true
};
await this.BulkInsertAsync(users, bulkCOnfig);
 var result = users.FirstOrDefault().Id;

}
But if I use BulkInsertOrUpdateAsync then I'm getting Ids all records except the last one, not sure why BulkInsertAsync not returning the ids.

Is it possible to update the MERGE operation to make use of OUTPUT clause to return the IDs of records inserted using BulkInsertAsync or BulkInsert ?


Fix LP formatting for Issue #36

Fix the formatting of the LP file reported in Issue #36.

Thanks for your contribution ! Published as v1.4.2


Titanium plate

Now titanium plate can't be made by IC or TE，do you want to add recipes of them？

I'll look into it.


Update to 0.4

Several latest changes introduce new features and change IIS Express process lifetime, so it's better to update the minor version number for the next release.
See #5 and #7.

Agreed, and done!


Stride api

https://developer.atlassian.com/cloud/stride/rest/

https://bitbucket.org/atlassian/go-stride/src/master/

Fixed by https://github.com/appscode/go-notify/pull/29

Either create Room Token:
https://developer.atlassian.com/cloud/stride/security/authentication/
or use CLIENT_ID and CLIENT_SECRET:
https://developer.atlassian.com/cloud/stride/security/authentication/


Update faq-customers.md - why no Entra branding for B2C

Added a question to FAQ - Why is Azure AD B2C not part of Entra ID/External ID?

Made changes to the doc after discussing with the Feature PMs  - Avni, Namita and Sasha.

@msmimart
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

emailed author

emailed author

#sign-off


repeated application of the same resource results in delete deployment

Describe the bug
When I updated the application resource of vela, I found that the deployment resource application was deleted by vela-core.
To Reproduce
apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: xxxx
  namespace: xxxx
spec:
  components:
    - name: xxxxxx
      type: webservice
      dependsOn: []
      properties:
        exposeType: NodePort
        image: xxxxxxx
        imagePullSecrets:
          - xxx-secret
        ports:
          - expose: true
            port: 80
      traits:
        - properties:
            domain: xxxxxxx
            class: nginx
            classInSpec: true
            http:
              /: 80
          type: gateway

apply application

$ kubectl apply -f demo.yaml

application.core.oam.dev/xxxx created

get pod

➜  ~ k get po -w
NAME                              READY   STATUS    RESTARTS   AGE
xxx-6cb857dc5f-nhnmt   0/1     Pending   0          0s
xxx-6cb857dc5f-nhnmt   0/1     Pending   0          0s
xxx-6cb857dc5f-nhnmt   0/1     ContainerCreating   0          0s
xxx-6cb857dc5f-nhnmt   0/1     ContainerCreating   0          3s
xxx-6cb857dc5f-nhnmt   1/1     Running             0          4s

Apply the same again yaml

kubectl apply -f demo.yaml

application.core.oam.dev/xxxx configured

get po

➜  ~ k get po -w
NAME                              READY   STATUS    RESTARTS   AGE
xxxx-6cb857dc5f-nhnmt   1/1     Terminating         0          8m35s
xxxx-6cb857dc5f-nhnmt   1/1     Terminating         0          8m36s
xxxx-6cb857dc5f-nhnmt   0/1     Terminating         0          8m37s

Expected behavior

Screenshots

KubeVela Version
vela-core : v1.3.2
Cluster information
Kubernetes Version [1.20.8]

This is quite strange. Is this app still reconciling? And the new Deployment haven't been created?

This is quite strange. Is this app still reconciling? And the new Deployment haven't been created?

yes， Very strange

You mentioned that the second time you apply the same application. If the app.spec are the same, kubectl report "app unchanged" instead of "app configured"
$ k apply -f app.yaml
application.core.oam.dev/app-4017 unchanged

Could you please check:

If the second app are the same with before.
What minimal properties and traits in application to reproduce.

The current file I did not do task changes

I'm try environment below. The second time when I apply the app.yaml kubectl just report "unchanged".

Same vela-core version

$ vela version
CLI Version: v1.3.5
Core Version: 1.3.2
GitRevision: git-cbed2b5
GolangVersion: go1.17.10

app.yaml like

apiVersion: core.oam.dev/v1beta1
kind: Application
metadata:
  name: app-4017
spec:
  components:
    - name: comp-1
      type: webservice
      properties:
        image: nginx
        ports:
          - expose: true
            port: 80
      traits:
        - type: gateway
          properties:
            domain: example.com
            class: traefik
            http:
              /: 80

If you are free, please check the what are the perperties are needed in minimal to reproduce the error. That will helps a lot for tracking the issue.

About Pod restart problem, it is duplicated with https://github.com/kubevela/kubevela/issues/3751. And it has been fixed here: https://github.com/kubevela/kubevela/pull/3758 which is included in v1.3.3. You can upgrade your vela-core and retry, see if problem still exists.
If upgrade don't solve it, feel free to post more information here.

@chivalryq Thank you！ I'll try the latest version first.


Fix/provide network restoring accounts

Try out this version of the Hiro Wallet - download extension builds.

In favour of #2194

Following the earlier conversation, we're going to close this PR for now, as it will result in some accounts resolving data key addresses, and thus hiding some existing addresses with balances.
@pradel @friedger sorry for the slow progress here, I didn't fully understand the impact of these changes. Though confident we can come up with a better solution, as discussed, whereby we figure out a way to transfer the names.


Lack of Information

It would be really nice to know what HTML tags are currently supported by the Label.

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

ID: f8a6273c-62b1-ac18-aab3-62f78bdbade8
Version Independent ID: 746bd180-ae67-313d-b949-538a508807b2
Content: Xamarin.Forms Label - Xamarin
Content Source: docs/xamarin-forms/user-interface/text/label.md
Product: xamarin
Technology: xamarin-forms
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and it's documentation can be found here.


To array and to sparse functions are ready for linearoperator

The default toarray and tosparse functions have been added to linearoperator.
Also, parallel tests for these functions have been written.

@mateomarin97  We see that you have made changes to the code after Elena's questions, but it is not clear what exactly they are for. Could you please answer explicitly?

@mateomarin97 We see that you have made changes to the code after Elena's questions, but it is not clear what exactly they are for. Could you please answer explicitly?

Hello. The changes were just aesthetic. I made the code more readable by implementing the suggestions Elena mentioned. The only thing stopping this function from being ready is that for the sparse matrix case, I could not write a Pyccel kernel to replace the for loop that appends the new matrices entries since Pyccel did not support Python lists.
As the code stands it works but it is too slow due to this problem.

@mateomarin97 We see that you have made changes to the code after Elena's questions, but it is not clear what exactly they are for. Could you please answer explicitly?

Hello. The changes were just aesthetic. I made the code more readable by implementing the suggestions Elena mentioned. The only thing stopping this function from being ready is that for the sparse matrix case, I could not write a Pyccel kernel to replace the for loop that appends the new matrices entries since Pyccel did not support Python lists.
As the code stands it works but it is too slow due to this problem.

Now that I think about it I am fairly certain I have a newer version of this branch that I did not push. I will push it later.


Feature/new opportunities page

What do you think about collapsing the borders on the desktop filters list (not the drop down)?   Does this look a bit better?
Collapsed

Not collapsed


feat(sink): support columns subset for cassandra and doris sink

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Fixes #16633

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
we can add partial_columns  in doris sink with option.
like
create sink with(doris.partial_columns = 'true')
https://doris.apache.org/zh-CN/docs/2.0/data-operate/update/update-of-unique-model/

@xxhZs Any updates?


Does this lib support jwks_uri?

Hello,
For cases where the authorization server rotates the JWT keys (e.g. Okta), does this lib support fetching keys from a jwks_uri?

Hey, please bring these issues to the GitHub Discussions of this project.
But yes, refreshing tokens shouldn't be an issue. You can manage this process in the token() callback similar to the way it's described in the NextAuth.js documentation.


use of xlsx

Couldn't we stop using xlsx, rather have each sheet in separate csv file and if we need to tight them together put them in the same folder?
It makes it much more difficult to merge master into a branch if there are changes in an xlsx file from both sides.

Thanks, Eva - that's what Emmanuel was discussing at today's meeting. See #1337 and PR #1432

Ahh, brilliant, thanks!


MAISTRA-1263 add replaces and olm.skipRange to csv files

Signed-off-by: rcernich<EMAIL_ADDRESS>

@rcernich
Will it be 1.0.9 replaced or 1.0.10?

@gbaufake, it will, once 1.0.10 is released, or once the show up in the maistra-1.0 branch.
Which reminds me, I forgot to update the cni version for 1.0.

Note, the changes to the servicemesh-operator.yaml file were made to minimize the differences between it and the maistra-operator.yaml file (so the only differences are the image names).

/retest


fix(reactivity): mutating a readonly ref nested in a reactive object should fail. (close #5042)

When we have a readonly object nested in another reactive object, attempting to write to a property of that readonly object fails, as expected.
const state = reactive({
  subState: readonly(reactive({ msg: 'Hello' }))
})
state.subState.msg = 'Bye' // => fails as `subState` is readonly

But if we have a readonly ref nested in a reactive object, it successfully writes to it.
const state = reactive({
  msg: readonly(ref('Hello' ))
})
state.msg = 'Bye' // => succeeds even though `msg` is a readonly ref.

This PR fixes that edge case.
Open questions

Is the behavior introduced by this PR the desired one?
Is this a "risky fix" as it may break code that works in current user projects?

fix: #5042

Such cases are extremely rare, you can expect no one will rely on it, so this change won't cause problems.
For me, I hope readonly is highest priority, because I don't want to memorize summary table.


Global setting for excluded folders.

Thank you for your good work in this plugin!
I wonder if the user can type down the excluded folders that the file cleaning does not touch.

Thank you for your feedback.
It's a pretty nice feature, unfortunately not yet, but I'll consider adding it.

The latest version supports "Excluded Folders".


fix unknown expression "x=?y"

fixes #21
I'm not confident this is the correct way to do this but the tests seem fine?
(it's a bit weird that test_compare/output_tokens.p8 now has two trailing newlines -- a separate bug to do with the recent expression joining maybe..?)

hm, on this commit, this input:
local a=?'x'
local b=2

gets minified to
local l,l=?"x"
,2

I'm surprised to say that the newline is actually fine here, but the repeated use of L as a variable name seems incorrect.
edit: ah actually it seems that shrinko8 is allowed to do this because neither variable ends up being used? weird! neat!

Thanks!
I'm taking the commit and - on top of it - changing print to be treated as a regular call in the AST (as that's what it is for all intents and purposes - it's even varargs - e.g. w,h=?"stuff").


Support Edit Order

Coinbase API allows editing limit orders that have time-in-force of good-until-cancelled.
https://docs.cloud.coinbase.com/advanced-trade-api/reference/retailbrokerageapi_editorder

I took a shot at implementing this. See https://github.com/KmiQ/coinbase-advanced-python/pull/54

Follow Up PR here @jonandtice : https://github.com/KmiQ/coinbase-advanced-python/pull/55


Stop using Docker and using Ubuntu 14.04 for this in CircleCI

This makes the test faster than current. The purpose of PR is just for notice.
Before merging, we have to change the setting in CircleCI.
After merging, further commits for existing PRs must rebase in order to pass the test.

Nice, when this is in, we should check if the  electron builder pr stops with the timeouts

The problem should be gone. The point was wine's configuration.
wineboot --init is executed when using electron-builder to setup wine environment, but it seems to block the process if there is X11 display (probably due to GUI). So I used unset DISPLAY && wineboot --init before using electron-builder.

Sounds promising, thank you.
On Sun, Sep 25, 2016, 13:13 Yuya Ochiai<EMAIL_ADDRESS>wrote:

The problem should be gone. The point was wine's configuration.
wineboot --init is executed when using electron-builder to setup wine
environment, but it seems to block the process if there is X11 display
(probably due to GUI). So I used unset DISPLAY && wineboot --init before
using electron-builder.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mattermost/desktop/pull/310#issuecomment-249415911,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFqyZEiUGS70yVDvBGTwTp1LaoLf9O--ks5qtldRgaJpZM4KEGIT
.


Polygonid : Credential not found, see slack channel

https://talao.slack.com/archives/C04QYN4UGSK/p1689364128043509

Issue not found...


server console

how to change the encoding of the console from cp866 to UTF-8??

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

You can't do that successfully in windows as far as I know, I've tried to do this once before, and the server will just freeze when it displays a unicode symbol. The basic Windows cmd console doesn't really support unicode at all, powershell does to some degree, but that isn't used in the server when it runs.

@cartman-2000 so could (in theory) nelson rewrite the server to support powershell?...

No. I could change the output of the output stream of the application but you would have to write a wrapper script for powershell. This is a windows limitation so i won't change it.


bump

bump to 13.8.1

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.25%. Comparing base (9c74f03) to head (574038d).
Report is 7 commits behind head on master.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #3485   +/-   ##
=======================================
  Coverage   98.25%   98.25%
=======================================
  Files          74       74
  Lines        8072     8072
=======================================
  Hits         7931     7931
  Misses        141      141

Flag
Coverage Δ

unittests
98.25% <ø> (ø)

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.


StageGL+Adobe Animate future.

Is StageGL expected to be used for "WebGL" publishing from Adobe Animate? Or if not, will it be possible to use it for Animate-based projects? With what limitations?

The idea with StageGL is you can swap the stage that is used. It is not clear at this time if Adobe will leverage it directly, but it will be really easy to swap stages to activate WebGL.
Note that it will work with any supported content, which includes Bitmaps, Sprite Sheets, and caches. Vectors and Text are not directly supported (must be cached first).
If your content is mostly bitmaps/images, or you are willing to do some work to properly cache Animate content, then it could be used.

In my testing, StageGL is not compatible with the way Animate outputs hi-dpi support for createjs. It ends up rending things offset by half the width/height of the stage.


Add license

Could you please add an MIT / BSD license?

Yes, we already have a BSD license!
https://github.com/psg-mit/micromarshall/blob/master/LICENSE

Oops, I think I was looking in the wrong place, sorry.
Thanks!


Add languages/ko.yml (korean version)

Hi! Thank you for the wonderful theme. I added new language file in Korean.

Thanks. :+1:


Refactor bindings generation

Nice! Can you verify whether we need a .gitkeep inside src/contract? Also, I think a clean command that deletes the contents of src/contract might make sense, for the user to call if they want.

forge is a little bit picky about the output binding directory. The output binding directory either has to not exist at all, or it has to be called with a --overwrite option. I suggest not to commit src/contract directory, thus no .gitkeep is necessary. So the crate would contain src/lib.rs, Cargo.toml and that's it. src/contract module will be created directory from the make bind command.
And yes, I'll add the clean action to Makefile.


CIS 3.2 Verify that docker.service file permissions are set to 644 or more restrictive

Set /etc/init.d/docker to 0644

Ran docker-bench and this isn't an issue


Wrong latitude for Eastern Hemisphere

Hi,
I'm in Stockholm, Sweden which is east of the the prime meridian. Using the fancy-asset-tracker/motion-tracker/main.ino code I've been getting GPS coordinates in the Atlantic ocean where I'm pretty sure I'm not. The fault seems to be that my longitude is negative where it should be positive (Eastern Hemisphere = positive longitude).
Looking thru the code I realise there's a hard coded - in front of longitude on row 214.
+ ",\"lon\":-"   + String(convertDegMinToDecDeg(GPS.longitude))
The best solution is probably to use the available GPS.lon value and add a minus sign if the value is 'W'. The same goes for the Latitude where there should be a minus sign if the GPS.latvalue is 'S'.
Cheers
Ruben

Thanks for reporting!  I think you're absolutely right, I'll fix this as soon as I get a chance, which should be next week.  Pull requests are of course welcome also if you like! :)

Thanks for a super quick reply.
I'm not git-savvy enough to pull of a pull request so I'll wait until you get the time.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

Coment

Coment


Mitigation for dependency confusion in pipreqs

Hi folks! I am a security researcher, and I believe I have found a way to perform a dependency confusion attack on pipreqs. This pull request aims to mitigate a larger portion of the impact, however, there are still some caveats left (will be discussed later).
Vulnerability description
Impact: Arbitrary code execution on all machines running the Python code with requirements.txt file that includes one of the vulnerable packages and was generated through pipreqs.
Difficulty: The exploit needs several prerequisites listed below.
tl;dr
Pipreq's remote dependency resolving mechanism (lines 447-449) can be abused to inject arbitrary packages into the final requirements.txt file.
There are three necessary conditions for triggering this behavior:

PyPI package name (later - pypi_package_name) and the names of its exported Python modules (later on exported_module_name) must differ.
exported_module_name:pypi_package_name mapping must be absent from the pipreq's hard-coded mapping file
exported_module_name must be available at PyPI as a package name

In-depth explanation
Let’s use the djangorestframework-simplejwt package as an example. It has over 1.3M monthly downloads, according to PyPIStats (https://pypistats.org/packages/djangorestframework-simplejwt). The package exports the rest_framework_simplejwt Python module.
Consider the following snippet of code from the above package's documentation.
from rest_framework_simplejwt.views import (
    TokenObtainPairView,
    TokenRefreshView,
)

urlpatterns = [
    ...
    path('api/token/', TokenObtainPairView.as_view(), name='token_obtain_pair'),
    path('api/token/refresh/', TokenRefreshView.as_view(), name='token_refresh'),
    ...
]

When pipreqs is run on the code above, the following things happen:

Pipreqs extracts the imported module name (rest_framework_simplejwt) and places it into the candidates variable at line 425.

Pipreqs tries to find the rest_framework_simplejwt value in the mapping file at line 429 through the get_pkg_names function, but it isn’t there, so the script assumes that the package name is rest_framework_simplejwt.

The script tries to find the rest_framework_simplejwt module in the exports of all locally installed Python packages at line 445 by default.
If the package djangorestframework-simplejwt is installed locally, the function will find it and assign it to the local variable.

If not, the function will return an empty array.

Pipreqs then compares the names inside candidates and local variables to populate a difference variable at line 447.
Since the candidates variable contains rest_framework_simplejwt, and the local variable is either empty or has the name of the PyPI package (djangorestframework-simplejwt), this comparison will always evaluate to True. Consequently, the code will simply copy the candidates entries into the difference variable.

Then get_imports_info function is triggered with the differencearray (['rest_framework_simplejwt']). It tries to find PyPI packages with the same names as in the passed array.
If a PyPI package name inside the difference array is missing from PyPI, the code will ignore it. However, if an attacker registers the name on PyPI, pipreqs will inject this malicious package to "requirements.txt". And, during the dependency installation, attacker-controlled code will be executed on the user's machine.

Proof of Concept
I've created a PoC using the rest_framework_simplejwt module mentioned above (https://pypi.org/project/rest-framework-simplejwt/).
If you run pipreqs on the code above, you will see that my package is injected into the requirements.txt:
pipreqs --print
djangorestframework_simplejwt==5.2.2 <--- legitimate package
rest_framework_simplejwt==0.0.2      <--- malicious package

INFO: Successfully output requirements

In fact, this behavior above is the source of a 3y.o. open issue (https://github.com/bndr/pipreqs/issues/218).
What can be done about this?
This pull request reworks the local package resolution. In particular:

get_locally_installed_packages function will now return local packages in a form of {'name':'package_name','version':'package_version','exports':['exported_module_1', 'exported_module_2', ...]}

get_import_local function will now search imports list entries in the exports and name fields (to account for pipreqs mapping) of the reworked get_locally_installed_packages function output.

init function will now compute the difference list entries (packages that are not found locally and have to be resolved remotely) accordingly to the changes made.

These 3 steps should improve the quality of the requirements.txt output for packages that are installed locally.
New pipreqs version's output for the same code above:
pipreqs --print
djangorestframework_simplejwt==5.2.2 <--- legitimate package only

INFO: Successfully output requirements

Unfortunately, there is still a fundamental issue with the Python packaging system that we cannot address in any way. get_imports_info function will anyway output a flawed requirements list if a correct package is not installed locally.
So, I added a warning message into the CLI output for users when using a remote resolution to check the list of the final requirements for the correct packages:
pipreqs --print
WARNING: Import named "rest_framework_simplejwt" not found locally. Trying to resolve it at the PyPI server.
WARNING: Import named "rest_framework_simplejwt" was resolved to "rest-framework-simplejwt:0.0.2" package (https://pypi.org/project/rest-framework-simplejwt/).
Please, verify manually the final list of requirements.txt to avoid possible dependency confusions.
rest_framework_simplejwt==0.0.2     <--- malicious package
INFO: Successfully output requirements

This bug is also a good reasoning to make --use-local flag a default option, and create a --use-remote flag to use a remote package name resolution. However, this change would violate one of the pipreqs' use cases, so I will leave this proposal for a public discussion.

Hey @adeadfed, I'm ready to merge your fix once the linting issues are fixed.
Cheers,
Vadim

Thanks @bndr! I've fixed the PEP8 linting, should be good now.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.25 :tada:

Comparison is base (f97d8b9) 86.71% compared to head (2103371) 86.97%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #364      +/-   ##
==========================================
+ Coverage   86.71%   86.97%   +0.25%
==========================================
  Files           2        2
  Lines         256      261       +5
==========================================
+ Hits          222      227       +5
  Misses         34       34

Impacted Files
Coverage Δ

pipreqs/pipreqs.py
86.82% <100.00%> (+0.26%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


SA1119 code fix needs to add whitespaces

At least if the code would become invalid otherwise. Example:
return(Tmp);

The code fix removes the parenthesis which results in
returnTmp;

grabbed.


URLSessionTask retain cycle and memory leak fix - all Linuxies

HI Apple,
I’ve noticed a memory leak using NSURLSessionTasks to fetch a URL in Swift 4 that wasn’t present in Swift 3.1.1. This is for Vanilla Linux as well as Android. This PR seems to resolve it.
There was a retain cycle between the URLSessionTask and its protocol handler resulting in it never being deinit’d and the internal data container never being freed up. This PR nils out the pointer to the protocol handler when the task completes, breaking the retain cycle and allowing both to deinit.
Fixes SR-5472

Sorry, I couldn’t resist fixing the many unnesssesary files created in /tmp for data requests. I don’t think this warranted another PR and really should be fixed since it was such a minor change to do so. Tested using a foundation test run.

Seems like we could leave it as the IUO, causing a crash if the protocol is ever unset in the other places (which seems like a precondition failure).

@swift-ci Please test

@parkera are you sure you want _protocol as a IUO? It effectively makes cancel, suspend, resume unusable now that _protocol can be nil if the request happens to have completed.

I'm not sure about the behavior on Darwin here, but that's what we should attempt to match.
The reason I was asking about removing the IUO here was that this leak fix also results in a behavior change to a silent failure if _protocol is nil, whereas as the IUO it would previously crash, right?

The leak fix doesn't change behaviour relative to darwin in itself. After the patch and using an IUO it does introduce a race condition where if you start a request but decide to cancel/suspend/resume after the request has already completed (and _protocol nil'd out) there will be a crash as opposed to just ignoring the cancel which is I image what the Darwin implementation would do.

If you wanted the precise startup behaviour you have make _protocol full optional as it can now be nil when a request has completed but make resume() as follows in case a protocol handle could not be found:
    /// Resume the task.
    ///
    /// - SeeAlso: `suspend()`
    open func resume() {
        workQueue.sync {
            self.suspendCount -= 1
            guard 0 <= self.suspendCount else { fatalError("Resuming a task that's not suspended. Calls to resume() / suspend() need to be matched.") }
            self.updateTaskState()
            if self.suspendCount == 0 {
                self.workQueue.async {
                    self._protocol!.startLoading()
                }
            }
        }
    }

Though I’m not sure why you’d want to do that. Better to add a check in the init().

Just checked. Darwin behaviour on resume for invalid protocol is call completion handler with error:
2017-09-01 19:54:15.646020+0100 zproto[13718:4235341] Fetch -1 nil Optional(Error Domain=NSURLErrorDomain Code=-1002 "unsupported URL" UserInfo={NSUnderlyingError=0x608000248ac0 {Error Domain=kCFErrorDomainCFNetwork Code=-1002 "(null)"}, NSErrorFailingURLStringKey=john://www.bbc.co.uk/news, NSErrorFailingURLKey=john://www.bbc.co.uk/news, NSLocalizedDescription=unsupported URL})

I think I’ve converged. Over to you @parkera

@swift-ci please test

The retain cycle was expected to be broken as done in this PR. Looks like we missed this during the refactoring in #968 The creation of temp files only when required (by download tasks) also looks good to me.

@swift-ci please test

@pushkarnk if you're happy with this, go ahead and merge

@swift-ci test and merge

@swift-ci please test and merge

Thanks @pushkarnk 👍 Did this fix make Swit-4.0?

No, it'll need a separate PR against swift-4.0-branch.

I can do a PR but Is it still open?

@parkera knows for sure. Even if 4.0 is closed though, I think it is quite possible there will be a 4.0.1 release soon after, cut from the same branch. There was a 3.0.1 and 3.0.2 last year.


slow start

VSCode Version: Code - Insiders 1.11.0-insider (4db944bc08e34fb15c5805927d9cb67470776606, 2017-03-02T07:04:47.508Z)
OS Version: Darwin x64 15.6.0
CPUs: Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz (4 x 2500)
Memory (System): 8.00GB (0.32GB free)
Memory (Process): 178.05MB working set (0.00MB peak, 123.60MB private, 95.11MB shared)
Load (avg): 2, 2, 2
VM: 0%
Initial Startup: yes
Screen Reader: no
Empty Workspace: no
Timings:

Component
Task
Time (ms)

main
start => app.isReady
138

main
app.isReady => window.loadUrl()
515

renderer
window.loadUrl() => begin to require(workbench.main.js)
571

renderer
require(workbench.main.js)
1978

renderer
-> of which require() node_modules
161

renderer
create extension host => extensions onReady()
1168

renderer
restore viewlet
456

renderer
restore editor view state
1523

renderer
overall workbench load
1945

main + renderer
start => extensions ready
4527

main + renderer
start => workbench ready
5178

:warning: Make sure to attach these files: :warning:
-/Users/klik/6832_main.cpuprofile.txt
-/Users/klik/6832_renderer.cpuprofile.txt

closing because profile-files are missing

6832_renderer.cpuprofile.txt
Uploading 6832_main.cpuprofile.txt…

@idkjs Sorry, the last file didn't finish uploading


Update Readme.txt

Sould update the psych txt

literally nobody reads this

why would you need this, nobody reads the readme.txt

no

nobody reads this + its used to credit the original game lmfao


HF_DATASETS_OFFLINE=1 didn't stop datasets.builder from downloading

Describe the bug
According to https://huggingface.co/docs/datasets/loading_datasets.html#loading-a-dataset-builder, setting HF_DATASETS_OFFLINE to 1 should make datasets to "run in full offline mode". It didn't work for me. At the very beginning, datasets still tried to download "custom data configuration" for JSON, despite I have run the program once and cached all data into the same --cache_dir.
"Downloading" is not an issue when running with local disk, but crashes often with cloud storage because (1) multiply GPU processes try to access the same file, AND (2) FileLocker fails to synchronize all processes, due to storage throttling. 99% of times, when the main process releases FileLocker, the file is not actually ready for access in cloud storage and thus triggers "FileNotFound" errors for all other processes. Well, another way to resolve the problem is to investigate super reliable cloud storage, but that's out of scope here.
Steps to reproduce the bug
export HF_DATASETS_OFFLINE=1
python run_clm.py --model_name_or_path=models/gpt-j-6B --train_file=trainpy.v2.train.json --validation_file=trainpy.v2.eval.json --cache_dir=datacache/trainpy.v2

Expected results
datasets should stop all "downloading" behavior but reuse the cached JSON configuration. I think the problem here is part of the cache directory path, "default-471372bed4b51b53", is randomly generated, and it could change if some parameters changed. And I didn't find a way to use a fixed path to ensure datasets to reuse cached data every time.
Actual results
The logging shows datasets are still downloading into "datacache/trainpy.v2/json/default-471372bed4b51b53/0.0.0/c2d554c3377ea79c7664b93dc65d0803b45e3279000f993c7bfd18937fd7f426".
12/16/2021 10:25:59 - WARNING - datasets.builder - Using custom data configuration default-471372bed4b51b53
12/16/2021 10:25:59 - INFO - datasets.builder - Generating dataset json (datacache/trainpy.v2/json/default-471372bed4b51b53/0.0.0/c2d554c3377ea79c7664b93dc65d0803b45e3279000f993c7bfd18937fd7f426)
Downloading and preparing dataset json/default to  datacache/trainpy.v2/json/default-471372bed4b51b53/0.0.0/c2d554c3377ea79c7664b93dc65d0803b45e3279000f993c7bfd18937fd7f426...
100%|██████████████████████████████████████████████████████████████████████████████████| 2/2 [00:00<00:00, 17623.13it/s]
12/16/2021 10:25:59 - INFO - datasets.utils.download_manager - Downloading took 0.0 min
12/16/2021 10:26:00 - INFO - datasets.utils.download_manager - Checksum Computation took 0.0 min
100%|███████████████████████████████████████████████████████████████████████████████████| 2/2 [00:00<00:00, 1206.99it/s]
12/16/2021 10:26:00 - INFO - datasets.utils.info_utils - Unable to verify checksums.
12/16/2021 10:26:00 - INFO - datasets.builder - Generating split train
12/16/2021 10:26:01 - INFO - datasets.builder - Generating split validation
12/16/2021 10:26:02 - INFO - datasets.utils.info_utils - Unable to verify splits sizes.
Dataset json downloaded and prepared to  datacache/trainpy.v2/json/default-471372bed4b51b53/0.0.0/c2d554c3377ea79c7664b93dc65d0803b45e3279000f993c7bfd18937fd7f426. Subsequent calls will reuse this data.
100%|█████████████████████████████████████████████████████████████████████████████████████| 2/2 [00:00<00:00, 53.54it/s]

Environment info

datasets version: 1.16.1
Platform: Linux
Python version:  3.8.10
PyArrow version: 6.0.1

Hi ! Indeed it says "downloading and preparing" but in your case it didn't need to download anything since you used local files (it would have thrown an error otherwise). I think we can improve the logging to make it clearer in this case

@lhoestq Thank you for explaining. I am sorry but I was not clear about my intention.  In other words, you can imagine that my storage has only read access but crashes on write access.
When run_clm.py is invoked with the same parameters, the hash in the cache directory "datacache/trainpy.v2/json/default-471372bed4b51b53/0.0.0/..." doesn't change, and my job can load cached data properly. This is great.
Unfortunately, when params change (which happens sometimes), the hash changes and the old cache is invalid. dataset would create a new cache directory with the new hash and create JSON builder there, even though every JSON builder is the same. I didn't find a way to avoid such behavior.
This problem can be resolved when using datasets.map() for tokenizing and grouping text. This function allows me to specify output filenames with --cache_file_names, so that the cached files are always valid (unless they are deleted).
This is the code that I used to fix cached filenames during tokenization. I wish I could do the same to datasets.load_dataset()
        tokenized_datasets = raw_datasets.map(
            tokenize_function,
            batched=True,
            num_proc=data_args.preprocessing_num_workers,
            remove_columns=column_names,
            load_from_cache_file=not data_args.overwrite_cache,
            desc="Running tokenizer on dataset",
            cache_file_names={k: os.path.join(model_args.cache_dir, f'{k}-tokenized') for k in raw_datasets},
        )

Hi ! load_dataset may re-generate your dataset if some parameters changed indeed. If you want to freeze a dataset loaded with load_dataset, I think the best solution is just to save it somewhere on your disk with .save_to_disk(my_dataset_dir) and reload it with load_from_disk(my_dataset_dir). This way you will be able to reload the dataset without having to run load_dataset


Fix module.json so it can be loaded on Foundry v11

I was helping someone on Discord manually install this module on Foundry v11. It needs some changes to the module.json file in order to work with v11. I can't say whether that's all that's necessary to make it work in v11, but at least you can install/enable the module with this change.

The authors property needs to be changed from an array of strings to array of objects. It should be:

  "authors": [
    {"name": "Benjaneer"}
  ],

The compatibility section lists the maximum version as "10", which obviously won't work with v11. Unless you know it won't work with a higher version, I'd use verified instead of maximum here.

  "compatibility": {
    "minimum": "10",
    "verified": "11"
  },

I'm still using v10, please let me know if that works for v11

Yes, the changes I suggested will still work with v10. It'll be compatible with both v10 and v11.

I mean that I made the changes, please validate.
(instead of verified 11 I removed the maximum, and updated the authors per your specification)

I was able to install and enable v1.2.2 of the module just fine on both my v10 and v11 setups. Thanks for fixing it so people can use it on v11.


With experimental-foreign-object-prototype, Java Map keys cannot be accessed from JS and Sets cannot loop

When enabling the experimental experimental-foreign-object-prototype flag, Java Maps behave oddly when querying their own properties from within Javascript and Java Sets do not allow looping over their content elements. The net result is that a Map's key set cannot be accessed by Javascript code.
Java Maps

Object.getOwnPropertyNames(map) lists the public properties of the Map implementation but not the keys of the map's actual content.
Similarly,  map.hasOwnProperty('get') and 'get' in map return true, but the same methods return false for any key of the map's content.

This behaviour is understandable from a technical viewpoint. However, it seems contrary to the intent of getOwnPropertyNames, in, etc. and will negatively affect the usability of the foreign object within Javascript code.
For example, the common JS pattern for(let key in Object.keys(map)) { ... } will not loop over the map's content as a JS developer would have good reason to expect. At this point, the developer has to interact with the object as a Java object (map.keySet()) and the idea of the foreign-object prototype seems lost.
A possibly unrelated issue is that map.propertyIsEnumerable(.) is unsupported.
Java Sets
However, here comes the next, even bigger gotcha: for(let key in map.keySet()) { ... } does again not loop over the map's content keys, but instead over the public methods of the Set implementation. Again, one can imagine the technical reasons for this behaviour.
Could one even work around this behaviour? How else would one loop over a Java Set within JS? Let's not even think about turning the set into an array and then looping over the array!
Mutatis mutandis, the same is true of map.entrySet(), bringing us to a final dead end.

M.

Thank you for taking time to report this issue. Basically all your expectations evolve around considering keys/elements of Java maps/sets as JavaScript (enumerable) own properties. Your code samples do not produce what you expect because they are not such properties. This is as designed and consistent with JavaScript Maps:
$ graalvm/bin/js
> var map = new Map(); map.set('key1', 'value1'); map.set('key2', 'value2');
Map(2){"key1" => "value1", "key2" => "value2"}
> Object.getOwnPropertyNames(map)
[]
> map.hasOwnProperty('key1')
false
> 'key1' in map
false

If you want to iterate over Java or JavaScript maps or sets then I suggest you to use for-of (not for-in statement): for (entry of map) print(entry[0] + '->' + entry[1]); resp. for (element of set) print(element);.

A possibly unrelated issue is that map.propertyIsEnumerable(.) is unsupported.

I agree that we can/should improve this method to handle Java (or other non-JavaScript objects) but this is an unrelated issue (as you metioned already). It would return false for map keys even if it worked for Java maps.

Uhhh! You are absolutely right, I should have used for-of instead of for-in. My mistake entirely and I am grateful for your decisive reply. I also had not understood that Java Map corresponds to JS Map and that then settles the matter.
However, for a Java Map map.keys() and map.entries() return empty sets. (In a feeble attempt for my defense, this is what set me off on the misguided exploration of a JS Object.) That's not not right for a JS Map, is it?

Uhh! You are absolutely right. for-in was incorrect and it makes perfect sense that a Java Map should correspond to a JS Map.
Apologies!!!
M.


Idea: remove dependence on VF Remoting javascript

I looked a little bit into VF Remoting, it's basically just EXTJS. Do you think it's worth the effort to remove VF Remoting and go with straight Angular $http? Regular Angular $http opens the door for interceptors and all the other Angular features when making calls back to SFDC. It also means we no longer have to include the VF component to get VfRemote.js included.

closing this issue due to inactivity


How heatmap works?

Hello,
I could not find a proper documentation for gmplot heatmap.
Is there a way, I can draw weighted heatmap?
For example: earthquake data with long, lat and intensity of earthquake.

@Fundoprajakta heatmaps show a concentration density, the more dense of points an area is the more colored is (the heat is higher). f.e. in http://sandrotosi.me/ingress/nyc/nyc_portals_heatmap.html the more read it is, the denser the area is (of points i'm interested in as part of this map), as you can zoom, you can see these areas tend to dilute.
to address your problem, since there's not a density involved, you may want to try and simulate it by using markers of different colors: red for higher h.q. intensity, blue for lower lever and so on

closing, feel free to reopen if you need more info (would be ideal to point to places where we can improve our doc)


Bump MSRV to 1.46.0

One of async-std's dependencies bumped their MSRV to 1.46, and tokio is going to do the same soon. 1.45 will continue to work with the sync version of the API

I think the required check thing has to be updated @amousset

I think the required check thing has to be updated @amousset

Indeed, done.


How can I get async behavior out of the Fluent SDK?

Hiya,
I'm creating VMs using the Fluent SDK.  Here's a code snippet:
            var vm = await this.azure

                .VirtualMachines

                .Define(vmName)

                .WithRegion(region)

                .WithExistingResourceGroup(resourceGroupName)

                .WithExistingPrimaryNetwork(network)

                .WithSubnet(subNetName)

                .WithPrimaryPrivateIPAddressDynamic()

                .WithoutPrimaryPublicIPAddress()

                .WithWindowsCustomImage(image)

                .WithAdminUsername(adminUserName)

                .WithAdminPassword(adminPassword)

                .WithSize(vmSize)

                .WithNewStorageAccount(storageAccountName)

                .WithTag(TagKeyNames.StorageAccountId, storageAccount.Id)

                .CreateAsync();

This task does not complete until the machine is entirely provisioned and booting - a period of over ten minutes.  That doesn't seem async to me.  Is there something I am doing wrong?  I want the call to return prior to the machine's availability.
For comparison, when provisioning a VM on Amazon EC2 returns nearly immediately, well before the machine is fully provisioned and booted.
I need from the call is the Id (vm.Id), the Primary Private IP (vm.GetPrimaryNetworkInterface().PrimaryPrivateIP), and the primary NIC Id (vm.PrimaryNetworkInterfaceId).  I do not need the machine to be ready.
Thanks!

@awesley we are working on providing callers a way to poll manually resource creation flow. Right now we do poll until resource provisioning status is succeeded or failed.

@hovsepm Understood.  I was able to add the multiThreaded: true arg to CreateAsync and reduce the time a bit, but my requestTimeout will still need to be extreme to manage this case (I'm using the SDK in ASP.Net Core).
My 0.02$ on desired behavior - the call to Azure should be quick, return the information that I described above, and that I could poll if I'd like to.
Due to the current behavior, if I spin up multiple VMs (conceptually, we could spin up 100 VMs at a time), I must introduce parallelization in my code -- which isn't a tremendous deal but an additional layer of complexity that I think could be hidden from API consumers such as myself.
Appreciate the response, feel free to close this Issue as you see fit.

@awesley did you consider to create a list of VM Creatables and then call Create on that list?
Manual polling is in the ToDo list for C# client runtime that we depend on. I should be also available in the future.

I have retry logic wrapping each VM creation request, so I prefer to have them as individual data structures.  That is a good suggestion though.  Thanks for the feedback!

@awesley We are looking for the ways to notify caller about resources that failed and succeed after Creatables.Create call.

My use case is to prepare a custom image which contains a self-announcing
service and request new VMs using that image.
Assuming that my request can be validated and that Azure returns the IDs of
all relevant resources (things created modified or otherwise used)
This ties back to polling as then I can long-poll those resource IDs as
needed (or some more sophisticated bi directional communication).
So really what I'm talking about here is to return the IDs rather than the
task object, and that I could query for each status or task object using
each ID.
In this way, consumers like me can choose to fire and forget, but if
desired could poll.
On Jul 6, 2017 2:34 PM, "Hovsep"<EMAIL_ADDRESS>wrote:

@awesley https://github.com/awesley We are looking for the ways to
notify caller about resources that failed and succeed after
Creatables.Create call.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Azure/azure-sdk-for-net/issues/3411#issuecomment-313481818,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADBMBXekwWOFqvtEY_XryyBf-mbWcKgjks5sLSjBgaJpZM4OD89_
.

@anuchandy another one related to the begin* pattern

This issue was moved to Azure/azure-libraries-for-java#8


[Demangler] Fix assertion failure.

It's illegal to call node->addChild() with a NULL child argument; it's possible to construct unexpected Node trees by passing invalid manglings, and in this case that was causing popTypeAndGetChild() to fail (because the top node was not a Type node), which then meant that the call to addChild had a NULL child argument.
The simplest fix is to use createWithChildren() to do the node construction, because that function checks its arguments for NULLs.
rdar://125350219

This is a cherry pick of #72653.

@swift-ci Please test


Malformed struct tag in api/http/handler/templates/template_file.go

The following struct tag is malformed (a blank space between : and the value) This might prevent to be correctly interpreted
https://github.com/portainer/portainer/blob/a0fa64781ae9eb1ea391d300ca9bf937e58c8dbc/api/http/handler/templates/template_file.go#L26

@chavacava
Thank you for the information.  I forwarded this to Engineering for review and an internal issue has been logged.
Thanks!

@chavacava
I wanted to follow up on this request.  The internal issue was resolved in 2.17.  Please re-open if needed.
Thanks!


clarification inline expect

We can make the "confusions" section collapsible with #6510 in the future.

Thanks @brian-carroll :)


Incompatible with non-git repositories

This is a rare case, especially when using GitHub, but there are options to use VCS other than git, e.g. Jujutsu. But gh-workflow relies on the git repository and git binary available here https://github.com/tailcallhq/gh-workflow/blob/11be6239df179680c65e778013044b3d09cb0a3d/src/generate.rs#L37-L42
Consider using other approaches to find proper root to put the workflow files. Maybe https://github.com/azzamsa/toor could be helpful

Should be a simple change. Feel free to raise a PR.

cargo metadata has the field: workspace_root, this could be of help, I think.
https://docs.rs/cargo_metadata/0.19.0/cargo_metadata/

Thanks @simonsan. Toor is designed just to find the root. If it's not a very large library we should go ahead.


Failed to install Wsl-IpHandler

Installation fails on Win11 via Terminal with:
Install-WslIpHandler -WslInstanceName "Ubuntu-22.04" -WslHostName ubuntu -GatewayIpAddress <IP_ADDRESS> -PrefixLength 25 -WslInstanceIpAddress <IP_ADDRESS> -DontModifyPsProfile -UseScheduledTaskOnUserLogOn
PowerShell installing Wsl-IpHandler to Ubuntu-22.04...
[sudo] password for admin:
bash: "C:UsersadminDocumentsPowerShellModulesWsl-IpHandlerScriptsBashinstall-wsl-iphandler.sh": No such file or directory
bash: line 1: ping -c1 windows 1>/dev/null && echo 1: No such file or directory
Test-WslInstallation on Ubuntu-22.04 Failed!
PowerShell finished installation of Wsl-IpHandler to Ubuntu-22.04 with Errors:
Pinging windows from WSL Instance: Ubuntu-22.04 failed:

Pinging ubuntu (WSL instance Ubuntu-22.04) from Windows failed.

Looks like escaping is broken somewhere as script trying to run script not via unix path
If I enter same command that fails in Debug, but in CMD:
wsl.exe -d Ubuntu-22.04 sudo -E env '"PATH=$PATH"' DEBUG=1 bash  $(wslpath "C:\Users\admin\Documents\PowerShell\Modules\Wsl-IpHandler\Scripts\Bash\install-wsl-iphandler.sh") "C:\Users\admin\Documents\PowerShell\Modules\Wsl-IpHandler\Scripts\Bash\wsl-iphandler.sh" /usr/local/bin "C:\Users\admin\Documents\PowerShell\Modules\Wsl-IpHandler\Scripts\Powershell\WSL-WinHostsEdit.ps1" /etc/wsl-iphandler.conf windows ubuntu <IP_ADDRESS>

It looks like do not fail.
At the end of debug output:
bash: line 1: ping -c1 windows 1>/dev/null && echo 1: No such file or directory
DEBUG: Test-WslInstallation [1663]: $wslTest:
DEBUG: Test-WslInstallation [1673]: Starting WSL instance Ubuntu-22.04 for testing ping from Windows.
DEBUG: Test-WslInstallation [1681]: $windowsTest result: True
Test-WslInstallation on Ubuntu-22.04 Failed!
PowerShell finished installation of Wsl-IpHandler to Ubuntu-22.04 with Errors:
DEBUG: Install-WslIpHandler [294]: ScriptStackTrace: at Test-WslInstallation, C:\Users\admin\Documents\PowerShell\Modules\Wsl-IpHandler\Wsl-IpHandler.psm1: line 1695
at Install-WslIpHandler, C:\Users\admin\Documents\PowerShell\Modules\Wsl-IpHandler\Wsl-IpHandler.psm1: line 290
at <ScriptBlock>, <No file>: line 1
Pinging windows from WSL Instance: Ubuntu-22.04 failed:

Thanks for reporting this.
However, it appears that the problem is coming from powershell itself rather that this module. I have submitted the issue at powershell-core repo.

Hi @wikiped, thank you!
For meanwhile as a workaround maybe start a cmd and inside of it lauchn wsl.exe? And after powershell-core fix applied revert logic bacK?

The bug comes with v7.3.*, so meanwhile I would recommend to downgrade powershell to 7.2.9: remove installed version 7.3 - and then install any older version.

The issue should have been fixed with latest commit. Has been tested to work with both 7.3.2 and 7.2.9.
How to update

Tested on both 7.3.2 and 7.2.9 it working as expected, thank you. And version 7.3.2 has an active history that can fill commands with just -> arrow, don't won't revert back, killer feature :)


TypeScript type for react-jsx-highmaps

Bug or Feature Request?
Feature request
Description
The library is really nice to use, but there are no TypeScript type definitions for react-jsx-highmaps. I was wondering if some can be added as it would be helpful to anyone who uses the library with TypeScript.
How to reproduce
Install the react-jsx-highmaps and you get a tslint error complaining about the lack of type definitions, it can be fixed with an @ts-ignore.
Live demo demonstrating bug

Versions

React JSX Highcharts version: 4.2.1
Highcharts version: 9.0.1
React version: 17.0.2
React JSX Highmaps version: 2.2.2

Thanks, I have started having a look at implementing something similar for react-jsx-highcharts.

I have opened #344 which adds the type definitions for react-jsx-highmaps.

@AspieCoder1 this is now released as<EMAIL_ADDRESS>can you verify that it works for you?

Yes, I am able to download it and it appears the type definitions are working.

Released as<EMAIL_ADDRESS>Thanks for the PR!


Make traverseC category-polymorphic.

Won't the plugin make the replacement?

We're not using the plugin (at least as far as this is concerned), so we look at concat-classes as a library for category-polymorphic programming directly. And traverseC can't be used that way, plus it defies the expectation set by the rest of the library.
Actually, I'd expect traverseC (and collectC) to also live in ConCat.Category, like zeroC does, but that's less of an issue for me.


Can we display a vertical line in the line graph?

Is your feature request related to a problem?

Not actually. I just didn't saw any example related to this in the docs.
Describe the solution you'd like

I would like to plot a graph like this one: https://stackoverflow.com/questions/42423167/custom-vertical-line-using-plotly-js.
Describe alternatives you've considered

None yet.
Additional context

Create this type of graph with vertical line.

Bump

It is not working by what I saw creating this sample project: https://github.com/64J0/Feliz.Plotly-testing-vertical-line/blob/master/src/Components.fs

I think that I have spotted the problem. In this line https://github.com/Shmew/Feliz.Plotly/blob/master/src/Feliz.Plotly/Bindings.fs#L26 by what I understood, all the values within the data object but this is wrong for the shapes. They must live inside of their own objects like shown in the example from the starting of this issue.

Shapes seem to be broken:
shapes in Props/Layout.fs
shape in Props/Shapes.fs
Example:
F#:
layout.shapes [shapes.shape [     shape.type'.line; shape.x0 1; shape.x1 4; shape.y0 7; shape.y1 12]
current JS:
shapes: { shape: { type: "line", x0: 1, x1: 4, y0: 7, y1: 12 }}
desired JS:
shapes: [{ type: "line", x0: 1, x1: 4, y0: 7, y1: 12 }]
Fix:
Props/Layout.fs:
static member inline shapes (shapes: obj list) = Interop.mkLayoutAttr "shapes" (shapes |> ResizeArray)
Props/Shapes.fs:
static member inline shape (properties: #IShapeProperty list) = (createObj !!properties)

@MichaelMay81 this project seems abandoned unfortunately


Not possible to control tilt

Home Assistant version (if applicable):
2021.2.3
Describe the bug
I have a problem to set the tilt position with a HMIP-BBL blind actuator.
This could be related to issue #241.
It looks like currently it's not possible to set_cover_tilt_position or close_cover_tilt/open_cover_tilt if the position and tilt values are not correct. It seems when I set_cover_tilt_position, it moves to the (wrong) position value and sets the tilt position on that position.
To Reproduce
Steps to reproduce the behavior:

Close the blind
Open the blind with the wall switch and stop it at some time. The position value will be 100.
Run close_cover_tilt
The blind will move all the way up instead of just close the slats

Expected behavior
With set_cover_tilt_position / close_cover_tilt / open_cover_tilt, only the slats should be closed or opened, but the blind should remain at the current position.

I confirm this issue.
Got seven BBL actors on latest firmware and they always drive up if any kind of tilt position is set.

It might be related to pull request #365

I downgrade to 2021.1.5 as it is using 0.1.70 of pyhomematic. It is latest version before #365 merged to 0.1.71

I downgrade to 2021.1.5 as it is using 0.1.70 of pyhomematic regarding manifest.json in homematic core.
It is latest version before #365 have been merged to 0.1.71

And does it work there correctly?

I can't say it for now. Next automation is triggered tomorrow morning. Then we will know more.
I'll keep you posted.

Okay with Version 2021.1.5 tilting is functional.

Now I might have the issue that cover.open_cover and cover.close_cover isn't working properly. Means I have to use cover.set_cover_position with absolute values to solve this.

This should be working with the new integration.


aur-search does not colorize when piped into a pager.

This is related with the approach used on #473.
It's not good enough to check if one of stderr or stdout is connected to a terminal to decide if colorization should be done or not.
Ideally they should work independently, always colorizing on the terminal, but never colorizing when redirected. (unless otherwise specified by the user)
Doing that might imply writing our own alternative to /usr/share/makepkg/util/message.sh

Possible workaround: disable colored output if either stdout or stderr is not connected to the terminal.
--- a/lib/aur-search
+++ b/lib/aur-search
@@ -116 +116 @@ source /usr/share/makepkg/util/parseopts.sh
-if [[ -t 1 && ! -o xtrace ]]; then
+if [[ -t 1 && -t 2 && ! -o xtrace ]]; then
@@ -122 +122 @@ opt_long=('any' 'info' 'search' 'desc' 'maintainer' 'name' 'depends'
-          'makedepends' 'optdepends' 'checkdepends' 'key:' 'raw')
+          'makedepends' 'optdepends' 'checkdepends' 'key:' 'raw' 'color:')
@@ -146,0 +147 @@ while true; do
+        --color)         shift; [[ $1 == always ]] && colorize ;;

#557


1/18 PM PUBLISH

Docs Build status updates of commit adbfee6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

docs-conceptual/azurermps-4.4.1/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-5.7.0/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.13.0/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-6.13.0)

[Warning-RepositoryOwnerPermissionInsufficient] Owner rloutlaw of repository https://github.com/MicrosoftDocs/azure-docs-powershell has lost Admin permission. Please use Transfer Owner via Docs portal https://ops.microsoft.com to assign a new owner.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit adbfee6:
:warning: Validation status: warnings

File
Status
Preview URL
Details

:warning:Warning

Details

docs-conceptual/azurermps-4.4.1/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-4.4.1)

docs-conceptual/azurermps-5.7.0/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-5.7.0)

docs-conceptual/azurermps-6.13.0/get-started-azureps.md
:white_check_mark:Succeeded
View (azurermps-6.13.0)

[Warning-RepositoryOwnerPermissionInsufficient] Owner rloutlaw of repository https://github.com/MicrosoftDocs/azure-docs-powershell has lost Admin permission. Please use Transfer Owner via Docs portal https://ops.microsoft.com to assign a new owner.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Simplify Test target to not create stub Directory.Build.* files

See the comment at https://github.com/dotnet/scenario-tests/pull/85#discussion_r1572100941

I'm not actually sure this is better/preferred. If you set ImportDirectoryBuildProps=false it disables importing Directory.Build.props completely e.g. even if you have it in a subdirectory. With the tests we just want to prevent importing the D.B.props outside of the test directory so having an empty one seems better to me.


Added tests and changed paths for password reset

Fixes issue #1396

I'll put them to a single it statement - not too keen on using a before since I want to do some assertions in the common part as well.

No need for another round of review if the test is passing 👍


Using for devices only, html5 video seems to conflict and disables Swiper.

I am using swiper container as my fallback for my portfolio on mobile devices. So on desktop it isn't being used.
It works great as I want, viewable on mobile but when I add html video to my desktop site, even though I am removing it for the mobile devices in media queries, it seems to brake Swiper? Have you seen this issue before? I'm not adding any new js, just what was provided.
var mySwiper = new Swiper ('.swiper-container', { 		// Optional parameters 		direction: 'horizontal', 		loop: true 	})
Any advice on why this could be happening would be great.

Would be good to see live example or JSFiddle

Typically, I cannot seem to re-create the bug. So don't worry about it, must be something else conflicting.


update usage for pyspark

Codecov Report
Patch and project coverage have no change.

Comparison is base (06e8900) 31.33% compared to head (6f45836) 31.33%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #89   +/-   ##
=========================================
  Coverage     31.33%   31.33%
  Complexity       78       78
=========================================
  Files            30       30
  Lines          1519     1519
  Branches        239      239
=========================================
  Hits            476      476
  Misses          973      973
  Partials         70       70

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Crash Issue

Mon_Jul_25-21.03.25-CEST_2022.log

"ReallyCheat/src/hachkingtohach1/reallycheat/checks/network" > Have you deleted this file in the source code of the plugin ?

The error got removed the time I deleted
But there is another error but I deleted it. Now There is Error.
Sat_Jul_30-17.38.25-CEST_2022.log
Sat_Jul_30-17.39.47-CEST_2022.log


Remove empty transaction from claimSimulationDataset

Tickets addressed: N/A
Review: By commit
Merge strategy: Merge (no squash)

Description

#723 removed the only remaining action wrapped in this transaction context - so now this transaction context is empty and can safely be deleted.
Verification

I'm relying on end-to-end tests that run simulation to verify this change.
Documentation

No existing documentation should be affected by this change.
Future work

#687 is still desirable - although after #723 and this PR, there are fewer moving parts between claiming and starting a simulation

I'm not sure why the test failed the first time - my second commit would include the error message
My best guess is that it was a recurrence of #685


When i try to run i get this error

I think it might be a problem with your code or just something im doing wrong
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: electron ./app
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?

Nevermind i found it out 😄


Support state expiration in storage interface

Closed by https://github.com/stellar/rs-soroban-env/pull/820


Change old "extensions/v1beta1" to new "apps/v1"

Some old API versions were deprecated in k8s 1.16: https://kubernetes.io/blog/2019/07/18/api-deprecations-in-1-16/
This PR allows to use kubed on Kubernetes 1.16 and higher
Fixing #369

Thanks @ruzickap !