optimization: let out.x have the same shape as x0

This is a suggestion for a backwards-incompatible change, so that's probably for SciPy 2.0.
All scipy optimization methods evaluate the objective function at points of shape (1,), even if the initial guess x0 is just a scalar. Likewise, the solution point is of shape (1,):
from scipy.optimize import minimize

def f(x):
    print(f"{x.shape = }")
    return x ** 2

x0 = 0.7

out = minimize(f, x0)

print(f"{out.x.shape = }")

x.shape = (1,)
x.shape = (1,)
x.shape = (1,)
x.shape = (1,)
x.shape = (1,)
x.shape = (1,)
out.x.shape = (1,)

I would like to propose that the shape of out.x and the evaluation points x should have the same shape as x0.

So in case of a scalar, you would like to return a scalar? I am not really in favour of the change. It's better to be consistent with the return types. We should actually do the opposite and force particular shapes at the input. Just like scikit-learn is doing.

Just like scikit-learn is doing.

Interesting. Can you point me to what they are doing?

We should actually do the opposite and force particular shapes at the input.

The domain of x can be anything (n dimensions), so we can't force anything at the input anyway. The solution of the optimization should be in the domain too, of course.

Just like scikit-learn is doing.

Interesting. Can you point me to what they are doing?

Here for instance. But they are doing this everywhere. You must feed an array of shape (n_samples, n_features). In the past they had a message inviting you to reshape and now they just raise if you don't do this.

We should actually do the opposite and force particular shapes at the input.

The domain of x can be anything (n dimensions), so we can't force anything at the input anyway. The solution of the optimization should be in the domain too, of course.

Sure, what I mean is to force a certain structure of the array and not allow different types of different shapes structure to be returned.

Well, I don't know the context of scikit-learn too well, but I can say that in optimization methods the structure of the domain is used nowhere. The only important thing is that it's an inner product space (in the sense that the operations +, -, scalar multiplication, and dot product make sense). Some methods don't even require the dot product. In principle, the shape of the x0 could be anything and all operations should work.

I am not sure I understand what you mean. Could you post some pseudo-code to explain what you propose and why it's important vs what we currently have? Also for breaking changes I suggest you use the mail list for that.

Let me adapt the original post.

@tupui I edited the original post and incorporated your suggestion to reject certain inputs (where necessary). I think the proposal makes more sense now, so thanks for the input.

Ok now I get it better, thanks for updating.
I think it's just not design from the start with that in mind. minimize assumes scalar inputs. The doc states: Minimization of scalar function of one or more variables. also look at the definition of x0 Array of real elements of size (n,), where ‘n’ is the number of independent variables..
So what you propose would require to implement some optimizer which handle vectorial inputs.

So what you propose would require to implement some optimizer which handle vectorial inputs.

All algorithms that don't require an inner product (Nelder-Mead and Powell, for example), can already do that with little or no change.

Yes but we would want to stay consistent along all optimizers. So it would need to be thought through in terms of API.
I would suggest you send an email to the dev mailing list with your proposal. Also state if you are actually proposing to do the work.

Yes but we would want to stay consistent along all optimizers.

Yeah, that's in the proposal, right? "Don't reshape the domain space, and reject input x0 for which there's no meaningful dot-product (if it's required in the algo)."

I would suggest you send an email to the dev mailing list with your proposal. Also state if you are actually proposing to do the work.

Will do.

minimize is for functions R^m -> R^1. As such I  think it makes sense that x has shape (m,). It doesn't make sense for x to be >1D because Hessians and gradients would have too many dimensions, and doesn't agree with R^m.
Whilst it's a pain if objective functions return more than a scalar it's not clear to me (from a pragmatic POV) what benefits are gained from x0 having the same shape as out.x.

Thanks Andy for taking a look!

minimize is for functions R^m -> R^1. As such I think it makes sense that x has shape (m,).

Right, that's a necessary restriction for some methods (namely those which require an inner product, e.g., CG, BFGS). The funny thing is that scipy will still accept other x0 (e.g., matrices) and will simply reshape them into "column-vectors" of shape (n, 1). I'd call the fact that this happens silently.. questionable. But realistically, that's probably not something we'll ever tackle. Other methods don't have that requirement (e.g., Nelder-Mead, Powell ) since they don't use an inner product. (They just scale, form averages etc.) Input points are still getting reshaped.

Whilst it's a pain if objective functions return more than a scalar

Well, optimization doesn't even make sense then. I think current scipy correctly raises and exception in this case. Anyway, this is a different issue.

what benefits are gained from x0 having the same shape as out.x.

For speed, there will be no difference. I'm only suggesting to remove some unnecessary reshapes. I think it's nothing more than natural if the solution out.x and all evaluation points x are in the same space as x0. That's the mathematical reality and could very well be reflected in code. The code would be cleaner, not only on scipy's side but also on the user side. For simple 1D minimization, I've always wondered why you need the [0] at the end:
from scipy.optimize import minimize

def f(x):
    return x ** 2 - 2

x0 = 1.5
out = minimize(f, x0)
sol = out.x[0]

For higher-dimensional problems, it saves a reshape at the end,
sol = np.,reshape(out.x, x0.shape)

This is actually my main motivation for suggesting the change.

I agree with @andfaff, and a -1 from me.  In short, x should be a strictly 1-dimensional array.
In more detail: Variables in a minimization problem should be a set.  Each variable has a scalar (and continuous) value and each must be unique.  You might consider that they have other attributes (name, bounds, standard error) and so be mapped Objects instead of plain floats -- we do this in lmfit, but it would be a large change to do that for scipy.optimize.   There might be benefits, but there would be costs too.
In principle the order of the variables does not matter (and any fit can reorder the variables and get the same result).  But it is completely reasonable to expect that an order will be maintained by a fitting algorithm.  In practice and historically (as the algorithms mostly originated in Fortran or perhaps early C implementation), it makes a lot of sense for the algorithm to combine these into a Vector or Array of floats that happens to map exactly to a 1D numpy array, and then it very much has a reliable order that cannot be changed and the only way to access individual variables is by indexing the array.
Importantly, a grouping of variables is not multi-dimensional: that second axis never has meaning, and there is never meaning to transposing or reducing the dimensionality in any way other than "flatten to strictly 1-dimension".
There is always ambiguity about whether arrays with 1-element are better interpreted as scalars.  For the specific problem here, please maintain that these, and all groupings of variables in scipy.optimize are 1-dimensional numpy arrays.


Implement the event horizon and ergosphere of Kerr black holes

🐞 What bugs you?
Event horizon is the point of no return in a black hole from which object can escape only with a speed greater than that of light.  In kerr black holes:

Formula for event horizon : RH = 0.5(Rs + sqrt(Rs2 - 4*a2))

See https://en.wikipedia.org/wiki/Kerr_metric#Important_surfaces

Formula for ergosphere :

See https://en.wikipedia.org/wiki/Kerr_metric#Ergosphere_and_the_Penrose_process
We can plot the erogosphere after the implementation.

🎯 Goal

Implement event_horizon() in utils/kerr_utils.py as a function of r, Rs, a and give option to return in radius in BL(Boyer-Lindquist) as well as Spherical Coordinates. BL->spherical can be implemented by doing BL->cartesian->spherical
Implement ergosphere() in utils/kerr_utils.py as a function of r, theta, Rs, a and give option to return radius in BL & spherical coordinates where Rs is schwarzschild radius.

Coordinate conversion utils are already implemented
📋  Steps to solve the problem

Comment below about what you've started working on.
Add, commit, push your changes
Submit a pull request and add this in comments - Addresses #114
Ask for a review in comments section of pull request
Celebrate your contribution to this project 🎉

Solving this issue is a step towards fixing #109

Fixes #109


Editing the Programming Exercise Timeline Makes the Viewport Jump to the Top of the Page

Describe the bug
When creating/editing programming exercises and editing the timeline options, the viewport jumps back to the very top of the page.
To Reproduce

Create/edit a programming exercise.
Change the programming language to Python, C, or Empty.
Edit the timeline options.

Expected behaviour
The screen should stay in the same position while editing. It does so in Chromium-based browsers.
Screenshots
https://user-images.githubusercontent.com/64250573/149141592-41db181f-fa30-4736-b4e1-d08e10e71ec8.mp4
What browsers are you seeing the problem on?
Firefox 96
Additional context
On current develop branch.
When using a programming language that has a package name setting, after closing the date-picker the scrollbar of the browser jumps to the very top, but then jumps back to the correct location.
Relevant log output
none

Can no longer be reproduced on Firefox 100 and Artemis 5.7.5.


Does TensorRT-LLM support input_embeds as input?

Does TensorRT-LLM support input_embeds as input?

Similar issue: https://github.com/NVIDIA/TensorRT-LLM/issues/2211#issuecomment-2370787495


Can't set management group parent using resource reference

Bicep version
Bicep CLI version 0.3.539 (c8b397dbdd)
Describe the bug
When creating a child management using Bicep, I'd expect to be able to pass the reference to the parent management group using the symbolic name of the parent management group. However, this doesn't work. I need to instead use the { id = parentManagementGroup.id } syntax.
To Reproduce
Template to reproduce the behaviour:
targetScope = 'tenant'

resource parentManagementGroup 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-parent'
  properties: {
    displayName: 'Example parent'
  }
}

resource childManagementGroup1 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child'
  properties: {
    displayName: 'Example child'
    details: {
      parent: parentManagementGroup // This does not work.
    }
  }
}

resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child-2'
  properties: {
    displayName: 'Example child 2'
    details: {
      parent: {
        id: parentManagementGroup.id // This does work.
      }
    }
  }
}

Notice that for childManagementGroup1, the parent property is a reference to the parentManagementGroup resource. When this is deployed, the parent-child relationship is not set. However, for childManagementGroup2, the .id property is used instead, and this successfully sets the parent-child relationship.
Additional context
Add any other context about the problem here.

This is correct behaviour. Bicep is unable to detect what that ID actually refers to - it knows only that property of management group expects a type string under key 'id'. what is expected by each resource provider is delivered by openapi (swagger) specs by each Azure team.
The parent/child relationship in bicep relates to the child types (like vnets->subnets). Here it's more a logical connection between 2 resources that depends on the particular resource logic, not the deployment itself.

I think this is a duplicate of #285. IMO if we want to support this with a high degree of accuracy, we would need a swagger annotation from RPs to mark places where a direct resource might be permitted. That unfortunately would a pretty huge effort.
A more general idea I was exploring a while back is to allow a shorthand for declaring an object with a single property by allowing . chars in object keys to auto-insert objects, e.g. so all of the following are semantically equivalent:
Original:
resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child-2'
  properties: {
    displayName: 'Example child 2'
    details: {
      parent: {
        id: parentManagementGroup.id
      }
    }
  }
}

Also works:
resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child-2'
  properties: {
    displayName: 'Example child 2'
    details: {
      parent.id: parentManagementGroup.id
    }
  }
}

Further nested:
resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child-2'
  properties: {
    displayName: 'Example child 2'
    details.parent.id: parentManagementGroup.id
  }
}

If there's interest in the above, I'd be happy to put together a formal proposal.
Alternatively, if we get around to doing #146, we could at least save some whitespace:
resource childManagementGroup2 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child-2'
  properties: {
    displayName: 'Example child 2'
    details: {
      parent: { id: parentManagementGroup.id }
    }
  }
}

Sounds good @anthony-c-martin.
I think one suggestion is - if this isn't going to be supported (and it's reasonable that it isn't), then is there anything we can do to tell the user quickly that they can't specify a resource, and they need to instead populate the .id property?
For example, can we return an error when you try to do this:
resource childManagementGroup1 'Microsoft.Management/managementGroups@2020-05-01' = {
  name: 'example-child'
  properties: {
    displayName: 'Example child'
    details: {
      parent: parentManagementGroup // This does not work.
    }
  }
}

For example, can we return an error when you try to do this:

Absolutely - we should already be blocking this; looks like a bug in the type checker!

looks like a bug in the type checker!

@anthony-c-martin - do you mind creating a new bug for this? I'm not exactly sure how to phrase the issue.
Going to close the issue since it is a dup of #285


Most basic display of how to revert to a previous PR

This does not have tests and does not necessarily look good. Feel free to pull in these changes though and style as you wish. This will provide your command to revert to a specific PR in the production branch from any of our developer's local machines.

This is explicitly reverting to the PR that is immediately above the line. Could use some styling/verbiage to better indicate that.

So this is if I need to rollback, I should use that? Or when I want to revert the rollback?
I'm not comfortable with this as is, because it isn't clear what it's for.

The command is to go to the point that the specified PR is at. As indicated in the description, this could use some slightly improved verbiage/styling to make that clearer.

Essentially saying "I want production to go to THIS point in its history".

I know I can work on styling differently later. But having something no one understands is arguably not much better than not having it at all, you know? That's why I care about the way it's worded, at least.

This PR is not required to be merged as is. It is simply proposing a solution that can be adapted for what makes the most sense for your team.

Verbiage updated, screenshot reflects that.

PR switched to a rollback.

Ok, so this is to rollback that deployment, and not have us in a screwed up state where it's difficult to redeploy, right?

That makes sense to me.

Merged it into my own branch https://github.com/usbsnowcrash/ruby-deployments-dashboard/pull/34


Error whilst importing .pmx

no idea how to fix

我也遇到了一模一样的问题

Thanks for the report, this is now fixed in the new release version 0.19.0.


RTL Support

Hello,
I want to is there any RTL support?

@AGhasemzadeh No yet. Do you know any framework or library has RTL support?

yes,
Qusar,Bootstrap,Zurb f6

Hello
I want to have RTL off-canvas but I after I fix all CSSs I can not see sidebar.
This is my SASSS code.

`// Off canvas menus
$off-canvas-breakpoint: 960px !default;
.off-canvas {
display: flex;
flex-flow: nowrap;
height: 100%;
position: relative;
width: 100%;
.off-canvas-sidebar {
background: $bg-color;
bottom: 0;
right: 0;
min-width: 10rem;
overflow-y: auto;
position: fixed;
top: 0;
transform: translateX(100%);
transition: transform .25s ease;
z-index: $zindex-3;
}
.off-canvas-content {
flex: 1 1 auto;
height: 100%;
padding: $layout-spacing 4rem $layout-spacing $layout-spacing;
}
.off-canvas-overlay {
background: rgba($dark-color, .1);
border-color: transparent;
border-radius: 0;
bottom: 0;
display: none;
height: 100%;
right: 0;
position: fixed;
left: 0;
top: 0;
width: 100%;
}
.off-canvas-sidebar {
&:target,
&.active {
transform: translateX(0);
}
&:target ~ .off-canvas-overlay,
&.active ~ .off-canvas-overlay {
  display: block;
  z-index: $zindex-2;
}

}
.off-canvas-toggle {
display: block;
right: $layout-spacing;
position: absolute;
top: $layout-spacing;
transition: none;
z-index: $zindex-0;
}
}
// Responsive layout
@media (min-width: $off-canvas-breakpoint) {
.off-canvas {
.off-canvas-sidebar {
flex: 0 0 auto;
position: relative;
transform: none;
}
.off-canvas-content {
  padding-right: $layout-spacing-lg;
}

.off-canvas-toggle {
  display: none;
}

}
}
`

Material UI supports RTL. They use jss-rtl to flip every rule related to x-axis and that's almost everything you need to support RTL unless there are javascript animations. For example:
float: right
right: 0.5rem
padding: 1rem 0.5rem 1rem 0

becomes:
float: left
left: 0.5rem
padding: 1rem 0 1rem 0.5rem


Bug fixes to AutoGEMM and DTRSM, DTRTRI

Fixed bugs in autogemm when "beta == 0"
Fixed bugs in DTRSM (and DTRTRI) caused by missing barriers.

@kknox @TimmyLiu This PR fixes all the bugs we are seeing in release mode with arrayfire. There is still another bug that only occurs in debug mode that we are investigating with a lower priority.

@pavanky thanks! Did you have a chance to do some performance benchmark? I am nervous about adding if statements within the kernel. It may add more cycles. But I am not sure since all thread (beta==0 or not ) should execute the same path.

@TimmyLiu We haven't explicitly benchmarked the code, but I don't think the if conditions add overhead because all threads take the same execution path. There is no thread divergence.

@pavanky I agree. Let me double check that it doesn't add more registers either. The performance can be sensitive to register count as well.


convert API to v1alpha1 from v1

Description of the Change
Convert the api/v1 package to api/v1alpha1 and convert CRD versions from v1 to v1alpha1
Motivation for the Change
This operator is still a work in progress and it hasn't been decided that this API is stable yet. Having an API version of v1 makes it seem that the API is stable and won't change significantly. Changing it to v1alpha1 conveys that this operator and its API is still in the alpha stage and is prone to change.

/approve


POST /flag/add endpoint

POST /flag/add
Parent Issue

#563

Steps
Branch - feature-flag/add

[ ] Create flag.js file for route, controller and models
[ ] Set /flag/add authentication to super_user and add /flag route to routes/index.js
[ ] Create controller and model (function name - addFlag for both)

Example of object to be added
title: dark-mode
enabled: true/flase
accessibility: {
  roles: [members, no_role, super_user],
  users: {
    "pallab id": true,
    "rohit id": false,
  },
}

Branch - feature-flag/add-tests-validator

[ ] Write integration and unit test
[ ] Add joi validation middle ware

Assign me this issue please!!

Reopening the issue as PR is in progress

Now Handled in the repo

https://github.com/Real-Dev-Squad/feature-flag-service


[go-machine-service] Adding machine driver fails with checksum error

Rancher Version:
v0.63.0
Docker Version:
1.9.1
OS:
Ubuntu 14.04-64bit
Steps to Reproduce
Add a new machine driver using the following parameters:
Download URL: https://github.com/janeczku/docker-machine-vultr/releases/download/v1.0.5/docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz
MD5 Checksum: 409c17282982c169c379645336b3caa4

Results
Fails with checksum error:

level=error msg="Error while downloading and verifying: https://github.com/janeczku/docker-machine-vultr/releases/download/v1.0.5/docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz Checksum provided: 409c17282982c169c379645336b3caa4 does not match calculated checksum : ece3d622b2d95f2f2d18074c6a5ced16 for driver with uri: /tmp/gms-010906735/docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz"

Expected
No error. The supplied checksum is correct.

@janeczku Lies? :smile:
mercury:tmp vincent$ curl -L -O https://github.com/janeczku/docker-machine-vultr/releases/download/v1.0.5/docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   629    0   629    0     0    642      0 --:--:-- --:--:-- --:--:--   641
100 2804k  100 2804k    0     0   944k      0  0:00:02  0:00:02 --:--:-- 2232k
mercury:tmp vincent$ md5 docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz
MD5 (docker-machine-driver-vultr-v1.0.5-linux-amd64.tar.gz) = ece3d622b2d95f2f2d18074c6a5ced16

Oh, your published list is the md5 of the .tar, not the .tar.gz that's actually downloaded.
BTW we used your driver for QA to test custom drivers, but apparently they didn't try the supplied checksums :smile:

:stuck_out_tongue_winking_eye: Haha, golden. Apparently Safari gunzips .tar.gz downloads and then stores them as .tar. And i almost believed in a checksum conspiracy....

Works as expected after supplying the correct checksum.


feat: implement transferable proxy pattern

since our marketplace contract does not yet hold any state, and only we will be using this contract, I am converting this into draft for now.


jekyll: 4.3.3 -> 4.3.4

https://github.com/jekyll/jekyll/compare/v4.3.3...v4.3.4
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351729

x86_64-linux

:white_check_mark: 3 packages built:

jekyll
jsonnet
jsonnet.doc

aarch64-darwin

:white_check_mark: 3 packages built:

jekyll
jsonnet
jsonnet.doc


fix: Sorting volume size by bytes size

Closes #94

Hi @Lalit3716, thank you very much for your contribution! The volumes are now successfully sorted by bytes size.
https://user-images.githubusercontent.com/15997951/199683936-df1fed2b-2499-42ff-ad6c-397306dda696.mov

Hi @Lalit3716 and thank you for stellar your contribution! In order to send you some Hacktoberfest-related swag, could you please send me an email to<EMAIL_ADDRESS>with your postal address and your t-shirt size? Thank you so much!

Done @AlbaRoza 🚀!


fix(file) populate port according to protocol in url

Populate port according to the protocol scheme in the url sugar
attribute. Default port of 80 is now populated to match Kong's
behavior in the same case.

cc @rainest


Updated package setup to use the latest mesh builder and install peer deps

Description
Using the latest mesh builder package and moving common deps to the CLI.
Related Issue
CEXT-2432
Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

seems some peer dependencies has to be added to get the build success

After Kris M's changes, the PR looks stable and working as expected. I have verified that the functionality is working as expected. Ready to merge.

to fix the string width error had to add resolution for jackspeak https://github.com/storybookjs/storybook/issues/22431


All locations of the uploader should support drag and drop

Is your feature request related to a problem? Please describe.
I'm always frustrated when I forget not all parts of the uploader supports drag and drop and the file I drag gets opened instead
Describe the solution you'd like
I'd like for the area where the added files thumbnails are to support drag and drop, I want when I drag and drop a file there its added and not opened

I do not understand exactly what you mean and what component, if it is the upload works properly when dragging an image

For convenience and user experience, the area where the thumbnails are displayed should also accept "drop" just as the area where it says "Click or drag files here"

it would even be better if it...

Also, when when the file limit is reached the upload area should be hidden until one or more items get removed
In the example below where the file limit is set to 1, "click here to upload photo" should be hidden until the file is remove (by clicking the X) instead of being disabled.

From a UX/UI perspective, these tips increase visual appeal

was updated and added many of the features, thank you very much for the contribution and help in the ux and ui
We did it similar to what you suggested, very good idea


Update dbeaver-community to 4.0.6

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

dbeaver-enterprise doesn't have a 4.0.6 release yet.

Thanks @fanquake.


[Request]:  a downlaod speed limiter , a donlwoad speed planifer

Is your request related to a problem?
No
How is the feature request related to the problem?
No response
Describe your proposed solution
i have a low internet connection i want to limite my download speed , i also want to set the speed for example 500kb/sec in the day
and 1mb/sec at night
Describe alternatives you've considered
no alternative
Additional context
No response

Hi, we've been tracking this request in #394, so feel free to check that for more info. Closing for now, thanks for your interest in the project!


New version: oneAPI v0.1.2

Registering package: oneAPI
Repository: https://github.com/JuliaGPU/oneAPI.jl
Created by: @maleadt
Version: v0.1.2
Commit: 87590fc3528cbd3a6b9d92c473ddf33b3c17fd9e
Reviewed by: @maleadt
Reference: https://github.com/JuliaGPU/oneAPI.jl/commit/87590fc3528cbd3a6b9d92c473ddf33b3c17fd9e#commitcomment-48344818

@maleadt license is missing :slightly_smiling_face:
[noblock]


Simplify datetime/timedelta check

As we already know that arr must be an array, simply check the array's type to see if it is datetime/timedelta. Thus this avoids the check of whether buf is an array.
TODO:

[ ] Unit tests and/or doctests in docstrings
[ ] tox -e py37 passes locally
[ ] tox -e py27 passes locally
[ ] Docstrings and API docs for any new/modified user-facing classes and functions
[ ] Changes documented in docs/release.rst
[ ] tox -e docs passes locally
[ ] AppVeyor and Travis CI passes
[ ] Test coverage to 100% (Coveralls passes)

Hello @jakirkham! Thanks for submitting the PR.

In the file numcodecs/compat.py, following are the PEP8 issues :

Line 37:80: E501 line too long (84 > 79 characters)
Line 52:80: E501 line too long (85 > 79 characters)
Line 62:80: E501 line too long (80 > 79 characters)
Line 63:80: E501 line too long (85 > 79 characters)
Line 65:80: E501 line too long (81 > 79 characters)
Line 69:80: E501 line too long (82 > 79 characters)
Line 76:80: E501 line too long (80 > 79 characters)
Line 86:80: E501 line too long (85 > 79 characters)
Line 94:80: E501 line too long (84 > 79 characters)
Line 95:80: E501 line too long (86 > 79 characters)
Line 96:80: E501 line too long (83 > 79 characters)
Line 97:80: E501 line too long (80 > 79 characters)
Line 115:80: E501 line too long (85 > 79 characters)
Line 123:80: E501 line too long (89 > 79 characters)
Line 127:80: E501 line too long (84 > 79 characters)
Line 141:80: E501 line too long (84 > 79 characters)

Sorry for pep8 noise, I've created .pep8speaks.yml here too now.


Feature/parsing cases

Codecov Report
Patch coverage: 94.16% and project coverage change: +2.15 :tada:

Comparison is base (19cce75) 86.39% compared to head (b8f4363) 88.55%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   86.39%   88.55%   +2.15%
==========================================
  Files           5        6       +1
  Lines         669      769     +100
==========================================
+ Hits          578      681     +103
+ Misses         91       88       -3

Flag
Coverage Δ

unittests
88.55% <94.16%> (+2.15%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/bioconverters/utils.py
94.00% <94.11%> (+4.44%)
:arrow_up:

src/bioconverters/constants.py
100.00% <100.00%> (ø)

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

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

@jakelever I am converting this to a draft pending approval for the other parser updates in #16


Blocks that shimmer when inside an electric field

Not sure if electric fields have been a thing in modded minecraft for a while, but the Tesla coil introduced by Immersive Engineering seems to generate a field which causes Fluorescent Tubes placed within that field, to shimmer.
This is a request to create a block (similar to the glowing FCB blocks) that will shimmer when placed within  an electric field.
thanks,

I'm not sure this makes sense for FCB, FCB tends to be a pretty basic 'vanilla' type mod, and this feature would require IE.
I'm not sure there is even a proper way to tie into them... but either way the first point stills stands, I'm not sure it belongs here.

Oh ok.  I asked blu as well.  You can read his response here.
https://github.com/BluSunrize/ImmersiveEngineering/issues/1733
Main reason why I asked it from FCB is because for my use case I intend to chisel the block, so for me the block has to be chiselable.  That way I can pepper the bits around my artwork and turn on/off groups of bits by switching on/off the telia coil.
Btw what is it about a block that makes it chiselable?  I've tried chiseling blocks like the scaffolding from I.E. but no joy.   Does a block have to opt in somehow?
Thanks

Blocks must basically be "normal full size with nothing special" Even if I did manage to add this feature to FCB, I would also have to add to C&B for it to work with it, and that is rather unlikely, because I'm trying to keep C&B away from potential performance issues.
As for options of swapping things around or animating, I hear you can use blood magics teleposer to move them around, maybe there are other blocks like that too that can help.

I see.  Thanks anyway for considering it.  Also I've learned much from this dialogue.
Would it be possible to add the IE scaffolding blocks to C&B... if they are not too special that is.    I think at least one of them has connected textures.   Does that disqualify it as chiselable?

It doesn't meet the basic criteria in some what, hard to say which one, or why.   Connected textures doesn't blatantly blacklist it, but they generally won't work.
If they are going to get white listed however @BluSunrize would likely be the one to ask, I don't white list peoples stuff on my side, if they want their things to work, they are free to do so however.

I believe I already stated that it's basically impossible to make a block texture "light up" next to a Tesla Coil. The only reason the Fluorescent Tube can, is because it's an entity render and has access to OpenGL context. Blocks don't.
There is little help in just carrying the problem to another moddev after I have already explained the nigh-impossibility of making this happen.

Sorry for the confusion Blu, I wasn't referring to that, I was referring to his request just above my last comment
"Would it be possible to add the IE scaffolding blocks to C&B... if they are not too special that is. I think at least one of them has connected textures. Does that disqualify it as chisel-able?"

Algo, I'm aware of why I was initially summoned. But the base issue that started this thread is something that I had already answered. And I found it quite rude to bother another developer with something I had already declared a technical impossibility.
I don't want people to bother you because I declined a request ^^

I guess, tho from the times it seems he asked both of us around the same time originally, I just hadn't replied until recently.

Sorry Blu, I was unaware that you maintained all knowledge about all animation techniques, and that no one else could possibly have alternative ideas.
I also much appreciate the fact that you've taken the role of acting as a Mother to all modders and you're now mediating discussions between players, and identifying rude behavior.  Apologies for my rudeness.
From this point forward, I will make it a point to contact you first for all my minecraft questions.   And if you say "technically impossible", I'll stop asking others.
Thanks again Blu
Algo, I'm already ashamed of my response above, but not enough to refrain.  Sorry I couldn't' resist, as I've seen this attitude from Blu on other threads.


make kubeless install more gently

If kafka-controller or kubeless-controller existing in the cluster, kubeless install should just print out a warning message instead of error and keep finishing the job. This should be very easy to do. You wanna take it @dbarranco?

sure @ngtuna.
Thanks!

we now removed kubeless install in favor of a kubectl create a manifest available in the release.


[Add]

Details
when hover the button will slide out like a brick
Screenshots

please assign this issue to me @Spyware007 @Souravvmishra
Type of Contribution

[ ] Update to an existing Animated Button
[X] Adding a new Button
[ ] Resolving a bug
[ ] Proposal to the Repository
[ ] Changes related to documentation or README.md
[ ] Other Changes

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue
[X] I am a GSSoC'23 contributor

Please check #1346

Would complete in a week thanks.

Sorry for the inconvinience, but we are closing all the issues! Please open a new issue if you want to contribute anything. Also now you can create a new button from the website itself! You will get points for those too! I will add the instructions on how to create a new button soon and create an empty pr for the same to get points!


Parse array types

Codecov Report

Merging #6 (724d5cc) into main (8567293) will increase coverage by 0.62%.
The diff coverage is 89.36%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
+ Coverage   79.34%   79.96%   +0.62%
==========================================
  Files           3        3
  Lines         644      654      +10
  Branches       14       14
==========================================
+ Hits          511      523      +12
+ Misses        127      125       -2
  Partials        6        6

Files
Coverage Δ

src/Escalier.Parser/ExprParser.fs
86.44% <89.36%> (+0.56%)
:arrow_up:


Improves backwards compatibility for safari

fixes #1897

that's right! no idea why though. probably also doing some styling via style attributes 🤷♂ . Switched to using classList.add and .hidden class which should've been done anyway. Waiting for TravisCI to come back from maintenance.

@MichMich ready to merge

Thanks. The reason why I haven’t merged yet is because I want to check out the code behind the header. If we aren’t going to display the header. Why add it to the dom? Adding a hidden class feels like a bandage on a issue that should be solved else where. There might be a good reason to do it this way, but I want to confirm before I merge this.
Thoughts?

I think this has been answered in the original issue https://github.com/MichMich/MagicMirror/issues/1897#issuecomment-577555074

As it seems, the problem with the style property is that it is considered read-only. Correctly, one should not write directly to style, but set individual sub-properties of style. This means:
element.style = "display: none;";
should become
element.style.display = "none";
Older versions of Safari apparently are more strict in this regard.

I think @MichMich is more reffering to wheather headers should be in the DOM at all. I understand @tbouron intent with doing so. I'm just here to help people reuse older devices instead of throwing them away. Hopefully there is a way of having everyone happy.

In fully understand that @MichMich is checking whether unnecessary headers can simply not be created instead of being created and then hidden. With my above reference to @tbouron's  comment I only intended to point @MichMich to why the current approach apparently had been implemented.
@sejka, I totally share your intention to improve compatibility with older browsers. I myself want to use MagicMirror in serveronly mode and use an old 3rd generation iPad running on iOS 9 as client. I tried to find out why some code does not work in Safari 9. This is what I have found out so far.

As mentioned above, Safari 9 is strict with HTMLElement.style being read-only. Therefore one should replace element.style = "display: none;"; with element.style.display = "none";.
Safari 9 does not understand ES6 arrow functions. They should therefore be replaced by ordinary functions.
Safari 9 does not understand ES6 let statements. They should be replaced by var statements.
Safari 9 does not understand ES6 concise method definitions as object properties. A definition such as obj = { foo(a, b) { … } }; should be replaced by obj = { foo: function(a, b) { … } };.
Safari 9 does not understand ES6 default parameters. I suggest replacing function(param = default) { … } by function(param) { param = param || default; … }.
Safari 9 does not understand class declarations. They should be replaced by ordinary class expressions.

Probably this list is incomplete. But in my fork (branch improve-compatibilty-with-old-browsers) of MagicMirror applying these replacements was enough to obtain a serveronly version running without apparent problems in Safari on my iPad running iOS 9.
I do not want to compete with your PR. I just wanted to share my results.

Wouldn't it be much better to use a transpiler (like babeljs or something) for compability in older browsers than forcing the code to be stuck in the past (IE6 flashbacks...)?

great work @aghster!
I think in most cases we'd like to use newest features of JS and not make devs think twice before using a feature. Honestly I didn't think there's so many changes - that's why I've only changed that one thing.
I think we have two separate conversations going on - to have headers in the DOM despite them being empty and general backwards compatibility for browsers.
When it comes to second one i totally support @buxxi approach with babel.

looked into babel a little bit - i think we would need to ditch class.js for es6 classes though. Also found this issue: https://github.com/MichMich/MagicMirror/issues/694 which suggests that @MichMich isn't interested in complicating codebase that much. Maybe something changed though.

Ditching custom class-creation for es6 classes doesn't sound like complicating it :smile:
Guessing babel would be a part of the gruntfile if it would be used. Could also be part of a specific Dockerimage to avoid having the core of MagicMirror have to support older browsers (and require a build-step) since the main target is electron.
But there really should be something in the Contributing Guidelines about what browsers it needs to work in or what features of JS can/can't be used.

Indeed. We will not use any pre-compilers. (see https://docs.magicmirror.builders/getting-started/manifesto.html) Sorry :)

I understand. What's your recommendation for users recycling their old devices for that use case? Should we create MagicMirrorLegacyDevices fork? Are we adding additional build step for legacy devices users? Maybe another docker image since most of older devices users are running host-client setup anyways?

You can always run an older version of MagicMirror.

Where will I get older version of magic mirror sorry for this silly
question but I am new to this
On Wed, 1 Apr 2020, 13:13 Michael Teeuw<EMAIL_ADDRESS>wrote:

You can always run an older version of MagicMirror.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MichMich/MagicMirror/pull/1908#issuecomment-607088409,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJTYL3T65H2ZXVPRRC5KXH3RKLWBZANCNFSM4KMJ3XWQ
.

Closing this for now because I don't see where we go with this. Feel free to reopen.


How to increase the number of columns in the generation of a file in xls

I need to generate an Excel file that will have approximately 1000 columns. But at the time of generation  can only generate 256 columns.
I've tried almost everything: I increased memory_limit, max_execution_time, ... But nothing worked so far.
Versions

PHP version: 7.0.26
Laravel version: 5.6
Package version: 2.1

Here is test code where I try to generate 1000 columns.
PHP
Excel::create("Test_A", function($excel) {

    $excel->setTitle("Only test");

    $excel->sheet("Test 1", function($sheet) {

        $sheet->loadView('excel');

    })->export('xls');

});

VIEW
<!DOCTYPE html>

    <meta charset="UTF-8">
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <meta http-equiv="X-UA-Compatible" content="ie=edge">
    <title>Test 1</title>

                @for ($i = 0; $i < 1000; $i++)
                    {{ $i }}
                @endfor

I found the answer. I need to simply create an XLSX file and not an XLS file.


Add NXP iMX RT1062 SVD file

This chip is used in the Teensy 4.0. I obtained the file from the SDK.

Thanks


Update to liberty arqillian plugin 2.1.0 and arquillian 1.7.0.Alpha12

Changes look good, but they won't work until the 2.1.0 release of Liberty Arquillian is published.


errors installing jsx (docker) on Mac

JSX Branch development
Following errors are thrown when installing jsx in a docker on mac. The installation succeeds afterwards
Error processing line 1 of /Users/dylan/Library/Python/3.7/lib/python/site-packages/zerohub-nspkg.pth:

  Traceback (most recent call last):
    File "/usr/local/Cellar/python/3.7.2_2/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site.py", line 168, in addpackage
      exec(line)
    File "<string>", line 1, in <module>
    File "<frozen importlib._bootstrap>", line 580, in module_from_spec
  AttributeError: 'NoneType' object has no attribute 'loader'

Remainder of file ignored
Error processing line 1 of /usr/local/Cellar/python/3.7.2_2/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site-packages/zerohub-nspkg.pth:

  Traceback (most recent call last):
    File "/usr/local/Cellar/python/3.7.2_2/Frameworks/Python.framework/Versions/3.7/lib/python3.7/site.py", line 168, in addpackage
      exec(line)
    File "<string>", line 1, in <module>
    File "<frozen importlib._bootstrap>", line 580, in module_from_spec
  AttributeError: 'NoneType' object has no attribute 'loader'

Remainder of file ignored

can you add steps lead to this case ?
is it a fresh installation, or there was already code on the machine

Fresh installation, no code on my machine

Is the installation with sudo ?

no

can you pull and try again ?
also if you may add your installation steps you used.

I pulled and tried again, had the same result. JSX installed did succeed afterwards.

Errors occurs only with my Python version. Which might be corrupt.


Plotsquared/reworking

Overview
Reworking of some PlotSquared docu pages
Fixing #44
Description

Reworking README, adding cover picture
Removing .yml from the page name of the setting docs
Small markdown typo
Fixing links
Updating infos
Renaming "Changelog v4 & v5" page
Updating "Why PlotSquared" page
Some typos

EDIT (06.01.2023)

Removing commands already removed from plugin
Updating source-code link to main branche

### Submitter Checklist
- [X] Make sure you are opening from a topic branch (**/feature/fix/docs/ branch** (right side)) and not your main branch.
- [X] Ensure that the pull request title represents the desired changelog entry.
- [X] New public fields and methods are annotated with `@since TODO`.
- [X] I read and followed the [contribution guidelines](https://github.com/IntellectualSites/.github/blob/main/CONTRIBUTING.md).

Can this be merged? I am already reworking the FAWE articles.

friendly ping @IntellectualSites/plotsquared-team :)


Angular is deprecated - please migrate to React

Hello :wave: from Grafana Labs
We have detected that your plugin might still be using AngularJS and recommend migrating it to React, so that it may continue to operate successfully in future versions of Grafana.
You can read more on why and how by visiting our documentation: https://grafana.com/docs/grafana/latest/developers/angular_deprecation/
You can reach out to us with any questions via slack: slack.grafana.com - channel #plugins or our plugin development community forum

Do the developers of this panel consider migration to React?

Yes it is done


[feature to main branch] Add binary format support with HNSW method in Faiss Engine

Description
https://github.com/opensearch-project/k-NN/pull/1815
https://github.com/opensearch-project/k-NN/pull/1781
Issues Resolved
N/A
Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed as per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

looks good, will approve once CI is finished


Update: improve instructions for some challenges

Improved the wording of a few of the js-loops challenges, adding better clarity.
This is based on grammatical rules as well as on feedback from students.
Also removed some irrelevant instructions from README files: no need to tell the students to install dependencies with npm i, when there are no dependencies listed in the package.json.

[...] Also removed some irrelevant instructions from README files: no need to tell the students to install dependencies with npm i, when there are no dependencies listed in the package.json.

I see the point. The reason for this unnecessary npm i is the migrate.js script: it makes sure that all challenges derived from the same template looks the same.
Every time we'll change the general structure of a template (in the case of the challenges involved here, the "js" template), we should run the migrate script again to ensure consistency.
The problem is the following: if we decide to change another template, we need to run the migrate script as well, but this will reset the changes suggested by this PR.
As far as I understand this, deleting these lines will be reset until the run the migrate script again because we have changed a template.
I think we should improve the migrate script instead: make sure that the npm i command is not part of the README.md file if there are no dependencies.

Thanks, @doemser and @PReichetanz. Sounds like an issue to be solved elsewhere. For now, I'll remove those changes.


AMPI: Clean up ROMIO configure step parameters

build ChaNGa still doesn't work.

Is it the same error message as before? Also, does ./buildold ... work?

If the commits I just pushed don't fix the issue, I'm stumped.


rustfmt fails with errors "left behind trailing whitespace"

I have rustfmt-preview installed on the nightly toolchain (rustc 1.30.0-nightly (3edb355b7 2018-08-03)) using rustup.
Running cargo +nightly fmt on the crate: https://github.com/frankmcsherry/differential-dataflow results in numerous "left behind trailing whitespace" errors. Some sample errors:
internal error: left behind trailing whitespace
  --> [..]/differential-dataflow/src/algorithms/prefix_sum.rs:70
   |
70 |         .iterate(|ranges|
   |                          ^
warning: rustfmt may have failed to format. See previous 1 errors.

  --> [..]/differential-dataflow/examples/seb.rs:37
   |
37 |       let cooccurrences =
   |                          ^
  --> [..]/differential-dataflow/examples/seb.rs:44
   |
44 |       let row_sums =
   |                     ^

  --> [..]/differential-dataflow/examples/dataflow.rs:35
   |
35 |             let (tweet_input, tweets) = scope.new_collection();
   |                                                                ^

rustfmt is failing due to this line.

wrap_str checks the width of formatted code, and veto if it exceeds max width. Currently it does not distinguish between normal code and comments. I think that it should skip checking the width of the comment.
https://github.com/rust-lang-nursery/rustfmt/blob/b2a80f99426363c092968ef1b4e771c159d7aa1b/src/utils.rs#L350-L358

Thanks! Looks like there was another issue when comment is between closure's argument list and its body.
E.g.
|foo|
// comment
foo + 3;

You're welcome. Before rustfmt 1.0, is there any plan to run https://github.com/rust-lang-nursery/crater but for rustfmt? That would actually highlight any bugs like this and also ones that result in compilation/test failures.

With the file below, the error is a bit different, but also more informative:
fn main() {
    let toto = |foo|
        // my comment
        foo + 3;
}

The error:
internal error: not formatted because a comment would be lost
 --> /home/yfful/documents/code/rustfmt/issue2896.rs:2
  |
2 |     let toto = |foo|
  |
  = note: set `error_on_unformatted = false` to suppress the warning against comments or string literals

Tested with rustfmt 0.99.2-nightly (2db2327a 2018-08-16).

Does not look related, but rustfmt is also failing to format the long lines https://github.com/frankmcsherry/differential-dataflow/blob/master/src/trace/implementations/ord.rs#L210 and https://github.com/frankmcsherry/differential-dataflow/blob/master/src/trace/implementations/ord.rs#L301. The rest of the file seems to have been correctly formatted. Should I open a new bug report?

Are bugs like this not going to be fixed before the 1.0.0 release? rustfmt 1.0.0-stable (4a4e508 2018-11-29) still has this bug and I guess the other open issues marked bug.

It will not - what's on beta is what will be in the 1.0. We can fix this post-1.0, since the warning can be fixed without being backwards incompatible

Could be a separate issue, but rustfmt fails with the following code.
fn main() {
    if 0 == ' '
    /* x */ as i32 {
    }
}

error:
error[internal]: left behind trailing whitespace
 --> /playground/src/main.rs:2:2:15
  |
2 |     if 0 == ' '
  |                ^
  |

warning: rustfmt has failed to format. See previous 1 errors.

Tested with rustfmt 1.2.2-nightly (5274b49 2019-04-24).

I also got 5 of these errors today after cargo fix and after updating my rustfmt to rustfmt 1.4.11-nightly (18382352 2019-12-03).

Same issue: https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=699a67c3936eb4c43b5d5533f500ea5c

How is this not fixed after two years? Random breakage in foundational tools is painful in Rust.

Pull request #4391 should resolve the issue.  See to be caused because when a comment at the end of a line is moved to a new line there is no trimming of the blanks before the comment.  Pull splits the result line, strip whitespaces from the end of each line and recombine the lines.

fyi, here is another example where I am hitting this issue
Error:
error[internal]: left behind trailing whitespace
  --> /Users/mesh/Exercism/rust/beer-song/src/lib.rs:11:11:0

With the error occurring on the long Strings. (manually wrapping the strings fixes the issue)
Here is the complete code:
pub fn verse(n: u32) -> String {
    let count: u32 = n;

    let mut out:String  = match count {
        2..=u32::MAX => {
            let s = if count > 2 {
                "s"
            } else {
                ""
            };

            format!(
                "{count} bottles of beer on the wall, {count} bottles of beer.\nTake one down and pass it around, {n_count} bottle{s} of beer on the wall.",
                count = count,
                n_count = count - 1,
                s = s
            )
        },
        1 => {
            String::from(
                "1 bottle of beer on the wall, 1 bottle of beer.\nTake it down and pass it around, no more bottles of beer on the wall.",
            )
        },
        0 => {
            String::from(
                "No more bottles of beer on the wall, no more bottles of beer.\nGo to the store and buy some more, 99 bottles of beer on the wall.",
            )
        }
    };

    out.push_str("\n");

    out
}

pub fn sing(start: u32, end: u32) -> String {
    let mut song_verses: Vec<String> = Vec::new();

    for i in (end..start + 1).rev() {
        song_verses.push(verse(i));
    }

    song_verses.join("\n")
}

@mikechambers, the error happens because line 11 is not empty but starts with whitespaces.   rustfmt does not trim lines that includes only whitespaces (I don't know if this is by design).  The error doesn't seem to be related to the long strings.

rustfmt does not trim lines that includes only whitespaces (I don't know if this is by design).

@davidBar-On While digging a bit into #4663, I found that doesn't seem to be true.  For example:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=df8ccaeebc257eb892fc014fc7586424
That code has trailing whitespace on line 2.  If you reformat the code, rustfmt removes that whitespace and produces no errors.  Did I misunderstand what you meant?

rustfmt does not trim lines that includes only whitespaces (I don't know if this is by design).

@davidBar-On While digging a bit into #4663, I found that doesn't seem to be true.  For example:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=df8ccaeebc257eb892fc014fc7586424
That code has trailing whitespace on line 2.  If you reformat the code, rustfmt removes that whitespace and produces no errors.  Did I misunderstand what you meant?

@davepacheco, I retried to format both the code in this issue and the code in #4663, and found that when using edition = "2018" in rustfmt.toml (current default) there is no error.  However, with edition = "2015" the error about the trailing whitespace occurs in both cases.  The rustplay link you provided sets edition=2018. but if you change it to edition=2015 the error occurs.
It seems that the whitespace error was fixed during last few months (as far as I remember, when I wrote the note on Nov., the error still occured).

@davepacheco, I retried to format both the code in this issue and the code in #4663, and found that when using edition = "2018" in rustfmt.toml (current default) there is no error.  However, with edition = "2015" the error about the trailing whitespace occurs in both cases.  The rustplay link you provided sets edition=2018. but if you change it to edition=2015 the error occurs.
It seems that the whitespace error was fixed during last few months (as far as I remember, when I wrote the note on Nov., the error still occured).

rustfmt does not trim lines that includes only whitespaces (I don't know if this is by design)
While digging a bit into #4663, I found that doesn't seem to be true. For example

Perhaps I can provide some clarity here... rustfmt is an AST-driven pretty printer type of code formatter which means that it visits and has to emit something for each node in the tree. In cases where rustfmt fails to format a particular node in the AST (which can happen for myriad reasons), the raw contents within the span of that node are emitted back out (i.e. the original snippet exactly as it was in the original input). This is done to prevent deleting parts of your code just because rustfmt failed to format those parts.
If the original input contained trailing whitespace in one of these failed-to-format cases, then accordingly the output/emitted formatting will too, and that in turn later triggers the left behind trailing whitespace error. When you see this it just about always means that there's a bug in how rustfmt is handling that particular kind of AST node, and you just happen to have trailing whitespace somewhere within the associated span.
Though that left behind trailing whitespace error message is the same, the bugs/causes and fixes are entirely unrelated (e.g. a case of this error in one place, like a match expression, is completely different than seeing the same error message in a different place, like a local stmt). As such, when a bug is fixed for one case, it  typically has zero impact on another.

rustfmt does not trim lines that includes only whitespaces (I don't know if this is by design)
While digging a bit into #4663, I found that doesn't seem to be true. For example

Perhaps I can provide some clarity here... rustfmt is an AST-driven pretty printer type of code formatter which means that it visits and has to emit something for each node in the tree. In cases where rustfmt fails to format a particular node in the AST (which can happen for myriad reasons), the raw contents within the span of that node are emitted back out (i.e. the original snippet exactly as it was in the original input). This is done to prevent deleting parts of your code just because rustfmt failed to format those parts.
If the original input contained trailing whitespace in one of these failed-to-format cases, then accordingly the output/emitted formatting will too, and that in turn later triggers the left behind trailing whitespace error. When you see this it just about always means that there's a bug in how rustfmt is handling that particular kind of AST node, and you just happen to have trailing whitespace somewhere within the associated span.
Though that left behind trailing whitespace error message is the same, the bugs/causes and fixes are entirely unrelated (e.g. a case of this error in one place, like a match expression, is completely different than seeing the same error message in a different place, like a local stmt). As such, when a bug is fixed for one case, it  typically has zero impact on another.

I am still getting this error on rustfmt 1.4.25.

I am still getting this error on rustfmt 1.4.25.

I am still getting this error on rustfmt 1.5.1-nightly (495b2166 2022-07-03)

It seems that the issue was fixed in version Two.  With version = "One" in rustfmt.toml I get the error, but with version = "Two" there is no error.

@davidBar-On and @pozm, would you mind sharing the code snippets that you're using to test / trigger the issue respectively. I want to make sure we're all on the same page here. As mentioned in https://github.com/rust-lang/rustfmt/issues/2896#issuecomment-765021508 there are potentially many different reasons for rustfmt to inform you that it "left behind trailing whitespace", so you each might be running into a different underlying issue.

@ytmimi, I used the code from @mikechambers comment above.


Commit changes to allow for offset: center

What does this do?

When using offset: center it'll try to vertically center the element in the viewport, as long as it's size is small enough, otherwise it's 0.

Is there a use case for this?

I'm currently creating a homepage, where page parts are teased in various sections followed one after another. On the side of the page are numbers, which when clicked jump to the corresponding section. This allows the numbered navigation and the sections to stay nicely aligned as well as those sections nicely replacing each other in the same space.

Can I see live demo or something? I don't mean to sound like I'm nitpicking, but I believe strongly in making every feature enhancement fight for it's right to be there.

Hey... just wanted to follow-up and see if you had a live demo I could look at.

http://yoga.kobrakai.de/

Thanks, makes sense. Let me just run a few tests, but this should be good to include. Thanks!

I've decided not to merge this for a few reasons:

It feels like a less common edge case that could be handled with the Smooth Scroll API instead of being baked into the core.
Your code is well written, but not in keeping with the style of the rest of the script or the contribution guidelines, so I would need to clean it up.
You submitted this PR when version 9 was the most recent version. I'm not onto v10 which uses  a very different approach, and would need to true this up with it.

Thanks for your work here, though.


update for auto-enter VR presentation, add Fullscreen handling, create Web-App manifest, add app icons (and glTF favicon), update <meta> tags, and include mentions of "Oculus Touch"

r+
excellent work!


Update Add Dynamic Entity Layer sample

Description
PR to revisit sample to reflect new changes in Kotlin toolkit.
Links and Data
Kotlin issue #3578
What To Review

[ ] Build and run the sample to check for any errors.
[ ] Remove MapViewpointOperation from MapView composable. https://github.com/Esri/arcgis-maps-sdk-kotlin-toolkit/pull/316
[ ] Update the MapViewModel to be able to work with the geo-compose module.
[ ] Update Toolkit version.
[ ] Add the line to the "Additional information section": This sample uses the GeoCompose Toolkit module to be able to implement a Composable MapView.
[ ] Replace the application parameter from the MainScreen with the use of LocalContext.current.applicationContext
[ ] Add the tags: geocompose, tookit
[ ] Update the sample's metadata.json file to reflect file and tags changes

How to Test
Run the sample on the sample viewer or the repo.

@shubham7109 Hi Shubham, just wondering, if this PR is something you are going to review?


ECONNRESET

Hi There,
First, as always I like to say thank you for your hard work on this library.
Recently I wrote a syncer script using jsforce, and I can't help but notice when executing a very large query on a very large table the connection gets reset multiple times.
Now I have built in recovery procedure into the script to handle the problem, but my Project Manager asked me if there was anything I could do to increase the timeout time on my end.
My very simple log file looks like this:
starting another job: Task
begin working on job Task
updated job Task status {running:true}
beginSync
{ [Error: socket hang up] code: 'ECONNRESET' }
gotResults
salesforce error: { [Error: socket hang up] code: 'ECONNRESET' }
beginSync
{ [Error: socket hang up] code: 'ECONNRESET' }
gotResults
salesforce error: { [Error: socket hang up] code: 'ECONNRESET' }
beginSync
{ [Error: socket hang up] code: 'ECONNRESET' }
gotResults
salesforce error: { [Error: socket hang up] code: 'ECONNRESET' }
beginSync
gotResults
updating job Task lastDate:2014-11-14T00:37:00.000+0000
continueSync
gotResults
updating job Task lastDate:2014-11-19T19:31:30.000+0000
continueSync
...

After three hangups, everything begins to work correctly. This has been a consistent pattern with one table and query.
Please do let me know if there is some setting on my end I can control to extend how long the connection is open for.
Thank you!

I am having same problem.
Did you find any solution??

Seen similar things.

##

@vikashk1 ?

Seen similar things? Any idea how to avoid this kind of errors for large queries.

@Harishgolla Haven't had any luck. I think something is wrong with the closing of jobs. I see them sit open from SF console even after they've finished sometimes. Maybe too many open ones leads to ECONNRESET

@rbaindourov What kind of retry code you wrote to handle this ?
Now I have built in recovery procedure into the script to handle the problem,

just check if err.code == ECONRESET and run it in a loop w/ try catch or .catch()

We submit a lot of batch jobs to Salesforce on a daily basis and I can tell you 1st hand that we see this error regularly.  We process about 100-200 batch jobs a day.  It's become much more frequent in the past 3-4 months.  I agree with @tejasmanohar in that we get these most commonly when trying to close jobs - As though Salesforce simply can't get the job done at times and so the API call will timeout.  All-in-all their API is extremely unreliable.
Currently forced to issue retries and/or alternative designs that limit the amount of jobs needed just to attempt to avoid the unreliable Salesforce job system.  Suggestions welcome however short of opening a support ticket with Salesforce I believe this is out of jsforce's control.  Please leave this ticket open for now in case a creative solution presents itself.

Encountered this issue when sending multiple queries through a Promise.all(). Interestingly the first retry hit an issue where a query returned 0 results (not possible with 'SELECT Id, DeveloperName FROM RecordType'). The next retry succeeded with no issue.  If this is a timeout issue, is there a way to define the timeout for the http.request() to allow more time for SF to complete the request at "busy" times?

This should be fixed in v3, see: https://github.com/jsforce/jsforce/discussions/1529


Feature/6.4.1/tdi 38096 salesforce pk chunking

What is the current behavior? (You can also link to an open issue here)
Currently can not use primary key chunking for large volumes of data in Salesforce.
https://jira.talendforge.org/browse/TDI-38096
What is the new behavior?
Add possibility to setup primary key chunking for Salesforce bulk connection.
Chunk size must be less than 250000. This feature should be used for volumes greater than 20 M records
Official documentation
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:

Pull request test result: failure :x:

Pull request test result: failure :x:

Builds are failed for the following reason.
Last master build was 5 days ago on the 28 of Apr
components-osgitest in nexus repository is outdated.
For some reason org.talend.components-components-api-service-osgi-tests.link file points not to latest version, but to the concrete version:
mvn:org.talend.components/components-api-service-osgi/0.18.0-20170428.122019-117//tests
But this version was already removed from nexus repository.
I've rebuilt components-osgitest locally. And restarted tests locally. They passed, however they failed before too.
Thus, we need master rebuild to fix this build

Pull request test result: failure :x:


2021 inventory with updated format

updated format using ForestGEO conventions (For Status_Year: LI = living, DS = dead, standing, DC = dead, down). will be better for computing & analysis
2021 values input from 3/9/2021
Survey of what trees are out of plot are marked with an "Out of plot" in the Notes column

Questions:

Should we switch the "Out-west" and "Out-east" to simply "West" and "East"?
For 1812 and 1813 in the Salt plot, it looks like you (@bpbond) switched the numbers in the data sheet. Does this mean the values were for the wrong tag or just that the rows are out of order?

Closed #11

Hello @stephpenn1 thanks for your work on this.

Yes
Right, this is the one I showed you in the field—the trees were 1813 then 1812 on the data sheet, so I initially entered the wrong (switched) values. Whatever the sheet shows now should be correct however.

Questions for you:

Why is Status_2021 not filled in (except for the DS and DC entries)? I feel like every one should have an entry, as in 2019 and 2020?
Let's take this opportunity to change the dates to unambiguous YYYY-MM-DD format?
Using the Notes field to record out of plot strikes me as a bad idea. I would propose just deleting those rows at this point, or if you want to keep them, let's add an In_plot true/false column.

@stephpenn1 and @bpbond you should double-check the sap flow tree IDs in the inventory file. When I extracted their DBHs yesterday, I noticed that some IDs were missing and others were duplicated:
Control | West | 1432 | 28.2 | 3/9/19 | FAGR | Yes | C5 | 28.2 | 3/10/20
Control | East | 1330 | 39.8 | 3/9/19 | LITU | Yes | C5 | 39.9 | 3/10/20

Thanks @ahopple , I bet they should be C5 and C15 instead. Will cross reference with our sapflow doc!


Release Upper Limit on Torch, Transformers, Accelerate

Currently the torch dependency in framework is upper bounded as "< 2.3", however in accelerate versions has problems supporting torch 2.2. The latest numpy versions (>=2.0) also has incompatibilities with the current torch version and is bounded here in #42. Hence, we should consider releasing the upper bound soon.
Also can consider releasing the upper limit on transformers and accelerate

we found out there is really no need to upper bound this torch dependency, as for us we are getting stuck only beause of this commit https://github.com/pytorch/pytorch/pull/121635.

so the workaround for us can be just to downgrade to nvidia-nccl-cu12==2.19.3

Also for transformers, we just have to be weary of the sliding window mask issue of SPDA, and keep track of it to see when it will be fixed.

FMS has fixed the TRL issue https://github.com/foundation-model-stack/fms-hf-tuning/pull/213

I think we need a lower limit on the bitsandbytes version that supports quant_storage. I have encourted that 0.41 didnt work, but 0.43 is ok

GPTQModel has a lower bound on transformers here but framework is bounded at transformers<4.40. This discrepancy should be resolved.

We have removed the upper bound on transformers in #58


Enable xcm transfer for Acala chain

Acala App has re-enabled Bridge feature => Enable xcm transfer for Acala chain on SubWallet

Version test: https://file.cdmteck.com/s/4oimk9zDJ8R5Aai/download?path=%2F&files=./koni-dev-build-0.6.6-1664880562620.zip
Validation passed!


problems bar not working

Issue Type: Feature Request
when i am in the text editor usually problems show up but suddenly they aren't showing up , please help
VS Code version: Code 1.56.2 (054a9295330880ed74ceaedda236253b4f39a335, 2021-05-12T17:13:13.157Z)
OS version: Windows_NT x64 10.0.19042

/needsMoreInfo


version 0.5.1 release

@dozed Is it ok to bump xsbt-web-plugin version from 2.0.4 to 2.1.0?

Let's merge this and release 0.5.1 now.

Done. https://oss.sonatype.org/content/repositories/releases/org/scalatra/sbt/scalatra-sbt_2.10_0.13/0.5.1/


Feature/standalone query iterator

This PR updates query and scan iterators to provide access to the current state of the operation. The iterators returned by scan and query gain a lastEvaluatedKey property, and those returned by parallelScan gain a scanState property that tracks this data for each segment of the scan. Sequential and parallel iteration/pagination is also factored into its own package to move that particular complexity out of the DataMapper class.

Currently trying (and failing!) to reproduce the error Travis is encountering locally. In Node 6.14.3, dynamodb-data-mapper/src/DataMapper.spec.ts appears to be run until the Travis container gives up after 10 mins of inactivity.

Based on #65, the way this PR is handling LastEvaluatedKey is not guaranteed to work for queries and scans operating on indices. The attributes included in LastEvaluatedKey for an index scan is not always entirely deterministic, and attempting to calculate the last evaluated key for a partially consumed page is dangerous.
The iterators in this PR should not offer a lastEvaluatedKey property -- to obtain this value, the user can drop down to the paginator, which will always have the LastEvaluatedKey returned by DynamoDB. To make this usable, there should be some way to tweak the Limit on each page to make sure a user can retrieve a fixed number of records


How to exclude some router/api url from jsonwebtoken

I am using node.js express app. jsonwebtoken for authentication. I want to exlude some api url from the jsonwebtoken verification. below is what I have tried and my code
router.use('/authentication', mountAllRoutes(authenticationModule));

// route middleware to verify a token
router.use((req, res, next) => {
const r = req;
const token = req.body.token || req.query.token || req.headers.authorization;
// decode token
if (token) {
    // verifies secret and checks exp
    jwt.verify(token, (req.app.get('superSecret')), (err, decoded) => {
        if (err) {
            // res.json({ success: false, message: 'Failed to authenticate token.' });
            res.status(401).send({
                success: false,
                message: 'Failed to authenticate token.'
            });
        } else {
            // if everything is good, save to request for use in other routes
            r.decoded = decoded;
            next();
            // console.log(decoded);
        }
        return {};
    });
} else {
    // if there is no token
    // return an error
    return res.status(403).send({
        success: false,
        message: 'No token provided.'
    });
}
return {};
});

router.use('/test', mountAllRoutes(testModule));
router.use('/other', mountAllRoutes(otherModule));
router.use('/data', mountAllRoutes(dataModule));

Here I have placed routes above middleware which I dont want to protect. and I have placed after middleware which I want to protect. But it is protected which I placed above middleware. In authenticationModule, login and user registration api comes. so for user registration it gives response no token provided

@ashrafmangalore I think you may like: https://github.com/auth0/express-jwt/

Another quick solution
https://github.com/nuxt-community/auth-module/issues/211#issuecomment-401486864


chore(deps): bump Microsoft.Web.WebView2 from 1.0.2045.28 to 1.0.2210…

….55 in /src/shell/dotnet/Shell

Replacing #512


Document different context for setup, test and teardown

In the documentation for JunitTestContext (i.e. https://vertx.io/docs/vertx-junit5/java/#_a_test_context_for_asynchronous_executions) it doesn't mention that any @BeforeEach/@AfterEach-annotated method gets a different context instance. I, at least, was surprised by this, so it's possibly worth mentioning.

My mistake. I just wasn't looking in the right place: https://vertx.io/docs/vertx-junit5/java/#_scope_of_code_vertxtestcontext_code_objects


Move Docker-specific log handling to DockerManager.

Eases the separation of Docker-specific code into the Docker-specific
runtime.

@satnam6502 the main semantic change here is that the logs symlink is now made for the pod infra container as well. Is this a problem? I already have a fix if it is, but want to make sure we need it first :)
/cc @dchen1107  @yifan-gu @satnam6502

I re-run travis. LGTM and will merge once it is green.

Thanks for the quick review @dchen1107!

LGTM!


the terminal process terminated with exit  code #259

the terminal exit with exit code after click the new terminal

/needsMoreInfo


Autorun improvements

For #833.
Features:

re-runs when code hasn't changed, both in autorun mode and in manual mode
spinner in autorun mode
removed toggle animation

UI notes:

double render in autorun mode on each keystroke (first one for the spinner, second for execution) seems fast enough; at least on my M1 macbook, hopefully it's ok on slower CPUs, someone should check
in manual mode, the spinner is gray if the code has stayed the same, or blue if the code has changed (similar to how this was already indicated with the button color); this happened by accident and I can make the spinner icon always-blue, but I guess it makes sense to indicate whether current execution is the re-run
spinner icon in autorun mode is the checkmark icon, I currently don't replace it with the rotating arrows icon; this might be controversial, and I'm not sure myself if I like it; but there's a reason for it: otherwise the icon would slightly flicker on each keystroke, even if execution is fast enough, and that's pretty annoying

OTOH, turning checkmark looks like rotating clock hands, which is kinda cool
@OAGr, what do you think?

Flickering example when execution is fast (but on the actual webpage it would be faster, gif framerate is too low):

Current rotating checkmark which doesn't flicker because the icon stays the same when execution is fast:

Codecov Report

Merging #889 (9ec84d2) into develop (fb0aa72) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #889   +/-   ##
========================================
  Coverage    57.96%   57.96%
========================================
  Files           73       73
  Lines         4094     4094
========================================
  Hits          2373     2373
  Misses        1721     1721

Continue to review full report at Codecov.

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

Some quick things:

When it's autorunning, I don't like the spinning checkmark. Any way you could instead just have this start after ~0.2ms or similar, instead?
re-runs when code hasn't changed -> When I try re-running it, it gives me the same exact outputs each time, which is weird. There should be randomness.

When it's autorunning, I don't like the spinning checkmark. Any way you could instead just have this start after ~0.2ms or similar, instead?

It's not doable with react, but should be doable with pure css animations, I'll experiment with those.

to change the opacity of the view to 0.5 or similar, while it's loading

Oh, right, forgot about that.

When I try re-running it, it gives me the same exact outputs each time, which is weird

Oops, I forgot to pass executionId to the hook, thanks for noticing!

When it's autorunning, I don't like the spinning checkmark. Any way you could instead just have this start after ~0.2ms or similar, instead?

It's not doable with react, but should be doable with pure css animations, I'll experiment with those.

A bit hacky, but it works, with 0.1s opacity animations. (Tricky part is that I had to hide the old icon with animations too, and animatiing visibility didn't work for some reason, so I went with opacity animation instead).

to change the opacity of the view to 0.5 or similar, while it's loading

Done.

When I try re-running it, it gives me the same exact outputs each time, which is weird

Fixed.

Better!
Maybe just don't change the "pause" to also be a spinner, I find that annoying.

I get it's a bit confusing. I think later we'll refactor the button from "Paused" to something a bit different.

Maybe just don't change the "pause" to also be a spinner too, I find that annoying.

Oops, that's a bug, fixed.

Great, things are green, merging.


Replace individual get, post, etc. defns with macro

@dakrone not sure if this is just stylistic preference =)

Hi @atroche
I think I'd rather stick with regular functions, since it makes jumping to definition (M-. in emacs) work much better, we've discussed this before in #223 (for some history).


Trouble with initializing Narrative Model

I am running relatio(0.3.0) in python 3.7.16 and I have followed the tutorials, and I have gotten until calling on the NarrativeModel function, but now am running into issues I havent been able to fix.
I dont have a strong preference for what Embedding Model to use, but I have tried multiple (Spacy, Sentece-Transformers, GloVe) and none of them work for me.
Thanks!
Johannes
I want to run:
from relatio.narrative_models import NarrativeModel

from relatio import Embeddings

nlp_model = Embeddings("spaCy", 'en_core_web_lg')

m = NarrativeModel(clustering = None,
                    PCA = True,
                    UMAP = True,
                    known_entities = top_known_entities,
                    assignment_to_known_entities = 'character_matching',
                    embeddings_model = nlp_model
                    )

and my error is:
✘ No compatible package found for '<relatio.embeddings.Embeddings
object at 0x7fb04acb3210>' (spaCy v3.3.3)

An exception has occurred, use %tb to see the full traceback.

SystemExit: 1

%tb
Traceback (most recent call last):

  File "/Users/johannesrenz/Documents/relatio_2CTA.py", line 123, in <module>
    embeddings_model = nlp_model

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/relatio/narrative_models.py", line 112, in __init__
    embeddings_type=embeddings_type, embeddings_model=embeddings_model

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/relatio/embeddings.py", line 84, in __init__
    self._embeddings_model = EmbeddingsClass(embeddings_model, **kwargs)

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/relatio/embeddings.py", line 199, in __init__
    spacy_download(model)

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/spacy/cli/download.py", line 68, in download
    version = get_version(model_name, compatibility)

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/spacy/cli/download.py", line 99, in get_version
    exits=1,

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/wasabi/printer.py", line 85, in fail
    title, text, style=MESSAGES.FAIL, show=show, spaced=spaced, exits=exits

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/wasabi/printer.py", line 249, in _get_msg
    title, text, color=style, icon=style, show=show, spaced=spaced, exits=exits

  File "/Users/johannesrenz/opt/anaconda3/envs/relatio2/lib/python3.7/site-packages/wasabi/printer.py", line 155, in text
    sys.exit(exits)

SystemExit: 1

Issue solved, i believe the problem was that i had previously downloaded the spacy model with spacy directly and the model version was incompatible with the narrative model.


pageserver - walreceiver sends CopyFail on error

When an error happens in the loop in walreceiver_main, the CopyBothDuplex struct owned by ReplicationIter (physical_stream) is dropped. The connection future notices this and sends a CopyFail message.
Sending CopyFail and retrying results in this log from postgres:
2021-08-23 22:54:13.040 GMT [12503] FATAL:  invalid standby message type "f"
2021-08-23 22:54:13.040 GMT [12503] STATEMENT:  START_REPLICATION PHYSICAL 0/F2527B8
2021-08-23 22:54:14.050 GMT [12504] FATAL:  invalid standby message type "f"
2021-08-23 22:54:14.050 GMT [12504] STATEMENT:  START_REPLICATION PHYSICAL 0/F2527B8
...

Postgres only expects CopyDone, Copy, and Terminate
https://github.com/postgres/postgres/blob/8e7811e952b4e0993330f7ac1fb1448a1ca69b9a/src/backend/replication/walsender.c#L1745

@arssher is this still relevant ?

It is.


Once any of the dns servers succeded try the failed servers again

In some environments where all the dns packets except the ones to the server by the ISP are filtered (and the ISP's DNS server returns crap in every response, but the direct connections to IPs work) this may be helpful.

There are no "failing servers", just RTT estimations, adjusted over time, and reset when certificates are refreshed.


Update to cadence v0.31.1

Description
Update to:

cadence/v0.31.1
flow-go-sdk/v0.31.1
flow-go/v0.29.3

For contributor use:

[x] Targeted PR against master branch
[x] Linked to GitHub issue with discussion and accepted design OR link to spec that describes this work
[x] Code follows the standards mentioned here
[ ] Updated relevant documentation
[x] Re-reviewed Files changed in the GitHub PR explorer
[x] Added appropriate labels

Holding this PR off a bit, since there is a newer patch version of cadence and we can update directly to it.

@SupunS can we update and get this merged so it doesn't get stale 🙏 thank you

Seems this is already done in https://github.com/onflow/flow-emulator/pull/287. Hence closing this PR.


50x errors due to pods termination

We are using this image m00nf1sh/aws-alb-ingress-controller:v1.2.0-alpha.1.
Our nodes are managed using an autoscaling groups of spot instances. We are using a spot termination handler to ensure that pods are evicted and the node is cordoned before it is killed. The application is exposed via cloudfront.
Every day we get a downtime of each application for about 4 minutes (monitored via an external health check). This happens if a node is removed from the autoscaling group.
The health check configuration of our target groups look like this:
Healthy threshold
2
Unhealthy threshold
2
Timeout
5
Interval
15

We always have 2 or more replicas running on different nodes.
My assumption would be that the alb ingress controller would remove a node timely from the target group in order to reduce the downtime to < 1 minute. According to our logs the alb ingress controller sometimes does not write any log entry about removing a target. This also looks strange.

these seems to be a similar issues
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/814
https://github.com/kubernetes-sigs/aws-alb-ingress-controller/issues/1065


Shaking screen when scrolling to video player

When scrolling to video player, window is shaking up and down rapidly and stops after about 2 seconds. Then window is shaking each time I try to scroll, making it unable to scroll anywhere so I need to restart app

Kiwi Browser Version 93.0.4577.17
Xiaomi Redmi note 10s

Could you screen record this issue and upload here? @SylwesterDu

Is text reflow / rewrap enabled ?

Could you please screen record this issue and upload it here? @SylwesterDu

https://drive.google.com/file/d/1-Ze70k3pvzFwE7NzTZ0MiLawKMQrV21u/view?usp=drivesdk

Is text reflow / rewrap enabled ?

No, it is disabled
Looks like this issue appears only in the site recorded on video. I think it is not site fault, other browsers, for example brave work correctly

No idea what's going wrong there, I think we would have to actually look into the site's source in terms of the video player to understand what's going on. A second site where this issue occurs would probably come in handy.

After updating to chromium 94 bug disappeared. Thanks for your work


Add --no-typecheck option for runtests/runtests-parallel

With this, you can run:
hereby runtests-parallel --no-typecheck

And it will start the tests immediately after bundling.
Maybe better would be to run typechecking in parallel off to the side and exit if that fails, but this is fine for a first attempt.
(--typecheck=false also works.)

it would be nice to have if the local build fails only on tsserver and tsc project errors instead?

That is possible, I just made it depend on buildSrc as you suggested. It could instead depend on buildTsc and buildTsserver and such instead without tests. I think we just need to overall define what it is we are expecting, because it seems like different people want the build to live at different positions on the "fast" vs "safe" scale.

user-scoped hereby files when?

I've updated the PR to also do this same thing for min/local and make them not need to depend on the entire buildSrc task.

user-scoped hereby files when?

Just when I thought I could keep that <600 lines.... 😅

user-scoped hereby files when?

That being said..... they are just ESM modules, so...
Oh no please don't let me get nerd sniped


phpLintMain task Failed

Dear Dmytro,
I have installed your plugin in my Project.
I think it's such a great feature, and very scarce in PHP project.
But, when i run the phpLintMain task there is an error.
The error message is :

What went wrong:
Execution failed for task ':phpLintMain'.

java.lang.Throwable (no error message)

Thanks for your attention.

Hi HieronyM,
Could you provide your build file ?
Thanks.

Dear Dmytro.
Here's my Build File
http://pastebin.com/LcxqG1d6
Thanks for your attention

I can't find in your build script this line which is required to load plugin
apply plugin: 'php'

You've just added dependencies but now actual load of plugin.

Dear Dmytro,
I think we can apply it using :

apply plugin: 'com.eltrino.gradle.php'

So I must added apply plugin : 'php'  ??
Thanks

i was not testing it with full name. short one should work for sure.
Could you try it ?

Dear Dmytro,
My workstation is still using by another.
Can I say the result tomorrow?
:D
And also I have suggestion to your project, Why don't add other static code Analysis like pdepend, phpcpd, phploc?
So 1 plugin can be use to do all static code Analysis.

np from my side :)
regarding suggestion, definitely yes, its in the plan, time is a problem :(

Dear Dmytro,
Are you familiar with make build file using Gradle?
Do you have any social media, so I can message You when I have problem?
hahhaha..

Dear Dmytro,
I have tried using apply plugin : 'php', but it seems like it not found plugin with id 'php'.

Could you try to add in buildScript section such line
configurations.classpath.resolutionStrategy.cacheChangingModulesFor 0, 'seconds'

Probably something cached incorrectly.

Dear Dmytro,
I have tried your suggestion, but error still same.
FYI, When I try phpMd task it gives error too..
This is the error :
Execution failed for task ':phpMdMain'.

java.io.IOException: Cannot run program "vendor/bin/phpmd": CreateProcess erro
r=2, The system cannot find the file specified

But it should specify bigger path, you have your $projectDir which some path. Actually where definition of this variable ?


refactor DockerSystem logic to make it support pass runner easy

Describe your changes
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Lint and apply fixes to your code by running lintrunner -a
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

/azp run


doc: make Readme more readable

Describe what this PR does / why we need it

a concise examples of the quick start
make introduction more brief
aggregation and consolidation of features

Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

Codecov Report

Merging #1594 (d929b59) into main (9882dbc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1594   +/-   ##
=======================================
  Coverage   11.82%   11.82%
=======================================
  Files          91       91
  Lines        8112     8112
=======================================
  Hits          959      959
  Misses       7030     7030
  Partials      123      123

Continue to review full report at Codecov.

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

./README.md:38: MD012 Multiple consecutive blank lines


Show due notes

Hello @rsandz
I think showing due notes could be useful. Are you interested in the feature?
The PR is still WIP so early feedback is welcome.

Hey @Jackymancs4, if you don't mind writing a bit on the expected UX for this, I think it'll help!
If my understanding is correct: we want to show ToDo Notes that have an alarm on a certain day?
If so, I think it's a cool feature 😄
As with the other note types, this feature should default to disabled until opt-in by user.
The current technical approach seems similar to the created and modified notes which looks good to me.


apicurio - support register schema via webapp

When registering an avro schema via the apicurio webapp, the schema has a different (generated) id than its fingerprint (AvroSchemaId). Plus, the revision metadata is not set.
This leads to different behavior of the AvroSchemaRegistry based on manual registration of the schema via webapp and a programmatic registration via AvroSchemaRegistry#register.
Idea: To work around this issue, we could use a composite registry where an in memory registry is put in front of the apicurio rest api, that loads all registered schemas and keeps a copy with the expected behavior.

fixed with b91add461e61ee5cf18f0e175f7982887ba1a4be