Unable to search.  function ts_match_locs_array(unknown, text, tsquery) does not exist

Zulip standard install method. All work fine, except for this:
The above exception was the direct cause of the following exception:
`
Traceback (most recent call last):
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/django/core/handlers/base.py", line 185, in _get_response
response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/django/views/decorators/csrf.py", line 58, in wrapped_view
return view_func(*args, **kwargs)
File "./zerver/lib/rest.py", line 131, in rest_dispatch
return target_function(request, **kwargs)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/django/utils/decorators.py", line 149, in _wrapped_view
response = view_func(request, *args, **kwargs)
File "./zerver/decorator.py", line 605, in _wrapped_view_func
return authenticate_log_and_execute_json(request, view_func, *args, **kwargs)
File "./zerver/decorator.py", line 587, in authenticate_log_and_execute_json
return rate_limit()(view_func)(request, user_profile, *args, **kwargs)
File "./zerver/decorator.py", line 747, in wrapped_func
return func(request, *args, **kwargs)
File "./zerver/lib/request.py", line 174, in _wrapped_view_func
return view_func(request, *args, **kwargs)
File "./zerver/views/messages.py", line 767, in get_messages_backend
rows = list(sa_conn.execute(query).fetchall())
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/base.py", line 948, in execute
return meth(self, multiparams, params)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/sql/elements.py", line 269, in _execute_on_connection
return connection._execute_clauseelement(self, multiparams, params)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/base.py", line 1060, in _execute_clauseelement
compiled_sql, distilled_params
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/base.py", line 1200, in _execute_context
context)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/base.py", line 1413, in _handle_dbapi_exception
exc_info
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/util/compat.py", line 203, in raise_from_cause
reraise(type(exception), exception, tb=exc_tb, cause=cause)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/util/compat.py", line 186, in reraise
raise value.with_traceback(tb)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/base.py", line 1193, in _execute_context
context)
File "/home/zulip/deployments/2018-09-25-02-55-09/zulip-py3-venv/lib/python3.5/site-packages/sqlalchemy/engine/default.py", line 507, in do_execute
cursor.execute(statement, parameters)
File "./zerver/lib/db.py", line 33, in execute
return wrapper_execute(self, super().execute, query, vars)
File "./zerver/lib/db.py", line 20, in wrapper_execute
return action(sql, params)
sqlalchemy.exc.ProgrammingError: (psycopg2.ProgrammingError) function ts_match_locs_array(unknown, text, tsquery) does not exist
LINE 2: ...ECT message_id, flags, subject, rendered_content, ts_match_l...
^
HINT:  No function matches the given name and argument types. You might need to add explicit type casts.
[SQL: 'SELECT /* get_messages */ anon_1.message_id, anon_1.flags, anon_1.subject, anon_1.rendered_content, anon_1.content_matches, anon_1.subject_matches \nFROM ((SELECT message_id, flags, subject, rendered_content, ts_match_locs_array(%(param_1)s, rendered_content, plainto_tsquery(%(param_2)s, %(param_3)s)) AS content_matches, ts_match_locs_array(%(param_4)s, escape_html(subject), plainto_tsquery(%(param_2)s, %(param_3)s)) AS subject_matches \nFROM zerver_usermessage JOIN zerver_message ON zerver_usermessage.message_id = zerver_message.id \nWHERE user_profile_id = %(param_5)s AND (search_tsvector @@ plainto_tsquery(%(param_2)s, %(param_3)s)) AND message_id <= %(message_id_1)s ORDER BY message_id DESC \n LIMIT %(param_6)s) UNION ALL (SELECT message_id, flags, subject, rendered_content, ts_match_locs_array(%(param_1)s, rendered_content, plainto_tsquery(%(param_2)s, %(param_3)s)) AS content_matches, ts_match_locs_array(%(param_4)s, escape_html(subject), plainto_tsquery(%(param_2)s, %(param_3)s)) AS subject_matches \nFROM zerver_usermessage JOIN zerver_message ON zerver_usermessage.message_id = zerver_message.id \nWHERE user_profile_id = %(param_5)s AND (search_tsvector @@ plainto_tsquery(%(param_2)s, %(param_3)s)) AND message_id >= %(message_id_2)s ORDER BY message_id ASC \n LIMIT %(param_7)s)) AS anon_1 ORDER BY message_id ASC'] [parameters: {'message_id_2': 368, 'param_7': 51, 'message_id_1': 367, 'param_3': 'aja', 'param_4': 'zulip.english_us_search', 'param_5': 11, 'param_6': 50, 'param_1': 'zulip.english_us_search', 'param_2': 'zulip.english_us_search'}] (Background on this error at: http://sqlalche.me/e/f405)
`
Postgresql 10.
Ubuntu 16 x64

@bashman are you using Docker, or some other platform?  In any case, the problem here is that you don't have the tsearch_extras package installed or setup in your database server.  As Steve implies, all supported Zulip installation methods include it, so I'm very curious how you installed your system?

@bashman are you using Docker, or some other platform? In any case, the problem here is that you don't have the tsearch_extras package installed or setup in your database server. As Steve implies, all supported Zulip installation methods include it, so I'm very curious how you installed your system?

Nope, Ubuntu 16  fresh install and postgresql 10.

Before install Zulip:

sudo apt-get install postgresql-server-dev-all  postgresql-common

@bashman Ubuntu 16.04 comes with Postgres 9.5 (and thus that's what Zulip expects to use), not postgres 10; how did you install postgres 10 on your system?  We recommend always using the version of dependencies that come with the distribution you're using.  (If you connect to a remote postgres server running postgres 10, that's fine, but the Zulip installer for running everything on one server expects to use the dependencies that it installs).
If you really want to use postgres 10 in an all-in-one instrallation on Ubuntu 16.04, in addition to installing https://github.com/zulip/tsearch_extras, you'd need to also do some tweaks to the puppet configuration.  I'd recommend instead just installing zulip 1.9.0-rc2 (which you can get from the GitHub "releases" tab) on Ubuntu 18.04, though, since that'll save you some upgrade work (and gets you Postgres 10 without any extra work).

We now support Postgres 10 via Ubuntu 18.04.  I don't think there's anything else here for us to fix, as the issue was a custom installation of postgres without our search extensions.

I saw this issue on a server where we upgraded it from 14.04 to 16.04 to 18.04, then a colleague upgraded Zulip. The fix was to run:
CREATE EXTENSION tsearch_extras SCHEMA zulip;
from scripts/setup/postgres-create-db and search is immediately working.

@troffasky just to make sure I understand, it sounds like you replaced the postgres version as part of that upgrade, and the extensions were not preserved in that migration so you had to recreate the one we use manually?

Yes, originally installed Zulip 1.7.1 w/Postgres 9.3 on Ubuntu 14.04, did two big do-release-upgrade's to get us to 18.04 with Postgres 10.5, then colleague upgraded us to Zulip 1.9.2 with upgrade-zulip zulip-server-latest.tar.gz.


Migrate users as they authenticate

When we authenticate a user through Digital Identity, check if we have
a record in our database for them.  If we do, all is well.  If not,
we're in one of two cases;

This user hasn't logged in to GOV.UK at all before
This user hasn't logged in to GOV.UK since the migration

We can distinguish the two cases by calling the UserInfo endpoint, and
checking the govuk-account claim.  If present, this claim holds the
pre-migration subject identifier, which we can use to look up and
migrate the user in our database.

Trello card

@alex9smith I realised this PR was slightly wrong, see b8e9d3fe74ebd87a8feab787a71195a2aeabf4c9

Good spot!


Blackberry 10: Dont validate select validation after change

Hi,
See this simple example.

http://jsbin.com/ujixil/2
Press submit, you get a validation error
select a value and the error should disapear, but on BB10 it dont.

Suggestions for workaround or reason for this bug?
/Johan

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.
To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


Generate NO_BOOSTS list by looking at setts with no emissions

Right now we are hard coding the NO_BOOSTS variable in order to sett which setts aren't counted in boost. This should be changed by only looking at setts which have no boosted emissions

Closed via #300


Fix pyflakes and pep8 errors

Use extra asserts to use previously unused variables in tests,
such as cass and response.
Fix only pyflakes errors in docs/conf.py

fwiw, the .travis.yml changes will be mostly unnecessary after #228

Looks like I caused some conflicts when merging #228.  Can you rebase?
Also, I noticed quite a few new asserts.  Is this purely for pep8 reasons?  Does flake8 require an assert per test or something?

pyflakes requires that all variables are used.
Often the tests had with blah as cass: something which did not access cass.  Adding an assert for cass fixes the problem.  The alternative is to remove .. as cass:, but I guessed it is more useful to add asserts than remove cass which may be useful for debugging or future asserts.

Ohh gotcha.
I think I would prefer to do the first option (remove ..as cass) since I prefer to have fewer asserts per test.  Asserts without much meaning to the test can dillute the meaning of the important assert.  I strive for a single assert per test but often don't achieve it :)

Thanks for the cleanup!


[TODO_BLOCKER] Red0ne's Iteration16 TODOs

Objective
Tackle all on-chain blockers assigned to red-0ne
Origin Document

https://github.com/pokt-network/poktroll/pull/577

Goals

Tackle all major TODO's in the codebase assigned to red-0ne

Deliverables
The output of the following command from https://github.com/pokt-network/poktroll/pull/577:
grep -r "TODO_BLOCKER(@red-0ne)" . | sed 's/^/- [ ] `/' | sed 's/:/`:/'

[x] ./x/tokenomics/keeper/keeper_test.go:   // TODO_BLOCKER(@red-0ne): Use the governance parameters for more
[x] ./x/tokenomics/keeper/keeper_test.go:   // TODO_BLOCKER(@red-0ne): Use the governance parameters for more precise block heights once they are implemented.
[x] ./x/tokenomics/keeper/keeper_test.go:   // TODO_BLOCKER(@red-0ne): Use the governance parameters for more precise block heights once they are implemented.
[x] ./x/tokenomics/keeper/keeper_test.go:   // TODO_BLOCKER(@red-0ne): Use the governance parameters for more precise block heights once they are implemented.
[x] ./x/supplier/config/supplier_configs_reader.go: // TODO_BLOCKER(@red-0ne): This hasn't been implemented yet but is part of the
[x] ./x/proof/keeper/msg_server_submit_proof.go:    // TODO_BLOCKER(@red-0ne): Fetch the correct ring for the session this relay is from.
[x] ./pkg/partials/payloads/rest.go:        // TODO_BLOCKER(@red-0ne): Implement this function
[x] ./pkg/partials/payloads/rest.go:        // TODO_BLOCKER(@red-0ne): Implement this function
[x] ./pkg/partials/payloads/rest.go:        // TODO_BLOCKER(@red-0ne): Implement this method
[x] ./pkg/partials/payloads/rest.go:        // TODO_BLOCKER(@red-0ne): Implement this method
[x] ./pkg/partials/payloads/jsonrpc.go:     // TODO_BLOCKER(@red-0ne): Implement this method
[x] ./pkg/partials/partial.go:// TODO_BLOCKER(@red-0ne): This function currently only supports JSON-RPC and must
[x] ./pkg/partials/partial.go:      // TODO_BLOCKER(@red-0ne): Handle other request types
[x] ./testutil/testproxy/relayerproxy.go:// TODO_BLOCKER(@red-0ne): This function only supports JSON-RPC requests and
[ ] ./e2e/tests/session.feature:# TODO_BLOCKER(@red-0ne): Make sure to implement and validate this test

Creator: @red-0ne

@red-0ne I recommend running grep -r "TODO_BLOCKER(@red-0ne )" . | sed 's/^/- [ ] /' | sed 's/:/:/' to update the deliervables after #577 is merged in.

Closed as per #694


Swift keys with slashes do not work

I had to regenerate my swift keys. If the key has a / or \ then rclone refuses to work and gives the following error:
./rclone lsd betaimages:/
2015/04/08 15:53:13 Failed to create file system for "betaimages:/": Operation forbidden
using the same config and using a swift key without \ and / works great

When you say your key do you mean the API key?
What auth URL are you using?

yes the API Key.
I am using rclone with swift. The server is the Rados Gateway to my Ceph cluster. It is swift v1 compatible.
My auth URL looks like:
http://ipaddress/auth/v1.0

I was able to figure out the problem was the \ or / because I read a lot about swift and OpenStack and this is a common issue in many Swift/S3 clients. the API key could contain special characters and those are not always managed correctly by client applications.

I tried to reproduce this with a swift cluster.  I tried passwords test/test and test\test but both of those worked fine with rclone and v1 auth.
Looking at the v1 auth code, all it does is put the API key into an http header which are allowed to have / and \ in.
I see the note you are referring to in the ceph docs

Important Check the key output. Sometimes radosgw-admin generates a key with an escape () character, and some clients do not know how to handle escape characters. Remedies include removing the escape character (), encapsulating the string in quotes, or simply regenerating the key and ensuring that it does not have an escape character.

That seems to suggest that for ceph, to use a password with \ in you should enter it in quotes, eg "test\test".  That seems like a ceph specific work-around as with swift it works fine with \ in passwords.
So my feeling is this is a bug/incompatibility in Ceph rather than a problem with the swift client.

Seems to be using common swift auth header:
http://ceph.com/docs/v0.67.9/radosgw/swift/auth/
Saverio, are you saying a Auth-User with \ isn't working with rClone?
curl -i swift.supercoolswiftstorage.com  -H "X-Auth-User:”test\test" -H
"X-Auth-Key:yourapikey"
On Tue, Apr 21, 2015 at 6:45 AM, Nick Craig-Wood<EMAIL_ADDRESS>wrote:

I tried to reproduce this with a swift cluster. I tried passwords
test/test and test\test but both of those worked fine with rclone and v1
auth.
Looking at the v1 auth code, all it does is put the API key into an http
header which are allowed to have / and \ in.
I see the note you are referring to in the ceph docs
http://ceph.com/docs/v0.67.9/radosgw/config/
Important Check the key output. Sometimes radosgw-admin generates a key
with an escape () character, and some clients do not know how to handle
escape characters. Remedies include removing the escape character (),
encapsulating the string in quotes, or simply regenerating the key and
ensuring that it does not have an escape character.
That seems to suggest that for ceph, to use a password with \ in you
should enter it in quotes, eg "test\test". That seems like a ceph
specific work-around as with swift it works fine with \ in passwords.
So my feeling is this is a bug/incompatibility in Ceph rather than a
problem with the swift client.
—
Reply to this email directly or view it on GitHub
https://github.com/ncw/rclone/issues/47#issuecomment-94742034.

@ncw try this password test\/test or this password test/\test

@zeshanb no the problem is with the key in the .rclone.conf file. There is not a problem with the username

@zioproto
I tried test\/test and test/\test and they both worked fine on a swift cluster.
Can you try my suggestion above?

That seems to suggest that for ceph, to use a password with \ in you should enter it in quotes, eg "test\test". That seems like a ceph specific work-around as with swift it works fine with \ in passwords.

Thanks
Nick

I've got a file named "Call Log Export: 01/13/15 - 04/21/15". rclone syncs it successfully but treats slashes as directory separators and creates a long path with data in the file called '15'. Native windows google drive replaces special characters with underscores and creates a file named "Call Log Export_ 01_13_15 - 04_21_15" instead. Somehow I like this approach better.

@lvmm Yes you are right... Would you mind making this into a separate issue please?  It isn't related to the swift keys discussed in this one.
Thanks
Nick

@zioproto I have finally managed to replicate this.
When you got your credentials out of ceph, you probably got a json dump which looks something like this
{
    "user_id": "xxx",
    "display_name": "xxxx",
    "email": "",
    "suspended": 0,
    "max_buckets": 1000,
    "auid": 0,
    "subusers": [],
    "keys": [
        {
            "user": "xxx",
            "access_key": "xxxxxx",
            "secret_key": "xxxxxx\/xxxx"
        }
    ],
    "swift_keys": [],
    "caps": [],
    "op_mask": "read, write, delete",
    "default_placement": "",
    "placement_tags": [],
    "bucket_quota": {
        "enabled": false,
        "max_size_kb": -1,
        "max_objects": -1
    },
    "user_quota": {
        "enabled": false,
        "max_size_kb": -1,
        "max_objects": -1
    },
    "temp_url_keys": []
}

Because this is a json dump, it is encoding the / as \/, so if you use the secret key as "xxxxxx/xxxx" in the above example it will work fine.
I'll add this to the docs for s3
Thanks
Nick

There is now a section about this in the docs: http://rclone.org/s3/
Thanks for the report


Small error

“FIPS 140 validated” means that the cryptographic module, or a product that embeds the module, has been validated (“certified”) by the CMVP as meeting as meeting the FIPS 140-2 requirements.
"as meeting as meeting" should be replaced by "as meeting".

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

ID: e93f07bf-c0dd-dd83-d7bf-8562e7f197c3
Version Independent ID: a6c44187-8ff1-6519-5291-146b4c1bf46c
Content: FIPS 140 Validation - Microsoft 365 Security
Content Source: windows/security/threat-protection/fips-140-validation.md
Product: w10
Technology: windows
GitHub Login: @DulceMontemayor
Microsoft Alias: dansimp

@RAJU2529 : As you can see directly above my comment, you have opened 2 identical Pull Requests for this resolution.
I assume you will follow up by closing your duplicate PR (#5649 ) and keep open the first one (#5648 ).

@illfated. Yes sir you are right, after creating the first PR, i seen  commit and pull request on top page in the green button, so i thought of something went wrong in earlier PR , so I pulled request. Thats why two PR is created

@bgolinvaux The issue has been addressed by PR #5648 via community contributor, RAJU2529. Thank you for providing feedback.


[bug] Thumbnail images not loading

As seen in the screenshot, a large portion of the thumbnails do not load. This has been reproduced on multiple networks, and on two different devices.
Device 1: LG G5 with Android 7.0
Device 2: Lenovo YT-X703F with Android 7.1.1
Both with the latest version of Neko

If you click into one of the manga does the cover load?

Yes, after 15-20 seconds

Ok try something for me.  Make sure you are on 1.3.2 and then clear cache, cookies, db in advanced settings then try.

Nice, that fixed most of it!

Yea the few that don't load is cause I am guessing what the extension is.  Jpg vs jpeg vs png.


(Spring Boot) Access to NativeWebRequest in Sink

I am implementing a Sink that stores the requests/responses in the database. I wanna link these logs to our internal models (e.g. organization). For identifying models like this, we are using HandlerMethodArgumentResolver from Spring. In order to reuse these resolvers, we need access to the NativeWebRequest from Spring.
In our Sink we only get access to the logbook request object. So basically my question is, is there a sane way to access the underlying Spring request from the Sink request object?
Update:
Access to PathVariables would also suffice. Is that possible?

Access to PathVariables would also suffice. Is that possible?

Unfortunately no. There are two possible integrations for Spring:

Servlet Filter (Spring Web)
Netty Channel Handler (Spring WebFlux)

Neither of those two ways has access to any Spring-specific details of the request. In fact they run both very early in the request processing chain, long before Spring is even processing it.

In order to elaborate a bit on why that has been done the way it has:
The Spring (Boot) integration is like a cherry on top of the lower level integrations that we have. They can be used directly without the need to have Spring (Boot) at all. As an example, every user of a standard-compliant Servlet container can use logbook and it works exactly the same. The other benefit from a maintenance perspective is that we don't have to re-invent the wheel for every framework/environment as long as we are able to support the underlying technology directly.
For Servlet contains that is based on the Servlet API spec. For Netty it's similar, but slightly different, because it's not a specification, but implementation. But Netty is so dominant in the NIO-framework space that it's the defacto standard.
I hope that clarifies some of the underlying design decisions and principles.

Thanks for the detailed explanation and for making logbook open source! We will find another way to extract the necessary information.


Cache subscriber callback clients

Purpose

$Title

Fixes #14425
Fixes #19494
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #19546 into ballerina-1.0.x will not change coverage.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.0.x   #19546   +/-   ##
================================================
  Coverage            15.73%   15.73%
================================================
  Files                   47       47
  Lines                 1265     1265
  Branches               197      197
================================================
  Hits                   199      199
  Misses                1053     1053
  Partials                13       13

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 6de9146...e881c58. Read the comment docs.


Timeseries dataset of network halts with approximate timestamps

To analyse network halts it is useful to have a timeseries with approximate timestamps of when network wide halts have occurred in one or more epochs. This data can then be crossed with validator and fullnode logs and makes it easier to find the cause of the halt.
Analyse which data we could use to define a halt. One suggestion is to use the /metrics endpoint of the prometheus setup, where a we define a halt as "the sum of committed block per second of the active validator set". The downside is the dependency of Prometheus.
Data should be made available to grafana for visualization.

Closed.
Result is repo: 0L-log-indexer


fix decoding of values in padded key generator

Scope & Purpose
Fix decoding of values in padded key generator when restoring data.
Added unit test.

[x] :hankey: Bugfix (requires CHANGELOG entry)
[ ] :pizza: New feature (requires CHANGELOG entry, feature documentation and release notes)
[ ] :fire: Performance improvement
[ ] :hammer: Refactoring/simplification
[x] :book: CHANGELOG entry made

Backports:

[x] Backports required for: (potentially) 3.6 and 3.7

Testing & Verification

[x] The behavior in this PR was manually tested
[x] This change is already covered by existing tests, such as shell_client, shell_server.
[x] This PR adds tests that were used to verify all changes:

[x] Added new C++ Unit tests

Link to Jenkins PR run:
http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13674/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13675/

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/13675/


Config externals shortcut

Is there a config shortcut to auto externals dependencies / peerDependencies?
Something like this
{
  "esbuild": {
    "external": "auto",
  }
}

Unfortunately there currently isn't, but I'll look into adding it

Hey @okikio , first of all thanks for the amazing work.
It would great if we can automatically add peerDependencies in the npm package.json into esbuild.external config.
For example, an React UI npm package usually has "react" and "react-dom" as peer dependencies, in the default config we should have:
{
  "esbuild": {
    "external": ["react", "react-dom"]
  }
}

@szhsin I'm hesitant to enable that by default for react, as people still check the raw bundle size of react, but what I will do instead is support an auto option that automatically marks peerDeps as external.
Sorry @flamrdevs I forgot to enable this feature last year, I'll be enabling later this week.


The website cannot be deployed

The website cannot be deployed at the moment. It seems to be related to Streamlit Cloud and is reported. An update will be released when the issue is resolved.

The issue seems to have resolved now!


Not showing errors

I have an API key but 0 tokens left - and I just don't get any response. It should probably say something 😊

Yes I need add that.

Fixed in main branch.


Add attr.resolve_types

This adds attr.resolve_types which can be used to resolve forward declarations in classes created using __annotations__
Fixes #265
Pull Request Check List
This is just a reminder about the most common mistakes.  Please make sure that you tick all appropriate boxes.  But please read our contribution guide at least once, it will save you unnecessary review cycles!

[x] Added tests for changed code.
[ ] New features have been added to our Hypothesis testing strategy.
[x] Updated documentation for changed code.
[x] Documentation in .rst files is written using semantic newlines.
[x] Changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives.
[x] Changes (and possible deprecations) have news fragments in changelog.d.

If you have any questions to any of the points above, just submit and ask!  This checklist is here to help you, not to deter you from contributing!

I'm not sure if I need to implement any Hypothesis things.  Please advise.
Also feel free to kibbitz on the naming of the new function.
I'm also open to not caching whether we've called the value and just letting the caller decide whether to do so or not.

Code looks good on a first look but is this really what we should do or is that a hack around something else we should be doing?  I’m honestly asking because the whole topic is utterly confusing to me 🤯 and the discussions that happened here didn’t really help. :)

Sure! So here's what I understand:
There are 2 ways to get the type annotations.  cls.__annotations__ and typing.get_type_hints(cls)
Currently __annotations__ gives you the python object after the :.  This may be one of:

a concrete type, e.g. List[A]
something with a forward ref inside it, e.g. List['A']
a string. e.g 'List[A]'
In the future, it will just always give you a string, see https://www.python.org/dev/peps/pep-0563/.

typing.get_type_hints(cls) will take that jumble from __annotations__ and try to return only concrete types.  However, in order for that to work it must be able to resolve all the names.  If it can't find a single name then it will raise NameError and return nothing.  This makes it something that should not be called automatically.  Specially since in some cases (e.g. Imports that only exist in if TYPE_CHECKING: blocks) they may never be resolvable. (Also importing typing is slow?)
Now as far as attrs is concerned field.type might as well be an opaque object.  attrs doesn't use it for anything so it's just some metadata attached to the field and most users won't care about it at all.
So I think attrs is fine to continue using __annotations__, even after PEP-563 is in force, with 2 exceptions:

In the case of auto_attrib=True you have to know if something is typing.ClassVar[...]  Right now the code checks str(annot).startswith("typing.ClassVar") which fortunately handles all three annotation styles (typing.ClassVar[int], typing.ClassVar['int'], 'typing.ClassVar[int]'). Though it should probably also check .startswith("ClassVar") to handle `'ClassVar[int]'``.  (I should probably file this.)

Users who want to resolve field.type to concrete types.  This includes third party tools that maybe want to do runtime type checking (Like we use internally at my company).  Or something like cattrs that needs types to structure and unstructure objects.

So for those in 2, this code would be useful.
And just to double check, @Tinche would this code help you at all?

Before I dive any deeper, how are data classes coping with this, @ericvsmith?

It's an open issue: https://github.com/ericvsmith/dataclasses/issues/92
The only place where dataclasses cares about the value of the annotation is to detect if it's a ClassVar or an InitVar. Other than that, if an annotation is a string or an actual type, I don't care. This is equivalent to point 1 above (https://github.com/python-attrs/attrs/pull/302#issuecomment-348724255).
As far as point 2 is concerned, I don't think this is a dataclass or attrs issue. My suggestion is to ignore it and let the user deal with it, just as they will have to for normal classes.
In an earlier discussion about it (that of course I can't find), I think it was Guido who suggested that if we find a string attribute, just scan the string for ClassVar (and now InitVar, too). This might have been an in-person conversation, which might be why I can't find it. The question remains: is that good enough? What if someone actually has a local type or alias called ClassVar?
I'm deliberately avoiding importing typing because it is so expensive. Maybe PEP 560 will help in that regard, but I doubt it. If you look at the implementation (https://github.com/ericvsmith/dataclasses/blob/10fb487befe96e5e5c25952bcb609dbe6f7aaaa1/dataclasses.py#L384), you'll see the trick I use to only check for a ClassVar if typing has already been imported, thus avoiding the import myself. This is the thing that breaks with string annotations.
I'm sort of waiting for PEP 563 to be accepted or not before I do anything. If it's accepted, maybe there will be better tools for this.
Thanks for pointing me to this issue.

Thanks for your input!  IOW: you just YOLO it for now and check a bunch of string.
As a fun aside, we have a gross ClassVar thingie too (for the same reason): https://github.com/python-attrs/attrs/blob/master/src/attr/_make.py#L186 if it makes you feel better. :)

For now I completely ignore the issue and never treat a string as a ClassVar. The YOLO string comparison (as ugly as it is) is a hack I haven't written yet.
Your hack does make me feel better! Whatever we end up doing, I'm hoping we can use the same approach.

@ericvsmith, PEP 563 is accepted, it's being implemented now, I'll finish this around Christmas time. The only thing left is really f-strings. Unparsing those is not super easy ;-)
@euresti, I would much prefer if you didn't reimplement get_type_hints(). I don't want to deal with differences in resolution later and there's going to be differences when we're dealing with nested scopes, etc. You raise a fair point about the entire call to get_type_hints() failing if a single forward ref cannot be resolved. We should handle this by adding a custom error handler to the call (sorta kinda like errors='ignore' or 'surrogateescape', etc. in text decoding).
PEP 563 requires updates to get_type_hints(), too. So, if @hynek could wait a week or two for a late Christmas present with all this landed, I would appreciate that.

Would anyone feel bad if we pushed this to 18.1?  I would like to get the fixes of 17.4 out and this seems to be still a bit in the air.

@ambv: I'm not sure what you're trying to do with f-string parsing, but there might be some helpers in the compiler and ast that can help. I wouldn't mind exposing some of them, sort of like string.Formatter.parse() does for "".format() strings.

Delaying is fine, and I'd love to see what happens to get_type_hints.

How are we gonna proceed here? 😇

I'm actually unsure how useful this is in the context of mypy.  Perhaps this would serve better as separate project, one in which you want runtime types.  Maybe part of cattrs or something new.  I'm fine with closing this without merge.

I’m gonna have to rely on your, @chadrik  and @ambv’s judgement.  So voice y’all opinions, I really don’t know. :)

PEP 563 is implemented, typing.get_type_hints() resolves all types for runtime use.  If there's anything that you don't like about typing.get_type_hints(), make an issue on BPO and assign to me (lukasz.langa).

Based on all the new developments (mypy support, typing.get_type_hints() etc) I'm ok closing this unmerged.   Basically all this code does is call get_type_hints and set the type arguments in the class attributes.  Basically I don't think this should be a feature of attrs anymore.

Alright, thanks again for all the work you put into it!

Ok this is ready. The only part I'm not super happy about is that my tests pass globals() and locals(). This is only necessary because the classes are defined in local scope.  If the classes were defined in module scope (which is probably how most classes are defined) you don't need to pass the arguments.  I could move the class declaration outside but then I can't parametrize the slots flag. Also it means the class is defined somewhere else.  Oh but this is pytest. I don't have to put the test in a class.  Don't know how you feel about that.

Very random thought: it would totally be useful if it worked as a class decorator too, no?
Like in:
@attr.resolve_types
@attr.dataclass
class C:
   X: Foo

Interesting.  That will work unless you've got a ForwardRef
e.g. This works
@attr.dataclass
class Foo:
   ...

@attr.resolve_types
@attr.dataclass
class C:
   X: Foo

But this won't:
@attr.resolve_types
@attr.dataclass
class C:
   X: Foo

@attr.dataclass
class Foo:
   ...

But not-too hard to implement.


Add support for per-KV entry content-type

As mentioned at https://blog.cloudflare.com/catching-up-with-workers-kv/, it should be possible to use getWithMetadata to get key metadata (including the content-type).
I'm not sure if wrangler is populating the KV entry metadata yet. I thought I remember seeing a PR for it, but can't find it to see if it was merged yet or not.

This is a very old issue so I'm going to close it for now. If you're still running into problems feel free to comment with more details and we can investigate further.


[FLINK-8029] Create WebMonitorEndpoint

What is the purpose of the change
The WebMonitorEndpoint is the common rest endpoint used for serving
the web frontend REST calls. It will be used by the Dispatcher and
the JobMaster to fuel the web frontend.
PR is based on #4986.
Verifying this change
This change is a trivial rework / code cleanup without any test coverage.
Does this pull request potentially affect one of the following parts:

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

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for the review @GJL. Merging this PR after Travis gives green light.


Further reduce stats lock contention

Avoid duplicate file open/close activity in the common case of updating the stats file under the lock. Instead, check if the file exists and open it with r+ mode to make in-place edits. Otherwise, if the file does not exist, open it with w mode to write the initial stats.

Thanks for the PR

Linking issue https://github.com/matus-chochlik/ctcache/issues/36


Use match statements for comparison chains

Signed-off-by: Lucas Steuernagel<EMAIL_ADDRESS>

I don't see that this is better, other than keeping clippy happy. However, it's not as awful as I remember.
Happy to merge this


network collector (net.go) uint32 -> uint64

BytesReceivedPerSec, BytesSentPerSec within Win32_PerfRawData_Tcpip_NetworkInterface are uint32.
https://msdn.microsoft.com/en-us/library/aa394293(v=vs.85).aspx
This seems to cause rollover when collecting from large, 10Gb+, interfaces. BytesTotal is uint64 and works fine.
I've been looking around for an alternate wmi path to expose this information as uint64, but nothing has jumped out yet.

egh, that link is to perfformatted data.
PerfRawData seems to be uint64 all the way down.
Closing this for now, the issue must be something else.

Looks like the issue is net.go is defining most of the items as uint32, similar to the above link. Changed all uint32's in Win32_PerfRawData_Tcpip_NetworkInterface struct to uint64 is working great.
seem reasonable? Further down it looks like that was the original intention.

Hi!
Thanks for reporting this. Indeed it looks like the data types are uint64, so it should be enough to just change the types in the Win32_PerfRawData_Tcpip_NetworkInterface struct. If you feel like submitting a PR, that would be great! Otherwise we'll try to get to it as soon as possible.

Fixed in #61, thanks!

The initial release of this failed to build due to a broken url. v0.2.2 should be finished shortly, including binaries.

Glad this got sorted out. I guess the microsoft docs are incorrect then? Over here, they are listed as 32bit:
https://technet.microsoft.com/en-us/security/aa394340(v=vs.80)

@martinlindhe Hm. It is possible they changed the type in later versions (the docs you linked seem to be for WinXP?). We should test this on both older versions of Windows, as well as different bitnesses. Also, what about the 32-bit version we ship?

Hm. Is it for XP? All i could deduce from the article is the metrics was introduced in XP / Windows Server 2003. Is there some newer docs showing them as 64bits?
If  the struct had different sizes for a 32-bit Windows, one might think they would all be 32-bit.
I just hope we don't need to have 2 versions of this :-)

Ah. No, on closer reading, this is the docs for Visual Studio v8.0, also known as Visual Studio 2015.
I checked my local machine, and it reports the types as uint64 (gwmi Win32_PerfRawData_Tcpip_NetworkInterface | gm -MemberType Properties | select Definition).
So, I dug into the WMI library we're using, and this line seems to indicate that all unsigned types are saved to uint64, regardless of their actual type? If I understood that correctly, we should be good. I think.

Okay, sounds good. I've been meaning to try the project out on some older Windows versions, I'll try to remember checking this out when I'm at it.


Laravel Reverb Message pusher.error

Reverb Version
@beta
Laravel Version
10.47
PHP Version
8.2
Description
I have installed Larave Reverb on Laravel 10  and after installing everything I websocket fetch pusher error message
{"event":"pusher:error","data":"{\"code\":4001,\"message\":\"Application does not exist\"}"}
Steps To Reproduce
My Event:
 public function broadcastOn()
    {
            return new PresenceChannel('reverb');
   }

import Echo from 'laravel-echo';
import Pusher from 'pusher-js';
window.Pusher = Pusher;

window.Echo = new Echo({
  broadcaster: 'reverb',
  key: import.meta.env.VITE_REVERB_APP_KEY,
  wsHost: import.meta.env.VITE_REVERB_HOST,
  wsPort: import.meta.env.VITE_REVERB_PORT ?? 80,
  wssPort: import.meta.env.VITE_REVERB_PORT ?? 443,
  forceTLS: (import.meta.env.VITE_REVERB_SCHEME ?? 'https') === 'https',
  enabledTransports: ['ws', 'wss'],
});

// PresenceChannel('reverb')

window.Echo.join('reverb')
  .here((users) => {
    console.log('here', users);
  })
  .joining((user) => {
    console.log('joining', user);
  })
  .leaving((user) => {
    console.log('leaving', user);
  });

FWIW, I'm having the same exact issue. This is using a fresh Laravel 10 project + Herd on windows 11. Everything is boilerplate using the code from the documentation pretty much.
Also when I dispatch an event, I get the following error
Pusher error: No matching application for ID [...]

@xerk

+1

same issue. I use laravel sanctum api setup with vue 3 as front end. stuck with
{event: "pusher:error", data: "{"code":4001,"message":"Application does not exist"}"} data :  "{\"code\":4001,\"message\":\"Application does not exist\"}" event :  "pusher:error"

I'm not using Echo/Pusher but in debugging I found that ws://localhost:8080/app/[app_id] which using the REVERB_APP_ID started working when [app_id] was replaced by the APP KEY - although in the configs with Echo above I see that everyone is supplying   key: import.meta.env.VITE_REVERB_APP_KEY so I'd think that would work...

What does the request to connect to the WebSocket server look like in your network tab?

@mousa4007 yes I solved it, I was using wrong key make sure u are using the key of reverb

@mousa4007 how you solve this?

@mousa4007 yes I solved it, I was using wrong secret make sure u are using the secret of reverb

can explain better how you fixed it?

Hello everyone, the problem went away when I deleted the config/reverb.php file.

Same error here, any solution?

找到问题了，修改 .env 参数需要重新运行 php artisan reverb:start

same issue. I use laravel sanctum api setup with vue 3 as frontend. stuck with
{event: "pusher:error", data: "{"code":4001,"message":"Application does not exist"}"} data : "{\"code\":4001,\"message\":\"Application does not exist\"}" event : "pusher:error"

In my case I was you using the wrong REVERB_APP_KEY in laravel-echo font end

I had the same error. The problem was in the config/reverb.php file. The app definitions need to be under the apps key, which should be under another apps key.
Correct:
   'apps' => [

        'provider' => 'config',

        'apps' => [
            [
                'key' => env('REVERB_APP_KEY'),
                 ...
            ],
        ],

    ],

My wrong config:
   'apps' => [
        'provider' => 'config',
         'key' => env('REVERB_APP_KEY'),
         ...
    ],


[ALLUXIO-2565 ] Improve concurrent UFS read efficiency PR1

PR1. see full PR #4799

thrift and protobuf changes.
Simple changes in common.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13590/Build result: FAILURE[...truncated 1654 lines...][JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/client/pom.xml to org.alluxio/alluxio-keyvalue-client/1.5.0-SNAPSHOT/alluxio-keyvalue-client-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/common/pom.xml to org.alluxio/alluxio-core-server-common/1.5.0-SNAPSHOT/alluxio-core-server-common-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/hdfs/pom.xml to org.alluxio/alluxio-underfs-hdfs/1.5.0-SNAPSHOT/alluxio-underfs-hdfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/hdfs/target/alluxio-underfs-hdfs-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-underfs-hdfs/1.5.0-SNAPSHOT/alluxio-underfs-hdfs-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/hdfs/target/alluxio-underfs-hdfs-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-underfs-hdfs/1.5.0-SNAPSHOT/alluxio-underfs-hdfs-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/swift/pom.xml to org.alluxio/alluxio-underfs-swift/1.5.0-SNAPSHOT/alluxio-underfs-swift-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/swift/target/alluxio-underfs-swift-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-underfs-swift/1.5.0-SNAPSHOT/alluxio-underfs-swift-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/swift/target/alluxio-underfs-swift-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-underfs-swift/1.5.0-SNAPSHOT/alluxio-underfs-swift-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/examples/pom.xml to org.alluxio/alluxio-examples/1.5.0-SNAPSHOT/alluxio-examples-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/s3a/pom.xml to org.alluxio/alluxio-underfs-s3a/1.5.0-SNAPSHOT/alluxio-underfs-s3a-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/s3a/target/alluxio-underfs-s3a-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-underfs-s3a/1.5.0-SNAPSHOT/alluxio-underfs-s3a-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/s3a/target/alluxio-underfs-s3a-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-underfs-s3a/1.5.0-SNAPSHOT/alluxio-underfs-s3a-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/server/pom.xml to org.alluxio/alluxio-core-server/1.5.0-SNAPSHOT/alluxio-core-server-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/common/pom.xml to org.alluxio/alluxio-keyvalue-common/1.5.0-SNAPSHOT/alluxio-keyvalue-common-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/pom.xml to org.alluxio/alluxio-underfs/1.5.0-SNAPSHOT/alluxio-underfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/glusterfs/pom.xml to org.alluxio/alluxio-underfs-glusterfs/1.5.0-SNAPSHOT/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/glusterfs/target/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-underfs-glusterfs/1.5.0-SNAPSHOT/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/underfs/glusterfs/target/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-underfs-glusterfs/1.5.0-SNAPSHOT/alluxio-underfs-glusterfs-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/integration/pom.xml to org.alluxio/alluxio-integration/1.5.0-SNAPSHOT/alluxio-integration-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/dependency-reduced-pom.xml to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT.pom[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/target/alluxio-core-protobuf-1.5.0-SNAPSHOT.jar to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/core/protobuf/target/alluxio-core-protobuf-1.5.0-SNAPSHOT-sources.jar to org.alluxio/alluxio-core-protobuf/1.5.0-SNAPSHOT/alluxio-core-protobuf-1.5.0-SNAPSHOT-sources.jar[JENKINS] Archiving /home/jenkins/workspace/Alluxio-Pull-Request-Builder/keyvalue/pom.xml to org.alluxio/alluxio-keyvalue/1.5.0-SNAPSHOT/alluxio-keyvalue-1.5.0-SNAPSHOT.pomchannel stopped
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13591/
Test FAILed.

@apc999 PTAL

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13608/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13611/
Test FAILed.

Merged build finished. Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13615/
Test FAILed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13632/
Test FAILed.

minor comments / questions left. otherwise this PR looks good to me.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13646/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13655/
Test FAILed.

Merged build finished. Test FAILed.

@calvinjia @apc999 i've addressed your comments except one about LockBlockResult. I will investigate that and ask Jiri tomorrow.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13669/
Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13670/
Test FAILed.

Merged build finished. Test FAILed.


Environment variables of dust path is missing when alacritty is started via Powertoys Run

Microsoft PowerToys version
0.84.0
Installation method
PowerToys auto-update
Running as admin
No
Area(s) with issue?
PowerToys Run
Steps to reproduce
Alacritty will miss the environment variable of dust path when launched by Powertoys Run, whereas windows terminal does not have this issue.
If I launch Alacritty directly, Alacritty can use dust normally.
I'm not sure if it's a issue with alacritty or with Powertoys Run.
✔ Expected Behavior

❌ Actual Behavior

Other Software
Alacritty 0.13.2 (installed via winget)
dust 1.0.0 (installed via winget)

Ran into this same issue with installing vim through win-get.
I added C:\Program Files\Vim\vim91 to the Path environment variables through sysdm.cpl, however when opening pwsh.exe or cmd.exe through PowerToys Run I would get this same error when trying to run vim from either open shell vim: The term 'vim' is not recognized as a name of a cmdlet, function, script file, or executable program..  Opening cmd or pwsh from the Windows Start Menu would not show this error and instead would start vim correctly. I checked the output of $env:PATH in pwsh opened from the Windows Start Menu and it displayed the entry C:\Program Files\Vim\vim91 that I had added earlier. Opening pwsh.exe or cmd.exe through PowerToys Run did not display an output for $env:path that was updated to include the entry which I had added.
Exiting PowerToys and opening the program again resolved the issue and PowerToys Run would then open cmd.exe or pwsh.exe with an updated Path variable that included the entry I had added while PowerToys was open. I assume this will also apply for other shells such as Alacritty.


Symbol Search should show metadata symbols in addition to source symbols

Environment data
dotnet --info output:
VS Code version: 1.19.1
C# Extension version: 1.13.0
Steps to reproduce
Open a script from dll
Cannot match symbols
Expected  behavior
Actual behavior

@rchande

Closing as there has not been a lot of demand for this enhancement.


Updated sort order

To keep related items instead of the current related kinds together it is proposed to sort based on the item path instead of the item string representation. See https://github.com/Enselic/public_items/pull/53#discussion_r846355796 for a possible implementation and more discussion.
Current situation:
- pub enum path::one::enum
- pub fn path::extra::enum::do_something()
- (possibly more fn)
- pub fn path::one::enum::do_something()

Proposed sorting
- pub fn path::extra::enum::do_something()
- pub enum path::one::enum
- pub fn path::one::enum::do_something()
- (possibly more fn)

In https://github.com/Enselic/cargo-public-api/pull/195 there is now proper grouping of all items, if the --unsorted flag is used. See example in that PR.
Here is my plan (subject to change):

Land that PR
Make a follow-up PR that by default uses the grouping rather than lexically sorting. Or put differently: The grouping will be considered being sorted already. That PR will close this issue.

PR that I think fixes this: https://github.com/Enselic/cargo-public-api/pull/198


CSV Import/Export: Implement RFC4180

RFC4180 (http://tools.ietf.org/html/rfc4180) from 2005 is the primary effort to standardise the CSV format. While it's still fairly loose, it does define enclosure escaping pretty well in a way that is at odds with SilverStripe's import/export scripts. SilverStripe's format is also incompatible with major CSV readers and creators like Microsoft Excel, which causes issues when providing exportable reports in the corporate environment.
The changes that would bring SS in line with RFC4180 would be the following:
// CSVParser#fetchCSVRow()
// ...
$value = str_replace(
    array('\\'.$this->enclosure, $this->enclosure.$this->enclosure,'\\'.$this->delimiter), // <--
    array($this->enclosure, $this->enclosure, $this->delimiter), // <--
    $value
);
// ...

Delimiters do not need to be escaped because RFC4180 states that they should always be surrounded by an enclosure, however this and the old enclosure escaping are left in for compatibility. It may be cleaner to remove these older ones and avoid any side effects that they may cause, though this will mean holding off until a minor version.
// GridFieldExportButton#generateExportFileData()
// ...
foreach($items->limit(null) as $item) {
    // ...
    $value = str_replace(array("\r", "\n"), "\n", $value);
    $columnData[] = '"' . str_replace('"', '""', $value) . '"'; // <--
}
// ...

Note that this breaks any reliance on the old format, so it will need to be changed in a minor version.
There may be more tweaks required for RFC4180 that I haven't noticed yet, but these two should work for most files.

Not implemented in 3 years, closing.
See https://github.com/silverstripe/silverstripe-framework/pull/3435


[Chore]: E2E tests should wait for spaces and orgs to finish deleting successfully

Background
We've recently discovered a number of issues with space and org deletion, where resources can be orphaned or the space/org can fail to ever finish deleting due to unfinalized resources in their namespace. This change will ensure that we catch these problems in the future if they recur
Action to take
Change the space and org teardown logic in the e2e tests so that it checks the job status for the deletion and ensures that it succeeds. Once #2604 and #2605 are complete, this will ensure that the spaces and orgs are actually deleted. Tests should fail if the deletion doesn't complete in a reasonable amount of time.
Impact
We'll catch issues with deleting spaces and orgs more quickly.
Dev Notes
The simplest way to accomplish this is likely by changing the deleteSpace and deleteOrg helpers

After some discussion amongst the AMER team, we don't think we should execute this chore, so I'm closing it.


transforming object keys

I'm curious if there's a convenient way of doing this in Ramda:
transform = (result) ->
  name: result.name
  id: result.id
  img: result.picture.data.url

I find myself doing this rather often. But at the same time, its a pretty simple function so it doesnt seem too unreasonable to just write out the function.

+1.
Using props and zipObj, this is pretty easy to achieve when the properties set is known. It's getting messy once you want to rename a single property (or a small subset).

I agree that writing ad hoc transformation functions isn't elegant, but I don't see a simple API that would handle your example, @ccorcos.

Well heres an attempt:
transform = R.pickPath({
  name: ['name']
  id: ['id']
  img: ['picture', 'data', 'url']
})

At the very least, its point free.

Well heres an attempt:

transform = R.pickPath({
  name: ['name']
  id: ['id']
  img: ['picture', 'data', 'url']
})

But might one also want to do something like
var transform = R.whatever({
    fullName = person => person.first + ' ' + person.last,
    id: R.prop('id'),
    img: R.path(['picture', 'data', 'url'])
})

... or some other transformation than simply prop/path?

R.pickPath is too narrow to be added to Ramda, in my view, and R.whatever is less convenient than a lambda. Lambda seems to be the right tool for this particular job.

I don't want to argue for whatever, as my rationale for bringing it up was only to show that pickPath was too limited to be really useful.
I don't think whatever with its implied API is generally useful enough for Ramda, but I also don't think that it's necessarily easy to replace it with lambdas.  If I had named functions above, for whatever reasons, then this would be very readable code (modulo a better name for the function):
var transform = whatever({
    fullName: extractName,
    id: getId,
    img: pullImg
});

compared to doing this via a lambda:
var transform = obj => {
    fullName: extractName(obj),
    id: getId(obj),
    img: pullImg(obj)
};

Still, I think that if someone wants to write this (and give it a good name), the best location would be the Cookbook.

Oh nice! The cookbook is a great idea!


Update python-package.yml

add support for Python 3.9 to the GitHub actions.
Reminders

[ ] Run black . to make sure the code follows the style guide.
[ ] Add tests for new features or tests that would have caught the bug that you're fixing.
[ ] If adding new functionality, add an example to the docstring, gallery, and/or tutorials.

Never mind. It looks like Python 3.9 is still not supported by PyTorch. I will close this for now and reassess in July 2021.


XML .plist example does not work

Line 11 and 12 in the .plist XML example have incorrect XML close tag. <string>...<string> should be <string>...</string>

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

ID: 870e6699-64bf-a816-ad17-1f3f1b9d229d
Version Independent ID: f51f0314-5ace-5041-002f-3ad464240b62
Content: How to schedule scans with MDATP for macOS - Windows security
Content Source: windows/security/threat-protection/microsoft-defender-atp/mac-schedule-scan-atp.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Fixed in this commit


Hotfix: arithmatic with off-midnight start-of-day

Codecov Report

Merging #27 (7dde6e1) into main (69be687) will increase coverage by 0.11%.
The diff coverage is 98.94%.

@@            Coverage Diff             @@
##             main      #27      +/-   ##
==========================================
+ Coverage   84.84%   84.95%   +0.11%
==========================================
  Files          77       77
  Lines        4994     5038      +44
==========================================
+ Hits         4237     4280      +43
- Misses        757      758       +1

Impacted Files
Coverage Δ

portfolyo/tools/__init__.py
100.00% <ø> (ø)

portfolyo/tools/standardize.py
89.02% <ø> (ø)

portfolyo/core/pfline/enable_arithmatic.py
92.10% <94.11%> (-0.63%)
:arrow_down:

portfolyo/core/pfline/interop.py
91.25% <100.00%> (ø)

portfolyo/core/pfline/multi_helper.py
97.72% <100.00%> (ø)

portfolyo/core/pfstate/pfstate_helper.py
96.96% <100.00%> (ø)

portfolyo/tools/intersect.py
96.29% <100.00%> (ø)

tests/core/pfline/test_multi_helper.py
100.00% <100.00%> (ø)

tests/tools/test_intersect.py
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add IotCentral ARM SDK

Description

Add C# SDK for Iot Central for the first time
Add Iot Central SDK tests

API Spec PR: https://github.com/Azure/azure-rest-api-specs/pull/3185

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] Please add REST spec PR link to the SDK PR
[x] I have read the contribution guidelines.
[x] The pull request does not introduce breaking changes.

General Guidelines

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[x] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[x] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (@emgarten: Uses latest, let me know if this should be changed. I based the generate files off other SDKs in the repo)
[x] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. (@emgarten: this is set in the csproj)

Thanks for the feedback @dsgouda I've updated the PR

//cc @vishwam

@dsgouda when will the package for this be on nuget.org?

@emgarten we don't publish the packages. Please read the instructions to publish the package here.


Counting over group join returns wrong result

A query of the form
            var result = (
                from f in db.Foos
                join b in db.Bars on f.FooId equals b.FooId into bars
                select new { Name = f.FooId, Count = bars.Count() }
                ).Take(15).ToArray();

returns the wrong result.
Steps to reproduce
Model
    public class MyFooEntity
    {
        [Key]
        public Guid FooId { get; set; }
    }

    public class MyBarEntity
    {
        [Key]
        public Guid BarId { get; set; }

        public Guid FooId { get; set; }

        public MyFooEntity Foo { get; set; }
    }

    public class MyContext : DbContext
    {
        public DbSet<MyFooEntity> Foos { get; set; }
        public DbSet<MyBarEntity> Bars { get; set; }

        protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        {
            optionsBuilder.UseSqlServer("Server=.\\;Database=mycdb;Integrated Security=SSPI;");
        }
    }

Test
        static void Main(string[] args)
        {
            var db = new MyContext();

            if (db.Foos.Count() == 0)
            {
                for (int i = 0; i < 20; ++i)
                {
                    var foo = new MyFooEntity() { FooId = Guid.NewGuid() };

                    db.Foos.Add(foo);

                    for (int j = 0; j < 20; ++j)
                    {
                        var bar = new MyBarEntity() { BarId = Guid.NewGuid(), FooId = foo.FooId };

                        db.Bars.Add(bar);
                    }
                }

                db.SaveChanges();
            }

            var result = (
                from f in db.Foos
                join b in db.Bars on f.FooId equals b.FooId into bars
                select new { Name = f.FooId, Count = bars.Count() }
                ).Take(15).ToArray();

            var text = JsonConvert.SerializeObject(result, Formatting.Indented);

            Debug.WriteLine(text);

            Debug.WriteLine("done.");
        }

Output
[
  {
    "Name": "f73a9dce-afeb-4269-9f65-0fe3258f9190",
    "Count": 15
  }
]

Expected are 15 objects with a count of 20 each, not one with a count of 15.
Further technical details
EF Core version: 1.1.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:
IDE: Visual Studio 2017

This has already been fixed in dev, I believe by #7823.
In 1.1.1 the Take call is being translated into a top statement in the sql which is limiting the number of results.  That is why you are only seeing 1 result with a 15 count.  In dev branch the count it is being applied clientside.
In dev this will currently not produce an optimal query since group by is not yet supported, but it does produce the expected result.

Yup, this is a dupe of https://github.com/aspnet/EntityFramework/issues/7497, closing

I'm glad to hear it!


Extended Edtition view

In the doodles project we have a contstruct that I think is interesting enough that I propose that it should be expressed as a seperate View in the standard.
(I am not even sure if edition is the right word to use here)
ExtendedEditions: a struct that is Edtionable and Retireable.

name: the edition have a name, for doodles this could be "Core" or "Unique"
type: the type of this edition, this can be that it is a "set" or a "position" or a "template" or a "season"
number: the number this item has in the edition
status: is this edition still open for minting or is it closed
maxNumber: a number signaling either final edition count or circulating supply if still active
traits: Additional traits/tags/attributes that this edition may have.

A concrete example

Wearables in doodles are in a set
each set has a given name and a counter to keep track of how many have been minted with that set
a set also has a creator

We tried modelling this as an Edition view and aditional Traits but that was confusing was the feedback I got so right now that data has been removed from the Views that wearables expose.
Contributors
We need more feedback on if this is something that should be represented by a standard view. If you want to see this as a view, please respond!

Is this related to #152 ? I'm sorry, but I don't fully understand the need for this either

In doodles i tried to expose the rich data model we have as a combination of traits and editions but got pushback from gaia and flowty that it was confusing. Thus the need for a richer view to expose the data.

If this is of no interest to the rest of the flow community we can always just expose it as a doodles specific view in our own Views file.

yeah, I think we would need to see other examples and needs to know if we need to make this a standard view or not, but I'm leaning towards no, but I'm happy to support it if a bunch of others think it is a good idea and we get some other examples

@bjartek Are you okay with me closing this or do you want to get some more opinions on it first?

Feel free to close


Describe table

Add this to the SQHTable buffer mapping. Something like
K - describe the table under the cursor

done


[Tracking issue] Addition of year-end & start-of-the-year SIG co-chair/lead activities

We have several activities that are performed close to year-end & start of next year. Some of them include:

Preparation of PR Wrangler schedule for the upcoming year
Preparation of Issue Wrangler schedule for the upcoming year
Pruning the OWNERS file
Preparing the annual report for the past year

It'd be helpful to have this documented somewhere in k/website or k/community.
Tagging @a-mccarthy since you already have a version of this documentation up on GDocs.
cc: @reylejano @natalisucks

/triage accepted

Adding a doc in k/community seems like the logical next step. I will be opening a PR and link it here.

@divya-mohan0209 can i help with this issue?

@iArchitSharma Hi there - unfortunately, these are tasks that @divya-mohan0209, @reylejano, and myself must complete as co-chairs of SIG Docs. However, there might be chances to help contribute to the annual report that we'll make known in the #sig-docs channel on k8s Slack when that work begins.


DOC: Minor doc formatting.

use continuation prompt ... for better parsing of example.
Use double backticks when snippet is not supposed to be a reference.
Mode a dot that ends a sentence outside of reference backticks.
Use Examples section to detect examples.
Properly indent "Returns" to be picked by numpydoc.

Thanks @Carreau, merged.  (Test failures were unrelated to this PR.)

Thanks!


[DOC]: how to infer by chatgpt models

📚 The doc issue
Hello, authors and developers，I have a problem with chatgpt. I train my model with single GPU, I want to know how to infer or predict the chatgpt model trained based on scripts such as examples/train_prompt.py, without related scripts or some dependencies。Thanks for your reply.

meet the same issue.
anyone can write a doc or any blog to show how to use colossalai to infer

Hi @Muzzypepper and @hurun Thanks for the feedback, we will provide it soon.

Please let me know if a doc is published. Thanks


v2.0 uses innerHTML, which removes nodes referenced in other code

I am trying to use this extension with highlightjs inside a ckeditor widget. The line numbering is added fine, but when ckeditor switches to source mode or attempts to save, it crashes. Without this extension, highlightjs works fine with ckeditor. I think it's because v2 uses innerHTML instead of DOM methods, and therefore nodes ckeditor expects to find are rewritten after they are already referenced.
Any chance v2 could be improved to use DOM methods like v1.1 did?

@jamespatterson, thanks for using the plugin and for your question!
Currently used new layout for numbers panel, so innerHTML remain in future.
I can advise solutions:

use v1.1
customize v2.0 for your cases
reinitialize ckeditor after switching

I ran into the a similar issue and using v1.1 instead of v2.x solved the problem, thanks @wcoder!


Redesigned firstchild as nav page: "Visual Studio Code for web development"

Rendered page for review:
https://review.docs.microsoft.com/en-us/microsoft-edge/visual-studio-code/

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

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/sources/opening-sources-in-vscode.md
:white_check_mark:Succeeded
View

microsoft-edge/visual-studio-code/index.md
:white_check_mark:Succeeded
View

microsoft-edge/visual-studio-code/microsoft-edge-devtools-extension.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 11d6bd7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/sources/opening-sources-in-vscode.md
:white_check_mark:Succeeded
View

microsoft-edge/visual-studio-code/index.md
:white_check_mark:Succeeded
View

microsoft-edge/visual-studio-code/microsoft-edge-devtools-extension.md
:white_check_mark:Succeeded
View

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


Display page-specific title in browser tab

Expected Behavior
When navigating to pages, there should be a page-specific title that appears in the browser tab. When someone bookmarks that page, that title will appear in the bookmark. For example, "About Us - Marmalade AI".
Actual Behavior
The browser is just displaying the default "title", which is the URL of the page. For example, "https://marmalade.ai/about".
References

Gatsby docs: Adding Page Metadata
Note: A static HTML page uses the title element:<html>
    <head>
        <title>About - Marmalade AI</title>
    </head>
    ...
</html>

React Helmet is a package that provides a React component interface to manage the data in the document head (i.e. metadata within the <head> element).

Open Issues

Should we add other metadata too? See other elements, besides <title>, that can be used inside the <head> element: : The Document Metadata (Header) element

Resolved by #68


working memorialize contract spec to new chain

doc for protobuf serialization https://github.com/cosmos/cosmos-sdk/blob/master/docs/architecture/adr-027-deterministic-protobuf-serialization.md - I am just using the default serialization for the TxRaw serialization, but it seems to be conforming to this spec (I also tried their 'Article' example and the serialization for that matched)
doc for proto tx signing https://github.com/cosmos/cosmos-sdk/blob/master/docs/architecture/adr-020-protobuf-transaction-encoding.md (we are using SIGN_MODE_DIRECT)
also fixed an issue with EventStream where now txs_results on the block_results rpc endpoint apparently can be nullable now if there are no transactions vs. an empty array...

Codecov Report

Merging #12 (b3db3b1) into main (1855d3c) will decrease coverage by 0.38%.
The diff coverage is 1.98%.

@@             Coverage Diff              @@
##               main      #12      +/-   ##
============================================
- Coverage     12.77%   12.38%   -0.39%
  Complexity       35       35
============================================
  Files            63       64       +1
  Lines          3132     3222      +90
  Branches        343      345       +2
============================================
- Hits            400      399       -1
- Misses         2678     2769      +91
  Partials         54       54

Impacted Files
Coverage Δ
Complexity Δ

...rovenance/engine/service/ChaincodeInvokeService.kt
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...provenance/engine/service/ProvenanceGrpcService.kt
0.00% <0.00%> (ø)
0.00 <0.00> (?)

.../kotlin/io/provenance/engine/stream/EventStream.kt
74.45% <25.00%> (-0.55%)
0.00 <0.00> (ø)

.../kotlin/io/provenance/engine/domain/Transaction.kt
19.23% <100.00%> (-1.93%)
0.00 <0.00> (ø)

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 1855d3c...b3db3b1. Read the comment docs.

Codecov Report

Merging #12 (d3a4046) into main (8e6ab27) will decrease coverage by 0.90%.
The diff coverage is 0.53%.

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


[Question]: How to solve the problem of garbled characters caused by pictures in the read doc

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
How to solve the problem of garbled characters caused by pictures in the read doc

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Problem with eloquent fresh method

Laravel Version: 5.5.19
PHP Version: 7.1

Description:
I have one model with global scope. For instance:
<?php

namespace App;

use Eloquent\NestedAttributes\Model;

class Facility extends Model {
    protected static function boot(): void {
        parent::boot();
        static::addGlobalScope('f1', function (\Illuminate\Database\Eloquent\Builder $builder) {
            $builder->select('facilities.*')->addSelect(\DB::raw('NOW() AS now_date'));
        });
    }
}

Now we can check in php artisan tinker:
>>> \App\Facility::get();
=> Illuminate\Database\Eloquent\Collection {#1183
     all: [
       App\Facility {#1185
         id: 1,
         name: "sss",
         description: null,
         created_at: "2017-11-22 14:58:00",
         updated_at: "2017-11-22 14:58:00",
         now_date: "2017-11-22 15:00:05",
       },
     ],
   }

>>> $f = \App\Facility::create(['name' => 'sss1']);
=> App\Facility {#1186
     name: "sss1",
     updated_at: "2017-11-22 15:00:14",
     created_at: "2017-11-22 15:00:14",
     id: 2,
   }

and fresh() method should return extra attribute now_date but it doesn't.
>>> $f->fresh()
=> App\Facility {#1176
     id: 2,
     name: "sss1",
     description: null,
     created_at: "2017-11-22 15:00:14",
     updated_at: "2017-11-22 15:00:14",
   }
>>>

Could you fix it?

This is intended:

fresh(): Reload a fresh model instance from the database

So it will load the row from the database not re-fire the query.

Maybe this is wrong way?
Now we can't pass a new row as eloquent object to Eloquent API Resources class with all data (scopes etc.).
$f = \App\Facility::create(['name' => 'sss1']);

return new FacilityResource(
  $f->fresh()
);

now in the FacilityResource we can't get any data from Facility scopes.
We can always pass new query:
$f = \App\Facility::create(['name' => 'sss1']);

return new FacilityResource(
  \App\Facility::find($f->id)
);

but now the wasRecentlyCreated attribute is always false and FacilityResource return status code 200 but should 201.

Yes, I know but I think it is a bug.
We have something like Resource class which return automatically a status code when an eloquent object is created or updated.
So if the create or fresh method can't get full data like query Model::find() then Resource class can't return full data and correct status code.
That's why I think it is a bug of the fresh method or ...
maybe we need another method like freshwhich will return all data like Model::find() with the wasRecentlyCreated flag with true value.

Fresh doesn't use scopes by default, this is by design, if you want a different behaviour you can add your own version of fresh in a base model.
To me that scope you're adding is not the best approach, using am eloquent mutator is a better approach.

Fresh doesn't use scopes by default, this is by design, if you want a different behaviour you can add your own version of fresh in a base model.

@themsaid would be swell if it could though, ran into the same limitation today.


Support COMPND record (pdb) and entity category (mmcif)

This includes (1) parsing the data from the files and (2) providing a mechanism to select parts of the structure based on it.
http://www.wwpdb.org/documentation/file-format-content/format33/sect2.html
http://mmcif.wwpdb.org/dictionaries/mmcif_pdbx_v40.dic/Categories/entity.html
Generally this kind of annotation should make it more easy for a user to select certain parts in big molecules/assemblies like the ribosome.

see #61


$bot->auth->isLoggedIn() works only if $bot->auth->login() was called

Auto login by cookie work only if i call $bot->auth->login() with particular username, password dont important, but i cannot call $bot->auth->login() without password argument

Hi @kolyalukin
I'm not sure what I've understood you correctly. So, what is the issue? How can I reproduce your problem?

Yes, sure. Feel free to provide a pull request with a new method for cookie login 👍


Surport mxnet train distributed  on Kubernetes

In Kubernetes pod, 	a virtual ip(svc ip) was used for communication. But this virtual ip can't used for bind.
So we suggest using <IP_ADDRESS> to bind scheduler port, and use the virtual ip for communication as before.
@mli

Thanks，we changed to use DMLC_USE_KUBERNETES and node.role to set schedule ip.
@mli


is this library under development yet ?

I see a lot of pull request and issues growing, But the last commit was on October 2014... (more than one year ago).
Is this library under development yet? or should I go with other ?
Is there an updated fork ?
Any update ? @mateuszmarkowski

The current version is stable and there are no planned updates.

Thanks.


Add a minimal example for the eFabless / sky130hd Caravel wrapper harness

This PR adds an example design which demonstrates how to build a minimal 'heartbeat' design into the Caravel wrapper harness used in eFabless' MPW tapeouts.
It contains a few changes which are required to pass all of the DRC/LVS checks in the MPW pre-check scripts, including some DEF post-processing and support for custom PDN scripts.
I tried to minimize changes to the tool drivers, but this 2-step build process uses TCL scripts for macro placement and PDN generation/connection, so the OpenROAD sc_floorplan.tcl script contains some minor changes to add optional support for those scripts.
In order to minimize the number of new files, there is also a CARAVEL_ROOT variable near the top of the build script. Users are expected to populate this with a filepath pointing to a clone of the Caravel repository on their disk.

I mostly followed your suggestions for the floorplan script, but I think that we should still perform the global placement step prior to reading a manual macro file, so I copied some of that logic into the 'else' block which runs if a floorplan DEF is not present.
Let me know what you think; there is a bit of duplication, and we don't currently need the case where a macro placement TCL file and a floorplan DEF are defined.

Looks good! I'm not too worried about the duplication, we could think about cleaning that up later.

we don't currently need the case where a macro placement TCL file and a floorplan DEF are defined.

Doesn't build_top() utilize both, or am I misreading something? In that case, keeping the extra block in both branches of the if-statement would be important I think (RE: your new inline comment).

Oh yeah, you're right - that's what I get for waiting a month to clean up and submit this branch.
I just double-checked that zerosoc can build with these changes, so I think they should be ready to merge.

Thanks for checking ZeroSoC, sounds good!

This is awesome to see happening! I'm excited to see if people use this to do silicon for the next SKY130 MPW run.


Allow share link to be selectable

The share link isn't selectable right now. This fixes it.
/cc @mdaxtman

:shipit:


uadk_engine: fix env poll timeout in abnormal scenarios

In abnormal scenarios, if the alg cores of hardware device are closed, the hw async task may return busy and can not finish the task. There is be a max loop count to avoid infinite loop. But there is already a 'MAX_POLL_TIMES' (1000) in uadk default poll policy. If engine use 'ENGINE_RECV_MAX_CNT' to control the poll loop count, the actual count will be:
MAX_POLL_TIMES * ENGINE_RECV_MAX_CNT
which is too large. This will cause the process use a very long time to exit. So modify the env poll loop time count.

uadk.h:26:1: error: unknown type name ‘define’
26 | define ENGINE_ENV_RECV_MAX_CNT 60000
| ^~~~~~
uadk.h:26:32: error: expected ‘=’, ‘,’, ‘;’, ‘asm’ or ‘attribute’ before numeric constant
26 | define ENGINE_ENV_RECV_MAX_CNT 60000


yarn start func start gives an error

When I tried to execute the command "yarn start func start" or "clocal-gcp func start" it gives an error like following.

I tried the command after "docker login" also. But it gave me the same error.

@rehrumesh  this was the issue regarding the missing docker image

Hey @shehand  looks like now image name is updated. Also why not you can build your own image using the Dockerfile :) then edit the image name in js file

yeah, @shalithasuranga thanks for the advice XD

I am closing this since this is fixed.


Document loader/debug

Description: Added debugging output in DirectoryLoader to identify the file being processed.
Issue: Need a trace or debug feature in Lanchain DirectoryLoader #7725
Dependencies: No additional dependencies are required.
Tag maintainer: @rlancemartin, @eyurtsev
This PR enhances the DirectoryLoader with debugging output to help diagnose issues when loading documents. This new feature does not add any dependencies and has been tested on a local machine.

thanks @rjarun8!


About input data for model inference

Hello, can I use GFS analysis data for model inference? If yes, how I can calculate toa_incident_solar_radiation parameter with its dataset?

Hi, so my recommendation in this case would be to use the pysolar library to build something as similar as possible to the toa_incident_solar_radiation, integrated for one hour up to the time of the input/target. The model should not be extremely sensitive to this parameter, and perhaps if you can get it to be below 1% difference from that of ERA5, the model is probably going to cope ok with it. You can always check against one of the dates for which ERA5 is available.
Alternatively, you may want ask ECMWF in case they already have some open source code, or can make available some code to compute that feature for any arbitrary date.
We may eventually switch to our own pysolar implementation for computing this, but it is unclear of what the timeline for open sourcing this code this would be.
Hope this helps!

Hi, so my recommendation in this case would be to use the pysolar library to build something as similar as possible to the toa_incident_solar_radiation, integrated for one hour up to the time of the input/target. The model should not be extremely sensitive to this parameter, and perhaps if you can get it to be below 1% difference from that of ERA5, the model is probably going to cope ok with it. You can always check against one of the dates for which ERA5 is available.
Alternatively, you may want ask ECMWF in case they already have some open source code, or can make available some code to compute that feature for any arbitrary date.
We may eventually switch to our own pysolar implementation for computing this, but it is unclear of what the timeline for open sourcing this code this would be.
Hope this helps!

Thank you very much! I will have a try with your suggestions.

Hi, so my recommendation in this case would be to use the pysolar library to build something as similar as possible to the toa_incident_solar_radiation

To add a comment here, I'm working on this problem myself.  pysolar is specialized for the idea of solar power deployments at the bottom of the atmosphere, so it needs some modification to give reasonable top of atmosphere radiation.
Some of the internal routines can give the solar angle at a particular location and time, then I found a Sandia National Labs page describing the extraterrestrial radiation component taking into account the eccentricity of Earth's orbit.
There's still a residual mismatch (of a fraction of a percent) with the ECMWF data, but it seems that IFS/ERA5 may have used a more complicated solar model that takes the solar cycle into account.
As an additional note about the data, toa_incident_solar_radiation is the accumulated solar radiation over the 1h leading up to the specified time.  For example, for an analysis or forcing dataset describing 2022-Jan-1 6Z, the solar radiation field would be the integrated radiation from 5Z to 6Z.  For a quarter-degree grid, integrating radiation with a 1m timestep over this period gives a smooth field without obvious stair-stepping (0.25 deg x 24h /  360 deg = 1m).  Taking an instantaneous radiation figure and multiplying by 3600s (converting W to J) will give a radiation field with the wrong shape.


Cactbot prompts to update latest, but already latest (not a huge issue)

Maybe double check to see whether your loaded CactbotOverlay.dll is pointing to the right one?

It seems very likely that the CactbotOverlay.dll that you are loading is the old one.

Is this still happening?

I've replaced the file, but it still prompts.

In the plugin listing, what version is your CactbotOverlay.dll?

Just checking back if this is still happening, and what it prints out for your CactbotOverlay.dll version in the overlayplugin log.

I'm going to close this for now.  Let me know if this is still happening.


Link from Sm Teams Managment console Learn more link arrives at this article - but article is only part of the help the link referenced.

The article is fine but I arrived at it from a learn more link inside the teams management console (Manage teams, channels, type column info (i) ) which said:
"Shows if a channel is a private, standard, or shared channel. [Learn more]" (https://docs.microsoft.com/en-US/microsoftteams/private-channels?WT.mc_id=TeamsAdminCenterCSH)
This article says nothing about standard or shared channels i.e. the help link is not helpful [I wanted to know about a shared channel for instance].

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

ID: ce34b12b-aa50-608b-bbfa-a1fb4bb79f8d
Version Independent ID: 9ddeb6f4-f30a-1e72-fc94-ae66def91f40
Content: Private channels in Microsoft Teams - Microsoft Teams
Content Source: Teams/private-channels.md
Service: msteams
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@Baldgyt thanks for the feedback. I'll pass this along to our content designers to look for a better article to link to.

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


There is a vulnerability in cron.utils 5.0.5 ,upgrade recommended

https://github.com/apache/dolphinscheduler/blob/ac15f45b0b51b115ebd02aa3dd739d53c99523b5/pom.xml#L72
CVE-2020-26238
Recommended upgrade version：9.1.3

is there any contributor who want to upgrade cron utils jar and test it?

is there any contributor who want to upgrade cron utils jar and test it?

Hello, I think I can do the job. Could you please assign it to me


compose: Show topic required error if topic is "(no topic)".

We already show the error if topic input is empty and it is
not allowed to send messages without topic in the organization,
and this commit also shows error when topic is "(no topic)".
The topic is set to "(no topic)" when someone sends a message
with empty topic input box and when it is allowed to send message
without topics in the organization.
This is not ideal behavior as we may want to treat "(no topic)"
differently from empty topic, but we can fix this in future and
this commit can be a short term fix.
Issue - #21344

GIFs or screenshots:

Looks good in manual QA -- thanks!


work around deprecations warning of apt

the install script triggers some W: --force-yes is deprecated, use one of the options starting with --allow instead. warnings. This PR is fixing this.

I wonder why it is forced at all?

Hi @wsteitz @ctietze
if I remember correctly, this was a pull request from somebody about a year ago, who might be installing these for money? It seemed he/she/they wanted to save a little extra time.


INSP-2225: Initial additions for dfp helper and responsive ads

@weddingwire/cms-crew wanna check it?


Feature/421 add color customisations for crop image activity

Add the issue linked to this PR
close #421
New Feature
Description:
Added options to customize toolbar color and activity background color of CropImageActivity
Check list for the Code Reviewer:

[x] CHANGELOG
[ ] README
[ ] Wiki
[ ] Version Number

@Canato I'm a bit rusty with the code. let me know if I need to do anything

One more, we need to add the features here https://github.com/CanHub/Android-Image-Cropper/blob/main/.documentation/features.md

One more, we need to add the features here https://github.com/CanHub/Android-Image-Cropper/blob/main/.documentation/features.md

Alright I'll update that too

I noticed the background suddenly being white. I had to set it to transparent to get the original theme color back. Why not setting it to transparent by default?


Add mdc-radio-group component

mdc-radio buttons should be placed inside of an <mdc-radio-group> unless the DOM structure would make that impossible (e.g., radio-buttons inside of table cells). The mdc-radio-group should have a value property that reflects the currently selected radio-button inside of the group.
Individual radio-buttons inside of a radio-group will inherit the name of the group.

Releasing as part of v0.39.1 bug fix release. It's the best method to resolve #1300


Add fscenario alias to focus example

This alias adds the focus: true metadata to scenarios. Most users
receive get this functionality via the Capybara gem; however, this
patch makes it available to all rspec-rails users.
I'm not sure how to test this change. Capybara uses this scenario. However, because rspec-rails has filter_run: :focus that won't work.

You can add a test for this by creating an example using the alias and checking it's metadata

Closing as I'm not likely to have time to work on this.


Publish man pages to /man/

We'd still need to link these in the index page.
I used the pages from the latest stable release from alpine/edge. This is a lot simpler/faster than cloning the sway repository and building the whole project.

Failure is due to CI trying to publish the site without permission.

Failure is due to CI trying to publish the site without permission.

Fixed in 087bb5ba0324b36a52046e0b0375d174e076acbf

Thanks for the PR! Overall looks pretty good.

(BTW, needs a rebase)

Updated and rebased

Alright, let's just go with that approach for now. We can always change later on to something else.
Here's a sed command to linkify references inside man pages, should make it easier to navigate:
sed -E 's,(sway[a-z-]*)\(([0-9])\),\1(\2),g'

Also we should probably add a new link on the homepage linking to the man pages? Should we just link to sway(1), or sway(5), or to an HTML page listing all man pages?


Wrong sub menu

[Enter feedback here]
The custom dns page should not be under the productivity score. "Monitor user productivity with Productivity score "

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

ID: 1f2e4e45-6ad7-eb79-aecb-0d7a89b06a03
Version Independent ID: 1059223e-5759-b130-41ac-5ea98cb5367e
Content: Add DNS records to connect your domain - Microsoft 365 admin
Content Source: microsoft-365/admin/get-help-with-domains/create-dns-records-at-any-dns-hosting-provider.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@limp15000 Thank you for the feedback. I will work internally with the author to get this sorted.
Thanks
Sri

@Sirkkuw We notice that the document - Microsoft Productivity score (https://docs.microsoft.com/en-us/microsoft-365/admin/productivity/productivity-score?view=o365-worldwide) has an incorrect submenu linking to this Custom DNS Page (https://docs.microsoft.com/en-us/microsoft-365/admin/get-help-with-domains/create-dns-records-at-any-dns-hosting-provider?view=o365-worldwide)
Could you please help in removing this article from the sub-menu?
CC: @pebaum @yogkumgit
Thanks
Sri

Ah yes. I will remove this. Cut and paste error...

@Sirkkuw Thank you for the update.
@yogkumgit This can be closed as the sub menu has been removed.
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @limp15000 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.