Re-direct Swarm's log to a file instead of stdout

One requirement of mine is to re-direct Swarm's log to a file instead of stdout; ">" is not the option as I'd like to run swarm in docker and retrieve the log by file. If this feature is here, I'd suggest to update it in Dockerfile

Just throwing this out there. Given the work put into the implementation in the docker-engine (multiple backends, etc), do you think it's feasible to turn that into a pkg and reuse it?

Just throwing this out there. Given the work put into the implementation in the docker-engine (multiple backends, etc), do you think it's feasible to turn that into a pkg and reuse it?

I think it's each component's work to log to a file; for example, zookeeper/marathon are using log4j which can be configured to append log message to a file.
But, I not sure this issue is to Swarm or log tool.

That should be easy to implement - we are using http://github.com/Sirupsen/logrus which supports many logging backends, including file.

@aluzzardi , thanks for your feedback. That'll be great if we pass conf to logrus in Swarm :).

@klaus1982 Do you have anyhave  interest in implementing this feature. :)

@jimmyxian , sure, my pleasure :). Any mentor/shepherd to guide me on this one?

@klaus1982 Cool.

First: setup a swarm development environment(https://github.com/docker/swarm/blob/master/README.md)
Second: contribute by this guide(http://docs.docker.com/opensource/code/)

Also, you can join our IRC #docker-swarm on freenode. Any questiones you can ping me there.

Some pointers:
cli.go is the main entry point for the CLI and this is where we handle global logging options (currently --log-level) and set up the logger:
https://github.com/docker/swarm/blob/master/cli/cli.go
What you probably want to do is:

Add a new flag, such as --log-file
When setting up the logger, open or create the log file and give the handle to log.SetOutput (https://godoc.org/github.com/Sirupsen/logrus#SetOutput)

Although as @MHBauer pointed out, Docker Engine recently added log drivers and perhaps we should use that instead.
/cc @docker/swarm-maintainers

Yeah, using log pkg from Docker Engine is another good choose. Prevent to Reinventing the wheel
https://github.com/docker/docker/tree/master/daemon/logger

Unfortunately it's not a pkg/ so it's private to the engine :/

Aha. Yeah...Not a pkg. :(

sure :). I'll work on this. Will you assign this to me?

Unfortunately it's not a pkg/ so it's private to the engine :/

@aluzzardi have you considered opening an issue in docker/docker to propose making it a pkg? Note; there's also this proposal around logging https://github.com/docker/docker/issues/18604

I would be happy to see Engine logging capabilities become more of a reusable pkg (cc @calavera).

Closing due to lack of activity. Please reopen if you wish to continue discussing it.


Code coverage doesn't work on files linked with "src="

I try to add code coverage to my project, but I use src= and it doesn't seem to be supported.
Maybe it's related to sourcemaps missing (https://github.com/eddyerburgh/vue-jest/pull/29#issuecomment-347252861).

Yeah, sorry. Source maps are missing at the moment. I'll add them soon

This was fixed in #131


Please invite me to the community

Name
Oguz Karademir
Additional Context
I would like to be in this community.
Kind regards

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is an open source solution for serverless platforms, it is designed for the development solutions and enhance creators to create their infrastructure 💻 Great to have you in our community ✨ Looking forward to your valuable contributions! 🤓 You can also check our community ✨ Github ✨ LinkedIn 💜 You can also share our community with other folks who want to learn development, DevOps, tech, and contribute together and with us 👉💜👈


fix(curriculum): feedback on Learn Css Variables By Building A City S…

…kyline instruction updated
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #47555

Hey @Sembauke what do you mean by fcc style?
Can you share some examples for reference

Can you please use a descriptive title? the title of the issue is not a descriptive title for the PR

@ieahleen can pls confirm, is it appropriate ?

And to add to that, it was decided to not divide in three steps, but to make the stepp less 'handolding', with the wording mentioned in https://github.com/freeCodeCamp/freeCodeCamp/issues/47555#issuecomment-1275090120

@ieahleen I've updated the pr. Do let me know if I missed anything.

Please follow the decision in the issue, do not do random changes

tbh I copied the changes from here

That was the breakdown for splitting the challenge in three different steps, as it's not what it was decided to do, you don't need to take that wording


aql query string has a syntax error, but I copy / pasted it from docs

This code:
db.query(aql`
    FOR i IN 1..10
        INSERT {
            LaTeX: CONCAT("A_", i),
            _key: i
        } IN mynodes-js`
).then(
    cursor => {
        cursor.all().then(
            vals => {
                // vals is an array containing the entire query result
                console.error(vals);
            });
    });

Is giving me:
node:internal/process/promises:225
          triggerUncaughtException(err, true /* fromPromise */);
          ^
ArangoError: AQL: syntax error, unexpected end of query string near 'js' at position 6:22 (while parsing)
    at new ArangoError (c:\Users\FruitfulApproach\Desktop\Quiver\quiver\node_modules\arangojs\error.js:115:21)
    at Object.resolve (c:\Users\FruitfulApproach\Desktop\Quiver\quiver\node_modules\arangojs\connection.js:406:32)
    at callback (c:\Users\FruitfulApproach\Desktop\Quiver\quiver\node_modules\arangojs\connection.js:191:26)
    at IncomingMessage.<a....

which doesn't make any sense.

    FOR i IN 1..10
        INSERT {
            LaTeX: CONCAT("A_", i),
        } IN mynodesjs`

This works; it didn't like the hyphen.

    FOR i IN 1..10
        INSERT {
            LaTeX: CONCAT("A_", i),
        } IN mynodesjs`

This works; it didn't like the hyphen.


Refactor dask tests

Pull Request Description
Refactor dask tests to not reuse a client across tests. Also use the "standard" pytest workflow of writing functions instead of classes. This was inspired by how lightgbm tests dask.
https://github.com/microsoft/LightGBM/pull/4159/files
I pushed a commit every three hours over the weekend and the tests always passed. Note the following:

A lot of the red x's are because the codecov/patch check  did not pass (I believe this is a "bug" stemming from the fact there was no diff over repeated commits. The issue fixed itself though).

This does not fix this issue https://github.com/alteryx/evalml/issues/2341, and it's why the windows unit tests failed for commit 5d57ebb. However, dask_sends_woodwork_schema did not fail.

I still do not have a 100% confident explanation of what was causing the original error. But I don't think this was a gh problem after all since I was finally able to repro the timeout locally.
My hand-wavy explanation is that the dask scheduler somehow is losing track of the data we've scattered. Since the client is the scheduler and we're reusing the same client across the test, it made me think that some global state was being corrupted. I saw that lightgbm and xgboost do not reuse clients across tests and it gave me the idea to try it here.
I filed an issue with dask/distributed as well https://github.com/dask/distributed/issues/4907

After creating the pull request: in order to pass the release_notes_updated check you will need to update the "Future Release" section of docs/source/release_notes.rst to include this pull request by adding :pr:123.

We should probably standby for a little bit to see whether turning off the broadcast option helps, like in PR #2376.  I've been tracking some of the speculation and resolution on the source issue, #2354 .  The dask issue that led me to try turning off broadcast was here.  I'd be hesitant to move away from the single client for testing only because it should work that way, haha.


feat(ingest): add support for Looker view built from SQL-based derived tables

This PR adds support for ingesting Looker views built from SQL-based derived tables (https://docs.looker.com/data-modeling/learning-lookml/derived-tables).
Tested with success on our Looker instance, with the lineage of Looker views built from SQL-based derived tables showing up in DataHub.
Also updates the lkml dependency to better parse .lkml files.
Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable)

@frsann the answer to your question seems to be yes: https://docs.looker.com/reference/view-params/derived_table#supported_database_dialects_for_derived_tables
I also found an issue with sql-metadata that I reported upstream: https://github.com/macbre/sql-metadata/issues/128

Considering that the sql parsing might not be water tight, should we put this behind feature flag?

Considering that the sql parsing might not be water tight, should we put this behind feature flag?

Agree here -- seems like the sql-metadata libs only supports a few dialects, whereas Looker appears to support many more?
Have we considered whether something like sqlparse is suitable?

I'm going to merge this in first. We can improve sql parsing in a follow up PR. Want to first prioritize the moving of this to the ingestion framework.

@jjoyce0510 sql-metadata actually uses sqlparse under the hood. Agree that we should have put this behind a feature flag since it's not yet fully baked.

@hsheth2 @jjoyce0510 I am trying to get some sql-metadata bugs fixed upstream. I'll open another PR here once a new version of sql-metadata is out.


Added support to use dot Notation

I used this technique to create a summary object of an other object.
It enables you to use @Field(name="prop.key") and the value will be filled in.

In order to merge this we need a test to understand what is being added/changed. Please reopen if you want to continue working on this.


docs: fix 'Tavily Search' import Error in tavily_search.ipynb

fix #25256

Thanks for this. This was fixed in https://github.com/langchain-ai/langchain/pull/24736 and will be released in the next version of langchain-community.
After release the existing import in the docs will work as TavilySearchResults is properly exported from tools.__init__: https://github.com/langchain-ai/langchain/blob/a82c0533f206548e0ebbdb1a8603e0d9a2eac370/libs/community/langchain_community/tools/init.py#L470


Fix initial collision pairs being cleared (2.1)

This was affecting both 2D & 3D physics and was a regression introduced by my physics mega-fix (#8999).
Fixes #9387.
By the way, apologies for the bug and thank you for the patience as this was delaying the new release.

For the good beginning I can confirm this fixed issue in my project (which I was unable to isolate in any way) :+1:


Exoplayer workload fails to run.

Describe the issue
When running the exoplayer workload with default settings it fails with:
TypeError(a bytes-like object is required, not 'str')
Run Log
run.log

Fixed with: https://github.com/ARM-software/devlib/pull/495


Add MarkDown formatting to examples/neural_doodle.py

Summary
Add MarkDown formatting to examples/neural_doodle.py.
Result

Related Issues
#12219
PR Overview

[ ] This PR requires new unit tests [y/n] (make sure tests are included)
[ ] This PR requires to update the documentation [y/n] (make sure the docs are up-to-date)
[x] This PR is backwards compatible [y/n]
[ ] This PR changes the current API [y/n] (all API changes need to be approved by fchollet)

Thanks for the PR.
We are postponing the addition of new examples on keras.io pending a future refactor of the docs website (which will have a dedicated examples section).


Added text2png for Strong Captcha

For Issue #784
Work in progress. Need suggestion @bkimminich

Following npm package has been used.
https://www.npmjs.com/package/text2png

Hi! We didn't want to replace the CAPTCHA in "Contact Us" with something stronger, but add a separate stronger CAPTCHA to another functionality, like "Recycling".


Fix broker.xml connector and connector-ref when using federation

Fix #177

check for the message Connection configuration is null for connectorName in the logs and I think you'll see this fail.

Connection configuration is null for connectorName

It was surrounded with double quotes, I committed the change removing them; also added a task to enforce  presence. Please give a look

Tested PR and works as expected

Tested PR and works as expected

Brilliant, thanks for all the help.


[pkg/otlp/metrics] Add support for metrics origin

What does this PR do?

Adds support for setting metrics origin on metrics based on the scope.

@mackjmr requesting your review since you reviewed #204, but this one is less urgent IMO than #231

A dedicated test in metrics_translator_test.go with something like:
	assert.ElementsMatch(t,
		consumer.metrics,
		[]metric{
			newCount(exampleDimsWithOrigin, uint64(seconds(1)), 30),
		},
	)

would be nice to have here, as this goes through the whole path.
Other than that LGTM!

All the tests using the files in pkg/otlp/metrics/testdata are already testing this. I don't have one specific for a metrics product, maybe I can add one of those?

All the tests using the files in pkg/otlp/metrics/testdata are already testing this. I don't have one specific for a metrics product, maybe I can add one of those?

Yeah, I think this would be good. Also, not sure if it is possible, but if we can set the scope in one of the tests it would be nice to include a known origin service in there.

Added a specific test for this, auto-merging....


Update packages: postcss, autoprefixer, csswring

Autoprefixer-core is depricated and autoprefixer is recommended as a replacement. This change required update of grunt-postcss and csswring versions to eliminate all errors at grunt tasks.

Current coverage is 48.25%

Merging #380 into master will not affect coverage as of d8d245c

@@            master    #380   diff @@
======================================
  Files           85      85
  Stmts         3384    3384
  Branches       320     320
  Methods          0       0
======================================
  Hit           1633    1633
  Partial         29      29
  Missed        1722    1722

Review entire Coverage Diff as of d8d245c

Powered by Codecov. Updated on successful CI builds.


reports is taking very long time

Hi, i want to know why lately the reports is taking long time to run.
there are reports for clients that finishes after 1-2 hours and there is reports that finishes after more than 5 hours.
why this is happening? did you have a long time reporting queues couple days ago ?
how much time i need to wait for a report to be created ? (currently i am waiting 5 hours but this seems to be not enough)

Can you provide the report-definition JSON object that you're using? That will help us determine some possible issues around why it's taking so long.


Remove deprecated component.json

Same as https://github.com/jashkenas/backbone/pull/3917
component.json has been deprecated since June
So there's no reason to keep maintaining it.
And as it is, @jashkenas does not like it.

Bower next ;D

@jdalton: Are you commenting on bower, or saying bower still uses component.json?

Are you commenting on bower

Commenting on bower. I'm pulling it from lodash v4 in favor of npm instead.

^^ agreed


Shortest Semantic Path

Currently, we don't know which semantic path is taken to get to a result. Ideally, this would be a meta value that one can get from the GQL results.
For example if the query iphone results in the obj Company with the name Apple. It should be able to show the path and distances.
e.g., iphone => macintosh => data obj.

Closing as duplicate of #1136


Inaccurate info - Needs update

under the section "Configure team site libraries to sync automatically" it describes "Copy library ID" which no longer exists in Office 365 platform
please update with accurate info

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

ID: 04ad2f16-c861-7a1b-5c1d-e4601e8f3855
Version Independent ID: c47df4cd-f706-d807-5eac-172fba1e500f
Content: Use Group Policy to control OneDrive sync settings
Content Source: OneDrive/use-group-policy.md
Service: one-drive
GitHub Login: @kaarins
Microsoft Alias: kaarins

@andocromn Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Any update on this? Critical function, and it's gone. How do I generate the ID?

@jeopardize95 I ended up using the incredibly awkward and backassward method described in this article
https://www.blackforce.co.uk/2019/09/12/sharepoint-copy-library-id-missing-for-team-site-libraries-sync
I was like NFW this is going to work! but it actually did.  Microsoft certainly needs to fix this though cause it's a workaround like removing and re-attaching a foot is a solution to foot pain

@andocromn I've tried. But then Intune won't sync the library. Have a case with MS on it, but doesn't seem to lead anywhere. This is ridiculous, this function is noted in Docs, but not working. Works in one tenant I have, but not the other. Hopeless.

@jeopardize95 sorry to hear that.  For what it's worth it does seem to be hit or miss, I just had to manually add it for a user myself.  Maybe I could have banged away at OneDrive and gotten it to work but the doc says something about 8 hours and even a saint doesn't have that kind of patience.  Best of luck

Hi @andocromn and @jeopardize95, thank you for your feedback.
I tested in two different tenants and it works for me:

I encourage you to continue working with support as this is the correct way to handle this problem.
Docs feedback is available to improve documentation and it seems to be an issue in your tenant.
Thanks again!

Hello Dario,
That’s strange. Would you like me to show you? Happens in three of my tenants :-(
Med vennlig hilsen
Andreas Aasheim
Arkitekt | Microsoft 365
Tlf: 98114408
Easify AS

Fra: Dario Woitasen<EMAIL_ADDRESS>Sendt: Tuesday, March 17, 2020 10:16:49 PM
Til: MicrosoftDocs/OfficeDocs-SharePoint<EMAIL_ADDRESS>Kopi: Andreas Aasheim<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Emne: Re: [MicrosoftDocs/OfficeDocs-SharePoint] Inaccurate info - Needs update (#1795)
Hi @andocromnhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fandocromn&data=02|01|andreas%40easify.no|ccab841edcee468bcd1608d7cab882b9|0c8060178696416998d7b89a4499da8a|0|0|637200766137106194&sdata=UvswH9Ymh3fPGexbQU%2BZzJL62iIhAVeulYU5D5IFJc8%3D&reserved=0 and @jeopardize95https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjeopardize95&data=02|01|andreas%40easify.no|ccab841edcee468bcd1608d7cab882b9|0c8060178696416998d7b89a4499da8a|0|0|637200766137106194&sdata=mNPCCwENhMwV65xCdRvbyLrCFlrvfV4qnxbEKV%2FHu40%3D&reserved=0, thank you for your feedback.
I tested in two different tenants and it works for me:
[image]https://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F33589238%2F76902611-2b016800-687b-11ea-9b87-2cbc6092b51a.png&data=02|01|andreas%40easify.no|ccab841edcee468bcd1608d7cab882b9|0c8060178696416998d7b89a4499da8a|0|0|637200766137116187&sdata=F0Z5Muo355Zrw0c8ILvQLJ4akn5ZNfbLcSC8hm42TgU%3D&reserved=0
I encourage you to continue working with support as this is the correct way to handle this problem.
Docs feedback is available to improve documentation and it seems to be an issue in your tenant.
Thanks again!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FOfficeDocs-SharePoint%2Fissues%2F1795%23issuecomment-600305550&data=02|01|andreas%40easify.no|ccab841edcee468bcd1608d7cab882b9|0c8060178696416998d7b89a4499da8a|0|0|637200766137116187&sdata=gbnR9enPA8aSlJR%2FQHsyHdECgWaM1nyyElaOzp%2BK5x4%3D&reserved=0, or unsubscribehttps://eur02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOWSWKZYDEAVJYUKRLCIHA3RH7SEDANCNFSM4K5EOIFQ&data=02|01|andreas%40easify.no|ccab841edcee468bcd1608d7cab882b9|0c8060178696416998d7b89a4499da8a|0|0|637200766137116187&sdata=fai6ZmXlY37ndPzBi8XPL4JfRBMxi5aMKTRaJLVPZOU%3D&reserved=0.

Sorry, I think you should continue working with support as I cannot provide you with this kind of assistance.

Hi @dariomws

Hi @dariomws, can you please provide further detail?  I have followed the instructions, Please see the screenshot showing that the Copy Library Link is not available.  thank you

The steps in the article worked for me. Please open or continue working with support as it's the correct way to get assistance on this issue.
Thanks.

I have followed the instructions in the article in two different tenants and it works for me.
This issue was reported before:
https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/issues/1326
And the screenshot was updated:
https://github.com/MicrosoftDocs/OfficeDocs-SharePoint/pull/1386
There you can find another way to get the library ID.
@kaarins could you kindly add something else here? Thanks!

Hi @dariomws, just to clarify I am not requesting support for this matter, I had the library ID and group policy working before submitting this report.  I am merely trying to help report the inaccuracy in the article so that it may be corrected.  I understand that #1386 describes other means of obtaining the Library ID, but the article in question does not provide the instructions in #1386
For the record I have attempted this in 5 major browsers and I have at least 2 tenants that do not have the "Copy Library ID" I can check more if it will prove the point but its really redundant.  #1326 seems to further indicate that extends beyond just the 2 tenants I recently setup

@andocromn Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.

I can confirm that the "Copy library ID" option appears. But only if you are using an administrator account.

I can confirm it still does not appear, admin or not. Seems to be tenant specific.

Hi @Pentao I have been testing licensed account with global admin permissions.  None of my tenants have the Copy Library ID option.  I suspect it may have to do with the time period in which the tenant or sharepoint site was created

@kaarins could you kindly help? There are people who are still experiencing this issue. Thanks!

I don't pretend to be an expert on SharePoint, but from my experience the site is created from a template... And this would be part of the template, right? So if it was removed from them template, these changes wouldn't apply to pre-existing sites created from the template, only sites created after this change was made to the template

@kaarins could you kindly help? There are people who are still experiencing this issue. Thanks!

@kaarins could you kindly help? There are people who are still experiencing this issue. Thanks!

@andocromn and any others, if you are still experiencing this issue, please contact Support because the team would like to investigate. We don't want to add the PowerShell instructions to this article because the feature should work as described. We will need you to collect and provide a Fiddler trace. When you contact support, please include a link to this issue. Thank you.


FIX: Defer more computation to the progress bar.

The only reason for the status object to be computing fraction, time_remaining, etc. is if it knows about some special case that the progress bar can't know. Otherwise we should just let the progress bar do its naive computation.

Codecov Report

Merging #438 into master will increase coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #438      +/-   ##
==========================================
+ Coverage    90.8%   90.86%   +0.06%
==========================================
  Files          48       48
  Lines        6700     6692       -8
==========================================
- Hits         6084     6081       -3
+ Misses        616      611       -5

Impacted Files
Coverage Δ

ophyd/status.py
93.29% <ø> (+2.59%)
:arrow_up:

Continue to review full report at Codecov.

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


looking to use mui with tailwind css

I'm always frustrated when i tried to use tailwind with naterial-ui
Describe the solution you'd like
i want to use tailwind with material-ui
<Card sx={{ flexDirection: 'row', maxWidth: 545, borderRadius: "20px", spacing: "125px", boxShadow: "7px", alignItems: 'stretch' , hover:bg - grey - 500 }} elevation={4} position="relative" >
Describe alternatives you've considered
need to get into something else eg. styled-components

This is not a Flowbite issue. Are you in the wrong repo?

Hey @cadentic,
You can check out Flowbite React as an alternative to Material UI + Tailwind CSS.


Removed bulb not showing a non-responsive

Issue

Hello Erik!
First I'd like to thank you for creating this amazing plugin! I'm very happy to be able to buy bulbs from different brands and all get them in my HomeKit through the Hue bridge. I've tried to find an answer to my question by reading through the issues, but I found nothing. So here goes.
One thing I've noticed and haven't been able to find an answer to is this: Whenever I remove a bulb from a lamp/socket it still shows in my HomeKit. Homebridge-hue does register the missing the bulb, as seen in the log below. It doesn't change the status in HomeKit tho, where I'm still able to turn the bulb on and off instead of it showing as non-responsive.
Is this normal or there a way for me to fix this? Thanks for reading and again, thanks for creating this amazing plugin!
Log Messages

[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: homekit on changed from 0 to 1
[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: homekit on changed from 1 to 0
[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: set homekit status fault from 0 to 1
[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: homekit on changed from 0 to 1
[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: homekit on changed from 1 to 0
[11/1/2019, 10:11:35 AM] [Hue] Innr lamp 2: homekit on changed from 0 to 1

Debug Files

No need, afaik.

No need, afaik.

The issue is that the Hue bridge doesn't know whether the light has been permanently removed, temporarily powered off, or simply temporarily unreachable because of hiccups in the ZigBee network.  The Hue API eventually reports the light as unreachable, when it doesn't respond to polling requests for its state.  This causes homebridge-hue to set Status Fault (as you can see in your log), but Apple's Home app ignores this (standard!) characteristic.  Other HomeKit apps, like Eve do show it.
I use Status Fault, because, unfortunately, you cannot push the fact that an accessory has become/no longer is unreachable to HomeKit: the Home app only sets this state when it actively queries the device.
By setting "wallswitch": true in config.json, homebridge-hue will force the light off when the bridge reports it as unreachable, under the (often wrong!) assumption that the light has been powered off by the 20th century wall switch.  If you permanently remove a bulb, you better remove it from the Hue bridge (e.g. using the Hue app) and restart homebridge (so homebridge-hue removes it from HomeKit as well).

So as I understand it's impossible to get the unresponsive state from a bulb when someone accidentally turned off the wall switch? Good to know. Thank you for the response.


Patch for issue with lifetime of subject's observers #324

Notice: lock()method of std::weak_ptr is executed atomically

@wutiantong
this passed the concurrency tests, looks great!
I would like to have tests for this leak, your comment in #324 is a great start. Would you like to add them in a separate PR?


Possible bug with PBE attachment code

https://www.elkarte.net/community/index.php?topic=5196.msg36961#msg36961

Have not been able to reproduce the issue .... it remains unconfirmed

This is a duplicate of #3415 ... will close this and track it there.


deps(WIP)!: upgrade to node v0.13.4

This PR gives celestia-openrpc parity with celestia-node. It also modularizes the project, so that users can create clients without needing to report all modules. This is good for integrations where the codebase wants to minimize dependencies and we only need blob module for example.
Want to write some integration tests finally before marking as ready for review

Now blocked on:
https://github.com/celestiaorg/celestia-node/pull/3405

Codecov Report
Attention: Patch coverage is 53.33333% with 28 lines in your changes are missing coverage. Please review.

:exclamation: No coverage uploaded for pull request base (main@ac20f5a). Click here to learn what that means.

Files
Patch %
Lines

builder/client.go
27.27%
16 Missing :warning:

types/blob/blob.go
65.71%
8 Missing and 4 partials :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #117   +/-   ##
=======================================
  Coverage        ?   13.88%
=======================================
  Files           ?       18
  Lines           ?      886
  Branches        ?        0
=======================================
  Hits            ?      123
  Misses          ?      738
  Partials        ?       25

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add item to context menu for report layout

Opening a designer for a report layout (.rdl or .docx) is a somewhat cumbersome task. You need to first open the file folder (using Reveal in Explorer ) to be able to open the layout file in its associated designer.
Increase our productivity and add a context menu item in VSCode to allow us to directly open the file in its designer.

This is a feature request for vscode. I have opened an issue there asking for the feature so you can upvote there if you want them to prioritize it.
https://github.com/Microsoft/vscode/issues/43641

Thanx, Dominik. DONE.

I'll leave this one open so that we have something to track

I have added a way to introduce this feature through our extension. Now you will be able to use:

Indeed I saw it. Great and thanx.


Changed pressure topic so it is the same as the autodiscovery message

If autodiscovery is used, the pressure is not read in Homeassistant, because it is called "press" at the autodiscovery and "pressure" at the MQTT message.

Thanks..
Planning to upgrade mqtt homeassistant auto discovery soon..


Error with disallowParenthesesAroundArrowParam

I updated an application to use<EMAIL_ADDRESS>then set "disallowParenthesesAroundArrowParam" to be true. Running the task causes numerous files in the application to break, though not all. It's likely that it's just the files that use arrow functions.
One stack trace is:
Error running rule disallowParenthesesAroundArrowParam: This is an issue with JSCS and not your codebase.
Please file an issue (with the stack trace below) at: https://github.com/jscs-dev/node-jscs/issues/new
TypeError: Cannot read property 'type' of undefined
    at /Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/rules/disallow-parentheses-around-arrow-param.js:62:59
    at Array.forEach (native)
    at Object.JsFile.iterateNodesByType (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/js-file.js:488:42)
    at Object.module.exports.check (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/rules/disallow-parentheses-around-arrow-param.js:57:14)
    at null.<anonymous> (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/string-checker.js:165:22)
    at Array.forEach (native)
    at StringChecker._checkJsFile (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/string-checker.js:161:31)
    at StringChecker.checkString (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/jscs/lib/string-checker.js:106:14)
    at DestroyableTransform._transform (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/index.js:72:22)
    at DestroyableTransform.Transform._read (/Users/jmeas/path/to/my_project/node_modules/gulp-jscs/node_modules/through2/node_modules/readable-stream/lib/_stream_transform.js:172:10) at pods/schedule-chart/views/chart-view/index.js :
     1 |import $ from 'jquery';
--------^
     2 |import _ from 'underscore';
     3 |import moment from 'moment';

All of the errors happen at line1, no matter what it is (even if it's, say, a comment).

Thanks for the report - I just fixed this in master. Same as https://github.com/jscs-dev/node-jscs/issues/1747

Thanks @hzoo! Sorry for the duplicate issue – I ran some searches and didn't find the original issue.

I'm having this issue also. I look forward to the next version bump! Thanks @hzoo :)


other: update sbt-sonatype from 3.9.5 to 3.9.7

Updates org.xerial.sbt:sbt-sonatype from 3.9.5 to 3.9.7.
GitHub Release Notes - Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

labels: sbt-plugin-update, semver-patch

Codecov Report

Merging #22 (7e2e7a8) into master (60aac93) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              master       #22   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        29        29
===========================================
  Files              1         1
  Lines            114       114
  Branches           8         8
===========================================
  Hits             114       114

Continue to review full report at Codecov.

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


eliminated udf and replaced it by simpler logic that works with test …

…data

Pre-review checklist
I have ensured that:

[x] Code runs successfully in the development environment.
[x] New code is tested to prove that it works. If not, justify reasoning.
[x] New code is documented using type hinting and following the numpydoc docstring format.
[x] New code follows the project naming standards.

Overview of changes

Post-review checklist
As I merge these changes, I have:

[x] Updated the related Jira ticket, including approximate time spent (including review).
[x] Moved the related Jira ticket to Done.

For example using window function instead of groupby see this branch


Fix openapi relative ref

Description
This fixes Internal error during $ref resolve when using relative file paths.
Checklist

[ ] Added failing tests for the change
[ ] All new and existing tests pass
[x] Added changelog entry (follow guidelines in CONTRIBUTING.rst)
[ ] Updated README/documentation, if necessary

Hi! Thank you, could you add a test case for this fix? Or share the schema you have (anonymised will work)

I added a test that fails without this PR. I included one of the paths (user_account.yaml) in my schema along with all it's $refs (parameters.yaml, schemas.yaml, responses.yaml). I can simplify the example further if you'd like but for now I kept all details to make it more "real".

Thank you so much! Yes, lets keep it as is :)


Register decodeDot11 for LinkTypeMetadata

In a (private) project of mine I have this code:
	dot11Layer := packet.Layer(layers.LayerTypeDot11)
	if dot11Layer == nil {
		return
	}

	dot11 := &layers.Dot11{}
	data := dot11Layer.LayerPayload()
	dot11.DecodeFromBytes(data, gopacket.NilDecodeFeedback)

which failed to decode properly: Packet decoding error: Unable to decode link type 105. This change fixes that issue.
If the issue is a result of a mistake on my side and this commit is the wrong way around it, please do tell me.

I signed it!


[Bug]: czkawka-gui is not updated to 7.0.0 due to archive name change

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues/PRs to ensure it has not already been reported or fixed.
[X] I have verified that I am using the latest version of Scoop and corresponding bucket.

Package Name
czkawka-gui
Expected/Current Behaviour
czkawka-gui is not updated to 7.0.0 due to archive name change, windows_czkawka_gui.zip ->  windows_czkawka_gui_gtk_410.zip
Steps to Reproduce
❯ scoop install czkawka-gui
Updating Scoop...
Scoop was updated successfully!
WARN  'czkawka-gui' (6.1.0) is already installed.
Use 'scoop update czkawka-gui' to install a new version.

Possible Solution
Update archive name in package manifest https://github.com/ScoopInstaller/Extras/blob/500ba541c0d32839ed46c03aecf6a0a5efebd08b/bucket/czkawka-gui.json#L25
Scoop and Buckets Version
v0.3.1 - Released at 2022-11-15

'extras' bucket:
500ba541c (HEAD -> master, origin/master, origin/HEAD) metasploit-framework: Update to version 6.3.58-20240226113509

'games' bucket:
4bbc1fed6 (HEAD -> master, origin/master, origin/HEAD) yuzu: Update to version 0-1727

'java' bucket:
0b128307 (HEAD -> master, origin/master, origin/HEAD) temurin8-nightly-jre: Update to version 8.0.412-4.0.202402231854

'main' bucket:
c9a5c65e2 (HEAD -> master, origin/master, origin/HEAD) poetry: Update to version 1.8.1

'naderi' bucket:
a7380487 (HEAD -> master, origin/master, origin/HEAD) chord: Deprecated

'nonportable' bucket:
0694d98 (HEAD -> master, origin/master, origin/HEAD) 10x-np: Update to version 1.0.129

'sysinternals' bucket:
e669cc0 (HEAD -> main, origin/main, origin/HEAD) chore(ci): switch to `pull_request_target` trigger

'versions' bucket:
4d4344c4c (HEAD -> master, origin/master, origin/HEAD) vscode-insiders: Update to version 1.87.0-insider+1708955303482

Scoop Config
last_update         use_external_7zip scoop_repo                              scoop_branch
-----------         ----------------- ----------                              ------------
26.02.2024 21:20:40              True https://github.com/ScoopInstaller/Scoop master

PowerShell Version
Name                           Value
----                           -----
PSVersion                      7.4.1
PSEdition                      Core
GitCommitId                    7.4.1
OS                             Microsoft Windows 10.0.22631
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Additional Softwares
No response

Looks like there is a question which version to use as "default": qarmin/czkawka#1217.

Fixed by #12915.


Preview page

Would be nice to have a preview page where we can see this in action before downloading/installing.

Working on it!
But this is just simple replacement of laravel auth and home.


Merge latest Library.Template

Bump xunit.runner.visualstudio from 2.4.5 to 2.5.0 (#210)
Bump xunit from 2.4.2 to 2.5.0 (#209)
Remove tool run from dotnet nbgv invocation
Dependabot to ignore dotnet-format

Codecov Report

Merging #42 (adc5765) into main (9e1d819) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #42   +/-   ##
=======================================
  Coverage   95.03%   95.03%
=======================================
  Files          90       90
  Lines        3804     3804
=======================================
  Hits         3615     3615
  Misses        189      189

Flag
Coverage Δ

LinuxHost
95.03% <ø> (ø)

Release
95.03% <ø> (ø)

WindowsHost
95.03% <ø> (ø)

macOSHost
95.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Add flag to the challenge!

Of course it has to be (arguably) the best password of them all: hunter2

@mc2squared Closing until we can confirm, please open a new PR when you are available to confirm your flag!


Shift+Enter to move to next line

Shift + Enter should be added in this list for the editing/authoring experience.

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

ID: 955cb34a-35ce-b369-a83b-b68195de2a63
Version Independent ID: e7503f23-33ab-2a1a-6796-ea6d96b64146
Content: Keyboard shortcuts for canvas apps - PowerApps
Content Source: powerapps-docs/maker/canvas-apps/keyboard-shortcuts.md
Service: powerapps
GitHub Login: @AFTOwen
Microsoft Alias: anneta

Added. Thanks, @thatapiguy! Closing issue.


unable to bold one word in a text

what i want is to unable jspdf to bold one word only or more than one word.. or you can have an option which you can choose what word you want to bold like this image i attached..
Need help. :(
Thanks....

help..

I am also having this issue and cannot find a way to do this. Is this not supported?

One way is this. Problem is, that you cant use fromHTML in nodeJS.
var doc = new jsPDF()

doc.setFontSize(40)
doc.fromHTML( 'Paranyan <b>loves</b> jsPDF', 35, 25)


Header / footer logo accessibility

The image link with the logo for Stanford Libraries has an aria-label in addition to an image with alternative text and both instances have a value of "Stanford Digital Repository" without including the visible Libraries text. (also: "There are images with an alternative text that does not provide a meaningful equivalent. The "Stanford Libraries" logo has "Stanford Digital Repository" as the alternative text.")

@astridu The logo is:

However, it links to https://sdr.stanford.edu/
That seems kinda contradictory (i.e., an image for SUL Libraries that links to SDR) and makes it unclear how to correct the labels.

Agree. Let's have the logo link to Stanford Libraries. I looked at other sites, and it looks like they link the 'Stanford University' logo back to the main university page, and then their department name 'Stanford | Engineering' goes to the engineering home. Could we do something similar here where our logo links to Stanford Libraries, and then our name 'Stanford | Digital Repository" goes to SDR home?

That'll work @astridu.


Update github CI workflow

This updates the branch name in our CI config. Are there any other steps to restore our CI, or is this enough?

I was going to ask since I saw things were hanging. Thanks for opening @yassere

Forgot about this. Thank you @yassere!


OM180 - initial check security fix

includex google/protobug 1.33.0 version, to fix

security fix - (EXPORTER) Fix Infinite loop
security fix - (EXPORTER) Fix Infinite loop

closing, incorrect pull request


Screenshots?

Hi there.... it seems like you've got a GUI in your project, but I can't tell for sure.  It would be great to see some screenshots in your readme if you've got a moment to add them.  They definitely grab the attention of visitors as it's not all that common for Python programs to have a GUI and when they do, people like seeing pictures.

Added SS to the readme

VERY NICE!
I had no idea!  I only knew you used PySimpleGUI and didn't know the extent.  These are awesome to see.  I like it! There's a lot more to your GUI than I thought there would be.

I'm going to be stealing your sunken text status idea.  It's a really nice touch and an easy way to draw attention to and group information.

I am very grateful for the excellent tool that is PySimpleGUI. Keep up the good work!!

Thank you... I will... seeing results like yours keeps the project going.

I used your status idea in this little program that downloads youtube videos.

Very clever idea you had.

I used your status idea in this little program that downloads youtube videos.

Very clever idea you had.

Very nice!! Thanks!


Update netskope-cloud-exchange-administration-console-tutorial.md

Just some minor changes to correct the documentation for this app.

@jeevansd

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


feat: add emulator support

Closes #3278.
Clients use generated classes directly for administration tasks, I had to add wrappers to enable emulator support. Clients should use new Bigtable\Admin\BigtableTableAdminClient instead of Bigtable\Admin\V2\BigtableTableAdminClient and Bigtable\Admin\BigtableInstanceAdminClient instead of Bigtable\Admin\V2\BigtableInstanceAdminClient if they need emulator.

Classes in Admin\V2 namespace have GENERATED CODE WARNING, I think this means that manual changes are strongly discouraged.
I've added emulator support for BigtableClient, but that's not enough. Clients also need at least BigtableTableAdminClient to create tables on emulated server.

That is true, but it also says it "is intended to hold manual additions to the generated class".
Changes you make there will not be overwritten. See ImageAnnotatorClient for a precedent. I know it's rather unintuitive.


website: Add v23.1 blog post

For some reason, I don't understand, the small changes I tried to push to #96 didn't work, so i recreated the PR here.


Laggy music

Im having a issue with a bot, the music is speeding up or lagging
Has anyone had any type of issue like this?

might be an issue with your internet

Mmmh, That seems little bit strange as im using an OVH dedicated server and it is practically empty. Any other idea?

it also depends upon your rtc latency (Discord voice latency)

100-110ms, is that too much?
El mié., 10 feb. 2021 9:07, Snowflake<EMAIL_ADDRESS>escribió:

it also depends upon your rtc latency (Discord voice latency)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/DevSnowflake/discord-ytdl-core/issues/19#issuecomment-776522887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMPAG6K553ADAD7E42CRND3S6I5EPANCNFSM4XKAHOHA
.

voice latency is different from regular ws latency, might be server region or something like that. This problem is not caused by this library. And I'm closing this issue now. If you have any issue(s) related to this library, feel free to open a new one 👍


How to handle dynamic IDs in examples

hey guys! :)
i've a few components that generates dynamic internal ids for their children, like:
import shortid from 'shortid';

class Field extends React.Component {
  constructor() {
    this.instanceId = shortid.generate();
  }

  render() {
    return (

        Example
        <input id={this.instanceId} />

    )
  }
}

Every time I start the style guide, I've a few red bars diffing only the ID.
Without moving it to a prop, there's any solution for this issue?
(I'm not worried about it tbh, just asking :P)

Hello @oieduardorabelo, thanks for your question.
Snapshots fail with pseudo-random values or values that change overtime and, apart from mocking the source of randomness, there is no solution.
In this case, as you said, the only way is passing the shortid generator as prop.


Isolate Workspaces into individual subscriptions

As an IT admin, I want each workspace deployed into their own subscription so that research teams can manage their own subscriptions, improve security isolation, billing isolation, and overcome subscription resource limitations.

Closing as duplicate of https://github.com/microsoft/AzureTRE/issues/1073
Please discuss on the existing issue. Thanks.


Missing step in AsyncGeneratorResolve

AsyncGeneratorResolve seems to be missing the following after step 9:

Set onFulfilled.[[PromiseCapability]] to promiseCapability.

I see the problem here. It looks like https://github.com/tc39/proposal-async-iteration/commit/e544dc869d0c2122d81a32dcdc40cb42ee7db3ed and https://github.com/tc39/proposal-async-iteration/commit/7bd31a53d3a33f2e50876a4cb206e4457b631685 are incorrect, because PerformPromiseThen uses its last parameter to store the value.


Fix most of gcc warnings from invalid trace format specifiers

Use %PRIu32 instead of %d to print uint32_t as compiler expects.
Warnings being fixed:
---8<---8<----
[Warning] mbed_trace.h@115,90: format '%d' expects argument of type 'int',
but argument 4 has type 'uint32_t {aka long unsigned int}' [-Wformat=]

rebuild

rebuild

rebuild


Unable to run to convert JPG to NDArray

Hello and thanks for this amazing project, I'm eagerly attempting to get something running.
I've attempted to read a JPG using the following tutorial and updated code snippet:
https://programmer.help/blogs/use-tensorflow.net-to-train-your-own-data-set-under-c.html
        private static NDArray ReadTensorFromImageFile(string file_name)
        {
            using (var graph = tf.Graph()/*.as_default()*/)
            {
                var jpg = tf.image.decode_jpeg(new IoApi().read_file(file_name));
                var casted = tf.cast(jpg, TF_DataType.TF_FLOAT);
                var dims_expander = tf.expand_dims(casted, 0);

                using (var sess = tf.Session(graph))
                    return sess.run(dims_expander);
                //return tf_with(tf.Session(graph), sess => sess.run(dims_expander));
            }
        }

As you can see, I've tried several variations of this to get it to work. It fails with a NullReferenceException because tensor.graph is null here https://github.com/SciSharp/TensorFlow.NET/blob/d1503cacf282b56ac9fb2ffa7f72b386e5c5e333/src/TensorFlowNET.Core/Graphs/Graph.cs#L221
I could probably read an image and convert the individual pixels to an NDArray, but is there a proper way to read an image?

@jdluzen Can you follow the updated code here: https://github.com/SciSharp/SciSharp-Stack-Examples/blob/master/src/TensorFlowNET.Examples/ImageProcessing/CnnInYourOwnData.cs


Integer problem starting with 0

Feature
0123

Integer starting with 0 should raise SyntaxError.
I think it should be fixed in lexer.rs
Expected Result
>>> 0123
  File "<stdin>", line 1
    0123
       ^
SyntaxError: invalid token

Actual Result
>>>>> 0123
123

lexer.rs sounds right.
Here is the CPython code that handles it https://github.com/python/cpython/blob/master/Parser/tokenizer.c#L1441


11 2022 release notes

[x] This contribution adheres to CONTRIBUTING.md.

What does this Pull Request accomplish?
Release notes for the 2022-11 release
Why should this Pull Request be merged?
Customers need this information to successfully upgrade.
What testing has been done?
Customer success will be testing this release notes while the PR is open, and we will be updating it as needed based on this feedback.

Should this be 2022-11 rather than 2022-09?

Fair chance I'm missed something. Is there a particular item this is in reference too

@JD-Robertson I'm also wondering if we should move our template into the release notes such that we have templates that are release specific rather than evergreen. Thoughts?

All dates in these file seem to be 2022-09, unless I'm going crazy.
I'm not sure how we decided on the current release notes format. I certainly don't object to any changes.


Issues with exporting and importing enums

TypeScript Version: 2.4.1
Code
file1.ts
export enum Colors {
  Red = "RED",
  Green = "GREEN",
  Blue = "BLUE"
}

export enum Numbers {
  ONE = 1,
  TWO = 2
}

file2.ts
import { Colors, Numbers } from "./file1";

Expected behavior:
Colors and Numbers should not be undefined in file2.
Colors should be:
{
  Red: "RED",
  Green: "GREEN",
  Blue: "BLUE"
}

Numbers should be
{
  ONE: 1,
  TWO: 2
  1: "ONE"
  2: "TWO"
}

Actual behavior:
Colors and Numbers are undefined.
Any enum entry requested fail with "Cannot read property 'enum entry' of undefined".
It seems that this only occurs in case of a circular dependency.
If you remove the circular dependency everything seems to work fine.

It seems that this only occurs in case of a circular dependency.
if file2 is an indirect dependency of file1 this still happens
it you remove the circular dependency everything seems to work fine.

@glazar is this similar to #112 ?
If not, would it be possible for you to create a minimal repo that reproduces this?

@kulshekhar
I do not know whether this is related to #112 or is another issue altogether.
I have created a small repo reproducing this issue.
Please have a look and let me know what you think.
This issue only happens for us only when running the unit tests.
We have worked around this issue by removing the circular dependency.
The problem is not visible when the code is bundled using webpack with ts-loader.

Thanks for the repo. I was able to replicate this issue and dig into it a bit.
I think that this comment in the Typescript repo explains the cause of this issue.
Given that this is more of a Typescript issue, I'm not sure there's much we can do here

From that comment, it appears that the TypeScript team has no intention of fixing this, and I don't really see how they could given that transpileModule has no consistently reliable way to resolve imports and access the enum definitions.
As this issue comment suggests, it isn't always safe to use transpileModule. I certainly understand the performance reasons why transpileModule was chosen for the preprocessor, though. The only solution I can think of would be to do a full compile ahead of time and then serve the transpiled JavaScript files as Jest requests them. That is not a trivial change, of course, and would require having a watch process in place as well.

Would it be workable to change the preprocessor to use a different compiler api - but just passing in the test file and a preconfigured list of files containing const enums?
I remember a comment in another issue which suggested they rewrote the const enums after compilation using a webpack plugin - maybe that would be more viable?

@rikkit if there's a workable solution, we can find a way to get that in

faced with same problem. But not with enums but with using my library (https://github.com/goloveychuk/tsruntime) which uses custom transformers api.
It emits types metadata and requires types from imported module. So it's any everywhere.
So, as I see, two options to workaround

run program.emit with filelist of all files. But this will affect performance since it will compile same files many times.
use LanguageServiceHost compiler api, and compile all files before jest stage starts.

https://github.com/goloveychuk/awesome-ts-jest
Proof of concept, which solves problem with imported enums and custom transformers.
With smart files updating (versioning) it should work pretty fast.

You can test with beta version (see #697) which handles const enum and others thanks to the language service (setting typeCheck: true).

I faced the same problem with enum in unit tests. The problem was the services with the enum var were mocked, and the mock did not have the enum var.
The solution was copy the enum also in the mocked service and export it so the classes that used the service can access to it.
on ServiceA.ts
export enum Colors {
  Red = "RED",
  Green = "GREEN",
  Blue = "BLUE"
}

and also on ServiceAMock.ts
export enum Colors {
  Red = "RED",
  Green = "GREEN",
  Blue = "BLUE"
}

I had the same problem.
Here is what helped me:

Make enum const.
export const enum OneTwoThreeEnum {  One = 'one',  Two = 'two',  Three = 'three' }
Use it like this:
"One" as OneTwoThreeEnum.One

@yoasia This issue is fixed with version 23.10 - try upgrading if you haven't already.

I can confirm this is still an issue for version "24.0.2".

Any update on this once, we're still facing the issue in our Vue project?

What's the issue @germain-receeve @thefill ? 23.10 works for me, referencing const enums in test files with no problem. What's the difference?

We ended up solving the issue by updating our jest.config.js file: we added core-js to setup files and isolated modules from ts jest.
  setupFiles: ["core-js"],
  globals: {
    "ts-jest": {
      isolatedModules: true
    }
  }

Hope this can help further readers

Why is it closed? I can't use exported enum in my tests
actions.ts:
export const enum DefinitionActionType {
    LOADING_DEFINITIONS = 'definitions/LOADING_DEFINITIONS',
    LOADED_DEFINITIONS = 'definitions/LOADED_DEFINITIONS',
}

When running normally it's fine, but when i run tests it fails:
 FAIL  src/reducers/form-definitions/reducer.test.ts
  ● Test suite failed to run

    TypeError: Cannot read property 'LOADING_DEFINITIONS' of undefined

      25 | export function reducer(state = initialState, action: DefinitionActions): State {
      26 |     switch (action.type) {
    > 27 |         case DefinitionActionType.LOADING_DEFINITIONS:
         |                                       ^
      28 |             return {

I'm seeing the same problem with version 24.0.2.
Seems to only happen when isolatedModules: true is used, but we need this setting to improve performance issue (#1115).

I found a workaround that sort of makes things okay:
It works if you have a module file that only exports enums

There is a note that if using enum inside .d.ts  won’t work, but const enum will work.

There is a note that if using enum inside .d.ts won’t work, but const enum will work.

i have same issue!

@ahnpnl @v7lin same here.

I'm seeing the same problem with version 24.0.2.
Seems to only happen when isolatedModules: true is used, but we need this setting to improve performance issue (#1115).

Const enum doesn’t work with isolatedModules: true because typescript transpile API doesn’t support it, you can check in the documentation.
@lucasriondel are you using isolatedModules: true ?

I still have same issue. what's the solution and is it documented somewhere?

I still have same issue. what's the solution and is it documented somewhere?

@safareli are you using isolatedModules: true ?

@safareli are you using isolatedModules: true ?

@ahnpnl, no I don't have it in my tsconfig.json (or any other place)

@ahnpnl, no I don't have it in my tsconfig.json (or any other place)

I meant ts-jest option isolatedModules in your jest config.
If you have it as true, enum won't work. If you have it as false (default) it should work.
If you have it as false and it still doesn't work, you can share a repo so I can help.

I meant ts-jest option isolatedModules in your jest config.
If you have it as true, enum won't work. If you have it as false (default) it should work.
If you have it as false and it still doesn't work, you can share a repo so I can help.

@safareli you gave me the idea of checking this in my code because I had the same problem. Turns out we had too many index.ts which re-exported stuff from sub-modules. So I specified my inports in those modules where the error occured and voilá: fixed it!
eg.
import { crudEntityFactory, ReduxEntities, RootState } from '@core/data';
became
import { crudEntityFactory, ReduxEntities, RootState } from '@core/data/redux';

I can confirm the issue remains in version "26.2.0".

Thank you for all suggestions, they solved my problem! I'll leave what helped me for others to find.
For me making the dependency tree a bit more granular helped, either:

mock definition require()-d in jest.mock() moved to a separate file (it was in a bundle with additional imports)
or
importing the enum from a different file than re-exported index.ts

I am also still seeing this issue. I just isolatedModules: true and isolatedModules: false and still running into Cannot read properties of undefined (reading 'All') for both.

I have the same issue with an enum from a third-party package.

same here
  ● Test suite failed to run

    TypeError: Cannot read properties of undefined (reading 'AUTH')

      4 |
      5 | @Module({
    > 6 |   imports: [RmqModule.register({ name: Services.AUTH })],
        |                                                 ^
      7 |   exports: [RmqModule],
      8 | })
      9 | export class AuthModule {}
      ```

      ```ts
      export enum Services {
  AUTH = 'AUTH',
  STATISTICS = 'STATISTICS',
}

Hi people.
On my end the issue was only happening for .ts files and not for .tsx
I duplicated the declaration on the .ts files then the test passed.

yes, i have this problem too, but I don't know how to solve it either.

Just put enum in separate file. It works for me.

It's weird. We use Jest in Angular (TypeScript) since years. And suddenly they are undefined from import? What's happened?

Still happen in ts-jest v29. Why is this issue closed ?

The only workaround I have found is to mock the enum in the test file and declare it as a object.  Trying to mock it in anyway in a mock file failed to work and declaring it as an enum in the test file also failed to work.

Still happens to:
"ts-jest": "^29.1.4"
"jest": "^29.7.0",
"jest-environment-node": "^29.7.0",

This is very annoyning and frustrating issue.

Only thing that works for me is putting the enum in a .ts file (and not in tsx)


Excessive Errors

I have installed the integration and it seems to be working.  The issue is that device goes into power saving mode and then the errors in HA pile up.   When not in power saving mode, it works great.  See below
Logger: custom_components.dlink_hnap.binary_sensor
Source: custom_components/dlink_hnap/dlink.py:126
Integration: dlink_hnap (documentation, issues)
First occurred: October 28, 2020, 10:09:17 PM (12330 occurrences)
Last logged: 3:17:49 PM
failed to update water sensor
Traceback (most recent call last):
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 936, in _wrap_create_connection
return await self._loop.create_connection(*args, **kwargs)  # type: ignore  # noqa
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1025, in create_connection
raise exceptions[0]
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1010, in create_connection
sock = await self._connect_sock(
File "/usr/local/lib/python3.8/asyncio/base_events.py", line 924, in _connect_sock
await self.sock_connect(sock, address)
File "/usr/local/lib/python3.8/asyncio/selector_events.py", line 496, in sock_connect
return await fut
File "/usr/local/lib/python3.8/asyncio/selector_events.py", line 528, in _sock_connect_cb
raise OSError(err, f'Connect call failed {address}')
OSError: [Errno 113] Connect call failed ('<IP_ADDRESS>', 80)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/config/custom_components/dlink_hnap/dlink.py", line 116, in call
result = await self.soap().call(method, **kwargs)
File "/config/custom_components/dlink_hnap/dlink.py", line 253, in call
resp = await self.session.post(
File "/usr/local/lib/python3.8/site-packages/aiohttp/client.py", line 480, in _request
conn = await self._connector.connect(
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 523, in connect
proto = await self._create_connection(req, traces, timeout)
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 858, in _create_connection
_, proto = await self._create_direct_connection(
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 1004, in _create_direct_connection
raise last_exc
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 980, in _create_direct_connection
transp, proto = await self._wrap_create_connection(
File "/usr/local/lib/python3.8/site-packages/aiohttp/connector.py", line 943, in _wrap_create_connection
raise client_error(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorError: Cannot connect to host <IP_ADDRESS>:80 ssl:default [Connect call failed ('<IP_ADDRESS>', 80)]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/config/custom_components/dlink_hnap/binary_sensor.py", line 142, in async_update
water_detected = await self._sensor.water_detected()
File "/config/custom_components/dlink_hnap/dlink.py", line 206, in water_detected
await self._cache_soap_actions()
File "/config/custom_components/dlink_hnap/dlink.py", line 192, in _cache_soap_actions
resp = await self.client.soap_actions(self.module_id)
File "/config/custom_components/dlink_hnap/dlink.py", line 106, in soap_actions
return await self.call("GetModuleSOAPActions", ModuleID=module_id)
File "/config/custom_components/dlink_hnap/dlink.py", line 112, in call
await self.login()
File "/config/custom_components/dlink_hnap/dlink.py", line 57, in login
resp = await self.call(
File "/config/custom_components/dlink_hnap/dlink.py", line 120, in call
self._bad_response()
File "/config/custom_components/dlink_hnap/dlink.py", line 126, in _bad_response
raise Exception("got error response from device")
Exception: got error response from device

I checked with DLink and there is not a way to prevent the sensor from going into power saver mode.

Yeah, all errors will produce log output right now. Would be great to improve this by not repeating unnecessary errors. PRs are welcome!

hi @postlund, I've installed your plugin and so far it doesn't seem to pull the data as expected.
020-11-02 14:45:47 WARNING (MainThread) [homeassistant.components.binary_sensor] Updating dlink_hnap binary_sensor took longer than the scheduled update interval 0:00:05 2020-11-02 14:45:52 WARNING (MainThread) [homeassistant.helpers.entity] Update of binary_sensor.basement_laundry is taking over 10 seconds 2020-11-02 14:45:52 WARNING (MainThread) [homeassistant.components.binary_sensor] Updating dlink_hnap binary_sensor took longer than the scheduled update interval 0:00:05 2020-11-02 14:45:53 ERROR (MainThread) [custom_components.dlink_hnap.dlink] Got an error, resetting private key 2020-11-02 14:45:53 ERROR (MainThread) [custom_components.dlink_hnap.binary_sensor] failed to update water sensor Traceback (most recent call last):   File "/config/custom_components/dlink_hnap/dlink.py", line 116, in call     result = await self.soap().call(method, **kwargs)   File "/config/custom_components/dlink_hnap/dlink.py", line 253, in call     resp = await self.session.post(   File "/usr/local/lib/python3.8/site-packages/aiohttp/client.py", line 504, in _request     await resp.start(conn)   File "/usr/local/lib/python3.8/site-packages/aiohttp/client_reqrep.py", line 860, in start     self._continue = None   File "/usr/local/lib/python3.8/site-packages/aiohttp/helpers.py", line 596, in __exit__     raise asyncio.TimeoutError from None asyncio.exceptions.TimeoutError During handling of the above exception, another exception occurred: Traceback (most recent call last):   File "/config/custom_components/dlink_hnap/binary_sensor.py", line 142, in async_update     water_detected = await self._sensor.water_detected()   File "/config/custom_components/dlink_hnap/dlink.py", line 206, in water_detected     await self._cache_soap_actions()   File "/config/custom_components/dlink_hnap/dlink.py", line 192, in _cache_soap_actions     resp = await self.client.soap_actions(self.module_id)   File "/config/custom_components/dlink_hnap/dlink.py", line 106, in soap_actions     return await self.call("GetModuleSOAPActions", ModuleID=module_id)   File "/config/custom_components/dlink_hnap/dlink.py", line 112, in call     await self.login()   File "/config/custom_components/dlink_hnap/dlink.py", line 57, in login     resp = await self.call(   File "/config/custom_components/dlink_hnap/dlink.py", line 120, in call     self._bad_response()   File "/config/custom_components/dlink_hnap/dlink.py", line 126, in _bad_response     raise Exception("got error response from device") Exception: got error response from device
I have the DCH-S160 (currently testing), and a DCH-S161 (currently disabled in HA).
I am wondering how long does the data poll to HA as I was putting a cup of water to the sensor to see if HA would change the binary state of the sensor, but unfortunately in the entity data it never shows any changes:

If you have ideas what could be wrong, that would be great.

Since the request times out I would assume the IP address is not correct. Double verify that it is correct.

Like dacorad0 says the problem is that the sensor goes into power save mode and does not stay connected to the wifi network. It will connect once in a while to report it's value. So that's why there are excessive errors. Catching those exceptions would be a start. It's normal to not get a response from the sensor if you go to it's ip address. The application mydlink will still give you the status of the sensor last time it reported. Reducing the error amount would also be a start. This behavior from the sensor is totally normal. It will only wake up if there is any reason for it to report a value. I might try myself at forking an PRing...

Like dacorad0 says the problem is that the sensor goes into power save mode and does not stay connected to the wifi network. It will connect once in a while to report it's value. So that's why there are excessive errors. Catching those exceptions would be a start. It's normal to not get a response from the sensor if you go to it's ip address. The application mydlink will still give you the status of the sensor last time it reported. Reducing the error amount would also be a start. This behavior from the sensor is totally normal. It will only wake up if there is any reason for it to report a value. I might try myself at forking an PRing...

Since the request times out I would assume the IP address is not correct. Double verify that it is correct.

I did check the setup, and the IPs are correct and the password to the device was setup correctly. I vaguely remember in December when i was troubleshooting the issue I believe the DCH-S161 uses a different protocol to communicate vs the DCH-s160.

Since the request times out I would assume the IP address is not correct. Double verify that it is correct.

I did check the setup, and the IPs are correct and the password to the device was setup correctly. I vaguely remember in December when i was troubleshooting the issue I believe the DCH-S161 uses a different protocol to communicate vs the DCH-s160.

Just installed the water leak sensor today and I’m getting the same error:
Got an error, resetting private key
And then my home assistant log getting just FILLED with errors until I disable the integration. Any ideas?

Old thread but I'm having the same issue with my sensor
But one thing I'll try is having my Ubiquiti UniFi system (not yet installed but will be soon) detect when my water alarm (and smoke alarm from another brand) connect and send notifications that way, will notify if I can figure something out


Update tutorial "One-class segmentation with 2D U-Net"

Following the integration of the new loader (#699), the tutorial One-class segmentation with 2D U-Net needs to be updated to match the new split (random seed is now applied to filenames and not participant_id so the split is different).
TODO:

[ ] Run the tutorial
[ ] Update the terminal output example in Train model
[ ] Update the terminal output example in Evaluate model
[ ] Update the filenames in the fsleyes command in Evaluate model
[ ] Update the output and ground truth segmentations images in Evaluate model

@mariehbourget From the original tutorial, I noticed that there are 10 subjects under the data_example_spinegeneric. However, for the evaluation step, only two subjects evaluated, do you have any ideas why is this and how to evaluate the whole 10 subjects?

only two subjects evaluated, do you have any ideas why is this and how to evaluate the whole 10 subjects?

It is probably because the evaluation step is only run on the testing dataset (which is a subset of the training dataset): see here for split: 60% for training, 20% for testing --> which matches up with "2 subjects"


refactor: use onConflictDoUpdate instead of repetative updates

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#43  👈
#42
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @techwithanirudh and the rest of your teammates on  Graphite


Current Actions URL is not useful when an action runs many times on the same commit, it shows many runs and it's impossible to say which one triggered the notification.

The current Actions URL point to (https://github.com/org/repo/commit/sha/checks)
For example https://github.com/prisma/language-tools/commit/1e465e035ebd81650ec70146fd14cd4a943b29a2/checks
This example has a quite a few actions runs on the same commit which makes it really hard to know which run triggered the notification

My current workaround is to change the footer with the link to the actual run like this
      - name: Set current job url in SLACK_FOOTER env var
        if: always()
        run: echo "SLACK_FOOTER=<$GITHUB_SERVER_URL/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID|Click here to go to the job logs>" >> $GITHUB_ENV

Which looks like this

It works very well but it would be amazing (and less YAML 😅) to have something like this directly implemented in this action.
I could see different options

Change the default Action URL
Same but only opt-in with a param?
Recommend this workaround (not the best)

What are your thoughts?

Same issue on me!
I read github actions doc a bit, and found GITHUB_RUN_ID
https://docs.github.com/en/actions/learn-github-actions/environment-variables

A unique number for each workflow run within a repository. This number does not change if you re-run the workflow run. For example,<PHONE_NUMBER>.

It can help if one can construct the URL to current workflow run with this ID as
https://github.com/<user>/<repo>/runs/<GITHUB_RUN_ID>?check_suite_focus=true , right?

cf. https://github.community/t/get-runs-url/16921

Oops, the url construction is already explained in the description :sweat:

This one has been resolved in #162, and would be available for use as soon as the new build is published. Credits to @shino for giving a workaround.

Thanks 🙌🏼 ! I'm curious to check this once it's released!


Releases and tags are not lined up, Latest release should be v2021.2

The releases don't have the latest release, I believe this causes some package managers to pull the wrong release.

I believe we have fixed this. with the latest releases.


Formatter option tuples introduction

no really ready for review yet just wanted to bring it out here

high level integration tests (benchee_test) pass

still needs to fix existing unit tests and add new ones for what
I added

also probably move some code around

[x] fix unit tests

[x] write new tests (this is just a sketch to see if the concept as a whole holds up)

[x] remove the Formatter use just stick with it as a behaviour and use output from somewhere

[x] fix documentation :muscle:

lil flakey
  1) test measure/1 will not leak processes if the applied function raises an exception (Benchee.MemoryMeasureTest)
     test/benchee/benchmark/measure/memory_test.exs:30
     Assertion with == failed
     code:  assert Enum.count(Process.list()) == starting_processes
     left:  93
     right: 94
     stacktrace:
       test/benchee/benchmark/measure/memory_test.exs:43: (test)
..
Finished in 4.5 seconds
111 doctests, 235 tests, 1 failure, 4 skipped

Blah gawd with my optional second argument dialyzer isn't happy with the output method... guess gotta remove it in this PR.
incidentally this is me right now so maybe another time:

Rebased to get rid off some bad commits and to be able to touch the changelog more easily

This might be done pending your reviews :angel:
Dreamy dog for a change of pace!

As only one behaviour is specified in those modules I decided to go with just an @impl true - it's the first time I'm using this so I might be missing something :)

I think @impl Benchee.Formatter just gives slightly nicer error messages since it knows to check that behaviour. @impl true will work just fine here for sure.
I think if we want to keep Elixir 1.5 compatability, we'll need to rewrite the functions in the console formatter, though, to remove the default argument. To the compiler in 1.5, those functions with default arguments have an arity of 1, which then means it's not implementing the behaviour properly.
We can do the following to get around it:
def write(suite), do: write(suite, %{})

@impl true
def write(suite, options) do
  #...
end

I have a feeling that this is essentially what the macro expanded to when there was default arguments in 1.5. We just need to be specific here to make it happy 😉

Grml.
I'll take a look when I get time, might want to get rid off the default argument again although it's quite nice if you wanna specify it as a function but then you shouldn't so we can just get rid off it

:boom:


Import Model Dropdown

Is your feature request related to a problem? Please describe.
Right now, the flow for importing models requires going to https://ollama.ai/library. It would be nice to have these models pulled via API so a user doesn't need to have 2 tabs open.
Describe the solution you'd like

Auto pull models from ollama.ai/library via API. Can probably use newest or most popular by default.
Maybe select model can also have a UI that displays https://ollama.ai/library as an iFrame OR recreate it with more stats could be cool so I can just select locally.

Describe alternatives you've considered
No alternatives, just have 2 tabs open
Additional context
n/a

Hi, Thanks for the suggestion. I'll close this issue for now because afaik there aren't any API endpoints listing all available models from Ollama.ai. However, if you find a way to get the list of available models from the website, feel free to make a PR. Thanks!


Add compatibility to common formats (PGN, FEN, etc.)

Issue:

the current implementation uses its' own data structures
allowing common formats would simplify the interaction with other chess programs

Suggested Solution:

add a new API function converting a PGN draw into a draw using a chess board
rework the API function VisualizeDraw() to support PGN output
add 2 new API functions for conversions between chess boards and FEN strings

create Python functions:

Board_FromFen(fen_str: numpy.ndarray)
Board_ToFen(board: numpy.ndarray, is_simple: bool)
ChessDraw_FromPgn(board: numpy.ndarray, pgn_str: str)
ChessDraw_ToPgn(ext_draw: int)


[WIP][SPARK-33200][CORE][K8S] Verify all shuffle blocks migrated in K8s integration testing

What changes were proposed in this pull request?
Verify that all of the shuffle files are migrated during integration testing.
Why are the changes needed?
More thorough testing.
Does this PR introduce any user-facing change?
No
How was this patch tested?
Test only change.

Test build #130070 has started for PR 30115 at commit 615053f.

Test build #130070 has finished for PR 30115 at commit 615053f.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

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

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

Merged build finished. Test FAILed.

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

The K8s failure is related to debian mirror being unreachable. So while I do expect to see a K8s failure, it's not that one.
Jenkins retest this please.

Jenkins retest this please.


Fix the coveralls step in the travis CI

The coveralls step in the Travis CI seems to be broken since few days.
The reason was:

Can't set personality: Operation not permitted
kcov: error: Can't start/attach to /home/travis/build/Geal/nom/target/debug/deps/nom-1bf2433837785ec3

Fore more details: travis-ci.org/Geal/nom#L2100
That related to travis-ci#9061.
The fix is to set sudo at required in .travis.yml.

Thanks!


Update http2-server to 11.0.1

Updates org.eclipse.jetty.http2:http2-server from 9.4.37.v20210219 to 11.0.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.eclipse.jetty.http2", artifactId = "http2-server" } ]

labels: library-update

We have no Servlet 5 plan yet.


Update content.json

Being a native speaker I have changed the translation for high in Gujarati as the previous translation meant superiority(ઉચ્ચ) and ઉચ્ચુ means high.

🚀


Printing fails after command is executed

I am writing a tool to do ability checks for D&D. I am trying to structure my code so I have one function that parses CLI arguments and returns the parsed args, then code elsewhere that does the logic I want. I only have the CLI related function wrapped with click decorators, but for some reason after my decorated function is executed, all printing to stdout fails. Is this normal?

@click.command()
@click.option("--players", required=True, help="Comma-separated string list of players (case sensitive) to send message to; use \"all\" for everyone",
              type=str)
@click.option("--check", required=True, help="The type of check", type=str)
@click.option("--dc", required=True, help="Difficulty Class of the check", type=int)
@click.option("--advantage", default="", help="Comma-separated string list of players with advantage on check; use \"all\" for everyone",
              type=str)
@click.option("--disadvantage", default="", help="Comma-separated string list of players with disadvantage on check; use \"all\" for everyone",
              type=str)
@click.option("--pass-text", default="", help="Text to send if player succeeds check", type=str)
@click.option("--fail-text", default="", help="Text to send if player fails check", type=str)
def cli(players, check, dc, advantage, disadvantage, pass_text, fail_text):
    """
    Parse CLI args into simple dict and return.
    """
    with open("config.json") as json_file:
        config = json.load(json_file)
    return Parameters(config, players, check, dc, advantage, disadvantage, pass_text, fail_text)

...

if __name__ == "__main__":
    parameters = cli()
    run_check(parameters)
    print("Can't print here; this doesn't show up in stdout!")
    click.echo("This also doesn't show up in stdout!")

Hello @jsgoller1
As per the documentation

standalone_mode – the default behavior is to invoke the script in standalone mode. Click will then handle exceptions and convert them into error messages and the function will never return but shut down the interpreter. If this is set to False they will be propagated to the caller and the return value of this function is the return value of invoke().

An example that does not work
import click

@click.group()
def cli(): pass

@cli.command()
@click.option('-s', '--string-to-echo')
def echo(string_to_echo):
    click.echo(string_to_echo)

if __name__ == "__main__":
    cli()
    click.echo("hello") # will not be printed as by default standalone_mode is set to true

But if you set the standalone mode to false
if __name__ == "__main__":
    cli(standalone_mode=False)
    click.echo("hello") # will  be printed

Double check the how to contribute and make sure you already checked the documentation and stackoverflow almost same question found here
Thanks

Got it, @aelbehe95 - thanks for the clarification!


Try it with the new RPI0 2!

Just a note to myself: I need to try it using the new RPI0 2.

Hey, great repo your documentation detail is really good! I am planning to recreate part of your project on the Pi Zero 2 W, have you since then already tested it? Am really interested in the power consumption since I will be applying it in a low power scenario.

Sadly I haven't yet. The Pi Zero 2 is arm7 (like the RPI 3) while the Zero 1 is arm6 (RPI 1 and 2), so I suppose it should be possible to directly install google coral packages without the need to compile certain things like I had to do for arm6. However, you may have trouble with versions as the latest RPI OS will be ahead of the python version coral is using. I had this problem with Ubuntu, but I think I solved using docker.


patches: update releases for October cycle

Update Patch releases for October cycle
/assign @justaugustus @puerco @Verolop @xmudrii
/cc @kubernetes/release-managers

should we update the EOL https://github.com/kubernetes/website/pull/29744/files#diff-3fbc599bc532616df1b24035ffaafddf482f185fdd2b04c3f1597e34d9ef62f3R138 to be the day of this Oct patch release? or we might make a final release in November?

Preview of updates page:

https://deploy-preview-29744--kubernetes-io-main-staging.netlify.app/releases/patch-releases/

if we need to change we can do in a follow up, remove the hold
/unhold


[Registry Preview] Opening a large file breakes window placement

Microsoft PowerToys version
0.69.0
Installation method
GitHub, PowerToys auto-update
Running as admin
No
Area(s) with issue?
Registry Preview
Steps to reproduce
Open a large .reg file (e.g. export of HKCU.)
Note: I can't reproduce it yet.
✔ Expected Behavior
For the large file a error message is shown. After closing the app I can open other smaller fiels without problems.
❌ Actual Behavior
The large file shows a taskbar button without window. After closing the app I can't use it anymore until deleting the settings file. The settings file contains a broken window placement setting.
{"appWindow.Position.X":-32000,"appWindow.Position.Y":-32000,"appWindow.Size.Width":199,"appWindow.Size.Height":48}

Other Software
No response

having a repro here and sample file here is critical.  Exporting HKCU IMO is not a realistic usage of the tool as well.
/needinfo

@crutkas
I was testing some othe things and opened the wrong fole by mistake. Normally the app shows a message that the file is to large. But un this case something triggered the wrong window size. Not sure what and why.
Maybe the large file isn't the reason.


NO-JIRA: Don't run ValidateMetrics on teardown if test has already failed

What this PR does / why we need it:
Don't run ValidateMetrics on teardown if test has already failed
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/lgtm

/hold
This won't run in cleanup phase. I think it's much easier to remove the t.Run() and not call teardownHostedCluster() twice rather than copying validateMetrics() to both places.

@stevekuznetsov we don't want to run it in Cleanup phase anyway, cleanup phase means there was an uncaught panic somewhere.
/unhold

Why not? Seems super hard to reason about what is and is not run in these cases ...

cc @enxebre - we've already had some chats on the subtests for these parts. It seems to me like this is a clear-cut spot where not using a subtest might be a good idea. Not only does it let us do logic on t.Failed() but it then also keeps us from having to have separate and somehow distinct execution paths through this (already fairly complex) set of logic.
From my past experience, keeping the framework execution as part of the outer test and sub-testing specific parts of the actual test logic (e.g. the PKI flow in CreateCluster, etc) really helps readability.

/lgtm cancel
Definitely seems weird to merge this in.

Why not?

why would you run another validation test if the test has paniced (aka failed)? its pretty straightforward if you read the code calling teardown, we have the happy path:

CreateCluster -> Validate -> Run the actual test -> Teardwon -> PostTeardown

And a failsafe case if there were a panic during the test execution:

CreateCluster -> Validate -> Run the actual test -> panic -> registered cleanup func is triggered -> Teardwon

you are suggesting that we need to run the ValidateMetrics in the 2nd case, why would we want that?
The discussion of whether to run the happy path Teardown in a sub-test or not seems totally unrelated for this PR.

the teardown logic is separated into 2 functions (h *hypershiftTest) teardown() and the helper func teardownHostedCluster() specifically so we don't pollute the actuall logic of tearing down a cluster via teardownHostedCluster() with unrelated sub-tests. So we don't have to use t.Failed() inside that func and keep it simple, which is what this PR fixes.

If you look at the code we run in the defer, we already (attempt to) run validations when there's been a panic, since we run postTeardown() in the deferred func. I'm not arguing for us to do something novel and different - I am simply arguing that we should be consistent. If we don't want to run validations after a panic, we should remove those across the board, rather than in this one place.
I'm not sure we derive value from the teardown -> teardownHostedCluster indirection, and, specifically, having to deal with "how many times have we torn down?" and "are we running in t.Cleanup() or not?" makes for a complex mental model of the flow for our tests.
I propose we:

remove the call to t.Cleanup()
remove the need to track how many times we've done teardown
simplify the teardown code to remove the distinction between different cleanup phases
remove the sub-tests, which allows us to simply use t.Failed() to check whether or not a validation should occur

postTeardown() will not run if there was a caught panic, because the test will be marked as failed with first and postTeardown() does check h.Failed() before running.
t.Cleanup() is always called, which is why we need to check if we have already torn down the cluster, because in the happy path, teardown() would have been already called and destroyed the cluster then we don't want to run it again, hence the check.

Again, this a fairly small PR fixing a simple the issue that ValidateMetrics is executed in the wrong func. I don't believe this is the right place to discuss changes on how we structure our tests.

Removing the t.Run() would be just as small of a change - so respectfully I don't want to get down the rabbit hole of what is or is not the right forum.

will not run if there was a caught panic, because the test will be marked as failed first with

I am pointing out that your argument for the refactor you made was that we did not need to bother running code in the post-panic handler, and I pointed out that your statement is inconsistent with the rest of the codebase, specifically because of what you write here - we already do that. If you feel strongly one way, let's make the code here consistent. The cleanup story in the existing end-to-end is pretty complex and I would appreciate us taking steps to making it less so, because I have trouble grok-ing it as it is.


change dns /port

"By default, Entralinked is configured to automatically use the local host of the system.
This approach is not always accurate, however, and you may need to manually configure it in config.json."
I'm trying to create a Nintendo "hub", is there a way to change the port and/or DNS on which the application works? how?

You can change the value of hostName in config.json and the DNS server will use that instead. The port cannot be changed because there's not much of a point since the DS wouldn't be able to connect if you changed it.

I was asking because maybe I'm ignorant, I set the 2 IPs from the PC settings, one for this and one for (forgive the spam) DNS-server which acts as a DNS hub, but even if you use 2 different IPs, it still gives me an error

You can change the value of hostName in config.json and the DNS server will use that instead. The port cannot be changed because there's not much of a point since the DS wouldn't be able to connect if you changed it.

I was asking because maybe I'm ignorant, I set the 2 IPs from the PC settings, one for this and one for (forgive the spam) DNS-server which acts as a DNS hub, but even if you use 2 different IPs, it still gives me an error

that is, logically I think that since they are different IPs, even if the port is the same it shouldn't cause problems, but it does

What kind of error are you getting, and what are you trying to do?

I'm trying to start two "DNS servers" on the same PC, one for the sync link of Pokemon and the other for the WFC servers, but even if they have different IPs it tells me that the port is already in use

hostName only changes what the DNS server points to, not what IP the program actually binds to.


Camper News gives "no url supplied" error on successful story submission

Hello, How can I assing this issue to myself ?

@vanirt Go right ahead.
https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/How-To-Fork-And-Maintain-a-Local-Instance-of-Free-Code-Camp
https://github.com/FreeCodeCamp/FreeCodeCamp/wiki/How-To-Create-A-Pull-Request-for-Free-Code-Camp

I experienced this issue in the past days but it wasn't so bad, because it ended up letting me correctly submit the link I wanted. Today (12-20-15) it seems to be worse. I had big troubles submitting a link, and I ended posting it twice (and I got no brownies at all). I also see a lot of links submitted twice in the news section.

I can't seem to reproduce it. @Em-Ant : What is the frequency of this issue, in terms of attempts?

@raisedadead I experienced the "no url supplied" on every post of mine in the last week (3 or 4 posts in total). Only the last submission ( dec-20 )  required three attempts. I've not posted anything after that.

I am getting this issue as well.  Confirmed.

I am having this issue. Has anyone resolved it?

I am unable to post link as well :(


gather_nd vs. take

I have an observation and I'm hoping someone can advise.
I have a scenario where I maintain a large table of vectors, so a basic (n,m) array of n vectors of size m. I then have some system that generates indices into this table, and I want to pull rows out of the table at those indices. (more background? A hash table version of NeRF is what we're making).
So, I have a set of indices, and I want to gather the rows out of the table to make use of elsewhere. There's two operators in mxnet that will do the job - gather_nd and take
I could have more than 100k, even 1000k indices:

at 100k  take will do a forward pass in less than 1 ms, but a backward pass will take about 45 ms. Meanwhile, gather_nd will do a forward pass in about 16 ms, and a backward pass in under 1 ms.
At 1000k indices, take is 4 ms fwd, 400 ms bwd, gather is 170 ms fwd, 1ms bwd. .

So... obvious question... is there a way to get the best of both worlds here? The fast forward pass of take, the fast backward pass of gather_nd?
Is there a better operator for gathering rows from the table?

Had another look through the documentation, and though I hadn't thought of it that way, the other option is to use the embedding function. And woderfully, that appears to be the best of both worlds that I wanted,


Set up CI

@vshabanov : Thanks for your work on this package!
Would be great to see some GitHub Actions here that perform continuous integration.

I am satisfied now with the state of the CI now.
It gives evidence for buildability with serveral GHCs (#61) and on several platforms (#59).
Loose ends:

CI using stack
Windows CI running testsuite

However, I am not planning to work more on this for now.

Thank you a lot for implementing the CI.
I fixed the Windows CI in #69. It turned out it was just a missing path to MSYS, which caused the test not to find the ICU .dll files.


GATK-SV sample finder script

Leaving this here as a draft - a self-documenting script to trawl through metamist and build a config file suitable for starting a new GATK-SV run.

Closing as superseded


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


gulp install throwing error.

Hi,
I am unable to install gulp on mac. I ran following command to install gulp
$sudo npm install --global gulp

CLI throws following error,
Couldn't install optional dependency: No compatible version found<EMAIL_ADDRESS>Thanks in advance..

This isn't a gulp issue.  It is most likely caused by your version of node.

@phated Thanks for the reply. I have latest node (v4.2.2) and npm(v3.4.1) on my mac. Kindly let me know if you have any suggestions to resolve this issue?