Failure to start - P100/Cuda 6.1 incompatibility?

Upon attempting to run version 0.2.12, it fails with the attached error. I tried a few flags, but none changed the error: --disable-flashinfer , --disable-flashinfer-sampling ,  --disable-cuda-graph . Given that the final line complains about a kernel image being unavailable, I suspect that there's a requirement for some particular version of cuda or some particular GPUs in the code, but I haven't been able to find that documented anywhere, and the error message itself is not exactly clear.
If I'm right about what the issue is, support for P100's/Cuda 6.1 would be appreciated as a feature request - They're literally the most economical way to run large models right now by a large margin. If p100's/Cuda6.1 are already supposed to be supported, I'd be eager to help track down whatever bug is happening here.
It wasn't clear to me what actual tests would help, so I didn't dig into it very deeply. I'm happy to try building the software myself, or install models beyond the one gptq model I happened to have downloaded. Just let me know what you need!
`
Exception in ControllerSingle:
Traceback (most recent call last):
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/managers/controller_single.py", line 166, in start_controller_process
controller.loop_for_forward()
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/managers/controller_single.py", line 103, in loop_for_forward
out_pyobjs = self.tp_server.exposed_step(recv_reqs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/managers/tp_worker.py", line 222, in exposed_step
self.forward_step()
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/managers/tp_worker.py", line 238, in forward_step
self.forward_prefill_batch(new_batch)
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/managers/tp_worker.py", line 452, in forward_prefill_batch
output = self.model_runner.forward(batch, ForwardMode.EXTEND)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/model_executor/model_runner.py", line 397, in forward
return self.forward_extend(batch)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/model_executor/model_runner.py", line 373, in forward_extend
return self.model.forward(
^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
return func(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/models/llama2.py", line 314, in forward
hidden_states = self.model(input_ids, positions, input_metadata, input_embeds)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/models/llama2.py", line 281, in forward
hidden_states, residual = layer(
^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/models/llama2.py", line 228, in forward
hidden_states = self.input_layernorm(hidden_states)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
return self._call_impl(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
return forward_call(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/vllm/model_executor/custom_op.py", line 13, in forward
return self._forward_method(*args, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/sglang/srt/layers/layernorm.py", line 45, in forward_cuda
out = rmsnorm(x, self.weight.data, self.variance_epsilon)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/llama/.pyenv/versions/3.11.9/lib/python3.11/site-packages/flashinfer/norm.py", line 52, in rmsnorm
return _kernels.rmsnorm(input, weight, eps)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: RMSNorm failed with error code no kernel image is available for execution on the device
`

Sorry. We primarily focus on sm80+ and sm75.

Please consider this a feature request, then. This hardware is literally the cheapest, most accessible way to run inference right now, so I'm of the opinion that it's the most important hardware to focus on. I'd also suggest you put the current restriction in your documentation, so that you don't get duplicate feature requests, then.

It makes nonsense for me.


awsome pokemon should go under games category?

just wondering if this should be changed or if there is a proper reason why its under misc. and not games

Gaming != Games
They are two separate things. Gaming section is game development oriented, not game oriented.

@aleksandar-todorovic Oh! my bad, guess i should close now


Rename name-related fields

A bunch of renaming, and preserving some more info starting at the header parser. New nomenclature:

name: @types/ember__object
typesDirectoryName: ember__object
libraryName: @ember/object (or nonNpmDescription for non-npm packages)
subDirectoryPath: ember__object/v2

It would be nice to know what the old nomenclature was, since I have a big change in flight that will benefit from this, but also need to update all its code.

Old nomenclature:

name: ember__object
fullNpmName: @types/ember__object
libraryName: Inconsistent, but was intended to be either the implementation library name or a human-readable representation of it. It wasn’t enforced to be the npm library name.
subDirectoryPath: ember__object/v2

I'm regenerating publishable output to ensure nothing broke here that wasn’t covered by the tests.


Use memory saving temporary files for transactions

The relANNIS converter can have huge number of tuples per transaction. The old implementation will hold these tuples in memory which won't work. As a workaround the converter committed transactions earlier to "flush" the tuples.
This pull requests adds a new alternative. Tuples are written to temporary files and append to the actual output file on commit or discarded if a commit is aborted. The old memory based implementation is still the default but the createTupleWriter(...) function of the factory now can have a parameter if the memory saving implementation should be used.
Please note the relannis4 branch of the relannis converter already assumes that this implementation is available.

Invalid, it's better to copy the new implementation to the relANNIS module itself.


Binding DateTime issue with Xamarin

Fixed an issue in Xamarin project that would bring up a "The registrar found an invalid type System.DateTime in signature for method ops. Use Foundation.NSDate instead." Error when building the project with the and trying to use the binding's ShareClient().

Can this be merged please? I need this fix also.

Can this be merged please? I need this fix also.

If you still need it there is a forked repo you can use here , it's also available via nuget.

Any reason why this was closed and not merged?


How can I use this repo to run all 50 models on a different dataset?

Dear Yang,
Assuming you are interested in the zero-shot baselines, we provide scripts in this directory to run all baselines discussed in the paper. Please let us know if you run into any issues.
Kind regards,
Pascal

Dear Yang,
I'm closing this issue as I believe the above addressed it. Please feel free to re-open if you have any further questions on this.
Best,
Pascal


ER training on a new dataset

Hi, I am training wavlm upstream for ER on a custom dataset. I am getting the following error while training
python3 run_downstream.py -n ExpName -m train -u wavlm -d emotion -c downstream/emotion/config.yaml -o "config.downstream_expert.datarc.test_fold='fold1'"

I am not sure about the wav files lengths in IEMPCAP as I don't have access to it. Here is the distribution of the wav file length in seconds for my dataset

Hello
Can you please explain how the data_root is supposed to insert?
I tried this format, but it does not work.

here is the code:


[issue 450] Deprecate terms

Description
Replace terms throughout the docs for OpenSearch REST APIs and settings deprecated in version 2.0: "master" is replaced with "cluster manager."
Issue requested to notify customers about the nomenclature change. Plan to add to What’s new post, and the Release Notes. Within the doc set, after first instance of “master”→“cluster manager”, there is a note to call out the change.
Issues Resolved
[FEATURE]Deprecate the "master" nomenclature #450
Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Update: I set PR as draft while I add a section.

Hi @tlfeng Tianli,
Could you check to see all items are documented in this PR for the issue #450?  Thanks

Hi @tlfeng and @Naarcha-AWS,
Just an update, we discussed creating a separate PR for documenting the list of deprecated APIs and settings in these doc locations:

Release Notes (2.0) & breaking changes
Deprecated APIs (separate page with full list)
REST API overview -  1 paragraph announcement:
"Several APIs and settings are set for deprecation with the next release (3.0). For the complete list, see ."


log: review logs for autodiscover and provider packages

The log level and the information printed has been reviewed for each log or added if missing.
The resources are printed between quotes for easier identification in the log line.

/dci-rerun


[WIP] Add support to provide custom claims in outbound JWT

Purpose
$subject
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/17812
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #17963   +/-   ##
=======================================
  Coverage   15.17%   15.17%
=======================================
  Files          46       46
  Lines        1206     1206
  Branches      188      188
=======================================
  Hits          183      183
  Misses       1009     1009
  Partials       14       14

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 b633bb2...c2ac398. Read the comment docs.


Web URL static identifiers (supposedly permalinks) are changing

Describe the bug
The auto generated ids for sec-certs urls pointing to specific certified product are not always persistent in time (https://sec-certs.org/cc/6207e27699db6695/). We do have at least two cases of id that changed in time - for example https://sec-certs.org/cc/6207e27699db6695/ vs. https://sec-certs.org/cc/56ef6c45d7eab7ee/ . The ID is generated from presumably static parameters of certificates, but it changed anyway. If these changes, we are loosing permalink people may have in their papers or watched certificates (email/RSS/Compare). We shall at least detect and add redirect. (and also investigate what is changing)
To Reproduce
Steps to reproduce the behavior:

Find some certificate (e.g., Strong Customer Authentication for Apple Pay on Apple Watch with S8 running watchOS 9.4(watchOS 9.4 (build 20T253)) ( ANSSI-CC-2023/60) https://sec-certs.org/cc/56ef6c45d7eab7ee/)
Wait some time
Check again based on url - if bug demonstrates itself, you will get Not Found
Search for certified device name again => observe different url https://sec-certs.org/cc/6207e27699db6695/

Expected behavior
Once indexed and inserted to database, the url static string shall stay same.
If change is anyway detected, notify admins and add automatic redirect to new url.
Desktop (please complete the following information):

web at sec-certs.org (end of 2023/begin 2024 -> October 2024)

Additional context
Known examples
NXP JCOP 4 P71 https://sec-certs.org/cc/2a45531c2dbd1ab8/ -> https://sec-certs.org/cc/f29f88756682e034/
Strong Customer Authentication for Apple Pay on Apple Watch with S8 running watchOS 9.4(watchOS 9.4 (build 20T253)) ( ANSSI-CC-2023/60) https://sec-certs.org/cc/56ef6c45d7eab7ee/ -> https://sec-certs.org/cc/6207e27699db6695/

This one will be tough to fix though, it's pretty deep in our design and we've repeatedly revisited this with no resolution.
@petrs If this is important to you, could you please fit it to Thu team meeting agenda? We should discuss possible solutions in person.

This one will be tough to fix though, it's pretty deep in our design and we've repeatedly revisited this with no resolution.
@petrs If this is important to you, could you please fit it to Thu team meeting agenda? We should discuss possible solutions in person.

Yes, I thing this is important feature to have (permalinks). It breaks email notification functionality as well, so it is real bug. Let's discuss in person

Let me put some notes here that I made on Matrix, just to not lose them.
During normal operation of the site it will never happen that a previously valid certificate pages gives a 404. If a certificate is removed from an update run (or its ID changes, etc., we can not really tell these things apart) the only thing that happens is that a note is added to the certificate update log, (see here for an example, here is the diff).
What happened to lead to the 404s
Some time ago CC portal changed the way they construct URLs to certification artifacts (PDFs). We were including these links in our primary key that we build the ID out of. Thus one update caused almost the whole database to turn over: "old" certificates were marked as removed and "new" certificates with new IDs took their place. The database was thus sort of duplicated. To clean this up, we decided to make a new fresh run and clear up the database (as well as make our ID construction algorithm agnostic to these sort of URL changes). We also added automated redirects from the old IDs to the new ones.
However, the CC portal also changed some other attributes of some certificates during this change. Since we did the fresh run we lost some certificates here, because the automated redirects were put in based on the new atrtributes (and lead to the wrong "old" IDs).
How to avoid this in the future?
Do not do fresh runs.
How to fix this now
Get pre-update dataset, examine which of its IDs are giving 404, quantify and create more redirects manually?
Bottom line
Even if we do no fresh runs in the future. The isue is not resolved fully.  Even if we had not done the fresh run which has caused the 404s, we would have had a different problem. The page with the old id would list the certificate as removed and there would be a new certificate in the database (and a page) with the new id. This is because to us, these are two different certificates, even though your human eye can tell they are the same. See below for a case study on this.
Case study
Name and cert ID change. :arrow_right:
STMicroelectronics ST33K1M5A and ST33K1M5M B02 to STMicroelectronics ST33K1M5A and ST33K1M5M B03

Removed: https://sec-certs.org/cc/8c7ccf48dcbd5398/, NSCIB-CC-2300112-01-CR
New: https://sec-certs.org/cc/83fc6586c8ac42f5/, NSCIB-CC-2300112-02-CR

Name and cert ID change. :arrow_right:
STMicroelectronics ST33K1M5C and ST33K1M5T C01 to STMicroelectronics ST33K1M5C and ST33K1M5T C02

Removed: https://sec-certs.org/cc/a6f7f2facccfcb2c/, NSCIB-CC-2300056-01-CR
New: https://sec-certs.org/cc/34d1675a7a079459/, NSCIB-CC-2300056-02-CR

Name and cert ID stayed. :heavy_equals_sign: Security target URL changed (...v1.0 to ...v1.1)
Junos OS 22.3R1 for ACX5448-M, EX4400-24T, EX4400-24P, EX4400-24MP, EX4400-48T, EX4400-48P, EX4400-48F, EX4400-48MP and QFX5120-YM

Removed: https://sec-certs.org/cc/9246f9e02631f69e/, Certificate Number: 2024/148
New: https://sec-certs.org/cc/7ff6ceb530a8d40b/, Certificate Number: 2024/148

Looking at the case studies I think a proper fix is to rethink what a certificate even is. In the cases above, which cases represent the same certificate and which present a new one? For example the first two where part of the cert ID changed are quite weird. See the respective certificates: old new.
Once we understand what a certificate even is, we can think about the changes to our methodology necessary to reflect that. I think it would have to go in the direction of more similarity based metrics and fuzzy stuff rather than hard IDs. Also, the tool is currently quite naive with respect to incremental updates and the web is doing a fresh run each time + some custom update tracking. This should be aligned, because without time information about the previous state of the dataset it is hard to say that some entries are really the same certificate.

Looking at the case studies I think a proper fix is to rethink what a certificate even is. In the cases above, which cases represent the same certificate and which present a new one? For example the first two where part of the cert ID changed are quite weird. See the respective certificates: old new.

I checked these two and I believe these shall be considered two different certificates:

cert ID is different (although only suffix) NSCIB-2300112-01 vs. NSCIB-2300112-02
Name of product is different (ST33K1M5M B02 vs. ST33K1M5M B03)
Assurance level/package is different (EAL 6 augmented with ALC_FLR.1 vs. EAL6 augmented with ALC_FLR.2)

I made the following change to the website to handle the cases where some of the primary key attributes changed, but not much. @petrs Is this a good-enough resolution for you?


Curving Velocities Doesnt work very well

It works, then it stops working and moves linearally I'm assuming ran out of array memory?
Reverting to last working script

Fixed it


Issue with product image sizing on home page.

If you add a product and add an image on desktop the image is way oversized. These values are being evaluated as null and the dom is assigning the css class Infinity.

Hi @chrisrijos
Thanks.


Lock Gemfile-shopify to money 0.14.x

The Gemfile.shopify file doesn't specify a version of money to use. I'm not entirely sure why that's the case, but it seems like an unnecessary risk to me that can just lead to headaches.
A particular headache I'm currently experiencing (and that's prompting me to lock this) is that Money 0.15 was just released (https://github.com/Shopify/money) and several tests began to fail because of that.
Let's keep the minor version locked for now to prevent breakages introduced by functionality changes.
We can bump the money gem to 0.15 in a follow up PR, for now we need to create a release for a fix for Gestpay https://github.com/activemerchant/offsite_payments/pull/352

Please do, I have concerns that locking this version without a follow-up fix will cause breakage in applications that use both offsite-payments and money.

Oh don't you worry, only core uses both, no biggie.

Please do, I have concerns that locking this version without a follow-up fix will cause breakage in applications that use both offsite-payments and money.

Oh don't you worry, only core uses both, no biggie.

@bdewater Do you have someone who can look at this immediately? Without setting the version the CI will continue to fail on master. And I don't have the bandwidth or context for this right now unfortunately.
If there are serious concerns and there is someone available shortly to do this I can wait to release for a bit until we can bump to 0.15. Question though, if there were any real problems in core we'd be experiencing them now since this gem (already) doesn't play nice with 0.15, right?

@bdewater Do you have someone who can look at this immediately? Without setting the version the CI will continue to fail on master. And I don't have the bandwidth or context for this right now unfortunately.
If there are serious concerns and there is someone available shortly to do this I can wait to release for a bit until we can bump to 0.15. Question though, if there were any real problems in core we'd be experiencing them now since this gem (already) doesn't play nice with 0.15, right?


Configurable drag-end event. Helps issue #1

This change enables the consumer page to catch a custom event, in order to know that a drag operation occurred (and is finished).
Solves issue #1

Thanks!


Variable assignment type component fixes

Need to update some of the components to use the new assignment type.

Turns out the problem is happening from inside zk.

Was fixed by updating to newer zk version.

Updated zk, and everything works fine now.


[release-2.6] Replace golang.org/x/text version

Refs:

https://github.com/stolostron/backlog/issues/26341

Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/hold Does this need to wait since there's a code freeze?

Also, the referenced issue is for 2.7. Was there a 2.6 issue?

There was no 2.6 issue, but in Slack it was mentioned that we should cherry-pick these kinds of fixes to the most recently released version.

There was no 2.6 issue, but in Slack it was mentioned that we should cherry-pick these kinds of fixes to the most recently released version.

Yeah, my bad. I actually deleted my comment because I remembered that an issue is only created for the latest version now.

@JustinKuli Is this ready to go? Looks like it had a hold on it so it didn't merge. Might need a rebase now to pick up my workflow updates.

@dhaiducek yeah, not sure how I lost track of this one.
/unhold


dry-run error :: can't dup NilClass (TypeError)

I already found the cause to this error, however I just wanted to post this here incase anybody else comes across the same issue. After the upgrade from cucumber 2.3.3 to 2.4.0 I ran into the error/stacktrace below. The culprit scenario file had not be changed and started failing only after the upgrade to 2.4.0, and was found during a --dry-run.
code:ruby
Scenario Outline: Lorem ipsum dolor sit amet
Given consectetur adipiscing elit
When proin non varius est <page_size> maecenas eu magna tempus
Then consectetur nibh eu, cursus diam  maecenas
Examples:
Examples:
  |page_size |page   |count |
  | 5        |  1    | 5    |
  | 10       |  2    | 10   |
  | 25       |  1    | 25   |

The error happens due to `Examples:` being listed twice for the `Scenario Outline`. And again I bring this up because we were not getting this error prior to upgrading to cucumber 2.4.0.

Env: `ruby 2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin13]`
Command: `bundle exec cucumber --dry-run`
Error: `can't dup NilClass (TypeError)`
Stacktrace:
```code:ruby```
└─[2] <git:(master 7c34199✱✈) $> cucumber --dry-run
can't dup NilClass (TypeError)
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:31:in `dup'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:31:in `initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:308:in `new'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:308:in `initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:274:in `new'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:274:in `block in initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:274:in `map'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:274:in `initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:133:in `new'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:133:in `block in initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:126:in `map'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:126:in `initialize'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:105:in `new'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:105:in `feature'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/ast_builder.rb:16:in `feature'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core/gherkin/parser.rb:28:in `document'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core.rb:27:in `block in parse'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core.rb:26:in `each'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core.rb:26:in `parse'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-core-1.5.0/lib/cucumber/core.rb:18:in `compile'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-2.4.0/lib/cucumber/runtime.rb:67:in `run!'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-2.4.0/lib/cucumber/cli/main.rb:32:in `execute!'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/gems/cucumber-2.4.0/bin/cucumber:8:in `<top (required)>'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/bin/cucumber:23:in `load'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/bin/cucumber:23:in `<main>'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/bin/ruby_executable_hooks:15:in `eval'
/Users/sgreen/.rvm/gems/ruby-2.2.2@nexpose-cucumber/bin/ruby_executable_hooks:15:in `<main>'

As Gherkin v3.2.0 (used in Cucumber-Ruby v2.3.3) does not allow empty Examples: sections, one would expect that the above feature should trigger a parser error in Gherkin v3.2.0. But what really happens is that the second Examples: line is parsed as a description (ast node) to the first Examples: line.
With Gherkin v4.0.0 the feature file is now (correctly) parsed as a scenario outline with two Examples: sections, one completely empty (which is allowed in Gherkin v4.0.0) and one with an examples table in it. As Gherkin now is allowing different variants of incomplete scenario outline, Cucumber-Ruby must also handle those cases, but as this issue points out - it does not do that yet.


win app should start even if settings xml file is damaged

Does this relate to the user.config file? This was removed recently which might make this issue irrelevant.


Allow/convert file:/// links to allow for drag & drop from Nautilus

Drag-and-drop is a really useful feature, having the ability to drag & drop directly from Nautilus (and similar file managers) would be really useful and should be fairly simple to implement.
I would even be willing to look into doing this and doing a pull request to add this feature if it's something that you think is fit to do.

I don't know how I missed this issue, but let's bring things up to date. The latest builds (from Git) include a local HTTP-based API that is accessible to other applications running under the user's account. This includes Nautilus and any extensions it provides. I have written one here that manually strips the file:/// from URLs before handing them to NitroShare, so it doesn't really solve your issue but works around it.
Is that okay or is this still a must-fix issue?


Replace nv_root finding via frames to using the first joint

When using crocoddyl_msgs with a custom URDF, the current method of finding the root joint by always selecting frames[1].parent does not work.
The change here is always using the second joint, i.e. model.joints[1]. This has fixed the issue for me, but I'm aware this is quite an important change that we may want consensus on.

Thanks, @james-p-foster! This looks better approach.


doc: open source the user-facing documentation

This is an open source version of the user-facing docs on developers.google.com/speed/pagespeed/module/.  I currently have it staged at modpagespeed.com/doc, which is probably where this should go when live.
It uses server-side includes for headers and footers, which isn't as nice as being fully static, but should be more maintainable.
This change doesn't include a landing page for mod_pagespeed.  I think that should probably go on modpagespeed.com / ngxpagespeed.com and not in the docs directly.  Having it in the docs is a weird artifact of how we were hosted on developers.google.com.  But this is not in this CL.
Mobile PSI score is 100 with mod_pagespeed and 89 without, so we're also doing a bit of demonstrating pagespeed.
For reviewability, this is split into three sequential changes:

https://github.com/pagespeed/mod_pagespeed/pull/1464 : straight copy-over of existing docs
https://github.com/pagespeed/mod_pagespeed/pull/1465 : formatting changes that apply in bulk to all html files
https://github.com/pagespeed/mod_pagespeed/pull/1459 (this change) : css and js to make the docs be usable and nice to look at

@jmarantz

What do you think about splitting this into two reviews.

In the first one, you pull over the django sources exactly, and I don't really review it :)
In the second one, you make the changes needed to render the documentation properly.

Done.  I've split it into three reviews:

#1464 : straight copy-over of existing docs
#1465 : formatting changes that apply in bulk to all html files
#1459 (this change) : css and js to make the docs be usable and nice to look at

@jlporter could you review the js in _footer.html?

At least add comments showing we are handling that case.

Done: 051489982

What happens if you issue an alert if you detect a level gap? Does it fire?

I'd need to check each of our doc pages, which would be a pain; we have a lot of doc pages.

Would that help keep the docs well-formed?

Probably not.  Either we'd use alert(), which would be a big pain for our users if we messed up, or we'd use console.log() and probably no one would notice.


OrangeHat - Firmware - Servo

Port the Servo library written previously for AT324 microcontroller for Unit Zero to the AT4809 for the OrangeHat

[ ] Giving power to the Servo bank causes a brownout on VSRVO.B and VRPI lines due to charge sharing. Regulator recovers within 20ms.

PROBLEM: Giving power to the servo banks, causes a brownout.

Servo driver now implements commands and a scalable number of channels. The interface is position in microseconds and speed in microsecond per seconds.

Eight servo channels operational. Image below shows sampling of channel one and channel eight

Channel 2 fix: It was a bad solder joint on the 4X1KOhm resistor bank (FIXED)

Problem with rotation servo is simply due to a skew in the Parallax 50RPM servomotor, that goes faster in one direction.
In the future I need to think about calibration and conversion. I think I'll do a class dedicated to convert from degrees or degrees per second or RPM to microseconds and embed calibration parameters inside that class, so that the driver just takes care of emitting a pure PPM waveform with 1500us zero and 20000us period.

Servo Driver is operational.
Moving on to advance other drivers to working state before adding bells and whistles to the Servo driver.


malloc: Heap corruption detected and segfault when running demo example

duckdb(24577,0x16ffd3000) malloc: Heap corruption detected, free list is damaged at 0x600003aacec0
*** Incorrect guard value:<PHONE_NUMBER>4145
duckdb(24577,0x16ffd3000) malloc: *** set a breakpoint in malloc_error_break to debug

and
zsh: segmentation fault  duckdb --unsigned

when running the demo example.
version = v0.9.2 3c695d7ba9
OS = Darwin Kernel Version 23.0.0: Fri Sep 15 14:41:43 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T6000 arm64

Thanks for reporting this. I will take a look!

Got the same thing today on v0.10 and Mac 14.3.1 (23D60) - M1 Max 64gb ram
duckdb(4508,0x1e1e7dc40) malloc: Heap corruption detected, free list is damaged at 0x6000023f6480
*** Incorrect guard value:<PHONE_NUMBER>6065
duckdb(4508,0x1e1e7dc40) malloc: *** set a breakpoint in malloc_error_break to debug
zsh: abort      duckdb -unsigned

Tried in an in-memory database & in a file database.

This is now fixed in https://github.com/ywelsch/duckdb-prql/commit/a1b156ddcdb8cc3d1abf53622e3c3d214c136d11, so in 30mins from now the binaries should be available for duckdb 0.10.0 (I'm only doing releases for the latest duckdb).


[Bug]: Reading additional metadata from SharePoint using GetFolderFilesByServerRelativeUrl

Describe the issue
I am trying to use the SharePoint module to read additional metadata on the items.
I see there is the following IntegrationEvent which refers to exactly what I want to do.
    [IntegrationEvent(false, false)]
    /// <summary>
    /// Process SharePointFile Metadata - Use to extract custom meta data into model record
    /// </summary>
    /// <remarks>Extend the "SharePoint File" table to store any custom data.</remarks>
    /// <param name="Metadata">__metadata node of SharePointFile Json Object</param>
    /// <param name="SharePointFile">SharePointFile temporary record.</param>
    internal procedure ProcessSharePointFileMetadata(Metadata: JsonToken; var SharePointFile: Record "SharePoint File" temporary)
    begin
    end;

It refers to extending the "SharePoint File" table but the table is not extendable. I could work round that but still think it needs fixing.
The bigger issues is the JsonToken that is passed is just the file Id and no metadata.
The below is take from codeunit 9106 "SharePoint File"
        if Payload.Get('ListItemAllFields', JToken) then begin
            Payload := JToken.AsObject();

            if Payload.Get('Id', JToken) then
                SharePointFile.Id := JToken.AsValue().AsInteger();

            SharePointClient.ProcessSharePointFileMetadata(JToken, SharePointFile);
        end;

Should the the Payload JsonObject not be the thing that is passed?
Expected behavior

Ability to extend the "SharePoint File" table
To be passed all metadata to the integration event

Steps to reproduce

I would expect the below to show Json of the metadata

    [EventSubscriber(ObjectType::Codeunit, Codeunit::"SharePoint Client", 'ProcessSharePointFileMetadata', '', false, false)]
    local procedure ProcessSharePointFileMetadata(Metadata: JsonToken; var SharePointFile: Record "SharePoint File" temporary)
    begin
        Message(format(Metadata));
    end;

Additional context
n/a

I'm not super strong in how the SharePoint module works (yet). We would have to rely on community members which have tried using it. Maybe create a draft PR to show your intended changes and see if anyone can assist?
@dynaway-kmi or @mynavblog, would you be able to assess if this change is good?

@JesperSchulz, thanks for the reply, will get the changes drafted up over the next few days.

Hi,
I have run into the same problems when trying to use metadata within our project.
I have already prepared a fix:
https://github.com/dynaway-kmi/BCApps/commit/cab44c0628059c4816a032ff8366b1455e8e1f3b
Should I make a PR out of it?

Hi,
I have run into the same problems when trying to use metadata within our project.
I have already prepared a fix: dynaway-kmi@cab44c0 Should I make a PR out of it?

Yes please! You can link to this issue.


public_metadata not passed for invite creation

When a user is invited to the application using the code below, the metadata is not passed properly.
res = s.invitations.create(
    request={
        "email_address": email,
        "public_metadata": {"role": "user"},
        "redirect_url": f"{config['FRONTEND_URL']}/accept-invitation",
        "notify": True,
        "ignore_existing": False,
    }
)

This seems to be related to the lines under the create function in src/clerk_backend_api/invitations.py below
if not isinstance(request, BaseModel) and request is not None:
    request = utils.unmarshal(request, models.CreateInvitationRequestBody)
request = cast(models.CreateInvitationRequestBody, request)

The request object returned by these lines contain an empty CreateInvitationPublicMetadata object.

Hey @jianminglok We are looking into this issue and will report back soon !


See rewards for a specific / range of Epochs

Story
As a user
I want to see what rewards I earned between specific epochs
So that I can use this information to account / track my earnings
Acceptance Criteria

[ ] There is a way to see a specific Epoch's reward earnings
[ ] There is a way to see a reward earnings in a range of epochs e.g. I want to see rewards between epoch 50 and 57

Tasks

[ ] Explore and sketch
[ ] Team and stakeholder review
[ ] Visual Design
[ ] Team review
[ ] Etc.

Sketch
Additional details / background info

Also consider if we do #4024


Bug: Preview show "[Process exited 0]" when delta is used

On linux

EDIT: I see previewers.new_termopen_previewer is a Telescope method, so I should probably report it there.

Your EDIT is entirely correct, nothing I can do about this message from my end 🙃

I came across this issue. Is there a solution for it?

Hi, there is no "fix" for this. Feel free to read up on it at the issue created by OP, which concerns the API I am using.
As far as I can tell that issue only contains hacky fixes which involve stripping the last line or similar. I am not willing to implement those just for aesthetics, others might consider the line useful: It is proof that delta ran successfully and no lingering terminal is running in the background. Further, this message actually comes from the underlying :term command used by the Telescope API which I in turn use here. You can try it  out yourself by running :term echo for instance.
Feel free to create an issue with Telescope, as anything I could do would basically patch their API response anyhow. If they respond similarly, you could decide to take it up in a Neovim issue. However, I hardly think they'll consider it: This would be a breaking change for plugins and scripts which might depend on that line being there.

I haven't tried it myself, but you would do the same as the commenters there: Set the previewer in the opts and Telescope should pick it up correctly. You can copy the previewer used by telescope-undo as a base and implement their changes on top or go for theirs entirely at first 👍🏻


TV show recipe reports handle id missing for shell theme.

Attempt to choose TV suggestion at ToT currently reports sporadic failure around shell theme handle as below. @sjmiles @shans
assert-web.js:11 Uncaught (in promise) Error: can't find handle of id !519203816017645:app-shell-8c5ltsmaehj5o8avf:1
    at assert (assert-web.js:11)
    at Arc._instantiateParticle (arc.js:258)
    at particles.forEach.recipeParticle (arc.js:397)
    at Array.forEach (<anonymous>)
    at Arc.instantiate (arc.js:397)

The handle id reported in the error message generally belongs to the Launcher Arc (which is supposedly unreferenced before the error itself fires [as part of instantiating a recipe in the TV Arc]).
I haven't figured out how this happens exactly (there are not supposed to be references to the old arc/handles anywhere).
It only seems to occur on the #shelltheme handle and only if data is written to that handle during Launcher Arc shutdown. PR #1345 prevents the error by setting the application arc to null during the shutdown, but this is a blind fix.


visualization

Hi ,
I hope this message finds you well. I'm writing to ask about the temp.json generated,I want to visualize the data in unity.Thank you in advance for your assistance.

Have you solved it yet? I have been trying for a long time without success, which is very important to me. Can you tell me how to visualize the generated JSON file using ai2thor? If possible, I would greatly appreciate it.


Add classroom theme picture

Simply Implemented with b9423f2c28738e38bc61c9e8fd1321d15f444538.
If we need to add an admin ui for setting and changing the picture, we'll have to add that later.


Problem with Multithreading

When I was training the agent, I observed that the process only used 1 core. Is multithreading supported in this code?

I tried multithreading (see lines 128-145) but it caused issue with the old version of SUMO, so I disabled it. Feel free to implement and test multithreading with later version of SUMO, though the speed improvement may be small due to the GIL.


Error when installing 1.28 deb

/usr/share/applications/code-url-handler.desktop: warning: key "MimeType" is a list and does not have a semicolon as trailing character:

I am having this issues too.
Is there a schedule to when this fix may be released?

Check https://github.com/Microsoft/vscode/pull/56727

This is pretty weird. I can't reproduce:
joao@joao-ubuntu-vm:~$ sudo apt install code
[sudo] password for joao:
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  code
0 upgraded, 1 newly installed, 0 to remove and 171 not upgraded.
Need to get 44.0 MB of archives.
After this operation, 196 MB of additional disk space will be used.
Get:1 http://packages.microsoft.com/repos/vscode stable/main amd64 code amd64 1.28.2-1539735992 [44.0 MB]
Fetched 44.0 MB in 1s (39.8 MB/s)
Selecting previously unselected package code.
(Reading database ... 170774 files and directories currently installed.)
Preparing to unpack .../code_1.28.2-1539735992_amd64.deb ...
Unpacking code (1.28.2-1539735992) ...
Processing triggers for mime-support (3.60ubuntu1) ...
Processing triggers for desktop-file-utils (0.23-1ubuntu<IP_ADDRESS>) ...
Setting up code (1.28.2-1539735992) ...
Processing triggers for gnome-menus (3.13.3-11ubuntu1.1) ...

But I also didn't fix anything.
The code looks good, as there is a semicolon in there: https://github.com/Microsoft/vscode/blob/03f7c83b237ceb2ea8fd5eac90c21bd3b2512ceb/resources/linux/code-url-handler.desktop#L12
When I download that exact same deb,  and extract it, I verify that the semicolon is there. :thinking:
Let me know if you still hit it, reopen if you can reproduce.

Hi @joaomoreno,
I had this issue too an I am still having it.
I removed an reinstalled it. At the end I listed the content of my .desktop file
nsol@MintMashine ~ $ date; sudo apt-get remove code
Mon Nov  5 07:00:30 CET 2018
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following packages will be REMOVED:
  code
0 upgraded, 0 newly installed, 1 to remove and 0 not upgraded.
After this operation, 196 MB disk space will be freed.
Do you want to continue? [Y/n] Y
(Reading database ... 284421 files and directories currently installed.)
Removing code (1.28.2-1539735992) ...
Processing triggers for gnome-menus (3.13.3-6ubuntu3.1) ...
Processing triggers for desktop-file-utils (0.22+linuxmint1) ...
Processing triggers for mime-support (3.59ubuntu1) ...
nsol@MintMashine ~ $ date; sudo apt-get install code
Mon Nov  5 07:00:42 CET 2018
Reading package lists... Done
Building dependency tree
Reading state information... Done
The following NEW packages will be installed:
  code
0 upgraded, 1 newly installed, 0 to remove and 0 not upgraded.
Need to get 0 B/44,0 MB of archives.
After this operation, 196 MB of additional disk space will be used.
Selecting previously unselected package code.
(Reading database ... 282479 files and directories currently installed.)
Preparing to unpack .../code_1.28.2-1539735992_amd64.deb ...
Unpacking code (1.28.2-1539735992) ...
Processing triggers for gnome-menus (3.13.3-6ubuntu3.1) ...
Processing triggers for desktop-file-utils (0.22+linuxmint1) ...
Processing triggers for mime-support (3.59ubuntu1) ...
Setting up code (1.28.2-1539735992) ...
/usr/share/applications/code-url-handler.desktop: warning: key "MimeType" is a list and does not have a semicolon as trailing character, fixing
nsol@MintMashine ~ $ date; cat /usr/share/applications/code-url-handler.desktop
Mon Nov  5 07:00:56 CET 2018
[Desktop Entry]
Name=Visual Studio Code - URL Handler
Comment=Code Editing. Redefined.
GenericName=Text Editor
Exec=/usr/share/code/code --open-url %U
Icon=code
Type=Application
NoDisplay=true
StartupNotify=true
StartupWMClass=Code
Categories=Utility;TextEditor;Development;IDE;
MimeType=x-scheme-handler/vscode;
Keywords=vscode;
X-Desktop-File-Install-Version=0.22

Regrads,
nsol

Since this is just a warning, I'll leave it closed. I still can't repro and I'm pretty sure the semicolon is in there inside the package... so... maybe @Tyriar has an idea?

It certainly has a semicolon at the end?
https://github.com/Microsoft/vscode/blob/beb5998ea3d6b4cae43b3a09ecdf11e3bd692667/resources/linux/code-url-handler.desktop#L11


Dependency on symfony/debug

Currently symplify/package-builder depends on symfony/debug. In Symfony 4.4 however this package is being replaced by the new symfony/error-handler component. As such symplify should aim to drop this dependency.
Why is the dependency even there? I can't find any usage of Symfony\Debug within the PackageBuilder.
See https://symfony.com/blog/new-in-symfony-4-4-errorhandler-component

The dependency was added in this commit but the ThrowableRenderer class is no longer present in the code, nor is any other usage of Symfony\Component\Debug string. I think this can be safely removed.


add password support to remote SSH access feature

Is your feature request related to a problem? Please describe.
Educates has the documented Enabling Remote SSH Access feature. However this requires the use of SSH key pairs and the learner to have to copy the file or key. This is cumbersome.
Being able to set a common or simple generated password and the learner do password authentication from a client on their machine would be an easier/simpler experience.
Describe the solution you'd like
Being able to set a common or simple generated password and the learner do password authentication from a client on their machine would be an easier/simpler experience.
Describe alternatives you've considered
Hosting the key in a webserver and having a generated script people could run that would download the key, chmod the permissions, etc.
Additional information
https://docs.educates.dev/custom-resources/workshop-definition.html?highlight=ssh#enabling-remote-ssh-access

The alternative you considered is already possible and was the compromise solution for now at least.
With a workshop session there is an environment variable CONFIG_PASSWORD. This is also available as data variable for interpolation in workshop instructions using config_password.
$ env | grep CONFIG_PASSWORD
CONFIG_PASSWORD=eHSWjbczrE17BFLC4iomyT6J0IVvlptM

That password can be used with special URL against a workshop session instance to get configuration related to the session.
Some details are found in:

https://docs.educates.dev/portal-rest-api/session-management#retrieving-session-configuration

The special URL prefix is the equivalent of $SESSION_URL/workshop/config/ where in a workshop session you can construct that from SESSION_URL environment variable or session_url data variable in workshop instructions.
For example:
$SESSION_URL/workshop/config/id_rsa?token=$CONFIG_PASSWORD

The configuration one can download using this includes the kubeconfig as well as SSH keys.
The idea thus was until work out a better solution, that any custom tunnel client for SSH access could be given the session URL and the config password. That client could then download the SSH key, saving it somewhere it can then use it when it actually runs SSH to access the session.
Nothing more has been done at this point and as sort of mentioned in SSH access docs the idea is to come up with a purpose built CLI, or as part of VS Code extension, for doing tunnelling, with samples for what is required provided for Python and Go. See tunnel.go and tunnel.py in:

https://github.com/vmware-tanzu-labs/educates-training-platform/tree/develop/tunnel-manager

So idea was to see first how people may want to use this before work out whether anything else should be done as part of Educates to make it easier.
As to having SSH directly accept a password, the challenge there was no readily available PAM module for a password file distinct from /etc/shadow. One can get an old package for Debian, but not Fedora.
Updating /etc/shadow to add a password is problematic as nothing runs as root in the workshop container. The only way around it is to use an init container running as root to update the file in that container and then using some volume mount tricks to inject the modified file into the main workshop container. Even then though that is not enough as /etc/shadow will not be readable by the workshop user (which sshd also runs as), so it couldn't authenticate the user against it anyway.
So short of find an existing PAM plugin module which allows use of a separate htpasswd, or maintaining our own such PAM plugin module, there is no ideal solution for having SSH access use a password, thus why SSH keys we used so far.
We can have a separate discussion about what you are trying to use SSH access for so can see where we go.


Add integration test for capacity controller

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Add integration test for capacity controller to ensure status is up to date with assignments.
Which issue(s) this PR fixes:
Part of #68
Special notes for your reviewer:

/hold
/assign @ahg-g

/retest

looks good, squash?

squashed

/lgtm

/lgtm
/hold cancel
/retest

Turns out I broke some unit tests by introducing a type alias :)

Turns out I broke some unit tests by introducing a type alias :)

ha!
/lgtm


how to use other plugin's shortcode in post for wechat?

Hello,
The following example is just for showing wechat post in wordpress site, and it works well.

在文章或页面编辑页中选择文本
在需要插入的地方填入 [wechat_material]
可选参数 col, 显示列数，默认为 3，可选为 2
可选参数 title, 是否显示标题，默认为 true，false 为不显示
可选参数 thumb, 是否显示缩略图，默认为 true，false 为不显示
可选参数 digest, 是否显示简介，默认为 true，false 为不显示
完整示例 [wechat_material col="3" title="true" thumb="true" digest="true"]

but,  I use the shortcode of other plugins in wordpress post, for example, I put a poll shortcode in wordpress post to show a poll, and the poll information will show up blog post at wordpress front-end,  but if I push the post to wechat, it will show that poll shortcode in Wechat APP not Poll text information, is it possible to resolve?
Thanks.
Alex

Other plugins is which one?
Tell me the plugin name, i will try it.
BTW, can you send your site more information to my mail (chinese.jk@gmail.com), i will help me.


Compute font size in onMeasure() to support height, line spacing, drawables, etc.

The previous version used a separate StaticLayout to determine if the
text was too large for the desired TextView height.  However,
accurately mimicing the TextView appearance (and thus size) is nearly
impossible when one considers full information, e.g., line spacing,
left/right/top/bottom drawables, replacement spans, etc.  Before API
16, information like line spacing was simply unavailable to
subclasses.
This commit uses a different approach. The onMeasure() method is
overriden to tell the superclass TextView to measure at its desired
height. If this is too tall or short (in pixels or lines), the font is
decreased or increased and the view measured again.  The same binary
search process as before is used.
This commit does not work when the 'android:singleLine' attribute is
set. The text is clipped. Instead, 'android:maxlines="1"' should be
set and the desired ellipsizing, horizontal scrolling, and single-line
transforms set manually.
The TextView treats 'singleLine' as a special case (i.e., it's handled
differently than 'android:maxLines="1"'), but does not expose this
state. Ideally, we would disable 'singleLine' in onMeasure(), before
calling super.onMeasure(), and reenable it after setting the fitted
font size. However, as there is no (supported) way to determine if the
'singleLine' property was set, this is not possible.  Simply disabling
maxLines and ellipsizing does not work---I belive because the TextView
uses the cached internal StaticLayout.
Ultimately, using 'singleLine' in an autosizing text view is weird
anyway, since 'singleLine' implies 'ellipsized'.  (Ellipizing after
the font size has reached the minimum is still supported).

Originally I left this since I was working on decoupling autofitting and TextView. Once I accomplished that with AutofitHelper, I meant to comment here but personal life took over.
It'd be great to see this re-invented with AutofitHelper in mind, but it'd probably have to be in the form of a completely different PR.
I'll leave this open for now, but if I don't hear a response I'll end up closing it and try looking at solving this myself if I have time.

Are you interested in the approach taken in this PR. i.e., a binary search within onMeasure() rather than over a separate static layout attempting to mimic the actual TextView?  The onMeasure() approach doesn't really fit the AutofitHelper approach, as far as I can see.

Unfortunately, no since this solution isn't portable to other types of TextViews like AutofitHelper. Thanks for your contribution!

Ok, no problem!


chore: delete unused file #523

What type of PR is this?

/kind cleanup
What this PR does / why we need it:
clean up unused files.  #523
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm

/approve


add request header for trace id

new request header trace-id

Hi @zffocussss can you explain why you need a separate header for only trace-id? Is it for routing? As I understand the spec, opentracing clients should be capable of extracting it from the trace-parent header (which contains both trace-id and span-id), WDYT?

WDYT

It is not mandatory, but it helps us add this info in logs when correlating apps tracing in some APM,such as elastic APM.
I am working on the kong logs, which logs can be modified by the request header.it is just a patch only.

I closed it,as it is not useful in common.

I closed it,as it is not useful in common.

Ok, as you wish. Thanks for your feedback, and if you really need it I don't mind merging :)


switcher: Split fast switcher into its own plugin

This only partially fixes #16 as it does not add touch support.

I think everything has been addressed.


[MS Teams] Team ID in context data is the same as the General Channel ID

Bot Info

Bot handle:
App ID:
SDK Platform: .NET
SDK Version: 3.9.1
Active Channels: MS Teams
Deployment Environment: Azure App Service

Issue Description
In context data, the Team ID is the General Channel ID and not the actual Team ID.
Expected Behavior
The team ID should be different from the General Channel ID, it should be the same team ID I get from Graph API.
Actual Results
E.g. when debugging my bot in production this is what I get @ MessageController.cs under ContextData

{
	"team":
	{
		"id"<EMAIL_ADDRESS>	}
}
{
	"channel":
	{
		"id"<EMAIL_ADDRESS>	}
}

Even though, when I look up these information using Graph API I get the following:
Joined Teams Request: https://graph.microsoft.com/beta/me/joinedTeams

Response:

{
<EMAIL_ADDRESS>"https://graph.microsoft.com/beta/$metadata#groups",
    "value": [
                {
            "id": "fd7daf35-be29-49e4-9820-8b105cc55eca",
            "displayName": "Testing Bot",
            "description": "Testing Bot"
        }
    ]
}

Channels Request: https://graph.microsoft.com/beta/groups/fd7daf35-be29-49e4-9820-8b105cc55eca/channels/

Response:

{
<EMAIL_ADDRESS>"https://graph.microsoft.com/beta/$metadata#channels",
    "value": [
        {
            "id": "b2041ccc-38ae-4dbd-8a2f-8e1646fb8c5a",
            "displayName": "Non-General Channel",
            "description": null
        },
        {
            "id": "0a10b552-8018-4b2d-a9ec-d25e828b935e",
            "displayName": "General",
            "description": "Testing Bot"
        }
    ]
}

Another question
What is the origin of the prefix (19:) & suffix (@thread.skype) in the ID field?
Can I expect these to be the same for all channels and teams in MS Teams?
The reason for this question is that I rely on these IDs to lookup up Team and Channel-specific information from Azure storage.
"channel":
{
"id"<EMAIL_ADDRESS>}

@amrElroumy
So, you are expecting team-id "b2041ccc-38ae-4dbd-8a2f-8e1646fb8c5a" to be returned.  Can you please confirm if there are no access issues between AAD application and GraphAPI in terms of getting necessary information from the group fd7daf35-be29-49e4-9820-8b105cc55eca?
Also can you please share the code? This will help us to understand and review the issue further?

@msmohan
I was expecting the team ID to be: fd7daf35-be29-49e4-9820-8b105cc55eca
However, I was getting the General Channel's ID when retrieving the Team ID.
I don't suspect that there's an issue with AAD and graph API concerning that particular group/team. I am getting the same result for any group/team I add this bot to. The returned team ID is always the general channel's ID and not the ID I get for the team from Graph API.
The code for parsing the Team and Channel ID is as follows:

        private static async Task<TeamsContext> InitializeTeamsContext(IActivity activity)
        {
            var tenantId = activity.GetTenantId();

            // teamID = general channel ID
            var teamId = (string)activity.ChannelData["team"]["id"];
            var channelId = (string)activity.ChannelData["channel"]["id"];

            var team = await _teamProvider.GetTeamByTeamsIdAsync(teamId);
            var service = await _serviceProvider.GetServiceByTeamsChannelIdAsync(channelId);

            return new TeamsContext(tenantId, channelId, team, service);
        }

It is invoked as part of a Luis Intent Method:
        [LuisIntent("Intent")]
        public async Task MyIntent(
            IDialogContext context,
            IAwaitable<IMessageActivity> activity,
            LuisResult result)
        {
            var message = await activity;
            var teamsContext = await InitializeTeamsContext(message);
            .....
        }

Thanks for bringing to us. The improvement to include the AAD group id to channel data is still in progress. We will let u know once it is released to public.

@amrElroumy  Closing this issue, feel free to reopen if required.

We are facing the same issue while creating a teams app. The returned teamId is the same as channel id. Is there any way go get the groupId, what could be then used in Graph API to query the team?

@attilabalazsy I believe the Team Id and General Channel id are the same by design.  If you are using the latest sdk and the bot is installed in the Team, you can call TeamsInfo.GetMembersAsync  to retrieve the Team members.  Please refer to this sample: https://github.com/microsoft/BotBuilder-Samples/blob/master/samples/csharp_dotnetcore/57.teams-conversation-bot/Bots/TeamsConversationBot.cs#L94
Note: this is a closed issue, and closed issues are not monitored by the Bot Framework Team.  If you require further assistance, please ask a question on Stack Overflow  The team monitors the BotFramework tag daily.

@EricDahlvang , thank you for your feedback. Based on it, we have realized, that we are using the old V3 bot framework as Microsoft still has this sample on the getting started page. We will update then our app to the new framework and will try your suggestion.


Loacale - Format datetime related issue

In my application I need to set locale as per user's culture. I am setting that using below code:
$.datetimepicker.setLocale('' + currentCulture + '');
$("#dateTimePicker").datetimepicker({
inline: false,
format: 'd-M-Y h:i A',
formatTime: 'h:i A',
step: 30
});
Above allows me to select datetime as per locale and format setting. After selecting datetime on blur it is selecting now(). I need to change my jquery.datetimepicker.full.js to keep my selected datetime.
Is that bug or m I missing some setting?

Hi there, I have faced with same problem.
I have found that when I use EN locale everything works as expected, but in my case with RU locale I also have problem that described above. (One more thing that can affect on this it's my english OS, but I'm not sure )
It looks like the problem in date parser logic, but unfortunately in present time I have not time to fix it in correct way - the parser creation logic is not clear for me from first view ... so I fixed it in rude way - I have replaced localized month's to EN analogue right before parser checks date. After this all works as expected.
To test my solution to your situation you should do next:

Open jquerydatetimepicker.full.js
Find next function

...
Date.parseDate = function(input, format) {
...
};
...

it should be between 2450 - 2470 lines
replace this function with next:
...
	Date.parseDate = function(input, format) {
                parseFunctions[format] || createParser(format);

                // -- temp date parsing fix begin --
                var ml = Date.monthNames.map(function(e){return e.substring(0, 3)});
                var mo = ['Jan','Feb','Mar','Apr','May','Jun','Jul','Aug','Sep','Oct','Nov','Dec'];
                for (var i=0; i<mo.length; i++) input = input.replace(ml[i], mo[i]);
                // -- temp date parsing fix end --

                return parseFunctions[format](input);
	};
...

Keep in mind that it's very rude fix, so use it on your own risk


Teamwork Contract

I added a number of items to the teamwork contract. Take a look and we can discuss when we meet if there are issues

I added a number of items to the teamwork contract. Take a look and we can discuss when we meet if there are issues

Looks great, I made a couple updates based on your suggestions. Is everyone good with publishing this on github?

Looks great, I made a couple updates based on your suggestions. Is everyone good with publishing this on github?

I think the teamwork contract looks great. I also added an extra sentence to 'c'. Yeah, I'm fine with publishing it on Github.

I think the teamwork contract looks great. I also added an extra sentence to 'c'. Yeah, I'm fine with publishing it on Github.

If we were to publish it on Github, I think we should remove it when we finish the project. The contract has too many personal details (strengths, weaknesses, interets)

If we were to publish it on Github, I think we should remove it when we finish the project. The contract has too many personal details (strengths, weaknesses, interets)

We could also put a link to the google doc instead and then just change the
permissions when the project is over. That way the history won't be
available.
On Wed, 13 Jan 2021 at 11:19, CalSchafer<EMAIL_ADDRESS>wrote:

If we were to publish it on Github, I think we should remove it when we
finish the project. The contract has too many personal details (strengths,
weaknesses, interets)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/UBC-MDS/532_Group_22/issues/2#issuecomment-759662623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALS2GJAEWH5IDYF2ROGS2DSZXW5BANCNFSM4V76DZXQ
.

We could also put a link to the google doc instead and then just change the
permissions when the project is over. That way the history won't be
available.
On Wed, 13 Jan 2021 at 11:19, CalSchafer<EMAIL_ADDRESS>wrote:

If we were to publish it on Github, I think we should remove it when we
finish the project. The contract has too many personal details (strengths,
weaknesses, interets)
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/UBC-MDS/532_Group_22/issues/2#issuecomment-759662623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALS2GJAEWH5IDYF2ROGS2DSZXW5BANCNFSM4V76DZXQ
.

All looks good to me.  My vote is for just leaving it on google docs to keep things simple.

All looks good to me.  My vote is for just leaving it on google docs to keep things simple.

Added a link to the README, we can include the link when we submit to canvas as well. I've changed the settings so it is viewable with a link, once the project is over I can change the settings so only we can view it.

Added a link to the README, we can include the link when we submit to canvas as well. I've changed the settings so it is viewable with a link, once the project is over I can change the settings so only we can view it.


adding boot entry tutorial only specifies how to add a newer os than windows vista

I want to add an older os to my boot entry than windows vista in my virtual machine.

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

ID: c2e888de-16af-86ce-3844-b55b519aea27
Version Independent ID: 97cbb0fb-8c60-afe6-5170-0bfcc75a8613
Content: Adding Boot Entries - Windows drivers
Content Source: windows-driver-docs-pr/devtest/adding-boot-entries.md
Product: windows-hardware
GitHub Login: @EliotSeattle
Microsoft Alias: eliotgra

@kevincrans - Thanks for reaching out to the Windows Driver Developer Documentation team. Unfortunately, we don’t have the resources to offer individual technical advice to the broad array of customers who use the driver documentation.
You may have success posting your question in the following forums, where other driver developers and Microsoft engineers can be found. The community there is quite responsive.
•	Windows Hardware WDK and Driver Development
Best of luck with this issue.


chore(pipeline): configure name and expose build project

Exposing the CodeBuild project in the pipeline will enable use cases
around creating monitoring resources. As an example, the build's
Duration and Failure metrics can be graphed on a CloudWatch dashboard.
Having a concrete name to the CodeBuild Project allows for the above
mentioned dashboards to reliably graph these across syntheses.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: e4f60e280ccdefe864e2c3c09786f51049848e88
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: 0faaf4a24993aca01507846a4250de5d2096cd4a
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

In the past I've avoided exposing the underlying resources of delivlib constructs cause it feels we are trying to keep the abstraction relatively tight.

I hear ya.
The other option I was considering was to expose the build metrics right out of the delivlib's pipeline.ts via an API, similar to what CodeBuild's Pipeline construct does. Do you think that would be better approach? Any other ideas?

I do think this is the right approach (exposing properties with underlying constructs) personally. Just wanted to spur any conversation that's worth having about it. We can see how this shakes out from the consumer side.


Improved Audible Audiobook support

Support Amazon Audible /hz/audible/mlp/mfpdp URLs:
These should have the cover image in its full resolution while Amazon's standard product pages (i.e. /dp URLs) provide only 500px images for Audible audiobooks.

Prefer /hz/audible/mlp/mfpdp pages for Audible products:
Look for Audible buttons on standard product pages, as we can only extract 500px images from these pages. In that case we immediately redirect the Amazon provider to /hz/audible/mlp/mfpdp pages which should have the same image in its full resolution.

Closes #322 (which is also one of the examples that I have used for testing).
I still remember that you had said that we should probably split Amazon audiobooks into a separate provider in case we would find another special case, which has happended now. Although I had considered this idea, I have not found a clean solution to do this as music and audiobook pages can't be told apart by their URLs.
It probably would not have a been a problem to have two providers for the same URL pattern before this change. But now we have to detect Audible product pages before we even try to extract an image because otherwise the provider would happily return the smaller image for Audible audiobooks. (All the other audiobook logic still can be run after all extraction attempts for a music release have failed.)
If you really want to split the provider, it would require having the audiobook provider run before the music provider. This would introduce a delay of an additional request for the common case of a music release if we don't want to introduce a cache for provider requests.

Hmm I'm not really in favour of having two separate Amazon providers with the same URL pattern being present in the dispatch map, that just seems confusing 😄 Is there another way we could reasonably split up the implementation to (clearly) separate it into digital/physical/audiobook? Preferably only by changing the Amazon provider. Simple factory came to my mind, but I don't think that'd be applicable as that would require changes to the dispatcher, IIRC.
(I'm sorry that I have to ask this question, I should know, it's my repository…but I've been doing these damn E2E tests for so long that I forgot what most of the other code looks like 😅 )

If you're cool with it, I can go tinker in this branch a bit to see if that'll work.

Sure, let's try that, just push your ideas to this branch, I won't do any further changes until you have checked that approach.

Done. I've also removed the thumbnail extraction fallback as it would otherwise silently succeed with inferior images (like #272), it's better to just fail completely so we can fix the extractor for everyone.
I've also locally regenerated all of the recordings and the tests still pass.

I've noticed that the placeholder cover detection only works on amazon.com, other domains use images in the local language, and it has a different filename. It's quite easy to fix, but I'm not sure whether it's worth it at this point. We can either:

Fix it in this PR
Open an issue and fix it later
Ignore it until someone complains about it

You decide 🙂

Open an issue and fix it later

Let's do it the half-lazy way 😇


ENH ppc64le

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

@conda-forge-admin rerender

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

Travis-CI times out. I'll make a manual build

Would it be possible to do an aarch64 build as well? Then we could try building llvmlite and numba on both using CI.

I don't have access to aarch64. Sorry


Projection leading to unexpected results

Overview
The CONUS404 netCDFs provided by Hytest are projected into a coordinate system that does not contain all necessary information to conduct spatial analysis with additional datasets in R. This leads them to plot in unexpected locations and makes spatial analysis impossible. I first documented this issue in an IWAAs repo on GitLab.
The netCDF files are available at caldera/hytest_scratch/scratch/conus404_monthly_netcdf/ and were made available for the IWAAs National group.
Viewing coordinate reference system
conus404 <- sds("01_fetch/out/conus404_monthly_010.nc")
crs(conus404, parse = T)
 [1] "PROJCRS[\"unnamed\","
 [2] "    BASEGEOGCRS[\"unknown\","
 [3] "        DATUM[\"unnamed\","
 [4] "            ELLIPSOID[\"Spheroid\",6370000,0,"
 [5] "                LENGTHUNIT[\"metre\",1,"
 [6] "                    ID[\"EPSG\",9001]]]],"
 [7] "        PRIMEM[\"Greenwich\",0,"
 [8] "            ANGLEUNIT[\"degree\",0.0174532925199433,"
 [9] "                ID[\"EPSG\",9122]]]],"
[10] "    CONVERSION[\"unnamed\","
[11] "        METHOD[\"Lambert Conic Conformal (2SP)\","
[12] "            ID[\"EPSG\",9802]],"
[13] "        PARAMETER[\"Latitude of false origin\",39.1000061035156,"
[14] "            ANGLEUNIT[\"degree\",0.0174532925199433],"
[15] "            ID[\"EPSG\",8821]],"
[16] "        PARAMETER[\"Longitude of false origin\",262.099998474121,"
[17] "            ANGLEUNIT[\"degree\",0.0174532925199433],"
[18] "            ID[\"EPSG\",8822]],"
[19] "        PARAMETER[\"Latitude of 1st standard parallel\",30,"
[20] "            ANGLEUNIT[\"degree\",0.0174532925199433],"
[21] "            ID[\"EPSG\",8823]],"
[22] "        PARAMETER[\"Latitude of 2nd standard parallel\",50,"
[23] "            ANGLEUNIT[\"degree\",0.0174532925199433],"
[24] "            ID[\"EPSG\",8824]],"
[25] "        PARAMETER[\"Easting at false origin\",0,"
[26] "            LENGTHUNIT[\"metre\",1],"
[27] "            ID[\"EPSG\",8826]],"
[28] "        PARAMETER[\"Northing at false origin\",0,"
[29] "            LENGTHUNIT[\"metre\",1],"
[30] "            ID[\"EPSG\",8827]]],"
[31] "    CS[Cartesian,2],"
[32] "        AXIS[\"easting\",east,"
[33] "            ORDER[1],"
[34] "            LENGTHUNIT[\"metre\",1,"
[35] "                ID[\"EPSG\",9001]]],"
[36] "        AXIS[\"northing\",north,"
[37] "            ORDER[2],"
[38] "            LENGTHUNIT[\"metre\",1,"
[39] "                ID[\"EPSG\",9001]]]]"

Plotting in ArcGIS

Plot origin at Null Island (0, 0)
Plotting in R
conus404 <- sds("01_fetch/out/conus404_monthly_010.nc")
plot(conus404$PREC_ACC_NC$PREC_ACC_NC_1)

Again, the plot origin is at (0, 0)
Possible solutions
I can think of two possible solutions: (1) provide additional CRS information so that the rasters plot as expected, or (2) provide a rasterized huc layer in the same projection as the rest of the CONUS404 data so that the CONUS404 data can be summarized to the HUC (HUC12 is preferable).

@pnorton-usgs, Amelia Snyder suggested that I tag you in this.

Below are the projections I've used for c404 and the c404 bias corrected datasets.

Conus 404
c404_proj = "+proj=lcc +lat_0=39.1000061035156 +lon_0=-97.9000015258789 +lat_1=30 +lat_2=50 +x_0=0 +y_0=0 +R=6370000 +units=m +no_defs"

Conus 404 bias corrected
c404bc_proj = "+proj=lcc +units=m +a=6370000.0 +b=6370000.0 +lat_1=30.0 +lat_2=60.0 +lat_0=40.0 +lon_0=-97.0 +x_0=0 +y_0=0 +k_0=1.0 +nadgrids=@null +wktext  +no_defs "

@amart90 I believe I have fixed the problem for the on-prem monthly netcdf files. See #143 for a description of the issue. At the moment I have not added a WKT attribute to the crs variable. With QGIS fixing the x and y coordinate variables fixed the projection problem. When you have a moment see if this fixes the problems you were experiencing. We could still add a WKT attribute if needed.

@pnorton-usgs, it worked! Correcting the typos in projection_x_coordinate and projection_y_coordinate as mentioned in #143 did the trick. I downloaded the new netCDFs from caldera and gave them a whirl in R.
conus <- terra::sds("C:/Users/ajmartinez/Downloads/conus404_monthly_010.nc")

us_states <- spData::us_states |>
  sf::st_transform(crs = sf::st_crs(conus[[1]]))

ggplot2::ggplot() +
  tidyterra::geom_spatraster(data = conus$ACETLSM$ACETLSM_5, maxcell = 500000) +
  ggplot2::geom_sf(data = us_states, fill = NA, color = "#7a7a7a", lwd = 0.2)

Plots and interacts with other datasets as expected