Switch shared parameters to either allocators or factories

Instead of receiving shared parameters, executor functions like then_execute() should receive allocators. This should eliminate the need for the awkward decay_construct() protocol and make it easy to package up constructor arguments by defining the appropriate kind of allocator.

Instead of allocators, let's try factories instead. It's difficult to create an object on the stack with an allocator.

The problem with choosing either one of these interfaces is that some executors will choose to statically create an object on the stack, and others will want to dynamically allocate an object via an allocator.

Using an allocator can sort of support either use case, awkwardly, with the helper make_value():
template<class Alloc>
typename std::allocator_traits<Alloc>::value_type make_value(const Alloc& alloc)
{
  using result_type = typename std::allocator_traits<Alloc>::value_type;

  Alloc alloc_copy = alloc;

  using storage_type = typename std::aligned_storage<sizeof(result_type), std::alignment_of<result_type>::value>::type;

  storage_type storage;

  result_type& construct_me = *reinterpret_cast<result_type*>(&storage);

  // call the constructor
  std::allocator_traits<Alloc>::construct(alloc_copy, &construct_me);

  result_type result(std::move(construct_me));

  // call the destructor
  std::allocator_traits<Alloc>::destroy(alloc_copy, &construct_me);

  return result;
}

We could augment factories to have an associated allocator -- .get_allocator() or something. If the executor needs to dynamically allocate, they can use the associated allocator to do so.
If the factory doesn't have a .get_allocator() function, then the executor can do its own thing.
boost::factory kinda works like this.

Changes to executors:

Change concrete executors to receive parameters named shared_factory instead of shared_init
Call shared_factory like a function to initialize the shared_parameter instead of using decay_construct()
Change executor_traits to receive parameters named shared_factory instead of shared_init
use ignore_factory (or unit_factory) instead of agency::detail::ignore when passing dummy parameters
eliminate decay_construct() function

Changes to bulk_invoke:

instead of calling make_shared_parameter_package_for_executor(), call make_shared_parameter_factory_tuple()


parhip on cluster for parallel graph partitioning

I have installed all the prerequisites: Scons, argtable and openmpi.
Now, I am able to run a simple MPI hello world program which communicates between it's other nodes. However when I run the parhip program as "senapati@ctl:~/KaHIP/deploy$  mpirun -n 2 -host cp-1,cp-2 parhip ../examples/delaunay_n15.graph --k=3 --preconfiguration=ecosocial"
(cp-1 and cp-2 are other nodes in the cluster), the following error occurs:
[cp-1:16051] *** An error occurred in MPI_Allreduce: the reduction operation MPI_SUM is not defined on the MPI_INTEGER datatype
[cp-1:16051] *** reported by process [2107572225,545460846592]
[cp-1:16051] *** on communicator MPI_COMM_WORLD
[cp-1:16051] *** MPI_ERR_OP: invalid reduce operation
[cp-1:16051] *** MPI_ERRORS_ARE_FATAL (processes in this communicator will now abort,
[cp-1:16051] ***    and potentially your MPI job)
[ctl.:15579] 1 more process has sent help message help-mpi-errors.txt / mpi_errors_are_fatal
[ctl.:15579] Set MCA parameter "orte_base_help_aggregate" to 0 to see all help / error messages
Please let me know how this issue can be resolved or why is this occurring.

Hi Monica,
it looks like you openmpi is not working correctly. Have you been able to run any other non-trivial code on that installation?
Best,
Christian

Yes I tried a simple hello program present in the examples folder of open mpi and it worked fine.
Thanks & regards,
Monica
On Sep 16, 2017, at 6:37 AM, Christian Schulz <notifications<EMAIL_ADDRESS>wrote:
Hi Monica,
it looks like you openmpi is not working correctly. Have you been able to run any other non-trivial code on that installation?
Best,
Christian
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/schulzchristian/KaHIP/issues/8#issuecomment-329963229, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AUVUmRgrcDIiqXpJ6_HLVq2L0x9OCUVTks5si7LjgaJpZM4PZOvk.

We now fixed the bug. Closing the issue.


added framer motion to blog cards

Fixed the issue : #336

Hey, @devvsakib just sync the code, pulled the code, and followed the steps mentioned in the conversation, But nothing is changed.

Is there any alternative way?


Crucible not accepting items after water refill BETA2

When the crucible has had it's water use up and is then refilled, adding another item does not work until one of the aspects already in there has been used up by one.

Well I'm dumb, should I get delete the issue or leave it for when someone else makes the same mistake?


Fixes for the latest version of the Android Gradle plugin

Hi,
Thanks for the great library!
I noticed that it stopped working with new projects so I opened a few pull requests (#3, #4, #5, #6) to fix some issues.
Since all the PRs are based on one another, if all the changes are OK, merging just #5 and #6 will merge them all in.

Hi, @mrPjer thanks for contributing.
Didn't have time to support plugin lately. Will look through your PRs and merge them in the next few days.


Handle corrupt caches

Coverage remained the same at 0.0% when pulling 99450b9bf887a9cde712a6a30ef06103aa434e5f on fix_corruption into b11ab02cdfc190a9639f4c33df0f8844a1ce051b on master.

Coverage remained the same at 0.0% when pulling 99450b9bf887a9cde712a6a30ef06103aa434e5f on fix_corruption into b11ab02cdfc190a9639f4c33df0f8844a1ce051b on master.


Type of a generic is not narrowed

🔎 Search Terms
generic narrowed class instance enum
🕗 Version & Regression Information

This is the behavior in every version I tried, and I reviewed the FAQ for entries about narrowed.

⏯ Playground Link
No response
💻 Code
enum Tag {
  A,
  B,
  C,
}

interface InterfaceByTag {
  [Tag.A]: {
    type: string;
  };
  [Tag.B]: {
    type: number;
  };
  [Tag.C]: {
    type: boolean;
  };
}

class C<
  T extends Tag = Tag,
> {
  tag: T;
  type: InterfaceByTag[T]["type"];

  constructor(tag: T, type: InterfaceByTag[T]["type"]) {
    this.tag = tag;
    this.type = type;
  }
}

const tagIsA = new C(Tag.A, "string");
const tagIsAError = new C(Tag.A, 123);

function test(instance: C) {
  switch (instance.tag) {
    case Tag.A: {
      const type = instance.type; // should get string, but get string | number | boolean.
      break;
    }
    case Tag.B: {
      const type = instance.type; // Same as above
      break;
    }
    case Tag.C: {
      const type = instance.type; // Same as above
      break;
    }
  }
}

🙁 Actual behavior
It is not narrowed in the switch statement. In all case, the type of instance.type is string | number | boolean.
🙂 Expected behavior
I think the specific type has been identified In the switch statement. What I expected:
  switch (instance.tag) {
    case Tag.A: {
      const type = instance.type; // string
      break;
    }
    case Tag.B: {
      const type = instance.type; // number
      break;
    }
    case Tag.C: {
      const type = instance.type; // boolean
      break;
    }
  }

Additional information about the issue
No response

This is working as intended. You do not specify the type parameter for C in your test function, so it falls back to the default Tag. That means the type is C<Tag>, which results in the type { tag: Tag, type: string | number | boolean }. This is not a union type that can be narrowed.

This is working as intended. You do not specify the type parameter for C in your test function, so it falls back to the default Tag. That means the type is C<Tag>, which results in the type { tag: Tag, type: string | number | boolean }. This is not a union type that can be narrowed.

I'm sorry that my example is a little wrong and you misunderstood it. I have updated the code. I mean the type in the switch statement, not the return type of test function

you misunderstood it.

I did not. The type of instance is C<Tag>, which results in the type { tag: Tag, type: string | number | boolean }. It's not the type { tag: Tag.A, type: string } | { tag: Tag.A, type: number } | { tag: Tag.C, type: boolean }, so it can't be narrowed the way you expect it to. It's perfectly valid to have the value { tag: Tag.A, type: true } assigned to C<Tag>.

you misunderstood it.

I did not. The type of instance is C<Tag>, which results in the type { tag: Tag, type: string | number | boolean }. It's not the type { tag: Tag.A, type: string } | { tag: Tag.A, type: number } | { tag: Tag.C, type: boolean }, so it can't be narrowed the way you expect it to. It's perfectly valid to have the value { tag: Tag.A, type: true } assigned to C<Tag>.

I see. Thank you for your explanation.
Now I'm using is to narrow it. But as the number of enumerations grows and each is function needs to be defined, the if expression looks unpretty:
function isTagA(instance: C): instance is C<Tag.A> {
  return instance.tag === Tag.A;
}

function test(instance: C) {
  if (isTagA(instance)) {
    const type = instance.type; // string
  } else if(isTagB(instance)) {}
  // more...
}

I tried to refactor it in other ways like using switch, and here's the problem.
Is there any way to associate the type of tag with  the type of type? I have no idea because  the type of C is a class instance and cannot be turned into a union type.

This approach is unsafe, as it does not align with your types. Again, { tag: tag.A, type: true } is a perfectly valid assignment to the type C, but using your isTagA type guard it would wrongly assume the type of type is string, when it's actually boolean.
You seem to wrongly assume that InterfaceByTag[T]["type"] is a type depending on what's assigned to tag, but that's not the case. The type depends on whatever T is, which can be a union type as well - and actually is a union type when your type argument defaults to Tag.
You need to use a union of the types you want:

C<Tag.A> | C<Tag.B> | C<Tag.C>, which results in { tag: Tag.A, type: string } | { tag: Tag.A, type: number } | { tag: Tag.C, type: boolean }
not C<Tag> or C (they're the same), which results in { tag: Tag, type: string | number | boolean }

You could use a helper type and make use of distributive conditional types to get a union based on your tags: type AllTagC<T extends Tag> = T extends Tag ? C<T> : never
Then use it: function test(instance: AllTagC<Tag>) { ... }
It's crucial to understand what structure your types actually look like.


Kepler Documentation

Describe the solution you'd like
Add documentation on:

About Kepler
Describe the modules, bpf, model server, collector, power, attacher etc

We can use one of these as a reference
Volsync
Microshift

@rootfs Please assign the issue to me. For some reason I still can't assign issues to myself.

@rootfs @nikimanoledaki @williamcaban I would appreciate your inputs on which docs would look good for Kepler. I have mentioned two references above. My preference is VolSync. It looks simple, intuitive and easy to navigate.

I guess you mean using the Sphinx theme. That's fine with me. I'm happy with anything that can use markdown-type files for quick iteration and easier collaboration on docs.

Please use this repo for documentation. Ideally we have doc to serve the following purposes:

For developers: an deep dive into the development environment, architecture, methodology, enhancement.
For end users:  Environment requirement, deployment process, dashboard setup and customization.
For integrators: Packaging, API, and extensions.

I am going to follow the this flow
Contents
Overview
Installation
Kubernetes
OpenShift
Usage
Developers
End Users
Integratiors

Hi, checking out this project after Kubecon EU. Im currently using Scaphandre, and scanning for alternatives.
Im looking for documentation on what exactly it is you're monitoring, and how you are calculating that, similar to:
https://github.com/hubblo-org/scaphandre/blob/main/docs_src/explanations/how-scaph-computes-per-process-power-consumption.md. Then I noticed this issue, so I would recommend having a section like that in your docs. As a user this is the first thing Im looking for.

Thanks @geertpingen for your suggestions. We are still in early stages but to give some context, we calculate power consumption of running containers in Kubernetes env based on cpu cycles/instruction/time/cache misses etc. Incorporating the details on how it is calculates is still WIP and will be available at our docs page here

@geertpingen thanks for the pointer, alternative is good for consumers :D
Kepler and Scaphandre both want to give users an answer on how much energy spent by a workload (process, container, or VM).
The difference is that, Scaphandre spends a good deal on breaking down the processes' CPU time to attribute the energy to the workloads. Kepler uses CPU time as well as CPU spends time on doing. At the end of the day, not all CPU instruction consumes the same amount of energy.

@husky-parul  @rootfs Thanks for the quick replies! Perhaps this is a question best asked in another setting (are you active on Slack or a similar platform?), but I was wondering how you obtain the coefficients here:
https://github.com/sustainable-computing-io/kepler/blob/a71dce95c1e76d1866f417c1ec8fbf20b58403fe/pkg/model/model.go#L47 (used at https://github.com/sustainable-computing-io/kepler/blob/79055d74185b9fad19dc974e6f65312f453ae72c/pkg/collector/reader.go#L320)

@geertpingen slack invite sent. The coefficients are hardcoded at the moment. They will be learned via online training by the model server (still under construction). If you are excited about building a model of your own, it is a place for collaboration


Make JsonOutputFormatter use camel case by default

It's not the 1990s, everyone expects their JSON to be camel case :-)
Can we get this to be the default so we don't have to do this: https://gist.github.com/shahiddev/e82fea1c85d5e3faa710
I am curious if anyone actually desires pascal case to even be an option.

/cc @JamesNK

+1 for camelCase by default

Word!

makeJsonCamelCaseAgain

While we're at it; can we propose to set NullValueHandling to Ignore as well? It's just a waste of bytes, IMO.
I assume that a high percentage of JSON returned from an API is consumed by JavaScript, where both null and undefined are falsy, so if (json.someProperty) { /* do something */ } would have the same results.
It should also be no problem if the consumer is using Newtonsoft.Json, where MissingMemberHandling is set to Ignore by default. This would simply set properties to default(T).

@khellang create new issue if you want, let's keep this one focused just on camelCase

Go Go Go just do it 😃

+3.14 :) And more :)

As a compliment to https://github.com/aspnet/Mvc/issues/1765 makes perfect sense. cc/ @danroth27

+1

+1

Take my vote

Thanks all. Make sure you use the upvote feature if you +1d

I see +1's are alive and well even after github reactions.
Camel case makes sense with JavaScript but I think JSON is more than just JavaScript. It is a simple change to enable camel case with Json.NET. I think the default is better left taking what the user has defined and making it clear in the documentation how to config MVC to use camel case everywhere if that is what the user wants.

I see +1's are alive and well even after github reactions.

Mobile version of page does not allow to add reaction.

@JamesNK is this an 80/20 thing? Like @khellang states above

I assume that a high percentage of JSON returned from an API is consumed by JavaScript...
Or as in the case of #1765
Can this be the default, and have the developer do the 20% exception?

If you do want to make this change then I think global naming needs a better feature in Json.NET. There are some problems with CamelCasePropertyNamesContractResolver that I haven't yet got around to fixing: it applies to both property names and dictionary names, it doesn't compose well and it always overrides the name from JsonPropertyAttribute.
A better solution would be adding some sort of INamingStrategy exposed on DefaultContractResolver  that supports finer grained control.

Where do I submit my PR for this? Make it so!

@jamesNK that would be awesome, we default to camelCase in elasticsearch's .NET client but need some hoops to make sure verbatim properties work e.g dictionaries/json properties/client mappings
https://github.com/elastic/elasticsearch-net/blob/master/src/Nest/CommonAbstractions/SerializationBehavior/GenericJsonConverters/VerbatimDictionaryKeysConverter.cs#L15
And
https://github.com/elastic/elasticsearch-net/blob/master/src/Nest/CommonAbstractions/SerializationBehavior/ElasticContractResolver.cs#L180
A more directly supported approach would be awesome!

+1

After much deliberation (and discussions with @JamesNK), we have decided by default to support:

Source images:

Camel
Bookcase

Yay! we're like everyone else now!

@rynowak - tell me the truth. Was the ❤ for the change, or for the amazing MS Paint job?

Ah, one issue is INamingStrategy isn't in 8.0.4-beta1

Yeah, "small" snafu... we would likely only do this feature if we had INamingStrategy because using the pre-existing techniques might be too big of a hammer (e.g. affecting dictionary keys, ignoring [JsonProperty], etc.).

Yay!!
On Fri, May 27, 2016 at 10:34 AM Eilon Lipton<EMAIL_ADDRESS>wrote:

Yeah, "small" snafu... we would likely only do this feature if we had
INamingStrategy because using the pre-existing techniques might be too
big of a hammer (e.g. affecting dictionary keys, ignoring [JsonProperty],
etc.).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Mvc/issues/4283#issuecomment-222207289, or mute
the thread
https://github.com/notifications/unsubscribe/AAInRC2CASZX117sMsR6rtQ9mkrV5aM1ks5qFysJgaJpZM4HwDDt
.

Here is NamingStrategy (I changed it to an abstract base class instead of an interface) mostly finished - https://github.com/JamesNK/Newtonsoft.Json/pull/922
It would be nice to get some other eyes on it if you have time for a code review.

@javiercn is working get us to @JamesNK's new https://www.nuget.org/packages/Newtonsoft.Json/9.0.1-beta1 version.

tell me the truth. Was the :heart: for the change, or for the amazing MS Paint job?

Huh, I wasn't even aware there was an image here. I usually browse github in text mode on a feature phone. This is a dope change, and I'm glad we finally agree with everyone else on the planet on how JSON is usually formatted.

62803d5
aspnet/Entropy@6c53784

I suggest you add a couple of unit tests for behavour of serializing dictionary keys (e.g. IDictionary<string, string>) and properties with JsonProperty("name") to make sure you're happy with how those serialize.

So... if I have existing ASP.NET 4.6 code that reads/writes JSON the default way it's been for many years now, and that JSON is stored as a string, and I want to use .NET Core to read it... will I be able to just read the existing data as-is, or do I have to do a convert-to-camel-case on my data as part of the migration to get it to deserialize into the .NET Core objects? Or is there a PascalCaseNamingStrategy I could use?

@SBArbeit I believe you can clear out the naming strategy and that'll work. Or, it might just work anyway, depending on the exact scenario (not everything is always case-sensitive).

When deserializing Json.NET will first attempt to match a property exactly, e.g. "firstName" in JSON will attempt to be deserialize to a .NET property called firstName, but if it doesn't find that it will do a case insensitive look for properties, and will match a property called FirstName.
You can set NamingStrategy to null.

@Eilon, @JamesNK Thank you for the clarification. That makes it super easy.

@JamesNK  @rynowak
I really don't like this change, this's will effect thousands of developers who will upgrade to dotnet Core, but seems like many people like came case ,I suggest some feature like this:
[cameCase] //should provide by json.net
public class Persion
{
    public int Id { get; set; }
    public string Name { get; set; }
}

I think this will give more controllable .
Json is not javascript And it not only use by javascript, it's a data formate now use in many languages .
And MVC shouldn't  limit the data by default.
Think about this :
// this is a line in project.json
"Microsoft.AspNetCore.Mvc": "1.0.0-rc2-final",

will be
"microsoft.AspNetCore.Mvc": "1.0.0-rc2-final",

Please rethink about this change.

[cameCase] //should provide by json.net

There is that feature.

Json is not javascript And it not only use by javascript, it's a data formate now use in many languages > And MVC shouldn't limit the data by default.

All those other languages are either camel casing or snake casing.
It's just a default and it is one line to turn it off.

Beyond just casing, I find CamelCasePropertyNamesContractResolver behaves better. I've had issues in the past where the default contract resolver would serialize private properties, might be fixed by now though.

@JamesNK  Can We use DefaultContractResolver in a few of specific Method  when the defalut CamelCaseResolver   turn on ?

Have we consulted the Xamarin and enterprise folks on this issue, yet? :wink:
So to summarize:

C#/Code is PascalCase by default.
Serialized version of said code (as data) is camelCase by default.

Please help me see what I am missing here that makes me say that this is inconsistent and fosters (more like festers) context-switching between two the areas of concern.
Maybe I found it here:

It's just a default and it is one line to turn it off.

Thank you for saving my sanity @JamesNK. :smile:  That's encouraging.  The problem is that C# increasingly appears to be the "odd duck" by holding onto PascalCase as its established default naming convention, and it seems that more and more tech -- MSFT tech, even and this is a good example -- is moving away from it.  This is especially so when you view the Azure code samples that are coming out.  It seems like everything BUT C# is camelCase.
But alas, we still use C# and hinge upon it for our solutions, generally speaking.
I am all for consistency, but if you are building your application in C#, you simply lose that consistency by deviating from its defaults.
FWIW, I have always felt this way.  I was not (and am not) a fan of .NET Configuration's (app/web.config) approach, either.
BTW, this is (yet another reason) why Xaml rocks so hard, as it has 100% parity to what it serializes/deserializes. :stuck_out_tongue:  The class definition IS the schema.

HTTP is not tied to a particular runtime stack, it is not a serialization
mechanism, it is a message transfer protocol. We are building solutions for
a HTTP clients to consume not .NET clients. The majority of the JSON APIs
in existence are camelCase. Why should a client have to know or care that
the backend implementation is .NET. The whole purpose of HTTP is to
abstract away the implementation details of the clients and servers.
On Fri, Jun 10, 2016 at 6:43 AM Mike-EEE<EMAIL_ADDRESS>wrote:

Have we consulted the Xamarin and enterprise folks on this issue, yet? 😉
So to summarize:

C#/Code is PascalCase by default.
Serialized version of said code (as data) is camelCase by default.

Please help me see what I am missing here that makes me say that this is
inconsistent and fosters (more like festers) context-switching between two
the areas of concern.
Maybe I found it here:
It's just a default and it is one line to turn it off.
Thank you for saving my sanity @JamesNK https://github.com/JamesNK. 😄
That's encouraging. The problem is that C# increasingly appears to be the
"odd duck" by holding onto PascalCase as its established default naming
convention, and it seems that more and more tech -- MSFT tech, even and
this is a good example -- is moving away from it. This is especially so
when you view the Azure code samples that are coming out. It seems like
everything BUT C# is camelCase.
But alas, we still use C# and hinge upon it for our solutions, generally
speaking.
I am all for consistency, but if you are building your application in C#,
you simply lose that consistency by deviating from its defaults.
FWIW, I have always felt this way. I was not (and am not) a fan of .NET
Configuration's (app/web.config) approach, either.
BTW, this is (yet another reason) why Xaml rocks so hard, as it has 100%
parity to what it serializes/deserializes. 😛 The class definition IS
the schema.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aspnet/Mvc/issues/4283#issuecomment-225185134, or mute
the thread
https://github.com/notifications/unsubscribe/AAInROyPkDp6_xQNSCqoFEKji-enJzO2ks5qKWn3gaJpZM4HwDDt
.

Yet we have chosen to tightly-couple our entire design and infrastructure upon HTTP, which seems a little incongruous.  Why must we know of HTTP within our code?  If there is any abstraction to be done, it would seem that designing a protocol-agnostic system is the place to start.  As great as HTTP is with its natural-lending verb and design elements, I still cringe whenever I see anything that requires it (or its concepts) to be in code.  I have always found it strange that no one questions why we are coupling design with a protocol.
But I digress. I do appreciate your explanation @glennblock.  Thank you for taking the time to do so.  I realize I am in the minority here, and I am not looking to reverse community consensus (ok, maybe a little :smile: ).  The best I can hope for is to point out the solution inconsistency we have introduced here from a .NET perspective.  And although I appreciate the aim of playing along nicely with others going forward, we have done so at a cost towards what has gotten us here to date.

yeah just do it and let the rest of the world waste their time searching for a solution. Great!!


The nexe loading log can be more useful

Here is what I get when I try to load an exe on arm:
$ ./daemonded -set vm.sgame.type 1 +devmap plat23
Using loader args:  nacl_helper_bootstrap nacl_loader --r_debug=0xXXXXXXXXXXXXXXXX --reserved_at_zero=0xXXXXXXXXXXXXXXXX -v -B irt_core-arm.nexe -e -i 100:25 -- sgame-arm.nexe 100
Warn: IPC: Socket closed by remote end

The message is meaningless.
If I run the command by hand, this is what I get:
$ ./nacl_helper_bootstrap nacl_loader --r_debug=0xXXXXXXXXXXXXXXXX --reserved_at_zero=0xXXXXXXXXXXXXXXXX -v -B irt_core-arm.nexe -e -i 100:25 -- sgame-arm.nexe 100
sel_ldr argument list:
nacl_loader --r_debug=0x0000000040002000 --reserved_at_zero=0x0000000040002000 -v -B irt_core-arm.nexe -e -i 100:25 -- sgame-arm.nexe 100
[471888,4149006400:03:28:04.970230] Error while loading "sgame-arm.nexe": CPU model is not supported
[471888,4149006400:03:28:04.971045] NaClPerfCounterInterval(SelMain __start__:SnapshotBlob): 1153 microsecs
[471888,4149006400:03:28:04.971180] NACL: Application output follows
[471888,4149006400:03:28:04.971212] NaClAppStartModule: module not loaded
Dumping vmmap.
In PrintVmmap
Done.

This is far more useful.

I guess it must be some kind of flushing problem. Either the process exits without flushing or we kill it before it can write and flush its output.

Wait a minute, did you actually open the log file? You just show the daemonded output in the OP but you're supposed to look in <homepath>/cgame.nacl_loader.log to see this stuff.

I think we want to put this not in the cgame.nacl_loader.log since I don't think this is enabled by default and print these types of errors to the console because that's where people look for errors?

I think we want to put this not in the cgame.nacl_loader.log since I don't think this is enabled by default and print these types of errors to the console because that's where people look for errors?

No, because that would be too much spam. On my machine it prints 17 lines for a normal startup at the default (lowest other than muted) log level.

It obviously always have to print to console, but maybe in the error case it can. This means that we have to always log to file and maybe read it back or something in the event of an error.

Wait a minute, did you actually open the log file? You just show the daemonded output in the OP but you're supposed to look in <homepath>/cgame.nacl_loader.log to see this stuff.

I didn't knew this file exists, I never used it and I never seen someone using it to troubleshoot IPC: Socket closed by remote end errors. I always helped people reporting IPC: Socket closed by remote end errors by blindly iterating some checklist (like verifying if NX bit is enabled in BIOS) without reading this file at all.
When such error happen there is even no message saying there is more information to be found in that file.
Can we dump this log file content in case of error?

I have asked people to send me the loader log to debug stuff, but the truth is it has rarely been useful. For example with the Slackware bug where the kernel configuration setting blocked the executable from loading at a low address, the process never even started so it couldn't log anything.

Well, even with my current case the log doesn't help me that much, but I'm just super-surprised we have more logs.
At least for me Error while loading "sgame-arm.nexe": CPU model is not supported confirms me it actually tries to load the right file and that it may be failing at detecting something about the CPU.


templates blow up of vault paths have hyphens

because i guess jinja don't like that?
easy solution: swap hyphens with underscores when rendering template?

aomi template foo foo.out vault/path Traceback (most recent call last): File "/usr/local/bin/aomi", line 9, in <module> load_entry_point('aomi==0.3.1', 'console_scripts', 'aomi')() File "/usr/local/lib/python2.7/dist-packages/aomi/cli.py", line 92, in main action_runner(operation, client) File "/usr/local/lib/python2.7/dist-packages/aomi/cli.py", line 79, in action_runner aomi.render.template(client, args[1], args[2], args[3]) File "/usr/local/lib/python2.7/dist-packages/aomi/render.py", line 20, in template output = template_src.render(**obj) File "/usr/local/lib/python2.7/dist-packages/jinja2/environment.py", line 989, in render return self.environment.handle_exception(exc_info, True) File "/usr/local/lib/python2.7/dist-packages/jinja2/environment.py", line 754, in handle_exception reraise(exc_type, exc_value, tb) File "<template>", line 7, in top-level template code TypeError: unsupported operand type
 for -: 'Undefined' and 'Undefined'


preliminary 1.1 rbac-groups doc has inconsistent namespace

Describe the bug
{{
https://preliminary.istio.io/docs/tasks/security/rbac-groups/
Step 1 identifies the namespace as "rbac-listclaim-test-ns" but the command uses "rbac-groups-test-ns"
}}
Expected behavior
{{
Consistent namespace
}}
Steps to reproduce the bug
{{
Follow "Before you begin" steps
}}
Version
{{
istio-1.1.0-snapshot.6
}}
Installation
{{
From "Before you begin"
$ kubectl apply -f install/kubernetes/istio-demo-auth.yaml
}}
Environment
{{
OpenShift Container Platform 3.11
}}
Cluster state
{{ N/A }}

The PR fixes this issue is at: https://github.com/istio/istio.io/pull/3364


[FEAT]: Bulk Link scraper improvement suggestions

How are you running AnythingLLM?
Docker (local)
What happened?
First of all, I love the idea of recurssively scraping a lot of content via a bulk link scraper.
I think it needs to be rethinked a little, I would recommend not giving people the ability to choose the amount of links to scrape but rather have a full recursive scrape of the whole website. The reason is that you already have the ability to choose what links you want to scrape IF you choose to go one on one.
I would argue that no one knows the amount of links they want to scrape (because you don't control what the scraper finds) within a full website so it becomes negligent because when you are scraping you want everything on that page or subdomain or folder.
Explanation:

Many times you have a usecase where the whole website needs to be scraped, you would usually not scrape 10 or 50 pages (and not knowing which pages have been scraped) but rather everything that the "scraper" finds on that url. Make a recurssive scraper for everything! Remove the option to add amount of links
When we are scraping we only get the popup saying "scraping, this might take some time". I would rather make this a brand new menu alternative in the backend called "Scraped websites" where you can see the amount of pages that are scraped, stop the scraping, pause the scraping or "play" the scraping again.
These files does not need to land in the document folder, they should be in a separate folder and EMBEDDED automatically.
Make scraped websites available via the settings menu scraped websites. Put a similar three dots as adding users to a workspace, this way you can now add different scraped websites to different workspaces outside of the document management solution.
In the data connectors bulk links "menu" -> create a link to the admin pages to see the scapings that are happening, that are done and/or paused.

I tried adding 50000 links to scrape and I have no idea now how to shut it down :) haha. It even said scrape failed and then it continues scraping in the background. This is why the above is key.
Are there known steps to reproduce?
No response

I second this!

Just one tip meanwhile how I control or monitor the scraping:
in your docker container you can run docker-compose logs --tail=200 --follow
Then you will see how many links are scraped from total, left and which link is currently scraped. At least a nice temporary solution :).


Uncaught TypeError: Cannot convert undefined or null to object

[Enter steps to reproduce below:]

Not sure how to repro consistently. I just got this exception when clicking on a commit in the time machine.

Atom Version: 1.6.1
System: Mac OS X 10.11.4
Thrown From: git-time-machine package, v1.4.1
Stack Trace
Uncaught TypeError: Cannot convert undefined or null to object
At /Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/marker-layer.js:132

TypeError: Cannot convert undefined or null to object
    at Function.keys (native)
    at MarkerLayer.module.exports.MarkerLayer.findMarkers (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/text-buffer/lib/marker-layer.js:132:21)
    at DisplayBuffer.module.exports.DisplayBuffer.findFoldMarkers (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:1146:36)
    at DisplayBuffer.module.exports.DisplayBuffer.screenPositionForBufferPosition (/Applications/Atom.app/Contents/Resources/app.asar/src/display-buffer.js:799:29)
    at TextEditor.module.exports.TextEditor.screenPositionForBufferPosition (/Applications/Atom.app/Contents/Resources/app.asar/src/text-editor.js:1278:33)
    at DiffViewEditor._createLineMarker (/Users/bsegraves/.atom/packages/git-time-machine/node_modules/split-diff/lib/build-lines.js:76:39)
    at /Users/bsegraves/.atom/packages/git-time-machine/node_modules/split-diff/lib/build-lines.js:65:55
    at Array.map (native)
    at DiffViewEditor.setLineHighlights (/Users/bsegraves/.atom/packages/git-time-machine/node_modules/split-diff/lib/build-lines.js:65:32)
    at Object.module.exports.SplitDiff.displayDiff (/Users/bsegraves/.atom/packages/git-time-machine/node_modules/split-diff/lib/split-diff.coffee:204:22)
    at Object.module.exports.SplitDiff.updateDiff (/Users/bsegraves/.atom/packages/git-time-machine/node_modules/split-diff/lib/split-diff.coffee:117:6)
    at Function.module.exports.GitRevisionView._splitDiff (/Users/bsegraves/.atom/packages/git-time-machine/lib/git-revision-view.coffee:147:15)
    at /Users/bsegraves/.atom/packages/git-time-machine/lib/git-revision-view.coffee:106:8
    at /Applications/Atom.app/Contents/Resources/app.asar/node_modules/underscore-plus/node_modules/underscore/underscore.js:666:47

Commands
     -1:33.8.0 core:move-right (atom-text-editor.editor.is-focused)
     -1:33.4.0 settings-view:open (atom-text-editor.editor.is-focused)
     -1:32.2.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:32.2.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:31.4.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:31.4.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:30.6.0 core:close (atom-text-editor.editor.mini.is-focused)
     -1:28.7.0 core:save (atom-text-editor.editor.is-focused)
     -1:27.6.0 fuzzy-finder:toggle-file-finder (atom-text-editor.editor.is-focused)
     -1:25.9.0 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:25.9.0 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:24.9.0 command-palette:toggle (atom-text-editor.editor.is-focused)
     -1:21.7.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -1:19 editor:consolidate-selections (atom-text-editor.editor.mini.is-focused)
     -1:19 core:cancel (atom-text-editor.editor.mini.is-focused)
     -1:16.1.0 git-time-machine:toggle (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "closeEmptyWindows": false,
    "disabledPackages": [
      "Handlebars",
      "open-last-project",
      "flex-tool-bar",
      "tabs-to-spaces"
    ],
    "ignoredNames": [
      ".git",
      ".hg",
      ".svn",
      ".DS_Store",
      "Thumbs.db",
      ".gitkeep",
      "~",
      ".gitignore",
      ".imdone",
      "tmp",
      "node_modules",
      "._.DS_Store",
      "schema-data.json"
    ],
    "themes": [
      "one-dark-ui",
      "atom-dark-syntax"
    ]
  }
}

Installed Packages
# User
Sublime-Style-Column-Selection, v1.5.1
atom-language-avro, v0.0.13
better-handlebars, v0.1.1
custom-folds, v1.5.2
file-icons, v1.6.20
fold-functions, v0.4.3
git-time-machine, v1.4.1
highlight-selected, v0.11.2
imdone-atom, v1.3.29
jsonpp, v0.1.1
minimap, v4.21.0
minimap-highlight-selected, v4.4.0
pigments, v0.26.0

# Dev
No dev packages

Unable to reproduce but I think this may have been fixed in the split-diff.


updated ARM info

The 7.10.3 compiler is now in the arch repos, and worked without having to hack llvm!  Thought that deserved an update here.  Haven't tested it extensively, just hello world on ghci and compiling a small project with cabal.

Thanks!  It's really good to hear that support is slowly getting better here


feat: add support for BigInt scalar

closes #59
TODO

[x] docs
[x] tests

Codecov Report

Merging #56 (9a9da4d) into main (c8b5f6c) will increase coverage by 0.78%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
+ Coverage   44.47%   45.25%   +0.78%
==========================================
  Files          26       27       +1
  Lines         407      411       +4
  Branches       81       81
==========================================
+ Hits          181      186       +5
+ Misses        206      205       -1
  Partials       20       20

Two things I want to see also happen in our support for this:

Mention which Node version is needed to support this.

watch out for this:

In order to support BigInt in JSON.parse and JSON.stringify, it is recommended to install this npm package together with this scalar. Otherwise, JavaScript will serialize the value as string. json-bigint-patch

To test that we're doing this right I would like us to add an integration test for BigInt and prove that the typeof check for a client response against a BigInt field returns BigInt and not String.

@iddan Heya, any update?

Updated docs regarding JSON patch and node version.

Please rebase, tests are fixed.

Do you know how to update snapshots?

@iddan I merged your other PR, so looks like some more conflicts for you to resolve. And the test snapshot update. Then we should be good to get this merged finally. Thanks!

Looks like a few more snapshot updates are needed.

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

npm package (@next-major dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Installation issues - HTTP error 401

> devtools::install_github("valentinitnelav/plotbiomes")
Using github PAT from envvar GITHUB_TOKEN
Error: Failed to install 'unknown package' from GitHub:
  HTTP error 401.
  Bad credentials

  Rate limit remaining: 56/60
  Rate limit reset at: 2021-10-06 23:56:59 UTC

Anything to be done other than cloning?
Originally posted by @technocrat in https://github.com/valentinitnelav/plotbiomes/issues/7#issuecomment-937326376

Hi @technocrat, I moved this as a separate issue from #7 - "Use GitHub pages for documentation", since that is a different issue.
To add a bit more, I also tried both under Linux (connected with GitHub) and Windows (not connected with my GitHub account) and I cannot reproduce your error.
I admit that I do not really have time to maintain this package, but not sure where the problem is. I had something similar once with issue #6 - "Failed to install - unsupported proxy error message".

Thanks for checking. Sorry to be late getting back to you.
I've now tried this on two other, unrelated networks and have the same
issue. It's not critical for me, since I can clone. Just wanted to let you
know.
On Fri, Oct 8, 2021 at 1:08 AM Valentin Ștefan @.***>
wrote:

Hi @technocrat https://github.com/technocrat, I moved this as a
separate issue from #7
https://github.com/valentinitnelav/plotbiomes/issues/7 - "Use GitHub
pages for documentation", since that is a different issue.
To add a bit more, I also tried both under Linux (connected with GitHub)
and Windows (not connected with my GitHub account) and I cannot reproduce
your error.
I admit that I do not really have time to maintain this package, but not
sure where the problem is. I had something similar once with issue #6
https://github.com/valentinitnelav/plotbiomes/issues/6 - "Failed to
install - unsupported proxy error message".
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/valentinitnelav/plotbiomes/issues/8#issuecomment-938435341,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAASHGEVBA24JCX42RBVB5TUF2RHLANCNFSM5FTAEYBA
.


FabricPlaceholderAPI support

Add possibility to use placeholders in chat formatting
https://github.com/Patbox/FabricPlaceholderAPI

Hi @Niketats,
Thanks for the suggestion! I've had a close look into this, but it appears that it would probably warrant a rewrite of how placeholders are handled.
I've refactored the Styling module to allow for easy support of adding new mod integrations. Right now, I've added LuckPerms placeholder support directly in #14.
Thanks,
Axieum.


A "res" folder is created always no matter the configuration

Hi.
I have just discovered the vscode extension and I have noticed that it always creates a folder called "res" to save the images no matter the configuration parameters.
By default:
res/image.png
If I try to save the image on a folder called images:
images/res/image.png
vcode version: 1.45.0
Ubuntu 18.04.4
Thanks

It seems function createImageDirWithImagePath hardcode res folder for a unknown reason in this line I thinks it would be more suitable give a parameter to add a folder (optional, it could be the actual res) or use the filename as a relative folder.


Scale Transform on Body Tag breaking all D3.mouse coordinates

I working on an app that has a scale (webkit-transform) set on the body tag and it is throwing all the coordinates off provided by d3.mouse(container).  I don't know if this is actually a problem with D3 or the with the browsers representation of clientX/Y or PageX/Y when using transform scale.  In short I can't get accurate coordinates for dragging while this scaling is involved.
This is largely a repost of the issue I originally created at D3 google group.  While this was resolved by using offset and doing the matrix transforms manually, I would like to use the mouse points to more easily manage boundaries within various viewports.
I have also created another test to showcase this behaviour.  The larger you set the container holding the svg the more obvious the discrepancy.
http://jsbin.com/kuhiy/6/ or edit
Chrome: 33.0.1750.154 m
OS: Win 7X64
D3: 3.4.3

Merging into d3/d3-selection#67.


Archive swift-issues-migration repositories?

Can the swift-issues-migration repositories be archived, so that duplicates aren't shown on the https://github.com/issues page by default?

swift-issues-migration org deleted.

One question: is the code of the migration tools still available?


bugfix: unit test failure (#1702)

Ⅰ. Describe what this PR did
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

Codecov Report

Merging #1708 into develop will increase coverage by 0.29%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             develop    #1708      +/-   ##
=============================================
+ Coverage      47.15%   47.44%   +0.29%
- Complexity      1785     1792       +7
=============================================
  Files            364      364
  Lines          13251    13251
  Branches        1642     1642
=============================================
+ Hits            6248     6287      +39
+ Misses          6324     6282      -42
- Partials         679      682       +3

Impacted Files
Coverage Δ
Complexity Δ

...a/server/session/file/FileBasedSessionManager.java
53.57% <0%> (+1.78%)
21% <0%> (+1%)
:arrow_up:

...o/seata/server/coordinator/DefaultCoordinator.java
55.9% <0%> (+5%)
28% <0%> (+1%)
:arrow_up:

...server/store/file/FileTransactionStoreManager.java
52.26% <0%> (+6.27%)
21% <0%> (+2%)
:arrow_up:

...in/java/io/seata/server/session/SessionHolder.java
59.49% <0%> (+6.32%)
16% <0%> (+1%)
:arrow_up:

...io/seata/server/session/DefaultSessionManager.java
95.23% <0%> (+9.52%)
8% <0%> (+1%)
:arrow_up:

.../server/store/AbstractTransactionStoreManager.java
50% <0%> (+25%)
2% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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

duplicate with https://github.com/seata/seata/pull/1492/files#diff-4518292ffa6917a7ea97757977a5014dR40

duplicate with https://github.com/seata/seata/pull/1492/files#diff-4518292ffa6917a7ea97757977a5014dR40

ok ,  Cancel this pr


Update NuGet spec to match the references that build the project

Another one I'm probably wrong about, but shouldn't the nuget specification version for Serilog match the package version the library is built against? I guess there aren't any breaking changes between 1.4.204 and 1.5.7, so there shouldn't be issues there.
Then again, I don't know if bumping the required version then requires bumping the serilog-web-classic package version too?
Ahh, package management.

Thanks for that! Yes, package management indeed :-)


Add way to get both nodes and relationships via helper methods

It's possible with a direct query, but it seems like you could have a method like .rels and .nodes on CypherNode that would return both the relationships and the nodes rather than one or the other.

Good point.  I've normally been working with ActiveNode, but I've been using straight up Query lately and that would be handy


Check return code

check return code of the spawn process

@lucafuji can you please squash your commits?

@bolkedebruin
Just squash all the commits

@mistercrunch
Yep, I agree there are some copy and paste code
However anyway you still need to overwrite the run() function
because when the _execute() exits, even it marks itself as failed,
the run function will still override the job status to be success

Then we should change BaseJob.run to try: and record failure when _execute raises, and make LocalJob._execute raise when it has a return code.

I changed according to comments.
Basically I add a on_failure callback when some exception happens during execution and implemented only for LocalTaskJob otherwise it will mark it self as success in the database

Weird, from the checks I see none are related to my diffs

@lucafuji, aside from the nits, this still looks useful. If you want to still contribute, you could rebase and we could check it against current tests. A unit test would be welcome. If we do not hear in 5 days, I will probably close as abandoned.

@artwr Sure, I will rebase from master and submit another diff soon


open-notifications-button does not work when you use browser back to go to previous notifications result page

Description
The "Open all unread" button from open-notifications-button looses functionality (but is still visible) when going to the previous search result page with the "Back" button of the browser, making it hard to open all issues of a repo in chronological order in new tabs (which is unfortunately my usecase).
Reloading the page after going back to it let's you work around this - then the button works again.
How to replicate the issue

Go to https://github.com/notifications, have many (25+) notifications for a repository
Click to see all notifications from repository, e.g. https://github.com/notifications?query=repo%3Aprisma%2Fprisma
(Maybe, not sure if that plays a role: Also switch to the "Unread" view: https://github.com/notifications?query=repo%3Aprisma%2Fprisma+is%3Aunread)
Click "Next" at the bottom right until you get to the last new notifications
Use the amazing "Open all unread" button from Refined GitHub, from feature open-notifications-button to open all of them (after confirming the prompt if I really want to do that)
Use the browser "Back" button to get to the previous search result page.
Observe how clicking the button does nothing (no output in console either)

Extension version
22.8.23
Browser(s) used
Chrome, Version 104.0.5112.81 (Official Build) (64-bit)

Thanks for the report. It's likely this:

https://github.com/refined-github/refined-github/issues/5871

We'll keep this issue open to know what's broken.

That sounds correct and relevant. The issue creation date also roughly matches to when I became aware of this not working any more as before - because this is definitely new and used to be fine in the past 👍
I also subscribed to the other issue for updates, but thanks for keeping this one open as a concrete example.


Support quantised RSQRT operator in TFLite

The commit tests _convert_unary_elemwise function for the quantized and
non quantized tensor for the RSQRT op.
Other operators will be tested in future (separated) commits.
cc @areusch @leandron @gromero @Mousius for reviews

Ass all the tests are passing here, and the issue is sitting here for a few days, I'll merge it and we can take any required changes as follow-up patches.
For the potential operators that can be implemented in future, benefiting from this PR, we raised https://github.com/apache/tvm/issues/9187.
Thanks @ophirfrish!


GUI left image doesn't update after using eraser tool

To Reproduce
Steps to reproduce the behavior:

Load image
Use eraser tool to erase something
Use arrow keys or scroll bar to attempt to change frames
Right image updates, but masks for left one don't until I use the brush tool again to paint something, or click a second time with the eraser. IDs do appear to update and change position (see included image). No errors are returned at any point.

Screenshots

Desktop (please complete the following information):

Python version: 3.9.6
OS: Windows 10
Cell-ACDC v1.2.4rc20

Hi Jordan, you are right, essentially it updates only when you deactivate the eraser. I accidentally added this when we changed the eraser behaviour to always show the overlaid segmentation masks. Essentially with the eraser active it still thinks it needs to show whatever image you had before activating the eraser. Of course this needs to be reinitialised when changing frame. Thanks for spotting that I will fix it today.

Hi Jordan, I released v1.2.4rc22 which has a fix for this issue. Let me know if it works, thanks!

Wow that was fast! Yes, it seems like it's back to normal now. Thanks Francesco!


Pulse audio error when starting Carla server

I am attempting to start the Carla server using the command ./CarlaUE4.sh  on Ubuntu Linux after using the precompiled installation. I get the following error, however, and then the server hangs.
4.24.3-0+++UE4+Release-4.24 518 0
Disabling core dumps.
xcb_connection_has_error() returned true
Failed to create secure directory (/run/user/40445/pulse): No such file or directory

Upon hitting command c:
FUnixPlatformMisc::RequestExitWithStatus
FUnixPlatformMisc::RequestExit
Exiting abnormally (error code: 130)

Upon Googling this, it seems to be an error relating Pulse audio. I am not sure why I am getting this error message, but I believe it is because I do not have access to the audio card. Are there any workarounds or fixes? Is Pulse even needed to run Carla?

@doterop could you follow up on this please?

Hi @Stephanehk
Are you getting this error only with Carla? It seems something related to the configuration of your machine.


Unable to resolve module react-native/Libraries/Components/ScrollView/InternalScrollViewType

Environment:
OS: macOS High Sierra 10.13.6
Node: 9.11.1
Yarn: 1.7.0
npm: 6.1.0
Watchman: 4.9.0
Xcode: Xcode 9.4.1 Build version 9F2000
Android Studio: 3.1 AI-173.4907809
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: 0.55.4 => 0.55.4
The project compiles ok, but when I execute the app receive this error
Loading dependency graph, done.
error: bundling failed: Error: Unable to resolve module react-native/Libraries/Components/ScrollView/InternalScrollViewType from /Users/javierolivieri/projects/govpredict/govpredict/node_modules/react-native-nested-scroll-view/lib/NestedScrollView.js: Module react-native/Libraries/Components/ScrollView/InternalScrollViewType does not exist in the Haste module map
This might be related to https://github.com/facebook/react-native/issues/4968
To resolve try the following:

Clear watchman watches: watchman watch-del-all.
Delete the node_modules folder: rm -rf node_modules && npm install.
Reset Metro Bundler cache: rm -rf /tmp/metro-bundler-cache-* or npm start -- --reset-cache.  4. Remove haste cache: rm -rf /tmp/haste-map-react-native-packager-*.
at ModuleResolver.resolveDependency (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/node-haste/DependencyGraph/ModuleResolution.js:161:1460)
at ResolutionRequest.resolveDependency (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/node-haste/DependencyGraph/ResolutionRequest.js:91:16)
at DependencyGraph.resolveDependency (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/node-haste/DependencyGraph.js:272:4579)
at dependencies.map.relativePath (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/DeltaBundler/traverseDependencies.js:376:19)
at Array.map ()
at resolveDependencies (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/DeltaBundler/traverseDependencies.js:374:16)
at /Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/DeltaBundler/traverseDependencies.js:212:33
at Generator.next ()
at step (/Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/DeltaBundler/traverseDependencies.js:297:313)
at /Users/javierolivieri/projects/govpredict/govpredict/node_modules/metro/src/DeltaBundler/traverseDependencies.js:297:473

The last version only works with react-native 0.56.0, but you wouldn't be able to compile, so that's why i am not sure, can you try an older version (6.0.1) or upgrade react-native to 0.56.0?

I need to implement in this version: react-native: 0.55.4 , I will try with 6.0.1 ....

@cesardeazevedo  I'm working with the 6.0.1 version, and when compiling i receive this:
"error: resource android:style/TextAppearance.Material.Widget.Button.Borderless.Colored not found.
error: resource android:style/TextAppearance.Material.Widget.Button.Colored not found.
error: failed linking references." please help

Can you compare with this file and apply the differences? such as compile 'com.android.support:design:25.3.1', compile 'com.android.support:appcompat-v7:25.3.1' , targetSdkVersion 25 and compileSdkVersion 25

ok thanks, that solution works but i have other issue. I will write an email to you

i'm having this trouble, but my React Native version is 0.60.5 :(


Test requirements need to be listed in setup.py

Environment

Qiskit Optimization version: 750c2c8
Python version: 3.9
Operating system: FreeBSD 13.1

In the FreeBSD port I need to fill the TEST_DEPENDS clause and it isn't clear what is a complete set of test dependencies.
What is happening?
n/a
How can we reproduce the issue?
n/a
What should happen?
n/a
Any suggestions?
n/a

I'm closing this for the same reason as Qiskit/qiskit-terra#8498 the requirement-dev.txt list includes all the test only requirements. If you need a list of test dependencies for FreeBSD packaging I would just use that.

requirement-dev.txt contains items irrelevant for testing.

setup.py has a dedicated variable tests_require just for this purpose.
If dev_require would have been sufficient, tests_require wouldn't have been defined.

All the test specifiers in setuptools, including tests_require have been deprecated for almost 3 years now and they're not recommend for use because they will be removed in a future release (https://setuptools.pypa.io/en/latest/references/keywords.html). They were only ever used by setuptools if you configure your setup.py to run tests (which we don't as that is quite a limited interface and deprecated).


auto-emit WINDOW_UPDATE frames on unprocessed DATA frames

fixes #1210

Thanks @vmagamedov - this all makes sense. Please see my last commits that should address all your comments.

@Kriechi Looks great! I don't have any other comments.


Revised animal excretion

Description
This is a fairly straightforward change providing a revised metabolic framework. The pre-existing metabolize and excrete methods have been reconfigured into:

metabolize: for the generating of excreta mass (metabolic wastes)
excrete: for the removal of nitrogenous wastes into soil pools (urination and analogs)
respire: for the removal of carbonaceous wastes into the atmosphere (breathing CO2)
defecate: for the removal of digestive wastes into soil pools

As we are not currently tracking nitrogen or water, the excrete trait and method does nothing but provide structure.
Fixes # (issue)
Type of change

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works
[ ] Relevant documentation reviewed and updated

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.96%. Comparing base (c60a201) to head (a81376a).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #460      +/-   ##
===========================================
+ Coverage    94.75%   94.96%   +0.20%
===========================================
  Files           70       70
  Lines         3793     3792       -1
===========================================
+ Hits          3594     3601       +7
+ Misses         199      191       -8

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


Hotfix/dockerwin

use another nanoserver container without security issue --> https://msazure.visualstudio.com/One/_workitems/edit/8291544
fix Roslyn upload to TSA
cut a few unnecessary pipeline checks and slighty improve build times

split pr


Delete custom category

You should display all custom categories on the Telegram UI to the user, and they must pick one which should subsequently be removed.
We leave the option of removing all data associated with the category, or keeping it as a historical record to you.

With the command "/category" the bot is able to manage categories now.
By entering "Delete", we can add a custom category.


Announcement/Feedback: Python@2 on all images will be deprecated starting of 04/06/2020

Hi!
Due to the recent deprecation of Python@2 we are going to deprecate it too on MacOS, Windows, Ubuntu images on 04/06/2020.
We give it 1.5 month to live so that developers could migrate their code.
If you have any concerns or questions please let use know through this issue.
Thanks!

FYI
https://github.blog/changelog/2020-02-27-github-actions-breaking-change-python-2-being-removed-from-all-virtual-environments/

We see a lot of customers' feedback about this announcement and decided to keep Python 2 on our images.
Cancelling this announcement. Thank you for your feedback

fwiw the blog post is still live -- might want to update that: https://github.blog/changelog/2020-02-27-github-actions-breaking-change-python-2-being-removed-from-all-virtual-environments/

@asottile , thank you for notice.
@thejoebourneidentity , could you please update blob post?


[FEATURE] Monitor Token usage in Chains and AIServices

Is your feature request related to a problem? Please describe.
I need to know how many tokens are used overall
Describe the solution you'd like
In extending the Chain interface and AIServices with a getTokenUsage method, token usage could be made available
Describe alternatives you've considered
One could directly use ChatLanguageModel#generate(ChatMessage...) but the necessary wrapper code in effect duplicates the Chain / AiServices
Additional context
I am currently using the following extended version of ConversationalRetrievalChain
public class ConversationalRetrievalChain implements Chain<String, String> {

   private final ChatLanguageModel chatLanguageModel;
    private final ChatMemory chatMemory;
    private final RetrievalAugmentor retrievalAugmentor;
    private TokenUsage               tokenUsage = new TokenUsage();
...
 @Override
    public String execute(String query) {

        UserMessage userMessage = UserMessage.from(query);
        Metadata metadata = Metadata.from(userMessage, chatMemory.id(), chatMemory.messages());
        userMessage = retrievalAugmentor.augment(userMessage, metadata);
        chatMemory.add(userMessage);

        Response<AiMessage> response = chatLanguageModel.generate(chatMemory.messages());

        tokenUsage = tokenUsage.add(response.tokenUsage());

        AiMessage aiMessage = response.content();
        chatMemory.add(aiMessage);
        return aiMessage.text();
    }
....
    public TokenUsage getTokenUsage() {
        return tokenUsage;
    }
}

This solution works for when an AiService returns a string, however how can i monitor the token usage for when an AiService returns a POJO or an enum, or anything else other than a string?
My use case is I'm looking to monitor and track usage amongst users, but I currently use the AiServices. Returning the token usage (optionally) from the AiServices would be ideal.

@jrsperry I have updated the example above as it was outdated (it used Response<AiMessage> instead of Result<T>).
You can access both POJO and TokenUsage when using Result<T> return type:
interface Assistant {

    Result<Pojo> chat(String userMessage);
}

awesome. Just what I was looking for.


Enable long form generation options

Sample generation
Before:

After:

Sample audio:
audio__bark__continued_generation__2023-05-04_16-07-49_long.webm
"""
Hey, have you heard about this new text-to-audio model called "Bark"?  Apparently, it's the most realistic and natural-sounding text-to-audio model out there right now. People are saying it sounds just like a real person speaking.
I think it uses advanced machine learning algorithms to analyze and understand the nuances of human speech, and then replicates those nuances in its own speech output. It's pretty impressive, and I bet it could be used for things like audiobooks or podcasts.
In fact, I heard that some publishers are already starting to use Bark to create audiobooks. It would be like having your own personal voiceover artist.
I really think Bark is going to be a game-changer in the world of text-to-audio technology.
"""

I can't use it, my generated audio is always max 14 seconds long. Any idea why?
I installed the webui today, so it was up to date.
My generation options:
History Prompt (voice) setting: or Use a voice: Use V2 English 9
Split prompt by lines
For each subsequent generation: Reuse history
Text temperature: 0,7
Waveform temperature: 0,7
Prompt: Hi - this is a text speech - I wonder how my voice will sound like [laughs], I hope it won't be strange... well, let's hear it then! In this test I'm also using Text temperature at 0,7 and Waveform temperature at 0,7. "Do I look angry now!?" she yelled angrily. (Some parts are always truncated so it isn't short).
And no errors in the command window.

Ok, I think the problem is - splitting prompt by lines. In the example you gave me:
Hi - this is a text speech - I wonder how my voice will sound like [laughs], I hope it won't be strange... well, let's hear it then! In this test I'm also using Text temperature at 0,7 and Waveform temperature at 0,7. "Do I look angry now!?" she yelled angrily.
There's no newline. Please try this prompt (with a new line) instead:

Hi - this is a text speech - I wonder how my voice will sound like [laughs], I hope it won't be strange... well, let's hear it then!
In this test I'm also using Text temperature at 0,7 and Waveform temperature at 0,7. "Do I look angry now!?" she yelled angrily.

The reason it doesn't do it automatically is because speaker's have different speeds of talking, etc, so it's hard to predict which amount of text will produce the best result.
The way to do it now is - a) generate and see where it stops, split there b) split by the closest sentence c) give approximate amount of characters.
In the future there will be better solutions to this, but not quite yet.
Also, in the newest UI you will see the generation in fragments at first, such as:
"""
Generated audio fragment... Hi - this is a text speech - I wonder how my voice will sound like [laughs], I hope it won't be strange... well, let's hear it then!
"""
This means that it still has not finished the entire prompt, and you should see the progress continue in the console.
I tested and the entire prompt seems to be around 25s.

Please let me know your results.

Thanks! Splitting the sentence worked.


fix fix cooldown datetime

Proposed changes
Issue Number: close #xxx

#16553 cooldown_ttl and cooldown_datetime cann't set at the same time.  If  cooldown_datetime is set, cooldown_ttl will be set to -1.
Tablet::_get_newest_cooldown_time  will always get storage_policy->cooldown_datetime , then all new rowsets will be  incorrectly cooldowned.

if cooldown_ttl is -1, then cooldown_datetime is used, original code is right. Could you show an bad case?


RDK-31068: HDMI Output HDCP Status Change Notification XiOne

Reason for change:
HDMI Output HDCP Status Change Notification XiOne
Thunder changes.
 Please submit this to the latest sprint branch if needed.


S15-nfg update tests made from test-gen.p6 for Unicode 9.0

[ ] mass-roundtrip-nfc.t
[ ] mass-roundtrip-nfd.t
[ ] mass-roundtrip-nfkc.t
[ ] mass-roundtrip-nfkd.t
[x] mass-equality.t

@jnthn I think you wrote the test-gen.p6 script.
When I try and generate with roast/S15-nfg/test-gen.p6 only one of the tests (mass-chars.t) actually seem to generate more than 2 passing tests, the rest all fail with either UCD 8.0 or UCD 9.0.
Maybe you can help?

This is done as of https://github.com/perl6/roast/commit/5844a89798414ab7c5dc6791b4257fa14a1fa56d
https://github.com/perl6/roast/commit/f8070c412275d13fe138b674f713759aa3bf84ac


Avoid PRIMARY notifications when disabled

This can cut down on unnecessary notifications when told not to handle PRIMARY. Contingent upon this.

Pending changes on https://github.com/cdown/clipnotify/pull/2 for now

clipmenu has moved on a lot since this was opened, please reopen based on current devel if still desired.


Fix #172: Prevent duplicate pagination events under certain cases.

I fixed this issue with minimal risk of regression bugs in the rendering logic (I am slightly concerned about the view changed + setViewSettings() proposed alterations), by debouncing the event at the tail end:
https://github.com/readium/readium-shared-js/issues/172#issuecomment-140357337
This has solved critical Media Overlays playback issues (TOC linking).
It actually makes some kind of sense from a design perspective, as the optional setViewSettings() call (only invoked when the view type has not changed) creates an intermediary event prior to opening the spine item (which generates the final PAGINATION_CHANGED event including the actual CFI target). When setViewSettings() is invoked outside of an initViewItem call, there is only a single call so the debounce introduces a small unnecessary delay.
Now, in the long term, I would prefer to review @JCCR's deeper design changes. I noticed this commit in the intel-contrib feature branch:
https://github.com/readium/readium-shared-js/commit/f132bf7b00517541b4e0e7429917750d937ab029
"Prevent duplicate pagination events under certain cases. A cleaned up version of PR #174"
This commit seems to cover other fixes, so I cannot comment on implementation details.
Once the various ongoing PRs (some of which include merges of several branches) are completed / ready for review, we can revisit the problem.

Can we merge this in then @danielweck ?

@JCCR Well, the pagination event is not duplicated in the develop branch anymore, so I am not sure what problem this PR would address. If anything, I would like to understand the design changes proposed here (there seems to be a few non-trivial code modifications). At the next opportunity, let's get together on a voice chat to review the underlying mechanics of the initViewForItem + setViewSettings code paths.

@danielweck
Sorry, commented on the wrong PR. Yea, I do remember about this issue addressed in your other commits.

ACTION: Close this pull request since it is now past-due and in conflict with changes made in the develop branch.


Add rmevans9 to Github Issue Task Force

Simply adding myself to the Github Issue Task Force so I can help out closing, tagging or whatever is needed for the issues.
Thanks!

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at<EMAIL_ADDRESS>Thanks!

By analyzing the blame information on this pull request, we identified @mkonicek and @melihmucuk to be potential reviewers.

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

Thanks for helping :)
Remember, with great power, comes great responsibility!!!
@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.

@satya164 I will use this power wisely. Thanks!


add support for FreeBSD

This adds support for FreeBSD and handles the following differences:

no 'fpm' package available (included in main PHP package)
no 'devel' package available (also included in main PHP package)
group 'root' is named 'wheel' on FreeBSD
additional software is installed under '/usr/local'
replace FreeBSD's system-wide extensions.ini with per-module configuration files

There are a few limitations though:

no support for manage_repos on FreeBSD (not useful on this plattform)
support for 'composer' and 'phpunit' is untested

Tested with the following hiera configuration:
php::ensure: 'installed'
php::manage_repos: false
php::fpm: true
php::dev: true
php::composer: false
php::pear: true
php::phpunit: false
php::fpm::config:log_level: 'notice'
php::composer::auto_update: true
php::settings:
  Date/date.timezone: 'Europe/Berlin'
php::cli::settings:
  PHP/memory_limit: '256M'
php::fpm::settings:
  PHP/short_open_tag: 'On'
php::fpm::service::ensure: 'stopped'
php::fpm::service::enable: false
php::package_prefix: 'php55-'
php::extensions:
  bcmath: {}
  xml: {}
  mysql: {}

Thanks! Looks good to me. :beers:
One question, though: You have set php::fpm::service::enable to false, but does managing the fpm service work? It might be a good idea to warn users if they want to use features that are not available on their platform.

@fpletz Thanks for your comments! I've updated my original submission:

additions to run php-fpm service on FreeBSD
replace FreeBSD's system-wide extensions.ini with per-module configuration files
update README.md to mention known FreeBSD-specific limitations

Besides that I've tested php-fpm on FreeBSD and hence made some additions to php-fpm.conf.erb. I consider php-fpm on FreeBSD to be fully working now.

Cheers @fraenki!
Looks good to me, @fpletz feel free to merge.

I'd like to test it in my FreeBSD environment these days and merge then, if that's OK...

@mmoll ack

I think we're having a slight ordering issue in this module that went maybe undetected on Linux:
root@bsdphp:~ # puppet apply --modulepath=/root/puppet/modules /root/puppet/lala.pp
Warning: Config file /usr/local/etc/puppet/hiera.yaml not found, using Hiera defaults
Notice: Compiled catalog for bsdphp.local in environment production in 5.76 seconds
Notice: /Stage[main]/Php/File[/usr/local/etc/php/extensions.ini]/ensure: removed
Notice: /Stage[main]/Php::Packages/Package[php56-extensions]/ensure: created
Notice: /Stage[main]/Php::Pear/Package[pear]/ensure: created
Notice: Finished catalog run in 7.76 seconds
root@bsdphp:~ # puppet apply --modulepath=/root/puppet/modules /root/puppet/lala.pp
Warning: Config file /usr/local/etc/puppet/hiera.yaml not found, using Hiera defaults
Notice: Compiled catalog for bsdphp.local in environment production in 5.67 seconds
Notice: /Stage[main]/Php/File[/usr/local/etc/php/extensions.ini]/ensure: removed
Notice: Finished catalog run in 1.94 seconds
root@bsdphp:~ # puppet apply --modulepath=/root/puppet/modules /root/puppet/lala.pp
Warning: Config file /usr/local/etc/puppet/hiera.yaml not found, using Hiera defaults
Notice: Compiled catalog for bsdphp.local in environment production in 5.67 seconds
Notice: Finished catalog run in 1.90 seconds

Please also add FreeBSD as supported OS in metadata.json

I've updated my original submission to include @mmoll's suggestions:

bump default php version to 5.6 on FreeBSD (to conform with FreeBSD's new default version)
fix ordering issue regarding 'extensions.ini' removal
add FreeBSD as supported OS in 'metadata.json'
rebased

Seems to work just fine here, thanks @fraenki! :ice_cream:

Awesome :tada:


Laravel 10.31.0 Shift

This pull request includes updates for the recent minor version release of Laravel as well as updates to your package dependencies. Feel free to commit any additional changes to the shift-ci-v10.31.0 branch.
Before merging, you need to:

Checkout the shift-ci-v10.31.0 branch
Review all pull request comments for additional changes
Run composer update
Thoroughly test your application (no tests?, no CI?)

:warning: The Shift build servers run PHP 8.1. If you are running a different version of PHP you should run composer update locally to ensure you have the proper versions of your dependencies to reflect your PHP environment.


Fix minor bug GRPC_TIMER_END before end of function

GRPC_TIMER_END should be called at the end of function instead of GRPC_TIMER_BEGIN

I signed it!

Thanks @zayoo! I'll merge once Travis goes green (which it should, this change is a no-op for Travis).

I don't know why it caused a test case to fail. I am looking at the test case.


peek~ is writing too slow

in comparison to tabwrite~ using [uzi] and a slider input (i have a patch)
peek is slow in drawing.pd.zip

Are the values slow in displaying, or is it actually not writing as fast?

yeah, it's just taking too long to display... it writes pretty fast

that's really evident with the slider + uzi in the example I uploaded

fixed with PEEK_TICK_TIME


Update popup not showing on checkForUpdate (iOS)

Hi!
I'm having some troubles displaying the update popup when running the checkForUpdate method. The code is straight from the guide and sessions as well as crash reports are registered in the HockeyApp dashboard. No errors are shown in the logs.
  componentWillMount () {
    HockeyApp.configure('APP_ID', true)
    console.log('WILL MOUNT', HockeyApp)
  }

  componentDidMount () {
    HockeyApp.start()
    HockeyApp.checkForUpdate()
  }

Steps:

Upload version 1.0.0 to HockeyApp
Install it on the iPhone
Upload version 1.0.1 to HockeyApp
Restart the app on the iPhone and wait for the popup (which never shows)

All help is much appreciated!

It started working, so I'm closing the issue.

@johanarnor  how it works for you?

I am having the same issue, anyone experienced this and got a solution? The android version does show the update dialogue.
Also anyone got the number of installs showing HockeyApp? Do we need to do something specific for it to work?

@johanarnor @Jaafar-abusair Any solutions? Initially I thought it had to do with the permission to show alert window. But using the react-native Alert.alert works fine on iOS.

I finally got it to work. Before using react-native-hockeyapp I used the official HockeyApp installation instructions for iOS.
Solution:

I had to revert the steps I had taken for integrating the official SDK.
I made the changes to AppDelegate.m (although this seems optional to me?)

If you wish to use Device UUID authentication or Web authentication, the following must be added to ios/AppDelegate.m
- (BOOL)application:(UIApplication *)application openURL:(NSURL *)url sourceApplication:(NSString *)sourceApplication annotation:(id)annotation {
  if( [[BITHockeyManager sharedHockeyManager].authenticator handleOpenURL:url
                                                        sourceApplication:sourceApplication
                                                               annotation:annotation]) {
    return YES;
  }

  /* Your own custom URL handlers */

  return NO;
}


Pulling from remote failed

I'm not really well versed handling things like this. I tried following the instructions under "Working with SSH repositories" but it doesn't seem to do anything. Any idea how I could solve this? Cheers.

Brackets 1.4.0-16380 (win), Git 2.5.0.windows.1 & Brackets-Git 0.14.32
Pulling from remote failed
Error: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

call: git fetch --progress origin
Error: Permission denied (publickey).
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
    at Object.exports.toError (file:///C:/Users/Aldi/AppData/Roaming/Brackets/extensions/user/zaggino.brackets-git/src/ErrorHandler.js:162:19)
    at NodeConnection.<anonymous> (file:///C:/Users/Aldi/AppData/Roaming/Brackets/extensions/user/zaggino.brackets-git/src/Cli.js:185:44)
    at j (file:///C:/Program%20Files%20(x86)/Brackets/www/thirdparty/thirdparty.min.js:559:26911)
    at Object.k.fireWith (file:///C:/Program%20Files%20(x86)/Brackets/www/thirdparty/thirdparty.min.js:559:27724)
    at NodeConnection._receive (/utils/NodeConnection.js:498:34)

No idea, I don't work with SSH repositories.

I get the same issue on a Mac, after setting up and configuring my SSH --> Github connection manually, only before seeing the error, I am prompted to enter the SSH key passphrase. (I can push to Github just fine manually from the terminal without entering a passphrase.)

Try turning on the debug mode in brackets-git settings and see what're the commands executed in brackets console, that might give you an idea what might be wrong.

Thanks! I made some SSH key changes and restarted Brackets. It's working now (so I'll never know what debug errors popped up). Thanks for this great tool!

@alignedleft what do you mean by "I made some SSH key changes"? Same issue here on Windows. That would be great to know more about your magic trick! :)

Ah, there was no magic. I think I generated new SSH keys and/or made sure Github was aware of my new key, then restarted. I've forgotten the details, sorry! It seemed I needed to reboot before Brackets noticed the change.

@zaggino do you have any advice to solve that problem? Doing it by opening the Git Bash terminal using the button from the extension works, but as soon as we push with the Git Push from the extension it doesn't work. Can you confirm that the extension uses the Windows command prompt to execute Git?

Thanks for your feedback @alignedleft ;)

Yes, the extension uses default command prompt on the given OS. I don't use SSH for Git so can't help here. You can turn on the debug mode in settings to get in and outs from command prompt in the brackets console.

I've only been able to get Brackets-git to work with SSH repositories by using my default id_dsa and id_rsa identities without passphrases. If I try to configure a unique identity for a server even though I have the identity added to my login session (ssh-add) and it works to git push, pull, etc from the terminal, it fails with the error above when using the buttons in Brackets-git.
My guess is that Bracket-git uses its own session that doesn't have access to the current ssh-agent.

@manthano @zaggino I can confirm, that it is not working when you protect your ssh key with a password (what you should btw). So using the terminal everything works fine. Using HTTPs everything works, but using ssh key which is password protected does not work but is the recommended and most secure method.


So does it evenly cycle?

I need to try to have as close as possible the same amount of viewers to each content so that I can properly chart the data. If I have 100 visitors, it would be ideal if 50 saw Content A and 50 saw Content B. Does your solution do this out of the box?
I tried messing with the sample, yet even after deleting the cookie and refreshing the page, I still kept getting 'blue'.

It sounds to me like the client isn't correctly hitting the server and always returning the control option. Can you confirm that the server is reachable from the client?

What server? I'm I ignorant of further requirements that are not in the readme?

Mark

On Jul 23, 2016, at 8:22 AM, Zack Kitzmiller<EMAIL_ADDRESS>wrote:
It sounds to me like the client isn't correctly hitting the server and always returning the control option. Can you confirm that the server is reachable from the client?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Oops, I am. Please disregard and delete or close this issue. lol

Mark

On Jul 23, 2016, at 8:22 AM, Zack Kitzmiller<EMAIL_ADDRESS>wrote:
It sounds to me like the client isn't correctly hitting the server and always returning the control option. Can you confirm that the server is reachable from the client?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

From the Readme:
PHP client library for SeatGeak's Sixpack ab testing framework.


feat: create empty Table without schema

Summary of Changes
We no longer raise an exception when a user creates an empty Table without specifying a schema. Table([]) is now allows.
add_row and add_rows must still be adjusted (#127) to allow adding rows to such an empty Table.

🦙 MegaLinter status: ❌ ERROR

Descriptor
Linter
Files
Fixed
Errors
Elapsed time

✅ PYTHON
black
2
1
0
0.72s

❌ PYTHON
flake8
2

1
0.43s

✅ PYTHON
isort
2
1
0
0.23s

❌ PYTHON
mypy
2

1
1.71s

❌ PYTHON
pylint
2

1
2.79s

✅ REPOSITORY
git_diff
yes

no
0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff
MegaLinter is graciously provided by

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

v0.8.0
GitHub release

Your semantic-release bot :package::rocket:


Fixing dasboard admin access

After a cron job run the public_wt_url is being overwritten.  This prevents the ability for dashboard admins to log into the extension to view the history.  Updating to latest version of auth0-express-extension-tools to handle isolated domains enabling login.

MD5: 693e9af84d3dfcc71e640e005bdc5e2e
NSP - Security Report
This is an automated security audit of this project. Please do not modify its contents
Vulnerabilities found: 3
Affected Modules: uglify-js, uglify-js, useragent
Summary:

Regular Expression Denial of Service
Incorrect Handling of Non-Boolean Comparisons During Minification
ReDoS via long UserAgent header

Detailed report
Regular Expression Denial of Service
Affected Module: uglify-js
Installed version: 2.2.5
Patched versions: >=2.6.0
Advisory: https://nodesecurity.io/advisories/48
Path<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>
Incorrect Handling of Non-Boolean Comparisons During Minification
Affected Module: uglify-js
Installed version: 2.2.5
Patched versions: >= 2.4.24
Advisory: https://nodesecurity.io/advisories/39
Path<EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>><EMAIL_ADDRESS>
ReDoS via long UserAgent header
Affected Module: useragent
Installed version: 2.1.6
Patched versions: >=2.1.13
Advisory: https://nodesecurity.io/advisories/312
Path<EMAIL_ADDRESS>><EMAIL_ADDRESS>


CPCS-1238 [Itinerary] Update logo images

Instead of referring to local copies, it points to our hosted images now, which should hopefully cut down on any future changes we'll have to make.
I've left the "theme_color" property alone in the manifest, as it's currently white, unlike the client which was purple and needed changing over.

@dnimmo @clicktravel-robin Do either of you have merge rights for this, please? Taj had a look at the PR, but couldn't actually merge the thing. 😀


connect-tcp-socket: add ConnectEx via WSAID_CONNECTEX

sorry, non-public sample

This is an example where having a guid feature could be useful.  (https://github.com/fireeye/capa-rules/pull/183#issuecomment-730618016)
I'll search my local repo to see if I have a sharable sample.  I should have enough information from the screenshot to dig.


fix(docs): fixed small typo in readme.

/cc @leogr

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

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


bug: Docstring Style Local Override Does Not Work

Description of the bug
Local override docstring_style for global configuration doesn't work.
To Reproduce
Declare in mkdocs.yaml:
plugins:
  - mkdocstrings:
      enabled: true
      default_handler: python
      handlers:
        python:
          options:
            docstring_style: google

-->
Inside file.md
::: path.to.module
    options:
      docstring_style: sphinx

Full traceback

Full traceback
N/A

Expected behavior

Override the default google style with the sphinx style for that one module.
Environment information

System: Windows-10-10.0.19045-SP0
Python: cpython 3.10.11
Environment variables:
Installed packages:

mkdocs v1.6.0
mkdocstrings v0.26.0
mkdocstrings-python v1.10.5
griffe v0.47.0

Additional context
I am using path:  to load the module and I'm using mkdocs-monorepo-plugin.

Thanks for the report @triley13 :slightly_smiling_face:
I suppose you expect all objects within path.to.module to be parsed as Sphinx-style, right? Currently we only re-assign the style to the specific object but not its members.
Ideally we would distinguish that a local option was used, and instead of re-assigning a style, we would use the provided local style to parse docstrings. The issue is that we're not able to differentiate a global option from a local one once we're in the render method of the handler :thinking:

We can re-assign the style to the object and all its members, but note that Griffe extensions that modify the cached and parsed version of the docstring will probably stop working (because such extensions would be initially parsing and caching the docstring with the global style, not the local one).
WDYT?

Hey @pawamoy, sorry I've been on vacation for a bit. Is caching necessary? If it's not, then I would suggest making an implementation that only works without caching. Otherwise, is there a workaround for parsing the docstrings for a single page, and limiting the style change to a particular webpage?

Hey again @triley13, thanks for your answer. I'm very inclined to do what you say and stop caching parsed versions of docstrings. Extensions should then never modify the parsed versions but only the "raw" docstring value. This is probably much less convenient though, programmatically speaking, since that means extensions have to manipulate a string instead of a proper data structure, but I'll think about it. Un-caching this is a no-brainer, it creates too many issues.

I have opened a relevant issue in Griffe, see https://github.com/mkdocstrings/griffe/issues/340 :slightly_smiling_face:


Adding an option to override SendGrid Client path

Fixes
Checklist

[x] I acknowledge that all my contributions will be made under the project's license
[x] I have made a material change to the repo (functionality, testing, spelling, grammar)
[x] I have read the [Contribution Guide] and my PR follows them.
[x] I updated my branch with the development branch.
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation about the functionality in the appropriate .md file
[x] I have added in line documentation to the code I modified

Short description of what this PR does:

At my job, we have a facade API that sits in between consumers and SendGrid; it does some various things that are important to our business. We overrode the host which worked fine, but it also added /v3 to the end, so that our URLs ended up being _http://company-email-api.domain.com/api/v1/email/v3_.
I thought this was a simple update to give users more flexibility and should be fully backwards compatible.
I also updated a couple pieces of the Dockerfile; The URL for prism bash script was a 404, so I updated to the current existing install script. I also added support for dos2unix, which coverts line endings. My host machine is windows, and I had lots of issues getting the docker container to run for tests because of line endings.
I didn't apt-get remove dos2linux because I still needed to use it for the unit tests, manually.
I am new to PHP, Docker, PHPUnit, so if I've done something wrong, please let me know

If you have questions, please send an email to Twilio Sendgrid, or file a Github Issue in this repository.

Thank you @erbaker!
I've added this to our backlog for a code review.


Adding a filter for SkipStatusCodePages and adding a sample

...pages
@yishaigalatzer

Needs unit tests for the attribute.

:watch:

@pranavkm fixed and updated.

:shipit: