How to force-include a char?

I need to force-include characters and keywords (e.g. +, -, ...) in my language. Do I just do @:'-' to do that, or is there a better way? (I could define each character as it's own rule, and probably will do so, but is there a way I don't know about?)

I'm not sure what you mean. Can you post a quick example of 1-2 strings and how you'd like it to be parsed?

@badicsalex Okay, so, as background, I'm writing an Ada parser.
This is my first time working with peginator, so I don't know how the AST would look, but as an example, from my understanding, terminals are normally discarded in productions (which makes sense: you (usually) don't need them). However, take the following Ada code (from the Arithmetic-Geometric Mean on RosettaCode):
with Ada.Text_IO, Ada.Numerics.Generic_Elementary_Functions;

procedure Arith_Geom_Mean is

   type Num is digits 18; -- the largest value gnat/gcc allows
   package N_IO is new Ada.Text_IO.Float_IO(Num);
   package Math is new Ada.Numerics.Generic_Elementary_Functions(Num);

   function AGM(A, G: Num) return Num is
      Old_G: Num;
      New_G: Num := G;
      New_A: Num := A;
   begin
      loop
         Old_G := New_G;
         New_G := Math.Sqrt(New_A*New_G);
         New_A := (Old_G + New_A) * 0.5;
         exit when (New_A - New_G) <= Num'Epsilon;
         -- Num'Epsilon denotes the relative error when performing arithmetic over Num
      end loop;
      return New_G;
   end AGM;

begin
   N_IO.Put(AGM(1.0, 1.0/Math.Sqrt(2.0)), Fore => 1, Aft => 17, Exp => 0);
end Arith_Geom_Mean;

In lines 16-18 and 25, we have operators that I need to keep (*, +, <=, ...). As I noted in my original comment, I could define each simple and compound operator like this:
@char
LessThan = '<';
// ... Others...

But I'm wondering if just doing @:'<=', for example, would work instead.

You can just
Relational = LessOrEqual | GreaterOrEqual | NotEqual | LessThan | GreaterThan | Equal;
LessOrEqual = "<=";
GreaterOrEqual = ">=";
NotEqual = "/=";
LessThan = "<";
GreaterThan = ">";
Equal = "=";

This will give you a nice little enum with empty members that you can use in later steps.
No need for @char, that's just a microoptimization anyway.
Just be sure careful with the order in your operator unions. An incorrect order:
Relational = LessThan |GreaterThan | LessOrEqual | GreaterOrEqual | NotEqual;
...

This would parse a <= b as the a token, then just <, and depending on the rest of the rules, either fail, or parse = and then b.

If the question was "can I put arbitrary expressions, or at least string literals after :, or at least after @:", then unfortunately the answer is no.
I was thinking about creating an anonymous rule feature (the expression part after : would create and immediately use an anonymous rule) to do this, but it was never actually needed; it always just made the grammar harder to understand.

@badicsalex Thanks for the help; I hope I do the parsing for expressions and such right (I did define all the keywords like And = i'and', so I'm good there). I have one more question - how does the parser handle diverging alternatives? (I'm unsure what else to call it.) E.g.:
@export
IteratedElementAssociation = For loop_spec:LoopParameterSpecification [Use key_expr:Expression] "=>" expr:Expression
| For iterator_spec:IteratorSpecification [Use key_expr:Expression] "=>" expr:Expression;

This also happens in expressions:
@export
Expression = relations:Relation {and:And relations:Relation} | Relations {and:And then:Then relations:Relation}
| relations:Relation {or:Or relations:Relation} | relations:Relation {or:Or else:Else relations:Relation}
| relations:Relation {xor:Xor relations:Relation};

Problem of course is that I think this is really messy. I might collapse the operators (and, and then, ...) into parenthesized blocks to make the tree easier to handle, but I was wondering how the parser handled things like this (the Ada grammar has things like this all over the place).

Alternatives are called "ordered choice" in PEGs, i.e. the first one that successfully parses is chosen, and parsing of the alternatives stop there.
I'd rewrite your first rule like this:
@export
IteratedElementAssociation = For (spec:LoopParameterSpecification | spec:IteratorSpecification) [Use key_expr:Expression] "=>" expr:Expression;

This way the spec field will be an enum that contains either a LoopParameterSpecification or an IteratorSpecification, and you can call match on it in your later logic.
And this is how I would do expressions:
@export
Expression = e:RelationExpression { op:LogicOperator e:RelationExpression};
RelationExpression = e:AddSubExpression { op:RelationOperator e:AddSubExpression};
AddSubExpression = e:UnaryExpression { op:AddSubOperator e:UnaryExpression};
...

LogicOperator = And | Or | Xor;
And = i'and';
Or = i'or';
Xor = i'or';
...

This would take care of operator precedence. See the calculator example, and also search for "operator precedence in PEG grammars"

@badicsalex Operator precidence is handled directly in the grammar of Ada; I don't think that's necessarily something that I need to worry about since the grammar encodes it directly. I suppose I could rewrite it to be something like:
@export
Expression = relations:Relation {op:(And | (And Then) | Or | (Or Else) | Xor) relations:Relation};

Not sure how this would be represented though. The reason I asked about ordered choice was because Ada does things like what I provided previously a lot, even when the items are entirely different, e.g.:
@export
FullTypeDeclaration = (Type ident:DefiningIdentifier [discriminant:KnownDiscriminantPart] Is definition:TypeDefinition
[aspects:AspectSpecification] ";")
| task_type_declaration:TaskTypeDeclaration
| protected_type_declaration:ProtectedTypeDeclaration;

Or:
@export
IterationScheme = While Condition
| For LoopParameterSpecification
| For IteratorSpecification
| [Parallel [AspectSpecification]]
For ProceduralIterator
| Parallel ["(" ChunkSpecification ")"] [AspectSpecification]
For LoopParameterSpecification
| Parallel ["(" ChunkSpecification ")"] [AspectSpecification]
For IteratorSpecification;

I assume the general strategy for this would be something like (using the first example):
@export
FullTypeDeclaration = normal_type_declaration:(Type ident:DefiningIdentifier [discriminant:KnownDiscriminantPart] Is definition:TypeDefinition
[aspects:AspectSpecification] ";")
| task_type_declaration:TaskTypeDeclaration
| protected_type_declaration:ProtectedTypeDeclaration;

Operator precidence is handled directly in the grammar of Ada
Ah, I didn't know that! I see there's a published context-free grammar. Be careful, | is very different in context-free and PEG grammars.

So unfortunately while field:("whatever" IDunno) would be intuitive, it's not supported. But you can always rewrite it by putting the part in the brackets in a separate rule: field:Whatever + Whatever = "whatever" IDunno.
One special thing you can do is if you'd have a rule like this:
MultiThing = "hmmm" thing:(Something | SomethingElse) "alrite";

You can instead write:
MultiThing = "hmmm" (thing:Something | thing:SomethingElse)  "alrite";

peginator will notice that you used thing in two different alternatives with two different types, and create an enum type just for the thing field, and will set it accordingly.
Also, if you write
MultiConti = "multi:" (num:Number | spec:Special | txt:Text);

peginator will generate the following structure:
struct MultiConti{
    num: Option<Number>,
    spec: Option<Special>,
    txt: Option<Text>,
}

and always set exactly one of the options. This will lead to error-prone parsing steps later, because you might forget a field or something.
If you do this instead:
MultiConti = "multi:" (m:Number | m:Special | m:Text);

peginator will create something like this:
struct MultiConti{
    m: MultiConti_m,
}
enum MultiConti_m{
    Number(Number),
    Special(Special),
    Text(Text),
}

And in later code you can just match (conti.m)  { MultiConti_m:Number(n) => ...}.
Alternatively:
MultiConti = "multi:" (@:Number | @:Special | @:Text);

enum MultiConti{
    Number(Number),
    Special(Special),
    Text(Text),
}

As for the type declaration, I'd write:
FullTypeDeclaration = @:NormalTypeDeclaration
| @:TaskTypeDeclaration
| @:ProtectedTypeDeclaration;

NormalTypeDeclaration = Type ident:DefiningIdentifier [discriminant:KnownDiscriminantPart] Is definition:TypeDefinition
[aspects:AspectSpecification] ";";

Oh, and BTW, in the peginator grammar, | has very low precedence, so you don't need the bracket in (And | And Then | Or | Or Else | Xor).
But you do need to change the order: (And Then | And| Or Else | Or | Xor)
BTW2: You only need to @export rules where you want to directly use the Rule::parse() function in rust. This is usually the root rule (I guess compilation in case of Ada?)

@badicsalex Thank you for that! I'll go through the grammar and modify it to add (pseudo)? rules to make the structure simpler. (I'll have to inline a lot of rules anyway -- there are lots of rules that are just alternatives, and I don't want an over-complicated AST later, but I'll worry about that later.)

By the way 4: peginator was designed so that it gives you very fine grained control over the resulting parse tree structure, so that later steps are easy. You should always think about what you will do with the parse tree when designing (or in this case, rewriting) the grammar. If you are looking to just syntax check Ada files or something, peginator may not be the tool of choice.
E.g. You can use @: rather liberally to reduce the depth of the AST.

@badicsalex I'm trying to write a compiler for it. Right now I just want to parse stuff; I'll work on minimizing it and making it easier to analyze later.

The fine-grained control and really good syntax is what made me want to try this library. I've got one last question about this and then I'll go try to refactor this a bit.
The Ada grammar has a rule called name. This Rule is the really "ambiguous" rule in the grammar and why I may need to do post-processing of the parse tree: most of the alternatives in that rule can match. As an example, the indexed_component rule is:
IndexedComponent = prefix:Prefix "(" exprs:Expression {"," exprs:Expression} ")";

But the Slice, FunctionCall, and GeneralizedIndexing rules also have similar syntaxes:
Slice = prefix:Prefix"(" range:DiscreteRange")";

GeneralizedIndexing = prefix:Prefix parameters:ActualParameterPart;

# ...

FunctionCall =FunctionName
| FunctionPrefix ActualParameterPart;

ActualParameterPart ="(" ParameterAssociation {"," ParameterAssociation}")";

ParameterAssociation =[FormalParameterSelectorName "=>" ] ExplicitActualParameter;

ExplicitActualParameter = Expression | VariableName;

So I might need to do some rewriting of the tree. (Or I could be crazy and go "Hey, let's build a symbol table with check  functions..."). Any recommendations for resolving this? (I'm not too worried about it right now, but in future I do want the parser to try to figure this stuff out on it's own. I'd refactor the grammar but whenever I do that I always worry that I might end up changing the language in some way I don't understand right away.)

Ooof, I looked at the link and this looks pretty complicated. I think you won't be able to distinguish between these cases in the parsing step. Take A(1)(2)... Is it a two dimensional array? An array of functions? A function returning an array? You will only know if you know what A is.
What I'd do is store it in some relatively generic form in the parsing step (like storing the prefix and the expression in the brackets), and then during a later step you check the type of A (which you should know by then), and act accordingly. So instead of directly match-ing on the syntax node enum, you do a smarter, dynamic dispatch and error handling.
I wouldn't exactly do "post-processing of the tree", as in "create a new tree where these are disambiguated, but everything else stays the same", but you will have to do some lowering step that completely rewrites the tree anyway, e.g. to get to SSA form, and you can just shove this dynamic dispatch there.
I don't think there's a way to have a list of parsed functions during parsing, because the checking functions (even the custom ones) have to be stateless because of the backtracking.

No problem, have fun :)


⚠ Unifi Controller has degraded performance

In bf30683, Unifi Controller (https://unifi.vflits.com/) experienced degraded performance:

HTTP code: 200
Response time: 10606 ms

Resolved: Unifi Controller performance has improved in 5057cb4 after 12 minutes.


Enhancement: Add iam_role_id as variable in root main.tf to permit referencing pre-built IAM roles instead of creating them

Right now, if you run the terraform-aws-vault role as a user who does not have permissions to create a role (but can assign them), then this build will not be successful due to the need to create a role that just grants describe-instances, describe-groups, and describe-tags. I'd like to be able to supply the iam_role_id, but that's not possible.
https://github.com/hashicorp/terraform-aws-consul/blob/master/modules/consul-iam-policies/main.tf
I will submit a PR later to implement this. Figured I would create the issue and reference it after.

I am currently working on version that would do this but am still testing:
https://github.com/thenom/terraform-aws-consul

Any updates on this? This is a huge gap for my use case.
I need special permissions to allow my instance to grab certificates at boot, which as it stands, cannot be modified with the current architecture.
Ideally, at least please let me specify additional policy rules, or override the module used.


Add some class recommendations

One student asked:

What are some CS electives you highly recommend? Are there any philosophy or ethics classes that do not have prerequisites and that a STEM major might like?

We answered:

Tons! You might consider CS 182 (Ethics, Public Policy, and Technological Change), Phil 13N (Justice Across Borders, though it’s not very STEM related), and CS 152 (Trust and Safety Engineering). You might also consider INTLPOL 268D (Online Open Source Investigation) and INTLPOL 268 (Hack Lab).

Perhaps we could add some other class recommendations?

Someone asked:
Future opportunities in CS + Social Good?


[cmake] Avoid passing -mcx16 to non-x86_64 compilations

We were checking the host instead of the target compiler, causing many warnings about passing -mcx16 while building the runtime for arm.

@swift-ci please test

I did a quick check locally by grepping the build.ninja file for "mcx16" before and after this change.  All the differences were in command-lines that use a non-x86_64 arch/target.

@compnerd the windows failure https://ci-external.swift.org/job/swift-PR-windows/9442/console doesn't say much about what happened, but it's during the stdlib build so presumably it's triggered by this PR.

I wonder if it is just a race condition?

@swift-ci please test Windows platform

Build failed
Swift Test OS X Platform
Git Sha - 9422c4b213dab81c234f38c177d458f74c83281f

@swift-ci please test macOS

Build failed
Swift Test OS X Platform
Git Sha - 9422c4b213dab81c234f38c177d458f74c83281f

@swift-ci please test macOS

I already had a pull submitted to fix this, @benlangmuir, mind testing and merging the remaining cleanup there, #35023? I will rebase for this change and then we can get it in.


Allow enabling of second factors by configuration

@DRvanR Please review.

@DRvanR Merging so I can get Joost back on track.

:+1: , had been discussed earlier.


Review and update python typing usage.

Codecov Report
Base: 28.70% // Head: 28.70% // No change to project coverage :thumbsup:

Coverage data is based on head (cfcf675) compared to base (bc17aba).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           stable     #994   +/-   ##
=======================================
  Coverage   28.70%   28.70%
=======================================
  Files          72       72
  Lines        5852     5852
  Branches      306      306
=======================================
  Hits         1680     1680
  Misses       4109     4109
  Partials       63       63

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

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


Spawn

Hi I'm Mirak.
About Mob Captain Spawn, how do i set the spawn distance so they don't spawn underground but on the surface?
1 more, how to manage their number to spawn?
That's all, thankyou!!

You can't yet determine how much they should spawn on the surface or in caves, that feature is planned.
To configure how many spawn, you can run /function ps-mob:config in chat.
In the menu you can then lower the cool down or increase the spawn change to increase how many MobCaptains spawn.

It was very helpful, thanks FYI!
Pada tanggal Kam, 15 Jun 2023 pukul 13.03 PuckiSilver <
@.***> menulis:

Closed #15 https://github.com/ps-dps/MobCaptains/issues/15 as completed.
—
Reply to this email directly, view it on GitHub
https://github.com/ps-dps/MobCaptains/issues/15#event-9535986799, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BASW2QYISW7GKMHFYHG77JLXLKQRVANCNFSM6AAAAAAZHHTZGI
.
You are receiving this because you authored the thread.Message ID:
@.***>


.Net: Microsoft.SemanticKernel.AI.ChatCompletion.ChatHistory is not deserializable

Describe the bug
ChatHistory is not deserializable due to ChatMessageBase being an abstract class:
[2023-08-14T17:33:25.624Z] fail: Microsoft.SemanticKernel.IKernel[0]
[2023-08-14T17:33:25.624Z]       Function call fail during pipeline step 0: RecommendProductPlugin.RecommendProduct. Error: Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Microsoft.SemanticKernel.AI.ChatCompletion.ChatMessageBase'. Path: $[0] | LineNumber: 0 | BytePositionInLine: 2.

To Reproduce
Steps to reproduce the behavior:
var chatHistory = JsonSerializer.Deserialize<ChatHistory>(chatHistoryString);

Fails with both System.Text.Json and Newtonsoft.Json.
Expected behavior
In the scenario where chat history is not stored on the server side it is important to easily pass the object back and forth serialized as a string.
Platform

OS: Mac
IDE:  VS Code
Language: C#
Source: Microsoft.SemanticKernel, 0.19.230804.2-preview

Additional context
Building a chatbot with the chat history stored on the client side.

Had to come with something like this to be able to unblock the development progress:
public class ChatHistoryMessageSurrogate
{
    public AuthorRoleSurrogate Role { get; set; }

    public string Content { get; set; }

    public ChatHistoryMessageSurrogate(AuthorRoleSurrogate role, string content)
    {
        this.Role = role;
        this.Content = content;
    }
}

public class AuthorRoleSurrogate
{
    public string Label { get; set; }

    public AuthorRoleSurrogate(string label)
    {
        this.Label = label!;
    }
}

public static ChatHistory? DeserializeChatHistory(string chatHistoryString, ILogger? logger)
{
    ChatHistory? chatHistory = null;

    try
    {
        var chatHistoryList = JsonSerializer.Deserialize<List<ChatHistoryMessageSurrogate>>(chatHistoryString)!;

        chatHistory = new ChatHistory();
        foreach (var chatHistoryMessage in chatHistoryList)
            chatHistory.AddMessage(authorRole: new AuthorRole(chatHistoryMessage.Role.Label), content: chatHistoryMessage.Content);
    }
    catch (JsonException ex)
    {
        logger?.LogError(exception: ex, message: "Exception deserializing ChatHistory: {chatHistory}", chatHistoryString);
    }

    return chatHistory;
}

@alexminza, thanks for finding this.  We will take a look.

The issue is still happening with the latest SK beta release and .NET 8:
Exception has occurred: CLR/System.NotSupportedException
Exception thrown: 'System.NotSupportedException' in System.Text.Json.dll: 'Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Microsoft.SemanticKernel.AI.ChatCompletion.ChatMessageBase'. Path: $[0] | LineNumber: 0 | BytePositionInLine: 2.'
 Inner exceptions found, see $exception in variables window for more details.
 Innermost exception 	 System.NotSupportedException : Deserialization of types without a parameterless constructor, a singular parameterized constructor, or a parameterized constructor annotated with 'JsonConstructorAttribute' is not supported. Type 'Microsoft.SemanticKernel.AI.ChatCompletion.ChatMessageBase'.

Platform

OS: macOS Sonoma 14.1.1
IDE:  VS Code 1.84.2
Language: C# 8.0.100
Source: Microsoft.SemanticKernel, 1.0.0-beta6

@markwallace-microsoft  can you take a look at this?


Can now use User.getUserDetailsForImpersonation

While working on https://github.com/jenkinsci/jenkins/pull/4848 I noticed that #22 has a pending item which is addressed by an API in https://github.com/jenkinsci/jenkins/pull/3074.

Done by #39.


update docs to show no longer compatible with later django

The patterns modules cannot be imported, as deprecated in django 1.8
this file
C:\apps\nested-formset\demo\nested_formset_demo\urls.py
Documentation here https://docs.djangoproject.com/en/1.9/ref/urls/#patterns
I'm running django 1.11

Wouldn't this be an easy fix by changing the urlpatterns to urlpatterns =[]?


Holographics Display bug

When you put a Envoy Location on another (One up of other) the holographics of the the top's one is bug.

Well Simple Answer Don't Them On Top Of Eachother

@PrimeMoose you got a point lol
@Juanete18 just dont double stack them then.


Updates to provide compatibility with Fog 2.4.0, take 2

Ooops, crunched my own PR #48 .  Trying again.
Remaining Issues:

Should Aliases to the old names be included here for better compatibility?
LoadBalancer test failures

@geemus new pull req, I cancelled the prev on accident.

K , thanks. I have some travel presently, but will try to get to it week after next.

FWIW, the way I've usually done renames in stuff has been to still support the old name, but when you use it there is a deprecation warning that then passes things along to the new version. So basically aliasing but a bit noisier. Not saying we have to do that here, but thought I would share it as a data point at least.

I'll work on a review of what's here, but could you maybe rebase this when you get a chance? I think that should bring in the github actions/ci config so that we can get the test run and see the load balancer or other issues.

Thanks again for taking this on, I know from experience how tedious it can be to get through these renames. I was finally able to find some time to review everything and it's looking good so far.
I think it's probably good to have the aliases like you do at least for the time being for backwards compatibility, it might be slightly nicer to have those classes actually warn and then handoff like I mentioned (I guess you could do it on initialize probably). What do you think?
Hopefully if we rebase the CI will run and then that can hopefully give us something to dig into around the load balancer stuff (though it looks like master/main CI is failing already anyway, so maybe it's a broader issues anyway).

I'll work on a review of what's here, but could you maybe rebase this when you get a chance? I think that should bring in the github actions/ci config so that we can get the test run and see the load balancer or other issues.

This pr is current on fog-rackspace:master, not sure why it isn't kicking in.

Oops, sorry about that then. Let me try to close/reopen, sometimes that does it.

Ok, so that does appear to have kicked off the tests again.
Unfortunately I think they are broken on main as well as this branch, which makes things harder certainly.
@vicomte Could you help point me to the particular LoadBalancer related breakage you were referring to so we can maybe try to sort that out at least?

The get_stats_test looks like something upstream changed what it is expecting. I looked at it briefly, but wasn't sure if changing this arg to just pass the attr hash was the correct action.
   tests/rackspace/requests/load_balancers/get_stats_tests.rb
      Fog::Rackspace::LoadBalancers | load_balancer_get_stats (rackspace)
    Initialization parameters must be an attributes hash, got Fog::Rackspace::LoadBalancers::VirtualIp     <Fog::Rackspace::LoadBalancers::VirtualIp
      id="204",
      address="246.294.440.598",
      type="PUBLIC",
      ip_version="IPV4"
    > (ArgumentError)
      /Users/mhassman/.rvm/gems/ruby-3.2.2/gems/fog-core-2.4.0/lib/fog/core/collection.rb:89:in `new'
      /Users/mhassman/.rvm/gems/ruby-3.2.2/gems/fog-core-2.4.0/lib/fog/core/collection.rb:79:in `block in load'
      /Users/mhassman/.rvm/gems/ruby-3.2.2/gems/fog-core-2.4.0/lib/fog/core/collection.rb:19:in `each'
      /Users/mhassman/.rvm/gems/ruby-3.2.2/gems/fog-core-2.4.0/lib/fog/core/collection.rb:19:in `each'
      /Users/mhassman/.rvm/gems/ruby-3.2.2/gems/fog-core-2.4.0/lib/fog/core/collection.rb:79:in `load'
      /Users/mhassman/Projects/fog-rackspace/lib/fog/rackspace/models/load_balancers/load_balancer.rb:111:in `virtual_ips='


Add more scopes to Dark+ and Light+ themes

Original context: https://github.com/jeff-hykin/cpp-textmate-grammar/issues/388
We should look into adding some of the scopes from https://github.com/microsoft/vscode-cpptools/blob/master/Themes/themes/cpptools_dark_vs.json into the Dark+ and Light+ themes to better highlight C++. If other grammars use those scopes it could improve highlighting for them too.

@Colengms I can go through https://github.com/microsoft/vscode-cpptools/blob/master/Themes/themes/cpptools_dark_vs.json and see if it makes sense to add all the scopes that the VS Code + themes don't have and try to translate the colors to match the existing theme colors, but if there are specific scopes that you think particularly important I can do those first.

I added the scopes you had in the themes to the dark+ and light+ themes. Some of the colors didn't quite match what we already have, so I tweaked them. @Colengms let me know what you think!

Thanks Alex!
@jeff-hykin Are there any C++ TextMate tokens that should be included in the default+ themes?

Scope
Should Have the Same Color As
Example

meta.embedded.assembly
string
asm("movl %a %b");

entity.other.attribute
entity.name.namespace
[[gnu::hot]]

entity.name.operator
keyword.other.operator
bool operator==()

keyword.operator.noexcept
storage.modifier
void func() noexcept;

I think keyword.operator.wordlike ( xor, and, or, etc) is the only one I'd add to matter's list.
I'd recommend not coloring meta.preprocessor, or really any meta.
I mostly recommend other themes because of the punctuation and operator highlighting. But that is kind of a fundamental difference since I think Dark+ theme doesn't color operators/punctuation in any languages.
The reason I recommend it in C++ is because it makes most white colored things (the unknown) variables and difficult-to-match types. Since variables can't (yet) be colored directly, this is the next closest thing to highlighting variables directly. By choosing for both punctuation and operators to be white, it can make for a decent number of lines of completely unhighlighted code like Type value = other_val + val2;
If you want to be advanced, you can color all of the function-related punctuation yellow, all of the class/struct/enum/union punctuation green.

@Colengms after talking with @aeschli about the history and current state of our themes, I've made a few changes to the scopes I added to better match our current colors and theme philosophy. You can see the changes in https://github.com/microsoft/vscode/commit/c1fdabc4f28f20fd63c6a6ef68d182617fb82813
I do have a few question/comments about some of the scopes that I removed:
variable.parameter: We currently don't do special coloring for any language for parameters and we want to stay consistent.
entity.name.function.operator: Can you give an example of this? If it makes sense, we could add it to one of our existing color groups.
entity.name.function.preprocessor: Since this one claims to be a function, we'd prefer to keep coloring it like a function. If it isn't actually a function, perhaps the scope could be renamed.
entity.name.operator.custom-literal: What should this be colored like? Is it a property?

@matter123 and @jeff-hykin I added the specific scopes you mentioned with the exception of the meta scope in https://github.com/microsoft/vscode/commit/6ba021ba320ee21bd533cd51dc756a6a56f92c4c

@alexr00
entity.name.function.operator is an operator overloaded as a function.  It will color both definition and use of the operator.
entity.name.function.operator.member is similar, but refers to when an operator is overloaded as a member.
entity.name.function.preprocessor is for preprocessor functions.  i.e. macros.
entity.name.operator.custom-literal is for user-defined literals.  https://en.cppreference.com/w/cpp/language/user_literal

@alexr00 the current color rule for meta.embedded causes assembly string literals to be colored white, overriding the previous scope of stting.quoted.double.

I've undone the change to have #include be blue instead of purple in dark (and the equivalent in light) because it is a dramatic change that is not also a strict improvement as it is already colored fairly specifically.
@Colengms and @matter123  I'll look at the remaining proposed changes in October!

Hi @alexr00 , We have https://github.com/microsoft/vscode-cpptools/issues/938 regarding the C++ alignof operator (keyword.operator.alignof.cpp) not being colored by default themes.  Could we add that also?

I added:
entity.name.operator.custom-literal, colored as a function
entity.name.function.preprocessor colored as preprocessor
meta.embedded.assembly colored as a string
I didn't add entity.name.function.operator* because it is already colored as a function, which seems ok.

Verified through integration tests.


Static arrays can not be captured.

Fixed https://github.com/BuilderIO/qwik/commit/7fadd186e2358c95cd5902a800c24b448e6d19c5


Settings are not saved on PC startup

Describe the bug
Each morning that I turn on my PC, the BeatBind window pops up and I must redo all of my settings including the client ID and device ID, as if starting BeatBind for the first time.
To Reproduce
Steps to reproduce the behavior:
Add the client ID, client secret, click on get devices and select my device. Then I unselect 'shift' in the modifiers for play/pause, previous track and next track. Then I update the play/pause key to be 'up'. Then I check 'start on windows startup' and 'start minimized'. Then I click 'save' and then I click 'start & close'.
Expected behavior
I expect my settings to save.
Desktop (please complete the following information):

OS: Windows 11 Home, 22H2, Build 22621.3155
Version 1.3.0

where are you keeping the BeatBind folder? If you're keeping it inside your Program Files folder then it needs to be ran in admin mode to be able to save and write files. I recommend placing it somewhere else

AH! That was it. I moved it from Program Files to a different folder and it has been fixed. Thanks for the prompt response!


How to style Outside days of month?

By following the extending example I was able to override Cell Days to add a button and style the day number text with the Outside bool property.
I would now like to have the entire day box of month view be stylied if it is Outside the current month. Is there a built in way to do this?
Thank you

You would override RenderCellContainer instead of RenderCellDayNumber.
Look at MonthView.razor in the source code (look in the main branch not dev branch).  Copy the existing method to use as a starting point.
However the Month View has been completely rewritten.  The changes you make now will not work when the next version is released.  So you might want to wait before doing anything.

Thank you for the information 👍@danheron


Remover Node Output

How can I remove Node Output, without having to remove it in sequence
Eg: I have [output_1, output_2, output_3]
if i remove it in sequence it works
if by chance I want to remove output_2 and then output_3
gives error

By removing output_2. Output 3 becomes 2.
Remove output_2 and remove output_2.
editor.removeNodeOutput(1, 'output_2');
editor.removeNodeOutput(1, 'output_2');


163 include open fee fines migration into migration tools close #163

Hi! Here's a mega PR! However it's mostly new code that only minimally touches the shared files.
For this to work we are going to need to upgrade the folio-uuid dependency https://pypi.org/project/folio-uuid/0.2.8/.

Out of scope:
https://github.com/FOLIO-FSE/folio_migration_tools/issues/616
https://github.com/FOLIO-FSE/folio_migration_tools/issues/617
https://github.com/FOLIO-FSE/folio_migration_tools/issues/618

I went ahead and bumped the folio-uuid version as part of this PR. Once that is verified and merged, I'll merge.

Codecov Report

Merging #615 (efbc4fd) into main (6b7bf60) will increase coverage by 0.31%.
The diff coverage is 77.81%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #615      +/-   ##
==========================================
+ Coverage   69.87%   70.19%   +0.31%
==========================================
  Files          97      101       +4
  Lines       10453    10789     +336
  Branches     1247     1283      +36
==========================================
+ Hits         7304     7573     +269
- Misses       2864     2931      +67
  Partials      285      285

Flag
Coverage Δ

unittests
70.19% <77.81%> (+0.31%)
:arrow_up:

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

Impacted Files
Coverage Δ

...n_tools/mapping_file_transformation/item_mapper.py
45.76% <ø> (ø)

...ls/mapping_file_transformation/ref_data_mapping.py
78.68% <ø> (ø)

...io_migration_tools/migration_tasks/batch_poster.py
18.20% <ø> (ø)

...ation_tools/migration_tasks/migration_task_base.py
31.79% <0.00%> (ø)

...ls/migration_tasks/manual_fee_fines_transformer.py
35.57% <35.57%> (ø)

...ing_file_transformation/manual_fee_fines_mapper.py
94.33% <94.33%> (ø)

src/folio_migration_tools/report_blurbs.py
100.00% <100.00%> (ø)

...ration_tools/test_infrastructure/mocked_classes.py
94.36% <100.00%> (+0.92%)
:arrow_up:

tests/test_manual_fee_fines_mapper.py
100.00% <100.00%> (ø)

tests/test_manual_fee_fines_transformer.py
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Remove width=940px

@rcaloras you still need this pushed through?

No, but I don't think the 940px does anything now. So might as well remove it!
👍 🍔 🍟


[FEATURE] JRTB-6: As a user, I want to see list of my group subscriptions

https://github.com/JiLaRiAtIoN/javarush-telegrambot/pull/22


Add e2e tests

Very early version, so excuse the lazy shell scripting, plus need to add a lot more test cases.
Use make dev/e2e  from root of the repo to run the tests.
Fixes #317
cc @v-thakkar @javierhonduco

fyi, the Go build in CI here fails since the test also requires the Parca server to be running.

You might want to run make go/deps to fix failing checks.


integrate user input of date of investment / liquidation

This has been preliminarily completed; just need to now create the observe event reactives for cross-tab tracking of the date input field.

The reactives are now taken care of.


Content enhancement: add guest guidelines

Summary
Cardano Catalyst Community Eastern Town Hall (C3ETH) Breakout rooms hosting Guidelines
Basic example
Include a basic example, screenshots, or links.
Motivation
Why are we doing this? What use cases does it support? What is the expected outcome?
Checks

[ ] Read Create a Pull Request
[ ] Supports all screen sizes (if relevant)
[ ] Supports both light and dark mode (if relevant)
[ ] Passes npm run test

thanks


mdbook: 0.1.8 -> 0.2.0

https://github.com/rust-lang-nursery/mdBook/releases/tag/v0.2.0
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

cc @havvy

Success on aarch64-linux (full log)
Attempted: mdbook
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0
shrinking /nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0/bin/mdbook
strip is /nix/store/zrs21zqcchgyabjf4xfimncdq16njizc-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0/bin
patching script interpreter paths in /nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0
checking for references to /build in /nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0...
/nix/store/agm6zc3l7kcbly6nfgqs1mwzl4xl8g6j-mdbook-0.2.0

Success on x86_64-linux (full log)
Attempted: mdbook
Partial log (click to expand)

installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0
shrinking /nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0/bin/mdbook
strip is /nix/store/1hi76hr87bd1y1q1qjk0lv8nmcjip1c8-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0/bin
patching script interpreter paths in /nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0
checking for references to /build in /nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0...
/nix/store/bjzs43916rdnnsg0875yk4m5qlz8m79p-mdbook-0.2.0

You're gonna want to keep at least 0.1.8 until we can get all the links updated in the Rust books.

Is there a way to find out, when this happen (like an issue?).

I've a tracking issue at https://github.com/rust-docs/team/issues/23

v0.2.3 is the latest now ...
It's not good for the nix pkgs ecosystem, that updates are hold back by such circumstances.
Should be solved by pinning nix pkgs or some other solutions ...

I opened #65890 to move this to 0.3.1 (which is the latest version).
Agree with @cw789 that external projects depending on an old version via Nix should look to pin that version separately.

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/50


okteto: 2.18.0 -> 2.18.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/okteto/okteto/releases.
meta.description for okteto is: Develop your applications directly in your Kubernetes Cluster
meta.homepage for okteto is: https://okteto.com/
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.18.2 with grep in /nix/store/mpg0ksr7y977nz6kbcni4df18d6dqz6j-okteto-2.18.2
found 2.18.2 in filename of file in /nix/store/mpg0ksr7y977nz6kbcni4df18d6dqz6j-okteto-2.18.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
okteto

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/mpg0ksr7y977nz6kbcni4df18d6dqz6j-okteto-2.18.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/mpg0ksr7y977nz6kbcni4df18d6dqz6j-okteto-2.18.2
ls -la /nix/store/mpg0ksr7y977nz6kbcni4df18d6dqz6j-okteto-2.18.2/bin

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

1 package built:

okteto

Maintainer pings
cc @aaronjheng for testing.

2.18.3 is out.


Create Subtree from selected Nodes not working

Bug Report
Bug Description
Currently the feature is not working (see Screenshot)

We should investigate if this is still an issues with the recent updates to the Web-GUI.

This is fixed in the web GUI Version 2.0.4


reports table is created.

Targets #1234

Description

Checklist:

[x] I have performed a self-review of my own code.

I am creating another pull request instead. That's why I am closing this PR.


Product flavors are not supported

Somewhat related to Issue #1 . Plugin crashes if project contains product flavors

Fixed in 0.2.0 (see #5)


[discuss] update slf4j to 2.x in 4.2 branch

Slf4j 1.7 has not been maintained for almost 2 years. Shall we update slf4j to 2.x in 4.2 branch? I am willing to send a PR. :)
@normanmaurer @chrisvest

ping @normanmaurer @chrisvest

@shoothzj is slf4j 2.x api compatible with slf4j 1.x ? or does it use different package names ?

@normanmaurer Yes it is. The "only" issue is that users have to upgrade their implementation (eg logback) as well.

@normanmaurer Yes, It's compatible. And I have update in other project. https://github.com/apache/bookkeeper/pull/4248

Log4j just update minor version. Logging configuration files don't need to change.

SLF4J API version 2.0.0 requires Java 8 and introduces a backward-compatible fluent logging API. By backward-compatible, we mean that existing logging frameworks do not have to be changed in order for the user to benefit from the fluent logging API.

I think that's fine... please send over a PR :)


Add heuristic logic to unify the oc block to optimize the peak memory consumption for LLM.

Description
Add heuristic  logic to unify the oc block to optimize the peak memory consumption for LLM.

OV PR:https://github.com/openvinotoolkit/openvino/pull/19575


Fixing a bug with a number of threads for the forward loop.

Hi Yaroslav,
Thank you for making effort to port RoI pooling to theano. There is a bug in your implementation. Number of threads in the forward pass should be equal to the size of an output tensor but not an input tensor.
Best Regards,
Minh

Thanks for fixing this!


什么时候更新

No description provided.

Thanks for your interest and patience. We release our inference codes at https://github.com/antgroup/animate-x.


WIP Docker

Quick dockerfile for turbot.
Definitely not optimized and creates a 1.28gb image, I suspect a lot of that is the scientific libraries though.
Needs instructions on use too.

Codecov Report

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

@@            Coverage Diff            @@
##            master      #154   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines         1148      1148
  Branches       224       224
=========================================
  Hits          1148      1148

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 410fc75...e3f5a89. Read the comment docs.

WDYT about using pip install . instead of installing it off PyPI?

Might also be worth reading thru https://pythonspeed.com/articles/pipenv-docker/ and especially the section titled Fast reproducible Docker builds with poetry. Maybe to support this we could add poetry export -f requirements.txt -o requirements.txt to the publish.sh script.
One thing I've been thinking about how to solve for a while now is how to make it so turbot knows if it's on a release or if it's on a development branch (or on master with changes past the last release). Right now if you do a turbot -v it'll give you the version of the last published release, even if there's been a ton of changes since then. Ideally I'd want it to be able to somehow know that it's a different version if things have been changed. Whatever process allowed this to be done would need to ensure that the generate requirements.txt file was also kept up-to-date so it would have to be done somewhere other than in publish.sh. Maybe as a git push hook perhaps? Dunno.

So my first port of call was trying pip install . but it does not seem to be able to handle the vendorized dep and breaks. Export could maybe work but I think it'll run into the same issue.

@shughes-uk Sorry, try again now that I've removed the vendorized dep. No idea why that just simply didn't work.


Update Radio Status Information for RFD868 Telemetry Radio

Adding radio status information for SI2, as refer from the pull request here: https://github.com/SEESAI/SeesInterface2/pull/332.

Also thanks for sorting out the tests - I had no idea we could skip some but still have them show in the check list, that's really neat.


Fix "undefined" in line 17

Change this.client.id to this.client.user.id

I haven't tested this so let's hope it works.

I haven't tested this so let's hope it works.


Grammar fixes in add-web-api-application.md

A couple small grammar fixes

Docs Build status updates of commit 4c83857:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory-b2c/add-web-api-application.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Looping from beginning is impossible with setPosition?

Describe the bug
In v4, If you use AKPlayer with looping and use setPosition API, the player loops between the position set with setPosition and the players total duration. What I expected and needed was to start from the position set by setPosition API and loop to the beginning when looping occurs. I am not sure if this is intentional or not but I need to be able to seek to a certain point in the Player and loop to the beginning and for now it seems that this is not a feature in AudioKit.
Is there a way or workaround to achieve this? I need to accomplish this as a requirement and I do not want to switch AudioKit just for this.
To reproduce here is an full playground example:
import AudioKitPlaygrounds
import AudioKit

let file = try AKAudioFile(readFileName: "3.aac")

//: Set up a player to the loop the file's playback
var player = AKPlayer(audioFile: file)
player.isLooping = true
player.buffering = .always

AKManager.output = player
try AKManager.start()

player.setPosition(player.duration - 2)
player.play(at: AVAudioTime.now())
// player loops between player.duration - 2, player.duration  forever. I want it to loop between 0, player.duration

import PlaygroundSupport
PlaygroundPage.current.needsIndefiniteExecution = true

Expected behavior
I expected it to loop back to the beginning.
Details:

AudioKit Version 4.11.2

Additional context
Tried to test this issue on v5-develop but apparently you can not loop sounds and use seek API in v5. I looked at the source code to verify this.

Okay strange enough, if we omit the line
player.buffering = .always
it does loop back to beginning but AFAIK this will not produce seamless loops which is needed for my case

Seems to work in v5 now.


invert marked color

I find it is super difficult to differentiate between selected file with media files, they both got the same text color. Maybe a good idea to invert the marked color.
Before:

After:

Thanks!


ZLIB 1.2.9 not found

NOTE:  I am a TOTAL noob with AWS Lambda.  Your patience is appreciated.
I set up serverless, followed the README instructions, and I get this error when trying to access any (valid) user's feed:
XML Parsing Error: not well-formed
Location: https://BLAHBLAH.execute-api.us-east-1.amazonaws.com/dev/rss?user=username
Line Number 1, Column 1:
{errorType=Runtime.UnhandledPromiseRejection, errorMessage=Error: /lib64/libz.so.1: version ZLIB_1.2.9' not found (required by /var/task/node_modules/canvas/build/Release/libpng16.so.16), trace=["Runtime.UnhandledPromiseRejection: Error: \/lib64\/libz.so.1: version ZLIB_1.2.9' not found (required by /var/task/node_modules/canvas/build/Release/libpng16.so.16)","    at process. (file:///var/runtime/index.mjs:1250:17)","    at process.emit (node:events:513:28)","    at emit (node:internal/process/promises:140:20)","    at processPromiseRejections (node:internal/process/promises:274:27)","    at processTicksAndRejections (node:internal/process/task_queues:97:32)"]}
^
I additionally tried compiling zlib 1.2.9 and moving libz.so.1.2.9 to ./node_modules/canvas/build/Release/libz.so.1 but I do not wish to replace the system install since it'll break a bunch of packages.
Is the only recourse building in a VM (with 1.2.9)?  Is there a bit of code or requirements I could change instead?

Sorry this hasn't been maintained due to lots of problems with accessing TikTok. I'll archive it.
However this is still working (mostly!) for me:
https://github.com/conoro/tiktok-rss-flat


Add bare bones service toolkit page

Adds a service toolkit presenter and view.
I've also taken the opportunity to remove the pointless service_manual_ prefix that's throughout this application which is the bulk of this diff – worth stepping through the commits one by one.
Relies on alphagov/govuk-content-schemas#473 being merged before this will pass on CI.
https://trello.com/c/GHY41UJi/546-create-service-toolkit-page

I clicked around a guide, topic, the homepage, the service standard whilst using the dummy content store. All seemed to work OK. Suggest we test relatively thoroughly on integration.

:+1: :shipit:


CalDAV: beforeCreateFile

Hello everyone!
I built a CalDAV server by implementing the caldav iBackend interface and initializing your calendarRoot with it. Most HTTP requests work very nicely with my Thunderbird Lightning plugin. However, creation of CalDAV calendar objects (HTTP-PUT request) fails.
From what I understand, this is caused by the caldav plugin "beforeCreateFile"-event handler expecting dataOrStream not to be null. It fails on validateICal if it is null, which is always the case, as httpPut calls createFile with dataOrStream = null.
One solution that I could see is the modification of DAV.handler#createFile in a way that it fetches the body prior to emitting the "beforeCreateFile" event. (For parent nodes that are not capable of handling streams).
Another solution could be to fetch the iCal content from the request body in the caldav plugin's beforeCreateFile method, just as the beforeWriteContent event handler does.
I tried the last solution and that worked for me.

fixed by merge


[KYUUBI #6171][FOLLOWUP] Restore flink-1.16 profile to recover Flink IT

:mag: Description
I forgot the Flink cross-version verification integration tests require using the target Flink version to bootstrap a Flink Mini Cluster, this PR restores flink-1.16 profile to recover Flink IT
Types of changes :bookmark:

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Pass GA.

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Verified in local, merged to master to recover CI.


bind_cols() gives "incompatible number of rows" error

To reproduce:
sc <- spark_connect("local")
d1 <- data_frame(
  x = 1:5
)
d2 <- data_frame(
  y = runif(5)
)
tbl1 <- copy_to(sc, d1)
tbl2 <- copy_to(sc, d2)

bind_cols(tbl1, tbl2)
## Error in cbind_all(x) : incompatible number of rows (1, expecting 11)

bind_cols() is provided by dplyr, but as far as I can tell it's not actually a generic, so we can't provide our own custom methods for it. Perhaps something to file upstream?

@richierocks would you mind sharing the use case for this? Implementing cbind would involving some plumbing due to partitioning of the data. See also https://github.com/rstudio/sparklyr/issues/35.

@kevinykuo This came about from trying to write a residuals method for ml_model objects. See https://github.com/rstudio/sparklyr/issues/557.
My first attempt looked like this:
residuals.ml_model <- function(x, ...) {
  actual <- sdf_register(x$data)  %>%
    select_(actual = x$response)
  predicted <- data_frame(predicted = predict(x))
  both <- bind_cols(actual, predicted)
  both %>%
      transmute(residual = predicted - actual)
}

The more general case is fortifying dataset with other tibbles. If you want to add a column to a tibble, use mutate. But if you want to add a whole tibble, it's natural to bind by columns.
Column binding is really just an inner join, so I think you can implement it by adding a dummy ID column (see how predict.ml_model() uses sdf_with_unique_id()), inner joining by that ID, then dropping the ID again.

sdf_bind_rows and sdf_bind_cols is merged. We asked for support in dplyr to make dplyr::bind_rows and dplyr::bind_cols generic, once we have that we can make this work as you would expect, Richie. In the meantime, sdf_bind_rows and sdf_bind_cols is the way to go.

Closing in favor of https://github.com/rstudio/sparklyr/issues/888


Upgrade from 1.7.1 to 1.8.1 incomplete bridge settings

Maybe it's only on Archlinux but I have updated my distribution and docker this morning, and I had lost any bridge (docker0) settings.
docker0 was present but no IP assigned, this command fix the problem :
#> ip addr add <IP_ADDRESS>/16 dev docker0
Thanks.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

docker version :
Client:
 Version:      1.8.1
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   d12ea79
 Built:        Sat Aug 15 17:29:10 UTC 2015
 OS/Arch:      linux/amd64

Server:
 Version:      1.8.1
 API version:  1.20
 Go version:   go1.4.2
 Git commit:   d12ea79
 Built:        Sat Aug 15 17:29:10 UTC 2015
 OS/Arch:      linux/amd64

docker info :
Containers: 80
Images: 1477
Storage Driver: btrfs
 Build Version: Btrfs v4.1.2
 Library Version: 101
Execution Driver: native-0.2
Logging Driver: json-file
Kernel Version: 4.1.5-1-ARCH
Operating System: Arch Linux
CPUs: 4
Total Memory: 7.625 GiB
Name: *******
ID: ICKU:HVPK:Y3RI:HBYT:KUOW:67DR:5PLY:3OEQ:TCXC:MEWD:VNJT:AEHO

uname -a :
Linux ******** 4.1.5-1-ARCH #1 SMP PREEMPT Tue Aug 11 15:41:14 CEST 2015 x86_64 GNU/Linux

Details environment:

Running on physical Archlinux, previously using docker 1.7.1.

Same problem here. My docker containers had lost their network connection after upgrading to docker 1.8.1 on my Arch Linux machines. I can't say what exactly the problem was, but restarting the docker service some times seems to have fixed the issue.

Yep, just discovered this problem yesterday. Arch Linux and Docker v1.8.1.
For what it's worth a workaround is sudo systemctl restart docker. The bridge comes up with the correct IP address after that.

I disabled docker on startup and when I tried to manually start it (via systemd) I got an error from systemd and these logs:
Aug 18 08:36:14 astuart docker[2061]: time="2015-08-18T08:36:14.466766687-07:00" level=info msg="Option DefaultDriver: bridge"
Aug 18 08:36:14 astuart docker[2061]: time="2015-08-18T08:36:14.466859605-07:00" level=info msg="Option DefaultNetwork: bridge"
Aug 18 08:36:14 astuart docker[2061]: time="2015-08-18T08:36:14.471006056-07:00" level=info msg="Firewalld running: false"
Aug 18 08:36:14 astuart docker[2061]: time="2015-08-18T08:36:14.549109778-07:00" level=fatal msg="Error starting daemon: Error initializing network controller: Error creating default \"bridge\" network: Failed to setup IP tables, cannot acquire Interface address: Interface docker0 has no IPv4 addresses"
Aug 18 08:36:14 astuart systemd[1]: docker.service: Main process exited, code=exited, status=1/FAILURE
Aug 18 08:36:14 astuart systemd[1]: Failed to start Docker Application Container Engine.

@andrewstuart @Zenithar thanks for reporting the issue. @jfrazelle was facing the exact same issue and we were suspecting kernel 4.1.4+ specific issue (and/or also maybe systemd related). We fixed a few netlink related issues in docker master recently. Can you try the latest master from https://master.dockerproject.org/ and let us know if it helps ?

@mavenugo Yes, the latest master does appear to have fixed the issue. :-)

@andrewstuart https://github.com/docker/docker/pull/15601 is marked for 1.8.2 release and please try out the 1.8.2 RC when it is out.

FYI: another workaround I've been using is to start docker with:
docker -d --bip=<IP_ADDRESS>/16

None of the above workarounds corrected the problem for me. Here's the workaround that fixed 172. address properly on my machine:
$ sudo systemctl stop docker

# Consider doing a "watch ip addr" to see the magic happen
$ sudo docker daemon -D
^C

@t-richards can you please confirm if you still see the issue using the latest master ? My earlier comment on this : https://github.com/docker/docker/issues/15630#issuecomment-132258988

@mavenugo Having trouble building the latest master at this exact moment in time. Probably a PEBKAC error. I'll report back if I manage to get it working :smile:

@mavenugo The latest master appears to configure the docker0 device properly.
$ docker --version
Docker version 1.9.0-dev, build 7f46dc0

@t-richards thanks for the confirmation. https://github.com/docker/docker/pull/15601 will be included in 1.8.2 and this issue will be taken care of.

@icecrime Given that the issue is resolved in the master and https://github.com/docker/docker/pull/15601 is planned for 1.8.2, should we close this issue now ?

@mavenugo Sure, thanks.

this worked for me too!

@andrewstuart restarting the docker service worked for me too. Everyone should keep in mind that I enabled port forwarding in my systemd-networkd service:
[Network]
IPForward=yes

@gngeorgiev Thank you very much, I should have read systemd.network more carefully

I'm facing the same problem but adding the IPForward=yes does not resolve my issue,
Actually every time a container is stopped the bridge lost his ip and i have to manually restart docker / re add ip addr to the bridge :/

@joelwurtz are you seeing it in 1.8.2 ? if not, I would strongly advice you to move to 1.8.2.

Yes i am on 1.8.2, with more search this may come from another bug with a conflict with "wicd" (as seen in http://stackoverflow.com/questions/28923743/dockers-docker0-device-dies-repeatedly-inet-addr-disappears), i have disable it for the moment and no problems occurs so far

I am running 1.9.1 and I am still experiencing this. I wonder if it is due to the systemd-networkd starting after docker and setting the flags. For now I can resolve it by restarting the docker service but that isn't very neat.


add default colors for dark-themed browsers

Still not deployed, right?

It is, but only effective for manpages that have been rendered since it was deployed. See e.g. https://manpages.debian.org/testing/freedombox/plinth.1.en.html
I’ll trigger a full re-render now, so that existing pages should be refreshed, too. It can take another few days for that change to actually make it out.

plinth.1.en.html looks fine. Thank you for explanation.


Clusters state lifted one level up

Only the last commit is relevant, the rest is to express dependency on #622

/clusters/clusters -> /clusters because there isn't any other property
at level /clusters any more. Previously there was
/clusters/loadInProgress.
ADD_CLUSTERS and REMOVE_MISSING_CLUSTERS actions replaced by single
SET_CLUSTERS since they always (once) occured together.

This change is

Rebase now that #621 is merged and LGTM.
Looks like it's unique enough to be in parallel with #622.

Rebase now that #621 is merged and LGTM.
Looks like it's unique enough to be in parallel with #622.

Unfortunately there is a dependency: both #622 and this patch changes src/reducers/templates.js

#622 is merged now

@jniederm rebase, please.

Rebased, it seems like ready to merge.


add ability to set culture at the Plot level

Is there a way to have date ticks with different culture notation?
I currently change the culture of current thread, but that's not a best approach for sure:
var savedCurrentCulture = Thread.CurrentThread.CurrentCulture;
Thread.CurrentThread.CurrentCulture = CultureInfo.GetCultureInfo("it");

var plt = new ScottPlot.Plot(1200, 500);
plt.PlotSignal(data, sampleRate: 1, xOffset: dateFrom.ToOADate(), lineWidth: 3, color: Color.Red);
plt.AxisAuto(0.05, 0);
plt.Axis(null, null, 0, Math.Round((data.Max() + 2000) / 1000) * 1000);
plt.Ticks(dateTimeX: true);
plt.XLabel("Day");
plt.YLabel("Value");
plt.SaveFig(statsPlotFile);

Thread.CurrentThread.CurrentCulture = savedCurrentCulture;

Hi @teejay-87, there's not currently a way to do that but adding a way would be a good idea! A way to set the culture at the plot level would be nice to set date format and numerical format of tick labels.
I'll add this shortly. Thanks for the idea!

This is what I came up with. Thanks for the suggestion @teejay-87!
I'll close this ticket when it's updated on NuGet
var culture = System.Globalization.CultureInfo.CreateSpecificCulture("de"); // German
plt.SetCulture(culture);

var culture = System.Globalization.CultureInfo.CreateSpecificCulture("hu"); // Hungarian
plt.SetCulture(culture);

Just pushed 4.0.18 to NuGet, it should be live a few minutes from now.
Let me know if this doesn't do the trick!

Thanks, I'll try it in the next days


Macho-extractor: support extraction of any object from fat Mach-O

The original tool was created to extract archives from fat Mach-O archives. But it would be useful, if it could extract any object from fat Mach-O. E.g. binary, that is not an archive.
Maybe, this is all that needs to be done:
---    valid = isStaticLibrary();
+++    valid = true; //isStaticLibrary();

But it needs to be checked before using it in master.

fixed in master


feat(es/minifier): Minify "undefined" == typeof x to "u" < typeof x

Description:
The only thing I'm worried about is IE returning "unknown" for some objects.
https://stackoverflow.com/questions/10982739/typeof-returning-unknown-in-ie
BREAKING CHANGE:

Related issue (if exists):

🤣 I never heard about unknown

I supposed it woulld be faster, since it's a common tricks in c/c++.
In 'undefined' > 'u', only comparing the first letter is sufficient to obtain the result.
In 'undefined' == 'undefined', typically the KMP algorithm needs to be executed, but even with the optimal algorithm, it still requires comparing 9 times to obtain the result.
If typeof x can be converted to compare directly with x and void 0, it should be faster, just like what we did in crates/swc_ecma_minifier/src/compress/optimize/bools.rs.
However, in actual speed tests, the results may fluctuate. So I speculate that its impact on speed is not significant.

perf.link

I'll profile runtime performance later today

@magic-akari it makes no sense to test the performance of something that can be removed as a dead code since it hasn't observable side effects.


Typing anything in editor generates error

@Vishv07, thanks for pointing this out, I'll add a patch soon. Generally, it has to do with new text creation workflow that is not there yet. Meanwhile, it is possible to open texts from files. You can try this one, for example - texts_selection.txt.

Works for file 👍


react-native run-android - completes and stops

Environment
React Native Environment Info:
System:
OS: macOS 10.14.2
CPU: (4) x64 Intel(R) Core(TM) i5-5350U CPU @ 1.80GHz
Memory: 19.61 MB / 8.00 GB
Shell: 3.2.57 - /bin/bash
Binaries:
Node: 11.6.0 - /usr/local/bin/node
Yarn: 1.12.3 - /usr/local/bin/yarn
npm: 6.5.0-next.0 - /usr/local/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
SDKs:
iOS SDK:
Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
Android SDK:
API Levels: 27, 28
Build Tools: 27.0.3, 28.0.3
System Images: android-27 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom
IDEs:
Android Studio: 3.2 AI-181.55<IP_ADDRESS>56338
Xcode: 10.1/10B61 - /usr/bin/xcodebuild
npmPackages:
react: 16.6.3 => 16.6.3
react-native: 0.57.8 => 0.57.8
npmGlobalPackages:
react-native-cli: 2.0.1
Description

Followed all the steps in getting started: https://facebook.github.io/react-native/docs/getting-started
Created a new project: react-native init AwesomeProject
Created a Virtual Device on Android Studio and started the app: react-native run-android
App Server runs and Completes. It spews Emulator terminal which start building the app on Emulator.
Emulator shows the welcome page successfully.
Issue is since app server Completes there is no good way to test any changes. I need to run the app server again:  react-native run-android
Why it is happening on Mac. I used the same setup on Windows and there it is working as intended. Emulator reloads for any changes in the app.

Reproducible Demo
Complete steps are specified above.
Any help will be much appreciated. I tested it on real device by USB debugging option. The same result on Android. Have not tested it yet on ios.
Output of react-native run-android:
Starting JS server...
Building and installing the app on the device (cd android && ./gradlew installDebug)...

Task :app:installDebug

Installing APK 'app-debug.apk' on 'Pixel_XL_API_27(AVD) - 8.1.0' for app:debug
Installed on 1 device.
BUILD SUCCESSFUL in 37s
27 actionable tasks: 27 executed
Running /Users/sushilbansal/Library/Android/sdk/platform-tools/adb -s emulator-5554 reverse tcp:8081 tcp:8081
Starting the app on emulator-5554 (/Users/sushilbansal/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell am start -n com.awesomeproject/com.awesomeproject.MainActivity)...
Starting: Intent { cmp=com.awesomeproject/.MainActivity }

What was the issue?

What was the issue?

I have a similar issue that when i run react-native run-android completed, My App has stopped

I don't know why but i will run js server in another terminal and install the app.

I don't know why but i will run js server in another terminal and install the app.

I'm trying fix it longtime but i can't find out


Feature request: Don't download Java when running the NPM artifact

Even if I run with JAVA_HOME pointed at a java 11 installation, it still tries to download one on the fly.
$ composeReport2Html -app STAR -overallStatsReport samples/star/build/compose_metrics/star_release-module.json -detailedStatsMetrics samples/star/build/compose_metrics/star_release-composables.csv -composableMetrics samples/star/build/compose_metrics/star_release-composables.txt -classMetrics samples/star/build/compose_metrics/star_release-classes.txt -o tmp
Downloading java runtime environment for version 11

@ZacSweers I'm not sure but I guess this issue is related to jdeploy. I'm also looking if how can we remove dependency on jdeploy and publish CLI independent of it.

Yeah it's due to jdeploy

Need help here regarding publishing this CLI without jdeploy!

any update?

For CLI, I'm still struggling to find alternatives to jdeploy. But the good news is I'm working on the gradle plugin, so that we don't need to use CLI

@ZacSweers I'm not sure but I guess this issue is related to jdeploy. I'm also looking if how can we remove dependency on jdeploy and publish CLI independent of it.

@ZacSweers exact issue isn't solved yet. But we have published a gradle plugin which does the report generation work automatically. This can help you to be independent from jdeploy.
Gradle plugin usage: https://patilshreyas.github.io/compose-report-to-html/use/using-gradle-plugin/


Test Method does not map with the test case in AzureDevOps

@JanuszNowak
I have created a small test method in Visual studio code which looks like this

I have used [TestFixture] instead of [TestClass] as it does not support [TestClass] and used [Test] rather than [TestMethod] as [TestMethod] is not supported
I have created a test case in the AzureDevops that has the test id as 185 and it cli shows success for mapping

but when I check in the AzureDevops, it is not mapped

@nimishbhuta good dot now .net 7 is working, instead --action List run or after run with --action Associate.
List - is scanning assembly for test to assign
Associate - is doing association

@JanuszNowak. I have now used "Associate" ,it ran successfully but did not map.

I see a similar problem. I have 3 test cases. One I assigned automation via Visual Studio. All I assigned through this CLI tool. The only difference between the VS assigned test automation and the other two updated via this CLI is the VS assigned has NeedUpdateInsert value set to False, while the two CLI assigned ones have NeedUpdateInsert value set to True. Checked documentation and code in this repo, but it's not clear what NeedUpdateInsert is used for or how to change its setting. The two test cases assigned automation using this CLI do not show any associated automation on their work item Automation tabs, but the one done with VS does.

@skeeler @nimishbhuta

first run is scanning assembly and shows that it found test automation in assembly (on image found automation for test case 185)
Second step is connecting to ado/tfs and is checking if this Assembly test method is connected to test 185, NeedUpdateInsert - True means that test automation is not assigned or that assigned automation is not the same like in assembly and need update
last step (Test association status) is updating/inserting automation and in status code returning StatusCode  OK-200 if it was able to update
@skeeler, @nimishbhuta what kind of project temple you are using for your project "Scrum/....."  ? This is ADO/TFS/ADO Server with version ?
To execute full tree steps --action Associate need to be run "Example --action "Associate" janono.ado.testcase.associate.cli.exe --authMethod PAT --authValue [yours PAT] --path D:\ExampleTestProject\bin\Release\net6.0\ExampleTestProject.dll --action Associate"


Detect if tests are running on test engineer's machine.

Would be a huge improvement! Looking forward to see this volkswagen 2.0

Any hints on how we would be able to achieve this?

Idea: Use presence of certain programs in $PATH as a heuristic. Test engineers are more likely to have afl-fuzz, valgrind or other testing tools installed.

There are several possible tests: check for an interactive shell, the presence of a window manager, matching running processes like browsers and so on.

My ideal: Check if the test machine is well configured.
Or we can check many other items, and score them with certain point. If the total score is above 60 , we dare to say it's a machine used by human. :laughing:

@archersmind actually a really interesting problem to solve :smiley:

@watson  Yes, Of course. Let's gather more opinions.
Just have the thought, it's going to be the Turing test problem ? Even more fun :computer:

The check can be as simple as checking for stdin, or $PS1 or a custom env var or tty -s which ends with no error on interactive shells. Best of all, these will work even on SSH, but definitely not in a batch command (script file).


listenerRegionIsChanging not being called.

listenerRegionIsChanging doesnt get called when I swipe around the map I can keep checking every second controller.bounds to do equivelent but that does get a bit annoying.

i will check it

can you provide small sample ?


No data in LAW

I have followed the install guide and doublechecked everything and I am still not seeing any of the custom logs. What else can I do to troubleshoot?

Hi @DeanGross144
Normally the first export from MPARR can take around 15 minutes to show the data in Logs Analytics, but you can see when you execute MPARR_Collector that every time that some information is sent to Logs Analytics appears at bottom the amount of that exported and the status, check the image below.

Thanks, the main logs finally showed up, but I am still not seeing the logs for labeldata, SITs, Azure AD Users, Roles, Domains and products. I ran all of those scripts without any errors. How can I troubleshoot ?

MPARR-SITData and MPARR-LabelData requires elevate privileges, you need to check if that was added or execute manually, in all the cases at the final of the execution you will see a message about how many records was sent to Logs Analytics.
Please check that first, and if you can share the images that can be helpful.

Appears that is working, check in 15 minutes your logs in Logs Analytics, you need to remember that Logs Analytics is not a traditional database, is only a kind of store, and all the records depends on datetime, in that order of ideas, any time that you make a query you need to have on mind the last time that the data was ingested.
In that order of ideas, in this GitHub site you can look for the queries used to get the data and you´ll see that the queries uses TimeGenerated the last 30 days to request the data.

it has been a few hours and none of the data from the manually run scripts has shown up

had you force to refresh the page? happens that sometimes you need to force to refresh the page to see the changes, the page is not updated automatically sometimes.

thanks- MUCHAS GRACIAS, I had switched from Logs to tables and back which did not refresh the list, doing a Ctrl + R got the info.

Enjoy MPARR :)

looking forward to testing it out and then showing my clients


Unable to run on older operating systems?

Windows XP 32bit.
I was able to install the program fully, associate a soundfont, and start Audiostation. It works until it throws some errors about WASAPI (Which only started appearing in Vista)
Is there no way to use this on a Windows XP system?

Thanks for you bug report.
I will take a look at this issue, maybe I can implement a fix in the next version

I found the problem, and why the application does not work on Windows XP.
In de latest released version (2.2) of the program this issue has not been fixed yet, I will release a new version (2.3) soon, in that version this issue will be fixed.

Hi, waiting for the 2.3 version!
maybe possible run it on Windows 98 ?

Hurrah! Thank you.
It throws two warnings on startup but otherwise works fine. Playing some mp3 files great.
The procedure entry point _except_handler4_common could not be located in the dynamic link library msvcrt.dll


Add as collaborator

Can you add me as collaborator so I can edit the project cards too?

I just sent the invite.


Support for Dead-for-now (DFN) code splitting

Does TeaVM support Dead-for-now (DFN) code splitting like GWT?
GWT.runAsync(new RunAsyncCallback() {
      public void onFailure(Throwable caught) {
         Window.alert("Code download failed");
      }

      public void onSuccess() {
         Window.alert("Hello, AJAX");
      }
});

Link to GWT docs on DFN code splitting:
http://www.gwtproject.org/doc/latest/DevGuideCodeSplitting.html
If not, do you consider such feature being implemented?
It suppose to precompile chunks of javascript code that will be loaded once the piece of code that marked as split has been reached.
Thus loading less JS code initially - and the user is gradually loading more JS code.
In big enterprise applications this is very important, since you don't know if the user will reach some parts of the application - and sometimes it can be a very large code base.
I think from debugger standpoint there's no problem, because you generate mapping for all the code - not regarding if it is being split or not.

Sure, it's possible to implement code splitting and it's planned. But there's no ETA, since there aren't users who really want it right now. Please, describe your use case. Do you have any large application you successfully ported to TeaVM? How large is the generated JS file? Did you try to minify and gzip it? How large it becomes after that? How much time does it take to load such application on desktop and mobile platforms?

Hi konsoletyper,
I just looked for code-splitting with TeaVM (WASM), and as you are asking for users, here I am.
Currently I have an GWT WebApp (https://m.brettspielwelt.de/) which dynamically loads the (source)code of the different games you can play there (only loading when used). Currently we have 80 different games, and I think even if every game compiles to just about 20-50K it is good for the user not to load about 3MB of code which will not be used.
Nonetheless I will try to convert the "app" to TeaVM in the next time and have a look at the "footprint".
Great work keep it going!


Order events on dojo page by date

Page: https://coderdojopotsdam.github.io
The events should be ordered by date. Having the next event at the end is confusing.

Same cause that https://github.com/CoderDojo/community-platform/issues/1139
A quick fix could be to re-order post-generation of the card, but the real fix is to provide the proper dates from the back-end instead of having the front generating them.

Add my vote to fix this please.

This should be fixed as we reworked the events-service (https://github.com/CoderDojo/events-service)


[ZEPPELIN-2919] fix: Fallback to table when vis is not available

What is this PR for?
fallback to table when vis is not available.
The erroneous situation can happen when

user enabled helium visualization
and open the visualization in paragraph
and disabled the helium visualization
and open again the paragraph.
then you will see the empty paragraph because the specific visualization is now not available

What type of PR is it?
[Bug Fix]
What is the Jira issue?
ZEPPELIN-2919
How should this be tested?

user enabled helium visualization
and open the visualization in paragraph
and disabled the helium visualization
and open again the paragraph with refresh

Screenshots (if appropriate)
Before

After

Questions:

Does the licenses files need update? - NO
Is there breaking changes for older versions? - NO
Does this needs documentation? - NO

CI failed, but irrelevant.

https://s3.amazonaws.com/archive.travis-ci.org/jobs/273164416/log.txt?X-Amz-Expires=30&X-Amz-Date=20170908T065857Z&X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAJRYRXRSVGNKPKO5A/20170908/us-east-1/s3/aws4_request&X-Amz-SignedHeaders=host&X-Amz-Signature=10a7491238b64313f21f40f14bf241e00fef1da11964882fda40996b34ac4b27

Results :

Failed tests:
  RemoteInterpreterServerTest.testStartStop:62 expected:<true> but was:<false>

Tests run: 58, Failures: 1, Errors: 0, Skipped: 0

LGTM!

Thanks for the review. Merge if no more discussion.


Translate tasks/administer-cluster/kubeadm/adding-windows-nodes.md into Korean

This is a Feature Request

What would you like to be added

Translate tasks/administer-cluster/kubeadm/adding-windows-nodes.md into Korean.
Why is this needed
There is no document of tasks/administer-cluster/kubeadm/adding-windows-nodes.md which translated into Korean.
Comments

Page to translate: https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/adding-windows-nodes/
/language ko

/assign


Null Reference Exception in PoshGitVcsPrompt

System Details

posh-git version/path: <IP_ADDRESS>  C:\tools\poshgit\dahlbyk-posh-git-9bda399\src
PowerShell version: 5.1.17763.771
git version 2.23.0.windows.1
OS: Microsoft Windows NT 10.0.17763.0

Issue Description
When I got to a repository that isn't clean on status, I'm getting a null ref exception
`
VERBOSE: 00092:Getting status
VERBOSE: 00195:Parsing status
VERBOSE: 00205:Status: ## master...origin/master
VERBOSE: 00219:Status: ?? Feature.MySQL.UnitTests/app.config
[Error: You cannot call a method on a null-valued expression.
VERBOSE: PoshGitVcsPrompt error details:
PSMessageDetails      :
Exception             : System.Management.Automation.RuntimeException: You cannot call a method on a null-valued expression.
at System.Management.Automation.ExceptionHandlingOps.CheckActionPreference(FunctionContext funcContext, Exception exception)
at System.Management.Automation.Interpreter.ActionCallInstruction2.Run(InterpretedFrame frame)                            at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame)                            at System.Management.Automation.Interpreter.EnterTryCatchFinallyInstruction.Run(InterpretedFrame frame) TargetObject          : CategoryInfo          : InvalidOperation: (:) [], RuntimeException FullyQualifiedErrorId : InvokeMethodOnNull ErrorDetails          : InvocationInfo        : System.Management.Automation.InvocationInfo ScriptStackTrace      : at Get-GitStatus, C:\tools\poshgit\dahlbyk-posh-git-9bda399\src\GitUtils.ps1: line 256                         at <ScriptBlock>, C:\tools\poshgit\dahlbyk-posh-git-9bda399\src\GitPrompt.ps1: line 330                         at <ScriptBlock>, C:\tools\poshgit\dahlbyk-posh-git-9bda399\src\GitPrompt.ps1: line 324                         at Global:Write-VcsStatus, C:\tools\poshgit\dahlbyk-posh-git-9bda399\src\GitPrompt.ps1: line 324                         at <ScriptBlock>, C:\tools\poshgit\dahlbyk-posh-git-9bda399\src\posh-git.psm1: line 56                         at <ScriptBlock>, <No file>: line 1 PipelineIterationInfo : {}
I haven't been able to track down what is exactly null at this point, the $filesAdded list exists, the $matches is filled in, and the $matches['path1] expression seems to have data.  Not really familiar with writing powershell scripts though.

As an additional note, this doesn't occur in the ISE environment and does not occur in the PowerShellCore 6.0 environment.

You are right in that the only way I can see getting that error is if $filesAdded were null but I'm not sure how that could be.  What does this return in your Windows PowerShell 5 console where you see this error:
$null -eq (New-Object System.Collections.Generic.List[string])

Is this still an issue?


Actions (Templates) Download links expired

https://github.com/GodotExplorer/ECMAScript/actions/runs/418803638

@Geequlim Can you release templates in Releases? Or at least have some doc about how to compile templates.

All download links expired, please we need compile instructions so maybe we help fixing the problems..
I am compiling 3.2.3-stable without tools on Ubuntu 16.04.3, 64bits and getting:
ImportError: No module named quickjs.builtin_binding_generator:
...
File "/home/godot-3.2.3-stableJS/modules/ECMAScript/SCsub", line 32: import quickjs.builtin_binding_generator
Any help please?

Does anyone have the export templates for HTML5?I am getting the following error:
modules\ECMAScript\quickjs\quickjs\version:1:1: error: expected unqualified-id

sorry not find time to update this project.
just follow the compile docs of official compilation.
The only difference is msvc is not supported yet.

Templates are uploaded to releases


docs: update the WeChat group QR code

Summary

docs: update the WeChat group QR code
docs: adding Chinese version link at Quick Start document

/cc @IronCore864 Please take a review at this pr.


Question SQL

Hi! I am a bit lost on what can be done through this line (using V8 engine!):
engine.AddHostObject("xHost", new ExtendedHostFunctions());
What .net types can I instantiate with this? Just system types? When I use
xHost.type('System.Collections.Generic.List") or xHost.type('System.DayOfWeek') it works just fine, but when I want to do more complex things, like open a sql connection using
xHost.type('System.Data.SqlClient')
I get "Could not find a specific type or generic type definition for 'System.Data.SqlClient'".
So what are the limitations in the use of ExtendedHostFunctions. Is there a way to make those things available in ClearScript without explicitly adding them with AddHostType ?
Thanks!

Hi @flat-eric147,

What .net types can I instantiate with this? Just system types?

The ExtendedHostFunctions.type(String, Object[]) method is based on Type.GetType(String), but because it's limited to namespace-qualified type names, it can only import types from core libraries such as mscorlib or System.Private.CoreLib.

Is there a way to make those things available in ClearScript without explicitly adding them with AddHostType ?

The host can expose any algorithm to the script engine, including algorithms for importing types. Here's a trivial example:
engine.Script.getType = new Func<string, object>(name => {
    switch (name) {
        case "System.Data.SqlClient.SqlConnection":
            return typeof(SqlConnection).ToHostType();
        // ...
    }
    return null;
});

engine.Execute(@"
    var SqlConnection = getType('System.Data.SqlClient.SqlConnection');
    var connection = new SqlConnection();
    // ...
");

Of course, if you really wish to give scripts the ability to import arbitrary types by name, a simple switch statement is probably impractical, but hopefully it gives you an idea of what's possible.
Good luck!

Hi @flat-eric147,

What .net types can I instantiate with this? Just system types?

The ExtendedHostFunctions.type(String, Object[]) method is based on Type.GetType(String), but because it's limited to namespace-qualified type names, it can only import types from core libraries such as mscorlib or System.Private.CoreLib.

Is there a way to make those things available in ClearScript without explicitly adding them with AddHostType ?

The host can expose any algorithm to the script engine, including algorithms for importing types. Here's a trivial example:
engine.Script.getType = new Func<string, object>(name => {
    switch (name) {
        case "System.Data.SqlClient.SqlConnection":
            return typeof(SqlConnection).ToHostType();
        // ...
    }
    return null;
});

engine.Execute(@"
    var SqlConnection = getType('System.Data.SqlClient.SqlConnection');
    var connection = new SqlConnection();
    // ...
");

Of course, if you really wish to give scripts the ability to import arbitrary types by name, a simple switch statement is probably impractical, but hopefully it gives you an idea of what's possible.
Good luck!

Brilliant hint! So I added to your sample an assembly parameter, a potential user can lookup type and assembly and with that, use getType(sting  assembly, string type).
The JavaScript code could look like:
var connetionString;
var SqlConnection = getType('System.Data.SqlClient', 'System.Data.SqlClient.SqlConnection');
connetionString = "Data Source=WIN-50GP30FGO75;Initial Catalog=Demodb;User ID=sa;Password=demol23";
cnn = new SqlConnection(connetionString);
cnn.Open();
MessageBox.Show("Connection Open  !");
cnn.Close();
(sample taken and adapted from https://www.guru99.com/c-sharp-access-database.html)
I just wonder if the respective assembly/dll needs to be somehow linked to the application or if it's sufficient to be present on the target system. If no explicit linking is necessary, this means we have almost the same power as VBScript's "CreateObject" (?)
Thanks for your valuable input!

Brilliant hint! So I added to your sample an assembly parameter, a potential user can lookup type and assembly and with that, use getType(sting  assembly, string type).
The JavaScript code could look like:
var connetionString;
var SqlConnection = getType('System.Data.SqlClient', 'System.Data.SqlClient.SqlConnection');
connetionString = "Data Source=WIN-50GP30FGO75;Initial Catalog=Demodb;User ID=sa;Password=demol23";
cnn = new SqlConnection(connetionString);
cnn.Open();
MessageBox.Show("Connection Open  !");
cnn.Close();
(sample taken and adapted from https://www.guru99.com/c-sharp-access-database.html)
I just wonder if the respective assembly/dll needs to be somehow linked to the application or if it's sufficient to be present on the target system. If no explicit linking is necessary, this means we have almost the same power as VBScript's "CreateObject" (?)
Thanks for your valuable input!

Hi @flat-eric147,

So I added to your sample an assembly parameter, a potential user can lookup type and assembly and with that, use getType(sting assembly, string type).

ClearScript provides ExtendedHostFunctions.type(String, String, Object[]), which takes an assembly name. This method also supports generic types.

I just wonder if the respective assembly/dll needs to be somehow linked to the application or if it's sufficient to be present on the target system.

That depends. If the assembly isn't already in memory, the CLR must be able to locate it on the local system, so unless it's installed as part of the CLR, it must be deployed with your application.
Note that the set of preinstalled assemblies, the assembly names, and the rules by which assemblies are located depend on the CLR you're using. For example, SqlConnection is in the System.Data assembly preinstalled with .NET Framework, but the same class on .NET Core is in System.Data.SqlClient and must be pulled in via a package reference.
Cheers!

Hi @flat-eric147,

So I added to your sample an assembly parameter, a potential user can lookup type and assembly and with that, use getType(sting assembly, string type).

ClearScript provides ExtendedHostFunctions.type(String, String, Object[]), which takes an assembly name. This method also supports generic types.

I just wonder if the respective assembly/dll needs to be somehow linked to the application or if it's sufficient to be present on the target system.

That depends. If the assembly isn't already in memory, the CLR must be able to locate it on the local system, so unless it's installed as part of the CLR, it must be deployed with your application.
Note that the set of preinstalled assemblies, the assembly names, and the rules by which assemblies are located depend on the CLR you're using. For example, SqlConnection is in the System.Data assembly preinstalled with .NET Framework, but the same class on .NET Core is in System.Data.SqlClient and must be pulled in via a package reference.
Cheers!

Thank you very much, it's clear to me now.

Thank you very much, it's clear to me now.


Fix missed references to removed helper functions

Hey
The previous pr https://github.com/mpociot/laravel-apidoc-generator/pull/572 missed a couple of references to the removed helper functions.
The fix in ResponseCallStrategy allowed it to generate docs in my Laravel 6 project.

Codecov Report

Merging #576 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master     #576   +/-   ##
=========================================
  Coverage     90.42%   90.42%
  Complexity      299      299
=========================================
  Files            14       14
  Lines           710      710
=========================================
  Hits            642      642
  Misses           68       68

Impacted Files
Coverage Δ
Complexity Δ

.../Tools/ResponseStrategies/ResponseCallStrategy.php
87.6% <100%> (ø)
41 <0> (ø)
:arrow_down:

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 741f6d1...c53c1c9. Read the comment docs.


Bump version for CSRF validation at Asgardeo website Google One Tap

Enabling CSRF validation at Asgardeo website for Google One tap

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/3563622730

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/3563622730
Status: success


Support $XDG_RUNTIME_DIR

$XDG_RUNTIME_DIR defines the base directory relative to which user-specific non-essential runtime files and other file objects (such as sockets, named pipes, ...) should be stored. The directory MUST be owned by the user, and he MUST be the only one having read and write access to it. Its Unix access mode MUST be 0700.

The lifetime of the directory MUST be bound to the user being logged in. It MUST be created when the user first logs in and if the user fully logs out the directory MUST be removed. If the user logs in more than once he should get pointed to the same directory, and it is mandatory that the directory continues to exist from his first login to his last logout on the system, and not removed in between. Files in the directory MUST not survive reboot or a full logout/login cycle.

The directory MUST be on a local file system and not shared with any other system. The directory MUST by fully-featured by the standards of the operating system. More specifically, on Unix-like operating systems AF_UNIX sockets, symbolic links, hard links, proper permissions, file locking, sparse files, memory mapping, file change notifications, a reliable hard link count must be supported, and no restrictions on the file name character set should be imposed. Files in this directory MAY be subjected to periodic clean-up. To ensure that your files are not removed, they should have their access time timestamp modified at least once every 6 hours of monotonic time or the 'sticky' bit should be set on the file.

If $XDG_RUNTIME_DIR is not set applications should fall back to a replacement directory with similar capabilities and print a warning message. Applications should use this directory for communication and synchronization purposes and should not place larger files in it, since it might reside in runtime memory and cannot necessarily be swapped out to disk.

@dag That sounds reasonable to me.


[Session Requests] exceeded the high volume rate limit

Hello,
I am seeing this message time by time when navigating through the UI, e.g.

Open channels and open one specific channel
Switch to channel details
==> Message about connection problems with peer pops up on the right top side.
==> Refresh and the above message shows up

Wait a minute and try again
==> everything works fine

I have the feeling that this happens more often when I navigate to quickly though the UI.
What is the meaning behind this error and is there a way to work around it?

@aMinuteMan It is controlled by the following two variables that can be overridden in the config
https://github.com/hyperledger-labs/fabric-operations-console/blob/main/packages/athena/json_docs/default_settings_doc.json#L160-L161
description - https://github.com/hyperledger-labs/fabric-operations-console/blob/ba3c7156888dfd41e813ea8d1d4c864f89a77e5c/packages/athena/json_docs/json_validation/ibp_openapi_v3.yaml#L13981

Thanks @varadgit !


refactor: adapt the code to bring the two networks back to life

Related:

#32

In this PR:

[x] remove big screens
[x] remove DynamicBar
[x] adapt API for the two networks
[x] improve README on how to run the project

/build-test

/build-test

It seems that /build-test use the workflow files from the default branch, not the PR branch. So I want to merge this PR first then have run /build-test here again.

/build-test


TypeScript: Could not find a declaration file for module 'contrast-color', it implicitly has an 'any' type.

Typings would be a great addition

Anyone coming from the Angular realm, you can import this library at the top of your file using:
declare var require: any;
const ContrastColor = require('contrast-color');

Call it from your template using,
  getContrastColor(bgColor: string) {
    return ContrastColor.contrastColor({ bgColor });
  }

Looks like typings were added at some point :)
https://www.npmjs.com/package/@types/contrast-color
import ContrastColor from 'contrast-color';

return ContrastColor.contrastColor({ bgColor });