Update tokio to 1.0

New version of tokio is released
reqwest vesrion of tokio is 0.2 Cargo.toml
Better upgrade, to avoid downloading two versions of tokio in projects

This actually seems to prevent upstream binaries from upgrading to tokio:0.3.0.
I tried upgrading my cli app bitbucket_server_cli, here is the result trying to run with incompatible tokio-versions
thread 'main' panicked at 'there is no timer running, must be called from the context of Tokio runtime', /Users/jensim/.cargo/registry/src/github.com-1acb6219da9ec827/tokio-0.2.22/src/time/driver/handle.rs:25:14
stack backtrace:
   0:        0x10edea394 - <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt::hd4b962ed89f71a03
   1:        0x10ee0de10 - core::fmt::write::h94ae1e793baa7a00
   2:        0x10ede448b - std::io::Write::write_fmt::h5c716758fdc3057f
   3:        0x10edebf45 - std::panicking::default_hook::{{closure}}::hc6119c7d16548caf
   4:        0x10edebc87 - std::panicking::default_hook::heae8b62897b351dc
   5:        0x10edec4e5 - std::panicking::rust_panic_with_hook::hc36596b4257bea99
   6:        0x10edec07b - std::panicking::begin_panic_handler::{{closure}}::h49e5ddc3f21ca2fb
   7:        0x10edea808 - std::sys_common::backtrace::__rust_end_short_backtrace::h9bd32c9ad3fad18f
   8:        0x10edec03a - _rust_begin_unwind
   9:        0x10ee2619f - core::panicking::panic_fmt::hcdc9362d34d55302
  10:        0x10ee2602a - core::option::expect_failed::h2cf6417a1cce758c
  11:        0x10eda9147 - tokio::time::delay::delay_for::ha5c963747a903eaf
  12:        0x10eceb48d - reqwest::async_impl::client::Client::execute_request::hf7a8c0de5eae6488
  13:        0x10ed284e2 - reqwest::async_impl::request::RequestBuilder::send::hcf6b3611e4279255
  14:        0x10ea49476 - bitbucket_server_cli::bitbucket::worker::BitbucketWorker::fetch_all_user_repos::{{closure}}::hddcb554493bccb0e
  15:        0x10ea5ad7d - <core::future::from_generator::GenFuture<T> as core::future::future::Future>::poll::hf40a34645155b35e
  16:        0x10ea32b81 - std::thread::local::LocalKey<T>::with::hc20d5bef6701544d
  17:        0x10ea389f7 - tokio::runtime::enter::Enter::block_on::h9eb328f39405ec67
  18:        0x10ea3cea5 - tokio::runtime::thread_pool::ThreadPool::block_on::h4174e4a4d5e0c544
  19:        0x10ea69bed - tokio::runtime::Runtime::block_on::h2bc0cb0537fac7ec
  20:        0x10ea354de - bitbucket_server_cli::main::h07f0683523971a5d
  21:        0x10ea6397a - std::sys_common::backtrace::__rust_begin_short_backtrace::h12ac5d6b20230f57
  22:        0x10ea6399c - std::rt::lang_start::{{closure}}::h771a80e1366e93a1
  23:        0x10edec880 - std::rt::lang_start_internal::hd0c760d47f593c0a
  24:        0x10ea355b9 - _main

Looking at reqwest dependency tree we are also blocked by the dependency chain

hyper -> h2 -> tokio:0.2
hyper -> tokio:0.2
hyper-tls -> tokio

➜  reqwest git:(master) cargo tree -i tokio
tokio v0.2.22
├── h2 v0.2.6
│   └── hyper v0.13.8
│       ├── hyper-tls v0.4.3
│       │   └── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)
│       └── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)
│       [dev-dependencies]
│       └── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)
├── hyper v0.13.8 (*)
├── hyper-tls v0.4.3 (*)
├── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)
├── tokio-tls v0.3.1
│   ├── hyper-tls v0.4.3 (*)
│   └── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)
└── tokio-util v0.3.1
    └── h2 v0.2.6 (*)
[dev-dependencies]
└── reqwest v0.10.8 (/Users/jenbri/vcs/github/reqwest)

https://github.com/meilisearch/meilisearch-rust/issues/57
https://github.com/Creatiwity/siren/pull/178
https://github.com/ThouCheese/cloud-storage-rs/issues/41
https://github.com/teloxide/teloxide/issues/292
https://github.com/khuey/bb8/issues/82
https://github.com/hyperium/hyper-tls/issues/76
https://github.com/graphql-rust/juniper/pull/794
https://github.com/hatoo/oha/issues/113

https://github.com/enarx/enarx-keepmgr/pull/10
https://github.com/robotty/recent-messages2/pull/68
https://github.com/dmolokanov/appinsights-rs/pull/131
https://github.com/1tgr/rust-websocket-lite/pull/118
https://github.com/IronCoreLabs/ironoxide/pull/194
https://github.com/IronCoreLabs/ironoxide-cli/pull/38
https://github.com/testcontainers/testcontainers-rs/pull/221
https://github.com/eldesh/dockworker/pull/99

https://github.com/seanmonstar/reqwest/pull/1076

N.B. this also means that trying to use reqwest from a tokio 0.3 runtime will give you this rather cryptic panic: https://github.com/tokio-rs/tokio/issues/1837

@seanmonstar Is there anything I can do to help get this done faster?  Could we do a point release?

This is waiting on hyper 0.14, which can be tracked here: https://github.com/hyperium/hyper/milestone/6
Progress keeps being made there :)

I got reqwest to compile by cloning patched versions of reqwest, hyper, and hyper-tls and modifying their Config.toml files:

https://github.com/messense/reqwest.git

git checkout #a3da4e094ede0f3c3ac45a2215742315b61dacca@tokio-0-3
Modify Cargo.toml and update dependency statements:

http-body = { git = "https://github.com/hyperium/http-body", rev = "5e434739" }
hyper = { path = "../hyper", default-features = false, features = ["tcp", "http1", "http2", "client"] }
hyper-tls = { path = "../hyper-tls", optional = true }

https://github.com/hyperium/hyper.git

git checkout #121c33132c0950aaa422848cdc43f6691ddf5785
Modify Cargo.toml and update dependency statements:

http-body = { git = "https://github.com/hyperium/http-body", rev = "5e434739" }

https://github.com/messense/hyper-tls.git

git checkout #07dc2306696b83d1a16476ea04c8c0755e6bff9d@bump-tokio
Modify Cargo.toml and update dependency statements:

hyper = { path = "../hyper", default-features = false, features = ["tcp", "client", "http1"] }

My project's integration test now builds and runs.  The Cargo.toml has:
[dependencies]
rustls = "0.18"
tokio = { version = "0.3", features = ["macros", "stream", "sync", "time"] }
tokio-rustls = "0.20"

[dev-dependencies]
reqwest = { path = "../deps/reqwest", features = [] }
tokio = { version = "0.3", features = ["fs", "macros", "net", "rt-multi-thread", "signal"] }
tokio-test = "0.3"

Thank you, all of you who are updating reqwest and its dependencies! :)

@mleonhard would it be possible to wrap all of that up into use of Cargo.toml [patch] section?
since you are overriding dependencies with patched forks, and then also overriding the dependencies of the patched forks themselves with each other.
This way users would not have to clone anything.

@mleonhard would it be possible to wrap all of that up into use of Cargo.toml [patch] section?
Since you are overriding dependencies with patched forks, and then also overriding the dependencies of the patched forks themselves with each other.
This way users would not have to clone anything.

This would be great!
I as a lib consumer would like to have at least a branch available here that I would be able to select in my application to prepare for tokio upgrade. Holding back versions will create a big latency, and should a security issue be unvailed somewhere in the dependency chain, us at the leaf level will be the last to patch, as always.
I would be super greatful if there was a patch-branch available here to point at in my patch branch..

Looks like we have tokio 1.0 available along with new hyper.

@seanmonstar Is there anything I can do to help get this done faster? Could we do a point release?

@sinanameh hyper-tls needs to have https://github.com/hyperium/hyper-tls/pull/79 merged and a new version released first.

hyper-tls has released v0.5.0 for tokio 1.0, so that's one blocker gone.
Any further blockers?

hyper-tls has released v0.5.0 for tokio 1.0, so that's one blocker gone.
Any further blockers?

This was done in #1076. Release coming today.

This was done in #1076. Release coming today.


Prepare for 0.8.2

Usual stuff

          1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger


feat: support import ts definition

Typescript support extension with .d.ts
But in current config of import , it should complain error when try import custom type.

Coverage remained the same at 97.802% when pulling eb355e9729d254cddf89086cec7dfe4fcdec9bb6 on davidNHK:feat/support-import-ts-definition into cf5573b5784a8b19c1a7c3e4003005dfaadc4375 on benmosher:master.

Coverage remained the same at 97.802% when pulling eb355e9729d254cddf89086cec7dfe4fcdec9bb6 on davidNHK:feat/support-import-ts-definition into cf5573b5784a8b19c1a7c3e4003005dfaadc4375 on benmosher:master.


Djfedos.ticket1

flipper.py iter. 1:
it accepts a file name in a CLI, flips the image and shows it in the window.
CLI is processed with python fire
image is processed with open cv following the https://docs.opencv.org/4.x/d1/da0/tutorial_remap.html example

@ydmeta


Accessing a colorpicker after the DOM has loaded

If I create a colorpicker element after the DOM has loaded, it does not seem to work. I would normally go and change all .click() events to .on("click", function() {}), but I see that the JS file already has this format. Is there any way I can add a colorpicker element to the page once the page has loaded?

Hi, this sounds very weird, have you take a look to the documentation and the samples?
Please, also take a look to the guidelines for contributing and publishing issues: https://github.com/mjolnic/bootstrap-colorpicker/blob/master/CONTRIBUTING.md
Then provide a non working example, reproducing your issue on a JSFiddle or similar.
Thanks


OLS-308: Add watsonx and Azure OpenAI support

Signed-off-by: Vladimir Belousov<EMAIL_ADDRESS>Description
Added support for IBM Watsonx and Azure OpenAI
Type of change

[ ] Refactor
[x] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library

Related Tickets & Documents

Related Issue #
Closes https://issues.redhat.com/browse/OLS-308

Checklist before requesting a review

[x] I have performed a self-review of my code.
[x] PR has passed all pre-merge test jobs.
[x] If it is a core feature, I have added thorough tests.

Testing

Please provide detailed steps to perform tests related to this code change.
How were the fix/results from this change verified? Please provide relevant screenshots or results.

/lgtm

/lgtm
/approve


feat: adds support for vanity subdomains

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

11 of 142 (7.75%)  changed or added relevant lines in 5 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.2%) to 56.228%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/vanity_subdomains/delete/delete.go
0
23
0.0%

internal/vanity_subdomains/check/check.go
0
26
0.0%

internal/vanity_subdomains/get/get.go
0
26
0.0%

cmd/vanitySubdomains.go
11
38
28.95%

internal/vanity_subdomains/activate/activate.go
0
29
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.2%

Covered Lines:
3354

Relevant Lines:
5965

💛  - Coveralls


spi-security: Simplify SecurityModuleAppUserRegistrationServiceAbstract

from https://github.com/isisaddons-legacy/isis-module-security/issues/46
bilgin:
What is the reason for SecurityModuleAppUserRegistrationServiceAbstract to have 2 abstract methods getInitialRole and getAdditionalInitialRoles?
Why not have only one method that returns Set? It will be easier to override. Otherwise, it makes you think that initial role is different that any any additional role.?

dan:
Yep, I agree. Was probably just how it evolved over time.
Obviously it'll be a breaking API change, but I'm less fussy about that so long as the changelog in the README explains how to fix (perhaps worth an additional section in the README to address this).
If you want to create a PR for this, I'll be happy to merge it in.
Thx


C++ gradients: reduce_min, reduce_max

Anyone already working on adding these two operators to the C++ gradients? Otherwise, I'll sign up for it.
/cc @bpiel @suharshs

@kbsriram Not me. Thanks for asking.


Add notebook to evaluate BART decoder experiments

Closes #24
Closes #28
Add notebook to evaluate BART decoder experiments.

Looks good!


Get toast service directly from bridge library

Thank you for your work on this excellent bridge.
I am using Typescript and had a little problem with the toast service. The library 'aurelia-materialize-bridge/toast/toastService' (from http://aurelia-ui-toolkits.github.io/demo-materialize/#/samples/dialogs/) could not be found.
import { MdToastService } from 'aurelia-materialize-bridge/toast/toastService';

.. not found compile failed

and
import { MdToastService } from 'aurelia-materialize-bridge';

compiled ok but error on run

So I tried this:
aurelia-materialize-bridge/src/index.js
export * from 'aurelia-materialize-bridge/toast/toastService';

And made className optional:
aurelia-materialize-bridge/src/toast/toastService.js
..
  show(message, displayLength, className?) {
..

Then this work:
import { MdToastService } from 'aurelia-materialize-bridge';

compiles ok run ok

Alternative is to add bridge/toast/toastService definition:
<EMAIL_ADDRESS>declare module 'aurelia-materialize-bridge/toast/toastService' {
    export class MdToastService {
        show(message: any, displayLength: any, className?: any): any;
    }
}

Thanks for mentioning, @Ullfis. I definitely need to use this with TypeScript more. :smile:
To be honest, I didn't really have time to take a deeper look at the babel-dts-generator so I can't decide what would be the best solution of the two you mentioned.
But I think the first proposal should work. I will try that by tomorrow - not sure if I can do it today.

This solution works but still throws an error:
Cannot find module 'aurelia-materialize-bridge/toast/toastService'."

It seems to be ignored, though.

fixed in  https://github.com/aurelia-ui-toolkits/aurelia-materialize-bridge/commit/797d3b1d26aca26879c0a3beed4f9828b6bdec5e

Now this comes back at me when running gulp doc-generate which is a dependency of gulp prepare-release:
Cannot find module 'aurelia-materialize-bridge/toast/toastService'.

I played around with this today and concluded that the export..
export * from './toast/toastService';

or

export * from 'aurelia-materialize-bridge/toast/toastService';

..in index.js is not needed in the definition file.
The code of MdToastService is included when gulp build-index is run. The definition file is created by gulp build-es6-temp based on the gulp build-index result file. The result file is not used anymore.
How to remove the export?

use remove tags

src/index.js:
// build-index-remove start
export * from './toast/toastService';
// build-index-remove end

and make some changes in the build/tasks/build.js file:
..line 15..
var jsName = paths.packageName + '.js';

// RegExp remove tags during insert.transform in build-index gulp task
var startTag = '//\\s*build-index-remove start';
var endTag = '//\\s*build-index-remove end';
var removeRegExp = new RegExp(startTag + '[^]+?' + endTag, 'g');

gulp.task('build-index', function(){
  var importsToAdd = [];

  return gulp.src(paths.source)
    .pipe(through2.obj(function(file, enc, callback) {
      file.contents = new Buffer(tools.extractImports(file.contents.toString("utf8"), importsToAdd));
      this.push(file);
      return callback();
    }))
    .pipe(concat(jsName))
    .pipe(insert.transform(function(contents) {
      return tools.createImportBlock(importsToAdd) + contents.replace(removeRegExp, '');
    }))
    .pipe(gulp.dest(paths.output));
});
..

Remember the
contents.replace(removeRegExp, '');

in

    .pipe(insert.transform(function(contents) {
      return tools.createImportBlock(importsToAdd) + contents.replace(removeRegExp, '');
    }))

Thanks a ton for investigating this!
Just as a reminder: you can also issue a pull request with that content. :smile:
I will apply that today when I have some more time.

Thanks for the fix! :)

merged in #140


Update script.js

15 seconds is not enough for the script to work properly in long chats

Done it, thanks for the tip


[SEDONA-535] Add the pull request labeler

https://github.com/actions/labeler
Did you read the Contributor Guide?

Yes, I have read the Contributor Rules and Contributor Development Guide

Is this PR related to a JIRA ticket?

Yes, the URL of the associated JIRA ticket is https://issues.apache.org/jira/browse/SEDONA-535. The PR name follows the format [SEDONA-XXX] my subject.

What changes were proposed in this PR?
Adds the pull request labeler to increase automation.
How was this patch tested?
Did this PR include necessary documentation updates?

No, this PR does not affect any public API so no need to change the documentation.

Hey @jiayuasu to finish this PR we need to add two new labels for root and licenses on the labels page
https://github.com/apache/sedona/labels


Validate workflow.metadata on PUT /api/holdingpen/<workflow_id>

Separated from #1450

PUT /api/holdingpen/ must validate and return validation errors in the format that ng2-json-editor expects

https://github.com/inspirehep/inspire-next/blob/d67c86279968cd8843d657071c38e0838d0780aa/inspirehep/modules/workflows/utils/__init__.py#L449 returns the errors in the right format already and they should be set to extra_data.validation_errors

Btw saving a workflow is somewhat complex on the client side https://github.com/inspirehep/inspirehep/blob/ffae4ded3d52f848d95044ce1c95869fe9545c6b/record-editor/src/app/holdingpen-editor/holdingpen-save-button/holdingpen-save-button.component.ts#L115-L188
So make sure we handle VALIDATION_ERROR type of errors for saving without callback url too.

Backend TECH NOTES:

Copy https://github.com/inspirehep/inspire-next/blob/d3ff3c8fb1fde8e94b7597fa4f16bb08f2811bb8/inspirehep/modules/workflows/utils/init.py#L449 into inveniosoftware-contrib/invenio-workflows-ui
on put we should validate https://github.com/inspirehep/inspire-next/blob/d3ff3c8fb1fde8e94b7597fa4f16bb08f2811bb8/inspirehep/modules/workflows/tasks/actions.py#L303-L312 as here


merge package template from arlac77/npm-package-template

package.json

chore: update package.json from template

.gitignore

chore(git): update .gitignore from template

Codecov Report

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

@@           Coverage Diff           @@
##           master     #241   +/-   ##
=======================================
  Coverage   77.31%   77.31%
=======================================
  Files           3        3
  Lines         194      194
=======================================
  Hits          150      150
  Misses         44       44

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 2c4d247...ce2058d. Read the comment docs.

Coverage remained the same at 67.883% when pulling ce2058d1a31a9d71217c0ee218fea5f59b5dc346 on template-sync-1 into 2c4d247d9d5491684eef1f89db9ed2361e53cdec on master.


added instructions for openseadragondemo

umm, i committed the openseadragon library in here cause it was needed for the demo
i'm not sure if we want it bundled that way. Perhaps we can merge just the gitbook file and demo changes and leave the openseadragon library out. I'm not sure.


Registration form

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #(234)
Replace 234 with the issue number which is fixed in this PR
Type of change
Created Registration using HTML AND CSS.
This form consist of
1 First Name
2 Last Name
3 Email ID
4 Phone number
5 Branch
6 Year
7 Gender
For every title created seperate classes.
"I havn't put background image because of copyright issue but have implemented this feature".

Please let me know anything required more in this.

Please let me know anything required more in this.

Mention Your Issue no with #

I fixes issue #237


Configure Codecov

This PR adds configuration for Codecov as in this Issue.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3b6ad3c). Click here to learn what that means.
 Consider uploading reports for the commit 494a3b8 to get more accurate results

Additional details and impacted files
@@          Coverage Diff           @@
##             master   #13   +/-   ##
======================================
  Coverage          ?     0
======================================
  Files             ?     0
  Lines             ?     0
  Branches          ?     0
======================================
  Hits              ?     0
  Misses            ?     0
  Partials          ?     0


Github backend requests both read and write access

When using the github backend, the scope requested is user, which includes both read and write permissions to user data: https://github.com/greenpau/caddy-auth-portal/blob/0f64ae51b30682557cfa1ffb9dedcc92dc689aaf/caddyfile.go#L270
This is not ideal, since caddy shouldn't be changing user data. Instead, we could use the scope read:user which requests only read access.
Github documentation: https://docs.github.com/en/developers/apps/building-oauth-apps/scopes-for-oauth-apps

@ziyaointl , thank you for the question! Please use generic provider instead of github. This way you can have Github integration with the more restrictive scope.

@ziyaointl , this sounds like a better default. Please see https://github.com/greenpau/caddy-auth-portal/issues/231#issue-1106200478 and re-open the issue in caddy-security.

@greenpau Thanks for such a swift reply!
I attempted to use the generic provider but unfortuantely was stuck on an error "authorization server response has no id_token field". Digging a bit deeper, I see that id_token seems to be required by the factory function that creates a backend, with no option to customize it in a Caddyfile.
https://github.com/greenpau/caddy-auth-portal/blob/f56ad05f79e9cb4a35495bf730444afa8e72b45b/pkg/backends/oauth2/backend.go#L76-L79
Here's my configuration for the generic oauth provider that generates the error:
backends {
      generic_oauth2_backend {
          method oauth2
          realm generic
          provider generic
          client_id <redacted>
          client_secret <redacted>
          base_auth_url https://github.com/login/oauth/
          authorization_url https://github.com/login/oauth/authorize
          token_url https://github.com/login/oauth/access_token
          identity_token_name access_token
          disable key_verification
          enable accept_header
          scopes openid email profile
      }
}

As I was typing this I see your new reply now :) Apologies for opening this in the wrong place! I'll open a new issue in caddy-security.

I see that id_token seems to be required by the factory function that creates a backend, with no option to customize it in a Caddyfile.

👍 will add it via the new issue 😄 Alternatively, once I finalize the migration, you could open a PR yourself. It should be pretty straight-forward.


Merge pull request #1 from CymChad/master

new
Thank you for contributing to BaseRecyclerViewAdapterHelper. Before pressing the "Create Pull Request" button, please consider the following points:

[1] Please give a description about what and why you are contributing, even if it's trivial.

[2] Please include the issue list number(s) or other PR numbers in the description if you are contributing in response to those.

[3] Please include a reasonable set of demo tests if you contribute new code or change an existing one. please make sure you have demo for working correctly.

1

1


Bump v1.0.1

In the current form, we can't merge this PR because it assumes published 1.0.1 links in the live website, whereas to publish 1.0.1 to NPM, we need to update the versions first and merge to master first. Docs update and Leaflet source update must be separate PRs.


.Net: Memory and document structure for semantic search should be extensible

This feature request could be a bit bigger.
As suggested in one of the last public SK office hours, it would be great if the memory management for semantic searches would be more extensible and more pluggable.
Some use cases that were discussed were as follows:

allow control of the collections used for document storage & retrieval
allow consuming of document stores created with LangChain (be able to use the document format from there)
allow asynchronous transformation of the original document before indexing (similiar to HyDE, but more like HyQE: Hypothetical Question Embeddings, you send the document to a model to generate a few questions about the document, and the questions will get indexed while the original document or a reference to that content is stored as metadata)
allow asynchronous re-transformation of documents after retrieval, before passing them back to the model (same scenario, the document was transformed at indexing, now we found the questions and need to return the original documents content, but we only stored a reference and need to fetch the contents from somewhere over a network)

This would mean introducing a new interface (i.e. DocumentTransformer) with two async methods that are called before storage and after retrieval. Or two distinct interfaces with a single method each.
My idea would be to register the transformer implementation type with the corresponding adapter. If a transformer is registered, the new methods would be called before passing the document to the adapter or after the adapter returned the found document. This of course means that the adapters must not make assumptions about the structure of the documents itself, as they might now work on LangChain documents that will be transformed to SK documents (or vice versa).

@dluc, do you have thoughts on the recommendation by @gingters?

please see https://github.com/microsoft/semantic-kernel/issues/1044

more control for collections/indexes should be already available, would love hearing some feedback
async transformation is available :-) see how the pipelines work, plus the ability to develop custom pipelines
consuming docs ingested by other solutions: work in progress

async transformation is available :-) see how the pipelines work, plus the ability to develop custom pipelines

Where is the documentation of custom pipelines? I searched https://learn.microsoft.com/en-us/semantic-kernel/. But I cannot find any related pages


add check-proto-version to verify if all the versions of related proto generation tools are expectd

Description
Add check-proto-version to verify if all the versions of related proto generation tools are expectd, this checking will be executed in target gen-proto: if the version is not correct, the build will exist with error notification.
the new added target check-proto-version will check these three version:

protoc: expected is 3.14.0
protoc-gen-go: expected is v1.25.0
protoc-gen-go-grpc: expected is 1.1.0

As command shows:
% protoc --version
libprotoc 3.14.0
% protoc-gen-go --version
protoc-gen-go v1.25.0
% protoc-gen-go-grpc --version
protoc-gen-go-grpc 1.1.0

Issue reference
See discussion in PR:
https://github.com/dapr/dapr/pull/3037
Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Unit tests passing
[ ] 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]

One special thing is that protoc-gen-go use stderr (NOT stdout) to output version information, so I have to redirect the stderr to stdout to catch the output version:
@test "$(shell protoc-gen-go --version 2>&1)" = "protoc-gen-go v1.25.0" \

@skyao look at the build errors

I added a checking in the build process which is similar to "go mod tidy check diff", as @artursouza suggested.
      - name: Run go mod tidy check diff
        if: matrix.target_arch == 'amd64' && matrix.target_os == 'linux'
        run: make modtidy check-diff
      - name: Run gen-proto check diff
        if: matrix.target_arch == 'amd64' && matrix.target_os == 'linux'
        run: make gen-proto check-proto-diff

But now see in the build there is no command for gen-proto/protoc-gen-go-grpc/protoc:
Run make gen-proto check-proto-diff
make: protoc: Command not found
make: protoc-gen-go-grpc: Command not found
/bin/sh: 1: protoc-gen-go: not found
please use protoc 3.14.0 to generate proto, see https://github.com/dapr/dapr/blob/master/dapr/README.md#proto-client-generation
make: *** [Makefile:287: check-proto-version] Error 1
Error: Process completed with exit code 2.

So should I remove this checking in build process?

I added a checking in the build process which is similar to "go mod tidy check diff", as @artursouza suggested.
      - name: Run go mod tidy check diff
        if: matrix.target_arch == 'amd64' && matrix.target_os == 'linux'
        run: make modtidy check-diff
      - name: Run gen-proto check diff
        if: matrix.target_arch == 'amd64' && matrix.target_os == 'linux'
        run: make gen-proto check-proto-diff

But now seems in the build evn there are no such commands( gen-proto/protoc-gen-go-grpc/protoc):
Run make gen-proto check-proto-diff
make: protoc: Command not found
make: protoc-gen-go-grpc: Command not found
/bin/sh: 1: protoc-gen-go: not found
please use protoc 3.14.0 to generate proto, see https://github.com/dapr/dapr/blob/master/dapr/README.md#proto-client-generation
make: *** [Makefile:287: check-proto-version] Error 1
Error: Process completed with exit code 2.

So should I remove this checking in build process?

The required binaries need to be installed upfront. There should be new steps installing those.

@skyao  Please, look at the broken build for this PR.

ping @skyao

The required binaries need to be installed upfront. There should be new steps installing those.

I don't know how to add the steps to install required binaries in the build env.


ci: fix push to dockerhub when tags are pushed

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Fixes pushing images to DockerHub when a tag is created and pushed.

What is the current behavior? (You can also link to an open issue here)

What is the new behavior (if this is a feature change)?

Other information:

Status

[ ] Code documentation for the relevant parts in the code have been added/updated by the PR author
[ ] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report
Patch coverage has no change and project coverage change: +20.85 :tada:

Comparison is base (8a4a58c) 68.14% compared to head (eb288d3) 89.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##           master     #948       +/-   ##
===========================================
+ Coverage   68.14%   89.00%   +20.85%
===========================================
  Files         280       11      -269
  Lines       34976     2400    -32576
  Branches     4679      306     -4373
===========================================
- Hits        23834     2136    -21698
+ Misses      10689      263    -10426
+ Partials      453        1      -452

see 269 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[chore] Error codes cleanup

Notes for Reviewers
This PR removed some of the unused error codes.
I would create new ones, but it seems like they weren't being used anywhere in the repo.
There is one warning on one error code: ErrInvalidRequestObjectCode which does exist, but the errorutil doesn't check if the error message format is appending new strings, or if the fields are nil.
I will be adding a check in meshkit for it.
If there are any error codes you would like to use here, please let me know, and we can make revisions.
Signed commits

[ ] Yes, I signed my commits.

Refreshing with changes from HEAD.
@Revolyssup your eyes on this are appreciated.

@nebula-aac did we end up with new issues to track enhancements on the errorutils? I can't remember...

I will create a new issue tracking errorutil.
//@leecalcote

Thank you much. 🙂


Performance issues

Tried searching, couldn't find an issue on it... Sorry if it exists already!
I was investigating why our eslint run is super slow, and it seems like eslint-plugin-react has the 3 worst offenders, by almost an order of magnitude (totalling 3/4 of our runtime). Is this known?
This is using<EMAIL_ADDRESS>if improvements have been made in 7, that's great!
$ TIMING=1 node_modules/.bin/eslint . --ext js,jsx
Rule                    | Time (ms) | Relative
:-----------------------|----------:|--------:
react/prop-types        |  2276.690 |    24.9%
react/no-multi-comp     |  2141.514 |    23.4%
react/sort-comp         |  2107.674 |    23.0%
max-len                 |   309.908 |     3.4%
indent                  |   284.646 |     3.1%
no-unused-vars          |   191.424 |     2.1%
keyword-spacing         |   119.781 |     1.3%
object-shorthand        |    80.173 |     0.9%
camelcase               |    69.895 |     0.8%
no-unexpected-multiline |    64.387 |     0.7%

Config: https://gist.github.com/SimenB/0d5a3f1db680364399c019eede0a713a

Just out of curiosity, how large is your project?
I see a lot of differences with my output, e.g. for me no-unused-vars is much slower but react/no-unused-prop-types and react/sort-comp are much faster.
Rule                           | Time (ms) | Relative
:------------------------------|----------:|--------:
import/no-nodejs-modules       |  6059.752 |    27.7%
react/no-unused-prop-types     |  1304.725 |     6.0%
react/prop-types               |  1191.717 |     5.4%
no-unused-vars                 |  1151.751 |     5.3%
react/no-direct-mutation-state |  1041.367 |     4.8%
react/no-string-refs           |   986.259 |     4.5%
max-len                        |   943.344 |     4.3%
react/sort-comp                |   655.701 |     3.0%
indent                         |   651.466 |     3.0%
react/no-deprecated            |   580.024 |     2.7%

This project has about 40k lines of code in 1000 files for the source. And about 50k lines of code in 750 files for the tests. Number of React components is around 225.

Doing cloc gives:
-------------------------------------------------------------------------------
Language                     files          blank        comment           code
-------------------------------------------------------------------------------
JavaScript                     352           3891           1039          23763

While there's always performance improvements to be had, unfortunately we didn't act on these helpful benchmarks back in 2017, so I'm going to close this. I'm happy to reopen it with updated benchmarks, if it's still an issue with the latest version.


Update open.mp SDK, add a build CI (using github actions)

An example of working ci in action: https://github.com/AmyrAhmady/Pawn.CMD/actions/runs/3861552296


Link typo Set-EcpVirtualDirectory.md

https://github.com/MicrosoftDocs/office-docs-powershell/issues/5104

Docs Build status updates of commit 14c9523:
:clock10: Full build: the average full build time is 12 min(s) 27 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 14c9523:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/client-access-servers/Set-EcpVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

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 cebecfa:
:clock10: Pending: waiting for processors (13 builds ahead of you)
:warning: Docs Build is busy, currently there are 13 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit cebecfa:
:clock10: Full build: the average full build time is 12 min(s) 28 sec(s), this is based on the last 1000 build(s) for this repository.

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/client-access-servers/Set-EcpVirtualDirectory.md
:white_check_mark:Succeeded
View (exchange-ps)

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 ObservedGeneration to the sub Resources

This patch does a few things:

it adds observedGeneration to the sub custom resources
it proposes to bump the observedGeneration at the beginning of the reconciliation loop (https://github.com/knative/serving/issues/4937)
it checks, at the top level, if theObservedGeneration matches with the metadata.generation assigned to the subCR(s)
before marking the DeploymentReadyCondition as True

Quite a few copy-paste errors. I'm not sure I caught all of them. Could you please go through the changes and make sure there aren't more of them?

Thank you, good catch, I think I fixed the leftovers at this point

/lgtm


Feeds don't show the title or header at the top

Describe the bug
Feeds no longer show the feed title or header at the top. Even the buttons to unpin or unsave are gone.
To Reproduce
Steps to reproduce the behavior:

From the bsky.app (web) main screen, click one of the feeds from the column on the right.

Expected behavior
You should see info about the feed at the top. Here is a screenshot from going to the feed's URL:

But if you click the list of pinned feeds, you only see "Bluesky":

Details

Platform: web - desktop Chrome
Platform version: Windows 11
App version: 1.68.0

This is intended, re: https://github.com/bluesky-social/social-app/pull/2775

Oh no! It seems like some use cases are now MUCH more challenging.
So now, there isn't a quick way to see the "About" info for a feed, which often has critical info like what hashtag or keywords to use and the rules of the feed.
There's also no quick way to send someone a link to a feed ("Hi fellow bird watcher. You might like the bird feed: [URL]").

This is all true — note these are also all problems that already exist on mobile. So it might be good to figure out some holistic solutions to these. I agree we should make the header useful again though.


[Feature Request] Keyboard Navigation when Sidecar Panel is focused

Hello @matthewturk !!!!
If possible, would you mind adding a simple keyboard navigation using ↑ ↓ when the focus is in the Sidecar Panel view.
I have long notes and would be spectacular to be able to  “scroll” using the keyboard keys.

Thanks for reading this FR!
I hope you like my idea 🙏

Hi!  Do you want the main note to scroll, or the sidecar panel?

Hello @matthewturk !!!!
Thanks for the reply :)

Hi! Do you want the main note to scroll, or the sidecar panel?

The sidecar panel :)

Let me see what I can do!  I honestly didn't realize it wouldn't scroll.

So I tried this out on my installation.  I've got a really really long sidecar panel note, and I've focused the panel it's displayed in.  Pressing up and down on my keyboard does scroll it.  Am I misunderstanding?

First of all forgive for the late reply, I literally burn my fingers while using the oven but I all good  now 🙏

So I tried this out on my installation. I've got a really really long sidecar panel note, and I've focused the panel it's displayed in. Pressing up and down on my keyboard does scroll it. Am I misunderstanding?

I can confirm the same behavior on 0.2.4 using the mouse to focus on the sidecar panel and after hitting arrow keys.
I think the issue is on my end and I think I know why. I kindly ask some time before closing this issue because I wanna do some tests.
Sharing my thoughts: I think the issue happens only when you use the keyboard to focus on the sidecar panel. That’s why is not working on my end. Let me see what I can do and I get back to you :)
Thank you very much for the help 🙏

Oh, interesting!  Maybe a hotkey to both focus the panel and focus the scrollable area would fix it?

Oh, interesting! Maybe a hotkey to both focus the panel and focus the scrollable area would fix it?

That would be perfect @matthewturk !!! 🙏 🙏 🙏
This is exactly what we have for all Core Plugins that has Sidebar interfaces File Explorer , Tags and Bookmarks :

Show File Explorer , Show Tags and Show Bookmarks will focus on the Sidebar and focus on the first item in the Sidebar

I hope it is feasible to implement this 🙏

@FelipeRearden I think I have an idea, related to the navigation variable on the ItemView object.  I'm working to test it.

@FelipeRearden I think I have an idea, related to the navigation variable on the ItemView object. I'm working to test it.

Thank you very much @matthewturk !!!!! I hope you have success 🙏 🙏 🙏 🙏 🙏 🙏 🙏 🙏 🙏

I wasn't totally able to fix this.  I'm going to see if I can get #2 first.

I wasn't totally able to fix this. I'm going to see if I can get #2 first.

Okay !!! Thank you very much for the feedback ☀


[BUG] BinaryData.ToString() throws nullreference exception after upgrading Microsoft.Extensions.Azure/Azure.Core

Library name and version
Azure.Core 1.44.0
Describe the bug
The dependency System.Memory.Data is upgraded in version 1.44.0 to 6.0.0, this version has very annoying bug. When BinaryData is empty, the .ToString() method throws. This is fixed in a later version of the System.Memory.Data library.
The previous version (1.42.0) used System.Memory.Data version 1.0.2, this version doesn't have the problem.
Expected behavior
BinaryData.ToString() should not throw
Actual behavior
It does :-(
Reproduction Steps
Add manual package dependency to Microsoft.Extensions.Azure 1.7.6
public async Task UpdateBuildingFileStatusActivity(
[BlobInput("path/to/emptyfile.txt", Connection = "MadasterStorage")] BlobClient blob)
{
var content = await blob.DownloadContentAsync();
var text = content.Value.Content.ToString();
}
Environment
No response

Change: https://github.com/Azure/azure-sdk-for-net/pull/46134 (this mentioned the bug)

@m-redding:  Would you please coordinate with the Storage team to make sure a release is going out this week with your fixes?

Hello @mlankamp the best way to fix this is to add a direct reference in your application on System.Memory.Data 8.0.0 or protect against the bug in the 6.0.0 package with a try/catch. Unfortunately, this is a bug in the System.Memory.Data package so it's not something that can be fixed in the Azure.Storage.Blobs package.

After internal discussion, it's clear there was an error in my initial analysis.  My understanding that the exception was internal to the Storage libraries was incorrect and, as Maddy noted, a new release won't resolve the issue.   Apologies for the confusion.
To add some additional context, there's a number of our dependencies that are in the process of being upgraded in order to ensure that we remain on supported versions and avoid vulnerabilities. Unfortunately, we also have to ensure compatibility with existing platforms such as Azure Functions and Azure PowerShell which add constraints to the versions that we're able to adopt.
In this case, our version of System.Memory.Data was flagged as needing to be upgraded.  Unfortunately, due to the dependency that it has on System.Text.Json, we're required to stick to the 6.x line for the immediate future.  Because .NET has chosen not to patch the ToString bug in the 6.x line and we're unable to move to the 8.x, there is friction between our priorities of ensuring that we are using safe dependencies and avoiding platform bugs.  To mitigate, we've updated calls toSystem.Memory.Data internal to the Azure SDK packages to ensure there's no behavioral change.
Unfortunately, we cannot do the same for application calls to ToString and regret the inconvenience caused by the behavioral change to your code.   As Maddy recommended, the best path forward is to take a direct dependency on the newest System.Memory.Data which contains the fix.  This will hoist the version used by the Azure SDK packages as well.  If you are unable to do so, for some reason (such as the application being an in-proc Azure Function), then guarding ToString use with a try/catch or a length check would be the best path forward.

@jsquire Thank you for the update. Maybe the release notes for https://github.com/Azure/azure-sdk-for-net/releases/tag/Microsoft.Extensions.Azure_1.7.6 should be updated, so that people know that, Azure.Core has been updated and with the side effect mentioned.

@jsquire Side effect with this (minor) upgrade Microsoft.Extensions.Azure from 1.7.5 --> 1.7.6 is/was that our azure functions running .net 8 inprocess won't handle servicebus triggers any longer. We didn't see any "toString()" but just the system.memory.data binary could not be found.
Due we see more issues with missing assembly's it cost some time to realize this small nuget update caused the issue. So if someone is also struggling with this could help to rollback to 1.7.5
The error which shows up:
Exception while executing function: ServicebusHandlerName Could not load file or assembly 'System.Memory.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The system cannot find the file specified.
Microsoft.Azure.WebJobs.Host.FunctionInvocationException:    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor+<ExecuteWithLoggingAsync>d__26.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a\_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionExecutor.cs:352)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor+d__18.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionExecutor.cs:108)
Inner exception System.IO.FileNotFoundException handled at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw:
at XX+d__3.MoveNext (XX.Functions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null)
at System.Runtime.CompilerServices.AsyncMethodBuilderCore.Start (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at XX.Functions.XX.Run (XX.Functions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null)
at lambda_method81 (Anonymously Hosted DynamicMethods Assembly, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null)
at Microsoft.Azure.WebJobs.Host.Executors.VoidTaskMethodInvoker2+<InvokeAsync>d__2.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a\_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\VoidTaskMethodInvoker.cs:20)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.WebJobs.Host.Executors.FunctionInvoker2+<InvokeAsync>d__10.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a\_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionInvoker.cs:53)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter1.GetResult (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor+d__33.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionExecutor.cs:581)
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)
at System.Runtime.CompilerServices.TaskAwaiter1.GetResult (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor+<ExecuteWithWatchersAsync>d__32.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a\_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionExecutor.cs:523)    at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at System.Runtime.CompilerServices.TaskAwaiter.GetResult (System.Private.CoreLib, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=7cec85d7bea7798e)    at Microsoft.Azure.WebJobs.Host.Executors.FunctionExecutor+<ExecuteWithLoggingAsync>d__26.MoveNext (Microsoft.Azure.WebJobs.Host, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35: D:\a\_work\1\s\src\Microsoft.Azure.WebJobs.Host\Executors\FunctionExecutor.cs:306)
Note. HTTP trigger within the same azure function project just works with no errors.

Also @bvmeer would you be mind copying & pasting what you wrote into a separate issue? Just in case anyone else running into this then it'll be easier to have it in one spot

It looks like on top of what @bvmeer mentioned this upgrade seems to completely break .NET 6 in-process model to the point where host refuses to start. From what Ive seen this issue seems to occur not only on local machine but also in Azure Function Apps.

Copy-pasted ver
Azure Functions Core Tools
Core Tools Version:       4.0.6280 Commit hash: N/A +421f0144b42047aa289ce691dc6db4fc8b6143e6 (64-bit)
Function Runtime Version: 4.834.3.22875

[2024-10-13T12:09:47.288Z] Found C:\Users\XDD\Desktop\Temp\Company.FunctionApp1\Company.FunctionApp1\Company.FunctionApp1.csproj. Using for user secrets file configuration.
[2024-10-13T12:09:48.546Z] A host error has occurred during startup operation '4d9a5020-76ed-4b92-bf08-6618f0cbfd38'.
[2024-10-13T12:09:48.547Z] System.Private.CoreLib: Exception has been thrown by the target of an invocation. Microsoft.Azure.WebJobs.Extensions.Storage.Queues: Could not load file or assembly 'System.Memory.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The system cannot find the file specified.
Value cannot be null. (Parameter 'provider')
Press any key to continue....[2024-10-13T12:09:49.700Z] A host error has occurred during startup operation 'f2002cfc-2a84-4fac-8a13-40778b32baf6'.
[2024-10-13T12:09:49.701Z] System.Private.CoreLib: Exception has been thrown by the target of an invocation. Microsoft.Azure.WebJobs.Extensions.Storage.Queues: Could not load file or assembly 'System.Memory.Data, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'. The system cannot find the file specified.

In my case upgrading any of the following packages seems to break. The worst part is that this is considered a patch upgrade, so it was auto-merged by Renovate app to master.

Package
Change

Azure.Storage.Blobs (source)
12.22.1 -> 12.22.2

Azure.Storage.Queues (source)
12.20.0 -> 12.20.1

Microsoft.Azure.WebJobs.Extensions.Storage (source)
5.3.2 -> 5.3.3

Here's an example repository I used for testing: https://github.com/Kiruyuto/.net6-inprocess-break.
All three packages in it are updated to their newest versions. Downgrading or rather not upgrading them appears to resolve the issue.

/unresolve

@mlankamp @Kiruyuto @bvmeer see: https://github.com/Azure/azure-functions-host/issues/10575

/unresolve

Because .NET has chosen not to patch the ToString bug in the 6.x line and we're unable to move to the 8.x, there is friction between our priorities of ensuring that we are using safe dependencies and avoiding platform bugs. To mitigate, we've updated calls to System.Memory.Data internal to the Azure SDK packages to ensure there's no behavioral change.

@jsquire, thanks for the explanation. Azure.Core 1.40.0 is pulled in with Azure.Identity 1.12.0. For us, the System.Memory.Data 6.0.0 is broken, since with it any empty response fails. Sure, we can attemp upgrading it everywhere it gets pulled in like this internal pull request, but it is not a great solution. If we miss it, empty responses are going to fail.
With .NET 6 out of support, is there a path forward for referencing a newer System.Memory.Data version from Azure.Core?

@cataggar:  To clarify, the .NET runtime 6.0 is out of support, but the 6.x line of packages are still actively maintained and supported.  We're currently working to bump the 6.x dependencies to 8.x, but doing so requires internal coordination to ensure we retain compatibility with the Azure Functions host and supported versions of PowerShell.
From context, I'm not sure if you're inquiring about an Azure Functions application (which is the context of this issue) or this is an unrelated question.  If you are asking about a stand-alone scenario, then you can take a direct reference to System.Memory.Data in your application and that version will take precedence over any of your dependencies, hoisting them up.
If you're asking about an Azure Functions application, then you do not have that option and will have to wait for the host dependencies to be confirmed upgraded and fully deployed so that we can safely bump the Azure SDK dependencies.

I'll create a new issue. My appologies for reopening this one.


Class implementing Interface throws error (cannot find error definition)

Hi,
Taking basic example from TypeScript handbook:
interface IClock {
    currentTime: Date;
}

class Clock implements IClock  {
    currentTime: Date;
    constructor(h: number, m: number) {
        return;
    }
}

TSLint throws error : default access modifier on member/method not allowed (member-access)
Screenshot:

I couldn't find what this error means

https://github.com/palantir/tslint#supported-rules

member-access enforces using explicit visibility on class members

You must declare the member as public or private or protected, etc.


Not working on ESP-01S with DOUT flash

After flashing firmware it prints errors on UART. Sketches from Arduino IDE work well after setting flash mode to DOUT.

What does it print?

Now I can't check it, because I flashed NodeMCU, but it depended on tool, which I used to flashing. When I used esptool built in Arduino IDE, It printed something with ets_main.c at end, something simillar to:
` ets Jan  8 2013,rst cause:2, boot mode:(3,6)
load 0x40100000, len 1396, room 16
tail 4
chksum 0xef
load 0x00000000, len 0, room 4
tail 0
chksum 0xef
ho 12 tail 0 room 4
load 0x00000000, len 0, room 12
tail 0
chksum 0xef
csum 0xef
csum err
ets_main.c `
When I used ESP8266 Download Tool, it printed errors all the time. It looked like numbers in registers.

Did you check the sha1 checksums? Try to download the zip and extract the binaries from there. Direct download sometime corrupts them.

Finally it works after flashing using NodeMCU PyFlasher. Other flashing software does not work. I combined all .bin files into one using hex editor, because PyFlasher doesn't have offset settings.


[Rosetta] Test updates required for Crescendo

Identify the updates required to run Rosetta with the crescendo upgrade. Open issues for any larger refactor/updates required.
Update flow-go to the stable cadence branch feature/stable-cadence

Looking over the test setup, please try uncommenting these tests and running it against a localnet
https://github.com/onflow/rosetta/blob/main/integration_test.py#L470-L490
and also running unit tests
go test ./...


Add a request to list all the tests within the current workspace

Fixes #611
rdar://98710526

@swift-ci Please test

A couple things I didn’t consider in the initial draft:

I think I didn’t include transitive subclasses of XCTest
I also included test methods that had parameters
Apparently, things work completely differently on Linux and Windows and subclasses of XCTestCase aren’t actually modeled as such in the index. Still need to investigate this one.

Does this work with swift-testing too?

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test Windows

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test Windows

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test Windows

https://github.com/apple/indexstore-db/pull/175
@swift-ci Please test Windows

@swift-ci Please test Windows


turtle show lfb how to make rem data jsons

see: #877

aye aye

Done; I'm prepared to construct the jsons after lfb obtains a machine's stone cost and the numbers and drop chances of the monsters in it (it's easy for me to do in my text editor so I offered to do it).
I also passed on the clarification note about what to do about monsters like elementals, who were once in the machine but now no longer appear: one big machine with 0 rate for everything.


NPM project Package addition

Initially, as it sits, The project won't start up.
You need to run
   npm i @tiptap/pm

before npm run dev, or it will fail.

thanks for making this issue, I haven't run into this, but I will try a fresh clone of the project and try to reproduce.  Are you on windows? mac?


User groups in access control rules

Introduce the concept of user groups to the SPI,
Add pluggable user group providers to the SPI,
Allow defining access control rules based on user groups.

@electrum If this version is OK, please merge it. I don't have the permissions for that.

Merged, thanks!


Consisten router and remote group

Hi, i was playing with gam little bit to allow routing of remote messages to using any routing strategy. Also included consistent router. Can you have a look if it is something that gam project would benefit from?
see examples/remoterouting/client/main.go how api looks like

I love it!

Merged! thanks


Fix Ruby 3.1 #freeze

Fixes #74.
I played around with the first solution I proposed there, i.e. defining all methods during the initialization by using super, but there are some behaviors of RecursiveOpenStruct that were too specific to make it work, like the preserve_original_keys option. The original implementation simply converts every key to a Symbol.
So I went with the second option: override Object#freeze to create all methods before freezing the object itself.
The gem could also provide a #deep_freeze that would freeze the object, as well as all its sub-elements, recursively.

LGTM, apologies on the delay for getting this reviewed and added.


Better Error messages when exporting to ORCID

Is your feature request related to a problem? Please describe.
I was testing out the export to orcid feature on dev and came across Could not export to ORCID: Not Found a lot. Seems like the orcid sandbox is really flakey and it eventually went completely down later (endpoint started working later once the sandbox was back). The message is confusing and makes it seem like the id of the Dockstore entry/version can't be found.
Describe the solution you'd like
The message should indicate it's a problem with orcid and not Dockstore.
Additional context
Hopefully it is just their sandbox that is really flakey, so this might only be annoying for developers.
┆Issue is synchronized with this Jira Story
┆fixVersions: Dockstore 1.12
┆friendlyId: DOCK-1764
┆sprint: SEAB 79- Gary (the snail)
┆taskType: Story

Could look at new message for review

Tests use hoverfly

Not sure how to reproduce barring spamming sandbox orcid until it goes down. Based on the error message and assuming e.getMeessage() cannot produce "Not Found". It seems:
https://github.com/dockstore/dockstore/blob/82f1dd03440a27610994affd517aa42e9c15981e/dockstore-webservice/src/main/java/io/dockstore/webservice/resources/EntryResource.java#L288
https://github.com/dockstore/dockstore/blob/82f1dd03440a27610994affd517aa42e9c15981e/dockstore-webservice/src/main/java/io/dockstore/webservice/resources/EntryResource.java#L303
are the only two possible sources for that message.
However, those two have been replaced with throw new CustomWebApplicationException("Could not export to ORCID.\n" + response.body(), response.statusCode());
which results in a fairly detailed message like:

seems to be a different issue. this issue is for better error message. that issue (with the infinite progress bar) implies no error message is displayed because the frontend didn't handle it

Sounds hard to reproduce and more likely to affect developers (sandbox) than users. Going to close, please re-open if seen again


How to set the line-height?

There is setting in new version, that's awesome!
But the line height is weird when I change the default font-size in setting.
How do I change the line-height?
Or you can set both of font-size and line-height in font.
Like font:18px/1.5em arial
Thanks.

English: Hi!!! It's true, its so easy, its like configure a text in web, i writed these settings in brackets.min.css, the zoom is quite ugly and hard to read, in order to resolve it, I set two styles properties: font-weight: 100!important; and line-height:1.6, this makes clear and easy to read!!!!
Español: Hola, el zoom de Brackets es muy malo, para resolverlo edité el archivo brackets.min.css, y edité .CodeMirror en la carpeta www/styles con los siguientes estilos, la clave es el peso de la fuente font-weight: 100!important;:
.CodeMirror {
line-height:1.6;
 font-family:"SourceCodePro-Medium","ＭＳ ゴシック","MS Gothic",monospace;//like sublime text
 font-weight: 100!important; //this make the text clear and easy to read!
}

These are the results:

This is easier to read than the brackets zoom default:

@jesuscmd Yup, I saw that in the Brackets post.  I will post back in that thread.  BTW, it would be easier if you had referenced this issue in the other thread instead of copying the content of the message.  That helps because you can navigate between both conversations by clicking on a link :)

Resolve the  problem :
1 -  go to :  AppData\Roaming\Brackets\extensions\user\FolderExtention
2 - Go to newtondark.css for example for me.
3 - add this line :
.CodeMirror {
line-height: 1.6;
}
4 - Be happy !

https://github.com/MiguelCastillo/Brackets-LineStyle


wph_moments_indices

When I run wph_op.wph_moments_indices(), I get a shape that doesn't match the coefficients returns by wph_op.apply().
Is this the desired behaviour?

This discrepancy should coincide with the number of scaling moments estimates ("L" class of moments).
You can check that the combination of wph_op.scaling_moments_indices and wph_op.wph_moments_indices gives you a consistent number of coefficients with what is returned by the apply method.


Add binding necklace to jewelrycount

-Since all charges are same ID, it will display a "?" until your current necklace breaks/or you right click check it

I think this should probably be in the rc plugin since it has such a limited use and also does not behave similarly to the other jewelry. Also doublecheck that the count value isn't already in varps.


Handling of various opcodes that generate message calls.

PR description
The RPC method trace_replayBlockTransactions with vmTrace option should handle correctly various opcodes that generate message calls. Specific operations are performed when a message call is present in a smart contract.
We should add some test cases to cover those calls not only with CALL opcode.
Added test cases:

CALLCODE
DELEGATECALL

Fixed Issue(s)
Jira ticket
Signed-off-by: Abdelhamid Bakhta<EMAIL_ADDRESS>

regarding .../src/test/resources/org/hyperledger/besu/ethereum/api/jsonrpc/trace/chain-data/blocks.bin - is there a JSON file describing how those blocks were generated? Didn't Meredith create a json format for that bin file so it's not a binary blob we have to import now? Binary blobs are very hard to maintain.

Unfortunately test code is still using binary format to setup the blockchain. We update a json file in a directory to reflect the content of the binary file.
https://github.com/hyperledger/besu/blob/abd94b8b05a8d6c8065a9b116c26c76fe996ca94/ethereum/api/src/test/resources/org/hyperledger/besu/ethereum/api/jsonrpc/trace/chain-data/src/blocks.json

In order to not stop progress I've moved my concerns about the .bin file to a separate Jira issue - [BESU-177][https://jira.hyperledger.org/browse/BESU-177]

Ok sounds good.


AJ-1695: (take 1) add public. qualifier to file/relation types

another draft, looking for early feedback. #653 is probably better than this one.

closing in favor of #653


[FEATURE] Update all AD Extension NamedXContentRegistry to SDK

Is your feature request related to a problem?
Because of the wide usage of the NamedXContentRegistry class, it is used in multiple classes and constructors, and changing the type to SDKNamedXContentRegistry and use of getRegistry() in one class results in dozens of other classes being impacted.
What solution would you like?
Make all the changes in a single PR for easier review.  This involves:

Change type globally by appending SDK to the type and changing import
For usage of the variable, add .getRegistry() to final usages in methods (not constructors)
For tests which use the xContentRegistry() default registry in OpenSearchTestCase, mock the registry and return that default via getRegistry()

Fixed in #529 and https://github.com/opensearch-project/anomaly-detection/pull/836


Refactor hardcoded string based animations to enum based #no-public-changes

This is in response to #436
Summary of Pull Request:
I've changed the hardcoded String model to use enums instead.
I've also removed the fatalError() in the opposing property of HollowState
This is a work in progress so let me know what you think.
~I put the #trivial tag in the title for now. I'm not sure if there's a better tag to use.~

          1 Error

:no_entry_sign:
Any changes to library code need a summary in the Changelog.

          1 Warning

:warning:
Consider adding supporting documentation to this change. Documentation can be found in the docs directory.

  Generated by :no_entry_sign: Danger

I'm not sure why this failed the second time.
The following build commands failed:
	CompileSwift normal x86_64 /Users/travis/build/IBAnimatable/IBAnimatable/IBAnimatableApp/TransitionTableViewController.swift
	CompileSwiftSources normal x86_64 com.apple.xcode.tools.swift.compiler
(2 failures)

The command "xcodebuild build -project IBAnimatable.xcodeproj -scheme IBAnimatableApp -sdk iphonesimulator -destination 'platform=iOS Simulator,name=iPhone 6,OS=latest'" exited with 65.
3.08s$ bundle exec danger

⚠ 1 Warning. Don't worry, everything is fixable.
Danger does not have write access to the PR to set a PR status.
Results:
Warnings:
- [ ] Consider adding supporting documentation to this change. Documentation can be found in the `docs` directory.

The command "bundle exec danger" exited with 0.
Done. Your build exited with 1.
/Users/travis/.travis/job_stages: line 150: shell_session_update: command not found

All changes fixed and if the build passes its ready to be merged.
Sorry for forgetting to make multiple commits as I fix the request changes.
I have a habit of trying to limit to commits.

@tbaranes am I supposed to merge this? I don't know if @JakeLin needs to signs off on it first as well. I just don't want to have to rebase this later 😁

@SD10 , Well done on cleaning up the demo app with enums 👍👍👍, that can help us pick up the error easily when we do refactoring.
No need to sign off from me. Merge it when you think it is ready. We all can make the decision.
But for this PR, I have some comments, please have a look. They are not your problem. I was lazy to leave some leftover for the string values. Can you please help us clean them up?

Well done @SD10 merge it when you think it is ready 🎉


[PS4]  [Error] Downloading stash for Synthesis

Any ideas whats going on? Is Procurement support PS4 or not?
DebugInfo.log

It seems that sessionID and Alias are ok. I dont know what I'm doing wrong :/

Ok im dumb... got it - https://github.com/Flenst/Procurement-PS4/releases

@Waciol after #1019 has been released you should be good to go using the normal version.


Dropdown menu - alignment issue when not enough room

Hello,
Consider this dropdown menu example (items copied from the foundation website):
http://codepen.io/anon/pen/jWWBvO
Normally, subitems are left aligned with the parent item. However, when there is not enough room for the subitems to appear (try shrink the window), the box is placed so that its right border touches to left border of the parent item, which gives a somewhat weird result:

I suppose the expected behavior should be right aligned, as indicated in the screenshot?

#7251

These issues were resolved in Foundation 6.2. If I update your CodePen to use the 6.2.0 CSS and JavaScript, the flipped menus align properly.


Hello Backend APP For API

Hello Rails Backend
In this Project, I have done these suggested requirements
Project requirements

Create a new Rails API app called 'hello-rails-back-end'.
Initialize your project with Git.
Make sure that your project has a Postgres database set up.
Create a table for storing your messages and create 5 different greetings.
Create an API endpoint that selects a random greeting from your table (you will need a controller with an action and rails route).

Create a new React app called 'hello-react-front-end'
Initialize your project with Git.
Create the App component with react-router.
Create the Greeting component that will display a greeting. Set it up as a route in your App component.
Create a store, an action and a reducer that will connect to you API endpont to get the random greeting.
Display the random greeting in your Greeting component.

You check Frontend repo here

Thank you @DammyShittu for you golden time.


Update Chroma class with new functionality for collection creation

The Chroma class has been updated with new functionality for creating and retrieving collections. The ensureCollection method now takes a new optional parameter 'newCollection', which allows the caller to specify whether or not to create a new collection. If 'newCollection' is true, a new collection will be created if one with the same name does not already exist. If 'newCollection' is false (or not provided), the method will return without doing anything if the collection already exists.
Additionally, a new method 'getVectorStore' has been added to the Chroma class. This method retrieves the vector store without attempting to create a new collection if one with the same name already exists. This can be useful in cases where the user wants to retrieve an existing vector store, but does not want to create a new one if it does not exist.
The constructor has also been updated to use the new 'ensureCollection' method with the 'newCollection' parameter set to true by default, ensuring that a new collection is created if one does not already exist.
Before:

After:

And it's a small change, but every millisecond counts.

that looks exactly like the PR I need :D

@arajooj thanks for the PR, there is an earlier PR which does the same without changing the API so I'll be closing this one #954

I thank you, my dear. I will continue improving what I have achieved!


⚠ Cioos Pacific CKAN Sitemap has degraded performance

In 91729b7, Cioos Pacific CKAN Sitemap (https://catalogue.cioospacific.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 10332 ms

Resolved: Cioos Pacific CKAN Sitemap performance has improved in de8f27d after 6 minutes.


Change clicking on taskbar icon to open the player window instead of a menu. [OSX]

I keep clicking on it hoping it will open the play window rather than making me click on the icon and then clicking show. after I get it set up I won't need quick access to Settings anymore, and almost never will I need to change the audio output.

I think this is a limitation of OSX, you can't have a context menu and have a separate onclick action.  I will look into it though :+1:

Oops

I tried to do this in #297, was told it's not possible on OSX.
So it's now only for Linux - well, when it's finally merged.
If it turns out to be possible, we should totally have it!

Closing due to #297 merging.
No other app on OSX (that I can think of) activates upon single click of the tray icon.
Skype -> Context menu
System Tools (Wifi, Bluetooth) -> Context Menu
Chrome -> Context Menu
Etc...

The Twitter Mac app has this as an option in the settings to either show the context menu or open the app: https://www.dropbox.com/s/2dvfha2519c278x/Screenshot 2016-03-09 09.06.44.png?dl=0

You can upload screenshots straight to github :+1:
Interesting, but there is no way to have both.  I.e. Left click toggle app, right click show context menu?

There always needs to be a way to get to the context menu and on mac it looks like you can't do both

(I'm a github n00b)
No, it's one or the other. But I feel I would more frequently need to just open the app than the context menu. And everything in the context menu is still accessible via just opening the Settings window from within the app.

Yes, but the reasoning people are giving to make the icon instantly show the app is to make it one click less...
But you think its reasonable to make the user click the icon, click the 3 bars to open the side nav, scroll down to desktop settings, click the desktop settings button, click on the "Audio" tab, click on the dropdown, click on the one they want and then close the desktop settings window.  Instead of just clicking on the icon and choosing from the context menu?
It's a lot to lose for a little to gain.  Also the core "show" button kind of relies on the context menu :+1:

I guess that depends on how frequently people are changing the audio settings. I personally would set it once (or never since the default is correct) and never touch it again.
Also, those settings are quicker accessible by clicking on the App name in the menu bar and clicking "Preferences" or even quicker, but hitting ⌘, which automatically opens that same window.

But the fact that we can't speak for all users and their habits / usage styles (I personally change audio output quite frequently as my headset is USB audio but my speakers are 3.5mm) means that we have to take the best compromise.  Which in this case is the quickest usage for each action.  One extra click for one method (show) and no extra clicks for any other method.
It has the least possible impact on the user regardless of how they use the app

Fair enough. It's your app so you get to be boss. It's still miles better than any other solution I've seen out there. Thanks for the replies! :)


Allow sorting interactions by adviser in endpoint

Description of change
This adds the ability to sort interactions by dit_adviser first_name or last_name in the collection endpoint.
The related test has been changed to make it always deterministic.

Codecov Report

Merging #1071 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           develop   #1071   +/-   ##
=======================================
  Coverage     96.2%   96.2%
=======================================
  Files          217     217
  Lines         6190    6190
  Branches       588     588
=======================================
  Hits          5955    5955
  Misses         166     166
  Partials        69      69

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 1ab4a44...0c711fd. Read the comment docs.


Add MO-Walker and MO-Swimmer

We could easily also add MO versions of the Walker and Swimmer robots, as in: https://openreview.net/forum?id=Ki4ocDm364

(!) Mujoco recently introduced breaking changes so we need to pin to 2.3.3. From >=2.3.4, we might need to move to mujoco v5 which is still unclear for Walker and Hopper from what I understood from the discussions within Farama.


Connection lookup single request

NOTE: Has #582 as a diffbase.
@tseaver This is based on my comment in the other PR. If you disagree, we can torpedo this PR. (FWIW I don't see a reason why we'd want this behavior on the Connection rather than associated with datastore.get, but maybe I'm missing something.)

Coverage remained the same at 100.0% when pulling 3b118d268f5b57602a1b320fc740ee538c5675ba on dhermes:connection-lookup-single-request into d858ff041192657c483e24fb904aade41996bf09 on GoogleCloudPlatform:master.

Coverage remained the same at 100.0% when pulling fd833ce234211cfba30361a0e55ba7616c772bed on dhermes:connection-lookup-single-request into d858ff041192657c483e24fb904aade41996bf09 on GoogleCloudPlatform:master.

@tseaver PTAL. I rebased on top of #582 and folded the list(lookup_response.deferred) part into the original request.

Coverage remained the same at 100.0% when pulling d517a7502e1405cda9fd50ee9e3c75bc813f2ffa on dhermes:connection-lookup-single-request into f854cd1cf9336cfbaf6c2a55041960b81e455d6f on GoogleCloudPlatform:master.

Coverage remained the same at 100.0% when pulling d517a7502e1405cda9fd50ee9e3c75bc813f2ffa on dhermes:connection-lookup-single-request into f854cd1cf9336cfbaf6c2a55041960b81e455d6f on GoogleCloudPlatform:master.

LGTM

Changes Unknown when pulling e9c6260f764e07e12c4cbf5b8999b5dd5959b127 on dhermes:connection-lookup-single-request into ** on GoogleCloudPlatform:master**.


[WIP] Upgrade coverlet

:white_check_mark: Build AElf <IP_ADDRESS>01 completed (commit https://github.com/AElfProject/AElf/commit/417618f52c by @)

close #2526, already change .net core version to 3.1.102


Serialization of None property

Package Name: cosmosdb-preview extension of azure-cli-extension
Package Version: under development
Operating System:
Python Version:

Describe the bug
A clear and concise description of what the bug is.
https://github.com/Azure/azure-cli-extensions/pull/3948#pullrequestreview-808184321
To Reproduce
Steps to reproduce the behavior:

Set AuthenticationMethodLdapProperties of DataCenterResourceProperties to None.
Serialize DataCenterResourceProperties
AuthenticationMethodLdapProperties still gets serialized as {}

Expected behavior
A clear and concise description of what you expected to happen.
I do not expect AuthenticationMethodLdapProperties key to be part of the serialized payload.
Screenshots
If applicable, add screenshots to help explain your problem.
Below is an example payload of DataCenterResourceProperties even when AuthenticationMethodLdapProperties is set to None.
{"properties": {"dataCenterLocation": "westeurope", "delegatedSubnetId": "/subscriptions/dd31ecae-4522-468e-8b27-5befd051dd53/resourceGroups/visunda-az/providers/Microsoft.Network/virtualNetworks/visunda-westeurope/subnets/nova-subnet", "nodeCount": 3, "authenticationMethodLdapProperties": {}}}
Additional context
Add any other context about the problem here.

@vivek-microsoft - is this an issue in Cosmos DB python SDK v4 ?


LED always ON!

Please you can make led to be always ON and when press the button the led blink and stay ON again?

@toshibochan Can you check the latest code to see if this works on your system?

Hi Leland thanks for update! The led work great! But I'm still cant call from iPhone to PI, only from PI to iPhone. Look like the resolution no change. I set to 1080p but look the same. Doorbell sound no play to.
I need to install something to play the sound?

@toshibochan I had a problem with the doorbell sound because pygame would throw an error when I tried to initialize the sound system, only when running and sudo. So I commented that out in the previous check in. However, I just checked this on a fresh install of the new version of Raspbian and it is now working. So that is put back.
However the video resolution is not yet implemented on video calls. It should use default. The RESOLUTION and VIDEORESOLUTION are used for the photos and recorded video, respectively. (I hope to get to this today, but will see.)
Thanks again for your input.

I cannot understand why i cant make calls to the pi. I'm using original Jessie come on sd card with the raspberry pi 3. The exemple on the linphone wiki site work for receive the call but your not. Is strange :-(

@toshibochan I do not yet understand this, either. I am working on making EVERYTHING configurable, via an actual config file(!) so you should only have to configure everything one more time, and after that just update the script and it will work. It was just getting too complicated.
After I finish this task, I'll look into this issue because it is affecting me, too. Meanwhile, if you're feeling adventurous, you could review my commit history on the main file, even pull down previous versions and see if those work for you. If you figure it out, will you please let me know? :)

for have access to .linphonerc need to install liphone desktop version?
i did all the linphone instalation but no .linphonerc
wget https://linphone.org/releases/linphone-python-raspberry/linphone4raspberry-3.9.0-cp27-none-any.whl
sudo apt-get install python-setuptools
sudo easy_install pip
sudo pip install wheel
sudo pip install --upgrade pip
sudo pip install linphone4raspberry-3.9.0-cp27-none-any.whl

@toshibochan .linphonerc is created when you install and run linphonec (command-line) or Linphone (desktop) programs.
You have to view hidden files to see it. So after you run Linphone you need to run:
💻 ls -al ~/
to see the config file (in the default location – your home directory). Or, to refer to it explicitly:
ls -al ~/.linphonerc


JBPM-5056 - Timer boundary events do not support cron expression given as variable, JBPM-5057 - Terminate end node in adhoc subprocess

+1


Based on https://github.com/an-tao/drogon/issues/627#issuecomment-727…

…768460

Based on the following comment I was able to resolve my cmake issues:
https://github.com/an-tao/drogon/issues/627#issuecomment-727768460
I thought adding these comments to the source code would reduce the troubleshooting time for newcomers.

Thanks for your patch


Using readium-js-viewer in asp.net

I was trying to build a sample project using readium js or readium-js-viewer in asp.net  but not able to know what is the function that load epub file. Is there anyone who can help me to know which function  is used to load epub file and what is js file name.
I am using your eadium-js-viewer sample.

Closing as stale.


How can I save/export current state of an paned/zoomed object?

Subject of the issue
I would like to save/export a current state to a json object so that I can apply it to later or to another object.
Could $elem.panzoom('option') work for me (which seems not currently)?
I've tried:
var v = $elem.panzoom('option')
but it returns undefined :(

OK...
So I should compile these two...
STEP 1:
var saved_state = $elem.panzoom("getMatrix");
// => [1, 0, 0, 1, 0, 0]
STEP2:
// Flip the element upside down
$elem.panzoom("setMatrix",  saved_state);
It is little counter-intuitive but still works for me. But these two methods are 'basically' internal as the author indicated in README.

Ahoy,
Also wondering this same thing. Did you get it solved somehow?
So, here's my problem.
I want to save a location on a floor plan with getMatrix and later show it with setMatrix. The problem is that the container sizes vary due to mobile and desktop usage.
So my setMatrix and getMatrix with the same location and zoom level are:
"capture-desktop.jpg"
$('#pohjakuva').panzoom("getMatrix");
Array [ "1", "0", "0", "1", "-222", "107" ]

"capture_mobile.jpg"
$('#pohjakuva').panzoom("getMatrix");
Array [ "1", "0", "0", "1", "-392", "-66" ]


enhancement: expression unify constructor by using List

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 48.22 seconds
stream load tsv:          510 seconds loaded<PHONE_NUMBER>9 Bytes, about 139 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>4 Bytes


Failed to detect Python function

Reporting some friction I encountered adopting this. I went to https://cloud.google.com/functions/docs/quickstart-python
copied this to func.py:
def hello_world(request):
    """Responds to any HTTP request.
    Args:
        request (flask.Request): HTTP request object.
    Returns:
        The response text or any set of values that can be turned into a
        Response object using
        `make_response <http://flask.pocoo.org/docs/1.0/api/#flask.Flask.make_response>`.
    """
    request_json = request.get_json()
    if request.args and 'message' in request.args:
        return request.args.get('message')
    elif request_json and 'message' in request_json:
        return request_json['message']
    else:
        return f'Hello World!'

then created requirements.txt with just Flask in it.
Then set the default builder to google's and ran:
GOOGLE_FUNCTION_TARGET=hello_world pack build gcr.io/ahmetb-demo/myfunc

===> DETECTING
[detector] ERROR: No buildpack groups passed detection.
[detector] ERROR: Please check that you are running against the correct path.
[detector] ERROR: failed to detect: no buildpacks participating
ERROR: failed with status code: 6

I am guessing documentation on how to structure functions is missing from the readme. File name might be the culprit?

Similarly I cloned the Functions Framework Python example
https://github.com/GoogleCloudPlatform/functions-framework-python/tree/master/examples/cloud_run_http and also doesn't work and fails with the same error.

It would be somewhat cool to have --verbose on these steps (maybe by setting GOOGLE_DEBUG=1). This has been my biggest pain point about CNCF Buildpacks (and also why I always kept trying and ran away from it). There's very little visibility.
Anything you can do to answer users’ "but whyyyy? what is it doing under the covers to fail to detect?" can help users tremendously to self-serve.

The first issue here is that you need to pass the environment variable into pack instead -- it doesn't inherit the environment variables from where it's ran:
$ pack build --env GOOGLE_FUNCTION_TARGET=hello_world test
latest: Pulling from buildpacks/builder
Digest: sha256:dc4892caa054a61d26ab4403bb87184285ef094b2763b6563cb69468d8a9cef9
Status: Image is up to date for gcr.io/buildpacks/builder:latest
latest: Pulling from buildpacks/gcp/run
Digest: sha256:3ad752a1ce30d7c343c805b451d52d4fb6d523e35c587c245d6aad0830979979
Status: Image is up to date for gcr.io/buildpacks/gcp/run:latest
===> DETECTING
[detector] 2 of 4 buildpacks participating
[detector] google.python.runtime             0.9.0
[detector] google.python.functions-framework 0.9.0
===> ANALYZING
[analyzer] Restoring metadata for "google.python.runtime:python" from app image
[analyzer] Restoring metadata for "google.python.functions-framework:functions-framework" from app image
===> RESTORING
[restorer] Restoring data for "google.python.runtime:python" from cache
[restorer] Restoring data for "google.python.functions-framework:functions-framework" from cache
===> BUILDING
[builder] === Python - Runtime<EMAIL_ADDRESS>===
[builder] Using latest runtime version: 3.8.2
[builder] === Python - Functions Framework<EMAIL_ADDRESS>===
[builder] Handling functions without dependency on functions-framework.
[builder] Dependencies cache hit, skipping installation.
[builder] ————————————————————————————————————————————————————————————————————————————————
[builder] Running "python3 -m compileall ."
[builder] Listing '.'...
[builder] Compiling './func.py'...
[builder] Done "python3 -m compileall ." (222.0063ms)
===> EXPORTING
[exporter] Reusing layer 'launcher'
[exporter] Adding layer 'google.python.runtime:python'
[exporter] Reusing layer 'google.python.functions-framework:functions-framework'
[exporter] Adding 1/1 app layer(s)
[exporter] Reusing layer 'config'
[exporter] *** Images (b6f3d9da4ee8):
[exporter]       index.docker.io/library/test:latest
[exporter] Adding cache layer 'google.python.runtime:python'
[exporter] Reusing cache layer 'google.python.functions-framework:functions-framework'
Successfully built image test

If you had done that, it would have built an image, but the image would have failed to run:
$ docker run -p8080:8080 test
Traceback (most recent call last):
  File "/layers/google.python.functions-framework/functions-framework/bin/functions-framework", line 10, in <module>
    sys.exit(_cli())
  File "/layers/google.python.functions-framework/functions-framework/click/core.py", line 829, in __call__
    return self.main(*args, **kwargs)
  File "/layers/google.python.functions-framework/functions-framework/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/layers/google.python.functions-framework/functions-framework/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/layers/google.python.functions-framework/functions-framework/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/layers/google.python.functions-framework/functions-framework/functions_framework/_cli.py", line 37, in _cli
    app = create_app(target, source, signature_type)
  File "/layers/google.python.functions-framework/functions-framework/functions_framework/__init__.py", line 130, in create_app
    raise MissingSourceException(
functions_framework.exceptions.MissingSourceException: File /workspace/main.py that is expected to define function doesn't exist

Which should indicate that you need to rename func.py to main.py. (You could also set GOOGLE_FUNCTION_SOURCE=func.py, but I'm not sure that should be exposed here.)
I think at the very least the buildpack should ensure GOOGLE_FUNCTION_SOURCE actually exists.

The first issue here is that you need to pass the environment variable into pack instead -- it doesn't inherit the environment variables from where it's ran:

this was it! Thanks Dustin.

GOOGLE_FUNCTION_SOURCE is not documented fwiw.

It would be somewhat cool to have --verbose on these steps (maybe by setting GOOGLE_DEBUG=1). This has been my biggest pain point about CNCF Buildpacks (and also why I always kept trying and ran away from it). There's very little visibility.

There is in fact a --verbose flag on pack which prints all the detection steps and why something opted in/out. Most of our buildpacks opt-out with a message.


fpga: cva6 fixes

Is this PR still relevant after the merged PR #478 or can we close it?

No I cherry picked the necessary commits into https://github.com/pulp-platform/snitch/pull/480 that is ready to be merged


Pod is not updated for swift 4.0 and higher

There's a pull request in for it if you need something in the meantime https://github.com/teodorpatras/SideMenuController/pull/168

hello buddy I don't have any issue with this project. I have used your JukeBox pod for my application now I need to play the offline downloaded file. but I could not play in your jukebox . please tell me a solution I cannot remove jukebox from my code .

hello its crashing on swift 5 xcode 11 EXEC_BAD_ACCESS here     fileprivate var sbw: UIWindow? {


Google calendar

What steps will reproduce the problem?

Change the email

What is the expected result?
My own email will connect with the extension calendar.
What happens instead?
My other email connected instead.
Screenshot URL:
Logs:
System Information:
(This can be captured automatically if you click on Report Issue from the Chrome extension)

Version:
Language:
Timezone Offset:
User Agent:

[ ] I have read and performed all the steps listed under the Troubleshooting section:
https://github.com/manastungare/google-calendar-crx/wiki/Troubleshooting

This extension pulls information from the account you have set up in Chrome. When you sign out of Google Calendar in Chrome using the Sign Out link, it does not sign you out of Chrome. (I agree this is a bit confusing and complicated).
If you’d like to see Calendar events from a different account, please use a separate profile. Instructions:
https://github.com/manastungare/google-calendar-crx/wiki/Frequently-Asked-Questions#i-have-multiple-google-accounts-how-can-i-see-events-from-calendars-in-both-accounts


wpewebkit, webkitgtk: Bump versions to 2.38.5

wpewebkit, webkitgtk: Bump versions to 2.38.5
Release notes:

https://wpewebkit.org/release/wpewebkit-2.38.5.html
https://webkitgtk.org/2023/02/15/webkitgtk2.38.5-released.html

This upgrade fixes also security issue CVE-2023-23529

The CI seems to be failing on dunfell / langdale branchs fetching woff2
Seems those branches need also patch 998173b2b32985d6bccad4af70892500b34fde49

Pushed to dunfell as 270599a765743c2db5a96195b7f83877cde5d499

Ok, nice.. Now the CI has issues with disk space
From https://github.com/Igalia/meta-webkit/actions/runs/4353737639/jobs/7610269946
Initialising tasks...done.
Sstate summary: Wanted 1821 Found 1801 Missed 20 Current 0 (98% match, 0% complete)
NOTE: Executing Tasks
NOTE: Running setscene task 1 of 1821 (/home/bot/yocto-webkit-dunfell/sources/meta-gstreamer1.0/recipes-devtools/orc/orc_0.4.32.bb:do_package_qa_setscene)
WARNING: The free space of /share/yocto/downloads/dunfell (/dev/mapper/vg-share--yocto) is running low (0.194GB left)
ERROR: No new tasks can be executed since the disk space monitor action is "STOPTASKS"!
WARNING: The free space of /share/yocto/sstate-cache/dunfell (/dev/mapper/vg-share--yocto) is running low (0.194GB left)
ERROR: No new tasks can be executed since the disk space monitor action is "STOPTASKS"!

The CI seems to be failing on dunfell / langdale branchs fetching woff2 Seems those branches need also patch 998173b

The CI seems to be failing on dunfell / langdale branchs fetching woff2 Seems those branches need also patch 998173b

Ok, nice.. Now the CI has issues with disk space From https://github.com/Igalia/meta-webkit/actions/runs/4353737639/jobs/7610269946
Initialising tasks...done.
Sstate summary: Wanted 1821 Found 1801 Missed 20 Current 0 (98% match, 0% complete)
NOTE: Executing Tasks
NOTE: Running setscene task 1 of 1821 (/home/bot/yocto-webkit-dunfell/sources/meta-gstreamer1.0/recipes-devtools/orc/orc_0.4.32.bb:do_package_qa_setscene)
WARNING: The free space of /share/yocto/downloads/dunfell (/dev/mapper/vg-share--yocto) is running low (0.194GB left)
ERROR: No new tasks can be executed since the disk space monitor action is "STOPTASKS"!
WARNING: The free space of /share/yocto/sstate-cache/dunfell (/dev/mapper/vg-share--yocto) is running low (0.194GB left)
ERROR: No new tasks can be executed since the disk space monitor action is "STOPTASKS"!

He he. Bad times for the CI.


need help for 10.0.22000.1

Hello,
my configuration on win 11 shows me not supported.
Can anyone help me ?
Thanks!

https://github.com/stascorp/rdpwrap/issues/1481