[AIRFLOW-1724] Add Fundera to Who Uses Airflow?

Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
I've been building & following a couple PR's, and have been helping out where I can, figured I should add my workplace to this list!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1724

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Added 'Fundera' to Who Uses Airflow README section

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
Readme update

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2694 into master will decrease coverage by 0.08%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2694      +/-   ##
==========================================
- Coverage   71.77%   71.68%   -0.09%
==========================================
  Files         154      154
  Lines       11814    11814
==========================================
- Hits         8479     8469      -10
- Misses       3335     3345      +10

Impacted Files
Coverage Δ

airflow/task_runner/bash_task_runner.py
93.33% <0%> (-6.67%)
:arrow_down:

airflow/utils/helpers.py
53.44% <0%> (-2.88%)
:arrow_down:

airflow/jobs.py
79.06% <0%> (-0.46%)
:arrow_down:

airflow/models.py
87.14% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Add a callback to bot.getMedia()

While I don't know about others, I use this API in combination with other Node packages. Trying to figure out how to use getMedia by itself is confusing enough, but trying to make it work with other things is a pain. Callbacks would help immensely.

Apologies for the late reply, time is not something I have atm due to school. This is the documentation for bot.getMedia(), but I can also add a callback if that's helpful to your usecase.

Can I enquire what your use case is for adding callbacks to this method? It seems rather unnecessary to me...

thedark1337 commented 3 days ago
Apologies for the late reply, time is not something I have atm due to school. This is the documentation for bot.getMedia(), but I can also add a callback if that's helpful to your usecase.

That'd be hecka helpful. Promises work too.

Hunchmun commented 3 days ago
Can I enquire what your use case is for adding callbacks to this method? It seems rather unnecessary to me...

Yeah, sure. I use this in conjunction with discord.js. I'm trying to make my Discord bot retrieve the current song from a room, and post it into a Discord server. So far, I've had no luck.  As for how to use getMedia it'd be something like
var media = bot.getMedia();
// use the media object now (such as for author, media.author, title is media.title. etc.

Example Media object:
{
    author: 'Freefire',
    cid: 'xxxxxxxx',
    duration: 333,
    format: 'x',
    id: 'xxxxxxxxxx',
    image: 'http://i.ytimg.com/vi/kT3vXUZ4TWc/default.jpg',
    title: 'Dataloss (Darth & Vader Remix)'
}

In addition to thedark1337's answer, can't this piece of code do what you want?
The following code will return the current song of your Plug.DJ account whenever a user will send "!song" in the Discord chat ( but you can trigger the action on others events like song change, skip, etc... ).
// Client = Discord.js bot // bot = PlugDJ.API bot

// create an event listener for messages on Discord.js library
client.on('message', message => {
  // if the message is !song,
  if (message.content === "!song") {
    // Retrieve song information from PlugDJ.API
    var media = bot.getMedia();
    // send the current song
    message.reply("The current song on Plug.DJ room is: " + media.title + " from " + media.author);
  }
});

@Talos51 I thought I had tried that, but it turns out to work.  Closing issue.

Thanks for your return, i'm glad that i was able to help you ^^ I let an admin close the ticket :)


[Feature] Add RabbitMQMessageQueue (RabbitMQ Integration)

This PR adds RabbitMQMessageQueue (naming might be redundant?)
Highlights:

Can use RabbitMQMessageQueue in place of SimpleMessageQueue in both LocalLauncher and ServerLauncher

The new examples/rabbitmq subdirectory contains installation and usage instructions as well as example scripts using both local and server launcher
Once this PR lands: will create a docker/k8s example using RabbitMQ

Uses the aio-pika client for RabbitMQ which is async first
Changes to BaseMessageQueue interface:

register_consumer() now returns a StartConsumingCallable
The consumer should invoke this function to start consuming from the queue

Looks pretty good to me!
Can you take a second pass through the README.md as well, and make sure nothing in there is outdated yet?

Absolutely! Thanks for the turn around time on the review!

Looks pretty good to me!
Can you take a second pass through the README.md as well, and make sure nothing in there is outdated yet?

Okay, I updated it. The main update was that mq.register_consumer() returns a StartConsumingCallable which is async and that needs to be called in order for the consumer to actually start consuming (well not really for SimpleMessageQueue since its a dummy callable thats returned, but we should keep the pattern consistent)


chore(web-modeler): update to version 0.5.1-beta

Which problem does the PR fix?
Updates Web Modeler to the latest beta version.
What's in this PR?
Version update to 0.5.1-beta.
Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] There is no other open pull request for the same update/change.
[x] The commits follow our Commit Guidelines.
[x] Tests for charts are added (if needed).
[x] The main Helm chart and sub-chart are updated (if needed).
[x] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[x] Did all checks/tests pass in the PR?

@CatalinaMoisuc no worries, all good :grin:


Create a nuget package to deploy the runner

Also, have it contain and msbuild props file that sets the StartupAction, etc for easier debugging.

Package created, thanks to @jmarolf, but we should add the targets file.

And done with the targets file in v1.0.14


[config] Log important config values on startup

We aren't logging the config because we print out the values before the
logger is set up.  Now, we should be able to add all values after
logging is set up that we care about.  The first value that we should
log is the chain id.

LGTM
do we have any plan to log the whole config? or do we want to just pick out the useful pieces individually?

The whole config is pretty large.  I think we can look at that next, but the most important piece was the chain id.

/land

Glad to at least get the chain id, but why not log the whole config btw?  Seems pretty useful.

Glad to at least get the chain id, but why not log the whole config btw? Seems pretty useful.

I figured it would be a massive log, and currently since we're dealing with some weird fallout from the logs, shorter is better at the moment.  But, I could log the entire thing

/stop

@aching @phlip9 Turned out to not be that big, so I've changed to just printing out the whole config (or at least what will serialize)
http://kibana.ct-1-k8s-testnet.aws.hlw3truzy4ls.com/app/kibana#/doc/9d3f1cd0-bcab-11ea-9157-7553013abcd5/kubernetes_cluster-2020.08.07?id=1cb819a6-5fa1-89f6-1b07-602d4a107595&_g=(filters:!(),time:(from:'2020-08-07T22:27:38Z',to:'2020-08-07T22:38:43Z'))

/land

:broken_heart: Test Failed - Build images and run cluster test

/land


Sort by column header in stats

I would love to be able to click on a column label to sort (Descending, ascending).

Definitely something on our radar, and shouldn't be too difficult. I don't know why you'd want to show an ascending view of any of the columns though? Is that significant in any way?

Sorting descending is now enabled for Confirmed, Deaths, and Mortality Rate for Global and US, and for Confirmed and Deaths at the US_Counties filter when in table view.


Areas are not all regular hexagons?

If I want to get a regular hexagon, is there any good way?

No, H3 does not support this. This is a result of H3's choice to use the gnomonic projection as well as other choices about how the cell subdivision process works.
If you did have regular hexagons I wouldn't expect them to appear as regular on a Mercator map because of distortion in that projection. Can you share what you would hope to gain with regular hexagons?

The cells are regular hexagons in hex2d space, which is the planar icosahedral gnomonic space scaled so that hex centers are one unit apart (see documentation). If you want regular hexagons that are all on a single icosahedron face I think you could get the cell boundaries in geographic coordinates and then use _geoToHex2d to transform the vertices to hex2d coordinates. If you want cells on multiple faces, you'd have to handle arranging the faces on the plane.

Thanks! I implemented an algorithm myself.  This is my solution:

Convert latitude and longitude to x and y coordinates . (https://wiki.openstreetmap.org/wiki/Mercator)
Calculate hexagon coordinates using mathematical equations
Conver x and y to latitude and longitude


Use replace directives in go.mod in development only

Problem
My go.mod file has:
replace github.com/channelmeter/send-to-rollbar-via-stdin => ../send-to-rollbar-via-stdin

this replace directive works in dev but not in prod. How can I ignore certain replace directives when running go install in prod? I want the replace directive to be ignored.
The error shows up as follows:

Step 27/43 : RUN go install -v  2>&1
 ---> Running in 1d1717578978
go<EMAIL_ADDRESS>parsing ../send-to-rollbar-via-stdin/go.mod: open /go/src/github.com/channelmeter/send-to-rollbar-via-stdin/go.mod: no such file or directory
The command '/bin/sh -c go install -v  2>&1' returned a non-zero code: 1

What version of Go are you using (go version)?

go version go1.14.4 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Ubuntu Linux 64 bit
go env Output
AR="ar"
CC="gcc"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_ENABLED="1"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
CXX="g++"
GCCGO="gccgo"
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/alex/.cache/go-build"
GOENV="/home/alex/.config/go/env"
GOEXE=""
GOFLAGS=""
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build016167139=/tmp/go-build -gno-record-gcc-switches"
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMOD="/home/alex/go/src/github.com/channelmeter/cp-go-api/go.mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/alex/go"
GOPRIVATE=""
GOPROXY="direct"
GOROOT="/home/alex/.local/go"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/home/alex/.local/go/pkg/tool/linux_amd64"
PKG_CONFIG="pkg-config"

What did you do?
I ran go install in prod, which breaks since prod fs in a docker container is not like my local fs.

I think this is entirely covered by https://github.com/golang/go/issues/26640.

Closing as a dup, since you gave a thumbs up.


Document how to use log crate with 2018 edition

The readme currently considers 2015 edition is used.
For the 2018 edition, the macro_use attribute and the extern crate declaration are superseded by a direct use of the macro (without a final question mark).
As such the doc should explain how to import the macros in such case.

Release a patch-version to update docs.rs?

@arve0 Yes, that's what 0da424552703 does.

Ah, I see. If you like, I can send a PR with updated docs.rs documentation?
Also, move order of 2015- / 2018-example? Or even remove 2015-edition?


Check a number is even or odd using if...else statement.java

NEED TO ADD THE HACKTOBERFEST TOPIC TO THE REPOSITORY, OR ADD THE
HACKTOBERFEST-ACCEPTED LABEL TO YOUR PR/MR.


Use modified knockd to set a cooldown on port 19132

Fixes #1969.
There are a couple details that may need to be addressed:

seq_cooldown can't be bigger than AUTOPAUSE_TIMEOUT_KN (see last commit). We might want to set a lower bound for both, or expose the cooldown timer as an environment variable.
I have no idea if the libpcap package has the same name for dnf and alpine, I know that apt has libpcap0.8 but the other scripts may have to be changed.

Yes, nothing to add if all looks good.


Add Kiali fix to Helm

This updates Kiali to use version 1.15 in helm and picks up changes from #22483 to Helm

@brian-avery: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

e2e-simpleTests_istio_release-1.4
242af067fafcf84ac92ac491e5f398b35a76bcb2
link
/test e2e-simpleTests_istio_release-1.4

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests_istio_release-1.4


style: Edge seems not supporting css shapes...

Describe the bug
Edge seems not supporting css shapes...
label: trivial
To Reproduce
Steps to reproduce the behavior:
open in edge https://cousquer.github.io/CardWebComponents/

Expected behavior

Remediation

give a try on vendor prefix
if it doesn't fix the problem small svg only for ie/edge... 😞

Desktop (please complete the following information):

OS: [windows 10]
Browser [edge]
Version [all]

Additional context
Add any other context about the problem here.

this can work for edge:
SVG clip-path for SVG elements
Specification

<clipPath id="clipping">
  <polygon points="98.4999978 153.75..."/>
</clipPath>
.item {
  clip-path: url(#clipping);
  }

/* Microsoft Edge Browser 12+ (All) - @supports method */

@supports (-ms-ime-align:auto) {
  .selector { property:value; }
}

resolved in #41


Backspace no longer works when typing Watchpoint

Hello! After updating to 0.2.4 I'm not longer able to backspace when typing in a Watchpoint. No error or anything. It simply doesn't do anything.

I am experiencing this same issue.  Also, arrow keys, delete button don't seem to work in the Watchpoints field either.  The work around is to right click and use the context menu which seems to be operational.  I noticed that the home and end buttons work.

Same issue,
Also can't cut and past with keyboard shortcuts.

I have same issue, when type some mistake character, don't work delete, backspace, and arrow keys left and right. I press Enter and delete mistake typing.

Facing the same problem. I have tried to disable all other plugins but it doesn't help.

This should be fixed in 0.2.5.


Remove plugin that was added by the test

necessary to remove artifacts for configurable cluster testing

Testing: ran plugin tests

@rschlussel

Did you run tests on the plugin/uninstall tests on a real cluster?

lgtm assuming testing

Yeah, it works on a real cluster


use import syntax to import Inferno namespace

Currently ts-transform-inferno always import Inferno namespace by using commonjs syntax (whatever user set the module in tsconfig.json). This will break in Rollup (Rollup enforce using esm syntax).
I've test usage in webpack and fusebox, both of them works correctly.
And this PR also fix issue #1

I haven't gotten this to work before but it seems there are no more webpack and fuse-box issues according to you? I'll just be releasing a major version, so nothing will break.
Thank you for this PR!


creation of theme presets

creation of a theme preset that people can use as a starting point for building their own theme.

I'm working on it


Document our solution for creating a ChangeLog

What
We need to document our solution for producing a ChangeLog for each release. Ideally, this solution should not require more than a few button clicks.

This is completed by #803 - we will not be using a ChangeLog for now, but will instead publish release notes (based on PR titles) automatically as part of the release action.


Problem loading extension: no module named nbformat

CRITICAL:root:Problem loading extensions from package wlauto.result_processors: No module named nbformat
Reverting 6266edad6f379446776a26af928868e92a86381d fixes the issue for me.

This error should be fixed by https://github.com/ARM-software/workload-automation/commit/b312bf317da051b1bcb601d3d185a76652e5f884. Please let me know if this resolved your issue.

Yes, this fixes the issue for me! :+1:


Release workflow without pushing directly to main

Simon brought this up after I pushed to main by mistake ;) It's probably a good idea.

It would also be good if Write permission was sufficient to perform a release.
I think something like this world work:

Create a release branch for a PR, e.g. release-0.16.0-rc3
Execute 1. and 2. from https://github.com/CosmWasm/cosmwasm/wiki/Releasing
Have a dry publish step for crates.io
Adapt CHANGELOG for the new version
Review and merge the PR
Tag the merge commit on main (this can be done by anyone with write access)
Publish the tag/merge commit to crates.io

We currently use cargo-release, but the PR workflow seems a bit nicer. We should investigate switching to release-plz instead, which also would align us with sylvia and storey.


[SUPPORT]used in test inveroment

Tips before filing an issue

Have you gone through our FAQs?

Join the mailing list to engage in conversations and get faster support at<EMAIL_ADDRESS>

If you have triaged this as a bug, then file an issue directly.

Describe the problem you faced
A clear and concise description of the problem.
To Reproduce
Steps to reproduce the behavior:

Expected behavior
A clear and concise description of what you expected to happen.
Environment Description

Hudi version :

Spark version :

Hive version :

Hadoop version :

Storage (HDFS/S3/GCS..) :

Running on Docker? (yes/no) :

Additional context
Add any other context about the problem here.
Stacktrace
Add the stacktrace of the error.

@c-f-cooper would you please provide more info?

Closing this due to inactivity.


Question: release schedule

tl;dr is there a release schedule or somewhere that the expected next release date is available?
We are looking to upgrade the version of pdf.js we use in our web application but thought it would be wise to wait if the next stable release is imminent. The wiki doesn't mention mention a release schedule but it looks like you have released at the end of October in 2017 and 2018. Is there a release schedule (aspirational or otherwise)?

[...] but it looks like you have released at the end of October in 2017 and 2018.

At the time of writing, there's been no less than three release in 2019 already (with the latest one being less than a month old); please see https://github.com/mozilla/pdf.js/releases
Also, please note that there's generally very little difference, as far as stability is concerned, between stable and pre-release versions (since no code lands without e.g. all tests passing).
Firefox Nightly, for example, is always using what's more-or-less the latest master version of the PDF.js library. Hence the decision to use "stable" or "pre-release" versions is really up each implementer (and note that unless something is really broken, patches are generally not back-ported).

Is there a release schedule (aspirational or otherwise)?

No regularly occurring one, as far as I'm aware. Since a lot of this project is driven by volunteers, releases usually happen when someone has time to create one (after e.g. a couple of [api-minor] patches and/or other significant changes has happened since the last release).

Closing since the comment above is accurate.


Generate cover image after selecting video[🌟]

Description
I hopen get cover image after selecting video.

https://www.npmjs.com/package/react-native-create-thumbnail


[BACKLOG-31765] Folder tree right-click style update

:gear: Build finished in 1m 37s with command:
mvn clean verify -B -e -Daether.connector.resumeDownloads=false -amd \
-Daudit -pl plugins/file-open-save-new/core

:ok_hand: All tests passed!

Total
Failed
Skipped

8
0
0

Details

:information_source:  This is an automatic message

@ddiroma

:gear: Build finished in 1m 24s with command:
mvn clean verify -B -e -Daether.connector.resumeDownloads=false -amd \
-Daudit -pl plugins/file-open-save-new/core

:ok_hand: All tests passed!

Total
Failed
Skipped

8
0
0

Details

:information_source:  This is an automatic message


main.c changes

read_file() exited without closing the file. Now it does, regardless the error status.
Slightly simplified file_exists().

Why do you think that transfer defines from main.c is a good practice?

I see httpd.c mainly as a library it could be used to easily create REST servers in C (which is my intent here). If that's the case, all the defines and declarations should be in the http.h header so that one can link httpd.c to different "main" for different purpose.

I'll close this pull request and open a new one.

@rdentato you don't need to create a new pull request every time! make changes within the same branch!


[ISSUE - 133] Enhance Search Box in the Navbar

Description👨💻
Enhanced the Search Box to match the theme of the website and also made some major bug fixes in the navbar.

The Changes made are Responsive for smaller screens as well.

Fixes Issue🛠

Closes #133

Type of change📄

[x] UI Enhancement

How this has been tested✅
I have done Alpha, Beta & Unit Testing
Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I am an Open Source Contributor

Screenshots/GIF📷

@Spandan-Bhattacharya This pull request is ready for review and merge. Additionally, I kindly request you to consider adding the JWOC and if possible, a HARD Difficulty label to this pull request as I have also fixed some major UI bugs in navbar to enhance its appearance.
Thankyou for the opportunity to contribute to this project.

Looks really good!
Just one thing,

When the search bar has something written in it, and then it is closed, the dropdown should collapse.

Hey @Spandan-Bhattacharya, I've resolved the issue. Could you kindly review the Pull Request and share your feedback?

Great Job!!

Thankyou 😄


Try to force Node version for old github-script action

📚 Documentation preview (see all builds) 📚

Demonstrates the problem, merging.


[1.27] google_grpc: add a runtime flag to disable TLSv1.3 (#32532)

Backport of https://github.com/envoyproxy/envoy/pull/32315

/retest

/retest


Add OSX builds on Travis-CI

This adds OS X builds on Travis-CI.
It takes way too long to install dependencies but I'm not sure what can be done about it...

Awesome! And I don't think the time issue is too big of a deal.
One small recommendation, could you change the name of the install script from before_install.sh to something like .travis_install.sh? I wouldn't want end-users thinking they have to run this script in order to install Boost.Compute itself.
And the test errors are something we'll have to fix before actually merging this (unless there is some way to temporarily disable testing on just OS X). The core issue is that on CPU-like devices on Apple platforms, any use of __local memory will cause the kernel to report that it only supports a maximum of 1 work-item and thus the clEnqueueNDRangeKernel() fails when we use it with algorithms designed for multiple work-items. I think the fix would to be to have separate implementations of these core algorithms (like copy(), sort(), and reduce()) which will dispatch to non-local-memory based algorithms by checking whether or not the device actually supports __local memory (which can be checked with device.get_info<CL_DEVICE_LOCAL_MEM_TYPE>() == CL_LOCAL).

We should test it again.

This can be closed.


Commented out code

private strings _Name and _Description also appear unused, as well as _TableConstant
https://github.com/HydrologicEngineeringCenter/HEC-FDA/blob/0c0e82868f7b7f9aa0a6e7aeb857b48f4f6dcad0/ViewModel/WaterSurfaceElevation/WaterSurfaceElevationElement.cs#L26

Good find, those are no longer needed after recent changes. I deleted them.


Polymorphism support for kotlinx.serialization

Describe the feature request
Given a polymorphic hierarchy like this:
@Serializable
sealed interface Event

@Serializable
@SerialName("PostAdded")
data class PostAdded(val message: String) : Event

@Serializable
@SerialName("PostDeleted")
data class PostDeleted(val id: Int): Event

I would expect the generated schema to contain a oneOf: PostAdded, PostDeleted for Event, along with a discriminator property, such as it does for Jackson.
Motivation
We have several events that used sealed interface and polymorphism. This maps cleanly to oneOf in OpenAPI and AsyncAPI, and should be supported.

Hey @geirsagberg thanks for the report.
The support for Kotlin Serialization is still in Beta. PRs are welcome, but that's something that was already in my TODO

Hey @geirsagberg,
I did a quick implementation that seems to work. Can you double-check that's properly working in your use case?

Nice, code seems to do the job far as I can tell, however we went for gRPC and Armeria for docs in the end, instead of RabbitMQ, so I can't test it unfortunately. But kudos for the quick support, I'm sure I won't be the last seeking this 😊


No device region option for CN470 and corresponding config file

Thanks for your great job.
@.***
在 2022年8月31日 +0800 16:49，Orne Brocaar @.***>，写道：

Closed #36 as completed via 0fa4071.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Keyboard shortcuts wonky

I've upgraded my machine to 14.3 and see that I'm also on 0.8.1 now and some things have changed.
I have my shortcuts setup as:
ctrl-alt-left = 'focus left'
ctrl-alt-down = 'focus down'
ctrl-alt-up  = 'focus up'
ctrl-alt-right = 'focus right'

alt-cmd-left = 'join-with left'
alt-cmd-down = 'join-with down'
alt-cmd-up  = 'join-with up'
alt-cmd-right = 'join-with right'

ctrl-cmd-alt-left = 'move left'
ctrl-cmd-alt-down = 'move down'
ctrl-cmd-alt-up = 'move up'
ctrl-cmd-alt-right = 'move right'

And as far as I can tell the join and move commands are working but for some reason the focus command just does not arrive and I can't rotate through windows anymore.
Did something change that could explain that? I ran ShortcutDetective which seems to pick up Aerospace of the second two sets, but not for the ctrl-alt shortcuts.

It seems my ctrl-cmd-alt-right command is also misbehaving because if I give the command through the terminal, the thing I expect to happen works perfectly.
Is there a way for me to enable verbose logging on aerospace to see what commands it is getting in?

OK. I think something is squatting on these combinations.
I remapped the actions to ctrl-alt-{1,2,3,4} and then they do work.
Not an Aerospace issue but still very annoying.

OK. Figured out what it was.
The reboot fired up skhd which was setup to catch many of the same shortcuts and was still lingering on my machine after trying out Yabai.

For anyone else ending up here, I had ctrl-(arrows) bound to focus (arrows) and they didn't work. They're actually bound by default in a fresh macOS installation for some Mission Control actions, which takes precedence over AeroSpace's hotkey handling.
Here's the delicious copy/pasta for dotfiles or if you just want to fix it via the CLI:
# Disable Ctrl + Left Arrow (Mission Control: Move left a space)
defaults write com.apple.symbolichotkeys AppleSymbolicHotKeys -dict-add 79 "<dict><key>enabled</key><false/></dict>"
# Disable Ctrl + Right Arrow (Mission Control: Move right a space)
defaults write com.apple.symbolichotkeys AppleSymbolicHotKeys -dict-add 80 "<dict><key>enabled</key><false/></dict>"
# Disable Ctrl + Up Arrow (Mission Control: Mission Control)
defaults write com.apple.symbolichotkeys AppleSymbolicHotKeys -dict-add 32 "<dict><key>enabled</key><false/></dict>"
# Disable Ctrl + Down Arrow (Mission Control: Application windows)
defaults write com.apple.symbolichotkeys AppleSymbolicHotKeys -dict-add 33 "<dict><key>enabled</key><false/></dict>"


[Make compatible with v0.47.1 of react-native] Remove redundant #createJSModules methods

Build fails if the #createJSModules method is implemented as has been removed upstream.
Didn't find any other blockers.

Please merge this with main branch if there are no issues with it. Android build fails because of @override notations otherwise.

Duplicates PRs #199 and #206.


Composer version inside .dep folder should override global composer command

Hello, I think that in the bin/composer set, the composer version inside the .dep folder should override the global version.
In this way you could have a "global" composer in the system but specify a composer version for a single project:
set('bin/composer', function () {
    // ===> this should be before the next check
    if (test('[ -f {{deploy_path}}/.dep/composer.phar ]')) {
        return '{{bin/php}} {{deploy_path}}/.dep/composer.phar';
    }
    // <===

    if (commandExist('composer')) {
        return '{{bin/php}} ' . locateBinaryPath('composer');
    }

    warning("Composer binary wasn't found. Installing latest composer to \"{{deploy_path}}/.dep/composer.phar\".");
    run("cd {{deploy_path}} && curl -sS https://getcomposer.org/installer | {{bin/php}}");
    run('mv {{deploy_path}}/composer.phar {{deploy_path}}/.dep/composer.phar');
    return '{{bin/php}} {{deploy_path}}/.dep/composer.phar';
});

Now as now it's impossibile to specify a composer version if you already have a composer version system-wide.
Thanks.

I agree. Can you create pr?

I agree. Can you create pr?

Yes, of course!
https://github.com/deployphp/deployer/pull/2371

Yes, of course!
https://github.com/deployphp/deployer/pull/2371


How to create task's arguments autocompletion?

I have found no answer to the question in title in documentation nor GitHub issues nor PRs.
Is it possible to do something like that at all and if yes, could you please point me to documentation, source code or anything which can explain how to archive task arguments completion?
Below is a long explanation with example of what I would like to be able to do.
Let's say I created a task in my justfile with name task-a which accepts one argument with values x, y, z.
At the moment even if I setup auto completions for just (i.e. with source <(~/.cargo/bin/just --completions bash) command in ~/.bashrc) I wasn't able to type just task-a  press  and get auto completion suggestions for  x, y, z. Or when I type just task-a w where w if first letter of x value and then press  I would like to get completion to the full value of x.
Another example of this could be when I have task-b which accepts one argument which is a list (or filtered list) of files in a specific project directory. Then typing just task-b some-fi and then  I would get a completion to just task-b some-file-name where some-file-name is a name of the file existing somewhere and implementation of task-b define where to find that file. In this case I don't assume just be smart enough to understand implementation of task-b I just want to explain (i.e. provide a path or mask to files) which can be used as potential variants for auto completion.
And thank you for the nice task runner! I really enjoy using it and extend my set of tasks almost every day!

I was looking for something similar recently, did you found anything till now?

No, I am still waiting for answers preferably from maintainers of the project. Meanwhile I started to look how it can be done. I think if there is nothing out there I will try to implement it and submit PR. But definitely it'll take a bit of time, but I hope to be able to do it during next month of so.

@Animeshz You can also upvote my question on StackOverflow may be somebody notice it there.

This could be implemented in just itself if the arguments were typed (debate-able if that should be in scope though).

It would definitely be nice to improve autocompletion here, probably with type hints which say things like "this argument is a path". Improving autocompletion probably won't happen until clap adds support for dynamic auto-completion which invokes just itself to figure out what the completions are. Our current approach of using generated autocomplete scripts which we then patch is too fragile to add new features to.

@casey What if we can generate scripts to call back to just with parameters for auto-completion? WDYT? Would you accept such PR or a prototype to further deeper understanding the feature and its implementation path?

I'm really hesitant to add explicit support for this to just. My experience with completion scripts has been that they're hard to write, hard to test, and break mysteriously, so I'd really like to avoid doing it ourselves and instead would like to lean on clap.


Adds attributes to startSpan

Description
[Describe what this change achieves]
Add attributes to the start span APIs. This is needed for attribute based decision making in Samplers.
Related Issues
Resolves #9197

Check List

[X] New functionality includes testing.

[X] All tests pass

[X] New functionality has been documented.

[X] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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

@reta Please take a look. Thanks!

@Gaganjuneja why do we need this change? Span has addAttribute(String key, String value) that could be used at any time.

Actually, addAttribute works post span creation. But for sampling if we want to take a sampling decision based  on attributes like trace=true then it has to be passed as an attribute while span creation.

Actually, addAttribute works post span creation. But for sampling if we want to take a sampling decision based on attributes like trace=true then it has to be passed as an attribute while span creation.

We should not based that on attributes, the Otel has dedicate concept called TraceFlags [1]:

TraceFlags contain details about the trace. Unlike TraceState values, TraceFlags are present in all traces. The current version of the specification only supports a single flag called sampled.

[1] https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/api.md

Yes, attributes would help in taking the sampling decision and that will be communicated through traceFlags to all the child spans.
https://github.com/open-telemetry/opentelemetry-java/blob/f52050bc2d3867df3e5005b579bddec2ee194723/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/SdkSpanBuilder.java#L201C59-L201C59
We need to implement the shouldSample method for custom samplers.
https://github.com/open-telemetry/opentelemetry-java/blob/f52050bc2d3867df3e5005b579bddec2ee194723/sdk/trace/src/main/java/io/opentelemetry/sdk/trace/samplers/Sampler.java#L105

Could you please resolve the conflicts as well? I think we are getting there :)

@Gaganjuneja heh, could you please send manual backport to 2.x? thank you


Consider switching to bitcoin based version

For example when bitcoin core 24.0 come out, add support for it and crate version could become 24.0-x.y

This seems like a good idea! The current setup is somewhat tricky to use, since you have to remember to specify a feature. So just specifying one version for the crate and getting the bitcoind version you want would be easier.

What you describe is not exactly what I had in mind. Features will stay and there will be no default feature enabled because by default I want to support setups where bitcoind executable is already in PATH or where BITCOIND_EXE env var is specified.
What I was considering with this issue is that bitcoind crate version 24.0-x.y is the first version supporting the feature 24_0

Ah, gotcha, makes sense. Would it make sense to split bitcoind into two crates, one that downloads bitcoind, and one that runs it? They seem like two distinct pieces of functionality. Then, the version of bitcoind that download the crates could use the same versioning scheme as bitcoind itself.

Then, the version of bitcoind that download the crates could use the same versioning scheme as bitcoind itself.

But do you imagine crate bitcoind_download version 23.0 can only download that version?
It's useful to test against the older versions

But do you imagine crate bitcoind_download version 23.0 can only download that version?
It's useful to test against the older versions

You could use a previous version of bitcoin_download to test older versions. And you could also use multiple versions to test different versions at the same time. I don't think that's possible with features.

Or, wait, I think it is, you would just have two dependencies with different names and with different features turned on.


Address external debug issue #11

Created PR from the discussion in issue #11

PTAL @ved-rivos @gokhankaplayan


Example on how to provide existing DKIM signature via Helm

Hi,
I'm trying to make it work that I can provide existing DKIM signatures for docker-postfix via the Helm Chart, but can't get it running.
Could you provide an example?
Do I have to specify the mountSecret?

You can use extraVolumeMounts to mount /etc/opendkim/keys.
See test_04_common_config.yml.


VirtualScroll Scrolling / display issues when filtering list

Ionic version:  (check one with "x")
[ ] 1.x
[X ] 2.x
I'm submitting a ...  (check one with "x")
[X ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

When a list is filtered by a search query, if you are at the top of the list when filtering all works fine and results update.  If you are part way down the list and the length of results returned by the filter is LESS than you would need to get to your scroll position BEFORE filtering you get a blank screen.  You can still scroll up and down but all you will see is white, scroll events don't fire from ion-content.  It's like the content area has been lost off the top of the screen somewhere.
If you then clear the search filter so the list goes back to it's original length (ie long enough to reach your original scroll position) scroll up and down a bit and all the items reappear and scroll events start firing again as usual.
Only happens with virtualScroll, ngFor lists behave as expected under the same test.
Expected behavior:

When you filter the list it updates and shows the filtered results correctly with the scroll position at the top of the filtered list.
Steps to reproduce:
Create a list of say 200 items so you have a decent amount of length.
Add an ion-search box and a basic search to filter the list as per the standard ionic component docs.
Scroll halfway down the list
Perform a search which will return less items than it takes to reach your previous position in the list (ie scroll down 50 items and perform a search that returns 5 items).
The screen will go white with no items shown, scroll events stop firing.
On iOS UIWebview the navbar will scroll off the top of the screen as if it is no longer fixed.  Basically it looks like ion-content has moved off the top of the view port somewhere and you can't get back to it until you restore the list of data to be longer than needed to reach your current scroll position.
I see the same symptoms on BOTH Android and iOS UIWebview.
Does not reproduce on plunker but here is a plunker setup with a simple search demo with virtual scroll, straight from the ionic demo's page but with a bit longer list of data and virtualScroll instead of ngfor.  Just copy and paste this into a new ionic app and you can reproduce the same behaviour on both iOS and Android.
http://plnkr.co/edit/OPYLBepHp000CDCOVCL6?p=preview
Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.4
Ionic CLI Version: 2.1.17
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.48
ios-deploy version: 1.9.0
ios-sim version: 5.0.13
OS: OS X El Capitan
Node Version: v6.7.0
Xcode version: Xcode 8.0 Build version 8A218a

Do you have a plunker?

@mpaland @brandyscarney - updated with ionic info and a plunker.  plunker does not reproduce the issue but you can start a new project and just copy and paste the template and ts into it and run on iOS emulator or an ios or android device and it will reproduce by simply scrolling down a few pages in the list and searching for "test" (which only returns 2 items so not enough to reach the pre-search scroll location).

+1 - Can verify that this happens.

Hi there.. having same issue on the tabs .. when i scroll down shows the rest content is blank
I have a slightly different ionic version..
Cordova CLI: 6.5.0
Ionic Framework Version: 3.0.1
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.1.0
ios-deploy version: 1.9.1
ios-sim version: 3.1.1
OS: macOS Sierra
Node Version: v6.3.1
Xcode version: Xcode 8.3.2 Build version 8E2002

Hi there!! Could figure it out.. my case was.. i had a order.html with tabs.. and a orderview.html with info that will be inside a tab.. in orderview.html i had just the html.. i wrapped with <ion-content> and the scroll is working withou the blank after scroll

@mariohmol this isn't the same issue I am talking about, I have ion-content everywhere it needs to be.  This relates specifically to virtual-scroll list behaviour when the list is filtered with a search query.

This behaviour is now slightly different.
Follow the same steps to reproduce though.
Create a virtualscroll list with a few hundred items
scroll down about 50 items
perform a search/filter on the list which will return 2 items
the screen goes blank.
the list does filter correctly but the returned 2 items are too far off the top of the screen and you cannot scroll to them.
if you then clear the search term to return to the unfiltered list the original virtualscroll list reappears with the scroll location the same as it was prior to searching/filtering.
The header no longer scrolls off the page and remains correctly fixed now.

I can confirm this issue. Is there any workaround?
In case not, can somebody point me to the part of the code that is causing this? I'd then try to fix it.

This issue still exists in Ionic version 3.6.0. The app also crashes which needs to be restarted. My app is run and tested on an iPhone 6S.

Unfortunately, this still happens with nightly 3.7.1-20171010. Is there any glimpse of hope? ;-)

I think this is fixed by #12917


Web API default interval is actually 0

The documentation states that the default interval for the web API is 30 seconds.
That seemed fine to me so I didn't add it to my configuration.
It turned out that when omitting the Web API interval in the config, the resulting interval is 0.
This causes some endless polling without a break.
Either the documentation should be adjusted or there's an issue with the handling of the default value of that property.

Ahh, good catch. I'm currently working on a new release, so I'll add in a quick fix for this. Thanks for reporting it!


Make cfg_if uses more explicit and consistent

This commit changes most uses of cfg_if as follows:

fallthrough else usage is avoided for architecture or OS specific
items
a comment is added in the final else clause to signal intent someone
modifying

It is safer to omit items than include ones for the wrong platform or
architecture.

Let's see if this works...
@bors: r+ c9dd87c7974e89bd7acf024dce3a1c9b9e843f7e

:hourglass: Testing commit c9dd87c with merge eb11bc1...

@bors: r-
gonna run some testing first...

@bors yay!

er still working on this, I'm gonna close this to see if I can make some progress (I think https://github.com/rust-lang/libc/pull/221 needs to land first), but I'll reopen once I figure bors out.

@bors: r- clean force

@bors: retry force clean

Sorry for the extra torrent of notifications @kamalmarhubi, but would you be ok resubmitting this? This PR seems to have confused @bors beyond belief...

Reopened as #224
Poor @bors!


updateDataSet with animate

Hello
tmpAdapter.updateDataSet(dataset, true) --> if animate parameter is true then updateDataSet function work only for the first time, when I called this function next times then only headers change its data but items none. The problem is only with animate true parameter.
greetings

@kasterlod, I can't reproduce it.
In which context you use it, with expandables or simple headers?
Try to enable logs before creating the Adapter (mAdapter.enableLogs(true)) and check what the adapter is doing. In general, the animation depends by some factors (all related each others):

most important is the difference of the new list
stableIds (should trigger a change for the items with same id),
the flag to animate
and of course the ItemAnimator linked to the RV or to the single ViewHolder as I just implemented.

However, if you see in the logs single notifications (or executing of notifications) then items should be animated with the item animator you set initially (for an insert/remove).
Did you play a bit with "Async Filter" and "Animations" use cases in the demoApp?

I remove headers from list --> sort list --> add new headers --> updateData
ids are indiviudal for all items
itemAnimator set as default
What does mean "the flag to animate"?
for the first time - headers and data updated:
D/FilterAsyncTask: doInBackground - started UPDATE
V/FlexibleAdapter: Animate changes! oldSize=14 newSize=14 limit=600
V/FlexibleAdapter: calculateRemovals total out=0
V/FlexibleAdapter: calculateAdditions total new=0
D/FilterAsyncTask: doInBackground - ended UPDATE
for the rest times - header updated but data no
D/FilterAsyncTask: doInBackground - started UPDATE
V/FlexibleAdapter: Animate changes! oldSize=14 newSize=14 limit=600
V/FlexibleAdapter: calculateRemovals total out=7
V/FlexibleAdapter: calculateAdditions total new=7
D/FilterAsyncTask: doInBackground - ended UPDATE
greetings

@kasterlod, the flag is the boolean for the method. I see that is true.
I understood the list is made of ISectionable + IHeader items. Some questions:

Why you remove headers, and how you do it?
What is the list of new items you pass? what contains?
From the 2nd refresh, I see that 7 items are removed and added, are they the headers?
Also did you initialized the Adapter with stableIds?

I remove headers because i want to sort items and after that headers should change position too. Everytime I add headers I create new headers and adding to flexibleItems list (7 in that example). I think now to try not creating new headers each time but add created before.

I don't know what you want to achieve. Keep in mind that you should have an original source of items which you manipulate first, then use the Adapter to display items. Sorting that list, should not be done through the adapter, it just receives the new list with updateDataSet(list, true).
As always, the new list should contains only the ISectionable items, that are the real data. Headers are just sections to organize the data.
That's why I asked, is the list different than the one currently displayed?
If the items are shuffled, they should be animated thanks to the single notifications. I can try to add this test case in the demo App, by tomorrow evening, but it's identical to the Filter use case,  They both use the same background "engine" to notify the modifications.

@kasterlod, are you still having problem with animation? could be related to the new issue #210?


2048 game

I want to add a 2048 game made using HTML, CSS and JavaScript

@Drishti-jain21 Assign to you


consolidate logger logic to shared

to: @chunyong-lin
cc: @airbnb/streamalert-maintainers
size: medium
Background
I'm going to be adding another Lambda function soon, and it pains me to copy/pasta the old logger creation again.
Changes
I've removed how loggers were previously created and centralized in the shared module.

Adding central/shared logger.py along with tests
Updating all modules to use shared logger logic
Updating apps lambda packaging logic to include shared module

Testing
Updating all unit tests

Coverage increased (+0.4%) to 97.31% when pulling 01e37c73a3c801e70b98ed2ac55bbe27501d44cf on ryandeivert-consolidate-logger into 777f423eb4bc00e15eef46acbd77e771501a8c98 on master.


feat: update capi and kube version in Tiltile

What this PR does / why we need it:
simple version bumps for dev environment
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
No open issues for this

No real tests to run outside of make tilt-up is launching the local management cluster.


Roll engine 9daf5c8d213a..cc9c6702da5b (3 commits)

https://github.com/flutter/engine/compare/9daf5c8d213a...cc9c6702da5b
git log 9daf5c8d213a..cc9c6702da5b --no-merges --oneline
cc9c6702d Only reject gestures to embedded UIViews when the framework says so. (flutter/engine#7307)
2fb2b2729 Roll src/third_party/skia 3605fde88b2a..f6e963ef91c2 (1 commits) (flutter/engine#7306)
220ea0e1e Roll version of Dart to d1817ddc91fd3aea061647b2e21860c47a5a5180 (flutter/engine#7286)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (liyuqian@google.com), and stop
the roller if necessary.

@liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/4798048179847168 , https://cirrus-ci.com/task/4938785668202496 , https://cirrus-ci.com/task/6064685575045120 , https://cirrus-ci.com/task/5501735621623808

Commit queue failed; closing this roll.


consider adding this website that helps manage microgrants for students

I built a website called Atila that makes it easy for people to manage microgrants for students.
So essentially Atila handles all the logistical stuff like helping the sponsor find students to apply, ranking and scoring applications, messaging finalists and winners, handling the payment etc.
This allows the sponsor to just focus on funding the microgrant, reviewing the applications and picking the winner.
Perhaps you could add my website to this repo as a resource for people who want to start a microgrant but need a tool to help them manage the logistics and they don't have to worry about all the admin work.
I call it a scholarship but actually I just realized that a scholarship is basically a microgrant for students.
For example, here is a microgrant we have for any student working on a side project.

Changed the title to be more polite 😄

thanks! this is cool, but doesn't quite fit the list of other microgrants.


Use different settings for windows checkout

Motivation
This hopefully makes the windows checkout more stable.
Description
I increased the http.postBuffer size, core.compression and pack.window with default values take much longer to checkout so I kept these values.
Test Plan
Checkout on windows works
TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed
[x] Update changelog as needed

bors merge

bors merge

bors merge

bors merge

PR seems to be stuck and not being picked up by bors, superceeded by https://github.com/spacemeshos/go-spacemesh/pull/6302


FN entity clustering - Rugged art

In the recent rugged art attack
The address was funded through TC and then transferred to another address that eventually deployed the malicious smart contract.
Funded address: https://etherscan.io/address/0x575d4871cf7e9214fb705a440f1ea99624690fb3#internaltx
Malicious address that deployed the contract: https://etherscan.io/address/0x9733303117504c146A4E22261F2685ddb79780eF
I am seeing that only the beta one alerted, here the alert - While checking the metadata I am not seeing any alert referring to funding although there are funding alerts
My question is: does the entity clustering logic working properly?

The issue was that it wasn't the first but the second tx of the funded address and that's why it wasn't considered a "new account".
I have adjusted the threshold in this PR to permit one additional transaction while still recognizing it as new.
In the future, we could update the threshold even more if necessary.


[openssl-unix, openssl-uwp, openssl-windows] Change version to deprecated

Describe the pull request

What does your PR fix?
Some people are confused that those ports have version 1.1.1h and think that vcpkg still supports openssl 1.1.1 on master. See for example #27275. The version deprecated expresses that the ports themselves are deprecated and not upstream's openssl 1.1.1.

People that still need openssl 1.1.1 should consider migrating their software to support openssl 3, use the versioning feature to pin an old version of openssl or create an overlay port to update to a newer version of openssl 1.1.1.

@FrankXie05 Maybe the vcpkg website should be regenerated if this PR gets merged?

@Thomas1664 LGTM!

Thanks!


Breadcrumb on windows filesystem

Hello,
When using the Filemanager on windows machine/hosting, The breadcrumb paths are absolute due to the different directory separator:

Thanks.

Merged!


There is a problem with one or more of your inputs, please check and try again

Hello.What happen wirh my wallet GPcjmzVyVwwPWykniVAMw6jQrc3v7GsTPh  ???
This is BTG address!I generated him yestarday and use for mine BTG.!
Today i try sign and cannot login.Why?Wallet not work or what?

NACK
This site is not for Bitcoin Gold. This should not be merged.


PageAndListRowFragment SettingsFragment - multirows for one header

the scroll doesn’t follow the current focused item inside a SettingsFragment  (if there are more than 3 rows)

This sample has been migrated to a new location and updates are happening there (check README for more information).
As recommended by GitHub, we are closing all issues and pull requests now that this older repo will be archived.
If you still see this issue in the updated repo, please reopen the issue/PR there. Thank you!


Fix package

[x] fix typo in package name
[x] add missing composer description
[x] move phpunit to require-dev
[x] apply some phpcs
[x] remove composer.lock as it should not be committed for library
[ ] fix travis

The BlobRestProxy does not keep the AccountName in the path when using the emulated storage. See:
https://github.com/Azure/azure-storage-php/blob/89e7bcd510c28027c6d8c692c3b90b60d8184531/azure-storage-blob/src/Blob/BlobRestProxy.php#L371-L384

It needs this https://github.com/Azure/azure-storage-php/pull/131 to be merged and released for the tests to pass.

I can confirm that with the fix I made on the storage SDK the tests pass with the emulated storage:
$ vendor/bin/phpunit
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.2.3 with Xdebug 2.6.0
Configuration: /Users/florent/workspace/flysystem-azure-blob-storage/phpunit.xml

.....                                                               5 / 5 (100%)

Time: 950 ms, Memory: 10.00MB

OK (5 tests, 14 assertions)

Generating code coverage report in Clover XML format ... done

Code Coverage Report:
  2018-03-21 09:48:42

 Summary:
  Classes:  0.00% (0/1)
  Methods: 27.27% (6/22)
  Lines:   70.41% (69/98)

\League\Flysystem\AzureBlobStorage::AzureBlobStorageAdapter
  Methods:  27.27% ( 6/22)   Lines:  70.41% ( 69/ 98)

Hello there, any news about this package? Everything has been fixed in the dependencies and it works well.

Hi @luxifer instead of the docker setup this package is going to be tested against the actual API. The coding style you've applied is not the coding style of this package. There are a few exception like spacer before !. I prefer not to have an external .sh file for travis setup because I find it less clear. The thing that indeed needs fixing is the lock exclusion and the require-dev. Could you maybe open a new PR for that instead?

I just see that you addressed the test problem with the 1.1 package for the azure SDK. So I think this PR is not needed anymore. I'm closing it. Have you planned to publish this package on packagist? I'm currently relying on my fork. Regards.


Download regrads

How cam i download this project?

We do not publish any artifacts from this repository and everything is built from source. Let me know if that didn't answer your question.


Is this repository abandoned like SimpleWebRTC?

Is this repository abandoned like SimpleWebRTC?

@lawlietmester not abandoned. I still use this and plan to continue using it in production. It's used as part of the broader otalk group of modules. Let me know if you need any particular help or have any issues. I'm going to close this just so it's not an open issue, but feel free to continue discussion here or open a new issue for something specific.


setup in branch:problems with cuda

Hi,
after switching to Ubuntu 18.4 and following the steps from the branch (https://github.com/ethnhe/PVN3D/tree/pytorch-1.5) I get the following Error after running python3 setup.py build_ext:
THCudaCheck FAIL file=/pytorch/aten/src/THC/THCGeneral.cpp line=47 error=100 : no CUDA-capable device is detected
Traceback (most recent call last):
  File "setup.py", line 30, in <module>
    'build_ext': BuildExtension
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/setuptools/__init__.py", line 153, in setup
    return distutils.core.setup(**attrs)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/core.py", line 148, in setup
    dist.run_commands()
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/dist.py", line 955, in run_commands
    self.run_command(cmd)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/dist.py", line 974, in run_command
    cmd_obj.run()
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 79, in run
    _build_ext.run(self)
  File "/home/benjamin/.local/lib/python3.6/site-packages/Cython/Distutils/old_build_ext.py", line 186, in run
    _build_ext.build_ext.run(self)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/command/build_ext.py", line 339, in run
    self.build_extensions()
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 649, in build_extensions
    build_ext.build_extensions(self)
  File "/home/benjamin/.local/lib/python3.6/site-packages/Cython/Distutils/old_build_ext.py", line 195, in build_extensions
    _build_ext.build_ext.build_extensions(self)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/command/build_ext.py", line 448, in build_extensions
    self._build_extensions_serial()
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/command/build_ext.py", line 473, in _build_extensions_serial
    self.build_extension(ext)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/setuptools/command/build_ext.py", line 196, in build_extension
    _build_ext.build_extension(self, ext)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/command/build_ext.py", line 533, in build_extension
    depends=ext.depends)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/distutils/ccompiler.py", line 574, in compile
    self._compile(obj, src, ext, cc_args, extra_postargs, pp_opts)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 396, in unix_wrap_single_compile
    cflags = unix_cuda_flags(cflags)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 372, in unix_cuda_flags
    cflags + _get_cuda_arch_flags(cflags))
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/utils/cpp_extension.py", line 1400, in _get_cuda_arch_flags
    capability = torch.cuda.get_device_capability()
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/cuda/__init__.py", line 309, in get_device_capability
    prop = get_device_properties(device)
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/cuda/__init__.py", line 314, in get_device_properties
    _lazy_init()  # will define _get_device_properties
  File "/home/benjamin/anaconda3/envs/pvn3d_b/lib/python3.6/site-packages/torch/cuda/__init__.py", line 190, in _lazy_init
    torch._C._cuda_init()
RuntimeError: cuda runtime error (100) : no CUDA-capable device is detected at /pytorch/aten/src/THC/THCGeneral.cpp:47

The message no CUDA-capable device is detected surprises me since I did follow the steps for the CUDA installatio.

I'm closing this issue because the problem seems to be that I am using Ubuntu - and therefore also nvidia and cuda 10.1 - with VirtualBox in a VM, which doesnt work. nvidia doesn't seem to have access to my GPU through the VM.


Fix storage type fixtures

I changed the fixtures and storage volume type to singular.
That should be correct according to the current swagger API example.
However, I do not know if this was always the case.

Can you edit your commit message to include the Signed-off-by: Firstname Lastname <email> line?
You can usually do so with git commit --amend -s followed by a git push --force

Codecov Report

Merging #497 (2a917c1) into master (1bbd68e) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #497   +/-   ##
=======================================
  Coverage   97.07%   97.07%
=======================================
  Files          56       56
  Lines        4101     4101
=======================================
  Hits         3981     3981
  Misses        120      120

Impacted Files
Coverage Δ

pylxd/tests/mock_lxd.py
100.00% <ø> (ø)

pylxd/models/storage_pool.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

Thanks!

Thanks!


custom icon server, how to load icons

Hey, First of all thanks a lot for iconify this icon library it is very powerful!
I'm having some confusion with my custom icon server, as follows:
I use https://github.com/iconify/api.js , run locally

Then I want to load the icon using the icon component of online mode
<Icon icon="mdi-light:home" />
But I found that this address will actually be requested https://api.iconify.design

Can you provide an api to replace the resource address, this can have the same Properties as the online icon

You need to change configuration for component to point to your API.
import { addAPIProvider } from '@iconify/vue';

addAPIProvider('', {
	resources: ['http://localhost:3000'],
});

Thank you very much 🙏


Task multi user assignment

Allow a task to be assigned to multiple volunteers
Moved Assign Volunteers to Task Detail from Activity
On Task Detail added All volunteers and Assigned Volunteers tabs
this completes #162

I'm reviewing this code now. @bcbeatty Can you fix the merge conflicts?


feat: Import as a Service API implementation

Related Issues

SITES-22159 Queue a new import job from the spacecat-api-service
SITES-22513 Implement API to query job status
SITES-22512 Enable download of import job archive once job is complete

Prerequisite
The following PR needs to be merged in order for the tests in this PR to pass: https://github.com/adobe/spacecat-shared/pull/259

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

v1.42.0
GitHub release

Your semantic-release bot :package::rocket:


Incorrect version and architecture assignment for list of packages in dnf_package

Description
When using dnf package resource for list of packages and providing version and/or architecture, it is expected for all packages in the list to use the defined version. But in safe_version_array and safe_arch_array only first element would get defined version/arch. Remaining packages in the list would get latest version.
Code:
        def safe_version_array
          if new_resource.version.is_a?(Array)
            new_resource.version
          elsif new_resource.version.nil?
            package_name_array.map { nil }
          else
            [ new_resource.version ]
          end
        end

Proposed fix to return list filled with version defined in resource:
        def safe_version_array
          if new_resource.version.is_a?(Array)
            new_resource.version
          elsif new_resource.version.nil?
            package_name_array.map { nil }
          else
            package_name_array.map { new_resource.version }
          end
        end

Chef Version
Chef Infra Client: 15.13.8
Platform Version
CentOS Stream release 8
Replication Case
Case can be replicated when installing not latest versions of packages available.
Client Output
Declared resource:
dnf_package("git, perl-Git, git-core, git-email, git-core-doc, gitk, git-gui") do
  package_name ["git", "perl-Git", "git-core", "git-email", "git-core-doc", "gitk", "git-gui"]
  action [:install]
  default_guard_interpreter :default
  declared_type :package
  cookbook_name "fbit_devtools"
  recipe_name "rhel"
  version "2.30.2-2.fb"
end

What actually it tries to do (check different versions for git and all other packages):
Begin output of ["dnf", "-y", "install", "git-0:2.30.2-2.fb.x86_64", "perl-Git-0:2.31.1-2.el8.noarch", "git-core-0:2.31.1-2.el8.x86_64", "git-email-0:2.31.1-2.el8.noarch", "git-core-doc-0:2.31.1-2.el8.noarch", "gitk-0:2.31.1-2.el8.noarch", "git-gui-0:2.31.1-2.el8.noarch"]

No, its intended that you need to have an array of versions that matches the package array.  Your code will also coerce the version to an array even if the package provider isn't given an array (if you try your fix lots of specs in spec/functional/resource/dnf_package_spec.rb should fail).
The correct way to do what you want is just this:
  version [ "2.30.2-2.fb" ] * 7

Or possibly less brittle but i haven't checked:
  version  [ "2.30.2-2.fb" ] * package_name.length

And the general case people wouldn't want that if they're installing multiple packages that have nothing at all to do with each other:
dnf_package "lsof", "tcpdump", "strace", "zsh", "git"

@lamont-granquist any reason not to create a cookstyle rule on this one?

@lamont-granquist If recipe should accept array of versions it should fail if value is not array or length of array doesn't match length of array of packages. This inconsistent behavior led to the configuration issue which was hard to identify.

version [ "2.30.2-2.fb" ] * 7
This is direct invitation for bug to happen when array of packages is extended.

Anyway, thanks a lot for looking into it!


iOS isKeyboardConnected always true

First of all, thank you for this library.
I'm using it to detect if the user has the keyboard connected and it works perfectly fine on Android, but I'm having some issues with iOS so I'd like some help - even simply pointing me in the right direction would mean a lot
Basically on iOS isKeyboardConnected always returns true, even if I don't have an external keyboard connected
Has someone ever experienced it?
Thanks in advance

Hello @consuelo-sanna,
I appreciate your usage and for reporting issue. I will look into the issue within the next few hours, with the latest resolution expected by the end of this week.
If you are primarily use the 'keyboard is connected' feature, I recommend to check the following library: react-native-is-keyboard-connected.
Furthermore, I have divided the functionality into several libraries, allowing you to use only specific features:

react-native-external-keyboard - used for keyboard a11y focus management and handling keypress events.
react-native-is-keyboard-connected - created for detecting the presence of a physical keyboard.
react-native-a11y-order - created for controlling screen reader a11y order.

If you have any additional needs, or have any ideas for implementation, feel free to share them here or personaly<EMAIL_ADDRESS>Sincerely,
Artur Kalach

Hello @consuelo-sanna,
I have checked isKeyboardConnected functionality on iOS and it works fine on latests versions of the library (0.4.0/0.4.0-rc, 0.3.1).
Could you also share additional information about:

ReactNative version
react-native-a11y version
What type of the keyboard do you use
Code example

If you have any progress, please share with me, it can help to improve the lib and a11y support in RN.
Sincerely,
Artur Kalach

Hello @ArturKalach,
thank you for your fast response and help, much appreciated!
I tried react-native-is-keyboard-connected and it works directly with this library, I don't know why it behave differently with react-native-a11y (at least that's my experience)
I'm using the following:
React Native: "0.71.7"
react-native-a11y: 0.3.2
I'm using a Fintie BT keyboard
I'll attach the part of code that was not working, I had no problem with the listener, my only problem was on start and this was the related code:
A11yModule.isKeyboardConnected()
      .then((enabled: boolean) => {
        __DEV__ && console.log('initial keyboard attached: ', enabled);  // ---> always returns true
        // eslint-disable-next-line promise/always-return
        rootStore.appDataStore?.setKeyboardAttached(enabled);
      })
      .catch(() => {
        rootStore?.appDataStore?.setKeyboardAttached(false);
      });

Anyway, as I said before it works with react-native-is-keyboard-connected, so thank you :)

Dear @consuelo-sanna,
Sorry for misunderstanding, I thought that you have problem with listeners. I will check this issue in fews days. It is belivable that there are issues with A11yModule.isKeyboardConnected() becouse it poor tested.
Even react-native-a11y will work in some days, but in case you need only the keyboard is connected feature, I would suggest to use react-native-is-keyboard-connected.
Thank you a lot for helping with the issue, your information will help to fix and improve the library, I really appreciate this.

Dear @consuelo-sanna,
The A11yModule.isKeyboardConnected has been fixed in the latest version of the lib: 0.4.1
I saw this error previously, but some why didn't fix it here.
Thank you a lot for investigation and helping with issue.
Regards,
Artur Kalach

Good morning,
just so you know I'm seeing a warning about not removing the listeners for the keyboardStatusListener
I tried both react-native-is-keyboard-connected and this package, both have the same issue
if I try to manually remove the listeners, TS warns me that

Property 'remove' does not exist on type 'void'

and the warning in the console does not disappear

new NativeEventEmitter() was called with a non-null argument without the required removeListeners method.

I'm using the following:
React Native: "0.73.2"
react-native-a11y: 0.4.3
I'm using a Fintie BT keyboard
Looks like the listener still does work on dev though

Hello @consuelo-sanna,
Thank you for information, I will check it in this week.

Hello @consuelo-sanna,
I tested the libs, and found out that there a mistake with types only when I start to write response)
There small mistake with types in A11yModule.types.ts.
It should be like this:
A11yModule.types.ts
...
 keyboardStatusListener: (callback: StatusCallback) => () => void;`
...

I will fix it in the next patch, but it can take some time.

What relates to warning....
new NativeEventEmitter() was called with a non-null argument without the required removeListeners method.

Honestly, I can't reproduce it... If you have any ideas why it shown or how reproduce please share them.

Thank you a lot for notifying issue, it helps me a lot.

Hello @consuelo-sanna,
The type for keyboardStatusListener has been fixed in react-native-a11y: 0.4.4.
I will create separate issue for react-native-is-keyboard-connected.

It appears that the issue is no longer reproducing. I am closing it due to the lack of activity and any other potential cases.
Please reopen the issue in case of issue reproducing.


Add ability to deploy to minikube for development env

Fixes: AAP-5261
Snippet from README (for testing purposes)

I ran into an error:
$ task minikube:build -- eda:001 task: [minikube:build] scripts/eda_kube.sh build eda:001 /Users/ben/git/ansible-events-ui/scripts/common/utils.sh: line 11: conditional binary operator expected task: Failed to run task "minikube:build": exit status

@ddonahue007 rebase please


Adding wait_for support

[x] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

Resolves #713.

@kessler-frost if documentation will be included in the PR - maybe change back to draft and mark as ready when done?


[Bug] Concurrent requests are being run sequentially on AMD MI60 (gfx906)

🐛 Bug
Hello team,
Thanks for creating such an amazing engine. I ran Llama-3-8B-Instruct-q4f16_1-MLC in server mode with different batch sizes (2-128) but I still see my requests are being run sequentially. With interactive chat mode, that model runs at ~80t/s on a single MI60 which is great. But when doing batch inference I expect it to be larger than 80t/s.
To Reproduce
Steps to reproduce the behavior:

python -m pip install --pre -U -f https://mlc.ai/wheels mlc-llm-nightly-rocm62 mlc-ai-nightly-rocm62
I started with mlc_llm serve HF://mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC  --mode server but it was too slow. Then, experimented with different options till I found out this one which worked at around 65t/s when batching:
mlc_llm serve HF://mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC --overrides "prefill_chunk_size=2048;max_num_sequence=128;context_window_size=4096" --mode server
Downloaded a benchmarking repo (MMLU-Pro) that accepts generic OpenAI API: git clone https://github.com/chigkim/Ollama-MMLU-Pro.git
Then, I ran batch benchmarking: python3 run_openai.py --url http://<IP_ADDRESS>:8000/v1 --category 'computer science' --verbosity 0 --parallel 64 --model HF://mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC

Each question is less than 4096 tokens. Each question took on average 2s to complete. It seems like mlc-llm is not doing batch inference even though it takes over 85% of 32GB VRAM. In comparison, I ran the same commands (except for 1 which was CUDA specific) RTX 3090 and I got over 700t/s in mlc-llm for the same benchmark. So, there is no issue with the benchmark or mlc server mode with CUDA backend. Only AMD/ROCm GPU seems to have no batch inference.
Here is the inference output I get with AMD MI60 when doing batch inference at 65t/s:

mlc_llm serve HF://mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC --overrides "prefill_chunk_size=2048;max_num_sequence=64;context_window_size=4096" --mode server
[2024-10-21 20:23:41] INFO auto_device.py:88: Not found device: cuda:0
[2024-10-21 20:23:42] INFO auto_device.py:79: Found device: rocm:0
[2024-10-21 20:23:42] INFO auto_device.py:79: Found device: rocm:1
[2024-10-21 20:23:43] INFO auto_device.py:88: Not found device: metal:0
[2024-10-21 20:23:44] INFO auto_device.py:79: Found device: vulkan:0
[2024-10-21 20:23:44] INFO auto_device.py:79: Found device: vulkan:1
[2024-10-21 20:23:44] INFO auto_device.py:79: Found device: vulkan:2
[2024-10-21 20:23:44] INFO auto_device.py:79: Found device: vulkan:3
[2024-10-21 20:23:45] INFO auto_device.py:88: Not found device: opencl:0
[2024-10-21 20:23:45] INFO auto_device.py:35: Using device: rocm:0
[2024-10-21 20:23:45] INFO download_cache.py:227: Downloading model from HuggingFace: HF://mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC
[2024-10-21 20:23:45] INFO download_cache.py:29: MLC_DOWNLOAD_CACHE_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-10-21 20:23:45] INFO download_cache.py:166: Weights already downloaded: /home/saidp/.cache/mlc_llm/model_weights/hf/mlc-ai/Llama-3-8B-Instruct-q4f16_1-MLC
[2024-10-21 20:23:45] INFO jit.py:43: MLC_JIT_POLICY = ON. Can be one of: ON, OFF, REDO, READONLY
[2024-10-21 20:23:45] INFO jit.py:158: Using cached model lib: /home/saidp/.cache/mlc_llm/model_lib/c53c4a7c987b8d7ea642bf287fbe03f6.so
[2024-10-21 20:23:45] INFO engine_base.py:192: The selected engine mode is server. We use as much GPU memory as possible (within the limit of gpu_memory_utilization).
[2024-10-21 20:23:45] INFO engine_base.py:200: If you have low concurrent requests and want to use less GPU memory, please select mode "local".
[2024-10-21 20:23:45] INFO engine_base.py:205: If you don't have concurrent requests and only use the engine interactively, please select mode "interactive".
[20:23:45] /workspace/mlc-llm/cpp/serve/config.cc:688: Under mode "local", max batch size 64 is specified by user, max KV cache token capacity will be set to 4096, prefill chunk size 2048 is specified by user.
[20:23:45] /workspace/mlc-llm/cpp/serve/config.cc:688: Under mode "interactive", max batch size 64 is specified by user, max KV cache token capacity will be set to 4096, prefill chunk size 2048 is specified by user.
[20:23:45] /workspace/mlc-llm/cpp/serve/config.cc:688: Under mode "server", max batch size 64 is specified by user, max KV cache token capacity will be set to 179536, prefill chunk size 2048 is specified by user.
[20:23:45] /workspace/mlc-llm/cpp/serve/config.cc:769: The actual engine mode is "server". So max batch size is 64, max KV cache token capacity is 179536, prefill chunk size is 2048.
[20:23:45] /workspace/mlc-llm/cpp/serve/config.cc:774: Estimated total single GPU memory usage: 27839.176 MB (Parameters: 4308.133 MB. KVCache: 22652.282 MB. Temporary buffer: 878.761 MB). The actual usage might be slightly larger than the estimated number.
INFO:     Started server process [40316]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     <IP_ADDRESS>:57132 - "POST /v1/chat/completions HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:57012 - "POST /v1/chat/completions HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:57066 - "POST /v1/chat/completions HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:57038 - "POST /v1/chat/completions HTTP/1.1" 200 OK
...

Expected behavior
I expect batch inference to run at least two times the speed of interactive inference.

Additional context
NA.

@MasterJH5574 Please, let me know. Thanks!


Model Trains on CPU but not on GPU - TensorDot/MatMul error

Click to expand!
Issue Type
Bug
Source
source
Tensorflow Version
tf 2.9.1
Custom Code
Yes
OS Platform and Distribution
Linux Ubuntu 22.04
Mobile device
No response
Python version
3.9.12
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
CUDA=11.2, cuDNN=8.1
GPU model and memory
RTX 8000 48GB
Current Behaviour?
I am trying to train a model using the GPUs on my computer. The model trains as expected on the CPU but I get the following error when trying to use the GPU:

Node: 'model/final_output/Tensordot/MatMul'
2 root error(s) found.
  (0) INTERNAL:  Blas xGEMV launch failed : a.shape=[1,2628288,150], b.shape=[1,150,1], m=2628288, n=1, k=150
	 [[{{node model/final_output/Tensordot/MatMul}}]]
	 [[Nadam/Nadam/group_deps/_111]]
  (1) INTERNAL:  Blas xGEMV launch failed : a.shape=[1,2628288,150], b.shape=[1,150,1], m=2628288, n=1, k=150
	 [[{{node model/final_output/Tensordot/MatMul}}]]
0 successful operations.
0 derived errors ignored. [Op:__inference_train_function_29232]

I have tried multiple versions of tensorflow gpu (2.5 through 2.10) and get the same error. I would expect the network to train the same on GPU as on CPU.

### Standalone code to reproduce the issue

```shell
The network that I have created is a bit cumbersome for posting on this platform. I am happy to send files if needed.

Relevant log output
Node: 'model/final_output/Tensordot/MatMul'
2 root error(s) found.
  (0) INTERNAL:  Blas xGEMV launch failed : a.shape=[1,2628288,150], b.shape=[1,150,1], m=2628288, n=1, k=150
	 [[{{node model/final_output/Tensordot/MatMul}}]]
	 [[Nadam/Nadam/group_deps/_111]]
  (1) INTERNAL:  Blas xGEMV launch failed : a.shape=[1,2628288,150], b.shape=[1,150,1], m=2628288, n=1, k=150
	 [[{{node model/final_output/Tensordot/MatMul}}]]
0 successful operations.
0 derived errors ignored. [Op:__inference_train_function_29232]

@willi3by,
To expedite the trouble-shooting process, could you please provide a complete code you are using. Thank you!

I'm having this bug as well in tf2.10.0, cudatoolkit 11.2.
Blas xGEMV launch failed : a.shape=[1,13238742,6], b.shape=[1,6,1], m=13238742, n=1, k=6
	 [[{{node model/attention/dense_2/MatMul}}]] [Op:__inference_train_function_1902]

Thanks for the response @tilakrayal  and for the second @markemus. I confirmed this issue by training the network with a batch size of 1 with no issues, as the matrix was smaller than 2^20 with only one sample. I will work on getting the code up soon.
Thanks

@tilakrayal There is a minimal code example here: https://github.com/tensorflow/tensorflow/issues/54463 for recreating the bug. I tried it on my setup and got the exception.

@willi3by,
Could you please take a look at this https://github.com/tensorflow/tensorflow/pull/56691 which fixed the Unable to register cuBLAS factory error. Please try this and let us know if the issue still persists. Thank you!

@tilakrayal
Thanks for the link. I am not sure what to try per se. I read through and it looks like this change was committed to the master branch of tensorflow on August 10. I only recently install tensorflow using pip. Does pip pull the most recent tensorflow master or do I have to compile from source?
Thanks

@willi3by , You need to install tensorflow nightly version for that PR to be available. Since this PR could not make it at the time of 2.10.0 brach cut.
It might be included if there is any patch release for 2.10.0 or it will be available in the next release, i.e 2.11.0.

@tilakrayal I apologize for the delay. I updated TF and am still getting the same error. My current version of TF is 2.11.0-dev20220928

Hi @willi3by,
Have you tried downgrading from CUDA 11.2 to CUDA 11.1?
If you are using conda environment, following installation steps in https://www.tensorflow.org/install/pip, conda install -c conda-forge cudatoolkit=11.1 cudatoolkit-dev=11.1 cudnn=8.1.0 -y sets up my environment perfectly. I have been doing this to circumvent the issue with CUDA 11.2 and you might find this helpful as well.

@arvindrajan92 That fixed it!! Thanks very much for your suggestion.


test(coverage): Add babel-istanbul for coverage

Linked to  #355 .
This adds npm run test:coverage which uses babel-istanbul.
The difference between istanbul and babel-istanbul is that it shows coverage on the code before it is compiled by babel.

By analyzing the blame information on this pull request, we identified @vvo, @bobylito and @redox to be potential reviewers

@Jerskouille could you also add the associated badge?

I am a bot! I will fix this

Good first steps, thanks! Do you already know how we can automate it and use any service (jenkings, coveralls..?) so that we track the coverage a little (badge, PR)?

I have no knowledge of coveralls. I've started looking, but I am not confident giving coveralls access to all the algolia repositories when I try to login with GitHub.

I have no knowledge of coveralls. I've started looking, but I am not confident giving coveralls access to all the algolia repositories when I try to login with GitHub.

We're also using coveralls on a bunch of projects, don't worry ;)

Ok let's move forward and add coverall etc in another PR since this one is "linked" and already useful

I think coverall is using their own tools to calculate the coverage automatically, we have nothing to add (except if we want to get the value before pushing to coverall)


CB-9097 fail with a more descriptive error if run as admin

cordova run will fail for windows if invoked from admin prompt. Currently we will wait until ActivateApplication stage and fail afterwards. We should fail fast and improve user experience.

@vladimir-kotikov Could you please help review and test this one?

LGTM

wfm! Merging.


Do not allow node to be dropped at source location based on source location properties.

Hi. Thanks for writing a nice tree for angular. Can you tell me the best way to not allow a node to be dropped on a target node? I thought maybe allowDrop but that is associated with the source node. (node to be dropped). I want to disallow drop of target if source node has certain properties.
Thanks

Never mind about this issue. I got it working by determining the source from the allowDrop after all.

Can you please elaborate more on how did you solve this issue?
This is the function that I am using for allowDrop
allowDrop: (element: any, {parent, index}) => {});

The element is what is being dragged, parent is the parent of the object that is being dragged and index is the index of the element that is being dragged.
There is no way to get the target object, i.e. where the dragged object is supposed to be dropped.
Can you please explain how did you manage to get the target properties and prevent allowDrop based on that because I am facing the same problem.
Thank you


Error when using register_data

I followed the steps on the documentation to try to use the MNist dataset, but I get an error when registering the dataset using federatedscope.register.register_data
The link to the document is https://federatedscope.io/docs/own-case/#data ;
the error is as follows:

Thank you for bringing this to our attention. It appears that the tutorial could indeed be slightly outdated. To align with the current state of the codebase, the correct implementation of the function should be like this:

Thank you for your answer, but I have a new problem after changing the code, the new problem is as follows:

The complete code is below:
from torchvision.datasets import MNIST
from torchvision.transforms import transforms

from federatedscope.core.data import BaseDataTranslator
from federatedscope.register import register_data

def load_my_data(config, client_cfgs=None):
    transform = transforms.Compose([
        transforms.ToTensor(),
        transforms.Normalize(mean=[0.1307], std=[0.3081])
    ])
    data_train = MNIST(root='data', train=True, transform=transform, download=True)
    data_test = MNIST(root='data', train=False, transform=transform, download=True)
    translator = BaseDataTranslator(config, client_cfgs)
    fs_data = translator([data_train, [], data_test])
    return fs_data, config

def call_my_data(config, client_config=None):
    pass

register_data("mydata", call_my_data)

I changed the value of data.type in the scripts.example_configs.femnist.yaml file to mydata and ran federatedscope.main.py with this config file.


Feature/cli

Codecov Report

Merging #26 (a643000) into main (2ea10f2) will decrease coverage by 1.45%.
The diff coverage is 50.00%.

@@             Coverage Diff             @@
##              main      #26      +/-   ##
===========================================
- Coverage   100.00%   98.54%   -1.46%
===========================================
  Files           18       20       +2
  Lines          466      480      +14
===========================================
+ Hits           466      473       +7
- Misses           0        7       +7

Impacted Files
Coverage Δ

tests/api/test_base_client.py
100.00% <ø> (ø)

tests/api/test_categories.py
100.00% <ø> (ø)

tests/api/test_decorators.py
100.00% <ø> (ø)

tests/api/test_releases.py
100.00% <ø> (ø)

tests/api/test_series.py
100.00% <ø> (ø)

tests/api/test_sources.py
100.00% <ø> (ø)

tests/api/test_tags.py
100.00% <ø> (ø)

src/fred/_util/cli.py
30.00% <30.00%> (ø)

src/fred/_util/__init__.py
100.00% <100.00%> (ø)

src/fred/_util/decorators.py
100.00% <100.00%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


[Parser] Statements must be separated by newlines or semicolons

Environment data

Language Server version: v2021.10.0
OS and version: Windows 10 Pro
Python version (& distribution if applicable, e.g. Anaconda): Python 3.9 with a Virtual environment

Expected behaviour
The "Statements must be separated by newlines or semicolons" error does not show.
Actual behaviour
The "Statements must be separated by newlines or semicolons" shows.

Logs
I get no additional output when putting "python.analysis.logLevel": "Trace" in my settings.json. If I had to guess, kivy, the GUI library I am using, is interfering with the output.
Code Snippet / Additional information
from kivy.uix.boxlayout import BoxLayout
from kivy.uix.button import Button
from kivy.uix.dropdown import DropDown
from kivy.uix.spinner import Spinner
from kivy.uix.splitter import Splitter

class RightLayout(Splitter):
    def __init__(self, root, **kwargs):
        super(RightLayout, self).__init__(**kwargs)

        self.root = root

        self.sizable_from = 'left'
        self.min_size = 15
        self.max_size = 270

        self.layout = BoxLayout(orientation = 'vertical')
        self.add_widget(self.layout)

Pylance complains that the last line would throw an error regardless of what I do. Adding a semicolon temporarily fixes it, however the error comes back whenever I add a new line. Python does not report a syntax error or any error whatsoever.

I'm not able to repro the problem with the above code sample.
I wonder if there's a non-printable character (perhaps a result of file corruption) that is present in the file? That's the only hypothesis that I have currently.
Here's what I suggest. Make a copy of the file to preserve the original contents so we can do further investigation if necessary. Then open the original file and try deleting and retyping the characters after and prior to the ). Does the error disappear?

Here's what I suggest. Make a copy of the file to preserve the original contents so we can do further investigation if necessary. Then open the original file and try deleting and retyping the characters after and prior to the ). Does the error disappear?

Alright, I've made a copy. Retyping everything hasn't fixed anything, the error still persists?

Let's try this. Create a new ".py" file, then copy the sample code above and paste it into the new file. Do you see the problem then?
What is the character encoding of the original file? Is it "UTF-8"? You can find the encoding by looking in the bottom right part of the VS Code window.

Here's what I suggest. Make a copy of the file to preserve the original contents so we can do further investigation if necessary. Then open the original file and try deleting and retyping the characters after and prior to the ). Does the error disappear?

I don't get the error anymore pasting it in, which is pretty odd?
I double checked and the encoding is UTF-8.

If you can provide us a file that repros the problem, we can investigate. Looks like you are unblocked for now. If this is still an issue please reopen this issue.

I have the same error now:

@Devourian, thanks for reporting this. We had trouble reproducing this issue last time. If you have a file that still shows this issue, could you try replying here and attaching the file (drag the file into your new comment). I'm hoping that that approach (rather than just providing the text of the file in a comment) will allow us to reproduce the issue on our computers.


Protein Powder not saving

I bought PP lvl1 and lvl2 after few rounds i had to reset game and only PP from all upgrades was not saved and we had again lvl0

The issue is that the level of protein powder is saved not with the current Protein Powder's level but Beekeeper's.
Regardless, the authors/contributors are aware of the issue and they will fix whenever possible.


Failing on Travis CI

Hey Guys,
I noticed that all builds of painload on Travis have failed as you have not configured how painload is to be tested by Travis CI. (http://travis-ci.org/#!/krebscode/painload)
Might I suggest you add a .travis.yml file and configure how the build should run (http://about.travis-ci.org/docs for more info) or maybe consider removing the service hook.
Have a great day,
Josh
p.s. feel free to pop into #travis on freenode for more info

won't fix at the current status of this project.

Hi makefu,
I find that a really odd response to a rather polite email.
If you do not want to fix the build, please consider removing the service hook as 363 fail builds shows you don't really care about using Travis.
Have a great day,
Josh


Add Maputo Meetup

Hey @rosariopfernandes we have conflicts that doesn`t allows me to merge, can you fix it?

My bad. Fixed it.


Seems Lunr.js doesn't support other language

When I search for English words, it works, but when I use Chinese or Japanese, it doesn't work. However, my blog is mainly in these two languages.
I guess it is Lunr.js doesn't support.
Was there any alternative？😂

Lunr.js definitely does support other languages. I don't know how that would be integrated into this project though.

@olivernn Yep, seems a lot of works to make this search engine support other languages


HF Hub dependency conflicts with latest tranformers version.

🐛 Bug
COMET has an explicit dependency for both transformers = "^4.17" and huggingface-hub = "^0.16.0" which will only allow versions up to 0.17.0, but the newest version of transformers (v4.36.1) has its own huggingface-hub dependency which is >=0.19.3,<1.0. This creates dependency conflicts for anyone trying to build a package on COMET and the latest transformers package. Can we do one of the following:

Remove the huggingface-hub dependency from COMET and just keep the transformers dependency given that the former is implicit in the latter;
Use the same style of huggingface-hub dependency that transformers uses (namely "huggingface-hub>=0.19.3,<1.0") to mitigate dependency conflicts;
At very least, update the huggingface-hub dependency to be ^0.19.3.

Hi @scottfleming! I like this suggestions but I rather go for 2. Because if for some reason HF decides to decouple the two packages, COMET will stop working (unable to download models).

I just released a new version that should solve this dependency issue. Ping me if the error still persists.
PS: I went for option 2

Thanks so much, @ricardorei! I'll test this out and let you know if I run into any problems.


Please update incorrect steps

Under "Add a partner to an existing subscription" this step is incorrect:

Go to Billing > Products & services and select a subscription.

"Products & Services" is not an option. Under Licenses > "Go to Products & services" there is no option for partners.
I would also like to add that it is extremely frustrating  how often I find mistakes like this in documentation. If the instruction manual isn't ready, the product isn't ready. The amount of compounded wasted time that results from all the pointless Office 365 GUI changes must be incalculable and astronomical.

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

ID: 6985ca13-b574-4d28-52f1-eca702b268bb
Version Independent ID: d15b3013-95b5-1e57-825c-055f50d45878
Content: Add a Partner of Record (POR) - Power Platform
Content Source: power-platform/admin/add-partner-record-subscription.md
Service: power-platform
GitHub Login: @jimholtz
Microsoft Alias: jimholtz

@ckssaas
Thanks for bringing this to our attention. The procedure has been corrected.


calculate and output the pool revenue from a given payout run

As requested by Nightguy1688#3119 in Discord: https://discord.com/channels/484437221055922177/799597981762453535/866456694540599343
Add logging to show:
Total Coinbase Generated: total block reward
Total User Transaction Fees Generated: fees collected from user transactions
Total Snark Fees Paid: fees paid out for snarks (includes fees from coinbase)
Net Coinbase Received: = Total Coinbase Generated - User Tx Fees - Snark Fees
Total Payouts Sent: sum of payout amounts
Total Payout Transaction Fees: sum of payout transaction send fees
Net MINA to Pool Operator: + Net Coinbase Received - Total Payouts Sent - Total Payout Transaction Fees

As requested by Nightguy1688#3119 in Discord: https://discord.com/channels/484437221055922177/799597981762453535/866456694540599343
Add logging to show:
Total Coinbase Generated: total block reward
Total User Transaction Fees Generated: fees collected from user transactions
Total Snark Fees Paid: fees paid out for snarks (includes fees from coinbase)
Net Coinbase Received: = Total Coinbase Generated - User Tx Fees - Snark Fees
Total Payouts Sent: sum of payout amounts
Total Payout Transaction Fees: sum of payout transaction send fees
Net MINA to Pool Operator: + Net Coinbase Received - Total Payouts Sent - Total Payout Transaction Fees

Net Coinbase Received: = Total Coinbase Generated - User Tx Fees - Snark Fees
it seems it's not correct. you should plus User Tx Fees instead of minus
Net Coinbase Received: = Total Coinbase Generated + User Tx Fees - Snark Fees

Closed w/ #71