Don't assume exception has a response

I was trying out the sample code and getting a really strange error:
irb(main):003:0> Raven.capture { 1/0 }
I, [2017-01-30T14:50:13.002654 #63295]  INFO -- sentry: ** [Raven] Sending event b9824a076643428786eebb7d96378993 to Sentry
E, [2017-01-30T14:50:13.401824 #63295] ERROR -- sentry: ** [Raven] Unable to record event with remote Sentry server (NoMethodError - undefined method `[]' for nil:NilClass)
E, [2017-01-30T14:50:13.401957 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/transports/http.rb:32:in `rescue in send_event'
E, [2017-01-30T14:50:13.401989 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/transports/http.rb:18:in `send_event'
E, [2017-01-30T14:50:13.402008 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/client.rb:40:in `send_event'
E, [2017-01-30T14:50:13.402025 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:81:in `send_event'
E, [2017-01-30T14:50:13.402040 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:126:in `capture_type'
E, [2017-01-30T14:50:13.402056 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:97:in `rescue in capture'
E, [2017-01-30T14:50:13.402071 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:92:in `capture'
E, [2017-01-30T14:50:13.402086 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/forwardable.rb:189:in `capture'
E, [2017-01-30T14:50:13.402106 #63295] ERROR -- sentry: ** [Raven] (irb):3:in `irb_binding'
E, [2017-01-30T14:50:13.402121 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/irb/workspace.rb:87:in `eval'
E, [2017-01-30T14:50:13.402136 #63295] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/irb/workspace.rb:87:in `evaluate'
E, [2017-01-30T14:50:13.402155 #63295] ERROR -- sentry: ** [Raven] Failed to submit event: ZeroDivisionError: divided by 0
ZeroDivisionError: divided by 0
    from (irb):3:in `/'
    from (irb):3:in `block in irb_binding'
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:93:in `capture'
    from (irb):3
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/console.rb:65:in `start'
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/console_helper.rb:9:in `start'
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:78:in `console'
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
    from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands.rb:18:in `<top (required)>'
    from bin/rails:4:in `require'
    from bin/rails:4:in `<main>'
irb(main):004:0>

I traced it down to the Raven::Transports::HTTP error handler. The underlying bug is that the SSL verification fails but the wrapped OpenSSL exceptions don't have a response:
#<Faraday::SSLError wrapped=#<OpenSSL::SSL::SSLError: SSL_connect returned=1 errno=0 state=error: certificate verify failed>>

Adding the extra check gets me an error that I can start troubleshooting:
irb(main):001:0> Raven.capture { 1/0 }
I, [2017-01-30T23:15:18.948759 #4498]  INFO -- sentry: ** [Raven] Sending event b9c8e4a11a624e6cb3a48da6c8b4133f to Sentry
E, [2017-01-30T23:15:19.338223 #4498] ERROR -- sentry: ** [Raven] Unable to record event with remote Sentry server (Raven::Error - SSL_connect returned=1 errno=0 state=error: certificate verify failed)
E, [2017-01-30T23:15:19.338362 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/transports/http.rb:35:in `rescue in send_event'
E, [2017-01-30T23:15:19.338392 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/transports/http.rb:18:in `send_event'
E, [2017-01-30T23:15:19.338411 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/client.rb:40:in `send_event'
E, [2017-01-30T23:15:19.338428 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:81:in `send_event'
E, [2017-01-30T23:15:19.338444 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:126:in `capture_type'
E, [2017-01-30T23:15:19.338460 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:97:in `rescue in capture'
E, [2017-01-30T23:15:19.338479 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:92:in `capture'
E, [2017-01-30T23:15:19.338495 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/forwardable.rb:189:in `capture'
E, [2017-01-30T23:15:19.338510 #4498] ERROR -- sentry: ** [Raven] (irb):1:in `irb_binding'
E, [2017-01-30T23:15:19.338525 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/irb/workspace.rb:87:in `eval'
E, [2017-01-30T23:15:19.338541 #4498] ERROR -- sentry: ** [Raven] /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/2.3.0/irb/workspace.rb:87:in `evaluate'
E, [2017-01-30T23:15:19.338561 #4498] ERROR -- sentry: ** [Raven] Failed to submit event: ZeroDivisionError: divided by 0
ZeroDivisionError: divided by 0
	from (irb):1:in `/'
	from (irb):1:in `block in irb_binding'
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/sentry-raven-2.3.0/lib/raven/instance.rb:93:in `capture'
	from (irb):1
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/console.rb:65:in `start'
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/console_helper.rb:9:in `start'
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:78:in `console'
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands/commands_tasks.rb:49:in `run_command!'
	from /Users/andrew/.rbenv/versions/2.3.1/lib/ruby/gems/2.3.0/gems/railties-<IP_ADDRESS>/lib/rails/commands.rb:18:in `<top (required)>'
	from bin/rails:4:in `require'
	from bin/rails:4:in `<main>'
irb(main):002:0>

Follow up to #585.

Huge +1. :+1:
This issue shadows and loses information about any real errors raised from Faraday. The  message easily leads people (like me) to chase a nondescript red herring.
In my case, my environment could not resolve a hostname for a proxy connection via getaddrinfo... which isn't very clear from reading NoMethodError - undefined method `[]' for nil:NilClass.
Thanks for putting this PR up there!


Inf-scroll: Change region to 'main' to ignore long tag/following list.

Issue:
Fixes #2789 Infinite scrolling should activate at end of listed claims, not subscription list
I believe it also closes #2957 scroll restoration issues when tags/sub list longer than page.  Looks to be working, and any quirks with Back will be a common (existing) one that is unrelated to extended Tag/Following size.
Remarks:
If you press End on the keyboard, you'll go to the bottom but only 1 extra page will be loaded.  If you scroll back up slowly, inf-scroll will continue to load one page at a time.  I think this is arguably a better behavior than trying to load all the way to the bottom in one-shot;  it's laggy enough as it is.

Ha. Nice fix!


[console-iot] integrate view metadata in device info

Type of change

Refactoring

Description

Replace the previous metadata list with ViewMetaData to view all metadata in a proper format as per latest wireframes

@drcoolsanjeev @ajaypratap003 @rkpattnaik780 @anukritijha    Hey,  I have been logged out of my red hat account and I can't seem to get in(VPN issues). I'm trying to find a fix for it, until then please contact me on<EMAIL_ADDRESS>


Docs: "down the file tree" => "up the file tree"

File trees, like many others in computing (1), usually grow upside-down 😉 The root is typically referred to as the "top", and descending "down" the tree means moving into subfolders... e.g., see the wording used in http://www.linfo.org/directory_tree.html, https://access.redhat.com/documentation/en-US/Red_Hat_Enterprise_Linux/3/html/Step_by_Step_Guide/s1-navigating-cd.html, and https://www.ibm.com/support/knowledgecenter/ssw_aix_61/com.ibm.aix.cmds1/cd.htm (2).
Since cosmiconfig searches parent folders rather than subfolders for config files ("the search continues in ./, ../, ../../, ../../../, etc."), the conventional wording would be that it walks "up the file tree" -- or even more idiomatically, "up the directory tree".
(1) Family trees sometime grow this way too (but only the descendancy kind).
(2) Did anyone else know that cd - toggles back and forth between two directories?!

@msegado Thank you! That does make sense. Would be interested in making a PR with these changes?

Heh, perhaps when I'm at a computer instead of on my phone ☺ Feel free to go for it if you want to though!


Adding problems related to Dynamic Programming

Issue Type:

[ ] Bug Report
[*] Feature Request
[ ] Documentaion

Describe the bug

A clear and concise description of what the bug is.

Possible solution

Describe the solution you thought of.

Screenshots

If applicable, add screenshots to help explain your problem.

@arpit456jain please assign it to me


Verify "notok"

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 34f684d
What command(s) is the bug in?
forge script script/Test.s.sol:Test --fork-url https://data-seed-prebsc-2-s2.binance.org:8545 --broadcast --etherscan-api-key 3......P -vv --via-ir --verify --watch
Operating System
macOS (Intel)
Describe the bug

Help me!!!

Note that you have your API key exposed, so someone can take it. I'd recommend generating a new one.
Without a repro it's hard to see what's happening, and this is bsc testnet which can be notoriously finicky with verification. What's your repo structure like?

this is repo
https://github.com/deed3e/fd-trading-contract

So i deploy in linea test , not verify

Closing as duplicate—this is another instance of verification issues


ZeusMonitor Fails to Initialize on HPC with MIG-Enabled GPUs Due to ValueError When Parsing GPU Indices

Describe the bug
I was trying to monitor energy usage during the training of a neural network using the ZeusMonitor on an HPC server with GPUs configured in Multi-Instance GPU (MIG) mode. However, the ZeusMonitor fails to initialize properly due to the presence of MIG identifiers, causing a ValueError when attempting to parse GPU indices. The expected behavior was for the ZeusMonitor to either correctly handle MIG identifiers or skip the GPU-related monitoring in such environments.
To Reproduce

Minimal training script

import torch
from zeus.monitor import ZeusMonitor

def main():
    device = torch.device('cuda' if torch.cuda.is_available() else 'cpu')
    print(f'Using device: {device}')

    monitor = ZeusMonitor()

    # Training loop and other logic
    monitor.begin_window("training")

    # Simulate some training process
    for epoch in range(5):
        pass  # Replace with actual training logic

    measurement = monitor.end_window("training")
    print(f"Entire training: {measurement.time} s, {measurement.total_energy} J")

if __name__ == "__main__":
    main()

# TODO

The command you ran

python train.py

Environment

Python version: 3.10.x
Relevant framework PyTorch 2.x
docker run command: -
GPU type: NVIDIA GPUs with MIG enabled (e.g., A100 with MIG configuration)

Additional context
The issue arises when the environment variable CUDA_VISIBLE_DEVICES contains MIG identifiers (e.g., MIG-b1e7cc10-ab7b-56d7-be62-934f743c936c) instead of standard GPU indices. This causes a ValueError in the ZeusMonitor when it attempts to parse these identifiers as integers. A possible workaround is to skip ZeusMonitor initialization when MIG is detected, but this is not ideal for environments where energy monitoring is required.

Hi @amirzenoozi, thanks for the issue.
Unfortunately, NVML, which is the NVIDIA-provided software library that gives us the GPU's power and energy consumption, does not support individually measuring MIG slices -- only the whole GPU's power/energy consumption can be measured. We'll at least provide better error messages when we notice CUDA_VISIBLE_DEVICES containing the string MIG.

Hi @jaywonchung, thanks for the reply.
So it mean that I'm not able to use it on the HPC servers? do you have any idea how can I measure the energy in this environment?

"HPC server" is too general for me to suggest anything concrete. However, assuming you are in some environment where you do not have access to the hardware directly nor do you have sudo, unfortunately there aren't many ways for you to get access to power metrics.
Then the next thing is to see whether approximations are acceptable. I think you would still be able to measure the entire GPU's power/energy consumption by temporarily swapping the value of os.environ["CUDA_VISIBLE_DEVICES"] to something like "0" just before instantiating ZeusMonitor and restoring it to the original value right after that. The power/energy consumption of the whole GPU will serve as an upper limit of what you're actually consuming, and the amount of error will be determined by whether jobs are scheduled on the same GPU on different MIG slices at the moment.

Hi @jaywonchung,
Thanks a lot, it works for me and now I am using ZeusMonitor to calculate the energy.


ADC example scaling wrong

The ADC examples scale the ADC codes with VREF/max_sample.
https://github.com/stm32-rs/stm32h7xx-hal/blob/a46841c4465978aeddef3795b4f571c6c25cfcc9/examples/adc.rs#L76-L80
max_sample = ((1 << bits) - 1) << lshift by itself is correct according to its docstring.
https://github.com/stm32-rs/stm32h7xx-hal/blob/a46841c4465978aeddef3795b4f571c6c25cfcc9/src/adc.rs#L790-L793
But the conversion is not. VREF corresponds to 1 << bits + lshift here as in all ADCs that I know of.
c.f. https://github.com/stm32-rs/stm32f4xx-hal/issues/362#issuecomment-941014100 and other places.
The best approach is probably to change max_sample() and its docstring.

I enjoyed the explaination on the f4 thread, thanks! Indeed this is clearly an error in the example, and a footgun in the API.
My preferred way of fixing this would be to add a new method, perhaps pub fn slope(&self) -> u32 that returns 1 << bits + lshift. After that, max_sample can be marked as depreciated.

Yes. Much better solution!


解决issue#6630问题：heartbeat配置bug

fix #6630

please fix the failed CI.
[INFO] There is 1 error reported by Checkstyle 8.9 with codestyle/checkstyle.xml ruleset.
[ERROR] src/main/java/org/apache/dubbo/registry/integration/RegistryProtocol.java:[99] (imports) AvoidStarImport: Using the '.' form of import should be avoided - org.apache.dubbo.remoting.Constants..
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:

@XiaoweiM hi, thanks for your contribution.
Please resolve conficts with the latest master branch.

ok

Close for long time no response.
Please feel free to reopen if you have any question.
If you think these changes still useful in the latest master branch, please submit a new pull request.


Merge ONNX decoder models

What does this PR do?

[x] Merge decoder and decoder_with_past in a single model proto
[x] Check what opset to use
[x] Apply ONNXSIM to simplify subgraphs
[x] Cross-subgraphs initializers deduplication for the merged model

Next steps:

[ ] Need to check if it works for large proto, and how shall we export it(but the support will be added in another PR)
[ ] Need to adapt ORTModels to be able to take benefits(New input use_cache, dummy inputs for past_key_values...)
[ ] Need to check if merged ONNX model works for IOBinding (introduces new input use_cache, but dlpack doesn't support dtype=bool )
[ ] Open discussion: Should it be applied automatically after export or before inference? (Guess no, optimizer won't be able to due with cross-subgraphs optimization)

Fixes #530

Snippets to test the merge:
# Load decoders
decoder = onnx.load(decoder_path)
decoder_with_past = onnx.load(decoder_with_past_path)

merged_model = merge_decoders(decoder, decoder_with_past)
onnx.save(merged_model, merged_model_path)

And test inference:
from transformers import AutoConfig, AutoTokenizer
from optimum.onnxruntime import ORTModelForCustomTasks

model_id = "gpt2"
tokenizer = AutoTokenizer.from_pretrained(model_id)
payload = tokenizer("I love burritos!", return_tensors="pt")
onnx_inputs = {
    "input_ids": payload.input_ids.cpu().detach().numpy(),
    "attention_mask": payload.attention_mask.cpu().detach().numpy(),
    "use_cache": np.full(1, True),
}
shape = (1, 12, 0, 64)
for i in range(24):
    onnx_inputs[f"past_key_values_{i}"] = np.random.uniform(low=0.5, high=13.3, size=shape).astype(np.float32)

sess = ort.InferenceSession(merged_model_path, providers=["CPUExecutionProvider"])
pred = sess.run(None, onnx_inputs)

After the merge, the merged decoder shall have almost the same size as decoder or decoder with past

GPT2

T5 small

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Will likely help for https://discuss.huggingface.co/t/reducing-latency-for-gpt-j/27547

from optimum.onnx.graph_transformations import merge_decoders

import onnx

decoder_path = "/home/fxmarty/hf_internship/optimum/t5_onnx/decoder_model.onnx"
decoder_with_past_path = "/home/fxmarty/hf_internship/optimum/t5_onnx/decoder_with_past_model.onnx"

# Load decoders
decoder = onnx.load(decoder_path)
decoder_with_past = onnx.load(decoder_with_past_path)

merged_model = merge_decoders(decoder, decoder_with_past)
onnx.save(merged_model, merged_model_path)

raises:
Traceback (most recent call last):
  File "<tmp 5>", line 43, in <module>
    merged_model = merge_decoders(decoder, decoder_with_past)
  File "/home/fxmarty/hf_internship/optimum/optimum/onnx/graph_transformations.py", line 260, in merge_decoders
    _unify_onnx_outputs(decoder, decoder_with_past)
  File "/home/fxmarty/hf_internship/optimum/optimum/onnx/graph_transformations.py", line 167, in _unify_onnx_outputs
    raise ValueError(
ValueError: Can not match 1233 with 1174. Make sure your model protos have same outputs, have same data types and are in the same order.

Transformers commit: 3ee958207a9830da6afcf2c20772ef9a2159ca30
Optimum commit: 9c71dd52f71f397bd97da2228691a3c7049f4452
Model exported with:
python3 -m optimum.exporters.onnx --model t5-small --task seq2seq-lm-with-past --for-ort t5_onnx

Can you add merge_decoders in onnx/__init__.py?

python3 -m optimum.exporters.onnx --model t5-small --task seq2seq-lm-with-past --for-ort t

from optimum.onnx.graph_transformations import merge_decoders

import onnx

decoder_path = "/home/fxmarty/hf_internship/optimum/t5_onnx/decoder_model.onnx"
decoder_with_past_path = "/home/fxmarty/hf_internship/optimum/t5_onnx/decoder_with_past_model.onnx"

# Load decoders
decoder = onnx.load(decoder_path)
decoder_with_past = onnx.load(decoder_with_past_path)

merged_model = merge_decoders(decoder, decoder_with_past)
onnx.save(merged_model, merged_model_path)

raises:
Traceback (most recent call last):
  File "<tmp 5>", line 43, in <module>
    merged_model = merge_decoders(decoder, decoder_with_past)
  File "/home/fxmarty/hf_internship/optimum/optimum/onnx/graph_transformations.py", line 260, in merge_decoders
    _unify_onnx_outputs(decoder, decoder_with_past)
  File "/home/fxmarty/hf_internship/optimum/optimum/onnx/graph_transformations.py", line 167, in _unify_onnx_outputs
    raise ValueError(
ValueError: Can not match 1233 with 1174. Make sure your model protos have same outputs, have same data types and are in the same order.

Transformers commit: 3ee958207a9830da6afcf2c20772ef9a2159ca30 Optimum commit: 9c71dd5
Model exported with:
python3 -m optimum.exporters.onnx --model t5-small --task seq2seq-lm-with-past --for-ort t5_onnx

Due to previous export bug. All good now.

The test for t5 can pass only of #601 is merged.

Merged!

I tried it on GPT-J to merge the models and I'm getting the error Detected 1 oom-kill event(s) in StepId=5333998.batch. Some of your processes may have been killed by the cgroup out-of-memory handler. although was running the process with 150GB of memory.


Clarify Section 5.0 - HF review

So the previous section, especially the content on model checking, only addresses 'calibration validation', which is about checking for goodness of fit, as well as any problems with the model fitting process.
This isn't a second round of validation, but is actually what we might think of the 'real' model validation process, whereby the model is tested against an independent set of data, and other evaluation tests are performed like sensitivity analyses, etc.
Meg has suggested having a diagram of the modelling process, and I was thinking about having a section in the manuscript that contains this diagram. Basically this section would outline the template's presumed model development workflow and means that we can shift a lot of the explanatory text out of the template, into that section.
In terms of actual use of the template, that modelling workflow description and diagram would then be presented as an accompanying vignette or document with the template.
Originally posted by @egouldo in https://github.com/egouldo/EcoConsPreReg/diffs

Hmm, I'm struggling to think about this sensibly, sorry. Does this happen in real life, outside of people keeping their training and testing data separate for validation?
Originally posted by @hannahsfraser in https://github.com/egouldo/EcoConsPreReg/diffs
That's ok, maybe we could have a phone chat to clarify. This happens all the time, but in ecology I think we always end up recombining all of the data to fit the model. I guess you could think of the previous checking section as evaluating the model's fit against the training data, and this section 5 as evaluating the model's ability to predict to independent data. Both sets of tests are testing different things, and therefore typically use different test statistics and procedures. The checking section also contains activities that ensure that the model is implemented as it is specified at the end of phase 3, model specification.
Originally posted by @egouldo in https://github.com/egouldo/EcoConsPreReg/diffs

f753664e0919c0cb04295fbb02aa0bf0a9a4827f


feat: spark renaming

We should follow the trademark guidelines: https://spark.apache.org/trademarks.html

The pipeline broke somehow - I will make a new PR: See #41


Not a valid enum value

Two solid days of troubleshooting and I still can't get Meteor to upload some text files.
"The provided value 'undefined' is not a valid enum value of type XMLHttpRequestResponseType" error in cfs_upload-http.js. Judging from peoples comments online, this is just broken right now.

I'm getting the same error. It doesn't seem to affect whether an object is uploaded as far as I can see but it certainly isn't clear what's causing it.


added js for the print button and fixed js link

Added javaScript to the print button and fixed link to the JS file.

No other changes to page so merge should work fine


use anchor element for social links when in game

fixes https://github.com/microsoft/pxt-minecraft/issues/2668
the in-game webview on macos doesn't support any of the window open APIs that the browser provides.<a> elements work though, so this PR replaces the social buttons with <a> elements.
normally i would just replace these for all platforms since <a> elements are usually better for accessibility, but the only way to open a pop up window is with js and i think that's the expected behavior for social buttons. doesn't matter for the in-game experience given that those all open in the browser outside the game anyhow.

@riknoll build failing

@abchatra i'll fix it, but leave this one out of the release anyhow. i'd like more people to test it first so let's save it for the hotfix

Sounds good.


[Template] Error in processing: "map has no entry for key 'advanced'"

Version
kind: OnPremises
distributionVersion: v1.27.9, v1.28.4, v1.29.4
furyctl: 0.29.8
Description
When running furyctl to create a cluster, the following error occurs during template generation:
ERRO error while creating cluster: error while executing cluster creation: error while executing kubernetes phase: error while executing phase: error preparing kubernetes phase: error copying from template: error generating from template files: error applying templates: error processing template: template: hosts.yaml.tpl:81:31: executing "hosts.yaml.tpl" at <.spec.kubernetes.advanced>: map has no entry for key "advanced"

Expected behavior
The template should properly handle the case where .spec.kubernetes.advanced doesn't exist, possibly by skipping the block or using default values.

Resolved from v0.29.8


Returning CDC equipment doc

Create doc with instructions on how to return CDC equipment

Doc created here, closing https://docs.google.com/document/d/1Nzs6TnqprjZv1eiJ3lJMdsUZxEZo2SiCUQakhSGL9bE/edit


Fix formatting for Powershell snippet

@mahilleb-msft,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@mahilleb-msft : Thanks for your contribution to the Azure documentation! The author, @cynthn, has been notified to review your proposed change.

#sign-off Thanks!


Adding a new CLI for consistent management of compression and encryption

Please check if the PR fulfills these requirements

[ ] The commit message is sensible and easily understood
[ ] Tests for the changes have been added (for bug fixes / features where relevant)
[ ] Docs have been added / updated (for bug fixes / features where relevant)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
This is a feature addition to try and partly address Issue #689 wherein we need to have a consistent usage of encryption tooling and standards.
What commits does this PR relate to?

Thank you for your contribution
We hope you stay around and connect with our growing community!

I added two single line comments to show one of the approaches that we can then reuse when this is applied elsewhere, it creates a simple boolean control included with the modified JSON payload. That should give us a simple if statement as a control, if the booleans are there, we decrypt and / or decompress, if they are not we have a plain text JSON.

Hi @lgriffin
I have tried just focussing on the simple compress/decompress functions initially. I can successfully integrate into the generator, and have verified that they work successfully, achieving approx 5x compression.
However, when the compressed file is fetched in the reader, we get "incorrect header check" consistently. In debug, we can see the correct number of raw bytes coming down, pako cannot inflate correctly.
Perhaps it is related to the compression/encoding? Particularly compatibility with http/text?
In addition, the controller strategy looks interesting. It could, however, be shortcutted perhpas by the upgraded generator generating a "tutors.bin" instead of "tutors.json". The reader then can attempt to fetch the "bin" version. If present, assume it is encrypted and compressed. If not present, then we fall back to fetching the json?


add tabs test

Please makes sure these boxes are checked before submitting your PR, thank you!

[ ] Make sure you follow Element's Contributing Guide.
[ ] Rebase before creating a PR to keep commit history clear.
[ ] Add some descriptions and refer relative issues for you PR.

@Leopoldthecoder fixed


Support Apple's APNS Token Authentication

(Issue originally opened by @EricNetsch in ParsePlatform/parse-server#3315)
I would like to start using Apple's new push notification authentication protocol instead of using traditional certs. Is this possible with parse-server yet? This would be a fantastic addition to the platform to simplify push certificate management.

Provider Authentication Tokens
To securely connect to APNs, you can use provider authentication tokens or provider certificates. This section describes connections using tokens.
The provider API supports the JSON Web Token (JWT) specification, letting you pass statements and metadata, called claims, to APNs, along with each push notification. For details, refer to the specification at https://tools.ietf.org/html/rfc7519. For additional information about JWT, along with a list of available libraries for generating signed JWTs, see https://jwt.io
A provider authentication token is a JSON object that you construct, whose header must include:
The encryption algorithm (alg) you use to encrypt the token
A 10-character key identifier (kid) key, obtained from your developer account
The claims payload of the token must include:
The issuer (iss) registered claim key, whose value is your 10-character Team ID, obtained from your developer account
The issued at (iat) registered claim key, whose value indicates the time at which the token was generated, in terms of the number of seconds since Epoch, in UTC
After you create the token, you must sign it with a private key. You must then encrypt the token using the Elliptic Curve Digital Signature Algorithm (ECDSA) with the P-256 curve and the SHA-256 hash algorithm. Specify the value ES256 in the algorithm header key (alg). For information on how to configure your token, search Xcode help for the phrase “Configure push notifications.”
A decoded JWT provider authentication token for APNs has the following format:
{
"alg": "ES256",
"kid": "ABC123DEFG"
}
{
"iss": "DEF123GHIJ",
"iat":<PHONE_NUMBER>
}
NOTE
APNs supports only provider authentication tokens that are signed with the ES256 algorithm. Unsecured JWTs, or JWTs signed with other algorithms, are rejected, and your provider server receives the InvalidProviderToken (403) response.
To ensure security, APNs requires new tokens to be generated periodically. A new token has an updated issued at claim key, whose value indicates the time the token was generated. If the timestamp for token issue is not within the last hour, APNs rejects subsequent push messages, returning an ExpiredProviderToken (403) error.

https://developer.apple.com/library/content/documentation/NetworkingInternet/Conceptual/RemoteNotificationsPG/CommunicatingwithAPNs.html
Please advise on any existing solutions/ implementations.

@EricNetsch I was planning to do this but I wasn't sure about the implementation. The dependency used here, APN, already supports using APNS authentication tokens. But it's a major update on their package, that has some breaking changes to the one currently used here.

@natanrolnik Ok, got it. It definitely adds some complexity to the token management and would require a cache per "defaultTopic" or bundle Id. This feature  is critical for one of our projects which supports multiple iOS applications on one parse-server (multiple bundle ID's). Please keep me updated and let me know how I can help

@EricNetsch if I remember correctly, you can send push to multiple applications with different bundle ids in the current versions. Take a look here, and check how you can pass an array of configurations.

I would also love to see this supported.

@NatanRolnik that's not good solution. We need to be able to support dynamic bundle ID's without re-deploying the server code every time we have a new app. Using the tokenized method is the right solution

@EricNetsch if I remember correctly, the module apn also requires the bundle id, otherwise it's not able to package the JWT. I might be wrong, but if that's the case, it would need re-deploying anyway.

Yes, it does require a bundle ID to create a token. The idea is that you could pass it the bundle ID (dynamically) to create a token under the same team ID. This would allow us to create tokens for new bundle ID's without having to re-deploy/update the server code. The bundle ID could be passed as a parameter when creating a token -> sending a push

There is a PR open that enables support for APNS Auth Token. You can look at it here: #52
However, @EricNetsch, the current implementation on that PR does require a deploy in order to update bundle IDs.

@natanrolnik with the release of v2.0.0 that features APNS v2 is that issue still relevant?

🙏🏻🎉


Fastlane match: creates a signing certificate for each Apple Id/Git branch combination

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I read the Contribution Guidelines
[ ] I read docs.fastlane.tools
[ ] I searched for existing GitHub issues

Issue Description
Fastlane match creates a signing certificate for each combination of Apple Id/Git branch. As an Apple Id can only have two signing certificates, this restricts us to having two branches in the Git repo that holds certificates/provisioning profiles.
Although, can get around this by creating additional Apple Ids (one per Git branch) is it possible for fastlane match to only create a signing certificate for each unique Apple Id rather than Apple Id/Branch combination.
This would allow me to use a single Apple Id and create a separate branch in Git repo for each product/team.

The reason it does this is that each team_id/apple id combo has its own organization per branch. Why would you want a separate branch when the team_id/appleid combo is the same?

Hi svanzoest. A separate Git branch in the certificates repo for each (Xcode) project, perhaps? Not a deal breaker by any means, just wondering if it was an option to support both scenarios.

@dh256 I see. I do not think that is currently supported as it wouldn't know if the certificate already exists in read-only mode.

@dh256 I believe @svanzoest latest response is correct so I'm going to close this issue for now 🙃


Can't filter point cloud with rqt

i work with velodyne as lidar and an optical camera and i want the topic of features_extraction to appear in rviz but the only points that appear are velodyne_points i want the calibration with my own data .   I have changed the params.yaml and camera_info.yaml according to my dataset.                                                                                distortion_model: "plumb_bob"
width: 1280
height: 720
D: [0.137822,0.35667,-0.035565,0.006467]
K: [975.093708  , 0,      649.525527 ,0.       ,<PHONE_NUMBER>8, 310.223634,0.           0.           1.        ]]
Topics
camera_topic: "/rtsp_stream"
camera_info: "/camera_info"
lidar_topic: "/velodyne_points"
Dynamic rqt_reconfigure default bounds
feature_extraction:
x_min: 3.0
x_max: 7.0
y_min: -3.0
y_max: 3.0
z_min: -1.7
z_max: 0.0
k: 50
z: 1
voxel_res: 0.4
Properties of chessboard calibration target
chessboard:
pattern_size:
height: 7
width: 5
square_length: 70
board_dimension:
width: 45.7
height: 58.7
translation_error:
x: 2
y: 2                                                                                I am currently trying your v2 package as well. Still the dynamic rqt window can't apply to my data. I can't see the pointcloud data change after I apply the filter. Do I suppose to wait longer? Btw, I made to few changes to let the package compile with my ROS Noetic version.

Has this issue been solved? @EMIR-Mh01


Feature request: Enhance Splitter to store positions to localstorage so users setup stays consistent after refresh

Not a high priority, but i figured it could be a cool enhancement.
If a user is finished using the new (AWESOME as it is!) splitter to setup the workspace to their liking, after a refresh, its back to defaults. I think it could be cool to have it remember the splitters new positions (:
Hope you have a good one!

in 4.2.27

[x] Splitter: add attribute data-save-state, required element ID. If true, panes sizes stored into Storage


Model request: a dedicated coding model

I could have sworn I saw CodeLlama 7B in WebLLM at some point, but it must have been a hallucination.
I was wondering if WebLLM could add support for a dedicated code instruct model. Something around 7B perhaps?
It seems there are discussions on Reddit about which coding model is good about every single week. The latest one being this one (May 2024).

As you have probably noted, Qwen2.5-Coder-1.5B and Qwen2.5-Coder-7B are added to the prebuilt list as of ^0.2.63. Will close this issue for now, feel free to open new ones for other requests!


Remove the ModelProvider interface

Motivation and context

It was only used in the Enterprise version, so I moved it there.
How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

Bug Fixes

Removed the ModelProvider interface, streamlining the management of model providers.
Removed the ModelProvider export, updating the module's interface.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 73.88%. Comparing base (12f886c) to head (f98db2a).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #8860   +/-   ##
========================================
  Coverage    73.88%   73.88%
========================================
  Files          408      408
  Lines        44104    44104
  Branches      3986     3986
========================================
+ Hits         32586    32587    +1
+ Misses       11518    11517    -1

Components
Coverage Δ

cvat-ui
78.35% <ø> (+<0.01%)
:arrow_up:

cvat-server
70.08% <ø> (ø)


Separate Timelocks scripts from Plutus scripts

There are several places in the codebase where it appears that we are handling generic scripts (ie both timelocks and plutus), but we are in fact only interested in one of the two types (or more precisely, one of the two constructors of Script). Where possible, we should use types that make this explicit and remove the need to filter.
The worst offender may be validateScript, which only validates timelocks:
https://github.com/input-output-hk/cardano-ledger/blob/2505b7f103c78ee5a230b3e302e31d9607fffbd5/eras/babbage/impl/src/Cardano/Ledger/Babbage.hs#L138-L139
Any use of isPlutusScript is also probably suspect:
https://github.com/input-output-hk/cardano-ledger/blob/2505b7f103c78ee5a230b3e302e31d9607fffbd5/eras/alonzo/impl/src/Cardano/Ledger/Alonzo/Scripts.hs#L137
And similarly isNativeScript:
https://github.com/input-output-hk/cardano-ledger/blob/2505b7f103c78ee5a230b3e302e31d9607fffbd5/libs/cardano-ledger-core/src/Cardano/Ledger/Era.hs#L153

closed by #2953


Adding accessPolicies to existing keyVault results in error validating property "name"

Bicep version
Bicep CLI version 0.1.226-alpha (eb3d221626)
Describe the bug
Trying to add an accessPolicy to existing keyvault using the name '${keyVault.name}/add'
This results in following error message when trying to build:
Error BCP036: The property "name" expected a value of type "'add' | 'remove' | 'replace'" but the provided value is of type "string".

The property name should be KeyVaultName/add, but bicep validates the whole name against the valid names add/remove/replace instead of just the child-part of the name.
To Reproduce
Add an accessPolicy to existing key vault with following bicep code:
resource keyVaultAccessPolicy 'Microsoft.KeyVault/vaults/accessPolicies@2019-09-01' = {
    name: '${keyVault.name}/add'
    properties: {
        accessPolicies: [
            {
                tenantId: functionApp.outputs.principalTenantId
                objectId: functionApp.outputs.principalId
                permissions: {
                    certificates: [
                        'get'
                        'list'
                        'create'
                        'update'
                    ]
                }
            }
        ]
    }
}

Where keyVault is a Key Vault I just created in the same bicep file and functionApp is a nested deployment that deploys a functionApp with a managed identity.
Additional context
Seems like this is related to the discussion about introducing a parent syntax in #657

thanks for reporting
this is a bug we'll need to fix, but in the meantime you can unblock yourself with the any() function:
resource keyVaultAccessPolicy 'Microsoft.KeyVault/vaults/accessPolicies@2019-09-01' = {
    name: any('${keyVault.name}/add')
    ...
}

Thanks @alex-frankel, your workaround worked like a charm as always!

@anthony-c-martin - will this be solved by #657? Or is this a KV swagger issue?

The swagger looks correct - #657 will fix it this.

I'm not sure if it's worth raising this here or as a seperate issue.
The workaround suggested by @alex-frankel is working fine for the actual deployment but it's failing for me when using the --confirm-with-what-if flag (using az deployment group create). I get the following error:
ERROR: DeploymentWhatIfResourceError - The request to predict template deployment changes to scope '/subscriptions/.../resourceGroups/...' has failed due to a resource error. See details for more information.
DisallowedResourceOperation - The operation 'read' on resource type 'vaults/accessPolicies' is disallowed.

I'm an owner on the subscription so shouldn't have any problems with resource permissions. Is it possible there's another issue besides just the parent-child/validation?

Fixed with #657

This issue still exists in bicep v0.10.61. If you are adding access policies and referencing the KeyVault as a "parent" you get the same error.


CM and CHT minor optimizations

Okay, basically we just introduced temporarily the concurrent chain so that we can set its tip, which is needed for the activation without removing concurrent chain from the code base.


NO data in tables ApimAbnormalResponseTimeAlert, ApimAbnormalBackendTimeAlert (APIM 3.2.0)

Hi,
we are using wso2 apim 3.0.0 and analytics ,  we want to work with alerts feature, above mentioned alerts are important for us, we have done configurations as per wso2 documentation, we getting other alerts except these two, not alert on email as well as in database no data.
what could be probable issue?

Related Duplicate issue: https://github.com/wso2/analytics-apim/issues/1476


Allow distribution on Curseforge

It looks like you've toggled this off on Curseforge project's settings:

Could you toggle it on again? It allows users to download your mod on external launchers like GDLauncher, which especially matters on modpacks (that would otherwise be partial).

Is there a reason you can't use modrinth? Consider also that massive mods like Iris and Sodium aren't even on CF anymore. The reason I have it on CF is because it gets more downloads than modrinth. If everyone just used modrinth (since it's better), then I wouldn't worry about CF at all.

As a user, of course I can.
As a modpack dev, I do use Modrinth as well, but would also like to give my users choices on what launcher to use.
The problem with this toggle is that it essentially locks CurseForge modpacks to using CurseForge launcher, because other launchers would only get a partial pack. But as you said, Sodium and Iris are currently no longer available on such launchers as well, so it is up to you.


Parameter of type "associative array"

Let's use the "pet store" application as example
Imagine that you can send multiple "find" requests in one. For that, you pass an array of associative arrays, which contain a list of integers
Text version:
criteria[0][status]: 1
criteria[0][tags]: 2,3
criteria[1][status]: 2
criteria[1][tags]: 4,5

POST /find HTTP/1.1
Host: petstore.com
Content-Type: application/x-www-form-urlencoded

criteria%5B0%5D%5Bstatus%5D=1&criteria%5B0%5D%5Btags%5D=2%2C3&criteria%5B1%5D%5Bstatus%5D=2&criteria%5B1%5D%5Btags%5D=4%2C5

How would you define that ?
I suggest the following syntax addition : adding a new property "indexType" to give the type of the keys in an array (default: integer, 0-based)
Example definition using this syntax
swagger: '2.0'
info:
  version: '1.0.0'
  title: Swagger Petstore (Simple)
  description: A sample API that uses a petstore as an example to demonstrate features in the swagger-2.0 specification
  termsOfService: http://helloreverb.com/terms/
  contact:
    name: Swagger API team
    email<EMAIL_ADDRESS>    url: http://swagger.io
  license:
    name: MIT
    url: http://opensource.org/licenses/MIT
host: petstore.swagger.wordnik.com
basePath: /api
schemes:
  - http
consumes:
  - application/json
produces:
  - application/json
paths:
  /pets/find:
    get:
      description: Returns multiple search query results at the same time
      operationId: findPets
      produces:
        - application/json
        - application/xml
        - text/xml
        - text/html
      parameters:
        - name: criteria
          in: query
          description: search criteria
          required: true
          type: array
          items:
            type: array
            indexType: searchCriteria
            uniqueItems: true
            items:
              type: array
              items:
                type: integer
          collectionFormat: csv
      responses:
        '200':
          description: pet response
          schema:
            type: array
            items:
              type: array
              $ref: '#/definitions/pet'
        default:
          description: unexpected error
          schema:
            $ref: '#/definitions/errorModel'
definitions:
  searchCriteria:
    type: string
    enum:
    - tags
    - status
  pet:
    required:
      - id
      - name
    properties:
      id:
        type: integer
        format: int64
      name:
        type: string
      tag:
        type: string
  errorModel:
    required:
      - code
      - message
    properties:
      code:
        type: integer
        format: int32
      message:
        type: string

Hi there,
Any news on that? Associative arrays are all over the place in APIs (at least then ones I worked/work on)
I suggest using the simple "keys" parameter:
name: pop
description: population by country
type: array
keys:
	type: string
	description: country code (2 characters)
items:
	type: integer
	description: population (in thousands)

Do you think it is feasible to add it? For now I am using "x-keys" with the same format, but it won't display in the generated documentation (unless I'm mistaken)

Can you reference any public standard that defines such a format?
Can you reference any public common API that exposes that format?
Right now, the usage of this format seems esoteric.

On the contrary, can you firmly state that absolutely no API would have any use for associative arrays?
Just one example (because I do not know each and every API out there):
https://apidocs.mailchimp.com/api/2.0/lists/webhooks.php
Here the "actions" is a definite list that keeps growing. What if one day you could have any number of "actions" here, or even custom actions? The solution you propose would be to change the format of returned data so that it is not an associative array anymore, but an indexed array instead. Which would break existing code calling that API.
Instead, having an associative array here (which is nothing more than a generic object), would allow for a seamless transition.
Just my 2 cents. And authorizing a new property "keys" for "type: array" data does not seem that much esoteric to me. As long as something is constant (here it would be the items/values), it maskes sense to allow its declaration imho.
Someone else asked for it here : https://github.com/swagger-api/swagger-spec/issues/289
The given example is a list of data by country, with the country 2 letters code being the key (this is the example I quoted in my comment).

I can't say that, but I can say that we don't aim to accommodate for any kind of API out there, so just because there is a way to do it, that's not reason enough for us to support it.
As for the mailchimp API, that doesn't look anything like what the OP requested. It's just an array of objects which is already supported by the spec (though please correct me if I'm wrong).
As for defining a generic object, you can do that today too. It's ill-advised, and we don't encourage it, but you can. That's pretty much saying you don't want to document your API (in most cases, not all).

JSON Schema has something like this:
name: pop
description: population by country
type: object
patternProperties:
  '[A-Z]{2}':
    type: integer
    description: the population of the country indicated by the property name.

This syntax is not imported by Swagger, though. I guess if this would be supported, that would be the way to go.

That's indeed what I ended up using after some research. If Swagger does
not support it already, I'm confident they will soon!
Thank you Paulo!
Le lun. 14 sept. 2015 à 16:50, Paŭlo Ebermann<EMAIL_ADDRESS>a
écrit :

JSON Schema has something like this:
name: popdescription: population by countrytype: objectpatternProperties:
'[A-Z]{2}':
type: integer
description: the population of the country indicated by the property name.
This syntax is not imported by Swagger, though. I guess if this would be
supported, that would be the way to go.
—
Reply to this email directly or view it on GitHub
https://github.com/swagger-api/swagger-spec/issues/380#issuecomment-140105023
.

The JSON API Spec actually has this for it's "field" query param.
/?fields[{TYPE}]={TYPE_FIELD1},{TYPE_FIELD2}&fields[{ANOTHER_TYPE}].....
http://jsonapi.org/format/#fetching-sparse-fieldsets

Rack is a quite common piece of software used in frameworks like Rails and Sinatra that has long parsed query parameters like array[]=1&array[]=2&array[]=3 into an array and hash[key]=value1&hash[another_key]=value2 into dict/hash types. Even then some parameters like hash[key_containing_array][] parse into a hash which contains an array.
This has been like this for a very long time. See https://github.com/rack/rack/blob/1.6.4/lib/rack/utils.rb#L111-L168 for their code.
I'd say there are tons of use cases out there like these where key and another_key are totally free-form. Probably less of those where hash is also completely free-form, but still existing.

:+1:

+1

Link to parent #565

+1
standard feature in all rails applications [1], also seems the only reasonable way to pass 1 or more! nested maps on through query strings. ie for essential for a GET request
[1] http://guides.rubyonrails.org/action_controller_overview.html#hash-and-array-parameters

Does anyone have a solution for this in the meantime? Obviously can't leave an endpoint undocumented.
I'm assuming something like this work work in the meantime?

I'm having a similar issue with array parameters. Does anyone have any suggestions how to describe an associative array in the query request such as:
http://example.com/api/rest/content?type=search&id=test&options[start]=20&options[limit]=10

Hello,
stripe uses this for search by date API
something like
GET /charges?date[gte]=2016-01-01&date[lte]=2016-01-02

@nemenemsrouge yes!, that's the same request pattern in the Stripe date API.
They are using date[gte] and date[lte] similar to how we are using options[offset] and options[limit] etc..
Is there any example of how to define that style date API GET request via the Swagger / OpenAPI spec?
I had a google but it appears the Stripe API docs are generated via a custom in-house tool ( https://www.quora.com/What-software-powers-the-Stripe-API-documentation )
Thanks for your reply.

@davidwhthomas  I m sorry that I have no idea about how to do that...
Do you think using a "query" parameter and a type=object can do the job?
Maybe have to try...

Technically, you can describe that today by naming the parameter date[gte] and setting the type to string.

... but that does not scale to M fields by N operators. date[gte], date[gt], date[lte], date[lt], date[ne], then same for fields a, b, c. ... x, y, z,  (We don't use notation such as this in our APis; I'm just noting that it is a stretch to enumerate all combinations.)

I didn't say it's a holistic solution, it can still be used to define explicit parameters if that's a requirement. I'm not familiar with Stripe's date API and it wasn't described as having varied inputs, only two. For two, it's definitely doable.

Thanks a lot. I test that today !

Is there any reason something like this won't work?
it works great when changing the parameter to body, it could just send the get param as
param1[0][name]=bla&param1[1][name]=foo
swagger: "2.0"
info:
  description: |
    This is a sample server Petstore server.
  version: "1.0.0"
  title: Swagger Petstore
  termsOfService: http://helloreverb.com/terms/
  contact:
    name<EMAIL_ADDRESS>  license:
    name: Apache 2.0
    url: http://www.apache.org/licenses/LICENSE-2.0.html
basePath: /v2
schemes:
  - http
paths:
  /test:
    get:
      summary: test
      description: test
      parameters:
        - name: param1
          in: query
          required: false
          description: test param1
          schema:
            type: array
            items:
              $ref: '#/definitions/bla'
      responses:
        200:
          description: OK
definitions:
  bla:
    type: object
    properties:
      name:
        type: string
        description: name of the object

See solution in https://github.com/OAI/OpenAPI-Specification/pull/804


Include the Line Cookbook as a resource

Describe the resource:
Please include the line cookbook
Why should this be included out of the box?:
It's almost as useful as the file cookbook
Current cookbook implementation:
Yes.
https://github.com/sous-chefs/line

Buried on @lamont-granquist's laptop is a CFEngine inspired set of file edit resources that we'd like to add to core instead of the specific ones in the line cookbook. We just need to clone @lamont-granquist so he has time to do that.

Yeah the answer to including line directly is 'nope' and this duplicates #7043.


chore: better upgrade messages

Adding an instruction into our "Update available" message, similar to how npm and yarn handle the case.
Before:

After:

Related: #212.

Hi, I've implemented the installation detection feature on my local clone.
But I can't push it directly to this branch because of permission issues. Could you edit this PR to allow edit from maintainers?	https://help.github.com/en/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork#enabling-repository-maintainer-permissions-on-existing-pull-requests


Clarifies that command removes all containers

Edit to clarify that govuk-docker compose rm -sv removes all Docker containers, not just govuk-docker containers.

Does it? Trying myself, I can't see that. Can you give an example?

Does it? Trying myself, I can't see that. Can you give an example?

It would seem that I can't - I'll chalk this one up to me being daft. Apologies!


Rename apidoc to api builder

Sorry, didn't see this PR until after I'd made the changes myself. Thanks for doing it!


Add support for reporting metrics to SignalFx

Adds a metrics reporting service to report the Kafka monitor metrics to SignalFx.
Suppress checkstyle of LinkedIn Corp. headers for SignalFx files

SignalFx is monitoring platform that performs operational intelligence for data-driven DevOps

Hey @rohitvyavahare, I just got reply from open source team at LinkedIn. It looks like we would liek you to confirm that you can license this patch to us under Apache 2 license. Would you be OK to delete “All rights reserved” and add “Licensed under the Apache 2 License”?
Ping @MBShannon for more comments.

@rohitvyavahare Another question here is that could you confirm this code is written by you? The open source team would like to double check.

@rohitvyavahare Another question here is that could you confirm this code is written by you as part of SignalFx? The open source team would like to double check.

@lindong28 Thank you for getting back to us. Yes, I am fine with changing "All rights reserved" to "Licensed under the Apache 2 License". For the second question, what form of confirmation you need?

@rohitvyavahare Thanks! For the second question, I guess it just needs verbal confirmation. @MBShannon asked about this.

@lindong28  @MBShannon I confirm that I have written above code on behalf of SignalFx, Inc. Also, I have updated files with Licensed under the Apache 2 License

@rohitvyavahare Thanks for the update. It seems that the patch is updated on July 30 to include the average per-broker latency. In general Kafka Monitor attempts to monitor Kafka cluster performance from client (i.e. producer and consumer) point of view and those users probably should not have to know urls of each broker in the cluster. I can see the value of monitoring per-broker latency for system administrator. On the other hand today our system administrator can already monitor this per-broker performance by watching e.g. kafka.network:type=RequestMetrics,name=LocalTimeMs,request={Produce|FetchConsumer|FetchFollower} which is available the Kafka server metrics.
So it seems that the newly added metrics kind of deviate from kafka-monitor's intended use-case and its functionality overlaps with the existing metrics in Kafka server. I am not sure we need this. Since main goal of this patch is to export the existing kafka-monitor metrics to SignalFx, can we remove this part from this patch and put it in a separate PR if needed?

@lindong28 I see your point. I have updated PR with only SignalFx metric reporter. I will create new PR for broker metrics. We can discuss more in that PR. Thanks

Thanks for the patch @rohitvyavahare. Merged to master branch.


Install problems pkg-config-0.28 on Mac (Mavericks)

Hi, I keep getting configuration errors / permission issues installing the pkg-config-0.28. file
Something about GLIB_CFLAGS and GLIB_LIBS seem to be wrong and 'have to be set to the correct values or pass'.
That's above my head to begin with, but after some searching I tried the following code in terminal:
pkg-config-0.28 Jon$ ./configure --with-internal-glib && make install
But still the permissions don't come thrue.
I'm a newby at the Github and with programming with terminal so any help in the right direction is more than welcome!
Here's my latest log:
terminal log_17-10.pdf

I see a few permissions errors, you may want to make sure you sudo the configure and make commands.
Are you unable to install Homebrew? It might greatly simplify the installation of pkg-config (and the rest).


ionic popover items have no tap effect with ng-click

However, with ng-href and ui-sref they do.

Greetings @PureSpider!
My sensors indicate that I require more information in order to assist you.

Can you provide a detailed description of this issue?
How would a robot go about replicating this issue?
On which platform does this happen occur? (iOS, Android, all, etc.)
On which platform versions does this occur? (Android 4.2, iOS 8, all, etc.)
Is this an issue with the Ionic Framework or Ionic CLI?

Additionally, providing a codepen that replicates the issue is extremely helpful.
If you wish to get help using the framework itself, I recommended visiting the forum, where many humans will be able to offer wisdom and advice.
Thank you for allowing me to assist you.

Here is a codepen showcasing the issue on android (seems to work fine on iOS): http://codepen.io/anon/pen/QwmLJZ
The lighter background is missing from the item with ng-click and it has no tap effect.

Ah, ok. So it's a display thing, not that the event isn't firing. Thanks.
Note to self: the activated class isn't being applied to elements in the android popver style that have ng-click attributes.

@perrygovier, did we ever get the grey background issue fixed with ng-click?

Hello all! Is this still an issue with the latest version of ionic? Thanks!

If the codepen is still relevant (assuming the URLs didn't change) then yes, it's still an issue.

What happened with this issue?

This issue was moved to driftyco/ionic-v1#3


Failed header parsing does not close the FileMetadata

In the case of some PE-files the format is recognized, however the header cannot be correctly parsed. One example of this is (VirusTotal hash d7e56505e1081c9818b7d742fb155deb52f322a33b58854444de6d283adb0fa6), which prints the Failed to parse PE headers: read out of bounds error in the UI log. Using the API's bn.binaryview.BinaryViewType['PE'].open(filename) or bn.binaryview.BinaryViewType.get_view_of_file(filename) returns None and the underlying BV's FileMetadata cannot be closed manually using the bv.file.close() nor is closed automatically as it is probably assumed that the file will be worked on later, as is the case of UI, which returns the Raw view with the FileMetadata handle reference. However this is an issue when these binaries are to be skipped in batch processing, as the underlying file handle leaks memory.
Additionally the bn.binaryview.BinaryViewType.get_view_of_file(filename) throws an exception as by default it tries to run the analysis of a NoneType. This can be easily mitigated using an if condition on the result of cls[available.name].open(filename).
if update_analysis and bv != None:
    bv.update_analysis_and_wait()
return bv

One solution to the file closing problem would be to return the raw view instead of None, when the parsing is unsuccessful. However this change would have a huge impact, as the documentation clearly states, that get_view_of_file returns the first available, non-Raw BinaryView available. and as such is an entry-point for many scripts written on API.
Using BN ver. 1.1.1259.
I would like to use this issue as a starting point for a discussion about the API's BinaryView creation, which is in comparison with the UI slightly confusing and less verbose.

Fixed in 1.3.2059-dev (ad37832dd4ffa112b10b523a151d3fa9b86b6c9d).


coloredlogs.install(level) VS logging.basicConfig(level)

Why do you choose to set log level on handler object instead of logger object?
There are significant performance difference when logging to disabled log levels.

Hi Alex, thanks for the feedback and sorry for the late reply.
Disregarding performance for a moment, I believe that loggers should perform no filtering of log messages at all because conceptually the decision to filter or not belongs with output handlers (that are unrelated to each other and have different concerns).
For example, I frequently like the tools I create and use to log up to INFO level on the terminal (by default) while logging up to DEBUG in the system log. If I run the tool with a --verbose option it should switch from INFO to DEBUG on the terminal, and if I run the tool with a --quiet option it should switch from INFO to WARNING on the terminal without affecting the verbosity logged to the system log.
Separate verbosity levels for different output handlers as described above avoid the problem of "I hit a serious issue but now can't reproduce it and I had the logging silenced so don't have any clues about what happened", basically it enables post-mortem debugging based on log messages, even if the user didn't see the log messages while running the tool.
Now I do acknowledge the performance aspect. What might be an option for coloredlogs is to take the "most verbose" logging level of all registered output handlers (even those not installed by coloredlogs) and set that level on the root logger object. This would be purely for performance reasons then (the reason you reported this issue, IIUC) so to be honest I'm not sure if it's worth the hassle and potential for introducing new issues (backwards incompatible changes breaking the assumptions made by existing users). I guess I'd have to benchmark the difference :-).
What do you think, would a change like described in the previous paragraph be an improvement for you? And would that be the change that convinces you to use coloredlogs while you might not have done so otherwise? (trying to gauge interest :-)

Hi again Alex. Recently I released coloredlogs 6.0 with two big changes:

When you do provide a logger to coloredlogs.install() it will not change the root logger's level.
When you don't provide a logger to coloredlogs.install() will (still) select the root logger and (still) increase the verbosity of the root logger, but now only up to the required log level instead of blindly setting the root logger's verbosity to logging.NOTSET.

I believe this implements the logic you desire, so I'm closing this issue now.


Ticket notifications don't link to exact ticket, just to tickets page

Is this still a bug @eatonphil ?


[CORE-1037] add startedTime marker for ListCommit

This PR adds support for marker based pagination for the ListCommit API. The number parameter is reused to limit the number of results returned from the API and a startedTime parameter is introduced as the marker. Consumers of the API can provide the commit started time of the last commit in their previous ListCommit call to get a new page of results.

 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.

Why did several random comments get modified in this PR?

This is auto-formatting changes from my IDE. The PR touched a few more files before I removed the go client changes so that's why you see those other changes here. I'll remove them before merging


BUG: Toggle button alignment in dark mode

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?

The toggle button alignment is good in light mode but it needs to be corrected in dark mode
Kindly assign me this issue.
Best regards
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Is there an existing issue for this?

[x]  I have searched the existing issues

What happened?
    The toggle button alignment is good in light mode but it needs to be corrected in dark mode
Kindly assign me this issue. Best regards
Record

[x]  I agree to follow this project's Code of Conduct
[x]  I'm a GSSOC'24 contributor
[x]  I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Update to embassy-executor 0.3.0

Thank you!
Thank you for your contribution.
Please make sure that your submission includes the following:
Must

[x] The code compiles without errors or warnings.
[x] All examples work.
[x] cargo fmt was run.
[x] Your changes were added to the CHANGELOG.md in the proper section.
[x] You updated existing examples or added examples (if applicable).
[x] Added examples are checked in CI

Nice to have

[x] You add a description of your work to this PR.
[x] You added proper docs for your newly added features and code.

Updates the build-in executors to use the newest embassy-executor crate. This change means that calling a pender no longer goes through a function pointer - but it needs a runtime branch to figure out which interrupt to fire.
I renamed a macro parameter as it wasn't very well named, so that increases the noise level a bit, but hopefully will reduce future confusion potential.
This PR does not change user-facing APIs. Hopefully, I've feature-guarded the __pender function correctly, too, in case someone wants to use their own executors. Unfortunately, it is not possible to mix one of ours (thread or interrupt) with a custom executor. This tradeoff was acceptable to the embassy project, so I hope this shortfall won't cause much problems here, either.
~Unfortunately, due to some transitive sadness, we have to use the git dependency until a new embassy-time is released.~ Thanks Jesse for the embassy-time release!

My test cases:

ESP32/embassy_multicore example
ESP32/embassy_multicore_interrupt example
my own S3 project

All 3 test cases seem to work well so hopefully I didn't mess anything important up. I've fixed the labels of the CI jobs I copy-paste-error'ed last time.


Hex view paint event is slow

We should improve hex view performance, which isn't perfect even on Release builds.
An idea for optimization: instead of drawing background for every byte separately we can do one call to fillRect for contiguous bytes with the same background.
Also, we should do some profiling to see what's really slow.

Fixed by https://github.com/codilime/veles/pull/280.


Mac Install Failed (behind a proxy)

Description:
I have tried and failed to install hydrogren on my Mac. I am on a company computer which needs to tunnel through proxy and firewall. Based on tips in other installation issues, I have made many changes and configurations, but am still running into problems. Alas, I am here to request help.
I have confirmed that apm is functioning and navigating my proxy by installing other atom plugins (atom-beautify, autocomplete-python, etc.)
Steps to Reproduce:
apm install hydrogen
Troubleshooting

Uninstall/resinstall Atom
Uninstall/Reinstall node
config set python2.7
config set https-proxy
brew uninstall/install pkg-config
brew uninstall/install zeromq
Reviewed every Mac installation issue and tried many other things..

At this point I'm at the end of my rope and nearly ready to pony up for PyCharm. But I'd prefer not.  Please help!
Versions:
atom --version
Atom    : 1.35.1
Electron: 2.0.18
Chrome  : 61.0.3163.100
Node    : 8.9.3

apm --version
apm  2.1.3
npm  6.2.0
node 8.9.3 x64
atom 1.35.1
python 2.7.10
git 2.21.0

Logs:
$❯ PYTHON=python2.7 apm install hydrogen
Installing hydrogen to /Users/{user}/.atom/packages ✗

><EMAIL_ADDRESS>install /private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-11934-44197-1rgzae5.8r7c/node_modules/Hydrogen/node_modules/zeromq
> node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)

prebuild-install WARN install self signed certificate in certificate chain

npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated in favor of @nteract/outputs.
npm WARN deprecated<EMAIL_ADDRESS>This package has been deprecated. Please access each transform through its own package.
{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }
npm WARN enoent ENOENT: no such file or directory, open '/private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-11934-44197-1rgzae5.8r7c/package.json'
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| ^15.0.0 but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of vega@* but none is installed. You must install peer dependencies yourself.
npm WARN apm-install-dir-11934-44197-1rgzae5.8r7c No description
npm WARN apm-install-dir-11934-44197-1rgzae5.8r7c No repository field.
npm WARN apm-install-dir-11934-44197-1rgzae5.8r7c No README data
npm WARN apm-install-dir-11934-44197-1rgzae5.8r7c No license field.

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/{user}/.atom/.apm/_logs/2019-04-05T02_31_17_300Z-debug.log

Full error log was posted to gist here: https://gist.github.com/jchaps/ed5f4068638f730e685aacbacee12294

That's frustrating. It's definitely something to do with your corporate network. I've seen people suggest this , but I dont know how unsafe that is. There might be a way to configure npm somehow to play well with with your internal network.

@jchaps Judging by this, I'm not sure if npm config set https-proxy does the trick. Is HTTPS_PROXY set in your machine?

Thanks for both for the input.
I do have strict-ssl=false set in my config:
$❯ npm config ls
; cli configs
metrics-registry = "{internal artifactory registry, redacted}"
scope = ""
user-agent = "npm/6.9.0 node/v10.15.3 darwin x64"

; userconfig /Users/{user}/.npmrc
https-proxy = "{proxy redacted}"
python = "python2.7"
registry = "{artifactory registry redacted}"
strict-ssl = false

; node bin location = /Users/{user}/.nvm/versions/node/v10.15.3/bin/node
; cwd = /Users/{user}
; HOME = /Users/{user}
; "npm config ls -l" to show all defaults.

I also have environment variables set, including HTTPS_PROXY, https_proxy, HTTP_PROXY, and http_proxy.  I had previously set apm config set https-proxy but not npm config set https-proxy=$HTTPS_PROXY... so I tried that just in case, but got the same result.
{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at Object._errnoException (util.js:1024:11)
    at _exceptionWithHostPort (util.js:1046:20)
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1182:14)
  code: 'ETIMEDOUT',
  errno: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }

Then it occured to me that the artifactory metrics registry might be the issue, so I did npm config delete registry.  I confirmed that I could curl https://registry.npmjs.org/ through the proxy successfully.  apm install hydrogen still fails with the same result.
I'm confident this is a network/proxy issue.  Just can't seem to figure out how to correct or work around it.

@jchaps Since you have npm (and hence node), would you be able to check if npm install zeromq yields the same error?

OK!  So, it looks like the issue is isolated to the zeromq install.
$ ❯ npm install -g zeromq

><EMAIL_ADDRESS>install /Users/{user}/.nvm/versions/node/v10.15.3/lib/node_modules/zeromq
> node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)

prebuild-install WARN install self signed certificate in certificate chain

{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1097:14)
  errno: 'ETIMEDOUT',
  code: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/{user}/.npm/_logs/2019-04-12T18_30_51_219Z-debug.log

Full log in gist here.  Any thoughts or recommendations? Or should I head over to https://github.com/zeromq/zeromq.js for assistance?

OK, so...
I manually installed the dependencies:
npm install -g nan
npm install -g<EMAIL_ADDRESS>
Still got an error from the prebuild-install.js script:
prebuild-install info begin Prebuild-install version 5.2.1
prebuild-install info looking for cached prebuild @ /Users/{user}/.npm/_prebuilds/5fc35e-zeromq-v5.1.0-node-v64-darwin-x64.tar.gz
prebuild-install http request GET https://github.com/zeromq/zeromq.js/releases/download/v5.1.0/zeromq-v5.1.0-node-v64-darwin-x64.tar.gz
prebuild-install http request Proxy setup detected (Host: {proxy-redacted}, Port: 8099, Authentication: No) Tunneling with httpsOverHttp
prebuild-install WARN install self signed certificate in certificate chain

{ Error: connect ETIMEDOUT <IP_ADDRESS>:443
    at TCPConnectWrap.afterConnect [as oncomplete] (net.js:1097:14)
  errno: 'ETIMEDOUT',
  code: 'ETIMEDOUT',
  syscall: 'connect',
  address: '<IP_ADDRESS>',
  port: 443 }
npm verb lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
npm verb lifecycle<EMAIL_ADDRESS>PATH: /Users/{user}/.nvm/versions/node/v10.15.3/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/{user}/.nvm/versions/node/v10.15.3/lib/node_modules/zeromq/node_modules/.bin:/Users/{user}/.nvm/versions/node/v10.15.3/lib/node_modules/.bin:/Users/{user}/miniconda/bin:/Users/{user}/miniconda/condabin:/Users/{user}/.nvm/versions/node/v10.15.3/bin:/Users/{user}/projects/dotfiles/bin:/usr/local/sbin:/usr/sbin:/sbin:/usr/local/opt/coreutils/libexec/gnubin:/usr/local/bin:/usr/bin:/bin:/usr/bin/getconf
npm verb lifecycle<EMAIL_ADDRESS>CWD: /Users/{user}/.nvm/versions/node/v10.15.3/lib/node_modules/zeromq
npm info lifecycle<EMAIL_ADDRESS>Failed to exec install script

I even found this issue and tried set NODE_TLS_REJECT_UNAUTHORIZED=0.  I've had to use strict-ssl=false previously due to the certificates in the proxy so this seemed reasonable,  but it still didn't resolve the issue.  (Notably, I have npm config set strict-ssl=false already set, but looks like prebuild-install maybe doesn't honor it?)
A potential workaround... I was able to download zeromq-v5.1.0-node-v64-darwin-x64.tar.gz directly from https://github.com/zeromq/zeromq.js/releases.  I put that in the path that prebuild-install was looking for (/Users/{user}/.npm/_prebuilds/5fc35e-zeromq-v5.1.0-node-v64-darwin-x64.tar.gz), and then npm install -g zeromq worked.  Finally.
So, now I have zeromq installed...
$ ❯ npm list -g zeromq
/Users/{user}/.nvm/versions/node/v10.15.3/lib
└──<EMAIL_ADDRESS>
... but apm install hydrogen is still throwing errors. Extract from the log file below...
8802 silly install<EMAIL_ADDRESS>8803 info lifecycle<EMAIL_ADDRESS><EMAIL_ADDRESS>8804 verbose lifecycle<EMAIL_ADDRESS>unsafe-perm in lifecycle true
8805 verbose lifecycle<EMAIL_ADDRESS>PATH: /Applications/Atom.app/Contents/Resources/app/apm/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-119312-51137-az1lhc.5inpl/node_modules/Hydrogen/node_modules/zeromq/node_modules/.bin:/private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-119312-51137-az1lhc.5inpl/node_modules/Hydrogen/node_modules/.bin:/private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-119312-51137-az1lhc.5inpl/node_modules/.bin:/Applications/Atom.app/Contents/Resources/app/apm/bin:/Users/{user}/miniconda/bin:/Users/{user}/miniconda/condabin:/Users/{user}/.nvm/versions/node/v10.15.3/bin:/Users/{user}/projects/dotfiles/bin:/usr/local/sbin:/usr/sbin:/sbin:/usr/local/opt/coreutils/libexec/gnubin:/usr/local/bin:/usr/bin:/bin:/usr/bin/getconf
8806 verbose lifecycle<EMAIL_ADDRESS>CWD: /private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-119312-51137-az1lhc.5inpl/node_modules/Hydrogen/node_modules/zeromq
8807 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c',
8807 silly lifecycle   'node scripts/prebuild-install.js || (node scripts/preinstall.js && node-gyp rebuild)' ]
8808 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
8809 info lifecycle<EMAIL_ADDRESS>Failed to exec install script
8810 timing action:install Completed in 17700ms
8811 verbose unlock done using /Users/{user}/.atom/.apm/_locks/staging-717c1c613337076f.lock for /private/var/folders/23/4xvmh1jd74z4q9z1b6ynn13h0000gp/T/apm-install-dir-119312-51137-az1lhc.5inpl/node_modules/.staging
8812 timing stage:rollbackFailedOptional Completed in 1649ms
8813 timing stage:runTopLevelLifecycles Completed in 178754ms

Why does Hydrogen insist on using its own copy of zeromq and not using what is already installed?

@jchaps

Full log in gist here. Any thoughts or recommendations? Or should I head over to https://github.com/zeromq/zeromq.js for assistance?

There is already an issue open at zeromq since October 2018 :disappointed: (https://github.com/zeromq/zeromq.js/issues/287)

Is it possible to either have Hydrogen use the existing install of zeromq, or use a similar workaround where I download the tarball and place it where it can get picked up?

Perhaps, something like the solution described here could work for you.
$ sudo apt install libzmq3-dev build-essential python2.7
$ apm install hydrogen --zmq-external

So, I was able to install zeromq on my Mac (zmq docs recommend homebrew on Mac, and my proxy allows it).
$ ❯ brew list zmq
/usr/local/Cellar/zeromq/4.3.1_1/bin/curve_keygen
/usr/local/Cellar/zeromq/4.3.1_1/include/ (2 files)
/usr/local/Cellar/zeromq/4.3.1_1/lib/libzmq.5.dylib
/usr/local/Cellar/zeromq/4.3.1_1/lib/pkgconfig/libzmq.pc
/usr/local/Cellar/zeromq/4.3.1_1/lib/ (2 other files)
/usr/local/Cellar/zeromq/4.3.1_1/share/man/ (66 files)

I confirmed that the library was installed:
$ ❯ ls -l /usr/local/lib |grep libzmq
lrwxr-xr-x   1 hxy618    admin   43 Apr 15 14:43 libzmq.5.dylib -> ../Cellar/zeromq/4.3.1_1/lib/libzmq.5.dylib
lrwxr-xr-x   1 hxy618    admin   37 Apr 15 14:43 libzmq.a -> ../Cellar/zeromq/4.3.1_1/lib/libzmq.a
lrwxr-xr-x   1 hxy618    admin   41 Apr 15 14:43 libzmq.dylib -> ../Cellar/zeromq/4.3.1_1/lib/libzmq.dylib

And I think this means I should be able to use external zmq rather than build. However, apm install hydrogen --zmq-external does not seem to work. I run into the same issues as when I don't use the --zmq-external flag. I found this portion in /node_modules/zeromq/scripts/prebuild-install.js...
if (process.env.npm_config_zmq_external == "true") {
  console.log('Requested to use external libzmq. Skipping download of prebuilt binaries.');
  process.exit(1);
}

...but for some reason it doesn't look like --zmq-external actually works.  Any guidance on what I'm doing wrong here?

@jchaps How about:
npm_config_zmq_external==true apm install hydrogen

?
And make sure you have installed libzmq3-dev (i.e. the header files needed for compilation).

Success!!!  Though it took some more trouble-shooting.  Info below...
I followed the standard Mac install for zeromq: brew install zeromq.  Then npm_config_zmq_external=true apm install hydrogen passed the correct flag for external libzmq. This was showing up in the logs:
Requested to use external libzmq. Skipping download of prebuilt binaries.
Requested to use external libzmq. Skipping libzmq build

But then I had issues with the python-interceptor.sh:
gyp verb check python checking for Python executable "/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh" in the PATH
gyp verb `which` succeeded /Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh /Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh
gyp verb check python version `/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh -c "import platform; print(platform.python_version());"` returned: "3.7.3\n"
gyp ERR! configure error
gyp ERR! stack Error: Python executable "/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh" is v3.7.3, which is not supported by gyp.
gyp ERR! stack You can pass the --python switch to point to Python >= v2.5.0 & < 3.0.0.
gyp ERR! stack     at failPythonVersion (/Applications/Atom.app/Contents/Resources/app/apm/node_modules/node-gyp/lib/configure.js:454:14)

I tried setting python to 2.7 in many ways... python=python2.7, --python=python2.7, python=/usr/bin/python2.7, and even npm_config_python=/usr/bin/python2.7, but none worked until I commented line 53 of apm according to this comment in atom/apm#179.  After that I could apm config set python=python2.7 and the config would stick.
Next, I was missing xcode. I had xcode command line tools, but not the full program, and needed the full xcode to run the build.  So I downloaded Xcode from the Mac App Store. I also ran sudo xcode-select -s /Applications/Xcode.app/Contents/Developer to ensure command line tools were linked to the full app.
After that, it finally worked!
$❯ npm_config_zmq_external=true apm install hydrogen
Installing hydrogen to /Users/hxy618/.atom/packages ✓

Clearly there are bugs in apm (python-interceptor.sh) and prebuild-install (not following the proxy and/or strict-ssl flags in apm/npm), but I don't think those are hydrogen issues.  I'm closing the issue as a result, but happy to engage with the dev team if you need me to run any testing to help isolate or fix the issues.

@jchaps Thank you for reporting back!

An update... I recently had to reinstall Atom and extensions on my work machine, and really wanted to get this done without disabling strict-ssl and opening up MITM risk. With a bit of research, I found a new setting that I thought may be useful (NODE_EXTRA_CA_CERTS). More info here.
I have proxy environment variables established (HTTP_PROXY and HTTPS_PROXY).
I have my proxy certificate file downloaded locally, and set a new env variable export NODE_EXTRA_CA_CERTS=/path/to/SystemCertificates.cer.
Once these env variables were set, I was able to apm install Hydrogen, without any other configured settings.
❯ apm --version
apm  2.4.3
npm  6.2.0
node 10.2.1 x64
atom 1.45.0
python 3.7.6
git 2.21.1

❯ apm config ls
; cli configs
globalconfig = "/Users/{user}/.atom/.apm/.apmrc"
metrics-registry = "{artifactory registry}"
scope = ""
user-agent = "npm/6.2.0 node/v10.2.1 darwin x64"
userconfig = "/Users/{user}/.atom/.apmrc"

; environment configs
python = "/Applications/Atom.app/Contents/Resources/app/apm/bin/python-interceptor.sh"

; project config /Users/{user}/.npmrc
registry = "{artifactory registry}"

; globalconfig /Users/{user}/.atom/.apm/.apmrc
cache = "/Users/{user}/.atom/.apm"
progress = false

; node bin location = /Applications/Atom.app/Contents/Resources/app/apm/bin/node
; cwd = /Users/{user}
; HOME = /Users/{user}/.atom/.node-gyp
; "npm config ls -l" to show all defaults.

A sanitized copy of my verbose install output is on gist for reference.


Certmaker should check for ./ca.{crt,key} before /root/nebula/ca.{crt,key}

Tracked instead in #2. Closing.


Added chain to exports

#392 This will allow people to actually import chain.

@vanbujm thanks for opening a PR, I have created a new one to get rid of the additional files that your editor included. All in all, thanks for catching it!


InlineDatePicker fails in 4.0.0+

Using version 4.1.0 of this Add-on, InlineDatePicker is not rendered.
Steps to reproduce:

Go to https://addonsv24.flowingcode.com/year-month-calendar/inline-date-picker

Observe that the component isn't rendered

Click to the "Month" tab, and click "InlineDatePicker" again.

Observe that the component is rendered

Expected behavior: the component should be rendered on first load.

The issue can also be reproduced with 4.0.0 (both in Vaadin 24.1 and 24.2).
On first load, the inner components of fc-inline-date-picker have no shadow root.


Can not connect to discordapp.com:443

I having this issue when running my bot:
  File "C:\Users\FabioNote\AppData\Local\Programs\Python\Python35\lib\site-
ges\aiohttp\connector.py", line 314, in connect
    .format(key, exc.strerror)) from exc
aiohttp.errors.ClientOSError: [Errno 1] Cannot connect to host discordapp.c
3 ssl:True [Can not connect to discordapp.com:443 [[SSL: CERTIFICATE_VERIFY
ED] certificate verify failed (_ssl.c:645)]]
Unclosed client session
client_session: <aiohttp.client.ClientSession object at 0x0000000004756588>

How to fix this?
Running on Windows 7 x64 and latest Pyton
Edit: This https://github.com/Just-Some-Bots/MusicBot/issues/281 dont solved.

Fixed

I'm sorry but i have the same problem and this solution don't worked for me...

In my case, the time of my notebook are not sync with the internet time, i sync and worked.

I'm running the bot on a Raspberry Pi 3, i'll watch if i have the same problem.


Hexagon shape and size does not allow proper spacing (remove border to correct)

In the design guidelines there are lots of examples of stacked hexagons.
Here's an example:

However, if you use the hexagons provided in the repo, they don't stack elegantly and it requires splitting pixels in order to align things correctly. Here's an example; notice how the bottom two hexagons don't have a white space between them like they do when placed next to the hexagon above:

Upon further investigation, it seems that the hexagons have a black 2px border on the outside of the shape. This is throwing everything off. If you remove the border and resize the hexagon correctly, the spacing is corrected:

Great point, love the detail, I've revised the template

Great point, love the detail, I've revised the template


Update mockito-core to 5.8.0 in java8

About this PR
📦 Updates org.mockito:mockito-core from 5.5.0 to 5.8.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.mockito", artifactId = "mockito-core" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.mockito", artifactId = "mockito-core" }
}]

labels: test-library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #538.