Capybara 3 rework

Enable provisional Capybara 3 support.

Fix up method collisions and amend specs as appropriate
Fix newly introduced #all functionality (We don't call #first)
Deprecate collection DSL definition

Can I get a review from @ineverov as well as @twalpole.
I've modified the merge_args method I "believe" correctly Thomas. But It's highly likely I've misunderstood some components.
@tgaff If you have capacity to review as well that would be awesome.

I've merged this in as is @twalpole. If I've made any glaring errors I'll try DM you and put them into another PR


Upgrade step for migrating from old created schema and cred def records

Records for created schemas and cred defs were previously being created in the routes.py for the respective objects. These records are now managed by AnonCredsIssuer directly to enable it to manage artifacts pending publication. The structure is very similar to the initial records but the old still need to be migrated to the new.

This was completed via the anoncreds upgrade endpoint.


Fixed plugin compilation using MSVC

Fixes #1245

@DennisHeimbigner Seeing a weird failure in nc_test4/tst_filter.sh, but only on windows.
On linux and OSX, I see the following:
$ nccopy -M0 -F /g/var,307,9,4 unfiltered.nc filtered.nc
$ ncdump -sh filtered.nc

As expected, the ncdump command shows me the special attributes, including (for float var(dim0, dim1, dim2, dim3) in group: g)
var:_Filter = "307,9,4" ;

If I follow the same series of commands in a bash shell on Windows, however, I do not see the _Filter attribute when I query the resultant filtered.nc file.  This is causing an error in the subsequent diff command invoked in the tst_filter.sh script.  Any ideas why nccopy under Windows would not be Filtering as expected?

Let me see if I can duplicate.

It seems to work for me.
Does the file build/Testing/Temporary/LastTest.log show anything?

You can see the failure here:

http://cdash.unidata.ucar.edu/testDetails.php?test=273805&build=7069

Is this possibly a mingw problem rather than visual studio (which is what I used)?
In any case, we need to see the complete output from the last
ncdump command:

/d/netcdf-c/build/ncdump/ncdump.exe -s filtered.nc
Can you modify the tst_filter.sh to cat out the output from that command?

Hi Dennis, I used Visual Studio as well; I'm not using mingw.  I'll follow up with the output.
For reference, I'm using Visual Studio 2017 w/ HDF5 1.10.4.

I am using MSYS which gives me a bash shell but it is still using the MSVC and standard Windows libraries.

@DennisHeimbigner here is the output captured from tst_filter.sh as per your request.

msvc_tst_filter.txt

I am completely baffled.

BTW, note that the _NCProperties attribute of that file says it was created
with hdf5 1.8.20.

I am installing hdf5 1.10.4 right now to see if that makes a difference.  What version of hdf5 are you testing with?

Apparently I am using 1.8.17 when I do a visual studio build.

I downgraded to 1.8.17 and saw the same error.  Are you running from a windows CMD window, or a cygwin window?  Also, do you see no error if you upgrade to a later (1.8.x or 1.10.x) version?

@WardF This PR can be closed as it is already merged.


Systest 9780 establish gs object

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: jbigel:x: jbigel277You have signed the CLA already but the status is still pending? Let us recheck it.


Slate color definition is missing from config.yml

I've noticed that the Slate color definition is missing from config.yml so it can be used in the admin panel. This results in Slate not being adopted as a token and not being available for Sass / CSS Variables to use. I can provide a PR with a fix is this is OK.
PS. Thanks for the great starter! I started using it yesterday and I'm loving it so far.

Good catch! Yes please, a PR would be great!

Closing this issue, since solution is merged.


Import less in js file error

Laravel Mix Version: 1.2.2
Node Version : 8.0.0
NPM Version :  5.3.0
OS: Windows 10

Description:
I import less file in app.js
import './theme/iview/index.less';
Error:

When you use mix.less(), we will automatically install the necessary dependencies. In your case, though, we have no way of knowing. So you need to run:
npm install less less-loader --save-dev

And you should be good to go.


Bumps up tar to 0.4.36

Minor changes - All tests pass and some manual tests

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @ehuss (or someone else) soon.
Please see the contribution instructions for more information.

Thanks!
@bors r+

:pushpin: Commit e4f09b6afaa9e945e44dcc4675a2f7f06ad4ead5 has been approved by ehuss

:hourglass: Testing commit e4f09b6afaa9e945e44dcc4675a2f7f06ad4ead5 with merge da8dd70c5c5b397322c8ef5a1ba891d7e7fe5be8...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing da8dd70c5c5b397322c8ef5a1ba891d7e7fe5be8 to master...


Why is the default folder layout different from Maven/SBT?

I notice that Mill defines its own folder layout for Java/Scala projects (in JavaModule), and although the folder layout conventionally used in Java/Scala projects can be used by extending MavenModule, that is not the default. Mill's documentation describes its preferred folder layout, but doesn't seem to explain why that's preferred over the Maven layout.

This is not an issue; discussion goes in the gitter channel


Round start Steel and LV cables for engineering.

About the PR
Gave engineers a stack of steel and a coil of LV wires. The LV wires will help ensure even if they forget them they can still set up sigulo.
Why / Balance
Shitganeering sometimes "forgor" to set up power or to fix hull breaches. So this should allow late joins and people who did not get to the mats soon enough something to work with. (one engineer grabs all the steel)
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Changelog

:cl:

add: Added steel and LV cables to Engineers round start

sci salivating at the free mats as we speak

engi hunting subdepartment in sci

Hey, we had a discussion among the maintainers and decided that we prefer loadouts not to include materials. As such I will be closing this PR. Thank you for your contribution.

And the station will remain full of holes as engi only cares about making tritium. As one TA steals all the mats and never uses them. I question you choice. Though you are the bosses. Thanks for the consideration.

And the station will remain full of holes as engi only cares about making tritium. As one TA steals all the mats and never uses them. I question you choice. Though you are the bosses. Thanks for the consideration.

If the 30 steel is really that desperate deconstruct some lockers with the screwdriver you spawn with. Also engie doesn't make trit, atmos does (well not anymore rip frezon). As for the LV you can find coils upon coils from tool vends, cable crates, and other vending machines. The only scenario where an engine wouldn't be able to get their hands on these materials within 5 minutes of arriving is when the station is either in a total blackout & depressurization or they died near arrivals.


Add migration guide to rpc

We need to tell the users how to migrate from previous version of the library to this one, as there are some breaking changes. Namely - we want to tell how to utilise GatewayClient (which will probably be most of use cases for now) instead of previous Client.


[Open for discussion] Rework/remove density_fn to support Gaussians.

issue
The density_fn in class Field takes different arguments than get_density (it takes positions instead of RaySamples).
As a result, it is not possible to use the ProposalNetworkSampler with networks that do not take positions as arguments. One example is: MipNerf360, which uses Gaussians in the proposal network.
More generally, I don't think it makes sense to have a function that is nearly identical to get_density, but that takes in non-standard arguments.
Proposed fix
remove/deprecate density_fn. It can nearly always be replaced with get_density. In places where it is not straightforward to replace it with get_density (e.g. the occupancy grid) use a custom wrapper function.
Problem with the fix
It is a breaking change, but a breaking change will be necessary if you want to support e.g. mipnerf360.
I don't know the policy for breaking changes, but one could always not immediately remove the density_fn and put a deprecation warning on it.

@Mxbonn Fully agree! #2242

Does anybody have a sense for how many (public) external repositories would be impacted by this?

I believe external repos should fix an NS version. The development breaks lots of stuff between versions so I think its fine. Perhaps keeping a deprecation warning for a couple of versions would be nice.

Do you agree to merge this as is, or should we add some deprecation warnings?

I believe external repos should fix an NS version.

This seems like a reasonable thing to recommend!

Do you agree to merge this as is, or should we add some deprecation warnings?

I skimmed a few repos, and at the very least it seems like the K-Planes extension will break from this PR:
https://github.com/Giodiro/kplanes_nerfstudio/blob/cc5867b14697732bef4dd1de5ba49fafd419b312/kplanes/kplanes.py#L189-L214
That said, I'm okay with merging without deprecation warnings. If we do this we can follow up with an official recommendation for external methods to pin a nerfstudio version/commit in the docs, and perhaps make PRs for fixing versions of external methods that we know are out there? Does this make sense? cc also @kerrj @tancik

So anyone willing to make a decision on this?

@brentyi , do you agree with merging?

I agree. I will add the error message soon.


Component Invoking

I am confused with the 'BlazorSample' reference in the following statement
DotNet.invokeMethod('BlazorSample', 'UpdateMessageCaller');
What does 'BlazorSample' refer/relate to?
Thanks in advance

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

ID: 7bdccf73-41ed-b996-86c1-d8b7838f8773
Version Independent ID: 32d99e69-dc2d-c452-cfdb-2bf1f1956e16
Content: Call .NET methods from JavaScript functions in ASP.NET Core Blazor
Content Source: aspnetcore/blazor/call-dotnet-from-javascript.md
Product: aspnet-core
Technology: aspnetcore-blazor
GitHub Login: @guardrex
Microsoft Alias: riande

The assembly name of the app. This particular app is "BlazorSample."
I'll update it to make that clear. Thanks for calling it out.


Usability for action when graph is empty

Idea is to indicate the immediate action when a user enters the graph editor for a newly created vocab or when the graph is empty.
For this, it is proposed to show a button for creating a class node with such focus that is

it shall be centred
and button with overlay on the view
or button centred on the canvas

Possible idea: Add an overlay above mxGraph component and display a button there.

Possible idea: Add an overlay above mxGraph component and display a button there.


Logging systems and notifications

The gist of what will be logged

Currency logs
Notifications for when something has affected you
Alerts (to announce major outages/changes)
Update logs (new concept as a way of showing the key details of whats changed in the bot, think of this as the pixel progression version of dank memer's website blogs)

At the moment, we have no plans to limit the amount of logs shown, but it is something we will keep in mind in the future if it becomes a problem. For reference, Dank Memer's notification lists are limited to a maximum of 200 stored per user. This number was much higher just a few years ago when the bot had less traction.
For logging mechanisms and notifications, there will be a paginator and it will have an index tracker (using the Pagination class).
For notifications, you will be able to access a specific notification via it's an ID which opens up the notification and a paginated list of the notification directly. This means when using the command to view a specific notification (/notification view id:55), it is equivalent to (/notification list) but with the paginator length set to 1, and the page index being 55 (or 56?). We would also need to implement a search subcommand, to search for a notification by keyword (checking if it is present in both the notification title)
For the currency log, there should also be a way of sorting by the methods of how the currency entered/left in the form of a dropdown.
What it could contain
A notification typically has the following attributes:

Title: briefly summarises what this notification is about
Description: expands on what actually happened
Created at: floating point epoch timestamp indicating when this was created

A currency log typically has the following attributes:

Quantity: value deducted/added from the user's currency
Source: how it happened
Type: database representable format of how this occured so that it can be used in the sorting dropdown
Who: if possible, who caused this to happen

Implementing loggers will only get harder as development continues, the sooner this is done the better.

Starting soon, trade requests will appear in notifications.
A single notification can contain the following:

Notification ID: ID of the notification for later retrieval
Notification title: Brief title to describe what happened
Notification content: The content of the notification
Timestamp: Epoch timestamp of when the notification was sent being set in the embed

Few commands to manage your notifications will be held in the notfications group.

/notifications view (id)
/notifications list

You will be able to filter out the notifications you want to see inside the menu.
Looping over each notification will display both the title and a trimmed preview of the content

The target is to have all of the following be stored in notifications:

Share receipts
Trade receipts
Robberies (both from and to you)

Trade requests eventually expire so a background task will need to be created to delete expired requests after the expiry timestamp.
New commands will be created to accept and reject trade requests.

Soon, trade requests will appear in notifications.
A single notification can contain the following:

Notification ID: ID of the notification for later retrieval
Notification title: Brief title to describe what happened
Notification content: The content of the notification
Timestamp: Epoch timestamp of when the notification was sent being set in the embed

Few commands to manage your notifications will be held in the notfications group.

/notifications view (id)
/notifications list

You will be able to filter out the notifications you want to see inside the menu.
Looping over each notification will display both the title and a trimmed preview of the content

The target is to have all of the following be stored in notifications:

Share receipts
Trade receipts
Robberies (both from and to you)

Trade requests eventually expire so a background task will need to be created to delete expired requests after the expiry timestamp.
New commands will be created to accept and reject trade requests.


Can't run way-cooler

Hey,
Cool project! I'm trying to run it but I'm failing. Where are the instructions on how to run it?
I'm using Ubuntu 16.04 LTS.
I'm attaching a log from running it.
way.txt

Also, are you not checking isatty before you output or should I create a log in some other way? I did way-cooler 2>&1 | tee way.txt

Running it in i3wm I get segfault after the following output:
way2.txt

Hey there.
Doing that way to log way cooler is fine, those characters that are escaped are color codes so that it's easier to see in e.g less. It looks a little strange if you try to view it directly in an editor (and since the program isn't made in a unixy way, e.g the output isn't designed to be piped to a program, I think this is an ok design decision personally).
Can you provide a coredump file or a backtrace at the segfault? Would help diagnose where the issue is. Does it segfault in the same place if you run it in a tty as well as embedded in x11, or is it different?

It doesn't segfault on tty. In the log it looks like it is shutting down instead of running. Do I need to do anything else than running ./target/*/way-cooler?
Here is a backtrace from when it segfaults in X:
(gdb) bt
#0  0x00007ffff5d6fac7 in wl_list_remove () from /usr/lib/x86_64-linux-gnu/libwayland-server.so.0
#1  0x00007ffff7bb8741 in wlc_xwm_selection_release (xwm=<optimized out>) at /home/niklas/git/wlc/src/xwayland/selection.c:460
#2  0x00007ffff7bb76e6 in wlc_xwm_release (xwm=xwm@entry=0x7ffff7dd53b0 <wlc+2544>) at /home/niklas/git/wlc/src/xwayland/xwm.c:853
#3  0x00007ffff7b999f3 in xwayland_event (listener=0x7ffff7dd5c00 <wlc+4672>, data=<optimized out>) at /home/niklas/git/wlc/src/compositor/compositor.c:318
#4  0x00007ffff7bb4bbc in wl_signal_emit (data=0x7fffffffcd9f, signal=0x7ffff7dd5e00 <wlc+5184>) at /usr/include/wayland-server-core.h:357
#5  wlc_xwayland_terminate () at /home/niklas/git/wlc/src/xwayland/xwayland.c:225
#6  0x00007ffff7bb54f5 in destroy_event (listener=<optimized out>, data=<optimized out>) at /home/niklas/git/wlc/src/xwayland/xwayland.c:209
#7  0x00007ffff5d6b17c in wl_client_destroy () from /usr/lib/x86_64-linux-gnu/libwayland-server.so.0
#8  0x00007ffff5d6b298 in ?? () from /usr/lib/x86_64-linux-gnu/libwayland-server.so.0
#9  0x00007ffff5d6d4b2 in wl_event_loop_dispatch () from /usr/lib/x86_64-linux-gnu/libwayland-server.so.0
#10 0x00007ffff5d6b955 in wl_display_run () from /usr/lib/x86_64-linux-gnu/libwayland-server.so.0
#11 0x00007ffff7bb274c in wlc_run () at /home/niklas/git/wlc/src/wlc.c:260
#12 0x0000555555837c89 in rustwlc::run_wlc () at /home/niklas/.cargo/registry/src/github.com-1ecc6299db9ec823/rustwlc-0.6.2/src/lib.rs:235
#13 0x000055555580f37b in way_cooler::main () at src/main.rs:215
#14 0x0000555555a93d4d in panic_unwind::__rust_maybe_catch_panic () at /checkout/src/libpanic_unwind/lib.rs:98
#15 0x0000555555a8d4ac in std::panicking::try<(),closure> () at /checkout/src/libstd/panicking.rs:458
#16 std::panic::catch_unwind<closure,()> () at /checkout/src/libstd/panic.rs:361
#17 std::rt::lang_start () at /checkout/src/libstd/rt.rs:61
#18 0x000055555582bb83 in main ()

Hey, sorry for getting back so late.
Have you tried running sway? If you have an issue with that, it's probably an issue in wlc and not something I can fix (eg if you are running proprietary Nvidia driver on an old wlc version, or multi card setup, or some other bug).
You shouldn't need to do anything other than run the binary.
The fact it's failing at wlc startup makes me think it's a wlc / environment issue

Hey, I'm not sure I will have time to try to run Sway. I also had to remove libwayland/wayland-protocol that I installed through PPA beacuse it was conflicting with other packages. (Seems like the PPA doesn't use the same packaging as Canonical). So I will probably wait with trying out way-cooler again until I've updated my system from Ubuntu 16.04. Feel free to close this issue.

Ok, sorry I couldn't help resolve your issue :-(


[FLINK-14660][sql cli] add 'SHOW MODULES' to SQL CLI

What is the purpose of the change
Add 'show modules' to sql client.
Since all the 'show xxx' are in sql client and not in sql parser, we add 'show modules' to sql cli for now. All the 'show xxx' commands should be moved parser together all at once
Brief change log

add 'show modules' to sql cli
add listModules API to executor

Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (yes)
If yes, how is the feature documented? ( docs later)

@xuefuz @lirui-apache @zjuwangg

LGTM but the compile failure seems related.

@xuefuz @lirui-apache Thanks! I fixed that. Will merge once CI passes


fix: changed maxlength groupname to 45 char

What influence will this change have to testing sites which are currently using a groupname between 45-50 characters? Will they see a helpful error message which explains what the problem is or will they suddenly see an error?

Hi @Ein-Tim ,
Teststellen with more than 45 Char cannot create tests - as their Teststellenname is stored encrypted in the datase, it exceeds the column size otherwise (currently 100 characters) due to the encryption.
The Teststellenbetreiber will recognize this the first time they create a test  - and probably will not pin it to the long name, the Teststelle is given.

@ascheibal

The Teststellenbetreiber will recognize this the first time they create a test - and probably will not pin it to the long name, the Teststelle is given.

Okay, wouldn't it make sense to show a hint to them to help them to mitigate the issue?

This is the hint they will get.

@kerstin-oppermann-tsi Okay thanks, sorry for the questions 😅


Handling secodnary fields

Closes #16

Codecov Report

Merging #17 into master will increase coverage by 0.06%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master      #17      +/-   ##
============================================
+ Coverage     77.56%   77.63%   +0.06%
- Complexity      576      579       +3
============================================
  Files            55       55
  Lines          1658     1663       +5
  Branches        164      166       +2
============================================
+ Hits           1286     1291       +5
  Misses          250      250
  Partials        122      122

Impacted Files
Coverage Δ
Complexity Δ

src/org/smof/parsers/ObjectParser.java
74.85% <100%> (+0.75%)
52 <1> (+3)
: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 1fdc6ca...b8206b0. Read the comment docs.


Update FSTConfiguration.java

The method asObject() could lead to an IndexOutOfBoundException if the byte array argument was less than 100 bytes.

thanks :)


Enhancements for FileTransfer window

There are some simple features/improvements that would make SuperPutty more usable as a WinSCP standin. Wanted to check which are feasible rather than raising as separate issues.

Allow typing the path in to the 'address' bar on local & remote
Just default to "c:" and "/" as the directories to open on each side to save having to set a 'home prefix'
Allow basic file operations on local & remote with a context menu (view/rename/create file or folder/copy/paste/move, chmod)
Optional Tree panel for directory structure on each side of a connection
Open in 'explorer' mode to show just the remote side and allow dragging and dropping in to windows explorer.
Speed of navigation seems very slow

[x] Allow typing the path in to the 'address' bar on local & remote
Yes This can be done, open a separate issue on this

[x] Just default to "c:" and "/" as the directories to open on each side to save having to set a 'home prefix'
Yes This can be added as an option in settings. Please open a new issue for this request.

[ ] Allow basic file operations on local & remote with a context menu (view/rename/create file or folder/copy/paste/move, chmod)
No PSCP does not support these types of operations, PSFTP however does.

[ ] Optional Tree panel for directory structure on each side of a connection
Maybe This would take considerable work since pscp connections are short lived, e.g. we make a connection, do an operation (list, or transfer) and exit. It would require caching the remote folder contents

[ ] Open in 'explorer' mode to show just the remote side and allow dragging and dropping in to windows explorer.
See above answer

[ ] Speed of navigation seems very slow
Sorry this is an inherent limitation of using pscp. A future implementation might require using a library such as the one provided by winscp to become a full fledged client.

PSFTP differs from PSCP in the following ways:
PSCP should work on virtually every SSH server. PSFTP uses the new SFTP protocol, which is a feature of SSH 2 only. (PSCP will also use this protocol if it can, but there is an SSH 1 equivalent it can fall back to if it cannot.)
PSFTP allows you to run an interactive file transfer session, much like the Windows ftp program. You can list the contents of directories, browse around the file system, issue multiple get and put commands, and eventually log out. By contrast, PSCP is designed to do a single file transfer operation and immediately terminate.

I would love to eventually implement better file transfer functionality, our pscp support was initially a quick dirty implementation that I didn't really think anyone besides myself would use. It seems to be a feature people do like and use even with its current limitations.

Makes sense now. I'm guessing you don't fancy switching over to psftp? Admittedly it would turn it in to a full FTP file manager app but you would certainly end up as a market leader if you offer the flexible windows, sessions with console and remote file management. (dream app)
Will open separate issues for the other requests, thanks.

@beirtipol The main issue is backwards compatibility, pscp supports both ssh1 and ssh2, psftp only supports ssh2, so there would be considerable amount of work supporting both which quite frankly could be better spend on a better implementation for file transfers in general. I am not opposed to this now its just a matter of finding the time. Personally I normally use winscp for most file transfers.
Thanks again for the feedback and ideas!


use helm install pulsar on k8s, how change the nodeport, then can connect from machine b.

What issue do you find in Pulsar docs?
from https://pulsar.apache.org/docs/zh-CN/kubernetes-helm/, use helm install pulsar on k8s, and get the services:

but the service, like pulsar-spatio-pulsar-manager, the port 31335 is random change, when uninstall then install again,
pulsar-spatio-pulsar-manager   NodePort       <IP_ADDRESS>            9527:31335/TCP                8s
because my k8s only 8*** port can connect from other machine, so how can i define the port, for example, define :
pulsar-spatio-pulsar-manager   NodePort       <IP_ADDRESS>            9527:8527/TCP                8s
What is your suggestion?
(For example, add explanations about xx, correct descriptions of xx, delete information about xx, etc)
Do you have any references?
(For example, website links, etc)?
Would you like to fix this issue?
(Welcome to contribute docs! Pulsar grows from a tiny seed to a big tree with every little effort by us! Your contribution is the sunlight, water, and air!)

@chlyzzo - Node ports are random if they are not declared in the yaml. The current helm chart does not allow for setting the node port for the pulsar manager. Are you trying to connect to it from inside the k8s cluster or from outside the cluster?  This is not possible in k8s, as node ports range from 30000 to 32767. https://kubernetes.io/docs/concepts/services-networking/service/#nodeport

@michaeljmarshall provided an answer to the problem as a resolution to this issue. Closing this.


#1424: Some rows in VE are taller than others

Tested on various IDE fonts and high DPI screen

:+1:


RIA_6437: set the field to generate Service request for payments for …

…AG appeal type
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

I think AG needs to be added to SubmitAppealCreateServiceRequestHandler too, at line 115


why i dont see my worker name in pool

hi
sorry for my bad English
i don't see the worker name in pool i am using Nano pool
and miner is working and gpu ram is used and temp is high every thing is ok but I don't see the worker in pool
I want to be sure if the miner working to my wallet not to another one
thanks for great job

Do you use a worker name or not? It should show up when you've mined at least 1 share.

I dont use any name cuz i am using the same build in many computers

Then it won't show up as a worker name but will only show the hashrate on nanopool.

Ok, what if i need to use same build in more pc
And want all  to show in pool

You want each individual computer to have a unique worker name or just one name? If just one name then just enter one and all will mine under the same worker name but if you want a unique one then you can enter {COMPUTERNAME} as the worker name and it will use the computers name as the worker name. You can also use {RANDOM} or {USERNAME} but generally {COMPUTERNAME} is the best one.

Yes, I want a different name, I will do that now, thank you


Fix typo

@changeworld : Thanks for your contribution! The author(s) have been notified to review your proposed change. @ddematheu

Looks good; thanks for this update @changeworld.


Update install-vscode.sh

Improvements and explanations:
Added comments: The code now includes comments that explain each step of the process, making it easier to understand.
Added package installation for libsecret-1-dev and libxkbfile-dev: These packages are dependencies for Visual Studio Code Insiders and are installed along with it.
Improved readability: The code is formatted to improve readability by using consistent indentation and line breaks.
Error handling: Error handling is not explicitly added in this code snippet. It's recommended to add error checking and handling to ensure the installation process proceeds smoothly. This can include checking if commands executed successfully and handling potential errors or failures appropriately.

@Jvr2022 If you'd like to help please read our Contribution Guide.


Clarify how this configuration play along when using MigrateDatabaseToLatestVersion

Hello everyone!,
Thank you for this docs, they are very helpful however I have ran into an issue
This works just fine when using as the docs suggest "the Package Manager Console, Enable-Migrations, Add-Migration and finally Update-Database" but what happens when you are using some sort of initialization such as MigrateDatabaseToLatestVersion? I have run into an issue where my app crash because the MigrateDatabaseToLatestVersion seem to not be aware of the new name of the migration history table hence it is trying to run the migrations again and saying "SomeDatabaseObject already exist on the database" I suppose is because the  Initializer is declared like this: MigrateDatabaseToLatestVersion<SomeDbContext, SomeDbMigrationsConfiguration> and DbMigrationConfiguration is not the same as a DBConfiguration which hold my custom migration history table name. I don't know how to make the Initializer be aware of my DbConfiguration on my context since it appears to be ignoring the Decorator.
Some Code:
// The DBContext
 [DbConfigurationType(typeof(FooDbConfiguration))]
    public class FooDbContext : DbContext
    {
        public FooDbContext () : base("FooDbContext")
        {
            // Notice here FooDbConfiguration is not the same as FooDbMigrationsConfiguration
            Database.SetInitializer(new MigrateDatabaseToLatestVersion<FooDbContext , FooDbMigrationsConfiguration>());
            Configuration.LazyLoadingEnabled = false;
        }
....

// The FooDbConfiguration
public class FooDbConfiguration : DbConfiguration
    {
        public FooDbConfiguration()
        {
            this.SetHistoryContext("System.Data.SqlClient",
                (connection, defaultSchema) => new FooMigrationHistoryContext(connection, defaultSchema));
        }
    }
...
// The FooMigrationHistoryContext
 public class FooMigrationHistoryContext : HistoryContext
    {
        public FooMigrationHistoryContext(DbConnection dbConnection, string defaultSchema)
                : base(dbConnection, defaultSchema)
        {
        }

        protected override void OnModelCreating(DbModelBuilder modelBuilder)
        {
            base.OnModelCreating(modelBuilder);
            modelBuilder.Entity<HistoryRow>().ToTable(tableName: "__FOO_MigrationHistory", schemaName: "dbo");
        }
    }
...
// The FooDbMigrationsConfiguration
internal sealed class Configuration : DbMigrationsConfiguration<FooDbContext>
    {
        public Configuration()
        {
            AutomaticMigrationsEnabled = false;
           // The FooSqlServerMigrationSqlGenerator just set some default values for some date fields
          // If you think this code may be relevant let me know to add it
            SetSqlGenerator("System.Data.SqlClient", new FooSqlServerMigrationSqlGenerator());
            CommandTimeout = 180;  // 3 minute timeout
        }
//

I can make the problem go away if I directly modify the  FooDbMigrationsConfiguration to tell it explicitly about my FooMigrationHistoryContext however, I don't like this approach because It violates DRY as I'm Setting the HistoryContextFactory twice
// The FooDBMigrationsConfiguraiton with the fix
    internal sealed class Configuration : DbMigrationsConfiguration<FooDbContext>
    {
        public Configuration()
        {
            AutomaticMigrationsEnabled = false;
            SetSqlGenerator("System.Data.SqlClient", new CustomSqlServerMigrationSqlGenerator());
            CommandTimeout = 180;  // 3 minute timeout
            // This is the fix, however is repeated code with my FooDbConfiguration
            SetHistoryContextFactory("System.Data.SqlClient",
                (connection, defaultSchema) => new FooMigrationHistoryContext(connection, defaultSchema));

        }

My question is, is there a better way to make both my DbConfiguration and DbMigrationsConfiguration use my custom Migration History table name?
Thank you!

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

ID: 86069e83-0f9e-6714-c287-73d8a2e26615
Version Independent ID: 5c976d32-f7a1-e8ed-c442-037d452a243b
Content: Customizing the migrations history table - EF6
Content Source: entity-framework/ef6/modeling/code-first/migrations/history-customization.md
Product: entity-framework
GitHub Login: @divega
Microsoft Alias: divega

Thank you for requesting a change to the EF6 documentation. We are prioritizing improvements to the EF Core documentation and at this time we are not making changes to the EF6 documentation.
However, we will consider well-written community pull requests to improve the EF6 documentation. The documentation source is located in the EntityFramework.Docs repository on GitHub. To submit a pull request:

Find the file to update and click on it to open

Click on the edit button (✏) to edit in your web browser

When done, add a title and description, then use the "Create a new branch and start a pull request." option.

(Note that you can also clone the docs repo and do a GitHub PR in the normal way.)


Laravel 11 php upgrade

Hi, I tried to install this package but couldn't because it wasn't compatible with Laravel 11.
For this reason, I want to collaborate to make it possible to install it with this version.
To make this pull request, I've based myself on this pull request (https://github.com/lerouse/laravel-repository/pull/3/files).
The changes are basic and very similar to the pull request mentioned (Add version 11).
In addition to that, I've modified the tests because it gave me warnings "metadata in doc-comments is deprecated [...] use attributes instead".
On the other hand, I've made changes in the "phpunit.xml.dist" file because when I ran the local tests it gave me a warning because the XML schema validation was deprecated.

Sorry, I accidentally deleted the branch wile testing to integrate it into my project. It's already restored in case the PR is accepted.


#001 Creating Youtube Rendering Functionality

[x]  Create a logger.
[x]  Create a custom exception.
[x] Youtube file to deal with this module.
[x]  Manual check in a jupyter notebook.

Done from development perspective


Contact Form

[ ] Mailing
[ ] Firebase
[ ] Javascript actions

https://github.com/brunoocasali/xyz/issues


navigation bar menu item overlapped by other menu item

OpenUI5 version: 1.13.1
Browser/version (+device/version):
Any other tested browsers/devices(OK/FAIL):
URL (minimal example if possible):
User/password (if required and possible - do not post any confidential information here):
Steps to reproduce the problem:

using iphone se for testing
opening one by one
there exists overlapped sub menu

Could you please share a link to a minimal sample either from Demo Kit or other code sharing platforms such as JSBin, Plunkter, etc..? What is a "navigation bar menu item" here?
OpenUI5 version 1.13.1 doesn't seem to be correct. Please try identifying (based on the minimal sample) since which version exactly the behavior is reproducible.

I have checked again and founding that the root cause is not relevant to ui5. Sorry for bothering you


Update add-a-pallet.md

Nicks pallet uses Event instead of RuntimeEvent in polkadot-v0.9.28.
won't compile if it's RuntimeEvent as it is not defined.
check https://github.com/paritytech/substrate/blob/polkadot-v0.9.28/frame/nicks/src/lib.rs

Better to move forward than to move back, so closing this PR in favor of updating the tutorial for v0.9.30.

its confusing for beginners that just copy and paste that's it


Support for avatar option in create/edit project

https://docs.gitlab.com/ce/api/projects.html#edit-project -> Currently the CreateProjectOptions does not contain an entry to send along an avatar, even thought it is supported in the API.
Would be great if we could upload an avatar :smile_cat:
Maybe we can also directly support the other new params ci_config_path and tag_list?

@Bouwdie I am about to add the missing options, but it's unclear from the docs that type of info is expected for the avatar field. As the type is mixed and it's not called avatar_url like in other locations of the API.
So do you know what it expects here? Does it expect an URL, or the image itself (so a string or bytes)?

@svanharmelen I totally agree with the confusion there, the documentation is lacking there. I will check tonight what setup works. Otherwise we could log an issue at Gitlab to get clarification on the field.

👍  Thx!

Sorry for the late reply, i dove into the code of the ce edition and found out they use a file upload, and not a url.
https://gitlab.com/gitlab-org/gitlab-ce/blob/master/lib/api/projects.rb#L28
https://gitlab.com/gitlab-org/gitlab-ce/blob/master/spec/requests/api/projects_spec.rb#L477
This means we have to send multipart i guess? Maybe you see a way to fit this nicely in the current client structure? https://matt.aimonetti.net/posts/2013/07/01/golang-multipart-file-upload-example/

In the original merge request i also found the following curl example of an avatar upload:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12143
curl --request PUT --header "PRIVATE-TOKEN: XXXXXXXXX" --form<EMAIL_ADDRESS>https://my-gitlab-instance.io/api/v4/users/2

I tried to upload a file and a json part, but no luck.

`▶ curl --request POST --header "PRIVATE-TOKEN: xxx" --form "avatar=@/somefile" -F "{"name": "else"};type=application/json" http://localhost:9090/api/v4/projects
{"error":"name, path are missing, at least one parameter must be provided"}% `

when we use form fields instead of json it does work (expected).

`▶ curl --request POST --header "PRIVATE-TOKEN: xxx" --form "avatar=@/somefile" -F "name=wow" http://localhost:9090/api/v4/projects
{"id":3, ...  `

gowise we sould be looking at something like this then :crying_cat_face:
`body := &bytes.Buffer{}
writer := multipart.NewWriter(body)
part, err := writer.CreateFormFile("avatar", filepath.Base(path))
if err != nil {
log.Fatal(err)
}
_, err = io.Copy(part, file)
writer.WriteField("name", "wow")
err = writer.Close()
fmt.Print(body)
if err != nil {
log.Fatal(err)
}
req, err := http.NewRequest("POST", uri, body)`
I guess we do not want to send a multipart form for all api actions.
Splitting the action into a file upload part and the json data might be an idea.
Looking forward to your suggestions!

Instead of the line req, err := http.NewRequest("POST", uri, body) you could something like this instead:
	req, err := s.client.NewRequest("POST", u, nil)
	if err != nil {
		log.Fatal(err)
	}

	req.Body = ioutil.NopCloser(body)

So first create a new request (like we currently do for every API endpoint) and then set the body to the created multipart...

So I finally got some time to look into this feature :sake:  The form multipart seems to be an option but a complex one. I find difficulties integrating this in the client NewRequest, which currently passes a simple json body as POST body. To go the form multipart way we have to figure out how to do a multipart post with a JSON and an image part. Or serialize to form values instead of json :thinking:
	if method == "POST" || method == "PUT" {
		bodyBytes, err := json.Marshal(opt)
		if err != nil {
			return nil, err
		}
		bodyReader := bytes.NewReader(bodyBytes)

		u.RawQuery = ""
		req.Body = ioutil.NopCloser(bodyReader)
		req.ContentLength = int64(bodyReader.Len())
		req.Header.Set("Content-Type", "application/json")
	}

Another challenge would then be detect which parts of the opt variable are files (instance of Reader?) and which parts are json/form-fields.
I also have seen some blog posts (not to Gitlab) where the file is actually passed as base64 encoded string in the json which would make our lives a bit easier i guess but comes with a price. + we don't know if the server-side implementation works like this.
What do you think would be the best way to go about this? :rescue_worker_helmet:

I wrote a python script to perform this using gitlab-python module and I was able to get this to work with multipart form encoding. See the requests library for python code which creates the request body.
I used following code if you want to step through:
import gitlab
from gitlab.types import FileAttribute

gl = gitlab.Gitlab('https://gitlab.instance`, private_token='XXXXX')
avatar_file = 'path/to/avatar.jpg'

# Work around the bug in gitlab module which does not return name
def createFileAttributeNamed(name):
    class CustomFileAttribute(FileAttribute):
        def get_file_name(self, attr_name=None):
            return name.split('/')[-1]
    return CustomFileAttribute

# Force project object to treat avatar as file upload
gl.projects._types = {'avatar': createFileAttributeNamed(avatar_file)}

gl.projects.create({
    'name': 'project-with-avatar',
    'avatar': open(avatar_file, 'b').read(),
})

We could add a client.NewFormRequest(path string, opt interface{}, fileFields []string) (*http.Request, error) method. This method should convert opt to map[string][]string using gorilla/schema and replace all fields mentioned in fileFields with actual files.

It would be nice to have this feature. And even not just for Projects but for Users and Groups too.
@Bouwdie @svanharmelen does it any moves in this direction?

I would suggest that we instead make NewFormRequest allow for opt to implement io.Reader and if such value is provided, then we use form multipart to send the data. This would match Do which has a special case if v implements io.Writer.

It seems that the issue is that it is not possible to remove the avatar.

I made https://github.com/xanzy/go-gitlab/pull/1318 to address this issue.

Hey @mitar thanks for your time working on this issue. I think the function to upload solely an avatar is a nice addition. I have played around with the idea to add readers for the create/update project/user/groups and when an object with a reader property gets passed (reflection I suppose) switch from json to a form multipart to do it all in one call.


issue42 ellipse fix

Description

Fix ellipse reading for the Endpoint field
Updated documentation

Update: I made the fix at my end. Instead of reading the point into EndPoint, it was loading it into a local variable.
Change made here:
DwgObjectSectionReader.cs
readEllipse()
Line 2222
ellipse.EndPoint = this._objectReader.Read3BitDouble();


[FEATURE] Video Content Placeholder

Feature Request
Description
We need to create a way for users to consume video instruction and series
Acceptance Criteria
Video player can be ported from a third party API. Will likely need a separate server to handle and a DB that can store video and media. We can make this initially available publicly, potentially gating content down the road.

Video component is now merged in prod


Set shebang in cgi-bin scripts in myperl suse build

Replace Shebang in the web-scripts from core/server/cgi-bin/ with myperl path, see also #156

The path has already been corrected in the files themselves to use '/usr/bin/perl'. MakeMaker automatically re-writes the script files (i.e. those specified in EXE_FILES) to use the path to the current perl binary. In the case of the cgi-bin scripts above, they aren't EXE_FILES because they don't belong in the scripts directory.


Edit dict_values: user input should be key, machine string should be value

Ryan and Jasmine completed this issue alongside tests. Tests are also working.
New return value is a list of all keys (user inputs) that match the machine's return value.
The dictionary consists of keys that are user inputs, and values that are sent and retrieved from the machine.

Closing issue now as it is resolved.

Reopening to link issue to pull request.


Fixes #1146
Fixes #1148

Thanks for the patch! Just a cosmetic comment.

Fixed

Thanks!
A test would've been nice, but I don't remember if this code is all stubbed out for testing.

Yeah, I think there should be tests for the expected behaviors of %r, %s, %f, and %p when starting and finishing builds in both dumb and smart terminal modes, but that seemed like a much bigger project.


Please Can I Join This Organisation? 🥺🙏"

Name:
Kawabata Kyo
Why do you want to join this organization?
I want to work together.
Pull request with your contribution
Replace this with a link to the pull-request with your contributions.
If you have not created a pull-request yet, please follow these instructions first.

Thank you for joining this community


json from fetch api is not working

While the example uses a js file with a variable in it I try to fetch data and return it a json. itemsjs does not recognize the array. Providing the identical data as js like in the example works fine.

It is more a problem of how you handle data after making request and not ItemsJS itself.
Let's try to stringify and parse an array and it's gonna work:
var data = JSON.parse(JSON.stringify(obj));


forbid-component-props: How to distinguish a component from a library?

Situation:
import {Layout as AntdLayout} from 'antd';
...
            <AntdLayout.Content className={contentClassNames}>
...

Adding AntdLayout nor AntdLayout.Content into allowedFor isn't working. I don't want to add Content, because then linter won't report an error when our Content component is used with className.

I would expect AntdLayout.Content to work; this seems like an omission.


add option for human readable output

Adding 200,000+ lines of dependencies just for some coloring seems a little excessive. Need to find out if there's way to mitigate this or just get rid of colors for now (helps readability IMO).

Stale. Closing.


Enable shopping cart functionalities

open source contributions
There is a bug found on the home page,
the shopping cart functionality is not working you have to contribute and make it enable.

@AbhiDiva96 assign me to this issue


let properties are implicitly final, so the open modifier is me…

…aningless.  Change them to public in order to avoid a warning from the compiler
Description of the pull request
Change let properties to public instead of open
Why is the change necessary?
Eliminate build warnings in Xcode 10 and later

Hi @paulw11. Thanks for the PR and sorry for the delay in getting back to you about it.
The changes in this PR have been merged in a recent PR therefore this PR is redundant and I'll close it.


Trigger production bundle release for v0.19.0

Triggering the production bundle release pipeline for EKS-A v0.19.0 release.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/approve

/unhold


Settings Editor: Give more contrast to backtick code blocks

Testing #55401
Interactive Playground:

GitHub:

Settings Editor:

It's much harder to scan through the setting description to find which word is code.

Good point! I thought that text was missing something. Hover has a background color for code segments, I wonder why we use one or the other. Thoughts @misolori?

I think I prefer for them to be consistent, but then that adds more confusion with our links. What if we kept the same styling as we do with the playground (has both links and code blocks) and made the links use our regular font and also match the naming structure?

I like that! I hadn't even thought about the names.

Opened https://github.com/Microsoft/vscode/issues/55695 for the link names.


infra and vars for update schema endpoint

adds api gateway infra for the PUT method for /data-product/{data-product-name}/table/{table-name}/schema which is used to update a table schema
Also adds lambda function to be invoked and variables for image.

LGTM - can you also update https://dsdmoj.atlassian.net/wiki/spaces/DataPlatform/pages/4557963328/Testing+user+journeys+WIP with a curl example when you test this?


Don't make vault start on boot by default

We really only want to set this manually after cloning the pot.

Sorry, I did this on github using their UI and didn't notice that there wasn't a pull request generated.

Thanks @grembo, I didn't see the file added and adjusted my side. Fixed.

Closed, see above


[1.7] Validate Gateway specs attempting to bind with <1024 port without root (#26699)

Fixes https://github.com/istio/istio/issues/26708

@howardjohn: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

unit-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

unit-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-telemetry-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

unit-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-telemetry-k8s-tests_istio

gencheck_istio_release-1.7
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

unit-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-telemetry-k8s-tests_istio

gencheck_istio_release-1.7
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio_release-1.7

unit-tests_istio_release-1.7
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-notes_istio

integ-security-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-security-k8s-tests_istio

integ-ipv6-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-ipv6-k8s-tests_istio

analyze-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test analyze-tests_istio

release-test_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test release-test_istio

integ-operator-controller-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-operator-controller-tests_istio

integ-distroless-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-distroless-k8s-tests_istio

gencheck_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio

integ-pilot-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-pilot-k8s-tests_istio

lint_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test lint_istio

integ-multicluster-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-multicluster-k8s-tests_istio

unit-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio

integ-telemetry-k8s-tests_istio
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test integ-telemetry-k8s-tests_istio

gencheck_istio_release-1.7
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test gencheck_istio_release-1.7

unit-tests_istio_release-1.7
91da0a3e98429787953bb1c060e50528aa23c7fb
link
/test unit-tests_istio_release-1.7

integ-multicluster-k8s-tests_istio_release-1.7
885f9034ae869ead2b791ad37daeec2b3b289c42
link
/test integ-multicluster-k8s-tests_istio_release-1.7

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Fix bugs in the createtasks function

Use multi-thread to improve the performance.

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

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

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

TTYL, AZPRBOT;

Can one of the admins verify this patch?


seq_len, label_len, pred_len for ECL

Hi, could you share the scripts for reproducing ECL results? Thanks!

Thanks for your attention, we will release the ECL parameters as soon as possible after processing the recent research work.


Update munit to 1.0.0-M9 in main

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

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

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta", artifactId = "munit" }
}]

labels: test-library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #123.


Add support for Pip's extra dependencies in YAML config.

Depends on: https://github.com/rtfd/readthedocs-build/pull/16.
Also closes #173.

Thanks for the PR. test_extra_requirements doesn't pass now: See https://travis-ci.org/rtfd/readthedocs.org/jobs/152807544#L238 for details. I guess you also need to adjust the validate_python method in https://github.com/rtfd/readthedocs-build/blob/master/readthedocs_build/config/config.py#L204.

Thanks @berkerpeksag for the heads up. I overlooked the dependency to readthedocs-build. I'll try to push a PR there soon.

Both PRs look good to me, thanks! I added the WIP label since we need to get https://github.com/rtfd/readthedocs-build/pull/16 in first.

All comments have been addressed. I think this PR is ready to be merged.
Just waiting for https://github.com/rtfd/readthedocs-build/pull/16 now...

https://github.com/rtfd/readthedocs-build/pull/16 has been merged upstream thanks to @ericholscher .
Should we wait for a proper release of the readthedocs-build package on PyPi before we can move forward on this PR?

We can just update the pip requirements to point at the commit for now -- we could also do a new release of rtd-build as well, if that is easier.

Could you please squash the commits?

@ericholscher: yep, that what I preemptively did in 5af1222c1c356c476ffeddeb29ffe05f2212a8cf. Hence @berkerpeksag proposal to squash commits.

@berkerpeksag: oh, BTW, is that my job to squash the commits? Just tell me what to do. But while we're at it I just wanted to point out that GitHub is now allowing repository owners to merge PR and squash commits in one go with a simple click. See: https://github.com/blog/2141-squash-your-commits

Yes, GitHub does the job for us now, but not every project use it by default. I sometimes forget to choose "Squash and merge" option for example :) So it would be nice to keep the history linear.

@berkerpeksag OK. Squashed all commits. Ready to be merged IMHO.

Will merge & fix typo. Thanks for the PR!

Thanks @ericholscher for the merge! :)

Sorry to bother you but, do we have an idea when this PR is going to hit the production servers of readthedocs.org?

I can get it deployed today.

I can wait a week or two. There is no urgency as the target is some sort of non-existential open-source projects. Just wanted to give a quick feedback about the effect of the PR.

Should be deployed now, let me know if it works :)

Sorry to bother you but I can't see any extra_requirements section being picked up by the build process. See this build, which I expected to use the corresponding readthedocs.yml config.

Hmm, did this work locally? I will try and take a look at it this week.
On Sat, Aug 27, 2016 at 4:03 AM, Kevin Deldycke<EMAIL_ADDRESS>wrote:

Sorry to bother you but I can't see any extra_requirements section being
picked up by the build process. See this build
https://readthedocs.org/projects/maildir-deduplicate/builds/4342502/,
which I expected to use the corresponding readthedocs.yml config
https://github.com/kdeldycke/maildir-deduplicate/blob/7dfb8412fc1d6c7747527a41528edef73651f3d1/readthedocs.yml
.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/rtfd/readthedocs.org/pull/2368#issuecomment-242910880,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABjpnJAic-6etBcQEeus71fIZY5W76kks5qkBmSgaJpZM4Jl3Wa
.

--
Eric Holscher
Maker of the internet residing in Portland, Oregon
http://ericholscher.com

@ericholscher Sorry. I just relied on unit-tests to cover this feature.

@kdeldycke, doesn't work for me too, see #2432

@ericholscher, @kdeldycke - it seems this does work for me on the local setup.
Here is the part of build log:
[07/Oct/2016 05:47:56] INFO [readthedocs.doc_builder.environments:102] Running: 'python /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/envs/latest/bin/pip install --use-wheel -U --cache-dir /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/.cache/pip sphinx==1.3.5 Pygments==2.1.3 setuptools==20.1.1 docutils==0.12 mkdocs==0.15.0 mock==1.0.1 pillow==2.6.1<EMAIL_ADDRESS>sphinx-rtd-theme==0.1.9 alabaster>=0.7,<0.8,!=0.7.5 commonmark==0.5.4 recommonmark==0.1.1' [/home/sk/rtd/checkouts/readthedocs.org]

[07/Oct/2016 05:49:57]"POST /api/v2/command/ HTTP/1.1" 201 8163
[07/Oct/2016 05:49:57] INFO [readthedocs.doc_builder.environments:102] Running: 'python /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/envs/latest/bin/pip install --ignore-installed --cache-dir /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/.cache/pip .[docs]' [/home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/checkouts/latest]

And in the web interface I have the following line:
python /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/envs/latest/bin/pip install --ignore-installed --cache-dir /home/sk/rtd/checkouts/readthedocs.org/user_builds/diofant/.cache/pip .[docs]

Please notice docs extra dependence and c.f. with the build on the rtfd.org (just plain pip install .):
http://readthedocs.org/projects/diofant/builds/4462416/
For funny reasons this build still was successful (some caching?), build most recent rebuilding from the same master commit was failed: http://readthedocs.org/projects/diofant/builds/4484132/
@ericholscher, Are you sure that the recent readthedocs-build was actually deployed?

Thanks @skirpichev for the feedback! Also suspecting a deployment issue on RTD production server since the beginning.

@ericholscher, support for .readthedocs.yml doesn't work too after recent update (#2442), so I bet the readthedocs-build package wasn't updated in production.  Could you check that?

Ok, now this finally works.  I'll close #2432.


CVE Fix: Update json-path version

Description
Apache Druid brings the dependency json-path which is affected by CVE-2023-51074.
Its latest version 2.9.0 fixes the above CVE.
This PR modifies pom file, license file, OrcReaderTest and OWASP Dependency Check suppression file.
Append function has been added to json-path and so the unit test to check for the append function not present has been removed.
Release Note:
JSON-Path upgrade now allows append functionality. However, we do not support json-path append function for ORC format (see https://github.com/apache/druid/pull/11722). Earlier, if the user tried to use JsonPath function append earlier, it would have not appended the item, but now the same function will return an exception. We are overriding the parse function used when append is invoked. So, row.getDimension("append") results in java.lang.UnsupportedOperationException: Unused coming from
druid/processing/src/main/java/org/apache/druid/java/util/common/parsers/FlattenerJsonProvider.java
This is a potential issue to watch out for when upgrading.

This PR has:

[x] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[x] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[x] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

I tried rerunning the flaky test. Didn’t seem to help. Since the failure doesn’t appear to related I’m ok merging this.


Multiple button support?

Is there a way to display multiple buttons on top of each other in such a way that they're stacked in the view? I briefly saw the documentation and saw that action-methods are supported, but how would one pass n-number of buttons, each with different delegate methods?

Hi,
You can use EKProperty.ButtonBarContent as data source of EKButtonBarView.
EKButtonBarView receives EKProperty.ButtonBarContent instance and inits itself accordingly.
If there are more than 2 buttons - they automatically stacked vertically as in the alert presets in the demo app.
EKProperty.ButtonBarContent instance contains an array of EKProperty.ButtonContent - one for each button. Each has an action.
More than one action (.touchUpInside) is currently not supported. But i'm open to suggestions if that's not enough IYO :)

Could this only be achieved with EKAlertMessageView?

That's an interesting point.
EKAlertMessageView only provides UI infrastructure for using it. It uses expandAnimately to expand the buttons by changing inner priorities.
Since EKButtonBarView doesn't handle the expansion itself, you need to do it yourself by doing:
buttonBarView.expand()
buttonBarView.layoutIfNeeded()

I should move the expand logic to the button bar view itself sometime soon. I have no doubt that logic should be there and not in its container.

Right, I was using the UI debugger on Xcode and setting the EKAttributes from
attributes.positionConstraints.size = .init(width: .offset(value: 20), height: .intrinsic)

to
attributes.positionConstraints.size = .init(width: .offset(value: 20), height: EKAttributes.PositionConstraints.Edge.constant(value: 100))

displays the buttons, but not vertically stacked with the third button. Also, using
buttonBarView.expand()

logs this error: 'expand' is inaccessible due to 'internal' protection level

By the way, I think this is the best presentation libraries out there on this planet 👍

Thanks a lot! ^^
I'll review the button bar view anyway and expose more of its methods.

I revised EKButtonBarView's expand method to contain all the expansion related logic.
Now, you basically only need to create an EKProperty.ButtonBarContent instance, initEKButtonBarView using it. Then call display method on SwiftEntryKit using your attributes.
It's available on version 0.3.3.

@HackShitUp
B.T.W: You still need to invoke expand on the EKButtonBarView instance.
I believe the requirements specified in this issue are satisfied. Let me know if you think otherwise.
Thanks!

@huri000  I am new to Swift and was trying to use your Library, what I was trying to achieve is the same thing which @HackShitUp, multiple buttons in the EKPopUpmessage.
I checked the Rating example but, what I perceive that it has buttonBarContent: <#T##EKProperty.ButtonBarContent#>), but the PopUp msg does not have that property, EKPopUpMessage(themeImage: <#T##EKPopUpMessage.ThemeImage?#>,
title: <#T##EKProperty.LabelContent#>,
description: <#T##EKProperty.LabelContent#>,
button: <#T##EKProperty.ButtonContent#>,
action<#T##EKPopUpMessage.EKPopUpMessageAction##EKPopUpMessage.EKPopUpMessageAction##() -> ()#>)
Please help how to accomplish this, I am ClueLess, if there is any example, it would be much Helpful.
Thanks.

still not any solution


Coordinator doesn't approve milstones

When I deploy new private tangle as this tutorial proposes, I have 3 nodes: Spammer, Coordinator and simple user node. Till the 30th of March - everything did look fine, but accidentally Coordinator was crashed. After many attempts of reloading, reinstalling, Coordinator refuses approve new milestone right after my custom node is synchronized with Coo. It just throws following exception:
failed to create: signature validation failed: %!w(<nil>)
Does anybody have an idea what this means?

It means the milestones in you network don't fit the configured coordinator address in your config file.
Easiest way would be to start with a fresh network. Delete all the databases of all nodes and bootstrap your network.
I would also recommend to use the latest develop version of HORNET, since all the breaking IOTA protocol changes in there will be live soon.