Linear RGB also comes in 32-bit and floating point

Hi @mm2,
In https://github.com/mm2/Little-CMS/commit/6b04ee0bab3ebe6f2c487195bac7b86efadd58d3 you added an automatic kill switch to disable optimization in case of a linear RGB space. In doing so, 32-bit and/or floating-point color spaces were left out, which broke Krita's out of gamut warnings as well as palette docker in the presence of OCIO.
This commit adds those two types of space to the check.
Fixes Krita bugs 442004 and 437429.
/cc @Lynx3d

The issue was originated by something completly different. Thanks for reporting.


"startsample.sh" does not give me enough time to authenticate

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:
The startsample script does not give me enough time to open the code website and enter the authorization code
What is the expected behavior?
I should have 30-90 seconds to enter the code
What behavior are you observing?
The script runs to completion without allowing for time to open the website
Provide the steps to reproduce the issue, if applicable:
after install, I run "sudo bash startsample.sh". The script runs to completion with pausing to allow me to enter the code into the website.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  master branch (1.8+?)

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ X] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[X ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

I needed to plug in my usb mic before starting the sample app. Leaving this here in case anyone else has the same problem.


[SPARK-8103][core] DAGScheduler should not submit multiple concurrent attempts for a stage

https://issues.apache.org/jira/browse/SPARK-8103
cc @kayousterhout (thanks for the extra test case)

(separate comment just to avoid putting this in the pr description for eventual merging)
@kayousterhout I agree that your test is much clearer than running jobs with various pauses.  I had tried to come up with something clearer but your version is much better.
I am a little torn, however -- this is certainly better for understanding this narrow case, but I feel it also gives a false sense of confidence b/c you only cover a narrow set of cases.  Eg., your new case covers the extra failure happening after the reduce stage is retried, but not if the failure happens before the reduce stage is retried, or after the reduce stage retry has completely finished.  To address that, I've been running a a variety of tests like this https://github.com/squito/spark/blob/combined_multi_attempt_fixes/core/src/test/scala/org/apache/spark/scheduler/DAGSchedulerFailureRecoverySuite.scala#L39
with some randomization in the parameters.  FWIW, with these changes and https://github.com/apache/spark/pull/6648, those tests pass consistently for me.  Of course, that is still getting limited coverage, and I'd like to expand it even more.  I'd like to find a better home for running tests like that (not just running them periodically on my laptop), but I guess that is separate issue.  So I think this can be merged as is, w/ the tests you wrote.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #34655 has started for   PR 6750 at commit b6bc248.

Test build #34655 has finished for   PR 6750 at commit b6bc248.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #34665 has started for   PR 6750 at commit ecb4e7d.

Test build #34665 has finished for   PR 6750 at commit ecb4e7d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #34689 has started for   PR 6750 at commit 9601b47.

Test build #34689 has finished for   PR 6750 at commit 9601b47.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #34692 has started for   PR 6750 at commit 89a59b6.

Test build #34692 has finished for   PR 6750 at commit 89a59b6.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Merged build triggered.

Merged build started.

Test build #34695 has started for   PR 6750 at commit 46bc26a.

Merged build finished. Test FAILed.

Test build #34695 has finished for   PR 6750 at commit 46bc26a.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #36042 has started for   PR 6750 at commit d8eb202.

Test build #36042 has finished for   PR 6750 at commit d8eb202.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #36118 has started for   PR 6750 at commit d8eb202.

Test build #36118 has finished for   PR 6750 at commit d8eb202.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test PASSed.

ping @kayousterhout @markhamstra @mateiz

Other than the question of whether doing more while holding the TaskSchedulerImpl lock is a concern, this LGTM.  I'll come back later to take a closer look at the new tets in DAGSchedulerSuite.

Looks sane to me, although I'm not really familiar with all the surrounding code.

Merged build triggered.

Merged build started.

Test build #36300 has started for   PR 6750 at commit ada7726.

Looking at this is on my to-do list...I'll take a look this afternoon or tomorrow!

Test build #36300 has finished for   PR 6750 at commit ada7726.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #36319 has started for   PR 6750 at commit ada7726.

Test build #36319 has finished for   PR 6750 at commit ada7726.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #36434 has started for   PR 6750 at commit 6542b42.

Test build #36434 has finished for   PR 6750 at commit 6542b42.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test PASSed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #36465 has started for   PR 6750 at commit 6542b42.

Test build #36465 has finished for   PR 6750 at commit 6542b42.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class SparkIllegalStateException(message: String, cause: Throwable)

Merged build finished. Test PASSed.

Finally got a chance to finish looking at this; looks mostly good with a few small comments!

Merged build triggered.

Merged build started.

Test build #36697 has started for   PR 6750 at commit a5f7c8c.

Merged build triggered.

Merged build started.

Test build #36698 has started for   PR 6750 at commit 19685bb.

@kayousterhout I think I addressed your comments.  I had some questions about the best way to make the attemptId condition cleaner -- I went ahead with one approach here https://github.com/squito/spark/commit/19685bba6d4615b9b725032a070249255caa0ff8
but I'm open to other suggestions.
Also on making the check for multiple active task sets faster -- there are a bunch of places that activeTaskSets is used with the existing structure.  It just makes the code more complicated to change all those usages.  So instead I added another variable for this purpose.  But I'm happy to change that to just be activeTaskSets if you prefer.  https://github.com/squito/spark/commit/b2faef52a5a2b49e8196a2f79f1f0c5349afd63e

Test build #36697 has finished for   PR 6750 at commit a5f7c8c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test build #36698 has finished for   PR 6750 at commit 19685bb.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Just checking in about this -- did you see my suggestion for using activeTaskSets rather than using a new data structure here https://github.com/squito/spark/pull/2?
Also it looks like this has fallen out of date.

oh, thanks for mentioning that, sorry I didn't see your suggestion!  I'll take a look later today and bring this up to date also, thanks.

Build triggered.

Build started.

Test build #37230 has started for   PR 6750 at commit f025154.

Merged build triggered.

Merged build started.

Test build #37235 has started for   PR 6750 at commit 906d626.

Test build #37230 timed out     for PR 6750 at commit f025154     after a configured wait of 175m.

Build finished. Test FAILed.

Test build #37235 has finished for   PR 6750 at commit 906d626.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class Least(children: Seq[Expression]) extends Expression
case class Greatest(children: Seq[Expression]) extends Expression

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #37252 has started for   PR 6750 at commit a21c8b5.

Test build #37252 has finished for   PR 6750 at commit a21c8b5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #37395 has started for   PR 6750 at commit c04707e.

@kayousterhout I made the changes to get rid of the extra data structure in TaskSchedulerImpl, see what you think.  Also I added some test cases to TaskSchedulerImplSuite that cover the cases I was mentioned that I think would break with your PR.  (those tests would pass before this change in any case, but I think they're good to have.)

Merged build triggered.

Merged build started.

Test build #37396 has started for   PR 6750 at commit 4470fa1.

Test build #37395 has finished for   PR 6750 at commit c04707e.

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

Merged build finished. Test FAILed.

Test build #37396 has finished for   PR 6750 at commit 4470fa1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

@squito I think there's a simpler fix here: I don't think TaskSchedulerImpl actually needs to know about all of the potential zombie task attempts.  Right now there are two things that happen in TaskSchedulerImpl.taskSetFinished: the task set is removed from the set of active task sets, and the schedulable is removed (for the purposes of fairness accounting).  What if we split this into two methods: one called taskSetNoLongerActive() that signals when a task set has become a zombie (so can be removed from activeTaskSets) and then rename taskSetFinished to allTasksInTaskSetFinished() or something, that removes the schedulable.  Then you could change activeTaskSets to be indexed on stage id and have a single task attempt, as in my original suggestion. I prefer that approach because it avoids tracking unnecessarily (and complex) state about all of the zombie task attempts.  Thoughts?

@kayousterhout I don't think that will work, but maybe I'm not seeing it.  I think the problem is, you still need some way get a handle on the zombie TaskSetManager to be able to call allTasksInTaskSetFinished().  Right now, taskSetFinished is ultimately getting a handle on that TaskSetManager by looking it up in activeTaskSets in statusUpdate().  So if those zombie task sets aren't in activeTaskSets, it seems like you'd still need to keep track of them somewhere in TaskSchedulerImpl.
I feel like part of the problem is that "active" task sets is somewhat vague.  You might not expect it to contain task sets that have already failed (from a fetch failed), but still happen to have tasks running.  I guess "zombie" is vague too, but in a way that is better since you aren't tricked into thinking you know what it means.

Ah shoot you're right. Do you prefer your original approach of creating a new data structure to track this or the current one of making activeTaskSets more complicated?

I suppose I slightly prefer the current one, I think its a bit more clear what is being tracked (with so many different ids floating around, its easy to lose track of what the id of a taskset is).  But no strong preference.  OTOH getting to the task set manager from a task id is now a lookup in 3 maps instead of 2, dunno if that will effect performance (or if it matters here).
really I don't have any preferences ...

I had one last idea for how to simplify this: https://github.com/squito/spark/pull/4, that I made sure doesn't break any tests this time! Let me know what your thoughts are.

Merged build triggered.

Merged build started.

Test build #37558 has started for   PR 6750 at commit e43ac25.

Test build #37558 has finished for   PR 6750 at commit e43ac25.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UnresolvedAttribute(nameParts: Seq[String]) extends Attribute
abstract class Star extends LeafExpression with NamedExpression
case class UnresolvedAlias(child: Expression) extends UnaryExpression with NamedExpression
case class SortOrder(child: Expression, direction: SortDirection) extends UnaryExpression
trait AggregateExpression extends Expression
trait PartialAggregate extends AggregateExpression
case class Min(child: Expression) extends UnaryExpression with PartialAggregate
case class Max(child: Expression) extends UnaryExpression with PartialAggregate
case class Count(child: Expression) extends UnaryExpression with PartialAggregate
case class Average(child: Expression) extends UnaryExpression with PartialAggregate
case class Sum(child: Expression) extends UnaryExpression with PartialAggregate
case class SumDistinct(child: Expression) extends UnaryExpression with PartialAggregate
case class First(child: Expression) extends UnaryExpression with PartialAggregate
case class Last(child: Expression) extends UnaryExpression with PartialAggregate
trait Generator extends Expression
case class Explode(child: Expression) extends UnaryExpression with Generator
trait NamedExpression extends Expression
abstract class Attribute extends LeafExpression with NamedExpression
case class PrettyAttribute(name: String) extends Attribute
abstract class LeafNode extends LogicalPlan
abstract class UnaryNode extends LogicalPlan

Merged build finished. Test PASSed.

Ok I did a last pass over this...just some last style comments, and it sounds like you're also going to do the cleanup of making the hashmap indexed on task id store the task set manager as a value? Then this looks good.

Merged build triggered.

Merged build started.

Test build #37655 has started for   PR 6750 at commit e01b7aa.

thanks for the last set of comments @kayousterhout .  I've updated based on your suggestions

There's just one lingering erroneous log message, and then this LGTM.

Thanks for all the iterations on this...I'm hopeful that if we made the scheduler code a little better / easier to read with each PR, these will become less painful over time.

Merged build triggered.

Merged build started.

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #32 has started for   PR 6750 at commit fb3acfc.

Merged build triggered.

Merged build started.

Test build #37663 has started for   PR 6750 at commit fb3acfc.

Test build #37655 has finished for   PR 6750 at commit e01b7aa.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test build #37663 has finished for   PR 6750 at commit fb3acfc.

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

Merged build finished. Test FAILed.

Jenkins, retest this please

Merged build triggered.

Merged build started.

Test build #33 has started for   PR 6750 at commit fb3acfc.

Merged build triggered.

Merged build started.

Test build #37668 has started for   PR 6750 at commit fb3acfc.

Test build #32 has finished for   PR 6750 at commit fb3acfc.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class StandaloneRecoveryModeFactory(conf: SparkConf, serializer: Serializer)
class RFormula(override val uid: String)
case class UnresolvedAttribute(nameParts: Seq[String]) extends Attribute
abstract class Star extends LeafExpression with NamedExpression
case class UnresolvedAlias(child: Expression) extends UnaryExpression with NamedExpression
abstract class LeafExpression extends Expression
abstract class UnaryExpression extends Expression
abstract class BinaryExpression extends Expression
case class SortOrder(child: Expression, direction: SortDirection) extends UnaryExpression
trait AggregateExpression extends Expression
trait PartialAggregate extends AggregateExpression
case class Min(child: Expression) extends UnaryExpression with PartialAggregate
case class Max(child: Expression) extends UnaryExpression with PartialAggregate
case class Count(child: Expression) extends UnaryExpression with PartialAggregate
case class Average(child: Expression) extends UnaryExpression with PartialAggregate
case class Sum(child: Expression) extends UnaryExpression with PartialAggregate
case class SumDistinct(child: Expression) extends UnaryExpression with PartialAggregate
case class First(child: Expression) extends UnaryExpression with PartialAggregate
case class Last(child: Expression) extends UnaryExpression with PartialAggregate
case class Pmod(left: Expression, right: Expression) extends BinaryArithmetic
      final class SpecificRow extends $
trait Generator extends Expression
case class Explode(child: Expression) extends UnaryExpression with Generator
trait NamedExpression extends Expression
abstract class Attribute extends LeafExpression with NamedExpression
case class PrettyAttribute(name: String) extends Attribute
case class Length(child: Expression) extends UnaryExpression with ExpectsInputTypes
case class FormatNumber(x: Expression, d: Expression)
abstract class LeafNode extends LogicalPlan
abstract class UnaryNode extends LogicalPlan
abstract class BinaryNode extends LogicalPlan

Merged build finished. Test PASSed.

Test build #33 has finished for   PR 6750 at commit fb3acfc.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
abstract class StandaloneRecoveryModeFactory(conf: SparkConf, serializer: Serializer)
class RFormula(override val uid: String)
case class UnresolvedAttribute(nameParts: Seq[String]) extends Attribute
abstract class Star extends LeafExpression with NamedExpression
case class UnresolvedAlias(child: Expression) extends UnaryExpression with NamedExpression
abstract class LeafExpression extends Expression
abstract class UnaryExpression extends Expression
abstract class BinaryExpression extends Expression
case class SortOrder(child: Expression, direction: SortDirection) extends UnaryExpression
trait AggregateExpression extends Expression
trait PartialAggregate extends AggregateExpression
case class Min(child: Expression) extends UnaryExpression with PartialAggregate
case class Max(child: Expression) extends UnaryExpression with PartialAggregate
case class Count(child: Expression) extends UnaryExpression with PartialAggregate
case class Average(child: Expression) extends UnaryExpression with PartialAggregate
case class Sum(child: Expression) extends UnaryExpression with PartialAggregate
case class SumDistinct(child: Expression) extends UnaryExpression with PartialAggregate
case class First(child: Expression) extends UnaryExpression with PartialAggregate
case class Last(child: Expression) extends UnaryExpression with PartialAggregate
case class Pmod(left: Expression, right: Expression) extends BinaryArithmetic
      final class SpecificRow extends $
trait Generator extends Expression
case class Explode(child: Expression) extends UnaryExpression with Generator
trait NamedExpression extends Expression
abstract class Attribute extends LeafExpression with NamedExpression
case class PrettyAttribute(name: String) extends Attribute
case class Length(child: Expression) extends UnaryExpression with ExpectsInputTypes
case class FormatNumber(x: Expression, d: Expression)
abstract class LeafNode extends LogicalPlan
abstract class UnaryNode extends LogicalPlan
abstract class BinaryNode extends LogicalPlan

Test build #37668 has finished for   PR 6750 at commit fb3acfc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.


fix "$pio eval" command

"$pio eval" command parameters are broken: package name in command does not match the package name in Evaluation.scala

@tom-chan I think @DanVisan is referring to https://github.com/apache/predictionio-template-attribute-based-classifier/blob/develop/src/main/scala/Evaluation.scala. That is what is being linked from this file right now.

Got it. In that case do you mind updating other instances of org.template.classification. to org.example.classification. on this page, as well as docs/manual/source/evaluation/metricbuild.html.md? Thanks.

Would be great if you also update docs/manual/source/evaluation/metricbuild.html.md in the same tutorial series. Thanks!

@tom-chan i updated docs/manual/source/evaluation/metricbuild.html.md. Thanks for noticing!

LGTM


[#855] 2) header-ui: request flow ui start inert (no backend nor frontend integration with it)

This one

header-ui: [#855] add inert request membership button+modal

Previous ones

settings-ui: [#855] set membership policy

Screenshots
Members page (records page for later)

Modal

rebased

Closed via https://github.com/inveniosoftware/invenio-communities/pull/1132


Consider optimizing NodeUtil#binarySearchForEdge()

The way that it's implemented right now isn't very GC friendly. It seems like it would be trivial to actually implement binary search directly.

This sounds reasonable. I will try to look into this for the next release. I'm hoping to get time for this in the next few weeks.

Cool, thanks!

Merged. Many thanks for the PR!
This will be included in the next release.


Problems with postField params

Hey,
I'm using Guzzle 5.3 and guzzle-services 0.5. When I describe a service with POST method and postField params it generates a 400 bad request error (Error executing command: Client error response). It works well when I use "query" params instead.
Here is my code (description):
$description = new Description([
                'operations' => [
                    'createUser' => [
                        'httpMethod' => 'POST',
                        'uri' => 'users',
                        'responseModel' => 'jsonResponse',
                        'parameters' => [
                            'first_name' => [
                                'type' => 'string',
                                'location' => 'postField',
                                'required' => true
                            ],
                            'last_name' => [
                                'type' => 'string',
                                'location' => 'postField',
                                'required' => true
                            ],
                            'email' => [
                                'type' => 'string',
                                'location' => 'postField',
                                'required' => true
                            ],
                            'password' => [
                                'type' => 'string',
                                'location' => 'postField',
                                'required' => true
                            ]
                        ]
                    ]
                  ]
            ]);

And this is how I use that service:
$r = $client->createUser(array("first_name" => "toto", "last_name" => "titi", "email" => "tata", "password" => "tutu"));

As I said, it works when I change 'location' to 'query' instead of 'postField' (but I have to use post params).
Thanks for your help.

Not sure why there would be an issue there. If you can capture debug or curl verbose output for each version (one using postField and one using query), that would help identify if there is an issue with this package or a misunderstanding of what the service actually needs.

Is this issue still valid? Can you provide the requested debug output?

Closed after no answer since four weeks. Feel free to reopen this issue.
Note: 1.0.0 of guzzle-services is out which is compatible with Guzzle6.


Support for multiple highlights in color and status bar info for highlights

We did this as a group project for our open source class, happy to hear any feedback you may have!

Are you looking to have this merged into this package or just feedback?

Potentially both?
On May 9, 2016 5:56 AM, "Richard Race"<EMAIL_ADDRESS>wrote:

Are you looking to have this merged into this package or just feedback?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/richrace/highlight-selected/pull/124#issuecomment-217823430

Ok. I'll give you feedback for merging it in. I'll have a look tonight about it (UK time zone).
One thing is I need the build passing from Travis before I can merge, thanks.

Points:

It breaks when you select non-word characters (e.g. -)
Status bar doesn't show
It breaks by doing:

Add cursor (do not double click on a word)
Double click on a word
Errors

To merge in you need to:

Undo changes to package.json
Ensure tests and coffee lint pass.
Add new tests to cover the new functionality.

Atom 1.48.0 (2020) broke this package. It is my most us package on Atom and I had to roll back. Could you please fix this? I can't really love without this on Atom.


xh: 0.19.1 -> 0.19.3

Diff: https://github.com/ducaale/xh/compare/v0.19.1...v0.19.3
Changelog: https://github.com/ducaale/xh/blob/v0.19.3/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 262636 run on x86_64-linux 1

1 package built:

xh


pixart HW

comfort: 4
completeness: 5
Instead of using $("#set-color").on("click", function(event){ event.preventDefault(); ............},
when I use $("#set-color").addEventListener("click", function(event){ ......... } and check on the console,
it says $("#set-color").addEventListener("click", function(event){ ......... } is not a function and not working.
Am I missing something if I use addEventListener?

:+1:  you are mixing dom methods and query methods.
addEventListener only works for DOM objects. .on() only works for jquery objects


Centering of team and clients

Hi,
Thank you for this amazing theme!
If the number of team members and clients are less than 3 and 4 respectively, it will still be left aligned. Is there any way to center this?
Thank you!

Hello,
You can use  justify-content-center like so:


Issue in isElementOverlapped with getting topRight element causing InspectorMode to hide edditing overlay

Describe the bug
The calculation in isElementOverlapped for getting the topRight element is not working as expected and resulting in hiding the edditing overlay in inspection mode.
In the current implementation it returns always the element next to element which is be testing.
It is a problem with using "document.elementFromPoint(x, y)" together with the right/top coordinates from "getBoundingClientRect" in utils.ts: https://github.com/contentful/live-preview/blob/3604517d40c15ea0e07277b9a4405a421feb78cf/packages/live-preview-sdk/src/inspectorMode/utils.ts#L364
To Reproduce
I have tried to build a jsfiddle for demonstraiting this behaviour:
https://jsfiddle.net/y2xmLq7p/2/
Expected behavior
Testing in isElementOverlapped for topRight-element is working when there is no overlapping element.
Context (please complete the following information):

Live preview SDK version number: 4.5.14

Hey @johollmann, thanks for highlighting this issue to us. We have it on our radar now and are working on a fix

I think this is the same issue. I noticed today that the Live Preview inspect overlay no longer appears for elements with a block child that has the top left coordinates...
The inspect overlay will not appear for something like this:

test

Add a padding to the parent or margin to the child and the inspect overlay will appear:

test 1

or

test

Hi @aodhagan-cf - we're experiencing the same problem as reported by dndrsn, are there any update on your side? Nested elements do not seem to receive the edit labels, unless you add some padding to them. For us it's a minimum of 2px, not 1px as in the other comment.

Issue should be fixed as part of this release: https://github.com/contentful/live-preview/releases/tag/%40contentful%2Flive-preview%404.5.15


https://docs.nuget.org/ has incorrect certificate

Apparently the website docs.nuget.org is hosted as azure website, and it has the default *.azurewebsites.net certificate, which is apparently marked as failure in my browser. (ERR_CERT_COMMON_NAME_INVALID).

This is fixed in the meanwhile.


Fix checking of name

Closes #144
Update includes:

Numbers not allowed within name
Hyphens and apostrophes allowed within name
Spaces in front and behind name not allowed

Test for trailing whitespace in TutorTest#isSameTutor() removed due to update in regex.

Codecov Report
Base: 73.94% // Head: 74.27% // Increases project coverage by +0.33% :tada:

Coverage data is based on head (ac1a89a) compared to base (929eaa8).
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #149      +/-   ##
============================================
+ Coverage     73.94%   74.27%   +0.33%
- Complexity      627      648      +21
============================================
  Files           100      102       +2
  Lines          1842     1893      +51
  Branches        215      224       +9
============================================
+ Hits           1362     1406      +44
- Misses          417      422       +5
- Partials         63       65       +2

Impacted Files
Coverage Δ

src/main/java/tuthub/model/tutor/Name.java
80.00% <ø> (ø)

src/main/java/tuthub/commons/util/StringUtil.java
90.62% <0.00%> (-4.12%)
:arrow_down:

src/main/java/tuthub/ui/MainWindow.java
0.00% <0.00%> (ø)

...odel/tutor/StudentIdContainsKeywordsPredicate.java
100.00% <0.00%> (ø)

...or/TeachingNominationContainKeywordsPredicate.java
100.00% <0.00%> (ø)

...ub/model/tutor/ModuleContainsKeywordPredicate.java

...b/model/tutor/ModuleContainsKeywordsPredicate.java
100.00% <0.00%> (ø)

...thub/model/tutor/TagContainsKeywordsPredicate.java
100.00% <0.00%> (ø)

...n/java/tuthub/logic/commands/FindByTagCommand.java
100.00% <0.00%> (ø)

...tuthub/logic/parser/FindByPrefixCommandParser.java
96.20% <0.00%> (+0.42%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add requested_at timestamps to output requests.

Describe your changes
The timestamps will be used to log waypoints in the server for debugging.
Check these boxes or delete any item (or this section) if not relevant for this PR.

[x] Client+Server: this change is compatible with old servers
[x] Client forward compatibility: this change ensures client can accept data intended for later versions of itself

Note on protobuf: protobuf message changes in one place may have impact to
multiple entities (client, server, worker, database). See points above.

@prbot approve


Carrier grade SBC but not certified by MS

Hello!
What if SBC is from a carrier-grade solution used in a CSP(Mobile Operator) network while not certified by MS? what would be missed in features especially in integration with MS-teams?
Thanks

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

ID: 1b1a1c01-10a9-7f69-1c7b-1a566b0f5112
Version Independent ID: 329ee759-40fc-98fb-b91e-87786c63024b
Content: Session Border Controllers certified for Direct Routing - Microsoft Teams
Content Source: Teams/direct-routing-border-controllers.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Fix issue with numpy 1.24

@CoreyCole does this work for you?

@CoreyCole does this work for you?

I tested this branch with the egs/ami/rttm_test_* and my own data and it is fixed for both using numpy==1.24
:+1:

@CoreyCole does this work for you?

I tested this branch with the egs/ami/rttm_test_* and my own data and #22 is fixed for both using numpy==1.24
👍

Thanks for testing it. I'll merge.


Match most specific route if multiple routes match by predicate

Ordering of routes based on PathPattern.SPECIFICITY_COMPARATOR so that if multiple routes' predicates match, the one with most specific path pattern wins.

But not all routes are matched on path.

Was thinking of how to do this.
All routes can have a path pattern explicit or implied. The ones without a path predicate can have /** as pattern else it will be the one in the path predicate.
Any recommendations?

It can't be ordered by path at all.

Other implementations of AbstractHandlerMapping in spring (AbstractUrlHandlerMapping, RequestMappingInfoHandlerMapping etc) have best match semantics based on path.
Would it be fine to add a nullable PathPattern field to the Route?
The routes can be sorted comparing first on order and then on pathPattern (nulls would come last)
If multiple routes match all predicates, the best matching route will be chosen during routing.

I'm not going add anything related to path to a route. I specifically created it this way to be able to match on other parts of the request.
Can we step back and ask why you want this?

Ok.
I'm mainly trying to route on longest prefix path matching. What I'm trying to do is
if path ~ "/abc/" =>             handler1()
if path ~ "/abc/special/" => handler2()
Is there a proper approach for this?

I meant
if path ~ "/abc/**" => handler1()
if path ~ "/abc/special/" => handler2()

If request comes to /abc/special, handler2 should be resolved.

Order handler 2 before 1

What version are you using?

	@Bean
	public RouteLocator customRouteLocator(RouteLocatorBuilder builder) {
		return builder.routes()
				.route(r -> r.order(1)
						.host("**.abc.org").and().path("/image/png")
						.filters(f ->
								f.addResponseHeader("X-TestHeader", "foobar"))
						.uri("http://httpbin.org:80")
				)
				.route(r -> r.order(-1)
					.path("/image/webp")
					.filters(f ->
							f.addResponseHeader("X-AnotherHeader", "baz"))
					.uri("http://httpbin.org:80")
				)
				.build();
	}

Closing this due to inactivity. Please re-open if there's more to discuss.

Hi @spencergibb , is there anything routeLocator.getMatchingPath(path) like zuul in spring cloud gateway ?

No. Zuul only worked with path. Gateway can match on many things, not just path

Thanks @spencergibb . Any recommend way if i need to match path , from routelocator.getRoutes() ? need to write own logic to check ?

No. All you have at that point is a predicate, it may be more than path.

Yeah Right, in predicate , i have more than path, which i need to parse and match probably.


Fix bug related to tag and group

The official repository has updated certain datasets by changing the key from group to tag. However, this update was applied inconsistently, resulting in some datasets using tag while others still use group. This inconsistency is causing issues during task evaluation, as the code isn't handling both cases properly.
To work around this, the easiest solution I've found is to dynamically retrieve either group or tag from the configuration, depending on which one is present.
Before:
tasks_and_groups[tag_group]

After:
tag_group = config["group"] if "group" in config else config["tag"]
tasks_and_groups[tag_group]

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 5 committers have signed the CLA.:white_check_mark: zxcvuser:white_check_mark: haileyschoelkopf:x: Ankush Rana:x: hrosegalb:x: PaulNdreiAnkush Rana seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Sorry, I did it by mistake.


Support dynamic color

Please support dynamic color feature.

I tried this feature when migrating to Material 3. Coffee only uses one color: <color name="coffee_brown">#4E342E</color>. IMO the connection between the app name and color is too strong to change it.
The quick tile is already themed on Android 12+.


Fix a few small bugs in CFLocale

Adds missing mutex unlock in an error path
Fixes use of an incorrect local variable
Addresses a few areas where _prefs member was assumed to be non-NULL

Fix to __CFLocaleGetMeasurementSystemGuts and __CFLocaleCopyTemperatureUnit
modelled after __CFLocaleCopyCollatorID
Fix to __CFLocaleEqual handles the case where one or both CFLocales
have NULL _prefs.

@parkera Please let me know if I should stop mentioning you on these CF patches, but it seems like you're the right person for these.

@swift-ci please test

@mark1smi That's fine, thanks.


Update logback-classic from 1.5.8 to 1.5.12

About this PR
📦 Updates ch.qos.logback:logback-classic from 1.5.8 to 1.5.12
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "ch.qos.logback", artifactId = "logback-classic" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "ch.qos.logback", artifactId = "logback-classic" }
}]

labels: test-library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #136.


docs(tutorial): update DCI

close https://github.com/blindnet-io/devrel-management/issues/81

Rewritten after merge because of a failed push (to update the commit message). Cf. d87ac90


Constraints not working properly - Swift 3

I am using flipper inside a view controller having navigation bar and tab bar. For now, I have a label at the top and an image below in the flipper detail view. Whenever I put top constraints to label in the detail page view (PageTestViewController's IB, as referred to in your example), the view shifts slightly down after loading, during runtime. No shifting happens if I keep the image horizontally and vertically centered and remove the label totally. I am unable to understand what's exactly happening here.
One more issue is that I need to load the flipview only once a button is clicked, not on page load. How to accomplish that?

Any chance you can create a sample project that illustrates the problem you are having? I am having trouble picturing how your view hierarchy is set up.

Will share a demo project shortly

I have uploaded demo project to this link: https://www.dropbox.com/s/ikl5m266cc4on7t/Demo.zip?dl=0

@Mamtak1811 thanks for uploading that. There seems to be something funky if you set up the flipperView via constraints. I'll have to look into that. A work around is to remove the flipperview from your TopStoriesScene and turn the viewcontrollers view into a DJKFlipperView. Also be sure to connect the outlet again. Check out the screen shot below.

I tried that too. It does not work with top constraints in the SportsScene.

I see now. It seems creating the pages that flip in the storyboard cause issues. It has something to do with the toplayoutguide being set dynamically.
If you remove the SportsFliperViewController from the storyboard and create a separate .xib for it, it should honor the top constraint correctly.

Thanks! That works perfectly fine.
Please also answer this question:  I need to load the flipview only once a button is clicked, not on page load. How to accomplish that?

@Mamtak1811 i'm glad that worked.
I pushed an update 0.1.9 setting the datasource no longer auto refreshes the view. You can set up your datasource and array, but call reload() whenever you want the flipperview to show.


Redirect cn.eslint.org to zh-hans.eslint.org site

The new Simplified Chinese site is ready, let's say goodbye to cn.eslint.org. Now it's time to take people to the new site.

It looks like there's no easy way to do this within Netlify DNS, so investigating other alternatives.

It might be possible to configure this using the _redirects file
https://docs.netlify.com/routing/redirects/

Yes, that's what I'm doing. I'm going to point both docs.eslint.org and cn.eslint.org to eslint.org and then use redirects to get them to the right place.


awesome_print wont work in Rails if ActiveRecord is not used

Hi Michael,
Like your awesome_print. It's truly awesome! We want to use it in our Rails app, but since we are not using ActiveRecord (it's not even loaded), when we run ap, it complains about missing constant ActiveRecord::Base. Here is the stack trace for this problem:
#Class:0x61e14e8: uninitialized constant AwesomePrintRails::ActiveRecord
from C:/Ruby/lib/ruby/gems/1.8/gems/activesupport-2.3.5/lib/active_support/dependencies.rb:440:in load_missing_constant' 	from C:/Ruby/lib/ruby/gems/1.8/gems/activesupport-2.3.5/lib/active_support/dependencies.rb:80:in const_missing'
from C:/Ruby/lib/ruby/gems/1.8/gems/awesome_print-0.1.2/lib/ap/mixin/rails.rb:17:in printable' 	from C:/Ruby/lib/ruby/gems/1.8/gems/awesome_print-0.1.2/lib/ap/awesome_print.rb:128:in awesome'
from C:/Ruby/lib/ruby/gems/1.8/gems/awesome_print-0.1.2/lib/ap/core_ext/kernel.rb:10:in send' 	from C:/Ruby/lib/ruby/gems/1.8/gems/awesome_print-0.1.2/lib/ap/core_ext/kernel.rb:10:in ai'
from C:/Ruby/lib/ruby/gems/1.8/gems/awesome_print-0.1.2/lib/ap/core_ext/kernel.rb:15:in `ap'
from (irb):12
We are running Rails 2.3.5 on Ruby 1.8.6 on Windows platform.
Do you think it's a bug and can be easily fixed?
Thanks,
Alex

i`d rather like to use AP without beeing forced to use ActiveRecord.

@thedanielhanke or @alexw668 How did you go about creating a Rails application which does not load AR?

@gerrywastaken We have apps that uses no-SQL database like MongoDB or Neo4j (a graph database) that don't deal with AR.

@alexw668 I mean, what is the process you used to stop it from loading. As in which file did you modify and how, so I can test the issue accurately and not end up making bad assumptions about your setup.

@gerrywastaken sorry for this short delay.
head config/application.rb -n 10
require File.expand_path('../boot', __FILE__)

require 'rails'
%w(
  action_controller
  action_view
  action_mailer
).each do |framework|
  begin
    require "#{framework}/railtie"

was all i needed to do.

can be fixed like this:
    def cast_with_active_record(object, type)
      cast = cast_without_active_record(object, type)
      return cast if !defined?(::ActiveRecord)

      if object.is_a?(::ActiveRecord::Base)

should become
    def cast_with_active_record(object, type)
      cast = cast_without_active_record(object, type)
      return cast if !defined?(::ActiveRecord::Base)       #< -----------------

      if object.is_a?(::ActiveRecord::Base)

or even by unless (defined?(::ActiveRecord) || defined?(::ActiveRecord::Base)

I'm not sure how this ended up closed. I haven't yet had a chance to look into it. Thanks for the info @thedanielhanke 👍

Ok so the original bug was created a long time ago so tracking it back, the problem was originally here:
https://github.com/awesome-print/awesome_print/blob/0.1.3/lib/ap/mixin/rails.rb#L17
It then moved here:
https://github.com/awesome-print/awesome_print/blob/0.2.0/lib/ap/mixin/active_record.rb
This commit should have then fixed the issue reported by @alexw668:
https://github.com/awesome-print/awesome_print/commit/05265cb4e53b48199241fe8a26bd97a47e3541ea#diff-e567c486212ebaee39cf4091c38a4d84R17
Then in this commit the AcitveRecord::Base check was swapped for an ActiveRecord check:
https://github.com/awesome-print/awesome_print/commit/dfbebd488769738ac854adf37c930ad1d31bb94f#diff-e567c486212ebaee39cf4091c38a4d84L17
Now this should not have affected @alexw668 as for him ActiveRecord isn't defined at all, so I'm unclear how is he would still be hitting the original bug.
@thedanielhanke Your bug appears to be that ActiveRecord is somehow getting defined by something (before awesome print is loaded), but it doesn't create ActiveRecord::Base.
Your fix is good, but I don't think it's the same issue as was reported here. I suspect the issue reported here was fixed quite some time ago. @alexw668 can you confirm?

ok i created a new empty project and yes - you are right.
the evil is msgpack-rails which messes around with stuff like
module ActiveRecord
  module Serialization
    include ActiveModel::Serializers::Msgpack
  end
end

i know that they fail, but i wonder if that "change" might help out - i guess there are many further bad written gems out there like msgpack-rails.
however, from my side this can be marked as closed. Thank you, @gerrywastaken

I'm just glad your issue was solved. I think it's a good change regardless and I suspect there will be others who are affected by this issue so I'll likely merge it in.
Cheers for investigating the root cause, I really appreciate that. 👍
I suspect @alexw668 was the one who originally closed this, but can't be sure as Github isn't telling me. I'll leave this open for a three more days to give @alexw668 a chance to respond.

you are welcome + cheers for this gem ;)


Rerun Migrations Issues.

If you run migrations more then one time then it will give you an error say this table is already exists. There is no validation in the code. It just put long lengthy errors.

@AlexAslam Migrations can generate idempotent scripts if you need them. If this isn't what you are looking for, then please provide all the information asked for in the issue template (see below) so that we can effectively understand and reproduce the issue you are seeing.
Describe what is not working as expected.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message:
Stack trace:

Steps to reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.
Console.WriteLine("Hello World!");

Further technical details
EF Core version: (found in project.csproj or packages.config)
Database Provider: (e.g. Microsoft.EntityFrameworkCore.SqlServer)
Operating system:
IDE: (e.g. Visual Studio 2017 15.4)

Seriously Guys, the thing is I run migrations multiple times on the specific context, then migration on specific context will give me the error table already exists in the database. I try to find solution of this error on the net but what I find is many guys facing this issue.
Please try to understand what I’m talking about.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10

From: Arthur Vickers<EMAIL_ADDRESS>Sent: Sunday, April 8, 2018 3:47:25 AM
To: aspnet/EntityFrameworkCore
Cc: AlexAslam; Mention
Subject: Re: [aspnet/EntityFrameworkCore] Rerun Migrations Issues. (#11580)
@AlexAslamhttps://github.com/AlexAslam Migrations can generate idempotent scriptshttps://docs.microsoft.com/en-us/ef/core/managing-schemas/migrations/ if you need them. If this isn't what you are looking for, then please provide all the information asked for in the issue template (see below) so that we can effectively understand and reproduce the issue you are seeing.
Describe what is not working as expected.
If you are seeing an exception, include the full exceptions details (message and stack trace).
Exception message:
Stack trace:
Steps to reproduce
Include a complete code listing (or project/solution) that we can run to reproduce the issue.
Partial code listings, or multiple fragments of code, will slow down our response or cause us to push the issue back to you to provide code to reproduce the issue.
Console.WriteLine("Hello World!");
Further technical details
EF Core version: (found in project.csproj or packages.config)
Database Provider: (e.g. Microsoft.EntityFrameworkCore.SqlServer)
Operating system:
IDE: (e.g. Visual Studio 2017 15.4)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/aspnet/EntityFrameworkCore/issues/11580#issuecomment-379504539, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AROi3nwxlbC-4BAAy4-wF6AH0W1YJfmsks5tmUH9gaJpZM4TK6h1.

@AlexAslam This is not how Migrations works. If you apply Migration A to Database A and then try and apply the migration again, it will be a no-op - We store a list of applied migrations in a table in the target database.


The program 'node' can be found in the following packages:

node
nodejs-legacy
Try: sudo apt-get install

Try doing sudo apt-get install nodejs-legacy and then node -v

Here's more info on this

v0.10.25 that is it response.

If node -v worked, can you try:
rm -rf node_modules
npm install

You might have to run rm -rf node_modules with sudo.

the npm install has finish now but when i goto localhost:5000 it does not work.

The app starts fine when you run npm start?

a new error. when i did npm start
jobinpal@Jobinpal ~/lets-chat> npm start

<EMAIL_ADDRESS>prestart /home/jobinpal/lets-chat
migroose

/home/jobinpal/lets-chat/migroose.js:10
if (err) { throw err; }
^
Error: failed to connect to [localhost:27017]
at null. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/server.js:555:74)
at EventEmitter.emit (events.js:106:17)
at null. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection_pool.js:150:15)
at EventEmitter.emit (events.js:98:17)
at Socket. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection.js:534:10)
at Socket.EventEmitter.emit (events.js:95:17)
at net.js:441:14
at process._tickCallback (node.js:415:13)
npm ERR! weird error 8
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian
npm ERR! not ok code 0
jobinpal@Jobinpal ~/lets-chat> npm start

<EMAIL_ADDRESS>prestart /home/jobinpal/lets-chat
migroose

/home/jobinpal/lets-chat/migroose.js:10
if (err) { throw err; }
^
Error: failed to connect to [localhost:27017]
at null. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/server.js:555:74)
at EventEmitter.emit (events.js:106:17)
at null. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection_pool.js:150:15)
at EventEmitter.emit (events.js:98:17)
at Socket. (/home/jobinpal/lets-chat/node_modules/mongoose/node_modules/mongodb/lib/mongodb/connection/connection.js:534:10)
at Socket.EventEmitter.emit (events.js:95:17)
at net.js:441:14
at process._tickCallback (node.js:415:13)
npm ERR! weird error 8
npm WARN This failure might be due to the use of legacy binary "node"
npm WARN For further explanations, please read
/usr/share/doc/nodejs/README.Debian
npm ERR! not ok code 0

You need MongoDB running :)

how do i do that please...

Here you go

Please I will continue tommorow.
On Feb 19, 2015 8:25 PM, "Houssam Haidar"<EMAIL_ADDRESS>wrote:

Here you go
http://www.mongodbspain.com/en/2014/08/30/install-mongodb-on-ubuntu-14-04/
—
Reply to this email directly or view it on GitHub
https://github.com/sdelements/lets-chat/issues/272#issuecomment-75118248
.

Thanks for your support @hhaidar but i went to the link you gave me the second step
2. Generate a file with the MongoDB repository url
$ echo 'deb http://downloads-distro.mongodb.org/repo/ubuntu-upstart dist 10gen' | sudo tee /etc/apt/sources.list.d/mongodb.list
deb http://downloads-distro.mongodb.org/repo/ubuntu-upstart dist 10gen
OUTPUT
deb http://downloads-distro.mongodb.org/repo/ubuntu-upstart dist 10gen
No command 'deb' found, did you mean:
Command 'xdeb' from package 'xdeb' (universe)
Command 'debc' from package 'devscripts' (main)
Command 'deb3' from package 'quilt' (main)
Command 'debi' from package 'devscripts' (main)
Command 'derb' from package 'icu-devtools' (main)
Command 'dwb' from package 'dwb' (universe)
Command 'dab' from package 'bsdgames' (universe)
deb: command not found

What OS are you running and which version?
Please have a look at mongodb's install docs: http://docs.mongodb.org/manual/installation/

You might want to look at running lets chat using a Docker container.

thanks  gues it is working but how do i do connect with other users and upload files.

You can tell users to make accounts on your instance. You can access the file menu from the sidebar.

Hi,
I'd like to add you to my professional network on LinkedIn.

Udeme

Accept: https://www.linkedin.com/blink?simpleRedirect=0SdjwVdzcVd3cPcPwQdj0QejARfkh9rCZFt65QqnpKqipns4lKs2R6hPRVpkJApn9xq7cCt7dBtmtvpnhFtCVFfmJB9C5QoORBt6BSrCAJt7dBtmsJr6RBfmtKqmJzon9Q9ClQqnpKimtBkClOs3Rx9CQRbnsOu6ZIqjxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
You received an invitation to connect. LinkedIn will use your email address to make suggestions to our members in features like People You May Know. Unsubscribe here: https://www.linkedin.com/blink?simpleRedirect=rjkJtP9UrSNFe6AJpTdIu64NfmhFpipjcQhIpTwUe3h8rk9Os7hipB56szkOskZzr49hlltAqkhAs3kOlBF4hBhaiPB1jTlTj6pqrAVOsPhUhBlMold8rmJ3cj9Lcn1WoAZHpRcJnSdhpDhEgRESckVOs51LrTxDokQPm4dWijt5pSwVjARRtllktBlmik5UhR8Utk5UoQtfhTEMpBlaukpqhTEUl6xej2R1kDBvkkVQsmR7hCsQlCxyj6kPrklMhnp6s39ds3xVlk51gmsUcnhmjQ4VilZGhB51fmhFrSMCt7dBtmtvpnhFtCVFfmJB9B4ScSh5hSxmhD9JhB51fmVBqSZkp6BJ9CVRr3RQ9CQRbnsOu6ZIqjxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead= Learn why we included this at the following link: https://www.linkedin.com/blink?simpleRedirect=e3wTd3RAimlIoSBQsC4Ct7dBtmtvpnhFtCVFfmJB9CNOlmlzqnpOpldOpmRLt7dRoPRx9CQRbnsOu6ZIqjxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
© 2015, LinkedIn Corporation. 2029 Stierlin Ct. Mountain View, CA 94043, USA

Udeme Samuel would like to connect on LinkedIn. How would you like to respond?
Accept: https://www.linkedin.com/blink?simpleRedirect=dzkUejoPejgPcPcUd3kMd3AVdjR4imVLqnhxt6BSrCAClT15rD0JhAsZumlbp6lOomxP9DhUpnhvt71BoSdxfndQ9CZOpmwZrioPfmBI9z4MnT9Bp6VFrmlOnThPpnlDnThPsCBCfmJB9ClQqnpKimtBkClOs3Rx9z8RbmFytCRBsPxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
View Udeme Samuel's profile: https://www.linkedin.com/blink?simpleRedirect=0VdjgRc38Vcj4Zh4BOpm9JpmQCt7xBt5ZBr6BCrT9MfndQ9CZOpmwZrioQfmBI9z4MnT9Bp6VFrmlOnThPpnlDnThPsCBCfmJB9DpMrzRQ9z8RbmFytCRBsPxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
You are receiving Reminder emails for pending invitations. Unsubscribe here: https://www.linkedin.com/blink?simpleRedirect=rmZzhj8BoDlEt6BDhj8BumNMpn8Md2kPem9ze3sPc6lzejoNoj8Vd69xpChCc34Nc30Mc30Mc6pzczBCp3wTdmoPoP0McPoQoClAe68Np3gMc34PpCkQcPgOcjkPdm5xpjcVe68Ppm8Nc312cylVr71BszRIqm5JpioMdz0MczRAqioSdjwVdzcVd3cPcPwQdj0QejARfmhFpOoNc5ZOpmhKqmRBsBZQsSlRpRZQsT9FpzRHpipQu6lQnS9RsSVRbjkJoDlPrDkJcj1vsClArCBJpn9vt7dBtmtvt7dOqmoJr6RBfmtKqmJzon9Q9CZLpPRQ9z8RbmFytCRBsPxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
You received an invitation to connect. LinkedIn will use your email address to make suggestions to our members in features like People You May Know. Unsubscribe here: https://www.linkedin.com/blink?simpleRedirect=38RbmFytCRBsPxFbmtPr7xxcjRAqmkCiTgTuCFnpBh2s5Z6d6NarStzc6lKgn1FekV2j4tTlRBVs4FTsDhldChgsz5ngkATi4hfcj18dksTp65ei65hojlvdkF6oT9Wkn1iqnFBsyR9iAhpk6lFkkpeiSwUcDdPnSkUhQJMj6NRpCxHnSFGdkNnpS8Nkldmpk9fikRJdkVRokQSsQ9UsCZEt4Fdp7Fvc6RzikFxu5BCr3hNoAd7hQwPh7B5jCBfdyRNnP0OqyRCuBl1gk5DsncRdl1hi4lvhklhgjRAqmZI9z4MnT9Bp6VFrmlOnThPpnlDnThPsCBCfmJB9B4ScSh5hSxmhD9JhB51fmVBqSZkp6BJ9CVRr3RQ9z8RbmFytCRBsPxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=This email was intended for sdelements/lets-chat. Learn why we included this at the following link: https://www.linkedin.com/blink?simpleRedirect=3wUdPgZp4BBr6dFt79x9z4MnT9Bp6VFrmlOnThPpnlDnThPsCBCfmJB9CNOlmlzqnpOpldOpmRLt7dRoPRx9z8RbmFytCRBsPxFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
© 2015, LinkedIn Corporation. 2029 Stierlin Ct. Mountain View, CA 94043, USA

Udeme Samuel would like to connect on LinkedIn. How would you like to respond?
Accept: https://www.linkedin.com/blink?simpleRedirect=0SdjwVdzcVd3cPcPwQdj0QejARfkh9rCZFt65QqnpKqipns4lKs2R6hPRVpkJApn9xq7cCt7xBt5ZQs6lzoS4ZsTgCrT9Bq3RJ9zcZqmMCcj1vsClArCBJpn9vt7dBtmtvp6VLoSlPfmJB9ClQqnpKimtBkClOs3Rx9zoQbjlHcnpBczBFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
View Udeme Samuel's profile: https://www.linkedin.com/blink?simpleRedirect=3ARd3kMczANcjR4in9BoCRBripQu6lQnSlIqmpLsD0ZsTgCrT9Bq3RJ9zgZqmMCcj1vsClArCBJpn9vt7dBtmtvp6VLoSlPfmJB9DpMrzRQ9zoQbjlHcnpBczBFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
You are receiving Reminder emails for pending invitations. Unsubscribe here: https://www.linkedin.com/blink?simpleRedirect=6RLoQkO9m9Rq7hFpQkO9nBIs6lOc3gBcPByoPwTcP1BoPAScm4OejhyompApz0Ncj0Mc30Mc31CoP8VpCgUdPlCcScMc3cSd69Bp3xycmgQc30NcSpBd3cQcz4RcPlxomkPejxycSlycj0Mgz8BumNMpn8Zr6BxrmkCc3oMc38Zp6ACdzkUejoPejgPcPcUd3kMd3AVdjRAqmsCcj1vsClArCBJpn9vt7dBtmtvp6VLoSlPfmJB9DhUpnhvoDlPrDkJdyRytndKtiQNc5ZOpmhKqmRBsBZQsSlRpRZArCZzpncJr6RBfmtKqmJzon9Q9CZLpPRQ9zoQbjlHcnpBczBFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
You received an invitation to connect. LinkedIn will use your email address to make suggestions to our members in features like People You May Know. Unsubscribe here: https://www.linkedin.com/blink?simpleRedirect=dzgJdmINtCkOemAJpTdIu64NfmhFpiplqnF4jAVntmFDgk4JhPlqk6BbkBlmj6JId39EolBylDxEh51Fcjhchmdpp5BWcAB4iRdkdCgOq4xzomlom4NNiCIRoTxhsPh6r5dSp4lvhP9Eq7dhkQ9Le4VUrT9MelZInTAMhQVfqkhDbk9CgQUNlQxCtTpDlAZ6qmNLoj5SsCcUs7hPr59Vhn0MhDFFmmJzp78RgQ4UqDoMtBZIcjcVhRhRbmFohClVqkNde51JgQkMlk51gl5boll9lRpblDF3hl51fmhFrSMCcj1vsClArCBJpn9vt7dBtmtvp6VLoSlPfmJB9B4ScSh5hSxmhD9JhB51fmVBqSZkp6BJ9CVRr3RQ9zoQbjlHcnpBczBFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=This email was intended for sdelements/lets-chat. Learn why we included this at the following link: https://www.linkedin.com/blink?simpleRedirect=e3wTd3RAimlIoSBQsC4Ccj1vsClArCBJpn9vt7dBtmtvp6VLoSlPfmJB9CNOlmlzqnpOpldOpmRLt7dRoPRx9zoQbjlHcnpBczBFbmtPr7xxcjRBfP9SbSkLrmZzbCVFp6lHrCBIbDtTtOYLeDdMt7hE&msgID=I5994054840039272448_500&markAsRead=
© 2015, LinkedIn Corporation. 2029 Stierlin Ct. Mountain View, CA 94043, USA


Feature Request: SetActiveByName/ByPid

Is there any way to make Window active by process name found? or/and by pid?
I've just tried
	pids, _ := robotgo.FindIds("gogland")
	if err == nil && len(pids)>0 {
		robotgo.ActivePID(pids[0])
   }
without any success.
Where I'm wrong?

Go version: 1.8.1
OS: Mac OS X 10.12.6

I will test it.


bump pins for dev build

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

This will make a gufe package on out dev channel that can be installed like this:
conda install -c conda-forge/label/gufe_dev -c conda-forge gufe


Big change in v1.7.x branch

Hello,
I made a big change in the roles Ansible.
Thus, the certificates are now generate on the local server and send to different servers.
The roles have also been reorganized.
I also remove the Vagrantfile, because I plan to rework them to have a "behavior" classic.
Maxence

Add ceph storage

@flemzord  Thanks you! I will checking.

Update @kairen ?


Version bump for new changes.

You're welcome to suggest another version number.

develop is on 0.4.4. But since I haven't been able to release a new stable I'll merge this in now.


[image-import] windows: bring disks online before translate

This happened when importing a non-uefi windows-2019 image: the secondary disk was offline, so it failed.
Just simply bring all offlined disk online to fix. Manually tested and it passed.

Why was the disk offline?

Romain and Andrew can answer it better. So far the theories are:

MBR disk may not be online automatically when boot disk (translator) is UEFI.
Devices on different bus may not be online automatically.

I had thought this may be because of the different partition tables (mbr vs gpt), but roman did some digging and this may instead be because the attached disk was on a separate bus, so maybe that's why this doesn't fail all the time?


ckeditor is incompatible with ManifestStaticFilesStorage

Hi guys, i have now an error with django-ckeditor 4.4.8 in my web server.
The webserver is webfaction, and my problem is the next:
I was using ckeditor in my page and work's fine, i have 4.4.7 and then updated for 4.4.8, i don't remember if i restarted the server but ckeditor works fine.
Then, i restarted the server and i try to used ckeditor but i couldn't see anything, see:

I tried all things that i think, re collect static files, re put ckeditor urls in urls.py, all, but, nothing working.
But, in local server work fine, shure, doesn't used collecstatic, don't need it, but in the server doesn't works this.
Finally, i installed 4.4.7 and works again.

Compare the urls from the bugs in the before image and see here:

The source are correct loading in this case, correct url, but, with 4.4.8 the statics urls are wrong.
Then, i think that is the problem of 4.4.8 with static urls, but, i don't know why.
Can anyone fix it?

Looks like no slash at the beginning of the path. The change made to URL generation of JS files was this one: https://github.com/django-ckeditor/django-ckeditor/commit/23b8a2ec1478162f96b7677c2d40194d5dcdd62c and it should work if your STATIC_URL is correctly set.

No, the problem is that first part of url is wrong, news/new/.../static isn't correct, the correct ist static/static/...

if the src is ="static/foo/bar.js" then it will be relative path which is wrong. if it would be "/static/foo/bar.js" then it should work correctly. What's the actual path in the raw HTML?

I don't understand your question.
I saw that ckeditor put the urls as page's url childs, in this case, news/new/ckeditor...., and these isn't the correct url.

Open the page source and check how they look. I have the latest version deployed in few places and all static files are working correctly. I assume you have something wrong with STATIC_URL - so check in the source, not in the rendered  DOM what's the URL. and check/post what's your STATIC_URL.

Wow, for now i can't do this, i fix the problem downgraded my django-ckeditor version, but tomorrow i try replicate this issue, thank you.

you can even do it without installing the new version. You can test the static() and STATIC_ROOT code example in manage.py/django-admin shell

STATICFILES_DIRS = ('/home/salahaddinal/webapps/tiempoturco/tiempoturco/statics',)
STATICFILES_STORAGE = 'django.contrib.staticfiles.storage.CachedStaticFilesStorage'
STATIC_ROOT = '/home/salahaddinal/webapps/tiempoturco_static/static'
STATIC_URL = '/static/static/'

I have this.

I had the same problem of the field no showing.
It worked when i added DefaultStorageFinder on settings.py
(i don't know why)
STATICFILES_FINDERS = (
     'django.contrib.staticfiles.finders.FileSystemFinder',
     'django.contrib.staticfiles.finders.AppDirectoriesFinder',
-    #'django.contrib.staticfiles.finders.DefaultStorageFinder',
+    'django.contrib.staticfiles.finders.DefaultStorageFinder',
 )

If you have a custom storage then it's likely it will need its own finder too.
I'll also check if there isn't any CachedStaticFilesStorage regressions in the latest release.

Ok, i will put the DefaultStorageFinder in my settings and try if works, thanks.

Ok, don't work yet.
See please:

You can see my settings and the wrong urls.

And now i have this issue:

I delete all static files and try collect it again, and, see.

I think you need slash on the end of STATIC_ROOT - https://docs.djangoproject.com/en/1.8/ref/settings/#std:setting-STATIC_ROOT

Ok, i'll try with this.

Don't works yet.

I am seeing this too. It can only be seen if DEBUG is False otherwise static takes a shortcut.
Note that the ckeditor.js and ckeditor-init.js get found fine, it's config.js that is not found. This is because the 'static' call transforms the name to eg ckeditor.677756974519.js (hashes so as not to serve a stale version when updates exist), so if ckeditor is relying on its name to be ckeditor.js to find config.js, this will trip it up.
The quick solution for those that are hitting this is to revert 23b8a2e and go back to the STATIC_URL version. Longer term would be to modify ckeditor.js to be smarter in how it finds its dependencies

So it's not as easy as it looked like.

Reverted in master.

it's seems very difficult but, will be better use static but search the way for fix this bug.
Thank you man.

is there any update here?

What update do you expect? At the moment no changes here. When I'll find some time I'll do some work on this package (there is a nice todo list).

Oh yes, but, i was talking about the problem with change static_url gettin.

It's hard to reproduce that config.js error with development server. Maybe circus/dj-static test setup could help test that.

It seems I have the same problem here. If debug = True, then all is ok, otherwise ckeditor.js, ckeditor-init.js are founded but not config.js (since the requested path by my browser is <current_path>/config.js instead of static/ckeditor/ckeditor/config.js).
I'm using the version 5.0.3 from pip, my relevant (I think) config vars are:
STATICFILES_STORAGE = 'whitenoise.django.GzipManifestStaticFilesStorage' # just copy-pasted from Heroku dev center, I don't really understand what does it mean
PROJECT_ROOT = os.path.dirname(os.path.abspath(__file__))
STATIC_ROOT = os.path.join(PROJECT_ROOT, 'staticfiles')
STATIC_URL = '/static/'
# Extra places for collectstatic to find static files.
STATICFILES_DIRS = (
    os.path.join(PROJECT_ROOT, 'static'),
)

Reading this thread, I feel this is the same problem as mine, but I didn't see any workaround for my issue.

@SalahAdDin can you maybe rename this issue to something more descriptive?
I suggest ckeditor is incompatible with ManifestStaticFilesStorage?
For all the guys out there that run into the same issue and turn to Google for help...

the changelog for 5.2.0 has a fix for ManifestStaticFilesStorage but I don't see this working (using 5.2.1).
https://github.com/django-ckeditor/django-ckeditor/blob/master/CHANGELOG.rst
5.2.0
-----
#. Django 1.10 updates
#. Development dependencies bump
#. CKEditor 4.6.1
#. Paste image support
#. Fix for ManifestStaticFilesStorage

The fix in release 5.2.2 still doesn't work for me. See my comment in pull request #377 .

This will be fixed in the next release. Please test the master branch (if you can) and report any problems in this issue tracker.
Thanks!

I just tried commit 2aaff4ed and it still has problematic Javascript code that attempts to read an HTML attribute before the DOM element has been loaded yet.
However, I followed the instructions in the new documentation committed in e080a82914bd23,  and that seemed to work for me!

Hmm, so you're saying that the code in ckeditor-init.js is running before the script tag with ID ckeditor-init-script (which references ckeditor-init.js!) is available to the DOM? That's really surprising! Do you have additional details about your environment that could help reproduce this?

@matthiask Ah, sorry, I just realised that I was actually testing 5.2.2. (For some reason, the deployment script I'm using didn't recognise a git+https:// dependency in requirements.txt.) I just tried out the new code on my development machine and it seems to work. Good work!

Great, thanks for testing!

e080a82  has worked for me as well. Both in Development and in Production.
Production: heroku, gunicorn, whitenoise
using django-ckeditor==6.0.0


System.ArgumentOutOfRangeException: Length cannot be less than zero. (Parameter 'length')

I ran into this error after updating to the renamed client in v4 - its triggered when trying to log out another exception I've encountered since the upgrade (which appears to be caused by passing a null value to an option set when updating a contact record, though I haven't pinned it down fully yet).
Microsoft.Xrm.Sdk.SaveChangesException: An error occured while processing this request.
 ---> System.AggregateException: One or more errors occurred. (Length cannot be less than zero. (Parameter 'length'))
 ---> System.ArgumentOutOfRangeException: Length cannot be less than zero. (Parameter 'length')
   at System.String.Substring(Int32 startIndex, Int32 length)
   at Microsoft.PowerPlatform.Dataverse.Client.DataverseTraceLogger.GetFirstLineFromString(String textBlock)
   at Microsoft.PowerPlatform.Dataverse.Client.ServiceClient.ShouldRetryWebAPI(Exception ex, Int32 retryCount, Boolean& isThrottlingRetry)
   at Microsoft.PowerPlatform.Dataverse.Client.ServiceClient.Command_WebExecuteAsync(String queryString, String body, HttpMethod method, Dictionary`2 customHeaders, String contentType, String errorStringCheck, Guid requestTrackingId)
   at Microsoft.PowerPlatform.Dataverse.Client.ServiceClient.Command_WebAPIProcess_ExecuteAsync(OrganizationRequest req, String logMessageTag, Boolean bypassPluginExecution, CancellationToken cancellationToken)
   --- End of inner exception stack trace ---
   at System.Threading.Tasks.Task`1.GetResultCore(Boolean waitCompletionNotification)
   at System.Threading.Tasks.Task`1.get_Result()
   at Microsoft.PowerPlatform.Dataverse.Client.ServiceClient.ExecuteOrganizationRequestImpl(OrganizationRequest req, String logMessageTag, Boolean useWebAPI, Boolean bypassPluginExecution)
   at Microsoft.PowerPlatform.Dataverse.Client.ServiceClient.Execute(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceContext.Execute(OrganizationRequest request)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceContext.SaveChange(OrganizationRequest request, IList`1 results)
   --- End of inner exception stack trace ---
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceContext.SaveChanges(SaveChangesOptions options)
   at Microsoft.Xrm.Sdk.Client.OrganizationServiceContext.SaveChanges()

Somehow Contains is returning true but IndexOf must be returning -1, which is why we get the length less than zero error:
if (textBlock.Contains(Environment.NewLine))
  return textBlock.Substring(0, textBlock.IndexOf(Environment.NewLine));

We have not been able to get this failure to replicate..
however, we have decided to add a bit of paranoia code in to the logic path in order to help narrow down the issue.
that will be included in the next drop we make publicly.


Fix the conflict on function naming in ddsproxy.

Fix #1070

Codecov Report
Base: 4.14% // Head: 17.19% // Increases project coverage by +13.05% :tada:

Coverage data is based on head (27086a7) compared to base (f3163b4).
Patch coverage: 94.24% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #1071       +/-   ##
===========================================
+ Coverage    4.14%   17.19%   +13.05%
===========================================
  Files          24       21        -3
  Lines        7140     5099     -2041
===========================================
+ Hits          296      877      +581
+ Misses       6844     4222     -2622

Impacted Files
Coverage Δ

nanomq/bridge.c
0.00% <0.00%> (ø)

nanomq/pub_handler.c
47.92% <0.00%> (+47.02%)
:arrow_up:

nanomq/sub_handler.c
62.71% <ø> (+17.88%)
:arrow_up:

nanomq_cli/dds2mqtt/mqtt_client.c
0.00% <ø> (ø)

nanomq/apps/broker.c
24.07% <25.00%> (+6.08%)
:arrow_up:

nanomq/apps/broker_test.c
100.00% <100.00%> (ø)

nanomq/pub_handler_test.c
100.00% <100.00%> (ø)

nanomq/sub_handler_test.c
100.00% <100.00%> (ø)

nanomq/unsub_handler.c
52.94% <100.00%> (+52.94%)
:arrow_up:

nanomq/unsub_handler_test.c
100.00% <100.00%> (ø)

... and 22 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Cannot Run Code Using GPUs

Description
Currently, I am trying to run the code using GPUs instead of CPUs to speed up the process, but I am unable to make changes to the code to make it work.
Code Output
`python scripts/train_vqgan.py -o viper_rl_data/checkpoints/atari_vqgan -c viper_rl/configs/vqgan/atari.yaml
WARNING:absl:GlobalAsyncCheckpointManager is not imported correctly. Checkpointing of GlobalDeviceArrays will not be available.To use the feature, install tensorstore.
WARNING:jax._src.xla_bridge:No GPU/TPU found, falling back to CPU. (Set TF_CPP_MIN_LOG_LEVEL=0 and rerun for more info.)
JAX process: 0 / 1
JAX total devices: 1
JAX local devices: 1
Logging to viper_rl_data/checkpoints/atari_vqgan
'
To Reproduce
I changed all mentions of CPU in code to GPU and made sure that CUDA was running. I also tried to change the packages installed to run CUDA specifically

I believe so. I checked using nvcc --version command and installed using the CUDA 11 pip command provided in the documentation.

Hm, this seems like an issue with your install / Jax not finding the right CUDA install path. I would try to set the following variables:
export LD_LIBRARY_PATH=/usr/local/cuda/lib64:$LD_LIBRARY_PATH
export LD_LIBRARY_PATH=/usr/local/cuda/include:$LD_LIBRARY_PATH

Unfortunately, my result is still the same after setting the following variables. I am not sure why it is still doesn't detect any GPUs being used.

I went back and try reinstalling the correct version of jax for cuda and it is currently running with no errors which is why I am closing this issue. Thank you so much for your help and I will make sure to reach out if I run into any more problems.

I went back and try reinstalling the correct version of jax for cuda and it is currently running with no errors which is why I am closing this issue. Thank you so much for your help and I will make sure to reach out if I run into any more problems. !

After i made it work it always states failed dnn library initialization and states "couldnt create cudnn handle" did you experience that at all?

@NonsansWD I'm having the same issue. Did you have any success resolving the issue?


Edge Runtime Does Not Work With TinaCMS Self Hosted.

Describe the issue that you're seeing. Any Loom videos or screenshots usually help a lot!
Currently you can use the tina client (with Tina Cloud) in a cloudflare worker / edge runtime because it only depends on fetch but if you wan't to use self hosted TinaCMS you will need to add NodeJS dependencies that will not work in the Edge Runtime.
There are two pieces of work to get TinaCMS self hosted  working with an an edge runtime.
1. The database client and @tinacms/graphql have Node.js Dependencies
The database client depends on @tinacms/datalayer and @tinacms/graphql both which have NodeJS dependencies.
We would have to move all Node.js dependencies to @tinacms/cli
2. A TinaNodeBackend  that uses Javascript Request and Response.
Currently to host the graphql / auth backend a route helper is exported. We would need to add another version of this handler built on  Javascript  Request and Response.
Reproduction
https://github.com/logan-anderson/nextjs-cloudflare/tree/self-host
Steps to reproduce

Clone the repo
Check out the self-host branch
See errors

System Info
N/A (All latest TinaCMS packages)

Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug.

Not stale


Micronaut 1.0.0.M4 Compatibility

Hello!
I'm trying to use this with Micronaut 1.0.0.M4 and it seems to fail when loading some class definition.
It works by just changing the version to the latest in this library and publishing it locally. All tests pass and it seems to work for my service. I suppose that since Micronaut generates the class definitions at compile time, and that the contract got broken in between milestones then this adapter won't work with different milestones, even though nothing is wrong with it.
Are there any plans to keep this up with the latest milestones or even final release?
Also, are you accepting contributions to this project? So far I'm interested in having this running with latest Micronaut and not looking into feature development.
Thanks!

Note that once we release the RC on the 30th of September we won't be making any changes to the generated byte code format again. So only one more release will be needed

@fmayoral Hello! Sorry for the delay and thank you for using this project. I agree with you, the generated class definitions do not seem to work with Micronaut 1.0.0.M4.
If the RC solves this problem, it's just a matter of time. In the meanwhile, I published a version 1.3.0 that supports Micronaut 1.0.0.M4. Feel free to use it.
I definitively will keep this library up-to-date! And all contributions are welcome, of course. :)
@graemerocher Good to know! Do you know whether generating class definitions directly in micronaut libraries (like this one) is bad practice or not? Is there an other way to develop libraries for Micronaut without bundling their class definitions with them?

@looorent Just to clear the RC won't fix the problem. The RC will finalise the binary class format that Micronaut uses so that we will ensure going forward that binary compatibility is not broken with new releases of Micronaut. So you will be able to do a release when the RC is out and that release will work with future versions of Micronaut
It is not considered bad practise and currently there is no way to build Micronaut libraries without bundling the class definitions

@graemerocher That's clear. Thank you!

Sweet! thank you very much @looorent ! and also tank you @graemerocher for your comments


Nvidia Control panel removed

Even if you reinstall your drivers it will not show up and you have to use Nvidia experience program instead of the normal control panel.
Can this be fixed please so it do not delete just that progam?
If you want Nvidia Experiance program it want you to add e-mail on their site and I do not want that when windows have a good normal program to control your Graphic card!
Only way to get it back is to login to windows store and get it that way, could not find that program anywhere else, if you did not want the Nvidia Experience program that is!
Except that I love the program and what it does!

I will add this in so that it does not remove it.

Added this to the whitelist. Please confirm it works.

I tried it yesterday and I can access my Nvidia Control panel

Great, I haven't had the time to reinstall my computer but the person before say it works so you I just want to thank you and close this issue!


UI: Updated lineage UI as per new mock

Describe your changes :

UI: Updated lineage UI as per new mock

Updated UI for the pipeline as edge
Added icon for showing function
Updated edit icon

#8521

Type of change :

[x] Improvement

Frontend Preview (Screenshots) :
https://user-images.githubusercontent.com/71748675/200386929-ffc38e4e-c60c-42d5-8db9-5f71ff323292.mov

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers

@ShaileshParmar11 Can we centre the DAG upon Expand All / Collapse All click action?
Is Expand All / Collapse All convey that it's related to column-level lineage? Sounds bit confusing to me.
cc: @harshach

@ShaileshParmar11 Can we centre the DAG upon Expand All / Collapse All click action? Is Expand All / Collapse All convey that it's related to column-level lineage? Sounds bit confusing to me. cc: @harshach

Yes its related to column lineage


Custom remote transport: documentation & example

There seems to be an undocumented extension point for the remote transport, which can allow people to implement their own transport instead of Jquery .ajax.
I've updated the API docs, and added a basic example.
If this extension point isn't ready for public consumption, I'd suggest that we still leave it documented, but note that it's likely to be reworked in the future with possible breaking changes.
Related issue: #907

:+1: any updates on this?


jquery dependency used?

I see the jquery dependency but I can't see where do you need it?

Hey @BTooLs I'm sorry we haven't gotten back to you. This project is now unsupported and we won't be actively developing it anymore.
I'm not sure why we have that dependancy, but you can fork this repo and remove it if you would like!


Build broken on python 3.13-dev

rpds-py cannot be built on Python 3.13.0a6+.
Steps to reproduce
pyenv install 3.13-dev
pyenv virtualenv 3.13-dev rpds-test
pyenv shell rpds-test

$ pip install rpds-py
Collecting rpds-py
  Downloading rpds_py-0.18.1.tar.gz (25 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Building wheels for collected packages: rpds-py
  Building wheel for rpds-py (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for rpds-py (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [52 lines of output]
      Running `maturin pep517 build-wheel -i /home/aciba/.pyenv/versions/3.13-dev/envs/rpds-test/bin/python --compatibility off`
      📦 Including license file "/tmp/pip-install-mpcdnjtn/rpds-py_c225fa28d6e746c5bdcd1b2440b5d7ec/LICENSE"
      🔗 Found pyo3 bindings
      🐍 Found CPython 3.13 at /home/aciba/.pyenv/versions/3.13-dev/envs/rpds-test/bin/python
      📡 Using build options features from pyproject.toml
         Compiling target-lexicon v0.12.9
         Compiling once_cell v1.18.0
         Compiling autocfg v1.1.0
         Compiling proc-macro2 v1.0.66
         Compiling libc v0.2.147
         Compiling unicode-ident v1.0.11
         Compiling parking_lot_core v0.9.8
         Compiling heck v0.4.1
         Compiling scopeguard v1.1.0
         Compiling smallvec v1.11.0
         Compiling cfg-if v1.0.0
         Compiling portable-atomic v1.6.0
         Compiling static_assertions v1.1.0
         Compiling triomphe v0.1.9
         Compiling indoc v2.0.4
         Compiling unindent v0.2.3
         Compiling archery v1.2.0
         Compiling lock_api v0.4.10
         Compiling memoffset v0.9.0
         Compiling rpds v1.1.0
         Compiling pyo3-build-config v0.20.3
         Compiling quote v1.0.31
         Compiling syn v2.0.32
         Compiling parking_lot v0.12.1
         Compiling pyo3-ffi v0.20.3
         Compiling pyo3 v0.20.3
      error: failed to run custom build command for `pyo3-ffi v0.20.3`

      Caused by:
        process didn't exit successfully: `/tmp/pip-install-mpcdnjtn/rpds-py_c225fa28d6e746c5bdcd1b2440b5d7ec/target/release/build/pyo3-ffi-70093274310161c8/build-script-build` (exit status: 1)
        --- stdout
        cargo:rerun-if-env-changed=PYO3_CROSS
        cargo:rerun-if-env-changed=PYO3_CROSS_LIB_DIR
        cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_VERSION
        cargo:rerun-if-env-changed=PYO3_CROSS_PYTHON_IMPLEMENTATION
        cargo:rerun-if-env-changed=PYO3_PRINT_CONFIG
        cargo:rerun-if-env-changed=PYO3_USE_ABI3_FORWARD_COMPATIBILITY

        --- stderr
        error: the configured Python interpreter version (3.13) is newer than PyO3's maximum supported version (3.12)
        = help: please check if an updated version of PyO3 is available. Current version: 0.20.3
        = help: set PYO3_USE_ABI3_FORWARD_COMPATIBILITY=1 to suppress this check and build anyway using the stable ABI
      warning: build failed, waiting for other jobs to finish...
      💥 maturin failed
        Caused by: Failed to build a native library through cargo
        Caused by: Cargo build finished with "exit status: 101": `env -u CARGO PYO3_ENVIRONMENT_SIGNATURE="cpython-3.13-64bit" PYO3_PYTHON="/home/aciba/.pyenv/versions/3.13-dev/envs/rpds-test/bin/python" PYTHON_SYS_EXECUTABLE="/home/aciba/.pyenv/versions/3.13-dev/envs/rpds-test/bin/python" "cargo" "rustc" "--features" "pyo3/extension-module" "--message-format" "json-render-diagnostics" "--manifest-path" "/tmp/pip-install-mpcdnjtn/rpds-py_c225fa28d6e746c5bdcd1b2440b5d7ec/Cargo.toml" "--release" "--lib"`
      Error: command ['maturin', 'pep517', 'build-wheel', '-i', '/home/aciba/.pyenv/versions/3.13-dev/envs/rpds-test/bin/python', '--compatibility', 'off'] returned non-zero exit status 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for rpds-py
Failed to build rpds-py
ERROR: Could not build wheels for rpds-py, which is required to install pyproject.toml-based projects

I don't personally test on 3.13 yet, though I'm of course happy to -- it's possible that PyO3 0.21 works on it / has a fix for this, but the GIL API changes were definitely not trivial for me, I haven't been able to get something working yet. I pushed my partial attempt to https://github.com/crate-py/rpds/compare/main...pyo3-21 -- PRs are welcome to finish that. Otherwise this will happens when it happens.

Thanks for the heads-up. I have tried with PyO3 0.21 with the same result. Per https://github.com/PyO3/pyo3/issues/4038, it looks like 3.13 support is a nice to have for PyO3 0.22.

Python 3.13 seems to be supported with the new 0.22 release of PyO3 https://github.com/PyO3/pyo3/releases/tag/v0.22.0
Would this be enough to make rpds compatible? Would love to help if possible :)

Python 3.13 seems to be supported with the new 0.22 release of PyO3 https://github.com/PyO3/pyo3/releases/tag/v0.22.0
Would this be enough to make rpds compatible? Would love to help if possible :)

@FlickerSoul you might wanna take a look at the failures in https://github.com/crate-py/rpds/pull/76

Yep that PR is definitely what needs help -- I pushed a branch with my own WIP here:
https://github.com/crate-py/rpds/tree/pyo3-21
which is the first part of this (updating to 0.21) -- you may or may not find that helpful to start with.
(n.b. things are undoubtedly slower at the minute on my end. https://www.linkedin.com/feed/update/urn:li:activity:7213574952485089280/ has the quick/not so quick explanation. Just mentioning as I apologize if responses and/or attention takes longer than usual, and/or if you're wondering when or if I'll get to this myself!)

Thanks for the information! I'll take a look later this week :)

Experimentally, this will compile on py3.13 with PYO3_USE_ABI3_FORWARD_COMPATIBILITY = 1 set.  Might be worth defaulting to that to make future versions less painful to deal with (but on the other hand, the limited API is not supported with freethreading builds).

PyO3 removes the Clone trait on PyObject by default. We can re-enable it by adding py-clone feature on PyO3. I made two branches for each case.
https://github.com/crate-py/rpds/compare/main...FlickerSoul:rpds:dev
https://github.com/crate-py/rpds/compare/main...FlickerSoul:rpds:clone-feature
Both versions work on the GIL version of CPython and all tests pass. However, they don't work on free-threaded one: Python crashes during PyO3 making the py model. This seems to be a bug in PyO3. I have filed an issue and I guess I can only wait for their responses.
Let me know if I miss anything. :)
Thanks!

Thanks for the work so far!
Two immediate notes:
I notice @edgarrmondragon also sent a draft PR, so it's likely wise to coordinate between the two of you on what's what and which/when they're ready for review.
On the free-threading question: I suspect most Python packages will not explicitly support the free-threading interpreter for the immediate future (as a maintainer I certainly don't plan on it in the short term, only perhaps quite a bit down the line, and certainly not without CI running both interpreters) -- so I'm totally fine sticking to things that work with "Traditional CPython", especially if it gets us more functionality there.
So yeah, I wouldn't intentionally break GIL-less, but if there's something which works on standard CPython we should keep it.
And yeah thanks again for your work so far it will be very helpful I'm sure in getting this over the line!

Thanks for the quick reply! We will discuss in the PR thread and will let you know when it's ready.
Yes, GitHub doesn't have a GIL-less CPython CI for now. I'll keep an eye on it while waiting updates from PyO3 side.
Thanks!

And 3.13.0 is out, no longer a dev version: https://www.python.org/downloads/release/python-3130/


[v3.5] Exclude completed pods from being workload endpoints.

https://github.com/projectcalico/libcalico-go/pull/1040


updatepip.py using deprecaated syntax to get installed packages

If you run 'pip install --upgrade pip', the updatepip.py script stops working because of the get_installed_distributions() method, which is no longer there in newer releases.
This is not really the right way to do anyway, I found doing it this way does work.  Here is the updated script (updatepip.py):
import pip
from subprocess import call
import pkg_resources

for dist in pkg_resources.working_set:
    call("pip install --upgrade " + dist.project_name, shell=True)

updatepip.py.txt

I don't think we use that updatepip.py anymore. I'm happy to delete it entirely

If not used, let's remove it then.  Otherwise it does make it a bit confusing for those exploring the internals. :)

I thought maybe you'd found some place I'd forgotten/missed. If you are just exploring then Yeh I'll get rid. Thanks.


Rename cart to carts

So are we sticking with the name cart or new name carts. It affects my PR.

I will take that as a yes.


Add fixed box constraint (Settings > Overlay)

In live caption tool, allow the option to set the bounding box to a fixed size, and have the font auto-scale. Refer to work done for STV.

Closing without resolution. This issue either affects too few users or there are too many issues with higher impact to be working on. It is possible to re-open or re-visit this issue at a later time. If you feel this issue is important, please give a thumbs-up reaction to the first post. You may track this issue from the pinned meta-tracker at the top of the issues home page.


Merge stable

This merges stable into master. There was a conflict in exe/facter.cc, between configurable facter and resolve order changes. @Magisus and @ahenroid could you please take a look at this and make sure it looks correct to both of you, since you made those changes originally

Waiting for CLA signature by @web-flow
@web-flow - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/
Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

Uhhhhh, I have no idea what triggered CLA bot here. I vote we ignore it?

Dueling bots. Woot! @MikaelSmith can/should we blocklist the web-flow user so the CLA bot doesn't comment on PRs with merge commits (of whatever exactly is happening here)?

Yes, probably should.

This looks right to me. From what I can tell, Andy's changes seem untouched, and all my stuff is in there.

Sorry, slow responder. Yes, the merged facter.cc looks great. Merge it.


It can be difficult to remove the PCB from the 125b enclosure

This issue affects the GuitarPedal125b design.  The TRS 1/8" Midi jacks on the PCB can cause the PCB to be quite difficult to be removed from the 125b enclosure.  This is because the end of the midi jack must protrude part way through the enclosure to allow the plug to make proper contact with the jack on the PCB.

I recently modified the Custom Drill Template to help reduce this issue. The hole for the midi jack is now larger and I changed the placement of the holes for the pots by 1mm to allow for a little more play in moving the PCB.   In my tests this makes the PCB much easier to remove from the enclosure and should resolve this issue.


Remove singleton shutdown hook on RythmEngine

This fixes #296.
Best practise is that it is the responsilibilty of the
instantiator/starting to destruct/shutdown an object. The library
doesn't know when a user is done with the RythmEngine.
Note that it is totally fine that Rythm instance has a shutdown hook
as it is singleton.

An alternative to this pull request is to add a new configuration option to disable the shutdown hook. At the same time, I really don't think that the shutdown belongs in the constructor of a class.

Also, after merging this it might make sense to publish a new version of the package to the maven repository.

Closing this. Fixed in a different way. See #296.