Object type with function properties

Not really sure what would be an appropriate title, please amend as you see fit.
Given this prelude:
/* @flow */
type SomethingDoer = {
  doSomething: (input: number) => string;
};

Exhibit A (passes):
function create() : SomethingDoer {
  return {
    doSomething() { // I would have expected this to conflict with SomethingDoer
      return '';
    }
  };
}

Exhibit B (fails):
function create() : SomethingDoer {
  return {
    doSomething(input : string) { // "This type is incompatible ..." as is expected
      return '';
    }
  };
}

Exhibit C (passes):
function create() : SomethingDoer {
  return {
    doSomething(input : boolean) { // I would have expected this to conflict with SomethingDoer
      return '';
    }
  };
}

Ideas? I'm on 0.4.0.

This can be expressed more simply:
type Contra = (x: number) => void;

var exA: Contra = function() {}
//var exB: Contra = function(x: string) {}
var exC: Contra = function(x: boolean) {}

I'm not an expert, so I'm probably not going to use the right words to explain this. I would appreciate it if someone would help correct my wording.
Example A follows because a function taking no arguments is a subtype of a function taking any arguments. This has a straightforward intuition, the function taking no arguments can't make any invalid assumptions—it can't make any assumptions at all.
Example C follows from the subtyping relationship number < boolean and the contravariance of function arguments. I believe the subtyping relationship exists so that numbers can be used in constructs like if.
In other words, this is the expected behavior.

Oh, thanks a lot. I totally forgot to consider the subtype relationships with my examples, and also managed to pick a particularly hairy Example C (as booleans are indeed complicated by the truthiness/falsyness of JavaScript).
It's quite clear now, sorry for the noise! :)


AudioStreamPlayer2D new stream types do not change property "playing" after finishing playing the audio 4.3.beta3

Tested versions
4.3.beta3 4.3.beta2 4.3.beta31
System information
Windows 11, RTX 3080, AMD 5800X3D
Issue description
So the new AudioStreamPlayer2D new stream types do not change property playing after finishing the audio
AudioStreamPlaylist
AudioStreamInteractive
AudioStreamSynchronized
I think there more problems with them like not having icons, I think they also skip more properties, but need some more testing
Steps to reproduce
Create a project
Create a scene
Create AudioStreamPlayer2D
Choose one of the new audio stream types
try this code:
func _process(delta: float) -> void:
    if !$AudioStreamPlayer2D.playing:
        $AudioStreamPlayer2D.play();
    pass;

Minimal reproduction project (MRP)
N/A

Possible duplicate of #94226 ?

My problem is not related to web export and only happens with the new types of audio stream

I can't reproduce this in v4.3.beta3.official [82cedc83c] with Windows 10.0.22631 - Vulkan (Forward+) - dedicated AMD Radeon RX 7700 XT (Advanced Micro Devices, Inc.; 31.0.24033.1003) - AMD Ryzen 7 5700X3D 8-Core Processor (16 Threads).
Can you add a MRP? That would help to see if it really doesn't reproduce on my system or if there's something different in our projects.

mrp.zip
Here is the small project I also added $AudioStreamPlayer2D.get_playback_position() to display that the position is restarted but the stream still shows playing

I can reproduce it now, thanks. My test was slightly different and probably not testing the right thing.

I'll see if I can fix this tonight before Godot Wild Jam starts, if it's not fixed in 7 hours someone else can pick it up as I'll busy with the jam for the next week. :)

Found the problem. Here's how it goes.
Here, AudioServer is determining that the stream is ending because what it received in bytes doesn't match the buffer size.
https://github.com/godotengine/godot/blob/97b8ad1af0f2b4a216f6f1263bef4fbc69e56c7b/servers/audio_server.cpp#L404-L417
But here, the stream is coded as if it's inactive, it will feed the requested frames, but silent.
https://github.com/godotengine/godot/blob/97b8ad1af0f2b4a216f6f1263bef4fbc69e56c7b/modules/interactive_music/audio_stream_synchronized.cpp#L207-L212
As AudioServer receives data, it never triggers the code that would end the stream.
Is this intentional @reduz?

@adamscott well, that is one way to end the audio, but streams also have an is_active() function that you can use check to see if they are doing anything.


inconsistence of big data

Note: People usually have question around learning Big data tools in initial phases of data science, I personally think, it's not necessary to learn big data tools in initial phases of data science, but if you're interested in learning it, you can learn it later on.

The phrase "big data" appears 3 times in the article, but only the first "big data" was written to "Big data".

The phrase "Big Data tools" is used to refer to the conceptual tools, like :-
Hadoop
Atlas.ti
HPCC
Storm
Cassandra
Stats iQ
CouchDB
Pentaho, etc.
The phrase "big data" is literally used for the tools which uses lots and lots of multi-dimensional data (which can be literally written as "big data").


Append chongg to blossom-ci whitelist [skip ci]

Signed-off-by: Chong Gao<EMAIL_ADDRESS>

why you change the title? all PRs will be added to the change log when release, there should be no personal info here

build


launch config for debugging

What does this do exactly?

Do you still want this to remain @primetheus ?  https://github.com/3rd-party-integrations/github-team-sync/blob/main/.github/CODEOWNERS#L2C1-L4C14
I am much less knowledgeable of this but I am happy to do my best to approve PR's and keep this functional.


[SPARK-44189][CONNECT][PYTHON] Support positional parameters by sql()

What changes were proposed in this pull request?
In the PR, I propose to extend the sql() method of Python connect client, and support positional parameters as list of Python objects that can be converted to literal expressions.
def sql(self, sqlQuery: str, args: Optional[Union[Dict[str, Any], List]] = None) -> DataFrame:

where

args is a dictionary of parameter names to Python objects or a list of Python objects that can be converted to SQL literal expressions. See the link regarding the supported value types in PySpark. For example: 1, "Steven", datetime.date(2023, 4, 2). The same as in Scala/Java API, a value can be also a Column of literal expression, in that case it is taken as is.

For example:
     >>> connect.sql("SELECT * FROM {df} WHERE {df[B]} > ? and ? < {df[A]}", [5, 2], df=mydf).show()
     +---+---+
     |  A|  B|
     +---+---+
     |  3|  6|
     +---+---+

Why are the changes needed?
To achieve feature parity with the PySpark API.
Does this PR introduce any user-facing change?
No, the PR just extends the existing API.
How was this patch tested?
By running new test:
$ python/run-tests --parallelism=1 --testnames 'pyspark.sql.tests.connect.test_connect_basic SparkConnectBasicTests.test_sql_with_pos_args'

and the renamed test:
$ python/run-tests --parallelism=1 --testnames 'pyspark.sql.tests.connect.test_connect_basic SparkConnectBasicTests.test_sql_with_named_args'

@HyukjinKwon @ueshin @cloud-fan @zhengruifeng @grundprinzip Could you review this PR, please.

@dongjoon-hyun May I ask you to review this PR.

@dongjoon-hyun May I ask you to review this PR.

Sure!

Merging to master. Thank you, @dongjoon-hyun for review.


Mono: mscorlib.dll.so not found

Hello,
I have a problem with generating mono glue.
With running command:
bin/godot.linuxbsd.tools.64.mono --generate-mono-glue modules/mono/glue
I get an error:

ERROR: Condition "!valid" is true. returned: false
at: _populate_object_type_interfaces (modules/mono/editor/bindings_generator.cpp:2401)

Here is the log:
Config attempting to parse: '/etc/mono/config'. (in domain Mono, info)
Config attempting to parse: '/home/bartlomiej/.mono/config'. (in domain Mono, info)
Image addref mscorlib[0x91c3c30] (asmctx DEFAULT) -> /usr/lib/mono/4.5/mscorlib.dll[0x91c2780]: 2 (in domain Mono, info)
Prepared to set up assembly 'mscorlib' (/usr/lib/mono/4.5/mscorlib.dll) (in domain Mono, info)
AOT: image '/usr/lib/mono/4.5/mscorlib.dll.so' not found: /usr/lib/mono/4.5/mscorlib.dll.so: cannot open shared object file: No such file or directory (in domain Mono, info)
AOT: module (null) is unusable: not compiled for debugging. (in domain Mono, info)
Assembly mscorlib[0x91c3c30] added to domain GodotEngine.RootDomain, ref_count=1 (in domain Mono, info)
Assembly mscorlib[0x91c3c30] added to domain GodotEngine.Domain.Scripts, ref_count=2 (in domain Mono, info)

Mono version:
Mono JIT compiler version <IP_ADDRESS> (tarball Tue Feb  4 21:20:20 UTC 2020)

In /usr/lib/mono/4.5/ is only mscorlib.dll file.
mscorlib.dll.so is in /usr/lib/mono/aot-cache/amd64/ directory.
I tried to move the file from amd64 dir to 4.5 dir, but then I got another error:

Config attempting to parse: '/etc/mono/config'. (in domain Mono, info)
Config attempting to parse: '/home/bartlomiej/.mono/config'. (in domain Mono, info)
Image addref mscorlib[0x7d0fdf0] (asmctx DEFAULT) -> /usr/lib/mono/4.5/mscorlib.dll[0x7d0e940]: 2 (in domain Mono, info)
Prepared to set up assembly 'mscorlib' (/usr/lib/mono/4.5/mscorlib.dll) (in domain Mono, info)
AOT: module /usr/lib/mono/4.5/mscorlib.dll.so is unusable: not compiled for debugging. (in domain Mono, info)
Assembly mscorlib[0x7d0fdf0] added to domain GodotEngine.RootDomain, ref_count=1 (in domain Mono, info)
Assembly mscorlib[0x7d0fdf0] added to domain GodotEngine.Domain.Scripts, ref_count=2 (in domain Mono, info)

I thought it was because of 'garbage' after previous Mono version.
I totally removed Mono (apt purge) and removed directories /usr/lib/mono & /etc/lib/mono.
Then installed Mono again using https://www.mono-project.com/download/stable/
After couple hours I have no idea what else can I do to fix it.
Does anyone know?

I believe that I'm seeing the same issue while trying to use the Mono Headless binary to build inside Docker. My mono logs contain similar messages about mscorlib.dll.so going missing. The message that appears at the CLI is "ERROR: Failed to export project: Failed to build project   at: godot_icall_GD_pusherror (modules/mono/glue/gd_glue.cpp:249)". More complete CLI output is:
root@b7e209a2660b:/ff2# godot ./godot/project.godot --export "HTML5"
Godot Engine v3.4.4.stable.mono.official.419e713a2 - https://godotengine.org

Mono: Log file is: '/root/.local/share/godot/mono/mono_logs/2022-06-18_22.41.56_17.log'
mono_project_debug_build: begin: Building project solution... steps: 1
        mono_project_debug_build: step 0: Building project solution
mono_project_debug_build: end
ERROR: Failed to export project: Failed to build project
   at: godot_icall_GD_pusherror (modules/mono/glue/gd_glue.cpp:249)
System.Exception: Failed to build project
  at GodotTools.Export.ExportPlugin._ExportBeginImpl (System.String[] features, System.Boolean isDebug, System.String path, System.Int32 flags) [0x0007e] in <cf61cc81be674d2a9f24e0a47c139a44>:0
  at GodotTools.Export.ExportPlugin._ExportBegin (System.String[] features, System.Boolean isDebug, System.String path, System.Int32 flags) [0x0000b] in <cf61cc81be674d2a9f24e0a47c139a44>:0
savepack: begin: <... truncated ...>
savepack: end

@TheKetrab Since this is an old issue you may not have this issue anymore, can you still reproduce it in a more recent version of the engine?
You also mention you are using Mono version 6.8, we currently use 6.12 to build the engine so an older version probably won't work.

@default-kramer Your error doesn't seem related to me, I can't see any mention of mscorlib.dll.so in the error log you provided. You should probably open a separate issue and fill the issue template.
What I can tell you from the error log is that the C# ExportPlugin is throwing an exception which could be due to many reasons so I'd recommend you to read the Mono log file in the path specified in the error log (/root/.local/share/godot/mono/mono_logs/2022-06-18_22.41.56_17.log) which may contain more detailed information about what went wrong.
However, when a project fails to export is usually because the C# project does not compile so also try exporting the game using the non-headless version of the engine if you can which will open error dialogs that may be more useful.

@raulsntos Sorry, I didn't include the mono log in my original post. Here is an excerpt from it:
root@dbcc163a3192:/ff2/godot# cat /root/.local/share/godot/mono/mono_logs/2022-06-18_23.49.17_16.log
Config attempting to parse: '/usr/local/bin/GodotSharp/Mono/etc/mono/config'. (in domain Mono, info)
Config attempting to parse: '/root/.mono/config'. (in domain Mono, info)
Image addref mscorlib[0x7bfbca0] (asmctx DEFAULT) -> /usr/local/bin/GodotSharp/Mono/lib/mono/4.5/mscorlib.dll[0x7bfa910]: 2 (in domain Mono, info)
Prepared to set up assembly 'mscorlib' (/usr/local/bin/GodotSharp/Mono/lib/mono/4.5/mscorlib.dll) (in domain Mono, info)
AOT: image '/usr/local/bin/GodotSharp/Mono/lib/mono/4.5/mscorlib.dll.so' not found: /usr/local/bin/GodotSharp/Mono/lib/mono/4.5/mscorlib.dll.so: cannot open shared object file: No such file or directory (in domain Mono, info)
AOT: image '/usr/local/bin/GodotSharp/Mono/lib/mono/aot-cache/amd64/mscorlib.dll.so' not found: /usr/local/bin/GodotSharp/Mono/lib/mono/aot-cache/amd64/mscorlib.dll.so: cannot open shared object file: No such file or directory (in domain Mono, info)
Assembly mscorlib[0x7bfbca0] added to domain GodotEngine.RootDomain, ref_count=1 (in domain Mono, info)
Assembly mscorlib[0x7bfbca0] added to domain GodotEngine.Domain.Scripts, ref_count=2 (in domain Mono, info)
Image addref GodotSharp[0x831ada0] (asmctx DEFAULT) -> /ff2/godot/.mono/assemblies/Debug/GodotSharp.dll[0x7ca5e00]: 2 (in domain Mono, info)
Prepared to set up assembly 'GodotSharp' (/ff2/godot/.mono/assemblies/Debug/GodotSharp.dll) (in domain Mono, info)
Assembly GodotSharp[0x831ada0] added to domain GodotEngine.Domain.Scripts, ref_count=1 (in domain Mono, info)

<... truncated, available on request ...>

I still feel like your issue is different since he's trying to compile the engine and you are trying to export a game project. Also, even though OP didn't specify I feel like they are using a commit of the master branch (which will be Godot 4.0) since they specified the name of the Godot binary is godot.**linuxbsd**.tools.64.mono (and in 3.x it would have been godot.**x11**.tools.64.mono). From your error log I know you are using 3.4.4 and given the many differences between the 3.x branch and the master branch I feel like it'd be better if you reported your issue separately.

Aside from that, after looking more closely to both of your mono logs it seems Godot actually does find the mscorlib assembly for both of you since the assembly is added to the engine's domain, Godot is simply trying to look for the mscorlib assembly using various paths (and if I remember correctly it tries both mscorlib.dll and mscorlib.dll.so) so it seems it eventually finds it after two tries.
The issue OP is having has to do with the C# bindings generator which is implemented in C++ and now I'm sure the issue is unrelated to yours since that C++ code will never be executed in your case.

@raulsntos Thanks, I believe you are correct that the message about the missing mscorlib.dll.so is not actually a problem. I just saw it in my log, wondered if it was a problem, found this issue via Google, and jumped to a conclusion. I will do more investigation and open a new issue if needed.


Rewrite in virtualsvc not working when authpolicy implemented

Bug Description
I am following some of the instructions in https://github.com/istio/istio/issues/40579 to setup Istio with an custom oauth2 provider with keycloak.
I have a main ingress which is sending all the traffic on one host to istio-ingressgateway
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: istio-ingress-main
  namespace: istio-system
  spec:
  ingressClassName: nginx
  tls:
    - hosts:
        - mlp.prod
      secretName: mlp-tls
  rules:
  - host: mlp.prod # A FQDN that describes the host where that rule should be applied
    http:
      paths:                            # A list of paths and handlers for that host
      - path: /
        pathType: Prefix
        backend:                      # How the ingress will handle the requests
          service:
            name: istio-ingressgateway # Which service the request will be forwarded to
            port:
              number: 80           # Which port in that service

My ingress gateway is defined as below
apiVersion: networking.istio.io/v1alpha3
kind: Gateway
metadata:
  name: prod-gateway
  namespace : istio-system
spec:
  selector:
    istio: ingressgateway
  servers:
    - port:
        number: 80
        name: http
        protocol: HTTP
      hosts:
        - 'mlp.prod'

One of my services is mlflow which is installed in mlflow namespace for which the virtual service is defined as below
apiVersion: networking.istio.io/v1alpha3
kind: VirtualService
metadata:
  name: gateway-vs-mlflow
  namespace: mlflow
spec:
  hosts:
    - '*'
  gateways:
    - istio-system/prod-gateway
  http:

    - match:
      - uri:
          prefix: "/mlflow"
      rewrite:
        uri: " "
      route:
      - destination:
          host: mlflow-service.mlflow.svc.cluster.local
          port:
            number: 5000

Now when i try to access the host mlp.prod/mlflow/, I am able to access MLFLOW without any issues and the UI comes up correctly.
However if i try to add an oauth provider in an authpolicy towards the /mlflow route, then I get 404 page not available after the oauth authentication is done
The authpolicy is as in the below
apiVersion: security.istio.io/v1beta1
kind: AuthorizationPolicy
metadata:
  name: oauth-policy
  namespace: istio-system
spec:
  selector:
    matchLabels:
      app: istio-ingressgateway
  action: CUSTOM
  provider:
    name: "oauth2-proxy"
  rules:
  - to:
    - operation:
        paths: ["/mlflow"]

Please assist in this issue. Is the rewrite in Virtual service supposed to work only without authpolicy with oauth2-proxy provider
Kindly help
Thanks,
Sujith.
Version
samuel@samuel-vm-001:~/install/nmlprsinfra/istio/istio-1.15.2/bin$ ./istioctl version
client version: 1.15.2
control plane version: 1.15.2
data plane version: 1.15.2 (8 proxies)
samuel@samuel-vm-001:~/install/nmlprsinfra/istio/istio-1.15.2/bin$ kubectl version --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.24.2
Kustomize Version: v4.5.4
Server Version: v1.22.9
WARNING: version difference between client (1.24) and server (1.22) exceeds the supported minor version skew of +/-1

Additional Information
No response

I was able to resolve this by setting the oauth config with the below new value
upstreams="static://200"
Once this was done the oauth2 started returning 200 for authenticated users and everything was fine.


Failed activation on Windows Server 2012 R2 Standard

Hey,
I'm trying to use activate_instance.ps1 to activate a Windows Server 2012 R2 Standard machine.
I get the following output:
Checking instance license activation status.
WIN-IKQRCC9Q6SQ needs to be activated by a KMS Server.
The variable '$get_product_details' cannot be retrieved because it has not
been set.
At C:\Dan\activate_instance.ps1:184
char:11
+           $get_product_details + '; no supported matches found for GCE.')
+           ~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (get_product_details:String) [
   ], RuntimeException
    + FullyQualifiedErrorId : VariableIsUndefined

Could not get the License Key for the instance. Activation skipped.

This is due to the fact that that on Windows 2012 R2 Standard, the following code
$script:product_name = (Get-ItemProperty -Path $reg -Name ProductName).ProductName
Returns: "Windows Server 2012 R2 Standard"
While the switch that checks for Windows versions checks for "Windows Server 2012 R2 Server Standard". This fails the check and goes to the default clause, that has another bug with an unknown symbol $get_product_details
I haven't checked it for the Non-R2 version, but I suspect it's the same. Notice the two "Server" in string "Windows Server 2012 Server Standard"
Thanks
Dan

Thanks for the PR to fix this. This is a bit of an odd area as we don't officially support anything other than 2008R2, 2012R2 and 2016 Datacenter. I'll merge your PR once you sign the CLA as it is a bug fix though.

New google-compute-engine-sysprep package with fix is in staging repos, will be pushed to stable today or tomorrow.

Thanks!


InputBlock returns class Object instead of parsable dict

while using slack_sdk.models.blocks import InputBlock, the inputBlock returns <slack_sdk.InputBlock> instead of InputBlock
Reproducible in:
slack-sdk==3.18.3

The Slack SDK version
slack-sdk==3.18.3
Python runtime version
Python 3.8.10
Steps to reproduce:
print(InputBlock(label=PlainTextObject(text="test"),
element=PlainTextInputElement(multiline=True, action_id="plain_text_input_action"))
Expected result:
{'element': {'action_id': 'plain_text_input_action', 'multiline': True, 'type': 'plain_text_input'}, 'label': {'text': 'SSH Key', 'type': 'plain_text'}
Actual result:
<slack_sdk.InputBlock: {'element': {'action_id': 'plain_text_input_action', 'multiline': True, 'type': 'plain_text_input'}, 'label': {'text': 'SSH Key', 'type': 'plain_text'}, 'type': 'input'}>

How do I parse this list?

HI @sharvesh06! Whether this class or another, any time you'd like to take a look at what methods/attributes are available on a given class, you can always reach for the built-in dir function:
thing_to_parse = InputBlock(label=PlainTextObject(text="test"),
element=PlainTextInputElement(multiline=True, action_id="plain_text_input_action"))

dir(thing_to_parse)

From here, we can see that there is a to_dict method. If you invoke it:
thing_to_parse.to_dict()

you will get the output you seem to be looking for.
Let us know if that resolves the issue you're facing!

I think that this question is already answered. Let us close this issue but please feel free to write in if you have follow-up questions.


RFC 0587 Envelope based on DIDComm Messaging

Signed-off-by: Troy Ronda<EMAIL_ADDRESS>

negotiating what we support

I proposed #576 a while back. I'm not fond of its details, but the key point is in communicating the supported suites in the OOB invitation when bootstrapping a connection. This can be complemented with the Features Discovery protocol as well (as per the discussion over there).

Will we need a community update approach?

That depends on the community.
SecureKey is comfortable with jumping into this without intermediate steps.

To clarify -- will we need a community update so that other frameworks that are deployed in the wild can move to this?  We need to keep interop going amongst the components for which interop is already working.
@andrewwhitehead - can you please take a look and assess how we would move to this?

@swcurran Here's an example community update:

Implement new envelope format and algorithms.
Add property to OOB to indicate envelope "version" (easier to define) or supported individual "suites" (harder to define).
Frameworks that support the newer version use the newer version when the OOB property indicates the other side does as well.
Transition out of supporting old format. (hopefully not too long of a timeline).

@swcurran I have created a sibling PR: https://github.com/hyperledger/aries-rfcs/pull/595 that adds an accept property to the OOB message. Using the accept property in the OOB message, the acceptable envelope type(s) can be determined by the receiver of an OOB message.

Updated with:

renamed to encryption-envelope-v2.
mime type clarification.
commentary about the DIDComm v2 transition.

Added clarification on the suites required by this RFC and context on the DIDComm Messaging requirements.

It's a bit verbose, but I think another option would be to wrap the current agent message format in a JWM, to avoid having to do content type detection. For example:
{
    "id": "518be002-de8e-456e-b3d5-8fe472477a86",
    "type": "https://didcomm.org/compat/1.0/agent_message",
    "from": "did:example:alice",
    "to": ["did:example:bob"],
    "created_time":<PHONE_NUMBER>,
    "expires_time":<PHONE_NUMBER>,
    "body": {
        "@type": "https://didcomm.org/trust_ping/1.0/ping",
        "@id": "518be002-de8e-456e-b3d5-8fe472477a86",
        "~timing": {
            "out_time": "2018-12-15 04:29:23Z",
            "expires_time": "2018-12-15 05:29:23Z",
            "delay_milli": 0
        },
        "comment": "Hi. Are you listening?",
        "response_requested": true
    }
}

@swcurran @TelegramSam @andrewwhitehead @llorllale I have updated this PR to state that the cty property of the JWE will distinguish DIDComm v1 vs DIDComm v2.

LGTM.


[Multiple File Upload] When using keyboard controls, when uploading or removing a file, focus is lost

This is very distracting to a screen reader user. To solve this we I'll probably add unique ids to all the "Remove" links, then when someone removes a file we'll return a URL containing the id for next/previous item in the list, or if none are in the list returns the ID for adding files. Similarly, when they add a file, we'll return the ID to add files (we could return the ID for the "remove" link for the just-added file, but that would jump them around in the page, so it's probably not the best).

Brainstorming with Melanie some possible fixes:
1- The fix I mention here, we'd have to embed this ID in a URL parameter, because it needs to survive the redirect that we do from add/Remove file to block edit.
2- Use local storage to do a similar trick (A bit weird, but doesn't require messing with our server side routes)
3 - Do something like 1 or 2, but simplify it to always the file upload widget (for both add and remove).
It doesn't seem possible to use hash tags in URLs for this because we do a redirect, that that removes the hash.

HTMX could also be used for the remove issue, but not for add. The reason being that the add ends up going through s3 servers, so every government would have to configure their servers to accept that AJAX call. Remove solely goes through our servers, so in theory could be done with HTMX.


Create insertionSort.py

I have added a python program that sorts the array using insertion sort.

Thank you for merging my code.
Can I add few more sorting algos?

Yeah sure, if you want to add some other algorithm, eg. search, dp,
knapsack, machine learning, etc,(Just make sure to add in appropriate
folder) that too is welcomed.
On Fri, Oct 1, 2021, 7:51 PM Ankan Mahapatra @.***>
wrote:

Thank you for merging my code.
Can I add few more sorting algos?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/MayankShrivastava17/algorithms-python/pull/1#issuecomment-932273560,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHV3I4E5QHAS2SFQQSZMXPLUEW7VLANCNFSM5FEZGRDA
.


LoadBalancer Service Struck in Pending State

Before you raise a new bug, please ensure you have visited the troubleshooting guide

[ ] Visited the troubleshooting guide

Setup Details
FIC Version : 0.1.9
CIS Version : 2.15.0
FIC Build: f5networks/f5-ipam-controller:latest
CIS Build: f5networks/k8s-bigip-ctlr:latest
BIGIP Version: Big IP x.x.x
AS3 Version: 3.x
Orchestration: K8S
Orchestration Version:
Additional Setup details: <Platform/CNI Plugins/ cluster nodes/ etc>
Description
IPAM controller pod is crashing with below message
unable to establish connection to DB, database is locked
Also none of loadbalancer's are getting provisioned. They are struck in pending state and the f5-cis-ctlr logs show below "skipping service '{ servicename namespace}' as its not used by any CIS monitored resource
Steps To Reproduce

Expected Result
successfully provisioned LB with an external IP from the range allocated IPAM
Actual Result
Service struck in pending state
Diagnostic Information
<Configuration files, error messages, logs>
Note: Sanitize the data. For example, be mindful of IPs, ports, application names and URLs
Note: The following F5 article outlines the information required when opening an issue.
https://support.f5.com/csp/article/K60974137

Observations (if any)

@chilu49 Is this issue still valid.

Hi
This issue is now resolved.
This was due to incorrect permission on PVC/PV which was a NFS storage
underneath
Thanks.
On Tue, Apr 2, 2024 at 7:57 AM Trinath Somanchi @.***>
wrote:

@chilu49 https://github.com/chilu49 Is this issue still valid.
—
Reply to this email directly, view it on GitHub
https://github.com/F5Networks/f5-ipam-controller/issues/152#issuecomment-2032281566,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE7BWHVDKV7IW65VA3LC6P3Y3LBMTAVCNFSM6AAAAABETSKC46VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMZSGI4DCNJWGY
.
You are receiving this because you were mentioned.Message ID:
@.***>


Import the MovieLens dataset (MovieLens App)

Tutorial URL: https://developers.sap.com/tutorials/hxe-aa-movielens-03.html
Step 10 - Validate Import
I'm trying to submit the number of rows that have been loaded into the table "ratings" - 100,836
It says the answer is incorrect. I have uploaded the small dataset found here - https://grouplens.org/datasets/movielens/latest/

Hi @stephenkringas,
As stated in step 1:
If you have additional capacity resources, you can run this tutorial series with the larger datasets, but the validations steps implemented were built based on the small dataset.
Which is why I provide links to the dataset files in Step 5: Import the dataset CSV files.
Please make sure to use the provided links instead.
Thanks

clarification added in https://github.com/SAPDocuments/Tutorials/commit/6688e84dda30a1c3b16a10b3adcc74f6c43e09fd

Thanks for the clarification. Regards,
Stephen


The method of writevideo is not hard to push. When writeaudio is added to push, the video cannot be played

The method of writevideo is not hard to push. When writeaudio is added to push, the video cannot be played
writeVideo方法推送视频没有生硬，加入writeAudio推送时视频无法播放
// type = 1,h264 video stream
Log.d("onEncodeResult", type + "   " + length);
if (type == 1) {
rtmpUSB.writeVideo(data, offset, length, Integer.valueOf((int) timestamp));
//                                FileUtils.putFileStream(data, offset, length);
}
// type = 0,aac audio stream
if (type == 0) {
//                                rtmpUSB.writeAudio(data, offset, length, Integer.valueOf((int) timestamp));
}

Thank you for the comment.
We right now cannot add new features or bug fixing for this repo unless there is something critical. I mean we don't have resource for that. Sorry.
If you make a PR, you're welcome.


Evaluation Code

Hi,
Thanks for your nice work on Point Cloud Generation!
Any plan to release your code for evaluation?
Best.

Hi, thanks for your comment.
For the evaluation, you can use your own evaluation code and your own setting.

Hi,
Thanks for your nice work on Point Cloud Generation!
Any plan to release your code for evaluation?
Best.

Hi, thanks for your comment.
For the evaluation, you can use your own evaluation code and your own setting.


Lines with closing braces and commas are marked as not covered

I'm seeing a lot of closing curly braces, parenthesis, commas, etc. being marked as not covered lines, when they should probably be uninteresting lines instead.
For an example, see: https://coveralls.io/builds/12552335/source?filename=src%2Fparser.rs
(Thanks for the great tool, this was the nicest setup I've ever had with getting code coverage!)

I'm aware of this issue. Still mulling it over how I want to deal with it given they're down as lines cause the line is tied to an actual instruction (although could be an end to a block and therefore a not really an executable instruction).
Thanks for the compliment, finding kcov a pain to use is why I did this :)

I've got another case of this here where some assert_eq!( in the tests are being marked as not-covered: https://coveralls.io/builds/12605055/source?filename=src%2Fparse_util.rs#L330

Hi @porglezomp, I've sorted the first part of your issue in a6cdd73e237851dc2b58bf40c6991ac9fb7219bc (well I just realised I missed out the comma but that'll be in the commit afterwards).
Once my testing is done the syntax analysis feature branch will be merged and I'll close this issue and release a new version of tarpaulin. I think the assert_eq! should have been sorted by some other changes between July and now but if not or if you see any other anomalies feel free to let me know :smile:


What is the definition of tol? Why are you setting tol=abs(0.16 * z1 / 1.6) in inference.py?

What is the definition of tol? Why are you setting tol=abs(0.16 * z1 / 1.6) in inference.py?
And I find there is a function to calculate the rotation angle with PCA method, but it seems that it doesn't be called. https://github.com/sunset1995/HorizonNet/blob/master/misc/post_proc.py#L178

@Hanoch666 did you have an answer for this ?


v0.2.7 'kubectl: command not found'

Downloaded 0.2.7
removed app dir in my home dir
re-launched 0.2.7 and created cluster.
...
k8s nodes list:
/Applications/CoreOS k8s Cluster.app/Contents/Resources/first-init.command: line 186: kubectl: command not found
...

Full output here:
https://gist.github.com/grempe/c3312e913e50a3c0a6a4
I also see this same error if I then close all windows and launch OS Shell from menu bar app.
# in os shell launched by app
bash-3.2$ which kubectl
bash-3.2$

No kubectl installed in bin dir:
bash-3.2$ ls -la ~/coreos-k8s-cluster/bin/
total 16888
drwxr-xr-x 6 glenn staff     204 Mar 16 16:29 .
drwxr-xr-x 9 glenn staff     306 Mar 16 16:26 ..
-rwxr-xr-x 1 glenn staff 7378924 Oct 29 10:23 etcdctl
-rwxr-xr-x 1 glenn staff 9066128 Feb 27 14:26 fleetctl
-rwxr-xr-x 1 glenn staff  153888 Mar 16 16:26 gsed
-rwxr-xr-x 1 glenn staff  686304 Mar 16 16:26 wget
bash-3.2$

I then did an updates > Update kubernetes cluster and OS X kubectl from the app menu and saw the following result:
https://gist.github.com/grempe/c2208714e7a9e183f120
This seems to have now installed 'kubectl' locally.  But this script threw an error as well at the very end:
...
/Applications/CoreOS k8s Cluster.app/Contents/Resources/update_k8s.command: line 82: fleetctl: command not found
fleetctl list-units:
/Applications/CoreOS k8s Cluster.app/Contents/Resources/update_k8s.command: line 84: fleetctl: command not found
...

No good deed goes unpunished.  :-)

@grempe thanks for find it out, now it for sure got fixed in v0.2.8

:+1:


Show visibility of storage variables and functions+params in a contract

User Story - As a developer, I'd like to see a summary of public/private variables, functions and parameters are public/private, so that I can confirm my design at a glance.
Function macros require users to specify "private", contrary to param vars in Noir that specify "pub" for public.
Words are clear, but sentiment can be confusing, eg here.
Note: a summary of public/private visibility is not expected to guarantee that no info is leaked, just a confidence check of function/var macros/specifiers at a glance.
Possibly a crude script to start, or an aztec-nargo command.
In the meantime, VSCode users can do Ctrl+K then Ctrl+2 (fold to depth 2), then Ctrl+K then Ctrl+J (to unfold)

@jzaki is this still relevant?


How to find the generated image

Hallo,teacher.I have finished your experiment,but i didn"t see the generated images,how can i view these images?

python visualize_results.py
More details in the README.


Fix nans in measurements list

Closes #56
Major changes:

The dimensionality reduction plugin now only takes rows from the measurements values that have no NaNs. This is done with the pandas DataFrame.dropna() command. At the same time, the information about which labels are included for this is stored.
The add_column_to_layer_tabular_data() is changed to handle this. It now receives a target layer into which the new data should be appended and a new dataframe that should be added. The new dataframe must have one column in common with the already available tabular data. In this case, this is the labels column. The new values are then sorted into layer.properties into the correct rows; the rows with no data from the dimensionality reduction will have a NaN entry.
The clustering procedure has been changed to support appending the cluster index to the data. However, replacing the NaN values is not necessary here as the implemented algorithms (kmeans and hdbscan) can apparently handle NaN and will give these data points the cluster index -1.

Code style changes

I moved the dimensionality reduction and the clustering functions out of the widget. Thus, they can be tested easier.
I added type annotations and default values to the clustering/dimensionality reduction functions as well as default arguments.
I added docstrings for the clustering functions which could make it easier in the future to also use these functions from code.

@Cryaaa : Would you mind giving this a brief look before I change it to ready for review? Tests are passing but...just to be sure.


WP-1803 Release sockjs-dart-client 0.3.2

JIRA: https://jira.webfilings.com/browse/WP-1803
Releases that need to go out at the same time (if any):
JIRA and PR's included in this release:
======= sockjs-dart-client 0.3.2 items =======
HY-4964  - strong mode + lints sockjs_client  - https://github.com/Workiva/sockjs-dart-client/pull/13
CP-2948  - Add smithy.yaml  - https://github.com/Workiva/sockjs-dart-client/pull/12
======= end =======
Diff Between Last Tag and Proposed Release: https://github.com/Workiva/sockjs-dart-client/compare/0.3.1...WP-1803_release_0.3.2
Diff Between Last Tag and New Tag: https://github.com/Workiva/sockjs-dart-client/compare/0.3.1...0.3.2

Raven
Number of Findings: 0

@Workiva/web-platform-pp

QA Resource Approval: +1

[ ] Dev +1's only needed for Release ticket

Ready for merge and tag.


Fixed nidmm_service.cpp build

What does this Pull Request accomplish?
Fixes nidmm_service.cpp build
Why should this Pull Request be merged?
Building grpc-device processes the following error:
D:\Repos\grpc-device\generated\nidmm\nidmm_service.cpp(1428): error C2440: 'static_cast': cannot convert from 'ViReal64' to 'nidmm_grpc::ApertureTime' [D:\Repos\grpc-device\build\SystemTestsRunner.vcxpro
j]

What testing has been done?
Built locally on Windows development machine.

The issue got fixed by another PR


[Typescript] DeepPartial usage is not always correct

I thing that problem is in DeepPartial recursion, when (at the recursion end) it is used for non-object properties.
Examples of such properties are:

any functional type properties (e.g. Palette.getContrastText)
React.CSSProperties['color'] (e.g. TypographyStyle.color)
etc.

Examples:
type TColor = React.CSSProperties['color']
type TColor2 = DeepPartial<TColor>
const c: TColor = 'red' //OK
const c2: TColor2 = 'red' //<<< ERROR

type TProc = () => void
type TProc2 = DeepPartial<TProc>
const p: TProc = () => { }
p() //OK
const p2: TProc2 = () => { }
p2() //<<< ERROR

Yuck! Any ideas how we can fix this?

I am afraid there does not exist generic solution (better DeepPartial) and it will be necessary to use special partial types for whole ThemeOptions type hiearchy. Are you interested in such solution?

Yeah, that's my feeling also. Would you care to submit a PR?

See https://github.com/mui-org/material-ui/issues/9442 pls

Using conditional type can solve the problem. Here is the code I have which solves the problem above.
export type DeepPartial<T> = T extends object
  ? T extends Function ? T : { [P in keyof T]?: DeepPartial<T[P]> }
  : T;

I’m not sure we want to make TS 2.8 a baseline requirement for users of this library yet.

@pelotom Could I please get some clarification regarding theme options?
Specifically, I'm updating the flow types over at flow-typed (https://github.com/flow-typed/flow-typed/pull/2655), and a bit confused about what is an isn't expected behavior regarding custom variable in theme options.
The docs only demonstrate custom variables being allowed at the top level (https://material-ui.com/customization/themes/#custom-variables), however after trawling through some issues I can it's at least reasonably common for people to add custom variables into the palette.
However, if I were to recursively use in-exact types (i.e. allow additional properties/keys everywhere) then it's pretty easy for users to make mistakes specifying what they think are official material-ui properties of theme (breakpoints options etc.), but are actually custom variables they've added in. We end up losing a lot of validation potential, so I'm of the opinion that allowing custom variables everywhere is probably(?) not what the material-ui team had in mind.
My gut feel is allow additional properties at the root, and palette, but nowhere else e.g.
  declare export type PaletteOptions = {
    primary?: PaletteColorOptions,
    secondary?: PaletteColorOptions,
    error?: PaletteColorOptions,
    type?: PaletteType,
    tonalOffset?: number,
    constrastThreshold?: number,
    common?: $Shape<CommonColors>,
    grey?: $Shape<Color>,
    text?: $Shape<TypeText>,
    divider?: string,
    action?: $Shape<TypeAction>,
    background?: $Shape<TypeBackground>,
    getContrastText?: (background: string) => string,
    [string]: any,
  };

  declare export type Palette = {
    common: CommonColors,
    type: PaletteType,
    contrastThreshold: number,
    tonalOffset: number,
    primary: PaletteColor,
    secondary: PaletteColor,
    error: PaletteColor,
    grey: Color,
    text: TypeText,
    divider: string,
    action: TypeAction,
    background: TypeBackground,
    getContrastText: (background: string) => string,
    augmentColor: (
      color: SimplePaletteColorOptions,
      mainShade?: number | string,
      lightShade?: number | string,
      darkShade?: number | string,
    ) => void,
    [string]: any,
  };

Does that sound about right to you?

@Benjamin-Dobell I think you'll also want to allow customization of the Typography and TypographyOptions at a minimum.
The way we do this in TypeScript without destroying type safety is via module augmentation. We define all "extensible" types as interfaces so as to allow 3rd parties to add properties where applicable, in a principled way which doesn't resort to catchall string indices. I'm not sure if Flow supports a similar mechanism.

@pelotom Thanks for the advice, super appreciated!
To be honest I'm not much of a Flow buff, but I think the "equivalent" (if you could call that) would be to put generics (with constrained parameter types) all over the place through-out the API... not particularly fun 😞


Ensure base images with 8u and 11_07 are removed from Azure.

None of the Azure Application offers should be referring to base images whose skuUrnVersion tuple includes either 8u or 11_07.  After the closure of #317 , none of them are.  This UNIX command can be used to verify this fact.
find arm-oraclelinux-wls arm-oraclelinux-wls-admin arm-oraclelinux-wls-cluster arm-oraclelinux-wls-dynamic-cluster -type f -name "*.json" -exec grep Oracle:weblogic {} \; -print | sed 's/^[[:space:]]*//g' | sort | uniq
None of the output should include 8u or 11_07.
This issue asks the assignee to make it so the following VM images are removed from Azure:
weblogic-141100-jdk11_07-ol76  Oracle       owls-141100-11_07-ol76   Oracle:weblogic-141100-jdk11_07-ol76:owls-141100-11_07-ol76:1.1.4   1.1.4
weblogic-141100-jdk8u251-ol76  Oracle       owls-141100-8u251-ol76   Oracle:weblogic-141100-jdk8u251-ol76:owls-141100-8u251-ol76:1.1.4   1.1.4
weblogic-122130-jdk8u131-ol73  Oracle       owls-122130-8u131-ol73   Oracle:weblogic-122130-jdk8u131-ol73:owls-122130-8u131-ol73:1.1.10  1.1.10
weblogic-122130-jdk8u131-ol74  Oracle       owls-122130-8u131-ol74   Oracle:weblogic-122130-jdk8u131-ol74:owls-122130-8u131-ol74:1.1.5   1.1.5
weblogic-122140-jdk8u251-ol76  Oracle       owls-122140-8u251-ol76   Oracle:weblogic-122140-jdk8u251-ol76:owls-122140-8u251-ol76:1.1.4   1.1.4

I suggest going into Partner Center and clicking the Stop Selling button for the corresponding offers.
I recall there was a time we did this already, but somehow it seems to have become undone.

These have already been marked with deprecated in the Partner Center.


Functionality to automatically create jira issues for accessibility problems reported in suite.

Context of the issue/feature
Enhancement, that aims at adding functionality to AET that would allow automatically creating jira bugs for accessibility issues found on pages in a given suite.
The work would consist of two main parts:

Implement code that realises the functionality of extracting and aggregating accessibility problems data. The scope is all issues in a given suite/raport. The issues would have to be categorised in a way to maintain readibility.
One solution is to categorise issues according to W3C guidelines, i.e by principle and guidelines, and then by tests' urls.
The idea is that e.g  when we would have 20 issues with code 1.2.4, the functionality would create one jira bug and link all the issues in description, rather than create 20 jira bugs.

Implementing OAuth into AET to connect to jira api in order to create accessibility issue jira bugs.

If you have suggestions or objections to the above text, please comment.

As the implementation can start with implementing such feature only for the accessibility tests, the design session for that solution should also cover other types of tests (like screenshot, source, w3c ...).
What needs to be designed is te UX part, how user can interact with AET to request creating jira tickets.
Is new report view required for that or this feature could somehow fit in the existing report?

@Skejven As far as I am informed, one suggestion is for a button to be placed somewhere in the upper area of the page, which would allow to create jira bugs for the suite, although I'm not sure whether it should be necessary to give user possibility to create bugs for a specific type of error only (screenshot, source, accessibility etc..), and how it would actually look like.

There are few options now, you can propose some solution i the comment and we can discuss about it here or you can organize a design session with this improvement stakeholders.
I'm just saying that this feature is much more than just raising a jira bug for accessibility. Even if the implementation will start only with accessibility part implemented, I'd like to see the solution for other types of checks.

2 features worth considering:

what about adding labels to jira automatically like accessibility or aet (we should think about the place where the user could define them)?
when the issues are reported to JIRA it would be good to get a list of them displayed somewhere in AET

Perhaps it's a good idea to mark somehow issues already raised and those new ones - it can be correlated to the improvement raised at 377.


[Bug]: Conflicting symlinks found when attempting to create venv.

What happened?
Using version 0.7.3 I am seeing
-----------------------------------------------------------------------------
Error:   × Unable to run command:
  ├─▶ Conflicting symlinks found when attempting to create venv. More than one
  │   package provides the file at these paths
  ├─▶ pip_deps_mcap_protobuf_support/site-packages/tests/test_writer.py
  ╰─▶ site-packages/tests/test_writer.py

These are the files
./bazel-voxel/external/pip_deps_mcap/site-packages/tests/test_writer.py
./bazel-voxel/external/pip_deps_mcap_protobuf_support/site-packages/tests/test_writer.py

In a typical pip installation the second file overwrites the first one.
Version
Development (host) and target OS/architectures: Linux x86-64
Output of bazel --version: bazel 7.1.0
Version of the Aspect rules, or other relevant rules from your
WORKSPACE or MODULE.bazel file: 0.7.3
Language(s) and/or frameworks involved:
How to reproduce
Adding these pip packages

mcap==0.0.15
mcap-protobuf-support==0.0.9

### Any other information?

_No response_

In a typical pip installation the second file overwrites the first one.

Are the file contents checked before overwriting, or only the name taken into consideration? If so, then which one wins? My concern is that the ordering of the dependencies matter here, and rules_py will need to ensure the same dependency ordering as pip, which may not be possible.
In this case, both those libraries should not be shipping test cases into the distribution package.

I get a similar error
Error:   × Unable to run command:
  ├─▶ Conflicting symlinks found when attempting to create venv. More than one
  │   package provides the file at these paths
  ├─▶ rules_python~~pip~rules_ros_pip_deps_38_pyyaml/site-packages/yaml/
  │   loader.py
  ╰─▶ site-packages/yaml/loader.py

Allowing these conflicts and printing a warning seems like a reasonable approach.

Is the intended solution here to use package_collisions='ignore' any time this happens? Or should we wait for a long term fix?
(I am hitting this when installing pandera[polars] and polars).


Show line numbers in diffs

Showing line numbers in a diff file is tricky as each line can have a different number before and after the patch is applied.
See for example, this diff from the example.diff file:
diff --git a/bin/main.ml b/bin/main.ml
index e03c4a4..cb71f31 100644
--- a/bin/main.ml
+++ b/bin/main.ml
@@ -1,7 +1,9 @@
 open Diffcessible

 let main () =
-  Interactive_viewer.start ()
+  let s = In_channel.input_all In_channel.stdin in
+  let patch = Patch.to_diffs s in
+  Interactive_viewer.start patch

 open Cmdliner

The line numbers are start at 1 in both files, as indicated by the @@ line, but then diverges due to there being more added lines than removed lines.
The line numbers are as follow:
1  1  open Diffcessible
2  2
3  3  let main () =
4    -  Interactive_viewer.start ()
   4 +  let s = In_channel.input_all In_channel.stdin in
   5 +  let patch = Patch.to_diffs s in
   6 +  Interactive_viewer.start patch
5  7
6  8  open Cmdliner
7  9

I would like to work on this @Julow. Please assign this to me.

Alright, this issue is assigned to you :)


Sync upstream master to github fork

Sync upstream master to github fork

@rhettg Do you have admin access to this repo?
Would you be able to grant write access to @github/engineering and admin access to @github/observability?

Would you be able to grant write access to @github/engineering and admin access to @github/observability?

Done


CSRNG dv docs and csrng_agent

CSRNG documentation and created/integrated files for csrng_agent. It compiles but most of the internals of the csrng_agent files are still TODO but that will come in future PRs, didn't want this PR to become any larger.

@rasmus-madsen can you please review this PR so that we get CSRNG a bit closer to V1 :)

@rasmus-madsen can you please review this PR so that we get CSRNG a bit closer to V1 :)

CI failed due this error.

UVM_ERROR @    535059 ps: (push_pull_if.sv:114) [ASSERT FAILED] ReadyIsKnown_A

CI failed due this error.

UVM_ERROR @    535059 ps: (push_pull_if.sv:114) [ASSERT FAILED] ReadyIsKnown_A

It appears to be due to the genbits agent if_mode change. I commented it out for the moment and just put a TODO in there cuz I won't have a chance to debug it right now.

It appears to be due to the genbits agent if_mode change. I commented it out for the moment and just put a TODO in there cuz I won't have a chance to debug it right now.


Docker deployment Cluster: web proxy error:  getaddrinfo ENOTFOUND

This started happening recently, so not sure if there was a change that caused it to break:
I'm currently using meteorhacks/cluster in a Docker image that I built that was based off the meteorhacks/meteord.  After a recently deploy, I noticed that the site will now constantly go up and down and log out Cluster: web proxy error:  getaddrinfo ENOTFOUND pretty regularly (even if no one is on the site doing anything).  Not sure what could be causing this, but as soon as I removed cluster, all was well again.  Some insight would be great

So just a minor update: I just updated from 1.6.3 -> 1.6.9 and still getting the same error.  Just says Cluster: web proxy error:  Connection droped now.
Still has the symptoms where it'll intermittently error and cause the meteor app to return 502's with a few actual page loads in between.

So woke up today and spun up a new environment exactly the same with fresh instances; seemingly whatever was going on is no longer happening....so closing this for now.  Perhaps I grabbed a busy instance?

This is actually still happening and seemingly more often


Contributor.md update (Phase4-BTC Miami Work)

From our call, update your file to resemble this Contributing file in our dwn-sdk-js repo. You should be able to primarily copy and paste and add some tweaks to tailor it to your repo.
Things to include:

Highlight labels they should be aware of 'help-wanted', 'good-first-issue', etc
how to know if it's being worked on WIP or not 'assigned to nobody' filter, navigate them there or add a link like the ex repo
Process to follow when working on an issue
Where they should be posting questions, feature request, etc, discord?

@dayhaysoos I know you're swamped I can write this up if it'll help and you can do a look over and tweak what doesn't work for you?


Feature/json pointers for describing object paths

Purpose
Issue https://openscience.atlassian.net/browse/OSF-4654
Small API docs fix: Use JSON pointers to describe object paths https://tools.ietf.org/html/rfc6901
Changes
Replaces object paths with JSONPointer.
Example:  links.self --> /links/self
Example:   parent.links.related.href --> /parent/links/related/href
Also, added more JSONPointers and some fixed inconsistencies, typos, and grammar issues in docs.

Fixed by https://github.com/CenterForOpenScience/osf.io/pull/4625


PowerToys Run - AI assistant

Description of the new feature / enhancement
Add the functionality to interact with an AI assistant (OpenAI GPT, Google Bard, or something similar) directly within the PowerToys Run interface.
Scenario when this would be used?
This functionality can be used to interact with this function in a more 'human-like' way.
Supporting information
No response

Duplicate of #23143, my bad


Flake8

And beautifulsoup4 no longer used.

Coverage decreased (-0.31%) to 43.58% when pulling c759a60e778ac45847e24992ccc0fec8391bde13 on hugovk:flake8 into 5083a4fc72700f7d3c13e5a21836ec0c26bb201f on unitedstates:gh-pages.

Coverage decreased because one covered line was removed:
from bs4 import BeautifulSoup

Previous build:
79 OF 180
RELEVANT LINES COVERED
https://coveralls.io/builds/2248127/source?filename=scripts%2Fgpo_member_photos.py#L17
This build:
78 OF 179
RELEVANT LINES COVERED
https://coveralls.io/builds/2248149/source?filename=scripts%2Fgpo_member_photos.py#L16

:+1:


Untie the HCS NetworkType from the network name

Currently, the network.NetworkInfo.Type value passed through to HCS (which must be one of network.NetworkType, i.e. hcn.NetworkType values) is simply the network name. This is pretty surprising, and also breaks the CNI spec if you want multiple networks of the same type, as each plugin or plugin-list must have a host-unique name.
It also means none of the example configs work, as they will all complain with something like the below:

hcnCreateNetwork failed in Win32: Invalid JSON document string. (0x803b001b) {"Success":false,"Error":"Invalid JSON document string. {{Type,UnknownEnumValue}}","ErrorCode":2151350299}

This appears to be deliberate, but it's not very user-friendly.
The obvious approach seems to me to have each of the executables specify their own NetworkType when they call into core.Core, since the network type used appears to be the only intended distinction between the executables. I started trying to plumb this approach through, but got lost in the abstraction layers, trying to work out how to carry that value without needing to put a hard-coded [string]NetworkType map somewhere in the system, or pull a bunch of stuff into the core package.
Another option would be to expose the network type as an additional parameter, and just produce one executable that covers all the use-cases, which is really what we have now anyway, but be intentional about it.
Another option is to have the executables named to match the NetworkType, and then we can just use config.Type instead of config.Name, and there's little-to-no surprise, as long as no one renames their own executable. Again, this is really just one executable compiled three times, so not really a great approach, I feel.
So I'm interested in guidance and suggestions. This has come up while working on porting BuildKit to Windows, as the only network layer it supports natively is CNI, so I needed minimally-functional CNI with NAT, ala the nat CNI plugin I found here. I used the lattermost option as my workaround, but without renaming binaries it only works for nat, not sdnoverlay or sdnbridge, as those are not NetworkType values.

I'm working on other projects but I'll try and slip this in this week.

@Keith-Mange FYI

Another option is to have the executables named to match the NetworkType, and then we can just use config.Type instead of config.Name, and there's little-to-no surprise, as long as no one renames their own executable. Again, this is really just one executable compiled three times, so not really a great approach, I feel.

FWIW this exact approach was marged in #94 as seen here.

 I wrote this part before the above comment was modified. Looks like it was updated with the same conclusion.
Interesting. Poking around, it doesn't seem that the issue was resolved that the Makefile produces binaries named sdnbridge and sdnoverlay, but those are not documented NetworkMode values. The examples seem to mostly expect binaries named l2bridge and l2tunnel (which aren't output here), but the flannel and dual-stack examples still reference the sdn* names, which I expect will still fail with the error in my original example.
Unless I'm overlooking some point where the sdn* names are translated into the HCN expected enum values... I note that the in-tree test code is not using the executable names, but passing the expected NetworkType values directly into util.MakeTestStruct, so the executable-name part isn't being covered by the test suite.

That said, since we've already gone down this path, perhaps simply changing the Makefile (and whatever else is lying around like the examples) to output executables to match the NetworkType enum is the shortest-path workaround.
Thinking further about it, and also in relation to this comment:
// getOrCreateNetwork
// TODO: Require network to be created beforehand and make it an error of the network is not found.
// Once that is done, remove this function.

perhaps the long-term solution is to complete this TODO, and then unifiy the plugins into a single type wincni. AFAICT from GitHub-browsing the source, if the network already exists, the type from the CNI config file is not actually used, we trust the type of the existing HCN network.
The only place I can see accessing the CNI config's Type is GetHostCompujteNetworkConfig, itself only called during CreateNetwork.
I'm not sure about the history of this comment or the decision to require pre-creating networks (That seems like CNI's role to my limited understanding) so this might not be a viable path. I also admit I don't know off-hand how I would pre-create the NAT network, for example.

Which suggests a different interesting issue: If you pre-create an Overlay network, you can trivially use that network's name in your CNI config, and a network type of nat (or NAT, or nAt, but that's a different issue), and it'll process it as an Overlay network based on the details retrieved from HCN. I suspect this is how this issue has slipped under the radar, if the general users of this repo are pre-creating their networks, then there's no failure with calling the sdnoverlay executable by CNI Config Type sdnoverlay but referencing an Overlay (or L2Bridge, or NAT) network.


Add (de)serialization of key and micro blocks

Closing this and will create a new PR against ng-delelop branch.


feat(monorepo): feat(critical) expose ignore settings in plugin configuration

Hi!
I'm not sure if this is a feature rather than anything else, please let me know if I need to change something.
PR Checklist
Please check if your PR fulfills the following requirements:

[X] The commit message follows our guidelines: https://github.com/scullyio/scully/blob/main/CONTRIBUTING.md#commit
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Other... Please describe:

What is the current behavior?

It's not possible to pass ignore configuration to the critical tool (see https://github.com/addyosmani/critical#critical)
What is the new behavior?
ignore configuration is documented in the plugin options interface and is passed to the critical tool
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I have updated my PR by making the other members of the ignore object optional


gcTrySegmentMergeDelta/gcTrySegmentMerge don't check the expected segments

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)

2. What did you expect to see? (Required)
we check the expected segments in gcTrySegmentMergeDelta/gcTrySegmentMerge.
3. What did you see instead (Required)
we check the abondoned segments in gcTrySegmentMergeDelta/gcTrySegmentMerge.
4. What is your TiFlash version? (Required)
nightly

Confirm this bug does not affect the release-6.1 branch.
https://github.com/pingcap/tiflash/blob/release-6.1/dbms/src/Storages/DeltaMerge/DeltaMergeStore.cpp#L1840-L1845


[broker][bugfix]Return if reset in progress

Motivation

Fix bug. If cursor reset in progress, the callback method will call resetFailed but without return. This will cause the callback be invoked again

Modifications

Just add return after callback invoke resetFailed  If cursor reset in progress

Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

[x] no-need-doc

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks


Release 2021-08-06

WiP

@ihaid this is ready for testing on Staging :)

LGTM
Curious if this is something critical?

LGTM
Curious if this is something critical?

I think this is no problem - this is a side-effect of some fuzzy logic we use to identify contracts that are possibly NEP141 compliant; this happens to be one that isn't actually one but we thought might be.  :+1:


train_dreambooth_lora.py on Colab

Describe the bug
I'm trying to run this script, starting with swapping the default train_dreambooth.py with the lora one in install requirements and run cells. Then I modify the settings to remove all the arguments lora doesn't use and adding the ones it needs, such as instance and class dirs and prompts, since it doesn't use concepts_list. I also change the learning rate to 1e-4. I find that I am able to complete the training, it saves the file and checkpoints but I am not seeing results in inference.
I don't know if it's working. Either the training is not working out or I am running inference the wrong way... To the inference cell I added:
pipe.unet.load_attn_procs(model_path)
pipe.to("cuda")
With the right path to embeds, but am just not seeing results.
Anyone have any suggestions?
Reproduction
null
Logs
No response
System Info
https://colab.research.google.com/github/ShivamShrirao/diffusers/blob/main/examples/dreambooth/DreamBooth_Stable_Diffusion.ipynb

the results in Lora training saves the images to the log folder and can only be seen in WandB.


[FLINK-36600][FLINK-36601][jdbc-driver] Improve Flink SQL JDBC

What is the purpose of the change
Improve Flink SQL JDBC
Brief change log
[FLINK-36600][jdbc-driver] Implement java.sql.Wrapper interface
[FLINK-36601][jdbc-driver] Add JobID property to FlinkResultSet
Verifying this change
This PR adds new unit tests or is cover by existing ones
Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@flinkbot run azure


Run Spring Boot example on Kubernetes

Only for branch sb-2.4.x

We won't execute the test with Kubernetes as it requires third parties and the database application is not compatible with K8s. Closing. FYI @jacobdotcosta


Compiler error when using profile-with-tracing with wgpu-core

wgpu-core uses profiling for instrumentation, and fails to compile if a downstream crate enables the profile-with-tracing feature on a local profiling dependency.
Repro steps

cargo init wgpu-error
Add a wgpu-core = "0.8.1" dependency to Cargo.toml
Add a profiling = { version = "1.0.1", features = ["profile-with-tracing"] } dependency to Cargo.toml
cargo build

This results in a lengthy set of compiler errors, the most prominent of which is failed to resolve: use of undeclared crate or module 'tracing' on each invocation of the profiling::scope macro, and expected type parameter 'B', found struct 'profiling::tracing::level_filters::LevelFilter' where the macro is invoked in a generic context.
The other feature flags don't exhibit this issue. Adding a tracing dependency to the wgpu-error binary crate causes the same outcome, as does using version = "1" for profiling to match its entry in the wgpu-core Cargo.toml, so this doesn't appear to be a local dependency or version mismatch issue.
Given that the tracing dependency is present in profiling as a re-export, and that its presence in wgpu-error makes no difference, I suspect that $crate is actually resolving to the nonexistent wgpu-core::tracing due to some quirk with macro expansion.
Relevant wgpu-core issue:
https://github.com/gfx-rs/wgpu/issues/1477

Published as 1.0.2


Attempt to fix compilation issues with Boost V1.76

Fixes #5666
@diehlpk please try whether this fixes the issue you reported

@hkaiser Thanks, that fixed the bug.

bors merge


Let infer_compression work for UPPERCASE extensions.

Addresses #594.

Happy to write an appropriate test, but would appreciate a kind contributor pointing me to where it should be added.

See fsspec.tests.test_compression . test_infer_custom_compression has some infer tests, but maybe just make a new function test_uppercase_compression with one or two of the standard suffixes.

On Apr 2, 2021, at 16:10, Ben Weeks @.***> wrote:
Happy to write an appropriate test, but would appreciate a kind contributor pointing me to where it should be added.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Added. Thanks for your guidance, @martindurant!


container: move lvm2 package installation

Before this patch, the lvm2 package installation was done during the
ceph-osd role.
However we were running ceph-volume command in the ceph-config role
before ceph-osd. If lvm2 wasn't installed then the ceph-volume command
fails:
error checking path "/run/lock/lvm": stat /run/lock/lvm: no such file or
directory

This wasn't visible before because lvm2 was automatically installed as
docker dependency but it's not the same for podman on CentOS 8.
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>(cherry picked from commit de8f2a9f83194e465d10207c7ae0569700345b9c)

jenkins test centos-container-podman


feat: add slug support for group, org, and projects API

Added slug support for groups, org, and projects APIs.
Usage
//v1beta1/groups/{slug}
//v1beta1/projects/{slug}
//v1beta1/organizations/{slug}

[x] Add Slug support for Get API
[x] Add Slug support for Update API
[x] Add Slug support for Add/Remove Admin(s) API

groups

[x] Add Slug support for List Group Users API
[x] Add Slug support for List User Group Relations API

@niharbansal02 can you please link the issue of this PR?


placeholderImage is empty, no show network image

Hey ,
function sd_setImageWithURL: placeholderImage:
Careless wrong placeholderImageName, The network will not show pictures.

i used the demo project to add a large image url in the detail view controller and the placeholder image showed correctly while the large image was loading. please beware that several large images from the demo project as not working. Add you own url to test

closing this. if you find that its still an issue please reopen


[Workers AI] Add support for deprecated model labels

Summary
Closes #15562
Screenshots (optional)

Documentation checklist

[x] The documentation style guide has been adhered to.
[ ]

Howdy @willswire, closing this out in favor of a #15574, but thanks for flagging and getting us started down this path.


Big List - Reporting Resources

Capturing screenshots of websites.
There are a variety of libraries for doing this. We should add this under remote recon or reporting.
https://github.com/ChrisTruncer/EyeWitness
https://www.trustwave.com/Resources/SpiderLabs-Blog/Using-Nmap-to-Screenshot-Web-Services/
https://github.com/SpiderLabs/Nmap-Tools/blob/master/NSE/http-screenshot.nse
https://github.com/segmentio/nightmare
Python mechanize
https://github.com/breenmachine/httpscreenshot

When handling a large-scale intrusion, incident responders often struggle with obtaining and organizing the intelligence related to the actions taken by the intruder and the targeted organization. Examining all aspects of the event and communicating with internal and external constituents is quite a challenge in such strenuous circumstances.
The following template for a Threat Intelligence and Incident Response Report aims to ease this burden. It provides a framework for capturing the key details and documenting them in a comprehensive, well-structured manner.

https://zeltser.com/cyber-threat-intel-and-ir-report-template/

Enjoyed this talk about the importance of writing when doing threat intelligence. I believe there are lessons to be transferred to risk assessment components of the reporting for SAFETAG. Pen-To-Paper and The Finished Report: The Key To Generating Threat Intelligence - CTI SUMMIT 2017


block zoom functions?

Good morning :)!
Thank you so much for this great useful tool. I have a question,
Can I block the zoom functions on a dashboard?
I searched through the whole documentation and I tried the 'lock' option and xml 'pageScale=0' function but without success.
When using compact dashboards with a few data points on it, the zoom function is very confusing for people when using it on a touch screen devices. Therefore it would be great if I could block all zoom functions for certain dashboards.

Does anybody know a possibility?


Bad lexing when using generic regex ".+"

On logos v 0.11.4 , I can't lex simple characters as token, they are captured by the global regex .+ , even if I use priority :
Test case ;
#[test]
  fn test() {
      use logos::Logos;
      #[derive(Logos, Debug, PartialEq)]
      enum Token {
      #[error]
      ERROR,
      #[token("c", priority = 100)]
      COMMENT,
      #[regex(".+")]
      VALUE,
      ROOT,
  }
  let mut lex = Token::lexer("cc");
  assert_eq!(lex.next(), Some(Token::COMMENT));
  }

This is working as intended, priority is only used to disambiguate patterns, in your case there is no ambiguity for inputs longer than 2. You can fix that by excluding strings that begin with c from your VALUE: [^c].*.

Thanks. There is still something I don't understand then, why is "cc" treated as a VALUE and not 2 COMMENT tokens ?
Is there implicit other rule that tokens should be space (or otherwise) separated ?

Sort of. The way to think about it is that the lexer is trying all currently matching patterns to find the longest possible match. If your input was just a single c then both patterns would match (which is where priority comes in), but since there are still bytes to read, and the second c matches ., that's the token you get.

Thank you for your clear explanations !


Support logical OR in routing rules

Do you want to request a feature or report a bug?
Feature
What did you expect to see?
As a better alternative to segmentation in the consul catalog provider (https://github.com/containous/traefik/pull/3796) I would like to be able to specify multiple rules and have these applied as a logical OR. So for example (not actual syntax):
Host: A;PathPrefix: /A OR Host: B;Pathprefix: /B
As discussed, this should wait for / be implemented by the general refactoring of the routing rule syntax. So this feature request is just a way of tracking that this functionality is implemented.

@hsmade,
Traefik v2 has full support for complex rule trees built with && and ||.
For this reason, I will be closing this issue.


Apache ORC 1.8.0 QA

This activity should track the snapshot and RC builds until we get an official Apache ORC 1.8.0 release.

[x] Dependency Audit (Especially, CVEs)
[x] Test on all supported OSes (Linux/Mac/Windows)
[x] Apache Spark Integration Test
[ ] Apache Iceberg Integration Test
[ ] Apache Arrow Integration Test
[ ] TBD

Thank you for your recent Apache ORC 1.8.0 QA activity. Could you take over this issue to become a release manager of Apache ORC 1.8.0 if you want, @williamhyun ?

Sure!

Thank you, @williamhyun .


Design support for multiple dynamic resource tokens

Refactor AzureData configuration to accept a TokenProvider.  Pseudocode (swift):
// more notes than actual code

public enum TokenPermissions {
   case read
   case readWrite
}

public enum TokenPermissionsLevel {
   case account
   case database
   case collection
   case resource
}

public struct Token {

   static let TokenRefreshSeconds = 600

   let token: String
   let tokenType: TokenType
   let permissions: TokenPermissions
   let permissionLevel: TokenPermissionsLevel
   let resourceLink: String?     // example: dbs/ToDoList/colls/Lists
   let resourceSelfLink: String? // example: dbs/PD5DAA==/colls/PD5DALigDgw=/
   let duration: Double?         // token liftime in seconds
   let expiration: Date?         // private set based on duration

   var isValid: Bool {
       return isMaster || expiration.subtract (Date()).seconds > TokenRefreshSeconds
   }

   var isMaster: Bool {
       return tokenType is .master && permissionLevel == .account
   }
}

public struct TokenProviderConfiguration {

   static let `default`: TokenProviderConfiguration = TokenProviderConfiguration()

   // get readWrite token even for read operations to prevent scenario of
   // getting a read token for a read operation then subsequently performing
   // a write operation on the same resource requiring another round trip to
   // server to get a token with write permissions.
   //
   // if this is set to true, should always request a readWrite token from server
   //
   // default: true
   var alwaysGetWriteToken: Bool = true

   // this specifies the at what level of the resource hierarchy
   // (Database/Collection/Document) to request a resource token
   //
   // for example, if this property is set to .collection and the user tries to
   // write to a document, we'd request a readWrite resource token for the
   // entire collection versus requesting a token only for the document
   //
   // default: .collection
   var tokenResourceLevel: TokenPermissionsLevel = .collection
}

// possibly a TokenDictionary class, and extensions on the Token type for
// comparing and other useful utilities to ensure implementing a TokenProvider
// is as easy as possible

public protocol TokenProvider {

    func GetToken(forResourceLink: String, withPermissions: TokenPermissions) -> Token

    init(config: TokenProviderConfiguration = TokenProviderConfiguration.default)

    init(tokens: [Token], config: TokenProviderConfiguration = TokenProviderConfiguration.default)
}

// AzureData.configure will look more like this:

public func configure (forAccountNamed name: String, withTokenProvider tokenProvider: TokenProvider)

public func configure (forAccountAt url: URL, withTokenProvider tokenProvider: TokenProvider)

// keep the master key configure options for quick start but maybe print to the log that
// these shouldn't be used outside local poc on a non-vital database instance
public func configure (forAccountNamed name: String, withMasterKey key: String, withPermissions: TokenPermissions)

public func configure (forAccountAt url: URL, withMasterKey key: String, withPermissions: TokenPermissions)

Reevaluating, we already have the following resources:

PermissionMode
Permission

No reason to recreate them, as this Permission type matches the Permission type served by Cosmos and the other SDKs (.Net, etc) which will most likely be actually serving the ~tokens~ Permissions.  Let's try to work with the existing Permission type opposed to the TokenPermissions and Token types mentioned above.

Don't want to copy everything here.  Refer to the complementary iOS issue for updated pseudo code and discussion.


What Are the Criteria for “No Blockchain-Related Lists” in Pull Request Guidelines?

Looking at the pull request requirements, there’s a condition stating “No blockchain-related lists.”
What exactly is the criterion for this?
There are already some “blockchain-related” items on the existing awesome list, and it seems like the current PR also includes such items.

There are already some “blockchain-related” items on the existing awesome list

The work was already done with those.


upgrade DockPanelSuite v.3

many changes after 2.9 version to 3.0 https://github.com/dockpanelsuite/dockpanelsuite/compare/Release_2.9...master

Thanks again.


SponsorBlock integration

SponsorBlock is an open-source crowdsourced browser extension and open API for skipping sponsor segments in YouTube videos. It also keeps track of non-music segments for music videos.
The idea would be enqueueing a song, using the API to search for non-music segments (e.g., cinematic parts in music videos, credits or intermissions) and automatically skip them when the playtime enters that segment.
Possible usage

!sponsorblock as an on/off toggle for this feature

Implementation
Triggering the seek action at specific timestamps might raise some issues.

Should the bot periodically (every second or less) fire an event checking whether the song entered a non-music segment?
We could also schedule events for those specific timestamps but we'd have to be extra careful adjusting the scheduling because of time altering commands like !pause and !seek.
...

Just found out yt-dlp supports SponsorBlock and you can remove certain categories. No need for FFmpeg gimmicks.
However, in order to run our own custom yt-dlp commands, we'd need to ditch Restartable::ytdl - instead download the song and pass it to Restartable::ffmpeg.

As an added bonus, supporting custom yt-dlp (or FFmpeg) commands might help us fix #88.

We have to see what Restartable::ytdl does again, with a proper look. If it simply uses yt-dlp for metadata and pipes that to Restartable::ffmpeg then it will be easy.

After experimenting for a few days, this revealed itself to be tougher than initially expected. This is because yt-dlp only does the Sponsorblock step after downloading the video. That's terrible because it now forces us to first download the track, trim it and then stream it (and I guess eventually delete it). Too much hassle for a small niche feature.
If we ever add playback support for local files, then we can revisit this. Closing it for now.


Separate images from JavaScript bundle.

This repo separates the inline images. The asset path is now user configurable, defaulting to versioned hosting via RawGit's production CDN.
Existing EmbedBox users upgrading will see an immediate file size drop without any configuration 😃. EmbedBox now sits at 40kb minified and gzipped!
I believe a version bump to 2.0.0 is appropriate given this alters EmbedBox to include outside resources.
Cheers,
Eric

Replaces JPG assets with PNGs
Fixes some legacy site CSS extract hacks.
Adds asset archive creation to build step.

TODO

[x] Test CDN in merged release.

Closes #124

👍 great job.


Improve the exception message for changing file attributes

Description: We have an issue with removing the ReadOnly file attribute, so we need to add more detailed information about the ArgumentException thrown when trying to set new attributes for a file.
Related docs:

about file attributes

about this exception

where the exception is thrown

Changelog:

src/Agent.Sdk/Util/IOUtil.cs — the SetAttributesWithDiagnostics and GetAttributesAsBinary methods are implemented

Some problems with GitHub. The changes are displayed incorrectly. This PR will be reopened.


Create Map/Dataset dialog for new dashboard

This PR sets up the Create map/dataset dialog that we use on Builder in the new dashboard to be able to launch it within the new dashboard.
Related to https://github.com/CartoDB/cartodb/issues/14313

Two issues found.

Styles are weird

When the map is being created, I can close the dialog. That's not possible in current Builder flow. (according to @jesusbotella )

retest this please

retest this please

According to me as well ( 😂), I was wrong, you can close the dialog in the current dashboard, even if it's weird. So if we want to fix it, we'll have to change current dashboard code.


Update Helm version in doc

This is a one-liner fix for Helm version in doc, in complement of [#10384 ]
Ref: https://github.com/Alluxio/alluxio/pull/10396

This should go into master then can be cherry-picked into 2.1 w/o manual work.

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/6708/
Test PASSed.

alluxio-bot, merge this please

@madanadit Up to you whether we want 0.5.2 to be in 2.1, or maybe patched versions of 2.1.

@jiacheliu3 I've created an issue to track this for the next patch release (https://github.com/Alluxio/alluxio/issues/10422). Unfortunately, updating the doc version before we release a new docker image is tricky for now.


fix: add possibility to filter test files by fileName

Hello,
fix #1797
I have created PR with a new option for "node-vite".
I added to vitest config new property "fileNamePattern", it is similar to "testNamePattern"
It is responsible for skipping test files when the pattern does not match.
I added documentation for "fileNamePattern" next to "testNamePattern" property.
I added additional information for the CLI help option about the new property.
I added a test case to check if the new functionality works as expected.
To test it you can:

test/core/vitest.config.ts -> remove "fileNamePattern" property. -> it will fail because I added failing file-name-pattern.test.ts
pnpm run test -p .test -> it will fail because I added failing file-name-pattern.test.ts

examples/playwright on CI / test (windows-latest, 16).
I think this is not related to my changes.

Hello,
I added a new folder test-skip, which is used to create the test scenario to check if filtering works correctly.
I set  fileNamePattern: '/core/test/',
Due to that it will match all tests inside the test folder, and will not match tests inside the test-skip folder.
I could not find any other method to create a test for this scenario, but I am open to discuss.
According to your review, I have updated the changeFilenamePattern method to filter files array and pass results to this.rerunFiles

According to your review, I have updated the changeFilenamePattern method to filter files array and pass results to this.rerunFiles

No, what I meant is that we don't need this property at all, because we already have include and exclude. When pressing p, we should just pass down correct files to rerunFiles.
I don't mind using regex for this.

@bartoszgolebiowski I pushed some changes to your PR. We don't have tests for stdin right now, unfortunately tho, and creating infrastructure for it is out of scope of this PR, I think. But would be great to have this stuff tested.
I also noticed that after rerunning it doesn't update test count, can you look into that?

Hello @sheremet-va
I added one failing test in the "skip-test" directory and new property "fileNamePattern"
in vitest.config.ts which was responsible for setting an initial value for filter https://github.com/vitest-dev/vitest/pull/1915/commits/93056db8120d83cdc5b8b5c11b32516c3566a273#diff-644652b4f2da0d83159d02c25d7368a9480fe293826cc3314d4f25cd26672ea6L82
This filter runs all tests inside "test" directory and does not run test inside "skip-test" directory.
I can see that the new test file and logic for injecting value are gone.
To resolve it, we can still use https://github.com/vitest-dev/vitest/pull/1915/commits/0fcdd52b6b0ca97f54e819cab88438a69b161bdf#diff-ff14f3092ee007d54e9c9f53fa4044149899b44fa12979cc92f1d182d1a2b057R45 and assigning this value to
https://github.com/vitest-dev/vitest/pull/1915/commits/93056db8120d83cdc5b8b5c11b32516c3566a273#diff-644652b4f2da0d83159d02c25d7368a9480fe293826cc3314d4f25cd26672ea6R26 and add failing test in the "skip-test" folder again.
let latestFilename = String(ctx.config.fileNamePattern || '')

BR
Bartosz Golebiowski

To resolve it, we can still use https://github.com/vitest-dev/vitest/commit/0fcdd52b6b0ca97f54e819cab88438a69b161bdf#diff-ff14f3092ee007d54e9c9f53fa4044149899b44fa12979cc92f1d182d1a2b057R45 and assigning this value to
https://github.com/vitest-dev/vitest/commit/93056db8120d83cdc5b8b5c11b32516c3566a273#diff-644652b4f2da0d83159d02c25d7368a9480fe293826cc3314d4f25cd26672ea6R26 and add failing test in the "skip-test" folder again.

Yes, but this doesn't actually test the interaction that user has - pressing "p" and inserting filter.

The file name filter doesn't work at all. Using vitest 0.25.5.


Fix documentation

https://github.com/mlperf/training/ is inconsistent with actual state of training and real rules. E.g., lists sentiment analysis as a benchmark. Please fix!

Bump. I'd like to get the readme updated so that people can understand what is going on in training! We have new people coming.
Assigning to victor so we can get it done on Thursday and then assign to someone else.


Consistently use tests as the module name for tests (again)

Since we should be consistent, and the idiomatic naming is tests:
https://doc.rust-lang.org/book/ch11-03-test-organization.html#unit-tests
https://doc.rust-lang.org/rust-by-example/testing/unit_testing.html
Fixes new instances added since #191.

@Malax Out of curiosity, do you hand type these test blocks, or use an IDE template? I wondered if it might be the latter that was using test, but it seems the IntelliJ Rust plugin does use tests (presuming this is the plugin you are using)?
https://github.com/intellij-rust/intellij-rust/blob/42deeaea73cb677c2f7f9bf2e5a7ae4191ca5723/src/main/resources/org/rust/ide/liveTemplates/test.xml#L45-L46

@edmorley I don't use a template for that - I just write it out quickly. :)


accept_whitelist and content_type_whitelist VS accept-whitelist and content-type-whitelist

in src/ApiFactory.php , because only accept_whitelist and content_type_whitelist are used, zf-content-negotiation data of non RPC services are not correctly assigned to $service->requestAcceptTypes and $service->requestContentTypes

This repository has been closed and moved to laminas-api-tools/api-tools-documentation; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-documentation/issues/7.


Jshint fixes

Fixed JSHint errors

As per in-person conversation with @ChakriCherukuri, the places where we used the square-brackets notation instead of the dot notation are the cases where we consider that the object has a "dictionary" semantic, i.e. corresponds to data, while the dot notation corresponds to an object semantic.
In order to avoid text editors with built-in linter integration to scream, we could add a /*jshint -W069 */ directive in the JavaScript file.

@sccolbert, what do you think of the above?

I agree, when the keys are not known. Thinking about it from a typings perspective, you type a dict-like object as:
interface MyDict {
  [key: string]: any;
}

But, if you expect certain keys to be present and finite you would type it like this (assuming all are optional):
interface MyOffset {
  value?: number;
  scale?: number;
}

You can also have a combination:
interface MyArgs {
  value?: number;
  scale?: number;
  [key: string]: any;
}

When framing things like that, I would say that if you know the expected structure of the object, treat it like struct with optional properties, rather than a dict. But it you don't know the structure, or need to dynamically query for keys, treat it as a dict.
From what I see in the diff, I would treat the objects like a struct instead of a dict.

Ok, so the following mark attributes

scales
preserve_domain
offset
interactions
[un]selected_styles

reflect dictionaries on the Python side, and I think that we should use square bracket notation to access their attributes.

IMHO it doesn't matter what they reflect on the Python side. They are separate languages. If the keys are more-or-less fixed, it's really more like a struct. If Python had JS-like object literals, you would probably be using those instead of a dict.
Again, this is just my opinion. This is your library :)

Thanks - merging.


run collection FS with 2 meteor app

Hello,
I have a issue when i try to insert image via URL.Basically i have two meteor applications use the same Mongo(mongohq): one for mobile app(1) and one for process data(2). Both of it can upload images. The problem is when app (2) try to upload image, it's crash app (1) and fail to upload.
Here logs on app(1):
W20150412-15:44:41.784(7)? (STDERR)
W20150412-15:44:41.785(7)? (STDERR) stream.js:94
W20150412-15:44:41.785(7)? (STDERR)       throw er; // Unhandled stream error in pipe.
W20150412-15:44:41.786(7)? (STDERR)             ^
W20150412-15:44:41.786(7)? (STDERR) Error: ENOENT, open '<path meteor app>/.meteor/local/cfs/files/_tempstore/images-fWc94zduSt34iKTQx-0.chunk'

Any ideas ? Thanks!

Duplicate of https://github.com/CollectionFS/Meteor-CollectionFS/issues/635
Follow this PR https://github.com/CollectionFS/Meteor-CollectionFS/pull/667

@rhyslbw Here my solution, it maybe help someone:
FS.debug = true;

var imageStore = new FS.Store.S3("images", {
   ...
});

ImagesCollection = new FS.Collection("images_alias", {
    stores: [imageStore],
    filter: {
        allow: {
            contentTypes: ['image/*'] //allow only images in this FS.Collection
        }
    }
});

ImagesOriginCollection = new FS.Collection("images", {
    stores: [imageStore],
    filter: {
        allow: {
            contentTypes: ['image/*'] //allow only images in this FS.Collection
        }
    }
});

ImagesCollection.on('stored', function(fileObj){
    Fiber(function() {
        var r = ImagesCollection.files.findOne(fileObj._id, {
            transform: function(doc){
                return doc;
            }
        });
        ImagesOriginCollection.files.insert(r);
    }).run();
});

Thanks for sharing @particle4dev . @bitomule This might be of use as a workaround

@particle4dev and how does this fixes the problem? You don't share ImagesOriginCollection between servers?
Thanks!

@bitomule I think the problem because CollectionFS use collection.files to tell FileWorker where the uploaded file is stored. When you upload file from app(2), the file doesn't exist on app(1) but FileWorker is still run because they use the same Mongo.
Solution:
App(2)

ImagesOriginCollection
ImagesCollection ( won't affect to app(1) )

App(1)

ImagesOriginCollection

And when the file is uploaded on ImagesCollection, i will copy it to ImagesOriginCollection. Hope that help.

@particle4dev so, on App(2) I always use ImagesCollection on publications and inserts no? App(1) only uses ImagesOriginCollection.

@bitomule use ImagesCollection on insert, still use ImagesOriginCollection on publications. Try it yourself and you will find out what best for you :)

@particle4dev thanks!

@particle4dev tried and doesn't work for me. I use transformWrite and it seems to be working bad with this fix. Also, this won't work if you have multiple instances of App(2) for example and it will create duplicated info in database.
Thank you for your help @particle4dev but I hope we can have a real fix to this issue ;)

@bitomule It work in my case because my app(2) is crawler job. Do you think that you should create a "image" service. It work like a restful service.  Just a idea because i think your app is big. :)

@particle4dev is already some kind of api service, as file is uploaded from iOS app but I need collectionFS in App(1) to get upload progress and client side helpers. I just hope great minds in working on this project can find a way to fix this before we start our iOS app beta :)


Functions in observables aren't typed correctly

Accessing functions in observables is returning a type error saying they aren't callable if the function is added via object spreading:

I can work around this by using
    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
    // @ts-ignore

If I add the function to the observable explicitly (both before and after the object spreading) it doesn't help as the object spreading seems to win out.

Oops, I thought I replied to this earlier! We're working on changing the way functions work and rewrote the types from scratch, which I think should fix this. You could try it in version 2.2.0-next.11 if you'd like? It's part of a pretty big change that may end up becoming a 3.0, so it'll be a little while before release. But I would be interested to know if this is solved in the new version. If it's not, could you share a code snippet that reproduces the problem? Thanks!

Sorry for taking so long to reply. Yes the 2.2.0-next.15 version has fixed this. Thanks.

Although now I'm seeing a different issue:
type ObservableComputed<T = any> = Readonly<ObservableNode<T>>;

I wonder if that's correct? Is the root Observable type ObservableNode or Observable in the rewrite?
I'm currently guessing Observable is but ObservableComputed doesn't seem to use it so that causes issues with my home made selector function that is typed to return Observable types that has a store containing ObservableComputed.

In the current types you might want to use ObservableReadable or ObservableWriteable as a check in selectors? Or can you show me what your selector looks like?

peerDependencies are missing in the next branch/release BTW.

Below is the selector code, we use hooks like useTile etc to allow us to have multiple instances of the same store so we don't need to use IDs or other such nonsense to namespace component instance data. We use contexts to access the stores. We pass in a selector function to the useTile hook to access the part of the store state we are interested in using the useSelector hook.
export function useTile<T>(
  selector: (state: TileStore) => Observable<T> | T
): T {
  const store = useContext(TileContext);

  if (!store) {
    throw new Error("Missing TileContext.Provider in the tree");
  }

  const storeMember = useSelector(() => selector(store));

  if (isObservable(storeMember)) {
    return storeMember.get();
  }

  return storeMember;
}

function isObservable<T>(key: Observable<T> | T): key is Observable<T> {
  return (key as Observable<T>)?.get !== undefined;
}

That's resulting in this error in the next branch:

This is the piece of state in the store:

So something about computeds has changed. They don't seem to be Observables any longer.

I see there is a isComputed function that might help but I can't import it.

What are you doing to functions in the new code? Are you wrapping them so they have the same API as all other observables and you have to call .get() on them to access the underlying function?


[Mod] kRPC for 1.12 by nullprofile

Mod info

Mod name: kRPC
Author(s): djungelorm, nullprofile
License: LGPLv3
Homepage: https://github.com/nullprofile/krpc
Download: https://github.com/nullprofile/krpc/releases

This fork is updated for KSP v1.12, but also includes some changes (mod integrations, mentioned in readme).

Hi @nullprofile, are you interested in having your kRPC fork added to CKAN? The current version in CKAN is from 2018, and your repo has been promoted recently by users on both of the currently active kRPC forum threads.

@Shadlock0133, please let us know if you are able to contact @nullprofile to get approval for this, and we can re-open it.