issue-3-handle-mixin-property

Issue : https://github.com/TWCable/grabbit/issues/3
WIP

Need to fix existing test case for JcrNodeWriterSpec and add a new one for writeToJcr in JcrNodeDecoratorSpec.... Still trying to figure that out..

@jbornemann : NOT ready for review.. Would discuss my changes and then mark it for review.. :)

@viveksachdeva  take a look at https://github.com/TWCable/grabbit/compare/e4d3656...issue-3-jeff
These were some of my ideas around how we could organize some of this stuff. I figured it would be easier to write code down, than attempt to explain all of it, but the jist :
1.) Trying to keep operations on data, close to where the data is organized. IMO, in general, if you see something like ProtoPropertyDecorator being passed into a method within JcrNodeDecorator, it probably means that whatever operations are being performed on ProtoPropertyDecorator should happen within ProtoPropertyDecorator.
2.) Since we do have to potentially search up the tree to find the root versionable node; For performance, it probably makes sense to have this be the exceptional case, rather than the norm. Scaling up the tree for deeply-nested, unversionable nodes would be expensive.
3.) If we find a versionable node, and check it out, we should check it back in when we are done with it.

@viveksachdeva .. playing catchup here.. but can you provide steps to reproduce the case that breaks without this change (of explicitly checking out the node and writing mixin)?

Nvm @viveksachdeva .. @jbornemann pointed me to https://github.com/TWCable/grabbit/pull/134#issuecomment-236684066 . Just trying to reproduce the original issue.

@jbornemann : When refactoring JcrProcessor, per the discussion that we had, ProtoNodeDdecorator should know how to write itself to Jcr node and JcrNodeDecorator would have JcrNode wrappers and ability to build proto node from it but as per https://github.com/TWCable/grabbit/compare/e4d3656...issue-3-jeff#diff-0e12e515926b4dbd7a7b92edebc8f7ecR66, we are delegating creating jcr node from proto to JcrNodeDecorator..

@jbornemann : Regarding 1) writeToNode is already there in ProtoPropertyDecorator.. is there something else that you are referring to in this comment.
2,3) If for checkout, it traverses till the root node, it would happen only once. From the next time, isCheckedOut() would return true and code for findingVersionableAncestor would not run again... IMO we do a check early for checkout as if that fails, we would anyway be getting the exception and then we would checkout the node and run same code again..
Moreover if we do it in exception block https://github.com/TWCable/grabbit/compare/e4d3656...issue-3-jeff#diff-412149750a01ec3f3187ff7ce351cd17R65, we would need to do it for mixins and other properties separately.. So I think this check should be done before we start to write anything in the node..
And related to node checkin, I think checkin should not be needed as we will be creating version of the new content may not be required... We have anyway modified what was read only, why set it back to same status..

1.) Yes. It's not about writeToNode being in ProtoNodeDecorator. ProtoNodeDecorator.writeToJcr should be there to gain a handle on a JcrNodeDecorator. What it should not do is have details on how JcrNodeDecorator is created....that should happen within JcrNodeDecorator. We are trying to create highly cohesive, SRP objects. Merge my commit, and it will be more clear - my commit reflects the direction I think we should be going in. It will be even more clear why this design direction is a good idea as my PR for users/groups goes in.
2/3) If we checkout a node, we should check it back in. Leaving all of the client's doors open is not acceptable. Furthermore, "If for checkout, it traverses till the root node, it would happen only once" - This is not true, because my previous point, and second we can not have any assumptions on the state of which a node is in - for all we know, the application is changing it all of the time. Every node we sync will be scaling up to either the root (if it is unversionable), or to a common versionable root. If we have to perform this behavior, again, lets only do it when we have to.

"ProtoNodeDdecorator should know how to write itself to Jcr node and JcrNodeDecorator would have JcrNode wrappers and ability to build proto node from it but as per e4d3656...issue-3-jeff#diff-0e12e515926b4dbd7a7b92edebc8f7ecR66, we are delegating creating jcr node from proto to JcrNodeDecorator.."
What I mean, is from the client perspective, ProtoNodeDecorator should have methods that allow me, the client, to get a JcrNodeDecorator from a ProtoNodeDecorator. As I client, I don't care how it happens. As the writer of ProtoNodeDecorator, I care how it happens, for the same reasons.
Think about it - if the details of building a JcrNodeDecorator change, where do those changes need to occur in the application?  If I have ProtoNodeDecorator (or any other client! JcrProcessor even) setting state, checking for state (e.g hasMixins) in order to build a JcrNodeDecorator correctly; than that means whenever representation of state changes for JcrNodeDecorator, I have to change all of the objects that act on that state. If JcrNodeDecorator builds itself by setting, and checking internal state; I only have to change JcrNodeDecorator.
The why we do this is we only want to give an object one reason to change, and therefore one reason to break - itself. We don't want objects breaking because other objects have changed, or have broken.  This is the Single Responsibility Principle.

Most of this taken care by @jbornemann in #65 .. Will create a new one in case something is left..


[SPARK-11971] Start py4j callback server for Java Gateway

See the thread 'pyspark does not seem to start py4j callback server'
This PR starts py4j callback server for Java Gateway
@davies

Test build #46647 has started for PR 9950 at commit 46c6db3.

Test build #46647 has finished for PR 9950 at commit 46c6db3.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

Test build #46651 has started for PR 9950 at commit f8ce99c.

Test build #46651 has finished for PR 9950 at commit f8ce99c.

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

Merged build finished. Test FAILed.

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


Migrating form os.path and hardcoded paths to pathlib for better cross-platform support

Implemented in Commit 2936686


proj_template is not working

In below files, nmigen should be replaced by amaranth, right?
CFU-Playground/proj/proj_template/cfu_gen.py
CFU-Playground/proj/proj_template/cfu.py
Whether nmigen_cfu should be replaced by amaranth or still use this module?

Hello @ggangliu ,
Is there an actual error so that proj_template cannot be built?   That would be surprising since we haven't seen failures in our GitHub Actions tests.
Yes, we will need to update the nMigen submodule and our code that references it at some point.

Hi @tcal-x,
I forgot attach error log. But I made below changes in cfu.py and cfu_gen.py. The issue was resolved.
proj/proj_template/cfu.py
# from nmigen import *
from  amaranth import *

proj/proj_template/cfu_gen.py
# from nmigen import *
from  amaranth import *
# from nmigen.back import rtlil, verilog
from amaranth.back import rtlil, verilog

Hi @tcal-x,
I forgot attach error log. But I made below changes in cfu.py and cfu_gen.py. The issue was resolved.
proj/proj_template/cfu.py
# from nmigen import *
from  amaranth import *

proj/proj_template/cfu_gen.py
# from nmigen import *
from  amaranth import *
# from nmigen.back import rtlil, verilog
from amaranth.back import rtlil, verilog

If needed, you can assign me to submit this fix. I can try to study the flow of contribution.

Hi everyone! I have the same errors during "making" proj_template. I did all you mentioned here above and it's still doesn't work. So is there anything else that should be switched due to switching to Amaranth?

Hi @spicker23  -- you didn't have any problem until you started making changes to switch over to Amaranth, right?

Hi @spicker23  -- you didn't have any problem until you started making changes to switch over to Amaranth, right?

Probably there was a problem with paths, because after I added it in shell everything worked as it should. It costed two days to find out.


Fix multicore settings for PromptReco

Fixes #7408
My goal with the latest changes in the Multicore settings is that a Multicore key/value at the top level should be propagated to all task types except LogCollect/Cleanup/Merge/Harvesting.
Since alcaSkim has to be single core, then we provide a new task value instead of using the spec multicore attribute.
Dirk, as a bonus you got a unit test (testPromptRecoWithSkims) fixed :-) It was looking for a task that isn't created in the spec.
@hufnagel @ticoann please review. I still want to write similar unit tests to the other specs though

The only WMCore specs without Memory/Multicore unit tests now are StoreResults and DQMHarvest. I noticed we don't set cores nor memoryRequirements for Harvest tasks. I'm not sure we'll ever have multicore harvesting jobs and or requiring more than the default amount of memory (1000MB AFAIK).
@ticoann @hufnagel @ericvaandering any insights here?

@ticoann please review and merge. I didn't run too many tests, but basic stuff seems Ok.

Looks good to me too.


Structural error at [property] should be equal to one of the allowed values allowedValues: array, boolean, integer, number, object, string

Hello, I was navigating through dndbeyond.com to record API requests and there are many structural errors when importing the resulting JSON into Swagger Editor similar to below:
Structural error at paths./font_picker.post.requestBody.content.application/json.schema.properties.organization_id.type
should be equal to one of the allowed values
allowedValues: array, boolean, integer, number, object, string

This particular request  looks like:
curl 'https://echidna.wirewax.com/font_picker' \
  -H 'accept: */*' \
  -H 'accept-language: en-US,en;q=0.9' \
  -H 'cache-control: no-cache' \
  -H 'content-type: application/json' \
  -H 'origin: https://player.vimeo.com' \
  -H 'pragma: no-cache' \
  -H 'referer: https://player.vimeo.com/' \
  -H 'sec-ch-ua: "Google Chrome";v="123", "Not:A-Brand";v="8", "Chromium";v="123"' \
  -H 'sec-ch-ua-mobile: ?0' \
  -H 'sec-ch-ua-platform: "macOS"' \
  -H 'sec-fetch-dest: empty' \
  -H 'sec-fetch-mode: cors' \
  -H 'sec-fetch-site: cross-site' \
  -H 'user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36' \
  --data-raw '{"environment":"player","organization_id":null}'

The resulting OpenAPI request body looks like:
requestBody:
       content:
         application/json:
           schema:
             type: object
             properties:
               environment:
                 type: string
               organization_id:
                 type: 'null'
             required:
               - environment
               - organization_id

As you can see null is not the same as 'null'. When submitting a value of null it's probably not possible to determine what the type signature should be. But rather than a string of 'null' it should probably just be an empty string to or one of the other allowed values to adhere to the spec.

Hi @jziggas, have you tried importing the spec into https://editor-next.swagger.io/ ? This error may be because the editor you're using isn't compatible with the latest OpenAPI 3.1 standard.
There used to be a nullable keyword. This has now changed, I believe type: 'null' is correct here https://stackoverflow.com/questions/48111459/how-to-define-a-property-that-can-be-string-or-null-in-openapi-swagger.

Oh interesting, thank you. I was not aware of that URL and was using the standard https://editor.swagger.io/ one. There is only one error when pasting into that next gen editor:

@jziggas Looks like the whitespace in the key names of securitySchemes is an issue. Fixed in v1.5.2, thanks for reporting.


Checkreturnvalue wrong on withFailMessage

Summary
Using the error prone compiler results in a failed build because withFailMessage does specify checkreturnvalue annotation, but that does not make sense.
Example
assertThat(result4).hasSize(1).withFailMessage("searched for: {}", searchStr);

No return value needed here.

withFailMessage has to be used before the assertion, otherwise it has not effect.

Ok that was not obvious for me, there is no example in the documentation.
So the correct assertion would be:
assertThat(result4).withFailMessage("searched for: {}", searchStr).hasSize(1);

Javadoc and website updated.
Thanks for reporting! 👍


Updated README with better R example and inferSchema flag

closes #79

Current coverage is 83.33%

Merging #103 into master will not effect coverage as of e1a661c

Coverage Diff
@@            master    #103   diff @@
======================================
  Files           10      10
  Stmts          384     384
  Branches       115     115
  Methods          0       0
======================================
  Hit            320     320
  Partial          0       0
  Missed          64      64

Powered by Codecov


Warning when mounting run

https://github.com/binpash/try/blob/3d5af35bc2d154bd20f18d80d64c10117fdb38d5/try#L51
mount: /tmp/tmp.1MYxsHblfI/temproot/run: filesystem was mounted, but any subsequent operation failed: Unknown error 5005.

Is this because /run is a tmpfs?


Build docker container failed

Build docker container failed with the file "docker\Dockerfile.demo_android"
please add below in line 60:
mv incubator-tvm tvm && \

PR filed #4363

Thanks @cszhz  for reporting the issue


Rabbitmq TransportURL CR support

Add logic to generate transport-url secrets
This implements logic to generate a k8s secret containing a transport-url for each TransportURL CR. The current implementation uses the default rabbitmq username, password, host information (similar to OSP today).

@dprince gave me the reasons why we are doing it this way.
Capturing them here for completeness:

One side is that we want to try to make this operator transparent in regards to the other operators' section of the spec, so we don't want the openstack-operator having to check the cinder-spec section to look for the rabbitmq information.

The other side is that, for security reasons, we don't want to give permissions to all the different operators to access the rabbitmqclusters secrets, so we want to keep it in the openstack-operator.


compiled String return type does not match actual return type

Describe the bug
The compiled typescript API query return type does not match the actual data returned for the echo lambda example.
Amplify CLI Version
4.17.2
To Reproduce
On an angular 8 amplify project
Follow this example: https://aws-amplify.github.io/docs/cli-toolchain/graphql#usage-1
amplify mock to create the API.service.ts
compare APIService.Echo return type (string) to actual data type returned ({echo: string})
Expected behavior
The generated APIService method return type should match the actual return type.
Desktop (please complete the following information):

OS: Mac
Node Version. v13.12.0

Hi @jnathanh
Is the APIservice query returning null type with an error in GraphiQL explorer?
Try the above function handler with a JSON parse statement to see if that works i.e.
exports.handler = function (event, context) {
  context.done(null, JSON.parse(event).arguments.msg);
};

Hi @jnathanh
Is the APIservice query returning null type with an error in GraphiQL explorer?
Try the above function handler with a JSON parse statement to see if that works i.e.
exports.handler = function (event, context) {
  context.done(null, JSON.parse(event).arguments.msg);
};

@akshbhu, I didn't observe any issue with a null type. The problem is the js API codegen. For the schema
type Query {
  echo(msg: String): String @function(name: "echofunction-${env}")
}

with return type string, and the lambda (the default created by amplify add function)
exports.handler = function (event, context) {
  context.done(null, event.arguments.msg);
};

which also returns a string, and the codegen API function Echo looks like
  async Echo(msg?: string): Promise<string | null> {
    const statement = `query Echo($msg: String) {
        echo(msg: $msg)
      }`;
    const gqlAPIServiceArguments: any = {};
    if (msg) {
      gqlAPIServiceArguments.msg = msg;
    }
    const response = (await API.graphql(
      graphqlOperation(statement, gqlAPIServiceArguments)
    )) as any;
    return <string | null>response.data;
  }

but the actual returned object shape is
{
   echo: string
}
```.
So, there is no way to correctly reference the returned object without triggering a typescript compilation error. Either the shape of the returned object should be different, or the return type of the generated `Echo` function should be `Promise<{echo:string}|null>`.


Fix FormatException reading .bcfg file in international.

Fixed "Input string was not in correct format" error reading .bcfg file on systems with different decimal separator (reported by Lutho).

Looks good. Have you tried any other testing with European language settings? e.g. round-tripping the .bcfg file to make sure that it is also written correctly?

Good point. We were also using the wrong CultureInfo when writing Decimal values to XML. Fixed.


[5.9][Macros] Plugin search options group

Cherry-pick #66650 into release/5.9

https://github.com/apple/llvm-project/pull/6999
@swift-ci Please test


Arrow functions don't work in super calls

Typescript version: 2.1.4
Playground: https://www.typescriptlang.org/play/index.html#src=class C {
	constructor(public f%3A { ()%3A string })%20%7B%0D%0A%0D%0A%09%7D%0D%0A%0D%0A%09%0D%0A%7D%0D%0A%0D%0Aclass%20B%20extends%20C%20%7B%0D%0A%09constructor()%20%7B%0D%0A%09%09super(()%20%3D%3E%20%7B%20return%20this.g()%3B%20%7D)%0D%0A%09%7D%0D%0A%0D%0A%09private%20g()%20%7B%0D%0A%09%09return%20%22hello%22%3B%0D%0A%09%7D%0D%0A%7D%0D%0A%0D%0Aalert(new%20B().f())%3B
Source:
class C {
	constructor(public f: { (): string }) {
	}
}

class B extends C {
	constructor() {
		super(() => { return this.g(); })
	}

	private g() {
		return "hello";
	}
}

alert(new B().f());

Expected result: "Hello" is alerted.
Actual result: calling f() fails with referenceerror.
The problem is that arrow functions are bound by the use of _this. Unfortunately, when there is a super call with no other this references, Typescript skips emitting the _this variable. _this doesn't exist so when the function is called, it fails.
If somehow this is the intended behaviour, the compiler should error on this references in super calls.

I guess there should be an error, because super must be called before accessing this.
Just change constructor of B to code below and you will get error (the code is equal)
super(this.g.bind(this));

This is basically working as intended because we have no way to know that the base class constructor invokes the function immediately. If instead the base class constructor stored a reference to the closure and then invoked it later (for example, maybe this function is an event handler), no error would occur and the code is fine.

I don't understand? The base class constructor does do exactly that in my sample- it stores the reference and then it is called later.

Sorry, misread. You're right - we should have emitted var _this = this; above the super call

Just for reference, it seems to me that even if the base class invokes the callback immediately, that should be safe as long as it doesn't return a new value for this?

Fix should be in<EMAIL_ADDRESS>or later.


ignore vscode

Reasons for making this change:
Visual Studio Code now supports Unity, so need ignore the folder created by it
Links to documentation supporting these rule changes:
https://marketplace.visualstudio.com/items?itemName=VisualStudioToolsForUnity.vstuc

You don't want to do it. Dir .vscode may contain several files you definitely want to check in. E.g. settings, workspaces, tasks
If you want to ignore some particular cache file from this dir, you should write more specific rule


New "Write Oso Policies" tutorial

PR checklist:

[x] Introduction
[x] Summary
[ ] Interlanguagization
[ ] More (complex) examples

All the changes look great! I know interlanguagization is next on your list, but I would publish this as is and take that as a follow-up. I think even with python-only examples this will add A LOT of helpful information to the docs so I'd rather not wait.

If we're going to publish this without fully inter-languagizerationing it, I think we should back out the {{< lang >}} shortcodes since there's currently some dissonance between the Oso API methods all being the Python versions and the text saying, e.g., "Java" if you have Java as your selected language.


Slack Link - Error when not logged in

NAME OF PAGE/COMPONENT
Slack Link
PART OF DESIGN NOT MATCHING
Clicking through reminder link sent to slack displays an error when you're not logged in.
Honesty store bug.

Do not fix


CI and HHVM fixes

This tests on the same versions as https://github.com/guzzle/promises/pull/110 (as well as 5.4), in particular, testing on HHVM 3 properly.

🍻


[BUG] Selecting a token to swap does not preselect

Describe the bug
When selecting a token to swap from it's token detail screen, does not preselect it in the swap page. It rather selects the native gas token
To Reproduce
Steps to reproduce the behavior:

Go to Token detail screen
Click on swap
See error

Expected behavior
Preselect the wished token

Version 29

QA done.


Fix(VIM-3116): Cannot return to Normal mode in read-only file with ideaglobalmode

Implemented using a focus listener, which has a disadvantage that it fires on every focus and not just when switching focus from a different editor. I considered FileEditorManagerListener.selectionChanged, but that event doesn't fire for editors outside the main editor tab pane, such as diff previews.

The most recent master commit the PR is based off has unit test failures.

Hi, thank you for your changes. However, this option is really experimental and we've noticed a lot of other issues when it's enabled. Currently, we don't have plans to invest in this option, so I'll close the PR.
I've added a note to the code saying that this option is experimental.


remove deprecated init container syntax

Init Containers were specified using custom annotations in k8s 1.5.  In
k8s 1.6, these annotations were replaced with the
Pod.Spec.initContainers collection and the old annotations were
deprecated in 1.6 and no longer supported in 1.8. There's no reason to
clutter the docs with the old annotation syntax.

/lgtm
/approve


Orchestrator doesn't work on 4.3-beta2 (yet)

Describe the bug
Orchestrator doesn't work on 4.3-beta2 (yet)
Expected behavior
Clicking on orchestrator and opening a .os file will allow me to edit it.
Actual behavior
Clicking on orchestra tor and opening a .os file will crash.
How to Reproduce?

download godot 4.3-beta2 from godot engine org
Install orchestrator (I tried upgrading the godot-cpp and it was failing.

Godot full version
4.3-beta2 (b75f0485ba15951b87f1d9a2d8dd0fcd55e178e4)
Orchestrator version
614a6f2cde0676aa59607c5afd1fd184fc41f814
Additional information
My friend was trying to use Orchestrator on 4.3-betas and wasn't able to.
See also: https://github.com/Vahera/godot-orchestrator/pull/422

Worked on this a little https://github.com/V-Sekai-fire/godot-orchestrator/tree/4.3-beta2

Unfortunately the latest godot-cpp 4.3-beta2 introduced a CRASH_NOW macro, and this is probably what your friend saw.
The issue with this change is that it breaks backward compatibility but, moreover, breaks the expectation that working in a C++ module should be nearly identical to GDExtension.  The fact you can no longer define a Godot object on the stack in GDE but you can in the engine seems unintended.

This should be fixed on main now.


.NET 7 Isolated: Logs missing from Application Insights / Log Analytics

Hello.
We are running an isolated Function using the latest packages. This is the list of relevant packages:

Microsoft.ApplicationInsights.WorkerService v2.21.0
Microsoft.Azure.Functions.Extensions v1.1.0
Microsoft.Azure.Functions.Worker v1.20.0
Microsoft.Azure.Functions.Worker.ApplicationInsights v1.1.0
Microsoft.Azure.Functions.Worker.Sdk v1.16.1
Microsoft.Extensions.Logging v7.0.0
Microsoft.Extensions.Logging.Abstractions v7.0.1
Microsoft.Extensions.Logging.Configuration v7.0.0

Dependency injection configuration looks like this:
builder.ConfigureFunctionsWorkerDefaults();
builder.ConfigureServices(services =>
{
   services.AddApplicationInsightsTelemetryWorkerService();
   services.ConfigureFunctionsApplicationInsights();
})

The host.json looks like this:
{
  "version": "2.0",
  "logging": {
    "applicationInsights": {

      "samplingSettings": {
        "isEnabled": true,
        "excludedTypes": "Request;Exception;Trace"
      }
    },
    "logLevel": {
      "default": "Information",
      "Function": "Information",
      "Host.Aggregator": "Information",
      "Host.Results": "Information"
    }
  },
  "extensions": {
    "http": {
      "hsts": {
        "isEnabled": true
      },
      "customHeaders": {
        "X-Content-Type-Options": "nosniff"
      }
    }
  }
}

However, logs generated from my code using Logger.LogInformation do not show up. Logs generated with Logger.LogWarning do show up.
There is a lot of disperse information out there about logging in Azure Functions:

Here it is assumed that everything just works, i.e. logs will show up in App Insights.
Here there is mention of some more advanced configuration, but I couldn't figure out how it applies to my use case (which I hope is a common, simple use case). There is also a huge "Important" warning which I think applies to my use case, but I couldn't find any mention on what to do about it. See image 1 below.
Here it mentions the configuration for ILogger logs from App Insights, but the JSON schema does not match the one in host.json. In host.json there doesn't seem to be possible to include the "LogLevel" property inside "logging.applicationInsights" (see Image 2 below)

Image 1

Image 2

I couldn't find any complete examples.
Could someone please point me to an example of how to configure an Isolate Azure Function so that any Logger.LogInformation shows up in AppInsights and Log Analytics (where Logger is an ILogger obtained via Dependency Injection).
P.S. I already checked that nothing is getting sampled away:

I also checked that the log entries show up in the "Log stream" of the function on the Azure Portal.

I think you're looking for this documentation:

However, by default, the Application Insights SDK adds a logging filter that instructs the logger to capture only warnings and more severe logs. If you want to disable this behavior, remove the filter rule as part of service configuration:

var host = new HostBuilder()
    .ConfigureFunctionsWorkerDefaults()
    .ConfigureServices(services => {
        services.AddApplicationInsightsTelemetryWorkerService();
        services.ConfigureFunctionsApplicationInsights();
    })
    .ConfigureLogging(logging =>
    {
        logging.Services.Configure<LoggerFilterOptions>(options =>
        {
            LoggerFilterRule defaultRule = options.Rules.FirstOrDefault(rule => rule.ProviderName
                == "Microsoft.Extensions.Logging.ApplicationInsights.ApplicationInsightsLoggerProvider");
            if (defaultRule is not null)
            {
                options.Rules.Remove(defaultRule);
            }
        });
    })
    .Build();

host.Run();

Thank you, @gabrielweyer. This worked.
This seems like a quite fragile way of doing things, involving:

A magic string, which I replaced with rule.ProviderName == typeof(Microsoft.Extensions.Logging.ApplicationInsights.ApplicationInsightsLoggerProvider).FullName
A hidden dependency not only on the classes of a separate package that evolves independently, but also on what that other code does at runtime.

I hope the Functions team finds a better way to wrap this up without exposing their dependency on the AppInsights SDK... perhaps a settings in their WorkerOptions?

I tried above solution and still don't get anything below "warning". I debugged and noticed that there isn't a single rule (options.Rules is NULL), therefore there's nothing to be removed. Ideas?

I tried above solution and still don't get anything below "warning". I debugged and noticed that there isn't a single rule (options.Rules is NULL), therefore there's nothing to be removed. Ideas?

I had a similar issue, and once I moved the ConfigureLogging to the end of the chain, it started working, maybe check that?

This makes this issue even more ridiculous

This is so convoluted.
It's good and understandable that AppInsights adds Warning level as default because we don't want it to spam too much.
However, the fact that we need to mess with AppInsights' internal filters to be able to add our own information messages for our own categories to App Insights is just ugly.
And what would be the consequences of removing that filter? Would we start seeing more internal .net core logging information than we want? Then what is the proper way to keep all the default log levels as Warnings (as it should be) and have our own logging configured by categories?


Chinese localization update

Chinese localization update

Github merge is being wacky again but I've added the new translation manually, thanks for the work.
I will push this out in the next update.


Keras time distributed dense layer output shape incorrect

Observed Issue:
A time distributed dense layer can be imported into DL4J without exceptions. However the output shape of the time distributed layer is 2d when it should be 3d. Values look right. Might just be a matter of adding a reshape after.
Not entirely sure if keras time distributed layer is supported. According to the docs it's not..
Maybe the docs just need updating.

I can take a look and work on this.


replace all instances of "sample" with "example"matching case as needed

PR Details
change sample to example
Description
All instances of "sample" found throughout the project have been changed to "example" while preserving case
Related Issue
#520  [hammer gh-pages]: Rename 11ty tag and label from 'sample(s)' to 'example(s)'
Motivation and Context
This change was made to resolve issue #520
How Has This Been Tested
I am running the site on my development box (Linux) and walked through the site to verify all changes were made.
Types of changes

[x] Docs change / refactoring / dependency upgrade

Checklist

[x] I have read the CONTRIBUTING document.

I have read the CLA Document and I hereby sign the CLA

recheckcla


[DOCFIX] Update table docs

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6568/
Test PASSed.

@ZacBlanco some minor doc changes

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6587/
Test FAILed.

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6588/
Test FAILed.

failed alluxio.client.fs.FreeAndDeleteIntegrationTest.deleteDir

Jenkins, test this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6608/
Test FAILed.

failed alluxio.client.cli.fsadmin.command.QuorumCommandIntegrationTest.quorumRemove

Jenkins, test this please

@gpang let's just merge this since this is DOCFIX?

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6609/
Test PASSed.

alluxio-bot, merge this please

alluxio-bot, cherry-pick this to branch-2.1 please


Add functions diag, diagonal, tril, triu, fliplu, fliplr

I have rebased this and included fixes for comments on the old PR.

Thanks. Given what fixed the tests, this might be a legitimate dangling reference problem with expression shapes.

This looks really good and adds a ton of features. I have made a few comments on a first pass.

Cc @JohanMabille  any idea how we may avoid temporary sequences in the two outstanding cases?

we could also work around with the X_OLD_CLANG flag?

Maybe. Honestly I am not sure if it is a bug or expected that the variables are moved already.

I think this PR is good enough to be merged. Then we can do an "optimization" review of the code and make a PR dedicated to performance only. From my point of view, the make_xgenerator function could be refactored so it can deduce its shape from its expression parameter if no shape is provided for instance. This would avoid a copy of the shape in the flipXX and triXX functions.
And we could add function generator for the fn_impl, diag_fn, etc, to avoid long templates types in the code.

Btw. we could also implement fliplr/ud as a view with the dynamic view functionality (or in the case of tensors it's probably already possible). That would make it a bit more consistent with numpy where those functions are implemented as views.

It depends wether you want the fliplr/ud to be assignable or not. The same could be done for triu / trid.
But that can be a future refactoring too, so the functionalities are already available in a "non-assignable" mode, and the refactoring won't break api compatibility.


Feature: Enhanced control of AppDomains (v5.6)

It has long been possible to instruct BizTalk to load your application's assemblies into a separate .NET AppDomain that does not interact with any other running BizTalk apps. The Deployment Framework has supported automatic configuration of that feature for many years, but did not expose one of the most important settings -- an AppDomain-specific .NET configuration file path. Need to configure Enterprise Library or store custom settings for a single app without modifying BTSNTSvc.exe.config? This feature is for you.
Add a new BizTalkAppDomain ItemGroup that allows one or more AppDomains to be configured for an app, and exposes explicit and regex-based assembly name assignment into those AppDomains. Also expose several of the most useful properties of the AppDomainSetup class including ConfigurationFilePath and ApplicationBase.
This work item was migrated from CodePlex
CodePlex work item ID: '10986'
Assigned to: 'tfabraham'
Vote count: '1'

[UnknownUser@8/22/2015]
Resolved with changeset 77157.


Add ability to override language for page title and main in Template

It is already possible to set the lang attribute on the html in Template. However according to https://github.com/alphagov/govuk_template/issues/342   there is a need to be able to set language of the page title and main separately.
Split from: https://github.com/alphagov/govuk-frontend/issues/971
Part of https://github.com/alphagov/govuk-design-system/issues/777

Needed as part of https://github.com/alphagov/govuk-design-system-backlog/issues/185


Update configuration files to use Webpack 2.6.0

Hi,
I just adapted the configuration files in order to have the examples working with Webpack 2.6 (installed on my system is 2.6.1).
Learnt a lot reading the examples, looking at the video, and running those examples, thanks!

💯 🎉


Hugo: Fix lists

Fix display of lists, add empty lines before them so that hugo formats
them correctly as lists and not as paragraphs.

/assign @steveperry-53


FLL Coach here (not an actual "issue")

Hello Peter. I thought you might be interested to know how I use your plugin. I am an FLL coach and we used your plugin all season, and it worked perfectly. Middle school kids were able to pick up the coding in no time at all. We advanced past our qualifier and did pretty good at our regioal championship (VA-DC).
I guess if anything that I could ask of you is, do you know a good online place where FLL coaches using Python on the Spike Prime hangout? I'd love to see how other coaches are doing this, and what kinds of cool programs they have written.
Anyway, sorry to hijack your support issues here, but please keep up the great work!
Skip Morrow
Coach Team 24277

Hey @MrGibbage , thanks for the kind words 😄  Glad that this plugin is helping you in your job as well as the kids you are teaching!
Sadly I'm not a coach myself and do not know of any FLL specific groups.


Launch 14 jan 2015

This issue is for @Floppy to merge tonight at 00:01 or close enough that it makes no difference
(If you can please check content first to make sure it is not broken/misaligned/useless)

This is checked and good to go. After midnight, first person to see this, hit the green merge button. :)


Docker error in New-BcContainer for BC23

This script worked yesterday for creating a BC22 sandbox container.
Today, I changed the version variable to create a BC23 sandbox container, but when I run the script it fails with a Docker error and the container is partially created but useable.
What is the likely cause?
Here is the script:
#-[ VARIABLES ]-----------------------
$bcVersion = '23.0.12034.13705'

$containerName = 'saas'

$bcAuth = "UserPassword"
$bcUserName = 'floyd'
$bcUserPassword = ConvertTo-SecureString 'floyd' -AsPlainText -Force

$dbServer = '<IP_ADDRESS>' # use Server IP instead of Server Name to avoid error: "Cannot establish a connection to the SQL Server/Database."
$dbInstance = ''
$dbSaName = 'sa'
$dbSaPassword = ConvertTo-SecureString 'XXX' -AsPlainText -Force

#-[ PREREQUISITES ]-----------------------
Import-Module -Name Az
Import-Module -Name bccontainerhelper

#-[ Get BC Artifact URL ]-----------------------
$artifactUrl = Get-BCArtifactUrl `
    -country 'US' `
    -version $bcVersion `
    -select Closest

$artifactUrl

#-[ Get BC License ]-----------------------
$licenseFile = 'T:\!_BACPAC\5356620.bclicense'

#-[ Create Multi-Tenant BC Container ]-----------------------
$userCredential = New-Object pscredential `
    -ArgumentList $bcUserName, $bcUserPassword
$dbCredential = New-Object pscredential `
    -ArgumentList $dbSaName, $dbSaPassword

$databaseParams = @{
    "databaseServer" = $dbServer
    "databaseInstance" = $dbInstance
    "databasePrefix" = "$($containerName)-"
    "databaseName" = "CRONUS"
    "databaseCredential" = $dbCredential
    "multitenant" = $true
}

Write-Host ('')
Write-Host ("Creating new container " + $containerName + "...") -ForegroundColor Yellow
New-BcContainer @databaseParams -replaceExternalDatabases `
    -accept_eula `
    -accept_outdated `
    -alwaysPull `
    -auth $bcAuth `
    -artifactUrl $artifactUrl `
    -containerName $containerName `
    -credential $userCredential `
    -enableTaskScheduler `
    -isolation hyperv `
    -licenseFile $licenseFile `
    -updateHosts `
    -useSSL $false

Write-Host ("New container created.") -ForegroundColor Yellow

Here is the output:

PowerShell Extension v2023.8.0
Copyright (c) Microsoft Corporation.

https://aka.ms/vscode-powershell
Type 'help' to get help.

PS W:\_REPOS\Qixas-Tools\Qixas Scripts>
PS W:\_REPOS\Qixas-Tools\Qixas Scripts> . {
#-[ VARIABLES ]-----------------------
$bcVersion = '23.0.12034.13705'

$containerName = 'saas'

$bcAuth = "UserPassword"
$bcUserName = 'floyd'
$bcUserPassword = ConvertTo-SecureString 'floyd' -AsPlainText -Force

$dbServer = '<IP_ADDRESS>' # use Server IP instead of Server Name to avoid error: "Cannot establish a connection to the SQL Server/Database."
$dbInstance = ''
$dbSaName = 'sa'
$dbSaPassword = ConvertTo-SecureString 'kr!tt3rs' -AsPlainText -Force

#-[ PREREQUISITES ]-----------------------
Import-Module -Name Az
Import-Module -Name bccontainerhelper

#-[ Get BC Artifact URL ]-----------------------
$artifactUrl = Get-BCArtifactUrl `
    -country 'US' `
    -version $bcVersion `
    -select Closest

$artifactUrl

#-[ Get BC License ]-----------------------
$licenseFile = 'T:\!_BACPAC\5356620.bclicense'

#-[ Create Multi-Tenant BC Container ]-----------------------
$userCredential = New-Object pscredential `
    -ArgumentList $bcUserName, $bcUserPassword
$dbCredential = New-Object pscredential `
    -ArgumentList $dbSaName, $dbSaPassword

$databaseParams = @{
    "databaseServer" = $dbServer
    "databaseInstance" = $dbInstance
    "databasePrefix" = "$($containerName)-"
    "databaseName" = "CRONUS"
    "databaseCredential" = $dbCredential
    "multitenant" = $true
}

Write-Host ('')
Write-Host ("Creating new container " + $containerName + "...") -ForegroundColor Yellow
New-BcContainer @databaseParams -replaceExternalDatabases `
    -accept_eula `
    -accept_outdated `
    -alwaysPull `
    -auth $bcAuth `
    -artifactUrl $artifactUrl `
    -containerName $containerName `
    -credential $userCredential `
    -enableTaskScheduler `
    -isolation hyperv `
    -licenseFile $licenseFile `
    -updateHosts `
    -useSSL $false

Write-Host ("New container created.") -ForegroundColor Yellow

}
WARNING: The names of some imported commands from the module 'Microsoft.Azure.PowerShell.Cmdlets.Network' include unapproved verbs that might make them
less discoverable. To find the commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved
verbs, type Get-Verb.
WARNING: The names of some imported commands from the module 'Az.Network' include unapproved verbs that might make them less discoverable. To find the
commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved verbs, type Get-Verb.
BcContainerHelper version 6.0.0
BC.HelperFunctions emits usage statistics telemetry to Microsoft
WARNING: The names of some imported commands from the module 'bccontainerhelper' include unapproved verbs that might make them less discoverable. To find
 the commands with unapproved verbs, run the Import-Module command again with the Verbose parameter. For a list of approved verbs, type Get-Verb.
https://bcartifacts.azureedge.net/sandbox/23.0.12034.13705/us

Creating new container saas...
BcContainerHelper is version 6.0.0
BcContainerHelper is running as administrator
HyperV is Enabled
UsePsSession is True
Host is Microsoft Windows 11 Enterprise - 10.0.22621.2715
Docker Client Version is 18.09.8
Docker Server Version is 18.09.8
Removing Desktop shortcuts
Setting database saas-tenant offline
Removing database saas-tenant
Setting database saas-CRONUS offline
Removing database saas-CRONUS
Starting Database Restore job from https://bcartifacts.azureedge.net/sandbox/23.0.12034.13705/us
C:\Program Files\WindowsPowerShell\Modules\bccontainerhelper\6.0.0\Import-BcContainerHelper.ps1
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
WARNING: Unable to find matching generic image for your host OS. Using mcr.microsoft.com/businesscentral:10.0.20348.2031
Building multitenant image false:sandbox-23.0.12034.13705-us-nodb-mt based on mcr.microsoft.com/businesscentral:10.0.20348.2031 with https://bcartifacts.azureedge.net/sandbox/23.0.12034.13705/us without database
Pulling latest image mcr.microsoft.com/businesscentral:10.0.20348.2031
10.0.20348.2031: Pulling from businesscentral
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.20348.2031 (ltsc2022)
Host OS Version: 10.0.22621.2715 (22H2)
Using hyperv isolation
Using license file T:\!_BACPAC\5356620.bclicense
Files in c:\bcartifacts.cache\mnhgbgbs.3gc\my:
Copying Platform Artifacts
c:\bcartifacts.cache\sandbox\23.0.12034.13705\platform
Copying ConfigurationPackages
C:\bcartifacts.cache\sandbox\23.0.12034.13705\us\ConfigurationPackages
Copying Extensions
C:\bcartifacts.cache\sandbox\23.0.12034.13705\us\Extensions
Copying Applications.US
C:\bcartifacts.cache\sandbox\23.0.12034.13705\us\Applications.US
c:\bcartifacts.cache\mnhgbgbs.3gc
Sending build context to Docker daemon  1.903GB

Step 1/6 : FROM mcr.microsoft.com/businesscentral:10.0.20348.2031
 ---> a28a216749db
Step 2/6 : ENV DatabaseServer=localhost DatabaseInstance=SQLEXPRESS DatabaseName=CRONUS IsBcSandbox=Y artifactUrl=https://bcartifacts.azureedge.net/sandbox/23.0.12034.13705/us filesOnly=False
 ---> Running in 3fcf65d78012
Removing intermediate container 3fcf65d78012
 ---> 066d9a18d5d2
Step 3/6 : COPY my /run/
 ---> e1d7be96115a
Step 4/6 : COPY NAVDVD /NAVDVD/
 ---> 68a33f39b93c
Step 5/6 : RUN \Run\start.ps1 -installOnly -multitenant
 ---> Running in 03e3ebf6363d
c:\run\my folder doesn't exist, creating it
Using installer from C:\Run\210-new
Installing Business Central
Installing from DVD
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
C:\NAVDVD\ServiceTier\Program Files
C:\NAVDVD\ServiceTier\System64Folder
Copying PowerShell Scripts
C:\NAVDVD\WindowsPowerShellScripts\Cloud\NAVAdministration
C:\NAVDVD\WindowsPowerShellScripts\WebSearch
Copying Web Client Files
C:\NAVDVD\WebClient\Microsoft Dynamics NAV
Copying ModernDev Files
C:\NAVDVD
Copying additional files
Copying ConfigurationPackages
C:\NAVDVD\ConfigurationPackages
Copying Test Assemblies
C:\NAVDVD\Test Assemblies
Copying Extensions
C:\NAVDVD\Extensions
Copying Applications
C:\NAVDVD\Applications
Copying Applications.US
C:\NAVDVD\Applications.US
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Skipping restore of Cronus database
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Installation took 26 seconds
Installation complete
New-Bcimage Telemetry Correlation Id: 605c6e73-a665-4dcc-9147-4fdb1f364af5
New-BcContainer Telemetry Correlation Id: 035768c6-63c9-4d9c-9f0b-5191870e8d28
DockerDo : re-exec error: exit status 1: output: time="2023-11-16T15:46:03-05:00" level=error msg="hcsshim::ImportLayer failed in Win32: The system
cannot find the path specified. (0x3)
path=\\\\?\\C:\\ProgramData\\docker\\windowsfilter\\9b02e9f68241373c78c4cdaf8964f6fec3430f10e188d2e95dbd35b9a212161a
folder=C:\\ProgramData\\docker\\tmp\\hcs030992335"
unexpected EOF
ExitCode: 1
Commandline: docker build --isolation=hyperv --memory 8G --no-cache --tag false:sandbox-23.0.12034.13705-us-nodb-mt c:\bcartifacts.cache\mnhgbgbs.3gc
At C:\Program Files\WindowsPowerShell\Modules\bccontainerhelper\6.0.0\ContainerHandling\New-NavImage.ps1:694 char:27
+ ...       if (!(DockerDo -command build -parameters @("--isolation=$isola ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
    + FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,DockerDo

PS W:\_REPOS\Qixas-Tools\Qixas Scripts>

Hey... I have exact the same issue since few days.
It fails in the same place as in example above (on docker build command). I know that some collegues from my company also have the same problem in other projects.
New-BcContainer Telemetry Correlation Id: d6bcc158-e217-43fb-909c-9f923a5876ff
Removing Desktop shortcuts
Run-AlPipeline Telemetry Correlation Id: a167d25b-e6a5-4fb8-964f-40a614856395
DockerDo : re-exec error: exit status 1: output: hcsshim::ImportLayer failed in Win32: The system cannot find the path
specified. (0x3)
ExitCode: 1
Commandline: docker build --isolation=process --memory 7G --no-cache --tag my:sandbox-23.1.13431.14059-pl-mt
c:\bcartifacts.cache\evie5cqv.in3
At C:\Program Files\WindowsPowerShell\Modules\bccontainerhelper\6.0.0\ContainerHandling\New-NavImage.ps1:694 char:27

...       if (!(DockerDo -command build -parameters @("--isolation=$isola ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,DockerDo

##[error]PowerShell exited with code '1'.

Hi I have exact same issue here. It started on Wednesday then I was able to run it during Thursday, but now it crashing again.
Installation complete
New-Bcimage Telemetry Correlation Id: 61aa7704-7249-4192-8eb8-f0aec0874335
New-BcContainer Telemetry Correlation Id: 981e0ad8-b998-42af-bbee-f842f6e2c8ff
Removing Desktop shortcuts
Run-AlPipeline Telemetry Correlation Id: 6397db8b-5eb3-45e0-9f45-92f2fe4b740c
##[error]DockerDo : re-exec error: exit status 1: output: hcsshim::ImportLayer failed in Win32: The system cannot find the path
specified. (0x3)
ExitCode: 1
Commandline: docker build --isolation=process --memory 8G --no-cache --tag my:onprem-21.2.49946.49990-se
c:\bcartifacts.cache\j1gnd3qv.hqp
At C:\Program Files\WindowsPowerShell\Modules\bccontainerhelper\6.0.0\ContainerHandling\New-NavImage.ps1:694 char:27

...       if (!(DockerDo -command build -parameters @("--isolation=$isola ...

            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Write-Error], WriteErrorException
FullyQualifiedErrorId : Microsoft.PowerShell.Commands.WriteErrorException,DockerDo
##[error]PowerShell exited with code '1'.

We also have the same problem on some of our build servers. It seems to be related to the creation of images. We have one server, which managed to pull the BC image yesterday, and it runs fine now. The other two servers however, fail at the exact same place as above.

@floyd-chan - the reason your script fails is a missing colon between -useSSL and $false in the last line:

-useSSL is a switch parameter which means that your $false will by powershell be used as a parameter to the imageName parameter - so your script will create an image (which fails).
Delete the last line and the terminating ` in the line above. (useSSL is false by default anyway)

The other problems seems to come from image creation - trying to replicate those, but there isn't much data to go with.
@Athystrup , @hannib84 and @swomkbb  please create a new issue and include script + full output to help me repro this issue.


Prefer better build system instead? 230 s (4 minutes) for a fresh build is unacceptable.

Fresh build times:
third_party/java/nailgun$ time mvn package
230.53 real 20.14 user 2.54 sys
third_party/java/nailgun$ tup
... elided ...
[2.297s] third_party/java/nailgun/nailgun-server: [JAVAC_JAR] nailgun-server.jar
... elided ...
230 seconds is about 4 minutes! Versus 2.2 seconds. You don't have a ton of dependencies
with nailgun. Please drop Maven and consider using a real build system:
https://github.com/facebook/buck or http://gittup.org/tup/.

@gorakhargosh I cannot reproduce your symptoms on a modest laptop:
$ time mvn clean package --quiet

real    0m7.522s
user    0m21.911s
sys     0m0.856s


Blood Dye Offhand Drygore Mace

When creating OH Drygore Mace (blood) it is checking for Drygore longsword not mace.

Fixed now.


daemon: add agent-runtime-config backup files to gitignore

Executing the unit tests in pkg/option generates agent-runtime-config.json backup files.
❯ go test ./pkg/option/...
ok      github.com/cilium/cilium/pkg/option     0.236s
ok      github.com/cilium/cilium/pkg/option/resolver    (cached)

❯ git status
On branch main
Your branch is up to date with 'origin/main'.

Untracked files:
  (use "git add <file>..." to include in what will be committed)
        pkg/option/agent-runtime-config.json

nothing added to commit but untracked files present (use "git add" to track)

This commit adds the config files to the gitignore file to prevent them from being added to version control.
Note: At a second look it seems that the config backup files are never stored into the Cilium StateDir. Writing, reading and validating the config files directly operators on the working directory. And the whole backup file rotation doesn't seem to work as expected as this one is working on the statedir directory where no backup files exists :thinking: But this PR fixes the issue that these config files shouldn't be added to version control (even though it would be nicer to be able to use a temp test dir - but this would require to change the config logic)
Fixes: https://github.com/cilium/cilium/pull/32981

/test


WIP Default to an empty dict instead of None

This avoids the following failure:
⋊> passpie help add
Traceback (most recent call last):
...
TypeError: argument of type 'NoneType' is not iterable

@marcwebbie: can you please help me write a test to this?


The Menu Bold Style Makes "The Width Inconsitent"

Example:

If you hover a menu text over that, the other text on that header is moving too (which is annoying).
I think each menu element must we wrapped in a fixed-width element.

Wow, nice catch!
@khairulanshar might be interested.
Anyone has suggestion?

Fixed by @khairulanshar ...by not using bold style.


Replace boost gregorian date

I don't believe you would be happy with the refactoring to the tests, but please help me out to get this right.
Unit tests also failed with missing py.test error that I wasn't able to figure out how to resolve in my setup.

Will work on the failing tests

Needs rework


empty authType SASL username leads to incorrect default bucket as datasource

If the user creates a new index and chooses the couchbase datasource type...
For example, perhaps they wish to index the "beer-sample" bucket...
And, the user then leaves the "authUser" field as empty (""), they can be inadvertently incorrectly connected up to the default bucket instead of the beer-sample bucket.

commit c5b3ea0


[Enhancement] Add more test coverage for DefaultMQPushConsumerImpl

Before Creating the Enhancement Request

[X] I have confirmed that this should be classified as an enhancement rather than a bug/feature.

Summary
Add more tests.
Motivation
Add more tests.
Describe the Solution You'd Like
Add more tests.
Describe Alternatives You've Considered
No
Additional Context
No response

link #8262

link #8262

OK, I didn't pay attention to this issue before, but I saw this RIP: https://github.com/apache/rocketmq/wiki/RIP-10-RocketMQ-Unit-Test
I will quote #8262 later.


feat: use cache kv manager for SchemaMetadataManager

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Add cache layer for SchemaMetadataManager
Checklist

[x] I have written the necessary rustdoc comments.
[x] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

Lack of prefetching the database ttl option, are you gonna do it in next PR?

Here: https://github.com/GreptimeTeam/greptimedb/pull/5053/files#diff-c8c39874ac506c24bf14a603286e832fd039cdc068f8abec26d8e4061c255f32R64-R66

Lack of prefetching the database ttl option, are you gonna do it in next PR?

Here: https://github.com/GreptimeTeam/greptimedb/pull/5053/files#diff-c8c39874ac506c24bf14a603286e832fd039cdc068f8abec26d8e4061c255f32R64-R66

@v0y4g3r do we need to prefetch the database options when opening the region in the datanode?

Lack of prefetching the database ttl option, are you gonna do it in next PR?

Here: https://github.com/GreptimeTeam/greptimedb/pull/5053/files#diff-c8c39874ac506c24bf14a603286e832fd039cdc068f8abec26d8e4061c255f32R64-R66

@v0y4g3r do we need to prefetch the database options when opening the region in the datanode?

I think it's ok for the first time?

Lack of prefetching the database ttl option, are you gonna do it in next PR?

Here: https://github.com/GreptimeTeam/greptimedb/pull/5053/files#diff-c8c39874ac506c24bf14a603286e832fd039cdc068f8abec26d8e4061c255f32R64-R66

@v0y4g3r do we need to prefetch the database options when opening the region in the datanode?

I think it's ok for the first time?

@v0y4g3r hmm... just some picky ones would complain about the first time performance. But let's not block this PR, I'll do it.


remove noble package in web sdk

Summary
Remove noble packages as moved native code to its own repo
How did you test your changes?

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

Review Error for bangtoven @ 2024-09-16 19:06:51 UTC
User failed mfa authentication, see go/mfa-help


Not an issue but to say thank you!

Hey, @shupershuff
THIS IS NOT AN ISSUE, JUST TO SHOW GRATITUDE!
Thanks for this amazing repo that you've made, amazing quality code, comments for modifying it and overall performance.
Wish you all the best, cheers!

Cheers mate! Appreciate it. Very cool to have strangers across the world find some use in my wee project :)


deriving support

hi!
I was just wondering if you did anything more on the deriving support?
Also you didn't raise a merge request against https://gitlab.com/fommil/scalaz-deriving/ so I can't see our work :smile:

For sure, some other stuff took priority, but I was actually able to get things working by adding code to the plugin - https://gitlab.com/carymrobbins/scalaz-deriving/tree/newtype
Here's a branch with some fixes for newtype that make the feature above work - https://github.com/estatico/scala-newtype/tree/deriving
As it currently stands I think the remaining work needed is to prevent the companion object from extending Function when a companion is not defined.

hmm. Disabling extending the function might be very complicated to pull off, since we're just duplicating what scalac does. The right thing to do there is to use class instead of case class.

Turned out to be pretty simple, let's see what you think - https://gitlab.com/fommil/scalaz-deriving/merge_requests/149


Stomp prompt should ask to delete ignored files on join

Right now, if you add a new pattern to your .flooignore/.gitignore/whatever, we don't try to remove the newly-ignored files from the workspace. This is really confusing to users.

Note: It should just remove these files from the workspace. It shouldn't send unlink: true.


feat: display reUploadedDataSize and newlyUploadedDataSize of each backup

What this PR does / why we need it

[x] Display Re-Uploaded Data Size and Newly Uploaded Data Size of each backup
[x] Highlight full backups by color tag

Issue
[UI][IMPROVEMENT] Display ReUploadedDataSize and NewlyUploadedDataSize of each backup on Volume page
Test Result

The table should display Re-Uploaded Data Size and Newly Uploaded Data Size
When creating a full backup, it will be highlighted with a blue tag

Summary by CodeRabbit

New Features

Enhanced table structure in the Backup List with new columns for 'Re-Uploaded Data Size' and 'Newly Uploaded Data Size'.
Improved visual representation of the 'Backup Mode' column using tags for better differentiation.

Style

Updated styles for Ant Design components, enhancing hover effects and alignment for tables and cards.
Introduced new styling for backup mode elements to improve appearance and usability.

Bug Fixes

Minor adjustments to rendering functions for better data presentation without altering core functionality.

Can the full and incremental information be shown in the snapshot(backup) chain of the volume page as well?

Can the full and incremental information be shown in the snapshot(backup) chain of the volume page as well?

Added backup mode: full / incremental in hover toolip?

Can the full and incremental information be shown in the snapshot(backup) chain of the volume page as well?

Do you mean adding backup mode: full / incremental in toolip?

Yes, you are right!

nit: BTW, the font size of the backup mode full and incremental looks smaller than other words

Can the full and incremental information be shown in the snapshot(backup) chain of the volume page as well?

Do you mean adding backup mode: full / incremental in toolip?

@a110605 @derekbit
Does this require API support? I don’t see the backup mode in v1/volumes/xxx?action=snapshotList.
Also, there can be multiple backups, how about show the user like this:
Backups:
backup-0411e509f5174d41,backup-4ed18b70d78d4875 (Full Backup)
backup-bad90ca310914546,backup-f1779c1e788f4f50

nit: BTW, the font size of the backup mode full and incremental looks smaller than other words

Yes, that is expected, as it's the design from Ant Design.
The node tag and disk tag are the same, so I would prefer to keep it as-is.

Can the full and incremental information be shown in the snapshot(backup) chain of the volume page as well?

Do you mean adding backup mode: full / incremental in toolip?

@a110605 @derekbit Does this require API support? I don’t see the backup mode in v1/volumes/xxx?action=snapshotList. Also, there can be multiple backups, how about show the user like this:
Backups:
backup-0411e509f5174d41,backup-4ed18b70d78d4875 (Full Backup)
backup-bad90ca310914546,backup-f1779c1e788f4f50

Hi @houhoucoop
Yes you are right, this info is not in the response of the API
We can handle it in another ticket and we can also discuss how to use color or something else to distinguish them as well.
Some users are also confused about that chain and don't know if it is a snapshot or a backup. We can refine it in the same ticket.
Can you help to create the ticket for it.
Something like: "Refine the snapshot(backup) chain chart in the volume page"
The goal is to refine the chart and help users to easily understand the meaning of the chart.
Thanks!

Hi @houhoucoop
The enhancement LGTM.
There is one small thing would like to ask for your help.
Can we allow users to "sort" the "Backup Mode" as well like "Snapshot Created"?
For the same "Backup Mode", we can sort them with "Snapshot Created" so the users can quickly find the latest full backup.
Thanks

Hi @houhoucoop The enhancement LGTM. There is one small thing would like to ask for your help. Can we allow users to "sort" the "Backup Mode" as well like "Snapshot Created"? For the same "Backup Mode", we can sort them with "Snapshot Created" so the users can quickly find the latest full backup. Thanks

Updated in commit https://github.com/longhorn/longhorn-ui/pull/817/commits/d2f07b0b1e378699929e00d386ce38bf37eba3bf.

We can handle it in another ticket and we can also discuss how to use color or something else to distinguish them as well.
Some users are also confused about that chain and don't know if it is a snapshot or a backup. We can refine it in the same ticket.

Issue created: https://github.com/longhorn/longhorn/issues/9864

Nit: can we make the font of the backup mode full and incremental consistent with creation state?

Nit: can we make the font of the backup mode full and incremental consistent with creation state?

Did you mean the font size or capitalize first letter? e.g. Full or Incremental

Did you mean the font size or capitalize first letter? e.g. Full or Incremental

Size and color

As discussed with @derekbit  offline, since the backup mode is sortable, highlighting Full Backup is unnecessary.
Updated in commit https://github.com/longhorn/longhorn-ui/pull/817/commits/951bafa7dcc0ef8b2b54254c5be4434eaaa92d46.

Thanks @houhoucoop for your first contribution to Longhorn🎉


Handle bad data and retry database queries more times

Oops. Didn't mean to have this auto-merge. I'll catch a review on the promote PR.


micromamba environment setup issue

Hi, thanks for sharing this repo.
I tried to setup an environment by following the instruction in README: micromamba create -f environment.yaml
However, I got the following errors.
I was able to resolve the issues of cudatoolkit, panel and bokeh by modifying the version, but not for pytorch.
Could you help me to address this issue?
nvidia/linux-64                                               No change
nvidia/noarch                                                 No change
conda-forge/noarch                                            No change
conda-forge/linux-64                                          No change
pytorch/noarch                                                No change
pytorch/linux-64                                              No change
pyviz/linux-64                                                No change
pyviz/noarch                                                  No change
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
warning  libmamba Problem type not implemented SOLVER_RULE_STRICT_REPO_PRIORITY
error    libmamba Could not solve for environment specs
    The following packages are incompatible
    ├─ bokeh 3.0.3**  is requested and can be installed;
    ├─ cudatoolkit 11.8.0**  is not installable because it conflicts with any installable versions previously reported;
    ├─ panel 0.10.3**  is not installable because there are no viable options
    │  ├─ panel 0.10.3 would require
    │  │  └─ bokeh >=2.2,<2.3 , which conflicts with any installable versions previously reported;
    │  └─ panel 0.10.3 conflicts with any installable versions previously reported;
    └─ pytorch 1.13.1*  is not installable because it conflicts with any installable versions previously reported.
critical libmamba Could not solve for environment specs

As for panel and bokeh, you can install any versions that work for you or not install them at all. These libraries are not needed to run scripts in the repository, they were used by the authors only in their private scripts and notebooks while working on the project.
Regarding CUDA and pytorch, can you try installing newer/older version of these packages? Overall, I expect the reported results to be robust to the versions of CUDA and pytorch.

@Yura52 Thanks for your prompt reply! I succeed to setup the environment by deleting panel, bokeh, cuda* and pytorch* in enviroment.yaml then installing cuda* and pytorch* manually (https://pytorch.org/get-started/previous-versions/).

Great! Feel free to reopen the issue if there are more questions.


Use stream when option is string and ssl/tls defined

Add transport class usage
Add try ... catch example
May be not optimal, but should work )))

Thanks a lot.
I've merged and optimized the code.

Great!
preg_match('/^(((tcp|udp|unix|ssl|tls):\/\/)?[\w\.\-]+)(\:(\d+))?/i', $options, $matches)

Does  udp|unix make sense ? Can C* listen on them?
By a way, if you use reg.exp. to parse string may be you can add user/password, like
ssl://user:password@host:port

ssl://user:password@host:port

It's a good idea.  but actually ssl protocol doesn't provide username and password argument.
Like mongodb or mysql, it should be
cassandra://user:password@host:port/keyspace

I hope $options argument should a valid fsockopen() argument.

udp|unix,
humm, maybe someday C* will provide unix socket way.  ^_^

 ssl protocol doesn't provide username and password argument.

Neither TCP does, those are not protocols but transports (from application layer POV), so I see no problem to add user:password
We are trying to do something useful  and usable, not http://en.wikipedia.org/wiki/Spherical_cow )))
Such solution will match both philosophy of user control and simplicity as well. I guess 99.(9)% of users will be happy just to use ssl://user:password@host:port without entering to philosophy at all )))

I hope it obey the standard, be clear and definite.
I don't think it's inconvenient to configure with array options. ^_^

For standard we should invent cassandra: and cassandras: )))


stream of msgpack-lite could be slow when working with socket stream

We found it could take more than 30s for transfer a 7M large of data through socket and then pipe to msgpack stream.
It could be solved, details are here: https://github.com/chemzqm/stream-issue

Hi @chemzqm!
Is this something you can implement into msgpack-lite? Would you be open to creating a PR?


Angular transformer deadlocks on _tests

After a few hours of debugging with @alorenzen, I've narrowed it down to here:
      await crawlImports(
          inputs,
          (id) => transform
              .readInputAsString(toBarbackAssetId(id))
              .catchError((_) => null));

... from build_barback. AFAICT, this was added by @natebosch for an internal customer that uses pub, but this same code causes circular dependencies in Angular itself to never succeed building when using with the _tests package:
// Add this to _tests/pubspec.yaml at the bottom
transformers:
  - angular/transform/reflection_remover:
      $include: test/**_test.dart
  - angular/transform/codegen
  - test/pub_serve:
      $include: test/**_test.dart

I don't know if its safe to remove this, or make it a flag, or what the proper fix should be.

If we remove this we're likely to end up hitting an issue later on.
Why is there a dependency from angular -> _tests? I wouldn't expect us to need any circular package level dependencies...

There is not a dependency, and I don't understand the details yet.
Only that all of my tests pass with that commented out, and they time out otherwise.

While we were looking at the internal case I had some uncommitted changes that I was using to track down where deadlocks were coming from - @alorenzen might still have the patch. It basically boiled down to keeping a a Map from an asset to the Set of assets it's waiting on (I think before we did package level but we could modify this to file level) and then running the build, waiting for it to settle, then assuming whatever is still waiting has the deadlock.
I'd be very interested in finding whether we can narrow it down to that level.
I'm surprised the build succeed after skipping that step. In general I wouldn't expect it to be safe because a deadlock there should mean we'd hit a deadlock at a later step anyway, but I could be misremembering the details.

Here is the high level reasoning behind that bit of code:
When we run the transformer with the Resolver we will end up needing to read transitive Dart imports at the moment we try to use the resolver. Grabbing a resolver adds a lock - only 1 Builder can be usign the Resolver at a time. Barback won't let us read any assets from packages which have not finished their Transformers yet so if we grab the Resolver lock, then can't do anything with it because one of the transitive imports is blocked by barback (and that package now can't get the resolver lock) we hit a deadlock.
This import crawling ensures the resolver is only "picked up" in dependency order so no resolver deadlock can happen. If we were to remove it then we'd be very likely to hit deadlocks in packages even where there is no cycle.

While we were looking at the internal case I had some uncommitted changes that I was using to track down where deadlocks were coming from - @alorenzen might still have the patch. It basically boiled down to keeping a a Map from an asset to the Set of assets it's waiting on (I think before we did package level but we could modify this to file level) and then running the build, waiting for it to settle, then assuming whatever is still waiting has the deadlock.

I guess I'm confused. At least for the AngularDart template building step (.template.dart) files, we don't care if the .template.dart file has been built or not for our dependencies. We don't analyze it, and we don't rely on it in anyway. We used to, but then we added the .ng_placeholder step.
Is the placeholder code what enables us to remove this check?

The limitation is that barback won't let you read any file from another package until it is done with its Transformers. Even if we don't care about generated files we won't be able to read source files either.

So as long as I don't have any circular packages (and I'm totally fine with making that a restriction of AngularDart 4.0), it should be safe to disable your check @natebosch?

Even with the check, it is already a requirement that there are no circular packages.

No, it will not be safe to disable the check. With this disabled the Resolver gets picked up in an arbitrary order - and if that order is not exactly dependency order you'll hit deadlock.

Then I guess I misunderstand why I need this check at all 😕

This check exists to enforce that the build runs serially in dependency order, which is the only way the Resolver can avoid deadlock.
In bazel this is enforced by the build system. In pub we had to enforce it ourselves.

Do you have a suggestion to proceed? We really need some tests enabled on travis in order to ensure 4.x is success, but I'm blocked with deadlocks as of now because of that check.

Like I'm totally fine if we need to add some sort of CLI-based checking, i.e.
$ pub run angular doctor

> Warning: You have a circular dependency. Bad bad bad!

In the immediate term we need to figure out which resources are deadlocked here, and either fix the circular dependency (which we don't expect should exist 😞 ) or fix this code to work in that condition.

Long term we want to move away from barback.

Wondering if this is going to be something specific to _tests. We might need to split compiler tests out from e2e tests, and only run the transformer on the later?

I guess the problem is even if I figure out where the problems are (I have a rough idea with @alorenzen), there is no good way to prevent these from re-occurring (because we are internal first).

Ok @natebosch and @alorenzen and I figured this out - having the reflection_remover transformer run before codegen causes a deadlock - codegen adds all crawled imports as a secondary input, which if this is after reflection_remover includes all generated files. That is a no-no!

If I remove the reflection_remover transformer from angular_test OR _tests, it works for the other one. If they both have reflection_remover, neither works.

weird... Have you tried looking for the places that import .template.dart in the checked in code and avoiding those tests?

Yeah I thought we had some, we don't.

OK I confirmed if I add a way to "filter" the crawlImports from build_barback, everything works @natebosch. I'm going to see if I can isolate this inside of AngularDart for right now (my idea now is to write a Builder that wraps and filters out .template.dart files.

Another thing: the version of build_barback on pub is 0.2.x, the version internally is 0.3.0-dev @natebosch. My assumption is that means... bad things?

Yes we will need to get a new version published. I don't recall if there are blockers or if we just never got around to it.

Considering we use it internally, I imagine it's safe. I'm going to try and publish with @kevmoo today.

This looks fixed @ build_barback 0.3.0. Working...

Yup, build_barback looks good.


Is Gunicorn server supported to deploy FastAPI on Windows Server 2022 as Production Server

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
gunicorn -w 4 -k uvicorn.workers.UvicornWorker

Description
I want to develop FastAPI on Windows sever 2022.
As I do some research on https://www.uvicorn.org/deployment/.
Is it possible for me to run this command   gunicorn -w 4 -k uvicorn.workers.UvicornWorker  to run the production server on window sever 2022.
I don't have experience in it so I don't know to get started with it.
**I have 4 question

Gunicorn server supported to deploy FastAPI on Windows Server 2022 ?
Howto set script file to run ( you can guide me to article relevant to this issue) , I can study it further
If it doesn't support Windows Server, It is more likely to have 2 way to get it done.
3.1 I install docker on the window server and deploy FastApo App to docker. (Let me know where I can download the image file)
3.2. Can I apply IIS Server for FastAPI on Windows?**

I hardly find any content that literally explains step-by-step how to deploy the FastAPI App to  the Windows server.
On My Window server 2020 that I am working on , I deploy Django App  on IIS Server as The first application , so that's why I have to  The FastApi App as second Applitoin on this window server as well.
Operating System
Windows
Operating System Details
Edition	Windows Server 2022 Datacenter Azure Edition
Version	21H2
Installed on	2/11/2022
OS build	20348.643
Processor	Intel(R) Xeon(R) CPU E5-2673 v3 @ 2.40GHz   2.40 GHz
Installed RAM	7.99 GB
System type	64-bit operating system, x64-based processor
Pen and touch	Pen and touch support with 3 touch points
FastAPI Version
fastapi 0.75.2
Python Version
python 3.9
Additional Context
No response

Gunicorn doesn't support windows.

Gunicorn doesn't support Windows.
Is there other ways to deploy fast API on window server?

There are posts in the Gunicorn repo on tweaking things to get it running in windows linux subsystem but that is probably difficult and buggy way to do things.
In 2022 Windows is not a popular OS to run new server infrastructure and IIS is even less popular since they are basically the home of legacy C#/Java enterprise applications. Thus it is doubtful you will find a full featured walk through on how to use IIS with an ASGI application.
You best bet if you absolutely must run it in Windows Server is docker. There is a performance penalty because I think windows docker runs things in a VM but it will be the easiest. Here are the docs for docker https://fastapi.tiangolo.com/deployment/docker/
If you use docker it will also answer your question 2 since the start command will be covered by the docker file and you will simply invoke it as any other docker container.

There are posts in the Gunicorn repo on tweaking things to get it running in windows linux subsystem but that is probably difficult and buggy way to do things.
In 2022 Windows is not a popular OS to run new server infrastructure and IIS is even less popular since they are basically the home of legacy C#/Java enterprise applications. Thus it is doubtful you will find a full featured walk through on how to use IIS with an ASGI application.
You best bet if you absolutely must run it in Windows Server is docker. There is a performance penalty because I think windows docker runs things in a VM but it will be the easiest. Here are the docs for docker https://fastapi.tiangolo.com/deployment/docker/ If you use docker it will also answer your question 2 since the start command will be covered by the docker file and you will simply invoke it as any other docker container.

it is understandable.
Thank you for your recommendation

Hi @technqvi
Just wondering if you proceeded with the docker solution as suggested above? I'm currently looking for a solution on this as well and just would like to check if you found the above stable to use in production or not.


Support French in localization file

Hello,
It'd be great to have french support as well.
I'll add the resx file to support them.
Is that ok for you?

Yes this is fine.  Please fork the repository and submit a pull request with your changes.

Hello Meddington,
I submitted again the https://github.com/Lastjp/OutlookPrivacyPlugin/commit/451878b0a0d0519ed8177cc316390b8df917b862
This is fixing some square characters for french accent. For some reason, the charset did not work well.
Could you take it for your next release.
Based on the latest code that you commited, you did good job:


Introduce intro videos for www.surveilr.com and LinkedIn posts

@ajaykumaran  Could you please assign a respective team member to begin the creation of the introductory videos? After that, ensure the videos are reviewed with Shahid. Thanks!

@pradeepcdlm @ajaykumaran before creating any videos we should prepare a list of the videos, where in the onboarding they will be added, and the actual scripts. Once we have a few outlined and purpose is known and agreed upon and we have scripts then we won't need any rework.


Slate App for Shopify Stores

One of the biggest pain-points of Slate's onboarding and setup process is connecting your project with an Shopify store. Currently it lives in the Slate Docs as a seven step process that requires you to navigate deep into your stores admin configuration and then manually copy values.
What if users could instead install the Slate App from the Shopify App Store and let it do the heavy lifting of connecting your project to your store. I believe the following workflow could be possible:

User installs Slate App on their store and is navigated to the Slate App Dashboard
User runs slate theme on their dev machine
User is asked for store myshopify.com URL and access key displayed in the Slate App Dashboard.
Slate requests data needed to auto-generate config.yml in the project via the an App Proxy

This could open the door to even more stream-lined integrations between an online store and Slate project.

An potential alternative approach would be for Slate to authenticate directly with a store. The user could run a command like slate login which would prompt the user for their store URL. After submitting the URL a browser window could open with the store login. The user could proceed to login and an authentication token would be created, connecting Slate CLI to store.


Feature: Reliable Log Fetching in the Face of Lying Nodes

In block streaming mode the algorithm needs to work like this:
We keep track of the last block we encountered an augur log: L
We get notified of a new block B
We turn around and request logs for blocks (L, B] matching our filters.
One of two cases occurs:

we get logs returned, in which case we know the node we spoke to has logs for that block and would have any known logs for blocks before B.

So we then take all logs from that range, and write them. We know any blocks with no logs do indeed have no data for augur.
We then update L to be B.

we get [], which means either no logs matched the filter, or the block didn't exist on that node.  In this cAse we don't commit the empty block, nor tell the UI a change was made.

Looks right


Mixer IP address values are passed to adapters as []byte

details:
Martin Taillefer:
Doug just pointed out that our access logs currently contain this kind of stuff:
{"level":"info","time":"2018-02-01T09:54:29.733870Z","instance":"accesslog.logentry.mixer-test50289c613cd04c4390cd9aab67","destinationIp":"AAAAAAAAAAAAAP//CjwJfQ==","
latency":"7.556ms","method":"GET","protocol":"http","requestSize":0,"responseCode":
200,"responseSize":178,"sourceIp":"CjwERA==","sourceUser":"cluster.local/ns/mixer-test50289c613cd04c4390cd9aab67/sa/default","url":"/details/0"}
Note the sourceIp and destinationIp values. Looks like we're getting confused about what we're logging.
The above is coming from the stdio adapter. I suspect the adapter is expecting a net.IP value but is
instead given a []byte.
I've lost track of where this kind of stuff is handled in Mixer proper. Who should be converting the
incoming []byte from the proxy to a net.IP for adapter use?
Sunny Gupta:
So the issue is, during request time, we (generated code) cast the byte[] to net.IP only of the type that are explicitly called out as standard_types.IPAddress inside the template.proto. For the dynamic types like dimensions, labels where the type is inferred from expression, we are still passing byte[] to the adapter since during request time we have no clue of the inferred type that we computed during config time.
I chatted to Oz and we have a simple temporary solution for now, which I am currently working on to evaluate the performance impact. Basically, as a temporary solution, during IL evaluation, we can check if final result is byte[], we can convert it into net.Ip (currently byte[] can only mean net.IP). Long term, according to Oz, the compiled expressions can
actually do better since they already should be knowing the expression type and we can just do the conversion localized
for IPAddress.

@ozevren @geeknoid  @douglas-reid  FYI

We should move quickly on the temporary hold-over solution, as I believe this is a regression.

This issue is fixed; the long term solution as suggested above applies to all types inside compiled expression and not just to the net.IP. The long term solution can be done as part of overall enhancement to the compiled expression; whenever we decide to do it.
Close this issue which is specific to net.IP


Set a timeout

Sometimes a certain timeout for connection attempts is needed. Having a function to set the timeout of websocket be a good functionality.

I could see that in specific cases. To be clear, the timeout would just be used on connecting to a server? NSStream doesn't have a timeout property that I am aware of, but Pull Requests are welcomed.

I tested different connection possibilities that would make a timeout useful. Configs test where:

Host doesn't exist
Host isn't a WebSocket server
Host is down.
Host is working properly

In all cases the socket responded in less than a second stating it could not connect. Is there a specific test case that would show the usefulness of the timeout?

Maybe I have a test case that would show a usefulness of a timeout. In my application I have this code below. I send a string to the Server and wait for response. The response time from the server is always variable. Therefore I implement a do while statement and wait until text is not nil. Otherwise I would work with an empty String.
       queue.addOperationWithBlock() {
            // do something in the background
            self.logData="{\"username\":\"\(username)\",\"password\":\"\(password)\"}"
            self.socket.writeString(self.logData)

            do {
                //do nothing, wait for response
            } while (self.text=="")

            self.response=self.text //websocketDelegate method didReceiveMessage
            println("Login \(self.response!)")

            NSOperationQueue.mainQueue().addOperationWithBlock() {
                // when done, update your UI and/or model on the main queue
               if self.text == "true" {
                         //do something
               }
             }
          }

@goki88 thanks for the example. This would be something app specific, not something that we would want to implement in Starscream. For the use case above, I would use a NSDate or/and maybe a NSTimer to watch the wait time. It could be something like the timer fires every 1 or 2 seconds to see if the 10 seconds have elapsed then do the connection failure logic.

in my case it is for synchronous requests (with a loading label), 10 seconds is not long enough or sometimes too. Right now I'm using a timer, it was just an idea to a cleaner code.

Alright cool. I think the timer is going to be the best solution at this point. I am certainly open to Pull Requests, so please do open one if a cleaner solution that fits inside the library presents itself.


ETC Mining settings for 2miners?

Hi, I configured the miner as follows to mine ETC on 2miners pool but my shares won't show up on the pool. I tried running the miner on several pcs for 24 hours which OpenCL and CUDA installed (4GB+ vRAM) but I don't get any share.
Connection: Stratums (SSL/TLS)
Pool: asia-etc.2miners.com:11010
Wallet Address: My trustwallet ETC address
Workername, password, extra: left blank
Idle mining: ON
Max GPU: 90%
Idle GPU: 90%
Mine ETC: ON
Can you help me with this? This is the pool config instructions: https://etc.2miners.com/help
Thanks.

It should work fine, can you run MinerCheck3.zip and post what it returns here?


Add fields to API key list

Scope of changes
This adds fields to the API key list response in Tenant and Quarterdeck. Most importantly:

Partial: Indicates whether or not the API key has all available permissions Full or partial permissions Partial
Status: Unused, Active, or Stale

Fixes SC-14899
Type of change

[x] new feature
[ ] bug fix
[ ] documentation
[ ] testing
[ ] technical debt
[ ] other (describe)

Acceptance criteria
Describe how reviewers can test this change to be sure that it works correctly. Add a checklist if possible.
Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[x]  Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated Shortcut story to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.
[ ] Are there any TODOs in this PR that should be turned into stories?

This PR will fix SC-15075

This PR will also fix SC-14746


Assortment of small partitioned-epoch rewards cleanups

Problem
While moving forward on SIMD-0018 implementation, I discovered a handful of small things to clean up.
Summary of Changes

Limit the pub-ness of calculate_rewards_for_partitioning. This just got missed in #553 when I moved in the compare submodule
Wrap long string literal that is hard to read
Remove sysvar account balance from log; the balance is a waste of bytes, now that it doesn't change on distribution
Update solana_account_decoder to stringify EpochRewards::parent_blockhash
Add EpochRewards sysvar getter, which will be helpful for partition recalculation work

If you want me to break any of this up, I totally get it; these things aren't really related. Just let me know.

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

Project coverage is 81.9%. Comparing base (baed522) to head (a1f07f4).
Report is 3 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #760   +/-   ##
=======================================
  Coverage    81.9%    81.9%
=======================================
  Files         851      851
  Lines      231100   231159   +59
=======================================
+ Hits       189395   189475   +80
+ Misses      41705    41684   -21


Changelog: update for 2.2.0

Please make sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's contributing guide (中文 | English | Español).
[ ] Make sure you are merging your commits to dev branch.
[ ] Add some descriptions and refer relative issues for you PR.

First of all. Congratulations.  More than 23,000 stars. They keep growing.
Second. There is a Spanish-language attribute on the menu that does not appear in English or Chinese. It's disabled.
Attached image.
Check it out, but I think I've done all the pending translations.

Ah, this is a mistake, I'll correct it. Thanks for pointing it out.

Hi, could you translate another sentence to Spanish?

Note: we use yarn to lock dependency versions, so you should install dependencies using yarn instead of npm install.

This is for people who want to contribute to Element. Thanks!

Nota: Usamos yarn para bloquear versiones de dependencias, por lo que debería instalar dependencias usando yarn en lugar de npm install.
That would be in Spanish.

@Leopoldthecoder
Whenever I can help, for this repository, another code, or for any other purpose just tell me.
It's a pleasure to be able to help. Always.

Appreciate it! Thanks!


Update Contributions

Move the templates to the .github folder.

 Thanks!
Since you are the master of .MD fixing, could you take a look at MagicMirror² Module Development Documentation if you have time?