parent.getAttribute is not a function error

After debugging this issue i have come to find that parent variable is being set to either Window or Document but neither of the two contain the getAttribute() function.
This issue breaks my code and prevents the render of the component. I have defined my LazyLoad component as so..

      <LazyLoad height={333} offset={300} overflow once>
        <Picture
          image={sectionImage}
          imageRetina={sectionImageRetina}
          imageMobile={sectionImageMobile}
          imageMobileRetina={sectionImage}
          screen={SCREEN_SIZES.SCREEN_SM_MAX} />
      </LazyLoad>

The parent div has a position: relative; styling
Have I not used the overflow prop correctly? because it seems that although the overflow prop os set, the returned object in parent doesn't contain the function which needs to be checked for the condition and therefore breaks at this point and throws an error.

Have i missed something?

If you use overflow props, you have to make sure there's overflow: auto; or overflow: scroll; set on <LazyLoad>'s parent, technically it could be html, but in that way you don't have to use overflow at all.

I applied overflow: auto across all the sections which has allowed the <LazyLoad> to behaviour as it should, only i've come across an edge case whereby I have an image within the section which overflows but once overlow: auto or overflow: scroll is applied the image is then clipped but without the overflow styling the image will not be lazy loaded.
In this case, is there another way I should be handling this particular section?

If ancestor hasn't got overflow: auto; or overflow: scroll;, you shouldn't add overflow props to <Lazyload>.

Do you change overflow style of LazyLoad's ancestor on the fly?

no, there not changed on the fly, the ancestor did have overflow: auto, so i removed the overflow prop from the <LazyLoad>, but now the image is clipped, this illustration might help to visualise the issue i'm having.

I had a work-around for this issue. Note that if you are setting overflow on your outer div through css, it is possible that the CSS is applied after the LazyLoad processes your div. It will therefore bubble up each parent until it reaches the root document.
I solved this with inline styles on my overflow component.
`
render() {
var overflowStyle = {
'overflowY': 'auto',
'overflowX': 'hidden'
}
return(

  { populateWithLazyLoad components }

)
}
`

I had the following structure
<overflow container>
    <container>
      <lazyload overflow={true}><img...
   </...>
</...>

When I try to remove the container element, this error occus:
Uncaught TypeError: parent.getAttribute is not a function

hi @yangg , can you post your css on overflow container and container?  I believe there is something wrong introduced by another bugfix.

this is style of my overflow container
{
    width: 377px;
    background-color: #fff;
    overflow: auto;
}

the container has no styles currently.

I assume you're using<EMAIL_ADDRESS>

@jasonslyvia  I've updated to 2.2.1, same error occurs.

Can you please try 2.2.2?

@jasonslyvia  It works now.

hi @cnolimit , can you please try version 2.2.2 to checkout if it solves your issue?

hi @jasonslyvia and @yangg  thanks for all your help, although since posting this issue i've changed the structure of the code and so i'm unable to check whether these solutions would work for me. but since refactoring I haven't ran into any errors and found it works much better now.


#167 update adopt district event text

Closes #167.
Please check:

[ ] Event cards
[ ] Event table rows
[ ] Event modals

The mobile views are still a bit of a mess.  I can fix them in this PR if you'd like, but honestly the entire site could probably use a bit of review and polish when it comes to some of the mobile layout so perhaps we should make it a new ticket?

I experimented with trying to also have the district/state subtitle. Let me know what you think

I definitely like the idea of keeping info if we can, but this layout doesn't quite seem right.  We're pushing some of the most important info to the right hand side.
What if we kept your 2col idea but made the layout vertical instead?  Might have to chop the description text a tad but it might work better.
e.g.
Home MoC (absent)                    Visiting MoC (present)
David Reichert                             Pramila Jayapal
(Republican, WA-8)
```


Fix unmatched param bug

Hi! I ran into a bug when using pydapper on my project when parameters are specified, but not used in the SQL query. In that case, it would throw a missing key exception. This might seem like a contrived situation, but it is an issue when using query_multiple where not all queries have parameters specified (which is what happened to me).
I also noticed the codecov package version in use is no longer available and was causing the install to fail so I updated it.

codecov actually deleted their python uploader from pypi, I'll put in a separate PR to fix that in CI

Sure, here's a minimum example:
param = {
    "val": 1,
}

sql = (
    "select 1",
    "select ?val?"
)

val1, val2 = commands.query_multiple(sql, models=(dict, dict), param=param)

This results in this exception:
KeyError: "Key '' can not be accessed on {'val': 1} or does not exist"
obj = {'val': 1}, key = ''

    def safe_getattr(obj: Any, key: str) -> Any:
        try:
            if isinstance(obj, dict):
>               return obj[key]
E               KeyError: ''

hey man - sorry I'm taking so long getting back to this, been a crazy few months for me. Lets try to get this merged this week.

No worries, thanks for reviewing!


v7 - Custom ssh config host names warnings

Deployer version: v7 beta 23
Deployment OS: unix

So I have one server with a few projects. When I try and pull them GitHub doesn't allow re-using deployment keys so I did some tricks of leveraging .ssh/config to name the host under a specific name.
Host project1
        ForwardAgent no
        User git
        HostName github.com
        IdentityFile /home/deploy/.ssh/project1

(~/.ssh/config)
So now the new v7 attempts to reach out to these projects even if they aren't valid domains.
task deploy:update_code
[prod] run command -v 'git' || which 'git' || type -p 'git'
[prod] /usr/bin/git
[prod] run ssh-keygen -F project1
[prod] run ssh-keyscan -H project1 >> ~/.ssh/known_hosts
[prod] getaddrinfo project1: Name or service not known
[prod] getaddrinfo project1: Name or service not known
[prod] getaddrinfo project1: Name or service not known
[prod] run [ -d /usr/share/nginx/html/.dep/repo ] || mkdir -p /usr/share/nginx/html/.dep/repo

Could you please share your "tricks of leveraging .ssh/config"? I am now facing the same situation and trying to get the .ssh/config to work.

# .ssh/config
Host name-you-use-in-deployer.php
        ForwardAgent no
        User git
        HostName realname-or-ip-address
        IdentityFile /home/deploy/.ssh/project1

# deployer.php
host('name-you-use-in-deployer.php')
    ->set('hostname', '<IP_ADDRESS>6')
    ->setRemoteUser('my-user')
    ->setConfigFile('~/.ssh/config')

I assume that this one is fixed as well by #2667. Can anyone confirm?

Well. No. But actually what @Schrank suggested is a solution.

@Schrank solution is what I was doing behind the scenes, which was broken in Deployer because it assumed it would be a URL and not a reference to a .ssh/config entry.
Either way, this PR does look to resolve that issue, but I no longer run Deployer to confirm.


Fix baseUrl for NODE_PATH=src

title

Out of curiosity, would this allow absolute imports in TS apps created with create-react-app?
I've been trying to get it working and remember setting the NODE_PATH for JS projects. Loving the Typescript support 🙏

Yes, this will start permitting absolute imports in TS apps.

Will this work with NODE_PATH=. as well?
I use this because it forces referencing src/, removing any ambiguity as to what is being imported.

Not sure if we'll allow . or not, there's always https://github.com/facebook/create-react-app/issues/5118 which will let you import via @/.

We use the same as @tkazec, so that instead of having to have a dozen different absolute paths (plus paths for files that were under src itself...I'm not sure how src/App.ts would be referenced without the src prefix present) we only had one, and thus had a lot less risk of a name clash between one of our folders and an npm package name.

@niekert
If you're looking for a way to enable absolute path imports, see https://github.com/facebook/create-react-app/issues/5118#issuecomment-439665737

Adding "baseUrl": "./src", to my compilerOptions options in tsconfig.json worked for me.

@jamesg1 That's been working for me for over a year too, but on this new project, whenever I run yarn start, the baseUrl entry is removed from tsconfig.json!
Anyone have a way to prevent this?

Yeah I think there was a change which resets the baseUrl might have to use react rewired.

@mindjuice The tsconfig is rewritten on every start. You have to use extends (https://github.com/facebook/create-react-app/issues/5645#issuecomment-461999138)

We now support setting baseUrl in tsconfig.json (and jsconfig.json). Currently you can only set it to node_modules (the default) and src. We are hoping to add support for aliases in the near future and may look at relaxing the restriction on baseUrl.
NODE_PATH is now deprecated and since you can just set baseUrl directly I think this can be closed.


Wrong size of unsigned long type

[Enter feedback here]
The size of unsigned long here must be 8 bytes, not 4 bytes. Its wrong here.

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

ID: 2a3252f3-2390-77d1-75ee-00ff08584173
Version Independent ID: 53786641-f25d-638f-18d5-8c7e19712776
Content: Data Type Ranges
Content Source: docs/cpp/data-type-ranges.md
Product: visual-cpp
Technology: cpp-language
GitHub Login: @corob-msft
Microsoft Alias: corob

@Pastelhorizon
All flavors of long in the Microsoft C/C++ compiler are 32-bit (4-byte) integers, the same size as int. This differs from some other compilers, such as gcc. If you require portable specific sizes of integers, use the explicitly sized types, such as uint64_t. I'm closing this issue as it doesn't appear to identify a real issue in the documentation.


Toggling between Week mode and Month mode does not redraw days

Thanks for creating this library. It is exactly what I need!
Whilst using it, I have noticed an issue which I was hoping you could help with.
Using Example1 of the samples, change line 52 of Example1Fragment.kt to:
val startMonth = YearMonth.of(2020, 8)
Run the example, enable week mode then disable week mode. The title is renamed to reflect the change in mode but the calendar is not redrawn to show all dates in the month. Instead it remains as it did when week mode was enabled

This is now fixed in 0.4.3

Works perfectly! Thanks for your help


Problem Creating Server Ctx Struct: Boost::Beast Way to write a Server_Ctx Struct

Effectively, my boost beast web server I have been working on for a while has multiple aspects to it including a io_ctx, thread pool, database management system, and ssl context etc. so I am trying to rewrite functions to use a server_ctx struct that effectively stores all these variables for me so I can access them easier. So I wrote the following:
...
//------------------------------------------------------------------------------
namespace beast  = boost::beast;      // from <boost/beast.hpp>
namespace http   = beast::http;          // from <boost/beast/http.hpp>
namespace asio   = boost::asio;          // from <boost/asio.hpp>
namespace ssl    = boost::asio::ssl;     // from <boost/asio/ssl.hpp>
namespace tuples = boost::tuples;     // from <boost/tuple/tuple.hpp>
namespace json   = boost::json;          // from <boost/json.hpp>
using tuples::tie;
using asio::awaitable;
using asio::use_awaitable;
using asio::detached;
using asio::experimental::as_tuple;
using asio::experimental::as_single;
using asio::redirect_error;
using tcp = boost::asio::ip::tcp;       // from <boost/asio/ip/tcp.hpp>
using namespace boost::urls;          // from <boost/urls.hpp> - Expiremental
//------------------------------------------------------------------------------
struct cassandra_ctx_t
{
    CassCluster* cluster;
    CassSession* session;
};
struct server_ctx_t
{
    asio::io_context   io_ctx;
    asio::thread_pool  tp_ctx;
    cassandra_ctx_t    cassandra_ctx;
    ssl::context       ssl_ctx;
    std::shared_ptr<std::string const> const doc_root;

    server_ctx_t(asio::io_context&    io_ctx_,
                 asio::thread_pool&   tp_ctx_,
                 cassandra_ctx_t&     cassandra_ctx_,
                 ssl::context&        ssl_ctx_,
                 std::shared_ptr<std::string const> const& doc_root_)
                 :
                 io_ctx(io_ctx_),
                 tp_ctx(tp_ctx_),
                 cassandra_ctx(cassandra_ctx_),
                 ssl_ctx(ssl_ctx_),
                 doc_root(doc_root_)
    {
    };
};

then I try and use this constructor in main via:
    struct server_ctx_t server_ctx(std::ref(io_ctx),
                                   std::ref(tp_ctx),
                                   std::ref(cassandra_ctx),
                                   std::ref(ssl_ctx),
                                   doc_root);

But I get the following error
In file included from main.cpp:1:
/home/warrenniles/Desktop/project/project-webserver/project-webserver/server.hpp: In constructor ‘server_ctx_t::server_ctx_t(boost::asio::io_context&, boost::asio::thread_pool&, cassandra_ctx_t&, boost::asio::ssl::context&, const std::shared_ptr<const std::__cxx11::basic_string<char> >&)’:
/home/warrenniles/Desktop/project/project-webserver/project-webserver/server.hpp:100:18: error: use of deleted function ‘boost::asio::io_context::io_context(const boost::asio::io_context&)’
  100 |                  io_ctx(io_ctx_),
      |                  ^~~~~~~~~~~~~~~
In file included from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/asio/detail/io_object_impl.hpp:23,
                 from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/asio/basic_waitable_timer.hpp:24,
                 from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/asio/steady_timer.hpp:22,
                 from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/beast/core/detail/stream_base.hpp:13,
                 from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/beast/core/basic_stream.hpp:14,
                 from /home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/beast/core.hpp:16,
                 from /home/warrenniles/Desktop/project/project-webserver/project-webserver/server.hpp:25,
                 from main.cpp:1:
/home/warrenniles/Desktop/project/project-webserver/external_libraries/include/boost/asio/io_context.hpp:633:3: note: declared here
  633 |   io_context(const io_context&) BOOST_ASIO_DELETED;
      |   ^~~~~~~~~~

The problem is that this doesn't seem to be feasible, but I need a clean way to do this. What would be an appropriate boost beast way to do this so I can use functions like:
template<class Body, class Allocator,class Send>
awaitable<void> handle_request(server_ctx_t& server_ctx,
                               http::request<Body, http::basic_fields<Allocator>>&& req,
                               Send&& send);

Why an asio::thread_pool & an asio::io_context? And why are you trying to copy them?
The latter is the error.
Also, why std::ref ?

Why an asio::thread_pool & an asio::io_context? And why are you trying to copy them?
The latter is the error.
Also, why std::ref ?

So the problem I'm having is the following: I want to package a bunch of these different variables into 1 data structure that I can pass to functions like handle request. My problems are as follows:

The problem is I can't simply write a struct that has let's say asio::io_context io_ctx as a member and then in main write something like

Asio::io_context io_context{5};
struct server_ctx myServerCtx;
server_ctx.io_ctx= io_ctx;

Because for example ssl::context requires passing tls version to the constructor.
2) So I know I can theoretically initialize members before the body of the constructor executes to try and get around this. Problem is I don't want to have to include all that logic within my constructor of setting the number of io threads. Setting the number of threads for the thread Pool for handling old legacy stuff etc.
I thought maybe copying once could get me the behavior I wanted but that hasn't worked either.
All I want essentially is to set up the asio::io_context, asio::thread_pool, ssl::context etc. In main.cpp, load them into 1 data structure that allows me to pass them all as references to various functions. I'm just having one hell of a time figuring out the best way to go about getting this desired behavior.

You cannot move execution_contexts such as io_context and thread_pool. That won't ever change, but you can use executors, which I think are made for your use-case.


feat: add vimeo provider

add vimeo provider support

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


Caching of missing group:name from a repository is not consistent with caching of missing group:name:version

Expected Behavior
When having multiple repositories defined in my project and adding a dependency with a dynamic version, such as org:foo:1.+ or org:foo:latest.milestone, I expect only the repository containing that org:foo to be refreshed when the dynamic versions cache expires.
Current Behavior
All repositories are checked as soon as the dynamic versions cache expires, which can be a performance concern when the build configures cacheDynamicVersionsFor 0, 'seconds' and contains multiple remote repositories.
Fixing this bug has one impact that could be surprising: If you start publishing new version of a dependency in a repository that was declared in your build already but did not contain any, you will need to force a refresh by using --refresh-dependencies. Because this feels like a corner case, fixing that performance bug should not have a negative impact on most builds.

Fixing this will improve the situation from #3049


Check demos 37, 41, 42, 45, 46, 47, 48

Fix #969
Programs added to the list of programs checked by ZDEMO_EXCEL_CHECKER: ZDEMO_EXCEL37, ZDEMO_EXCEL41, ZDEMO_EXCEL42, ZDEMO_EXCEL45, ZDEMO_EXCEL46, ZDEMO_EXCEL47 and ZDEMO_EXCEL48.
If a Web Repository object doesn't exist, the concerned program now appears in the list of programs with DIFF and SAVE buttons

Pull request no more needed in this repo, now done in the new Demos repository → https://github.com/abap2xlsx/demos/pull/32.


Add getDisplayMedia support check

I noticed that on Android, the getDisplayMedia function is undefined. Therefore I added a more verbose error message.

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

That's great! Thank you for your contribution :)


Build Search Component which will search by item name

Description
The Search component should provide a facility to search items by name. It should fetch all the items which contains the search keyword from BPPs.

Goals

Buyer should be able to fetch all the items searched by keyword.

Expected Outcome
Satisfy the above mentioned goals.

Acceptance Criteria

There should be a input box where Buyer can put keyword to search for.
On clicking search action button, it should fetch the items containing the keyword in their name or description from BPPs

Product Name
Beckn

Project Name
BAP-Boilerplate-SDK

Domain
Culture and Tourism

Tech Skills Needed:
NodeJS, ReactJS

Mentor(s)

nikhil2128

Complexity
Medium

Category

UI/UX/Design
Integrations
Features
Deployment
Test

Sub Category

API
Database
Analytics
Frontend
Backend

Hey @nikhil2128, is only the Backend required or should I provide an basic React Native UI as well ?


Dropzone styling improvements

Move all dropzone styles to separate file
Fix white background in arc-green
Fix rendering of non-square images and previews

Before:

After:

Added a few more improvements. Increased thumbnail preview quality 4-times and replaced the hideous blur effect dropzone does with opacity, this is how it looks now when hovering a thumbnail:


Increase versions, build fixes for mobile beta release

Note that the latest Android build number in production is 28.
Fixes an Android build issue by creating a missing libs directory,.
Fixes an Apple failure to strip symbols from the tun2socks framework by not embedding it. This seems to be an issue with Go/gomobile: https://github.com/golang/go/issues/28997.

@bemasc, in case you want to take a look, the latest two commits contain build fixes. I have updated the PR description.


🐛 BUG: named exports will crash

What version of Wrangler are you using?
2.0.0
What operating system are you using?
mac
Describe the Bug
⚡ echo "const a = () => {};export {a};" > index.js && wrangler dev index.js

will result in:
  {
    text: 'Received a bad response from the API',
    notes: [
      {
        text: "Uncaught SyntaxError: Unexpected token 'export'\n" +
          '  at line 4\n' +
          ' [code: 10021]'
      }
    ],
    location: undefined,
    kind: 'error',
    code: 10021
  }

Ideally, in a modern module system named exports wouldnt throw an error even if they are unused

Good catch. In this scenario, wrangler is detecting that you're developing a service-worker format worker, since you don't have a default export. Our runtime throws an error when those workers have named exports. We do log a warning when you do this, which would look like:
▲ [WARNING] The entrypoint index.js has exports like an ES Module, but hasn't defined a
default export like a module worker normally would. Building the worker using "service-worker" format...

The workaround here is to remove any exports. As a long term fix, wrangler could do better and simply remove those exports itself, which we'll do soon. Thanks for reporting this issue!

Actually, I don't think Wrangler should patch these exports. Instead this should be a hard error, rather than just the warning it is now.

Why?
To elaborate on my usecase: I'm creating an Astro integration for cloudflare. This means that I dont have complete control over the build output. The build output in this case includes some exports. In a modern module system environment, this would be fine, and the exports would just be ignored, however, in cloudflare this would then apparently be a hard error? This means I now have to write extra code and run an additional build to post-process my already modern es module

Hmm. As I understood it, the named exports should not cause the error. It is the lack of default export that is the problem.
That is because without it, this is not a Worker.

It is the lack of default export that is the problem.

Interesting, using wrangler 1 this didnt seem to be a problem

@thepassle - could you provide a reproduction of the error that I could try out?

const a = () => {};
export {a};

This is not a valid Cloudflare Worker.

Thats the most minimal repro I could make that highlights the error

I think wrangler2 could do a better job here, I'll chat with @petebacondarwin and we'll come up with something

Oh! I see. Sorry if I was coming across as argumentative. I was trying to work out why you would get the error.
You are actually creating a "service worker" style Worker, but happen to also have exports, which causes Wrangler 2 to think it is a "ES module worker" style Worker.
So in that case, we should improve our "format sniffing". I.E. if there is no default export (that contains a fetch method) and there is a call to addEventListener() then we should consider that a "service worker" style format.

We landed a fix that shouldn't throw the error anymore in https://github.com/cloudflare/wrangler2/pull/941, you can try it with wrangler@beta. We'll do a proper release in a few days. Thanks for filing this issue!


Option to display .modaal-close button text

I think it would be nice to be able to opt to display the clipped 'close' text on the close button.
Doing so may require that the before and after be positioned right rather than left.

Hi @lewismcarey, thanks for the suggestion. This feels like it's more of a case by case basis for each developer to make, however I also feel like it's a small enough enhancement that could provide valuable flexibility to the plugin and developers alike.
I'd like to leave this ticket open for a bit to gauge  interest in this idea. For anyone else reading, if you would like to see this added, please give a +1 thumbs up reaction.


Merge chunks.RootTracker interface into chunks.ChunkStore

You can't fully specify RootTracker without referring to the
ChunkStore interface, so they should just merge.
Fixes #3402

PTAL

Whoops, tests build now. PTAL


Unclear error message when reassigning tuple fields

When reassigning the field of a tuple like so:
actor Main
  new create(env: Env) =>
    let foo: (U64, String) = (42, "foo")
    foo._2 = "bar"

you get the following error message:
main.pony:5:8: can't assign to a let field
    foo._2 = "bar"
       ^
Error:
main.pony:5:12: left side must be something that can be assigned to
    foo._2 = "bar"
           ^

I know that this is explained in the tutorial, but I think that this message should be more clear, maybe cannot reassign an element in a tuple.

I think this shows the issue a bit better:
actor Main
  new create(env: Env) =>
    var foo: (U64, String) = (42, "foo")
    foo._2 = "bar"


[Screen Reader - Cloud Power Shell - Open Editor] Narrator is reading the incorrect position information at files tree items.

User Experience:
If Narrator is reading the incorrect position information at files tree items then the users with visual disabilities will get confuse while navigating through the tree items.
Repro steps:

Open URL https://ms.portal.azure.com/  and login via valid credentials
Navigate to the cloud shell button on the top menu bar and select it.
Cloud Shell popup is launched.
Navigate to "Welcome to Azure Cloud Shell" popup and select "PowerShell(Linux)" button.
Navigate to "You have no storage mounted" popup window and select "Show advanced settings" button.
Navigate to "Subscriptions" dropdown and select "Microsoft Azure" list item and create button.
Navigate to "Open Editor" button of "CloudShell-PowerShell Editor" window and select it.
Navigate to files tree item and verify the issue.

Actual Result:
Narrator is reading the incorrect position information at files tree items.
Narrator is reading as 1 of 8 at .azure tree item but there are 9 items in the tree. even after expanding the items in the first level it is reading the incorrect position at the items in level 2.
Expected result:
Narrator should read the correct position information at files tree items.
Narrator should read as 1 of 9 at '.azure' tree item as there are 9 items in the tree. After expanding the items in the first level, it should consider the items at level 2 as another set and it should read as 1 of 9 at level 2 if there are 9 items at level 2.
Note:
JAWS in Browse mode is not navigating to the tree items individually.
MAS Reference:
MAS 1.3.1 - Info and Relationships (40)
Check out Keros! - Identify accessibility bugs before check-in and make bug fixing faster and easier.

Issue is copied from Azure Cloud shell - Bug 2992185


update/insert columns -> assign

closes #235
I think I made a mistake in splitting "set_column" into "insert_column" and "update_column":

first, because having tried using the Standard to write code a while now, it's really tedious to have to distinguish them
because I don't think any existing dataframe library has this distinction
a users has brought this up too (#235)

In keeping with the goal of making the standard feel familiar, let's look at what existing dataframe libraries do:

pandas and its clones: either __setitem__ (which the consortium doesn't want) or DataFrame.assign
pyspark / polars: DataFrame.with_columns
ibis: DataFrame.mutate
datatable: update / extend

I don't really mind which one we go with. I'm suggesting with_columns here as I think it's probably the most explicit

approved in today's call (but consensus on assign instead of with_columns)


Error: Could not read the plist for postgresql!

System version:
Description:	Ubuntu 14.04.3 LTS
Linuxbrew ( here I installed Linuxbrew not the Homebrew ):
Homebrew 0.9.5 (git revision dc1c2; last commit 2016-04-15)
source code: https://github.com/Linuxbrew/linuxbrew
I installed Linuxbrew, and then installed the homebrew-services with the command:
brew tap homebrew/services
I installed PostgreSQL with Linuxbrew, with the brew info postgresql command, got the install info:
postgresql: stable 9.5.2
Object-relational database system
https://www.postgresql.org/
Conflicts with: postgres-xc
/home/snow/.linuxbrew/Cellar/postgresql/9.5.2 (3,109 files, 36.3M) *
Built from source with: --without-perl
From: https://github.com/Homebrew/linuxbrew/blob/master/Library/Formula/postgresql.rb
==> Dependencies
Required: openssl ✔, readline ✔, libxml2 ✔, libxslt ✔, util-linux ✔
==> Options
--32-bit
Build 32-bit only
--with-dtrace
Build with DTrace support
--with-python
Build with python support
--with-tcl
Build with Tcl support
--without-perl
Build without Perl support
==> Caveats
If builds of PostgreSQL 9 are failing and you have version 8.x installed,
you may need to remove the previous version first. See:
https://github.com/Homebrew/homebrew/issues/2510
To migrate existing data from a previous major version (pre-9.0) of PostgreSQL, see:
http://www.postgresql.org/docs/9.5/static/upgrading.html
To migrate existing data from a previous minor version (9.0-9.4) of PosgresSQL, see:
http://www.postgresql.org/docs/9.5/static/pgupgrade.html
You will need your previous PostgreSQL installation from brew to perform pg_upgrade.
Do not run brew cleanup postgresql until you have performed the migration.
Then with brew services list command, it works:
Name       Status  User Plist
postgresql stopped
But with brew services start postgresql command, it failed:
sh: 1: list: not found
Error: Could not read the plist for postgresql!
I don't know what's the problem?
Is the directory problem? Need to set the custom_plist  or something?
Help! Thanks!

homebrew-services does not support Linux, sorry.

Hi, same error on Linux. Do you have any plan to support Linuxbrew?

@unionx We'd accept pull requests but almost certainly not as it's very tightly bound currently to launchctl.


Updating README.md

Remove \n

Update README file, add a new line and remove the leading \n sign.


added support for simplified Chinese

Have added support for simplified Chinese.
BTW. Where can we add simplified Chinese to the survey locale property options in the editor?

Does the build failure have anything to do with my codes? Although does not seem to...

@xraywu I have confirmed the pull request. The issue in our functional tests. They are not 100% stable yet. We will take a look at them.
Thank you,
Andrew


Why those two links redirect to bioRxiv rather than Medrxiv?

Thanks for noticing! That's a mistake we should fix - do you want to submit a PR with a fix? :)

Thanks for noticing! That's a mistake we should fix - do you want to submit a PR with a fix? :)

Thanks, proposed!
check https://github.com/embeddings-benchmark/mteb/pull/187


low recall on sift1M dataset

hello,
i deploy single-node vearch using https://github.com/vearch/vearch/blob/master/docs/BinaryInstallation.md
i found that both hnsw & ivfpq got very low recall on sift1M dataset, just about 12%,
however, when i lower the vector number of sift1M dataset from 1,000,000 to 10,000, recall got normal which can be up to 99%, so any idea to fix this?

How did you create the table?

create table:
{
    "name": "test",
    "partition_num": 1,
    "replica_num": 1,
    "engine": {
        "name": "gamma",
        "retrieval_type": retrieval_type,
        "retrieval_param": retrieval_param
    },
    "properties": {
        "field1": {
            "type": "vector",
            "index": True,
            "dimension": dimension
        }
    }
}

ivfpq retrieval_param:
{
            "ncentroids": ncentroids, # 512, 1024, 2048, 4096, 8192
            "nsubvector": 64,
            "metric_type": "L2",
            "nbits_per_idx": 8
}

ivfpq bulk search:
{
            "query": {
                "sum": [{
                    "field": "field1",
                    "feature": features,
                }]
            },
            "size": 10,
            "sort": [{
                "_score": {"order": "asc"}
            }],
            "nprobe": nprobe # 50, 100, 200, 400, 500, and with case nprobe = ncentroids
}

hnsw retrieval_param:
{
            "metric_type": "L2",
            "nlinks": 16,
            "efConstruction": efConstruction, # 80, 200
            "efSearch": efSearch # 10, 20, 30, 40, 50, 70 and with case efSearch = efConstruction
}

hnsw bulk search:
{
            "query": {
                "sum": [{
                    "field": "field1",
                    "feature": features,
                }]
            },
            "size": 10,
            "sort": [{
                "_score": {"order": "asc"}
            }]
}

when create table, if you don't set max_size, then the default value of max_size is 100000, so you just insert 100000 into the table. You can set it like this:
"engine": {
"name": "gamma",
"index_size": 70000,
"max_size": 1100000,
"id_type": "String",
"retrieval_type": "IVFPQ",
},

thank you, i will have a try

fixed


[Feature][Sort] Support rowdata way of sort InLong message format base

Description
Support rowdata way of sort InLong message format base
Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

It has merged into master


output and debug

Has anyone started including these two processes on a back-end and if so what kind of data and/or debugging info do you return? Maybe we could define this better or simply provide some examples in the description.

AFAIK no back-end except the GEE back-end has implemented the Subscriptions API yet, which is the basis for these processes. I'm still thinking about how these processes can look like, but I think these outputs highly depend on the back-end, too. In the GEE driver the output of output could be equal to what the GEE print operation offers.

For Geopyspark, it will not be easy to provide something meaningful. So I don't have these processes on my planning. I also don't know what GEE prints as debugging output, that might provide some ideas...

For interoperability reasons it could be useful to provide stubs anyway, @jdries, so that process graphs using these functions work anyway, but without doing anything. But that's probably something for fine-tuning at a later stage.
Some GEE examples:

Reworked in commit 57c7a0cc503e98bc64e25cfd4f94fa8dc1b17c58


fix: properly calculate content-length for cfn custom resource responses

Ref: https://github.com/aws/aws-cdk/pull/24501

Custom Resources need to write their response into a S3 object.
This is implemented as a PUT request to a pre-signed URL and has to specify the content-length of the response object.
Previously the CustomResource code would use responseBody.length.
However this returns the number of graphemes, not bytes.
If any utf8 characters with graphemes != bytes are part of the response, CloudFormation would fail the deployment with a Response is not valid JSON error.

Oh interesting. Let me ask the team to take a look.


Javascript Capstone: Set up Kanban Board

Below is the link to our team's Github project:
https://github.com/evillaz/CapstoneProject/projects/1
Add a note telling the reviewer how many people there are in your team:
There are two people in my team: Student A: @evillaz and Student B: @adarachel

Approved ✔
Hi @adarachel and @evillaz,
Great job, Your project is complete!  There is nothing else to say other than... it's time to close this issue and start working on the project :shipit:
GOOD LUCK 🤞

(Changes Required) Aspects to improve: ♻

N/A

[Optional] suggestions:

N/A

Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
**Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear, and please remember to tag me in your question so I can receive the notification**
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Using a local LLM with LMStudio but it requires embeddings, any embedding model or a specific one?

Using docker and setting the BASE_URL to my host LLM running inside LMStudio I was able to make it work at least part of it.

But it stops to work the moment he searches for embeddings:
[2024-03-15 22:08:09.682] [ERROR] Unexpected endpoint or method. (POST /v1/embeddings). Returning 200 anyway
So, does it search for a specific embedding model? Or any available models? Cause in the latter case it could work using Ollama, since Ollama has embeddings Nomic Embedding for instance is pretty fast, has better performance text-embedding-ada-002 and text-embedding-3-small performance on short and long context tasks.
Usually accessed with this:
curl http://localhost:11434/api/embeddings -d '{
  "model": "nomic-embed-text",
  "prompt": "The sky is blue because of Rayleigh scattering"
}'

Tested with nomic-embed-text and Ollama and works as intended.

Hey @Mayorc1978 where can I find the file or configuration to use it with Ollama?
Hope you have a fantastic day!

@Mayorc1978 can you please provide the changes you made to make it work with Ollama...thanks!

            # Get Ollama host and embedding model from environment variables
            # Set default value for OLLAMA_HOST
            ollama_host = os.getenv("OLLAMA_HOST", "http://host.docker.internal:11434")
            # Set default value for OLLAMA_EMBEDING_MODEL
            ollama_embedding_model = os.getenv("OLLAMA_EMBEDING_MODEL", "nomic-embed-text")

            _embeddings = OllamaEmbeddings(model=ollama_embedding_model, base_url=ollama_host)

change embeddings.py works for me


Set up many virtual hosts in main app container

Hi, Is there a way to set up many virtual hosts pointing to to diferent paths in app/ through image or environment configuration?
I'm very new to Docker and have been trying to do this with this php-docker-boilerplate (which I found very usefull btw) and couln't figure it out.

No that's not possible without building a custom image.
But usually you would use 1 container per vhost. If you're concerned about port 80 being occupied by the first container, take a look at https://github.com/jwilder/nginx-proxy or https://jtreminio.com/blog/traefik-on-docker-for-web-developers/


Bets No Longer Authorized on Alpha!

Solved with:
https://github.com/openpredictionmarkets/socialpredict/pull/30


test all tags in manifest for regexp match

I am new to this tool, but as I was setting it up, I had some unexpected results with a repo that had a mix of:

untagged images
tagged images (with a single tag)
tagged images (with multiple tags)

I noticed that the shouldDelete function only checks the first tag it finds in the manifest.
This change allows the regex to match against any of the manifest Tags in the entire slice

I'm going to close out this PR because the code has changed a bit, and I'm not convinced this is a change we should support since it alters the API behavior.


Upgrade phoenix, ecto, timex and ex_machina to latest version using elixir 1.3

This upgrades the whole app to the latest phoenix and ecto versions, plus other libraries in the process, to get all tests green again. All tests pass for me. LMK if you have issues.

Hi @behrendtio, thank you so much for this PR, I was planning to upgrade everything :)
I'll review it ASAP and merge it :metal:
Cheers!

Hm yeah, didn't see it in the console, since I was always logged in and the tests didn't show the error either. Should be fixed with the latest commit - there were several encoding issues between JSON <> Timex <> Poison...


New package: DWANGO.NAIR version latest

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:
(Automated response - build 869.)

Manual Validation ended with:

The installation isn't unattended. Is an additional installer switch needed?


Adding additional firefox colorways

Hello there, I just want to say firstly thank you so much for this project! Thanks to your help, I was able to restore the old colorways I missed. I saw that this project had colorways I didn't see before, but entering them into the firefox config section worked well. I was able to extrapolate from the colorways presented, and bring back the colorways that I was familiar with. I'll add them here:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>`
I'm not sure if there are any colorways that I'm missing, but those are the ones that I remember. I hope these help!

Thanks @awarmwildfire, I'm glad you found it helpful.
I updated the README file to reflect your themes.

Thank you! 😄


10.0.19041.1499 not work

termsrv.zip

termsrv.zip

The offsets added in test mode - try it and let me know if it work or not.
https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini

Thank you. Working.

Is that dll from Insider build?


[alpha8] Documentation needs to be update

Yes, book is currently non-functional.

Update: book is now in template form.

Update: book is now in template form.

Cross-reference #469.

Cross-reference #469.

Close in favor of #678.


[EventRequest] OnRefreshOnAfterGetRecordOnAfterGetPurchLines in Purchase Order Statistics

Hello, please consider adding this event to the Purchase Order Statistics page
    [IntegrationEvent(false, false)]
    local procedure OnRefreshOnAfterGetRecordOnAfterGetPurchLines(PurchaseHeader: Record "PurchaseHeader"; var TempPurchLine: Record "Purchase Line" temporary)
    begin
    end;

and raise it after PurchPost.GetPurchLines(Rec, TempPurchLine, i - 1); in the RefreshOnAfterGetRecord procedure, similarly to how it's done in the Sales Order Statistics page.
Thank you

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48467.


Missing Parameter -ShareByEmailEnabled in Docs

In PowerShell and in EXAMPLE 5 there is a parameter -ShareByEmailEnabled
This paremeter is missing in the Command Reference at the top and in the parameter description.
Please have a look.

@officedocsbot assign @yogkumgit

Hi @PowerAppMac, thank you for your feedback.
Could you please kindly provide me the link to the article? Or at least the cmdlet name.
Thanks!

Hi.
Sure, it’s the “New-PnPSite” cmdlet:
https://docs.microsoft.com/en-us/powershell/module/sharepoint-pnp/new-pnpsite?view=sharepoint-ps
Regards
Martin
From: Dario Woitasen<EMAIL_ADDRESS>Sent: Montag, 9. September 2019 22:04
To: MicrosoftDocs/office-docs-powershell<EMAIL_ADDRESS>Cc: PowerAppMac<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/office-docs-powershell] Missing Parameter -ShareByEmailEnabled in Docs (#3952)
Hi @PowerAppMac https://github.com/PowerAppMac , thank you for your feedback.
Could you please kindly provide me the link to the article? Or at least the cmdlet name.
Thanks!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/office-docs-powershell/issues/3952?email_source=notifications&email_token=AMGE4MKFFWRNQJBNPG3QQK3QI2TZXA5CNFSM4IUCOTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6I3IWY#issuecomment-529642587 , or mute the thread https://github.com/notifications/unsubscribe-auth/AMGE4MJNASYDM3MQCVUGZJDQI2TZXANCNFSM4IUCOTBA .  https://github.com/notifications/beacon/AMGE4MMWMFGBQCZHZNM3R7LQI2TZXA5CNFSM4IUCOTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6I3IWY.gif
[ { "@context": "http://schema.org", "@type": "EmailMessage", "potentialAction": { "@type": "ViewAction", "target": "https://github.com/MicrosoftDocs/office-docs-powershell/issues/3952?email_source=notifications\u0026email_token=AMGE4MKFFWRNQJBNPG3QQK3QI2TZXA5CNFSM4IUCOTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6I3IWY#issuecomment-529642587", "url": "https://github.com/MicrosoftDocs/office-docs-powershell/issues/3952?email_source=notifications\u0026email_token=AMGE4MKFFWRNQJBNPG3QQK3QI2TZXA5CNFSM4IUCOTBKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6I3IWY#issuecomment-529642587", "name": "View Issue" }, "description": "View this Issue on GitHub", "publisher": { "@type": "Organization", "name": "GitHub", "url": "https://github.com" } } ]

Thank you @PowerAppMac,
It works for me:

@PowerAppMac sorry my mistake, I read your comment again and I will update this article adding this parameter.

@VesaJuvonen Please merge this PR #2251

@PowerAppMac - thank you for the proposal. The PnP cmdlet references are a little different than the rest of the references. The PnP 'source' for the reference actually lives in the cmdlet code itself and each release the reference content is copied over here so it is rendered alongside the rest of the Office cmdlets. The source file for this particular cmdlet is located here: https://github.com/SharePoint/PnP-PowerShell/blob/master/Commands/Files/AddFile.cs The repo is community driven and maintained and I know they would very much appreciate your contribution. I am going to go ahead and close this and let me know if you have any questions or need me to help or research anything on the PnP side.


Filtering with suffix and fixing root paths inclusion

This PR:

Filters out any other file except for.tiff, .tif or (.tdb in case of exportation) from dir URIs given for ingestion or exportation.
Fixes a previous mistake on unsorted-undefined order slicing of a bucket's ls.

Disclaimer: The next PR will concern a complete test suite for these part of codebase.

Failures are transient server-side errors; merging to get this out to the people.


[QA Review]:  Address static code scan analysis

[x] API/Geoproc - Remove any instance of eval() in python code
[x] UI - remove console.log in download-bundle.js

cc @jeffsuperglide

Cumulus-API
CWE-244 HEAP INSPECTION(2) (GO)

async_listener_listener/main.go

line 26
DBPass string is an attribute to the Go Struct Config

api/config/config.go

line 14
DBPass string is an attribute to the Go Struct Config

CWE-94 CODE INJECTION (PYTHON)

async_packager/src/cumulus-packager/configurations.py

line 70
removed eval replacing with ternary operator

async_packager/src/cumulus-packager/configurations.py

line 64
removed eval replacing with ternary operator

async_packager/src/cumulus-packager/configurations.py

line 29
removed eval replacing with ternary operator

CWE-798 HARDCODED PASSWORD?CREDENTIALS (PYTHON)

utils/send_process_msg.py

line 91
Local development testing products to simulate sending a message to the SQS queue for the geoprocessor
Code not used in application

utils/send_process_msg.py

line 84
Local development testing products to simulate sending a message to the SQS queue for the geoprocessor
Code not used in application

utils/geoprocess_message.py

line 47
Local development testing products to simulate sending a message to the SQS queue for the geoprocessor
Code not used in application

CWE-400 REGEX DENIAL OF SERVICE (JAVASCRIPT/NODE.JS)

docs/async_packager/html/search/worker.js

line 62

Cumulus-geoproc
CWE-94 CODE INJECTION (PYTHON)

src/cumulus-geoproc/configurations.py

line 69
removed eval replacing with ternary operator

src/cumulus-geoproc/configurations.py

line 63
removed eval replacing with ternary operator

src/cumulus-geoproc/configurations.py

line 30
removed eval replacing with ternary operator

cumulus-ui
CWE-117: LOG FORGING (1) (JAVASCRIPT /NODE.JS)SI-10: INFORMATION INPUT VALIDATION

src/app-bundles/download-bundle.js

line 44
✅ removed console.log()
this was silly, the request word flagged, but was not a variable

@adamscarberry @jeffsuperglide this is great, thanks.  I'm adding my response to the ISSO team below, would you mind making sure I'm not misstating anything?  Thanks.
Cumulus Scan Result Response
Cumulus-API
CWE-244 Medium - Heap Inspection
async_listener/listener/main.go
api/config/config.go
DB_Pass is pulled from the environment and stored in a Go struct for access by other methods that need to connect to the database.  I suppose we could refactor so that the value is pulled from the env when needed (i.e. when config.connStr() is called) but this will require a change in how we approach connection information.  Looking for advice from ISSO on if this is actually a big issue since a heap dump would indicate bigger problems in my opinion.
CWE-94 High - Code Injection
async_packager/src/cumulus_packager/configurations.py
Addressed by refactor.
CWE-798 Medium - Hardcoded Credentials
utils/send_process_msg.py
utils/geoprocess-message.py
Code in question in utils/send_process_msg.py:
84 -> mClient = Minio(
    "localhost:9000",
    access_key="AKIAIOSFODNN7EXAMPLE",
    secret_key="wJalrXUtnFEMI/K7MDENG/bPxRfiCYEXAMPLEKEY",
    secure=False,
)
91 -> bClient = boto3.resource(
    "sqs",
    endpoint_url="http://localhost:9324",
    region_name="elasticmq",
    aws_secret_access_key="x",
    aws_access_key_id="x",
    use_ssl=False,
)

Code in question in utils/geoprocess-message.py
47 -> CLIENT = boto3.resource(
    "sqs",
    endpoint_url="http://localhost:9324",
    region_name="elasticmq",
    aws_secret_access_key="x",
    aws_access_key_id="x",
    use_ssl=False,
)

Credentials are for local testing, not actual credentials. Is there a way to flag as not an issue or should we refactor?
CWE-400 Medium - Regex Denial of Service (REDOS)
docs/async_packager/html/search/worker.js
Non-issue, the value that is sent to the regex is the content of our documentation, not user input. The regular expression is used by a library to index contents and make searchable by a user.  User input is managed by the library to perform searches and is processed to search the index.  Additionally all regular expression searches are performed client side so a invalid user input would only affect that user.
Cumulus-Geoprocessor
CWE-94 High - Code Injection
src/cumulus_geoproc/configurations.py
Addressed by refactor.
Cumulus-UI
CWE-117 Low - Log Forging
src/app-bundles/download-bundle.js
Addressed by refactor.

@willbreitkreutz This all looks correct to me.

@adamscarberry cool, thanks!


Add deployment scripts and CI

This PR adds deployment scripts and a GitHub Action workflow to deploy all agents in the repo, if necessary.
Closes ENG-4281

Force push removes server-side encryption flags from S3 calls

Force push removes accidentally-committed testing code

Force push fixes initial manifest creation branch


master-next: update for SVN r330216

sys::Process::GetArgumentVector has been removed.  It is unclear if it
was ever needed in the first place.  Simplify the code to avoid the use.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

CC: @bob-wilson

@swift-ci please test

Before I got distracted last week, I was testing that same change locally along with this:
`--- include/swift/Basic/LLVMInitialize.h
+++ include/swift/Basic/LLVMInitialize.h
@@ -19,15 +19,16 @@
#ifndef SWIFT_BASIC_LLVMINITIALIZE_H
#define SWIFT_BASIC_LLVMINITIALIZE_H
+#include "llvm/Support/InitLLVM.h"
#include "llvm/Support/ManagedStatic.h"
#include "llvm/Support/PrettyStackTrace.h"
#include "llvm/Support/Signals.h"
#include "llvm/Support/TargetSelect.h"
#define PROGRAM_START(argc, argv) \

llvm::sys::PrintStackTraceOnErrorSignal(argv[0]); \
llvm::PrettyStackTraceProgram _INITIALIZE_LLVM_STACKTRACE(argc, argv); \
llvm::llvm_shutdown_obj _INITIALIZE_LLVM_SHUTDOWN_OBJ

do { \
llvm::InitLLVM _INITIALIZE_LLVM(argc, argv); \
} while (0)

#define INITIALIZE_LLVM()
do {
`
It didn't work and I haven't gotten back to debugging it yet.

This definitely was sufficient to perform a build and run the test suite.

OK, well go ahead with that part then. I do think the InitLLVM change will be needed on Windows, but I can work on that separately.

That might explain it. I'll try taking that out.

Thanks, Michael. You were right. I took out the do-while and tested a build with both my change and compnerd's and everything passed. https://github.com/apple/swift/pull/16133


Fix issues with IHNC (newton-mod) Integral equation method

I messed up some signs when rewriting that part to catch some rare events. This is the fix.

@votca-bot changelog: fix issues with IHNC (newton-mod) Integral equation method

@felipeZ please review!


Send back errors

#17

Codecov Report

Merging #18 (0583968) into main (28e4fd1) will decrease coverage by 14.62%.
The diff coverage is 17.24%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main      #18       +/-   ##
===========================================
- Coverage   64.61%   50.00%   -14.62%
===========================================
  Files           4        6        +2
  Lines          65       92       +27
  Branches        4        8        +4
===========================================
+ Hits           42       46        +4
- Misses         23       46       +23

Impacted Files
Coverage Δ

nats_client/handlers.py
0.00% <0.00%> (ø)

nats_client/clients.py
32.50% <21.42%> (-6.79%)
:arrow_down:

nats_client/exceptions.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix rollback on only one migration

Fix #1059
I think error comes with this commit: https://github.com/robmorgan/phinx/commit/bba44b381fef9756c1d2f5a6c65ddc6757e7f6bb
If only only migration is left the first migration is selected as target instead of 0. So nothing is done.
I found a test case for this, but don't know why the test does not work:
https://github.com/robmorgan/phinx/blob/master/tests/Phinx/Migration/ManagerTest.php#L712

A negative assertion, only tests what shouldn't happen, not what does happen. Hmmm.

I played with the assertions. So ether add to assert not contains No migrations to rollback
or to contains some of this:
==<PHONE_NUMBER>5330 TestMigration: reverting
 ==<PHONE_NUMBER>5330 TestMigration: reverted 0.0000s

Hi.
If you can make the test assert positively (i.e. that when there is 1 or 0 migrations, the result is appropriate, then I can merge this.

I added both now. Positive assertion for the "reverting" and "reverted" status output.
Also the negative assertion to not contain "no migrations", so for what ever could go wrong, the test could fail on this.

Thank you.

@moritz-h (since you are frequent contributor): do not link to branches, use git tag (or commit hash) to link to sourcecode, because the line (or even file) referenced gets out of sync, so the links in issue make no sense in the future.

@glensc ok, I try to do this in future.


Column names not being translated

Column names are not being translated using provided translator instance.


A way to add extra keypoints to the current annotation format

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
There are 17 body points only in the COCO keypoint format. Is there a way that I can add more points to the same one detected person by using CVAT?
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response


fix: force useRegistry return type

Currently building nuxt-scripts makes all registry scripts function type return any in declaration files. Forcing the return type of useRgistryScript fix this issue
fix #72

Thanks!


docs: update CODEOWNERS to remove myself (boyney123)

Description
Sorry I have not been very active, I don't have much time at the moment, and it's unfair to have me on this list still.
Good luck with everything, I'm still watching/using everything AsyncAPI does and sharing the love with EDA folks.

related: https://github.com/asyncapi/cli/issues/671

/rtm


Metadata providers for manhwa and webtoons

There are many popular manhwa and webtoons that have very limited metadata on anilist, mangaupdates and MAL. Adding official (?) or more robust metadata providers for these types of content would be great.

afaik official providers usually have less metadata than MU. MU usually is best, but anime-planet could be an option, their collection is reasonably full and their thumbnails have a much higher resolution than MU.

Mangadex or Anime-planet seem to be the only decent sources for OEL currently.

closed in favor of #86


Made the changes for issue 125

🗣 Description
Addresses the comments for MEET.3.1v0.1 policy statement wording.
💭 Motivation and context
This change makes the policy wording more concise, but adding more context to the rationale
Fixes #125

@LaurenBassett Feel free to review. We can merge once we have concurrence from you and @buidav.


TypeError: ch_names must be a list, tuple, or int

When I run the sample code in Python 2.7，the error happened as follow：
import numpy as np
import matplotlib.pyplot as plt

from sklearn.pipeline import Pipeline
from sklearn.discriminant_analysis import LinearDiscriminantAnalysis
from sklearn.model_selection import ShuffleSplit, cross_val_score

from mne import Epochs, pick_types, find_events
from mne.channels import read_layout
from mne.io import concatenate_raws, read_raw_edf
from mne.datasets import eegbci
from mne.decoding import CSP

print(__doc__)

tmin, tmax = -1., 4.
event_id = dict(hands=2, feet=3)
subject = 1
runs = [6, 10, 14]  # motor imagery: hands vs feet

raw_fnames = eegbci.load_data(subject, runs)
raw_files = [read_raw_edf(f, preload=True, stim_channel='auto') for f in
             raw_fnames]
raw = concatenate_raws(raw_files)

# strip channel names of "." characters
raw.rename_channels(lambda x: x.strip('.'))

# Apply band-pass filter
raw.filter(7., 30., fir_design='firwin', skip_by_annotation='edge')

events = find_events(raw, shortest_event=0, stim_channel='STI 014')

picks = pick_types(raw.info, meg=False, eeg=True, stim=False, eog=False,
                   exclude='bads')

epochs = Epochs(raw, events, event_id, tmin, tmax, proj=True, picks=picks,
                baseline=None, preload=True)
epochs_train = epochs.copy().crop(tmin=1., tmax=2.)
labels = epochs.events[:, -1] - 2

scores = []
epochs_data = epochs.get_data()
epochs_data_train = epochs_train.get_data()
cv = ShuffleSplit(10, test_size=0.2, random_state=42)
cv_split = cv.split(epochs_data_train)

lda = LinearDiscriminantAnalysis()
csp = CSP(n_components=4, reg=None, log=True, norm_trace=False)

clf = Pipeline([('CSP', csp), ('LDA', lda)])
scores = cross_val_score(clf, epochs_data_train, labels, cv=cv, n_jobs=1)

class_balance = np.mean(labels == labels[0])
class_balance = max(class_balance, 1. - class_balance)
print("Classification accuracy: %f / Chance level: %f" % (np.mean(scores),
                                                          class_balance))
Automatically created module for IPython interactive environment
Extracting EDF parameters from C:\Users\user\mne_data\MNE-eegbci-data\physiobank\database\eegmmidb\S001\S001R06.edf...
EDF file detected
EDF annotations detected (consider using raw.find_edf_events() to extract them)
Setting channel info structure...
Creating raw.info structure...
Reading 0 ... 19999  =      0.000 ...   124.994 secs...
Extracting EDF parameters from C:\Users\user\mne_data\MNE-eegbci-data\physiobank\database\eegmmidb\S001\S001R10.edf...
EDF file detected
EDF annotations detected (consider using raw.find_edf_events() to extract them)
Setting channel info structure...
Creating raw.info structure...
Reading 0 ... 19999  =      0.000 ...   124.994 secs...
Extracting EDF parameters from C:\Users\user\mne_data\MNE-eegbci-data\physiobank\database\eegmmidb\S001\S001R14.edf...
EDF file detected
EDF annotations detected (consider using raw.find_edf_events() to extract them)
Setting channel info structure...
Creating raw.info structure...
Reading 0 ... 19999  =      0.000 ...   124.994 secs...
Setting up band-pass filter from 7 - 30 Hz
l_trans_bandwidth chosen to be 2.0 Hz
h_trans_bandwidth chosen to be 7.5 Hz
Filter length of 265 samples (1.656 sec) selected
Setting up band-pass filter from 7 - 30 Hz
l_trans_bandwidth chosen to be 2.0 Hz
h_trans_bandwidth chosen to be 7.5 Hz
Filter length of 265 samples (1.656 sec) selected
Setting up band-pass filter from 7 - 30 Hz
l_trans_bandwidth chosen to be 2.0 Hz
h_trans_bandwidth chosen to be 7.5 Hz
Filter length of 265 samples (1.656 sec) selected
Trigger channel has a non-zero initial value of 1 (consider using initial_event=True to detect this event)
Removing orphaned offset at the beginning of the file.
89 events found
Event IDs: [1 2 3]
45 matching events found
No baseline correction applied
Not setting metadata
0 projection items activated
Loading data for 45 events and 801 original time points ...
0 bad epochs dropped
Traceback (most recent call last):

  File "<ipython-input-1-212a350e05c6>", line 65, in <module>
    scores = cross_val_score(clf, epochs_data_train, labels, cv=cv, n_jobs=1)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\model_selection\_validation.py", line 321, in cross_val_score
    pre_dispatch=pre_dispatch)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\model_selection\_validation.py", line 195, in cross_validate
    for train, test in cv.split(X, y, groups))

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\parallel.py", line 779, in __call__
    while self.dispatch_one_batch(iterator):

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\parallel.py", line 625, in dispatch_one_batch
    self._dispatch(tasks)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\parallel.py", line 588, in _dispatch
    job = self._backend.apply_async(batch, callback=cb)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\_parallel_backends.py", line 111, in apply_async
    result = ImmediateResult(func)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\_parallel_backends.py", line 332, in __init__
    self.results = batch()

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\parallel.py", line 131, in __call__
    return [func(*args, **kwargs) for func, args, kwargs in self.items]

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\model_selection\_validation.py", line 437, in _fit_and_score
    estimator.fit(X_train, y_train, **fit_params)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\pipeline.py", line 257, in fit
    Xt, fit_params = self._fit(X, y, **fit_params)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\pipeline.py", line 222, in _fit
    **fit_params_steps[name])

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\externals\joblib\memory.py", line 362, in __call__
    return self.func(*args, **kwargs)

  File "C:\Users\user\Anaconda2\lib\site-packages\sklearn\pipeline.py", line 589, in _fit_transform_one
    res = transformer.fit_transform(X, y, **fit_params)

  File "C:\Users\user\Anaconda2\lib\site-packages\mne\decoding\mixin.py", line 33, in fit_transform
    return self.fit(X, y, **fit_params).transform(X)

  File "C:\Users\user\Anaconda2\lib\site-packages\mne\decoding\csp.py", line 172, in fit
    class_, reg=self.reg, method_params=self.cov_method_params)

  File "C:\Users\user\Anaconda2\lib\site-packages\mne\cov.py", line 1586, in _regularized_covariance
    info = create_info(data.shape[-2], 1000., 'eeg') if info is None else info

  File "<string>", line 2, in create_info

  File "C:\Users\user\Anaconda2\lib\site-packages\mne\utils.py", line 729, in verbose
    return function(*args, **kwargs)

  File "C:\Users\user\Anaconda2\lib\site-packages\mne\io\meas_info.py", line 1745, in create_info
    raise TypeError('ch_names must be a list, tuple, or int')

TypeError: ch_names must be a list, tuple, or int

@Rockingsnow I cannot replicate with current master branch:
alex@:mne-python(master)$ ipython
Python 2.7.12 |Anaconda custom (x86_64)| (default, Jul  2 2016, 17:43:17)
Type "copyright", "credits" or "license" for more information.

IPython 5.0.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

In [1]: %run debug5
Module created for script run in IPython
Classification accuracy: 0.933333 / Chance level: 0.533333```

@agramfort Why can you run successfully？ Is my development environment not configured right or what code is missing?

This is my version information:
import sys
print sys.version
print sys.version_info
2.7.13 |Anaconda, Inc.| (default, Sep 19 2017, 08:25:59) [MSC v.1500 64 bit (AMD64)]
sys.version_info(major=2, minor=7, micro=13, releaselevel='final', serial=0)
Should I  update my python to version 2.7.12 or 3.0 ?

what MNE version are you using?
what does :
mne.sys_info()
gives you?

Recently I have updated my MNE version to the latest one —version 0.16.2 :
NAME
mne - MNE software for MEG and EEG data analysis.
FILE
c:\users\user\anaconda2\lib\site-packages\mne_init_.py
PACKAGE CONTENTS
annotations
baseline
beamformer (package)
bem
channels (package)
chpi
commands (package)
connectivity (package)
coreg
cov
cuda
data (package)
datasets (package)
decoding (package)
defaults
dipole
epochs
event
evoked
externals (package)
filter
fixes
forward (package)
gui (package)
inverse_sparse (package)
io (package)
label
minimum_norm (package)
misc
parallel
preprocessing (package)
proj
realtime (package)
report
selection
simulation (package)
source_estimate
source_space
stats (package)
surface
tests (package)
time_frequency (package)
transforms
utils
viz (package)
DATA
version = '0.16.2'
VERSION
0.16.2

you need to use the latest master version (unreleased)
See https://mne-tools.github.io/dev/advanced_setup.html#advanced-setup
use something like this:
pip install --upgrade --no-deps
https://api.github.com/repos/mne-tools/mne-python/zipball/master

I don't know there is also a newer version of MNE!
All right, I will try soon.

Also the output of:
>>> import mne
>>> mne.sys_info()

Would be helpful. This is what @agramfort was asking for. It looks like you pasted something different above.

After updating my MNE to the latest version——mne-0.17.dev0 :
Platform:      Windows-10-10.0.15063
Python:        2.7.13 |Anaconda, Inc.| (default, Sep 19 2017, 08:25:59) [MSC v.1500 64 bit (AMD64)]
Executable:    C:\Users\user\Anaconda2\pythonw.exe
CPU:           Intel64 Family 6 Model 63 Stepping 2, GenuineIntel: 28 cores
Memory:        31.9 GB
mne:           0.17.dev0
numpy:         1.13.1 {lapack=mkl_rt, blas=mkl_rt}
scipy:         0.19.1
matplotlib:    2.0.2 {backend=TkAgg}
sklearn:       0.19.0
nibabel:       2.2.1
mayavi:        Not found
cupy:          Not found
pandas:        0.20.3
dipy:          Not found
The same error still occurred as follow:
File "C:\Users\user\Anaconda2\lib\site-packages\mne\utils.py", line 797, in verbose
return function(*args, **kwargs)
File "C:\Users\user\Anaconda2\lib\site-packages\mne\io\meas_info.py", line 1783, in create_info
("list, tuple, or int"))
File "C:\Users\user\Anaconda2\lib\site-packages\mne\utils.py", line 2915, in _validate_type
', got %s instead.' % (type(item),))
TypeError: ('ch_names', ' must be an instance of ', 'list, tuple, or int', ", got <type 'long'> instead.")

can you open your meas_info.py file (in your mne install) and see if it
works
when you replace this line:
https://github.com/mne-tools/mne-python/blob/master/mne/io/meas_info.py#L1780
by
if isinstance(ch_names, integer_types):
making sure you import integer_types from six like this:
from ..externals.six import string_types, integer_types

This process is effective and my code have run successfully as well, I think that this problem may be caused by the incompatible of integral type between python 2.7 ( support int and long type) and the code of version 3.5(only support int type).
your advice does solve my problem eventually，thanks！

great
@Rockingsnow do you want to give a try to contribute the fix to MNE with a pull request? or shall we do it?

I hope so, but I don't know the specific operation process or you can help me deal with it.

this should help
https://mne-tools.github.io/dev/contributing.html?highlight=contributing
and more generally for learning github :
https://help.github.com/articles/creating-a-pull-request-from-a-fork/

@Rockingsnow if you need help when attempting you swing by https://gitter.im/mne-tools/mne-python

I have the same issue. However, @agramfort 's fix
if isinstance(ch_names, integer_types):

doesn't solve the issue.
mne.sys_info():

Platform:      Windows-7-6.1.7601-SP1
Python:        2.7.15 |Anaconda custom (64-bit)| (default, May  1 2018, 18:37:09) [MSC v.1500 64 bit (AMD64)]
Executable:    C:\Users\Public\Anaconda2\pythonw.exe
CPU:           Intel64 Family 6 Model 78 Stepping 3, GenuineIntel: 4 cores
Memory:        7.9 GB
mne:           0.16.2
numpy:         1.12.1 {lapack=mkl_rt, blas=mkl_rt}
scipy:         1.1.0
matplotlib:    2.1.0 {backend=Qt5Agg}
sklearn:       0.19.1
nibabel:       2.0.2
mayavi:        Not found
pycuda:        Not found
skcuda:        Not found
pandas:        0.23.1
When I run this...
import mne
from mne.decoding import CSP

import numpy as np
from numpy import random

import pandas as pd
import matplotlib.pyplot as plt
import os

from sklearn.model_selection import cross_val_score
from sklearn.model_selection import ShuffleSplit
from sklearn.pipeline import Pipeline
from sklearn.discriminant_analysis import LinearDiscriminantAnalysis

def train_test_split(data, test_ratio):
  random.seed(42)
  shuf_ind = random.permutation(len(data))
  test_size = int(len(data) * test_ratio)
  test_idx = shuf_ind[:test_size]
  train_idx = shuf_ind[test_size:]
  return train_idx, test_idx

##############
# setting-up #
##############

# get the data
os.chdir(r'C:\Users\John Doe\surfdrive\Alpha\data\ML_format\features')
all_trials_raw = mne.read_epochs('all_trials_raw-epo.fif', preload=False)

# get the labels
os.chdir(r'C:\Users\John Doe\surfdrive\Alpha\data\ML_format\labels')
labels = pd.read_csv('response.txt',
                     delim_whitespace=True,
                     header=None,
                     names=['subjectID', 'condition', 'ISI', 'response'])
#labels.info()

# split training and test set
train_idx, test_idx = train_test_split(all_trials_raw, 0.2)
xtrain = all_trials_raw[train_idx]
ytrain = labels.response.iloc[train_idx]

## first, to test CSP, filter data in alpha range (no filter bank yet)
#xtrain.load_data().filter(l_freq=8.0, h_freq=13.0)

## after filtering, discard everything but 1 sec pre-stimulus (i.e. dump the buffer zones that were only there to protect against filter artifacts)
#xtrain.load_data().crop(tmin=-1.0, tmax=0)

# now let's apply CSP to the data and plot the filters
csp = CSP()
csp.fit(xtrain.get_data(), ytrain)

... the following error occurs:
runfile('C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py', wdir='C:/Users/John Doe/surfdrive/Alpha/code/pre_processing')
Reading all_trials_raw-epo.fif ...
Found the data of interest:
t =   -3000.00 ...    2996.09 ms
0 CTF compensation matrices available
3520 matching events found
Applying baseline correction (mode: mean)
3520 matching events found
Applying baseline correction (mode: mean)
Not setting metadata
0 projection items activated
Loading data for 2816 events and 1536 original time points ...
Traceback (most recent call last):
File "", line 1, in
runfile('C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py', wdir='C:/Users/John Doe/surfdrive/Alpha/code/pre_processing')
File "C:\Users\Public\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 705, in runfile
execfile(filename, namespace)
File "C:\Users\Public\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 87, in execfile
exec(compile(scripttext, filename, 'exec'), glob, loc)
File "C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py", line 65, in
csp.fit(xtrain.get_data(), ytrain)
File "C:\Users\Public\Anaconda2\lib\site-packages\mne\decoding\csp.py", line 172, in fit
class_, reg=self.reg, method_params=self.cov_method_params)
File "C:\Users\Public\Anaconda2\lib\site-packages\mne\cov.py", line 1586, in _regularized_covariance
info = create_info(data.shape[-2], 1000., 'eeg') if info is None else info
File "", line 2, in create_info
File "C:\Users\Public\Anaconda2\lib\site-packages\mne\utils.py", line 729, in verbose
return function(*args, **kwargs)
File "C:\Users\Public\Anaconda2\lib\site-packages\mne\io\meas_info.py", line 1745, in create_info
if not isinstance(ch_names, (list, tuple)):
TypeError: ch_names must be a list, tuple, or int

The integer_types is probably not as robust as:
import operator
try:  # integer
    ch_names = [operator.index(ch_names)]
except TypeError:
    pass

can you try that fix?
Also updating to latest mne master you will get a more informative message saying what the errant type is

Still throws the same error when I
import operator

and insert
try:  # integer
    ch_names = [operator.index(ch_names)]
except TypeError:
    pass

immediatly above https://github.com/mne-tools/mne-python/blob/master/mne/io/meas_info.py#L1780

@JohnDoenut can you tell us what is ch_names (type and value) before the try?

the data.shape[-2] argument taken by create_info in meas_info.py pertains to
xtrain.get_data()
xtrain.get_data().shape[-2]

gives 62L

What is type(np.long(10)) and type(operator.index(np.long(10)))? I'm surprised that the operator.index fix does not take care of this.

Actually instead could you try #5603 to see if it fixes your issue?

I applied the fix again, this time correctly as in https://github.com/mne-tools/mne-python/pull/5603.
I didn't remove
if isinstance(ch_names, int):

before.
This solves the ch_names issue.
However, another error occurs:
runfile('C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py', wdir='C:/Users/John Doe/surfdrive/Alpha/code/pre_processing')
Reading all_trials_raw-epo.fif ...
Found the data of interest:
t =   -3000.00 ...    2996.09 ms
0 CTF compensation matrices available
3520 matching events found
Applying baseline correction (mode: mean)
3520 matching events found
Applying baseline correction (mode: mean)
Not setting metadata
0 projection items activated
Loading data for 2816 events and 1536 original time points ...
Estimating covariance using EMPIRICAL
Done.
Estimating covariance using EMPIRICAL
Done.
Traceback (most recent call last):
File "", line 1, in
runfile('C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py', wdir='C:/Users/John Doe/surfdrive/Alpha/code/pre_processing')
File "C:\Users\Public\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 705, in runfile
execfile(filename, namespace)
File "C:\Users\Public\Anaconda2\lib\site-packages\spyder\utils\site\sitecustomize.py", line 87, in execfile
exec(compile(scripttext, filename, 'exec'), glob, loc)
File "C:/Users/John Doe/surfdrive/Alpha/code/pre_processing/FBCSP.py", line 65, in
csp.fit(xtrain.get_data(), ytrain)
File "C:\Users\Public\Anaconda2\lib\site-packages\mne\decoding\csp.py", line 198, in fit
eigen_values, eigen_vectors = linalg.eigh(covs[0], covs.sum(0))
File "C:\Users\Public\Anaconda2\lib\site-packages\scipy\linalg\decomp.py", line 493, in eigh
" computed." % (info-b1.shape[0]))
LinAlgError: the leading minor of order 62 of 'b' is not positive definite. The factorization of 'b' could not be completed and no eigenvalues or eigenvectors were computed.


Hi Adam, great work you have here. I would like to know if and how I can use this api from an android device,  I am building an android app for face recognition and AR, that I intend to run online by uploading an image online for processing

face_recognition version:
Python version:
Operating System:

Description
Describe what you were trying to get done.
Tell us what happened, what went wrong, and what you expected to happen.
What I Did
Paste the command(s) you ran and the output.
If there was a crash, please include the traceback here.

There's an simple example of running this as a web service here: https://github.com/ageitgey/face_recognition/blob/master/examples/web_service_example.py


How to erase the number sign on the generated files?

How to erase the number sign on the generated files?

I too have the same problem, did you find the solution?

I'm not quite sure what I'm testing in the index.js file and it looks for this line with the exact text signImage(#${editionCount}); and you delete it.


not able to install ruby not valid windows application.

hello

Hi, you need to give more information. Errors, steps to reproduce, etc.
Give me some more information and I'll reopen this issue.


Fix clearing of different directions.

When placing different (creative) energy containers it can become possible that directions will become added multiple times (EAST x2 etc).
This small PR will delete and remove the directions that are not yet checked.
The issue can be reproduced by placing 2 or more lazuli flux containers, which can become an issue with large counts of valid connections:
https://github.com/GabrielOlvH/Industrial-Revolution/blob/master/src/main/kotlin/me/steven/indrev/utils/machineinteractions.kt#L125

A log of the line I linked. 😄

Thanks for the PR!


DAO: Hit the cache for entity flagged as removed too

I came along this part of the code and I don't see any reason why the cache should not be used when fetching with the "removed" ones. It will help decrease the number of DB queries.
It can be merged in many CS versions

@DaanHoogland I agree that's the best practice though there is a glitch here. Following code returns null if _cache is not null but element is null,
    T result = null
    if (_cache != null) {
        final Element element = _cache.get(id);
        if (element != null) {
            result = (T)element.getObjectValue();
        }
    } else {
        result =findById(id, true, null);
    }
    return result;

Probably intention is to,
    T result = null
    if (_cache != null) {
        final Element element = _cache.get(id);
        if (element != null) {
            result = (T)element.getObjectValue();
        } else {
        result = findById(id, true, null);
    }
    return result;

@sateesh-chodapuneedi not quite, you're missing a case. If the cache is null (not configured) you will always return null, which is not the goal.

I pushed a change to write the method following the good practive you're mentioning. I also updated the sibling method findById to follow the same practice.

@marcaurele @sateesh-chodapuneedi You are both right probably more then I will ever be so please take with a grain of salt:
    T result = null;
    if (_cache != null) {
        final Element element = _cache.get(id);
        if (element != null) {
            result = (T)element.getObjectValue();
        }
    }
    if(result == null) {
        result = lockRow(id, null);
    }
    return result

same pattern would apply to the findIncludingRemoved variety. I think this is the cleanest though Marc's version should be semantically equal given a good optimising compiler.
@swill  I will start integration tests on this next.

LGTM based on code, CI pending

Thanks @DaanHoogland.  👍

@swill I am getting to many false positives in my environment to grant it value, can you schedule this for integration testing?

@DaanHoogland I am also getting more false negatives than I had in the past.  I am also pushing my environments a bit harder than I was before, so I think that is part of it.  I will add this one to my CI.

CI RESULTS
Tests Run: 94
  Skipped: 0
   Failed: 2
   Errors: 0
 Duration: 7h 45m 13s

Summary of the problem(s):
FAIL: Create a redundant VPC with 1 Tier, 1 VM, 1 ACL, 1 PF and test Network GC Nics
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_vpc_redundant.py", line 605, in test_04_rvpc_network_garbage_collector_nics
    self.check_routers_state(status_to_check="BACKUP", expected_count=2)
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_vpc_redundant.py", line 353, in check_routers_state
    self.fail("Expected '%s' routers at state '%s', but found '%s'!" % (expected_count, status_to_check, cnts[vals.index(status_to_check)]))
AssertionError: Expected '2' routers at state 'BACKUP', but found '1'!
----------------------------------------------------------------------
Additional details in: /tmp/MarvinLogs/test_network_BG7UFE/results.txt

FAIL: test_04_rvpc_privategw_static_routes (integration.smoke.test_privategw_acl.TestPrivateGwACL)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 277, in test_04_rvpc_privategw_static_routes
    self.performVPCTests(vpc_off)
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 324, in performVPCTests
    self.check_pvt_gw_connectivity(vm1, public_ip_1, vm2.nic[0].ipaddress)
  File "/data/git/cs2/cloudstack/test/integration/smoke/test_privategw_acl.py", line 559, in check_pvt_gw_connectivity
    "Ping to outside world from VM should be successful"
AssertionError: Ping to outside world from VM should be successful
----------------------------------------------------------------------
Additional details in: /tmp/MarvinLogs/test_network_BG7UFE/results.txt

Associated Uploads
/tmp/MarvinLogs/DeployDataCenter__May_10_2016_19_40_27_1Z9M75:

dc_entries.obj
failed_plus_exceptions.txt
runinfo.txt

/tmp/MarvinLogs/test_network_BG7UFE:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_privategw_acl_JUTOR4:

failed_plus_exceptions.txt
results.txt
runinfo.txt

/tmp/MarvinLogs/test_vpc_routers_009P6U:

failed_plus_exceptions.txt
results.txt
runinfo.txt

Uploads will be available until 2016-07-11 02:00:00 +0200 CEST
Comment created by upr comment.

I reran the tests that failed and they succeeded the second time, so CI is in a good place on this one.  I need one more code review on this one.  @marcaurele can you please force push this PR to kick of Jenkins again so when we get another code review we can merge right away.  Thanks...

@swill did a force push by changing the last commit description

Thank you sir.  Unfortunately travis failed this time.  Would you mind trying it again.  Thanks...

@marcaurele instead of force push you can close and reopen after a few seconds. This usually works as well and seves you switching windows browser->terminal->browser ;)

I don't get what's the relation between my change and the jenkins build failling. Should I trigger another one?

@marcaurele I don't think it is related to your code either. can you open/close your pr to see if it works. If not, maybe we need to empty the jenkins-workspace. The problems are not clear to me, might be jenkins, our job or something messy in our code. Any pointers are welcome.

Once I get #1539 merged, I will ask you to re-push since we are working to fix the issue that is causing Travis to fail in that PR right now.  Thx...

@marcaurele I have merged fixes to Jenkins and Travis, can you force push again to kick off those runs again.  Sorry for the inconvenience, hopefully this is the last time.  :P

Sorry to do this to you again @marcaurele.  Can you rebase and force push again.  I have disabled the test in master that was blocking this in Jenkins because we have verified that there are problems with that test.  Thx...

checks are finally ok...

I am counting my own code review on this one given the scope of the change and the fact that a review is the only thing holding this up.  I will merge this one...

@swill you should always count the opinion of the RM as that of a real memeber of the community ;)


Revert "Externalize kvm agent storage timeout configuration"

Reverts apache/cloudstack#4585

@blueorangutan package

@nvazquez a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian. SL-JID 590

@blueorangutan test

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1319)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 40004 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr5218-t1319-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Smoke tests completed. 89 look OK, 0 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

Lgtm


[WFLY-15900] 'connector-ref' configuration in ejb3 subsystem not acce…

…pted
https://issues.redhat.com/browse/WFLY-15900

Closing, as the associated JIRA was rejected.


PulpSolver Error with lscp_###.solve function

I plan to use the lscp.ipynb as a part of lab material in my class. After IT staff successfully installed spopt on desktops in the lab, we got an "PulpSolver Error: cannot execute cbc.exe" every time we ran lscp_###.solve functions.
After trying multiple ways, I find one way to run successfully as follows:
1, conda uninstall pulp
2, pip install pulp
3, conda install spopt
But still, I am curious about any way to eliminate the errors at the very beginning.

I plan to use the lscp.ipynb as a part of lab material in my class. After IT staff successfully installed spopt on desktops in the lab, we got an "PulpSolver Error: cannot execute cbc.exe" every time we ran lscp_###.solve functions.
After trying multiple ways, I find one way to run successfully as follows: 1, conda uninstall pulp 2, pip install pulp 3, conda install spopt
But still, I am curious about any way to eliminate the errors at the very beginning.

Hi Xin I think we had a previous issue #347.
Adding a line of code might solve the problem.
'solver = pl.PULP_CBC_CMD(keepFiles=True)

Also: xref #348.

solver = pl.PULP_CBC_CMD(keepFiles=True)

Thanks very much!

Got it! Thanks!


Local paths included in hierarchy.

Currently the hierarchy root is resolved to a path on my local machine.
Instead, it should default to the project root.
However, if "basePath" is specified within typedoc.json it should use that as the root.

fixed v4.0.1

Thanks, great theme! But, unfortunately, the same problem... Mac os m1, v4.1.2

I solved this problem for myself as follows:
navigation.js
const navigation = (context) => (props) => {
let categories = formatFileHierarchy(props.model.project.children || []);

do {
    var key = Object.keys(categories.categories)[0]
    categories = categories.categories[key]
}
while (key != "front")

thereby removing the unnecessary branch above "front"

Just checked
Apparently this is because entryPoints is set in relative in my configuration
"entryPoints": ["../../front/"],

in fact, even if I do
{
  "entryPoints": ["src"],
  "entryPointStrategy": "Expand",
  "out": "public/docs",
  "plugin": ["typedoc-theme-hierarchy"],
  "theme": "hierarchy",
  "tsconfig": "./tsconfig.json",
  "name": "Project name"
}

it will still capture the "src" folder


Fix directory and reports stop channel urls

Report stop channel and Directory stop channel methods uses a different service path then their default class. When the lib build the full service url, the service path is duplicated.

I signed it!

How about a test for this?

@bshaffer, is it ok now? Please, where can I report that the service URL at these four pages should be fixed?

https://developers.google.com/admin-sdk/directory/v1/reference/channels/stop: service URL and API Explorer;
https://developers.google.com/admin-sdk/reports/v1/reference/channels/stop: service URL and API Explorer;
https://developers.google.com/admin-sdk/directory/v1/guides/push#stopping-notifications: service URL;
-https://developers.google.com/admin-sdk/reports/v1/guides/push#stopping-notifications: service URL


Refactor clang-tidy file for tests

Make Tests/.clang-tidy inherits from the parent .clang-tidy. Disable checks for magic number and identifier length

Description

Fixes #170
Fixes #173

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (7a0d43a) 81.81% compared to head (2e6197b) 81.81%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #174   +/-   ##
=======================================
  Coverage   81.81%   81.81%
=======================================
  Files          16       16
  Lines         451      451
=======================================
  Hits          369      369
  Misses         82       82

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Define defaults for React

We might need some default set for each framework, in order to skip the post framework selections step with a flag.
This issue is about picking the defaults for React

Could I take this issue?

Of course! @giuseppesciacca  Thank you 😄

PR: https://github.com/Giuliano1993/make-js-component/pull/90


Button overflow on OTP Screen

Describe the bug
A send verification otp button is overflowing in the OTP screen.
Expected behavior
The button should be a little more correct in dimension so the text wont overflow
Screenshots

Smartphone :

Device: Realme 3 Pro

Kindly Assign this issue to me Under GSSoC 2023

Kindly assign me this issue under GSSoC 2023. I can fix it  easily.

@m0hit-kumar kindly assign this issue under GSSoC 2023
I have a good experience in flutter and can solve this easily!!

please assign me this issue under GSSoc 2023

please assign me this issue under GSSOC

Assign me this issue under IWOC'24 @Pratik-11

Assign me this issue under IWOC'24 @Pratik-11

sure go ahead and look for other things on the onboarding that are overflowing and fix it

@m0hit-kumar  you have assigned this issue to the wrong person ! can you reassign it?

Hi! I 'm Anjali.
Approach:

flexible
Expanded
Can you assign it to me under IWOC 24


Deck name character limit isn't checked when importing decks

I was building the official starter decks, as I considered they should perhaps be easily available somewhere. But when naming the first deck, "Starter Deck 1: War of Darkness", I was disappointed to see there is a 30 character limit to deck names.
Because the character limit seemed arbitrary, I tried editing it manually. I exported the deck and opened the resulting fecudk file. I changed the name field from "Starter Deck 1" to "Starter Deck 1: War of Darkness". I then reimported the deck.
The deck seemed to import without any problems. I could swap the deck in and out. I could add and remove cards. I could even save changes to the main character. The only problem was, if I tried to edit the deck name, I could not save a new (or the same) deck name greater than 30 characters long.
I can think of two solutions:

Increase (or remove) the deck name character limit. If the limit is indeed arbitrary, I think it should be greater than 30 characters.

Perform deck name sanitization when importing decks. If there are potential risks to increasing the character limit, then decks with names longer than the allowed length should be rejected or their names truncated.

The deck name length is 100% arbitrary, it was actually put at @ZaFwamingDwagon 's suggestion to keep me from having to truncate deck names in the drawer. As I never cared if a user manually edited a deck file, and editing the deck file would result in no app breaking issues since strings are by default sanitized in react, I didn't think to clean decks on import.
Any suggestions @ZaFwamingDwagon ?
PS: Except for the newest Roy deck, all the other decks I had already put together and linked in the forum post, I guess I should probably link to them on the github page itself too. Here's the link if you are curious https://drive.google.com/folderview?id=0B1sqS5ok8pcDeWc5NG5BeUJXOEU&usp=sharing

Ah, somehow I missed that you'd already made the decks. Thanks for re-sharing the link.

We can remove the limit. Visually, I suppose we should implement truncation in the deck drawer if we do so. We might have to investigate in sanitation as suggested eventually.

There isn't really any more sanitation to be done. This isn't a website where someone would actually put effort into somehow injecting code to steal personal data. This is a self contained app. Even if somehow they bypassed the sanitation that happens when JSON is parsed, and then the sanitation that happens when react displays a string, it's their own fault and it only affects them. Like, the source code is out there, there'd be no reason for them to try to inject code in an app where they can simply edit the source code.
If anything, we can make sure the app doesn't throw a fit if they make a deck without a name by hand, or if they have invalid ID's by hand for the cards in a deck.

This should be fixed in v0.9.0


Formatter is broken

While working on a file it randomly stopped working and I got this:
"Traceback (most recent call last):
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/runpy.py", line 87, in _run_code
exec(code, run_globals)
File "/Users/mpoplow/c_formatter_42/c_formatter_42/main.py", line 63, in
main()
File "/Users/mpoplow/c_formatter_42/c_formatter_42/main.py", line 43, in main
print(run_all(content), end="")
File "/Users/mpoplow/c_formatter_42/c_formatter_42/run.py", line 29, in run_all
content = clang_format(content)
File "/Users/mpoplow/c_formatter_42/c_formatter_42/formatters/clang_format.py", line 74, in clang_format
raise RuntimeError(f"clang-format error: {err.decode()}")
RuntimeError: clang-format error: Configuration file(s) do(es) not support Objective-C: /Users/mpoplow/prj/getnextline_WORK/.clang-format
42-c-format: Please read README and check requirements."
Idk, I tried restarting, reinstalling of both the extension and VS-Code, restarting and switching the computer and much more. I don't know how to fix it.

RuntimeError: clang-format error: Configuration file(s) do(es) not support Objective-C: /Users/mpoplow/prj/getnextline_WORK/.clang-format

It seems there might be an issue with your clang-format configuration file. Can you check it?


chore(neo4j): minor amend to tests

Reasoning 💡
After a read through @balazsorban44 's refactor I made a few minor amends. Thanks btw, I leadned a lot!

Add to tests to fail if userId foreign key exists on session or account.
CreateSession, don't add userId on Session node
For the above change to CreateSession, amend format.to to allow nested objects
On create statements, = can be uses instead of +=

Checklist 🧢

[ ] Documentation
[x] Tests
[x] Ready to be merged

Affected issues 🎟
Following on from PR https://github.com/nextauthjs/adapters/pull/314

No problem. Thanks for the review. Cheers


Allow Node.js LTS ‘Gallium’ (v16) release version

This PR lowers the minimum Node.js 16 and npm versions allowed
Why? Across our projects we have a mismatch in our supported Node.js versions. This makes npm link a bit painful with tools like nvm and asdf as they partition dependencies by Node.js version
lts/gallium → https://github.com/alphagov/govuk-frontend/blob/main/.nvmrc
16.14.2 → https://github.com/alphagov/govuk-design-system/blob/main/.nvmrc
16.16.0 → https://github.com/alphagov/govuk-prototype-kit/blob/v13/.nvmrc
Before
Latest Node.js LTS ‘Gallium’ (v16) release when https://github.com/alphagov/govuk-frontend/pull/2842 was opened
"engines": {
  "node": "^16.17.0",
  "npm": "^8.5.0"
},

After
All Node.js LTS ‘Gallium’ (v16) releases from initial LTS launch are now supported
"engines": {
  "node": "^16.13.0",
  "npm": "^8.1.0"
},

Also opened:
https://github.com/alphagov/govuk-design-system/pull/2399
https://github.com/alphagov/govuk-prototype-kit/pull/1712

Update now https://github.com/alphagov/govuk-frontend/pull/2939, https://github.com/alphagov/govuk-prototype-kit/pull/1712 and https://github.com/alphagov/govuk-design-system/pull/2399 are merged:
lts/gallium → https://github.com/alphagov/govuk-frontend/blob/main/.nvmrc
lts/gallium → https://github.com/alphagov/govuk-design-system/blob/main/.nvmrc
lts/* → https://github.com/alphagov/govuk-prototype-kit/blob/v13/.nvmrc
Once development environments are upgraded, package.json engines field changed, GitHub workflows bumped up, tests pass (and Heroku stacks upgraded to match) etc we may also want the lts/* alias everywhere