Add support for carthage

Closes #137, at least for now.  This is necessary for my work on Aquifer.

Don't merge this yet.  I'm still working on how to make Carthage behave.

This is now ready!  The only thing is that Carthage can't directly build Swiftz for some reason, so any users have to attach it as a dependent subproject to their own.  It'll still fetch it, place it in its proper location, and keep it up to date, though.  :)

The same thing is happening with the Basis.    Weird that both cases work with Travis but not that

Before we do this, if you want to rename the target we had best do it here and now.  Perhaps capital Swiftz?

Sure! I'll do this as soon as I'm back at my keyboard.  :)

Seems only one of the targets got a name change.  No matter.  We can sort this out later when I get around to part 2 or #116.

That is to say :boat:


Resolve "Extend getitem"

Resolves #58.

Hello @mattpitkin! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file psrqpy/search.py:

Line 493:1: W293 blank line contains whitespace

In the file test/query_test.py:

Line 139:1: W293 blank line contains whitespace
Line 142:1: W293 blank line contains whitespace
Line 147:1: W293 blank line contains whitespace


Enhanced support Metadata template data types [hyperlinks]

In the Metadata template data types, the following is documented
https://www.weave.works/docs/scope/latest/plugins/#report-data-structures
Metadata Templates section

dataType specifies the type of data, and determines how the value is displayed. Possible values for this attribute are: “number”, “ip”, “datetime” and “” for strings.

It would be great if there was support for URLs that support direct linking to resources.   The use case is that we have a Node that represents a Deployment of some service.   This service exposes a set of links for administration, logging, etc.
Food for thought
If there was a more pluggable extension model, templates could define their own data types backed by the plugin to support other data types as well.

Hi, thanks for the suggestion!
How do you envisage this would look, both in the code and on the screen?

Hello,
I work with Todd.
We've implemented the feature by first running a plugin that posts metadata or mulit-column data including the dataType: link field. The frontend then conditionally renders either a link or the original component.
Our implementation is here: https://github.com/weaveworks/scope/pull/3105

Resolved by #3105.


VCR+Webmock+Curb+on_body failing

When using all these together, the body stored in the cassette is empty.
I think it's because the Curl::Easy client doesn't set the easy.body_str if you add a callback to easy.on_body

VCR itself doesn't have any curb-specific code.  It just uses APIs provided by WebMock, so I imagine the bug is there.
Can you come up with some example code demonstrating the issue (I've never used curb)?

I'm going to close this since it's a bug in the curb/webmock integration that VCR delegates to.  Please file an issue on the webmock issue tracker.


noticket/create random db for tests

I delete db on resource release, and I am not 100% sure if it is right to do it for Prod and for Tests. Pls, correct me if I wrong.

I delete db on resource release, and I am not 100% sure if it is right to do it for Prod and for Tests. Pls, correct me if I wrong.

You should do it only for Tests. Add Cleanup resource, just like we do for DynamoDB, and add RocksDB test env.


IE11 - vendor.bundle.js - fuction combine() - Unable to get property 'apply' of undefined or null reference

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

@angular/cli: 1.0.0
node: 7.8.0
os: win32 x64
@angular/cli: 1.0.0
@angular/common: 4.0.2
@angular/compiler: 4.0.2
@angular/compiler-cli: 4.0.2
@angular/core: 4.0.2
@angular/forms: 4.0.2
@angular/http: 4.0.2
@angular/platform-browser: 4.0.2
@angular/platform-browser-dynamic: 4.0.2
@angular/router: 4.0.2
npm: 4.2.0
windows 7
IE11
Repro steps.

unknown, it works fine with chrome and firefox. it is only happens in IE.
The log given by the failure.

Format:
Desired functionality.

IE compatible.
Mention any other details that might be useful.

Please visit this website with IE11.
Angular 4 app

Please try adjusting the polyfills.ts file according to your browser requirements.  It is well commented to assist you in the process.

@clydin , thanks for helping.  it solved my problem.  I suggest to update the ReadMe so that other people aware of this.

@clydin  I am beginner in angular development.
Can you please shed some light on adjusting polyfill.js.
Thanks.

@tata-abhi  ,  polyfill.js is well commented, just follow the instruction to install the dependencies and uncomment the relevant lines.

@jefftham  I am getting same issue in IE and Safari , I uncomment  the relevant lines in polyfill.js after this it working on IE but  safari still showing same error.
Can you please suggest how to resolve this in safari.
Thanks

@ruchisamit , I am sorry that i am not familiar with the polyfill support for safari.

In IE 10, I got the following error: (but OK in IE 11)
TypeError: Unable to get property 'ngMetadataName' of undefined or null reference
The property 'ngMetadataName' has been found in vendor.js.
The polyfills.ts is as the following:
/**

This file includes polyfills needed by Angular and is loaded before the app.
You can add your own extra polyfills to this file.

This file is divided into 2 sections:

Browser polyfills. These are applied before loading ZoneJS and are sorted by browsers.

Application imports. Files imported after ZoneJS that should be loaded before your main

 file.

The current setup is for so-called "evergreen" browsers; the last versions of browsers that
automatically update themselves. This includes Safari >= 10, Chrome >= 55 (including Opera),
Edge >= 13 on the desktop, and iOS 10 and Chrome on mobile.

Learn more in https://angular.io/docs/ts/latest/guide/browser-support.html
*/

/***************************************************************************************************

BROWSER POLYFILLS
*/

/** IE9, IE10 and IE11 requires all of the following polyfills. **/
import 'core-js/es6/symbol';
import 'core-js/es6/object';
import 'core-js/es6/function';
import 'core-js/es6/parse-int';
import 'core-js/es6/parse-float';
import 'core-js/es6/number';
import 'core-js/es6/math';
import 'core-js/es6/string';
import 'core-js/es6/date';
import 'core-js/es6/array';
import 'core-js/es6/regexp';
import 'core-js/es6/map';
import 'core-js/es6/weak-map';
import 'core-js/es6/set';
/** IE10 and IE11 requires the following for NgClass support on SVG elements */
import 'classlist.js';  // Run npm install --save classlist.js.
/** IE10 and IE11 requires the following for the Reflect API. */
import 'core-js/es6/reflect';
/** Evergreen browsers require these. **/
// Used for reflect-metadata in JIT. If you use AOT (and only Angular decorators), you can remove.
import 'core-js/es7/reflect';
/**

Web Animations @angular/platform-browser/animations
Only required if AnimationBuilder is used within the application and using IE/Edge or Safari.
Standard animation support in Angular DOES NOT require any polyfills (as of Angular 6.0).
**/
import 'web-animations-js';  // Run npm install --save web-animations-js.

/**

By default, zone.js will patch all possible macroTask and DomEvents
user can disable parts of macroTask/DomEvents patch by setting following flags
*/

// (window as any).__Zone_disable_requestAnimationFrame = true; // disable patch requestAnimationFrame
// (window as any).__Zone_disable_on_property = true; // disable patch onProperty such as onclick
// (window as any).__zone_symbol__BLACK_LISTED_EVENTS = ['scroll', 'mousemove']; // disable patch specified eventNames
/*

in IE/Edge developer tools, the addEventListener will also be wrapped by zone.js
with the following flag, it will bypass zone.js patch for IE/Edge
*/
(window as any).__Zone_enable_cross_context_check = true;

/***************************************************************************************************

Zone JS is required by default for Angular itself.
*/
import 'zone.js/dist/zone';  // Included with Angular CLI.

/***************************************************************************************************

APPLICATION IMPORTS
*/

Can you please suggest how to resolve this for IE 10.
Thanks

In IE 10, I got the following error: (but OK in IE 11)
TypeError: Unable to get property 'ngMetadataName' of undefined or null reference
I have  updated the angular-cli  from 6.0.8 to 6.2.3 (Latest) and it solved the above error.
This link #24339 might be helpful.


Split up the createChildMethod_ refinement.

Java and swift need refinements on createChildMethod_ to merge method info from
parent. There's a subtle difference between swift and java when checking
hasOwnProperty. It's up for debate which platform's behaviour is more
correct and once we've decided on that, we should try to get the two
platforms to match.

Will review.

Ping?

Gonna close this in favour of https://github.com/foam-framework/foam2/pull/1407


chat render bare asterisk

Keybase has a weird display bug on Windows and android clients. the following string contains 3 x-s and 2 asterisks, "x * x * x," but Keybase is only visibly displaying the x-s, but if you copy this text and paste it somewhere else, the asterisks are on the clipboard. where an asterisk is adjacent to something, it's displayed just fine: x* x*x *x
steps to reproduce

type the following string into a chat session without quotes: "* x* * x*x * *x *"
observe the following is displayed: " x xx x *"
observe what is displayed is not what was pasted
copy the displayed chat to the clipboard
paste into another application that accepts text
observe that the original string was pasted

tested with other printable ascii chars, and seems to only be an issue with the asterisk:
Dec Char Bare
32-
33-! !
34-" "
35-# #
36-$ $
37-% %
38-& &
39-' '
40-( (
41-) )
42-* *
43-+ +
44-, ,
45-- -
46-. .
47-/ /
48-0 0
49-1 1
50-2 2
51-3 3
52-4 4
53-5 5
54-6 6
55-7 7
56-8 8
57-9 9
58-: :
59-; ;
60-< <
61-= =
62-> >
63-? ?
etc.

This is because wrapping text between asterisks formats the text as bold. You can prevent this by putting a backslash before one of the asterisks: x * x * x
There are other characters that cause formatting in a similar way. If you look just below the text entry box and to the right on the desktop client you'll see a note about this.

Came to second @kf5grd the middle x is bold because it's between to asterisks, this is expected behavior. #WONTFIX


Make container environment variables accessible as application variables

Prefixes all container env vars with SPIN_VARIABLE. This means that Spin components can get access to these variables if specifically configured in the Spin manifest to have access. For example, say a a component wants access to the KUBERNETES_SERVICE_ADDRESS container environment variable, it would configure this in the Spin.toml as follows:
[variables]
kubernetes_service_address = { required = true }

[component.example]
source = "target/wasm32-wasi/release/example.wasm"
allowed_outbound_hosts = []
[component.example.variables]
kubernetes_service_address = "{{ kubernetes_service_address }}"

Then, it can be accessed in the Spin app using the variables SDK:
let addr = variables::get("kubernetes_service_address")?;

Should affect/reflect the "Configuring Runtime Options" SKIP https://github.com/spinkube/skips/pull/4

I updated the implementation to check the spin.toml for the variables first and only prefix the associated env vars with SPIN_VARIABLE. For casing, it is expected that the container env vars are the uppercased version of the application variable. Spin variables must always be in lowercase snake ID format but for the CLI must be provided by an uppercased env var. For example: SPIN_VARIABLE_FOO_BAR spin up works but SPIN_VARIABLE_foo_bar spin up` does not work.

Just rebased. @Mossaka @jsturtevant @devigned @radu-matei this is ready to review.


Create ic_flags_vietnam.xml

38B61D99-CA86-4F91-ADE2-6F1FCC4046C5

Hi @trustwalletvn there doesn't appear to be any data in this PR.
Recommend you take the flag SVG code I posted previously, create a .svg file with it, then do 'Import Vector' in AndroidStudio on the SVG. The default flag size should be 40 x 40.
Can you make this a single PR with #1980

Closing as consolidated into #1980


Separate bindData and bindList calls

Currently, binding data to placeholder elements and binding iterable data into lists is done through the same bind function, and separate calls might be easier to understand in some situations.

I'm aiming to release this functionality as part of the v1 release, due to the complex and sometimes confusing nature of performing all data binding within one bind() function, as currently implemented.
Calling a different bind* method depending on what data you want to bind makes so much more sense.
Currently planning on implementing:

bindKeyValue
bindData
bindList
bindNestedList


Add Recycle Bin to Navigation Bar

What feature or improvement do you think would benefit Files?
So before I've used Files; I've used a program called Wintoys and in this program there was a option to add the Recycle Bin to Explorer navigation bar.
it look like this:

So I was wondering if we could implement the recycle bin into Files in the navigation bar in Files
Requirements
Add Recycle Bin to the navigation bar.
Add Recycle Bin in the right-click menu if the navigation bar.
I'm referring to this:

Files Version
<IP_ADDRESS>
Windows Version
22631.4037
Comments
Navigation Bar = Sidebar.
BTW I would be nice if we can move the sidebar items (WSL, Drives...) whatever we want in the sidebar.

Yeah ik but I would be nice if we put the recycle bin in a seperate section instead of pinning it.

Can you share more about your usecase for having Recycle Bin as a top level item?

It's easier to access the recycle bin especial when you click on the recycle bin on the desktop when the "Set Files as the default file manager" flag is turn off (I will open file explorer if you did that) + if you open file explorer and the recycle bin is pinned and it's also added the navigation bar (using Wintoys) it will look weird...
I'm referring to this:


Setup the Environment: bug on path

On the setup The Environment page, you say that we should launch the following command.
root@docker$ ~/pipeline/bin/initial/RUNME_ONCE.sh
Instead of the previous command, the correct one is
root@docker$ ~/pipeline/bin/RUNME_ONCE.sh
By the way, thank you for your example.

@dalvarom hey there!
good catch.  you caught us in that rare moment where the wiki docs were updated BEFORE the PR was merged!
in other words, if you do a git pull right now, you would find that script in bin/initial.
thanks again for keeping us honest.  :)


Run workflows in case of changes in specific directories

Run acceptance tests if changes in any file in src, utest, atest directories and its subdirectories, excluding any changes in *.rst files.
Run unit tests if changes in any file in src, utest directories and its subdirectories, excluding any changes in *.rst files.

Based on https://help.github.com/en/actions/reference/workflow-syntax-for-github-actions#onpushpull_requestpaths

That's a good point. I haven't noticed that they weren't started here. In my own fork, they were started when I pushed this commit.

No need to run acceptance tests if there are changes only under the utest directory. Commits only containing such changes are pretty rare, sometimes there are cleanups etc.

Codecov Report

Merging #3529 into master will decrease coverage by 1.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #3529      +/-   ##
==========================================
- Coverage   73.63%   72.55%   -1.08%
==========================================
  Files         199      197       -2
  Lines       16115    16033      -82
  Branches     2642     2478     -164
==========================================
- Hits        11865    11631     -234
- Misses       3817     3933     +116
- Partials      433      469      +36

Impacted Files
Coverage Δ

src/robot/utils/unic.py
61.91% <0.00%> (-34.92%)
:arrow_down:

src/robot/utils/compat.py
50.00% <0.00%> (-25.00%)
:arrow_down:

src/robot/utils/encodingsniffer.py
63.08% <0.00%> (-21.53%)
:arrow_down:

src/robot/utils/robottypes.py
81.25% <0.00%> (-18.75%)
:arrow_down:

src/robot/utils/encoding.py
66.67% <0.00%> (-17.64%)
:arrow_down:

src/robot/utils/etreewrapper.py
67.57% <0.00%> (-14.86%)
:arrow_down:

src/robot/utils/misc.py
86.21% <0.00%> (-12.06%)
:arrow_down:

src/robot/utils/robotpath.py
62.27% <0.00%> (-11.32%)
:arrow_down:

src/robot/version.py
55.56% <0.00%> (-11.11%)
:arrow_down:

src/robot/utils/robotio.py
40.39% <0.00%> (-7.69%)
:arrow_down:

... and 21 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 c2f2fe7...b9a5d59. Read the comment docs.

No need to run acceptance tests if there are changes only under the utest directory. Commits only containing such changes are pretty rare, sometimes there are cleanups etc.

Fixed. I forgot that you told me about that in Slack.


Redis Client Side Caching?

Hello everyone,
I recently came across this post discussing the drawbacks and trade-offs of using Redis.
Redis offers a server-assisted client-side caching feature that provides a solution to combine local cache with Redis.
Would the team be open to exploring the potential benefits of this feature?
Thanks!

Something worth looking into down the line if there's a lot of interest and people are brushing up against the tradeoffs of having a purely distributed cache. CSC is particularly useful when you have very hot keys that you don't want to go back to your distributed cache for. If that becomes a common problem, this could be a very useful solution.


Remove watermark from registrations

The large "read-only" watermark is distracting. I propose removing it.

I think the un-dismissable blue alert at the top of the page fulfills the purpose of indicating that a node is a registration. We might also add an icon next to the project title to make it even more clear.
Pinging @jlcohoon @caneruguz @bnosek  for thoughts on this.

@GaryKriebel another one that got pushed into the backlog abyss -- update/close?

Would still like to see further movement on this, so would prefer not to close.

Let's see what @JeffSpies has in terms of legal updates.

Why are we removing the watermark?
On Mon, May 11, 2015 at 1:12 PM, Gary Kriebel<EMAIL_ADDRESS>wrote:

Let's see what @JeffSpies https://github.com/JeffSpies has in terms of
legal updates.
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/673#issuecomment-100981494
.

Because @sloria felt 10 months ago that it was distracting and that the blue floating bar does enough. The last item from you said we should wait until changing interface and lawyer talking could impact, so to defer until that was done.

Caught up, sorry. Yeah, unless there's user feedback about it not differentiating, then this should be closed for the time being.

Closing this issue.  Will open a new issue if user feedback warrants it.

For better or worse, we receive very little user feedback on UI concerns like this (thus far, our UI decisions have been informed by our own usage of the site).
Given that others agreed that there might be better alternatives to the status quo, I'm not sure why this was closed. It can definitely be put on hold, but I think this is low-hanging fruit for improving the presentation of registrations.

I agree with Steve. We do have one comment from UI review the other day
about the background being unattractive. We could see if the interns say
anything as well?
On Mon, May 11, 2015 at 2:11 PM, Steven Loria<EMAIL_ADDRESS>wrote:

For better or worse, we receive very little user feedback on UI concerns
like this (thus far, our UI decisions have been informed by our own usage
of the site).
Given that others agreed that there might be better alternatives to the
status quo, I'm not sure why this was closed. It can definitely be put on
hold, but I think this is low-hanging fruit for improving the presentation
of registrations.
—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/673#issuecomment-101002568
.

I think the grey background is unattractive and doesn't do what we need it
to. If there is serious concern on usability and aesthetic, then we can
consider making it a priority when our designers get to the base OSF L&F.
Until then, unless there is a better solution proposed that addresses both
usability and aesthetic, then we can keep this closed.
If anyone want spend time thinking about UI, let's focus on the things that
users actually see--right now, registrations is not one of them. Lower
hanging fruit is hanging elsewhere--let's prioritize. To me, this is low on
the priority list as compared to those things.
J.
On Tue, May 12, 2015 at 9:30 AM, jlcohoon<EMAIL_ADDRESS>wrote:

I agree with Steve. We do have one comment from UI review the other day
about the background being unattractive. We could see if the interns say
anything as well?
On Mon, May 11, 2015 at 2:11 PM, Steven Loria<EMAIL_ADDRESS>wrote:

For better or worse, we receive very little user feedback on UI concerns
like this (thus far, our UI decisions have been informed by our own usage
of the site).
Given that others agreed that there might be better alternatives to the
status quo, I'm not sure why this was closed. It can definitely be put on
hold, but I think this is low-hanging fruit for improving the
presentation
of registrations.
—
Reply to this email directly or view it on GitHub
<
https://github.com/CenterForOpenScience/osf.io/issues/673#issuecomment-101002568

.

—
Reply to this email directly or view it on GitHub
https://github.com/CenterForOpenScience/osf.io/issues/673#issuecomment-101278594
.


TreeView reveal should not trigger warnings

Issue Type: Bug
Steps to reproduce:

Have a view with a when context that causes it to be show/hidden. For example, by using onDidChangeActiveTextEditor to hide the view when the languageId isn't supported.

Have the view's data provider trigger a command that executes TreeView.reveal() just prior to the view being hidden. In my case, this is caused by listening to onDidChangeTextEditorSelection.
This is the command handler:

LogService.info('onRevealTreeItem: isVisible=' + this.view.visible);
const item = this.provider.findTreeItemAt(position);
if (item !== null) {
  this.view.reveal(item, { select: true });
}

As far as I can tell, when selecting a different text editor this occurs:

onDidChangeTextEditorSelection is triggered for the original editor (with a 0,0 selection) prior to being triggered again on the new editor. The first event is therefore unstoppable from the provider side.
The command handler for the first event executes. At this point view.visible is still true, meaning that the call to reveal is unstoppable from the view side.
The onDidChangeActiveTextEditor event then executes, which hides the view, since reveal is asynchronous.
The queued reveal then executes, and should fail (the context is correct), while triggering the warning.

Relevant log entries:

[info] onDidChangeTextEditorSelection (file:///c%3A/Users/Matt/Documents/VSCode/Projects/redacted/test.js)
[info]   start={3, 0}, end={3, 0}
[info]   execute revealTreeItem
[info] onRevealTreeItem: isVisible=true
[info] onDidChangeTextEditorSelection (file:///c%3A/Users/Matt/Documents/VSCode/Projects/redacted/test.js)
[info]   start={0, 0}, end={0, 0}
[info]   execute revealTreeItem
[info] onRevealTreeItem: isVisible=true
[info] context changed
[info] onDidChangeTextEditorSelection (file:///c%3A/Users/Matt/Documents/VSCode/Projects/redacted/readme.md)
[info]   start={12, 0}, end={12, 0}

Actual result:
rejected promise not handled within 1 second: Error: Unknown view customView
stack trace: Error: Unknown view customView
    at e.isVisible (file:///C:/Users/Matt/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:5320:911)
    at e.toggleViewVisibility (file:///C:/Users/Matt/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:5873:597)
    at e.openView (file:///C:/Users/Matt/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:5872:580)
    at i.openView (file:///C:/Users/Matt/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:3553:624)
    at e.openView (file:///C:/Users/Matt/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:5332:168)
    at processTicksAndRejections (internal/process/task_queues.js:85:5)

Expected result:
Since this is technically how it's supposed to work, just catch and ignore the exception. Alternatively, don't trigger a fake onDidChangeTextEditorSelection event, but that wouldn't solve similar cases.
VS Code version: Code - Insiders 1.44.0-insider (2f3fe9c9c36f935ec83113ef749d1a62ff7d07b6, 2020-03-11T20:45:29.164Z)
OS version: Windows_NT x64 10.0.18362
Extensions (3)

Extension
Author (truncated)
Version

vscode-eslint
dba
2.1.1

vscode-icon-theme
jtl
1.6.6

material-theme
zhu
3.2.3

@mattacosta is there more to that stack trace? Is it possible for you to handle the failed reveal in your extension?

Nope, that's the entire stack trace, and yes is possible to chain a rejection callback that silently ignores the problem. Adding that to every reveal() that could potentially cause the issue isn't ideal though.

As you say adding a rejection callback to every reveal() that could end up like this isn't ideal, but the contract of reveal() is that the view will be revealed. If it isn't revealed, the extension should know that (by the exception).

I disagree. If the reveal() method can fail by design, then in practice, the contract is only that it will attempt to reveal a view (and the result should just be returned instead). Furthermore, if an extension really needs to know the state of the view after calling reveal() in its current implementation, then it is already possible for the extension to re-check the visible property, making the exception pointless.


Update create-child-flows.md

Added: known issue about Child Flows getting blocked by HTTP Connector DLP

@MSFTMan : Thanks for your contribution! The author(s) have been notified to review your proposed change. @MSFTMAN


Future tools

[ ] SubPhaser: https://github.com/zhangrengang/SubPhaser
[ ] Merqury: https://github.com/marbl/merqury
[ ] findZX: https://github.com/hsigeman/findZX
[ ] NGenomeSyn: https://github.com/hewm2008/NGenomeSyn; Consensus vs Haplotypes

compleasm is another name for miniBUSCO. miniBUSCO does not support all the BUSCO modes yet and it is in beta release.
SubPhaser is a phasing tool. The phasing QC part is also covered by Merqury which uses Meryl for k-mer counting


Explanation / clarification for run at startup behaviour option

Inform user that the run at startup setting only triggers opening of settings app - does not affect weather Modern Flyouts will be applied after computer is restarted.
i think it is currently a bit ambiguous, and some users may turn it on, believing it necessary for the app to work after reboot

No, it will run the app at startup.

Why should there be an option to open settings window after the app is started. That will be annoying for the users.

I think, in your machine the app is started twice after Windows startup (the first one making the ModernFlyouts run and the second one triggering settings window). The app itself handles the startup logic via Registry startup. There could be another method (startup shortcut, startup task) [possibly made by you] which causes this second startup.
I would recommend you to check Task Manager's startup tab and report whether you see two entries of ModernFlyouts (with the run at startup enabled in the settings).
Turning of the Run at startup option will prevent your issue for now.

Sorry, I was really puzzled by a setting which appeared to cause the settings window to open at boot.  It seemed like a really weird feature.
You were correct, a close look at start-up tab reveals that for some reason two entries were created, one in HKCU:Run (which is the one that the settings menu enables and disables) and another which appears in startup common. Not sure why, i never told it to. uninstalling then reinstalling doesn't remove the 'Startup common' entry. I suspect it may have been left over from a previous version as i had V0.2 installed prior to V0.3.
Disabling the "startup common" entry resolves the problems.
 I'm always welcome to contributors.


Calculate Estimated CO2 Using  near Real Time Actual Grid Intensity When Available

Feature Suggestion
The current co2 estimation uses hard coded values. The GSF carbon aware SDK has made it possible to now use much more accurate estimates based on real time grid intensities.
Describe the solution you'd like
An option to use near realtime region based carbon intensity measurements when estimating co2.
Describe alternatives you've considered
We are implementing near real time estimates on our own project.
What we are doing using go.
func CalulcateCarbon(input *CalulcateCarbonInput) *CalulcateCarbonOutput {
	output := CalulcateCarbonOutput{}
	minWattsAverage := .74
	maxWattsAverage := 3.5
	averageCpuUtilization := float64(50)
	averageWatts := float64(minWattsAverage + (averageCpuUtilization/100)*(maxWattsAverage-minWattsAverage))
	durationInS := float64(input.Duration / 1000 % 60)
	memorySetInMB := float64(input.MemorySize)
	functionWatts := float64(averageWatts * durationInS / 3600 * memorySetInMB / 1792)
	output.Electricity = functionWatts
	output.Carbon = functionWatts * float64(input.Intensity)
	return &output
}

Same algorithm but rather than use hard coded region estimates real time grid intensity obtained every 5 minutes from the GSF Carbon Aware SDK.
Additional context
Accuracy is an important part of measuring carbon. Therefore, using real time estimates should be highly considered.
Possible Implementation
Use intensities inside this report.
Renewable Report: https://store.hedge.earth/renewable-report/report.json
One option given that the calculations are not done in real time for CCC is to create a job and populate your own database with intensities within the report. Another would to provide you with an API to pull a report(s) within a time window. You could also forget the report provided above and implement your own strategy using the Carbon Aware SDK.

Hi, I'm interested in picking this up, I believe this may be a great addition to increase the accuracy of measurement. Additionally, especially if one is starting to think about demand shifting / shaping techniques, I imagine it's currently really hard measure potential gains. Is there any chance we can have a chat about this?


Fix Node3D gizmo names

Commit 5ef62e546f485e5f2798230dcfbea2b4ddecbe6e renamed EditorNode3DGizmoPlugin "get_name" to "get_gizmo_name", but it wasn't done everywhere. Because EditorNode3DGizmoPlugin inherits from Resource, it was then using Resource::get_name() which resulted in null names.
Fixes godotengine/godot#46349

Thanks!


MACH-O:  non-private labels cannot appear between .cfi_startproc / .cfi_endproc pair

Refer to: https://github.com/llvm/llvm-project/issues/82261

Fixed by upgrading the rust to latest nightly.


Question: By using this, can we run shadcn/ui inside Svelte?

By using this, can we run shadcn/ui inside Svelte?

Yes, but It depends on which component on how easy it is to use.
Some components such as <Dialog/> are using React.Children.only internaly and then you can use that component directly inside a svelte file and you'd need to write a custom wrapper component.
I'd recommend looking into https://shadcn-svelte.com/

@bfanger thanks for the answer, ya the idea is to use native shadcn/ui inside the Svelte since that already very good and avoid using the porting version of shadcn-svelte. That's just an idea :)


services/friendbot: fix race condition in Bot.Pay

This PR fixes an issue in the friendbot Pay function. Previously, the minion index and update were not being locked. As a result, tests that attempted concurrent access detected a race condition, as seen here

@debnil What’s the error when running this test?

I think that test is just fine because it simulates starting multiple go routines of Pay and that's happening in the FriendbotHandler.loadResult method. The issue detected by race detector is the following:
WARNING: DATA RACE
Write at 0x00c0000da7f8 by goroutine 9:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:28 +0x2d9
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func3()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:66 +0x50

Previous read at 0x00c0000da7f8 by goroutine 8:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:25 +0x181
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func2()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:61 +0x50

and it's really a race condition in Pay function:
// Pay funds the account at `destAddress`.
func (bot *Bot) Pay(destAddress string) (*hProtocol.TransactionSuccess, error) {
	log.Printf("Selecting minion at index %d of max length %d", bot.nextMinionIndex, len(bot.Minions))
	minion := bot.Minions[bot.nextMinionIndex] // line 25
	resultChan := make(chan SubmitResult)
	go minion.Run(destAddress, resultChan)
	bot.nextMinionIndex = (bot.nextMinionIndex + 1) % len(bot.Minions) // line 28
	maybeSubmitResult := <-resultChan
	close(resultChan)
	return maybeSubmitResult.maybeTransactionSuccess, maybeSubmitResult.maybeErr
}

bot.nextMinionIndex is accessed from multiple go routines. We need to add a mutex that locks access to it to a single go routine.

@nikhilsaraf it's the data race documented in the trace below. This test was originally defined to create such a data race. I think we discussed why, because the friendbot.Pay function would be called through separate calls to a deployed endpoint, this would not happen with the deployed friendbot.
?   	github.com/stellar/go/services/friendbot	[no test files]
2019/06/12 18:15:33 Selecting minion at index 0 of max length 1
2019/06/12 18:15:33 Selecting minion at index 0 of max length 1
2019/06/12 18:15:33 Selecting minion at index 0 of max length 1
==================
WARNING: DATA RACE
Write at 0x00c0000da7f8 by goroutine 9:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:28 +0x2d9
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func3()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:66 +0x50

Previous read at 0x00c0000da7f8 by goroutine 8:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:25 +0x181
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func2()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:61 +0x50

Goroutine 9 (running) created at:
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:65 +0x5a6
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:827 +0x162

Goroutine 8 (running) created at:
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:60 +0x552
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:827 +0x162
==================
==================
WARNING: DATA RACE
Read at 0x00c0000da7f8 by goroutine 8:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:28 +0x293
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func2()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:61 +0x50

Previous write at 0x00c0000da7f8 by goroutine 9:
  github.com/stellar/go/services/friendbot/internal.(*Bot).Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot.go:28 +0x2d9
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay.func3()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:66 +0x50

Goroutine 8 (running) created at:
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:60 +0x552
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:827 +0x162

Goroutine 9 (running) created at:
  github.com/stellar/go/services/friendbot/internal.TestFriendbot_Pay()
      /go/src/github.com/stellar/go/services/friendbot/internal/friendbot_test.go:65 +0x5a6
  testing.tRunner()
      /usr/local/go/src/testing/testing.go:827 +0x162
==================
--- FAIL: TestFriendbot_Pay (0.03s)
    testing.go:771: race detected during execution of test
FAIL
FAIL	github.com/stellar/go/services/friendbot/internal	0.043s

@bartekn thanks for the explanation and quick review!! I accidentally read the stack trace backwards and fixed the test alone, thinking that was sufficient.

NP! Now we can review and merge https://github.com/stellar/go/pull/1398.

locking access is the right fix, thanks @debnil and @bartekn


Limit assumed role session name to 64 characters

AWS limits the length of session names, when assuming roles to 64 characters.
With Account IDs being 12 digits long, role names having a 40 character limit, there's only 12 characters left for the username a random number and 3 separators.

Thank you!


using SC_TECHNOLOGY_DIRECTWRITE rather than SC_TECHNOLOGY_DEFAULT

SC_TECHNOLOGY_DIRECTWRITE performs better font linking than SC_TECHNOLOGY_DEFAULT.
In font-missing situation, SC_TECHNOLOGY_DIRECTWRITE can show more fonts.
Florian Balmer wrote (in e-mail)...

Notepad2 5.0.26-beta4 makes use of SC_TECHNOLOGY_DIRECTWRITE [0],
which seems to perform better font linking than the traditional
SC_TECHNOLOGY_DEFAULT mode, and may thus fix some GDI quirks? The same
problem seems to appear in Notepad2 5.0.26-beta4 if DirectWrite is
disabled [1].
[0] http://www.scintilla.org/ScintillaDoc.html#SCI_SETTECHNOLOGY
[1] http://www.flos-freeware.ch/development-releases/notepad2-FAQs.html#d2d-rendering
Best wishes, Florian.

Random changes in the files.
Also, I see no need to have a new macro. Finally, the place you decided to add the call is pretty random.

With SC_TECHNOLOGY_DIRECTWRITE, Scintilla becomes more likely to be fully capable for handling multilingual characters. Maybe It's a legacy bug in GDI.
It matters, but that's not the point.
For many years, CPU-based font rendering codes are developed to look almost equal to printed ones. On printers, outline vector fonts are processed with a component called RIP(raster image processor). This difference has been a long last pain for font designers, print layout designers and application GUI designers.
DirectWrite uses GPU during the rasterization phase, and this makes the rendered result much

With SC_TECHNOLOGY_DIRECTWRITE, Scintilla becomes more likely to be fully capable for handling multilingual characters. Maybe It's a legacy bug in GDI.
It matters, but that's not the point.
For many years, CPU-based font rendering codes are developed to look almost equal to printed ones. On printers, outline vector fonts are processed with a component called RIP(raster image processor). This difference has been a long last pain for font designers, print layout designers and application GUI designers.
DirectWrite uses GPU during the rasterization phase, and this makes the rendered result much

With SC_TECHNOLOGY_DIRECTWRITE, Scintilla becomes more likely to be fully capable for handling multilingual characters. Maybe It's a legacy bug in GDI.
It matters, but that's not the point.
For many years, CPU-based font rendering codes are developed to look almost equal to printed ones. On printers, outline vector fonts are processed with a component called RIP(raster image processor). This difference has been a long last pain for font designers, print layout designers and application GUI designers. DirectWrite uses GPU during the rasterization phase, and this makes the rendered result seem much more accurate because of features like, for example, ClearType sub-pixel positioning.
No reason to avoid use of DirectWrite.
BTW, I agree with @bluenlive that Direct2D feature can be enabled or disabled manually. It's because of the behavioral difference between GDI and DirectWrite. See below, the first and the second images compared. Both are set Consolas 10pt. Also both the third and the fourth images are set Inconsolata 11pt. Non-Latin graphemes shown are Korean syllables. With GDI used, the first and the third images seems Batang (Microsoft Korean default font like Times New Roman) is used for fallback on Korean syllables. With DirectWrite used, the second and the fourth images seems Malgun Gothic (Microsoft Korean default font like Segoe UI) is used for fallback.
Korean syllable graphemes are expected to have exact twice the width of Latin graphemes when they are used with monospace fonts like Consolas or Inconsolata. Although GDI is used, the first one shows a good example of rendered result, because the rectangular selection seems like an exact rectangle. On the other hand, the fourth one is better rendered than the third one. I don't know what makes this difference, but I am certain that this is not a bug.

Executable binaries used to take these screenshots are those @bluenlive distributes with Korean language patches.
Conclusion: making GDI/DirectWrite a manual choice would dramatically help users with various fallback fonts.
Thanks.

This PR contains changes not related to what addressed, see https://github.com/zufuliu/notepad2/commit/b7ad639fa6e576db2df04ce72e2b69f409f386bd for a clean implementation and an easy way to switch between different rendering technologies.


[feature][broker] Support cgroup v2 by using jdk.internal.platform.Metrics in Pulsar Loadbalancer

Master Issue: #16601
Motivation
The Pulsar load balancer detects CPU limits using cgroup v1 API, and the jdk.internal.platform.Metrics already support cgroup (V1, v2) so we should use jdk.internal.platform.Metrics to get the cgroup metrics.
Reference: https://code.yawk.at/java/17/java.base/jdk/internal/platform/
Modifications
Use jdk.internal.platform.Metrics to get the cgroup metrics in the LinuxInfoUtils.
Verifying this change

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

This change is already covered by existing tests, such as testCGroupMetrics.
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

I don't understand something. You're using an internal class from the JDK - it's internal - you're not supposed to use it as is no?
How is the JDK itself using this - and maybe we can this info in a more public API way?
Can you describe in more detail the motivation to get those metrics and why normal MBean metrics don't cut it?

I don't understand something. You're using an internal class from the JDK - it's internal - you're not supposed to use it as is no? How is the JDK itself using this - and maybe we can this info in a more public API way? Can you describe in more detail the motivation to get those metrics and why normal MBean metrics don't cut it?

The OperatingSystemMXBean using jdk.internal.platform.Metrics to get CPU load.
But OperatingSystemMXBean only exports CPU load, we also need the limit of CPU unfortunately, JDK did not export it.

And, for the CPU usage we have our own calculation formula, which is not exactly the same as OperatingSystemMXBean#getCpuLoad.
Can you please explain why it is different? I do see that eventually, the load balancer needs the percentage right? how much was the CPU used relative to the limit which is 100%? getCPULoad of sun implementation does try to calculate that, no?

Can you please explain why it is different? I do see that eventually, the load balancer needs the percentage right? how much was the CPU used relative to the limit which is 100%? getCPULoad of sun implementation does try to calculate that, no?

We need to record usage and limit, but OperatingSystemMXBean#getCpuLoad only is the percentage.
And for usage we calculate is a mean value, more please see:
https://github.com/apache/pulsar/blob/380c7587d0cefdd763030f348246fef711bfd58c/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LinuxBrokerHostUsageImpl.java#L139-L144

Ok, I did some reading and this is what I found:

Usage calculation in OperatingSystemMXBean which delegates to OperatingSystemImp gives you the percentage of usage of CPU relative to the limit imposed on the container group, as can be seen here:

            long quota = containerMetrics.getCpuQuota();
            long share = containerMetrics.getCpuShares();
            if (quota > 0) {
                long numPeriods = containerMetrics.getCpuNumPeriods();
                long quotaNanos = TimeUnit.MICROSECONDS.toNanos(quota * numPeriods);
                return getUsageDividesTotal(cpuUsageSupplier().getAsLong(), quotaNanos);

which calls
        private double getUsageDividesTotal(long usageTicks, long totalTicks) {
            // If cpu quota or cpu shares are in effect. Calculate the cpu load
            // based on the following formula (similar to how
            // getCpuLoad0() is being calculated):
            //
            //   | usageTicks - usageTicks' |
            //  ------------------------------
            //   | totalTicks - totalTicks' |
            //
            // where usageTicks' and totalTicks' are historical values
            // retrieved via an earlier call of this method.
            if (usageTicks < 0 || totalTicks <= 0) {
                return -1;
            }
            long distance = usageTicks - this.usageTicks;
            this.usageTicks = usageTicks;
            long totalDistance = totalTicks - this.totalTicks;
            this.totalTicks = totalTicks;
            double systemLoad = 0.0;
            if (distance > 0 && totalDistance > 0) {
                systemLoad = ((double)distance) / totalDistance;
            }
            // Ensure the return value is in the range 0.0 -> 1.0
            systemLoad = Math.max(0.0, systemLoad);
            systemLoad = Math.min(1.0, systemLoad);
            return systemLoad;
        }

So we can obtain the percentage of CPU used relative to CPU allocated to the container (container group).
As you can see the code pasted above also does delta calculation, relative to last time it was called. It is the same as we do as can be see in Pulsar code here:
    private double getTotalCpuUsageForCGroup(double elapsedTimeSeconds) {
        double usage = getCpuUsageForCGroup();
        double currentUsage = usage - lastCpuUsage;
        lastCpuUsage = usage;
        return 100 * currentUsage / elapsedTimeSeconds / TimeUnit.SECONDS.toNanos(1);
    }

The main point here is that JDK does provide relative usage or as you said "mean" value.

The biggest difference I see is that when we report usage percent, we do so relative to the entire host: we take the CPU Usage for CGroup as reported by the operating system  (measured in microseconds), only the delta from last time measured, and divide that by elapsed, so in effect, it is CPU used relative to the entire host.

    public static long getCpuUsageForCGroup() {
        try {
            if (metrics != null && getCpuUsageMethod != null) {
                return (long) getCpuUsageMethod.invoke(metrics);
            }

Which delegates to
    /**
     * Returns the aggregate time, in nanoseconds, consumed by all
     * tasks in the Isolation Group.
     *
     * @return Time in nanoseconds, -1 if unknown or
     *         -2 if the metric is not supported.
     *
     */
    public long getCpuUsage();

The limit we report is the percentage of CPU allocated, relative to the entire host.
It actually looked a bit odd to me: Why do we report usage relative to host and limit relative to host? Why not just report usage in percent relative to the allocated container limit?
So I looked at how we are using it, as you also mentioned above. So I saw those references:
            if (entry.getValue().limit > 0 && entry.getValue().limit > entry.getValue().usage) {
                Double temp = ((entry.getValue().limit - entry.getValue().usage) / entry.getValue().limit) * 100;
                percentAvailable = temp.intValue();
            }

In the above code you can see it doesn't really need it the percent to be relative to host nor does it need the limit. It just needs to say: CPU usage in percent relative to container allocated CPU limit < 100%, which the JDK gives.
           double cpuChange = (newUsage.cpu.limit > 0)
                            ? ((newUsage.cpu.usage - oldUsage.cpu.usage) * 100 / newUsage.cpu.limit)
                            : 0;

In the above code, you can see it only needs to know how much percent has changed - same thing - it can use the JDK load percent.
The biggest issue I have is only with the reporting
    private void printResourceUsage(String name, ResourceUsage usage) {
        spec.console().println(name + " : usage = " + usage.usage + ", limit = " + usage.limit);
    }

I don't really understand why would some need to know the limit. Why isn't the CPU percentage enough relative to the container allocation?
I understand all I mentioned here is a bit of technical debt to fix in this PR. I'm saying that we didn't mind not reporting the limit, we didn't need to be very operating system specific, and write Linux-based reporting classes and read the info of it from the file system and now through JVM internal classes. We could just use OperatingSystemMXBean.
@heesung-sn What do you think?

Please note that I haven't found any other open source that uses jdk.internal.platform.Metrics - I tried searching using TabNine code search and also GitHub search.

@heesung-sn What do you think?

According to this article,
https://developers.redhat.com/articles/2022/04/19/java-17-whats-new-openjdks-container-awareness#recent_changes_in_openjdk_s_container_awareness_code
It seems like OperatingSystemMXBean already provides cpu usage in percentage relative to cgroup (for both v1 and v2) (I will call it as cpu_usage_in_percent_cgroup for our discussion)
I agree with you. From my understanding, load balancer cares about cpu_usage_in_percent_cgroup in the end for the cpu usage computation.
However, LB also requires other signals such as memory_usage_in_percent_cgroup, network_usage_in_percent_cgroup, which require separate limits for their percentage computation.
As you pointed out, the problem is that LB uses the same generic code (requiring limit) to compute the resource percentage.
Maybe we can tweak the code to ignore the limit and use the signal as-is if they are already in the *_usage_in_percent_cgroup form.

However, LB also requires other signals such as dict_memory_usage_in_percent_cgroup, network_usage_in_percent_cgroup, which require separate limits for their percentage computation.
Can you please expand on that? I don't any method providing usage and limit of direct memory in Pulsar. I do see usage and limit for physical memory (in the container) which as we said we can take from the mxBean which provides total and usage of memory.
Regarding network usage - yes, that I was surprised to see that the operating system didn't include, and we resort to reading operating system specific files to obtain that. I posted a question in the OpenJDK general discussion mailing to get the motivation for that ( I did search their JIRA issue database and also googled the mailing list site but found nothing).

Maybe we can tweak the code to ignore the limit and use the signal as-is if they are already in the *_usage_in_percent_cgroup form.
I guess that requires a separate PR. Maybe we can push that PR in but add an issue to refactor (delete it out) the dependency on limit and only use percentage (which should be enough) or refactor to avoid using limit and be more specific like ResourceUsage.CpuUsage, MemoryUsage, NetworkUsage, each providing it's own method, where the CPU would only provide percentage.

Can you please expand on that? I don't any method providing usage and limit of direct memory in Pulsar. I do see usage and limit for physical memory (in the container) which as we said we can take from the mxBean which provides total and usage of memory.

I see this method is used to collect direct memory usage.
https://github.com/apache/pulsar/blob/535415302ef6d1a9017f6ec25b87b24afd081155/pulsar-broker/src/main/java/org/apache/pulsar/broker/loadbalance/impl/LoadManagerShared.java#L228-L229

I guess that requires a separate PR. Maybe we can push that PR in but add an issue to refactor (delete it out) the dependency on limit and only use percentage (which should be enough) or refactor to avoid using limit and be more specific like ResourceUsage.CpuUsage, MemoryUsage, NetworkUsage, each providing it's own method, where the CPU would only provide percentage.

Yes, a separate PR makes sense to me.

Any updates? I think this PR makes sense to use the cgroup v2.

I will update this PR as soon.

I guess that requires a separate PR. Maybe we can push that PR in but add an issue to refactor (delete it out) the dependency on limit and only use percentage (which should be enough) or refactor to avoid using limit and be more specific like ResourceUsage.CpuUsage, MemoryUsage, NetworkUsage, each providing it's own method, where the CPU would only provide percentage.

Yes, we need a separate PR to refactor it.

We need this to be included in branch-3.0 and branch-2.11 asap. I backported the changes to branch-2.10 with all dependencies in #20659.
This is becoming urgent.
AKS Kubernetes 1.25+ switches to use cgroup v2:
https://github.com/Azure/AKS/releases/tag/2023-03-05
AKS Kubernetes 1.24 goes End-of-life on July 30, 2023.
GKE contains to have a way to select between cgroup v1 & cgroup v2:
https://cloud.google.com/kubernetes-engine/docs/how-to/node-system-config#cgroup-mode-options
GKE will default to cgroup v2 in new Kubernetes 1.26 clusters or node pools.
AWS EKS v1.26 nodes will default to cgroup v2.


Italian language for Data-Locale

Added Italian Language

Thank you very much. I add this to 4.2.8


Dmwatch 1.7 + base repo owner change

The DMWatch org account owner is no longer around, might as well do the development from the dev account anyhow.
The website that the plugin will now interact with is a live end point that gets the name updates of scammers, relying on github  has a 5 minute cache delay.
By offering closer to real time data by default we will help protect people from serial scammers that name change 20 times in a day.

Do not approve this.


[Q] How can I get an updated list?

The current list is outdated and doesn't contain ranges such as  <IP_ADDRESS>/16.

https://github.com/v2fly/geoip/raw/release/text/ir.txt


Pop ups are never popping up in front of player alwayas behind RE2

If I interact with something so the inventory popups or any other in game popup it almost always at the opposite side of what I am looking at. Its annoying to always have to turn around to see the menu,

Use a newer build. This was fixed a while ago.
https://github.com/praydog/REFramework-nightly/releases


Add explicit check for number of channels

Example why you need to check it:
M = torch.randint(low=0, high=2, size=(6, 64, 64), dtype = torch.float)
When you put this input through to_pil_image without mode argument, it converts to uint8 here:
if pic.is_floating_point() and mode != 'F':
            pic = pic.mul(255).byte()

and change the mode to RGB here:
if mode is None and npimg.dtype == np.uint8:
            mode = 'RGB'

Image.fromarray doesn't raise if provided with mode RGB and just cut number of channels from what you have to 3

Also in this Pytorch forum discussion.

@ademyanchuk thanks for the PR ! I'd add the check before to numpy conversion, before line 194, as if pic.shape[-3] > 4.
It would be good to add a test on that too, for example here:
https://github.com/pytorch/vision/blob/aa4cf03961f93fd8290b8a57ca1052d01dfc0f5c/test/test_transforms.py#L989
and
https://github.com/pytorch/vision/blob/aa4cf03961f93fd8290b8a57ca1052d01dfc0f5c/test/test_transforms.py#L993
Thanks

@vfdev-5 thanks for comment! I thought where to put the check for number of channels and decided to put it after conversion to numpy as at this point we know the channel dimension position is .shape[2]. Given the function takes as input both torch.tensor and numpy.array and assumes tensor is channel first, we have to add check two times for numpy with ndim==3 and for torch with ndim==3. Do you think I should add the check two times? Or perhaps is it better to check only once after conversion?
Sure, I would like to add tests. I will see how can I add those. It is my first PR :)

Given the function takes as input both torch.tensor and numpy.array and assumes tensor is channel first, we have to add check two times for numpy with ndim==3 and for torch with ndim==3.

Yes, true. For tensor the assumption is that number of channels is -3 and for numpy is -1. Idea is that IMO it would be better to raise exception before making data processing ...

Sure, I would like to add tests. I will see how can I add those. It is my first PR :)

Sounds good. Feel free to ask questions if needed. Please, take a look at drafted CONTRIBUTING, if needed.

Idea is that IMO it would be better to raise exception before making data processing ...

This is totally make sense. So I will proceed.

Ok. I refactored the checking to be done before processing and added a test for invalid number of channels :)

Ok. It seems like I manage to make all discussed changes :)

@vfdev-5 , sorry to bather. How could I do python setup.py develop so it compiles with cuda version 10.2?

How could I do python setup.py develop so it compiles with cuda version 10.2?

It will build cuda extensions with the same cuda version as torch has. If you'd like to disable it, just set
CUDA_VISIBLE_DEVICES="" python setup.py develop

It will build cuda extensions with the same cuda version as torch has

Apparently in my case it is not :(
I did conda install pytorch -c pytorch-nightly
it install torch with cuda 10.2
but python setup.py develop compile cuda11 version

Anyway, I made the changes from your last comments :)

It will build cuda extensions with the same cuda version as torch has

Apparently in my case it is not :(
I did conda install pytorch -c pytorch-nightly
it install torch with cuda 10.2
but python setup.py develop compile cuda11 version
On my local machine cuda 11 is installed - could it be issue?
So, as it is WSL2, i just delete my local cuda11. And torchvision compiled with matched version of cuda as in torch nightly!

It would be nice to have a detailed instructions to be able to reproduce your situation. If you could reproduce the issue, please file an issue about that.

Could you also please merge master to your branch. There is unexpected modification here : https://github.com/pytorch/vision/pull/3013/files#diff-b5826ce59a98ee20294c65d06e09363609879221494b5ff7fbb7087b75f3e159L7

Hi, I did merge master into my branch patch-1. I also manually checked this file. But I have no diff with pytorch vision master. This diff that you show the link for is from this commit (as I understand) https://github.com/pytorch/vision/pull/3020/commits/365f1594fc12937e31b0844261d95ca31025c1dd

It would be nice to have a detailed instructions to be able to reproduce your situation. If you could reproduce the issue, please file an issue about that.

I will try to reproduce the issue. If I succeed, I would file the issue.

Thanks @vfdev-5 for a valuable comments and review. Appreciate! So, is it done? How the merge in master is proceeded?

It appears to me like most of recent pull requests fail on the same travis ci job on the same place.


Depth image encoding is hard-coded

The current implementation appears to hard-code the expected image encoding of the depth image:
https://github.com/ros-industrial/industrial_reconstruction/blob/2cf137d2941be6425bc4908e70c0cc1a2fc34c63/industrial_reconstruction/industrial_reconstruction/industrial_reconstruction.py#L288-L288
it doesn't do this for the regular RGB images:
https://github.com/ros-industrial/industrial_reconstruction/blob/2cf137d2941be6425bc4908e70c0cc1a2fc34c63/industrial_reconstruction/industrial_reconstruction/industrial_reconstruction.py#L289-L289
Looking at various drivers, there are two main 'types' of depth images:

16-bit unsigned with depth in mm (16UC1)
32-bit float with depth in m (32FC1)

At least in ROS 1, the depth_image_proc/convert_metric nodelet can be used to convert between the two types.
The hard-coded 16UC1 (and the existence of the depth_scale parameter) implies the first is expected and has been used when developing industrial_reconstruction. The code as-is is incompatible with the second type.
Would it be desirable to remove the hard-coding and rely on what is set in sensor_msgs::msgs::Image::encoding field instead?

Would it be desirable to remove the hard-coding and rely on what is set in sensor_msgs::msgs::Image::encoding field instead?

Yes, this would definitely be preferable. Upon initial creation of this node I had some issues trying to do that, so I just ran with what would work for the initial application. It sounds like you and  @Aashish-TUD have an application using different camera encodings than what I've used in the past, if you guys have a working change for that I would definitely want to merge that in.


Embedded Binaries signing problem

I added the framework as a embedded binary.. But when i make archive the framework does not signing via my provisioning profile. What is the problem ?

Hey @testudorithm, in your project under Targets > Build Phases, is PermissionScope marked as a dependency?

no i add to project like this,

Is this a manual integration as opposed to cocoapods or carthage?
You should add PermissionScope to the dependency list as well if it's manual.

i get the framework in example project, and i add to embedded binaries also add Build Phases>Copy Files

I would highly encourage you to use one of the other integration options because they do the hard work for you.
If you must integrate manually like you're doing now, add the project as a dependency in your Build Phases, and make sure 'Code Sign on Copy' is checked in your Copy Files phase.

When i try adding as a dependency, i does not see the framework in selection

Oh, you just grab the framework rather than the project.
Instead of using the framework, drag the permissionscope project file into your project and take the same steps. This'll build the framework and include it for you.

Okay thanks :-)


New package: DifferentiationInterfaceTest v0.1.0

Registering package: DifferentiationInterfaceTest
Repository: https://github.com/gdalle/DifferentiationInterface.jl
Created by: @gdalle
Version: v0.1.0
Commit: e3bc10d5190b6e1c21788b1a55e403490edfd482
Git reference: HEAD
Description: An interface to various automatic differentiation backends in Julia.

[noblock] will be fixed by https://github.com/gdalle/DifferentiationInterface.jl/pull/162


Enabling Windows Insider Page And Services

Hi,
I would like to know how to enable the Insider page on Windows Server 1809 standard Sept 2020 Update. I enabled developer mode via group policy. And there are some reports that the page can be enabled via the registry. Put it doesn't work of server
Any Help? I enabled the service.
https://www.howto-connect.com/fix-insider-program-settings-blank-empty-missing-windows-10/
https://www.windowsphoneinfo.com/threads/add-or-remove-windows-insider-program-settings-page-in-windows-10.8061/
https://www.tenforums.com/tutorials/63585-enable-disable-insider-build-settings-windows-10-a.html
These do not work on Server.

Please file a feedback through Feedback Hub about it.
Currently, Windows Server builds don't support receiving preview builds from Windows Update and, therefore, you'll need to upgrade using an ISO image.

@bleblancMSFT

This repository tracks documentation issues. Doesn't appear this falls into that bucket, so recommend closure @yogkumgit.


In http.js, do not end request for keep-alive connections

The request.end(data) line was preempting maintaining a persistent connection, even when Connection: 'keep-alive' was specified. My proposal is to handle the data via the body option, and only end the request when keep-alive is not specified.

Thanks a lot for your contribution @mgorczyca - we appreciate it a lot :+1:
The TravisCI Build is failing due to failing tests in node v0.12 and io.js - in order to test it with your local repo it would be necessary to use nvm (at least I guess that there is no other/better solution) and install node v0.12 as well as io.js and run npm test in these environments too.

The build for v0.8.28 is failing with the following:
Error: Cannot find module 'readable-stream/duplex'

The npm test completes successfully in my nvm environment for the same node.js version. I'd appreciate any suggestion or solution.

Sorry for the late reply - I restarted the Travis CI tests and everything is passing (sometimes Travis CI seems to have some hickups while running tests).
Please, in order to get your PR merged in quickly, follow the steps lined out in our Guideline on Submitting a Pull Request. The points which are left to merge your PR are the following:

Please, squash your commits into a single one. If you don't know how to do this there are links to the excellent GitHub help on this topic in our Guidelines.

Please, please, please ** do not close a pull request due to a request to rebase** (instead, read through our short r Guidelines and find all necessary information & help in order to squash/rebase your commits :+1:

Thanks for the helpful information and links!

Thanks a lot :+1:


Highlight node

Emphasise clicked node and those connected to it in some way.
Examples:
http://bl.ocks.org/d3noob/5141528
http://plnkr.co/edit/3fv3IR3S3Kp86rAummnz?p=preview
http://www.coppelia.io/2014/06/finding-neighbours-in-a-d3-force-directed-layout-2/

Updated highlightNode function to:

function highlightNode(){

        if (toggle == 0) {

            d = d3.select(this).node().__data__;

            node.style("opacity", function (o) {
                return neighboring(d, o) | neighboring(o, d) ? 1 : 0.15;
            });
            toggle = 1;

            link.style("opacity", function(o) {
                return o.source === d || o.target === d ? 1 : 0.15;
            });

        } else {
            node.style("opacity", 1);
            link.style("opacity",1);
            toggle = 0;
        }

    updateInfoBox(this);

}

You have to click another node to get it to toggle back to unhighlighted. Ideally clicking background would trigger toggle

Fixed by adding click event to outer svg (inner svg didn't work for some reason).
 osvg = d3.select('#forceGraph')
                .on('click',function(){
                    node.style("opacity", 1);
                    link.style("opacity",1);
                    toggle = 0;
                }), //select the svg

Also had to add    d3.event.stopPropagation(); to highlightNode function


Can not launch minecraft

Describe the Bug
I can not launch my minecraft when only installing this mod and fabric api
The crash report shows that there is a java.lang.NullPointerException at com.simibubi.create.infrastructure.gui.OpenCreateMenuButton.method
Reproduction Steps

install minecraft 1.19.2 and fabric 0.15.6
install create-fabric-0.5.1-c-build.1160+mc1.19.2 and fabric-api-0.77.0+1.19.2
launch minecraft
get a crash report

Expected Result
I expect I can see the minecraft main menu rather than crash report.
Thanks for your fixing.
Screenshots and Videos
Please see the last line:

Crash Report or Log
https://gist.github.com/baiyu-cs/4c3ee68e3fa08969238ff8996d147d6d
Operating System
Windows 10
Mod Version
0.5.1c
Minecraft Version
1.19.2
Other Mods
fabric-api-0.77.0+1.19.2
create-fabric-0.5.1-c-build.1160+mc1.19.2
Additional Context
No response

thank you very much


Distros fail to launch: File system error -2145103860

Your Windows build number:
Microsoft Windows [Version 10.0.16299.492]

What you're doing and what's happening:
After installing WSL, rebooting, downloading all three linux distros from the "Downloading distros"
section of https://docs.microsoft.com/en-us/windows/wsl/install-manual, each distro .appx fails to launch with File system error -2145103860. Should these be able to launch as a regular user, or do I need to have admin rights to launch?
check sums and sizes in bytes:
$ cksum *
3981138175 208978356 openSUSE-42_v1.appx
4197424015 208258653 SLES-12_v1.appx
3421032464 201749452 Ubuntu.1604.2017.711.0_v1.appx

What's wrong / what should be happening instead:
I see the following error dialog instead of a bash shell (or whatever is supposed to appear when launch succeeds).

Strace
Sorry, not sure how to run strace on a windows machine and the linux part isn't launching.

For WSL launch issues, detailed logs:
I think only an admin/elevated user can get these logs. I do not have admin rights.

Try this:

Uninstall Ubuntu app (right click on tile -> Uninstall)
Rename Ubuntu.1604.2017.711.0_v1.appx to Ubuntu.1604.2017.711.0_v1.zip and extract all.
Run Ubuntu.exe inside the extracted folder.

Does it work now?

Yes! Installing an app within ubuntu now. Thanks! Why did it work this way instead of the original?

@fappy012 - you are likely hitting some issue related to the Microsoft Store. Extracting the appx and running the executable directly bypasses that issue.

 I thought by downloading it and doing a 'manual' install would avoid any attempt to reach the Store. But I guess those extra steps were necessary too.

Try this:

Uninstall Ubuntu app (right click on tile -> Uninstall)
Rename Ubuntu.1604.2017.711.0_v1.appx to Ubuntu.1604.2017.711.0_v1.zip and extract all in to a folder.
Run Ubuntu.exe inside the extracted folder.

Does it work now?

This works me. How do you uninstall with this method?

There is an additional requirement - the extracted folder needs to be on the same drive on which Windows OS is installed. The above does not work if the extracted folder is on a different drive.

@brucify You can uninstall the distro with wsl.exe --unregister <DistributionName> altough beware because it deletes everything inside the linux distro.
@tagMacher I have distros installed on G: (being C: the system drive) without problem. That should be an only Microsoft Store limitation for now.

@onomatopellan
Thank you, it worked for me:

i am sure this question is little unrelated but when i install any app let's say windows terminal i get this below error file system error "12030"

i am using https://store.rg-adguard.net/ to download apps, but i can see exe's when i extract them

@smjltd Even If the setup file is a msix like in Windows Terminal I'm afraid you can't choose the folder where to install it.

I am sure this question is little unrelated

This is fully unrelated. If you're downloading msix, you can use 7zip to extract it anywhere.

wow that did work you have to basically extract twice, thanks mate 👍🏼

@smjltd If the setup file is a .msix like in Windows Terminal I'm afraid you can't choose the folder where to install it.
You can install Windows Terminal via Chocolatey
choco install microsoft-windows-terminal

Chocolatey - Windows Terminal
Github - Windows Terminal

I tried to extract AppxBundle and run without internet x64.appx or x64.msix with 7zip and some UWP apps successfully installed it.
It worked with Files UWP - Beta, Ear Trumpet, Fluent Terminal, but didn't work for XAML Studio

I tried to extract AppxBundle and run without internet x64.appx or x64.msix with 7zip and some UWP apps successfully installed it.
It worked with Files UWP - Beta, Ear Trumpet, Fluent Terminal, but didn't work for XAML Studio

i tried x64.appx behind firewall one that has full access it always fails with 12030 error.

@brucify You can uninstall the distro with wsl.exe --unregister <DistributionName> but beware because it deletes everything inside the linux distro.
@tagMacher I have distros installed on G: (being C: the system drive) without problem. That should be an only Microsoft Store limitation for now.

Thanks a lot @onomatopellan
i'm on Windows 10 Enterprise LTSC (1809) i was able to install wls just unzipping the appx package, but i'm unable to uninstall it, my wsl command has very few parameters :( i do not even have lxrun.
Thanks


feat: bifrostWallet support

@antondalgren Please enable changes from maintainers on this PR

Closing for now, but you can open another PR with maintainer edits enabled.


Error: Activity has leaked IntentReceiver

In Activity's onCreate() I call:
        connectivityManager = getSystemService(Context.CONNECTIVITY_SERVICE) as ConnectivityManager
        RxNetwork.connectivityChanges(this, connectivityManager)
                .subscribe({ connected: Boolean -> if (connected) refreshApp() })

when I press back button this error shows up:
Activity com.mypackage.MainActivity has leaked IntentReceiver io.andref.rx.content.OnSubscribeBroadcastRegister$1@c58a89f that was originally registered here. Are you missing a call to unregisterReceiver()?
                                                                     android.app.IntentReceiverLeaked: Activity com.mypackage.MainActivity has leaked IntentReceiver io.andref.rx.content.OnSubscribeBroadcastRegister$1@c58a89f that was originally registered here. Are you missing a call to unregisterReceiver()?
                                                                         at android.app.LoadedApk$ReceiverDispatcher.<init>(LoadedApk.java:1030)
                                                                         at android.app.LoadedApk.getReceiverDispatcher(LoadedApk.java:817)
                                                                         at android.app.ContextImpl.registerReceiverInternal(ContextImpl.java:1256)
                                                                         at android.app.ContextImpl.registerReceiver(ContextImpl.java:1236)
                                                                         at android.content.ContextWrapper.registerReceiver(ContextWrapper.java:583)
                                                                         at io.andref.rx.content.OnSubscribeBroadcastRegister.call(OnSubscribeBroadcastRegister.java:65)
                                                                         at io.andref.rx.content.OnSubscribeBroadcastRegister.call(OnSubscribeBroadcastRegister.java:28)
                                                                         at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:50)
                                                                         at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:30)
                                                                         at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:50)
                                                                         at rx.internal.operators.OnSubscribeLift.call(OnSubscribeLift.java:30)
                                                                         at rx.Observable.subscribe(Observable.java:8759)
                                                                         at rx.Observable.subscribe(Observable.java:8726)
                                                                         at rx.Observable.subscribe(Observable.java:8549)
                                                                         at com.mypackage.MainActivity.onCreate(MainActivity.kt:52)

It looks like, it comes from OnSubscribeBroadcastRegister's method call()
Thanks

I have the same issue. Maybe you need to release the subscriber to fix this issue


Update beautysh.py

Proposed fix for issue 57 (https://github.com/lovesegfault/beautysh/issues/57). Tested in my local environment and working there.

Thank you so much for your contribution! Could you please add a test so this doesn't regress in the future?

Hmm, this seems to just undo #7, which was a fix to #2. It may require some fiddling to make sure both the test cases presented in #2 and #57 work.

Yes, I realised that after submitting the PR. My version fixes the #57 test case, but not the #2 test case. So it seems a more nuanced fix is needed. Sorry…
FWIW, the #2 test's if/elif statement is at the top level of the file, while the #57 test's statement is in a function. I'm seeing the problem in functions, too. So the issue isn't just "elif", it's "elif in a function".

Yeah, I needed to test more. I've added my findings as a comment on #57 and will close this PR. Sorry to waste your time!


extended precision python input

extended precision floating point values can be passed into devsim as strings

processing on Python side can be done with gmpy2 with mpfr and ieee(128) context
https://gmpy2.readthedocs.io/en/latest/mpfr.html


PR for issue #237

As discussed, I have resolved all the conflict and worked on the modification for issue #237. Kindly review and let me know for any concern.

In csc-site-ssapp/code/templates/shipment/tab-navigator.html  the tab should be re-indexed but there is another task for this: #263


[composer] edge - debugging shows "could not connect to debugger" after success debug

Description:
Please see below,

Affected Versions:
GA
OS, DB, other environment details and versions:
Windows 10 | Edge
ballerina-platform-windows-installer-x64-0.970.0

This issue is not reproducible in ballerina 0.981.2-SNAPSHOT so closing this.


Enable an ArchUnit test that validates that there are no package cycles

Another small tool to keep the API well designed. It adds an architecture test that validates that there are no package cycles.
The package design is specified in the PlantUml file design.puml. All classes must use only dependencies that are allowed by the package graph in that file. The package graph must be an acyclic directed graph.

Cool! Never thought plantUML can be used in this way, thanks!


ProcessException: ProcessException: %1 is not a valid Win32 application.    Command: D:\Easy a Delivery\mobile\development\deliverer\android\gradlew.bat -v

command
flutter run
exception
ProcessException: ProcessException: %1 is not a valid Win32 application.
Command: D:\Easy a Delivery\mobile\development\deliverer\android\gradlew.bat -v
#0      runAsync (package:flutter_tools/src/base/process.dart:232:33)
<asynchronous suspension>
#1      runCheckedAsync (package:flutter_tools/src/base/process.dart:251:34)
<asynchronous suspension>
#2      _initializeGradle (package:flutter_tools/src/android/gradle.dart:225:9)
<asynchronous suspension>
#3      _ensureGradle (package:flutter_tools/src/android/gradle.dart:206:37)
<asynchronous suspension>
#4      _readGradleProject (package:flutter_tools/src/android/gradle.dart:134:31)
<asynchronous suspension>
#5      _gradleProject (package:flutter_tools/src/android/gradle.dart:111:34)
<asynchronous suspension>
#6      getGradleAppOut (package:flutter_tools/src/android/gradle.dart:105:29)
<asynchronous suspension>
#7      AndroidApk.fromAndroidProject (package:flutter_tools/src/application_package.dart:160:23)
<asynchronous suspension>
#8      ApplicationPackageFactory.getPackageForPlatform (package:flutter_tools/src/application_package.dart:47:32)
<asynchronous suspension>
#9      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:352:56)
<asynchronous suspension>
#10     HotRunner.run (package:flutter_tools/src/run_hot.dart:251:39)
<asynchronous suspension>
#11     RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:469:37)
<asynchronous suspension>
#12     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:478:18)
<asynchronous suspension>
#13     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:383:33)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#15     _rootRun (dart:async/zone.dart:1124:13)
#16     _CustomZone.run (dart:async/zone.dart:1021:19)
#17     _runZoned (dart:async/zone.dart:1516:10)
#18     runZoned (dart:async/zone.dart:1463:12)
#19     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#20     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:375:20)
#21     CommandRunner.runCommand (package:args/command_runner.dart:197:27)
<asynchronous suspension>
#22     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:396:21)
<asynchronous suspension>
#23     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#24     _rootRun (dart:async/zone.dart:1124:13)
#25     _CustomZone.run (dart:async/zone.dart:1021:19)
#26     _runZoned (dart:async/zone.dart:1516:10)
#27     runZoned (dart:async/zone.dart:1463:12)
#28     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#29     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:356:19)
<asynchronous suspension>
#30     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:112:25)
#31     new Future.sync (dart:async/future.dart:224:31)
#32     CommandRunner.run (package:args/command_runner.dart:112:14)
#33     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:242:18)
#34     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:22)
<asynchronous suspension>
#35     _rootRun (dart:async/zone.dart:1124:13)
#36     _CustomZone.run (dart:async/zone.dart:1021:19)
#37     _runZoned (dart:async/zone.dart:1516:10)
#38     runZoned (dart:async/zone.dart:1500:12)
#39     run.<anonymous closure> (package:flutter_tools/runner.dart:60:18)
<asynchronous suspension>
#40     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:153:29)
<asynchronous suspension>
#41     _rootRun (dart:async/zone.dart:1124:13)
#42     _CustomZone.run (dart:async/zone.dart:1021:19)
#43     _runZoned (dart:async/zone.dart:1516:10)
#44     runZoned (dart:async/zone.dart:1463:12)
#45     AppContext.run (package:flutter_tools/src/base/context.dart:152:18)
<asynchronous suspension>
#46     runInContext (package:flutter_tools/src/context_runner.dart:56:24)
<asynchronous suspension>
#47     run (package:flutter_tools/runner.dart:51:10)
#48     main (package:flutter_tools/executable.dart:62:9)
<asynchronous suspension>
#49     main (file:///C:/Users/Abderrazzak/development/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#50     _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:299:32)
#51     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:172:12)

flutter doctor
[✓] Flutter (Channel stable, v1.7.8+hotfix.3, on Microsoft Windows [Version 10.0.16299.431], locale ar-MA)
    • Flutter version 1.7.8+hotfix.3 at C:\Users\Abderrazzak\development\flutter
    • Framework revision b712a172f9 (11 days ago), 2019-07-09 13:14:38 -0700
    • Engine revision 54ad777fd2
    • Dart version 2.4.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\Abderrazzak\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\Users\Abderrazzak\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio Preview\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 3.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 23.2.1
    • Dart plugin version 171.4424
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b01)

[✓] Android Studio (version 3.4)
    • Android Studio at C:\Program Files\Android\Android Studio Preview
    • Flutter plugin version 32.0.2
    • Dart plugin version 183.4588.61
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)

[!] IntelliJ IDEA Community Edition (version 2018.3)
    • IntelliJ at C:\Program Files\JetBrains\IntelliJ IDEA Community Edition 2018.3.2
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.dev/intellij-setup/#installing-the-plugins

[✓] VS Code, 64-bit edition (version 1.34.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.1.0

[✓] Connected device (1 available)
    • SM J200F • 92012898d9238337 • android-arm • Android 5.1.1 (API 22)

! Doctor found issues in 1 category.

This is fixed on master.


Virgin Radio TV

Source
https://www.superguidatv.it/programmazione-canale/oggi/guida-programmi-tv-virgin-radio/461/
Channels
Virgin Radio TV - VirginRadioTV.it
Notes
No response

The linked source requires a new config file.
I've added the EPG info from guidatv.sky.it in the meantime - https://github.com/iptv-org/epg/pull/1587/commits/75e6b6a0a1d73a42d82c432ccec23ff77cb67b9a

The guide for Virgin Radio TV is now available at this link:
https://iptv-org.github.io/epg/guides/it/superguidatv.it.xml


This command is also necessary to enable indexes

Seems ambiguous to me.


Web UI should render footers on attachments

This is a Feature Proposal
Description
Web UI doesn't currently render attachment footers, it should.

As discussed, we will have to filter out footers that don't make sense in the web UI, such as the listening state footer. Since there are no channels (group messaging) in the web UI, we wouldn't want to show a listening state.


fix: negative zero for DefinePlugin

What kind of change does this PR introduce?
bugfix, do not lose the sign of negative zero
Did you add tests for your changes?
Yes
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
Nothing

Thanks


az web app up error: AttributeError: 'int' object has no attribute 'upper'

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up
Errors:
'int' object has no attribute 'upper'
Traceback (most recent call last):
python3.7/site-packages/knack/cli.py, ln 206, in invoke
    cmd_result = self.invocation.execute(args)
cli/core/commands/__init__.py, ln 603, in execute
    raise ex
cli/core/commands/__init__.py, ln 661, in _run_jobs_serially
    results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/__init__.py, ln 652, in _run_job
    cmd_copy.exception_handler(ex)
...
cli/command_modules/appservice/custom.py, ln 2993, in webapp_up
    create_app_service_plan(cmd, rg_name, plan, _is_linux, False, sku, 1 if _is_linux else None, location)
cli/command_modules/appservice/custom.py, ln 1390, in create_app_service_plan
    sku = _normalize_sku(sku)
cli/command_modules/appservice/utils.py, ln 20, in _normalize_sku
    sku = sku.upper()
AttributeError: 'int' object has no attribute 'upper'

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az webapp up --sku {} -n {} -l {}

Expected Behavior
Environment Summary
Darwin-19.0.0-x86_64-i386-64bit
Python 3.7.5
Shell: bash

azure-cli 2.0.76

Additional Context

Duplicate of 11221

Duplicate of #11221

Fixed by https://github.com/Azure/azure-cli/pull/11019


ClientArea>Manage Product Pterodactyl link and credentials

Adding login credentials a link to the panel install and eventually some basic controls as in start/stop/restart would be helpfull

There is already Go To Service button allowing you to go to the service on Pterodactyl's side and manage it.

But there's none for the clientarea

Added in https://github.com/TrixterTheTux/Pterodactyl-WHMCS/commit/34d7bb8b80eaeb50ae696bbf8f2485e5a1740351


Question concerning training RNNs

Hi I have some questions with regards to generating the train, val and test sets for the temperature forecasting problem.

When using the timeseries_dataset_from_array  tool, we passed the argument shuffle= True. I thought we had to maintain the order of things within the data, so I expected shuffling to be harmful to the model. It turns out however that if I do not shuffle I instead get a worst model.
When building our model, from my understanding of the code, the input data still has the temperature column within it, even though we are using temp as our targets. Is that not some kind of data leak ?

Hi @PelFritz

shuffling in this sense does not mean you are shuffling the order within each sequence, but the order of sequences drawn. Assume your sequence data is [0, 1, 2, 3, 4, 5, 6, 7, 8, 9] and your sequence_length = 2, then shuffle=False would draw your sequences in chronological order [0, 1], [1, 2], [2, 3], [3,4], ... while shuffle=True could lead to [3, 4], [0, 1], [5, 6], [2, 3], ...

You see that the order within sequences is still intact.

I don't own the book yet, so I'm answering this without further context. I would assume that the temperature column in the input data being still present does not matter (rather the opposite: it is essential) because you are trying to predict the temperature at a point in the future. That's why you have the delay variable for data=raw_data[:-delay] and targets= temperature[:delay].

Hi @pkienle thank you for your insights.


Paging does not work for the export

When a search contains more results than the page size, the paging did not work for me in the exporter. I only ever got the documents from the first page exported.
I am totally new to the search client and fixed the problem by replacing the code in the PartitionExporter.ExportPartitionAsync with the following. I am not sure this is the best approach?
Thanks for the repository, other than this problem it worked very well!
       var options = new SearchOptions
        {
            Filter = partition.Filter,
            Size = _pageSize,
            Skip = 0,
            IncludeTotalCount = true,
        };
        AddSelect(options);
        options.OrderBy.Add($"{_partitionFile.FieldName} asc");

        SearchResults<SearchDocument> searchResults = null;
        do

        {
            searchResults = await _searchClient.SearchAsync<SearchDocument>(searchText: string.Empty, options: options,
                cancellationToken: cancellationToken);

            await _partitionWriter.WritePartitionAsync(partitionId, searchResults, cancellationToken);
            options.Skip += _pageSize;
        } while (options.Skip < searchResults.TotalCount);

@anev-auror  it almost fix the problem. You should also modify the FilePartitionWriter and IFilePartitionWriter to include a page number.
I did this and worked:
        public async Task WritePartitionAsync(int partitionId, int page, SearchResults<SearchDocument> searchResults, CancellationToken cancellationToken)
        {
            if (!Directory.Exists(_directory))
            {
                Directory.CreateDirectory(_directory);
            }

            string exportPath = Path.Combine(_directory, $"{_indexName}-{partitionId:D3}-{page:D3}-documents.json");
            using FileStream exportOutput = File.Open(exportPath, FileMode.OpenOrCreate, FileAccess.Write, FileShare.Read);

            await foreach (Page<SearchResult<SearchDocument>> resultPage in searchResults.GetResultsAsync().AsPages())
            {
                foreach (SearchResult<SearchDocument> searchResult in resultPage.Values)
                {
                    JsonSerializer.Serialize(exportOutput, searchResult.Document);
                    exportOutput.WriteByte((byte)'\n');
                }
            }
        }

Sorry it's not obvious how to use it. You need to invoke it from the command line like so
 dotnet run partition-index

Description:
  Partitions the data in the index between the upper and lower bound values into partitions with at
  most 100,000 documents.

Usage:
  export-data partition-index [options]

Options:
  --endpoint <endpoint> (REQUIRED)      Endpoint of the search service to export data from. Example:
                                        https://example.search.windows.net
  --admin-key <admin-key>               Admin key to the search service to export data from. If not
                                        specified - uses your Entra identity
  --index-name <index-name> (REQUIRED)  Name of the index to export data from
  --field-name <field-name> (REQUIRED)  Name of field used to partition the index data. This field must
                                        be filterable and sortable.
  --lower-bound <lower-bound>           Smallest value to use to partition the index data. Defaults to
                                        the smallest value in the index. []
  --upper-bound <upper-bound>           Largest value to use to partition the index data. Defaults to
                                        the largest value in the index. []
  --partition-size <partition-size>     Maximum size of a partition. Defaults to 100,000. Cannot exceed
                                        100,000 [default: 100000]
  --partition-path <partition-path>     Path of the file with JSON description of partitions. Should
                                        end in .json. Default is <index name>-partitions.json []
  -?, -h, --help                        Show help and usage information

Once you partition the index, then you can export. With the modifications you have made, the export will not work in parallel
 dotnet run export-partitions

Description:
  Exports data from a search index using a pre-generated partition file from partition-index

Usage:
  export-data export-partitions [options]

Options:
  --partition-path <partition-path> (REQUIRED)     Path of the file with JSON description of
                                                   partitions. Should end in .json.
  --admin-key <admin-key>                          Admin key to the search service to export data from.
                                                   If not specified - uses your Entra identity
  --export-path <export-path>                      Directory to write JSON Lines partition files to.
                                                   Every line in the partition file contains a JSON
                                                   object with the contents of the Search document.
                                                   Format of file names is <index name>-<partition
                                                   id>-documents.json [default: .]
  --concurrent-partitions <concurrent-partitions>  Number of partitions to concurrently export. Default
                                                   is 2 [default: 2]
  --page-size <page-size>                          Page size to use when running export queries.
                                                   Default is 1000 [default: 1000]
  --include-partition <include-partition>          List of partitions by index to include in the
                                                   export. Example: --include-partition 0
                                                   --include-partition 1 only runs the export on first
                                                   2 partitions []
  --exclude-partition <exclude-partition>          List of partitions by index to exclude from the
                                                   export. Example: --exclude-partition 0
                                                   --exclude-partition 1 runs the export on every
                                                   partition except the first 2 []
  --include-field <include-field>                  List of fields to include in the export. Example:
                                                   --include-field field1 --include-field field2. []
  --exclude-field <exclude-field>                  List of fields to exclude in the export. Example:
                                                   --exclude-field field1 --exclude-field field2. []
  -?, -h, --help                                   Show help and usage information

You will unfortunately have to undo your modifications, otherwise it might not work

I also created a code sample and fixed the reported bug
#15