Endpoint hook for custom routing

We're making a hook that interprets things into our non-restful API.
The Sails endpoint:
/something/:id

passes the information that I need (model name, id, request method) through the controller, waterline, and the adapter to our hook.
But if I want to redirect for an endpoint like:
/something/:id/sub-somethings

I lose the sub-somethings information and cannot build my route.
Any ideas how to get that data?

It looks like that this might even be a waterline issue. It seems that the /something/:id/sub-somethings is being interpreted into: something.find(:id).populate('sub-somethings') instead of sub-somethings.where('something': :id). I'll move the issue to the waterline repo if I'm off base her, what do you think?

I agree, and in addition, if you feel like it, it might be a good idea to create a generic request method that simply exposes the underlying restify client. Care to send a PR?


There's lots of there things that are "static" - csv dumps, logisim files, and images. Might be worth giving this another go and looking for more docs about it.


Mixtral OutOfMemoryError with 2 GPUs

I'm trying to run Mixtral (Mixtral Hermes) with two 48GB GPUs but it seems that sglang server is not using my second GPU.
CUDA_VISIBLE_DEVICES="0,1" python -m sglang.launch_server --model-path /workspace/model --port 30000
errors out with
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Special tokens have been added in the vocabulary, make sure the associated word embeddings are fine-tuned or trained.
Process Process-1:
Traceback (most recent call last):
router init state: Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/manager.py", line 68, in start_router_process
    model_client = ModelRpcClient(server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 448, in __init__
    self.model_server.exposed_init_model(0, server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 54, in exposed_init_model
    self.model_runner = ModelRunner(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 229, in __init__
    self.load_model()
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 272, in load_model
    model = model_class(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 322, in __init__
    self.model = MixtralModel(config, linear_method)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 285, in __init__
    [
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 286, in <listcomp>
    MixtralDecoderLayer(config, i, linear_method=linear_method)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 238, in __init__
    self.block_sparse_moe = MixtralMoE(config=config, linear_method=linear_method)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 99, in __init__
    [
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 100, in <listcomp>
    MixtralMLP(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/models/mixtral.py", line 55, in __init__
    self.w2 = ReplicatedLinear(
  File "/usr/local/lib/python3.10/dist-packages/vllm/model_executor/layers/linear.py", line 108, in __init__
    self.linear_weights = self.linear_method.create_weights(
  File "/usr/local/lib/python3.10/dist-packages/vllm/model_executor/layers/linear.py", line 55, in create_weights
    weight = Parameter(torch.empty(output_size_per_partition,
  File "/usr/local/lib/python3.10/dist-packages/torch/utils/_device.py", line 77, in __torch_function__
    return func(*args, **kwargs)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 112.00 MiB. GPU 0 has a total capacty of 44.35 GiB of which 9.38 MiB is free. Process 2776126 has 44.33 GiB memory in use. Of the allocated memory 44.02 GiB is allocated by PyTorch, and 14.25 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_CUDA_ALLOC_CONF

detoken init state: init ok

I had the same error on Modal with 2 x 80GB GPUs.
Thanks for the support

You have to add arguments --tp-size 2 to indicate the server to run the tensor on 2 GPUs.

Ok thank you that seems to get the second GPU to work. However I still get an error :
python -m sglang.launch_server --tp-size 2 --model-path /workspace/model --port 30000
server started on [<IP_ADDRESS>]:10004
server started on [<IP_ADDRESS>]:10005
accepted ('<IP_ADDRESS>', 47232) with fd 28
welcome ('<IP_ADDRESS>', 47232)
accepted ('<IP_ADDRESS>', 48754) with fd 24
welcome ('<IP_ADDRESS>', 48754)
router init state: Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/manager.py", line 68, in start_router_process
    model_client = ModelRpcClient(server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 469, in __init__
    rets = [obtain(x) for x in executor.map(init_model, range(tp_size))]
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 469, in <listcomp>
    rets = [obtain(x) for x in executor.map(init_model, range(tp_size))]
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 621, in result_iterator
    yield _result_or_cancel(fs.pop())
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 319, in _result_or_cancel
    return fut.result(timeout)
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 458, in result
    return self.__get_result()
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 467, in init_model
    return self.model_servers[i].init_model(i, server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/netref.py", line 240, in __call__
    return syncreq(_self, consts.HANDLE_CALL, args, kwargs)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/netref.py", line 63, in syncreq
    return conn.sync_request(handler, proxy, *args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 718, in sync_request
    return _async_res.value
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/async_.py", line 108, in value
    raise self._obj
_get_exception_class.<locals>.Derived: Trying to create tensor with negative dimension -789: [-789]

========= Remote Traceback (1) =========
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 359, in _dispatch_request
    res = self._HANDLERS[handler](self, *args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 837, in _handle_call
    return obj(*args, **dict(kwargs))
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 54, in exposed_init_model
    self.model_runner = ModelRunner(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 230, in __init__
    self.init_memory_pool(total_gpu_memory)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 300, in init_memory_pool
    self.req_to_token_pool = ReqToTokenPool(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/memory_pool.py", line 11, in __init__
    self.mem_state = torch.ones((size,), dtype=torch.bool, device="cuda")
RuntimeError: Trying to create tensor with negative dimension -789: [-789]

detoken init state: init ok
goodbye ('<IP_ADDRESS>', 47232)
goodbye ('<IP_ADDRESS>', 48754)

I'm not sure if it's related to multi GPU or not

Ok thank you that seems to get the second GPU to work. However I still get an error :
python -m sglang.launch_server --tp-size 2 --model-path /workspace/model --port 30000
server started on [<IP_ADDRESS>]:10004
server started on [<IP_ADDRESS>]:10005
accepted ('<IP_ADDRESS>', 47232) with fd 28
welcome ('<IP_ADDRESS>', 47232)
accepted ('<IP_ADDRESS>', 48754) with fd 24
welcome ('<IP_ADDRESS>', 48754)
router init state: Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/manager.py", line 68, in start_router_process
    model_client = ModelRpcClient(server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 469, in __init__
    rets = [obtain(x) for x in executor.map(init_model, range(tp_size))]
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 469, in <listcomp>
    rets = [obtain(x) for x in executor.map(init_model, range(tp_size))]
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 621, in result_iterator
    yield _result_or_cancel(fs.pop())
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 319, in _result_or_cancel
    return fut.result(timeout)
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 458, in result
    return self.__get_result()
  File "/usr/lib/python3.10/concurrent/futures/_base.py", line 403, in __get_result
    raise self._exception
  File "/usr/lib/python3.10/concurrent/futures/thread.py", line 58, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 467, in init_model
    return self.model_servers[i].init_model(i, server_args, port_args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/netref.py", line 240, in __call__
    return syncreq(_self, consts.HANDLE_CALL, args, kwargs)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/netref.py", line 63, in syncreq
    return conn.sync_request(handler, proxy, *args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 718, in sync_request
    return _async_res.value
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/async_.py", line 108, in value
    raise self._obj
_get_exception_class.<locals>.Derived: Trying to create tensor with negative dimension -789: [-789]

========= Remote Traceback (1) =========
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 359, in _dispatch_request
    res = self._HANDLERS[handler](self, *args)
  File "/usr/local/lib/python3.10/dist-packages/rpyc/core/protocol.py", line 837, in _handle_call
    return obj(*args, **dict(kwargs))
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_rpc.py", line 54, in exposed_init_model
    self.model_runner = ModelRunner(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 230, in __init__
    self.init_memory_pool(total_gpu_memory)
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/managers/router/model_runner.py", line 300, in init_memory_pool
    self.req_to_token_pool = ReqToTokenPool(
  File "/usr/local/lib/python3.10/dist-packages/sglang/srt/memory_pool.py", line 11, in __init__
    self.mem_state = torch.ones((size,), dtype=torch.bool, device="cuda")
RuntimeError: Trying to create tensor with negative dimension -789: [-789]

detoken init state: init ok
goodbye ('<IP_ADDRESS>', 47232)
goodbye ('<IP_ADDRESS>', 48754)

I'm not sure if it's related to multi GPU or not

When meeting this error, what is the machine? I think 48G x 2 is not enough for a 8 x 7B model,

Yes thank you that seems to be the issue. Got it working on modal with 2x80GB and --tp-size 2
code
import sys

from modal import Image, Stub, gpu, method

MODEL_DIR = "/model"
BASE_MODEL = "NousResearch/Nous-Hermes-2-Mixtral-8x7B-SFT"

GPU_CONFIG = gpu.A100(memory=80, count=2)

def download_model_to_folder():
    import os
    from huggingface_hub import snapshot_download
    from transformers.utils import move_cache

    os.makedirs(MODEL_DIR, exist_ok=True)

    snapshot_download(
        BASE_MODEL,
        local_dir=MODEL_DIR,
        ignore_patterns="*.pt",  # Using safetensors
    )
    move_cache()

app_image = (
    Image.micromamba()
    .apt_install("build-essential")
    .apt_install("procps")
    .micromamba_install(
        "cudatoolkit=11.8",
        "cuda-nvcc",
        channels=["conda-forge", "nvidia"],
    )
    .pip_install(
        "transformers",
        "hf_transfer",
        "huggingface_hub[hf_transfer]",
        "sglang[all]",
    )
    .env({"HF_HUB_ENABLE_HF_TRANSFER": "1"})
    .run_function(download_model_to_folder)
)

stub = Stub("sglang-mixtral")

@stub.function(
    gpu=GPU_CONFIG,
    timeout=60 * 10,
    container_idle_timeout=60 * 10,
    allow_concurrent_inputs=10,
    image=app_image,
)
def test():
    # taken from https://modal.com/docs/examples/vllm_mixtral
    if GPU_CONFIG.count > 1:
        # Patch issue from https://github.com/vllm-project/vllm/issues/1116
        import ray

        ray.shutdown()
        ray.init(num_gpus=GPU_CONFIG.count)

    from sglang import function, system, user, assistant, gen, set_default_backend, Runtime

    from transformers import AutoTokenizer

    tokenizer = AutoTokenizer.from_pretrained('NousResearch/Nous-Hermes-2-Mixtral-8x7B-SFT', use_fast=True)

    # taken from https://modal.com/docs/examples/vllm_mixtral
    if GPU_CONFIG.count > 1:
        import subprocess

        RAY_CORE_PIN_OVERRIDE = "cpuid=0 ; for pid in $(ps xo '%p %c' | grep ray:: | awk '{print $1;}') ; do taskset -cp $cpuid $pid ; cpuid=$(($cpuid + 1)) ; done"
        subprocess.call(RAY_CORE_PIN_OVERRIDE, shell=True)

    runtime = Runtime(model_path=MODEL_DIR, tp_size=2)
    set_default_backend(runtime)

    @function
    def text_qa(s, question):
        s += "Q: " + question + "\n"
        s += "A:" + gen("answer", stop="\n")

    states = text_qa.run_batch(
        [
            {"question": "What is the capital of the United Kingdom?"},
            {"question": "What is the capital of France?"},
            {"question": "What is the capital of Japan?"},
        ],
        progress_bar=True
    )

    print(states)
    runtime.shutdown()

@stub.local_entrypoint()
def main():
    test.remote()


feat(Paragraph): Add long type

Resolves #1476

Hmm, curious if we should just introduce a lineHeight prop with default, short, long as options and just mark short for deprecation. Would make it more clear they are mutually exclusive. What do you think @Barsnes ?

Hmm, curious if we should just introduce a lineHeight prop with default, short, long as options and just mark short for deprecation. Would make it more clear they are mutually exclusive. What do you think @Barsnes ?

I don't think we should, since line-height is a prop in css, users might think they can send any value in here.
variant, or something along those lines would be better, I think 🤔

Hmm, curious if we should just introduce a lineHeight prop with default, short, long as options and just mark short for deprecation. Would make it more clear they are mutually exclusive. What do you think @Barsnes ?

I don't think we should, since line-height is a prop in css, users might think they can send any value in here. variant, or something along those lines would be better, I think 🤔

hmm, we already have weight on Label which kinda acts the same, expect its just a suggested subset of the actual values for font-weight 🤔
I kinda dont want to use variant for this as its what we normaly use to denote primary, secondary, tertiary variants for our components.
Hmm, any other suggestions for names? Or we just keep it long and short...

Hmm, curious if we should just introduce a lineHeight prop with default, short, long as options and just mark short for deprecation. Would make it more clear they are mutually exclusive. What do you think @Barsnes ?

I don't think we should, since line-height is a prop in css, users might think they can send any value in here. variant, or something along those lines would be better, I think 🤔

hmm, we already have weight on Label which kinda acts the same, expect its just a suggested subset of the actual values for font-weight 🤔
I kinda dont think we should use variant for this as its what we normaly use to denote primary, secondary, tertiary variants for our components.
Hmm, any other suggestions for names? Or we just keep it long and short...

If we can't think of anything better I suggest we keep it as long and short - and we can look into this further later 😄

Hmm, curious if we should just introduce a lineHeight prop with default, short, long as options and just mark short for deprecation. Would make it more clear they are mutually exclusive. What do you think @Barsnes ?

I don't think we should, since line-height is a prop in css, users might think they can send any value in here. variant, or something along those lines would be better, I think 🤔

hmm, we already have weight on Label which kinda acts the same, expect its just a suggested subset of the actual values for font-weight 🤔
I kinda dont think we should use variant for this as its what we normaly use to denote primary, secondary, tertiary variants for our components.
Hmm, any other suggestions for names? Or we just keep it long and short...

If we can't think of anything better I suggest we keep it as long and short - and we can look into this further later 😄

variant is also used in Figma so what the heck, lets use that. Better than having two props which are mutually exclusive.


Add nadid.is-cool.dev subdomain

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.
[x] There is no NS Records (Enforced as of Sepetember 4th, 2024)

Description

Created 'domains/nadid.is-cool.dev.json' for the approval of the subdomain 'nadid.is-cool.dev'
Link to Website

https://nadid-wasique.netlify.app/

I have read the CLA Document and I hereby sign the CLA


BUG:navbar links not working in return policy page

https://github.com/user-attachments/assets/9ca36fd5-d707-4326-8be8-d39b97f05277
solution:
linking the nav bar links to their exact locations
@Tejashri-Taral  Assign me this issue, I would like to fix it

As this is not a level1 task
I'm giving you one more task - Make navbar like home page

@Tejashri-Taral  sure I'll do it


Parse idAttribute to set model.id

Background
When a record is retrieved from the database, it is immediately run through the model's parse function, then passed to the model's set function. If the record contains the model's idAttribute, then the value of that attribute is assigned to model.id within the set function. See discussion for more about why this is important.
Problem
A problem arises if the model's idAttribute is specified in such a way that it matches the raw database record, and model.parse() mutates keys (e.g. conversion to camelCase). In this case, the parsed record passed to set will not contain the model's idAttribute (since the record has already been mutated).
Solution
Upon model.extend, compute and store a parsedIdAttribute that is equal to the specified idAttribute with the model's parse function applied to it. Use parsedIdAttribute within set to set model.id.
Discussion
This PR was motivated by issues with using hasMany relations, as documented in #818 and #957. In particular, if model.id is not set properly, then relation.eagerPair() will not be able to associate retrieved relations with their parent models. For this reason, I have also added an assertion in eagerPair() to check that all retrieved relations have been paired with their parent models: I can't think of any situations in which unpairable relations would be retrieved, since they are retrieved according to their parent model's attributes.
Also, as explained in #818, it is not possible to simply specify the parsed (.e.g camelCase) form of the idAttribute when defining the Model. This is because relation.eagerKeys() retrieves query binding values from the raw parentRecord, which has unparsed attribute keys.
Test
For an illustration of the issue that this fixes, cherry-pick 23ba8c6 and run npm test. It shows that model.id is not set when the parse function mutates attribute keys.

Bump :+1:

Any ETA on merging this? Currently the combination of snake_case in the database and camelCase on the models is broken because of the issue fixed in this PR.

Hi,
Needed to fix the commented test on #1380


GROOVY-7951: MethodCallExpression.transformExpression does not copy generic types

PR created from patch supplied in GROOVY-7951.  It seems like a reasonable change to make, though I did have trouble thinking up a way to test it that would be meaningful.  If anyone can nudge me in the right direction I can try to write up a test if you think it's warranted in this case.
I considered whether genericsTypes.clone() should be used, but it didn't seem to be common and most uses of the array seem to be read only.

What about something like:
import org.codehaus.groovy.ast.ClassNode
import org.codehaus.groovy.ast.ClassCodeVisitorSupport
import org.codehaus.groovy.ast.expr.MethodCallExpression
import org.codehaus.groovy.control.CompilerConfiguration
import org.codehaus.groovy.control.SourceUnit
import org.codehaus.groovy.control.customizers.CompilationCustomizer
import org.codehaus.groovy.classgen.GeneratorContext
import static org.codehaus.groovy.control.CompilePhase.CANONICALIZATION

def config = new CompilerConfiguration()
def checker = new CompilationCustomizer(CANONICALIZATION) {
  void call(SourceUnit src, GeneratorContext ctxt, ClassNode cn) {
    new ClassCodeVisitorSupport() {
      void visitMethodCallExpression(MethodCallExpression mce) {
        if (mce.objectExpression.text == 'java.util.Collections' &&
            mce.method.text == 'emptyList') {
            assert mce.genericsTypes != null && mce.genericsTypes*.name == ['Date']
        }
        super.visitMethodCallExpression(mce)
      }
      SourceUnit getSourceUnit() { src }
    }.visitClass(cn)
  }
}
config.addCompilationCustomizers(checker)

new GroovyShell(config).parse '''
    Collections.<Date>emptyList()
'''

Currently the macro transform (so 2.5 only) is stealing the generics information away but after the fix it stays put.

Thanks for the test case @paulk-asert, that was helpful.  I added the ClassCodeExpressionTransformer transformer so that the test would break in 2_4_X (without relying on the macro transform being in place) since this fix seems like something that could go in that branch as well as master, but would be interested to know what you think.

Nice. +1 for both branches.


wired behavior about makefile

if you run make under /cri and you got containerd and ctr binary in _output, but if you run make again next,  and try to remove the binary containerd under _output, you will get rm: cannot remove 'containerd': Text file busy. I use lsof ` to check, and find a process using the binary, if you kill the process, and you will remove it.
Based on that, I think it's a little wired, and need to report here.
I hope that it's ok to run 'make' command twice.

odd... text file? busy?
it should be a program.. and I see no reason for it to be in use.
behavior not seen here:
mike@mike-VirtualBox:~/go/src/github.com/containerd/cri$ sudo make
[sudo] password for mike:
go build -o _output/containerd \
	-tags 'seccomp apparmor' \
	-ldflags '-X github.com/containerd/cri/vendor/github.com/containerd/containerd/version.Version=3ba04c0-TEST' \
	-gcflags '' \
	github.com/containerd/cri/cmd/containerd
go build -o _output/ctr \
	-tags 'seccomp apparmor' \
	-ldflags '-X github.com/containerd/cri/vendor/github.com/containerd/containerd/version.Version=3ba04c0-TEST' \
	-gcflags '' \
	github.com/containerd/cri/cmd/ctr
mike@mike-VirtualBox:~/go/src/github.com/containerd/cri$ ls _output
containerd  ctr
mike@mike-VirtualBox:~/go/src/github.com/containerd/cri$ sudo rm _output/containerd
mike@mike-VirtualBox:~/go/src/github.com/containerd/cri$ sudo make
go build -o _output/containerd \
	-tags 'seccomp apparmor' \
	-ldflags '-X github.com/containerd/cri/vendor/github.com/containerd/containerd/version.Version=3ba04c0-TEST' \
	-gcflags '' \
	github.com/containerd/cri/cmd/containerd
mike@mike-VirtualBox:~/go/src/github.com/containerd/cri$

my go version:
go version go1.11.2 linux/amd64

suggest doing a sudo make clean in case you golang cache is messed up somehow.. then try again?

I'm not sure it's related about docker service I used, because I found make install  will install the containerd binary into /usr/local/bin which may pollute the environment.

DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=16.04
DISTRIB_CODENAME=xenial
DISTRIB_DESCRIPTION="Ubuntu 16.04.6 LTS"
NAME="Ubuntu"
VERSION="16.04.6 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.6 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"
VERSION_CODENAME=xenial
UBUNTU_CODENAME=xenial

go version go1.10.4 linux/amd64

We had to move up to go 1.11.x a while back, I don't remember all of the issues, but you may be seeing go build dependency issues with our code base and vendor tree.
"pollute"... the latest docker is compatible with the latest containerd.. it should run just fine with your master built version of containerd.

@ZYecho We just call go build to build binaries in make. And I can't reproduce it.
This seems to be something wrong in your development environment.

close now. thanks every one


latest 4.12.0 version doesn't support nodejs 6.9.1

NPM version (npm -v): 3.10.8
Node version (node -v): 6.9.1
Node Process (node -p process.versions): 6.9.1
Node Platform (node -p process.platform): Linux
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): 4.12.0
npm node-sass versions (npm ls node-sass): 4.12.0
npm install which will install latest node-sasss 4.12.0 version, failed with below errors.
4.11.0 version is fine.
Detail logs:
222424 verbose lifecycle<EMAIL_ADDRESS>CWD: C:\frontend\node_modules\node-sass
222425 silly lifecycle<EMAIL_ADDRESS>Args: [ '/d /s /c', 'node scripts/build.js' ]
222426 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
222427 info lifecycle<EMAIL_ADDRESS>Failed to exec postinstall script
222428 verbose unlock done using C:\Users\wuhongwe.CORPDOM\AppData\Roaming\npm-cache_locks\staging-23a59feab1bf2de9.lock for C:\frontend\node_modules.staging
222429 silly rollbackFailedOptional Starting
222430 silly rollbackFailedOptional Finishing
222431 silly runTopLevelLifecycles Finishing
222432 silly install printInstalled
222433 warn optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\chokidar\node_modules\fsevents):
222434 warn notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
222435 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid OS:    darwin
222435 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid Arch:  any
222435 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual OS:   win32
222435 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual Arch: x64
222436 warn optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\sane\node_modules\fsevents):
222437 warn notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
222438 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid OS:    darwin
222438 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid Arch:  any
222438 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual OS:   win32
222438 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual Arch: x64
222439 warn optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\nodemon\node_modules\chokidar\node_modules\fsevents):
222440 warn notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
222441 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid OS:    darwin
222441 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid Arch:  any
222441 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual OS:   win32
222441 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual Arch: x64
222442 warn optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\watchpack\node_modules\chokidar\node_modules\fsevents):
222443 warn notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
222444 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid OS:    darwin
222444 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Valid Arch:  any
222444 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual OS:   win32
222444 verbose notsup SKIPPING OPTIONAL DEPENDENCY: Actual Arch: x64
222445 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222446 verbose If you need help, you may report this error at:
222446 verbose     https://github.com/npm/npm/issues
222447 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222448 verbose If you need help, you may report this error at:
222448 verbose     https://github.com/npm/npm/issues
222449 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222450 verbose If you need help, you may report this error at:
222450 verbose     https://github.com/npm/npm/issues
222451 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222452 verbose If you need help, you may report this error at:
222452 verbose     https://github.com/npm/npm/issues
222453 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222454 verbose If you need help, you may report this error at:
222454 verbose     https://github.com/npm/npm/issues
222455 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222456 verbose If you need help, you may report this error at:
222456 verbose     https://github.com/npm/npm/issues
222457 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222458 verbose If you need help, you may report this error at:
222458 verbose     https://github.com/npm/npm/issues
222459 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222460 verbose If you need help, you may report this error at:
222460 verbose     https://github.com/npm/npm/issues
222461 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222462 verbose If you need help, you may report this error at:
222462 verbose     https://github.com/npm/npm/issues
222463 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222464 verbose If you need help, you may report this error at:
222464 verbose     https://github.com/npm/npm/issues
222465 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222466 verbose If you need help, you may report this error at:
222466 verbose     https://github.com/npm/npm/issues
222467 warn<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
222468 verbose If you need help, you may report this error at:
222468 verbose     https://github.com/npm/npm/issues
222469 warn<EMAIL_ADDRESS>license should be a valid SPDX license expression
222470 verbose If you need help, you may report this error at:
222470 verbose     https://github.com/npm/npm/issues
222471 verbose stack Error<EMAIL_ADDRESS>postinstall: node scripts/build.js
222471 verbose stack Exit status 1
222471 verbose stack     at EventEmitter. (C:\frontend\node\node_modules\npm\lib\utils\lifecycle.js:255:16)
222471 verbose stack     at emitTwo (events.js:106:13)
222471 verbose stack     at EventEmitter.emit (events.js:191:7)
222471 verbose stack     at ChildProcess. (C:\frontend\node\node_modules\npm\lib\utils\spawn.js:40:14)
222471 verbose stack     at emitTwo (events.js:106:13)
222471 verbose stack     at ChildProcess.emit (events.js:191:7)
222471 verbose stack     at maybeClose (internal/child_process.js:877:16)
222471 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:226:5)
222472 verbose pkgid<EMAIL_ADDRESS>222473 verbose cwd C:\frontend
222474 error Windows_NT 10.0.16299
222475 error argv "C:\frontend\node\node.exe" "C:\frontend\node\node_modules\npm\bin\npm-cli.js" "install" "--proxy=http://web-proxy.houston.softwaregrp.net:8080"
222476 error node v6.9.1
222477 error npm  v3.10.8
222478 error code ELIFECYCLE
222479 error<EMAIL_ADDRESS>postinstall: node scripts/build.js
222479 error Exit status 1
222480 error Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js'.
222480 error Make sure you have the latest version of node.js and npm installed.
222480 error If you do, this is most likely a problem with the node-sass package,
222480 error not with npm itself.
222480 error Tell the author that this fails on your system:
222480 error     node scripts/build.js
222480 error You can get information on how to open an issue for this project with:
222480 error     npm bugs node-sass
222480 error Or if that isn't available, you can get their info via:
222480 error     npm owner ls node-sass
222480 error There is likely additional logging output above.
222481 verbose exit [ 1, true ]

Please post the full log output. The error is not in the output you've
provided
On Sun., 28 Apr. 2019, 3:24 pm kadinwu<EMAIL_ADDRESS>wrote:

NPM version (npm -v): 3.10.8
Node version (node -v): 6.9.1
Node Process (node -p process.versions): 6.9.1
Node Platform (node -p process.platform): Linux
Node architecture (node -p process.arch): x64
node-sass version (node -p "require('node-sass').info"): 4.12.0
npm node-sass versions (npm ls node-sass): 4.12.0
npm install which will install latest node-sasss 4.12.0 version, failed
with below errors.
4.11.0 version is fine.
[ERROR] npm ERR! node v6.9.1
10:58:18 [ERROR] npm ERR! npm v3.10.8
10:58:18 [ERROR] npm ERR! code ELIFECYCLE
10:58:18 [ERROR]
10:58:18 [ERROR] npm ERR<EMAIL_ADDRESS>postinstall: node
scripts/build.js
10:58:18 [ERROR] npm ERR! Exit status 1
10:58:18 [ERROR] npm ERR!
10:58:18 [ERROR] npm ERR! Failed at the<EMAIL_ADDRESS>postinstall
script 'node scripts/build.js'.
10:58:18 [ERROR] npm ERR! Make sure you have the latest version of node.js
and npm installed.
10:58:18 [ERROR] npm ERR! If you do, this is most likely a problem with
the node-sass package
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/sass/node-sass/issues/2652, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAENSWEQ7XDXGAMERTUY2TTPSUYI5ANCNFSM4HI5VRTA
.

why this error is coming while installing node-sass any version.

+3. We are using node-sass in our application and 3 of our rpi user are facing the same issue. I have requested them to provide the full log but npm doesn’t provide more detailed log than you already have it here, ie. ‘222480 error Failed at the<EMAIL_ADDRESS>postinstall script 'node scripts/build.js’.

hi all, here is the log file https://www.dropbox.com/s/etdeio90ca62s0r/2019-05-13T18_11_26_044Z-debug.log?dl=0

I found this googling the same issue : "Failed at the<EMAIL_ADDRESS>postinstall script."
node version 11.13.0
npm version 6.9.0
sass version 1.20.1

just want to confirm that there is still an issue even after running 'npm cache clear', 'npm cache verify' and 'npm install'

I just completely reinstalled (from erased drive) MacOS Mojave (10.14.5) and did little else but install node and npm (direct download from their site) and then gulp and then node-sass.  I still get the "Failed at the<EMAIL_ADDRESS>postinstall script." Log of install below.
2019-05-23T04_34_55_052Z-debug.log

Hi. The issue still has awaiting response label, so just wanted to confirm, if it is still waiting for any response from us?

npm install --unsafe-perms


Site tutorial: Part 1

Adds a tutorial section to guide the newcomers through the new lib!
It has nice things like uses cases and exercises to the reader (with solutions!)
The tutorial is divided in two parts: One talks about the reading and writing operations (read, write and append). The other one talks more about file manipulation operations like temp files, file creation and deletion, etc.

Pending for discussion:

Find an alternative to the gist problem
Should I add the tutorial hell advice to the introduction or delete it completely?

* Find an alternative to the gist problem

I think we found it

* Should I add the tutorial hell advice to the introduction or delete it completely?

Delete it. Also, avoid using "hell" as a term.

Changes done!


Improve ID Auto-Population Logic for Facility and Facility Group Creation

Description
Enhance the facility and facility group creation process in the Facilities app to auto-populate the ID field based on the name only during the initial entry. Subsequent changes to the name should not affect the ID field, providing a more consistent and predictable user experience.

Once user edits facility ID field, it should no longer auto populate based on user input.

Assigning this ticket to @priyanshee1604 for verification.
CC: @dt2patel


Add Labels And Annotations From Pods To ServiceInstance Metadata

https://github.com/spring-cloud/spring-cloud-kubernetes/issues/1162

Also please fix the checkstyle errors

ok

A couple of other things:
1.) We would need to add the same functionality to the Fabric8 implementation 2.) I am still unsure of what will happen when labels/annotations from the pod and service conflict. Seems like they will override either other, last one added will win. I am not sure how we could resolve this.

I think there is no good way to solve the conflict, but only through the user to control themselves, because the metadata in the ServiceInstance is just a Map<String, String>

We still need documentation

correlating the the address to the pod is not complicated, for example:
kubectl get -o json endpoints spring-cloud-kubernetes-client-configmap-event-reload
{
    "apiVersion": "v1",
    "kind": "Endpoints",
    "metadata": {
        "annotations": {
            "endpoints.kubernetes.io/last-change-trigger-time": "2023-02-21T14:01:05Z"
        },
        "creationTimestamp": "2023-02-21T14:00:55Z",
        "labels": {
            "app": "spring-cloud-kubernetes-client-configmap-event-reload"
        },
        "name": "spring-cloud-kubernetes-client-configmap-event-reload",
        "namespace": "default",
        "resourceVersion": "613",
        "uid": "2d017d3a-a18a-4ddb-ae2f-05c3f61ab117"
    },
    "subsets": [
        {
            "addresses": [
                {
                    "ip": "<IP_ADDRESS>",
                    "nodeName": "c577996ade89",
                    "targetRef": {
                        "kind": "Pod",
                        "name": "spring-cloud-kubernetes-client-configmap-deployment-event-jw5jz",
                        "namespace": "default",
                        "uid": "a499484e-0d8c-4668-be9c-f4df52c550d3"
                    }
                }
            ],
            "ports": [
                {
                    "name": "http",
                    "port": 8080,
                    "protocol": "TCP"
                }
            ]
        }
    ]
}
/ #

notice that each address (if it has a targetRef), has :
                    "targetRef": {
                        "kind": "Pod",
                        "name": "spring-cloud-kubernetes-client-configmap-deployment-event-jw5jz",
                        "namespace": "default",
                        "uid": "a499484e-0d8c-4668-be9c-f4df52c550d3"
                    }

As to returning the metadata from pods, we could introduce a new record in DefaultKubernetesServiceInstance, for example PodMetadata that would contain things like pod-name and pod-labels. Of course you have to opt it, to get such data.
@ryanjbaxter if you agree, I would continue this at some point in time.

I've added such functionality to the fabric8 client, pending review here. I will, of course, add such an option to the k8s-client too later.

Addressed in https://github.com/spring-cloud/spring-cloud-kubernetes/pull/1254


5.0.2 maintenance

push 5.0.2 release branch on to the maintenance branch

@yinlinchen :  this doesn't look right: The PR should be merging 5.0.2-RC into 5.x-maintenance


Should I use your theme in my editor?

I like Dracula.  However, my use case is to build themes directly into an editor that I built, rather than to send them to someone else.
I'd like to build Dracula into my editor, the way I have Penumbra and Solarized and some others.  I'm not sure, in this case, if it's appropriate: you charge for your stuff.  I am uncertain if I am supporting you or pirating you.
Please clarify.  May I add Dracula to my web-based editor?  Thank you.

Only the Dracula Pro themes are paid content. The regular Dracula theme is free.

Hi @StoneCypher 👋🏻
You can integrate the Dracula theme into your editor. 😅
Zeno Rocha created Dracula Theme under the MIT License; that is, you are allowed to:

Commercial use
Modification
Distribution
Private use

Just what I would ask you to collaborate with the project by adding the steps to activate it in your editor; this is also a way to give visibility to your project.
Now, Dracula Pro is a closed-source project, and you don't have those legal permissions in this one.
I hope this clarifies your doubt. 🧛🏻

It does. Thank you. I will try to follow your process.
I got themes working yesterday, so, probably over the next week or two.


prefer_const_constructor in packages/flutter/bin

@Hixie can you merge this is or is it waiting on something?


TreeView: Refactor slots

Describe your changes here.
Closes # (type the issue number after # if applicable; otherwise remove this line)
Screenshots
Please provide before/after screenshots for any visual changes
Merge checklist

[ ] Added/updated tests
[ ] Added/updated documentation
[ ] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

You can comment on the pull request or remove the stale label to keep it open. If you do nothing, this pull request will be closed in 7 days.

Closing this for now to avoid merge conflicts. We'll revisit this when we prioritize work on slots.


Explicitly close database connection

Hi, I cannot find any documentation on when a connection closes. I see that a pool will stop when it is no longer referenced, but what about "normal" Conn?
Is there something like Conn::close that will close the connection? From what I understood from the docs, the method with that signature does not.

Oh, sorry. COM_QUIT is executed in impl Drop for Conn. I'll update the docs.


applying black to HARK

This PR applies black to reformat all the code in the main HARK module.

[x] Update CHANGELOG.md with major/minor changes.

@sbenthall can you fix up the merge conflicts and then merge this in.

@sbenthall Should I merge this in ?


option to disallow surrender [fixes #32]

fixes: pappde/bmai#32 option to disallow surrender
added to the parser a command to disable the ability to surrender
added unit tests around the new behavior
requires the test_framework branch pappde/bmai#34
major consideration is the adding of virtual to a method so that I could override it. I dont like adding stuff in the source code simply to make testing easier but I couldn't see a less invasive way at the moment

there needs to be a larger discussion around testing. the current patterns are very difficult to test. we should decide if we want to migrate towards more testable patterns

BONUS: this also includes some major GHActions workflow improvements including auto build/tag/releasing the code on main branch weekly. the community wants a way to use this without compiling so I thought it would be great if they could just head over to the releases and grab a recent copy

DONT FORGET TO SQUASH

major consideration is the adding of virtual to a method so that I could override it. I dont like adding stuff in the source code simply to make testing easier but I couldn't see a less invasive way at the moment

there needs to be a larger discussion around testing. the current patterns are very difficult to test. we should decide if we want to migrate towards more testable patterns

A service injection pattern would be cleaner, and could always be added as needed. For example a "MessageSender" - but that is a lot more code. Instead, changing some of these classes/methods to virtual is pretty safe and conceptually appropriate
In some cases, the added behavior would be reasonable to add to the core code - even if only the test code uses. For example, you could change BMC_Parser to cache the last message sent, then the test code would have access to this without having to override the behavior. Of course, I can appreciate it's safest to not even touch the base class.


Fixed test_dft_1d so it passes

Found the problem causing #52, and fixed it so tests now pass.
It was an issue with the tests not the code. It was because iterating over a dataarray iterates over the values in the array, rather than over the names of the dimensions as the test assumed.
How did the failing tests get merged? Did this behaviour change in xarray at some point? I'm using the development master version of xarray. Does xrft need to specify minimum versions of its dependencies?

Codecov Report

Merging #53 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   88.92%   88.92%
=======================================
  Files           1        1
  Lines         271      271
  Branches       90       90
=======================================
  Hits          241      241
  Misses         15       15
  Partials       15       15

Continue to review full report at Codecov.

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


fix bindings type options

change the binding option of type pthread_* to opaque_type. it makes the structure containing the pthread_* varibles to derive Copy trait

would you please update the PR?


Introduce ZYTE_API_MAX_REQUESTS

An attempt on https://github.com/scrapy-plugins/scrapy-zyte-api/issues/111 to gather feedback.
This has minimal changes that only addresses the limits on ZAPI Requests.
In the future, we also might want to impose other limits like the following which may affect the implementation:

item count
per data type
per page type
etc

TODO

[x] README

I think the approach is fine, though I haven't thought about the download_req_count calculation.

Hey folks, I've unmarked this one as draft.
I think we can think about other things to set as max like item count, per data type, per page type, etc in another future PR.

Thanks @BurnzZ!


Update spanish translations

Changes / Comments
Spanish translations for D&D_3-5 sheet.
IMPORTANT: I tried to keep english string lengths for the spanish translations. Please, check if translations fit in place before merging.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[ ] Does the pull request title have the sheet name(s)? Include each sheet name.
[ ] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[ ] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

@helio-huete You'll need to speak with the original sheet author if you'd like to have the spacing checked on the sheet. Do you want me to hold off on merging this until you contact them?

@clevett Thanks for the info. We can wait some time for @suldae to comment on this PR (I don't know how to contact her in other way).

I've never been asked to verify translations before; so I just dropped it into the translation tab on the game and looked at what it looked like.  Hopefully that replicates things well enough.
The only translation I see that doesn't appear to fit is:
"power-points": "Puntos de Poder",
it gets cut off after the "o" in Poder.
I went ahead and created a pull request to increase the size of that field slightly so it would fit nicely.

@suldae thank you very much for the test you have done. I think this is enough, so feel free to merge the PR.

@clevett feel free to merge the PR.
Thanks.

@helio-huete
This will be merged in the coming week's pull.
If you are interested in doing further translations please send me an email at<EMAIL_ADDRESS>. We use Crowdin to process translations it is safer and easier to do than editing a JSON. :)
Thanks,
Cassie

Hey Cassie, did something happen with my pull request?  It seems to have just disappeared; I don't see that the sheets were updated and even my branch no longer shows the change.
Should I try again?  Is editing the file in my branch through the browser a bad way to do it? (Didn't want to use the desktop app because I have a bunch of pending changes for a different sheet in there which I haven't pushed up to the server yet).


Add visionOS support

TODO

Update UIMenuController code to use UIEditMenuInteraction on iOS 16+ / visionOS

Summary
Add visionOS support via react-native-test-app. This mostly just worked... so I'm throwing up the PR while I'm on a good streak :D
As it turns out, most of where we ifdeffed stuff out for macOS, is also where we need to ifdef out for visionOS. So a simple conversion of the ifdefs from "if !macOS" to "if iOS" does the job.
Test Plan

This PR works as far as I can tell, save custom menus. The method for showing custom actions in a context menu on iOS 16 / visionOS 1.0 changed (use UIEditMenuConfiguration instead of UIMenuController), so I'm seeing If I can refactor that bit. Otherwise, this change might be able to land without custom menu support

Hello @Titozzz / @jamonholmgren , I was wondering if there was anything I could do to help move this PR forward? Appreciate your time spent maintaining this repo :)


remove old popularity tags when new is assigned

not tested

Looks good, better with the regex. No testing like production ;)
Someday I'll mock the GH api. That day is not today.

@erasche would this work?

@martenson beautiful, thank you!

@erasche thanks for leading me and merging


Master

Added the flag prod-timeout with a default value of 3 seconds
Added the flag alternate-timeout with a default value of 1 second

Thanks a lot! it will be a good fix. There are some minor typos, and need to run "go fmt ".


I don't know how to code in fnf psych

Describe your problem here.
thats it
Are you modding a build from source or with Lua?
Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
no

just… look it up on YouTube for fucks sake

Describe your problem here.
thats it

Then take coding lessons no brainer

If you want to learn source code then the language they use is called HScript. I recommend taking courses for it somewhere online...

If you want to learn source code then the language they use is called HScript. I recommend taking courses for it somewhere online...
The luascripting API uses hsScript, the game itself just uses haxe.

ok

If you want to learn source code then the language they use is called HScript. I recommend taking courses for it somewhere online...

The luascripting API can use hsScript, the game itself just uses haxe.

oki


feat: configuring an extensible PR auto-labeler

Configuring this nice automated labeler https://github.com/actions/labeler.
Setting a few rules to get started, but it should be easy to grow and add more conditions as we go.

Looks awesome! Cant wait to pile on more configs for localization, tests, database support, etc etc.


Starknet 0.6.10 Constants Access Issue on final package

Describe the bug

constants don't expose StarknetChainId nor any other as const+type exports on the final build
Local usage is working (from /src) in the tests.

The imminent conclusion is that namespaced as const+type exports don't get bundled properly.
To Reproduce
import { constants } from "starknet";
const a=constants.StarknetChainId.SN_SEPOLIA;

Expected behavior
Example should pass build

Node version [e.g. 21.0.0]
Starknet.js version 0.6.10

hey! I would like to tackle this one


Can't install app

Any workarounds?

You need to manually install the certificate,
Enable Developer Mode in Settings, then Right click on appx -> properties -> Digital Signatures -> click on Pinguin2001 and click Details, then on the details screen click on View Certificate and then you install the Certificate on local machine and trusted root certificates, and then try again installing the appx and it should work


[libdisasm] add license

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Full license text is visible here: https://sourceforge.net/p/bastard/code/HEAD/tree/trunk/libdisasm/LICENSE

Thanks!


feat: support --parse-as flag

Resolves #67

Thank you for your PR. @another-rex is on vacation and we will review it early next year and then release as part of the next minor release 1.0.x release.

yup all good - fwiw I might do another PR or two over the holiday break; btw I think you mean 1.x release, since 1.0.x is for fixes not minors :)

The intention of that issue was to automatically detect such cases without user intervention.

I thought about that when implementing the requirements.txt parser and came to the conclusion that so long as such file does not have a well-defined identifier, it's not feasible because any file could be intended to be used as a requirements.txt - so you'd end up having to open every file, and either assume when you come across a line that doesn't look like it belongs in requirements.txt that it's not that kind of file or error, which if you choose to report to the user will be extremely verbose and if you don't then it can lead to unexpected behaviour as a user could think a file is being checked when it's not.
In addition, currently the requirements.txt parser (like all the parsers) tries to be forgiving so it just skips any line that doesn't denote a requirement - so i.e. a markdown file with a valid requirement line would get parsed as a valid requirements.txt with that as a package.
Finally, there are also cases where a user might not want a requirements.txt-like file to be scanned - the first one that comes to mind which is pretty active is the requirements.in file used by pip-tools/pip-compile; while that specific file name could be detected, like requirements.txt users are free to name the file whatever they like (and for similar reasons as with requirements.txt, there could be multiple i.e requirements.dev.in).
So I personally think it's better to mirror the behaviour of pip: pick up requirements.txt by default, and require users to explicitly name any other files that should be parsed as such, which they need to pass explicitly to pip anyway whenever they're using that file so it should be a familiar behaviour.
One improvement that could be made to the requirements.txt parser that I'd consider related to this would be supporting referring to other requirements.txt files and constraints files - I had not implemented them yet because I wasn't sure if it was correct to just treat them as being part of the original requirements.txt or if they should be somehow pushed out as their own lockfile (effectively having the parser report them as a heuristic for identifying other requirements.txt files).

Option 3. Seems like the best one to me, in a way that's consistent with our existing CLI format, without introducing redundancies. WDYT?

Sounds ok, but note it's not actually consistent with the existing CLI flags - currently everything is based on "last value wins", i.e. doing the equivalent as your example comment but with --config will result in the last --config being used for all lockfiles.
Note that there is a tradeoff too: you're removing the ability for users to confidently override/enforce certain options which can be useful in automatic tooling where the command line arguments are being generated (i.e. ensuring that the --config the tooling generates is used regardless of what's in the repo), or in project scripts when debugging/trying something.
Given that, I'd prefer to land this PR as-is and then follow it up with another PR changing the overall flag logic to work as you described (if that is still something you'd like).

The intention of that issue was to automatically detect such cases without user intervention.

I thought about that when implementing the requirements.txt parser and came to the conclusion that so long as such file does not have a well-defined identifier, it's not feasible because any file could be intended to be used as a requirements.txt

Thanks for explaining. Yeah, it seems like a difficult problem to solve perfectly. One compromise solution here is to account for any filename matching *requirements*.txt.
This particular issue came up when we tried scanning https://github.com/home-assistant/core, where they have a bunch of requirements_*.txt files in the root that we didn't pick up.
This is also the approach that syft takes: https://github.com/anchore/syft/blob/e3d6ffd30e44428b898675922a0474221a7f7dc7/syft/pkg/cataloger/python/cataloger.go#L16

Sounds ok, but note it's not actually consistent with the existing CLI flags - currently everything is based on "last value wins", i.e. doing the equivalent as your example comment but with --config will result in the last --config being used for all lockfiles.

Good call on the inconsistency with --config. Unfortunately because of our SemVer adherance we can't change the behaviour. We'll need to call this out as a global level config that can only be specified once in our CLI documentation.
That said, with --config we already provide a similar per-lockfile level mechanism to override things via osv-scanner.toml files living in the same directory.

Note that there is a tradeoff too: you're removing the ability for users to confidently override/enforce certain options which can be useful in automatic tooling where the command line arguments are being generated (i.e. ensuring that the --config the tooling generates is used regardless of what's in the repo), or in project scripts when debugging/trying something.

Not sure I fully understand this in the case of --parse-as. If the rules are clearly documented and understood, then automated tooling can still ensure the same outcome?

Given that, I'd prefer to land this PR as-is and then follow it up with another PR changing the overall flag logic to work as you described (if that is still something you'd like).

Unfortunately as above we need to ensure backwards compatibility for all of our releases, so we can't change the behaviour of a flag like so after we merge (and release this).

That said, with --config we already provide a similar per-lockfile level mechanism to override things via osv-scanner.toml files living in the same directory.

I assume you mean that provided --config isn't provided; since if it is then osv-scanner.toml files won't be read (which is the correct behaviour imo)

If the rules are clearly documented and understood, then automated tooling can still ensure the same outcome?

Yes, but you increase the amount of work required to ensure that outcome because now users and tools have to examine the command more closely, figure out what rules are in place, and modify the whole command to ensure their outcome; whereas currently, if you just stick --config at the end you know regardless of if there are configs that exist in the directories being scanned or if an explicit config is being used in the command, your config will get used because it's the last option specified.
I think in the case of --parse-as it's less likely to be an issue, but I'm also aware how often other people come up with perfectly valid ways of using tools that do get impacted by these sorts of things; really what I'm driving at here is I think the general behaviour of how all CLI flags are applied should be as consistent as possible to reduce the cognitive load when using the tool (and also sadly in my experience people don't read the documentation enough to rely on that alone)

because of our SemVer adherance we can't change the behaviour.
Unfortunately as above we need to ensure backwards compatibility for all of our releases

Actually so long as the behaviour is not documented (which it currently isn't), you can make the argument that the change is a bugfix as it was always the desired behaviour - of course I agree care should be taken to minimize disruptions, but I personally think it would be pretty confusing to have some CLI flags behave so differently/inversely to others; it is of course your call though, so I can try and find some time to change this.

That said, with --config we already provide a similar per-lockfile level mechanism to override things via osv-scanner.toml files living in the same directory.

I assume you mean that provided --config isn't provided; since if it is then osv-scanner.toml files won't be read (which is the correct behaviour imo)

Yep!

I personally think it would be pretty confusing to have some CLI flags behave so differently/inversely to others; it is of course your call though, so I can try and find some time to change this.

Thanks for the detailed rationale. I see your point on the inconsitency wrt flags, but I also see a global --parse-as as inconsistent when our CLI examples have cases like https://github.com/google/osv-scanner#example-2
$ osv-scanner --lockfile=/path/to/your/package-lock.json --lockfile=/path/to/another/Cargo.lock

Adding a --parse-as to this would not make very much sense.
Thinking more -- is requirements.txt the only edge case where the filename is unpredictable (and commonly ends in .txt)? If so, then perhaps we're overthinking all of this, and we can just do something like parse all *.txt files as requirements.txt without needing this additional flag.

--parse-as would apply to all subsequent --lockfile arguments, and can be specified multiple times.

I've done some initial looking and it seems that urfave/cli doesn't support this as it does not expose the position each flag was passed in to the command.

then perhaps we're overthinking all of this, and we can just do something like parse all *.txt files as requirements.txt without needing this additional flag.

I don't mind saying that this solves a separate issue, but I think there is value in having the flag in either form (and even if its hidden) because it makes the tool more flexible - consider that right now you have to name your file based on the lockfile for a particular ecosystem but people could use exotic systems which could use alternate names (for better or worse), and we are already doing that here with the fixtures for lockfile.

Another option similar to option 1 is to have parse-as argument where you specify the additional mapping in value rather than the flag itself, e.g.:
--parse-as=requirements_extra.txt:requirements.txt

And you can then specify multiple of these similar to how it can be done with other flags.

This seems like a reasonable compromise that allows the flexibility we want without being inconsistent with our CLI examples.
I'd probably invert the order though:
--parse-as=requirements.txt:/path/to/requirements_extra.txt

To make this more consistent/easier to parse in case there are ":" characters in the input paths.

that actually also enables the original form of this i.e. --parse-as=requirements.txt could be applied to everything, because there is no :.

that actually also enables the original form of this i.e. --parse-as=requirements.txt could be applied to everything, because there is no :.

This could, but I think we should try to have a single way of doing something (i.e. the full --parse-as=requirements.txt:...), rather than multiple ways (i.e. also supporting the original --parse-as=requirements.txt --lockfile=... ), as that can end up being confusing in the future.

Yeah I wasn't meaning supporting both of those ways (especially since the second way is not possible with the current CLI library being used), but that because --parse-as=x:y is effectively a way of passing a map, you could make --parse-as=x (without the : delimiter) set the default value when there isn't an item in the map (which right now is effectively "", which has lockfile infer the parser to use based on the filename)

@oliverchang @another-rex I've implemented this per our discussions - it's come out pretty nicely; a couple of thoughts:

I've tried to implement what I feel is the current core of the flag in a way that should hopefully allow for a bit of semver-flexibility for exploring future improvements like the ability to apply a mapping to a whole directory, setting a default parser, etc that I think could be useful but probably require too much additional consideration to be worth blocking this PR over

alternatively, this PR could do a "starts with" path check instead of an equality check which should immediately support at least those two features - the trade-off being what if someone had a file or folder named something like package-lock.json or Gemfile.lock2 etc

This should enable support for at least reading CSV files with only a few more lines of code - I've deliberately not done that in this PR because its self-standing (and you might want to go with a different format than what lockfile currently expected), but I can do a PR straight after this one lands
I considered if the installed parser should be included in the map within lockfile and then have it only used if the related lockfile functions are called explicitly with installed, which felt like it could be right. but I was concerned about edge-cases and just general ikyness in other parts of the codebase - I think it makes sense to go with what I've done for now, and reconsider the structure if/when similar special-case parsers are introduced

Thanks! Probably for a follow up PR, but something we can add as well is instead of passing in the full path, you only have to pass in the file name, or even a glob to match against. This PR seems structured quite nicely to support those cases.

(I realise I may not have communicated very clearly about what I had thought --parse-as  with the proposed format would do -- sorry about that!)

Also resolves #124

@G-Rath I think we just have two minor comments here to resolve, and we're good to go?

@oliverchang yup I've been a bit slammed between personal and work stuff, so not had a chance to do anything big but tomorrow's 20% time so should be able to pick it up then

note the documentation is a little ham-y right now, but I think that'll be easier to improve once #168 is landed because then there'll be two cases like this.

thanks a ton as always @G-Rath !

Hi,
When is this feature expected to be released (i.e. included in ghcr.io/google/osv-scanner:latest)?

@agmond A new version is planned to be released this week, we will also publish a releasing schedule soon.


Generated code does not do what was intended

Short description of the problem:
My source code:
  private gotoHomepage(data: any): void {
    console.log('gotoHomepage!');
    let test:number = 0;
    let activePage: ViewController = this.nav.getActive();
    if (!activePage || activePage.component['name'] === 'LoginPage') {
      test = 1;
      this.nav.setRoot(HomePage, data);
    }
    console.log('Test is now ', test)
  }

takes me to my home page and outputs 1 when I run ionic serve
When I run ionic run android I get the output 0 from the generated code:
        t.prototype.gotoHomepage = function(t) {
            console.log("gotoHomepage!");
            var e = 0
              , n = this.nav.getActive();
            n && "LoginPage" !== n.component.name || (e = 1,
            this.nav.setRoot(Ltt, t)),
            console.log("Test is now ", e)
        }

Which @ionic/app-scripts version are you using?
0.0.23
Cordova CLI: 6.3.1
Gulp version:  CLI version 1.2.2
Gulp local:
Ionic Framework Version: 2.0.0-rc.0
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
OS: Distributor ID:	Ubuntu Description:	Ubuntu 16.04.1 LTS
Node Version: v6.7.0

Ooops - this is to do with name mangling, not incorrect code generation.


Autoupdates on user login (fixes #3395, fixes #3392)

@normanjaeckel What should be the default for disabling the last login field? I would prefer True for disabling.

@normanjaeckel All done. Ready for your last review. As discuss, I have to merge it now. Feel free to remark something later. I marked this pull request with new label "needs review" for you.


backlinks are broken for notes with spaces in the filename

The commit 8eca1e60f78f40d6b598aa397bb315adfc53d9ad solved the issue for links to the notes, but if you enter a note with a space in its filename, it won't have any backlinks listed

here is an example: https://flatrick.github.io/moc/Note-taking/
The note's original name is "Note taking.md", and the link to the note works, but if you look at its list of backlinks, the list is empty. However, it is definitely linked to ( https://flatrick.github.io/ )

searches also seem to be broken for notes with spaces in the filename

the graph is also unable to identify which page you're currently on for notes made with a space in the filename.

hey @flatrick, thanks for reporting this issue! Will take a look tonight when I get the chance to

check if the latest version solves your bug :))

It works perfectly now! Thanks @jackyzha0


Installer Downloads No Longer Available

There are no releases available for download.

@mpunderw i can see them all there?
This is the latest
https://github.com/lithnet/miis-autosync/releases/tag/v1.2.8002

That's crazy, when I followed the link from here in github and from the lithnet.io website the list of releases was blank. I can now see them. I even tried on multiple computers. I can see them now. Thanks for your help.

closing


Fix one-click-pr-or-gist private free repos pr creation

Closes #6391
This page affects PR creation in private repos

Screenshot
Without the fix

With the fix

It would be better to check if the "unable to create draft" button exists. Since now if the dom changes at any point we won't see any errors

I think that we just want to leave the drop-down untouched in that case.
Can you craft a selector for that "teams" label and add it as a function in the exclude array? Find "exclude" on https://github.com/refined-github/refined-github/blob/main/contributing.md#featuresadd for more info

Hi @fregante 👋
I'd like to propose an alternative rather than disabling this feature.
If the goal is to keep promoting GitHub for its paid plan, couldn't we just recreate a button next to it that would continue to open the promotion modal? Adding the attribute data-show-dialog-id="drafts-upgrade-dialog" to any button would open it.

No, there's no point in keeping the feature enabled for a button that doesn't work.
We can't hide it (as in your original screenshot) and we should not hide the inline documentation/explanation. So leaving it alone is the easiest thing to do (also because there will be no one around to fix it when it breaks)


Windows-implementation

Is there any windows pre-compiled binaries for this project?
Is it being being developed along-side dnscrypt-proxy?
How do i submit a dnscrypt-server running to resolvers btw?

Hi!
It should work out of the box on Linux, BSD and MacOS, but some changes would be required for Windows.
The best way to submit a server would be a pull request here: https://github.com/dnscrypt/dnscrypt-resolvers


Custom transition for specific NuxtChild

Types of changes

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Hello friends.
I'm playing with named views in Nuxt (2.18) - i would like to attach a specific transition only to one view.
So far i could not find a way to give any  a specific transition.
Looking at the source code it apparently always inherit either parent's or global transition.
Above is a simple change i made that allowed me to pass a transition object to my NuxtChild.
<div id="page">
    <nav id="navigation">
        <NuxtChild name="left" :transition="{name:'nav'}"/>
    </nav>

    <main>
        <NuxtChild/>
    </main>
</div>

Do you think this is worth pursuing with this change ?
Or maybe there is a native solution to this.
Thank you, good day
Checklist:

[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

As Nuxt 2 is currently in maintenance-mode until June 30, 2024, we are not aiming to merge any more features and so, regretfully, I am closing this PR. My apologies that we weren't able to include it in v2.17 🙏


Improve the release process so Token sources will be available on Etherscan

Currently, the release process leaves Token contracts on test networks but for these Token contracts you cannot find the source code on Etherscan.  This issue keeps track of improving the release process so that the token contract sources will be visible on Etherscan.

The first task is to perform a small experiment.  Does Etherscan like Token.sol?

Yes. Etherscan verification works when I manually resolve imports in CustomToken.sol.
Currently etherscan_verify cannot find the addresses of these token contracts easily.  It can find the TokenNetworkRegistry, but it does not provide a list of tokens (for good reasons).

One idea is to read events from TokenNetworkRegistry and figure out which token address is involved.
For the service contracts the situation is easier because these contracts have a publicly readable variable token.

Perhaps, there should be files deployed_token_kovan_0xaabbccdd....json that describes each deployed token.  And then etherescan_verify.py can use the file to call Etherscan.


Update cargo target dir caching

As rustc always rebuilds artifacts between each rustc releases.

Ping for reviewing!

Things never merged. When in doubt, close it!


Update and unpin some dependencies

Our dependencies in main group were pinned since adding the poetry (which means - they were pinned always). This is a bad practice because it doesn't allow the user to install any other version of the used by us library.
I also hope that bumping coverage dependency will fix flaky Windows tests that are pretty randomly crash with AttributeError: 'NoneType' object has no attribute 'startswith' (it points to coverage.py files).

I also didn't update tox to v4 because of two reasons:

tox-poetry was deprecated and there is no stable version that supports v4. (Deprecation because Poetry included official support of tox)
We have a deprecation warning in CI, which would be an error in v4.

I personally have no motivation to fix these issues and I would better go with #488 as solution for those.

Python version was bumped from 3.8.0+ to 3.8.1+ because of similar bump in flake8-bugbear.

I did not take time to investigate the linter error from pyright, but the dependencies look good.

I did not take time to investigate the linter error from pyright, but the dependencies look good.

I already fix those in a2d553180d61b9a492e1fa9d98a20fd134255d8c, so I will just cherry-pick the commit.


It's error, only Chrome(60.0.3112.101), it's fine on firefox or edge. it's sudden.

I'm submitting a ...

[ ] bug report
[ ] feature request
[ ] other (Please do not submit support requests here (see above))

Current behavior:

Expected / new behavior:

Minimal reproduction of the problem with instructions:

AngularJS version: 1.x.y

Browser: [all | Chrome XX | Firefox XX | Edge XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Anything else:

angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:121 Error: [$compile:tpload] http://errors.angularjs.org/1.5.8/$compile/tpload?p0=%2FHome%2FDashboard&p1=-1&p2=
    at angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:9
    at angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:159
    at angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:134
    at m.$eval (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:148)
    at m.$digest (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:145)
    at m.$apply (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:149)
    at l (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:100)
    at J (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:105)
    at XMLHttpRequest.e (angular?v=SoymK9Y9ucGPQikPKd-Bve0nwIqic7YUxFep-8U94IA1:106)

only Chrome(60.0.3112.101),

@shenjielx As mentioned on gitter, it's going to be easier to help you if you can reproduce it.
Also be sure to check:

The Chrome Developer Tools might also be helpful in determining why the request failed.

I'm going to close this issue because we haven't got any feedback. Feel free to reopen this issue if you can provide new feedback.


⚠ Bofrost.lu has degraded performance

In ee0a0eb, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1880 ms

Resolved: Bofrost.lu performance has improved in 25ef1ab after 8 minutes.


Enable chef robotic vacuum cleaner device

Add a chef device - Robotic Vacuum Cleaner.
this device supports rvc  run mode cluster, rvc clean mode cluster, rvc operational state cluster, and power supply cluster.

All read callbacks seem wrong: The requirement from emberAfExternalAttributeReadCallback at https://github.com/project-chip/connectedhomeip/blob/master/src/app/util/generic-callbacks.h#L81 says
 * If the application was successfully able to
 * read the attribute and write it into the passed buffer, it should return a
 * value of InteractionModel::Status::Success

you must fill/read the buffer not just return uninitialized data

Done

@andy31415 can you please take a look at your convenience, thank you


Start documenting the std library functions

We should find a way to publish some API docs, somewhere.

With https://github.com/jkcfg/website/pull/25 I consider what we have as the "start" this issue is about. There are more documentation efforts that should happen but writing documentation is not quite as fun as adding features. So well, little by little :)


self-signed KeyStore usage in spring-security-saml

I have question of following code, what the purpose of it?
If it is only needed for local debug, what should we do for production release?
Thanks
@Bean
public KeyManager keyManager() {
    DefaultResourceLoader loader = new DefaultResourceLoader();
    Resource storeFile = loader.getResource(samlKeystoreLocation);
    Map<String, String> passwords = new HashMap<>();
    passwords.put(samlKeystoreAlias, samlKeystorePassword);
    return new JKSKeyManager(storeFile, samlKeystorePassword, passwords, samlKeystoreAlias);
}``

This method is used at runtime for locating the Java Keystore.  The password is used open the keystore itself and on the individual items in the keystore.  Technically there could be two different passwords.  Not supported by this code though.

Hi @rmkellogg thanks for your answer. Could you please explain more about this part? For example, why we have to declare this KeyManager here? What is this piece of code supposed to do?
I just no idea how to deal with this part when we need to release our app to production, a self-signed keystore seems not a good idea, but what should be used instead?
Thanks


We should fail duplicate registration of storage drivers

At the moment we just overwrite any existing registration (see driver-factory.ts line 38). Failing if there's already a registration will help make our tests more hermetic.

Additionally, we should have a global after-each test that either deregisters everything automatically or asserts that they are deregistered. We have nearly-global test-environment-test.ts files that would need to be threaded in the right place when generating the test-runner in sigh.ts to make them genuinely global.


Experiment assets not created/showing on portal

I am trying to create an asset in my experiment using the https://experimenters.organicity.eu portal and although it says asset created successfully I cannot see them on the list of Experiment Assets.
I thought the issue was that I tried to add an asset with a latitude/longitude outside the experiment region but this was not the reason.
Thank you for your help.

Issue fixed by @boldt
Thanks.


can't build this version

some errors in the pom.xml files.

I change to eclipse sts,  that environment builds successfully.
something may be wrong with my idea environment.


Question: what is the batch size for the wav2seq pretrained model.

In section 3.4 in the wav2vec paper, it says the model is pretrained for 400k steps with 8 gpus. I am curious what is the batch size for each gpu? I couldn't find this information in the wav2vec example here, either.

Also, is the pretrained module freezed during finetuning or its parameters are allowed to change?

@alexeib ?

Any follow-up?

also confused about it.

The default batch size per gpu is 8. I ran it and used pdb to check.

In section 3.4 in the wav2vec paper, it says the model is pretrained for 400k steps with 8 gpus. I am curious what is the batch size for each gpu? I couldn't find this information in the wav2vec example here, either.

Hi, what do you mean by 400k steps? Are that epochs?

In section 3.4 in the wav2vec paper, it says the model is pretrained for 400k steps with 8 gpus. I am curious what is the batch size for each gpu? I couldn't find this information in the wav2vec example here, either.

Hi, what do you mean by 400k steps? Are that epochs?

No, I think it means 400k steps of mini-batches fed into the model during training.

In section 3.4 in the wav2vec paper, it says the model is pretrained for 400k steps with 8 gpus

here how 400k steps is considered. if we train on 8 GPU with batch size 8 then progress bar during training show num_update parameter updated by 1 value in 1 iteration but effectively it should be increased by 8  because in 1 iteration the update is done parallely on 8 batches. if this is the scenario then should we train for 400K/8 i.e 50K update steps or not ????


SPM support

This PR completes #309.
I had to dance a little with some project files as the framework contains mixed languages (Swift + Objective-C). Therefore the package contains a standalone lib for the hex2bin converter which the main lib DFULibrary depends on.
Cocoapods and Carthage both do not need that lib, that's why we use #if canImport(...) which is only required for SP.
The header file is also moved into its own include directory or otherwise SPM would complain about the missing directory.
Also note that I didn't called the library iOSDFULibrary because we do support macOS and it does not make sense to call it that way.

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

cc @philips77

To try out building the project with SPM you need Xcode 11. It's enough to double click on Package.swift. Xcode will open not just the file but an entire project structure like that. You can select the scheme you want and just try building.

@philips77 an optimisation would be if you could re-write the Objective-C code in to Swift, then we could remove the Hex2BinConverter and keep it as an implementation detail as intended. However right now it leaks to the consumer.

Many, many thanks! We will review it after drinking the champagne!
Rewriting the Hex2BinConverter is a good idea. Let me do this first before merging this PR so we could have clean SPM setup.

Regarding the iOS in the name, this is mainly to distinguish from the Android lib, which has Android in the name. macOS support was added... accidentally, as it just works. But I support the idea for publishing as DFULibrary.

@philips77 by the way the package also compiles just fine for tvOS and watchOS.

@philips77 now that we have the chance, we could also use a better name then something with a Library suffix. Maybe NordicDFU?

Hi, have a look at https://github.com/NordicSemiconductor/IOS-Pods-DFU-Library/blob/feature/swift-hex-converter/iOSDFULibrary/Classes/Utilities/HexToBinConverter/IntelHex2BinConverter.swift
(branch feature/swift-hex-converter, commits:
https://github.com/NordicSemiconductor/IOS-Pods-DFU-Library/commits/feature/swift-hex-converter)

@philips77 I only had a very quick glance at the code. I don't really have time atm. to evaluate the correctness of the code according to its specifications, but for that you build some tests which is usually a good thing. :)
If you're confident with the changes feel free to merge it into my branch (you have the rights to do so as a maintainer of the original project).
After that is done, I'll clean up Package.swift file.

I'll merge tomorrow. Yes, let's call them NordicDFU.

Ok, I managed to merge latest commits into your branch, but you need to review it. I discarded all changes in Pods.xcodeproj/project.pbxproj, as they were related to IntelHex2BinConverter.h, but I might have missed something. I had to rebase the branch.
Could you check if it still works and fix if needed? Also, as discussed, let's set the name to NordicDFU.

@philips77 I updated the project. I also added the tests target to the package and fixed some compile-time errors when targeting watchOS (except for one package issue related to watchOS and tests - see comment in Package.swift).
The tests are also re-routed to use the URL of the test file instead grabbing the resources from the bundle. This change enables us to run them from both SPM and from .xcworkspace.

Amazing! Thanks again for all the work.
Is there anything else blocking from releasing? Is there a need for a tag after releasing?

Is there anything else blocking from releasing?

I don't think so. In the future one could clean up the project structure and CocoaPod/Carthage module names, but that's not relevant now.

Is there a need for a tag after releasing?

No sure, but I would guess that it's needed.


Existing plugins can no longer be extended

In sidekick manifest v2, projects were allowed to extend existing plugins, for example hide the "publish" plugin for documents in the drafts folder. This no longer works in the manifest v3 version.
For example, it should be possible to have the following config to only show the publish button on non-draft URLs in preview:
{
  "id": "publish",
  "excludePaths": [ "**/drafts/**" ],
  "environments": ["preview"]
}

:tada: This issue has been resolved in version 6.2.0 :tada:
The release is available on:

GitHub release
v6.2.0

Your semantic-release bot :package::rocket:


[GSSoC'23] Large Factorial Calculator Project Completed

Title and Issue number

Title - Large Factorial Calculator
Issue Number - 2995
Close #2995

Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

@MohitGupta121 @pranjay-poddar @Saurabh-kumar09 sir, please review this pr


sublist is clashing with other elements

list settings are currently impacting majority of lists in the structure, rearange

This sounds like a CSS specificity problem
On Tue, 19 Jan 2016, 11:17 AM Alex Lee<EMAIL_ADDRESS>wrote:

list settings are currently impacting majority of lists in the structure,
rearange
—
Reply to this email directly or view it on GitHub
https://github.com/bvn-architecture/styleguide/issues/137.


playground play button should not be available if the models is not downloaded

The user is able to navigate to the playground of a model which is not on disk leading to an error.
What are the expected behaviour we want to achieve ? We need to improve the playground page in the case the models is not on disk.

Prevent the user to access the playground page if not available on disk (hide the playground tab)
Add a download button in the playground page

cc @jeffmaury

rather than hide the tab, i'd prefer if the extension can check if model is on disk, and if not download it - probably ask user if they want to download it first -  it takes a few mins to download so also give a message that it's downloading.

Add a download button in the playground UI looks better

No longer required


BKR-1089 Fixing Custom Facts added to environment on Windows

This version of the fix removes the unnecessary space

Can one of the admins verify this patch?

Related to #1376

Both #1376 and #1375 are fixes for issue BKR-1089.  Just need to decide which is preferable.

closed in favor of #1376


Compressor FX hover is wrong color

Usually when hovering over one of these icons, it will produce a slightly brighter/highlighted color. However with the compressor, it shows the same color as when it is inactive/bypassed.

Im trying to find the parameter name in Vital 1.5. I did this in the skin editor of 1.0.

Okay I think it was the "Lighten Screen" parameter that was the wrong value in the vitalskin file.
I'll check it out and remove the line later.


Issue running protractor test

I get the following error while running npm run protractor command.

<EMAIL_ADDRESS>protractor /Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat
protractor e2e-tests/protractor.conf.js

[18:51:45] I/local - Starting selenium standalone server...
[18:51:45] I/launcher - Running 1 instances of WebDriver
[18:51:45] E/launcher - Process exited with error code 1
/Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:654
throw error;
^
Error: Server terminated early with status 1
at Error (native)
at /Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/remote/index.js:210:20
at Promise.invokeCallback_ (/Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:1329:14)
at TaskQueue.execute_ (/Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:2790:14)
at TaskQueue.executeNext_ (/Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:2773:21)
at /Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:2652:27
at /Users/akanksha.singh/Documents/AngularWorkspace/angular-phonecat/node_modules/protractor/node_modules/selenium-webdriver/lib/promise.js:639:7
at process._tickCallback (node.js:369:9)
npm ERR! Darwin 13.1.0
npm ERR! argv "/usr/local/bin/node" "/usr/local/bin/npm" "run" "protractor"
npm ERR! node v4.5.0
npm ERR! npm  v2.15.9
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>protractor: protractor e2e-tests/protractor.conf.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>protractor script 'protractor e2e-tests/protractor.conf.js'.
npm ERR! This is most likely a problem with the angular-phonecat package,
npm ERR! not with npm itself.

This error is not informative. It seems more related to selenium or protractor though. Also make sure you have all the dependencies installed (including Java) and the port used by selenium is available.
You could also try setting directConnect: true in your protractor.conf.JS and see if the problem goes away.

directConnect: true helps, thanks!

l trydirectConnect: true,works!thanks @gkalpak !

OK, since we haven't heard from the OP and people seem to be able to work around any issues with directConnect: true, I am going to close this.
(We should consider including directConnect: true by default.)

I am having same issue while using browser internet explorer.
directConnect: true - Doesn't work with internet explorer. Any other leads for IE?


Training with new dataset from previous .dat file

I am following the example "dnn_metric_learning_on_images_ex.cpp" to train the face recognition model with my additional dataset. I see when it runs, it creates / resumes from a sync file in the line:
 trainer.set_synchronization_file("face_metric_sync", std::chrono::minutes(5));
and save the training output to
 serialize("metric_network_renset.dat") << net;
However, I am curious whether it is possible to train the face-recognition model resuming from previous pretrained model provided by dlib named dlib_face_recognition_resnet_model_v1.dat.
Any suggestion will be highly appreciated.

The primary curiosity for this if the accuracy could be improved further by training with the newly released VGGFace 2 dataset containing 3.3M+ faces or parametric tuning on layers.

Unfortunately, I didn't save the sync file.  The public model I saved was converted from one containing batch norm layers to affine, so the batch norm state was lost, which makes loading it into a model with batch norm layers annoying.


Add support for Swift 3

Xcode 8 is out, and it requires that you use either Swift 3, or Swift 2.3 (which is a transitional version intended, if I got it right, to help "modernizing" Swift 2.2 code).

Anyone knowing how to update this project to support Swift 3?


[FR] vim-like navigation in move file to folder switcher

The title says it.
I would appreciate Vim-like navigation with Ctrl-j and Ctrl-k in the move file to folder switcher like what is already available in the recommended search switcher.

Thank you for your FR, @kometenstaub :)
Yes, I'll release it next release 👍

Released in v6.1.0 🚀


Fix(docatl): Trailing slash resulting in 404

fixes: #8

nice thanks :tada: there might be a better solution where we sanitize urls in general after we combined them but for now this is good enough


Reversed order for decorators

This is going to solve #26.

Coverage remained the same at 71.569% when pulling a02121122b1f4402a52df29f03e2d1f1acb85ee0 on puentesarrin:decorators_order into a6746adf778d0782e8f8ebbd196753d196f3764b on bodbdigr:master.

thank you, applying pr

Thanks Valery! I'd love to get a new minor release with this update. Shall I send a new pull request?

sure, shoot :)
On Fri, Jan 6, 2017 at 3:56 PM, Jorge Puente-Sarrín <
<EMAIL_ADDRESS>wrote:

Thanks Valery! I'd love to get a new minor release with this update. Shall
I send a new pull request?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/bodbdigr/restea/pull/27#issuecomment-270919839, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AARKQjK7CJpSh1EESGKMoTbxE1ZsTv0Aks5rPlY2gaJpZM4LcQOl
.