Refresh pagination when using AJAX column filters

Hello, I'm not very proficient in angular and I can't figure out what exactly is happening in your code.
Essentially, I am getting large data sets asynchronously and I want to be able to refresh the pagination when they're done loading. I also want to reload pagination for example when searching through the filtered content.
Can anyone give me a hint on how to achieve this?
Here is my failed attempt, forked from the AJAX filter example - https://codepen.io/anon/pen/JNVWqr

OK, I just figured it out - the mdt-trigger-request actually works fine, the problem was mentioned here - https://github.com/iamisti/mdDataTable/issues/196#issuecomment-261004813
For whatever reason in function paginatorCallback(page, pageSize, options) { ... }  - page appears to be undefined. Simply doing if(!page) { page = 1 } makes it work fine
Hope this can help someone until a fix is out


Final DataSource design change.

This should be the last time we have to talk about this.
Yesterday we figured out that we can use multiple Application classes to do dependency injection.  See snippets below.  DataSource no longer needs to be a proper singleton with DataSource.getInstance() - it should actually be an Interface.  Our first implementation of DataSource would be a DummyDataSource that just does everything in memory - no files, no elasticsearch, maybe not even adding data (just hard coded defaults).  Luckily it'll only be a small change to the datasource uml at this point since we have no code.
class TravelTrackerApp extends Application {
  private DataSource ds;  // never changes!
  public DataSource getDataSource() {  return ds; }
}

class DummyApp extends TraverTrackerApp {
  public void onCreate() {
    ds = new DummyDataSource();
  }
}

class ProductionApp extends TraverTrackerApp {
  public void onCreate() {
    ds = new ElasticSearchDataSource();
  }
}

Objections? Critiques?

That looks good to me. Hopefully this will let us test easily.

I haven't fully researched this, but we might need our different Application classes to inherit from MockApplication for testing purposes.  If this is the case, then we probably have only one actual TravelTracker Application, and as the project progresses we would just change the ds = new WhateverDataSource() line in that actual Application.
That's still compatible with the above - the key is that DataSource is an interface that we implement in different ways.

@kdbanman Thats the way I envisioned it. Dependency injection is only possible in the testing environment where we can inject mock applications. In the main source it cant be injected. I don't think that will cause an issue for us.

:+1:


Elixir-LS is installed every time but does nothing

Precheck

Do a quick search and make sure the issue has not yet been reported
Do not use the issues tracker for guidance, questions or support (try Elixir Forum, Stack Overflow, Slack, etc. instead)
If using a client other than VS Code, please try VSCode's "ElixirLS: Elixir support and debugger" extension. Does it reproduce your failure?

Environment

Elixir & Erlang versions (elixir --version):
Erlang/OTP 26 [erts-14.0.2] [source] [64-bit] [smp:12:12] [ds:12:12:10] [async-threads:1] [jit:ns]

Elixir 1.15.4 (compiled with Erlang/OTP 26)

Elixir Language Server version: 0.18.1
Operating system:  Windows 11 Pro
Editor or IDE name (e.g. Emacs/VSCode): Nvim and VSCode
Editor Plugin/LSP Client name and version: in VSCode the plugin, v0.18.1, in NVim elixir-ls v0.18.1

Current behavior
I'm getting no autocompletion, suggestions, nothing out of the client, everytime I open a .ex file I'm getting in the logs that it's installing the lsp server.
Here in VSCode:
Installing ElixirLS release v0.18.1
Running in ~dir/to/the/project~
Install complete

In Nvim it is being printed to stderr, and the client is not being attached to the buffer, the message is the same as in VSCode:
"stderr"	"Installing ElixirLS release v0.18.1\n"
"stderr"	"Running in ~dir/to/the/project~\n"
"stderr"	"Install complete\n"

Installing ElixirLS release v0.18.1
Running in ~dir/to/the/project~
Install complete

Beside of the previous logs, it just prints this before: [Trace - 4:45:11 PM] Sending request 'initialize - (0)'.

Installed elixir with choco -> choco install elixir
commands are working well
This is the repo I'm trying
https://github.com/FdHerrera/learning-elixir
Expected behavior

It do nothing

Please install a supported OTP version. OTP 26.0.2 is not supported on Windows. See https://github.com/elixir-lsp/elixir-ls?tab=readme-ov-file#support-matrix


[BUG] Internal 68.2.0 change breaks all ROS2 python packages

setuptools version
68.2.0
Python version
3.10
OS
Ubuntu
Additional environment information
No response
Description
This recent change in setuptools for 68.2.0 described as "Record normalized requirements in dist.metadata object"
https://github.com/pypa/setuptools/commit/a641c78c7c4e91d85a8772434d046db56abd1576
Probably breaks all downstream python packages built for ros2 using colcon.
https://github.com/colcon/colcon-python-setup-py/issues/56#issue-1888257679
This is going to cause a bunch of cascading failures in other projects.  For example:
https://github.com/ArduPilot/ardupilot/issues/24862#issuecomment-1714048295
Stack Overflow is about to get really busy because the tutorials are also broken.
https://robotics.stackexchange.com/questions/104054/building-example-library-of-ros-2
This design flaw in colcon has been "known" since 2021:
https://github.com/colcon/colcon-core/issues/454
But the ROS ecosystem is very large and this will probably take a while to fix.
Unfortunately the root cause is that colcon is feeding output from setup.py into a compiler and expected compilable code but now gets a string representation which is not compilable. The extra <> characters now cause a syntax error.
Expected behavior
Builds using 68.1 do work. Since this is an internal meta-data structure change which seems to be an optimization? I would expect the easiest fix is to revert.
How to Reproduce
Try to build any ROS2 python package built with 68.2
Can add more details in a bit.
Output
 '_normalized_install_requires': {'setuptools': <Requirement('setuptools')>}
                                                ^
SyntaxError: invalid syntax

Have you tried 68.2.1?

This design flaw in colcon has been "known" since 2021:
colcon/colcon-core#454

About using the deprecated setup.py CLI: I understand the stance of the project... as long as they understand the consequences of using a deprecated feature, that is OK.
However this seems to be unrelated to using setup.py per-se:

Unfortunately the root cause is that colcon is feeding output from setup.py into a compiler and expected compilable code but now gets a string representation which is not compilable. The extra <> characters now cause a syntax error.

Without deep inspection, it seems that serialising arbitrary objects with repr and expecting them to parse as Python would be a very bold decision... I cannot it is not error prone. I am not sure if colcon is doing exactly that, but at least this is how it sounds by looking at this problem.

Thank you very much @asasine for confirming.
@owend, please let me know if that is enough for this issue. If that is OK, we can go ahead and close.

Fixed with 68.2.1, thanks!


Building on Visual Studio doesn't reuse existing build server

This is because we use AppDomain.CurrentDomain.BaseDirectory to compute the default pipe name. We should be using path to rzc instead.

@mkArtakMSFT we need to fix this for preview2. Assigning this to myself.

7bc54b99e084ec15880ca35aab23a574aec0775a


Openning IPython Notebook with Light Table

I'm really excited to see that I can open the IPython Notebook (version 2.3) in Light Table by using a Browser in Connections. However, I've noticed that when clicking on a link to the actual notebook, it opens up the notebook in a different window instead of as a tab. What is more peculiar is that if I type in the address for the notebook, which has displayed fine in the seperate window, the page no longer loads. Here is the error from the console.
Uncaught Error: Mismatched anonymous define() module: [object Object] http://requirejs.org/docs/errors.html#mismatch
require.js?v=07d7db4d3fd6519996822665e4239282 [166]	makeError
require.js?v=07d7db4d3fd6519996822665e4239282 [1221]	intakeDefines
require.js?v=07d7db4d3fd6519996822665e4239282 [1408]	anonymous

Is this a Light Table issue, or an IPython issue?
I would be great to be able to work in the editor and with IPython notebooks in the same program!

Closing as iPython no longer works for more recent versions and support is being tracked in https://github.com/LightTable/Python/issues/35


What is the Open Source License for this code?

I'd like to use this code, but it lacks a license.  Can you tell me what license it is distributed under?

I appreciate this, but without a license, it's a copyright violation use your code: you fully own the copyright.  What a license does is formally declare that we have the right to you use your code and under what conditions.
If you're in the category of "do whatever you want with this", I suggest Apache 2.0, which is a license just like this.  Just slap the Apache 2.0 license text in the top directory of the project, called LICENSE, and call it a day.  Don't even bother modifying the file headers, it's not that important.

Added Apache licence.

I'm working on a small project for adding PD to Mozzi on the Arduino.  I'm looking for simple PD versions for all the casio waves.  BTW, I think you've been replicating waves from the front panel of the CZ series, but you might be interested that the front panel lies: the way the waves work is entirely different.  In fact, there are eight waveforms, only one of which is "resonance" so to speak (it's often called "multi-sine"), and waves consist of pairs of waveforms plus a window function: there are eight window functions.  So there are technically 8^3 combinations, not 6, though many are duplicates and others are useless.  It'd be really nice to have distortion generators for all eight (well, 7) of the basic waveforms, the rest is easy to put together.  See the Casio CZ-1 Splunking page, which is good at explaining it, though it doesn't give any clues as to the specific functions for doing distortion.  This guy got good generators for Saw and Square, and built a resonance function but that's not how resonance works in the CZ: it's just increasingly high-frequency Sine run through a triangle window function.  I've not found any other useful code.

I am confused a bit by your PD code.  I think you have a fixed PD generator for (say) sawtooth in the form of a lookup table, and what you're doing is cross-fading between it and sine, rather than using parameterizable function generator?  I could be mistaken?  It's not super clear.  This might work in a pinch and I might do that but was hoping to actually find a fast (non-float, non-division, sine only via lookup table) way to produce the distortion functions.

There are a couple of mechanisms going on here. The 'resonance' waves are implemented pretty much how the hardware works.
For the other waves my approach was to 'move' through a sine wave with the speed dictated by the phase distortion look-up table. I then sampled the phase distorted waves from my Casio CZ, built a little program to calculate the look-up table, and stored that.
So I'm not cross-fading between the phase distortion look-up and the sine wave; I'm dynamically adjusting the speed I play back the sine wave based on the scaled phase distortion wave. It's phase distortion, but implemented very differently to the Casio.
I wasn't really aiming for accuracy - more to make a fun homage to the Casio that would work in a Eurorack situation. The sounds tend to be more aggressive than the original and work well in a monophonic situation whereas the original Casio excels as a polyphonic synth (to me, anyway).

I wonder if that would work in my situation. Lookup tables always good -- I'm on an arduino with no hardware division nor floating point -- but the whole architecture might be too computationally costly... [I'm doing AE Modular GRAINS].

That makes it a lot harder - I am way too lazy to work with fixed point maths. I will generally try and avoid divides anyway, unless they can be implemented as integer bit shifts.
I haven't delved massively deeply into the Casio architecture (the patent document is available but is incredibly complicated). However I am fairly sure they would have used a much simpler method than me for performance on older processors.
I also think that one thing that has a fairly significant impact on the sound is the analog LP filter on the output. I don't know what they are using exactly - it is marked 'LPF17K' on the schematic, so I assume it is filtering at 17 KHz.

I'm guessing this was an attempt to eliminate aliasing? Or high frequency DAC artifacts?    Probably not filtering prior to downsampling?

On Dec 8, 2023, at 3:32 PM, dchwebb @.***> wrote:
That makes it a lot harder - I am way too lazy to work with fixed point maths. I will generally try and avoid divides anyway, unless they can be implemented as integer bit shifts.
I haven't delved massively deeply into the Casio architecture (the patent document is available but is incredibly complicated). However I am fairly sure they would have used a much simpler method than me for performance on older processors.
I also think that one thing that has a fairly significant impact on the sound is the analog LP filter on the output. I don't know what they are using exactly - it is marked 'LPF17K' on the schematic, so I assume it is filtering at 17 KHz.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

I'm not exactly sure. The DAC goes to an 'Expander Circuit' and then a 'Sample/Hold Circuit', before being filtered and output. I haven't really looked into what these do but possibly create artifacts that need to be filtered.


Small improvements to index buffer/texture loader

Add initialData and setData to indexBuffer. Works the same way as with vertexBuffer (VB already has it).
Allow texture loader to load from blob URLs (window.URL.createObjectURL)

Looks fine to me. @daredevildave?


[RFC]Fix progress bar

Fixes #3969
Recalculate epoch_length not to exceed 100%
before:

[gen@localhost chainer]$ python3 examples/ptb/train_ptb.py --test
#vocab = 10000
     total [#####################################################] 107.69%
this epoch [..................................................]  0.00%
         6 iter, 42 epoch / 39 epochs
   0.10037 iters/sec. Estimated time to finish: -1 day, 23:59:55.730007.

after:

[gen@localhost chainer]$ python3 examples/ptb/train_ptb.py --test
#vocab = 10000
     total [##################################################] 100.00%
this epoch [..................................................]  0.00%
         6 iter, 42 epoch / 39 epochs
  0.099782 iters/sec. Estimated time to finish: 0:00:00.

Sorry, late reply.
How about using min? I think it is more simple to understand.

@ankokumoyashi Do you have time to fix this PR?

Sorry for my late reply.
Thank you for pointing it out.
How about this? Is it alright?

Thanks You are correct. I will check this.
jenkins, test this please.

Codecov Report

Merging #4152 into master will decrease coverage by 0.52%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #4152      +/-   ##
=========================================
- Coverage   89.12%   88.6%   -0.53%
=========================================
  Files         320     320
  Lines       21211   21007     -204
=========================================
- Hits        18905   18613     -292
- Misses       2306    2394      +88

Impacted Files
Coverage Δ

chainer/training/extensions/progress_bar.py
15.71% <0%> (-0.47%)
:arrow_down:

chainer/serializers/hdf5.py
24.65% <0%> (-73.98%)
:arrow_down:

chainer/functions/theano/theano_function.py
28.57% <0%> (-71.43%)
:arrow_down:

chainer/links/theano/theano_function.py
25% <0%> (-59.38%)
:arrow_down:

chainer/testing/serializer.py
93.33% <0%> (-6.67%)
:arrow_down:

...ner/functions/normalization/layer_normalization.py
94.73% <0%> (-5.27%)
:arrow_down:

chainer/functions/array/stack.py
95.65% <0%> (-4.35%)
:arrow_down:

chainer/functions/array/broadcast.py
95.77% <0%> (-4.23%)
:arrow_down:

chainer/functions/math/erf.py
93.93% <0%> (-3.04%)
:arrow_down:

chainer/functions/math/erfc.py
93.93% <0%> (-3.04%)
:arrow_down:

... and 21 more

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 1759406...860067a. Read the comment docs.

LGTM!


Function-id/table based ecall/ocall dispatching.

Edger8r generates ocall and ecall function tables in the host and enclave sides respectively.
Each ocall and ecall function is associated with an enumeration value.. Currently the functions are assigned id's in the order of their declaration.
Functions are called by their ids instead of their names.
In the enclave side, oecore expects two symbols _oe_ecalls_table and _oe_ocalls_table_size to be generated by the edger8r.
In the host side, there can be many tables since the host can load many different enclaves and each edl file results in an unique ocalls table on the host side.
The registration of the ocalls table happens at enclave creation time; the ocalls table is now a parameter to oe_create_enclave.
To hide all the internals, edger8r generates oe_create__enclave as function that the user must call to create an enclave.
The infrastructure that the generated code uses on the host and enclave sides have been moved to
edger8r/host.h and edger8r/enclave.h.
Note:
No existing behavior is deprecated.
Marshaling has not been changed.

bors try

Addresses #894

@paulcallen I have addressed your feedback. Can you take another pass?

bors try

bors r+

bors r+

bors try

bors try

bors r+

bors r+


Re-rendering issue with ember-engines 0.8.2 & link-to

After updating to<EMAIL_ADDRESS>&<EMAIL_ADDRESS>code like this started to fail in our app:
{{#link-to routeName}}...{{/link-to}}

Where routeName could change. The cause of this, as far as I can see, is the _prefixProperty() in the link-to, as it seems to be triggering a "invalid rerendering" error. I tried to recreate it as a test in this repo, but couldn't - so not entirely sure what exactly triggers this. But it happened in multiple places in our app.
I fixed it by changing it to e.g.
<LinkTo @route={{readonly routeName}}>...</LinkTo>
Which is OK, but feels like it shouldn't be necessary.
I guess overall it would be nicer if link-to wouldn't need to be monkey patched this way, but not sure how exactly that could look like :/

Oops, somehow I didn't see #667, I guess that's a duplicate - sorry.


NoMatchingExpectationException removes spaces, causes frustration.

A lot.
I've spent some time figuring out why this code didn't work;
$this->mockedObject->shouldReceive('push')->with('foobar');

1) Test\MyTest
Mockery\Exception\NoMatchingExpectationException: No matching handler found for Mockery_7_Test::push("foobar"). Either the method was unexpected or its arguments matched no expected argument list for this method

HOWEVER it turned out that my code was not sending "foobar" but "foo bar" and the spaces were stripped from the output.
$this->queue->shouldReceive('push')->with('foo bar');

OK (1 test, 3 assertions)

I understand where this might come from (because huge dumps are hard to read) but this is very frustrating.
I would suggest not stripping spaces - or any whitespace at all, to prevent this kind of stuff.

Hmmm, I can see why that would be annoying!
What version are you using? I just tried a quick test and couldn't replicate it in master:
<?php

require "vendor/autoload.php";

$mock = Mockery::mock();
$mock->shouldReceive('push')->with('foobar');

$mock->push('foo bar');

# PHP Fatal error:  Uncaught exception
# 'Mockery\Exception\NoMatchingExpectationException' with message 'No matching
# handler found for Mockery_0::push("foo bar"). Either the method was unexpected
# or its arguments matched no expected argument list for this method

Got it - the issue is when passed variables are in arrays. See this example;
<?php

require "vendor/autoload.php";

$mock = Mockery::mock();
$mock->shouldReceive('push')->with(['foobar', 'var 2']);

$mock->push('foo bar', 'argument 2', ['var' => 'Array value 2']);

# PHP Fatal error:  Uncaught exception
# 'Mockery\Exception\NoMatchingExpectationException' with message
# 'No matching handler found for Mockery_0::push("foo bar", "argument 2", array('var'=>'Arrayvalue2',)).
# Either the method was unexpected or its arguments matched no expected argument list for this method

I think there's some whitespace / newline trimming regex cast on var_export's somewhere..

Another fun fact - booleans in arrays are displayed as string (true => 'true') in those exceptions.

@RobQuistNL please review #504 if you get a moment

Tested the case, and that PR solves a lot of issues! Markup is better, no more oldskool array() notation, bools are fixed and the spaces as well. :+1: for you, great work!


[ENH, WIP] Add dipole plotting to mne.viz.Brain

This adds the plotting of dipoles similar to mne.viz.plot_alignment to mne.viz.Brain. I just extended it to support plotting the forward model as well:

I touched tutorials/inverse/35_dipole_orientations.py so if this messed things up (especially scale) for plot_alignment we can fix that.

Eh thinking on it, you shouldn't be able to change the colors in the forward operator, I'll separate it out.

Okay this should be good now. I like this because it allows you to scale and change the alpha on the forward dipoles so that you can see them better and they look good using Brain.
There is also access to coloring the dipoles, so you could pass colors from a colormap with the amplitudes or gofs as arguments. Pretty nice I think.

The dipole artifact looks different than before but I actually think I fixed a bug where most of them weren't showing
https://42048-1301584-gh.circle-artifacts.com/0/dev/auto_tutorials/inverse/35_dipole_orientations.html?highlight=dipole

Okay, all done with changes, should be all greens and good to go.

Tests passed if anyone wants to review

The dipole artifact looks different than before but I actually think I fixed a bug where most of them weren't showing

I think main is even worse than you give it credit for -- these should all have some component that points directly outward, because the forward is surface oriented:

In particular, it should be the z component that points outward. I think that Z is blue (as I think RGB is mapped to XYZ, i.e., RAS in non-surf-oriented space, which matches the non-surf-oriented image you pasted above), and to my eye these blue arrows correctly point out of cortex (unlike main where some are just missing?):

Thanks for the fixes @alexrockhill !

So @larsoner, in response to your comment about the forward plotting on main, I was struggling with getting the quiver3d to show up. I had to set scale_mode='scalar' and scalars=[1] * len(dipoles). Perhaps this should be the default as a fix in quiver3d or maybe there is a fix or a more root problem. I can open an issue.


C++ APIView NOTICE.txt generation and artifact publishing

Supersedes https://github.com/Azure/azure-sdk-tools/pull/5249/files
NOTICE.txt generation fails when opening a PR from a fork

Keep this open

Closing this until there's a need for it.


Release Google.Cloud.Notebooks.V1Beta1 version 1.0.0-beta03

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.Notebooks.V1Beta1-1.0.0-beta03


Suggesting a different name for struct "DEBUG" in xcontainer.hpp

Latest xtensor 0.20.4 fails to build for me, due to a clash with my debug build defines. At least on Windows it is relatively common to have a define "DEBUG" for debug builds, analog to "NDEBUG" for release builds. This clashes with the name of aforementioned struct. Would it be possible to rename:
    template <class T>
    struct DEBUG;

from xtensor-0.20.4/include/xtensor/xcontainer.hpp? Thank you for your consideration!

This is a forgotten artifact for debugging templates, it should not have been committed.
I will remove it ASAP

Thanks @JohanMabille ! Closing this issue as fixed.


FailureMessages should use String.format to replace the %n with corre…

The %n is not properly replaced with the system specific line separator if the string is not formatted with String.format method.

Hi timmikk,
thanks for the pull request. :)
I did not check every possible path, but at least in some execution parts the String is formatted in ExpectedException#expectMessage.
Cheers,
Pascal

Of course formatting it directly does not hurt and makes it clearer to the reader, so it's probably a good idea to apply your pull request.

I agree, it does not hurt.
Thanks

Hi,
Thanks for quick response.
I just came by this when solving failing AssertJ-Swing JUnit test:
org.assertj.swing.internal.assertions.images.Images_assertHasSize_Test#should_Fail_If_Actual_Is_Null.
That test uses directly the FailureMessages#actualIsNull without proper string formatting.

Thanks @timmikk.


How to enable OpenMP on macOS (clang)

How to enable OpenMP on macOS (clang)?

Node Version: node v20.12.2, npm 10.5.0
Platform:  Darwin MacBook-Pro.local 23.5.0 Darwin Kernel Version 23.5.0: Wed May  1 20:12:58 PDT 2024; root:xnu-10063.121.3~5/RELEASE_ARM64_T6000 arm64
Compiler:

cc -v
Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)
Target: arm64-apple-darwin23.5.0
Thread model: posix
InstalledDir: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin

Module: Any with OpenMP

Working example:
export CPPFLAGS="$CPPFLAGS -Xpreprocessor -fopenmp"
node-gyp rebuild

How to specify CPPFLAGS in binding.gyp config?
More info: https://open-box.readthedocs.io/en/latest/installation/openmp_macos.html

Could anyone help to solve the issue?


How to get recording video size?

I want to check whether space is available or not in device. so can you guys please help me about How can i get Currently recording video size ?
so i can notify user that you don't have enough space in your device.
Thanks in Advance.

Try to use this

@kumarpramod017 , I don't wat to get video recording duration nut i want the live video recording size.
suppose record duration reach to 1 min than video size will be 20 mb, if video duration reach to 2 min than video size will be 50 mb.
actually i want to check recording video size.

size = bit_rate * current_duration


yq-go: 4.30.4 -> 4.30.5

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/mikefarah/yq/releases.
meta.description for yq-go is: "Portable command-line YAML processor"
meta.homepage for yq-go is: "https://mikefarah.gitbook.io/yq/"
meta.changelog for yq-go is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.30.5 with grep in /nix/store/h8sli4hsmxw67wcc3hza19f1h56pvnx7-yq-go-4.30.5
found 4.30.5 in filename of file in /nix/store/h8sli4hsmxw67wcc3hza19f1h56pvnx7-yq-go-4.30.5

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
k3s
yq-go

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h8sli4hsmxw67wcc3hza19f1h56pvnx7-yq-go-4.30.5 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A yq-go https://github.com/r-ryantm/nixpkgs/archive/e99fa4a086dcc9766e414c26be10c0ac1987f20e.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/h8sli4hsmxw67wcc3hza19f1h56pvnx7-yq-go-4.30.5
ls -la /nix/store/h8sli4hsmxw67wcc3hza19f1h56pvnx7-yq-go-4.30.5/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

k3s
yq-go

Maintainer pings
cc @nlewo @SuperSandro2000 for testing.

@ofborg eval


FYI: I'm going on paternity leave in a month

Just realized I should let y'all know that I will likely be at a severely reduced capacity from mid Feb to mid Apr. I'm expecting baby 2 in about a month! I'm not sure exactly what my availability will be but am hoping to be off for some time. Just wanted others here to know in case I am non responsive then. I trust y'all to continue to improve the project and community in my absence 🙂
Feel free to close this - just wasn't sure how else to get you all this information

🎉 👶🏻 🎉

Congratulation and take care of your family!
We'll try to keep the boat floating n your absence ;-)

@choldgraf are you back in action these days?  I'm under a series of deadlines this month, a release is overdue, and there are a few tough blockers I've been struggling to find time to work on:

#1416
#1398
#1317

the last one is the big one, the 2 first can be dismissed for a downstream patch. I'll have more time in September for review and bug fixes but for now I am completely under water.

look closer at #1416, it breaks ALL of our on-page-load JS.

I'm sort of back but definitely not reliable. I wouldnt depend on myself for anything code related right now unfortunately 😞


vdirsyncer throws errors when syncing with one calendar

With one calendar I get the following errors:
These are the errors at the end when it is set to local wins:
warning: Race condition detected with server p01-caldav.icloud.com, consider using an alternative.
Copying (uploading) item 8374708D-8CF1-4A04-89CE-BA00005328659FEF to private_calendar_local
Unhandled exception occured for private_calendar.
Traceback (most recent call last):
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 141, in _upload_impl
    return fpath, get_etag_from_fileobject(f)
  File "/usr/lib/python3.4/contextlib.py", line 66, in __exit__
    next(self.gen)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 120, in _open
    self.commit(f)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 140, in commit
    move_atomic(f.name, self._path)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 72, in move_atomic
    return _move_atomic(src, dst)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 26, in _move_atomic
    os.link(src, dst)
FileExistsError: [Errno 17] File exists: '/home/nik/.khal/calendars/home/tmpygrlp2rd' -> '/home/nik/.khal/calendars/home/8374708D-8CF1-4A04-89CE-BA00005328659FEF.ics'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.4/site-packages/vdirsyncer/cli/tasks.py", line 77, in sync_collection
    force_delete=force_delete
  File "/usr/lib/python3.4/site-packages/vdirsyncer/sync.py", line 175, in sync
    action(a_info, b_info, conflict_resolution)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/sync.py", line 197, in inner
    dest_href, dest_etag = dest.storage.upload(item)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/base.py", line 17, in inner
    return f(self, *args, **kwargs)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 118, in upload
    fpath, etag = self._upload_impl(item, href)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 144, in _upload_impl
    raise exceptions.AlreadyExistingError(existing_href=href)
vdirsyncer.exceptions.AlreadyExistingError

And this is what I get when I set that remote wins
Doing conflict resolution for item 9B46D59C-D719-467E-800E-1E9445244877...
...private_calendar_remote wins.
Copying (updating) item 9B46D59C-D719-467E-800E-1E9445244877 to private_calendar_local
Copying (uploading) item 8374708D-8CF1-4A04-89CE-BA0000005328659FEF to private_calendar_local
Unhandled exception occured for private_calendar.
Traceback (most recent call last):
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 141, in _upload_impl
    return fpath, get_etag_from_fileobject(f)
  File "/usr/lib/python3.4/contextlib.py", line 66, in __exit__
    next(self.gen)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 120, in _open
    self.commit(f)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 140, in commit
    move_atomic(f.name, self._path)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 72, in move_atomic
    return _move_atomic(src, dst)
  File "/usr/lib/python3.4/site-packages/atomicwrites/__init__.py", line 26, in _move_atomic
    os.link(src, dst)
FileExistsError: [Errno 17] File exists: '/home/nik/.khal/calendars/home/tmp34i_i9w_' -> '/home/nik/.khal/calendars/home/8374708D-8CF1-4A04-89CE-BA0000005328659FEF.ics'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.4/site-packages/vdirsyncer/cli/tasks.py", line 77, in sync_collection
    force_delete=force_delete
  File "/usr/lib/python3.4/site-packages/vdirsyncer/sync.py", line 175, in sync
    action(a_info, b_info, conflict_resolution)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/sync.py", line 197, in inner
    dest_href, dest_etag = dest.storage.upload(item)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/base.py", line 17, in inner
    return f(self, *args, **kwargs)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 118, in upload
    fpath, etag = self._upload_impl(item, href)
  File "/usr/lib/python3.4/site-packages/vdirsyncer/storage/filesystem.py", line 144, in _upload_impl
    raise exceptions.AlreadyExistingError(existing_href=href)
vdirsyncer.exceptions.AlreadyExistingError

When local wins no file-fixing (TZOFFSETFROM is wrong +5328 instead of +005328; a script I got from @untitaker fixes it for me) is needed, when remote wins file-fixing is needed (which is obvious). The problem exists with only one calendar, but there with lots of events.
This is how my vdirsyncer-config looks like:
[general]
status_path = ~/.vdirsyncer/status/
[pair private_calendar]
a = private_calendar_local
b = private_calendar_remote
conflict_resolution = b wins

[storage private_calendar_local]
type = filesystem
path = ~/.khal/calendars/home
fileext = .ics
[storage private_calendar_remote]
type = caldav
url = https://p01-caldav.icloud.com/this/is/a_path/
auth = guess
username =<EMAIL_ADDRESS>password = yeah_right

And this is the corresponding part in the khal.conf:
path = ~/.khal/khal.db
[calendars]
[[home]]
path = ~/.khal/calendars/home
color = light green
[locale]
local_timezone = Europe/Berlin
default_timezone = Europe/Berlin

timeformat = %H:%M
dateformat = %d.%m.
longdateformat = %d.%m.%Y
datetimeformat = %d.%m. %H:%M
longdatetimeformat = %d.%m.%Y %H:%M

firstweekday = 0

[default]
default_command = calendar
default_calendar = home

And this is the event that is shown in the errors above (there are far more but I hope one event is enough as a sample) after the script ran that fixes the wrong TZOFFSETFROM.
BEGIN:VCALENDAR
VERSION:2.0
PRODID:-//CALENDARSERVER.ORG//NONSGML Version 1//EN
BEGIN:VTIMEZONE
TZID:Europe/Berlin
X-LIC-LOCATION:Europe/Berlin
BEGIN:STANDARD
DTSTART:18930401T000000
RDATE;VALUE=DATE-TIME:18930401T000000
TZNAME:CEST
TZOFFSETFROM:+005328
TZOFFSETTO:+0100
END:STANDARD
BEGIN:DAYLIGHT
DTSTART:19160430T230000
RDATE;VALUE=DATE-TIME:19160430T230000
RDATE;VALUE=DATE-TIME:19400401T020000
RDATE;VALUE=DATE-TIME:19430329T020000
RDATE;VALUE=DATE-TIME:19460414T020000
RDATE;VALUE=DATE-TIME:19470406T030000
RDATE;VALUE=DATE-TIME:19480418T020000
RDATE;VALUE=DATE-TIME:19490410T020000
RDATE;VALUE=DATE-TIME:19800406T020000
TZNAME:CEST
TZOFFSETFROM:+0100
TZOFFSETTO:+0200
END:DAYLIGHT
BEGIN:STANDARD
DTSTART:19161001T010000
RDATE;VALUE=DATE-TIME:19161001T010000
RDATE;VALUE=DATE-TIME:19421102T030000
RDATE;VALUE=DATE-TIME:19431004T030000
RDATE;VALUE=DATE-TIME:19441002T030000
RDATE;VALUE=DATE-TIME:19451118T030000
RDATE;VALUE=DATE-TIME:19461007T030000
TZNAME:CET
TZOFFSETFROM:+0200
TZOFFSETTO:+0100
END:STANDARD
BEGIN:DAYLIGHT
DTSTART:19170416T020000
RRULE:FREQ=YEARLY;UNTIL=19180415T010000Z;BYDAY=3MO;BYMONTH=4
TZNAME:CEST
TZOFFSETFROM:+0100
TZOFFSETTO:+0200
END:DAYLIGHT
BEGIN:STANDARD
DTSTART:19170917T030000
RRULE:FREQ=YEARLY;UNTIL=19180916T010000Z;BYDAY=3MO;BYMONTH=9
TZNAME:CET
TZOFFSETFROM:+0200
TZOFFSETTO:+0100
END:STANDARD
BEGIN:DAYLIGHT
DTSTART:19440403T020000
RRULE:FREQ=YEARLY;UNTIL=19450402T010000Z;BYDAY=1MO;BYMONTH=4
TZNAME:CEST
TZOFFSETFROM:+0100
TZOFFSETTO:+0200
END:DAYLIGHT
BEGIN:DAYLIGHT
DTSTART:19450524T020000
RDATE;VALUE=DATE-TIME:19450524T020000
RDATE;VALUE=DATE-TIME:19470511T030000
TZNAME:CEMT
TZOFFSETFROM:+0200
TZOFFSETTO:+0300
END:DAYLIGHT
BEGIN:DAYLIGHT
DTSTART:19450924T030000
RDATE;VALUE=DATE-TIME:19450924T030000
RDATE;VALUE=DATE-TIME:19470629T030000
TZNAME:CEST
TZOFFSETFROM:+0300
TZOFFSETTO:+0200
END:DAYLIGHT
BEGIN:STANDARD
DTSTART:19460101T000000
RDATE;VALUE=DATE-TIME:19460101T000000
RDATE;VALUE=DATE-TIME:19800101T000000
TZNAME:CEST
TZOFFSETFROM:+0100
TZOFFSETTO:+0100
END:STANDARD
BEGIN:STANDARD
DTSTART:19471005T030000
RRULE:FREQ=YEARLY;UNTIL=19491002T010000Z;BYDAY=1SU;BYMONTH=10
TZNAME:CET
TZOFFSETFROM:+0200
TZOFFSETTO:+0100
END:STANDARD
BEGIN:STANDARD
DTSTART:19800928T030000
RRULE:FREQ=YEARLY;UNTIL=19950924T010000Z;BYDAY=-1SU;BYMONTH=9
TZNAME:CET
TZOFFSETFROM:+0200
TZOFFSETTO:+0100
END:STANDARD
BEGIN:DAYLIGHT
DTSTART:19810329T020000
RRULE:FREQ=YEARLY;BYDAY=-1SU;BYMONTH=3
TZNAME:CEST
TZOFFSETFROM:+0100
TZOFFSETTO:+0200
END:DAYLIGHT
BEGIN:STANDARD
DTSTART:19961027T030000
RRULE:FREQ=YEARLY;BYDAY=-1SU;BYMONTH=10
TZNAME:CET
TZOFFSETFROM:+0200
TZOFFSETTO:+0100
END:STANDARD
END:VTIMEZONE
BEGIN:VEVENT
UID:8374708D-8CF1-4A04-89CE-BA000000000000005328659FEF
DTEND;TZID=Europe/Berlin:20081009T153000
SUMMARY:Fahrschule
DTSTART;TZID=Europe/Berlin:20081009T143000
DTSTAMP:20110908T161944Z
SEQUENCE:0
TRANSP:OPAQUE
END:VEVENT
END:VCALENDAR

As recommended in https://github.com/geier/khal/issues/140#issuecomment-118603502 I did a
vdirsyncer repair private_calendar_remote/home
After that the problems persist. I also tried again with local/remote wins.
The repair worked on the items and didn't throw any error

The vdirsyncer repair-command told me that it processes 1320 items. After a sync with remote-sync which needs fixing I count 1229 ics-files that get re-downloaded apparently

Could you

delete all local calendars
delete the status folder
vdirsyncer sync

Also I've just fixed a bug in master where an execution of the repair command could've tripped up vdirsyncer massively.

vdirsyncer sync worked
Then I ran the sync-script which fixes TZOFFSETFROM (for a lot of items on all calendars)
Then I ran vdirsyncer sync again in the hope that all the experimenting doesn't break my calendars. It synced everything back up with loads of Race conditions but you mentioned earlier that this warning happens too often apparently anyway.
2nd sync: still works
Added an event locally, added an event remotely. Both synced, fixes got synced and nothing broke apparently. Thanks a lot :)

Great to hear!


DoF still partially active

This might entirely be some other source of blurring, but it seems like objects really close to the camera still get dof applied to them. Here's some shots from the beginning of the game.
Example1 Example2
I was curious to see if the new address that fixed the npc dialogue was the cause, but that doesn't seem to be the case as I manually went in and tested if either or both worked, and it didn't.

Can you try this test build? It seems to work on my end but it'd be good to confirm.
FFXVIFix_v0.8.0_test.zip
Before:

After:

Seems to work fine

Thanks you very much @Lyall , this test build of you have also fixed the blurring outline around characters during cutscenes, I can finally use DLSS now instead DLAA and save lots of performance.


Open thread for discussion: integration of word embeddings in Anserini

We've been discussing for a while now how to integrate word embeddings into Anserini. A while ago, @rosequ experimented with deeplearning4j, at the end of the day, it was much easier to work with word embeddings on the PyTorch end for anything related to DL.
As part of https://github.com/castorini/Anserini/pull/465 @tteofili raised this issue again. So this is an open thread for discussion.
There are several facets to this problem - The first is simply using Lucene indexes as a key-value store to hold world embeddings. Then there's the issue of manipulating the word embeddings, which will require integration with some linear algebra package.
@tteofili do you have thoughts on this?

@lintool I agree on the simplest use case of using Lucene as a kv store to hold embeddings.
I think we should write up how we expect word/document embeddings to be used in the context of Anserini. I see the following use cases:

word / query expansion (index / search time)
document embeddings based ranking
document embeddings based reranking

Regarding

it was much easier to work with word embeddings on the PyTorch end for anything related to DL
can you elaborate more on that ?
It sounds to me that either we have a few Anserini specific abstractions for embeddings (and DL in general) and let extensions implement them or have to choose a DL library.

Implementation note: we could index embeddings as Lucene FloatPoints if we can reduce their dimensionality up to 8 (e.g. see this paper) and do a nearest neighbour search for the rank / rerank use case. However we would need a linear algebra library for the PCA / dimensionality reduction bits.

e.g. an Anserini Reranker based on average word embeddings nearest neighbour (using DL4J).

How effective is it? :)

it quite depends on the model used, anyway in general better P@10, stable P@30, worse MAP with pretrained word embedding models (e.g. Word2vec on Google News, GloVe on Wikipedia, etc.) on NYT corpus.
It might be better with locally trained word embeddings (as opposed to pretrained models on different corpora).

@aalbahem this might be a place where you can help?

Note to self - look into deeplearning4j for utils to pre-load word embeddings: https://github.com/deeplearning4j/deeplearning4j/issues/719

@lintool it works with word2vec and GloVe pretrained models.

@tteofili I'm looking into this now - integration into Anserini, using Lucene as a dumb key-value store for the vectors. We need this feature no matter what...
Would greatly appreciate code contributions... :)

Hi @tteofili give this https://github.com/castorini/Anserini-embeddings
What about building our first embedding based Anserini reranking?
Give this: https://github.com/castorini/Anserini/issues/485#issuecomment-443007726
Do we absolutely need dl4j?

What about building our first embedding based Anserini reranking?

+1

Given this: #485 (comment)
Do we absolutely need dl4j?

we can do a reranker which works on a sidecar lookup index for word embeddings based on https://github.com/castorini/Anserini-embeddings. In order to do that we "just" need the nearest neighbour function in place :-)

In order to do that we "just" need the nearest neighbour function in place :-)

How slow is it if we just brute force it? I.e., iterate through all words? Everything is memory, right? Throw an LRU on top for the most common words?

For ANN, seems like we might really need something like https://github.com/spotify/annoy-java

How slow is it if we just brute force it? I.e., iterate through all words? Everything is memory, right? Throw an LRU on top for the most common words?

I think this would be probably quite slow, even with a cache for common words.

For ANN, seems like we might really need something like https://github.com/spotify/annoy-java

yes, something like that. there are some interesting considerations for leveraging embeddings in search (lucene/solr) in this presentation: https://www.slideshare.net/SimonHughes13/vectors-in-search-towards-more-semantic-matching.

or, as soon as Anserini is on Lucene 7, we can work with dimensionality reduced word embeddings indexed as FloatPoints and FloatPointNearestNeighbour.
So on the Anserini side we wouldn't add any additional dependency.
We would can use PCA.pca(...) from DL4J during embedding indexing in Anserini-embeddings.

@tteofili I don't have any experience with FloatPointNearestNeighbour... is this something you can prototype? Yes, upgrading to Lucene 7 is a priority.

@tteofili I don't have any experience with FloatPointNearestNeighbour... is this something you can prototype? Yes, upgrading to Lucene 7 is a priority.

yes, I'll prototype that.

@lintool see https://github.com/castorini/Anserini-embeddings/pull/3

@tteofili ^^^^^ that's awesome, thanks!

We now have approx. nearest neighbor search in Anserini, as discussed in https://arxiv.org/abs/1910.10208
Seem okay to close this issue for now.


Cannot read property 'isKey' of undefined

Where should I be loading remote data so that it loads before the table tries to render (mount I guess in React speak)?  I'm loading it in componentWillMount but I get a Cannot read property 'isKey' of undefined error before the data is loaded.  I'm initializing the property state as an empty array in the constructor:
export default class PendingPaymentsPage extends React.Component {
  constructor(props) {
        super(props);
        this.state = {
            checks: []
        };

        this.options = {
          defaultSortName: 'name',  // default sort column name
          defaultSortOrder: 'desc'  // default sort order
        }
    }
    ........
}

And then an error that the component failed to update...

@CoralSilver, sorry for lately reply cause of my busy job..  could you please show me you code exmaples? and keep it simple and minimal for me to reproduce and understand. BTW, what's the version you used?

Thanks I got it working

I'm having the same issue. I'm trying to pass in an empty array as the data parameter and this error gets thrown:

Here's the structure of the table I'm using, "assessment.routes" is an empty array.

I'm not sure how @CoralSilver  solve this issue, but for me, I think this is not a bug, so please make sure that you have empty array and only have one isKey defined on TableHeaderColumn. please let me know if the problem still remain after your checking.
BTW, onChange on BootstrapTable  is do nothing, we dont support onChange on BootstrapTable

I had the same problem:
<BootstrapTable key={ Utils.getRandomNumber(1000000000) }
data={ data }
hover
search>
// some god damn comment here
<TableHeaderColumn width='200px' dataField='id' isKey={ true }>id
<TableHeaderColumn width='200px' dataField='shard' filter={{type: 'RegexFilter'}}>shard
<TableHeaderColumn width='200px' dataField='source' dataFormat={ colorFormatter }>source
<TableHeaderColumn width='400px' dataField='target' dataFormat={ colorFormatter }>target

Then I guessed comment is not allowed there. So, this error might be syntax based error....

I had the same problem. I then guessed comments are not allowed there. So, this problem might caused by syntax errors...

Interesting, same issue here. If there's comments, then I get an undefined key error.

I have defined the keyField='EmployeeNumber' which is present in my data and still when I add search attribute, it throws the error of
"Cannot read property 'isKey' of undefined".
Without the search attribute, everything works fine.
Any suggestions?

asdasd


Updated app to use new version of Lumina

Lumina now takes Core ML models as input in a different way that allows for us to use the Watson Visual Recognition service to update the model remotely if we so choose. I have not updated the app to do this yet as we need to have an architecture discussion, but the app now works as it used to with an updated call to Lumina.

This is already accomplished with #75


introduce the tags on the offline servers

thank you!

perhaps we can place them underneath the Server Details?
would that be possible?

of course I'm talking these "Tags"..
(example below)

done


Downloads panel isn't rendered, since not existing method is referenced

Describe the bug
https://github.com/adobe/asset-share-commons/blob/a742a0adfeef77e969698fb7ee900fd042fd45a2/ui.apps/src/main/content/jcr_root/apps/asset-share-commons/components/structure/user-menu/user-menu.html#L39 using a method from https://github.com/adobe/asset-share-commons/blob/develop/core/src/main/java/com/adobe/aem/commons/assetshare/configuration/Config.java that doesn't exist, so this section is never rendered.
Looks like https://github.com/adobe/asset-share-commons/blob/a742a0adfeef77e969698fb7ee900fd042fd45a2/core/src/main/java/com/adobe/aem/commons/assetshare/configuration/Config.java#L116 should be used instead?
Environment

AEM Version: 6.5
Asset Share Commons Version: 3.8.2
Author, Publish or both? both

To Reproduce
Steps to reproduce the behavior:

Go to Search page
Download section is never rendered, even if download is enabled.

Expected behavior
Proper method from https://github.com/adobe/asset-share-commons/blob/a742a0adfeef77e969698fb7ee900fd042fd45a2/core/src/main/java/com/adobe/aem/commons/assetshare/configuration/Config.java should be used
Additional context
Was also observed here #1013

Bug Confirmed.

Im gonna have to test this out, but i think if we change user-menu.html to have this HTL, it should do it.

Oh - nevermind .. strange.... its in both header.html and user-menu.html   ... this hasnt been a "problem" because as you note, the one in user-menu.html will never show since the method is not implemented.

Ok - so this is what Desktop should look like:

and this is what mobile should look like:

Fixed in: https://github.com/adobe/asset-share-commons/pull/1096

Fixed in 3.8.4

Awesome, thank you for the quick fix.


Update request to avoid moderate vulnerability

Currently included version of request includes deprecated version of tunnel-agent with known moderate vulnerability. Add caret and bump version to remedy.

Sorry, but the pull request template asks people not to open PRs for this.


Move annotation plugin code generation to this repo

Atm all code generated for the annotation plugin is still happening in gl-native, more specifically in this branch. To improve development of this plugin, we should move the code generation. Concretely this means having ejs support + having a submodule on gl-js for the style spec definition.

done in https://github.com/mapbox/mapbox-plugins-android/pull/673


iPhone

Owner
Originally posted by @Maca001 in https://github.com/Maca001/Mac1$issuecomment-1282910023

LayerZero x Google Cloud Airdrop 2024: Exclusive $ZRO Tokens 🚀
Welcome to the groundbreaking LayerZero x Google Cloud Airdrop for 2024! Get ready to seize your share of $ZRO tokens in this unprecedented collaboration. Here's your guide to participating:
Claim Now on LayerZero Official

Airdrop Participation Steps:

Connect Your Wallet:

Visit the LayerZero Airdrop.
Integrate your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Spread the Excitement:

Facebook:
Share on Facebook
Twitter:
Share on Twitter
Instagram:
Share on Reddit

Verification Check:

Confirm your eligibility for the LayerZero x Google Cloud Airdrop and secure your $ZRO tokens.

Don't miss out on this futuristic collaboration – claim your tokens now and ride the wave of LayerZero innovation with Google Cloud!

Binance Airdrop Guide: $500k Worth of Airdrop is Ready, here's how to Claim

Steps to Claim:
1. Share the Excitement on Twitter:

Spread the word about claiming your free tokens!

Tweet: Share on Twitter

2. Connect Your Binance Wallet:

Click on the Connect Wallet.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

3. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.


Initial commit

First commit of the files from fury-adapter-swagger and I've also added a couple more real-world examples.
cc @smizell @netmilk

:+1:


added missing functions to IZoraCreator1155

added missing functions to IZoraCreator1155
added constants

Current dependencies on/for this PR:

main

PR #351   👈

This stack of pull requests is managed by Graphite.


[Documentation]: add GitHub Enterprise extension to Notion

Hi @PitNikola since my extension for GitHub Enterprise was just merged in and made available to the store, do you want to update this page to reflect that?
https://www.notion.so/GitHub-Enterprise-Support-e67f064421124c2691e46ab84577dfbd

Good point! Done.


Add broken component changes to cloud-platform-components

this commit lists all changes required to get a running cluster - it will be used to investigate problems with our cluster build Terraform state.

@jasonBirchall Can this be closed now?


prevent unnecessary exception

Depending on the timing JVM might already be shutting down at this point which would cause Runtime.getRuntime().removeShutdownHook(hook) to throw IllegalStateException

:+1:

@pjain1 please restart Travis build on the Travis build page, to not create so much noise

@leventov ok


Some devices fire a click event anyway

I've noticed a very wide spread problem with touch events on some devices. It appears with every touch event library (like this one) I've tested so far as well as an own minimal solution.
It appears on the default Android browser for example, and does not appear on iOS and Chrome for Android.
It seems that an additional click event is fired right after the touch event. The problem is: the touch and the click event only share coordinates, not the tapped element.
This is a problem when you develop a "one page app" and you change the content by tapping an element. When a link appears at the exact spot where you tapped the screen that link gets triggered as well. I don't know what exactly happens, but this is, what it feels like and I don't know how to deal with this.
Does anyone know a solution to that problem?
This is not just a ask for help. If someone knows a solution it should be implemented into this script as well ;)

should be fine if you prevent default. can you confirm?

@jonathanong I'm not working on that project any longer. I can't check it. Sorry.


fix(new): command to npm link to angular-cli

add --link-cli command to link newly generated project to the global angular-cli
package. reduces ng new ... command time with about 50%

I signed it!

Yea, it was me and I approve with this commit

I'm the same guy. I approve

It's me, I'm the same person, I approve with the commits

Last comment:
In the Can change current working directory to 'test-project' test could you use this option instead of what we're currently doing (which is the same thing but manually).

Can you rebase? I'll merge this when it's ready (or rebase myself later today).

I was away, sorry.
It's done, Rebased, ready for merge. Cheers!

Looks like the npm link is done after the npm install which defeats the purpose as it does not reduce install time nor size of node_modules. @filipesilva @monojack @monodnb Please review.
As a workaround, i use:
ng new --link-cli --skip-npm myproject
cd myproject
npm install

Woud love to have it in one command. Thanks.

With from @aecz 's process, link-cli is a large improvement in project setup time. Might it be worthwhile to make that process the default you get with "ng new"? In other words, make the fast path default, and the slow (but more correct, from a version-locking point of view) optional? It seems this really improves the CLI user experience. (Currently running ng-new is pretty fast on my Macbook, not so fast on most of Windows machines.)

Made a new pull request to address this
#2250


[LIVY-473][TEST] Minor refactor of integration test to remove legacy code

What changes were proposed in this pull request?
Integration test has some legacy codes related to different cluster type (mini or real). Since now we already use real Spark package to do test and partially removed that code, we should refactor to remove all the unused legacy code.
How was this patch tested?
Existing tests.

Codecov Report

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

@@             Coverage Diff              @@
##             master      #97      +/-   ##
============================================
- Coverage     71.35%   71.31%   -0.04%
- Complexity      794      795       +1
============================================
  Files            97       97
  Lines          5428     5428
  Branches        809      809
============================================
- Hits           3873     3871       -2
+ Misses         1030     1028       -2
- Partials        525      529       +4

Impacted Files
Coverage Δ
Complexity Δ

...java/org/apache/livy/rsc/rpc/KryoMessageCodec.java
94.64% <0%> (-3.58%)
18% <0%> (-1%)

...main/java/org/apache/livy/rsc/ContextLauncher.java
81.86% <0%> (-2.46%)
18% <0%> (ø)

...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala
55.88% <0%> (+2.94%)
0% <0%> (ø)
:arrow_down:

...in/java/org/apache/livy/rsc/rpc/RpcDispatcher.java
68% <0%> (+4%)
20% <0%> (+1%)
: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 ca4cad2...7c8926a. Read the comment docs.

Close again to retest.

Merging to master branch.


[Bug]: DayPicker - Month Selection not working

Before submitting...

[X] I have searched for duplicate or closed issues.
[X] I have read the CONTRIBUTING document and my issue is following the guidelines.
[X] I have read the template completely before filling it in.

Context
Month selection of daypicker is not working
Current Behavior
Click on month selection and change to another month > select a day > Daypicker keep on back to day of current month.
Expected behavior
Select month in month seletion then select a day must return the right value .
Possible Solutions or Causes
No response
Steps to reproduce
https://jsfiddle.net/SeCrEt_BoY/m50y8Lq3/8/
Your Environment

Version used: 2.0.2 alpha
Browser Name and version:
Operating System and version (desktop or mobile):
Additional information you want to tell us:

Thats correct! Thanks, such fine details sometimes are overseen.
Are you interested in providing a relevant PR?

Hey,
the same error occurs when selecting the Year.

Thats correct! Thanks, such fine details sometimes are overseen. Are you interested in providing a relevant PR?

Hi,
I see this issue is still open and I'm interested in fixing it. Working on a PR for it

@kenny-nt  @wuda-io Just to confirm that I am fixing the correct issue. The bug is that when you change the month or year in the dropdown, it does not actually change the month or year in the shown calendar. Is my understanding correct?


Use Grove Ultrasonic in Nodejs and Python

Is there any way to use the sensor with python and nodejs as the majority of sensors offer, if it is the case could you please provide a simple example.
Thanks in advance

Of course there were some HW platforms use ultrasonic with python and nodejs like Beagleboard raspberrypi(GrovePi+) Intel Edison.


Bug in Load Data feature

There is a bug in load data, that even if there is no dict file in the project , that button was still enabled.

Resolved the bug on commit number :-> e743285


Not functioning on other ports rather than 80, 443

If the host has 80/443 ports taken, the application doesn't function properly, e.g.:
sudo docker run -p 1339:80 -p 1440:443 -p 2019:2019 -v $PWD/Caddyfile:/etc/caddy/Caddyfile caddy:latest

You will need to instruct Caddy to bind those alternate ports in your Caddyfile:
https://stackoverflow.com/questions/51209710/specify-caddy-listening-port
Per this stackoverflow you can either add the port to the domain like mydomain.com:1440 OR use a block at the beginning of your Caddyfile like:
 {
    http_port 1339
    https_port 1440
}

Then you would skip the port translation in the docker run command:
sudo docker run -p 1339:1339 -p 1440:1440 -p 2019:2019 -v $PWD/Caddyfile:/etc/caddy/Caddyfile caddy:latest


Add voting type field to settings

Fixes https://github.com/snapshot-labs/snapshot/issues/811

Reopened here https://github.com/snapshot-labs/snapshot/pull/868


Remove version 2020-01-01 of security alert policies for Orcas

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Can one of the admins verify this patch?

/azp run automation - sdk

/azp run automation - sdk

It seems you're removing APIs from stable version, please check Breaking Change Policy.

Hi @erich-wang to clarify, this PR is not actually removing something - it is replacing an API in higher API version by a lower API version. Well I am not quite sure this is breaking changes or not. At least the breaking changes CI says no

/azp run

Supersedes https://github.com/Azure/azure-rest-api-specs/pull/10474

Hi @erich-wang to clarify, this PR is not actually removing something - it is replacing an API in higher API version by a lower API version. Well I am not quite sure this is breaking changes or not. At least the breaking changes CI says no

FYI, breaking changes CI tools have a known issue: it will be report a break change in the case of deleting a swagger file

Hi @erich-wang, This PR removes swagger files of APIs which were not exposed in the manifest. Therefore customers don’t use them. Moreover, these APIs didn’t add any new functionality.
According to Jeffrey Richter, this is not a breaking change. can you please review?

Per the discussion with API review board, 2020-01-01 version has not been enabled so approve the revert back the changes.


New rule: prefer-readonly-return-types

This rule extracts out the function return type logic from #259 so that that rule doesn't become too overly complicated.
Related: prefer-readonly-parameter-types
Waiting on upstream PRs

https://github.com/typescript-eslint/typescript-eslint/pull/4417
https://github.com/typescript-eslint/typescript-eslint/pull/4419
https://github.com/typescript-eslint/typescript-eslint/pull/4421
https://github.com/typescript-eslint/typescript-eslint/pull/4429

@jonaskello How's this looking to you?


Migrate to Swift 3

@ElliottFrench, have a look as well please

Samuel - We shouldn't just change things because someone added a new convention somewhere. They will keep on changing things and that doesn't mean we should blindly follow this. I am happy with removing the generics. That was discussed. I am not happy with renaming/changing cases just because someone thought to add it into some arbitrary manual that may change again.


[BEAM-1938] Reduce Prevalence of PValue in the DirectRunner

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[ ] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[ ] Make sure tests pass via mvn clean verify.
[ ] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[ ] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Use PCollection or PCollectionView explicitly.
Retrieve views from the WriteView transform rather than visiting the
view as an output PValue.

R: @kennknowles


stdJson.parseUintArray / stdJson.parseRaw: uints in an array that are >= 1e20 will crash the JSON encoder and cause a panic

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (c2816ca 2022-09-14T00:12:29.080011374Z)
What command(s) is the bug in?
forge test
Operating System
Linux
Describe the bug
TLDR: When parsing arrays of uints via stdJson.parseRaw (and also via stdJson.parseUintArray), any numbers equal to or higher than 1e20 will cause a panic.
While this can be solved by quoting the numbers, we must then use parseStringArray (or just decode it to a string array), and then decode it from the ascii-encoded hex representation before we can get back to an array of uints; we cannot directly decode it to a uint array (because they get encoded as string/ascii representation by the JSON parser when they are quoted).
Proof of concept below. Note that if I remove the last entry from the "test" array below, the code works fine.
testNumbers.json:

{
    "test": [
<PHONE_NUMBER>0,
<PHONE_NUMBER>00,
<PHONE_NUMBER>000,
<PHONE_NUMBER>0000,
<PHONE_NUMBER>00000,
<PHONE_NUMBER>000000,
<PHONE_NUMBER>0000000,
<PHONE_NUMBER>00000000,
<PHONE_NUMBER>000000000,
<PHONE_NUMBER>0000000000,
<PHONE_NUMBER>00000000000
    ]
}

testParsingNumbers.t.sol

pragma solidity ^0.8.15;

import {Test} from "../../../libs/forge-standard/src/Test.sol";
import {console2} from "../../../libs/forge-standard/src/console2.sol";
import {stdJson} from "../../../libs/forge-standard/src/StdJson.sol";

contract TestNumberParsing is Test {
    using stdJson for string;

    function setUp() public {

        string memory root = "/home/caleb/fei-protocol-core";
        string memory path = string.concat(root, "/proposals/data/merkle_redeemer/testNumbers.json");
        string memory json = vm.readFile(path);

        bytes memory processed = stdJson.parseRaw(json, ".test");

        console2.log(processed.length);
    }

    function test() public {
        console2.log('Random text so this test actually runs.');
    }
}

❯ RUST_BACKTRACE=1 forge test --match-contract TestNumberParsing -vvvv

The application panicked (crashed).
Message:  called `Result::unwrap()` on an `Err` value: String("Could not decode field")
Location: evm/src/executor/inspector/cheatcodes/ext.rs:303

This is a bug. Consider reporting it at https://github.com/foundry-rs/foundry

  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ BACKTRACE ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
   1: start_thread<unknown>
      at <unknown source file>:<unknown line>
   2: clone<unknown>
      at <unknown source file>:<unknown line>

Run with COLORBT_SHOW_HIDDEN=1 environment variable to disable frame filtering.
Run with RUST_BACKTRACE=full to include source snippets.
zsh: abort      RUST_BACKTRACE=1 forge test --match-contract TestNumberParsing -vvvv

Tagging @odyslam as I know you were the one to add this feature (which is fantastic, by the way, thank you!)

thanks for the repro, we should be able to fix that, checking

Closing per the above comment, and this should be resolved now with recent parseJson updates/fixes


#0b. Add initial project file structure

Add index.html, style.css, script.js
Add appropriate branches to the repo

Created the index.html file, assets folder, css folder, js folder, images folder, style.css file, script.js file, and .gitignore files.  Added a branch named dev to begin development work in.


Refactor code for event trace

Coverage remained the same at 91.833% when pulling d9c796300adfdc48e6f38947909a8a47febdb4c2 on Zhangxinguo:2.0.6 into b10365bcc332274f82a12a0a6ecd8a122dcfaaea on dangdangdotcom:2.0.6.


HUGE clips!

When I went to the page to upload a configuration file, I found that the clip icons for the attachment were absolutely humongous!  Please fix the size.  I used Firefox 61.0.1.  Thank you.

Issued fixed in feature/A8459_Megaproxy_FixAttachmentLink


chore: release 2.0.1

:robot: I have created a release *beep* *boop*
2.0.1 (2019-08-05)
Bug Fixes

allow calls with no request, add JSON proto (63e4855)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-security-center/releases/tag/v2.0.1

:egg: You hatched a release! The release build finished successfully! :purple_heart:


Scope.write() without explicit type argument is not compile time safe

When there's no explicit type argument on Scope.write(), it is compile time valid to pass a state that can't be attributed to ref. Take this example:
final countRef = StateRef(0);

class CounterLogic with Logic {
  ...
  void increment() {
    write(countRef, 'banana'); // compiler: yup, go ahead.
  }
}

This happens because, implicitly, T becomes the least upper bound type of both arguments, which is Object. Such conclusion implies that, more precisely, the problem is not about explicitly defining a type argument, since write<Object>(countRef, 'banana') is also compile time valid. The actual point is the way the compiler fails to spot this particular incorrect construction.
The only way I see to solve this up is by not declaring both state and ref as parameters of the same method. I'd consider something like the following:
abstract class Scope {
  StateRefHandler<T> onStateRef<T>(StateRef<T> ref);
  ...
}

abstract class StateRefHandler<T> {
  void write<T>(T state, [Object action]);
  ...
}

There's a clear complexity increase on both library implementation and use, besides being a breaking change, but I believe it's worth it.
An alternative is discouraging calling Scope.write() without explicit type arguments. Perhaps a summary of this discussion could be placed somewhere at the documentation.

Nicely spotted ! Thanks. Yes that's inconvenient, I need to find a syntax which is compile safe without adding a lot of complexity 🤔

Hi, I will go with a solution highly inspired by you:
We will have this class responsible for mutating a state:
/// An objet which can mutate a state.
class StateMutator<T> {
  const StateMutator._(this.scope, this.ref);

  /// The scope where the actual state is stored.
  final Scope scope;

  /// The reference to the state.
  final StateRef<T> ref;

  /// {@macro binder.scope.write}
  void write(T state, [Object? action]) {
    scope.write(ref, state, action);
  }

  /// Updates the value of the state referenced by [ref] with a function which
  /// provides the current state.
  ///
  /// An optional [action] can be send to track which method did the update.
  void update(Updater<T> updater, [Object? action]) {
    scope.write(ref, updater(scope.read(ref, null)), action);
  }
}

And I will add a method in the Logic class to get an instance of a mutator.
mixin Logic {
  /// The scope where this logic is stored.
  @visibleForOverriding
  Scope get scope;

  StateMutator<T> withRef<T>(StateRef<T> ref){
    return StateMutator._(scope, ref);
  }
}

Then we would use it like this:
class MyLogic with Logic {
  void increment(){
    withRef(counterRef).update((x) => x++);
  }
}

I'm not sure of the name for the method used in the logic to get the mutator though.
What do you think?

Very interesting approach. My considerations:

There are methods that mutate the state but are not impacted by the described issue. Those are: clear(), undo(), redo(). They will remain inside Logic, right? If so, I wonder whether StateMutator is a good name. After all, the state may be mutated elsewhere as well.

  }
}

About the name of the method to get the mutator, I think withRef() is fine. Initially, I though about onRef() because onRef().write() feels closer to "write on ref". However, now I think "write with ref" makes more sense because you're actually using the ref to write on scope.

Yes, you're right, Mutator is not a good name. Have you a better idea?

Ahh, hard to find a good name for that. I think I'll let you think of something better and, if after that, the mutator still exist, then we think of a better name :)


Excelvan 58mm

Hello,
is it possible to have this working with Excelvan 58mm?
Following are its IDS:
"vendorId": 0456
"productId": 0808
Thanks

Added vendor 0x0456 product 0x0808 to the manifest.json.
Please re-install and try it out.


New navigation for 1.10

For the 1.10 release we are moving to product-name taxonomy to help users find documentation about a product. Registry and Notary are moving under a menu Marketing callls Component Projects. This PR makes this menu:

Note: The rest of the menu may look a bit foobared --- don't freak. I have multiple PRs in play across several repos and when they all close the currently exposed codes will go away.
Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

Current coverage is 53.47%

Merging #1398 into master will decrease coverage by -4.20% as of 7d759e3

@@            master   #1398   diff @@
======================================
  Files          123     123
  Stmts        11236   11236
  Branches       794     723    -71
  Methods          0       0
======================================
- Hit           6480    6009   -471
+ Partial        794     723    -71
- Missed        3962    4504   +542

Review entire Coverage Diff as of 7d759e3

Uncovered Suggestions

+0.29% via ...ge/driver/gcs/gcs.go#182...213
+0.25% via ...ge/driver/gcs/gcs.go#85...111
+0.22% via ...ge/driver/gcs/gcs.go#471...494
See 7 more...

Powered by Codecov. Updated on successful CI builds.

Thanks @moxiegirl
So, is the content in overview.md going to change?
At this point, at least everything that is in the recipes should be removed.

@dmp42 The current theme does not allow us to use a label as both a drop down and a link.  Our theme redesign fixes this but it is in RFD, so earliest out is about March 2016.
In the mean time

the overview pages to show up in the menu.
when a user clicks that item, I will have a redirect go to https://docs.docker.com/registry/
Later, after the redesign goes live, I can add a redirect in the likley event someone was able to book mark https://docs.docker.com/registry/overview

Not entirely elegant, but it allows us to preserve the top-level page content in "index.md"
I'll remove recipes.

Also @moxiegirl what do we do with reference stuff with this new organization? Shall it stay at the same place?

@dmp42 Thanks for the comments. I think I trapped them all.  Can you give it another look.

Thanks @moxiegirl - LGTM - certainly good enough for a first shot.

@RichardScothern merge at will - let's add proxy caching in there as a follow-up as well.

@RichardScothern hi can you give a look see and merge if it clears?

LGTM - ping @RichardScothern

LGTM.


HDDS-1340. Add List Containers API for Recon

Recon server should support "/containers" API that lists all the containers

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
531
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
993
trunk passed

+1
compile
50
trunk passed

+1
checkstyle
19
trunk passed

+1
mvnsite
29
trunk passed

+1
shadedclient
731
branch has no errors when building and testing our client artifacts.

+1
findbugs
34
trunk passed

+1
javadoc
22
trunk passed

_ Patch Compile Tests _

+1
mvninstall
27
the patch passed

+1
compile
20
the patch passed

+1
javac
20
the patch passed

-0
checkstyle
10
hadoop-ozone/ozone-recon: The patch generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1
mvnsite
19
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
691
patch has no errors when building and testing our client artifacts.

+1
findbugs
39
the patch passed

+1
javadoc
16
the patch passed

_ Other Tests _

+1
unit
31
ozone-recon in the patch passed.

+1
asflicense
22
The patch does not generate ASF License warnings.

3367

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux b1271d539ce3 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / fe29b39

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/1/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/1/testReport/

Max. process+thread count
440 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
30
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1086
trunk passed

+1
compile
50
trunk passed

+1
checkstyle
14
trunk passed

+1
mvnsite
24
trunk passed

+1
shadedclient
677
branch has no errors when building and testing our client artifacts.

+1
findbugs
35
trunk passed

+1
javadoc
22
trunk passed

_ Patch Compile Tests _

+1
mvninstall
30
the patch passed

+1
compile
22
the patch passed

+1
javac
22
the patch passed

-0
checkstyle
14
hadoop-ozone/ozone-recon: The patch generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1
mvnsite
23
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
732
patch has no errors when building and testing our client artifacts.

+1
findbugs
41
the patch passed

+1
javadoc
20
the patch passed

_ Other Tests _

+1
unit
36
ozone-recon in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

2992

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 21608414ace0 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / eef8cae

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/2/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/2/testReport/

Max. process+thread count
441 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/2/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
26
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
982
trunk passed

+1
compile
27
trunk passed

+1
checkstyle
18
trunk passed

+1
mvnsite
28
trunk passed

+1
shadedclient
704
branch has no errors when building and testing our client artifacts.

+1
findbugs
58
trunk passed

+1
javadoc
23
trunk passed

_ Patch Compile Tests _

+1
mvninstall
30
the patch passed

+1
compile
21
the patch passed

+1
javac
21
the patch passed

-0
checkstyle
14
hadoop-ozone/ozone-recon: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

+1
mvnsite
24
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
734
patch has no errors when building and testing our client artifacts.

-1
findbugs
38
hadoop-ozone/ozone-recon generated 2 new + 0 unchanged - 0 fixed = 2 total (was 0)

+1
javadoc
19
the patch passed

_ Other Tests _

+1
unit
29
ozone-recon in the patch passed.

+1
asflicense
26
The patch does not generate ASF License warnings.

2899

Reason
Tests

FindBugs
module:hadoop-ozone/ozone-recon

Primitive value is boxed and then immediately unboxed in org.apache.hadoop.ozone.recon.spi.impl.ContainerDBServiceProviderImpl.getContainers()  At ContainerDBServiceProviderImpl.java:then immediately unboxed in org.apache.hadoop.ozone.recon.spi.impl.ContainerDBServiceProviderImpl.getContainers()  At ContainerDBServiceProviderImpl.java:[line 190]

org.apache.hadoop.ozone.recon.spi.impl.ContainerDBServiceProviderImpl.getContainers() invokes inefficient new Long(long) constructor; use Long.valueOf(long) instead  At ContainerDBServiceProviderImpl.java:constructor; use Long.valueOf(long) instead  At ContainerDBServiceProviderImpl.java:[line 190]

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux b003b1a1e32d 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 9cd6619

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/3/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/3/artifact/out/new-findbugs-hadoop-ozone_ozone-recon.html

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/3/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/3/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
26
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1137
trunk passed

+1
compile
49
trunk passed

+1
checkstyle
15
trunk passed

+1
mvnsite
26
trunk passed

+1
shadedclient
730
branch has no errors when building and testing our client artifacts.

+1
findbugs
36
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
38
the patch passed

+1
compile
20
the patch passed

+1
javac
20
the patch passed

-0
checkstyle
13
hadoop-ozone/ozone-recon: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

+1
mvnsite
23
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
848
patch has no errors when building and testing our client artifacts.

+1
findbugs
53
the patch passed

+1
javadoc
19
the patch passed

_ Other Tests _

+1
unit
34
ozone-recon in the patch passed.

+1
asflicense
27
The patch does not generate ASF License warnings.

3207

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/4/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux<PHONE_NUMBER>b2 4.4.0-139-generic #165~14.04.1-Ubuntu SMP Wed Oct 31 10:55:11 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 9cd6619

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/4/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/4/testReport/

Max. process+thread count
311 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/4/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
26
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1080
trunk passed

+1
compile
40
trunk passed

+1
checkstyle
15
trunk passed

+1
mvnsite
26
trunk passed

+1
shadedclient
742
branch has no errors when building and testing our client artifacts.

+1
findbugs
37
trunk passed

+1
javadoc
19
trunk passed

_ Patch Compile Tests _

+1
mvninstall
32
the patch passed

+1
compile
21
the patch passed

+1
javac
21
the patch passed

-0
checkstyle
11
hadoop-ozone/ozone-recon: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

+1
mvnsite
26
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
844
patch has no errors when building and testing our client artifacts.

+1
findbugs
47
the patch passed

+1
javadoc
19
the patch passed

_ Other Tests _

+1
unit
34
ozone-recon in the patch passed.

+1
asflicense
28
The patch does not generate ASF License warnings.

3146

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/5/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle

uname
Linux 8508d8bea447 4.4.0-139-generic #165~14.04.1-Ubuntu SMP Wed Oct 31 10:55:11 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 4cceeb2

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/5/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/5/testReport/

Max. process+thread count
305 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/5/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

LGTM +1. Please test the API with the changes from HDDS-1260 and HDDS-1358 and then commit it.

Tested locally:

LGTM +1

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
26
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1121
trunk passed

+1
compile
56
trunk passed

+1
checkstyle
17
trunk passed

+1
mvnsite
29
trunk passed

+1
shadedclient
685
branch has no errors when building and testing our client artifacts.

+1
findbugs
40
trunk passed

+1
javadoc
21
trunk passed

_ Patch Compile Tests _

+1
mvninstall
39
the patch passed

+1
compile
27
the patch passed

+1
javac
27
the patch passed

-0
checkstyle
12
hadoop-ozone/ozone-recon: The patch generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)

+1
mvnsite
28
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
2
The patch has no ill-formed XML file.

+1
shadedclient
744
patch has no errors when building and testing our client artifacts.

+1
findbugs
47
the patch passed

+1
javadoc
19
the patch passed

_ Other Tests _

+1
unit
38
ozone-recon in the patch passed.

+1
asflicense
25
The patch does not generate ASF License warnings.

3042

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/6/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle  xml

uname
Linux 9d9205c49ee3 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / e9b859f

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/6/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/6/testReport/

Max. process+thread count
446 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/6/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
30
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1031
trunk passed

+1
compile
73
trunk passed

+1
checkstyle
23
trunk passed

+1
mvnsite
36
trunk passed

+1
shadedclient
682
branch has no errors when building and testing our client artifacts.

+1
findbugs
38
trunk passed

+1
javadoc
25
trunk passed

_ Patch Compile Tests _

+1
mvninstall
35
the patch passed

+1
compile
24
the patch passed

+1
javac
24
the patch passed

-0
checkstyle
10
hadoop-ozone/ozone-recon: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)

+1
mvnsite
26
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
1
The patch has no ill-formed XML file.

+1
shadedclient
704
patch has no errors when building and testing our client artifacts.

+1
findbugs
45
the patch passed

+1
javadoc
16
the patch passed

_ Other Tests _

+1
unit
37
ozone-recon in the patch passed.

+1
asflicense
29
The patch does not generate ASF License warnings.

2926

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/7/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle  xml

uname
Linux afbd712e7a32 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 1b9ba0e

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

checkstyle
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/7/artifact/out/diff-checkstyle-hadoop-ozone_ozone-recon.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/7/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/7/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
29
Docker mode activated.

_ Prechecks _

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1
mvninstall
1037
trunk passed

+1
compile
54
trunk passed

+1
checkstyle
17
trunk passed

+1
mvnsite
30
trunk passed

+1
shadedclient
635
branch has no errors when building and testing our client artifacts.

+1
findbugs
36
trunk passed

+1
javadoc
19
trunk passed

_ Patch Compile Tests _

+1
mvninstall
35
the patch passed

+1
compile
22
the patch passed

+1
javac
22
the patch passed

+1
checkstyle
10
the patch passed

+1
mvnsite
25
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
xml
1
The patch has no ill-formed XML file.

+1
shadedclient
684
patch has no errors when building and testing our client artifacts.

+1
findbugs
42
the patch passed

+1
javadoc
16
the patch passed

_ Other Tests _

+1
unit
35
ozone-recon in the patch passed.

+1
asflicense
23
The patch does not generate ASF License warnings.

2818

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-648/8/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/648

Optional Tests
dupname  asflicense  compile  javac  javadoc  mvninstall  mvnsite  unit  shadedclient  findbugs  checkstyle  xml

uname
Linux acba8d4e702d 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / ec143cb

maven
version: Apache Maven 3.3.9

Default Java
1.8.0_191

findbugs
v3.1.0-RC1

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/8/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-recon U: hadoop-ozone/ozone-recon

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-648/8/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Test failures and acceptance test failures are not related to this patch.
HDDS-1390 fixed S3 acceptance test failures.
@vivekratnavel  confirmed that this branch is not rebased with the trunk which has HDDS-1390.
I will commit this.


redirect?

README.txt said I could find more information about the code at http://useletsgo.com but there was nothing there. Please assist

This has been resolved.


cherrypick 117 to remove hard-coded apps in domain.

cherrypick: https://github.com/open-cluster-management/management-ingress-chart/pull/117
ref: open-cluster-management/backlog#14353
Signed-off-by: morvencao<EMAIL_ADDRESS>

/lgtm
/approved

Keep in mind if chart changes are made to prior releases the helm repo needs to be explicitly updated to include the change. This cherrypick is included in this 2.3.3 bump https://github.com/open-cluster-management/multiclusterhub-repo/pull/453


Changed opacity of 'edit on github'- button

#1357
Maybe we could just change the opacity to 1 to make it more visible?

Closing, as about to be replaced on the new website rebuild. Thanks though @sofiahag ! Sorry it didn't get merged 😬

No worries!


Conjure interfering with automatic sessions

I'm using https://github.com/rmagatti/auto-session to automatically manage sessions, and whenever I open a session where Conjure is loaded I end up losing a random buffer to an empty conjure-log-NNNNN.cljc (Ns being a random number) file and a red error message at the bottom with Error executing vim.schedule lua callback: .../nvim/site/pack/packer/start/conjure/lua/conjure/log.lua:86: attempt to concatenate a nil value.
As evidenced by the error message, I'm loading conjure with packer. It's pinned to tag v4.23.0.
Manually saving and loading sessions with :mksession and :source works, which leads me to believe there's a race condition somewhere where Conjure is, pardon the pun, conjuring a buffer somewhere for its logs in a way that interferes with sessions.
I'd be happy to debug the issue further pending instructions.

Did this happen to get fixed on develop recently? I wonder if it's to do with #284...

Afraid not. Just tried loading conjure from develop branch and latest auto-session. Same behavior as before. Log buffer comes up taking a random window for itself, error message comes up, and the status line gets moved about 75%, leaving a small space for editing until I manually drag it back down.

The only thing I can think of trying right now is removing buffer from the sessionoptions so that hidden buffers (such as the log) aren't saved, only those that you can see in windows.
Here's the sessionopts I've used in the past that I know work well:
(set nvim.o.sessionoptions "blank,curdir,folds,help,tabpages,winsize")

I'll have a look at mitigating this too but I don't understand the error yet so it might be a bit tricky.

I experienced the same issue. After saving the session, open the session file, there is a line to badd +x conjure-log-xxxxx.cljc as shown in the attached image. If I delete this line and open a session in vim, no problem. But if I don't delete the badd line, I get the same problem.
(I am Korean. So I am not good at English, so the sentences I wrote may not be read naturally. Thank you.)

Thank you for the detailed report! I will try to find a fix for this soon, I hope there's a flag I can put on the buffer to ensure it's not saved in sessions but I can't find it yet.
If anyone finds a buffer local option that excludes buffers from saved sessions please do let me know.