Trying to use nodemailer in Angularjs and but failed with "undefined is not a function"

Hi,
I am new to nodejs and just trying to use Nodemailer in AnguarJS.
My code in controllers.js looks like this:
      var nodemailer = require('nodemailer');
      var smtpTransport = require('nodemailer-smtp-transport');

      var transporter = nodemailer.createTransport(smtpTransport({
            host: 'smtp.gmail.com',
            service: 'gmail',
            port: '465',
            secure: true,
            auth: {
                user: 'username',
                pass: 'password'
            }
        }));

      transporter.sendMail({
            from<EMAIL_ADDRESS>// sender address
            to<EMAIL_ADDRESS>// list of receivers
            subject: 'Hello ✔', // Subject line
            text: 'Hello world ✔', // plaintext body
            html: '<b>Hello world ✔</b>' // html body
      });

When I try to send an email, it will give me such error message:

TypeError: undefined is not a function
at SMTPConnection.connect (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:779:28)
at SMTPTransport.send (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:1750:16)
at Nodemailer. (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:12290:30)
at Nodemailer._processPlugins (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:12410:16)
at Nodemailer. (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:12286:14)
at Nodemailer._processPlugins (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:12410:16)
at Nodemailer.sendMail (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:12275:10)
at l.controllers.controller.$scope.send (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/js/bundle.js:167:19)
at hb.functionCall (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/bower_components/angular/angular.min.js:198:424)
at Cc.(anonymous function).compile.d.on.f (file:///Users/zhiyzuo/Documents/web/zhiyzuo.github.io/static/bower_components/angular/angular.min.js:227:394)

And the corresponding line in bundle.js is:

this._socket = tls.connect(this.options.port, this.options.host, opts, this._onConnect.bind(this));

I test the webpage after I run this command:
$ browserify controllers.js > bundle.js

What's wrong?!! Please help me !! Thank you!!

You can't open sockets from a web bage, you need to run Nodemailer in your server and not in a browser.

Thank you for the reply!
Does this mean that I cannot use nodeamailer on Github user page?

On Feb 2, 2015, at 1:09 AM, Andris Reinman<EMAIL_ADDRESS>wrote:
You can't open sockets from a web bage, you need to run Nodemailer in your server and not in a browser.
—
Reply to this email directly or view it on GitHub https://github.com/andris9/Nodemailer/issues/384#issuecomment-72414519.

No, you can't. It is not possible to run anything in the browser that requires access to tcp sockets

I see. Thank you so much!
On Monday, February 2, 2015, Andris Reinman<EMAIL_ADDRESS>wrote:

No, you can't. It is not possible to run anything in the browser that
requires access to tcp sockets
—
Reply to this email directly or view it on GitHub
https://github.com/andris9/Nodemailer/issues/384#issuecomment-72525882.

--
Best,
Zhiya


Added Hiligaynon (hil)

@bobreyes First time I tried to add my commit in conjunction of yours (PR201) in order to make the PR works. I am closing this one.


[SPARK-6521][Core] Bypass unnecessary network access if block managers share an identical host

Refactored #5178 and added unit tests.

Merged build triggered.

Merged build started.

Test build #45071 has started for PR 9478 at commit 849eaa7.

Test build #45071 has finished for PR 9478 at commit 849eaa7.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test FAILed.

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

Merged build triggered.

Merged build started.

Test build #45106 has started for PR 9478 at commit 5c00a5a.

Test build #45106 has finished for PR 9478 at commit 5c00a5a.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test FAILed.

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

Merged build triggered.

Merged build started.

Test build #45117 has started for PR 9478 at commit 8d0a2f0.

Test build #45117 has finished for PR 9478 at commit 8d0a2f0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test FAILed.

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

retest this please

retest this please

Merged build triggered.

Merged build started.

Test build #45163 has started for PR 9478 at commit 8d0a2f0.

Merged build triggered.

Merged build started.

Test build #45166 has started for PR 9478 at commit 211f0b9.

Merged build triggered.

Merged build started.

Test build #45170 has started for PR 9478 at commit cebc895.

Test build #45163 has finished for PR 9478 at commit 8d0a2f0.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test FAILed.

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

Test build #45166 has finished for PR 9478 at commit 211f0b9.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

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

Merged build finished. Test PASSed.

Test build #45170 has finished for PR 9478 at commit cebc895.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test PASSed.

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

@andrewor14 Could you review this and give some suggestions?

@maropu Thanks for taking this over. I took a pass and it seems that the abstractions can be simplified a little. Please also fix the style issues that I pointed out. I believe the high level approach is correct, however.

By the way, have you done some benchmarking to measure how much this actually saves? I wonder if the gains are actually all that significant. Maybe it actually matters if we're fetching many small blocks, but even then I'm not 100% sure.

Thx for your reviews.
I'll fix it and also do benchmarks .

Merged build triggered.

Merged build started.

Test build #45591 has started for PR 9478 at commit 6b8f7bf.

Test build #45591 has finished for PR 9478 at commit 6b8f7bf.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n  * public class JavaLBFGSExample \n  * class LDA @Since(\"1.6.0\") (\n  *   case class Metadata(\n  *       require(className == expectedClassName, s\"Error loading metadata: Expected class name\" +\n  * class SlidingRDDPartition[T](val idx: Int, val prev: Partition, val tail: Seq[T], val offset: Int)\n  * class SlidingRDD[T: ClassTag](@transient val parent: RDD[T], val windowSize: Int, val step: Int)\n

Merged build finished. Test FAILed.

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

@maropu any updates? Did you have a chance to do the benchmarks?

@andrewor14 I tried quick benchmarks though, I saw little difference, so I'm taking various tests on it.
I'll get the result in a day.

Test build #46739 has started for PR 9478 at commit 1653691.

Test build #46741 has started for PR 9478 at commit 4ca2d72.

Test build #46739 has finished for PR 9478 at commit 1653691.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #46741 has finished for PR 9478 at commit 4ca2d72.

This patch fails from timeout after a configured wait of `250m`.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n  * public final class OneWayMessage implements RequestMessage \n  * case class Cast(child: Expression, dataType: DataType) extends UnaryExpression \n  * class SparkPlanInfo(\n  * class SQLMetricInfo(\n  * case class SparkListenerSQLExecutionStart(\n  * case class SparkListenerSQLExecutionEnd(executionId: Long, time: Long)\n

Merged build finished. Test FAILed.

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

Test build #46794 has started for PR 9478 at commit ba94687.

Test build #46794 has finished for PR 9478 at commit ba94687.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Merged build finished. Test FAILed.

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

Test build #46806 has started for PR 9478 at commit 303abcd.

@andrewor14 ISTM that this pr has a little effect on performance even
in case of many partitions involved in shuffle.
Test settings:

10 test runs except for the first run
4 workers in single c4.8xlarge (each worker has 12g mem)
spark.driver.memory=1g
spark.executor.memory=12g
spark.shuffle.bypassNetworkAccess=true
spark.sql.shuffle.partitions=1000
total shuffle size = around 1.5g
query executed:

import org.apache.spark.sql.types._
import org.apache.spark.sql.Row

val schema = StructType(
  Seq(
    StructField(s"col0", IntegerType, true),
    StructField(s"col1", StringType, true)
  )
)

val rdd =
  sc.parallelize((1 to 36), 36).flatMap { j => (1 to 5000000).map ( i =>
    Row(i, s"${i}"))
  }

sqlContext.createDataFrame(rdd, schema).registerTempTable("shuffle")

sqlContext.sql("cache table shuffle")

def timer[R](block: => R): R = {
  val t0 = System.nanoTime()
  val result = block
  val t1 = System.nanoTime()
  println("Elapsed time: " + ((t1 - t0 + 0.0) /<PHONE_NUMBER>.0)+ "s")
  result
}

(0 until 12).map { i =>
  timer { sql(s"""select col0, col1 from shuffle cluster by col0""").queryExecution.executedPlan(1).execute().foreach(_ => {}) }
}

w/o bypassing
Elapsed time: 9.344261745s
Elapsed time: 10.039675768s
Elapsed time: 11.644405718s
Elapsed time: 9.600332115s
Elapsed time: 9.745275262s
Elapsed time: 11.469732737s
Elapsed time: 11.495929827s
Elapsed time: 9.036307879s
Elapsed time: 9.159725732s
Elapsed time: 9.142030108s

w/ bypassing
Elapsed time: 8.529972448s
Elapsed time: 9.302905215s
Elapsed time: 9.933504313s
Elapsed time: 7.681615877s
Elapsed time: 9.16150229s
Elapsed time: 9.655643012s
Elapsed time: 8.948393461s
Elapsed time: 8.048391038s
Elapsed time: 8.780075475s
Elapsed time: 9.826335759s

I think that this patch might avoid issues caused by netty (i.e., gc pressures and network troubles) in a corner case though,
there is no big performance gain in this quick benchmarks.

Test build #46806 has finished for PR 9478 at commit 303abcd.

This patch fails from timeout after a configured wait of `250m`.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):\n  *   case class GetLocalDirsPath(blockManagerId: BlockManagerId) extends ToBlockManagerMaster\n

Build finished. Test FAILed.

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

@maropu thanks for running the benchmarks. It seems that the gains are not really significant enough to warrant all the complexity this patch adds. This is to a certain extent as expected since network is generally pretty fast, and Spark tends to bottleneck on CPU rather than I/O or network.

Can you close this PR?

@andrewor14 Okay and thanks. Also, can you close SPARK-6521?


Please add configure options for WWW_USER and /var/www/run/bgplgd.sock

As of writing, neither WWW_USER nor /var/www/run/bgplgd.sock can be overriden using options for ./configure. To increase the portability, it could make sense to change this, because what the www user to OpenBSD is, is e.g. www-data to Debian. And socket /var/www/run/bgplgd.sock is also not on a path that Linux distributions usually prefer. If this is considerated, the man pages also would need placeholders then.

Happy to do so, I knew that -portable may need something but was not sure what.
Do we also need to provide extra knobs for BGPLGD_USER and bgpctlsock (-S option default)?

Do we also need to provide extra knobs for BGPLGD_USER and bgpctlsock (-S option default)?

Ehm…aren't there already --with-bgplgd-user=user for BGPLGD_USER and RUNSTATEDIR (via --runstatedir=/run/bgpd) for bgpctlsock?

I see, I looked in the wrong tree :)
So the idea is to add a --with-www-user=user (defaults to www) and a --wwwrunstatedir=path which defaults to /var/www/run. Does that make sense?

Merged #57


Stop trying to make "Fetch" a thing.

This PR focuses on the Haskell data model, providing a stub definition of the 'Cacheable' monad. If the model doesn't offend anyone's sensibilities I'll commit the true 'Cacheable' implementation, including all implementations of 'Cacheable'-valued record fields.

This looks fine to me


Status of the game on other platforms

The Colorful Creature is available on 4 platforms as far as I know:

Steam
Itch.io
Game Jolt
Google App Store

However, only Steam is receiving constant updates.
What is the status of other platforms? Should other platforms also receive updates when new release is available here on GitHub?

Steam stays as is as always. Gamejolt and Itch.io could possibly be removed or stay as is since keeping updates for them is difficult as I don't know if there is a way to automatize the process for updating, I think it's better having the steam version since it handles packages and autoupdates pretty well, there's no DLC content such as donation dlcs, soundtrack included so that also makes it difficult. As for Android, updates are currently postponed because I lost an android key for it so I can't update the game, but I will bring the android version back to normal.

Thanks for the explanation. I guess you can write some announcement/message on itch.io and game jolt that game won't receive updates there.
You can keep this issue or create new one for having Android app updated. I know there is also some approval on Google side after every update, but it shouldn't take long.

Alright I updated the gamejolt and itch.io pages


entity not available

Thank you so much for your amazing work.
If I connect my plug before starting home assistant everything is working fine. if I plug the Tapo with home assistant running I have to restart it, otherwise I get "entity not available" and it doesn't seem to look up for the device afterwards.
I'm really new to home assistant and I don't know if this is normal behaviour for other integrations as well.
Is it possible to look for connected Tapo's while home assistant is running?
Also is it possible to have more than one Tapo plug? (I don't have more at the moment and cannot test)

Can't answer the first part but you can have more than one tapo plug, just repeat the config.
## Plug 1
  - platform: tapo_p100_control
    ip_address: 192.168.x.x
    email: !secret tapo_email
    password: !secret tapo_password
## Plug 2
  - platform: tapo_p100_control
    ip_address: 192.168.x.x
    email: !secret tapo_email
    password: !secret tapo_password

I have the same problem as mentioned above.
If the Tapo switch or light is offline at the time of starting HomeAssistsant services, the entities will show as 'entity not available' in the card I.E entity is removed from the system.
And will remain this way (even if the device goes back online) until the service is restarted again.
I also have Hive lights, and in this case they just show as 'unavailable' and the service does not need to be restarted once they come back online.
I understand why this is happening - because homeassistant does not continuously read the config file and try load entities that could not be found at the time of starting, but I just wonder if there is a way around this?


MM-44048: Fix Suggestion.Undefined

Summary
This PR fixes SUGGESTION.UNDEFINED in command provider and emoji provider. Also adds default Message as Suggestion for the missed cases
Ticket Link
Fixes https://github.com/mattermost/mattermost-server/issues/21358
JIRA: https://mattermost.atlassian.net/browse/MM-44048
Screenshots
Commands

before
after

Emojis

before
after

Release Note
Fixes an issue where suggestion dividers are displayed as undefined

Hello @sinansonmez,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

E2E tests not automatically triggered, because PR has no approval yet. Please ask a developer to review and then try again to attach the QA label.

Thanks @mickmister as your suggestion is to use suggestion.default, I removed types that I previously inserted for command related suggestions. Anyhow these cases will be handled with default message as seen below
https://github.com/mattermost/mattermost-webapp/blob/e3decea0fd146c094c2837abcf9231fc7512af30/components/suggestion/suggestion_list.jsx#L190-L202

Sorry for the confusion @mickmister . Taking into account @matthewbirtch 's comments. This should be ready for second round of review. Feel free to let me know if there are still some things to be fixed

This PR has been automatically labelled "stale" because it hasn't had recent activity.
A core team member will check in on the status of the PR to help with questions.
Thank you for your contribution!

LGTM. I just noticed the issue today and was happy to see a PR fixing it!

@mickmister please let me know if anything is pending related to your comments. Otherwise it is good to go

@Dhaussermann kindly pinging for review

/update-branch

Thanks @DHaussermann, yeah same here no more undefined in suggestions 🎉
Could you please merge the PR and close the issue?

@mickmister or @ashishbhate would be needed to perform the merge.

Thank you @sinansonmez !


Deps+Everywhere: Upgrade discord.js to v14

Additionally, every command now

uses the SlashCommandBuilder instead of directly providing JSON.
have their handlers properly typed.

Some parts of this commit are based on @bgianfo's work in #647.

You have no idea how happy I am to see this PR, thank you! :blue_heart: :green_heart: :purple_heart: :yellow_heart:


exploration: design a logo for the Aibyss

@yurijmikhalevich  review this if it looks good or not?

hello @yurijmikhalevich assign me this work :)

@yurijmikhalevich review this if it looks good or not?

is this looks good ? @yurijmikhalevich please answer me so we can put it as logo on site


[analyzer] optimize imports doesn't remove the complete statement

Start with two files, while enabling format and optimize imports in IntelliJ.
a. very_long_name_which_needs_to_span_multiple_lines.dart

class A{}

b. b.dart
import 'package:someflutter/very_long_name_which_needs_to_span_multiple_lines.dart'
    as a;

void f() {
  a.A();
}

Now comment a.A();

Expected:
The file is compilable.
Actual:
s a;

void f() {
  //a.A();
}

There is a reponse from the DAS:
1652779666587:Req:{"id"::"23","method"::"analysis.updateContent","params"::{
  "files"::{"C::\\use\\someflutter\\lib\\main.dart"::{"type"::"add",
  "content"::"s a;\n\nvoid f() {\n//  a.A();\n}\n"}}},
  "clientRequestTime"::1652779666584}

Dart SDK version: 2.18.0-120.0.dev (dev) (Mon May 16 13:04:31 2022 -0700) on "windows_x64"

There is a reponse from the DAS

Just to be clear, the line you gave from the instrumentation log records a request sent from the client (requests are always from the client) to the analysis server.
Earlier in the log file there should be a request to format the code followed by a response showing what server sent to the client as the formatted content of the file. If you can reproduce the issue and find those lines, then we'll know whether this is a bug in the formatter or somewhere else.


Fix typo in DateFormatter method dateFormat:fromTemplate

Surprisingly this is actually wrong on Darwin too: https://developer.apple.com/documentation/foundation/dateformatter/1408112-dateformat
As we try to match Darwin as closely as possible I don't think we should change this.

Might require @parkera to do another documentation fix first.

This is the internal name so it should be fine to change it. I'll just fix it up in the Foundation header for a future release.

Ah, now I remember why it is spelled this way. I believe it was an Objective-C++ compatibility hack, as template is a reserved keyword.

@parkera so do we want to close this, or would changing to a different name (e.g. tmplt) make sense?

Let's just leave this as-is.


Inherit from gym.Spaces

See this thread for information regarding the changes: #28
During your review please focus on test_dict.py and tuple.py.
The test case in the old tf and theano modules differs from the test case in the akro module. Please advise on which to keep.
Within the constructor for a Tuple it attempts to convert each component into a TF object and subsequently calls .dtypes on that object. For some reason, a TF object is never created (returns None) without any warnings or exceptions and I'm not sure why.

I've noticed much of the added functionality in each akro Space (e.g. flatten, unflatten, etc) currently exists in garage/src/garage/envs/util.py.  I'm assuming the intent is to use the implementations in each class rather than relying on util.py.  Is that correct?

yep -- we want to eliminate those and use utilities attached to the types instead

No worries.  Thanks for all the responses!

@ryanjulian @krzentner I think this is ready to be merged.  Please take one last look over it when you get a chance.

I don't believe there are any objections to merging this, so I'm going to merge it now.
Additional changes can be made in a later PR, if necessary.

It looks like @sud0nick pushed a merge of the original change and the rebased change, which explains the confusing history. I've squashed all of the changes together manually and pushed them back up.


Condense BSides events

Fixes #228. CC @koronkowy
Adds or updates the following companies, events, or universities:

Condense numerous BSides events into one

Checklist
Company updates

[ ] I have put the most recent relevant date in the "Last Update" column
[ ] I have linked to the article about the change, not the company's homepage (Please put N/A if there is no public post)

Event updates

[x] I have linked to the article about the change, not the event's homepage

University updates

[ ] I have linked to the article about the change, not the university's homepage

I believe this is a problematic change.
BSides events are all wildly different events and only loosely have a similar name / org.
Nothing that the core of BSides does effects how the events are run, and the events vary from small to large.
This is not like a DevOpsDays situation where they are more localized.
I understand that there are a lot of BSides, but knowing that BSides Charm for example is cancelled, does a disservice to the BSides that are happening. Ex. BSides Seattle is moved to October. They have no affiliation, overlap of staff, or otherwise.

Here's the link to where that information lies for BSides, so far: http://www.securitybsides.com/w/page/12194156/FrontPage

I understand that there are a lot of BSides, but knowing that BSides Charm for example is cancelled, does a disservice to the BSides that are happening. Ex. BSides Seattle is moved to October.

That's why I want to point to the BSides site rather than trying to list each one individually. Since there is a centralized list for all BSides events, it makes a lot more sense to link there to me.
Would it be better if we linked to the link in your second comment instead of the wiki page from the issue?


Feature request: Warn users when adding multiple values to dimension before flushing

Use case
In reference to this post, when a user adds multiple values to a single dimension before flushing metrics, only the last value set on the dimension wins.
Solution/User Experience
It would be nice if the metrics object would do a console.warn when multiple distinct values are used on a single dimension before flushing. Current behavior uses the last value added to the dimension.
Alternative solutions

Acknowledgment

[x] This feature request meets Powertools for AWS Lambda (TypeScript) Tenets
[ ] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

Hi @allenheltondev, nice to see you here!
I agree with the discussion you and @leandrodamascena had in the linked thread, we should emit a warning when this happens.
I have opened a PR to introduce the change, once merge it'll be included in the next release.


Support Symfony 4?

Hello.
Have you tried using this bundle with Symfony 4?
I tried install composer require codecloud/symfony-shopify-bundle:dev-master
but installation was interrupted with such a message
Your requirements could not be resolved to an installable set of packages.
Problem 1

Installation request for codecloud/symfony-shopify-bundle dev-master -> satisfiable by codecloud/symfony-shopify-bundle[dev-master].
codecloud/symfony-shopify-bundle dev-master requires symfony/framework-bundle ^3.0 -> satisfiable by symfony/framework-bundle[3.0.x-dev, ..., v3.4.5] but these conflict with your requirements or minimum-stability.

Are you going to support Symfony 4?
I think the main changes will be related to DependencyInjection and Security.
UPGRADE FROM 3.x to 4.0
Thanks!

Hi there!
I installed this library on symfony 4 successfully without 'dev-master'.
Who will do install must to do just - 'composer require codecloud/symfony-shopify-bundle'.
For author 'codecloud/symfony-shopify-bundle' respect!


Crypto.org

@Zaggim, I'm closing the issue as it seems unrelated to this project. If this is not correct, please provide more details about the issue you are facing and how to reproduce it


Deploy Control and 3 Client nodes via Chef in Packet

[x] Deploy packet nodes manually for control and client nodes
[x] Launch Chef driven OpenStack services on control node (non-ha)
[x] Launch Chef drive OpenStack client nodes (3 replicas)
[ ] Verify all network functions:
-- [ ] VM access to upstream (public service)
-- [ ] Access to VMs from upstream (via floating IP)
-- [ ] VM-VM access
[ ] Document process for delivering running system

What's left:

document and verify network functions
document process for delivering system
commit to folder
tag issues in commits, by adding ‘[#100]’ to the first message to link to issue #100

Primary docs for deploy in https://github.com/cncf/cnfs/blob/master/comparison/chained_nf_test/openstack/deploy_openstack/README.md


AttributeError: module 'tensorflow.python.framework.ops' has no attribute '_TENSOR_LIKE_TYPES'

I installed the latest version of keras and tensorflow on anaconda.
The problem looks like the same as the following and it is still awaiting a response
https://github.com/tensorflow/tensorflow/issues/38589

from keras.datasets import mnist
from keras.utils import np_utils
from keras.models import Sequential, Model
from keras.layers import Input, Dense, Dropout, Activation, Flatten
from keras.layers.advanced_activations import LeakyReLU
from keras.optimizers import Adam, RMSprop
import numpy as np
import matplotlib.pyplot as plt
import random
from tqdm import notebook

# Dataset of 60,000 28x28 grayscale images of the 10 digits, along with a test set of 10,000 images.
(X_train, Y_train), (X_test, Y_test) = mnist.load_data()

# Preprocessing

X_train = X_train.reshape(60000, 784)
X_test = X_test.reshape(10000, 784)
X_train = X_train.astype('float32')/255
X_test = X_test.astype('float32')/255

# Set the dimensions of the noise
z_dim = 100

# Optimizer
adam = Adam(lr=0.0002, beta_1=0.5)

g = Sequential()
g.add(Dense(256, input_dim=z_dim, activation=LeakyReLU(alpha=0.2)))
g.add(Dense(512, activation=LeakyReLU(alpha=0.2)))
g.add(Dense(1024, activation=LeakyReLU(alpha=0.2)))
g.add(Dense(784, activation='sigmoid'))  # Values between 0 and 1
g.compile(loss='binary_crossentropy', optimizer=adam, metrics=['accuracy'])

d = Sequential()
d.add(Dense(1024, input_dim=784, activation=LeakyReLU(alpha=0.2)))
d.add(Dropout(0.3))
d.add(Dense(512, activation=LeakyReLU(alpha=0.2)))
d.add(Dropout(0.3))
d.add(Dense(256, activation=LeakyReLU(alpha=0.2)))
d.add(Dropout(0.3))
d.add(Dense(1, activation='sigmoid'))  # Values between 0 and 1
d.compile(loss='binary_crossentropy', optimizer=adam, metrics=['accuracy'])

d.trainable = False
inputs = Input(shape=(z_dim, ))
hidden = g(inputs)
output = d(hidden)
gan = Model(inputs, output)
gan.compile(loss='binary_crossentropy', optimizer=adam, metrics=['accuracy'])

AttributeError                            Traceback (most recent call last)
 in
3
4 g = Sequential()
----> 5 g.add(Dense(256, input_dim=z_dim, activation=LeakyReLU(alpha=0.2)))
6 g.add(Dense(512, activation=LeakyReLU(alpha=0.2)))
7 g.add(Dense(1024, activation=LeakyReLU(alpha=0.2)))
~\anaconda3\envs\tf\lib\site-packages\keras\engine\sequential.py in add(self, layer)
164                     # and create the node connecting the current layer
165                     # to the input layer we just created.
--> 166                     layer(x)
167                     set_inputs = True
168             else:
~\anaconda3\envs\tf\lib\site-packages\keras\backend\tensorflow_backend.py in symbolic_fn_wrapper(*args, **kwargs)
73         if _SYMBOLIC_SCOPE.value:
74             with get_graph().as_default():
---> 75                 return func(*args, **kwargs)
76         else:
77             return func(*args, **kwargs)
~\anaconda3\envs\tf\lib\site-packages\keras\engine\base_layer.py in call(self, inputs, **kwargs)
444                 # Raise exceptions in case the input is not compatible
445                 # with the input_spec specified in the layer constructor.
--> 446                 self.assert_input_compatibility(inputs)
447
448                 # Collect input shapes to build layer.
~\anaconda3\envs\tf\lib\site-packages\keras\engine\base_layer.py in assert_input_compatibility(self, inputs)
308         for x in inputs:
309             try:
--> 310                 K.is_keras_tensor(x)
311             except ValueError:
312                 raise ValueError('Layer ' + self.name + ' was called with '
~\anaconda3\envs\tf\lib\site-packages\keras\backend\tensorflow_backend.py in is_keras_tensor(x)
693     ```
694     """
--> 695     if not is_tensor(x):
696         raise ValueError('Unexpectedly found an instance of type ' +     697                          str(type(x)) + '. '
~\anaconda3\envs\tf\lib\site-packages\keras\backend\tensorflow_backend.py in is_tensor(x)
701
702 def is_tensor(x):
--> 703     return isinstance(x, tf_ops._TENSOR_LIKE_TYPES) or tf_ops.is_dense_tensor_like(x)
704
705
AttributeError: module 'tensorflow.python.framework.ops' has no attribute '_TENSOR_LIKE_TYPES'

@ClearLens
There seems to be some weird compatibility issues between keras and tensorflow.keras. So please try with tensorflow.keras instead.
from tensorflow.keras.datasets import mnist
from tensorflow.python.keras.utils import np_utils
from tensorflow.keras import Sequential
from tensorflow.keras import model
from tensorflow.keras.layers import Input, Dense, Dropout, Activation, Flatten
from tensorflow.keras.optimizers import Adam, RMSprop

Thanks!

I got same error. After I tried with tensorflow it worked.
Tensorflow version : '2.3.1'
Tensorflow.keras version : '2.4.0'
Thanks.

@ClearLens
There seems to be some weird compatibility issues between keras and tensorflow.keras. So please try with tensorflow.keras instead.Also, please let us know Tensorflow version you are using?
from tensorflow.keras.datasets import mnist
from tensorflow.python.keras.utils import np_utils
from tensorflow.keras import Sequential
from tensorflow.keras import model
from tensorflow.keras.layers import Input, Dense, Dropout, Activation, Flatten
from tensorflow.keras.optimizers import Adam, RMSprop

Thanks!

Replace 'keras' by 'tensorflow.keras' worked for me today.


System audit table fields are only usable from BC17 onwards, but VS Code / AL do not error when targeting BC15...!! SystemCreatedAt, SystemCreatedBy, SystemModifiedAt, SystemModifiedBy

According to here, the fields SystemCreatedAt, SystemCreatedBy, SystemModifiedAt, SystemModifiedBy - are only usable from BC17 onwards.
So we merrily used them... in a project targeting BC15+. Which builds fine, and of course installs fine against later BCs as we tend to develop against.
Then, when we go to install that to a BC15 customer...

Please fix this, so that the AL extension will error if we are referencing any system/audit fields that aren't available in the minimum runtime/platform version we have specified in app.json. Please also check if related methods like RecordRef.SystemCreatedAtNo() also need to be guarded in the same way - I imagine they do.
Thank you!
Name: AL Language
Id: ms-dynamics-smb.al
Description: AL development tools for Dynamics 365 Business Central
Version: 9.5.674382
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-dynamics-smb.al

Hi, I can't seem to reproduce this. Audit fields were indeed shipped with BC17 or runtime version 6.0, so when I target for example runtime 4.0 (corresponding to BC15) I do get an error specifying that such fields are not defined for a table on VSCode.
Is there any other information that you can provide to reproduce this issue - like which runtime version are you targeting in the app.json or a code example?

Thanks for looking, I appreciate it!
What error did you get, when you do what? :-)
Yes, this is still a problem AFAICS,: Latest AL targeting runtime 4.0 does not warn that I'm adding those fields to pages, when the server will be unable to compile said page:

Name: AL Language
Id: ms-dynamics-smb.al
Description: AL development tools for Dynamics 365 Business Central
Version: 10.2.719883
Publisher: Microsoft
VS Marketplace Link: https://marketplace.visualstudio.com/items?itemName=ms-dynamics-smb.al

Hi, we have internally discussed this issue and have decided not to fix at the moment to prioritize other issues. The problem is that this is a form of backwards compatibility that we do not currently support.
Right now we (try to) guarantee that code that used to compile will continue to compile with newer compilers. I.e. a new compiler will not break existing app code.
This however is a different form of backwards compatibility: that code that used to not compile will continue to not compile. The cost of implementing this is too high to be considered at the moment. For now, the recommendation to use the compiler that is shipped with the version you are building for if you require strict compatibility is probably the best workaround.
As usual, you can raise this on BCIdeas though. If there's enough support for this in the partner community, that would add weight to us being able to prioritize the resources required to do this.


Changed definition of field trashed to allow be null

Allow trashed to be null.

Codecov Report

Merging #24 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #24   +/-   ##
=========================================
  Coverage     48.12%   48.12%
  Complexity       21       21
=========================================
  Files             3        3
  Lines           133      133
=========================================
  Hits             64       64
  Misses           69       69

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 b8fc9e5...02b60ac. Read the comment docs.


In the Github Sims actions, stop running the sims with badgerdb.

Summary
Remove badgerdb from the matrices in the sims github actions/workflow/jobs.
Problem Definition

The runsim jobs often take 15 minutes longer on badgerdb than on the others. So removing badgerdb Those are the last jobs that finish. So removing badgerdb from them will noticeably speed up completion of all checks. not actually true. The cleveldb ones are the longer ones. The badgerdb ones take 20-30 minutes, while cleveldb takes 40 to 50.
The go-test-multi-db jobs run on both mac and linux. The mac runners are limited to 5 for the entire repo (possibly org). So those often end up getting queued up, and we have to wait longer on them. In those tests, there are 2 tests and per OS per db type. Currently there are 3 db types. So there are more tests (2*3=6) for Mac than we can have runners running at once. So removing badgerdb from that matrix will help things complete sooner too (much less waiting on queued jobs).
No one is using badgerdb for provenanced. And tendermint is heavily considering removing support for it (and most likely will).
There's some added complexity in those jobs that can then be removed because we no longer have to build and install badgerdb. not actually true.

Proposal
Remove badgerdb from the runsim and go-test-multi-db job matrices defined in .github/workflows/sims.yml.

For Admin Use

[ ] Not duplicate issue
[ ] Appropriate labels applied
[ ] Appropriate contributors tagged
[ ] Contributor assigned/self-assigned

I've update the problem definition with a couple findings.

The longest running ones are actually runsim with cleveldb. Those take 40-50 minutes. The runsim with badgerdb ones take about 20-30 minutes.
There isn't any added complexity for the badgerdb tests. Unlike rocksdb and cleveldb, it doesn't need a shared library.

Since the runsim tests only run on a linux runner, and they're not the longest running ones, it's probably okay to keep badgerdb in that matrix. We still want to remove it from the go-test-multi-db matrix though to free up those mac runners.

After a bit of discussion, we think it's fine to only run the sims using goleveldb, and also not run them on a mac runner. That should significantly speed up overall test run and remove the mac runner bottleneck from more important jobs that need a mac runner (e.g. the bulid/release).


Allow the use of '--load' without an error

If you used the arguments '--load [name]' it would respond with 'cannot use --choose with --load'. This fixes that.

Confirmed working, solves #19


Adding a table of exercises

I removed the "vertical" column because I don't think it add a lot of information and less is better. What do you think?

Definitely, less is more. Luz verde!


Forked Repos don't allow actions to run

Check out https://github.com/meltano/meltano/pull/6314
From chatting with @WillDaSilva we found https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks , and https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target
It looks like we need to implement the pull_request_target event in our github actions.
What's a bit confusing to me is that I would think I could run the actions on the AutoIDM fork without much issue to Meltano (as I wouldn't have access to any of the secrets from Meltano's repo)

Maybe caused by https://www.githubstatus.com/  ?

Was caused by the GitHub Actions outage.


[Brownies] Fix [botname] in cog docstring

Type

[x] Bugfix
[ ] Enhancement
[ ] New Feature

Description of the changes
bleh

Thank


Return error in Runtime::migrate method

Overview

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 85.165% when pulling 2aca9075956e9ccb25c8e88b7f269a2e14aded89 on vitvakatu:return-error-for-migration into 56fc548f336416c49b8eee31e89f1cf74051a9df on exonum:master.


Add GraphQL package that sits on top of Aqueduct/ORM

Looking for using  Aqueduct with trending Graphql :bulb:  will be more awesome! any samples available?
https://github.com/angel-dart/graphql
https://github.com/hourliert/graphql_client
***** https://thosakwe.com/new-features-coming-to-angel-in-version-2-0-0/ *****

There aren't any plans to add a GraphQL layer to Aqueduct, but I'd love to see a package that does it - the ORM has a lot of the features needed to support it. I'll leave this open if anyone is interested and I personally would be interested in supporting it.

Dear @joeconwaystk
Thank you for creating such robust framework for server side.
I think It would be good to create graphql
dependency.

The only source for Dart implementation of GraphQL spec is at angel mono repo as graphql as the mentioned graphql_client is only the client side (as far as I know).
Best course of action would be relying on angel graphql package or create/use standalone package like zino-app/graphql.

any new news?


Add support for rate limiting on core queues

Core queues leveraging priority queues have improved handling higher priority messages accordingly, solving the prioritization problem from the controller perspective, however, switches that are managed by kytos have their own implementation on how to handle messages, so to mitigate entirely starvation on switches as well the controller can't overwhelm them by sending too many messages, that's why rate limiting is needed on core queues (this issue here) and on Flask endpoints (issue #225).
The proposed solution that will be further discussed in this thread in the future must be compatible with the current queues infrastructure that have been delivered on PR #226 and #235. Configuration options and granularity of rate limiting will also be assessed, currently we know that from NApps perspective FlowMods can generate significant spikes 200+ in a few seconds.

We could use this package to implement the ratelimiting, then make a subclass of KytosEventBuffer that uses the ratelimiter.

Landed on
https://github.com/kytos-ng/kytos/issues/454
https://github.com/search?q=org%3Akytos-ng+is%3Aissue+label%3Aepic_rate_limit&type=issues
Now we've bot core building blocks for pacing soft rate limiting, at the moment, they're using to limit flow mods, but then can be used in other places where it makes sense too.


charts version upgrade of jenkins-x/exposecontroller-service

change jenkins-x/exposecontroller-service to version 1.0.7

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: rawlingsj
If they are not already assigned, you can assign the PR to them by writing /assign @rawlingsj in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Hi @tokeskovgaard. Thanks for your PR.
I'm waiting for a jenkins-x member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.


Mj-section_del_one_aside

mj-section_del_one_aside
A "warning" aside is duplicated early in packages/mjml-section/README.md. This deletes one of them.
Further, this shortens the wording in that aside. It removes awkward wording and clarifies.
It gets the same message across better.
No change to any JavaScript file.

Flawed.


Remove old logs and tempfiles

Better matches Rails' default bin/setup:
https://github.com/rails/rails/blob/v4.2.0/railties/lib/rails/generators/rails/app/templates/bin/setup#L11

I'm all for consistency, but I didn't realize that 1) bin/setup should be run more than once, and 2) it should delete files. With a -f, even!
Overall, this doesn't bother me, but I do find (1) and (2) surprising, and I follow the POLA.

bin/setup should be run more than once
Yup, it's been idempotent for awhile. Things like the db seeds or ENV variables in .sample.env can change. I find myself running bin/setup on projects a few times a week, after pulling new code, as long as I don't have development data I'm currently working with.

It may not be surprising to me because I think of "setup" as "I'm setting myself up for this programming session I'm about to sit down for." rather than "I'm setting up this project for the first time and I'll never need to think about seeds and ENV variables again."

it should delete files. With a -f, even!

Yeah, I can see the concern with that.
I actually don't feel strongly about this one way or the other. Saw it in the default Rails bin/setup and wanted to put it up for discussion.

I know that bin/setup is idempotent, but I run it once per project unless one of the other devs specifically asks me to run it again because they changed something.
I'm not against this, but I also don't think this gains us much - is it so bad to have old logs and tmp files?

Yeah, I don't feel strongly. I think I'd rather not maintain this code or open up possibility for breakages. Closing without merging.

For what it's worth, I also run bin/setup regularly, and my answer to many local development question is "did you try running bin/setup again?"


fix: don't animate initially if defaultOpen is set to true

Hi, I believe this has potentially added another issue where the animation is happening after change the page width. After loading, if I then pull the page in from the side, I'm seeing the animation fire. Is anyone else experiencing this?


about baseline in paper

Hi, thanks for your works. When reading your paper, i have few questions about the baseline:

Baseline is build based on CenterNet with an extra head predicting tracking offset map O_B. As mention in #36 , backbone network just take a single image (frame at timestamp t) as input instead of previous_frame + current_frame + previous_detection_heatmap like CenterTrack? So I wonder how to predict offset map O_B with just one frame. My initial guess is that feature map of frame t and t-T is calculated independently, but attaching offset-map head on an aggregated feature map (e.g simple stack/average/subtraction) of t-T and t?
In Fig. 4, since Baseline doesn't have O_C (w/o CVA), does it means Baseline == Baseline + CVA or O_C == O_B in the baseline context?

Thanks for your interest! The baseline in the paper should refer to just adding a simple tracking head on CenterNet. The O_B in baseline should be predicted from the single current image only.


Option to hide 'Implement missing function' option

Any plans to add an option to remove this option from quick fix?

I have set Enable Code Actions to false, but that doesn't seem to hide this option.

I thought, that's crazy, too!

This is from the TypeScript extension: https://github.com/microsoft/vscode/blob/613ad56601c62ff8fea04a62444217470fbc496d/extensions/typescript-language-features/src/languageFeatures/quickFix.ts#L380-L381

Do we need to open a thread there?

Yes, please open an issue there. Also make sure to explain why you want to disable it


[PR][CLOSED] Activitated CODEOWNERS for automated review list

 Issue by geunsik-lim
Monday Jul 16, 2018 at 04:52 GMT
Originally opened as https://github.sec.samsung.net/STAR/TAOS-CI/pull/129

This commit is to activitate the CODEOWNERS template of github.sec.samsung.net.
Changes proposed in this PR:

Activate CODEOWNERS template

Signed-off-by: Geunsik Lim<EMAIL_ADDRESS>
[Template] PR Description
In general, github system automatically copies your commit message for your convenience.
Please remove unused part of the template after writing your own PR description with this template.
$ git commit -s filename1 filename2 ... [enter]

Summarize changes in around 50 characters or less

More detailed explanatory text, if necessary. Wrap it to about 72
characters or so. In some contexts, the first line is treated as the
subject of the commit and the rest of the text as the body. The
blank line separating the summary from the body is critical;
various tools like `log`, `shortlog` and `rebase` can get confused
if you run the two together.

Further paragraphs come after blank lines.

**Changes proposed in this PR:**
- Bullet points are okay, too
- Typically a hyphen or asterisk is used for the bullet, preceded
  by a single space, with blank lines in between, but conventions vary here.

Resolves: #123
See also: #456, #789

**Self evaluation:**
1. Build test: [ ]Passed [ ]Failed [*]Skipped
2. Run test: [ ]Passed [ ]Failed [* ]Skipped

**How to evaluate:**
1. Describe how to evaluate in order to be reproduced by reviewer(s).

Add signed-off message automatically by running **$git commit -s ...** command.

$ git push origin <your_branch_name>

geunsik-lim included the following code: https://github.sec.samsung.net/STAR/TAOS-CI/pull/129/commits

 Comment by git-bot-sec
Monday Jul 16, 2018 at 04:52 GMT

Thank you for submitting PR #129. Your PR has to pass all verificiation processes of cibot before getting a review process from reviewers. We recommend that you read instruction manual in . If you want to monitor the progress status of your PR in detail, visit http://aaci.mooo.com/.

:octocat: cibot: Thank you for posting issue #153. The person in charge will reply soon.

 Comment by git-bot-sec
Monday Jul 16, 2018 at 04:52 GMT

:octocat: cibot: @STAR, Oooops. The commit has an invalid executable file. The file is .github/CODEOWNERS. Please turn the executable bits off. Run chmod 644 file-name command.


Add log trance command

Context
Add log trance command. This command will be used to trace the log of the target smart function.
Usage: jstz log trace [OPTIONS] <SMART_FUNCTION>

Arguments:
  <SMART_FUNCTION>

Options:
  -f, --format <FORMAT>
  -l, --level <LEVEL>
  -s, --search <SEARCH>
  -h, --help             Print help

Manually testing the PR
Ran the command and confirmed that the args get parsed properly
cargo run log trace myaddress -f pretty -s sometext

There seems to be a conflict with jstz trace PR.

Closing in favour of #79


Support regex in redirections

Context
Documentation projects have to write long redirection files when a folder is moved. Exactly, one new line per page changed.
In this situation, most times redirections are only added for the most relevant pages moved, and the rest are left with a 404 errors.
Proposal
Allow a subset of regular extensions in the redirections file. By doing so, Sphinx could redirect all files that follow a similar pattern (e.g. all documents under a folder) to the same URL using one line. For example, this could look like:
faq/*: "https://website.com/forum/faq/$source"

Implementation
Research if the extension https://documatt.gitlab.io/sphinx-reredirects/usage.html#source-wildcards is a good fit for the project.

GitHub pages only supports static pages. There is no way to make it execute server-side code and thus it's not possible to create regex re directions.


Dependency Installation

Dependencies to install regex:
yum install -y gcc
yum install -y python-devel

If you are using alpine (docker), you will need the following:
apk add gcc python3-dev musl-dev


fix ncat sctp test and update CI image

see commits

@edsantiago I hope this unblock us hopefully

LGTM once it passes CI

@edsantiago @mheon PTAL again

My bad.
/lgtm


Stop auto startup

It's really annoying to start Dockstation up and have multiple projects auto start.
Please make auto start an option and not the default.

@sonicviz did you ever solve this? i would like to have a certain project start when dockstation starts.  since that doesn't seem possible what was causing your containers to start?

Sorry, never did solve it.


PLATFORM-6406 hooks: avoid firing post registration hooks multiple times

https://fandom.atlassian.net/browse/PLATFORM-6406
We could revert this change after webhooks are fixed

It introduces a bug. Fixed in https://github.com/Wikia/kratos/pull/38


Stuck Dark Theme Switch To The Top Left & Added Hover Effect To Contributor Cards

PR Description 📜
I have fixed the dark mode to the top left and made cursor pointer for it.
I have added hover effects for each contributor card.
Preview
Dark Mode Switch Button

Contributor Hover Effect

Fixes #2772

Mark the task you have completed ✅

[X] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[ ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ ] I have added screenshot for website preview in assets/images
[ ] I have added entries for my game in main README.md
[ ] I have added README.md in my folder
[ ] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @5h0ov ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Thank you  @5h0ov , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


mass.play_media failing on radio

What version of Music Assistant has the issue?
2.0.0b144
What version of the Home Assistant Integration have you got installed?
2024.4.0
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
Using the mass.play_media automation, fails when trying to start a radio. The error generated is:
Invalid command: music/radio/library_items
The error only happens when using the automation to start a radio, and starting an artist, an album, a track, or a playlist works perfectly. The radio is a TuneIn provided radio.
Already tried troubleshooting steps:

uninstall and reinstall the addon
check the name of the radio in the automation
playing the radio from the MA GUI (works)
restarting HA
simplifying the automation and running only the mass.play_media portion manually

The last time I tried running this automation (successfully) is maybe 2 weeks ago. I have migrated my HA from a raspberry pi to an x86 machine since. I'm running HAOS and have followed the guide for installation.
How to reproduce

Create an automation that runs the following automation (code generated from the GUI):

service: mass.play_media
metadata: {}
data:
  media_type: radio
  media_id: Radio Swiss Pop
target:
  entity_id: media_player.living_room_hifi

Run the automation.
Music Providers
TuneIn: Does not work
I do not have other radio providers.
Player Providers
Snapcast: Works all the time during all my tests (except when the error is generated, but I don't think it's the issue)
Full log output
Here is the log generated by me running the automation 4 times in a row, once for an artist, an album, a playlist, and finally for a radio:
2024-05-04 09:37:56.048 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue living-room-hifi - crossfade: False
2024-05-04 09:38:15.331 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue living-room-hifi - crossfade: False
2024-05-04 09:38:43.204 INFO (MainThread) [music_assistant.streams] Start Queue Flow stream for Queue living-room-hifi - crossfade: False
2024-05-04 09:38:54.281 WARNING (MainThread) [music_assistant.webserver] Invalid command: music/radio/library_items

Additional information
Please let me know if I can provide any more information. I have tried looking at the problem from all the angles I can think of. If I can support in any way, please let me know.
Thank you for making this wonderful piece of software.
What version of Home Assistant Core are your running
2024.5.1
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

@sp0ngeboobs tunein as well or radio browser?

@sp0ngeboobs are you using a manually added radio station?

I tried it with the same radio stream URL as @sp0ngeboobs :

I can add the stream as a radio.
It plays perfectly when started from the MA UI.
When I pass the direct link to mass.play_media it plays without issues.
When I only give it the name, it logs the same error as above: Invalid command: music/radio/library_items

If I can help narrow the issue further, let me know :)

Same issue with me:
service: mass.play_media
target:
  entity_id: media_player.front_room_speakers_mass
data:
  media_id: More FM Canterbury
  media_type: radio

Failed to call service mass/play_media. Invalid command: music/radio/library_items

Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:239
integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 7:36:21 AM (14 occurrences)
Last logged: 10:55:25 AM

[281472083526336] Unexpected exception
[281472083589952] Unexpected exception
[281472098671168] Unexpected exception
[281471966229824] Unexpected exception
[281472219492800] Unexpected exception
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 239, in handle_call_service
    response = await hass.services.async_call(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2738, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2779, in _execute_service
    return await target(service_call)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 975, in entity_service_call
    single_response = await _handle_entity_call(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 1047, in _handle_entity_call
    result = await task
             ^^^^^^^^^^
  File "/config/custom_components/mass/media_player.py", line 510, in _async_play_media_advanced
    if item := await self._get_item_by_name(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/config/custom_components/mass/media_player.py", line 561, in _get_item_by_name
    result = await func(search=searchname)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/music_assistant/client/music.py", line 341, in get_library_radios
    await self.client.send_command(
  File "/usr/local/lib/python3.12/site-packages/music_assistant/client/client.py", line 176, in send_command
    return await future
           ^^^^^^^^^^^^
music_assistant.common.models.errors.InvalidCommand: Invalid command: music/radio/library_items

The service works in b143, but not current b144
Radio is added as a favorite from TuneIn:

Fixed in the latest integration version 2024.5.0


Feedback attribute breaks InputGroups

Hi, I just noticed that adding a feedback attribute for validation purposes to an Input breaks it from its suffix element in an InputGroup.
I have noticed that changing the order of the elements fixes the issue.
Generated HTML (broken) :

    <input id="totalWeight" class="form-control is-invalid" type="text" name="" placeholder="100">

Ce champ est obligatoire

    kg

Working HTML (fixed by swapping div.invalid-feedback and span.input-group-text) :

    <input id="totalWeight" class="form-control is-invalid" type="text" name="" placeholder="100">
    kg

Ce champ est obligatoire

Ah thanks @Meknassih - hmm that's a tough one to fix since the feedback would be rendered by the Input, and can't split the elements like that.  Am assuming this is the best workaround for now:
<InputGroup>
    <Input id="totalWeight" invalid placeholder="100" />
    <InputGroupText>kg</InputGroupText >
   <FormFeedback>Ce champ est obligatoire</FormFeedback>
</InputGroup >

I'll look and see if some better option, thanks for the heads up.

Ah thanks @Meknassih - hmm that's a tough one to fix since the feedback would be rendered by the Input, and can't split the elements like that.  Am assuming this is the best workaround for now:
<InputGroup>
    <Input id="totalWeight" invalid placeholder="100" />
    <InputGroupText>kg</InputGroupText >
   <FormFeedback>Ce champ est obligatoire</FormFeedback>
</InputGroup >

I'll look and see if some better option, thanks for the heads up.


Create Bot2RedForReal, updated  autonomous for mecanum driving

Before issuing a pull request, please see the contributing page.

Testing comments


Subscription eoseTimeout causes premature EOSE

Suppose I have nostr.mutinywallet.com in my relay list. This relay sends EOSE immediately on any sub. So eoseTimeout fires very early and cuts data coming from other relays. eoseTimeout should probably fire only if there was X ms from last event on this sub, not from last eose - this should handle the dead relays, but still not cut the data.

this was fixed on the NDK 1.0 release
now it expects a minimum number of relays to EOSE (iirc 50%) before timeouts are considered and then the timeout is reduced by the number of relays that are missing an EOSE.
All timeouts are ignored while events for the subscription are being received.
And subscriptions set to close on EOSE are closed at the relay level when the relay has sent the EOSE, instead of the previous functionality of waiting for the relay set to EOSE before closing subscriptions at the relay level.


ashm_multi_rainbow_plot broken?

The example for this function in our vignette produces just a very busy legend.

I think this is related to ggplot changing the behaviour of scale_X_manual functions, as per:
https://community.rstudio.com/t/scale-colour-manual-does-not-drop-unused-levels/109094/4

Confirmed that installing ggplot2 from github resolves this. Not really an ideal solution.

I have added  a solution for this in #73 that will always sunset colors to the keys actually present in the data ✅


HTTP Server fails to start

Using a raspberry pi 3, i can't get the server to run. Appimage, or compile from source always results in the same error.
pi@[REDACTED]:~/browservice/release/bin $ ./brow* --http-listen-addr=<IP_ADDRESS>:80 --vice-opt-http-auth=[REDACTED]:[REDACTED]
WARNING @ src/config.cpp:241 -- The command line option --http-listen-addr is a deprecated alias for --vice-opt-http-listen-addr
INFO @ src/main.cpp:200 -- Loading vice plugin retrojsvice.so
INFO @ src/main.cpp:207 -- Initializing vice plugin retrojsvice.so
INFO @ retrojsvice.so src/context.cpp:281 -- Creating retrojsvice plugin context
INFO @ src/xvfb.cpp:64 -- Starting Xvfb X server as child process
INFO @ src/xvfb.cpp:135 -- Xvfb X server :0 successfully started
INFO @ retrojsvice.so src/context.cpp:331 -- Starting plugin
INFO @ retrojsvice.so src/http.cpp:620 -- Starting HTTP server (listen address: <IP_ADDRESS>:80)
PANIC @ retrojsvice.so src/http.cpp:625: Starting Poco HTTP server failed with exception: I/O error: Permission denied Aborted
Any idea what I might be doing wrong?

You get the error because by default, only root can listen to privileged ports (that is, ports in range 1-1023). This is why all the examples use port 8080 instead (for security reasons, you should never run browservice as root). If you really want to use port 80 for Browservice, there are a few options:

You can use authbind to give your user permission to listen to that port.
You can listen to a non-privileged port and forward port 80 to it (for example by using iptables).

@ttalvitie I tried using authbind with this command authbind --deep ./browservice-v<IP_ADDRESS>-armhf.AppImage --vice-opt-http-listen-addr=<IP_ADDRESS>:666 (also tried to manually set the depth), and got the same error with no difference). higher ports work fine though, so thanks for helping me already

You get the error because by default, only root can listen to privileged ports (that is, ports in range 1-1023). This is why all the examples use port 8080 instead (for security reasons, you should never run browservice as root). If you really want to use port 80 for Browservice, there are a few options:

You can use authbind to give your user permission to listen to that port.
You can listen to a non-privileged port and forward port 80 to it (for example by using iptables).

Thanks for clarifying this. Marking as closed.


WIP - Add option for measurement frame range

Refactor #553 to elevate logic from FpsInfo and bring up to date with dev.  From @tomped01's original PR:

This patch adds the option to specify start and stop frames for
secondary FPS measurements. In addition, options for aborting replay
after the end of the measurement range is reached and for flushing all
work at range boundaries have been added.
The framerange support is very useful when investigating performance for
large traces as creating new trimmed variants for every scene in the
trace can be very time consuming (and quickly becomes unmaintainable for
large sets of traces).
The abort option is very handy on slower development platforms where
replay of relatively small traces can take hours, this allows for
skipping work one might not be interested in for a particular run.

CI gfxreconstruct build queued with queue ID 13882.

CI gfxreconstruct build # 1352 running.

CI gfxreconstruct build # 1352 failed.


fix(windows): fix crash in getIpAddressSync

Overview
Even if not properly documented, some properties of windows.networking.connectivity.connectionprofile can throw. It's the case of NetworkAdapter since it's then calling check_hresult which will throw if the returned hresult is an error one.
Since RNCNetInfo::getIpAddressSync is marked as noexcept, any exception thrown in this method will call std::terminate.
This commit is to return "unknown" in this case.
Test Plan
Unfortunately, I wasn't able to reproduce the crash some of our users face myself so I can't validate it fixes it.
Though I imported these changes in our own RN Windows project and made sure IP info is still working in the general case!
For reference, here is the stack trace we get in our crash reporting system:
RNCNetInfoCPP!__GSHandlerCheck_EH4+0x64 [D:\a_work\1\s\src\vctools\crt\vcstartup\src\gs\amd64\gshandlereh4.cpp @ 86]
RNCNetInfoCPP!winrt::throw_hresult+0x200 [node_modules@react-native-community\netinfo\windows\RNCNetInfoCPP\Generated Files\winrt\base.h @ 4688]
RNCNetInfoCPP!winrt::check_hresult+0x448<EMAIL_ADDRESS>@ 111] [inlined in RNCNetInfoCPP!winrt::ReactNativeNetInfo::implementation::getIpAddressSync+0x4a0<EMAIL_ADDRESS>@ 111]]
RNCNetInfoCPP!winrt::ReactNativeNetInfo::implementation::RNCNetInfo::GetNetworkStatus$_ResumeCoro$1+0x75c<EMAIL_ADDRESS>@ 191]
RNCNetInfoCPP!winrt::ReactNativeNetInfo::implementation::RNCNetInfo::GetNetworkStatus$_InitCoro$2+0x194<EMAIL_ADDRESS>@ 15732480]
RNCNetInfoCPP!winrt::ReactNativeNetInfo::implementation::RNCNetInfo::GetNetworkStatus+0x44<EMAIL_ADDRESS>@ 15732480]
RNCNetInfoCPP!<lambda_07af3bc3acbb5d7aa0533ab2f666cdc6>::operator()+0x78<EMAIL_ADDRESS>@ 147]
RNCNetInfoCPP!winrt::impl::delegate<winrt::Windows::Networking::Connectivity::NetworkStatusChangedEventHandler,<lambda_07af3bc3acbb5d7aa0533ab2f666cdc6> >::Invoke+0xc [node_modules@react-native-community\netinfo\windows\RNCNetInfoCPP\Generated Files\winrt\Windows.Networking.Connectivity.h @ 791]

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


cannot remove reaction via menu

re Microsoft/vscode#69321

add a reaction from the menu, reaction is added to comment
select the same reaction from the menu again
:bug: reaction isn't removed

This has been fixed, no longer repros


Cannot delete an Entry

Just going through the docs here and I can't see any examples of the SDK deleting Entries despite it being supported by the REST API. I'm searching things like "Delete", "Delete entries", "Deleting an entry", "Deleting entries".
Can anyone advise of a workaround or point me to the correct piece of documentation if this is indeed possible.

Closing, need to use the management API, not delivery API.


Merge master into qa

This should allow the GitVersion build changes to get into the qa build (and eventually the live build once we merge qa into live).

This change is

LGTM

Reviewed 17 of 17 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Add something similar to DecodeResult.NOT_FOUND to ServiceInvocationHandler

A ServiceCodec can tell the session layer that the request doesn't actually belong to the enclosing Service so that the other Service takes over, by returning DecodeResult.NOT_FOUND.
It will be nice if ServiceInvocationHandler has something similar. For example, a user could bind an HttpFileService and a TomcatSerivce under the same path prefix. If HttpFileService fails to find the file, TomcatService could handle it, making it possible to serve the static resources using Armeria. (TomcatService could also handle the 404 response from Tomcat specially so that the next service takes over.)
/cc @delegacy

Agreed. However, I'm not sure how to generalize a trigger to take over requests for the next service.

This issue will require two following changes:

Make DecodeResult.NOT_FOUND not fail immediately. Let HttpServerHandler try again until it tries all matching services.
Define a mechanism for ServiceInvocationHandler that does the same job with DecodeResult.NOT_FOUND.

Perhaps we could define some exception type? We have ServiceNotFoundException already, so we could use it for this purpose. It's actually an exception the invocation promise will get when HttpServerHandler gets DecodeResult.NOT_FOUND, so it seems to make sense to me.
Another issue to consider is the case where a service wants to tell the session layer that the path does not exist and it does not want other services to take the request over. However, I'm not sure if we should consider this because usually they will just end up hitting the catch-all path mapping or making HttpServerHandler return 404.

Another issue to consider is the case where a service wants to tell the session layer that the path does not exist and it does not want other services to take the request over. However, I'm not sure if we should consider this because usually they will just end up hitting the catch-all path mapping or making HttpServerHandler return 404.

I think we need to consider the case when a user does want the last service to catch 404 and response with the custom error page. In this case, propagation of 404 should not be done.

This actually was fixed in #60 slightly differently.


Changing response in domain throws error Detected mismatch in domain specification

Rasa version: 1.9.5
Rasa SDK version (if used & relevant):
Rasa X version (if used & relevant):
Python version: 3.7
Operating system (windows, osx, ...): osx
Issue:
Ive been changing a response to include custom json but whenever I make edits and re-train the model then run rasa shell, I get errors about a mismatched domain.
Error (including full traceback):
  File ".../lib/python3.7/site-packages/rasa/core/domain.py", line 756, in compare_with_specification
    f"Domain specification has changed. "
rasa.core.domain.InvalidDomain: Domain specification has changed. You MUST retrain the policy. Detected mismatch in domain specification. The following states have been
         - removed: slot_webex_device_3,slot_device_type_5
         - added:

Command or request that led to error:
rasa shell

Content of configuration file (config.yml) (if relevant):

Content of domain file (domain.yml) (if relevant):

Thanks for the issue, @JustinaPetr will get back to you about it soon!
You may find help in the docs and the forum, too 🤗


bindings: counter: extend binding with channel specifier

Introduction
(This is an RFC that follows from the following PR: https://github.com/zephyrproject-rtos/zephyr/pull/77650)
Make it possible to define the channel in the device tree when using a hardware counter.
Problem description
When a device tree node uses a hardware counter as a property, it is not possible to specify the channel for this counter. This makes it that the channel needs to be hardcoded in the driver implementation, which limits the portability of the driver.
Proposed change
Add a counter.yaml file that specifies a #counter-cells property. This can than be included and used to add the channel to the specifiers. A proposal can be found in the following commit: https://github.com/zephyrproject-rtos/zephyr/pull/77650/commits/6efd291435ed373da704d9526eb3b52d218dd489. Also extend all device tree nodes that describe a hardware counter to specify #counter-cells.
Detailed RFC
Include the_counter.yaml_ file in each vendors counter.yaml file, and add the channel  counter-cells. Each device tree node that defines a hardware counter needs to be adapted to specifiy the number of counter-cells (#counter-cells = <1>;). I already adapted the device tree nodes from st in this commit: https://github.com/zephyrproject-rtos/zephyr/pull/77650/commits/f6e9d3361964e361ec9dcabac4178d95ab3c1e6d
Proposed change (Detailed)
When a driver needs to access to one or more hardware counters, it can benefit from using different channels on the same hardware counter (e.g., using counter_api_set_alarm). As of today, these channels need to be hardcoded in the driver. It would however be beneficial to define these channels in the device tree. This would increase the drivers portability.
Simple usecase
Imagine you want to design a full-duplex bitbanged transeiver using hardware timers. Your device tree node could then look something like the following:
bitbang_bus: bitbang-bus {
	status = "okay";
	compatible = "my,compatible"
	baudrate = <15000>;

	rx-gpios = <&gpio0 0 0>;
	tx-gpios = <&gpio0 1 0>;
	counters = <&counter0 0>, <&counter0 1>;
}

The driver will consist of 3 main parts:

A tx counter callback, scheduled periodically to toggle the tx gpio for transmission
A rx gpio interrupt, used to detect the start of incoming data (neglecting clockdrift for simplicity). This interrupt starts the rx counter so it is triggered at the desired sample point
A rx counter callback, sampling each time it is triggered, and rescheduling itself for the next data symbol.

The main advantage of adding the channels as counter-cells, is that when a hardware counter supports multiple channels, these can be specified in the device tree, instead of in the driver itself, increasing portability.
A real life scenario would be a Dali bitbanged bus, like described here (https://github.com/zephyrproject-rtos/zephyr/issues/71544). Using busy waits will block the cpu for too long of a time due to the low baudrate of dali (1200).
Concerns and Unresolved Questions
A large portion of the tree will be affected, as all device tree nodes describing a hardware counter need to be extended.

Looks good, and gpio binding is similar, but it uses gpio-controller.yaml. But this requires more discussion, that is, waiting for a response from the maintainer of this subsystem.


Download scan report does not work

On 2.5-head - commit id: db350a63

Install CIS v2 app
Run RKE permissive scan on the cluster.
Clicking on Download Scan report - No API call is made.
Error in console:

This should be fixed with this PR: rancher/dashboard#1981

On 2.5-head - commit id: 395b4fe77

Download report for one time scan and scheduled scan works.
Report/reports are downloaded


[$1000] Press and hold reaction emoji highlights emoji shortcode

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open an existing chat
Send message
React to the message with emoji
Press and hold the reaction emoji
The emoji shortcode (:fire) is selectable

Expected Result:
The emoji shortcode shouldn't be selectable
Actual Result:
shortcode on the emoji is selected
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[x] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.11-2
Reproducible in staging?: y
**Reproducible in production?:**new feature
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/236692550-f502fff4-db9d-4078-bb6f-05aba3f7e991.mp4
https://user-images.githubusercontent.com/43996225/236692556-359da89f-50c4-4e24-bc96-444d65cfb760.mp4
Expensify/Expensify Issue URL:
Issue reported by: @natnael-guchima
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1683360430748899
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0158e77952c96c0f58
Upwork Job ID:<PHONE_NUMBER>583827968
Last Price Increase: 2023-05-08

When pressing down a button (in this case the emoji reaction), the selection shouldn't happen. We have a discussion and decided to wait for the new version of rn-web here https://github.com/Expensify/App/issues/14139#issuecomment-1379475849 which will completely fix the issue.
We also have a similar issue here https://github.com/Expensify/App/issues/13908 and this selection issue happens on everywhere, except the button that we apply a workaround by disabling the selection.
cc: @parasharrajat

Thanks for the insights @parasharrajat I'll close this issue.

Hello @parasharrajat @Christinadobrzyn, I see that an issue that have the same nature as I reported here is being fixed here, and the one I reported is closed. I am curious to know the difference between the issue I reported here and the one fixed here https://github.com/Expensify/App/issues/18661


Unable to test on real device due to port issue

I have the flutter app with all the necessary configuration such as integration_test/appium_test.dart file. When I built and run the app using the following command flutter run ios --target integration_test/appium_test.dart --flavor qa --release I see the following lines in the terminal
flutter: [APPIUM FLUTTER]  Appium flutter server is listening on port 9000
flutter: 00:00 +0: appium flutter server

So far so good.
And in my automation script in the desired caps, I have also made sure to use
"appium:flutterSystemPort", 9001
When I run the test,  I see 9001 port number is NOT reachable. See the logs below.
[5211e896][AppiumFlutterDriver@d4bb] Checking if flutter server is running on port 9001
[5211e896][WD Proxy] Matched '/status' to command name 'getStatus'
[5211e896][WD Proxy] Proxying [GET /status] to [GET http://<IP_ADDRESS>:9001/status] with no body
[5211e896][iProxy@00008030:9001] Connection was refused to port 9001
[5211e896][WD Proxy] socket hang up
[5211e896][AppiumFlutterDriver@d4bb] FlutterServer not reachable on port 9001, Retrying..
[5211e896][WD Proxy] Matched '/status' to command name 'getStatus'
[5211e896][WD Proxy] Proxying [GET /status] to [GET http://<IP_ADDRESS>:9001/status] with no body
[5211e896][iProxy@00008030:9001] Connection was refused to port 9001
[5211e896][WD Proxy] socket hang up

It keeps on scanning for next port numbers like 9002, 9003 etc but to no avail.
My dev code has
dev_dependencies:
  appium_flutter_server: 0.0.27

and I am using latest version of appium v2.12.1, Java client is 9.3.0 and flutter integration version<EMAIL_ADDRESS>(automationName 'FlutterIntegration').
What's wrong with my setup?
To the contrary if I clone appium-flutter-server repo and change the file server.dart, line # 39 to const PORT_RANGE = [9001, 9020]; and have the server location referred in pubsec.yaml as
  appium_flutter_server:
    path: ../server

<IP_ADDRESS>:9001 is reachable and rest everything works fine.
This is just the workaround.
Please let me know the ideal fix for this situation.
Thanks!

+1 for this issue

@suryak-cs could you share the full appium server logs as gist?


[GSSOC'23] - Add -- Add to Cart

Adding Add to Cart in the heading bar

@codervivek5 Please assign this issue to me.

Adding Add to Cart in the heading bar  @codervivek5 Please assign this issue to me.

I have added on click add to cart I don't think there is any need of Add to cart in header

Adding Add to Cart in the heading bar  @codervivek5 Please assign this issue to me.

I have added on click add to cart I don't think there is any need of Add to cart in header

I want to add this in the heading bar where all the user items are listed. IDK where you want to add this. Could you attach your issue here.

here is the video - @komala777 Screencast from 28-06-23 10:59:52 AM IST.webm


Subaru Forester 2019

This PR adds support for Subaru Forester 2019 (Global Platform). Tuning values and testing by Discord user dgranger15
Route ID for replay tests: c321c6b697c5a5ff|2020-06-23--11-04-33
Highway drive link: https://my.comma.ai/cabana/?route=c321c6b697c5a5ff|2020-06-23--11-04-33&exp=1624469159&sig=haDYm%2FnNboH2X%2BhlY4XFcbUVGXqykYgPAAlz3x9oSHk%3D&max=17&url=https%3A%2F%2Fchffrprivate-vzn.azureedge.net%2Fchffrprivate3%2Fv2%2Fc321c6b697c5a5ff%2Fff13c873e05ba2f5137fa23ad4126f7a_2020-06-23--11-04-33

Uploaded the route to the CI bucket, tests should pass now.


Bump the minimum required version for loader-fs-cache

1.0.0 naively uses mkdirp without listing it as dependency

This is totally not required as semver will get the patch. ^1.0.0 will get us latest 1.x, so 1.0.1 is good already! Thanks for fixing this!


Added TYPE_CHECKING for lazy loading

Adresses this issue: #https://github.com/pyglet/pyglet/issues/286
Following the suggestion from one of the pylance maintainers: #https://github.com/microsoft/pylance-release/issues/474
This change makes static analysis tools like pylance able to provide code completion for all of pyglet. Without it they fail to understand the lazy loading and don't work for the  submodules.

It seems a bit odd to use TYPE_CHECKING here, since we're not using any type definitions in pyglet. Looking at the source code of the typing module, however, it just defaults to TYPE_CHECKING = False. It should be fine to make this change, if it will help with their inspection.


Evaluate bottleneck of training process

Training a naive GNN using ~140k data points took almost two days on Snellius, using CPU only. We are evaluating which is the bottleneck, either the way in which data points are loaded from the hdf5 files or the fact that we're still not using GPUs. In the first case, we may evaluate if keep using hdf5 format as it is for storage is really convenient.
Blocks #315

Here's a detailed report about deeprank-core training phase performances.
100 samples dataset
Timings for running 1 epoch, trainer.py:603(_epoch method), 1 call:

CPU/s only

num_workers = 0: 0.967 s
num_workers = 16: 1.447 s
num_workers = 64: todo

GPU + CPU/s

num_workers = 0: 0.305 s
num_workers = 16: 0.710
num_workers = 18: todo

Timings for running dataloader.py:1286(_next_data), 1 call:

CPU/s only

num_workers = 0: 0.060 s
num_workers = 16: 0.071 s
num_workers = 64: todo

GPU + CPU/s

num_workers = 0: 0.050 s
num_workers = 16: 0.075 s
num_workers = 18: todo

140k samples dataset
Timings for running 1 epoch, trainer.py:603(_epoch method), 1 call:

CPU/s only

num_workers = 0: ~9600 s
num_workers = 16: ongoing
num_workers = 64: ~1954 s

GPU + CPU/s

num_workers = 0: ~9500 s
num_workers = 16: ongoing
num_workers = 18: ~354 s

Timings for running dataloader.py:1286(_next_data), 1 call:

CPU/s only

num_workers = 0: ongoing
num_workers = 16: ongoing
num_workers = 64: 0.018 s

GPU + CPU/s

num_workers = 0: ongoing
num_workers = 16: ongoing
num_workers = 18: 0.057 s


Testing infrastructure with Github Actions using custom-runners on nf-core infra

The goal of this task is to

Use GitHub actions
Setup a test nomad cluster
Use CSI-hostpath volumes

Hi @tomiles  and @matthdsm , please feel free to add details/subtasks to this task as you see fit. I've marked this as part of v0.1.0
@jagedn and I would focus on the local dev cluster and the mock testing setup for the initial milestones.

Phase-1: Nomad cluster on Azure

Host Volume / CSI volume setup using Azure FileShare
One Azure VM
A Nomad auth certificate
Nomad ACL minimal (https://developer.hashicorp.com/nomad/docs/other-specifications/acl-policy)

Phase-2:

Add other Azure VMs and iterate on ACL

for the record
from now we have a GH action able to run pipelines using the Azure cluster, see https://github.com/nextflow-io/nf-nomad/actions/runs/10614308658

for the record
from now we have a GH action able to run pipelines using the Azure cluster, see https://github.com/nextflow-io/nf-nomad/actions/runs/10614308658

I'm looking into setting up scripts so that the runner creates and destroys the nomad cluster each time, so you can test against a nomad cluster


Dataloader error when using DeepSpeed

🐛 Bug
After https://github.com/PyTorchLightning/pytorch-lightning/pull/8527 PR is merged, I tested DeepSpeed with pytorch lightning nightly version and got dataloader error as below:
Traceback (most recent call last):
  File "/home/shared/workspace/multimodal-matching/multimodal-matching/train-CLIP/boring_example/deepspeed_dataloader.py", line 114, in <module>
    test_x(tmpdir)
  File "/home/shared/workspace/multimodal-matching/multimodal-matching/train-CLIP/boring_example/deepspeed_dataloader.py", line 112, in test_x
    trainer.fit(model)
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 546, in fit
    self._run(model)
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/trainer/trainer.py", line 853, in _run
    self.accelerator.setup_environment()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/accelerators/gpu.py", line 30, in setup_environment
    super().setup_environment()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/accelerators/accelerator.py", line 80, in setup_environment
    self.training_type_plugin.setup_environment()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/ddp.py", line 166, in setup_environment
    self.setup_distributed()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/deepspeed.py", line 343, in setup_distributed
    self._format_config()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/deepspeed.py", line 543, in _format_config
    self._format_batch_size_and_grad_accum_config()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/deepspeed.py", line 553, in _format_batch_size_and_grad_accum_config
    batch_size = self._auto_select_batch_size()
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/pytorch_lightning/plugins/training_type/deepspeed.py", line 564, in _auto_select_batch_size
    train_dataloader = self.lightning_module.train_dataloader()
  File "/home/shared/workspace/multimodal-matching/multimodal-matching/train-CLIP/boring_example/deepspeed_dataloader.py", line 90, in train_dataloader
    return DataLoader(self.mnist_train, batch_size=BATCH_SIZE)
  File "/opt/conda/envs/clip_cuda11.1/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1130, in __getattr__
    raise AttributeError("'{}' object has no attribute '{}'".format(
AttributeError: 'LitMNIST' object has no attribute 'mnist_train'

The code works fine without deepspeed.
I'm guessing when using deepspeed, it does not call setup() method in LightningDataModule class.
Please reproduce using the BoringModel
To Reproduce
Since deepspeed is not working in interactive mode as Colab, I attached an example code: deepspeed_dataloader.py.
Also, the code is based on MNIST example because the BoringModel does not include detailed dataloader implementation (e.g., setup(), train_dataloader()).
Expected behavior
It works without deepspeed, and the expected behavior is that the code works with deepspeed as well.
Environment

CUDA:
- GPU:
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- Tesla V100-SXM2-32GB
- available:         True
- version:           11.1
Packages:
- numpy:             1.20.1
- pyTorch_debug:     False
- pyTorch_version:   1.9.0
- pytorch-lightning: 1.4.0rc1
- tqdm:              4.59.0
System:
- OS:                Linux
- architecture:
- 64bit
- ELF
- processor:         x86_64
- python:            3.8.8
- version:            SMP Fri Dec 18 16:34:56 UTC 2020

Additional context

I confirm this is no longer a problem and got fixed. I verified it by taking the provided code sample and running it with both PL 2.0 and 1.9.4. I also converted the code to a DataModule since the OP also mentioned it.
Here is the code I used to verify (only imports and Trainer API was updated from OPs outdated code):
import os

import torch
from torch import nn
from torch.nn import functional as F
from torch.utils.data import DataLoader, random_split, Dataset
from torchvision.datasets import MNIST
from torchvision import transforms
tmpdir = os.getcwd()
from lightning import Trainer, LightningModule, LightningDataModule

PATH_DATASETS = os.environ.get('PATH_DATASETS', '.')
BATCH_SIZE = 256
train_ds = MNIST(PATH_DATASETS, train=True, download=True, transform=transforms.ToTensor())

class LitMNIST(LightningModule):

    def __init__(self, data_dir=PATH_DATASETS, hidden_size=64, learning_rate=2e-4):

        super().__init__()

        # Set our init args as class attributes
        self.data_dir = data_dir
        self.hidden_size = hidden_size
        self.learning_rate = learning_rate

        # Hardcode some dataset specific attributes
        self.num_classes = 10
        self.dims = (1, 28, 28)
        channels, width, height = self.dims
        self.transform = transforms.Compose([transforms.ToTensor(), transforms.Normalize((0.1307, ), (0.3081, ))])

        # Define PyTorch model
        self.model = nn.Sequential(
            nn.Flatten(), nn.Linear(channels * width * height, hidden_size), nn.ReLU(), nn.Dropout(0.1),
            nn.Linear(hidden_size, hidden_size), nn.ReLU(), nn.Dropout(0.1), nn.Linear(hidden_size, self.num_classes)
        )

    def forward(self, x):
        x = self.model(x)
        return F.log_softmax(x, dim=1)

    def training_step(self, batch, batch_idx):
        x, y = batch
        logits = self(x)
        loss = F.nll_loss(logits, y)
        return loss

    def validation_step(self, batch, batch_idx):
        x, y = batch
        logits = self(x)
        loss = F.nll_loss(logits, y)
        preds = torch.argmax(logits, dim=1)

        # Calling self.log will surface up scalars for you in TensorBoard
        self.log('val_loss', loss, prog_bar=True)
        return loss

    def test_step(self, batch, batch_idx):
        # Here we just reuse the validation_step for testing
        return self.validation_step(batch, batch_idx)

    def configure_optimizers(self):
        optimizer = torch.optim.Adam(self.parameters(), lr=self.learning_rate)
        return optimizer

    ####################
    # DATA RELATED HOOKS
    ####################

    def prepare_data(self):
        # download
        MNIST(self.data_dir, train=True, download=True)
        MNIST(self.data_dir, train=False, download=True)

    def setup(self, stage=None):

        # Assign train/val datasets for use in dataloaders
        if stage == 'fit' or stage is None:
            mnist_full = MNIST(self.data_dir, train=True, transform=self.transform)
            self.mnist_train, self.mnist_val = random_split(mnist_full, [55000, 5000])

        # Assign test dataset for use in dataloader(s)
        if stage == 'test' or stage is None:
            self.mnist_test = MNIST(self.data_dir, train=False, transform=self.transform)

    def train_dataloader(self):
        return DataLoader(self.mnist_train, batch_size=BATCH_SIZE)

    def val_dataloader(self):
        return DataLoader(self.mnist_val, batch_size=BATCH_SIZE)

    def test_dataloader(self):
        return DataLoader(self.mnist_test, batch_size=BATCH_SIZE)

class MyDataModule(LightningDataModule):
    def __init__(self, data_dir=PATH_DATASETS):
        super().__init__()
        self.data_dir = data_dir
        self.transform = transforms.Compose([transforms.ToTensor(), transforms.Normalize((0.1307, ), (0.3081, ))])

    def prepare_data(self):
        # download
        MNIST(self.data_dir, train=True, download=True)
        MNIST(self.data_dir, train=False, download=True)

    def setup(self, stage=None):

        # Assign train/val datasets for use in dataloaders
        if stage == 'fit' or stage is None:
            mnist_full = MNIST(self.data_dir, train=True, transform=self.transform)
            self.mnist_train, self.mnist_val = random_split(mnist_full, [55000, 5000])

        # Assign test dataset for use in dataloader(s)
        if stage == 'test' or stage is None:
            self.mnist_test = MNIST(self.data_dir, train=False, transform=self.transform)

    def train_dataloader(self):
        return DataLoader(self.mnist_train, batch_size=BATCH_SIZE)

    def val_dataloader(self):
        return DataLoader(self.mnist_val, batch_size=BATCH_SIZE)

    def test_dataloader(self):
        return DataLoader(self.mnist_test, batch_size=BATCH_SIZE)

if __name__ == '__main__':
    model = LitMNIST()
    datamodule = MyDataModule()
    trainer = Trainer(
        devices=1,
        accelerator="cuda",
        max_epochs=1,
        precision=16,
        strategy="deepspeed_stage_2"
    )
    trainer.fit(model, datamodule)


Nested Model in Union gives AttributeError: model_fields

Apologies if this is the wrong way to communicate but I am trying get an example working that matches the pydantic models I am trying to render in a dash app.
I have taken the example in your Discriminate Unions doc section and extended it by creating an Office class which is then used as the type for an additional attribute called 'options' in HomeOffice. I have done this only because I am stuck with multiple pydantic models that have this structure. I have noticed that if HomeOffice instead inherits Office then it works.
But is there any way of getting the pydantic model structure below to work with dash-pydantic-form ??

Hey @wynneevans, thanks for bringing this to my attention. I am able to reproduce this issue and indeed there are issues with nested models in discriminated unions. I'll find a fix for this and let you know when there is progress on this!

That would be really amazing. Thank you for making this open source! If you do commit time to this issue then it would be even more amazing if it could handle multiple layers of nesting. i.e. In the above example, instead of Office inheriting BaseModel it inherited another class, which inherited another class, etc, until a class that inherits BaseModel. I've attached an example of the type of pydantic model I am trying to render in dash. This only in case you would like to stress test dash-pydantic-form against a real-world horrible structure. Changed to .txt file because .py files cannot be uploaded - it's not strictly working code just illustration of model structure.
post.txt

@wynneevans it all seems to be working in 0.3.3 on my end, feel free to re-open if you still have issues with this 🙂


Update the dependency to core-java 0.10.81-SNAPSHOT.

Auto-generated PR to update core library dependency to the latest version.

Closing as the version has already been updated in https://github.com/SpineEventEngine/jdbc-storage/pull/114.