Splined radial integral

@agoscinski do you want (and have the time) to give this code a look? This is using Hermit splines instead of the natural cubic splines of librascal.
Also, do you remember how many points where required to reach a given accuracy in librascal? Here I need around 80 for 1e-5 and 320 for 1e-8 with realistic GTO parameters.

Benchmark result versus non splined GTO (use cargo bench --bench soap-radial-integral -- --save-baseline basis and then critcmp basis -g ".*(GTO)(.*)" to reproduce)
group                                                       basis/                   basis/Splined
-----                                                       ------                   --------------
GTO (per neighbor)/n_max = 14, l_max = 1                    12.26     2.6±0.07µs     1.00    210.8±4.48ns
GTO (per neighbor)/n_max = 14, l_max = 15                   20.69    11.0±0.57µs     1.00   529.7±11.06ns
GTO (per neighbor)/n_max = 14, l_max = 7                    18.40     6.8±0.20µs     1.00   367.0±22.76ns
GTO (per neighbor)/n_max = 2, l_max = 1                     4.52   697.4±32.33ns     1.00    154.4±3.80ns
GTO (per neighbor)/n_max = 2, l_max = 15                    9.91      2.1±0.11µs     1.00    215.6±3.78ns
GTO (per neighbor)/n_max = 2, l_max = 7                     7.79  1348.7±31.73ns     1.00    173.2±3.86ns
GTO (per neighbor)/n_max = 8, l_max = 1                     9.59  1722.2±37.31ns     1.00    179.5±4.87ns
GTO (per neighbor)/n_max = 8, l_max = 15                    16.42     6.4±0.22µs     1.00   387.3±18.10ns
GTO (per neighbor)/n_max = 8, l_max = 7                     13.55     3.8±0.06µs     1.00    278.0±5.65ns
GTO with gradients (per neighbor)/n_max = 14, l_max = 1     12.14     4.6±0.07µs     1.00    379.9±9.79ns
GTO with gradients (per neighbor)/n_max = 14, l_max = 15    12.50    12.6±0.08µs     1.00  1011.3±41.43ns
GTO with gradients (per neighbor)/n_max = 14, l_max = 7     12.15     7.9±0.08µs     1.00   653.3±13.72ns
GTO with gradients (per neighbor)/n_max = 2, l_max = 1      4.47  1364.9±51.65ns     1.00    305.0±9.04ns
GTO with gradients (per neighbor)/n_max = 2, l_max = 15     8.37      3.2±0.08µs     1.00    385.0±9.69ns
GTO with gradients (per neighbor)/n_max = 2, l_max = 7      5.87  1963.2±36.92ns     1.00    334.2±5.70ns
GTO with gradients (per neighbor)/n_max = 8, l_max = 1      9.16      3.1±0.07µs     1.00    334.1±6.77ns
GTO with gradients (per neighbor)/n_max = 8, l_max = 15     10.61     7.3±0.30µs     1.00   687.9±10.92ns
GTO with gradients (per neighbor)/n_max = 8, l_max = 7      9.20      4.6±0.11µs     1.00   500.6±15.89ns

The speedup varies between 4x faster for small n_max/l_max to 20x faster for high n_max/l_max.


Suggestion: Change the server name variable to a constant on settings.h

With this constant, will be more easy to change the Crow server name.

Are you sure this will really make it easier to change the Crow server name? With the displacement of this variable to a preprocessor constant I think this has the potential to slow down compile time. And cause unnecessary recompilation of object files in the incremental linking process if the server name is changed.

Thank you for the PR, but I don't think this feature is needed. I think it's better to expose API at App class level if someone want to change server name.


Experimentation of model compression

Closes #746

The work is currently done in https://github.com/betagouv/publicodes/pull/291


Update CLI extensions available doc

Update CLI extensions available doc.
Triggered by Azure/azure-cli-extensions - TRAVIS_BUILD_ID=556961118
https://github.com/Azure/azure-cli-extensions/commit/b30734d056a2a1fe56793967f1186f0f763ee53b

Docs Build status updates of commit d0ffc5b:
:clock10: Full build: the average full build time is 7 min(s) 9 sec(s), this is based on the last 129 build(s) for this repository.

Docs Build status updates of commit d0ffc5b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit d0ffc5b:
:clock10: Full build: the average full build time is 7 min(s) 9 sec(s), this is based on the last 130 build(s) for this repository.

Docs Build status updates of commit d0ffc5b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/azure-cli-extensions-list.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add hide/show parcellation button on the chip

I've commented on tests for hiding delineations. I will move them to the state if we agree about this approach.

Hmm, I have a feeling that this is way more complicated than it needs to be.
I don't think it needs to be on the global store. The more I think about this, the more I think using a directive to track, and act on the toggle might be a simpler approach.

Hmm, I have a feeling that this is way more complicated than it needs to be.
I don't think it needs to be on the global store. The more I think about this, the more I think using a directive to track, and act on the toggle might be a simpler approach.

What do you think about the service if you do not want to make the state heavier? if we use directive it should be adapted to the element (to change icon on toggle, also tooltip will not be dynamic). Current state parcellation is will not be available in the application.

or maybe we can keep only parcellationVisible in the state?

Feature is moved to the directive

@fsdavid can you ping me when you have finished this PR and it is again ready for review?
Thanks for the work so far.

@fsdavid can you ping me when you have finished this PR and it is again ready for review?
Thanks for the work so far.

Sure :)

@xgui3783 I have made some changes to the tests.
I could not make a fix  for the '> calls schedulRedraw' and comment it

@xgui3783 It should be fixed now

Thanks.
I will take a look maybe today, maybe tomorrow.
I will also investigate why scheduleRedraw no longer gets called.

superceded by #1169 , specifically 83b7609bfa772968e846c0c4377365dabd6e1ca0


Crypto Mail Alert Program (Automated with API)

An Automated API-based a Crypto Mail Alert Programme
Script Description
User can set whether he wants to target the above or below the range of a particular Crypto Exchange and trigger the program to automate the process of sending a mail alert to the provided email_id whenever it crosses the target limit.
Instructions

Create a new folder for your script and the file/folder name should be appropriate.
Create a README.md in your folder for program Instructions
add requirements.txt if needed
add pytest/unittest if possible

Please add/delete options that are not relevant.

[x] Adding New Code
[x] Improving Documentation

Programming Language

[x] Python

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server  - https://discord.gg/FXyh2S3
Happy Coding,

Made a PR
Kindly Review.


feat(minifier): constant fold nullish coalescing operator

#5761  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Boshen and the rest of your teammates on  Graphite


Segment Trees in Python

It is mostly used in competitive programming

Can I work on this?

Thanks for Contributing


Not showing phone position on Android 11

Hi,
I installed a Traccar server at home and a gps tracker in my car.
I connect to it from my pc (using my personal website address) and my phone.
A useful feature is when I click on the Current Location button (the one with a compass icon) and a blue dot appears with my current position, using phone gps. In this way I can compare my current position and my car position and easily find where my car is parked.
When I use Traccar Manager on my Oneplus Nord phone with Android 11, I cannot see my current phone position.
The Current Location button does nothing.
In phone settings, Traccar Manager location permission is on, when app is in use (I cannot choose "always").
I tried to install Traccar Manager on an old tablet with Android 7 (using same server, same account, etc.), and it works flawlessly. I see my position when I enable Current Location.
In tablet settings, Traccar Manager location permission is always on.
I thinks there's an issue with Android 11 and maybe new permissions system.

Are you connected to HTTPS server URL?

Are you connected to HTTPS server URL?
Yes, I am.

The issue is not with permission. It should be "when app is in use", so that part is correct.

Don't know what else to check.
Does the app provide logs?
I tried to look into Logcat while tapping on Current Location, but found nothing relevant about 'traccar' or 'location'.

The feature is basically provided be the WebView, so there's not much to log. I guess the only option I see is to debug the code (both web and native) to see if all callbacks are getting called.

I just discovered that if I open Chrome on the phone and connect to my Traccar url, the Current Location button works perfectly: after tapping on it, and allowing the "this site wants to know your location" popup, the blue dot appears.
Therefore, issue is only related to the Traccar Manager app.
At least I can use Chrome as a workaround.


XCTest plugin should support a component under test that has dependencies

Currently, when the component under test has a dependency on a library, linking of the test bundle fails.

This is wrapped up in https://github.com/gradle/gradle/pull/3410


[Feature request] Add time with date

Some form fields need time with date.This datetime feature will complete this package. Waiting for it.

Hi @shahreazebd 👋
Thank you for using our package.
Sorry for the delay. Thank you for this feature request. Rest assured that we will implement it very soon. This feature is really needed and already comes up in many issues. It will probably be included in the new versions.
PRs are always welcome 🎉.

@onesine @shahreazebd
Hi guys, since PRs are welcome ;), follow a PR in below just doing exactly that:
https://github.com/onesine/react-tailwindcss-datepicker/pull/236


(vlc) Fix 32-bit install on 64-bit, 64-bit install on 32-bit platforms

Fix for #1858

Description

Motivation and Context

How Has this Been Tested?

Screenshot (if appropriate, usually isn't needed):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[ ] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[ ] I have updated the package description and it is less than 4000 characters.
[ ] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[ ] The changes only affect a single package (not including meta package).

Original Location

Original Repository
Open Issues Add the different issues underneath, and tick those that are fixed in this PR

[ ] Issue 1 link
[ ] Issue 2 Link

[ ] Include the link to the opened PR that removes the package from the original location
[ ] The migration guidelines have been followed

:white_check_mark: Package verification completed without issues. PR is now pending human review

Thank you for opening a PR to fix the issue, one thing though.
We are trying to move towards not using globbing patterns when possible (this is one package that haven't been updated yet), would it be possible for you to instead update the replace the actual files in the argument instead?
See the following locations for an example of what we do with the cmake.install package which would need to be updated to here:
https://github.com/chocolatey-community/chocolatey-packages/blob/4738d9be544c595129bf07b1ea9e2f968404988b/automatic/cmake.install/tools/chocolateyInstall.ps1#L8-L9
https://github.com/chocolatey-community/chocolatey-packages/blob/4738d9be544c595129bf07b1ea9e2f968404988b/automatic/cmake.install/update.ps1#L22-L25

:white_check_mark: Package verification completed without issues. PR is now pending human review


minor fixes

fix popup to show content or overview depending on what fields are present in the article.
fix language in form to add event -- cta still says "add a link"

dupe. closing.


[BUG] auto complete is a bit up and down with working right

recent mac osx updates for the past 2 commits have really knocked autocomplete on its side. it doesnt work great and is quite testy.
sometimes things work, other times, not
i did post about it before. but just making it known again ;)

Not true, autocomplete on osx is currently not working at all.  Actual autocomplete will probably be back tomorrow.
PS: Sorry for loosing your previous issues. Recreated the repository so it isn't marked as a pure fork of vscode which made it not count the commits for the authors until they are merged back. Forgot about rescuing the issues :(

also to add. auto complete does not really work on new class files if the project has been built before adding it

Should be better by now.


Update Storybook UI theme to improve readability

Closes #136
Changes

Use Avenir Next as default font for all Storybook text
Sentence case for story names
Darker background colors
Lighter text color
CFPB seal for branding

How to test this PR

Check out the screenshots below
yarn && yarn start -> Poke around, see how things feel: are they readable?

Screenshots

Before
After

I'm not understanding why the netlify deploy of Storybook is failing above??
yarn build-storybook which translates to the same storybook build -o dist-storybook that is run in netlify and succeeds locally...

@meissadia I scope creeped this PR a bit by adding a commit that moves the repo to Yarn v3. Yarn v1 has issues with ES modules apparently.


Support for OpenTelemetry instrumentation grpc

Hello, may I ask whether you could add support for grpc instrumentation? Thanks.
[1] https://github.com/open-telemetry/opentelemetry-python-contrib/tree/main/instrumentation/opentelemetry-instrumentation-grpc

This is not currently a estimated date at which point this will be added as part of the distro. However, you can still trigger the instrumentation yourself after calling configure_azure_monitor. For instance, SQLAlchemy is not a part of the distro, but you can still use it with the distro. See the associated sample here

Migrating to Azure sdk: https://github.com/Azure/azure-sdk-for-python/issues/32496


[ServiceBus] stress optional rotate logs

Fix TODO before adding rotating logs back in as default.

API change check
API changes are not detected in this pull request.


Set key for duration of context

I'd like to set a redis key for the duration of the context. The problem I'm facing is that once the context is cancelled, I can't delete the key using Del (since ctx is cancelled). I don't control the context lifecycle (it is being cancelled since gRPC stream server is closing). Is there a way to accomplish this?

ended up just creating a new root context like
ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second)
err := cache.Client.Del(ctx, minfo.BuildKey()).Err()
cancel()


Allow usage on git < 2.23.0

Switch to old commands for branch switching to allow for versions of git < 2.23.0

Fixes #48


date: support relative --date

A simple example of inputs/outputs of what GNU's Coreutils is capable of is visible at https://www.cyberciti.biz/tips/linux-unix-get-yesterdays-tomorrows-date.html
This issue was suggested by @tertsdiepraam in https://github.com/uutils/coreutils/issues/2685#issuecomment-1111308354 (which might be interesting to solve in the same PR).

I think this was solved by https://github.com/uutils/coreutils/pull/4790

Yes, #4790 implements most of the functionality, though there is still some functionality missing (for example, "next week" and "last week" are not recognized yet).


Adding support for metadata in State API Requests

Description
Adding support for metadata in Get/Save/Delete State Requests.
Issue reference
Closes #1909
Closes #1967
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[x] Unit tests passing
[x] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test

Where's the HTTP support?

Where's the HTTP support?

More specifically, in HTTP Get/BulkGet.

Where's the HTTP support?

HTTP Get metadata support was already there. However missing was Delete one that I added now.

Where's the HTTP support?

HTTP Get/BulkGet metadata support was already there. However missing was Delete one that I added now.

Ok, it looks like we need to update our docs about how to send metadata for GET requests, as this is different, not being on the body.

Where's the HTTP support?

HTTP Get/BulkGet metadata support was already there. However missing was Delete one that I added now.

Ok, it looks like we need to update our docs about how to send metadata for GET requests, as this is different, not being on the body.

Ok. I have created issue https://github.com/dapr/docs/issues/767 (in 0.11 milestone ) to track this and have self assigned.

@shalabhms We also update the sdk to support meta in state delete. I find that in go-sdk, no metadata in delete api.  And I need this now to delete some state which metadata is required. I have summit a issue and a PR:
https://github.com/dapr/go-sdk/issues/81


aria2.aria2 version v1.36.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The Windows Package Manager policies requires that the InstallerUrl comes directly from the ISV's release location.

https://learn.microsoft.com/en-us/windows/package-manager/package/repository#error-labels
Please add Urls from the official source.

original url only provide zip format. I believe providing it as an installer url wont work.

You can add a PR for .zip installers using PRs #88065, #88067 as reference. As to when those PRs will get accepted, you can track the PSA #87300.

You can add a PR for .zip installers using PRs #88065, #88067 as reference. As to when those PRs will get accepted, you can track the PSA #87300.

thank you :)


Create a new pull request by

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.806%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1982

Relevant Lines:
2043

💛  - Coveralls


lombok and @builder

This seems to be a different issue than reported before. While running javadoc my build actually fails if I try to return the lombok generated builder class as a return value from a method. Java doc cannot find the dynamically created builder class and fails with "cannot find symbol". I have not figured out a way around this besides turning off javadoc (for now).

+1
Only solution found is to turn javadoc to regular comments to avoid the whole project javadoc to crash

+1


Instrument chef-form-field to watch its input

:nut_and_bolt: Description: What code changed, and why?
Rather than having to manually instrument every input to watch for
its value being edited so that it matches its original value, this bakes that into the component. Here's the details embedded in the code itself:
 // A FormFieldComponent can optionally track whether an input field (FormControl)
  // being edited is the same as, or different from, the field's initial value.
  // That is, it knows whether the user has made a change or not,
  // and marks the pristine/dirty state accordingly, so that information may be used
  // in the parent form to e.g., disable the `Save` button, per our UX guidelines.
  // Example:
  // <form [formGroup]="updateForm">
  //   <chef-form-field
  //       [watch]="updateForm.controls.name.value"
  //       [control]="updateForm.controls.name">
  //     Name *
  //     <chef-input formControlName="name" type="text"></chef-input>
  //   </chef-form-field>
  // </form>
  // <chef-button primary [disabled]="!updateForm.valid || !updateForm.dirty">
  //
  // You need to supply these two parameters:
  @Input() watch: string; //  dynamic value to watch
  @Input() control: FormControl; // control to set back to pristine when needed

Note that this functionality is optional.
Any existing <chef-form-field> elements will function exactly as they did before.
:chains: Related Resources
NA
:+1: Definition of Done
The new functionality is wired into editing:

a project name (Settings >> Projects >> <project> >> Details)
a team name (Settings >> Teams >> <project> >> Details)

:athletic_shoe: How to Build and Test the Change
Rebuild automate-ui.
Edit a project or team name as indicated above.
Edit the value.
➡ Observe Save becomes enabled.
Restore the original value.
➡ Observe Save becomes disable.
:white_check_mark: Checklist

[x] Tests added/updated?
[x] Docs added/updated?

:camera: Screenshots, if applicable

Don't merge just yet... Scott thought he had some ideas to improve this.

There's some more test cleanup I'd like to do but this is a working example of what I had in mind: https://github.com/chef/automate/compare/sc/pristine-form-controls
Adds a directive that will track the original value of a FormControl and update the dirty/pristine state accordingly. Main advantage being this won't require changing how chef-form-field or other input related elements are used.

Closing in favor of https://github.com/chef/automate/pull/1171


(suggestion) check filepath.Walk misusage

https://golang.org/pkg/path/filepath/#Walk
func Walk(root string, walkFn WalkFunc) error
type WalkFunc func(path string, info os.FileInfo, err error) error

Sometimes people ignore err in their own WalkFunc.
real-world example: https://github.com/docker/docker/issues/23004
So I suggest checking whether err is ignored in WalkFunc.
I can work on this if I can get positive feedback.

Is this not a problem that errcheck would detect already?

In my understanding errcheck cannot detect unchecked errors in arguments (not in returned value).
package main

import (
        "os"
        "path/filepath"
)

func main() {
        err := filepath.Walk("/dummy", func(d string, fileInfo os.FileInfo, e error) error {
                return nil // ignore e
        })
        if err != nil {
                panic(err)
        }
}

$ go get -u github.com/kisielk/errcheck
$ errcheck a.go
(empty result)

Should I make a suggestion to errcheck community rather than staticcheck?

I think errcheck to be fixed to handle this.

Do propose the check to the errcheck dev first please. I'm happy to add this to staticcheck if he rejects the idea.

OK, thank you all!


Semantic Versioning in 0.5.0 Release

Starting with version 0.5.0 we are committing to semantic versioning for Terminus. ( Read more about semantic versioning at http://semver.org/ )
Versions
In keeping with the standards of semantic versioning, backward-incompatible fixes will be targeted to "Major" versions. "Minor" versions will be reserved for significant feature/bug releases needed between major versions. "Patch" releases will be reserved only for critical security issues and other bugs critical to stabilizing the release.
After a new major version is released, previous major versions will be actively supported for 1 year.
What qualifies as a backward incompatible change
Our initial commitment will be to command compatibility and parameter compatibility. However, since on the command line STDOUT and STDERR are essentially APIs we will make a best effort to keep machine-readable output compatibility, meaning if your code interfaces with Terminus via --json or --bash formats we will try our best to ensure these are stable and compatible between minor release. However, changes to the STDOUT, like success and fail messages, will not be consider incompatible.
Version branches
If you are using terminus in a production environment you should be deploying the executable for the latest release. ( github.com/pantheon-systems/cli/releases )
Ongoing development on the next planned release will be on the master branch and should not be considered stable as changes will be taking place on a daily basis.
We will maintain a separate branch for all minor point releases going forward, i.e. 1.0.x, 0.5.x. Any critical patches should be submitted against those branches. If the fix is applicable to master a separate pull request can be made.
What this means for users
0.5.0 will include only changes that are backward incompatible. After it's released we will create a 0.5.x branch that will be used for any critical bugs or patches that need to be addressed. All other bugs/features/issues will be addressed in the next major point  release 1.0.0. The master branch will effectively become the 1.0.x branch. There is currently no plan for a version 0.6.x.

LGMT

:+1:


feat: add random baseline

The previous chembench version had several JSON files with different scores; I guess they were to test the script. In this PR I only included one file with overall scores similar to the other models

The previous chembench version had several JSON files with different scores; I guess they were to test the script. In this PR I only included one file with overall scores similar to the other models

I think the different files have been for different mcq_strategy choices in the baseline (e.g. first vs. random). But we anyhow used only one, so your PR is helpful. Thanks!


feat: bond enough aleo but make a delegate address be a validator

Motivation
Bond enough aleo but make a delegate address be a validator. solve the problem of asset security for Validators.

TODO:

Some solution:// The committee_state struct tracks the total stake of the validator, and whether they are open to stakers. struct committee_state:     // The amount of microcredits bonded to the validator, by the validator and its delegators.     microcredits as u64;     // The boolean flag indicating if the validator is open to stakers.     is_open as boolean;     // The delegate address of the validator bft.     delegate_address as address.public;

Can you answer the following concerns:

What happens if a validator has less than 1M credits initially? Doesn't this imply they can squat all 200 validator slots with no stake
What happens if the delegator unbonds their full stake and the validator now has nothing at stake?

Thanks for your response.
The purpose of this Pull Request  is to split the validator address into owner and worker addresses.
Here are answers to some questions you raised:

During initialization, the validator must be provided with a single address that contributes more than 1 million points, this address will be recorded as the delegate address (i.e., the `owner address).

The validator is required to execute the unbond_delegator_as_validator operation on all bond addresses before performing the unbond_validator operation on the delegate_address (i.e., owner address).

There might be better ways to make modifications, but in order to keep code changes minimal, the original logic has been reused.

Closing because ARC-0037 resolves these concerns - https://github.com/AleoNet/snarkVM/pull/2385


Make ESLint tests run alongside unit tests and appear in the browser report

This is how Ember test setup does it, and it's super helpful. Lets you catch lint errors as early as possible.

Thanks for the suggestion! Is this slecifically for when the browser tests are being run? I haven't yet seen a set up that integrates linting results and test results into an HTML reporter. That sounds really cool, though.
Do you think you might have time to put together a PR, or otherwise link to other resources that might help get this added?
iirc Ember uses Brocolli and something other than Mocha so it may not be as easy to do here. But I'm down to add this if it's possible!

Is this slecifically for when the browser tests are being run?

Yes.

Do you think you might have time to put together a PR

Nope. I'm merely a user, not familiar with test pipeline internals.  Ember indeed does some non-trivial manipulation. I only know its test pipeline is based on Testem.

Cool. Thanks for the info! I'll definitely add this to my list of things to research. I think it would be a great addition to the boilerplate.

After some initial research this seems to be a feature that would require adding in Testem. It seems like it supports merging all of the results from different sources (linting, unit testers) together into 1 output.
I'll look more into how much complexity it adds to the build step. @paulfalgout, do you have any initial thoughts?

oh wow. like the idea..  news to me.


Provide an accuracy testing interface?

The project is excellent. Can you provide an accuracy testing interface? It can save much time in accuracy testing.

Thank you for interesting in our work! The instruction is given in README. If you are interested in reproducing the reported number if our paper,
For LongBench, you can obtain the result by
# In our paper, K_BITS==V_BITS==2, GROUP_LENGTH==32, RESIDUAL_LENGTH==128
bash scripts/long_test.sh {GPU_ID} {K_BITS} {V_BITS} {GROUP_LENGTH} {RESIDUAL_LENGTH} {MODEL_NAME}
python eval_long_bench.py --model {MODEL} # MODEL is the dir name under pred/ Currently it support Llama family model and Mistral model.

For tasks like GSM8K, CoQA, TruthfulQA, you can obtain the result by
git checkout -b lmeval
git pull

cd lm-evaluation-harness
pip install -e .
cd ..

# We report TASK in {coqa, truthfulqa_gen, gsm8k} in our paper.
# If use KIVI implementation, set K_BITS and V_BITS to 2 or 4.
# If use baseline, set K_BITS and V_BITS to 16.
bash scripts/lmeval_test.sh {GPU_ID} {K_BITS} {V_BITS} {GROUP_LENGTH} {RESIDUAL_LENGTH} {TASK} {MODEL_NAME}

Let me know if you cannot reproduce our results or have further questions.

Which metric is used in the paper? I have got bleu_acc, bleu_diff, etc. for truthfulqa_gen, but not sure which one is reported


Can not find a way to display data with sub-second resolution with Grafana Cloud

What happened:
I am trying to display Time Series visualization with sub-second resolution using Grafana Cloud. I have the latest plugin version (v2.8.0) activated in my Grafana Cloud instance.
I am collecting MQTT messages with AWS IoT Core and routing those using Rule Engine to a Kinesis Firehose stream.
Inside Rules Engine i have setup a rule like: SELECT *, timestamp() AS aws_timestamp, FROM 'topic_name'
This rule simply injects a millisecond resolution timestamp telling when exactly the message has been received by AWS servers.
I can confirm sub-second resolution of my database by making a query inside AWS Athena console as well as by making the query inside my Grafana Cloud account using Table format (query in Grafana: SELECT aint AS time, along AS value, 'test' AS metric FROM "some_db"."test" ORDER BY time ASC)
However when i switch to Time Series format, the same query returns: "Data is missing time field"
By modifying the query to SELECT FROM_UNIXTIME(aint/1000) AS time, along AS value, 'test' AS metric FROM "some_db"."test" ORDER BY time ASC the Time Series get rendered, however only with second resolution.
What you expected to happen:
I expected that i can render a Time Series with sub-second resolution as my data has sub-second precision.
Tried a bunch of things for 2 days now, without reaching my original goal.
Screenshots
Table format confirms sub-second precision of the timestamp contained in the dataset

Time Series format does not render using the same query

Time Series format can render only at second resolution using a modified query

Anything else we need to know?:
Environment:

Grafana version: Grafana Cloud
Plugin version: 2.8.0
OS Grafana is installed on:
User OS & Browser: Windows 10x64, Chrome 111.0.5563.147
Others:

Hi @idea--list thanks for reporting this, I've managed to reproduce it. We're taking a look, will let you know when we have updates!

Hi again @idea--list! The problem with Time Series in this case is that your column isn't a "time" column type, so our backend doesn't render it as such
Have you tried to run the query without "/1000" operation from the last screenshot? From_unixtime() returns, from what I can see, a timestamp(3), which should store milliseconds. If that doesn't work, is it because you're getting an error or another reason?

Closing this issue as it hasn't had activity in more than 30 days


CI: Enable pullapprove

Require two approvals from documentation team members before a PR
can land.
The configuration file is the same as those used for the other repos,
except for the approval team name.
Fixes #2.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

Ping @grahamwhaley, @kata-containers/documentation.

Pullapprove is unhappy with the .pullapprove.yml, which is odd considering it's valid yaml and it's identical  to all the other kata pullapprove configs with the exception of the group name.
I've tried removing and re-adding the pullapprove github link and force pushing but still no joy. The error reported by pullapprove is:
"[]" is an invalid value for groups

It might have been because the documentation team had no repositories associated with it - I just associated it to the documentation repo - so maybe try a refresh and see if that fixed it?

@grahamwhaley - good catch, that must have been it!


Revoke badge improvements

Basically, if the project description is updated that is considered as an update, not just adding a new projectUpdate entity.
So this simplified the develop and I just consider the project's updatedAt Column.
UpdatedAt column is also automatically updated when a new projectUpdate is added, updated or removed.

@mohammadranjbarz the ropsten testnet seems to be disabled or down, I know mateo did a migration on the subgraph. So maybe we hit the same deprecation issue.
I removed some tests that I thought were not required as we had goerli test below. And others that I think it would be good to rewrite them as they are an interesting case.

@mohammadranjbarz the ropsten testnet seems to be disabled or down, I know mateo did a migration on the subgraph. So maybe we hit the same deprecation issue. I removed some tests that I thought were not required as we had goerli test below. And others that I think it would be good to rewrite them as they are an interesting case.

Thanks man, you did a good thing


Add Cargo.lock

To make builds more reproducible, a Cargo.lock should be added to the project.

To which project? as there are 3 of them in this repo :)

I think you can just add it as a toplevel lockfile, if I am not mistaken.


Allow specifying column list for foreign key ON DELETE SET actions

Problem
There are situations where a table references another table by more than one column (compound foreign key).
For example:
model Department {
  id          String           @default(cuid())
  namespaceId String
  strategy    Strategy?  @relation(fields: [namespaceId, strategyId], references: [namespaceId, id], onDelete: SetNull)
  strategyId  String?

  @@id([namespaceId, id])
}

model Strategy {
  id          String           @default(cuid())
  namespaceId String

  @@id([namespaceId, id])
}

As you can see I want to set Department.strategyId to null when referenced record is deleted from Strategy table. However the database also tries to set the department.namespaceId to null but that column is a shared and NOT NULLABLE column. So the operation will fail.
Suggested solution
As mentioned here, recent PostgreSQL versions support specifying which columns must be set to null when referenced record is deleted.
So it would be great if Prisma also supported it with something like this
onDelete: SetNull(strategyId)

However it's not always necessary to explicitly declare this because for example in my situation because Department.namespaceId is not nullable but Department.strategyId is, then Prisma can detect the problem automatically which can warn user when migrating or fix it implicitly by changing SQL file to something like this:
     CREATE TABLE "Department" (
        ...
        FOREIGN KEY (namespaceId, strategyId) REFERENCES Strategy ON DELETE SET NULL (strategyId)
    );

Please comment if you know that other databases also support a similar feature.
Alternatives
Without suggested solution, I have to add extra column to my Department table:
(Which still does not fully satisfy my requirements because now I also need to have some kind of constraints to be sure both namespaceId and strategyNamespaceId are equal when defined.)
model Department {
  id          String           @default(cuid())
  namespaceId String
  strategyNamespaceId String?
  strategy    Strategy?  @relation(fields: [strategyNamespaceId, strategyId], references: [namespaceId, id], onDelete: SetNull)
  strategyId  String?

  @@id([namespaceId, id])
}

Oh wow, interesting feature. Do you have a documentation link for PostgreSQL for this?
That this is quite a new feature will make supporting this reliably quite a challenge as Prisma itself is mostly version unaware (for now).

Yes it is very new and just released in PostgreSQL 15 beta. This feature has been mentioned in this page:

SET NULL [ ( column_name [, ... ] ) ]
Set all of the referencing columns, or a specified subset of the referencing columns, to null. A subset of columns can only be specified for ON DELETE actions.

It also works for SET DEFAULT:

SET DEFAULT [ ( column_name [, ... ] ) ]
Set all of the referencing columns, or a specified subset of the referencing columns, to their default values. A subset of columns can only be specified for ON DELETE actions. (There must be a row in the referenced table matching the default values, if they are not null, or the operation will fail.)

I know this feature probably takes a long time to be implemented in Prisma however It's very useful.
The weird thing is PostgreSQL during migration, does not complain about my tables schema that I've mentioned in my first post.

Previously we could use this PostgreSQL feature by editing migration files manually, but now Prisma 4 complains about the schema itself:

Error parsing attribute "@relation": The onDelete referential action of a relation must not be set to SetNull when a referenced field is required.
Either choose another referential action, or make the referenced fields optional.

any updates?


More Information

Can you update the README.MD with some further info such as ports that are used, maybe a compose example or just your docker run example.

Can you update the README.MD with some further info such as ports that are used, maybe a compose example or just your docker run example.

I second this. This would be super helpful.

Sorry for very late response, i did not received any notification from github.
i'm running this docker in unraid environment, so it just runs with simple run command. and my MC runs behind Apache reverse proxy too.
all config options are in data folder config.json. there are so many options, that i didn't want to start describe it one by one.
here is my docker run command:
docker run -d --name='meshcentral2' --net='bridge' -e TZ="Europe/Kiev" -e HOST_OS="Unraid" -p '82:80/tcp' -p '455:8443/tcp' -p '466:443/tcp' -v '/mnt/user/appdata/meshcentral2new/data2/':'/home/node/meshcentral/meshcentral-data':'rw' -v '/mnt/user/backups/meshcentral-files2/':'/home/node/meshcentral/meshcentral-files/':'rw' -v '/mnt/user/appdata/meshcentral2new/web/':'/home/node/meshcentral/meshcentral-web':'rw' -v '/mnt/user/appdata/meshcentral2new/backup':'/home/node/meshcentral/meshcentral-backup':'rw' --hostname=meshcentral.example.com --log-opt max-size=50m --log-opt max-file=1 'uldiseihenbergs/meshcentral:0.5.85'


fix rbenv plugin to work with homebrew

Fixes #4998 and #3093.

Hi, any update on getting this PR merged?


Enhance shared convergence steps

This PR makes it possible for all convergence tests to leverage shared convergence steps and implements those shared convergence steps in all convergence tests.
Checklist

[ ] User's Guide has been updated
[ ] Developer's Guide has been updated
[ ] API documentation in the Developer's Guide (api.md) has any new or modified class, method and/or functions listed
[ ] Documentation has been built locally and changes look as expected
[X] Testing comment in the PR documents testing used to verify the changes
~~ New tests have been added to a test suite ~~

Testing
I have run the following convergence tests on chrys with intel, openmpi:

ocean/spherical/icos/cosine_bell
ocean/planar/inertial_gravity_wave
ocean/planar/manufactured_solution

@xylar Can you take a look at this when you have a chance and let me know if you are on board with these changes before I write up the documentation?

@xylar Thanks for the detailed review! I'm on board with all the changes. I'll make them and retest next week.

@xylar This is ready for your (re)review. I also added a convergence suite which is convenient for testing changes to the convergence steps.

@cbegeman, if you want to either clarify or remove the print statement I flagged above, I think that's the last thing needed before this can be merged.

@xylar Thank you for your review! I removed the print statement and will merge once the checks are finished.


Hybrid mode for astro SSR does not prerender pages on Mac

What version of starlight are you using?
0.23.4
What version of astro are you using?
4.10.1
What package manager are you using?
npm
What operating system are you using?
Mac
What browser are you using?
Edge
Describe the Bug
Astro seems not able to pre-render pages in a hybrid environment.
According to the manual, astro pages are always pre-rendered which should make it compatible with an hybrid installation.
However, using such config leads to no starlight page pre-rendered. All other astro pages, outside starlight, are prerendered. I found this noticing that during the build, pagefind was not finding any html page.
output: "hybrid",
  adapter: nodejs({
    mode: "standalone",
}),

Without the hybrid mode:
 generating static routes
12:52:26 ▶ @astrojs/starlight/404.astro
12:52:26   └─ /404.html (+8ms)
12:52:26 ▶ src/pages/schema/[...slug].astro
12:52:26   ├─ /schema/1/index.html (+6ms)
12:52:26   ├─ /schema/2/index.html (+4ms)
12:52:26   └─ /schema/3/index.html (+4ms)
12:52:26 ▶ @astrojs/starlight/index.astro
12:52:26   ├─ /guides/example/index.html (+4ms)
12:52:26   ├─ /index.html (+14ms)
12:52:26   └─ /reference/example/index.html (+4ms)
12:52:26 ✓ Completed in 62ms.

and with the hydrid mode enabled:
 prerendering static routes
12:52:54 ▶ src/pages/schema/[...slug].astro
12:52:54   ├─ /schema/1/index.html (+13ms)
12:52:54   ├─ /schema/2/index.html (+3ms)
12:52:54   └─ /schema/3/index.html (+3ms)
12:52:54 ✓ Completed in 35ms.

I am working to a reproducible example. The one on Stackblitz seems to work, but the exact same repo on my local macbook M3 does not.
Link to Minimal Reproducible Example
https://stackblitz.com/~/github.com/LunaticMuch/fuzzy-giggle
Participation

[X] I am willing to submit a pull request for this issue.

Thanks for your report.
Is this the exact same repro you're testing or something similar? I'm asking because I just tried downloading the repro and building it on macOS and did not get the issue you're seeing.
The only change I made was removing the ../../assets/houston.webp mention in src/content/docs/index.mdx as I assume it's just an asset missing in StackBlitz.
I used the "Download ZIP" button in the file explorer to download the repro and ran npm install and npm run build as the configuration already contained output: "hybrid", and the Node.js adapter.
If you re-download the repro and try again, does it still fail for you?

It's the same one running from my local.
I built it because I have another repo which actually behave bizarrely in the same way. Only difference, I locally used a company proxy for downloading dependencies instead of the public npm registry.

If you re-download the repro and try again, does it still fail for you?

Yes, I actually tried also from my own macbook...same issue. Do you know how could I potentially debug/trace it?

I think it's an astro related problem. I will open a bug there. Right now you can look the following logs for MacOS and Codesandbx
Same repo, different behaviour. The only real difference is the underlying OS.

Problem identified. Not a Starlight problem.


support Typescript version 4

With TS4 isImportDeclaration check for node.kind has changed it's value of SyntaxKind.ImportDeclaration enum from 254 to 265.

https://github.com/microsoft/TypeScript/blob/5f017df609b61f017baa61ff7679aabb39d089fb/src/compiler/types.ts#L312
https://github.com/microsoft/TypeScript/blob/918f0ef4040b372df357c5439601576baaeeec4c/src/compiler/factory/nodeTests.ts#L592-L594

@banyudu This plugin is great.
I'm a bit in a hury to make it work with TS4.
So I proposed a PR, which installs typescript as a peer dependency, so that the SyntaxKind enums are always that of the locally installed TS version.


Collecting requests from multiple GET requests from various coroutines

Hi,
I am just trying to make multiple GET requests to some urls and collect responses to parse the results. The example shown in High level requests matches what I need but it prints the result. Is there any way to collect the results of all the coroutines into a table. I tried doing this-
all_data={}
for _, host in ipairs(list) do
results=copas.addthread(handler, host)
table.insert(all_data,results)
end
copas.loop()
But all_data ends up being a collection of threadlists. I am not sure how to get the results from this thread. I see in general coroutines, result is retrieved by coroutine.resume but I don't see any copas.resume() method?
Also, for my purpose, i don't have any server implementation and just require an asynchronous client requesting different URLs. I feel for my application, I can just implement a simple coroutine example given in LUA coroutine tutorial instead of copas library. I am just curious if using copas for my application would add enough robustness/error handling so that I can overlook the overhead added by copas library or is copas library primarily used for server handling applications. It would be great if overhead can be estimated.
Thanks,
Aditya

@Aditya23456 addthread will return the newly created coroutine. You should store the result at the end of your handler function.
local all_data = {}  -- create an upvalue

local handler = function(...)
  -- do the usual stuff here

  -- when done, and the result is complete, store it
  table.insert(all_data, my_new_result)
end

for _, host in ipairs(list) do
  results=copas.addthread(handler, host)
  -- by now, we created the handlers, but they aren't running yet, let alone have a result
end
copas.loop()  -- will exit when all handlers have finished

-- since all handlers have completed now, the results should be in the list

Hii..
Thank you very much for your reply. I have another issue. I am using a lua lanes multithreaded library to launch 3 threads. One of the thread uses copas to make async calls to urls. At my handler function, I parse and update an SQLLite database with the result of URL response. I don't know why but my other threads can access SQLite database without any issue but the DB transactions done in the handler gets stuck with "LuaSQL: database is locked" error. I made sure my database connection and closing are done after the asynchttp statement so that entire transaction is done by current handler and then control is yielded to the other async request handler. Am I still doing something wrong? I am planning to send all my results by storing in upvalue to the main thread and then do the SQLite transaction for each response(I am not sure if it works but this increases time complexity). Do you have any idea how I can get through this ?
Thank you once again,
Regards,
Aditya

I think this question should better be posted on the Lua mailing list, preferably with some code that shows the issue you're having.
see https://www.lua.org/lua-l.html

iirc it is not implemented, I have no time to check the code right now. See also #33


fix DecoderBlock bug

Fix shape mismatch bug reported in issue42 caused by DecoderBlock when stride is odd. For example: when kernel_size=10, stride=5

outpadding should be set as output_padding = stride % 2, so it will only work when stride is odd.


Add support for TCP fallback when we receive a truncated DnsResponse

Motivation:
Sometimes DNS responses can be very large which mean they will not fit in a UDP packet. When this is happening the DNS server will set the TC flag (truncated flag) to tell the resolver that the response was truncated. When a truncated response was received we should allow to retry via TCP and use the received response (if possible) as a replacement for the truncated one.
See https://tools.ietf.org/html/rfc7766.
Modifications:

Add support for TCP fallback by allow to specify a socketChannelFactory / socketChannelType on the DnsNameResolverBuilder. If this is set to something different then null we will try to fallback to TCP.
Add decoder / encoder for TCP
Add unit tests

Result:
Support for TCP fallback as defined by https://tools.ietf.org/html/rfc7766 when using DnsNameResolver.

This fixes https://github.com/netty/netty/issues/9127

@trustin @ejona86 any comments ?


Incorrect Documentation related to multiple subnets example in Network Settings

*Supposed CSV file contents are shown as below with the Subnet field as Identity.
Identity, Mask, SiteID
<IP_ADDRESS>, 24, Redmond
<IP_ADDRESS>, 24, Chicago
<IP_ADDRESS>, 25, Vancouver
<IP_ADDRESS>, 28, Paris
But the Powershell code for the multiple subnets is referencing to "SubnetID" field. This either needs to be changed in the CSV file example with "Identity" replaced with "SubnetID" or in the Powershell code with "$.SubnetID" replaced with "$.Identity"
Also, Powershell code for multiple subnets should include a space between $_.SubnetID and -MaskBits.
Current Powershell:
Import-CSV C:\subnet.csv | foreach {New-CsTenantNetworkSubnet –SubnetID $_.SubnetID-MaskBits $_.Mask -NetworkSiteID $_.SiteID}
Change proposal:
CSV File field updated to SubnetID:
SubnetID, Mask, SiteID
<IP_ADDRESS>, 24, Redmond
<IP_ADDRESS>, 24, Chicago
<IP_ADDRESS>, 25, Vancouver
<IP_ADDRESS>, 28, Paris
PowerShell with added space:
Import-CSV C:\subnet.csv | foreach {New-CsTenantNetworkSubnet –SubnetID $_.SubnetID -MaskBits $_.Mask -NetworkSiteID $_.SiteID}

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

ID: eef19426-f330-e8ea-dd48-571954b23176
Version Independent ID: d4357caa-b5c6-a034-3b81-f409678f5ee3
Content: Manage your network topology for cloud voice features in Microsoft Teams - Microsoft Teams
Content Source: Teams/manage-your-network-topology.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@nithinmulley Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@LanaChin please can you have a look at this one?

Hi @nithinmulley - Thanks so much for your feedback and helping to improve our documentation. We've updated the PowerShell command in the article.


Add refresh token API end point to document and update .gitignore

Is your feature request related to a problem? Please describe.
I'm suggesting two improvements:
Add documentation for the refresh token endpoint.
Update the .gitignore file to prevent unnecessary files from the .idea folder from being uploaded to version control.
Describe the solution you'd like
Document the refresh token endpoint to provide better guidance.
Modify the .gitignore file to exclude all files within the .idea folder, as these files are specific to the IntelliJ IDEA IDE and should not be included in version control.

Thanks for reporting this, @erzohrakhan.  I have it assigned to you!


chore: update to Cosmos SDK 0.46.7 and related dependencies

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added ! to the type prefix if API or client breaking change
[ ] added appropriate labels to the PR
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Isn't there a required migration listed in the 0.46.7 release notes?

This will come in a separate PR, the migration is not needed for local testing.


have the two bin/ cli tools accept -h flags

have the two bin/ cli tools accept -h flags

(parts of this PR were extracted from https://github.com/dart-lang/intl_translation/pull/189)

dart bin/extract_to_arb.dart -h
Accepts Dart source files and produce intl_messages.arb as output.

Usage: extract_to_arb [options] <files.dart>

-h, --help                           Print this usage information.
    --[no-]suppress-last-modified    Suppress @@last_modified entry.
    --[no-]suppress-warnings         Suppress printing of warnings.
    --[no-]suppress-meta-data        Suppress writing meta information
    --[no-]warnings-are-errors       Treat all warnings as errors, stop processing
    --[no-]embedded-plurals          Allow plurals and genders to be embedded as part of a larger string, otherwise they must be at the top level.
                                     (defaults to on)
    --[no-]transformer               Assume that the transformer is in use, so name and args don't need to be specified for messages.
    --locale                         Specify the locale set inside the arb file.
    --[no-]with-source-text          Include source_text in meta information.
    --output-dir                     Specify the output directory.
    --output-file                    Specify the output file.
    --sources-list-file              A file that lists the Dart files to read, one per line.The paths in the file can be absolute or relative to the location of this file.
    --[no-]require_descriptions      Fail for messages that don't have a description.

and:
dart bin/generate_from_arb.dart -h
Usage: generate_from_arb [options] file1.dart file2.dart ... translation1_<languageTag>.arb translation2.arb ...

-h, --help                         Print this usage information.
    --[no-]json                    Generate translations as a JSON string rather than as functions.
    --[no-]code-map                Generate translations as a JSON string rather than as functions.
    --[no-]flutter                 Generate localization file that uses Flutter locale split.
    --[no-]suppress-warnings       Suppress printing of warnings.
    --output-dir                   Specify the output directory.
    --generated-file-prefix        Specify a prefix to be used for the generated file names.
                                   (defaults to "")
    --[no-]use-deferred-loading    Generate message code that must be loaded with deferred loading. Otherwise, all messages are eagerly loaded.
                                   (defaults to on)
    --[no-]null-safety             Generate null safe code vs legacy (pre-null safe) code.
                                   (defaults to on)
    --codegen_mode                 What mode to run the code generator in. Either release or debug.
                                   [release, debug (default)]
    --sources-list-file            A file that lists the Dart files to read, one per line. The paths in the file can be absolute or relative to the location of this file.
    --translations-list-file       A file that lists the translation files to process, one per line. The paths in the file can be absolute or relative to the location of this file.
    --[no-]transformer             Assume that the transformer is in use, so name and args don't need to be specified for messages.

(neither this nor #194 need to land before publishing)


[WIP] Change DB Tab Order

Make Databases be the first tab and link destination from DB list table

I see this message when landing on the Metrics tab currently:

2nd screenshot is invalid config on local (chatted on slack rq)
i don't think we should do this change until we have charts merged though and other backend changes merged up

I got this fixed by changing my .env file and now i see this:


[Machine Logs]

Problem Description
From Machines Tab, click a machine to view the logs.
There is no indication on the resulting screen which machine you are looking at.
Solution
Append the machine name to the Title "Machine Logs"
e.g. "Machine Logs - talos.controlplane1"
Alternative Solutions
No response
Notes
No response

This was fixed


JsonWidgetData.fromDynamicList not passing registry argument to maybeFromDynamicList

In the major rewrite of the JsonWidgetData class for version 7, new static methods were added to generate JsonWidgetData objects from a dynamic List namely fromDynamicList and maybeFromDynamicList. Both methods take a (dynamic) list and optional (JsonWidgetRegistry) registry as arguments.
fromDynamicList calls maybeFromDynamicList but does not pass the registry argument, so if you call fromDynamicList with an alternative registry instance maybeFromDynamicList will just use the default registry instance.
  static List<JsonWidgetData> fromDynamicList(
    dynamic list, {
    JsonWidgetRegistry? registry,
  }) {
    final result = maybeFromDynamicList(list);
    if (result == null) {
      throw Exception(
        '[JsonWidgetData]: requested to parse from dynamic list, but the input is null.',
      );
    }

    return result;
  }

should be:
static List<JsonWidgetData> fromDynamicList(
    dynamic list, {
    JsonWidgetRegistry? registry,
  }) {
    final result = maybeFromDynamicList(list, registry: registry);
    if (result == null) {
      throw Exception(
        '[JsonWidgetData]: requested to parse from dynamic list, but the input is null.',
      );
    }

    return result;
  }

Thanks for the detailed report!  Fixed tonight with 7.0.6


Allow multiple tokens

I didn't notice that slack only generated tokens, so every bot used would be a different token. So I need to be able to accept from a list of tokens.
Additionally, make a setup instructions that says to use the install script on Debian, and then says what to do as far as tokens and updates are concerned.

Also point out that the server will run on port 9117.

Fixed on d82415593b09da9283cca554e17bd8a3753d30f7

That commit may have also completely broken everything. :/ I need to test this.


Feature/webhooks rebased

Please take a look.

Codecov Report

Merging #114 into master will decrease coverage by 0.89%.
The diff coverage is 7.75%.

@@            Coverage Diff            @@
##           master     #114     +/-   ##
=========================================
- Coverage   20.68%   19.79%   -0.9%
=========================================
  Files         127      131      +4
  Lines        8387     8730    +343
=========================================
- Hits         1735     1728      -7
- Misses       6481     6845    +364
+ Partials      171      157     -14

Impacted Files
Coverage Δ

messaging/types/webhook.go
0% <ø> (ø)
:arrow_up:

system/internal/service/user.go
0% <0%> (ø)
:arrow_up:

messaging/internal/service/message.go
12.04% <0%> (-0.07%)
:arrow_down:

messaging/internal/repository/webhook.go
0% <0%> (ø)

messaging/internal/service/error.go
0% <0%> (ø)
:arrow_up:

system/internal/service/error.go
0% <0%> (ø)
:arrow_up:

messaging/internal/service/webhook.go
10.23% <10.23%> (ø)

messaging/internal/service/service.go
68.18% <100%> (+3.18%)
:arrow_up:

messaging/internal/service/command.go
12.5% <12.5%> (ø)

system/internal/repository/util.go
75% <33.33%> (ø)
:arrow_up:

... and 8 more

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 2952ceb...3510381. Read the comment docs.


model: xgboost: Add console example

#930

Codecov Report

Merging #934 (c2cfa43) into master (2c127ec) will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #934      +/-   ##
==========================================
- Coverage   85.89%   85.81%   -0.08%
==========================================
  Files         141      141
  Lines        9086     9082       -4
  Branches     1516     1516
==========================================
- Hits         7804     7794      -10
- Misses        958      964       +6
  Partials      324      324

Impacted Files
Coverage Δ

dffml/util/data.py
74.03% <0.00%> (-2.77%)
:arrow_down:

tests/util/test_net.py
98.24% <0.00%> (-1.76%)
:arrow_down:

dffml/noasync.py
78.18% <0.00%> (-1.13%)
:arrow_down:

dffml/util/testing/consoletest/commands.py
77.12% <0.00%> (-0.06%)
:arrow_down:

dffml/source/df.py
76.54% <0.00%> (ø)

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 2c127ec...c2cfa43. Read the comment docs.


[client] Add single player game modes

[x] Vs bot (currently is launched immediately on start, see #11)
[ ] 20/40/120 lines - best time
[ ] 2 minutes - highest score
[ ] Free play - no time or score constraints

After each success game, the replay is sent to web-api.

Related to #21


Reviewing for Gaia usage

Reviewing for use of gaia is challenging. Our current standard is that apps that write any data to gaia receive full marks. Even with this very low and simple to meet standard, it's not always clear if an app doesn't use Gaia or we are unable to find that app’s use of Gaia.
Proposal: As part of the monthly app mining submission process, developers should explain how Gaia is used by their app . Reviewers will follow those instructions to verify Gaia usage.

The implementation for this is probably similar to #138.

@larrysalibra Do you want to prevent server processing of data completely?
We had years of pushing cloud computing and many many libraries and tools to make life easier while putting processing on the server-side.
Take image processing as an example; we can not easily put OpenCV in every user mobile phone. Even if we can, battery life and network usage will be problematic.
I think this issue contains two parts, privacy and data ownership.
Data ownership will be easily done with bounding servers to do the write with a separate encryption key for every call and clients verify that.
But client-side encryption will exclude almost all cloud computing and leave us with cloud storage.
I love to see if we can find a way to do cloud computing, just like Blockstack did with solving identity and storage.

@larrysalibra how would you expect devs to declare this info? How would you test and score it?

@larrysalibra suggestion from @zone117x is that we can ask apps to attest to the following: all user data encryption and storage with Gaia is handled client-side.

Sounds like this isn't a change to rules or testing, just adding some info on app registration that allows NIL to test easier and faster. Moving to review and also adding to App.co backlog. @markmhx can you please ref this issue in an App.co ticket.


Calculation fails when the end time of a charge is very close to the beginning of the hour

Hi, awesome project, but ran into an issue using the Energinet provider. I had a charge that finished at the following timestamp:
2023-04-02 13:00:19.299.
When calculating it failed at the following line:
https://github.com/MattJeanes/TeslaMateAgile/blob/179ddd7c201abd178c0a1c79d42e24bb6149aa63/TeslaMateAgile/Helpers/PriceHelper.cs#LL127
With the error:
Charge calculation failed, pricing calculated for 364 / 365, likely missing price data
To fix it, I change the last timestamp of the charges table to 2023-04-02 13:01:19.299 via. psql, that is one minute later. I assume there's some issue related to charging finishing within the first minute of an hour. I didn't dig any deeper as the database modification solved my issue this time.
Let me know if any other information is needed.

It looks like the Energinet provider is requesting data down to the minute precision level and not to the second which would explain the behaviour you are seeing. There is a window of +/- 2 hours already to account for timezones as we cannot specify the date to come back in UTC but it seems during CEST we're right on the edge of that which is why we're only seeing this now as Denmark only switched to CEST a week ago or so.
I've pushed image version v1.11.3 - would you please give it a go by setting the cost field to NULL for that charging process (and reset the time back to the original time if needed) and see if it works?
Cheers!

Thank's for the quick fix, it seemed to work as intended!
dbug: TeslaMateAgile.PriceHelper[0]
      Calculated charge cost for 04/02/2023 11:00:00 UTC - 04/02/2023 12:00:00 UTC (unit cost: 0.94817500875, fee per kWh: 0): 10.572977343900205207444027852 for 11.15087114333333266884 energy
dbug: TeslaMateAgile.PriceHelper[0]
      Calculated charge cost for 04/02/2023 12:00:00 UTC - 04/02/2023 13:00:00 UTC (unit cost: 0.83773750125, fee per kWh: 0): 7.1852287972819619459826124506 for 8.57694538749999876048 energy
dbug: TeslaMateAgile.PriceHelper[0]
      Calculated charge cost for 04/02/2023 13:00:00 UTC - 04/02/2023 14:00:00 UTC (unit cost: 0.84313750, fee per kWh: 0): 0.00000000 for 0 energy

I assume the 0 energy on the last one is to be expected, the last charges event show 0 current.
Again, thank you for the awesome project!

That's correct, yeah
No worries, glad to hear that's sorted it out! 🙂


New users look shocking

Adding new users is gonna be interesting.

New user isn't visible unless they opt in to community. Should probably do something with onboarding.

I've inadvertently solved this elsewhere.


support for 2019 Chrysler Pacifica non-Hybrid

Add support for 2019 Chrysler Pacifica non-Hybrid thanks to @xeb
Add fingerprint for 2017 Pacifica non-Hybrid thanks to irpilot
I ran selfdrive/test/test_fingerprints.py and it ran fine:
Found 79 individual fingerprints
TEST SUCESSFUL

@adhintz , thanks! Any chance to share cabana drives for those cars?

CHRYSLER PACIFICA 2018
136 exact key/value matches
2 mismatches
Mismatches: {564: Unsupported: 4, this: 8; 924: Unsupported: 3, this: 8
I see mismatches in the fingerprint. I would suggest to split the 2017 and 2018


Twitter header for Zulip

I have uploaded the .svg and .png files for a twitter header design for the Zulip Account :)
Tested it in my twitter account.

Merged, thanks @kleidieski!  @rishig can you take care of uploading it?

Thanks @kleidieski!
Unfortunately now that I look more closely, this background doesn't quite match our logo:

This is a nice backdrop though, and I can imagine it being useful inspiration for other things.

Hi @rishig!
The background is indeed based on the same colour gradients used across the zulipchat.com website.
I thought that extending that style to social media channels would help unifying the brand.
Feel free to use, mix, and remix however you may see fit :).
Cheers.


Fix issues with cyclic type argumetns

@xRodney: Thank you so much for the analysis and the test.
@andreaTP: This should address the issues you had.

@iocanel thanks a ton!!! Can you please notify when a new release will be available to test?


Fixed Scale for Markers (feature)

Hey,
Would be great to see a fixed scale for markers, to cover a consistent area between zooms.
I did try to implement a solution in the marker_layer.dart, togglable by a new variable fixedScale, but unable to adjust size bigger than 100x100.
Ideas:
map.getZoomScale(map.zoom, map.zoom) always returns 1.0 given the duplicated arguments, so currently its not doing anything in current code. I think it was placed here for this particular feature in the future. I believe the starting zoom should populate the second argument to maintain the starting size of the widget(image).
Current Implementation:
My current implementation looks something like this (considering my starting zoom is 17.0).
Screen capture on youtube
Trouble I found is that the marker Width and Height have to be within 0-100 to work. My marker's widget is a AssetImage in a container.
Widget build(BuildContext context) {
[...]
          var width = markerOpt.width;
          var height = markerOpt.height;

          var pixelPosX =
              (pos.x - (markerOpt.width - markerOpt._anchor.left)).toDouble();
          var pixelPosY =
              (pos.y - (markerOpt.height - markerOpt._anchor.top)).toDouble();

          if (markerOpt.fixScale) {
            width = markerOpt.width * map.getZoomScale(map.zoom, 17.0);
            height = markerOpt.height * map.getZoomScale(map.zoom, 17.0);

            pixelPosX = (pos.x -
                    (width -
                        (markerOpt._anchor.left *
                            map.getZoomScale(map.zoom, 17.0))))
                .toDouble();
            pixelPosY = (pos.y -
                    (height -
                        (markerOpt._anchor.top *
                            map.getZoomScale(map.zoom, 17.0))))
                .toDouble();
          }

          if (!latlngBounds.contains(markerOpt.point)) {
            continue;
          }

          markers.add(
            new Positioned(
              width: width,
              height: height,
              left: pixelPosX,
              top: pixelPosY,
              child: markerOpt.builder(context),
            ),
          );
        }
[...]

Got any ideas on how to increase the size?

Do you mean fixed scale as in scale the markers up/down while the map scales? To me fixed scale means to keep them at a fixed scale, which is already the case.
Polygons scale with the map. You can look at them for ideas. They just use a painter, but the scaling logic is there.
Anyway the plugin system is there for this reason - make a plugin and @johnpryan could integrate it at some point into the core. This way the plugin system will get some love too. :)

as @avioli mentioned, It looks like you have a solution that can be implemented using the plugin api. I'm going to close but feel free to reopen if you feel that doesn't fit your needs.


all: remove duplicate and simplify code

The code is the same, just removed some un-needed MySQL logic and consolidated duplicate code.

@masseelch please check it again, I've addressed your comments

I'd like to see more comments in the code :-)

I'd like to see more comments in the code 😉

I've added comments even to the unexported functions now.


Negative Values for some KOs after normalization?

Hello,
Thanks for creating this tool! We have come across a situation where we get negative counts for some of the KOs after normalization and learn_model correction. Many of these genes appear to be house-keeping genes (e.g. ribosomal proteins) so our current hypothesis is that they are single copy genes that aren't classified by MUSiCC as single-copy. Before we move forward, I wanted to check to make sure that you haven't seen negative counts show up for some other reason and that we aren't making some error in usage.
The data are from soil and rhizosphere samples so we are using learn_model rather than the HMP model.
Here is the data that we used to run the program: https://drive.google.com/file/d/1rp5hnWqF-AsXa1WA-L09-D2lemkvpG-w/view?usp=sharing
This is the command we used to run it:
run_musicc.py gene_abundance.csv -if csv -of csv -o gene_abundance_normalized.csv -n -c learn_model -perf -v

Thanks for your help!

Hi,
Just wanted to let you know we're looking into this issue. We haven't come across this issue in the past, so I don't have any immediate advice for you.
I'll let you know if we do discover the underlying cause for this issue.

Sounds great, thanks for the help! I should mention, that it's only the learn_model step that generates the negative values. The initial normalization works fine.
Also, in case it's relevant, the first 12 samples are bulk soil samples and the second 12 are rhizosphere samples. We thought at first that maybe learn_model was failing because the samples from different environments needed to be treated differently. Since rhizospheres tend to have more fast-responding species than bulk soil (and so potentially have different base-line copy number variation). Unfortunately, when we separate the rhizosphere from bulk soil samples, we still see negative values.
The other thought we had was that perhaps there is something odd about the annotations or mapping for gene counts. (Though I wouldn't say I really understand MUSiCC's algorithm well enough to be confident in how annotation methods might or might not have downstream impacts). The annotations were made using the eggnog emapper.py script and the read counts were generated with salmon.
Let me know if there's any other information that might be helpful.

I have an update, and a possible solution.
Based on testing with your data (mainly focusing on your first sample at the moment), it looks like there are some genes expected to be universal single-copy genes that are extreme outliers (e.g. 0 abundance/absent in the sample, or very low abundance). This seems to be impacting model learning, causing the model to predict very slight negative correction factors, which will then result in negative abundances. However, by removing all 0 abundance KOs (which will include those outlier universal single-copy genes) from a sample, it looks like the learned model no longer makes negative value predictions.
I don't know the underlying reason for why some of your samples have such extreme low abundance for what we expect to be normally universal single-copy genes, but I think the simplest solution for the moment is to split your samples into separate files, remove all rows for 0 abundance KOs, run MUSiCC on each abundance file separately, then recombine.
Regarding your concern about samples from different environments being an issue, I should note that MUSiCC learns a different model for each sample, it never takes into account multiple samples when correcting a single sample. The two types of correction (intra vs. inter) refer to, respectively, correcting abundances due to potential sequencing/abundance calculation biases within the sample and correcting abundances based on universal single-copy gene abundance so that corrected abundances are comparable between samples.
Hope that helps!

This helps a ton! Thank you so much!


IndexError: list index out of range

I am running DAN-msa/pick_final_models.div.py and I get the following error
Traceback (most recent call last):
File "/share/dev1fs1b/chenxl/software/anaconda3/envs/folding/lib/python3.7/multiprocessing/pool.py", line 121, in worker
result = (True, func(*args, **kwds))
File "/share/dev1fs1b/chenxl/software/anaconda3/envs/folding/lib/python3.7/multiprocessing/pool.py", line 44, in mapstar
return list(map(*args))
File "/share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/DAN-msa/pick_final_models.div.py", line 72, in calc_lddt_dist
lddt_1 = float(os.popen("%s/lddt/lddt -c %s %s | grep Glob"%(script_dir, pose_i, pose_j)).readlines()[-1].split()[-1])
IndexError: list index out of range
"""
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/DAN-msa/pick_final_models.div.py", line 114, in
raw_dist = pool.map(calc_lddt_dist, args)
File "/share/dev1fs1b/chenxl/software/anaconda3/envs/folding/lib/python3.7/multiprocessing/pool.py", line 268, in map
return self._map_async(func, iterable, mapstar, chunksize).get()
File "/share/dev1fs1b/chenxl/software/anaconda3/envs/folding/lib/python3.7/multiprocessing/pool.py", line 657, in get
raise self._value
IndexError: list index out of range
How should this problem be solved? thanks you verv much

/bin/sh: /share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/lddt/lddt: No such file or directory
/bin/sh: /share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/lddt/lddt: No such file or directory
/bin/sh: /share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/lddt/lddt: No such file or directory
/bin/sh: /share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/lddt/lddt: No such file or directory
/bin/sh: /share/dev1fs1b/chenxl/RoseTTAFold/RoseTTAFold/lddt/lddt: No such file or directory


[Additional Feature] Encrypted Assertions

It would be better to add the possibility to encrypt the SAML assertions.
See:
https://github.com/IdentityPython/pysaml2/blob/master/src/saml2/server.py#L574
https://github.com/IdentityPython/pysaml2/blob/master/src/saml2/server.py#L675

Added here:
https://github.com/peppelinux/djangosaml2idp/commit/af69dc1edb245f0a5b32cf24cb04f822d0e8291b
needs settings parameter, works as expected!
I suggest you to implement this

That is indeed a good addition, I see no drawbacks for this.

merged with version 0.6.0


Proposal to track disclosure type in addition to safe harbor type

A company's willingness to disclose reported vulnerabilities can be of interest to researchers when evaluating programs, in addition to safe harbor. As described in the disclose.io terms, disclosure types can include coordinated disclosure, discretionary disclosure, and non-disclosure:

Coordinated Disclosure: A researcher can share details of the vulnerability after a fix has been applied and the program owner has provided permission to disclose, OR after 90 days from submission, whichever is sooner,
Discretionary Disclosure: The researcher or the program owner can request mutual permission to share details of the vulnerability after approval is explicitly received, or
Non-Disclosure: Researchers are required to keep vulnerability details and the existence of the program itself confidential.

To that end, I suggest adding a public_disclosure field to each program. As with the safe_harbor field, the public_disclosure field can have one of three values: "coordinated" for coordinated disclosure, "discretionary" for discretionary disclosure, and "" or "none" for non-disclosure.
Such a change will both aid researchers and enable tracking disclosure trends across VDPs over time.

@cablej Agreed. Feel free to commit a PR to the schema and the main list (cc: @codingo)

My 10 cents.
I'd update to this definition for 'Coordinated Disclosure'.

Coordinated Disclosure: A researcher can share details of the vulnerability after a fix has been applied and the program owner has provided permission to disclose, OR after (at maximum) 90 days from submission, whichever is sooner

I don't think it should be called 'coordinated disclosure' if the organization has a policy that extends beyond 90 days.

I'm still really not a fan that all 3 of these choices leave a researcher in a state where their protections from CFAA are predicated upon conforming to the disclosure policy of the company.
Is there some 4th option where a company can say that they won't pursue legal action even in the case of what they deem to be 'irresponsible disclosure'?
Google does this, & they will even pay if you disclose before the vuln is fixed. What category would their policy fall into in this case?

@JLLeitschuh This is confusing the disclosure dynamics  of vulnerability disclosure (i.e. the superset, and disclose.io's focus) with bug bounty (i.e. a subset).

This is confusing the disclosure dynamics of vulnerability disclosure (i.e. the superset, and disclose.io's focus) with bug bounty (i.e. a subset).

Sure, the way I phrased it above, I agree.
Let me put it another way, I'd like to make my companies VDP not require that researchers sync their disclosure with ours to make them feel like we'll protect them from CFAA. I don't want to couple safe-harbour with disclosure. What's the best way to do this and which category would such a policy fall into?


bindings/go: Fix bug with incomplete reads

Test plan
Added regression test

Did this go test failure just start happening? or was this from before?

That failure is entirely unrelated to this PR AFAICT, not sure why it's coming up but will look into it tomorrow before merging.

Perhaps an artifact of targeting a branch other than main, doesn't seem to be any issue any more.


fixes #267: [Multidatabase] Source: Implement “from” syntax / config

Fixes #267
This PR introduces the support for multi-tenancy for Neo4j 4.0 Enterprise Edition
Proposed Changes (Mandatory)
A brief list of proposed changes in order to fix the issue:

added a new module test-support that is used only in tests for Source (and in the future sink). This module contains an extension of the Neo4jContainter for testing enterprise features; in this case in order to provide the test we need to ship the built jar to the container so I also added Maven Wrapper in order to build the project from Kotlin.
added the from syntax
split the KafkaEventRouterIT in the same way we did for the sink plugin in order to simplify the management
updated the documentation

Thanks a lot @moxious for the feedback!


Meson dependencies pattern should pick list items

Please refer to this issue for details.

Hi, thanks for the PR. OpenBMC code contributions go through gerrit and then are synched into github once merged. Please see https://github.com/openbmc/docs/blob/master/CONTRIBUTING.md#submitting-changes-via-gerrit-server-to-openbmc

Thanks. Please have a look at change 34422. Will close this PR.


Fix deletion snackbar visibility issue.

Snackbar message is now clearly visible on deletion.

Updated title of pull request and provided screenshots.

You've changed the title of the PR, change the title of the commit too. You can also add some more details such as "in dark mode" and "in file manager".

Update the commit message as well.
git commit --amend will open up an editor to change the commit message. After amending, you'll have to force push the changes.

Last thing: Add this change to the changelog


added neqr

added neqr encoding, adapted qunatum image class to take in account how many color qubits i want to define. in main added dependecy from sixe inout to measure
Proposed Changes
adds neqr encoding

Codecov Report
Attention: Patch coverage is 88.33333% with 7 lines in your changes are missing coverage. Please review.

Project coverage is 61.78%. Comparing base (7f13237) to head (c69ed81).

Files
Patch %
Lines

src/qimp/main.py
0.00%
7 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
+ Coverage   56.37%   61.78%   +5.40%
==========================================
  Files           6        6
  Lines         298      348      +50
  Branches       34       50      +16
==========================================
+ Hits          168      215      +47
- Misses        129      132       +3
  Partials        1        1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Start using proper documentation software

I suggest a documentation pipeline that works with the following tools via Github Actions:

doxygen
sphinx (RTD theme)
breathe plugin

It could publish auto-generated documentation to a github-pages site.

SGTM


#3338 fix meta url construction

Fixes Issue
Fix #3338
Description of Change
Handle .json.zip urls in addition to .json.gz.
Have test cases been added to cover the new functionality?
yes

These changes are not enough to fix the issue. I'm continuing the investigation.


rename BuildDataTest til BuildDataUnitTest to avoid confusion as to w…

I have run into people being confused on numerous occasions using BuildDataTest in integrationTests causing flaky tests. Therefore I think the naming should be more explicit to prevent the confusion.

I've been confused by this several times as well. It's not clear if to use BuildDataTest or TestDataBuilder from the name alone.
My "workaround" is to check the Javadoc every time, but most don't, which causes problems.

Agreed, the naming on these two are pretty atrocious. A way to better differentiate them would definitely be an improvement to the API.


Feature request: Dark Mode for UI

It would be nice to have light, dark, system options in settings for UI.

https://github.com/pocketbase/pocketbase/issues/407


feat: AWS Secret Engine

Adds support for AWS Secret Engine with tests and docs
I tried to re-use existing code pattern and modules and tests, feel free to comment if it does not match somehow so code can be kept clean
Implementation is being used by a https://github.com/novadiscovery/novops branch, it works fine so far :)

Hey @jmgilman any chance you can take a look at this? :)

Looking forward to this

Hey @PierreBeucher Nice work I will merge it and include the changes in the next release.


A

[Enter feedback here]

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

ID: ba002086-e5b7-023a-4f1c-d8912c67cecd
Version Independent ID: 6bab0cb3-7eae-644d-36ab-8f8676e78932
Content: Common Data Model format - Azure Data Factory & Azure Synapse
Content Source: articles/data-factory/format-common-data-model.md
Service: data-factory
Sub-service: data-flows
GitHub Login: @kromerm
Microsoft Alias: makromer

@saepul85
Could you please elaborate the query for better understanding. So, we can assist you further.


Error on downloading model

Before downloading the model, create the models folder under server folder.

Fixing it soon

Simple fix.

In the server directory run mkdir -p models
In privateGPT.py replace the def download_and_save(): method with the following code:

def download_and_save():
    url = 'https://gpt4all.io/models/ggml-gpt4all-j-v1.3-groovy.bin'
    filename = 'ggml-gpt4all-j-v1.3-groovy.bin'
    models_folder = 'models'
    n_ctx = 1024  # Replace this with correct value

    response = requests.get(url, stream=True)
    total_size = int(response.headers.get('content-length', 0))
    bytes_downloaded = 0
    file_path = f'{models_folder}/{filename}'
    if os.path.exists(file_path):
        return jsonify(response="Download completed")

    with open(file_path, 'wb') as file:
        for chunk in response.iter_content(chunk_size=4096):
            file.write(chunk)
            bytes_downloaded += len(chunk)
            progress = round((bytes_downloaded / total_size) * 100, 2)
            print(f'Download Progress: {progress}%')

    callbacks = [StreamingStdOutCallbackHandler()]
    llm = GPT4All(model=file_path, n_ctx=n_ctx, backend='gptj', callbacks=callbacks, verbose=False)
    return jsonify(response="Download completed")

Then run python privateGPT.py

Fixed now


ModuleNotFoundError: No module named 'mpl_toolkits.basemap'

Hi, I was running the "Estimate Climate Driver Influence on Rainfall" notebook and it seems like the base map package has not been proper installed in the system. Please help double-check.
Thanks,
Daniel

Hey @Daniel-Trung-Nguyen , thanks for the report! What environment were you using when you got this issue? The NCI/VDI or DEA Sandbox? @MatthewJA wrote this notebook, so might have some thoughts on why it doesn't work depending on the environment.

Hey @Daniel-Trung-Nguyen , thanks for the report! What environment were you using when you got this issue? The NCI/VDI or DEA Sandbox? @MatthewJA wrote this notebook, so might have some thoughts on why it doesn't work depending on the environment.

Hi @robbibt and @MatthewJA,
I got this issue on the DEA Sandbox. The "import mpl_toolkits.basemap" loaded successfully with no issues on the NCI environment.
Cheers,
Daniel

Hi @robbibt and @MatthewJA,
I got this issue on the DEA Sandbox. The "import mpl_toolkits.basemap" loaded successfully with no issues on the NCI environment.
Cheers,
Daniel

This notebook will only work on the NCI - the data aren't available on Sandbox.

This notebook will only work on the NCI - the data aren't available on Sandbox.

Looks like that notebook needs updating anyway to use cartopy.

Looks like that notebook needs updating anyway to use cartopy.


Add support for bike mtb:scale accessibility

Triggered by the discussion in https://community.openstreetmap.org/t/consuming-highway-path-take-2/118175 I detected that currently we do not support the mtb:scale tag for the bicycles yet at all. This PR adds support for the detection of the accessibility. This does make sense in order to increase the safety of the suggested routes.
I was thinking about adding additional support for control of the priority detection, but came to the conclusion that this is not necessary as according to taginfo we are already handling the most often combined tags. Handling the prioritization stuff twice does not make sense.

Why can't we use mtb_rating in the different bike custom models?

Until now I only de-priorities encoded values and never blocked it. Would total blocking be possible via a custom rule, e.g. by assigning a speed of 0? Additionally the sac_scale is also handled in the parser and for symmetry we should do the same as this avoidance better should be available even without any custom model.

It would be nice if we can move it out of the parser too as it is already partially. As you can see with the mtb scale, the special handling with + and - needs to be duplicated here again.

Thank you. OK, I will try it. But what is the best way to test if a modified custom rule actually works as intended? Do I need to find and test this with real world examples?

Now I moved the sac_sale and mtb:scale handling for bicycles to the custom models. The PR therefore became much more complex then before. But now I think I understand what you mean when you say that the long term goal is to move the weighting completely into the custom models.

@karussell Did you miss this PR? Usually you responded very fast, but lately a couple of PRs are hanging around. I hope you are doing fine?

Did you already have time to test it for some real world cases? If yes, can you point me to one hike and one mtb case where this PR is important and makes a difference?

Yes for mtb:scale=2, you find the example with screenshots of the result in #3049 . For the sake of completeness here is the example link.
The result for sac_scale should be the same compared to before as only the implementation has changed. Here is an example I found via overpass

Should we really hard exclude those ways for bike? Isn't it the same like for steps that one could get off the bike and push (or even carry) the bike? But probably seldom in practise that a way with mtb:scale is the only path to the destination.
I'm undecided about this. What would you suggest? Is
 { "if": "mtb_rating > 2",  "multiply_by": "0.1" },
better?

But maybe we keep it as you suggested for now and if there are problems with this fix this according to real world examples?

Given the long time we didn't consider the tag mtb:scale this might take another 10 years :smiling_face_with_tear: , but I'm fine with it.


Get Tailwind CSS

Finding the project really interesting and useful. Thanks.
Am using the command 'editor.runCommand('get-tailwindCss')' to get the tailwind CSS used in the editor.
As the docs says, this outputs to the console, but can't find a way of getting the CSS as a variable I can save to a database.

After looking into the code, settled on the following:
const doc = editor.Canvas.getDocument();
doc.head.querySelectorAll('style').forEach(el => {
el.innerText.includes('tailwind');
twcss = el.innerText;
});
document.getElementById("styles").value = twcss;

Use this to get css
editor.runCommand('get-tailwindCss', { callback: async (twcss)=> { console.log(twcss } })


Cards are not shown when sent in a resolving Promise on ConsoleConnector

Hi! I'm using Node.js library version 3.1.1.
I'm facing problems on my app, so I did a quick test as a POC to help replicate it.
I copied the Skype Bot example for sending a ThumbnailCard (/actions, /weather and .beginDialogAction calls as well)
Also, I added a self-resolving promise. Something like what's below. And the card is not sent on ConsoleConnector. But it works on ChatConnector (testing on Facebook).
`
Promise.all([
new Promise((done, reject) => {
setTimeout(done, 1000);
}),
new Promise((done, reject) => {
setTimeout(done, 2000);
}),
new Promise((done, reject) => {
setTimeout(done, 3000);
})
        ]).then((body) => {

            var msg = new builder.Message(session)
                .textFormat(builder.TextFormat.xml)
                .attachments([
                    new builder.ThumbnailCard(session)
                        .title("Hero Card")
                        .subtitle("Space Needle")
                        .text("The <b>Space Needle</b> is an observation tower in Seattle, Washington, a landmark of the Pacific Northwest, and an icon of Seattle.")
                        .images([
                            builder.CardImage.create(session, "https://upload.wikimedia.org/wikipedia/commons/thumb/7/7c/Seattlenighttimequeenanne.jpg/320px-Seattlenighttimequeenanne.jpg")
                        ])
                        .buttons([
                            builder.CardAction.dialogAction(session, "weather", "Seattle, WA", "Current Weather")
                        ])
                ]);
            session.send(msg);

        });

`

To be fair... no cards are being displayed on ConsoleConnector...

Any input on this?
This is quite frustrating, as I need to send a card based on the response of a WebService call.
Ever since I implemented it, I can't use the ConsoleConnector to debug my code, and I have to push my code live and test it on Facebook...

@Stevenic is the expert on this, but I do not think the SDK supports setTimeout / closures / promises in this way. Are you trying to "proactively" message your user, at a later time?

It's not proactive. I fire a http request and build a sentence out of its
answer. So I need to send a message inside a promise.
That works fine with Facebook (UniversalConnector) but not with
ConsoleConnector
On 12 Aug 2016 19:51, "Dan Marshall"<EMAIL_ADDRESS>wrote:

@Stevenic https://github.com/Stevenic is the expert on this, but I do
not think the SDK supports setTimeout / closures / promises in this way.
Are you trying to "proactively" message your user, at a later time?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/761#issuecomment-239578468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACGGJ5tRoJEHkLI8wSjQaLhZ9e-WuNpdks5qfPjWgaJpZM4JQNiX
.

I just took a pull request which should fix this. There were a couple of bugs in the ConsoleConnector related to this. It wasn't properly calling done()

Closing this issue. If it's not fixed, please either open a new issue or reply to this one asking to reopen. thanks!

Hi!
Has it been released? I'm still facing this as of version 3.2.3.
Example code:
                    /*Building the card*/
                    const message = new ZipController.builder.Message(session);
                    message
                        .textFormat(ZipController.builder.TextFormat.plain)
                        .attachments([
                            new ZipController.builder.ThumbnailCard(session)
                                .title("[[Text]]")
                                .subtitle("[[Subtitle]]")
                                .buttons([
                                    ZipController.builder.CardAction.postBack(session, "sim", "sim"),
                                    ZipController.builder.CardAction.postBack(session, "não", "não")
                                ])
                        ]);

                    /*Simulating call to external service and sending card when the request is finished*/
                    new Promise((res) => {
                        setTimeout(()=>res(), 1000);
                    }).then(()=>{
                        console.log("sent");
                        builder.Prompts.choice(session, message, "sim|não");
                    });

I get "sent" printed on the console, but no card.

Sorry... It has now. Check the latest 3.3 release.


[DT-744] Add option to modify email preferences

Addresses
https://broadworkbench.atlassian.net/browse/DT-744
Summary
Add option to modify email preferences:
https://github.com/user-attachments/assets/cba8b194-0f6b-4b29-9728-d7d9e596265d

Have you read Terra's Contributing Guide lately? If not, do that first.

Label PR with a Jira ticket number and include a link to the ticket
Label PR with a security risk modifier [no, low, medium, high]
PR describes scope of changes
Get a minimum of one thumbs worth of review, preferably two if enough team members are available
Get PO sign-off for all non-trivial UI or workflow changes
Verify all tests go green
Test this change deployed correctly and works on dev environment after deployment

LGTM - thx!