Create a module/plugin to support deployment with the Ledger Hardware Wallet

Outline
The goal of this task is to create a module/plugin for Embark that supports deploying smart contracts authorizing each of them to deploy with a Ledger Hardware Wallet.
The deployment config in config/contracts.js should be something like:
module.exports = {
  ...
  "develop": {
      "deployment": {
        "accounts": [
          {
            "HardwareWallet": "Ledger"
          }
      ]
    }
  }
}

then, with the Ledger connected, doing embark run should deploy the contracts. Each deployment needs a confirmation from the ledger device".
Important:
Keep in mind that the default environment in contracts.js will be merged with any other environment that's being used (develop being the default), which connects to the locally installed blockchain by default. It's required that it doesn't matter what Embark connects to.
resources

API: https://github.com/LedgerHQ/ledgerjs
Provider example which is key: https://github.com/LedgerHQ/ledgerjs/blob/master/packages/create-dapp/template/src/wallets.js

Acceptance Criteria

It should be created as an isolated Embark module (lib/modules) & All major functionality should be there
The scenario described above should work for the testnets (e.g kovan) and livenet

@iurimatias ready to start working on this and turn it around quickly, so hoping i'll be considered. thanks 🙂

@mul1sh approved. However please joins us at the gitter channel to chat about this.

@StatusSceptre definitely 🙂  and thanks for approving to work on this bounty, will work hard to deliver my very best

Hi @mul1sh , please feel free to drop by the gitter channel, we can give you some pointers on how to best approach this.

@iurimatias  thanks will do 👍

hmm @gitcoinbot already submitted a PR

@gitcoinbot yep i'm rebasing the code 🙂

@gitcoinbot still rebasing 🙁

@gitcoinbot 1 more commit remaining 😬

@mul1sh looks like @PascalPrecht is asking for feedback in #901 on if you need anymore help! Let us know if you're still working on this.

@StatusSceptre sorry for the delay with this issue, i'll sort it in less than 24 hours. Thanks

@PascalPrecht Could you confirm if this bounty is complete? It went many different directions and i'm not sure if i'm able to payout.

@StatusSceptre No unfortunately, after several iterations, this PR was not fully functional. Latest news on that is that the contributor broke his Hardware Wallet and therefore "paused" the work.
We decided to keep the PR open in case somebody wants to pick it up.

Closing this one for now. The effort has been put on a break


Bump version to 0.9.0

Can you release a new version? If there is no reason.

https://github.com/wltsmrz/nim_zstd/releases/tag/v0.9


chore: Testing some CodeCov configurations

Overview

Test Plan

Changelog

Review requests

Risk assessment

Conclusions: I can't figure out how to get rid of the CodeCov ✅/❌ check without getting rid of the annotations in the "Files changed" tab. :(


[bug] never gonna give you up

Describe the bug
No response
To Reproduce
No response
Expected Behavior
No response
Screenshot/ Video
No response
Additional context
No response

What? 😂


Return to the upstream CUB

Upstream CUB was cleaned to reduce the size of the history and uses 20 MB now.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!
Also, do please check out our forum (and Chinese version) for general "how-to" questions.


a bunch of updates

Changes:

old .unitypackage deleted
AndroidWrapperProject added

Note:

We don't need old .unitypackage, because we have latest .unitypackage (sorry, they have different names)
We need to keep AndroidWrapperProject otherwise nobody will be able to change android wrapper for Unity

:+1:


restart debugging for Debug R-File broken

The restart button (Ctrl+Shift+F5) seems broken when using Debug R-File.
System
docker image rocker/r-ver:4.0.0
Workaround
Stopping debugging and restarting seems to work.

Thanks for catching that!
The Restart button is currently being "abused" to exit a function without terminating the R session itself (the "Stop" button always terminates both the function and the R session itself).
This was meant as a temporary workaround, which I forgot about. I'll look into that, should be easy to fix :)


Add functionality for Search Option in backend

A search bar is present in navbar but it doesn't perform any functionality now so Create an API for search option in backend so it returns the notes with titles that are matching with the text present in search bar.

Can you assign this to me @Sujal-Sahu ?

I have implemented it, you can view in the pull request #6

Great @subinsk


Feat/tertiary button updates

pr-build broken on branch, changes moved to new branch
https://github.com/Availity/element/pull/136


fix issue 5569.

I think issue #5569 has been fixed.
I didn't find which code cause Illegal instruction on riscv64 directly.
But I fix some code will cause some other problem.
Which will lead reg alloc been different and something else.
Right now We can pass the test.

When I investigated that issue I think it was due to a veneer being inserted right in the middle of a load_constant sequence, that uses fixed offset jumps. Its likely that the other fixes changed the generated code enough that that particular test case no longer triggers that same issue.
I'll run the fuzzer again to find another counterexample.

Yeah, sounds reasonable. @afonso360

I think I have locate the problem.
Here I put a Minstrel::RawData.
https://github.com/bytecodealliance/wasmtime/blob/main/cranelift/codegen/src/isa/riscv64/inst/emit.rs#L80
And when emit I try to emit_island.
https://github.com/bytecodealliance/wasmtime/blob/main/cranelift/codegen/src/isa/riscv64/inst/emit.rs#L473
Those load constant instruction must execute together.
I also think this struct LoadConstant should only used in emit stage not lower stage.
I test at 299be327d.

That looks like it could fix the issue! I'm going to run the fuzzer for a while to check if it finds another counter example on this branch!

It ran the fuzzer overnight without crashing, It looks like it's fixed!

@afonso360 I think this issue is fixed.:-)

@afonso360 What are we waiting for?

Hey, someone else has to review this and merge it.

@afonso360 ok, thanks


update regex to accomodate the new format for playlists while maintai…

…ning backwards compat for the old structure

@garetmckinley Thanks for merging. Will you create a release so this update is available through package managers?

Yes :) currently doing some maintenance on another repo, I'll switch up and cut a release later this evening. Thanks for the PR :trophy:

@stvedt just released v2.1.0, sorry for the delay.

Thanks so much!!

@garetmckinley Not sure if I'm missing something but the latest release on npm does not include the latest source. The tag on GH certainly does but https://registry.npmjs.org/gatsby-remark-embed-spotify/-/gatsby-remark-embed-spotify-2.1.0.tgz doesn't appear to have the new regex. Please advise.

ah, I think I see what happened. It looks like you modified the regex of the compiled index.js file and not src/index.js. So when I published it, the build step must have overridden the changes. If you wanna make a new PR, feel free too. Otherwise I'll be able to make the change myself later on tonight :ok_hand:

Oh, snap. That would do it. I'll submit another PR shortly!
On Thu, Oct 31, 2019 at 1:02 PM Garet McKinley<EMAIL_ADDRESS>wrote:

ah, I think I see what happened. It looks like you modified the regex of
the compiled index.js file and not src/index.js. So when I published it,
the build step must have overridden the changes. If you wanna make a new
PR, feel free too. Otherwise I'll be able to make the change myself later
on tonight 👌
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/garetmckinley/gatsby-remark-embed-spotify/pull/4?email_source=notifications&email_token=AAOHF4M6BE62NYLPNZJH4O3QRMFT7A5CNFSM4JA6JY4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECYQSUA#issuecomment-548473168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOHF4ODDIQZ2PV3CJFVV23QRMFT7ANCNFSM4JA6JY4A
.


CLOUDSTACK-10026 Page for Internal LB VM stucking while loading

When the page for internal LB VM's is opened, it is continuously loading but not giving list of internal LB VM's

LGTM for Testing.
Internal LB VMs are listing properly.

LGTM.


Philippines disease download scripts

I will work on this. -Cecille

Pull requests performed for PH_malaria.py (download script) and PH_malaria_clean.py (cleanup script)


Remove deleted pages from S3

Currently, deleted pages remain on S3 unless the user manually removes them. We should detect 404'd pages and delete them from the bucket.
Reported at https://staticword.press/t/deleted-pages-dont-disappear-from-s3/477

I didn't used to care about this too much and still don't like the idea of the plugin trying to decide what needs to be deleted (maintain state, which we can't unless we were the one to create the S3 bucket and it was never modified by any other process).
What I would like, to solve the use case of clients requiring this behaviour, is to instead, offer an option to "Empty bucket before deployment".
As they'll have Cloudfront in-front, shouldn't cause site issues thanks to caching. The CF invalidation request comes at end, at which point, site is in expected state.
If that's not enough for a user's requirements, I'd propose that they are a power/enterprise user to the point that they could either:

setup a blue/green deployment script, which publishes to a new bucket, then updates the CF distribution

or

employ the WP2Static crew to setup an Enterprise solution for them.

Emptying the bucket won't work because CloudFront's caching is "lazy". Most of the site will not be in-cache in most edge locations and will result in 404s.
I think it's okay for WP2Static to maintain the state of the objects that it created. This seems like expected basic functionality to me that really should be there, but I think there isn't a way to track 404s to do this properly. That may change with the advanced-crawling features, and then this will be pretty simple to implement.
Blue/green would be ideal though.

Oh, bummer about the CF caching.
OK, sounds totally reasonable, thanks @john-shaffer!
Could there be an option:

[ ] delete files we know should be deleted
[ ] delete any files not from our deployment

Or am I over-complicating things further?

Hi, just checking if there was any progress on this feature request?
I agree with @john-shaffer about considering this a basic functionality. Let me share a few examples where it's essential for us:
We have a few brands in the tourism industry.  There are several situations where we need to remove a page. The most frequent ones are:

When a promo is no longer active: we run promotions at certain times of the year, which are also referenced from social media or partner websites. We need to be able to remove those promo pages when they expire.
When a tour/product is on hold or permanently removed
On the Jobs section, when a position is no longer available

Our current workaround is creating a redirection to the homepage, but for certain situations (usually for security or privacy reasons) we need the websites to return a 404 error. In this last case, the content managers have to contact our hosting provider (different timezone) and coordinate a time when they can empty the S3 bucket, and at the same time, we export a new version of the site.
Hope those comments help to give more context and ideas as to why some of us consider this an important feature.


Support inet/cidr operators

As pr. Gitter chat.
For a project, I need to be able to perform containment/comparison operations on inet/cidr types, so the operators in the below documentation would be awesome to have :).
https://www.postgresql.org/docs/9.3/static/functions-net.html

@LordMike can you please provide some more detail on exactly which operators you need in your application?

For this specific project, contains-or-equals operators. So: <<, <<=, >> and >>=.

It looks like this could be implemented in the same way as #323 (extension stubs + attributes).
@roji If no one is actively working on this, I'm happy to volunteer to take this one.

@austindrenski you're welcome to work on this, it should be even easier than ranges (no generics). However, please take into account my comment on the design before starting to implement, so we're agreed on how we want translators to look like.

This feature was completed in #407 and #535.


Options for removing or changing prefix commands for clearing ACLs

@fischa made mention of desiring to output Cisco ACLs without any of the included purging commands such as no ip access-list foo
This is to help in use with config creation and deployment tools that can do the proper exchange of configs on their own.
We should be careful in how we handle this as it could make for breaking changes for other users.
At first glance options may be a good solution but this could have a chance to explode the number of options. There is a desire to add a clear statement in https://github.com/aerleon/aerleon/issues/207
Another option would be to allow for prefix commands or strings to be allowed at both the filter and policy level.
@jtwb for visibility.
In either case this likely will need to be put on the back burner till we move the project over to YAML only as the options as they are now in the pol format are not workable.

Thx for picking up this topic. I think it is easier to understand the idea by clarifying on the two use cases I see:

updating/replacing an existing filter (config snippet approach)
updating a full configuration incl. filters (full config approach)

Aerleon is serving currently only case 1 and for case 2 the additional commands for cleaning out/replacing a filter is not required or might even break things. So case 2 is mostly an include of the "pure" generated filter.


Add feature to let selected programs bypass the User Account Control prompt when run

Description of the new feature / enhancement
A new feature that would allow the user to disable the User Account Control prompt for certain programs.
Scenario when this would be used?
This has been a feature that has been requested to Windows for years. A lot of people wanting to automate tasks or scripts have been frustrated with the UAC prompt adding an extra step. It seems like the only solution is to still create a scheduled task to work around it.
Supporting information
https://superuser.com/questions/1607838/disable-uac-for-a-specific-program-windows-10
https://answers.microsoft.com/en-us/windows/forum/all/disabling-uac-approval-for-specific-programs-in/745a3740-679a-42f9-9c51-59475c9cd2ff
https://answers.microsoft.com/en-us/windows/forum/all/disable-uac-for-1-program-only/14a7698b-ca8f-42e7-9426-57b5345a2453
https://answers.microsoft.com/en-us/windows/forum/all/how-to-always-allow-a-certain-program-to-bypass/e9bbd7c3-ea32-4b75-b11a-5c9073b0612a
https://answers.microsoft.com/en-us/windows/forum/all/disable-uac-prompt-per-program-without-shortcut/b55285dc-29c0-46e1-81b1-b86030875767

Apart from the huge security risks I see with such a feature, is this something that PowerToys can't do. It would have to be built into Windows.
/feedbackhub


[WIP] Introduce Traceformat to allow JSON format log

Things I plan to do but haven't done yet:

[ ] When verbosity is set to message, ignore params on Request / Notification messages
[ ] Expose traceFormat to client's initialization params

Questions:

[ ] Can we rename this to [langId].lsp.trace? server is misleading as the logs are actually logged from the client perspective. I'll write documentation for this too so I hope we can get this right.

@dbaeumer Would like to get some high-level insight - let me know if this is the right direction, what more things you would want to me to add, etc.
More details are at https://github.com/Microsoft/language-server-protocol-inspector/issues/8

@octref goes absolutely into the right direction. Only one little comment so far.

Hey Dirk, there are a bunch of updates that happened, so I'm wondering if you can take another look at this PR. Also /cc @aeschli in case you are interested.
High Level Changes
This PR remain as it is, so:

trace.server continues to work
trace.server can be an object now, for example:"mls.trace.server": {
  "verbosity": "verbose", // values same as old `trace.server`
  "format": "json" // or `text` for old behavior
}

For streaming support, it can be implemented trivially by Language Server authors using the existing API LanguageClientOptions.outputChannel. For example: https://github.com/octref/vscode-language-server-template/commit/4181af806233f98f2dd654968e4171ee1dabcc33.
This goes against what we initially agreed on, e.g:
{
  "format": "json" | "text",
  "verbosity": "off" | "messages" | "verbose",
  "output": "channel" | "log" | "websocket"
}

the benefit of leaving out logging / websocket-streaming is that we don't add dependencies (ws for websocket) and we give ext authors more freedom on handling logs.
How Things Work Now

There are 3 repos:

octref/vscode-lsp-inspector: The Inspector in webview with streaming support. It:

launches the web app in webview
opens a Websocket server to listen to incoming connections that would send LSP log data
sends LSP log data into the web app and update UI.

Microsoft/language-server-protocol-inspector: I'll merge octref/vscode-lsp-inspector into this. So this repo will have 2 build targets. One is on LSP Website. The other one is the webview extension.
Microsoft/vscode-languageserver-node: People using the new version of the lib can specify output format of the LSP log. They can then save the log, stream it into the Inspector or do whatever they want.

Play With It

Download the VSIX: https://github.com/octref/vscode-lsp-inspector/releases/tag/v0.0.1
Build and Run the stream branch of: https://github.com/octref/vscode-language-server-template/tree/stream
Open a MD file. The LS should connect to the LSP Inspector Webview extension's WebSocket and start streaming data into it.

@octref provided by comments.

Addressed comments, but left some questions too.


Refactoring Twitter tokenizers

Twitter tokenizers go into io.anserini.analysis package. Ported test case from https://github.com/lintool/twitter-tools

not very familiar with this, but LGTM


Simplify temporary_directory and use safe_rmtree

Resolves: https://github.com/python-poetry/poetry/issues/3493

[x] Added tests for changed code.
[ ] Updated documentation for changed code.

Unit test fails without changes and passes with changes 🎉

Yes


Add Zulip support for message URI

This is needed for file issue to record where the command was given
Opened by @jayvdb

Rather than add it to utils, I've created an issue upstream https://github.com/zulip/errbot-backend-zulip/pull/6, and sent a WIP-ish patch upstream https://github.com/zulip/errbot-backend-zulip/pull/7 .
I suspect the Zulip backend cant easily build these URLs.  At least some URL creation algorithms need to be added to the python-api-library.
c.f. https://github.com/coala/corobo/issues/394
As this may take a while, https://github.com/coala/corobo/issues/392 is the better solution for now.

Merged upstream


Update entrypoint.sh

replaced hardcoded fedora release by regex  - this way the entrypoint scripts don't have to be different for fc34 and fc35

@stefanleh this is a great idea! 🎉


Convert idea and eclipse plugin to lazy configuration

Context

See: https://github.com/gradle/gradle-native/issues/630
Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation
[ ] Recognize contributor in release notes

This PR is ready for another round of reviews.

If I read the CI error properly, it seems the issue is flaky and is disabled in master. So we can merge this once we get another round of review.


Init version for Oracle with visitor

For #4308.

Pull Request Test Coverage Report for Build 1732

0 of 205 (0.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 62.028%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleTCLVisitor.java
0
5
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/OracleParserEntry.java
0
9
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/impl/OracleDCLVisitor.java
0
22
0.0%

shardingsphere-sql-parser/shardingsphere-sql-parser-oracle/src/main/java/org/apache/shardingsphere/sql/parser/visitor/OracleVisitor.java
0
169
0.0%

Totals

Change from base Build 855:
-0.7%

Covered Lines:
11049

Relevant Lines:
17813

💛  - Coveralls


The fetch_links=True in beanie queries doesn't work

So after getting a document calling fetch_all_links() works however when using the keyword argument it doesn't. Thus you can't do queries comparing the linked document properties. For example the below would not work:
doc = await House.find_one(House.front_door.color == 'Blue', fetch_links=True)

This looks like same issue with mongomock's implementation of DBRef (which is the core of beanie's links) as discussed in #5. I've created issue at mongomock (mongomock/mongomock#759), but I don't really want to patch internals of mongomock in this library. So currently support for links are limited.

Thanks so much! In the interim, I changed my code to use fetch_all_links in an async for loop and for cases where I need to use fetch_links I skipped these unit tests for now. I have integration tests to cover these cases. However it would be great to have this working.

Hello, Can you fix this?
it shows NotImplementedError: Although 'pipeline' is a valid lookup operator for the aggregation pipeline, it is currently not implemented in Mongomock.

What exactly is not working for you? This issue refers to the internal implemetation of beanie that doesn't support some operations with DBRef that mongo supports (and beanie uses).


Bad quality of synthesis audio

I trained the model with default params for a new 10-hour dataset but the audio sounds not good. Also with the demo of pre-trained model, the synthesis audio sounds not very good. Anyone know the reason  and how to improve it?  Thank you.

Solved it because of my wrong complie.


feat: text and branding

This PR adds the landing page text and makes some updates to the branding. The CityLAB is now also visually included in header and footer.

I will have a look after lunch! Pretty excited already! :D

Self reminder: replace the link of the CityLAB logo in the footer with the correct one before merging.

@dnsos I really like the new TTN info box. Fits perfectly to what we are saying in the paragraph to the left and doesn't distract from the main content. good job

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


Survey

It's a short survey, I should help us build our ROADMAP.

Moving it to an internal Google Docs.


Updated Damian's profile

New location and more accurate bio.

Docs Build status updates of commit a0ea3ee:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

advocates/damian-brady.yml
:white_check_mark:Succeeded
View

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


Update of Polish translations

Adding missing translation strings.
ps. Maybe you should use some external translation management system?

Thanks. I want to build a CLI for translation management. Haven't found any other good systems yet.


Fix #18 - Enable passing JSON payload to Slack App Route

Summary
The goals of this PR is to enable BotToken users to leverage the payload to pass json to a bot and resolves #18
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

@thatrandybrown Thank you so much for sending this PR in! It looks great. Love that you added some examples too


⚠ Samsung has degraded performance

In b9c9e06, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2955 ms

Resolved: Samsung performance has improved in d99aea8.


Update build.sh to use PWD

Prerequisites

[x] I have written a descriptive pull-request title
[x] I have verified that there are no overlapping pull-requests open
[x] I have verified that I am following the Nancy code style guidelines
[x] I have provided test coverage for my change (where applicable)

Description
This will set the path to PWD so that bootstrappers/serializers can call dependencies/Nancy/build.sh and all will be well

@jchannon guessing build.ps1 needs updating as well?

Also ❤ this is going to make our lives SO much easier 😄

Yup but you can do that for me on Windows ;)

Aight let's pull this one then


SASS in css

Issue description:
SASS $vars are still present in the css file:
width:calc(100% * $count / $perpage)

Also with my build was getting an error until the $vars inside the calc where wrapped (in the scss file) ie:
width:calc(100% * #{$count} / #{$perpage})

Tiny-slider version:   2.8.0

Added a PR https://github.com/ganlanyuan/tiny-slider/pull/227

thanks


After update to 0.131.9 mg2mqtt vehicle template throws fatal invalid key error

Describe the bug
Due to the changes in #17650 the mg2mqtt vehicle template does not wok anymore, as the template needs the user and vin properties. But when configuring the vehicle (which worked in 0.131.8), now there is a fatal error when trying to start evcc.
After the refactoring the template only inherits vehicle-common, which does not include user and vin.  vehicle-base can not be used either, because the mg2mqtt template does not require a password, which is set as required in vehicle-base .
Steps to reproduce

Configure vehicle using template
Start evcc
Fatal error when starting

Configuration details
vehicles:
  - name: mg_zs_ev
    type: template
    template: mg2mqtt
    title: MG ZS EV
    capacity: 50.3
    user: *REDACTED*
    vin: *REDACTED*
    phases: 1

Log details
[main  ] FATAL 2024/12/12 13:09:19 cannot create vehicle 'mg_zs_ev': cannot create vehicle type 'template': invalid key: vin

[main  ] FATAL 2024/12/12 13:09:19 will attempt restart in: 15m0s

[main  ] INFO 2024/12/12 13:24:21 evcc 0.131.9

[main  ] INFO 2024/12/12 13:24:21 using config file: /etc/evcc.yaml

[db    ] INFO 2024/12/12 13:24:21 using sqlite database: /root/.evcc/evcc.db

[mqtt  ] INFO 2024/12/12 13:24:21 connecting evcc-577169755 at tcp://mosquitto:1883

[main  ] INFO 2024/12/12 13:24:21 listening at :7070

[main  ] FATAL 2024/12/12 13:24:22 cannot create vehicle 'mg_zs_ev': cannot create vehicle type 'template': invalid key: user

[main  ] FATAL 2024/12/12 13:24:22 will attempt restart in: 15m0s

What type of operating system are you running?
Linux
Nightly build

[X] I have verified that the issue is reproducible with the latest nightly build

Version
evcc version 0.131.9

Closed in https://github.com/evcc-io/evcc/commit/95d04fa7129c4b00a68d817522d1a8af830fcb74


howto: Run timer unaffected by X11 activity?

Is it possible to use xidle to run a timer that isn't affected by whether X is idle or not?
Like a pomodoro timer that runs the screen locker after 25 minutes of the screen being unlocked?
this would be a separate timer from the one that is running after 5 minutes of inactivity.

I'm not sure I understand, you want it to run the screen locker 25 minutes of the screen being unlocked, regardless of activity?

Yeah, with a primary timer based on inactivity that locks after 5 minutes resetting both timers, or vice versa. It's basically a way to enforce the pomodoro technique so I don't spend 6 hours sitting at my computer and forgetting to do things like eat.

Ah, so you want to lock every 25 minutes, unless you leave your computer for 5 minutes which will also lock?
So, correct me if I'm wrong, a day for you could look like:

25 minutes work, then lock
take a 3 minute break
25 minutes work, then lock
take a 3 minute break
7 minutes work, then you go take a coffee
5 minutes inactivity, then lock
you unlock and continue work
25 minutes work, then lock
... and over

Then I'd recommend using a bash script for that 25 minute timer, something like
while true; do
    sleep 25m
    xidlehook-client --trigger --socket <path>
    <wait until some ipc message sent by your lock script after unlock>
done

thanks, I'll get this implemented

I am currently experimenting with a library-based xidlehook which would allow you to make your own module that does what you want.

xidlehook has been rewritten to be a library (see !6). This lets you register any kind of timer using traits. Feel free to PR something that adds such a module if you deem it useful to the majority of xidlehook users. You can also extend xidlehook yourself using the awesome library system, essentially wrapping it with a more fine-tuned configuration using rust code.


Time does not update (quickly) after app is backgrounded.

To repro: start the app, send it to the background, bring it to the foreground some time later, the stars will be out of position.

This is an old bug, right? I seem to remember this being an issue back in
the day.
On Thu, Jun 2, 2016 at 11:47 PM, John David Taylor<EMAIL_ADDRESS>
wrote:

To repro: start the app, send it to the background, bring it to the
foreground some time later, the stars will be out of position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/120, or mute the thread
https://github.com/notifications/unsubscribe/ALFRPUD0f50FwoEzMHW1t1bXqpW2Ean9ks5qH6O8gaJpZM4ItLeC
.

Could be a dupe - I just noticed it when testing the gyro stuff.
On Fri, 3 Jun 2016 at 04:37 KevinSerafini<EMAIL_ADDRESS>wrote:

This is an old bug, right? I seem to remember this being an issue back in
the day.
On Thu, Jun 2, 2016 at 11:47 PM, John David Taylor <
<EMAIL_ADDRESS>
wrote:

To repro: start the app, send it to the background, bring it to the
foreground some time later, the stars will be out of position.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/120, or mute the
thread
<
https://github.com/notifications/unsubscribe/ALFRPUD0f50FwoEzMHW1t1bXqpW2Ean9ks5qH6O8gaJpZM4ItLeC
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sky-map-team/stardroid/issues/120#issuecomment-223556995,
or mute the thread
https://github.com/notifications/unsubscribe/APq7C9-hW-U0VFncHtBLN9uyaWqtSkCIks5qIBHjgaJpZM4ItLeC
.


Make python-seco-range compatible with both Python 2 and 3

The goal of this is to make python-seco-range compatible with Python 3 while
still remaining compatible with Python 2.

Opened this PR in case any other poor souls out there still need python-seco-range now that Python 2 is EOL

For anyone else that finds this, I'm sorry you're also still stuck using Range. I've also uploaded a purely Python 3 version of python-seco-range to PyPi named python3-seco-range. It probably has bugs but feel free to make PRs against my fork I guess. Good luck and god-speed.

Merged! Is range really that horrible for you? It's been a while, but I loved it when I was using it.

@nmeyerhans Wow, I actually didn't expect my PR to get merged, thanks!
Range itself is not horrible, it actually works quite well. The problem is that it's not maintained as evidenced by my PR being the first commit in 6 and half years. And there is very limited documentation (unless there's documentation somewhere I haven't found).


Test PR

This is a test PR.

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


V0.12.0 throws error with EXPO SDK 52

Unable to resolve "src/core/utils" from<EMAIL_ADDRESS>

I need this fix ASAP

Fixed for expo sdk 52. Tested by creating new expo sdk 52 project, installing dependencies, and testing library. Please reopen if any issues.


Do we need country (currently called source) as an input to init Survey?

right now only US+Canada (country=3) is implemented
did Chu implement country=1 and country=2?

Related to #93

I'll close this and label #93 high priority.


Intervals not respecting dashboard interval

I did not understand the use case of having interval option as part of the UI.

I always thought that interval should come from grafana and we should use that

You can see an example here: https://github.com/grafadruid/druid-grafana/blob/master/dashboard.json#L171
You have to use ${__from:date:iso} as interval start and ${__to:date:iso} as interval stop.
Basically the component will detect the $ and will then use the variable and print the variable as the input field placeholder.

You can see an example here: https://github.com/grafadruid/druid-grafana/blob/master/dashboard.json#L171
You have to use ${__from:date:iso} as interval start and ${__to:date:iso} as interval stop.
Basically the component will detect the $ and will then use the variable and print the variable as the input field placeholder.

All the plugins on grafana respect that time range filter. It would be odd to expect users to put this in.
I think what we can do is if no interval is selected then default it to grafana. If interval are user entered then use that.
So this way normal requests are handled without user having to understand intervals.

All the plugins on grafana respect that time range filter. It would be odd to expect users to put this in.
I think what we can do is if no interval is selected then default it to grafana. If interval are user entered then use that.
So this way normal requests are handled without user having to understand intervals.

Here it is @Gauravshah : https://github.com/grafadruid/druid-grafana/pull/31
Feel free to review it :)
Basically it defaults to the Grafana interval if empty.

Here it is @Gauravshah : https://github.com/grafadruid/druid-grafana/pull/31
Feel free to review it :)
Basically it defaults to the Grafana interval if empty.

thanks @jbguerraz

thanks @jbguerraz


[FLINK-4359] [table] Add INTERVAL type

Thanks for contributing to Apache Flink. Before you open your pull request, please take the following check list into consideration.
If your changes take all of the items into account, feel free to open your pull request. For more information and/or questions please refer to the How To Contribute guide.
In addition to going through the list, please provide a meaningful description of your changes.

[x] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[x] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[x] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

This PR introduces INTERVAL types for Table API/SQL. Here are some examples how intervals can be used in the APIs. More examples can be found in TemporalTypesTest.
 testAllApis(
    1.year,
    "1.year",
    "INTERVAL '1' YEAR",
    "+1-00")

testAllApis(
    'f0 + 2.day,
    "f0 + 2.day",
    "f0 + INTERVAL '2' DAY",
    "1990-10-16")

@aljoscha would be great if you could have a quick look on it.

Is the INTERVAL prefix needed to make it work with the parser? Otherwise it would be nice to just write 2 DAYS, or something like that.
By the way, the reason for the two different interval types is that the millis type does not have enough range to represent years?

The first line is Scala Table API, second line is Java Table API and the string with the INTERVAL prefix is standard SQL. So we cannot change the standard ;)
Yes, I think that is the reason. Calcite does it like that and Oracle too.
https://docs.oracle.com/cd/B19306_01/server.102/b14200/sql_elements003.htm#i38598

Ok, I figured as much. A bit ugly but I guess everyone has to do it like this. 😅
From a quick glance the API and implementation look good. 👍

Merging this...


Unable to move row and row select

Hello Paul i hope you are well
i updated the jexcel to the most recent release and i can't tell is trully a success
Yes now we can edit a cell while pressing the shift key
but my users and i experience now a down feature wise
we cannot drag and drop row anymore and when we try the row is selected
we can even select more then one row
i kept a previous version so for now we are good
i downloaded yesterday the 13 november

You are right, sorry about it... it has been fixed.


Add text origin

@ajpmaclean : Please Review.

@ajpmaclean : Please Review.

Missing TestTextOrigin.png

@ajpmaclean : Updated test image added. Please review.

Thankyou again.


Enhanced Ui for login.html

This pull request enhances the UI of the login page.
Improved the design of the register.html page by adding better styling for a modern look.
Updated the CSS for more responsive layouts.
-Added hover effects at each logos
-Animation effect while visiting the page
Added enhanced form validation for password strength and matching.
Testing:
Tested on Chrome, Firefox, and Edge.
The form works as expected with visual improvements.
UI BEFORE:

UI AFTER:

@sohailshk Great contribution to Hacktoberfest 2024! 😊 Please include the issue number next time to improve tracking. Thanks for your effort! 🙌


raftstore: directly write kvs rafther than ingestion when merging small regions.

What is changed and how it works?

Issue Number: Close #17412, #17376

What's Changed:
Previously, snapshots of regions, with the exception of the `lock_cf` column family,
were flushed to SST files, while `lock_cf` was flushed to plain files. This approach could
lead to a high volume of `ingest` tasks for merging regions, especially when the TiKV
node contained a large number of small-sized regions.  As we all know, an excess of `ingest`
tasks can significantly impair the overall performance of the TiKV node.

To address this issue, this pull request introduces a new configuration option called
`snap_min_ingest_size`. When a region's size is below the threshold set by `snap_min_ingest_size`,
its corresponding snapshot will now be applied without `ingest` SST files. This change
is expected to reduce the frequency of ingest tasks and thereby enhance the TiKV node's processing efficiency.

And the following comparison shows that this pr can greatly reduce the duration of merging numerous small regions.

Nightly(222e0a)
This PR

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

To mitigate the high apply wait duration problem, it directly dumps and flushes key-value pairs in sst files of snapshots when merging small regions rather than ingestion.

Proved to be effective to reduce the triggers of ingestion when merging small regions.

Is it backward compatible?

yep, the newly introduced trait use_plain_file, used to mark whether the persisted snapshot uses plain file or not, is a new member of kvproto::raft_serverpb::SnapshotMeta. And it will be serialized into the meta file of the snapshot, where the proto has ensured the compatibility.

... while raft_cf was flushed to plain files.

Do you mean "lock_cf"?

To address this issue, this pull request introduces a new configuration option called
snap_min_approximate_size. When a region's size is below the threshold set by snap_min_approximate_size,
its corresponding snapshot will now be written to plain files instead of SST files. This change
is expected to reduce the frequency of ingest tasks and thereby enhance the TiKV node's processing efficiency.

Another approach could be that the senders continue sending SST files while the receivers check the SST file size and iterate over the file if it’s small. This way, we wouldn’t need to modify kvproto.

Do you mean "lock_cf"?
Yep, thx for reminder, it's corrected.

Another approach could be that the senders continue sending SST files while the receivers check the SST file size and iterate over the file if it’s small. This way, we wouldn’t need to modify kvproto.

Maybe it can be further simplified with the following approach, without modifying kvproto =>

It still uses the following strategy to generate the sst files or plain files when building snapshot, depending on the size of the region.
When applying, firstly, try snap_io::apply_plain_cf_file(...) and then snap_io::apply_sst_cf_file(...). If the first apply_plain_cf_file is success, the given cf file should be a plain file. And if the first try is failed but the second apply_sst_cf_file is success, it should be a sst file. In this way, we do not introduce any extra implementations on dumping sst files when receiving either. Moreover, if the both tries on applying were failed, it should be that the given file is broken or invalid for applying.

Another approach could be that the senders continue sending SST files while the receivers check the SST file size and iterate over the file if it’s small. This way, we wouldn’t need to modify kvproto.

Accepted. The whole implementation has been refactored based on this proposal.

/test pull-unit-test

/approve

/cc @zhangjinpeng87 PTAL, thx.

/needs-cherry-pick-release-8.1

/needs-cherry-pick-release-7.5


Sync join and sync leave documentation

@tianyuan129

@tianyuan129 anything I should add? I'm know other documentation are longer but not sure what specifically to explain.

Don't these new documents have to be referenced from the index or some other document? I believe sphinx raises a warning otherwise (and also, how does the reader discover their existence?)

Thanks Davide--you are completely correct. I have now linked them into specification.rst.

I think it's good to go. Thank you so much for the effort!


Update README.md to add Kommander

Project URL
Kommander
Description
Kommander is a Swift library to manage the task execution in different threads. Through the definition a simple but powerful concept, Kommand.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

This change is

          1 Warning

:warning:
Please update the Pull Request title to contain the library name

  Generated by :no_entry_sign: Danger

Thank you for contributing to awesome-ios 😄


Adds role="presentation" to unfocusable anchors

Minor: was trying out this extension from Microsoft (recommended!) and it caught this: we're laying an unfocusable anchor tag over the dropdown items so that we can have the keypress handler open the submenu and any mouse click go to the href — since if you're using a pointing device, the submenu will open on hover.
Adding role="presentation" overwrites the semantics of the given element and ensures it's ignored (it wasn't focusable anyway so I don't believe this was an actual issue but dotting the i here).

Looks good, and always great to learn more about accessibility!


New-AzureRmADApplication - sample code

For my devops project; I'd like to see some sample code that registers my application on apps.dev.microsoft.com as a Converged application rather than just the Azure AD only application.  In the interim, can you point me to an example?

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

ID: af593b9c-8d39-96b4-2812-10c52a79ea2e
Version Independent ID: b5d4e4d7-4b3c-391a-52c4-5103feabb07a
Content: Azure PowerShell Samples - App Service
Content Source: articles/app-service/app-service-powershell-samples.md
Service: app-service
GitHub Login: @SyntaxC4
Microsoft Alias: cfowler

@MouthOfMadness Thanks for the feedback! We are currently investigating and will update you shortly.

I have a couple of accounts, and not all of them show the Converged application, so here’s a screen shot from apps.dev.microsoft.com
<EMAIL_ADDRESS>From: YutongTie-MSFT<EMAIL_ADDRESS>Sent: Wednesday, June 6, 2018 2:38 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: James Mattern<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] New-AzureRmADApplication - sample code (#9797)
@MouthOfMadnesshttps://github.com/MouthOfMadness Thanks for the feedback! We are currently investigating and will update you shortly.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/9797#issuecomment-395188465, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ab4Q-g-GfU2FSnYGRr-0_mKu-RxKOv_sks5t6C-CgaJpZM4Uc3I6.

@MouthOfMadness Can you please post your feedback to the below article? This article is about the Azure Web App product, which is different than AAD applications. This will ensure the AAD doc authors receive the feedback and take the necessary steps. Thank you for your understanding.
https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/what-is-application-management

@BryanTrach-MSFT this doesn't address my issue, as the New-AzureRMADApplciation registers the
application here, just not as a converged application.  I'll attach the original image.
.

I’m not looking to use the Web App Portal, as it’s not something I can automate with a VSTS Pipeline.
The New-AzureRdAdApplication registers with that portal as an AAD application only; work greats I have registration in that portal as well as my Azure subscription.
I just need it to be a converged application.
From: Bryan Trach<EMAIL_ADDRESS>Sent: Thursday, June 7, 2018 4:35 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: James Mattern<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] New-AzureRmADApplication - sample code (#9797)
@MouthOfMadnesshttps://github.com/MouthOfMadness Can you please post your feedback to the below article? This article is about the Azure Web App product, which is different than AAD applications. This will ensure the AAD doc authors receive the feedback and take the necessary steps. Thank you for your understanding.
https://docs.microsoft.com/en-us/azure/active-directory/manage-apps/what-is-application-management
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/9797#issuecomment-395572632, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ab4Q-nC8pAC_pFFFoEn_eAHXsw1OCx3Pks5t6Zx2gaJpZM4Uc3I6.

@MouthOfMadness Converged section only displays the application which are registered with V2 endpoint and the only way to register Azure AD v2.0 applications programmatically at this point in time is through the Microsoft Graph API beta endpoint. However, as this is a beta endpoint it may change in the future and is not recommended for production.
You can refer to MS Graph Getting Started for more information

Thank you.
From: SaurabhSharma-MSFT<EMAIL_ADDRESS>Sent: Friday, June 8, 2018 12:54 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: James Mattern<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] New-AzureRmADApplication - sample code (#9797)
@MouthOfMadnesshttps://github.com/MouthOfMadness Converged section only displays the application which are registered with V2 endpoint and the only way to register Azure AD v2.0 applications programmatically at this point in time is through the Microsoft Graph API beta endpointhttps://developer.microsoft.com/en-us/graph/docs/api-reference/beta/resources/application. However, as this is a beta endpoint it may change in the future and is not recommended for production.
You can refer to MS Graph Getting Startedhttps://developer.microsoft.com/en-us/graph/docs/concepts/get-started for more information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/9797#issuecomment-395838390, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ab4Q-i5YYq9dkRdowc0kuy5W4nuuR8EGks5t6roygaJpZM4Uc3I6.

I'm very successful in getting the token for this uri and registering an application:
https://graph.windows.net/$tenant/$($resource)?api-version=1.6
Unfortunately, this is not a valid token for the beta website.  https://graph.microsoft.com/beta/$tenant/$($resource)
If I use the beta website to login, I get the exception:
Exception calling "AcquireToken" with "4" argument(s): "authority_not_in_valid_list: 'authority' is not in the list of valid addresses"
Any guidance on how to generate a valid token for the beta website?
From: SaurabhSharma-MSFT<EMAIL_ADDRESS>Sent: Friday, June 8, 2018 12:54 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: James Mattern<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] New-AzureRmADApplication - sample code (#9797)
@MouthOfMadnesshttps://github.com/MouthOfMadness Converged section only displays the application which are registered with V2 endpoint and the only way to register Azure AD v2.0 applications programmatically at this point in time is through the Microsoft Graph API beta endpointhttps://developer.microsoft.com/en-us/graph/docs/api-reference/beta/resources/application. However, as this is a beta endpoint it may change in the future and is not recommended for production.
You can refer to MS Graph Getting Startedhttps://developer.microsoft.com/en-us/graph/docs/concepts/get-started for more information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/9797#issuecomment-395838390, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ab4Q-i5YYq9dkRdowc0kuy5W4nuuR8EGks5t6roygaJpZM4Uc3I6.

Was the beta API recently disabled?
I started receiving this error:  The remote server returned an error: (405) Method Not Allowed.
From: SaurabhSharma-MSFT<EMAIL_ADDRESS>Sent: Friday, June 8, 2018 12:54 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: James Mattern<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] New-AzureRmADApplication - sample code (#9797)
@MouthOfMadnesshttps://github.com/MouthOfMadness Converged section only displays the application which are registered with V2 endpoint and the only way to register Azure AD v2.0 applications programmatically at this point in time is through the Microsoft Graph API beta endpointhttps://developer.microsoft.com/en-us/graph/docs/api-reference/beta/resources/application. However, as this is a beta endpoint it may change in the future and is not recommended for production.
You can refer to MS Graph Getting Startedhttps://developer.microsoft.com/en-us/graph/docs/concepts/get-started for more information
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/9797#issuecomment-395838390, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Ab4Q-i5YYq9dkRdowc0kuy5W4nuuR8EGks5t6roygaJpZM4Uc3I6.


Migrate to KX 13.0 NET4.8

Hey Mike!
I noticed you're working on a KX13 upgrade for this package but it is in a state of not working due to the requirements of System.Web features and trying to target .netstandard2.0.
You'll either need to use compiler directives to use the same source to produce code that will be able to access web.config with all the APIs you're used to, and work in ASP.NET Core.
I made a quick update to target KX13 and avoided the issue by directly targeting net48 and leaving out handling ASP.NET Core.
I really think, if that's still a goal, it should be a separate project and the IConfigurationHelper and WebConfigSettingsRepository live in a net48 project while a netstandard2.0 project contains all the shared types.
The ASP.NET Core project (likely targeting netcoreapp3.1 or preferably net5.0) would only need a ISettingsRepository implementation.
As things stand in my brand everything is working (tests pass) for using this in the Content Management app.
The README.md still needs updated.

Thank you!. I'm merging this branch and I'll upload the nupkg.  I had a feeling folks where waiting for this.
I'll take a closer look at your recommendation after when I get a breather.  Trevor brought up the need to include this in .NET Core projects so that it can filter staging tasks like users created during registration.

Thank you!. I'm merging this branch and I'll upload the nupkg.  I had a feeling folks where waiting for this.
I'll take a closer look at your recommendation after when I get a breather.  Trevor brought up the need to include this in .NET Core projects so that it can filter staging tasks like users created during registration.


Improve Cookie Max-Age

In https://tools.ietf.org/html/rfc6265#page-20 written:

If delta-seconds is less than or equal to zero (0), let expiry-time
be the earliest representable date and time.  Otherwise, let the
expiry-time be the current date and time plus delta-seconds seconds.

This means that the value of Max-Age can be negative. In this case, browsers should set early represented date and time
I left an exception when the Max-Age < -1 because all the other cases do not make sense and is interpreted in the same way as -1. If you set Max-Age=-89219 it looks like a bug (I doubt at this point)
Correct me if I am wrong anywhere.
https://github.com/jayway/rest-assured/issues/580

Thanks for helping out and looking this up.


small typo in truck_meshalgo

in the first sentence in lib.rs it should be "Mesh algorithms" not "Mesh algorighms"

Thank you very much! Now we fixed it!


Update volumes.md

Updating the link for the Kubernetes vSphere Cloud Provider and adding a more info link to the vSphere CSI documentation.

@lparis : Would you please be able to make the requested changes?

@lparis : Would you please be able to make the requested changes?

/sig storage

@lparis Could you please addressed the requested change and rebase occurring for continuing this.
Thanks

@lparis 👋🏻 Please feel free to /reopen when you're ready to rebase and respond to feedback.
/close


mat: don't clobber optimisation for TriDense matrix input

Please take a look.
Fixes #1448.

I didn't look. I will now.

~I didn't look. I will now.~ There are no benchmarks that cover this AFAICS.

Would be nice to have some :)


Long-range teleport doesn't load properly

When teleporting a long distance in the same dimension, the target area doesn't load correctly (even if a chunkloader is present). The area will appear similar to when you fly beyond the edge of the world during chunk generation. However, the server reports that those chunks are loaded. Restarting the client fixes the problem.
Relevant Versions:
Forge: <IP_ADDRESS>8
Ender Core: <IP_ADDRESS>
Ender IO: <IP_ADDRESS>5_beta

It's possible that this is a FastCraft-related bug. Just in case it's related, I'm using fastcraft 1.22-ctest14

Note: Teleporting, then immediately teleporting back works correctly. However, letting the area unload (even if a chunkloader is set up), causes the bug.

This sounds like a performance issue, sometimes MC just takes a long time to generate a completely new area. How long have you waited?

Closing due to inactivity, also possibly fixed by ada5b4fc4189817ba20c1e9c93bdb4c1e8eb8cc7


Junctions and failures inside of them

See RT #131118.
Basically, we should simply mention that failures inside Junctions are OK, but they will blow up when you try to use these Failures unintentionally

Thanks. Fixed in https://github.com/perl6/doc/commit/7628708384bdc049d74c997fc2059cf57fcf5a57


Added partial support for Color temperature, RGB, RGBW lights

I've added code for color temperature and color control.
Color temperature and color can be controlled after changing the device type of a added 'Light' from Generic Component Dimmer to Generic Component CT or Generic Component RGB or Generic Component RGBW in Hubitat.
But only statuses of 'level' and 'switch' are shown in Hubitat.

Nice addition.  Thank you!  I am merging this pull request as it is 100% backwards compatible.  At this time, the user has to manually change the driver of the child device to the proper "Generic Component RGBW" or "Generic Component CT" driver in order to issue these color commands from HE to HA.  At this time, as noted, this is a partial implementation.  Since it requires manual intervention to change the drivers to use the two new functions that this PR adds, there is no risk to existing installations.


execute ./eosio_build.sh

🍺  /usr/local/Cellar/boost/1.66.0: 13,101 files, 435.2MB
Pinning boost libraries.
Boost 1.66.0 successfully installed @ /usr/local.
Checking MongoDB C++ driver installation.

% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
100   616    0   616    0     0    568      0 --:--:--  0:00:01 --:--:--   567
0 7535k    0 34370    0     0     74      0 28:58:00  0:07:39 28:50:21     0
curl: (56) LibreSSL SSL_read: SSL_ERROR_SYSCALL, errno 54
mongo-c-driver-1.9.3/configure: (Empty error message)
tar: Error exit delayed from previous errors.
Unable to unarchive file mongo-c-driver-1.9.3.tar.gz.
Exiting now.

@lilinkai You have a network error and were unable to connect to and clone the github MongoDB C++ driver. Check your network settings or consult with your ISP.


Inference => RuntimeError: index out of range: Tried to access index 300 out of table with 299

Task:
Inference for librispeech
Command:
python examples/speech_recognition/infer.py /home/ahmed/projects/speech_data/librispeech_final --task speech_recognition --max-tokens 250000 --nbest 1 --path model_lib/checkpoint_best_run1.pt --beam 20 --results-path results_dir --batch-size 20 --gen-subset test-clean --user-dir examples/speech_recognition/
Error:
RuntimeError: index out of range: Tried to access index 300 out of table with 299 rows. at /opt/conda/conda-bld/pytorch_1579022027550/work/aten/src/TH/generic/THTensorEvenMoreMath.cpp:418

@okhonko, can you take a look?

While debugging i notice that the dimension of active_bbsz_idx suddenly changes from 320 (1620) to 300 (1520) and this is when the error happens.
In: fairseq/fairseq/sequence_generator.py

Output:
torch.Size([16, 20])
torch.Size([16, 40])
torch.Size([16, 20])
torch.Size([320])
torch.Size([16, 20])
torch.Size([15, 40])
torch.Size([15, 20])
torch.Size([300])
I am still not sure why this happens!

@AhmedTElthakeb @myleott Hi, do u find a solution for this issue? When inference (use the infer.py in examples/speech_recognition), I got a similar exception when the bzs greater than 1. It seems good when bsz fixed with 1, while it is really slow.... almost cost half an hour for my computer to finished all Aishell's test corpus.

@AhmedTElthakeb I apologize for the delayed reply.
I was able to obtain the model which replicate the bug you mentioned. I am working on the fix now.


Added only_catch_better_iv and only_catch_better_cp

Added ability for the bot to do a basic check if a Pokemon to be caught is a higher IV or CP than the one we have (highest one we have) and see if we need to try and catch it.

I don't understand the need of this... Just catch... Can earn some stardust... Later use optimizer to release... This sound redundant....

Agreed, but look at #6154 why someone wants to change catch parameters all the time

this is better than #6154

works only around IV and CP, but only see pokemon_id? or also family_id?

I am not liking the way this is coded:
if pokemon_config.get('only_catch_better_cp', False):
This will fail if the key is missing in json. So we need to avoid config fails whenever something new is added. Please default the values and allow the key not to be in config.

This will fail if the key is missing in json.
It doesn't with me? I believe I provide a default value with the False here, so if no value found it will default to False?

@Jcolomar it works based on a Pokemon, not on a family of Pokemon, so no family_id

@davidakachaos sorry missed the false there :).

@davidakachaos so u need a rule for each pokemon...

Yes, I mean this rule isn't for every Pokémon? Like the candy rule, it
works like that, so it checks the Pokémon about to be caught versus the
rule(s) matched, so it would be able to match multiple Pokémon?
Op 1 aug. 2017 14:38 schreef "Jhero"<EMAIL_ADDRESS>
@davidakachaos https://github.com/davidakachaos so u need a rule for
each pokemon...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PokemonGoF/PokemonGo-Bot/pull/6162#issuecomment-319357988,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAD5Y3xxA3__ruEOtTVV-VwAsN3blob5ks5sTxxIgaJpZM4OnZPX
.

@davidakachaos @Jcolomar You want to merge this pr?

Ya np. We kept opened but @davidakachaos is in vacation.


Allow permanent USB pass-through configuration

This is a follow-up of this discord discussion.
USB-device pass-through can be configured manually when a VM has stated by clicking on the USB icon at the top right of a running VM window. This has to be done each time.
Note that  I added the following to the VM config in the QEMU section: -device usb-host,vendorid=0x067b,productid=0x2303 (I got the vendor/product IDs from macOS "System Information". However this doesn't work and manually attaching the USB device is still required each time the VM is started.
The idea would be to configure this "permanently", i.e. the configured USB device (based on vendorid/productid or hostbus/hostaddr) would be attached as pass-through to the VM when it is started. This is similar to what VirtualBox does with filters.
According the reply I got from @conath on Discord, one implementation idea could be to add “bookmarked” USB device passthrough so the device is automatically passed when the VM starts.
Thanks in advance for considering this feature request.

Anyone found a workaround for this? I badly need it as well.

Workaround: https://gist.github.com/JurajNyiri/df4bacd1ccfc00105843e67f09b2fc80
How to run:

Open Automator
Create new application
Choose "Run AppleScript"
Copy & paste the code above
Change application_name to your VM name
Change usb_select to the prefix (or exact match) of your USB device, for example if you are using ConBee II, you can put in conbee or con as long as you do not have other device like connected usb device, it basically just writes the text and hits enter after context menu is shown.

I will try @JurajNyiri' solution but it would be great to have this out of the box - why isn't that the case?!

Does the workaround work for multiple devices without having to duplicate the script?

@protoEvangelion it controls your mouse and keyboard so as long as you do not run it at the same time it will work

This also worked for me (thank you @JurajNyiri  for the idea):
tell application "System Events"
	repeat with theButton in button of toolbar 1 of window "ha" of application process "UTM"
		set theButtonDescription to description of theButton
		if theButtonDescription is "USB" then
			click theButton
			keystroke "Sonoff"
			key code 36
		end if
	end repeat
end tell

(replace "ha" with the vm name and "Sonoff" with the start of the menu item text)

can u do this in headless mode .. i am thinking not because the usb+ button wont there?

can u do this in headless mode .. i am thinking not because the usb+ button wont there?

@krycor This has also bothered me, but I figured out you can now also connect USB devices to headless VMs via the utmctl command line tool.
Make sure USB Support is on in the VM Settings (Input section).
In a Terminal:
# List VMs - you can use either the ID or the name
utmctl list

# List USB Devices - Copy the ID you want to connect
# The ID is the vendor ID and product ID of the USB device, this won't change
utmctl usb list

# Connect USB device (when VM is running)
# This connects USB device with ID AAAA:BBBB to VM named "Home Assistant"
utmctl usb connect "Home Assistant" "AAAA:BBBB"

So this does not solve the need for a permanent configuration since you still have to run the connect command whenever you restart the VM or after unplugging the device.
But it is much nicer than being forced to have a virtual display and running Automator scripts to connect a device via the UI.

Thanks.. i was actually looking at the AppleScript option but my lack of experience with it meant its a bit awkward for me to use. Found this again via google which helped as i can use the utmctl to quickly get a list of devices and so i script a connect to vm.
one thing i see is you can only really do this when the vm is running which means i'd have make the vm bootable excluding usb device / disk,  run a exec script  kicked off on guest vm via utmctl to do the mappings etc. not too bad as i can package it in a script on eitherside.

Any hope of getting this feature?

Should help others in this thread but also perhaps the feature in UTM: this can be done with a LaunchAgent.

Thanks, that's helpful for connecting a device to an already running VM!
I think one thing the issue is asking for is better UTM support for the opposite order of events, or ideally both. When a USB device is already connected to the host and I start a VM, I want the device to be connected to the VM immediately without running a script.
I believe the automation currently possible is to start the VM and trigger the USB connection both from a script (not sure about the necessary timing here), or to trigger the connection from the guest system on boot in some way.

can u do this in headless mode .. i am thinking not because the usb+ button wont there?

@krycor This has also bothered me, but I figured out you can now also connect USB devices to headless VMs via the utmctl command line tool.
Make sure USB Support is on in the VM Settings (Input section).
In a Terminal:
# List VMs - you can use either the ID or the name
utmctl list

# List USB Devices - Copy the ID you want to connect
# The ID is the vendor ID and product ID of the USB device, this won't change
utmctl usb list

# Connect USB device (when VM is running)
# This connects USB device with ID AAAA:BBBB to VM named "Home Assistant"
utmctl usb connect "Home Assistant" "AAAA:BBBB"

If utmctl is not found, make sure to install it: https://docs.getutm.app/scripting/scripting/#command-line-interface If UTM was installed via homebrew, this shouldn't be necessary, since homebrew already adds the binary to its standard bin folder (i.e. /opt/homebrew/bin on Apple Silicon)
So this does not solve the need for a permanent configuration since you still have to run the connect command whenever you restart the VM or after unplugging the device. But it is much nicer than being forced to have a virtual display and running Automator scripts to connect a device via the UI.

Thank you for this information!
I was able to use automator to make an "app" to start up at login to start my VM's (including HA) and connect the USB devices to HA once I knew their ID's
Here's a resource on how to create an "automator app": https://truptivbhatt.medium.com/how-to-run-your-shell-script-automatically-after-logging-to-mac-da789bb78d21
This is the shell script that I used and is working great for me:
# Set current working directory
cwd=$(pwd)
# Change directory to UTM app
cd /Applications/UTM.app/Contents/MacOS/
# Start VM machine/s
./utmctl start VM1
./utmctl start VM2
./utmctl start VM3...
# Add a 45 sec wait timer (not sure if needed but added just in case)
sleep 45
# Mount Desired USB's
./utmctl usb connect "HomeAssistant (or Name of VM)" "AAAA:BBBB"
./utmctl usb connect "HomeAssistant (or Name of VM)" "AAAA:BBBB"
# Change directory to original location
cd $cwd

I hope this helps others as well!

Yup doing exactly this.. one weird thing i noticed is that to "list" usb devices a VM has to be active albeit i just wanted to list them so i know what ids to use.
Currently I use this to hook-in hdds into a nas vm with fstab set to non-auto mount.. but this does mean i have to kick off crypttab/mount services from the cmd of the host which worries me as it does mean i can execute whatever i want from the host.
Is there a way to lock this down somehow.. so the host can't inject a commands.. i'd prefer it executing a known & allowed script only.. instead of the current allow anything.


Logging for test failures

Logging related to #6175

Codecov Report

Merging #6177 (ea864f3) into main (9ae7d16) will increase coverage by 0%.
The diff coverage is 84%.

@@          Coverage Diff          @@
##            main   #6177   +/-   ##
=====================================
  Coverage     72%     72%
=====================================
  Files        399     399
  Lines      26869   26885   +16
  Branches    3954    3958    +4
=====================================
+ Hits       19424   19440   +16
+ Misses      5800    5795    -5
- Partials    1645    1650    +5

Impacted Files
Coverage Δ

...lient/datascience/jupyter/kernels/cellExecution.ts
70% <ø> (ø)

...ient/datascience/jupyter/kernels/kernelSelector.ts
87% <76%> (-4%)
:arrow_down:

...rc/client/datascience/commands/notebookCommands.ts
69% <100%> (+1%)
:arrow_up:

...science/jupyter/kernels/kernelDependencyService.ts
89% <100%> (+<1%)
:arrow_up:

...ience/notebook/emptyNotebookCellLanguageService.ts
72% <0%> (-4%)
:arrow_down:

...ent/datascience/jupyter/kernels/kernelExecution.ts
67% <0%> (-2%)
:arrow_down:

...t/datascience/kernel-launcher/localKernelFinder.ts
80% <0%> (-2%)
:arrow_down:

...tascience/jupyter/liveshare/hostJupyterNotebook.ts
44% <0%> (-2%)
:arrow_down:

src/client/datascience/jupyter/kernels/kernel.ts
75% <0%> (-1%)
:arrow_down:

.../datascience/notebook/notebookControllerManager.ts
86% <0%> (-1%)
:arrow_down:

... and 9 more

Sorry couldn't quite tell from the title. Is this just something you were running to get logging for debugging temporarily? Or did you want to submit this permanently?

Not sure if this was just for temp logging, but fine to submit either way :)

@IanMatthewHuff in #6175, a test seems to fail (this was always passing, but noticed this being flaky after the VS Code changes to outputs).
Hnece adding logging for tests.


[Event leak] UpdateProfilesList  in particleTextureBlock

If disposed before the request has ended, the loadend event is not being removed when using this function.
See:
https://dev.azure.com/babylonjs/ContinousIntegration/_build/results?buildId=18896&view=logs&j=cfa20e98-6997-523c-4233-f0a7302c929f&t=e990613a-5b65-57a5-9f28-c08f539cef73&l=23

This one is not a leak (per se). The WebXRMotionControllerManager has a static function (UpdateProfilesList) which is not attached to any scene or engine. The request will clean itself upon resolve (or failure)
I think we should update that test to ignore this occurence.

Test updated.


feat: add mongodb 1.16.0 release

update mongodb with release 1.16.0

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


SLO-106 Added index on status field for posts table

There are slow queries on the posts table that take up to 5 minutes to browse posts for customers with a large number of posts.
Sentry link: https://ghost-foundation.sentry.io/performance/?isDefaultQuery=false&metricSearchSetting=metricsOnly&project=1879038&query=transaction%3A"GET+%2Fghost%2Fapi%2Fadmin%2Fposts%2F"&statsPeriod=7d&utc=true
There are two slow queries and an index on [status, type] will improve the performance of both of these queries.

@vershwal Thanks for creating this!  Can you attach EXPLAIN ANALYZE output of this query before and after the change so we can understand the impact it's having on the query analyzer?  Can you also test this before/after on a database in staging that's having this problem and share what improvement this is making for the customer as experienced from the browser?

Current behaviour:-
-> Limit: 20 row(s)  (cost=6015 rows=20) (actual time=876..876 rows=20 loops=1)
    -> Sort row IDs: (case when (posts.`status` = 'scheduled') then 1 when (posts.`status` = 'draft') then 2 else 3 end), (case when (posts.`status` <> 'draft') then posts.published_at end) DESC, posts.updated_at DESC, posts.id DESC, limit input to 20 row(s) per chunk  (cost=6015 rows=7991) (actual time=876..876 rows=20 loops=1)
        -> Filter: ((posts.`type` = 'post') and (posts.`status` = 'draft'))  (cost=6015 rows=7991) (actual time=0.615..875 rows=800 loops=1)
            -> Table scan on posts  (cost=6015 rows=7991) (actual time=0.604..872 rows=15000 loops=1)
-> Select #2 (subquery in projection; dependent)
    -> Aggregate: count(distinct members_click_events.member_id)  (cost=1091 rows=1) (actual time=0.0633..0.0633 rows=1 loops=20)
        -> Nested loop inner join  (cost=1000 rows=908) (actual time=0.063..0.063 rows=0 loops=20)
            -> Covering index lookup on redirects using redirects_post_id_foreign (post_id=posts.id)  (cost=1.6 rows=5.29) (actual time=0.0629..0.0629 rows=0 loops=20)
            -> Index lookup on members_click_events using members_click_events_redirect_id_foreign (redirect_id=redirects.id)  (cost=175 rows=171) (never executed)

After adding index:-
-> Limit: 20 row(s)  (cost=280 rows=20) (actual time=120..121 rows=20 loops=1)
    -> Sort row IDs: (case when (posts.`status` = 'scheduled') then 1 when (posts.`status` = 'draft') then 2 else 3 end), (case when (posts.`status` <> 'draft') then posts.published_at end) DESC, posts.updated_at DESC, posts.id DESC, limit input to 20 row(s) per chunk  (cost=280 rows=800) (actual time=120..121 rows=20 loops=1)
        -> Index lookup on posts using status_type (status='draft', type='post')  (cost=280 rows=800) (actual time=0.662..119 rows=800 loops=1)
-> Select #2 (subquery in projection; dependent)
    -> Aggregate: count(distinct members_click_events.member_id)  (cost=1091 rows=1) (actual time=0.0395..0.0395 rows=1 loops=20)
        -> Nested loop inner join  (cost=1000 rows=908) (actual time=0.0391..0.0391 rows=0 loops=20)
            -> Covering index lookup on redirects using redirects_post_id_foreign (post_id=posts.id)  (cost=1.59 rows=5.29) (actual time=0.0389..0.0389 rows=0 loops=20)
            -> Index lookup on members_click_events using members_click_events_redirect_id_foreign (redirect_id=redirects.id)  (cost=175 rows=171) (never executed)```

Query 2
SELECT COUNT(DISTINCT posts.id) AS aggregate
FROM `posts`
WHERE (
  `posts`.`status` in ('draft') AND `posts`.`type` = 'post'
)

Current behavior:-
-> Limit: 200 row(s)  (cost=6744 rows=1) (actual time=882..882 rows=1 loops=1)
    -> Aggregate: count(distinct posts.id)  (cost=6744 rows=1) (actual time=882..882 rows=1 loops=1)
        -> Filter: ((posts.`type` = 'post') and (posts.`status` = 'draft'))  (cost=6736 rows=79.9) (actual time=1.53..880 rows=800 loops=1)
            -> Table scan on posts  (cost=6736 rows=7991) (actual time=1.51..878 rows=15000 loops=1)

After adding index
-> Limit: 200 row(s)  (cost=207 rows=1) (actual time=20.5..20.5 rows=1 loops=1)
    -> Aggregate: count(distinct posts.id)  (cost=207 rows=1) (actual time=20.5..20.5 rows=1 loops=1)
        -> Covering index lookup on posts using status_type (status='draft', type='post')  (cost=127 rows=800) (actual time=0.126..0.938 rows=800 loops=1)

The estimated cost has decreased from 6744 to 207
The actual time taken has decreased from 882 ms to 20.5 ms


Fix reference marking of values merged with unresolvable type-only imports

In #50853, I identified several cases where a symbol was incorrectly being reported as unusable in emit locations due to being a type-only import, when it was actually a local value merged with a type-only import:
// @Filename: a.ts
export interface A {}

// @Filename: b.ts
import type { A } from "./a";
const A = {};
A; // Error: A cannot be used as a value because it was imported with 'import type' 👎

The original code simply looked to see if the symbol resolved to any type-only import or export, which was obviously insufficient for merged symbols. Ideally, we wanted to know which declarations contributed which meanings to the merged symbol’s flags. That information isn’t tracked, though, so in #50853 I implemented a bit of proxy logic to get close: if the yet-unresolved alias symbol has a value meaning, and the type-only import symbol doesn’t, then we know the value meaning could not have come from the type-only import, so it’s safe to use. In code, that was something like
symbol.flags & SymbolFlags.Value &&
!(resolveAlias(getTypeOnlyAliasDeclaration(symbol).symbol).flags & SymbolFlags.Value)

So in the example above, symbol.flags would have Alias|Variable (and Variable counts as a Value), and the resolution of A starting at the type-only import would have Interface. The new logic correctly allowed A to be referenced as a value.
So we were kind of picking apart the pieces of the merged symbol, performing some alias resolution independently on the parts, and using the process of elimination to determine where the SymbolFlags.Value meaning must have come from. (The key thing that makes this work is that separate values are not allowed to merge across modules through an import—e.g., even though a function and a namespace can merge in the same file, you can’t import a function f and then declare an instantiated namespace f. That’s always been an error.)
However, this logic breaks down if the type-only import cannot be resolved, as in ts.transpileModule. The symbol returned by resolveAlias for an unresolvable alias is unknownSymbol, which has all symbol flags, to denote that it could be anything. So looking at the example through the old code again:
symbol.flags & SymbolFlags.Value &&
!(resolveAlias(getTypeOnlyAliasDeclaration(symbol).symbol).flags & SymbolFlags.Value)

symbol.flags & SymbolFlags.Value is still true, but this time, resolveAlias(getTypeOnlyAliasDeclaration(symbol).symbol) is unknownSymbol, which also has SymbolFlags.Value, so the incorrect assumption is that the value meaning for A must have come from the type-only import.
This PR replaces that strategy with one that collects flags starting at the merged symbol, and stops just before it gets to the resolution of the type-only import.
Fixes #54526

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this


0.9.4: DeprecationWarning warnings on use pytest-sugar

Looks like latest version emits DeprecationWarning warnings
: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
  xdist_version = LooseVersion(xdist.__version__)
/usr/lib/python3.8/site-packages/pytest_sugar.py:170: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.

It looks like this was fixed by ee02ada.

Just tested whole https://github.com/Teemu/pytest-sugar//pull/230
Here is result:
+ PYTHONPATH=/home/tkloczko/rpmbuild/BUILDROOT/python-pytest-sugar-0.9.4-3.fc35.x86_64/usr/lib64/python3.8/site-packages:/home/tkloczko/rpmbuild/BUILDROOT/python-pytest-sugar-0.9.4-3.fc35.x86_64/usr/lib/python3.8/site-packages
+ /usr/bin/pytest -ra -p no:twisted -p no:xkill
Test session starts (platform: linux, Python 3.8.13, pytest 7.1.2, pytest-sugar 0.9.4)
rootdir: /home/tkloczko/rpmbuild/BUILD/pytest-sugar-0.9.4
plugins: sugar-0.9.4, forked-1.4.0, xdist-2.5.0, rerunfailures-10.2
collecting ...
 test_sugar.py ✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓✓                                                                                100% ██████████
============================================================================= warnings summary =============================================================================
test_sugar.py: 34 warnings
  /home/tkloczko/rpmbuild/BUILDROOT/python-pytest-sugar-0.9.4-3.fc35.x86_64/usr/lib/python3.8/site-packages/pytest_sugar.py:250: PytestRemovedIn8Warning: The (startdir: py.path.local) argument is deprecated, please use (start_path: pathlib.Path)
  see https://docs.pytest.org/en/latest/deprecations.html#py-path-local-arguments-for-hooks-replaced-with-pathlib-path
    lines = self.config.hook.pytest_report_header(

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html

Results (6.38s):
      28 passed

And if you are around 😋
Id it possible to sort out https://github.com/Teemu/pytest-sugar/issues/226 ?

I think this is fixed afaik


Add optional setting to disable commit and push after adding link

What:
Add ability to include commitAndPush in your package.json which determines if a commit and push happens after adding a link. Should default to true if not specified.
Why:
You can't stop the commit and push if you don't want it to happen.
How:
Set doCommitAndPush to equal packageJson.commitAndPush !== false and check it before doing commitAndPush()
Checklist:

[x] Documentation
[x] Tests (edited the equivalent lines in my node_modules and it appeared to work
[x] Ready to be merged

A few extra lines appear to have changed as a result of automatic pre-commit formatting.

Being able to check for my own errors one last time before pushing is my reason for wanting it.

Being able to check for my own errors one last time before pushing is my reason for wanting it.

Hmmmm... If there's an error (which is rare for me, but it does happen) them I just fix it manually and push. 🤷♂

Hmmmm... If there's an error (which is rare for me, but it does happen) them I just fix it manually and push. 🤷♂

🤷♂ I can also edit the node_modules install to not have commitAndPush if it's too niche of a request for the library itself. No big deal on my end.

🤷♂ I can also edit the node_modules install to not have commitAndPush if it's too niche of a request for the library itself. No big deal on my end.

Yeah, this is a simple package and I want to keep it that way. Thanks though!

Yeah, this is a simple package and I want to keep it that way. Thanks though!


e2e test for github source

Expected Behavior
Have e2e test coverage for github source.
Actual Behavior
There is no e2e test for github source.
/kind bug

/remove-lifecycle stale

moved to https://github.com/knative-sandbox/eventing-github/issues/83


PublishTestResults@2 version 2.170.0 fails to upload tasks due to missing PTR_TEST_RUNSUMMARY.json file

Question, Bug, or Feature?
Bug
Enter Task Name: PublishTestResults@2
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:
Account: MQAutomate
Team Project Name: MOSAIQLab
Build Definition Name: Prereq Test
Build number - all

Agent - Private (VM Environment) on Windows Servers:

Issue Description
Task version 2.170.0 fails to upload tasks, even though it completes successfully.
"Normal" log:
C:\azagent\A1\_work\_tasks\PublishTestResults_0b0f01ed-7dde-43ff-9cbb-e48954daf9b1\2.170.0\modules\TestResultsPublisher.exe @C:\azagent\A1\_work\_temp\40166ab0-a773-11ea-b9d3-91ff84b586bc.txt
Finishing: Upload test reports

With diagnostic logs on, we see the following:
##[debug]Exit code<PHONE_NUMBER> received from tool 'C:\azagent\A1\_work\_tasks\PublishTestResults_0b0f01ed-7dde-43ff-9cbb-e48954daf9b1\2.170.0\modules\TestResultsPublisher.exe'
##[debug]STDIO streams have closed for tool 'C:\azagent\A1\_work\_tasks\PublishTestResults_0b0f01ed-7dde-43ff-9cbb-e48954daf9b1\2.170.0\modules\TestResultsPublisher.exe'
##[debug]Exit code of TestResultsPublisher:<PHONE_NUMBER>
##[debug]Agent.Version=2.169.1
##[debug]Agent.TempDirectory=C:\azagent\A1\_work\_temp
##[debug]Unable to publish the test run summary to evidencestore, error details:Error: ENOENT: no such file or directory, open 'C:\azagent\A1\_work\_temp\PTR_TEST_RUNSUMMARY.json'
##[debug]task result: Succeeded

It appears that the code is looking for the PTR_TEST_RUNSUMMARY.json but this file does not exists. I took a quick scan through the repo and can only find this instance of reading the file - but not where the file is produced.
Task logs
Inline above.
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
No error logs.

@colindembovsky The debug statement is actually misleading. Since there are no tests found for the task, so there's no summary may be we should change those lines to avoid confusion.
To root cause the original issue.
What are you trying to publish .trx, .xml (vstest, nunit, xunit or junit) tests ? If possible can you share the pipeline details or the test reports to help you further ?

The pipeline is publishing junit test result files that are generated by Pester. We have this pipeline in 2 orgs: one is a "master" org and the other is for testing.
When we run the pipeline in the master org, the 2.169.0 version of PublishTestResults@2 runs and we get tests in the summary as expected. When we run the same pipeline in the test org, version 2.170.0 of the PublishTestResults@2 runs and we get the error above in DEBUG and there are not test results.
When there have been problems with the test results file (or this file is not found) there is usually a warning saying that no test results were found. The 2.170.0 task is finding test results (I can see this in the log) but it is erroring as above.
A run of the pipeline that works (in master org with test results as expected, using 2.169.0) is here.
Unfortunately, we've deleted the team project with an example of the pipeline that does not work but I will add a link to that when we have one. By the way, we've deleted and created the Team Project and pipeline multiple times and every time PublishTestResults@2 runs on version 2.170.0 we do not get any test results.

@colindembovsky can we get to see the entire debug logs ?
and are you using the self hosted docker container agent ?

@navin22 I don't have a log for a non-working pipeline. In our original org the version that runs is 2.169.0 which works. New orgs seem to get 2.170.0. As soon as I have a log for a failure again I will scrub and post. However. I'm sure just uploading a junit XML results file will allow you to repro?
This job is running in a VM environment (yml deployment job) so it's a private agent.

@colindembovsky Okay.
I see the TestResultsPublisher.exe tool is exiting with different exit code than usual, that we saw only docker hosted agent. Can you try passing PublishTestResults.OverrideExeFlow  = true to your pipeline and run to see if exe is causing the issue.?

Hi @navin22 - looks like that switch helped!
Here's a run without the switch: link
Here's a run with diagnostics enabled and the switch: link
Here's a snippet of the log from the second pipeline (that works as expected):
##[debug]PublishTestResults.OverrideExeFlow=true
##[debug]OS type: Windows_NT
##[debug]Reading test results from file 'C:\azagent\A4\_work\1\s\tests\SQL-Prereqs.results.xml'
Timestamp is not available for one or more testsuites. Total run duration is being calculated as the sum of time durations of detected testsuites
##[debug]Processed: ##vso[results.publish type=JUnit;mergeResults=false;runTitle=SQL Prereqs;publishRunAttachments=true;resultFiles=C:\azagent\A4\_work\1\s\tests\SQL-Prereqs.results.xml;failTaskOnFailedTests=false;testRunSystem=VSTS - PTR;]
##[debug]task result: Succeeded
##[debug]Processed: ##vso[task.complete result=Succeeded;]
##[debug]Release.ReleaseUri=undefined
##[debug]Release.ReleaseId=undefined
##[debug]Build.BuildUri=vstfs:///Build/Build/13
##[debug]Build.Buildid=13
##[debug]Processed: ##vso[telemetry.publish area=TestExecution;feature=PublishTestResultsTask]{"builduri":"vstfs:///Build/Build/13","buildid":"13","osType":"Windows_NT","testRunner":"JUnit","failTaskOnFailedTests":"false","mergeResultsUserPreference":"false","config":null,"platform":null,"testResultsFilesCount":1,"dotnetVersion":"","subFeature":"publishTestResultsTaskConsolidatedCiEvent"}
Async Command Start: Publish test results
Publishing test results to test run '2'
Test results remaining: 35. Test run id: 2
##[debug]Uploading test run attachements individually
Published Test Run : https://tfsprodcus5.visualstudio.com/Aab78a5bf-b4fe-4d61-94c4-3a08dc88f8f2/MOSAIQ%20Training%20Lab/_TestManagement/Runs?runId=2&_a=runCharts
TestRunPublisher.IsTestRunFailed: Getting test run summary with run id: 2 using stats api
TestRunPublisher.IsTestRunFailed: checking if test run is failed using run summary input
Number of failed tests which are non-flaky is: 12
Failed Results are published
##[debug]RESOURCE_URIS:
##[debug]Setting task variable METADATA_8e005faf-00a8-404a-8d4c-ab0cbb50243e: {"name":"4a70e4e3-6377-4168-8999-a34982821314","resourceUris":[],"metadata":{"description":"","relatedUrls":[{"url":"https://dev.azure.com/MQAutomate/MOSAIQ Training Lab/_build/results?buildId=13","label":"pipeline-url"}],"humanReadableName":"Test Results from Publish Test Results utility","serializedPayload":"{\"testId\":\"PublishTestResults\",\"testTool\":\"JUnit\",\"testResultAttestation\":{\"total\":35,\"failed\":12,\"passed\":20,\"skipped\":0},\"testDurationSeconds\":0.0,\"testPassPercentage\":\"57.14285714285714\",\"relatedUrls\":[{\"url\":\"

@colindembovsky good that you have workaround now to unblock yourself.
Meanwhile I will work on the root cause. I don't have acess to the link you have provided. Can you please download the entire logs and share here for the one without switch which is causing the problem.

@navin22 Can you give me an email address to share the logs?

@colindembovsky it's<EMAIL_ADDRESS>

@navin22 just emailed logs to you

@colindembovsky seeing a similar issue on 2.170.0
After running the tests I can see the results
pwd
Directory: D:\a\1\s\WebApp\ClientApp\test-results
ls
Contents of directory:
Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        6/17/2020   3:23 AM            120 junit.xml

When attempting to publish the results:
No Result Found to Publish 'D:\a\1\s\WebApp\ClientApp\test-results\junit.xml'.

@MuckT I think that's a different issue. If it cannot fine the XML file, you'll get that error message. My issue was that the file was found but there was some sort of error when uploading it.
You should check the path that you're publishing the test results to or that your searching to upload.

@colindembovsky This exit code(2148734720 ) is related to .net Framework. (I feel it's related to https://github.com/microsoft/azure-pipelines-tasks/issues/12224)
What's the .net framework on VM? Can you try upgrading to 4.7?

@colindembovsky can you please confirm if you have tried upgrading the .net framework as mentioned above?

I have the same issue with Publish Test Results version 2.171.0 and .Net 4.0. For me this issue looks like this one : #12300.

@colindembovsky upgrading to .net framework 4.7.2 should fix the problem. We will update the docs and now closing this issue.


docs: add security policy

Add Security Policy
Pull request template

Add security.md
Motivation and context
Complete the community standards and provide a way to communicate security issues
https://github.com/empathyco/x/community

[ ] Dependencies. If any, specify:
[ ] Open issue. If applicable, link:

Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that causes existing functionality to not work as expected)
[x] Change requires a documentation update

What is the destination branch of this PR?

[x] Main
[ ] Other. Specify:

How has this been tested?

Tests performed according to testing guidelines:
Checklist:

[ ] My code follows the style guidelines of this project.
[ ] I have performed a self-review on my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] New and existing unit tests pass locally with my changes.

Looks good to me

Best avatar ever, I can't get tired of it.
Thanks for the approval!


Content length of inputstream is 0 at CommitStage

Config
 - PhantomJSFetcher : Only HTML(Default)
 - keepDownload : false

I can't handle contents of url(ex. images) in queueAddition() method.
I tried to read it but Its length is 0 so returns nothing.
I confirmed PhantomJSFetcher pass the url(mime type : image) to GenericDocumentFetcher.
Is there any option to pass content inputstream to queueAddtion method?
Thank you.

The Importer module parses every file and extracts any text it can.  By default, the Committer should not receive binary streams (e.g., images).  So it may just be that no text content could be extracted from the image (just metadata) and this is why you have a zero-length stream.  You may want to investigate enabling OCR.  Have a look at GenericDocumentParserFactory.
If you want to get the raw image stream, you have to make sure images are not parsed.  You can try the following:
  <documentParserFactory>
      <!-- if you want OCR -->
      <ocr path="/app/tesseract/">
          <languages>en, fr</languages>
          <contentTypes>application/pdf</contentTypes>
      </ocr>

      <!-- Do not parse images: -->
      <ignoredContentTypes>
          image/.*
      </ignoredContentTypes>
  </documentParserFactory>

If you do so, make sure you have no import handlers that expect text.  Use the "restrictTo" on your handlers.

Thank you. It is helpful.
I'll close this.
Thank you.


Filter applied in get() returns 400065: Bad Request

I'm trying to filter by name a user, group or project, I'm getting following error message.
400065: Bad Request
	The filter query contains a key which is not recognized for this api version.

Environment:
server-client-python  version: 0.14.0
server version: 3.2 (Not working in any version from 2.0 to 3.2)
Code used:
tableau_auth = TSC.TableauAuth(user, password, site_name)
server = TSC.Server('https://mysite.com')
server.use_server_version() #3.2

with server.auth.sign_in(tableau_auth):
        req_option = TSC.RequestOptions()
        req_option.filter.add(TSC.Filter(TSC.RequestOptions.Field.Name,
                                     TSC.RequestOptions.Operator.Equals,
                                     'THE_USER_NAME'))
       all_entities, pagination_item = server.users.get(req_option)

@ludwigrubio what's the version of Tableau Server that you are connecting to? I've done a few tests on 2020.4 and getting users with the name filter seems to work fine, but it might be version-specific.

@bcantoni Tableau Server Version: 2018.3.4 64-bit Windows

Looks like the same issue as #732

This should be fixed in v0.14.1.


Another whitespace fix

What type of PR is this?
Add one of the following kinds:

bug

What this PR does / why we need it:
PR to fix trailing whitespace issue in #23 commit
Which issue(s) this PR fixes:
trailing whitespace issue in #23 commit
Special notes for reviewers:
Changelog input
 release-note

Additional documentation
This section can be blank.
docs

@javierlozallu this time checked on a forked repository with the same linter workflow - third time lucky!


main property in bower manifest should point to runnable code

We use main-bower-files with gulp to inject our dependencies into our application and just suffered an outage because the main property in angular-img-fallback's bower points to ES6 code. I'd suggest pointing it towards the transpiled minified version instead.
Until this fix is deployed, the workaround is adding the following to your applications bower file:
"overrides": {
    "angular-img-fallback": {
      "main": "angular.dcb-img-fallback.min.js"
    }
}

This need to be fixed, it breaks the build process when you use angular-fullstack-generator and took me a lot of time to debug. Just because there are some ES6 fat arrows in the main file.

+1 - this broke the gulp-uglify part of my build

@nhim175 @jordajm can you please share the errors you are getting?

@dcohenb - the error was something like "unrecognized character '>'" - I assume from the ES6 fat arrows as mentioned above. @thethomaseffect's workaround fixed the issue.

This is a collision with this issue:
https://github.com/dcohenb/angular-img-fallback/issues/12
In short, the other one states that the main file should be the unminified version...


VersionBot: Tag only bumping

We're looking at adding VersionBot to the product keyframes repo. However, we don't have a changelog file (indeed, it's not something required as it's a machine repo), or any other file to update. VersionBot will currently refuse to bump versions if versionist comes back without any files (or no changelog exists).
What we actually could do is just skip the update if no files change, and still merge then tag. This would solve this issue.

CC @brownjohnf so he knows the situation.

Wouldn't it be better to just force an empty commit with the version bump?

Sorry, I don't think I was clear. By skip the update, I mean the check for files. We'd still have to do an empty commit to ensure that an entry occurred in the git log, yep.


Project is missing an OSS license

Please add an open source license for this project

@patrickhernon thank you for raising this issue.  This was an oversight, and one that is being corrected in the next release.


New package: Gtk4 v0.1.0

Registering package: Gtk4
Repository: https://github.com/JuliaGtk/Gtk4.jl
Created by: @jwahlstrand
Version: v0.1.0
Commit: dc77e63a882a1aca5a5089068cf046938d8493f0
Git reference: HEAD
Description: Experimental Julia package for Gtk4, using GObject introspection

@jwahlstrand
please see the message above.  The name is fine, but you need to address the compat issue. Once you fix that you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]