Windows 7 Crash on Start

I've been using Toggl Desktop for a while now and it was working fine this morning. This afternoon however, it crashes when I try to start it up with the message "TogglDesktop has stopped working". Here's a screenshot of the error message, if it helps:

Thanks for letting us know about the problem!
Could you send us your log files to<EMAIL_ADDRESS>They may help us understand what is going on.
You can find the files here: C:\Users<username>\AppData\Local\TogglDesktop\toggldesktop.log
Please include in the email that the log file is for me (Paul), to make sure I see it as soon as possible.
Please also let us know what version of Windows you are using.
Sorry about the inconvenience!

After first encountering this problem I reinstalled Toggl Desktop in the hopes that this would help, but it didn't. Now, however, I'm thinking that was a bad idea since there are no log files in that folder.
I'm using an up to date version of Windows 7.

Hi, I am having the same problem but on Windows 10, for the last couple of days - I'm not sure what started it. I have tried completely reinstalling and have checked the log files but neither show anything since the last successful launch (19th of August). I suppose there must a problem with permissions or the process is crashing before it initializes the logging code?
Any help would be much appreciated :smile:

James

Could you guys try starting Toggl Desktop from a command prompt and see if you get any error messages or other information that way, that is not included in the log files?

Sure, here:
ation system failed to initialize ---> System.Configuration.ConfigurationErrorsE
xception: Root element is missing. (C:\Users\JHorowitz\AppData\Local\Toggl\Toggl
Desktop.exe_Url_nmykuwnaiwey5jctue0o2u4gqyvvdoqq\7.2.281.0\user.config) ---> Sys
tem.Xml.XmlException: Root element is missing.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
   at System.Xml.XmlTextReaderImpl.Read()
   at System.Xml.XmlTextReader.Read()
   at System.Configuration.XmlUtil..ctor(Stream stream, String name, Boolean rea
dToFirstElement, ConfigurationSchemaErrors schemaErrors)
   at System.Configuration.BaseConfigurationRecord.InitConfigFromFile()
   --- End of inner exception stack trace ---
   at System.Configuration.ConfigurationSchemaErrors.ThrowIfErrors(Boolean ignor
eLocal)
   at System.Configuration.BaseConfigurationRecord.ThrowIfParseErrors(Configurat
ionSchemaErrors schemaErrors)
   at System.Configuration.BaseConfigurationRecord.ThrowIfInitErrors()
   at System.Configuration.ClientConfigurationSystem.OnConfigRemoved(Object send
er, InternalConfigEventArgs e)
   --- End of inner exception stack trace ---
   at System.Configuration.ConfigurationManager.PrepareConfigSystem()
   at System.Configuration.ConfigurationManager.GetSection(String sectionName)
   at System.Configuration.PrivilegedConfigurationManager.GetSection(String sect
ionName)
   at System.Net.Configuration.DefaultProxySectionInternal.GetSection()
   at System.Net.WebRequest.get_InternalDefaultWebProxy()
   at System.Net.WebRequest.get_DefaultWebProxy()
   at Bugsnag.Notifier..cctor()```

Thanks!
It might be that the programs settings file is corrupt. Deleting the directory C:\Users\<username>\AppData\Local\Toggl might fix this (the program will re-create the files).
Can you check if that does the trick?

Great, that seems to have fixed it! Thanks for the quick replies

It has indeed fixed it, thank you!

Great to hear, and thanks for your patience!


Updated JS code to show views in addition to votes

Updated JS code to show views in addition to votes on the rendered chart.

Not needed


Feat/cache in bundle

Q
A

Bug fix?
N

New feature?
Y

BC breaks?
Y

Deprecations?
Y

Fixed tickets
N

We won't save cached file in storage services anymore but directly in public/bundles/emscache

I would approve but there seems to be a conflicting file Processor.php

I would approve but there seems to be a conflicting file Processor.php

Fixed


fix(pointcloud): Width and Height become negative after dragging

:tada: This PR is included in version 1.16.1-alpha.1 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.12.2-alpha.1 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.16.1 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.12.2 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.9.0-alpha.1 :tada:
The release is available on npm package (@alpha dist-tag)
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on npm package (@latest dist-tag)
Your semantic-release bot :package::rocket:


Broken link on Grid page

Link to commit.
https://github.com/RedHat-UX/red-hat-design-system-site/commit/6824d21b2dd968c59a8a08d743109c21bf716b99

Link to commit.
https://github.com/RedHat-UX/red-hat-design-system-site/commit/6824d21b2dd968c59a8a08d743109c21bf716b99


Get Rid of Monorepo

In the current architecture, we don't need a monorepo-approach. Because of that we can get rid of it.
This enables us also to drop the requirement for pnpm. Techincally the user could use any package-manager then.

Let's do it! I initially planned on a monorepo approach with services/api, services/db, etc directories. However, as I've talked with more and more people, it became clear a simple SvelteKit-based directory structure is preferred.

Use Webstone to enhance your SvelteKit app


Cannot upgrade to CUDNN 5.1 (How to compile using CUDNN 5 and CUDA 7.5)

I am using a University Cluster with Tesla K80 with CUDA 7.5 and CUDnn 5.0.5. I don't have root access and cannot upgrade either CUDA or CUDnn. I have tried compiling TF 1.0 from sources using bazel by configuring CUDnn to 5.0.5. I am still experiencing this error while running deep mnist.
2017-02-23 12:29:43.883764: E tensorflow/stream_executor/cuda/cuda_dnn.cc:352] Loaded runtime CuDNN library: 5110 (compatibility version 5100) but source was compiled with 5005 (compatibility version 5000).  If using a binary install, upgrade your CuDNN library to match.  If building from sources, make sure the library loaded at runtime matches a compatible version specified during compile configuration.
2017-02-23 12:29:43.883914: F tensorflow/core/kernels/conv_ops.cc:654] Check failed: stream->parent()->GetConvolveAlgorithms(&algorithms)
Aborted
However, the mnist with simple linear model worked. Please let me know a way around this and also if you need anything else.
Thanks

Hello,
I have copied the CUDNN 5.1 binaries to a virtual environment in my local space for CUDA 7.5. I have tried installing Tensorflow version 0.10.0 using wheel given by the tensorflow official website and it worked.
I will try and install Tensorflow 1.0 from source again and will post any updates here.
Thanks

Hi, [TF 1.0 with CUDA 7.5 and CUDNN 5.1 on Tesla K80 with compute capability 3.7] successfully installed from sources without root access but using virtualenv.  Upgrading to CUDNN 5.1 fixed my problem temporarily. but building the tf source when built with CUDNN 5.0 failed.
I am closing this issue for now as upgrading to CUDNN 5.1 was a breeze even without root access. Anyone can request the wheel for TF 1.0 with CUDA 7.5 and CUDNN 5.1, compute capability 3.7 on a Tesla K80 from me directly.


Add Status field to CSIVolume API and read CSIVolumeCRs while coming up

This PR fixes the issue when Kubernetes sends a Publish Volume request on a node for a new pod with same VolumeID before unpublishing the old one.
Following changes have been implemented:

Status field has been added to CSI Volume CR.
Mountpath is also being compared
CSI Volume CRs are being read while coming up

Codecov Report

Merging #27 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   53.72%   53.72%
=======================================
  Files          12       12
  Lines         631      631
=======================================
  Hits          339      339
  Misses        272      272
  Partials       20       20

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 609aa62...329a736. Read the comment docs.


Request version info from device

This string is sent by android app during pairing
{"data":"","sid":"1654467840319","type":3}
SPA responds with
{"sid":"1654467840319","data":"{\"ip\":\"192.168.x.x\",\"uid\":\"0K040210272020102000008062\",\"dtype\":\"spa\"}","result":"ok","type":3}

We found out here
https://community.home-assistant.io/t/intex-pure-spa-wifi-control/323591/25
that last four numbers in uid are probably device model - my app says i have Bubble SPA V28062
Could you please implement this query in your lib?
Thanks

Nice finding ! There might even be more info to retrieve from this spa response.
Did you identify one or many use cases in which the communication with the spa depends on the spa device model ? It would help me to priorize the feature.
I intend to deliver a first release containing essential features only, allowing me to work on a home assistant integration.
So if this is only "nice to have" I could keep if for subsequent releases.

Hi, i use this command to have "nice" device name in home assistant integration. It's not needed for cotrolling the spa.
In thread on hass forum, some people used telnet to control SPA using commands I captured on my network and it worked for them, so it seems, there is only one version of their protocol now.

Thank you for your answer.
I'll offer a way to name the integration and I'll keep this feature request for subsequent releases.

Hi @Elkropac, what is the commercial name of your Spa ?
Mine is Carbon 4 pax (Intex ref. 28458EX) and I receive this:
{'ip': 'a.b.c.d', 'uid': '0K040210392021030300016637', 'dtype': 'spa'}
So it does not match the commercial name nor the Intex reference
I'd like to create a map of it

Mine is called Pure Spa - Bubble Greywood Deluxe AP 4 in CZ. It's last year model.
What does the original app say about your spa? Does the model correspond to the ending numbers in uid?


Update manage-topics.md

Added new section to describe Origin filter more fully.
Strings originally reviewed with Ruth in this file: https://microsoft.sharepoint-df.com/:w:/t/ECMandSearchinODSP/EdcR6qRlG1ZFuIn_XZaN3dIBCgfxKJaqcVP9G3kkyXSnoQ?e=JmJKbR

@ruthholls
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


How to retrieve a string inside a tag from xml contains namespaces

Hi,
I am unable to retrieve a value inside a tag from an xml, containg name spaces.here my intention is"not processing namespaces",just to get the required tag value.I referred tutorial,not able to understand,how to do it?
Please provide the solution if possible.

Please use the mailing list when asking questions


[SPARK-34454][SQL] Mark legacy SQL configs as internal

What changes were proposed in this pull request?

Make the following SQL configs as internal:

spark.sql.legacy.allowHashOnMapType
spark.sql.legacy.sessionInitWithConfigDefaults

Add a test to check that all SQL configs from the legacy namespace are marked as internal configs.

Why are the changes needed?
Assuming that legacy SQL configs shouldn't be set by users in common cases. The purpose of such configs is to allow switching to old behavior in corner cases. So, the configs should be marked as internals.
Does this PR introduce any user-facing change?
Should not.
How was this patch tested?
By running new test:
$ build/sbt "test:testOnly *SQLConfSuite"

Test build #135200 has started for PR 31577 at commit 111635d.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39781/

Test build #135200 has finished for PR 31577 at commit 111635d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135200/

Test build #135226 has started for PR 31577 at commit 84aed2f.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39808/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/39808/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/39808/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/135226/

Seems OK to me if we're saying nobody should be able to set it? but would we want to forbid it?

but would we want to forbid it?

If someone has a good reason to have a legacy conf as non internal, she/he can add an exception to the test. But I believe we should have a mechanism to catch the situation when someone adds "external" legacy config.


Renamed “nouns” list

The “nouns” list is misnamed, as it contains other things that nouns. This patch renames it to “english”, in the same way that “french” and “german” lists exist.

Shouldn't we make it a full list too then?


Dashboard could not load logs

Time:

Tested between 8:15 and 8:21 (CET)

Further Info:

The dashboard crashed multiple times while trying to load the latest logs. Tested on systems 7 or 5.
All other tabs are working

Observation:

Loading logs runs with no result (spinning loading indication)
Exit to Log-In-Prompt

Additional info

In parallel, a download of the last 24h of measurement data is running

Actions to resolve the issue:

None

Update 8:57:
Loading the logs works again. The download has finished in the meanwhile. A download of the last 24h currently takes >1h.

Adding logs to such issues helps a lot in finding the root cause. I think for this one, both server and dashboard logs would be interesting. If you run Tenta in development you have the logs directly in the terminal. You can find the production server logs in LogTail or DigitalOcean in your setup, and the dashboard logs in the browser console.

Server ran out of memory. Fix it by giving the server at least 512 Mb of memory. #26 will fix it too.


Add WBE app

This change is

@hlngo Thank you for your pull request.
Please merge your code with the code that is now in the repository.  I have made a folder structure where pnnl has its own directory.  I apologize for this step I should have done this before you started working on it.
Please do the following:

Change the directory for the wbe agent.  Generally we start the agent directories with an upercase letter.  I would like to also say that it should probably be WholeBuildingEnergyAgent or WBEAgent (the former is my preference).  Also put it in the pnnl folder.
Update the branding and copyright in all of the code to be 2016 (if appropriate or earlier)
Add an rst document (README.rst) or other for the application to document what it does an provide users a full example of how to use the code.  This should include what the purpose of the agent is as well as what it reads off of the message bus and how it is configured.


New version: nzbget.nzbget version 24.0

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Getting a hash error with a Winget upgrade between 23.0 and 24.0
Found NZBGet [nzbget.nzbget] Version 24.0
This application is licensed to you by its owner.
Microsoft is not responsible for, nor does it grant any licenses to, third-party packages.
Downloading https://github.com/nzbgetcom/nzbget/releases/download/v24.0/nzbget-24.0-bin-windows-setup.exe
██████████████████████████████ 12.2 MB / 12.2 MB
Installer hash does not match.

https://github.com/microsoft/winget-pkgs/blob/master/manifests/n/nzbget/nzbget/24.0/nzbget.nzbget.installer.yaml
InstallerSha256: E152747BCAA057C9A3736BE648FB6DBD7C0E06C11C45BB1EFFD87C3A79C2D922
While downloading the installer gives me (SHA-256) : D213945D9C5CFFECBA26D8F672204940F8574FC948D4013F87C1D0AC7534329E


add flexibility to RDS sampling

Since several studies have strategy where they follow RDS networks till they die out but then enroll more seeds if the sample size is too small, we need to allow RDS sample to allow this in RDS sampling. This is essential for sparse networks especially

implemented in #25. Needs to be checked

For the Vincent method (link-tracing) we need to allow RDS with the following features:

Many seeds
Targeting sample size
Restricting number of waves (useually ~2)
Allow adding seeds

This is currently not supported by sample_rds() and needs to be implemented ASAP

The last item is very important since we are about to get RDS+ data.


Relative Alias Not Found

In webpack.config.js, I specify the following alias:
{
    common: Path.join(__dirname, '../Common')
}

Then, I import from a path using that alias:
import * as Module from 'common/Modules/Module';

In my compiler, there are no errors, and runtime logging demonstrates that the module is successfully imported.
However, in Atom, with atom-typescript, the import is reported as having the error 'cannot find module'. I've also tested this in ide-typescript, which reports the same error.
I imagine that both modules are probably using another module which is responsible for this bug, but I'm not familiar with the internals of these plugins, and so I don't know where the source of the bug lies.
Any help would be much appreciated.

TypeScript compiler has no idea about webpack, so your webpack config is somewhat irrelevant here. Most importantly, it's not webpack that provides IDE niceties, but TypeScript. You most likely need to set up TypeScript's module resoltution algorithm to work roughly the same as webpack does in your case. Please see here: https://www.typescriptlang.org/docs/handbook/module-resolution.html#path-mapping

Hey, thanks for the pointer! I was aware that TypeScript itself has no knowledge of Webpack, but I was not familiar with paths. It seems as though I had also setup a (perhaps unnecessary) Webpack alias which just happened to work, without the help of paths, which was a little confusing.
For the benefit of anyone who may find this in the future, this is what I went with in tsconfig.json:
{
    "paths":
    {
        "common/*": ["../Common/*"]
    }
}


Add YoDeck https://www.yodeck.com/pricing/

Yodeck puts SSO behind their enterprise tier of pricing increasing cost over 63% for a security feature.

What is the vendor's name?
Yodeck.com
What is the vendor's pricing site?
https://www.yodeck.com/pricing/
What is the base pricing? Use the lowest tier that looks sane for a small business customer, not free or personal tiers.
7.99mo per screen
What is the minimum pricing for SSO support?
12.99mo per screen 62.57% increase.
Does this pricing info come from a quote or other non-public source?
From website.
Are there any caveats we should list in the footnotes?

Hey @jhiggason @alexzuehlke ,
as this repository was not maintained for a long time, I decided to start a new initiative under https://ssotax.org/. The data is already way more up2date than in this repository! Moreover I integrated feedback from the community like a Friends of SSO page.
Feel free to contribute there. I will merge changes quickly, I promise!


feat: add user mfa and update public access

Description
Adding User MFA setting and updating public access setting to use IBM provider.
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

Associated with #157

/run pipeline

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

GitHub release
v2.5.0

Your semantic-release bot :package::rocket:


Edit Chart Access to Non Admin / Non Owner role User

Hello Team,
Need to know if there is a way for us to provide "Edit Chart" access to Non Admin / Non Owner role user in Superset.
How to reproduce the bug

Go to 'Dashboard -> Chart -> Edit Chart'

Expected results
User with appropriate role / access to the dataset/dashboard, should be able to Edit Chart
Actual results
User is not able to Edit chart.
Environment

browser type and version: Chrome
superset version: 2.1.0
python version: 3.8

Yes. You can use assign the required permissions using the Roles module.


Tech blog cards should be clickable and open up the tech blog in new tab

The learn more button redirects to the articles specific page.


fix: profiles were not refreshed

:tada: This PR is included in version 0.11.0-beta.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Vim tmp directory not created

When the vim setup script runs it does not create a tmp directory under .vim/,  this causes vim to complain every time you open a file.  Create tmp/ as part of the setup.

Removed the .vim/ directory and put all temporary files under ~/.cache/vim/.


Lock slider

It would be cool if the function "confirmable controls" could also lock the sliders - not just the switches.
I'm often switching some of my lights on when trying to scroll, because I accidentally touched a slider.

That's a good idea. I'll put it on my list.

Fixed in 20171214


"apps retrieved" status for app nodes before any action has been taken

What version of Ctrl-Q NR are you using?
0.1.0
What version of Node-RED are you using?
No response
What operating system are you running Node-RED on?
No response
What Qlik Sense versions are you using?
No response
Describe the Bug
Node status should be initialised to empty.

Expected Behavior
No response
To Reproduce
No response

Cancel that, the current behaviour is actually standard Node-RED behaviour.


Automatic master discovery

What has been done? Why? What problem is being solved?
I didn't forget about

[ ] Tests
[ ] Changelog
[ ] Documentation

Closes #409

Locally, some map call tests had failed for me. I'll explore it after the holidays.
1) truncate.backend:"vshard".backend_cfg:{identification_mode = "name_as_key", master = "auto"}.engine:"vinyl".test_non_existent_space
...thub/tarantool/crud///test/integration/truncate_test.lua:31: expected: nil, actual: true
stack traceback:
  ...thub/tarantool/crud///test/integration/truncate_test.lua:31: in function 'truncate.backend:"vshard".backend_cfg:{identification_mode = "name_as_key", master = "auto"}.engine:"vinyl".test_non_existent_space'
  ...
  [C]: in function 'xpcall'
2) truncate.backend:"vshard".backend_cfg:{identification_mode = "uuid_as_key", master = "auto"}.engine:"vinyl".test_non_existent_space
...thub/tarantool/crud///test/integration/truncate_test.lua:31: expected: nil, actual: true
stack traceback:
  ...thub/tarantool/crud///test/integration/truncate_test.lua:31: in function 'truncate.backend:"vshard".backend_cfg:{identification_mode = "uuid_as_key", master = "auto"}.engine:"vinyl".test_non_existent_space'
  ...
  [C]: in function 'xpcall'
Ran 24 tests in 13.557 seconds, 22 succeeded, 2 failed


Reduce docker image size of standalone

ui-only is 30mb :heavy_check_mark:
standalone is ~712mb :x:
Let's see that we can reduce the image size as much as possible.
It feels a target of <100mb does not sound improbable.

standalone now at ~693mb - still very large :x:

standalone:v0.0.4-clientserver-v0.9.5) now at ~731mb - got larger :x:

Maybe inspecting it with dive will help. From what I've seen, there are a lot of build components that get committed to the final image. Adding a new build stage and just copying the built/compiled artifacts may help reduce the size.


Incorrect file name mentioned in section "Set and hit a breakpoint in the client-side React code"

Section: Set and hit a breakpoint in the client-side React code
Point number: 3
Line: "Switch to Visual Studio and set a breakpoint in app-bundle.js code in"
Fix required: The file name is actually "app.tsx" instead of "app-bundle.js".

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

ID: e5b3b809-00c3-5637-ab09-0180827ffe4c
Version Independent ID: 2e5782ba-8234-95d5-cc22-1a68c3b4ae34
Content: Create a Node.js and React app - Visual Studio
Content Source: docs/nodejs/tutorial-nodejs-with-react-and-jsx.md
Product: visual-studio-dev15
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@arpitadessai Thanks for the feedback! I'll take a look at it.

This is by design. Step 8 has more info about setting and hitting breakpoints in app.tsx


Write down the typing rules for MLTT

Basically repeat these rules:
https://gitlab.com/JasonHuZS/practice/-/blob/master/MLTT/Statics/Full.agda
One can use notations to make them look nicer in Coq, c.f. https://coq.github.io/doc/v8.16/refman/user-extensions/syntax-extensions.html#coq:cmd.Notation .

Make sure to read https://dl.acm.org/doi/10.1145/3167091 first!


add function to enable a clean require in a single call

I thought it'd be nice to be able to write the below:
var foo = clearRequire.require('./foo')

vs.
clearRequire('./foo')
var foo = require('./foo')

Perhaps this crosses a grey area of "we don't ever want to replace require, just an easy way to clean it". It's useful for testing purposes however.

https://github.com/sindresorhus/require-uncached ?

All that searching and I never found that module...


Implement DM4

Implement DM4 message handling and associated testing for validation. #15

This work was completed with the check in #142


State of Linux->Windows cross compilation

Hi, what's the current state of cross compilation from Windows? The README says:

Note that using this crate on non windows platform is undefined behavior. It does not contain safeguards against doing so.

What actions would be necessary to get this to work on Linux? If this is not easily possible, can at least some safeguards please be added?

My build.rs looks something like this:
fn main() {
    output_dir();
    gen();
    match std::env::var("CARGO_CFG_TARGET_OS").unwrap().as_str() {
        "linux" | "macos" => { /* Nothing to do here */ }
        // If building *FOR* Windows
        "windows" => {
            generate_manifest();
            generate_rc();
            let mut rc = winres::WindowsResource::new();
            rc.set_manifest_file(&(String::from(env!("CARGO_MANIFEST_DIR")) + "/" + MANIFEST_OUT));
            rc.set_resource_file(&(String::from(env!("CARGO_MANIFEST_DIR")) + "/" + RC_OUT));

            #[cfg(not(target_os = "windows"))]
            {
                rc.set_ar_path("x86_64-w64-mingw32-gcc-ar");
                rc.set_windres_path("x86_64-w64-mingw32-windres");
            }

            rc.compile().unwrap()
        }
        _ => panic!("Target not supported"),
    }
}

Cross-compilation should now work out of the box in v0.1.13 of my fork. I updated the README as well, it had some buggy instructions (target_os is the host OS when you compile build.rs).


bazel: update to latest rolling release

https://github.com/bazelbuild/bazel/releases/tag/6.0.0-pre.20220414.2
As discussed in Slack, there are a number of disadvantages to staying on LTS releases.
This can also help unblock Envoy Mobile's iOS releases: https://github.com/envoyproxy/envoy-mobile/pull/2217
Risk Level: Moderate, changing the bazel version, though issues are most likely to be caught at build/test time
Testing: Existing, CI

/retest

For years I hand curated my Bazel versions (somewhat out of necessity in the early days of Bazel on aarch64 Linux).


changed publishNotReadyAddresses to service spec from annotation

Motivation

publishNotReadyAddresses is a service spec and not a service annotation. This is mentioned in the K8s API docs at https://v1-17.docs.kubernetes.io/docs/reference/generated/kubernetes-api/v1.15/#servicespec-v1-core

Modifications

Modified publishNotReadyAddresses from annotation to service spec

Verifying this change

[x] Make sure that the change passes the CI checks.

I really appreciate if someone can review this PR. Thank you.

@zymap @wolfstudy  Would you please also help review this change?


Tesseract with live camera

I've been trying this wrapper and it works perfectly with photos. Now I would like to start a live camera and use tesseract to OCR like the following: https://github.com/yushulx/dotnet-barcode-qr-code-sdk/blob/main/example/maui/Platforms/iOS/CaptureOutput.cs
The problem I am facing is that this wrapper supports PNG/JPG only and I'm getting InvalidImage error. I've tried to convert CGImage to UIImage and get the NSData as PNG. I'm not getting an error, but the text is not getting recognised.
How would one approach this situation?

I need more information
Are you able to save the converted png/jpeg image byte buffer to file with png/jpeg file extension?

Do the bytes produce correct image
Are the characters clear in this saved image

What kind of words are you trying to recognize? (single word, sentences, single characters)

Page segmentation mode might change the recognized result if you are not looking for sentences

Your example code link
I am quite busy and with a quick look I am not really sure if I understand the code correctly.
Is this the process that is ran for every frame your camera captures? What camera library you are using to get the input?
My thoughts
I would look if the converted NSData can be saved as png/jpeg byte array and then recognized as image path.
If that works problem is probably not a problem with encoding.
It is very hard to say without an example. It would help if you could produce simple example project.
I am very busy for next like 4 weeks and might not be able to help very much. iOS is also a platform that I am not very familiar with and only have emulator with remote connection as my iOS development device. Remote connection causes very bad performance with emulator so live camera will make testing pain.
I will try to help the best I can, but it might not be very much.
You can ask @sivanmg if he is willing to help with iOS technical side. I know better the tesseract side and he knows iOS side.
Regards,
Henri


get_physical_device_memory_properties return invalid heap_index

In ash 0.38 memory for VkPhysicalDeviceMemoryProperties became uninitialized
let mut memory_prop = mem::MaybeUninit::uninit();
This change produce invalid data in (at least) memory_types field (specifically heap_index becomes random) which violates restrictions in VkMemoryType
As for ash 0.37 it is fine as memory is zeroed
let mut memory_prop = mem::zeroed();
Driver 545.29.06-0ubuntu<IP_ADDRESS>

Let us first link some relevant sources:
The PR that made this MaybeUninit::uninit(): #798
The bit of code that you're bringing up for discussion:
https://github.com/ash-rs/ash/blob/331724c216b62d2914479d44e16c977e685e0749/ash/src/instance.rs#L440-L452
Also relevant to mention that the "545.29.06 driver" you're mentioning is an Nvidia driver. Other hardware vendors, which also implement the Vulkan API, have a different numbering scheme.

Have you looked at the memory_type_count field of vk::PhysicalDeviceMemoryProperties? It describes how many elements in memory_types are initialized when Instance::get_physical_device_memory_properties() returns. Keying off of heap_index on its own wouldn't make much sense (if it was mem::zeroed()) as heap_index 0 is valid if there's at least one heap.
Specifically we added slice getters in #858 to make it easier to extract the valid portion of this struct.

Now, given that the fields of struct vk::PhysicalDeviceMemoryProperties are safe to access, I don't know if unsafeness of Instance::get_physical_device_memory_properties() allows us to say that the caller is responsible for only looking at the first ..memory_type_count number of elements in memory_types (same for heaps) in the returned struct instance.

https://doc.rust-lang.org/std/pin/struct.Pin.html#method.get_unchecked_mut might be a nice example of this, where the safety docs don't specify any preconditions to uphold, but do say how you have to handle the returned value to not cause any UB.
On the other hand the safety docs for MaybeUninit::assume_init() specify that T must be fully initialized, which is not the case here.
Maybe this is one of the few cases where we should move back to ::zeroed() though these uninitialized invalid values do force callers to actually look at the _count field or (better yet) use our helper slice getters.

Thanks for answering
You are right I didn't check memory_type_count so it works with zeroed memory but not with uninitialized
As it is my bad I guess issue is resolved

I started inventorizing all static-sized arrays that are runtime length bounded (either by a field or for strings a NULL terminator) at https://github.com/ash-rs/ash/compare/static-runtime-bounded-array-maybeuninit. Might get a little nasty to deal with this properly, as all accessor functions now have to be unsafe because we can never assume that the "bound" was initialized or (by safely writing a slice) tampered with.


Wrap code blocks

Is there a way to wrap text in the code blocks? I'd appreciate if you can point me to the lines in CSS that need to be changed, I can do that after I export to html.   --Thanks!

Yeah, this would best be done outside of Grip since GitHub doesn't offer text wrapping in code blocks.
Since all code blocks are <pre> elements, you can add something like this:
pre {
    white-space: pre-wrap;       /* Since CSS 2.1 */
    white-space: -moz-pre-wrap;  /* Mozilla, since 1999 */
    white-space: -pre-wrap;      /* Opera 4-6 */
    white-space: -o-pre-wrap;    /* Opera 7 */
    word-wrap: break-word;       /* Internet Explorer 5.5+ */
}


Prepare v2.0.5 release

Pre-Review Checklist for the PR Author

[x] Add a second reviewer for complex new features or larger refactorings
[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[x] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

Prepare v2.0.5 release

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Relates #2099

ROS 2 CI:

Linux
Linux-aarch64
Windows


polytone should be imported to ibc-apps repo.

From Jake / noah,
https://github.com/DA0-DA0/polytone
TODO

[ ] Wiki Content copy paste
[ ] issue pull over
Add to @cosmos/daodao_ibc-apps_write -> Noah, Jake, Artimix, and bekauz
[x] getting them in as codeowners
[x] Artifact Publish on tag of cosmwasm/polytone

DAODAO team will review and ship the PR.

I believe this can be closed as of #169? cc @Reecepbcups


Why can't you access the chatGptAPI interface suddenly? how to solve it?

request to https://api.openai.com/v1/chat/completions failed, reason: connect ETIMEDOUT <IP_ADDRESS>:443
It didn't show up yesterday, but it's a problem today. I don't know what's going on.

@YJHXZ thanks for using the extension. There could be a limitation to ports in your region. Could you check this reply from @yibuxulong here: https://github.com/gencay/vscode-chatgpt/issues/176#issuecomment-1453020918
This reply from the other thread may help:

thanks for your replying! @gencay I found out that may be because the chatgpt is also locked to the Chinese port in the API as that in the website, which was not the case a week ago. I have restored normal use by switching the port. I hope it can help other users. Thank you again for your work and time.


[mbedtls]: update to 2.28.3

Fixes #31666

[X] Changes comply with the maintainer guide
[X] SHA512s are updated for each updated download
[ ] ~The "supports" clause reflects platforms that may be fixed by this new version~
[ ] ~Any fixed CI baseline entries are removed from that file.~
[ ] ~Any patches that are no longer applied are deleted from the port's directory.~
[X] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[X] Only one version is added to each modified port's versions file.

[4/100] C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1435~1.322\bin\Hostx64\arm64\cl.exe    -ID:\buildtrees\mbedtls\arm64-windows-dbg\include -ID:\installed\arm64-windows\include -ID:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\include -ID:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\library /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /MP  /W3 /utf-8 /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1 /showIncludes /Folibrary\CMakeFiles\mbedcrypto.dir\aesni.c.obj /Fdlibrary\CMakeFiles\mbedcrypto.dir\mbedcrypto.pdb /FS -c D:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\library\aesni.c
FAILED: library/CMakeFiles/mbedcrypto.dir/aesni.c.obj
C:\PROGRA~1\MICROS~1\2022\ENTERP~1\VC\Tools\MSVC\1435~1.322\bin\Hostx64\arm64\cl.exe    -ID:\buildtrees\mbedtls\arm64-windows-dbg\include -ID:\installed\arm64-windows\include -ID:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\include -ID:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\library /nologo /DWIN32 /D_WINDOWS /W3 /utf-8 /MP  /W3 /utf-8 /D_DEBUG /MDd /Z7 /Ob0 /Od /RTC1 /showIncludes /Folibrary\CMakeFiles\mbedcrypto.dir\aesni.c.obj /Fdlibrary\CMakeFiles\mbedcrypto.dir\mbedcrypto.pdb /FS -c D:\buildtrees\mbedtls\src\v2.28.3-daafc1cf77.clean\library\aesni.c
C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Tools\MSVC\14.35.32215\include\immintrin.h(15): fatal error C1189: #error:  This header is specific to X86 and X64 targets

see #34744


⚠ Bofrost.fr has degraded performance

In e083f34, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1304 ms

Resolved: Bofrost.fr performance has improved in 39ef249.


[5.4] Cross-connection has not working

Laravel Version: 5.4.6
PHP Version: 7.1.0
Database Driver & Version: MySQL 5.7.16

Description:
I have a similar setup as mentioned in #17515 (two schema's on a single server and a couple of relationships between those). I explicitly specified both connections but as soon as I try a has (and probably whereHas too) query from the model on the first connection to a model on the second connection it fails stating that the second model's table can not be found (using the first connections schema), which is correct but wrong, since the relationship should know to look (since it is explicitly specified) in the second connections schema (probably a simple (pseudocode)
if ($model1->getConnectionName() != $model2->getConnectionName()) {
  $model2->{fullyQualifiedColumnName} =
$model2->getConnection()->getSchema().".".$model2->{fullyQualifiedColumnName};
}

would suffice to fix the issue (assuming both schema's are on the same database server (otherwise it would never work, since it is near impossible to do subqueries across different server connections))
Steps To Reproduce:

Create two schema's (ie schema1 & schema2)
Create two models one in each database (ie model1 (model1 $connection == 'schema1connection') & model2 (model2 $connection == 'schema2connection'))
Create a relationship between those models (ie on model1: $this->hasMany(Model2::class))
Execute Model1::has('model2relation')->get()

Expected results
Since both models have their connections explicitly specified I'd expect to see it reflected in the resulting (sub)select (ie
WHERE [`schema1`.]`model1table`.`model1column` = `schema2`.`model2table`.`model2column`

) with schema1 optional since the query runs on this connection

A (cleaned up) error message:
Illuminate\Database\QueryException with message 'SQLSTATE[42S02]: Base table or view not found: 1146 Table 'schema1.model1' doesn't exist (SQL: select * from `model2` where exists (select * from `model2` where `model1`.`id` = `model2`.`model1_id`) limit 1)'

This was already reported here: https://github.com/laravel/framework/issues/4649

The report you reference seems to be about 2 different servers, my report is about 1 server 2 schema's which is a different situation (since cross schema (ie. on the same server) JOINs/SUBSELECTs are possible, JOINs/SUBSELECT(s) on a different server not)...

Hello,
I have the same problem at the moment. I can get for example cars with motors (i use the with function and can also say wich motors i want) without any problems accross two databases on the same server.
But if i want for example only cars with motors where the motor has a specific power (the whereHas function) it is not working.
The real code is about adminlogs and the user -> player. I want to search for logs where the player->name is like something. (adminlog and user are in one database and player is in another, but on the same server). You can see the code on the attached screenshot. A workaround is that i can do it only with the with function and filter through the collection and remove the null relations. But that is not a solution because i call later other where and orderBy functions + paginate and this collection can be very big in the end.
sorry for my english.

Does this issue continue existing or it was fixed? Regards

I didn't tested it until now and that project with the problem is not running anymore. Maybe someone else could try it out (with a very basic example).
Regards

I added package to manage this issue:
https://github.com/hoyvoy/laravel-cross-database-subqueries


Rename to airtable-export

If it also does JSON - #2 - this is a better name. Best to rename it before anyone else is using it.

I'm just going to delete airtable-to-yaml from PyPI, I won't bother creating a redirect for it.

Done: https://pypi.org/project/airtable-export/


Optimize for size

I've been considering to rewrite the AppImage runtime in Rust for a while. Looks like you're faster! :+1:
One question: Is here a way to optimize for binary size in Rust, so that we might get well under 1 MB?

@probonopd
Hi! Yes, I can optimize the size a bit if I remove mksquashfs from the release (or hide it behind a separate --features mksquashfs). Then in that case the SquashFS-only runtime version would be less than 1 MB (~960 KB).
And the version with support for both SquashFS and DwarFS will have a size of ~6 MB

Yes I think that would be a good improvement.

@probonopd
Added a lite version for AppImage uruntime without mksquashfs in v0.0.9

@probonopd
The main weight here is from assets, and rust code weighs only about ~290 KB
This approach is chosen because there is no native library for DwarFS for rust.
And I also noticed some problems in RunImage container when using AppImage runtime with fuse and SquashFS native libraries.
The problem was that some vulkan layers didn't work (e.g. mangohud overlay didn't show up in games and vkbasalt effects didn't work) and these problems didn't occur when remounting the container image with squashfuse.

Do you optimize all of the embedded binaries with -Os -s?
I don't think that Vulkan issues are caused by the runtime, more like by the contents of the AppImage (probably something is missing?)

Now, that is really interesting!

Probably almost all possible methods of size reduction have already been applied, unless i can also rebuild dwarfs-universal and all static libraries it depends on with the same size optimizations. And i can also try to remove support for some compression algorithms, though I wouldn't really want to cut functionality in this regard.

At the moment, dwarfs-universal is taken from the builds of the official repository.

Unfortunately I noticed a big speed degradation of squashfuse built on musl libc, so I had to build it on glibc to prevent this. As a result, the uruntime size is a bit bloated.
@probonopd
I also tested AppImage type2-runtime (which is also now compiled on musl) and it too has a lower read speed than the squashfuse+glibc.

Interesting observation @VHSgunzo. Do you have measurements? What kind of percentage are we talking about? I wonder whether Cosmopolitan would perform better

https://github.com/AppImage/AppImageKit/issues/1105

@probonopd

Do you have measurements? What kind of percentage are we talking about?

the difference in my test was ~15%
squashfuse+glibc vs type2-runtime
The size of the image along with the runtime is ~1.97 GB, when decompressed it is 5.07 GB, zstd compression, 1 level and 128 Kb block size used.
The test consists of simply copying the contents of the mounted image to tmpfs (with pre-clearing of the page cache, dentries and inode cache).

I wonder whether Cosmopolitan or Chimera would perform better

I'd be interested in that too, but I haven't compared them yet

Chimera

I don't understand why it'd be relevant, fBSD utils are more bloated compared to Busybox, Toybox or Sbase.
And Chimera is also Musl-based, just like Alpine. I don't think there will be any advantages to using a less-tested system, which is also unstable, and not prepared for containers.
Of course, this is just my opinion, as someone who's daily driven FreeBSD, Alpine Linux (until recently) and Chimera.
Actual testing should be done nonetheless

I don't understand why it'd be relevant, fBSD utils are more bloated compared to Busybox, Toybox or Sbase.

Because chimera uses mimalloc: https://github.com/orgs/chimera-linux/discussions/2480

I don't understand why it'd be relevant, fBSD utils are more bloated compared to Busybox, Toybox or Sbase.

Because chimera uses mimalloc: https://github.com/orgs/chimera-linux/discussions/2480

You can use jemalloc too in Alpine, @probonopd is already compiling the programs we need in an Alpine CI, he'd have to add the jemalloc package and make all programs link with it.


Undefined symbol: OBJC_CLASS$_GREYActionBlock

I am implementing some functional UI tests using EarlGrey 2 where I need to get the text value of an element and check if it matches with X string. I followed this example but I am getting this error when running the UI test:
Undefined symbols for architecture x86_64:   "_OBJC_CLASS_$_GREYActionBlock", referenced from:       objc-class-ref in GetText.o ld: symbol(s) not found for architecture x86_64 clang: error: linker command failed with exit code 1 (use -v to see invocation)
Here's the method for getting the text from label
public class Element {
  var text = ""
}

func grey_getText(_ elementCopy: Element) -> GREYActionBlock {
  return GREYActionBlock.action(withName: "get text", constraints: grey_respondsToSelector(#selector(getter: UILabel.text))) { element, errorOrNil -> Bool in
    let elementObject = element as? NSObject
    let text = elementObject?.perform(#selector(getter: UILabel.text), with: nil)?.takeUnretainedValue() as? String
    elementCopy.text = text ?? ""
    return true
  }
}

method call:
lazy var usernameText = EarlGrey.selectElement(with: grey_accessibilityID("username-label"))

func getUsername() -> String {
    let element = Element()
    usernameText.perform(grey_getText(element))
    return element.text
 }

 func isUserLoggedIn(username: String) -> Self {
    userProfileDrawer.perform(grey_tap())
    GREYAssert(getUsername() == "u/\(username)", "Username: \(username) did not upload correctly")
    return self
 }

Am I missing something? Thanks in advance!

Were you able to fix it?


Remove redundant API level checks, modernise some code

@PatrykMis
Talkback works in this PR, however because we are fully targeting API 31 now the permission READ_PHONE_STATE is no longer granted automatically/by default (as far as I can tell).
Because I don't know how to; a way to ask the user to grant access to the Phone permission (for READ_PHONE_STATE) needs to be implemented. It should be trivial.
Otherwise, the app will crash as the previous behaviour was it assumed it already had it so the "permission check" always returned true. READ_PHONE_STATE is considered a "dangerous" permissions and Play Store apps aren't allowed to have this by default. A user-facing pop up asking to grant the Phone permissions is necessary here to prevent Talkback from crashing upon launch.
The check is in https://github.com/talkback-foss-team/talkback-foss/blob/main/talkback/src/main/java/com/google/android/accessibility/talkback/CallStateMonitor.java#L167
It needs to be made to ask the user for Phone permission and return that with error handling as it simply assumes it has it already by returning true.
For accessibility purposes, the READ_PHONE_STATE permission needs to be granted by default. Seems to require the service to be rewritten.
The crash if the permission isn't granted:
2021-12-05 06:17:48.850 2715-2715/com.android.talkback E/AndroidRuntime: FATAL EXCEPTION: main
    Process: com.android.talkback, PID: 2715
    java.lang.SecurityException: getCallState: Neither user 10105 nor current process has android.permission.READ_PHONE_STATE.
        at android.os.Parcel.createExceptionOrNull(Parcel.java:2425)
        at android.os.Parcel.createException(Parcel.java:2409)
        at android.os.Parcel.readException(Parcel.java:2392)
        at android.os.Parcel.readException(Parcel.java:2334)
        at com.android.internal.telecom.ITelecomService$Stub$Proxy.getCallStateUsingPackage(ITelecomService.java:2578)
        at android.telecom.TelecomManager.getCallState(TelecomManager.java:1803)
        at android.telephony.TelephonyManager.getCallState(TelephonyManager.java:5707)
        at com.google.android.accessibility.talkback.CallStateMonitor.getCurrentCallState(CallStateMonitor.java:137)
        at com.google.android.accessibility.talkback.VoiceActionMonitor.getCurrentCallState(VoiceActionMonitor.java:131)
        at com.google.android.accessibility.talkback.eventprocessor.AccessibilityEventProcessor.shouldDropEvent(AccessibilityEventProcessor.java:363)
        at com.google.android.accessibility.talkback.eventprocessor.AccessibilityEventProcessor.onAccessibilityEvent(AccessibilityEventProcessor.java:209)
        at com.google.android.accessibility.talkback.TalkBackService.onAccessibilityEvent(TalkBackService.java:533)
        at android.accessibilityservice.AccessibilityService$2.onAccessibilityEvent(AccessibilityService.java:2170)
        at android.accessibilityservice.AccessibilityService$IAccessibilityServiceClientWrapper.executeMessage(AccessibilityService.java:2376)
        at com.android.internal.os.HandlerCaller$MyHandler.handleMessage(HandlerCaller.java:44)
        at android.os.Handler.dispatchMessage(Handler.java:106)
        at android.os.Looper.loopOnce(Looper.java:201)
        at android.os.Looper.loop(Looper.java:288)
        at android.app.ActivityThread.main(ActivityThread.java:7839)
        at java.lang.reflect.Method.invoke(Native Method)
        at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:548)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1003)
     Caused by: android.os.RemoteException: Remote stack trace:
        at android.app.ContextImpl.enforce(ContextImpl.java:2178)
        at android.app.ContextImpl.enforceCallingOrSelfPermission(ContextImpl.java:2206)
        at android.content.ContextWrapper.enforceCallingOrSelfPermission(ContextWrapper.java:905)
        at com.android.server.telecom.TelecomServiceImpl.canReadPhoneState(TelecomServiceImpl.java:2324)
        at com.android.server.telecom.TelecomServiceImpl.access$300(TelecomServiceImpl.java:78)

Hard code the newest Android NDK version as it would constantly get an old version (21)
Replaced API 30 intent flag from PendingIntent.FLAG_UPDATE_CURRENT to API 31's PendingIntent.FLAG_IMMUTABLE
Update some dependencies
Enable R8 full mode optimisation
Set manifest file to app sandbox 2 and use new biometrics API
Remove lots of redundant checks and old API hacks as we are not
targeting API <25

Signed-off-by: June<EMAIL_ADDRESS>

because we are fully targeting API 31 now the permission READ_PHONE_STATE is no longer granted automatically/by default (as far as I can tell).

It seams for accessibility service like TalkBack, this permission is beeing granted automatically while enabling it.

You were right. Phone permission must be granted before enabling accessibility service; otherwise, the service is constantly crashing.

Closing as it's no longer needed, but thank you for your useful suggestions.


cannot port forward

please give me full guide about port forwarding on wg server on windows

Hey @arshad2374, I'm afraid I don't completely understand your question. The main port forwarding that I'm aware of in reference to WireGuard is from your router to your WireGuard server machine (regardless of Window or Linux).
Most people with home internet will have all of their personal devices behind NAT, meaning that each device gets assigned an internal IP address, and your whole network is represented by one external IP address (aka WAN address), managed by your router. In order to connect to your home network from outside, you must direct requests to your WAN IP. However, if you want your requests to be forwarded to a particular device (in this case, the device running the WireGuard server), you need to specify a port so that your router knows how to direct the request.
WireGuard normally uses port 51820, although that's customizable. Whichever port you pick, you should forward it from your router to your WireGuard server machine. Unfortunately, that process will be different depending on your router. The README has one example for FiOS routers, pictured below, if it helps. But you'll probably be better off Googling: <your router brand> port forwarding.

I apologize if you were referring to another kind of port forwarding that I am unaware of!

@arshad2374, did you get the information you were looking for? If so, I will close this issue.

I am going to close this issue as no further questions or details have been provided by the submitter.


Any object user / game / service / team must have unique uuid field

It's need for export/import objects from to simular systems

@sea-kg Do you suggest changing the id to a uuid?
if yes, then no problem)

@IgorPolyakov
Anyway need uuid - will be changed id to uuid field or added just a new field - don't matter. How are will be right?

I would change the number ID to uuid. At least so as not to duplicate the keys
It's safer that way. In addition, postgres normally works with uuid as the primary key

Okay

done in #18


CMake: don't set a predefined build type

Description
The CMakeLists.txt should not set a CMAKE_BUILD_TYPE by default because
multiple generators do not use this variable (e.g. Visual Studio).
This makes the build process inconsistent between the CMake
configuration and the IDE.
Example with current behaviour:

User runs CMake for Visual Studio, CMake generates Release
User selects Debug in Visual Studio

The build process is inconsistent.
Tasks

[x] Tested on Linux

I disagree. It's still inconsistent if you set nothing and then get multiple options. It's just expected behavior that some generator will provide multiple types.

I agree that CMake projects should not hardcode such details and that it makes more sense to default to a debug build.

This has been closed 3.5 years ago 🙂
If you think we should change something, maybe open a new issue about the topic?


Going to page 2 in PDF takes 2 mins+ in IE11 for a given pdf file

Link to PDF file (or attach file here):
TextTest.pdf
Configuration:

Web browser and its version:
IE11

Operating system and its version:
Win10

PDF.js version:
Latest in https://mozilla.github.io/pdf.js/web/viewer.html

Is an extension:

Steps to reproduce the problem:

go to https://mozilla.github.io/pdf.js/web/viewer.html
Upload this pdf file:
TextTest.pdf
Try going to page 2

it will take 2mins +
What is the expected behavior? (add screenshot)
Should be immediate, like in other browsers.
What went wrong? (add screenshot)
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

banguero,
https://github.com/mozilla/pdf.js/issues/7001 - I think it's same problem
This bug from 17 Feb 2016. From 1.5 to 1.7 no changes. Performance in IE11 veeeeeeery sloooooow

Please note that IE11 is really old, and haven't received meaningful updates/improvements in a couple of years. Besides, it's general JavaScript performance was never great to begin with, a problem that's further compounded by its lack of support for many modern JavaScript features which thus requires polyfills (which are slower than native code).
One possible explanation, besides general slowness of the browser itself, is that the amount of asynchronous code in PDF.js is difficult for IE11 to handle. However, the PDF.js code has/will only become more async over time (also the case with many other JavaScript applications), so there's nothing that can be done here.
This issue is essentially asking for PDF.js developers/contributors to somehow improve the general JavaScript performance of the IE11 browser itself, which is obviously not possible unfortunately; hence closing this as INVALID seems like the appropriate resolution here.


move or consolidate scripts

We now have 5 scripts in the root directory:

computed_kubectl.sh
down.sh
max_k8s_version.sh
up.sh
update.sh

Could we consolidate these into a bin directory?

There is an issue for this on the board already.
So, yes!
On Jun 30, 2017 1:14 PM, "Joe Julian"<EMAIL_ADDRESS>wrote:

We now have 5 scripts in the root directory:

computed_kubectl.sh
down.sh
max_k8s_version.sh
up.sh
update.sh

Could we consolidate these into a bin directory?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/samsung-cnct/k2/issues/592, or mute the thread
https://github.com/notifications/unsubscribe-auth/AApXmsOtItMBoTkVt8Lr1xBDjhgVbgBcks5sJVc6gaJpZM4OLAcu
.

dup of #525


Bash if condition fix in a few scripts

Replaced all 'if [ $(...) -ne 0 ]; then' style if conditions with 'if ! (...) ; then' across a number of bash scripts.
Fixes issue #174

Kicked off two Jenkins jobs (8 and 9):

https://ci.adoptopenjdk.net/view/work in progress/job/openjdk8_x86-64_linux_mani_pr_175/2/console
https://ci.adoptopenjdk.net/view/work in progress/job/openjdk9_x86-64_linux_mani_pr_175/2/console

OpenJDK8 successful
...
openjdk version "1.8.0-internal"
OpenJDK Runtime Environment (build 1.8.0-internal-jenkins_2017_10_02_19_12-b00)
OpenJDK 64-Bit Server VM (build 25.71-b00, mixed mode)

Removing unnecessary files now...
Fetching the first tag from the OpenJDK git repo...
Currently at '/home/jenkins/workspace/openjdk8_x86-64_linux_mani_pr_175/openjdk/build/linux-x86_64-normal-server-release/images'
moving j2sdk-image to jdk8u144-b01
Finished removing unnecessary files from jdk8u144-b01
Archiving the build OpenJDK image...
OpenJDK repo tag is jdk8u144-b01
Your final .tar.gz was created at /home/jenkins/workspace/openjdk8_x86-64_linux_mani_pr_175/openjdk/build/linux-x86_64-normal-server-release/images
Moving the artifact to /home/jenkins/workspace/openjdk8_x86-64_linux_mani_pr_175/OpenJDK8_x64_Linux_201702101909.tar.gz
All done!
Triggering projects: openjdk8_test_x86-64_linux
Archiving artifacts
Recording test results
None of the test reports contained any result
Finished: SUCCESS

OpenJDK9 successful

'/mnt/jenkins/workspace/openjdk9_x86-64_linux_mani_pr_175/openjdk/build/linux-x86_64-normal-server-release/images/jdk' found

openjdk version "9-internal"
OpenJDK Runtime Environment (build 9-internal+0-adhoc.jenkins.openjdk)
OpenJDK 64-Bit Server VM (build 9-internal+0-adhoc.jenkins.openjdk, mixed mode)

Removing unnecessary files now...
Fetching the first tag from the OpenJDK git repo...
Currently at '/mnt/jenkins/workspace/openjdk9_x86-64_linux_mani_pr_175/openjdk/build/linux-x86_64-normal-server-release/images'
moving jdk to jdk-9+181
Finished removing unnecessary files from jdk-9+181
Archiving the build OpenJDK image...
OpenJDK repo tag is jdk-9+181
Your final .tar.gz was created at /mnt/jenkins/workspace/openjdk9_x86-64_linux_mani_pr_175/openjdk/build/linux-x86_64-normal-server-release/images
Moving the artifact to /mnt/jenkins/workspace/openjdk9_x86-64_linux_mani_pr_175/OpenJDK9_x64_Linux_201702101909.tar.gz
All done!
Archiving artifacts
Recording test results
None of the test reports contained any result
Finished: SUCCESS

@gibfahn it worked on Jenkins but it didn't work on local environment (for me at least - guessing might have been the same for others).
But we rarely build locally - is that right?

@gibfahn it worked on Jenkins but it didn't work on local environment (for me at least - guessing might have been the same for others).

Okay, it seems like you get an error (on my shell), but the if condition doesn't run, so I guess that explains why it didn't break Jenkins.
$ if [ a -eq 0 ]; then echo true; else echo false; fi
[: integer expression expected: a
false

Yes thats how I remember seeing in the log files, when I noticed the issue

@neomatrix369 I'd strongly suggest putting it back the way it was, with the command on a previous line and testing $? as it's vastly more readable.

@sxa555 I'll revert it to the original lines, as mentioned earlier I only changed it as shellcheck was raising warnings on those lines.
@gibfahn What is your view on the original implementation?

I'd strongly suggest putting it back the way it was, with the command on a previous line and testing $? as it's vastly more readable.

I disagree that it's more readable. Using $? is arguably slightly more readable for those familiar with bash, but less readable for people familiar with any other programming language.

@gibfahn What is your view on the original implementation?

I prefer it as it is now. If the worry is long lines, then break it up like:
     if ! (bash ./configure --prefix="${WORKING_DIR}"/"${OPENJDK_REPO_NAME}"/installedfreetype "${FREETYPE_FONT_BUILD_TYPE_PARAM}" && $MAKE all && $MAKE install); then

becoming:
if ! (./configure --prefix="${WORKING_DIR}"/"${OPENJDK_REPO_NAME}"/installedfreetype "${FREETYPE_FONT_BUILD_TYPE_PARAM}" &&
  $MAKE all && $MAKE install); then

I don't see how this
./configure --prefix="${WORKING_DIR}"/"${OPENJDK_REPO_NAME}"/installedfreetype "${FREETYPE_FONT_BUILD_TYPE_PARAM}" &&
  $MAKE all && $MAKE install
if [ $? -ne 0 ]; then

is cleaner.
For shorter conditions I think the if ! (command) is much cleaner, i.e.
if ! (wget "${JAVA_SOURCE}"); then

rather than
wget "${JAVA_SOURCE}"
if [ $? -ne 0 ]; then

I'd be interested to hear what others think though.

I see the same clarity with the  if ! (command), although now our scripts have two different conventions, so best to agree to one and I'll be happy to steer it to the agreed one.

@sxa555 @gibfahn Should we do a drive to standardise/unify this implementation across all our scripts, once we have come to an agreement? I#ll be happy do change it for us.

SGTM


Fix Compatibility with React Native 56.0

can this be processed?

@NitroGhost The repository seems not to want any contributors, so you will have to wait until the returns to maintain this library (or create your own fork).

At least there should be a guide for common issue with RN 56.0


http: implement skip_if_present in HeaderValueOption

Description: Adds a skip_if_present option to HeaderValueOption so that response_headers_to_add and request_headers_to_add can optionally skip setting/appending a header if the header is already present in an HTTP message.
Risk Level: low
Testing: added
Docs Changes: added
Release Notes: added
Fixes #9525

lgtm pending @derekargueta deciding if he wants to go ahead and move append as well. @snowp do you want to take a final pass?


Refactor Phalcon\Annotations

Refactor Phalcon\Annotations

I've started here: https://github.com/les-enovateurs/phalcon/tree/T2-annotations
But there is a C part so if someone is more expert than me, you can take this part.
I will be focused on other more simple Phalcon part.

Resolved in https://github.com/phalcon/phalcon/pull/302

Resolved in https://github.com/phalcon/phalcon/pull/302


Instrumentation

This pull request adds the ability to trigger faults during the coverage testing so that we can exercise each of the failure paths and verify their function.   Because there are many fault paths, this request is going to take a while to complete.
This is currently a test pull built on classhints2 and serves as the test bench to verify that the final core rewrite is actually covering all of the paths.
In order to trigger faults, the compiler is set to compile in fault hooks on enable coverage.   This converts all of the JP_PY_TRY and JP_TRACE_IN statements into fault hooks.  It uses C++ const expression to convert the name of the function into a hash code which can be triggered with a specific fault.  As a coverage method this doesn't help much as the coverage tool recognizes the difference between our fake fault triggers and the real condition and thus calls it just another untested branch.   But it does allow us to test some of the more deeply buried error handling.  Most faults will trigger a SystemError which represents Pythons error for something just short of fatal.  The test bench then attempt to trigger this fault with SystemError then again with any errors that are expected and finally again without an error to make sure we hit each path and managed to exit unscathed.
Thus far it has turned up about a dozen bad error handling paths and exposed about an equal number of bugs by allowing an easy way to generate off normal conditions or unusual conditions (using partially unpopulated objects).  Thus it is likely worth the effort.

Codecov Report

Merging #592 into master will decrease coverage by 1.73%.
The diff coverage is 40.85%.

@@            Coverage Diff             @@
##           master     #592      +/-   ##
==========================================
- Coverage   52.42%   50.69%   -1.74%
==========================================
  Files          98      107       +9
  Lines        7656     8021     +365
  Branches     3459     3588     +129
==========================================
+ Hits         4014     4066      +52
- Misses       1316     1443     +127
- Partials     2326     2512     +186

Impacted Files
Coverage Δ

native/common/include/jp_encoding.h
33.33% <ø> (ø)
:arrow_up:

native/common/include/jp_array.h
100% <ø> (ø)
:arrow_up:

native/common/include/jp_arrayclass.h
100% <ø> (ø)
:arrow_up:

native/common/include/jp_platform_linux.h
43.75% <ø> (ø)
:arrow_up:

native/python/pyjp_value.cpp
39.01% <ø> (+2.22%)
:arrow_up:

native/common/jp_encoding.cpp
64.65% <ø> (ø)
:arrow_up:

native/common/include/jp_env.h
33.33% <ø> (ø)
:arrow_up:

native/common/include/jp_value.h
100% <ø> (ø)
:arrow_up:

native/common/include/jp_tracer.h
0% <ø> (ø)
:arrow_up:

native/common/include/jp_exception.h
25% <ø> (+5%)
:arrow_up:

... and 112 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 602050f...9c26f4d. Read the comment docs.

@marscher  I think this one should likely go to review and start another PR as I finally lost the coverage diffs which means I will need another baseline to establish further progress.  Or if you want me continue on this one until I run out of bugs (may be a few weeks at current rate) I would have to perform a force push.  Sadly 1100 tests is still not enough to get us over 60% coverage.
We also need to clear that appveyor branch.  I am not sure what it is doing but since it appeared nothing is clearing the CI.
Tasks done:

Add fault system to trigger faults.  Faults are triggered for JP_TRACE, JP_PY_TRY,  JP_FAULT, and JP_JAVA_CHECK to try to get at parts of the code.
Fixed float OverflowError to follow Java rules.
Discovered a lot of strange edge cases in the type system when I pushed the test matrix out.  Still a fair number of FIXMEs that will need to be dealt with.
Reorgs the tests to make it easier to use a diff tool to copy between different types (oh my was that boring)
Removed piles of dead code.
Reworked exceptions from Release* methods.  Exceptions are not permitted in those paths, so I had to do some rework to handle it better.
Reorged a portion of the test harness.  We had a lot of redundant harness code that needed to be consolidated.   I put everything that had to do with fields and common methods into jpype.common.Fixture.

Yes.  I will remove those.   Unfortunately, debug prints do not show in Pytest unless you give the “-s” switch so they tend to get lost.

The OSX Travis job fails, because we need to manually install ant on OSX now. Seems like they have changed the image/target.

The OSX image was pretty busted a while back.   I disabled one copy of it and forced the brew pattern to update which was the required kludge.   But that added 5 minutes to the build and then they updated it again.   I will make another run at it tonight.

On 3/9/20 10:53 PM, Karl Nelson wrote:

Thrameos commented on this pull request.

@@ -6,6 +6,9 @@ This changelog only contains changes from the first pypi release (<IP_ADDRESS>) o

Next version - unreleased

java.util.Map conforms to Python abc.Mapping API.

float properly follows Java rules for conversion from double.

floats outside of range map to inf and -inf.

I think that it is just adding a bypass flag on PyJPNumber_new.   But I want to do this as a separate PR rather than part of a huge change ball.

Sounds more reasonable.

On 3/9/20 10:51 PM, Karl Nelson wrote:

Thrameos commented on this pull request.

+-include $(DEPFILES)
+""", file=fd)
+
+
+# Customization of the build_ext
+class BuildMakefileCommand(build_ext):

"""
Override some behavior in extension building:

handle compiler flags for different compilers via a dictionary.

try to disable warning -Wstrict-prototypes is valid for C/ObjC but not for C++

"""

extra compile args

copt = {'msvc': [],

Okay I will have to find out which disttools is causing the problem.  I checked on Ubuntu under windows and it works so will have to make a wider scan of versions to see if I can find a stable pattern.
I am not very happy with Python disttools.   Their API requires overriding stuff and it doesn't seem very stable.  I can make it do stuff that I need but it almost always needs to go into the machinery  which seems like poor planning on their part.

the distutils lib is crappy indeed. We committed our fate into
setuptools, as it tries to solve most of distutils issues. But as it
tries to preserve as much (weakly designed) API as possible, it cannot
solve all problems. The most horrible thing is, that distutils classes
are not even new-style objects, so we cannot rely on super() calls for
invoking parent implementations.
Just out of curiosity, would Ubuntu notice/behave differently when
emulated under Win?

Ubuntu under WLS is an improvement over cygwin as it offers all the capabilities of Ubuntu with good package manager and at the same time lets you run windows commands just by calling the exe.   The only differences have been with some kernel level stuff.  For example docker does not work from emulated linux, so I have to run a separate windows docker system.   The good news is I can run the docker command from within the linux, but it launches the windows container manager.  It is good enough to allow me to test both windows and linux stuff from the same window and often within the same command script so it is a big time saver.

I applied autopep8 and added comments to address review concerns.   The rest were transferred to issues. Then once the CI clears I will merge.


fix incorrect  xxMapper class name in example

This is a simple fix to one of the examples where PersonMapper was used instead of MyClassMapper (as the class name would dictate as well as the text leading up to the example).
Tip: If you prefer to use MyClass.fromJson over MyClassMapper.fromJson, add the fromJson and fromMap methods directly to your class like this:

Thanks


Barbarian assault: plugin enhancements

Includes some features to enhance the barbarian assault minigame gameplay experience.
Barbarian Assault Configuration

Show call change timer
Shows time to next call change

Chat colors
Enable game chat colors on messages announced by this plugin
Show death times
Shows the time all penance monsters of a certain type are killed in an info box, the chat, or both

Show duration
Displays duration after each wave and/or round

Reward points
Gives summary of reward points in the chat after each wave and/or round

Rewards breakdown
Gives summary of advanced points breakdown in the chat

Show eggs loaded on hopper
Displays the amount of loaded eggs on cannon hoppers

Highlight mode
Define the mode of all inventory highlights
     Outline         Overlay

Attacker
Highlight arrows
Highlights arrows called by your teammate

Highlight arrow color
Configures the color to highlight the called arrows
Highlight attack style
Highlights the attack style called by your teammate

Highlight attack style color
Configures the color to highlight the attack style
Defender
Highlight called bait
Highlights bait called by your teammate

Called bait color
Color to highlight the bait called by your teammate
Show runner tick timer
Shows the current cycle tick of runners

Collector
Show number of eggs collected
Displays current number of eggs collected

Healer
Highlight called poison
Highlights poison food called by your teammate

Called poison color
Configures the color to highlight the correct poison food
Highlight incorrect notification
Highlights incorrect poison chat notification

Notification color
Configures the color to highlight the notification text
Show number of hitpoints healed
Displays current number of hitpoints healed

Show teammate health bars
Displays a health bar where a teammate's remaining health is located

Ground Items
Barbarian assault
Barbarian Assault mode
Enables or disables the mode inside barbarian assault
Highlight eggs
Highlight egg colors on the ground when performing the Collector role
The following gif loops between the available configurations:

Called only
Highlight all
Highlight none
Barbarian assault mode off/on

Highlight bait
Highlight bait dropped on the ground when performing the Defender role
Highlight logs and hammer
Highlight logs and hammer when performing the Defender role
Menu Entry Swapper
Item Swaps
Collection bag
Swap Look-in with Empty on the collection bag inside barbarian assault
Collector horn
Swap Use with Tell-defensive on the collector horn inside barbarian assault
Collector eggs
Swap Use with Destroy on eggs inside barbarian assault
Get-rewards
Swap Talk-to with Get-rewards for the Commander Connad
NPC Swaps
Commander Connad
Swap Talk-to with Get-rewards for the Commander Connad

It might be preferred to put this on the pluginhub, since this looks like a very wide reaching and intrusive change.

I initially had it setup as an external plugin. But the ground items highlighting was conflicting with the existing ground items plugin, with features (and code) having to be duplicated.
While being on pluginhub, is the conflict solvable at all?

Highlighting ground items in a plugin is not too hard / it is not unreasonable to duplicate the code for that. It is basically an overlay + also hooking to the spawn/despawn events I think? and is not too much code; I would recommend just duplicating that.

Code duplication was not that bad tbh, the main issue was it conflicting with the ground item plugin
Should i ask the users to manually put all items (hammer/logs, red/green/blue/yellow eggs) on its hidden item list?

You could, or if you really wanted you could just append it to the hidden item list from your plugin (it is just a config opt of the grounditems plugin you could modify).

I'll append it to the list then, thanks. The main issue i see is the hammer/logs that are also used outside barbarian assault
What about the new menu entry swaps? Should it go into the menu entry swap plugin, or the external plugin?

I don't see why they can't just be in the external plugin?

Requested pull request as an external plugin: runelite/plugin-hub/pull/639
Closing


[guidelines] Writing tests for the Azure SDK

This PR contains a single file at documentation/writing_tests.md that contains the proposed test guidelines. Please submit feedback!
TODOs:

~Update the template project to reflect these guidelines, once enough feedback is obtained.~ Done.
[guidelines] Write down how to use the VSCode debugger with the Azure SDK #8088
~Move common test utilities to a shared folder/project, then add documentation on how to use these utilities.~ This could be done after merging an initial version of the guidelines.
~A PR polishing what's already merged with an introduction and other final touches~ Done!.

PRs to this PR:

~[guidelines] [writing_tests.md] Recommended tools https://github.com/sadasant/azure-sdk-for-js/pull/14~ merged.
~[guidelines] [writing_tests.md] Structure of tests https://github.com/sadasant/azure-sdk-for-js/pull/15~ merged.
~[guidelines] [writing_tests.md] Writing test cases https://github.com/sadasant/azure-sdk-for-js/pull/16~ merged.
~[guidelines] [writing_tests.md] Engineering setup https://github.com/sadasant/azure-sdk-for-js/pull/17/files~ merged.

Self feedback (still pending resolution):

Done: The scripts unit-test and integration-test should ~only do what they're supposed to do. Only integration tests should reach to live resources, and these scripts should force this to happen.~ be renamed into test and test-ci.

Do you think it would be beneficial to recommend/enforce any minimum % of code coverage?
It'll be hard to achieve high % of code coverage, especially if we're focusing mostly on integration tests, but wanted to hear everyone's thoughts around this.
I've personally seen good and bad from enforcing coverage, specially if the threshold is too high, it may encourage some bad practices. On the good side, your project ends up better guarded against regressions.

Do you think it would be beneficial to recommend/enforce any minimum % of code coverage?
It'll be hard to achieve high % of code coverage, especially if we're focusing mostly on integration tests, but wanted to hear everyone's thoughts around this.
I've personally seen good and bad from enforcing coverage, specially if the threshold is too high, it may encourage some bad practices. On the good side, your project ends up better guarded against regressions.

Maybe a polite alternative is to have us check in some kind of coverage summary file (similar to the api extractor output) so that it's obvious if we're regressing coverage / adding lots of new untested surface but at the same time if it's not meaningful to test that surface, it's not enforced to be tested.
I remember there used to be a CI bot that would do this and tell you how your PR would affect code coverage. Rather than enforcing a target, it just makes you (and reviewers) more aware of the impact you're having.

@sadasant.. might have to provide the test inputs to the karma.conf in the template  to get the CI green!

@ramya-rao-a Hi! I think that, when we have time to come back to this, one good way to follow up and fix our projects could be to make new ESLint rules to help us detect test-guidelines related inconsistencies (at least see what makes sense to automate and what not). I wanted to leave this here so that we could pick this up later 🌞

Closing this. See you in space, branch 🚀🪐


After ExplictTarger Color Projection: No color

Two file saved, .mlt and .obj.
This is text of .mlt:

Right now ExplictTarger Color Projection is saving color into vertex color, but I made an update already in which I modified the algorithm to support direct projection on to texture map, and another algorithm to convert vertex color into texture map.
Those will be available in the next update, cheers 👍

Was this updated?  Having the same issue. Even if save the mesh as .glb im not seeing the color


Weird behaviour when font size up and down in WSL

Environment
Windows build number: [run `[Environment]::OSVersion` for powershell, or `ver` for cmd] 1909
Windows Terminal version (if applicable): From 0.10 to latest master

Any other software?

Steps to reproduce
I was playing around since we fixed #5479 recently. But I found that the content became weird and duplicate when I'm scaling font up & down.
Originaly there's only one prompt line. Scale the font to a very large one (I mean large enough to trigger line wraps) and down. Repeat.

Expected behavior
Well, anything less confusing would be good.

Actual behavior
After a couple of times, you got something like this:

I can only reproduce this in WSL. Distro does not seem to matter.

I'm not really concerned about the issue itself. I mean, it would appear very few people would actually do this in real life. What concerns me is the underlying issue that may affects other things. And I don't really get it why Powershell & cmd are exempt from this behaviour.

So I'm thinking this is definitely a /dupe of #5450, and I'm thinking honestly that both of these are related to #3088
This seems less like it's something we're doing, but more an artifact of how bash handles reprinting it's prompt. Compare gnome-terminal with conhost:

As soon as the prompt wraps onto a third line, you bash seems to handle unwrapping particularly poorly.


Weight refunds for staking extrinsics potentially using SortedListProvider

#9507 introduced the SortedListProvider for keeping an ordered list of nominators. For extrinsics using the SortedListProvider this dramatically increased their worst case weight; in some cases though certain logic paths have dramatically less weight. Benchmarks should be updated to document less expensive code paths and give relevant refunds.
(rel: https://github.com/paritytech/polkadot/pull/3872#discussion_r711915842)

I came across this again while thinking about all of this. I think this is quite important, and we should do it before we deploy this on Polkadot.

cc @emostov something for you to consider on the side.

Might also be a good FRAME entry point for @rossbulat

I am happy to be assigned with this and play with the benchmarking.

Looking closer, the only scenario for SortedListProvider::on_update where we should see a significant weight reduction is if the voter does not change bags. The current weight setup  is
	/// - the node to be updated (r) is the head of a bag that has at least one other node. The bag
	///   itself will need to be read and written to update its head. The node pointed to by r.next
	///   will need to be read and written as it will need to have its prev pointer updated. Note
	///   that there are two other worst case scenarios for bag removal: 1) the node is a tail and
	///   2) the node is a middle node with prev and next; all scenarios end up with the same number
	///   of storage reads and writes.
	///
	/// - the destination bag has at least one node, which will need its next pointer updated.

As the above mentions, the other worst cases are that in the source bag the node is the tail or the node is a middle node with a prev and next pointer. This means that in any scenario where the node moves we will see the same count of read/writes for the operations related to the source bag. For the destination bag, the only cheaper scenario is that the bag does not already exist, which I hypothesize to be a relatively rare occurrence.
With this in mind, I propose that SortedListProvider::on_update return a bool or enum indicating if the node changed bags (this is an easy change, since do_rebag returns an option indicating this). We can then make the refund conditional on the returned value.
If this approach sounds ok, I see 3 parts to implementing this, (which could be broken into separate PRs)

change the signature of SortedListProvider::on_update to return a bool or enum indicating if the node changed bags
create new benchmarks for staking extrinsics that use SortedListProvider::on_update. The benchmarks would be a scenario where the node doesn't change bags
implement refund logic by return the weight from the new benchmarks if no refund happened


Get nix compiling on rust beta

Blocked on https://github.com/rust-lang/cargo/issues/1478

Rust is currently compiling on beta, but there is more work to do: #98


Update MD file on new branch 2017-34-17_15-12-02

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


canceling requests

Hi, I have not found any way in TRON to cancel requests.
Is there any API for that?

Hey!
Sure, each performRequest method returns Alamofire.Request, which can be cancelled:
let token = request.perform(withSuccess: ..., failure: ...)

token.cancel()

If you use Rx extension for TRON, requests are also automatically cancelled when subscriptions get disposed.


JAWS does not read the tab text when switching between tabs in a group using the arrow keys.

Bug Report
Description
JAWS does not read the tab text when switching between tabs in a group using the arrow keys.
Steps to Reproduce

TBD

Additional Context / Screenshots
Identified as part of functional verification on this PR.
Expected Behavior
JAWS should read tab text when switching between tabs ina group using arrow keys.
Environment

Component Name and Version: terra-tabs
Browser Name and Version: TBD

Duplicate to https://github.com/cerner/terra-framework/issues/482


WFLY-9678 Expand clustering test cases to 3-node cluster

The core of the problem is that since all cases run against 2-node cluster a get(..) going remote is never being tested given that numOwners defaults to 2 so both nodes are always an owner.
Jira
https://issues.jboss.org/browse/WFLY-9678

@msve I have not looked in any detail on this yet, but I spoke to @pferraro in Brno, and he mentions that this PR does not introduce any new functionality. It is mainly internal refactorings to get this new test stuff up. However it

brings in a jgroups upgrade needed for later work
replaces some protocols
drops custom protocols (whatever they may be)
possibly more stuff I've not seen yet

Does QE need analysis etc. for these?

@kabir the comment is intended for https://github.com/wildfly/wildfly/pull/10746, in order to move forward with other PRs, we need to incorporate each PR in another one...

@rhusar I am sticking the hold label on this until  #10746 gets merged

@rhusar needs a rebase

Yes, definitely related, started to fail frequently after the Infinispan upgrade merge (#10746).
Caused by: org.infinispan.util.concurrent.TimeoutException: Timed out waiting for topology 10

Retest this please

Let me mark this as hold until we have a fix for for https://issues.jboss.org/browse/WFLY-9890 -- which this PR on our CI can reliably reproduce.

Updated the branch to test with Infinispan upgrade to 9.2.0.Final.

retest this please

@pferraro Please approve.
We need this to prevent issues such as WFLY-9920/WFLY-9890 :-/

@wildfly/mergers-full this is ready.


[Feature]: Update supported Algorand ASA list on Ledger Live UI.

Library name
https://github.com/LedgerHQ/app-algorand/blob/56c42a91c7113ee4be38d17f425d8ad783fc7a75/app/src/algo_asa.c#L52
Describe your feature request
Multiple assets have been added to the ASA list for Algorand, that do not show up on the ledger live UI. APP supported, just not reflecting in the UI of Ledger live mobile / desktop.

The list showing the assets not reflected.

Hi @JakeNexus
sometimes this bug occurs.
Best thing to do to resolve it is open the defi protocol page and let the agent on the live chat know.
They can then make a proposal to complete your request and fix the bug via Defi Protocol


Master

Issue #44

Hello @AndrewJWashington, Thank you for submitting the Pull Request !

In the file neurodsp/burst.py, following are the PEP8 issues :

Line 121:1: W293 blank line contains whitespace
Line 136:5: E303 too many blank lines (2)
Line 136:27: E231 missing whitespace after ':'
Line 137:32: E231 missing whitespace after ':'
Line 138:31: E231 missing whitespace after ':'
Line 139:32: E231 missing whitespace after ':'
Line 139:49: E226 missing whitespace around arithmetic operator
Line 140:23: E231 missing whitespace after ':'
Line 140:35: E226 missing whitespace around arithmetic operator

Do see the Hitchhiker's guide to code style

Ready to merge!

Tutorial adds new data to satisfy issue #43


Dropdown doesn't close on click outside fix

Hi there,
Dropdown component doesn't close on click outside in my app.
I debugged a little bit problem and found that componentWillUpdate uses incorrect.
It accepts nextProps and nextState and not prevState and nextState
https://facebook.github.io/react/docs/component-specs.html#updating-componentwillupdate

Correct, you are totally right!!
Can you also update the File components/menu/Menu.jsx, as that's the one I used as base, and incurres in the same error?

Probably we changed the method name and forgot to update the arguments. Can you do as @epilgrim says and update the Menu too before merging? :dancer:

@javivelasco , done!

Great, thanks! :)


x/mobile: use GOOS=ios and remove the build tag ios usages

This is a proposal of refactoring. I propose to replace the current build tag ios usages with specifying GOOS=ios. This change means that Go 1.16 or newer will be necessary. So, I'm fine if we have to wait more (e.g., Go 1.17 is released)
Background: GOOS=ios was introduced at Go 1.16. Before this, gomobile used a build tag ios for iOS projects. In Go 1.16 and newer, standard libraries have *_ios.go files, and apparently they also specify // +build ios explicitly at the same time. Then, we don't have an urgent issue around this.
@hyangah, @ydnar, What do you think?

Sure, I think it makes sense.
Would you want the current iOS tests to pass as well?
It would probably need some additional updates, and ideally macOS CI so it doesn't continue to bit-rot.

cc @eliasnaur

Now that Go 1.17 is released (and thus 1.15 has become unsupported), there shouldn't be a reason not to do this. Also see discussion in #47952.


test removing wcf certs

Warnings

:warning:
Please include a description of your PR changes.

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

  Generated by :no_entry_sign: dangerJS against 16a831ba713795307f34a562f7f3ff37f5e2d974


Editor does not update items on URL change

Open playlist with multiple videos
Open editor
While editor is open, delete a video from URL (address bar) and press Enter

Current result:
Deleted interval is still in editor, but pressing j/k does not reach it.
Expected result:
Deleted interval is removed from editor.

Turns out there issue was caused by a duplicate image interval without time, eg:
#i=NduXs68.gif&v=T0rs3R4E1Sk&t=23s;30s&i=NduXs68.gif

Hm.. seems that it still does not work with regular videos: editing URL does not update editor  (https://github.com/lidel/yt-looper/commit/657c94616da65c7bf172137acc676d9e8778ba32 only fixed drag&drop)


Added fingerprint of 'ASSA ABLOY iRevo 06ffff2027' to zigbee-lock driver according to CHAD-8531

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

Looks good. Please use the "squash and merge" option when merging your commits unless you want to squash locally and force push.


DONOTMERGE: Use batchcli

JIRA:
Overview:
Testing:
Roll Out:

[ ] Remember to deploy both s3-to-redshift and s3-to-redshift-fast

@ahsojar made changes to move to sfncli here too.

superseded by #116


Tally: Decrypt Tally with Compensated Guardians

(Followup from #53 )
Allow an encrypted tally to be decrypted when all guardians are NOT present, but at least the required quorum is. Consists of:

An endpoint for a guardian to decrypt compensation shares for one or more missing guardians
An endpoint for a mediator to combine compensation shares into a missing guardian's share

These endpoints will be used in conjunction with the endpoints from #53 as follows:
Consider that 3/5 guardians are present.

Each present guardian (guardians 1-3) decrypts their own share.
Each present guardian decrypts the compensation shares of the missing guardians (e.g. guardians 1-3 each decrypts compensation shares for guardians 4 and 5).
The mediator decrypts the share for each missing guardian, using the compensation shares from step 2.
The mediator decrypts the tally using the shares from steps 1 and 3.

This was completed with some other changes.