Still can't make it work

I have been following this package for quite some time now. A few months ago I opened a bug report about not being able to make it work (show anything).
I just tried it again and I still can't make it work (can't make it output anything at all).
I'm using a composer project (which is detected very nicely by Intellij IDEA). I even see the package doing things in the dev console (messages like "Building index"), but I don't see any popups with an autocomplete.
I even tried https://github.com/Peekmo/atom-autocomplete-php/pull/78 and it doesn't show any warnings, which confirms that my project is alright.
Is there anything I can do to make this plugin tell me why is it not working? Is there some "verbose/debug" mode?

Hello
These are some things I noticed that can cause problems (disclaimer: I'm not a core developer, more of a user and minor contributor):

When a syntax error is present in a file being saved, autocompletion can stop working.
IIRC the package requires that the project follows the PSR-2 standard with relation to naming classes, methods, and so on (maybe @Peekmo can confirm this).
Not every corner case works yet, but simple $this-> autocompletion should work fine (inside class method definitions).
Are you sure the classmap and autoload.php paths in your configuration are set to the correct location (relative to your project root)?
Is there actually any content in the autoload_classmap.php file or is it just an empty array? If it is empty, I believe you can forcibly fill it using composer dumpautoload -o. It should be automatically filled by Composer but I've found that with some composer.json configurations it is not.

AFAIK there is no verbose switch, but if you are familiar with the developer tools of Chromium (as for building websites), you can also use that in Atom by viewing the package source, placing 'debugger' statements in the CoffeeScript code and then refreshing your PHP project using Ctrl+R with the developer tools open (or just use the Chromium Debugger to manually place breakpoints). You can use this to examine output from PHP and see what the code is doing as a last resort. (It requires no additional software or anything.)
Hope this helps.

Hello,
@hotoiledgoblinsack has list every thing that could be a problem for the plugin (thank you).
Anyway, can you try to clone this repo : https://github.com/hotoiledgoblinsack/php-autocomplete-test and (after a composer install), for example, in this file, at this line : https://github.com/hotoiledgoblinsack/php-autocomplete-test/blob/master/src/TestNamespace/SomeBaseClass.php#L52 write $this->
If something is showed, then, the problem is from your project.
Regards,
Peekmo

Will try to test all this tomorrow, I have been extremely busy those days. Sorry for the delay.

Ok, tested the repo you linked and indeed, the autocomplete worked. I must debug my project and find where/why it's failing.

Btw, will this plugin work with PSR-4?

Hello,
Yes, I'm following PSR-4 in all my projects, and it works

Ok, let's give this another angle. I'm using this project with Yii2, so let me see if I can create an empty Yii2 project and make this plugin work.

I just tried Yii2 framework.
The plugin does not support this framework for a simple reason : This framework does not rely on composer for its autoloading at all (no section 'autoload' in its composer.json and see vendor/yiisoft/yii2/Yii.php).
So, sorry, but it will never be supported (at least for PHP5, I have better hopes for PHP7) in this plugin.

I'm not really sure how/what you tested, but here is what I see/do:
composer global require "fxp/composer-asset-plugin:~1.0.3"
composer create-project --prefer-dist yiisoft/yii2-app-basic basic

This creates the folder basic that contains a new Yii2 project. The composer.json file at the root folder is supposed to be edited by the developer and fill in the fields related to the project (name, version, etc).
The folder vendor contains (among others) a folder called yiisoft, which contains the Yii2 structure. The file yii2/composer.json contains an autoload section (which is what you're looking for).
Yii2 itself is using composer, as all Yii2's plugins and external libraries. Why I can't get autocomplete when using Yii2 if composer is used?

The composer.json in the root folder in the one that download all vendors (like Yii), and they have all a composer.json. But if the project would be managed with composer, you'll have a section "autoload" with your personal source files in a composer.json. But there is not. There's a section "autoload" with Yii vendor source code in yii2/composer.json but not your source files (backend, common, console, ...)
Play the command composer dump-autoload in your project and have a look into vendor/composer/autoload_classmap.php and you'll see that there's an empty array... So the main composer doesn't manage the autoloading of your sources, just those of your vendors (check to write $this-> in a yii2 vendor's class, it should work.

So this plugin is actually able to work with Yii2, it's just that it's missing a little bit of glue when there is no autoload section in the "main" composer.json file.
Would it be difficult to make this plugin scan the vendors folder and try to "manually" load what it can?
Also, is this plugin just relying on the content of the autoload_classmap.php or is there some more composer-related magic happening behind the curtains?

Yes, it will be difficult because it needs an autoloader to be able to work with PHP reflection (the plugin does not parse all PHP files), and the composer classmap to easily find all files. The classmap could be done manually, but not the autoloader. Moreover, the plugin parse the autoload section of your composer to add the right namespace to your files.

Sorry for insisting, but aside from the "autoload" section in the main composer.json file I can't really see any difference between a new symfony project (with which this plugin works) and a new yii2 project (with which this plugin doesn't work).
This is how a new symfony project looks like: http://ibin.co/2GjuJSCR6MoN
And this is how a new yii2 project looks like: http://ibin.co/2Gjutz1sSigI
So, both projects have autoload_(classmap, files, namespaces, psr4) and all of them are filled with the (I assume) correct namespaces, files, paths and so on. So, aside from the "autoload" section, there isn't really any other difference. Hence, why Symfony is working, but Yii2 isn't?
I'm sorry for being repetitive, but I just can't really understand where exactly is the problem.

The autoload section is the most important.
If you see a symfony2 project, you have something like that in the autoload section "psr-4": { "": "src/" }
It tells to composer every classes under src are prefix by nothing and so, in autoload_namespaces, ""will match .../src So, the class files within src folder will be autoloaded by composer.
In this plugin, we require in the proxy (https://github.com/Peekmo/atom-autocomplete-php/blob/master/php/parser.php#L80) an autoloader (composer ones by default) to be able to find every classes by reflection in PHP. If your source files (like in Yii2) are not autoloaded by composer, when we try to do a reflection on it, it fails, because the script doesn't know your class, because we didn't require it.
You could try to add vendor/yiisoft/yii2/Yii.php in the configuration of the plugin (section "Autoloader file"), 'cause it seems to be the main autoloader of Yii2, but even if it's partially works, you will miss some features because the autoload_classmap will not contains your source files.

As far as I know (perhaps I read it somewhere in Yii's docs) Yii2 uses spl_autoload_register. Basically, it tells composer that it will use it's own class mapper/importer/loader, but iirc composer is still able to understand and work with that. Not sure if that help somehow.

spl_autoload_register doesn't tell anything to composer. It's a PHP feature, used by composer itself (in vendor/composer/autoload_real.php) It just tell PHP that "When this class is instanciated, look into those files".

This is an old issue, but hare are my findings.
In my case, Yii's clases would be indexed correctly (because of this) but it would not load my personal classes...
So, I just added this to my composer.json (assuming that app is my root namespace).
    "autoload": {
        "psr-4": {"app\\": ""}
    }

Dont' forget to  composer dump-autoload
You might also need to restart Atom to force a reindexing (By the way @Peekmo , we are missing a command for that)
I am still not sure what impact it can have on the autoloading on the project itself. But if you follow Yii's recommendations it should be good.

When using the Yii autoloader together with other autoloaders, you should include the Yii.php file after all other autoloaders are installed. This will make the Yii autoloader the first one responding to any class autoloading request.

atom-autocomplete-php would then use psr-4 from composer to autoload/parse the project, but when the project is running, it would still be using Yii's autoloader.

Oh, @alexandernst I just saw you other issue here: https://github.com/Peekmo/atom-autocomplete-php/issues/124
which is basically what I just explained :)


Use different ports behind nginx

Hi, i'm trying to run this on google cloud run which requires to run on port 8080. Right now this port is used internally so it doesn't run when trying to use PORT=8080. Would it be possible to move the internal ports into a less frequently used port range? (like e.g. 32543)

I'm not experienced with Google Cloud, but the Docker container internally uses port 80 and AFAIK the host should not interfere with the internals of the container. When you start the container e.g. with docker run -p "<IP_ADDRESS>:8080:80" jgontrum/spacyapi:en_v2, you map the local port 8080 to the container port 80. So if 8080 doesn't work for you, can't you try -p "<IP_ADDRESS>:32543:80"?

Unfortunately the requirements Cloud Run has is for the container is to listen to whatever port is specified in the PORT env variable. Unfortunately I cannot specify which port they use - they say that they use "8080" but this shouldn't be relied on, the app should support the PORT env variable at best.
I realize this is a bit of a limitation from the Cloud Run side as you usually have something to route to the container (using -p for docker bridge network, some reverse proxy, kubernetes service, ...) and it seems this is a bit unique to Cloud Run.
Would be cool if this could be changed here to support the use case, since otherwise it would require forking and it seems like it wouldn't break anything for current users to support it. Since you already have support for the PORT env variable it would only require changing the internal ports of the frontend and service which are not exposed right now.
To summarize, this is basically a feature request to support using PORT=8080 with the container :)
docker run -e PORT=8080 -p "8080" jgontrum/spacyapi:en_v2
BTW thank you very much for packaging this all up in an easy to use container! It's a fantastic time saver :blush:

I'm having a similar issue when trying to use Cloud Run. Has anyone had any success with deploying this on GCP or have any recommendations?

@Kamshak I am facing the same issue here deploying on Cloud Run.
Basically Cloud Run prefers listening on the port 8080 and as the container is using nginx i have replaced the default.conf file to make nginx expose port 8080 as given on the link - https://stackoverflow.com/questions/47364019/how-to-change-the-port-of-nginx-when-using-with-docker#:~:text=If you want to change,conf file inside the container.&text=navigating to localhost%3A3333 in,to include the default nginx.
** Dockerfile **
FROM jgontrum/spacyapi:base_v2
RUN pip install wheel
ENV languages "en_core_web_lg"
RUN cd /app && env/bin/download_models
COPY default.conf /etc/nginx/conf.d/
COPY nginx.conf /etc/nginx/
EXPOSE 8080
The problem is that it gives a 502 error on hitting any api. Here are the logs:
/usr/lib/python2.7/dist-packages/supervisor/options.py:461: UserWarning: Supervisord is running as root and it is searching for its configuration file in default locations (including its current working directory); you probably want to specify a "-c" argument specifying an absolute path to a configuration file for improved security.
'Supervisord is running as root and it is searching '
2020-06-25 22:20:43,107 CRIT Supervisor is running as root. Privileges were not dropped because no user is specified in the config file. If you intend to run as root, youcan set user=root in the config file to avoid this message.
2020-06-25 22:20:43,107 INFO Included extra file "/etc/supervisor/conf.d/supervisor.conf" during parsing
2020-06-25 22:20:43,132 INFO RPC interface 'supervisor' initialized
2020-06-25 22:20:43,132 CRIT Server 'unix_http_server' running without any HTTP authentication checking
2020-06-25 22:20:43,133 INFO supervisord started with pid 7
2020-06-25 22:20:44,136 INFO spawned: 'nginx' with pid 10
2020-06-25 22:20:44,142 INFO spawned: 'api' with pid 11
2020-06-25 22:20:44,145 INFO spawned: 'frontend' with pid 12
2020-06-25 22:20:45,333 INFO success: nginx entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2020-06-25 22:20:45,334 INFO success: api entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2020-06-25 22:20:45,334 INFO success: frontend entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2020-06-25 22:20:49,475 INFO exited: frontend (exit status 2; expected)
2020-06-25 22:21:34,723 INFO exited: api (exit status 2; expected)
I can't figure out what is the problem here. Any help is appreciated.


build: auto release

This PR adds automated release functionality to our CI process. This takes away most of the manual work of creating a release.
We use electron-builder for this.
After all the tests have passed on CI, electron-builder will check if a draft release exists on github and if it finds one it will build the project and attach the build artifacts to the release node.
This runs on travis (builds on/for osx and linux) and appvoyer (builds on/for windows).
Results look like this:

In order to integrate this you need to create a github access token with the public_repo permission and set it in the travis and appvoyer environment variables as GH_TOKEN.
Then, to trigger automate build publishing for an upcoming release:

Draft a new release. Set the “Tag version” to the value of version in your application package.json, and prefix it with v. “Release title” can be anything you want. For example, if your application package.json version is 1.0, your draft’s “Tag version” would be v1.0.
Push some commits. Every CI build will update the artifacts attached to this draft.
Once you are done, publish the release. GitHub will tag the latest commit for you.
The benefit of this workflow is that it allows you to always have the latest artifacts, and the release can be published once it is ready.

This follows the electron-builder recommended release flow:
https://www.electron.build/configuration/publish#recommended-github-releases-workflow

You can see an example of a successful build from there here:
https://travis-ci.org/mrfelton/zap-desktop/builds/407861306

utACK 37bb20d

utACK 37bb20d

Pull Request Test Coverage Report for Build 1122

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+29.0%) to 63.636%

Totals

Change from base Build 1119:
29.0%

Covered Lines:
5

Relevant Lines:
7

💛  - Coveralls

@Empact @JimmyMow I've given this PR some pretty extensive testing and now have the automated deployment process fully working. Once merged, every build on the master branch will build all electron targets and then attach to a github draft release whose tag matches the version number in package.json if one exists.
You can see the latest results of this here:
Travis build that triggered the deployments:
https://travis-ci.org/LN-Zap/zap-desktop/builds/408641606
Draft github release with builds attached:
https://github.com/LN-Zap/zap-desktop/releases/tag/untagged-6f67776ef89cc44961a1
As a follow up to this we will need to do https://github.com/LN-Zap/zap-desktop/issues/615 so that the builds are properly signed.

utACK 1ac5271


check(object, Schema.object) passes function in tests

Hello, I'm notice unexpected behavior of SS with check in tests. I've used this schema
Then, there is a server Method, which contain such code:
"accounts/addressBookAdd": function (address, accountUserId) {
    check(address, ReactionCore.Schemas.Address);
    check(accountUserId, Match.Optional(String));
...
});

If I trying to call method in such way:
expect(function () {
  return Meteor.call(
    "accounts/addressBookAdd",
    () => {console.log("test")},
    () => {}
  );
}).toThrow();

It is successfully passed the check. Is this expected? This was not reproduced in browser.

What does the method receive? Since function can't be sent over wire, I'm assuming address and accountUserId are probably undefined in your test. It's possible that no error is thrown when the first arg to check is undefined. Is that the issue?

Hi, in situation with browser, yes, method receive undefined or null as address.  I did not remember. But in Velocity's tests method receives a function and the main problem here - SS passes this function through.

Closing this since it doesn't really seem like an issue with this package.


Dependency don't resolve in release v.

Release version has package dependency issues:
 Problem 1
    - friendsofphp/php-cs-fixer is locked to version v2.15.1 and an update of this package was not requested.
    - friendsofphp/php-cs-fixer v2.15.1 requires php ^5.6 || ^7.0 -> your php version (8.1.6) does not satisfy that requirement.
  Problem 2
    - phpunit/phpunit is locked to version 8.2.5 and an update of this package was not requested.
    - phpunit/phpunit 8.2.5 requires php ^7.2 -> your php version (8.1.6) does not satisfy that requirement.

You can also try re-running composer require with an explicit version constraint, e.g. "composer require proget-hq/apple-activation-lock:*" to figure out if any version is installable, or "composer require proget-hq/apple-activation-lock:^2.1" if you know which you need.

It looks like the issue has been resolved in the latest commits, so you should just need to push a new release version to resolve.

These are only dev dependencies, it does not really affect the tagged release. What are you trying to achieve?


Refactor parsing to return error instead of using panic/recover

Currently the parser code relies on recover function to catch errors and return them to the end user. The recommended go approach is to return errors from functions and check them. This commit refactors all log.Panicf calls to actually returning errors.
Fixes #33.
@robfig as promised some time ago, ptal
@nd2s @lukescott fyi

I've also extended the test cases, currently the test coverage is almost 100%.

I guess that doesn't look so much worse than the panic version... Just a small comment or two. Thanks!

I've addressed your comments and rebased the PR on top of #73.

Thank you

Thank you!


refactor: not simplifier

Description
This PR adds the refactoring of the simplifier for not.
It changes the entire perspective, to instead of adding simplification functions per CommonModel property, we process the JSON Schema keywords which then affects the CommonModel in some way.
It adds

simplifyNot, to interpret the not schema keyword.

Files this PR is supposed to replace
This does not directly replace a single file, however, it fixes a few bugs with the current simplifier.
Todo

[ ] Refactor docs
[ ] Solve code smells
[ ] Add logging to the process

Related issue(s)
Blocked by #184

Coverage decreased (-0.4%) to 94.197% when pulling a1a0f9cd352cbba158a11fc998da82e3873dd61f on jonaslagoni:feature/not_simplifier into f47276c22db4c9e987a2f8e77b9ecdcee74aff54 on asyncapi:master.


Setup custom webhook link is broken

The Setup a custom webhook link is broken on this section. Perhaps it should point to https://docs.microsoft.com/en-us/microsoftteams/platform/webhooks-and-connectors/how-to/add-incoming-webhook?

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

ID: 28316a67-22fa-8a05-1ae0-fbe1c5720ef9
Version Independent ID: 8a250e46-2a30-af86-839d-9f83a8b48041
Content: Create and send messages - Teams
Content Source: msteams-platform/webhooks-and-connectors/how-to/connectors-using.md
Product: msteams
GitHub Login: @laujan
Microsoft Alias: lajanuar

Hi @LanJosh - We are not facing this issue from our side. It's redirecting properly. So, could you please check again?

Hmm, yesterday I was getting a 404 when I hit this link but not experiencing it now either. I'll close this!

Ah, it was patched already https://github.com/MicrosoftDocs/msteams-docs/pull/3745/files


Fixing level 3 page + logic improvement in all the levels.

Implementing the below features in page 3 as done same in page 1.

Multiple retires in the questions page.
Explanation when user chooses wrong answer.
Redirect to explanation page if user loses.
Amazon kid friendly voucher card text.

/assign

@Talha-ai  are you working on this ?!

@Talha-ai are you working on this ?!

This issue is fixed and merged

Yeah Yeah its fixed @Vinay-Khanagavi , sorry I forgot to link this. @Talha-ai from next when you open a pr, write closes # and the issue number with no space, so when we merge ur pr. The issue is automatically closed.

Yeah Yeah its fixed @Vinay-Khanagavi , sorry I forgot to link this. @Talha-ai from next when you open a pr, write closes # and the issue number with no space, so when we merge ur pr. The issue is automatically closed.

Got it. thx


reorganize unit tests

Some functions have moved around quite a bit within the Kernel Tuner code base, but the test suite still reflects the original file structure. It would be nice if the tests could be reorganized such that it will be easier to find the tests that are responsible for a single module.
Also some tests may be considered integration tests and should perhaps be moved to a separate place (I'm talking about tests that actually call run_kernel or tune_kernel).
I'm perfectly fine with the fact that some unit tests require a GPU (and use skiptest when they're not present). I know this goes into general unit testing opinions that think unit tests should not depend on external factors, but it's something we can't avoid when working with GPU codes.

Just discussed with @egpbos about using some of the examples as integration tests as well. The idea is to separate the setup of tunable parameters from some of the examples so we can call the examples with reduced search spaces.

cleaning up the issues a bit, this one is quite old, will pick this up again at a later point


Linux Installation Bug | Ubuntu 22.04

Hey there webui-dev!
I was trying to build the program by using this method:
git clone https://github.com/webui-dev/webui.git
cd webui/build/Linux/GCC
make

But it gave me this error:

fatal error: client/webui.h: No such file or directory

Is there a way to solve this problem?
Thanks,
Mehmet Mert

Also, opened the vscode to the source directory and had this error:
[{
	"resource": "/home/mehmetmertgunduz/webui/src/webui.c",
	"owner": "C/C++: IntelliSense",
	"code": "1696",
	"severity": 8,
	"message": "cannot open source file \"client/webui.h\"",
	"source": "C/C++",
	"startLineNumber": 19,
	"startColumn": 1,
	"endLineNumber": 19,
	"endColumn": 26
}]

This repo has a new build process that still needs to be completed. We are still voting for it.
For now, you can use the stable released version. Simply download Source code (tar.gz).

Thanks a lot @AlbertShown 🚀

Fixed, now simply clone and build.
No dependency is needed (Unless you want to modify webui_bridge.ts)
https://github.com/webui-dev/webui/commit/096b0cf8369fe6cea30d366abbdc41de83dd36e5


Remove IAP from staging and prod

Similar to the previous change on dev, we're removing the IAP code and adding new OAuth variables in order to enable moving to Signon

Superceded by https://github.com/alphagov/govuk-knowledge-graph-gcp/pull/480


Webpacker helpers

Possibly going to rename to Webpacker Helpers.

This change is

Probably never going to merge this one...


User Registration

This task includes implementing the user registration functionality. The scope of this task would be to handle an incomplete sign up case

Completed


Throwables.propagate is deprecated

The issue tracker is ONLY used for bug report and feature request.
Any question or RocketMQ proposal please use our mailing lists.
BUG REPORT

Please describe the issue you observed:
Throwables.propagate is deprecated

Please tell us about your environment: Windows, VsCode

can I raise PR for this  ?

@Oliverwqcwrw I am able to open rocketmq-dashboard in local but this issue is popping up ?

Regards

It looked like the connection to the NameServer failed, please check your cluster if it is working.


Feature Request: Export SqlDump #205

Code changes for Feature Request #205. New UI screen for exporting SQL Insert and Create statements in Preview and Download mode.

@tcbuzor What's it look like with 10 columns ?

Screen with 11 columns attached.

@tcbuzor 1 more option on the bottom please... "remove whitespace from column names"
Also, there's a nice feature to have... if I facet on only 24 rows of my 100 rows... I expect that the Export to SQL will only preview those 24 rows that I have a Text facet on.  When I tried this, it was previewing all rows, no matter my Facet options.

@thadguidry Check the latest code. Added the trim and facet options.

@tcbuzor umm... I wasn't saying to ignore the facets, but to respect them ?

It respects them by default. Try it now.
On Sun, Mar 25, 2018 at 10:07 PM, Thad Guidry<EMAIL_ADDRESS>wrote:

@tcbuzor https://github.com/tcbuzor umm... I wasn't saying to ignore
the facets, but to respect them ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/pull/1552#issuecomment-376034004,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Af2f5YK5fePyHC1oxrpmuMb4B9qqAYCXks5tiFtxgaJpZM4S6ACw
.

@tcbuzor ok, the Facet respecting works now.  But not the trim on column names for preview or download..
CREATE TABLE clipboard (
record VARCHAR(255),
Column 1 VARCHAR(255),
values 2 VARCHAR(255),
values 3 VARCHAR(255),
values 4 VARCHAR(255),
values 5 VARCHAR(255),
values 6 VARCHAR(255),
values 7 VARCHAR(255),
values 8 VARCHAR(255)
);
INSERT INTO clipboard (record,Column 1,values 2,values 3,values 4,values 5,values 6,values 7,values 8) VALUES
( 'record','record','Alfredo Santamaria 2013','Spain','Castilla-y-León','Cigales	','Alfredo Santamaria Arias, S.L. (Producer)','+34 983 58 50 06','www.bodega-santamaria.com' ),
( 'record','record','Al-Ria Reserva 2015','Portugal','Algarve','Vinho Regional	','Casa Santos Lima (Producer)','+351 2 63 76 90 93;+351 2 63 76 06 21','www.casasantoslima.com' ),
( 'record','record','Amaren Tempranillo Reserva 2009','Spain','Rioja','Rioja Alavesa	','Araex Rioja Alavesa SL (Producer)','+34 945 15 05 88','www.araex.com' ),
( 'record','record','Aponte Reserva 2008','Spain','Castilla-y-León','Toro	','Bodegas Frontaura SLU (Producer)','+34 983 88 04 88','www.bodegasfrontaura.com' ),
( 'record','record','Arzuaga Reserva Especial 2011','Spain','Castilla-y-León','Ribera del Duero	','Bodegas Arzuaga Navarro SL (Producer)','+34 983 68 11 46','www.arzuaganavarro.com' ),
( 'record','record','Baigorri Belus 2013','Spain','Rioja','Rioja Alavesa	','Bodegas Baigorri (Producer)','+34 945 60 94 20','www.bodegasbaigorri.com' ),
( 'record','record','Bottega Il Vino degli Dei 2012','Italy','Veneto','Amarone della Valpolicella DOCG	','Bottega SPA (Producer)','+39 04 38 40 67','www.bottegaspa.com' ),
( 'record','record','Burro Loco Rosado 2016','Spain','Castilla-y-León','Vino de la Tierra	','Concejo Bodegas SL (Producer)','+34 983 50 22 63','www.concejobodegas.com' ),
( 'record','record','Cancellaia 2012','Italy','Toscana','Toscana IGT	','Az. Agr. Pakravan-Papi (Producer)','+39 3356 00 44 46','www.pakravan-papi.it' ),
( 'record','record','Cardela 2014','Spain','Castilla-y-León','Ribera del Duero	','Bodegas Bohórquez, S.L. (Producer)','+34 915 64 37 51','www.bodegasbohorquez.com' ),
( 'record','record','Casa Ferreirinha Vinha Grande Red 2014','Portugal','Porto E Douro','Douro	','Sogrape Vinhos SA (Producer)','+351 227 86 80 08;+351 227 85 03 00','www.sograpevinhos.com' ),
( 'record','record','Casa Mayor Old Vines Cabernet Sauvignon 2016','Chile','Valle de Colchagua','Viña Casa Silva SA (Producer)','+56 7 22 71 65 19','www.casasilva.cl' ),
( 'record','record','Castello di Vicarello 2012','Italy','Toscana','Toscana IGT	','Castello di Vicarello (Producer)','+39 0564 99 04 47','www.castellodivicarellovini.com' ),
( 'record','record','Castillo Labastida Ermita Santa Lucia 2016','Spain','Rioja','Rioja Alavesa	','Araex Rioja Alavesa SL (Producer)','+34 945 15 05 88','www.araex.com' ),
( 'record','record','Castroviejo 2013','Spain','Rioja','Rioja DOCa	','Bodegas Pastor Diaz Sl (Producer)','+34 941 14 23 90','www.pastordiaz.com' ),
( 'record','record','Cava Amorany Brut Gran Cuvée','Spain','Cataluña','Cava	','Vid Vica SL (Producer)','+34 933 10 22 62','www.vidvica.com' ),
( 'record','record','Champagne Bauchet Millésime Premier Cru 2009','France','Champagne','Champagne	','Champagne Bauchet (Producer)','+33 3 26 28 17 72; +33 3 26 58 17 72','www.champagne-bauchet.com' ),
( 'record','record','Champagne Fresne Ducret La Grande Hermine 2009','France','Champagne','Champagne	','Champagne Fresne Ducret/SCEV JA MILAUR (Producer)','+33 3 26 49 24 60','www.champagne-fresne-ducret.com' ),
( 'record','record','Champagne Jean Dumangin Brut Millésime 2009','France','Champagne','Champagne	','EARL Jean Dumangin (Producer)','+33 3 26 03 42 17','www.champagne-jean-dumangin.fr' ),
( 'record','record','Champagne Marc Perla-Rosa 2012','France','Champagne','Champagne	','Champagne Marc (Producer)','+33 3 26 58 46 88','www.champagne-marc.fr' ),
( 'record','record','Champagne Vincent Lamoureux Rosé','France','Champagne','Champagne	','EARL Lamoureux Vincent (Producer)','+33 3 25 29 39 32','www.champagne-lamoureux-vincent.fr' ),
( 'record','record','Château La Fleur Peyrabon 2014','France','Bordeaux','Pauillac	','Château Peyrabon (Producer)','+33 5 56 59 57 10','www.chateau-peyrabon.com' ),
( 'record','record','Château Moulin du Terrier 2016','France','Bordeaux','Bordeaux Rouge	','GAEC Forcato (Producer)','+33 6 71 21 57 71','www.chateau-lary.com' ),
( 'record','record','Château Ventenac Carla Rosé 2016','France','Languedoc-Roussillon','Cabardès	','Maison Ventenac (Producer)','+33 4 68 24 93 42','www.maisonventenac.fr' )

@thadguidry

@tcbuzor I think this is feature complete now from my side.
@jackyq2015 @ettorerizza could you test locally also and give it a review also ?

Some observations:

DROP TABLE triggers an error if the table does not exist. Ideally, DROP IF EXISTS should be used, but I do not know if it's standard SQL.

The SQL script exported produces an error in the database ("all VALUES must have the same number of terms") if the OR project contains null (try with file-attached). Possible solution: transform all null into empty strings just before export.

The system does not handle records (one to many relationship). In my example, it would ideally be necessary for the column "instanceof", which is multivalued, to become a separate table linked to the principal by a foreign key. If it's too complicated, then the ideal would be to perform a "fill down ALL" just before the export, in order to get a correct denormalized table. But this function does not exist yet in Open Refine (the fill in is also imperfect and to be used with caution).

@ettorerizza I would rather not force opinion but think we should instead give the user an option to "Possible solution: transform all null into empty strings just before export."
The "handle records as relationships" for SQL export would be an enhancement, and THAT would have a multitude of options since users differ on their needs.  I speak from experience with this.
@ettorerizza So... Go ahead and open an new enhancement issue for your design thoughts on "handle records as relationships" and provide drawings even if you can about how you ideally would see an interface to deal with that and we can have @tcbuzor Tony work on that in a future enhancement...and ideally once we get our front / backend separated.

I would rather not force opinion but think we should instead give the user an option

@thadguidry I agree that the user should have the choice of how he treats nulls. But for the moment, he has absolutely no choice. If a cell is null, the SQL script will not work.
In other words, if you have this:

The export will produce this, which is incorrect.
CREATE TABLE clipboard (
Col1 VARCHAR(255),
Col2 VARCHAR(255)
);
INSERT INTO clipboard (Col1,Col2) VALUES
( 'A','B' ),
( 'C' ),  <------------ incorrect
( 'D','E' )

Go ahead and open a new enhancement issue for your design thoughts

I will try to draw that. The opinion of @gobertm could be interesting on this question.

@wetneb , @ettorerizza I am working on the reported bug. I will add an option to "convert null cell to empty string" that way a valid sql insert statement will be generated. This will work for VARCHAR, TEXT and CHAR types only. Need to decide what to do when the user selects a numeric type. Return an error maybe?

@tcbuzor or you could just change the table schema so that null values are accepted in all columns, maybe? SQL has support for NULL, so why not use it?

@ettorerizza Sorry for late reply . Didn't  see the notification.
I think the best solution is to insert null.
INSERT INTO clipboard (Col1,Col2,Col3) VALUES
( 'A','B','B' ),
( 'C',null,'D' ),
( 'D','E','E' )

This will be correct in regards of the input. Replacing by an empty string is not the same. An empty string is not null.
If I perform a Select * from clipboard where Col2 is null  I wont' get the line if it's replaced with an empty string.
Moreover it may be useful to add a "Not null" option in the export columns options. Because by default a CREATE TABLE t (   Score INT(2) );  allows null values. It will therefore be CREATE TABLE t (  Score INT(2) NOT NULL);  or even allowing the user to enter a default value in case of undesired null values in the input CREATE TABLE t (  Score INT(2) NOT NULL DEFAULT 0 );

@tcbuzor Just insert a null, as discussed above. We all upvoted.

Just checked in the latest export code. I added 2 columns(Allow Null , Default Value). If the user indicates the column is not Nullable then he has the option of adding a default value, which will be used in the insert statement when the field value is NULL.

@jackyq2015 I'll need your eyes on this as final verify, please.  Take a look. Thanks.


RLN Proof generation for demo

@0xmad I think we need to mark this issue with #104 PR right?

@iskdrews we can mention it, it's not a complete feature.


ApiModelProperty annotation generates automatically example for Boolean

I'm working in Scala.
I defined my case class as follow:
import io.swagger.annotations.{ApiModel, ApiModelProperty}

@ApiModel(value = "TokenIntrospectionResponse")
case class TokenIntrospectionResponse(@(ApiModelProperty@field) @BeanProperty val active: Boolean)

When I read the generated schema, I see:
"definitions": {
   "TokenIntrospectionResponse": {
      "type": "object",
      "properties": {
        "active": {
            "type": "boolean",
            "example": false
         }
      }
   }
}

If I replace the type of active property from Boolean to String, the result of the Swagger doc generation is:
"definitions": {
   "TokenIntrospectionResponse": {
      "type": "object",
      "properties": {
         "active": {
            "type": "string"
         }
      }
   }
}

If I remove the ApiModelProperty annotation, the example is not generated for type Boolean:
"definitions": {
   "TokenIntrospectionResponse": {
      "type": "object",
      "properties": {
         "active": {
            "type": "boolean"
         }
      }
   }
}

Is it a bug ?
Is there a way to keep this ApiModelProperty annotation and delete this example property for boolean property ?
If not, is there an existing workarround ?

For your information, I need this annotation to specify this property is required.
I need to erase this example property because Amazon API Gateway is not supporting this one (source):

API Gateway supports most of the Swagger specification, with the following exceptions:

[...]
The example tag is not supported.

Thanks in advance.

is there someone to help me please ?

@lhauspie Yeah I think this is a bug too, ApiModelProperty defaults example to the empty string, for all numeric Properties this is being mapped to null as parse is throwing NumberFormatException and not modifying the null example.  However Boolean#parseBoolean behaves differently parsing the empty string (and null) as false.  I believe that BooleanProperty#setExample needs to treat these as special cases.

I've created the following pull request
https://github.com/swagger-api/swagger-core/pull/2561


Request trove classifier

Request to add a new Trove classifier.
The name of the classifier you would like to add

Your :: Trove :: Classifier

Why do you want to add this classifier?

It looks like this was opened accidentally. If you'd still like to request a classifier, please fill out the requested information and re-open.


Some images broken on case sensitive platforms

A number of images are looking into the folder /Art instead of /art on case sensitive platforms causing 404 errors.
Error: Invalid Asset modules/starfinder-field-test-for-pf2e/Art/npc-art/tokens/pzo9500-25_cybernetic_zombie-t_v2_1-5.webp
Error: Invalid Asset modules/starfinder-field-test-for-pf2e/Art/npc-art/tokens/corpsefolk_operative-t_v2_1-5.webp
Error: Directory modules/starfinder-field-test-for-pf2e/Art/maps does not exist or is not accessible in this storage location

Already fixed, duplicate of #13


20 reestablish socket communications for front and backend

Update the and reids socket logic so the frontend can receive notifications, messages and progress updates from rair-node, rair-sync and rair-stream.

QA verified


D365 Finance and Operation On-Premises Installation

Hi,
I completed production and sandbox environment installation.
I can sign in  and using sandbox environment, But I can't sign in production.
ADFS login page not open for production.
In ADFS Server:
I have 2 application, and check application link. Its correct.
I checked production topology.xml and I dont have a problem for ADFS Production App.
I checked LCS, Service Fabric, AOS Event, ORCH events and I don't see any problem.
what can i do for this problem?
Thanks.

Thanks for contacting us. Unfortunately, we're unable to handle Support requests for the product here. To contact Support, consider one of the options described here: Support contact options. In particular, you might want to go to the Dynamics 365 for Finance and Operations community page to get help with your questions from the Dynamics community.


enhancement update FilterEstimation algorithm

Proposed changes

after filter "T1.A=c", column stats for A should be updated
skip DORIS_DELETE_SIGN=0 in stats derive,
after filter, we need to update column ndv. use new formula:
newNdv = originNdv * (1 - Math.pow(1 - sel, originRowCount / originNdv))

Issue Number: close #xxx
Problem summary
Describe your changes.
Checklist(Required)

[ ] Does it affect the original behavior
[ ] Has unit tests been added
[ ] Has document been added or modified
[ ] Does it need to update dependencies
[ ] Is this PR support rollback (If NO, please explain WHY)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall

run buildall


Docs refer to makeConfig but should be initConfig?

Docs: https://ohdsi.github.io/Ulysses/articles/start_study.html#config-yml
makeConfig doesn't exist, but I checked and initConfig appears to serve this same purpose.

whoops...I forgot to change this in the docs. thanks for catching that.
Follow allow this vignette to set up the config.yml file https://ohdsi.github.io/Ulysses/articles/credentials.html

Should be correct in v006 see #22
Closing

Warning message for addConfig still refers to makeConfig


Update SDK to preview 7

Addresses aspnet/AspNetCore-Internal#2774

(Still wondering why we have to do this manually...)

@bricelam because the SDK doesn't exist in a Maestro++ channel and Arcade's minimum SDK lags, then lags some more.

Should we re-open dotnet/arcade#811? (or at least file a new issue)

The flow from Arcade works. The problem I was talking about was just them updating the SDK reactively i.e. when someone mentions it's lagging. Then, they need to notify Arcade users and wait a bit before changing. We generally want to move faster.
That said, I could open another Arcade PR to start their process. Is that important at this point?

I’m fine with it as it is (since I’m not the one doing it) it just feels like arcade didn’t deliver the feature we asked for

I.e. we should be able to opt into the latest SDK, not a “minimum required” version


port to .net framework 4.7.x

we suggest creating a separate branch with library version for the classic .net framework
we have ported library under .net 4.7.2

it says I want to merge that to ArkEcosystem:master
that obviously not my intention, need separate branch for that

This is a rather nasty and impractical solution, a better approach is to add multi-framework support by using TargetFrameworks.


Get/Create/Join spaces by user account address

Is your feature request related to a problem? Please describe.

I need to search list of spaces by user account address
I need to create and join spaces using snapshot.js but the documentation only has a placeholder .... When will the api be ready?

Describe the solution you'd like
await client.spaces.get("client account address")
await client.spaces.create("awesome.eth", "client account address")
await client.spaces.join("awesome.eth", "client account address")
Additional context
The graphql query for multiple spaces doesn't allow searching by members. Can add there as well.

Yes we should document how to follow and unfollow spaces using snapshot.js

bump to document this


Contact Send Now issue solved

Related Issue
Fixes:  #1272
Description
I have solved the unnecessary rendering issue that occurred when clicking the 'Send Now' button. The form now displays a 'Message sent' notification, providing a better user experience.

Type of PR

[x] Bug fix
[ ] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)

Checklist:

[x] I have made this change from my own.
[x] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@huamanraj please check it and merge !!

Conflicting files
index.html
Please see the conflicts highlighted in the image above. Resolve the conflicts in the mentioned files. Ensure you pull all the latest changes before pushing your feature/bug fixes to the main branch. After resolving the conflicts, please open a new PR for review.
Thank you!


Chore/add codecov integration

:tada: This PR is included in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Enable Snyk scanning

addresses #2087

Ouch…236 open issues.  That’s painful.
The three “high” issues are the same thing and are likely fine.  We’ll need guidance on how to mark “uninteresting” issues — presumably that is the docs.
Thanks for doing this, @ryjones — awesome stuff.
Aside — I had to edit the “github-code-scanning” comment, as the link to the overview was bad — unusable in email.  Anything we need to do about that?

Several of the low severity ones would be taken care of by #2310, I think

@swcurran I think the reason it didn't work was it didn't exist yet

@swcurran I think the reason it didn't work was it didn't exist yet

I don’t think so.  When I fixed the link, I could navigate to the content, so it existed. The link started with “hyperledger/aries-cloudagent-python…” — it didn’t have github.com in the URL. Might have been related to the link going into an email.  Anyway, we can deal with that later.

@swcurran sounds good. Please approve and merge


Add the setup_test_with_server/1 hook for test_with_server.

Solves Issue #29

Coverage increased (+0.003%) to 95.631% when pulling 08ef822df63484eebbc0bbafcd55e463197e8df7 on fcapovilla:setup_test_with_server into eebca5ca4d40b2937cd04a641de76b23c04f72d7 on bernardolins:master.

Coverage increased (+0.003%) to 95.631% when pulling 08ef822df63484eebbc0bbafcd55e463197e8df7 on fcapovilla:setup_test_with_server into eebca5ca4d40b2937cd04a641de76b23c04f72d7 on bernardolins:master.


test: prevent resource leaks from go timeout

Fixes #N/A
Description
When go timeouts happen, the test suite panics and does not enter after suite clean up steps. This allows the ginkgo timeout to be run before the go timeout, which actually runs the after test clean up.
How was this change tested?
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 79.105%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
91.67%

Totals

Change from base Build<PHONE_NUMBER>:
-0.01%

Covered Lines:
5940

Relevant Lines:
7509

💛  - Coveralls


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/149 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit 1077448:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Cauldron JEI Integration

Hello!
The Cauldron JEI Integration no longer works in the latest versions of JEI.
Pressing U on a Cauldron yields this:

Although it does in fact have functional recipes.
Versions

Inspirations-1.12.2-0.2.3
jei_1.12.2-<IP_ADDRESS>

Please give me a copy of your game log, pretty sure this is the same as another reported issue

Here you go: https://gist.github.com/NillerMedDild/5b8c033eecd0d3495ad7a5d67e0f7298
I updated JEI to 245, no change.

Yep, duplicate of #97. You should be able to fix by removing the missing item from the anvil smashing recipes (possibly mulch if you disabled that). I'll have a proper fix along shortly

Oh I see, thank you.


spec: dependencies should have "size"/"length"

Dependency references should have an optional "size" or "length" field to allow clients to prevent against DOS attacks  (e.g.) when discovering and retrieiving them.

/cc @vbatts @philips

LGTM.
As an aside, it might be worth extending the discovery handling to cover doing HEAD requests to the ACI endpoint to look up the size as well. LMK, I could write that up.

@krobertson sounds great!


[Tests] NFC: Disable @preconcurrency conformances executable test o…

…n backdeployment runtime

@swift-ci please smoke test


fix: when use LogViewer with big size(more then 30M maybe) logs cause…

… maximun call stack error
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

fix https://github.com/backstage/backstage/issues/21927


date fields in FireFox

fantastic proof of concept, unfortunately FireFox (latest stable) does not support the datetime input type and the validation message therefore becomes confusing to the user

Yes, this is right. Thanks! I plan to do more work on the theming. In this case I think that the best solution is to provide an additional widget that renders year/month/day[/hour/minute/second] selectors.

I was wondering if rendering multiple input's using object or collection name convention could work and be bound to a specific get set on the date?
Date component
fieldname.year or fieldname[year]
fieldname.month or fieldname[month]
fieldname.day or fieldname[day]
Time component
fieldname.hour or fieldname[hour]
fieldname.minutes or fieldname[minutes]
fieldname.seconds or fieldname[seconds]
That would also make it a little easier to truncate to say the nearest minute or hour, or even skip the date altogether.
Personally I use jQuery UI datepicker for dates; I'm aware it's a bit old-school, but as it's externally maintained, cross-browser compatible to quite an extent, works on mobile and is somewhat accessible; it seems like the easiest way to get dates out the door. For times I usually use two-three separate inputs, but have also been known to use select boxes because of their keyboard and mobile experience.
I get that jQuery UI might bloat this, and I wish firefox just had half-decent support for the date and datetime input types, but it's looking like that won't change any time soon, so maybe a better error message or UI helper, or even third-party API could fix?

Yes, something like what they have in https://mozilla-services.github.io/react-jsonschema-form/ (Date&Time section) would work I think. We already support arrays, and it is not a problem to have deeper hierarchies. I guess that the first step would be to have a widget that is collection of selects, and make the value that is actually stored a string, kind of what mozilla is doing. And then provide an additional option to store the state as a collection of individual values month/day/year.
I also thought about external dependencies for datepickers. I use them (I personally don't share the hate many people express to jQuery based solutions), but I lean against having jQuery + a datepicker as a dependency.
In the particular project where the need for this library was born we use a very custom theme, with selectors for year, month and day, and for time pickers (not date time, only time), the customer required sliders with 15minute steps, so in practice the widgets don't have many restrictions, redux-form is flexible enough so we can do almost anything we want.

Re-reading your comment and to clarify: object would work, yes, but it would render the fields in vertical, and my guess is that writing a widget for this (similar to collection) would be more what the user expects.

@nacmartin by object syntax I am only talking about the names of the output display components to a single component definition, not using multiple component definitions. The JSON would look identical. It'd be the way that is rendered that would change and require a more complex object.

I have added two new widgets: compatible-date and compatible-datetime.
compatible-datetime is now on the main page of the docs (https://limenius.github.io/liform-react/#/), and both of them are shown in the "All widgets" section of the examples: https://limenius.github.io/liform-react/#/examples/all-widgets
At the end I opted to store always dates and date-times internally as a string, because the library ajv for validation likes that format.
I guess that these two provide some basic date inputs that people can use. After all, if someone wants to use their favourite datepicker they can just write a widget and add it to the theme locally.
I am closing this issue, but feel free to reopen if you feel like.


pyparsing.exceptions.ParseSyntaxException: Expected ')', found end of text

Describe the bug
pyparsing.exceptions.ParseSyntaxException: Expected ')', found end of text
To Reproduce
My code (to put in a launch.py file)
domain.txt
problem.txt

import unified_planning
from unified_planning.shortcuts import *
from unified_planning.io import PDDLReader, PDDLWriter

reader = PDDLReader()
pddl_problem = reader.parse_problem('./domain.pddl', './problem.pddl')
print(pddl_problem)

# Get the compiler from the factory

dcr_kind = pddl_problem.kind

dcr_problem = pddl_problem

# create a version of the "problem" without negative preconditions
with Compiler(
        problem_kind = dcr_kind,
        compilation_kind = CompilationKind.NEGATIVE_CONDITIONS_REMOVING
    ) as negative_conditions_remover:
    # After we have the compiler, we get the compilation result
    ncr_result = negative_conditions_remover.compile(
        dcr_problem,
        CompilationKind.NEGATIVE_CONDITIONS_REMOVING
    )
    ncr_problem = ncr_result.problem
    ncr_kind = ncr_problem.kind

    assert not ncr_kind.has_negative_conditions()

    with OneshotPlanner(name='pyperplan') as planner:
        result = planner.solve(ncr_problem)
        if result.status == up.engines.PlanGenerationResultStatus.SOLVED_SATISFICING:
            print("Pyperplan returned: %s" % result.plan)
        else:
            print("No plan found.")

then
python launch.py (also put the domain.pddl and problem.pddl in same folder, see files attached)
domain.txt

I tried your code and -after a long parsing- I get this plan by pyperplan:
Pyperplan returned: SequentialPlan:
    a4022moins74plus102moins290
    a169plus290
    a3516moins290
    a1382plus102plus180plus290
    a323plus74moins290
    a274moins74plus102plus290
    a33moins180moins290

I avoided the problem print because it's quite long.
Do you still have the issue with the files you sent here?

I think what's take so much time is the compilation for taking off the neg preconditions. It seems to work now anyway. Thanks


Widgets/InputGroup children type checking

Would be great if InputGroup didn't explicitly check for your own "Input" type. Now it's incompatible with any other Component which might decorate your Input component. eg. http://redux-form.com/6.0.0-alpha.4/docs/api/Field.md/

Bump.

+1


Fix buffer overflow error in KryoBackedDecoder [Backport 6.x]

See #18316 and #18338

PR is merged


ARROW-12110: [Java] Implement ZSTD compression

Add ZSTD codec implementation.
Removes dependency on netty by using ArrowBuf methods.
Updates docs and archery test (will wait on CI to run archery if that fails will do more debugging.)

I ran archery tests for java locally and zstd seems to pass with this PR.  I'm not sure how these changes could have affected the archery/crossbow tests.
@dianaclarke I think you might have been the last person to touch this area?  Any thoughts?

I ran archery tests for java locally and zstd seems to pass with this PR. I'm not sure how these changes could have affected the archery/crossbow tests.
@dianaclarke I think you might have been the last person to touch this area? Any thoughts?

I'm pretty new to Arrow, and I'm not quite sure I follow. What's failing?
The Archery & Crossbow / Archery Unittests and Crossbow Check Config check passed.
Are you referring to the Integration / AMD64 Conda Integration Test (pull_request) check? It appears to be out of disk space.
npm ERR! nospc ENOSPC: no space left on device, write
npm ERR! nospc There appears to be insufficient space on your system to finish.

I'm probably too n00b to be of much help on this issue, but good luck!
That said, I do hope to find some time to collaborate on the following PR with you all.
ARROW-10031: [CI][Java] Support Java benchmark in Archery
https://github.com/apache/arrow/pull/8210

I would be fun to get these Java benchmarks hooked up to Conbench. I have a blog post coming soon that better explains the Arrow/Archery/Conbench integration. Until then:
https://conbench.ursa.dev/
https://github.com/ursacomputing/conbench
https://github.com/ursacomputing/benchmarks

Cheers & stay safe!

@dianaclarke hmm, it looks like "Archery & Crossbow / Archery Unittests and Crossbow Check Config (pull_request" passed this time, it seems like it might be flaky (i don't know that there is a great way to see old runs).  Yeah, I think the integration test failure is completely unrelated and there is already an open JIRA for it.

I am curious to know about the performance of this codec (I suppose it should be good, as there are no on-heap/off-heap data copies).
Maybe we can run some benchmarks after ARROW-11901 checks in the benchmarks.

Yes, I expect it to be good.  I'm going to try to run some end-to-end experiments and I can report back numbers.

@dianaclarke https://github.com/apache/arrow/pull/9822/checks?check_run_id=2219940455

@dianaclarke https://github.com/apache/arrow/pull/9822/checks?check_run_id=2219940455

Ah, now I see. I didn't write that test. I think @bkietz did, but I can try to find some time this week to dig into it.

@dianaclarke https://github.com/apache/arrow/pull/9822/checks?check_run_id=2219940455

Ah, now I see. I didn't write that test. I think @bkietz did, but I can try to find some time this week to dig into it.

I see what the issue is. That test is assuming order, when it can't. I'll create a ticket and submit a PR shortly.
[Developer][Archery] Flaky test: test_static_runner_from_json
https://issues.apache.org/jira/browse/ARROW-12145

@liyafan82 I opened https://issues.apache.org/jira/browse/ARROW-12163 to track making compression levels configurable, any other concerns?

@liyafan82 I opened https://issues.apache.org/jira/browse/ARROW-12163 to track making compression levels configurable, any other concerns?

@emkornfield My only concern is performance. If we are going to address it in ARROW-11901, this PR is ready for merge IMO.

@emkornfield My only concern is performance. If we are going to address it in ARROW-11901, this PR is ready for merge IMO.

Yes, I think functionality should come first and we can measure and iterate on performance.  Thank you for your review.

going to merge.


IPNS support for immutable share links

Is your feature request related to a problem? Please describe.
When linking directories or files in IPFS, it's better to change the content without the link changing, You can use those URLs for downloads for smart contracts.
Describe the solution you'd like
We currently use IPNS so that way we can update the data under the hood and keep an immutable pointer to the directory. I wonder if you could take a similar approach with shared resources?
Describe alternatives you've considered
Just using s3!

You can certainly use IPNS outside of the minio gateway to refer to the content that you store on IPFS via the gateway. However the issue with using IPNS within the gateway is that it is exceptionally slow, and the publish times would make using s3x as a backend unrealistic.

Right, I was thinking just that, as part of the share download functionality

Share download functionality? Could you explain more about this. I wasn't aware S3 had a method of sharing access to folders and objects

I meant to enable https://docs.min.io/docs/minio-client-complete-guide#share
Share downloads

ah i see thanks for the clarification, this is certainly something that we will look into


Add dropin file support for .network files for networkd

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 7
Ruby: 2.7
Distribution: Ubuntu
Module version: 7.1.0

Since systemd 232 it has supported dropin files for network interfaces. It would be good to get support here for those as well.
eg /etc/systemd/network/eth0.network.d/routes.conf

I guess the implication is that you're not puppet-managing the .network files, but you just want to puppet-manage some modifications using the dropin. I haven't found a need for this because I have been managing the .network files, but I suppose this could be useful in some use cases. It's analogous to managing dropin files for service units, but not managing service units themselves because the package management system does that.


sched/sched_timerexpiration.c: change from sched_time to eventtime

Summary
change from sched_time to eventtime to overcome the build error: 'struct sporadic_s' has no member named 'sched_time' , becuse the member to save  the last time that the scheduler ran is "eventtime" rather than "sched_time"
Impact
sporadic sched
Testing
ci

@zouboan to avoid the break in the future, it's better to enable the sporadic policy in some config

@zouboan will you add a sample config (for sim or any other target) that enables sporadic policy in this PR? If yes, then I will postpone the merge of this PR until you add a config.

Please squash to a single commit.

@pkarashchenko I enabled sporadic sched in sim:foc , because i have tesed sporadic sched in my custom board about foc


Update storageclass apigroup in csv

Fixes the Failed to watch error logs
Failed to watch *v1.StorageClass: unknown (get storageclasses.storage.k8s.io)

Fixes https://github.com/konveyor/volume-snapshot-mover/issues/120

Codecov Report

Merging #766 (98f9f72) into master (5b41bea) will decrease coverage by 2.74%.
The diff coverage is 7.77%.

@@            Coverage Diff             @@
##           master     #766      +/-   ##
==========================================
- Coverage   36.96%   34.22%   -2.75%
==========================================
  Files          16       16
  Lines        3219     2840     -379
==========================================
- Hits         1190      972     -218
+ Misses       1926     1778     -148
+ Partials      103       90      -13

Impacted Files
Coverage Δ

controllers/datamover.go
30.30% <0.00%> (-4.48%)
:arrow_down:

controllers/registry.go
29.23% <0.00%> (-10.97%)
:arrow_down:

pkg/common/common.go
100.00% <ø> (ø)

pkg/credentials/credentials.go
23.69% <ø> (+0.28%)
:arrow_up:

controllers/velero.go
48.28% <43.75%> (-0.64%)
:arrow_down:

Continue to review full report at Codecov.

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

/retest

/retest

/retest

Don't forget to ping @rayfordj to update CSV downstream


arrayCumSum function

This PR adds cumulative sum function.
When you are working with some specific data arrays is useful to store them using deltas. When CH stores these delta encoded arrays the compression ratio is higher than the one achieved with the original data (in some cases it's several orders of magnitude in near-constant arrays) and allows to save disk space (and likely execution time even with the cumsum overhead).
arrayCumSum allows decoding those delta encoded arrays so you can work with them as regular arrays.
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Great!

I've tried this variant of inner loop:
        size_t pos = 0;
        for (size_t i = 0; i < offsets.size(); ++i)
        {
            Result sum{};

            for (; pos < offsets[i]; ++pos)
            {
                sum += data[pos];
                res_values[pos] = sum;
            }
        }

It is more obvious. But there is no difference in performance (or it is slightly worse). So, I will leave your variant.

Looking forward for arrayCumReduce function :)

what do you mean with arrayCumReduce? something like cumsum but more generic?:
:) select arrayCumReduce(x, y -> y - x, [1, 2, 3, 4]
[1, 1, 1, 1]

I was thinking about the reverse method:
:) select arrayDiff([100, 102, 103, 104])
[100, 1, 1, 1]

Other things I think would be useful are fillna (to fill nan values) but I don't know if spending time on those methods is actually useful since they could be a mix of general functions + array join
Thanks for testing that variant, it's indeed simpler to read

Examples (along with arrayReduce function, that is already exists in ClickHouse):
arrayReduce('avg', [1, 2, 3]) = 2
arrayCumReduce('avg', [1, 2, 3]) = [1, 1.5, 2]

arrayReduce('uniq', [10, 11, 11, 12]) = 3
arrayCumReduce('uniq', [10, 11, 11, 12]) = [1, 2, 2, 3]

arrayReduce('corr', [1, 2, 3], [1, 0, 1]) = 0
arrayCumReduce('corr', [1, 2, 3], [1, 0, 1]) = [nan, -1, 0]

Implementation looks very straightforward. Please give it a try!

Ok, I get it, makes sense.
I was thinking about (and sorry about the offtopic) working on some methods to allow easier geospatial filtering/analysis. I worked with CH in the past and worked really great (I wrote a blogspot  about it which BTW is on your front page, thanks!)
You recently added support for pointInPolygon which is one of the pillars of geospatial analysis, I think it would be interesting to add:

quadkey/hilbert curves support to allow indexing. Kind of easy and works great with CH indexing mechanism.
knn search. In general closer stuff is correlated so to do fast analysis having that kind of thing is interesting.

With that CH would be the best geospatial analytical database out there.


Improve ProductClass implementation

[x] Rename Product Class to Product Type in UI
[x] Add more verbose labels to particular attributes fields - common for all variants on product attributes field and specific for each variant for variant attributes
[x] Rename Migrated product class to Unnamed product type
[x] Display more details on product types list (product and variants attributes)
[x] Simplify URL scheme, now we have /products/base/<class_pk>/add Change to /products/add/<class_pk>
[x] Add a help text that will explain why user needs a product class

@artursmet  "Add a help text that will explain why user needs a product class" - where that info should be placed?

@krzysztofwolski We can initially place this text below the product class form. As you suggested, we should find a way to display similar help messages in other dashboard views. But this task requires more time for research purposes.

@weronka Could you look at product class list? I'm not sure if my solution for presenting attributes is good enough. Could you suggest something better?


[fix][cli] Dedicated docs tools

This closes #18932.
This is a follow-up to https://github.com/apache/pulsar/pull/20687. We parse ApiPropertyModel annotations when generate docs in the site repo.

https://github.com/apache/pulsar-site/blob/e0557ec306143b50d150b7b143ca7fcb9ef0f89e/tools/pytools/lib/execute/config_doc_generator.py
https://github.com/apache/pulsar-site/actions/runs/5428385621

Factor out a module to handle dependency issues. Also payback some debt that inlines CmdDocGen in core modules.
Verified locally.
I'm preparing a counterpart in the site repo now. See https://github.com/apache/pulsar-site/pull/628
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

I'm trying to solve https://github.com/apache/pulsar/issues/18932 also and avoid compatibility hack in the site repo. I have an idea and am implementing it.

Codecov Report

Merging #20700 (1577436) into master (e360379) will increase coverage by 0.41%.
The diff coverage is 58.04%.

@@             Coverage Diff              @@
##             master   #20700      +/-   ##
============================================
+ Coverage     72.69%   73.11%   +0.41%
- Complexity    31878    32092     +214
============================================
  Files          1871     1868       -3
  Lines        138982   138937      -45
  Branches      15283    15275       -8
============================================
+ Hits         101037   101585     +548
+ Misses        29873    29306     -567
+ Partials       8072     8046      -26

Flag
Coverage Δ

inttests
24.19% <23.52%> (+0.07%)
:arrow_up:

systests
24.98% <21.00%> (-0.06%)
:arrow_down:

unittests
72.40% <58.04%> (+0.46%)
:arrow_up:

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

Impacted Files
Coverage Δ

...in/java/org/apache/pulsar/PulsarBrokerStarter.java
34.13% <ø> (ø)

.../org/apache/pulsar/PulsarClusterMetadataSetup.java
72.51% <ø> (ø)

...g/apache/pulsar/PulsarClusterMetadataTeardown.java
75.75% <ø> (ø)

...org/apache/pulsar/PulsarInitialNamespaceSetup.java
75.00% <ø> (ø)

...ava/org/apache/pulsar/PulsarStandaloneStarter.java
35.38% <ø> (ø)

...sar/PulsarTransactionCoordinatorMetadataSetup.java
63.15% <ø> (ø)

...n/java/org/apache/pulsar/PulsarVersionStarter.java
42.30% <ø> (ø)

...pache/pulsar/broker/tools/GenerateDocsCommand.java
77.27% <ø> (ø)

...a/org/apache/pulsar/broker/web/RequestWrapper.java
54.54% <ø> (ø)

...pache/pulsar/broker/web/ResponseHandlerFilter.java
57.14% <ø> (ø)

... and 20 more

... and 118 files with indirect coverage changes

Pending to merge...


[Question] Auto scaling with IRSA enabled - is this supported for self-hosted?

Hi,
For a kops deployed cluster with IRSA enabled is there any recommendation on how to address auto-scaling. I'm seeing the node auto scaling group successfully provision a new instance, but it subsequently fails to join the cluster, I assume due to the node role only have assumeRole permissions.
Are you aware of any workarounds/activity around this on the kops/IRSA sides?
Thanks,
John

For a kops deployed cluster with IRSA enabled is there any recommendation on how to address auto-scaling. I'm seeing the node auto scaling group successfully provision a new instance, but it subsequently fails to join the cluster,

I don't think that IRSA is related because the webhook and auth flow only operates on pod creation.
How are you doing node authentication?

I assume due to the node role only have assumeRole permissions.

What role does your node assume? Are you trying to pull any ECR images? Did you grant ECR pull permissions?

How are you doing node authentication?

I'm not clear on what the "node authentication" is you're referring to in this context, could you clarify?

I assume due to the node role only have assumeRole permissions.

What role does your node assume? Are you trying to pull any ECR images? Did you grant ECR pull permissions?

Kops initially creates a node role with EC2 list, S3 list and ECR read (needed for its nodeup utility to complete cluster node bootstrapping). After cluster creation has completed I've dropped those permissions and the node role just has sts:AssumeRole with the OIDC trust relationship. This works just as expected with the webhook and pods.
Now when the EC2 autoscaler starts a new cluster node in a node group, it uses the same node role as above, which now only has sts:AssumeRole. The nodeup tool still needs access to the EC2/S3/ECR resources, but since the node role is now reduced this fails. I assume I'll have to update nodeup to use AssumeRoleWithWebIdentity and somehow generate a signed JWT to assume a new role with the same permissions as the original node role - this would be happening outside of the cluster, not via the webhook. It's this last part I'm not clear on, whether there may be some alternative I'm not aware of (very likely).
Thanks for providing feedback, hopefully this is a bit clearer.

Kops initially creates a node role with EC2 list, S3 list and ECR read (needed for its nodeup utility to complete cluster node bootstrapping). After cluster creation has completed I've dropped those permissions and the node role just has sts:AssumeRole with the OIDC trust relationship. This works just as expected with the webhook and pods.

Ah right so, kops requires that S3 permission so that nodes can get the cluster CA and other certificates. I think that is the source of your problem. The node role will still need those permissions you removed for new nodes to join.
When you say

the node role just has sts:AssumeRole with the OIDC trust relationship. This works just as expected with the webhook and pods.

is the AssumeRole permission in the trust policy or the role policy?

Now when the EC2 autoscaler starts a new cluster node in a node group, it uses the same node role as above, which now only has sts:AssumeRole.

The "it uses" part, is that the autoscaler using itself? Or the role it assigns to new nodes in the ASG?

I assume I'll have to update nodeup to use AssumeRoleWithWebIdentity and somehow generate a signed JWT to assume a new role with the same permissions as the original node role - this would be happening outside of the cluster, not via the webhook. It's this last part I'm not clear on, whether there may be some alternative I'm not aware of (very likely).

JWTs for pods are created by the kube-api server (POST /api/v1/namespaces/<namespace>/serviceaccounts/<service account>/token) by the kubelet. The Kubelet will always be the root of trust for pods, and you'll need a way for the kubelet to authenticate to the cluster. Since you're using Kops, that is S3.
You'll need to re-enable the default IAM permissions on the node role for kops


Make mdanish-kh and russellbanks community moderators

cc @denelon
Microsoft Reviewers: Open in CodeFlow

@denelon / @stephengillie

We still need one back-end change as well.


WIP: Multi-peer conformance tests

Closes: #369

[ ] Referenced an issue explaining the need for the change
[ ] Updated all relevant documentation in docs
[ ] Updated all code comments where relevant
[ ] Wrote tests
[ ] Updated CHANGES.md

The test currently panics at 'Pre-condition of mark_witness_as_faulty violated: peer_id != self . primary && self . witnesses . contains (& peer_id)' (here in code)
This is probably because it marks the primary provider faulty here(?)
Maybe @romac can help here!
Run test:
cargo test run_multipeer_tests -- --nocapture
Gives output:

This is probably because it marks the primary provider faulty here(?)

Yes, it's a flaw in the current PeerList API, which I've attempted to fix in #397. I think you can disregard this error until we merge #397 and adapt this PR to the new, improved API.

On second thought, perhaps that's an actual bug in the supervisor. Will take a closer look.

So yeah it's a bug. We should check whether the faulty peer is the primary or not and call the appropriate method if that's the case (swap_primary), but the PeerList API currently does not let us perform this check. In #397, we now expose this information, but could envision having a single method on PeerList which would perform the check and dispatch to the right method for us.

Scratch that, the issue is actually the faulty witness is not part of the witnesses list. Need to investigate further to see why that's the case.

After merging #397 in this branch, the error goes away (likely due to a bug that was fixed PeerList where it would not perform primary/witness swap properly).
Tests then fail with ErrorKind::NoTrustedState(Trusted), because the light store was initialized with a Verified block instead of a Trusted one.
After fixing this, the first two tests pass but the third one fails with:
Running Test Case: Case: Trusted height=1, found conflicting valid commit at height=11 from only one of the two witnesses, should not expect error
with Primary Peer: node::Id(BADFADAD0BEFEEDC0C0ADEADBEEFC0FFEEFACADE)
Witness: ADBEEFC0FFEEFACADEBADFADADE0BEFEEDC0C0AD
Witness: BADFADBEEFC0FFEEFACADEAD0BEFEEDC0C0ADEAD
[light-client/tests/supervisor.rs:145] e = Error(
    Context {
        kind: NoWitnessLeft,
        source: None,
        backtrace: Some(

/* snip */

To be investigated further.

@Shivani912 Just merged master in now that #397 has been merged, and fixed the trusted state issue. We are now left with the NoWitnessLeft error.

Thanks @romac for your help here! I'll take care of the failing test

This is interesting... So the test case here is that there is a primary peer and 2 witnesses. One of the witnesses supports the data sent by the primary one and the other witness doesn't. Looking at the backtrace of the test, supervisor actually tries to replace the primary peer marking it "faulty". Shouldn't it mark the witness who sent conflicting data faulty?

Merged master in after merging #407. This got rid of the error above, but the test now fails because when we detect an actual fork, we raise a ForkDetected error, which the test is not expecting.
@Shivani912 @josef-widder @OStevan @melekes: What do you think should be the proper behavior when we detect a fork? Here is what we currently do:

If verification with the primary fails for any reason, we replace the faulty primary and perform verification again
Otherwise, if the fork detector suspects

a) a faulty witness, we replace the faulty witness and perform verification again
b) that a witness timed out, we replace the faulty witness and perform verification again
c) an actual fork, we report the fork, raise an error and abort

Case a) and b) seem correct to me, but perhaps we should replace the witness and keep trying to verify in case c), rather than abort?

@Shivani912 @josef-widder @OStevan @melekes: What do you think should be the proper behavior when we detect a fork? Here is what we currently do:

If verification with the primary fails for any reason, we replace the faulty primary and perform verification again

Otherwise, if the fork detector suspects

a) a faulty witness, we replace the faulty witness and perform verification again
b) that a witness timed out, we replace the faulty witness and perform verification again
c) an actual fork, we report the fork, raise an error and abort

Case a) and b) seem correct to me, but perhaps we should replace the witness and keep trying to verify in case c), rather than abort?

After our discussion on the call, it seems appropriate that if we detect an actual fork, option (c) is the right expectation here and test should be modified to expect so. This reveals an inconsistency between Go (v 0.33) and Rust code as the Go light client does not raise an error on finding a fork and simply ignores it, marks it faulty and moves to verify headers from other witnesses. Also, this have been improved in their latest (master) code and reports on finding a fork. (So not raising an issue for this anymore @milosevic )

After fixing this, the first two tests pass but the third one fails with:
Running Test Case: Case: Trusted height=1, found conflicting valid commit at height=11 from only one of the two witnesses, should not expect error
with Primary Peer: node::Id(BADFADAD0BEFEEDC0C0ADEADBEEFC0FFEEFACADE)
Witness: ADBEEFC0FFEEFACADEBADFADADE0BEFEEDC0C0AD
Witness: BADFADBEEFC0FFEEFACADEAD0BEFEEDC0C0ADEAD
[light-client/tests/supervisor.rs:145] e = Error(
    Context {
        kind: NoWitnessLeft,
        source: None,
        backtrace: Some(

/* snip */

Also, this test fails with NoWitnessLeft error which should instead be something like ForkDetected or FoundConflictingHeaders. This fix is outside the scope of this PR, but just noting it here.

Case a) and b) seem correct to me, but perhaps we should replace the witness and keep trying to verify in case c), rather than abort?

We're considering it as a future improvement. See https://github.com/tendermint/tendermint/issues/4694. However, it won't be so simple if it's a full fork (2/3+ forked the chain), in which case it's probably safer to stop everything.

Yes it seems like we want to do C and have this be an error instead of a panic.
We really should have clarity on the proper termination behaviour of light clients. I made an issue in the spec repo here https://github.com/tendermint/spec/issues/114.

Yes it seems like we want to do C and have this be an error instead of a panic.

Perhaps I wasn't clear, but the bullet points above describe the current behavior, not alternatives to be chosen from, so that's already what we are doing.
We indeed raise an error, and it's then up to the consumer of the API to either handle it or let it bubble up and eventually panic in the main function.

@brapse Should we merge this already until we get clarification in tendermint/spec#114?


Metasploit-framework crashed in latest upgrade for Kali Rolling-2016.1

Hi everyone,
Recently I upgraded my Kali Linux which was previously a version Kali-Rolling and after an upgrade I got Kali-2016.1. However, after a successful upgrade, I tried to run "msfconsole" to launch a Metasploit-Framework. However, it wasn't a successful startup and it crashed. I then removed it completely and then downloaded and set up a whole new master version of metasploit framework from GitHub. After performing all necessary set ups, I again tried to run "msfconsole" and had some new issues. I have given a snapshot of what I got in terminal. If anyone have got any solution or anything, please revert back with a comment. Thanks!
root@kali:~# msfconsole
Your bundle is locked to rake (11.2.2), but that version could not be found in any of the sources listed in your Gemfile. If you haven't changed sources, that means the author of rake (11.2.2) has removed it. You'll need to update your bundle to a different version of rake (11.2.2) that hasn't been removed in order to install.
Run `bundle install` to install missing gems.
root@kali:~#

If it's possible for you to reduce to a simpler reproduction case, that would be helpful to us - otherwise it'll probably take us longer to get around to trying to recreate from within the metasploit-framework.

Hi.
I was thinking, could the problem be with Rake (11.2.2)?

Or RVM? as first I used rbenv to install metasploit, however I was getting issues with it. And when I run this command: user@user-pc-SATELLITE-C55-A:/opt/metasploit-framework$ bundle exec gem list I get this:
rbenv: version ``2.3.1' is not installed ``(set by /opt/metasploit-framework/.ruby-version)
user@user-pc-SATELLITE-C55-A:/opt/metasploit-framework$

However, I did uninstall rbenv with sudo apt-get remove rbenv, so I'm just completely lost now.
Ty

Hi everyone,
At last, I did configured metasploit-framework with successfully working environment. Actually the problems were with Rake-11.2.2 and much more. However, I did uninstall it 7-8th time, and then, instead of installing it manually through termial, I preferred to do so using other install methods. As part of an installation process, I first downloaded Metasploit-Framework installer which comes with an extension as .run file type. Hence, installing this file wasn't much hectic for me and that gave me an installation interface. It is working now with regular updates.
Thanks all for your responses!

I'll close this issue then, if you need more help or want to re-open it, just say so :)


How to perform double flush

Hi,
is it possible to perform double flush on DocumentManager<
I have my own class with little modified behavior on flush method
/**
     * Flushes all changes to objects that have been queued up to now to the database.
     * This effectively synchronizes the in-memory state of managed objects with the
     * database.
     *
     * @param object $document
     * @param array $options Array of options to be used with batchInsert(), update() and remove()
     * @throws \InvalidArgumentException
     */
    public function flush($document = null, array $options = array())
    {
        if (null !== $document && !is_object($document) && !is_array($document)) {
            throw new \InvalidArgumentException(gettype($document));
        }
        $this->errorIfClosed();

        $this->unitOfWork->commit($document, $options);

        foreach($this->afterFlushDocuments as $documentTemp){
            $this->merge($documentTemp);
            $this->refresh($documentTemp);
            $this->persist($documentTemp);
        }

        $this->unitOfWork->commit($document, $options);

        $this->afterFlushDocuments = [];
    }

But instead of updating in the second flush, the objects are inserted.  Should  I call resfresh on the UnitOfWork?
The scenario is to automatically update some documents, but to perform this action I need the ID keys and I don't want to use listeners.

The scenario is to automatically update some documents, but to perform this action I need the ID keys

Document's IDs are generated during initial persist() and are available just after it, I think there's no need to hack flush method for this :)

Thank you, I wasn't aware of the fact, that ID is generated by doctrine. My initial thought was that MongoDB is handeling this stuff, just like the MySQL generate ID after insert if strategy is set to AUTO.
Tahnk you for this information. I find it really usefull. :-)

@OndrejPiza if you're using AUTO strategy you'll be getting new \MongoId()casted to string as an identifier and you would have to be a terribly unlucky man to get a collision


wishlist: camera: orthographic or perspective

option to change the camera type (and can be changed in notebook)

started working on this in the orthographic branch

[ ] integrate CombinedCamera (perspective & orthographic) including methods to switch
[ ] fix hyperball impostor (just noise)
[ ] fix sphere impostor (bounding box too small)
[ ] integrate OrthographicTrackballControls (also, make feel more similar to TrackballControls)
[ ] fix multi-sample antialiasing (needs offset for OrthographicCamera)
[ ] add camera switch to preferences

ongoing work in PR https://github.com/arose/ngl/pull/115

merged the PR, testing appreciated

can you create a branch in NGLView with updated NGL code for camera? And how to change camera type? I can test it but prefer to use notebook. :D

stage.setParameters({cameraType: "orthographic"})

so I did
cd ngl/build
`sh make.sh
then copy js/build/{ngl.js, three.custom.min.js} to NGLView. Nothing shows up. Open JS console and saw
[.CommandBufferContext]GL ERROR :GL_INVALID_FRAMEBUFFER_OPERATION : glClear: framebuffer incomplete

done. work nicely.


Added one-click reaction documentation

Updated documentation for https://mattermost.atlassian.net/browse/MM-26478:

Channel Basics: added headings, inline images, and one-click reactions
Using Emoji: added one-click reactions and inline images
Managing Account Settings: removed unnecessary content duplication in tabs, added one-click user setting

@justinegeffen - We'll be updating the existing "one-click" terminology in-product for this feature to "quick reactions" in a separate PR.


UIROLES-107: add tenantId to the query key to invalidate list of roles after deleting role

Purpose

UIROLES-107 - Authorization role is not removed from roles list right after being deleted.
Approach

Since to fetch roles list data used tenantId parameter provide the same parameter to delete mutation hook to invalidate roles list data query.

Pre-Merge Checklist
Before merging this PR, please go through the following list and take appropriate actions.

[ ] I've added appropriate record to the CHANGELOG.md
Does this PR meet or exceed the expected quality standards?

[ ] Code coverage on new code is 80% or greater
[ ] Duplications on new code is 3% or less
[ ] There are no major code smells or security issues

Does this introduce breaking changes?

[ ] If any API-related changes - okapi interfaces and permissions are reviewed/changed correspondingly
[ ] There are no breaking changes in this PR.

If there are breaking changes, please STOP and consider the following:

What other modules will these changes impact?
Do JIRAs exist to update the impacted modules?

[ ] If not, please create them
[ ] Do they contain the appropriate level of detail?  Which endpoints/schemas changed, etc.
[ ] Do they have all they appropriate links to blocked/related issues?

Are the JIRAs under active development?

[ ] If not, contact the project's PO and make sure they're aware of the urgency.

Do PRs exist for these changes?

[ ] If so, have they been approved?

Ideally all of the PRs involved in breaking changes would be merged in the same day to avoid breaking the folio-testing environment.  Communication is paramount if that is to be achieved, especially as the number of intermodule and inter-team dependencies increase.
While it's helpful for reviewers to help identify potential problems, ensuring that it's safe to merge is ultimately the responsibility of the PR assignee.

This works but it feels brittle, requiring the namespace value here on line 11 to be synchronized with useAuthorizationRoles line 25 while these hooks are not connected in any obvious way 😬 . Likewise, do we need the same in useCreateRoleMutation line 18?
At the very least we should add some comments in both places to point out the synchronization. Would it be better to export a common function, or try to establish a pattern for this kind of thing where we have closely related hooks for create/delete/read?

You are right. Just synchronized query and mutations by removing specified parameter key to namespace


<EMAIL_ADDRESS>does not build on Discover

<EMAIL_ADDRESS>currently does not build on Discover.
Workaround in place: after creating an environment that uses<EMAIL_ADDRESS>run spack remove<EMAIL_ADDRESS>followed by spack add<EMAIL_ADDRESS>If spack concretize was run before remove /path/to/env/spack.lock and rerun spack concretize.

Why doesn't MAPL build on Discover? Is it the compiler, or MPI? Strange that it doesn't build there of all places.

Missing libraries/includes (netcdf stuff). No idea/time to look into why.
mapl 2.11.0 builds fine (but it doesn’t build on macOS therefore the move to 2.12.3).
I would suggest that post release 1, we get the mapl spack update from NASA, hopefully this will solve the problems.

On Jun 22, 2022, at 12:13 PM, Kyle Gerheiser @.***> wrote:
Why doesn't MAPL build on Discover? Is it the compiler, or MPI? Strange that it doesn't build there of all places.
—
Reply to this email directly, view it on GitHub https://github.com/NOAA-EMC/spack-stack/issues/213#issuecomment-1163454486, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5C2RNDJ2K6QNIUI5V3EY3VQNJTJANCNFSM5Y76QQFQ.
You are receiving this because you were assigned.

The duct-tape solution to build<EMAIL_ADDRESS>instead on Discover was applied in https://github.com/NOAA-EMC/spack-stack/pull/214. Will keep this issue open for the future (not as part of the first release of spack-stack).

Funny (well ...) - with spack-stack-1.0.0-rc2, I can build<EMAIL_ADDRESS>on Discover without problems?!? Error not always reproducible? Sigh.


Small thing in Stability in Linear Rational Expectations Models

A small thing regarding Stability in Linear Rational Expectations Models. In section Illustration: Cagan’s Model subsection Alternative Code, you run a LinearStateSpace() simulation, but graph the results of the previous code.
Thanks.

Thanks @spspitze !
Could someone please put  a PR together to fix this?  CC @shlff @Harveyt47 @sayaikegawa @mtiley

Thanks @spspitze and @jstac . Sure!

Hi @spspitze @jstac will do I believe
T = 100

# simulate using LinearStateSpace
x, y = ss.simulate(ts_length=T)

# plot
plt.figure()
plt.plot(range(T), x[1], label='$m_t$')
plt.plot(range(T), y[1], label='$p_t$')
plt.xlabel('t')
plt.title(f'λ={λ}, α={α}, $ρ_1$={ρ1}, $ρ_2$={ρ2}')
plt.legend()
plt.show()

Should fix this. I'll make a PR

I think the relevant info is all in y. So it should be y[0, :] and y[1, :]
iirc.
On Sat, May 9, 2020, 11:50 PM Harvey Thompson<EMAIL_ADDRESS>wrote:

Hi @spspitze https://github.com/spspitze @jstac
https://github.com/jstac will do I believe
T = 100
simulate using LinearStateSpace
x, y = ss.simulate(ts_length=T)
plot
plt.figure()
plt.plot(range(T), x[1], label='$m_t$')
plt.plot(range(T), y[1], label='$p_t$')
plt.xlabel('t')
plt.title(f'λ={λ}, α={α}, $ρ_1$={ρ1}, $ρ_2$={ρ2}')
plt.legend()
plt.show()
Should fix this. I'll make a PR
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/QuantEcon/lecture-python/issues/204#issuecomment-626268980,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AETYS7ZQJY7UT3QXBOXVWW3RQYP6ZANCNFSM4M46UFFQ
.

Yeah @spspitze that is nicer. Thanks!

I'll make the PR @shlff


Gemini does not respect the order of the properties in a schema

Description of the bug:
Let's imagine a situation where you want an answer and a rationale from the LLM. You want it to be in a JSON format like
{
  "choice": 4,
  "rationale": "The answer is 4 because..."
}

If the LLM completes the choice property first, then the next rationale can be made up to justify the choice.
However, if the rational comes first like
{
  "rationale": "The answer is 4 because...",
  "choice": 4
}

Then, the choice will be determined by rationale. It is similar to CoT.
It is important that the LLM follows the given order of the properties in an object. However, I found that Gemini SDK ignores it.
    properties: MutableMapping[str, "Schema"] = proto.MapField(
        proto.STRING,
        proto.MESSAGE,
        number=3,
        message="Schema",
    )

I think this one better to be a repeated, not a map field, to keep the order of the properties.
Actual vs expected behavior:
Gemini SDK ignores the order of the properties in an object.
Any other information you'd like to share?
No response

Any updates? also there is another issue:
https://github.com/google/generative-ai-go/issues/211

I am having the same issue that Gemini ignores the order; it tends to hallucinate

You can give a propertyOrdering property with a list of the property names, then it will respect that order.

@httplups  The object it returns is unordered. It may just be printed alphabetically, the order it generates them may just be incidental.

propertyOrdering

The API could add something like that to solve the problem.

Any update?


Add PHP / MySQL example

Hi
Nice work you've done here. I wish you added a docker-compose example with a MySQL & phpfpm containers, but I can device that myself.
Cheers,

I'll have one up by the end of the day!

@Skilgarriff still waiting :)


Allow Subject_Sets to be sortable

Subject_sets cannot currently be sorted by updated_at, among other attributes.

we can do this but what's the use case for sorting on the update_at attribute. I'm just a little hesitant adding more indexes to the db if it's trivial use case.

This is for the pullout news tab on the new signed in home page. Currently, the code is making an api call, getting all the subject sets for each project, and then sorting them by date when returned and plucking the most recent. It would be more efficient if the call could sort by updated at, then it would be able to only return the one most recent.

got it, we could certainly look at adding this (you can look at page_size=1 for each call OR we could do a bulk call to load by the set of project ids).
Out of interest, why is it using subject sets to find out if new data has been changed / released. I'm not sure this is the best way of signaling this change though.

Making as few calls as possible would be great, because right now it is making a call for the subject sets of each project, which is slowing things down a bit. Doing a bulk call would be nice.
I was thinking making a call for a project's subject sets and seeing the most recently updated set would be an indicator of the most recent update. However, I'm all ears if there's a better or more efficient way of checking when a project has received new subjects, especially by solely using the project resource.

we have a most recently updated projects filter (projects page) already, /api/projects?page=1&sort=-updated_at&launch_approved=true&cards=true&include=avatar

What causes a project to be updated though? If an owner tweaks the title or changes the workflow, is it still updated? I'm mainly curious if new subject sets have been added, but perhaps that is being too picky?

yeah a bunch of things like that (modification of the project resource via links, etc). Re change name, i can do the same thing on a subject set too.

Yeah, considering we're only trying to know if new subjects have been added, sorting a subject set might be the best bet.

sounds like a plan.

In talking with a couple people about this (Jim and Sarah), we might just sort projects by updated_at for the news pullout, which would save us from having to make the extra API call and would speed things up a bit. We can always explore how much quicker a bulk API call would be in the future, or we might be satisfied with how the page looks displaying the most recently updated projects.

Close this for now then, re-open if we need it.


Feature: Add Scripts and Dockerfile for Cortex CPP

This PR adds the necessary scripts and Dockerfile to set up and run Cortex using Docker. The following files are included:
1. entrypoint.sh:

Installs the cortex.llamacpp engine.
Starts the Cortex server.
Keeps the container running by tailing the log files.

2. Dockerfile:

Uses Ubuntu 22.04 as the base image.
Installs necessary dependencies.
Downloads and installs the latest versions of cortex.llamacpp and cortex.cpp.
Copies the entrypoint script and sets it as the entrypoint for the container.
Exposes port 39281 and sets up a health check.

3. download-cortex.llamacpp.sh:

Fetches the latest version of cortex.llamacpp if not specified.
Downloads various cortex.llamacpp engine versions and stores them in /opt/cortex.llamacpp.

4. download-cortex.cpp.sh:

Fetches the latest version of cortex.cpp if not specified.
Downloads and installs cortex.cpp to /usr/local/bin.

These additions will facilitate the setup and deployment of Cortex in a Docker container, making it easier to manage and run the Cortex server.
Related issue #365


Hacktoberfest

Added code of disjoint set union in python as dsu.py

@ankitcdry1612 contribute here https://github.com/Ishaan28malik/Get-Dev-Resources