[AIRFLOW-1755] Url prefix support for flower and webservice

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1755

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

I would like to submit this change that allows Airflow to support accessing easily the web UI from an endpoint such as /airflow and Flower with /flower.
For Flower, it is already possible to use the flowerconfig.py to convey the url_prefix setting, but I prefered aligning it
I propose this change that add 2 new settings on airflow.cfg and the CLI arguments:
# Root URL to use for the web server
web_server_url_prefix = /airflow
...
# The root URL for Flower
flower_url_prefix = /flower

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

I am open to suggestion on how to add unit tests on this PR. Tests will be done on a preproduction cluster running Kubernetes.
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2723 into master will increase coverage by 0.09%.
The diff coverage is 45%.

@@            Coverage Diff             @@
##           master    #2723      +/-   ##
==========================================
+ Coverage   72.37%   72.46%   +0.09%
==========================================
  Files         154      154
  Lines       11815    11822       +7
==========================================
+ Hits         8551     8567      +16
+ Misses       3264     3255       -9

Impacted Files
Coverage Δ

airflow/models.py
87.14% <ø> (+0.04%)
:arrow_up:

airflow/bin/cli.py
53.66% <0%> (-0.25%)
:arrow_down:

airflow/configuration.py
84.4% <100%> (+0.36%)
:arrow_up:

airflow/www/app.py
96.51% <100%> (-0.05%)
:arrow_down:

airflow/settings.py
92.53% <100%> (ø)
:arrow_up:

airflow/www/views.py
70.93% <14.28%> (ø)
:arrow_up:

airflow/jobs.py
79.51% <0%> (+0.45%)
:arrow_up:

airflow/utils/helpers.py
56.32% <0%> (+2.87%)
:arrow_up:

airflow/task_runner/bash_task_runner.py
100% <0%> (+6.66%)
:arrow_up:

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 f271d43...aaa51b5. Read the comment docs.

 I was looking for taskinstance.index_view actually :)

PR updated after your recommendations. I have tested it on my home dev environment, which is not the labs where i needed these url_prefix changes. So, manual tests looks ok. I will stress it next week again.

-1 for the moment, I have encountered a bug when testing on our Kubernetes instance. Static files are not properly prefixed, while on my dev machine it works fine

I have found the root cause of the static file url issue. The fact to use url_for for static files didn't worked because of bad initialization of the Flask(...).

@ashb @msumit What do you think about my Pull Request?

Hello. Here is the result of some addiotional tests on Kube 1.8:

the use of url_for has a strange impact compared to the way it url_prefix handled in Flower, that is completely takes care of url rewriting: Access log shows something like;
GET /celery/admin/ HTTP/1.1" 200 68476 "http://airflow.tls.renault.fr/celery/admin/
On Flower, I need to request the load balancer to strip the pathprefix (With traefik: traefik.frontend.rule.type: PathPrefixStrip annotation

Action: I would like to align both behavior

experimental api is not prefixed, for exemple /api/experimental/latest_runs

I will work on it asap.

Hello. Fixed the experimental api url. i have also updated the documentation about reverse-proxying airflow

@ashb Can you do a review? I don't know who I can contact, I really would like to integrate this pull request in airflow, for myself, I need it since we cannot execute airflow/celery smoothly in our internal kubernetes cluster at a root path (airflow.mycompany.com is not allowed, but I can put it in mydeps.intra.mycompany.com/myteam/airflow/).
Thanks

I'll see if I can take a look today, I can't make any promises.
(The reason you aren't getting much of a response is was Thanksgiving yesterday and many people will have taken today off as holiday as well.)

A couple of small comments on code. The changes look okay. Caveat: I haven't run the code myself.
One minor comment: The whitespace-only changes made it harder to review. I don't know how the core team feel about them. (I am not a member of the core team)

removed the whitespace change. They have been made by my editor, sorry

(I know the feeling. I have to configure my editor to not do the same when working on Airflow)

Do you know who I can contact or what I can do, on the mailing list there is no a big momentum on this PR :(

Nope, my only suggestion would be to try again on Monday, towards the end of our days, which will hopefully coincide with more Americans being active. Sorry

Reviews are good, I saw and fixed issues related to blueprints: the root redirection was broken and the /health wasn't under the prefix.
Thanks for you help anyway !

Hi. My local environment has no more problem, if you want to test this PR do not hesitate. I wish it could be merged soon. Thanks

I just came across this pull request while searching for a way to define the Flower url_prefix in Airflow, so I'd like to give a big +1 to this.  The extra docs on nginx config are great too.

+1

Thank you for your contribution! I tested this out locally and looks good overall.
I came across an encoding issue when I attempt to click the trigger dag button on the homepage. This was introduced with this PR; but let me know if you can't replicate it.
Traceback (most recent call last):
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask-0.11.1-py2.7.egg/flask/app.py", line 1988, in wsgi_app
    response = self.full_dispatch_request()
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask-0.11.1-py2.7.egg/flask/app.py", line 1641, in full_dispatch_request
    rv = self.handle_user_exception(e)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask-0.11.1-py2.7.egg/flask/app.py", line 1544, in handle_user_exception
    reraise(exc_type, exc_value, tb)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask-0.11.1-py2.7.egg/flask/app.py", line 1639, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask-0.11.1-py2.7.egg/flask/app.py", line 1625, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask_Admin-1.4.1-py2.7.egg/flask_admin/base.py", line 69, in inner
    return self._run_view(f, *args, **kwargs)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask_Admin-1.4.1-py2.7.egg/flask_admin/base.py", line 368, in _run_view
    return fn(self, *args, **kwargs)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Flask_Login-0.2.11-py2.7.egg/flask_login.py", line 755, in decorated_view
    return func(*args, **kwargs)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/apache_airflow-1.10.0.dev0+incubating-py2.7.egg/airflow/www/utils.py", line 262, in wrapper
    return f(*args, **kwargs)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/apache_airflow-1.10.0.dev0+incubating-py2.7.egg/airflow/www/utils.py", line 309, in wrapper
    return f(*args, **kwargs)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/apache_airflow-1.10.0.dev0+incubating-py2.7.egg/airflow/www/views.py", line 948, in trigger
    return redirect(origin)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Werkzeug-0.13-py2.7.egg/werkzeug/utils.py", line 372, in redirect
    location = iri_to_uri(location, safe_conversion=True)
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/Werkzeug-0.13-py2.7.egg/werkzeug/urls.py", line 670, in iri_to_uri
    ascii_iri = to_native(iri).encode('ascii')
  File "/Users/joyg/Dev/airflow-source/venv/lib/python2.7/site-packages/future-0.16.0-py2.7.egg/future/types/newbytes.py", line 381, in __getattribute__
    raise AttributeError("encode method has been disabled in newbytes")
AttributeError: encode method has been disabled in newbytes

As for writing unit tests, you can test out the webserver configuration by (1) configure web_server_url_prefix (2) call the web/api endpoints with the custom url prefix (3) check 200 is returned.

I do not have any problem with any button. I however do all my test on Python 3, and this seems to be related to future package.

@Stibbons it works in PY3 perfectly fine, the error only shows up in PY2. The future library converts str to newbytes, which made get_url_prefix() return a newbytes type, and werkzeug isn't equipped to handle this type.
A kind of patchy way to work around this is instead of:
def get_url_prefix():
    url_prefix = get('webserver', 'WEB_SERVER_URL_PREFIX')
    if not url_prefix or url_prefix == '/':
        url_prefix = ""
    return url_prefix

return the native type:
def get_url_prefix():
    url_prefix = get('webserver', 'WEB_SERVER_URL_PREFIX')
    if not url_prefix or url_prefix == '/':
        url_prefix = ""
    return url_prefix.__native__()

There may be a cleaner fix that i'm not aware of though.

Something from the six package is the usual way of dealing with py2/py3 issues.
@Stibbons could you add a test case that exercises whatever code path was triggering the problem?

I am on it, I do not see any problem here with Python 2.7, I'll add a unit test to cover the fact get_url_prefix returns a str in python 2.7.
url_prefix.__native__() does not work, url_prefix is a unicode string in my tests. I'll update this PR asap

I have found the issue and fixed it. It works in when the string is empty and when it is not, in python 2.7.
So now should work in python 2.7, 3.5 and 3.6, at least.
I have some difficulties to add unit tests, I'll try to do something this afternoon.

I think the dask test issue has been fixed, so rebasing should help.


Update docs for IE conditionals

Add an example of conditional comments wrapping the main stylesheet
and conditional IE stylesheets.
Link to examples in the GOV.UK template and GOV.UK elements.
@jonhurrell, let me know if this would have resolved #280.
(I know it's closed, but think the updates will benefit others).

@gemmaleigh Yes, some examples would be great, it's a hard one to spot inline.

In the future I was thinking we might want to split out some of the documentation into guides/tutorials, rather than being inline with individual mixins, eg, "How GOVUK Template supports internet explorer" or "Setting up GOVUK Template with Rails".
Thats quite a bit of work, and will probably come with larger refactoring of govuk_template and co, but this is a big improvement in the mean time 👍


Changes to optionally return distances with .sel

Changes include:

accessor optionally takes in a "distances_name" str with which to name the new variable in results to store calculated distances from query()
query() now returns distances
distances are returned (always?) in radians from the trees, but are converted to kilometers in query.
If a DataArray was input to .sel with "distances_name" input, a Dataset will be returned to contain the new variable
added example to intro notebook
added a test to test_accessor
also fixed an import in conftest

Notes:

I did not test this with the dask approach

Sorry for all the additions. I have continued to use the code in the project I'm working on and have realized more about the existing code and better approaches as I go.

I forgot to say this was meant to address https://github.com/xarray-contrib/xoak/issues/26.

@benbovy or @willirath are either of you available to take a look at this? I'm hoping to be able to use these changes in a version update of xoak with a package of mine that uses xoak. Thanks for your time.

@benbovy I'm completely open to the API and detailed choices in the code if you want to change anything. I just need access to the distances! I am aware of the new indices in xarray but haven't seen them in action to see how they work or how to use them yet.

I am aware of the new indices in xarray but haven't seen them in action to see how they work or how to use them yet.

Yes this is pretty new and still largely undocumented. Xoak is a good place to try this feature, but we need a smooth transition so I don't expect it to be straightforward.

I'm completely open to the API and detailed choices in the code if you want to change anything.

Sorry I could have provided more details in my previous comment. I was thinking about this alternative:

Keep Dataset.xoak.sel(indexers=None, **indexers_kwargs) unchanged so that it will be easier to later switch to using Dataset.sel((indexers=None, **indexers_kwargs) with a custom Xarray index implemented in xoak. We might want eventually provide custom options to Dataset.sel but it is still not clear how would look like the API (https://github.com/pydata/xarray/issues/7099).

Add a Dataset.query(indexers=None, **indexers_kwargs) that returns both distances and indices as Xarray DataArray and Dataset objects (#32).

This would look like this (reusing the examples in the documentation):
>>> ds_mesh
# <xarray.Dataset>
# Dimensions:  (x: 100, y: 100)
# Coordinates:
#     lat      (x, y) float64 ...
#     lon      (x, y) float64 ...
# Dimensions without coordinates: x, y
# Data variables:
#     field    (x, y) float64 ...

>>> ds_trajectory
# <xarray.Dataset>
# Dimensions:    (trajectory: 30)
# Dimensions without coordinates: trajectory
# Data variables:
#     latitude   (trajectory) ...
#     longitude  (trajectory) ...

>>> ds_mesh.xoak.set_index(['lat', 'lon'], 'sklearn_geo_balltree')

>>> distances, indices = ds_mesh.xoak.query(
...    lat=ds_trajectory.latitude,
...    lon=ds_trajectory.longitude,
... )

>>> distances
# <xarray.DataArray (trajectory: 30)>
# ...
# Dimensions without coordinates: trajectory

>>> indices
# <xarray.Dataset>
# Dimensions:  (trajectory: 30)
# Dimensions without coordinates: trajectory
# Data variables:
#     x        (trajectory) int64 ...
#     y        (trajectory) int64 ...

Having a query method that returns a (distances, indices) tuple is consistent with scipy.spatial.KDTree.query(), sklearn.neighbors.KDTree.query(), sklearn.neighbors.BallTree.query(), etc.
The distances and indices Xarray objects returned here are a bit confusing as they don't have the same type and structure, but it allows doing:
>>> results = ds_mesh.isel(indices).assign(distances=distances)
>>> results
# <xarray.Dataset>
# Dimensions:  (trajectory: 30)
# Coordinates:
#     lat        (trajectory) float64 ...
#     lon        (trajectory) float64 ...
# Dimensions without coordinates: trajectory
# Data variables:
#     field      (trajectory) float64 ...
#     distances  (trajectory) float64 ...

Now, this is clearly more verbose than what is proposed in this PR:
>>> ds_mesh.sel(
...     lat=ds_trajectory.latitude,
...     lon=ds_trajectory.longitude,
...     distances_name="distances"
... )

Also, because Dataset.xoak.query() doesn't return a single Xarray object it wouldn't be easy to chain it with other Dataset / DataArray methods (it is possible using pipe -> merge but this is less straightforward than just using Dataset.xoak.sel()). So maybe we could support returning the distances via both .xoak.query() and xoak.sel() after all? What do you think @kthyng @willirath?

So maybe we could support returning the distances via both .xoak.query() and xoak.sel() after all?

If we're going to support both, I think we can either merge this PR and then refactor #32 or the other way around.

@benbovy I like having the indices and distances explicitly available with
distances, indices = ds_mesh.xoak.query( ...

as you suggest. One thing I didn't like about my change is that when a DataArray is input but distances are returned, a Dataset is returned to contain both variables. It's been messing up my code here and there.
Maybe it would be better to keep it simple with .sel (no changes) and if you want to have the distances and indices explicitly, you have to do the two step process you laid out of
distances, indices = ds_mesh.xoak.query(
...    lat=ds_trajectory.latitude,
...    lon=ds_trajectory.longitude,
... )
results = ds_mesh.isel(indices).assign(distances=distances)

But I do think it should be up to you @benbovy and @willirath so let me know what you prefer and I can try to implement, or you can go ahead with it, either way.

I think leaving .xoak.sel() unchanged and as close to the standard xarray .sel() API as possible is what we should do. The proposal to have .query() returning, both, distances and indices sounds like the best way to implement this.


RFC: Better API for custom animations

Goal
This is an RFC (Request for comments) on improving the custom animations API. We're looking for your thoughts on how you use this feature, and how you think it could be improved. If you have encountered any issues not mentioned here, feel free to comment them as well.
Current overview
Currently, the workflow involves:

Creating a new Scene with an AnimationPlayer as root
Set RESET and custom animations on that player
Manually run SceneManager.set_animation_player(YourAnimationPlayerScene) once, early in your game.

I think the first two steps are good, since they allow users flexibility to move their files around as they please and create the scene as they like it, however the third step seems inconvenient.
It definitely gets the job done, but it creates two things I don't like:

It makes users have to call set_animation_player on a script that is potentially completely unrelated to scene transitions (and potentially makes users have to create a script solely for this purpose)
It adds a somewhat unnecesary API method (since it's only called once by a fraction of users)

Proposed Solutions
I can think of two alternatives that would solve these issues:
Plugin popup solution
There are plugins around which will add custom screens to the godot editor itself to configure plugin settings, such as Dialogic does with their dialogue preview options. We could implement something similar in a much smaller scale (perhaps a simple popup with a text input?) to set the path of the animation_player. We would save this data into a .tres file that is loaded when the game starts, so we instantly pick up the path and load it into SceneManager.
The main disadvantage of this approach is basically the difficulty of implementing it. In my experience doing similar plugins for Godot 3, these sorts of "Editor plugins" are a bit annoying to implement since the developer has to reload the plugin every time they make a change and the changes don't always apply on first try. I'm also not sure if a popup triggered from, for example, one of Godot's menu dropdowns at the top of the editor, can be achieved. The closest I've seen is a new "main screen" (such as the 2D, 3D, Script and AssetLib tabs), which seems a little intrusive and way too big for the use-case I'm describing.
Hardcoded path solution
We could set a "hardcoded" path to the AnimationPlayer scene. This could be something like res://SceneManagerAnimationPlayer.tscn or res://addons/scene_manager/SceneManagerAnimationPlayer.tscn, and we could even have both with the second one as a fallback for the first one. If this was implemented, this would work in addition to the current set_animation_player method, so the user could either use these paths OR set a custom one. This would also allow the (in my opinion) not recommended option of manually modifying the scene_manager code to just add paths directly there.
The disadvantages for this approach are several, it's more of a patch than a solution, since we're only adding a couple of places were users CAN just drop the AnimationPlayer and have it work instantly. However it's much easier to implement and it does solve the issue for some people.

Can it not be a custom Project Setting at worst?

That's actually a better idea, I hadn't thought of that

Animations seem to be mostly working properly, and there's not much demand for this feature. I will be closing it for now.


update turn speed to use the delta time multiplier

While the car speed being different based on fps is fixed. The turn speed is still slowed by lower fps.

Not exactly the behaviour I am looking for. Movement is now too fast on low fps. Resolved(ish) in 758b7028a5766e93df3e6f3f91cca210060afe71 . I will re open this later.


Bootstrap Horizontal Form Wrapper Issue.

The div wrapper (form-group) for the horizontal form is not passing the row class.  This causes some responsive issues with the layout.  The first being on that on large screens there is no bottom-margin on the div.  The second being that if a label spans over two lines it will readjust layout at certain widths to have the label for the next row at the right of the form-group prior. Adding the row class via wrapper_html: { class: 'row' } fixes both these issues, and conforms to Bootstraps best practices of having cols reside within rows.

Quoting bootstrap horizontal forms docs:

Use Bootstrap's predefined grid classes to align labels and groups of form controls in a horizontal layout by adding .form-horizontal to the form (which doesn't have to be a <form>). Doing so changes .form-groups to behave as grid rows, so no need for .row.

This tells us the row class shouldn't be necessary, so there might be something else going on on your form. I'm giving this a close, please use the mailing list if you need help, if we end up figuring it's a real Simple Form issue, we can reopen later. Thanks!


requireArrowFunctions: don't check AssignmentExpression or Property

Fixes #1413
Current idea is don't check for those.
// should use method shorthand syntax instead, otherwise `this` becomes undefined.
var a = {
  b: 1,
  normalFunction: function() { return this.b; },
  arrowFunc: () => { return this.b; },  // undefined.b;
  methodShorthand() { return this.b; }
};
var a.arrow = () => { return this.b }; // undefined.b;

Or we figure out if this is being used or some other idea.

Coverage increased (+0.0%) to 98.86% when pulling 3926df09a4facb8bfdb027d9bc03cd4d7a3cadf4 on hzoo:i-1413 into b0a55b5731bdf1752ea3abbd0d4ce5412ab1155a on jscs-dev:master.

This is a good catch. +1 for ignoring. Computing valid  scopes sounds like a JSHint feature more than a style choice.

Yeah the best test was to take the rule and run it through some code to find these issues!


Update sbt-mdoc to 2.3.0

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

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta", artifactId = "sbt-mdoc" } ]

labels: sbt-plugin-update, early-semver-minor, semver-spec-minor, commit-count:1

You'll need to change 2.13.6 to 2.13.7 or 2.13.8 probably to fix the build.
https://github.com/outwatch/outwatch/blob/54c6df3ad38c94b006d44e09eb4ccc96b25f5fac/build.sbt#L212-L216

Superseded by #584.


usePrevious only tracks the previous of the last render and is affected by other hooks

If i have other hooks in a component that are unrelated to what the usePrevious is tracking, some update that causes a rerender from that hook will sync the previous value to the current value. To see an example of this, here's a sandbox. To see the behavior, click + a few times and then click toggle on to toggle unrelated state. Not that the value for prev will update.
In this sandbox I've implemented an alternative usePrevious that tracks a more accurate version of previous.
I can make a PR but wanted to know if maybe we call this new version something else to support the previous version of usePrevious.
I understand that usePrevious tracks the previous render value, but this version may be more strictly useful in an application. Maybe usePreviousDistinct?

:tada: This issue has been resolved in version 11.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

Oh! I knew I've been wanting this hook without even knowing how to think about it. I just saw it pop in my VSCode autocomplete and realized this was just what I needed!
Thank you for this! @streamich  @psachs21


ReadOnlySpan.Split support

Is the feature request related to a problem
This feature request would allow some performance optimizations for NET 6+ targets before NET 9.
Describe the solution
SpanSplitEnumerator was merged and the PR basically contains the needed code. Of course would need preprocessor directives to check for NET 6 or greater and less than NET 9 (to get SearchValues and friends easily).
Describe alternatives considered
One can always split with allocations 💣
Additional context
I'm unsure if I have the capacity to create the PR, even though I fully understand it would be desirable from maintainer's point of view. The coding conventions and rules, I'm not yet familiar with.

6.6.0 should hit nuget in approx 20min

Thank you, great addition!

i found one more mem tweak https://github.com/SimonCropp/Polyfill/pull/217. so u prob want 6.6.1


Servo set_position is implemented and tested on rpi3

Implemented set_position function for servo, using same style as origin gpiozero.
Tested on raspberry pi3.

thanks i will take a look and test a bit later


Using multiple ecto repos causes timeout on unique_index

I have created a sample repository to show this issue at https://github.com/Gazler/ecto_index_timeout
The gist of the issue is that if I create a migration with a unique index:
defmodule FirstApp.Repo.Migrations.CreateUsers do
  use Ecto.Migration

  def change do
    create table(:users) do
      add :name, :string
    end

    create unique_index(:users, [:name])
  end
end

And a model:
defmodule FirstApp.User do
  use Ecto.Model

  schema "users" do
    field :name, :string
  end
end

Then use the same model in an umbrella application with a different repo in the tests:
defmodule FirstAppTest do
  use ExUnit.Case

  test "creating a user" do
    FirstApp.Repo.insert!(%FirstApp.User{name: "foo"})
  end
end

defmodule SecondAppTest do
  use ExUnit.Case

  test "creating a user" do
    SecondApp.Repo.insert!(%FirstApp.User{name: "foo"})
  end
end

This causes a timeout. Interestingly, if the same Repo is used in both instances then an Ecto.ConstraintError is raised.

Actually - on after another test - it seems like the umbrella part is irrelevant. Having the unique index and two repos causes the same issue. I'll make another github repo to demonstrate this.

Here's another version that has the same problem but doesn't use an umbrea:
https://github.com/Gazler/ecto_index_timeout/tree/one_app

To run tests in consistent way, pool Repo use one connection and
transaction is started: (** BEGIN [] OK** )
Ecto.Adapters.SQL.begin_test_transaction(FirstApp.Repo)
After tests in first_app this connection stil have active transaction.
In second app we have DB deadlock because is inserted same "foo" on uniqe index.
add on_exit in first test:
setup do
    on_exit fn ->
      Ecto.Adapters.SQL.rollback_test_transaction(FirstApp.Repo)
    end
    :ok
end

and tests works fine
First output:
petrica@mac ~/tmp/elixir-ecto/ecto_index_timeout > mix test
==> first_app

09:12:27.914 [debug] BEGIN [] OK query=0.3ms

09:12:27.915 [debug] SAVEPOINT ecto_sandbox [] OK query=0.3ms

09:12:27.962 [debug] INSERT INTO "users" ("name") VALUES ($1) RETURNING "id" ["foo"] OK query=0.9ms
.

Finished in 0.1 seconds (0.1s on load, 0.01s on tests)
1 test, 0 failures

Randomized with seed 949991
==> second_app

09:12:28.022 [debug] BEGIN [] OK query=0.2ms

09:12:28.023 [debug] SAVEPOINT ecto_sandbox [] OK query=0.4ms

  1) test creating a user (SecondAppTest)
     test/second_app_test.exs:4
     ** (exit) exited in: GenServer.call(#PID<0.194.0>, {:query, "INSERT INTO \"users\" (\"name\") VALUES ($1) RETURNING \"id\"", ["foo"]}, 5000)
         ** (EXIT) time out
     stacktrace:
       (elixir) lib/gen_server.ex:539: GenServer.call/3
       (postgrex) lib/postgrex/connection.ex:102: Postgrex.Connection.query/4
       (ecto) lib/ecto/adapters/postgres/connection.ex:31: Ecto.Adapters.Postgres.Connection.query/4
       (stdlib) timer.erl:197: :timer.tc/3
       (ecto) lib/ecto/adapters/sql.ex:246: Ecto.Adapters.SQL.query/7
       (ecto) lib/ecto/pool.ex:159: Ecto.Pool.do_run/4
       (ecto) lib/ecto/adapters/sql.ex:231: Ecto.Adapters.SQL.query/6
       (ecto) lib/ecto/adapters/sql.ex:209: Ecto.Adapters.SQL.query/5

Finished in 5.0 seconds
1 test, 1 failure

Randomized with seed 42019

@Tica2 Thanks for the advice, this fixes the issue.
I was also informed that it is better to use a single shared repo to take advantage of query caching.


add: windscribe-alpha-gui-deb

Fixes #1586

CI fails because docker doesn't support systemctl

Send a screenshot of a working install.


python311Packages.pytest-examples: 0.0.10 -> 0.0.12

Description of changes
Changelog: https://github.com/pydantic/pytest-examples/releases/tag/v0.0.12
cc @fabaff

Things done

Built on platform(s)

[ ] 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
[ ] 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.

Result of nixpkgs-review pr 324548 run on aarch64-linux 1

8 packages marked as broken and skipped:

khoj
khoj.dist
open-webui
open-webui.dist
python311Packages.langchain-chroma
python311Packages.langchain-chroma.dist
python312Packages.langchain-chroma
python312Packages.langchain-chroma.dist

32 packages built:

pentestgpt
pentestgpt.dist
python311Packages.instructor
python311Packages.instructor.dist
python311Packages.langchain
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python311Packages.langsmith
python311Packages.langsmith.dist
python311Packages.pytest-examples
python311Packages.pytest-examples.dist
python312Packages.instructor
python312Packages.instructor.dist
python312Packages.langchain
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist
python312Packages.langsmith
python312Packages.langsmith.dist
python312Packages.pytest-examples
python312Packages.pytest-examples.dist
shell-gpt
shell-gpt.dist

Result of nixpkgs-review pr 324548 run on x86_64-darwin 1

4 packages marked as broken and skipped:

khoj
khoj.dist
open-webui
open-webui.dist

36 packages failed to build:

pentestgpt
pentestgpt.dist
python311Packages.instructor
python311Packages.instructor.dist
python311Packages.langchain
python311Packages.langchain-chroma
python311Packages.langchain-chroma.dist
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python311Packages.langsmith
python311Packages.langsmith.dist
python311Packages.pytest-examples
python311Packages.pytest-examples.dist
python312Packages.instructor
python312Packages.instructor.dist
python312Packages.langchain
python312Packages.langchain-chroma
python312Packages.langchain-chroma.dist
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist
python312Packages.langsmith
python312Packages.langsmith.dist
python312Packages.pytest-examples
python312Packages.pytest-examples.dist
shell-gpt
shell-gpt.dist

Result of nixpkgs-review pr 324548 run on x86_64-darwin 1

4 packages marked as broken and skipped:

khoj
khoj.dist
open-webui
open-webui.dist

36 packages built:

pentestgpt
pentestgpt.dist
python311Packages.instructor
python311Packages.instructor.dist
python311Packages.langchain
python311Packages.langchain-chroma
python311Packages.langchain-chroma.dist
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python311Packages.langsmith
python311Packages.langsmith.dist
python311Packages.pytest-examples
python311Packages.pytest-examples.dist
python312Packages.instructor
python312Packages.instructor.dist
python312Packages.langchain
python312Packages.langchain-chroma
python312Packages.langchain-chroma.dist
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist
python312Packages.langsmith
python312Packages.langsmith.dist
python312Packages.pytest-examples
python312Packages.pytest-examples.dist
shell-gpt
shell-gpt.dist

Result of nixpkgs-review pr 324548 run on aarch64-darwin 1

2 packages marked as broken and skipped:

khoj
khoj.dist

2 packages failed to build:

open-webui
open-webui.dist

36 packages built:

pentestgpt
pentestgpt.dist
python311Packages.instructor
python311Packages.instructor.dist
python311Packages.langchain
python311Packages.langchain-chroma
python311Packages.langchain-chroma.dist
python311Packages.langchain-community
python311Packages.langchain-community.dist
python311Packages.langchain-core
python311Packages.langchain-core.dist
python311Packages.langchain-text-splitters
python311Packages.langchain-text-splitters.dist
python311Packages.langchain.dist
python311Packages.langsmith
python311Packages.langsmith.dist
python311Packages.pytest-examples
python311Packages.pytest-examples.dist
python312Packages.instructor
python312Packages.instructor.dist
python312Packages.langchain
python312Packages.langchain-chroma
python312Packages.langchain-chroma.dist
python312Packages.langchain-community
python312Packages.langchain-community.dist
python312Packages.langchain-core
python312Packages.langchain-core.dist
python312Packages.langchain-text-splitters
python312Packages.langchain-text-splitters.dist
python312Packages.langchain.dist
python312Packages.langsmith
python312Packages.langsmith.dist
python312Packages.pytest-examples
python312Packages.pytest-examples.dist
shell-gpt
shell-gpt.dist


Not getting with Binary gap

Hello Roseliux,
I have entered 547 and binary is<PHONE_NUMBER>100011, it means we have 2 binary gaps here
1-000-1-000-11
But I am getting output 3

hello @udkumar sorry for the late response, this repository was just to keep my solution not planning on maintaining the repository, but feel free to share your solution and I will add to the repository

hello @udkumar , try this out.
def solution(n)
  n =  n.to_s(2)
  n_array = n.chars
  x_array = []
  c = 0
  for i in n_array
  	if i == "0"
  	  c += 1
  	else
  	  x_array.push c
  	  c = 0
  	end
  end
  return x_array.max
end

@roseliux , please let me know what you think.

Hello Roseliux,
I have entered 547 and binary is<PHONE_NUMBER>100011, it means we have 2 binary gaps here
1-000-1-000-11
But I am getting output 3

I think response 3 is right, according to the explanation, 3 is the longest binary gap.

@jwaiswa7 It looks good but take a look at this cleaner solution
https://github.com/roseliux/codility-ruby-solution/blob/master/Lesson 1/BinaryGap2.rb#L1

It looks beautiful


methodOwner is incorrect on nested isSelf methods

# typed: true

class A
  def self.foo
    def self.bar
      puts T.reveal_type(self) # error: Revealed type `T.class_of(A)`
    end
  end
end

A.foo # ok
A.bar # ok

This test currently fails and should pass.
→ View on sorbet.run

Fixed in #2009


Add hour to nightly release name [phab:D422]

Created by schroederc at 2015-07-23 21:19:40:

shahms accepted this revision at 2015-07-23 21:21:02

schroederc closed via commit e9b974770384b92174e5921135f256e3ca4ea06c at 2015-07-23 21:24:11


can't build

Fedora Rawhide
#...
[ 33%] Generate JSON tl source files
[ 33%] Building CXX object tddb/CMakeFiles/tddb.dir/td/db/SqliteKeyValueAsync.cpp.o
[ 34%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/HttpOutboundConnection.cpp.o
[ 34%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/HttpQuery.cpp.o
[ 34%] Building CXX object tddb/CMakeFiles/tddb.dir/td/db/detail/RawSqliteDb.cpp.o
[ 34%] Built target tl_generate_json
Scanning dependencies of target example
Scanning dependencies of target bench_actor
[ 35%] Linking CXX static library libtddb.a
[ 35%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/HttpReader.cpp.o
[ 35%] Building CXX object benchmark/CMakeFiles/bench_actor.dir/bench_actor.cpp.o
[ 36%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/Socks5.cpp.o
[ 36%] Building CXX object tdactor/CMakeFiles/example.dir/example/example.cpp.o
[ 36%] Linking CXX executable bench_actor
[ 37%] Linking CXX executable example
[ 37%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/SslFd.cpp.o
[ 37%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/TcpListener.cpp.o
[ 37%] Built target tddb
/usr/lib/gcc/x86_64-unknown-linux/6.2.0/../../../../x86_64-unknown-linux/bin/ld: cannot open output file example: Is a directory
collect2: error: ld returned 1 exit status
make[2]: *** [tdactor/CMakeFiles/example.dir/build.make:99: tdactor/example] Error 1
make[1]: *** [CMakeFiles/Makefile2:1012: tdactor/CMakeFiles/example.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[ 38%] Building CXX object tdnet/CMakeFiles/tdnet.dir/td/net/Wget.cpp.o
Scanning dependencies of target binlog_dump
[ 38%] Linking CXX static library libtdnet.a
[ 39%] Building CXX object tddb/CMakeFiles/binlog_dump.dir/td/db/binlog/binlog_dump.cpp.o
[ 39%] Built target tdnet
[ 39%] Built target bench_actor
[ 39%] Linking CXX executable binlog_dump
[ 39%] Built target binlog_dump

You probably should run cmake from a separate empty directory.
mkdir build
cd build
cmake ..
make

Nevermind, it was a issue of my flatpak manifest. I forgot to add the "builddir": true, option. Thanks.


Upload FHIR survey results to GCP

This PR adds functionality for surveys created using the Android FHIR SDK to upload their results as FHIR QuestionnaireResponse JSON to Firebase Firestore.

Thanks @ibnzterrell! That is a great idea - something that we should discuss with @kellynguyvn


dbeaver/dbeaver#20513 Fix uuid value manager for panel

incorrect id is shown as a header in JSON view

verified


Speed up combo fields in the editor

Each field requires several heavy queries to the database, resulting in very slow load of the editor, sometimes 10-20 seconds. This needs to be fixed in 0.7. What we can do:

Pre-calculate common combobox values after loading the data, in background.
Alternatively, query the database on opening the editor in the background, and update the values when done. And cache them!
Cache fields (and reset cache after loading data) — would speed up their extraction as well, e.g. translations.

Turns out simple caching is enough. We'll see if there are reports about slowness after 0.7.


Tag new 5.3 patch version with bugfixes

Can you update 5.3 branch to new patch version with these bugfixes - https://github.com/laravel/framework/commit/666a752dd9679448905a38f2a975a881060c4178

Issue should be renamed to "support 5.3 if it is allegedly still supported (or tell people that yeah, they'll have to migrate their applications every 6 months)".

As a matter of fact, 5.3 has been released on 23 Aug 2016, that's seven months...
So technically, it's already on "extended support" (security fixes only) and it shouldn't be expected to receive bugfixes such as those you referenced (#17983)...
(and yeah, I'm tired of constantly migrating)

Though, #17983 had been made in the six-month main support window.
(it's already fugacious, don't make it shorter :trollface:)

Thanks, will tag a release soon.


Need a -D flag, and option for enabling "continue" option in gradle.properties

Expected Behavior
Can we have "-D" flag for --continue, just like we have for enabling parallel builds (org.gradle.parallel=true), like this (org.gradle.continue=true). - I checked the docs and couldn't find it.
Current Behavior (optional)
No response
Context
need this option to enable globally for users and on CI Systems

This capability is available for parallel builds - org.gradle.parallel=true
Looking for the same capability for continue - org.gradle.continue=true

Thank you for your interest in Gradle!
This feature request is in the backlog of the relevant team, but this area of Gradle is currently not a focus one, so it might take a while before it gets implemented.

@jbartok I have completed this feature, can you please review this commit.

@Korov, thank you for your contribution. We will, but it might take some time.

@jbartok I have completed this feature, can you please review this commit.

@Korov  - Thank you so much for your contribution


Switch on PROD feature flag to turn off clarification questions

This needs to be deployed at 5pm today.

:+1:


Fixed PIT joins

Fixed an issue in joins when satellites without a load end date were used to build a PIT table

BigQuery Testing is passed.

Please use the variable alias, not the hardcoded ledts. Here in Intersport we use a different naming convention for the LOAD END DATE TIMESTAMP, so this will not work when trying to build a PIT with a s0 satellite here.


rasterization_inria_wrapper

There are rasterization and rasterization_inria_wrapper in gsplat.render, why only used rasterization in splatfacto.py (SplatfactoModel.get_outputs) different from the Inria-3DGS using rasterization_inria_wrapper

Anyone can help me?


Typescript type definition generation using Rosetta

Rosetta features automatic typescript definition generation (.d.ts file) from ScriptSharp code. You can view the default output here: https://gist.github.com/janis-veinbergs/4aec76d100de72eeb6784700314da530
I actually had to remove empty [ScriptNamespace("")] attributes from Lookup, Option and Utility classes, otherwise Rosetta throws The ScriptNamespace attribute contains an overriden namespace value which cannot be accepted!. I don't know the implications of it.
Here is the commit that generates .d.ts on build: https://github.com/janis-veinbergs/SparkleXrm/commit/9015ac1c85f13d152cbc9d40d6925d6980e4606d
I'd like some feedback from @scottdurow - what is the implicatjion of removing [ScriptNamespace("")] for said files?
I'd like some additional feedback from whoever has done this before - are those .d.ts files usable OOTB?
I'v yet to see if this plays nicely with XrmDefinitelyTyped and if SparkleXrm definitions should be somehow changed to play nicely with those definitions.
I opened this issue to inform the comunity on a promising approach. + I know michaelaird has tried this out (from here) and ideally would like some feedback from him.
P.S. It was most feature-complete/successful methodology of generating typescript definitions. I tried other tools too, but they weren't providing useful results for ScriptSharp code:

netjs
dts-gen
tsc --declaration --emitDeclarationOnly --allowJs (Typescript compiler) by feeding in SparkleXrm.js

I actually don't expect someone would notice or care or provide feedback, as the world has moved on, but anyways... even if it helps someone with information, profit! #47

Ha, ScriptName attribute is not honored by Rosetta. I'll try to fix that and and let's see if owner will include those changes in Rosetta: https://github.com/andry-tino/Rosetta/pull/59
There are other issues when I tried generating mscorlib definitions. So definitions are still work-in-progress.


flush semantics are unclear

The semantics of flush is a little unclear to me, i.e. when I call flush(cb) I'd expect that all data that I have write(chunk) would be flushed when cb is invoked. However, unlike flushSync this is not the case. flush doesn't actually flush the buffer to the worker, it just waits for whatever already has been flushed to worker is done.

I notice there is actually a todo for this already:
  // TODO write all .buf


Observatory not placing data in data units after first 1000 units

After completely filling a data unit from the observatory, every attempt to place data on any data unit will remove the data from the observatory buffer but will not add data to the data unit.

Just tried with a non-empty chip (extra data from satellite that would not fit in ADP buffer) and it transferred data. Will post new developments as they occur.

so if the chip lists as storing "Some Random data" it removed data from the observetory without adding it to the chip?

Yes.

A video may be helpful, I'm not able to replicate this

Video is up unlisted.
Also, I am providing a modlister link for this collection of mods in case that helps any.
Video: https://youtu.be/xRGsNTGENek
Modlister: https://modlister.com/user/GuyRunningSouth

Will try to get to this tomorrow

Thank you very much for your help!

You're quite welcome. Thank you for such an awesome mod.


Feature RQ: Config options

Hi! I enjoy this mod quite a bit but do have one issue in that nothing about the generation is customizeable. Blacklisting dimensions is one thing, as well as perhaps being able to adjust the rates of specific types and their sizes. Personally I wouldn't mind if this was either nice fancy modmenu integration or simply datapack compat, just keeping the rockin' rocks from generating in unfitting modded dimensions

I personally would like to be able to disable this in modded biomes in the Nether but not the Overworld. Because the mod I use to add biomes to the Nether already adds lots of little stuff to the ground of its own biomes but not the Vanilla ones so this mod makes them all equally interesting. Meanwhile it helps all of the overworld so I wouldn't want to turn it off entirely from modded biomes.


The text after selecting icon always inside the em or i tags

When we selecting the icons and after that we type the text, the text always goes to inside the  or  tags, and cannot put text outside that tags.
So, here is the example :

 sd

The text sd is wrapped inside the  tags, that i should not. This is what we expected

sd

Btw Also looks like the plugin didn't support fa version 5, the icons are not showing if we use version 5 font-awesome.

I don't think there's anything that can be done for the text problem, this is a problem with redactor itself I believe.
I noticed if you add your icons after typing the text you don't have that problem

Ha I didn't see your PR included a fix for that, I added it now


Validating timestamp value in interval

It is hard to validate timestamp value exactly, but in most cases tester can expect timestamp in some interval. Especially in combination with citrus:currentDate().
<Timestamp>@between(2001-12-17T09:30:47.0Z,2001-12-17T09:30:50.0Z)@</Timestamp>
I haven't found anything like this in the documentation so I assume this is not yet possible.

No there is no such validation matcher available, I'll see if we can have something like this in upcoming versions

i am drunk
On Tue, Jul 14, 2015 at 10:53 PM, Christoph Deppisch <
<EMAIL_ADDRESS>wrote:

No there is no such validation matcher available, I'll see if we can have
something like this in upcoming versions
—
Reply to this email directly or view it on GitHub
https://github.com/christophd/citrus/issues/42#issuecomment-121382278.

--
Miroslav Matejovsky

Done, see commit https://github.com/christophd/citrus/commit/1aecd240053489be0867623a81c88c98daaf7310


Cannot read property ‘firstChild’ of undefined

I am getting this error.
The web says this is caused by having 2 different versions of React.
I have tried every way to make sure that I only have one version, and I am still getting this error.
It is happening on the second render to a component, IF I am using opal-rspec in the browser.
I do not believe this is happening unless I am running opal-rspec.
I have done some basics like putting a console.log statement at the start of the react.js (unminimized) file and according to that log react.js is only loading once.
I have also made sure that react-source is not in my gem file.
Any clues or things to try would be greatly appreciated.

Okay it seems like this is something to do with react's state between the rspec tests.   If I move the "failing" test to be the first test, everything is fine.
I do have after(:each) { React::API.clear_component_class_cache } in the test but it makes no difference.
So I am pretty sure somehow the opal-rspec environment is clobbering react

Okay I got this figured out -- the problem occurs if a component that was used in a static rendering is later asyncronously updated.  That seems to screw everything up.
I don't know yet if this is a problem with opal, react.rb, or perhaps with reactjs

That's weird, would you mind pasting the code which failing the test?

This issue was moved to reactrb/reactrb#27


Inline errors when bad username during login

@cjb I tried to clean up what I had so far but I'm still not exactly happy in how the error is propagated and later how it's decided what component to display. Would you mind taking a look? Thanks!

@zapu Looks good! But let's do the decoding of error number into a string in the username container.js rather than the display component, and add a bool for whether to display the sign up link which is set by the container too? It can be a function in that file that's called by mapStateToProps.
Does that make sense? The display components should usually only receive things like bools and strings.

Thanks! I moved logic to shared/provision/username-or-email/container.js, but I'm not sure if I didn't try to be "too clever" there. Can you take another look?

@zapu Pretty good! I'm surprised that flow types successfully -- inlineError is a required prop of the Username component in index, but it's only returned via the spread if there was an error.  Any idea what I'm missing?
It does seem better to have mapStateToProps show what it's returning explicitly, with something like this (untested), what do you think?
const decodeInlineError = inlineRPCError => {
  let inlineError = ''
  let inlineSignUpLink = false
  if (inlineRPCError) {
    switch (inlineRPCError.code) {
      case RPCTypes.constantsStatusCode.scnotfound:
        inlineError = "This username doesn't exist."
        inlineSignUpLink = true
        break
      case RPCTypes.constantsStatusCode.scbadusername:
        inlineError = 'This username is not valid.'
        inlineSignUpLink = false
        break
    }
  }
  return {inlineError, inlineSignUpLink}
}

const mapStateToProps = state => {
  const {inlineError, inlineSignUpLink} = decodeInlineError(state.provision.inlineError)
  return {
    error: state.provision.error.stringValue(),
    inlineError,
    inlineSignUpLink,
    // So we can clear the error if the name is changed
    submittedUsername: state.provision.username,
  }
}

I think its because its nullable. Ill take another look in a bit, thanks!

Wiadomość napisana przez Chris Ball<EMAIL_ADDRESS>w dniu 23.03.2019, o godz. 16:01:
@zapu Pretty good! I'm surprised that flow types successfully -- inlineError is a required prop of the Username component in index, but it's only returned via the spread if there was an error. Any idea what I'm missing?
It does seem better to have mapStateToProps show what it's returning explicitly, with something like this (untested), what do you think?
const decodeInlineError = inlineRPCError => {
let inlineError = ''
let inlineSignUpLink = false
if (inlineRPCError) {
switch (inlineRPCError.code) {
case RPCTypes.constantsStatusCode.scnotfound:
inlineError = "This username doesn't exist."
inlineSignUpLink = true
break
case RPCTypes.constantsStatusCode.scbadusername:
inlineError = 'This username is not valid.'
inlineSignUpLink = false
break
}
}
return {inlineError, inlineSignUpLink}
}
const mapStateToProps = state => {
const {inlineError, inlineSignUpLink} = decodeInlineError(state.provision.inlineError)
return {
error: state.provision.error.stringValue(),
inlineError,
inlineSignUpLink,
// So we can clear the error if the name is changed
submittedUsername: state.provision.username,
}
}
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

I tried making it inlineError: string instead of inlineError: ?string and it still passed flow :)  Seems like a Flow bug, though usually when I feel that way it turns out to be a cjb bug..

thanks this is perfect now!

@patrickxb can you take a look at more cleanups in engines regarding logging in with e-mail? thanks


Pin hubot-yammer version to git master

Pinning hubot-yammer version to master branch of https://github.com/athieriot/hubot-yammer: since we can’t publish new versions to npm and I wouldn’t want to bug @athieriot with this too much, it would be better if we fetch the yammer adapter directly from git.
We do it with some other adapters, too; nothing new here.

Updated the readme to include pushing latest, too.


comparison to None will result in an elementwise object comparison

We should probably address the warning:
"FutureWarning: comparison to None will result in an elementwise object comparison in the future.
if N_k == None::"
Sooner rather than later so that people don't worry.  Levi, can you manage that? Also, Levi, not sure why you don't appear in the possible assignee lists.  Perhaps I need to add you . . .

Added @Lnaden to maintainers and developers list.

Be sure to make a PR---don't commit to this repo directly!

There are several comparisons that will incur this error, specifically the first instance of kwarg == None where kwarg's type is a numpy array. I know this is to set the default value of the kwarg in a more complex way than the function's def statement will allow.
There are 2 ways I can fix this, depending on what we expect the user to feed in.

Change all the == None statements that can be arrays to is None. If any array is set to the kwarg, this test returns False without tripping the warning.
Test for both the None object, an empty array, or an array of None objects.

The first case I can come up with a PR to address the instances that will trip the warning pretty quick. The second case will also check if the user passes in an empty array or an array of None, but I don't have a clean solution that does not trip the warning yet.
I think the first case would be better since the treatment would be the same to an empty array or None array would be the same, unless we want to handle those two arrays differently.

PR #201 should resolve this. I did not go through the old_mbar.py file and make changes since I figured people will not care about a future warning on old files.


ETH withdrawn

When sending ETH / Tokens from the GUI the 0.0001 fee does not need to be applied as in ETH the full amount is sent (the fee is paid from your balance but not from the UTXO [there are no UXTO in ETH]): here is a sample transction with above parameters:
etherscan.io/tx/0x29fccae8b2015769b5c6536c84451f96aecd48338a400537e77ee78ae85bf0e7
BTW: it works!!

https://github.com/KomodoPlatform/BarterDEX/commit/fb27483bba2d3ffdd03e7b136eed83274f99e27c


ambiguous debug log

excerpt from iptables man-page:
-A, --append chain rule-specification
-I, --insert chain [rulenum] rule-specification
-D, --delete chain rulenum

-A does not accept rulenum

@zhangyoufu Yes, I will fix this!


Check the pinned comment at the top of the issues page.

Check the pinned comment at the top of the issues page.
Originally posted by @MW2K in https://github.com/gibbed/SteamAchievementManager/issues/239#issuecomment-974799716

Thank you!


Update centos 7, Ubuntu 8 and DataHub Jupyter

Summary

Related issue :

Enforcing merge as images expire today


Fix for ft-transfer mismatching arguments

Closes: #869
Description
See the issue for a discussion of the scenario for this bug.

For contributor use:

[X] Updated the Unreleased section of CHANGELOG.md with the issue.
[ ] If applicable: Unit tests written, added test to CI.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

Codecov Report

Merging #870 (cdebabf) into master (c32393a) will decrease coverage by 0.0%.
The diff coverage is 0.0%.

@@           Coverage Diff            @@
##           master    #870     +/-   ##
========================================
- Coverage    43.5%   43.4%   -0.1%
========================================
  Files         170     170
  Lines       12655   12683     +28
========================================
  Hits         5507    5507
- Misses       7148    7176     +28

Impacted Files
Coverage Δ

relayer-cli/src/commands/tx/transfer.rs
0.0% <0.0%> (ø)

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 c32393a...cdebabf. Read the comment docs.


Filter Navigation Menu based off of permissions

Do not display a menu item if user is missing the permissions for actions against it.

added 2m of time spent

mentioned in commit 45cc34284a7b89aa3d61e61cfa3a12c73165127b


Mobaxterm incorrectly converts UNIX newline chars

cat displays the scripts correctly. In less, they appear as M. This causes all tests to fail with error message:
$'\r': command not found

No longer using MobaXterm


split-attrs: equalise-attributes

Possible need ?
At least must ensure that the operation makes sense with the new split-form attributes,
( i.e. using existing equalise code with the new 'cube.attributes' combined access approach : #4983 )
However, we may need to handle newly-possible collisions between matching local+global settings in a more intelligent way

possibly provide automatic resolution of some such problems  -- or detect + error/warn
possibly add extra keywords to control different resolution modes

The spec for this, according to the docstring, is
This function deletes any attributes which are not the same for all the given cubes.  The cubes will then have identical attributes, and the removed attributes are returned.
The implementation uses iteration over keys, so that could in some cases break the contract.
E.g. with cube_a.attributes.globals['x'] = 3 and cube_b.attributes.locals['x'] = 3, it will conclude that these are equal, so neither cube will be modified.  But still, afterwards, "cube_a.attributes != cube_b.attributes" .
This could cause breaking changes when cubes are loaded from different sources (at least, if any are netcdf), or constructed manually.
And this is potentially important, because then the cubes would not merge.
( since merge does equality testing of attributes -- see here  )
Probably


🏃Add release notes utility / update controller-tools & CRDs

Signed-off-by: Vince Prignano<EMAIL_ADDRESS>

What this PR does / why we need it:
This PR adds the make release-notes target
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

oh boy, the tools controller-runtime is different than the actual controller-runtime dev 🙃
they should be kept in sync

/lgtm
/hold
holding for @chuckha

/hold cancel


Snippets as separate files

Requesting better snippet storage ystem:
The current implementation of snippets' data is narrow and awkward. By clustering all snippets by language into JSON files it makes it difficult to share, version, and create snippets per projects. SublimeText's implementation of file-per-snippet in a user-defined folder structure is much more flexible.
Perhaps a middle ground would be allowing multiple namespaced JSON files to be concatenated and loaded. E.g. less.json, less_SomeName.json, less_ProjectName.json, etc
Version of VS Code 1.18.1

closing this as dupe of #22661 (snippets knowing their scope/language) and #13182 (global snippets)


add RTL support by 'transformDirection' prop

Related issue: #453
Adding support for RTL direction websites

Has this been included yet?

I dont think, i need to add support for resizing layout with rtl driection.

Can i help?

sure

Where should we chat?
My first finding is that this package depends on React-Resizable,
which has to be modified based on direction to subtract instead of add delata.X
on this line
or if we shouldn't mess around much, we could add onResize direction handler ?
along with a set of new css for react-resizable-handle class

Whatsapp, Email, Zoom, Skype?
This is my email address<EMAIL_ADDRESS>Send me your phone number via mail with the country prefix to add you to the whatsapp.

I have sent to you my number in an email.
I have committed the changes to your repo.
But unfortunately it'll depend on @STRML to accept my pull request on React-Resizable

i am looking for this support of rtl too

I'm also looking for RTL support. @Roshdy have you ever gotten it to work?


Dev landing merge

Merge recent changes from dev_landing into master. Changed master to branch to deploy from

It looks like you are attempting to push a change to the part repository, for help with deploying please start a new thread in the discussions.

Thank you. I was about to close this to ignore. This pull request was created in error.


Unable to test anything after successful login on h2database [hammerhead not loaded after redirect]

Are you requesting a feature or reporting a bug?
Bug
What is the current behavior?
After a successful login on a memory-only database (h2-1.4.196 from h2database.com), it's impossible to perform any action with the browser anymore as apparently the hammerhead status bar does not show up. This happens after a redirect from the successful login.
What is the expected behavior?
I would expect the hammerhead status bar to stay alive and to be able to complete any new action.
How would you reproduce the current behavior (if this is a bug)?
Download h2-1.4.196.jar
Launch the local server: java -jar h2-1.4.196.jar
Testcafe with either ie,edge,chrome or firefox will fail (use test-h2-login.js below)
Provide the test code and the tested page URL (if applicable)
Tested page URL: http://localhost:8082
Test code
// test-h2-login.js:
import { Selector } from 'testcafe';

fixture('First fixture')
  .page('http://localhost:8082');

test('First test', async t => {

  await t
    .typeText('form#login input[name=driver]','org.h2.Driver',{replace:true})
    .typeText('form#login input[name=url]','jdbc:h2:mem:',{replace:true})
    .typeText('form#login input[name=user]','sa',{replace:true})
    .click('form#login input[type=submit][value=Connect]')
    .debug() // this allows to see the debug is not functional after the redirect. Any other interaction with the redirected page is also impossible

});

Specify your

operating system: Win10 enterprise
testcafe version: 0.20.2
node.js version: 8.11.1

Hi @cmichon! Unfortunately the H2DB control page is built upon the obsolete  feature, and we don't support pages that contain the <frame> and <frameset> tags. Only <iframe> is supported.

Hi @AndreyBelym: that would discard entirely TestCafe from my comparative study. This is unfortunate...
Though I understand the decision taken (obsolescence of several legacy html markers), there are plenty of sites out there still using these markers.
FWIW, Cypress and CodeceptJS have no issue at all supporting H2 database web application and its frame/frameset legacy markers.


feat: add container

closes #133

upps, it should be max-with, changing it...

I think container is more like a preset of max-width in different viewport?

@hannoeru It is here: https://tailwindcss.com/docs/container, we can move to here layout module.

Should I change to?:

layouts.ts  renamed to overflow.ts
container.ts with the 2 rules here.

I think container is more like a preset of max-width in different viewport?

yeah, sorry, testing...

I keep container rule on size.ts module.

ok, it is wrong, updating again...

@hannoeru  Sorry for the delay, I switch to my work, I'm back.
So the container rule will add { 'width': '100%'} and all the corresponding @media max-width declared on breakpoints, but it also needs to be itselft responsive, that is, we need also add bp variants to container, for example lg:container. In that case we need to rewrite the styles using only breakpoints from the current and down in the current list (from current and up in size), following with the lg:container example, the resulting rules will contain only lg, xl and 2xl styles for @media max-width.
Should only handle : or also support - variant?

@hannoeru @antfu The problem with this rule is that is not a normal breakpoint rule, by default , it will configure the max-width for each theme breakpoint, but can also be used with breakpoint variant (from the breakpoint and up, see tailwind docs here https://tailwindcss.com/docs/container#responsive-variants).
The container rule will result on this:
/* class="container" or container attr */
.container,[container=""]{width:100%}
@media (min-width:640px) {.container,[container=""]{max-width:640px}}
@media (min-width:768px) {.container,[container=""]{max-width:768px}}
@media (min-width:1024px) {.container,[container=""]{max-width:1024px}}
@media (min-width:1280px) {.container,[container=""]{max-width:1280px}}
@media (min-width:1536px) {.container,[container=""]{max-width:1536px}}
/* class="md:container" or md:container attr */
@media (min-width:1024px) {.lg\:container,[lg\:container=""]{width:100%;max-width:1024px}}
@media (min-width:1280px) {.lg\:container,[lg\:container=""]{max-width:1280px}}
@media (min-width:1536px) {.lg\:container,[lg\:container=""]{max-width:1536px}}

I have changed breakpoint variant to exclude container to be processed on variantBreakpoints, then added its own module on rules and handle the breakpoint variant on the new module. It also includes a cache, but we can remove it.
I don't know the implications aboout reusing the @media (min-width...) {}: I cannot use Variant and cannot allow breakpoints variant to process it, so the only way I have found is to exclude from latter.

we should also review if we need to add the center and padding options: https://tailwindcss.com/docs/container#customizing

Missing ; in end of css property?
.container,[container=\\"\\"]{width:100%}

.container,[container=\\"\\"]{width:100%;}

I don't add it, see for example animate for single or last entries.


Fix Param Naming Convention In RegisterBasicUserAPIRoute

Change the parameter naming convention to be consistent with what we have for Parameters (camelCase)

Fixed Here


Async response / std::move stopped working

Dear all,
after an exhaustive review, this is the last commit (https://github.com/oktal/pistache/commit/c613852cb2595f9a1c9a4847ed0b0aaa017e0d81) that worked for async responses using std::move as suggested in https://github.com/oktal/pistache/issues/197.

Please let me know if there will be any fixes for that.
Thanks,
Pablo.

There is a unit test ("async_test") that passes.  Odd.
Please create a PR that implements a minimal unit test that replicates the problem.  I will then clone the PR and attempt to diagnose on my own system.

I have test for that now, but cannot create branches in pistache.
Could you please add me as collaborator?

Hi,
Any update on that? As I said I have a minimal test that reproduces this problem.
If you don't want add me as collaborator to add this PR, that's OK, I can upload a git patch this issue.
I just need to know if you will be able to take care of this in the next few weeks?
Thanks,
Pablo.

Sorry, I missed your earlier message.  This is what you should do:

Create a fork of pistache into your own github account.
In your fork, create a branch.
Add your unit test to your branch.
In your branch, in the github UI, there is a button to create a pull request taht will place the PR into the upstream (eg, oktal/pistache).
From there, I (and/or others) can checkout your branch and dig into the problem.


Revert "fix: DEV-2364: Annotation history name/timestamp spacing improvement"

Reverts heartexlabs/label-studio#2433

/help


feat: make typesafe library transient

Checklist

[ ] I've run ./gradlew test from the root directory to see all new and existing tests pass
[ ] I've followed the harvest-client code style
[ ] I've read the [Contribution Guidelines][contribution guidelines]
[ ] I've updated the documentation if necessary.

Motivation and Context
The Harvest client needs a com.typesafe.config.Config object input as configuration, so it will be the best to expose com.typesafe.config dependency as a transient library so that projects that uses this library could use this client without adding any additional dependencies
Description

Codecov Report

Merging #14 (56ae16e) into develop (103733f) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #14   +/-   ##
==========================================
  Coverage      87.50%   87.50%
  Complexity       382      382
==========================================
  Files             74       74
  Lines           1353     1353
  Branches          87       87
==========================================
  Hits            1184     1184
  Misses           128      128
  Partials          41       41

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 103733f...56ae16e. Read the comment docs.


mark msgpack-charset as a Charset

remove all reflection

Thanks for the patch.

I thought "remove all reflection" was pretty clear, but I will try to think
of something more verbose to say :)
On Tue, Nov 17, 2015, 12:22 AM Eugene Ma<EMAIL_ADDRESS>wrote:

Thanks for the patch.
—
Reply to this email directly or view it on GitHub
https://github.com/edma2/clojure-msgpack/pull/19#issuecomment-157306568.

I think you provided a sufficient reason (less reflection is better, obviously). I'm mostly curious about what prompted the change. Did this fix a linter warning, or was it just something you found by reading the code?

Hey, sorry about that, I saw this as I was waking up, made a snarky reply, and then forgot about it. Not cool. Anyhow, I think I ran 'lein check' on my own project, and saw the reflection warnings coming from clojure-msgpack and saw that there were some java-interop calls that were compiling into reflective calls because they couldn't infer the type of msgpack-charset.

I see, thanks :)


feat(form-elements): icon change

should this have its own ui test?

not sure we need...
@YonatanKra WDTS?

should this have its own ui test?

not sure we need... @YonatanKra WDTS?

@YonatanKra this raises the question of how success can be authored while error should be triggered

@yinonov - I think handling the success message is not part of the scope of this PR. This PR only changes the icons.


IP v6 issues

Hi,
Thank you for this great script. But there are some things out there, if you know, please answer them for me because I'm just an end user, not a Cloud or network expert.
This script runs fine on an Ubuntu VPS from Vultr provider but if they find out I run multiple proxies on the VPS, they will ban my account.
So I want to run this script on another provider's VPS like Linode or Digital Ocean, but unfortunately it cannot be used because v6 IPs when created, they cannot be accessed from the internet, specifically I can not ping v6 IPs created by your script.
Is there some network configuration that needs to be changed?
Thanks.

I don't know about Linode, but DigitalOcean doesn't give you full /64 subnet - only 16 addresses.
For example:


show task with a particular id

One can just query a particular task using its id to check for more details ie. who created, when was it created even a task description if the param could be added in the future

@Pheon-Dev Some cool updates are on the way in episode 02. We'll be introducing filters, timestamps (for both creation and last modification, including the user id and time).
Regarding this PR, I'm a bit uncertain. The base::show(...) doesn't appear to return anything. Is that intentional?
Don't we use base::get(..) to retrieve entities by their id?

@Pheon-Dev please, check the official rust-web-app repo at
https://rust10x.com/
https://github.com/rust10x/rust-web-app
And there is temporary branches, like the nx-e02-tmp with the new workspace layout and all, like this one.
This is where things are happening now. This was a old prep branch.


Connectors cant be created for federated

SUMMARY
The issue is related to activemq_ha_enabled and activemq_cluster_discovery.
Federated isn't HA, nor is it Clustered. To get connectors created, I have to set activemq_ha_enabled=true and activemq_cluster_discovery=static or provided:

https://github.com/ansible-middleware/amq/blob/5cc8e62337c00b4adc27033cd68b3e9324dc315c/roles/activemq/tasks/main.yml#L7-L33

But doing that will then create cluster-connections, which I don't want:

https://github.com/ansible-middleware/amq/blob/5cc8e62337c00b4adc27033cd68b3e9324dc315c/roles/activemq/tasks/connectors.yml#L62-L74

If I don't set either flags, the broker starts up with federation created but not connectors.
If I set those flags, I get the below error when running the ansible playbook:
TASK [redhat.amq_broker.amq_broker : Create instance amq1 of amq_broker] **************************************************************************************************************************************************************************
[WARNING]: Module remote_tmp /opt/amq/apache-artemis-7.12.0/.ansible/tmp did not exist and was created with a mode of 0700, this may cause issues when running as another user. To avoid this, create the remote_tmp dir with the correct
permissions manually
fatal: [ip-11-0-1-231.eu-west-2.compute.internal]: FAILED! => {"changed": true, "cmd": ["/opt/amq/amq-broker-7.12.0/bin/artemis", "create", "/opt/amq/amq1", "--name", "amq1", "--clustered", "--cluster-user", "amq-cluster-user", "--cluster-password", "amq-cluster-pass", "--max-hops", "1", "--message-load-balancing", "ON_DEMAND", "--failover-on-shutdown", "--staticCluster", "--require-login", "--user", "amq-broker", "--password", "amq-broker", "--host", "localhost", "--http-host", "<IP_ADDRESS>", "--http-port", "8161", "--no-autocreate", "--queues", "queue.in,queue.out", "--data", "/opt/amq/amq1/data", "--nio"], "delta": "0:00:01.228351", "end": "2024-09-24 09:31:03.340651", "msg": "non-zero return code", "rc": 1, "start": "2024-09-24 09:31:02.112300", "stderr": "Expected parameter for option '--static-cluster' but found '--require-login'", "stderr_lines": ["Expected parameter for option '--static-cluster' but found '--require-login'"], "stdout": "\nUsage: artemis help [<args>...]\nuse 'help <command>' for more information\n      [<args>...]", "stdout_lines": ["", "Usage: artemis help [<args>...]", "use 'help <command>' for more information", "      [<args>...]"]}

fatal: [ip-11-0-1-201.eu-west-2.compute.internal]: FAILED! => {"changed": true, "cmd": ["/opt/amq/amq-broker-7.12.0/bin/artemis", "create", "/opt/amq/amq2", "--name", "amq2", "--clustered", "--cluster-user", "amq-cluster-user", "--cluster-password", "amq-cluster-pass", "--max-hops", "1", "--message-load-balancing", "ON_DEMAND", "--failover-on-shutdown", "--staticCluster", "--require-login", "--user", "amq-broker", "--password", "amq-broker", "--host", "localhost", "--http-host", "<IP_ADDRESS>", "--http-port", "8161", "--no-autocreate", "--queues", "queue.in,queue.out", "--data", "/opt/amq/amq2/data", "--nio"], "delta": "0:00:01.304106", "end": "2024-09-24 09:31:03.466758", "msg": "non-zero return code", "rc": 1, "start": "2024-09-24 09:31:02.162652", "stderr": "Expected parameter for option '--static-cluster' but found '--require-login'", "stderr_lines": ["Expected parameter for option '--static-cluster' but found '--require-login'"], "stdout": "\nUsage: artemis help [<args>...]\nuse 'help <command>' for more information\n      [<args>...]", "stdout_lines": ["", "Usage: artemis help [<args>...]", "use 'help <command>' for more information", "      [<args>...]"]}
fatal: [ip-11-0-1-71.eu-west-2.compute.internal]: FAILED! => {"changed": true, "cmd": ["/opt/amq/amq-broker-7.12.0/bin/artemis", "create", "/opt/amq/amq3", "--name", "amq3", "--clustered", "--cluster-user", "amq-cluster-user", "--cluster-password", "amq-cluster-pass", "--max-hops", "1", "--message-load-balancing", "ON_DEMAND", "--failover-on-shutdown", "--staticCluster", "--require-login", "--user", "amq-broker", "--password", "amq-broker", "--host", "localhost", "--http-host", "<IP_ADDRESS>", "--http-port", "8161", "--no-autocreate", "--queues", "queue.in,queue.out", "--data", "/opt/amq/amq3/data", "--nio"], "delta": "0:00:01.643840", "end": "2024-09-24 09:31:03.850280", "msg": "non-zero return code", "rc": 1, "start": "2024-09-24 09:31:02.206440", "stderr": "Expected parameter for option '--static-cluster' but found '--require-login'", "stderr_lines": ["Expected parameter for option '--static-cluster' but found '--require-login'"], "stdout": "\nUsage: artemis help [<args>...]\nuse 'help <command>' for more information\n      [<args>...]", "stdout_lines": ["", "Usage: artemis help [<args>...]", "use 'help <command>' for more information", "      [<args>...]"]}

Sample inventory:
---
federated:
  hosts:
    ip-11-0-1-001.eu-west-2.compute.internal:
      amq_broker_instance_name: amq1
      amq_broker_connectors:
      - name: artemis
        scheme: tcp
        address: ip-11-0-1-001.eu-west-2.compute.internal
        port: 61616
      - name: amq2
        scheme: tcp
        address: ip-11-0-1-002.eu-west-2.compute.internal
        port: 61616
      - name: amq3
        scheme: tcp
        address: ip-11-0-1-003.eu-west-2.compute.internal
        port: 61616
      amq_broker_federations:
        - name: amq1-federation
          upstreams:
            - name: amq2
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq2
            - name: amq3
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq3
          queue_policies:
            - name: TEST
              includes:
                - queue_match: TEST
                  address_match: TEST
    ip-11-0-1-002.eu-west-2.compute.internal:
      amq_broker_instance_name: amq2
      amq_broker_connectors:
      - name: artemis
        scheme: tcp
        address: ip-11-0-1-002.eu-west-2.compute.internal
        port: 61616
      - name: amq1
        scheme: tcp
        address: ip-11-0-1-001.eu-west-2.compute.internal
        port: 61616
      - name: amq3
        scheme: tcp
        address: ip-11-0-1-003.eu-west-2.compute.internal
        port: 61616
      amq_broker_federations:
        - name: amq2-federation
          upstreams:
            - name: amq1
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq1
            - name: amq3
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq3
          queue_policies:
            - name: TEST
              includes:
                - queue_match: TEST
                  address_match: TEST
    ip-11-0-1-003.eu-west-2.compute.internal:
      amq_broker_instance_name: amq3
      amq_broker_connectors:
      - name: artemis
        scheme: tcp
        address: ip-11-0-1-003.eu-west-2.compute.internal
        port: 61616
      - name: amq1
        scheme: tcp
        address: ip-11-0-1-001.eu-west-2.compute.internal
        port: 61616
      - name: amq2
        scheme: tcp
        address: ip-11-0-1-002.eu-west-2.compute.internal
        port: 61616
      amq_broker_federations:
        - name: amq3-federation
          upstreams:
            - name: amq1
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq1
            - name: amq2
              user: amq-broker
              password: amq-broker
              static_connectors:
                - amq2
          queue_policies:
            - name: TEST
              includes:
                - queue_match: TEST
                  address_match: TEST

Sample playbook:
---
- name: Deploy AMQ in 3 broker federation setup
  hosts: federated
  vars:
    ansible_ssh_private_key_file: my_keypair.pem
    amq_broker_ha_enabled: true
    amq_broker_cluster_discovery: "provided"
    amq_broker_nio_enabled: true
    amq_broker_cors_strict_checking: false
    amq_broker_prometheus_enabled: false
    amq_broker_addresses:
      - name: TEST
        anycast:
          - name: TEST
      - name: DLQ
        anycast:
          - name: DLQ
      - name: ExpiryQueue
        anycast:
          - name: ExpiryQueue
    amq_broker_acceptors:
      - name: artemis
        bind_address: "<IP_ADDRESS>"
        bind_port: "{{ amq_broker_port }}"
        parameters:
          tcpSendBufferSize: 1048576
          tcpReceiveBufferSize: 1048576
          amqpMinLargeMessageSize: 102400
          protocols: CORE
          useEpoll: true
          amqpCredits: 1000
          amqpLowCredits: 300
          amqpDuplicateDetection: true
          supportAdvisory: false
          suppressInternalManagementObjects: false
      - name: amqp
        bind_address: "<IP_ADDRESS>"
        bind_port: "{{ amq_broker_port_amqp }}"
        parameters:
          tcpSendBufferSize: 1048576
          tcpReceiveBufferSize: 1048576
          protocols: AMQP
          useEpoll: true
          amqpMinLargeMessageSize: 102400
          amqpCredits: 1000
          amqpLowCredits: 300
          amqpDuplicateDetection: true
  roles:
    - redhat.amq_broker.amq_broker

The correct setting for federated (no ha), would be
activemq_ha_enabled: false
activemq_cluster_discovery: provided
activemq_cluster_connections: []

The problem is in configure_artemis at instance creation because the static list of connector-ref is empty (it is provided in federation not in cluster_connnections)

with those suggested options, the broker starts but still no connectors. artemis.log shows:
2024-09-24 10:22:37,267 INFO  [org.apache.activemq.artemis.core.server] AMQ221012: Using AIO Journal
2024-09-24 10:22:37,342 INFO  [org.apache.activemq.artemis.core.server] AMQ221057: Global Max Size is being adjusted to 1/2 of the JVM max size (-Xmx). being defined as<PHONE_NUMBER>
2024-09-24 10:22:37,362 WARN  [org.apache.activemq.artemis.core.server] AMQ222226: Connection configuration is null for connectorName "amq2"
2024-09-24 10:22:37,362 WARN  [org.apache.activemq.artemis.core.server] AMQ222175: Bridge amq2 could not find configured connectors
2024-09-24 10:22:37,362 WARN  [org.apache.activemq.artemis.core.server] AMQ222226: Connection configuration is null for connectorName "amq3"
2024-09-24 10:22:37,363 WARN  [org.apache.activemq.artemis.core.server] AMQ222175: Bridge amq3 could not find configured connectors

Can you compare the configuration with that of federation molecule test? (you'll need to grab the PR code)
I also added a task to ensure connectors top element exists, fixed a double quote around connector-refs which was suspicious.
I do not get the warnings and broker.xml seems correct ( i do not get any logs though):

[...]
    <connectors>
      <connector name="instance1">tcp://instance1:61616?</connector>
      <connector name="instance2">tcp://instance2:61616?</connector>
    </connectors>
    <federations>
      <federation name="amq-federation">
                <upstream name="instance1" user="amq-broker" password="amq-broker">
                                    <static-connectors>
                                <connector-ref>instance1</connector-ref>
                            </static-connectors>
                                            </upstream>
                                                <queue-policy name="TEST">
                        <include queue-match="TEST" address-match="TEST"/>
                                </queue-policy>
                              </federation>
    </federations>
  </core>

Also mind the I think the federation name must be unique (and same on all instance). Thanks


Tab support

Add support for tabbing directories.  Probably because I need it myself

Added in the latest version v0.0.18.


Run vsftpd as standalone

Thanks for this image !
My goal is to have this container always open, and to have only "virtual_users.txt" updated by our application.
So, it would be great to launch vsftpd as a service.
When trying to start it with systemctl I have this error :
[root@d865273a4eae init.d]# systemctl start vsftpd.service Failed to get D-Bus connection: Operation not permitted
I want to have it as standalone to be able to restart it when the file "virtual_users.txt" is updated with a cron.
Do you have an idea ?
Thank you by advance.

I'm not sure I understand this request. Is there any reason for not restarting the container itself?

Avoiding killing users which are transferring files ?

I think you are mistaking restarting with reloading (your request is about the former). In any case, I think service management should be done outside the container. If you are interested in using systemd, I'd recommend having a look at https://hub.docker.com/_/centos section Running a systemd enabled app container.


Trouble with changes saving and persisting.

Describe the bug
I'm having trouble getting the db view to save info that I enter or add. If I add a new column/field and change the name then click away from the file and come back it doesn't save. Or if I enter a new value into a field it doesn't write. Basically any change I make wont save until after some long amount of time (or perhaps some other action that I haven't been able to identify). Even if I leave the view open for 5-10 mins it wont save.
Expected behavior
Changes save and persist within 5-10 seconds at max.
Desktop (please complete the following information):

OS: Mac OS Monterey 12.3.1

try now with 1.1.2 to confirm if its fixed

Yes, it is working now! 🙏


name plugin repo if dockerfile is at root

Be sure and add yourself to AUTHORS along with your changes.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@cb37a80). Click here to learn what that means.
The diff coverage is 50%.

@@            Coverage Diff            @@
##             master     #905   +/-   ##
=========================================
  Coverage          ?   78.08%
=========================================
  Files             ?       52
  Lines             ?     5545
  Branches          ?        0
=========================================
  Hits              ?     4330
  Misses            ?     1215
  Partials          ?        0

Impacted Files
Coverage Δ

vent/api/plugins.py
80.51% <50%> (ø)

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 cb37a80...8aaec69. Read the comment docs.


Picking commits from #328

@hannes-ucsc let me know if you want me to open an issue for this. I spoke with @jvivian and he is blocked on some of the fixes in #328, while we debug the spark service update. I figured it'd be wise to open a PR with those commits picked over, so we can merge the Toil 3.2.1 fixes and get @jvivian on his merry way.

Thanks @fnothaft !

LG2M (assuming tests pass)

No issue needed.

Thank you; you are a good person. May your day be joyous.

@hannes-ucsc — Ready for review


removes example game reference

The code for the example game is still in the project - no need to
delete that.

Closing issue. (Orestis)


share diagnostic data doesnt wrap to small screen

The Issue
some word doesnt wrap on small screen , for example : if user have 1024*768 screen resulution .
Expected Behaviour
would be nice its supported many monitor resulution
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Suggested fix is in the PR referenced above.


Method cache makes the tests fail

The ASTKernel key is likely incomplete.
Repro with:
ENABLE_METHOD_CACHE=1 GPU=1 python3 test/test_ops.py

Simpler repro
ENABLE_METHOD_CACHE=1 GPU=1 python3 test/test_ops.py TestOps.test_cmp_ge TestOps.test_cmp_le

Fixed by https://github.com/geohot/tinygrad/commit/5dc227dba613be7b366c164397b0b45fedce5abf


⚠ Default Queue Latency has degraded performance

In 0d5333b, Default Queue Latency ($WORKER_HEALTH_URL/default/latency) experienced degraded performance:

HTTP code: 200
Response time: 167 ms

Resolved: Default Queue Latency performance has improved in d7b75bf after 8 minutes.


error on python setup.py install

mbp-dev:gazu-publisher des$ python setup.py install
error in gazupublisher setup command: 'install_requires' must be a string or list of strings containing valid project/version requirement specifiers; Expected version spec in qtazu@git+https://github.com/cgwire/qtazu.git#egg=qtazu at @git+https://github.com/cgwire/qtazu.git#egg=qtazu

Thx for your feedback !
This bug has been fixed, so it shouldn't be a problem anymore. The gazu publisher still remains WIP so let us know if the error persists. If you want to, we've also written a procedure of installation here to make things easier : https://gazu.cg-wire.com/publisher.html

ok! lets move on this one!
os x mojave, 10.14.6
pyside2 installed, pip from python27
mbp-dev:gazu-publisher des$ pip install https://github.com/cgwire/gazu-publisher.git --user
DEPRECATION: Python 2.7 reached the end of its life on January 1st, 2020. Please upgrade your Python as Python 2.7 is no longer maintained. pip 21.0 will drop support for Python 2.7 in January 2021. More details about Python 2 support in pip, can be found at https://pip.pypa.io/en/latest/development/release-process/#python-2-support
Collecting https://github.com/cgwire/gazu-publisher.git
Using cached https://github.com/cgwire/gazu-publisher.git
ERROR: Cannot unpack file /private/var/folders/my/nn47_t4d365fr1p3rj6p0r7r0000gn/T/pip-unpack-rrb_dW/gazu-publisher.git (downloaded from /private/var/folders/my/nn47_t4d365fr1p3rj6p0r7r0000gn/T/pip-req-build-ARbn8n, content-type: text/html; charset=utf-8); cannot detect archive format
ERROR: Cannot determine archive format of /private/var/folders/my/nn47_t4d365fr1p3rj6p0r7r0000gn/T/pip-req-build-ARbn8n

Can you try the command pip install git+https://github.com/cgwire/gazu-publisher.git --user, instead of pip install https://github.com/cgwire/gazu-publisher.git --user (add git+ in front of the url) ?
The mistake is in our tutorial, it was corrected but the PR with the fix has not been merged yet, sry for that. Let me know if it worked !

pip install -U setuptools did the trick for me, after the git+https worked just fine in --user mode.
mac mojave 10.14.6, python27, apple python build, no brew


Added docs for SmartMap_Address fieldtype

Hey Erik,
I've added the cheat sheet references for my Smart Map "Address" field type. If you want to test out the plugin for whatever reason, feel free to hit me up at<EMAIL_ADDRESS>Thanks man, you've built a great little tool here! :)

@lindseydiloreto Looks great! I tested things locally and it works like a charm.
The only change I'll make is the "extra" info you provided. I'm going to wrap that p tag in a div and move the class name. We have styles setup to tweak the font just a tad based on that cascading.

Thanks man, you've built a great little tool here! :)

Great to hear. Thank you!

@lindseydiloreto Alright, it's officially in the build!

Awesome, thanks Erik!


Failing to install via yarn

Subject of the issue
Cannot install the package via yarn.
It throws an error "Error: [ERROR] You need npm version @>=5 but you have 3.5.2"
3.5.2 is the most recent npm version
Environment
Node version: 8.10.0
NPM version: 3.5.2

npm v3 is so old, upgrade plz

I'm closing this, since it has been answered.


Inconsistency in supported integer types on GPU

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
not relevant

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Linux 4.9.0-8-amd64 #1 SMP Debian 4.9.110-3+deb9u4 (2018-08-21) x86_64 GNU/Linux
VERSION_ID="9"
VERSION="9 (stretch)"

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:

TensorFlow installed from (source or binary):
1.10.1 (from pip binary)

TensorFlow version (use command below):
tf.VERSION = 1.10.1
tf.GIT_VERSION = v1.10.1-0-g4dcfddc5d1
tf.COMPILER_VERSION = v1.10.1-0-g4dcfddc5d1

Python version:
Python 3.5.3

Bazel version (if compiling from source):

GCC/Compiler version (if compiling from source):

CUDA/cuDNN version:
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2017 NVIDIA Corporation
Built on Fri_Nov__3_21:07:56_CDT_2017
Cuda compilation tools, release 9.1, V9.1.85

GPU model and memory:
Tesla P100-PCIE-16GB

Exact command to reproduce:

Describe the problem
It appears that the kernel of tf.reduce_sum is not registerd for GPU's if the type of the tensor to be summed is int64 (only registered for tf.int32)
Moreover the kernel of tf.tile is not registered for the case where the tensor to be tiled is of type tf.int32 (only registered for tf.int64)
Why is there this inconsistency?

tf.tile has int32 implementation (not int64)
tf.reduce has no int* implementations. There are kernels registered on GPU with int32 but they run on CPU since in most common case the tensors are small and launching GPU kernel would be inefficient.
If you indeed need an int32 op that runs on GPU you are out of luck.
Generally, there is an inconsistency since for some ops registering an int32 gpu kernel would lead to many small  ops  being launched  on GPU.

Nagging Assignee @azaks2: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.


Placing Tag Template in the Localizable folder

Getting this issues
/Users/ianwarner/.rbenv/versions/2.0.0-p247/lib/ruby/gems/2.0.0/gems/middleman-core-3.1.3/lib/middleman-core/sitemap/extensions/proxies.rb:56:in `proxied_to_resource': You can't proxy tag/car-insurance to tag/template.html which is itself a proxy.

I think it is because of the subject reason.
Any help / guidance appreciated.

I got around my issues by setting the options for the dynamic pages in the config.rb
Please read
https://forum.middlemanapp.com/t/middleman-tagpages-template-not-getting-correct-locale/2605/2
Closing - there is a wider i18n ticket getting done that may fix some of this


Change default serverDir to buildDir

This continues the discussion in https://github.com/xing/hops/pull/726#discussion_r239585775, I think. As already mentioned there, I would not be too keen on creating a sparse dist folder even in development mode.
Also, I would not want to change the internal config structure so soon before 1.0 - not without extremely convincing reasoning.

@ZauberNerd I just had another look at your proposal. By using a setting defined by @untool/webpack in the @untool/express preset, we are introducing a new dependency. We will at least have to add said dependency to the express preset’s package.json.
This would, of course, establish a circular dependency between the two. Generally, I would love to keep the possibility of using the Express stuff without Webpack at least with static builds.