[BUG] Getting CPU allocation error after the update on 12/15/2023

Describe the bug
Already installed the update on 12/15/2023 via "pip3 install gemini-torch --upgrade". Running the example code generates the following error.  -- Maybe Torch cannot be forced to run on the GPU? I have an NVidia GeForce RTX 2070.
"RuntimeError: [enforce fail at alloc_cpu.cpp:80] data. DefaultCPUAllocator: not enough memory: you tried to allocate<PHONE_NUMBER>2800 bytes."
To Reproduce
Steps to reproduce the behavior:

Install Gemini-Torch per Readme.md
Run the example.py
See error

Expected behavior
The example completes without error and displays a matrix.
Screenshots
Please see screenshots attached.

Upvote & Fund

We're using Polar.sh so you can upvote and help fund this issue.
We receive the funding once the issue is completed & confirmed by you.
Thank you in advance for helping prioritize & fund our backlog.

@corous try to use a better cpu this a big model I can try adding dynamic cpu usage but it will take some time

Got it.
So, there’s no way to force it to run on the GPU? Or, reduce the dimension / parameters or size of the model to allow it to run?
If not, I can run out to buy myself a new CPU for Christmas. Or maybe run it on a VM.

@corous yes just lower the dimension and depth and sequence length and maybe vocabulary size and it should work

Thank you so much. I really appreciate this implementation and your hard work on responding to the issues.
So, I reduced the dimension, depth, sequence length and max tokens (I'm guessing this is what you meant by vocabulary size). I'm getting a "Failed in forward method:  too many values to unpack (expected 5)" error now.
The code and error message are copied and pasted below.

Code:
import torch
from gemini_torch import Gemini
Initialize the model
model = Gemini(
num_tokens=4096,
max_seq_len=1024,
dim=256,
depth=8,
dim_head=128,
heads=24,
use_abs_pos_emb=False,
alibi_pos_bias=True,
alibi_num_heads=12,
rotary_xpos=True,
attn_flash=True,
attn_kv_heads=2,
qk_norm=True,
attn_qk_norm=True,
attn_qk_norm_dim_scale=True,
)
Initialize the text random tokens
x = torch.randint(0, 4096, (1, 1024))
Apply model to x
y = model(x)
Print logits
print(y)

Error:
Failed in forward method:  too many values to unpack (expected 5)
ValueError                                Traceback (most recent call last)
Input In [2], in <cell line: 28>()
25 x = torch.randint(0, 4096, (1, 1024))
27 # Apply model to x
---> 28 y = model(x)
30 # Print logits
31 print(y)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\gemini_torch\model.py:158, in Gemini.forward(self, text, img, audio, *args, **kwargs)
155     else:
156         model_input = self.decoder.forward(text)[0]
--> 158     return self.decoder(model_input, padded_x=model_input[0])
159 except Exception as e:
160     print("Failed in forward method: ", e)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\zeta\structs\auto_regressive_wrapper.py:320, in AutoregressiveWrapper.forward(self, x, return_loss, **kwargs)
317     mask = ~torch.zeros_like(inp).scatter(1, indices, 1.0).bool()
318     kwargs.update(self_attn_context_mask=mask)
--> 320 logits = self.net(inp, **kwargs)
322 loss = F.cross_entropy(
323     rearrange(logits, "b n c -> b c n"),
324     target,
325     ignore_index=ignore_index,
326 )
328 if return_loss:
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\gemini_torch\transformer.py:1599, in Transformer.forward(self, x, return_embeddings, return_logits_and_embeddings, return_intermediates, mask, return_mems, return_attn, mems, pos, prepend_embeds, sum_embeds, return_attn_z_loss, attn_z_loss_weight, **kwargs)
1582 def forward(
1583     self,
1584     x,
(...)
1597     **kwargs,
1598 ):
-> 1599     b, n, device, num_mem, emb_frac_gradient = (
1600         *x.shape,
1601         x.device,
1602         self.num_memory_tokens,
1603         self.emb_frac_gradient,
1604     )
1605     return_hiddens = (
1606         return_mems | return_attn | return_intermediates | return_attn_z_loss
1607     )
1609     # absolute positional embedding
ValueError: too many values to unpack (expected 5)

@corous I just tried fixing it by removing [0] in the forward method, try it out and lmk if it works for you

Hey @kyegomez, thank you for the continued support and prompt response.
I ran a "pip3 install gemini-torch --upgrade".
Now I'm getting a "'tuple' object has no attribute 'shape'" error.

Code:
import torch
from gemini_torch import Gemini
Initialize the model
model = Gemini(
num_tokens=4096,
max_seq_len=1024,
dim=256,
depth=8,
dim_head=128,
heads=24,
use_abs_pos_emb=False,
alibi_pos_bias=True,
alibi_num_heads=12,
rotary_xpos=True,
attn_flash=True,
attn_kv_heads=2,
qk_norm=True,
attn_qk_norm=True,
attn_qk_norm_dim_scale=True,
)
Initialize the text random tokens
x = torch.randint(0, 4096, (1, 1024))
Apply model to x
y = model(x)
Print logits
print(y)

Error:
Failed in forward method:  'tuple' object has no attribute 'shape'
AttributeError                            Traceback (most recent call last)
Input In [1], in <cell line: 28>()
25 x = torch.randint(0, 4096, (1, 1024))
27 # Apply model to x
---> 28 y = model(x)
30 # Print logits
31 print(y)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\gemini_torch\model.py:158, in Gemini.forward(self, text, img, audio, *args, **kwargs)
155     else:
156         model_input = self.decoder.forward(text)
--> 158     return self.decoder(model_input, padded_x=model_input)
159 except Exception as e:
160     print("Failed in forward method: ", e)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\zeta\structs\auto_regressive_wrapper.py:307, in AutoregressiveWrapper.forward(self, x, return_loss, **kwargs)
284 def forward(self, x, return_loss=True, **kwargs):
285     """
286     Forward pass of the autoregressive wrapper.
287
(...)
305
306     """
--> 307     seq, ignore_index = x.shape[1], self.ignore_index
309     inp, target = x[:, :-1], x[:, 1:]
311     if self.mask_prob > 0.0:
AttributeError: 'tuple' object has no attribute 'shape'

@corous i think i might have fixed it but i don't know forsure please try again. It consumes too much memory even for me

@kyegomez  Hey, I ran a "pip3 install gemini-torch --upgrade", but it doesn't look like anything got updated during the process. I'm also still getting the "Failed in forward method:  'tuple' object has no attribute 'shape'" error.

Code:
import torch
from gemini_torch import Gemini
Initialize the model
model = Gemini(
num_tokens=4096,
max_seq_len=1024,
dim=256,
depth=8,
dim_head=128,
heads=24,
use_abs_pos_emb=False,
alibi_pos_bias=True,
alibi_num_heads=12,
rotary_xpos=True,
attn_flash=True,
attn_kv_heads=2,
qk_norm=True,
attn_qk_norm=True,
attn_qk_norm_dim_scale=True,
)
Initialize the text random tokens
x = torch.randint(0, 4096, (1, 1024))
Apply model to x
y = model(x)
Print logits
print(y)

Error:
Failed in forward method:  'tuple' object has no attribute 'shape'
AttributeError                            Traceback (most recent call last)
Input In [1], in <cell line: 28>()
25 x = torch.randint(0, 4096, (1, 1024))
27 # Apply model to x
---> 28 y = model(x)
30 # Print logits
31 print(y)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\gemini_torch\model.py:158, in Gemini.forward(self, text, img, audio, *args, **kwargs)
155     else:
156         model_input = self.decoder.forward(text)
--> 158     return self.decoder(model_input, padded_x=model_input)
159 except Exception as e:
160     print("Failed in forward method: ", e)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1518, in Module._wrapped_call_impl(self, *args, **kwargs)
1516     return self._compiled_call_impl(*args, **kwargs)  # type: ignore[misc]
1517 else:
-> 1518     return self._call_impl(*args, **kwargs)
File E:\Anaconda3\lib\site-packages\torch\nn\modules\module.py:1527, in Module._call_impl(self, *args, **kwargs)
1522 # If we don't have any hooks, we want to skip the rest of the logic in
1523 # this function, and just call forward.
1524 if not (self._backward_hooks or self._backward_pre_hooks or self._forward_hooks or self._forward_pre_hooks
1525         or _global_backward_pre_hooks or _global_backward_hooks
1526         or _global_forward_hooks or _global_forward_pre_hooks):
-> 1527     return forward_call(*args, **kwargs)
1529 try:
1530     result = None
File E:\Anaconda3\lib\site-packages\zeta\structs\auto_regressive_wrapper.py:307, in AutoregressiveWrapper.forward(self, x, return_loss, **kwargs)
284 def forward(self, x, return_loss=True, **kwargs):
285     """
286     Forward pass of the autoregressive wrapper.
287
(...)
305
306     """
--> 307     seq, ignore_index = x.shape[1], self.ignore_index
309     inp, target = x[:, :-1], x[:, 1:]
311     if self.mask_prob > 0.0:
AttributeError: 'tuple' object has no attribute 'shape'

@corous please forgive me there was a whole vast array of issues trying to enable omni-modality transformation from multiple dimensions into text dimension, it should work now, but there may be alot of room for optimization please let me know what you think. in the model file!

@kyegomez   No worries. Totally understand. I tried it once last week and again today. Restarting and doing a "pip install gemini-torch --upgrade" both times.
I'm getting the error below now. If it's too much trouble to account for multi-modal, no worries. I will just try to run it on a different VM environment.

@corous hey try again the implementation has been completed


Add a new configuration to ignore query params in cache keys

This adds a new configuration option:
CONFIG proxy.config.http.cache.ignore_query INT 0

This is off by default, but it is overridable. The typical use case for this would be to use e.g. conf_remap.so plugin, to set this for services / URLs which shouldn't use the query parameters in the HTTP cache key calculations.
This would be significantly less expensive (less CPU) than using e.g. the cachekey.so plugin to simply strip the query parameters from the key.

Moderately tested, but first without setting this configuration:
leif@Hel ~/ » curl -D -s -o /dev/null "http://hel.ogre.com/bench/100.bmp" | grep HTTP/1.1
HTTP/1.1 200 OK
leif@Hel ~/ » curl -D -s -o /dev/null -H "Cache-Control: only-if-cached" "http://hel.ogre.com/bench/100.bmp" | grep HTTP/1.1
HTTP/1.1 200 OK
leif@Hel ~/ » curl -D -s -o /dev/null -H "Cache-Control: only-if-cached" "http://hel.ogre.com/bench/100.bmp?foo=bar" | grep HTTP/1.1

Then setting it via @plugin=conf_remap.so @pparam=proxy.config.http.cache.ignore_query=1 :
leif@Hel ~/ » curl -D -s -o /dev/null "http://hel.ogre.com/bench/100.bmp" | grep HTTP/1.1
HTTP/1.1 200 OK
leif@Hel ~/ » curl -D -s -o /dev/null -H "Cache-Control: only-if-cached" "http://hel.ogre.com/bench/100.bmp" | grep HTTP/1.1
HTTP/1.1 200 OK
leif@Hel ~/ » curl -D -s -o /dev/null -H "Cache-Control: only-if-cached" "http://hel.ogre.com/bench/100.bmp?foo=bar" | grep HTTP/1.1
HTTP/1.1 200 OK

So, clearly getting the cache hit on the request with the query params. I tested with various other query params, and they of course also hits the same cache object.

Benchmarking this against the cachekey plugin, the difference is surprisingly small. This option is about 10-15% faster than the plugin. But it's an simple addition, so worth it I think.

[approve ci centos]


systecm restarts k3s, which causes   coredns dropping configmap configuration

OS  Version:   ubuntu 18.04
k3s Version:   1.21.4+k3s1
kubectl describe   configmaps   coredns  -n kube-system
Name:         coredns
Namespace:    kube-system
Labels:       objectset.rio.cattle.io/hash=bce283298811743a0386ab510f2f67ef74240c57
Annotations:  objectset.rio.cattle.io/applied:
{"apiVersion":"v1","data":{"Corefile":".:53 {\n    errors\n    health\n    ready\n    kubernetes cluster.local in-addr.arpa ip6.arpa {\n  ...
objectset.rio.cattle.io/id:
objectset.rio.cattle.io/owner-gvk: k3s.cattle.io/v1, Kind=Addon
objectset.rio.cattle.io/owner-name: coredns
objectset.rio.cattle.io/owner-namespace: kube-system
Data
Corefile:
.:53 {
errors
health
ready
kubernetes cluster.local in-addr.arpa ip6.arpa {
pods insecure
fallthrough in-addr.arpa ip6.arpa
}
hosts /etc/coredns/NodeHosts {
<IP_ADDRESS>  datamart.tool.com
<IP_ADDRESS> bigdata.tool.com
<IP_ADDRESS> employee.tool.com
ttl 60
reload 15s
fallthrough
}
prometheus :9153
forward . /etc/resolv.conf
cache 30
loop
reload
loadbalance
}
command : systecm restarts k3s ,  After restarting k3s, the following configuration is lost(Rebooting the server can also cause configuration loss)
<IP_ADDRESS>  datamart.tool.com
<IP_ADDRESS> bigdata.tool.com
<IP_ADDRESS> employee.tool.com

How can this be solved?
What is causing the problem?
Thank You!

The coredns configmap is managed by K3s; any changes made to it will be reverted on startup. If you want to inject some additional values into the configmap, you can do so by creating a configmap named coredns-custom. See this issue for more information:

https://github.com/k3s-io/k3s/pull/4397#issue-1044719103

The coredns configmap is managed by K3s; any changes made to it will be reverted on startup. If you want to inject some additional values into the configmap, you can do so by creating a configmap named coredns-custom. See this issue for more information:

[Feature] Customizing CoreDNS ConfigMap with overrides and additional servers #4397 (comment)

What causes restarting k3s to overwrite coredns configmap?
Thank You!

It is part of the coredns manifest. All  of the manifests for packaged components are re-deployed on startup.

It is part of the coredns manifest. All of the manifests for packaged components are re-deployed on startup.

I tried to modify /var/lib/rancher/k3s/server/manifests/coredns.yaml
and then restarting still doesn't seem to work
New coredns-config.yaml in /var/lib/rancher/k3s/server/manifests, how to make it work?
Thank You

If you want to modify it, you should copy it to a new file and then start k3s with --disable=coredns

If you want to modify it, you should copy it to a new file and then start k3s with --disable=coredns

--disable=coredns
Isn't disabling not deploying coredns?

Correct, but it will instead deploy coredns from the manifest that you copied and customized...


README spelling changes

Fixes # .
Changes proposed in this pull request:

Coverage remained the same at 100.0% when pulling e6ff3d6fc1ab6e21479d33107d62ae88f7ec6cec on masudrahman:patch-1 into 10a0c15f6fb3208ae4de9b59b1ff82058af55b6c on zalando:master.

:+1:

😨


Add TransactionRecordQuery calls to capture fail transaction costs

Signed-off-by: Nana Essilfie-Conduah<EMAIL_ADDRESS>Description:
Add TransactionRecordQuery calls to capture fail transaction costs

update executeTransaction catch with TransactionRecordQuery logic
update executeGetTransactionRecord catch with TransactionRecordQuery logic

Related issue(s):
Fixes #592
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report
Base: 71.39% // Head: 70.88% // Decreases project coverage by -0.50% :warning:

Coverage data is based on head (52e8272) compared to base (54bb7d4).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #593      +/-   ##
==========================================
- Coverage   71.39%   70.88%   -0.51%
==========================================
  Files          16       16
  Lines        1115     1123       +8
  Branches      192      194       +2
==========================================
  Hits          796      796
- Misses        267      275       +8
  Partials       52       52

Impacted Files
Coverage Δ

packages/relay/src/lib/clients/sdkClient.ts
6.50% <0.00%> (-0.33%)
:arrow_down:

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

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


how to resize the GWorkspace

Great start!  But do you know how to make the desktop larger in VirtualBox?
Using in a VirtualBox VM:  The desktop can be resized with
$ xrandr -s 1024x768

but I cannot figure out how to resize GWorkspace to fit the new size.  Also, once popped up, the Calendar
obscures the gworkspace bar too.
If I find a workaround I will share it.

in windowmaker you can use ctrl or alt and lmb or rmb for move and resize, then there is also the command defaults read


Problem with testers, projectors, and implications

Why doesn't this type-check?
val find: f:('a -> Tot bool) -> l:list 'a ->
      Tot (res:(option 'a){is_Some res ==> f (Some.v res)})
let rec find f l = match l with
  | [] -> None
  | hd::tl -> if f hd then Some hd else find f tl

The error message is:
ex4e-find.fst(12,53-12,56) : Error
Expected expression of type "_122:(option ){(is_Some _122)}";
got expression "res" of type "(option 'a)"

The following alternative type does work for the same thing:
val find: f:('a -> Tot bool) -> l:list 'a ->
      Tot (res:(option 'a){forall x. res = Some x ==> f x})

Problem is gone now.


Image of dogs surround the table

Creating image that having dogs surround the table.

yeah, that is the image we need in create event page and create table page


[Config Request] Speed of Growth

Is there any way you can add a config option to adjust the speed at which a tree completes a growth cycle?  It seems very fast right now, for the purposes of my pack at least.

There is no global modifier for the growtime (yet), but you can configure the json based trees individually.
I'll add that global modifier and a CraftTweaker method for non-json trees.
Would that work for you?

Does configuring the json based trees myself allow me to adjust their growth time?

Yes, it would. They have a growTimeMultiplier field. You'd simply extract the json files in the types.d folder to your config/bonsaitrees/types.d folder and overwrite/add their growTimeMultiplier field.
That has some disadvantages though. If I change the original json in the jar yours still has priority and would overwrite my changes.
At this point I'd recommend waiting a bit until I get the above mentioned features in and put out a new release. Probably sooner (i.e. today) than later (at most Sunday).

Build is uploaded to curse, it takes a moment to get approved.
Enjoy!

Awesome work!  Already tested it out and it works splendidly!  Nice job!


GPG Signing Key for PyPI and GH Releases

Starting I think in 1.3.0, I upload to PyPI using twine's -s facility, but using my own key.
Should we create an 'official' GPG key for uploading to PyPI?
This key will then also be used to create the .asc files to be released alongside the .tar.gz and .whl via GitHub Releases.

I'm not sure.
I know that typically I've seen folks have personal signing keys.
The humanitarian side of me says that it's good to trust people and trusting folks involved with the project is :+1:
The Red Team in me asks what happens if someone gets angry, or approached by a gov't entity, or or or or...
I honestly don't know what (if any) the "right" answer is -- but I lean towards folks having their own keys and just publishing a list of keys that are valid for signing. :shrug:

At this moment I actually already did exactly like you suggested: Signing with my own key, and providing info about the key in both README and PyPI.
I was just throwing some ideas into the wind 😄
I'm okay with any decision.


Public Link Data Service Command

Disclaimer
Pushing for tracking. Beware, replaces to local path.
Dependency tree:

https://github.com/owncloud/ocis-reva/pull/258

https://github.com/cs3org/reva/pull/822

this does not compile for me:
rm -rf config/identifier-registration.yaml
go clean -i ./...
go<EMAIL_ADDRESS>requires
<EMAIL_ADDRESS>parsing ../../refs/reva/go.mod: open /home/vincent/Private/Work/refs/reva/go.mod: no such file or directory
make: *** [Makefile:62: clean] Error 1

Tried to isolate the deps update here https://github.com/owncloud/ocis/pull/328

Aas

Obsolete, we don't need a data service for public links as the public storage provider relays everything to the regular storage provider + regular data service, see https://github.com/cs3org/reva/pull/822#issuecomment-648227346


Issue 175

Addresses #175

Added check for inconsistent row lengths

Added checks for correct value types after splitting on expecting single space separator.

The latter will also catch invalid separator usage.

Updated affected tests.

@arokem - could you have a look?

LGTM
On Tue, Jul 12, 2016 at 4:10 PM, Chris Filo Gorgolewski <
<EMAIL_ADDRESS>wrote:

@arokem https://github.com/arokem - could you have a look?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/INCF/bids-validator/pull/177#issuecomment-232182565,
or mute the thread
https://github.com/notifications/unsubscribe/AAHPNutqGtL85LBwCiy0P6ZBxTE52Ax_ks5qVALjgaJpZM4JJq3D
.


Not compatible with Python 3.12

netneurotools.datasets.utils depends on pkg_resources which has been removed from Python 3.12.

Fixed in #127

@liuzhenqi77 sorry to insist.
Are there any other pending issues / PRs preventing a new release? We are very keen on releasing a version of Clinica compatible with Python 3.12, but it is currently blocked by upgrades of nilearn to version 0.10.x and netneurotools to the current revision.
Thanks 🙏

Hi @ghisvail , sorry for the delay.
About the dependency issue, if I understand it correctly, the dependency is actually brainstat, and it would by default install the latest version of netneurotools? I can see that your current PR has a working fix by installing netneurotools directly from a certain commit. We would actually recommend you to use this fix as a temporary solution for now.
The main issue on our side is, we are currently re-organizing the structure of netneurotools internally. There are currently some untested functions in the package that we would be hesitate to put into a new release. Additionally, there would be a major breaking release in the next few weeks/months. This means you would probably have to make changes in brainstat to avoid incompatibility by then.
What we'd suggest is that it's probably best to use the git/github commit-based installation method for now. After we release the breaking change, we could submit a PR to brainstat and coordinate with their team to make a new release to solve the dependency issue completely. Then you could fully remove the version constraints in Clinica.
Please feel free to let us know how you think. Thanks!

Le 13 février 2024 00:34:36 GMT+01:00, Zhen-Qi Liu @.***> a écrit :

Hi @ghisvail , sorry for the delay.
About the dependency issue, if I understand it correctly, the dependency is actually brainstat, and it would by default install the latest version of netneurotools?

You are correct.

I can see that your current PR has a working fix by installing netneurotools directly from a certain commit. We would actually recommend you to use this fix as a temporary solution for now.

The working fix is only useful for running tests in our CI. It is not possible to publish packages on PyPI with unreleased dependencies, such as a Git snapshot.
Sadly though, that means this solution is not satisfactory for us, as it would prevent us from making a release compatible with Python 3.12.

The main issue on our side is, we are currently re-organizing the structure of netneurotools internally. There are currently some untested functions in the package that we would be hesitate to put into a new release. Additionally, there would be a major breaking release in the next few weeks/months. This means you would probably have to make changes in brainstat to avoid incompatibility by then.

Could you please provide a bug fix release whilst working on your major refactoring? We have already provided the necessary triaging and fixes to make that happen, and don't want to spend too much time tracking down regressions from further big changes.

What we'd suggest is that it's probably best to use the git/github commit-based installation method for now. After we release the breaking change, we could submit a PR to brainstat and coordinate with their team to make a new release to solve the dependency issue completely. Then you could fully remove the version constraints in Clinica.

As explained above, this is not satisfactory for us. We need a proper release of netneurotools compatible with Python 3.12 or we, and other packages using brainstat, are stuck.

Please feel free to let us know how you think. Thanks!

Thank you for considering our viewpoint.
Cheers 👍

Hi @ghisvail, we have released a new version as requested. Hope it helps.


Birthdays are missing?

My widget does not show any birthdays, for which I could not find any option in the settings. I feel like remembering that a year ago or so the widget showed me birthdays. Feel free to correct me :)

Hi @minlength-is-0
Are you still using the Calendar-Widget app or you migrated to ToDo Agenda?
In the latter case please look at similar questions here: https://github.com/andstatus/todoagenda/issues/68

Hi @minlength-is-0
Are you still using the Calendar-Widget app or you migrated to ToDo Agenda?
In the latter case please look at similar questions here: andstatus#68

Still using the Calender-Widget but now migrated to ToDo Agenda and everything works like a charm. Thanks


Don't install peerDependencies which can be resolved by Webpack

If you're shimming a peer dependency via resolve.alias, e.g. like Preact and Inferno do with their React compatibility modules, then you don't want the peer installed and know if can be safely ignored.

Additional thought: should you give Webpack a chance to try to resolve peerDependencies instead of going straight for an install? This would make the shimming via resolve.alias use case just work.

If peerDependencies were handled like any other normal module the plugin tries to install and Webpack was able to resolve the package name, they wouldn't be installed.

Relevant from the closed PR:
If, instead of being installed up-front, peerDependencies were handled like any other normal module the plugin tries to install and Webpack was able to resolve them package via aliases, fallback module directories etc. they wouldn't be installed.
We can't ignore peerDependencies because Plugin#resolveModule() skips sub-dependencies - could install() return something the plugin could use to check if it should attempt to resolve/install sub-dependencies which are known to be peerDependencies?

Anyone got a module which demonstrates a solid use case for needing to install peerDependencies?
I have a build which disables up-front installation and has lots of debug logging to show what's going on,  and it works just fine in example/webpack2 and my nwb preact case.


Property 'color' does not exist on type 'Framebuffer2D'

I am trying to convert the sprites.js example to TypeScript. The first issue I encountered is #602, and the second is that on line 159 it says SPRITES[(tick) % 2].color[0], but the Framebuffer2D class does not have a color property.

This could be the line where the color property is assigned https://github.com/regl-project/regl/blob/master/lib/framebuffer.js#L635.


Disable flame profile

We're having some account issues related to generating the flame graph. This PR disables it, hopefully we can add it back in eventually.

bors r+


Displaying Tasks in a Table as a Column Like Plugin (Folder DB)

Problem
We like to have the idea and vision of putting tasks on a schedule for review and reminders.
Sometimes we have important tasks to accomplish, so if it is displayed in the dataloom table, it will be something important and special.
Solution
Like Plugin (Obsidian Folder DataBase) It can also display tasks in a table as a column.
Additional details
Thank you :)

@7Codez Could you provide a screenshot of this behavior in Folder DB?

Thanks for providing that screenshot. I understand what you mean now.


Correct method to use for setting TableView Cell view data

I have a custom tableview cell defined in its own class and I have linked few outlets. setLayoutState seems to be working without any problem, but when i am trying to access the outlets in
  func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell

I get nil outlets when scrolling, seems to be happening when the cell is reused. Is there any method which is called when the cell is shown

@TwunTee you might need to call setState() or update() on the cell node before trying to reference the outlets.

I am accessing the outlets right after setState

    @IBOutlet var likeButton: DOFavoriteButton!
    var viewModel: StyleCellViewModel?
    func refreshState() {
        self.layoutNode?.setState(self.viewModel?.model!)
                        if let liked = self.viewModel?.model?.liked {
            self.likeButton?.isSelected = liked
        }
    }

I am calling refresh state method of the cell in
func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell
But likeButton seems to be nil when scrolling, its not nil initially

@TwunTee hmm, that all looks correct. Can you upload a sample project so I can investigate?

It was a mistake on my end. One of the object inside the struct was nil and i was trying to access its property and that was causing the error. Thanks for helping out!!


Viro3DObject disappear when reopen the application in the phone

Now I'm using an Android phone to develop, and I found the demo called 'ARSimpleSample' has a bug. when the user has installed the app on the phone and the USB is not connected to the PC, the 3DObject will disappear if the user kills the app and reopen it. it will only show the object once after installation. Not only 'ARSimpleSample' but also 'ARDrivingCarDemo' has this problem. the car will also disappear when reopening the app. I wonder why.


Adding a changelog to the project

I think that a project like youtube-dl could use a changelog to state major changes with each release, it will benefit everyone especially people checking the project after some time without going through each commit.

Duplicate of #1626.


Add ImageSpec.from_env

Tracking issue

Closes https://github.com/flyteorg/flyte/issues/5889
Why are the changes needed?

Resolve notebook environment mismatch described in https://github.com/flyteorg/flyte/issues/5889
What changes were proposed in this pull request?

This PR adds a ImageSpec.from_env into ImageSpec.
How was this patch tested?

I added unit test to this PR and ran this workflow:
from pathlib import Path

from flytekit import current_context, task, workflow, ImageSpec
from flytekit.types.file import FlyteFile

image_spec = ImageSpec.from_env(
    pinned_packages=["joblib"],
    name="myimage",
    registry="localhost:30000",
    packages=[
        "git+https://github.com/thomasjpfan/flytekit@41a6a74df4bab1b7db4f9ff13eb249e94fe7a2e4"
    ],
    apt_packages=["git"],
)

@task(container_image=image_spec)
def create_file() -> FlyteFile:
    working_dir = Path(current_context().working_directory)

    file = working_dir / "outer_file.txt"
    file.write_text("this is some text")

    return file

@workflow
def main() -> FlyteFile:
    return create_file()

@eapolinario There seems to be a unrelated error: https://github.com/flyteorg/flytekit/actions/runs/12020940988/job/33514788383?pr=2895
Should this be fixed before we can merge this PR?

@eapolinario There seems to be a unrelated error: https://github.com/flyteorg/flytekit/actions/runs/12020940988/job/33514788383?pr=2895
Should this be fixed before we can merge this PR?

yes we can, this is an airflow module error I guess


Offline

Is it currently offline for some reason? Should I be able to tell why?

You can tell if it's offline if it isn't listed as being online on your server. I haven't set up an status webpage or anything like that (yet?).
It does seem to be down, and unintentionally so. Unfortunately, I likely won't be able to fix it today.

Thanks for letting me know!

Should be back up now. Seems to be an issue with Discord resetting the bot's connection after it's been running for a while. Reopening this to remind me to figure out some way to fix this.

Thanks for letting me know. Tonight I should be able to look into this, and hopefully fix it for good.

It's offline again

I'm not sure if the underlying issue is a Discord.py issue or a Discord issue, but hopefully we should be able to get around it now by just reconnecting once the connection is dropped.


Cannot login with session keys (using FormAuthInfo)

Hi all,
I tried to crawl a website using Drupal. However, that Website requires 3 more parameters (besides the username and the password) to login. These parameters are session keys (gotten from hidden fields in the source page).
I cannot override the method "doFormLogin" of the class PageFetcher to extend this method, because it is a private method.
Do you have other ways to do login in similar cases?
Thanks a lot.

Not currently.
The authentication module (FormAuthInfo) could use a good overhaul to
include several login details as well as additional fields
On Mon, Sep 24, 2018 at 5:55 PM mxphu<EMAIL_ADDRESS>wrote:

Hi all,
I tried to crawl a website using Drupal. However, that Website requires 3
more parameters (besides the username and the password) to login. These
parameters are session keys (gotten from hidden fields in the source page).
I cannot override the method "doFormLogin" of the class PageFetcher to
extend this method, because it is a private method.
Do you have other ways to do login in similar cases?
Thanks a lot.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yasserg/crawler4j/issues/351, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABrbW8NyzKFA5YQD0AUAuzvfyYLiaeTzks5uePJigaJpZM4W21BZ
.


Scoring functions in gnina

Hi,
I am sorry for the naive question. What built-in scoring function does gnina have? Also, if I use --scoring to specify alternative scoring function, will it be applied through out the Monte Carlo sampling and full refinement?
Thank you very much!

The default CNN scoring function is an ensemble of 5 models selected to balance pose prediction performance and runtime: dense, general_default2018_3, dense_3, crossdock_default2018, and redock_default2018. More information on these various models can be found in the papers listed in the README.

--cnn_scoring determines at what points of the docking procedure that the CNN scoring function is used.

none - No CNNs used for docking. Uses the specified empirical scoring function throughout.
rescore (default) - CNN used for reranking of final poses. Least computationally expensive CNN option.
refinement - CNN used to refine poses after Monte Carlo chains and for final ranking of output poses. 10x slower than rescore when using a GPU.
all - CNN used as the scoring function throughout the whole procedure. Extremely computationally intensive and not recommended.

If the CNN scoring function isn't being used then the empirical scoring function is used, which defaults to Vina.

Yes, the default is to use the default ensemble.

how can I run the

default2018.def

in /gnina/gninascr with

redock2018_0.def

in /gnina/gnianscr/weights  in a conda python=3.6 environment?

You can see the mapping between commandline names and model weights here: https://github.com/gnina/gnina/blob/master/gninasrc/lib/cnn_data.cpp


experiments collection

Almost all previous Canvas Experiments in one place behind feature switches. All of them are enabled by default.
https://utopia.pizza/p/971e8889-cord-cowl/?branch_name=experiments/collected

Tagged and closed


unify mri and jruby with c++ core

simplify maintence with one core alogrithm for ipog written in c++11.  ffi bindings for both the mri and jruby runtimes.

as of 0.2.0 mri has an extension written in c++11.

compiling the c++11 extension on the jruby runtime is a pain because jruby does not implement the MakeMakefile module.  i need to write a makefile just for jruby.  it's unlikely the makefile would be portable to systems without g++ (gnu compiler collection).  so the user would need to install gcc.
jruby-launcher is a good example of bundling a native extension in jruby.


Support dark theme?

My app is dark by default. The permissions dialog kinda sticks out like a sore thumb :/

Hi @AdityaAnand1
The permissions dialog is native and we don't have any control over it. As it is right now, there is no way we can customize it ourselves.

Oh.


Fix offline.js and caching of talks

For some reason, Firefox seems to always determine that it is offline. (Reproduced from inside company network; need to test this at home as well)
As a result the talks are never updated, and if they are not present, they are not loaded

I'm closing this for now, I think my workplace has some issues where it keeps breaking the localstore - I've not had any trouble from firefox on my other machines


Kernel implementation

I have a weird request before I review this pull request. Can you merge all your pull requests but this one? Then, you'll be given the option to update this branch from main. That way, now all 19 files are here, and it's easier for me to review.


[Bug]: Error generating interferograms in Yamchi Dam notebook example

Describe the bug
Generation starts as expected however seems to stop before the first queue has been generated completely. Only thing that I have changed is the asf log-in details.
To Reproduce
Pull new docker image (https://hub.docker.com/r/pechnikov/pygmtsar) and run YamchiDam.ipynb example.
System and software version:

OS: Linux Mint
PyGMTSAR version: docker.io/pechnikov/pygmtsar (tag = latest)

Processing Log
yamchi_dam_output.txt
yamchi_dam_output2.txt

Try to remove this code completely (to use the default NetCDF5 library) and re-run the example:
if os.path.exists('/.dockerenv') and not 'google.colab' in sys.modules:
    # use different NetCDF backend in Docker containers
    from pygmtsar import datagrid
    datagrid.netcdf_engine = 'netcdf4'

Please let me know about your results. Docker containers work tricky sometimes... For me and some other users, the default NetCDF5 library returns similar error and the NetCDF4 library works well.

Your suggestion resolves this issue.

@benwright-davra Great! Please specify your hardware for reference. It is x64 or arm64 architecture based or something different?

Thanks and sure, please see below:
Kernel: 5.15.0-91-generic x86_64 bits: 64 compiler: gcc v: 11.4.0 Desktop: Cinnamon 5.6.8
Processor: 14-core (6-mt/8-st) model: 12th Gen Intel Core i7-12700H bits: 64 type: MST AMCP, Arch: Alder Lake


Copy line no longer pastes correctly on Insiders

Have a document with your cursor placed at |
hello w|orld

Ctrl+c and ctrl+v. I would expect to get two lines of hello world, but instead I get
hello whello world
orld

Duplicate of #181489


The registry key path for network protection is wrong.

[Enter feedback here]
The registry key path for network protection that is described on the following part of this document is wrong.
It should be corrected with the right registry path.
https://learn.microsoft.com/en-us/microsoft-365/security/defender-endpoint/enable-network-protection?view=o365-worldwide#check-if-network-protection-is-enabled
Current : -> HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\ Windows Defender Exploit Guard \Network Protection
1). Select the Start button in the task bar and type regedit to open Registry editor.

2). Choose HKEY_LOCAL_MACHINE from the side menu.

3). Navigate through the nested menus to SOFTWARE > Policies > Microsoft > Windows Defender > Windows Defender Exploit Guard > Network Protection.

If the Key is missing, Navigate to SOFTWARE > Microsoft > Windows Defender > Windows Defender Exploit Guard > Network Protection.

4). Select EnableNetworkProtection to see the current state of network protection on the device:

0, or Off
1, or On
2, or Audit mode

Actual:
Registry key path of network protection is "HKEY_LOCAL_MACHINE\SOFTWARE\Policies\Microsoft\Windows Defender\Policy Manager"

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 673970ca-2f79-a288-978c-ceb4845c5062
Version Independent ID: 2a58449f-b0d4-bf17-1209-d7b8ec0481b6
Content: Turn on network protection
Content Source: microsoft-365/security/defender-endpoint/enable-network-protection.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@mutsun1 Thank you for your feedback. I will update the document/screenshot with the new registry key.
Thanks
Sri

@mutsun1 Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9980
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @mutsun1 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Export with current style? (cvs/pdf)

Hey Allen,
I want to export with current css style applied to it. Right now, it exports just as text without any styling at all. Is it possible to keep certain format? (font color/background color)?
Also, is there anyway to export as pdf as well? Thanks

@rkdmen, we don't support this kind of feature, PR will be welcome, thanks

Hi,
I was looking into this and checked some online resources.
Here are the thought process.

We cannot add color or any formatting information to CSV as it's just a pure textual content and this is more thoroughly mentioned in https://tools.ietf.org/html/rfc4180
There are some libraries which help you generate xlsx from javascript, but that would mean a whole new way of doing the data manipulation and also.

Regards,
Nagaraj

hello,
I actually had to create whole new code using exceljs on backend. I think that was only way to customize the style way I want. It was time consuming to apply all the styles but I think it's the only way.

@rkdmen Yes, perhaps very true. There has to be another approach.
@AllenFang Can go ahead and close this, as the feature to be requested would be to export as xlsx and not csv with color.

@tan31989 @rkdmen thanks you guys sharing 👍


Response builder for different channels in ms-botframework

What would be the approach to achieving separate response based on the channel of interaction.
Say I have three response builder .js files (webchat_response.js, Skype_response.js & Cortana_response.js)
How do I trigger those response based on the channel of interaction. Something like..
if(message.channel=='webchat'){ // trigger responses from webchat_response.js }
regards,
ram

Content of response builder files;
module.exports = function(controller) {     controller.hears('qq', 'message_received', function(bot, message) {         bot.startConversation(message, function(err, convo) {             convo.say('Lets get to know each other a little bit!')             convo.ask('Which is more offensive? Book burning or flag burning?', function(res, convo) {                 convo.next()                 convo.ask('How often do you keep your promises?', function(res, convo) {                     convo.next()                     convo.ask('Which is bigger? The Sun or the Earth?', function(res, convo) {                         convo.say('Thank you, that is all for now')                         convo.next()                     })                 })             })         })     }) }


Remove debugger statement in RollbarController

I develop other VS Code extensions but have the CMake Tools extension installed. This debugger; statement tends to fire whenever I exit my VS Code instance under test, which steals focus back to the VS Code instance running the debugger even though it's not my code.

Is there any other debugger; statements?

I'm not doing anything fancy. I'm able to repro when debugging the hello world VS Code extension and opening a folder with a single empty CMakeLists.txt, waiting for status bar items to appear, and then closing the instance under test.

Replacing this PR with #2415.  More details are in the PR description.

Replacing this PR with #2415.  More details are in the PR description.


Uncaught Error: not opened

[Enter steps to reproduce below:]

...
...

Atom Version: 1.7.2
System: Mac OS X 10.10.5
Thrown From: Ensime package, v0.38.1
Stack Trace
Uncaught Error: not opened
At /Users/atubionoya/.atom/packages/Ensime/node_modules/ensime-client/node_modules/ws/lib/WebSocket.js:219

Error: not opened
    at WebSocket.send (/Users/atubionoya/.atom/packages/Ensime/node_modules/ensime-client/node_modules/ws/lib/WebSocket.js:219:16)
    at postString (/Users/atubionoya/.atom/packages/Ensime/node_modules/ensime-client/lib/client.js:84:26)
    at post (/Users/atubionoya/.atom/packages/Ensime/node_modules/ensime-client/lib/client.js:87:16)
    at Object.getSymbolAtPoint (/Users/atubionoya/.atom/packages/Ensime/node_modules/ensime-client/lib/client.js:119:16)
    at ShowTypes.showExpressionType (/Users/atubionoya/.atom/packages/ensime/lib/features/show-types.coffee:41:13)
    at /Users/atubionoya/.atom/packages/ensime/lib/features/show-types.coffee:20:10

Commands
     -1:04.8.0 core:move-up (atom-text-editor.editor.is-focused)
  2x -1:04.4.0 editor:newline (atom-text-editor.editor.is-focused)
     -1:00.8.0 core:move-up (atom-text-editor.editor.is-focused)
     -1:00.4.0 editor:move-to-end-of-screen-line (atom-text-editor.editor.is-focused)
     -1:00.1.0 editor:select-to-first-character-of-line (atom-text-editor.editor.is-focused)
     -0:59.8.0 core:backspace (atom-text-editor.editor.is-focused)
     -0:59.1.0 core:move-down (atom-text-editor.editor.is-focused)
  2x -0:58.8.0 core:backspace (atom-text-editor.editor.is-focused)
     -0:50.8.0 command-palette:toggle (div.ensime-log-view.scroll-view.native-key-bindings)
     -0:49.3.0 core:move-down (atom-text-editor.editor.mini.is-focused)
     -0:48.8.0 core:confirm (atom-text-editor.editor.mini.is-focused)
     -0:48.7.0 ensime:start (div.ensime-log-view.scroll-view.native-key-bindings)
     -0:40.3.0 core:close (div.ensime-log-view.scroll-view.native-key-bindings)
  4x -0:09.9.0 core:move-down (atom-text-editor.editor.is-focused)
     -0:06.8.0 editor:newline (atom-text-editor.editor.is-focused)
  4x -0:05.9.0 core:undo (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "scala-worksheet"
    ]
  },
  "Ensime": {
    "runServersDetached": true,
    "sbtExec": "/usr/local/Cellar/sbt/0.13.11/bin/sbt"
  }
}

Installed Packages
# User
Ensime, v0.38.1
browser-plus, v0.0.60
hyperclick, v0.0.36
intentions, v1.0.5
language-scala, v1.1.6
linter, v1.11.4

# Dev
No dev packages

Sorry about the missing steps
I have fresh new installation of atom and the package ensime.
I did sbt gen-ensime, it seems fine, but once I do Ensime: start I keep getting this errors every second or so

I am closing this one, since the last upgraded version worked.

getting the same error on ubuntu 14.04.03 LTS
atom version 1.8.0
ensime 0.38.1

I also just got this, it happened randomly as well

I did same on other machine it works and did not work my machine.

I've just started getting this after running atom/ensime for about a week with no issues on a new mac
Weirdly, I can't think of anything that might have caused this

Had same issue, removing my .ensime_cache appeared to solve the issue...


Allow to disable mouse wheel switch in scene tabs

Editor allows to switch scene tabs with mouse wheel since #47700
However, just like with #65302, 100% of the times I used this functionality it was by accident (and it happens surprisingly often).
This PR adds a interface/scene_tabs/change_tabs_with_mouse_wheel editor setting that disables this function.

From the RC discussion, I'd just make it activate with a modifier key, like Ctrl, but otherwise leave it on.

Modifier would make this much less discoverable.

Modifier would make this much less discoverable.

I don't think the fact that you can use a mouse wheel over the tabbar is discoverable in the first place, unless by accident. If it's not rebindable, then the setting makes it hostile towards user, i.e. "you either use it like this, or you can't use it at all".

I wouldn't require using a modifier to use the mouse wheel either, as the behavior then becomes inconsistent with other apps that support this (patched Chromium, KDE apps, etc).

Should we make the setting disabled by default then? I don't see much demand for this feature, there was no request referenced in the original PR, and the chance to trigger it accidentally is evidently high.
My suggestion for the modifier key is to remove the accidental element without disabling the feature. If we don't want that solution, then disabling it by default is the only option.

Should we make the setting disabled by default then?

Settings that are disabled by default have poor discoverability – they might as well not exist :slightly_smiling_face:

I changed the PR to remove the feature entirely instead. It technically reverts #47700, but clean revert was impossible due to conflicts, so not sure if it counts 🤔

Thanks!


Flexible pricing: allow searching for users with any status

As a customer support rep, I'd like to be able to find a user's financial assistance request without having to know their current status.
If I do a search without choosing a status at all, this works.
But once I choose a status, I can't remove it. From then on I'm limited to searches filtered by status.

This works OK, as long as the admin doesn't choose a status. One they have chosen a status, there's no way to clear it, except for reloading the page.


Fix bug in signature creation

Purpose
$Subject
Fixes: https://github.com/ballerina-platform/ballerina-extended-library/issues/575

@kalaiyarasiganeshalingam It would be great if you can push only the fix related changes separately (at least as a separate commit) instead of sending along with all the formatting changes, which will make it easier to isolate the fix.


https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/matchAll

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

When run the first example in the page, the message error is showing: Error: str.matchAll is not a function.

It works fine for me; are you using Nightly or Beta Firefox, or release Firefox? matchAll() is really new (was only implemented in Fx67) so your browser might not be new enough.

You're right, my Firefox version is 66.0.3 and need be updated. Thanks for
answering me.
Em seg, 29 de abr de 2019 às 05:32, Chris Mills<EMAIL_ADDRESS>escreveu:

It works fine for me; are you using Nightly or Beta Firefox, or release
Firefox? matchAll() is really new (was only implemented in Fx67) so your
browser might not be new enough.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mdn/sprints/issues/1413#issuecomment-487492016, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACR3YNW4FEYBG3XLSUUXAF3PS2W75ANCNFSM4HI5UFSA
.

You're welcome.


Checkpoints list

This PR adds the checkpoints list to the proof of stake consensus state. It does not update the chain selection rule to use the checkpoints. The design is as follows.

There is a number of desired checkpoints per year, currently set to 12.
This implies that we want to take a checkpoint every W slots, breaking time up into "checkpoint windows" each of W slots.
We select the first block in each checkpoint window to go into the checkpoints list.

In terms of implementation, I did the following
Each state has a bit in it which says whether it has an ancestor in the same checkpoint window. If this bit is false, the state-hash of that state should get consed onto the checkpoints list as soon as the chain gets extended.

ping @nholland94


Errors on package deletion still removes the package resource

Repro:

Install a package
Delete a CRD manually
Delete the package

Expected:
The package deletion fails, the package resource (as well as the resources created by the package) remains (we'd need a way to force it)
Actual:
The package is deleted but the resources created by the package remains.

closing as dup


improvement: the folder deletion

resolves #6487

bring back delete confirmation
move delete outside for faster access

https://github.com/Kong/insomnia/assets/7845001/3f37fa1b-3c3b-4e44-bdcc-76d56536460e

How about only counting as "confirmation" if the time difference between two clicks are greater than an interval, say 0.5 seconds? This saves users from an unintended double-click. An animation can be added to make the behaviour doesn't look weird.
https://github.com/Kong/insomnia/assets/14835950/f5562b37-0a7a-4864-8a70-601fbed79bc7

Closing as this is already implemented


QEMU: Add support to qpc script for multiple os

ref https://github.com/AdoptOpenJDK/openjdk-infrastructure/issues/1538
I will be creating more qemu disk images, which will involve different OSs for already supported architectures.
Im currently adding a ubuntu 18 on arm64 qemu disk image onto infra-softlayer-ubuntu1804-x64-1. To accommodate the changes I have made to the script, I will rename the xz compressed disk images to support the OS.ARCHITECTURE.dsk.xz style.

@Willsparker Do you know the OS for the riscv disk image? All I get is Debian from
10:37:04     "msg": [
10:37:04         "inventory_hostname: localhost ",
10:37:04         "ansible_hostname: debian",
10:37:04         "ansible_fqdn: debian",
10:37:04         "ansible_user: linux",
10:37:04         "ansible_default_ipv4.address: <IP_ADDRESS>",
10:37:04         "ansible_os_family: Debian ",
10:37:04         "ansible_distribution: Debian ",
10:37:04         "ansible_distribution_major_version: unstable ",
10:37:04         "ansible_architecture: riscv64 ",
10:37:04         "ansible_processor_vcpus: 1 ",
10:37:04         "ansible_processor_cores: 1 ",
10:37:04         "Domain: adoptopenjdk.net ",
10:37:04         "Jenkins_Username: jenkins ",
10:37:04         "Superuser_Account: Enabled",
10:37:04         "Vendor_File: Disabled",
10:37:04         "Nagios_Plugins: Enabled ",
10:37:04         "Nagios_Monitoring: Enabled",
10:37:04         "Nagios_Master_IP: <IP_ADDRESS>"
10:37:04     ]

but no version number

Nevermind, from the qemu docs I see that its Debian Buster

@Haroon-Khel I believe it's Debian 'bullseye', which will be the one after buster, AFAIK

@Willsparker Ive put the RISCV image's distro down as debian 11, according to https://wiki.debian.org/DebianBullseye, do you object?

At https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/ Im running a QPC job with modified parameters to allow the inclusion of an OS parameter (it looks quite ugly so I will be looking for alternatives)

New qpc job running at https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/99/
Looks promising. All disk images boot and have the playbook run on them, except for ubuntu18.aarch64. This image worked fine when ran on infra-softlayer-ubuntu1804-x64-1 on its own, but has trouble when run as part of the jenkins job.

@Willsparker fyi the riscv job has failed twice at
12:48:39 TASK [Common : Add the openjdk repository to apt for openjdk7] *****************
12:48:51 fatal: [localhost]: FAILED! => {"changed": false, "cmd": "apt-key export DA1A4A13543B466853BAF164EB9B1D8886F44E2A", "msg": "E: gnupg, gnupg2 and gnupg1 do not seem to be installed, but one of them is required for this operation", "rc": 255, "stderr": "E: gnupg, gnupg2 and gnupg1 do not seem to be installed, but one of them is required for this operation\n", "stderr_lines": ["E: gnupg, gnupg2 and gnupg1 do not seem to be installed, but one of them is required for this operation"], "stdout": "", "stdout_lines": []}

ubuntu18.aarch64 disk wasnt booting on the jenkins job due to the jenkins user not permitted to access the image; fixed now.
Also added a task to Common/tasks/Ubuntu.yml which installs gnupg2 on aarch64; the playbook was getting caught up on TASK [Common : Add the openjdk repository to apt], which failed due to gnupg2 not being on the machine. This same fix may likely fix the riscv failure mentioned above

Latest qpc job running at https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/100/

@Haroon-Khel The RISC-V playbook is yet to be merged (the fixes are at #1483).

@Haroon-Khel I checked to see why the aarch64 U18 platform kept failing in QPC, and I'm getting this error message when running the qemu-system-aarch64 command manually:
WARNING: Image format was not specified for '/home/vagrant4/U18_aaarch64_test/openjdk-infrastructure/ansible/pbTestScripts/qemu_pbCheck/UBUNTU18.AARCH64.dsk' and probing guessed raw.
         Automatically detecting the format is dangerous for raw images, write operations on block 0 will be restricted.
         Specify the 'raw' format explicitly to remove the restrictions.
qemu-system-aarch64: Initialization of device cfi.pflash01 failed: Could not reopen file: Permission denied

When you specify the format to be raw, the first error message goes away, but there is still the permission denied error.

See that's odd, because I managed to get it working on that machine with the jenkins user by changing the owner of the flash1.img and QEMU_EFI-flash.img files in /qemu_base_images/arm64_tools/ to the jenkins user

Now im getting a different permission error when booting the disk up as the jenkins user
qemu-system-aarch64: -device virtio-blk,drive=drive0,bootindex=0: Could not reopen file: Permission denied

Ok, ive got the qpc script to work as the jenkins user. I think this is a case of the machine needs to be disconnected and reconnected from jenkins in order to update its permissions. @sxa what do you think?

Disconnecting/reconnecting the machine from jenkins did not work. The job failed yet again at https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/ARCHITECTURE=aarch64,OS=ubuntu18,label=vagrant/109/console

@Willsparker despite the ubuntu18.arm64 job not working, can this still be merged? The changes to the qpc script are still vaild, since the other jobs run without error. Im eager to get this in because the qpc jenkins job is configured to use my fork and branch, which is holding up testing other changes to the qpc script, such as https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1566

Changes the qpc job config back to original in order to test the pr https://github.com/AdoptOpenJDK/openjdk-infrastructure/pull/1566

It works! https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/ARCHITECTURE=aarch64,OS=ubuntu18,label=vagrant/120/console
Confusingly, though the jenkins user had read and write access and ownership of the files in /qemu_base_images/arm64_tools/, changing the permissions to -rw-rw-rw- seemed to do the trick.
@sxa can this finally be merged?

Hmmm I'm not comfortable with anything that seems to require world-writable files around

Ive made the permissions of the contents of /qemu_base_images/arm64_tools/ to -rw-rw-r-- and the job fails as before, https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/122/ARCHITECTURE=aarch64,OS=ubuntu18,label=vagrant/console.
In the logs, it seems that the jenkins node runs the job in the /home/vagrantX directory. and not in /home/jenkins as I thought. This may be why we are getting a permissions error.
How would you feel if I added the vagrantX users into the jenkins user group, on that machine? I think this may solve the problem

Ive moved the disk images from /qemu_base_images to /home/jenkins/qemu_base_images. Instead of copying over the compressed disk image, the script will copy the contents of a folder nameed OS.ARCHITECTURE, which will include the compressed disk image and any other files used in the booting of the vm, into the working directory, and then boot the image.
As of https://ci.adoptopenjdk.net/job/QEMUPlaybookCheck/136/console, this seems to work!

Given that it's now doing the sensible thing and copying everything from the read-only repository and not writing directly into /qemu_base_images I'm not sure it's too necessary to add that into the documentation (which is unlike me to say something like that !)


Ability to get line content before a newline is sent?

Hi - Is there any way to get line content before a newline is sent?

You could conceivably (ab)use the hints or completions mechanisms, which pass the current buffer each time it is changed.


Selected items are cleared when filtering KubeObjectItemList

https://user-images.githubusercontent.com/40840436/150874879-7d8e7fb1-e7b6-446a-989b-7a022edc8f4b.mov
This is a regression and may be related to https://github.com/lensapp/lens/pull/4653#issuecomment-1012457220
Happens on master

This has been fixed on master, noticed while working on #4798


Revert "Add ContractResolver to JsonSerialization, so that non-option…

…al fields are required (#852)"
This reverts commit 86581a2c2171c55db8902350631a2ec5fa4c2afd.
This change needs to be re-submitted after refactoring the compiler config to match the optional fields that are currently documented.

@marina-p hello. Could you please give advice whom to mention to make my pr reviewed? pr #847


postgresql: where statement in  sql attribute can not be execute. If the sql statement has no where condition,It works well. I am upset with it. Thank you!

#!/bin/sh
DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" && pwd )"
bin=${DIR}/../bin
lib=${DIR}/../lib
echo '{
"type" : "jdbc",
"jdbc" : {
"url" : "jdbc:postgresql://<IP_ADDRESS>:5432/cmdbuild",
"user":"dbuser",
"password":"123456",
"sql":"select v."Code" as _id,v."Code" as "ComputerName" from "VirtualServer" v where v."Status"='A' and v."DeviceStatus"=990",
"interval":"5s",
"index_settings" : {
"index" : {
"number_of_shards" : 1,
"number_of_replicas":1
}
},
"index":"profiles",
"type":"Server"
}
}
' | java
-cp "${lib}/*"
-Dlog4j.configurationFile=file://localhost/C:/elasticsearch-jdbc-<IP_ADDRESS>-dist/elasticsearch-jdbc-<IP_ADDRESS>/bin/log4j2.xml
org.xbib.tools.Runner
org.xbib.tools.JDBCImporter

By the way,I copy the sql statement to postgresql,It works well too.

Hi
Try this workaround: make a view on your SQL and query that view from the in the "sql"-parameter above.

Thank you,akobler,I have solved the question,your suggetion is right, But how do you solved the problem like the URL shows?TKS
https://github.com/jprante/elasticsearch-jdbc/issues/511

TanQ so much @akobler. Solved my issue.


Error in .local(.Object, ...) : Failed to obtain 'hguid' cookie even with local .ucscCache file

Error occurred:
library(Gviz)

saveRDS(Gviz:::.ucscCache, file=sprintf("ucscCache.%s.rds", "hg19"))
.ucscCache <- readRDS(sprintf("ucscCache.%s.rds", "hg19"))
assignInNamespace(".ucscCache", .ucscCache, ns="Gviz")

from <- 9217084
to <- 10698980

knownGenes <-UcscTrack(genome = "hg19", chromosome = "chr4",
                       track = "knownGene", from = from, to = to,
                       trackType = "GeneRegionTrack",
                       rstarts = "exonStarts", rends = "exonEnds",
                       gene = "name", symbol = "name",
                       transcript = "name", strand = "strand",
                       fill = "#8282d2", name = "UCSC Genes")

Error in .local(.Object, ...) : Failed to obtain 'hguid' cookie

My session info:
R version 4.3.1 (2023-06-16)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Ubuntu 20.04.6 LTS

Matrix products: default
BLAS:   /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.9.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.9.0

...

attached base packages:
[1] grid      stats4    stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] Gviz_1.44.1          GenomicRanges_1.52.0 GenomeInfoDb_1.36.3  IRanges_2.34.1       S4Vectors_0.38.1     BiocGenerics_0.46.0

loaded via a namespace (and not attached):
  [1] DBI_1.1.3                   bitops_1.0-7                deldir_1.0-9                gridExtra_2.3
  [5] biomaRt_2.56.1              rlang_1.1.1                 magrittr_2.0.3              biovizBase_1.48.0
  [9] matrixStats_1.0.0           compiler_4.3.1              RSQLite_2.3.1               GenomicFeatures_1.52.2
 [13] png_0.1-8                   vctrs_0.6.3                 ProtGenerics_1.32.0         stringr_1.5.0
 [17] pkgconfig_2.0.3             crayon_1.5.2                fastmap_1.1.1               backports_1.4.1
 [21] dbplyr_2.3.3                XVector_0.40.0              utf8_1.2.3                  Rsamtools_2.16.0
 [25] rmarkdown_2.24              bit_4.0.5                   xfun_0.40                   zlibbioc_1.46.0
 [29] cachem_1.0.8                progress_1.2.2              blob_1.2.4                  DelayedArray_0.26.7
 [33] BiocParallel_1.34.2         jpeg_0.1-10                 parallel_4.3.1              prettyunits_1.1.1
 [37] cluster_2.1.4               VariantAnnotation_1.46.0    R6_2.5.1                    stringi_1.7.12
 [41] RColorBrewer_1.1-3          rtracklayer_1.60.1          rpart_4.1.19                Rcpp_1.0.11
 [45] SummarizedExperiment_1.30.2 knitr_1.44                  base64enc_0.1-3             Matrix_1.6-0
 [49] nnet_7.3-19                 tidyselect_1.2.0            rstudioapi_0.15.0           dichromat_2.0-0.1
 [53] abind_1.4-5                 yaml_2.3.7                  codetools_0.2-19            curl_5.0.2
 [57] lattice_0.21-8              tibble_3.2.1                Biobase_2.60.0              KEGGREST_1.40.0
 [61] evaluate_0.21               foreign_0.8-82              BiocFileCache_2.8.0         xml2_1.3.5
 [65] Biostrings_2.68.1           pillar_1.9.0                filelock_1.0.2              MatrixGenerics_1.12.3
 [69] checkmate_2.2.0             generics_0.1.3              RCurl_1.98-1.12             ensembldb_2.24.0
 [73] hms_1.1.3                   ggplot2_3.4.3               munsell_0.5.0               scales_1.2.1
 [77] glue_1.6.2                  Hmisc_5.1-0                 lazyeval_0.2.2              tools_4.3.1
 [81] interp_1.1-4                BiocIO_1.10.0               data.table_1.14.8           BSgenome_1.68.0
 [85] GenomicAlignments_1.36.0    XML_3.99-0.14               latticeExtra_0.6-30         AnnotationDbi_1.62.2
 [89] colorspace_2.1-0            GenomeInfoDbData_1.2.10     htmlTable_2.4.1             restfulr_0.0.15
 [93] Formula_1.2-5               cli_3.6.1                   rappdirs_0.3.3              gwasglue_<IP_ADDRESS>00
 [97] fansi_1.0.4                 S4Arrays_1.0.6              dplyr_1.1.3                 AnnotationFilter_1.24.0
[101] gtable_0.3.4                digest_0.6.33               rjson_0.2.21                htmlwidgets_1.6.2
[105] memoise_2.0.1               htmltools_0.5.6             lifecycle_1.0.3             httr_1.4.7
[109] bit64_4.0.5

Same problem here, Yesterday was working perfectly fine. I reboot the computer and now IdeogramTrack() and browserSession() functions, at least are throwing the error:
Error in .local(.Object, ...) : Failed to obtain 'hguid' cookie

Related to this, when I try to use IdeogramTrack get this error:
Error in (function (classes, fdef, mtable)  :
unable to find an inherited method for function 'displayPars<-' for signature '"numeric", "list"'
This issue started two days ago

Hi, I believe it is related to the issue of rtrackalyer package as they are suing http instead of https: https://github.com/lawremi/rtracklayer/pull/92

I updated the rtracklayer package, but the issue still persists. I also tried using
options(Gviz.ucscUrl="http://genome-euro.ucsc.edu/cgi-bin/")
but it didn't solve the problem.

Anyone have any fixes for this? Getting the same error and I can't find a workaround.

For me the error disappeared the same way it appeared, 2 days later it started to work fine again 🤷

Same error for me around. Please update with fix when available thanks!

I have same problem.

have the same problem

I'm also having the same issue.

The same problem. Please update with fix when available.
Thanks!

This has been fixed upstream in rtracklayer in both release (Bioc 3.18): 1.62.0 and devel (Bioc 3.19) version: 1.63. 0


Remove StartOptions

Currently the only option is Sampler. We should remove the option struct and just pass sampler directly to the NewSpan methods (not the convenience methods that presently don't take StartSpanOptions).

We are going to keep StartOptions and use it to add #525


Added accout api key limit info block to api_key_permissions.md

Description of the change: Added accout api key limit info block to
Reason for the change: in all my other pull requests I said it was because Kaylyn requested it but I now see that it was Carley
Link to original source: https://sendgrid.com/docs/Classroom/Basics/API/api_key_permissions.html
@ksigler7

Hello @debri174,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


[Cosmetic Bug] Connected texture not working on a single line Launch Pad (1.11.2-1.2.2)

As say in the title if we put only one line of Launch Pad there is no connected texture.

Is this still a issue? Please comment if it is and i will reopen if if you are unable? We are closing old issues to see what is still relevant and needs to be fixed.

Can confirm


fix for 'No such namespace: nrepl.middleware.print'

(find-var 'nrepl.middleware.print/*print-fn*) causes an Exception "No such namespace: nrepl.middleware.print" when the namespace is not found

I just pushed again with your fix applied – thanks!

thank you too :)


Sync not working

I created a fork, moved 4 commits back and made a force push.
Not I created the action, which run successfully.
Sadly the master branch did not change.
(I put the MartinX3 branch with my action as default branch).
Repo: https://github.com/MartinX3/github-do-not-ban-us

Sorry, my bad.
I wrote my own repo into the "source repo" field.
Is there any way to auto fetch the fork of the project? @wei ?


[🐞] 'Client-only' code is imported in entry.express

Which component is affected?
Qwik Optimizer (rust)
Describe the bug
In one of our components, we are showing a map to the user.  The functionality of the map used to be wrapped in a useClientEffect$ .
 useClientEffect$(async ({ track }) => {
    track(() => props.mapVisible);
    if (props.mapVisible) {
     // do some map related stuff
    }
  });

However we upgrade from:
<EMAIL_ADDRESS>"0.16.2",
<EMAIL_ADDRESS>"0.0.128",

to
<EMAIL_ADDRESS>"0.21.0",
<EMAIL_ADDRESS>"0.5.3",

This forced us to replace useClientEffect$ by useVisibleTask$.
 useVisibleTask$(async ({ track }) => {
    track(() => props.mapVisible);
    if (props.mapVisible) {
     // do some map related stuff
    }
  });

Actual behaviour:
After building the application we see that the import of leaflet is added to entry.express.mjs. This is causing the application to crash at startup:
ReferenceError: window is not defined
    at eval (file:///home/projects/qwik-starter-vtm9gw/node_modules/leaflet/dist/leaflet-src.js:230:19)
    at eval (file:///home/projects/qwik-starter-vtm9gw/node_modules/leaflet/dist/leaflet-src.js:7:66)
    at Object.eval (file:///home/projects/qwik-starter-vtm9gw/node_modules/leaflet/dist/leaflet-src.js:10:3)
    at Object.function (https://qwikstartervtm9gw-mylu.w-credentialless.staticblitz.com/blitz.6df6cd42ba0b81666017ec43c5c96e159ca2f943.js:15:141457)

We where able to work around this by using a lazy loaded import const { getMapUtils : _getMapUtils} = await import('./add-features');
Expected Behaviour
This import should not be bundled in the entry.express.mjs' if the code is wrapped in a useVisibleTask$` hook
Reproduction
https://stackblitz.com/edit/qwik-starter-vtm9gw
Steps to reproduce
Open the url.
System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 16.14.2 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 7.17.0 - /usr/local/bin/npm
  npmPackages:
    @builder.io/qwik: 0.21.0 => 0.21.0
    @builder.io/qwik-city: 0.5.3 => 0.5.3
    undici: 5.20.0 => 5.20.0
    vite: 4.1.4 => 4.1.4

Additional Information
No response

Why is the lib files outside the src directory?
The issue is not related to useClientEffect or useVisibleTask, now both behave the same, the problem started with a change to make serverQrl work, which involved emiting a side effect import to local files referenced.
In this case, it fails because:

it's a relative import, but outside the src directory
the dependency has side effects

A work around you can do is to:

move the function inside the src directory
use isBrowser to wrap the usage of leaflet

Need to fix how to solve this issue

Some mitigations:

resolve relative imports, to figure out if they point to a file outside the src directory and treat them like external imports
mark external deps has sideEffect free (could be problematic)

@manucorporat

A work around you can do is to:

move the function inside the src directory
use isBrowser to wrap the usage of leaflet

Indeed the first solution seems to work. Thx for the hint. However, we are building in the context of an NX monorepo and this code is situated in a lib (obviously outside of the src folder).


Provide a Tutorial Guide showing how to "ping a collector install" with a dummy signal (e.g., metric) without the use of a language SDK or other component

The current Getting Started guide includes a demo that use Zipkin, Jaeger and Prometheus. I am looking for a more simpler guide. Something that would show how to send a metric(s) using a utility like CURL and seeing the metric being recorded in a local file. This would help confirm that the installation of OTEL Collector is working normally.

I think something like https://github.com/equinix-labs/otel-cli might be useful for this under the collector quick start as it can send gRPC (which is, presumably, what most people using a collector + OTLP will be using).

Yes otel-cli looks like a great and capable OTEL utility to quickly and simple send metric data to OTEL, however, I'm looking something even more simpler, without GO and Jaeger. I am looking to confirm that OTEL is installed and working correctly.
Is there a curl command that can be used, with dummy random metric data, to send to the OTEL collector listener and confirm the receipt of data to a local file? This will confirm that OTEL is up and running and accepting connections.

I'm not sure if the metric receiver supports measurements in formats other than gRPC at this point? Might be a better question for the collector repository. You can confirm receipt of data pretty easily with the logging exporter on the collector though.

Thank You! I'm just starting my journey with Opentelemetry . I am wondering if a simple curl command can be formatted using Opencensus format to send a dummy metric to OTEL? If there was a guide on how to do this it would be immensely helpful.

A bit wordy, but I renamed this issue to be a little more clear about the intent. We do offer what you're asking for in language docs today (e.g., the python docs show how to set up a local collector and send data to it), but there's nothing that's purely collector-focused and doesn't involve any other components.

Hi @cartermp ,
we use telemetrygen from opentelemetry collector contrib Repo.
It works for us.
Regards, Sascha

@theletterf This may be addressed with an updated getting started guide for the collector as well (https://github.com/open-telemetry/opentelemetry.io/issues/1695)

fixed via https://github.com/open-telemetry/opentelemetry.io/issues/1695


Document default/overridden values for extras on discoverable extractor and loader pages

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2513
Originally created by @DouweM on 2021-01-12 23:11:17

Some discoverable extractors have a default value for the select extra: https://gitlab.com/meltano/meltano/-/blob/84aaa6eee51efcda492862ab8ebcfec72bf0f007/src/meltano/core/bundle/discovery.yml#L18. Since the user will need to be aware of this when overriding the value in their own meltano.yml, we should explicitly document the value.
Some discoverable loaders have default values for dialect and target_schema extras: https://gitlab.com/meltano/meltano/-/blob/84aaa6eee51efcda492862ab8ebcfec72bf0f007/src/meltano/core/bundle/discovery.yml#L1047-1048. Since these values may affect the plugin's behavior and interaction with other plugins, we should explicitly document these default values.

View 5 previous comments from the original issue on GitLab

We can add this in a Hub update. Closing for now.


Missing prop predefinedPlacesAlwaysVisible

I'm using the predefinedPlacesAlwaysVisible support in my project, but upgrading from v1.4.1 to 1.6.0 I received an error saying that the above prop is missing and I believe it was forgotten to add it to the types file, right?
I believe that the correct place to put this type is below the following line:
https://github.com/FaridSafi/react-native-google-places-autocomplete/blob/645ec590e1dd3c6e5da8568b2f566e91970993bc/GooglePlacesAutocomplete.d.ts#L359

I'm closing this issue because it was added in PR #555

Just re-opening this until a fix is merged in.


Reuse timers with sync.Pool

Here is the top out for alloc space of heap profile:
      flat  flat%   sum%        cum   cum%
 5845.32MB 42.91% 42.91%  5845.32MB 42.91%  github.com/sasha-s/go-deadlock.callers (inline)
 2851.22MB 20.93% 63.84%  2851.22MB 20.93%  time.NewTimer
 2243.68MB 16.47% 80.32%  8089.69MB 59.39%  github.com/sasha-s/go-deadlock.lock

I haven't figured out how to optimize callers. But for time.NewTimer the change seemed to be quite simple. So I made it.

This change is

Reviewed 1 of 2 files at r1, all commit messages.
Reviewable status: 1 of 2 files reviewed, 1 unresolved discussion (waiting on @millfort)

deadlock.go line 207 at r1 (raw file):

func releaseTimer(t *time.Timer) {
	if !t.Stop() {
		<-t.C

Would for range t.C {}
be cleaner here?

Stop does not close the channel


Add pip syncing

Sync pyproject with pip freeze via bash script

Ensure it makes a back-up of pyproject.toml before edits, but that it's excluded (pyproject.toml.bak) in the .gitignore

Nevermind, I didn't realise poetry add and poetry remove exist - include this in CONTRIBUTING.md and other docs. Should regularly use poetry update, though.


split grafana loading phase into its own script

There are deployments in which, due to firewall rules or other issues,
once has issues pulling docker images. Or alternatively, an user may
already have a prometheus/grafana installation that they may want to
reuse.
For situations like that, it will be useful to extract the container
setup phase from the data loading phase. Users can then call just the
latter.
Signed-off-by: Glauber Costa<EMAIL_ADDRESS>

Looks good
@glommer can you send PR for the readme, and/or update the Wiki with instructions?
Users will not guess it there otherwise.


Add plugin for lighthouse-audit-service

Hey, I just made a Pull Request!
This PR implements a plugin which can act as a frontend for the newly open-sourced lighthouse-audit-service. This frontend enables you to view all previously run audits, as well as kick off new audits.

:heavy_check_mark: Checklist

[x] All tests are passing yarn test
[x] Screenshots attached (for UI changes)
[x] Relevant documentation updated
[x] Prettier run on changed files
[x] Tests added for new functionality

🕵 Things missing at this point

[x] look into a more elegant solution for those rollup hacks
[x] find and replace the hacky inline styles I added with classes and withStyles
[x] write tests for the plugin
[x] write a blog post on the Lighthouse plugin and add the link to the intro
~create nice "no audits yet exist" (e.g. empty) state~ #514
~create a better "lighthouse-audit-service is unreachable" state which helps guide users to setting up the service~ #514
~create a more general purpose plugin intro which directly pulls from README sections to render content~ #515

Sorry but this needs updating now that InfoCard is converted to TS.

Could we do the testUtil separation in a separate PR? feeling that needs some more thought and it would be good to get the plugin merged. We def want it in some form though.

Hey! Regarding this PR and https://github.com/spotify/backstage/pull/487, you'll notice on my next commit that I'm pretty dependent on leveraging wrapInThemedApp (whatever it's called! I'm on my phone) to make the tests work well, which is why this PR is rebased on it. If there's a simpler way to get access to it, like exposing the testUtils from core, that you'd prefer to get this in, fine by me. But I didn't want to land this without pretty complete tests, as I assume we're about to iterate both on the plugin and on the larger application, and I think having a test baseline is critical to avoid breaking this given that.
As for theme - circular dependency broke lerna's brain, it turns out that on a fresh install, it couldn't figure out which order to run yarn build. so, theme has to happen at the same time as test utils if it is to happen.

@Rugvip @freben @stefanalund ok, this PR is ready to review! I'm gonna split things I didn't get done into feature tickets and finish writing my blog post now. Thanks for the intermittent reviews and the patience!

:shipit:

m e r g i n g !

OMG THIS IS AWESOME THANK YOU VERY MUCH


🍒 Cherry pick PR #4593 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/4593 to staging 🍒

🚀 Cherry-picked to staging by @roryabraham in version: 1.0.85-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @isagoico in version: 1.0.85-9 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Add support for mounting a directory with additional config files

Right now, a user has to overwrite the whole httpd.conf for customizations (e.g. for named vhosts). This requires the user to docker cp the httpd.conf out of a container and keeping it up to date with container updates, or to create his/her own Dockerfile extending the image. Both ways carry the risk of the image going stale as well as being extra effort.
This change adds support for mounting a directory containing *.conf files similar to the conf.d mechanism in Debian/Ubuntu (where the idea was actually taken from).
Example usage:
docker run -d \
-p 80:80 \
-v /srv/httpd/htdocs:/usr/local/apache2/htdocs:ro \
-v /srv/httpd/conf:/usr/local/apache2/conf/optional:ro \
httpd:2.4

While this functionality is interesting, I'd really prefer not to deviate from upstream's officially supplied configuration if we can help it.  I think this should be reasonably simple to accomplish with a short Dockerfile (combined with something like automated builds to keep it up to date), something like:
FROM httpd:2.4
RUN mkdir -p "$HTTPD_PREFIX/conf/optional"
RUN echo 'IncludeOptional conf/optional/*.conf' >> "$HTTPD_PREFIX/conf/httpd.conf"

@tianon the approach with even a short Dockerfile is something I'd prefer to not have to set up. This is a maintenance burden upon users, as there's yet another layer of complexity (having to maintain a Jenkins/whatever automator, plus a Nexus or other repository). By incorporating this functionality I can simply have v2tec/watchtower watch for changes in the official image.

For the watchtower use case, you could supply a full httpd.conf or use a slightly more complex command in order to accomplish what you want without too much trouble, as in:
$ docker run ... httpd:2.4 sh -c 'mkdir -p "$HTTPD_PREFIX/conf/optional" && echo "IncludeOptional conf/optional/*.conf" >> "$HTTPD_PREFIX/conf/httpd.conf" && exec httpd-foreground'

I can see that this has been brought up a number of times.  I don't find the reasoning sufficient however.  I'm commenting here rather than opening a new issue or PR.

Adding the IncludeOptional to the (docker-only) is trivial and PRs have been supplied.
From a user perspective maintaining a docker file of any size is miles beyond image: httpd.  Especially when Docker Hub image retention policy goes into place.

I agree that it would be preferable for upstream to add something like this, but until such a time... most of us who pull the image are wondering why we have to COPY a full config file.  Right now there thousands (at least) copies of the base config floating around, likely not being updated, many with very few lines of additional configuration added.

I can see that this has been brought up a number of times.  I don't find the reasoning sufficient however.  I'm commenting here rather than opening a new issue or PR.

Adding the IncludeOptional to the (docker-only) is trivial and PRs have been supplied.
From a user perspective maintaining a docker file of any size is miles beyond image: httpd.  Especially when Docker Hub image retention policy goes into place.

I agree that it would be preferable for upstream to add something like this, but until such a time... most of us who pull the image are wondering why we have to COPY a full config file.  Right now there thousands (at least) copies of the base config floating around, likely not being updated, many with very few lines of additional configuration added.


Drone Makeflow GitHub Actions failing again due to Terraref BETYdb changes

The current behavior or issue
The GitHub Actions for testing BETYdb are failing
The steps taken to reproduce the behavior or issue, or specify a location where the steps were recorded
See https://github.com/AgPipeline/drone-makeflow/actions/runs/1734662039
Expected behavior
The Actions run successfully to completion
Screenshots
N/A
Add other supporting information that may be useful
N/A
Completion criteria
Able to test BETYdb successfully

https://github.com/AgPipeline/drone-makeflow/actions/runs/1742007338


Unable to get it to work

I've done some extensive table editing and no matter what I do, I cant seem to get it to work, even following the examples. Are there any specifics to this, does the integer for the chance to drop need to be formatted in a specific way if it's less than one? IE Chance = 0.10f or .1 or something else?
The indexes I've created all start at 0 and go from there, should I be starting them at a later number if the mob already has drops or something?

Post your config and lets take a look.
The chance should be something like
Chance = 1
Chance = 0.5
Chance = 1.0

[Greyling.0]
ItemName = Raspberry
AmountMin = 1
AmountMax = 2
Chance = 0.10
OnePerPlayer = false
LevelMultiplier = true
Enabled = true
[Greydwarf.0]
ItemName = Blueberry
AmountMin = 1
AmountMax = 2
Chance = 0.10
OnePerPlayer = false
LevelMultiplier = true
Enabled = true
[Greydwarf.1]
ItemName = Thistle
AmountMin = 1
AmountMax = 1
Chance = 0.10
OnePerPlayer = false
LevelMultiplier = true
Enabled = true
[Greydwarf_Shaman.0]
ItemName = Carrot
AmountMin = 1
AmountMax = 2
Chance = 0.25
OnePerPlayer = false
LevelMultiplier = true
Enabled = true

drop_that.tables.cfg.txt
That's my current table config file.

Ah, I think I found the issue.
Remove all the sections with "?[header]" and it should have an easier time loading. Several of your configs are already applied, but it has trouble with things like the Greyling due to following section.

Closing due to no follow-up


Fix issue with download-all-languages command

Codecov Report

Merging #638 into develop will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff            @@
##           develop     #638   +/-   ##
========================================
  Coverage    88.89%   88.89%
========================================
  Files           59       59
  Lines         3269     3269
  Branches       613      613
========================================
  Hits          2906     2906
  Misses         260      260
  Partials       103      103


setup.py makes restrictive assumptions about the location of libevent.*

What steps will reproduce the problem?
1. Install a system-wide libevent at /usr/local/lib/libevent.*
2. python setup.py install

What is the expected output? What do you see instead?
The expected output is a completed installation. We see, "couldn't find
libevent installation or build directory", instead.

What version of the product are you using? On what operating system?
pyevent-0.3 with python 2.4.4 on linux

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 8 Oct 2007 at 1:40

This patch was incorporated at some point.


Add CycloneDX Editor Validator Tool by Festo

We currently promote our tool to different German companies, which all want to use it. Therefore, we would like to have this included here, as e.g. the capability of providing a custom schema for validation is something not every tool is able to.

This is now live on the catalog https://sbom-catalog.openssf.org/
Thank you for the update!

@joshbressers Thanks for the fast response and merge!


Foundation 5.2: unbound variable $column-gutter when building standalone block-grid

I'm getting following error when trying to build block-grid:

Warning: bower_components/foundation/scss/foundation/components/block-grid:18: error: unbound variable $column-gutter

I have to include accordion in order to make the custom build work.

Just put settings SCSS file import in the bottom of global.scss. Works perfect for me


Initialize repository with commits from gateway-api

Replay commit history of gwctl folder from https://github.com/kubernetes-sigs/gateway-api.
Additional changes include:

Changing the package name from sigs.k8s.io/gateway-api/gwctl -> sigs.k8s.io/gwctl to allow the code to compile
Commits from https://github.com/kubernetes-sigs/gateway-api/pull/3244 to serve as the base of future changes.
Sample .gitignore file

/lgtm


Wires voiding energy from things

https://dl.dropboxusercontent.com/u/134976541/ShareX/2016-01/2016-01-24_16-46-43.mp4
Fluxed Redstone-<IP_ADDRESS>
The energy storage pictured is from neotech.

Same issue is not present when using BC energy pipes, so it must be fluxed redstone wires.

The power is being transferred into the wires, not a lot I can do without rewriting the powernet

One way to make the problem less ovious would be to reduce the amount of power each cable can store

It's not just being transferred, energy is getting voided. Wouldn't be an issue if the pipes only stored power, the issue is them losing it somewhere.

I've got an idea of where this might be coming from.

I think im going to rewrite the energy transfer code, it will be more fendily on the sevrer and fix issues like this.


Use masked_array to handle no data values

now the default is to load the data as a np.masked_array using rasterio's option
update the set_ndv and test_set_ndv accordingly

For now, both normal numpy arrays and masked arrays are allowed, but the default should probably be only masked_arrays which means:

updating data.setter to check type
check that all comparison to no data value are moved to using self.data.mask

Solves #90

Solves #89

looks good, and solves 2 issues! :+1:

looks good, and solves 2 issues! +1

Yes, and in the meantime I added at least 3 issues!