[Simplification request] Top-level existential elimination

Simplification
Existential quantification in top-level positive constraints can be eliminated and the bound variables can be replaced with some fresh variables.
Example: #Exists X. { P(X) } => P(X0), for some X0 fresh.
I am assuming that this is a sound simplification when doing matching-logic proofs since it is sound when doing first-order logic proofs, and matching logic captures first-order logic.
This simplification cannot be applied under negation.

https://github.com/runtimeverification/haskell-backend/blob/master/kore/src/Kore/Variables/Fresh.hs for refreshing variables


Update Starship maiden launch date

According to this article, SpaceX now targets 2021 as the year Starship will be first launched.
first_flight: "2021-12-01"

See #226.

@jesusrp98 updated the date, #226 doesn't apply because the rocket details don't pull from the wiki


Getting errors while pip setup.py install

python setup.py install
/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/dist.py:760: UserWarning: Usage of dash-separated 'author-email' will not be supported in future versions. Please use the underscore name 'author_email' instead
% (opt, underscore_opt)
/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/dist.py:760: UserWarning: Usage of dash-separated 'home-page' will not be supported in future versions. Please use the underscore name 'home_page' instead
% (opt, underscore_opt)
Traceback (most recent call last):
File "setup.py", line 19, in
'tqdm>=4.28.1'
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/init.py", line 87, in setup
return distutils.core.setup(**attrs)
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/_distutils/core.py", line 122, in setup
dist.parse_config_files()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/dist.py", line 851, in parse_config_files
self, self.command_options, ignore_option_errors=ignore_option_errors
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 167, in parse_configuration
meta.parse()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 446, in parse
section_parser_method(section_options)
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 417, in parse_section
self[name] = value
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 238, in setitem
value = parser(value)
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 552, in _parse_version
return expand.version(self._parse_attr(value, self.package_dir, self.root_dir))
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/setupcfg.py", line 371, in _parse_attr
package_dir.update(self.ensure_discovered.package_dir)
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/_collections_abc.py", line 720, in iter
yield from self._mapping
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/expand.py", line 458, in iter
return iter(self._target())
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/expand.py", line 448, in _target
self._value = self._obtain()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/expand.py", line 418, in _get_package_dir
self()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/config/expand.py", line 408, in call
self._dist.set_defaults(name=False)  # Skip name, we can still be parsing
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/discovery.py", line 330, in call
self._analyse_package_layout(ignore_ext_modules)
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/discovery.py", line 363, in _analyse_package_layout
or self._analyse_flat_layout()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/discovery.py", line 420, in _analyse_flat_layout
return self._analyse_flat_packages() or self._analyse_flat_modules()
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/discovery.py", line 426, in _analyse_flat_packages
self._ensure_no_accidental_inclusion(top_level, "packages")
File "/home/zeus/anaconda3/envs/vfs/lib/python3.7/site-packages/setuptools/discovery.py", line 455, in _ensure_no_accidental_inclusion
raise PackageDiscoveryError(cleandoc(msg))
setuptools.errors.PackageDiscoveryError: Multiple top-level packages discovered in a flat-layout: ['pytest', 'results', 'davis2017'].
To avoid accidental inclusion of unwanted files or directories,
setuptools will not proceed with this build.
If you are trying to create a single distribution with multiple packages
on purpose, you should not rely on automatic discovery.
Instead, consider the following options:

set up custom discovery (find directive with include or exclude)
use a src-layout
explicitly set py_modules or packages with a list of names

To find more information, look for "package discovery" on setuptools docs.

Tried running python setup.py install packages=['davis2017'] and got zsh: no matches found: packages=[davis2017]

Update: just had to modify line 20 in setup.py to be the following :
   ],packages=['davis2017'])

@andlyu Could you please inform me whether you are able to reproduce their results with Resnet18 backbone on Davis benchmark? The best accuracy that I got on my implementation of this paper was 65.36 (J&Fmean).

Yes, I was. Thank you

Confirming this solution works
https://github.com/davisvideochallenge/davis2017-evaluation/issues/11#issuecomment-1336366024


Update README.md of backup_nhg_test

Update the test plan to focus on interested scenarios:

Replace next-hop IP in the backup NHG with a routing instance.

@nflath to LGTM.
@liulk to approve.
Thanks!

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 70.423%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1050

Relevant Lines:
1491

💛  - Coveralls

LGTM


Using the RefreshDatabase trait errors with dropping multiple tables

Hi,
When attempting to run a series of tests that involve the RefreshDatabase trait, I am greeted with the following error:
Illuminate\Database\QueryException : SQLSTATE[HY000]: General error: 1706 Feature 'Dropping more than one table in a single query' is not supported by SingleStore.

This only applies within tests, as manually doing php artisan migrate:fresh doesn't cause this. However my tests all rely on refreshing.

Hey @heychazza! That's kinda confusing to me, because it looks like the RefreshDatabase trait defers to migrate:fresh under the hood. (https://github.com/laravel/framework/blob/9.x/src/Illuminate/Foundation/Testing/RefreshDatabase.php#L73)
When you're running tests, are you using the SingleStore driver? If your test DB is set up to use the MySQL driver but is pointed at a SingleStore db, you might get that error.

Hey @heychazza! That's kinda confusing to me, because it looks like the RefreshDatabase trait defers to migrate:fresh under the hood. (https://github.com/laravel/framework/blob/9.x/src/Illuminate/Foundation/Testing/RefreshDatabase.php#L73)
When you're running tests, are you using the SingleStore driver? If your test DB is set up to use the MySQL driver but is pointed at a SingleStore db, you might get that error.

Apologies in the delay, that was a good spot! Always annoying when its that pesky phpunit.xml file with the override.


Navbar

Navbar needs to be done

We need links to /products /customers as of now..
http://themes.getbootstrap.com/products/dashboard (i really like this one)

Ok I'll work on that today then. So you want a horizontal navbar on the left side?
(I can't 'assign' this task to myself)

Well at first the normal top navbar would be okay but if you are up for the navbar to the left would be good.

I'll do a top navbar first then swap it to a sidebar later on

Maybe we should consider a full css framework switch ? (maybe adminlte or similar)
https://almsaeedstudio.com/preview
@papajo @DanBonehill


better iPad layout

Pre-requisite #14 #19 - discussion still on-going, so please don't send pull requests 😘
Inspiration
Contacts app by Apple with show and detail view on the same page. For this app, the list view on the left will be the list of events / repos and the detail view on the right will contain the details.

Current look

cc @cheeaun
Not sure if a modal view with a scroll bar would look simple. Also next time, there will be a map that will get the user's location, etc etc. Better to not have modal no?

Here's my vision 😄
Two columns:

Modal:

OMG 😍 i love it!! ok then mobile how? tabbed view or long list? hhmmm

Tabbed view 👍 💥 💥 💥 💥

summary:

mobile view #20
iPad view #22


Add shell shebang in .multitask file

Adding a shell shebang at the top of the .multitask file will allow the user's editor to give correct syntax highlighting and LSP hints.

Added in f5aa11b.


Sections are hidden by the navbar when clicking on links (Safari)

When clicking on Deno.close for example on Safari:

Whereas it's working fine on Chromium:

I'm not sure where the problem comes from, maybe it comes from Vercel Links.
I haven't had much time to look into it more in depth but I'll try to spend some more time finding what the issue is and I'll try to fix it.
Feel free to give my any hint if you think what the problem might be :)

Fixed as header was removed. (At least on desktop).


Update to 0.8.11, add network-agnostic deployment script, add hardhat support, add package.json and dependency control

I believe this PR brings a number of improvements to the protocol, namely adding Hardhat and a network-agnostic deployment script that is extensible to any network. Please note some added scripts:

npm run deployOptimismKovan: deploys from the deploy folder the Arrow ERC20 to selected hardhat network.
npm run verifyOptmismKovan: verifies network deployments automatically on etherscan. Make sure to add "-- CONTRUCTORARG1, 2, etc." after the command in the terminal so the plugin knows constructor params.

Please note, Brownie CI is behaving oddly after I updated to Solidity 0.8.11 and started driving imports off of OpenZeppelin's published node packages (best practice AFAIK). I in no way wanted to break this, brining this up here for @BrassLion 's reference.
If you want @thomasgarrison, I can spin up an issue and try to port the testing suite to hardhat and add continuos integration on that front/real-time testing.
Please find the example Arrow contract on Optimism Kovan that can be deployed with the above scripts and custom parameters (edit deploy script) here.

Btw, I don't think there will be an issue w/ ProxyAdmin/ContractAdmin collision if we do it right. On the TransparentUpgradeableProxy standard this is solved using a dedicated ProxyAdmin contract and perhaps on this new UUPS implementation that OZ is using in 0.4 it might not be needed at all...
https://docs.openzeppelin.com/contracts/4.x/api/proxy#transparent-vs-uups
Please allow me to do some more research here.

Closes #8

Generally LGTM, with my limited knowledge of Hardhat. I've added a couple of minor things that could be addressed. CI should pass before we merge this.
FYI, I have a competing deployment PR here, so we should decide on the best deployment route and use one moving forward.

Thanks for your input! Very helpful. I just resolved a few of the items and am leaving the open issues unresolved. Getting this testing suite in Mocha and moving the CI and testing to Hardhat may be the next step - what do you think about that?
Once the CI is resolved and I write the verification check you asked for, this should be good to merge in :)

Generally LGTM, with my limited knowledge of Hardhat. I've added a couple of minor things that could be addressed. CI should pass before we merge this.
FYI, I have a competing deployment PR here, so we should decide on the best deployment route and use one moving forward.

Thanks for your input! Very helpful. I just resolved a few of the items and am leaving the open issues unresolved. Getting this testing suite in Mocha and moving the CI and testing to Hardhat may be the next step - what do you think about that?
Once the CI is resolved and I write the verification check you asked for, this should be good to merge in :)

Just wanted to make sure that we don't do duplicated works. I've already wrote a chai test suite (see  #12) and have created a GitHub Action for running npx hardhat test  whenever a push or pull request is made.

@bghughes Any ways I can help out on moving this along?


[Refactor][Enhancement] DataStreamRecvr::SenderQueue phase 4: introduce lazy deserialization mechanism

What type of PR is this：

[ ] bug
[ ] feature
[ ] enhancement
[x] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Introduce a lazy deserialization mechanism for PipelineSenderQueue, the chunk deserialization operation is not performed on the add_chunks interface, but is postponed to the get_chunk interface
benefit:

The latency of the transmit_chunk request will be reduced because of removing such a heavy operation, which may increase the throughput of transmit_chunk
If a short-circuit operation occurs, some unnecessary computational overhead can be avoided

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_admit_test

run starrocks_be_unittest

run starrocks_admit_test

run starrocks_admit_test


feat(Algebra/TensorAlgebra): support towers of algebras

This is pre-work towards a base-change of clifford algebras.
The main result here is
@[nolint unusedArguments]
instance instAlgebra {R A M} [CommSemiring R] [AddCommMonoid M] [CommSemiring A]
    [Algebra R A] [Module R M] [Module A M]
    [IsScalarTower R A M] :
    Algebra R (TensorAlgebra A M)

Note that strictly the IsScalarTower R A M argument isn't needed, but I'd claim the instance doesn't make any sense without it.
In order to prevent diamonds in the algebraMap fields of the Int and Nat algebra instances, we have stop having natCast  as an irreducible_def, and we have to add a missing intCast customization for RingQuot. In order to prevent diamonds in the smul fields there and elsewhere (such as a complex tensor algebra being a real algebra), we have to stop having the smul definition as an irreducible_def. We already had to make a similar refactor to prevent diamonds for Algebra R (Polynomial A).
If we backport any of this to mathlib3, we'd additionally have to change the smul definition to not use pattern matching. Thankfully, structure eta in Lean 4 makes that unnecessary.
These diamonds are tested with inline examples.

[x] depends on: #6066
[x] depends on: #6072

Thanks!
bors merge


Problem: checking if job shall be executed is inconsistent

Solution: combine all checks in a helper function
Additionally, update dependencies to latest versions.

Coverage remained the same at 100.0% when pulling 3407204bc2756165722048907115d7178eb69c6e on refactoring into d26d030fa3417bc26409ec8d3085189b0ec5d52e on master.

Coverage remained the same at 100.0% when pulling 3407204bc2756165722048907115d7178eb69c6e on refactoring into d26d030fa3417bc26409ec8d3085189b0ec5d52e on master.

Coverage remained the same at 100.0% when pulling 3407204bc2756165722048907115d7178eb69c6e on refactoring into d26d030fa3417bc26409ec8d3085189b0ec5d52e on master.

Coverage remained the same at 100.0% when pulling 3407204bc2756165722048907115d7178eb69c6e on refactoring into d26d030fa3417bc26409ec8d3085189b0ec5d52e on master.

Looks great


Fixes #22

Add update_metdata method, test and documentation.

@chrismetcalf @timwis take a look at this. Also, I've added you both as collaborators, so that either of you in the future can merge code or close issues.

I tested a couple of keys in the metadata, and for most it changed them. For some others though, it doesn't. An example of this is "category". If the user tried to set a category to one that isn't in the prefined list in the UI, then category will simply be undefined. This is a bit confusing, because the api returns a 200 response, but I guess that is a case that you guys will handle when redesigning this endpoint.
In summary, I'm just returning whatever the server returns, for better or worse.

This is great! Thanks so much for working on this. I love that the actual code for the method is only 2 lines :P
Regarding the category, I noticed that with create as well, and noted in the docs that it must exist in /admin/metadata.


Phone numbers sometimes include commas (especially in extensions)

Imported from Google Code issue #467 created by<EMAIL_ADDRESS>on 2014-05-14T23:07:08.000Z:

Parse this phone number: +1888 294-9010 ext 3,1
Desired/expected result: +1<PHONE_NUMBER> ext. 3,1
Actual result:
The number is considered invalid.
isPossibleNumberWithReason() returns TOO_LONG
Partial workaround:
Preprocess numbers to remove commas. Then this example returns: +1<PHONE_NUMBER> ext. 31
In some cases this may be sufficient. In others cases the loss of information is important.
Perhaps the biggest reason for commas in phone numbers is to indicate to a modem or other automatic dialing device the need to pause. This use case applies outside of the normal meaning of an extension, but it's very similar. Certainly the comma is not strictly part of the "phone number" needed to make a connection. But it's part of the complete information needed to get to where you need to be.
It would be a useful enhancement for libphonenumber to handle commas.

Another question: is it possible this person is trying to indicate two separate numbers, one with extension 1 and one with extension 3?
If so we could solve this as part of issue #1591

We have end users that use commas in extension to indicate wait.
At this time, due to this bug, we are only using libphonenumber in the "warning" state. So when it rejects a phone number we only show to end user a warning that it may not be valid, but still allow user to proceed and save the phone number as entered - with commas.
We are only using Java library and I have created a PR to fix this bug there.

The format it issue #1591 is very different - it includes a letter after slash. So, even with comma, two extensions would be specified something like<PHONE_NUMBER> ext: 302, 2303.
This is exactly the reason why my PR did not allow comma to be the trailing character as human can place it there in many situations.
Comma in the middle of the extension can be included for readability (when it is long)<PHONE_NUMBER> ext: 1,501,234 or when it is for the modem. In both cases it is distinct from the "multiple extensions" case by not having a space.

What do you mean for the modem?
People routinely don't add spaces where they "should" for readability. We should find examples of real numbers online and see what the comma meant in the individual cases, particularly for the "findNumbers" use-case.

For any solution: if they want to indicate wait, then we need to start allowing commas to be saved in the extension field too, not just stripped when we parse.

That is exactly what my PR did. The commas are part of the extension, they are not rejected, they are kept in the parsed out phone number. That is exactly what our end users (people) need.

So we don't keep commas as "diallable" chars, which makes me nervous keeping them in an extension. However, I notice they are indeed diallable on my phone - maybe we should be adding them there too? What about semicolon, also used to indicate waiting in dialling sequences?
Looking at RFC 3601 they aren't part of E164 but they are mentioned (as p) as part of a possible dialling sequence. We don't format extensions in E164 format anyway though.
Also: changing the code in the way you did: would this not affect the findNumbers use-case? Would need a test - we don't want 135789x345,456345 for example to have the latter extracted as part of the number if it's on a web page
We should test how more methods react to their presence - particularly any that do normalisation. Do we want to consider 1678x1,2 to match 1678x12 for instance?
[Meta-comment: the PR would need to cover JS and C++, even if you use only Java.]

My PR would not affect find number in 135789x345,456345 because of the two conditions

commas are only allowed in extension
the max extension length is 7 characters.
So 345,456345 will not match the extension regex, but I am happy to add a test to cover this case.
The shortest number allowed in the library is 3 digits long. So, we can get to fail to detect the numbers that way if the second number is a short one and there is no delimiter rather than space. I, however, would argue that a human may also not detect that the tailing triplet is a number, unless it is a well known one, like 911.
In order to get there all of those should be true:

no spaces or any other symbols between the numbers, other than comma
a very short extension (one or two digits)
a very short phone number (3 or 4 digits)

Does it makes sense to you?

I have pushed another commit adding a test for this case.

So, we have discussed and in theory we are okay allowing extensions to have commas in them. However, for consistency, we think it should also allow ";" unless you know of a reason why not?
The PR would need to:
a) be in Java, JS and C++
b) have tests demonstrating that other methods are unaffected (e.g. formatting a number from the various methods works when the extension has a comma in it, number matching works when the extensions only differ by the comma)
c) ensure that findNumbers doesn't find 135789x345,45 (note shorter than I originally proposed because of your good point about the maximum length of 7). We care about this because it is very unlikely that a number in the wild embedded in text is supposed to be interpreted with the "," here as a wait command, instead it is more likely to be in a larger context and being followed by e.g. an address (45 Some St) and the user has omitted a white space.
d) have good comments explaining why "," and ";" are allowed, including in the normalizeDiallableCharsOnly function (and friends) if we don't add them to the list of diallable chars (we can say something about them being only used in extensions)
I think the best approach is to get the PR right for Java (assign me to it when you feel you have met (b), (c) and (d) and I can review)  - also update the pending_code_changes.txt file too - and then when that looks good, you can port to JS and C++. Even if you have little experience with these languages, the porting of a small change shouldn't be too onerous.

As a status update, I have all the conditions meat except the one about the findNumbers. So far I was not able to devise a sane way to not find 135789x345,45 as a number with a pause in the extension.
I have not lost hope on it yet, though. The suggestions are welcome.

d) was not met either
For findNumbers, we have different patterns at the moment: PhoneNumberUtil.EXTN_PATTERNS_FOR_MATCHING is used to find phone numbers. At the moment, the only difference is the singleExtnSymbols, but I would change the method createExtnPattern to take in a second parameter, notably the symbols allowed inside the extension bit, instead of using CAPTURING_EXTN_DIGITS. (also that field probably needs a rename).
Note it will not permit us to parse "800 345 2343-456,554#" since CAPTURING_EXTN_DIGITS is not used before the # symbol, but I think this is okay - opinion?


Go through the existing iTX and explore what needs to be worked on

This involves:

Looking at existing issues
Evaluating their severity
Eventually get a repro working
Plan what should be tackled

Required:

https://github.com/prisma/prisma/issues/8668

Worth taking a look:

https://github.com/prisma/prisma/issues/11565
https://github.com/prisma/prisma/issues/10615
https://github.com/prisma/prisma/issues/10886
https://github.com/prisma/prisma/issues/11036

Nice to have:

https://github.com/prisma/prisma/issues/11511
https://github.com/prisma/prisma/issues/9073
https://github.com/prisma/prisma/issues/9083

Can probably be closed:

https://github.com/prisma/prisma/issues/8739
https://github.com/prisma/prisma/issues/9846
https://github.com/prisma/prisma/issues/11107

Unknown:

https://github.com/prisma/prisma/issues/8708

Required:

https://github.com/prisma/prisma/issues/7918
https://github.com/prisma/prisma/issues/6156

Unknown:

https://github.com/prisma/prisma/issues/5729


[BUG] Build 5.8.2 on NodeJS fails on extensions.test()

There is a bug in extensions test in the latest build.
const extensions = /^([^.]+$|\.(asp|aspx|cgi|htm|html|jsp|php)$)/;
in reducedLinks definition we got:
if (... && extensions.test(link.pathname)) {
}

But, it fails for example when link is like:
http://blablabla.bla/index.php?route=product/category&path=255
Even when link.pathname is just /index.php.
Example site: http://hurt-winter.pl
NodeJS v10.16.0

Resolved in latest.

Resolved in latest.


error with filter

Reference:
https://stackoverflow.com/questions/15876259/typeerror-filter-object-is-not-subscriptable?utm_medium=organic&utm_source=google_rich_qa&utm_campaign=google_rich_qa

Thanks! This is a part of the py2 to py3 migration cases.


fix: multi-architecture builds

fixes #140

ARM64 image now contains ARM64 binary:
docker pull quay.io/numaio/numaflow-go/source-simple-source:test
docker create quay.io/numaio/numaflow-go/source-simple-source:test
docker cp 47b6d52f3fb91667bc346c50a21d53006b5f84c2d8c0f13c9b81d08bdb171d89:/bin/simple-source-example .
file simple-source-example

Gives us:
simple-source-example: ELF 64-bit LSB executable, ARM aarch64, version 1 (SYSV), statically linked, Go BuildID=HAQPKt0Cbx8AC5Tf_M3K/siYBCYGh18kTcjLgTEJM/DEsZ9_DBxT1WHWosA9pg/JgAVEZP6x8i5MyjWinb6, with debug_info, not stripped

@yhl25 @BulkBeing please review.


build v0.3.0 fails at docker cp step

Describe the bug
make build fails
In fairness, make ci succeeds, but it was unexpected to have make build fail
To Reproduce

git checkout v0.3.0 (master fails in the same way, but I would certainly expect a tagged version to build)
make build (or ./.dapper --debug build)
observe the kaboom

Expected behavior
exit 0
Additional context
The full build.log: https://pastebin.com/TJa5swKR
Although I first experienced this error using docker-machine on macOS, I can reproduce it in a "traditional" Linux, too:
$ head /etc/os-release
NAME="Ubuntu"
VERSION="18.04.2 LTS (Bionic Beaver)"

$ docker version
root@ip-10-0-1-228:~# docker version
Client:
 Version:           18.06.1-ce
 API version:       1.38
 Go version:        go1.10.3
 Git commit:        e68fc7a
 Built:             Tue Aug 21 17:24:51 2018
 OS/Arch:           linux/amd64
 Experimental:      false

Server:
 Engine:
  Version:          18.06.1-ce
  API version:      1.38 (minimum version 1.12)
  Go version:       go1.10.3
  Git commit:       e68fc7a
  Built:            Tue Aug 21 17:23:15 2018
  OS/Arch:          linux/amd64
  Experimental:     false

Closing due to age.


[BUG] "Share PDF" shares file location not PDF

Version
v. 1.1.2 Build 13
What platform are you using?
iOS
OS Version
iOS 17.3.1
What happened?
When sharing a PDF it shares the file location not PDF

Relevant log output
No response
Code of Conduct

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

@jacobgonzales20 i see. Will see how i can remove that and put a title instead


2nd level pages not working, neither display in navigation

I must say awesome work done by you.Issue am having is that 2nd level and third level post are not working with AMP.
for example this one is not working
http://vettimes.dev/article/gastrointestinal-foreign-bodies-your-cat-ate-what/
while this one work well
http://vettimes.dev/behind-the-scenes-at-fitzpatrick-referrals/
Can you please help ?

Thank you for creating the issue and reporting the bug.
We are working on a major update release and we will push it in few days.
After this update, I will take a deeper look at how we can use amp on custom endpoints.
Please stay tuned as the update is just around the corner.

Am waiting for your release today :) are we on time?
On Thu, Sep 1, 2016 at 6:16 PM, Mohammed Kaludi<EMAIL_ADDRESS>wrote:

Thank you for creating the issue and reporting the bug.
We are working on a major update release and we will push it in few days.
After this update, I will take a deeper look at how we can use amp on
custom endpoints.
Please stay tuned as the update is just around the corner.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ahmedkaludi/Accelerated-Mobile-Pages/issues/65#issuecomment-244075102,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAB63J1bMmt-4JFodIbnS4Yp1hKNzBmvks5qltAngaJpZM4JyApX
.

--
Regards,
Syed Zeeshan Haider Shah
mobile:+92<PHONE_NUMBER>
email<EMAIL_ADDRESS>Please Consider the Environmental Loss Before Printing Anything.

@Zeeshan can you please post the live URL instead of vettimes.dev In order to recreate the issue

Our Tested Results :

3rd level page : http://take.ms/MgX98
2nd level page : http://take.ms/K0fFm
@Zeeshan we have tested it please save your permalinks http://take.ms/PXzi4


Unexpected end of request content during multipart/form-data upload

We're using AspNetCore 2.1.2 on production where multipart/form-data uploads are sometimes (~1% out all the upload requests) failing with following exception:
Unexpected end of request content.

Exception: Microsoft.AspNetCore.Server.Kestrel.Core.BadHttpRequestException

source: Microsoft.AspNetCore.Server.Kestrel.Core
target: Throw
module: Microsoft.AspNetCore.Server.Kestrel.Core.dll

   at Microsoft.AspNetCore.Server.Kestrel.Core.BadHttpRequestException.Throw(RequestRejectionReason reason)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.Http1MessageBody.PumpAsync()
   at System.IO.Pipelines.PipeCompletion.ThrowLatchedException()
   at System.IO.Pipelines.Pipe.GetReadResult(ReadResult& result)
   at System.IO.Pipelines.Pipe.GetReadAsyncResult()
   at System.IO.Pipelines.Pipe.DefaultPipeReader.GetResult(Int16 token)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.MessageBody.ReadAsync(Memory`1 buffer, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpRequestStream.ReadAsyncInternal(Memory`1 buffer, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.WebUtilities.BufferedReadStream.EnsureBufferedAsync(Int32 minCount, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.WebUtilities.MultipartReaderStream.ReadAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.WebUtilities.FileBufferingReadStream.ReadAsync(Byte[] buffer, Int32 offset, Int32 count, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.WebUtilities.StreamHelperExtensions.DrainAsync(Stream stream, ArrayPool`1 bytePool, Nullable`1 limit, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Http.Features.FormFeature.InnerReadFormAsync(CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Mvc.ModelBinding.FormValueProviderFactory.AddValueProviderAsync(ValueProviderFactoryContext context)
   at Microsoft.AspNetCore.Mvc.ModelBinding.CompositeValueProvider.CreateAsync(ActionContext actionContext, IList`1 factories)
   at Microsoft.AspNetCore.Mvc.ModelBinding.CompositeValueProvider.CreateAsync(ControllerContext controllerContext)
   at Microsoft.AspNetCore.Mvc.Internal.ControllerBinderDelegateProvider.<>c__DisplayClass0_0.<<CreateBinderDelegate>g__Bind|0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.InvokeInnerFilterAsync()
   at Microsoft.AspNetCore.Mvc.Internal.ResourceInvoker.InvokeNextExceptionFilterAsync():

{"method":"POST","path":"/api/v1/file","query":"","accept":"application/json, text/plain, */*","userAgent":"Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) LoopEmail/4.2.0-beta2 Chrome/59.0.3071.115 Electron/1.8.3 Safari/537.36"}: {"received":2503398,"checkpoint-rendered":0}
ENDRQ: HTTP 500 /api/v1/file
size=0

Checkpoints:
    (start)
0ms:
    rendered
125782ms:
    awaited
0ms:
    (log)

at: LogMiddleware.Invoke:210

Also unable to reproduce it the lab using tight upload loops with both small and large test files.

That's the expected error when the client disconnects before finishing the upload.

Ok, then I suppose we'll just update logging on our side to treat it as such, thanks.


Printing execution plan for merge operation with python API

Hi, is there a way to see the execution plan at this level?
e.g. deltaTable.alias('t1').merge(update_df.alias(t2), 't1.id = t2.id').explain()
I see the example from Databricks was by creating temp view, and executed from SQL, as in spark.sql('EXPLAIN ...')

Yep. This sounds a useful API.

@zsxwing would something like this work:
  def explain(): Unit = {
    val sparkSession = targetTable.toDF.sparkSession
    val mergeIntoCommand = getMergeIntoCommand(sparkSession)

    // only simple is needed to describe the merge into command
    val explain = ExplainCommand(mergeIntoCommand, mode = ExplainMode.fromString("simple"))
    val plan = SparkSession.active.sessionState.executePlan(explain).executedPlan.executeCollect()
      .map(_.getString(0))
      .mkString("\n")
    // scalastyle:off println
    println(plan)
    // scalastyle:on println
  }

^ based off of spark dataset.explain()...
where the output looks like the following:
== Physical Plan ==
Execute MergeIntoCommand
   +- MergeIntoCommand Project [_1#468 AS key2#473, _2#469 AS value2#474], Relation [key1#477,value1#478] parquet, Delta[version=0, ... :/private/var/folders/gy/dy_2wchs6wz7223r9nzckf980000gq/T/spark-5bd85319-3f6e-491e-9c6b-2cd52ce9481f], (key1#477 = key2#473), [Update [actions: [`key1` = key2#473, `value1` = value2#474]]], [Insert [actions: [`key1` = key2#473, `value1` = value2#474]]], StructType(StructField(key1,IntegerType,true), StructField(value1,IntegerType,true))

Would this be sufficient for explain command?


Reorganise module structure to use semantic imports

Depends on #34. To view just the changes introduced in this PR, use this link: https://github.com/thoughtbot/react-native-template/pull/36/files/f4b2ded513f3adb582e3e0f6f8d4af7d0372615f..f65db2837ada7c7c3e41392effc9de0f962981a3.
This change uses a combination of TypeScript's flexible paths compiler option (for compile time import checking) and babel-plugin-module-resolver (for runtime module resolution) to provide semantic imports throughout the project:
-import { Welcome } from "../app/screens";
+import { Welcome } from "screens";

-import * as counterActions from "app/redux/modules/counter/actions";
+import { actions } from "modules/counter";

It also more consistently uses index files to hide details about project structure from clients who just want a semantic import.

 Reorganise module structure to use semantic imports


Fix proposal for cordova/vue.js/webpack 'T.buffer is undefined' error

Issue #172 fix proposal

Thanks! I'll ask the team to review.

The problem with the above fix is that it won't work on nodejs since module will not be undefined and thus it will try to execute res.blob(), which causes an error on node since it should use res.buffer(). The below solution solves the issue and allows the same code to run on either the browser or nodejs. I tested this on Browserify (which had the same T.buffer error) and it worked, but have not tested with webpack.
return (typeof WorkerGlobalScope !== 'undefined' && self instanceof WorkerGlobalScope) || (typeof module === 'undefined' || typeof window !== 'undefined');

Hi, Indeed, sorry, I am not familiar with node.js . I can confirm that it worked with webpack however. Do you want me to modify my pull request with your fix, or you integrate it directly ?

Thanks! Would be good for the SDK team to incorporate some kind of fix so we can write browser and nodejs code that can also be deployed throuth Webpack or Browserify. There are various ways to detect if running in Webpack or Browserify.  Below is an interesting read which states 'Both Browserify and Webpack define a process.browser field which returns true, whereas in Node it’s falsy.' However, the downside is that since the file contains reference to the Node built-in process module, both Browserify and Webpack will automatically include the polyfill for the entire module in the bundle, which increases its size. The typeof window is a quick fix that doesn't require extra bundling.
https://nolanlawson.com/2017/01/09/how-to-write-a-javascript-package-for-both-node-and-the-browser/

Thanks for the additional information!

can this be merged?  After digging into the minified dist js, i added this and it fixed my issue as well.

Thanks! I can't offer a timeline right now, but this is open with the team.
cc/ @pran1990

thanks for the effort. just throwing in that this issue is a blocker for us to upgrade to the latest version as were using Webpack. would be great if this can be merged sometime soon

Sorry, I'll be looking at this PR today!
On Tue, Apr 10, 2018, 1:49 AM Yoav Niran<EMAIL_ADDRESS>wrote:

thanks for the effort. just throwing in that this issue is a blocker for
us to upgrade to the latest version as were using Webpack. would be great
if this can be merged sometime soon
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dropbox/dropbox-sdk-js/pull/173#issuecomment-380023780,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE2ScHpokkqdXfNGvUxYujj1AuLgpX7xks5tnHH7gaJpZM4R1PTz
.

big blocker here too

Thanks for the effort and patience @yoavniran @shaun-sweet @mkirkland4874 @nicoduj  !

thanks @pran1990 !

+1 for merging this! I was completely stuck figuring out why filesDownloaded() was unusable in the browser with Webpack but this fixes it. Using webpack-replace loader as a temporary way to hack in the fix for now until it's up on NPM

Thanks all! For reference, this is released now in the latest version.


tensorflow.lite.python.convert.ConverterError: :0: error: loc("Func/StatefulPartitionedCall/input/_0"): requires all operands and results to have compatible element types

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
Tensorflow version: 2.3.0
TensorFlow installed from (source or binary): Binary (Anaconda)

Command used to run the converter or code if you’re using the Python API
converter = tf.lite.TFLiteConverter.from_saved_model(saved_model_dir,signature_keys=['serving_default'])
converter.optimizations = [tf.lite.Optimize.DEFAULT]
converter.experimental_new_converter = True
converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]
quantized_model = converter.convert()

The output from the converter invocation
2020-08-05 14:53:54.038060: I tensorflow/stream_executor/platform/default/d
so_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-08-05 14:53:56.073223: I tensorflow/stream_executor/platform/default/d
so_loader.cc:48] Successfully opened dynamic library libcuda.so.1
2020-08-05 14:53:58.130905: I tensorflow/stream_executor/cuda/cuda_gpu_exec
utor.cc:982] successful NUMA node read from SysFS had negative value (-1),
but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.131826: I tensorflow/core/common_runtime/gpu/gpu_device
.cc:1716] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla K80 computeCapability: 3.7
coreClock: 0.8235GHz coreCount: 13 deviceMemorySize: 11.17GiB deviceMemoryB
andwidth: 223.96GiB/s
2020-08-05 14:53:58.131898: I tensorflow/stream_executor/platform/default/d
so_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-08-05 14:53:58.134264: I tensorflow/stream_executor/platform/default/d
so_loader.cc:48] Successfully opened dynamic library libcublas.so.10
2020-08-05 14:53:58.136456: I tensorflow/stream_executor/platform/default/d
so_loader.cc:48] Successfully opened dynamic library libcufft.so.10
2020-08-05 14:53:58.136904: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcurand.so.10
2020-08-05 14:53:58.139311: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusolver.so.10
2020-08-05 14:53:58.140510: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusparse.so.10
2020-08-05 14:53:58.145672: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudnn.so.7
2020-08-05 14:53:58.145878: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.146748: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.147536: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0
2020-08-05 14:53:58.147999: I tensorflow/core/platform/cpu_feature_guard.cc:142] This TensorFlow binary is optimized with oneAPI Deep Neural Network Library (oneDNN)to use the following CPU instructions in performance-critical operations:  AVX2 FMA
To enable them in other operations, rebuild TensorFlow with the appropriate compiler flags.
2020-08-05 14:53:58.158009: I tensorflow/core/platform/profile_utils/cpu_utils.cc:104] CPU Frequency:<PHONE_NUMBER> Hz
2020-08-05 14:53:58.158281: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x558637af4d10 initialized for platform Host (this does not guarantee that XLA will be used). Devices:
2020-08-05 14:53:58.158342: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Host, Default Version
2020-08-05 14:53:58.207301: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.208313: I tensorflow/compiler/xla/service/service.cc:168] XLA service 0x558637b08a60 initialized for platform CUDA (this does not guarantee that XLA will be used). Devices:
2020-08-05 14:53:58.208367: I tensorflow/compiler/xla/service/service.cc:176]   StreamExecutor device (0): Tesla K80, Compute Capability 3.7
2020-08-05 14:53:58.208657: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.209465: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla K80 computeCapability: 3.7
coreClock: 0.8235GHz coreCount: 13 deviceMemorySize: 11.17GiB deviceMemoryBandwidth: 223.96GiB/s
2020-08-05 14:53:58.209534: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-08-05 14:53:58.209670: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcublas.so.10
2020-08-05 14:53:58.209798: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcufft.so.10
2020-08-05 14:53:58.209882: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcurand.so.10
2020-08-05 14:53:58.209990: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusolver.so.10
2020-08-05 14:53:58.210115: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusparse.so.10
2020-08-05 14:53:58.210180: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudnn.so.7
2020-08-05 14:53:58.210338: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.211290: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.212074: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0
2020-08-05 14:53:58.212147: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-08-05 14:53:58.765400: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1257] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-08-05 14:53:58.765463: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1263]      0
2020-08-05 14:53:58.765635: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1276] 0:   N
2020-08-05 14:53:58.766139: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.767126: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:53:58.767958: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 10618 MB memory) -> physical GPU (device: 0, name: Tesla K80, pci bus id: 0000:00:04.0, compute capability: 3.7)
2020-08-05 14:54:09.899095: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.899586: I tensorflow/core/grappler/devices.cc:69] Number of eligible GPUs (core count >= 8, compute capability >= 0.0): 1
2020-08-05 14:54:09.899811: I tensorflow/core/grappler/clusters/single_machine.cc:356] Starting new session
2020-08-05 14:54:09.900545: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.900959: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1716] Found device 0 with properties:
pciBusID: 0000:00:04.0 name: Tesla K80 computeCapability: 3.7
coreClock: 0.8235GHz coreCount: 13 deviceMemorySize: 11.17GiB deviceMemoryBandwidth: 223.96GiB/s
2020-08-05 14:54:09.901027: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudart.so.10.1
2020-08-05 14:54:09.901113: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcublas.so.10
2020-08-05 14:54:09.901170: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcufft.so.10
2020-08-05 14:54:09.901247: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcurand.so.10
2020-08-05 14:54:09.901349: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusolver.so.10
2020-08-05 14:54:09.901427: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcusparse.so.10
2020-08-05 14:54:09.901501: I tensorflow/stream_executor/platform/default/dso_loader.cc:48] Successfully opened dynamic library libcudnn.so.7
2020-08-05 14:54:09.901648: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.902096: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.902468: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1858] Adding visible gpu devices: 0
2020-08-05 14:54:09.902524: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1257] Device interconnect StreamExecutor with strength 1 edge matrix:
2020-08-05 14:54:09.902550: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1263]      0
2020-08-05 14:54:09.902571: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1276] 0:   N
2020-08-05 14:54:09.902728: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.903208: I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:982] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
2020-08-05 14:54:09.903590: I tensorflow/core/common_runtime/gpu/gpu_device.cc:1402] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 10618 MB memory) -> physical GPU (device: 0, name: Tesla K80, pci bus id: 0000:00:04.0, compute capability: 3.7)
2020-08-05 14:54:10.067218: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:816] Optimization results for grappler item: graph_to_optimize
2020-08-05 14:54:10.067284: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:818]   function_optimizer: Graph size after: 2978 nodes (2644), 3263 edges (2922), time = 94.971ms.
2020-08-05 14:54:10.067309: I tensorflow/core/grappler/optimizers/meta_optimizer.cc:818]   function_optimizer: function_optimizer did nothing. time = 1.906ms.
2020-08-05 14:54:13.144913: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:313] Ignored output_format.
2020-08-05 14:54:13.144989: W tensorflow/compiler/mlir/lite/python/tf_tfl_flatbuffer_helpers.cc:316] Ignored drop_control_dependency.
loc("Func/StatefulPartitionedCall/input/_0"): error: requires all operands and results to have compatible element types
Traceback (most recent call last):
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/convert.py", line 196, in toco_convert_protos
    model_str = wrap_toco.wrapped_toco_convert(model_flags_str,
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/wrap_toco.py", line 32, in wrapped_toco_convert
    return _pywrap_toco_api.TocoConvert(
Exception: <unknown>:0: error: loc("Func/StatefulPartitionedCall/input/_0"): requires all operands and results to have compatible element types
<unknown>:0: note: loc("Func/StatefulPartitionedCall/input/_0"): see current operation: %1 = "tf.Identity"(%arg0) {device = ""} : (tensor<1x?x?x3x!tf.quint8>) -> tensor<1x?x?x3xui8>

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "scratch.py", line 8, in <module>
    quantized_model = converter.convert()
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/lite.py", line 1076, in convert
    return super(TFLiteConverterV2, self).convert()
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/lite.py", line 899, in convert
    return super(TFLiteFrozenGraphConverterV2,
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/lite.py", line 629, in convert
    result = _toco_convert_impl(
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/convert.py", line 569, in toco_convert_impl
    data = toco_convert_protos(
  File "/opt/conda/envs/tf-detect/lib/python3.8/site-packages/tensorflow/lite/python/convert.py", line 202, in toco_convert_protos
    raise ConverterError(str(e))
tensorflow.lite.python.convert.ConverterError: <unknown>:0: error: loc("Func/StatefulPartitionedCall/input/_0"): requires all operands and results to have compatible element types
<unknown>:0: note: loc("Func/StatefulPartitionedCall/input/_0"): see current operation: %1 = "tf.Identity"(%arg0) {device = ""} : (tensor<1x?x?x3x!tf.quint8>) -> tensor<1x?x?x3xui8>

Also, please include a link to the saved model or GraphDef
model.zip
Any other info / logs
Model architecture is SSD-Mobilenet V2 with input size of 96x96, created by Object Detection API
Model exported according to instructions

The fix from https://github.com/tensorflow/tensorflow/issues/41877 did not resolve this

+1

@dtch1997
Please provide complete code to replicate the issue faced or if possible share a colab gist to replicate the issue reported.

@dtch1997 @korabelnikov can you install tf-nightly and try again? And also update the input shape?
Specifically this solution posted here: https://github.com/tensorflow/tensorflow/issues/42114#issuecomment-671593386

Thanks! It worked after I installed tf-nightly.

Issue still exists for TF version 2.3.0 and EfficientDet model. Is anyone facing the same problem?

@smohan10 We are tracking TFLite support for these models here. SSD-only for now, EfficientDet is work-in-progress.

For me just adding following line from this comment fixed the issue.
   converter.target_spec.supported_ops = [tf.lite.OpsSet.TFLITE_BUILTINS, tf.lite.OpsSet.SELECT_TF_OPS]
I am using tensorflow~=2.4.1


Drop-down list with media players at start

It would be great if a user can choose media players when starting the application.
Also it would be doubly great if Syncplay could automatically find media players (e.g. using standard paths or using registry).

Normally Syncplay already does search for compatible media players in default paths.
Could you describe in details the situation were it apparently doesn't work? In particular:

What operating system you are using and what version of Syncplay.
Which players you have installed and in what paths.
What exactly was the process in which you started the Syncplay and concluded that it doesn't have this feature? This might seem like a silly question, but it's important for us to know in what exact way the user interface has misled you. We cannot really make such observations ourselves given that we know the software inside and out, so a new perspective is invaluable.

Windows 7 x64; Syncplay 1.6.7 (release 94)
a) C:\Program Files\MPC-HC\mpc-hc64.exe
b) C:\Program Files (x86)\VideoLAN\VLC\vlc.exe
с) C:\Program Files (x86)\K-Lite Codec Pack\MPC-HC64\mpc-hc64.exe  (friend's)
It was Start menu, i.e. explorer.exe I guess.

Seems Syncplay saves previous path if I changed it. So yes, I can confirm that there is drop-down list. But I'm not sure how to check now what was at the beginning.

I think that as per https://syncplay.pl/guide/client/ you can run Syncplay with the --clear-gui-data switch to reset this data.
According to https://github.com/Syncplay/syncplay/blob/master/syncplay/constants.py the first two paths you specified should be listed by default. For K-Lite we didn't have the 64-bit version so I'll add that

It was Start menu, i.e. explorer.exe I guess.

@MarisKori sorry, I guess I didn't state my question clearly and simply :) I just wanted to know what you, as an end user, see in the Syncplay window, what you clicked, what you checked etc. In essence I would like to get a perspective of why you thought that Syncplay doesn't have a list of existing players. Such description could also help us in actually reproducing the bug (as it does look like one).
Like at @Et0h mentioned - the dropdown should be there and it should have shown first two players you had. Possibly including additional custom path you set in the past.

Friend is sure that there was no drop-down list.
I guess that I personally could miss it but I remember too that there was no drop-down list.
I can check it again and even make a video if you tell me how to make Syncplay forget previously chosen paths. I think it's smart desision to save all typed paths but now I can't help you to test it.
Btw the friend (a girl) chose unins000.exe by mistake and Syncplay saved it's path. It's a kind of issue.

@MarisKori like Et0h mentioned, you can use command line option --clear-gui-data to reset whatever is in that dropdown menu. I think on Windows the easiest way to do that is to make a temporary shortcut to Syncplay executable and add that option to "Target" field.
It would be great if you also managed to test latest development version of Syncplay after the change Et0h made - it should detect all the paths you listed earlier. You can download it here.

I checked 1.6.7 and 1.6.8 versions (reinstalled it). There is drop-down list in both versions. Sorry for the wrong suggestion.
Btw I found syncplay.ini directly in C:\Users\user\AppData\Roaming.
I think there should be a subfolder like "C:\Users\user\AppData\Roaming\Syncplay" or "C:\Users\user\AppData\Roaming\TeamName". But it doesn't really matter.


Cherry-pick recent migrator/IDE changes into swift-4.0-branch.

@swift-ci please test

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 01c8ae12702f9a68b6e4e6cdafd6fd960bc3281a
Test requested by - @nkcsgexi

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 01c8ae12702f9a68b6e4e6cdafd6fd960bc3281a
Test requested by - @nkcsgexi

@swift-ci please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 1bba86b7278229ac2492e788c3cd8ad1ff650ce7
Test requested by - @nkcsgexi

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - 1bba86b7278229ac2492e788c3cd8ad1ff650ce7
Test requested by - @nkcsgexi

Time out. @swift-ci please test linux platform


[SMALLFIX] Clean up client connect messages

Mode did not provide any additional and useful information.

Merged build finished. Test PASSed.

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


Name uploaded files with its hash

Currently uploaded files are stored with a random UUID as its file name.
If we change this so that file names are a hash of its content, we can reduce duplication by checking if a file hash exists as a file already at upload.
This can be done for uploaded music and artwork.

This a way better soloution for solving duplication to what i had in mind reduces processing time aswell as hashes will only be computed once.
This feature would be easier to implement after #28 is completed.

After updating reworking the file uploading to include user folders i realised it would be alot easier to implement this feature before #28

Completed Thank you for youre suggestion


Secured rpc

Hi.
I can implemented topic class with "SecuredTopicInterface" and secure() method.
In the case of Rpc, do you propose doing validation in each method?
thanks

You would need to implement some kind of ACL system in your RPC classes, there isn't a SecuredTopicInterface counterpart for RPC.  RPC classes/methods are really kind of like controllers when comparing to the HTTP request/response model, and since the websocket system isn't running with a notion of Symfony's firewalls in place, your best bet is to do ACL checks in each method.
Personally, the way I handle it ends up looking something like this (which uses the roles that can be set onto user accounts through the admin interface):
use Gos\Bundle\WebSocketBundle\Client\ClientManipulatorInterface;
use Gos\Bundle\WebSocketBundle\Router\WampRequest;
use Gos\Bundle\WebSocketBundle\RPC\RpcInterface;
use Ratchet\ConnectionInterface;
use Symfony\Component\Security\Core\Authorization\AccessDecisionManagerInterface;
use Symfony\Component\Security\Core\Exception\AccessDeniedException;

class SecuredRpc implements RpcInterface
{
    private ClientManipulatorInterface $clientManipulator;
    private AccessDecisionManagerInterface $accessDecisionManager;

    public function __construct(
        ClientManipulatorInterface $clientManipulator,
        AccessDecisionManagerInterface $accessDecisionManager
    ) {
        $this->clientManipulator     = $clientManipulator;
        $this->accessDecisionManager = $accessDecisionManager;
    }

    public function getName(): string
    {
        return 'secured';
    }

    /**
     * @throws AccessDeniedException if the user is not allowed access to this event
     */
    public function privilegedAction(ConnectionInterface $connection, WampRequest $request, array $params): array
    {
        $token = $this->clientManipulator->getClient($connection);

        if (!$this->accessDecisionManager->decide($token, ['ROLE_ELEVATED'])) {
            throw new AccessDeniedException('You are not authorized to perform this action.');
        }

        // Do stuff

        return [];
    }
}

Thank you


Cleaning up "TOC Contributors" list

Please see https://github.com/cncf/toc/issues/867 for context
Signed-off-by: Davanum Srinivas<EMAIL_ADDRESS>

As part of this, I assume I should remove the "TOC Contributors" section from this page?

@cjyabraham yes please!

Done

thanks @cjyabraham !

Let's land this now @cjyabraham
cc @amye @caniszczyk


Kubernetes Horizontal Pod Autoscaler (kubectl get hpa) fails to get CPU utilization

Rancher Version:
1.2.0-pre1
Docker Version:
1.11
OS and where are the hosts located? (cloud, bare metal, etc):
Azure
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
single node Rancher with local file volume
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Kubernetes
Steps to Reproduce:

Create a brand new Kubernetes environment
Update the Kubernetes stack to 1.3 via the Stack upgrade button
Follow the steps here: http://blog.kubernetes.io/2016/07/autoscaling-in-kubernetes.html:

>kubectl run php-apache --image=gcr.io/google_containers/hpa-example --requests=cpu=500m,memory=500M --expose --port=80
service "php-apache" created
deployment "php-apache" created

> kubectl get deployment
NAME           DESIRED   CURRENT   UP-TO-DATE   AVAILABLE   AGE
php-apache     1         1         1            1           23m

>kubectl autoscale deployment php-apache --cpu-percent=50 --min=1 --max=10
deployment "php-apache" autoscaled

> kubectl get hpa

Results:
CPU utilization doesn't appear:
NAME         REFERENCE               TARGET    CURRENT     MINPODS   MAXPODS   AGE
php-apache   Deployment/php-apache   50%       <waiting>   1         10        13m

Expected:
CPU utilization should appear:
NAME         REFERENCE               TARGET    CURRENT     MINPODS   MAXPODS   AGE
php-apache   Deployment/php-apache   50%       0%   1         10        13m

You have to install Heapster in order to get Pod Autoscaling to work. In order to get Heapster to work with current Rancher Kubernetees (1.3.0-rancher3) I suggest you to deploy my version of Heapster: https://github.com/bonifaido/heapster
git clone https://github.com/bonifaido/heapster.git
cd heapster
kubectl apply -f deploy/kube-config/standalone/

Thanks for the tip, will try it out

Instructions updated, sorry I missed out a step.

Closing issue as it seems like question answered.


Script works fine, but on iOS I get this error when invoking any method

ERROR: ERROR ReferenceError: Can't find variable: LxVolumeManager
SAMPLE CODE:
muteButton() {
    this.volume.mute();
}

unmuteButton() {
    this.volume.unmute();
}

volumeUp() {
    this.volume.volumeUp();
}

volumeDown() {
    this.volume.setVolume(0.5);
}

Sorry this has been 2 years since you posted this issue. I believe (without looking yet) that the LxVolumeManager was through a CocoaPod that is required for the plugin to work. I'll check.

I did confirm LxVolumeManager is a CocoaPod that was used in this project. I had initially used another but from my recollection I found the LxVolumeManager to be more popular or suited my needs better, or possibly both.


Quick summary card

User story
As a policy-maker, I want to get a quick overview of the projections for my selected nutrient, so I can decide whether I want to explore those projections in more details
Summary
Provide a quick summary of projections
Recent changes (1-27-21)

[ ] Clarify the Estimate heading - Estimate (2010). The reason Sherwin suggests 2010 is that all the scenarios will have that same historic base then, and we can base it back to underlying data from FAO.

[ ] Remove the option to switch between person, household and nation. It will always be just Person

[ ] Default selection should be mg per person per day. It is the most used indicator.

[ ] Added mobile layout mockup

Component

Location
Explore projections tab

Details

[ ] Show the current estimate which is based on the SSP2 (middle of the road) scenario (it changes based on measuring unit selected)
[ ] Show the target - recommended nutrient intake (NRI) for this nutrient (it changes based on measuring unit selected)
[ ] Show the difference between estimate and target  (it changes based on measuring unit and time period selected)
[ ] You can change measuring units and it updates the figures displayed
[ ] You can change the time period and it displays the figures displayed
[ ] Show the year when the 'business as usual' will reach the target
[ ] Hovering question near target date, you get more details about this figure

[ ] The card is responsive

Resources

Target number explanation text

Using moderate assumptions of economic and population growth, and no interventions (scenario SSP2), per capita consumption of [micronutrient name] micronutrient is expected to reach recommended levels by 2021.

Acceptance criteria

[ ] People using the maps app for the first time can get a quick overview of the projections and decide whether they want to explore them in more detail

Sprint Ready Checklist

[x] Acceptance criteria defined

[ ] Team understands acceptance criteria

[ ] Team has defined solution / steps to satisfy acceptance criteria

[ ] Acceptance criteria is verifiable / testable

[ ] External / 3rd Party dependencies identified

Is there API/data for this available?


Forward to Host header

This is a question. I'd like someone to help me write a configuration that would forward requests to hosts based on Host header (reverse proxy). We could test this from localhost with this command:
$ curl -H 'Host: target.com' http://localhost:5000

Here is my setup:

I use Docker for Mac (local development setup).
I deployed two echo http servers echo1 and echo2 (returns just Hello world 1/2).
I also created a custom network where containers can communicate with each other and I attached echo1 and echo2 to the new network.

$ docker network create proxytest
$ docker network connect proxytest echo1
$ docker network connect proxytest echo2
$ docker exec -it echo1 /bin/sh -c 'curl http://echo2:8080'
$ docker exec -it echo2 /bin/sh -c 'curl http://echo1:8080'

Now I need to deploy an envoy that would be exposed to localhost at some port (e.g. 5000) and would route traffic to these two echo servers based on Host header. The configuration must not be static so Envoy should dynamically route based on request's Host: header. The effect would be the same as in K8S setup where this already works.
$ curl -H 'Host: echo1' http://localhost:5000
$ curl -H 'Host: echo2' http://localhost:5000

What I currently have is this configuration:
static_resources:
  listeners:
  - name: listener_0
    address:
      socket_address:
        protocol: TCP
        address: <IP_ADDRESS>
        port_value: 5000
    filter_chains:
    - filters:
      - name: envoy.http_connection_manager
        typed_config:
          "@type": type.googleapis.com/envoy.config.filter.network.http_connection_manager.v2.HttpConnectionManager
          stat_prefix: ingress_http
          route_config:
            name: local_route
            virtual_hosts:
            - name: local_service
              domains: ["*"]
              routes:
              - match:
                  prefix: "/"
                route:
                  cluster: dynamic_forward_proxy_cluster
          http_filters:
          - name: envoy.filters.http.dynamic_forward_proxy
            typed_config:
              "@type": type.googleapis.com/envoy.config.filter.http.dynamic_forward_proxy.v2alpha.FilterConfig
              dns_cache_config:
                name: dynamic_forward_proxy_cache_config
                dns_lookup_family: V4_ONLY
          - name: envoy.router
  clusters:
  - name: dynamic_forward_proxy_cluster
    connect_timeout: 1s
    lb_policy: CLUSTER_PROVIDED
    cluster_type:
      name: envoy.clusters.dynamic_forward_proxy
      typed_config:
        "@type": type.googleapis.com/envoy.config.cluster.dynamic_forward_proxy.v2alpha.ClusterConfig
        dns_cache_config:
          name: dynamic_forward_proxy_cache_config
          dns_lookup_family: V4_ONLY

I deployed this to Docker and connected it with my custom network. The curl command works for public domains (e.g. google.com), but for local names throws no healthy upstream (e.g. echo1).
Thanks for all the hints.

I've figured this out. For envoy configuration above to work, the container must be deployed to Docker with --network proxytest.
$ docker run -d --name proxyapp --network proxytest -p 10000:10000 proxyapp

The configuration won't work if you try to deploy the container in two steps:
$ docker run -d --name proxyapp -p 10000:10000 proxyapp
$ docker network connect proxytest proxyapp2

Why is that so?


Completed

I was curious why it says that everything has been started, but not completed. Is there something I should be doing to mark lessons complete?
Thanks,
This is an awesome site. I'm so grateful that you put this together.

Try clicking the boxes that are started to change them to complete

Duh, thanks that worked.


Re-export the ReferenceObject type of popper.js with the Popper component

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Summary 💡

Should be able to import the ReferenceObject interface from material-ui instead of popper.js.
Examples 🌈

Current way:
import { ReferenceObject } from 'popper.js'
import { Popper } from '@material-ui/core'

export default function ExamplePopperWrapper({
  anchorEl
}: {
  anchorEl: ReferenceObject
}) {
  return <Popper anchorEl={anchorEl} />
}

Proposed way:
import { Popper, PopperReferenceObject } from '@material-ui/core'

export default function ExamplePopperWrapper({
  anchorEl
}: {
  anchorEl: PopperReferenceObject
}) {
  return <Popper anchorEl={anchorEl} />
}

I don't know where it should be exported from, but somewhere in the material-ui core package. Otherwise, the abstraction is broken.
Motivation 🔦

Typing my code properly without breaking abstraction or adding dependencies that are not directly used.
Thanks :)

Would NonNullable<PopperProps['anchorEl']> work as well?

@eps1lon I'm not sure what you mean, sorry

Something like
import Popper, { PopperProps } from '@material-ui/core/Popper'

export default function ExamplePopperWrapper({
  anchorEl
}: {
  anchorEl: NonNullable<PopperProps['anchorEl']>
}) {
  return <Popper anchorEl={anchorEl} />
}

@eps1lon's approach sounds perfect, @DaniGuardiola let us know if you face any limitations with it.

@eps1lon perfect, I haven't had the chance to use it yet (been rewriting most of my project), but I'll let you know if I face any limitations. Looks good though. Thank you! :)


Keyboard combination like shift + button

It would be great to add case sensitivity or special characters to your service, for instance. Thank you.

I think this isn't what the service should do. You see, repo is called "keycodes" cause it brings to you the keyCode property, very useful for web developers in some situations, However, you're asking for character codes (case sensitive, special chars and so on). According to W3C (http://www.w3schools.com/jsref/event_key_keycode.asp):

The difference between the two code types:

Character codes - A number which represents an ASCII character
Key codes - A number which represents an actual key on the keyboard

It's more useful and (I should say) ordinary for devs to know which key is being pressed no matter what modifier is being pressed too. For example: when you're pressing W key in your keyboard, I do not care what else is being pressed, cause W key will be always... the W key! When a developer wants to know what text was (or should be) printed after key down, there are other properties and a lot of algorithms.

This would be nice to have - going to figure out regional keyboard support first though.

dupe of another issue -will come :)


Ability to choose the encoding of the database

Thanks for the great addition! I will run a quick check on it and then merge it.

Thanks again @eoger !

Thank you for the fast merge. Happy hacking!


Add JWT tutorial

resolve https://github.com/emqx/nanomq/issues/1221

Codecov Report
Patch coverage: 5.86% and project coverage change: +12.97 :tada:

Comparison is base (f3163b4) 4.14% compared to head (d1d4950) 17.12%.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1225       +/-   ##
===========================================
+ Coverage    4.14%   17.12%   +12.97%
===========================================
  Files          24       17        -7
  Lines        7140     5105     -2035
===========================================
+ Hits          296      874      +578
+ Misses       6844     4231     -2613

Impacted Files
Coverage Δ

nanomq/bridge.c
0.00% <0.00%> (ø)

nanomq/cmd_proc.c
0.00% <0.00%> (-12.50%)
:arrow_down:

nanomq/conf_api.c
0.00% <0.00%> (ø)

nanomq/rest_api.c
0.00% <0.00%> (ø)

nanomq/web_server.c
0.00% <0.00%> (ø)

nanomq/webhook_inproc.c
0.00% <0.00%> (ø)

nanomq/apps/broker.c
21.07% <7.69%> (+3.08%)
:arrow_up:

nanomq/pub_handler.c
49.70% <27.50%> (+48.79%)
:arrow_up:

nanomq/sub_handler.c
61.37% <45.00%> (+16.54%)
:arrow_up:

nanomq_cli/dds2mqtt/mqtt_client.c
63.58% <62.50%> (+63.58%)
:arrow_up:

... and 1 more

... and 10 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Array iteration methods

The Array API defines pairs of iteration methods such as repeated_combinations and each_repeated_combination. The former, and shorter of the two, returns an Array, while the later returns an Iterator and can take a block. This is not optimal. And really, there shouldn't be a need for both.
All that is needed is a method in Iterator to "manifest" all the results. In Ruby that method is #to_a. That doesn't quite fit for Crystal. So maybe #all?
array.repeated_combinations => Iterator
array.repeated_combinations.all => Array

I get the motivation for using each_, it makes it more obvious which methods return an iterator and can take a block. But using the iterator is so much better in most cases that having the longer method name for it is not good b/c it acts as a deterrent.
Aside, why not?
array.combinations(repeated: true)

In many cases returning an array directly instead of going through an iterator is much faster, this is why there are two versions. You can try and benchmark it to see this.
Ideally we'd have something like Ruby's Enumerable and Enumerator, so you could get an Iterator out of a method implemented with just yield very easily, but this turns out to be much slower (like 1000x slower)

Or we could go with C# approach and transform that nice yielding code in the iterator version when invoked without block.
But that would not remove the returning array versions. Just the block/iterators.

You have an example? I ran some benchmarks and have yet to find an example where the speed is much different. (And that's avoiding the use of things like #first which would easily give the iterator the advantage.)
Also, the performance isn't just about speed. Memory usage is also significant. In my benchmarks, though the speed varied little, in many cases the non-iterator hit a memory wall.
a = (1..10).to_a

a.permutations.each{ |x| x }
# 0m4.246s

a.each_permutation{ |x| x }
# 0m3.784s

a.permutations.each{ |x| x }
# 0m5.076s

But...
a = (1..12).to_a
a.permutations.each{ |x| x }

produces
Too many heap sections: Increase MAXHINCR or MAX_HEAP_SECTS
Aborted (core dumped)

Also if the speed difference is so bad (1000x, wat?) then something is wrong with the implementation. That's just not reasonable.

It's true, there doesn't seem to be much different. Maybe we can make permutation and combination be the same as Ruby then. But I don't know, I always felt those two methods were out of place, every other method starts with each: each_slice, each_cons, each_with_index, each_with_object`. But I wouldn't mind keeping the same syntax and semantics as in Ruby.
About the 1000x difference, I meant to have just a single permutation implementation that would yield values, and then a generic way to turn that into an Iterator. The way Ruby does this is with fibers: it pauses and resumes the current fiber on yield. We could do the same by also using fibers and channels, but it turns out it's 1000x slower than implementation the Iterator manually, like it's done now.

I always felt those two methods were out of place, every other method starts with each: each_slice, each_cons, each_with_index, each_with_object.

That's an interesting point, in fact I recall asking that Ruby's each_with_object be renamed to just with_object a long time ago. And really the same is more or less true for all of these methods -- each is a bit obvious and thus a tad redundant if the method name uses the plural (e.g. #slices). But I am guessing you won't want to move that far away from Ruby's naming?
That being the case and you want to stick with each_ for consistency, okay. But maybe since these two don't correspond with Ruby anyway, things can be improved a little just by using more concise names, #each_combo, #combos, #each_perm, and #perms.

It is possible to transform a method that yields with a block into a method that returns a stateful iterator which implements the same logic using only AST transformations inside the compiler. In a similar manner to what http://facebook.github.io/regenerator/ does for ES6 (the code becomes a state machine). The performance impact would probably be very minimal (as efficient as a hand-written iterator, I suppose), but this is far from trivial to implement.

I don't think there's anything to do here other than a feature request for very efficient generator functions, which are rewritten to Iterators at compile time. This is a feature i'd love to have, however it seems quite difficult to implement correctly. I've created an issue for that here: https://github.com/crystal-lang/crystal/issues/4438.


Katex.org can't be accessed with Chrome 85.0.4158.4 dev 64bit.

Describe the bug:
The website can't be accessed with Chrome 85.0.4158.4 dev 64bit. I confirmed it's not caused by extensions. Because it's still unable to access with Incognito mode. But it's able to accessed with Edge or Firefox. I think it's caused by some updates from Chrome.
To Reproduce:
Open the website https://katex.org with Chrome 85.0.4158.4 dev 64bit.
Expected behavior:
The website should be able to accessed with it.
Screenshots:

Environment (please complete the following information):

Device: Desktop
OS: Windows 10
Browser: Chrome
Version:  85.0.4158.4 dev 64bit

I can verify that Chrome 85.0.4158.4 (Windows) will not access https://katex.org/. It also fails to access https://www.netlify.com/.
@kevinbarabash, @ylemkimon Is this something we need to worry about?

It's ok now. Thank you.

I can access it yesterday. But it can't be accessed again today.

I assume this is working again now. Let us know if not!

Sorry for not being around to check on this.  I'm going to assume there was a netlify outage.

Sorry for the late update. This was an issue between HTTP/2 GREASE and Netlify. See https://community.netlify.com/t/cannot-access-netlify-sites-with-chrome-85/16059 and https://bugs.chromium.org/p/chromium/issues/detail?id=1094308 for more detail.

Thank you. It's working now.


Cant get it to boot into the installer

So I am able to begin loading the macos installer but halfway through it crashes and reboots. Have you encountered this issue?

check built-in ssd → micron 2200s causing this issue

check built-in ssd → micron 2200s causing this issue

Disable pc611 hard disk


preview_site_jekyll: Fix breakage due to Ubuntu-installed jekyll

Proactively pin versions
Error on current gh-pages (d61cc1d06dbe994464c61c95e455c4b362c536fb):
$ git clean -fxd
$ ./preview_site_jekyll.py
...
{base}/styleguide/_gems/gems/jekyll-3.9.0/lib/jekyll/drops/document_drop.rb:8: warning: already initialized constant Jekyll::Drops::DocumentDrop::NESTED_OBJECT_FIELD_BLACKLIST
/usr/lib/ruby/vendor_ruby/jekyll/drops/document_drop.rb:8: warning: previous definition of NESTED_OBJECT_FIELD_BLACKLIST was here
{base}/styleguide/_gems/gems/jekyll-3.9.0/lib/jekyll/drops/drop.rb:8: warning: already initialized constant Jekyll::Drops::Drop::NON_CONTENT_METHODS
/usr/lib/ruby/vendor_ruby/jekyll/drops/drop.rb:8: warning: previous definition of NON_CONTENT_METHODS was here
/usr/lib/ruby/vendor_ruby/jekyll/hooks.rb:56: warning: constant ::Fixnum is deprecated
Configuration file: {base}/styleguide/_config.yml
Configuration file: {base}/styleguide/_config.yml
Configuration file: {base}/styleguide/_config.yml
jekyll 3.1.6 | Error:  undefined method `config=' for #<Jekyll::Site:0x00005637e7de1bb8>
Did you mean?  config
Traceback (most recent call last):
  File "./preview_site_jekyll.py", line 92, in <module>
    main()
  File "./preview_site_jekyll.py", line 88, in main
    run_env(["bundle", "exec", "jekyll", "serve", "--no-watch"])
  File "./preview_site_jekyll.py", line 80, in run_env
    run(args, env=env, check=True)
  File "/usr/lib/python3.6/subprocess.py", line 438, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['bundle', 'exec', 'jekyll', 'serve', '--no-watch']' returned non-zero exit status 1.

This change is

If we are going to be using gems, there is a lot we could do differently, especially plugin-wise. I don't think your comments are right about bundler, though. I use it frequently and it is completely isolated. Also rbenv covers both gems and ruby versions, but the default ruby version is system (you are confusing ruby-build, I think),

rvm is scary, though. That's why rbenv exists.

The issue with bundler is that I don't know how to install it without polluting something.
Debian's version of bundler is too old to support the Gemfile that's there, and I don't feel like to trying to make things work with the older bundler.
If that were fixed, then yeah, I'd be down. But I still want to isolate bundler itself.

Then probably you would have to go rbenv, with or without bundler (with is nicer), but I see what you mean.

Not obvious what is too advanced in the Gemfile, they have not changed the format too much over time, but if bundler is not happy it must be something.

My naive reading of rbenv is that I still have to futz with environment variables to isolate my changes from the user (e.g. interpreter clash). I don't see anything about setting up an explicit root, aside from RBENV_ROOT.
I'll modify my docs, but I don't see that usage as anything better. (Unless it solves the --local problem, which I don't want to test right now.)

I'm not saying change, just what my. natural workflow is. Keep everything as is.

rbenv is nice enough that someone wrote a Python equivalent (https://github.com/pyenv/pyenv), so I do certainly recommend it, if you were ever to do significant Ruby development.

I'll take a look at this later this week on my Drake day, but also just to clarify the endgame here in my head...
I want to move the integration of the cppguide into our website from this repository's GH pages into the @drake//doc:build tool, such that we have a unified preview & deploy engine inside Drake itself.  (Our WORKSPACE already fetches the styleguide repository for cpplint tooling; it will also then use the documentation contents in this new world.) Perhaps this preview tooling just moves into drake/doc as part of that (as a subroutine), or perhaps Drake's existing jekyll deploy gains this one markdown file to eat as well.
So in short, it's possible that this preview tool has a short shelf live, so I'm not going to worry yet about polishing it too much, instead just confirming that it works.

Sounds great to me! When you do so, can you also confirm it still fails on your machine on gh-pages if you have jekyll installed via apt?

You should just need only the following and nothing else, and if you use bundle, you should be fine.
source 'https://rubygems.org'
gem 'github-pages', '~> 212'

I need to look at what we are doing that is unusual.

(also echo 'gem: --no-document' > ~/.gemrc will save you a load of time)

Just push changes to your fork and let GitHub do the work.

FYI I have a different solution ready to PR to Drake soon.  I have a change to @drake//doc:build which rolls the styleguide publishing into the main deployment process, without using the styleguide's GitHub pages under the anymore.  So I'd advise holding off on any more work here until we see if that pans out.

Oooh, that'd be awesome!

Just push changes to your fork and let GitHub do the work.

Aye, suboptimal, but works.

(or buy a Mac)

wayyyyyy suboptimal for me :P
Closing for now!


Use of recaptcha.net to load captcha library to support it in china.

I'm using this component in my directive and I see it is loading re-captcha google library from google.com  domain. As google is blocked in china, its not working. However there's way to overcome this problem and we can load required library from another domain i.e. www.recaptcha.net as per google developers forum.
Ref - https://developers.google.com/recaptcha/docs/faq
Please have a look at question Can I use reCAPTCHA globally? in above link.

Both of these places explicitly check for the google domain, we should probably make the domain configurable and just default to the google domain (to avoid a breaking change)
https://github.com/VividCortex/angular-recaptcha/blob/73d01e431d07f2fc9b84c5e62677e8ed5b41a377/src/service.js#L149
https://github.com/VividCortex/angular-recaptcha/blob/73d01e431d07f2fc9b84c5e62677e8ed5b41a377/src/service.js#L162


Episode: cool gems

This is new.
Everyone is asked during registration, if they would like to share something cool, so could everyone of us.

@everyone : What would you like to share beyond the CodeRefinery trainer materials?

We did not get much input for this via the registration, so I will ask this again via email closer to the event and explain a bit more.
Any ideas on how to make it " less scary " and more inviting very welcome :)

From co-working session: Share something from a course you liked

Currently proposed topics:
PO: sure, but I cannot think of any. Can use CFF but its not made by me
EC: Yes, I can talk about how I used audio data online for my course https://smc-aau-cph.github.io/SPIS
AVC: I would be able to share about learner personas I use to design a set of teaching material (which I already have prepared). Or things I recently learned during last three months delivering them (which I expect to have as part of my post-training task by that day))
MM: Yes with 50% probability (it might be that what I want to talk about is just already known and widely discussed and I am not aware of that, I need to do some research first)
RB: I could talk about how I share history and/or how I plan to containerize the teaching environment.
AVM, RD: Yes

Here is what I plan to demo: https://github.com/bast/teaching-setup

Thank you @bast for adding some text to the page!


Symbol not found: OBJC_CLASS$_PFACL

I was using Parse with iOS 8.4 and Xcode 6.4 successfully. The parse version which I was using 1.7.5. Recently, I have updated my pods to latest and parse version is now 1.9.1 and also project settings to compile for 9.1. Secondly, I am using Xcode 7 now.
Everything compiles good and build is successful. However, when I am trying to run the application I get below error.
dyld: Symbol not found: OBJC_CLASS$_PFACL Referenced from: /Users/ankur/Library/Developer/CoreSimulator/Devices/5924BE2D-5302-4140-B9C1-4E1D671B3C1F/data/Containers/Bundle/Application/9C046EDE-986D-4A6E-B461-38D7F859F5C7/myapp.app/myapp Expected in: flat namespace in /Users/ankur/Library/Developer/CoreSimulator/Devices/5924BE2D-5302-4140-B9C1-4E1D671B3C1F/data/Containers/Bundle/Application/9C046EDE-986D-4A6E-B461-38D7F859F5C7/myapp.app/myapp
I looked for PFACL and it is part of PODS. I can find it under PODS.
Not sure what is wrong here but looks like something missing?
Thanks for the help - Ankur

Hey @ankurarora, looks like you are integrating Parse SDK as a dynamic framework via CocoaPods.
Could you double check if the framework generated by CocoaPods did properly integrate with your end application? The first version that was opensource is 1.8.0, meaning that know everything compiles from source when you build your application (Parse SDK including), and it might have affected the installation of CocoaPods if you did not fully integrate it.
Let me know how it goes. If you can provide a consistent repro project that has this problem - I could jump right in and fix it, but otherwise - the only thing that comes to my mind is to double check proper integration of CocoaPods in your project.

Hey @nlutsenko , Thanks for the response. I investigated little more on this and found that earlier version was able to run without [Parse enableLocalDatastore] however with newer version the moment I put that extra line of code the app is running perfectly. Though there are not errors while compiling the app in both cases. If it is mandatory to enable local datastore then I think having a meaningful error message would help to understand the reason of error better.
Thanks for helping, the problem is sorted
Ankur

Hey @nlutsenko , I investigated further and found that earlier version was able to run without [Parse enableLocalDatastore] however with newer version the moment I put that extra line of code the app is running perfectly. Though there are no errors while compiling the app in both cases.
I think it should be handled greacefully if enabling local datastore is mandatory.
Thanks for helping, the problem is sorted
Ankur

Sounds super strange and definitely not something we intetfully enforced.
Any chance you can add a retro project to this?

Yes I can try to do a retro project but give me some time plz


[ruby] Method Bindings & Constructor Modifier

Methods require binding/ref edges with their corresponding type declaration in dynamic languages to correctly model their object-like behaviour. This PR implements these bindings following pysrc2cpg's createBinding function.
Additionally, adds the CONSTRUCTOR modifier to constructors.
Resolves #4593

(On hold, checking codescience tests)

Hmm I see a data-flow test failing on codescience Ruby tests, but not from this PR so I'll merge


[2.4.0-beta.1] Break nested module resolve

Describe the bug
@vueuse/router import a cjs version vue-router

Reproduction
https://github.com/odex21/vite-2.4.0-beta.1-repo
yarn
cd packages/main
yarn dev

check the link http://localhost:3000/#/?id=123

System Info
System:
    OS: Windows 10 10.0.19042
    CPU: (16) x64 AMD Ryzen 7 5800H with Radeon Graphics
    Memory: 2.16 GB / 13.87 GB
  Binaries:
    Node: 14.16.1 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - C:\Program Files\nodejs\yarn.CMD
    npm: 6.14.12 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.59)
    Internet Explorer: 11.0.19041.1

Used Package Manager
yarn
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Probably same as #4005, see PR being reviewed https://github.com/vitejs/vite/pull/3753

@odex21 would you check if your issue is resolved in<EMAIL_ADDRESS>(just released)?

@odex21 would you check if your issue is resolved in<EMAIL_ADDRESS>(just released)?

Yes, it works fine now in 2.4.0-beta.2


Corrections to remove confusion in 302 notebook

Download dataset to data directory. It addresses the feedback:

The dataset is downloaded in output/ folder which is a little bit counter-intuitive since we put the dataset in dataset/ or data/ generally. Output/ is usually for the results

Corrected text about pre-training model. Explicitly say about loading a pre-trained weight instead of running tuning.

@helena-intel @ryanloney Should I fix something in the notebook to pass CI?
It seems there's issue not related to the notebook's code: Could not install packages due to an OSError

Thanks for this improvement @ljaljushkin!

@helena-intel @ryanloney Should I fix something in the notebook to pass CI? It seems there's issue not related to the notebook's code: Could not install packages due to an OSError

It is very likely a hiccup with Github Actions. I reran the CI, but still see the issue. I'll check again later today.

Thanks @ljaljushkin ! This is a nice improvement and I appreciate the extra explanation. I only have some minor language suggestions.

Thank you for your suggestions! really appreciate it


FlannBasedMatcher add single

Hi, I wonder how could I add a single Descriptor set to a trained Matcher, with the common interface. Because the docs says, that you have to pass in a vector.
So I need to pass a vector with a single descriptor, right? Ok, then my question is, which imageIdx will have the matches related to this descriptor?

@dropfen , the Q&A forum is more appropriate place for questions: http://answers.opencv.org/


Show Thumbnails of Walls on Main Wall Page

Walls should be shown as thumbnails on the main wall page.

Closing this as this has been implemented.

Where is the thumbnail? Where can we test it?

Anyways, there is more important stuff. So, I am adding this issue to a later milestone.

The idea is that a thumbnail is like a screenshot of the actual wall. I think this feature is not a top priority. We can reopen it in the future, when other more important things are done.


Docker Hub repository name
ivancurkovic046/excalidraw-docker-extension
Terms of services

[X] I accept the term of services

/validate

/validate

/validate


Delete resources so reverse migrations will work

All credit here goes to @TheRealHaoLiu

@AlanCoding im not taking the blame for this...


feat(51000) - Flag Deprecation Plan

Fixes #51000

Many tests don't set target explicitly and rely on the ES3, therefore, the PR contains many changes related to tests
https://github.com/microsoft/TypeScript/blob/896b9228710941f875294ebee2da998d500c8614/src/harness/harnessIO.ts#L393
https://github.com/microsoft/TypeScript/blob/896b9228710941f875294ebee2da998d500c8614/src/compiler/utilities.ts#L6361-L6366
Is it acceptable to set globally ignoreDeprecations for tests?

Many tests don't set target explicitly and rely on the ES3, therefore, the PR contains many changes related to tests

can you create a new pr to explicitly set tests target to es5?

can you create a new pr to explicitly set tests target to es5?

I can, however many changes will remain due to the difference in output between es3 and es5.

can you create a new pr to explicitly set tests target to es5?

I can, however many changes will remain due to the difference in output between es3 and es5.

but that PR won't need to be reviewed clearly

I suggest to deprecate --experimentalDecorators and add a new flag --legacyDecorators to migrate

In general, I think this code is "too maintanable" as it were. We'll need to update these things every 2.5 years or so, so simply hardcoding the disallowed values altogether should be simpler to read IMO. The "phases" as we have them don't need to be manifested in the code; when 5.5 comes around we'll (hopefully) simply be deleting values from enums. Hope that makes sense and can make this a bit shorter.

I've removed the parse versions in favor of the hardcoded versions and left only the type required for CompilerOptions. Maybe you have ideas for further simplification...

"too maintainable"

😂 - this is the first time I've seen such a polite synonym for "these changes don't meet expectations".

I think the above comment is the only one I have left - LGTM otherwise. Great work!

🎉!


Apply and enforce clang format

Description
So far, no consistent layout is used. This might sometimes lead to pure layout changes of the code instead of functional changes. This makes MRs harder to review.
Possible Solution and Definition of Done

[x] Write and add a clang format files
[ ] Add specification on which version of clang format to use, i.e. via a python virtual environment
[x] apply the clang format to the existing code base
[ ] Enforce the right clang format, i.e. during testing
[ ] Add information on how to integrate the clang format into your IDE to the project (readme or wiki?)

Related Issues and Merge Requests

Blocks
Is blocked by
Follows #15
Precedes
Related to
Part of
Composed of

Additional Information

Interested Parties

@mayrmt
https://embeddedartistry.com/blog/2017/11/02/a-strategy-for-enforcing-formatting-with-your-build-server/
I used and tested the strategy in the above link mentioned by @NoraHagmeyer. The "code check" part of the build fails when the formatting is incorrect. Before starting a merge request, should we update to using clang-format version 14.0.0, as already done by BACI? Currently, we are using version 6.0.1.

Since it does not seem possible/easy to apply specialized clang format in VSCode, @mayrmt found a few python scripts that could be adapted to aid in the development process.
https://github.com/mongodb/mongo/blob/a751344c0b46fa7ea121fb74aa931797b783ef2b/buildscripts/clang_format.py
https://embeddedartistry.com/blog/2017/11/02/a-strategy-for-enforcing-formatting-with-your-build-server/

@RShaw026 Yes, updating to a more recent version of clang-format makes sense. Good idea!


fix wrong qrel path for tripclick/train/head/dctr

fix wrong qrel path for tripclick/train/head/dctr

Thanks @thongnt99! I also pushed up corresponding changes to the integration tests.
Perhaps I should add a test for all datasets that ensures query ids in the query set and qrels overlap to avoid problems like this in the future.