Reduce size of published wheel files (_pywrap_tensorflow_internal.so)

Please make sure that this is a feature request. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:feature_template
System information

TensorFlow version (you are using): 2.2.0
Are you willing to contribute it (Yes/No): N/A

Describe the feature and the current behavior/state.
The size of the wheel files published for Tensorflow over the 2.x.x releases has increased dramatically and is reaching somewhat absurd levels. Taking for example the cp37 manylinux2010 wheel:
2.0.0: 84 MB
2.1.0: 422 MB
2.2.0: 516 MB
Presently, anyone depending on these wheels may have seen their download and build times quintuple over the 2.x.x lifespan, with resulting images etc also now significantly larger than before.
The increase in size appears to be dominated by a single 1.2 GB shared library file _pywrap_tensorflow_internal.so. Presumably this increase might relate to the switch from swig to pybind, with this .so file containing nearly half a million symbols. Many of these symbols are stacks of variations of parameters to single functions.
Obviously just asking for a file to be smaller isn't much help, but I would hope this can prompt some discussion about ways this can be remedied? Assuming it is related to the pybind switch, is there anything that can be learned from the prior swig architecture and applied in the current process?
Will this change the current api? How?
No
Who will benefit with this feature?
All users who depend on the Python .whl files published to PyPI. Users with custom builds of Tensorflow will likely also have this large .so issue.
Any Other info.
There may already be an issue or discussion related to this, but GitHub search didn't reveal much.

Hey @aaliddell So the migration to pybind11 is not the primary cause of the size increase. It's the CUDA compute capabilities that we package with each version. I think we may have packaged less with 2.0.0, but if you look at 1.15 we were hovering around 412MB as well.
That being said we are looking into packaging compressed versions of the computes. @chsigg has submitted this and we have already seen a size decrease in the tf-nightly binaries to around 350MB.

Ok, that's great to hear and I'm glad it's being look at; I see the .so file is now halved in size in the nightly wheel that you've linked. I guess other inputs to that library have been looked at to see if similar gains are available? I don't have much visibility into what now dominates the size of this file.
In terms of this issue: is it worth holding to track actions for reducing the size, or are there better more-specific issues available to track this? I'll trust your judgement on what you'd like to do with this one.

So we have internal trackers and goals to monitor and reduce the size. Since the particular issue you referenced is not the root cause, and we have significantly decreased the size for now, I'll close this issue and link it internally. Hope that is okay. Thanks for filing the issue and looking into this!


How to generate sequence with more than one word.

In cobra/cobra/models/mamba/modeling_mamba.py line 1772:
assert hidden_states.shape[1] == 1, "Only support decoding with 1 token at a time for now"

which prevent me from generating sequence. But in the paper there are examples of sequence generating like Fig. 4 in page 13.
So, how can I change the code to achieve this?

maybe set cg =True when generating is a solution.


Update next

Merges latest updates (incl. v2.2.1 release) from develop into next.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Admin authorisation

Implement middlewares: What can an admin user account do? Which data can the admin change and see and have access to, and where can do a GET, POST, PUT, or DELETE occur?

Story Points: 13
See the discussion in Backend S2
Powered by Parabol


Use json-stable-stringify for deterministic JSON files and specify typescript as peerDependency

One thing I've learned from doing very similar work on Jest is that the recording of VCR files can cause a lot of churn. When files are updated they often don't contain any real changes but are simply written with the keys in a different order.
Using json-stable-stringify helps a bit with this to reduce some churn so I made this PR for that.

BTW, congrats @palmerj3  🎉 , you're the very first contributor to this project 👍

Thx @palmerj3 ❤  PR is merged, will release it shortly with patch version update


measurements & data present in multiple databases

Bug report
System info:
InfluxDB 1.0 beta 3
CentOS/7
Steps to reproduce:
> create database db1
> use db1
Using database db1
> insert test1,foo=bar value=1i
> create database db2
> insert test1,foo=bar value=2i
> use db2
Using database db2
> insert test1,foo=bar value=3i
> select * from test1

Expected behavior:
name: test1
-----------
time			foo	value
1470868506225707010	bar	3

Actual behavior:
name: test1
-----------
time			foo	value
1470868283107824299		2
1470868506225707010	bar	3

Additional info:
Any data inserted into any measurement after a database is created will be present in that database. Only data which was inserted before a database was created will not show up in that database.

I'm afraid I'm unable to reproduce this with your test example. I tested using the docker image for 1.0.0-beta3. How did you insert the data? I used the CLI and typed in the exact commands.

I'm no longer able to reproduce this. I did it once exactly as you see above, but can't seem to do it again. My only guess is that prior to my test, influxdb somehow got into a bad state, but I don't know how that would have happened.
Close?


Version requirements update for pysqlcipher3

Fixing install on Mac OS X since version has now increased.

thanks for contributing :),
wouldn't it be better if we fix the version to be a specific version instead of the dependency manager trying to upgrade it to some version which we may not not if it will work or not :)

closing this in favor of https://github.com/tasdikrahman/tnote/pull/23 :)


[ERROR DPL-0036] Detailed placement failed.

Description
I got the following error and process exited abnormally during implementing my design.
[ERROR DPL-0036] Detailed placement failed.
Error: resizer.tcl, 78 DPL-0036
[ERROR]: during executing openroad script /openlane/scripts/openroad/resizer.tcl
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
child process exited abnormally
Environment
OpenLane environment has been installed as part of
Caravel User Project - mpw-5c release
"python3 ./env.py issue-survey" failed as follows...
Traceback (most recent call last):
File "./env.py", line 255, in
main()
File "./env.py", line 251, in main
commandsargs[0]
File "./env.py", line 154, in issue_survey
% get_tag()
File "/home/shc/Research/efabless/openlane/dependencies/get_tag.py", line 66, in get_tag
raise e
File "/home/shc/Research/efabless/openlane/dependencies/get_tag.py", line 48, in get_tag
f"Cannot determine git branch. Please specify OPENLANE_IMAGE_NAME manually.\nFull output: {branch_name_data.stderr.decode('utf8').strip()}"
dependencies.get_tag.NoGitException: Cannot determine git branch. Please specify OPENLANE_IMAGE_NAME manually.
Full output: error: unknown option `show-current'
usage: git branch [] [-r | -a] [--merged | --no-merged]
or: git branch [] [-l] [-f]  []
or: git branch [] [-r] (-d | -D) ...
or: git branch [] (-m | -M) []
or: git branch [] (-c | -C) []
or: git branch [] [-r | -a] [--points-at]
or: git branch [] [-r | -a] [--format]
Reproduction Material
openroad_issue_reproducible.tar.gz

git clone https://github.com/Hagiwara-shc/marmot_asic.git
Install OpenLane and PDK as described at https://github.com/efabless/caravel_user_project/blob/mpw-5c/docs/source/quickstart.rst
make marmot

Expected behavior
Flow finish without error.
Logs
...
set_max_fanout 5.0000 [current_design]
[INFO]: Setting RC values...
[INFO RSZ-0027] Inserted 637 input buffers.
[INFO RSZ-0028] Inserted 296 output buffers.
[INFO RSZ-0058] Using max wire length 2319um.
[INFO RSZ-0034] Found 4 slew violations.
[INFO RSZ-0036] Found 2 capacitance violations.
[INFO RSZ-0038] Inserted 14 buffers in 4 nets.
[INFO RSZ-0039] Resized 92648 instances.
[INFO RSZ-0042] Inserted 196 tie sky130_fd_sc_hd__conb_1 instances.
[INFO RSZ-0042] Inserted 14 tie sky130_fd_sc_hd__conb_1 instances.
[INFO DPL-0034] Detailed placement failed on:
[INFO DPL-0035]  138361
[INFO DPL-0035]  138866
... (snip, about 1000 lines)
[INFO DPL-0035]  115609
[INFO DPL-0035]  115632
[INFO DPL-0035] message limit reached, this message will no longer print
[ERROR DPL-0036] Detailed placement failed.
Error: resizer.tcl, 78 DPL-0036
[ERROR]: during executing openroad script /openlane/scripts/openroad/resizer.tcl
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
child process exited abnormally

@Hagiwara-shc
Regarding "python3 ./env.py issue-survey" failed as follows... this issue,
Try to use python3.6 and above version

@Hagiwara-shc
set ::env(FP_CORE_UTIL) 50 enable this in config.tcl and try lower value to utilize more core for placement, cell placement congestion high. Try set ::env(FP_CORE_UTIL) 30

@Hagiwara-shc
With following updates in config.tcl am able to complete the flow successfully.
set ::env(FP_CORE_UTIL) 30
set ::env(CELL_PAD) 2
set ::env(PL_BASIC_PLACEMENT) 0

@vijayank88
Thank you for your comment.
After I updated config.tcl as you suggested, [ERROR DPL-0036]  disappeared.
However, the flow stopped due to another problem.
Reproducible packaged and the log is as follows:
openroad_issue_reproducible_2.tar.gz
[INFO DRT-0181] Start track assignment.
[INFO DRT-0184] Done with 484755 vertical wires in 9 frboxes and 283467 horizontal wires in 10 frboxes.
[INFO DRT-0186] Done with 60026 vertical wires in 9 frboxes and 75809 horizontal wires in 10 frboxes.
[INFO DRT-0182] Complete track assignment.
[INFO DRT-0267] cpu time = 00:05:10, elapsed time = 00:11:47, memory = 3979.47 (MB), peak = 4457.57 (MB)
[INFO DRT-0187] Start routing data preparation.
[INFO DRT-0267] cpu time = 00:00:01, elapsed time = 00:00:03, memory = 4002.71 (MB), peak = 4457.57 (MB)
[INFO DRT-0194] Start detail routing.
[INFO DRT-0195] Start 0th optimization iteration.
Completing 10% with 0 violations.
elapsed time = 00:04:12, memory = 4361.95 (MB).
[ERROR]: during executing openroad script /openlane/scripts/openroad/droute.tcl
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
child killed: kill signal
Do you have any clue to fix the problem?
Should I file as another issue?

How much RAM does your machine have?
On Sat, Mar 19, 2022, 4:32 AM Hagiwara-shc @.***> wrote:

@vijayank88 https://github.com/vijayank88
Thank you for your comment.
After I updated config.tcl as you suggested, [ERROR DPL-0036] disappeared.
However, the flow stopped due to another problem.
Reproducible packaged and the log is as follows:
openroad_issue_reproducible_2.tar.gz
https://github.com/The-OpenROAD-Project/OpenLane/files/8308035/openroad_issue_reproducible_2.tar.gz
[INFO DRT-0181] Start track assignment.
[INFO DRT-0184] Done with 484755 vertical wires in 9 frboxes and 283467
horizontal wires in 10 frboxes.
[INFO DRT-0186] Done with 60026 vertical wires in 9 frboxes and 75809
horizontal wires in 10 frboxes.
[INFO DRT-0182] Complete track assignment.
[INFO DRT-0267] cpu time = 00:05:10, elapsed time = 00:11:47, memory =
3979.47 (MB), peak = 4457.57 (MB)
[INFO DRT-0187] Start routing data preparation.
[INFO DRT-0267] cpu time = 00:00:01, elapsed time = 00:00:03, memory =
4002.71 (MB), peak = 4457.57 (MB)
[INFO DRT-0194] Start detail routing.
[INFO DRT-0195] Start 0th optimization iteration.
Completing 10% with 0 violations.
elapsed time = 00:04:12, memory = 4361.95 (MB).
[ERROR]: during executing openroad script
/openlane/scripts/openroad/droute.tcl
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
child killed: kill signal
Do you have any clue to fix the problem?
Should I file as another issue?
—
Reply to this email directly, view it on GitHub
https://github.com/The-OpenROAD-Project/OpenLane/issues/1010#issuecomment-1072876829,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOQKCMIWFYRB5E4H43SNJLVAUDRRANCNFSM5RAQUFSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

RAM is 4.8GB, and I added 19GB swap file.
$ free -h
total        used        free      shared  buff/cache   available
Mem:           4.8G        735M        1.1G         61M        3.0G        3.8G
Swap:           19G        953M         19G

Increase your RAM size it will work. I will confirm later how much RAM
exactly you need to complete the routing. Also as per current configuration
flow failed with hold violations
On Sat, Mar 19, 2022, 7:58 AM Hagiwara-shc @.***> wrote:

RAM is 4.8GB, and I added 19GB swap file.
$ free -h
total used free shared buff/cache available
Mem: 4.8G 735M 1.1G 61M 3.0G 3.8G
Swap: 19G 953M 19G
—
Reply to this email directly, view it on GitHub
https://github.com/The-OpenROAD-Project/OpenLane/issues/1010#issuecomment-1072922011,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUOQKCIJLRH4MEKHEFEG4MDVAU3TDANCNFSM5RAQUFSA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Ok, I will increase RAM size and try again.
Thank you.

@Hagiwara-shc
Peak memory took to complete routing
[INFO DRT-0267] cpu time = 02:14:29, elapsed time = 01:10:16, memory = 32422.46 (MB), peak = 35505.38 (MB)

Closed to 35GB need

@vijayank88
My trial with larger RAM (48GB) has just completed! (with hold violations...). The log indicates 35GB peak too.
Thank you for your support.

Where can I find the config.tcl? I am facing a similar issue.


Build fails due to OpenIddict

My initial run of npm start failed. So I figured I'd go right into the api project, removing all node concerns and the project doesn't build. I'll spend a bit more time on it tonight, but I figured I'd let you know as it could be something really simple that you've already sorted.
`PS C:\Projects\net\aspnet-core-react-template\api> dotnet build
Project api (.NETCoreApp,Version=v1.0) will be compiled because expected outputs are missing
Compiling api for .NETCoreApp,Version=v1.0
C:\Program Files\dotnet\dotnet.exe compile-csc @C:\Projects\net\aspnet-core-react-template\api\obj\Debug\netcoreapp1.0\d
otnet-compile.rsp returned Exit Code 1
C:\Projects\net\aspnet-core-react-template\api\Models\ApplicationUser.cs(6,36): error CS0246: The type or namespace name
'OpenIddictUser' could not be found (are you missing a using directive or an assembly reference?)
C:\Projects\net\aspnet-core-react-template\api\Models\DefaultDbContext.cs(8,18): error CS0311: The type 'vipper.Models.A
pplicationUser' cannot be used as type parameter 'TUser' in the generic type or method 'OpenIddictDbContext'. The
re is no implicit reference conversion from 'vipper.Models.ApplicationUser' to 'Microsoft.AspNetCore.Identity.EntityFram
eworkCore.IdentityUser'.
C:\Projects\net\aspnet-core-react-template\api\Controllers\AuthController.cs(19,26): error CS0246: The type or namespace
name 'OpenIddictUserManager<>' could not be found (are you missing a using directive or an assembly reference?)
C:\Projects\net\aspnet-core-react-template\api\Controllers\AuthController.cs(23,13): error CS0246: The type or namespace
name 'OpenIddictUserManager<>' could not be found (are you missing a using directive or an assembly reference?)
Compilation failed.
0 Warning(s)
4 Error(s)
Time elapsed 00:00:00.9974886`

@cramhead Did you first run npm install ?  That should install node packages and also run dotnet restore ./api && dotnet restore ./api.test.  It looks like build is failing because OpenIddict is missing but it is specified as a dep: https://github.com/bradyholt/aspnet-core-react-template/blob/master/api/project.json#L23

FYI, OpenIddictUserManager and OpenIddictUser have been removed in the beta1 bits.
You can find an up-to-date tutorial here and a password flow sample here.

Thanks Brady.
I did npm install but it initially failed as docker was down. I re-ran it and I thought that everything had gone ok. Maybe it didn't. Do you know if there is a way to run as a fresh install? I'm imagining just remove node_modules for the node portion. I ran dotnet restore --no-cache but it didn't store OpenIddict and the same error resulted

@cramhead Can you specify "OpenIddict": "1.0.0-alpha2-0441" here: https://github.com/bradyholt/aspnet-core-react-template/blob/master/api/project.json#L23 and then run npm install again?

That work for getting the nuget package. The dotnet projects now build correctly.
The Helloworld does present, but there is a problem finding the types. The @type/* are under node_modules as expected.
`   ERROR in [default]
Cannot find type definition file for 'node'.
ERROR in [default]
Cannot find type definition file for 'react'.
ERROR in [default]
Cannot find type definition file for 'react-dom'.
ERROR in [default]
Cannot find type definition file for 'react-router'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\client-react\components\Landing.tsx:2:12
Cannot find name 'require'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\History.d.ts:2:
Cannot find module 'history'.
ERROR in [default]<EMAIL_ADDRESS>:3:19
Cannot find module 'history'.
ERROR in [default]<EMAIL_ADDRESS>19
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\Link.d.ts:3:19
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\Redirect.d.ts:3
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\Route.d.ts:3:25
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\Router.d.ts:6:2
Cannot find module 'history'.
ERROR in [default]<EMAIL_ADDRESS>:2:19
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\createMemoryHis
y.d.ts:1:19
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\match.d.ts:1:19
Cannot find module 'history'.
ERROR in [default] C:\Projects\NET\aspnet-core-react-template\node_modules@types\react-router\lib\useRouterHistor
.ts:1:71
Cannot find module 'history'.
ERROR in [default]<EMAIL_ADDRESS>9
Cannot find module 'history'.
Child html-webpack-plugin for "index.html":
Asset    Size  Chunks       Chunk Names
index.html  574 kB       0
chunk    {0} index.html 538 kB [rendered]
[0] ../~/html-webpack-plugin/lib/loader.js!../client-react/index.ejs 974 bytes {0} [built]
[1] ../~/lodash/lodash.js 537 kB {0} [built]
[2] ../~/webpack/buildin/module.js 241 bytes {0} [built]
I figured problem could be because of the previous npm install failure, due to docker not being up, so  I tried wiping node_module andnpm install. The there were no errors listed in the install, but the same errors were present. I tried modifying the tsconfig to add typesRoot and 'moduleResolution="node", but that didn't change anything.

Interesting.  What version of TypeScript do you have installed?  What does tsc --version output?

PS C:\Projects\net\aspnet-core-react-template> tsc --version
Version 2.0.6
PS C:\Projects\net\aspnet-core-react-template> node --version
v6.9.1
PS C:\Projects\net\aspnet-core-react-template> npm --version
3.10.8

Most of these issues were specific to Windows and some were not.  When I initially setup this project I was using OSX and everything worked fine.  The following commits on master should fix all the issues on Windows.  If you pull down the latest and run npm install && npm start you should be good to go.
e637786040784190cadf2573b273005fd41f95bb
53c1c875ebf7f31867c7417483d89c449ad18d6b
2f46ccdf1f6ef3b083e342a3ccd97af709481d9a
b713e2b74e653ce6d024d07cab7fa25314806c5c

Thanks.
I tested as well. It worked perfectly.
Marc
On Thu, Nov 3, 2016 at 8:08 AM, Brady Holt<EMAIL_ADDRESS>wrote:

Closed #2
https://github.com/bradyholt/aspnet-core-react-template/issues/2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bradyholt/aspnet-core-react-template/issues/2#event-846523245,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAXj83NW8eJqUqLyO18GJJ3P1MvEyvXBks5q6fjVgaJpZM4KnhtS
.


test-branch/execute-job/wrong-commit-sha-on-pr/800e9109-ff7c-4b3a-bce0-e6666762155d

PR for testing a3139599f57f0b83655615936ce27a09828a36ae

/canonical/self-hosted-runners/run-workflows 931ad21dd9ace68aaa0a7d341e94de96ad93fa6b


Terminal mode cannot enter characters normally

Terminal mode cannot enter characters normally
I can’t manually enter characters and then ctrl+v to copy characters, I can only right-click and paste characters into the terminal

What version of the extension are you using and can you show a video of what you're referring to? I have just tested with the most recent version of the extension and confirmed that it is working as expected on my machine.
Thanks!

Same problem here. Version of the extension is v0.10.230825001.

If you need more infomation, just let me know.
Thanks!

@Luke-Guangzhong @hongshui3000 Do you happen to know what version of the extension last worked for you?

@Luke-Guangzhong @hongshui3000 I may have diagnosed the issue, I'm investigating now and hope to merge a fix that will go into pre-release tomorrow. Thanks for catching this quickly!

The latest version worked for me is v0.10.0. But it has another problem. --> Double New Line when press Return/Enter in terminal mode (Linux) · Issue #124 · microsoft/vscode-serial-monitor (github.com)

@Luke-Guangzhong These issues should be fixed in the most recent release.


Backward compatibility checks for Queries

withSchema block should accept an optional minimumVersion parameter. If specified, troy should validate the query against previous schema versions as well.

https://github.com/cassandra-scala/troy/pull/95


Crash on fetch timeout

Reported here by @ribose-jeffreylau :
https://github.com/CalConnect/csd-transcription-systems/issues/85

fixed in v 1.8.1


cargo vendor copies unnecessary crates if crate has dependency that uses "?" syntax

Problem
cargo vendor copies necessary crates into vendor directory. But when dependencies crates use "?" syntax in feature's dependencies, this commands ignores "?". It means that this command copies all unnecessary crates into vendor directory.
Cargo.toml
[package]
name = "vendor-test"
version = "0.1.0"
edition = "2021"

[dependencies]
icu_capi = { git = "https://github.com/unicode-org/icu4x", rev = "548158ebd39b72f0149e98161dc7f82b227a5caf", default-features= false, features = ["any_provider", "icu_segmenter", "compiled_data"] }

icu_capi's Cargo.toml
This crate uses "?" syntax in dependencies of features.
..
[features]
..
icu_segmenter = ["dep:icu_segmenter"]
compiled_data = [
    "icu_calendar?/compiled_data",
    "icu_casemap?/compiled_data",
    "icu_collator?/compiled_data",
    "icu_datetime?/compiled_data",
    "icu_decimal?/compiled_data",
    "icu_displaynames?/compiled_data",
    "icu_list?/compiled_data",
    "icu_locid_transform?/compiled_data",
    "icu_normalizer?/compiled_data",
    "icu_plurals?/compiled_data",
    "icu_properties?/compiled_data",
    "icu_segmenter?/compiled_data",
    "icu_timezone?/compiled_data",
]
..
[dependencies]
icu_segmenter = { workspace = true, features = ["auto"], optional = true }

Steps

Run cargo init then, modify cargo.toml

[package]
name = "vendor-test"
version = "0.1.0"
edition = "2021"

[dependencies]
icu_capi = { git = "https://github.com/unicode-org/icu4x", rev = "548158ebd39b72f0149e98161dc7f82b227a5caf", default-features= false, features = ["any_provider", "icu_segmenter", "compiled_data"] }

Run cargo vendor

Some unnecessary crates such as icu_datetime crate are copied into vendor directory unfortunately.
Possible Solution(s)
No response
Notes
No response
Version
- cargo 1.74.0 (5a856bf98 2023-09-04)
- cargo 1.72.0 (103a7ff2e 2023-08-15)

I'm guessing this is another manifestation of #10801.

Yes. Same root cause. Closing in favor of #10801
Thanks for the report :)


OnBoardingTracingFragment: asyncIsEnabled() results in fatal crash if GMS connection fails

Checklist

[x] Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
[x] If this PR comes from a fork, please Allow edits from maintainers
[ ] Set a speaking title. Format: {task_name} (closes #{issue_number}). For example: Use logger (closes # 41)
[ ] Link your Pull Request to an issue (if applicable)
[x] Create Work In Progress [WIP] pull requests only if you need clarification or an explicit review before you can continue your work item.
[x] Make sure that your PR is not introducing unnecessary reformatting (e.g., introduced by on-save hooks in your IDE)
[x] Make sure that your PR does not contain changes in strings.xml (see issue #72)
[x] Make sure that your PR does not contain compiled sources (already set by the default .gitignore) and / or binary files

Description

Executing InternalExposureNotificationClient.asyncIsEnabled() may also result in an exception of com.google.android.gms.common.api.ApiException if connection to GMS isn't possible. Not catching this exception results in fatal crash.
Steps to reproduce:

Execute App on Android VM (Android R with Google Play Services).
Click through Onboarding until OnBoardingTracingFragment is created.

Expected result:
The app catches the exception like all other GMS exceptions and opens a popup with that exception.
Actual result:
App crashes fatally with the following exception:
2020-06-08 16:33:29.693 20904-20904/de.rki.coronawarnapp E/AndroidRuntime: FATAL EXCEPTION: main
    Process: de.rki.coronawarnapp, PID: 20904
    com.google.android.gms.common.api.ApiException: 17: API: Nearby.EXPOSURE_NOTIFICATION_API is not available on this device. Connection failed with: ConnectionResult{statusCode=UNKNOWN_ERROR_CODE(39507), resolution=null, message=null}
        at com.google.android.gms.common.internal.ApiExceptionUtil.fromStatus(com.google.android.gms:play-services-base@@17.3.0:4)
        at com.google.android.gms.common.api.internal.ApiExceptionMapper.getException(com.google.android.gms:play-services-base@@17.3.0:2)
        at com.google.android.gms.common.api.internal.zah.zaa(com.google.android.gms:play-services-base@@17.3.0:18)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zaa(com.google.android.gms:play-services-base@@17.3.0:211)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zaa(com.google.android.gms:play-services-base@@17.3.0:217)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.zaa(com.google.android.gms:play-services-base@@17.3.0:115)
        at com.google.android.gms.common.api.internal.GoogleApiManager$zaa.onConnectionFailed(com.google.android.gms:play-services-base@@17.3.0:79)
        at com.google.android.gms.common.internal.zag.onConnectionFailed(com.google.android.gms:play-services-base@@17.3.0:2)
        at com.google.android.gms.common.internal.BaseGmsClient$zzf.zza(com.google.android.gms:play-services-basement@@17.3.0:6)
        at com.google.android.gms.common.internal.BaseGmsClient$zza.zza(com.google.android.gms:play-services-basement@@17.3.0:25)
        at com.google.android.gms.common.internal.BaseGmsClient$zzc.zzc(com.google.android.gms:play-services-basement@@17.3.0:11)
        at com.google.android.gms.common.internal.BaseGmsClient$zzb.handleMessage(com.google.android.gms:play-services-basement@@17.3.0:49)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at com.google.android.gms.internal.common.zzi.dispatchMessage(com.google.android.gms:play-services-basement@@17.3.0:8)
        at android.os.Looper.loop(Looper.java:223)
        at android.os.HandlerThread.run(HandlerThread.java:67)

Proposed change:
Move if (InternalExposureNotificationClient.asyncIsEnabled()) { into the try-catch-block.

Dear @lenke182,
thanks for pointing this out and providing a PR for the issue, this is merged and addressed in #288
since your branch tests failed unfortunately so I could not merge it to dev branch directly.
Thanks and best,
Marc

@marcmuschko Just noticed: The exact same problem exists in SettingsTracingFragment.kt in the function private fun startStopTracing() on line 128. Clicking on the switch in the tracing settings results in the same fatal crash. I suggest to also put that line into the try-catch-block.
https://github.com/corona-warn-app/cwa-app-android/blob/dev/Corona-Warn-App/src/main/java/de/rki/coronawarnapp/ui/settings/SettingsTracingFragment.kt#L128


Game crashes when I press the key to open the chat

Describe the bug
When I press "t" to open the chat my game crashes.
Modpack version
6.40-alpha.1
Launcher
CurseForge App
Confirm

[X] This bug report is about an issue with the modpack itself, I have not modified the mods nor added any resource packs. If this is not the case, I know that I should post the issue to modpack's Discord support channel instead.

Additional context
https://mclo.gs/cYwIswE

Did you update your instance from 1.21.1? It is not recommended to update from stable to alphas and betas.
Either way, the workaround is to disable "Show server safety" in No Chat Reports settings for now.


Does Realm Support custom RLMObject initializers ?

i want to initialize some ignored properties when query RLMObjec, i don't know how to achieve it .
NSArray* joinedTopis = [[Topic objectsWhere:@"joined == 1"] toNSArray];
i don't know which initializers should be override.

You can not override the initializer that is called by queries since it is private.
But the behavior you expected is possible by overriding the getter of properties. Because ignored properties just behave as normal properties. So you can override getter methods to return the default value if the property has no value.
@interface Topic : RLMObject
@property NSString *title;
...
@end

@implementation Topic

- (NSString *)title {
    if (!_title) {
        _title = @"DefaultTitle";
    }
    return _title;
}

+ (NSArray *)ignoredProperties {
    return @[@"title"];
}

@end


[SYSTEMML-764] Add Univar-Stats.dml labeled console output

Added console output. Removed the existing table listing number and name of univariate statistics to avoid redundancy (the console output is self-documenting).

Can an Admin verify this patch?

LGTM.
Please close the JIRA once this is merged.

Thank you @sandeep-n !


Styles not being recompiled when using --watch mode

Describe the bug
When building my library in watch mode it compiles the css on first build but when rebuilding on change it removes the css file from the folder and only compiles the js files, anybody got an idea why this is happening and how i could resolve this?

Reproduction
https://github.com/stigvanbrabant/vite-issue-repro

System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:
  System:
    OS: Linux 5.8 Ubuntu 20.04.2 LTS (Focal Fossa)
    CPU: (16) x64 AMD Ryzen 7 3700X 8-Core Processor
    Memory: 3.20 GB / 15.64 GB
    Container: Yes
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 14.16.1 - ~/.nvm/versions/node/v14.16.1/bin/node
    npm: 7.10.0 - ~/.nvm/versions/node/v14.16.1/bin/npm
  Browsers:
    Chrome: 90.0.4430.72
    Firefox: 87.0
  npmPackages:
    @vitejs/plugin-vue: ^1.2.1 => 1.2.1
    vite: ^2.2.1 => 2.2.1

Used package manager: npm
Logs

❯ npm run build:watch

><EMAIL_ADDRESS>build:watch
> vite build --watch

vite v2.2.1 building for production...

watching for file changes...

build started...
transforming (1) src/bundle.js
warn - Tailwind is not purging unused styles because no template paths have been provided.
warn - If you have manually configured PurgeCSS outside of Tailwind or are deliberately not removing unused styles, set `purge: false` in your Tailwind config file to silence this warning.
warn - https://tailwindcss.com/docs/controlling-file-size/#removing-unused-css
✓ 4 modules transformed.
dist/style.css       3157.76kb / brotli: skipped (large chunk)
dist/my-lib.es.js   0.24kb / brotli: 0.17kb
dist/my-lib.umd.js   0.46kb / brotli: 0.26kb
built in 9038ms.

build started...
✓ 4 modules transformed.
dist/my-lib.es.js   0.24kb / brotli: 0.17kb
dist/my-lib.umd.js   0.46kb / brotli: 0.26kb
built in 1347ms.

Before submitting the issue, please make sure you do the following

[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] Provide a description in this issue that describes the bug.
[ ] 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.
[ ] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

@stafyniaksacha You build the feature in #1449 :slightly_smiling_face:
Do you know how we can fix that? Maybe you could provide a PR?

@Shinigami92 Yes, I got the issue this morning too, I am going to investigate

fixed by #3887, 2.4.0-beta.0 and later versions  work fine.


[TASK] Alert when one of main branch checks fails

Overview
The Distro team should be alerted if one of the GH Action checks fails on the main branch. Ideally, that should be via Slack.
Actions

### Sub-tasks
- [ ] TBA

hey can you give me more detail


End-to-end Simulation in Main

List of tasks to do for end-to-end simulation to get into main:

[x] Snitch core CSR decoding > @rgantonio PR #7
[x] Snitch core cluster and cluster connections > @rgantonio PR #6
[x] Add HWPE MAC to bender > @xiaoling-yi PR #10
[x] Add HWPE files > @xiaoling-yi #12
[x] Attach MAC unit > @xiaoling-yi #13
[x] Modify templates and schema > @xiaoling-yi @rgantonio #14
[x] Add SNAX configuration > @xiaoling-yi @rgantonio #14
[x] Add working CSR test > @rgantonio #15

Done!!!


Ready never called, user not logged in?

Not quite sure what's going on here
I'm trying to get SFTP to work.
var c = new ftp();
c.on('error', function(err){
	console.log(err);
});
c.on('greeting', function(msg){
	console.log(msg);
});
c.on('close', function(){
	console.log('closed');
});
c.on('end', function(){
	console.log('ended');
});
c.on('ready', function(){
	console.log('ready');
	c.list(function(err, list){
		console.log('listing');
		if(err) return cb(err);
		console.log(list);
		c.end();
	});
});
var key = fs.readFileSync('./ftp-key').toString();
console.log(key);
c.connect({
	host: '<IP_ADDRESS>',
	port: 22,
	user: 'ganna',
	password: 'password',
	secure: true,
	secureOptions: {
		rejectUnauthorized: false,
		pfx: key,
		// key: key,
		// cert: key,
		// ca: key,
	}
});

output:

the ftp server is on my local machine (using filezilla). I can see someone not logged in connect. Then they time out after enough time. Why does this not log in?
'ready' is also never called. No errors ever displayed...

Are you sure you're not confusing FTPS with SFTP (port 22)? They are two separate protocols. FTPS is just FTP over SSL/TLS, whereas SFTP is a totally different protocol over SSH. The only thing they share in common are the 3 letters :-)
If you do need SFTP, there's ssh2 which does both SSH and SFTP.

after enough finicking around i got ssh2 to work with the private key.
However, I am getting the exact same issue. Ready is never called, looks like client never tried logging in. https://github.com/mscdex/ssh2/issues/280

Ok, so perhaps it's not SSH either. Can you share what your server settings are?

Eventually I am looking to support all. FTP, TLS & FTP, SSH & FTP.
I have an actual server that I've PuTTy'ed in through SFTP and a .ppk file, however ssh2 says malformed private key (expected sequence) when I use the .ppk & connect to that server...

I thought it would be easier to try to setup a local server first & go from there...
local server configs:

(uses same files as the client is trying to use)
1 user:

with setup home folder:

Ok, so it definitely is FTPS then, not SFTP. Can you set debug: console.log in your connection config object and post the output (censoring sensitive information)?

Also, if you're using port 22 as you've set up in the server, that's for implicit SSL/TLS connections, so make sure you use secure: 'implicit' in your connection config. Otherwise use port: 21 and secure: true to use explicit SSL/TLS.

ok, with secure: 'implicit'

with port: 21 & secure: true (hey it got the greeting!)

ok switched the key to key from pfx. Worked. Thanks a lot!


Setup: research using organization secrets

Could we use organization secrets to avoid having to set secrets for an individual's repo? If so, when the user is filling out the form how do we check if their repo has organization secrets available? Do we ask them for the repo name first and then... I'm not sure what the API for that looks like. I think you'd just have the same syntax and it would access the org secrets. https://docs.github.com/en/actions/reference/encrypted-secrets#creating-encrypted-secrets-for-a-repository
Apparently org workflows are just workflow templates. I suppose we could do that and have our code pull that workflow template from the org. Not sure that's really useful to us. It would at least set some basic stuff like BASE_URL ~Could we have an interview for org admins to set up a workflow for their org? https://docs.github.com/en/actions/learn-github-actions/sharing-workflows-with-your-organization. If so, how do we limit the workflow to just some repositories? A config in the repo? We don't need to worry about a config for the repo or filename. The first will be clear by which repo is being pushed to. The second is in the test files.~

Closed by #149


[backport] Backport #6595, #7099 and #7102

Backport of #6595, #7099 and #7102

[automatic post] Jenkins, test this please.

LGTM

Jenkins CI test (for commit 71ffb069716beed7cd6e3284a8abdd84da19e970, target branch v6) succeeded!


Return a Stringable from the Translator instead of a regular string

This pull request adds the ability to return a Stringable from the Translator instead of normal strings. This allows the developer to easily manipulate strings before being displayed, for example to change their case or to concatenate translations.
Example
Consider the following translation file:
return [
    'file' => 'File',

    'phrases' => [
        'select' => 'Select',
        'search' => 'Search',
    ]
];

Now, the developer wants to display the text 'Select file'. In the old situation, the developer would have to do something like this:
{{ Str::of(__('phrases.select'))->append(' ')->append(__('file'))->lower()->ucfirst() }}

// or

{{ Str::ucfirst(__('phrases.select')) }} {{ Str::lower(__('file')) }}

When returning a Stringable, the developer will now be able to do something like this:
{{ __('phrases.select')->append(' ')->append(__('file'))->lower()->ucfirst() }}

Another example:
{{ Str::upper(__('phrases.search')) }}

// becomes

{{ __('phrases.search')->upper() }}

The beauty is that PHP will always coerce a Stringable to a string when needed, so it will not be a big backwards incompatible change, whilst still providing developers with a lot of fluent methods on a translation.
I noticed that there are several validation errors failing in the testsuite. I suspect that is because of it trying to do an operation on a Stringable that is only allowed on a string. I've really looked into this, but I can't find the place where the problem is. I would be grateful if someone with more experience can explain where the problem lies, so that I can fix it, or if that person would be willing to fix it himself. Thanks!

It seems like the failure comes from the Validator:
78) Illuminate\Tests\Validation\ValidationValidatorTest::testFailOnFirstError
Failed asserting that two arrays are equal.
--- Expected
+++ Actual
@@ @@
 Array (
     'baz' => Array (
-        0 => 'validation.required'
+        0 => 'validation.custom.baz.required'
     )
     'age' => Array (
-        0 => 'validation.min.string'
+        0 => 'validation.custom.age.min.string'
     )
 )

D:\a\framework\framework\tests\Validation\ValidationValidatorTest.php:6914

(This is only one of the many failed tests)

@Jubeki Thank you! Yes, I saw that it was coming from the Validator (and the MessageBag). It isn't removing a part like 'custom.XXXXX', but I meant more the internal part of the Validator that's responsible for this.
I noticed that if I changed only one line of the Translator (line 155) back to return $line, all tests work again.

Too much of a breaking change IMO.


Synthesis failed for java-billing

Hello! Autosynth couldn't regenerate java-billing. :broken_heart:
Here's the output from running synth.py:
ay obtain a copy of the License at\n \\*\n \\* http://www.apache.org/licenses/LICENSE-2.0\n \\*\n \\* Unless required by applicable law or agreed to in writing, software distributed under the License\n \\* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express\n \\* or implied. See the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmp6ba5uszg/google-cloud-billing-v1-java/proto-google-cloud-billing-v1-java/src/main/java/com/google/cloud/billing/v1/BillingAccountName.java.
2020-08-06 02:39:41,626 synthtool [INFO] > Replaced '/\\*\n \\* Copyright \\d{4} Google LLC\n \\*\n \\* Licensed under the Apache License, Version 2.0 \\(the "License"\\); you may not use this file except\n \\* in compliance with the License. You may obtain a copy of the License at\n \\*\n \\* http://www.apache.org/licenses/LICENSE-2.0\n \\*\n \\* Unless required by applicable law or agreed to in writing, software distributed under the License\n \\* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express\n \\* or implied. See the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmp6ba5uszg/google-cloud-billing-v1-java/proto-google-cloud-billing-v1-java/src/main/java/com/google/cloud/billing/v1/SkuName.java.
2020-08-06 02:39:41,629 synthtool [INFO] > Replaced '^package (.*);' in /tmpfs/tmp/tmp6ba5uszg/google-cloud-billing-v1-java/grpc-google-cloud-billing-v1-java/src/main/java/com/google/cloud/billing/v1/CloudBillingGrpc.java.
2020-08-06 02:39:41,629 synthtool [INFO] > Replaced '^package (.*);' in /tmpfs/tmp/tmp6ba5uszg/google-cloud-billing-v1-java/grpc-google-cloud-billing-v1-java/src/main/java/com/google/cloud/billing/v1/CloudCatalogGrpc.java.
2020-08-06 02:39:41,716 synthtool [INFO] > Running java formatter on 19 files
2020-08-06 02:39:44,859 synthtool [INFO] > Running java formatter on 2 files
2020-08-06 02:39:47,244 synthtool [INFO] > Running java formatter on 47 files
2020-08-06 02:39:53,763 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/trusted-contribution.yml
.github/workflows/ci.yaml
.kokoro/build.bat
.kokoro/build.sh
.kokoro/coerce_logs.sh
.kokoro/common.cfg
.kokoro/common.sh
.kokoro/continuous/common.cfg
.kokoro/continuous/dependencies.cfg
.kokoro/continuous/integration.cfg
.kokoro/continuous/java11.cfg
.kokoro/continuous/java7.cfg
.kokoro/continuous/java8-osx.cfg
.kokoro/continuous/java8-win.cfg
.kokoro/continuous/java8.cfg
.kokoro/continuous/lint.cfg
.kokoro/continuous/propose_release.cfg
.kokoro/continuous/samples.cfg
.kokoro/dependencies.sh
.kokoro/linkage-monitor.sh
.kokoro/nightly/common.cfg
.kokoro/nightly/dependencies.cfg
.kokoro/nightly/integration.cfg
.kokoro/nightly/java11.cfg
.kokoro/nightly/java7.cfg
.kokoro/nightly/java8-osx.cfg
.kokoro/nightly/java8-win.cfg
.kokoro/nightly/java8.cfg
.kokoro/nightly/lint.cfg
.kokoro/nightly/samples.cfg
.kokoro/populate-secrets.sh
.kokoro/presubmit/clirr.cfg
.kokoro/presubmit/common.cfg
.kokoro/presubmit/dependencies.cfg
.kokoro/presubmit/integration.cfg
.kokoro/presubmit/java11.cfg
.kokoro/presubmit/java7.cfg
.kokoro/presubmit/java8-osx.cfg
.kokoro/presubmit/java8-win.cfg
.kokoro/presubmit/java8.cfg
.kokoro/presubmit/linkage-monitor.cfg
.kokoro/presubmit/lint.cfg
.kokoro/presubmit/samples.cfg
.kokoro/release/bump_snapshot.cfg
.kokoro/release/common.cfg
.kokoro/release/common.sh
.kokoro/release/drop.cfg
.kokoro/release/drop.sh
.kokoro/release/promote.cfg
.kokoro/release/promote.sh
.kokoro/release/publish_javadoc.cfg
.kokoro/release/publish_javadoc.sh
.kokoro/release/snapshot.cfg
.kokoro/release/snapshot.sh
.kokoro/release/stage.cfg
.kokoro/release/stage.sh
.kokoro/trampoline.sh
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
java.header
license-checks.xml
renovate.json
samples/install-without-bom/pom.xml
samples/pom.xml
samples/snapshot/pom.xml
samples/snippets/pom.xml
2020-08-06 02:39:53,922 synthtool [DEBUG] > existing pom file found (samples/pom.xml) - keeping the existing
2020-08-06 02:39:53,923 synthtool [DEBUG] > existing pom file found (samples/snippets/pom.xml) - keeping the existing
2020-08-06 02:39:53,923 synthtool [DEBUG] > existing pom file found (samples/install-without-bom/pom.xml) - keeping the existing
2020-08-06 02:39:53,924 synthtool [DEBUG] > existing pom file found (samples/snapshot/pom.xml) - keeping the existing
2020-08-06 02:39:56,972 synthtool [DEBUG] > Wrote metadata to synth.metadata.
2020-08-06 02:39:57,040 autosynth [INFO] > Changed files:
2020-08-06 02:39:57,041 autosynth [INFO] > M .kokoro/release/publish_javadoc.cfg
 M .kokoro/release/publish_javadoc.sh
 M synth.metadata
2020-08-06 02:39:57,041 autosynth [DEBUG] > Running: git log dd230c816f88d0141fcd0be83498986287220d1b -1 --no-decorate --pretty=%s
2020-08-06 02:39:57,045 autosynth [DEBUG] > Running: git log dd230c816f88d0141fcd0be83498986287220d1b -1 --no-decorate --pretty=%b%n%nSource-Author: %an <%ae>%nSource-Date: %ad
2020-08-06 02:39:57,050 autosynth [DEBUG] > Running: git add -A
2020-08-06 02:39:57,055 autosynth [DEBUG] > Running: git status --porcelain
2020-08-06 02:39:57,061 autosynth [DEBUG] > Running: git commit -m Add cloud-RAD for Java

Co-authored-by: Jeff Ching<EMAIL_ADDRESS>
Source-Author: Les Vogel<EMAIL_ADDRESS>Source-Date: Thu Jul 30 13:09:50 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: dd230c816f88d0141fcd0be83498986287220d1b
Source-Link: https://github.com/googleapis/synthtool/commit/dd230c816f88d0141fcd0be83498986287220d1b
[autosynth-169 67d3650] Add cloud-RAD for Java
 3 files changed, 184 insertions(+), 6 deletions(-)
2020-08-06 02:39:57,070 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at 67d3650 Add cloud-RAD for Java
2020-08-06 02:39:57,076 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-08-06 02:39:57,082 autosynth [DEBUG] > Running: git diff HEAD..autosynth-169 -- . :(exclude)synth.metadata
2020-08-06 02:39:57,087 autosynth [DEBUG] > Running: git diff HEAD autosynth-169
2020-08-06 02:39:57,092 autosynth [DEBUG] > Running: git apply /tmpfs/tmp/tmprwrmsui_/autosynth-169.patch
2020-08-06 02:39:57,096 autosynth [DEBUG] > Running: git add -A
2020-08-06 02:39:57,101 autosynth [DEBUG] > Running: git status --porcelain
2020-08-06 02:39:57,108 autosynth [DEBUG] > Running: git commit -m Add cloud-RAD for Java

Co-authored-by: Jeff Ching<EMAIL_ADDRESS>
Source-Author: Les Vogel<EMAIL_ADDRESS>Source-Date: Thu Jul 30 13:09:50 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: dd230c816f88d0141fcd0be83498986287220d1b
Source-Link: https://github.com/googleapis/synthtool/commit/dd230c816f88d0141fcd0be83498986287220d1b
[autosynth 67d3650] Add cloud-RAD for Java
 3 files changed, 184 insertions(+), 6 deletions(-)
2020-08-06 02:39:57,116 autosynth [DEBUG] > Running: git diff autosynth-169..autosynth-171 -- . :(exclude)synth.metadata
2020-08-06 02:39:57,121 autosynth [DEBUG] > Running: git diff autosynth-169..autosynth-171 -- synth.metadata
2020-08-06 02:39:57,125 autosynth [DEBUG] > Running: git diff autosynth-171..autosynth-177 -- . :(exclude)synth.metadata
2020-08-06 02:39:57,129 autosynth [DEBUG] > Running: git diff autosynth-171..autosynth-177 -- synth.metadata
2020-08-06 02:39:57,133 autosynth [DEBUG] > Running: git diff autosynth-177..autosynth-189 -- . :(exclude)synth.metadata
2020-08-06 02:39:57,137 autosynth [DEBUG] > Running: git diff autosynth-177..autosynth-189 -- synth.metadata
2020-08-06 02:39:57,142 autosynth [DEBUG] > Running: git log -1 --no-decorate --format=%B
2020-08-06 02:39:57,146 autosynth [DEBUG] > Running: git push --force origin autosynth
remote:
remote: Create a pull request for 'autosynth' on GitHub by visiting:
remote:      https://github.com/googleapis/java-billing/pull/new/autosynth
remote:
To https://github.com/googleapis/java-billing.git
 * [new branch]      autosynth -> autosynth
2020-08-06 02:40:01,090 autosynth [DEBUG] > Running: git log -1 --pretty=%b
2020-08-06 02:40:04,100 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 690, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 539, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 670, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 400, in synthesize_loop
    synthesize_inner_loop(fork, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 444, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 273, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


[Fix #5838] 'rubocop:enable all' does not match previous more specific disable

Issue based upon CommentConfig#extra_enabled_comments returning individually enabled cops after finding enable all where the default state would have been for them to be enabled anyway.
This caused comment to contain rubocop:enabel all but the name of the offence to match the individual cop and a failure in cop_name_indentation to index the position.
Workaround is to reject offences in the cop containing enable all in the comment.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

Seems another person already provided a fix for this - #5810.


Variables within #optional body are reported as unknown

In the Contract Grammar field, variables within #optional body are reported as unknown. E.g. (with "String name optional" in a model), the following reports: "Unknown property: name File text/grammar.tem.md line -1 column -1":
  {{#optional name}} Name: {{name}} {{/optional}}

No variables are evaluated within the #optional Body.
Expected Behavior
In https://docs.accordproject.org/docs/markup-templatemark.html#optional-blocks a similar sample is used, but it does not work. Expected behavior is that if name is set in ContractData to value x, then Name: x should be displayed.
Current Behavior
False error message: "Unknown property: name File text/grammar.tem.md line -1 column -1".
It also contains wrong line and column.
Steps to Reproduce

Use e.g. standard Hello world model which opens on start of Accord Studio
Make model variable "name" optional
Add to the grammar, e.g. as a last line:     {{#optional name}} Name: {{name}} {{/optional}}

Context (Environment)
It occurs in any context (Browser, OS, ...) in latest Accord Studio.

The background for this is that I am currently evaluating the use of Accord for International Data Spaces IDS Data Usage Contracts, see https://internationaldataspaces.org/.
Such a contract has many clauses which are activated by contract model variables when set, and the value of such a variable should be displayed within the the text for that clause.

Hi @norweis,
When you write an optional block:
{{#optional name}} Name: {{name}} {{/optional}}

This changes what variables are in scope for the content of that block:
// here `name` is in scope
{{#optional name}} /* here the _content_ of `name` is in scope /* {{/optional}}

So in the case where name is an atomic type you can use the special variable {{this}} which should contain the name:
{{#optional name}} Name: {{this}} {{/optional}}

For a complex type though you would most likely write something like this:
{{#optional name}} Name: {{firstName}} {{lastName}} {{/optional}}

Assuming name is something like:
concept Person {
 ...
 o Name name
 ...
}

concept Name {
  o String firstName
  o String lastName
}

Thank you. It works. But it is not documented.

A small error remains: Unknown property: name File text/grammar.tem.md line -1 column -1":
It should report the correct line, column of the occurence of name, and possibly the hint : do you mean {{this}}

This issue isn't about template studio. Moving it to Cicero, more relevant.


Added README

Added a README to help other people in the future who might want to use this.
The steps described here allow me to run the library in a fresh package. My only regret is that I'm not able to give particular requirements in terms of which Windows SDKs and Visual Studio tools need be installed (I installed a whole bunch in the course of setting up noble-uwp and can't tell which ones are necessary for this library)

Thank you!

No Windows SDKs or Visual Studio tools should be required for this package...

Awesome! This lib is a joy to use compared to uwp


Fix a bug with TriggerInstance payload not able to contain special characters such as "{{"

See #1492 for context and details.

Fixed on master.


Changing Protocol Version

Hello, I can't figure out how to change the protocol version, for example switch from 1.8 to 1.7.2 protocol version. If this hasn't been already added, it would be a nice feature and it would make your library even better!
Greetings, xTACTIXzZ

There's already code for it in https://github.com/Steveice10/MCProtocolLib/tree/1.7.4

This isn't really feasible to do in a single library jar without a lot of pain and code duplication. You'll have to use a build from the 1.7.4 branch as previously mentioned.


Think about public API

I'm thinking this lib could be pulled into a project repo, like Moolah. Then you'd write resources / store migrations there to configure the app that the project uses.
Also, instead of having two databases, there can probably just be one DB that stores info about itself. That would keep things simpler.

Alright, one idea:

Add this project as a dependency

npm install api-pls --save

Create a .plsrc file with the URL to your DB, and the location of your resource files

Run pls bootstrap to bootstrap the database.

Run pls start to start the API

Later, run pls migrate to run migrations

Later, there could be a service that just spins up an API for you, which could be useful for keeping everything outside of your project's repo.

The above seems about right, although there might be small tweaks. #41 will begin implementing this, so I'm going to close this out.


nix-doc,nix-plugin: fix by pinning to nix 2.5 (2.6 breaks)

Description of changes
See:
https://github.com/NixOS/nixpkgs/pull/157618#issuecomment-1059727303
Unbreak the two packages.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

cc https://github.com/shlevy/nix-plugins/issues/10 for nix-plugin .

cc @lf- as nix-doc maintainer.
Please update to match the latest nix.

(split into two commits, thanks!)

Noted. I will look into this upstream.

Hi! Thanks for pinging me. I have fixed the Nix compat issues upstream. For some reason (which I really do not understand; the relevant headers are definitely included), it was not resolving the appropriate functions. I have put in a shim to fix it.


Button to load more months not appearing on mobile view

Description
This problem occurs when using the datepicker with default settings on small viewports (mobile).
It requires a lot of swiping to trigger more months to be displayed. There is an PR ( https://github.com/krystalcampioni/vue-hotel-datepicker/pull/63 ) that addresses this by introducing a button to load more months, however I didn't find any way to make it appear, also in the examples I couldn't find any with that button.
Code sample
Just the default datepicker
Datepicker Version
2.6.4
Found the cause. The styles are there but they are applied to a button tag where as the next--mobile class sits on a <div type="button" . If I change div to button everything works just fine. Still needs a fix though.

It appears that writing some text in the button, while it's a div, will make the button appear.


Code Injection vulnerability found in js-yaml dependency

 tachyons-cli > tachyons-build-css > cssnano > postcss-svgo > svgo > js-yaml
To remediate:
Upgrade js-yaml to version 3.13.1 or later
https://npmjs.com/advisories/813
Thank you!

This shouldn't happen if you install globally
npm install --global tachyons-cli
see: https://github.com/tachyons-css/cli#installation

It is meant to be installed globally with the --global flag, and requires v4 of node or higher.

If you get this error, uninstall  tachyons-cli [dev]:
npm uninstall tachyons-cli
then use the global installation command, to install the package properly. That will suppress the warning(s).

Exactly. Oversight on my part. Reinstalled globally and it works without warning anymore. Thanks.


Revert "Move generation of kea config json to generation object"

Reverts ministryofjustice/staff-device-dns-dhcp-admin#226

Reverted this commit. It causes issues relating to config verification. We can look at a refactor like this later.

Reverted this commit. It causes issues relating to config verification. We can look at a refactor like this later.


Make it clear from where in the theme does each widget get their default style

Use case
For instance, when I create a RaisedButton, where are all the default styling coming from the theme?
Now for a RaisedButton, it's possible to change any styling from the widget properties and also it's sometimes possible to intuitively figure it out, but widgets like datePicker, timePicker and Scrollbar don't have styling properties and don't tell you where from the theme they get their color from.
The process of finding this information can be very cumbersome and time consuming. I've sometimes resolved this by searching google for the info and when that's not possible, digging into the underlying code of the widget.
Proposal
In the docs, have a section where it mentions all styling and theme info for every widget.
I understand that for some widgets it could need "if then" clauses to know exactly where the default style comes from, but that is the exact point. Developers need to know this info and there's no good way to know as far as I know.

@johnpryan I created this PR for the last issue I mentioned in flutter/website#4306 (comment). Just wanted to inform you since you've helped with the other issues.

This is really similar to #70935, and I couldn't agree more.  It takes far too much effort to figure out which TextStyle I have to override for my application's AppBar, for example. It'd be nice to be able to look at the documentation for each widget and see what the defaults are for its colors, font weight / size / family, etc., so that I can create comprehensive custom themes that keep my application DRY.


[bug]: Adding zsh-eza without eza installed prevents zsh startup

Environment
Ubuntu 23.10
Reproduction steps
1. Do not install eza yet.
2. Add z-shell/zsh-eza to your zsh plugins. For example, with antidode: `echo "z-shell/zsh-eza" >> .zsh_plugins.txt`
3. Run zsh

Expected behavior
4. zsh-eza prints an error message, but leaves zsh running

Current behavior
4. zsh-eza prints an error message and zsh exits

Code snippet
# .zshrc
source $HOME/dotfiles-packages/antidote/antidote.zsh
antidote load $HOME/.zsh_plugins.txt

# .zsh_plugins.txt
z-shell/zsh-eza

Additional information
If exec zsh is used in step 3 above in a login shell or over ssh, this will quit the shell and prevent login/ssh without working around a user's default shell.
Self-service

[ ] I'd be willing to address this documentation request myself.

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines.

Are you familiar with the Contributor Covenant Code of Conduct?

[X] I have read the Contributor Covenant Code of Conduct.

Contact Details
No response

Second this issue. Particularly annoying working with something like distrobox. It pulls my .zshrc but doesn't have access to the zsh binary, which crashes distrbox.


Adding DataPollManager to manage the (periodic) data poll transmission

This commit introduces a new class DataPollManager. This class
contains all the logic related to sending of the data poll messages
and managing the data poll transmission period.
The data poll interval is derived based on combination of value
from user (from SetExternalPollPeriod()) and the internal state
of the OpenThread. Data poll manager may send data polls more
frequently during attach process or when expecting a control-message
from parent or in case of data poll transmission failures or data
poll timeouts.

Codecov Report

Merging #1508 into master will increase coverage by 0.1%.
The diff coverage is 64.49%.

@@            Coverage Diff            @@
##           master    #1508     +/-   ##
=========================================
+ Coverage   69.52%   69.62%   +0.1%
=========================================
  Files         146      148      +2
  Lines       19951    20027     +76
  Branches     2432     2451     +19
=========================================
+ Hits        13870    13943     +73
- Misses       5171     5177      +6
+ Partials      910      907      -3

Impacted Files
Coverage Δ

src/core/thread/mle.cpp
81.83% <ø> (+0.93%)
:arrow_up:

src/core/thread/mle.hpp
94.44% <ø> (ø)
:arrow_up:

src/core/api/link_api.cpp
26.81% <0%> (ø)
:arrow_up:

src/core/thread/data_poll_manager.hpp
0% <0%> (ø)

src/core/api/thread_api.cpp
46.32% <0%> (ø)
:arrow_up:

src/core/thread/mesh_forwarder.hpp
60% <100%> (+60%)
:arrow_up:

src/core/thread/data_poll_manager.cpp
64.51% <64.51%> (ø)

src/core/thread/mesh_forwarder.cpp
76.77% <87.5%> (+2.45%)
:arrow_up:

src/core/coap/coap_client.cpp
50% <0%> (-17.96%)
:arrow_down:

src/core/coap/coap_client.hpp
57.89% <0%> (-15.79%)
:arrow_down:

... and 18 more

Continue to review full report at Codecov.

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

Rebased on master to fix merge conflicts with recent merge of 77fed906ac66cb329c7173c15cd0e92395ef3a78 (moving the otLinkSendDataRequest to link_api.cpp)


UnicodeDecodeError: 'utf-8'

Hi
Getting an error with a file
Exactly which folders & files should I include in .gitignore_ _ r_godot.md
Traceback (most recent call last):
  File "/opt/tid2md/tid2md.py", line 413, in <module>
    main(args.files, args.update, args.delete, args.output_directory,
  File "/opt/tid2md/tid2md.py", line 375, in main
    if migrate_tid_file(tid_path, update, output_directory, tables,
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/tid2md/tid2md.py", line 316, in migrate_tid_file
    lines = f.readlines()
            ^^^^^^^^^^^^^
  File "<frozen codecs>", line 322, in decode
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte

Thanks for reporting! Could you attach this file to allow me to reproduce it? If the file content is private, you could delete everything except the first line (or word).

Thanks for the reply. Here is the file.
Exactly which folders & files should I include in .gitignore_ _ r_godot.tid.zip

Thank you!
I couldn't reproduce this error:
$ tid2md.py "Exactly which folders & files should I include in .gitignore_ _ r_godot.tid"
Exactly which folders & files should I include in .gitignore_ _ r_godot.md

1 tiddlers migrated to Markdown.

Which command/arguments did you use?
Are you using the latest version of tid2md?
Which Python version do you use?
Which operating system do you use?

Hi
Getting an error with a file
Exactly which folders & files should I include in .gitignore_ _ r_godot.md
Traceback (most recent call last):
  File "/opt/tid2md/tid2md.py", line 413, in <module>
    main(args.files, args.update, args.delete, args.output_directory,
  File "/opt/tid2md/tid2md.py", line 375, in main
    if migrate_tid_file(tid_path, update, output_directory, tables,
       ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/tid2md/tid2md.py", line 316, in migrate_tid_file
    lines = f.readlines()
            ^^^^^^^^^^^^^
  File "<frozen codecs>", line 322, in decode
UnicodeDecodeError: 'utf-8' codec can't decode byte 0xff in position 0: invalid start byte

I think "Exactly which folders & files should I include in .gitignore_ _ r_godot.md" was created successfully and the next file caused this error. It's unfortunate that the name of the file leading to this error isn't printed. I'll change this.
Maybe you can find the file causing this error by just printing all .tid files with ls *.tid and then checking which is the one after "Exactly which folders & files should I include in .gitignore_ _ r_godot.md".

If you use the latest commit on the main branch, you should get the name of the file starting with 0xff printed.


inventory setup not equiping certain items or withdrawing everything

Type
GPU
Operating System
None
OS Version
No response
Bug description
sometimes doesnt withdraw everything properly causing it to be stuck
Screenshots or videos
No response
RuneLite version
asd
Logs
No response

This should be fixed in the latest update


Firewall Use Cases

Submitted on slack:

Ability to model internal network (corporate subnets and GCP VCPs) in one place and reference elsewhere in rules. Routable over VPN tunnels on GCP shared VPC

Ability to model external IP of corporate egress gateways or VPC NAT gateways

Ability to model and track other external IPs of clients/partners (reference inventory) and use in rules

Blacklist set of ports (ssh, rdp, etc) that should not be exposed to internet (<IP_ADDRESS>/0) but in some cases may be allowed to restricted (external set) due to need to use private VPC (GKE is prime case at moment but better support of private VPC seems to be coming)

Enforce restricted port lockdown for specific projects

Some means to handle exceptions / overrides (whether this is implemented as hierarchy exceptions to org level policies or some other means)

We will perform a gap analysis of this request against our current firewall scanner.


Fix uninitialized mipmap levels

Fix #21391 and some related problems with low mipmap levels not being initialized.
generate_mipmaps always fails to generate the last mipmap, _generate_po2_mipmap fails to generate mipmaps of rectangular textures where one of the input dimensions is 1px, and the Squish compressor would fail in the same rectangular texture situation due to the input dimension being reduced to zero.

Needs a second commit to fix an issue with rectangular textures, hold off for now...

Thanks!


Remove links reachability validation.

Neat idea, but have some issues in practice.

Too long to execute even on simple datasets.
Services tend to ban too fast requests (OH HAI GITHUB).
It is inevitable that links will stale over time failing the validation.

Hm ... What does too long means?
Is that really annissue if build takes 1 minute or 10 minutes?
IP address blocking could be an issue though. If we ever reach 100 episodes
the most links have in show notes link to twitter profiles. Lets assume 5
twitter links for each episode: so 500 links, twitter has 180 request limit
for each 15 minutes.
So could we have a basic regex matching test at least to verify urls?
Artur Dryomov<EMAIL_ADDRESS>schrieb am Mo., 18. Juni 2018,
20:53:

@ming13 https://github.com/ming13 requested your review on:
TheContext/podcast-ci#7 https://github.com/TheContext/podcast-ci/pull/7
Remove links reachability validation..
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/TheContext/podcast-ci/pull/7#event-1686941018, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAjnroKM36u8kTDhDowpFr4sntPxvPLBks5t9_cagaJpZM4UsSoX
.

What if we cache results in memory? ie for twitter links those are mostly same between episodes
How faster would that be? It'll also reduce chances to get banned

Hm ... What does too long means?

I take this back, it is not so long in practice.

IP address blocking could be an issue though.

Just checked and GitHub blocked me via 429 pretty fast on fetching 20 links a couple of times.

So could we have a basic regex matching test at least to verify urls?

HttpUrl does the checking in its own weird way.

What if we cache results in memory? ie for twitter links those are mostly same between episodes

It is an optimization for sure but the majority of links we have are different.
Please don’t forget the stale issue. Since the content is being re-generated every time all links must be checked. Disappearing resources would become an issue pretty fast. Alternatively we can change the workflow to split validation and generation completely, validating only a chosen episode (the last one). Might be tricky to automate though and it will skip potential changes in other files.

Yeah ok
On Mon, Jun 18, 2018, 8:28 PM Artur Dryomov<EMAIL_ADDRESS>wrote:

Hm ... What does too long means?
I take this back, it is not so long in practice.
IP address blocking could be an issue though.
Just checked and GitHub blocked me via 429 pretty fast on fetching 20
links a couple of times.
So could we have a basic regex matching test at least to verify urls?
HttpUrl does the checking in its own weird way.
What if we cache results in memory? ie for twitter links those are mostly
same between episodes
It is an optimization for sure but the majority of links we have are
different.
Please don’t forget the stale issue. Since the content is being
re-generated every time all links must be checked. Disappearing resources
would become an issue pretty fast. Alternatively we can change the workflow
to split validation and generation completely, validating only a chosen
episode (the last one). Might be tricky to automate though and it will skip
potential changes in other files.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/TheContext/podcast-ci/pull/7#issuecomment-398263010,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA7B3CYo6qcZWA6llPjuaRdSEa2jpQJpks5t-G_PgaJpZM4UsSoX
.

Indeed, Stale problem is a problem. I would say some regex check (or let
httpurl do the job) is sufficient. Also I still veri much like the idea of
previewing markdown files by attaching them to pull request for new
episodes (I can work on that) where reviewer can click on links to verify
that they are pointing to the right place.
Artem Zinnatullin :slowpoke<EMAIL_ADDRESS>schrieb am Di., 19.
Juni 2018, 09:06:

Yeah ok
On Mon, Jun 18, 2018, 8:28 PM Artur Dryomov<EMAIL_ADDRESS>wrote:

Hm ... What does too long means?
I take this back, it is not so long in practice.
IP address blocking could be an issue though.
Just checked and GitHub blocked me via 429 pretty fast on fetching 20
links a couple of times.
So could we have a basic regex matching test at least to verify urls?
HttpUrl does the checking in its own weird way.
What if we cache results in memory? ie for twitter links those are mostly
same between episodes
It is an optimization for sure but the majority of links we have are
different.
Please don’t forget the stale issue. Since the content is being
re-generated every time all links must be checked. Disappearing resources
would become an issue pretty fast. Alternatively we can change the
workflow
to split validation and generation completely, validating only a chosen
episode (the last one). Might be tricky to automate though and it will
skip
potential changes in other files.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<https://github.com/TheContext/podcast-ci/pull/7#issuecomment-398263010
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AA7B3CYo6qcZWA6llPjuaRdSEa2jpQJpks5t-G_PgaJpZM4UsSoX
.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/TheContext/podcast-ci/pull/7#issuecomment-398296723,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAjnroBjVDYxqZLPN7PM2C0Yh6sRXwFRks5t-KLxgaJpZM4UsSoX
.

I think we can merge this, can't we?
Removing it is fine for me ...


Control + Q fails with read_parquet(http in notebooks?

Try Control+Q shortcut with:
SELECT * FROM READ_PARQUET('https://www.timestored.com/data/sample/pq/candle.parquet');

fixed.


Android 2.7.0 samples

@jeffswartz @robjperez could you review them? Also, I think we have to update README and doc


SFT Tokenizer Fix

After experiencing issues with GPTQ Conversion, discussed in this issue https://github.com/huggingface/trl/issues/1138.
I was able to resolve it by passing the Tokenizer into the SFTTrainer.
I also noticed that lowering gradient_accumulation_steps helps resolve the linalg.cholesky error experienced with pytorch.
I've left a note under Best Practices in the sft_trainer.mdx file.
We can decide where to best put this note in regards to GPTQ Conversion.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

@younesbelkada, updated the commit with the pre-commit fixes. Let me know if any other issues.


Add calculation for Battery-as-a-Service (Teravoima)

Added calculation for Battery-as-a-Service plans to compare how a plan like Teravoima could be used.
For consumption it is a simple fixed price plus monthly price calculation, but for production the TeraVoima plan works by "charging" a virtual battery with the excess production which can then be used any time in the future. The calculation takes into account that excess production is only used after it has been produced, so it's not just a simple subtraction of total production from total consumption over the time period.
I am not affiliated with Teravoima, I just wanted to see how such a plan would work with my usage and production compared to fixed and spot pricing.

Cool stuff and thanks for the pull request. I'll try to check this out tomorrow in detail.

Rebased via local branch, code is accepted as is.


[MS2] Finish function construct_course

[x] Finish code of construct_course.py
[x] Create unit test for construct_course function
[x] Create PR for approval
[x] Code merged to main branch

Code added and unit tested, merged to main so close to issue


Update Exec Task to be compatible with the Provider API

I want to set up the Exec task to use Gradle Providers as inputs to configure

Environment variables
System properties

When I try and use providers as inputs, the Exec task will accept them, but it will not correctly compute the values, and will instead just call toString() on the provider.
Example
// build.gradle.kts

val someInputFile = objects.fileProperty()

someInputFile.set(file("build.gradle.kts"))

val myExecTask by tasks.registering(Exec::class) {
    workingDir(projectDir)
    commandLine("echo", "\$SOME_INPUT_FILE")
    // windows
    //commandLine("cmd", "/c", "echo %SOME_INPUT_FILE%")
    environment("SOME_INPUT_FILE", someInputFile.map { it.asFile.invariantSeparatorsPath })
}

Expected output:
/path/to/my/project/build.gradle.kts

Current Behavior
Actual output shows that the value is not unwrapped.
map(property(org.gradle.api.file.RegularFile, fixed(class org.gradle.api.internal.file.DefaultFilePropertyFactory$FixedFile, path/to/my/project/build.gradle.kts)))

Context
Reproduced on Gradle 7.6 and 8.0.2
These issues are related:

https://github.com/gradle/gradle/issues/11534
https://github.com/gradle/gradle/issues/15168

Workaround
As a workaround I have to compute the property in a doFirst {} block. This is unintuitive and confusing.
// build.gradle.kts

val someInputFile = objects.fileProperty()

someInputFile.set(file("build.gradle.kts"))

val myExecTask by tasks.registering(Exec::class) {
    workingDir(projectDir)
    commandLine("cmd", "/c", "echo %SOME_INPUT_FILE%")

    doFirst {
        environment("SOME_INPUT_FILE", someInputFile.get().asFile.invariantSeparatorsPath)
    }
}

This produces the correct result
/path/to/my/project/build.gradle.kts

Thank you for your interest in Gradle!
This issue will be closed as a duplicate of

https://github.com/gradle/gradle/issues/12247

Please add your use case and 👍 to that issue.


"instruments " can't used when construct DataHandlerLP

hi,
I use staticDataLoader to load dataframe in memory, then construct DataHandlerLP. But " instruments " can't used. codes are like this:
codes:
sdl = StaticDataLoader(config=df)  # df is a dataframe
ds = DataHandlerLP(instruments='csi300',start_time='20100101',end_time='20120601',data_loader=sdl)
ds construct filed.
but if i remove "instuments='csi300'" ,ds is ok.
I wonder why?  and how can I use "instuments" to choose stock sample?
thanks

Can you give your full error traceback?
Thanks


Added Pm1 and Metablog Data to biolog_carbon_microbacterium.csv

Updates to reflect CarveMe and new update_model.yml file


beforeAll and beforeEach run in parallel and cannot depend on context set by earlier hooks

Describe the bug
All beforeAll and beforeEach hooks are run in parallel which means a hook can't depend on results of an earlier hook.
For example I defined a allWithTmp() hook to create a temp folder, and I wanted to create a allWithCert() hook that would create a self signed cert in the tmp directory. I expected hooks to run sequentially in order.
allWithTmp()
allWithCert()

function allWithTmp() {
  beforeAll(async (ctx) => {
    const tmp = ctx.tmp = await mkdtmp('suite-')
    return () => rm(tmp, { recursive: true })
  })
})

function allWithCert() {
  beforeAll(async (ctx) => {
    // join throws an error because ctx.tmp is undefined.
    await makeCert(join(ctx.tmp, 'cert.pem'))
  })
})

Reproduction
https://stackblitz.com/edit/vitest-dev-vitest-jyerpn?file=test/basic.test.ts
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    vite: latest => 3.2.2
    vitest: latest => 0.24.5

Used Package Manager
npm
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] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

beforeAll and afterAll don't have access to any context. But they do provide an access to suite information, which I don't think is documented, and might be a bug.

This behavior also applies to beforeEach.
I guess before all does not need to set state in context, or suite, because it runs exactly once and state can be set in variables in the describe closure.

This behavior also applies to beforeEach.

beforeEach and afterEach have documented access to test context. This context is unique for each test, so you cannot access it in beforeAll and afterAll.

@kristiandupont started a discussion a long time ago about similar issue:

https://github.com/vitest-dev/vitest/discussions/1364

I think if we want to implement non-parallel execution, we should follow this vision.

Recently I migrated a couple of projects that were using Jest to Vitest and bumped into the same issue. As you mention in the other issue the hooks are executed in parallel.
 ...await Promise.all(getHooks(suite)[name].map(fn => fn(...(args as any)))),

We had some utility functions to setup the test environment that internally used beforeAll and relied on the fact that Jest executed them in the order they were declared in.
We implemented a "custom hook" function to work around this, but making this behaviour configurable or default would be appreciated.
Abbreviated example:
let previousPromise: Promise<unknown> = Promise.resolve();

export function seqBeforeAll(fn: () => void | Promise<void>, timeout: number = getDefaultHookTimeout()): void {
  const p = previousPromise;
  previousPromise = new Promise<void>(async (resolve) => {
    beforeAll(async (): Promise<void> => {
      await p;
      await withTimeout(fn, timeout, `Hook timed out in ${timeout}ms.\nIf this is a long-running test...etc`)();
      resolve();
    });
  });
}

I also created a custom solution which is not very elegant but it does work. My friend Lars created an alternative solution: vitest-fixture Since it seems that @sheremet-va is ok with it, I will try to make PR.

Since it seems that @sheremet-va is ok with it, I will try to make PR.

I have a working prototype with configurable options. Will make a PR today, would you be able to review it?

Definitely, I will look at it right away. Thank you, @sheremet-va !


Refactor cli

This PR more cleanly separates arg parsing and execution of code.  It also extracts repeated coded into functions, and addresses some flake8 style issues.
There are a lot of changes here so I'm happy to address any issues or nitpicks.

Awesome. Thanks! I'll be able to review this early next week.

I've had a quick look at the changes. Superficially they all look reasonable and are definitely in the right direction.
However, with these code changes, an innocuous typo (e.g., renaming a CLI param.) can break fteproxy for lots of users. The most popular use case is with Tor [1], but it will probably be quite challenging to mimic the Tor Browser environment.
As an alternative, we may want to just test fteproxy in a few a cases: (1) with only client|server specified, (2) with IP addresses, and (3) with keys.
It'd be great if you could address this. If not, I'll have time to work on it this weekend.
[1] https://gitweb.torproject.org/builders/tor-browser-bundle.git/tree/Bundle-Data/PTConfigs/linux/torrc-defaults-appendix#n2
[2] https://travis-ci.org/kpdyer/fteproxy

I would be happy to write test cases covering these changes and more. Is there an established method of measuring test coverage?
I have lots of experience with testing in Javascript but almost none in python. Is there any documentation  on writing tests for fteproxy? Even a simple outline would go a long ways in getting me up to speed.

The idea is that we'll be able to write unit tests that are executed when you run python setup.py test.
I'll present an example this weekend. However, it will roughly look something like following:
https://github.com/kpdyer/fteproxy/blob/master/fteproxy/tests/test_relay.py
but instead of invoking the client/server via the start() method, they'd be invoked via the command line.

when can we expect this to land?

@kpdyer I'm happy to pick this up again.  Can you elaborate on your preferred approach to testing?

fteproxy usage for Tor these days is so low that we won't really break anyone if make a non-backwards compatible change. AFAICT, everything here looks good. Thanks!


Reorganizes Validation Files

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
Organizes validation into separate files based on the resource type.
Does this PR introduce a user-facing change?:

NONE

@danehans I think https://github.com/kubernetes-sigs/gateway-api/pull/668 solves part of this. Please rebase and re-organize if needed.

@danehans now that #668 is in, this PR needs to be rebased. It looks like #668 may have already done most of this reorganizing, but if there's anything it missed I'd love to get it in.

@danehans not sure if there's anything left in this PR with #668 merged in, but if so, this needs to be rebased.

Closing this issue since https://github.com/kubernetes-sigs/gateway-api/pull/668 took care of the reorganization.


Consider dockerizing spec tests for reproducible builds

I spent a good chunk of time just battling different python3, pip, and venv verioning issues to try to get the spec tests running locally. Eventually I had to run https://github.com/ethereum/consensus-specs/issues/3304 to deal with a Py03 module error.
This made me wonder if dockerizing the spec tests could make them easier to interact with. Each version could be a container that you pull and run locally, which would have a specific python version and the respective packages that we know work. Just an idea, but if there is value here I would be happy to explore :-)

Hi @michaelneuder and @hwwhww,
I also faced issues to run the project, and I did the following:

a Dockerfile to build an image that store the project + all dependencies.
scripts to replace make command.

The Dockerfile store in consensus-specs/docker:
FROM python:3.11.0-slim-bullseye as base

RUN mkdir /consensus-specs
WORKDIR /consensus-specs

FROM base as builder
ENV DEBIAN_FRONTEND=noninteractive
RUN apt update && apt install -y git
RUN python -m pip install --upgrade pip

COPY . .

RUN pip install -r requirements-dev.txt -r requirements_preinstallation.txt
RUN python setup.py pyspecdev

A requirements-dev.txt store at the root:
pytest==7.4.0
pytest-cov==4.1.0
pytest-xdist==3.3.1
lru-dict==1.2.0
remerkleable==0.1.27
py_ecc==6.0.0
py_arkworks_bls12381==0.3.4
milagro_bls_binding==1.9.0
trie==2.0.2
eth-hash==0.5.2
pycryptodome==3.15.0

A script named run_tests.sh  stored in consensus-specs/scripts to replace make test:
# /bin/sh

# constants
ALL_EXECUTABLE_SPECS=("phase0" "altair" "bellatrix" "capella" "deneb" "eip6110" "whisk")
TEST_PRESET_TYPE=minimal

COV_HTML_OUT=.htmlcov

# default flag value
version=latest
number_of_core=4

# generate coverage scope
name="eth2spec"
coverage_scope=()
for spec in "${ALL_EXECUTABLE_SPECS[@]}"
do
    coverage_scope+=("--cov=${name}.${spec}.${TEST_PRESET_TYPE}")
done

# Parse flags
while getopts v:n: flag
do
    case "${flag}" in
        v) version=${OPTARG};;
        n) number_of_core=${OPTARG};;
        \?) echo "not valid -$OPTARG:${OPTARG}";;
    esac
done

# Get IF of container that run the image `consensus-specs:$version`.
get_container_name() {
  echo $(docker ps -a -q --filter ancestor="consensus-specs:$version" --format="{{.ID}}")
}

# Stop and remove all container that use the `consensus-specs:$version` image
cleanup() {
  echo "Stop and remove current container"
  docker rm $(docker stop $(get_container_name))
}

copy_coverage_report() {
  docker cp $(get_container_name):./$COV_HTML_OUT ./$COV_HTML_OUT
}

## Equivalent to `make test`
docker run -w //consensus-specs//tests//core//pyspec consensus-specs:$version \
    pytest -n $number_of_core --disable-bls $coverage_scope --cov-report="html:${COV_HTML_OUT}" --cov-branch eth2spec

# Get coverage report form container instance
$(copy_coverage_report)

# Stop and remove all containers that use the `consensus-specs:$version` image when exiting
#  the script. It helps user to limit container instances.
trap cleanup EXIT

# for testing purpose
$SHELL

Then we can run the following at the root of the repo:

docker build -f ./docker/Dockerfile -t consensus-specs:0.0.0 .
run_tests.sh -v 0.0.0 -n 4 if you are located to consensus-specs/scripts or if you are at the root of the project pushd && cd ./scripts && run_tests.sh -v 0.0.0 -n 4 && popd

I am currently testing how to get the coverage report after the end of the tests (not working yet).
What do you think about this process above @michaelneuder ?

Hey @wenceslas-sanchez
I’m not a docker master, but it looks good.
It would be nice to add a CircleCI job to verify if the Dockerfile is correct, but no need to run all the tests in CI. Perhaps just have a dockerfile_test.sh that runs linter or so.

Hi @hwwhww
All right, I will make a PR soon 👍


line 58 of abi.py

decoded = decode(self.outputs, output_data)
should be
decoded = decode(self.outputs, bytes(output_data))
right?
because it kept throwing error that saying it's not bytes type

it looks like the problem has gone


feat: update Fuelet Wallet connector data

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: mpoplavkov:x: LuizAsFightYou have signed the CLA already but the status is still pending? Let us recheck it.

@mpoplavkov I move the content of this PR over here: https://github.com/FuelLabs/fuels-wallet/pull/881


Imbalance dataset

Hi,
I am using scArches to create and update  our reference dataset and since I am using semisupervised SCANVI model, I think I must ask my question here.
My question is about training on imbalance dataset.
Should I use other techniques to balance my dataset or this model has an option like 'focal loss' that I can use or...?
grateful for your suggesstion.

Hi @farnoush-shh! If you are on our master branch, we just added an n_samples_per_label argument to the train method of scanvi. If set, then each epoch, it will subsample the number of cell used to train per label class. If a class has fewer than n_samples_per_label it will just use all the cells in that that class for each epoch.
Soon, we might also add support for weighted sampling each epoch (#883)

Hi @farnoush-shh! If you are on our master branch, we just added an n_samples_per_label argument to the train method of scanvi. If set, then each epoch, it will subsample the number of cell used to train per label class. If a class has fewer than n_samples_per_label it will just use all the cells in that that class for each epoch.
Soon, we might also add support for weighted sampling each epoch (#883)

@farnoush-shh, I just wanted to add, we haven't fully tested this new subsampling scheme. In the paper, the authors just subsample the number of labeled training examples prior to training which is something worth trying as well.

@farnoush-shh, I just wanted to add, we haven't fully tested this new subsampling scheme. In the paper, the authors just subsample the number of labeled training examples prior to training which is something worth trying as well.

@galenxing , many thanks for your support! Looking forward for weighted sampling .

@galenxing , many thanks for your support! Looking forward for weighted sampling .

This is currently on the master branch and we will be releasing in the next few days.


workers are not visible in resque-web

We added resque-mutli-job-forks to our project today, however, it appears that the workers disappear from the resque-web UI.
When the workers first start they show up in the UI, but after the fork for the second time, they all start to appear to die off.  Looking at top the workers are obviously working and the jobs are being processed in the queue.
We are using resque 1.10.0

We have the same problem using resque 1.19.0 on rails 3.1.3 using the latest git version of resque-multi-job-forks.
Did you manage to find a solution, @mguterl ?

@Narnach I don't believe we found any solution, we're still using an older version of Resque but plan on upgrading soon.

I just tested the commits by @mguterl and the work on top of that by @dolores and together they seem to solve the problem, at least when doing simple tests in development mode.
The relevant commits are: recruitmilitary/resque-multi-job-forks@93b4f6c994994c1d462f42ee903d4444a6709fea and dolores/resque-multi-job-forks@547935e1c60b65b90e1c5c2093833b191002fd7a


Use own api in more places

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is 95.65%.

@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   98.16%   98.16%
=======================================
  Files          61       61
  Lines        7571     7571
=======================================
  Hits         7432     7432
  Misses        139      139

Impacted Files
Coverage Δ

iommi/menu.py
99.24% <ø> (ø)

iommi/table.py
98.09% <90.00%> (ø)

iommi/debug.py
84.21% <100.00%> (ø)

iommi/form.py
99.09% <100.00%> (ø)

iommi/page.py
97.24% <100.00%> (ø)

iommi/traversable.py
96.68% <100.00%> (ø)

tests/test_03_traversable.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Update link to Kops docs site

Kops has a docs site so we should link to it when possible

/assign @wongma7

/lgtm

/approve


Review variables/units and add relevant nomenclature for GENeSYS-MOD

This will review the first set of variables/units and add a list of variables relevant for GENeSYS-MOD. It should not add (at this stage) all required variables (to not have a PR too large to review), but rather see whether the process for adding variables and units is sufficiently simple and intuitive.
depends on #4

closing per merge of #36 and #48


Expose a method for deriving the index name from cache name rather th…

…an using a constant

Whoops forgot to make sure it had a JIRA :D

Thanks Will!


Format Promise in CHANGES.md to be consistent

Thanks for the suggestion, but I prefer the original formatting.

No worries @mbostock thanks for your consideration.
I was mainly try to keep it consistent with this style
var x = d3.scaleBand()
    .domain(["a", "b", "c"])
    .range([0, width]);


Predictions files are written regardless of --test-fraction

The help message for --test-fraction says the following:
--test-fraction TEST_FRACTION
                        Output model predictions for a held-out fraction of data. This should be used for model
                        selection purposes. By default, no data will be held out during training, and predictions
                        will not be written to file.

However, when the --test-fraction flag is unset, MTZ files called *predictions*.mtz are still written out, and have test columns that are exclusively 0-valued.
I think we should either correct the --test-fraction help message to match the existing behavior, or modify the behavior to match the help message.

yeah -- I'm fine with keeping it. The correlation between Iobs and Ipred seems like it could be a useful metric, even in the absence of a holdout set.