consider using krew-release-bot for auto releasing allctx plugin on krew

Hi
I manage krew-release-bot, a bot to automatically do version bump PR with krew-index repo once new release is made.
request you to kindly consider including the bot in your release process. FYI, it does not require any credentials as such to work.
a sample PR: kubernetes-sigs/krew-index#1271
Thanks
Rajat Jindal

Thank you for letting me know about your bot. I don't think this plugin will ever need to be updated.


Base: add back proposal details quorum %

@yayay927 my initial thought process is to just add the gov quorum param on the side like so

@ryuash Hmm it doesn't really make sense to me to add it in the corner. And it's (quorum 33%) not even in the same height as title (voted/ total). Maybe we can ask designer to take a really quick look about where is a good place to show quorum information?

How about somewhere here?

Spoke with Design team and they said they want a ? icon in the top right corner (similar to the original picture) but when you hover over, a small tooltip will show up and say something like For a proposal to be considered valid, voting must reach at least xx% quorum
So the icon can be like

@ryuash Okay~ I think it's good to hide information first and when users want they can check.


Flexbox implementation

It would be worth considering an implementation purely using flexbox.

how to say? any idea for using it to improve this component with bootstrap?


New version: Mozilla.Firefox.DeveloperEdition.br version 132.0 (132.0b4)

Created by 🥟 Dumplings.
Microsoft Reviewers: Open in CodeFlow

Verify by proxy: https://github.com/microsoft/winget-pkgs/pull/179431#issuecomment-2400690391


Documentation improvements

Various minor documentation improvements, as suggested by @benjaminrose in this post in the JOSS review.

[ ] Make clear(er) that sntools is a command line app.
[x] Compiled version of documentation.
[ ] First example should run more quickly.
[ ] Simplify arguments of first example

@JostMigenda, I like the current example becuase it shows many sntools options. I would keep it along side an improved first run experience.

@benjaminrose Perhaps it’s best to have two separate examples—an introductory one (that’s very quick and uses few optional arguments for clarity), and a more advanced example (that can run for longer and demonstrate many common arguments)?
E.g. sntools fluxes/intp2001.data --format nakazato --endtime 50 as an introduction (that runs in ~15 s on my machine) and sntools fluxes/intp1301.data --format nakazato --ordering normal --distance 20 --detector SuperK --verbose

Regarding compiled version of the LaTeX documentation:
I’ve uploaded a pdf file to release v0.7 and have added an action that should automatically generate a pdf file for every new release and attach it.

Sounds great!

I’ve updated the examples as discussed, though I decided to use --channel es instead of --ordering normal for the second one: That makes the output shorter and easier to explain because it’s only simulating 4 channel/flavour combinations (instead of 14 for normal mass ordering and all channels).
Also noted explicitly that it’s a command line app.
After those commits, I think #18 covers all the requested documentation improvements and I will merge it soon if there are no additional comments.


Manylinux wheels for packages like pandas and numpy cannot be replaced by zappa

I was trying to zappa deploy a python 3.8 function to AWS Lambda using MacOSX development environment.
The python 3.8 function depends on numerous packages, including pandas.
When I was doing development, and cross check the deployment package of zappa, i found that the pandas library in the package is still darwin's wheel, expected to be manylinux wheel.

Expected Behavior
Expect manylinux wheels for pandas and numpy will be replaced when calling zappa deploy / update.
Actual Behavior
Pandas and numpy inside the virtualenv is still darwin package
Possible Fix
Change the line at https://github.com/zappa/Zappa/blob/1cdbd7f828b643b2a70c32fd334ec4f0662642de/zappa/core.py#L320
from:
f'^.{self.manylinux_suffix_start}-manylinux({"|".join(manylinux_suffixes)})_x86_64.whl$'
to
f'^.{self.manylinux_suffix_start}(.*?)(-|.)manylinux({"|".join(manylinux_suffixes)})_x86_64.whl$'
Steps to Reproduce

write a python 3.8 lambda handler
create an environment with pip install the latest pandas
try zappa deploy in macos, with delete_local_zip = false
unzip the generated package, check the wheel tag for pandas
Find the wheel tag is cp38-cp38-macosx_10_9_x86_64, which is unexpected

Your Environment

Zappa version used: latest (0.53)
Operating System and Python version: Darwin 20.5.0 Darwin Kernel Version 20.5.0: Sat May  8 05:10:33 PDT 2021; root:xnu-7195.121.3~9/RELEASE_X86_64 x86_64 i386, Python 3.8.6
The output of pip freeze:
argcomplete==1.12.3
boto3==1.17.112
botocore==1.20.112
certifi==2021.5.30
cfn-flip==1.2.3
charset-normalizer==2.0.2
click==8.0.1
durationpy==0.5
future==0.18.2
hjson==3.0.2
idna==3.2
jmespath==0.10.0
kappa==0.6.0
numpy==1.21.0
pandas==1.3.0
pep517==0.10.0
pip-tools==6.2.0
placebo==0.9.0
python-dateutil==2.8.2
python-slugify==5.0.2
pytz==2021.1
PyYAML==5.4.1
requests==2.26.0
s3transfer==0.4.2
six==1.16.0
text-unidecode==1.3
toml==0.10.2
tqdm==4.61.2
troposphere==3.0.1
urllib3==1.26.6
Werkzeug==0.16.1
wsgi-request-logger==0.4.6
zappa==0.53.0
Link to your project (optional):
Your zappa_settings.json:

Same issue on Win 10, Python 3.7, zappa==0.53.0 with numpy. Verified wheel tag is cp37-cp37m-win_amd64 in the zip file.
I made change to the code you suggested however that caused no manylinux wheels to be installed.
It looks like this may be a relevant issue on the numpy issue tracker. Based on that I downgraded to numpy 1.19.4.
Downgrading to numpy==1.19.4 resolved the issue.
When deployed it causes an error like:
Original error was: No module named 'numpy.core._multiarray_umath'
Logs:
[1628009634217] Instancing..
[1628009635324] [ERROR] ImportError: Unable to import required dependencies:
[1628009635324] numpy:
[1628009635324] IMPORTANT: PLEASE READ THIS FOR ADVICE ON HOW TO SOLVE THIS ISSUE!
[1628009635324] Importing the numpy C-extensions failed. This error can happen for
[1628009635324] many reasons, often due to issues with your setup or how NumPy was
[1628009635324] installed.
[1628009635324] We have compiled some common reasons and troubleshooting tips at:
[1628009635324] https://numpy.org/devdocs/user/troubleshooting-importerror.html
[1628009635324] Please note and check the following:
[1628009635324] * The Python version is: Python3.7 from "/var/lang/bin/python3.7"
[1628009635324] * The NumPy version is: "1.21.1"
[1628009635324] and make sure that they are the versions you expect.
[1628009635324] Please carefully study the documentation linked above for further help.
[1628009635324] Original error was: No module named 'numpy.core._multiarray_umath'
    "Unable to import required dependencies:\n" + "\n".join(missing_dependencies)e

I am seeing something similar with psycopg2-binary.  It looks like psycopg2 switched to using PEP600:
https://www.psycopg.org/docs/news.html#what-s-new-in-psycopg-2-9
So now instead of having a filename like this:
psycopg2_binary-2.8.6-cp38-cp38-manylinux1_x86_64.whl
It now looks like this:
psycopg2_binary-2.9.1-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
It looks like the the manylinux_wheel_file_match regular expression almost finds the PEP600-formatted filename using the legacy suffix except that now there is a period before "manylinux" instead of a hyphen.  It looks like the possible fix recommended above could work.

I was still having this issue with 0.54.0 and pandas==0.25.2. Getting macOS wheel tag (cp39-cp39-macosx_10_15_x86_64) in the generated package. This version of Zappa has the following regexes:
        # AWS Lambda supports manylinux1/2010 and manylinux2014
        manylinux_suffixes = ("2014", "2010", "1")
        self.manylinux_wheel_file_match = re.compile(
            f'^.*{self.manylinux_suffix_start}-(manylinux_\d+_\d+_x86_64[.])?manylinux({"|".join(manylinux_suffixes)})_x86_64[.]whl$'
        )
        self.manylinux_wheel_abi3_file_match = re.compile(
            f'^.*cp3.-abi3-manylinux({"|".join(manylinux_suffixes)})_x86_64.whl$'
        )

Upgrading pandas to 1.3.4 fixed this, so it would seem as if the current regex is no longer backward-compatible with some older wheels. Not sure if it ever was.
However, I'm still unable to get the Linux wheel for psycopg2==2.9.1.

Whoops, I didn't have psycopg2-binary in my dependencies. That fixed it.

Thanks for the update @dpmccabe - closing this issue as there are no more issues reported so far 👍🏽

Confirmed my original reported issue is gone. Thanks all.


Errors parsing CSS

@simolus3
how do i exclude certain folders and files from being analyzed by sass_buider
build.yaml
targets:
  $default:

    #sources:
      #exclude:
        #- "web/assets/css/**"
        #- "web/styles.css"

    #  sass_builder
    builders:
      sass_builder:
        generate_for:
          - "lib/**"       # not work
        options:
          outputStyle: compressed

ngdart on web/styles.css: Errors parsing CSS:
error on line 29, column 12: parsing error expected {
   ╷
29 │  @supports selector(::-webkit-scrollbar) {
   │            ^^^^^^^^
   ╵
error on line 29, column 20: expected }, but found (
   ╷
29 │  @supports selector(::-webkit-scrollbar) {
   │                    ^
   ╵
error on line 29, column 40: expected {, but found )
   ╷
29 │  @supports selector(::-webkit-scrollbar) {
   │                                        ^
   ╵
error on line 29, column 42: expected }, but found {
   ╷
29 │  @supports selector(::-webkit-scrollbar) {
   │                                          ^
   ╵
error on line 42, column 2: premature end of file unknown CSS
   ╷
42 │  }
   │  ^
   ╵
[SEVERE] ngdart on web/styles.css: Null check operator used on a null value

web/styles.css
 /* no phosphor 1 é \f31a  .ph-caret-down:before
  no phosphor 2.0.3 é \e9fe .ph-caret-down:before*/
 .dropdown-toggle::after {
     display: inline-block;
     margin-left: .25em;
     vertical-align: -.125em;
     font-family: var(--icon-font-family);
     font-size: 1em;
     line-height: 1;
     content: "\e9fe";
 }

 html {
     --scrollbar-color-thumb: rgb(98, 98, 98);
     --scrollbar-color-track: rgb(31, 31, 33);
     --scrollbar-width: thin;
     --scrollbar-width-legacy: 10px;
 }

 /* Modern browsers with `scrollbar-*` support */
 @supports (scrollbar-width: auto) {
     html {
         scrollbar-color: var(--scrollbar-color-thumb) var(--scrollbar-color-track);
         scrollbar-width: var(--scrollbar-width);
     }
 }

 /* Legacy browsers with `::-webkit-scrollbar-*` support */
 @supports selector(::-webkit-scrollbar) {
     html::-webkit-scrollbar-thumb {
         background: var(--scrollbar-color-thumb);
     }

     html::-webkit-scrollbar-track {
         background: var(--scrollbar-color-track);
     }

     html::-webkit-scrollbar {
         max-width: var(--scrollbar-width-legacy);
         max-height: var(--scrollbar-width-legacy);
     }
 }

pubspec.yaml
name: esic_frontend_site
description: Frontend (WEB) do Sistema Eletrônico de Informações ao. Cidadão (e-SIC)
version: 2.0.0
publish_to: none

environment:
  sdk: ^3.2.1

dependencies:
  ngdart: 8.0.0-dev.4
  ngrouter: 4.0.0-dev.3
  ngforms: 5.0.0-dev.3

  intl: any
  chartjs2: ^2.0.0
  http: any
  js: any
  pdf: ^3.10.8

  esic_core:
    path: ../core
  #bootstrap_sass: any

dev_dependencies:
  build_runner: ^2.1.2
  build_test: ^2.1.3
  build_web_compilers: ^4.0.0
  lints: ^2.1.0
  test: ^1.24.0
  sass_builder: ^2.2.1

Applying generate_for: ['lib/**'] should stop the sass builder from running on web/ files. You said it doesn't work, what exactly is going wrong?

After some time trying to solve this, I finally got it to work correctly like this:
targets:
  $default:
    builders:
      ngdart:
        generate_for:
          exclude:
            - "web/assets/css/**"
            - "web/assets/js/**"
      sass_builder:
        options:
          outputStyle: compressed

It would be nice if this was documented in the sass_builder README.md

Wait a second:

ngdart on web/styles.css: Errors parsing CSS:

sass_builder isn't even involved in that error! If the errors were coming from sass_builder, your original build.yaml would have prevented them too.
Maybe this is something that the AngularDart project should document then?


Angular/cli issues

I tried building an angular/cli-app using radix. i had the following issues:

node_modules/radixdlt/build/module/modules/wallet/RadixKeyPair.d.ts(15,17): error TS2304: Cannot find name 'Long'.
I was able to fix it by adding "import Long from 'Long';" to this file.

node_modules/radixdlt/build/module/modules/universe/RadixNodeConnection.d.ts(5,8): error TS1192: Module '"events"' has no default export.
Fixed it by adding "esModuleInterop": true to the tsconfig.json.
(angular version: 7.2.0, typescript version: 3.2.2)

Closed since this library is now deprecated.


Fix bug in attack_random_noise.py in adversarial competition toolkit.

This is outside of the cleverhans/ or tutorials/ folders, LGTM


Add twiddler finger by hydra

Song Name
twiddler finger
Artist Name
hydra
Source
Youtube
Youtube Link
liI9X0voRQh_zl2e
Direct File Link
No response
Song ID
700211
Start Offset [ms]
1
Extra
Created from Auto Nong.
Created for level: 28225110

idk how to put it in geometry dash or idk if i did put it in

accept

pls accept

reject
invalid yt id.


chip-tool-darwin Support Needed for OTA-R Subscribe Events command

Problem
When testing TC-SU-2.7, each step needs support for chip-tool-darwin to run a subscribe-event state-transition command for OTA-Requestor .
An example of this command is Step 1 verification step: ./out/chip-tool-darwin/chip-tool-darwin otasoftwareupdaterequestor subscribe-event state-transition 5 10 0x1234567890 0 --timeout 65535
Current supported commands for chip-tool-darwin
16-MacBook-Pro:connectedhomeip liam$ ./out/chip-tool-darwin/chip-tool-darwin otasoftwareupdaterequestor
[1652372264756] [93441:7162147] CHIP: [TOO] Missing command name
Usage:
  ./out/chip-tool-darwin/chip-tool-darwin otasoftwareupdaterequestor command_name [param1 param2 ...]

  +-------------------------------------------------------------------------------------+
  | Commands:                                                                           |
  +-------------------------------------------------------------------------------------+
  | * announce-ota-provider                                                             |
  | * read                                                                              |
  | * write                                                                             |
  | * subscribe                                                                         |
  +-------------------------------------------------------------------------------------+
[1652372264758] [93441:7162147] CHIP: [TOO] Run command failure: ../../examples/chip-tool-darwin/third_party/connectedhomeip/examples/chip-tool/commands/common/Commands.cpp:86: Error 0x0000002F

Chip-tool Example
16-MacBook-Pro:connectedhomeip liam$ ./out/chip-tool/chip-tool otasoftwareupdaterequestor subscribe-event
[1652373953034] [95132:7195802] CHIP: [TOO] Missing event name
Usage:
  ./out/chip-tool/chip-tool otasoftwareupdaterequestor subscribe-event event-name [param1 param2 ...]

  +-------------------------------------------------------------------------------------+
  | Events:                                                                             |
  +-------------------------------------------------------------------------------------+
  | * state-transition                                                                  |
  | * version-applied                                                                   |
  | * download-error                                                                    |
  +-------------------------------------------------------------------------------------+
[1652373953035] [95132:7195802] CHIP: [TOO] Run command failure: ../../examples/chip-tool/commands/common/Commands.cpp:105: Error 0x0000002F

Test Case
3.8.1. [TC-SU-2.7] Tests Events generated by OTA-R(DUT)

@liamgonyea for chip-tool-darwin currently there is no plan to support for individual subscriptions for events. The way to test this is to subscribe to all events.


[AutoPR iothub/resource-manager/deviceprovisioningservices/resource-manager] [merge on 4/2] Basic Sku API version update for IotHub

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/2695

(message created by the CI based on PR content)

@lmazuel following up on this ^

Merged in the wrong branch..... Will be back in https://github.com/Azure/azure-sdk-for-python/pull/2324 soon


Fix blend mode and opacity

Note that it doesn't work if the opacity is driven from the graph. However if we sampled the opacity from the monitor node then it wouldn't work for images (as the opacity applied to an image is not stored).

UX feedback for some edge cases:

[ ] If an individual node (rather than a layer) is selected, we want the widgets disabled instead of showing a dash.

[ ] However it would be good for a selected Opacity or Blend Mode node to each show its value in its respective widget.

[ ] The backwards probing logic should stop when it reaches another layer, rather than following it up that layer's node chain


Fix critical: update solid-js

Description
Users cannot deploy and use, due to an error on export solid-js, this fixes.
Linked Issues
This solves the great sum of issues regarding
https://github.com/anse-app/anse/issues/117
https://github.com/anse-app/anse/issues/105
Additional context
Fix critical: update solid-js, this did not allow none to deploy or even use it!
Special thanks
To the creators and maintainers! We are the heart of Github opensource projects! Merry Christmas and a Happy New Year 🎉🎄

@ddiu8081, @yzh990918, @eliassebastian Thanks for your time and dedication, here is my small contribution to houndreds of users at my university trying to deploy this amazing tool!


Icon Request: Crop

Name: Crop
Use Case: Used for photo editing.
(I think we all know what a crop icon looks like 😉)

Hey @colebemis, how are you?
I can help with this one!
Can we contribute or should I wait to you finish the 'design guidelines and contribution instructions.'?

@thiagotomasi I have a very specific process for adding icons that I haven't documented yet. Because of this, at the moment we are not accepting pull requests containing icons but that doesn't mean you can't contribute. The best way to contribute an icon is to create an issue with a screenshot and link to an SVG of your icon. If I use your design, I'll document it and make sure you get credit 👍

@colebemis, awesome!!!
Thanks for your quick reply!!! 😄

Hey @colebemis, here it is!!!
crop-icon.zip
I made 2 options, following the same style, round border, strokes, and size of the other ones,
you can choose which one you want, my choice would be the 'crop.svg' because it is more simple and closer to the others.
crop.svg

crop-2.svg

And please, if you want any change or fix, just let me know.


fix: jit-times - Timestamp arithmetic + add warnings for negative times

Plus and minus operators for Timestamp type can produce invalid Timestamps. All of Timestamp parts should be either (0 or positive) or (0 or negative), but statements like result.milliseconds-- don't take this value into account in the next "if".

The reporting there is broken - the tool parses methods.txt in assumption it is produced by a single-threaded app, and it seems that it's not the case in reality. That's why many "self" timings are negative.

I truncate all negative self-times to zero while calculating total.

@dotnet-policy-service agree

/azp run


No webcam detected

i used the code below and when executed it does not detect any webcam i am using Mac OS and java version 15
Webcam  webcam -= Webcam.getDefault();
if (webcam != null){
System.out.println("Webcam :"+webcam.getName());
}
else
{
System.out.println("No webcam detected");
}

If this is indeed the code you're using, there is an error with your variable assignment.
[code]Webcam webcam -= Webcam.getDefault()[/code]
should be ...
[code]Webcam webcam = Webcam.getDefault()[/code]

@djovla please let us know if the fix proposed by @shinobisoft resolved your issue.

I have the same thing. It should detect 3 webcams but no webcam is detected.
I don't have the -=.
OSX 10.15.7

Same issue.  No cams found.
System.out.println("Webcams: "+Webcam.getDiscoveryService().getWebcams(10, TimeUnit.SECONDS).size() );
Webcams: 0

Same issue but Im also getting an error regarding SLF4J (Maybe because Im using a Mac with M1 Chip)
Mac OS Big Sur 11.6
    Webcam webcam = Webcam.getDefault();

    System.out.print("Webcam is ---> ");
    System.out.println(webcam);

SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder".
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Webcam is ---> null
I guess I'll try using my Windows machine to check if it works there. (Good luck my app has to work in windows and im not building something for MAC)

Hi, please try this driver, it should work with recent MacOS versions https://github.com/eduramiba/webcam-capture-driver-native


[Bug] Can't implement themes even if images are in themes folder

This is insane. For some reason, BetterDiscord doesn't detect any images I put in its themes folder. I have googled images that are good for themes. When I drag and drop them into my themes folder, BetterDiscord doesn't detect these files.

You're likely on the wrong repo, you should be using Bandaged BD if you're not.
Images are not themes. Themes require CSS; images alone do not work as themes. You can find themes on the BetterDiscord Discord server. For info on installing themes you can refer to this guide.

Please read the above. @OPAmqericaHPE


language filter for descriptions.

Is your feature request related to a problem? Please describe.
As the pics shared in this issue : https://github.com/CircuitVerse/CircuitVerse/issues/561
many users on the platform use profanity which shoud be restricted.
Describe the solution you'd like
I want to restrict the use of profanity in at least descriptions.
Describe alternatives you've considered
We can use https://github.com/chrisvfritz/language_filter
Additional context
Ultimately It's my opinion and is totally up for discussion. Please let me know your thoughts on whether the description needs a profanity filter.
Are you working on this? (Yes/No)
GCI

Agree with having a profanity filter for public projects, comments etc

I think this should be a task


fix the dependency injection

Hello,
This is my first contribution to this awesome project. This pull request is a continuation of the work started on the ActorSystem #497 to get rid of the static objects.
I fixed the tests. I removed the need of passing the system in the Props. The DefaultSpawner method is back in the Props class. This requires me to change the delegates (Spawner and Sender) to pass the ActorSystem.
In the TestKit, because there is an inplicit operator, I added a little static class to create an static ActorSystem instance wich is shared between the TestKitBase classes.
Description

Purpose
This pull request is a:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist

[X] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


xml driver needs paging

The xml driver currently only harvests the first page of metadata. Shahre Farang is a collection with multiple pages e.g. https://shahrefarang.com/en/feed/?paged=2
We need to make paging work in a flexible way so that we can use it on any kind of xml data with paging or we need to design specific drivers for rss_xml, atom_xml, etc.

Since they are using Atom for linking, it would be great if they could use paging by adding a <link rel=next ..> like:
<atom:link href="https://shahrefarang.com/en/feed/?paged=2" rel="next" type="application/rss+xml" />

Then our XML driver could follow it's nose to the next URL if paging is on? Otherwise I guess we could add some URL pattern to the intake configuration.


ci: require Tui prefix for type alias

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Refactoring
[x] Build or CI related changes
[ ] Tests related changes
[ ] Documentation content changes

@waterplea please, review me


luigi.is-cool.dev

Requirements

[x] You have completed your website.
Site is up at luigicastro.pro, since I am using traefik the main site cannot be reached unless fqdn is used.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] Your website is not hosting on the following due to SSL issues: Vercel, Netlify

Description
I would like to migrate my current domain to this request. I am planning on using it for my online presence/portfolio and to generate certs for LE
Link to Website

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

"I have read the CLA Document and I hereby sign the CLA"

I have read the CLA Document and I hereby sign the CLA


Connection between num_parallel_samples and make_evaluation_predictions?

I notice that when running make_evaluation_predictions on a multi-core system, only one core is active. I'd like to parallelize this step (if it's not entirely done on the GPU with ctx='gpu'), and notice the argument num_parallel_samples for an estimator object. I'm not sure of the connection between num_parallel_samples and make_evaluation_predictions (if there is any).
Is it possible to go into more detail on what the argument num_parallel_samples actually does? And how it might be possible to speed up make_evaluation_predictions with the argument, or other methods?

@lotsoffood the num_parallel_samples argument decides how many sample paths the network used by the resulting predictor will produce at once. However, the total number of sample paths that the predictor uses to make forecasts can be set to a completely different value when invoking predict, in which case an appropriate number of “batches” of sample paths will be generated.
So I guess the “parallel” here may be a bit misleading: it just indicates how many sample paths at once the network draws.

Also, please note #301 for a related discussion and potential upcoming API changes.

Yes, In principle these samples should be parallelized by mxnet. Not sure why this is not giving higher cpu utilization. In any case, you can wrap your predictor in a ‘ParallelizedPredictor’, which should use all cores if you predict many series.


Fixes for svd with __array_function__

Fixes a couple of small issues that were introduced with __array_function__-only cases in the works of #6616 and #6623 .

Failure seems completely unrelated to this PR, any ideas about the origin for the error?

Agreed that the failure looks unrelated. I haven't seen that failure before and I wasn't able to reproduce it locally.

Oh, I take that back. When repeating the test 500 times I did eventually get a failure. I didn't have --pdb turned on though so it's gone.
Anyway, I'll take a look at it separately.

Thanks @TomAugspurger for checking and merging!


Add ability to save and load layouts

I have been looking for an opportunity to integrate this into my project. It appears currently, layouts have be generated at runtime from C++, it would be really great to be able to save layouts and load them as an asset.

I'm not sure how would that work (I'm not saying it's impossible), as layouts work with pointers from C++, thus it would need to somehow serialize.
For what is worth, the layout system heavily borrows layouts from wxWidgets, and the best way (IMHO) to operate with it is to use wxFormBuilder to generate the Widgets and layouts inside the editor, and the editor generates a piece of C++ (or XRC file) to generate the same widgets with that layout in your code.
For more advanced uses (e.g. where the UI is procedurally genrated), I still use wxFormBuilder as a template to cut and paste the original code into the real one.
As for Colibri, I still use wxFormBuilder to sketch the layouts and then I finally type the Colibri layout code by hand. Since the behavior is almost 1:1 (but not exactly) it is a very efficient method to quickly create new UIs.

Ok, thanks for the info.


Fix: fixes a broken link and some missing styling in the main docs

Which issue does this PR close?

Closes #1012 .
Rationale for this change

Makes the docs somewhat clearer and easier to use.
What changes are included in this PR?

Are there any user-facing changes?

Test failure on https://github.com/apache/arrow-rs/runs/4474279550?check_suite_focus=true  looks like a right nightly bug


Domains under 'Setup'.

In the admin center, go to the Settings > Domains page.
should be
In the admin center, go to the Setup > Domains page.

at least that's in my case...

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

ID: f087dd43-183d-08bb-37d7-d893bede6efb
Version Independent ID: be429837-0300-3de8-6c7d-2da2fabc2c17
Content: Update DNS records to keep your website with your current hosting provider - Microsoft 365 admin
Content Source: microsoft-365/admin/dns/update-dns-records-to-retain-current-hosting-provider.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@nchokoev - Thank you for submitting feedback.
From our understanding, the issue has been resolved based on this merged commit https://github.com/MicrosoftDocs/microsoft-365-docs/commit/dc3a6dbb393b68828addf13012ddc3763109d6ef.
Thank you for your contribution to make the docs better! Much appreciated!


build: pass down the cgroup manager to buildah

Pass down the cgroup manager to use to buildah.
Closes: https://github.com/containers/libpod/issues/3938
Signed-off-by: Giuseppe Scrivano<EMAIL_ADDRESS>

/test images

LGTM
although I don't know why the tests are failing.
/retest

Build failures in podman-remote

Changes LGTM otherwise though

Build failures in podman-remote

I am not sure what is the proper fix for it, I've pushed a new version that at least builds now

/lgtm


Unable to Delete Token

The first token I created, I entered an incorrect value. I now have both the correct and incorrect token. When I right click the first token and select "delete", nothing happens. I have been able to create and delete other tokens.
Any suggestions? Is there a way to delete the token from a preference file?
Thanks!

Also, the token is dated, Dec 31, 1969

Sorry for delay. Could you upload some more details such as a screen recording GIF or a set of screenshots to help me reproduce this bug? Thank you very much.

After clicking the "Delete" nothing changes.

Okay thank you.

On Apr 17, 2016, at 06:33, nothing123456<EMAIL_ADDRESS>wrote:
After clicking the "Delete" nothing changes.
—
You are receiving this because you were assigned.
Reply to this email directly or view it on GitHub https://github.com/TongKuo/Authenticator/issues/8#issuecomment-210919488

Thank you!

Seeing the same behaviour with a token that has date 1 Jan 1970

Oh i' so sorry guys, i'll fix this problem asap. The bug fixes version is coming soon.
@anthonygreen all of keys were stored at ~/Library/Containers/home.bedroom.TongKuo.Authenticator/Data/Library/Application Support/Authenticator/Vaults/Defaults. BTW, it's an AES-256 encrypted vault file.

I Have the same problem! Please help. When will be the next Update? BEst, Marco

@officerMa May you please provide me with either (a) the combination of Secret Key/Account Name/Service Name or (b) the QR code you scanned which has the bug emerge? I suppose it'd be far easier to reproduce and then locate this bug since I've never encountered the same problem before. For the security consideration, before you give me such secret information, be aware that invoke them in your website's account settings.
BYW: Here's the roadmap of v2.0.

Hello Torin,
see the screenshot. Maybe can you tell me how to reset the app for a new beginning?
Because im unable to delete.
The Date is always on 01.01.1970
Thanks, Marco
Marco Wessendorf
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> sent from a small mobile device designed in Cupertino

Am 27.03.2017 um 14:12 schrieb Torin Kwok<EMAIL_ADDRESS>@officerMa https://github.com/officerMa May you please provide me with either (a) the combination of Secret Key/Account Name/Service Name or (b) the QR code you scanned which has the bug emerge? I suppose it'd be far easier to reproduce and then locate this bug since I've never encountered the same problem before. For the security consideration, before you give me such secret information, be aware that invoke them in your website's account settings.
BYW: Here https://github.com/TorinKwok/Authenticator/issues/12's the roadmap of v2.0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/TorinKwok/Authenticator/issues/8#issuecomment-289435163, or mute the thread https://github.com/notifications/unsubscribe-auth/AZeblTmIFzaXeikCXOhHN5oMaFSfYdB5ks5rp6eZgaJpZM4IDr97.

Hi @officerMa,
All of keys were stored at ~/Library/Containers/home.bedroom.TongKuo.Authenticator/Data/Library/Application Support/Authenticator/Vaults/Defaults.

Open the Finder and press Command + Shift + G
Copy and paste the path above. Hit Enter
Move whatever you see into Trash
Restart the app

You'll see the following:

Thank! That worked.
I have put the Secret Key manually. Now it works.
Best ,
Marco
Marco Wessendorf
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS> sent from a small mobile device designed in Cupertino

Am 04.04.2017 um 07:25 schrieb Torin Kwok<EMAIL_ADDRESS>Hi @officerMa https://github.com/officerMa,
All of keys were stored at ~/Library/Containers/home.bedroom.TongKuo.Authenticator/Data/Library/Application Support/Authenticator/Vaults/Defaults.
Open the Finder and press Command + Shift + G
Copy and paste the path above. Hit Enter
Move whatever you see into Trash
Restart the app
You'll see the following:
https://cloud.githubusercontent.com/assets/4256649/24642474/1fce5076-193a-11e7-95b6-29dd4126f68b.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/TorinKwok/Authenticator/issues/8#issuecomment-291396791, or mute the thread https://github.com/notifications/unsubscribe-auth/AZeblZgn3kTgDYX8zSZtOnJzr9gzpmE9ks5rsdRigaJpZM4IDr97.

@officerMa Glad to hear 🍺.

Still seeing this issue. I have the latest version on the mac App Store. If any code is entered via QR code, then it is incorrect and has the date Dec 31, 1969.


Static NAT, Port forwarding, Firewall support

Now vagrant-cloudstack can set a port forwarding rule when the virtual machine is deployed.
But there are some limitations.

only 1 rule can be set
unable to use in VPC because openfirewall is always true
can't specify source CIDRs

This patch enables to create Static NAT, Port forwarding, Firewall rules  like Security Groups.
It also delete rules where the instance is destroyed.

I would like to allow to specify port forwarding IP address directly instead of uuid.
Can you take a look?

Merged into https://github.com/schubergphilis/vagrant-cloudstack

Thanks!

Since we ended up maintaining the original repo, I'll merge this one here too.


Implement Cloud Functions API

https://developers.google.com/apis-explorer/#p/cloudfunctions/v1beta1/
(Internal & whitelist only right now, just planting the flag that we need/want it )

I've created API Implementation Tracker which I've included this request in. Feel free to subscribe to that issue to stay updated!


Suggestion: state what scripts do

Scripts can delete files that users want to keep, and the user may want to know exactly what the script is doing/do it themselves. So rather than leaving the user to open the script up in a text editor and work it out from that, the guide should simply list what the script does.

I don't really feel like this is necessary. Users don't always need to know the specifics of what a program/script is doing in the background as long as it works, and those who care about all of the specific details can just read it as you have pointed out is possible.


add flag to tarpaulin coverage command

context:
https://github.com/xd009642/tarpaulin/issues/756

unresolved atm but tackling in other branches


🐞 dagger develop should fail if no dagger.json is found

What is the issue?
Some users have reported some issues while getting started with dagger about dagger develop not apparently working as expected. In most of those cases we found out that they didn't run dagger init before calling dagger develop which yielded to some unexpected results.
@shykes suggested (https://discord.com/channels/707636530424053791/1215438073694003300/1215471630630002818)  making dagger develop to fail if no dagger.json is found  which seems reasonable
ref: https://discord.com/channels/707636530424053791/1215438073694003300/1215438073694003300
ref: https://discord.com/channels/707636530424053791/1215318377329983519
Dagger version
v0.10.1
Steps to reproduce
No response
Log output
No response

It seems like this was added in https://github.com/dagger/dagger/pull/6386 a while ago (in v0.9.8), so asking for more info.
In an empty directory, if I do dagger develop --sdk=go:
Error: failed to get configured module: no dagger.json found in directory . or any parents up to git root

In the first discord ref, it seems the user did have a dagger.json, but doesn't know how it got there.
In the second discord ref, it seems the user used develop (albeit without init first) but didn't do what they thought so they created the .py file manually and go confused on dagger call.

There's some polish planned for init/develop so I want to take a general look in the develop flags in relation to the usage of those commands, but that's a matter for another issue.
Polish
For example:

https://github.com/dagger/dagger/issues/7065
https://github.com/dagger/dagger/issues/6666

There's been discussions for not not using --sdk in init, but that would require two commands dagger init && dagger develop --sdk=... when you want to generate code for authoring so it's been decided to leave the init convenience so far.
We could also have develop do the init automatically if there's no dagger.json, but that can also be confusing. In that case you'd use dagger develop --sdk to author, or dagger init just for dependencies.
It would be less confusing to split, but frequent users like the init convenience.


plan-all hides errors

Hey,
When using plan-all, if one of the internal modules has a parsing/validation error it will be hidden and not shown by plan-all.
Example:
For this module:
resource "aws_route53_zone" "main" {
  name   = "${var.domain_name}"
  vpc_id = "${aws_vpc.main.id}"
}

resource "aws_route53_zone" "main" {
  name   = "${var.domain_name}"
  vpc_id = "${aws_vpc.main.id}"
}

There is an error because the two resources have the same name, but running plan-all won't show it. (Running plan will).

What does the command do in that case? Does it show any messages? Could you paste some example log output?

The plan command shows this:
1 error(s) occurred:

* module root: 1 error(s) occurred:

* aws_route53_zone.main: resource repeated multiple times

plan-all shows this:
...
...
[terragrunt] [module/vpc] 2017/07/06 14:44:44 Running command: terraform plan -var-file=/env/test/vpc/../environment.tfvars
[terragrunt] [module/vpc] 2017/07/06 14:44:54 Module /env/test/vpc has finished with an error: exit status 1
[terragrunt] 2017/07/06 14:44:54 Encountered the following errors:
exit status 1

And the error doesn't appear anywhere in the log output above?

Nope.
There is only 1 line after the plan command and I pasted it

I believe this was fixed by #277.


node crashes when image comparison difference is larger than the tolerance threshold.

Hi There,
I ran below error when I do visual testing via resemblehelper. Can anyone help to take a look? Thanks a lot.
The helper works well when the difference is under the tolerance indicator, only crashes when the image comparison difference is bigger than the tolerance.

The project dependencies as below :

"devDependencies": {
"mocha": "^5.2.0",
"mocha-multi": "^1.0.1",
"puppeteer": "^1.15.0"
},
"dependencies": {
"codeceptjs": "^2.6.5",
"codeceptjs-resemblehelper": "1.9.0",
"mocha-junit-reporter": "^1.18.0",
"mochawesome": "^3.1.1"
}

And node version is : 8.11.3

@gtycherry any specific reason why you use node v8?

Hi @puneet0191
I ran into some other issue when I use V10:

I ran into above issue even without image comparison. My colleague also have the same V10 codeceptjs issue and V8 resemblehelper issue.

@gtycherry please try to use codeceptjs 1.9.0, I think you use the latest 1.9.1 which could be having this issue, and the issue with node 10, is because you have some old files in bcrypt, try to do it on a clean node_module folder, https://stackoverflow.com/questions/46384591/node-was-compiled-against-a-different-node-js-version-using-node-module-versio

Hi @puneet0191
Thanks for recommendation. But seems I was without luck.
I think you meant resemble helper version should be 1.9.0. I tried both 1.9.1 and 1.9.0 Both didn't work.
For the V10 issue, per your tips, the original error disappeared, and all the test ran successfully. But there was a new issue come out when generating test report.

@gtycherry I am not sure how this is related to the helper, please check if you get this error without using the helper?

@puneet0191  I think without the helper ,codecepjs itself can't do image comparison.
I refered to the doc here https://codecept.io/visual/#using-resemble-helper

@gtycherry I am just trying to understand, if the problem is because of helper or something else in generating the report.

@puneet0191
seems i confused you here, the reporter issue is for V10 testing run(this is to reply to you about why i must use V8). But my original priority one issue is nodejs crashes when visual testing found difference bigger than the specified threshold.
2 different problem, maybe we can ignore the V10 issue at the moment. But only focus on the crash issue.

@gtycherry are you still able to reproduce this? I haven't encountered any reports, please feeel free to reopen in case this still is a problem for you, thanks a lot for reporting.


[BEAM-10232] Restrict rsa package version.

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @aaltay @udim

Thanks to a speedy patch of google-auth, this is no longer necessary.


Suggestion: Delete reports/datasets from the workspace that are not in the repository

Not an issue, but a suggestion for an additional action:
To keep the workspace clean, we have a PowerShell step that compares the workspace reports and datasets with the PBIX files within the repository. If it is in the workspace, but not in the repos then we will delete both report and dataset from the workspace. Otherwise a report or dataset will never disappear because DevOps is often the only way to manage the workspace content.

Hi Joost, if you could share the code I can maybe incorporate it as an additional action in the extension when I have time.

sampe code is in your mailbox

I have added an additional action to remove reports. It is currently in beta, would you mind testing it?

No problem, I'm curious about what you have created. Let me know how to test it

If you share your organization url with me I can make it accessible.

If you share your organization url with me I can make it accessible.

see your email

@ssisjoost , @maikvandergaag
But there is a minor drawback with this approach, once you delete the report of the dataset, then you cant download the .pbix file from power bi service, but then again if you are using version control then no big deal, just pointing out an observation from our testing.

Its moved to production

Can you please share the pipeline to achieve this
currently im using below but, its difficult to send all the deleted name, it should find form the repo

task: maikvandergaag.maikvandergaag-power-bi-actions.PowerBIActions.PowerBIActions@5
displayName: 'Power BI Action: DeleteReport'
condition: eq(variables['Demo_deleted'], True)
inputs:
PowerBIServiceEndpoint: 'powerbi-actions'
Action: DeleteReport
WorkspaceName: 'Demo-workspace'
ReportName: 'Demofile-1'

What would you like to achieve deleting all the reports? Then you would have to write a powershell script to retrieve all reports save that in a variable and use that for the deleting.

In repo, im deleting the few files
eg: out of 4 pbix files, i have deleted 2 files in repo , so those two should be removed from PowerBI workspace as well.
im using the below script to get the diff which files got deleted
Write-Host "Source file '$sourceFile' does not exist. Marked for deletion..."
$destsubfolder = Split-Path $sourceFile
$deletedFolderPath = Join-Path $deletedfolder $destsubfolder
$dummyFileName = Split-Path $sourceFile -Leaf
$dummyFilePath = Join-Path $deletedFolderPath $dummyFileName
if (-not (Test-Path -Path $deletedFolderPath)) {
$null = New-Item -ItemType Directory -Path $deletedFolderPath
}
$null = New-Item -ItemType File -Path $dummyFilePath
}
but if i want to send these in the azure devops pipline, i cannot hardcode thevalues in the repot names, how do i pass dynamically with array or loop..?
task: maikvandergaag.maikvandergaag-power-bi-actions.PowerBIActions.PowerBIActions@5
displayName: 'Power BI Action: DeleteReport'
condition: eq(variables['Demo_deleted'], True)
inputs:
PowerBIServiceEndpoint: 'powerbi-actions'
Action: DeleteReport
WorkspaceName: 'Demo-workspace'
ReportName: 'Demofile-1'

What was the solution provided to "[ssisjoost]" to delete reports from the powerBi workspace that are not in repository..?


no validation for missing redirect uri with openid scope

There is no validation for a missing redirect uri when the authorization request has an "openid" scope

close with #420


Deal with horizontally overflowing cell output in GitHub Pages

I found it necessary to handle overflow behaviour not just for links but for output_html in general. Perhaps the default for customstyles.css could be like so:
.output_html {
    overflow-y: scroll;
}

Otherwise, the static github.io page can trivially break, also wrt. the sidebar menu.

Doesn't sound like it would break anything to me. Would you mind suggesting a PR with that change?

@mikhas can you provide a repo or example where the overflow occurs in an undesireable way?  That way will give me something I can try to fix rather than trying to fix a problem in the abstract.  Thanks for your help!


Feature Support: Generic Query, How to get schema and data while query?

Hi,
Clickhouse is an OLAP database and I want to make a generic query engine where users can enter any query SQL and return query results, including returning schema and data, and then the application layer displays the data. When querying, the user does not have to specify the data type.
The query result is a two-dimensional data, where one array is the column name and the other data is also a two-dimensional data, e.g., but the schema is not fixed
{
	"schema": ["id","name", "age","birthday"],
	"data": [
		["20"， ”james1, "18", "2000-01-01"],
		["21"， ”james2, "19", "2000-01-01"],
		["22"， ”james2, "10", "2000-01-01"],
		["23"， ”james4, "30", "2000-01-01"],
		["24"， ”james5, "26", "2000-01-01"]
		...
	]
}

How to do that? thanks.

Hi @loyd
Can you answer this question?
thanks

Sorry for the late response, there were health problems.
Relates to https://github.com/loyd/clickhouse.rs/issues/44, but for SELECTs instead of INSERTs. This crate doesn't provide support for dynamic columns and whether it will be supported is questionable (because the crate is based on types). You can use https://github.com/suharev7/clickhouse-rs instead.

Thanks for your response。
I used the https://github.com/suharev7/clickhouse-rs  before， but that do support decimail38+，and amlost stop update now。

Sadly to hear it.
If you need only SELECTs, it theoretically can be done now, but it's a little tricky:

Need to request a schema
Need to write a struct over Vec<Value> (value should be from serde_value, serde_json or valuable crates).
The struct must implement Deserialize by calling methods according to the fetched schema.

Anyway, after https://github.com/loyd/clickhouse.rs/issues/10 the codec will implement deserialize_any and it will be possible just to use some special Value (dedicated to SQL types). At first sight, implementing deserialize_any is not so hard, maybe several evenings, I need to look it closer.


refactor(multiple): run typed forms migration

Runs the typed forms migration over all our code.

Thanks! Would someone kick off a g3 TGP, or let me know how to do so for components? (@crisbeto suggested we need one, since a component is affected)


Added timeout to port server

Second attempt at fixing #7145
I've confirmed that establishing a tcp connection without sending an http request will block the port_server from servicing requests.  This adds a socket timeout.

test this please

LGTM


Add xray policy and watch resources

I have read the CLA Document and I hereby sign the CLA

@ryndaniels - we are aware of the slackbot issue.

recheck cla

recheckcla

@ryndaniels - looks pretty good. Can you supply a simple TF I can use to test locally? I ran all your tests and everything passes with flying colors.

@ryndaniels - looks pretty good. Can you supply a simple TF I can use to test locally? I ran all your tests and everything passes with flying colors.

Sure, I guess I'd try something like:
resource "xray_policy" "test" {
	name  = "test-policy-name"
	description = "test policy description"
	type = "security"

	rules {
		name = "rule-name"
		priority = 1
		criteria {
			min_severity = "High"
		}
		actions {
			block_download {
				unscanned = true
				active = true
			}
		}
	}
}

resource "xray_watch" "test" {
	name = "test-watch-name"
	description = "test watch description"
	resources {
		type = "all-repos"
		name = "All Repositories"
	}
	assigned_policies {
		name = xray_policy.test.name
		type = "security"
	}
	watch_recipients =<EMAIL_ADDRESS>}

which should get both watches and policies tested.

@ryndaniels Here is the tf I put together and this is what I got:
# Required for Terraform 0.13 and up (https://www.terraform.io/upgrade-guides/0-13.html)
terraform {
  required_providers {
    artifactory = {
      source  = "registry.terraform.io/jfrog/artifactory"
      version = "2.2.6-alpha"
    }
  }
}
provider "artifactory" {
  url = "http://localhost:8082/artifactory"
  username = "admin"
  password = "password"
}

resource "xray_policy" "test" {
  name  = "test-policy-name"
  description = "test policy description"
  type = "security"

  rules {
    name = "rule-name"
    priority = 1
    criteria {
      min_severity = "High"
    }
    actions {
      block_download {
        unscanned = true
        active = true
      }
    }
  }
}

resource "xray_watch" "test" {
  name = "test-watch-name"
  description = "test watch description"
  resources {
    type = "all-repos"
    name = "All Repositories"
  }
  assigned_policies {
    name = xray_policy.test.name
    type = "security"
  }
  watch_recipients =<EMAIL_ADDRESS>}

Results:
christianb@christianb-mac terraform-provider-artifactory % terraform init

Initializing the backend...

Initializing provider plugins...
- Using previously-installed jfrog/artifactory v2.2.6-alpha
- Using previously-installed hashicorp/random v3.0.1
- Finding latest version of hashicorp/xray...

Error: Failed to install provider

Error while installing hashicorp/xray: provider registry registry.terraform.io
does not have a provider named registry.terraform.io/hashicorp/xray

christianb@christianb-mac terraform-provider-artifactory %

Am I missing something?

@chb0github - I'm not entirely sure why it's trying to pull down a hashicorp/xray provider. Could be that it expects all resources in the artifactory provider to begin with artifactory_. Assuming you have the 2.2.6-alpha version locally (since I don't see it in the registry), you could try changing that version so the new resources start with resourceArtifactory instead of resourceXray and see if that works. It may be that this is one of the limitations of trying to put multiple categories of resources into one provider, but beyond that, the details of how terraform init chooses what to look for internally isn't something I'm terribly familiar with.

@ryndaniels - I had to make some changes  to accomodate the problem we saw earlier.
see this SoF question.

I added a testable terraform file - i've been working on it a bit.
I realized that you had the URL as "http://host/xray" and that worked fine for xray but not for artifactory. I made changes to accomodate them both.
I merged in some changes from origin/master and did some refactor that was hurting my eyes.

Overall, very little of your code needed to change. Please comment as needed.

@chb0github - the changes you made look good from my end - glad you were able to get this working to work with both the artifactory and the xray bits 👍

@ryndaniels anything to add on this ?

Hi @chb0github and @ryndaniels, I'm happy to implement something, but I need some direction on the schema changes.
One option would be to add 2 additional optional properties.
For example:
resource "xray_watch" "test" {
	name  = "%s"
	description = "%s"
	resources {
		type = "repository"
		name = "%s"
		bin_mgr_id = "%s"
		filters {
			type = "package-type"
			value = "%s"
		}
                property_filter { # 1 or more of these can be defined
                  key = "someKey"  # required field
                  value = "someValue" # required field
                }
                path_filter { # 0 or 1 of these can be defined.
                  exclude_patterns = ["some/path"] # default value of []
                  include_patterns = ["some/path"] # default value of []
                }
	}
	assigned_policies {
		name = xray_policy.test.name
		type = "security"
	}
}

Cheers

Give me some time to consider. I will get you an answer tomorrow


Property 'getVideoData' does not exist on type 'Player'

Hi, for some reason I can call methods like playVideo() and seekTo(), but getVideoData() doesn't work on my local machine. Calling the method on the Plunker demo works fine.
I have @types/youtube installed if that's relevant.

Sorry, it appears getVideoData() is depreciated.
The new way to get a video's name and author is to parse the string from http://www.youtube.com/get_video_info?video_id={id}, if anyone was having the same problem.

you can use preview version of @types/youtube: 0.0.28  it still working. But in the latest at this moment there is not type for getVideoData(). hope this help.


⚠ Bofrost.de has degraded performance

In 30ec460, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1586 ms

Resolved: Bofrost.de performance has improved in c3f20fa after 6 minutes.


ok

ok

Hmm, it's not clear what this pull request does. If it is something to consider, please open a new one with a more comprehensive description!


Submit/fix vega lite

Motivation for this change
vega-lite binaries were not able to run, because they could not find the vega dependency. This change fixes this and adds a test to assure future changes will provide running executables.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ x] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Sorry for my ignorance. I've never done a pull request before. Do I have to to anything to apply the suggested changes or are they added to the pull request automatically?

Feel free to squash them, or we could squash all commits on merge

Ok I squashed them. Hope I did it right.


"rake aborted jpicker.js has already been required" at assets:precompile

Hi,
I installed Jpicker as indicated on rails 3.2.13 and when I do a:
rake assets:precompile
I get an error message: "rake aborted jpicker.js has already been required"
What's wrong?
Thank you for your help

Maybe my fork could resolve your issue

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin Airdrop?
If you're interested in holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great opportunity. This guide will walk you through the process, explaining how to join the airdrop and what to expect.

🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop.
Connect your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the airdrop here.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your participation.

Open your connected wallet.
Navigate to the Chainlink $LINK Airdrop contract.
Locate the 'Claim' or equivalent button.
Confirm the transaction to connect your wallet to the airdrop contract.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any questions in the comments below. Let's make this process a breeze for everyone!
Winners: @pearcec, @ElliotVilhelm, @alexwibowo, @kartikay18, @abdlhay, @AinuX, @JUd9in

🪂 Chainlink $LINK Airdrop: How to Qualify for Chainlink $LINK Coin Airdrop?
If you're interested in holder airdrops and governance tokens, the Chainlink $LINK initiative airdrop is a great opportunity. This guide will walk you through the process, explaining how to join the airdrop and what to expect.

🚀 Steps to Secure Your Chainlink $LINK Airdrop:

Connect Your Wallet:

Visit the Chainlink $LINK Airdrop.
Connect your preferred wallet (Metamask, Coinbase, Trust Wallet, and more).

Eligibility Check:

Confirm your eligibility for the airdrop here.

Interact with the Contract:

Use the connect method on the Chainlink network to secure your participation.

Open your connected wallet.
Navigate to the Chainlink $LINK Airdrop contract.
Locate the 'Claim' or equivalent button.
Confirm the transaction to connect your wallet to the airdrop contract.

Engage for Extra Rewards:

Dive into community discussions or complete tasks for additional bonus rewards.

🌈 Bonus Tips for Chainlink $LINK Airdrop Success:

Community Assistance:

Need help? Reach out via Telegram or other social media platforms.

Stay Informed:

Watch for updates on the airdrop process via official channels.

Patience Pays Off:

Airdrop distribution may take some time. Stay calm and keep an eye out for updates.

Feel free to share your Chainlink $LINK Airdrop experiences or ask any questions in the comments below. Let's make this process a breeze for everyone!
Winners: @jwwildebeast, @phamquiluan, @arbeen, @BenQoder, @Nawabmeeravali, @ACollectionOfAtoms, @Takonan


User primary key UUID and wagtail 2.13 comments

I integrated wagtail in a project where the user has a UUID as primary key. This was working fine in 2.12 but in 2.13 we got the new comment system.
'''TypeError: keys must be str, int, float, bool or None, not UUID'''
If you turn of the comment system:
WAGTAILADMIN_COMMENTS_ENABLED = False
Then the problem go's away
Technical details

Wagtail version:2.13

Hi @onno-timmerman - this has been fixed in Wagtail 2.13.1. #7216

yeah I just see it in the release notes :)


Migrated cart data retriving from Firestore

Fix the Retrieving data from firestore.
Description
Previously implementation of the cart database relies on a Realtime Database, It has been Migrated to Firestore. We have to retrieve the data from firestore.
Expected Behavior

Make Sure CURD Operations work smoothly and fine.
Make sure the curd operation changes reflect on the website.

@MAVRICK-1 can you please add labels and assign this to me.

@MAVRICK-1  I had created the PR. I can you please check.


How to deal with doctest formatting issues? (Consider adding pytest-accept)

In this repo we are using doctest to check whether the examples in the documentation still produce the expected output. But sometimes we make small modifications and the output is supposed to change. In that case, we need to update the expected outputs.
One option is to run the example code in your own IDE, and copy it to the docstrings. Another option is to copy the output of the test. However, in both cases, we run into small formatting errors e.g. related to trailing whitespace.
One option could be to consider pytest-accept. Or, we can look into whether we can configure the tests to ignore whitespace issues.

Apparently doctest has a flag which allows for normalizing whitespaces. See;
https://stackoverflow.com/questions/42372351/how-to-insert-trailing-spaces-in-a-doctest-so-that-it-doesnt-fail-even-when-ac
https://docs.python.org/3/library/doctest.html#doctest.NORMALIZE_WHITESPACE

Please bear in mind this answer on SO that advises against doing this. But I think we have a valid usecase, as long as we manually verify that the updated test output indeed corresponds to what we would expect.

A quick note about the solution using # doctest: +NORMALIZE_WHITESPACE. The doctest used in ci/cd is provided by sphinx. It can pass the test (locally, called via make doctest) with # doctest: +NORMALIZE_WHITESPACE and it will not show up in the doc. However, since it still looks quite ugly in the code...Let's look for something elegant first.


Audio sync does not work with latest ffmpeg builds

ffpmeg thrown an unknown command error when trying to splice the trimmed sections together. Tested an old build of ffpmeg and it runs fine

Hmm, could you test on 1.0.0

I tested with several versions available and I was unable to get it to run without downgrading my ffmpeg


Error starting llama3 -  external llama server: Executable not found in $PATH

What is the issue?
Description
When attempting to run the llama3:instruct model using the ollama run command, I encountered an error indicating that the executable ollama_llama_server could not be found in the $PATH.
Steps to Reproduce

Updated via curl -fsSL https://ollama.com/install.sh | sh
Pulled the llama3:instruct model using ollama pull llama3:instruct.
Verified successful pull with ollama list, which showed the model as available.
Attempted to run the model using ollama run llama3:instruct.
Received an error message.

me@me-MS-7C56:~/ollama/models/meta/llama3/7b_instruct$ ollama pull llama3:instruct
pulling manifest
pulling 00e1317cbf74... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████▏ 4.7 GB
pulling 4fa551d4f938... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████▏  12 KB
pulling 8ab4849b038c... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████▏  254 B
pulling c0aac7c7f00d... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████▏  128 B
pulling db46ef36ef0b... 100% ▕████████████████████████████████████████████████████████████████████████████████████████████████▏  483 B
verifying sha256 digest
writing manifest
removing any unused layers
success

me@me-MS-7C56:~/ollama/models/meta/llama3/7b_instruct$ ollama list
NAME                     ID              SIZE      MODIFIED
llama3:instruct          71a106a91016    4.7 GB    49 seconds ago
mistral-7b-pro:latest    27ebf620ae7f    7.7 GB    4 weeks ago

me@me-MS-7C56:~/ollama/models/meta/llama3/7b_instruct$ ollama run llama3:instruct
Error: error starting the external llama server: exec: "ollama_llama_server": executable file not found in $PATH

Environment
Operating System: Ubuntu 22.04
ollama version: 0.1.32
Any ideas how to fix this issue?
OS
Linux
GPU
Nvidia
CPU
AMD
Ollama version
0.1.32

same here. any solution?

Can you share your server log?

@alfredsam-nbfc
fixed by reinstalling via curl -fsSL https://ollama.com/install.sh | sh .

Can you share your server log?

Sure
$ journalctl -u ollama
Mar 15 14:55:03 me-MS-7C56 systemd[1]: Started Ollama Service.
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: Couldn't find '/usr/share/ollama/.ollama/id_ed25519'. Generating new private key.
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: Your new public key is:
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: ssh-ed25519 AAAAC3NzaC1lZDI1NTE5AAAAIKgl9RPeBzmro+/SC+7DM5D+wFklrz21Jj1OHuiTeUGB
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:03.727Z level=INFO source=images.go:806 msg="total blobs: 0"
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:03.727Z level=INFO source=images.go:813 msg="total unused blobs removed: 0"
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:03.727Z level=INFO source=routes.go:1110 msg="Listening on <IP_ADDRESS>:11434 (version 0.1.29)"
Mar 15 14:55:03 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:03.729Z level=INFO source=payload_common.go:112 msg="Extracting dynamic libraries to /tmp/ollama2055899656/runners ..."
Mar 15 14:55:06 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:06.931Z level=INFO source=payload_common.go:139 msg="Dynamic LLM libraries [cpu_avx cuda_v11 cpu_avx2 rocm_v60000 cpu]"
Mar 15 14:55:06 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:06.931Z level=INFO source=gpu.go:77 msg="Detecting GPU type"
Mar 15 14:55:06 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:06.931Z level=INFO source=gpu.go:191 msg="Searching for GPU management library libnvidia-ml.so"
Mar 15 14:55:07 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:07.106Z level=INFO source=gpu.go:237 msg="Discovered GPU libraries: [/usr/lib/x86_64-linux-gnu/libnvidia-ml.so.535.161.07]"
Mar 15 14:55:07 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:07.119Z level=INFO source=gpu.go:82 msg="Nvidia GPU detected"
Mar 15 14:55:07 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:07.119Z level=INFO source=cpu_common.go:11 msg="CPU has AVX2"
Mar 15 14:55:07 me-MS-7C56 ollama[1355023]: time=2024-03-15T14:55:07.124Z level=INFO source=gpu.go:119 msg="CUDA Compute Capability detected: 8.6"
Mar 15 16:04:31 me-MS-7C56 ollama[1355023]: [GIN] 2024/03/15 - 16:04:31 | 200 |    9.220005ms |       <IP_ADDRESS> | HEAD     "/"
Mar 15 16:04:31 me-MS-7C56 ollama[1355023]: [GIN] 2024/03/15 - 16:04:31 | 200 |  107.027478ms |       <IP_ADDRESS> | GET      "/api/tags"
Mar 15 16:33:40 me-MS-7C56 ollama[1355023]: [GIN] 2024/03/15 - 16:33:40 | 200 |       37.21µs |       <IP_ADDRESS> | HEAD     "/"
Mar 15 16:33:41 me-MS-7C56 ollama[1355023]: [GIN] 2024/03/15 - 16:33:41 | 200 |   60.647614ms |       <IP_ADDRESS> | GET      "/api/tags"
'''

as I mentioned to @dhiltgen, reinstalling fixed the issue.

Let me know if you need anything further.

How can I fix the same error in Windows?

Facing similar problem on Mac m3
Error: error starting the external llama server: fork/exec /var/folders/64/w9tycd0x3mv243jr9zb1glg00000gn/T/ollama97142749/runners/metal/ollama_llama_server: no such file or directory

Facing similar problem on Mac m3 Error: error starting the external llama server: fork/exec /var/folders/64/w9tycd0x3mv243jr9zb1glg00000gn/T/ollama97142749/runners/metal/ollama_llama_server: no such file or directory

For me a reinstall Ollama

@asif-kaleem I think that's a different problem.  The MacOS tmp cleaner removed the file out from underneath us.  It should self-correct on the next model load.  Are you seeing it get stuck in this state and no longer work?

Quick update - @asif-kaleem I have a PR up that should resolve the problem you noticed - #3846

I believe this should be resolved in the next release with #3846 and #3850 merged.

Same here on Windows 11 Pro
time=2024-04-28T20:50:09.953+03:00 level=ERROR source=server.go:285 msg="unable to load any llama server" error="error starting the external llama server: exec: \"ollama_llama_server.exe\": executable file not found in %PATH% "

The pre-release for 0.1.33 is available now, which should resolve these exe missing problems on windows.

in macOS Sonoma, reinstall fix the issue

Same here. It happened on Debian.
Error: error starting the external llama server: exec: "ollama_llama_server": executable file not found in $PATH
The workaround is: sudo systemctl restart ollama

@bsdnet if you upgrade to 0.1.33 this defect should be resolved and no longer require restarting the service to work around it.  If you still see this persisting, please let us know.

Thank you.  Let me give it a try and get back to you if I still see the same issue

i have the same problem with mac
reinstall doesn't change anything, please reopen

@didlawowo can you share your server log?

Same issue, MacOS
Error: error starting the external llama server: fork/exec /var/folders/hs/ ... /T/ollama501987923/runners/metal/ollama_llama_server: no such file or directory

On a Mac (running Sonoma), I opened activity monitor and closed all Ollama related processes (there were 3). I then started the model up again and it worked fine. (No reinstall needed.)

@bsdnet if you upgrade to 0.1.33 this defect should be resolved and no longer require restarting the service to work around it. If you still see this persisting, please let us know.

I ran into this issue with 0.1.34 on Linux (RHEL 8) and a 'systemctl reset ollama' fixed it. Let me know if creating an issue or sharing logs is helpful; the server is on an air-gapped system so a little more effort than throwing them in this comment.

@JoshInLisbon Thanks! Closing Ollama in the menu bar did help.

@Bangkokian what version of Ollama were you running?  This should have been resolved in 0.1.33 and newer, but maybe there's some other corner case I missed.
@jshbmllr can you share your server log for the failure?
Restarting the app or quitting the menu item shouldn't be necessary, so if folks are still having to do that to work around this then there's still a bug here somewhere I need to fix.

@dhiltgen Disregard! This was indeed a version issue; I was on 0.1.32. I'm transitioning from a metal install to a container deploy and got my numbers crossed. Sorry!

ollama run llama3
Error: error starting the external llama server: fork/exec /var/folders/d1/6g8w22ms0874r1qjd314m8yc0000gn/T/ollama3689840524/runners/metal/ollama_llama_server: no such file or directory
This happened my MAC OS, when giving command: ollama run llama3.
Env is python 3.11 which is just settled for LLMs locally.

@T8354HQ which version are you running?  This should be fixed in 0.1.33.

@dhiltgen I can confirm that upgrading 0.1.32 -> 0.1.37 resolved the problem on macOS Sonoma 14.5

I'm going to go ahead and re-close this, as it does in fact look like it's resolved.


[WW-5344] Un-deprecates Sitemesh plugin and upgrades Sitmesh to ver 2.5.0

Closes WW-5344

Never used this plugin myself - doesn't seem to have any unit test coverage but presumably covered by the showcase app acceptance tests?

Me neither, but looks like it's used by the Showcase App and we use the app to perform integration tests (part of the app)


Contracts update

There are currently 2 FIXME items to address
FIXME 1:

after including the liquidationsStorageNode as an argument for makeVaultManagerKitInternal an error is triggered by missing that attribute
this change was implemented on commit b4c8a21a607249b8cc633f47652351869cf89442
vaultDirector

FIXME 2:

the type of auctionEndTime {TimestampRecord | undefined} needs to match the type of endTime {Timestamp}
this change was implemented on commit 21bb9ca37a76f30d010921f47540ea7ce4a081ab
vaultManager

Changes made to original code

When the makeVaultManagerKit function of vaultManager is executed by the vaultDirector, a child node of manager{id} will be created with the key liquidations and used as argument for the makeVaultManagerKitInternal function.

  /**
   * @param {Omit<
   *   Parameters<typeof makeVaultManagerKitInternal>[0],
   *   'metricsStorageNode' | 'liquidationsStorageNode'
   * >} externalParams
   */
  const makeVaultManagerKit = async externalParams => {
    const [metricsStorageNode, liquidationsStorageNode] = await Promise.all([
      E(externalParams.storageNode).makeChildNode('metrics'),
      E(externalParams.storageNode).makeChildNode('liquidations'),
    ]);

    return makeVaultManagerKitInternal({
      ...externalParams,
      metricsStorageNode,
      liquidationsStorageNode,
    });
  };
  return makeVaultManagerKit;

When the liquidateVaults method of vaultManager self facet is executed by the vaultDirector, it will pass as an argument the timestamp that corresponds to the auction start time.

        makeLiquidationWaker() {
        return makeWaker('liquidationWaker', _timestamp => {
          // XXX floating promise
          allManagersDo(vm => vm.liquidateVaults(auctioneer, _timestamp));
        });
      },

...

postAuctionState WiP
The planned structure of postAuctionState was:
Two different approaches are:

dynamic ...
static ...


Internal error merging refs/heads/ during model merging in Papyrus.

Issue and tracking information
Developer's time Estimated effort to fix (hours):
Developer's Actual time spent on fix (hours)
Issue reporter to provide a detailed description of the issue in the space below
While verifying issue 415, we encountered a bug that occurs in both the performance fix and without the performance fix.
When attempting to perform a model merge from one branch onto another, sometimes this error message will appear:

Once this message appears, it is impossible to continue with the model merge.
This issue is reproducible in the modelmergerproducer repo. First, pull that repo in a Papyrus workspace, switch to the "feature_branch" branch and attempt to merge the "conflicting_branch" branch onto "feature_branch" branch. This will always result in the above error being thrown. However, interestingly enough, if you perform the opposite and switch to "conflicting_branch" and attempt to merge the "feature_branch" branch onto the "conflicting_branch" branch, then the model merge continues as normal with no issue.
Here is the event details for the above error:

Interesting. I must have tried only the direction that worked. I'll take a look.

I can reproduce it as well.
Looks like this error occurred even before merging the PR for Issue #415, so at least we know that that PR didn't introduce this error.
Looking at the exception trace, it seems like it is something in EMF Compare. I'll keep investigating.

And it looks like it may have something to do with elements from an template module instantiation, in particular with some element(s) in the instantiation of the Typed package/module from the DDS_DCSP connector library. I don't know if it's a problem specific to that library or in general with template instantiations.
Also, the exception occurs during merging doing some internal check on a reference change which involves one of those elements instantiated, and which happens to have a pathmap reference. I don't know yet if that might be the reason either but it may be relevant.
Nevertheless, it seems like EMF Compare shouldn't be trying to merge the library elements. But in this case it's attempting to merge the instantiated elements from the library. If these elements are never changed by the user, then maybe we should prevent EMF Compare from merging them. So the question is: do users change the elements of these template module instantiations such as DDS_DCPS::CCM_DDS::Typed (TestData_conn in the BasicPubSub model)?

No, users almost never (that I can think of) modify instantiated templated modules after they have been instantiated. We will run the "reimplement" refactor feature from time to time, however. But I don't think we want to disable the merge feature for it in-case it ever becomes a use-case in the future.

Ok, I'll proceed with the assumption that we may want to merge template instantiations as well.
I found something that seems related.
When I checkout the master branch, which is the parent of the conflicting_branch but not the parent of the feature_branch, I get an exception shown below. Furthermore, as the trace shows, the problem seems to be when loading the DDS_Read element, which is in the TestData_conn template instantiation, and which is the container of the element that produces the merge exception (the element DDS_Read::status which is an InterfaceRealization of the PortStatusListener interface).
I also noticed that in the XMI of BasicPubSub.uml of that master branch, the 'client' attribute of the InterfaceRealization, has the id of the client duplicated, but that is not the case for neither the conflicting_branch nor the feature_branch nor the very first commit (the common parent to the master and feature_branch branches):
              <interfaceRealization xmi:type="uml:InterfaceRealization" xmi:id="_IK-WwAEWEe2YDp68O4JWJA" name="status" client="_IK3CAQEWEe2YDp68O4JWJA _IK3CAQEWEe2YDp68O4JWJA">
                <supplier xmi:type="uml:Interface" href="pathmap://DDS4CCM_DCPS_CONNECTOR_LIBRARIES/DDS_DCPS.uml#_h0qncGRnEd-Wc9hRqhM12g"/>
                <contract xmi:type="uml:Interface" href="pathmap://DDS4CCM_DCPS_CONNECTOR_LIBRARIES/DDS_DCPS.uml#_h0qncGRnEd-Wc9hRqhM12g"/>
              </interfaceRealization>

This suggests that the commit pointed to by the master branch may be corrupted. So the question is how did that client field get duplicated values.
Furthermore, I tried to cherry-pick the commit pointed to by the master branch on another branch I created, and got the same exception:
The exception trace:
Unable to load the resource with the uri platform:/resource/ModelMergeReproducer/ModelFiles/BasicPubSub.uml from the storage BasicPubSub.uml
!STACK 0
org.eclipse.emf.ecore.resource.Resource$IOWrappedException: Value 'org.eclipse.uml2.uml.internal.impl.ClassImpl@5a13ad0f (name: DDS_Read, visibility: <unset>) (isLeaf: false, isAbstract: false, isFinalSpecialization: false) (isActive: false)' is not legal. (platform:/resource/ModelMergeReproducer/ModelFiles/BasicPubSub.uml, -1, -1)
	at org.eclipse.emf.ecore.xmi.impl.XMLLoadImpl.handleErrors(XMLLoadImpl.java:77)
	at org.eclipse.emf.ecore.xmi.impl.XMLLoadImpl.load(XMLLoadImpl.java:185)
	at org.eclipse.emf.ecore.xmi.impl.XMLResourceImpl.doLoad(XMLResourceImpl.java:261)
	at org.eclipse.emf.ecore.resource.impl.ResourceImpl.load(ResourceImpl.java:1563)
	at org.eclipse.emf.compare.ide.internal.utils.NotLoadingResourceSet.loadFromStorage(NotLoadingResourceSet.java:254)
	at org.eclipse.emf.compare.ide.internal.utils.NotLoadingResourceSet.load(NotLoadingResourceSet.java:427)
	at org.eclipse.emf.compare.ide.internal.utils.NotLoadingResourceSet.load(NotLoadingResourceSet.java:394)
	at org.eclipse.emf.compare.ide.internal.utils.NotLoadingResourceSet.create(NotLoadingResourceSet.java:155)
	at org.eclipse.emf.compare.ide.ui.internal.logical.ComparisonScopeBuilder.createScope(ComparisonScopeBuilder.java:519)
	at org.eclipse.emf.compare.ide.ui.internal.logical.ComparisonScopeBuilder.create(ComparisonScopeBuilder.java:298)
	at org.eclipse.emf.compare.ide.ui.internal.logical.EMFResourceMappingMerger.mergeMapping(EMFResourceMappingMerger.java:261)
	at org.eclipse.emf.compare.ide.ui.internal.logical.EMFResourceMappingMerger.merge(EMFResourceMappingMerger.java:143)
	at org.eclipse.emf.compare.egit.internal.merge.RecursiveModelMerger$ModelMerge.run(RecursiveModelMerger.java:441)
	at org.eclipse.emf.compare.egit.internal.merge.RecursiveModelMerger$ModelMerge.access$1(RecursiveModelMerger.java:436)
	at org.eclipse.emf.compare.egit.internal.merge.RecursiveModelMerger.mergeTreeWalk(RecursiveModelMerger.java:249)
	at org.eclipse.jgit.merge.ResolveMerger.mergeTrees(ResolveMerger.java:1259)
	at org.eclipse.emf.compare.egit.internal.merge.RecursiveModelMerger.mergeTrees(RecursiveModelMerger.java:135)
	at org.eclipse.jgit.merge.ResolveMerger.mergeImpl(ResolveMerger.java:357)
	at org.eclipse.jgit.merge.Merger.merge(Merger.java:233)
	at org.eclipse.jgit.merge.Merger.merge(Merger.java:186)
	at org.eclipse.jgit.merge.ThreeWayMerger.merge(ThreeWayMerger.java:96)
	at org.eclipse.jgit.api.CherryPickCommand.call(CherryPickCommand.java:129)
	at org.eclipse.egit.core.op.CherryPickOperation$1.run(CherryPickOperation.java:106)
	at org.eclipse.core.internal.resources.Workspace.run(Workspace.java:2292)
	at org.eclipse.core.internal.resources.Workspace.run(Workspace.java:2317)
	at org.eclipse.egit.core.op.CherryPickOperation.execute(CherryPickOperation.java:118)
	at org.eclipse.egit.ui.internal.commit.command.CherryPickUI$1.performJob(CherryPickUI.java:147)
	at org.eclipse.egit.ui.internal.jobs.RepositoryJob.run(RepositoryJob.java:59)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: org.eclipse.emf.ecore.xmi.IllegalValueException: Value 'org.eclipse.uml2.uml.internal.impl.ClassImpl@5a13ad0f (name: DDS_Read, visibility: <unset>) (isLeaf: false, isAbstract: false, isFinalSpecialization: false) (isActive: false)' is not legal. (platform:/resource/ModelMergeReproducer/ModelFiles/BasicPubSub.uml, -1, -1)
	at org.eclipse.emf.ecore.xmi.impl.XMLHandler.setFeatureValue(XMLHandler.java:2715)
	at org.eclipse.emf.ecore.xmi.impl.XMLHandler.handleForwardReferences(XMLHandler.java:1193)
	at org.eclipse.emf.ecore.xmi.impl.XMLHandler.endDocument(XMLHandler.java:1282)
	at org.eclipse.uml2.uml.internal.resource.UMLHandler.endDocument(UMLHandler.java:57)
	at org.eclipse.emf.compare.ide.internal.utils.ForwardingXMLDefaultHandler.endDocument(ForwardingXMLDefaultHandler.java:67)
	at org.eclipse.emf.compare.ide.internal.utils.NotifyingParserPool$NamespaceDeclarationNotifyingXMLDefaultHandler.endDocument(NotifyingParserPool.java:217)
	at java.xml/com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.endDocument(AbstractSAXParser.java:746)
	at java.xml/com.sun.org.apache.xerces.internal.impl.XMLDocumentFragmentScannerImpl.scanDocument(XMLDocumentFragmentScannerImpl.java:539)
	at java.xml/com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:888)
	at java.xml/com.sun.org.apache.xerces.internal.parsers.XML11Configuration.parse(XML11Configuration.java:824)
	at java.xml/com.sun.org.apache.xerces.internal.parsers.XMLParser.parse(XMLParser.java:141)
	at java.xml/com.sun.org.apache.xerces.internal.parsers.AbstractSAXParser.parse(AbstractSAXParser.java:1216)
	at java.xml/com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl$JAXPSAXParser.parse(SAXParserImpl.java:635)
	at java.xml/com.sun.org.apache.xerces.internal.jaxp.SAXParserImpl.parse(SAXParserImpl.java:324)
	at org.eclipse.emf.ecore.xmi.impl.XMLLoadImpl.load(XMLLoadImpl.java:175)
	... 27 more
Caused by: java.lang.IndexOutOfBoundsException: targetIndex=1, size=1
	at org.eclipse.emf.common.util.BasicEList.move(BasicEList.java:657)
	at org.eclipse.emf.common.notify.impl.NotifyingListImpl.doMove(NotifyingListImpl.java:1338)
	at org.eclipse.emf.common.notify.impl.NotifyingListImpl.move(NotifyingListImpl.java:1311)
	at org.eclipse.emf.ecore.xmi.impl.XMLHelperImpl.setValue(XMLHelperImpl.java:1189)
	at org.eclipse.emf.compare.ide.internal.utils.ForwardingXMLHelper.setValue(ForwardingXMLHelper.java:205)
	at org.eclipse.emf.compare.ide.internal.utils.NotifyingParserPool$NotifyingXMLHelper.setValue(NotifyingParserPool.java:298)
	at org.eclipse.emf.ecore.xmi.impl.XMLHandler.setFeatureValue(XMLHandler.java:2710)
	... 41 more

I have always had a sneaking suspicion that some of our models were partially corrupted after the migration from RSA. What the cause of this is, I don't know, but we have always seen strange behavior with our models that we can't reproduce, so we tend to not raise the issues with your team. For the most part CX and Papyrus do not have an issue with these models, but us users have no idea that something might be malformed inside the model. I would love a model validation extension that would do this sort of extra introspection.
Have you tried manually removing the extra ID specification to see if that fixes your exception?

I have tried it and indeed, that is the culprit. It turns out there where several duplicated ids in the 'client' attribute of the InterfaceRealization of several of the instantiated template elements: DDS_Read, DDS_Get, DDS_Listen and DDS_StateListen.
After removing the duplicated ids, I amended the commit for 'master' creating a new 'master1' branch, and then cherry-picked the commits for 'feature_branch' and 'conflicing_branch' into branches 'f1' and 'c1' respectively on top of the new 'master1'. (See the attached updated git repo.) Then I tried merging both ways, 'c1' into 'f1' and 'f1' into 'c1' and both work.
So we can conclude that the cause of the problem is indeed those duplicate ids. The question is how did they get duplicated? Your suggestion of this being an artifact from the RSA migration is a possibility, except that in this case, the first commit in the repo, the common parent to 'master' and 'feature_branch' does not have these duplicate ids. So somehow between the first commit and the commit for 'master', these ids were introduced. The author of these commits was David Ashenden. Perhaps he might remember what he might have done? I imagine he might not, as the repo was made 9 months ago, but perhaps he might remember if something odd happened when he did that commit.
In the meantime, my working theory is that it may have something to do with template instantiation, since the commit for 'master' added a new template instantiation for a DataSpace "NewMessage_conn", and it is precisely the elements inside this package that have the duplicate ids. I'll do some tests to see if template instantiation causes these duplicates.
As for validation, indeed, that would be ideal. Our meta-model already has some validation built-in, but surely this particular case was missing. I'm going to check if I can find validation for these duplicates. If the problem is produced by template instantiation, then it might make sense to run validation after it, and in fact after any operation modifying the model. I believe that some of this validation is done "live", but perhaps not all of it, as it may be too costly.
modelmergereproducer2.zip

Well, I've been trying unsuccessfully several things to reproduce the duplicated client ids, including:

Creating a new template instantiation
Copying and pasting an existing template instantiation
Apply every refactoring available in the "Model refactor" menu on both BasicPubSub and a fresh test model

Unfortunately I still can't find any action that would result in the duplicated ids. I'll keep trying.
I've also done a quick inspection of the code that manipulates InterfaceRealizations and nothing has come up yet. But perhaps it's something very subtle.

I think I can add some validation for it.
I'll also continue to see if I can figure out how it could have been duplicated.

Just to give you an update: validation turns out to be trickier that expected. It turns out that when the model is loaded, the duplicated id is removed, so the model in memory has only one reference to the target element, and therefore normal validation fails to detect the problem.
I suppose that somehow, when Git operations such as checkout and merge are performed, the model is loaded in a different way that preserved the duplicated ids causing the error. I'm looking into that.

Thanks for the update

A new update. It seems like the duplicate ids were just a red herring and not the real cause of the exception.
The duplicated ids do indeed result in an exception when opening the model and performing certain EGit operations like checkout and cherry-picking. But my original attempt to reproduce without the duplicate ids was flawed. A more accurate reproduction without duplicate ids still yields the original exception. It does happen with the same elements in the model, which is why I thought it was the duplicate ids, but EMF Compare has a problem with a different feature, not the one with duplicate ids. It does seem to be a problem with EMF Compare itself, so I'm digging in deeply trying to understand how it works exactly.
Furthermore, while the duplicate ids raise an exception, the model does seem to be correctly loaded in memory. So validation for duplicate ids doesn't seem to be that important now, at least for this issue. Besides, with the model correctly loaded in memory, the only way to detect such duplicated ids is to do it outside of the validation framework. So for now, I think it's more important to focus on the real cause of the original exception, and leave the potential validation for later.

An update: the problem is related to InterfaceRealization elements whose supplier Interface is external (a pathmap reference to a library element) and which exist in the branch being merged but not in the receiving branch.
I still do not understand why this is the case as it goes deeply into the inner workings of EMF Compare, but I was able to create a minimalistic git repo that reproduces the error. See the attached zip file.
In this repo there is a model with a struct S1, a locally defined interface I1 and a port type PT1. Then there are two sets of branches: "internal" and "external". In the "internal" ones, the interface realization of the "provides" interface of PT1 is the local interface I1. In the "external" branches, the interface realization is "PortStatusListener" from the DDS_DCSP library (and therefore an external pathmap reference). Then for each set, there are two conflicting branches "-a" and "-b". The conflict is on the type of the attribute in the S1 struct. Additionally the "*-b" branches add a new port type PT2, and as before, its interface realization is I1 in the "internal-b" branch and PortStatusListener in "external-b" branch.
When merging "internal-a" onto "internal-b" it works as expected.
When merging "internal-b" onto "internal-a" it works as expected.
When merging "external-a" onto "external-b" it works as expected.
but when merging "external-b" onto "external-a" we get the exception.
So this clearly rules out a problem with interface realizations with references to local interfaces.
It's not clear why the situation is asymmetric. But at least this should simplify the process of diagnosing the problem.
issue444min.zip

Thank you for the update. I am glad you were able to find a much simpler reproducer

A new update: It has been very difficult to track down the cause, but I think I may have discovered it, or at least part of it. I've found a fundamental asymmetry embedded deep inside the core of EMF Compare, during a phase called "premerge", which seems to behave differently depending on the direction of the merge. I think it may be a fundamental bug in EMF Compare itself, but I need to investigate and test further to confirm this.
I'll provide more details on Monday.

By the way, under Preferences -> EMF Compare -> Merge there is a "Pre-merge" checkbox. If one unchecks it, premerge is not performed and there is no exception, at first. This is, the comparison + conflict detection will succeed and the user will be able to open the Merge Tool and see the differences. However, once you make the desired changes to resolve the conflict and try to save, the actual merge will be executed leading to the same exception. Therefore, unchecking the "Pre-merge" checkbox is not a real workaround, and the underlying problem remains.

And more info: when merging "external-a" to "external-b" (the one that works, where the new elements are on "external-b" already), after the comparison, and before the pre-merge, the list of differences is empty! This results in the pre-merge being skipped, which is why there is no exception in that case. This looks like another unexplained asymmetry, most likely a bug.

I've made some modifications to the method that causes the exception and it succeeds, but there are two caveats:

My solution works when there is at most one element in the list that's being checked, which in this case is the list of "suppliers" of the InterfaceRealization. I need to test whether adding more suppliers will work, but I doubt it. Nevertheless, that shouldn't be a problem for this case, as typically an InterfaceRealization has only one Interface. But I don't know if there are other cases.
The solution requires an update to a core part of EMF Compare, which may not be customizable, so we may be forced to create a fork of EMF Compare. I'm investigating if we can avoid the fork.

It looks like it is possible to override the problematic code without having to form EMF Compare. I've opened a PR with the proposed solution, but as mentioned in my previous comment, it may not work in some cases, so I'm testing to see what happens in those cases.

After some testing, the case for multiple suppliers seems to work, but I'm never hitting the code that I think could cause trouble.
Nevertheless, you can test the build with the proposed solution which you can get here.

I've continued to test it and it seems to work in all cases I've tried. I still cannot reproduce a case that hits a potentially problematic piece of the code, and it looks like that code deals only with some corner cases, not typical cases, so, since the solution works for all cases that we have encountered, if it works for you, we could merge this solution and deal with the remaining corner cases in another issue, or we can keep this issue open and I'll continue to investigate when I get back on July 10th. Paul has reviewed the solution and is aware of the corner cases. He can merge this one if you'd like to. Let us know.

Have you attempted to use the proposed solution with our original modelmergereproducer example? When I use it, it no longer throws an error, which is great. What is strange is that I do get a conflict, and attempting to resolve the conflict has an extra step I wouldn't expect. The conflict is on the remove "color" enumeration. Are you seeing the same thing?

Hi. I have, and I see the conflict, and in fact I get the same conflict in both directions. The conflict was expected because the feature_branch added, in the 'defn' package, the 'ComplicatedTestData' struct which has a 'color' attribute of type Color, which is the Enum deleted in the conflicting_branch.

Ah, that makes some sense. I guess that is a tough thing for a modeling merging tool to understand. If we remove color in one spot but use it in another, how do we resolve that conflict easily.
In any case, the issue that this ticket was to resolve appears to be fixed, so I am happy with merging and closing this ticket.

Great! Should I merge it both to the papyrus branch and the maintenance branch?

Yes, I think merging it to both would be best for us. Thanks!

The commit has been cherry-picked onto the streams/v2.4.x-maintenance branch and merged onto the papyrus branch.


Add libdisni.so in wiki to test performance of SparkRDMA

We have found some issue about the wiki page in https://github.com/Mellanox/SparkRDMA/wiki/Running-HiBench-with-SparkRDMA which may be fixed easily.
In section "Experiment #1: TeraSort", we need to add not only spark-rdma-3.1-for-spark-SPARK_VERSION-jar-with-dependencies.jar but also libdisni.so.
Add to HiBench/conf/spark.conf:
spark.driver.extraClassPath /PATH/TO/spark-rdma-3.1-for-spark-SPARK_VERSION-jar-with-dependencies.jar
spark.executor.extraClassPath /PATH/TO/spark-rdma-3.1-for-spark-SPARK_VERSION-jar-with-dependencies.jar
spark.shuffle.manager org.apache.spark.shuffle.rdma.RdmaShuffleManager
spark.shuffle.compress false
spark.shuffle.spill.compress false
spark.broadcast.compress false
spark.broadcast.checksum false
spark.locality.wait 0

It should be changed to this.
Add to HiBench/conf/spark.conf:
spark.driver.extraLibraryPath /PATH/TO/libdisni.so
spark.executor.extraLibraryPath /PATH/TO/libdisni.so
spark.driver.extraClassPath /PATH/TO/spark-rdma-3.1-for-spark-SPARK_VERSION-jar-with-dependencies.jar
spark.executor.extraClassPath /PATH/TO/spark-rdma-3.1-for-spark-SPARK_VERSION-jar-with-dependencies.jar
spark.shuffle.manager org.apache.spark.shuffle.rdma.RdmaShuffleManager
spark.shuffle.compress false
spark.shuffle.spill.compress false
spark.broadcast.compress false
spark.broadcast.checksum false
spark.locality.wait 0

On the README page we've mentioned:
libdisni.so must be in java.library.path on every Spark Master and Worker (usually in /usr/lib)

So if libdisni is in default jaava.library.path on each node (e.g. in /usr/lib) then no need to configurre additionally.