Update cats-laws, cats-testkit to 1.6.0

Updates

org.typelevel:cats-laws
org.typelevel:cats-testkit

from 1.5.0 to 1.6.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention @scala-steward in the comments below.
Have a nice day!

Codecov Report

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

@@           Coverage Diff           @@
##           master      #49   +/-   ##
=======================================
  Coverage   44.63%   44.63%
=======================================
  Files          31       31
  Lines         289      289
=======================================
  Hits          129      129
  Misses        160      160

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 26da815...d4b1453. Read the comment docs.


 Closing this issue.

fixed in https://github.com/slsa-framework/example-package/pull/373


Fix folder deletion with VSCode

VSCode has issue (however by design at current version) where removed folders doesn't send any notifications of removed files because file watcher has some kind of performance optimizations to ignore them.
However watching folders removal is possible. This PR adds support for omnishar-roslyn side so clients can notify removed directory instead of single files if needed -> which can be impelemented properly with current VSCode file watcher.
This fixes one of most common issue that requires omnisharp restart on any directory rename/move/... operation.
Vscode side implementation: https://github.com/OmniSharp/omnisharp-vscode/pull/3829

Ready for review @filipw @JoeRobich @david-driscoll


API to add/remove annotations from policies

Category
User level API features/changes
Describe the feature you'd like to request
I'd like to programmatically add and remove annotation from policies. However there is no such API in the policy struct .
Describe alternatives you've considered
The alternative I considered was to serialize the policy and change the annotations at the JSON level.
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

I think this would be solved by an implementation of #366


Find why the plugin crashed when the file size is -1

The [api]{http://manpages.ubuntu.com/manpages/trusty/man2/read.2.html} indicates that the EOF file is indicated by returning a zero. This is true when the size of the file is sys.maxint. As soon as the file size is -1, makes it a huge number and returning zero does not work.  [This]{http://stackoverflow.com/questions/12275831/why-is-the-st-size-field-in-struct-stat-signed} tells that the size should never be -1 but I haven't been able to find a correct explanation.

It looks like the file size has to be a number greater than the actual size of the file


NeonAnimatable element does not appear when added programmatically to the NeonAnimatedPages

NeonAnimatable element is not added programmatically to already existing NeonAnimatedPages.
Here's the html code:
<paper-button on-click = "addPage">Add page</paper-button>

<neon-animated-pages id = "mainPages" selected ="0"
                                 entry-animation="slide-from-right-animation"
                                 exit-animation="slide-left-animation">

    <neon-animatable on-click = "goToNextPage">Page One</neon-animatable>
    <neon-animatable on-click = "goToNextPage">Page Two</neon-animatable>

</neon-animated-pages>

Basically every time the button is clicked, a new page should be added.
And it is added as a child, but still doesn't appear on the web page. The rotation I do with the goToNextPage method does not include new pages and continues to switch between the first two pages which are initially put into html file.
Dart code:
@PolymerRegister('pages-test')
class Pages extends PolymerElement {
  NeonAnimatedPages mainPages;

  Pages.created() : super.created() {
    mainPages = querySelector("#mainPages") as NeonAnimatedPages;
  }

  @reflectable
  void goToNextPage(event, [_]) {
    mainPages.selectNext();
  }

  @reflectable
  void addPage(event, [_]) {
    NeonAnimatable newPage = new NeonAnimatable();
    newPage.text = "New page";
    newPage.onClick.listen(goToNextPage);
    mainPages.children.add(newPage);
  }
}

pubspec.yaml:
environment:
  sdk: '>=1.9.0 <2.0.0'

dependencies:
  browser: ^0.10.0
  polymer_elements: ^1.0.0-rc.3
  polymer: ^1.0.0-rc.6
  reflectable: ^0.3.3
  web_components: '>=0.11.3 <0.13.0'
  polymer_interop: ^1.0.0-rc.5

Is that a bug or am I missing something?

You have to use the Polymer dom apis when manipulating the dom of Polymer elements in order to ensure that all side effects are taken into account. In this case that would look like this:
Polymer.dom(mainPages).appendChild(newpage);


Object is possibly undefined when optional key is conditionally assigned

TypeScript Version:  4.0.3

Search Terms:
typescript object might be undefined when using conditional assignment
Code
// CASE 1
enum FOOBAR_KEYS {
    FOO ="foo",
    BAR = "bar",
    BAZ = "baz"
}
type Foobar = Record<FOOBAR_KEYS, number>;

let obj: Partial<Foobar> = {}
obj[FOOBAR_KEYS.BAR] = obj[FOOBAR_KEYS.BAR] || 0;
obj[FOOBAR_KEYS.BAR]++; // Object is possibly 'undefined'.(2532)

// CASE 2
interface FooInterface {
    key: { [K in FOOBAR_KEYS]?: number }
}

let obj_2: FooInterface['key'] = {};
obj_2[FOOBAR_KEYS.BAR] = obj_2[FOOBAR_KEYS.BAR] || 0;
obj_2[FOOBAR_KEYS.BAR]++; // Object is possibly 'undefined'.(2532)

// CASE 3
if(obj[FOOBAR_KEYS.BAR] === undefined) {
    obj[FOOBAR_KEYS.BAR] = 0;
}

obj[FOOBAR_KEYS.BAR]++; // Object is possibly 'undefined'.(2532)

Expected behavior:
Typescript understands that the key cannot be undefined as undefined will get 0 to be assigned
Actual behavior:
Object is possibly 'undefined'.(2532)
Playground Link:
https://www.typescriptlang.org/play?noUnusedLocals=true&noUnusedParameters=true#code/KYOwrgtgBAYg8nAQgQQEoH0DSBRAmgZSgG8BYAKCktgSgF4AiAMwHtn6AacqqFVOqegCMAhgCcOXKigBa-IcIBe9cgF9y5AC4BPAA7BYrEaP6pgAY2aiAJgB54SNFjz52UcBEHBRAPgDc6sgAbYA0oZkEAKwAuKAAFMQ0AS2FAu0Mxb34iNTJwiIBte14nAgA6XgBdfjzChGKcMsqoAB9mqAAGf1zI2ocMBvxytAqAahGu8kSQDS9GYTN9GFYASWnZ+f1SCioAa2AtGKIofMwoKeo+kvwKgH4Y909jHJzyYNC89AAmGKXmVZnRHMFvkAOR7LQgqq0YgqLofT69erOIaoKFhSJfRGOAYoqqtDpwjEIorY5GVMYTMgBRKMAAUNRJ-TJwzotGhYBAVmAjCmwCsAEpiJJKAy6qTGizoe1VAFRZcceTxkA
Related Issues:
https://github.com/microsoft/TypeScript/issues/31060

Related (perhaps duplicate?): #10530

It's a true duplicate; the only reason this doesn't work is because this kind of expression doesn't generate a CFA node

It's a true duplicate; the only reason this doesn't work is because this kind of expression doesn't generate a CFA node

Hi @RyanCavanaugh and thanks for the comment.
Could you elaborate on what a CFA node is and maybe provide some additional reading/references for this?


Remove Members listed from Team

Please remove members listed on teams page

Amy Burcham from Marketing
Khairul Akmal from Design
Sush from Design

Done it, will push when has ready for engineering label.


Raspberry Pi Error Startup W/ Emulation Station (Help me bill gates)

I have a Raspberry Pi B and yesterday I configured my Pi to start up  "To CLI". I reset it though the gooey and I started up Emulation Station. While I was playing, I unplugged The power cord. After I plugged it in, it would boot up to a what I would assume is a crash screen. When I try to do anything on this ''Crash Screen'',  it does absolutely nothing. I tryed startx, but to no avail. Someone help ne!

While I was playing, I unplugged The power cord.
There's your problem. The SD card is likely corrupt from the power loss while in use. You may need to reinstall to  a new SD card.


Resolving actionpack issue

Bundler could not find compatible versions for gem "actionpack":
In Gemfile:
actionpack (~> 6.0)
routing-filter (~> 0.6.2) was resolved to 0.6.2, which depends on
  actionpack (>= 4.2, < 6)

If you change action pack from 6.0, then a lot more stuff is not compatible.

Gemfile
source 'https://rubygems.org'
git_source(:github) { |repo| "https://github.com/#{repo}.git" }
ruby '2.6.4'
Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
#gem 'rails', '~> 6.0'
Use sqlite3 as the database for Active Record
gem 'sqlite3', '~> 1.4'
Use Puma as the app server
gem 'puma', '~> 3.11'
Use SCSS for stylesheets
gem 'sass-rails', '~> 6.0'
Transpile app-like JavaScript. Read more: https://github.com/rails/webpacker
gem 'webpacker', '~> 4.0'
Turbolinks makes navigating your web application faster. Read more: https://github.com/turbolinks/turbolinks
gem 'turbolinks', '~> 5'
Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
gem 'jbuilder', '~> 2.7'
Use Redis adapter to run Action Cable in production
gem 'redis', '~> 4.0'
Use Active Model has_secure_password
gem 'bcrypt', '~> 3.1.7'
Use Active Storage variant
gem 'image_processing', '~> 1.2'
Reduces boot times through caching; required in config/boot.rb
gem 'bootsnap', '>= 1.4.2', require: false
group :development, :test do
Call 'byebug' anywhere in the code to stop execution and get a debugger console
gem 'byebug', platforms: [:mri, :mingw, :x64_mingw]
end
group :development do
Access an interactive console on exception pages or by calling 'console' anywhere in the code.
gem 'web-console', '>= 3.3.0'
gem 'listen', '>= 3.0.5', '< 3.2'
Spring speeds up development by keeping your application running in the background. Read more: https://github.com/rails/spring
gem 'spring'
gem 'spring-watcher-listen', '~> 2.0.0'
end
group :test do
Adds support for Capybara system testing and selenium driver
gem 'capybara', '>= 2.15'
gem 'selenium-webdriver'
Easy installation and use of web drivers to run system tests with browsers
gem 'webdrivers'
end
Windows does not include zoneinfo files, so bundle the tzinfo-data gem
gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]
#Spree
gem 'spree', '~> 4.0.0.beta'
gem 'spree_auth_devise', '~> 4.0.0.rc1'
gem 'spree_gateway', '~> 3.6'
#Solidus
gem "solidus_auth_devise"
gem 'solidus_i18n', '~> 2.0'
gem 'rails-i18n', '~> 5.1'
gem 'kaminari-i18n', '~> 0.5.0'
gem 'routing-filter', '~> 0.6.2'
#Actionpack
gem 'actionpack', '~> 6.0'


Suggestion: Separate creative tab for Portable Tank fluids

Exactly what it says on the tin. Quality of life improvement if you got mods like Minechem installed that add a bazillion liquids.
Noticed it was quite laggy to scroll over the creative tab because of that.

You can disable the tanks by setting the "prefilledPortableTanks" config property to false :)


Using a  standards compliant date format.

plottr/plottr/plottr.py  line 38:
TIMEFMT = "[%Y/%m/%d %H:%M:%S]"
is not a standard date format.
I think it should be:
TIMEFMT = "[%Y-%m-%d %H:%M:%S]"
so that it is both a valid RFC2822 and valid ISO 8601.
Just a small change to be standards compliant

agreed, fixed :)


Update Pydantic integration to match Django's integration

@la4de proposed a great way of integrating strawberry with Django:
@strawberry.django.type(models.Fruit)
class Fruit:
    id: auto
    name: auto
    color: 'Color'

This is better than what we have with Pydantic as you might get a better autocomplete, better type definition (if we add mypy support for it) and it makes it easier to override fields (even when you only want to override the name or description, as you can use auto as the type).
We should do the same thing[1] for Pydantic and promote it be a non-experimental feature.
Also worth mentioning that adding Pydantic support might mean that we also get support for https://github.com/tiangolo/sqlmodel which is probably going to be super helpful!
[1] We should keep the old way of passing fields as deprecated for now, and remove in version 1 or 2 :)

Perhaps we could use an optional argument to the decorator to tell it to copy the existing fields as is, that way the user could pick between the more explicit version with better autocomplete or the implicit version with less boilerplate?

yes, I think we can allow that 😊
@strawberry.django.type(models.Fruit, all_fields=True)
class Fruit:
    ...

something like this, or maybe more explicit expose_all_fields=True :)

Another option:
@strawberry.django.type(models.Fruit, fields="__all__")
class Fruit:
    ...

would that allow this:
@strawberry.django.type(models.Fruit, fields=["id", "type"])
class Fruit:
    ...

?

Yeah, fields take either a list or a string.

Yeah, fields take either a list or a string.

This is what wee do already though 😊 not sure I like the idea how having two ways of declaring fields (auto vs fields list)

Oh I see what you mean, I was just thinking about the current API. For the new API all_fields is probably better.


Error Run notebooks without some estimators

Hello
I have noticed ElasticNet (EN) method is computationally expensive. Since I'm only planning to use results with Lasso, I commented the lines that refer to EN in the provided notebook examples. For instance:
estimators = {
    "lasso": lasso_cv,
    "alasso": alasso_cv,
   # "en": en_cv,
   # "sgl": sgl_cv,
    "stabl_lasso": stabl,
    "stabl_alasso": stabl_alasso,
 #   "stabl_en": stabl_en,
 #   "stabl_sgl": stabl_sgl,
}

However, this results in an error. Please, see below:
Run CV on SSI dataset
Traceback (most recent call last):
  File "\someaddress\Stabl\Notebook examples\run_cv_SSI_modified.py", line 113, in <module>
    multi_omic_stabl_cv(
  File "\someaddress\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\sklearn\utils\_testing.py", line 158, in wrapper
    return fn(*args, **kwargs)
  File "\someaddress\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.9_qbz5n2kfra8p0\LocalCache\local-packages\Python39\site-packages\stabl\multi_omic_pipelines.py", line 144, in multi_omic_stabl_cv
    en = estimators["en"]
KeyError: 'en'

Hello @lazaromsanchezr ,
If you want to remove some models from the run, you just need to comment their names in the variable models. You have to keep them uncommented in the variable estimators.
For example, in the file run_cv_CFRNA.py, the variable models is set to:
# Associated model names. Here you can comment what you don't want to test ()
models = [
    "STABL Lasso",
    "Lasso",
    "STABL ALasso",
    "ALasso",
    # "STABL ElasticNet",
    # "ElasticNet"
]

So both ElasticNet models are not used.
I hope it is more clear now.
PS: I change the name of the issue so it is more adequate to the error.

Hi Xavier,
Thank you, that worked!


Latexify: Replace two new-line characters with  HTML tag

Description of change

Added feature to replace two \n characters with <br /> HTML tag, so \n\n means break line.
Pull-Request Checklist

[ ] Code is up-to-date with the main branch
[ ] npm run lint passes with this change
[ ] npm run test passes with this change
[ ] This pull request links relevant issues as Fixes #0000
[ ] There are new or updated unit tests validating the change
[ ] Documentation has been updated to reflect this change
[ ] The new commits follow conventions outlined in the conventional commit spec

Codecov Report

Merging #4 (0d47221) into main (505c033) will increase coverage by 40.14%.
The diff coverage is 80.00%.

@@             Coverage Diff             @@
##             main       #4       +/-   ##
===========================================
+ Coverage   50.00%   90.14%   +40.14%
===========================================
  Files           2        4        +2
  Lines          12       71       +59
  Branches        1       15       +14
===========================================
+ Hits            6       64       +58
- Misses          6        7        +1

Impacted Files
Coverage Δ

src/index.tsx
95.23% <50.00%> (ø)

src/utils/latexify.tsx
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


oem: ibm: fileTable: Add lid 81e00670 (#488)

Add read-only lid 81e00670 which is a partition table that hostboot uses to get information about the other partition files.
Tested: Verified this lid was read by hostboot on a p10bmc system and that the system powered on.
Change-Id: If09ec0ae0938925f8a147dc28ea2d8b24f94c13d

Cherry-pick the following 1060 commit into 1050:

https://github.com/ibm-openbmc/pldm/commit/4971dcff3b810e1048a97d7bca016112f9b6d805

jenkins run tests please


Update dlp-conditions-and-exceptions.md

Line 88 Recipient is member of needs to be changed to reflect what we have in the DLP policy configurator. As for the Sender is member of, M365 groups are not actually supported. What we see in the EXO DLP policy configurator is:
Looks for messages sent to members of distribution groups, dynamic distribution groups, or mail-enabled security groups.

Learn Build status updates of commit 71d053e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-conditions-and-exceptions.md
:white_check_mark:Succeeded

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

Learn Build status updates of commit 77a9a7a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/compliance/dlp-conditions-and-exceptions.md
:white_check_mark:Succeeded

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

this article will be merged into DLP policy  reference in Q4 FY23


[backport] Update caffe.rst

Backport of #5090

[automatic post] Jenkins, test this please.

Jenkins CI test (for commit d7b1c56f9451e93f73c52214d341043124d4551c) succeeded without errors!

LGTM


"Substring Search" that searches for individual words?

"Substring Search" mode come closest to working for me, but fails a personal critical use case:  searching on multiple substrings, each separated by space. An illustration:

Sample tab or bookmark name: Account Health Report (AHR) notes + links
Substrings I'd like Quick Tabs to match: ahr notes ornotes ahr
Substring that must currently be used to match: ahr) notes

Note that other search types may work, but I find that they return too many responses or are more sophisticated than what I'm looking for.
This is the one feature that blocks me from using Quick Tabs which otherwise looks like an incredibly powerful extension. I'm currently using Popup My Bookmarks 50+ times a day. Search results are great but PMB doesn't support open tabs and has a few other small-to-medium challenges. Would love to switch to QT but search results are a blocker at the moment.
Happy to provide more details, examples, etc. Many thanks in advance!!

Have you tried the word-based Fuse search?


Block non-PDF files in PDF attachment upload process

From https://github.com/NASA-IMPACT/nasa-apt-frontend/pull/486#pullrequestreview-1456970164, comment 3
Current Behavior
Users can upload any file in the PDF Attachment step. The file successfully uploads.
Expected Behavior

[ ] Only PDF files are accepted
[ ] An error toast (with https://github.com/NASA-IMPACT/nasa-apt-frontend/issues/490) appears if non-PDF files are attempted to be uploaded.

@bwbaker1 this is ready for your review on staging.

@wrynearson This is nice! I have a question though. Do we need an error toast if someone has an uploaded pdf and then starts to upload a file but decides to cancel the upload? See the video for more details.
This isn't a huge deal, but it caught my attention while testing.
https://github.com/NASA-IMPACT/nasa-apt-frontend/assets/56319064/c263a4a0-f599-465c-bae5-10b3997e75e3

Oh good catch @bwbaker1, let's see if @frozenhelium can easily change the error toast to only appear when an incorrect file is uploaded (and not when the user cancels an upload like you showed).


Add missing 'Serializer/Context' class to the list of "annotation to attribute" rule

This  MR add a missing class to the list of configured "annotation to attributes" set list: the class Symfony\Component\Serializer\Annotation\Context, which was not yet configured.

Thank you :+1:

Thank you :+1:


Capital letters in GitHub username prevent login

we check whether the url matches the username, and if there are capital letters in the username then this fails

can you tag me on the PR?


How can I set Some rows Disable Edit   When I choose A row In EditForm.

Hi ,Nice to see you!
When I  choose One row  In EditForm. such like this

Can I get the choose value to make some row Edit disable. (as that say attribute [Updatable(false)])....
Is the XYZDialog.ts ? Or  Where Should I code and Code what ?
I need your help , Thank you Sincerely !

In your dialog constructor
form.SomeField.change(e => {
   Serenity.EditorUtils.setReadOnly(form.SomeAnotherField, true);
});

Thank you.. what you said  make a prefect effect !

form.SomeAnotherField.getGridField().toggle(false)

I can't meet the toggle method, just show like this

You need to add parenthesis () after getGridField


Move state to context

The shared state in components.jsx was causing me some trouble. In my isomorphic app, I wanted to be able to load the translated strings I needed dynamically at runtime when a page is requested, but components.jsx was storing the messages in a variable global to the module, which meant that that state was being shared across requests. It seemed to make more sense to me that the list of translations would be stored in a React context, which is a good mechanism for passing data down to child components.
In this branch, the way you set things up is to wrap your main app container in a parent component:
On the server:
let strings = require.context('../shared/translations/bundles', true, /.js$/)(`./${locale}.js`);

 let componentMarkup = renderToString(
     <I18NContainer messages={strings}>
        <RouterContext {...renderProps} />
    </I18NContainer>
);

And similarly in the entry point on the client:
 <I18NContainer messages={strings}>
    <Router history={browserHistory} routes={RootRoute} />
 </I18NContainer>

If you want to declare that you'd like one of your child components to receive the translation context, you can simply do this:
import React from 'react';

let { translated } = require('jsxlate/build/components').default;

class BaseComponent extends React.Component {
    componentWillMount() {
        // .. do some stuff
    }
}

BaseComponent.contextTypes = {
    existingContext: React.PropTypes.object.isRequired
};

export default translated(BaseComponent);

I think translated could easily be converted into an es2016 decorator as well. The reason for this function is to avoid having to declare the boilerplate Class.contextTypes = { ... } stuff every time you create a new component.
Finally, to be able to support i18n('some text'), the babel plugin replaces occurrences of that function with this.context._i18n. I realize this isn't ideal, because it requires this to refer to the component, and I'm sure that there's a cleaner way to do this but I haven't figured out what would be best yet. If you think this overall approach is interesting, I'd love to discuss this part of it more and make it less brittle.
I've tested this setup against my existing app and it's working pretty well. I'm sure there are some other approaches that could also work, but I wasn't able to come up with any other approach that would remove the global module-level state and keep it isolated. I'd love to hear your suggestions.

Hi Steven, So.. This context is a great means to achieve that, and it is totally where I started with jsxlate. As you can see in https://github.com/drd/jsxlate/blob/8634a593818ade9c0c931a49a06bc1c055e146a2/examples/simple/app.jsx, I was setting the context, but also directly setting the messages property of the i18n() function to the bundle. Because both functions and components need to be able to look up the current bundle, I thought it was weird and like I was using context just for the sake of context.
What we do in our isomorphic app is, on the server:
import I18N from 'jsxlate/build/components';

function handleRequest(req, res) {
  const locale = localeFromRequest(req);
  const bundle = bundleForLocale(locale);
  I18N.setMessages(bundle);
  return renderApp(req, res, bundle);
}

And on the client:
const locale = __server_data__.locale;
const bundle = require('i18n/bundle-' + locale);
I18N.setMessages(bundle);
... render app...

This hasn't ever caused us any problems :)

@drd Sorry for my delay in getting back to you. I was out of town for 2 weeks and I'm getting back to this today.
I understand that this approach hasn't caused any issues for you guys so far, but the concern I have is that it creates global state that is manipulated on every single request, and global state makes apps much more difficult to test. It also just instinctively feels strange to me that every single request to a web application server would modify the global state of the application server, rather than generating an individualized response for the request. Context might not be the right way to go here, but I don't think that global module-level state is correct either. At the very least, being able to tie this state to a Flux implementation would make it more in line with existing best practices for state management in React. Does your team use Flux for your app, or are you handling state in a different way?
In an ideal situation, I would be able to simply pass the correct messages to my component and write a test to verify that the replacement happens correctly. From what I can tell right now, setMessages is never exercised by the existing test coverage. To do that would require you to blow away the messages state in the module before every test run, which is kind of ugly. It would be better to use dependency injection here and to be able to compose the component with the necessary pieces in order to be able to write tests.
Is this something you'd be willing to talk about more and consider, or are you pretty set on this approach? I need to work something out to contain the state, and I'd hate to get into a situation where I have to fork so far from the master source.

I'm definitely open to alternatives to the current setMessages behavior, but I don't think that the context solution is right, due to the issues with the non-React i18n function. I'm not sure that I would want to force all translated text to be inside of a React component... Do you have any other ideas?


Supporting type ZodUndefined in OpenAPI/Swagger.

Related to the issue #98

This will not be merged since

There is an .optional() refinement.
There is no need for this, since the original issue can be fixed in a more simple way: #104


Syntax Coloring

Creating custom syntax coloring for the antimony modeling language.
Using past syntax files from Vscode-Antimony project

All syntax coloring from the vscode-antimony project has been implemented into the web editor.


docs(readme): fixup links and bundlephobias

Why
Some links are broken in the global readme of the repository

bundlephobia pages are not working because package does not export entrypoints
they are broken links in growthbook package

How

fix global readme links
re-add main entry to make bundlephobia working

@QuiiBz What is your opinion on re-adding main key on package ?

Ok I will change that


Target dnx451

This reverts e57e1772244b340bc281d6e2205db5848ae55c24 for DNX

Recalling. After chatting with @Eilon, we decided that adding the facades to user's project.json is an acceptable workaround.


Where to use this

Hi, thanks for the scripts. Just wondering where to use them? Wasn’t aware we could use npm modules on CF workers?

Yes, you can use npm modules on a CF worker. Just use Webpack to bundle one final script.
If this was your worker script worker.js:
import cloudflareEdgeProxy from "cloudflare-edge-proxy";

const config = {
    defaultBackend: "https://example.com"
};

const proxy = cloudflareEdgeProxy(config);

addEventListener("fetch", event => {
    event.respondWith(proxy(event));
});

Then you could build a single bundle for CF with a Webpack config something like this:
module.exports = {
    entry: { worker: "./worker" },
    target: "webworker",
    module: {
        rules: [
            {
                test: /\.js$/,
                exclude: /(node_modules)/,
                use: {
                    loader: "babel-loader",
                    options: {
                        presets: [
                            [
                                "@babel/preset-env",
                                {
                                    targets: { chrome: "70" }
                                }
                            ]
                        ]
                    }
                }
            }
        ]
    }
};


Error in /auth/logout

Any time I hit the logout endpoint I get this error. If I refresh a couple of times the errors goes away I I'm able to logout.
'LogoutView' should either include a `serializer_class` attribute, or override the `get_serializer_class()` method.

django = 1.7.4
djoser = 0.2.0
django rest framework = 3.0.3

oh nevermind if I post to it, it works fine.


Provide open parameters schema

Let the CDS data store return an open parameters schema according to https://github.com/dcs4cop/xcube/blob/master/docs/source/storeconv.md.

Implemented as part of #1 .


Add pages to show summary coverage status

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Hi @joeyjiaojg,
Please write some commit description about what this is doing and motivation for this change.
This driver_path_map.json looks too non-scalable and dependent on exact kernel version. Even if somebody will describe whole kernel once (who?) it will get out of date quickly, so will be constant maintenance cost and pain. Have you considered extracting this info automatically from kernel sources?

@googlebot I signed it!

@googlebot I signed it!

driver_path_map.json

I think it can only be used as an example. For each kind of kernel tree, it might be different and I didn't see any way to pulled it from kernel source file.

@googlebot I signed it!

driver_path_map.json

@dvyukov I removed the driver_path_map.json, and it's not a must dependency for the change now.
If anyone added it in their own repo, it will show more rich data.

My concern is that since this does not work out-of-the-box, nobody besides you will use it. We generally try to make things scalable and work for everybody out-of-the-box rather than requiring manual tuning, manual bruteforcing and duplicating work across all users. Say, you are specifying what is "sound" subsystem in your private config, but sound subsystem is mostly for the for any linux version.
A notion of "subsystem" come up multiple times: #544, #550, #608, #1545. We already can attribute kernel bugs to files and files should be attributed to subsystems.
The current most promising idea is to do this attribution based on mailing lists in MAINTAINERS files. For a private fork one could then patch MAINTAINERS accordingly to accommodate new files, etc. If you implement this, this feature will immediately useful for all syzkaller users.

My concern is that since this does not work out-of-the-box, nobody besides you will use it. We generally try to make things scalable and work for everybody out-of-the-box rather than requiring manual tuning, manual bruteforcing and duplicating work across all users. Say, you are specifying what is "sound" subsystem in your private config, but sound subsystem is mostly for the for any linux version.
A notion of "subsystem" come up multiple times: #544, #550, #608, #1545. We already can attribute kernel bugs to files and files should be attributed to subsystems.
The current most promising idea is to do this attribution based on mailing lists in MAINTAINERS files. For a private fork one could then patch MAINTAINERS accordingly to accommodate new files, etc. If you implement this, this feature will immediately useful for all syzkaller users.

@dvyukov These ideas looks great, my change didn't intent to do that of #544, we internally use the commit to compare coverage between builds. And MAINTAINERS won't help us to fix those bugs, that is to say we manage all issues and fix all internally.

I also checked the MAINTAINERS list which is large and most of compnents are not our interesting.
I welcome if there is any future update on the dashboard, perhaps a script to auto generate the driver_map file based on the list.
Thanks

I welcome if there is any future update on the dashboard, perhaps a script to auto generate the driver_map file based on the list.

That's a reasonable argument. If we "plug" automatic subsystem detection later, it may work. Let me think about this.

Let's also name this "subsystem" consistently. I see you use term "driver" in one place and "component" in another place.
This is generally not about "drivers", e.g. "net/tcp" is not a driver.
This is generally referred to as "subsystem" by kernel community:
https://lore.kernel.org/lkml/?q=subsystem
https://lore.kernel.org/lkml/?q=component

Now /filecover and /subsystemcover are used with kernel_subsystem set from like adb.cfg, and with all comments resolved above.

Looks good to me. Thanks for bearing with me.


AP-1818Remove CCMS connectivity test

Remove CCMS connectivity test
Link to story
Remove the code to carry out the CCMS connectivity test.  (The button to execute this code was removed in AP-1269)
Checklist
Before you ask people to review this PR:

[ ] Tests and rubocop should be passing: bundle exec rake
[ ] Github should not be reporting conflicts; you should have recently run git rebase master.
[ ] There should be no unnecessary whitespace changes. These make diffs harder to read and conflicts more likely.
[ ] The PR description should say what you changed and why, with a link to the JIRA story.
[ ] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[ ] You should have checked that the commit messages say why the change was made.

@stephenrichards does views/admin/ccms_connectivity_tests/show.html.erb also need removing?


Clip cover_multistep_options's option policies with limits of action space.

This allows for flexibility in defining variable step policies in cover_multistep_options because the Pick() and Place() ground truth option policies now clip the low-level-action by the bounds of the action space. If we want policies that take more steps to go from point to point, we can reduce the limits of the action space.

LGTM after tom's suggestion


feat(balance): display frozen balance on jars

Resolves #634.
Displays the frozen or locked balance in the Jars component.
Before

After

This is really just a suggestion. Any suggestion on how to better display frozen funds is highly appreciated.
Maybe one can display a few ice cubes in Jar with frozen UTXOs.. :ice_cube: :ice_cube: :ice_cube: ?
Heads up: The build errors are fixed in #621

for the time beeing its already usefull and really quickly implemented, plus its already motivating the user to click on it to "look inside"
never the less i will think about the other idea and play around with it


EKS bootstrap blindly assumes IPv4 for cluster if B64_CLUSTER_CA and APISERVER_ENDPOINT are set

What happened:
Deployed an IPv6 enabled EKS cluster using https://github.com/terraform-aws-modules/terraform-aws-eks
What you expected to happen:
IPv6 DNS to correctly be set, instead it is using the 172.x.x.x address
How to reproduce it (as minimally and precisely as possible):
Deploy cluster using https://github.com/terraform-aws-modules/terraform-aws-eks and set it to IPv6. It will create a user data that includes the B64_CLUSTER_CA and APISERVER_ENDPOINT at which point:
https://github.com/awslabs/amazon-eks-ami/blob/60550f3eeed450d5f57b591f6c6ea76d1c494439/files/bootstrap.sh#L377-L381
Will assume that the only option is IPv4.
User data:
#!/bin/bash
set -e
echo "Fetching AWS CLIv2"
(
  cd /tmp
  curl "https://awscli.amazonaws.com/awscli-exe-linux-x86_64.zip" -o "awscliv2.zip"
  unzip awscliv2.zip
  ./aws/install
  rm -rf /bin/aws
  ln -s /usr/local/bin/aws /bin/aws
)
echo "Starting bootstrap stage"
B64_CLUSTER_CA=[...]
API_SERVER_URL=https://hexdata.us-gov-west-1.eks.amazonaws.com
/etc/eks/bootstrap.sh test-aws-gov-k8s --kubelet-extra-args '--node-labels=group=management' --b64-cluster-ca $B64_CLUSTER_CA --apiserver-endpoint $API_SERVER_URL
cd /tmp
sudo yum install -y https://s3.amazonaws.com/ec2-downloads-windows/SSMAgent/latest/linux_amd64/amazon-ssm-agent.rpm
sudo systemctl enable amazon-ssm-agent
sudo systemctl start amazon-ssm-agent

Anything else we need to know?:
Environment: govCloud

AWS Region: us-gov-west-1
Instance Type(s): t3a.large
EKS Platform version (use aws eks describe-cluster --name <name> --query cluster.platformVersion): eks.5
Kubernetes version (use aws eks describe-cluster --name <name> --query cluster.version): 1.21
AMI Version: ami-03e76509bb9349ef0
Kernel (e.g. uname -a):  Linux ip-10-58-130-206.gov.aws.test.example.internal 5.4.181-99.354.amzn2.x86_64 #1 SMP Wed Mar 2 18:50:46 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux
Release information (run cat /etc/eks/release on a node):

BASE_AMI_ID="ami-04149a3294d95f9d1"
BUILD_TIME="Wed Mar  9 04:02:56 UTC 2022"
BUILD_KERNEL="5.4.181-99.354.amzn2.x86_64"
ARCH="x86_64"

Thanks for the the feedback. The issue here is that when adding arguments we want to make sure the behavior supported with just the pre-existing arguments doesn't change. This guarantees backward compatibility with the AMI once you have correctly set the arguments for any future AMI Release.

I believe this is resolved with https://github.com/awslabs/amazon-eks-ami/pull/860 - cc @bwagner5 / @suket22

This issue won't be resolved with this. It will still set the IP_FAMILY to ipv4.
The script when provided the B64_CLUSTER_CA and APISERVER_ENDPOINT should ask the kube API what the service address range is, rather than blindly accepting to move forward with IPv4.

I am not sure that I agree that it will be fixed with that issue. After that issue the bootstrap script still assumes IPv4 if the B64_CLUSTER_CA and APISERVER_ENDPOINT endpoint are set, because this line:
https://github.com/literalice/amazon-eks-ami/blob/a2d4c4c924e9576e57cf8aaaa6e57a25015b8e5f/files/bootstrap.sh#L331
Falls through, and then:
https://github.com/literalice/amazon-eks-ami/blob/a2d4c4c924e9576e57cf8aaaa6e57a25015b8e5f/files/bootstrap.sh#L370
This sets the family to IPv4.

Thanks for the the feedback. The issue here is that when adding arguments we want to make sure the behavior supported with just the pre-existing arguments doesn't change. This guarantees backward compatibility with the AMI once you have correctly set the arguments for any future AMI Release.

That's perfectly acceptable, and I agree with that. Given the cluster CA/apiserver endpoint we could interrogate k8s to find out what the service address is and whether that is IPv4 or IPv6... vs right now blindly assuming that if those are passed that it must be an IPv4 cluster.

This issue won't be resolved with this. It will still set the IP_FAMILY to ipv4.

correct, users have to opt into "IP_FAMILY=ipv6" to enable IPv6 support which was just fixed in #860

The disappointing thing is that it still doesn't bring parity with IPv4, you either need to provide the DNS IP address manually, OR you need to provide the service address range.
Getting that service address range though means calling the aws command to fetch the address range from the EKS cluster by name, rather than the script doing it for us.
With IPv4 there is no requirement to pass in the service address range at all, it is assumed to be in 10/8 or 172.16/12 (if the former is in use in the VPC).
If I could pass --ip-family=ipv6 and it just did the right thing that would be great, and would mean I don't have to smuggle data into the cloud-init script to get it to join the cluster correctly.


Record format with Kafka

Infrastructure details
Java Version:1.8
Scala Version:2.11
Spark version:2.2
OS version:win7+centos6.7
Cluster mode
I am using kafka with StreamDM
suppose there is a data set for traning with format(e.g. csv) as following:
1,14.23,1.71,2.43,15.6,127,2.8,3.06,.28,2.29,5.64,1.04,3.92,1065
1,13.2,1.78,2.14,11.2,100,2.65,2.76,.26,1.28,4.38,1.05,3.4,1050
1,13.16,2.36,2.67,18.6,101,2.8,3.24,.3,2.81,5.68,1.03,3.17,1185
the first element is label and the others could be assembled to feature
so,what is the format of these records if i use producer to send them to topic?
e.g.
if i have a sample with
label = '1'
feature='14.23,1.71,2.43,15.6,127,2.8,3.06,.28,2.29,5.64,1.04,3.92,1065'
how could i construct ProducerRecord with label and feature?

Dear @zhangjinrui2718,
I am sorry for taking too long to provide you with an answer. I was going to update the KafkaReader class, but due to higher priority aspects of the project I end up not getting to work on that yet. So far, the best I can assist you with is to direct you to the current KafkaReader version, specifically you will need to setup your instance such that it complies with the code in getExamples().
Please let me know if that works for you
Best Regards,
Heitor


New udp interface does not build on AVR32

Make  this a compile time option

This seemed like a waf specific issue only, but i included both sys/socket.h and arpa/inet.h in the requirements for if udp build.


Fix: lookup for cached translations with unknown key

@mpakus could you review this, please

@mpakus also as I'm not an owner of this repo, it looks like only you can merge changes

done, permission added


Include DateTime ISO 8601 change as a breaking change

:heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Updated .gitignore to allow CMake on VSCode to generate build files

Bro, update pls?

Sure! Be on that right away.

I guess the commit history became too clumsy... Can you create clean your commits / create a new branch and open a new PR?


Add button to override no internet lockout

The reboot workaround for the no connectivity lockout should be replaced by a button or some other kind of interaction that's just annoying enough to incentivize users to still keep up to date.

Simple math problem? Add 2x random numbers together and enter the result into Comma device.

High-level logic to achieve annoyance while allowing update bypass:

After 14 days, shows "Update Later" button to allow the user to switch to onroad state after mouse click and set snooze_counter to 1. Device will not allow to switch to onroad state unless up to date or mouse click on "Update Later" button.
Once the device switches to offroad state, set the snooze_counter to 0, "Update Later" button reappears, and does not allow the user to switch to onroad state unless the device is up to date or mouse click on the "Update Later" button.


Adds inspec runner for pronto to runner overview, re. #254

This adds my pronto runner for inspec tests to the runner overview in README.md, "solves" #254

Thank you @stiller-leser 😁


Height of a Binary Tree

Allow me to work on this issue

@sujal111 mention the language in which you want to code.

Hello,
Please assign me this task. I am a GSSoC'21 participant. I can add the solution in c++.

@sujal111 mention the language in which you want to add the code.

Hi,
Please assign this task to me. I am a GSSoC'21 participant. I can solve the problem using Java too.

Can I solve this using C++?

I am gonna do it in Python

please Assign me this , I would love to do it in C++.

I would love to contribute to solving the problem, I am a GSSOC'21 participant. Please assign it to me. I will try my best and provide proper comments and implement different test cases for better understanding. I will code in Python as specified.

Hey, I am a GSSOC'21 participant. I would like to contribute the code of this problem statement in PYTHON language. Please assign this issue to me.

Hey, I am Divya, a GSSOC'21 participant and I want to work on this issue in the Java language. I will provide a neat and well explanatory code for this using some example test cases as well. Please assign me this issue.
Thank you

#348    #698
I would like to contribute in your Project by working on Tree (Find the Depth ) (Binary Tree ) Problems using Recursive Approach in JAVA .

My Approach is Optimize yet Easy to understand .
Single Line Comments makes it easy to read.
Required Test Case are added at the end.
Can make it More Explanatory using the Image

Please assign me this issue under the label GSSOC21 , Looking Forward To your Response @tarun26091999

I am a participant of GSSOC'21
I would like to contribute in this

If this issue is open to work still please assign me this issue.
I will code in C++.
And try my best to provide explanation and proper Test cases.


Unexcepted write amplification

Hi,
We are merging polkadot 0.9.37 into Khala network and testing if we can ship paritydb as the default database.
We have found some unexcepted write amplification that would kill the SSD.
During the test, I bought 2 brand new HP FX900 Pro 4T SSD and used them to sync Kusama and Khala from the beginning. I set a RAID1 ZFS pool and the node VM used xfs as the FS. I noticed that the substrate node had written over 50T within 24 hours.

Also a heavy, intolerable fragmentation happened. It only takes 24 hours to make ZFS fragmentation to 59% on a brand new SSD.

Should I provide any information to debug with it?

Was the sync peformed in archive or default mode?

Was the sync peformed in archive or default mode?

For Phala/Khala mining, the substrate node should be in archive mode explicitly.

And for kusama? As far as I understand the figures were taken after syncing Kusama and Khala

Yes both Khala and Kusama.

Even the sync is blocked by block import validation, the IO is still insane.

paritydb is a hashtable, and not an LSM. Each insertion requires at least two random sector write with addional IO for the write-ahead log. So write amplification could be as high as 10x when inserted values are small. 50T is still too high though. I'm not that familiar with zfs, but as far as I undestand default record size of 128k means that each small write will touch at least 128k.
Could you setting zfs records size and block size to the physical sector size and doing anoter sync from scratch?

Closing due to inactivity


Calculate total count of _User data using parse_ruby_client

Hi All,
I am using parse_ruby_client gem for Parse API calls.
In below rails code i want to calculate total count of users data, but it doesn't works. Please help me.
def total_count
    Parse.init :application_id => Rails.application.config.parse_application_id, :api_key => Rails.application.config.parse_api_key

    # build a query on the User class
    object_query_count = Parse::Query.new("_User")
    # no limit on the results
    object_query_count.limit = 0
    # check to make sure the email is not blank
    object_query_count.exists("email")
    # get the count
    object_query_count.count
    # store the count
    count = object_query_count.get["count"]
    # print the count
    Rails.logger.info "Total with Email: #{count}"
    # calculate how many iterations are needed to go through all the records

    redirect_to root_path
  end

@aditya-mandal
I believe setting limit == 0 means you return 0 results.
Also, ruby hash (what is returned by a query object where count == true) has no #get function (unlike python's dict). Instead, just use object_query_count['count'], which returns a number.
Note that if you've got 1000 > users > 100, you should set your limit to the max (1000). If you've got > 1000 users, you'll have to page through results.
Does that make sense?


Javascript Capstone Kanban Set Up

Dear reviewer,
We are a team of two partners and we have just completed setting up our kanban project. Kindly review it for us so we can kick start. Thanks in advance.
Kanban Link: Kanban Link
Project Link: GitHub code Link

Hi @mudasiri 👋,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that doesn't contain the right link to your Kanban board project ( the link to the board itself) below 👇 is an example

Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hello team 👋, the link to the kanban board is broken:

STATUS: INVALID REQUEST
Hi team👋,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project that doesn't contain a link to the kanban board
Your Code Review Request will be marked invalid in your Dashboard, so please submit a new one once ready 🙏.
Invalid requests don't count as a review, so don't worry about that 😄!
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me @Gopxfs in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Hi @mudasiri,
Good job so far 👍🏼
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights
✔ Used Kanban board
✔ Board has all the tasks
Required Changes ♻

[ ] all cards should be assigned to team members, kindly consider assigning every card to a team member, and keep in mind to distribute the tasks in a fair way

Check the comments under the review.

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Thanks for the review and approval.


Order of Collection Metadata to Match Work Show Page

Reorders the metadata fields to match the order of the attribute rows from the work show partial. You can use the attribute rows as a list for guidance.
https://github.com/psu-stewardship/scholarsphere/blob/develop/app/views/curation_concerns/base/_attribute_rows.html.erb

Coverage remained the same at 54.73% when pulling e4fa9f4f4951df79a65c1e8716ab508c24fc1baf on i#1016 into 3c14ee8408b9d0c92b88a64cc8bd745a16916429 on develop.


Check for case where matched is nil.

the value for matched can be nil in the case where the user has updated their version of Xcode but has not yet opened the GUI to agree to the license agreement.

Associating with issue #229

Looks like CI isn't quite setup all the way.  Anything I can do to help?

Looks good, thanks :+1:


Mssql

How to connect to mssql remotely?

enter the serverip on
$dbhostname = ""; //MsSQl Server Addres
you may use XX.XX.XX.XX/MSSQL or something like that search on google.... you can find better anwser.

Her put me that the request is too long

well, i do some research, you need to enable remote connection on SQL server, just search on google and you will find something.
there are what you need -> https://goo.gl/RuL5td


[stable/prometheus-pushgateway] Follow the Call for maintainers

As this Chart is bad/not maintained and also buggy I would like to take
ownership for this Chart as my team relies on this Chart.
Signed-off-by: cstaud<EMAIL_ADDRESS>
Is this a new chart
No
What this PR does / why we need it:
The actual maintainer seems not to be active anymore and there are a lot of things which should be improved about this Chart.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #13867

Special notes for your reviewer:
There was already some discussion in #18187
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/assign @gianrubio
/assign @zanhsieh

/assign @maorfr

@cstaud See "Owning and Maintaining A Chart" at https://github.com/helm/charts - please add the OWNERS file to .helmignore

/assign

/verify-owners

/verify-owners

/ok-to-test

/lgtm


Remove --between-tags option

@olleolleolle I bet you are interested in collaborating with me on this one?

Hello!
The metadata of a change is quite important to this project.
A description which explains how something is wrong and what has been done to right it and why this all matters to a user. That's part of communicating your intent to others who wonder.
Can you find and link to any discussion which weighs the issue somewhere?
I'm sure there's context to this somewhere, but I can't find it from this PR.

There you go: #450 @olleolleolle

Thank you for this PR!

@olleolleolle I guess there are some post-development to do for this removal. Let me see what I could find out!

@olleolleolle @Lucashuang0802 Is this change a bugfix or a breaking change? If the latter, then the next version of GCG will be v2.0.0 to account for this, yeah?


[AUTOPATCHER] Patches CVE-2021-31829 for kernel

        Patches CVE-2021-31829 for kernel.
        This patch and PR was automatically generated and needs human verification before merging.

Disregarding Kernel patch.  Kernel patching is handled through other means.


Support the TeleTAN and GUID verification server flow

Current Implementation

The current implementation provides a "fake" verification server implementation. This supports the /version/v1/tan/verify' REST endpoint ("fake" because it accepts only the single TAN of "edc07f08-a1aa-11ea-bb37-0242ac130002").
Suggested Enhancement

The real verification server also has the following REST endpoints:

/version/v1/registrationToken
/version/v1/tan
/version/v1/testresult

In order to test clients against the test server, it would be useful if the server could also support these endpoints, potentially supporting just a finite set of "fake" inputs, as is already the case for /version/v1/tan/verify'.
Expected Benefits

This would be a useful change for the test server, and for building clients against it.
It would have no effect on the real server implementation.

Hello llewelld,
thank you very much for your input and sorry you didn't get a response so far. We have been rather busy with implementing stuff, and it's only now we have a little more time on our hands.
Regarding your PR: We have another setup internally that we use to test the clients against. This is also the reason we didn't need that change. However, since you took the effort in creating this ticket and the PR, we at least didn't want to close it without giving you feedback—for which we didn't find the time.
Now I want to thank you in the name of the team of the CWA for your interest and your effort, even if won't come around to use it. I hope this doesn't discourage you from further participating here or in other open source projects.
Stay healthy!
Jeremias


Add feature to export configuration which closes #663

This PR closes #663

Coverage remained the same at 90.972% when pulling e93132da753e5d466b0486f6ca65aba1f51c2446 on ocadaruma:feature-server-export-config into e41a39bfd7f4337c6b8aa7d808ba23466392a273 on redpen-cc:master.

👍

Thank you very much for the valuable contribution @ocadaruma! I will look into the changes. This pull request is  merged in few days.

LGTM! Thank you again for the great contribution @ocadaruma!


Add unit test for open search client bulk requests

What changed?
Refactor bulk add method, separate callback func for better testability
Add unit test for open search client bulk requests

Why?
Unit test converage

How did you test it?

Potential risks

Release notes

Documentation Changes

Pull Request Test Coverage Report for Build 018f547e-b777-41d6-a318-9218648d0b88
Details

24 of 30 (80.0%)  changed or added relevant lines in 1 file are covered.
26 unchanged lines in 7 files lost coverage.
Overall coverage increased (+0.08%) to 68.208%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

common/elasticsearch/client/os2/client_bulk.go
24
30
80.0%

Files with Coverage Reduction
New Missed Lines
%

common/task/weighted_round_robin_task_scheduler.go
1
89.05%

service/history/task/transfer_standby_task_executor.go
2
86.21%

common/task/fifo_task_scheduler.go
2
87.63%

service/matching/taskListManager.go
2
81.16%

service/history/execution/mutable_state_util.go
2
78.52%

service/history/queue/timer_gate.go
3
95.83%

service/history/execution/mutable_state_task_refresher.go
14
64.56%

Totals

Change from base Build 018f5467-b337-442d-bcf0-8ad761c213c7:
0.08%

Covered Lines:
100188

Relevant Lines:
146887

💛  - Coveralls


put commentary into qconsole workspaces

QC example workspaces need more commentary to explain what's going on for each tab.

Added some good commentary, I think.


MultiChoice checkboxes not applying theme

Hi,
I'm using a dialog as the following snippet of code. My theme is Theme.AppCompat.NoActionBar so I'm put Theme.Light to customize the dialog. The content is ok, but the checkboxes unpressed styled remains with dark theme (white color).
Could you take a look?
Thank you.
    eventMultiDialog = new MaterialDialog.Builder(getActivity())
                .title(R.string.choose_tasks_c)
                .titleColorRes(R.color.app_blue)
                .theme(Theme.LIGHT)
                .widgetColorRes(R.color.app_green)
                .items(R.array.task_mock_items)//TODO replace with data from service
                .itemsCallbackMultiChoice(new Integer[]{}, new MaterialDialog.ListCallbackMultiChoice() {
                    @Override
                    public boolean onSelection(MaterialDialog materialDialog, Integer[] integers, CharSequence[] charSequences) {

                        return true;
                    }
                })
                .positiveText(R.string.done_u)
                .positiveColorRes(R.color.app_green)
                .build();

Unfortunately not too much can be done about that.
The dialog is inflated with whatever theme you specify with the theme(Theme) Builder method (or LIGHT by default). If the dialog view was inflated using the Activity theme, then there would be a lot of visual issues (e.g. wrong colors for scroll indicators and other elements).

You can use the ContextThemeWrapper to inflate your layout within a specific theme

@alorma that's what used to be done, but like I said above, it causes many visual issues.

Actually, I have an idea

@alorma not sure why I didn't think of using a different context to inflate the individual list item layouts before.
This issue is fixed in https://github.com/afollestad/material-dialogs/commit/c5e79ec6d2c1bd5c5fc63007f11e85a6167c613c.

@afollestad nice!

Thank you for the work! But it still working in the bad way. Is there any workaround to solve this? What context do you suggest to use? Thank you!

@josalvdel1 you haven't gotten the fix yet, it's not published

https://github.com/afollestad/material-dialogs/releases/tag/v<IP_ADDRESS>


[DOC] Fix doc for pyhive client.

Why are the changes needed?
Based on my actual tests and pyhive source code, the variable "user" should be "username".
pyhive source code

Thanks.

Codecov Report

Merging #4960 (db39786) into master (768b8ff) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4960   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         563     563
  Lines       30964   30964
  Branches     4035    4035
======================================
  Misses      30964   30964

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks, merged to master/1.7

@BruceWong96 Please link your email to your GitHub account, Bruce<EMAIL_ADDRESS>otherwise, your contribution will not be counted by GitHub

Thanks.


(PUP-10231) Use EvaluatingParser singleton in PopsBridge

Avoid sharing an EvaluatingParser across threads in PopsBridge by using the thread-local singleton instance instead.

jenkins please test this with servertests


api: add docs for targets and alertmanagers api

Short docs on the recently added targets and alertmanagers API.
@fabxc @brian-brazil
Closes #649

LGTM in general 👍


How could I remove mraa and install it again?

I've changed some files of the library manually, so I've problem with mraa now.
How could I remove mraa and install & build it again?

Just run make install again? It'll wipe all the old libs and headers.
On 21 Dec 2017 10:22, "Mohamad777"<EMAIL_ADDRESS>wrote:

I've changed some files of the library manually, so I've problem with mraa
now.
How could I remove mraa and install & build it again?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/intel-iot-devkit/mraa/issues/848, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAfZZkJafF9PdveS3EG8sO1e5CXuoZa0ks5tCiNGgaJpZM4RJjaH
.

It didn't work.
I've removed /usr/lib/arm-linux-gnueabihf/libmraa.so file and now it is not available even after make install again.
And I get this error in run time:
error while loading shared libraries: libmraa.so.1: cannot open shared object file: No such file or directory
I'm using Ubuntu mate on RPI3.

Are you sure make install succeeds? If you could post a console log for the whole process, that could be helpful for understanding what's going on. The process is way to simple to break :), so I wonder what could've gone wrong there.

Hi,
Can I usr make uninstall to remove mraa?


New-CEApiToken Fails because $Session is empty

New-CEApiToken fails when logged in with username with error "The Argument is Null or Empty". this is caused because $Session is empty. we can add the logic to fetch '$Session' from '$script:Sessions' just like in 'Get-CESession' function.

Can you provide the error output and your steps to recreate the error? I'd like to try and provide a fix for this issue.


Gesture detector not working if inside a row

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
create a sample project
-create a column and move the code to the bottom of the screen
and then try to tap on the second button
Expected results
I expect the second button show be clickable as well. Second button is not clickable
Actual results
second button is not registering the click event
Code sample
Code sample
              Row(
                mainAxisAlignment: MainAxisAlignment.center,
                crossAxisAlignment: CrossAxisAlignment.start,
                children: [
                  Text(
                    'By continuing, you agree to ',
                    style: RiverTextStyle.st01.copyWith(
                      height: 1,
                      fontSize: 3.ws(context),
                      color: RiverColorPalette.riverBlue05,
                    ),
                  ),
                  RiverTextWithUnderlineWidget(
                    label: 'T&C',
                    labelStyle: RiverTextStyle.r2.copyWith(
                      height: 1,
                      fontSize: 3.ws(context),
                      color: RiverColorPalette.riverBlue05,
                    ),
                    onTap: () {
                      OnboardingTermAndConditionRoute().go(context);
                    },
                  ),
                  Text(
                    ' & ',
                    style: RiverTextStyle.st01.copyWith(
                      height: 1,
                      fontSize: 3.ws(context),
                      color: RiverColorPalette.riverBlue05,
                    ),
                  ),
                  RiverTextWithUnderlineWidget(
                    label: 'Privacy Policy',
                    labelStyle: RiverTextStyle.r2.copyWith(
                      height: 1,
                      fontSize: 3.ws(context),
                      color: RiverColorPalette.riverBlue05,
                    ),
                    onTap: () {
                      PrivacyPolicyRoute().go(context);
                    },
                  ),
                ],
              ),

              class RiverTextWithUnderlineWidget extends StatelessWidget {
  const RiverTextWithUnderlineWidget({
    required this.label,
    required this.labelStyle,
    this.lineThickness = 1, // 1.5
    this.onTap,
    super.key,
  });

  final String label;
  final TextStyle labelStyle;
  final double lineThickness;
  final VoidCallback? onTap;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: onTap,
      child: IntrinsicWidth(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: [
            Text(
              label,
              textAlign: TextAlign.center,
              style: labelStyle,
            ),

            /// If the labelStyle font weight is w700
            /// make the underline thin
            Container(
              height: lineThickness.pixelScale(context) *
                  (labelStyle.fontWeight == FontWeight.w700 ? 1.5 : 1),
              color: labelStyle.color,
            ),
            SizedBox(
              height: lineThickness.pixelScale(context) *
                  (labelStyle.fontWeight == FontWeight.w700 ? 0.5 : 1),
            )
          ],
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.2, on macOS 13.4.1 22F770820d darwin-arm64,
    locale en-IN)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 14.3.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.81.1)
[✓] Connected device (2 available)
[✓] Network resources

• No issues found!```

</details>

Hi @dishankjindal1. Thanks for filing this. I'm unable to reproduce this issue on the latest flutter versions with the code sample below. Kindly confirm this on your end by checking with this or modifying it to describe your issue.
Code Sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(home: MyHomePage());
  }
}

class MyHomePage extends StatelessWidget {
  MyHomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: Row(
          mainAxisAlignment: MainAxisAlignment.center,
          crossAxisAlignment: CrossAxisAlignment.start,
          children: [
            Text(
              'By continuing, you agree to ',
            ),
            RiverTextWithUnderlineWidget(
              label: 'T&C',
              labelStyle: TextStyle(),
              onTap: () {
                print("continuing");
              },
            ),
            Text(
              ' & ',
            ),
            RiverTextWithUnderlineWidget(
              label: 'Privacy Policy',
              labelStyle: TextStyle(),
              onTap: () {
                print("Privacy");
              },
            ),
          ],
        ),
      ),
    );
  }
}

class RiverTextWithUnderlineWidget extends StatelessWidget {
  const RiverTextWithUnderlineWidget({
    required this.label,
    required this.labelStyle,
    this.lineThickness = 1, // 1.5
    this.onTap,
    super.key,
  });

  final String label;
  final TextStyle labelStyle;
  final double lineThickness;
  final VoidCallback? onTap;

  @override
  Widget build(BuildContext context) {
    return GestureDetector(
      onTap: onTap,
      child: IntrinsicWidth(
        child: Column(
          mainAxisSize: MainAxisSize.min,
          children: [
            Text(
              label,
              textAlign: TextAlign.center,
              style: labelStyle,
            ),

            /// If the labelStyle font weight is w700
            /// make the underline thin
            Container(
              height: 10,
              color: Colors.green,
            ),
            SizedBox(height: 10)
          ],
        ),
      ),
    );
  }
}

Move it to the bottom of the screen, and test it on iphone 14 pro. For keeping the scenario similar

Hi @dishankjindal1. Thanks for your response. I moved to the bottom of the screen as you proposed, tested on an iPhone 14 Pro and still can see the GestureDetector work very fine.
https://github.com/flutter/flutter/assets/53122008/be868fd4-de1e-4c9e-afc3-582892e1af41
For this issue to be workable, we'd need you to kindly provide a minimal complete reproducible code sample that doesn’t include 3rd party plugins or complex production code and if possible a screen record of the behaviour observed on your end.

There is a auth guard logic which was wrongful present within the navigation logic of the privacy screen prevented it from getting pushed to the top of navigation stack.
After adding some debugger and log statements, was able to come on this conclusion and fixed it.
No problem in the core flutter.


How to deploy lambda function without creating API Gateway endpoint?

I don't always need the API Gateway endpoint. One of my lambda function is to watching for changes happening in S3 bucket. How can I disable Gateway endpoint creation when I do: zappa deploy dev?

Related to this: https://github.com/Miserlou/Zappa/issues/61

Closing as duplicate in fact


ERROR: cannot send statistics to Home Assistant: error unknown_error: Invalid statistic_id

I get the following error when running this.
Do you know how you can fix this? The output from ESB seems correct and the sensor was created as documented.

Actually I get this error with the provided sensor config:
Invalid config for [template]: required key not provided @ data['trigger'][0]['platform']

Did you configure home assistant sensor as described in https://github.com/Lorentz83/esb2ha/blob/main/documentation/index.md#configuring-home-assistant ?
If you do the quick and dirty test using a dockerized home assistant  (https://github.com/Lorentz83/esb2ha/blob/main/documentation/index.md#i-just-wan-to-give-it-a-quick-try) does it work?


[5.5] Revert "Add multibyte functions where needed in Support/Str"

This reverts #21207.
Parsing with the multibyte functions is actually useless.

ping @jakebathman @sisve

Thanks, saw your comment earlier and was gonna make this later tonight.

I can confirm it could be safely reverted. 👍
But I think the multibyte string tests should be maintained so we can assure they'll never break in the future for whatever reason. 😄

Updated PR to keep the tests.
Theoretically they should be useless, but who knows, better having too many tests than the contrary.


ErrorException  Use of undefined constant SIGINT - assumed 'SIGINT'

php artisan telebot:polling

   ErrorException

  Use of undefined constant SIGINT - assumed 'SIGINT' (this will throw an Error in a future version of PHP)

  at *******\vendor\westacks\telebot\src\Laravel\Artisan\LongPollCommad.php:72
     68▕     }

php -v

PHP 7.4.4 (cli) (built: Mar 17 2020 13:49:19) ( ZTS Visual C++ 2017 x64 )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies

"require": {
        "php": "^7.3|^8.0",
        "fruitcake/laravel-cors": "^2.0",
        "guzzlehttp/guzzle": "^7.0.1",
        "laravel/framework": "^8.75",
        "laravel/sanctum": "^2.11",
        "laravel/tinker": "^2.5",
        "westacks/telebot": "^1.14"
    },

p.s.  "westacks/telebot": "1.13.0"  - OK
version > 1.13.0 - error

Hello! Do you have OS/PCNTL extension installed on your PHP 7.4?

Windows,  OpenServer

Will be fixed ASAP, thanks for reporting

Should be working in 1.14.5


Error: XPath parse error

   Then response body path $.codigoLoja should be 1717
erro:

my postman response body:
{
"codigoLoja": 1717,
"codigoTest": 249966,
}

I have the same issue. XPath parsing is not working anymore. Is this project still maintained in any way ?

Could you please try with v3.0.3? I've upgraded a number of packages to see if those created any regressions. All tests seem to be passing - see https://github.com/apickli/apickli/actions/runs/10819988204/job/30019118511

It seems to work fine with v3.0.3 on my side. Thank you! :)

Should you create a new release for v3.0.3 ? I don't see it in https://github.com/apickli/apickli/releases


Proxy server pushes

Currently, it seems that caddy does not proxy server pushes.
So if my app relies on server push, I cannot put it behind a caddy-proxy, because the browser won't get the pushes that way.
Will you implement this in the future?

You can use Preload links and Caddy will push resources to client event if using proxy.

That won't work for all applications.
I'm currently implementing a web push server, that implements the web push protocol.
The protocol makes heavy use of server push.
For example, to listen for push notifications, the listener makes a never-ending request to the server, and then the server pushes notifications via server push over that request.

We could look into this, but I don't know how to do it right now. I think we're going to implement auto-push first, but then maybe @wendigo and/or I could look into this?

@mholt, @VanCoding -
I had a similar question on twitter, and would love to see Caddy support this so more of my project's users could put their Ponzu apps behind Caddy and still leverage all of h2.
My problem sounds similar to @VanCoding's in which typical server push usage does not apply. Rather than using server push for resources like CSS/JS/etc to make a "page load" faster, Ponzu provides a Push mechanism to basically use the network for n+1 queries. As mentioned in the tweet, an example of this can be illustrated below:
Say you have 2 data types, Song and Artist, where each song has an artist field, referencing the Artist type:
type Song struct {
	Title   string `json:"title"`
	Year    int    `json:"year"`
	Artwork string `json:"artwork"`
	Artist  string `json:"artist"`
}

type Artist struct {
	Title   string   `json:"title"`
	Logo    string   `json:"logo"`
	Members []string `json:"members"`
}

In a Ponzu CMS app, the Pushable interface will allow the programmer to define which fields from the receiver struct to server push, as long as they are URLs.. (the Artist field in the Song struct would be a URL to the JSON representation of the Artist, i.e. ?type=Artist&id=42)
So, by adding the Push method to Song, like below, a request for a Song would also send a server push for the embedded Artist endpoint too, so the artist information for that song is available to the client, without making an additional roundtrip:
func (s *Song) Push() []string {
	return []string{
		"artist", // this being the json tag for the Artist field in Song struct
	}
}

e.g. /api/content?type=Song&id=21
{
"data": [{
    "title": "Where's the Revolution",
    "artist": "?type=Artist&id=42",
    "year": 2017,
    "artwork": "/api/uploads/2017/02/single-cover.png",
 }]
}

would send a push promise for the data[0].artist node as per the Push method to send down the JSON at ?type=Artist&id=42
{
"data": [{
    "title": "Depeche Mode",
    "logo": "/api/uploads/2017/01/depeche-logo.png",
    "members": ["Dave Gahan", "Martin Gore", "Andrew Fletcher"]
 }]
}

Currently, Caddy will not proxy the my explicit h2 server push to the client which I believe is what @VanCoding is also trying to achieve, versus the implicit server pushes done through an "auto-push" mechanism, or through the Preload headers which would require direct access to the ResponseWriter.
This certainly could be done, and may be achieved with some of Caddy's outstanding PRs for 1.8 HTTP/2 stuff.
@VanCoding - does this sound at all like the higher level of flexibility from server push you're looking for?

@nilslice yes, exactly.
As I understood it, with Go 1.7, this would not have been possible, but Go 1.8 does provide the features needed to implement this. So this seems to be the right time to ask for this :P

As far as I know reverse proxy (https://golang.org/pkg/net/http/httputil/) does not support http push handling as of go 1.8. I don't know if it can be achieved straight away.

Seems this issue would be your best bet: https://github.com/golang/go/issues/18594

Exactly @mholt. I think that best option for now is to leverage Link headers so instead of backend pushing resources - if it's running behind the proxy it should emit Link headers that will be handled by Caddy.

@wendigo - that does seem to be the only option until go's h2 client and reverse proxy support reading the push_promise frames.

Closing for now, but labeled as deferred so we won't forget about it.

So @mholt currently caddy won't work with push in reverse proxy mode using either config or Link headers right?

If a backend responds with Link headers, Caddy will push those resources, but currently it won't send push frames from the backend.

If a backend responds with Link headers, Caddy will push those resources

I modified Discourse to add the Link headers but I don't get any pushes.
My test bed is the main resource at https://forum.overwatchbrasil.org/

I suppose the first question is if these are in the correct syntax/format?

I do believe they are :thinking:

Also, the server header, though easily overwritten/spoofed, is "nginx", not "Caddy" as commonly left in Caddy configurations. Are you definitely serving the site you linked from Caddy?

Yes I am, nginx is listening a sock and Caddy is the owner of 80 and 443 port.

@wendigo You said:

Exactly @mholt. I think that best option for now is to leverage Link headers so instead of backend pushing resources - if it's running behind the proxy it should emit Link headers that will be handled by Caddy.

Does that work on latest master, or is a planned feature? I added all the Link headers on my response, but Caddy doesn't push.

@xfalcox I missed your reply; but that feature is released now; you can use Link headers upstream to initiate pushes downstream.


Add unit testing for package acid_dto

Add unit testing for internal/stackql/acid/acid_dto.
Acceptance Criteria:

85% unit test coverage.

Hey @jeffreyaven can I work on this issue ?

certainly @doshi-kevin

Hey @jeffreyaven , I am new to the go language. After forking the repo and making changes the code, could you please provide me the code to run the tests. It would be really helpful.

@general-kroll-4-life would you be able to help @doshi-kevin

Hiya @doshi-kevin,
Thank you for joining in!  Your question is valid and exposes a shortfall in our documentation.  The short of things is:

Install golang on your system if you do not already have version >= 1.21, per the golang doco.
Install python on your system if you do not already have version >= 3.11, available from the python website and numerous package managers.
Install robot framework, per the robot doco.
Each of these should be run from the repository root:

Get required go libs: go get ./....
Run unit tests: python cicd/python/build.py --test.
Build the application: python cicd/python/build.py --build (this will generate an executable at ./build/stackql).
Run mocked functional tests: python cicd/python/build.py --robot-test.  This will subject the executable to the automated testing regimen.

Running linting locally is also a nice-to-have, and can be done:

Install golangci-lint v1.59.1 (the current version we use in CI) per the golangci-lint doco.
From the root directory of the repository: golangci-lint run.

Please do reach out if you get stuck.  The CI is the source of truth on build and test.  All the permutations for different local systems are not well documented just yet.

Hi @doshi-kevin ,
Just following up.  I missed some detail in my above directions, so added this section in our docs to cover off some gaps: https://github.com/stackql/stackql/blob/main/docs/developer_guide.md#developer-quickstart
Please use this over my comments.  There is room for improvement no doubt; reach out any time...  Best of luck 😎

Sure ! I will start working on this a as soon as I am done with my semester exams


Unclear documentation about outgoing traffic

This page only seems to focus on ingoing traffic, ie traffic that starts on the internet and ends in a backend machine. But what about the reverse? If a VM is behind an application gateway, and it doesn't have it's own public IP, and it tries to make an outgoing connection to some service on the internet, will that traffic go through the application gateway? If yes, how does one configure that? And if no, is a public IP the only way to get it that access to the internet?

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

ID: 0373061e-623f-f6ca-7d42-e32ea0299c66
Version Independent ID: 20930963-7740-a05e-8743-2210404675da
Content: How an application gateway works
Content Source: articles/application-gateway/how-application-gateway-works.md
Service: application-gateway
GitHub Login: @abshamsft
Microsoft Alias: absha

@batanun , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

@batanun The application gateway is a reverse proxy, it only works for inbound connections to the application workloads. If you need to inspect outbound connections, you should look to products such as the Azure Firewall or forward proxies.

#reassign:surajmb

#please-close


Use custom contract resolver overrides for Dictionary and Array contracts

Changes

added test for custom contract resolver that overrides default array contract
only treat as array or dictionary when current NJsonSchema logic and contract resolver agree

The serialization by json.net in regards to dictionaries and arrays is not always desired. There are some cases that NJsonSchema differs (e.g. when you have extended a dictionary and added a property). When using both json.net and NJSonSchema and customizing serialization behavior, the contract resolver is a way to that both serialization and schema generation can be customized to do something differently.
These changes works towards customization being treated the same way for serialization and schema generation. Further changes should be made so that the custom treating of objects (e.g. dictionaries) by schema and code generation is done via custom contract revolvers rather than separate logic.

I think there's a compile error:

https://ci.appveyor.com/project/rsuter/njsonschema-jlw0p/build/1135

Apologies. I had some c#7 in there.

No problem. Thanks for the fix... Ill have to do more testing until we can merge the branch... I also have to change NSwag to support these changes.


Fixes #320

Restores extendedGoals and modifies the progress bar for situations when achievements are greater than the goal value. Fixes #320

@stiplady :+1: Thanks for the pull requests ! I will test that ;)
(This feature will be recreated inside the new stravistix UI - Dashboard section in future)


ssh.exe uses 25% of CPU (quad core processor)

When copying file via rsync -ahvzP --append-verify ./source user@host:/destination, the process ssh.exe consumes 25% of CPU.

Thanks @darkdragon-001 for posting.  To clarify -- is this unexpected given your particular network setup?  rsync uses ssh to transfer data securely over the network, and ssh does some heavyweight encryption.  If the network connection between your machines is fast, and/or your CPU is slow, then it's very possible that ssh won't be able to encrypt data fast enough to keep up.  In which case you'll see it using a full CPU core (25% on a quad-core machine).
ssh can be configured to use less CPU-intensive (and theoretically less secure) encryption, if that's what you're looking for.

8 MBit/s = 1MB/s upstream, Intel Core 2 Quad Q9550 @ 2.83 GHz
Maybe I setup a computer connected with Gigabit to test maximum transfer speed...

For what it's worth, the Core 2 Quad Q9550 is an 8-year-old chip.  It's relatively old and slow compared to newer processors.
I feel like that chip ought to be able to push a little more than 8mbits over ssh, but not a whole lot more, and maybe newer versions of ssh are negotiating more CPU-intensive encryption schemes?
Do you have reason to believe that this machine could transfer data significantly faster and/or with less CPU utilization using a real Ubuntu 14.04 installation or VM?

A 1.86ghz Core 2 should be able to push around 139MiByte/s using aes128-ctr, which is the default mode for ssh on Ubuntu. You can do ~13MByte/s on a 200Mhz Pentium II. No idea why his core is maxed, mind.

It has not been my experience that ssh itself achieves those speeds on this hardware...  Do you have a benchmark of it (rather than an alternate implementation of just the algorithm) that you can cite?

Hm...  I think perhaps the truth is somewhere in the middle:
I used to use an old MacBook Pro with a Core 2 2.33ghz as my main computer.  I just dug it out and tried running rsync from it using the OP's arguments.  I was able to sustain in the neighborhood of 25MB/sec.  Which is considerably slower than 139MB/s, but considerably faster than 1MB/sec.
I'm seeing around 15MB/sec from WSL, on a machine with a Core i7-6500U.  Not that old-Mac to new-WSL is a very meaningful comparison...  The receiving server was running native Ubuntu 14.04 in both cases.
It's worth observing that the OP is using a compressed data stream (-z), which probably uses nontrivial CPU time, but that happens in the rsync process, not the ssh process.  (Unless any of the OP's ssh config files contain Compression yes...)
@darkdragon-001 -- do you have any customizations in /etc/ssh/ssh_config or ~/.ssh/config that would affect this connection?

~/.ssh/config does not exist
/etc/ssh/ssh_config contains
Host *
SendEvn LANG LC_*
HashKnownHosts yes
GSSAPIAuthentication yes
GSSAPIDelegateCredentials no

So I don't think this should affect the speed...

What kind of machine are you rsync'ing to?  Another machine running WSL and Ubuntu's OpenSSH?  An Ubuntu server (what version)?  Some other Linux?
If you're willing to post the output of ssh -vv user@host, that might contain some useful information.  Or if not the whole output, at least post what encryption is being negotiated.

Your new numbers seem about right. Here's a dude from 2009 getting 23.7MByte/s between his AMD Athlon 4600 and a Core 2 Mac. These are all pre AES-NI instruction machines. On my circa 2009 i7 with AES-NI I am getting around 150MB/s (~1.2Gbit/s) on WSL with scp over localhost.

For the next test, I omitted the -z of rsync since copying a file full of zeros lead to 80 MB/s over my 1 MB/s internet connection. Further, I used dd if=/dev/urandom to create a file filled with random content to do the tests. The rsync command is always run on WSL.
Test1: Ubuntu 16.04 with SSD connected via Gigabit Ethernet to Windows 10 WSL with SSD (without -z):
~ 42.5 MB/s, rsync and ssh have between 5% and 20% CPU.
Test2: Windows 10 WSL with SSD connected via Gigabit Ethernet to Ubuntu 16.04 with SSD (with -z):
~ 85 MB/s, rsync about 25% and ssh nearly 0% CPU
Test2.5: Windows 10 WSL with SSD connected via Gigabit Ethernet to Ubuntu 16.04 with SSD (without -z):
~ 70 MB/s, rsync about 10% and ssh nearly 25% CPU
Test3: Windows 10 WSL with SSD connected via Internet (50 MBit/s down, 10 MBit/s up) to Ubuntu Server 16.04.01 LTS (without -z):
~ 1 MB/s, rsync about 0% and ssh about 25% CPU

Hm...  That's very interesting.
If you run rsync with --rsh="ssh -vvv", ssh will print out connection debugging information.  Do you see any interesting differences between its debug output in Test2 vs Test3?
Also, would it be possible to test copying from the Ubuntu 16.04 with SSD machine to the Ubuntu Server machine?  Just to see how ssh behaves with WSL out of the picture entirely.

Great tests. I wonder we're really just seeing something like #981 here. How about  just:
WSL$ scp you@native-server:bigfile /dev/null

@therealkenc -- ah, another good test.  For what it's worth, if none of the above tests turn up anything, I'm wondering if it's related to #971 (which I'm not immediately sure how to test for).

Unfortunately, I have some problems with my server so I could only do local tests. I used scp command this time. Writing to SSD or to /dev/null did not make any difference.
3 machines with SSDs connected via Gigabit Ethernet:

WSL (Core2Quad -> maximum 25% CPU)
U+ (Ubuntu 16.04, Corei7 -> maximum 25% CPU)
U- (Ubuntu 16.04, Pentium 4 -> maxium 100% CPU)

* indicates that scp command was run on this machine
1: (ssh: 12%, scp:  2%) `U+`   --55MB/s--> *`WSL` (ssh: 20%, scp:  3%)
2: (ssh: 25%, scp:  7%) `WSL`* --75MB/s-->   `U+` (ssh: 10%, scp:  5%)
3: (ssh: 25%, scp:  2%) `WSL`* --40MB/s-->   `U-` (ssh: 50%, scp:  4%)
4: (ssh: 45%, scp:  3%) `U-`   --40MB/s--> *`WSL` (ssh: 15%, scp:  2%)
5: (ssh: 10%, scp:  1%) `U+`*  --70MB/s-->   `U-` (ssh: 45%, scp: 15%)
6: (ssh: 40%, scp:  8%) `U-`*  --60MB/s-->   `U+` (ssh: 15%, scp: 4%)
7: (ssh: 14%, scp:  1%) `U+`   --55MB/s-->  *`U-` (ssh: 40%, scp: 12%)
8: (ssh: 40%, scp:  7%) `U-`   --75MB/s-->  *`U+` (ssh: 17%, scp:  4%)

I noticed the following:

SSH on WSL is bottleneck when sending (25% CPU) [2; 3]
WSL somehow limits speed to U- which is capable of more speed [3/5; 4/8]
Sending data is usually faster then fetching data (and needs less CPU) [2/1; 5/7]

Great you took the effort to test this more. Can you explain [3/5; 4/8] -- apologies, it will be obvious once you do (threads?). These numbers look pretty reasonable, considering, no? I mean, based on the numbers I've seen online for "older" machines (ie without AES-NI) you're being limited by AES not your GigE local net. We're getting pretty pedantic at this point, but you could try doing the same U- scp tests with a Windows ssh (open or MS) and see if it is any different than WSL on the same box.
One thing I left unmentioned up-post is that AES proper (any implementation) isn't going to cross out of userspace, so this shouldn't theoretically be in WSL's wheelhouse. The WSL emulation layer doesn't even know your data is encrypted. What I was really looking for was WSL related overhead in either the fs or tcp system calls. That's really the only place you should be seeing any WSL influence here.

This issue went stale without comment from the team, but I can confirm something is going on here; I am just not sure what. Note the 374.88K bytes/sec below.
This is rsync into a chroot folder but I doubt that's a factor. It isn't hardware factors (cpu speed, network speed) because I happened to do the same operation in a Ubuntu VM on the same machine just a few minutes earlier and the operation was 10x faster. Sorry I don't have exact stats; the VM results are out of my scroll buffer. But in a VM it was "didn't even think to look" speed while in WSL it was "taking so long I left" speed. I am really starting to think it's something related to #981, which I am also seeing in spades. This is in VolFs.
# emerge-webrsync
Fetching most recent snapshot ...
Trying to retrieve 20160929 snapshot from http://gentoo.mirrors.tera-byte.com ...
Fetching file portage-20160929.tar.xz.md5sum ...
Fetching file portage-20160929.tar.xz.gpgsig ...
Fetching file portage-20160929.tar.xz ...
Checking digest ...
Getting snapshot timestamp ...
Syncing local tree ...

Number of files: 207,023 (reg: 179,492, dir: 27,531)
Number of created files: 207,022 (reg: 179,492, dir: 27,530)
Number of deleted files: 0
Number of regular files transferred: 179,492
Total file size: 415.23M bytes
Total transferred file size: 415.23M bytes
Literal data: 415.23M bytes
Matched data: 0 bytes
File list size: 4.06M
File list generation time: 0.007 seconds
File list transfer time: 0.000 seconds
Total bytes sent: 180.33M
Total bytes received: 3.55M

sent 180.33M bytes  received 3.55M bytes  374.88K bytes/sec
total size is 415.23M  speedup is 2.26
Cleaning up ...

Performing Global Updates
(Could take a couple of minutes if you have a lot of binary packages.)

I have experienced this also while compiling Ruby. It seems there are several issues related, not to a specific command, but rather overall to WSL being limited to 25% of CPU. Is this limit an intentional feature of WSL? I'd like to increase this limit if possible. See also #358

Essentially a discussion thread that ran course. A single threaded compile or single threaded ssh can (and will) max out a CPU if that's the bottleneck (contrast disk or network io). There's no known limitation specific to WSL in that regard (if WSL couldn't use all CPUs people's heads would explode). It is possible (but improbable) there are scenarios where WSL behaves different from the Real Thing for reasons, but in the event that is the case what would be needed to track further would be a test case with a tight repro.


Karma: Explore running tests in parallel

Task Description

As was mentioned multiple times over the course of this project and again this week, it might be beneficial to split up Karma tests across multiple CI jobs in parallel to speed things up and shorten the feedback loop.
karma-parallel looks like a promising candidate for that.
We might be able to use it in combination with a test matrix to pass a custom shardIndex, but not sure if that works.

Assigning @samwhale because he expressed interest in this.
/cc @littlemilkstudio @BrittanyIRL @merapi for visibility


Lona Hot Reload

Hi,
Are there any plans to support hot reloading for Lona apps ?
it would be tremendously helpful to have that features for developers. Currently, i use watchfiles  to detect any code changes in the folder and restart the script, but this produces a side effect that server shows disconnected in the browser.
Thank you!

Hi @eldjoko,
Not having hot-reload was an active design-decision: https://lona-web.org/1.x/faq.html#why-has-lona-no-live-reload-feature-like-aiohttp-devtools
Maybe I would be open to a solution when someone had a good concept for testing such a feature and would be willing to maintain it.


⚠ BeyondHD has degraded performance

In d0dff88, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 4977 ms

Resolved: BeyondHD performance has improved in ca97c6c after 6 minutes.


Add support for TOP before ALL/DISTINCT

Adds support for dialects (Redshift) that expect the TOP keyword in a SELECT statement to appear before the ALL/DISTINCT option. For example: SELECT TOP 2 DISTINCT FROM tbl;
https://docs.aws.amazon.com/redshift/latest/dg/r_SELECT_list.html

Pull Request Test Coverage Report for Build<PHONE_NUMBER>4
Details

23 of 25 (92.0%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.391%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/dialect/mod.rs
1
2
50.0%

src/dialect/redshift.rs
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
30587

Relevant Lines:
34217

💛  - Coveralls

Thanks @yoavcloud and @iffyio


Require pm^4.7

Hello, and thank you for leaving my project a PR. I see that you're requiring a more recent version of pocketmine, could you explain how this benefits my library or plugin authors who use this library?

Thank you, its nice hearing that my project is helpful. But why should it target pm ^4.7 instead of 4.0.0? Are you having issues with composer require-ing the project. Does this fix an issue?

Oops, I intended for the version to be ^4.0 which would fix your problem. Would you mind changing your PR to target version ^4.0 instead, so the library isn't restrained to 4.7 and up? Once you do I think the PR will be ready to merge and I will realease a new version.

Thank you, expect Sounds v1.2.1 soon!

I made a slight hiccup, but now Sounds v1.2.2 instead of v1.2.1 should work for your project! 😄


changes to parameterised azure blob connections

This exact change is present in the forked branch of @sauravmndl  too.
https://github.com/cloudfoundry-incubator/service-fabrik-backup-restore/compare/master...sauravmndl:feature/azure_cost_reduction?expand=1
@swativerma1  Do you need this change urgently?

there are different changes as compared to https://github.com/cloudfoundry-incubator/service-fabrik-backup-restore/compare/master...sauravmndl:feature/azure_cost_reduction?expand=1
The earlier one has bugs.
Anyways, raised another PR with additional more changes, so closing this