enforce: whitespace after catch

I realized we forgot to add catch to the enforce whitespace after operator list.
switch was already sticking to the style guide. So this is the result of a simple sed:
sed -i "s/catch(/catch (/" **/*.d

Btw a short look the D spec yielded the following statements - do we consider one of them as operator?
mixin
version
scope
synchronized
with

LGTM

LGTM

Auto-merge toggled on


Fixing merge problems

The headers in the collectives module where added concurrently to the assertion module, thus where missed while fixing the related #include changes.

I'll go ahead and merge this to make master green on CircleCI.


Use an actual database for all docs examples

It would be extremely helpful if the entire documentation set could use one actual database that you could run queries against.
Some benefits of this approach would include:

Better reproducibiliy and manual testability when writing docs, e.g., right now I am working on #2555 and I am re-populating a table "by hand" (possibly on 3 different binary versions) to test a single code snippet wrapped in a transaction

Better automated testing (see #182)

Better practices can be enforced if everything is in one jointly-maintained DB (see #190)

Automation of various kinds can be explored that are not possible right now (related in spirit to #2819)

Users could run the examples very easily while learning the DB.  For example, if this hypothetical DB were a cockroach gen output, they could have their own copy of the exact DB used to document everything running locally, eliminating any time lost to "guess and check" with the example code on the site.  The user can just copy-pasta it right into the prompt and see the exact same results.

Examples would be "cut from the same cloth", in much the same way that C.J. Date's books use the parts database.  This helps reader comprehension since the context for the example is more easily understood.  Once you learn the parts database you can just focus on understanding the section about JOINs or whatever, you aren't thinking too much about other aspects of the tables.

A drawback is that it might not be that easy to design an example database that covers all of our documented features.  And it could prove hard-ish to maintain for various other reasons, such as needing to update cockroach gen (if we went that way, but it seems like that would be the best way in principle to accomplish this).
However, I think the benefits would outweigh the drawbacks.  I think it would be a big win for users.

@rmloveland, I think this is a larger project that we're going to have the time for in 2.1, so I'm marking this as "later."  But it's definitely something we should investigate more, perhaps in conjunction with Product.

This is the database we used for the 2.0 meetup
Bandwidth permitting we are moving things related to MovR here

Now that Movr has been open-sourced, it's the obvious choice here: https://github.com/cockroachdb/movr

@danhhz built this into the database directly at this point. This should remove a lot of boilerplate from certain docs.
Nathaniels-MacBook-Pro:~ nate$ cockroach demo movr
#
# Welcome to the CockroachDB demo database!
#
# You are connected to a temporary, in-memory CockroachDB
# instance. Your changes will not be saved!
#
# Web UI: http://<IP_ADDRESS>:57764
#
# Server version: CockroachDB CCL v19.2.0-alpha.20190606-273-g95979f7 (x86_64-apple-darwin14, built 2019/06/21 13:46:32, go1.12.5) (same version as client)
# Cluster ID: 2d7ba835-2cc0-4e2a-bab2-dc6074c619e0
#
# Enter \? for a brief introduction.
#
root@<IP_ADDRESS>:57763/movr> select * from users limit 1;
                   id                  |   city    |    name     |      address       | credit_card
+--------------------------------------+-----------+-------------+--------------------+-------------+
  c28f5c28-f5c2-4000-8000-000000000026 | amsterdam | Maria Weber | 14729 Karen Radial |<PHONE_NUMBER>
(1 row)

Time: 996µs

root@<IP_ADDRESS>:57763/movr>

As the new title suggests, we'll roll this change out in phases...
Breaking down Phase 1 into smaller tasks:

[ ] Update first example: Learn CockroachDB SQL
[ ] Identify other examples that can use the movr dataset as-is (starting with SQL statement examples).
[ ] Identify how the movr dataset needs to be updated to accommodate other examples.
[ ] Start updating first batch of "as-is" examples (likely SQL statement examples).

... Phase 1 might take a couple sprints to complete.
Tentative Phase 2 (not in the scope of this sprint):

[ ] Update movr dataset to accommodate all examples.
[ ] Identify next batch for updates.

I'm going to close this issue.
See #5040, #5041, and #5061 for issue coverage.
See #5075 for first-round updates.


Add guidelines for naming patterns when creating a new content

When creating a new content type, whenever the naming convention is not followed there is a field validation error.
The suggested enhancement would be to have the suggestion instead of the Invalid_pattern message, display a guideline e.g. Invalid pattern. Naming convention should be in snake case

Good point! Will add description bellow the input field


Update for Travis CI to test against spark 2.0

Address #359

Well look at that -- awesome! 👍

Well that was straightforward 🎉  thanks @boazmohar ! Can you add a note to the README as part of this PR saying that it works with Spark 2.0.0?

Yep, was very easy. I guess the parts that changed a lot in 2.0 are DataFrame and Dataset not RDD's.

I've been thinking that it would be interesting to see if we could get any speed-ups by switching to DataFrames -- haven't had to play around with it yet though.

I've been thinking that it would be test whether there might be any speed-up in switching things to DataFrames -- though I haven't had any time to play with it.

Yeah agreed @jwittenbach, we should definitely look into that, I think the Dataset API may be what we want to look at. But in the meantime let's merge this in, good to make clear it's working with the latest version.


PyPI not really pure python

Since PyPI requires Numpy and SciPy, calling PyPI a "pure python" implementation is a bit of a misnomer. Both Numpy and SciPy make heavy use of "C" code. Anybody searching for a "pure java" will be looking for functions that they can run in an environment (such as jython) that does not require "C" or other non-"truly pure python" implementations. I would suggest removing the "pure python" vocabulary from the webpage and project documentation or at least put a disclaimer up front near the "pure python" claims.

Hi @mstupak. Are you confusing PyPI ("The Python Package Index") with pypy (self-hosting Python)?

Thanks @di ! Obviously I got a little crazy following links and ended up in an unintended repository.


Reset async_event_loop_ only if initialized (#354)

Pick https://github.com/triton-inference-server/python_backend/pull/354 PR into r24.04


Fix flex-grid-column with nowrap child

On the last version of chrome, the child of a flex-grid-column(null) with
white-space: nowrap;
overflow: hidden;

doesn't have a limited size, and the full content is showed in one line exceeding the row.
min-width: 0; fix this problem, probably by remembering to chrome to evaluate the column width with a 0px content width (it's what flex: 1 1 0px should do).
https://stackoverflow.com/questions/34934586/white-space-nowrap-and-flexbox-did-not-work-in-chrome

Yeah, you really need more contributors ;)

Nice catch

Just an example on how bad the min-width: 0 turns out!
http://codepen.io/DaSch/pen/YqgPBY
Adding min-width: initial; fixes the problem!

@DaSchTour Thanks. I'll try to resolve it.


Didn't work for bounds (Bounding Box)

I tried to use it with the bounds property: bounds=sw_latitude,sw_longitude|ne_latitude,ne_longitude
but it returns: { statusCode: 400,
data: '{"error": {"text": "One or more parameters are invalid in request", "id": "INVALID_PARAMETER", "field": "bounds"}}' }
Any help to make this work? I tried like this:
yelp.search({ term: 'food', bounds: '42.455242, -71.239708, 42.341274, -70.993417'  })

Nevermind, I used the wrong format it was supposed to be like this:
yelp.search({ term: 'food', bounds: '42.455242, -71.239708 | 42.341274, -70.993417' })
It is working, closing this now.


Invalid password format message for valid passwords including spaces

Current behavior:

Configure a React web site with awsmobile/aws-amplify configuration as documented on aws-amplify readme + https://docs.aws.amazon.com/aws-mobile/latest/developerguide/web-add-user-sign-in.html
Run the site with "npm start"
On the "Sign In Account" page click the "Sign Up" option in the bottom left of the screen
Add any user name (TestUser works) and "test TEST 01234!@#$%" (without the outer quotes) as a password.
Click the "Sign Up" button. The page prompts with an "Invalid password format" message.
Remove the spaces from the password ("testTEST01234!@#$%") and click Sign Up - the warning message goes away.
Checking the network tab after selecting the "Sign up" button shows two network request (first OPTION, then POST).  to https://cognito-idp.us-east-1.amazonaws.com/   The reply is 400 (Bad Request).    Headers include X-Amz-Target:AWSCognitoIdentityProviderService.SignUp and X-Amz-User-Agent:aws-amplify/0.1.x js

Hi @bizarosteve
the following link points to a reference to the password policy. space is not allowed in the password.
https://docs.aws.amazon.com/cognito/latest/developerguide/user-pool-settings-policies.html


Addon registration with gitlab can't work

Hi, Team
Describe the bug
I want to use gitlab as my addon registry. But when I registered Gitlab to Vela, I found that the registration failed. Could you help
me  fix it, thanks a lot.
KubeVela Version
CLI version: 1.7.4
Core version: 1.7.4

You can add a gitlab registry with the following command:
vela addon registry add my-repo --type gitlab --endpoint=http://gitlab.cmss.com/xxx/catalog --path=addons --gitlabRepoName=catalog --gitToken=<git token>
note:

the gitlabRepoName must be set
the path  must be set when addons are not in root of registry
the endpoint must be set the values contains repo name


Get schedule notification id

How can I get the schedule notification Id?
I need to schedule weekly notifications, but I need to remove them when I want it. Any suggestion?

you can assign id when you schedule notification. use that Id to cancel


Is_chief

Hello,
I have some problem with training.
When I start 2 workers with command :
Python iccgan.py walk --rank 0 --world_size 8
Python iccgan.py walk --rank 1 --world_size 8
The training don't start, it's like last issue posted 2 days before (stop in time.wait)
I checked json file path, ok
But if I check in agent_wrapper (iccgan.py), rank is always -1 then is_chief is always False...
Do you know what ?
(When I force rank to 0, it can start 1 worker)
Thanks a lot

The world_size option is the total number of workers that are going to start. If you just launch two worker threads with rank 0 and 1, the program will wait for the other 6 threads to start because you set world_size as 8.

Oh !!! I'm dumb...
Thanks a lot !


STG80

API changes have been detected in azure-storage-file-share. You can review API changes here
API changes
+     ivar provisioned_bandwidth_mi_bps: int

API changes have been detected in azure-storage-blob. You can review API changes here
API changes
+             include_system: bool = ...,
+             include_system: bool = ...,

API changes have been detected in azure-storage-file-share. You can review API changes here
API changes
+     ivar provisioned_bandwidth: int


AS3 container v3.5.0 - "delete single tennants" feature missing

ISSUE TYPE

Feature Idea

AS3 BUILD/ VERSION
Docker Container - v3.5.0

BIGIP VERSION
BIG-IP 14.0.0 Build 0.0.2187 Final

SUMMARY
Using guide https://clouddocs.f5.com/products/extensions/f5-appsvcs-extension/3/refguide/as3-api.html#delete-ref, I'm missing a "delete single tennants" function using the Docker container.

I can delete all AS3 applications on a specific host using a POST request to "https://as3-container:8443/mgmt/shared/appsvcs/declare", but I cannot POST to "https://as3-container:8443/mgmt/shared/appsvcs/declare/<tennant>".

STEPS TO REPRODUCE
example declaration:
POST https://as3-container:8443/mgmt/shared/appsvcs/declare
{

    "class": "AS3",
    "action": "deploy",
    "targetHost": "<IP_ADDRESS>",
    "targetUsername": "as3-admin",
    "targetPassphrase": "as3-admin",
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.0.0",
        "id": "container",
        "label": "Sample 1 in a container",
        "remark": "Simple HTTP application with RR pool",
        "AS3_App_01": {
            "class": "Tenant",
            "VS_App01": {
                "class": "Application",
                "template": "http",
                "serviceMain": {
                "class": "Service_HTTP",
                "virtualAddresses": [
                    "<IP_ADDRESS>"
                ],
                "pool": "Pool_App01"
                },
                "Pool_App01": {
                "class": "Pool",
                    "monitors": [
                        "http"
                    ],
                "members": [{
                    "servicePort": 80,
                    "serverAddresses": [
                        "<IP_ADDRESS>",
                        "<IP_ADDRESS>",
                        "<IP_ADDRESS>"
                    ]
                }]
                }
            }
        }
    }
}

Deleting this tennant could be done the following way:
POST https://as3-container:8443/mgmt/shared/appsvcs/declare/AS3_App_01
{

    "class": "AS3",
    "action": "remove",
    "targetHost": "<IP_ADDRESS>",
    "targetUsername": "as3-admin",
    "targetPassphrase": "as3-admin"

}

EXPECTED RESULTS
Get a success message, which indicates the tennant has been removed.
ACTUAL RESULTS
Error message:
{
    "code": 400,
    "message": "method \"Post\" is currently not allowed on path /AS3_App_01"
}

Hi @net47 ,
I am sorry to hear that issue happened to you, and thank you for raising up .
I  have created an internal issue, and I am going to check and look at the issue, and meantime if you have any more concerns, feel free to let me know (or direct ping me Le in our slack channel #f5-appsvcs-extension) for any AS3 container questions.
Thanks,
-Le

Hi @net47
After I have talked with the team, I have some updates that with AS3 container, you can remove the specific tenant is do a POST with tenant body empty. In your case, you can try this declaration
POST https://as3-container:8443/mgmt/shared/appsvcs/declare
{
"class": "AS3",
"action": "deploy",
"targetHost": "<IP_ADDRESS>",
"targetUsername": "as3-admin",
"targetPassphrase": "as3-admin",
"declaration": {
"class": "ADC",
"schemaVersion": "3.0.0",
"id": "container",
"label": "Sample 1 in a container",
"remark": "Simple HTTP application with RR pool",
"AS3_App_01": {
"class": "Tenant"
    }
}

}
FYI:
AS3 has no POST on tenant path currently supported, but I'm opening an issue and will prioritized it. Stay tuned.
Please let me know if you have other questions.
Thanks,
-Le

Hi Le,
sorry for the late response, I just tried that and it's working.
Response:
{
    "results": [
        {
            "message": "success",
            "lineCount": 23,
            "code": 200,
            "host": "<IP_ADDRESS>",
            "tenant": "AS3_App_01",
            "runTime": 22892
        }
    ],
    "declaration": {
        "class": "ADC",
        "schemaVersion": "3.0.0",
        "id": "container",
        "label": "Sample 1 in a container",
        "remark": "Simple HTTP application with RR pool",
        "controls": {
            "archiveTimestamp": "2018-11-12T15:00:16.649Z"
        }
    }
}

Thanks for your help!
br
Daniel

No problem @net47 Daniel.
I'm glad it wok and always happy to help.
Please let us know if there are other issues or you can find us through the Slack chance f5-appsvcs-extention.

Closing this issue as the workaround verified works.  We are tracking the feature enhancement part internally.
Please keep an eye out for new releases. Thanks!


ENH: sparse: Add indexing for 1D arrays

Adds support for indexing with 1D arrays.
Builds on #19833 (csr-1d) to provide indexing for 1D CSR arrays, though this code also works for DOK with minor changes after #19715 (dok-1d) gets merged.
The diff will be much easier to read after #19833 is merged. Look at only the last commit to see the diff for just this PR. I will rebase as needed, but I think the CSR PR will be merged soon and the diff here will become cleaner.

adds test_indexing1d.py
refactors IndexMixin._validate_indices to use an ndim-independent approach for both 1d and 2d.
the supported formats will need methods _get_int, _get_slice and _get_array. These are dispatched to after processing the index.
this tries to incorporate all improvements from the recently merged #19957 (and doesn't change those tests).
the ndim-independent approach means some helper functions are no longer needed and/or have been inlined. The two remaining helper functions are now methods so they have access to self.ndim. They are _as_indices and _compatible_boolean_array.

I've changed the tests to use np.testing assert_equal and assert_allclose in these new tests. I'll create a separate PR to change the new-but-merged-tests to use them too.
I've also added indexing support for np.newaxis/None (though it can't make 3D+). And that was an excuse to revamp some of the getitem/setitem code to be easier to move to nd.  The ndindex library gives almost everything we need, but doesn't support boolean sparse arrays as indexes. So we can now index 2d and 1d sparse arrays.
I believe the test failures are not related to this PR.
Note: This PR does not change reduction operations to have them return 1D sparse arrays. That will be a separate PR.
Note: Another separate PR: We need to decide how to handle 0D. A[3,4] should return a 0D object as per array_api. But should it return a numpy or scipy.sparse 0D object? In general, an array type "should" return it's own type when indexed. But sparse is not a standard array type -- we don't expect the entire array api to be implemented. But the answer to this question will impact this same code.

I've made the suggested changes. Thanks for the note about Python 3.10+ being supported (3.9 dropped)

I updated as suggested (and realized why the else block suggestion was correct -- we don't need to convert the dtype at that location.
This should be ready. The failing test seems unrelated.
@stefanv you looked at this some a while back -- can you take another look?

This looks ready to merge from my end. @stefanv feel free to push the button if you agree.

Thoughts for release notes:
In New features section for sparse:

sparse arrays now support indexing for 1D-arrays using ints, slices and arrays.
Methods min, max, nanmin, nanmax return 1D COO arrays when axis
is specified.

Getting ahead of myself, but feedback is welcome:
Once the migration guide document is available we can make a Highlights entry something like:

sparse arrays are now fully functional replacements for sparse matrices.
Consider converting your code to use sparse arrays. See migration_to_sparray.rst
for guidelines to this process and open an issue if you have questions. We have
dedicated reviewer bandwidth to help with these conversion over the next two releases.

Thank you, @dschult!


⚠ Fortnite has degraded performance

In 3c5d8d6, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1546 ms

Resolved: Fortnite performance has improved in c56b274.


Conversation: decorate v

This isn't intended to necessarily be the final resting place for this functionality; I'd just like to submit that the DX of building the small vella projects I've attempted thus far has been improved using this API. The single import is certainly nice, as is the cohesive feel of the v prefix for all v/S activities.

demi-godref invalidates this. Long live the Godref 😏


[SPARK-39795][SQL] New SQL function: try_to_timestamp

What changes were proposed in this pull request?

Add a new function try_to_timestamp. It is identical to the function to_timestamp, except that it returns NULL result instead of throwing an exception on string parsing error under ANSI SQL mode.
Why are the changes needed?

Similar to the other try_* functions from https://issues.apache.org/jira/browse/SPARK-35161, users can manage to finish queries without interruptions in ANSI mode.
The function to_timestamp is popular. There is a try_to_timestamp function in snowflake as well: https://docs.snowflake.com/en/sql-reference/functions/try_to_timestamp.html
Does this PR introduce any user-facing change?

Yes, a new function try_to_timestamp.  It is identical to the function to_timestamp, except that it returns NULL result instead of throwing an exception on string parsing error.
How was this patch tested?

UT

late LGTM


Remove does not work on repeated field

syntax = “proto3”;

message Items {
  repeated Item items = 1;
}
message Item {
  int32 id = 1;
}

Using remove on the generated code does not work correctly:
final items = Items(
  items: [
    Item(id: 1),
    Item(id: 2),
    Item(id: 3),
    Item(id: 4),
  ],
);

print(items.length); // prints 4
items.items.removeAt(2);
print(items.length; // prints 4 should be 3

Nvm it seems to work now, not sure what was wrong


[MINOR] Add timeout for github check test-hudi-hadoop-mr-and-hudi-java-client

Change Logs
Adds 40 minute timeout for github check test-hudi-hadoop-mr-and-hudi-java-client
Impact
NA
Risk level (write none, low medium or high below)
low
Documentation Update
NA
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

Failures are from flaky test. Going ahead w/ merging to unblock other CI runs.


minimal kernel version of merbridge?

With https://github.com/kubernetes/system-validators/pull/37 in v1.32, kernel version should be 4.19+; recommend version 5.8 for cgroup v2.

The minimum supported version is 5.7, if it is 5.8 then it is fully supported.


Configure Azure Pipelines

This pipeline configuration builds the application and produces a build for the Release configuration.

 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.:x: maxfjacob sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Intermittent bug: loop count limit error

LogMass: Error: Reached loop count limit while flushing commands

Haven't seen in a while, closing


Fix/#544 modal style chaining

related to ticket #544

Codecov Report

Merging #563 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master   #563   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          55     55
  Lines         567    567
=====================================
  Hits          567    567

Impacted Files
Coverage Δ

packages/modal/src/Modal.js
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@bertya ✅LGTM, but I don't see an approve button anywhere.


DCTL failed with error C2679 when build by msvc on Windows

DCTL failed with error C2679 when build by msvc on Windows, I use latest version 2ee0699 on master branch. Could you please help take a look at this? Noted that this issue only found when compiles with unreleased vctoolset, that next release of MSVC will have this behavior.
You can repro this issue as the steps below:

git clone https://github.com/rhalbersma/dctl.git D:\DCTL\src
git clone https://github.com/rhalbersma/xstd.git D:\DCTL\xstd
git clone https://github.com/rhalbersma/int_set.git D:\DCTL\int_set
extract boost_1_67_0.zip to D:\DCTL\boost_1_67_0\x64
open a VS 2017 amd64 command prompt as admin and browse to D:\DCTL
cmake -G "Visual Studio 15 2017 Win64" -DCMAKE_SYSTEM_VERSION=10.0.17134.0 -DBOOST_ROOT=D:\DCTL\boost_1_67_0\x64\boost -DBOOST_INCLUDEDIR=D:\DCTL\boost_1_67_0\x64 ..\src\
msbuild /m /p:Configuration=Release;Platform=x64 dctl.sln /t:Rebuild

Our workaround:
database.hpp:
std::copy(m_files.begin(), m_files.end(), std::ostream_iteratorstd::string(std::cout, "\n"));
=>
std::copy(m_files.begin(), m_files.end(), std::ostream_iterator<std::wstring, wchar_t>(std::wcout, L"\n"));
Error info:
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\xutility(1784): error C2679: binary '=': no operator found which takes a right-hand operand of type 'std::filesystem::path' (or there is no acceptable conversion)
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\iterator(232): note: could be 'std::ostream_iterator<std::string,char,std::char_traits> &std::ostream_iterator<std::string,char,std::char_traits>::operator =(std::ostream_iterator<std::string,char,std::char_traits> &&)'
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\iterator(232): note: or       'std::ostream_iterator<std::string,char,std::char_traits> &std::ostream_iterator<std::string,char,std::char_traits>::operator =(const std::ostream_iterator<std::string,char,std::char_traits> &)'
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\iterator(208): note: or       'std::ostream_iterator<std::string,char,std::char_traits> &std::ostream_iterator<std::string,char,std::char_traits>::operator =(const _Ty &)'
with
[
_Ty=std::string
]
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\xutility(1784): note: while trying to match the argument list '(std::ostream_iterator<std::string,char,std::char_traits>, std::filesystem::path)'
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\xutility(1821): note: see reference to function template instantiation '_OutIt std::_Copy_uncheckedstd::filesystem::path*,_Iter(_InIt,_InIt,_OutIt)' being compiled
with
[
_OutIt=std::ostream_iterator<std::string,char,std::char_traits>,
_Iter=std::ostream_iterator<std::string,char,std::char_traits>,
_InIt=std::filesystem::path *
]
D:\DCTL\src\include\dctl\egdb\database.hpp(53): note: see reference to function template instantiation '_OutIt std::copy<std::_Vector_iterator<std::_Vector_val<std::_Simple_types<_Ty>>>,std::ostream_iterator<std::string,char,std::char_traits>>(_InIt,_InIt,_OutIt)' being compiled
with
[
_OutIt=std::ostream_iterator<std::string,char,std::char_traits>,
_Ty=std::filesystem::path,
_InIt=std::_Vector_iterator<std::_Vector_val<std::_Simple_typesstd::filesystem::path>>
]
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\filesystem(2066): note: see reference to class template instantiation 'std::chrono::time_pointstd::filesystem::_File_time_clock,std::filesystem::_File_time_clock::duration' being compiled
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\xstring(1689): note: see reference to class template instantiation 'std::basic_string_view<wchar_t,std::char_traits<wchar_t>>' being compiled

Thank you for taking an interest in my library. I have been told by Microsoft that Visual Studio 15.9 Preview 3 should correctly compile and run all unit tests.
Unfortunately, I haven't access to Visual Studio myself, and I rely on AppVeyor testing for my C++ testing.
As soon as 15.9 Preview 3 is available (see https://github.com/appveyor/ci/issues/2724) I will re-run the tests and see whether your build errors are still present.

I have been told by Microsoft that Visual Studio 15.9 Preview 3 should correctly compile and run all unit tests.

The compiler error here isn't an MSVC issue - it's a platform-specific assumption in this line of code:
https://github.com/rhalbersma/dctl/blob/05c10653fb0c0ae0c89104f2441d8b49727aa713/include/dctl/egdb/database.hpp#L53
This std::copy() call is attempting to implicitly convert std::filesystem::path to std::string, which assumes that the path's string type is narrow. That's true on other platforms, but not on Windows, where the path's string type is std::wstring.
To make this code portable, you can call path::string() in a loop, although that will convert to the current locale. If you need to preserve Unicode, you'll need more complicated code.

@StephanTLavavej To sidestep this entirely, I think the easiest fix is to just use std::ostream_iterator<std::filesystem::path::string_type>(std::cout, "\n")

Thank you very much!
@rhalbersma, I have verified on the latest version 95abd6e of DCTL, this issue seems still alive, could you look at this?
error.txt
database.cpp
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\iterator(209,1): error C2679: binary '<<': no operator found which takes a right-hand operand of type 'const _Ty' (or there is no acceptable conversion)
with
[
_Ty=std::filesystem::path::string_type
]
*_Myostr << _Val;
^
C:\Program Files (x86)\Microsoft Visual Studio\2017\Enterprise\VC\Tools\MSVC\14.14.26428\include\ostream(438,39): note: could be 'std::basic_ostream<char,std::char_traits> &std::basic_ostream<char,std::char_traits>::operator <<(std::basic_streambuf<char,std::char_traits> )'
basic_ostream& __CLR_OR_THIS_CALL operator<<(_Mysb _Strbuf) { // insert until end-of-file from a stream buffer

@QuellaZhang I don't quite understand the error message here. I'd have to isolate this example for filesystem, it doesn't seem related to any of my own library code. Keeping this issue open for the time being.

The error is saying that std::ostream_iterator<std::filesystem::path::string_type>(std::cout, "\n") is attempting to stream a wide string (on Windows, path::string_type is wstring) to narrow cout.
Here is a possible fix:
C:\Temp>type meow.cpp
#include <algorithm>
#include <array>
#include <filesystem>
#include <iostream>
#include <iterator>

int main() {
    const std::array<std::filesystem::path, 3> arr = {{
        R"(C:\cute.txt)",
        R"(D:\fluffy.jpg)",
        R"(E:\kittens.png)"
    }};
    std::ostream_iterator<std::filesystem::path> iter(std::cout, "\n");
    std::copy(arr.begin(), arr.end(), iter);
}

C:\Temp>cl /EHsc /nologo /W4 /MTd /std:c++17 meow.cpp
meow.cpp

C:\Temp>meow
"C:\\cute.txt"
"D:\\fluffy.jpg"
"E:\\kittens.png"

@StephanTLavavej Thanks, using the filesystem::path directly is of course much easier. I think when this code was first created, I still used Boost.FileSystem and later <experimental/filesystem>. At some point, gcc or clang didn't accept printing out the filesystem::path as template argument for std::ostream_iterator but they did implicitly convert to string. Can't really know for sure, since I never checked in those failing attempts to print out file names. Hopefully fixed now!

Thank you @rhalbersma . This issue has gone now.


Add python3 requirements.txt

Coverage remained the same at 26.647% when pulling caaa27aa64df895f83fe124a790367322e7eb437 on py into 7e9f6ef088150e57fc136a8212606be51680c33c on master.


Typo fix in README.md

This PR fixes a typo in README.md (missing commas in json snippet).

Thanks very much :+1:


Support multi character "brackets"

For example, some languages use "if" and "end if" to then end the branch.
Add support for these types of languages, so they could be added in the future. Currently, these "brackets" must be single character.

Support will be added in 0.11, to allow SQL language BEGIN / END block


Tic-tac-toe command

Relevant Issues

Closes #300
Description

I added tictactoe command like described in issue, + added tictactoe history|log to show a list of current session games and tictactoe history|log show <id> to show the board of the game.
Reasoning

Because there was approved issue.
Screenshots

Did you:

[x] Join the Python Discord Community?
[x] If dependencies have been added or updated, run pipenv lock?
[x] Lint your code (pipenv run lint)?
[x] Set the PR to allow edits from contributors?

Hey @ks129, are you still on this?

@Akarys42 Yes, just after finished these Dewikification PRs I didn't have time to apply these changes because of school.

@kosayoda Review parts is now applied, but I didn't added this newline for compact mode, because this will make emojis sooo small in normal mode what makes this hardly readable.

@ks129 Hello, what's the status of this PR?
If you happen to need an extra set of eyes/hands for reviewing/writing code, I'm down.

@ks129 Hello, what's the status of this PR?
If you happen to need an extra set of eyes/hands for reviewing/writing code, I'm down.

@ks129 Hello, what's the status of this PR?
If you happen to need an extra set of eyes/hands for reviewing/writing code, I'm down.

@ks129 Hello, what's the status of this PR?
If you happen to need an extra set of eyes/hands for reviewing/writing code, I'm down.

@Xithrius This PR is waiting review.

@Xithrius This PR is waiting review.

@Xithrius This PR is waiting review.

I will review once I wake. Thank you for the response, have a good one!

@Xithrius This PR is waiting review.

I will review once I wake. Thank you for the response, have a good one!

Sorry, completely forgot about this PR. I will test things after reviewing the code itself.

Sorry, completely forgot about this PR. I will test things after reviewing the code itself.

@Xithrius Board is now moved to embed.

@Xithrius Board is now moved to embed.

@Xithrius Board is now moved to embed.

Alright. I'll test it myself as soon as you mark this as ready for a review.

@Xithrius Board is now moved to embed.

Alright. I'll test it myself as soon as you mark this as ready for a review.

@Xithrius What do you mean? This isn't Draft PR.

@Xithrius What do you mean? This isn't Draft PR.

Sorry, miscommunication I my end, I meant whenever you wanted me to review (marking it ready for me).

Sorry, miscommunication I my end, I meant whenever you wanted me to review (marking it ready for me).


fix a bunch of issues I saw with the ui.

Did not really go through the issue list, but I saw a lot of issues with the UI that needed addressing.  Also fixed mobile menu and logo placement issues I saw.

Thats enough hound issues -_-... they are way too petty.

I don't see any issues on the list that address what I did, everyone was just ok with all the issues I found I guess.  It was a lot of materialize not being used ( or misused like rows or col) and it made content go off the page.  I pretty much went through all the pages and fixed it so on mobile the pages didn't bleed off the page and have a horizontally moveable scrollbar.   Added "MENU" to the side menu hamburger.  Changed a few colors but nothing significant.
If you actually look at the files the changes are pretty minimal and its mostly cleaning up code.  Found a lot of weird html and I fixed some I saw.

Thanks again for your hard work!


✨ feat: add config command to CLI

Checklist:

[!IMPORTANT]
Please review the checklist below before submitting your pull request.

[X] Please open an issue before creating a PR or link to an existing issue
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas

Description
This PR offers quality of life improvements for developers who are looking to debug their manuscript without extensive navigation of the file system. Simply, it sets the standard for manuscript configuration directly from the CLI. This can be tremendously helpful for gathering information about errors.
Current improvements include the following:

clean up/refactor commander.go to use helper functions
new config command

Shows config file location when run without subcommands
Path resolves to $HOME/.manuscript_config.ini

new config show subcommand!

Displays full configuration in JSON format
Added --summary/-s flag for condensed view showing:

Config location and base directory
Manuscript counts (disk vs config)
Horizontal listing of manuscripts with port assignments
Discrepancy detection between disk and config entries

new config clean subcommand!

Implements selective manuscript removal from config
Added flags:

--all: Remove all manuscripts
--force: Skip confirmation prompts

Preserves system settings (BaseDir, SystemInfo)
Uses new SaveConfigFresh function for atomic writes

Added SaveConfigFresh to manuscript_config.go to handle clean config writes without merging

Future Considerations
In the future the following should be considered:

Allow direct manipulation of other configuration settings from the CLI , i.e. (config --set-base-dir)
Allow hard delete / purging of manuscript files on disk with an additional --destructive flag (config clean -d)
Rework list function to look at manuscripts in the specified base_dir in config file and secondarily manuscripts in a chosen directory (taken as an argument) as it worked in #96 . Looking at manuscripts in config file using list is now redundant.

Related Issue
Partially Addresses 2 out of 4 Issues Raised in #101

[x] a config command that allows a user to DISPLAY and CHANGE aspects of their config file. Also, tells a user its path on disk.
[x] a clean command or flag that allows a user to WHOLLY wipe out their config file, or WIPE all manuscripts
[ ] a start command where a user can deploy any manuscript from a config without listing the entire path or parameters for the manuscript in question (this would speed up deploys & make better use of the manuscript_config.ini)  ref: #101
[ ] a global --config flag that allows most commands to run from the specified config file instead of one in the default location

Base Command Usage
$ manuscript-cli config
📁 Manuscript config location: /home/user/.manuscript_config.ini

Sub Command 1 Usage
$ manuscript-cli config show
📋 Manuscript Configuration:
{
  "BaseDir": "/home/user/manuscripts",
  "SystemInfo": "linux",
  "Manuscripts": [
    {
      "Name": "demo",
      "Port": 8081,
      ...
    }
  ]
}

$ manuscript-cli config show --summary

📊 Manuscript Configuration Summary
──────────────────────────────────
📁 Config Location: /home/kagemnikarimu/.manuscript_config.ini
📂 Manuscripts Directory: /home/kagemnikarimu/manuscripts

📈 Statistics:
   • Manuscripts on disk: 0
   • Manuscripts in config: 0

🔍 Configured Manuscripts:

Sub Command 2 Usage
# Force remove specific manuscripts (no confirmation)
$ manuscript-cli config clean manuscript1 manuscript2 --force
Removed manuscript 'manuscript1'
Removed manuscript 'manuscript2'
🧹 Successfully removed 2 manuscript(s)

# Force remove all manuscripts (no confirmation)
$ manuscript-cli config clean --all --force
🧹 All manuscript configurations cleaned successfully!

# Clean all interactively (without force)
$ manuscript-cli config clean --all
⚠  Warning: This will remove all manuscript configurations. Continue? (y/N): y
🧹 All manuscript configurations cleaned successfully!

# Clean specific manuscripts interactively (without force)
$ manuscript-cli config clean manuscript1 manuscript2
Remove manuscript 'manuscript1'? (y/N): y
Removed manuscript 'manuscript1'
Remove manuscript 'manuscript2'? (y/N): n
Keeping manuscript 'manuscript2'
🧹 Successfully removed 1 manuscript(s)

Screenshots
config show --summary before running config clean --all :

config show -s after running config clean --all:

Type of Change

[X] New feature
[X] Enhancement, including but not limited to code refactoring, performance optimization, Test and CI

very cool~


getting a directory, not a web page

installed oui  and i get a directory listing, not a web page
![image](https://user-images.githubusercontent.com/52370034/130043621-067c2b56-eba6-4670-9295-9e16bfada550.png)

Please confirm whether there are other HTTP servers running in your device, occupying port 80.

fixed, removed legacy uhttpd


Library updating issue

Steps to reproduce
When I update the library for new anime, it just loads some of them and leaves all other thus new animes never get updated and i need to update all of them manually by going to each anime's episode view section.
Expected behavior
It should update all the new animes eposodes and also send a notification about the updated episode.
Actual behavior
It loads 5 animes and never updates the rest even tho new episode is out.
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
Android 12
Device
Samsung galaxy tab S6 lite
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Same problem

https://github.com/aniyomiorg/aniyomi/issues/1346#issuecomment-1907149996
This solution fixed this same problem reported in another issue for me

It's either an extension issue, or it's related to smart updates


Mixup of two projects

Details

Project URL: https://readthedocs.org/projects/solidity
Build URL (if applicable): https://solidity.readthedocs.io/en/develop/
Read the Docs username (if applicable):  chriseth

Expected Result
All links in the "sidebar" of https://solidity.readthedocs.io/en/develop/ and the search results should link to sub-pages of https://solidity.readthedocs.io/en/develop/
Actual Result
Some links go to http://solidity-482solution.readthedocs.io, for example the version links in the sidebar go to http://solidity-482solution.readthedocs.io/en/v0.4.15/ and search results as seen on https://solidity.readthedocs.io/en/develop/search.html?q=events&check_keywords=yes&area=default go to http://solidity-482solution.readthedocs.io/en/develop/structure-of-a-contract.html?highlight=events
This could be security critical as users might not notice the change of the domain.

@axic just realized that the other project - solidity-482solution has a russian language setting while solidity.readthedocs.io only has "en" and "zh" and this russian translation is somehow mixed into solidity.readthedocs.io

Closing this as the user hasn't responded or given more information, feel free to reopen with more information if you are still having this problem :).

@chriseth thanks for the update :)


unstar(url) is buggy

Request: http://phonetworks.com:1338/unstar?url=http://localhost:8080/docs.html???&public_id=16D58CF2-FD88-4A49-972B-6F60054BF023
Response: PhoNetworksAutogenerated\UserOut\Star is not multiplicable and there's already an edge from 4b688921975b3f46544c487d268e24d6 to 54a26ffcaf7a36c3aa0161ccd4317b78

Same as #15 closing.


Fix advanced filter controls

These advanced filter controls appeared for the first time today:

Does anyone find them useful?
Maybe they could be hidden by default. Also, there is a grey rectangle above them that should be hidden.

I find them all useful except "To me" because pretty much all my mail is to me. But the rest could definitely be useful.


Sharing Mod lists does not share order correctly

When sharing mod lists, it tells the most position via having ';x' at the end, but if players have different mods this can cause mods to be in the wrong order, causing incompatibility that can be hard to fix

Situation we ran into, mod list of:
2879757612|2790124693|2806164655|2859968660|2860107529|2883357983|2854346056;20|2794696516|2831188313|2789904335|2789858755|2789872749|2798367988|2792940826|2789850100|2789845079|2789854789|2852724610|2794896622|2889271044|2789862740|2855837519|2790584452|2789903475|2790563688|2789948438|2890499031|2789904423|2860849259|2859369022|2792939724|2858251286|2875227597|2881678598|2856202634|2789857945|2858832821|2789893818|2790147047|2859646091|2790754811|2863153005|2853058175|2790986410|2854819509|2862719717|2855364225|2860940476|2854403011|2884798300
Our my_mods files:
my_mods.txt
my_mods.txt
In this case, '!derpy_noinruneguardian.pack' and 'immortal_landmarks_IE_v1.pack' swap order as one of us had one more mod


Date and Number Fields Should Be Text Fields

The failure to treat them uniformly:

Created a lot of unneeded duplication as much of the code was simply delegating.
Made working with the specialized variants difficult. For example, DNU #identifier when used in an expression like: response onReturn setValueTo: aDate mmddyyyy thenTriggerChangeOf: self myDateField

Coverage remained the same at 100.0% when pulling c5e4885942a6c082d8343546139ad34c0f678ca5 on seandenigris:clean-date-and-number-fields into edd4b857b074908c1e30164cde4b3bd2ac19635d on ba-st:master.

@seandenigris take a look at #129 , I've changed a bit the code because in this branch you can for example instantiate a DateField with the multiline methods and that make no sense... So let me know if this covers your use case or we need to changes something elese.


chore(RHINENG-9387): update page titles

Description
Associated Jira ticket: # https://issues.redhat.com/browse/RHINENG-9387
Updates page titles according to 'Current concept' column, single line treatment in this sheet
How to test the PR
Please include steps to test your PR.
Before the change
After the change
Dependent work link
Checklist:

[ ] The commit message has the Jira ticket linked
[ ] PR has a short description
[ ] Screenshots before and after the change are added
[ ] Tests for the changes have been added
[ ] README.md is updated if necessary
[ ] Needs additional dependent work

Codecov Report
Attention: Patch coverage is 50.00000% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 51.55%. Comparing base (a6131bf) to head (7bfa6c1).

Files
Patch %
Lines

src/Routes/Dashboard/index.js
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #276   +/-   ##
=======================================
  Coverage   51.55%   51.55%
=======================================
  Files          86       86
  Lines        1286     1286
  Branches      271      271
=======================================
  Hits          663      663
  Misses        623      623

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


Cross-Linking CellGuide and GE

We aim to cross-link with established database curators such as the Human Protein Atlas, the CL Group, HubMAP, and work towards other more niche areas such as cbio portal. We expect this will drive traffic and improve our Google ranking, which will also compound and drive more organic traffic.
~1.9M users visit ebi.ac.uk monthly, ~100,00 users visit Human Protein Atlas monthly, ~1,000 users visit HuBMAP monthly.
This work will be carried out by external teams who we have already established partnerships such as CL and HuBMAP.
Product Stories:
As a user, I want to link back to CellGuide when visiting CL to search for cell types or gene info.
As a user, I want to link back to CellGuide when visiting HubMAP to search for cell types or gene info.
UX Design
Figma

@carolinechoi can you move this ticket to the proper swim lane please? thank you!


[TypeScript]: Need to (F12/Goto definition) twice on destructured imports to get to actual definition

From @jumpinjackie on June 17, 2016 9:45

VSCode Version: 1.2.1
OS Version: Windows 8.1 x64

Steps to Reproduce:

Import a module member using destructuring syntax
F12 / Goto definition on the destructured import symbol

The first F12 takes you to the destructuring expression. The second F12 then takes you to the actual definition
See attached gif for an example

Possibly also related is that tooltips on destructured import symbols show the import statement instead of its definition

Compare with non-destructured version

Copied from original issue: Microsoft/vscode#7806

Moving to TS team.

Duplicate of https://github.com/Microsoft/TypeScript/issues/3669, should be fixed by https://github.com/Microsoft/TypeScript/pull/6952.


risk_free_rate in max_sharpe() can't be setup

Describe the bug
The risk_free_rate in max_sharpe() always equal to its default value as 0.02 no matter the parameter value I input, which causes sharpe ratio calculated wrong.
Expected behavior
If risk_free_rate=0, Expected annual return=14.6%, Annual volatility = 10.6% , Sharpe Ratio shall be 14.6% / 10.6% =1.377. But ef.max_sharpe() returns sharpe ratio as 1.19 which is calculated as (14.6%-2%)/10.6%.
If risk_free_rate=0.05, Expected annual return=19.4%, Annual volatility = 14.8% , Sharpe Ratio shall be (19.4%-5%) / 14.8% =0.973. But ef.max_sharpe() returns sharpe ratio as 1.18 which is calculated as (19.4%-2%)/14.8%.
Screenshots

Operating system and python version:
Win 10, python 3.7.3

Hi @Meng-QQ,
Thanks for raising this! The problem is not in max_sharpe, but in ef.portfolio_performance() (this line).
max_sharpe is indeed using the correct risk_free_rate, but ef.portfolio_performance redefines a local risk_free_rate variable. So to get the right behaviour, you would need to do:
ef.max_sharpe(risk_free_rate=0.05)
ef.portfolio_performance(verbose=True, risk_free_rate=0.05)

This is terrible design on my part. I will push a fix that makes risk_free_rate a field of the EfficientFrontier object (or something like that), so that it has more sensible default behaviour.
Cheers,
Robert

Thanks Robert. It works as you explained. Cheers!

Fixed in v1.3.0

Fixed in v1.3.0


Add 'tolerate' feature to autocom

If a user has something like
    `cpu.percent`^

where the ^ is the cursor, and press enter they get
    cpu.percent^

instead of what they (probably) expect,
    `cpu.percent`
    ^

This PR fixes this issue.
In addition, it offers a slight performance improvement when there's a large number of options, by calling the autocompletion function about 3x less.
@drcapulet
(Tested locally)

:+1:


Possibility to split

run demo/group/index.html for demo.

When splitting hunks such that one hunk only has one line, the resulting line will have incorrect margins:

In the basic demo, the + icons are still visible and clickable, but they do not do anything. I think we need a property to enable/disable the buttons

I implemented the changes you proposed.


Partition stdout redirection message should always comes from partition 0

Original issue: https://charm.cs.illinois.edu/redmine/issues/462

Charm++> Running in SMP mode: numNodes 1024,  14 worker threads per
process
Charm++> The comm. thread both sends and receives messages
Charm++> Using rank ordered division (scheme 0) for topology aware
partitions
Redirecting stdout to files
/lustre/atlas1/bio024/scratch/jimp/output/0/apr10_stmv2fs_128rep_1024_nop_0.log
through 127
Note that it's printing the partition 0 case, vs with topology it's 116:
Charm++> Using recursive bisection (scheme 3) for topology aware
partitions
Charm++> TORUS A SIZE 25 USING 9 10 11 12
Charm++> TORUS B SIZE 16 USING 2 3 4 5 6 7 8 9
Charm++> TORUS C SIZE 24 USING 0 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
18 19 20 21 22 23
Charm++> TORUS MINIMAL MESH SIZE IS 4 BY 8 BY 24
Redirecting stdout to files
/lustre/atlas1/bio024/scratch/jimp/output/116/apr10_stmv2fs_128rep_1024_def_116.log
through 127
_pmiu_daemon(SIGCHLD): [NID 10498] [c23-4c2s1n2] [Fri Apr 11 11:41:37
2014] PE RANK 289 exit signal Segmentation fault
[NID 10498] 2014-04-11 11:41:37 Apid 4847657: initiated application
termination
Also, the topology version crashes at the first replicaBarrier.

Original date: 2014-04-11 16:13:22

OK, so the +stdout filename strangeness is just due to this:
  if ( CmiMyNodeGlobal() == 0 ) {
    printf("Redirecting stdout to files %s through %d\n",stdoutpath,CmiNumPartitions()-1);
  }

Global node 0 isn't always on partition 0.  This is annoying but not fatal.
The crash looks like a crash on one of the nodes.

Original date: 2014-04-14 18:17:40

Nikhil, could you triage this?

Original date: 2014-04-14 18:40:16

Jim further reported that it works for apoA1, but not for stmv. So we are not sure of which part is broken. I plan to look at it post SC deadline.

Original date: 2014-04-24 22:07:08

Now that we're post-SC, could you re-examine this?

Original date: 2014-05-05 18:52:53

I am not able to reproduce this issue: just ran 100M stmv on BW with 4 partitions and topology scheme 3 on 16 nodes with and without SMP. Should I try some other configuration?

Original date: 2014-05-26 05:17:43

I am inclined to mark this as resolved (rejected?). Jim, please confirm.

Original date: 2014-11-04 21:07:20

Jim, Nikhil, is this still an issue?

Original date: 2014-11-05 20:55:09

I think the issue that global node 0 isn't always on partition 0 is still there, but it only affects this one message.

Original date: 2014-11-11 19:20:28

Retitled; I assume the crash in question was long ago resolved as independent of this?


The schema generated by amplify api gql-compile is not correct

Edit amplify/backend/api/xxxx/schema.graphql

type Advertisement @model(mutations: null, subscriptions: null){
  productID: ID!
  productDescription: String
  redPackets: [SharedRedPacket] @connection(name: "AdsRPs")
}

type User @model(mutations: null, subscriptions: null) {
  userEmail: ID!
  balance: Float
  hasSharedRP: Boolean
  city: String
}

type SharedRedPacket @model(mutations: null, subscriptions: null) {
  userEmail: ID!
  advertisement: Advertisement @connection(name: "AdsRPs")
  sharedDoneFlag: Boolean
  sharedDetails: [SharedDetail]
}

type SharedDetail {
  userEmail: String!
  value: Float!
}

Run amplify api gql-compile, and the schema is generated
The following is the generated amplify/backend/api/xxx/build/schema.graphql

type Advertisement {
  productID: ID!
  productDescription: String
  redPackets(filter: ModelSharedRedPacketFilterInput, sortDirection: ModelSortDirection, limit: Int, nextToken: String): ModelSharedRedPacketConnection
}

type User {
  userEmail: ID!
  balance: Float
  hasSharedRP: Boolean
  city: String
}

type SharedRedPacket {
  userEmail: ID!
  advertisement: Advertisement
  sharedDoneFlag: Boolean
  sharedDetails: [SharedDetail]
}

type SharedDetail {
  userEmail: String!
  value: Float!
}

enum ModelSortDirection {
  ASC
  DESC
}

type ModelAdvertisementConnection {
  items: [Advertisement]
  nextToken: String
}

input ModelStringInput {
  ne: String
  eq: String
  le: String
  lt: String
  ge: String
  gt: String
  contains: String
  notContains: String
  between: [String]
  beginsWith: String
  attributeExists: Boolean
  attributeType: ModelAttributeTypes
  size: ModelSizeInput
}

input ModelIDInput {
  ne: ID
  eq: ID
  le: ID
  lt: ID
  ge: ID
  gt: ID
  contains: ID
  notContains: ID
  between: [ID]
  beginsWith: ID
  attributeExists: Boolean
  attributeType: ModelAttributeTypes
  size: ModelSizeInput
}

input ModelIntInput {
  ne: Int
  eq: Int
  le: Int
  lt: Int
  ge: Int
  gt: Int
  between: [Int]
  attributeExists: Boolean
  attributeType: ModelAttributeTypes
}

input ModelFloatInput {
  ne: Float
  eq: Float
  le: Float
  lt: Float
  ge: Float
  gt: Float
  between: [Float]
  attributeExists: Boolean
  attributeType: ModelAttributeTypes
}

input ModelBooleanInput {
  ne: Boolean
  eq: Boolean
  attributeExists: Boolean
  attributeType: ModelAttributeTypes
}

input ModelSizeInput {
  ne: Int
  eq: Int
  le: Int
  lt: Int
  ge: Int
  gt: Int
  between: [Int]
}

input ModelAdvertisementFilterInput {
  productID: ModelIDInput
  productDescription: ModelStringInput
  and: [ModelAdvertisementFilterInput]
  or: [ModelAdvertisementFilterInput]
  not: ModelAdvertisementFilterInput
}

type Query {
  getAdvertisement(id: ID!): Advertisement
  listAdvertisements(filter: ModelAdvertisementFilterInput, limit: Int, nextToken: String): ModelAdvertisementConnection
  getUser(id: ID!): User
  listUsers(filter: ModelUserFilterInput, limit: Int, nextToken: String): ModelUserConnection
  getSharedRedPacket(id: ID!): SharedRedPacket
  listSharedRedPackets(filter: ModelSharedRedPacketFilterInput, limit: Int, nextToken: String): ModelSharedRedPacketConnection
}

type ModelUserConnection {
  items: [User]
  nextToken: String
}

input ModelUserFilterInput {
  userEmail: ModelIDInput
  balance: ModelFloatInput
  hasSharedRP: ModelBooleanInput
  city: ModelStringInput
  and: [ModelUserFilterInput]
  or: [ModelUserFilterInput]
  not: ModelUserFilterInput
}

input SharedDetailInput {
  userEmail: String!
  value: Float!
}

type ModelSharedRedPacketConnection {
  items: [SharedRedPacket]
  nextToken: String
}

input ModelSharedRedPacketFilterInput {
  userEmail: ModelIDInput
  sharedDoneFlag: ModelBooleanInput
  and: [ModelSharedRedPacketFilterInput]
  or: [ModelSharedRedPacketFilterInput]
  not: ModelSharedRedPacketFilterInput
}

Run amplify push
Got error message

The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelStringInput' [@73:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIDInput' [@89:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIntInput' [@105:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelFloatInput' [@117:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelBooleanInput' [@129:1].

The generated schema file is obviously wrong, it does not have ModelAttributeTypes defined.
Amplify CLI Version
4.10.0
Desktop (please complete the following information):

OS: MAC
Node Version. v12.13.1

@JoeShi taking a look

I added brand new type/model to existing schema and amplify push didn't update graphql mutation and queries.
So I tried amplify api gql-compile and same again. Nothing was changed.
amplify codegen models did update model based on new schema.
amplify cli v4.17.2

I have also encountered same problem.
Resource Name: zaic7uucg5fcjnuysj35hs4ba4GraphQLSchema (AWS::AppSync::GraphQLSchema)
Event Type: update
Reason: Schema Creation Status is FAILED with details: Found 6 problem(s) with the schema:
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelStringInput' [@151:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIDInput' [@167:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIntInput' [@183:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelFloatInput' [@195:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelBooleanInput' [@207:1].
The input value type 'ModelPRODUCT_TYPEInput' is not present when resolving type 'ModelCartItemFilterInput' [@224:1].

@isheejong Are you using the latest version of the CLI (4.18.1)? The fix mentioned above is available in that release.

Here is my version
isheejong@gmail.com:~/environment $ amplify -v
4.18.1
type Cart
@model(queries: null, mutations: null, subscriptions: null)
@aws_api_key
@aws_cognito_user_pools
@key( fields : ["sessionId"], name: "sessionId")
{
id : ID!
memberId:String
sessionId: ID
cartType: CART_TYPE!
cartName: String!
orderSummary: AWSJSON
orderSummaryInfo: AWSJSON
createTime: AWSDateTime
updateTime: AWSDateTime
cartItems: [CartItem] @connection(name:"CartCartItem")
}
type CartItem
@model(queries: null, mutations: null, subscriptions: null)
@aws_api_key
@aws_cognito_user_pools
{
id: ID!
productNo: String!
itemNo: String!
productType: PRODUCT_TYPE!
packageNo: String!
sellerId: String!
productName: String!
itemName: String!
itemPrice: Float!
itemQty: Float!
cartItemInfo: AWSJSON
cart: Cart @connection(name:"CartCartItem")
}
enum CART_TYPE {
MEMBER
SESSION
}
enum PRODUCT_TYPE {
UNIT
PACKAGE
}
type Query {
getMyCart:Cart
getCartBySessionId(sessionId:String!):Cart
}
input CreateCartInput{
cartType: CART_TYPE!
cartName: String!
orderSummary: AWSJSON
orderSummaryInfo: AWSJSON
}
input UpdateMyCartInput{
memberId: String!
orderSummary: AWSJSON
orderSummaryInfo: AWSJSON
}
input UpdateCartInputBySessionId{
sessionId: String!
orderSummary: AWSJSON
orderSummaryInfo: AWSJSON
}
type Mutation{
createCart(input: CreateCartInput!):Cart
@aws_cognito_user_pools
@aws_api_key
updateMyCart(input: UpdateMyCartInput!):Cart
@aws_cognito_user_pools
updateCartBySessionId(input: UpdateCartInputBySessionId!):Cart
@aws_api_key
}
Resource Name: zaic7uucg5fcjnuysj35hs4ba4GraphQLSchema (AWS::AppSync::GraphQLSchema)
Event Type: update
Reason: Schema Creation Status is FAILED with details: Found 6 problem(s) with the schema:
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelStringInput' [@151:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIDInput' [@167:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIntInput' [@183:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelFloatInput' [@195:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelBooleanInput' [@207:1].
The input value type 'ModelPRODUCT_TYPEInput' is not present when resolving type 'ModelCartItemFilterInput' [@224:1].
URL: https://console.aws.amazon.com/cloudformation/home?region=ap-northeast-2#/stacks/arn%3Aaws%3Acloudformation%3Aap-northeast-2%3A786923391258%3Astack%2Famplify-kestore-heejong-25054-apikestore-Y8NY1TR62L19%2F48c738a0-85d7-11ea-ab05-06ad40dfc020/events

@isheejong Are you using the latest version of the CLI (4.18.1)? The fix mentioned above is available in that release.

Here is my version of amplify CLI used
isheejong@gmail.com:~/environment $ amplify -v
4.18.1

and this is schema.graphql for generations.
type Cart
@model(queries: null, mutations: null, subscriptions: null)
@aws_api_key
@aws_cognito_user_pools
@key( fields : ["sessionId"], name: "sessionId")
{
  id : ID!
  memberId:String
  sessionId: ID
  cartType: CART_TYPE!
  cartName: String!
  orderSummary: AWSJSON
  orderSummaryInfo: AWSJSON
  createTime: AWSDateTime
  updateTime: AWSDateTime
  cartItems: [CartItem] @connection(name:"CartCartItem")
}

type CartItem
@model(queries: null, mutations: null, subscriptions: null)
@aws_api_key
@aws_cognito_user_pools
{
  id: ID!
  productNo: String!
  itemNo: String!
  productType: PRODUCT_TYPE!
  packageNo: String!
  sellerId: String!
  productName: String!
  itemName: String!
  itemPrice: Float!
  itemQty: Float!
  cartItemInfo: AWSJSON
  cart: Cart @connection(name:"CartCartItem")
}

enum CART_TYPE {
  MEMBER
  SESSION
}

enum PRODUCT_TYPE {
  UNIT
  PACKAGE
}

type Query {
  getMyCart:Cart
  getCartBySessionId(sessionId:String!):Cart
}

input CreateCartInput{
  cartType: CART_TYPE!
  cartName: String!
  orderSummary: AWSJSON
  orderSummaryInfo: AWSJSON
}

input UpdateMyCartInput{
  memberId: String!
  orderSummary: AWSJSON
  orderSummaryInfo: AWSJSON
}

input UpdateCartInputBySessionId{
  sessionId: String!
  orderSummary: AWSJSON
  orderSummaryInfo: AWSJSON
}

type Mutation{

  createCart(input: CreateCartInput!):Cart
  @aws_cognito_user_pools
  @aws_api_key

  updateMyCart(input: UpdateMyCartInput!):Cart
  @aws_cognito_user_pools

  updateCartBySessionId(input: UpdateCartInputBySessionId!):Cart
  @aws_api_key

}
-----------------------------------------------------------------
Resource Name: zaic7uucg5fcjnuysj35hs4ba4GraphQLSchema (AWS::AppSync::GraphQLSchema)
Event Type: update
Reason: Schema Creation Status is FAILED with details: Found 6 problem(s) with the schema:
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelStringInput' [@151:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIDInput' [@167:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelIntInput' [@183:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelFloatInput' [@195:1].
The input value type 'ModelAttributeTypes' is not present when resolving type 'ModelBooleanInput' [@207:1].
The input value type 'ModelPRODUCT_TYPEInput' is not present when resolving type 'ModelCartItemFilterInput' [@224:1].
URL: https://console.aws.amazon.com/cloudformation/home?region=ap-northeast-2#/stacks/arn%3Aaws%3Acloudformation%3Aap-northeast-2%3A786923391258%3Astack%2Famplify-kestore-heejong-25054-apikestore-Y8NY1TR62L19%2F48c738a0-85d7-11ea-ab05-06ad40dfc020/events
-------------------------------------------------------

But amplify codegen models is working well
isheejong@gmail.com:~/environment/korea-ecommerce-store (master) $ amplify codegen models
The following types do not have '@auth' enabled. Consider using @auth with @model
         - Cart
         - CartItem
Learn more about @auth here: https://aws-amplify.github.io/docs/cli-toolchain/graphql#auth
GraphQL schema compiled successfully.

I'm out of idea how to resolve that problem. :)

Hi there
Thank you for interesting my problem. I 'm working on developing e-commerce
site with customer.
I have replied from your comment, could you check my comment?
Thank you
Heeejong
2020년 4월 25일 (토) 오전 1:09, Edward Foyle<EMAIL_ADDRESS>작성:

@isheejong https://github.com/isheejong Are you using the latest
version of the CLI (4.18.1)? The fix mentioned above is available in that
release.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/aws-amplify/amplify-cli/issues/3141#issuecomment-619104169,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZ77HFGCZGLQD7OZT3MH6TROG2SPANCNFSM4KD2GWOQ
.

--
<EMAIL_ADDRESS>Best Regards
HeeJong

following up in #4081

I added brand new type/model to existing schema and amplify push didn't update graphql mutation and queries.
So I tried amplify api gql-compile and same again. Nothing was changed.
amplify codegen models did update model based on new schema.
amplify cli v4.17.2
Solved by using amplify codgen

I can confirm the exact same scenario for amplify version 4.50.2


ROM mapping location is incorrect

https://github.com/pudii/gba-ghidra-loader/blob/main/src/main/java/gba/GBALoader.java#L140
Currently the ROM gets loaded into 0x80000000, but it should be loaded into 0x08000000.

Hi, thanks for the issue. I'll have a look into that as simply changing the address breaks something.


Voltage space calibration bug fix

I think these changes are included in #94. If that's the case then you can close this @ieivanov. Thanks!


Abandon additional transport method parameter

Most of the time, users won't care what is the underlaying communication protocol.
So, I propose:
func NewPlugin(path string, params ...string) *Plugin

To change the default communication protocol, expose:
func SetProto(proto string)

The API becomes clearer. E.g.:
p := pingo.NewPlugin("plugins/hello-world/hello-world")

Agreed. This would also help implementing automatic switching of protocol.

Would be nice to use RPC over stdin/stdout too.

@kdar: I'll make a new issue for this.


Magnetometer values are off

System info
Platform the Issue occurs on: iOS
Plugin name: sensors_plus
Plugin version: 1.3.0
Steps to Reproduce
It seems that Flutter's implementation of the Magnetometer is off when compared to iOS devices.
I don't know yet if it's due to some additional flags that are set on the iOS side to calibrate the magnetometer ( https://stackoverflow.com/questions/28365121/how-to-get-magnetometer-data-using-swift ), but results for Flutter are way higher than for iOS devices. And actually sometimes when the Flutter app detects metals the values go down instead of going up.
You can easily test it out by downloading those 3 apps below and seeing the results by yourself.
iOS implementation
https://apps.apple.com/us/app/stud-finder-wall-detector/id1530653768
https://apps.apple.com/us/app/teslameter/id976926346
Flutter implementation
https://apps.apple.com/us/app/stud-finder/id1613688190
iOS

Flutter

You can see on the screenshots that iOS values are lower than those on Flutter. All of those screenshots were made in the same phone position, with the phone lying flat.
One note here: Look at the X, Y, Z values here, because the Flutter app has the main displayed wrong.
cc: @Zabadam because I've seen that you were implementing this feature

I have an extremely limited knowledge of iOS and as such the testing for that platform was less than thorough.
If I take another look, it may be as simple as calling values from sensor manager incorrectly.
It seems Android has moved things around over the years, and iOS may be similar; whereby I could have been referencing irrelevant docs.

Possibly, I'll also take a look at what could be wrong here

I have finally sat down to look into this.
It seems like it may be a simple fix. We are after this value, the calibrated magneticField property of a CMDeviceMotion object that "unlike the magneticField property of the CMMagnetometerData class . . . reflect the earth’s magnetic field plus surrounding fields, minus device bias."

https://github.com/Zabadam/plus_plugins/commit/0b72713a476d18b1bd49c99be319038f448095b8

Awesome!
I'll check it out later. @Zabadam
Just FYI, in the meantime, I've used this library: https://pub.dev/packages/motion_sensors and it seems to have a proper implementation.

Just FYI, in the meantime, I've used this library: https://pub.dev/packages/motion_sensors and it seems to have a proper implementation.

I remember when I was building a sensors based package I had to make the decision between that package and sensors_plus.
You can see now the decision that I ultimately made was to add the magnetometer support I needed to this official package instead of using motion_sensors as per its lack of web support.

Barring a check for availability of sensors and providing a means to alter temporal accuracy, this implementation should accomplish the same expected functionality as from motion_sensors:

I wish I had an iOS device to verify this fix. Could be checked by someone else if they have the time:
dependencies
  # Testing calibrated magnetometer on iOS
  sensors_plus:
    git:
      url: https://github.com/Zabadam/plus_plugins.git
      path: packages/sensors_plus/sensors_plus

@Zabadam Thanks for your effort!
I just used this dependency and it seems that it's still showing higher values than the other apps or the motion_plus package.

@Zabadam Thanks for your effort!
I just used this dependency and it seems that it's still showing higher values than the other apps or the motion_plus package.

Because the shared section of code is nearly identical (albeit Swift instead of ObjC) to motion_sensors, it seems the magnetometer implementation here may not be the source of issue after all.

The PR that implements this fix will likely soon be merged. As the code is technically correct, especially when compared to your other linked package that implements iOS magnetometer support, I'm unsure the status on this Issue.
I would love an update on the status of this fix.

Sorry, didn't notice your previous message.
It's hard to test those things since we don't really know what's the correct result. The implementation looks good to me, I say: merge it.

Wow, I learned a lot just by reading this excellent comment on StackOverflow.
Snipped below:

I have put a Magnet-O-Meter demo app on gitHub which displays some of these differences. It's quite revealing to wave a magnet around your device when the app is running and watching how the various APIs react:

my original solution

CMMagnetometer doesn't react much to anything unless you pull a rare earth magnet up close. The onboard magnetic fields seem far more significant than local external fields or the earth's magnetic field. On my iPhone 4S it consistently points to the bottom left of the device; on the iPad mini it points usually to the top right.

my proposed solution

CMCalibratedMagneticField is the most steady in the face of varying external fields, but otherwise tracks it's Core Location counterpart CLHeading.[x|y|z] pretty closely.

...but it seems that

CLHeading.[x|y|z] is the most vulnerable (responsive) to local external fields, whether moving or static relative to the device.

And this is the surprising bit. It would seem, according to these tests, that the Core Location CLHeading provides geomagnetic data that both filters out the device hard iron and also is more responsive to external elements (such as is desired for a stud finder) than is the Core Motion CMDeviceMotion's CMCalibratedMagneticField.
Their final note:

CLHeading.magneticHeading - Apple's recommendation for magnetic compass reading - is far more stable than any of these. It is using data from the other sensors to stabilise the magnetometer data. But you don't get a raw breakdown of x,y,z

Still, how these values compare to Android implementation is mystery to me. It is almost funny to me that by now I have learned far more about how iOS and Apple handle sensors and this sort of data than the platform I actually use. That just means I have more to learn!

The PR with the magnet sensor changes has been merged and release. Let me know if this ticket can be closed.

@Zabadam have you checked further to see if it gives the correct values?
i checked today and magnetometerEventStream still returns incorrect values that are much higher


missing value reports

WIP for adding missing value reports to all transformers that may make suspect computations over missing values.
This extends the DataSetTransformationOutput and SingleValueTransformationOutput types to include a MissingValueReport of the following form:
type MissingValueReport = {
  missingValues: MissingValueLocation[];
  extraInfo?: string;
};

type MissingValueLocation = {
  collection: string;
  attribute: string;
  itemIndex: number;
};

The report is optional, though I think it might be useful implementation-wise to have transformers just always produce the report and have it ignored when it is empty (no missingValues).
I am prototyping this with just Mean at the moment and using confirm() to allow the user to either abort the computation when they notice it has missing values, or proceed with it in which a case the transformer output & report are generated.
I haven't tried adding this to updating because I think there are some questions there to address:

Are MVRs updated when transformers update? Or is a new report generated? Or nothing at all?
What if an MVR is deleted and then the input dataset is updated?

I'm also not entirely convinced of this way of indicating location--collection and attribute is good, but I feel like we need to explain the "item number" aspect and even then it will get confusing for hierarchical datasets.

Here's a screenshot of what it can look like now, using a table to present a summary of which values were missing. I had to include context info because some transformers consume multiple contexts and therefore would make it ambiguous.

Also worth noting that although this looks nice, it is not the layout in which the components were thrown onto the CODAP workspace (which was quite spread out and random looking). This might have an effect on whether we keep this table view or revert to a single textbox with all the info.

Note that at the moment missing value reports do not interact with updating at all. I also want to demo the textbox + table behavior for sk to see what he thinks before merging.

Also worth noting: The missing value table for formula transformers does not contain collection/attribute info, but only indicates which rows of the input for which the formula evaluated to a missing value. This is because there is no one particular attribute/collection involved in the computation, since a formula can access any of the attributes across the table.

This good to merge? @stardust66 @P-bibs


fix: make markdown css not conflict with lion components

Prevent styles from applying to lion components that are rendered in light dom

would you mind updating the mdjs dependency?

would you mind updating the mdjs dependency?

Done :)

thx 👍


Add logic to deal with ABCIndexClass being renamed to ABCIndex

Pandas 1.3.x renamed its abstract base class for indexes from ABCIndexClass to ABCIndex, which is messing up some of our type checks. This PR adds some logic to work around that renaming. I'm using exceptions as control flow, which is a bit ugly, but the alternatives are also ugly.

Okay, that didn't work. Will try something else.

Fixed some bugs in SpanArray and TokenSpanArray that new regression tests in Pandas 1.3.0 brought to light. Now we're down to 62 failing tests.

Update: Most of the test failures seem to be due to a bug in Pandas 1.3.0. Due to some performance optimizations introduced in https://github.com/pandas-dev/pandas/pull/40353, Pandas turns DataFrame.iloc[slice(x, y, z)] into __getitem__((..., slice(x, y, z)) on the ExtensionArray that backs any column defined with an extension type.
Code to reproduce:
from pandas.api.extensions import ExtensionArray,ExtensionDtype

class MyExtensionDtype(ExtensionDtype):
    """Minimal extension dtype"""
    def __init__(self):
        pass

    @property
    def type(self):
        return int

    @property
    def name(self) -> str:
        return "MyExtensionDtype"

    @classmethod
    def construct_array_type(cls):
        return MyExtensionArray()

class MyExtensionArray(ExtensionArray, ExtensionScalarOpsMixin):
    """Minimal extension array that logs calls to __getitem__()"""
    @property
    def dtype(self):
        return MyExtensionDtype()

    def copy(self):
        return MyExtensionArray()

    def __len__(self):
        return 5

    def __getitem__(self, key):
        print(f"__getitem__ called with key '{key}'")
        return 42

arr = MyExtensionArray()
df = pd.DataFrame({"a": arr})
_ = df.iloc[:3]

which prints out:
__getitem__ called with key '(Ellipsis, slice(None, 3, None))'

It should print the following instead:
__getitem__ called with key 'slice(None, 3, None)'

I'll put in a workaround tomorrow and file a bug with Pandas.
FYI @BryanCutler @ZachEichen @PokkeFe @Crushellini

Update:

Opened https://github.com/pandas-dev/pandas/issues/42430 to cover the slicing issue described above.
Added a workaround for the Pandas issue to TensorArray.__getitem__()
We're now down to 53 failing tests.

Update: Fixed another minor bug. Now we are down to 49 failing tests.

Update:

Filed https://github.com/pandas-dev/pandas/issues/42437 and temporarily disabled the tests that that bug was causing to fail, since a workaround exists. See #221 for a workaround for that issue.
Fixed a minor issue in TensorArray.isna() that was causing a number of regressions.

Now we're down to 2 failing tests.

All tests passing against Pandas 1.3.0 now. Merging this PR to unblock other PRs.


Installing on CentOS: Missing Dependencies

Hello,
I am installing GoAccess from source on CentOS 7. As per the documentation, I have installed the following packages:

ncurses-devel
geoip-devel
tokyocabinet-devel
openssl-devel
"Development Tools" yum group

I am configuring GoAccess with: ./configure --enable-utf8 --enable-geoip=mmdb
However, the output is as follows:

./configure --enable-utf8 --enable-geoip=mmdb
checking for a BSD-compatible install... /bin/install -c
checking whether build environment is sane... yes
checking for a thread-safe mkdir -p... /bin/mkdir -p
checking for gawk... gawk
checking whether make sets $(MAKE)... yes
checking for gcc... gcc
checking whether the C compiler works... yes
checking for C compiler default output file name... a.out
checking for suffix of executables...
checking whether we are cross compiling... no
checking for suffix of object files... o
checking whether we are using the GNU C compiler... yes
checking whether gcc accepts -g... yes
checking for gcc option to accept ISO C89... none needed
checking for style of include used by make... GNU
checking dependency style of gcc... gcc3
checking whether gcc and cc understand -c and -o together... yes
checking for pthread_create in -lpthread... yes
checking build system type... x86_64-unknown-linux-gnu
checking host system type... x86_64-unknown-linux-gnu
checking whether to build with rdynamic for GNU ld... yes
checking for MMDB_open in -lmaxminddb... no
configure: error:
*** Missing development files for libmaxminddb library.

Am I missing a package or doing something else wrong?
Thanks

I found the issue. I had to install the libmaxminddb-devel.x86_64 package as well. Maybe this should be added to the docs for CentOS?
Thanks!

This has been added to the download page. Thanks for heads up!


there should be a way to download the audio as mp3 or wav to export it to another application

If you run the server example (https://github.com/bryanjenningz/record-audio/tree/master/record-server-example), you should be able to record and download audio. Once you have the audio file, you should be able to use it however you want. Is this what you mean?

got you. i sort it out anyways. thanks.

How did you end up doing it without the server is their code that can save it to the file

yes i convert the file to mp3 and save it


[NPM] Support pod controller to improve reliability

Reason for Change:

This PR is to support pod controller in npm to improve reliability according to pod events.
For example, when event handling was failed, we need to retry them.
Issue Fixed:

Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[x] adds unit tests

Notes:
First put code layout to support pod controller.
Will need to fill up business logics.

Close this PR and create a new one ([NPM] Support pod controller and unit test to improve reliability #836
) to resovle conflict issue easily.


Fix for info() command

A fix for info command as the type is 0 and it tries to compare with a response that has the ? in the command, but the consts do not.  So this will compare only the size of the command

Thanks for the PR.


logo of XCELTOKEN PLUS (XLAB)

In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[*] Filename in the lowercase format
[*] Image format: png
[*] Image size: 256 by 256 px

Please update your fork for next PR, for now added new Xcel logo here https://github.com/TrustWallet/tokens/commit/c4fb3d11e8b67944220b426b098617ad68cd5a54


Support for exponential retry on failure

This is useful when there are datacenters with somewhat unstable connections. Previously range_repair.py stopped running if repair failed. Default behaviour is still the same (that is, fail on first), however this pull request introduces support for retries with exponential backoff.
Additional comments:

I decided to keep the retry logic in the same range_repair.py file to make it easy to copy the file to Cassandra nodes for execution.

What do you think? Tests pass (except for #37).

Retry in action in production:

Works like a charm. 🎉


How to train model that takes mutiple tensors as input

I code a model for testing which takes 2 differnt tensors as input as follows:
            Tensor input_wide = keras.Input(5);
            Tensor input_deep = keras.Input(6);

            var hidden1 = keras.layers.Dense(30, activation: keras.activations.Relu).Apply(input_deep);
            var hidden2 = keras.layers.Dense(30, activation: keras.activations.Relu).Apply(hidden1);
            var concat = keras.layers.Concatenate().Apply(new Tensors(input_wide, hidden2));

            var output1 = keras.layers.Dense(1, activation: keras.activations.Relu).Apply(concat);
            var output2 = keras.layers.Dense(1, activation: keras.activations.Relu).Apply(hidden2);

            var model = keras.Model(new Tensors(input_wide, input_deep),
                new Tensors(output1, output2)
            );

            model.compile(optimizer: keras.optimizers.Adam(),
                loss: keras.losses.MeanSquaredError(),
                new[] { "accuracy" });

Here I construct a Tensors with 2 tensor, which is assigned to the inputs of model. This model can be compiled without problems. Then I try to predict by this model:
                // Just test data
                var x1 = np.array(new float[,] { { 1, 2, 3, 4, 5 }, { 1, 3, 5, 7, 9 } });
                var x2 = np.array(new float[,] { { 1, 2, 3, 4, 5, 6 }, { 1, 3, 5, 7, 9, 11 } });
                var x = new Tensors(x1, x2);
                var pred = model.predict(x);

Here I got an exception System.Collections.Generic.KeyNotFoundException:“The given key '3' was not present in the dictionary.”.Did I make a mistake or is it a bug? Many thanks.

Yes, model can be compiled but cannot be used.
Unfortunately, there is only version of Predict that takes "Tensor", not "Tensors"
I need to be able to feed multiple inputs into Tensorflow model too.
Dear SciSharp/Tensorflow/Keras team, please consider prioritizing adding support for multi-input models.
Thank you!

Yes, please.
Not being able to train/predict models with more than one input blocks many tasks.

Yes, model can be compiled but cannot be used.
Unfortunately, there is only version of Predict that takes "Tensor", not "Tensors"
I need to be able to feed multiple inputs into Tensorflow model too.
Dear SciSharp/Tensorflow/Keras team, please consider prioritizing adding support for multi-input models.
Thank you!

The issues of tf.net have been delayed for a long time because short of hands. I'd like to help on this problem. What I understand is that there is a compiled keras.Model, a list of Tensors is supposed to passed to model.fit. Is that the point of this issue?

Yes, and model.predict too
My use-case is - neural network input is an image and a a float. It requires 2 separate tensors of a different shape as an input. It works in python, it works when converted to ONNX, but in it cannot be used in Tensorflow.Net.
I wonder whether adding an option to pass "Tensors" instead of "Tensor" would solve the issue

OK. I'll try to implement it and I'll tell you once I complete it.

Now the multiple inputs of model.fit and model.predict has been supported by #996 and #1000 .
A simple example with multi-inputs LeNet could be found here.

Seems to support 2 inputs and no more than 2?

Seems it can only accept 2 max (not multiple), and the inputs have to be of the same size (ie, can't do [1,28,28,1] for A, and [1] for b). Still trying to figure it out, real weird.


Update ClosureTreeRepositoryTrait.php

Bug fix - use name for parent column from config instead of fixed name 'parent'

thanks, but master branch may not be released, I suggest to use 2.4.x

Ok, can you suggest a way of extending a custom entity repository and having your closure tree repository functionality as the same time? The custom repository is a base repository developed by a third party, so there's no way to use it as a trait. Your library is used only for one entity, maybe a few more in the future, but I need to keep the consistency throughout all repositories that all inherit from the mentioned custom repository class. Or are you going to implement traits in 2.x branch? Thank you.

copy the source code in repository. it will not change much in the future.


Test old state support

In RPC methods like starknet_call which offer the user to use an old block state specifier.
The support for this is introduced with #104 but not thoroughly tested.

@FabijanC please can you add more description and what to do for this issue? It will be my please to tackle it

@vibenedict description updated

Hi @FabijanC, looking at the linked PR, I can't seems to find the file where BlockId::Tag(BlockTag::Latest) was tested. It would make my work faster if you could add link to the file in the description. Thanks

E.g. starknet_call is tested in https://github.com/0xSpaceShard/starknet-devnet-rs/blob/ed53233079c9dc338ea432d18e2eeb4e43ab85b2/crates/starknet-devnet/tests/test_call.rs#L28. Would be cool to test the case where there are e.g. 3 blocks, and you want to call with block_id being number 2. Also testing by hash would be good. Just be sure to spawn the background devnet with --state-archive-capacity full (see other instances of it).


Function Calling:  unexpected behavior within ShellGPT's REPL mode when using the --repl --shell options

I've encountered a potential issue or unexpected behavior with ShellGPT's REPL mode when using the --repl --shell options. I'm not 100% sure if this is a bug or if I'm not using the feature correctly.
Issue Description
When using the sgpt --repl --shell command and then asking it to list files in the current directory, the command's behavior is inconsistent between executions. Initially, the command executes but does not produce any output. Repeating the same command immediately after results in the expected output being displayed.
Steps to Reproduce
Start ShellGPT in REPL mode with the shell option: sgpt --repl --shell
Input the command: list files in this dir

Entering REPL mode, press Ctrl+C to exit.
>>> list files in this dir

> @FunctionCall `execute_shell_command(shell_command="ls -l")`

>>> list files in this dir
Here is the list of files and directories in the current directory:
total 388
drwxr-xr-x. 1 cal cal    554 Feb 15 10:24 bin
drwxr-xr-x. 1 cal cal     28 Jan 13 16:35 blog
drwxr-xr-x. 1 cal cal      0 Feb 15 09:54 Desktop
drwxr-xr-x. 1 cal cal    880 Feb 15 09:15 Documents

Note the command I issued 'list files in this dir' twice and how the first time didn't product output following the "@FunctionCall" notice.
Expected Behavior
I expected the command run a function and list the files and directories upon the first execution without needing to repeat the command for output.
Actual Behavior
The command needs to be executed twice to produce the expected output. The first execution runs the function without any visible output, and the second execution displays the correct information without seemingly running the function again.
Additional Information
I observed similar behavior with other commands that produce output, such as interactions with Task Warrior, which worked impressively well without additional configuration.
Is this behavior a bug, or am I missing a step in setting up or using the --repl --shell mode incorrectly?

This is actually a bug, --shell should not have access to functions. It is not particularly useful in the context of generating pure shell commands. Functions should only work with the default and custom roles. I tested it with default role sgpt --repl temp, and it works as expected. We will remove function calls from repl --shell/code in upcoming releases.

in powershell, it acts as follows:
>>> what is current folder?
```powershell
Get-Location

e

powershell : The term 'powershell' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the
spelling of the name, or if a path was included, verify that the path is correct and try again.
At line:1 char:1

  + CategoryInfo          : ObjectNotFound: (`powershell:String) [], CommandNotFoundException
  + FullyQualifiedErrorId : CommandNotFoundException

I attempted to replicate the issue using the command from the initial comment. It appears that --shell used immediately after --repl is just creating a session called "--shell".
Instead, the issue seems to be broader and occurs whenever the --chat or --repl flags are used and a function is called. sgpt outputs the function string, but then it doesn't manage to output the expected answer.


When LINECOLORS > 9, only label colors up to nine colors can be used.

Reported by ansley b manke on 17 Jan 2012 17:26 UTC
When we add more line colorse with MODE LINECOLORS, the lines get the new colors, but labels colored with pen number are black starting with pen 10. It looks like the command-parsing is looking for only one digit after @P for the pen color.  Here is a script:
yes? SET MODE LINECOLORS:10
yes? PPL COLOR,10,60,0,100
yes? plot/color=10/i=1:10 i
yes? label 5 10 -1 0 .2 @P10Here is a label in color 10

The label that is drawn is black and the text is "0Here is a label in color 10" so clearly the 0 is being parsed as part of the label. (How to decide what is really the intent?)
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1911

Comment by ansley.b.manke on 17 Jan 2012 19:22 UTC
In fact, we can specify any pen number. We just need more links and cross-references to this section of the documentation:
http://ferret.pmel.noaa.gov/Ferret/documentation/users-guide/appendix-b-pplus-guide/PPLUS-COMMAND-FILES#_VPID_639
Pen Selection

The pen may also be selected by giving the change pen command @Pn, where n is the character 1-9 and A-G. This allows the selection of up to 16 pens/colors. The color and font is reset to the default font and previous color after the character string is drawn. The PEN command can be used to change the default color by typing PEN,0,default_color.
If you need to select a color index beyond the range of P1 through PG, you can use the change color command @Cnnn where "nnn" is a 3-digit color index. (It must be 3 digits.)

Attachment from ansley.b.manke on 17 Jan 2012 17:27 UTC
the image from the script
REPLACE THIS TEXT WITH UPLOADED FILE ./attachments/TRAC_1911_GIT_1183/bug1911.gif


Unable to pass image to model using fastapi

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI, UploadFile, File
import uvicorn

import torch
import torchvision
from torchvision import transforms

from PIL import Image
from build_effnet import build_name_model
import torch.nn.functional as F

import io
from io import BytesIO
import numpy as np
import cv2

app = FastAPI()

class ClassificationServer:
    def __init__(self):
        self.class_names=['F_AF', 'F_AS', 'F_CA', 'F_LA', 'M_AF', 'M_AS', 'M_CA', 'M_LA']
        self.idx_to_class={i:j for i,j in enumerate(self.class_names)}
        self.class_to_idx={value:key for key,value in self.idx_to_class.items()}
        self.image_transforms={"test":transforms.Compose([
            transforms.oTensor(),
            transforms.Normalize([0.485,0.456,0.406],[0.229,0.224,0.225])
            ])}
        self.device=torch.device("cuda:0" if torch.cuda.is_available() else "cpu")
        self.model=build_name_model()
        self.model.load_state_dict('CharacterClass_effnet_SGD.pt',map_location='cpu')
        self.model.eval()
        self.model.to(self.device)
    def read_image(self,image_payload):

        stream=BytesIO(image_payload)
        image=np.asarray(bytearray(stream.read()),dtype="uint8")
        image=cv2.imdecode(image,cv2.IMREAD_COLOR)

        if (isinstance(image,np.ndarray)):
            img=Image.fromarray(image)
        return img

    def classify(self,payload):
        img=self.read_image(payload)
        img=self.image_transforms["test"](img)

        with torch.no_grad():
            ps=self.model(img.unsqueeze(0))
            ps=F.softmax(ps,dim=1)
            topk,topclass=ps.topk(1,dim=1)
        x=topclass.view(-1).cpu().view()
        return self.idx_to_class[x[0]]

    @app.get("/hello")
    def get(self):
        return "Welcome to Image classfiication server."
    @app.post("/classify_image")
    async def classify_image(self,file:UploadFile=File(...)):
        image_bytes=await file.read()
        #return self.classify(image_bytes)
if __name__=="__main__":
    uvicorn.run(app,host="<IP_ADDRESS>",port=8000)

Description
On executing the code,when I pass image to the server via curl command as shown below
curl -L -F<EMAIL_ADDRESS>http://<IP_ADDRESS>:8000/classify_image

I end up recieving the error
{"detail":[{"loc":["query","self"],"msg":"field required","type":"value_error.missing"}]}

Plus,I am unable to upload the image to the server.it seems that there is an issue with passing the file.Am I writing the code correctly.
I tried testing the upload funcitonality using the following code
from fastapi import FastAPI, File, UploadFile
import uvicorn

app = FastAPI()

@app.get("/files/")
async def create_file(file: bytes = File(...)):
    return {"file_size": len(file)}

@app.post("/uploadfile/")
async def create_upload_file(file: UploadFile):
    return {"file_size": file.filename}
if __name__=="__main__":
    uvicorn.run(app,host="<IP_ADDRESS>",port=8000)

But when I click on the server <IP_ADDRESS>:8000 I end up getting method not allowed.Passing the file to server via curl returns its filename
Most of the docs end up using the same snippet,I also pointed the upload issue in previous topic.but this is the main task I am trying to solve,which is upload image to server,and then get model predictions.
Any help will be appreciated
Operating System
Windows
Operating System Details
No response
FastAPI Version
0.75.1
Python Version
3.9.7
Additional Context

In your code, the get and classify_image methods are inside the class which I think should not be. I made a minimal example which seemed to work:
 # code omitted
app = FastAPI()

class ClassificationServer:
    def __init__(self):
       # code omitted
        pass

    def read_image(self, image_payload):
        # code omitted
        stream = BytesIO(image_payload)
        image = np.asarray(bytearray(stream.read()),dtype="uint8")
        return image

    def classify(self, img):
        # code omitted
        return 1

@app.get("/hello")
def get():
    return "Welcome to Image classfiication server."

@app.post("/classify_image")
async def classify_image(file: UploadFile=File(...)):
    image_bytes = await file.read()
    classifier = ClassificationServer()
    img = classifier.read_image(image_bytes)
    return classifier.classify(img)

if __name__=="__main__":
    uvicorn.run(app,host="<IP_ADDRESS>",port=8000)

Visit localhost:8000/docs to access api endpoints.

thanks @arunppsg  it worked


Possible Issue with Bulk Edit - File Fields

Hi Norm,
As I'm a newbie I'm not sure if its something I'm doing wrong or whether I've found an issue with the bulk edit not working on file fields?
https://www.loom.com/share/aa561424cc9e4984b97dae163e2a29b0?sid=3c641e3f-fd46-4230-8d05-8a096cdc303f

You're right Carl!  File type is not supported with the API.
I have removed the checkboxes for those, so we can't select them anymore.
Thanks for reporting,
Norm