Symbolical Reshape target_shape

The problem I am facing is that the input size is variable and I don't know how to get a symbolic shape in the middle of the graph.
 Is there something better?
x_shape = [10, None, None, None, 3]   # batch + 4 axes
model = models.Sequential()
model.add(Convolution2D(64, 3, 3, border_mode='same', input_shape=x_shape[1:]))
# Will this cause the theano graph to be duplicated??
sh = model.get_output().shape
# Is there an easier way with core.Reshape??
model.add(Lambda(lambda x: x.flatten(ndim=2), output_shape=flattened_shape))
model.add(Activation(K.softmax))
reshape = Lambda(lambda x: x.reshape((sh[0], sh[1], T.prod(sh[2:]))))

# this is what I'd like to do instead
# model.add(Reshape((x.shape[0], x.shape[1], x.shape[2]*x.shape[3])))

Get the symbolic shape of a layer

In Theano:
layer.output.shape
In the general case:
K.shape(layer.output)

Specify a symbolic (i.e. variable at run time) target_shape in Reshape(target_shape)

Can you not already do that with Reshape?

In the general case:
K.shape(layer.output)

There doesn't seem to be a layer.output
my_conv = Convolution2D(64, 3, 3, border_mode='same', input_shape=[3, None, None])
my_conv.output

*** AttributeError: 'Convolution2D' object has no attribute 'output'

Did you mean to use get_output(), like this?
my_conv = Convolution2D(64, 3, 3, border_mode='same', input_shape=[3, None, None])
sh = K.shape(my_conv.get_output())

What happens if I have a non-deterministic layer in the model and I call get_output() to get the shape of a feature map in the middle of the definition of the model? Will this cause the compilation of two separate theano graphs, i.e. the first one when I get the shape tensor and the second when the output of the whole model is used by Keras? Or is Keras smart enough to save the intermediate graph after the first call to get_output() and reuse it when get_output() is called for the second time?

Can you not already do that with Reshape?

I think I tried Reshape(10, 20, -1) (which is Lasagne's syntax for "whatever else") but I guess I didn't try to use a symbolic variable right away. I will try to combine it with get_output(), thanks for the pointer!

You seem to be using an outdated version of Keras.
Also: reshape takes a tuple of integers, e.g. Reshape((10, 20, -1))
On 26 April 2016 at 12:56, Francesco<EMAIL_ADDRESS>wrote:

In the general case:
K.shape(layer.output)
There doesn't seem to be a layer.output
my_conv = Convolution2D(64, 3, 3, border_mode='same', input_shape=[3, None, None])
my_conv.output
*** AttributeError: 'Convolution2D' object has no attribute 'output'
Did you mean to use get_output(), like this?
my_conv = Convolution2D(64, 3, 3, border_mode='same', input_shape=[3, None, None])
sh = K.shape(my_conv.get_output())
What happens if I have a non-deterministic layer in the model and I call
get_output() to get the shape of a feature map in the middle of the
definition of the model? Will this cause the compilation of two separate
theano graphs, i.e. the first one when I get the shape tensor and the
second when the output of the whole model is used by Keras? Or is Keras
smart enough to save the intermediate graph after the first call to
get_output() and reuse it when get_output() is called for the second time?
Can you not already do that with Reshape?
I think I tried Reshape(10, 20, -1) (which is Lasagne's syntax for
"whatever else") but I guess I didn't try to use a symbolic variable right
away. I will try to combine it with get_output(), thanks for the pointer!
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/2509#issuecomment-214867359

You seem to be using an outdated version of Keras.

You are right, I just updated but it will take me some time to port my code to the new version of Keras.

Also: reshape takes a tuple of integers, e.g. Reshape((10, 20, -1))

Yes, sorry for the typo. What I am trying to do is to pass a symbolical argument to reshape. Is it possible? Something like:
my_conv = Convolution2D(64, 3, 3, border_mode='same', input_shape=[3, None, None])
sh = K.shape(my_conv.get_output())
Reshape((sh[0], sh[1], -1))

Thank you for your help.

@fvisin Have you solve this problem, I face the same problem to reshape with symbolic size?

same here! any solution?

No, as far as I can tell there is no solution unless you want to dig into Keras and modify the Reshape layer.
The Reshape layer only accepts integers. If you instead try to reshape the tensor directly in Theano (to avoid the Reshape layer completely), Keras fails to get information about the input shape of the next layer and crashes. The fact that Keras maintains a graph of node parallel to that of Theano seems like an unnecessary burden. This is not the case in other frameworks.

Are there any plans to update Reshape to allow for this type of functionality? Thanks.

Ok i did dig into this:
this function would reshape for conv2d in the best way it could:
def ReshapeForConv2D(inp):
  sh_int = K.int_shape(inp)

  if (sh_int[1] is not None and sh_int[2] is not None):
    if K.image_dim_ordering() == 'th':
      out = Reshape((1, sh_int[1], sh_int[2]))(inp)  # -1 for batch size....
    else:
      out = Reshape((sh_int[1], sh_int[2],1))(inp)  # -1 for batch size....
  else: #undefined lengths...
    sh = K.shape(inp)

    if (sh_int[1] is not None):
      sh[1] = sh_int[1]
    if (sh_int[2] is not None):
      sh[2] = sh_int[2]

    # Is there an easier way with core.Reshape??
    if K.image_dim_ordering() == 'th':
      out = Lambda(lambda x: K.reshape(x, (-1, 1, sh[1], sh[2])))(inp) #-1 for batch size....
    else:
      out = Lambda(lambda x: K.reshape(x, (-1, sh[1], sh[2], 1)))(inp)
  return out

But if some dimension is not specified, later calling something like Flatten()  orTimeDistributed(Flatten()) would fail not knowing the dimensions...


Make CONTENT_TYPE_DOES_NOT_EXIST error more specific.

Fixes https://github.com/contentful/contentful-cli/issues/94#issuecomment-582357151

:tada: This PR is included in version 1.0.12 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Problem of implementation of InstanceNorm1d in PyTorch Geometric

❓ Questions & Help

hello @rusty1s @magnetar99:
I manage to implement the InstanceNorm1d shown in #687 . And I just write a demo program... here is my code.
my implementation and some error...
my code
# layers_1.py

import torch
import torch.nn as nn
import torch.nn.functional as F
import torch_geometric
import torch_geometric.nn as gnn

import torch_scatter

# class GraphBatch(nn.Module):
#     def __init__(self):

#     def forward(self, x, edge_index):

if __name__ == '__main__':
    torch.manual_seed(0)
    x = torch.randn(30, 40)  # 30 nodes, with 20 dimensions
    batch = torch.tensor([0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 1, 1, 1, 1, 1, 1, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2, 2])
    # 30 nodes, and they are in 3 different graphs, use torch.repeat_interleave to make it much better.
    # 3 different graphs
    edge_index_0 = torch.randint(0, 12, [2, 40])
    edge_index_1 = torch.randint(12, 18, [2, 20])
    edge_index_2 = torch.randint(18, 30, [2, 50])
    edge_index = torch.cat((edge_index_0, edge_index_1, edge_index_2), dim=1)
    p = torch.randperm(110)
    edge_index_ = edge_index[:, p]  # shuffle the input edge_index, note that features are not shuffled.... (just for a simple test...)
    mu = torch_scatter.scatter_mean(x, batch, dim=0)
    sigma = torch_scatter.scatter_std(x, batch, dim=0)
    x_mu, x_sigma = mu[batch], sigma[batch]
    x_norm = (x - x_mu.unsqueeze(1)) / x_sigma
    x0 = x_norm[:12]
    x1 = x_norm[12:18]
    x2 = x_norm[18]

This is my code, and I think this code is like the code provided in #687 .
error?
And I want to check what is happening, or, what mean and std is, I found a problem? 👇ScreenShots

what it should be
as we all know that after normalization, the output's mean should be 0. and output's standard deviation should be 1. .
and why such kind of thing happen? (I know that this "can be regarded as" a small problem (since there is a slight differece with what it should be), but I just wonder why.
?? is there any papers, or ideas about "normalization on graph data?" ??
I know that in thomas kipf's gcn paper, A+I is called "renormalization trick", but perhaps this trick is not the normalizaiton in this issue. So... I just want to ask if there is some paper about normalization on "features"(not the normalization of "structure", i means, methods like "renormalization trick") of the graph?
Appendix: mean and std in numpy and pytorch
When I am finding out the reasons, I tried the mean and std in numpy and pytorch
Take such "simple" 1d array/tensor [1., 2., 3., 4.]as an example.
numpy
import numpy as np
x_np = np.array([1., 2., 3., 4.,])
print (x_np.mean())
print (x_np.std())
print ()
x_mean_ = x_np.sum() / np.prod(x_np.shape)
print (x_mean_)
x_std_ = (np.sum((x_np - x_mean_)**2) / np.prod(x_np.shape)) ** 0.5

ScreenShot👇

pytorch
import torch
x_t = torch.tensor([1., 2., 3., 4.])
print (x_t.mean())
print (x_t.std())
x_mean_ = x_t.sum() / 4
x_std_ = (torch.sum((x_t - x_mean_)**2) / (4 - 1)) ** 0.5  # NOTE(WMF): -1!!!
print (x_mean_)
print (x_std_)

ScreenShot👇

pytorch_scatter
import torch
import torch_scatter
x = torch.tensor([1., 2., 3., 4.,])
x_ = x.unsqueeze(0)
xT= x_.transpose(1, 0)
batch_0 = torch.tensor([0, ])  # NOTE(WMF): I know that there is ONLY ONE node in the graph, with 4 features.
batch_1 = torch.tensor([0,0,0,0])  # NOTE(WMF): In another view

# the following 2 are wrong examples... DO NOT USE LIKE THAT...
x_mean_0 = torch_scatter.scatter_mean(x_, batch_0, dim=0)
x_std_0 = torch_scatter.scatter_std(x_, batch_0, dim=0)

print (x_mean_0)
print (x_std_0)
print ()

# this time can be all right
x_mean_1 = torch_scatter.scatter_mean(xT, batch_1, dim=0)
x_std_1 = torch_scatter.scatter_std(xT, batch_1, dim=0)

print (x_mean_1)
print (x_std_1)
print ()

# or it can be written as:
x_mean_2 = torch_scatter.scatter_mean(x_, batch_1, dim=1)
x_std_2 = torch_scatter.scatter_std(x_, batch_1, dim=1)

print (x_mean_2)
print (x_std_2)
print ()

ScreenShot👇 (sorry, this is too long, and i do not capture the code part)

What I want to express here is:
In Image's Normalization, we sum up all the pixel's values, and then calculate it. (Batch Normalization or Instance Normalization) But here, when i operate the scatter_std, the feature_dimension, does not reduce(onlydim='s dimension reduce)?
And I think that /4 or /(4-1) is not the key point, since it is a problem of statistics, or numerical analysis. (when lim n->∞, the 2 approximations can be viewed as the same...)
final note:
#684 may be meaningless, and some of the thoughts may have no use. (perhaps some thoughts are too ahead of time, i.e. few people may pay attention to that. And I close the issue. Many sorry....
yours sincerely,
@wmf1997

Hi,
There is a weird unsqueeze(1) in your computation which yields incorrect results. Computing x_norm via
x_norm = (x - x_mu) / x_sigma

yields the correct results. Regarding normalization, it can be applied by normalizing the aggregation (as in GCN) or applied afterwards on the node features (like in GIN). For normalization on node features, I have yet to see anything other than BatchNorm.

hello @rusty1s
an apologize
First, I apologize that unsqueeze(1), I tried some other methods, but i forgot to change that. Now it is fixed. And is the result is okay..?
In [3]: x_norm.mean()
Out[3]: tensor(-1.0928e-09)

In [4]: x_norm.std()
Out[4]: tensor(0.9491)

In [6]: x0.mean()
Out[6]: tensor(0.0016)

In [7]: x0.std()
Out[7]: tensor(0.9515)

In [8]: x1.mean()
Out[8]: tensor(0.0005)

In [9]: x1.std()
Out[9]: tensor(0.9334)

In [10]: x2.mean()
Out[10]: tensor(-0.0018)

In [11]: x2.std()
Out[11]: tensor(0.9564)

But... the mean is close to 0.0 , and std is close to 1.0 . That is what I wonder. I guess it is the computational error? Or something else?

Well, first of all you index wrong features IMO
x0 = x_norm[:12]
x1 = x_norm[12:18]
x2 = x_norm[18]

but your first example has only 11 nodes. In addition, for std being exactly 1 you need much more samples. You get the same numerical instabililties when using regular mean() and std() calls.

oops! I got that!
I should have use
batch = torch.repeat_interleave(torch.tensor([12, 6, 12]))

and after I fixed it, all the means are allright.
so... this is alright. I should be more careful next time.


Thumbnail option does not work in Bootstrap 4.x

Bootstrap Version: 4.2.1
bootstrap-select Version: 1.13.5
jQuery Version: 3.3.1
Browser: Chrome 71.0.3578.98, Edge 41.16299.820.0
Operating System: Windows 10 (1709) - 16.299.904
I started by using the example referenced at this URL:
https://thdoan.github.io/bootstrap-select/examples.html
That example uses an older version of bootstrap-select, jQuery, and Bootstrap.  I created a working jsfiddle example using the code on that page.  I then proceed to update the scripts.  Changing to jQuery 3.3.1 did not change the result.  Upgrading to the latest bootstrap-select and Bootstrap causes the drop down to be display but with no thumbnail.
https://jsfiddle.net/ddelella/xqygf86n/9/

bootstrap-select doesn't have a thumbnail option. The version you linked appears to be a modified version of bootstrap-select.

I've just been caught out by the same thing.. I need to allow a user to select one image from a list, and I liked the idea of using a styled select list with image thumbnails.
@caseyjhol, I've searched the issues and PRs and I don't see anything about this, so just in case I missed it, is there any plan to add (or explicitly not add) such a feature to bootstrap-select?
at the time of writing this, @thdoan's fork is "19 commits ahead, 518 commits behind snapappointments:master", so if I can rebase/adapt those changes into this project, would you be willing to accept a PR for this?


checking connection and forcing reconnection

Hi, first of all, thanks for this amazing library.
I'm just wondering, how can I force a reconnection attempt as my project will lose connection and it will take up to 20 seconds to reconnect.
My project is interactive so when people use it, it uploads the data to firebase and a user waiting 20 seconds will make them walk away.
I am using Firebase.reconnectWiFi(true); but I would like to be able to monitor my connection and force it to reconnect when the connection is lost.
How would I check the connection status?
How can I force it to reconnect if disconnected?
thanks so much.

The time interval for Firebase library to take care for the WiFi reconnection is every 10 seconds.
Why 10 seconds? Because this is the interval to prevent the mcu to wait all the time (blocking operation) if the WiFi can't connect all the times and if the wait timeout reached (approx. 5 seconds), exit the wait loop.
The WiFi connection can't be handle directly as ESP chip but through WiFi101 library which control the communication process with ATWINC 1500 WiFi chip.
If you don't want the wait in Firebase library, you should check the WiFi status by yourself before call the Firebase functions and in the loop too.


Won't work with<EMAIL_ADDRESS>

Environment

node -v: v8.2.1

npm -v: 5.3.0

npm ls react-scripts (if you haven’t ejected): empty

Operating system: macOS sierra

Steps to Reproduce
npm install<EMAIL_ADDRESS>Restart vscode and it works fine
npm install<EMAIL_ADDRESS>Restart vscode and it doesn't work
``
Expected Behavior
Color dot should appear aside each it
Possible error message should display
Actual Behavior
Non of these appear.
On devtools console appears this error
Cannot read property 'testRegex' of undefined: TypeError: Cannot read property 'testRegex' of undefined

Tried to help running the project but with no luck :/
I have no experience developing vscode extensions, sorry about that
Cloned, yarn install, opened with vscode, F5 and nothing (no second window opened)
just this message on the debug console

If it's not something obvious / easy to fix, just focus on the issue, not on my lack of experience XD

PS: I installed<EMAIL_ADDRESS>because 20.0 throws EMFILE error and was trying to avoid everyone having to run brew install watchman

I'm closing the issue having been unable to reproduce this today. We're long past Jest 20.1.-delta.5.


Unable to import gorgonia.org/gorgonia/examples/mnist

go: module gorgonia.org/gorgonia@upgrade found (v0.9.17), but does not contain package gorgonia.org/gorgonia/examples/mnist

You need to run the command from inside the example directrory /gorgonia/examples/convnet$ go run . -epochs 1 2 and it will work.
Reason being inside the example there is another go.mod.
I had the same problem and for me the solution was to open /gorgonia/examples/convnet with a new vscode instance.

just to add some more context, in particular after this addition https://github.com/gorgonia/gorgonia/commit/bc72cf782765090e64fc334d2fa23820f6ebe8b6#diff-20fabdc98fb65350ab9b5e5be6d09b920f1f48bb44b1c81084459958f8c7f762 the import of gorgonia.org/gorgonia/examples/mnist broke. Removing the go.mod seems to fix the issue but there might be a good reason why those mods (one for each example) are there?

Can you please clarify on how we can fix this issue :
go: finding module for package gorgonia.org/gorgonia/examples/mnist
practice/practice imports
gorgonia.org/gorgonia/examples/mnist: module gorgonia.org/gorgonia@latest found (v0.9.17), but does not contain package gorgonia.org/gorgonia/examples/mnist
How do I run the command from inside  that you stated, and how do i run another code in VSC as stated by you :
(I had the same problem and for me the solution was to open /gorgonia/examples/convnet with a new vscode instance.)


www.google.com and google.com should be the same.

When people mention google.com or www.google.com (#82 or #92), they are talking about the same exact website and so i think combining counts for both of them makes sense. You can rename the entry to "google.com or www.google.com"

Dutifully noted. Thanks for feedback!

On Apr 25, 2016, at 7:07 AM, Chintan Patel<EMAIL_ADDRESS>wrote:
When people mention google.com or www.google.com http://www.google.com/ (#82 or #92), they are talking about the same exact website and so i think combining counts for both of them makes sense. You can rename the entry to "google.com or www.google.com http://www.google.com/"
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/antontarasenko/smq/issues/1

The same holds for many of the news.ycombinator.com vs ycombinator.com links in this list


Observation of unstable scanning of physical web in Android, Chrome 49

Does physical web wakes only when google.com has the permission allowed or it also needs to be able to fetch the location?

Yes, you need to have a location provider enabled.

Does physical web need permission specifically for google.com or any other website will also do?

You just need to have granted the location permission to chrome.  The most common way that users do this is by visiting a site (like google.com) that requests it.  You could also enable the Physical Web in privacy preferences -- we will request location permission if you go through that flow.

Will the physical web wake up if the location permission is disabled on google.com?

We can't scan without the location permission.

If there is absence of internet, will physical web even wake up, though it may not be able to fetch metadata and instead just show loading?

We may scan, but we will not show results or send you a notification.

Does slow internet affects the scannability of chrome to wake physical web?

You should still see results, even with a slow connection.

Are there any other permissions that the user needs to grant?

No.

What is the best procedure and condition for a user to get physical web notification when he comes in the range of an URL broadcasting beacon?

Chrome 50 will have a diagnostics page at chrome://physical-web to diagnose these issues.  For now, the best way is to go through the settings -> privacy settings -> physical web -> enable flow to ensure we request location permission.

How can we reach to a stage where the user just turns on bluetooth and gets physical web?

Right now, we need the location permission, a location provider, and a data connection.  If you have suggestions on how to expedite any of these, we are open to ideas.

Thank You for the answer here.
Few observations here:
1| I saw a lot of smartphones with chrome 49 but there is no option for physical web under privacy setting.
2| In smartphones where there is physical web option in privacy, when I switch it on,it does not asks for location permission. I had to manually open google.com and get location permission.

I did a fresh installation of chrome 49.
Then I went to privacy setting and I could not see physical web there.

Then I went to chrome flags and changed the state of physical web flag from default to enable, and immediately upon relaunching chrome, I could see physical web.
If this works out, then why haven't we made the physical web flag to enable instead of default.
I think can help us detecting physical web easily.

One reason I could think of for why in chrome 49 we cannot see physical web in every phone is because it is still not rolled out to everyone.
Are we still rolling it out or Google chrome has completed physical web roll out to everyone?

This is a bug, we're fixing it in Chome 50

Thank You Scott.  Does that mean the physical web flag will be set to enabled by default when a user update chrome or the physical web option will be present in privacy settings


Update strings.xml

Fantastic work! You also made me find some place where I was defining some of the strings directly in the code instead of having them in the resources files. I'll migrate a few there and let you know which ones need translating.
Also question for you: this is the sync button if I leave it as is. Do you think it's better to have it bigger so that the word sincronizar fits into one line, or do you have a shorter version of the word that's commonly used like sinc or something?

Like this doesn't look good, hahahah. And the words we can use to replace it are also long...

"Sincronizar": Sinc (but it's not an abbreviation that is commonly used in my opinion and most people will not push it)
"Actualizar" (means update): This one doesn't have an abbreviation
"Refrescar" (means refresh): This one doesnt' have an abbreviation

As another possibility, would it be possible to replace sinc for a circle arrow as used on web browsers to refresh the page? I think is a symbol globally understood and eliminates the problem with the word or their abbreviations, because I don't know which one can be used

I might consider an icon indeed if we get into more trouble with the other languages. But for now I like that this builds the knowledge of what words we use in bitcoin for these types of operations, so I'll leave it as is and see how it goes with other languages before making anymore changes there.


Add temporary parse error for @helper directive to avoid confusion.

Currently @helper is unavailable in Razor 4 but we should create a temporary error to notify users. This will also server to avoid unrecognizable errors such as the one mentioned here.

Ahh wrong repo.


modprobe vboxsf fails, virtualbox-guest-additions

Issue description
Attempt to install virtualbox-guest-additions. Tried linuxPackages_latest and linuxPackages_testing and also virtualbox v5.2.18.
dmesg | grep vbox
[    0.885435] stage-1-init: loading module vboxsf...
[    0.887157] vboxsf: loading out-of-tree module taints kernel.
[    0.887182] vboxsf: Unknown symbol VBoxGuest_RTMemTmpFree (err 0)
[    0.887183] vboxsf: Unknown symbol VBoxGuestIDC (err 0)
[    0.887185] vboxsf: Unknown symbol VBoxGuest_RTSemFastMutexRequest (err 0)
[    0.887189] vboxsf: Unknown symbol VBoxGuest_RTSemFastMutexRelease (err 0)
[    0.887190] vboxsf: Unknown symbol VBoxGuest_RTLogRelGetDefaultInstanceEx (err 0)
[    0.887192] vboxsf: Unknown symbol VBoxGuest_RTStrCopy (err 0)
[    0.887193] vboxsf: Unknown symbol VBoxGuest_RTErrConvertToErrno (err 0)
[    0.887197] vboxsf: Unknown symbol VBoxGuest_RTSemFastMutexCreate (err 0)
[    0.887198] vboxsf: Unknown symbol VBoxGuest_RTSemFastMutexDestroy (err 0)
[    0.887204] vboxsf: Unknown symbol VBoxGuest_RTAssertShouldPanic (err 0)
[    0.887206] vboxsf: Unknown symbol VBoxGuest_RTLogLoggerEx (err 0)
[    0.887208] vboxsf: Unknown symbol VBoxGuest_RTMemTmpAllocTag (err 0)
[    0.887211] vboxsf: Unknown symbol VBoxGuest_RTAssertMsg1Weak (err 0)
[    0.887214] vboxsf: Unknown symbol VBoxGuest_RTAssertMsg2Weak (err 0)
[    0.900375] stage-1-init: modprobe: ERROR: could not insert 'vboxsf': Unknown symbol in module, or unknown parameter (see dmesg)
[   13.594264] vboxdrv: Found 12 processor cores
[   13.613084] vboxdrv: TSC mode is Invariant, tentative frequency<PHONE_NUMBER> Hz
[   13.613085] vboxdrv: Successfully loaded version 5.2.18 (interface 0x00290001)
[   17.545456] IPv6: ADDRCONF(NETDEV_UP): vboxnet0: link is not ready

Steps to reproduce
{
  virtualisation = {
    virtualbox = {
      guest = {
        enable = true;
      };
    };
  };
}

Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
 - system: `"x86_64-linux"`
 - host os: `Linux 4.18.9, NixOS, 19.03.git.d5e99fa (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.1pre6377_954d1f4d`
 - channels(root): `"nixos-19.03pre152634.218ce4de508"`
 - nixpkgs: `/home/xps/Downloads/nixpkgs`

Actually seems to be caused by a problem in my configuration, I'll close for now and post the problem if I ever get around to looking for it.

I met this error now. @eadwu did you find out what caused this?

For me, I was running virtualisation.virtualbox.guest.enable = true on my host machine which doesn't really make sense so I changed it to virtualisation.virtualbox.host.enable = true while virtualisation.virtualbox.guest.enable = true was set in my VM, like in here, in my gitlab, where it is enabled in nixos/modules/virtualisation/virtualbox.

I'm definitely using a Virtualbox guest and set the guest enable and still got this error.

Are you able to use the functionality given by the guest additions regardless of the error?

howdy
I just ran "/mnt/VBoxLinuxAdditions.run" after mounting as root and got the same errors shown with dmesg. Seems modprobe does also not work: --- snip ---
VirtualBox Guest Additions: Building the VirtualBox Guest Additions kernel modules.  This may take a while.
update-initramfs: Generating /boot/initrd.img-4.18.0-0.bpo.1-amd64
VirtualBox Guest Additions: Running kernel modules will not be replaced until the system is restarted
VirtualBox Guest Additions: Starting.
VirtualBox Guest Additions: modprobe vboxsf failed --- snap ---
I noticed this swapping phenomenon on an earlier VBox releases already: that the vboxsf module was (sometimes) not (always) included/compiled/running after VBoxLinuxAdditions.run. Someone some clue on this?
Your help is very welcome.
Kind regards,
Roger


add a timeout to the requests

For no reason. Because there's no way we're actually still using this gem. Nope.

:+1:


Mistral embedding rate limit exceeded

$config = new OpenAIConfig();
$chat = new MistralAIChat($config);
$embeddingGenerator = new MistralEmbeddingGenerator();

$dataReader = new FileDataReader(__DIR__.'/myfile.pdf');
$documents = $dataReader->getDocuments();
$splittedDocuments = DocumentSplitter::splitDocuments($documents, 3072);
$formattedDocuments = EmbeddingFormatter::formatEmbeddings($splittedDocuments);
$embeddedDocuments = $embeddingGenerator->embedDocuments($formattedDocuments);

returns

PHP Warning:  Undefined array key "data" in llphant\vendor\openai-php\client\src\Responses\Embeddings\CreateResponse.php on line 49

According to Mistral AI limits, we can do 1 request per second. In the embedDocuments function, there is a loop over Documents that calls embedDocument. embedDocument calls embedText that calls LLM to do the work. This is process is too fast.

This is the test
<?php

declare(strict_types=1);

namespace Tests\Integration\Embeddings\EmbeddingGenerator;

use LLPhant\Embeddings\EmbeddingGenerator\Mistral\MistralEmbeddingGenerator;

it('can embed some stuff with pause', function () {
    $llm = new MistralEmbeddingGenerator();
    for($i=0; $i<=5; $i++) {
        sleep(2);
        $lastEmbedding = $llm->embedText('This number is '.$i);
    }
    expect($lastEmbedding[0])->toBeFloat();
});

it('can embed some stuff', function () {
    $llm = new MistralEmbeddingGenerator();
    for($i=0; $i<=5; $i++) {
        $lastEmbedding = $llm->embedText('This number is '.$i);
    }
    expect($lastEmbedding[0])->toBeFloat();
});

The problem is real, and I think it's not limited to Mistral, but probably could happen using OpenAI too. Unfortunately the solution isn't trivial, since rate limits depend on your subscription type, so we can't wait for a fixed amount of time.
A possible workaround could involve batches: https://github.com/openai-php/client?tab=readme-ov-file#batches-resource
I think that another solution could be to intercept HTTP code 429, but at the moment I can't understand how. @MaximeThoonsen , do you have any idea?

@f-lombardo  embedText into src/Embeddings/EmbeddingGenerator/EmbeddingGeneratorInterface.php should accept array. Not only a string.
See openai api docs and mistral api docs.
ollama accepts only a string.

hey @cirolaferrara you are right, we can batch now: https://docs.mistral.ai/capabilities/embeddings/
So we could send a batch of something like ~50 embedding (maybe even more).
Do you want to suggest a change for this one?

I had a look at the code from openai-php, I didn't see how we could batch. So :

either we make a PR on openai-php project's
or we use another client to make the call

what do you think?

I went for using guzzle.
Does it work for you @cirolaferrara ?

In these days I am doing some testing. What do you think about using a dedicated client for each LLM?
Mistral,  Ollama Openai?

My goal is to keep everything as simple as possible. So I'd prefer if we could avoid that.
But if we need it, no problem

@MaximeThoonsen would you accept some kind of static class which keeps track of the X-RateLimit-* headers, and then users of the library can easily access that information and act upon it. I only use OpenAI but I assume each LLM exposes similar headers?


Fixed TEST_DIR management

TEST_DIR was set in script even if already set in environment. Added a test so that it is not set if already defined.

thanks @c4s4, that's a useful feature.  As you submitted the pull request to my fork as well, I added my comments in there: https://github.com/cirulls/xspec/pull/11

I merged this pull request in my fork and add a bats unit test testing that TEST_DIR is overridden when provided externally.


Remove connect-mincer dep

Use https://github.com/nodeca/mincer/tree/master/examples as a starting point for a replacement. conncet-mincer is heavily outdated and hasn't received any updates since december 2013.

Obsolete as of 88c9125aa82ae2250627ecf0749ebcae97e3089e.


Update packages and remove unused packages

This just cleans things up a bit to keep everything up to date. I looked over all the revision notes, and in some cases the commits between releases, and all these bumps look sane. I also ran all of the gulp tasks (except for gulp spec-server as noted in pivotal-cf/pui-react-tools#32). I did not bump react dependencies or babel-loader, which will make more sense to do once we're consuming a pui-react-starter that includes pivotal-cf/pui-react-tools#34

Oops, thought I already merged this. It is no longer relevant with version 3


two changes i had to make to get the basics to work

html change:
the two buttons can not have the same class. the "feedback_button" class is only for opening the layer and not for sending the request to the backend
js change:
I had to suppress the default function of the feedback_button. otherwise, the layer pops up and the page goes to the /feedback/ url
I am not that happy about the nature of these changes. Do you know any better way to make this work?

It brokes my instance. I don't fully understand what problem is it solving. Can you please describe better?

I have changed the class for feedback_button, which was same as feedback pop-up button, so it probably fixes your first problem.
The second problem might be caused by lack of feedback/jquery.form.js import in your form.

Yes, problems should be fixed this way.


Fix not ASCII shows not found error

Fixes Flexget/Flexget#762, Fixes Flexget/Flexget#759.

Closed since this has been merged into dev


neorpc: adjust SignerWithWitness marshalling scheme

Close #3059. C#'s rules for RPC-server signers unmarshalling are not as strict as for core signers. Not sure whether it's worth bug in C# node, because it's a valid behaviour. Let's fix it in our side.

I'd expect to see WitnessScope.UnmarshalJSON modification only, but if it's really unmarshaled differently in different contexts then I love C#, yay.

I've even added a unit-test to C# node to ensure that witness scope of a simple signer can't be unmarshalled from number (only from enum's string representation).


Not possible to use selector operators with json selector property

It doesn't appear that there is a way to use an operator other than the implied = with json, which uses
"selector": { "name:", "foo", "value": "bar"}.

forked from #3623

Dupe of #341 (yes, 3 digit issue).


Improve support for Sequelize

Improve support for Sequelize. Currently only  .sequelizerc is supported, but .sequelizerc.js and .sequelizerc.json should also be supported.
Changes proposed:

[X] Add

@JimiC Done! Thanks for the fast response! Hopefully it is correct now.


Implement control flow graph

This is great! I'd love to add some documentation in the README.md that goes with the implementation to explain how to implement this - which bits to copy for another language, the reasoning behind which bits to add, etc. Some explanation of the terminology would be good too, e.g. "completion"


irmin-pack: avoid assuming the encoding of atomic write keys

We currently use Repr.of_bin_string in Irmin_pack.Atomic_write to construct a "zero" value that is assumed never to be constructed by the user. This doesn't extend to non-hash keys (as will be required by https://github.com/mirage/irmin/pull/1389), so this commit changes the API to be more upfront about what properties are required.

Thanks @icristescu for the review.


fix: remove ibc connection coreum with sei

This channels have not been ever used

It there a site that shows that it's never been used?

Hey @JeremyParish69
{
"amount": {
"denom": "ibc/81BD95B0890B8D0130755E8338CF4AA48C7CBBD149C1D66EC9F9B62AFAE5C4F3",
"amount": "16928523"
}
}
This is the total supply that was bridged (about 16 SEI in total (quite a small amount) the first days the channel was open (and that we bridged for testing) and before it expired because it wasn't maintained not used by anyone. The channel hasn't been maintained for months.


new toolbar

sorry fo this commit. I merge it accidentally.
I want add fontselect fontsizeselect.
вт, 9 окт. 2018 г. в 15:11, Christoph Schweppe<EMAIL_ADDRESS>
Is there any specific reason why you removed all those functions?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/the-control-group/voyager/pull/3614#issuecomment-428167871,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AbNOE2ehUHo5tUztc8npMhs8XMcpaQnUks5ujJJigaJpZM4XTFZk
.


please give me the db initial sql，thanks！

PM> Add-Migration Initial
Startup project 'Pluto.Domain' targets framework '.NETStandard'. There is no runtime associated with this framework, and projects targeting it cannot be executed directly. To use the Entity Framework Core Package Manager Console Tools with this project, add an executable project targeting .NET Framework or .NET Core that references this project, and set it as the startup project; or, update this project to cross-target .NET Framework or .NET Core. For more information on using the EF Core Tools with .NET Standard projects, see https://go.microsoft.com/fwlink/?linkid=2034705
PM>

Add-Migration Initial -Context MainDbContext
Microsoft.EntityFrameworkCore.Model.Validation[30000]
No type was specified for the decimal column 'Price' on entity type 'Product'. This will cause values to be silently truncated if they do not fit in the default precision and scale. Explicitly specify the SQL server column type that can accommodate all the values using 'HasColumnType()'.
Microsoft.EntityFrameworkCore.Infrastructure[10403]
Entity Framework Core 2.2.3-servicing-35854 initialized 'MainDbContext' using provider 'Microsoft.EntityFrameworkCore.SqlServer' with options: using lazy-loading proxies
Your target project 'Pluto.API' doesn't match your migrations assembly 'Pluto.Repository'. Either change your target project or change your migrations assembly.
Change your migrations assembly by using DbContextOptionsBuilder. E.g. options.UseSqlServer(connection, b => b.MigrationsAssembly("Pluto.API")). By default, the migrations assembly is the assembly containing the DbContext.
Change your target project to the migrations project by using the Package Manager Console's Default project drop-down list, or by executing "dotnet ef" from the directory containing the migrations project.


replace image vector component descriptor creation with component-cli

How to categorize this PR?

/area delivery
/kind task
/priority normal
What this PR does / why we need it:
The images defines in the images.yaml are now added to the component descriptor using the component-cli.
The new cli command understands the semantics of the images.yaml and adds the needed information to the componentdescriptor v2.
The new script basically does the same thing as the old bash script so all current automation will stay in place.
In addition, it adds new labels that are later used generate the image overwrites without the need of the gardener images.yaml during deployment.
For detailed information how the resources are translated see https://github.com/gardener/component-cli/blob/main/docs/reference/components-cli_image-vector_add.md .
For the detailed implementation see https://github.com/gardener/component-cli/blob/main/pkg/imagevector/imagevector.go#L29
This is the first step towards testing release candidates.
Special notes for your reviewer:
cc @AndreasBurger @ccwienk
Release note:

NONE

FYI: There is more computation for the component descriptor in the following lines, I guess you are aware. What's the reason this was not adapted?

FYI: There is more computation for the component descriptor in the following lines, I guess you are aware. What's the reason this was not adapted?

FYI: There is more computation for the component descriptor in the following lines, I guess you are aware. What's the reason this was not adapted?

It was currently not adopted because the overwrite is not correctly used: see this issue https://github.com/gardener/gardener-extension-provider-aws/issues/234
Therefore we wanted to first see how we can fix this issue.

FYI: There is more computation for the component descriptor in the following lines, I guess you are aware. What's the reason this was not adapted?

It was currently not adopted because the overwrite is not correctly used: see this issue https://github.com/gardener/gardener-extension-provider-aws/issues/234
Therefore we wanted to first see how we can fix this issue.

/hold

/hold

/unhold
after some clarifications, lgtm
I would however like to improve handling of "sourceRepository" attr (parse for repo-url, add rel-path, optionally add path to dockerfile)

/unhold
after some clarifications, lgtm
I would however like to improve handling of "sourceRepository" attr (parse for repo-url, add rel-path, optionally add path to dockerfile)


Architecture: Risks and Technical Debt

Section Risks and Technical Debt (design/risks.qmd). Contains more detailed information about possible risks and technical debts, as well as ways we plan to mitigate them.

Closed in #103.


Update Dockerfile

Workaround on Docker Hub pull limits by leveraging ECR Public Gallery
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Same as other PRs. Will circle back and review again if we decide to address this. Thanks again!


feat: add managed resource metrics

Description of your changes
This PR adds two new metrics that can be used for exposing information about reconciliations and request to aws per managed resource:

managed_resource_reconciles_total
aws_api_calls_reconcile_managed_resource_total

Which is especially helpful for managed resources which use relatively expensive api calls, like service catalog.
This two metrics turn on by feature flag --enable-managed-resource-metrics or env variable ENABLE_MANAGED_RESOURCE_METRICS.
For now these metrics are implemented only in ProvisionedProduct.
Metrics sample:
aws_api_calls_reconcile_managed_resource_total{api_version="servicecatalog.aws.crossplane.io/v1alpha1",controller_operation_type="create",kind="ProvisionedProduct",resource_name="kirill-test-2"} 1
aws_api_calls_reconcile_managed_resource_total{api_version="servicecatalog.aws.crossplane.io/v1alpha1",controller_operation_type="delete",kind="ProvisionedProduct",resource_name="kirill-test-2"} 0
aws_api_calls_reconcile_managed_resource_total{api_version="servicecatalog.aws.crossplane.io/v1alpha1",controller_operation_type="observe",kind="ProvisionedProduct",resource_name="kirill-test-2"} 27415
aws_api_calls_reconcile_managed_resource_total{api_version="servicecatalog.aws.crossplane.io/v1alpha1",controller_operation_type="update",kind="ProvisionedProduct",resource_name="kirill-test-2"} 0
managed_resource_reconciles_total{api_version="servicecatalog.aws.crossplane.io/v1alpha1",kind="ProvisionedProduct",resource_name="kirill-test-2"} 5483

I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

Could we avoid changing each manage resource (at least more than a few lines)? Perhaps this belongs in provider-runtime?
See also https://github.com/crossplane/crossplane-runtime/issues/555


Cannot build NodeJS module in latest Raspberry Pi A+ (Raspbian 4.1)

When trying to build mraa from source i run in this error when it gets to compiling the nodejs module:
http://www.elec-tron.org/wp-content/uploads/2015/10/log.txt
These are my installation procedures from scratch -> http://www.elec-tron.org/?p=996
I have been able to get MRAA working on the previous Raspbian 3.18 kernel but on Raspbian 4.1 i run into the above.... Any suggestions?

Looks like a v8 version incompatibility to me. Which node.js and v8 versions do you have on Raspbian 4.1 and on an older one that worked?

Yes correct, ill downgrade. Thanks

I have left step by step build instructions on my blog here:
http://www.elec-tron.org/?p=996


Revamp amp-access to be simpler and more generic

Rephrase docs to be more generic around authentication, rather than publisher-specific.
Make it clear that amp-access response allows tiered access / ACL (through powerUser property).
Simplify login page, remove Google login (to be added as separate sample in future PR).
Remove logout page (use redirect instead).
Use email of logged in user in auth response instead of "Charlie".
Make it possible for login to fail if email is invalid.
Make it clear the login page needn't be AMPHTML.
Remove unused amp-access backend handlers from hotel.go.

Why Google sign in has been removed? It was easy to show around that it's possible to use Google sign in with AMP

Next PR (that I'm working on right now) will bring it back for the comments sample. I just wanted to separate it from the "normal" login, to simplify the amp-access demo.

Update: #1475 "brings back" Google sign-in, so I'll merge this after that PR, as to not end up in a state where we remove it.

@fstanis Can we merge this?


🐛 Bug Report: the backstage identity is showing up as guest even though I have authenticated using the github auth provider

📜 Description
the backstage identity is showing up as guest even though I have authenticated using the github auth provider
👍 Expected behavior
the user entity should show up as my username
👎 Actual Behavior with Screenshots

 this is the actual behavior
👟 Reproduction steps

configure auth provider
login using the cred
click on settings button
check the backstage identity

📃 Provide the context for the Bug.
No response
🖥 Your Environment
No response
👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Hi @gowthamshankar99, I tried to answer your question on Discord - https://discord.com/channels/687207715902193673/1128212589378273290 - but I need some information from you. Can you share your auth.ts file contents please? Also, do you have User entities in your catalog?

@awanlin  apologize for the delay. Here is the auth.ts contents
import {
  createRouter,
  providers,
  defaultAuthProviderFactories,
} from '@backstage/plugin-auth-backend';
import { Router } from 'express';
import { PluginEnvironment } from '../types';

export default async function createPlugin(
  env: PluginEnvironment,
): Promise<Router> {
  return await createRouter({
    logger: env.logger,
    config: env.config,
    database: env.database,
    discovery: env.discovery,
    tokenManager: env.tokenManager,
    providerFactories: {
      ...defaultAuthProviderFactories,

      // This replaces the default GitHub auth provider with a customized one.
      // The `signIn` option enables sign-in for this provider, using the
      // identity resolution logic that's provided in the `resolver` callback.
      //
      // This particular resolver makes all users share a single "guest" identity.
      // It should only be used for testing and trying out Backstage.
      //
      // If you want to use a production ready resolver you can switch to
      // the one that is commented out below, it looks up a user entity in the
      // catalog using the GitHub username of the authenticated user.
      // That resolver requires you to have user entities populated in the catalog,
      // for example using https://backstage.io/docs/integrations/github/org
      //
      // There are other resolvers to choose from, and you can also create
      // your own, see the auth documentation for more details:
      //
      //   https://backstage.io/docs/auth/identity-resolver
      github: providers.github.create({
        signIn: {
          resolver(_, ctx) {
            const userRef = 'user:default/guest'; // Must be a full entity reference
            return ctx.issueToken({
              claims: {
                sub: userRef, // The user's own identity
                ent: [userRef], // A list of identities that the user claims ownership through
              },
            });
          },
          // resolver: providers.github.resolvers.usernameMatchingUserEntityName(),
        },
      }),
    },
  });
}

@awanlin  - i see what you are saying -
const userRef = 'user:default/guest';

is there any documentation/reference of how to point this back to the logged in user ?

@gowthamshankar99 this is what the providers.github.resolvers.usernameMatchingUserEntityName(), does for you. It will look up a user in the catalog with the username matching the entity name in the catalog. If this is not what you want or need to do, you'll need to implement something that can do something similar but fits your use case depending on what users in the catalog are called.
Do you have all the users in the catalog?

Change it like below conf:
github: providers.github.create({
signIn: {
/* resolver(_, ctx) {
const userRef = 'user:default/guest'; // Must be a full entity reference
return ctx.issueToken({
claims: {
sub: userRef, // The user's own identity
ent: [userRef], // A list of identities that the user claims ownership through
},
});
},*/
resolver: providers.github.resolvers.usernameMatchingUserEntityName(),
},
}),
},
});
}

This worked! thanks a ton!

@gowthamshankar99 this is what the providers.github.resolvers.usernameMatchingUserEntityName(), does for you. It will look up a user in the catalog with the username matching the entity name in the catalog. If this is not what you want or need to do, you'll need to implement something that can do something similar but fits your use case depending on what users in the catalog are called.
Do you have all the users in the catalog?

yes I do - i was able to fix this issue

Awesome, glad you got it working @gowthamshankar99! I'm going to close this issue then

For others experiencing this issue, ensure you run through this guide as well:
https://backstage.io/docs/auth/
You will need to add a sign-in page component to App.tsx in order to get the profile page to link in properly


page.waitForSelector() with hidden === true always succeeds in some situations

Puppeteer version: 1.20.0
Platform / OS version: macOS 10.14.6
URLs (if applicable): (private)
Node.js version: 12.7.0

What steps will reproduce the problem?
If I run the following TypeScript code, Selector found will always be logged to the console no matter what selector is waited for.
import puppeteer = require('puppeteer');

(async () => {
  const browser = await puppeteer.launch({
    headless: false,
    //slowMo: 100
  });
  const page = await browser.newPage();
  await page.goto('https://www.google.com/');
  const navigationPromise = page.waitForNavigation();
  //await navigationPromise // uncomment to fix
  await page.waitForSelector('any-text-you-like', { hidden: true, timeout: 0 });
  console.log('Selector found')
})();

This is not the case if hidden: true is changed to visible: true.
In my particular use case there may be pauses and more than one redirect before the correct webpage (which doesn't have fixed URL) loads so I'm using waitForSelector() with a hidden element like <div id="my-secret-element-id" style="display: none;"> to detect when the correct web page is loaded. But this is not working due to this issue and awaiting e.g. page.waitForNavigation({'waitUntil': 'networkidle0'}); doesn't solve the issue.
What is the expected result?
Console should not show Selector found unless the selector has actually been found.
What happens instead?
Console does show Selector found.

Looks like hidden: true doesn't work at all. I don't have any redirects in my test and it still doesn't work as expected.

Reading the docs for waitForSelector, I see this bit from where it talks about hidden:

wait for element to not be found in the DOM or to be hidden

To me that means that using { hidden: true } will result in a success if the selector is not found on the page, meaning the element was not found in the DOM.
That said, if it's not meant to work that way then I'd be interested to know, because I've been using it with that assumption.

It definitely works. I tested this rigorously after suspecting some errors, but it turns out I wasn't handling race conditions very well. As suggested by @paperbackdragon, waitForSelector(..., { hidden: true }) may fail if the node is not in the DOM. You should proceed a call of hidden: true with a await waitForSelector(...) to ensure you are correctly waiting for the node to be visible before testing that it is hidden, if you expect it to appear then disappear in your test.


autosplits not working

had autosplits working a week ago for speedrunning haydee 2, now it doesnt work, maybe cause new update for livesplits? but deactivating and reactivating them in edit splits and redownloading livesplits and even trying it from the raw code text file doesnt work at all, any idea why autosplitting doesnt work anymore for haydee 2? the only time the timer activates auto is from starting the game from steam, never from resetting runs or even clicking start from new game.

Yes, it's because of the update... The autosplitter (any autosplitter actually) needs adjustment with every update. This takes a few hours, and I don't know if and when I have the time. You should let the developer know that this is a problem (which could easily be fixed from his side). It seems there are only like 2 people who ever used the autosplitter, though.

The developer of what? Livesplits? Or something else? Also thanks for replying, I really utilize the autosplits in my runs. Amazingly done.
-------- Original message --------From: Enthusiast @.> Date: 2021-12-09  11:14 a.m.  (GMT-06:00) To: EnthusiastNT/haydee2 @.> Cc: Mista-Roy @.>, Author @.> Subject: Re: [EnthusiastNT/haydee2] autosplits not working (Issue #2)
Yes, it's because of the update... The autosplitter (any autosplitter actually) needs adjustment with every update. This takes a few hours, and I don't know if and when I have the time. You should let the developer know that this is a problem (which could easily be fixed from his side). It seems there are only like 2 people who ever used the autosplitter, though.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.

Oh, I meant the Haydee developer. Maybe post in the Steam forum? Just so they are aware.

Note, you can install a previous version from Steam:
https://www.reddit.com/r/Steam/comments/611h5e/guide_how_to_download_older_versions_of_a_game_on/

Thank you for the idea, but I tried it and it still doesn't work, the splits auto start only when I start the game and not when in game, but im not sure which manifest to use, I only could find one by going to the database and finding the new update manifest then seeing there's been a manifest change and I used that ID, not sure how to find older ones.
-------- Original message --------From: Enthusiast @.> Date: 2021-12-19  6:33 a.m.  (GMT-06:00) To: EnthusiastNT/haydee2 @.> Cc: Mista-Roy @.>, Author @.> Subject: Re: [EnthusiastNT/haydee2] autosplits not working (Issue #2)
Note, you can install a previous version from Steam:
https://www.reddit.com/r/Steam/comments/611h5e/guide_how_to_download_older_versions_of_a_game_on/
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Should work now. Have fun!

It works, hopefully they'll stay working from now on, thank you for your
help, I'll be practicin runs all day.
On Mon., Dec. 27, 2021, 12:24 p.m. Enthusiast, @.***>
wrote:

Should work now. Have fun!
—
Reply to this email directly, view it on GitHub
https://github.com/EnthusiastNT/haydee2/issues/2#issuecomment-1001690001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVK6BZQFCFXYQQGSMQDYPNDUTCVPPANCNFSM5JWSTNPQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>


Serializing Resources for storage in UserDefaults

I'm trying to use the Serializer to store objects in UserDefaults so we can load resources up without always doing API requests.
I'm trying to serialize an array of objects that each contain toOne/toMany relationships.
So right now I'm doing:
func persistShoppingList() {
    let serializer = getShoppingListSerializer()
    let shoppingListData = try! serializer.serializeResources(contents, options: [.IncludeID, .IncludeToOne, .IncludeToMany])
    UserDefaults.standard.set(shoppingListData, forKey: "shoppingListArray")
}

However when I deserialize this, I have none of the attributes of the relationship objects inside of contents, only their ids. Here is how I'm deserializing:
  func checkShoppingListPersistence() {
    let serializer = getShoppingListSerializer()

    if let dataFromDefaults = UserDefaults.standard.data(forKey: "shoppingListArray") {
      let contentsJson = try! serializer.deserializeData(dataFromDefaults, mappingTargets: contents).data
      contents = (contentsJson as? [DFPContents])!

      tableView.reloadData()
      tableViewHeight.constant = tableView.contentSize.height
    } else {
      getShoppingList()
    }
  }

I'm not exactly sure what mappingTargets is for. Maybe I'm using that wrong?
How would I go about including the attributes of these relationships when serializing/deserializing?

@Croge32 UserDefaults isn't really appropriate for persisting objects like this. However the issue you're experiencing is related to not implementing encoding. See:
https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/Archiving/Articles/codingobjects.html

@Croge32 have you an example of the JSON which has been written to UserDefaults?

@markst The structure of the object's model is as follows:
class DFPContents: Resource {
  var quantity: NSNumber?
  var sequence: NSNumber?
  var wasConsumed: NSNumber?
  var calories: NSNumber?
  var foodId: NSNumber?
  var servingId: NSNumber?

  var dfpMeal: DFPMeal?
  var food: Food?
  var portion: Portion?

  override class var resourceType: ResourceType {
    return "dfp_contents"
  }

  override class var fields: [Field] {
    return fieldsFromDictionary([
      "quantity": Attribute(),
      "sequence": Attribute(),
      "calories": Attribute(),
      "wasConsumed": BooleanAttribute().serializeAs("was_consumed"),
      "foodId": Attribute().serializeAs("food_id"),
      "servingId": Attribute().serializeAs("serving_id"),
      "dfpMeal": ToOneRelationship(DFPMeal.self).serializeAs("dfp_meal"),
      "food": ToOneRelationship(Food.self),
      "portion": ToOneRelationship(Portion.self),
    ])
  }

...

I'm trying to serialize an array of these DFPContents objects. They are not coming from JSON, which I suspect is part of the problem. Basically when I just serialize the array itself, the children do populate and their ID and types are assigned, but each of their attributes are nil. I see that in the library itself, it looks like responses are able to be parsed into a JSONAPIDocument object where included resources can be specified, which I'm unable to do on my level.
I did however find a workaround. I noticed that if I serialize the object AND it's children all in a Resource array, the parents children are also populated with the children's attributes. So I'm able to pass an items array of type [Resource] into the serializer and then deserialize and pick out just the objects I want.
Here is how I'm serializing in order to get the child attributes to be non-nil:
func persistShoppingList() {
    let serializer = self.getShoppingListSerializer()

    var items = [Resource]()
    let contents = self.contents as [Resource]
    let foods = self.contents.flatMap {
      return $0.food!
    } as [Resource]
    let portions = self.contents.flatMap {
      return $0.portion!
    } as [Resource]

    items.append(contentsOf: contents)
    items.append(contentsOf: foods)
    items.append(contentsOf: portions)

    let data = try! serializer.serializeResources(items, options: [.IncludeToMany, .IncludeToOne, .IncludeID])
    cache["shoppingListArray"] = data as NSData
    tableView.reloadData()
    tableViewHeight.constant = tableView.contentSize.height
  }

I have also abandoned UserDefaults and am using a caching library now.
Any further insights? I know this probably sounds a bit confusing.

@markst Any ideas at all? Any suggestions on how I can achieve data persistence using Spine?

It's most probably not a reason, but have you remember about adding:
 serializer.registerResource(Food.self)
 serializer.registerResource(Portion.self)

Etc.?


fixed deprecations(connected property on MongoClient) in MongoQueue

[Mongo] This class has been DEPRECATED as of version 1.3.0. Relying on this feature is highly discouraged. Please use MongoClient instead.
MongoClient deprecate connected method. Fixed the deprecation issue.

MongoQueue::getDatabase(): The 'connected' property is deprecated

Happy to pull in this PR, but I'm not sure why the connection method is a static method. Also not sure why there is a connection->is in there


Add: Peer TV Südtirol

Channel Name
Peer TV Südtirol
Alternative Names (optional)
Peer TV;PeerTV;Peer.tv,peer.tv,peertv,PEERTV
Network (optional)
Peer TV
Owners (optional)
Peer GmbH / SRL
Country
IT
Subdivision (optional)
IT-32
City (optional)
No response
Broadcast Area
r/EU;c/IT;s/IT-32
Languages
deu
Categories (optional)
culture;documentary;lifestyle;outdoor;travel;weather;news
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.peer.tv/
Logo
https://www.peer.biz/peertv-iptv/peer-tv-suedtirol.png
Notes
No response

Same remarks as in https://github.com/iptv-org/database/issues/11055

I have also adjusted this. Now the data should suit :)


Dynamic imports and  shorter syntax for style modules

Dynamic imports is very powerfull features. We can split build to several chunks. But we need to support of this syntax import(/* webpackChunkName: "user" */ './@routes/user')

Just little improvement for ability using such name of style module file styles.m.scss not only this one styles.module.scss

@Razzwan thanks! looks good


Is a GUI required to run?

I'm trying to setup Dicaffeine on a NUC style device.  I've tested with the original install of the OS (which has a GUI) and I want to make the system as minimal as possible, so I was wondering if a desktop is required for this to work?

Hi, it's not if you're fine with the command line tools. You can compile Yuri2 binary which serves as a backend for the Dicaffeine. Short guide how to use it and how to compile the Yuri2 binary is here:
https://dicaffeine.com/guide


Add options which allows us to use difft as Apache Subversion's external diff tool

svn,  Apache Subversion's command line client also has option to use external diff tool for svn diff subcommand.  However it assumes that diff tool has  diff compatible -L (label) and -u option.
This implements -L and -u options.

Neat, it'd be nice to have svn support.
I'm OK with adding label options, but I don't like having multiple ways of setting the display mode.
Could we support this syntax similar to git support? If it's always the same arguments in the same order, we could allow those arguments without allowing -u in all circumstances.

Thank you for being interested in this PR.
-u option is passed only if the user does not specify any additional options.  This is done by svn_io_run_diff2() C API function.   In the case,  -u options is always the first argument, 0 - 2 -L LABEL options is followed and then finally 2 paths,  however this order is not documented.
How about  hiding -u option from help and just ignore it in the argment processing ?

I'd be interested!

@futatuki can you rebase your diff, please?

Yes, I'm willing to do later, with pleasure.

Yes, I'm willing to do later, with pleasure.

Thanks! I am using your patch, it is really helpful. I hope @Wilfred sees this in time and can maybe integrate it :)

Rebase done.


Update feature-gates-lifecycle.md

🚀 Deployed on https://63f6cdb4a1fa502d5365f61b--cozy-fox-0defec.netlify.app


More info on "Failed to load the tool-bar package" issues. Temporary solution for users.

I am too getting the same exact error, but I want to give a bit more information than the other issues.
Not 100% sure, but it looks like tool-bar can't generate the dist/iconsets.css file.
In the 551d2ce8cda320be0b881f164139bcdc97cbe9bd commit, if I revert the change done to the styles/tool-bar.less file, the error disappears, but the tool-bar doesn't show up.
So I tested a cloned a few versions and compared commits.
For me up to commit ab847968c66399f30328b2e9786248f32450d787 tool-bar worked.
Commit  f6170fafcf59baba7083480cb6fb0638b388cd00 was suddenly when the tool-bar didn't even show up, but no errors.
And commit 551d2ce8cda320be0b881f164139bcdc97cbe9bd was when the error appeared.
TL;DR
Something broke and doesn't load right. If you want to get tool-bar working again then you could try to downgrade your version by downloading the build from an older version like v1.3.0 or the lastest working commit

Thank you for the workaround! I just released 1.4.1 which should fix the issue :)


Can I skip not created resouce methods?

Route::resource('test','TestController);
But I created only index function in this controller and other function are not created (create, show, edit, update, destroy).
when I run artisan command, it shows the error:
  [ReflectionException]
  Method create does not exist

Try this
Route::resource('test', 'TestController', ['only' => ['index']);


[PE-D][Tester B] Inconsistent error messages when editing nicknames with duplicates

There is some discrepancies when editing to create duplicate nicknames. The error message for the other duplicate cases mention nickname as an example but the actual duplicate nickname error message does not use that.
Users with the same full name, with one being assigned a duplicate nickname:

Users being assigned a duplicate email results an error message that describes duplicate 'nicknames' as a potential case, but that message is not assigned to the case of duplicate nicknames

Labels: type.FunctionalityBug severity.Low
original: itsme-zeix/ped#1

See Issue #255


Add CVE-2024-4990

This PR adds the CVE-2024-4990 in yiisoft/yii2.
This is my first contribution, I hope I did it right, especially the branches key :sweat_smile:

It seems like the 2.0.49.x branch got updated and version <IP_ADDRESS> contains the security fixes: https://github.com/yiisoft/yii2/pull/20183

Thanks for all your efforts on this @DBX12 !


Pls do help on this

Pls do help

Please fill in the issue template so we have more information to help you.

cmd isn't in your path, hence spawn cmd ENOENT. Read this so post.


Removes memory_resource.

What is the next planned release that will include this fix?

Most users seem to be using the develop branch so I did not bother merging in master and creating a new release. I will prepare a new pre-boost release this weekend.

Just made a release tarball v1.4.2. Docs are not updated yet.

Thank you.
The companies I've worked with require release checkpoints for open source projects so this release helps.

Note that CMakeLists.txt still has "VERSION 1.4.1" in this 1.4.2 tarball.


Add bigger font size and bold text to product prices

Make the font-size a bit larger and font-style bold on the product prices in Products.tsx file using Tailwind CSS classes.

Hello. I would like to take on this issue.

Hello. I would like to take on this issue.

Sure, go ahead.

There seems to be a weird bug in your project with tailwind. Arbitrary values don't work.
Like this:
"text-[2.5rem]"
Maybe a fresh install of tailwind will help? I'll try it.

There seems to be a weird bug in your project with tailwind. Arbitrary values don't work. Like this: "text-[2.5rem]"
Maybe a fresh install of tailwind will help? I'll try it.

Sorry, i forgot to mention in the instructions that you have to run npx postcss src/index.css -o src/output.css to apply the changes. After that it will work.

I see. So you used the Tailwind CLI installation? Why not use the Framework install with Vite using React?

That's a good point, that would definitely be much more convenient solution that always use npx build. I might change that to make it easier in the future since im already using Vite.

That's great. Btw, how big do you want it to be?
Like this?

That's great. Btw, how big do you want it to be?
Like this?

Maybe just a bit bigger font and it should be fine.

That's great. Btw, how big do you want it to be?
Like this?

Maybe just a bit bigger font and it should be fine.

How about this? The middle one is the new one. So you can see the difference.

That's great. Btw, how big do you want it to be?
Like this?

Maybe just a bit bigger font and it should be fine.

How about this? The middle one is the new one. So you can see the difference.

That looks great.

Done. I made a pull request.


[BUG] deploy the hugo project into github pages with github actions. but encounter error "failed to load modules"

Describe the bug
I try to deploy the hugo project into github pages with github actions. but encounter error "failed to load modules"
To reproduce
Steps to reproduce the behavior:

add  .github/workflows/hugo.yaml to loacl repo and push to remote
Click on gitHub  'Action' to check the process
I see failed result
See error

Expected behavior
hugo
--gc
--minify
--baseURL "https://teholic.github.io/"
shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
env:
HUGO_VERSION: 0.122.0
GITHUB_PAGES: true
HUGO_ENVIRONMENT: production
HUGO_ENV: production
hugo: downloading modules …
hugo: collected modules in 18666 ms
Total in 18667 ms
Error: failed to load modules: module "hinode" not found in "/home/runner/work/teholic.github.io/teholic.github.io/themes/hinode"; either add it as a Hugo Module or store it in "/home/runner/work/teholic.github.io/teholic.github.io/themes".: module does not exist
Error: Process completed with exit code 1.
Log file
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Hinode version: v 0.22.5
Host OS: ubuntu-latest
Node version: 20.11.1
Hugo version: hugo v0.122.0-b9a03bd59d5f71a529acb3e33f995e0ef332b3aa+extended darwin/arm64 BuildDate=2024-01-26T15:54:24Z VendorInfo=brew
Browser: [e.g. Google Chrome Version 108.0.5359.124 (Official Build) (arm64)]

Additional context
[Add any other context about the problem here.]

It's difficult to reproduce the error with the information given, but I suspect you need to vendor your modules prior to the build. Try running hugo mod vendor prior to hugo --gc --minify --baseURL "https://teholic.github.io/".
If it still fails, would you mind sharing a (minimal) repo or the exact steps to reproduce the error? In particular, your custom action .github/workflows/hugo.yaml would be helpful, as well as if you are using Hinode as a regular theme or with npm.


Strategic retreat on multi-sorted type theory

The original vision was to construct multi-sorted type theory based on refinements (as in Darin and my recent paper), in which we would have things like Pi(A; x.B) refine σ ⇀ τ when A refines σ and when the family B(x) refines τ, etc. But there is trouble in paradise:

I had also thought that each sort would give rise to its own hierarchy of universes, but now, I do not know if this makes any sense. Each universe refines exp (the sort of type expressions), but it gets confusing beyond that... The first τ-universe contains the types that refine τ, but the second τ-universe contains those types as well as the first τ-universe itself; so there is a universe which contains some types that refine one sort (τ), and some types that refine another sort (exp). At this point, it is totally unclear to me what it means to quantify over types.

Issues with universes aside, if we intend to use Red JonPRL as a program logic, it might make a lot of sense to follow the program we laid out, but I would suggest it would be not so useful until we have generalized to a polymorphic or dependently-sorted syntactic framework (as opposed to mere ABTs, which correspond in some sense to a nominal version of the simply typed lambda calculus).

In any case, I think it's time to take a page out of the proceedings of the 10th Congress and make a strategic retreat.
I am proposing the following:

All of the CTT-related stuff should be put into the exp sort; things like tactic scripts, level expressions, etc. shall live in their own sorts, just like they do now.

For now, we should only have a single universe hierarchy, the hierarchy of types that refine exp. Since all type expressions are of sort exp, it is not even clear what a universe hierarchy for another sort would even mean. Now, this does mean that only types that refine exp inhabit a universe—but recall that the definition of typehood in CTT is not really inhabitance in a universe, but rather the imposition of a partial equivalence relation on the terms of a particular sort. Fixing a universe hierarchy has more to do with the PRL than it has to do with the actual type theory. So, there are still types that don't refine exp, but they just won't inhabit a universe. I don't think that this will be a big problem in general, and we can also add special universes in the future that include types refining a different sort. Additionally, there is no reason why we cannot relax the postsupposition of the PRL judgment H >> P from P shall inhabit a universe to P shall be a type.

Computational approximation, equivalence etc. must remain multi-sorted! There is no reason to restrict these to exp.

I spoke with @freebroccolo a bit over IRC last night, but I'm curious if anyone else has feedback. /cc @jozefg @david-christiansen

I would add that my current/evolved philosophy concerning the multi-sorted framework is that it should "especially" be used to capture the syntax of phenomena which are exotic in some sense to the type theory; this could include:

tactic scripts
level expressions
dimension expressions (for computational cubical type theory)
names of exceptions and assignables (e.g. how do you keep a local exception from being used as an assignable, or vice versa? by giving it a different sort)

I see all these things, in fact, as sort of syntactic implements that we're using in order to capture some aspect of type theory that we can't really write down. For instance, we impose a predicative hierarchy on type theory not because that is the way that the mental constructions work, but because it is a way to ensure that the constructions we write down correspond to ideal mental constructions; likewise, we use the presheaf style dimension labels in order to formally manipulate things that would correspond to ideal mental constructions of very high dimension. So, the purpose here is to separate the algebraic/syntactic (incidental, existing only because of encoding) from the behavioral (essential).

Here's the to-dos:

[ ] remove the TRIV sort
[ ] make AX : EXP
[ ] for now, remove the sort-indexing from universes


[FEA]: Upgrade to PyTorch 2+

Is this a new feature, an improvement, or a change to existing functionality?
Improvement
How would you describe the priority of this feature request
Medium
Please provide a clear description of problem this feature solves
To enable Hopper support (with sm_90) we need to upgrade to PyTorch 2.0+ with CUDA 11.8.
Describe your ideal solution
The PR for this issue should include:

Test whether or not we can install PyTorch via Conda

Currently we use pip to install PyTorch since there was no overlap between the version of CUDA they required and our own version of CUDA. With the latest PyTorch, they now support CUDA 11.8 which is our current version.

Upgrade to the latest available PyTorch that works with our set of dependencies
Update any code that had breaking changes due to the transition from PyTorch 1.13 -> 2.0+

This should include any examples such as the GNN example

Describe any alternatives you have considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open feature requests and have found no duplicates for this feature request

Open Q: while doing this, does it make sense to default the torch memory pool to use RMM? Reducing memory usage when combining torch with cuDF is always a win.
Happy to open this discussion in a different place, just figured it might be efficient while we're touching torch anyway.

@jarmak-nv That is interesting and certainly something we should explore, but I think that we should break this up into two issues. Using the RMM allocator has other benefits too (We could then directly access PyTorch tensors from C++ like we do with cuDF). Exploring those possibilities would be outside of the scope of a simple version upgrade.

Thought that might be the case, I'll open an issue so we can prioritize and work on as we see fit.


Footer looks bad on empty discover page

The logo should sit in the blue part under the diagonal

Fixed in https://github.com/dolthub/doltlab-issues/releases/tag/v0.3.0


[SPARK-33566][CORE][SQL][SS][PYTHON] Make unescapedQuoteHandling option configurable when read CSV

What changes were proposed in this pull request?
There are some differences between Spark CSV, opencsv and commons-csv, the typical case are described in SPARK-33566, When there are both unescaped quotes and unescaped qualifier in value,  the results of parsing are different.
The reason for the difference is Spark use STOP_AT_DELIMITER as default UnescapedQuoteHandling to build CsvParser and it not configurable.
On the other hand, opencsv and commons-csv use the parsing mechanism similar to STOP_AT_CLOSING_QUOTE  by default.
So this pr make unescapedQuoteHandling option configurable to get the same parsing result as opencsv and commons-csv.
Why are the changes needed?
Make unescapedQuoteHandling option configurable when read CSV to make parsing more flexible。
Does this PR introduce any user-facing change?
No
How was this patch tested?

Pass the Jenkins or GitHub Action

Add a new case similar to that described in SPARK-33566

The original case described in SPARK-33566 as follows:
data:
"h1","h2","h3"
"one","two","three"
"abc","^@Referral from Joe Smith.<A0> Fred is hard working.<A0> Super smart, though you wouldn't know it at first.<A0> 6 months, and we sold this project.<A0> Phooey he said to me!<A0> What's up with you people.<A0> You'll say anything for a sale!<A0> Until he met me of course....haar haar!
<A0>
Internet is spotty
Working while at home so.<A0> Will be applied this weekend. <A0>
<A0>
On Bill Recovery and 20 yr warranty added.
<A0>
Kindness made this deal happen!
<A0>","xyz"

opencsv and commons-csv parse row2 of h2 as follows:
^@Referral from Joe Smith.<A0> Fred is hard working.<A0> Super smart, though you wouldn't know it at first.<A0> 6 months, and we sold this project.<A0> Phooey he said to me!<A0> What's up with you people.<A0> You'll say anything for a sale!<A0> Until he met me of course....haar haar!
<A0>
Internet is spotty
Working while at home so.<A0> Will be applied this weekend. <A0>
<A0>
On Bill Recovery and 20 yr warranty added.
<A0>
Kindness made this deal happen!
<A0>

Spark  parse row2 of h2 as follows:
^@<span style=""font-family: tahoma

cc @HyukjinKwon

Test build #131848 has started for PR 30518 at commit b025271.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36445/

Test build #131848 has finished for PR 30518 at commit b025271.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Test build #131857 has started for PR 30518 at commit 1770c56.

Test build #131858 has started for PR 30518 at commit ca79a48.

Test build #131860 has started for PR 30518 at commit 1646adb.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36455/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36454/

Test build #131863 has started for PR 30518 at commit ca2900d.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36457/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/36460/

Merged to master.

Thanks @LuciferYang.

Test build #131857 has finished for PR 30518 at commit 1770c56.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131857/

thx @HyukjinKwon

Test build #131858 has finished for PR 30518 at commit ca79a48.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #131860 has finished for PR 30518 at commit 1646adb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131858/

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131860/

Test build #131863 has finished for PR 30518 at commit ca2900d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Build finished.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/131863/


I can't move the screen with the mouse

The installation went well, the file execution as well.
But when using the mouse to change screens or enter apps it just doesn't work

https://github.com/Genymobile/scrcpy/blob/master/FAQ.md#mouse-and-keyboard-do-not-work


details: fix missing files header

Missing header on files when restricted, see image

there is now unnecessary margin on the top of the box, below the header (white space), could you remove it?


Empty string hash fragment # is stripped from to prop

Version
5.2.0
Test Case
https://codesandbox.io/s/react-router-forked-z49on
Steps to reproduce
Create this link
<Link to="/about#">About</Link>
Expected Behavior
For the href rendered in the browser to be .../about#
Actual Behavior
The herf rendered in the browser is .../about with the # stripped.
Why
The empty string hash fragment # means scroll to top.
https://html.spec.whatwg.org/multipage/browsing-the-web.html#target-element
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/a#attr-href

You should use an a, not Link, to do this.

In https://github.com/rafgraph/react-router-hash-link, which wraps <Link> and provides hash link scroll functionality, I use # for scroll to top functionality as per the spec. react-router-hash-link receives the to prop first so it's able to know if # is present, but when it's render in the DOM the # is stripped which is confusing to the user, as well analytics tracking, etc.
Using an a tag only works if on the same the same path, or don't provide a path, otherwise it causes a full page load. If you use an a tag with href /about# on the /topics page it causes a full page load. Also using it with react-router-hash-link means you a get a lot of nice scroll functionality for free.
What's the reason for stripping it?
Seems kind of harmless to be spec compliant in this instance, and allow other libraries to add functionality that react-router has chosen not to implement.

So this is causing a bug in react-router-hash-link https://github.com/rafgraph/react-router-hash-link/pull/69 for <NavHashLink> because when the # is stripped the location doesn't doesn't match the to prop so the link isn't active.
@timdorr would you consider reopening this? Thanks.


BENCH-1228 Add updated data collections notebook

Refactor data collections notebook to guide users through creation of a new workspace, conversion to data collection, and publication of versions.

The working_with_data_collections.ipynb notebook isn't loading in both github and on jupyter for me. Please fix
File Load Error for working_with_data_collections.ipynb
Unreadable Notebook: /home/jupyter/repos/terra-axon-examples/first_hour_on_vwb/working_with_data_collections.ipynb NotJSONError('Notebook does not appear to be JSON: \'{\\n "cells": [\\n {\\n "cell_type": "m...')

I have now fixed this.

A few high-level comments:

it now makes sense to link somewhere in this nb to the doc on this now it's out, e.g. https://support.workbench.verily.com/docs/how_to_guides/creating_data_collections/#how-to-create-a-data-collection-and-manage-its-versions
Somewhat related, I think the discussion of policies could use a bit of rework.  You might want to link here: https://support.workbench.verily.com/docs/how_to_guides/creating_data_collections/#add-a-data-collection-policy
Basically, they can set group policies themselves via the web UI, ideally before they share the workspace, but need to contact us to set region policies.
You should also clarify (or point to docs) regarding the difference between setting a group policy, and sharing the workspace with users or groups, as they're 2 diff things.

A few high-level comments:

it now makes sense to link somewhere in this nb to the doc on this now it's out, e.g. https://support.workbench.verily.com/docs/how_to_guides/creating_data_collections/#how-to-create-a-data-collection-and-manage-its-versions
I have linked to this doc in the notebook section "Restricting discovery access to a data collection"
Somewhat related, I think the discussion of policies could use a bit of rework.  You might want to link here: https://support.workbench.verily.com/docs/how_to_guides/creating_data_collections/#add-a-data-collection-policy
I refactored the content about policies and linked to the indicated guide
Basically, they can set group policies themselves via the web UI, ideally before they share the workspace, but need to contact us to set region policies.
done
You should also clarify (or point to docs) regarding the difference between setting a group policy, and sharing the workspace with users or groups, as they're 2 diff things.
done
Regardless of whether there's a group policy, you probably want to spell out that once they've created a dc, they need to share the underlying workspace with the right target audience so that they can see the dc. (sorry if I missed that).
done


block process on alert ?

hi
i use swal in a _.each loop, is it possible to act like a real alert and so block the loop until i clicked the confirm or cancel button ? now it s just replacing the content of the error dialog so i only see the last "error" in my loop not each errors :/
thanks

Hi @smolleyes !
Thanks for submitting this issue! After some thinking I'm still not sure if this functionality really needed in swal2. Could you please describe you use case so I can better understand it.
Personally, as user, I'd rather not to have loop of popups. In Chrome for exaple you have prevent this page from creating additional dialogs checkbox.

reopen if needed


'function' object has no attribute 'model'

I am facing this issue while visualizing a decision tree using dtreeviz . I follow your demo of sklearn from READ.md. I am using vscode and download jupyter notebook as extension.
this is what I got after executing it
AttributeError                            Traceback (most recent call last)
Cell In[37], line 1
----> 1 viz_model = dtreeviz.model(tree_classifier,
2                            X_train=dataset[features], y_train=dataset[target],
3                            feature_names=features,
4                            target_name=target, class_names=["perish", "survive"])
AttributeError: 'function' object has no attribute 'model'

Hmm...must be an old version of dtreeviz?

this is my package details
Name: dtreeviz
Version: 2.0.0
Summary: A Python 3 library for sci-kit learn, XGBoost, LightGBM, Spark, and TensorFlow decision tree visualization
Home-page: https://github.com/parrt/dtreeviz
Author: Terence Parr, Tudor Lapusan, and Prince Grover
Author-email<EMAIL_ADDRESS>License: MIT
Location: c:\users\UserName\appdata\local\packages\pythonsoftwarefoundation.python.3.10_qbz5n2kfra8p0\localcache\local-packages\python310\site-packages
Requires: colour, graphviz, matplotlib, numpy, pandas, pytest, scikit-learn
Required-by:

@HarshKapadia I suspect it's an import issue. Please double check there, because your code sees the dtreeviz as a function, as it was in the old version of the library.
Maybe this could help also : https://stackoverflow.com/questions/75346507/dtreeviz-nameerror
If you followed all the above suggestions and still no success, providing a collab notebook or some sharable code to reproduce the issue would help us :)

I am also having the same issue while trying to visualize a representative tree of xgboost classifier.
AttributeError                            Traceback (most recent call last)
 in
----> 1 viz_model = dtreeviz.model(best_model_O,
2                            X_train=X_train_O, y_train_O=y_train_O,
3                            feature_names=X_train_O.columns.tolist(),
4                            target_name='Risk Level',
5                            class_names=best_model_O.classes_)
AttributeError: module 'dtreeviz' has no attribute 'model'
@parrt and @tlapusan any solution?


Unable to recompile with wider qubit type

HI! Thank you for developing this tool.
I was trying to run benchmarks with more than 128 qubits, and then the following message was shown by the program:
what():  Requested too many qubits to be handled by the DD package. Qubit datatype only allows up to 128 qubits, while 232 were requested. If you want to use more than 128 qubits, you have to recompile the package with a wider Qubit type in export/dd_package/include/dd/Definitions.hpp!
Thus, I changed the qubit type in Definitions.hpp from std::int_fast8_t to std::int_fast16_t and recompiled the package. However, the compilation was not successful and the following error message was provided:
qcec/extern/qfr/src/algorithms/Grover.cpp:41:69: error: no matching function for call to ‘max(long unsigned int, int)’
41 |         auto target = static_castdd::Qubit(std::max(nqubits - 1, 0));
Using other types does not work either. Am I missing something out?
Thank you!

Thanks for reaching out!
Could you try changing the line in the error message (i.e. qcec/extern/qfr/src/algorithms/Grover.cpp:41:69) to
auto target = static_cast<dd::Qubit>(std::max(static_cast<dd::QubitCount>(nqubits - 1), static_cast<dd::QubitCount>(0)));

and see if this resolves the error (or if other errors pop up)?
I am able to compile the code on my machine when changing the qubit type from std::int_fast8_t to std::int_fast16_t without any problems. This might be because the actual types underlying these std::int_fastx_t types are system dependent which might in turn lead to the compiler complaining since it can't find the definition for a certain standard library function (in the case above std::max).

Thanks for reaching out!
Could you try changing the line in the error message (i.e. qcec/extern/qfr/src/algorithms/Grover.cpp:41:69) to
auto target = static_cast<dd::Qubit>(std::max(static_cast<dd::QubitCount>(nqubits - 1), static_cast<dd::QubitCount>(0)));

and see if this resolves the error (or if other errors pop up)?
I am able to compile the code on my machine when changing the qubit type from std::int_fast8_t to std::int_fast16_t without any problems. This might be because the actual types underlying these std::int_fastx_t types are system dependent which might in turn lead to the compiler complaining since it can't find the definition for a certain standard library function (in the case above std::max).

This just works!
Thank you for your help!


Multicam synchronization without HW connections

Before opening a new issue, we wanted to provide you with some useful suggestions (Click "Preview" above for a better view):

Consider checking out SDK examples.
Have you looked in our documentations?
Is you question a frequently asked one?
Try searching our GitHub Issues (open and closed) for a similar issue.

All users are welcomed to report bugs, ask questions, suggest or request enhancements and generally feel free to open new issue, even if they haven't followed any of the suggestions above :)

Required Info

Camera Model
{  D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Linux (Ubuntu 14/16/17)

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C++ }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
I'm working with multi(3) D435s and the synchronization puzzles me. For the task is not that demanding in aspect of synchronization,  I would prefer carrying it out without HW sync. Then which timestamp should I check when aligning framesets from different source device? Is there any tricks about the poll_for_frame and wait_for_frame method over this problem?
What I've tried: Suppose the answer to the 1st question is back-end timestamp, I select a master cam and wait_for_frame of it at the start of every loop, and then use poll_for_frame to update the slaves(if it returns false, use the frameset of the former loop/cycle). If the time interval seems too long, I would wait_for_frame of one of the slaves.  (for example, the stream is 30Hz and if the frameset I poll comes over 20 ms earlier than master, I would  call wait_for_frame, so the slave's frameset comes later but with shorter time interval)

The subject of 'software sync' without hardware is discussed in the link below.
https://github.com/IntelRealSense/librealsense/issues/2148

Thanks. @MartyG-RealSense
To be more specific, what function/metadata should I call to get the host timestamp caculated from RS2_FRAME_METADATA_BACKEND_TIMESTAMP - (RS2_FRAME_METADATA_FRAME_TIMESTAMP - RS2_FRAME_METADATA_SENSOR_TIMESTAMP))

My interpretation of the very last comment by @dorodnic in the discussion that I linked to is that the RealSense developers incorporated the formula that you quoted into the SDK from version 2.22.0 onwards as.a feature called Global Camera Time.
https://github.com/IntelRealSense/librealsense/pull/3909


asynchronous processing event caused a claim data index error

asynchronous processing event caused a claim data index error.

already fixed

fixed


the cmake version file had a comment after the version number

and that effed up everything for me in #19 . Finally working, yay!

Fixed in #22


fix for #1260

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

Left a blocking comment for this PR. Also, could you add unit tests covering the lines of code added? The PR can't be merged if the diff coverage isn't greater than 100%. Thanks.

unit test failed to cover 100% because the tests/covalent_tests/dispatcher_plugins/local_test.py runs after the covalent server is started in the git actions. So I had written this test to pass even if the server was running and thought of letting you know afterward. If it is ok can I push the modified test case? However, it will fail until we change the git actions.

Left a blocking comment for this PR. Also, could you add unit tests covering the lines of code added? The PR can't be merged if the diff coverage isn't greater than 100%. Thanks.

unit test failed to cover 100% because the tests/covalent_tests/dispatcher_plugins/local_test.py runs after the covalent server is started in the git actions. So I had written this test to pass even if the server was running and thought of letting you know afterward. If it is ok can I push the modified test case? However, it will fail until we change the git actions.

@AlejandroEsquivel I have the changes that @FyzHsn suggested. Regarding the test coverage, as I mentioned I have written a test case to pass when a workflow is dispatched when the server is not running. As expected, the server didn't start in the git actions and the tests passed but codecov failed to meet its requirement. The reason behind this is it did not cover the code block which will execute only when the server is started. So is there a way to cover this code block without running the server?
Thanks

@kessler-frost and now this can be closed since you merged mine!

Tagging @santoshkumarradha here as well.


BIOM shape field shows incorrect number of rows

Thanks. Check if the latest commit fixes it for you! If not, please do re-open this issue.