Sprint May 2017 - Scrum update

SCRUM for Week  01-05-2017 to 15-05-2017
Availability- Global Test Team members

George (100%)
Sivas (75% / 1 week vacation)    @sivassrr
Rahul (100%)  @rahulmah
Prashant(100%)  @prkatti1
Sridevi (50% Automation)   @SrideviRamesh
Sweta (50% Test + automation / 50% ramping up OpenBMC training, education )  @swe12345
Leo (80% Automation + 20 % support Dev fixes on demand)  @lgon
Saqib (???)  @saqibkh
Triz (30 % / Since he is winding up on development sprint work)   @mtritz
Gunnar (0% since still doing development sprint work)  @gtmills
Sathyajit (???)

GEORGE
Completed:

System Test Jenkins job : https://github.com/openbmc/openbmc-test-automation/issues/582
Power Restore test : https://github.com/openbmc/openbmc-test-automation/issues/576 /  https://github.com/openbmc/openbmc-test-automation/issues/575
Skip list : https://github.com/openbmc/openbmc-test-automation/issues/571
New REST test cases added: https://github.com/openbmc/openbmc-test-automation/issues/567
HTX hardbootme : https://github.com/openbmc/openbmc-test-automation/issues/553
Sub Task:
#542 [ Merged ]
#558 [ Merged ]
#561 [ Merged ]
#565 [ Merged ]

In Progress / TBD:

Network stress test : https://github.com/openbmc/openbmc-test-automation/issues/588
Base Fan test code :https://github.com/openbmc/openbmc-test-automation/issues/587  ( In review )
Validate params and HTX log file https://github.com/openbmc/openbmc-test-automation/issues/580 ( In review )
HTX Installation base code https://github.com/openbmc/openbmc-test-automation/issues/573 ( In review )
Driver stability test https://github.com/openbmc/openbmc-test-automation/issues/583 (In Progress)
Halt HTX test ( Keep HTX running on error )  https://github.com/openbmc/openbmc-test-automation/issues/579   ( In review )

Blockers If any / Comments:

MFG tool test ( Need to discuss with MFG folks )
Tarball enablement code update / CT  https://github.com/openbmc/openbmc-test-automation/issues/461 ( Need to try out using different install binaries PATH)
CT Metrics jenkins integration https://github.com/openbmc/openbmc-test-automation/issues/592 ( TBD - Will schedule if time permits)

SIVAS
Completed:

None

In Progress / TBD:

On vacation will update latter

Blockers If any / Comments:

PRASHANT
Completed:

None

In Progress / TBD:

PGood tests for IPMI https://github.com/openbmc/openbmc-test-automation/issues/520  ( In review )
Network refactoring https://github.com/openbmc/openbmc-test-automation/issues/429
Ethtool automation https://github.com/openbmc/openbmc-test-automation/issues/522

Blockers If any / Comments:

Since the network code is yet available for testing, we could use time to develop XCAT infra structure this week, Will discuss on this.

SRIDEVI
Completed:

IPMI power off/on : #521
PDGB tool manual execution: #489

In Progress / TBD:

RAS TC with autoreboot option disabled : #449   ( In review - Rebase and comments to address)
Lab FVT systems OS IP setup (In progress)
Outband IPMI poweron/off https://github.com/openbmc/openbmc-test-automation/issues/593

Blockers If any / Comments:

PDGB utility implementation : #577

SWETA
Completed:

None

In Progress / TBD:

Call out test : https://github.com/openbmc/openbmc-test-automation/issues/586
Errorlog test case : https://github.com/openbmc/openbmc-test-automation/issues/562  (In review)
Release build test https://github.com/openbmc/openbmc-test-automation/issues/591 (In progress)

Blockers If any / Comments:

LEO
Completed:

None

In Progress / TBD:

Display software levels/inventory and verify required tools installed   https://github.com/openbmc/openbmc-test-automation/issues/549 ( In review )
Verify System state https://github.com/openbmc/openbmc-test-automation/issues/550 ( Under discussion )

Blockers If any / Comments:

https://github.com/openbmc/openbmc-test-automation/issues/550 needs inputs from system test

GUNNAR
After discussion, due to Development sprint work committed it is unlikely he'll do automation this sprint.

TRIZ
Completed:

None

In Progress / TBD:

CT metrics https://github.com/openbmc/openbmc-test-automation/issues/358  (Will pick up once Dev story is completed)

Blockers If any / Comments:

RAHUL
Completed:

None

In Progress / TBD:

Will update once list are given..

Blockers If any / Comments:

SATHYAJIT
Completed:

None

In Progress / TBD:

OpenBMC GUI manual execution

Blockers If any / Comments:

Completed:

IPMI power off/on : https://github.com/openbmc/openbmc-test-automation/issues/521
PDGB tool manual execution: https://github.com/openbmc/openbmc-test-automation/issues/489

In Progress / TBD:

RAS TC with autoreboot option disabled : https://github.com/openbmc/openbmc-test-automation/issues/449

Blocker:

PDGB tool implementation : https://github.com/openbmc/openbmc-test-automation/issues/577

Reviews in gerrit which needs attention

RAHUL
Completed:

OCC power supply redundancy and derating value TC: https://gerrit.openbmc-project.xyz/#/c/3940/
Daily CT for Witherspoon, Barreleye, and Palmetto (Monday, Tuesday)
Issue logged -
OpenBMC - 1580, 1579, 1572, 1578
OpenBMC automation - 584

In Progress / TBD:

Daily CT for Witherspoon, Barreleye, and Palmetto (for rest of the week)
Test system leds of Witherspoon system.
Method to identify a new REST interfaces
SOL enablement
SOL test cases
Turbo allowed value

Blockers If any / Comments:

SOL parameter TC - (Dev code in progress)

RAHUL
Completed:

OCC power supply redundancy and derating value TC: https://gerrit.openbmc-project.xyz/#/c/3940/
Daily CT for Witherspoon, Barreleye, and Palmetto (Monday, Tuesday)
Issue logged -
OpenBMC - 1580, 1579, 1572, 1578
OpenBMC automation - 584

In Progress / TBD:

Daily CT for Witherspoon, Barreleye, and Palmetto (for rest of the week)
Test system leds of Witherspoon system. (Code in review)
Method to identify a new REST interfaces (Code in review)
SOL test cases (Code in review)
Turbo allowed value (Code in review)
Evaluate existing issues which are getting skipped. (Need to start from  10th May)

Blockers If any / Comments:

SOL enablement -  Once we stop SOL we are unable to connect SOL next time in automation. Trying to resolve the problem.

RAHUL
Completed:
OCC power supply redundancy and derating value TC https://github.com/openbmc/openbmc-test-automation/issues/527
In Progress / TBD:

Daily CT for Witherspoon, Barreleye, and Palmetto - https://github.com/openbmc/openbmc-test-automation/issues/601
Test system leds of Witherspoon system. https://github.com/openbmc/openbmc-test-automation/issues/529
Method to identify a new REST interfaces https://github.com/openbmc/openbmc-test-automation/issues/410
SOL test cases - https://github.com/openbmc/openbmc-test-automation/issues/411
Turbo allowed value - https://github.com/openbmc/openbmc-test-automation/issues/527 (Code in review)
Fix for power restore test case not getting skipped https://github.com/openbmc/openbmc-test-automation/issues/590
Evaluate existing issues which are getting skipped. https://github.com/openbmc/openbmc-test-automation/issues/303 (Need to start from 10th May)

Blockers If any / Comments:
SOL enablement https://github.com/openbmc/openbmc-test-automation/issues/602 - Once we stop SOL we are unable to connect SOL next time in automation. Trying to resolve the problem.

Action item (31 May 2017):

[ ] @sridevi to open issue related to wsbmc006 -   Quiesce state
[ ] esel TCs to be reviewed - AI George, rahul, Prashanth
[ ] XCAT test using release driver - AI Prashanth
[ ] Security Scanning - AI Prashanth
[ ] Need to check on OPAL Dump - AI Sivas ( May not be available for 1st GA).

Closing this sprint..


Typescript enhancements

Checklist

[x] Use a separate branch in your local repo (not master).
[ ] Test coverage is 100% (or you have a story for why it's ok).

Please see issue #611
Re: Test coverage. Were you guys looking for type tests on these? Didn't see any other ones.

@nicolodavis @delucis
Updated with your feedback. Let me know if you'd like me to organize my commits differently prior to merging. I can squash the pr feedback into one of the first three commits if we want commit history to be cleaner.

All requested changes should be addressed now


Investigate how Teku peer constraints are configured

For example, here are the upper-bound / lower-bound peering options: https://docs.teku.consensys.net/en/latest/Reference/CLI/CLI-Syntax/#p2p-peer-lower-boundy version

https://github.com/ConsenSys/teku/blob/ce6e13e32f2c5029aa05895f400c00a9c72a3f38/networking/p2p/src/main/java/tech/pegasys/teku/networking/p2p/connection/TargetPeerRange.java#L23

Can we replace Besu's PeerRequirement functional interface (it's only used in one spot plus tests) with something like Teku's TargetPeerRange - this also would be easy to test

https://github.com/hyperledger/besu/pull/4200


Inconsistencies in "Citing"

Hi all,
the citing page lists the following entries (highlights from me):

Jones E, Oliphant E, Peterson P, et al. SciPy: Open Source Scientific Tools for Python, 2001-, http://www.scipy.org/ [Online; accessed 2019-08-12].

@Misc{,
author =    {Eric Jones and Travis Oliphant and Pearu Peterson and others},
title =     {{SciPy}: Open source scientific tools for {Python}},
year =      {2001--},
url = "http://www.scipy.org/",
note = {[Online; accessed ]}
}

Travis E. Oliphant. Python for Scientific Computing, Computing in Science & Engineering, 9, 10-20 (2007), DOI:10.1109/MCSE.2007.58 (publisher link)

And in the section NumPy

Travis E, Oliphant. A guide to NumPy, USA: Trelgol Publishing, (2006).

You will notice that the name first name and initials Travis E Oliphant are inconsistent. In no. 4, first and last name are switched (compared to other references).
Cheers!

Thanks, @adsche, we've improved the citing page.


Give the user hability to set ingress as needed

Removed unecessary IngressClass.
If no IngressClassName passed, use cluster's default class
Renamed ingressclass with IngressClassName. Is the standard name used for it.
Included custom annotations for Ingress. This way user can set any custom annotation for the ingress only.

Codecov Report
Patch and project coverage have no change.

Comparison is base (b1feb4e) 7.75% compared to head (1f257af) 7.75%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1417   +/-   ##
======================================
  Coverage    7.75%   7.75%
======================================
  Files          47      47
  Lines        2953    2953
======================================
  Hits          229     229
  Misses       2714    2714
  Partials       10      10

Files Changed
Coverage Δ

config/configStructs/tapConfig.go
0.00% <ø> (ø)

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


Disable (or adjust) dependabot.

The dependabot settings are inherited from upstream and this bot does not necessarily help for forked project like this.
The reason is that we get spam of updates of single package updated on EACH release branch (rate limited by dependabot). This causes issues:

It's not scalable to update single package in random time on random release branch especially manually. In fact we were not touching it so bot paused:

Every single update is essentially a "patch" that will make it much harder to sync e.g. patch version from Prometheus later on. It is also unsure if we should trigger expensive and time consuming release for each update or not?

I would suggest we disable the bot for now. If we need security updates on certain deps on certain release we might want to rather fix upstream OR manually patch version and only do that if absolutely necessary.
If we really want spam of PRs with dep updates and we want to all release branches to have all the latest deps (I don't think we want/need that), I would suggest we reconfigure dependabot to do batch upgrades, so it's managable.

Why? We've been merging Dependabot issues in the prometheus-engine repository. They're really useful, especially since at the end of this month, we'll need to start merging those to meet vulnerability requirements.

Yea, sorry for not explaining why, I should have updated the description. Edited it in (:


Add controller to generate addon label on cluster

The related issue: https://github.com/open-cluster-management/backlog/issues/15279
Signed-off-by: Yang Le<EMAIL_ADDRESS>

/assign @qiujian16
/assign @deads2k


Rename XButton1 and XButton2 into Extra1 and Extra2

This is to avoid repeating Button when using those values now that the sf::Mouse::Button enumeration is a scoped enumeration.
Follow-up of review comment https://github.com/SFML/SFML/pull/2822#discussion_r1428646949

SFMLBuildMaster: Build this please.

I'd actually be tempted to call them Back and Forward. Or does anyone actually know a mouse where this modern de-facto standard won't match? I think most if not all popular browsers map this for example.

All mice I use do not have extra buttons, but I see from wikipedia those buttons can have other meanings such as changing mouse sensitivity.

Not exactly. If they influence sensitivity, they're not mapped to those buttons (or they only do so with a modifier/switch). But yes, it's not really standardized.


Version 1.0.0

Using swagger V2.0
TAG added
Operations can be defined in MRO
Static files removed as version dependant

Wow! That's a lot of changes! Thanks for your work.
Could you please squash your commits properly? You are also trying to merge with master instead of version-1.0.0 branch.

I fact no as I'm not happy with the code...
Several reasons we can discuss :

No test... no comment
Operation can be defined only as decorator. If using an inherited
function, as all informations are set on the method (the
operation)so it doesn't work... As shared methods are often used a
model API, this is very annoying
I haven't add the model introspection (from SQLAlchemy) as I did'nt
know how you wanted to to it... (any specific design request..?)

So I need to discuss more with you about how to make it evolving
Regards
Le 03/04/2016 18:54, Sobolev Nikita a écrit :

Wow! That's a lot of changes! Thanks for your work.
Could you please squash your commits properly? You are also trying to
merge with master instead of |version-1.0.0| branch.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/flask-restful-swagger/flask-restful-swagger/pull/8#issuecomment-205009195

--
Guillaume DOUMENC
FPR / CoWorks
Your Internet Enabler
http://www.fpr-coworks.com

Sorry I created a pull request on root project, not yours.. Let you
remove it..
Le 03/04/2016 18:54, Sobolev Nikita a écrit :

Wow! That's a lot of changes! Thanks for your work.
Could you please squash your commits properly? You are also trying to
merge with master instead of |version-1.0.0| branch.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/flask-restful-swagger/flask-restful-swagger/pull/8#issuecomment-205009195

--
Guillaume DOUMENC
FPR / CoWorks
Your Internet Enabler
http://www.fpr-coworks.com

Closed the root PR.

This is not good, but since we don't have tests at all, this is acceptable (for now).

Sorry, I didn't get the idea about your's number 2. Can you please provide code samples?

I haven't dig this out yet. It was a good proposal on the main project. I thought that it might be a nice feature to implement, but no rush for this one.

About static files. I have used the proper version of the swagger-ui which supports both versions of swagger's specs. Were there any issues for you?

;)
If you define the get method for example by inheritance, the get operation (which will be the same method for all resources) will be used several times. Then adding informations on this method will be overridden by last used. Using decorator, this won't occur as the decorator is set on the root method not on the inherited class...
Using inherited methods forces to change replacing resource.origin__.dict__ by dir(resource.origin) in:

def extract_operations(resource):
    operations = []
    for k in dir(resource.orig):
        v = getattr(resource.orig, k)

At least we should provide import depending on ORM used (as done for swagger version)
Not quite sure I got an issue with this... I will check again using your files. Anyway I think we should not provide the static files but write in the doc how to get it from swagger-ui.

Oh, I got it. Thanks for the fix.
I will experiment with ORM and models this week.
I disagree with you about that we should not provide static files at all. My plan was to have a gitmodule to contain these files. Also, it is possible to integrate collectstatics command to have them in one place. Because why not? If we document how to change them (there are static_url and static_dir options), how to serve them in production and storing them as a submodule - it is totally fine by me. So it is easy to setup up and test.

It is not fixed.. We should provide another implementation for having operation informations on resource not on operation...
Ok for the static files, I will reset. Also good idea you planned to have gitmodule..

@gdoumenc @sobolevn Question on how Swagger can be used by to pass auth_token or some form of auth in header or as cookies to API server ? And if its possible someone can integrate their own function/module to allow authentication .. So swagger UI can use it ?

With the old version you have to fork-and-customize the code.
But yes, it is possible with this version. What you have to do is to create your own swagger-ui folder and set it with template_folder and/or static_folder. We will have this feature documented when the new version will be released.

@sbololevn Thanks for the update! ill keep an eye out for the next release and docs


Add syntax highlighting

Brief description
Just add syntax highlighting in the quick start code
Developer Certificate of Origin

[x] I certify that these changes according to the Developer Certificate of Origin 1.1 as described at https://developercertificate.org/.

Great, thank you for the update!


Remove forEach method from NodeList

According to MDN forEach() isn't in any Spec. It's only implemented by some Vendors, to be nice to us developers. Even though there is a wide adoption, this shouldn't be in Flow.
At the guardian we are trying to align the flow behavoir with polyfill.io so that all developers can rely on the type definitions being eventually polyfilled.
Opinions?

Sorry for the test failures. I can't see how they relate to the actual change. Do you have a hint how to fix them?

Note the different line numbers in the error messages. You removed a line and that broke all of the tests whose error messages reference lines after your alteration.
I would add a blank line at your removal point and run the tests to remove all of the line numbering noise so that you can see what tests your changes have broken. Once you understand the implications of your changes and have repaired the tests, then the runtests.sh shell program has a -r flag for updating the tests' .exp files (execute .runtests.sh --help for specifics).

@popham Thanks for the help! I checked the tests with the empty line instead and it seems no tests are affected ✨. Regenerated the .exp file for dom.

Hmm, this is a tricky situation. According to the linked spec, there is a candidate recommendation spec which gives all iterables in the webidl spec, including NodeList, a forEach method. When new things show up in the spec, we could in theory make them nullable, so that people have to make sure that they exist before using them. In practice, though, we don't.
So I'm leaning against taking this PR, since this feature is in the candidate spec.
What do you think?

NodeList is defined as "iterable". Per https://heycam.github.io/webidl/#es-forEach (emphasis mine):

If the interface has any of the following:

an iterable declaration
a maplike declaration
a setlike declaration

then a property named “forEach” must exist with attributes { [[Writable]]: true, [[Enumerable]]: true, [[Configurable]]: true } and whose value is a function object.


Unable to use Removable USB devices with Kali Linux

Windows Version
Microsoft Windows [Version 10.0.22621.1555]
WSL Version
WSL version: <IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
Linux version <IP_ADDRESS>-microsoft-standard-WSL2
Distro Version
Kali GNU/Linux Rolling on Windows 10 x86_64
Other Software
usbipd-win version 2.4.1
Repro Steps

Plug in drive and run GET-CimInstance -query "SELECT * from Win32_DiskDrive"

mount the disk with wsl --mount and fail

output from dmesg https://pastebin.com/QEdF3sC8

Expected Behavior
To be able to see the drive mounted on the the Linux Distro
Actual Behavior
IT failed to mount with many errors.
Error file: https://pastebin.com/QEdF3sC8
Diagnostic Logs
WslLogs-2023-04-23_18-37-21.zip

Step 1 shows that the device has a partition, but you didn't specify --partition.
If the partition is NTFS the expected failure will occur.
Use --bare instead of --partition, and mount it manually.

Ah ok, that fixed it


Load multiple models for Inference?

Hi, I was wondering that, is that possible for me to load multiple models together for camera inference? I need to feed the output of one model as an input to another model while the camera keeps streaming, but I got error like 'Cannot load model during processing' 'Cannot unload model during processing', is there any suggestion to solve it? Very grateful!

@Aileenlingyu That's kind of possible, depends on your use case. Please look into InferenceEngine class inside inference.py. You can load multiple models by calling load_model() several times. Existing CameraInference and ImageInference are just wrappers around InferenceEngine.

@dmitriykovalev

That's kind of possible, depends on your use case.

What use cases will this work for? I'm trying to achieve a similar thing but feeding a camera stream into two different models in the same script. Looking at the source code (inference.py) it looks like calling load_model() multiple times will overwrite the previous loaded model.
Is there a way to run two models simultaneously?

@ritwikbiswas Each model has unique identifier (model_name in python code), it's possible to load multiple models at the same time and then use that identifier to reference them. The tricky part is that you can only use one model for camera inference (at least for now), you cannot process each camera frame with more than one model.

Hi @dmitriykovalev , thanks for the suggestion! Is that possible to process the each camera frame with only one model but add a post-processing using another model?

@Aileenlingyu Currently this is not possible but could be implemented in future. I'll mark this issue with "enhancement" label.

Any update on this?

This is something that can be done but that's not the highest priority right now.


BUG: Fix captain selection on Map veto

Discovered this bug when developing a fork. When a match is loaded, players are not swapped to their correct teams.
However, the map veto process assumes they are, so when the veto starts it will select captains based on in-game teams.
Example:

Match loaded
team1 is Terrorists
team2 is CTs
A player from team2 is current on the Terrorist team
Map veto starts
The team2 player that is currently on Terrorist is selected as the captain for team1 (wrong)

All this does is swap players to their correct teams and enforce jointeam restrictions during the veto process.

Thanks for catching it and for the fix!


Make a custom biblatex style that supports various modern and custom citation types

[ ] @githubissue: GitHub issue, PR, or commit
[ ] @uip: Urbit improvement proposal
[ ] @software: Software project or library
[ ] @blog: Blog or social media post

Not quite there yet but you can see what I've added to the mss.bib file now.


new rule: react-a11y-iframe

All iframe elements must have a title:
• https://www.w3.org/TR/WCAG20-TECHS/H64.html
• http://oaa-accessibility.org/wcag20/rule/10/
Title attributes for frames must be unique.
• http://oaa-accessibility.org/wcag20/rule/11/
Frames should not be hidden or empty.
• http://oaa-accessibility.org/wcag20/rule/12/
Reference: https://www.w3.org/TR/WCAG10/wai-pageauth.html#tech-frame-longdesc
Notes:
• frame/iframe longdesc attribute is deprecated and not supported by React.
frame element not supported by React

@JoshuaKGoldberg @HamletDRC I'll try this one if it's ok... :)

@noamyogev84 go for it!


server TCP connection

Hi,
should server initiate the TCP connection binding to port 830  (RFC6242) ?  As I can see now it uses a random allocated port.

Not sure what exactly you mean. It does listen on port 830 by default but generates a random port for each connection, which is simply how it works. You cannot have all the clients connected on a single port, obviously. Or what exactly is the problem?


Allow deletion

Allow deletion of entries.
Suggestion: -d as a flag for deletion.
If more than one entries match the search term, a confirmation message appears mentioning the number of found matches.
Example 1 (one result):
memb -d "tar"
Deleted statement: WHOLE STATEMENT HERE

Example 2 (multiple results):
memb -d "tar"
2 statements match the given search-term, are you sure you want to delete all 2 entries? [y/n]

y -> Deleted 2 statements.
n -> Do nothing.

Fixed, although only deletion per line number is possible at the moment.


Own properties

Is there no way to pass own props? Like this

this.$notify({
    img: 'https://pbs.twimg.com/profile_images/848471660860538880/pevXVsIp_400x400.jpg',
    link: '/profile/'
    title: 'Message',
    text: 'You got new message'
})

Nope, i dont think there is a way to do it right now.


[feature request] select support label in value

Existing Component
是
Component Name
select
Description
set select return value what include label and value, which default return value.

It's useful. I get options by the remote search then select multiple options and search again. when I just want to get the labels I selected. I have to maintain a label list or get labels by $ref.children which is a hassle.


Run performance test against fork point commit

Context
This closes https://github.com/gradle/gradle-private/issues/1526
Basically, the idea is, we should run feature branch performance tests against the distribution which is built from the "fork point commit". This can reduce the effects of performance regression which is not introduced by the feature branch.
Two tasks are added into the build and depended by performance task:

determineForkPoint, run git command to determine the fork point commit. It's done during the execution phase, then it sets the commit for buildForkPointDistribution task.
buildForkPointDistribution, try to build a distribution from the fork point commit.

buildForkPointDistribution is cachable so it only needs to run once. Its input is the fork point commit, output is two files: a binary distribution and a tooling api jar. The generated binary distribution is used to run individual worker tests.
It looks like this in performance report:

@blindpirate when do you think this will be ready for review again?

In https://github.com/gradle/gradle-private/issues/1526#issuecomment-424712119, Lorant also suggest to add an optional input directory for the distribution location. I would prefer that very much over using the magic location as we do now.

@lptr I think this is ready for another review.

Thanks @wolfs @eskatos @lptr ! Can you please take another look?

I don't, @big-guy do you have anything to say?


Not able to run rttm viewer in colab

Hi,
I followed the steps in your repo and tried to run in google colab. But some error is pooping during install.py. Please check and if possible give an example on colab

Also when removing the install.py line, it runs but not able to view the result

PS: runs flawlessly on local environment. Looks great!
Thank You

Hi @AMITKESARI2000,
The viewer currently doesn't run on colab because it requires access to the browser.
The goal of the installation script is to make the viewer available through a right click on an RTTM file, so it doesn't apply to colab either.
Glad to hear it works locally!

Just a suggestion, I saw that optuna library can plot interactive graphs like yours in a jupyter notebook as well, example.
If possible, hope to see it in future!


Terminus becomes unresponsive over time

After about a day or two Terminus becomes very slow in switching tabs and unresponsive to the degree I have to close the application are re-start it.
I am using about 8 tabs with ssh connection to cloud servers.
Version: 1.0.77
Platform: win32 10.0.17134
Plugins: none

Since you open/close a lot of tabs this should be resolved with #1263 :wink:


[DRAFT] Exit test suite gracefully

Ref issue #28.
Note that this PR is stacked on top of PR #26, which is also in draft state.

Closing because #94 addresses this.


Use zig as compiler instead of clang

What is zig?
Zig is a general-purpose programming language and toolchain for maintaining robust, optimal and reusable software.
It's really three things:

Programming language Not important for hazel
Build system (build.zig) replacing makefiles/cmake/ninja/etc
C/C++ compiler with an emphasis on cross-compilation

Why use zig over clang?
These 2 blog posts describe the reasons better than i could, but heres a tldr:

cross-compilation from any major os+isa to any major os+isa
small archive (~45mb on linux) with single static executable under mit license, that ships with common libraries like libc, libcxx, compiler-rt, tsan, libunwind
builtin caching
package manager
better defaults (enables UBSAN, uses all supported cpu features, e.g. SSE or AVX)
drop-in replacement

Guide on setting up zig as c/c++ compiler

Compile a C/C++ Project with Zig

Overview of the main features of zig as c/c++ compiler

zig cc: a Powerful Drop-In Replacement for GCC/Clang

Links and further reading
Official website

https://ziglang.org

Examples

Example of small project using zig as compiler and toolchain
Example of larger project using zig as compiler and toolchain

I thought Hazel mainly uses MSBuild/MSVC and not Clang. How would you integrate this into Visual Studio?

@Kirdow

I thought Hazel mainly uses MSBuild/MSVC and not Clang. How would you integrate this into Visual Studio?

I came here after chernos lastest video Hazel Engine ON LINUX!. The new linux builds use clang.
Using zig would drop the need for extra linux ci/cd or simplify it greatly since there would be only one runner using zig compiling for all platforms and not one runner for each platform.

That video you linked is for Hazel 3D/Big Hazel. Not Hazel 2D/Game Engine series which this repository is about. The repository you're after is under StudioCherno available to Patrons.

That video you linked is for Hazel 3D/Big Hazel. Not Hazel 2D/Game Engine series which this repository is about. The repository you're after is under StudioCherno which is available to Patrons.

Ok, sorry my bad. I'll close the issue then.


Recognition of random words by speech recognition from microphone with endpoint detection

I am using this script for building live speech recognition API. But it is recognizing some random words in between. I have attached a figure which shows random words are getting recognized at indices 2,4 and 10. No words have been uttered at this indices.

Could you use
https://k2-fsa.github.io/sherpa/ncnn/pretrained_models/zipformer-transucer-models.html#csukuangfj-sherpa-ncnn-streaming-zipformer-en-2023-02-13-english
for testing if you are speaking English?
Also, please have a look at
http://github.com/k2-fsa/sherpa-onnx
which supports more models.


Feature Request: switch back & forth in Pay to Many to allow copy & paste of several addresses

Currently, when you click on pay to many, it will not allow you to go back to your transaction history or addresses tabs to allow copying addresses then going back to paste these back in pay to many. I think this would be a very useful feature. Thanks!

The trouble with making dialogs non-modal is that if you click anywhere else on the application, they disappear behind the window that was clicked on. As trivial as it seems, it can be non-obvious that the dialog needs to be retrieved from that state. It's possible to make the windows underneath interactive and still keep the dialog on top, but then it appears on top of all windows on the desktop, which is generally not what's wanted (you may want to copy addresses from another application).
I'll consider this further though - can you elaborate on the use case?


add IServiceProvider to the constructor to generate parameters by Dep…

…endency injection for the CustomRepository method And Test Ef 5

It's already supported by the library.
EF Core has an internal IServiceProvider which can be accessed through DbContext.GetInfrastructure() with the main purpose of resolving services directly using the service locator pattern.
The library checks if any repository (custom or generic) has been registered into the MS Built-In container and also tries to resolve it from there, in case the repository hasn't been registered it uses the internal factory in order to instantiate the repository. This behavior can be found here and here.
The library also counts on some extensions to help developers registering repositories. The extensions can be found here and here.
Both features can/should be used together to make it happen.
I appreciate the time you've spent on this feature, but unfortunately, it's already covered.


Tweak dependent package's versions

WHY
WHAT
REF

Codecov Report

Merging #187 into master will decrease coverage by 0.14%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #187      +/-   ##
==========================================
- Coverage   61.72%   61.57%   -0.15%
==========================================
  Files          39       39
  Lines        1395     1395
==========================================
- Hits          861      859       -2
- Misses        475      476       +1
- Partials       59       60       +1

Impacted Files
Coverage Δ

pkg/grapiserver/cmux.go
100% <100%> (ø)
:arrow_up:

pkg/grapiserver/gateway.go
75% <0%> (-3.58%)
:arrow_down:

Continue to review full report at Codecov.

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


rfc: use trait to export method on type

in itertools, num_trait, the libraries provide method by trait.
so import the trait, user can directly use the methods by dot.
use library::a_trait

t.library_method()

pros: unify 2d and 3d vector and else
example:
trait Arr<T> {
    fn add(self, o: T) -> Self;
}
impl Arr<[i32; 2]> for [i32; 2] {
    fn add(self, o: [i32; 2]) -> Self {
        [self[0] + o[0], self[1] + o[1]]
    }
}
impl Arr<[i32; 3]> for [i32; 3] {
    fn add(self, o: [i32; 3]) -> Self {
        [self[0] + o[0], self[1] + o[1], self[2] + o[2]]
    }
}

fn main() {
    [1, 2].add([2, 3]);
    [1, 2, 3].add([2, 3, 4]);
    println!("Hello, world!");
}

for any dimensiion
trait Arr<T, const N: usize> {
    fn add(self, other: T) -> Self;
}

impl<const N: usize> Arr<[i32; N], N> for [i32; N] {
    fn add(self, other: [i32; N]) -> Self {
        std::array::from_fn(|i| self[i] + other[i])
    }
}

pub fn main() {
    // Test with different array sizes
    let arr1 = [1, 2].add([2, 3]);
    let arr2 = [1, 2, 3].add([2, 3, 4]);
    let arr3 = [1, 2, 3, 4].add([2, 3, 4, 5]);

    println!("2D array result: {:?}", arr1);
    println!("3D array result: {:?}", arr2);
    println!("4D array result: {:?}", arr3);
}

Thanks for your suggestion. I made a vecn.rs file to put such general codes for static array vectors.
https://github.com/nobuyuki83/del-geo/blob/main/del-geo-core/src/vecn.rs


Output molecular weights in MusicBox configurations

MusicBox configuration data includes species with molecular weight = 0. Update the configuration generating scripts to calculate MW based on the chemical formula

Closing as this does not apply to the new version of chemistry cafe.


Mobile Example App

This ticket likely needs further breakdown.
https://github.com/livepeer/livepeer-demo-react-native
Currently, the existing example app currently doesn't work because it hits the old API. I think this a good point to reflect on the experience we want to deliver with this sample application and decide on the right approach.
Requirements for a mobile sample application:

Should allow users to quickly understand Studio's capabilities including broadcasting, playback, and On Demand.
Should allow users to quickly replicate the functionality in the application in their own app
Should be compatible across devices (iOS / Android)
Should allow users to input a stream key to begin broadcasting
Should allow users to seamlessly switch between front and back cameras
Should allow users to use a default API key to upload and playback video.
Should use the Livepeer Player

Questions:

Is the existing example app salvageable? Do we want to use this as an example app in the future? The app in the demo looks good from a design perspective but is missing On Demand?

Here an update about the React Native app.
The app has not been really updated in 1.5 years. So it's broken in multiple ways.
Here the issues:

the API urls are pointing to the old endpoints, easy fix done on my branch
the api authentication flow is broken, easy fix done on my branch too
the codebase does not build on Apple Silicon processors (M1, M2, etc). This is mainly due to the Camera and Video dependency being deprecated and not updated for latest macs (react-native-nodemediaclient). It needs to be replaced. I am in the process of doing this
React Native is way too old to actually make a migration. Everything breaks. So I created  a new React Native project and transferred all the still broken code from the old app and I am fixing it

What I am suggesting is that I take an extra day to fix all these issues (just replacing the camera is actually outstanding), then we can publish and updated React Native app that works and that is usable in hackatons and such.
Later, depending on priorities, we can decide to dedicate some time and improve the codebase quality (modernise it a bit, medium effort) and add new features.


stroring an encrypted BSON::Binary

I started using mongodb in my application and I'm now looking for an easy way to encrypt some sensitive binary data and store it in the mongodb. When I do it for my MySql DB I use the attr_encrypted, but I guess that with mongoid, I'll need some other gem or method to do it, especially with a non-string field (BSON::Binary ).
Any suggestions?

There's no way out fo the box to do encryption with Mongoid (we don't have an attr_encrypted), so I would recommend encrypting your value before setting it in a field as either a BSON::Binary or as a String.


E/AudioAttributes(20124): Invalid stream type -2147483648 for AudioAttributes

I keep getting this error, but apart from the message it works properly. Should I bother?
E/AudioAttributes(20124): Invalid stream type -2147483648 for AudioAttributes
W/SoundPool(20124): App requested 100 channels
I/OMXClient(20124): IOmx service obtained
W/AudioTrack(20124): AUDIO_OUTPUT_FLAG_FAST denied by server; frameCount 0 -> 4829
I/OMXClient(20124): IOmx service obtained
W/AudioTrack(20124): AUDIO_OUTPUT_FLAG_FAST denied by server; frameCount 0 -> 4829
I/OMXClient(20124): IOmx service obtained

I've the same issue.
After investigating the problem occurs when you use PlayerMode.LOW_LATENCY:
https://github.com/luanpotter/audioplayers/blob/master/android/src/main/java/xyz/luan/audioplayers/AudioplayersPlugin.java#L157
At line 248 : AudioManager.USE_DEFAULT_STREAM_TYPE causes this error because the constant exists but is not is not handled :
https://github.com/luanpotter/audioplayers/blob/master/android/src/main/java/xyz/luan/audioplayers/WrappedSoundPool.java#L240
https://android.googlesource.com/platform/frameworks/base/+/refs/heads/master/media/java/android/media/AudioAttributes.java#882
In my opinion it's just a trick to let Android choose the default stream type: https://android.googlesource.com/platform/frameworks/base/+/refs/heads/master/media/java/android/media/AudioManager.java#694

Closing this due to reports of this being fixed on newer versions. Feel free to open a new issue in case of this still happening


name SSM doc

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Only issue I see here is accounts with multiple stacks deployed will conflict on the name...

Need a review on this


Error Reduce of empty array with no initial value

This is coming from Flo integration into SCDF UI. Moving to SCDF UI: https://github.com/spring-cloud/spring-cloud-dataflow-ui/issues/1289


thread 'main' panicked at 'called Result::unwrap() on an Err value: InvalidHeaderValue { _priv: () }', src\libcore\result.rs:997:5

🐛 Bug Report
Environment

operating system: Windows 10
output of rustc -V: rustc 1.35.0 (3c235d560 2019-05-20)
output of node -v: v10.15.3

Steps to reproduce

create a new rust project from the template
put account id into wrangler.toml
wrangler publish

What did you expect to see?
Either my project published or an understandable error
What did you see instead?
with RUST_BACTRACE:
> wrangler publish
 Compiling your project to WebAssembly...
[INFO]: Checking for the Wasm target...
[INFO]: Compiling to Wasm...
    Finished release [optimized] target(s) in 0.05s
[INFO]: Installing wasm-bindgen...
[INFO]: Optional fields missing from Cargo.toml: 'description', 'repository', and 'license'. These are not necessary, but recommended
[INFO]: :-) Done in 0.38s
[INFO]: :-) Your wasm pkg is ready to publish at .\pkg.
thread 'main' panicked at 'called `Result::unwrap()` on an `Err` value: InvalidHeaderValue { _priv: () }', src\libcore\result.rs:997:5
stack backtrace:
   0: <unknown>
   1: <unknown>
   2: <unknown>
   3: <unknown>
   4: <unknown>
   5: <unknown>
   6: <unknown>
   7: <unknown>
   8: <unknown>
   9: <unknown>
  10: <unknown>
  11: <unknown>
  12: <unknown>
  13: <unknown>
  14: <unknown>
  15: <unknown>
  16: <unknown>
  17: <unknown>
  18: BaseThreadInitThunk
  19: RtlUserThreadStart

duplicate of https://github.com/cloudflare/wrangler/issues/389; closing.


Grading UI/UX: Grading Overview Showing X Entries Per Page Incorrect

The showing X entries per page drop down box has a bug where it only shows 10 entries no matter what value is selected.

the feature was added by #2787, which is the most likely cause of the bug


../hw/per/psp/pc/psp_tmr.c:13:21: fatal error: pthread.h: No such file or directory

Hi, this looks like a great library. I'm having an issue when I try and build:
17:43:43 ** Incremental Build of configuration Debug for project lv_pc **
make all
'Building file: ../hw/per/psp/pc/psp_tmr.c'
'Invoking: Cross GCC Compiler'
gcc -I"C:\Users\Josh\Documents\proj_pc" -O0 -g3 -Wall -c -fmessage-length=0 -MMD -MP -MF"hw/per/psp/pc/psp_tmr.d" -MT"hw/per/psp/pc/psp_tmr.o" -o "hw/per/psp/pc/psp_tmr.o" "../hw/per/psp/pc/psp_tmr.c"
../hw/per/psp/pc/psp_tmr.c:13:21: fatal error: pthread.h: No such file or directory
#include <pthread.h>
^
compilation terminated.
hw/per/psp/pc/subdir.mk:21: recipe for target 'hw/per/psp/pc/psp_tmr.o' failed
make: * [hw/per/psp/pc/psp_tmr.o] Error 1
17:43:44 Build Finished (took 947ms)

It seems you are using  windows. In this case you nee to download pthread.dll too.
http://sourceware.org/pthreads-win32/

I tried it on Windows again and need to do the following steps:

Download pthread from here: ftp://sourceware.org/pub/pthreads-win32/pthreads-w32-2-9-1-release.zip
Copy Pre-built.2/include/ content to C:/MinGW/include
Copy Pre-built.2/lib/x64/ content to C:/MinGW/lib (If you have a 64 machine)
Copy Pre-built.2/dll/x64/ content to {eclipse_worksapce}/lv_pc/Debug/
I got an error: 'struct timespec' redefined. To solve this: Project properties -> C/C++ Build -> Settings -> GCC C++ Compiler -> Miscellaneous -> Others flag and add  -DHAVE_STRUCT_TIMESPEC (it could be a pthread issue)
Download the development libraries of SDL from https://www.libsdl.org/download-2.0.php here Development Libraries: SDL2-devel-2.0.5-mingw.tar.gz
To get it work I need to use the i686 (32 bit) libraries even on 64 bit system. So if you get undefined reference to SDL_init() error try the x86_64_w64_mingw folder
Copy i686-w64-mingw32/include/SDL2 folder to C:/MinGW/include
Copy i686-w64-mingw32/lib/ content to C:/MinGW/lib
Copy i686-w64-mingw32/bin/SDL2.dll content to {eclipse_worksapce}/lv_pc/Debug/

Does it working now?

In v5.0.0 pthread is already not used. More info


Add 'thiram' to anti-infective list

See https://github.com/PHI-base/phipo/issues/414
https://github.com/PHI-base/phi-eco/issues/40

Done


Refactor GO gRPC file generation to support a folder per service under app/interface/rpc/proto

The makefile $(PB_GO_FILES) rule supports all proto files under app/interface/rpc/proto. Now:
Given: The makefile $(PB_GO_FILES) rule
When: It finds proto files under app/interface/rpc/proto<<service>> folders
Then: The makefile $(PB_GO_FILES) rule generates the *pb.go and ancillary files in their app/interface/rpc/proto<<service>> folders

Done under #3, using BUF


Implement simple daily show-stopping backup & restore, for business apps used during work-hours

The backup strategy is to archive the contents of the data/ directory.
To restore a backup, checkout the repository again, and extract the data/ directory backup into it.
After running vagrant up all the data should be there, the username and password should be the same etc.
The simplest way of automating a backup is to shut-down the box and compress the psql.vdi file, then storing it in the backups directory.
Yes, this is a 'show-stopper', but for my use-case I don't have a problem with it. It will be running business applications which can safely go offline out of standard work hours for a couple of hours.
What needs to happen? Ideally:
make backup

What does it do:?

Ensure the Vagrant box is halted, if not: exit
Compact psql.vdi, using vboxmanage modifymedium <filename> --compact
Compress the whole data/ directory into a .tar.xz file
Store the .tar.xz file in the backups/ directory

Ideally, it should also:

Rotate backups, keeping 7 daily, 4 weekly, 6 monthly (customisable?)
Run third-party scripts to transfer backups to off-site location
Remove monthly backups if data goes over size

Then to restore a backup, it should be very simple:
make restore

What does it do?

Extract the latest available backup from the backups/ directory

Minor problems which may be had:

UUID of psql.vdi changing between instances of vagrant the box (e.g. move from one host to another, or doing a destroy) - this should be handled properly by the vagrant persistent plugin.

Ok this has been implemented now.


Deb package fails to install via apt module

Hi Jeff!  First, thanks for all the great work you do and for sharing it!
While working to add Ubuntu 18.04 support to my Check_MK module, I'm running into a strange error that I hope you can help me understand.
I suspected the .deb file, but it installs fine manually using "dpkg -i" and my local Ansible playbook run.
For clarity, I will use separate comments for the error in full "-vvvv" mode, followed by the same task run successfully in my demo environment.
Is there a new convention in 18.04 I'm missing?
I see something about AnsiballZ in your output but not mine - is that a clue or red herring?

From within docker-ubuntu1804-ansible:
TASK [role_under_test : Install Check_MK RAW | APT] *******************************************************************************************************************************************
task path: /etc/ansible/roles/role_under_test/tasks/main.yml:49
<localhost> ESTABLISH LOCAL CONNECTION FOR USER: root
<localhost> EXEC /bin/sh -c 'echo ~root && sleep 0'
<localhost> EXEC /bin/sh -c '( umask 77 && mkdir -p "` echo /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403 `" && echo ansible-tmp-1546394919.06-26203703697403="` echo /root/.ansi
ble/tmp/ansible-tmp-1546394919.06-26203703697403 `" ) && sleep 0'
Using module file /usr/local/lib/python2.7/dist-packages/ansible/modules/packaging/os/apt.py
<localhost> PUT /root/.ansible/tmp/ansible-local-2356aElf7n/tmpPec1Xf TO /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403/AnsiballZ_apt.py
<localhost> EXEC /bin/sh -c 'chmod u+x /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403/ /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403/AnsiballZ_apt.py && sleep 0'
<localhost> EXEC /bin/sh -c '/usr/bin/python /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403/AnsiballZ_apt.py && sleep 0'
<localhost> EXEC /bin/sh -c 'rm -f -r /root/.ansible/tmp/ansible-tmp-1546394919.06-26203703697403/ > /dev/null 2>&1 && sleep 0'
The full traceback is:
WARNING: The below traceback may *not* be related to the actual failure.
  File "/tmp/ansible_apt_payload_zywB21/__main__.py", line 622, in install_deb
    pkg = apt.debfile.DebPackage(deb_file)
  File "/usr/lib/python2.7/dist-packages/apt/debfile.py", line 61, in __init__
    self.open(filename)
  File "/usr/lib/python2.7/dist-packages/apt/debfile.py", line 71, in open
    control = self._debfile.control.extractdata("control")

fatal: [localhost]: FAILED! => {
    "changed": false,
    "invocation": {
        "module_args": {
            "allow_unauthenticated": false,
            "autoclean": false,
            "autoremove": false,
            "cache_valid_time": 0,
            "deb": "/opt/check-mk-raw-1.5.0p7_0.bionic_amd64.deb",
            "default_release": null,
            "dpkg_options": "force-confdef,force-confold",
            "force": true,
            "force_apt_get": false,
            "install_recommends": null,
            "only_upgrade": false,
            "package": null,
            "purge": false,
            "state": "present",
            "update_cache": null,
            "upgrade": null
        }
    },
    "msg": "Unable to install package: There is no member named 'control'"
}

From within my local VirtualBox environment:
TASK [kso512.install-check_mk-server : Install Check_MK RAW | APT] ****************************************************************************************************************************
task path: /home/kso/Documents/git/projects/ansible/roles/kso512.install-check_mk-server/tasks/main.yml:49
Using module file /usr/local/lib/python2.7/dist-packages/ansible/modules/packaging/os/apt.py
<uad-00016> ESTABLISH SSH CONNECTION FOR USER: None
<uad-00016> SSH: EXEC ssh -vvv -C -o ControlMaster=auto -o ControlPersist=60s -o KbdInteractiveAuthentication=no -o PreferredAuthentications=gssapi-with-mic,gssapi-keyex,hostbased,publickey -o PasswordAuthentication=no -o ConnectTimeout=10 -o ControlPath=/home/kso/.ansible/cp/b5908d20b0 uad-00016 '/bin/sh -c '"'"'sudo -H -S  -p "[sudo via ansible, key=cxklccjahyfkswlhshwzigkldekzxyfj] password: " -u root /bin/sh -c '"'"'"'"'"'"'"'"'echo BECOME-SUCCESS-cxklccjahyfkswlhshwzigkldekzxyfj; /usr/bin/python'"'"'"'"'"'"'"'"' && sleep 0'"'"''
Escalation succeeded
<uad-00016> (0, '\n{"invocation": {"module_args": {"dpkg_options": "force-confdef,force-confold", "upgrade": null, "force": true, "force_apt_get": false, "install_recommends": null, "package": null, "autoclean": false, "purge": false, "allow_unauthenticated": false, "state": "present", "autoremove": false, "update_cache": null, "default_release": null, "only_upgrade": false, "deb": "/opt/check-mk-raw-1.5.0p7_0.bionic_amd64.deb", "cache_valid_time": 0}}, "diff": {"prepared": "Selecting previously unselected package check-mk-raw-1.5.0p7.\\n(Reading database ... 179616 files and directories currently installed.)\\nPreparing to unpack .../check-mk-raw-1.5.0p7_0.bionic_amd64.deb ...\\nUnpacking check-mk-raw-1.5.0p7 (0.bionic) ...\\nSetting up check-mk-raw-1.5.0p7 (0.bionic) ...\\nProcessing triggers for ureadahead (0.100.0-20) ...\\nProcessing triggers for systemd (237-3ubuntu10.10) ...\\nProcessing triggers for man-db (2.8.3-2ubuntu0.1) ..."}, "changed": true, "stderr": "update-alternatives: using /omd/versions/1.5.0p7.cre to provide /omd/versions/default (omd) in auto mode\\nAdding system group omd\\nNew default version is 1.5.0p7.cre.\\nInstalling systemd service: omd.init\\nDisabling startup through SysV (/etc/init.d/*) scripts\\nActivating startup during system boot\\n", "stdout": "Selecting previously unselected package check-mk-raw-1.5.0p7.\\n(Reading database ... 179616 files and directories currently installed.)\\nPreparing to unpack .../check-mk-raw-1.5.0p7_0.bionic_amd64.deb ...\\nUnpacking check-mk-raw-1.5.0p7 (0.bionic) ...\\nSetting up check-mk-raw-1.5.0p7 (0.bionic) ...\\nProcessing triggers for ureadahead (0.100.0-20) ...\\nProcessing triggers for systemd (237-3ubuntu10.10) ...\\nProcessing triggers for man-db (2.8.3-2ubuntu0.1) ...\\n"}\n', 'OpenSSH_7.9p1 Debian-4, OpenSSL 1.1.1a  20 Nov 2018\r\ndebug1: Reading configuration data /etc/ssh/ssh_config\r\ndebug1: /etc/ssh/ssh_config line 19: Applying options for *\r\ndebug1: auto-mux: Trying existing master\r\ndebug2: fd 3 setting O_NONBLOCK\r\ndebug2: mux_client_hello_exchange: master version 4\r\ndebug3: mux_client_forwards: request forwardings: 0 local, 0 remote\r\ndebug3: mux_client_request_session: entering\r\ndebug3: mux_client_request_alive: entering\r\ndebug3: mux_client_request_alive: done pid = 7119\r\ndebug3: mux_client_request_session: session request sent\r\ndebug3: mux_client_read_packet: read header failed: Broken pipe\r\ndebug2: Received exit status from master 0\r\n')

changed: [uad-00016] => changed=true                                                                                diff:                                                                                prepared: |-                                                                                Selecting previously unselected package check-mk-raw-1.5.0p7.                                                                                (Reading database ... 179616 files and directories currently installed.)                                                                                Preparing to unpack .../check-mk-raw-1.5.0p7_0.bionic_amd64.deb ...                                                                                Unpacking check-mk-raw-1.5.0p7 (0.bionic) ...
      Setting up check-mk-raw-1.5.0p7 (0.bionic) ...
      Processing triggers for ureadahead (0.100.0-20) ...
      Processing triggers for systemd (237-3ubuntu10.10) ...
      Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
  invocation:
    module_args:
      allow_unauthenticated: false
      autoclean: false
      autoremove: false
      cache_valid_time: 0
      deb: /opt/check-mk-raw-1.5.0p7_0.bionic_amd64.deb
      default_release: null
      dpkg_options: force-confdef,force-confold
      force: true
      force_apt_get: false
      install_recommends: null
      only_upgrade: false
      package: null
      purge: false
      state: present
      update_cache: null
      upgrade: null
  stderr: |-
    update-alternatives: using /omd/versions/1.5.0p7.cre to provide /omd/versions/default (omd) in auto mode
    Adding system group omd
    New default version is 1.5.0p7.cre.
    Installing systemd service: omd.init
    Disabling startup through SysV (/etc/init.d/*) scripts
    Activating startup during system boot
  stderr_lines:
  - 'update-alternatives: using /omd/versions/1.5.0p7.cre to provide /omd/versions/default (omd) in auto mode'
  - Adding system group omd
  - New default version is 1.5.0p7.cre.
  - 'Installing systemd service: omd.init'
  - Disabling startup through SysV (/etc/init.d/*) scripts
  - Activating startup during system boot
  stdout: |-
    Selecting previously unselected package check-mk-raw-1.5.0p7.
    (Reading database ... 179616 files and directories currently installed.)
    Preparing to unpack .../check-mk-raw-1.5.0p7_0.bionic_amd64.deb ...
    Unpacking check-mk-raw-1.5.0p7 (0.bionic) ...
    Setting up check-mk-raw-1.5.0p7 (0.bionic) ...
    Processing triggers for ureadahead (0.100.0-20) ...
    Processing triggers for systemd (237-3ubuntu10.10) ...
    Processing triggers for man-db (2.8.3-2ubuntu0.1) ...
  stdout_lines: <omitted>

I went ahead and updated a branch with the changes and pushed it to Travis CI, in the hopes it would shed some light on the situation.
Here's the result, showing the same "Unable to install package: There is no member named 'control'" error:
https://travis-ci.org/kso512/install-check_mk-server/builds/474288340
Thanks!

Add xz-utils to your install_check_mk_server_prereqs. This should help.

Hi Timon,
Thanks!!  This helps a lot, and now I think I'm on track.
I'm curious:  what clue lead you to suggest that xz-utils might be involved?
Thanks again,

kso512

@kso512 actually I was facing the same issue some time ago and was just following the error message:
  File "/usr/lib/python2.7/dist-packages/apt/debfile.py", line 71, in open
    control = self._debfile.control.extractdata("control")

TL;TR:

This is part of python-apt which was called inside Ansible and the apt functionality (function install_deb).

In apt.DebPackage._get_content you can see how it's trying to extract the data.

With some knowledge about the Debian package system you know that the data archive in deb files can vary (Wikipedia) and the filename is data.tar.XX.

Using hexdump you can find your data file and see which compression is used.
Using check-mk-raw-1.5.0p11_0.stretch_amd64.deb as example:

Now just ensure that you have the correct applications installed to decompress the archive.

Enjoy.

Wow, that would have taken me a LONG time to figure out on my own.  Big thanks!
This resolves my issue so I'll close it.


Add dialog to let user pick recon service when marking cells as new

Closes #4985.
Screenshot

Maybe I could add a Cypress test as well.

I plan to merge this this week, with the intention of releasing it in 3.7-beta1, unless there are any concerns.

Tried it, works fine!


Alpha channel in WebM files is not used during the compositing

Godot version:
3.0.2
OS/device including version:
Windows 10 x64
GPU : GTX1080
Issue description:
I try to use some WebM video files in my project, but more precisly WebM with Alpha Channel. This is super useful when you want to live composite some "lowerthird" other background elements (or other video streams).
I'm a professionnal broadcast engineer who try to integrate Godot 3 in my workflow for realtime purpose.
I already read the whole topic about the integration of WebM in Godot (thank you for that, Zaps166 !) :
https://github.com/godotengine/godot/issues/5886
Steps to reproduce:
Load the provided project and run it
Minimal reproduction project:
I provide a really simple test project, containing a WebM file with an alpha channel. The RGB channels was written in "Straight" mode (not premultiplied), so you don't have to divide the color values by the Alpha (to unpremult the color) before the compositing. By this way, I guess it should be easier for you to implement this feature. 😃
WebM_AlphaChannel.zip
You can drag and drop this *.webm file in Google Chrome to check the content, premultiplied by the alpha. Thank you for your wonderful work !

Might be related to #8334 ?

https://godotengine.org/qa/25657/webm-video-with-alpha-channel

Thank you for you feedbacks ! I'm happy not to be the only one to ask for this feature :)

Any progress on this feature?

@Crystalwarrior There's no ETA on this feature. Anyway, we're now closing feature proposals as they need to be reopened on the Godot proposals repository :slightly_smiling_face:

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Improvements for emacs

The window creation behaviour has changed, you can recover the old behaviour by putting the
following in your init.el:
(add-to-list
 'display-buffer-alist
 '(\"*ghcid*\"
   (display-buffer-reuse-window   ;; First try to reuse an existing window
    display-buffer-at-bottom      ;; Then try a new window at the bottom
    display-buffer-pop-up-window) ;; Otherwise show a pop-up
   (window-height . 18)      ;; New window will be 18 lines
   ))

Improvements:

Don't assume that nlinum-mode is available.
Respect user configuration when displaying the buffer. Now it works with purpose-mode and custom
display-buffer-alists
Automatically detect height of ghcid window.
Use the standard with-selected-window to temporarily select the ghcid window.
Use term-line-mode instead of term-char-mode, so the usual navigation shortcuts work in the ghcid
buffer.
Resets the directory of the ghcid process if you call ghcid again.
Fix bug where the next-error would not always go the the right place in a file.

Thanks for the pull request!

If you care, my PR preferences are at https://github.com/ndmitchell/neil#contributions, but they're all guidelines, and I'm not too fussy - you don't have to read them.

@WraithM - can you review? Totally out of my depth here.

@ndmitchell This is great! It works so much better than my version.
Thanks @yoricksijsling. Well done!

Thanks!


Instructions for adding APP ID /Credentials are missing in documentation README

Describe the bug
The README needs to provide instructions on how to setup AppID credentials for the TakeTwo
To Reproduce
There is no specific steps to reproduce. README is missing the required documentation on how to fill 3 fields
CLIENT_ID=
SECRET=
OAUTH_SERVER_URL=
Expected behavior
N/A
Screenshots
Attached is the section of the  README that needs updated documentation
Copy .env.sample file to .env file in the same directory. Fill out the CouchDB database and IBM App Id service credentials.
#CouchDB Credentials
DB_HOST=
DB_PORT=
DB_NAME=
DB_USERNAME=
DB_PASSWORD=
#IBM AppID Credentials
CLIENT_ID=
SECRET=
OAUTH_SERVER_URL=
Desktop (please complete the following information):
N/A
Smartphone (please complete the following information):
N/A
Additional context

Hi @shobhavijaystar thanks for this. I will add in some more information about how to get started with AppID on IBM cloud. We do want to make this part optional (#15) but it would still be good for people that want to use AppID.

@josiemundi I am not sure I understood the line "it would still be good for people that want to use AppID." I tried the steps I get an error on reading of DB_PORT as "ValueError: invalid literal for int() with base 10: 'None'". I have given the port number as 5984. I suspect this is not the right place and move it to Slack channel? Are you saying we really do not need this AppID section?

@shobhavijaystar are you trying to deploy the API. You have your env file but it's not working. The part you are referring to is for couch db not AppID and should look as follows:

Right now you need to have AppID on IBM cloud to get the demo working. Do you have AppID credentials? If not it won't work. We are working this next sprint to remove this dependency.
Hope this helps. Please reach out on Slack if you need any other help getting this up and running :)

reopening - not yet complete.


ASSET make docstring raw python strings to avoid escape characters

Coverage remained the same at 84.695% when pulling c8a918a0f77b90da0c6a165214d6ba22f5694128 on INM-6:fix/asset_rawstring into 191a9c0c6f8763e04f38a17149e43f8c0cadaf68 on NeuralEnsemble:master.


[Explorer] Data selector enhancement and refactoring adoptions

Description
Data selector in core has a set of enhancements and refactoring which require the same counterpart in explorer to also change accordingly.
https://github.com/opensearch-project/OpenSearch-Dashboards/pull/6571]
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

It looks like all of the integration test actions are failing, including the ones that weren't failing before -- Can you investigate?


adding confirm password in sign up page

Is your feature request related to a problem? Please describe.
Hi, I would like to add Confirm Password field in sign up page where the user has to enter the same password as the Password field. If the Confirm Password is not equal to Password, it would show error in real time. This would prevent typos.
Record

[X]  I have read the Contributing Guidelines
[X]  I'm a GSSOC'24 contributor
[X] I want to work on this issue

@sk66641 i already solved this issue kindly accept my merge request

@me-nilakshi
Could you please give an update over this issue as it has been pending for over two weeks?

I'm working on it, will finish it by today


v1.12.0: Failed to execute 'fetch' on 'Window': Illegal invocation

We upgraded our web-based, Vite/React SPA to the new Kiota v1.12.0 and upgraded our generated API client. The dependencies in our project were updated to include (as dictated by kiota info ...):
@microsoft/kiota-abstractions: 1.0.0-preview.44
@microsoft/kiota-http-fetchlibrary: 1.0.0-preview.43
@microsoft/kiota-serialization-json: 1.0.0-preview.44

Running the updated project, API requests fail with the following message and stack:
TypeError: Failed to execute 'fetch' on 'Window': Illegal invocation
    at CustomFetchHandler.<anonymous> (http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:598:29)
    at Generator.next (<anonymous>)
    at http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:185:67
    at new Promise (<anonymous>)
    at Object.__awaiter (http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:167:10)
    at CustomFetchHandler.execute (http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:597:24)
    at HeadersInspectionHandler.<anonymous> (http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:1600:44)
    at Generator.next (<anonymous>)
    at http://localhost:3000/node_modules/.vite/deps/@microsoft_kiota-http-fetchlibrary.js?v=8cce6e32:185:67
    at new Promise (<anonymous>)

Our client initialization code has been:
const myApiClient = createMyApiClient(new FetchRequestAdapter(new AnonymousAuthenticationProvider()))

Modifying the client initialization to use an explicitly bound window.fetch function "fixes" the error and gets us back to a working integration:
const myApiClient = createMyApiClient(new FetchRequestAdapter(new AnonymousAuthenticationProvider(), undefined, undefined, KiotaClientFactory.create(window.fetch.bind(window))))

Are we initializing our API client correctly? Or is this an unintentional issue with Kiota v1.12.0?
(Edit: And I should note- downgrading to the previous version of Kiota also works.)

Hi @seniorquico
Thanks for using kiota and for reaching out.
Yes the initialization is correct as far as I can tell.
It's strange that your tests can't find fetch which is supposed to be defined both on global (that's the one being referenced) and on window.
We had removed an older version of node-fetch as a dependency, for sanity we cleaned it up with #1092 but for some reason we forgot to publish that change. I've just released new versions with #1100.
Can you upgrade and see whether you still need the workaround please?

@baywet I upgraded to the following dependency versions and reverted the initialization code, but I still get the the "illegal invocation" error. The workaround still works with these new versions, as well.
@microsoft/kiota-abstractions: 1.0.0-preview.45
@microsoft/kiota-http-fetchlibrary: 1.0.0-preview.44
@microsoft/kiota-serialization-json: 1.0.0-preview.45

Very odd!

@baywet And just to confirm- reverting to Kiota v1.12.0-preview.202402220002, the following dependency versions, and the original initialization code (no explicit window.fetch binding to window) works as expected- no "illegal invocation" error.
@microsoft/kiota-abstractions: 1.0.0-preview.43
@microsoft/kiota-http-fetchlibrary: 1.0.0-preview.42
@microsoft/kiota-serialization-json: 1.0.0-preview.43

This was what we had been testing with prior to upgrading to Kiota v1.12.0.

Thanks for the additional information.
And you're seeing that behavior only in vitest, not when running the code in browser and getting the data from the actual API?
when looking at the changes between both versions the only things that jump to mind are default values for the middlewareFactory and httpClient, I suspect that somehow an undefined is being kept instead if using parameters default values.
Maybe the httpClient constructor should get defaulted with fetch as any like other places do.
Regardless, I think a regression was introduced, if a custom is in fact passed to the constructor parameter, we'll end up with two CustomFetchHandlers and I think the body of the constructor should be mostly reverted to what it was besides the renames.
And as for the default values, maybe fetch ?? (window && window.fetch) as any instead (or some variation of that) would be better adapter (we should also include global.fetch for workers then).
Do you think you could explore those avenues by cloning the repository locally and testing those changes?

@baywet No, this is happening in both development & production builds of the app in the web browser. I actually haven't tested it yet with vitest.
Happy to try a local, modified build. This may take a few days, but I'll get back as soon as I can.

I'm experiencing this error as well using the latest version 1.12.0 and latest preview of the different npm's

Thanks everyone for the additional input. It helps a lot.
I'm currently traveling for a conference, so I won't have time to look into that before next week at the very best.
If one (or more) of you could get the investigation started locally and start putting together a PR, it'd help fast-track things.

Adding this to the requestAdaptor KiotaClientFactory.create(window.fetch.bind(window)) also solved it for me as a temp solution

I just tested with Kiota 1.13.0-preview.202403280001 and manually updated to the following:
@microsoft/kiota-abstractions: 1.0.0-preview.50
@microsoft/kiota-http-fetchlibrary: 1.0.0-preview.49
@microsoft/kiota-serialization-json: 1.0.0-preview.50

I can confirm it's fixed and working without the explicit bind. Thanks, all!


inform about ALPN/NPN support on local/remote

Hello,
I encountered the following issue when connect to Apple push notification service with hyper. Current APNS endpoint uses HTTP/2. Connects from python 2.7.10 works good, but when I try to connect from python 3.4 I just get ConnectionReset error every time. After spending some hours for investigation I ended up with the following. APNS does not support NPN (which is considered obsolete as I understand) and only support ALPN. And ALPN support in python ssl module is only started from python 3.5.  In hyper.tls we have the following code
    # ALPN is newer, so we prefer it over NPN. The odds of us getting different
    # answers is pretty low, but let's be sure.
    with ignore_missing():
        proto = ssl_sock.selected_alpn_protocol()

    with ignore_missing():
        if proto is None:
            proto = ssl_sock.selected_npn_protocol()

What we see here is that absense of ALPN support is silently ignored. And since NPN is not supported at endpoint hyper follows http1.1 connection scheme though current endpoint is HTTP2.
I think that it is better to allow developers using hyper to specify wether they want to ignore_missing or not and raise exceptions when this happens. At least they will not spent a day figuring out why it works in one python version and not in other.
What do you think?
(looks like for my case I will need to use local copy of the hyper with hardcoded 'h2' proto by default, and I'm not happy about it)

@mylh Yeah, this is admittedly a concern. The debuggability of it is a real issue, but it's mildly problematic to resolve in a way that doesn't have some really unfortunate usability problems:

Either we add a configuration flag for this, which is a slightly tricky thing to do in a way that is clear.
We can assert in situations where neither NPN or ALPN are available (not really ideal, and wouldn't have fixed this problem as NPN is available)

I'm aware this is an issue though, and have been considering for a little while whether we should extend ignore_missing to log at warning level in situations when one of the two protocols is not present.
As to what to do in your specific case: where are you getting your Python from? It would obviously be ideal if you could use either Python 2.7.9+ or 3.5+, either of which does have support for ALPN in the standard library. Unfortunately, if you're on Python 3.4 you're in a tricky middle stage where currently hyper uses the standard library because NPN is available, rather than falling back and requiring PyOpenSSL.
We also have a patch that is currently languishing (#197) that basically just lets you override what hyper thinks is going on. That patch is probably suitable for your use case, so you may want to run a local copy of hyper with that patch applied until such time as I'm able to ship an update to hyper that includes it.
How does that seem to you?

Well I should search for APNS in the repo :) I think I would go with the patch. The problem arises with Ubuntu 14.04 where python is in 2.7.6 and 3.3. and this is the default Ubuntu LTS release now :(
I will think about a way to implement this in hyper considering your input. If I have ideas will definitely share

Arg, yeah, the Ubuntu LTS releases are a problem. You'd find that even with newer Pythons your OpenSSL is too old, and doesn't support ALPN anyway (ALPN was added in OpenSSL 1.0.2). So you really have no option but to just circumvent it.

I was getting the exact same problem on Python 3.4 on Heroku. Luckily I can simply bump up the runtime. So now when testing on 3.5.1 I'm still getting:
Traceback (most recent call last):
  File "<console>", line 1, in <module>
  File "/app/.heroku/python/lib/python3.5/site-packages/requests/sessions.py", line 511, in post
    return self.request('POST', url, data=data, json=json, **kwargs)
  File "/app/.heroku/python/lib/python3.5/site-packages/requests/sessions.py", line 468, in request
    resp = self.send(prep, **send_kwargs)
  File "/app/.heroku/python/lib/python3.5/site-packages/requests/sessions.py", line 576, in send
    r = adapter.send(request, **kwargs)
  File "/app/.heroku/python/lib/python3.5/site-packages/hyper/contrib.py", line 68, in send
    resp = conn.get_response()
  File "/app/.heroku/python/lib/python3.5/site-packages/hyper/common/connection.py", line 124, in get_response
    return self._conn.get_response(*args, **kwargs)
  File "/app/.heroku/python/lib/python3.5/site-packages/hyper/http11/connection.py", line 198, in get_response
    self._sock.fill()
  File "/app/.heroku/python/lib/python3.5/site-packages/hyper/common/bufsocket.py", line 172, in fill
    raise ConnectionResetError()
ConnectionResetError

Any configuration I need to change?

In this situation the connection is getting forcibly closed. The problem here seems to be that hyper is sending a HTTP/1.1 request, presumably because either you don't have ALPN available (and so no protocol was negotiated) or because you used a non-https URL.
The version of hyper in the master branch allows you to tell hyper to ignore the ALPN negotiation and to assume a given result, but by default you won't be able to use that with the requests adapter provided in contrib. I'd welcome a patch to fix that, at which point you should be able to use the master branch until such time as I draft a release.

@Lukasa thanks for the quick response! Any idea why ALPN isn't available here despite using python 3.5.1?

So ALPN is only available on newer OpenSSLs, as you've discovered. My bet is that Heroku by default uses Ubuntu 14.04 for its base OS, which contains OpenSSL < 1.0.2: sadly, ALPN was introduced in OpenSSL 1.0.2.
I'm not sure how troublesome it will be to provide yourself with a new OpenSSL on Heroku, but the answer is probably "somewhat troublesome".

@Lukasa that's what i'm trying to figure our right now. I've built custom buildpacks before, but need to refresh my memory. I'll update here for posterity once I figure it out :) thanks!

For anyone else coming here looking how to run Python with ALPN support (OpenSSL 1.0.2) on Heroku, see instructions here: Compiling Python with custom OpenSSL on Heroku. Hope this helps :)

Thanks for that note @matangover! :sparkles:

@Lukasa
Hey bro. I am using 0.7 hyper for APNS services.How can i ignore the ConnectionResetError error to continue using hyper?
Thanks a lot.
Why i am doing this , it's because I got a ** ValueError: Could not unserialize key data.**.I've been searching for the entire Internet that how can i update the openssl version in the Ubuntu.When i done that , i can't let python use the new version openssl (Although i change the setup.dist in the python building directory , don't know which part did wrong). So i didn't use the latest version of the openssl.And come up with this problem.
Thanks for listening my issue here..

You can't ignore a ConnectionResetError: that occurs because the connection was forcefully closed by the remote peer. How are you hitting the problem?

@Lukasa
I am using a Python2.7 which output ssl version OpenSSL 1.0.1f 6 Jan 2014
When i using the .p8 file provided by Apple , send the request through hyper , the cryptography error raise up. (https://github.com/pyca/cryptography/issues/3232).
So i break the .p8 lines then it works.
but now i am facing the problem i describe.

I think the problm is the Apple server doesn't recognize the .p8 file which i break the line...

No, the problem is yours: OpenSSL 1.0.1f does not support ALPN, so the negotiation fails. You'll need to install OpenSSL 1.0.2 or later.

@Lukasa  Solving the problem by upgrade ubuntu to the new version and the openssl automatically update to 1.0.2.
Thanks for helping!


Update README.md

Explain how to create a config.
Improve readability.

Closes #50.

hmm generally it is unnecessary to download the sample since matrix-registration is shipped with the config file and even has a setup process that sets the most important options if you just execute:
matrix-registration

when no config file is found.
that said on some weird setups it doesn't find this file in which case I guess you would need to download it yourself

@ZerataX Thanks for the feedback. I've updated my patch to include your comments. Hope it fits better.

@ZerataX Thanks for the feedback. I've updated my patch to include your comments. Hope it fits better.

Thank you!

Thank you!


Rimuru Tempest added

Looks like there are some merge conflicts in your Fork. Please sync your Fork and then open the Pull Request. Thank you.


Color picker displaying off page and resizing page

Hi,
I've met another issue ;) The color picker is extending the page so the white background appears. We should detect if there is enough space to display balloon below or maybe we should display it above ;)
Sandbox: https://codesandbox.io/s/suneditor-baloon-position-4sn8m

Best wishes,
Maciej

The 2.28.1 version has been updated.
Only the left position of the menu has been modified.
The issue displaying the menu at the above button will be modify in the next version.
Thank you.

Hi
The 2.28.2 version has been updated.
Thank you.


CustomNodes strategy refactor

Task 2 from https://github.com/gnosis/gp-v2-services/issues/1407
Important changes:

CustomNodes strategy doesn't use crate transaction_retry nor rpc::submit file anymore. It relies completely on the new Submitter struct and its algorithm. (gas_price_stream.rs, retry.rs, rpc.rs are deleted)
Trait 'TransactionSubmitting' now returns custom error type (SubmitApiError) so we can better handle different situations in Submitter
Submitter doesn't get web3 as input but rather reuses web3 from contract object (not completely sure if this is safe in all use cases?)

Todos:

[ ] Implement cancellation for CustomNodes strategy (skipping for now the cancellation for all strategies)
[ ] Add unit tests for Submitter (no need for unit tests)
[x] Cleanup of unused old code

Codecov Report

Merging #1476 (722592f) into main (ac53588) will increase coverage by 0.50%.
The diff coverage is 8.44%.

@@            Coverage Diff             @@
##             main    #1476      +/-   ##
==========================================
+ Coverage   66.07%   66.58%   +0.50%
==========================================
  Files         162      162
  Lines       31038    31519     +481
==========================================
+ Hits        20508    20986     +478
- Misses      10530    10533       +3

Looks good to me (I like that the overall LOC count was drastically reduced)!
I'm still not entirely sure how this abstraction allows us to mix and match between strategies. How would a strategy be configured that uses Eden and Flashbots in parallel?

We have two options, either to have Vec<Submitter> or Vec<&'a dyn TransactionSubmitting> inside Submitter. Will be considered in the next PR.

We have two options, either to have Vec or Vec<&'a dyn TransactionSubmitting> inside Submitter

This will work for the parallel strategy, but how about a strategy where we only want to send certain txs to the public nodes (e.g. the ones not having any or only relatively fail safe interactions)?

I was thinking of having these structures dynamically filtered for each settlement and possibly for every iteration in run loop (for example if we want to remove eden in case the gas_price > X gwei)


fix:Illegal creation of two SpiderSubnets with the same 'spec.subnet'

Signed-off-by: iiiceoo<EMAIL_ADDRESS>

CodeFactor — 18 issues found.


Error while running example script

Having issues while testing the example script (side note the slack link is expired)
Versions:
Python 3.8.10
PyTorch  1.8.1+cu102
NumPy 1.21.4
python3 attack_cifar10_gradinversion.py --batch_size 16 --tv 0.05 --bn_reg 0.001 --reconstruct_labels
{'reconstruct_labels': True, 'signed_image': False, 'mini': False, 'large': False, 'BN_exact': False, 'attacker_eval_mode': False, 'defender_eval_mode': False, 'total_variation': 0.05, 'epoch': 0, 'bn_reg': 0.001, 'attack_lr': 0.1}
Global seed set to 1234
[ToTensor(), Normalize(mean=(0.4914, 0.4822, 0.4465), std=(0.2023, 0.1994, 0.201))]
Files already downloaded and verified
Files already downloaded and verified
Loaded data!
Traceback (most recent call last):
File "attack_cifar10_gradinversion.py", line 199, in
pipeline, attack_hparams = setup_attack()
File "attack_cifar10_gradinversion.py", line 77, in setup_attack
model = create_lightning_module(
File "/home/ubuntu/.local/lib/python3.8/site-packages/gradattack/models/init.py", line 493, in create_lightning_module
assert os.path.exists(ckpt), f"Failed to load checkpoint {ckpt}"
AssertionError: Failed to load checkpoint checkpoint/vanilla_epoch=1-step=1531.ckpt

Hi,
Please replace the path of the checkpoint (checkpoint/vanilla_epoch=1-step=1531.ckpt) with the checkpoint of your own model :)
Please let me know if you have further questions. Thanks!


[SMALL, Reveng] Add ability to deal with temporal tables for SQL Server 2016

Fixes #4697.

@natemcmaster  can you take a look at this one

:shipit:

Checked in with commit e96af05.


How can I get the datas of every frame?

Hello.
You have to modify UsbCamera.cs a little.
Try these changes below and let me know it works fine or not.
Step1. Add this block around line 80.
public Action<Bitmap> BitmapCaptured
{
    get { return CaptureGrabberCallback.Buffered; }
    set { CaptureGrabberCallback.Buffered = value; }
}
private SampleGrabberCallback CaptureGrabberCallback;

Step2. Replace this block from
private Func<Bitmap> GetBitmapFromSampleGrabberCallback(DirectShow.ISampleGrabber i_grabber, int width, int height, int stride)
{
    var sampler = new SampleGrabberCallback(width, height, stride);
    i_grabber.SetCallback(sampler, 1); // WhichMethodToCallback = BufferCB
    return () => sampler.GetBitmap();
}

to
private Func<Bitmap> GetBitmapFromSampleGrabberCallback(DirectShow.ISampleGrabber i_grabber, int width, int height, int stride)
{
    CaptureGrabberCallback = new SampleGrabberCallback(width, height, stride);
    i_grabber.SetCallback(CaptureGrabberCallback, 1); // WhichMethodToCallback = BufferCB
    return () => CaptureGrabberCallback.GetBitmap();
}

Step3. subscribe BitmapCaptured.
camera.BitmapCaptured += (bmp) =>
{
    // called here every frame captured.
};

good,thanks!it works very well.

it works,but it isn't thread-safe.

Could you show me the code that produce the problem?

like this:
 camera.BitmapCaptured += (bmp) =>
  {
      this.textBox1.Text = "hello";
      this.BackgroundImage = bmp;
  };

You're better off implementing it with events

Are you saying that the line below cause InvalidOperationException? (Cross-thread operation not valid.)
this.textBox1.Text = "hello";

If so, it is your responsibility to call cross-thread control in a thread-safe way.
textBox1.Invoke((Action)(() =>
{
    this.textBox1.Text = "hello";
}));

I did it.And it can work.But I have many controls to process.So I will do more work.It's not just convenient.

I use delegates to handle those controls, so I do a lot of things. Using Invoke is much easier.thanks!


Add User test implementation

Add User test layer

Looks great! Check all the empty lines in the test methods to make them more consistent


Fix align issue on providers list

Description
Fix align issue on providers list
Tested
On Android
How others should test
Go to cash in
Related issues

Fixes https://github.com/celo-org/celo-monorepo/issues/7843

Closed in favor of #384


fix: Cannot mock @backstage/core-components dependency

fix: Cannot mock @backstage/core-components dependency
#20713

Hello @Rugvip can you please review this and correct me if more changes needs to be done.

@MeenuyD Check out the errors first and let's try to get it green.

Hey @MeenuyD :wave:
Thanks for this contribution, but I ended up fixing this in a separate PR to get it shipped as a patch release quickly.
Thanks for taking the time to have a stab at this, hopefully there's some other good first issue labels that you'd like to pick up? :pray:
Going to close this as a duplicate.


Multiple layout bugs in Mid 2013 YouTube layout

Description:
I discovered, that Mid 2013 YouTube layout has multiple layout bugs.
UPDATE 28.11.2023: Tested on Firefox only!
Status of bugs
Home Screen:

[ ] Empty button under "What to Watch" button
[ ] Hovering over a video thumbnail on the home screen also shows a paid promo tooltip
[ ] The live tooltip is unnecessarily stretched out due to a missing icon
[ ] The premiere tooltip is unnecessarily stretched out due to a missing icon
[ ] The "Show more" in the library section is misaligned to the separator
[ ] The X button on the search bar is larger than before

Search screen:

[ ] The "About these results" button isn't following old YouTube styles, and is misaligned to the separator
[ ] The "Filters" button isn't following old YouTube styles and is misaligned to the separator

Watch page:

[ ] Big gap between the About & Save button
[ ] Expanded live chat isn't following the old YouTube styles
[ ] Links are blinking when hovering at the description
[x] Share button is showing the wrong text (Won't fix)
[ ] The Autoplay button isn't fitting to old player style
[ ] The "Enable Return YouTube Dislike compatibility" option isn't working correctly
[ ] The "Join" button isn't following old YouTube styles
[ ] The timeline slider on the player toolbar isn't stretched out correctly on screen with a 2560x1440 resolution

If one gets fixed, then I (or someone else) will mark it as completed.
Screenshots:
Home screen:
Empty button under "What to Watch" button

Hovering over a video thumbnail on the home screen also shows a paid promo tooltip

The live tooltip is unnecessarily stretched out due to a missing icon

The premiere tooltip is unnecessarily stretched out due to a missing icon

The "Show more" in the library section is misaligned to the separator

The X button on the search bar is larger than before

Search screen:
The "About these results" button isn't following old YouTube styles, and is misaligned to the separator

The "Filters" button isn't following old YouTube styles and is misaligned to the separator

Watch page:
Big gap between the About & Save button

Expanded live chat isn't following the old YouTube styles

Links are blinking when hovering at the description
https://github.com/lightbeam24/CustomTube/assets/69351811/edbb045b-b3b4-494c-b2f2-f6dc401f7f85
The Autoplay button isn't fitting to old player style

The "Enable Return YouTube Dislike compatibility" option isn't working correctly

The "Join" button isn't following old YouTube styles

The timeline slider on the player toolbar isn't stretched out correctly on screen with a 2560x1440 resolution

Won't fix:

Share button is showing the wrong text

Sorry, my bad. That is probably intended to be a counter of shares

hey you forgot about this one

First of all: Layout inaccuracies != bugs.

Empty button under "What to Watch" button

Cannot reproduce.

Hovering over a video thumbnail on the home screen also shows a paid promo tooltip

Do not use CustomTube with Inline Playback enabled. You can turn it off here: https://www.youtube.com/account_playback

The live tooltip is unnecessarily stretched out due to a missing icon
The premiere tooltip is unnecessarily stretched out due to a missing icon

These are the same issue, but are very low priority.

The X button on the search bar is larger than before

The X button didn't even exist in 2013, so in order to be the most accurate, it would need to be removed altogether, which I think would be a bad change.

The "About these results" button isn't following old YouTube styles, and is misaligned to the separator
The "Filters" button isn't following old YouTube styles and is misaligned to the separator
Big gap between the About & Save button
Expanded live chat isn't following the old YouTube styles

These are not bugs, and should not be reported here. Trust me, I am well aware of every tiny inaccuracy in this extension, it just takes time to fix them.

The Autoplay button isn't fitting to old player style

Duplicate. Also not a bug.

Links are blinking when hovering at the description

This is literally the intended result. Actually, the links are only supposed to be blue when hovering over the description. The fact that they're blue when you're NOT hovering over the description is the issue, but is an EXTREMELY low priority one.

The "Enable Return YouTube Dislike compatibility" option isn't working correctly

Return YouTube Dislike itself does not work with an experiment that your account was flagged with.

Sorry, my bad. That is probably intended to be a counter of shares

No, that is the dislike count. That is an issue with Return YouTube Dislike itself, due to the aforementioned experiment.

The "Join" button isn't following old YouTube styles

There is an option to hide the Join button. (Maybe I should make it enabled by default)

The timeline slider on the player toolbar isn't stretched out correctly on screen with a 2560x1440 resolution

It is impossible to make the progressbar full width while in fullscreen to my knowledge. It doesn't have anything to do with your resolution.

Sorry, but with only a couple of exceptions, these are all feature requests, not bugs. I'm not taking feature requests anymore, which I explained here: https://github.com/lightbeam24/CustomTube/discussions/121
As a result, I'm closing this issue.

hey you forgot about this one

You have an experiment that a few others have gotten. I don't know what experiment it is, so I can't reproduce nor fix that bug as of now.

hey you forgot about this one

You have an experiment that a few others have gotten. I don't know what experiment it is, so I can't reproduce nor fix that bug as of now.

you don't understand. this is a monitor display problem. i use chrome. my display is  1024 x 768. also did you reply my gmail suggestions?

hey you forgot about this one

You have an experiment that a few others have gotten. I don't know what experiment it is, so I can't reproduce nor fix that bug as of now.

you don't understand. this is a monitor display problem. i use chrome. my display is 1024 x 768. also did you reply my gmail suggestions?

hold on. when i switched to my alt channel, it looks like it has been fixed. but when i go back to my main channel, it's not fixed! what the hell is going on?

also did you reply my gmail suggestions?

I don't check that email often. I only created it because the Chrome Web Store requires that you have a contact email.

hold on. when i switched to my alt channel, it looks like it has been fixed. but when i go back to my main channel, it's not fixed! what the hell is going on?

I already told you, one of your accounts has an experiment that's doing that.


Update time component

Reviewer: @Poltergeist

This is a cool component :) Nice work @mlunoe!

Looks pretty good! Just some small comments and I think it's good!


Activation visual feedback through TimeBounceInterpolator

Added new class: TimeBounceInterpolator that interpolates forth and back
Modified TimeFloatInterpolator to support adding callback functions, for TimeBounceInterpolator to function.
Activating an entry with primary/secondary input should now provide a "bounce" visual feedback.
Add some commments for other classes

merged


FAQ page is missing some URLs

The list of file hosting services on the FAQ page is missing some URLs (such as light-theme-hurts.my-ey.es)

Will be fixed in #14. You can find a text file in the root of the repository too with a parseable list of domains in this PR.
Leaving this open until merge.

For the lib devs, what's a good way and/or interval to update this? I'd normally say on init, but for long running processes this won't work as well if it gets updated during runtime of the process.

Probably could do it with a cron-job on a build server that will automatically fetch and apply every 30min or so if there are changes or something.
Libraries should leave the URL decisions up to the developer, though. I don't see any need to force developers to pick from a list if the list is constantly changing. The file is mostly meant for people who quickly need to see the list and application developers.

#14 is closed, closing this issue.