C cpp

Description 📜
I have added documentation for function overloading in C++
Fixes # <2173>

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[x] C/CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

@deepali2002-max  Add a line in readme of directory

@deepali2002-max
unfortunately this is a duplicate issue for #1508
Kindly search for an issue next time & add a proper title
I can't merge this... because that one was made before this

Okay thanks


Eliminate -Wimplicit-fallthrough triggers

Replace fallthrough comments with ABSL_FALLTHROUGH_INTENDED, allowing clean building with -Wimplicit-fallthrough.

Thanks for the PR!


User/shaynie/fix11615

Overview
Update the render order to reflect the latest changes in graphics tools.  This makes the cursor visuals very steady in editor and on device.  See https://github.com/microsoft/MixedReality-GraphicsTools-Unity/pull/182 for details.
Changes

Fixes: #11615  .

Verification

This optional section is a place where you can detail the specific type of verification
you want from reviewers. For example, if you want reviewers to checkout the PR locally
and validate the functionality of specific scenarios, provide instructions
on the specific scenarios and what you want verified.
If there are specific areas of concern or question feel free to highlight them here so
that reviewers can watch out for those issues.
As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

Thank you!! 🎉🎉🎉


zfs error:  cannot receive: local origin for clone does not exist

I cannot use zfs send/receive to copy the LXD ZFS dataset in certain cases after copying and deleting containers.
This must be a ZFS bug, but I can only reproduce it with LXD.  Could you please find out the equivalent zfs commands that reproduce this problem, so we can pass it on to ZFS.
Version information
Distribution:  Debian 10.5, with backports, snapd, and and ZFS installed, as per:
https://openzfs.github.io/openzfs-docs/Getting Started/Debian/index.html
I also verified it with Ubuntu 20.04 and Ubuntu 18.04.
uname -vrmo
4.19.0-10-amd64 #1 SMP Debian 4.19.132-1 (2020-07-24) x86_64 GNU/Linux
zfs version
zfs-0.8.4-2~bpo10+1
zfs-kmod-0.8.4-2~bpo10+1
snap list
Name    Version      Rev    Tracking       Publisher   Notes
core    16-<IP_ADDRESS>  9804   latest/stable  canonical✓  core
core18  20200724     1885   latest/stable  canonical✓  base
lxd     4.4          16926  latest/stable  canonical✓  -
Steps to reproduce:
Assume LXD is installed with ZFS dataset z/lxd:
lxc launch images:alpine/3.12 a1
lxc snapshot a1 packages
lxc copy a1/packages a2
lxc stop a1
lxc delete a1
zfs snapshot -r z/lxd@copy
on another system (I tried Debian and Ubuntu 18.04), with ssh access to "pin", and an empty ZFS dataset z/test, run the following:
ssh pin sudo zfs send -R z/lxd@copy | sudo zfs receive -F z/test/lxd
Result:  I get an error:
cannot receive: local origin for clone z/test/lxd/containers/a2@copy does not exist

A similar issue was reported here:  https://github.com/openzfs/zfs/issues/10135

I edited the description and removed the second "lxc snapshot", which is not needed.

Can you show zfs list -t all -o name,origin,clones? Maybe this will show some oddity?
I'm closing the issue as this really feels like a ZFS bug. I subscribed myself to the openzfs issue so will get notified if upstream asks for something that's related to LXD.


Request for documentation on how node-redis handles connections on non-graceful disconnects

Issue
I was unable to find documentation on the importance of the client needing to call client.quit/client.end before disconnecting, or any such warning thereof (which leads me to assume that the connection automatically disconnects after some time). If you forget to call client.quit or your server unexpectedly crashes, will the redis client automatically disconnect after some time? or will you have leaked connections and you'll finally hit the rate limit? Either way I think a sentence or two on this in the docs will largely clarify the matter.
Thank you.

client.quit() is the way to go for closing a connection to gracefully exit. Perhaps we should highlight this in the documentation.
However, Redis shouldn't timeout/disconnect when everything is good with the server (it's a persistent connection on purpose). You shouldn't also be creating adhoc connections - it's a major antipattern to create a connection per operation in Redis.

We'd be open to a a PR on fixing the documentation about graceful exiting.


No block source available - stuck 6 hours behing

I have all required peers with latest build on testnet. (win10x64)
I already tried -reindex

Did you try to delete peers.dat before relaunching?

it did not help. :(

Please send your debug.log to<EMAIL_ADDRESS>

done

Please update to
https://dashpay.atlassian.net/builds/browse/DASHW-DEV-498/artifact/JOB1/gitian-win-dash-dist/
and add these nodes to your dash.conf
addnode=[2001:1608:10:25:10::101e:7241]:19999
addnode=[2001:1608:10:25:10::10d2:f4d1]:19999
addnode=[2001:1608:45:3::242:ac11:2]:19999
addnode=[2001:1608:45:3::242:ac11:3]:19999
addnode=[2001:1608:45:3::242:ac11:4]:19999
addnode=[2001:1608:45:3::242:ac11:5]:19999
addnode=[2001:1608:45:3::242:ac11:6]:19999
addnode=[2001:1608:45:3::242:ac11:7]:19999
addnode=[2001:19f0:9000:2042::2]:19999
addnode=[2001:19f0:9000:2049::2]:19999

I had similar problem, i was forked, deleted all folders and files and restarted and let it sync, all good.

How many blocks are necessary to keep wallet stuck with wrong chain?
This seems to be an issue with consensus protocol. Probably in bitcoin this issue persists as well.

This is not a issue with the consensus protocol, it only applies to Dash testnet where only 20 CPU cores are mining and you can fork the network by running a single rough GPU miner :)
Is your sync problem solved?

Yes, by deleting the blockchain, I already had last build.


Integration with Jax-XC

Jax-XC  allows to convert LibXC functionals to JAX. It would be cool to allow integration with our library, I think.

I contacted the developers to see the best way to expose the energy densities. Let's wait and see...


null safe library

The library 'package:flutter_profile_picture/flutter_profile_picture.dart' is legacy, and should not be imported into a null safe library.  Try migrating the imported library.

The library 'package:flutter_profile_picture/flutter_profile_picture.dart' is legacy, and should not be imported into a null safe library. Try migrating the imported library.

Haven't tried yet it but this seems like the only way for now to use non-null safe packages in null-safe apps or packages
https://dart.dev/null-safety/unsound-null-safety

Heya I bumped into this issue today - so I have raised a PR just now to try and migrate it over.


License for repo?

I noticed you have no LICENSE file in your repo. Is your intent to make this available as an OSS project? If so, I recommend at a minimum adding an MIT license file like so:

Copyright 2023 [Your full name]
Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation files (the “Software”), to deal in the Software without restriction, including without limitation the rights to use, copy, modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software is furnished to do so, subject to the following conditions:
The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.
 IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Hi Mike
Finally got round to this.
Thx
Alastair
On 2023-03-08 21:19, Mike Deats wrote:

I noticed you have no LICENSE file in your repo. Is your intent to make
this available as an OSS project? If so, I recommend at a minimum
adding an MIT license file like so:
Copyright 2023
Permission is hereby granted, free of charge, to any person obtaining a
copy of this software and associated documentation files (the
"Software"), to deal in the Software without restriction, including
without limitation the rights to use, copy, modify, merge, publish,
distribute, sublicense, and/or sell copies of the Software, and to
permit persons to whom the Software is furnished to do so, subject to
the following conditions:
The above copyright notice and this permission notice shall be included
in all copies or substantial portions of the Software.

IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY
CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT,
TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION WITH THE
SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.
--
Reply to this email directly, view it on GitHub [1], or unsubscribe
[2].
You are receiving this because you are subscribed to this
thread.Message ID: @.***>

Links:
[1] https://github.com/spicysomtam/k3s-aws-cluster/issues/5
[2]
https://github.com/notifications/unsubscribe-auth/AAIWRRGRMOX43DMZWNVSAMLW3DZXXANCNFSM6AAAAAAVUIU4ZQ


[WIP] ProcessFlagMount without ProcessFlagV

What's the current status of this?

Sorry @AkihiroSuda currentky, I am not well. Will get back once I get well

@manugupt1 Do you plan to continue to work on this?

@AkihiroSuda sorry! I do not have any bandwidth right now. I will close this


BOTS: Add Withdraw using Partial Self-Liquidation to harvest profits from Leverage Bots

As a user, I want to be able to withdraw my profits from my leverage position so that I am able to collect profits without closing my curve leverage position.

Allow user to specify how much they want to liquidate
Convert the amount to liquidate into the required percentage
Provide UI/UX for Withdraw on the Mybots page for Leverage Bots

Why not just borrow_more? that's equivalent to taking profit without closing it

Why not just borrow_more? that's equivalent to taking profit without closing it

So we can borrow_more on this leverage contract so that we can take profits on the loan. Follow-up questions:

With this method, we have more interest rate cost on the loan. Correct?
With this method, user can just withdraw whatever token they wish, the swap happens on the back end. Yes?

Closing as confirmed that health band management is the next phase, but we will focus on Renzo bot for now.

Reopening, as this is being deployed now

@fieldtheory123 we need the epic for this now ser.

Closing as we focus on llamalend.


Come up with a servicing story for a given SDK version

Need to define a servicing story for a given SDK version (branch structure, code flow, versioning etc).

Well, there are handful of givens that I think we might as well enumerate.
Assuming that #517 resolves to have some sort of semantic versioning:

The latest major version will be in support unless the whole product falls out of support.
Individual patches are never altered, the support story is to adopt the most recent patch in the support band.
We may allow ourselves to have multiple support bands. I.e. Just because there is a major version 7 doesn't mean we won't publish more minor version releases for version 6. However, the back support may be limited to supporting more services or making security patches.
Our version numbers will not be directly tied to the Azure Service version numbers. As much as possible, if Azure is still supporting a means of calling them, we would like to support it. (Despite the fact that this is not how things work today)

Assuming that we decide not to use semantic versioning:

At that point, all of our bits would be identified by Git branches and commit hashes alone.
Each support band would be captured by a branch. (Though, at the moment nothing comes to mind in terms of support bands outside of master)
Inside of a support band, the support model would always be, "take the latest commit or switch support bands."
We would still like to devise a way of targeting the different Azure versions.

So, hopefully the above narrows the question we have to answer to, "How long will we support different support bands?" For instance, in semantic version world, the question is: "How many major versions back do we support?"

To close this issue, we'll need to modify the README to have a section on our servicing story.


Having Problem while Running it.

I'm trying to run train_ego_pred but I get this error
Traceback (most recent call last):
File "/home/dev007/Desktop/tad-IROS2019/train_ego_pred.py", line 41, in
val_gen = data.DataLoader(val_set, **dataloader_params)
File "/home/dev007/.local/lib/python3.10/site-packages/torch/utils/data/dataloader.py", line 351, in init
sampler = RandomSampler(dataset, generator=generator)  # type: ignore[arg-type]
File "/home/dev007/.local/lib/python3.10/site-packages/torch/utils/data/sampler.py", line 107, in init
raise ValueError("num_samples should be a positive integer "
ValueError: num_samples should be a positive integer value, but got num_samples=0

I just downloaded the A3D dataset
On Wed, May 31, 2023, 08:30 Yu (Brian) Yao @.***> wrote:

Did you download the dataset properly? Why you have num_samples=0?
—
Reply to this email directly, view it on GitHub
https://github.com/MoonBlvd/tad-IROS2019/issues/43#issuecomment-1569443756,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASAUBDUXBO66NT7G5OHUQV3XI23NLANCNFSM6AAAAAAYNVMIWY
.
You are receiving this because you authored the thread.Message ID:
@.***>


Donation Link in README broken

The donation link is broken in the documentation. Wanted to find out if you would consider fixing it so we could donate to show appreciation for your continued work.
Also, would you ever be available for some consulting work?

Hmm, working for me from github readme page. Maybe temporary paypal issue. Regarding consultation. I don't normally do that, but I have few spare weeks, so if you want contact me through twitter @mauron85.

Closing this, link seems to be working fine.


git repo initialization directive

This is a follow up to comments in #5 from 170623.

It would be great to have a directive available in Plaster that could initialize the project as a git repository. This would help users of my Plaster template to get going even faster. More stuff ready more happy users.

Thank you.

I'd like to use libgit2sharp for this so we don't have to rely on Git being in the path.  We will need to redist the libgit2sharp assembly.  @daviwil the license for the project is MIT, will this be a problem?
BTW it looks like this project will be adding support for .NET Core shortly.  I'll wait for them to get their .NET Core support in place.  Hopefully that won't be too long.

@rkeithhill reads like a total great idea :-).


ID pins to pinout header

more pins to io_conf

We need to move the logic out of port.cpp, but this is good for now.


Change appVersion to 0.2.0

Codecov Report

Merging #433 (0cb4b8b) into batching/main (36bf50d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 0cb4b8b differs from pull request most recent head 2ce2c33. Consider uploading reports for the commit 2ce2c33 to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@              Coverage Diff               @@
##           batching/main     #433   +/-   ##
==============================================
  Coverage          46.26%   46.26%
==============================================
  Files                 32       32
  Lines               8398     8398
==============================================
  Hits                3885     3885
  Misses              4513     4513

Flag
Coverage Δ

test
46.26% <ø> (ø)

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

Continue to review full report in Codecov by Sentry.

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


ISSUE: Active menu not working exactly

By default, a menu item is considered active if any of the following holds:
The current path matches the url parameter
The current path is a sub-path of the url parameter
If it has a submenu containing an active menu item

But not active xxx.com/admin/permission is a sub-path of the xxx.com/admin/permission?page=2

Bro i want to know how you achieve to build roles and permissions

Several laravel packages exist who are able to help you with this. The permission package by Spatie is one of them.


feat(form-pristine): add setPristine validation method

[Current coverage][cc-pull] is 100%

Merging [#132][cc-pull] into [master][cc-base-branch] will not change coverage

@@           master   #132   diff @@
====================================
  Files          18     18
  Lines         288    289     +1
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits          288    289     +1
  Misses          0      0
  Partials        0      0

[][cc-pull]

Powered by Codecov. Last updated by [9b05aec...2455e70][cc-compare]
[cc-base-branch]: https://codecov.io/gh/leroy-merlin-br/garden/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/leroy-merlin-br/garden/compare/9b05aec5e5128c8051b932507d3768b93c44dc04...2455e70c5b3e50dd427e27f4870a8098a5a69ebb
[cc-pull]: https://codecov.io/gh/leroy-merlin-br/garden/pull/132?src=pr


Cluster pgen Cleanup

We identified a few aspects of the cluster setup in #2 that could be cleaned up, although they are low priority

[x] Unify the calculation mean molecular masses from helium mass fraction
[ ] Simplify/avoid the passing around of hydro_pkg for the different cluster functions
[ ] k_0 and k_100 have bad orders of magnitude, <-100. This could be partially rectified by dividing these quantities by mean particle mass
[x] Make determining  the radial points where the 1D HSE profile is integrated more robust without the r_sampling parameter

[ ] check source term handling of (updated) prims and cons


add --auto-nonce switch

This will make the program ask for the next nonce to be used via JSONRPC.
We need to provide the nonce to use infura.

Added some tests. Please have a look at it again.

The auto-nonce might also be misinterpreted by someone not using a local private key, he will want to nonce to be automatically filled by the web3 provider but will get an error if he uses --auto-nonce.

I don't know how to get the default address of the unlocked account if there is any (and didn't care enough to look it up). If we know how to do that, we can add the code into the get_nonce function.
Though I do think it's not worth it. We will use a local private key anyway.


fix: avoid call stack recursion with large error (fix: #3060)

Fixes #3060
Full details in original issue. I have tested this with the same code used therein - I did not include that code in here because it relies on too many external libraries to make such a massive error output.

@sheremet-va are you the right person to review this? I see you tagged my original issue. Am keen to close this out as it's preventing me from moving to vitest, which I'd love to do.

@sheremet-va are you the right person to review this? I see you tagged my original issue. Am keen to close this out as it's preventing me from moving to vitest, which I'd love to do.

I don't really understand how this fixes it? Why do we need an if? Why can't we always do what is done for 5000 characters?

You mean just replace the current implementation with my change for <5000? I'm happy to do so if you'd prefer that. I assumed (maybe wrongly) there was a reason why the code c.red() the entire error message instead of just the FAIL part.
It seems to me that logger.error(${c.red(c.bold(errorName))}: ${error.message}) is the better code for all instances instead of an if, but I didn't want to just get rid of the existing implementation if there was a good reason for it. Happy to do so though! Lemme know and I can re-raise a PR with that change.

Ah, now I see 🤦🏻
I don't know then 🤔 Maybe it's possible to fix the recursion itself? Maybe use tail recursion 🤔

I don't think fixing the underlying recursion will be possible without a change to picocolors, not vitest. The real issue here is that RTL also colors its output - so we end up running a color-fied output through c.red() which then fills the stack with a zillion calls. It seems to only break at >7000 characters (the default output size for RTL actually, so with the default you're usually fine) but I chose 5000 as a safe buffer.
My assumption was:
if output.length > 5000, it's likely that the output comes from something (like RTL) that already has done formatting/coloring, and we shouldn't .red() it anyway.
I understand your hesitance for the if/else - I think if you're not good with that approach I'd rather submit to just not c.red() the error itself at all - and only do the FAIL part - it seems like a "nice to have" (red error details) is not worth a stack overflow (ability to actually see full output from RTL, a very common use case).
Thoughts?

@sheremet-va any tweaks you want me to make? I really do want to use vitest in our project but this issue is a blocker. Thanks!

I don't think fixing the underlying recursion will be possible without a change to picocolors, not vitest

We have a forked version in the monorepo in utils/src/colors.ts file. It's used mostly for runtime, not for Node.js logic, but I guess we can set it up here. The only problem is that we need to set it up somewhere, and I'm not sure where.

I don't think fixing the underlying recursion will be possible without a change to picocolors, not vitest

We have a forked version in the monorepo in utils/src/colors.ts file. It's used mostly for runtime, not for Node.js logic, but I guess we can set it up here. The only problem is that we need to set it up somewhere, and I'm not sure where.

Is that something I should wait for or is there an action item for me here on this PR?

How about I raise a PR for just the code path with red() only on FAIL and not on the rest of the message (so no if/else) so that we can unblock this issue, and then if there are future changes to be made, we can raise a new issue for them? Would that work @sheremet-va ?

So to be clear @sheremet-va - there's nothing for me to do here and at some point this issue will be closed by a different PR? If so, I shall wait and then migrate our code base once that is done.

So to be clear @sheremet-va - there's nothing for me to do here and at some point this issue will be closed by a different PR? If so, I shall wait and then migrate our code base once that is done.

Tinyrainbow still has the bug. Feel free to fix it there

So to be clear @sheremet-va - there's nothing for me to do here and at some point this issue will be closed by a different PR? If so, I shall wait and then migrate our code base once that is done.

Tinyrainbow still has the bug. Feel free to fix it there

So to be clear @sheremet-va - there's nothing for me to do here and at some point this issue will be closed by a different PR? If so, I shall wait and then migrate our code base once that is done.

Tinyrainbow still has the bug. Feel free to fix it there

@sheremet-va I've played around with tinyrainbow a little and I don't think it's going to be realistically solvable there either - the fundamental problem is that there is - with large pre-colored inputs - a stack overflow no matter what.
Tail recursion does not solve this issue, it just will kick the can down the road (so an even larger pre-colored input) still will break it) and most js interpreters can't even optimize for tail recursion as far as I can tell.
I don't think this is the solution we want, and I think that with others now facing the same issue outside of just RTL, it would be a good idea to use my original PR: a simple test to say "probably color this, but if the input is very large assume it's already colored and don't bother doing so." This would unblock our adoption and solve the issue for other people as well.
I really want to onboard to vitest, but this is a critical blocker for us. I cannot use your library without it being solved.

@sheremet-va any way to re-run the CI test that's failing? It does not look to be a real failure:
⎯⎯⎯⎯⎯⎯ Unhandled Error ⎯⎯⎯⎯⎯⎯⎯
Error: connect ECONNREFUSED <IP_ADDRESS>:65049
❯ TCPConnectWrap.afterConnect [as oncomplete] node:net:1494:16

@sheremet-va sorry to bother - when should we anticipate a merge/release? Thanks in advance!


Add application to the current project (monorepo)

Context
When we are in a multi-project context at some point, we would like to create a new application inside this monorepo.
The idea will be to provide a generator to create an otter application inside a monorepo.
Proposal
The idea is to extend the already existing angular app generator and add the o3r core on top of it, plus some additional steps (listed below).
The command to run will be:
npx ng g application  which behind will call the angular generator for an application inside a monorepo:

npx ng g @schematics/angular:application
followed by:
npx ng add @o3r/core --project

On top of the modification done today by the ng add @o3r/core this command will:

adapt application to Otter recommendations ( add a package.json to the generated app - this is not added by the angular generator: Hint: prepare a template (here is an example of the template from ng-cli which we can use) with the needed deps for a brand new app generated and get the versions of the packages from the root package.json);
add otter schematics at the new created app level (modify the angular.json by adding the schematics field to the new application workspace)

Requires #633


(RFC) INFRA-22713: testing automated PR code analysis with SonarCloud

Let's close this one for now. There are some limitations on the SonarQube plugin and I won't be able to focus on this for the next few days.


Filename Issue in Tutorial_2_Drug_Property_Pred_Assay_Data.ipynb

Hi Kexin,
There is an issue on loading the HIV data. After I ran the following commands:
X_drugs, y, drugs_index = dataset.load_HIV(path = './data')
print('Drug 1: ' + X_drugs[0])
print('Score 1: ' + str(y[0])

It gave me an error about FileNotFoundError.
The code /DeepPurpose/dataset.py tried to fin the file hiv.csv under data folder. However, after unzipping the hiv.zip file, it results in the HIV.csv file. Therefore, there is a name mismatch issue here (hiv.csv vs HIV.csv).
Afterward, I changed the filename from HIV.csv to hiv.csv, and it stopped getting me the filename error.
Best,
Ken

Hey, good catch, there was a data server change in the last month so there might be some data files get messed up. Would you like to do a PR for it? or I can also change it now. Let me know!
Also, we have released another project TDC, with much more datasets of DTI, ADME,T, HTS and so on. It is also integrated with DeepPurpose framework, check out this tutorial

Hi Kexin, I did a PR. Thanks :)


License.txt doesn't define the licensor / copyright holder, nor the years

A common thing for Apache licensed software is to provide the "short" version in the readme.
This can easily be copied for open source attribution pages.
The source code would potentially suggest this is that text:
Copyright (c) 2021 Apple Inc. and the Swift project authors

Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License.
You may obtain a copy of the License at

    http://www.apache.org/licenses/LICENSE-2.0

Unless required by applicable law or agreed to in writing, software
distributed under the License is distributed on an "AS IS" BASIS,
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
See the License for the specific language governing permissions and
limitations under the License.

It's also possible that it should have a variation to mention the runtime exception, or link to the Swift.org mirrored license

Quick note: The snippet above used to have an incorrect copyright notice. This notices in this repository read Copyright (c) <years> Apple Inc. and the Swift project authors.

Ah, sure. I was extrapolating from swift-log, where I posted a similar issue.
Admittedly that looks like a pure Apache license, and is obviously an older project.


[Bug] Repeat

The repeating icons dont change .. and repeat one is not working playing one song even if i skip

The icons is changing correctly now but if repeat one song is on skipping to a next song the name changes but the previous song is repeated

The icons is changing
repeat(1) is not working correctly
if i enable repeat(1)
skip or go to previous song
song title changes to the next song but the previous song is repeated always
so its repeating the track when repeat(1) was enabled


Scrollbar locked on mobile

Hi,
Is anyone experiencing scrollbar locked on mobile? Not sure what the problem is. No error on the console, have checked all css as well.
The issue is only on mobile browsers. Have tested on Safari iOS 13, Chrome on iOS and Android phones. The result is the same. The scrollbar is locked.
EDITED: I noticed modal on that page with form also is blocked.
Thank you in advance.

I'm also having this issue.
@devs - could this be resolved asap?
Thanks!

I get this error as well.. Is there a fix?

This component appears to hijack all touchmove (and mousemove) events for the whole containing document, calling e.preventDefault(); as its first order of business.  That clobbers anything to do with touch for the rest of the page, which likely isn't what we want to happen.
I fixed it by modifying the move function (that gets bound to that touchmove event on the document), so that it only calls preventDefault if we're actually dragging the slider:
				if (this.flag && this.dragFlag)
				{
					e.preventDefault();

					if (this.stopPropagation)
					{
						e.stopPropagation();
					}
				}

This component appears to hijack all touchmove (and mousemove) events for the whole containing document, calling e.preventDefault(); as its first order of business. That clobbers anything to do with touch for the rest of the page, which likely isn't what we want to happen.
I fixed it by modifying the move function (that gets bound to that touchmove event on the document), so that it only calls preventDefault if we're actually dragging the slider:
				if (this.flag && this.dragFlag)
				{
					e.preventDefault();

					if (this.stopPropagation)
					{
						e.stopPropagation();
					}
				}

Do you have a fork with this fix?

+1 I also need this fix

Do you have a fork with this fix?

No.  I just changed my local copy.
Those few lines are the entire thing.

Is it possible that this is the same problem?
https://i.gyazo.com/f70cee1c511e8b89ad0ddc83fd3312ed.mp4

я поменял атрибут "passive: true" с "passive: false" в функции
function addEvent(el, type, fn, capture) {
    el.addEventListener(type, fn, {
      passive: true,
      capture: !!capture
    });
  }

Is there any solution to this problem without editing source code?

я поменял атрибут "passive: true" с "passive: false" в функции
function addEvent(el, type, fn, capture) {
    el.addEventListener(type, fn, {
      passive: true,
      capture: !!capture
    });
  }

И собственно все заработало. Либо можно пойти более длинным и сложным путем, меня отдельную логику для
addEvent(document, EVENT_TOUCH_MOVE, this._move);

this works.

There is PR https://github.com/xwpongithub/vue-range-slider/pull/34 but it's not merged still ((


how to use comma / space-bar key for adding tag?

I added "32", "44" in the default keycode area, and add some function like enterKeyPress.
But it won't works...
then I tried add one line:  "32  : 'enter!',"  after 13  : 'enter!'. and change e=c===13?"":d.val(); to e=c===13||c===32?"":d.val()
The space-bar key add tag successfully, but the e=c===13||c===32?"":d.val() need a trim().
I tried e=c===13||c===32&&a.trim(d.val())!==""?"":d.val(), but it returns  don't have trim method...
Is that possible for you to add a function that comma / space-bar key can also add a tag?

:+1:


[Vectorized][Enhancement] use simd to speed up coalesce and if_not_null function

Proposed changes
Issue Number: close #7685
Problem Summary:
Describe the overview of changes.
Checklist(Requried)

Does it affect the original behavior: (No)
Has unit tests been added: (No Need)
Has document been added or modified: (No Need)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

LGTM!


fix(quick-preview): WI description gets lost on updating WI fields.

Fixes https://github.com/openshiftio/openshift.io/issues/3842

Removed duplicate code for deactivating description editor.

Ike Plugins (test-keeper)
Thank you @divyanshiGupta for this contribution!

It appears that no tests have been added or updated in this PR.
Automated tests give us confidence in shipping reliable software. Please add some as part of this change.
If you are an admin or the reviewer of this PR and you are sure that no test is needed then you can use the command /ok-without-tests as a comment to make the status green.
Your plugin configuration is stored in the file.

@divyanshiGupta Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 and visit http://localhost:5000 to access it.

@divyanshiGupta Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 and visit http://localhost:5000 to access it.

@divyanshiGupta The issue remains.

Make a change to a field
Move away from the field without saving it
Change Area or Iteration. Observe that the entered value in step 1 is lost.

@rgarg1 according to our discussion yesterday let us create a task for UX to decide upon if the dynamic fields should remain open on updating any other WI field.
I have also added the test please have a look.

The PR is approved from the functionality perspective. The tests need to pass before the PR could be merged.
I will raise another UX task to track the case of dynamic fields clearing on WI reload.

@divyanshiGupta please do not remove the default template. Fill in the description and issue no. in the default template.

@Raunak1203 sure

Thanks @divyanshiGupta

@divyanshiGupta Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 and visit http://localhost:5000 to access it.

@divyanshiGupta Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 and visit http://localhost:5000 to access it.

@divyanshiGupta Your image is available in the registry. Run docker pull registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 && docker run -it -p 5000:8080 registry.devshift.net/fabric8-ui/fabric8-planner:SNAPSHOT-PR-2714 and visit http://localhost:5000 to access it.


carthage support watchOS

I added watchOS Target to support Carthage.
And, fixed Podfile and project file,
because Nimble and Quick is not support Bitcode.
thanks.

Fixed in #115


[QA] OldGuyBBQ - Cannot access rooftop level

Explorer Alpha build versions:
v0.20.0
Issue Description:

On scene /goto-97,-95 we can access a second floor and a rooftop level. Currently, the button to reach the rooftop is unresponsive.
STR:

/goto-97,-95
Press the button of Party Portal
At the next level, press the button with the arrow to access the rooftop

Expected behaviour:
The player should go to the selected level
Current behaviour:
The button is unresponsive
Evidence:
https://github.com/user-attachments/assets/ca7e8f62-b1e6-4ef2-995c-b2a44c182866
Additional Notes:

Notes by the user:
From Unknower: looks like that button is being generated by the Virtual Land Manager dependency after the scene's initial load in. It's using the movePlayerTo() function, which works on another click event in the scene. That rooftop button is working in browser/old client but not new client it seems

@Ludmilafantaniella can you please add evidence of what is the expected behavior in Explorer web?

The expected behavior is for the button to take you to the rooftop
https://github.com/user-attachments/assets/6ffa00c8-1e06-4a57-bd76-1be2809e8c37

We fixed teleport to, lets review this again on main @anicalbano

We fixed teleport to, lets review this again on main @anicalbano

On latest (checked 0.29.0) the button is still not working, doesn't take you to the rooftop by pressing it

@fcolarich lets verify the function being called is actually the one we fixed and see if we can see why this specific case isn't working.

The button press Input IS being sent through the CRDT but there is no "move player to" command coming from the scene... so seems like something is broken there...
This scene mentions it uses something called **Virtual Land Manager dependency **...
Do we have any way to look at the scene source code?

This one is really tricky to debug, because the issue seems to be in the underlying use of an outside tool that "manages" these buttons and functionality. And seems that some functionality of that tool does not work properly with the new client.
We would need to see the source code of the scene to continue debugging it, but its a very case specific issue sadly. Ideally the creator can just update the scene and put a proper button with a proper move player to method in it and that's all.
But if we are really willing to invest the time, we need the scene source code to check why its failing to properly link the button with the method call.

I talked to OldGuy who said we can close that ticket, he wants to revamp the all scene and will start working on it in a few days. Should the issue persist afterwards, he will reach out to us


Cannot access ignition

I cannot access the ignition key when in the drivers seat. i have no other cruiser mods and even tried turning off letmelookdown.
ill try using it with no other mods and report back.
cruiser is unusable with this issue though as i cannot start the car

Do you have any emote mods like TooManyEmotes? Some mods that change the player animator can cause players to stand up in the seat.


Feature: Get the URL to stream a video

I'm an amateur when it comes to OO design, so i wasnt sure if this was the best way to structure this. If you have any advice please let me know.
jurassic_park = plex.library.section('Movies').get('Jurassic Park')
url = StreamUrl(server=plex, key=jurassic_park.key, max_bitrate=128,
                         max_resolution="1280x720", offset=0)
print(url)

# http://123.456.789.864:10041/video/:/transcode/universal/start?copyts=1&offset=0&path=http%3A%2F%2F<IP_ADDRESS>%3A32400%2Flibrary%2Fmetadata%2F8&mediaIndex=0&X-Plex-Platform=Chrome

We should definetly add this.  However, I still deciding if this should in fact be a separate class as opposed to directly ingrained in the Video object.  It  seems quite complex, give me a few days to ponder this. :)

I think it should be in the video object as well, i just wanted to get it working first without thinking about any abstraction or inheritance.

I feel jurrasic_park.stream_url should return the URL to stream - there's no need to create an object, which in this instance is effectively a (not really) subclass of a String.

@munnerz That's what I was thinking as well. I have the code (a very simplified version of simonwjacksons version) almost ready to go, but can't actually get it to stream in VLC yet, so need to figure out what is wrong there.

@mjs7231 You may be missing a parameter in the URL that is preventing plex to encode the stream.

I added this without using a separate StreamURL object.  The function Video.getStreamURL was added using your code as the example, thanks!


Update CondaCommand.settings to use new file format

This update modifies CondaCommand.settings to use the new platform-specific file format.  Under the current system, Windows users won't be affected as it references the correct file, but for Linux and OSX, sublime.load_settings is getting a reference to a file name that no longer exists.
I tested this locally and verified that specific user settings were used if defined, and that defaults were used for everything else.
Please let me know if you would like me to make any changes or add additional documentation.  Sorry this was not included in the original PR.

No problem. I'm surprised the last PR worked without this change.


Provide default mocks in table_meta

Problem
At the moment we still need a lot of repetitive code to define our tests: Each from_mocks call needs to specify all the input table mocks even if their data would not change between tests.
What changed
We added a default_inputs argument to the table_meta decorator. This allows to pass reasonable default to the model.
Here is an example from the docs:
@table_meta(
    query_path="./examples/test_query.sql",
    default_inputs=[UserTable([]), SubscriptionTable([])] # We can provide defaults for the class if needed.
)
class MultipleSubscriptionUsersTable(BigQueryMockTable):
    user_id = col.Int(default=1)

# Setting up different scenarios to demonstrate the use of defaults
users = UserTable.from_dicts([{"user_id": 1}, {"user_id": 2}])
subscriptions = SubscriptionTable.from_dicts(
    [
        {"subscription_id": 1, "user_id": 1},
        {"subscription_id": 2, "user_id": 1},
        {"subscription_id": 2, "user_id": 2},
    ]
)

# Utilizing the default inputs set in the table_meta
res = MultipleSubscriptionUsersTable.from_mocks(input_data=[])
res = MultipleSubscriptionUsersTable.from_mocks(input_data=[users]) # Using only users, defaults for others
res = MultipleSubscriptionUsersTable.from_mocks(input_data=[users, subscriptions]) # Overriding defaults

What to look for
You can ignore the changes in docs/ since they are auto generated from the docsource/ markdown file changes.
Instead, have a look at:

What do you think about the "API" of providing defaults this way
Are the docs and examples understandable
General correctness

@karolinastawicka thanks for your review! I loved the suggestions and tried to add them with my changes.


Editor grabs focus when opening a file

Description of your problem
What steps will reproduce the problem?

In spyder open a couple of files in editor
Close spyder and restart

What is the expected output? What do you see instead?
After the main window appears, it is grabbing focus for each file that is reopened from the previous session. In addition each of these files is put to the front of the editor window.
Reopening files in a thread is a good idea to reduce startup time. However, at the moment this is not effective, as the user will not be able to do anything (neither in spyder nor in any other application) until all files are loaded.
Please provide any additional information below
Versions and main components

Spyder Version: 4.0.0dev 858193f
Python Version: 2.7.12
Operating system: osx 10.10

... more over, the files are opend twice if a project was active (I guess first spyder opens it's list and then later the project) -> should this be put in a separate issue?

Thanks for reporting. I opened an issue about files being opened twice at startup when a project is active. We'll fix this problem in 3.0.2.
I don't know what to do about this one though. It seems a problem when you have a lot of files opened in your last session (30, 50?), but nobody has complained about it before :-)

I think even with a handful of files it's slightly annoying though it wasn't annoying enough for me to complain about.

I think the solution would be to prevent our load function to steal focus at startup. Let's assign one of our juniors to it :-)

OK. Maybe I am too impatient … ;-) and maybe there is something wrong with my setup.
Anyway I can see the files open one by one (about 1-2 per second). Don’t know if others observe the same. Since I am developing a plugin, restart spyder frequently, 10 seconds waiting for my 10 files to open (20 when I have a project active) is getting on my nerves…
I imagine the quick fix would be, to not put each newly opened file on the front tab (e.g. open the one that was front last time first, then add tabs in the background without setting the focus). A way to speed up things may be to skip parsing each file (at least an outline is generated for each while loading) and deferr that to the moment the user switches to the file…. what do you think?
Cheers,
C.

On 24 Oct 2016, at 03:55, Carlos Cordoba<EMAIL_ADDRESS>wrote:
Thanks for reporting. I opened an issue about files being opened twice at startup when a project is active. We'll fix this problem in 3.0.2.
I don't know what to do about this one though. It seems a problem when you have a lot of files opened in your last session (30, 50?), but nobody has complained about it before :-)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

--
Christoph Gohle
<EMAIL_ADDRESS>


Use setup-go@v3 to avoid golangci-lint error: "Cannot open: File exists"

Changing the version of setup-go to v3
as the v4 version is leading to errors such as:
Cache Size: ~633 MB (663986965 B)
/usr/bin/tar -xf /home/runner/work/_temp/c621c0af-3135-4a1c-9118-
827d7db5e99b/cache.tzst -P -C /home/runner/work/ramen/ramen
--use-compress-program unzstd
Error: /usr/bin/tar<EMAIL_ADDRESS>yaml_test.go: Cannot open: File exists
Builds with the error:

15198188098

Fixes: #993

CI run after the change: https://github.com/RamenDR/ramen/actions/runs/5620511816
Before the change: https://github.com/RamenDR/ramen/actions/runs/560983082

@Reviewers Please notice that we have had 5 consecutive  successful builds and one failed build (unit tests, known issue) - all without the golangci error: https://github.com/RamenDR/ramen/actions


dask.sizeof very wrong for MultiIndex

Noticed in distributed diagnostics page when scattering large multi-index series:
import sys
import pandas as pd
from uuid import uuid4

def object_size(x):  # from dask.sizeof.register_pandas
    if not len(x):
        return 0
    sample = np.random.choice(x, size=20, replace=True)
    sample = list(map(sizeof, sample))
    return sum(sample) / 20 * len(x)

x = pd.MultiIndex.from_product([[uuid4().hex for i in range(5)], range(1_000_000)])
print(x[0])
pd.Series([sys.getsizeof(x), x.memory_usage(deep=True), object_size(x)]) / 1e6

Output:
0      74.944361
1      74.944337
2    1525.000000

Not sure how serious of an issue this is as I don't know what all these estimates are used for.  The scatter itself does seem to be affected for some reason:
%time future = client.scatter(<series>, broadcast=False)
Wall time: 1min 10s

%time future = client.scatter(<series>.reset_index(), broadcast=False)
Wall time: 2.06 s

The object_size there looks a little strange for MultiIndex. That forces a conversion to an ndarray of objects, which likely isn't what we want.
@mrocklin do you recall the reason for not just using index.memory_usage(deep=True)?

memory_usage is very slow when the object is large, so the subsampling approach seems reasonable; it just doesn't work in this case because repeat values aren't stored multiple times. Is it even feasible to get a good estimate of the size here without traversing the whole array?

Mmm, though the current way of np.random.choice converting to an ndarray of tuples isn't exactly fast :)
IMO, I think we're fine with doing deep memory usage here. If we think that number of unique values per level is smallish, then pandas implementation of
        level_nbytes = sum(i.memory_usage(deep=deep) for i in self.levels)

With a MI like
In [29]: idx = pd.MultiIndex.from_product([list(range(1000)), ['a', 'b']])

In [30]: idx.memory_usage(deep=True)
Out[30]: 55204

the deep part only affects a Index(['a', 'b'].memory_usage(deep=True)). Not too bad.

If we really think deep=True is problematic in practice, then we'll need to reimplement pandas' way of doing things, but sample the object levels.

If we are concerned with the cost of memory usage on a level with many values, I think we could define a MultiIndex-specific sizeof that applies the current sizeof_pandas_index to each level of the MultiIndex, and then adds in the size of the .codes (.labels for older pandas). Does that sound reasonable @bnaul?

I think that's probably the way to go; changing my example to 10^6 UUIDs, I get
[ins] In [105]: %time sys.getsizeof(x)
CPU times: user 1.54 s, sys: 10.3 ms, total: 1.55 s
Wall time: 1.56 s

[ins] In [106]: %time x.memory_usage(deep=True)
CPU times: user 1.45 s, sys: 10.3 ms, total: 1.46 s
Wall time: 1.47 s

[ins] In [108]: %time object_size(x)
CPU times: user 125 µs, sys: 16 µs, total: 141 µs
Wall time: 136 µs

Seems like the version you suggested would be about as fast as what we currently have (i.e. way faster than the full computation). I can try taking a stab at it in the next week or so.


Bug at proxy.rb:182: nil stub being invoked

The logic at line proxy.rb:174 yields false if stub is nil.  The very next condition, at line 180, tests only expectation, and then invokes the nil stub at line 182.  This leads to an internal exception: ArgumentError: wrong number of arguments (0 for 2).

@aks: can you provide an example use of rspec-mocks that triggers this sequence?

I'm under a deadline, or I would have tried to create a simple example.  Below are snippets of the code.
This is the only test that fails.
The failure is induced in a controller spec that is attempting to confirm proper handling of AR StaleObjectErrors
The controller method instance variable, @ffi_flow_function holds a decorated (see the Draper gem) FFI::FlowFunction instance.  Hence the .object method.
The rspec test doubles the flow function object, and expects the various messages needed in the sequence to test the conflict detection, which causes an StaleObjectError.
 context "with conflicting saves" do
      let(:new_attrs1) { { name: "My New Name1", description: "My New Desc 1" } }
      let(:new_attrs2) { { name: "My New Name2", description: "My New Desc 2" } }

      it "intercepts conflicting saves" do
        ff1 = flow_function
        ff2 = flow_function
        ff1.update_attributes new_attrs1
        ff1.save
        ffdbl = double(ff2)
        expect(FFI::FlowFunction).to receive(:find).with(any_args).and_return(ffdbl)
        expect(ffdbl).to receive(:decorate).and_return(ffdbl)
        expect(ffdbl).to receive(:object).and_return(ffdbl)
        expect(ffdbl).to receive(:updated_by_id=).with(any_args)
        expect(ffdbl).to receive(:status).and_return(1)
        expect(ffdbl).to receive(:update_params).with(any_args).and_raise(ActiveRecord::StaleObjectError)
        put :update, {:id => ff2.to_param, :ffi_flow_function => new_attrs2 }, valid_session
        expect(flash[:error]).to be_present
      end
    end

The controller code being tested is here:
  # PATCH/PUT /ffi/flow_functions/1
  # PATCH/PUT /ffi/flow_functions/1.json
  def update
    ff_params = ffi_flow_function_params

    respond_to do |format|
      begin
        ffobj = @ffi_flow_function.object
        update_result = ffobj.update_params(ff_params)
        if update_result
          format.html { redirect_to @ffi_flow_function.object.as_parent, notice: 'Flow function was successfully updated.' }
          format.json { render :show, status: :ok, location: @ffi_flow_function}
        else
          format.html { render :edit }
          format.json { render json: @ffi_flow_function.object.errors, status: :unprocessable_entity }
        end
      rescue ActiveRecord::StaleObjectError => e
        log_debug "StaleObject caught on PATCH/PUT FFI::FlowFunction"

        @ffi_conflict = FFI::Conflict.create! parameters: ff_params,
                                              class_name: @ffi_flow_function.class.to_s,
                                              record_id: @ffi_flow_function.object.id,
                                              user_id: current_user_id
        flash[:error] = "Conflict detected on update! Conflict(#{@ffi_conflict.id})"
        format.html { render 'ffi/conflicts/show' }
        format.json { head :no_content }
      end
    end
  end

Here is a stacktrace:
=> ["/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/activerecord-4.2.5/lib/active_record/errors.rb:128:in `initialize'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:149:in `exception'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:149:in `raise'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:149:in `block in and_raise'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:660:in `call'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:660:in `block in call'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:659:in `map'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:659:in `call'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:554:in `invoke_incrementing_actual_calls_by'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/message_expectation.rb:411:in `invoke'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/proxy.rb:182:in `message_received'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/method_double.rb:77:in `proxy_method_invoked'",
 "/Users/aks/.rbenv/versions/2.2.3/gemsets/borg/gems/rspec-mocks-3.4.1/lib/rspec/mocks/method_double.rb:64:in `block (2 levels) in define_proxy_method'",
 "/Users/aks/src/ecorithm/BORG/app/controllers/ffi/flow_functions_controller.rb:100:in `block in update'",

Finally, the :update_params method is defined within the AR subclass itself, but is added on through an included module.  The module does this:
module FFI::FlowFunctionUtilities
  FFI::FlowFunction.class_eval do
    ...
    def update_params in_params
      ...
    end
  end
end

Perhaps it's possible that the method validator in rspec is being confused by the dynamically injected method?

This is not a bug.
The problem is that ActiveRecord::StaleObjectError requires two arguments, and was being raised without any arguments.
Closing this issue.

Glad you figured it out!


rotate a matrix

Rotate clockwise and anti-clockwise
This implementation first defines two functions, rotate clockwise and rotate counterclockwise,
to rotate the image by 90 degrees clockwise and counterclockwise, respectively.
Each function takes a 2D vector image as input and modifies it in place.
The printImage function is used to print the image for demonstration purposes.
Finally, the main function initializes an example image, prints it, rotates
it by 90 degrees clockwise, prints it again, rotates it by 90 degrees counterclockwise,
and prints it one last time to verify that the rotations worked as expected.

@Kumar-laxmi assign this to me under SSoC'23

This is a LeetCode Question, please don't raise issues to solve LeetCode, gfg or hackerrank questions


[Question][Workflow Definition ]  Could I have a DAG Group function that categories will be  set between different tasks on Workflow Definition ?

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
No response
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Can you expand on the description of the problem?


i18n

Bruno currently only supports English Locale.
If you feel that multiple locale's would help. Please upvote this issue and comment the locale you would like to be supported.

fr_FR

import openapi not display chinese

import openapi not display chinese

deplicated https://github.com/usebruno/bruno/issues/671

zh_CN

need zh_CN

zh_CN please

Please add zh_CN language support, preferably internationalization support instead of just zh_CN

I'd be more willing to pay for it if client-side multi-language support is available


Broken links to hardfought

The dumplog links to hardfought on my profile are broken, e.g. the ascension from 2020-11-16 links to: https://.hardfought.org/userdata/d/deki/tnnt/dumplog/1605445381.tnnt.txt (-> the eu part is missing in the domain, as I was playing on that server).

thanks for the report! this has since been fixed :)

thanks for the report! this has since been fixed :)


Support insert_all

just give option to replace create! with insert_all please https://apidock.com/rails/v6.0.0/ActiveRecord/Persistence/ClassMethods/insert_all

@lubosch I have implemented this feature in our fork https://github.com/renuo/seed_dump. Maybe it helps.

gem "db_seed_dump", git: 'https://github.com/renuo/seed_dump', require: 'seed_dump'


Edit pybryt.utils.save_notebook to only save in the classic Jupyter Notebook interface

The method used to force-save a notebook in pybryt.utils.save_notebook is incompatible with JuppyterLab and VSCode. This method should only be used when a user is in the classic notebook interface. The function should determine which interface a user is using and force save differently based on that information.

Implemented in #29


Fix computeTangents

I load a gltf file, then assert failed at this point:

After spending some time to investigate, the problem is found here:

This is the log's out puts:

Asking @prideout to confirm the fix :)


KAFKA-2722: Improve ISR change propagation.

The patch has two changes:

fixed a bug in controller that it sends UpdateMetadataRequest of all the partitions in the cluster.
Uses the following rules to propagate ISR change: 1) if there are ISR changes pending propagation and the last ISR change is more than five seconds ago, propagate the changes. 2) if there is ISR change at T in the recent five seconds, delay the propagation until T + 5s. 3) if the last propagation is more than 1 min ago, ignore rule No.2 and propagate ISR change if there are changes pending propagation.

This algorithm avoids a fixed configuration of ISR propagation interval as we discussed about in KIP-29.

@junrao This seems a problem that needs to fix in 0.9. The patch is not big. Do you mind take a look?

The test failure is irrelevant.

Thanks for the patch. LGTM


Unable to use Witchcraft, iex (1.8.1)

Hello 👋
I have an issue when trying to get into an interactive session with {:witchcraft, "~> 1.0"}
Interactive Elixir (1.8.1)
Erlang/OTP 21 [erts-10.2.5]
See recording
It gets stuck after Generated witchcraft app and my computer starts to behave so badly.

Note. Never used Witchcraft before

A couple questions:

What machine/OS do you use?
Do other apps compile ok with iex -S mix?
Did you tried rebooting your computer? (This may sound stupid, but I mostly suspend my laptop, and only restart it once a couple months, or when needed, and sometimes this did solve issues.)
Do you have a repo online that could be cloned to try it out?

Hi, thanks for the answer

macOS 10.14.2
Everything else works like a charm
I did many times, just in case, but did't make any difference
Y don't, I have nothing, just created a new project, added witchcraft to deps as described in the documentation.

Were you able to solve this? Sorry for the couple months of delay.

I think I have an idea: your own application has the same name as the package you are using. Normally Elixir would fail to compile (as it tries to compile the same module twice), however IEx allows to recompile and reload previously compiled modules (for new versions) hence you don't get the same behaviour. If you pay attention to the last part, you'll see it says:
Generated witchcraft app                                                                    ==> witchcraft                                                                              Compiling 1 file (.ex)                                                                      warning: redefining module Witchcraft (current version loaded from _build/dev/lib/witchcraft/ebin/Elixir.Witchcraft.beam)                                                                 lib/witchcraft.ex:1

warning: redefining module Witchcraft
Solution: rename your project.
I'm almost 100% certain about this, but feel free to let me know if I was wrong ;)


Always offer the install target names for nvtx3

Description
This is required so that consumers of nvtx3 can safely use nvtx3::nvtx3-cpp consistently no matter if they are using nvtx3 from source or an installed version.
Checklist

[x] I am familiar with the Contributing Guidelines.
[x] New or existing tests cover these changes.
[x] The documentation is up to date with these changes.
[x] The cmake-format.json is up to date with these changes.

@robertmaynard do we also want to add this to the FINAL_CODE_BLOCK of the rapids_export() calls? I recall we also needed to do this in get_arrow.cmake.

@robertmaynard do we also want to add this to the FINAL_CODE_BLOCK of the rapids_export() calls? I recall we also needed to do this in get_arrow.cmake.

We won't need to do that with nvtx3 since rapids-cmake is generating the build and install nvtx3-config.cmake and nvtx3-targets.cmake.

/merge


Profile page web

I haven't done a pull request to the main branch, figured I might as well. Nothing here is set in stone, there's some obvious bugs, and it's not linked to the API yet.
The main files I edited/created.

web/src/components/Dialogs/CollectionDialog.tsx
web/src/components/Surfaces/CollectionCard.tsx
web/src/pages/Profile/collections.tsx
web/src/pages/Profile/index.tsx

I'm not sure if the other diffs are worth mentioning, but I can try and change them if required.

Should probably also mention how to access this profile page:
http://localhost:3000/Profile

I'm not sure if the other diffs are worth mentioning, but I can try and change them if required.

Went ahead and fixed this!
Yarn lint:

45:15  Error: Missing "key" prop for element in iterator  react/jsx-key
This is okay, because it only applies to a temporary solution. It will be nuked as soon as it's actually connected to the API.

Not sure if this is your doing or not, but we have both pages/profile.tsx and pages/Profile/index.tsx, which is not allowed.


coverage: add exception for omit canary retry extension

Commit Message: adding back the per file exception as master is currently red due to this.
Risk Level: low
Testing: CI
Signed-off-by: Jose Nino<EMAIL_ADDRESS>

I rolled it into #12067 to fix master


Menu has double entries of Outbox as an option

+1 metoo bugspam. here's my gmx.com folders:

The first one is in Russian Исходящие and the second's in English OUTBOX.

K-9 Mail uses an internal folder to store messages that have yet to be sent. Your server most likely contains a folder named "Outbox" as well.
The next stable version of K-9 Mail will display the internal folder using a special icon. The other folder will most likely be displayed with a regular folder icon.


Add profile for 01 Communique (IronCAP)

Automatically generated pull request to add profile for 01 Communique (IronCAP) according to application pkic/members#210.
This closes pkic/members#210

01 Communique is already listed.


Better "New Contributor" Pagination

Purpose
Add pagination to to "Add Contributor" modal when contributors are imported from the parent (closes #3556) and fixes bug with persisting paginatior buttons when user selects "Most Common Projects" or "Most Recent Collaborators" as contributor searches.
Changes
makes all search use paginator fetchResults() and adds function that collates results for paginator pages.
Side Effects
None that I know of.

@Johnetordoff all the pagination calculation logic should be on server side. You can just modify the url by make a new observable, this.url, change this url valuable in different method before the fetchResult is called.

It's a little more complicated then that because search_contributor is the only search that actually makes an elastic search query, so it the only one that has sever-side pagination calculation already, that's why they weren't originally paginated, I could go add individual pagination calculation to all them, but obviously a single function is easier.
Also why should the url be an observable? It's only be used internally, right?

@Johnetordoff Getting this up to date with develop should fix the build. Can you do that?

@sloria its up to date now

@Johnetordoff first pass done.

Just a few driveby comments; I haven't fully reviewed this. Ping @chennan47 for second pass.

@Johnetordoff @sloria second pass done. Some minor changes.

The New Contributor UI has diverged significantly since this PR was made. Closing this for the time being.


fix openFirewallPorts to build the network name with the same format …

…used in createInstance, allowing the user to pass a fully-qualified url for network and subnetwork resources

Description

https://github.com/docker/machine/pull/4335 added support for fully qualified network and subnetwork names in the google driver. This pr builds on that to fix the formatting of the network name in the openFirewallPorts method.
Currently, the driver will append the fully qualified network name to the globalUrl instead of taking it as-is.
Related issue(s)

https://github.com/docker/machine/issues/4344

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "master" git@github.com:perryao/machine.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.

This PR will be merged?

Can we get this? Is anyone yet support docker machine project? I do not think so :)


How to add Defaults? - e.g. Keyed  =  Ui region wrapper (List (String, Html msg))

Module Order
-> Prior art. How do other libraries do it?

now added a file under Layout; might move to Restrictive/Html/Keyed because it's not just layout. In that case, same with Region bc it has an analogious status to Html.


Where is Conda.sublime-build installing from Package Control?

Under windows, I have to add shell: true to Conda.sublime-build to make matplotlib show the plots.
Installing through git, no problem at all.
Installing through Package Control, can't find Conda.sublime-build anywhere.

When installed via Package Control, the sublime-build file will be located in C:\User\AppData\Roaming\Sublime Text 3\Installed Packages. Let me know if you don't find it there.


Usage as model extension

Hi,
More info as how to use SoapService as model, atm with the example from docs i get this error:

Non-static method App\Soap::functions() should not be called statically, assuming $this from incompatible context

Thanks!

You need to use the model in your controller;
use App\Model;

I also would like a bit more info on using in a model.
for example, I have a model that requires passing some options for SSL
'ssl' => array(
'verify_peer'   => false,
'verify_peer_name' => false,
'allow_self_signed' => true
)
I know how this can be done in a controller but not in a model and therefore I get unable to load errors on the WSDL, but putting these calls into a model would save me alot of lines of coding if possible.


python310Packages.scikit-survival: 0.17.1 -> 0.17.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.scikit-survival is: "Survival analysis built on top of scikit-learn"
meta.homepage for python310Packages.scikit-survival is: "https://github.com/sebp/scikit-survival"
meta.changelog for python310Packages.scikit-survival is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 0.17.2 with grep in /nix/store/cf673v56mw93qganyf1c4nmg0c3qbabh-python3.10-scikit-survival-0.17.2
found 0.17.2 in filename of file in /nix/store/cf673v56mw93qganyf1c4nmg0c3qbabh-python3.10-scikit-survival-0.17.2

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 x86_64-darwin rebuild(s)

First fifty rebuilds by attrpath
python310Packages.scikit-survival
python39Packages.scikit-survival

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/cf673v56mw93qganyf1c4nmg0c3qbabh-python3.10-scikit-survival-0.17.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.scikit-survival https://github.com/r-ryantm/nixpkgs/archive/8e20059ffa99aa2bd06fc2a75bf3d789c072e5dc.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/cf673v56mw93qganyf1c4nmg0c3qbabh-python3.10-scikit-survival-0.17.2
ls -la /nix/store/cf673v56mw93qganyf1c4nmg0c3qbabh-python3.10-scikit-survival-0.17.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python310Packages.scikit-survival
python39Packages.scikit-survival

Maintainer pings
cc @GuillaumeDesforges for testing.

@GrahamcOfBorg eval


Hide on click

Hi I think it would make sense to hide these buttons on click as now (in chrome) there's a blue outline on click and once the button hides. Also this is the way it works in OsX and its really good UX.
This can be easily achieved by :active css selector. I can fork & update if you are willing to accept?
@kivylius

I think this is an issue about the outline of the button and balloon? outline: 0 on the balloon and button would get rid of the blue outline on Chrome. On Android (Nexus 5x) Chrome,  I also see a thin orange outline that makes the balloon a bit awkward looking. Setting outline to 0 or none would probably fix that. However, getting rid of the outline would be bad for accessibility so I would also recommend what @kivylius suggested, to add an active/focused style on :active and :focus.
I will attach a screenshot later if anyone is curious ...

@Kivylius @yanandcoffee My concern is exactly about accessibility. To prevent the awkward outline effect, the outline: none property would need to lie on the parent tooltip element. If this element is a button, link or any other interactive element, this would lead to users not knowing when it is focused (which is not a good thing).
I think its be better to let the developer decide when removing it or not. If one really wants to remove the outline on one's project, they could just add this rule at their CSS and its done:
[data-balloon] {
    outline: none;
}

Or maybe replace the outline with a better looking effect.

For static text is fine, but majority of people will be using this for button or links. Also what about a hiding on balloon on click :active & `:focus'?

@Kivylius It doesn't seem possible to disable this on only the balloon on-click without disabling it in general. If you do outline: none; on the :active, it doesn't do anything. If you do it on the :focus, it just disables it the same way as putting it on the parent tooltip element.
@kazzkiq  I agree with you that I don't think there's any right way to solve this without hurting accessibility. I think that on mobile, people may be clicking for the tooltip, but the super thin line on Chrome is barely noticeable and purely aesthetic. On desktop, it seems more obnoxious, but it is possible for the developer to intentionally solve this with an override and for the developer to take responsibility of accessibility concerns. Here is one possible solution/suggestion for the styling of :focus and :active if you do want to take away the outline...

It's not possible to put a border around the little triangle from what I've tried, otherwise an overall border around the entire bubble may be a more visible style...

Yea the blue outline might have to stay. There is another issue that we are not addressing:
On hover on <button>; balloon shows, but once that same button is clicked balloon should hide itself.

I know this is kind of an issue a little bit beside the discussion, but as the title say "Hide on click" I'm wondering how to make it hide on touch devices, because after click it stays visible.

@almadsen That happens because the when you tap on touch devices, the browser simulate the :hover effect. Thats why when you tap over it the tooltip keeps visible.
To avoid this kind of behavior in mobile browsers you can use one of those approaches:
http://stackoverflow.com/questions/2851663


Update to use AliasedSolrQuerySet for member and item list views

Codecov Report

Merging #289 into develop will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #289      +/-   ##
===========================================
+ Coverage    98.45%   98.45%   +<.01%
===========================================
  Files          133      135       +2
  Lines         6267     6272       +5
  Branches         5        5
===========================================
+ Hits          6170     6175       +5
  Misses          97       97


feat(prometheus-collector): release for 2/15

What type of PR is this?
feat(prometheus-collector): release for 2/15

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[x] includes documentation
[x] adds unit tests
[x] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.313%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
2285

Relevant Lines:
2881

💛  - Coveralls


fix motion events detection for iOS 13.4, which are broken

This will fix immersive-web/webxr-polyfill and basically any app out there which uses that webxr-polyfill (including the webxr-examples when apply the one another pre-grant permissions condition on iOS).
This fix is needed because devicemotion is broken in Safari for iOS 13.4, (as it was on Chrome m65).
So this PR adds the double condition to the actual isWithoutDeviceMotion variable.

I'd love to put in a vote for this, as this is a key component in getting aframe cardboard mode working again in iOS 13.4 (https://github.com/aframevr/aframe/issues/4554)

Is this fixed in Safari for versions after 13_4, if there are any? I don't have iOS devices to test, but I'll get this in

Landed in de9678d8f077ceeddf9873bb81ee43b36e9d9651 in 1.0.19


MySQL works fine on dev box, but {"message": "Forbidden"} on AWS

Hello!
I have diligently followed every step here...
https://docs.google.com/presentation/d/1aYeOMgQl4V_fFgT5VNoycdXtob1v6xVUWlyxoTEiTw0/edit#slide=id.g1a01adb9b7_0_159
And here...
https://jinwright.net/how-deploy-serverless-wsgi-app-using-zappa/
...but still cannot get my Flask app to load on AWS once I add a database layer to the code.
Using Python 3.6 & yesterday's most recent version of Zappa.
Site works perfectly (accessing a mysql database on AWS!) from my desktop computer, but when uploaded to AWS I am met with
{"message": "Forbidden"}

when visiting certified domain or
{"message": "Internal server error"}

when visiting the original AWS-issued domain.
I have no clue what's happening here but I'm not far from switching from Flask to Django just to see if that fixes the problem (which is less than ideal because I don't want to re-write that code but I need a working app). That may be what I work on while waiting for a response here.
Steps to Reproduce
Here is all code involved:
app.py
from flask import Flask, render_template
from flask_mail import Mail
from flask_security import Security, SQLAlchemyUserDatastore, \
    UserMixin, RoleMixin, login_required
from flask_social import Social
from flask_sqlalchemy import SQLAlchemy

# Create app
app = Flask(__name__)
app.config['DEBUG'] = True
app.config['SECRET_KEY'] = 'super-secret'

# Configure Security
app.config['SECURITY_PASSWORD_HASH'] = 'plaintext'
app.config['SECURITY_REGISTERABLE'] = False
app.config['SECURITY_RECOVERABLE'] = False
app.config['SECURITY_CHANGEABLE'] = False
app.config['SECURITY_CONFIRMABLE'] = False

# MySQL Config
app.config['SQLALCHEMY_DATABASE_URI'] = 'mysql://spio_mysqluser:spio_mysqlpass@spio-zappa-users.cpk9wqigsbjj.us-east-1.rds.amazonaws.com:3306/dev'
app.config['SQLALCHEMY_TRACK_MODIFICATIONS'] = False

# Create database connection object
db = SQLAlchemy(app)

# Define models
roles_users = db.Table('roles_users',
        db.Column('user_id', db.Integer(), db.ForeignKey('user.id')),
        db.Column('role_id', db.Integer(), db.ForeignKey('role.id')))

class Role(db.Model, RoleMixin):
    id = db.Column(db.Integer(), primary_key=True)
    name = db.Column(db.String(80))
    description = db.Column(db.String(255))

class User(db.Model, UserMixin):
    id = db.Column(db.Integer, primary_key=True)
    email = db.Column(db.String(255))
    password = db.Column(db.String(255))
    active = db.Column(db.Boolean())
    confirmed_at = db.Column(db.DateTime())
    roles = db.relationship('Role', secondary=roles_users,
                            backref=db.backref('users', lazy='dynamic'))

# Setup Flask-Security
user_datastore = SQLAlchemyUserDatastore(db, User, Role)
security = Security(app, user_datastore)

# Create a user to test with
@app.before_first_request
def create_user():
    db.create_all()
<EMAIL_ADDRESS>password='password')
    db.session.commit()

# Views
@app.route('/')
def home():
    return render_template('home.html')

if __name__ == '__main__':
    app.debug = True
    app.run()

and zappa_settings.json
{
    "dev": {
        "app_function": "app.app",
        "aws_region": "us-east-1",
        "binary_support": false,
        "certificate_arn": "arn:aws:acm:us-east-1:735853783919:certificate/e57ef6c8-c524-44bc-af63-b590a1730e26",
        "cors": false,
        "debug": true,
        "domain": "stockpicker.io",
        "profile_name": "default",
        "s3_bucket": "zappa-stockpicker-1",
        "x-amazon-apigateway-integration": {
            "contentHandling": "CONVERT_TO_TEXT"
        },
        "vpc_config": { // Optional VPC configuration for Lambda function
            "SubnetIds": [ "subnet-7f1b7353", "subnet-fc4727a6", "subnet-a0ea1cc4", "subnet-bf4e3db3" ], // Note: not all availability zones support Lambda!
            "SecurityGroupIds": [ "sg-e3638593" ]
        }
    }
}

Zappa 0.43.1
Flask 0.12.2
Python 3.6.2
Output of pip freeze (really messy because I'm still developing and haven't eliminated any packages yet but have explored many):
argcomplete==1.8.2
Babel==2.4.0
base58==0.2.4
blinker==1.4
blist==1.3.6
boto3==1.4.4
botocore==1.5.40
cassandra-driver==3.11.0
certifi==2<IP_ADDRESS>
chardet==3.0.4
click==6.7
cql==1.4.0
docutils==0.13.1
durationpy==0.4
envs==1.2.2
facebook-sdk==3.0.0a0
first==2.0.1
Flask==0.12.2
Flask-BabelEx==0.9.3
Flask-CQLAlchemy==1.2.0
Flask-Login==0.4.0
Flask-Mail==0.9.1
flask-mongoengine==0.9.3
Flask-OAuth==0.12
Flask-Principal==0.4.0
Flask-Security==3.0.0
Flask-Social==1.6.2
Flask-SQLAlchemy==2.2
Flask-WTF==0.14.2
future==0.16.0
futures==3.1.1
google-api-python-client==1.6.2
hjson==2.0.7
httplib2==0.10.3
idna==2.5
itsdangerous==0.24
Jinja2==2.9.6
jmespath==0.9.3
kappa==0.6.0
lambda-packages==0.16.1
MarkupSafe==1.0
mongoengine==0.13.0
mysqlclient==1.3.10
oauth2==1.9.0.post1
oauth2client==4.1.2
passlib==1.7.1
pip-tools==1.9.0
placebo==0.8.1
pyasn1==0.3.1
pyasn1-modules==0.0.10
pymongo==3.4.0
python-dateutil==2.6.1
python-slugify==1.2.4
pytz==2017.2
PyYAML==3.12
requests==2.18.2
rsa==3.4.2
s3transfer==0.1.10
six==1.10.0
speaklater==1.3
SQLAlchemy==1.1.12
terminaltables==3.1.0
thrift==0.10.0
toml==0.9.2
tqdm==4.14.0
troposphere==1.9.5
Unidecode==0.4.21
update==0.4.4
uritemplate==3.0.0
urllib3==1.22
uuid==1.30
Werkzeug==0.12.2
wsgi-request-logger==0.4.6
WTForms==2.1
zappa==0.43.1

Okay looking forward to any ideas you guys might have. :) Really eager to move on to the next problem lol.

This might be something. In my cloudwatch  logs every time there's a request:
Syntax error in module 'handler': invalid syntax (uuid.py, line 138)

I have looked at the code and it is well beyond my own wizardry to understand its content. If there's a typo in that code I lack the skill to see it. This may help or it may be nothing, I have no idea where in the code that function is even called.

Here is the code in question (lines 133-152):
        if fields is not None:
            if len(fields) != 6:
                raise ValueError('fields is not a 6-tuple')
            (time_low, time_mid, time_hi_version,
             clock_seq_hi_variant, clock_seq_low, node) = fields
            if not 0 <= time_low < 1<<32L:
                raise ValueError('field 1 out of range (need a 32-bit value)')
            if not 0 <= time_mid < 1<<16L:
                raise ValueError('field 2 out of range (need a 16-bit value)')
            if not 0 <= time_hi_version < 1<<16L:
                raise ValueError('field 3 out of range (need a 16-bit value)')
            if not 0 <= clock_seq_hi_variant < 1<<8L:
                raise ValueError('field 4 out of range (need an 8-bit value)')
            if not 0 <= clock_seq_low < 1<<8L:
                raise ValueError('field 5 out of range (need an 8-bit value)')
            if not 0 <= node < 1<<48L:
                raise ValueError('field 6 out of range (need a 48-bit value)')
            clock_seq = (clock_seq_hi_variant << 8L) | clock_seq_low
            int = ((time_low << 96L) | (time_mid << 80L) |
                   (time_hi_version << 64L) | (clock_seq << 48L) | node)

Line 138 (in Atom) is:
if not 0 <= time_low < 1<<32L:

0 <= time_low < 1<<32L is valid syntax for Python 2.7 but not for Python 3.6.
Maybe that really old package: https://pypi.python.org/pypi/uuid/1.30 is not really compatible with 3.6. I would try to remove uuid package from requirements file and rely on uuid module that comes with python distribution.

This is an excellent bug report! Zappa doesn't need that uuid package, I think..

Oh excellent! Any ideas for syntax on editing that out or should I stay posted for a patch?

If it's not in zappa it may be somewhere in Flask security. I'm about 1/3 of the way into migrating to Django and may just see that through, seems like a more stable framework for my purposes.

Good news everyone! I got it working in PostgreSQL, which I'm pretty sure is better anyway. Somewhere in the past 36 hours of debugging and migrating platforms, some line of code somewhere got fixed and some subnet got the right config and now everything is playing nice. Gracias por la ayuda.

Can you please provide me more details? How did you fixed following issue?
{"message":"Forbidden"}

When you create a public accessible database in RDS, the inbound rule of the security group will not have <IP_ADDRESS>/0 as allowed sources, but the IP address from your machine, which were used when the DB was created. Under these circumstances the db can be accessed from the dev machine just fine, but not from zappa (or any other machine for that matter).


Added new testcase for multiple volume backup at once

What this PR does / why we need it:
Which issue(s) this PR fixes (optional)
Closes #
Special notes for your reviewer:

Task put on hold. Will raise a new PR


Improvements to how dvsim.py runs jobs

This patch queue tries to unpick some of the logic in Deploy.py which has got a little tangled. There are quite a few commits (9), because I wanted to merge incremental, bisectable changes rather than a giant "rewrite the world" commit.
The main thrust of the set of patches is to move the scheduling logic into a new file (Runner.py) and untangle all the "scheduley" bits from the code in the Deploy class. There are a few main changes:

Job tracking is now done in a scheduler class, rather than in the Deploy class, which also represents jobs to be run.
Jobs now have a notion of dependencies, rather than the ability to spawn new jobs. This matches other build systems much more closely, and also simplifies how we do coverage runs.
Signal handling (intercepting SIGINT) is now done by the scheduler rather than dvsim.py.

There are a few user-visible changes (and hopefully no bugs!):

If you have 100 tests that depend on 1 build and the build fails, you don't have to wait 100 / parallelism seconds for dvsim to crank through the tests it can't run.
The timer doesn't reset to zero when starting coverage collection.

This refactoring also makes it possible to make other improvements in the future. For example, it would now be much simpler to add LSF job array support: we could essentially just dump the commands for a given target into shell scripts and then fire them all off at once. In the code, this would be represented by an LSFTargetRunner class (subclassing TargetRunner) with a specialized (and simplified!) run method.
Another possible future improvement (which is slightly at odds with LSF magic) is to allow different "targets" (the phases of the run) to run at once, which would allow you to fire off some runs as soon as their build was ready. This would make no difference on my laptop (with few cores and fewer EDA licences), but might make a significant difference on a big compute cluster, where you can get started on the big list of tests as soon as some can be run. Of course, this is getting to look awfully like a Make replacement: if we move further in that direction, I suggest dumping commands into shell scripts and writing out the dependency DAG to a Makefile.
@cindychip, @msfschaffner: As usual, I can't try out some of the linter commands or the FPV commands locally. If you have a minute, I'd be really grateful if you could try out this PR to make sure I haven't broken anything with a silly typo. I think that most of the cleverness is on the simulation side (which I could test), so I'm hopeful this will work.

Force-push rebases on top of the code in PR #3948, which clashes with this PR.

Force-push fixes --run-only (before the fix, I'd accidentally left dependencies from the run jobs on the un-queued build jobs).

Force-push rebases onto master (no longer depending on a merged PR)

Force push has no interesting changes: just a weekly rebase to avoid file conflicts.

Hey Rupert, I am super sorry for the late reply.. i tried it on both FPV and VeribleLint, I think they are working fine. And the changes look very cool to me. I think would be good to wait until Sri has a chance to review this.

Hi Cindy, Thanks for the response. I think that Sri wants to take a very close look at any dvsim changes (we don't have any test suite and he understandably doesn't want to break this project-critical infrastructure). So I'm just rebasing once a week and leaving things in a working state until he has a chance to look at it.

Force-push is just a weekly rebase to keep everything from bit-rotting.

Another week, another rebase. No intentional changes, although I did have to spend some time fixing things up when rebasing past recent dvsim changes. It's a bit of a pity that this is gently bit-rotting.

Force-push re-does the _queued change (it only needed to change to a list once we split up the scheduler).

Another week, another rebase. No intentional changes, although I did have to spend some time fixing things up when rebasing past recent dvsim changes. It's a bit of a pity that this is gently bit-rotting.

It indeed is unfortunate. It has been already communicated that non-urgent / non-bug-fix enhancements like these, especially when its a mishmash of commits that make changes to several different parts of the code, will be reviewed with a much lower priority. There are at least 4 discrete meaningful changes in here that could have been submitted as separate PRs, which would have been a lot easier / smoother to review. Separate commits in the same PR is not at all helpful, if the diff in latter commits is against your own code from previous commits!
I have already communicated before - large, non-urgent PRs will be looked at with a lower priority (expect an SLO of ~60 days). If that is unacceptable, please justify. If it is urgent, please file an issue so that the priority is understood.
@domrizz0 @asb @sjgitty @tomroberts-lowrisc

@sriyerg: Would you prefer me just to close this PR? I suggested splitting it up yesterday, but you've replied to a previous message. What would you like me to do? Obviously, I don't want to cause more work for you (or unnecessary friction). And I'm sorry for letting any frustration appear in a GitHub message.

@sriyerg: Would you prefer me just to close this PR? I suggested splitting it up yesterday, but you've replied to a previous message. What would you like me to do? Obviously, I don't want to cause more work for you (or unnecessary friction). And I'm sorry for letting any frustration appear in a GitHub message.

No need to close this PR (or the others) - I am reviewing this one now. I think these are pretty good enhancements. To be clear, my intention is certainly not to discourage anyone from making improvements to the tool. It's just that it will take some time for me to get to them. How long depends on how easy it is to review the changes.

Hi Sri. Thanks for the review comments. I've addressed them (I think) in the latest force-push. I don't think that the PR splits very easily, or at least I think smaller PRs would end up clashing lots, so we'd still have to merge them in order.
Would you prefer me to split out an initial chunk, so that you can review it separately there? The first 2 commits are kind of trivial: maybe they'd be a good initial candidate?

Like you said, those are trivial - I am fine either way. I am done reviewing those 2. I am looking at the latter commits now.

Force-push is just a rebase over some recent dvsim changes that git couldn't resolve automatically. No functional change.

Force-push rebases over Weicai's cov_unr changes.

Force-push is just a weekly rebase to keep everything applying cleanly.

Force-push rebases over the change to how we load configs.

Force-push rebases over recent remote scratch changes.

Happy New Year! Force-push is just a rebase.

Happy New Year! Force-push is just a rebase.

LGTM, will take the fix-if-something broke approach (nothing seems to be broken for now). Thanks for these changes. The splitting of the scheduling logic was definitely needed, which will be a step towards supporting a wide variety of load sharing tools in a more natural way.
At Google, LSF is getting deprecated. We need to completely switch to using GCP within a month's timeframe. The way jobs interact with GCP is of proprietary nature and hence, cannot be exposed to the open repo. However, it is not too different from how LSF works, so it should not be too hard.
The way I am envisioning it - we need to support local dispatch and LSF as backends, both of which can live in the open source repo. In addition, if a company wants to use their own compute dispatch method (GCP in our case), they should be able to easily extend the TargetScheduler class and override (or add) appropriate APIs to be able to interact with the dispatched job. This extended class will be private to the company. The goal is to make it easy to get dvsim to dispatch to AWS, Azure etc. in future based on each OT partner's needs. I will be making updates to that logic to this end in the coming weeks.
I suppose you have some more ideas on improving dvsim. It will be great if we could document what's being planned and when is it expected to land, so that we are not blindsided by each other's updates. Let me know how do you want to track it. Thanks!

LGTM, will take the fix-if-something broke approach (nothing seems to be broken for now). Thanks for these changes. The splitting of the scheduling logic was definitely needed, which will be a step towards supporting a wide variety of load sharing tools in a more natural way.
At Google, LSF is getting deprecated. We need to completely switch to using GCP within a month's timeframe. The way jobs interact with GCP is of proprietary nature and hence, cannot be exposed to the open repo. However, it is not too different from how LSF works, so it should not be too hard.
The way I am envisioning it - we need to support local dispatch and LSF as backends, both of which can live in the open source repo. In addition, if a company wants to use their own compute dispatch method (GCP in our case), they should be able to easily extend the TargetScheduler class and override (or add) appropriate APIs to be able to interact with the dispatched job. This extended class will be private to the company. The goal is to make it easy to get dvsim to dispatch to AWS, Azure etc. in future based on each OT partner's needs. I will be making updates to that logic to this end in the coming weeks.
I suppose you have some more ideas on improving dvsim. It will be great if we could document what's being planned and when is it expected to land, so that we are not blindsided by each other's updates. Let me know how do you want to track it. Thanks!

Thanks for the review, Sri. About migrating from LSF: I'd really suggest doing something a bit like an LSF job array. So the scheduler just dispatches a list of jobs and waits until they've all finished. This can be piped into something like xargs to run locally, LSF if you have it (we don't), or presumably whatever GCP script you're going to use at Google. I think this is probably a nice simple interface that won't need too much magic glue code.

Thanks for the review, Sri. About migrating from LSF: I'd really suggest doing something a bit like an LSF job array. So the scheduler just dispatches a list of jobs and waits until they've all finished. This can be piped into something like xargs to run locally, LSF if you have it (we don't), or presumably whatever GCP script you're going to use at Google. I think this is probably a nice simple interface that won't need too much magic glue code.


Residents Progress

https://docs.google.com/document/d/1vg95Bw4Pzsqe4DXPusiyD2hAU-HCI-37c0AAZSXxC68/edit?disco=AAAAGf_2o-o
3 - exchange 10 transaction with resident/reputable org

Done