[Help] [Bug?]

Example of bug is on my site: https://sarahsdigitalgarden.netlify.app/
My github repo with my site: https://github.com/sarahsimionescu/digitalgarden
Many but not all of my front links do not work. Also the file navigation does not work either for some reason, let me know if this is a bug or if I am doing something wrong!
I thought the cause could be a lack of support for the vertical pipe command [[Article#Header|Put text here]] which I use quite often, but it seems like there are some links that work.
Please let me know if you know how to fix this! Thank you!

There seems to be an issue when the plugin is processing your note before uploading it to github 🤔
This file for example, should look something like this:
---
{"dg-publish":true,"permalink":"/compsci-2-xc-3/0-5-introduction/recursion-vs-iteration/"}
---

**Recursive implementations (generally) take longer to run the iterative implementations.**

I'm not sure why this happens. What platform are you on? Windows, Mac, Linux, iOS or Android?

Hello! Sorry for this late reply, I had a busy weekend. Thank you so much for your feedback. I am using Windows. Any advice on how to fix the issue?

No worries!
I’m suspecting it may have something to do with the some of the folder names. Can you try creating a copy of the note I mentioned above in the root of your vault, and publish that? Let me know when you’ve done that, and I’ll have a look in your repo to see if it looks correct or not.

So that note seem to have been correctly added. This could suggest that this has something to do with how Windows handles folder names or something. I tried replicating the issue myself, but I couldn't. But I'm on a Mac, so I need to see if I can find a Windows computer to test this.
Out of curiosity: Does any of the files show up under the "Changed" section in the Publication Center, even if you've published them?

Nope, the Publication Center works as expected. Thank you so much for your time.

Everything suddenly works! Thank you!

Oh, okay 😅 Great 👍


[BUG] RCON Command string

Describe the bug
Setting RCON parameters in General Settings do not generate the required command line string for RCON, to allow commands to be run.
To Reproduce
Steps to reproduce the behavior:
Set RCON Port and admin password
Expected behavior
Expect RCON to be enabled either via command line args or ini.
Currently have to manually match settings with extra question mark parameters to get RCON to work

I also have the same problem with RCON.
After a few minutes you can no longer send commands.
The server can no longer be shut down either. You have to close the server window manually so that the server goes offline.

Hello, sorry that you guys are having this problem, can you supply more context, config.json (of the server profile) gus.ini and what is the startup command. Please remove any sensitive information before sending it.
@ELSalvadore @Kaffiend

Hey JensvandeWiel  @JensvandeWiel ,
attached the two dates.
I had to rename the GUS.ini to GUS.JSON, otherwise I wouldn't have been able to upload it here.
Maybe you will find the error.
Thank you and best regards
config.json
GameUserSettings.JSON

Could it perhaps be because the admin password appears twice in the config.json?
See also your wiki:
https://github.com/JensvandeWiel/ArkAscendedServerManager/wiki/Custom-Configuration
},
"adminPassword": {
"type": "string"
},
"serverAdminPassword": {
"type": "string"
When I change the password in the manager it only makes a change to adminPassword but it doesn't change to serverAdminPassword.

Hello Everyone, @ELSalvadore, @JensvandeWiel, @Kaffiend
I've been looking into the duplicated INI values problem and believe to have a solution for this as well. I found that when it came time to save the new configs on startup it was just appending new key=value pairs above the old ones and not replacing values in the INI file, thus not changing anything (as well as the duplicates!).
I managed to fix the problem on my end and will submit a PR after I do some testing and a build to make sure I did not break anything else.

That's weird


Update load-balancer-standard-overview.md

Added the link to VM's SLA to clarify the meaning of "Implicit in VM SLA"

#sign-off

@michaelnemtsev - The article was updated in July to use an include file for this section of the content. Would you resubmit your change against the include file: includes/load-balancer-comparison-table.md? Thanks
Cc @KumudD

I have added the link outside of this PR.
#please-close


default window position

Hello,
I have looked for it but I haven't found it. Is there an option to set the default window position? I was looking for something similar to default_size.

I'd personally be more interested in an absolute positioning, since figuring out the screen size should be rather trivial.
However the possibility for both would be best I guess. A case selection depending on the parameter type isn't hard either, I believe.


should we disable server side cursor for redshift connection by default?

PostgreSQL driver use server side cursor by default since django-1.11.
https://docs.djangoproject.com/en/1.11/ref/databases/#server-side-cursors
Should we set DISABLE_SERVER_SIDE_CURSORS = True ?

@shimizukawa What is the downside of having server-side cursors on Redshift? It has a serious advantage for large querysets on PostgreSQL.

@emyller Redshift documentation documentation recommends against using sever side cursors.
https://docs.aws.amazon.com/redshift/latest/dg/declare.html#declare-performance

For what it's worth, not really sure....
I just started working on a dashboard for a redshift project and am prototyping with rails and django. The DISABLE_SERVER_SIDE_CURSORS = True setting got me  past a “cursor ”django_curs*" does not exist” error.
I was getting that error on a ForeignKey association in admin and in a filtered list view I was building

me again, even with DISABLE_SERVER_SIDE_CURSORS = True i'm seeing the exception in django_admin for a model with a site = models.ForeignKey(Site..) when navigating to the edit page.
I am setting the PK and FK in the redshift schema for documentation and query optimization.
Is anyone else seeing this?

I just ran into this issue which setting DISABLE_SERVER_SIDE_CURSORS = True resolved:
I received the following error when rendering a bound form (i.e. re-displaying a populated form) that had a ModelMultipleChoiceField pointing to a model utilizing Redshift (note that an unbound, blank form rendered with no issue):

cursors can only be used within the transaction that created them

Strangely, googling that quoted phrase returns no results which made troubleshooting difficult. This error is thrown during the handling of the above exception while trying to close the cursor:

OperationalError: cursor "_django_curs_140089065723648_1" does not exist

I'm not using the admin for any of my models in Redshift, but I tried registering and testing one and it seemed to work okay regardless of the setting of DISABLE_SERVER_SIDE_CURSORS.

DISABLE_SERVER_SIDE_CURSORS does work, as you know. I started having the issues when I added another db connection and had to move that setting to the specific redshift connection. Thanks


Emby: some requests fail with "upstream connect error"

Bug description
Since upgrading to istio 1.5.1, some requests to a pod running Emby (with an istio-proxy sidecar) are consistently returning 503 errors with the message "upstream connect error or disconnect/reset before headers. reset reason: connection termination". I've verified that emby is serving the requests correctly. The same pods were working fine with istio 1.4.3.
Expected behavior
Requests to emby pods running istio-proxy return the upstream response.
Steps to reproduce the bug

Apply the following manifest to install emby:

apiVersion: apps/v1
kind: Deployment
metadata:
  name: emby
spec:
  replicas: 1
  selector:
    matchLabels:
      app.kubernetes.io/name: emby
  template:
    metadata:
      labels:
        app.kubernetes.io/name: emby
    spec:
      containers:
        - name: emby
          image: "emby/embyserver:<IP_ADDRESS>"
          ports:
            - name: http
              containerPort: 8096
              protocol: TCP
          livenessProbe:
            httpGet:
              path: /
              port: http
          readinessProbe:
            httpGet:
              path: /
              port: http
          resources:
            limits:
              memory: 512Mi
            requests:
              cpu: 100m
              memory: 256Mi
          env:
            - name: UID
              value: "1000"
            - name: GID
              value: "1000"
---
apiVersion: v1
kind: Service
metadata:
  name: emby
spec:
  type: ClusterIP
  ports:
    - port: 80
      targetPort: http
      protocol: TCP
      name: http
  selector:
    app.kubernetes.io/name: emby

From another pod in the same namespace, run curl -v emby.

When istio sidecar injection is not present, I get the following:
*   Trying <IP_ADDRESS>:80...
* Connected to emby (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: emby
> User-Agent: curl/7.69.1-DEV
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 302 Found
< Location: web/index.html
< Date: Fri, 17 Apr 2020 00:58:07 GMT
< Transfer-Encoding: Chunked
<
* Connection #0 to host emby left intact

When the istio sidecar injection is present, I get the following:
*   Trying <IP_ADDRESS>:80...
* TCP_NODELAY set
* Connected to emby (<IP_ADDRESS>) port 80 (#0)
> GET / HTTP/1.1
> Host: emby
> User-Agent: curl/7.67.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 503 Service Unavailable
< content-length: 95
< content-type: text/plain
< date: Fri, 17 Apr 2020 01:13:48 GMT
< server: envoy
< x-envoy-upstream-service-time: 89
<
* Connection #0 to host emby left intact
upstream connect error or disconnect/reset before headers. reset reason: connection termination

When I examine the internal logs in the pod (at /config/logs/embyserver.txt), I can see the requests with a 302 response. Also, requests to some other paths succeed as expected:
/ # curl -v emby/web/index.html > /dev/null
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>:80...
* TCP_NODELAY set
* Connected to emby (<IP_ADDRESS>) port 80 (#0)
> GET /web/index.html HTTP/1.1
> Host: emby
> User-Agent: curl/7.67.0
> Accept: */*
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 302 Found
< location: index.html?start=wizard#!/wizard/wizardstart.html
< access-control-allow-headers: Accept, Accept-Language, Authorization, Cache-Control, Content-Disposition, Content-Encoding, Content-Language, Content-Length, Content-MD5, Content-Range, Content-Type, Date, Host, If-Match, If-Modified-Since, If-None-Match, If-Unmodified-Since, Origin, OriginToken, Pragma, Range, Slug, Transfer-Encoding, Want-Digest, X-MediaBrowser-Token, X-Emby-Token, X-Emby-Client, X-Emby-Client-Version, X-Emby-Device-Id, X-Emby-Device-Name, X-Emby-Authorization
< access-control-allow-methods: GET, POST, PUT, DELETE, PATCH, OPTIONS
< access-control-allow-origin: *
< server: envoy
< date: Fri, 17 Apr 2020 01:15:36 GMT
< content-length: 0
< x-envoy-upstream-service-time: 35
<
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
* Connection #0 to host emby left intact

Version (include the output of istioctl version --remote and kubectl version and helm version if you used Helm)
$ istioctl version --remote
client version: 1.5.1
control plane version: 1.5.1
data plane version: 1.5.1 (11 proxies)

$ kubectl version
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-25T14:58:59Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-25T14:50:46Z", GoVersion:"go1.13.8", Compiler:"gc", Platform:"linux/amd64"}

How was Istio installed?
istioctl manifest apply -f "$DIR"/../manifests/istio.yaml

With the following manifest file:
apiVersion: install.istio.io/v1alpha1
kind: IstioOperator
spec:
  values:
    global:
      mtls:
        enabled: true
    sidecarInjectorWebhook:
      rewriteAppHTTPProbe: true
    gateways:
      istio-ingressgateway:
        type: LoadBalancer
    global:
      k8sIngress:
        enabled: true
        enableHttps: true
        gatewayName: ingressgateway

And the following PeerAuthentication config:
apiVersion: security.istio.io/v1beta1
kind: PeerAuthentication
metadata:
  name: default
spec:
  mtls:
    mode: STRICT

Environment where bug was observed (cloud vendor, OS, etc)
Running on an on-prem environment with OpenStack train (K8s installed through kubeadm).

This is an Envoy bug: https://github.com/envoyproxy/envoy/issues/10041. I can confirm this is fixed in 1.6, and sounds like it was working in 1.4 so must have been a regression that slipped in?

And thanks for the thorough report by the way! if you had left out any of the details you had in there I probably wouldn't have figured this out

Would you mind trying out 1.6? I verified it myself but would be good to get a second set of eyes on it

We are observing this same bug with istio 1.6 OSS. Can you validate on your end if this has been fixed in 1.6 ASM?

I have verified this is fixed on 1.6 from reproducing the config above. Its possible you are experiencing a different issue?

@howardjohn the exact same services which failed for us on 1.5.4 ASM failed on 1.6 OSS, and were fixed with a rollback to 1.4.9 ASM/OSS in the respective scenarios. So it is most likely due to the Transfer-Encoding: Chunked issue which has been documented on envoy repository which you shared in the above comments.

@andrijaperovic do you have any info suggesting it is this issue aside from it breaking on 1.5? Do you see the same dispatch error: http/1.1 protocol error: unsupported transfer encoding error?

From what I can tell by this pull request, the changes were on a release-1.5 branch in envoyproxy/envoy:
https://github.com/envoyproxy/envoy/pull/10055
Not sure how this repository is being shipped with OSS/ASM istio or what the release/branching strategy is.

It was added to 1.5 branch a few days ago, it has not yet shipped in any 1.5.x release

We did originally observe the dispatch error on the 1.5.4 ASM install, I haven't found a similar snippet on 1.6, will have to re-install on a test cluster to validate:
[external/envoy/source/common/network/io_socket_error_impl.cc:29] Unknown error code 104 details Connection reset by peer [Envoy (Epoch 0)] [2020-06-02 20:13:34.626][30][trace][connection] [external/envoy/source/common/network/raw_buffer_socket.cc:38] [C176] read error: Resource temporarily unavailable [Envoy (Epoch 0)] [2020-06-02 20:13:34.626][30][debug][client] [external/envoy/source/common/http/codec_client.cc:127] [C176] protocol error: http/1.1 protocol error: unsupported transfer encoding

This will be shipped out in 1.5.6 which should be out pretty soon - within a week or so, I am not sure the exact schedule though

Closing as I think this is fixed in all versions now. If not please reopen/open a new issue. Thanks!

@howardjohn this appears to be still broken in 1.6.5, please see my comment on istio/envoy:
https://github.com/envoyproxy/envoy/pull/10055#issuecomment-666867460


Wrong title in diff editor

Type: Bug
STR:

Make some changes in git connected project
View diff
Find changed lines and hover mouse over line want to revert
There are 2 different text titles and types, see screenshot

VS Code version: Code 1.87.1 (1e790d77f81672c49be070e04474901747115651, 2024-03-06T00:21:36.258Z)
OS version: Windows_NT x64 10

/duplicate of #206784

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Clarification questions

Hi, I have a few questions regarding TransCoder's training data and optimization setting.

From the paper, it is clear that TransCoder is trained using Standalone functions during the DAE+BT training stage. But is TransCoder only trained using Standalone functions in the MLM stage too?
During the MLM stage, only the encoder part of TransCoder is pre-trained, right?
For the MLM pre-training, max_epoch and epoch_size are set to 100k. If I understand correctly, epoch_size basically refers to the number of instances used in each epoch. Is it correct? Also, for MLM pre-training, the following are set:

--validation_metrics _valid_mlm_ppl \
--stopping_criterion '_valid_mlm_ppl,10'

So, I am assuming TransCoder pre-training is stopped based on the stopping_criterion. Before, the MLM pre-training was stopped, how many optimization steps were executed?

Unlike the MLM pre-training stage, for the DAE+BT stage training, there is no stopping_criterion is set. And the epoch_size was set to 50000 and the max_epoch was set to 10000000. So, when the training stops? How many optimization steps were executed during this stage?

We train on whole files for the MLM stage
Yes but we reload the encoder pre-trained with MLM for the decoder when training TransCoder
It stopped at 557 epochs on 32 GPUs for the model we release in this repo.
We stopped the training when it stopped improving and selected the best model using the validation set or stopped it after two days of training for the translation models trained from DOBF. The TransCoder_model_1.pth corresponds to the 30th epochs and TransCoder_model_2.pth to the 33rd.

@brozi one more question, MLM pre-training is done with a batch size 32. Is it per-GPU-batch-size or total batch size? If it is the total batch size, then I am curious to know why did you need 32 V100 GPUs? Or, actually, the total batch size is (32 x 32) = 1024?

It's per GPU so yes the actual batch size is 32 * 32

One more question for my understanding. You said, The TransCoder_model_1.pth corresponds to the 30th epochs and TransCoder_model_2.pth to the 33rd (and the epoch_size was 30k instead of 50k). I want to understand this part clearly.
Does epoch size 30k mean one epoch composed of 30,000 standalone functions? If yes, then stopping after 30 epochs mean, the model was trained with (30,000 x 30) = 900k functions? (I doubt this is the case)
OR, epoch size 30k means 30k updates (each update is associated with 1024 instances, which is the batch size)? In this case, 30.72M functions are used in each epoch. Stopping after 30 epochs would mean the model is trained with 921.6M functions.
From the TransCoder paper, it seems like there are 60M Java, 60M C++, and 109M Python Standalone functions (15%, 50%, 50% of the total Java, C++, and Python functions). So, in total there are (approx.) 229M Standalone functions. So, if the above-mentioned calculation is correct, TransCoder went through the entire collection 4 times (roughly).

The epoch size is supposed to be the number of samples you train on.
However there is something that's a bit confusing in our code. We always use increase the sentences counter by the batch_size parameter when training but it's not always the actual batch size.

for stream datasets (e.g. for MLM) we use the batch parameter so epoch_size is really the number of samples (per GPU so you need to multiply by 32)
for DAE and BT we fix the number of tokens per batch instead of the epoch size to avoid OOMs due to batches of varying sizes so the batch size is not constant. In that case epoch_size is num_updates/32 (32 is the default batch size parameter), which doesn't really cause any issues but is quite confusing.
I'll think about changing the behaviour of this parameter to have something more coherent in a way that minimizes the changes people will need to make to the parameter.


Nearby: Flawed initial loading of pins.

Currently, in nearby, when a user scrolls (which also includes resizing) to a new position in the map), a query is made for items in the rectangular geographical area. Although the query is lightweight (made for only P625), it goes on endlessly until timeout for large enough areas.
When a user zooms out enough, the initial query is made for a large area, so the query takes a long time only to end up in a timeout. When the user then zooms in, a second query should be initiated (discarding the first), but it never happens as per the current logic, which ideally shouldn't happen.
This part is easy to fix (by ignoring currently running query after a shorter timeout).
But ideally, the expensive query should never be made in the first place. Reason:
Say, the area is small enough for the query to return (but still large enough for it to return much greater than 1000 items).
Then the resulting items are sorted in-app (by their distance from the map centre), and only the nearest 1000 items are actually utilized.
A workaround would be to first query for the items count. If it's much greater than 1000, then query on a smaller square bound centered on the user (in the same manner as a binary search). Do it till the number of items in the queried area is under $1000 \times \frac{4}{\pi}$.
Here's an example:

The initial query in its current form
The proposed initial query (which returns only the count)

Also, if the area is absurdly big, then even the count-binary-search queries (proposed above) can be dropped and a simple overlay saying "zoom in to display pins" can be shown (maybe similar to how chromium based browsers show "No Internet Connection").

Great ideas! I totally agree with all of this.
Do you want to be assigned? (please mention it when creating issues)


pyfastllm 内存泄露

测试脚本：
from typing import List
import json
import time
import os
import sys

cur_dir = os.path.dirname(os.path.abspath(__file__))
fastllm_dir = os.path.join(cur_dir, "fastllm")
sys.path.append(fastllm_dir)

import pyfastllm

glm2_model_path = os.path.join(fastllm_dir, "chatglm2-6b-fp16.flm")

glm2_model = pyfastllm.create_llm(glm2_model_path)

def _receive(idx: int, content_list: List[bytes]):
    #print(idx)
    pass

def gen_llm_config(llm_config):
    _gen_config = pyfastllm.GenerationConfig()
    _gen_config.enable_hash_id = True
    _gen_config.max_length = llm_config["cfg_max_length"]
    _gen_config.top_k = llm_config["cfg_top_k"]
    _gen_config.top_p = llm_config["cfg_top_p"]
    _gen_config.temperature = llm_config["cfg_temperature"]
    _gen_config.repeat_penalty = llm_config["cfg_repeat_penalty"]
    _gen_config.last_n = llm_config["cfg_last_n"]
    return _gen_config

def response(model, prompt_input, config):
    gmask_token_id = 130001
    bos_token_id = 130004

    input_ids = model.weight.tokenizer.encode(prompt_input)
    input_ids = input_ids.to_list()
    input_ids.extend([gmask_token_id, bos_token_id])
    input_ids = [int(v) for v in input_ids]
    # print(input_ids)

    handle = model.launch_response(input_ids, config)
    continue_token = True
    ret_byte = b""
    ret_str = ""

    while continue_token:
        resp_token = model.fetch_response(handle)
        continue_token = (resp_token != -1)

        content = model.weight.tokenizer.decode_byte([resp_token])
        ret_byte += content
        ret_str = ret_byte.decode(errors='ignore')

    return ret_str

def main():
    with open("data.json", "r") as f:
        d = json.load(f)
    with open("params.json", "r") as f:
        d2 = json.load(f)
    prompt = d["data"][0]
    config = d["config"]
    llm_config = gen_llm_config(config)
    for i in range(0, 100):
        glm2_model.response(prompt, _receive, llm_config)
        #time.sleep(1)
    print("done 1")
    """
    for i in range(0, 100):
        ret = response(glm2_model, d2["query"], llm_config)
        print(ret)
        #time.sleep(1)
    print("done 2")
    """
    time.sleep(10000000)

if __name__ == "__main__":
    main()

使用google-perftools定位FillLLMInputs一直在分配内存但是没有释放，如下图分配30多G：

请问是我的使用方式不对吗

附带data.json的内容如下：
{
    "data": [
        "[Round 0]\n\n\u95ee\uff1a\u4ee5\u4e0b\u7528\u4e09\u4e2a\u5f15\u53f7\u5305\u542b\u7684\u5185\u5bb9\u662f\u6587\u7ae0\u5185\u5bb9:\n\"\"\" Sample International Contract for Sale of Goods, pursuant to the United NationsConvention on Contracts for the International Sale of Goods TERAMATE,Ltd. with its principal office West Road Drive27, Hopson Chart, Briston, AN44FL,UKrepresented by Matt Wattson, on the basis of Power of Attorney from 23 June 2008(hereinafter referred to as the ,,Seller\" on the first side) and AGFH,a.S. ID:78333998 having its principal office at: Palachova 152, Prague 2, Zip Code: 12000registered in the Commercial Register,Section B, Entry No. 4127 maintained by theMunicipal Court, Prague acting by: Ing. Karel Nekola, Chairman of the Board of Directors(hereinafter referred to as the ,,Buyer\" on the second side) (Seller and Buyer referred to also as the \"Contracting Parties\"or separately each the\"Contracting Party\") have entered on the day, month and year as bellow, pursuant to the United NationsConvention on Contracts for the International Sale of Goods (hereinafter referred to as\"Convention\"), into the following CONTRACT FOR SALE OF GOODS I.Subject-matter of the Contract The Subject-matter of this Contract is particularly the obligation of the Seller to deliver goodsspecified in the Exhibit No. I hereto to the Buyer and to transfer the property in goods to theBuyer under the terms and conditions herein and the obligation of the Buyer to accept thedelivered goods from the Seller and to pay the agreed purchase price. II.Sale of Goods 1.The Seller hereby agrees to deliver the Buyer goods (movables) specified in Exhibit No.1hereto(hereinafter referred to as the ,,Goods\") and in the time,quality and quantity specifiedin Exhibit No. I hereto. The Buyer shall collect the Goods and pay Seller for Goods thepurchase price specified in the Article II. hereof.2. The Seller fulfils his obligation to deliver the Goods when the Goods have been madeavailable to the Buyer at the place of business of the Seller. The Parties have agreed that theBuyer shall arrange for carriage of the Goods from the place of business of the Seller througha carrier the name of which Buyer shall notify Seller. The Seller shall arrange the loading ofGoods, and the Goods shall be packed in the manner set forth in Exhibit No. 2. Unless\n otherwise expressly provided herein, the Goods shall be packed in manner adequate to protectthe Goods. 3.The Seller shall dcliver the Goods to Buyer's carrier on 15 December 2008 during regularworking hours (08.00 to 16.00 hours). Seller shall notify Buyer regarding the delivery ofGoods to carrier by fax message sent to phone No<PHONE_NUMBER>94.The title in the Goods shall pass to Buyer immediately upon delivery of Goods to theBuyer's carrier. Risk of damage to or loss of the Goods shall pass to the Buyer at the time ofdelivery.5. The Buyer hereby declares he received all information regarding the Goods necessary toarrange insurance coverage.6.Seller shall send the Buyer documents related to the Goods within 10 days after delivery ofGoods and at the Buyer's address set out in herein. III.Purchase Price 1.The Buyer shall pay the Seller the purchase price of the goods amounting EUR 50000(hereinafterreferredto         asthe,,Purchase      Price\").2. The Purchase Price shall be due upon the invoice issued and sent by the Seller not later than10 days from delivery and collection of Goods by the Buyer. The invoice shall be payable notlater   than   21   days   from   the   issue   of   the   invoice   by   Seller.3. If the Buyer fails to pay the purchase price, the Seller shall have the right to default interestat the rate of 0,1 % of outstanding amount for each day of default without prejudice to anyclaims for damage pursuant to the Article 74 of the Convention. IV.Product Liability 1.The Seller shall be liable for any lack of conformity in Goods which exists at the time whenthe risk passes to the Buyer and which occurs within 24 months from the date of delivery ofGoods by the Buyer's carrier. The Seller declares that the Goods during a period of 24 monthsfrom the date of collection by the Buyer's carrier will remain fit for the purposes for whichthe Goods would ordinarily be used or during this period will retain specified qualities(hereinafter referred to as the ,,Warranty Period\").2.The Seller shall not be responsible for the defects arising out of the failure to followoperation instructions, for the defects caused by improper storage after the Goods weredelivered or for the defects caused by circumstances that were beyond the reasonable control.3.The Buyer shall, immediately upon delivery of the Goods by the carrier, duly examine theGoods and if the defects of Goods were apparent upon the collection of Goods, the Buyershall promptly give notice on this to the Seller.4. Should the Buyer discover any defects during the Warranty Period, the Buyer shall givewritten notice of the defect to the Seller and not later than within 15 days after such defect hadbeen detected. In a written notice specifying the defects he shall have the following options:-replace of defective Goods by delivery of non-defective Goods;demand to- repair the defective Goods if the defects are repairable;demand- appropriate Purchase Price reduction; or\n to withdraw from the- Contract.5.The Seller, upon receipt a notice from the Buyer stating the defect, promptly shall give awritten statement and reply whether he accepts the claim for defects or not. VI.Exclusion of Liability 1.A party is not liable for a failure to perform any of his obligations if he proves that thefailure was due to an impediment beyond his control and that he could reasonably be expectedto have taken the impediment into account at the time of the conclusion of the Contract or tohave ayoided or overcome it or its consequences. The exemption provided by this Article haseffect for the period during which the impediment exists.2. The non-pefforning party shall give prompt written notice to the other party of the reasonfor its failure to perform and the extent and duration of its inability to perform. VII. Arbitration Clause All the disputes resulting from this agreement or in conjunction with it, will be decided finallyin the arbitration procedure before one arbitrator Mgr. Marie Pluharova, registered in the listof arbitrators of Czech Arbitration Centre s.r.o., ID 28163427, Klatovska 515/169,32100Plzen-Litice and registered in the list of arbitrators of Ministry of Justice Czech Republicreg.nr.69 and appointed in accordance with Act No. 216/1994 coll. of Laws, on ArbitrationProcedure and Execution of Arbitration Awards, and with the Proceeding Rules of CzechArbitration Centre announced at it's websites www.arbitrators.cz. The parties authorize thearbitrator to settle the dispute based on the principles of natural equity. Compensation forarbitration costs (including the expenses of the contractual parties) will be awarded by thearbitrator based on the principle of success in the dispute. VIII.Final Provisions 1.This Contract shall enter into force and shall take effect on the day when it is executed.2.The Contracting Parties hereby agree that entering into this Contract and performing dutiesunder this Contract have been duly approved by the relevant company bodies of theContracting Parties in a compliance with legal regulations, by-laws and other internalregulations of the Contracting Parties; and no other approval or consent shall be required.3.The Contracting Parties agrees to respect the legitimate interests of the other Party, shallconduct in accordance with the purpose of this Contract and shall not counteract such purposeand they shall perform all legal and other actions that may prove necessary to reach thepurpose of this Contract. 4. All documents in writing shall be mailed at the address of the Contracting Parties set forth\n in the heading of this Contract unless either of the Contracting Partics shall give a writtennotice to the other Party on changing its address. Whatever papers the delivery of which isrequired, assumed or is made available by this Contract and regardless of any other availableway allowed by the legal regulations to prove such a delivery, shall be deemed to have beenserved if such had been delivered to the other Contracting Party at the address set forth in theheading of this Contract or at the address noticed in written form by either Contracting Partyto the other Party.5. Any changes and amendments to this Contract shall require a written form.6.If any provision of this Contract is determined to be invalid or unenforceable,the validityor enforceability of the other provisions either of this Contract as neither a whole nor otherprovisions will be affected unless such an invalid or unenforceable provision is severable.Contracting Parties herby agrees to supersede such an invalid or unenforceable provision by anew valid and forceable provision that most closely matches the intent and the purpose of theoriginal provision.7.This Contract and the relations arising from shall be govermed by the Law of the CzechRepublic,particularly by the United Nations Convention on Contracts for the IntermationalSale of Goods.8.This Contract had been made in two duplicates whereby each Contracting Party shall retainone copy each. e TERAMATE, Ltd.Karel NekolaDone in Prague on 13 June 2019 ml AGFH,a.S.Matt Wattson Done in Prague on 13 June 2019\n\"\"\"\n\u7528150\u5b57\u4ee5\u5185\u7684\u4e2d\u6587\u603b\u7ed3\u4ee5\u4e0a\u6587\u7ae0\u5185\u5bb9:\n\n\u7b54\uff1atime_stamp:169762201245954754816808106421951820652289425540685"
    ],
    "config": {
        "cfg_max_length": 4096,
        "cfg_top_k": 1,
        "cfg_top_p": 0.8,
        "cfg_temperature": 0.1,
        "cfg_repeat_penalty": 1.0,
        "cfg_last_n": 64
    }
}

内存飞涨

是基于cpu运行的吗

是基于cpu运行的吗

基于GPU的

我也遇到这个问题了， gpu版的， 跑完以后就报错了， 释放GPU内存的代码似乎有问题
Error: CUDA error when release memory!
CUDA error = 4, cudaErrorCudartUnloading at /fastllm/src/devices/cuda/fastllm-cuda.cu:1493
'driver shutting down'

me too，Mark.


Enable integration testing via Flask Application Factory pattern

Connected to #79, #81, #35
This PR introduces the Application Factory pattern to AIPscan, which resolves our linting errors, enables multiple application configurations (e.g. for production, development, and testing), and  provides us the ability to use an in-memory test database for tests that require accessing database state. It also includes some new test fixtures (see more below) and an example integration test.
Along the way, this work also had the following consequences:

Changes to how Celery is configured:

Celery is now initialized in AIPscan.extensions prior to being provided Flask application context in AIPscan.celery.configure_celery.
The configuration details for the Celery result backend and broker were moved to their own config file and are now environment-aware.
The Celery worker should now be started with: celery worker -A AIPscan.worker.celery --loglevel=info (the instructions in the README have also been updated)

The db.create_all() call in the Aggregator Blueprint's __init__ module had to be removed, so the database must be explicitly created prior to running the application.
It was necessary to remove the @app.route("/") handler from the Aggregator component. To avoid any user-facing changes, I created a very small Home blueprint to add a route for the root ("/") path which simply redirects to the Aggregator view function @app.route("/") was previously decorating.
Tests that invoke Flask or SQLAlchemy-related code now need to be aware of their application context. The app_instance fixture (added in this work to the Aggregator and Data components) gives a demonstration of how to provide this application context without requiring changes to our tests (edited to account for latest commit).

Ideally we would want to define fixtures such as app_instance once rather than separately in each Blueprint, but I think that would require co-locating our tests within a single module (e.g. AIPscan.tests) in order to use fixtures defined in a conftest module in tests for any of our Blueprints. If we did go that way, we might keep the per-Blueprint structure via sub-modules within AIPscan.tests. I'd like some input on whether that sounds reasonable, or if there might be other consequences that I'm not seeing.
And perhaps the nicest consequence:

Hi @ross-spencer - I've made all the changes I think need to be made based on the review comments. That includes reverting to automatically creating the database, which I'm glad you pushed for - it is a nicer experience this way, and moving our conftest.py up into the AIPscan package so that our app_instance pytest fixture is available across all of the blueprints.
In addition, I've moved the application factory into AIPscan.__init__ (largely just so it's where other developers with Flask experience will expect it and to reduce the number of modules, though there may be other advantages as noted in the commit message) and moved the blueprint imports into the application factory function.
Tests and linting are all still passing following the changes, so if you're happy I think all that remains is to rebase on #89 after it's merged.


error: SSL_connect

doesn't happen every time... but this is the second time it happened today. so I figured it was worth reporting here.
[info] found cache entry for >https://glia.net/the-blog?format=rss<
[info] adding header If-None-Match (etag) >W/"0fc4cbe2e2ab08cd539e871f3f48f9d2"< for conditional GET

*** error: SSL_connect SYSCALL returned=5 errno=0 state=SSLv3/TLS write client hello

##[error]Process completed with exit code 1.

Thanks for reporting the SSL / HTTPS connection errror.

I guess the problem is, not so much the connection error, as the build failing because of that connection error.

I see - good point - I try to add the error to the list of "recovered" catched errors that get logged but processing (on the next feed) continues.  The rescue line is here https://github.com/feedreader/pluto/blob/master/pluto-feedfetcher/lib/pluto/feedfetcher/cond_get_with_cache.rb#L42.


Add twig filetype

Hey!
It would be nice if HTML tags were automatically renamed in Twig files.
Léo

Hi there!
It appears the primary maintainer is currently preoccupied and hasn't been responding (https://github.com/windwp/nvim-ts-autotag/issues/163#issuecomment-2107013155). Not a dig against them, they've provided a great plugin at no charge.
In the meantime, I'll be maintaing a fork until they're back @ https://github.com/PriceHiller/nvim-ts-autotag/.
I may be incorporating your PR at that repository. If you have any issue with me doing this, please let me know 🙂.

Hello! Since my last comment, I've now become a maintainer to help out in this repo. This was merged into main as a co-authored commit from my fork as part of #173.
Commit can be found here: https://github.com/windwp/nvim-ts-autotag/commit/5d6845d508aae2bda09712287126ed834f3d0084
I'll be closing this out, please ping me if it shouldn't have been closed.
Thanks!


add React Native (ClojureScript and re-frame)

Repo: https://github.com/flexsurfer/conduitrn
DEMO: https://www.youtube.com/watch?v=aq1QMq6_E14&list=PLRSmGfvI8FLo-EgYJw4PoFEbpnKNA0dh9
Development process: https://www.youtube.com/watch?v=5k2T126JFCY
DEMO APK: https://github.com/flexsurfer/conduitrn/blob/master/app-release.apk

hi @EricSimons any chance we can merge this one ? :) there is a video demo, also its a port of clojurescript web version, so it has the same functionality as already added and approved https://github.com/jacekschae/conduit , thanks

hi, @anishkny is this project still alive? :)

Related to #578

Hi, @flexsurfer! 😄
Terribly sorry for the long delay. Your PR is now merged and The README should be autogenerated within 24 hours.

Hi, @flexsurfer! 😄
Terribly sorry for the long delay. Your PR is now merged and The README should be autogenerated within 24 hours.


Response of head()-function is null

In my case a
Restangular.one('locations', location.id).head().then(function(response) {
    console.log(response.headers);
});

does not return a response and therefore I get a TypeError: Cannot read property 'headers' of undefined. What's the exact usage of head() so I can access for instance the ETag value?

Yep, that's how it should be done :) Glad you figured that out by yourself.


Zoom

This is the first approach to make a zoom component over the plot.
Scroll over the plot updates the selected positions (left and bottom are default) padding.
https://deploy-preview-99--amazing-kilby-d9c1d7.netlify.app/?path=/story/plot-plot-zoom--control

This is going to be implemented in another way and another moment


Change SASS for classic CSS

We don't need SASS now that the majority of the browsers support CSS nesting.

I'm working on the work described in that Issue right now, like the file at this link
https://github.com/biomejs/website/pull/1203/files#diff-da27d798d2b0283a7abb796fbf9a588f5ed817ce08efb4edd7aaaac29029dc7dR1-R23
I have a file that is written in scss proprietary notation, do you want to change this so that it can be expressed in naitive css?


Generate AWS Advanced Templates directly with gen library

Initial support for creating AWS templates with dcos_generate_config.sh using the argument --aws-cloudformation and a limited subset of config.yaml options.
Known bugs:

[x] Setting aws_template_upload: true doesn't make aws_template_storage_access_key_id and friends mandatory / validate
[x] Need to tell user about config_id folder + path information
[x] bootstrap_url is set wrong (Shouldn't contain the unique config_id path)
[x] repository_url is set wrong (Shouldn't contain the unique config_id path)

Sample config.yaml:
---
aws_template_storage_bucket: cody-test-advanced
aws_template_storage_bucket_path: some/path
aws_template_storage_region_name: us-west-2
aws_template_upload: true
aws_template_storage_access_key_id: <your_access_key_id>
aws_template_storage_secret_access_key: <your_secret_access_key>

Includes: #656

@cmaloney this is ready for a review. I'm now using the same codepath used by the release library to generate the AWS artifacts. Separately, we are creating integration tests which will test the generated templates. You can generate yourself with:
wget https://downloads.dcos.io/dcos/testing/pull/442/dcos_generate_config.sh
chmod +x dcos_generate_config.sh
mkdir -p genconf
echo -e "---\ncloudformation_s3_url: https://s3-us-west-2.amazonaws.com/jeremy-testing/dcos" > genconf/config.yaml
./dcos_generate_config.sh --aws-cloudformation

The resulting directory structure looks as follows:
genconf/serve/aws.html
genconf/serve/bootstrap/d7552bd48f051a987e30808acb5cd58f4aef707d.active.json
genconf/serve/bootstrap/...
genconf/serve/cloudformation/el7-zen-1.json
genconf/serve/cloudformation/...
genconf/serve/packages/dcos-metadata/dcos-metadata--setup_45bd2e731989db9b3f2b4b15ab852c47ce886bb0.tar.xz
genconf/serve/packages/dcos-metadata/...
genconf/serve/packages/dcos-config/dcos-config--setup_5f3dea04dd07405651d0d815be3364b50454ac88.tar.xz
genconf/serve/packages/dcos-config/...

The open questions in my mind are:

Should we provide something analogous to channel_commit_path for users building template artifacts locally? The user can control the exact path with the cloudformation_s3_url config option by setting it to a unique path on each build. However, it would be easy to overlook this and I think there is a high probability that end-users will overwrite important template artifacts unintentionally. Any suggestions here?

How should we handle config option validation? For these generated templates we can only support a subset of the standard options... for example cloudformation_s3_url and resolvers can be set, but other options like exhibitor_address, s3_bucket, s3_prefix, etc. can not be modified currently and we should error. We have validation code in both the gen library and the installer, but it's not clear to me where the best place is to do the validation for this new configuration mode.

cc: @spahl and @malnick

Reuse or Not
My PR adds three new arguments, that could possibly be conflated with current gen parameters - cloudformation_s3_bucket, cloudformation_s3_key, cloudformation_s3_region. These are user-facing, and I'm not sure if I should re-use the current s3_bucket and s3_path parameters already present, or create these new ones for CF generation only.
Validation
@lingmann As for validation, and this is part of another PR since it'll be a big change, we should work towards using the code-paths that on-prem generation uses: validation and calculation funcs should go in gen/calc.py.
resolvers is already in there, but when we use the AWS provider I do not think its validation function is leveraged yet.

I think this is now to a baseline MVP in terms of end to end code
This needs some hand testing and probably a user test before we can land it. Integration test before 1.8 ships, although after we do this as a baseline. There is a unit test included.
Need to go though / double check the code changes for some of the bits shared with release / aren't too dangerous and are correct.

The s3 bucket used for this needs a bucket policy which allows the stacks being launched from it to be read from. Probably want to make a mandatory opt-in/out parameter for users which if opted in adds this default policy (ideally a more restrictive one that only allows things in the account, but this would be good enough for the usability need):
{
  "Version":"2012-10-17",
  "Statement":[
    {
      "Sid":"AddPerm",
      "Effect":"Allow",
      "Principal": "*",
      "Action":["s3:GetObject"],
      "Resource":["arn:aws:s3:::<bucket_name>/*"]
    }
  ]
}

Managed to successfully launch a cluster using the generated templates.
Will leave the s3 bucket policy usability fix. Going to double check a downstream dcos distribution works with this, then commit later this evening/

Filed JIRAs for the major outstanding bits of work.


Changed/Deleted workspace should turn into untitled workspace

I think we need to listen to changes to the workspace config file and migrate a window into "Untitled Workspace" once:

the underlying config file gets deleted or renamed or moved to a different location

Looks like the a similar transition that we now have for Untitled => Workspace needs to be supported the other way around as well.

After discussion we will wait for more user feedback on this issue.

With our new workspace config format, it is more easy to get into this state.

After discussion we think we can install a watcher on the workspace file and if the file gets deleted:

show a dialog telling the user
provide an option to delete the workspace (which would bring the window back to empty state)
provide an option to restore the workspace (which would flush the last known configuration model back to the file)

@sandy081 one idea was to handle the deleted case more gracefully and not crash. I can see that when I delete a workspace while is is opened, immediately this crash happens:
shell.js:228 Cannot read property 'map' of undefined: TypeError: Cannot read property 'map' of undefined
    at parseWorkspaceFolders (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/platform/workspace/common/workspace.js:108:33)
    at Object.toWorkspaceFolders (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/platform/workspace/common/workspace.js:99:32)
    at WorkspaceService.onWorkspaceConfigurationChanged (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/configuration/node/configuration.js:427:53)
    at file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/configuration/node/configuration.js:219:110
    at CallbackList.invoke (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/callbackList.js:58:43)
    at Emitter.fire (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:91:40)
    at WorkspaceConfiguration.<anonymous> (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/workbench/services/configuration/node/configuration.js:536:132)
    at CallbackList.invoke (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/callbackList.js:58:43)
    at Emitter.fire (file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/common/event.js:91:40)
    at file:///Users/bpasero/Development/Microsoft/monaco/out/vs/base/node/config.js:152:53

It seems that the defaultConfig in here  is not a good choice because it will not have any folder set. Should be at least a config that maybe has an empty array for folders.

@bpasero Agreed showing empty workspace is better than what we have now. I would think still showing a message to user is good.

Pushed a change to have a better defaultConfig fallback that does not result in a crash.


Elixir: add Ockam.Worker.route/1,2 to generalize sending messages from workers.

Currently when workers send messages they need to create a message and call Ockam.Router.route/1
Ockam.Router.route/1 does not have information about the worker and the worker needs to create a message with all the metadata first to route it.
If we were using something like Ockam.Worker.route(message, state) then we could use some generic code to access the state and add metadata to the message and/or apply access control rules maybe preventing message sending.
This is a refactoring task to implement Ockam.Worker.route(message, state) as just calling Ockam.Router.route(message) and changing all usages of Ockam.Router.route/1 from context of workers to Ockam.Worker.route/2

Please assign to me

@lordB8r awesome, all yours, let us know if you have any questions.

@mrinalwadhwa Are there any examples of the state that this task will run? Or is this issue just to completely refactor from using Ockam.Router.route/1 -> Ockam.Worker.route/2, and a future PR will handle implementation of the state?

Are there any examples of the state that this task will run? Or is this issue just to completely refactor from using Ockam.Router.route/1 -> Ockam.Worker.route/2, and a future PR will handle implementation of the state?

state argument is there for future use.

Apologies for the long radio silence, just a quick question about tests.
In Ockam.Router.Tests, the describe block for testing the Router.route/1 would be changed to Worker.route/2. Should these tests then be moved to Ockam.Worker.Tests, even though the ultimate functionality is still testing the Router (we're just using the Worker as a way to handle State)

I'm going to leave the tests and just add a comment, but I'll add a test in Worker.Tests to cover these changes as best as I understood this issue.

@hairyhum @mrinalwadhwa I have the code ready, tested, and rebased, but cannot push because of permissions. Please advise.

@lordB8r that's awesome. We follow GitHub's fork and pull requests workflow so you can create a pull request from a fork without needing our permission. We'll review the request and then merge.
More info here: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork
Let us know if you more questions.

@mrinalwadhwa Is there anything else that I need to do to get a review? tests fail on my repo because I don't have permissions to run them. Unclear what next steps y'all need from me.


Cancel and speed up transactions

@colourfreak do you have the latest UI designs?
https://github.com/AlphaWallet/alpha-wallet-android/issues/1541#issuecomment-666378719 and the one in zpl://screen?pid=5d0356be2f25b319dcc5944b&sid=5f27e5cd3bd29121031f7f28 aren't consistent.

https://zpl.io/VDj1zeg

I did not get what is inconsistent. @hboon Please take a look one more time, maybe I missed this part.

I did not get what is inconsistent. @hboon Please take a look one more time, maybe I missed this part.

@colourfreak speed up is good now. Thanks. Do we have a similar design for cancelling a transaction? I think other than the additional button to "Cancel", the screens would be the same except for the copy.

https://zpl.io/addw4w5


Imports from multiple "rxjs" cases increases build file size

Bug Report
Current Behavior
I previously reported an issue in seeing large bundle sizes when building rxjs with webpack and typescript.

While diving back down this road to shrink the overall bundle, I noticed that it seemed that I had multiple sets of rxjs imports. Lo and behold, despite having only the one installed rxjs package, I had a typo in my project that pulled from "Rxjs" instead of "rxjs" in the import statement.
Correcting the case of the import shrunk the pre-minifcation size of the bundle by 40 KB and results in just the one section.

I'm not well-versed in all the variations of import statements these days (except to be thankful that they Just Work in my IDE) so I'm not entirely sure if there's anything you can do about this, but ideally you should only be able to import from "rxjs" and not other permutations of the case.
A clear and concise description of the behavior.
Reproduction

REPL or Repo link:
https://stackblitz.com/edit/rxjs-d4xgv9

Note that with the mixed case, Stackblitz will complain, but putting the same in my local VS Code instance will build without error (and show that unexpected bundling).
import { of } from "rxjs";
versus
import { of } from "Rxjs";
Expected behavior
I would expect that a warning be thrown indicating that there appears to be more than one case used across different imports of the same thing (though this may also be something better reserved for a linter) or, ideally, it Just Works and automatically uses the same case for all imports.
Environment

Runtime: Chrome 75.0.3770.90 64-bit
RxJS version: 6.3.3
Webpack: 4.28.2

Possible Solution
Tell me to go file this under tslint's project instead or perhaps as a bug for VS Code (given that Stackblitz properly catches the mismatched import names).
Additional context/Screenshots
See inlined image above.

I don't think who exports (rxjs) can control imports site (like Rxjs): this would be probably same across any import you do (i.e Lodash vs lodash). When we exports we can't prohibit other way around. Also weird to hear compiler doesn't catch it: as you noted stackblitz says message about it, this is more like compiler / bundler's job.

Perhaps TypeScript's forceConsistentCasingInFileNames compiler option would help prevent this sort of thing. I agree with @kwonoj: this is not a problem that RxJS can solve.


Work Package wp-fk.csv

I would like to work on this please! on the issue please!

Results of Phase 1
wp-fk-results.csv

I completed this work package.

I would like to work on this issue please!

Results of Phase 1
wp-fk-results.csv

Results of Phase 1
wp-fk-results.csv

I would like to work on this issue please!

Results of Phase 2
wp-fk-results.csv


Exchange to zero-th worker?

Hi, I've been using Timely for a while, while recently something wired happened, relating to the Exchange behavior.
To let all data go to the 0th worker for debugging purpose, I explicitly write:
exchange(|_| 0_u64) .inspect(|...| info!("Worker {} get results.", index))
in the dataflow. When I ran 4 workers (in one machine), it is fine. However, when I ran 5 or 6 workers,  I found out that it was Worker 4 (instead of 0) who printed out the results, as:

Any clue why? Thanks.

Thanks for the report! I just tried to reproduce this with a simple .exchange(|_| 0u64).inspect(...) and could not (on five and six workers). Do you have a reproduction you can say more about?

Ah! I've never seen this before. The thing I want to do (in a graph) is: to search n-cycles (or more precisely 3-cycles, ..., n-cycles) from a bunch of starting vertices, and print the number of n-cycles of the given vertices. The fragment of codes are:
     pub struct CycleParams {
        /// The starting vertices to start the traversal
        pub start_vertices: Vec<DefaultId>,
        /// The hop limit while traversing the graph
        pub n: usize,
        /// The top-k results to return
        pub k: usize,
        /// Control whether to search out neighbors only, or both neighbors
        pub is_out_neighbors_only: bool,
        /// Whether it is count_only
        pub is_count_only: bool,
    }

    lazy_static!{
        pub static ref GRAPHS: Vec<Arc<GraphStore>> = init_graph(DATA_PATH);
        pub static ref PARAMS: CycleParams = CycleParams::from_json_file(&(PARAM_PATH))
        .expect("Must specify EXP_PARAM_PATH=<cycle_param_dir>");
    }

    let num_workers = // compute from timely args;
    timely::execute_from_args(std::env::args(), move |root| {
        let index = root.index();

        let mut input = InputHandle::new();
        let mut probe = ProbeHandle::new();
        let mut results = HashMap::<DefaultId, Vec<PathData>>::new();
        let mut counts = HashMap::<DefaultId, usize>::new();
        let route_last =
            Arc::new(move |path: &PathData| {
                    let last = *path.last().unwrap();  (last * num_workers + last % num_workers) as u64 } );
        let route_first =
            Arc::new(move |path: &PathData| {
                    let first = *path.first().unwrap();  (first * num_workers + first % num_workers) as u64 } );

        root.dataflow::<u32, _, _>(|scope| {
            let stream = scope.input_from(&mut input).map(|v| vec![v]);
            let mut output_streams = Vec::new();

            let (handle, cycle) = scope.feedback(1);

            // go records all that does not form cycles
            // stop records all the forms cycles
            let (go, stop) = stream
                .concat(&cycle)
                .exchange(move |path: &PathData| route_last(path))  // PathData = Vec<usize>
                .flat_map(move |path| {
                    let v = path.last().unwrap();
                    let i = index % num_workers;
                    let graph = (*GRAPHS).get(i).unwrap();

                    let mut vec_iters = vec![graph.get_out_vertices(*v)];

                    if !(*PARAMS).is_out_neighbors_only {
                        vec_iters.push(graph.get_in_vertices(*v));
                    }

                    let iters = IterList::new(vec_iters);
                    let path_clone = path.clone();

                    iters
                        .map(|v| v.get_id())
                        .filter(move |u| {
                            let iter = if path_clone.len() < 3 {
                                Iter::from_iter(path_clone.iter())
                            } else {
                                Iter::from_iter(path_clone.iter().skip(1))
                            };
                            for exist in iter {
                                // remove duplicate
                                if u == exist {
                                    return false;
                                }
                            }
                            true
                        })
                        .map(move |u| {
                            let mut new_path = path.clone();
                            new_path.push(u);
                            new_path
                        })
                })
                .branch(|_, path| path.first() == path.last()); // split on whether forming cycle

            // stop.inspect(|cycle| println!("stop: {:?}", cycle));
            // Continue go if not reaching hop limit
            go.branch(move |_, path| path.len() <= (*PARAMS).n)
                 .1
                // .inspect(|path| println!("go checked hop limit: {:?}", path))
                .connect_loop(handle);

            // Insert stop into output stream
            output_streams.push(stop);

            scope
                .concatenate(output_streams)
                .unary_notify(
                    PactExchange::new(move |path: &PathData| route_first(path)),
                    "Cycle collecting",
                    None,
                    move |input, output, notificator| {
                        input.for_each(|time, data| {
                            notificator.notify_at(time.retain());
                            let cycles: Vec<PathData> = data.replace(Vec::new());
                            for cycle in cycles {
                                let first = cycle[0];
                                let result = results.entry(first).or_insert_with(Vec::new);
                                *counts.entry(first).or_insert(0) += 1;

                                if !(*PARAMS).is_count_only {
                                    if result.len() < (*PARAMS).k {
                                        result.push(cycle);
                                    }
                                }
                            }
                        });

                        notificator.for_each(|time, _, _| {
                            let mut session = output.session(&time);
                            if !(*PARAMS).is_count_only {
                                session.give_iterator(results.drain());
                            } else {
                                session.give_iterator(
                                    counts.drain().map(|(v, count)| (v, vec![vec![count]])),
                                )
                            }
                        });
                    },
                )
                .exchange(|_| 0_64)
                .inspect(move |(v, cycles)| info!("Worker {} {}: {:?}", index, v, cycles))
                .probe_with(&mut probe);
        });

        if index == 0 {
            for vertex in &(*PARAMS).start_vertices {
                input.send(*vertex);
            }
        }

        input.advance_to(1_u32);
        root.step_while(|| probe.less_than(input.time()));
    })
    .unwrap()
    .join();

Nothing stands out as an obvious culprit here. But, there are certainly things I don't understand going on. If you are able to minimize the reproduction to something I can build and run, I can take a peek. Also, the exchange operator is really simple, so you could in principle dive in there and see what is going on internally (e.g. the error seems to be present by that point or not). It just uses the Exchange pact and it would be surprising to learn that this pact is not correct.

Ha, it is really my bad. I exchange to 0_64 rather than 0_u64. Thank you your help anyway.


Registry v2 API util update

can this PR will fix /repositories issus ?
https://github.com/vmware/harbor/issues/170

@tinytub this is the expected behavior, that the repo_name parm has to contain the namespace of the repository

sorry for my last comment
i try branch https://github.com/ywk253100/harbor/tree/registry_api_util from @ywk253100
but https://github.com/ywk253100/harbor/tree/registry_api_util_update maybe not work fine.
I got an error like https://github.com/vmware/harbor/issues/202
and it has merged to harbor's master


Fix widget Radio::class, add documentation.

Q
A

Is bugfix?
✔

New feature?
✔

Breaks BC?
❌

Fixed issues
https://github.com/yiisoft/form/issues/97

👍


feat: sort husky hooks

Not sure is this #20 requested, we should sort them anyway.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CLI: Add a signal handler to reset the cursor on interrupt

In CLI we use cargo-generate for init and new.
cargo-generate uses things like Select from dialoguer.
Those things hide your terminal's cursor for aesthetic purposes.
If you press Ctrl + C while those things are active, you will end up with a cursorless terminal.
See: https://github.com/console-rs/dialoguer/issues/294
This PR adds a ctrlc signal handler to reset the cursor and exit.

Thanks! Much better


add djv validator to json schema suite

https://github.com/atomiqio/json-schema-test-suite
also to readme

https://github.com/ebdrup/json-schema-benchmark/pull/24


Genomics is wrong

@baimingze we have in the facet a type Geneomics(0) is must be Genomics.

done


tapOn taps on the view twice

We have a tag component that you can toggle on and off.
when using tapOn, maestro seems to tapping on that view twice, one selecting the tag and again unselecting it.
Is there a way to fix that?
Here's the screen record video: https://photos.app.goo.gl/R9JEjvDMnt3fFFpa9

Yes you can disable this behavior using retryTapIfNoChange: false:
- tapOn:
    text: your-text
    retryTapIfNoChange: false

Thanks Leland. I actually fixed the issue by setting a different a11y label for each selected/unselected state (i.e. Add Layout tag and Remove Layout tag).
But it's great to hear there's also a retryTapIfNoChange param.
I'm seriously blown away with this tool.
We spent weeks setting up Appium/Webdriver and it took me 1 hour last night to write 4x less code with Maestro.
At this point, the only blocker for us to migrate is Browserstack support to run on physical devices in the cloud.

That's awesome to hear @sregg. On physical device support - this is something that is on our radar!


Add more background on our myriad_compile suggested params

In https://github.com/luxonis/depthai-docs-website/blob/master/_tutorials/openvino_model_zoo_pretrained_model.md#run-myriad_compile, we suggest a number of argument values for myriad_compile, but we don't provide background on why.
Provide this information.

We have a notification in the API that warns the user about the optimal amount of shaves that should be used with a current pipeline to improve performance.
Also, we have a dedicated section about model compiling here, where we link to official OpenVINO sources the compile commands are mostly network-specific.
However, the tutorial section about myriad_compile here would need more description, as mostly there are just code block now

I'll assign myself to this task and circle back with PR that adds some comments on the arguments used


Is an update of react-query to v5 planned?

Hi, I'm currently working on updating react-query. MR should be coming soon


Getting 404 after running npm install

Hello,
I forked this project to use and followed the doc provided. However, after running npm install I get the following error:
$ npm install
npm ERR! code E404
npm ERR! 404 Not Found - GET https://registry.npmjs.org/files-pipe/-/files-pipe-0.0.7.tgz - Not found
npm ERR! 404
npm ERR! 404  'files-pipe@https://registry.npmjs.org/files-pipe/-/files-pipe-0.0.7.tgz' is not in this registry.
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

Am I doing something wrong? Any help is appreciated.

Hi @overgames
The issue has been resolved. Pull the newest changes and you should be able to install dependencies 😄


Turn off default OSM Layer

I'm using ol3 with my WMS, I need turn off the layer of OSM which is loading by default to use only my custom wms layer. How can I do that?
https://github.com/tombatossals/angular-openlayers-directive/blob/master/examples/040-layers-change-tiles-example.html
In this example the layer "osm" have set an active property and the layer visibility is switching by the button, but in my case and need just load my wms tile layer and I want to the OSM layer that OL3 isn't load by default, the example property is not working for me because if I set the OSM (which is loading apart of the default) if I set off manually the layer OSM the OSM default don't turn off.
I used this example
https://github.com/tombatossals/angular-openlayers-directive/blob/master/examples/053-layers-image-wms-example.html
<openlayers ol-center="campos.setDefault.marker" height="480px" width="100%">
		    		<ol-layer ol-layer-properties="wms"></ol-layer>
		    		<ol-layer ol-layer-properties="osm"></ol-layer>
		    		<ol-marker lat="campos.setDefault.marker.lat" lon="campos.setDefault.marker.lon"></ol-marker>
		    	</openlayers>

angular.extend($scope, {
        center: {
            lat: 19.839,
            lon: -90.52,
            zoom: 4
        },
        wms: {
                active: true,
                    source: {
                        type: 'ImageWMS',
                        serverType: 'mapserver',
                        url: 'http://<IP_ADDRESS>/cgi-bin/mapserv?map=/var/www/mymap.map',
                        crossOrigin: null,
                        params: {
                            LAYERS: '-----',
                            TILED: true,
                            FORMAT : 'image/jpeg',
                            attribution: '----'
                             },
                        serverType: ('mapserver')
                    }
                },
        osm :  {
                        name: 'OpenStreetMap',
                        active: false,
                        source: {
                            type: 'OSM'
                        }
        }
    });

Hi, have you tried setting the attribute custom-layers="true" in the  directive?

Thank you, with this attribute my problem was solved


Update cache of website

The website of Hugo uses Cloudflare to cache the assets. It's known that the cache will not be updated automatically if changes are made in stylesheets. A manually reset is necessary.
Currently, we use style-tags as an ugly workaround. Affected are:

Firefox menu fix #2036
The Algolia search form
The How to contribute to Hugo tutorial #2044
The updated quickstart #1888

@spf13: Please leave a note if you find time to update the cache. I would like to move the (temporary) CSS snippets in the content files.

I have started using https://github.com/carlmjohnson/scattered as an asset hasher for my personal Hugo blog. (Basically, I just run it and pass the hash to Hugo as an environmental variable that gets injected into the template.) Would there interest in having something like that integrated into Hugo? It would probably need to be a new built-in function to retrieve the name of the hashed asset.

@carlmjohnson,
There's an existing feature request for cache busting already open: https://github.com/spf13/hugo/issues/621.

Thanks for pointing me at the right issue.

@digitalcraftsman
I've refreshed the cache on cloudflare.
I believe that the problem here was actually a corrupt git repo on the server which was preventing automatic deploys of the website from happening (origin, not edge problem).
@carlmjohnson That would be a great feature addition for Hugo.
Ideally the implementation wouldn't require any template changes or support to work, but simply a config value or flag.
The hugo transform functionality is likely to be part of a good solution here. https://github.com/spf13/hugo/tree/master/transform
See how the absurl transformer is written https://github.com/spf13/hugo/blob/master/transform/absurlreplacer.go

I moved the CSS code from above into style.css. See ebefa47691d6419c1923d71946439c870c38aa14 and 48fce6110d965eac29a20ea57c4824e4a579ac17.


Price List Query API stale without VPC Endpoints when running karpenter in private subnets

Tell us about your request
When running Karpenter in an isolated private subnet (without any Internet egress via IGW or NAT gateways), we see the following error, which presumably means we are using stale pricing data.
ERROR   controller.aws.pricing  updating on-demand pricing, RequestError: send request failed
caused by: Post "https://api.pricing.us-east-1.amazonaws.com/": dial tcp <IP_ADDRESS>:443: i/o timeout; RequestError: send request failed
caused by: Post "https://api.pricing.us-east-1.amazonaws.com/": dial tcp <IP_ADDRESS>:443: i/o timeout, using existing pricing data from 2022-08-17T00:19:52Z  {"commit": "4b5f953"}

It does not seem like there is a VPC endpoint available for the Price List Query API, according to the list of services in https://docs.aws.amazon.com/vpc/latest/privatelink/aws-services-privatelink-support.html.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Since there is no VPC endpoint officially supported yet for the pricing API, at least a doc update on https://aws.github.io/aws-eks-best-practices/karpenter/#amazon-eks-private-cluster-without-outbound-internet-access would be great to warn that price data will be stale without.
Are you currently working around this issue?
No workarounds, just using the stale data.

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Why you say "warn that price data will be stale without", do you mean in the error message?
If so it already does that. The rest of the error message should tell you pricing data from what date will be used instead.

Sorry for the ambiguity, to clarify, I think it would make sense to warn this on the documentation itself linked above, so that users don't have to find out only at runtime, if up-to-date price data were important to them.

I opened two PRs to describe this issue: one in the Karpenter Troubleshooting guide and the other in Karpenter Best Practices.


Making annotation values readonly has being commented out in JDK11 migration and needs to be re-enabled

Description:
https://github.com/ballerina-platform/ballerina-lang/blob/master/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/desugar/Desugar.java#L1086
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Reopening since the fix was reverted with https://github.com/ballerina-platform/ballerina-lang/pull/26476 due to https://github.com/ballerina-platform/ballerina-lang/issues/26478.
We can fix this when fixing https://github.com/ballerina-platform/ballerina-lang/issues/26478.

Fixed in https://github.com/ballerina-platform/ballerina-lang/pull/28447.


[BUG] Custom Icon

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[ ] I did not answer truthfully to all the above checkboxes.

What problem did you encounter?
Well this isnt really a problem.. But anyways is there like a way to make the custom bloxstrap icons bigger because i think its too small for me. and i want it to be bigger is there a way to do that or maybe add like a bigger size format for the custom icons and thats really it I hope someone Responds Soon..
Bloxstrap Log
N/A

I don't really think you can do that.

Show a screenshot of what your bootstrapper looks like please


[$1000]  The tooltip has capital 'C' in the word Call unlike other tooltips with three words

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to web chrome
Go to any chat
Send message and hover over the 'Copy to clipboard' and 'Mark as unread'.
Notice that only the first letter is in the capital form
But now hover over the call icon and notice that the first letter of the Call is in the capital. (Even in web messenger, it's 'Start a video call', where 'c' is small)

Expected Result:
To maintain consistency between the other tooltips, the 'C' should be small in the tooltip for the call icon
Actual Result:
The 'C' is in the capital letter in the call icon tooltip
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number:  1.2.98-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/43996225/230997926-518af286-0318-4f13-a3ad-879f7a4fd7ab.mp4

Expensify/Expensify Issue URL:
Issue reported by: @priya-zha
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1681139252495869
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ecbf592ef39ed301
Upwork Job ID:<PHONE_NUMBER>278033920
Last Price Increase: 2023-04-11

Triggered auto assignment to @sakluger (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

@Expensify/design should we be following the same capitalization scheme for the Start a call hover text as we use for other hover texts? If so, I think I can easily change this myself in the code here.

I agree that we should be using sentence case everywhere, not title case. That's probably more of a marketing question but I think they would answer the same way.

Okay cool, I'll add external to get this moving. Should be quick and easy!

Current assignee @sakluger is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @0xmiroslav (External)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Capitalization of three-word tooltips is not consistent.
What is the root cause of that problem?
The literal defined in language file is not as expected:
https://github.com/Expensify/App/blob/67097aa547de76f86c91b6405d8c902e688c8159/src/languages/en.js#L110
What changes do you think we should make in order to solve the problem?
Modify the literal in the above mentioned line as expected: Make a call.
I think if we agree upon this capitalization format (i.e. Sentence-style Make a call not Title-style Make a Call), we should inspect the whole language file and modify where necessary.
Some of the same issues: https://github.com/Expensify/App/issues/17201, https://github.com/Expensify/App/issues/17243.
What alternative solutions did you explore? (Optional)
NA

Triggered auto assignment to @thienlnam (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
The tooltip has capital 'C' in the word Call unlike other tooltips with three words
What is the root cause of that problem?
Start a call button tooltip rendered via this line
https://github.com/Expensify/App/blob/5df7225e347253fdb537d282585774dc6cb4c040/src/components/VideoChatButtonAndMenu/BaseVideoChatButtonAndMenu.js#L105
Within translation file it consist this value
https://github.com/Expensify/App/blob/5df7225e347253fdb537d282585774dc6cb4c040/src/languages/en.js#L197-L199
We can see at line 197 there is a C in capital for Start a Call. This is the root cause of the problem.
What changes do you think we should make in order to solve the problem?
We have to make c small as shown below:
 tooltip: 'Start a call',

So this will solve the problem.
Note: I will also suggest to change translation key from tooltip to startACall so key remain consistence with other button keys like zoom and googleMeet etc.
What alternative solutions did you explore? (Optional)
None

Proposal
Please re-state the problem that we are trying to solve in this issue.
Inconsistent capitalization of the 'Start a Call' tooltip, unlike other three word tooltips.

^inconsistent with:

What is the root cause of that problem?
The tooltip references this string which is not defined correctly the en.js file.
https://github.com/Expensify/App/blob/67097aa547de76f86c91b6405d8c902e688c8159/src/languages/en.js#L110
What changes do you think we should make in order to solve the problem?
We should change the String for the tooltip to 'Start a call'
What alternative solutions did you explore? (Optional)
NA
(Hi, first time contributor here, I like React Native and have been wanting to contribute but I am intimidated with the complexity of the codebase. Since this is a simple change, Can I be assigned this as submitting this PR will help build my confidence.)

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01b1623e7bd66df3df

✅ Contributor details stored successfully. Thank you for contributing to Expensify!

@0xmiroslav  First proposal suggested to Change it Make a call actually it should be `'Start a call'. I think what I suggested https://github.com/Expensify/App/issues/17239#issuecomment-1503682900 is correct grammar to change.
I also suggested  to change translation key from tooltip to startACall so key remain consistence with other button keys like zoom and googleMeet etc.

@0xmiroslav For such small correction such grammar matters so I requested to consider it.

Great catch @0xmiroslav, I agree we should just take care of all mis-capitalization in this issue - I've commented on those other issues and so can we get another proposal here to handle all these capitalization differences in our language files?

Updated solution should check en.js and es.js and make sure we're following the pattern of just the first word capitalized

@thienlnam Wanted to ask, will I not be considered for the issue related to the 'Start a Call' bug?

@priya-zha The bug you reported was for incorrect capitalization, which was already reported a few days ago here.  It was for a different copy, but since it basically handles the same issue (incorrect capitalization) it is considered a duplicate and won't be eligible for a reporting bonus

📣 @dukenv0307 You have been assigned to this job by @thienlnam!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@0xmiroslav I just updated my proposal with some similar issues. Please help to check and confirm which one we should fix
cc @thienlnam

Great idea to review similar issues in the language file. Let's update numbers 1, 3, 4, 6, and 12.
Going through the reasoning for each proposed change:

Transfer balance - Yes, let’s use sentence case for both the link and the header
Tap Settings - No, “Settings” is the button name so it should be capitalized
Terms of Service - Yes, we should standardize to Terms of Service
Choose account - Yes, use sentence case
Expense Management/Spend Management/Expense Reports/Company Credit Card/REceipt Scanning App/Bill Pay - No, these are in the footer of the sign in page and should have Title case
Delete account - Yes, use sentence case
CPA Card - No, footer copy should be Title case
Press Kit - No, footer copy
No, footer copy
No, footer copy
Your Expensify Cards are ready to go - no, Expensify Cards is a product name
Uploading attachment - Yes, use sentence case

@0xmiroslav @thienlnam The PR is ready for review

@sakluger Is a better contact for confirming these copies - if the PR is delayed due to us the bonus amount will still apply so no worries there

Sorry for the delay. Yes, let's do sentence case for "Debit Card", "Business Days" and "Bank Account". So Debit card, Business days, Bank account.

By the way, since the delay here was my fault, this will still qualify for the efficiency bonus as long as we merge by tomorrow.

By the way, since the delay here was my fault, this will still qualify for the efficiency bonus as long as we merge by tomorrow.

Yeah I agree, going to notify Kavi to consolidate the duplicate issues for capitalization in here

Hi @0xmiroslav, @thienlnam, coming from the 17539 issue, if it is also solved along with this issue, will I be eligible for reporting bonus?

Sorry, same applies here since this is a duplicate
https://github.com/Expensify/App/issues/17239#issuecomment-1503880492

Hi @thienlnam , my point here is although the bug is about incorrect capitalization, 17239 or any discussion till now didn't focused on connect bank account validation capitalization. If not for 17539 report, we would have not worked on it and it may have again come up in future as new issue which would have been worked separately.
Also one small part different from capitalization in the report was step number 15 in action performed. It is about wrong terminology.

Yes, though that would apply to any part of the product that has inconsistent capitalization and any of the previous issues that mentioned incorrect capitalization.
Since there were so many different reports we consolidated the issues addressing inconsistent capitalization and are addressing all of them from our constants file. All these are providing different examples, but if some are missed that would be a regression from this issue which aims to resolve all 'inconsistent capitalization in product'

I appreciate the initiative of working all these issues together, it will rectify many issues at once but am not sure if its fair to every reporter as they have contributed in it but won't be rewarded only because all of them are worked under one PR. Can you also let me know if point 15 of action's performed in #17539 will also be taken care in this issue? It is not about capitalization.

Can you also let me know if point 15 of action's performed in https://github.com/Expensify/App/issues/17539 will also be taken care in this issue? It is not about capitalization.

Thanks for pointing this out - I actually reopened your issue because it is not related to capitalization and cleaned up the issue there.

I appreciate the initiative of working all these issues together, it will rectify many issues at once but am not sure if its fair to every reporter as they have contributed in it but won't be rewarded only because all of them are worked under one PR.

As for this, I understand the concern about fairness - I'm consolidating them because the underlying issue for all of these is incorrect capitalization which can be resolved by just looking at the constants file where all of our strings exist and so here we're just giving the bug report to the first person that reported the problem.
For example, if there is a bug in a component which is used in multiple locations - there are typically multiple bug reports for a problem in different parts of the app. In this case, we'll resolve the bug for the first issue and then we just close the rest of the bug reports because they were duplicates.

@0xmiroslav with "Tax ID Number", "PO Box" it should be all in normal-case or "ID", "PO" is still in capital

@sakluger ping again in case you miss it, please help to check this comment

Yes "P" should be capitalized.

Reviewing label has been removed, please complete the "BugZero Checklist".

The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.4-0 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:

https://github.com/Expensify/App/pull/17327

If no regressions arise, payment will be issued on 2023-05-02. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.

[ ] External issue reporter
[ ] Contributor that fixed the issue
[ ] Contributor+ that helped on the issue and/or PR

As a reminder, here are the bonuses/penalties that should be applied for any External issue:

Merged PR within 3 business days of assignment - 50% bonus
Merged PR more than 9 business days after assignment - 50% penalty

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ] [@0xmiroslav] The PR that introduced the bug has been identified. Link to the PR:
[ ] [@0xmiroslav] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
[ ] [@0xmiroslav] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:
[ ] [@sakluger] Determine if we should create a regression test for this bug.
[ ] [@0xmiroslav] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ] [@sakluger] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:

Just noticed this one never switched back to Daily and I missed it, sorry about that! I'm reviewing now to see what is outstsanding before paying out.

Based on the timeline including times waiting for copy, I think this should qualify for the efficiency bonus.
@ayazhussain79 @dukenv0307 @0xmiroslav I've sent you all offers in Upwork.

@sakluger offer accepted

All paid.


There is already a Construct with name 'adminGroupRole' in AmplifyUserPoolGroupStackOutputs

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting bug reports.
[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.

App Id
d3u8gr5p05kbef
Region
ap-northeast-2
Amplify Console feature
Backend builds
Describe the bug
This error occurs in the process of pushing and building a new source code in the amplify console. I don't know where the problem came from. I'd appreciate it if you could help me.
Expected behavior
Backend build should be completed normally and frontend build should be started.
Reproduction steps

push the new source code to github
Provisioning from the amplify console
Setup build completed.
Backend build error.

Build Settings
version: 1
backend:
  phases:
    build:
      commands:
        - '# Execute Amplify CLI with the helper script'
        - amplifyPush --simple
frontend:
  phases:
    preBuild:
      commands:
        - yarn install
    build:
      commands:
        - yarn run build
  artifacts:
    baseDirectory: out
    files:
      - '**/*'
  cache:
    paths:
      - node_modules/**/*

Additional information

Hi @seunghwako, sorry for the delay in response. I see you were able to successfully build not long after submitting this issue. Were you able to resolve the problem?

Closing this due to inactivity


Migrate rke launched cluster to rancher launched

What kind of request is this (question/bug/enhancement/feature request):
Question
Question
I currently have a relatively big rancher cluster that was launched using RKE, however, since a lot of people are using this cluster and we'd like to administrate the cluster using the ui instead of config files. A rancher launched cluster has the option in the to update/take snapshots and much more straight from the ui which we also would like. Setting up rke on everyone's pc who wishes to administrate the cluster is time-consuming and the person needs prior knowledge of rke to use it, and we'd like to avoid that if we can. Would it be possible to convert a rke launched cluster to the "rancher launched" version instead?

Not at this time, this is covered in https://github.com/rancher/rancher/issues/22763


Add aarch64[arm64] support

Fix: Add support to upload aarch64 docker image on docker hub. Closes https://github.com/github-changelog-generator/docker-github-changelog-generator/issues/33
You need to feed docker hub credentials into Travis builds as DOCKER_USERNAME and DOCKER_PASSWORD to upload the docker images on docker hub.
Signed-off-by: odidev<EMAIL_ADDRESS>

Can you please review and let me know if any changes are required?

Can you please review and let me know if any changes are required?

Thanks for the PR! Now that Travis limited build minutes for OSS projects, we might be better moving to GH actions. I'll think about that...

Thanks for the PR! Now that Travis limited build minutes for OSS projects, we might be better moving to GH actions. I'll think about that...

Closing in favor of using Docker Inc. actions. I'm working on those right now.


Cannot read property '0' of undefined

While doing a readFile for .xls file following error occurs.
switch(state[state.length-1][0]) {
^
TypeError: Cannot read property '0' of undefined
at parse_xlml_xml (/home/abc/node_modules/xlsx/xlsx.js:8815:32)
at parse_xlml (/home/abc/node_modules/xlsx/xlsx.js:9171:53)
at readSync (/home/abc/node_modules/xlsx/xlsx.js:11393:21)
at Object.readFileSync (/home/abc/node_modules/xlsx/xlsx.js:11403:9)
at Object. (/home/abc/readXL.js:2:20)
at Module._compile (module.js:541:32)
at Object.Module._extensions..js (module.js:550:10)
at Module.load (module.js:458:32)
at tryModuleLoad (module.js:417:12)
at Function.Module._load (module.js:409:3)
please help me out

I meet the same problem. I solve it by save the document again.

@vineetl @chenvan can you share a file that shows this problem?  I suspect those are really HTML files that happen to have the XLS extension, since that particular error and codepath mean the file started with a < (which the library guesses is a SpreadsheetML file)

@SheetJSDev you guess is maybe right. My document is exported from a website which is about chinese stock markets.   I don't know how to send to you? An email?

@chenvan You can send an email to<EMAIL_ADDRESS>

@chenvan received the file.  As I guessed, it was an HTML file.   This export is particularly interesting because of the number format hints in the style:
<td style="text-align:right;mso-number-format:0.00;">

We'll keep this issue open until we decide how to handle HTML files.


Add countdown timer until users can buy more packs

Need to rework this to use UTC so timezones don't mess it up


[Bug] : Can we add resize to the graphic element?

Version
5.4.3
Link to Minimal Reproduction
https://echarts.apache.org/examples/en/editor.html?c=line-graphic&edit=1&reset=1
Steps to Reproduce
I want to resize the parent graphic element
Can we achieve this for the graphic elements?
Current Behavior
NA
Expected Behavior
Resize event for the graphic elements
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

You need to do this part of the logic yourself, and the general strategy is as follows:

Specify the width
Calculate the number of characters in a line based on its width
Calculate the number of rows based on the total number of rows
Calculate the height based on the row height.

Can you update in the sample? @RypAsuka-Akira

updated Example
📌 please close issue if problem solved.

@helgasoft  not on click of graphic element you are resizing it I want to resize on click but not statically like we have text element resize same behavior I want for the graphic element


"Inherit AL XML Documentation Comment" results in 'badly formed XML' if parameter type object name contains spaces (and therefore surrounded by quotes in the signature)

Dear @365businessdev,
If you use the command "Inherit AL XML Documentation Comment" the parameter types are added in the reference to the original procedure in the interface.
If a parameter type is an object with a name that needs to be surrounded by quotes, e.g. "Temp Blob", then this results in a warning message:

Also, hovering over the procedure does not give the documentation in this case.
Could you look into this?

Perhaps you could escape the quotes with "?

Sorry for the delay in fixing your supported issue!
We have recently released the version 1.0.5 which fixed this issue.

Sorry for the delay in fixing your supported issue!
We have recently released the version 1.0.5 which fixed this issue.


TypeError: needs a 'context' argument.

I'm using mac and jsdom version 4.0.1
and following is my code
   jsdom.env({
                    html: body,
                    scripts: ['http://code.jquery.com/jquery-1.6.min.js']
            , done: function(err, window){
                    var $ = window.jQuery;

                    console.log($('title').text());
                    res.end($('title').text());
                  }
            });
    });

I get the following error
TypeError: needs a 'context' argument.
at new Window (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/jsdom/lib/jsdom/browser/Window.js:32:26)
at Object.exports.jsdom (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/jsdom/lib/jsdom.js:46:16)
at processHTML (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/jsdom/lib/jsdom.js:220:24)
at Object.exports.env.exports.jsdom.env (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/jsdom/lib/jsdom.js:122:5)
at Request._callback (/Users/lgmagic/Techwalk/nettuts/nodetube/app.js:31:15)
at Request.self.callback (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/request/request.js:344:22)
at Request.emit (events.js:98:17)
at Request. (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/request/request.js:1239:14)
at Request.emit (events.js:117:20)
at IncomingMessage. (/Users/lgmagic/Techwalk/nettuts/nodetube/node_modules/request/request.js:1187:12)

What version of io.js are you using?

version 0.1.6

Hmm, there is no 0.1.6 version of io.js, so I am not sure what you are saying.

I guess i skipped a few things here.. Im new to Node and IO. I thought you were referring to io module of Node.js
i was trying to build with Node.js and
just read the documentation that it is no longer supported with Node.js and works only with io.js.
My bad. i think you should mark this issue invalid.


Update README.md

Add CodeGeeX, an AI coding extension developed by Zhipu

We also include codegeex on line 47 - wanna merge the entries?

I see the situation, the existing entry is the model, and he also has a vscode extension called CodeGeeX. So, are you suggesting to save it in the form of a record plus notes?

I see the situation, the existing entry is the model, and he also has a vscode extension called CodeGeeX. So, are you suggesting to save it in the form of a record plus notes?

sounds good


style fix

closes #146 #94 #154 #147

Hey @hanbyul-here, why would you prefer to use if/else? We can chat about this in our UI polish session today if you'd like.

@meghanhade  I tried using {{#if}}{{else}}{{/if}} but coudln't get it working, it could only catch true :( I also think it is better to use if/else if it works, please feel free to edit this!

@hanbyul-here related: https://github.com/transitland/feed-registry/issues/174