Adding missed out request/response parameter change to new version 2021-06-01 for Azure Site Recovery.

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Changelog
Please ensure to add changelog with this PR by answering the following questions.

What's the purpose of the update?

[ ] new service onboarding
[ ] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[x] Other, please clarify : In Azure Site Recovery we raised swagger PR New Version 2021-06-01 for Azure Site Recovery. #14794. We forgot to add 1 change: resource naming(custom target disk, NIC names).This PR is to add those changes.

When you are targeting to deploy new service/feature to public regions? Please provide date, or month to public if date is not available yet.
When you expect to publish swagger? Please provide date, or month to public if date is not available yet.
If it's an update to existing version,  please select SDKs of specific language and CLIs that require refresh after swagger is published.

[x] SDK of .NET (need service team to ensure code readiness)
[ ] SDK of Python
[ ] SDK of Java
[ ] SDK of Js
[ ] SDK of Go
[ ] PowerShell
[ ] CLI
[ ] Terraform
[ ] No, no need to refresh for updates in this PR

Contribution checklist:

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
[ ] Ensure to copy the existing version into new directory structure for first commit (including refactoring) and then push new changes including version updates in separate commits. This is required to review the changes efficiently.
Adding a new service

[ ] Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable or public preview version with Breaking Change Validation errors
[ ] Updating API(s) in public preview over 1 year (refer to Retirement of Previews)

Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.

This PR is to add an additional [commit ]https://github.com/Azure/azure-rest-api-specs/pull/15199/commits/f848781966df9c9173d739cc15acdffdd33c8708) containing changes that were missed out in the original PR  which has been reviewed for break changes and passed by ARM team. This PR should only be merged after the original PR. This PR should only be checked for the additional commit.
Tagging @chunyu3, @zhenglaizhang  , for FYI regarding older PR.

This PR is to add an additional [commit ]f848781) containing changes that were missed out in the original PR  which has been reviewed for break changes and passed by ARM team. This PR should only be merged after the original PR. This PR should only be checked for the additional commit.
Tagging @chunyu3, @zhenglaizhang , for FYI regarding older PR.

@prsadhu-ms-idc  thanks for in the info, so I will start reviewing when the original PR merged, so that I could see the correct diff at that time.

This PR is to add an additional [commit ]f848781) containing changes that were missed out in the original PR  which has been reviewed for break changes and passed by ARM team. This PR should only be merged after the original PR. This PR should only be checked for the additional commit.
Tagging @chunyu3, @zhenglaizhang , for FYI regarding older PR.

@prsadhu-ms-idc thanks for in the info, so I will start reviewing when the original PR merged, so that I could see the correct diff at that time.

@zhenglaizhang please check the last [commit ]f848781) which is the difference, rest all commits are same as the other PR which is already approved. Please let us know if any changes are required on the last commit.

This PR is to add an additional [commit ]f848781) containing changes that were missed out in the original PR  which has been reviewed for break changes and passed by ARM team. This PR should only be merged after the original PR. This PR should only be checked for the additional commit.
Tagging @chunyu3, @zhenglaizhang , for FYI regarding older PR.

@prsadhu-ms-idc thanks for in the info, so I will start reviewing when the original PR merged, so that I could see the correct diff at that time.

@zhenglaizhang please check the last [commit ]f848781) which is the difference, rest all commits are same as the other PR which is already approved. Please let us know if any changes are required on the last commit.

@chunyu3 so are you fine to merge the older PR, so that I would easily review this PR with all latest validation results , as it's based on the old PR.

This PR is to add an additional [commit ]f848781) containing changes that were missed out in the original PR  which has been reviewed for break changes and passed by ARM team. This PR should only be merged after the original PR. This PR should only be checked for the additional commit.
Tagging @chunyu3, @zhenglaizhang , for FYI regarding older PR.

@prsadhu-ms-idc thanks for in the info, so I will start reviewing when the original PR merged, so that I could see the correct diff at that time.

@zhenglaizhang please check the last [commit ]f848781) which is the difference, rest all commits are same as the other PR which is already approved. Please let us know if any changes are required on the last commit.

@chunyu3 so are you fine to merge the older PR? so that I would easily review this PR with all latest validation results , as it's based on the old PR.

We are waiting for ARM manifest changes to happen before we merge the older PR.

@zhenglaizhang , @chunyu3 , please review and merge this PR. I have rebased with upstream/master, as the other PR  is already merged.

I have seen the older PR was just merged 4 hours ago, so should no sdk released, no need to do breaking change review.


Feature request: FlashAttention

Hey,
There is an implementation for this memory efficient version of attention here.
And I was wondering if it is possible to somehow integrate it into Flax.

Instead of flash attention, maybe consider memory efficient attention? https://arxiv.org/abs/2112.05682
I tested that flash attention and noticed it's both slower and seem to take more memory. Tests were done on TPU, so it's possible the higher memory usage is due to flash attention slicing causing more padding.
Memory efficient attention have shown a decrease in memory. The paper actually comes with jax implementation. Google research's implementation can be found https://github.com/google-research/google-research/tree/master/memory_efficient_attention

It's supposed to be both faster and less memory intensive. lucidrains version is implemented in jax instead of at a low level. So it is slower than the normal version.

+1
Any update on this requst ?

jax.experimental has an implementation of FlashAttention, written by Pallas kernels and therefore usable in both GPU and TPU.
We can probably upstream this to Flax attention if jax.experimental doesn't scare us (it means the API could still change).

jax.experimental has an implementation of FlashAttention, written by Pallas kernels and therefore usable in both GPU and TPU.
We can probably upstream this to Flax attention if jax.experimental doesn't scare us (it means the API could still change).

Does that support both backdrop and forward prop?

jax.experimental has an implementation of FlashAttention, written by Pallas kernels and therefore usable in both GPU and TPU.
We can probably upstream this to Flax attention if jax.experimental doesn't scare us (it means the API could still change).

Does that support both backprop and forward prop?

Yes, it seems so.

jax.experimental has an implementation of FlashAttention, written by Pallas kernels and therefore usable in both GPU and TPU.
We can probably upstream this to Flax attention if jax.experimental doesn't scare us (it means the API could still change).

Yes, but it seems missing some parameters like the dropout, mask and bias, if I am not mistaken.

One more point Jax has two implementations:

Fused attention: https://github.com/google/jax/blob/main/jax/experimental/pallas/ops/attention.py
Flash attention: https://github.com/google/jax/blob/main/jax/experimental/pallas/ops/tpu/flash_attention.py

Update: my comment was made before Pallas was an option, where low level access is simply not possible.
The official jax flash attention implementation is built off on Pallas and thus not subjected to the same issue from 5+months ago. Flash attention should be variable right now, and likely faster than the older mem eff attention I suggested.

Looks like the two attention kernels are for different platforms - one for TPU and another for GPU. The implementations are slightly different for performance reasons, and they are also of different APIs.
It would be cool if we can unify the two and provide a stable attention API, either at Flax level or at JAX level. I will look more into that and collaborate with JAX folks.
For now, feel free to just use one of those kernels based on your platform needs.

Thanks a lot @IvyZX  for integerating flash attention.
I am just afraid that it is still missing some parameters like the dropout, mask and bias, if I am not mistaken.

@IvyZX  could you please share the link for the current PR ?

Dropout is not currently available in Pallas kernels, as it is yet to support PRNG keys. Causal masking can be turned on with causal=True, and the TPU version has attention bias (the arg ab - unclear naming IMO).
I wonder what the real-life use cases for attention bias are, as many LLMs nowadays doesn't use bias any more. Could anyone share why they would like to have it?

If I am not mistaken, many of the state of the art Embedding method requires the attention bias like ALibi and relative attention encoding for T5 models.

It seems that this triton implementation supports attention bias, so there is nothing that prevents the algorithm from supporting it.
https://github.com/Dao-AILab/flash-attention/blob/main/flash_attn/flash_attn_triton.py
Additionally, this implementation of block-parallel attention (faster than FA-1) also supports attention bias:
https://github.com/young-geng/EasyLM/blob/main/EasyLM/bpt.py


Add support for Azure, OpenAI, Palm, Anthropic, Cohere Models - using litellm

Proposed changes
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request. If it fixes a bug or resolves a feature request, be sure to link to that issue. 👀🔧
This PR Adds support for  Azure, OpenAI, Palm, Anthropic, Cohere Models - using litellm
I'm the maintainer of litellm https://github.com/BerriAI/litellm - a simple & light package to call OpenAI, Azure, Cohere, Anthropic API Endpoints
This PR adds support for models from all the above mentioned providers
Here's a sample of how it's used:
from litellm import completion

## set ENV variables
# ENV variables can be set in .env file, too. Example in .env.example
os.environ["OPENAI_API_KEY"] = "openai key"
os.environ["COHERE_API_KEY"] = "cohere key"

messages = [{ "content": "Hello, how are you?","role": "user"}]

# openai call
response = completion(model="gpt-3.5-turbo", messages=messages)

# cohere call
response = completion("command-nightly", messages)

# anthropic call
response = completion(model="claude-instant-1", messages=messages)

Types of changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply 😄🚀

[ ] Bugfix (non-breaking change which fixes an issue) 🐛
[x] New feature (non-breaking change which adds functionality) ✨
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected) 💥
[ ] Documentation Update (if none of the other choices apply) 📖

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code. ✅

[x] I have read the CONTRIBUTING.md 📚
[ ] I have added tests that prove my fix is effective or that my feature works ✅✔
[ ] I have added necessary documentation (if appropriate) 📝

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc... 💡❓
References and related issues (e.g. #1234)
N/A 📌

@RayVentura if this looks good, happy to add tests and docs on how to use this with other model providers 😊

@ishaan-jaff @RayVentura any updates on this?

@ishaan-jaff worked this and tested as much as i could (i am only a student) everything works fine can add documents and test case if wanted to be, would like to work with you guys on this for actual experience

That's mean no need to use openai api key. Great, because I don't have purchased api for openai :v


Merge composite and compound expr test cases

Fixes test failure on main due to conflicting representations of compound expressions after landing from #1600 and #1551

cc @alamb
FYI @ayman-sigma @goldmedal

❤


add test kitchen support

this patch allow to converge cookbook in Test Kitchen. Lets fix all other object the same way.

any updates?

@scopenco  will check tomorrow

did you checked?

hi, any updates?

@scopenco sorry about that, been busy lately. from the glimpse of it, kitchen does not like the method objects. i am skeptical about the code change to get kitchen to pass the convergence.

test kitchen right now is the most popular method of cookbook testing. I dont understand why you are not agree to adopt version to test kitchen. i'm ready to create pull requests for this.

@vkhatri This error is not related to test-kitchen overall. It is reproduced on Chef Client 12.0.3.
So, this is a Chef syntax error, meaning that ruby code (and method calls in particular) should not be placed to value of resource's attribute.
The similar problem: http://stackoverflow.com/questions/15743546/chef-create-template-with-dynamic-variable
Method's name doesn't make sense, and there are two way to fix that:

To avoid method calls in the resource's attribute (like @scopenco have suggested in this pull-request)
Either just use "Lazy attribute evaluation"
https://docs.chef.io/resource_common.html#lazy-attribute-evaluation

Please, fix it by whatever you like.
P.s. By the way, what do you think about enabling 'Issues' in this repo? It will be very useful for issue reporting. Thanks!

Issues should be reported to dev.icinga.org where all Icinga developers organize their tasks. We won't enable a secondary issue tracker here, sorry.
https://dev.icinga.org/projects/chef-icinga2

@scopenco i am not oppose to fixing kitchen test. it make more sense now to know that it causes issue with Chef 12.x. this cookbook is not yet tested for Chef 12. @legal90 thanks for pointing the issue.


[xcb] Unknown sequence number while processing queue - XInitThreads -ubuntu

SQL Operations Studio Version: 0.28.6

Steps to Reproduce:
Installed Ubuntu version of it, While launching the application getting below error.
[xcb] Unknown sequence number while processing queue
[xcb] Most likely this is a multi-threaded client and XInitThreads has not been called
[xcb] Aborting, sorry about that.
/home/sunila/sqlops-linux-x64/sqlops: ../../src/xcb_io.c:274: poll_for_event: Assertion `!xcb_xlib_threads_sequence_lost' failed.
Aborted (core dumped
Sometimes it is working but not launching app.

Are below all libraries needed?
alsa-lib (alsa-lib-x205ta)
cairo (cairo-git, cairo-ubuntu, cairo-dfb, cairo-minimal, cairo-infinality, cairo-nomesa, cairo-infinality-ultimate)
curl (curl-http2-git, curl-git, curl-ssh)
fontconfig (fontconfig-git, fontconfig-infinality, fontconfig-ubuntu, fontconfig-srb, fontconfig-infinality-ultimate)
gcc-libs (gcc-libs-multilib-x32, gcc-libs-git, gcc-libs-multilib-git)
gconf (gconf-gtk2)
glibc (glibc-git)
gtk2 (gtk2-patched-gdkwin-nullcheck, gtk2-patched-filechooser-icon-view, gtk2-ubuntu)
libsecret
libunwind (libunwind-git)
libxkbfile
libxss
libxtst
nss (nss-hg)
python
krb5

I tried to run it but it keeps freezing the VM so I have to terminate it each time. I tried reinstalling but I get the same issue. I type 'sqlops' into terminal and I get the following message: update#setstate idle
and then it eventually freezes.
I have tried installation using Debian as well as tar. Also increased VM size to 16 GB RAM & 4 cores.

@sajagekar I tested the July release on a default Ubuntu installation and it worked as expected installed either from the .deb and .tar.gz file.  This could be a conflict with some other software installed on your machine.


ims registrtion failed with sipp

Hi @herlesupreeth,
Issue:
registration failed with sipp, 403 Authentication failed
sipp(UE simulator) ------> PCSCF + ICSCF + SCSCF + pyHSS

  1. UE Configured in pyHSS with required info ki, opc, amf
  2. sipp does not provision for "OPC", it has an "op" configuration option
  3.  but pyHSS does not have an "op" configuration option. I suspect this could be the root cause.

reg.zip
please suggest, if is it possible to configure "op" in pyHSS

You can use this repo - https://github.com/PodgroupConnectivity/kiopcgenerator to get OPc for a given Ki and OP value

Hi @herlesupreeth,
Issue: registration failed with sipp, 403 Authentication failed sipp(UE simulator) ------> PCSCF + ICSCF + SCSCF + pyHSS

  1. UE Configured in pyHSS with required info ki, opc, amf
  2. sipp does not provision for "OPC", it has an "op" configuration option
  3.  but pyHSS does not have an "op" configuration option. I suspect this could be the root cause.

reg.zip please suggest, if is it possible to configure "op" in pyHSS

Hey, how did you manage to register the SIPP simulator? I'm trying to do that but it's not possible? Can you share your SIPP configs and commands?


Fix problem with chmod for insiders test on mac

Fixes #14553
The step should only run on linux for the insiders and smoke tests

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #14558 into main will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main   #14558      +/-   ##
==========================================
- Coverage   65.01%   64.99%   -0.03%
==========================================
  Files         539      539
  Lines       25050    25050
  Branches     3533     3533
==========================================
- Hits        16287    16281       -6
- Misses       8102     8104       +2
- Partials      661      665       +4

Impacted Files
Coverage Δ

src/client/common/utils/platform.ts
60.00% <0.00%> (-12.00%)
:arrow_down:

.../locators/services/virtualEnvironmentIdentifier.ts
94.73% <0.00%> (-2.64%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
:arrow_down:

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 b30b044...d9cf607. Read the comment docs.


Update devtools

Connected to: https://github.com/LLNL/serac_tests/pull/12

Updates ATS to include the fix for the misparsing of the SLURM version
Fixes and uses the sphinx we build and install
Includes a bandaid fix for spack activate being broken and the -f fix not fully working without adding all the dependencies by hand (minus the ones that can't be activated)
Parses the last ATS junit file in the current submodule for FAIL to trigger failure since ATS does not currently return an error code (ATS issue created)
Removes user-specific information from summary file and moves it to serac output.  This will reduce rebaseline conflicts.

Codecov Report

Merging #735 (8fa0ad4) into develop (175ae62) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #735      +/-   ##
===========================================
- Coverage    94.83%   94.81%   -0.03%
===========================================
  Files          135      135
  Lines         8699     8699
===========================================
- Hits          8250     8248       -2
- Misses         449      451       +2

Impacted Files
Coverage Δ

src/serac/infrastructure/cli.cpp
87.50% <100.00%> (ø)

src/serac/infrastructure/variant.hpp
96.36% <0.00%> (-3.64%)
:arrow_down:

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 175ae62...8fa0ad4. Read the comment docs.

This is now ready to be reviewed.


Add Constant type/value properties

This is some draft of the edits I've made (currently only to v26 as that's the version of the locally generated docs) to be able to extract a list of constants from a crate as mentioned in https://github.com/obi1kenobi/trustfall-rustdoc-adapter/issues/279
Unresolved question:

currently I'm using serde json to de/ser the Type  since there is no conversion to trustfall::FieldValue implemented for this type, but that seems clunky, so what's the easiest way to have a proper support of this conversion (is it only possible in the trustfall main crate?)

it's clunky since on the receiving side I can only try_into_struct

  struct Output {
    // type_  	: rustdoc_types::Type, // Type→FieldValue not implemented, use ↓ json→string
    type_     	: String,
  }
let results:Vec<_> = trustfall::execute_query(&schema, adapter.clone(), query, variables.clone()).expect("failed to run query")
    .map(|row| row.try_into_struct::<Output>().expect("shape mismatch")).collect();

into a string field, can't make it deserialise from json automatically).
(haven't done any formatting/tests yet)

Also, are you interested in associated constants as well, or just regular ones? If the associated constants are missing fields, I'd be happy to take a PR there as wel

just the regular ones, the associated ones already have the default value though seems to miss the type_ field
AssocConst
Fields
type_: Type
default: Option<String>

Done the last thing I wanted -  updated a test, though had to split logic for Constants with more properties like expr/value and for GlobalValues that don't have those and only test name/path as before
(although I've only run the local test for this updaed const crate, not the whole test suite)
Then pending the unresolved doc formatting edits should be more or less fine?
And as far as I understood, porting to other rustdocs versions is semi-auto-done with the help of your magic script?

Looks great. Going to merge this, add the newline in the docs, and cut a new release that you should be able to use.
Thanks for the help! Let me know how this all works out in your use case.

And as far as I understood, porting to other rustdocs versions is semi-auto-done with the help of your magic script?

Yup! I'll take care of porting it and cut a release for all the rustdoc versions that cargo-semver-checks currently supports: v24, v26, v27.

New versions are available on crates.io!


Numbered parameter mismatched Error strings

Two basic SyntaxError strings:
-> { _1; -> { _2 } }

Expected SyntaxError ((?m-ix:numbered parameter is already used in))
but got: SyntaxError ((eval):1: Numbered parameter is already used in outer scope)

AND:
 _1 = 0
  proc { _1 }.call("a")

Expected SyntaxError ((?-mix:_1 is reserved for numbered parameter))
but got: SyntaxError ((eval):1: Token reserved for a numbered parameter)

The second one has been fixed on main. @eregon any chance you can update ruby/spec such that it checks for that regex case-insensitive?

We can adapt the specs if needed to allow either.
But shouldn't Prism be as much as feasible compatible in its SyntaxError messages? (unless the CRuby message is pretty unclear, then it can be worth to improve it, but there is almost always a compatibility cost to change those things).
I think exception messages starting with an uppercase letter are very rare if they exist at all in CRuby, i.e. I think they all/almost all start with a lowercase letter.

It's a complete mix.
 It's not at all standardized.
On the other hand, we have a style guide for our error messages and are attempting to standardize them. I'd prefer that we keep them capitalized since we have consistency there.
I agree with the idea of compatibility to a certain extent, but at the same time we're going to provide many more error messages and many more specific error messages than the current Ruby parser, so this is a place where we're going to have to deviate.

From experience with exception messages in TruffleRuby, it's almost never worth the cost to deviate, there is basically always a gem depending on an exception message and it's a pain to update those.
Of course for completely new errors or so it's more free.

 It's not at all standardized.

I think these Invalid are about the only uppercase error messages.
All TypeError, ArgumentError etc from CRuby start with a lowercase letter, isn't it? I would strongly recommend to not try to change that but instead change the style guide, because it's only going to cause needless friction towards making Prism the default parser in CRuby and using Prism in tools instead of Ripper/parser gem.
Getting some quick stats in ruby/ruby:
git grep -P 'rb_raise[^"]+"[A-Z]' | wc -l
154
git grep -P 'rb_raise[^"]+"[a-z]' | wc -l
1350

And amoung these 154 many of them are class/module names.
git grep 'compile_error(' also shows most (all?) of the SyntaxError are lower-case.
Invalid break comes from compile.c, which may be the reason for the inconsistency.

Does it make sense to have two "modes" of the parser for raising exception messages - CRuby compatible and not compatible modes?
As a replacement of the current CRuby parser - we need compatibility. But when Prism is used as a gem for linters/formatters/etc - such compatibility isn't required. And the more a message is informative/helpful the better.

I would really prefer to avoid "modes" to be honest. I'm generally scared of the divergence. I've pushed back on this for the syntax tree as well for this same reason.
What I would prefer would be to provide a rich enough interface that it can satisfy all consumers. In this case I believe that eventually means providing different error classes that subclass SyntaxError so people can rescue individually (since right now, people rescue SyntaxError and then parse out the message).


Why specify C compiler if there is no C code in the project?

https://github.com/tenstorrent/tt-umd/blob/6deb8d7d2c6513af090d91c58e3ace53b4564b4e/CMakeLists.txt#L10

@blozano-tt status?

Its possible some of the dependencies have C code.
Lets just leave it until it becomes a problem.


Create withUserContainer helper method

Protip for doing this:

Create a helper function, we usually call them containers

export const withUserContainer = (cb) => {
  return async (
     _parent: void,
     arg: ...,
     { req }: Context
   ): Promise... => {
         const { id } = arg
         const userId = req.user?.id
         if (!authorId) throw new Error('No User')
         return cb(_parent, arg, {req, userId} )
}

Then now you can use this container for every resolver you would wish to make sure userId exists:

export const flagExercise = withUserContainer(async () => {} )

export const removeExerciseFlag = withUserContainer( async () => {} )

Originally posted by @songz in https://github.com/garageScript/c0d3-app/pull/2012#discussion_r906688879

Nice.

Reopened for now, will close again once withAdminContainer is merged


public ip should be persisted between restarts

It is possible that the IP will not be reported, if IP has changed but Telegram API was not available at that time.


cc1plus

Hi all,
luarocks install cutorch-scm-1.rockspec gives:
gcc: error trying to exec 'cc1plus': execvp: No such file or directory
CMake Error at THC_generated_THCBlas.cu.o.cmake:207 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-5116/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCBlas.cu.o
make[2]: *** [lib/THC/CMakeFiles/THC.dir/THC_generated_THCBlas.cu.o] Error 1
make[2]: *** Waiting for unfinished jobs....
gcc: error trying to exec 'cc1plus': execvp: No such file or directory
CMake Error at THC_generated_THCHalf.cu.o.cmake:207 (message):
Error generating
/tmp/luarocks_cutorch-scm-1-5116/cutorch/build/lib/THC/CMakeFiles/THC.dir//./THC_generated_THCHalf.cu.o
Cheers,
Zhong

I'm setting export CC=gcc-4.8 down from 4.9 and getting the same errors. Can't see anything recent in the commit log which would cause it, so possibly an issue with our local configurations.

Installing g++ 4.8 seems to do the trick. sudo apt-get install g++-4.8.
Even though I still have g++ 4.9 set in environment.
-- The C compiler identification is GNU 4.8.4
-- The CXX compiler identification is GNU 4.9.2
-- Check for working C compiler: /usr/bin/gcc-4.8
-- Check for working C compiler: /usr/bin/gcc-4.8 -- works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++
-- Check for working CXX compiler: /usr/bin/c++ -- works

Although do get a bunch of warnings, like:
CMake Warning at /usr/share/cmake-3.0/Modules/FindCUDA/make2cmake.cmake:66 (message):
   Removing non-existent dependency file: generic/THCStorage.h

You don't need to worry about these warnings, it's a known problem with CMake. If you don't want to see them anymore just run luarocks install FindCUDA.

You don't need to worry about these warnings

They're just it looking for CUDA 7.0 tensor types right? Not implemented in 6.5.

It's just an artifact of how the CUDA build system works, not connected to anything to do with implementations or versions.


[Question]: Excuse me, does the llamaindex support custom http calls Embedding and LLM?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I have an LLM service that exposes only http post requests. The same goes for Embedding.
How do I use llamaindex to call them?

@WhyMEliu implement a custom llm and custom embedding model that wraps your API requests
https://docs.llamaindex.ai/en/stable/examples/embeddings/custom_embeddings.html
https://docs.llamaindex.ai/en/stable/module_guides/models/llms/usage_custom.html#using-custom-llm-advanced

Thank you very much for your help.

@WhyMEliu implement a custom llm and custom embedding model that wraps your API requests https://docs.llamaindex.ai/en/stable/examples/embeddings/custom_embeddings.html https://docs.llamaindex.ai/en/stable/module_guides/models/llms/usage_custom.html#using-custom-llm-advanced

Hi Logan, Sorry to open this issue again.
If I build custom LLM from this way, and I use this LLM with response_synthesizer: tree summarize,
How can I make this LLM can be called by tree summarize async?

@xjw1001001 you'd have to define the async method in the custom LLM (check out the base class for what I mean)
async def acomplete()
async def achat()
Etc.
For these methods to be truly async (and not fake async) there should be some await used in these methods

@logan-markewich  Hi Logan, thank you for your advice. Here is my implementation for reference
class Claude3_Sonnet(CustomLLM):
context_window: int = 180000
num_output: int = 4000
model_name: str = "custom"
dummy_response: str = "My response"
system_message: str = "You are a helpful assistant"
api_key: str
@property
def metadata(self) -> LLMMetadata:
    """Get LLM metadata."""
    return LLMMetadata(
        context_window=self.context_window,
        num_output=self.num_output,
        model_name=self.model_name,
    )

@llm_completion_callback()
def complete(self, prompt: str, system_message: str ="None", **kwargs: Any) -> CompletionResponse:
    api_url = "https://?????/gpt/v2/claude-3-sonnet-20240229-v1"
    if not self.api_key:
        raise ValueError("API key is not set.")
    if system_message== "None":
        system_message = self.system_message
    headers = {"X-APIKey": self.api_key}
    payload = {
        "max_tokens": self.num_output,
        "messages": [{"role": "user", "content": prompt}],
        "system": system_message,
        "anthropic_version": "bedrock-2023-05-31"
    }

    response = requests.post(api_url, headers=headers, json=payload)
    if response.status_code == 200:
        response_text = response.json().get("content", [{}])[0].get("text", "")
        return CompletionResponse(text=response_text)
    else:
        return CompletionResponse(text=f"Error: {response.status_code} - {response.reason}")

@llm_completion_callback()
def stream_complete(self, prompt: str, **kwargs: Any) -> CompletionResponseGen:
    raise NotImplementedError("This method has not been implemented yet.")

@llm_completion_callback()
async def acomplete(self, prompt: str, system_message: str = "None", **kwargs: Any) -> CompletionResponse:
    api_url = "https://???????/gpt/v2/claude-3-sonnet-20240229-v1"
    if not self.api_key:
        raise ValueError("API key is not set.")
    if system_message == "None":
        system_message = self.system_message
    headers = {"X-APIKey": self.api_key}
    payload = {
        "max_tokens": self.num_output,
        "messages": [{"role": "user", "content": prompt}],
        "system": system_message,
        "anthropic_version": "bedrock-2023-05-31"
    }

    async with httpx.AsyncClient() as client:
        response = await client.post(api_url, headers=headers, json=payload)
        if response.status_code == 200:
            response_text = response.json().get("content", [{}])[0].get("text", "")
            return CompletionResponse(text=response_text)
        else:
            # Use httpx.codes to get a textual reason for the status code
            reason_phrase = httpx.codes.get_reason_phrase(response.status_code)
            return CompletionResponse(text=f"Error: {response.status_code} - {reason_phrase}")

@logan-markewich
Update: Need to add retry and max time for the code:
`
@llm_completion_callback()
async def acomplete(self, prompt: str, system_message: str = "None", **kwargs: Any) -> CompletionResponse:
api_url = "https://???????/gpt/v2/claude-3-sonnet-20240229-v1"
if not self.api_key:
raise ValueError("API key is not set.")
if system_message == "None":
system_message = self.system_message
headers = {"X-APIKey": self.api_key}
payload = {
"max_tokens": self.num_output,
"messages": [{"role": "user", "content": prompt}],
"system": system_message,
"anthropic_version": "bedrock-2023-05-31"
}
    # Define an inner async function for making the HTTP request with retries and timeout
    @backoff.on_exception(backoff.expo,
                        httpx.ReadTimeout,
                        max_tries=3)
    async def make_request_with_retries():
        # Set a custom timeout (e.g., connect=5.0 seconds, read=10.0 seconds)
        timeout = httpx.Timeout(120)
        async with httpx.AsyncClient(timeout=timeout) as client:
            response = await client.post(api_url, headers=headers, json=payload)
            return response

    # Call the inner function
    response = await make_request_with_retries()
    if response.status_code == 200:
        response_text = response.json().get("content", [{}])[0].get("text", "")
        return CompletionResponse(text=response_text)
    else:
        # Use httpx.codes to get a textual reason for the status code
        reason_phrase = httpx.codes.get_reason_phrase(response.status_code)
        return CompletionResponse(text=f"Error: {response.status_code} - {reason_phrase}")

`


Some ByteStringSerializer's don't have a corresponding ByteStringDeserializer

Is there a reason that for example ByteStringDeserializer[Long] is missing?

Yes, because you can't write a ByteStringDeserializer[Long] for any ByteString.

So why do you have a ByteStringDeserializer[Double] then, that one also does not work for any ByteString?

Hum I see, you are right.
DoubleRedis should/could be hidden? (It should be hidden as it is Redis specific)
https://github.com/etaty/rediscala/blob/master/src/main/scala/redis/Converter.scala#L183-L193

Honestly, I think you should just have deserialisers for all of the serialisers. Even if they might throw exceptions. Users can always opt for the String version if they want to be safe.
Are you sure the RedisDouble is Redis specific? As far as I know there is no such thing as double in Redis. It's just parsing the jave Double#toString() output.


bugs in data.tsv (and upstream yaml) from use_modular_gd.py

[ ] if the pattern looks like a list, make it a enumeration/pulldown

solved by processing MIxS after NMDC because the list-like patterns came from NMDC and get overwritten by MIxS?
[ ] they are still in MIxS string serialization

[ ] add hierarchical indentation of enumerated values

I have some old code for that somewhere
probably requires sem-sql code and database (vs a SPARQL solution)
[ ] use any ontologies besides EnvO?
see #58

[x] Add support for partial date columns and time columns

[x] align section composition and ordering with @mslarae13's Example Use tab

that info will be added to other more structured tabs

[x] tidy the descriptions

[x] remove quotes (from NMDC, like env_local_scale. When MIxS is processed after NMDC, the quote-free MIxS annotations supersede.) See https://github.com/microbiomedata/nmdc-schema/issues/229
[x] remove square brackets (from NMDC, like GOLD sample identifiers) See https://github.com/microbiomedata/nmdc-schema/issues/228
[x] 'TODO' description on some NMDC terms. See https://github.com/microbiomedata/nmdc-schema/issues/231

[x] add meanings for enumerated values

[x] lookup with enum_annotator (see example in Makefile)
[x] expose as Ontology IDs
[x] some enum labels from MIxS have added parenthetic content that makes string matching difficult: meadows (grasses,alfalfa,fescue,bromegrass,timothy)

[x] add more patterns based on

[x] string_serialization
[x] slot's range... pretty thorough at this point

[x] populate examples column?

[x]  but https://github.com/microbiomedata/nmdc-schema/issues/233

[x] are terms being included even though they are marked skip on nmdc_biosample_slots ?

[x] Ontology ID

[x] full URLs (prefer prefixed)

gen-yaml NMDC and MIxS schemas, yq delete imports sections

[x] MIxS slots with NMDC URLs like https://microbiomedata/schema/mixs/ph

[x] terse labels (from apparent prioritization of NMDC over MIxS annotations?)

[x] parent classes with "https:" prefixes

long-term solution: align section composition and ordering below
short-term solution comes from full URLs (prefer prefixed) above

[x] seems like number of required fields too low

added requirements from slot usages

[x] elaborate on the use of regular expressions in the guidance column. Also include the string serialization?

[x] Where is the default PV in the sample_type enum coming from

@click.option('--default_data_status', default="default", show_default=True)

[x] what does the Null values section in the double-click header help mean? see https://github.com/cidgoh/DataHarmonizer/issues/244

shows the contents of the data status column in data.tsv, which I was populating with --default_data_status

[x] take advantage of min and max values for pH (anything else?)

[x] whose id-like fields should be used? The ones from NMDC or ones created by @mslarae13

using identifiers from biosample_identification_slots

MIxS slots with NMDC URLs like https://microbiomedata/schema/mixs/ph

just put the MIxS term requests second in the tasks dict

Ontology ID/full URLs (prefer prefixed)

example: https://microbiomedata/schema/ecosystem
would prefer for that to appear as nmdc:ecosystem
Current definition of nmdc prefix::
nmdc:
    prefix_prefix: nmdc
    prefix_reference: https://microbiomedata/meta/

enums included so far, with enrichment success

cur_land_use_enum medium
drainage_class_enum poor
fao_class_enum good
profile_position_enum errors out on first term, backslope
soil_horizon_enum poor
tillage_enum errors out on first term, chisel

Tabulation of ranges for NMDC and MIxS as-is only
To-do
external identifier       3
double                    1

enums
cur_land_use_enum         1
drainage_class_enum       1
fao_class_enum            1
profile_position_enum     1
soil_horizon_enum         1
tillage_enum              1

Handled already
string                   39
    xsd:token by default
quantity value           16
date                      3
    xsd:date
    see notes above regarding partial dates and times

New tabulations:
TABULATION OF SLOT RANGES, for prioritizing range->regex conversion
string                           38
quantity value                   16
external identifier               3
date                              3
oxygen_relationship_enum          1
storage_condt_enum                1
soil_horizon_enum                 1
sample_type_enum                  1
samp_biotic_relationship_enum     1
profile_position_enum             1
fao_class_enum                    1
growth_facility_enum              1
env_package_enum                  1
drainage_class_enum               1
cur_land_use_enum                 1
analysis_type_enum                1
tillage_enum                      1
dtype: int64

TABULATION OF STRING SERIALIZATIONS, for prioritizing serialization->regex conversion
<none>                                                                                30
{PMID}|{DOI}|{URL}                                                                                20
{text}                                                                                8
enumeration                                                                                7
{float} {unit}                                                                                4
{termLabel} {[termID]}                                                                                4
{text};{float} {unit}                                                                                3
{integer}                                                                                2
{timestamp}                                                                                2
{text}:{text}                                                                                2
{text};{timestamp}                                                                                1
[summit|shoulder|backslope|footslope|toeslope]                                                          1
{PMID}|{DOI}|{URL}|{text}                                                                               1
{termLabel} {[termID]}|{text}                                                                           1
{{text}|{float} {unit}};{float} {unit}                                                                  1
[O horizon|A horizon|E horizon|B horizon|C horizon|R layer|Permafrost]                                  1
{float}                                                                                1
{float} C                                                                                1
{float} {float}                                                                                1
{termLabel} {[termID]}; {timestamp}                                                                     1
{term}: {term}, {text}                                                                                1
[Acrisols|Andosols|Arenosols|Cambisols|Chernozems|Ferralsols|Fluvisols|Gleysols|Greyzems|Gypsisols|     1
[very poorly|poorly|somewhat poorly|moderately well|well|excessively drained]                           1
[cities|farmstead|industrial areas|roads/railroads|rock|sand|gravel|mudflats|salt flats|badlands|pe     1
{boolean};{Rn/start_time/end_time/duration}                                                             1
HH:MM:SS                                                                                1
{text};{float} {unit};{timestamp}                                                                       1
[drill|cutting disc|ridge till|strip tillage|zonal tillage|chisel|tined|mouldboard|disc plough]         1
dtype: int64

Looks like all tasks here are complete except for the one about hierarchical enums which is covered by other issues. Closing.


Add startpage

Code of Conduct

[X] I agree to follow this project's Code of Conduct.

Submission validity

[X] This is not a personal support request, that should be posted on discussions community.
[X] I checked current issues and this request isn't a duplicate of an existing issue, opened or closed.

Summary
Currently, when visiting the / URI of yourls I get a HTTP 500 Forbidden by the underlying Apache server.
This is surely a valid and secure response in these cases (no shortened or backend URL was given).
But it would be nice to be able to show a logo or a text and/or a link to the login (/admin) page of yourls.
Motivation
Convenience: I set up a go.mydomain.com URL for yourl which is short and looks great and familiar.
Familiarity: A visitor may look at the domain (e.g. go.mydomain.com) and it would be nice to have some cool/professional logo/page there.
Consistency: When I visit that URL it would be nice to have some HTTP 200 OK page that ensures that everything works and a site logo. The same goes for very simple+cheap HTTP(S) checkers that can be then pointed directly to the root URL.
Describe the solution
Show a nice and simple landing page, e.g. with the yourls logo as the default and a link to the backend login.

http://yourls.org/#FAQ;noindex


provide a way to extending server

Added a KeplerMapper class to extend the server. ref #7

complete #7


Add backend stuff for feature 'setting the page owner'

import backend stuff for feature 'setting page owner' from working branch from #371
functions will be called with current #378 client

@api {post} /page_owners.activate Activate page owner configuration
@api {post} /page_owners.deactivate Deactivate page owner configuration
@api {get} /teams.list List all team
@api {post} /teams.create Create team
Add bodyConfirmed field to edit form

functions will not be called with current #378 client

@api {post} /pages.updateLifetime Set lifetime to page
@api {get} /teams.get Get team with given ID
@api {post} /teams.edit Edit team

@sotarok I've fixed! Could you review again?
I've removed addUser & deleteUser. There are replaced by edit.

I'll recreate if requires this feature.


route config error after upgrade barbarous:typescript from 0.3.1 to 0.3.3

Hi, thanks for the hard work for the community, my app was running well till I updated
barbatus:typescript           upgraded from 0.3.1 to 0.3.3
barbatus:typescript-compiler  upgraded from 0.5.12 to 0.6.1
it gives me this error:
browser_adapter.js:77 Error: Route config should contain exactly one "component", "loader", or "redirectTo" property.
I am using the deprecated router.

It's certainly not related to the compiler, search by the error message and fix your router configuration.


Adopt new assert classes

What does this PR do?
Adopt new assert classes
What issues does this PR fix or reference?
#
The PR fulfills these requirements:
[X] Tests for the proposed changes have been added/updated.
[X] Code linting and formatting was performed.
Functionality Before
<insert gif and/or summary>
Functionality After
<insert gif and/or summary>

@albarivas looks like you have some syntax errors:
Error SlackServiceTest Method does not exist or incorrect signature: void isTrueEquals(String, String) from the type System.Assert (68:[20](https://github.com/trailheadapps/ready-to-fly/actions/runs/3901131958/jobs/6668639191#step:7:21))
 Error SlackServiceTest Method does not exist or incorrect signature: void isTrueEquals(String, String) from the type System.Assert (64:20)
 Error SlackServiceTest Method does not exist or incorrect signature: void isTrueEquals(String, String) from the type System.Assert (59:20)
 Error SlackServiceTest Method does not exist or incorrect signature: void isTrueEquals(String, String) from the type System.Assert (63:20)

Did you do a bad search and replace?


fix(event): fix event display problem

root@VM-0-9-ubuntu:/home/ubuntu# kubectl  get event
LAST SEEN   TYPE     REASON                    OBJECT                                      MESSAGE
...
50m         Normal   NodeHasSufficientPID      node/k8s-dra-driver-cluster-worker          Node k8s-dra-driver-cluster-worker status is now: NodeHasSufficientPID
50m         Normal   NodeAllocatableEnforced   node/k8s-dra-driver-cluster-worker          Updated Node Allocatable limit across pods
50m         Normal   RegisteredNode            node/k8s-dra-driver-cluster-worker          Node k8s-dra-driver-cluster-worker event: Registered Node k8s-dra-driver-cluster-worker in Controller
50m         Normal   Starting                  node/k8s-dra-driver-cluster-worker
50m         Normal   NodeReady                 node/k8s-dra-driver-cluster-worker          Node k8s-dra-driver-cluster-worker status is now: NodeReady
48m         Normal   GPUDriverUpgrade          node/k8s-dra-driver-cluster-worker          Successfully updated node state label to [upgrade-done]%!(EXTRA <nil>)

root@VM-0-9-ubuntu:/home/ubuntu# kubectl logs -f gpu-operator-64bdf76697-8d8kr -ngpu-operator | grep "Successfully "
{"level":"info","ts":1733661826.0248199,"logger":"controllers.Upgrade","msg":"Successfully changed node upgrade state label","node":"k8s-dra-driver-cluster-worker","new state":"upgrade-done"}

friendly ping @adrianchiris @vasrem
/PTAL


No Local Storage in Game and No High Score Feature

Description
Currently, the game does not implement local storage, and as a result, all game data is lost when the user refreshes the page. Additionally, there is no concept of a high score. By adding local storage and introducing a high score feature, we can ensure that the game state is preserved, and players can see their best performance across sessions.
Steps to Reproduce

Play the game and make progress.
Refresh the browser window.
Notice that all game data is lost.

Expected Behavior

The game should save progress to local storage, so when the user refreshes the page, the game can restore the previous state.
A high score feature should be added where the user's highest score is stored in local storage and displayed in future sessions.

Possible Solution

Implement local storage to save game data (e.g., player progress, current score, etc.).
Introduce a high score feature, saving the highest score in local storage and displaying it when the game is reloaded.

Screenshot

Additional Context
Adding local storage and a high score feature will significantly enhance the user experience by preserving progress and providing players with a goal to beat their previous best score.

Please assign this issue to me

@YadavAkhileshh  please assign me this issue under hacktoberfest accepted


Add notes about pre-stabilization to contributor unstable docs

This PR is meant to add more direction for contributors on the path to stabilization for unstable features. It adds a section titled Pre-Stabilization to the unstable contributor docs.
The idea for this came out of the discussion about when and how to stabilize workspace inheritance. The notes that are being added were derived from the above comment as well as the the adding of the Call for Testing section to TWiR. This comment gives more information as well.
As for the requirement of testing notes, there is still discussion about if they are needed.
While what was added is not comprehensive it is meant as a guide for what to do as each feature has different requirements for stabilization
r? @epage

Yep! I'll make that change!

It looks like this failed for network connectivity issues. Can someone rerun this for me?

@bors r+

:pushpin: Commit 5418d581b78939f8ea6c2fad8e4930cc8ad53cef has been approved by weihanglo

:hourglass: Testing commit 5418d581b78939f8ea6c2fad8e4930cc8ad53cef with merge a4c1cd0eb6b18082a7e693f5a665548fe1534be4...

:sunny: Test successful - checks-actions
Approved by: weihanglo
Pushing a4c1cd0eb6b18082a7e693f5a665548fe1534be4 to master...


Modifying dataTransfert.effectAllowed

Hello ! Thank you for this really useful module.
I'm facing a problem when trying to change the effect when dragging and dropping.
Is there any way to change the config by passing an argument like 'effectAllowed' ?
For example, something like that :
<div dnd-sortable-container [sortableData]="el" [dropZones]="['zone1']">
    <div *ngFor="#e of el; #ie=index" dnd-sortable [sortableIndex]="ie" [effectAllowed]="copy">
        <div>{{e}}</div>
    </div>
</div>

Thanks in advance !

I will do it in next release.

Perfect, thanks !
When have you planed to do it ? (no pressure, just to have an idea :) )

I'm a bit busy for now but I schedule to release no later than this weekend.

Perfect, thanks !

Hi,
Sorry for delay.

I have added input properties:

effectAllowed
effectCursor
Can you check and close if this one was sorted.

Hi,
Can I close this issue?

Hi,
Yes, thanks for the update !


Add Spanner's Boomsticks Continued from Kerbal Stuff

This pull request was automatically generated by Kerbal Stuff on behalf of LORDPrometheus, to add Spanner's Boomsticks Continued to CKAN.
Please direct questions about this pull request to LORDPrometheus.

Closed by https://github.com/KSP-CKAN/NetKAN/pull/2450


Integration Tests for s3-sink related to Github-issue#1048

Description
Integration tests for #1048
Issues Resolved
Resolves: Integration tests for #1048
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

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.

Codecov Report

Merging #2401 (22cb4f1) into main (048db07) will decrease coverage by 0.02%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##               main    #2401      +/-   ##
============================================
- Coverage     93.86%   93.85%   -0.02%
+ Complexity     2081     2079       -2
============================================
  Files           244      244
  Lines          5789     5789
  Branches        470      470
============================================
- Hits           5434     5433       -1
  Misses          239      239
- Partials        116      117       +1

see 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Review how missing data is handled in inversions

If no footprint or obs data is found for a site, that site is dropped. This works well for monthly inversions, but less so for annual inversions.
@joe-pitt had some runs where NaNs due to missing footprint data were propegated to H and caused a confusing pymc error.
Possible improvements could include:

adding options to fill or drop NaN values
modifying the code that drops sites when data isn't found so that it works for annual runs (might be more difficult?)

Also, we should investigate what happens if some obs data is missing. Typically footprint data won't be missing, but we often have missing obs, and the runs work despite this.

Quick update on this - if a month of fp data from the middle of the dataset is missing from the object store (e.g. you have April 2016 and June 2016, but you are missing May 2016) you get the confusing pymc error:
pymc.exceptions.SamplingError: Initial evaluation of model at starting point failed!
If there is a month of fp data missing at the end of the dataset (e.g. MHD obs run to November 2023, but the MHD footprints only run to October 2023) you get a different error:
File "/user/home/zh21490/openghg_inversions/openghg_inversions/hbmcmc/inversionsetup.py", line 100, in addaveragingerror
    fp_all[site.upper()].mf_repeatability.values ** 2
ValueError: operands could not be broadcast together with shapes (870,) (1,943)

In this case there are no nans in fp_all, but the timeseries is truncated to the end of the fp data, which causes these subsequent issues. I think the first step would be to raise a more informative error when either of these cases occur, e.g. "Some observations could not be matched to footprints - check that all footprints are loaded in object store". Even better if it can tell you which footprints are missing!

If the error happens during "add averaging error", then that might be because the obs data is loaded again from the object store for this part, so probably obs were dropped when the ModelScenarios are made, and then we try to add some averaging error based on the full obs time series.

Yes I think that is what is happening. So maybe it would run if averaging error was turned off, but by default I think we probably want it to fail, or at the very least give us a strong warning that some of the obs were removed due to missing fps

For now, you might just want to turn off "addaveragingerror", because when it loads the obs, it specifies an averaging period, so the averaging error of that data (over the same period) is zero... so I don't think it does anything. I haven't got around to fixing that, but there is an issue open https://github.com/openghg/openghg_inversions/issues/42

OK cool thanks! Although in my case I found the missing fps so I'm running fine now

I'm having this issue as well @brendan-m-murphy : it looks like when I have times that have footprints/no obs, no problem but if there's obs/no footprints I get the same issue as joe (pymc.exceptions.SamplingError: Initial evaluation of model at starting point failed!). Ideally we'd want a way of skipping the timestamps without fp in the inversion (presuming this is what happens when there are no obs?). Any idea where that would be best?
Looks related to #96 with the logp -inf issue, as well

It should be possible to fix this using 'dropna("time")' on the model scenarios before they're converted to numpy. I'll have a look Monday.


Retire Fuzzy zkapp tests in CI

Fuzzy tests gave us too many false failures. Those tests are also long and blocks releases. We tried to maintain them and expand timeouts (individual and total). Therefore we decided to remove them from CI.

!ci-build-me

!ci-nightly-me

Delete buildkite/src/Command/FuzzyZkappTest.dhall as well?

@deepthiskumar Maybe i'm blind :( but i don't see https://github.com/MinaProtocol/mina/tree/dkijania/remove_fuzzy_zkapp_unit_tests_from_ci/buildkite/src/Command/FuzzyZkappTest.dhall. I target my branch to master which hasn't got src/Command/FuzzyZkappTests.dhall file (in contrary to develop and compatible)

Weird that it is not in master

!approved-for-mainnet

!approved-for-mainnet

!ci-build-me


anyone have updater?

&offsets::functions::WorldToScreen
//"83 EC 10 56 E8 ? ? ? ? 8B 08",
//"E8 ? ? ? ? 8D 44 24 10 50 8B 44 24 54"
no work anymore.
i look for Unknow cheat but "8B CE E8 ? ? ? ? 83 7E 3C 00 75 17" no work for me, game  crash.
so, plz help for updater bot

I have told you many times that I will not update that pull request. WorldToScreen method takes an ecx parameter after the new update. This indicates that the method is now called with the __thiscall calling convention and an instance must be passed to the method. Open the method in IDA and reverse it.

When I look at the unknowncheats forum, I see that they have already shared the current type definition of the worltoscreen function. No one will give you a ready made code for this source.
learn c++, don't embarrass yourself on the forums


Validation for Form Schemas

Definitions

Form Schema (AKA "Schema") A JSON document which describes a form (e.g. a Contact Name & Address Form)

Form Schema Specification (AKA "Spec") - A JSON Schema document which can be used to validate a Form Schema.

Issue
There should be a Spec against which a Schema can be validated.  angular-schema-form creates the form from the schema and also validates the form fields against the same schema.

There's the beginnings of a form schema specification here. It's a little difficult to generalize things because different formly components can require such different things, but I think most of the common stuff is there.

There's the beginnings of a form schema specification here. It's a little difficult to generalize things because different formly components can require such different things, but I think most of the common stuff is there.


"SharedArrayBuffer is not defined" error after a web build with a custom export template and Thread Support disabled

Tested versions

Reproductible in b75f0485b

System information
Godot v4.3.beta (b75f0485b) - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3070 Ti (NVIDIA; <IP_ADDRESS>61) - AMD Ryzen 5 5600X 6-Core Processor (12 Threads)
Issue description
I'm trying to reduce the size of a web export in order to upload the game to Poki. To do this I pulled the godot source code of the 4.3 beta 2 release and made a custom export template with only 3d disabled. I made sure to disable Thread Support and enable Cross Origin Isolation.
Despite this, I get a "SharedArrayBuffer is not defined" error in the console.
When using the default web export template it works fine
Steps to reproduce
Pull commit b75f0485b of the godot repo. (4.3 beta 2)
Make a custom web export template with
scons platform=web tools=no target=template_release (using emscriptem 3.1.61)
Use the newly compiled export template to export your game. Disable Thread Support and enable Cross Origin Isolation
When I upload the game here https://inspector.poki.dev/, I get the "SharedArrayBuffer is not defined" in the browser's console
Minimal reproduction project (MRP)
N/A

As far as I know the thread support is not something that can be turned on and off on the specific template, you need to compile it with the correct option as it's part of the compiled code

Indeed, you need to compile with threads=no. The checkbox impacts which official export template is used, but if using a custom one, you have to make sure it's built with the matching option.
@adamscott We need to add instructions for re: threads to https://docs.godotengine.org/en/latest/contributing/development/compiling/compiling_for_web.html

That's done in:

https://github.com/godotengine/godot-docs/pull/9392

This seems to have done the trick! Thanks for the quick response! Closing


Unpack output

Lot's of thanks for creating the library. I'm not used to the transformers library, ¿how can I unpack the output?

Upvote & Fund

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

Another issue: ¿where can I find the robotics datasets? Thanks


More than one third-level nest causes alignment issues

You can see here with CSS from develop branch:
https://canjs-slickmap.glitch.me/index-develop.html
As well as here with CSS copied from the live demo:
https://canjs-slickmap.glitch.me/index.html

swapping out to top: calc(-100% + 25px); here works well (at least for the default metrics in this lib):
https://github.com/astuteo/slickmap/blob/669e623d0e60939911ccadb9ebd632c129fa0d17/slickmap.css#L229-L237


Resource Gallery

See https://www.figma.com/file/yv0VtHTvMoGPqkv0RAYcVp/PDP?type=design&node-id=6697-2613&mode=dev
Which block is this based on?
Requirements

Elements:

Heading
Tertiary button, toggle
Video

Thumbnail
Play button
Title
Metadata text

Document

Link/location
Title
Metadata text

Styling

On the default (collapsed) state, a maximum of 6 videos and 6 document files will be exposed

Both videos and document files are in their own list. If there are 4 videos, the document files remain in place.

If the video title length exceeds one line, the text will truncate with ellipsis (text-overflow to be used)
Colours change according to context

Behaviour

Toggle button

“View All” button will reveal all content. On tablet and mobile, the carousel will be replaced with a list view.
“View Less” will revert the component back to the default state

Tapping a video thumbnail will open a modal, showing the video

Here's an example of a similar "More Resources" block with 6+ images/CTAs aligned 3 columns, 2+ rows:
https://www.volvotrucks.us/trucks/vnr-electric/specifications/

Agile Requirement
As a business owner I want to be able to display documents and videos in a gallery way so that user's can navigate through the videos and download the documents easily.
Description
Create the Resource Gallery block https://www.figma.com/file/yv0VtHTvMoGPqkv0RAYcVp/PDP?type=design&node-id=6697-2613&mode=dev. There's an existing design which can be used as code base https://www.volvotrucks.us/trucks/vnr-electric/specifications/.
Acceptance Criteria

There is a new block available with the follwing elements:

Heading
Tertiary button, toggle
Video
Thumbnail
Play button
Title
Metadata text
Document
Link/location
Title
Metadata text
Styling

The styling of the block is as follows:

On the default (collapsed) state, a maximum of 6 videos and 6 document files will be exposed.
Both videos and document files are in their own list. If there are 4 videos, the document files remain in place.
If the video title length exceeds one line, the text will truncate with ellipsis (text-overflow to be used)
Colours change according to context.

The behaviour of the block is as follows:

Toggle button.
“View All” button will reveal all content. On tablet and mobile, the carousel will be replaced with a list view.
“View Less” will revert the component back to the default state.
Tapping a video thumbnail will open a modal, showing the video.

Check the volvo trucks existing behavior to see if we can reuse the code if it's not to much trouble. https://www.volvotrucks.us/trucks/vnr-electric/specifications/

@DanielaPedrochevd
"On tablet and mobile, the carousel will be replaced with a list view"
I cannot find any carousel in the design nor in specification. If this is not mistake, please clarify where and which carousel to use.

@ozzy-krishnan @csulitzer Can you help clarify the above question?

@markovukiceviccn @ZoeBeykirch are you referring to a carousel for the resources section? I'm not sure if we have a component for it in the component library, but I have a screenshot of the resources video in a carousel on mobile view :)

@markovukiceviccn As you can see from Ozzy's comment, videos in the mobile and tablet view are displayed in a carousel way. Users should be able to scroll horizontally through the list of videos while on desktop, videos are displayed in rows of 3 in a list view.


Switch to a faster http framework

Possibly fiber or fasthttp?
Not sure if serving 1000 requests per second more really makes a difference. We won't expect more than two thousand per minute or so.

Even on a DDoS scale, the network interface would be the bottleneck:
Tiny test:
Server Hostname:        localhost
Server Port:            8080

Document Path:          /
Document Length:        4045 bytes

Concurrency Level:      6
Time taken for tests:   0.005 seconds
Complete requests:      100
Failed requests:        0
Total transferred:      414100 bytes
HTML transferred:       404500 bytes
Requests per second:    19952.11 [#/sec] (mean)
Time per request:       0.301 [ms] (mean)
Time per request:       0.050 [ms] (mean, across all concurrent requests)
Transfer rate:          80685.26 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0    0   0.0      0       0
Processing:     0    0   0.1      0       1
Waiting:        0    0   0.1      0       1
Total:          0    0   0.1      0       1

Percentage of the requests served within a certain time (ms)
  50%      0
  66%      0
  75%      0
  80%      0
  90%      0
  95%      0
  98%      0
  99%      1
 100%      1 (longest request)

Huge test:
Server Hostname:        localhost
Server Port:            8080

Document Path:          /
Document Length:        4045 bytes

Concurrency Level:      1000
Time taken for tests:   4.018 seconds
Complete requests:      100000
Failed requests:        0
Total transferred:      414100000 bytes
HTML transferred:       404500000 bytes
Requests per second:    24886.06 [#/sec] (mean)
Time per request:       40.183 [ms] (mean)
Time per request:       0.040 [ms] (mean, across all concurrent requests)
Transfer rate:          100637.86 [Kbytes/sec] received

Connection Times (ms)
              min  mean[+/-sd] median   max
Connect:        0   14   3.2     15      24
Processing:     0   26  13.4     24     194
Waiting:        0   20  13.7     19     185
Total:          0   40  13.2     39     207

Percentage of the requests served within a certain time (ms)
  50%     39
  66%     40
  75%     42
  80%     42
  90%     46
  95%     50
  98%     60
  99%    139
 100%    207 (longest request)


Refactor WinUI Helix functionality into reusable WinUI.Helix nuget package

Since starting to use Helix to execute WinUI tests, we have built up a good amount of scaffolding to sit on top of Helix that builds on its out-of-the-box functionality.
This code is generally useful to other teams who want to use Helix to run TAEF tests. The WinUI 2 and WinUI 3 repos both have copies of this code, but it has diverged over time. Also, other teams outside of WinUI have made forks of this code (such as Terminal).
So, I refactored the code in the WinUI3 repo to separate out the Xaml specific code from the general purpose code. The general purpose code is packaged into a nuget package (Microsoft.Internal.WinUI.Helix).
The copy of the code in this repo is removed and we use the nuget package instead.
WinUI.Helix Features
TAEF Support
Out of the box, Helix has support for XUnit which is the test runner that the dotnet team use to run
their tests. WinUI required TAEF support which was implemented on top of Helix's base functionality. There are two parts
to the TAEF support:
TAEF Test Execution
RunHelixWorkItem.ps1 from WinUI.Helix invokes TAEF. It then converts TAEF's .wtl log output to the XUnit log format that
Helix supports. This allows us to execute TAEF tests in Helix and get the test results reported back to the Azure Pipeline.
TAEF Test Discovery
It is possible to hard-code the set of Helix Work Items to run. But this can be fragile when you have a large amount of
tests. GenerateHelixWorkItems.ps1 from WinUI.Helix is a script that can be run from Azure Pipelines to discover TAEF tests
and generate the work items for Helix.
Retry Logic
In the ideal world, tests would be 100% reliable and would never fail except when catching a legitimate error. However, sometimes it is necessary to handle a certain amount of unreliability in a suite of tests. WinUI.Helix includes some retry logic. A test that fails is initially re-run and is only reported as a true failure if it fails multiple times on re-run.
Currently this retry-logic is always enabled. But it could be made opt-out if that was needed.
Test Output as Pipeline Artifact
To aid debugging test failures, the full log output of the executed tests along with any supporting files (e.g. screenshots)
is uploaded as an Azure Pipeline artifact. This is done by WinUI.Helix's ProcessHelixFiles.ps1.
Extension Points
If you need to run some custom logic, but still want to use WinUI.Helix's TAEF execution, you can provide scripts that
will be called at various points in the execution. This is useful if you need to install some dependencies on the test
machine before executing the tests.

/azp run

/azp run

/azp run


Representing data in django using matplotlib

Article checklist
Pre-submission checks

Please ensure that you have only one open topic suggestion (issue) + in-review article (linked pull request) at a time. This will ensure that we complete the review process in a timely manner from inception to publishing

If you have not already, please go over our Resources Page for more tips and tricks on how to improve your overall technical writing so reviews are swifter, increase the chance of a payout and provide more value to our readers.

For first-time contributors and for more details on our submission guidelines, see our Contributing Guidelines.

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes

Remove the backticks and add the issue number below to link and close your Topic Suggestion (issue) when your article has been published (PR has been merged). See this video for more details.

This closes #4170

Finally, delete the article checklist notes in blockquotes and submit your PR. We look forward to reviewing your article.

Done...

I'll complete my final review by EOD

@JohnKiguru
Can you ping me the GitHub repo that contains the full code?
Meanwhile, I'll try to complete the final changes.

Sure thing.
On Wed, 3 Nov 2021, 17:00 Srishilesh P S, @.***> wrote:

@JohnKiguru https://github.com/JohnKiguru
Can you ping me the GitHub repo that contains the full code?
Meanwhile, I'll try to complete the final changes.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/section-engineering-education/engineering-education/pull/4499#issuecomment-959149733,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASTGUEYHJ4HDF4WJ5K625QLUKE57LANCNFSM5GKKW3LA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

https://github.com/JohnKiguru/Working-with-matplotlib-and-Django

@srishilesh
Can you elaborate on what final changes you are working on?
Does this PR still need more edits or revisions ?
If so we can consider closing it to allow the author the time to make the required edits. 🚀

@srishilesh
Can you elaborate on what final changes you are working on?
Does this PR still need more edits or revisions ?
If so we can consider closing it to allow the author the time to make the required edits. 🚀

@hectorkambow
I'm working on reviewing the codes.
Will try to complete the review by today.

Thanks

Merged #4499 into master


Feature Request: Intensified Spell Metamagic

please add Intensified Spell, the +5 Dice damage cap makes it pretty much a must have for magus builds

this will be in 1.11.0 which will be out for WotR 1.2.0


[terra-layout] - Regenerate jest snapshots

Summary
The following pull request introduced a root selector prop that caused the terra-layout snapshot tests to fail.
https://github.com/cerner/terra-core/pull/1983
Additional Details
Regenerated the jest snapshots to account for the default root selector.
Thanks for contributing to Terra.
@cerner/terra

There were some breaking changing to the action-footer within this component as well, I'll regenerate the screenshots on the next commit.


[ ] Yes
[ ] No

If No, please run rush change before, this is necessary.
If adding a new feature, the PR's description includes:

[ ] Reason for adding this feature
[ ] How to use
[ ] A basic example

Does this PR introduce a breaking change? (check one)

[ ] Yes
[ ] No

If yes, please describe the impact and migration path for existing applications:
Summary
Detail
How to test it

Website Preview: https://tiktok.github.io/sparo/_preview/pr-76/

🚀 Deployment Update
The preview of your changes has been successfully deployed! 🎉
🌐 Website Preview
https://tiktok.github.io/sparo/_preview/pr-76/
Please review the changes at the above link. If everything looks good, you can proceed with the merge. If you encounter any issues, please address them and push the changes.
Happy coding! 😃

🚀 Deployment Update
The preview of your changes has been successfully deployed! 🎉
🌐 Website Preview
https://tiktok.github.io/sparo/_preview/pr-76/
Please review the changes at the above link. If everything looks good, you can proceed with the merge. If you encounter any issues, please address them and push the changes.
Happy coding! 😃

Close due to the fact that we don't want to deploy preview page with tiktok.github.io domain.


Workflow fix

Pull Request summary:
Fixing issue with worklfows not starting with only .md files changed.
Description of changes:
Delated in .yml files:
    paths-ignore:
      - '**.md'

and switched branch name from master to main.
Checklist:

[x] Make sure to run pre-commit before commiting changes
[x] Make sure all checks have passed
[x] PR description is clear and comprehensive
[x] Mentioned the issue that this PR solves (if applicable)
[x] Make sure you test the changes

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/175


[topgen] Inter-module validation code

topgen didn't have a validation code for inter-module signal.
This commit is to implement basic signal format part for inter_module.
It checks below:

Check if req/rsp format is module.signal{[index]} ({} means
optional)
Check if request is array and matched the number of entries
Check if response has an array index, and request is not an array

i think it looks good, but i'd really appreciate an example or two (see comment), i can make sure i'm reading it right.


How to fix overflow problem.

Dear
I had run nmt model without attention, I can get result for my data.
But when I add attention to the model, the tensorflow will give me a error as "step 1200 overflow, stop early".
I had try with all 4 type of the attention function, all of them will overflowed at step 1X00.
Any one can give me some tips to fix the problem?
thanks
my model setting is as blow
python  -m nmt.nmt
--src=lf --tgt=tf
--vocab_prefix=/data/xxx
--encoder_type=bi
--attention=normed_bahdanau
--unit_type=gru
--out_dir=/out/xxx
--embed_prefix=/data/xxx
--train_prefix=/data/xxx.space.train
--dev_prefix=/data/xxx.space.test
--test_prefix=/data/xxx.space.test
--num_train_steps=6000
--steps_per_stats=100
--num_layers=2
--num_units=128
--dropout=0.2
--metrics=bleu

I had fix this problem ,by small learning rate


Add an Option to Always Show Progress on 'scrollParent: true' and 'autoCenter: false'

Currently, if the waveform is initialized with 'scrollParent: true' and 'autoCenter: false' progress will not scroll with the waveform. It would be great to have an option to always show progress onscreen, without autoCenter.
So, if progress goes offscreen, the waveform automatically scrolls to show progress on the very left of the displayed waveform.

For anyone looking for how to implement this, this is what I ended up with, inside a setInterval (which I had already been using):
`if (wavesurfer.current.isPlaying()) {
    const progressPixels = Math.round(
      wavesurfer.current.getCurrentTime() * wavesurfer.current.params.minPxPerSec *
      wavesurfer.current.params.pixelRatio,
    );

    const parentWidth = wavesurfer.current.drawer.getWidth();
    const clientWidth = wavesurfer.current.drawer.wrapper.clientWidth;

    if (wavesurfer.current.drawer.getScrollX() + parentWidth < progressPixels) {
      wavesurfer.current.drawer.wrapper.scrollLeft = progressPixels / wavesurfer.current.params.pixelRatio;
    }
  }`


Create modal overlay for user deleting their profile

Tasks

Create modal overlay with style guide design for when user clicks delete button to delete their profile on Profile page
Include a secondary Cancel button and a primary Confirm button saying "Yes, Delete Profile"
Title heading could be "Delete Your Profile?"
Body text could be "Are you sure you want to delete your profile? This action is permanent and cannot be undone."
Add prototype interactions for delete buttons (display overlay), cancel buttons (close overlay) and confirm button (user is taken back to home)

- [ ] Mobile
- [ ] Desktop
Design

Structure: Centered on the screen with a semi-transparent background overlay.
Size: 400-600px wide, depending on content.
Actions: Include primary and secondary buttons for confirmations and cancellations.
Follow style guide for headings and body texts

Parent ticket #108

Completed.

Review completed.


[2.0.0] New Plan

PRs to cherry pick/port over:

https://github.com/Incendo/cloud/pull/203
https://github.com/Incendo/cloud/pull/96

Feature Requests:

https://github.com/Incendo/cloud/issues/66 (More info in replies)
https://github.com/Incendo/cloud/issues/164 (More info in replies)
https://github.com/Incendo/cloud/issues/191

Other Ideas:

Rewrite the examples and move them into a separate repository
Continue rewriting the tests and also test Brigadier support

Every parser should have a test (including platform specific parsers)
Test individual units of code instead of having ugly monolith tests

Support suggestion tooltips
Currently Cloud only allows for string suggestions. This should not be a hard limitation, and the system should be able to support semi-generic suggestions, potentially through the use of a new interface (Suggestion) which can then be used to give platform specific suggestions (potentially a BrigadierSuggestion that extends StringSuggestion)

Move argument requirement into the tree
This is part of the bigger idea of removing all "metadata" from command arguments, and instead storing that in the command tree directly. However, there's an even more powerful version of this, where command arguments are removed as a concept and we only deal with command parsers.
This would allow us to transform
manager.command(
  manager
    .commandBuilder("command", Description.of("Test cloud command using a builder"), "alias")
    .argument(StringArgument.of("input"))
    .argument(IntegerArgument.<CommandSender>newBuilder("number").withMin(1).withMax(100).build())
    .handler(context -> {
      String input = context.get("input");
      int number = context.get("number");
      context.getSender().sendMessage(String.format(
        "I am %d%% hyped for %s!",
        number,
        input
      ));
    })
);

into
manager.command(
  manager
    .commandBuilder("command", Description.of("Test cloud command using a builder"), "alias")
    .required("input", StringArgument.parser())
    /* can we get rid of that explicit generic arg? */
    .required("number", IntegerArgument.<CommandSender>parser().withMin(1).withMax(100))
    .handler(context -> {
      String input = context.get("input");
      int number = context.get("number");
      context.getSender().sendMessage(String.format(
        "I am %d%% hyped for %s!",
        number,
        input
      ));
    })
);

However, required(CommandArgument) and optional(CommandArgument) could still exist, where CommandArgument<CommandSender, ArgumentType> could be a holder of argument metadata. This allows for this pattern:
private final CommandArgument<Player, Integer> ITEM_AMOUNT = CommandArgument.of("amount", IntegerArgument.<CommandSender>parser().withMin(1).withMax(100)));

which then allows for  builder.required(ITEM_AMOUNT)

It would also be possible to bring in https://github.com/Incendo/cloud/pull/237, although I am not sure I love the approach. It's worth discussing.


Removed the deletion test until the service deletion works reliably

Describe what this PR does and why we need it:
Changes proposed in this pull request

We are seeing too many un-erasable project on our CI openshift.
So I am removing the test that is generating these projects until we fix the underlying culprit

This will be renabled when https://issues.jboss.org/browse/AEROGEAR-2176 is done


[Suggestion] Acidic radrock spouting recipe

Acidic radrock usually takes 4 radrock and a bucket of acid in a crafting table.
It would be a lot more convenient to be able to spout 250mb of acid onto radrock to get acidic radrock!

Good idea

Should be added in latest commit


Some bug fixes

just some misc bugs i fixed (mostly due to trouble on seals)
:warning: DRAFT

This is probably safe to delete since Seal Codex is now Seal Wildbook. This was in relation to this issue: https://community.wildme.org/t/no-matches-issue/2777


Add aarch64 platform to the nightly/release pipelines.

Relates to https://github.com/AdoptOpenJDK/openjdk-website/issues/251
We changed to incrementally release builds to the website (to ensure nightlies were regular, as we had some platforms that were failing to build for various issues, that would block all platforms from being released).  By doing so, the aarch64 build is no longer being pushed to the website, as the
aarch64 platform is being built via a shell script at present, and was getting picked up for deployment prior to the pipeline and release tool changes.
We should add it to the pipeline scripts, so that the build/test/deploy story for all platforms is consistent.

@smlambert any updates on this?

Sorry, I actually delayed to add them, because it wasn't clear whether @karianna build rewrite was going to use the build pipeline scripts or change up the underlying stuff radically.  But I will proceed with my PR and deal with any changes later.

So an update, is that we did add it to the build pipelines, and the build and tests are triggering properly, the problem must lie with the checksum/release_tool step, investigating further now.

fixed by #318


Activity log for an app

As a cloud.gov developer I want to be able to see the most recent activity on my instance so that I can diagnose problems.

Moving this to #189


Ensure flatbuffers serialization works

Currently using custom serialization code, it will be more maintainable to use flatbuffers, and most of the work was done already.

I've actually done this, and I'm using flatbuffers now.


Custom Templates for Search

It'd be great to be able to use custom templates for the search module. It might work well to spec a template like this:
$(".ui.search").search({
   ...
   template: "myTemplate"
});

{{title}}

If this already exists and I missed it, apologies. Didn't see anything like this in the docs, tho.

You can modify search templates by altering $.fn.search.settings.templates.
They are currently simple JS concatenated strings, but could also work with compiled handlebar templates, although i haven't tested
If you want to add your own template just specify a new template name as type on init.
For example
$.fn.search.settings.templates.special = function(response) {
 // do something with response
};
$('.ui.search')
  .search({
    type: 'special'
  })
;

You can also specify on init, but you'd have to do this each time.
$('.ui.search')
  .search({
    type: 'special'
    templates: {
      special: function(response){

      }
  })
;

Awesome, thanks for that info. I'll go that route. Should I leave this open so it makes its way into the docs at some point?

Yes

Added to 2.0 docs with explanation

Should not these settings be found on the settings page?

I tried to use custom template (using items). But I can't control dropdown size. How to solve this?


[BUG] "No module named 'robotcode.plugin'" error when using robotcode cli

Describe the bug
when attempting to run the robotcode command from the pypi package, the following error occurs:
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\Users\user\project\.venv\Scripts\robotcode.exe\__main__.py", line 4, in <module>
  File "C:\Users\user\project\.venv\Lib\site-packages\robotcode\cli\__init__.py", line 9, in <module>
    from robotcode.plugin import (
ModuleNotFoundError: No module named 'robotcode.plugin'

To Reproduce
run the following commands:

pip install robotcode
robotcode

Expected behavior
robotcode cli runs on my project like it does in the vscode extension
Desktop (please complete the following information):

RobotCode Version 0.71.0
OS: windows 11
Python Version 3.12
RobotFramework Version 7.0

Additional context
i'm not sure if the cli is even intended to be used outside of the extension like this, but my use case is to add a robotcode check in my CI to match the checks the vscode extension does locally

Thanks for the issue, I have corrected it.
The RobotCode Command Line Interface is used internally by RobotCode, some parts are already finished but others not yet, I'm still working on it.
What is still missing as a major point is the analysis of a Robot Framework project, i.e. "Keyword not found" and something like that, the language server already does this, but it doesn't work on the command line yet.
At RoboCon 2024 Online I will be giving a tutorial on RobotCode CLI Tools, I want to have it ready by then, so please be patient ;-) By then there will also be a little more documentation.
But as a short outlook:
Installing the "robotcode" package alone is useless, it is only the common interface to the individual tools. Since you don't need all the tools in a CI environment, e.g. the language server, you have to specify the actual tool or install the package directly.
So for the RobotCode Runner e.g.
pip install robotcode-runner

or
pip install robotcode[runner]

there are the following tools that currently work:

robotcode-runner
robotcode-debugger
robotcode-language-server

the following will follow:

robotcode-analyse
robotcode-templates

As I said, you have to be patient until RoboCon Online...
On https://robotcode.io you can already find a version of the documentation under the tab CLI

Thanks for the issue, I have corrected it.

is this fix in 0.76.0? because i still get the same error in that version:
> robotcode --help
Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "C:\project\.venv\Scripts\robotcode.exe\__main__.py", line 4, in <module>
  File "C:\project\.venv\Lib\site-packages\robotcode\cli\__init__.py", line 9, in <module>
    from robotcode.plugin import (
ModuleNotFoundError: No module named 'robotcode.plugin'

What did you install?

pip install robotcode

this is now really fixed with 0.76.2.
It is not enough to only install the robotcode package you need to specify also a tool like runner, debugger or so.
like pip install robotcode[runner] or pip intstall robotcode-runner
or  pip install robotcode[debugger] or pip intstall robotcode-debugger
or if you want to install all use pip install robotcode[all]

works now, thanks!


Simplify default export commands

Include .pdf in the filename to bypass interactive menus
Specify export filename in both example commands
Avoid using "publish" as filename since resume publish is a different, valid command

This is targeted at the simplest use case, hope it is helpful.

Thanks, @mchelen! Just now getting back to this. Yeah, my plan was to fix the scroll issue by just either slightly widening the container or setting the element to "horizontal-scrolling=no."
"


Complex to float conversion warnings; errors when disabling jax's jit regarding int to float conversion

Hey, it's me again.
While working with this library and debugging some of my code I found the following small issues which can lead to errors in future versions of jax
Note that some of the issues only appear when running jax with config.update("jax_disable_jit", True)
https://github.com/astro-informatics/s2fft/blob/26ff37a8e05bab8adc31dd9efc3c73081ebed8ff/s2fft/utils/rotation.py#L37-L40
I think dl should be of type jnp.float64 because the $d$ matrix is real and is also implemented as float64 in risbo.py, otherwise I get warnings that the complex128 to float64 conversion is deprecated and will become an error in the next update.
A few times there is a (-1) taken to a negative power which issues an error when jax's jit is disabled
https://github.com/astro-informatics/s2fft/blob/26ff37a8e05bab8adc31dd9efc3c73081ebed8ff/s2fft/transforms/spherical.py#L299
https://github.com/astro-informatics/s2fft/blob/26ff37a8e05bab8adc31dd9efc3c73081ebed8ff/s2fft/transforms/spherical.py#L660
https://github.com/astro-informatics/s2fft/blob/26ff37a8e05bab8adc31dd9efc3c73081ebed8ff/s2fft/transforms/wigner.py#L262
Suggestion: substitute (-1) with (-1.0)
I get another warning about an invalid escape sequence \e
https://github.com/astro-informatics/s2fft/blob/26ff37a8e05bab8adc31dd9efc3c73081ebed8ff/s2fft/recursions/risbo.py#L106-L114
Suggestion: make it a raw string literal with r"""...
Btw thanks a lot for the rotate_flm implementation, it is exactly what I needed.

Hi again 👋 thanks for the debugging, definitely correct about the Wigner matrices. I'll work through these points today and update the codebase ready for v1.0.2 release.

Closing as this should have been addressed in PR #189


remove link to gitlist.org

In the file README.md in root-directory the site gitlist.org ist still mentioned. But the domain is parked or somehow else inactive.
So remove the link and if a replacement is available that would be great.

Hell David. gitlist.org works correctly for me. Try checking your DNS settings.

@David in the Netherlands it works fine for me aswell.

Fine, don't know what went wrong but now I can see it too and it looks like being correct.
Before I saw a page that the domain was expired.
Ok, issue can be closed ;-)


Remove group role from focus wrapper

This PR removes the role="group" from the router's focus wrapper element. The group role results in some screen readers reading through nested content without context provided by the markup which is pretty confusing as a general setting.
App developers can still pass a role prop into router if this setting works better for specific use cases, but it seems better to remove it as the default behavior.

Looks good, I'm happy to just put this straight into a minor release w/o a beta.


Monitor dependencies with Depedency CI

Not too interesting with 0 dependencies, but it also keeps track of lincensing
issues which should be useful in the future.
https://dependencyci.com/github/woofwoofinc/csearch

Very stylish!