Custom code snippet not working in react

VSCode Version: Code 1.9.1 (f9d0c687ff2ea7aabd85fb9a43129117c0ecf519, 2017-02-09T00:26:45.394Z)
OS Version: Windows_NT ia32 6.3.9600
Extensions:

Extension
Author
Version

vscode-eslint
dbaeumer
0.10.18

Steps to Reproduce:

Create a .js file
Create a custom code snippet inside of javascript.json such as the one below:

"Target Blank": {
	"prefix": "_b",
	"body": [
		"_blank\" rel=\"noopener noreferrer\" "
	],
	"description": "Prevent new tab from having access to original window."
}

In a react javascript file with the .js extension, inside of the

render() {
     return (
          <THIS AREA>
     )
}

try to type a custom made code snippet (_b). It won't work.
3. Outside of  return area type _b. It will work. (inserts _blank" rel="noopener noreferrer").

Interesting... I see the snippet when I hit F1 > Insert Snippet but not when I press Ctrl+Space


How to run one specific "it" case by command without using "only" function?

I know how to run one specific file from the wiki, but how to run one "it" case?

it.only or describe.only is what you're looking for, I think.

@ematipico I mentioned it in the title, I don't want to write down "only" in the script.
My question was, is there any command like below?
$ mocha test\demo3.js (some "it" case)

@li-01 You can use --grep or --fgrep.

@outsideris thank you very much!


Still copies of npmcdn.com

https://github.com/Farata/angular2typescript/search?utf8=✓&q=npmcdn.com

https://github.com/BoostIO/Boostnote/pull/127/files


Make BaseAdapter describe the mandatory adapter interface

BaseAdapter should document the mandatory interfaces for implementing your own adapter from scratch for a different HTTP library. Currently requests requires all the parameters from HTTPAdapter implementation of send to be defined so copying them and relevant documentation over to the base adapter. Also adding relevant parts of documentation on what close is to the base class

There's one small note from @sigmavirus24 but I'd be happy to merge this when that's resolved.


Timeout getting IP address - Self-signed certs

Rancher Versions:
Server: 1.3.0
healthcheck: 0.2.0
net: holder & 0.8.1
network-manager: 0.4.0
scheduler: 0.5.0
kubernetes (if applicable): NA
Docker Version:
1.12.5
OS and where are the hosts located? (cloud, bare metal, etc):
Ubuntu 16.04 LTS, virtual machines/bare metal, air-gapped network
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
1 rancher server
2 rancher hosts
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Swarm
Steps to Reproduce:
Generated self signed certs (cn name is rancher-server), built nginx proxy container per rancher instructions w/ self signed certs, linked to rancher-server container.
Able to use https to get to the rancher-server, change API settings for air-gapped registry, build new swarm environment. Go to add host to the swarm environment, able to grab all the containers needed for agent, but it never fully comes up.
Results:
Error logs for rancher-server container:
2017-01-06 17:29:48,880 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153->instanceHostMap:153] [instance.start->(InstanceStart)->instancehostmap.activate] [] [ecutorService-8] [c.p.e.p.i.DefaultProcessInstanceImpl] Agent error for [compute.instance.activate.reply;agent=1]: Timeout getting IP address  2017-01-06 17:29:48,880 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153] [instance.start->(InstanceStart)] [] [ecutorService-8] [i.c.p.process.instance.InstanceStart] Failed [1/3] to Starting for instance [153]  2017-01-06 17:29:48,904 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153->instanceHostMap:153] [instance.start->(InstanceStart)->instancehostmap.activate] [] [ecutorService-8] [c.p.e.p.i.DefaultProcessInstanceImpl] Agent error for [compute.instance.activate.reply;agent=1]: Timeout getting IP address  2017-01-06 17:29:48,904 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153] [instance.start->(InstanceStart)] [] [ecutorService-8] [i.c.p.process.instance.InstanceStart] Failed [2/3] to Starting for instance [153]  2017-01-06 17:30:20,682 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153->instanceHostMap:153] [instance.start->(InstanceStart)->instancehostmap.activate] [] [ecutorService-8] [c.p.e.p.i.DefaultProcessInstanceImpl] Agent error for [compute.instance.activate.reply;agent=1]: Timeout getting IP address  2017-01-06 17:30:20,682 ERROR [f2978b68-6612-4ae5-9a63-25c5de1632ea:3833] [instance:153] [instance.start->(InstanceStart)] [] [ecutorService-8] [i.c.p.process.instance.InstanceStart] Failed [3/3] to Starting for instance [153]  2017-01-06 17:30:31,711 ERROR [f62079cf-9fa8-4947-b4e6-405025eef70c:1873] [service:10] [service.upgrade] [] [cutorService-13] [c.p.e.p.i.DefaultProcessInstanceImpl] Expected state running but got error: Timeout getting IP address  2017-01-06 17:30:31,717 ERROR [:] [] [] [] [cutorService-13] [.e.s.i.ProcessInstanceDispatcherImpl] Expected state running but got error: Timeout getting IP address
Expected:
All containers for agent up, no errors.

Can disregard. All the containers came up after about 10 minutes. Now when adding a host, it comes up immediately.
Having an issue with swarmkit now. Telling me the tokens have expired.

I'll close this ticket out then.


Client authentication in webhook mode

Hello. We have a bot, and we would want it to use webhook mode instead of long polling. However we are currently concerned that it is hard to determine if the incoming request is legitimate (sent by Telegram API) or not (sent by hacker who knows IP adress of our webhook server).
As far as I understand, we have only two options available right now:

Embed some secret value into URL.
Verify client IP address.

The first approach is pretty insecure - it is hard not to leak token due to a timing attack, or in the request log.
The second approach is much better, but IP is vulnerable to low-level transport attacks, and we would like to defend against them too.
To achieve full security, I propose to add some means of authentication support to webhook mode. It may be TLS client certificate, HMAC signature based on request payload or any other kind of authentication.
If you agree that client authentication is useful, I am ready to implement it and send pull request.
Please note that we use public telegram-bot-api instance, not local. I still open this issue here because, AFAIU, they share the same code base and any enhancement added to this repository will eventually appear in api.telegram.org too.

Are you concerned exclusively about your own bot, or that it's [too] easy for other people to improperly secure their bots?

Yes, our primary concern is our bot. We want it to be secure against as many attacks as possible, but without using complex architecture like managing local telegram-bot-api instance.

it is hard not to leak token due to a timing attack

If your webhook is https://example.com/mybot/110201543:AAHdqTcvCH1vGWJxfSeofSAs0K5PALDsaw, and you route all requests to https://example.com/mybot/* to your handler, then capture the "110201543:AAHdqTcvCH1vGWJxfSeofSAs0K5PALDsaw" and compare it to your current token using a constant-time algorithm (like hmac.compare_digest), does that meet your needs?

or in the request log.

Can you restrict access logs to people who already have access to the bot's token?
Are there other issues you're concerned with?

@MikailBag

The first approach is pretty insecure - it is hard not to leak token due to a timing attack, or in the request log.

I would say that it is really hard to leak the token due to timing attacks if you are concerned about them. For example, @nmlorg suggested an absolutely secure way to mitigate timing attacks. Also, if an intruder can read files on your server, for example, request log, then your security is already compromised.
You can also use IP range check as additional security measure, so only a provider-wide intruder can try to attack you.

If you agree that client authentication is useful, I am ready to implement it and send pull request.

I don't think that client authentication is useful, because there are simpler and more efficient solutions. If you think that you can't protect the secret URL path from leaking, then switching to a local Bot API server is a reasonable security measure. Also, you can continue to use getUpdates, which has no downsides compared to webhooks.

Thanks for your comments. Thinking about it more, I agree that checking both secret and source is secure enough.


All documents and external links should open in a new tab

I think all links to pdfs and other sites should open in a new tab, links to other pages on this site should open in the current tab.

Users need to re-scroll down the page when they navigate back to the community profile.


fix: double tap on currency selects AFN

Details

Fixed Issues

$ https://github.com/Expensify/App/issues/30508
PROPOSAL: https://github.com/Expensify/App/issues/30508#issuecomment-1782954641
Tests

Go to Create money request flow
On amount page, click on currency symbol multiple times
Verify if currency is selected only after the page has transitioned and user selects the currency

[X] Verify that no errors appear in the JS console

Offline tests

QA Steps

[X] Verify that no errors appear in the JS console

PR Author Checklist

[X] I linked the correct issue in the ### Fixed Issues section above
[X] I wrote clear testing steps that cover the changes made in this PR

[X] I added steps for local testing in the Tests section
[X] I added steps for the expected offline behavior in the Offline steps section
[X] I added steps for Staging and/or Production testing in the QA steps section
[X] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[X] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[X] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[X] I included screenshots or videos for tests on all platforms
[X] I ran the tests on all platforms & verified they passed on:

[X] Android: Native
[X] Android: mWeb Chrome
[X] iOS: Native
[X] iOS: mWeb Safari
[X] MacOS: Chrome / Safari
[X] MacOS: Desktop

[X] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[X] I followed proper code patterns (see Reviewing the code)

[X] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[X] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[X] I verified that comments were added to code that is not self explanatory
[X] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[X] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[X] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[X] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[X] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[X] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[X] I verified the JSDocs style guidelines (in STYLE.md) were followed

[X] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[X] I followed the guidelines as stated in the Review Guidelines
[X] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[X] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[X] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[X] I verified that if a function's arguments changed that all usages have also been updated correctly
[X] If any new file was added I verified that:

[X] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[X] If a new CSS style is added I verified that:

[X] A similar style doesn't already exist
[X] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[X] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[X] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[X] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[X] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[X] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[X] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/27822551/6e6f7800-a284-49c5-bab4-baa229d434be

Android: mWeb Chrome
https://github.com/Expensify/App/assets/27822551/e8fdfc90-7404-4985-aee2-48220c8b2fa4

iOS: Native
https://github.com/Expensify/App/assets/27822551/8a6ec7a4-fb7d-4f36-8ec2-2e309a2f7bc7

iOS: mWeb Safari
https://github.com/Expensify/App/assets/27822551/b3ed1bca-07bd-4d4d-afb9-4c52b87cf18f

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/27822551/1852ba45-d21c-4894-aead-a14c6682d96e

MacOS: Desktop
https://github.com/Expensify/App/assets/27822551/02406f5d-c737-4258-bcb5-607daf584600

@BhuvaneshPatil could you please complete the checklist?

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@BhuvaneshPatil seems like your checklist is outdated, mind checking https://raw.githubusercontent.com/Expensify/App/main/.github/PULL_REQUEST_TEMPLATE.md?

Ignore me, I'm just building this PR to verify whether this follow up PR introduced a regression

:test_tube::test_tube: Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! :test_tube::test_tube:

Android :robot:
iOS :apple:

https://ad-hoc-expensify-cash.s3.amazonaws.com/android/31225/index.html
https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/31225/index.html

Desktop :computer:
Web :spider_web:

https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/31225/NewExpensify.dmg
https://31225.pr-testing.expensify.com

🚀 Deployed to production by https://github.com/luacmartins in version: 1.4.1-13 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Found a pretty big error

@andrewram4287 @borcuttjahns I found a pretty big error in how I was handling the centering—I was subtracting the means instead of adding them back in! I have now added some tests to check this code more thoroughly. As you can imagine, this error could have affected a lot of late observations. Apologies.
I will merge this now, but wanted you to be able to see the diff.

Actually... hold on. My test may be the backwards case here.


Add Reformstark Martin

using the magento graphql endpoint
maybe have some look at the units - they have to be parsed out of the product name and not working well for some products
categorization done

sind laut api etwas über 4000

Added categories


Remove unnecessary assert on sub_module.training

Related: https://github.com/Lightning-AI/pytorch-lightning/issues/19467
Why were these asserts added? nn.Module.training is for controlling forward() behavior. I have never seen it used to control backward() behavior, let alone raise an error because of it.
To me and the users in the ticket I linked, these checks are very unexpected. It seems like training is being co-opted for something beyond what it was originally intended for. If I just put my whole model into train mode before calling backward, I stop seeing these errors. How is training expected to be set on partially frozen models and why?

When removing these asserts it allows me to launch my training script. Yet unaware of whether it has any consequences further down though.

I tried removing these asserts as well but now I get an error later on parameter_offload.py:
File "/.local/lib64/python3.11/site-packages/deepspeed/runtime/zero/partitioned_param_coordinator.py", line 316, in fetch_sub_module     assert param.ds_status == ZeroParamStatus.AVAILABLE, param.ds_summary()

Does it get merged? It is really ridiculous to have such an assertion

Does it get merged? It is really ridiculous to have such an assertion

Last I checked, it doesn't work even if you remove the assertion. That is why I gave up on this.

Does it get merged? It is really ridiculous to have such an assertion

Last I checked, it doesn't work even if you remove the assertion. That is why I gave up on this.

@ringohoffman, @Boltzmachine, apologies for missing this.
First, I want to affirm your observations:

The assertion is incorrect and a problem for backward pass of frozen weights.
We hijacked .training for module prefetching optimization by have separate prefetcher for (1) forward+backward trace (i.e., training) and (2) forward trace (i.e., eval/inference).
It makes sense that removing these assertions uncovers problems previously unknown to us, since we have not tested those execution paths. We have previously assumed .train() workaround to make things work. However, creating gradients on frozen weights is not a long-term solution, so requires fixing.

Second, here are my thoughts for next steps:

Remove these assertions and the misuse of .training for prefetching optimization. This will enable correct handling of backward on frozen weights and avoid user confusions (such as those discussed in the Lightning forum).
Fix issues arising from the above change. @ringohoffman, are you able to revive this PR or share those failures? I am curious if disabling prefetching would address the failures you observed.
Improve prefetching robustness:

A different way to distinguish between forward+backward trace and forward trace.
Eliminate prefetching across forward/backward boundary. Instead have separate prefetchers for forward and backward traces. We need to understand the performance implications of this.

We would really appreciate your help on the above plan. We also understand this might no longer be priority for you.
@tohtana, FYI


for_education: Add better screenshots with proper size.

The screenshots we have currently don't really fit into the width available and the font looks too small. This commit adds newer screenshots that have been scaled to fit a width of 400px such that the font is readable with a small amount of content.
Details of how the screenshots were taken:

200% zoomed-in browser on a 4k monitor.
Window width small enough for both the left and right sidebars to be hidden.
The contents of the message wrapped at around 9-10 words per line.

@alya @timabbott Also added a dark mode screenshot. :)

@timabbott The build error is definitely not me. ./tools/lint --fix didn't really return any output.

I rebased this so that CI passes, and merged it.  Huge thanks for making these screenshots happen!


Version_3.0b3 with cocopods

Is there a way to use the Version_3.0b3 with cocopods.  Tried a number of different ways but can get the version 3.x to load.
pod 'XCGLogger', 'Version_3.0b3'

You can grab the Swift 2 development branch, if that's what you are after:
pod 'XCGLogger', :git => 'https://github.com/DaveWoodCom/XCGLogger.git', :branch => 'swift_2.0'

Thanks @zbeckman, I don't actually use CocoaPods, so wasn't even sure how it was done.


Problems with map.locate()

i am try open
https://leafletjs.com/examples/mobile/
to see how map.locate({setView: true, maxZoom: 16});
its not work for me in sudan .
i am get this erorr
Geolocation error: Unknown error acquiring position.

Duplicate of https://github.com/Leaflet/Leaflet/issues/7087 . As I said there, use navigator.geolocation directly. If you're unable to get a geolocation with that API, then it isn't Leaflet's fault.


metrics: total borrowed/created connections

I'm testing a move from dbcp2 to hikari. Thanks for writing another alternative for pooling.
In dbcp2 i've been monitoring my pools via JMX for borrowedCount (the amount of times connections have been borrowed/used) and createdCount (the amount of times a connection is created).
These are running counts, and i do deltas on the counts for my monitoring.
I'm wondering if we can get something like this in Hikari.
Thanks.

@abiacco What does the delta of these two values tell you?  I.e. how is that metric informative?

@abiacco You can get one of those values, the number of times connections have been used.  If you enable Dropwizard Metrics, the "pool.Usage" metric is a com.codahale.metrics.Histogram, which has a count value indicating the number of borrow/return pairs.
We don't track the number of connection creation events.  I am still curious how the delta of these two values is useful for monitoring.

+1 for number of creation events. By monitoring that counter I am able to determine that we are creating and dropping more connections than I expect.  It is a hugely useful counter to have.

 I am working on a PR that will fix the pool overfill issue and this is really the metric I need to verify that the fix is working correctly

Send a pull and we'll take a look.

@abiacco @samhendley In HikariCP v2.6.0 there is a new DropWizard/Prometheus metric for connection creation.  In DropWizard, the metric is called <poolname>.pool.ConnectionCreation. In Prometheus, the metric is hikaricp_connection_creation_millis.


This document is outdated, obsolete, and seemingly abandoned by MS..

When was the last time someone saw "Connection String" or  "user name" anywhere inside the Cosmos Web Console??
Also, when was the last time someone at MS tried to connect to Cosmos using Studio 3T..probably a year ago?

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

ID: 21d6cd84-d0c4-eea9-c2c6-e6f3be91dad8
Version Independent ID: 6fa411bf-934b-fc27-8fef-09f07c3f0299
Content: Connect to Azure Cosmos DB's API for MongoDB using Studio 3T - Azure Cosmos DB
Content Source: articles/cosmos-db/mongodb-mongochef.md
Service: cosmos-db
Sub-service: cosmosdb-mongo
GitHub Login: @sivethe
Microsoft Alias: sivethe

@LastTribunal Thank you for bringing this to our attention. I have assigned this to the content owner to evaluate and update as appropriate.

@LastTribunal Can you be a bit more descriptive with regard to the items within the document that no longer are applicable. With the latest version of Studio 3T, I am seeing that the authentication modes are a little different but, everything else applies.

If you are referring to "Connection String" or "user name", I was referring to Cosmos, There are no such things there...

@Mike-Ubezzi-MSFT and thank you for prompt and diligent response to my complaint

@LastTribunal I think you are looking at the UI for a SQL API account (it's keys for this type of account). I just created a new Mongo DB API account and validated that the UI looks as it is in the doc. I don't see any update to the doc as of now.

I don't see any fix to the doc at this point, will close this issue.
#please-close


Creating the Farm Role - ERROR

We are running into issues where kitchen will fail on 'Creating the Farm Role'
output of kitchen
-----> Creating ...
Creating farm with name: KITCHEN_FARM_default-CentOS7_xxxxxx-xxx
Success: farmId is 1111
Getting the imageId for image CentOS7-Linux-November-2016
The image id is xxxxxxxxxxxxxxxx
The image os id is centos-7-x
The image platform is ec2
The image Location is us-east-1
Creating a role with name KITCHEN-ROLE-xxxxxxxxxxxxx
The role id is 1234
RoleImage association created
Creating the Farm Role

------Exception-------
Class: Kitchen::ActionFailed
Message: 1 actions failed.
Failed to complete #create action: [409 Conflict] on default-CentOS7

Hello,
What version of Scalr are you running? Could you join a safe version of your kitchen.yml file?

Scalr version 7.4.
I not sure I understand the kitchen.yml question.

Oh sorry, I was asking you to attach the kitchen.yml file (sanitized so that you don't leak any internal information) so that we could take a look at the config and understand the issue.

driver:
name: scalr
provisioner:
name: chef_zero
platforms:

name: CentOS7
driver:
scalr_api_url: 'https://xxx.xx.com'
scalr_env_id: '1'
scalr_project_id: 'x5xxxxxx'
scalr_server_image: CentOS7-Linux
scalr_location: 'us-east-1'
scalr_server_instanceType: 'm3.medium'
scalr_platform: 'ec2'

suites:

name: default
run_list:

recipe[test::default]
attributes:

Wild guess: any policy enforcement on the environment you use?

the only policy we have is forcing the AWS build to specific VPC

Ok thanks for raising the issue. VPC are not currently handled by the plugin, and we are waiting for the next Scalr release to implement this support along with multi-cloud support. Please verify that everything is working when this policy is not enforced, we will add support for VPC in the following releases of kitchen-scalr.

any eta on when all of that will hit?

Nothing accurate yet, a few weeks probably.

On 2 Mar 2017, at 19:18, lcc2207<EMAIL_ADDRESS>wrote:
any eta on when all of that will hit?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/momohawari/kitchen-scalr/issues/1#issuecomment-283734768, or mute the thread https://github.com/notifications/unsubscribe-auth/AF91NnR17maQRvsNC0EHwjO7q5CncAr3ks5rhwf1gaJpZM4MQMKq.

Ok it was way shorter actually. It should work with Scalr 7.5 and kitchen-scalr 0.2.0. We introduced the scalr_base_farm_role configuration parameter in the kitchen.yml file that should allow you to put a VPC with something like:
driver:
name: scalr
provisioner:
name: chef_zero
platforms:

name: CentOS7
driver:
scalr_api_url: 'https://xxx.xx.com'
scalr_env_id: '1'
scalr_project_id: 'x5xxxxxx'
scalr_server_image: CentOS7-Linux
scalr_location: 'us-east-1'
scalr_server_instanceType: 'm3.medium'
scalr_platform: 'ec2'
scalr_base_farm_role:
networking:
networks:
- id: 'some-id-specific-to-vpc'
suites:
name: default
run_list:
recipe[test::default]
attributes:

The network id can be retrieved by using scalr-ctl farm-role get with an existing farm-role that is configured with the right VPC.
Closing the ticket now


Replace a11r with AspirinSJL in client_channel OWNERS file.

Juanli has been working pretty extensively on this code, so it makes more sense for her to be an owner, especially with David on vacation.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences


400 Bad Request with proxy configuration

Hi !
I have a problem when using this SDK with a proxy configuration.
Datadog requests don't reach at all the proxy.
When events are send, I've got this error on XCode (I don't have any errors on JS side) :
nw_http_connect_process_response [C<IP_ADDRESS>:3] http connect Proxy received status: HTTP/1.1 400 Bad Request
What I don't understand is that the requests are send very well with plain local server, I have this :
nw_http_connect_process_response [C21.1.1:3] http connect Proxy received status: HTTP/1.0 501 Unsupported method ('CONNECT')
I can the see the requests in my local server.
This is the configuration that I'm using for testing
datadogConfiguration.proxyConfig = new ProxyConfiguration(ProxyType.HTTPS, '<IP_ADDRESS>', 443)

Version of the SDK: 1.8.5
Version of React Native: 0.71.6

Thanks for the help !

Hi @martinezguillaume, we currently use the native iOS URLSession and Android okhttp proxy functionalities that rely on CONNECT to forward the requests.
If you're interested in us adding GET/POST methods, I recommend you to reach out to our support team so we can better prioritize this feature :)
Let me know if you need help on anything else!

Hi there,
What is the ProxyType used for? It seems that selecting HTTPS or HTTP both ends up in creating a tunnel using HTTP CONNECT method (no difference).
Is there a configuration that will not use CONNECT, resulting in something similar as using browser RUM proxy configuration ?

Hi @fguffon,
As mentioned in this similar issue, we're planning to add soon a customEndpoint parameter for each feature (RUM, Logs, Trace) that would enable you to achieve something similar to what the browser proxy is doing.
In the current state of the SDK there is no such feature, the proxy configuration will always use CONNECT requests.
Indeed the ProxyType HTTPS and HTTP have the same behaviour today. I think it was intended to have a different behaviour on iOS, and the HTTPS type should not create an http tunnel. We'll make that change as well in the same effort.
I hope that answers your questions, let me know if I can help further!

Hi @martinezguillaume @fguffon!
We released a customEndpoints configuration option in 2.1.0 version of the SDK.
It enables you to specify an endpoint for data to be sent to. For instance if you specify in your configuration:
config.customEndpoints = {
  rum: 'https://my-custom-endpoint.com',
  logs: 'https://my-custom-endpoint.com',
  trace: 'https://my-custom-endpoint.com',
}

Your RUM data will be sent to https://my-custom-endpoint.com/api/v2/rum, your logs data to /api/v2/logs and your trace data to /api/v2/spans.
One important note is that this is not an official alternative to the proxy configuration yet - making it so requires more time and we have more pressing items in our backlog.
So if you decide to use it for this purpose keep in mind that:

Although it should work well, we haven't tested all scenarios extensively. Feel free to open an issue or a PR if you notice some data isn't reported as expected.
As this isn't officially documented the path we send your data to might change in the future. If so, it would be mentioned in the release notes.

Let me know if this fixes your use case!

hi @louiszawadzki
Thanks it's perfectly doing the job for us.
The only feedback I have is that comparing to the configuration used on web RUM proxy configuration, there is a discrepancy retrieving the full path (/api/v2/rum) that is added in the ddforward parameter, but for now the change allowed us to make it work !


Add screenshots of wikipedia diffs to posts

It's nice to get a preview of the diff in your feed instead of just a notice that something has changed. See #151.

Any word on merging this @edsu?

I'm really sorry for the delay. I've been swamped at work, and haven't had the time to invest in learning Kubernetes so I can use the latest Node in Wikimedia Labs #154.
But there's no reason for me to delay merging this right now. I can always deploy it later. Thanks again for all your work putting this together, it's a very important addition to provide more transparency about the edits that are being announced.

Ah, thanks, and good luck with the Kubernetes switchover! Additionally, thank you for being so responsive and clear in your communication! It's been very pleasurable.
I'm looking forward to seeing the screenshots in action!


bug: [static_shock_docs] Throws on generation

Running dart run in the static_shock_docs project results in the following error:
Unhandled exception:
NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: []("flutter-bounty-hunters")
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:38:5)
#1      getItem (package:jinja/src/defaults.dart:23:18)
#2      Context.item (package:jinja/src/context.dart:66:31)
#3      StringSinkRenderer.visitItem (package:jinja/src/renderer.dart:321:20)
#4      Item.accept (package:jinja/src/nodes/expressions.dart:498:20)
#5      StringSinkRenderer.visitAttribute (package:jinja/src/renderer.dart:225:28)
#6      Attribute.accept (package:jinja/src/nodes/expressions.dart:540:20)
#7      StringSinkRenderer.visitFor (package:jinja/src/renderer.dart:520:34)
#8      For.accept (package:jinja/src/nodes/statements.dart:55:20)
#9      StringSinkRenderer.visitOutput (package:jinja/src/renderer.dart:714:12)
#10     Output.accept (package:jinja/src/nodes.dart:109:20)
#11     Template.renderTo (package:jinja/src/environment.dart:569:10)
#12     Template.render (package:jinja/src/environment.dart:556:5)
#13     JinjaPageRenderer._renderJinjaTemplate (package:static_shock/src/plugins/jinja.dart:241:37)
#14     JinjaPageRenderer.renderContent (package:static_shock/src/plugins/jinja.dart:116:5)
#15     StaticShock._renderPages (package:static_shock/src/static_shock.dart:520:28)
<asynchronous suspension>
#16     StaticShock.generateSite (package:static_shock/src/static_shock.dart:227:5)
<asynchronous suspension>
#17     main (file:///C:/Users/dicke/static_shock/packages/static_shock_docs/bin/static_shock_docs.dart:42:3)
<asynchronous suspension>

I'm working on a new plugin, having a working example project to test with would be nice

I see you've opened an issue here https://github.com/ykmnkmi/jinja.dart/issues/27
Guess we'll just have to wait for an error message

These kinds of errors come up with a lot with minor configuration issues when using Jinja templates. It's often something like you expect that a variable exists but it doesn't. I see a reference to "flutter-bounty-hunters" in your error message.
I would recommend that you look for wherever that value appears in one of the Jinja files.


kmsdrm: modesetting support for initial video mode

Add initial modesetting for the video KMSDRM driver via environment variables (SDL_VIDEO_KMSDRM_CRTCID, SDL_VIDEO_KMSDRM_MODEID, SDL_VIDEO_KMSDRM_MODELINE) and unlock aspect ratio flag decoding via DRM, so that all possible modes are exposed to the KMSDRM driver.

Awesome. Thanks very much


Pkr and Parkour not registered as parkour commands.

Compiled as of https://github.com/A5H73Y/Parkour/commit/7dea8ea280a20f32f7850c3c07c66ee32662dca1
  EnforceParkourCommands:
    Enabled: true

Players inside parkour mode can't use /parkour or /pkr commands to leave. Is it only registering /pa as parkour commands? I removed the /pa alias since I have another plugin that uses /pa for PVP arenas.

"/parkour leave" works fine for me, as does the GUI item to leave the course.
You could always add pkr as a whistlisted command:
"/pa whitelist pkr".
/pkr can then be used in-game.

@mibby Please validate what @steve4744 has suggested, before I raise this problem, thanks both.

My apologies. It seems /parkour does work but /pkr is not registered as a parkour command. I get the Non-Parkour commands have been disabled! output in chat when trying to type /pkr leave.

The pkr alias is defined in the plugin.yml with pa - as you removed pa, did you leave the pkr alias definition?
As a test you could try the original parkour and plugin.yml to see if /pkr works. As I said it works for me in-game.

Yes, I left the pkr alias definition in the plugin.yml.
Are you opped / grouped admin who inherits bypass permissions and certain you don't have /pkr in the whitelist list? As me testing without any inherited parkour permissions, wildcards, or OP bypasses, was not able to use /pkr.

You are correct in that pkr was not considered a Parkour command, I've fixed in next commit.
However @steve4744  suggested that you whitelisted the command, which would have fixed the problem.

Whitelisting would have indeed solved the problem, but being that it is a registered alias, just thought it should be allowed by default, especially for people who are unaware and wouldn't have added it to the whitelist.
Being that it is now set in the latest commit, I will close this ticket. :)


p

If you are reporting a bug, please fill in below. Otherwise feel free to remove this template entirely.
Can you reproduce the problem with latest npm?
Many errors, especially related to "missing modules", are due to npm bugs.
If you're using Windows, follow these instructions to update npm.
If you're using OS X or Linux, run this to update npm:
npm install -g npm@latest

cd your_project_directory
rm -rf node_modules
npm install

Then try to reproduce the issue again.
Can you still reproduce it?
Description
What are you reporting?
Expected behavior
Tell us what you think should happen.
Actual behavior
Tell us what actually happens.
Environment
Run these commands in the project folder and fill in their results:

npm ls react-scripts (if you haven’t ejected):
node -v:
npm -v:

Then, specify:

Operating system:
Browser and version:

Reproducible Demo
Please take the time to create a new app that reproduces the issue.
Alternatively, you could copy your app that experiences the problem and start removing things until you’re left with the minimal reproducible demo.
(Accidentally, you might get to the root of your problem during that process.)
Push to GitHub and paste the link here.
By doing this, you're helping the Create React App contributors a big time!
Demonstrable issues gets fixed faster.

Seems like this issue was created by mistake! I'll go ahead and close it. 😄
Please feel free to open another if you're having issues.


Issue with k8s.io/docs/tasks/configure-pod-container/configure-pod-configmap/

EXAMPLE INJECT CONFIGMAP IN POD
_Use envFrom
https://kubernetes.io/docs/tasks/configure-pod-container/configure-pod-configmap/_
Right:
apiVersion: v1
kind: Pod
metadata:
  name: my-pod-with-env-configmap
  labels:
    name: use-configmap-env
spec:
  containers:
  - envFrom:
    -configMapRef:
	 name: my-config-map-name
    image: nginx
    name: my-http-server

wrong in documentation
  containers:
    - name: test-container
      image: k8s.gcr.io/busybox
      command: [ "/bin/sh", "-c", "env" ]
      envFrom:
      - configMapRef:
          name: special-config

/language en

/language en

This suggestion doesn't make sense to me.

This suggestion doesn't make sense to me.

@ivanruizalcuadrado can you clarify how making that change would improve the page at https://k8s.io/docs/tasks/configure-pod-container/configure-pod-configmap/ ?
/triage needs-information

@ivanruizalcuadrado can you clarify how making that change would improve the page at https://k8s.io/docs/tasks/configure-pod-container/configure-pod-configmap/ ?
/triage needs-information

This is right code. The cofigmaps are an object array:

envFrom:
-configMapRef:
name: special-config*

wrong in the documentation documentation:
  envFrom:
  - configMapRef:
      name: special-config

El sáb, 9 ene 2021 a las 14:45, Tim Bannister<EMAIL_ADDRESS>escribió:

@ivanruizalcuadrado https://github.com/ivanruizalcuadrado can you
clarify how making that change would improve the page at
https://k8s.io/docs/tasks/configure-pod-container/configure-pod-configmap/
?
/triage needs-information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/25922#issuecomment-757206058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7SSLDZQDRIPLIETCFX5WDSZBMX7ANCNFSM4VRVAQEA
.

This is right code. The cofigmaps are an object array:

envFrom:
-configMapRef:
name: special-config*

wrong in the documentation documentation:
  envFrom:
  - configMapRef:
      name: special-config

El sáb, 9 ene 2021 a las 14:45, Tim Bannister<EMAIL_ADDRESS>escribió:

@ivanruizalcuadrado https://github.com/ivanruizalcuadrado can you
clarify how making that change would improve the page at
https://k8s.io/docs/tasks/configure-pod-container/configure-pod-configmap/
?
/triage needs-information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/issues/25922#issuecomment-757206058,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7SSLDZQDRIPLIETCFX5WDSZBMX7ANCNFSM4VRVAQEA
.


Drop old preprocessor and converter api

This looks good, @vigsterkr ?

This looks good, @vigsterkr ?

python build failed because log exceed maximum length
appveyor failed because of timeout

restarting ...

I'll leave it to @vigsterkr to merge this though

i had to fix some errors in the feature branch... it should be good now, as soon as things gets green in CIs i'm merging this and then i merge the whole feature branch into develop


Wrong test execution time for parallel test execution

Expected Behavior

I wood like to see real tests execution time.
Current Behavior

Gradle test report show me just sum of all test execution times.
Context

Steps to Reproduce (for bugs)

For example I create two class with two test for each class in one sample package.
Each test contains 5 second sleep, so in test execution each class takes me 10 second for execute.
First:
public class TestClassOne {
    @Test
    public void testOne() throws InterruptedException {
        Thread.sleep(5000);
    }

    @Test
    public void testTwo() throws InterruptedException {
        Thread.sleep(5000);
    }
}

Second:
public class TestClassTwo {
    @Test
    public void testOne() throws InterruptedException {
        Thread.sleep(5000);
    }

    @Test
    public void testTwo() throws InterruptedException {
        Thread.sleep(5000);
    }
}

I set maxParallelForks property in test task equals to 2:
test {
    maxParallelForks = 2
}

After all setup I run test task in console.
./gradlew clean test --tests \*sample\*
And test executed in parallel.
At the end I open report and it says me that I spend 20 second to test but actually only 10.

Your Environment

------------------------------------------------------------
Gradle 4.4
------------------------------------------------------------

Build time:   2017-12-06 09:05:06 UTC
Revision:     cf7821a6f79f8e2a598df21780e3ff7ce8db2b82

Groovy:       2.4.12
Ant:          Apache Ant(TM) version 1.9.9 compiled on February 2 2017
JVM:          1.8.0_152 (Oracle Corporation 25.152-b16)
OS:           Mac OS X 10.13.2 x86_64

@rosolko To be very clear, the "duration" at the top of the report "20s" is wrong. The duration for each test class "10s" is correct. Do you agree?

@eriwen Yes, correct. Just overall duration time is wrong.

Same issue for gradle 7.2 and parallel test execution with junit5.


Configure renovate to pin k8s and controller-runtime for dev preview and pin ocp

pin k8s and controller runtime to be in sync with lib-common and configure renovate to keep the pin
pin ocp api version to a hash in sync with lib-common instead of the nonexistent v3.9.0+incompatible tag

/lgtm

/test heat-operator-build-deploy-kuttl

/retest Requesting bear token: invalid status code from registry 500 (Internal Server Error)

/retest
Requesting bear token: invalid status code from registry 500 (Internal Server Error)


[Bug] Dropdown menus don't close when clicking on monitors other than the one with the dropdown.

Describe the bug
Opening a widget like the dasboard menu on one monitor and then mousing over to another without closing the widget causes the widget to stay open and prevents you from typing
To Reproduce
Have 2 monitors, open dashboard panel, click on second monitor, attempt to type
Expected behavior
Ideally the widget closes and you are able to type
Desktop (please complete the following information):

Distribution: Arch Linux
Window Manager/Desktop Environment: Hyprland

Additional context
I have a 3 monitor setup and two of my monitors a vertical. I don't see how that could matter in any way but just for your awareness.

is this still an issue

Yep, this still happens.
I cannot, however, reproduce not being able to type in my second monitor while the menus are open


cargo install cargo-crev fails.

   Compiling crev-data v0.17.0
error[E0599]: no function or associated item named `from_bytes` found for struct `ed25519_dalek::Signature` in the current scope
  --> /home/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/crev-data-0.17.0/src/id.rs:80:59
   |
80 |                 let signature = ed25519_dalek::Signature::from_bytes(&sig_bytes)?;
   |                                                           ^^^^^^^^^^
   |                                                           |
   |                                                           function or associated item not found in `ed25519_dalek::Signature`
   |                                                           help: there is an associated function with a similar name: `to_bytes`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use ed25519_dalek::ed25519::signature::Signature;`

error[E0599]: no method named `verify` found for struct `ed25519_dalek::PublicKey` in the current scope
  --> /home/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/crev-data-0.17.0/src/id.rs:81:24
   |
81 |                 pubkey.verify(&content, &signature)?;
   |                        ^^^^^^ method not found in `ed25519_dalek::PublicKey`
   |
   = help: items from traits can only be used if the trait is in scope
   = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
           `use ed25519_dalek::Verifier;`

error[E0599]: no method named `sign` found for struct `ed25519_dalek::Keypair` in the current scope
   --> /home/rust/.cargo/registry/src/github.com-1ecc6299db9ec823/crev-data-0.17.0/src/id.rs:198:22
    |
198 |         self.keypair.sign(msg).to_bytes().to_vec()
    |                      ^^^^ method not found in `ed25519_dalek::Keypair`
    |
    = help: items from traits can only be used if the trait is in scope
    = note: the following trait is implemented but not in scope; perhaps add a `use` for it:
            `use ed25519_dalek::Signer;`

error: aborting due to 3 previous errors

Caused by #368 . Install git version please. :)

Thanks, already done it.

I hit this issue as well.

Caused by #368 . Install git version please. :)

Maybe it would make sense to publish a bug fix release and yank the broken version?

I think this is no longer a problem.


Is it possible to only include imported material components to the bundle?

Bug, feature request, or proposal:
I'm using angular-cli beta.32 und angular-material 2 beta 2. Even when only import material components that are actually used in my templates, source-map-explorer is showing me, that all angular-material components are getting included in the bundle.
I don't know if this is an angular-cli or webpack problem, I couldn't find this specific topic mentioned anywhere. #webpack/webpack
If this is due to the webpack tree-shaking problem ( https://github.com/webpack/webpack/issues/2867 ) feel free to close this issue. Its my first angular-cli project and I don't have much experience yet.
What is the expected behavior?
Only include components in bundle that are used in components of the app.
What is the current behavior?
All components of angular-material are included
What are the steps to reproduce?
I have this component in my app, no other component is importing MaterialModule anywhere. The bundle produced by angular-cli still contains components like tabs, select, slider, etc.
import {NgModule} from '@angular/core';
import {CommonModule} from '@angular/common';
import {TelefonanlageRoutingModule} from './telefonanlage-routing.module';
import {TelefonanlageComponent} from './telefonanlage.component';
import {
	MdButtonModule, MdCardModule, MdCoreModule, MdDialogModule
} from '@angular/material';
import {WarteschleifeCardComponent} from './warteschleife-card/warteschleife-card.component';
import {TagesstatistikComponent} from './tagesstatistik/tagesstatistik.component';

@NgModule({
	imports: [
		CommonModule,
		MdCoreModule,
		MdButtonModule,
		MdCardModule,
		MdDialogModule,
		// MaterialModule,
		TelefonanlageRoutingModule
	],
	declarations: [TelefonanlageComponent, WarteschleifeCardComponent, TagesstatistikComponent],
	entryComponents: [TagesstatistikComponent]
})
export class TelefonanlageModule {
}

What is the use-case or motivation for changing an existing behavior?
Smaller production bundles
Which versions of Angular, Material, OS, browsers are affected?
angular cli beta 32, angular material 2 beta 2, angulaer 2.4.8
Is there anything else we should know?

I think this is a webpack issue that it doesn't get tree-shaken properly. I'm not sure there's much we can do about it from the material side unfortunately.


Add gun broadcast command

We need to be able to broadcast a signed transaction. The most important bit is that we can broadcast PSBTs but I guess we should also support broadcasting a raw transaction hex. I think the command should be:
gun broadcast [<tx-file>]

<tx-file> can be a path to either a PSBT encoded as base64 or binary. Or it can be a raw consensus encoded transaction. If <tx-file> is - gun should read it from STDIN.
If  is missing and psbt-output-dir from https://github.com/LLFourn/gun/issues/62 is set then gun should go through each .psbt file in psbt-output-dir and broadcast it if it looks finalized.

I'm closing this issue for now since I'm not sure we need it (yet).


Fix issue regarding potential race condition at connection initialization.

Add *.opendb to gitignore
Upgrade compiler to VS2015
Attempt to fix issue found regarding message being sent prior to initial connection
being verified.  This can cause incorrect message parsing.
Add aligned array - can be used when interop with native code.
Fix one issue with mutable variable not being captured by closure (VS2012 does not compile)

Will create new one with new title.


unknown flag: --configStore2URL

Hi all,
When you tried to install istio with chart version 0.2.13-chart4, you can't because of this flag. In the help of the mixer you have
--configStoreURL string
In the template templates/deployment/mixer.yaml you must replace configStore2URL by configStoreURL
To reproduce it:
ISTIO_CHART_VERSION=0.2.13-chart4
ISTIO_IMAGE_VERSION=0.8.0
helm upgrade -i istio incubator/istio --reuse-values --set istio.install=true,istio.release=$ISTIO_IMAGE_VERSION,rbac.install=false --devel --version $ISTIO_CHART_VERSION --namespace istio-system

Log of my pod:
kubectl logs -n istio-system istio-mixer-687668dc55-rgt5b -c istio-mixer
Error: unknown flag: --configStore2URL
Usage:
  mixs server [flags]

My kubectl version:
kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.4", GitCommit:"5ca598b4ba5abb89bb773071ce452e33fb66339d", GitTreeState:"clean", BuildDate:"2018-06-06T08:13:03Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"10+", GitVersion:"v1.10.4-gke.2", GitCommit:"eb2e43842aaa21d6f0bb65d6adf5a84bbdc62eaf", GitTreeState:"clean", BuildDate:"2018-06-15T21:48:39Z", GoVersion:"go1.9.3b4", Compiler:"gc", Platform:"linux/amd64"}

Auth is enable and I'm deploying istio on gcloud.
Thx

This flag no longer exists in any helm chart. We also don't have istio-mixer-* pods anymore.
What version of istio are you using? What is the incubator/istio?

Yes incubator/istio as previously mentionned but I figure this out. I was using chart version 0.2.13-chart4 and then I saw that istio-0.3.0 version exist here https://kubernetes-charts-incubator.storage.googleapis.com/.
The thing is when you try to deploy istio version 0.8.0 this charts istio-0.3.0, you can't because there is no docker ca image version 0.8.0 on docker hub.
kubectl logs -n istio-system istio-istio-ca-7d9bd9b585-xwctk
Error from server (BadRequest): container "ca" in pod "istio-istio-ca-7d9bd9b585-xwctk" is waiting to start: trying and failing to pull image

The last version is 0.7.1. In the CA deployment chart you got:
image: "{{ .Values.ca.deployment.image }}:{{ .Values.istioRelease }}"
So if you guys can create it, it will be great or maybe I missed something. For now I will try to install 0.7.1 for my testing purpose :)

In 0.8.0 CA is called Citadel

I finally make version 0.8.0 works but following only istio documentation on istio website.


Getting Cannot Find Error when building project

In the "ModEntitySpawn.java" class the package "net.fabricmc.fabric.mixin.object.builder.SpawnRestrictionAccessor" is causing a cannot find symbol error. I looked at the Fabric API and didn't see this package there anymore.

Ensure you are using the latest Fabric API and version.


New optimizers fail to load CUDA installed through conda

System information.

Have I written custom code (as opposed to using a stock example script provided in Keras): no
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04 (WSL)
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): 2.11
Python version: 3.9
Bazel version (if compiling from source): N/A
GPU model and memory: RTX 2080 Ti
Exact command to reproduce:

Create a new environment, following the official installation instructions from here https://www.tensorflow.org/install/pip#linux:

conda install -c conda-forge cudatoolkit=11.2 cudnn=8.1.0
export LD_LIBRARY_PATH=$LD_LIBRARY_PATH:$CONDA_PREFIX/lib/
pip install tensorflow

Run the beginner MNIST tutorial (or any other tutorial that calls fit) from here https://keras.io/examples/vision/mnist_convnet/

Describe the problem.
An error is raised:
libdevice not found at ./libdevice.10.bc

Note that if you switch to using the legacy optimizers, by switching this line
model.compile(loss="categorical_crossentropy", optimizer="adam", metrics=["accuracy"])

to this
model.compile(loss="categorical_crossentropy", optimizer=keras.optimizers.legacy.Adam(), metrics=["accuracy"])

then the example runs successfully.
Describe the current behavior.
An error occurs when running the example.
Describe the expected behavior.
The example should run without error, as it does when using the legacy optimizers.

Do you want to contribute a PR? (yes/no): no

Standalone code to reproduce the issue.
https://keras.io/examples/vision/mnist_convnet/
Source code / logs.
Full stack trace of the error:
    File ".../tmp.py", line 47, in <module>
      model.fit(x_train, y_train, batch_size=batch_size, epochs=epochs, validation_split=0.1)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/utils/traceback_utils.py", line 65, in error_handler
      return fn(*args, **kwargs)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/engine/training.py", line 1650, in fit
      tmp_logs = self.train_function(iterator)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/engine/training.py", line 1249, in train_function
      return step_function(self, iterator)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/engine/training.py", line 1233, in step_function
      outputs = model.distribute_strategy.run(run_step, args=(data,))
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/engine/training.py", line 1222, in run_step
      outputs = model.train_step(data)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/engine/training.py", line 1027, in train_step
      self.optimizer.minimize(loss, self.trainable_variables, tape=tape)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 527, in minimize
      self.apply_gradients(grads_and_vars)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 1140, in apply_gradients
      return super().apply_gradients(grads_and_vars, name=name)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 634, in apply_gradients
      iteration = self._internal_apply_gradients(grads_and_vars)
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 1166, in _internal_apply_gradients
      return tf.__internal__.distribute.interim.maybe_merge_call(
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 1216, in _distributed_apply_gradients_fn
      distribution.extended.update(
    File "/home/drasmuss/mambaforge/envs/tmp2/lib/python3.9/site-packages/keras/optimizers/optimizer_experimental/optimizer.py", line 1211, in apply_grad_to_update_var
      return self._update_step_xla(grad, var, id(self._var_key(var)))
Node: 'StatefulPartitionedCall_4'
libdevice not found at ./libdevice.10.bc
         [[{{node StatefulPartitionedCall_4}}]] [Op:__inference_train_function_1026]

Likely related to:

https://github.com/tensorflow/tensorflow/issues/56927
https://github.com/tensorflow/tensorflow/issues/59013

I had this problem as well and the fix above worked.

I ran into the same problem on Linux Mint victoria 21.2 x86_64 after creating a new environment with conda and installing tensorflow-gpu version 2.12.1 from the conda-forge channel.
As suggested by @SuryanarayanaY I used his attempt but without specifying the cuda-ncc version (it installed 12.3.52) and it worked.
`# Install NVCC
conda install -c nvidia cuda-nvcc
Configure the XLA cuda directory
mkdir -p $CONDA_PREFIX/etc/conda/activate.d
printf 'export XLA_FLAGS=--xla_gpu_cuda_data_dir=$CONDA_PREFIX/lib/\n' >> $CONDA_PREFIX/etc/conda/activate.d/env_vars.sh
source $CONDA_PREFIX/etc/conda/activate.d/env_vars.sh
Copy libdevice file to the required path
mkdir -p $CONDA_PREFIX/lib/nvvm/libdevice
cp $CONDA_PREFIX/lib/libdevice.10.bc $CONDA_PREFIX/lib/nvvm/libdevice/`
Thank you very much again for the solution, @SuryanarayanaY !

Same issue on Fedora 39 with fresh tensorflow from conda-forge, SuryanarayanaY fix works

Same issue, any official fix yet?


3602.2.0 files overwrite change of behaviour

Description
AWS AMI image based in EU-WEST-2 (ami-0bb8a98c666ad7adf) for release 3602.2.0 appears to have changed the behaviour for writing files to core os settings
Attempting to spin up an instance results in an error in the ingnition-files service
CRITICAL : Ignition failed: failed to create files: failed to create files: error creating /sysroot/etc/flatcar/update.conf: error creating file "/sysroot/etc/flatcar/update.conf": A file exists there already and overwrite is false

I am using Flatcar for a clusters that can't have random restarts so running with REBOOT_STRATEGY=off
variant: flatcar
version: 1.0.0
storage:
  files:
    - path: /etc/flatcar/update.conf
      contents:
        inline: |
                    REBOOT_STRATEGY=off

If I remove that then the AMI boots fine or I can specify overwrite: true
I also however set some custom limits eg
variant: flatcar
version: 1.0.0
storage:
  files:
    - path: /etc/security/limits.conf
      mode: 0644
      append:
        - inline: |
            root soft nofile 1048576
            root hard nofile 1048576

and that is now reporting the same error message
CRITICAL : Ignition failed: failed to create files: failed to create files: error creating /sysroot/etc/security/limits.conf: error creating file "/sysroot/etc/security/limits.conf": A file exists there already and overwrite is false

I can fix it by changing to contents from append and setting overwrite
    - path: /etc/security/limits.conf
      mode: 0644
      overwrite: true
      contents:
        inline: |
            root soft nofile 1048576
            root hard nofile 1048576

Impact
I have work arounds for everything that was failing however it required changes to the build pipelines that were working previously
Environment and steps to reproduce

Set-up: Start up a new EC2 instance using release 3602.2.0
Task: Create a new EC2 instance with some custom settings involving the overwriting of config files
Action(s):
a. Start up ec2 instance using custom configuration

variant: flatcar
version: 1.0.0
storage:
  files:
    - path: /etc/security/limits.conf
      mode: 0644
      append:
        - inline: |
            root soft nofile 1048576
            root hard nofile 1048576

Error: ignition-files service is unable to write the custom configurations and the boot process crashes

CRITICAL : Ignition failed: failed to create files: failed to create files: error creating /sysroot/etc/security/limits.conf: error creating file "/sysroot/etc/security/limits.conf": A file exists there already and overwrite is false

Expected behavior
With previous version 3510.2.8 this applied the custom configurations
Additional information
Please add any information here that does not fit the above format.

Thank you for the report. This will have been caused by switching to using an overlay for /etc, which provides distro defaults for config files. The overlay is created before ignition runs (right @pothos?) instead of config files being created via systemd-tmpfiles after ignition runs.
I think the current behavior is what we want to have though, with users needing to explicitly specify overwrite: true. I do want to apologize for breaking your pipeline. This is a tricky area for us, it is challenging to preserve compatibility with possible user modifications to config files and at the same time shipping updated default configs for users that dont have modifications.

Thanks @jepio
My only concern is that overwrite:true  doesn't allow for append mode?

Is that only the case for /etc/security/limits.conf? The issue could be that that file is a symlink by default, and this may actually be a bug now that we're using an overlay.

I only use append mode for /etc/security/limits.conf so i'm not sure sorry
I had an issue with etc/flatcar/update.conf as well but that was just an inline which I could set overwrite: true on


Not compatible with Output Biomass or Output Biomass Reclass

Error message pasted below:
2017-04-27 11:25:56,812 -    Reading initial communities map "./initial-communities.gis" ...
2017-04-27 11:25:57,119 -    Opening a NECN-H log file "NECN-H-prob-establish-log.csv" ...
2017-04-27 11:25:57,123 - Loading Output Biomass Reclass extension ...
2017-04-27 11:25:57,139 - The data type of site variable "Succession.BiomassCohorts" is Landis.Library.BiomassCohorts.ISiteCohorts, not Landis.Library.BiomassCohorts.ISiteCohorts

Which versions exactly?  Which installers?

The installer that's on the PSU Shares server at the user folder level.  Not on line to see version # sorry. Then used latest code on repo and compiled new DLL.

Alec, did you copy the deploy directory over from the 'no-ecoregions' branch?  Melissa, when you say, 'the installer' - which installer for which extension?  Latest code for which extension?

I did not copy the deploy folder over yet, its on the list for tomorrow. You gave me an installer for NECN H in January that Melissa used this morning to install, that still resides on the I: drive.
Sent from my iPhone

On Apr 27, 2017, at 5:11 PM, Robert Scheller<EMAIL_ADDRESS>wrote:
Alec, did you copy the deploy directory over from the 'no-ecoregions' branch? Melissa, when you say, 'the installer' - which installer for which extension? Latest code for which extension?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

Ah, that is definitely outdated.  Once we get deploy in place, should be easy.

Ok sounds good. I'll copy it over first thing tomorrow
Sent from my iPhone

On Apr 27, 2017, at 5:34 PM, Robert Scheller<EMAIL_ADDRESS>wrote:
Ah, that is definitely outdated. Once we get deploy in place, should be easy.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or mute the thread.

The new installer for NECN_Hydro works now, but it still doesn't work with output biomass.
Error message below.
2017-05-09 12:57:38,729 - Loading Output Biomass extension ...
2017-05-09 12:57:38,773 -    Opening species biomass log file "spp-biomass-log.csv" ...
2017-05-09 12:57:38,804 - The data type of site variable "Succession.BiomassCohorts" is Landis.Library.BiomassCohorts.ISiteCohorts, not Landis.Library.BiomassCohorts.ISiteCohorts

Make sure you get the latest version of Output Biomass from Bruce (or GitHub).

I updated to the latest version of Biomass and Reclass from GitHub and lo and behold.. they both work!


no field messageSet error while getting task from queue

this happens in a cutout to png images task
-----------end pipeline----------------
err = ErrorException("type SQSError has no field messageSet")
typeof(err) = ErrorException
ERROR: LoadError: type SQSError has no field messageSet
 in wait_ref(::Base.RRID, ::Int64) at ./multi.jl:1138
 in call_on_owner(::Function, ::Future, ::Int64, ::Vararg{Int64,N}) at ./multi.jl:1128
 in wait at ./multi.jl:1145 [inlined]
 in #remotecall_wait#610(::Array{Any,1}, ::Function, ::Function, ::Base.LocalProcess, ::Dict{Symbol,Any}, ::Vararg{Dict
{Symbol,Any},N}) at ./multi.jl:1083
 in remotecall_wait(::Function, ::Base.LocalProcess, ::Dict{Symbol,Any}, ::Vararg{Dict{Symbol,Any},N}) at ./multi.jl:10
83
 in #remotecall_wait#614(::Array{Any,1}, ::Function, ::Function, ::Int64, ::Dict{Symbol,Any}, ::Vararg{Dict{Symbol,Any}
,N}) at ./multi.jl:1105
 in remotecall_wait(::Function, ::Int64, ::Dict{Symbol,Any}, ::Vararg{Dict{Symbol,Any},N}) at ./multi.jl:1105
 in macro expansion at /root/.julia/v0.5/ChunkFlow/scripts/main.jl:25 [inlined]
 in (::##22#24{ChunkFlow.Execute.#execute,Dict{Symbol,Any}})() at ./task.jl:360

...and 3 other exceptions.

 in sync_end() at ./task.jl:311
 in macro expansion; at ./task.jl:327 [inlined]
 in anonymous at ./<missing>:?
 in include_from_node1(::String) at ./loading.jl:488
 in process_options(::Base.JLOptions) at ./client.jl:262
 in _start() at ./client.jl:318
while loading /root/.julia/v0.5/ChunkFlow/scripts/main.jl, in expression starting on line 19

have not seen such error later, should be fixed by adding retry.


/admin-preview route doesn't work?

Hi,
Re the page preview link (underneath the module title when editing a module):
If a module is set as draft, the URL is as follows:
mydomain.com/admin-preview/project/myproject
The /admin-preview route doesn't seem to work, and on my frontend it goes to 404. (When the module is set to live, the preview URL changes to mydomain.com/project/myproject and it works fine).
What mistake am I making here?
I'm now getting around it by redirecting via a front-end route.
~BTW I managed to get around this by overriding getPermalinkBaseUrl() in my admin module controller (removing the line that adds the /admin-preview route), but I don't feel very comfortable doing that!~
Thanks,
Seb

Hi @sebastianlenton,
This behavior should be disabled by default, and enabled only through configuration.
Right now it also seems to be attached only to models using revisions, which is incorrect. We will release a new version of Twill this week and a fix for this will be pushed right after.

Thanks @ferpetrelli!

Has this actually been fixed? This issue still seems to be present in the latest twill

There is nothing to fix there @Tofandel. We can certainly make it configurable, but this is meant to be a way for developers to set up an authenticated route in their frontend to preview draft records outside of the CMS, eg. for external users. Developers are free to set up something like basic HTTP authentication or any sort of auth for that route in the frontend, and of course that's out of Twill's scope.
I agree it lacks documentation, is opinionated, and could be improved, don't get me wrong, but I wanted to provide some context.

There is other ways to achieve this, it could be done with a query parameter and a signed route for example, then in the published scope we can ignore it if the conditions are met, this would put this feature entirely in twill, without a custom route and without burden to the user while still being shareable


With Textformfield the cursor cannot move at the middle of a word.

Hi! With TextFormField and TextField if tap a word inside of the field the cursor is not moved at the place. It moves only when I type a white space.
Is it voluntary from Flutter, how cannot we change this behavior?
I have been dealing with this behavior on all flutter channels for a long time.

I can change cursor position by tapping on Android. On iOS I can't change it by tapping, but I can drag the cursor, the same and for native android apps.
I cannot change cursor position in web and desktop (tried MacOS desktop)

flutter doctor -v
[✓] Flutter (Channel master, 1.19.0-4.0.pre.47, on Mac OS X 10.15.5 19F101, locale en-GB)
    • Flutter version 1.19.0-4.0.pre.47 at /Users/nevercode/dev/flutter
    • Framework revision 28138ec5e3 (5 hours ago), 2020-06-04 00:33:01 -0400
    • Engine revision 2608f2ee9f
    • Dart version 2.9.0 (build 2.9.0-13.0.dev 54adfeb93f)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /Users/nevercode/Library/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 11.5, Build version 11E608c
    • CocoaPods version 1.9.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 3.6)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b4-5784211)

[✓] VS Code (version 1.45.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.11.0

[✓] Connected device (4 available)
    • Redmi Note 7 • 6345c469   • android-arm64  • Android 9 (API 28)
    • macOS        • macOS      • darwin-x64     • Mac OS X 10.15.5 19F101
    • Web Server   • web-server • web-javascript • Flutter Tools
    • Chrome       • chrome     • web-javascript • Google Chrome 83.0.4103.61

• No issues found!

@GENL do you mean web?

No not web. This behavior happens on Android devices.

@VladyslavBondarenko your answer helped me. When I read you I asked myself if the problem wasn't by my side. So I looked up through my app and I found out in the ThemeData of the MaterialApp, I had set platform: Platform.ios instead of Platform.android.
Everything is ok now. Thanks

@GENL
I'll reopen it in order to keep track the issue about web and desktop

@mdebbar
I left the issue open not because of iOS, but because of web and desktop (tried with MacOS desktop), where I can't move cursor to the middle of words as well.
Tried with current beta 1.19.0-4.1.pre, current dev 1.19.0-5.0.pre, current master 1.19.0-6.0.pre.57.

code sample
import 'package:flutter/material.dart';

void main() => runApp(MaterialApp(home: MyApp()));

class MyApp extends StatefulWidget {
  @override
  _MyAppState createState() => _MyAppState();
}

class _MyAppState extends State<MyApp> {
  @override
  Widget build(BuildContext context) {
    return SafeArea(
      child: Scaffold(
        body: Center(
          child: TextField(),
        ),
      ),
    );
  }
}

I have the same issue. I'm on macOS and I can't move the cursor to the middle of the word by my mouse(It's flutter web).
But I have tested it on the different platforms and noticed that flutter web when it's running on some other OS(for example Windows) has the behavior of an android app where I'm able to put the cursor in the middle of the word. And macOS have the behavior of the ios app.  I think flutter web inputs should have a unique solution for this, the user should be able to edit his input by clicking on the typo place by mouse. Currently, there is a possibility for macOS users to edit input word by placing cursor with arrow buttons.

Originally (about a month-45 days ago) we didn't had this issue.

Hi @GENL @VladyslavBondarenko
I cannot reproduce the issue anymore on latest master channel.

Closing as probably fixed, if you disagree please feel free to write a comment providing more details bellow and I will reopen it.
Thank you
flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.117, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.117 at /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision 15c009b805 (10 hours ago), 2020-12-22 20:09:03 -0500
    • Engine revision b7cd3f8e70
    • Dart version 2.12.0 (build 2.12.0-173.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    ! CocoaPods 1.9.3 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.1.5
    • Dart plugin version 203.5981.152

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 87.0.4280.88

! Doctor found issues in 1 category.
Process finished with exit code 0

Because I thought the bug was here again
==> Normal behavior on iOS
For "Hello world"

If you only tap, the cursor is placed at the beginning or at the end of one of your words
If you "tap & hold" you can go inside the word (and at the same time, "Paste | Select All" becomes visible)
It's ok for me (iOS 14.5.1 / flutter stable channel 2.2.0)

is this normal on ios, tap and hold only go inside the word, cause normally you should just tap the word and then should go inside right? I think this is a bug right?

This is really a major bugs, most of the ios app they can go inside the text only flutter cannot this is really not user friendly


Websockets support

Requires either of #29, #9 first.
Client side support for live endpoints with content-diffs.
Support for notifications of updating live endpoints.

Along the lines of h11 and h2: https://github.com/python-hyper/wsproto

What would be the short term solution to this feature?

What is currently blocking this? Or what is required? Both #9 and #29 are closed.

It seems that uvicorn now accepts websockets response, would be nice to let apistar handle websocket too.

Closing this off given that 0.6 is moving to a framework-agnostic suite of API tools, and will no longer include the server. See https://discuss.apistar.org/t/api-star-as-a-framework-independant-tool/614 and #624.


Add buttons to widget docs

I'm adding buttons at the top of widget docs which take users to the component's Marketplace page. The button pops a bit more than a link, and can let our users navigate to Marketplace faster.
If I have any shortcode issues, I'll work them out with @NatasaKralj when she's back from sick leave.

@ConnorLand , is this PR still relevant?


feat: print resolved address for localhost

BREAKING CHANGE: server.printUrls is now async.
Description
#8543 used verbatim: true to make localhost resolved by DNS.
But it was removed by #8642 for compatibility reasons with node ecosystem.
This PR changes the hostname printed when server starts.
The resolved address is used when the node.js resolver differs from DNS.
The printed url differs from the config so it looks a bit weird.
But this can be avoided by calling dns.setDefaultResultOrder('verbatim') in vite.config.js.
This will fix the hijacking issue which #8543 tried to solve.
Why does this solve them?
before
Vite: <IP_ADDRESS>:3000
Other server: [::]:3000
Log output: localhost:3000 => [::1]:3000 Mismatch
#8543
Vite: localhost:3000 => [::1]:3000
Other server: [::]:3000
Log output: localhost:3000 => [::1]:3000
This PR
Vite: localhost:3000 => [::1]:3000
Other server: [::]:3000
Log output: [::1]:3000 (if DNS resolves to [::1], localhost:3000 will be printed)
Additional context
I'll update the docs in #8634.

What is the purpose of this pull request?

[ ] Bug fix
[x] New Feature
[ ] Documentation update
[ ] Other

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

[x] Read the Contributing Guidelines.
[x] Read the Pull Request Guidelines and follow the Commit Convention.
[x] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[x] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

TODO: update migration.md


Request body without required = true should not have the @NotNull annotation

Given a [request body(https://spec.openapis.org/oas/v3.0.3#request-body-object) with "required": false:
   "paths": {
      "/request-body/optional": {
         "post": {
            "requestBody": {
               ...
               "required": false
            },
            ...
         }
      }

when generating JAX-RS code with OpenApi2JaxRs, method parameters representing the request body have the @NotNull annotation, as if they were required:
  @Path("/optional")
  @POST
  @Consumes("application/json")
  void optionalRequestBody(@NotNull Content data);

I believe the @NotNull annotation should only be present when the request body has the "required": true property.

I don't know whether the code generator should add a @Nullable annotation when required is false (or missing), or no annotation at all.


Ensure every axiom has provenance

See manual for background
Every axiom (logical and non-logical) should have provenance. We do this using source properties for now.
Particularly high priority for logical axioms (SubClassOf)

This would involve querying for all the subclassOf relations that don't have a source annotation.
If the provenance is not there though, I am not sure where to find it?

This will be sorted by the Mondo ingest pipeline.

Slurping will go through source and remove all the existing source annotations/provenance
source annotations will be added to subclassOf axioms

need to figure out - for synonyms, will add EXCLUDE if it has been removed by the source

for subclassOf axioms - for the source annotation, if it is no longer supported, we will add some sort of tag


Update compatibility matrix in readme

Issue
NA
Description
Update compatibility matrix in readme

Gravitee.io Automatic Deployment
🚀 A prerelease version of this package has been published on Gravitee's private artifactory, you can:

use it directly by updating your project with version: 4.2.1-gaetanmaisse-patch-1-SNAPSHOT
download it from Artifactory here

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

4.2.2
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.2.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add images: Android Basics with Compose

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-add-images?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-1-pathway-3%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-add-images#1
In which task and step of the codelab can this issue be found?
2
Describe the problem
The link  to download the image returns 404 from Github
Steps to reproduce?

click on the " link "
new tab opens with github
page shows 404

Versions
N/A
Additional information

Hi @AlexRantos
Thank you for writing in with this issue.
The lesson has been updated to address it and has recently been republished.
Hope you are enjoying the course!
Regards.


Deterministic cudnn algorithms

Hi all,
I see other frameworks such as pytorch have some actions to have a flag to force cudnn to only use deterministic algorithm, specially for convolution and maxpooling:
like in pytorch:
example
Some references to cudnn:
maxpooling
cudnnConvolutionBackwardData
cudnnConvolutionBwdFilterAlgo_t
I am doing a medical product and reproducibility is an issue. Therefore, this feature will be very important for me.
One solution is to have a flag like MXNET_CUDNN_AUTOTUNE_DEFAULT.
Anyone is doing it? I have reviewed the current code and want to do it.
Any suggestion or better solution?
Thanks

Thank you for suggesting this @wenyangchu , we will look into this, @sandeep-krishnamurthy , requesting to label this under CUDA.

I prototyped this functionality last summer but did not PR it.  I found at the time that the regression suite would not run if only deterministic algorithms were permitted.  I'm not sure if the situation has improved in terms of universal coverage.  Some questions to think about:

Assuming an environment variable, should it be MXNET_PREFER_DETERMINISM with a warning issued if no deterministic implementation exists, or MXNET_REQUIRE_DETERMINISM with a fatal error if no deterministic implementation exists?
Should the directive be only for cudnn, or for the entire platform?
Should the environment variable be setting a default, with new parameters added to the operators for individual operator instance control?
What should the priority of determinism be relative to other constraints like "choose fastest algo" or "limit workspace"?  [Determinsim should win out I would think]

Hi @DickJC123 ,
Thanks for your reply,
I did an implementation this week due to my urgent need. I just put it into a pullrequest meant for discussion for now:
@https://github.com/apache/incubator-mxnet/pull/11361
Please check the last 2 commits.
For your questions:

If MXNET_PREFER_DETERMINISM is set and it can not find a deterministic algorithm, I suppose it has to have a fatal error because user's need is not to be able to be satisfied.

I think it is a good idea to have it for the entire platform but I will try to solve it with cudnn first because it is the most used one I suppose? I do not see other obvious issue in other backends yet maybe anyone else can suggest where can be not deterministic?

I have tested CPU version with intel MKL in a limited scenarios and it was deterministic for training. I think we need advice and tests to figure out which part of any other backends is not deterministic.

I think it is good to have a global determinism control if feasible. If it is possible to have control over individual layers, I think it is also very good to have.

In the pullrequest I added deterministic parameter (default = False) to Maxpooling:
nn.MaxPool2D(pool_size=(3,3), strides=(2,2) ,deterministic=True)
Added env parameters to select Deterministic algorithms for Conv back propagation algorithm
os.environ["MXNET_CUDNN_AUTOTUNE_DEFAULT"] = "3"
Old:
#Value of 1 chooses the best algo in a limited workspace
#Value of 2 chooses the fastest algo whose memory requirements may be larger than the default workspace threshold
Added:
#Value of 3 choose the deterministic best algo in a limited workspace
#Value of 4 chooses the deterministic fastest algo whose memory requirements may be larger than the default workspace threshold
They could be replaced by a global deterministic flag.

As you see above, I actually think it is good to let user to select deterministic algorithm according to constraints: speed or memory size.

The problem of this solution is that, if cudnn chooses different deterministic algos, it can fail repeatability. I think it is good to have another mechanism to let user to select cudnn algorithm directly if available.

Hi @DickJC123 , I have little knowledge on the regression test in mxnet. Could you please let me know how you ran the test?

I often use tests/jenkins/run_test_ubuntu.sh to compile MXNet and run the regression tests.  You may need to set DEV=0 in that script to get past compile warnings treated as errors.

Hello @wenyangchu @DickJC123
Can you please let me know next steps here? How you think we can take it forward?

+1. Sometimes we need to be able to reproduce results even at the cost of worse performance. Is there a plan or next step? Thanks.

@wenyangchu Since I have not got the response from you, I took the liberty to implement this feature in MXNet due to other urgent requests. Thank you very much for your investigation and suggestion. You are more than welcome to review my PR and provide other valuable suggestions.

@apeforest Thanks for taking action to make it happen. Sorry the notification was sent to my gmail and filtered.


⚠ Bofrost.lu has degraded performance

In c4161f9, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1803 ms

Resolved: Bofrost.lu performance has improved in 8c82420 after 24 minutes.


Prevent failures in the Annotation code if the Rect array contains indirect objects (issue 7115)

Note that in the PDF files provided by the reporter, this issue was limited to Rect arrays in AcroForm entries (which we currently don't support).
However, since a bad PDF generator could create this problem in any kind of annotation, the reduced test-case included here uses a simple LinkAnnotation instead.
Fixes #7115.
A note for the reviewer: If you need a copy of the original PDF files for verification purposes, please let me know!

/botio-linux preview

/botio test

/botio unittest

/botio makeref

Thanks!


FileParseException does not log the actual exception

This is a sub-issue for issue #150.
FileParseException in utils.py does not print more than bad request in the console.
Maybe I am missing some development instructions, and it need to be added to README.

sorry my bad


Seem to be some missing with this fork

I am upgrading my previous app from 4.6.2 to core2, Thus changing over to this CommandLineParser from Gio's as the previous didn't seem to support Core. Now i have an issue of specifics could not be found
`      #region Properties
  [ParserState]
  public IParserState LastParserState { get; set; }

  #endregion

  #region Functions

  [HelpOption]
  public string GetUsage()
  {
        //Output usage plus any errors if found.
        return HelpText.AutoBuild(this, current => HelpText.DefaultParsingErrorsHandler(this, current));
    }

    #endregion
}

}`
Any help or documentation direction would be great.
Thankyou

Hi Jason,
perhaps same problem like https://github.com/commandlineparser/commandline/issues/182
Regards
Oliver


Player health bars are drawn behind enemies

Health bars should be drawn on a layer above all players and enemies. They need to be drawn after all players and enemies are drawn. Make my_game draw the health bars instead of the draw() method in the Player.

dealt with in pr #71


Search the scenarios and show a list of matching scenarios

As a testmanager I would like to search for scenarios having a certain (combination of) tags so that I can view a list of those scenarios
e.g. a list of all scenario's tagged with @manual and @regressie
In search results, maintaining the grouping into solutions would be nice

A small update on this issue:
I have decided to switch the storage from SQL Server to RavenDB. RavenDB allows for indexing the Json formatted data by multiple tags, something which comes in quite handy. The additional upside is that it can run embedded, so there is no need to install and maintain a separate database. (Licensing has been taken care of, as Hibernating Rhinos has been kind enough to provide an free license for this open source project.)
With the holidays having come to an end, I expect the switching of the database will take a couple of days. After which the search functionality should be easy to implement.

Klinkt goed!
Vriendelijke groeten
Harry Nieboer
Op 4 jan. 2015 om 13:43 heeft marktaling <notifications<EMAIL_ADDRESS>het volgende geschreven:
A small update on this issue:
I have decided to switch the storage from SQL Server to RavenDB. RavenDB allows for indexing the Json formatted data by multiple tags, something which comes in quite handy. The additional upside is that it can run embedded, so there is no need to install and maintain a separate database. (Licensing has been taken care of, as Hibernating Rhinos has been kind enough to provide an free license for this open source project.)
With the holidays having come to an end, I expect the switching of the database will take a couple of days. After which the search functionality should be easy to implement.

Reply to this email directly or view it on GitHubhttps://github.com/marktaling/Augurk/issues/5#issuecomment-68631581.
Op dit e-mailbericht is de disclaimer van Info Support van toepassing, zie http://www.infosupport.com/disclaimer


Revert "Update travis tests for new openmm-forcefields package to fix failing travis tests"

Reverts choderalab/perses#623

@hannahbrucemacdonald: I must have missed the original issue somehow: Why was the revert needed, and if I continue to fix these tests in a separate PR, should I base that PR on master or another branch?

I accidentally merged the changes into master, so I reverted master so that it could be merged properly when it’s ready - sorry

Totally fine, thanks!