Confusing 'conditional expression without else clause' error with a yaml error before the if statement

What steps did you take:
I attempted to template a yaml file with some invalid yttt syntax:
...
  - #@ "--webhook-port={}".format(webhook_port <-  MISSING PARENTHESIS
#@ if image_pull_secrets_enabled:
imagePullSecrets:
- name: #@ image_pull_secrets_name
#@ end

What happened:
An error that suggested that my if block was missing an else clause was thrown:
ytt: Error:
- conditional expression without else clause
    deployment.yaml:48 |       #@ if image_pull_secrets_enabled:

What did you expect:
Ideally, an error that would point out the line that has the error.
Failing that, an error that would suggest that the issue might be before the if block as the current error seems to strongly suggest that the issue is with the if block.
Environment:

ytt version: v0.36.0

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Thanks for the simple example (reproduced in Playground).
Wow, this is an unfortunate error message.
TL;DR — at this point changing this behavior may be very hard.
So, under the covers, ytt uses a slightly modified Starlark. That runtime is what's compiling and executing this code.
One of the modifications is the inclusion of the end keyword and therefore whitespace/indent-insensitive parsing.
In Starlark, the if keyword can either be a statement or a conditional expression.
Some inside baseball: ytt compiles all YAML templates into Starlark programs that describe how to build the resulting YAML.
For example, this template gets compiled to:
...
__ytt_tpl2_start_node(6)
__ytt_tpl2_set_node(6, ("--webhook-port={}".format(webhook_port))
if image_pull_secrets_enabled:
__ytt_tpl2_start_node(7)
__ytt_tpl2_start_node(8)
__ytt_tpl2_start_node(9)
__ytt_tpl2_start_node(10)
__ytt_tpl2_start_node(11)
__ytt_tpl2_set_node(11, (image_pull_secrets_name))
end
__ytt_tpl2_start_node(12)
...

(compiled template listings are included in the --debug output)
It's subtle, but given that there's one fewer parens on that second line of the listing and that Starlark in ytt is whitespace/indentation-insensitive (meaning that the next line beginning with if ... appears to the parser to be a continuation of the previous line), that subsequent if keyword appears to be modifying the second parameter of the __ytt_tpl2_set_node() call...
i.e. the if is marking a conditional expression ... and thus "technically" the error message is correct.
I absolutely get that this is not how it reads to the eye within the template. So, I want to chew on this for a bit.  As it stands, tho, I'm currently not bullish it's feasible to fix this error message. 😬

Thanks for the detailed reply 👍🏼

@pivotaljohn hmm, i wonder if we can run set node lines through its own ast parse... should that always succeed, i think so... in theory it should be self-contained (though symbols wont be resolved).

@pivotaljohn hmm, i wonder if we can run set node lines through its own ast parse... should that always succeed, i think so... in theory it should be self-contained (though symbols wont be resolved).

🤔  nice.  Does this hold for when we're replacing too? I think so... hmmmmm.

This is related to other less-than-great UX in templates:

Is it possible to pass the params of a function on several lines to improve the readability. Something like
 --- #@ generate_certificate(harbor_core_internal_tls_cert,
                             harbor_namespace,
                             harbor_cert_duration,
                             harbor_cert_renew_before,
                             harbor_core_internal_tls_secret,
                             [harbor_organization],
                             harbor_core_common_name,
                             generate_dns_names(harbor_core_common_name, harbor_namespace),
                             harbor_ca_issuer)

src: https://kubernetes.slack.com/archives/CH8KCCKA5/p1649264496571349


[Feature]: Datanode support to manage both SSD and HDD disks

Contact Details
<EMAIL_ADDRESS>Is there an existing issue for this?

[X] I have searched all the existing issues

Is your feature request related to a problem? Please describe.
In our use cases, there are only few machines available to deploy cfs cluster , eg 3 or 5. Those machines are both equipped with SSD and HDD disks.  For some cold data, we want to migrate them from SSD to HDD.
As I know, the developing Hybrid-cloud branch support data cool-down feature, but need to deploy at least two zones , in each zone the nodes are limited to configure same type of disks.  when migrating, the cold data are transfered from SSD zone to HDD zone.
The solution now  is not friendly for small cfs clusters,  eg. nodes less than 6 .  Also, for machines with both SSD and HDD disks, the migration process can not leverage Locality to imporve performance.
Describe the solution you'd like.
As a solutiion,  I think the datanode should have the ability to manage both SSD and HDD which seems more reasonable.  As a result,  even the cluster with 3 nodes (with both SSD and HDD, one zone) can experience the data cool-down feature, Meanwhile, cubefs can optimize cold data migration performance. When migrating cold data, prefer to choose HDD directories that on the same node as the destination,  which can reduce  network traffic and improve migration performance greatly.
Describe an alternate solution.
As a alternate solution, we can deploy cfs in containers, so we can deploy two zones on three nodes cluster. Nodes in each zone manage a single type of disks.   Nodes configure secondary IP also can help.
But can't use locality to reduce  network traffics caused by data migration
Anything else? (Additional Context)
No response

We have a rough idea :

a datanode can configrue different type of disks, so zone also can have multiple media types.
when create volume , if specified two kind of storage class,  prefer to create  paired SSD and HDD data parittion on the same data node .
when lcnode trigger migration task.  First try to transite cold data from SSD srcDp to HDD dstDp which located on the same data node with SSD srcDp.
a.  get extents list of the migrating file
b.  group the extents by data partition (srcDp)
c.  for each srcDp , select a  dstDp which is co-located to srcDp .then build a extentsLocalTransition request which contains the local transition context and will be sent to datanode
d.  send request to target datanode on which first dp replica located (repl-protocol)
e.  all datanodes with dp replicas  perform local transition , read from srcDp  extents and write to extents on dstDp . And first replica datanode return the inode migrated extents list to lcNode
g. lcNode batch update inode metadata of the migrating file

4)  If local transition failed , then use the original workflow to transite data across nodes

we have  roughly implement a draft version , and verify the idea. https://github.com/cubefs/cubefs/pull/3243
The local transition process described above is faster than the original and even more faster than file upload process. In my environment（4c,8G ,1000M netwrok）,a 4GB file local migration takes about more than 10 seconds(even faster than s3 put which takes about 30 seconds) , and the original cross-node migration process takes about 90 seconds.

This solution requires significant changes, and we can only consider whether to incorporate this approach after the completion of the first phase of the HybridCloud project.


Error "request signature does not match" in docker build

While troubleshooting another issue, I'm getting intermittent failures when trying to download a file from PyPI. I've gone to the netifaces project and found the source file URL for the latest version (netifaces-0.11.0.tar.gz). I downloaded the file in one environment, but needed it in a Linux environment, so created this docker file:
FROM jaraco/multipy-tox

RUN pipx install httpie
RUN http https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz

When I build that file, I get the following:

 draft $ docker build --no-cache --progress plain .
#1 [internal] load build definition from Dockerfile
#1 sha256:45cdf4a0d3c13eec3e3eee801bcdeabc79a6bd6c31b932948ceba681c19a379f
#1 transferring dockerfile: 236B done
#1 DONE 0.0s

#2 [internal] load .dockerignore
#2 sha256:2f53bf71ca425d8b0cab4302f38b5bbfe98f422372dd31e6b9a9b1525ef7149c
#2 transferring context: 2B done
#2 DONE 0.0s

#3 [internal] load metadata for docker.io/jaraco/multipy-tox:latest
#3 sha256:f99bb5b255811eb5e0f34320a22686fd80d95c00183c070bbbbd89d1ab9edebe
#3 DONE 0.0s

#4 [1/3] FROM docker.io/jaraco/multipy-tox
#4 sha256:6d547c8e790803b81751627135b715e872a53824cb7b59ca5cda8bdbb227b897
#4 CACHED

#5 [2/3] RUN pipx install httpie
#5 sha256:e4331792e715ba4e6453fb3d4aa344c15e4155afccf34804ced7561d93e5fd44
#5 0.234 creating virtual environment...
#5 0.275 installing httpie...
#5 4.622   installed package httpie 3.2.1, installed using Python 3.11.0rc1
#5 4.622   These apps are now globally available
#5 4.622     - http
#5 4.622     - httpie
#5 4.622     - https
#5 4.622 done! ✨ 🌟 ✨
#5 DONE 4.7s

#6 [3/3] RUN http https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz
#6 sha256:7a080ea1ed046076fe2ac044f5af8de21be7c0aac41c98ef01512678583ffb89
#6 0.559 <?xml version='1.0' encoding='UTF-8'?><Error>SignatureDoesNotMatch<Message>The request signature we calculated does not match the signature you provided. Check your Google secret key and signing method.</Message><StringToSign>POST
#6 0.559
#6 0.559 application/json
#6 0.559 Mon, 05 Sep 2022 14:19:08 GMT
#6 0.559 /pypi-files/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz</StringToSign></Error>#6 DONE 0.6s

#7 exporting to image
#7 sha256:e8c613e07b0b7ff33893b694f7759a10d42e180f2b4dc349fb57dc6b71dcab00
#7 exporting layers
#7 exporting layers 0.1s done
#7 writing image sha256:e897f3a1550025b72c3ee0b0fef8a515004eaeeeeb5f201afc06f59bc6805a69 done
#7 DONE 0.2s

Use 'docker scan' to run Snyk tests against images to find vulnerabilities and learn how to fix them

In particular, the request fails with:
#6 [3/3] RUN http https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz
#6 sha256:7a080ea1ed046076fe2ac044f5af8de21be7c0aac41c98ef01512678583ffb89
#6 0.559 <?xml version='1.0' encoding='UTF-8'?><Error>SignatureDoesNotMatch<Message>The request signature we calculated does not match the signature you provided. Check your Google secret key and signing method.</Message><StringToSign>POST
#6 0.559
#6 0.559 application/json
#6 0.559 Mon, 05 Sep 2022 14:19:08 GMT
#6 0.559 /pypi-files/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz</StringToSign></Error>#6 DONE 0.6s

Curiously, if I run the same http command interactively or as part of the CMD, it works.
The error message seems a little suspicious, as I'm not contacting Google and I'm not supplying a secret key nor signing anything, so it seems to me it's something between warehouse and its backend that's failing.

Aha, so when I run the http command with -v, I get a hint as to what's going wrong:
#6 [3/3] RUN http -v https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz
#6 sha256:32b52cd0eaaf7afc5f1109412d771137b8bed3a091b67b33d2612d9e3aca13c1
#6 0.507 POST /packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz HTTP/1.1
#6 0.507 Accept-Encoding: gzip, deflate
#6 0.507 Connection: keep-alive
#6 0.507 Content-Length: 0
#6 0.507 User-Agent: HTTPie/3.2.1
#6 0.507 Accept: application/json, */*;q=0.5
#6 0.507 Content-Type: application/json
#6 0.507 Host: files.pythonhosted.org
#6 0.507
#6 0.507
#6 0.507
#6 0.507 HTTP/1.1 403 Forbidden
#6 0.507 Connection: keep-alive
#6 0.507 Content-Length: 426
#6 0.507 Content-Type: application/xml; charset=UTF-8
#6 0.507 Server: UploadServer
#6 0.507 Accept-Ranges: bytes
#6 0.507 Date: Mon, 05 Sep 2022 14:25:15 GMT
#6 0.507 X-Served-By: cache-bfi-krnt7300064-BFI, cache-iad-kiad7000036-IAD
#6 0.507 X-Cache: MISS, MISS
#6 0.507 X-Cache-Hits: 0, 0
#6 0.507 X-Timer: S1662387916.806833,VS0,VE119
#6 0.507 Strict-Transport-Security: max-age=31536000; includeSubDomains; preload
#6 0.507 X-Frame-Options: deny
#6 0.507 X-XSS-Protection: 1; mode=block
#6 0.507 X-Content-Type-Options: nosniff
#6 0.507 X-Permitted-Cross-Domain-Policies: none
#6 0.507 X-Robots-Header: noindex
#6 0.507
#6 0.507 <?xml version='1.0' encoding='UTF-8'?><Error>SignatureDoesNotMatch<Message>The request signature we calculated does not match the signature you provided. Check your Google secret key and signing method.</Message><StringToSign>POST
#6 0.507
#6 0.507 application/json
#6 0.507 Mon, 05 Sep 2022 14:25:15 GMT
#6 0.507 /pypi-files/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz</StringToSign></Error>#6 DONE 0.5s

It seems that when http is invoked non-interactively, it's setting the Content-Type to application/json, but it doesn't do that when I run it interactively:
 draft $ http -v https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz
GET /packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz HTTP/1.1
Accept: */*
Accept-Encoding: gzip, deflate
Connection: keep-alive
Host: files.pythonhosted.org
User-Agent: HTTPie/3.1.0

HTTP/1.1 200 OK
Accept-Ranges: bytes
Access-Control-Allow-Headers: Range
Access-Control-Allow-Methods: GET, OPTIONS
Access-Control-Allow-Origin: *
Age: 5881416
Cache-Control: max-age=365000000, immutable, public
Connection: keep-alive
Content-Length: 30106
Content-Type: application/x-tar
Date: Mon, 05 Sep 2022 14:25:55 GMT
ETag: "3146dcb3297dd018ae5eb9a52b440419"
Last-Modified: Mon, 31 May 2021 08:33:03 GMT
Server: UploadServer
Strict-Transport-Security: max-age=31536000; includeSubDomains; preload
X-Cache: HIT, HIT
X-Cache-Hits: 7, 1
X-Content-Type-Options: nosniff
X-Frame-Options: deny
X-Robots-Header: noindex
X-Served-By: cache-bfi-krnt7300065-BFI, cache-iad-kiad7000176-IAD
X-Timer: S1662387955.306892,VS0,VE1
X-XSS-Protection: 1; mode=block
x-goog-generation:<PHONE_NUMBER>154165
x-goog-hash: crc32c=jupVYg==, md5=MUbcsyl90BiuXrmlK0QEGQ==
x-goog-metageneration: 1
x-goog-stored-content-encoding: identity
x-goog-stored-content-length: 30106

+-----------------------------------------+
| NOTE: binary data not shown in terminal |
+-----------------------------------------+

Aha. So it seems the issue is GET vs POST. I can replicate the failure interactively with this command:
http -v POST https://files.pythonhosted.org/packages/a6/91/86a6eac449ddfae239e93ffc1918cf33fd9bab35c04d1e963b311e347a73/netifaces-0.11.0.tar.gz

It's surprising to me that http would selectively issue a POST vs GET action based on when the command is run, but there's probably a good reason for it.
So the basic issue is not rooted in warehouse, but it the usage of HTTPie.
Still, I recognize some things that warehouse could do to make this issue less mysterious:

Provide a better error message when an invalid POST request is made. Instead of "signature does not match", something like "empty POST to this resource is invalid; did you mean GET?"
Avoid forwarding requests to Google when they won't be expected to succeed.

I don't feel strongly that anything in warehouse needs to be done, so I leave it to maintainers to ascertain the next steps from here (if any).


Update to allow for Disabled Scrolling in WeekView

Fixes: #522 and #556
This PR allows for propagation of the disableScrollingBeforeDate() and disableScrollingBeyondDate() functions in .weekView

Please someone accept, very useful :)


UI control for showCoverageOnHover (for clustered data)

Related to  https://github.com/bcgov/smk/issues/103/
It would be beneficial for SMK-CLI (SMK Editor) users to be able to toggle on showCoverageOnHover for a given vector layer.
Value should be false by default, please.

This issue was addressed by pull request https://github.com/bcgov/smk-cli/pull/75 and merged to master.


[RFC] Intel GPU (XPU) Support with Triton Implementation

🚀  1. The feature
This RFC proposes using Triton language to implement Intel GPU (torch.xpu) kernels.
1.1. Motivation
This RFC aims to discuss the XPU kernel implementation with the Triton language. The Triton kernel will contain most of the device-agnostic logic, and the device-specific logic will be implemented separately.
The only package requirement for this is triton package. However, as triton is by default the requirements of torch, we could ignore this package requirements in torchvision.
2. Op List

[ ] nms
[ ] roi pool
[ ] roi align
[ ] ps roi pool
[ ] ps roi align

3. File Structure
torchvision
├── csrc
│   └── ops   # This will contain the current op registration in C++, will be no change
│       └── nms.cpp
└── ops        # This folder is pure Python
  ├── triton # The shared Triton kernel implementation. It won't register the kernel.
  │   └── nms_kernel.py
  └── xpu    # The device-specific kernel implementation and registration
    └── nms_kernel.py

4. Details
4.1. Register Custom Kernel Implementation
The kernel implementation consists of two parts. Firstly, we aim to use Triton to implement device-agnostic logic to cover as much of the logic as possible. Secondly, we will use ATen operations for the logic that Triton does not support and device-specific logics.
4.1.1. Triton Kernel Implementation
To extend the generality, we propose to implement the main logic using the Triton language. This is device-agnostic. As long as the device/platform support Triton, the Triton kernel should work.
# torchvision/ops/triton/nms_kernel.py
@triton.jit
def triton_nms_kernel(boxes, scores, output_ptr, threshold, num_boxes, BLOCK_SIZE: tl.constexpr):
  pid = tl.program_id(0)
  block_start = pid * BLOCK_SIZE
  offsets = block_start + tl.arange(0, BLOCK_SIZE)
  ...

4.1.2. XPU Kernel Implementation and Registration
This will implement device specific logic. There are two aspects:

Triton unsupported ops. For example, tensor initialization, argsort, etc.
Device-specific Triton configuration. This would affect the performance of the kernel (e.g., pass Triton config to the kernel).

The developer will have to wrap the above logic into a custom kernel and register the kernel to PyTorch using torch.library.register_kernel(kernel_name, device).
# torchvision/ops/xpu/nms_kernel.py

@torch.library.register_kernel("torchvision::nms", "xpu")
def xpu_custom_nms_triton_kernel(boxes: Tensor, scores: Tensor, threshold: float) -> Tensor:
  # 1 : Perform operations that Triton doesn't support:
  # - Tensor initialization, which may be device-specific
  output = torch.ones(num_boxes, dtype=torch.int32, device=boxes.device)
  # - Use ATen calls for Triton unsupported ops (argsort in this case)
  order = torch.argsort(scores, descending=True)
  boxes = boxes[order]
  ...

  # 2 : Call Triton Kernel
  # Hardware vendors can decide which configuration is optimal for their device.
  # For example, BLOCK_SIZE in this case.
  grid = lambda meta: (triton.cdiv(num_boxes, meta['BLOCK_SIZE']),)
  triton_nms_kernel[grid](boxes, scores, mask, threshold, num_boxes, BLOCK_SIZE=meta['BLOCK_SIZE'])

  # 3 : Post Processing Logic
  output = torch.narrow(...)

  return output

In this way, the user could call the torchvision.ops.nms with both eager mode and torch.compile.
5. Run The Kernel
5.1. Eager Mode
The user can directly call the kernel in eager mode. The nms call will fall into the custom implementation in this scenario.
output = torchvision.ops.nms(boxes, scores, threshold)
print(output)

5.2. torch.compile Mode
The user can call torch.compile with the following:
nms_compile = torch.compile(torchvision.ops.nms)
compiled_output = nms_compile(boxes, scores, threshold)

The torch.compile will use inductor compiler to decompose ops into primitive ops. When the op does not have registered the decomposition, it will fallback to ATen call. i.e., In this scenario, the inductor won't decompose our custom kernel, and it will behave the same as eager mode.
6. Alternatives

Native C++ Implementation:

Currently, all the kernel backends are in torchvision/csrc/ops and are written in C++. This way requires every device backend developer to implement using their own compiler/language (for example, CUDA with nvcc). The pros of native C++ are that it could reach peak performance, and it will have Windows support.
By comparison, Triton has some limitations. For example, Triton does not support manually putting the tensor into the shared memory, thus the logic in CUDA nms kernel could not be implemented in Triton. However, the pros of Triton are that Triton is more general with reasonable performance. The optimization logic in Triton can be shared by different devices.
7. Limitations

Windows Support: Currently, Triton does not support Windows. Thus, kernels implemented with Triton will fail on Windows. However, when the Triton supports the Windows platform, we don't need any additional efforts.
Performance Alignment: Since the Triton has some limitations, the performance may not fully align with native C++ implementation.

8. Additional context

What if one backend needs to call Triton kernel when torch.compile and C++ implementation when in eager mode?

In this case, one may register as a custom lowering. Like the following code:
@register_lowering(torchvision.ops.nms)
def nms(...):
  if device == 'xpu' and USE_ATEN_BACKEND:
     aten_cpp_nms(...)
  elif device == 'xpu' and USE_TRITON_BACKEND:
     triton_nms(...)
  else:
     ...

cc: @EikanWang @fengyuan14  @riverliuintel

@NicolasHug , may I know your comments?

This is interesting! ❤


[ Issues 10786 ] Fixed flaky test testTransactionMetaStoreAssignAndFailover (#10786)

Fixes #10786
Master Issue: #10786
Motivation
Fixed flaky test.
Modifications
Replaced check after sleep to untilAsserted and extended atMost time to 10 seconds(default of Awaitility)

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks

/pulsarbot rerun-failure-checks


R model with string input and output.

Can someone give me an example of how an R function with string input and output would look like.
I have tried:
pred_fn = function(str) {
	return("value")
}

and then tested like:
curl -X POST --header "Content-Type:application/json" -d '{"input": "value"}' <IP_ADDRESS>:1337/app1/predict
It didn't work and I see this error:
 what():  Number of model outputs: 1 does not equal the number of inputs: 2
But I see both of these are strings of same length.

nevermind, I was doing something wrong, in the above function I wasn't passing and return a list which is essential for API-compatibility.
So I did:
pred_fn = function(str) {
	return(str)
}

and test with:
curl -X POST --header "Content-Type:application/json" -d '{"input": "c(\"Very Poor\")"}' <IP_ADDRESS>:1337/app1/predict

We use adaptive batching to figure out the optimal batch size for a model. So the input to your function should be a list of str instead of just a single string; it should return a list of string as well.
See https://github.com/ucbrise/clipper/issues/513 for more info.


Update VS Code settings

bazel.lsp.enabled is no longer a valid setting. bazel.lsp.args should be first to be alphabetical.

Related, do we even need to mention bazel.lsp.args, since the default is an empty array?

Thanks! I think we can remove bazel.lsp.args as well. Mind also updating the minimum required version in the README from 0.9.0 to 0.10.0?


Add a go mode indicator

Ideally account for various goals as well.

for egg: what is all actually required to defeat the boss? according to the logic, "d9" (i assume egg) boss is locked behind OR(SWORD, BOW, MAGIC_ROD, BOOMERANG, HOOKSHOT):
this, to me, means that:

if we have all items logically needed to open egg, or have all needed items (for instruments-in-item-pool setting, or seashells); OR the requirement is no-egg
have one of those d9 items
can logically reach egg

we can enable a check for "winning" at the top: if you can see that check, it would basically be "go mode". the only goals here i cant wrap my head around are the bingo ones: they would require a bit of work, especially considering that i havent actually played bingo mode yet, and dont know what the goals are.

okay, they seem easy enough to handle case-by-case.. just tedious to set up. wonder if there's logic checks in the rando for finishing goals...

I think the D9 boss here is actually the Color Dungeon boss (Hardhit Beetle) - Color Dungeon is sometimes referred to as D0 and sometimes as D9.
I believe the egg boss's (Dethl) requirement is in the overworld logic: AND(MAGIC_POWDER, SWORD, OR(BOOMERANG, BOW))
But yeah, I think you're right that tying a check to that spot in logic is probably the best bet. That may be a little easier said than done, but it shouldn't be too bad.
Bingo is definitely a lot trickier. The board can in theory be read by the autotracker or imported from a spoiler log (after modifying LADXR to put the board in the spoiler log). The autotracker can also detect which goals are complete, but using logic to tell which goals are currently doable sounds like a daunting task. I'm not familiar enough with how the goals are incorporated into logic (if they even are - the "beatable" setting may not be compatible with bingo) to say how much work it would involve.

there's two ways i could see LADXR handling bingo:

the game is generated as normal, and then some random bingo goals are added without regard to logic
the game is generated with bingo goals in mind

and if i had to guess on it, i'd say the first: occam's razor. it makes this end of things kind of suck, though.

Yeah, the first option seems to be my reading, unfortunately.
I see bingo_goals get set here, then only ever referenced here, which just patches the ROM.

Going to call this good enough for now since apparently LttP just made it a manual button. The egg has a logical icon that accounts for everything but bingo, and there's a manual button in the default dungeon item layout.


reuse port in Bjoern adapter

This affords a considerable saving in terms of system resources.

Looks save to do. Thanks


Is it possible to know when a SO variable value has changed?

Is there any way to do this without adding a ValueChanged event to every script that changes the value?
Big fan of the asset. It's the first thing I import in every project.

You can drag and drop an SO variable into a GameEvent listener.  Or in code you can subscribe to the SO variable with it's AddListener method.


[Feature] How to deploy a RayService with deployment to exist RayCluster ?

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Hi teams, i want to deploy directly a application via rayservice to exist raycluster, what can do it ?
Now exist:

Use case
You can asume i have a new application serving by rayserve, the name is rayservice-app-one .
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

You can connect to the Ray cluster and then use serve run to deploy your serve applications

via common "deployment" to kuberay cluster

For prototyping, you can run serve run. However, for production, you should create a RayService CR with a new RayCluster. To achieve RayService high availability, all Pods must be configured with readiness probes which are different from RayCluster CR.

feel free to reopen the issue if the reply doesn't answer your questions.


New package: Bitly v0.1.0

Registering package: Bitly
Repository: https://github.com/tbeason/Bitly.jl
Created by: @tbeason
Version: v0.1.0
Commit: 9aac8cbfa8aa8fd00ceaea9fcb9b1103ee539318
Reviewed by: @tbeason
Reference: https://github.com/tbeason/Bitly.jl/commit/9aac8cbfa8aa8fd00ceaea9fcb9b1103ee539318#commitcomment-43334153

[noblock] I place the blame on the name similarity more on the existing package than on mine. I'm providing access to the Bitly API, so I think the name is appropriate.


[WIP] Have another go at isolating the sierra_merger tests

Replaces #1723, another part of #1643.

Closing due to inactivity.


Update favicons to have white background color

Details
This fixes the favicons on web by giving them a white background.
Fixed Issues
Fixes https://github.com/Expensify/Expensify.cash/issues/2595
Tests

Visually inspect the favicon in web when you have no unread messages, make sure it's updated.
Visually inspect the favicon in web when you do have unread messages, make sure it's updated.

Tested On
This is web only.

[x] Web
[ ] Mobile Web
[ ] Desktop
[ ] iOS
[ ] Android

Screenshots
Web
Unreads:

No unreads:

Mobile Web
N/A
Desktop
N/A
iOS
N/A
Android
N/A

🚀 Deployed to staging in version: 1.0.32-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Tailwind styles being reset when adding Novel as an npm package

The button is white because Novel's Tailwind is resetting the styles:

I've seen a lot of PRs related to styling but none about this specifically. Would love a quick fix.

Any update on this? Facing same problem with MUI.

https://novel.sh/docs/introduction novel now exports unstyled components, you can style it for your specific use case


[Go] Random segmentation faults when calling Read() on a pqarrow.RecordReader

Hello!
I am currently using github.com/apache/arrow/go/v16/parquet to read the records of a downloaded s3 parquet file (75KB, stored in bytes.Buffer). My implementation is the following:
mem := memory.NewCheckedAllocator(memory.DefaultAllocator)
pf, err := file.NewParquetReader(bytes.NewReader(buf.Bytes()), file.WithReadProps(parquet.NewReaderProperties(mem)))
if err != nil {
     return nil, err
}
defer pf.Close()
reader, err := pqarrow.NewFileReader(pf, pqarrow.ArrowReadProperties{Parallel: true, BatchSize: pf.NumRows()}, mem)
if err != nil {
     return nil, err
}
rr, err := reader.GetRecordReader(ctx, nil, nil)
if err != nil {
     return nil, err
}
defer rr.Release()
rec, err = rr.Read() <---- problem line
if err != nil && err != io.EOF {
     return nil, err
}
if rec == nil {
     return nil, nil
}
defer rec.Release()

... parse the file

I am reading the same file each time and majority of the reads into rec are successful. However, on occasion, I get a segmentation fault inside of rr.Read().  I have confirmed that the file is successfully downloaded each time and that buf.Bytes() is the same on successful and failed reads. I have also confirmed that I can get the schema from the file on successful and failed reads which leads me more to believe something is happening inside the RecordReader.
schema := pf.MetaData().Schema
log.Info(fmt.Sprintf("Schema:%s", schema)) <--- prints out the right schema each time

Here are some logs from the stack trace that I thought could be helpful for debugging.
SIGSEGV: segmentation violation
PC=0x4cb0c8 m=11 sigcode=1 addr=0x7ffbfdf94013e8

goroutine 150888 gp=0x4006db0a80 m=11 mp=0x4000780808 [runnable]:
github.com/apache/arrow/go/v16/parquet/internal/bmi.extractBitsGo(0xffffffffffffffff?, 0xffffffffffffffff?)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/internal/bmi/bmi.go:242 +0xcc fp=0x41bc72bae0 sp=0x41bc72bae0 pc=0x12818ac
github.com/apache/arrow/go/v16/parquet/internal/bmi.ExtractBits(...)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/internal/bmi/bmi.go:38
github.com/apache/arrow/go/v16/parquet/file.defLevelsBatchToBitmap({0x45f221c000?, 0x1?, 0x1?}, 0x400, {0xbc72bbb8?, 0x41?, 0x0?, 0x874c?}, {0x3b0f7d0, 0x41bcba5cc0}, ...)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/file/level_conversion.go:155 +0x180 fp=0x41bc72bb70 sp=0x41bc72bae0 pc=0x12f2ad0
github.com/apache/arrow/go/v16/parquet/file.defLevelsToBitmapInternal({0x45f221c000, 0x400, 0x2c000}, {0x1?, 0x0?, 0x0?, 0x1?}, 0x41bc72bcc0, 0x1)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/file/level_conversion.go:175 +0x198 fp=0x41bc72bc40 sp=0x41bc72bb70 pc=0x12f2d68
github.com/apache/arrow/go/v16/parquet/file.DefLevelsToBitmap(...)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/file/level_conversion.go:186
github.com/apache/arrow/go/v16/parquet/file.(*recordReader).ReadRecordData(0x41bb5c8000, 0x11)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/file/record_reader.go:545 +0x218 fp=0x41bc72bd40 sp=0x41bc72bc40 pc=0x12f8aa8
github.com/apache/arrow/go/v16/parquet/file.(*recordReader).ReadRecords(0x41bb5c8000, 0xce)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/file/record_reader.go:632 +0x294 fp=0x41bc72bde0 sp=0x41bc72bd40 pc=0x12f8e84
github.com/apache/arrow/go/v16/parquet/pqarrow.(*leafReader).LoadBatch(0x41bb5c8060, 0xce)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/column_readers.go:104 +0xd8 fp=0x41bc72be30 sp=0x41bc72bde0 pc=0x1767e48
github.com/apache/arrow/go/v16/parquet/pqarrow.(*listReader).LoadBatch(0x41bc72bee8?, 0x41bc72bf3c?)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/column_readers.go:360 +0x2c fp=0x41bc72be50 sp=0x41bc72be30 pc=0x17690fc
github.com/apache/arrow/go/v16/parquet/pqarrow.(*ColumnReader).NextBatch(0x41b9013190, 0xce)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/file_reader.go:131 +0x34 fp=0x41bc72be70 sp=0x41bc72be50 pc=0x176e9d4
github.com/apache/arrow/go/v16/parquet/pqarrow.(*recordReader).next.func1(0x5, 0x41bc72bf38?)
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/file_reader.go:655 +0x50 fp=0x41bc72bef0 sp=0x41bc72be70 pc=0x17729a0
github.com/apache/arrow/go/v16/parquet/pqarrow.(*recordReader).next.func2()
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/file_reader.go:708 +0x100 fp=0x41bc72bfd0 sp=0x41bc72bef0 pc=0x1772850
runtime.goexit({})
	/root/.gimme/versions/go1.22.5.linux.arm64/src/runtime/asm_arm64.s:1222 +0x4 fp=0x41bc72bfd0 sp=0x41bc72bfd0 pc=0x4df0a4
created by github.com/apache/arrow/go/v16/parquet/pqarrow.(*recordReader).next in goroutine 253
	/go/pkg/mod/github.com/apache/arrow/go/v16@v16.1.0/parquet/pqarrow/file_reader.go:699 +0x2e8
...

It seems that the segmentation fault is happening inside of (*recordReader).next so was curious if anyone familiar with this library had some insight on why this was happening.  I can share a longer stack trace if that would be helpful. I am also using v16 but saw the same error in v13 as well. Thanks in advance!
Component(s)
Go

@reiades I've been skimming through the open Issues here and saw this one. Just wanted to poke you to see if you can get back with any information that might help us reproduce this issue so we can address and fix it.


[VSCode] Add folder to drop custom kamelets

Add folder to the configuration so we can drop custom kamelets. Only for VSCode?

Duplicated for #527

this is not the right number of issue which is duplicated


sched/pthread: Remove unused pthread_sem_xxx functions

Summary
sched/pthread: Remove unused pthread_sem_xxx functions
Impact
Testing

[Experimental Bot, please feedback here]
This PR appears to partially meet the NuttX requirements. Here's a breakdown:
Strengths:

Summary: The summary clearly states the change (removing unused functions) and the affected area (sched/pthread). However, it would be beneficial to add:

Why: Briefly explain the reason for removing these functions (e.g., code cleanup, reducing binary size).
Related issues: If this change addresses an existing issue in NuttX, provide a link to it.

Testing: The PR provides sections for testing logs, which is good practice.

Areas for improvement:

Impact: This section is entirely missing.  You need to analyze and address each point:

New/Changed features: Are any features affected by this removal?
Impact on users, builds, hardware, documentation, security, and compatibility: Since it involves removing code, analyze potential impacts on each of these aspects. Even if the impact is "NO", state it explicitly.
Other considerations: Mention any other relevant aspects.

Testing:

Provide actual testing logs: The sections for testing logs are currently empty. Populate them with relevant logs from your testing before and after the change.
Testing details: Specify the build hosts and targets used for testing. Include operating systems, architectures, boards, and configurations.

Recommendation:
Before submitting, complete the "Impact" section and provide detailed information in the "Testing" section.


VA Profile Sync: Implement SSL Auth

Need @k-macmillan to add detail

Please add your planning poker estimate with ZenHub @ianperera

Invoking the lambda from the AWS Lambda console works.  I have the public pem certification used for unit testing in SSM as placeholder data, but we still need a production public certificate from the Profile team.
Unit tests pass for the POST functionality with JWTs.  The PUT functionality will be unit tested as part of #704 and perhaps also as part of this ticket.  I need to modify the lambda code to use the correct certificates for mTLS.


Consider submitting this code to Apache Jena

Hi
I'm one of the committers on the Apache Jena project and read your paper with much interest and have reviewed your code.
We would love to get your work integrated into Apache Jena so that it is available to our end users.  Please could you consider putting together a pull request that would add the relevant parts to ARQ so that they can be used by the wider Jena community.
Thanks,
Rob

+1

Dear Rob,
Thank you very much for the suggestion.
We currently just don't have the time for the integration. Luckily, I am going to have some free time during June, so I think we can start integration at beginning of June. We also have some possible improvements in our minds, and during integration we may get some guidance also. Please let me know if it is okay to integrate it at June.
Thanks again for the interest.  We are very delighted to be noticed and for the suggestion.
Best,
Tahir

We don't have any specific time frame in mind we'd just love to see this in Apache Jena at some point in the future.
Please reach out on the dev mailing list as and when you need assistance/guidance with the integration
Best Regards,
Rob


Feature/run in local dev

Steps from the beginning for running hyrax from the hyrax directory

All good :+1:


.14.2 compatibility

The output of /version on my server is:
>version
[18:39:37] [Server thread/INFO]: This server is running CraftBukkit version git-Spigot-093165d-ba575a5 (MC: 1.14.2) (Implementing API version 1.14.2-R0.1-SNAPSHOT)
[18:39:37] [Server thread/INFO]: You are running the latest version

The output of /version denizen on my server is:
>version Denizen
[18:39:42] [Server thread/INFO]: Denizen version 1.0.6-SNAPSHOT (build 827-DEV)
[18:39:42] [Server thread/INFO]: Authors: mcmonkey, Morphan1 and aufdemrand

The issue is:
[18:31:26] [Server thread/WARN]: [Denizen] This build of Denizen was built for a different Spigot revision! This may potentially cause issues. If you are experiencing trouble, update Denizen and Spigot both to latest builds! If this message appears with both Denizen and Spigot fully up-to-date, contact the Denizen team (via GitHub, Spigot, or Discord) to request an update be built.
[18:31:26] [Server thread/WARN]: [Denizen] -------------------------------------

Updated yesterday to Spigot 1.14.2
Spigot then helpfully changed the NMS mapping ID in the very next commit. gg
Fixed, thanks!


Initial generation capacities 2030

Initial generation capacities (e.g., wind) too small for some countries.

Silke updated existing capacities to those of Entso-E’s ERAA 2022 National Estimates for 2030 (27 January 2023)

Added some load shedding units per country for industry load shedding (called DSR).

Fossil power plants deactivated in base-roe

All thermal units are deactivated in High_VRE alternative

Decided to keep the 2019 hydro capacity data, as new hydro capacities in ENTSO-E data should also influence inflows

A few notes:

Hydro: PHS not in DK/FI/NL, ROR not in DK/FI/NL, reservoir not in BT/DK/NL.
Offshore wind: Not in AT/CH/CZ.
Initial batteries: Not in DK/NO/PL/PT
Gas: Not in CH/NO

Updated:

PHS: CZ/DE/FI/FR/GB capacity, conversion factor, unit size, vomCosts data
ROR: NO cost data
Batteries: DE costs
Nuclear: some updates

ToDo:
Need to check all existing capacity data. Some mismatches at least in Nuclear, Hard coal, Lignite, Gas, Oil, Others non-renewable, Biofuel, Batteries.
In D2.1, Romania capacities missing from BK and SE3 capacities missing from SE and DE wind/solar/others capacities also not the same as in ENTSO-E data. Wind and solar potentials take into account initial capacities so remember also to check/update maxUnitCounts.

Moved load shedding units to dsr alternative. Created bio-out and new-waste-out units to deactivate initial and new biofuel units and new waste units. High_VRE alternative does not anymore deactivate all thermal units.

Updated all capacities based on Entso-E’s ERAA 2022 National Estimates for 2030, including hydropower. Old hydropower data had very high capacities e.g. for CZ and FI. Need to check inflows - and for example BT hydro discharge unit.

Updated also initial battery storage capacities

I enabled hydro reservoir and PHS spillage (max 5% of reservoir turbine and PHS OL turbine capacity).

Added thermal_reduced alternative to be used in S3 and S4. Biofuel, waste and nuclear capacities reduced by 50%.

NL wind and solar capacities updated

added alternative "ERAA2022-germany" in p_gnu_io with reduced DE wind onshore, offshore and solar PV capacities that corresponds to ERAA2022. It is much lower than the current more ambitious expansion targets. yet, in order to simulate investment it might be desirable not to have such high capacities.


Tagging releases on github

Hi,
For distro package maintainers it's nice to be able to quickly find the version of the source that we package from hackage. It's useful for applying bugfixes and security patches if needed. Doing a git blame on the version string doesn't tell us if the next commit is also part of the release without manually checking the changes.
Thank you,
David

Thanks for the heads up. I just added a tag for the new version (0.8.0). I'll try to be better about this going forward.


Can annotorious used in OpenSeaDragon like the old version?

Hello, It's nice to find out that someone have rebooted the annotorious. I have used the old version of annotorious in my project for the Openseadragon Viewer, so I'm wondering can this new version use in the OpenSeaDragon like the old one?

Eventually :-) Everything's still a bit under construction. But the basics work. If you can let me know which particular features of the old version you need, I can answer more specifically.
There is one major difference in terms of how the project is set up, however. The version in this repository works only for images. The OpenSeadragon plugin now has a separate repository (and is a separate JavaScript file):
https://github.com/recogito/annotorious-openseadragon

Thanks for your fast reply! Unfortunately, my project is using the DZI on OpenSeaDragon not the image type. I am looking for a way to add a label or button beside the annotation when create the annotation just like the image below. Do you have any advise about dealing with this? Anyway thanks for your help!

As I said: you need to check out https://github.com/recogito/annotorious-openseadragon for the  version!
You might be able to hack that labels into the annotationlayer yourself. If they have to show all the time, this might be tricky. If only mouseover is needed, that would be easy to do by using the onEnterAnnotation and onLeaveAnnotation events.

P.S. I looked into the code quickly. This is where the OpenSeadragon plugin draws the shape:
https://github.com/recogito/annotorious-openseadragon/blob/master/src/annotations/OSDAnnotationLayer.js#L81
drawRect is an imported function from the normal Annotorious package (see here). You‘d need to replace that with your own SVG drawing code. Then you‘d have all the freedom to add extra SVG shapes, add labels based on Annotation contents etc.
The only downside at the moment is that you‘ll be creating a separate fork of Annotorious. It‘s not possible to do this without hacking it, I‘m afraid. But probably an ok solution nonetheless for now.


Duplicate edit actions in settings

Refs: #31448

Hover over the edit action in the settings margin
Notice double hover

Also sometimes there are multiple edit actions in the margin, even though there should always be exactly one.

Also when executing those actions I am offered multiple choices so it seems that actions are lying on top of each other and they all get executed upon click.

@isidorn May I know how do you get into this state?

I could reproduce it.

Ok great


bash script

Hi, I have a question about your script.
https://github.com/ghacksuserjs/ghacks-user.js/blob/master/updater.sh
There are 2 mentions of two packages, wget and curl. These are packages that load something (in our case, user.js).
In this script, they are selected to choose from, because there is an "if" condition. The first package is "curl", but if it is not available (not installed or broken?), Then "wget" comes into play. However, what if there are none. ALL. NOT What then? second "if" is not there?
this is not about debian distributions (for which they are available by default), but about (say) archlinux.
On it, as far as I remember, you need to manually install it (although I could be wrong, I did not use it for a long time).
and so it is possible to supplement the conditions on the verification (presence or absence) of these packages? Is there a guru js here?
I am not a guru in js, and maybe I’ll repeat myself I don’t understand. Explain to me?
I hope I wrote without errors :cry:

Are you using Arch, right? It's in Extra Repo: https://www.archlinux.org/packages/?name=wget

now there is no (win10), But I used it. and I don’t remember that he installed these packages by default

Sound like a good idea to me.
It still might be a good plan to add a check for simply aborting when even perl is missing, though. There are a gazillion distros out there and more will spawn on time, we can't possibly know for sure whether they all have/will have one of those or not.

Good point - do you think we should keep the wget and curl options, or just switch to perl entirely?

At just 2 lines per additional check I'd just keep them all :) Then again, I'm not too finicky when it comes to etiquette and saving bytes and such... :sweat_smile:

Sounds good! It looks like @claustromaniac already added a check in #540, so we're good there.
If curl is that widely used, the pearl option probably doesn't add a whole lot, but I can throw it in!

Feel free to add a perl fallback but I think every system with Firefox installed will almost certainly also have curl or wget. At least so far nobody reported a problem and OP only guessed about Arch not having it.

@earthlng

and OP only guessed about Arch not having it

I am (practically) sure. 60 percent.
in base and base-devel they are not.
(I don’t know if you are using Linux, but) archlinux is installed along this guide https://wiki.archlinux.org/index.php/Installation_guide.

@bogachenko pacman is a base package and curl is a dependency of pacman

new version of the updater.sh is in the works so I'm closing this


Sign out issue

Upon signout, it throws the user to the invenio server that returns a 'Not found' page.
Instead it should return gracefully back to the Login page (Home page is acceptable, but not ideal because some libraries may prefer to not have open search access, but only within their communities)
Also plz ensure browser Back or Refresh, after logout, does not allow access to pages. The session management seems to work in general, but once or twice I encountered an error. In any case, needs to be tested (could get tricky if multiple tabs or windows are open).

Hi, it would be great if you more details about this.
Can you please describe how to reproduce it?

Hi, it would be great if you more details about this.
Can you please describe how to reproduce it?

Hi,
When you "Sign out" either from the frontsite or backoffice, the screen I get is attached below. Obviously, doesn't make sense to be taken to invenio server 404 page and if you login here, it goes to the server admin page - instead should be taken back to the ILS login page upon logout. Hope it's clear.

Hi,
When you "Sign out" either from the frontsite or backoffice, the screen I get is attached below. Obviously, doesn't make sense to be taken to invenio server 404 page and if you login here, it goes to the server admin page - instead should be taken back to the ILS login page upon logout. Hope it's clear.

#https://github.com/inveniosoftware/invenio-app-ils/pull/1159


Errors while buiding environment

While installing graspAPI with pip install graspnetAPI,error occured as:
ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
lang-sam 0.1.0 requires numpy<2.0.0,>=1.24.2, but you have numpy 1.20.3 which is incompatible.
supervision 0.18.0 requires numpy>=1.21.2, but you have numpy 1.20.3 which is incompatible.
How can I solve this problem?Thanks.

That error can be ignored as long as the numpy version is 1.20 and graspnetAPI is installed. You can proceed to the installation verification step and let us know if you are facing any issues.

That error can be ignored as long as the numpy version is 1.20 and graspnetAPI is installed. You can proceed to the installation verification step and let us know if you are facing any issues.

Now I'm using numpy 1.20.3, however, ImportError: cannot import name '__all__' from 'numpy.linalg' (/home/sun/miniconda3/envs/grasp-env/lib/python3.9/site-packages/numpy/linalg/__init__.py)

Could you try installing numpy==1.23.0 and see if this works?

Could you try installing numpy==1.23.0 and see if this works?

Thanks, I have solved the problem in this way


vue-tsc: Add --onSuccess argument that runs command after every successful compilation triggered by --watch

Similarly to  tsc-watch, support an --onSuccess COMMAND argument that when paired with --watch runs a command on every successful compilation.
Useful when developing local packages with yalc and need to run yalc push on every change.

tsc-watch already support pass tsc path, you can try tsc-watch --compiler vue-tsc/bin/vue-tsc.js.


Merge release-1.1 into master branch

Do not automerge, or it will be squashed

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: howardjohn
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
29858e51b278b3938c94733d62e9ce3b652d078d
link
/test istio-unit-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
ecffca727615c1c757e0cf7e37c1366da9c89165
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-integ-k8s-tests

@howardjohn: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
ecffca727615c1c757e0cf7e37c1366da9c89165
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@duderino @rshriram can you take a look when you get a chance?

@costinm @linsun can you take a look

Closing for https://github.com/istio/istio/pull/13996


Concurrency with multiple CR

Feature Request
I am currently working on an operator that now involves over 30 different custom resources.
Even when passing a --max-worker=<n> argument, I would still get at least n workers per CR.
Sometimes, the k8s module would just timeout - which I suspect has to do with lots of simultaneous queries, as it mostly shows when restarting the container, and 30+ playbooks starting together.
While the existing max-worker flag is pretty nice already ( or max-concurrent-reconciles, as of operator-framework/operator-sdk/issues/3360 ?), it might be interesting to have something similar, that would limit the overall amount of plays that can run simultaneously.

That issue is still there.
I've been updating my operator base image to 1.1.0: no change.
Restarting/starting an operator, while 10s of CR exist, all playbooks would start together, k8s modules would randomly timeout, none of your playbooks would complete, ... until such time playbooks run are spaced enough to get enough CPU.
Meanwhile, as of using 1.1.0 (ansible 2.10.2), most my playbooks would fail anyway, with some "maximum recursion depth exceeded while calling a Python object" error.
It would be nice for those operator-sdk images to be reliable, at some point.

Running 1.5.0, I still have tens of playbooks starting at once. CPU load reaching over 40 when the operator container restarts.
On a node that has 4 CPUs, and is now dedicated to running my operator (otherwise, even an nginx ingress controller ends up in crashloopbackoff, pretty much any liveness/readiness probe would randomly fail, usually due to timeouts).
Is there any rational to close this case?
Should I open another one, reporting that exact issue?
What can be done, enforcing proper concurrency?


tsconfig.json lib option doesn't work on windows

When I provide a "lib" field in tsconfig.json, it works as long as the OS uses slashes (/) as separator for paths. Typescript uses the LanguageServiceHost.getDefaultLibFileName as a way to get the location of the lib directory. On Windows the path will contain backslashes () . Typescript uses slashes internally and when it tries to get the directory from a path, it uses slash to find the last path component and failing to return the correct path.
https://github.com/jrieken/gulp-tsb/blob/114c0da84301e0a597ac81a4e4c20c6cb4333be0/src/builder.ts#L561
Typescript uses the getDefaultLibFileName as a backup, if getDefaultLibLocation is not available on the LanguageServiceHost. However, when Typescript gets the LanguageServiceHost from gulp-tsb, it wraps the object in a sort of proxy, which is missing the getDefaultLibLocation forward.
So a solution would be either:

Modify the getDefaultLibFileName to always return a path with slashes.
Implement the forwarding of getDefaultLibLocation in Typescript and implement the getDefaultLibLocation in gulp-tsb as well, using (or not) platform dependent separators.

I think number 1 should be the easiest and would also follow Typescript's way of using slashes on all platforms.

Fixed in 2.0.3 (see #53)


Restart workers for queues that change

https://ocd.appscale.com:8080/job/Daily Build/1768/

https://ocd.appscale.com:8080/job/Daily Build/1769/


[Feature]: JSfunction to select table row (&fill Table.selectedRow.{})

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Goal: navigate to a CRUD page with a selected item (by default)
Why should this be worked on?
Example use case: a page that lists todo's also lists the owner (IDx) of the todo (IDy). In case of clicking on the 'edit owner' button we want to redirect the user to the crudUser page with the table row corresponding to user with IDx selected. This allows him to directly change that user.
Quick and dirty workaround

add a 'defaultFilter' param to the navigateTo request
in the target page set the defaultSearchFilter to appsmith.URL.queryParams.defaultFilter

Drawbacks of this workaround:
A). undesired from user experience POV as the user might not be aware that a filter is applied when he wants to show different users/ table rows as well
B). the table search in general (without changing query), for a text field. Theses are non-unique. This for this function to work properly the text field/ query shall be adopted every time such that the primary key can also be search and ensure only one unique results

@unknown1337 setting the selected row won't help here because if the data set is server-side paginated, you won't be able to set the selected row since the row may not be present in the first load. The best way to achieve this is:
pass the id of the user you want to select in the appsmith store using storeValue and in the query check if the id has been set and query rows that are -5 and + 5 ids from the id that is set. Then onPageChange, you can clear the ID that is set. If you want the id to even be selected, you can find the index of the id in the table and set it in the default selected row property. Do let me know if this doesn't solve the problem for you.


Check custom deps e2e

What does this PR do?

Fixes #<issue_number>
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

@akihironitta mind also check this one? seems similar to #15429

Closing this PR as the reason for the hang was because of the very outdated docker image used in CI as I confimed with Mansy last week in #15417.


Can't install chromedp

What versions are you running?
latest
root@Proxies:~# go get -u github.com/chromedp/chromedp
unrecognized import path "embed": import path does not begin with hostname

$ go version
<!--replace this line with the output-->
root@Proxies:~# go version
go version go1.14 linux/amd64

#### What did you do? Include clear steps.

<!--Put your source code in fenced code block like this:-->

```go
put your source code here

What did you expect to see?
What did you see instead?

go1.14 is too old and not supported any more. See  https://github.com/chromedp/chromedp/releases/tag/v0.8.2.


Refactor to reduce the number of lines in bgp test module

          Let's open an issue to tackle this

Originally posted by @gmuloc in https://github.com/aristanetworks/anta/pull/813#discussion_r1795090252

Completed per #888 and #914


[FEATURE] Make the background blurry when opening sidebar (mobile view)

Is your feature request related to a problem? Please describe.
Making the background blurry whenever the sidebar opens.
Describe the solution you'd like
Currently when sidebar opens, the background is not blurred which is not providing a good user experience. Making the background blur whenever sidebar opens will help users to just focus on the sidebar content nowhere else.

Hii @anand346
I am willing to work on this.can I get Assigned?

Ok @anand346, I had reviewed the code and understand your problem, after opening sidebar its distracting and not feeling good.
Assign me this issue under Hacktoberfest 2023, I will try to resolve it far better way.

Hi @bhalerao-2002 , I'm assigning this issue to you. Kindly raise a PR when you are done 👍🏻

Hi @bhalerao-2002 , is there any update on this issue?

Yes, I am trying to do better work, I will raise PR after 1 Oct, as with respect to, it will be counted for Hacktoberfest

Thank you @anand346 and That not problem let @bhalerao-2002  can work Its my permillage to contribute in OS.

Thanks @Basavarajakjbasuu, I really appreciate your efforts. Please feel free to work on any other issue, thanks.

is this open?

Hello i can help by resolving this issue can you assign it to me @anand346

Fixed in #35


Folding links into text, CC0 addition

It's much nicer if URLS are folded as links into the text.
Added a CC0 paragraph under the "...open science...what does that mean?".
Some other hopefully minor wording changes.

Thanks Abram. I agree about the links. I just didn't know how to do it. I
can make that change. I'm still pretty clueless on GitHub.
Thanks for the wording and the CCO para! That's fantastic. I haven't had a
chance to look yet, but I'm assuming this was done in a branch that I would
have to merge?
On Wed, Nov 11, 2015 at 8:53 AM, Abram Connelly<EMAIL_ADDRESS>wrote:

It's much nicer if URLS are folded as links into the text.
Added a CC0 paragraph under the "...open science...what does that mean?".
Some other hopefully minor wording changes.
You can view, comment on, or merge this pull request online at:
https://github.com/PGPHarvard/FAQ/pull/6
Commit Summary

minor wording changes. Folding links into text
Updating Coriell link

File Changes

M README.md https://github.com/PGPHarvard/FAQ/pull/6/files#diff-0
(24)

Patch Links:

https://github.com/PGPHarvard/FAQ/pull/6.patch
https://github.com/PGPHarvard/FAQ/pull/6.diff

—
Reply to this email directly or view it on GitHub
https://github.com/PGPHarvard/FAQ/pull/6.


Create Commits page

The commits page must have a list, where each row will have:

plugin name
github commit link
potentially other info if there's no limit amount of API calls I can make


iPad does not work

Does not work on iPad
screenfull returns false

The fullscreen on the iPad is completely different from what's used here. Once upon a time Apple invented their own way to do fullscreening for videos, and only videos. That's what the iPad uses. I'm not adding support for it as it's legacy, properitary and only supports fullscreening videos.


Important options gone

Greetings. Sorry my basic English.
When the setting "Parse Gamelists only" is used, RetroArch disappears from main screen, and Option menu options mostly disappear, only remaining Advanced + Bluetooth + Update + Wifi.
The gamelist.xmls I'm using were all created with ScreenScraper Windows version.

Don't use "Parse Gamelists only".  It doesn't save much in initial loading time anyway.

First of all, thank you for your assistance.
I’m only alerting for the consequences of losing all those settings.

An alert should be issued,
It should not happen (?) since gamelists are for games (as the name implies) and it took me hours to figure out why the settings were missing (I reinstalled ArkOS from scratch 4x)
I’m not alone. Found other people online complaining about the same missing settings with no answer.

Thanks and keep up the good work!
—Laurentino Martins

No dia 20/04/2023, às 00:12, christianhaitian @.***> escreveu:

Don't use "Parse Gamelists only". It doesn't save much in initial loading time anyway.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.

Thank you for your help and your excellent work. I've donated for the project.
Related to the above question, in Amiga, I've created .m3u files for all multi-disk games, since PUAE uses the .m3u to auto-install the disks in multiple drives. It works fine.
Now the annoyance: The game launcher (EmulationStation?) lists these games twice.
Note: gameslist.xml only lists the .zip file.
Without the "Parse Gamelists only" problematic setting, is there a way to ignore the .m3u files?
Thanks in advance

You can manually edit the es_systems.cfg file located in /etc/emulationstation.  Just remove the extensions for Amiga you do not wish to show.  Make sure you use a unix/linux compatible emulator.  Not the default notepad or wordpad editor in Windows since that will add additional characters that linux/unix don't like.

Sorry for the delays in the answer, I've been quite busy.
Finally I installed Ubuntu in my Parallels Mac mini and managed to edit the file you mentioned.
I removed both the .m3u and .M3U references in Amiga and the duplications disappeared.
Thank you very much!

Awesome.  You could've also just used a ssh terminal program to remote into ArkOS while powered on to make that change as well.

(Do you have a personal email address I could use?)
You may want to know you are distributing ArkOS with a pre-configured item.
Last time I installed ArkOS from scratch I found a WIFI Network or a Bluetooth device pre-configred.
I'm not sure which, it was some time ago.
You may want to remove this in the next release.
Thanks, and keep up the good work.
Sorry my basic English.

Thanks for reporting this.  I'm sure it wasn't wifi.  The one preconfigured item in the wifi menu is for ethernet based connections which is needed if you ever plan to connected a hardwired ethernet dongle.  I'll double check that bluetooth doesn't have a preconfigured device in there.

I have an RG353M with integrated WiFi, so this is why I never understood what that WiFi connection was for in the WiFi menu.
Thanks
Another question, regarding the access with SSH, on the FAQ you talk about "Go to the Options menu and select NETWORK INFO so you can see your the assigned IP to your device.".
I must be blind. I can't find this "NETWORK INFO" menu?

I need to update the faq.  I removed that as you can just hit start and scroll to the bottom of the screen to see your ip info.

Ah, that explains it :)
Sorry, I hope I'm not abusing your assistance, but I have one final question:
Do you have any ideia why the port Cannonball (Out Run) doesn't save or can't load (?) the user settings? I read somewhere that might have to do with permissions.
Thanks

What format is your roms2 card in?

ExFAT
Do you know anyone that is able to save the settings?
I've searched all around the partitions and failed to find a configuration file (although I wasn't sure what I was looking for)

We'll have to test and see what's going on with this.


Provide documentation about what to insert in the schema and about how to read the uploaded files in the code

I use graphql-go as graphql server but there is no documentation about what i should add to my schema file nor what i should add to a resolver to get the file bytes.

@mjarkk Thanks! I will add some documentation about it soon.

@mjarkk look inside example.go source

Thank you!


New Type RGB and PPM

Hi ElectronicCats Team
Thank you for your awesome CayenneLPP library. For a project we needed some additional value Types, and maybe you'll be interested in this changes.
New Values Type that we added:
uint8_t CayenneLPP::addPPM(uint8_t channel, uint32_t value); // 1 PPM unsigned - PPM means Parts per million 1PPM = 1 * 10 ^-6 = 0.000 001
uint8_t CayenneLPP::addRGB(uint8_t channel, uint8_t r, uint8_t g, uint8_t b); // R: 255 G: 255 B: 255
addPPM(0, 401);  = 00 8F 01 91
{
"ppm_0": 401
}
and
addRGB(0, 1, 145, 255); = 00 90 01 91 FF
{
"rgb_0": {
"b": 255,
"g": 145,
"r": 1
}
}

eh yes the build check fails but it shouldn't come from my changes, because I didn't change anything in the Basic_library_TTN/Basic_library_TTN.ino Example - the errors on this are mostly because of this definition, placeholders etc:
#define loraSerial Serial1 -- some Boards don't know this Serial1 port and should be "defined"
#define freqPlan REPLACE_ME // 'REPLACE_ME' was not declared in this scope
TheThingsNetwork ttn(loraSerial, debugSerial, freqPlan); //in expansion of macro 'freqPlan' - because of the error above

Ppm is urn:oma:lwm2m:ext:3325	3325	3325	Concentration
And rgb is urn:oma:lwm2m:ext:3335	3335	3335	Colour should i rename it to addColour and addConcentration?

Is ok for me, only
3325  change please name to "addConcentration"
3335  change please name to "addColour"

Perfect I'll do that tomorrow

Commits:

Changed the Implementation according to open mobile alliance
Fixed Typo in Concentration

Type                      IPSO    LPP     Hex     Data Size
Concentration       3325    125     7D      2
Colour                   3335    135     87      3

Code tested with with the decode / decodeTTN sketch, and with our sensors and sending it to the TTN Backend.


Refactor map component

First few thoughts trying out a refactored more 'general' component for map
Changed from vanilla js mapbox library to react-map-gl pacakge. I've found it is WAY more react-typescript friendly and felt very little friction with the type-system
This is pretty rough and tbh the more I played with ReactMapGL I questioned if even it's necessary to have a generic in-app Map component and just use ReactMapGL component directly when you need to - their API and documentation is pretty comprehensive and they give lots of examples that offers a lot of great patterns
https://visgl.github.io/react-map-gl/examples/geojson
https://visgl.github.io/react-map-gl/docs
Also used some things from Brad's PR https://github.com/codeforboston/mattapan-mapping/blob/b909bec69422fbd94183af1a95c88824305d8511/src/components/Map/Map.tsx v useful
This PR is in draft for now, I'll spend a couple more shower thoughts and see if I can polish this further. The 'Explore' page is playground for now to see how an instance of Map component could play out (still using not-curated-data)

Also, I deleted package.json in this changeset incase things conflate with yarn packages (I think I had it from when originally I set up CRA)

Thanks for choosing our opinion(s) on the Mapbox library setup, @laudickson!
Not sure why the deploy preview situation is so overloaded but this is the one that built I think, though the mapbox token might not be right still https://deploy-preview-69--mattapan-mapping.netlify.app/#/
Do you happen to have 1-3 netlify's going in your account there?

I'll go over and share more in-depth with group tomorrow, but I'm somewhat settling on this pattern for now -

All Map-like components in MapAtoms; I don't think having Map in organism makes sense anymore
Whenever you want to make a map with a geojson data-type (or multiple geojsons), the geojson you want to import has to have a corresponding 'layer' with it; basically a json that describes how the geojson will be styled (follows LayerProp as described by react-map-gl`

Ex. pattern loosely looks like
import geoJsonData1 from './geoJsonData1';
import geoJsonData2 from './geoJsonData2';

const layer1 = { id: 'foo', type: 'line', paint: { 'line-color': 'orange' } };
const layer2 = { id: 'bar', type: 'fill', paint: {fill-color': 'purple' } };

      <Map mapStyle={ Theme.map.dark }>
        <>
          <MapSource data={ geoJsonData1 } layer={ layer1 } />
          <MapSource data={ geoJsonData2 } layer={ layer2} />
        </>
      </Map>

Concerns/issues:

Type casting geojson data before able to actually put into as prop value (omitted in example)
necessary fragment brackets because React children only expects 1 child (how to get around?)
questioning the whole pattern itself / appropriate in MapAtoms?

Thanks for choosing our opinion(s) on the Mapbox library setup, @laudickson!
Not sure why the deploy preview situation is so overloaded but this is the one that built I think, though the mapbox token might not be right still https://deploy-preview-69--mattapan-mapping.netlify.app/#/ Do you happen to have 1-3 netlify's going in your account there?
Final build was most successful I guess: https://mattapan-mapping.netlify.app/explore

@thadk good look there, it has been working locally for me pretty much all this time but turns out it was related to this https://github.com/visgl/react-map-gl/issues/1266 and this version of mapbox isn't transpiring correctly. Lots of suggested solutions in that thread, but this one seemed to work as far as from looking at the deploy preview (craco ftw once again!)

https://github.com/mapbox/mapbox-gl-js/issues/10173#issuecomment-793462984
https://deploy-preview-69--mattapan-mapping.netlify.app/explore

and move assets and boundaries into public folder and use from there


Support Chrome's UNORDERED_NODE_ITERATOR_TYPE

Hi Martijn, just getting back on the wagon after a long time away from Enketo.  Here's a fix for a weird bug (either in enketo, or Chrome ;¬) which I'm probably the only person in the world to suffer from!  Please let me know if it's suitable for the enketo mainline, and if not if there is anything I can do to improve it.

When chrome's XPath evaluate() method is asked for an ORDERED_NODE_SNAPSHOT_TYPE
it can return an empty UNORDERED_NODE_ITERATOR_TYPE.  This throws an error when
snapshotLength is called on
TypeError: Failed to read the 'snapshotLength' property from 'XPathResult': The result type is not a snapshot.

On the other hand, there are other types which will throw a similar error if
iterateNext() is called on them.

Welcome back! 🌟
Did you mean to add a test with that XForm?

Ahem * no!  Didn't mean to add the form at all (¬;

Ok, will try to reproduce by creating a test that fails with grunt karma:browsers. So just a 'nodes' result that is empty, I presume?

Ok, will try to reproduce by creating a test that fails with grunt karma:browsers.

I'm not sure that will work - the problem shouldn't appear when using XPathJS because the native evaluator won't be called: https://github.com/medic/enketo-core/commit/d889014f214fbde968b4229009d09ffe95e2ed14#diff-3647f2f20ebf195842b4f1664089ed79
Because of that commit, I modified our openrosa-xpath-evaluator to delegate requests for *_NODE_*_TYPE to the browser's evaluator, which caused the bug fixed above.  I hope this delegation is reasonable, at least barring bugs in Edge.
Hence my initial comment that I'm probably the only person in the world to suffer from this bug.  Apologies for failing to provide more context earlier.

I think I should look into this a little further though - on re-reading my code there's a chance I've caused this by changing the requested resultType for the evaluate() call from ORDERED_NODE_SNAPSHOT_TYPE to ANY_TYPE.  I'll re-open if I can't fix at this end!

Ah thanks!
If you find it's an issue, we should definitely be able to write a test that forcefully evaluates with the native XPath evaluator. Enketo also sends native XPath expressions as much as possible to the native evaluator (in a much less intelligent way, if the parameter tryNative is true and no custom function calls are detected).


Enable GET requests on RPCs

Hi,
I have this case where I need GET requests on rpcs rather than POST.
I would like to call: http://localhost:3000/rpc/myfunc?param1=value1&param2=value2
any plans to implement GET requests on rpcs?

It was discussed in https://github.com/begriffs/postgrest/issues/328#issuecomment-162029728 for stored procs which are marked as immutable. I haven't looked at that issue for some time, so I don't recall the consensus, but have a look at it and see what you think about the proposals there.

I've commented on the issue. I'm not a haskell developer. so I can't really contribute to this project by code. so the only thing I can do is giving my opinion as a user/dba from that perspective.

what are your thoughts? I would love to hear from you that you will implement this. I really have a use case and am looking forward seeing this feature in the next release/and or master.

See my proposal on #328 if we all agree on the interface I could implement this feature.

duplicate of #328


Literal from macro expansion is not accepted in typescript_custom_section since rust 1.17.0

Describe the Bug
Since the update to rust 1.17, typescript_custom_section doesn't work with literal captures expanded by macro.
In previous versions of rust, everything worked as expected.
The reported error is
Expected a string literal to be used with #[wasm_bindgen(typescript_custom_section)]

Reproduction
macro_rules! generate_ts {
    ($lit:literal) => {
        #[wasm_bindgen(typescript_custom_section)]
        const _: &str = $lit;
    };
}

generate_ts!("TEST");

The snippet above represent the code that used to work, but doesn't anymore. I've printed the actual AST that syn::ItemConst::macro_parse received, and got this:
Group(
  	ExprGroup {
		attrs: [],
		group_token: Group,
		expr: Lit(
			ExprLit {
				attrs: [],
				lit: Str(LitStr { token: "TEST" })
			}
		)
	}
)

Which means, now rust expands the literal (and possibly other) captures as an expression group with one item, and this should probably be handled in various places in the parser.
Current workaround
I managed to fix it in my specific case by using tt capture instead of literal, but this is obviously more general than i would like.

Hm can you confirm what version of Rust you're using? 1.17 is quite old, and I'd recommend using something more in the 1.40 range

@alexcrichton oh sorry, that was totally a typo on my side, the version is 1.48.0, I have confirmed it not working since 1.47.0, and working on 1.46.0.

Aha makes sense! I'll try to investigate this this week.


Land scape mode added successfully.

Send me your email or discord channel so that i can share you the detailed video.

You ahve to share it here.
just drag and drop video in comment box it will be uploaded.

Video size is getting out of limit... It is unable to load a 30 sec video..
That's why I was unable to share the video here...
Here are some snaps of my work.

Sir i made few changes which were necessary.
These are the changes I made :-
1 change in Mainfest file of orientation mode into unspecified
2 change in leaderboard fragment , connecting the leaderboard frag with an activity to avoid app crash
3 added scroll view in signup page and in chats fragment plus button
4 change in Mainfest file main activity to resume activity on state change.
These are the changes I made.
I can breify explain you in detail if you want.

Sir tell me any update what change I need to do or everything fine.

LGTM @Yash-Parsana

https://drive.google.com/file/d/1U1kRvbHQ250fHE-uTQL55BLazBY6OAqD/view?usp=drivesdk
Video link

https://drive.google.com/file/d/1U1kRvbHQ250fHE-uTQL55BLazBY6OAqD/view?usp=drivesdk
Video link

LGTM


[helm3]  helm3 ls -o json ,return without "next" or "total" field.

helm3 ls -o json  return has no field "next" or "total"
but helm2 ls --output json has.

Helm 2 relied on the gRPC API, so the results were paginated. In Helm 3, the results come directly from the Kubernetes API and require no pagination.
related: #6801

Closing as answered.


Don't cross feature

Is it possible to prevent slider bullets crossing each other?

any updates on this one?

yes

I plan to add the crossable_handles option in the next version, but for now it would be cool if you can check the development version and let me know if it works for you. The option is documented in README.MD as can be seen on https://github.com/lokku/jquery-nstslider. You can clone the repository and use /dist/jquery.nstSlider.js or /dist/jquery.nstSlider.min.js directly as they've been updated.
Reference commit

Thank you. That is what I was looking for how ever there is bug in my app right now which I'm not sure it's because of that last commit or not. When I drag the right handles over the other one they don't cross witch is pretty much expected, and when I drag the right handle back I can't move the left handle to left any more.
Do you have the same issue? I don't know if I could make it clear what the problem is or not.

Working fine in the demo page under demo/ I have just added crossable_handles = false to the initialization. Hmm, can you give me more information?

what browser are you using?
how do you initialize the slider?

Maybe consider creating a jsfiddle that illustrates the case, you could fork and update this one with your code to get it done quickly.

Yea you are right demo works perfect but It's wired and it seems like setting direction: rtl; on body and crossable_handles: false" on your config object will cause left handle to only be able to move to right!
I'm testing under chrome.

Good point! fixing!

Now should work

Thanks for your grate support dude!
We have a project on dead line and you just saved our asses on that.


BRIDGE-1068 Change the medication tracking to include two step question about timing

Modifications to the hardcoded logic to allow for two questions about medication timing.

Looks good


Doesn't start after shutting down/restarting (windows)

🧐 Have you read the FAQ and viewed similar issues?

[ ] I have read the FAQ
[X] I have checked older issues, open and closed
[ ] I did a clean install of SpotX as described in the FAQ

Сountry of your account
India
What is your Spotify Plan?
Free
Do you have Spicetify installed?
No
ℹ Computer information
- Spotify: <IP_ADDRESS>9.ga3ae4...(spotify doesn't open can't check version from inside used winget to see)
- Windows: Windows 10 22H2 19045.3086
- PowerShell: 7.4.0-preview.3

📝 Description
1.Install spotx - didn't disable updates, tried both enabling and disabling "podcasts, episodes and audiobooks" either way "Didn't find variable patches.json.others.podcast_ad_block in Spotify.exe" error was raised
2.Works fine
3. shut down/ reboot
4. Spotify does not start
works fine after reinstalling spotx
📸 Screenshots
No response

read FAQ

I can't, the site doesn't load with a Secure Connection Failed error


How to set proper candidates for tuning a triton kernel?

I see the matmul kernel example here: https://github.com/triton-lang/triton/blob/main/python/tutorials/03-matrix-multiplication.py
You set 8 candidates for fp16 gemm.
I wonder how do you find the 8 sets of parameters?
Is there any other parameters more efficient for some specific shape gemm ?
How can I search in a large search space to get a better kernel?  Or how can I set the values of the parameters?
Thank you!
[
        triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 256, 'BLOCK_SIZE_K': 64, 'GROUP_SIZE_M': 8}, num_stages=3,
                      num_warps=8),
        triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 256, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4,
                      num_warps=4),
        triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4,
                      num_warps=4),
        triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 64, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4,
                      num_warps=4),
        triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 128, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4,
                      num_warps=4),
        triton.Config({'BLOCK_SIZE_M': 128, 'BLOCK_SIZE_N': 32, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=4,
                      num_warps=4),
        triton.Config({'BLOCK_SIZE_M': 64, 'BLOCK_SIZE_N': 32, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=5,
                      num_warps=2),
        triton.Config({'BLOCK_SIZE_M': 32, 'BLOCK_SIZE_N': 64, 'BLOCK_SIZE_K': 32, 'GROUP_SIZE_M': 8}, num_stages=5,
                      num_warps=2),

(I am not from the triton team, so just sharing my two cents)
I think you can just generate all combinations of the autotuning parameters in python with
import itertools
itertools.product(...)

to search for a large space (blindly)
To hand-tune for the best parameters (instead of relying on the autotuning process to search blindly), I think you can generate the nsight computer profiling for running a triton kernel, and see how a certain change of the parameter could affect the lower level GPU performance metrics (on the [nsight compute app](https://developer.nvidia.com/nsight-compute)); which needs the domain knowledge of GPU performance optimization (which is actually what triton aims to free you from via compiler optimization) and can be time consuming.

(I am not from the triton team, so just sharing my two cents)
I think you can just generate all combinations of the autotuning parameters in python with
import itertools
itertools.product(...)

to search for a large space (blindly)
To hand-tune for the best parameters (instead of relying on the autotuning process to search blindly), I think you can generate the nsight computer profiling for running a triton kernel, and see how a certain change of the parameter could affect the lower level GPU performance metrics (on the nsight compute app); which needs the domain knowledge of GPU performance optimization (which is actually what triton aims to free you from via compiler optimization) and can be time consuming.

Thank you!  I'm now trying the blind search and it really gives fast kernels.except that the search space is so large and it takes a lot of time. But that's ok given the fact that it saves me a lot of time to tune it by hand.
BTW, are you familiar with tuning kernels using nsight compute? I used it, but I can't get an effective advice from it. because the metrics are so many. Do you have any advice for that, or some tutorials? Thank you!

yeah autotuning (blind search) is very powerful in my experiences as well; I do not have concrete stats, but I think it is slow primarily due to repeated compilations (for each shape being profiled here, a JIT compilation will be triggered)
I am also a beginner in nsight compute profiling; I have enjoyed this series of nvidia tutorials on youtube like:

https://www.youtube.com/watch?v=GCkdiHk6fUY
https://www.youtube.com/watch?v=uHN5fpfu8As

https://www.youtube.com/watch?v=uHN5fpfu8As

Thank you for the sharing!
I tried triton searching, and it's really powerful, I generate gemm with triton and it's faster than cublas.


Add ability to subscribe to multiple keys or any change in a given suite

This can be useful when you need to handle changes to multiple keys the same.
Should support optionally giving an array or keys or if not given subscribe to all changes.
Something like this:
Defaults.observeAll<T: Codable>(
	_ keys: Defaults.Key<T>,
	options: NSKeyValueObservingOptions = [.initial, .old, .new],
	handler: @escaping () -> Void
) -> DefaultsObservation

Defaults.observeAll<T: Codable>(
	suite: UserDefaults = .standard,
	options: NSKeyValueObservingOptions = [.initial, .old, .new],
	handler: @escaping () -> Void
) -> DefaultsObservation

They should include change objects.
It should also prevent infinite recursion if you call, for example, Defaults[.foo] = true inside the handler as that would normally emit an event, which would call it again infinitely. If it doesn't make sense as default, at least there should be an option for this.
The suite: overload would get all the keys in the given suite and listen to those events. It should not use UserDefaults.didChangeNotification.
I think it could be useful with an overload that listens to all keys defined in Defaults.Keys.
Feedback wanted.

IssueHunt Summary

Backers (Total: $64.00)

 sindresorhus ($64.00)

Become a backer now!
Or submit a pull request to get the deposits!
Tips

Checkout the Issuehunt explorer to discover more funded issues.
Need some help from other developers? Add your repositories on IssueHunt to raise funds.

IssueHunt has been backed by the following sponsors. Become a sponsor

@sindresorhus  observeAll or publisherAll for user-defined keys, should be possible using Mirror reflection.

It should also prevent infinite recursion if you call, for example, Defaults[.foo] = true inside the handler as that would normally emit an event, which would call it again infinitely. If it doesn't make sense as default, at least there should be an option for this.

I found this. The observeValue method would be called once again, but wouldn't propagate changes to later handlers.

Thanks for chiming in. That sounds good.


Entity manager's spurious injection

The orm entity manager is injected into the drest manager object but may be required to determine default behaviours on a number of drest features.
Currently the entity manager is being injected and passed around a number of function calls in ClassMetaData, RouteMetaData and ExposeFields. Implement a cleaner way of allowing access (service locator? dependency injector?)

Using an Entity Manager Registry to solve this. Also benefits the user as they can register a number of managers / connections.


Frontend ~ editable text

NOTE, Merge first the pull request: https://github.com/filecoin-project/mir/pull/527
New Feature:
Added the possibility to modify the logs that each MIR node sends by modifying the incoming message. (For now, only the JSON leaves are modifiable. )
All the JSON types have been taken care of:

it's important to notice that once a 'null' field is clicked to modify, it will become a 'string type afterward'.
For other types, the type will be preserved if the newly modified value is acceptable for that specific type. For example, inserting 'true' into a boolean field will remain a boolean, but using 'tru' will result in it becoming a string field.

Usage Example:
Modifiable incoming log (highlighted by the border line):

Once clicked it becomes editable as a string:

By clicking 'Replace' the modified version will be saved (and we will receive the next log in the 'Incoming Log' section):

If we click simply 'Accept' we will accept the original version of the incoming log  (and we will receive the next log in the 'Incoming Log' section):

If we click 'Decline' will just decline and drop the message (even if modified) (and we will receive the next log in the 'Incoming Log' section)

Finally almost ready to merge this @andreaj00 ! :-)
It just needs to be rebased on top of the visualization branch. There are some conflicts that can probably be trivially resolved (by you, not by me, even though I tried to have a look at it myself).
I'll merge it as soon as the conflicts are resolved.


Do not error when skipping natively supported device

Is your feature request related to a problem? Please describe:
Errors indicate problems that need to be fixed, but skipping devices natively supported by Homekit does not need to be fixed, yet it's logged as an error.
Describe the solution you'd like:
Skipping natively supported devices be logged as info messages or not logged at all.
Describe alternatives you've considered:
None.
Additional context:
None.

I agree, just haven't had the time to fix it. Closing as a dupe of #71 :)


[ZEPPELIN-2970] Enhance Authentication decrypting key generation

What is this PR for?
Overview : The random number generator implemented by nextInt() cannot withstand a cryptographic attack.
In the file IdHashes.java similar issues were on line numbers 74
Comments : Use of SecureRandom is recommended
What type of PR is it?
[Refactoring]
What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2970

How should this be tested?
CI should be green

LGTM

Thanks for the review will merge this if no more discussion.
CI fails for Job #470.2 with following exception; which is unrelated
1) Search block e2e Test correct count of selections
[INFO]   - Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])
[INFO]
[INFO] 2) Search block e2e Test correct matches count number
[INFO]   - Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])
[INFO]
[INFO] 3) Search block e2e Test counter increase and decrease correctly
[INFO]   - Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])
[INFO]
[INFO] 4) Search block e2e Test matches count changes correctly after replace
[INFO]   - Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])
[INFO]
[INFO] 5) Search block e2e Test replace all works correctly
[INFO]   - Failed: No element found using locator: By(xpath, //div[@class="new-paragraph" and @ng-click="insertNew('below');"])

@felixcheung  yes there were four more, have updated those as well.


Added support for creation of resources with repository-supplied paths.

Resolves https://jira.duraspace.org/browse/FCREPO-1588

Resolved with: https://github.com/fcrepo4-labs/fcrepo4-client/commit/0375aec44ae0037636497e1820e0dde6885d352b


Using for intune managed MacOS devices?

Hello. We use intune for managing our MacOS devices and would be really cool to use your script, i've tested it on a mac and seems to work great. Great job on this!
Wondering if you have any ideas for deploying this to MacOS managed devices? We can ofcourse deploy the script itself, but we would like some sort of the data (maybe the log it generates) pushed back to us in order to recieve some sort of status or log back to ensure that there were no errors, and X or Y was successfully patched and so forth.
Any ideas here?

Would something like this work for you?
Login on Microsoft Intune admin center.
Click on Devices > macOS >Shell scripts.
Click on any Script deployment name.
Go to either Device status or User status under Monitor.
Click on the Device Name or User name > A Pane will open on the right-hand side.
Click on Collect logs.
You’ll need the absolute path to the log file to collect, but this would be one way.
We currently don’t enroll our computers into Intune so I can’t verify or know if this action can be automated or added onto your #app-auto-patch script deployment.
Additionally, depending on how deep you want to go, you could either have a Teams integrated webhook to send you failures or successes (there’s a good example of this in #setup-your-mac), or maybe a PowerAutomate webhook workflow, sending your logs to PA and getting them where they need after that. For questions on that you can check the #webhooks channel


dbgate: init at 5.2.8

Description of changes
Closes #306756
Release notes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

> nix build .#dbgate
> tree result/
 result
└──  bin
   └──  dbgate

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3950

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

@Misterio77 Since the package had a new release and the PR is still opened I thought it was a good idea to bump the version to 5.2.8 (without the need to make a new pr in a hopefully short time)
The approval should be fast enought

Result of nixpkgs-review pr 311405 run on x86_64-linux 1

1 package built:

dbgate

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