Hello world —> Hello home?

Hello, I'm writing from 2020, currently, we have a problem called COVID-2019. The world health organization recommends staying at home as one of the recommended measures to prevent a pandemic of COVID-2019.
We, as developers could be part of the advocacy of this recommendation!
Let's have a temporary current branch Hello Home with changing Hello world text in snippets to Hello home involve and all developers around the world to contribute to this change to distribute the importance of this recommendation worldwide?

Hello Sweden ...
I'd say talking, like this, and using voip, messages and everything is more important than ever.

Changing hundreds of examples to replace a word with another word is too big of a scale to be done. However, feel free to fork this repository and transform them yourselves.


path to kdedefaults invalid

The path to kdedefaults is invalid.
(BTW, does this parse as
[Invalid object] [path: //home/<user>/.config/kdedefaults/kdeglobals], or
[Invalid object path:] [//home/<user>/.config/kdedefaults/kdeglobals]?)
$ yin-yang --toggle
No such schema “org.gnome.gedit.preferences.editor”
Successfully applied the color scheme BreathDark to your current Plasma session
QDBusConnection: error: could not send signal to service "" path "//home/<user>/.config/kdedefaults/kdeglobals" interface "org.kde.kconfig.notify" member "ConfigChanged": Invalid object path: //home/<user>/.config/kdedefaults/kdeglobals

If the theme changes correctly, then that is just the bug in lookandfeeltool I reported here.


#1065 Fix handling of reusable bitmaps

closes #1065
So the problem was that on 10 < API < 19 the BitmapPool returned bitmaps for reuse without checking if size is identical, which is a hard requirement before API 19. This was unlikely to show up in production apps since most bitmaps are tiles which have identical sizes anyway.
With this change, we have to trust that the tilesize in the different ITileSources is correct. Determining the size from the file or inputstream would add significant overhead.

awesome work


Fix/duplicate team invite error

This PR will fix the issue reported by @rbr .
The issue with invites not being sent and seeing the error unable to send invites Cannot return null for non-nullable field TeamInvite.id was caused by not raising a GraphQL ExecutionError in the backend. Unless this error is raised it is always expecting a return value in a query. In this case this was the TeamInvite id, which was not there because it wasn't created.
@MeijerM1 I found out that it is done the wrong way in most cases. In many responses the errors field is part of the response, which is not what the Apollo client is looking for. Maybe something for you to look into when you have time.

Somehow the checks for travis are not included in this PR. The build is green however:
https://travis-ci.org/kabisa/kudo-o-matic/builds/649465906?utm_medium=notification&utm_source=github_status


stringtoslug "self"

First off, awesome plugin, thank you for writing it!
I'm trying to have a live slug input field where a user can type anything and have the string automatically stringToSlug-ified. It works, but only with focus blur change events. I would like it to be a little more realtime, so that when a user hits Enter to submit the form the changes are applied. Plus, it's nice visual feedback for the user, too, having the value automatically updated to reflect what the slug will look like.
However, using keypress, keyup and/or keydown as events makes it pretty much unusable. For starters, the plugin trims trailing spaces, so it never gets to replace spaces for dashes. Also, text selection in the input is no longer possible.
I didn't see this particular scenario in the examples. Is it possible?

Hi @appel, you'd like to slug an input while user is typing? So you won't need to save the string, only the slugged?
Well, you can lock spaces for typing with the follow code:
$("#StreamUrl").keydown(function (e) {
     if (e.keyCode == 32) { // space key
       $(this).val($(this).val() + ""); // remove spaces
       return false; // return false to prevent space from being added
     }
});

That did it, thanks!


Use export tag instead of goog.exportProperty

The build file (dist/ol3cesium.js) is the same as before.

Thanks, please merge.


Calling non-private functions that have return values results in unexpected execution

I have a smart contract that has a public method and is expected to be called by users, however I lazily used the same function internally rather than placing the code in a generic private function which is called by both public functions. This resulted in unexpected program behaviour or error.
Example TEALScript:
import { Contract } from '@algorandfoundation/tealscript';

class Demo extends Contract {
    isTrue(value: boolean): boolean {
        return value;
    }

    call(value: boolean): boolean {
        if (this.isTrue(value)) {
            return true;
        }
        return false;
    }
}

In this example the program fails with the following error: logic eval error: retsub executed with stack below frame. Did you pop args?. Details: pc=107, opcodes=concat; log; retsub; label6:
Is it possible to maybe prevent users calling non-private functions if they return values?

Hmm... I'd actually expect this to work. I'll play around with it and see if I can get it working

Actually I realized that this is because ABI subroutines don't technically return anything... they just log it. I think modifying them to dupe the actual value before ABI return is reasonable.

That makes sense. A return would remove everything from the stack except the top value, and a retsub (assuming you're using proto) would also remove anything that's not intended to be returned. Although as I'm thinking about it I don't think there would necessarily be anything unexpected on the stack from duping the return value.


Suggestion's Position

It'd be great if we could choose whether the suggestions appear above / under the textfield. On small phones when the keyboard comes up it is kinda difficult to see the suggestions.

hey @tomcajot , I would like work on it

hey @tomcajot , I would like work on it

I don't really get what you mean?

@tomcajot The issue here is when rendering the suggestions we render downwards, if we render them above the textfield we have to reverse the rendering direction from bottom to top. Is this the functionality you had in mind?

#86


[htlcswitch]: Set AddRef on failed packets

This PR fixes a long-standing, albeit benign, issue within the HTLC switch. The channels and switch maintain persistent references, essentially log pointers, to any htlcs that may need to be reforwarded internally after the switch is restarted or a peer reconnects. In this PR, we fix a instance where one of these references was not set for any packets failed by the switch. The end result is that we would continue to retransmit them internally with each restart/connection and never cleanup these references.
The switch fails all packets using a single method failAddPacket in multiple places. To test that this reference is being set properly, I chose to extend TestChannelLinkMultiHopUnknownNextHop which is one place where this method is used.
In doing so, I noticed that this test was also broken and has now been fixed. Before, the sent payments would never clear Alice's node because we were using Dave's pubkey (which isn't even in the test lol). This would result in handleLocalDispatch returning an error, though the test would succeed because we were expecting the same error from the remote node.
With the tests fixed, we then inspect Bob's forwarding packages to ensure that the AckFilter containing the add Packet has been fully acked. The case here we are specifically testing is when a packet is failed by Bob's switch, and ensuring that the ack is persisted in the forwarding package of the incoming link.
The first commit fixes TestChannelLinkMultiHopUnknownNextHop and adds the assertions that test if the AckFilter has been fully acked. Running the test on this commit will cause the test to fail. The second commit properly sets the AddRef on packets failed by the switch, which causes the test to pass.

This PR is split out from #1698 as it not directly related to cleaning up past references, though it should prevent more dangling references from appearing due the missing AddRefs going forward.

Confirmed as well that without the second commit, the added test in the first commit fails due to a non-fully-acked forwarding package.

Pushed commit to have the mock server drop messages if the link is not found, let's see how it fairs on travis


Allow specifying a query for associations

The documentation suggests that you are allowed to specify a query instead of a schema - when running this locally I always hit:
def related_from_query(queryable, name) do
  raise ArgumentError, "association #{inspect name} queryable must be a schema or " <>
    "a {source, schema}. got: #{inspect queryable}"
end

I have changed the code to support the documentation. I tried writing a test but I couldn't get the integration tests to run:
$ mix test integration_test/cases/assoc.exs

== Compilation error in file integration_test/cases/assoc.exs ==
** (CompileError) integration_test/cases/assoc.exs:4: module Ecto.Integration.Case is not loaded and could not be found
    (elixir) expanding macro: Kernel.use/2
    integration_test/cases/assoc.exs:4: Ecto.Integration.AssocTest (module)

I'm probably doing something wrong - if someone could help me get the tests running I'll add a test showing the functionality seems to be currently broken.
Thanks.

Sorry for closing - I accidentally submitted without a description.

I'm guessing this isn't the full solution because I am getting

Compilation failed because of a deadlock between files.
The following files depended on the following modules:

errors when compiling my project (only after rm -rf _build).
A simplified version of my schemas:
defmodule Post do
  schema "posts" do
    has_many :comments, Comment
    has_one :latest_comment, from(c in Comment, order_by: [desc: c.id])
  end
end

defmodule Comment do
  schema "comments" do
    belongs_to :post, Post
  end
end

I have a feeling the has_one is causing the deadlock but if I comment out belongs_to :post, Post and replace it with field :post_id, :integer I can also get around the problem.

We have recently tried to add this and ran into many issues, so we settled on supporting some keys individually. For example, today you can pass where to your associations. You can see the implementation for that feature here: https://github.com/elixir-ecto/ecto/pull/2520

I saw where and while it's a great addition it doesn't solve my problem. Would you considering a pull request if I added order_by?

Sure. Just keep in mind is a relatively high amount of work. Here is the first PR, which allowed you to pass where via query, before we changed it to pass where as a separate option: https://github.com/elixir-ecto/ecto/pull/2472/files

Maybe it would be easier is someone else implemented it lol. I'm happy to give it a go because I need this functionality. Also has_one seems to have implicit ordering right now - which pg it seems to return the first result. I'm not sure if that is true on all databases though.
The alternative would be a virtual field on Post and I populate with a second query - it's not terrible but it's a bit of work that I'd prefer to not have to do in several spots.

Actually PG has no ordering guarantee, so it may be the first result but it is not guarenteed. MySQL orders by ID, IIRC.


[Feature Request] Collection groups

Background
I'm a heavy user, developer and love the app.
The Problem
I have a screen full* of collections with a screen full* of tasks with a screen full* of todo-items.
* Almost
A Solution
There's only one way to go, and that is up.
There should be a way to implement a collection grouping to organize collections especially when the amount of collections is huge.

It should work around or integrate synchronization. Preferably the latter.
Device calendar integration may be impossible for this.

I plan to contribute to this project. I've started contributing with the synchronization guides and will contribute code if there's enough time and the existing community agrees.
I don't plan to save this issue for myself if someone is willing to take it.


emails: Fix email styling regression.

This issue was introduced after we changed the library we use for
inlining CSS into email HTML. For some reason, the styles in
email.css were not applied earlier but were applied after we
migrated to css-inline. With this commit, we have fixed the
regression in email styles.
Fixes: #25083.

Screenshots and screen captures:

According to @zulipbot, it seems like I have not mentioned the issue in the commit, but I actually did. Could it be because one of the commits should explicitly state that it fixes the entire issue instead of just a part of it?

Yeah, that's correct.

Thanks, @timabbott. Fixed commit message accordingly.

Merged, thanks @shameondev! It's possible we should do a bit of a systematic audit for similar CSS quirks; might be worth discussing in Zulip.


mrubyc3

Fix load.c, mruby3 was changed field size of irep->ilen and irep->slen

Fix bytecode padding
Fix mrblib.c using mruby3


drivers: flash: stm32h7: Data to write might not be nbytes aligned

Deal with the case of non-nbytes-aligned write input buffer.
Fixes #59029

@erwango, if the read is unaligned it will never align and it would be needed to memcopy inside the loop. The simplest would be just to always do that.
BTW it is strange that an unaligned end is accepted, shouldn't this be errored out in the range check ?

I think this is a broader problem than just the stm32h7 driver. I am encountering the same on stm32u585xx and it's probably not just limited to those. Do we need to change each driver or is there a place we can make a broader change?

I do not think this is correct fix.

Fully agreed

NVS, where the problem occurs in reported isseu uses write alignment it reads from flash driver (write_block_size).
This means that NVS gets incorrect info on alignment
Instead of fixing this by overwriting data several times, as I suspect happens here, correct write alignment should be reported for subsystems writing to the flash.

On this point, I'd be glad you were right, but this is set as:
https://github.com/zephyrproject-rtos/zephyr/blob/0773cc88c881bce51ae510ec150f03b69501e6d1/dts/arm/st/h7/stm32h7a3.dtsi#L18-L20
which is inline with ref man (RM0455.pdf):
Flash memory programming by 128 bits (user area) and 16 bits (OTP area)
Am I missing something?

I think I am now missing what us happening. The problem is not that the flash operation is unaligned, the problem is that the data is unaligned.
The data that is passed to a flash operation should not be aligned, if the flash driver needs this to be aligned it has to do that itsself.

A little test program:
#include <zephyr/kernel.h>
#include <zephyr/device.h>
#include <zephyr/drivers/flash.h>
#include <zephyr/devicetree.h>

#define FL_OFFSET 0xff800
#define FL_SIZE 0x200
static const struct device *flash_device =
	DEVICE_DT_GET_OR_NULL(DT_CHOSEN(zephyr_flash_controller));

int main(void)
{
	int rc, i;
	char tst_data[32] = "aaaaaaaaaaaaaaaabbbbbbbbbbbbbbbb";
	printk("Hello World! %s\n", CONFIG_BOARD);

	rc = flash_erase(flash_device, FL_OFFSET, FL_SIZE);
	printk("Flash erase %s\n", rc == 0 ? "[OK]" : "[ERR]");
	for (i = 0; i < 16; i++) {
		rc = flash_write(flash_device, FL_OFFSET + i * 16, &tst_data[i], 16);
		if (rc != 0) {
			break;
		}
	}

	printk("Flash write %s at %i\n", rc == 0 ? "[OK]" : "[ERR]", i);
	return 0;
}

This errors for i=1 (on stm32h747i_disco_m7).

@erwango I have tested two of the possible solutions to this issue on STM32H7 series, so please take a look and see if any of these are worth implementing as a fix upstream:

Use memcpy to copy the source data into temporary write-block-size-aligned buffer, and then write it to flash: https://github.com/zephyrproject-rtos/zephyr/compare/main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver

This approach adds an additional space overhead (write block size temporary buffer on stack) and processing overhead (extra memcpy-ing)

Use UNALIGNED_GET macro to make compiler take care of unaligned read access to source buffer: https://github.com/zephyrproject-rtos/zephyr/compare/main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver-v2

This approach adds a slight performance penalty while accessing the source buffer (probably negligible compared to duration of flash write)
I'm also not sure if UNALIGNED_GET macro is available on non-GNU toolchains (it is defined in gcc.h) - other STM32 flash drivers use it (G0, L4 and WB), so compatibility with other toolchains could already be broken for these series

The same issue could also affect G4, L5, U5 and H5 flash drivers (I didn't check it on actual hardware, just inspected the code):

G4
L5/U5/H5

@erwango I have tested two of the possible solutions to this issue on STM32H7 series, so please take a look and see if any of these are worth implementing as a fix upstream:
* Use `memcpy` to copy the source data into temporary `write-block-size`-aligned buffer, and then write it to flash: [main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver](https://github.com/zephyrproject-rtos/zephyr/compare/main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver)

memcpy to intermediate is only needed for unaligned source. In all other cases you can just copy directly from source address. This would give you two paths of execution and increased flash size, but aligned wrights would be faster.

@erwango I have tested two of the possible solutions to this issue on STM32H7 series, so please take a look and see if any of these are worth implementing as a fix upstream:
* Use `memcpy` to copy the source data into temporary `write-block-size`-aligned buffer, and then write it to flash: [main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver](https://github.com/zephyrproject-rtos/zephyr/compare/main...tmilkovic51:zephyr:fix-h7-unaligned-read-in-flash-driver)

memcpy to intermediate is only needed for unaligned source. In all other cases you can just copy directly from source address. This would give you two paths of execution and increased flash size, but aligned wrights would be faster.

I would skip the trial to optimize and standardize on UNALIGNED_GET(), if compilers do not support it it should be added (it is also used in soc_flash_nrf.c). If on some devices it is not needed to do this the macro could be modified to do nothing (for that device).

* This approach adds an additional space overhead (write block size temporary buffer on stack) and processing overhead (extra `memcpy`-ing)

There is already a buffer for that called unaligned_datas so there is no additional space overhead. I don't think there will be a measurable overhead, given the flash write time is around ~5000 CPU cycles. IMHO it's best to keep the code simple (which also means smaller) and avoid too many code paths.

What UNALIGNED_GET basically does is read data, pointer by pointer, through intermediate variable of type the pointer provided to UANLIGED_GET is; so if you just go down to:
https://github.com/zephyrproject-rtos/zephyr/blob/e1efafa31db942f1a2557b967a22da522c4526ee/drivers/flash/flash_stm32h7x.c#L344-L352
And replace
flash[i] = data[i];

with:
uint64_t temp = data[i];
flash[i] = temp;

that should do the same thing, if temp gets aligned on stack to its type.

Superseded by #61549


[docgen] displays "entry" in the function headers of entry functions

"entry" keyword was missing in the function headers
this PR displays the keyword "entry" properly

Motivation
To properly display the "entry" keyword in the function headers
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
cargo test

By the way, it seems that both of the following functions satisfy is_script() and is_entry():
(1) a entry function a regular module
(2) a function in a script module
Is it intended?
Shouldn't it be like only (1) satisfies is_entry(), and only (2) satisfies is_script()?

By the way, it seems that both of the following functions satisfy both is_script() and is_entry():
(1) a entry function in a regular module
(2) a function in a script module
Is it intended?

My understanding is that script function is deprecated and replaced with entry function, so I am okay with is_script and is_entry being semantically equivalent. (and maybe deprecate is_script in the future?)


Support custom annotations/labels

Closes #45 by adding support to custom annotations/labels at the target component level, like:
apiVersion: "io.jaegertracing/v1alpha1"
kind: "Jaeger"
metadata:
  name: "labels-and-annotations"
spec:
  all-in-one:
    labels:
      some-label: some-label-value # this is applied to the deployment, pods, services and ingresses
    annotations:
      some-annotation: some-annotation-value # this is *also* applied to the mentioned objects

Results in:
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  annotations:
    deployment.kubernetes.io/revision: "1"
    prometheus.io/port: "16686"
    prometheus.io/scrape: "true"
    some-annotation: some-annotation-value
  labels:
    app: jaeger
    jaeger: labels-and-annotations
    some-label: some-label-value
  name: labels-and-annotations
  namespace: default
spec:
  selector:
    matchLabels:
      app: jaeger
      jaeger: labels-and-annotations
  template:
    metadata:
      annotations:
        prometheus.io/port: "16686"
        prometheus.io/scrape: "true"
        some-annotation: some-annotation-value
      labels:
        app: jaeger
        jaeger: labels-and-annotations
        some-label: some-label-value
    spec:
---
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  annotations:
    some-annotation: some-annotation-value
  creationTimestamp: 2018-10-08T09:47:33Z
  generation: 1
  labels:
    some-label: some-label-value
  name: labels-and-annotations-query
---
  kind: Service
  metadata:
    annotations:
      some-annotation: some-annotation-value
    creationTimestamp: 2018-10-08T09:47:33Z
    labels:
      app: jaeger
      jaeger: labels-and-annotations
      some-label: some-label-value
    name: labels-and-annotations-agent
---
  kind: Service
  metadata:
    annotations:
      some-annotation: some-annotation-value
    creationTimestamp: 2018-10-08T09:47:33Z
    labels:
      app: jaeger
      jaeger: labels-and-annotations
      some-label: some-label-value
    name: labels-and-annotations-collector
---
  kind: Service
  metadata:
    annotations:
      some-annotation: some-annotation-value
    creationTimestamp: 2018-10-08T09:47:33Z
    labels:
      app: jaeger
      jaeger: labels-and-annotations
      some-label: some-label-value
    name: labels-and-annotations-query
---
  kind: Service
  metadata:
    annotations:
      some-annotation: some-annotation-value
    creationTimestamp: 2018-10-08T09:47:33Z
    labels:
      app: jaeger
      jaeger: labels-and-annotations
      some-label: some-label-value
    name: labels-and-annotations-zipkin

Signed-off-by: Juraci Paixão Kröhling<EMAIL_ADDRESS>

This change is

Sorry I should have provided some more guidance on #45.
I think one of the main drivers for this feature (at the moment) is to potentially replace the tracing helm chart in Istio.
Looking at these values/charts, I believe we need the potential to set labels/annotations on:

a per deployment basis - so for all-in-one there is only a single deployment, so that could exist under the all-in-one/deployment node - but possibly for production we may need to be able to set labels/annotations individually on the separate deployments

within a deployment, the labels/annotations for the deployment may be different to those for the template

a per service basis - this could be independent of the deployment strategy, so could be defined under a service specific node in the CR

we may also want labels/annotations that can be applied across all three services

similarly, for the production strategy, we may want some labels/annotations to be common across all three deployments (but specific to the deployment or template?).

So this is looking more complicated, to be able to provide the fine grained configuration of labels/annotations against the various components.

Allowing custom labels/annotations to be set for services, but not for deployments (or ingresses, or deployment templates) means that we'd expose this kind of detail about how a "jaeger" instance is composed. What are the concrete use cases you'd be looking to solve with the custom annotations/labels?

Thought it might be best to provide a concrete example from Istio (the Istio 1.0.2 istio-demo.yaml). This file is generated as part of the release build, and is used for demo/test purposes. It is also currently possible for users to specific their own set of annotations that would be used on the jaeger-query service - although not clear what specifically those annotations would be used for.
apiVersion: extensions/v1beta1
kind: Deployment
metadata:
  name: istio-tracing
  namespace: istio-system
  labels:
    app: istio-tracing
    chart: tracing-1.0.1
    release: RELEASE-NAME
    heritage: Tiller
spec:
  replicas: 1
  template:
    metadata:
      labels:
        app: jaeger
      annotations:
        sidecar.istio.io/inject: "false"
        scheduler.alpha.kubernetes.io/critical-pod: ""
        prometheus.io/scrape: "true"
        prometheus.io/port: "16686"
    spec:
      containers:
        - name: jaeger
          image: "docker.io/jaegertracing/all-in-one:1.7"
          imagePullPolicy: IfNotPresent
          ports:
            - containerPort: 9411
            - containerPort: 16686
            - containerPort: 5775
              protocol: UDP
            - containerPort: 6831
              protocol: UDP
            - containerPort: 6832
              protocol: UDP
          env:
          - name: POD_NAMESPACE
            valueFrom:
              fieldRef:
                apiVersion: v1
                fieldPath: metadata.namespace
          - name: COLLECTOR_ZIPKIN_HTTP_PORT
            value: "9411"
          - name: MEMORY_MAX_TRACES
            value: "50000"
          livenessProbe:
            httpGet:
              path: /
              port: 16686
          readinessProbe:
            httpGet:
              path: /
              port: 16686
          resources:
            requests:
              cpu: 10m

      affinity:
        nodeAffinity:
          requiredDuringSchedulingIgnoredDuringExecution:
            nodeSelectorTerms:
            - matchExpressions:
              - key: beta.kubernetes.io/arch
                operator: In
                values:
                - amd64
                - ppc64le
                - s390x
          preferredDuringSchedulingIgnoredDuringExecution:
          - weight: 2
            preference:
              matchExpressions:
              - key: beta.kubernetes.io/arch
                operator: In
                values:
                - amd64
          - weight: 2
            preference:
              matchExpressions:
              - key: beta.kubernetes.io/arch
                operator: In
                values:
                - ppc64le
          - weight: 2
            preference:
              matchExpressions:
              - key: beta.kubernetes.io/arch
                operator: In
                values:
                - s390x

As you can see, the deployment and template metadata have different sets of labels, and the template has annotations not on the deployment metadata. (BTW Also note the affinity node, which is something else we would probably need to deal with - as well as other things like resources, priority class name and image pull secrets).
And services
apiVersion: v1
kind: List
items:
- apiVersion: v1
  kind: Service
  metadata:
    name: jaeger-query
    namespace: istio-system
    annotations:
    labels:
      app: jaeger
      jaeger-infra: jaeger-service
      chart: tracing-1.0.1
      release: RELEASE-NAME
      heritage: Tiller
  spec:
    ports:
      - name: query-http
        port: 16686
        protocol: TCP
        targetPort: 16686
    selector:
      app: jaeger
- apiVersion: v1
  kind: Service
  metadata:
    name: jaeger-collector
    namespace: istio-system
    labels:
      app: jaeger
      jaeger-infra: collector-service
      chart: tracing-1.0.1
      release: RELEASE-NAME
      heritage: Tiller
  spec:
    ports:
    - name: jaeger-collector-tchannel
      port: 14267
      protocol: TCP
      targetPort: 14267
    - name: jaeger-collector-http
      port: 14268
      targetPort: 14268
      protocol: TCP
    selector:
      app: jaeger
    type: ClusterIP
- apiVersion: v1
  kind: Service
  metadata:
    name: jaeger-agent
    namespace: istio-system
    labels:
      app: jaeger
      jaeger-infra: agent-service
      chart: tracing-1.0.1
      release: RELEASE-NAME
      heritage: Tiller
  spec:
    ports:
    - name: agent-zipkin-thrift
      port: 5775
      protocol: UDP
      targetPort: 5775
    - name: agent-compact
      port: 6831
      protocol: UDP
      targetPort: 6831
    - name: agent-binary
      port: 6832
      protocol: UDP
      targetPort: 6832
    clusterIP: None
    selector:
      app: jaeger

The same labels are used across each service apart from the jaeger-infra.
So if we couldn't provide this level of customisation, then we would need to find out from the Istio community whether it was important - otherwise we wouldn't be able to provide the operator as an alternative.

I'm closing this one, as I'm not sure this can be reused to solve the requirements.


Improving the performance of TF models for aarch64.

The aim of this PR is to enhance the efficiency of TF models. The following chart showcases the progress achieved through our modifications by presenting a contrast between ACL and Eigen across different NN architectures and shapes.

Hi @penpornk - some performance improvements to the AArch64 build with oneDNN, would you be able to take a look please?


About the latest release

👋 looks like 1.1.0 release was not completed, ie the release notes did not get created. Create this issue to raise some awareness. Thanks!
cc @aarongable
relates to https://github.com/Homebrew/homebrew-core/pull/171334

cc @aarongable

Fixed! Thanks for the note. https://github.com/jsha/minica/releases/tag/v1.1.0

Thanks! Close via https://github.com/Homebrew/homebrew-core/pull/171334


'platform' module conflict with python built-in module

$ cat a.py
import platform
print platform.uname()[0]
$ python a.py
FreeBSD
$ python -m plop.collector a.py
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/runpy.py", line 162, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/usr/local/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/usr/local/lib/python2.7/site-packages/plop/collector.py", line 104, in main
    exec f.read() in globals(), globals()
  File "<string>", line 2, in <module>
AttributeError: 'module' object has no attribute 'uname'

This is because collector pass globals() to user script, but __package__='plop' too.

Thanks, I've fixed this.


Issue with AWS SignatureDoesNotMatch error when generating presigned URLs for filenames with spaces

Hey there,
First off, kudos for the fantastic work on this library!
I've encountered a snag recently while using it. Specifically, I'm encountering an AWS "SignatureDoesNotMatch" error when attempting to generate presigned URLs for filenames containing spaces. Oddly enough, everything works like a charm when the filename doesn't include any spaces.
Here's the relevant code snippet I'm employing:
@default_opts [ttl: 86_400, body_digest: "UNSIGNED-PAYLOAD"]

def get_signed_url(full_object_path, opts \\ @default_opts) do
  opts = Keyword.merge(@default_opts, opts)

  %{
    access_key: access_key,
    secret_key: secret_key,
    region: region,
    bucket: bucket,
    s3_base_url: s3_base_url
  } = aws_config()

  datetime = :calendar.universal_time()
  method = Keyword.get(opts, :method, "GET")
  service = Keyword.get(opts, :service, "s3")
  url = "#{s3_base_url}/#{bucket}/#{full_object_path}" |> URI.encode()

  signed_url =
    :aws_signature.sign_v4_query_params(
      access_key,
      secret_key,
      region,
      service,
      datetime,
      method,
      url,
      opts
    )

  %{
    file_path: full_object_path,
    file_path_signed_url: signed_url,
    file_name: Path.basename(full_object_path)
  }
end

Upon investigation, I've found that if I refrain from calling URI.encode, the library throws an invalid URI error when dealing with filenames containing spaces.
The generated presigned URL for a file like dog copy 2.jpg looks like this:
https://s3-ap-southeast-1.amazonaws.com/<bucket>/dog%20copy%202.jpg?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAWS67RBQH6KEY26HV%2F20240224%2Fap-southeast-1%2Fs3%2Faws4_request&X-Amz-Date=20240224T131209Z&X-Amz-Expires=86400&X-Amz-Signature=5c10426d39b7f897c5c6fe66c7dd712ca5eba301800658250048f8d0a48d96be&X-Amz-SignedHeaders=host

Error when accessing the generated pre-signed URL
<Error>
SignatureDoesNotMatch
<Message>The request signature we calculated does not match the signature you provided. Check your key and signing method.</Message>
<AWSAccessKeyId>AKIAWS67RBQH6KEY26HV</AWSAccessKeyId>
<StringToSign>AWS4-HMAC-SHA256 20240224T131209Z 20240224/ap-southeast-1/s3/aws4_request bbb393cfd4efea6c106efa8a3c0c0d22afbc9a215fe3641cd6843e7b30d915a4</StringToSign>
<SignatureProvided>5c10426d39b7f897c5c6fe66c7dd712ca5eba301800658250048f8d0a48d96be</SignatureProvided>
<StringToSignBytes>41 57 53 34 2d 48 4d 41 43 2d 53 48 41 32 35 36 0a 32 30 32 34 30 32 32 34 54 31 33 31 32 30 39 5a 0a 32 30 32 34 30 32 32 34 2f 61 70 2d 73 6f 75 74 68 65 61 73 74 2d 31 2f 73 33 2f 61 77 73 34 5f 72 65 71 75 65 73 74 0a 62 62 62 33 39 33 63 66 64 34 65 66 65 61 36 63 31 30 36 65 66 61 38 61 33 63 30 63 30 64 32 32 61 66 62 63 39 61 32 31 35 66 65 33 36 34 31 63 64 36 38 34 33 65 37 62 33 30 64 39 31 35 61 34</StringToSignBytes>
<CanonicalRequest>GET /forumfiles/IF23/activities/221831/dog%20copy%202.jpg X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAWS67RBQH6KEY26HV%2F20240224%2Fap-southeast-1%2Fs3%2Faws4_request&X-Amz-Date=20240224T131209Z&X-Amz-Expires=86400&X-Amz-SignedHeaders=host host:s3-ap-southeast-1.amazonaws.com host UNSIGNED-PAYLOAD</CanonicalRequest>
<CanonicalRequestBytes>47 45 54 0a 2f 66 6f 72 75 6d 66 69 6c 65 73 2f 49 46 32 33 2f 61 63 74 69 76 69 74 69 65 73 2f 32 32 31 38 33 31 2f 64 6f 67 25 32 30 63 6f 70 79 25 32 30 32 2e 6a 70 67 0a 58 2d 41 6d 7a 2d 41 6c 67 6f 72 69 74 68 6d 3d 41 57 53 34 2d 48 4d 41 43 2d 53 48 41 32 35 36 26 58 2d 41 6d 7a 2d 43 72 65 64 65 6e 74 69 61 6c 3d 41 4b 49 41 57 53 36 37 52 42 51 48 36 4b 45 59 32 36 48 56 25 32 46 32 30 32 34 30 32 32 34 25 32 46 61 70 2d 73 6f 75 74 68 65 61 73 74 2d 31 25 32 46 73 33 25 32 46 61 77 73 34 5f 72 65 71 75 65 73 74 26 58 2d 41 6d 7a 2d 44 61 74 65 3d 32 30 32 34 30 32 32 34 54 31 33 31 32 30 39 5a 26 58 2d 41 6d 7a 2d 45 78 70 69 72 65 73 3d 38 36 34 30 30 26 58 2d 41 6d 7a 2d 53 69 67 6e 65 64 48 65 61 64 65 72 73 3d 68 6f 73 74 0a 68 6f 73 74 3a 73 33 2d 61 70 2d 73 6f 75 74 68 65 61 73 74 2d 31 2e 61 6d 61 7a 6f 6e 61 77 73 2e 63 6f 6d 0a 0a 68 6f 73 74 0a 55 4e 53 49 47 4e 45 44 2d 50 41 59 4c 4f 41 44</CanonicalRequestBytes>
<RequestId>SJED7CWD0J00AP6N</RequestId>
<HostId>4Mv/pvPxwEZVJkcCc1Oby6DVk/MJiYJjm0uHAGlSraYC9LzIo9qRQA9GZvxmkGCnjY7cUy1zYMI=</HostId>
</Error>

It seems that the encoding of spaces in the filename might be the root cause of this issue.
Would greatly appreciate any insights or solutions to resolve this hiccup.
Cheers!

Hi @Arp-G Thank you and thank you for a very detailed report! I was not involved in the referenced issue.
I can try to replicate this later tonight or early tomorrow. 👌(And look into the referenced issue)
Do you happen to have the generated url without URI.encode ? AWS can be finicky with spaces I've seen this on the past as well. We'll figure it out 👌👍

@Arp-G I've had some fiddling around now and can confirm that spaces are problematic.👌 I noticed that the CanonicalPath is double encoded twice in at least the Erlang version of this: aws_s3_presigned_url. I don't have an Elixir environment hooked up to AWS at the moment but your Elixir snippet does the same URI.encode/1 prior to sending it down towards aws_signature so we should be in the same boat here ⛵
Example: test onno ends up being test%2520onno which is the same as if you were to call URI.encode/1 twice. (Or the Erlang equivalent aws_util:encode_uri(aws_util:encode_uri(<<"test onno">>)).
The reason for this is simple, your snippet calls URI.encode/1 BUT since you're not passing :uri_encode_path: false in the options, aws_signature encodes it a second time at: https://github.com/aws-beam/aws_signature/blob/3eb4f66218bba99c7a9c86793b33f19a9312acdc/src/aws_signature.erl#L253 resulting in a mismatch in the URL and the one that was used to sign the request.
Rather than changing that default behaviour, the solution should be:

Pass :uri_encode_path: false as part of your opts 👌
Fix aws_s3_presigned_url to do the same 👌 (I'll fix 👍)

Awesome, thanks a lot for your quick response and clear explanation about the double encoding problem.
Adding uri_encode_path: false to the options passed to :aws_signature.sign_v4_query_params/8 solved the issue for me.
I am super thankful to people like you who maintain such libraries for free, thanks again 🙇


[FLINK-27399][Connector/Pulsar] Change initial consuming position setting logic for better handle the checkpoint.

What is the purpose of the change
Brief change log
This task modifies the flink-connector-pulsar module, adds some new mechanisms to set the initial consuming position.

Change StartCursor, add new useful methods, and deprecate the confused fromMessageTime() method.
Change StopCursor, and add new useful methods.
Introduce a new SplitAssigner for assigning the splits among the Pulsar readers. Make the partition assignment logic clear and testable.
Change the start position seeking mechanism from Pulsar consumer API to Pulsar admin API. Didn't reset the start position when the topic has a subscription.

Verifying this change
This change is already covered by existing tests, such as:

PulsarSourceITCase
PulsarSourceEnumeratorTest
PulsarOrderedPartitionSplitReaderTest

We add new tests for new partition assign logic:

NormalSplitAssignerTest
SharedSplitAssignerTest

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

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduces a new feature? (yes)
If yes, how is the feature documented? (docs)

@PatrickRen Can you also have a look at this PR? I've understood that this PR should help resolve this blocker test stability FLINK-26721

@MartijnVisser Thank you. I'll review the patch in this week. Actually I ever try to request myself as a reviewer but forget several times >_<

@syhily Can you resolve the latest review comments?

@tisonkun Finally, the ci turns green.


Change default operation name and make it overridable

Operation name for handler spans contains only method name, like GET or POST. This makes trace analysis quite difficult. I changed default operation name to include the path as well: tracing handler on /somePath route will now generate a span with operation name HTTP GET /somePath. I also made it overridable from children classes.
Also changed attribute map key from severMap -> serverMap.

@dimitrijer hi, thanks for the contribution.
Using HTTP paths for operation names isn't a good default value. There is a problem with wildcards e.g. for endpoint /users/:id there would two spans with operation names users/1 and users/2 representing the same operation.  For more see https://github.com/opentracing/specification/blob/master/specification.md#tracer
The best solution is to set the operation name to wildcard or http verb and wildcard, however I am not sure if we can get that from the routing context.
This instrumentation provides also span decorator which allows you to change operation name to any value.

Hi, @pavolloffay!
Thanks for quick review. I can see why setting path is a bad idea. I guess I'll provide wildcard paths as operation names via decorators then.
Is there any danger of span being finished before my decorator changes the operation name (before decorator's onRequest is executed)?

If you can manage to implement a decorator or even directly set the operation name to wildcard in tracing handler that would be awesome!

Is there any danger of span being finished before my decorator changes the operation name (before decorator's onRequest is executed)?

The span shouldn't be finished when calling https://github.com/opentracing-contrib/java-vertx-web/blob/master/opentracing-vertx-web/src/main/java/io/opentracing/contrib/vertx/ext/web/WebSpanDecorator.java#L24

Also, do you think it would be a good idea to pass RoutingContext to decorators instead of routingContext.request()?

There is a little danger when passing routing context e.g. one could call next.

Yeah, that makes sense.
I tried extracting wildcard route with routingContext.currentRoute().getPath(). The problem is that at the time we set context path, either directly or with decorators, no route is matched yet, meaning that routingContext.currentRoute().getPath() will return null. It works fine when actual route is matched, that is, call to that method returns wildcard path when invoked from handler body.
Hm, I'll just make some hacky solution for my use case at this time. If I figure out how to do it properly, I'll let you know here.
Again, thanks for the patience (first time doing OpenTracing here) and thanks for being so responsive.


⚠ ClamAV has degraded performance

In 0dd1b2b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: ClamAV performance has improved in 1828fe1.


Corrected a typo

Docs Build status updates of commit 6f1effc:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Odata-docs/webapi/filter-in-expand.md
:warning:Warning
View
Details

Odata-docs/webapi/filter-in-expand.md

Line 3, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 3, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add output_buffer_size to multicore.Pool

Codecov Report

Merging #305 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #305      +/-   ##
==========================================
+ Coverage   94.51%   94.52%   +0.01%
==========================================
  Files          30       30
  Lines        9376     9393      +17
==========================================
+ Hits         8861     8878      +17
  Misses        515      515


Getting "nan" error

I am trying to implement a simple example with scaled conjugate gradient. This is my code
`
dataset        = [Instance( [0,0], [0] ), Instance( [1,0], [1] ), Instance( [0,1], [1] ), Instance( [1,1], [0] )]
settings       = {
"n_inputs" : 2,
"layers"   : [  (2, sigmoid_function), (1, sigmoid_function) ]
}
network        = NeuralNet( settings )
training_set   = dataset
test_set       = dataset
cost_function  = cross_entropy_cost

scaled_conjugate_gradient(
# Required parameters
network,                     # the neural network instance to train
training_set,                # the training dataset
test_set,                    # the test dataset
cost_function,               # the cost function to optimize

# Optional parameters
ERROR_LIMIT          = 1e-3, # Error tolerance when terminating the learning
max_iterations       = (),   # Regardless of the achieved error, terminate after max_iterations epochs. Default: infinite
print_rate           = 1000, # The epoch interval to print progression statistics
save_trained_network = False # Whether to ask the user if they would like to save the network after training
)

`
The output at each epoch looks something like this:
[training] Current error: nan 	Epoch: 1000
It never changed from nan and I can't figure out why

I'm recently working on this and found the same issue while working on the scaled conjugate gradient. I have modified the library with Python3 and also made a few updates to the data structure based on my understanding of sklearn library. I've found that it could be because the cost function didn't produce meaningful error according to the outputs and targets in the original binary_cross_entropy_cost. I've added another cost function to use while predicting the stock price as below:
def mean_squared_error(outputs, targets, derivative=False, epsilon=1e-11 ):
    if derivative:
        return 2 * (outputs - targets)
    else:
        return np.mean(np.sum(np.square(outputs - targets), axis=1))


Add maven support

Maven support should be added. This requires to add corresponding needles in maven pom file in jhipster generator too.

@atomfrede : hope this https://github.com/jhipster/generator-jhipster/pull/2469 will help u

Definitely! Already started to implement it.


Replace join in metadata computation with is_latest_issue

Here:
https://github.com/cmu-delphi/delphi-epidata/blob/eec9c7e96a4bfc0b0bbe602a504a96f42fa3b651/src/acquisition/covidcast/database.py#L264

brilliant!

brilliant!


Intellij error while analyzing the Angular repository.

Analyzer Feedback from IntelliJ
Version information

IDEA IC-173.4301.25
2.0.0-dev.39.0
IC-173.4301.25, JRE 1.8.0_152-release-1024-b11x64 JetBrains s.r.o, OS Mac OS X(x86_64) v10.13.3 unknown, screens 3840x2160

Exception
Please append the contents of:
file:///private/var/folders/ls/1jj80hm96sb7j_wjh2tb2bnm005b3f/T/report.txt
Dart analysis server, SDK version 2.0.0-dev.39.0, server version 1.20.0, error: Analysis failed: /Users/alorenzen/angular/examples/github_issues/test/issue_body_test.dart
RangeError (byteOffset): Invalid value: Not in range 0..1860, inclusive:<PHONE_NUMBER>
#0      _ByteDataView.getUint32 (dart:typed_data-patch/dart:typed_data/typed_data_patch.dart:4278)
#1      BufferContext._getUint32 (package:front_end/src/base/flat_buffers.dart:72)
#2      StringReader.read (package:front_end/src/base/flat_buffers.dart:663)
#3      _FbGenericList.[] (package:front_end/src/base/flat_buffers.dart:826)
#4      Object

Tried again this morning, and got a similar crash, but on a different file:
Dart analysis server, SDK version 2.0.0-dev.39.0, server version 1.20.0, error: Analysis failed: /Users/alorenzen/angular/_tests/test/core/linker/deferred_view_with_ngmodel.dart
RangeError (byteOffset): Invalid value: Not in range 0..1860, inclusive:<PHONE_NUMBER>
#0      _ByteDataView.getUint32 (dart:typed_data-patch/dart:typed_data/typed_data_patch.dart:4278)
#1      BufferContext._getUint32 (package:front_end/src/base/flat_buffers.dart:72)
#2      StringReader.read (package:front_end/src/base/flat_buffers.dart:663)
#3      _FbGenericList.[] (package:front_end/src/base/flat_buffers.dart:826)
#4      Object

@scheglov (in case you have any ideas)

Looks like a damaged cache.
Remove ~/.dartServer/.analysis-driver and restart Analysis Server.

I did that a few times, once just restarting Analysis Server, the other restarting Intellij completley, and am still getting these errors.
Dart analysis server, SDK version 2.0.0-dev.39.0, server version 1.20.0, error: Analysis failed: /Users/alorenzen/angular/dev/bin/pub.dart
RangeError (byteOffset): Invalid value: Not in range 0..1980, inclusive: 262186
#0      _ByteDataView.getUint32 (dart:typed_data-patch/dart:typed_data/typed_data_patch.dart:4278)
#1      BufferContext._getUint32 (package:front_end/src/base/flat_buffers.dart:72)
#2      StringReader.read (package:front_end/src/base/flat_buffers.dart:663)
#3      _FbGenericList.[] (package:front_end/src/base/flat_buffers.dart:826)
#4      Object

What version of Angular plugin do you use?
@MichaelRFairhurst is this version with the cache race condition fixed?

This isn't a damaged cache...it's fixed but unpublished still... I'll be
publishing this week hopefully.
The problem is from latest intellij using the navigation API in a way that
crashes
On Tue, Mar 20, 2018, 12:06 PM Konstantin Scheglov<EMAIL_ADDRESS>wrote:

What version of Angular plugin do you use?
@MichaelRFairhurst https://github.com/michaelrfairhurst is this version
with the cache race condition fixed?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/32599#issuecomment-374719767,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjWe1-K8hPadve1C-l7PpHgKpXBRKTdks5tgVMngaJpZM4SxFq4
.

@jwren @alexander-doroshko

I should be clearer that the problem was on my end and not the intelliJ
end. But it broke without me noticing at some point since I hadn't tested
it on the EAP.
On Tue, Mar 20, 2018, 12:38 PM Brian Wilkerson<EMAIL_ADDRESS>wrote:

@jwren https://github.com/jwren @alexander-doroshko
https://github.com/alexander-doroshko
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/sdk/issues/32599#issuecomment-374729842,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABjWezo7kHaIntCY7X3uUJ5HoaoQDqTnks5tgVq8gaJpZM4SxFq4
.

Fixed in 0.0.14


插入图片后直接就崩溃了

index.js:3402 Uncaught TypeError: this.props.getContainerNode is not a function
at Image.calcToolbarOffset (index.js:3402)
at Image. (index.js:3197)
at callCallback (react-dom.development.js:16433)
at commitUpdateEffects (react-dom.development.js:16472)
at commitUpdateQueue (react-dom.development.js:16460)
at commitLifeCycles (react-dom.development.js:17364)
at commitAllLifeCycles (react-dom.development.js:18736)
at HTMLUnknownElement.callCallback (react-dom.development.js:149)
at Object.invokeGuardedCallbackDev (react-dom.development.js:199)
at invokeGuardedCallback (react-dom.development.js:256)
at commitRoot (react-dom.development.js:18948)
at react-dom.development.js:20418
at Object.unstable_runWithPriority (scheduler.development.js:255)
at completeRoot (react-dom.development.js:20417)
at performWorkOnRoot (react-dom.development.js:20346)
at performWork (react-dom.development.js:20254)
at performSyncWork (react-dom.development.js:20228)
at batchedUpdates$1 (react-dom.development.js:20443)
at batchedUpdates (react-dom.development.js:2151)
at dispatchEvent (react-dom.development.js:4905)

升级到2.3.1后出现这个
index.js:1 The above error occurred in the  component:
in Image
in Unknown (created by DraftEditorContents)
in div (created by DraftEditorContents)
in div (created by DraftEditorContents)
in DraftEditorContents (created by DraftEditor)
in div (created by DraftEditor)
in div (created by DraftEditor)
in div (created by DraftEditor)

2.3.1也有这个问题？ 截下报错的问题截图看看

我这样插入图片的，报的就是上面那个错误

2.2.10 是正常的

吐槽一下Media本来就是复数形式了就不要加s了

已经修复了哈


ARROW-16006: [C++] Row conversion helpers and example

This PR is meant to improve the ease of converting between Arrow tabular structure and row-based data, both by convenience functions and doc examples.
Summary of Changes

Added abstract arrow::ToRowConverter and arrow::FromRowConverter, based on design discussed in https://docs.google.com/document/d/174tldmQLMCvOtjxGtFPeoLBefyE1x26_xntwfSzDXFA/edit
Migrated arrow::RecordBatchBuilder to use Result. Also added the class to the C++ API docs
Added example of implementing and using the row converters to convert between vectors of rapidjson::Document and Arrow tables.

Not sure if this is related to the appveyor segfault, but some batch sizes produce a segfault:
# segfault for me
rapidjson-row-converter 1000 500

# runs fine
rapidjson-row-converter 1000 800

Just rebased. CI failure is unrelated.
@emkornfield @pitrou Would one of you be willing to review soon?

@wjones127 I don't know if that would be painful or not, but if not, would you like to submit the RecordBatchBuilder API changes in a separate PR that we can merge quickly?

I don't know what the exact requirements for this are, but if performance is not important then do note there are already conversion utilities in stl.h that go back and forth between arrow::Table and std::vector<std::tuple<...>>.

I don't know what the exact requirements for this are, but if performance is not important then do note there are already conversion utilities in stl.h that go back and forth between arrow::Table and std::vector<std::tuple<...>>.

One of the intended use case is designing performant connectors for NoSQL databases, who typically provide results as a vector or iterator of row / document objects. For example, DynamoDB's C++ SDK returns Aws::Map<Aws::String, Aws::DynamoDB::Model::AttributeValue>. Cassandra has a similar class. More generally, there are a lot of existing systems that work off of rows that an adopter of Arrow may need to connect to.
Looking that the utilities in stl.h, one thing I haven't explored is whether arrow::stl::ConversionTraits<T> and arrow::CTypeTraits<T>.

Generally I have mixed feelings about exposing and maintaining this API. On the one hand it can ease writing conversions a bit, on the other hand it doesn't really abstract the task a lot anyway (mostly it seems to translate between iterator <-> record batch sequence <-> table).

I agree that the new API doesn't do that much on the task; the translation it does between iterator, RB, and tables is mostly for convenience. The main thing it does is establish a pattern of converting one batch at a time, and potentially operating on a smaller batch size than a table is currently grouped in for cases where the inner conversion loops performs better with smaller batches.
It's hard to make an API that general enough that handles most of the conversion; that's why most of this PR is an example demonstrating how to use Arrow's VisitXInline methods, type traits, and conversions to C types to implement row conversions. Those features make writing the row-conversions a lot easier, but they aren't well-documented for end users. If we think the new API doesn't add much, I could just fold its logic into the example and this could just be a docs PR 🤷 .

Oops, I didn't actually reply to the use case part.

One of the intended use case is designing performant connectors for NoSQL databases, who typically provide results as a vector or iterator of row / document objects. For example, DynamoDB's C++ SDK returns Aws::Map<Aws::String, Aws::DynamoDB::Model::AttributeValue>. Cassandra has a similar class. More generally, there are a lot of existing systems that work off of rows that an adopter of Arrow may need to connect to.

It would be useful to make this use case easier, but I'm still not sure if it needs to be a new API, vs a code sample or suggestions on how to approach it. row_conversion.h here is very minimal and could frankly just be copy-pasted into an application easily enough without having to make it a public Arrow API. That said I don't necessarily object to having it if it's going to be a common enough pattern and/or if that makes it easy to point to.

I have refactored this PR to only create an example, rebased so that the RecordBatchBuilder changes are separated, and rewrote most of the prose in the example.

Do we also want to put the example in the Cookbook somehow? I feel like it's a bit too long/involved to go there directly, but maybe we can cross-reference it.

Yeah, I can create a link to it somewhere under the "Creating Arrow Objects" section I think.

This LGTM though I'm unsure how many people would actually follow such a long and non-trivial example.

It's a common use case and I tried to add narrative docs that broke up the example to highlight useful utilities. But it is unfortunate we weren't able to find any general abstractions to make this operation short and trivial.
I think this is ready to be merged, unless @emkornfield has any objections?

sorry never got through a full review so if other reviewers are happy with it I'm fine checking it in.  Is there a summary for why we don't think there is a path forward for a generalized method?


Add agnostic int/string logic for time and id

As well as two small additions to readme this mainly adds function converting id to string and time to number.
Added this after scratching my head for better part of an hour debugging and wondering why my notifications weren't firing; Used integer IDs which didn't fire. Convert to string and presto!

Looks good, thanks for the contribution!


How to compile?

My steps:

git clone https://github.com/ReSwift/CounterExample-Navigation-TimeTravel.git
cd CounterExample-Navigation-TimeTravel
carthage update

Got the following error message:

*** Building scheme "ReSwiftRecorder" in ReSwiftRecorder.xcodeproj
** BUILD FAILED **
The following build commands failed:
CompileSwift normal arm64 /Users/zhouhao/Projects/Swift/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/RecordingStore.swift
CompileSwift normal arm64 /Users/zhouhao/Projects/Swift/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistorySliderView.swift
CompileSwift normal arm64 /Users/zhouhao/Projects/Swift/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistoryCollectionViewCell.swift
CompileSwift normal arm64 /Users/zhouhao/Projects/Swift/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistoryView.swift
CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(5 failures)

Hey @zh-wowtv,
would you mind trying the following carthage install command?
carthage update --no-use-binaries --platform ios

The following build commands failed:
CompileSwift normal arm64 /development/github/ios/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/RecordingStore.swift
CompileSwift normal arm64 /development/github/ios/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistorySliderView.swift
CompileSwift normal arm64 /development/github/ios/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistoryCollectionViewCell.swift
CompileSwift normal arm64 /development/github/ios/CounterExample-Navigation-TimeTravel/Carthage/Checkouts/ReSwift-Recorder/ReSwiftRecorder/UI/StateHistoryView.swift
CompileSwiftSources normal arm64 com.apple.xcode.tools.swift.compiler
(5 failures)
I have tried 'carthage update --no-use-binaries --platform ios'. Still unable to kick-off.

@goldenbearkin @zh-wowtv
Really sorry for the slow follow up on this! I just tested this on my end and I know remember how I set up this project. Instead of including the carthage build product I'm only using carthage for fetching the projects.
This means the checkout command is sufficient to get this project to compile:
carthage checkout

I'll make sure to add this to the documentation.
Sorry for the issues, and please let me know if this command worked for you!

@zh-wowtv @goldenbearkin to be more specific, the full command should be
carthage update --no-build && carthage checkout

Could you let me know if this works for you?

@Ben-G  I got the same issue by doing just
carthage update
now after using
carthage update --no-build && carthage checkout
everything worked fine :)


More questions than issues...

Thanks again for making another awesome media server utility.
I had a few questions about the bernard google drive monitorig.
I see it now relies on json files. Does this mean it will not work with "My Drive" since service accounts, as far as I know, can't be granted rights on my drive?
Second, does Drive ID with Bernard mean the actual drive name or the drive ID string, e.g. the team drive ID from rclone?

Autoscan indeed does not work with "My Drive". Instead, it exclusively works with Shared Drives and Service Accounts.
With Drive ID we mean the drive ID string indeed.


Issue with docs.zowe.org/stable/user-guide/configure-zos-system/

Description
The page shows garbled markup (:::notes and :::):

:::notesNotes

in Zowe. :::

Pages to Update
https://docs.zowe.org/stable/user-guide/configure-zos-system/

@ghannington-Rocket -- thanks for bringing this to our attention. Fixing with PR 3556.


failed to isolate traffic between two environments in same cluster-Failed to watch *unstructured.Unstructured: can't watch configmaps.v1.: forbidden

Please describe your use case / problem.
We have migrated our ambassador to emissary ingress 3.5.1. The installation and upgrade is successful.
We are facing an issue in emissary ingress to handle traffic for two different namespaces installed in same cluster?
Our Dev and TEST environment is deployed in same k8s cluster. We have also created two different ambassador id via emissaryyns.yml file and associated this ambassador id with the resoureces related to this i.e- Host,Mapping. But still we are getting
below error:
E0620 13:18:13.187653      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch configmaps.v1.: forbidden
E0620 13:18:14.709696      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch tcpmappings.v3alpha1.getambassador.io: forbidden
E0620 13:18:16.415064      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch kubernetesendpointresolvers.v3alpha1.getambassador.io: forbidden
E0620 13:18:17.183580      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch devportals.v3alpha1.getambassador.io: forbidden
E0620 13:18:17.371023      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch endpoints.v1.: forbidden
E0620 13:18:22.402303      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch ratelimitservices.v3alpha1.getambassador.io: forbidden
E0620 13:18:25.015539      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch modules.v3alpha1.getambassador.io: forbidden
E0620 13:18:25.338950      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch mappings.v3alpha1.getambassador.io: forbidden
E0620 13:18:28.107542      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch ingresses.v1.networking.k8s.io: forbidden
E0620 13:18:31.418093      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch kubernetesserviceresolvers.v3alpha1.getambassador.io: forbidden
E0620 13:18:34.725394      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch authservices.v3alpha1.getambassador.io: forbidden
E0620 13:18:35.489461      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch ingressclasses.v1.networking.k8s.io: forbidden
E0620 13:18:40.297993      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch logservices.v3alpha1.getambassador.io: forbidden
E0620 13:18:41.353277      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch consulresolvers.v3alpha1.getambassador.io: forbidden
E0620 13:18:43.445850      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch tlscontexts.v3alpha1.getambassador.io: forbidden
E0620 13:18:43.791595      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch services.v1.: forbidden
E0620 13:18:46.688648      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch listeners.v3alpha1.getambassador.io: forbidden
E0620 13:18:48.875253      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch tcpmappings.v3alpha1.getambassador.io: forbidden
E0620 13:18:51.229065      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch hosts.v3alpha1.getambassador.io: forbidden
E0620 13:18:52.702868      1 reflector.go:138] pkg/kates/client.go:466: Failed to watch *unstructured.Unstructured: can't watch endpoints.v1.: forbidden
has context menuComposeParagraph
Describe the solution you'd like
We want to run two different emissary for our DEV and TEST environment to serve the traffic in same cluster.
Describe alternatives you've considered
Please provide the steps
Additional context
not required

Team,
any update on this?

@MOUM02 , can you check your annotations and stored version to make sure there's no v1 resources there, as in this issue?  Testing with just one deployment in the cluster does it work without these errors?

Its working with one deployment. Whenever we deployed same service in test with providing the ambassador-id previous one got failed new one works fine

I see that in a previous issue the problem was with RoleBinding. In that case the ClusterRole was defined using an aggregator, and the name of the Role referenced in the binding didn't contain the actual RBAC rules. Fixing this binding to reference the -watch Role made it work.  Can you check your permissions for this cluster?  I think you can do something like kubectl cluster-info dump | grep authorization-mode

Hi @MOUM02 , are you still having issues with this?  We do have a weekly live help session for Emissary on Thursdays at 2:30pm ET.  Please let me know if you'd like to join sometime.


[ODB] root/view.delta gives useless token while reach the end

Hi,
I use /drive/items/root/view.delta to get changes from ODB, everything is ok if I'm using next link to receive changes.
But when I reach the end of my storage, I'll get an useless token like 1;%234;%231;3;64d77af3-ac65-4a82-9162-fd5e71086f0b;636032100807700000;15048766;%23 from the response. If I put this token into a new view.delta request the only response I can get is read timeout.
Is this an issue of onedrive for business?
GET https://cloudqnap-my.sharepoint.com/_api/v2.0/me/drive/root/view.delta params={'top': 1000, 'token': u'1;%234;%231;3;64d77af3-ac65-4a82-9162-fd5e71086f0b;636032100807700000;15048766;%23', 'select': 'id,name,size,fileSystemInfo,webUrl,file,folder,parentReference,lastModifiedDateTime,remoteItem,package,deleted'},headers={}
Encounter error HTTPSConnectionPool(host='cloudqnap-my.sharepoint.com', port=443): Read timed out. (read timeout=60)

Hi @verybada, are you still seeing timeouts? That certainly sounds like a service issue.
If possible, could you increase your timeout to something insane and wait patiently for the query to come back. Hopefully it will eventually, in which case you should be able to grab the request-id header out of the response. We can use that to take a look at what's actually going on.


No way to set function/method body?

Would there be any interest in allowing a body to be set?
PhpMethod::create('__construct')->setBody(function($property) { ?>
    $this->get('<?= $property ?>');
<?php });

I don't fully understand you... yet. Could you help me by sketching the expected output of this?

public function __construct()
{
    $this->get('<property>');
}

where  is whatever is passed to the anonymous function, possibly via passing params added to setBody:
PhpMethod::create('__construct')->setBody(function($property) { ?>
    $this->get('<?= $property ?>');
<?php }, 'whatever');

Then results in:
public function __construct()
{
    $this->get('whatever');
}

You can just do this:
PhpMethod::create('__construct')->setBody('$this->get('. $property .');');

which will result in:
public function __construct() {
    $this->get('whatever');
}

Oh, I must have missed that method.  It looked like there was no way to set it.  Thanks.


After setting the aspect ratio, user should not be able to change it.

If I am setting the aspect ratio to 16:9 , because i want to force user to maintain that aspect ration while selecting the image.
User is able to change the aspect ratio to something else other than 9:16
There should be Boolean variable for example shouldLockAspectRatio , If that variable is set to be true , User should not be able to change the aspect ratio.

I have changed the code and created pull request for the same.

hi @mihirpmehta, please see example,
That was text mistake

Your Pull request was broke master!
Pull request has been removed

You should make pull to get latest source and correct apply your changes


Improved search in the generated website

The search in schedule, tracks, speakers & rooms page was a little faulty.
e.g. If you search "Keynote", then you get the results as soon as you type till "Keyn" and obviously when you type "keynde", you won't get the results. But when you erase the added wrong characters and search string becomes "Keyn", you still don't get results even after writing full search string "keynote", you won't get the results. Here is the gif showing the error. Please open to see the error.

This PR fixes that error.
Before Changes:
https://demolp.github.io/FOSSASIASummit/index.html
After Changes:
https://demolp.github.io/FOSSASIAnew/index.html

Codecov Report

Merging #1303 into development will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##           development    #1303   +/-   ##
============================================
  Coverage        66.08%   66.08%
============================================
  Files                9        9
  Lines             1551     1551
  Branches           244      244
============================================
  Hits              1025     1025
  Misses             526      526

Continue to review full report at Codecov.

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

@Lplenka Can you please specify the issue which you are fixing with this PR?

@geekyd no such issue was reported :sweat_smile: . I found this and went for the PR to solve the issue. The issue is as much I have explained above.

@Lplenka Good job! But, for a number of reasons we require devs to open issues before they make a PR. Please have a look at our best practices here: http://blog.fossasia.org/open-source-developer-guide-and-best-practices-at-fossasia/
Some reasons: It helps to align all developers and keep the communications flowing. Developers can give feedback to specific points and discuss best approaches where relevant. It also helps to keep a documentation of activities. You will probably find a lot of projects where this is done like that and in companies like IBM they also work in that way. So, please make it easy and transparent what is being worked on and open an issue and match this issue in the pull request.
Thanks.

@mariobehling sure, will open an issue now for this.

Well-Done.

Thanks @mariobehling  @geekyd


Support Python version 3.12, drop 3.8

We may need to make changes like these:
https://github.com/ansys/ansys-api-mapdl/pull/11
https://github.com/ansys/pydpf-composites/pull/360/commits/ce41fa42f6411e215c79e341da5a9e419e64ea40

Related #2096


Allow customizable timeout for SQL Server connections

This change allows you to specify a timeout for the SQL Server connection.  The default of 120 seconds can be too short for long running operations such as restoring a large database backup.

Thanks


g-Function becomes negative for coaxial pipe depending on fluid

I've was playing around with a coaxial pipe flows/dimensions, when the g-function suddenly became negative.
G-function is positive for 20 degC water (I think it is a laminar flow in the inner pipe as well as the outer annulus)
It is negative for 60 degC water (I think it is laminar flow in the inner pipe and turbulent in the outer annulus)
I based my code on the examples in the documentation, but I could of course have made mistakes.
Below is my code:
# -*- coding: utf-8 -*-

import numpy as np
import pygfunction as gt
from scipy.constants import pi
import matplotlib.pyplot as plt
from matplotlib.ticker import AutoMinorLocator

# -------------------------------------------------------------------------
#  EXAMPLE OF SETTINGS -
# -------------------------------------------------------------------------

B = 4               # Distance between boreholes (m)
D = 4.0             # Borehole buried depth (m)
r_b = 0.1143 / 2    # Borehole radius (m)
H = 200.            # Borehole length (m)

# ---- DETAILED SETTINGS -----

# g-Function calculation options
nSegments = 8
method = "similarities"
boundary_condition= "MIFT"

Jmultipoles = 4 # Used by pipes module

# Simulation parameters
dt = 3600.                  # Time step (s)
tmax = 100.*8760. * 3600.    # Maximum time (s)

# Load aggregation scheme
cells_per_level = 10 # 5 is defualt value
nSources=1

# Borefield geometry
N_1 = 1           # Number of boreholes in the x-direction (columns)
N_2 = 1             # Number of boreholes in the y-direction (rows)

# Ground properties
T_g = 10.0          # Undisturbed ground temperature (degC)
k_s = 3.1           # Ground thermal conductivity (W/m.K)
Cp_vol = 2160000    # Volumetric heat capacity (J/m3.K)
alpha = k_s/Cp_vol      # Ground thermal diffusivity (m2/s)

# Pipe dimensions and properties:
# pos, r_in, r_out, borehole, k_s, k_g, R_ff, R_fp, J

# Pipe properties
epsilon = 1.5e-6    # Pipe roughness (m)
k_p = 0.42           # Pipe thermal conductivity (W/m.K)

# Pipe positions
# Coaxial pipe (x, y)
pos = (0., 0.)

# Pipe dimensions (coaxial)
# Not: Outer pipe is 0.5 mm smaller than borehole and 0.001 mm thick
r_in_in = 90/2/1000    # Inside pipe inner radius (m)
r_in_out = r_in_in + 6 / 1000    # Inside pipe outer radius (m)
r_out_out = r_b - 0.5 / 1000   # Outer pipe outside radius (m)
r_out_in = r_out_out - 0.001 /1000    # Outer pipe inside radius (m)

# Grout properties
k_g = 1.5           # Grout thermal conductivity (W/m.K)

# Fluid properties
m_flow_borehole = 0.14      # Total fluid mass flow rate per borehole (kg/s)
media = "Water"
Tmedia = 60. # Media average tmperature (degC)

nBoreholes = N_1 * N_2

# g-function
segment_ratios=gt.utilities.segment_ratios(nSegments, end_length_ratio=0.02)
options = {'nSegments': nSegments,
           'segment_ratios' : segment_ratios,
           'disp': True}

# Fluid properties
m_flow_network = m_flow_borehole*N_1*N_2    # Total fluid mass flow rate (kg/s)

fluid = gt.media.Fluid(media, 0, Tmedia)
cp_f = fluid.cp     # Fluid specific isobaric heat capacity (J/kg.K)
rho_f = fluid.rho   # Fluid density (kg/m3)
mu_f = fluid.mu     # Fluid dynamic viscosity (kg/m.s)
k_f = fluid.k       # Fluid thermal conductivity (W/m.K)

# -------------------------------------------------------------------------
# Define a rectangular borehole field
# -------------------------------------------------------------------------

# Rectangular field
boreField = gt.boreholes.rectangle_field(N_1, N_2, B, B, H, D, r_b)
nBoreholes=len(boreField)

# -----------
# Calculate Coaxial BHE  (based on  example in PyG-function):
# ---------------

# Vectors of inner and outer pipe radii
# Note : The dimensions of the inlet pipe are the first elements of
#        the vectors. In this example, the inlet pipe is the inside pipe.
r_inner = np.array([r_in_in, r_out_in])     # Inner pipe radii (m)
r_outer = np.array([r_in_out, r_out_out])   # Outer pip radii (m)

# Pipe thermal resistance
# (the two pipes have the same thermal conductivity, k_p)
# Inner pipe
R_p_in = gt.pipes.conduction_thermal_resistance_circular_pipe(
    r_in_in, r_in_out, k_p)
# Outer pipe
R_p_out = gt.pipes.conduction_thermal_resistance_circular_pipe(
    r_out_in, r_out_out, k_p)

# Fluid-to-fluid thermal resistance
# Inner pipe
h_f_in = gt.pipes.convective_heat_transfer_coefficient_circular_pipe(
    m_flow_borehole, r_in_in, mu_f, rho_f, k_f, cp_f, epsilon)
R_f_in = 1.0 / (h_f_in * 2 * pi * r_in_in)
# Outer pipe
h_f_a_in, h_f_a_out = \
    gt.pipes.convective_heat_transfer_coefficient_concentric_annulus(
        m_flow_borehole, r_in_out, r_out_in, mu_f, rho_f, k_f, cp_f,
        epsilon)

R_f_out_in = 1.0 / (h_f_a_in * 2 * pi * r_in_out)
R_ff = R_f_in + R_p_in + R_f_out_in

# Coaxial GHE in borehole
R_f_out_out = 1.0 / (h_f_a_out * 2 * pi * r_out_in)
R_fp = R_p_out + R_f_out_out

# All borehole collectors in the borefield (Coaxial borehole collector)
CoaxBHCs = []
for borehole in boreField:
    CoaxBHC = gt.pipes.Coaxial(pos,
                               r_inner, r_outer,
                               borehole,
                               k_s, k_g,
                               R_ff, R_fp,
                               J=Jmultipoles)
    CoaxBHCs.append(CoaxBHC)

# Evaluate and print the effective borehole thermal resistance for the first borehole in borefield
R_b = CoaxBHCs[0].effective_borehole_thermal_resistance(
    m_flow_borehole, fluid.cp)

# Visualize the borehole geometry (and save the figure)
CoaxBHCs[0].visualize_pipes()

# Build a network object from the list of UTubes
network = gt.networks.Network(boreField,CoaxBHCs,
                              m_flow_network=m_flow_network,
                              cp_f=cp_f,
                              nSegments=nSegments,
                              segment_ratios=segment_ratios)

# Simulation parameters
Nt = int(np.ceil(tmax/dt))  # Number of time steps
time = dt * np.arange(1, Nt+1)

# Load aggregation scheme
LoadAgg = gt.load_aggregation.ClaessonJaved(dt, tmax,
                                            nSources=nSources,
                                            cells_per_level=cells_per_level)

# Get time values needed for g-function evaluation
time_req = LoadAgg.get_times_for_simulation()

# -------------------------------------------------------------------------
# Calculate g-function
# -------------------------------------------------------------------------

# Calculate g-function
gFunc = gt.gfunction.gFunction(network, alpha,time=time_req,
                               method=method, boundary_condition=boundary_condition,
                               options=options)

gFunc.visualize_g_function()

print(f'Coaxial Borehole thermal resistance: {R_b:.4f} m.K/W')

Thank you @OskarRaftegard for reporting this issue.
The issue seems to be related to the pipe model. I tested the evaluation of the fluid temperature profiles for the two temperatures, continued from your code. There is a non-physical increase of the fluid temperature at the bottom of the borehole when Tmedia=60 and the flow is turbulent. In that case, both the fluid to fluid (inner to outer) and the fluid to ground (outer to wall) are small and of the same scale (a few 0.01 m-K/W).
See the code and figures below.
I will delve deeper into the coaxial pipe model.
# -------------------------------------------------------------------------
# Plot fluid temperature profiles
# -------------------------------------------------------------------------

# Evaluate temperatures at nz evenly spaced depths along the borehole
# at the (it+1)-th time step
nz = 20
z = np.linspace(0., H, num=nz)
T_b = 0.
T_f_in = 1.

T_f = CoaxBHCs[0].get_temperature(
    z, T_f_in, T_b, m_flow_borehole, cp_f)

# Configure figure and axes
fig = gt.utilities._initialize_figure()

ax = fig.add_subplot(111)
# Axis labels
ax.set_xlabel(r'Temperature [degC]')
ax.set_ylabel(r'Depth from borehole head [m]')
ax.set_title(f'Tmedia = {Tmedia:.0f} [degC]')
gt.utilities._format_axes(ax)

# Plot temperatures
ax.plot(np.array([T_b, T_b]), np.array([0., H]), 'k--')
ax.plot(T_f, z, 'b-')
ax.legend(['Borehole wall', 'Fluid'])

# Reverse y-axes
ax.set_ylim(ax.get_ylim()[::-1])
# Adjust to plot window
plt.tight_layout()

Here is a higher resolution figure for Tmedia=60. There appears to be numerical noise, most likely due to a round-off error when subtracting large values calculated by pipe._general_solution() (which involves calls to exp(beta*z), sinh(beta*z) and cosh(beta*z)).

Sounds reasonable with a numeric error. I have continued to play around, but with the double-U collector. I get negative temperatures with the double-U as well, but it do not look like the same kind of numerical error. T_f slowly drops below T_b with a few decimals.  The fig is plotted with nz=1000.  More or less same "settings" as above.
# Pipe dimensions and properties
r_out = 0.016      # Pipe outer radius (m)
r_in = r_out - 0.003       # Pipe inner radius (m)
D_s = 0.074 / 2         # Shank spacing radius (m)
epsilon = 1.5e-6    # Pipe roughness (m)
k_p = 0.42           # Pipe thermal conductivity (W/m.K)

# Pipe positions
# Double U-tube [(x_in1, y_in1), (x_in2, y_in2),
#                (x_out1, y_out1), (x_out2, y_out2)]
pos = [(-D_s, 0.), (0., -D_s), (D_s, 0.), (0., D_s)]

# Grout properties
k_g = 1.5           # Grout thermal conductivity (W/m.K)

# Fluid properties
m_flow_borehole = 0.14      # Total fluid mass flow rate per borehole (kg/s)
media = "Water"
Tmedia = 60. # Media average tmperature (degC)

I seem to a found a promosing approximation for when gamma * z is large in pipes.SingleUTube.coefficients_temperature.
For now, it is only done for a_in:
# -------------------------------------------------------------------------
# Plot fluid temperature profiles
# -------------------------------------------------------------------------

# Evaluate temperatures at nz evenly spaced depths along the borehole
# at the (it+1)-th time step
nz = 200
z = np.linspace(0., H, num=nz)
T_b = 0.
T_f_in = 1.

T_f = CoaxBHCs[0].get_temperature(
    z, T_f_in, T_b, m_flow_borehole, cp_f)
a_in, a_b = CoaxBHCs[0].coefficients_temperature(
    z, m_flow_borehole, cp_f, nSegments)
f1 = CoaxBHCs[0]._f1(z)
f2 = CoaxBHCs[0]._f2(z)
F4 = CoaxBHCs[0]._F4(z)

delta = CoaxBHCs[0]._delta
gamma = CoaxBHCs[0]._gamma
beta1 = CoaxBHCs[0]._beta1
beta2 = CoaxBHCs[0]._beta2
beta = CoaxBHCs[0]._beta
beta12 = CoaxBHCs[0]._beta12
A = (1 - 0.5 * (beta1 + beta2) / gamma * np.tanh(gamma*H)) / (1 + 0.5 * (beta1 + beta2) / gamma * np.tanh(gamma*H))
z_crit = -0.5 / gamma * np.log(1e-8)
a_in[z > z_crit, 0, 0] = 0.5 * beta12 / gamma * np.exp((beta + gamma) * z[z > z_crit] - 2 * gamma * H) + 0.5 * (1 - A * beta12/gamma + delta) * np.exp((beta - gamma) * z[z > z_crit])
a_in[z > z_crit, 1, 0] = 0.5 * (1 + delta) * np.exp((beta + gamma) * z[z > z_crit] - 2 * gamma * H) + 0.5 * (A * (1 - delta) + beta12/gamma) * np.exp((beta - gamma) * z[z > z_crit])
T_f2 = a_in @ np.atleast_1d(T_f_in) + a_b @ np.full(nSegments, T_b)

# Configure figure and axes
fig = gt.utilities._initialize_figure()

ax = fig.add_subplot(111)
# Axis labels
ax.set_xlabel(r'Temperature [degC]')
ax.set_ylabel(r'Depth from borehole head [m]')
ax.set_title(f'Tmedia = {Tmedia:.0f} [degC]')
gt.utilities._format_axes(ax)

# Plot temperatures
ax.plot(T_f, z, 'b-')
ax.plot(T_f2, z, 'r-')
ax.plot(np.array([T_b, T_b]), np.array([0., H]), 'k--')
ax.plot(np.array([0., 1.]), np.array([z_crit, z_crit]), 'k:')

# Reverse y-axes
ax.set_ylim(ax.get_ylim()[::-1])
# Adjust to plot window
plt.tight_layout()

In the figure below, the blue line is the original method, red is the approximation which only applied below the dotted line, and the dashed line is the borehole wall.

It is still needed for the coefficient a_b, for example if we set T_f_in = 0. and T_b = 1:

To do:

[ ] Approximation of a_in
[ ] Approximation of a_b
[ ] Validation (using a finite difference method?)

To do :

[ ] Implement in SingleUTube class
[ ] Test g-function generation
[ ] More validation cases for the fluid temperature profiles

The initial implementation is on branch https://github.com/MassimoCimmino/pygfunction/tree/issue266_negativeGFunctionsCoaxial
It seems to solve the issue of negative g-functions. Something interesting is that nSegments=8 does not seem do be enough in the case of the initial post. Here is the g-function for Tmedia=60. using nSegments=24.


Add sync_reduce

Codecov Report
Attention: Patch coverage is 93.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 99.86%. Comparing base (bc19e8c) to head (787ce98).

Files
Patch %
Lines

src/karbonn/distributed/ddp.py
93.33%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@             Coverage Diff             @@
##              main     #175      +/-   ##
===========================================
- Coverage   100.00%   99.86%   -0.14%
===========================================
  Files           67       68       +1
  Lines         1423     1453      +30
  Branches       152      158       +6
===========================================
+ Hits          1423     1451      +28
- Partials         0        2       +2

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


feat: update landing page CTA to show fill button or cta button

Overview

Updates CTA to take a buttonType prop to determine whether to show a CTAButton (default) or a regular FillButton. Also refactors PagePrefooter to use newer gamut components.

PR Checklist

[x] Related to designs: https://www.figma.com/file/J5yB1jyELiNt3Vb47T9GcE/Job-Readiness?node-id=2%3A43
[x] Related to JIRA ticket: [REACH-1327]
[x] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.87-alpha.db6d2e.0@codecademy/gamut-labs@20.2.1-alpha.db6d2e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://614a0d18aee02e00bb888798--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.87-alpha.2d96ff.0@codecademy/gamut-labs@20.2.1-alpha.2d96ff.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://614a166c48c55b07e4bbfa8d--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.5.92-alpha.cf412e.0@codecademy/gamut-labs@23.1.1-alpha.cf412e.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://614e1f6e0abf3a2357760da4--gamut-preview.netlify.appDeploy Logs


Can't set a directory with a space in the name

I'm having trouble setting the JRNL location to be my ~/Dropbox (Personal)/jrnl folder. Escaping the characters and adding quotes are taken as text symbols and creates a new folder.
What's the proper way to set the location in my .jrnl_config file?

I fixed the issue. I wasn't including the filename when typing in a new location.

@charliesneath I'm trying to do the same, but even pointing to the file in my Dropbox folder leads to the creation of a new folder. Mind sharing your jrnl_config file?

I ran into this as well. If anyone else reads this, you should not escape the path in jrnl_config:
...
"default": "/Users/username/Dropbox (Personal)/Journals/journal.txt"
...


Get stats command (/gs) is broken?

Not really sure why, but /gs brings an error now when executing it.

error from EMPC, has to be updated

Fixed, plugin has been updated for next restart.


chore: add JSS owners group to Secure CICD

/hold
we will do this for all repos

superseded by https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/pull/1562


Question about the num_sample setting

hello，I'm very glad to learn your code in this project， but there's a hesitation of how to set the num_sample if I want to use all the points in the depth map without sample?

Hi, num_sample=0 will lead to no sampling, so you can use all the input points!

Ok, I'll try it, thank you!


feat: 🎸 shrink pacakge dependencies to remove the duplicates

Checklist
All items have to be completed before a PR is merged

[x] Adhere to Contribution Guidelines
[ ] Updates to Decision Records considered?
[ ] Updates to documentation at DX@Scale Guide considered?
[x] Tested changes?
[x] Unit Tests new and existing passing locally?

Codecov Report
Base: 60.76% // Head: 60.17% // Decreases project coverage by -0.59% :warning:

Coverage data is based on head (db6bffc) compared to base (b195766).
Patch coverage: 13.04% of modified lines in pull request are covered.

:exclamation: Current head db6bffc differs from pull request most recent head 494ea8a. Consider uploading reports for the commit 494ea8a to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1197      +/-   ##
===========================================
- Coverage    60.76%   60.17%   -0.59%
===========================================
  Files           59       59
  Lines         1993     2014      +21
  Branches       431      436       +5
===========================================
+ Hits          1211     1212       +1
- Misses         782      802      +20

Impacted Files
Coverage Δ

...ore/src/dependency/TransitiveDependencyResolver.ts
66.66% <13.04%> (-20.64%)
:arrow_down:

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

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


Payment cards :has selector does not work on Firefox

I have discovered that in example/cards/payment-method.tsx, the [&:has([data-state=checked]))]:border-primary class does not work in Firefox, because it is not yet supported.
So I managed to fix it by tweaking the code a bit and using the peer variant.
So my code now looks like this for each RadioGroupItem
<div>
    <RadioGroupItem
        value="card"
        id="card"
        className="peer sr-only"
        aria-label="Card"
    />
    <Label
        htmlFor="card"
        className="flex flex-col items-center justify-between rounded-md border-2 border-muted bg-transparent p-4 hover:bg-accent peer-data-[state=checked]:border-primary hover:text-accent-foreground [&:has([data-state=checked])]:border-primary"
    >
        <svg
            xmlns="http://www.w3.org/2000/svg"
            viewBox="0 0 24 24"
            fill="none"
            stroke="currentColor"
            strokeLinecap="round"
            strokeLinejoin="round"
            strokeWidth="2"
            className="mb-3 h-6 w-6"
        >
            <rect width="20" height="14" x="2" y="5" rx="2" />
            <path d="M2 10h20" />
        </svg>
        Card
    </Label>
</div>

Great job @alonsourena! It's always Firefox 😄. We already had a problem with the :has pseudo-class on another component.
Could please create a PR for this?

Sure thing! Never done one before but I'm going to figure out a way to do it 👍🏻.

Fixed by https://github.com/shadcn-ui/ui/pull/1209


Changes from Group 1 - Code Duplication

This fixes:

#79
#82
#89
#97
#108
#72

@rilling The PR is mergeable but we received a feedback that it is not. Could you please check?

Since this PR is created from the master branch, it also has commits for CS3 now as they have been merged into the fork.


no connection

hi i have an mk3s+ , now i have install in my raspi zero 2 this: https://github.com/prusa3d/Prusa-Link
it´s working good but i can not use it with home assistant have anyone an idea?  thanks
i have this error in the log: 2022-12-03 11:32:48.048 ERROR (MainThread) [homeassistant.components.prusalink.config_flow] Could not connect to PrusaLink: 403, message='Forbidden', url=URL('http://192.168.178.**/api/version')
{"api": "0.9.0-legacy", "server": "0.7.0rc2", "original": "PrusaLink I3MK3S", "text": "PrusaLink 0.7.0rc2", "firmware": "3.12.0-RC1-5576", "sdk": "0.7.0rc2", "capabilities": {"upload-by-put": true}, "hostname": "prusa"}
is the plugin only comaptible with prusa mini? if so then it was misnamed :)

The goal is to make it generic but I only have a mini to test it with.

I can help you :)

First is to verify your API key is valid. 403 = invalid key.

I have to disappoint you, I entered the api from the server settings, it doesn't accept it, neither api key, nor prusalicer key....I think that's where the error is
API keys: jdfeW94N2Hfaxx
PrusaSlicer key: 4M95HERJAKTWxxxx
Host: 192.168.178.xx

Hi,
Same problem here.
I think it could be nice to specify this library only works with Prusa Mini now.
Currently, I'm with the same problem than @yasars, my configuration is identical (Prusa MK3S and Raspberry Pi Zero), the code of the API is absolutely not the same .
We have:

https://github.com/prusa3d/Prusa-Link PrusaLink is a compatibility layer between 8-bit Prusa 3D printers (MK2.5, MK2.5S, MK3, MK3S and MK3S+) and PrusaConnect, which lets you control and monitor your 3D printer from anywhere.
https://github.com/prusa3d/Prusa-Firmware-Buddy This repository includes source code and firmware releases for the Original Prusa 3D printers based on the 32-bit ARM microcontrollers.

Both have the same name but target completely different machines and architecture, absolutely nothing says that PrusaLink version 2 for MK3S and consort will have the same format, that's why it would be good to indicate additional information about it.
In the end, it would be ideal to support both, having an MK3S, I can give you a hand.
Thank you.

I also could not get pyprusalink to cooperate with my SL1S due to authentication issues with API keys. I forked the project and implemented Digest authentication so that I can add the printer to HA. Not sure if that is relevant to your printer models. A PR into pyprusalink was rejected.

The goal is to make it generic but I only have a mini to test it with.

It seems that we're beginning to gather a few people with different printers interested in contributing to this goal.

It seems that we're beginning to gather a few people with different printers interested in contributing to this goal.

:+1:

Support for the new v1 API endpoint and Digest Authentication this is mostly addressed with https://github.com/home-assistant-libs/pyprusalink/pull/63.


Missing License

This repository does not have a license.
Could you add a license, so we know if we can use your code for research?
Here you can read up on different open licenses.

please feel free to use the code for research.

Thanks, that is great to hear!
Would you still consider to add a license? That would really help us and any future users of the code.

Thanks for choosing a license.


rescue Exception incorrectly reports untyped code error

Input
→ View on sorbet.run
# typed: strong
extend T::Sig

sig {params(x: Integer).void}
def foo(x)
  begin

  rescue StandardError

  end
end

Observed output
editor.rb:8: This code is untyped https://srb.help/7018
     8 |  rescue StandardError
                 ^^^^^^^^^^^^^
  Note:
    Support for typed: strong is minimal. Consider using typed: strict instead.
Errors: 1

Expected behavior

No untyped code error should be reported under StandardError

closing
dupe of https://github.com/sorbet/sorbet/issues/6878


Add flashrom support

Pending patch approval: https://review.coreboot.org/c/flashrom/+/67878

Patch approved


franka_ros2 compatibility

Hello, I wanted to know if the ros2 humble branch is up to date and can be used. The documentation states that franks ros 2 is a beta software and the installation is provided for ros2 foxy.

Yes, the humble port is usable. It's a fairly recent release so I think the Documentation just hasn't been updated yet.

Hi @drewskoots , the humble port throws the following error while building franka_hardware package:
Could not find a configuration file for package "Franka" that is compatible
with requested version "0.11.0".
The following configuration files were considered but not accepted:
/usr/lib/cmake/Franka/FrankaConfig.cmake, version: 0.9.2
/lib/cmake/Franka/FrankaConfig.cmake, version: 0.9.2

However, when I attempt to install libfranka 0.11.0, the branch is not found. I changed the version in the CMake to 0.10.0 and 0.9.2, but then the following error occurs:
fatal error: franka/active_control.h: No such file or directory
25 | #include <franka/active_control.h>
Could you please take a look into this issue.

you need to switch to the fr3-develop branch in the libfranka. Latest release of franka-ros2 depends on fr3-develop branch. https://github.com/frankaemika/franka_ros2/releases/tag/v0.1.3

okay thanks! it worked now


Why change the collection of rgb in upsampler?

Hi! Noticed that in a recent commit, you change this line, but isn't the original version with >= correct? It returned the rgbs with size greater than input size, while now it returns less than or equal to input size.

@XavierXiao wasn't thinking clearly this morning :disappointed: should be fixed !


Removed surrounding quotes for raw string types

I tried sending a raw string as the payload in an attempt to display custom HTML, but the quotes are a problem here. Because the format is raw, it would be better for the caller to decide if they want to include surrounding quotes for strings.

Would my proposed change cause any breaks to existing features?

This won't work. Looks like I'm hitting the same branch when dumping strings. The string contents should be properly HTML escaped, and the server should accept a new $type or a $html: '...' as the value for displaying unescaped html.

Looking fine 👌

This looks good to me - thanks for the PR! I will merge it right away, and include it in the next extension release.


add ability to stop and restart a game in progress

"Start/Stop Game" button currently stops the game, but starts it as a new game when restarted (adding a duplicate set of players) rather than continuing with the same state

duplicate of issue #4, created accidentally during project setup


Clarify definition of releasable.

From Dave Farley:
I would add the idea that a pipeline "should be definitive for release”.
If the pipeline says everything looks good, that should be enough - it forces the focus on what “releasable” means.

Merged PR