[BUG] cargo make test fails out of the box

What is the bug?
Following https://github.com/opensearch-project/opensearch-rs/blob/main/DEVELOPER_GUIDE.md, run cargo make test. The certs tests are failing (container failed to start?).
~/source/opensearch-project/opensearch-rs/dblock-opensearch-rs (test-matrix)$ cargo make test
[cargo-make] INFO - cargo make 0.35.11
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: test
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Running Task: legacy-migration
[cargo-make] INFO - Execute Command: "./.ci/run-opensearch.sh"
INFO: Running opensearch latest
INFO: Creating network search-rest-test if it does not exist already
7061dca675c92cccd6c40bc70b0f4b682766a51182f6ebbf52fb4291240c8647
INFO: Take down node if called twice with the same arguments (DETACH=true) or on seperate terminals
INFO: Removing volume instance-rest-test-data
INFO: Removing volume instance-rest-test-data
instance-rest-test-data
0: 9200 https://localhost:9200
INFO: building opensearch container
cluster is opensearch
Sending build context to Docker daemon  2.719MB
Step 1/6 : ARG STACK_VERSION
Step 2/6 : FROM "opensearchproject/opensearch:$STACK_VERSION"
 ---> 1f7403158bcf
Step 3/6 : ARG SECURE_INTEGRATION
 ---> Using cache
 ---> 97b17916d521
Step 4/6 : ARG opensearch_path=/usr/share/opensearch
 ---> Using cache
 ---> f009b16cbc97
Step 5/6 : ARG opensearch_yml=$opensearch_path/config/opensearch.yml
 ---> Using cache
 ---> 053b2daeb2ea
Step 6/6 : RUN if [ "$SECURE_INTEGRATION" != "true" ] ; then $opensearch_path/bin/opensearch-plugin remove opensearch-security; fi
 ---> Using cache
 ---> ee5c8c68c3b9
Successfully built ee5c8c68c3b9
Successfully tagged opensearch-secure-true:latest
INFO: Starting container instance
+ [[ true == \t\r\u\e ]]
+ healthcmd='curl -vvv -s --insecure -u admin:admin --fail https://localhost:9200/_cluster/health || exit 1'
++ echo curl -vvv -s --insecure -u admin:admin --fail https://localhost:9200/_cluster/health '||' exit 1
+ docker run --name instance --network search-rest-test --env 'ES_JAVA_OPTS=-Xms1g -Xmx1g' --env node.name=instance --env cluster.name=search-rest-test --env cluster.initial_master_nodes=instance --env discovery.seed_hosts=insta
nce --env cluster.routing.allocation.disk.threshold_enabled=false --env bootstrap.memory_lock=true --env node.attr.testattr=test --env path.repo=/tmp --env 'repositories.url.allowed_urls=http://snapshot.test*' --env action.destr
uctive_requires_name=false --env node.name=instance --volume instance-rest-test-data:/usr/share/opensearch/data0 --publish 9200:9200 --ulimit nofile=65536:65536 --ulimit memlock=-1:-1 --detach=true '--health-cmd=curl -vvv -s --i
nsecure -u admin:admin --fail https://localhost:9200/_cluster/health || exit 1' --health-interval=2s --health-retries=20 --health-timeout=2s --rm -d opensearch-secure-true
66342aac22ecedbe3df93b51e3b8cec3e8028dc20e2ea1494984f9ced2dc2612
+ set +x

INFO: waiting for node instance to be up
INFO: Removing network search-rest-test
search-rest-test
SUCCESS: Cleaned up and exiting

ERROR: Failed to start instance in detached mode beyond health checks
ERROR: dumped the docker log before shutting the node down

SUCCESS run-tests
[cargo-make] INFO - Execute Command: "cargo" "test" "-p" "opensearch"
    Finished test [unoptimized + debuginfo] target(s) in 0.10s
     Running unittests src/lib.rs (target/debug/deps/opensearch-9c704994a214f91a)

running 24 tests
test http::request::tests::bytes_body_returns_usable_buf ... ok
test http::request::tests::bytes_body_writes_to_bytes_mut ... ok
test http::request::tests::bytes_slice_body_writes_to_bytes_mut ... ok
test http::request::tests::bodies_into_ndbody_writes_to_bytes ... ok
test http::request::tests::string_body_writes_to_bytes_mut ... ok
test http::request::tests::serialize_into_jsonbody_writes_to_bytes ... ok
test http::request::tests::string_slice_body_writes_to_bytes_mut ... ok
test http::request::tests::vec_body_writes_to_bytes_mut ... ok
test http::response::tests::deserialize_error_string ... ok
test http::response::tests::deserialize_index_not_found_exception ... ok
test http::transport::tests::connection_url_with_no_path_and_trailing_slash ... ok
test http::transport::tests::connection_url_with_no_path_and_no_trailing_slash ... ok
test params::tests::serialize_slices_auto ... ok
test http::transport::tests::connection_url_with_trailing_slash ... ok
test http::transport::tests::connection_url_with_no_trailing_slash ... ok
test http::response::tests::deserialize_illegal_argument_exception ... ok
test params::tests::serialize_slices_count ... ok
test tests::build_search_on_all_indices_and_types ... ok
test tests::build_search_on_selected_indices ... ok
test tests::build_search_on_selected_indices_and_types ... ok
test root::bulk::tests::serialize_bulk_operations_with_different_types_writes_to_bytes ... ok
test tests::percent_encode_characters ... ok
test root::bulk::tests::serialize_bulk_operations_with_same_type_writes_to_bytes ... ok
test http::transport::tests::invalid_pkcs12_cert_credentials ... ok

test result: ok. 24 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

     Running tests/auth.rs (target/debug/deps/auth-9256b3c24e58b89c)

running 3 tests
test basic_auth_header ... ok
test bearer_header ... ok
test api_key_header ... ok

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.30s

     Running tests/cert.rs (target/debug/deps/cert-333d1668bb1e5deb)

running 8 tests
test certificate_certificate_ca_validation ... FAILED
test full_certificate_ca_validation ... FAILED
test none_certificate_validation ... FAILED
test full_certificate_ca_chain_validation ... FAILED
test default_certificate_validation ... FAILED
test fail_certificate_certificate_validation ... FAILED
test certificate_certificate_validation ... FAILED
test full_certificate_validation ... FAILED

failures:
---- certificate_certificate_ca_validation stdout ----
Error: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) }) }
thread 'certificate_certificate_ca_validation' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/082e4ca49770ebc9cb0ee616f3726a67471be8cb/library/test/src/lib.rs:185:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- full_certificate_ca_validation stdout ----
Error: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) }) }
thread 'full_certificate_ca_validation' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/082e4ca49770ebc9cb0ee616f3726a67471be8cb/library/test/src/lib.rs:185:5

---- none_certificate_validation stdout ----
Error: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) }) }
thread 'none_certificate_validation' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/082e4ca49770ebc9cb0ee616f3726a67471be8cb/library/test/src/lib.rs:185:5

---- full_certificate_ca_chain_validation stdout ----
Error: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })) }) }
thread 'full_certificate_ca_chain_validation' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/082e4ca49770ebc9cb0ee616f3726a67471be8cb/library/test/src/lib.rs:185:5

---- default_certificate_validation stdout ----
thread 'default_certificate_validation' panicked at 'Expected error message to contain 'unable to get local issuer certificate' but was 'error sending request for url (http://localhost:9200/): error trying to connect: tcp connect error: Connection refused (os error 111)'', opensearch/tests/cert.rs:67:13

---- fail_certificate_certificate_validation stdout ----
thread 'fail_certificate_certificate_validation' panicked at 'Expected error message to contain 'unable to get local issuer certificate' but was 'error sending request for url (http://localhost:9200/): error trying to connect: tcp connect error: Connection refused (os error 111)'', opensearch/tests/cert.rs:285:13

---- certificate_certificate_validation stdout ----
thread 'certificate_certificate_validation' panicked at 'Expected error message to contain 'unable to get local issuer certificate' but was 'error sending request for url (http://localhost:9200/): error trying to connect: tcp connect error: Connection refused (os error 111)'', opensearch/tests/cert.rs:241:17

---- full_certificate_validation stdout ----
thread 'full_certificate_validation' panicked at 'Expected error message to contain 'unable to get local issuer certificate' but was 'error sending request for url (http://localhost:9200/): error trying to connect: tcp connect error: Connection refused (os error 111)'', opensearch/tests/cert.rs:193:17

failures:
    certificate_certificate_ca_validation
    certificate_certificate_validation
    default_certificate_validation
    fail_certificate_certificate_validation
    full_certificate_ca_chain_validation
    full_certificate_ca_validation
    full_certificate_validation
    none_certificate_validation

test result: FAILED. 0 passed; 8 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.68s

error: test failed, to rerun pass '-p opensearch --test cert'
[cargo-make] ERROR - Error while executing command, exit code: 101
[cargo-make] WARN - Build Failed.

What is the expected behavior?
Tests to pass.

My first problem was that the docker container won't start. To debug:
docker run --name test_1 -d opensearch-secure-true
docker logs test_1

There was an error in there saying my vm.max_map_count was too low.
Ran these from https://github.com/opensearch-project/opensearch-rs/blob/main/.github/workflows/test-integrations.yml#L24.
sudo swapoff -a
sudo sysctl -w vm.swappiness=1
sudo sysctl -w fs.file-max=262144
sudo sysctl -w vm.max_map_count=262144

Tests now try to get the wrong URL, setting OPENSEARCH_URL helps resolve some failures.
OPENSEARCH_URL=https://localhost:9200 cargo make test

running 8 tests
test full_certificate_ca_chain_validation ... FAILED
test certificate_certificate_ca_validation ... FAILED
test full_certificate_ca_validation ... FAILED
test none_certificate_validation ... ok
test fail_certificate_certificate_validation ... ok
test default_certificate_validation ... ok
test certificate_certificate_validation ... ok
test full_certificate_validation ... ok

---- full_certificate_ca_chain_validation stdout ----
Error: Error { kind: Http(reqwest::Error { kind: Request, url: Url { scheme: "https", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("localhost")), port: Some(9200), path: "/", query: None, fragment: None }, source: hyper::Error(Connect, Ssl(Error { code: ErrorCode(1), cause: Some(Ssl(ErrorStack([Error { code: 337047686, library: "SSL routines", function: "tls_process_server_certificate", reason: "certificate verify failed", file: "../ssl/statem/statem_clnt.c", line: 1913 }]))) }, X509VerifyResult { code: 20, error: "unable to get local issuer certificate" })) }) }
thread 'full_certificate_ca_chain_validation' panicked at 'assertion failed: `(left == right)`
  left: `1`,
 right: `0`: the test returned a termination value with a non-zero status code (1) which indicates a failure', /rustc/082e4ca49770ebc9cb0ee616f3726a67471be8cb/library/test/src/lib.rs:185:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

running 8 tests
test certificate_certificate_ca_validation ... ok
test full_certificate_ca_chain_validation ... ok
test full_certificate_ca_validation ... ok
test default_certificate_validation ... ok
test fail_certificate_certificate_validation ... ok
test none_certificate_validation ... ok
test certificate_certificate_validation ... FAILED
test full_certificate_validation ... FAILED

Interesting, I was able some of yours, but failed others.

@dz-s I think this is something about cert setup out of the box, needs debugging

This is because by default tests try to connect to a live endpoint.


Scala Common Enrich: add validation that currency fields are ISO 4217

Using http://www.joda.org/joda-money/

Migrated to https://github.com/snowplow/enrich/issues/189


kaizen: clean up state finite automata

addresses #197
This gets rid of the DFA/NFA distinction, everything is now an NFA. The stretch goal is to fix #188, but the messy sprawl of DFA-and-NFA was getting in the way of understanding.
This includes a bunch of commented-out code for pretty-printing state machines, which I found totally essential in migrating to all-NFA.  I'm going to create an issue to bring this code out of comments and make it possible to switch it off/on to facilitate the creating and merging different kinds of patterns.

I see that I accidentally lost a couple of unit tests, although the coverage didn't go down.  Will clean this up subsequently.  There's a huge amount of quite difficult work here that I want to get checked in so I have a stable base to continue work on #188 and then some new features.

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

Project coverage is 96.32%. Comparing base (a716bc1) to head (8351137).
Report is 68 commits behind head on main.

Files
Patch %
Lines

nfa.go
94.82%
1 Missing and 2 partials :warning:

shell_style.go
95.45%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #308      +/-   ##
==========================================
+ Coverage   96.19%   96.32%   +0.13%
==========================================
  Files          17       17
  Lines        2129     1634     -495
==========================================
- Hits         2048     1574     -474
+ Misses         54       34      -20
+ Partials       27       26       -1

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

Having thought about it, I can see why this is a difficult PR to review.  It's all down in the finite-automaton weeds and not that many people care about this stuff or have exposure to it.    So I will probably break the rules and go ahead and merge if nobody is up to doing a real review in the next day or so.

+1


decomp some small files, do some cleanup

this pr is a bit of a random combination of various things i've been working on/cleaning up idly by looking at the smallest remaining .s files

@Pheenoh fixed merge, sorry for the misc PR with lots of conflict points 😅

@Pheenoh fixed merge, sorry for the misc PR with lots of conflict points 😅


Question: how to generate and use AuthorizedUser wrt CreateAuthorizedUserCredentialsFromJsonFilePath

Answering these questions before submitting your bug report will help us give
you a quicker answer.  Thank you!
If one or more of these questions are not applicable, feel free to remove them.
Does this issue affect the google-cloud-cpp project?
Yes, stuck at using the API CreateAuthorizedUserCredentialsFromJsonContents or CreateAuthorizedUserCredentialsFromJsonFilePath
What component of google-cloud-cpp is this related to?

Cloud Storage.

What version of google-cloud-cpp are you using?
latest HEAD git rev-parse HEAD
66cb4f9f5b4b6f18ba222cd44f9287ff25571260
What compiler and version are you using?
gcc-5.3.1
Please include the output of g++ -v or the equivalent command-line flag.
Using built-in specs.
COLLECT_GCC=gcc-5
COLLECT_LTO_WRAPPER=/usr/lib64/gcc/x86_64-suse-linux/5/lto-wrapper
Target: x86_64-suse-linux
Configured with: ../configure --prefix=/usr --infodir=/usr/share/info --mandir=/usr/share/man --libdir=/usr/lib64 --libexecdir=/usr/lib64 --enable-languages=c,c++,fortran,ada,go --enable-checking=release --with-gxx-include-dir=/usr/include/c++/5 --enable-ssp --disable-libssp --disable-libvtv --enable-libmpx --disable-plugin --with-bugurl=http://bugs.opensuse.org/ --with-pkgversion='SUSE Linux' --disable-libgcj --with-slibdir=/lib64 --with-system-zlib --enable-__cxa_atexit --enable-libstdcxx-allocator=new --disable-libstdcxx-pch --with-default-libstdcxx-abi=gcc4-compatible --enable-version-specific-runtime-libs --enable-linker-build-id --enable-linux-futex --program-suffix=-5 --without-system-libunwind --enable-multilib --with-arch-32=x86-64 --with-tune=generic --build=x86_64-suse-linux --host=x86_64-suse-linux
Thread model: posix
gcc version 5.3.1 20160301 [gcc-5-branch revision 233849] (SUSE Linux)
What operating system and version are you using?
SLES12SP3
If you are using a Linux distribution include the name and version of the
distribution too.
What were you trying to do?
I am trying to write up an app using End User authentication scheme only. So I created a bucket under the user's storage account and  I generated a OAuth Client ID and Key under Cloud Console -> APIs & Services -> Cloud Storage with type "Other".  Then I added that OAuth Client ID as Members for that bucket and authorized it to be "Storage Admin".
Within my google-cloud-cpp application,  I've constructed a json contents string with client_id, client_secret as follows (XXXX and YYY are actual strings from the OAuth output json file):
std::string const AUTHORIZED_USER_VALID2_CRED_CONTENTS = R"""({
"client_id": ".apps.googleusercontent.com",
"client_secret": "",
"refresh_token": "https://oauth2.googleapis.com/token",
"type": "authorized_user"
})""";
Then I passed it to
auto creds_json = gcs::oauth2::CreateAuthorizedUserCredentialsFromJsonContents(AUTHORIZED_USER_VALID_CRED_CONTENTS);
Client client(*creds_json);
What did you expect to see?
I expect the client to be constructed and list-objects, read-object operation should return OK
What's the proper way to write an app with only End User authentication?  And how do you actually configure an end user to have access to a bucket and also obtain the JSON file containing the correct credentials I could use to call CreateAuthorizedUserCredentialsFromJsonFilePath with?
What did you see instead?
Instead I see the following errors:
Storage Client Credential creation, status= [OK]
Standard C++ exception raised: Permanent error in ListObjects: {
"error": "invalid_grant",
"error_description": "Bad Request"
} [INVALID_ARGUMENT]
Anything else you would like us to know?
Using a service account Json file, it seems to be working.
Include here information about your environment that is not captured above, or
any other information you think might be relevant.

What were you trying to do?
I am trying to write up an app using End User authentication scheme only.

Since I am no expert in OAuth 2.0 let me make sure I understand what that means: you have an application that needs to access GCS on behalf of some user, and therefore you want the application to use an access token for that user when talking to GCS.  Is that accurate?

So I created a bucket under the user's storage account and I generated a OAuth Client ID and Key under Cloud Console -> APIs & Services -> Cloud Storage with type "Other". Then I added that OAuth Client ID as Members for that bucket and authorized it to be "Storage Admin".

I am not sure I follow:

Did you add those members as what?  serviceAccount:blah-blah ? user:blah-blah?
If you are going to authenticate as the end-user, why grant your "application" (or Client ID) access to the Bucket?

Within my google-cloud-cpp application, I've constructed a json contents string with client_id, client_secret as follows (XXXX and YYY are actual strings from the OAuth output json file):
std::string const AUTHORIZED_USER_VALID2_CRED_CONTENTS = R"""({
"client_id": ".apps.googleusercontent.com",
"client_secret": "",
"refresh_token": "https://oauth2.googleapis.com/token",

This should be an actual token that you obtained via the OAuth flow.

"type": "authorized_user"
})""";

I do not see XXXX or YYYY here, presumably those are (part of) the client_id and client_secret values?

What did you expect to see?
I expect the client to be constructed and list-objects, read-object operation should return OK

That seems reasonable.

What's the proper way to write an app with only End User authentication?

AFAIK, you just create the JSON file (or contents) with the secret and refresh token from the user and you are good to go.  It is different if you want to act on behalf of the user but using your client id. Then you need to go through the OAuth flow to get an access token given your client ID and then end-user authorization code. And then present that access token to GCS.

And how do you actually configure an end user to have access to a bucket and also obtain the JSON file containing the correct credentials I could use to call CreateAuthorizedUserCredentialsFromJsonFilePath with?

If you are asking about how to get a refresh token and secret for a user that is a bit beyond the scope of the GCS client library.  You need to follow one of the OAuth flows for that.

What did you see instead?
Instead I see the following errors:
Storage Client Credential creation, status= [OK]
Standard C++ exception raised: Permanent error in ListObjects: {
"error": "invalid_grant",
"error_description": "Bad Request"
} [INVALID_ARGUMENT]

Ack.  That looks like the OAuth server rejecting the request to convert what (should be) a refresh token into an access token.

Anything else you would like us to know?
Using a service account Json file, it seems to be working.

Okay, that is good news.
Have you tried to use this with your own account?  For example, use gcloud auth application-default login will create a valid authorized user credentials file in $HOME/.config/gcloud/application_default_credentials.json.  I have good reason to believe it works, but I use these all the time.
It is possible that this is not what you need, but maybe it can help us clarify any misconceptions I may have.

the "gcloud auth application-default login" trick did the work!! Thanks.


Waqadahmad

Coverage decreased (-0.05%) to 56.099% when pulling bb461b0a7ca0701d3aee90df595a3d71ac680d1c on waqadahmad:patch-1 into 47436f5e77652282b8c947fc46338a39841ead13 on vmware:pr/3775.


Add more events to CollectionView items: long tap, swipe right

Hi!
It would be great to add such events to CollectionView: long tap for opening context menu, swipe right fo removing item from list, for example

@sand123 you can already do this with the current API. Below is my code for your reference. In both cases, "composite" is a Composite widget appended in a Cell of the CollectionView. You can also refer to the documentation on Touch Events (https://tabrisjs.com/documentation/latest/touch) for a list of available events you can use to implement what you need.
Longpress
composite.on("longpress", function(widget, gesture) {
    if (gesture.state === "start") {
        // do what you want here
    }
});

Slide to Delete
This allows you slide left or right. If you slide more than 100 pixels, it will slide the composite out of the screen and show a confirm or undo button. Anything less than 100 pixels, it will revert back to its original position.
composite.on("pan:horizontal", function(this, gesture) { // Remove item
    this.set("transform", {translationX:gesture.translation.x});
    if (gesture.state === "end") {
        if (Math.abs(gesture.translation.x) > 100) {
            this.animate({transform:{translationX:(gesture.translation.x<0 ? -1 : 1)*1000}},{duration:1000});
            var confirmationComposite = tabris.create("Composite", {
                layoutData: {centerY:0, centerX:0}
            }).appendTo(this.parent());
            tabris.create("Button", {
                layoutData: {centerY:0, left:confirmationComposite.children().last()},
                text: "Remove"
            }).on("select", function() {
                // do what you want here
            }).appendTo(confirmationComposite);
            tabris.create("Button", {
                layoutData: {centerY:0, left:confirmationComposite.children().last()},
                text: "Undo"
            }).on("select", function() {
                this.animate({transform:{translationX:0}},{duration:1000});
            }).appendTo(confirmationComposite);
        } else {
            this.animate({transform:{translationX:0}},{duration:1000});
        }
    }
});

@wilsonna Ok, as a workaround. But

for the performance reasons, I think it's better to delegate items events (just like select) to CollectionView itself, rather then bind same listeners to every element.
for simple list, I don't want to wrap cell content into Composite just to have these events fired - they should not depend on cell template

@sand123 I don't think that @wilsonna's solutions are workarounds, that's the recommended way to attach listeners, see the swipe-to-dismiss example.

Why do you think that delegating cell events to the CollectionView could improve performance? Cell is the widget that actually receives the event.
You can add the listener to the Cell itself, there's no need for wrapping.

If this API doesn't make sense for you, could you draft an example snippet of what you'd have expected?

@sand123 for the sliding event, we may need to have different implementations under different scenarios. For example, instead of sliding to delete, I may instead wish to slide to reveal more information underneath the composite. Furthermore, even for delete, what we want actually do when "slide-out" also varies based on our needs.
What I did to cut down on my boiler plate code is to create a function (Deleter.register) which encompasses the code I wrote above. I then use it to register a "deleter", with a callback function. e.g.
var collectionView = tabris.create("CollectionView", {
	layoutData: {top:headerComposite, bottom:0, left:0, right:0},
	refreshEnabled: true,
	itemHeight: Global.itemHeight,
	initializeCell: function(cell) {
		var composite = tabris.create("Composite", {
			layoutData: {top:0, bottom:0, left:0, right:0}
		});

		Deleter.register(composite, function(widget) {
			var cell = widget.parent();
			var session = cell.get("item");
			Service.deleteSession(session.id, function() {
				console.log("Session " + session.id + " deleted");
				collectionView.remove(cell.get("itemIndex"));
			});
		});
                    ....
	}
});

This way, I can reuse my Deleter under various scenarios and implementations

Regarding @ralfstx second point about attaching events to the Cell instead of a wrapper (i.e. another composite), I actually did it for a not-so-significant reason. It's because I've also appended another composite (not shown in my code above) to the Cell to serve as a row divider. If I were to attach the "pan" event to the Cell, sliding the cell would also slide the divider out of the screen. As such, I had to create a wrapper so that only the wrapper would slide, but the divider would stay. :smile:

OK, thanks for clarification. I thought that longpress and swipe should also be fired by CollectionView just like select - it seems to be more consistent API. Anyway, cell inherited events are enough


Add prismaClientModule option to config

📝 Description
In my product, I changed the output option of the generator client config in my schema.prisma to output = "./path/to/client". However, in factories.ts, the import path for Prisma and PrismaClient is still "@prisma/client", and it can't find the module. It should locate my ./path/to/client module. I resolved this issue by adding the prismaClientModule config to the generator factory config.
Thank you for the great library!
🔗 Links
na
✅ Checklist

[x] Run tests and linters.
[ ] Add a changeset if it is required.

I will add a changeset


Navbar Fixing

Is there an existing issue for this?

[X] I have searched the existing issues

What went down?

Need of proper allignment of navbar.
Record

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

@Apurv7777 It helps me more to assign you, when you share the screenshot of the result.

I hope you're doing great! As a GSSoC contributor, I'm keen on ramping up my involvement for GSSoC'24. Could you please assign this issue to me? I'm excited to continue contributing to the project. Thanks a bunch!


Config parser improvements

Free memory properly even during errors.
Extract the gigantic parser function.
Re-use memory where possible.
Better error handling.
Cleaner and more readable code around memory management and list stuff.
Make it easier to extend.

Also see the #45 for splitting the parsing and scanning stages.

This will need extensive testing.


Updated transaction application model: staged ledger diff application and witness generation

Part of https://github.com/MinaProtocol/mina/pull/11760.

merged into https://github.com/MinaProtocol/mina/pull/12548


SpinLock.TryEnter fail fast for timeout 0

Port of https://github.com/dotnet/coreclr/pull/6952
Resolves https://github.com/dotnet/corert/issues/2026

Thanks!


Read and execute commands

Created a simple, fast execution loop for demonstration purposes. Can be expanded on as additional features are added. Currently accepts a shortened/simplified version of the Test config file and executes the steps provided using the chrome browser.

For v. 0.1.0 pytest will be removed and replaced with custom test discovery and execution loop for tests/test cases


Allow specifying HTTP verbs to expose

I have a need for exposing a PUT-only function URL. This PR does a light refactor to make it possible to explicitly set the HTTP verbs for function urls.

Thanks for raising this PR. The code change look fine, so give me a moment to merge it and release a new version of it.
I didn't see the need to set a CI for it, but if more PRs are coming, I may need to.

@decompil3d you changes are available in v1.0.3


Fixed response continuation token limit get and set APIs

Fixes this issue : https://github.com/Azure/azure-sdk-for-java/issues/12115

Are there central SDK guidelines on measures KB vs Kb?

is it KB or Kb?
now that we are fixing this, please check that KB vs Kb, we should be consistent with other places (if any).

Checked everywhere else in the code, we don't expose anything like this anywhere else.
So I think we can change it however we want.
I am fine with either of those.
@kirankumarkolli - SDK central team didn't raise any issues with this API in the api review phase. So I am guessing they are fine with Kb


5 months ago this brilliance was created - can you tell us all how to make colored boxes responsive?

Flexbox in the middle part would be great (the colored boxes that open/animate section) -- really interested in flexbox
PROBLEM: when the browser is made small the boxes all cram together.
Question: Is there a way to have them align all in a row when browser LARGE -- ie. setting a width i guess?
SO: then when the browser is small they would be one above the other.
Can you help us all with this? Grid needed i think.
whatever happens great thanks!
Don

This is unrelated to the library itself, though here is what you can do:
When browser width goes below your threshold width, make all boxes -> width: 100%


Need copy to clipboard button on get started page

How would this new feature help increase access to p5.js?
Most appropriate sub-area of p5.js?

[x] Accessibility (Web Accessibility)
[ ] Build tools and processes
[ ] Color
[ ] Core/Environment/Rendering
[ ] Data
[ ] DOM
[ ] Events
[ ] Friendly error system
[ ] Image
[ ] IO (Input/Output)
[ ] Localization
[ ] Math
[ ] Unit Testing
[ ] Typography
[ ] Utilities
[ ] WebGL
[ ] Other (specify if possible)

New feature details:
Need copy to clipboard button for copying snippets and CDN on get started page

@limzykenneth @lmccart  Is this issue open? If possible I would like to work on this

Yes, it's open!


Charactor's Equipment

I modify the folder of the name of value, and replace the Character to Player

It is in order to writing tools system


Bad type inference for polymorphic records

I'm noticing a trend where parameterized records usually can't infer the proper label. Using field @"label" works fine consistently. I need to find some reproducible cases so I'll report back as I find them.

Turns out I had merely forgotten to import the instances 😊


Discovery through hotspot

I'm having trouble discovering zeroconf services that hosts connected to the iPad using its hotspot are serving.
Here's the scenario:

iPad is in hotspot mode
device with zeroconf server is connected to the iPad personal hotspot using wifi (to simplify testing i've been using xamarin's build host as the zeroconf server since our own host requires a separate piece of hardware)
a call is made to BrowseDomainsAsync but always shows up empty

I tested the same code with both the iPad and the device connected to the same wifi network and had no trouble discovering the service being offered, but if I go into hotspot mode it doesn't appear to be broadcasting the request to the internal wifi network, just the cellular network, so my app never finds the hosted service.
Any clue on this?  Would you expect this to work?  I briefly read through the code and it appears you're broadcasting such that it should go to every network.  Maybe a modification can be made (on my end if necessary) to allow me to specify the netmask or some portion of the IP range to broadcast to (e.g. 172.20.10.*)?

Too add some info to this I wrote an iOS app that pumped out some information from the GetAllNetworkInterfaces() call and when there is a personal hotspot enabled the bridge100 interface shows up and appears to be the internal network interface (with IP <IP_ADDRESS>).
Maybe I end up implementing "GetBestInterface" for iOS and pick that interface if it exists, otherwise pick the default?  I'm not sure if the index you get back from the interfaces call matches what you're passing in to the SetSocketOption for MulticastInterface or if I'd have to translate the index somehow.  You're just giving it zero and I don't get zero in the interfaces call so I assume zero tells it to pick a default or the "best" as far as it knows, or there is a mismatch with the indices.

Honestly, I'm not sure how iOS/mono picks the best interface in that scenario. It is using a multicast transmit/receive, so it does have to bind to a specified interface; it doesn't use all of them at once.
If you write code to do what you need, I'd be happy to take a PR.

We ended up just creating an interface for interaction with Zeroconf and implementing an iOS specific class using the iOS bonjour libraries.
On Jun 8, 2015 7:25 AM, Oren Novotny<EMAIL_ADDRESS>wrote:
Honestly, I'm not sure how iOS/mono picks the best interface in that scenario. It is using a multicast transmit/receive, so it does have to bind to a specified interface; it doesn't use all of them at once.
If you write code to do what you need, I'd be happy to take a PR.
—
Reply to this email directly or view it on GitHubhttps://github.com/onovotny/Zeroconf/issues/11#issuecomment-109976254.

Can you please retry using the latest versions? There were some changes to the code to send over every interface and this may be resolved.

Closing for now as we should be scanning all interfaces now. Please re-open if this is still an issue.

I have noticed this issue as well.   My setup is similar to what the original post identified.  I am using the personal hotspot on iphone XS with iOS 13.  Code written with Xamarin.iOS.  Everything works great under other circumstances (different wifi network/host).  I'm new to github, so I'm not sure if this is how I should report this or not.  I know this is an old thread and that is somewhat frowned upon.


Add missing BuiltinType::getType() overloads

Some types are not supported for extraction.
BuiltinType is missing two getType() functions, namely for BYTE and CHAR which currently resolve to OTHER (as they are not overloaded like the other types) which prevents the correct type comparison in Variant::extract().
This PR adds those overloads. However, some stuff might be debatable:

I overloaded BYTE using unsigned char. Another possibility is using the C++17 std::byte class which is not as backwards compatible but semantically nicer.
CHAR is overloaded using char. This makes semantic sense but has one quirk: char can be a signed char or unsigned char (the same type as BYTE, although distinct in the sense that it does not break the template).
I cannot think of when it could be problematic but there may be cases.

Thank you, I don't know how I missed them ;)

Why you did this to me? :(
How could you?
http://build.ros.org/job/Ldev__ros_type_introspection__ubuntu_xenial_amd64/41/changes

Oh man, I am so sorry! :(
I wrote this from a Windows machine on which I could not build.
Because it was so simple, I would not have imagined it would break the build.
What could be the reason?

Ah I see, the types point to the same stuff.

No problem, I was kidding, I will fix it later.
Cheers

Thanks, that's a bit reassuring, but, I still trashed your master... :)
Because I feel very bad, I did remove all occurrences of BYTE in this commit.
Although I understand if you cannot trust me anymore. ☺


Translate interface to Poland

[ ] Translate all the strings

@juzraai is there a translation file?

@KrzysztofMadejski of course, here: https://github.com/petabyte-research/redflags/blob/master/redflags-web/src/main/resources/ - see messages_LANG.properties files.
I've just written a cookbook chapter for adding a new language:
http://docs.redflags.eu/developer/cookbook/#adding-a-new-language-to-the-webapp

I'm wondering about When you are sure you won't need for example IndicatorX gear, you don't have to translate and can remove all message lines which contain IndicatorX. and the fact that the message keys don't have language key.
I foresee that we may reuse the code of your indicators in Poland (and name them the same), because they will have very simillar functionality, but there may be differences in legal framework, thus a bit different name will be needed.
Then message keys should be prefixed with the language, ie. flag.hu.CountOfInvOpsNoCondIndicator.desc and then we may reuse it in flag.pl.CountOfInvOpsNoCondIndicator.desc or skip it in the translation files..
@juzraai What do you think?

@KrzysztofMadejski  Well, I agree.

This way language files can be extended instead of losing information (overwriting/deleting lines of current Hungarian indicators).
Though the possibility of differencing indicators by language/country is already existing (classname), it would be more rational to difference them inside language files too (and istead of in the classname), as they are also separated in the package structure (...gear.indicator.hu).

These message keys are generated and used automatically at many points inside the system, so I have to think over exactly where/how do I have to refactor the code. Then the documentation must be updated.
And also I will have to introduce this change to the current live system behind http://redflags.hu/ .

Okay, I've refactored the code and updated the docs as well. The most important part:
The indicator identifier is generated from the name of the direct parent package and the simple class name, e.g. hu.petabyte.redflags.engine.gear.indicator.pl.CountOfInvOpsNoCondIndicator will have pl.CountOfInvOpsNoCondIndicator as its id, so its message keys will have the prefix flag.pl.CountOfInvOpsNoCondIndicator.*.

Amazing!

Missing translations:

[ ] With the financial support of the Prevention of and Fight against Crime Programme
European Commission - Directorate-General Home Affairs
[ ] Read the summary of the project!
[ ] This work is licensed under a Creative Commons Attribution-NonCommercial-ShareAlike 4.0 International License.

Other:

[ ] notice.tedlink - rename to "check in source"

https://github.com/epforgpl/redflags/commit/499b231fc311b0e4e492f302b5f86b36c3b027c2

@juzraai FYI I've changed notice.tedlink to notice.checkSource so it will be valid for all notice sources. -> https://github.com/epforgpl/redflags/commit/90b20f46090dd7073d976a4b4a77ef740085a9b1


Support of Multiple Function Location Jars

We have a use case where Indivisual functions are developed in their seperate Jars due to Isolation reasons and we want to deploy these multiple functions combined in a single deployer.
Single Deployer should maintain child application contexts and loads the corresponding function jar.
Ex-
spring.cloud.function.location=target/upper-1.0.0.jar;target/lower-1.0.0.jar
spring.cloud.function.definition=upperFunction;lowerFunction

Similar Issue mentioned in
https://github.com/spring-cloud/spring-cloud-function/issues/564

We have the same use case. Is there any plan to add this feature?

We have spring-cloud-function-deployer module that doers exactly what you are asking for and in fact was designed for that. Here is the test from it https://github.com/spring-cloud/spring-cloud-function/blob/main/spring-cloud-function-deployer/src/test/java/org/springframework/cloud/function/deployer/FunctionDeployerTests.java.
Also, here are the docs for it https://docs.spring.io/spring-cloud-function/reference/spring-cloud-function/deploying-a-packaged.html

Is this about allowing multiple locations instead of a single JAR?

Any updates on this?


About the rotation in the inspector

How does the rotation work?
Description

What unit does the object inspector use? Is it rad or deg?  or pure quaterion ?
For sanity sake i think that it should be in deg.

Version

0.97.10

Platform

Windows [X]

Hello
Internally quaternions are used but in the editor a euler rotation is presented is can be used in rad or degree it is configurable in the settings menu.
By default radians are used.

Thank You, didnt noticed that option .

No problem here to help. Thanks


get_class() expects parameter 1 to be object, array given getting this error while trying to insert data in product table

What were you using for a client?

I'm sending data using post request is there any particular formate in which we have to pass data for insert

I am posting on route http://<IP_ADDRESS>:8000/odata.svc/Products using Post method
@c-harris i am sending data in JSON format

@CyberiaResurrection we should probably get cracking on json testing and checking.
@dsv4890 give us a little bit to track this one down

ok

@dsv4890 , thanks for the reply.  I strongly suspect the cause (see @c-harris comment above), but if you can, could you try submitting your post request in atom format, rather than json?

can you please provide any demo for that?

postman

Can you please provide any kind of example or sample data for post request!

It looks like you might have tripped over a difference in JMS serialiser versions - to check, you're running minimum-stability: dev on both your test projects?
I've pushed a fix to the odata-metadata project - can you please composer update and see what pain persists?

ok

I have run composer update command but still getting the same error
It would be good if you can send me demo data so i can verify I'm inserting data in proper format

OK, here's the best I can find on line (from the OData v3 dox - http://www.odata.org/documentation/odata-version-3-0/atom-format/ , section 6.1) for atom format:
<entry>
  <id>http://services.odata.org/OData/OData.svc/Products(0)</id>
  <title />
  <summary />
  <updated>2012-03-30T07:11:05Z</updated>
  <author>
    <name />
  </author>
  <link rel="edit" title="Product" href="Products(0)" />
  <link rel="http://schemas.microsoft.com/ado/2007/08/dataservices/related/Category" type="application/atom+xml;type=entry" title="Category" href="Products(0)/Category" />
  <link rel="http://schemas.microsoft.com/ado/2007/08/dataservices/related/Supplier" type="application/atom+xml;type=entry" title="Supplier" href="Products(0)/Supplier" />
  <category term="ODataDemo.Product" scheme="http://schemas.microsoft.com/ado/2007/08/dataservices/scheme" />
  <content type="application/xml">
    <metadata:properties>
      <data:ID m:type="Edm.Int32">0</data:ID>
      <data:Name>Bread</data:Name>
      <data:Description>Whole grain bread</data:Description>
      <data:ReleaseDate metadata:type="Edm.DateTime">1992-01-01T00:00:00</data:ReleaseDate>
      <data:DiscontinuedDate metadata:type="Edm.DateTime" metadata:null="true" />
      <data:Rating metadata:type="Edm.Int32">4</data:Rating>
      <data:Price metadata:type="Edm.Decimal">2.5</data:Price>
    </metadata:properties>
  </content>
</entry>

sending data in the same format i'm getting following error
assert (): Property content must be instanceof ODataPropertyContent. failed

Is crud operation not possible using this package?

Hi
I'm waiting for a response on my issue about POST request
error  get_class() expects parameter 1 to be object, array given

@dsv4890 I am very sorry this has taken so long to resolve. sadly my own life kind of bulldozed my open source commitments of late. I underwent treatment for cancer this year and the ongoing side effects have been greatly slowing down all my development work. however, I am investigating your issue today. please expect a reply in the next 7 days (upper bound, could be sooner) with a solution.

ok Thank you


Add documentation for default use cases

Description
https://github.com/opensearch-project/flow-framework/issues/496 added default use cases to the flow framework plugin through substitution, this is a follow up for our substitution param PR we had.
nit: currently links in default link are all set to same link, will update to correct links but rest is ready for review
Checklist

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

Thank you for providing the PR, @amitgalitz! After you are done with the tech review, please let me know and I will do a doc review on the PR. Thanks!

Linking https://github.com/opensearch-project/flow-framework/issues/541 to this PR.

For the table headers I made the fix required by style check but style check is persisting there. Also for the openai comment, this is already set to openai in code for use_case so can't be changed to OpenAi. Also all use cases should be all lower case.


Error in processing timeseries data in your pipeline

I was trying to run your main.ipynb file using Quarto. I encountered the following error, probably because one of the pages have changed since you ran your pipeline. It comes from this cell:
header, data = scraping.get_results("https://bases.athle.fr/asp.net/liste.aspx?frmbase=resultats&frmmode=1&frmespace=0&frmcompetition=282742", 13)

Looking at utils/scraping.py, I found the error comes from there:
data["hours"] = data["Chrono"].dt.hour
data["minutes"] = data["Chrono"].dt.minute
data["seconds"] = data["Chrono"].dt.second

Which I  identified as coming from the first row of your dataset where html parsing did not end well:

Is it possible for you to fix that problem ?

The results page has indeed changed, with the organizers adding a new line for performance points counts. I hadn't seen this in previous pages, which might be because this race is international. As a result, the current scraping script isn't robust enough to handle these changes.

I am working on it to fix it

If you have a page that you know is working well, it is ok to me. I know webscraping can be unstable and its hard to prevent problems in such pipeline.
What I want to do is see your quarto output, which should work fine if you propose me a page that is not generating this bug

The provided result page should now work properly. This update also covers standard events such as 5K, 10K, half marathons, and marathons. However, for more complex meetings that include multiple competitions race formats and types like throws and hurdles, there may still be some issues with robustness.

Thanks for the quick action, I have been able to reproduce your app. You can close the issue


feat(timeout): add job elapsed time

This commit adds support to set a job's elapsed time. This may be used to assert if a job got executed within some desired time interval.
An experiment has been added that verifies if 50 config maps get created by a job and all of this should happen within 10 seconds.
In addition, several fixes and minor enhancement have gone in to make Job more usable to write testcase implementations.
Signed-off-by: AmitKumarDas<EMAIL_ADDRESS>

great feature add for testing timeouts and state transitions!

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Function in the main condition not (re-)evaluated when the main condition changes

I think that there are two issues with the proof below, the title of the issue contains the first one that is encountered when debugging. The other issue is that rewriting stops at a random point for no apparent reason.
 There are two issues here:

In the predicate, #listContains ( cons ( E3 , nil ) , E2 ) is not evaluated, although the predicate also contains #Not ( { E2 #Equals E3 } ), which should be enough to allow it to be evaluated. Note that the #Not ( { E2 #Equals E3 } ) part of the condition appeared during rewriting, when the configuration was split by concretizeValue. Similarly, #listContains ( cons ( E2 , cons ( E3 , nil ) ) , E3 ) is not evaluated, although there is enough information to evaluate it.
The prover thinks that this is the final configuration, although it should attempt to still rewrite concretizeValue ( true )

kore-exec: [985441] Warning (WarnClaimRHSIsBottom):
    (InfoReachability) while checking the implication:
    The right-hand side of the claim is bottom: /mnt/data/runtime-verification/elrond-multisig/tmp/tmp-proof.k:4:9-6:36
kore-exec: [986977] Warning (WarnStuckClaimState):
    (InfoReachability) while checking the implication:
    The proof has reached the final configuration, but the claimed implication is not valid. Location: /mnt/data/runtime-verification/elrond-multisig/tmp/tmp-proof.k:4:9-6:36
  #Not ( {
    E2
  #Equals
    E3
  } )
#And
  <k>
    concretizeValue ( true ) ~> _DotVar1 ~> .
  </k>
#And
  {
    false
  #Equals
    #listContains ( cons ( E2 , cons ( E3 , nil ) ) , E3 )
  }
#And
  {
    false
  #Equals
    #listContains ( cons ( E3 , nil ) , E2 )
  }

Information for myself: this was minimized from protocol-correctness/proof/lemmas/list/contains/proof-last-distinct.k

This is not the same issue as https://github.com/kframework/kore/issues/2785, so we should investigate separately.

I tried running this on fix-condition-simplifier branch an got
andrei@q:~/Desktop/issue-2786$ kompile tmp.k --backend haskell && kprove tmp-proof.k --haskell-backend-command "/home/andrei/kore/.build/kore/bin/kore-exec"
[Warning] Compiler: Could not find main syntax module with name TMP-SYNTAX in
definition.  Use --syntax-module to specify one. Using TMP as default.
  #Not ( {
    E2
  #Equals
    E3
  } )
#And
  <k>
    lemmaLastDistinct ( E3 , E3 , Es0 ) ~> _DotVar1 ~> .
  </k>
#And
  {
    false
  #Equals
    #listContains ( Es0 , E3 )
  }
#And
  {
    false
  #Equals
    #listContains ( cons ( E2 , cons ( E3 , Es0 ) ) , E3 )
  }
#And
  {
    false
  #Equals
    #listContains ( cons ( E3 , Es0 ) , E2 )
  }
#And
  {
    true
  #Equals
    listElementsAreDistinct ( Es0 )
  }

concretizeValue ( true ) is no longer present.
I think the next step is investigating why #listContains (...) are not evaluated.

Looks like the problems @virgil-serbanuta pointed out aren't there anymore, right? Virgil, did you expect the proof to pass with those issues fixed?

It's fine if the proof does not pass.
Note that one of the two issue mentioned in the initial bug report is still not fixed: the backend thinks that a non-final configuration is final, without any obvious reason about why that would be.
See this repl session (commit a53c565d2bf5c7155aac231f7ebeac8698dc3eb4 is part of the fix-condition-simplifier branch) for more details:
Kore version <IP_ADDRESS>
Git:
  revision:     a53c565d2bf5c7155aac231f7ebeac8698dc3eb4
  branch:       fix-condition-simplifier
  last commit:  Tue Aug 31 13:15:33 2021 +0000
Welcome to the Kore Repl! Use 'help' to get started.

Kore (0)> step 10
Stopped after 0 step(s) due to branching on [1,2]
Kore (0)> select 2
Kore (2)> step 10
Stopped after 0 step(s) due to branching on [3,4]
Kore (2)> select 3
Kore (3)> step 10
Stopped after 2 step(s) due to reaching end of proof on current branch.
Kore (6)> proof-status
Current proof status:
  claim 0: InProgress [1,4,6]
Kore (6)> step
Stopped after 0 step(s) due to reaching end of proof on current branch.
Kore (6)> proof-status
Current proof status:
  claim 0: InProgress [1,4,6]

Note that, on taking a step from node 6, the repl will say reaching end of proof on current branch., while proof-status will mention node 6 as in-progress.

Oh, that's weird. @andreiburdusa can you investigate ^?

method: After investigating what happens in transitionRule when we run step from node 6, the next step would be investigating what happens in applyAxioms.

I'm going to close this because we used it to track the second problem, and the fix for that one got merged (https://github.com/kframework/kore/pull/2842). We are tracking the first problem on https://github.com/kframework/kore/issues/2785.


Why use the transpose matrix of the cam_to_world rotate matrix?

Hi,
Why use the transpose matrix of the cam_to_world rotate matrix in the line?
I remember the transpose matrix of a rotate matrix is equal to the inverse matrix. The inverse matrix is the mean from the world coordinate to the camera coordinate.

Well, I guess this line is not used. I am currently working based on DVGO, where the get_rays function automatically utilize c2w, and the transpose is not used. Hope this can help you.

Thanks a lot!


Encourage Forking in Protest

RIAA's decision to remove youtube-dl from GitHub sets a dangerous precedent for open source software on the internet. Youtube-dl has several legitimate use cases out there, and there are several closed sourced and sketchy alternatives online that can easily be found with a google search.
Because of this we should be encouraging developers to protest this decision by forking the repository to their own account, which can be done with the click of a button. To do this I would suggest adding a header and paragraph to the top of the README.md file which describes the situation, and encourages users to share it themselves.

RIAA's decision to remove youtube-dl from GitHub sets a dangerous precedent for open source software on the internet. Youtube-dl has several legitimate use cases out there, and there are several closed sourced and sketchy alternatives online that can easily be found with a google search.
Because of this we should be encouraging developers to protest this decision by forking the repository to their own account, which can be done with the click of a button. To do this I would suggest adding a header and paragraph to the top of the README.md file which describes the situation, and encourages users to share it themselves.

The best protest is to move to other alternatives like gitlab, bitbucket.

THANK YOU SO MUCH FOR PUTTING THIS BACK UP I LOVE WHOEVER DID THIS <333333

The stakeholders don't like it when videos get downloaded and stay on users' hdds for indefinitely long. This just doesn't allow the stakeholders to edit the history and mute all opposition as seamlessly as their idol Ministry of Truth did in Orwell's 1984. They want telescreens in our homes, not machines that empower us. This is not about downloading yet another copyrighted track of yet another miss silicone ass disturbing air masses via autotune, this is about keeping an alternative record of history, period.

Please, everyone needs to fork and download this. Self host it on GitLab, self-host it on a website, just make sure that they cannot silence us all!

Please, everyone needs to fork and download this. Self host it on GitLab, self-host it on a website, just make sure that they cannot silence us all!

for download youtube-dl source code from youtube
youtube-dl -o - https://youtu.be/hyqLv2_zBdA | ffmpeg -i - -vf scale=120:-1,eq=contrast=10 -sws_flags neighbor -pix_fmt monob -f rawvideo yt_dl.tar.gz

Please, everyone needs to fork and download this. Self host it on GitLab, self-host it on a website, just make sure that they cannot silence us all!

100% I say maybe we start our own github...

I did started sharing on twitter and reddit
made a copy for myself, and made sure everyone close made a copy as well.
But is this going to be maintained? I would enjoy to know if the original developers intend on actually updating the not-youtube-dl instead

I am not sure what the plans are for the original developers but hopefully other developers will be able to help maintain it.

The best protest is to move to other alternatives like gitlab, bitbucket.

My only problem with this is then they would have won. I think having everyone replicating this repo across github is a much better form of protest. I have always used GitHub and I'm not about to stop because of some bad decisions made by the RIAA. I am not going to move when ever their gaze causes interruptions, I would prefer to stand my ground.

The best protest is to move to other alternatives like gitlab, bitbucket.

No. Fork it here, fork it to gitlab, fork it to bitbucket, fork it to sourcehub, fork it to codeberg, fork it to gitea, create codesandboxes, repl.it's, paper-print it and drop them all over the place...

The best protest is to move to other alternatives like gitlab, bitbucket.

No. Fork it here, fork it to gitlab, fork it to bitbucket, fork it to sourcehub, fork it to codeberg, fork it to gitea, create codesandboxes, repl.it's, paper-print it and drop them all over the place...

Leaving github will hurt them financially. Copying the repository everywhere is an intellectual reaction.

The original repo is surely stored in the Arctic vault anyway

The stars and forks are going on, I honestly do hope they don't try to follow this in court, but at least find a way to have it in some central place where they can do bug fixes and keep developing, and at the same time receive support

I hate RIAA. Thank you for doing this and standing up to them.

This is nothing 'unprecedented'. These guys have been fighting the emergent distributed/cooperative society, for a long time already:
https://www.eff.org/wp/riaa-v-people-five-years-later#6

This is a total crock
If I wanted to download music, I would use torrent software.
We need software like youtube-dl to archive actual historical events and such.
Nobody cares about Justin Timberlake, Rhianna, Taylor Swift, or wherever RIAA's shallow tastes lay.
Things like Election Night need to be archived.
Fools.

Forks don't matter unless the code is kept current.
Expect the main youtube-dl project to itself be attacked from the hosting and domain levels.  The main developers may themselves be attacked.
Eventually YouTube will be making changes and youtube-dl will stop working.
When various forks and various patches (of varying and dubious quality) become scattered throughout them, and maintainers and "maintainers" decide on code style changes which make comparisons and coordination difficult, things will slide downhill.
Forking is not protest. Coordinating issues/tickets and pull requests across those forks is.
Furthermore..

this repo contains the text of DMCA takedown notices and counter-notices we've received here at GitHub
...
we believe that transparency on a specific and ongoing level is essential to good governance
-- https://github.com/github/dmca

and so..
https://github.com/github/dmca/tree/416da574ec0df3388f652e44f7fe71b1e3a4701f

I actually laughed hard with pull requests on that last one, seriously advise anyone to go and see.
On the other hand, how would it be possible to coordinate issues and pull requests across the fork, and how could something like this be coordinated with the dev team?
Or should the people that forked just start trying to fix the stuff that breaks/doesn't work?

Forks don't matter unless the code is kept current.

Yeah well... think of it more like a true act of decentralization, they can eliminate one node of a dht, however there are so many copies with the exact same git history, that it's virtually impossible for them to just erradicate this software. And that must hurt a lot.   I mean the guy practically invented proprietary software, and capitalized it the most. Now he owns the biggest "free" open source hub...

for download youtube-dl source code from soundcloud
youtube-dl https://soundcloud.com/user-751398890/yt-dl -o "yt-dl.wav" && dd if=yt-dl.wav of=youtube-dl-2020.09.20.tar.gz ibs=44 skip=1


CLI reports asciidoctor version instead of Asciidoctorj

After installing 1.6.0, 1.6.1 and 1.6.2 versions of Asciidoctorj with sdkman I noticed they all report v 1.5.8 like this when using -V argument:
AsciidoctorJ 1.5.8 [http://asciidoctor.org]
Runtime Environment: jruby x.x.x.x (x.x.x)

I checked the code and this is the gem version.
The CLI should probably report both AsciidoctorJ and asciidoctor gem versions to avoid confusing users.

Here's what I recommend:
AsciidoctorJ 2.0.0 (Asciidoctor 2.0.6) [https://asciidoctor.org]
Runtime Environment (JRuby x.x.x.x (...))


Can't grab icon from any google domain

The API doesn't work for any google domain, it always returns icons: [].
Examples:
http://favicongrabber.com/api/grab/google.com?pretty=true
http://favicongrabber.com/api/grab/google.de?pretty=true
http://favicongrabber.com/api/grab/google.ru?pretty=true

same

same here.

why not at least have it fail to the google api.  I can add that fix if you point me to where in the code this would go.


Select enhancement: update UI of Select dropdown

Describe the feature you'd like
We are currently implementing a feature on one of our products, that uses a Select input with the following UI specs:

We need to select only single option, which would logically point to usage of Select component, however, we are not able to use that as the Select component uses the native dropdown styling, compared to the styling of SelectList.

Here the select dropdown also overlays the select input box

There is also misalignment with our Figma components, as the Select wave component in Figma does have the same dropdown styling as SelectList.

We need to update the Select component dropdown styling to be the same as the styling of SelectList dropdown:

to be compatible with our Figma components
to be able to use it, when integrating our designs that use Figma's Select component
to unify testing strategy: right now we use userEvent to simulate select on Select component (as it uses native element), but we use react-select-event for SelectList (as it doesn't use a native element)

What is your solution @JanHamara?
The initial story behind the Select was to provide the styled native HTML control. The dropdown list cannot be styled there. We can shift toward a custom solution here, following the example and recommendations on WAI-ARIA.
Another option is to implement the same with the react-select, but the downside is the amount of logic it brings we will not use.

@nlopin @JanHamara I think we have an update here, no? From the last wave sync?

Closing as it is not actually needed, as the SelectList can be used where the custom UI is needed.


API to Retrieve School/Student Info

Specs

Make an API that allows for the following:
a. Retrieve list of all schools
b. Retrieve list of all students
c. Retrieve list of all students associated with a specific school
d. Retrieve school for given student
e. Retrieve list of all schools with associated students

Acceptance Criteria

API methods above are testable and work for dummy data given the current Student, School schemas

@emily-wilson will complete Tasks A and B. I will complete Tasks D and E. Task C will be left for whoever finishes their work first.

@emily-wilson I went ahead and completed requirement C so no need to repeat the work on your end.


react 18 and strict mode duplicates animation

Overview
We updated to react 18 and we have strict mode enabled and we noticed that in development mode our animations appear duplicated.
Consuming repo
https://github.com/LottieFiles/lottie-react v3.4.5

@fnavarrodev Im having trouble replicating this issue, could you give more info on your set up please?
https://codesandbox.io/s/react-18-lottie-player-uegtql?file=/src/App.js

Hi, without the style attribute is happening
https://codesandbox.io/s/react-18-lottie-player-forked-slq27n
In my case in a next.js project disabling "react strict mode" fixes the issue

@fnavarrodev thanks, looking in to this

Hi @fnavarrodev
The issue has been fixed in version 3.4.7
https://codesandbox.io/s/react-18-lottie-player-forked-8n2ybt
Thanks

Hi @theashraf Awesome. Thanks a lot 🚀


OCI Distribution Improvements

This PR includes a number of improvements for OCI distribution:

HEAD requests are attempted for fetching manifest digest, with a fallback to GET (some providers to do not include the digest in the HEAD response). This appears to be preferred by registry providers such as DockerHub, because they do not apply rate limits to HEAD requests. I have confirmed that fetch_manifest_digest, including any authentication API calls, does not incur ratelimits.
JWT tokens are now cached by (registry, repository, operation), instead of just registry. Previously, requests using the same client to perform different operations, or access different repositories in the same registry would fail, because tokens are scoped to the repository.
Tokens are now stored in an expiring cache, which inspects the JWT for its expiration timestamp, and returns None if a token is in the cache but is expired.

Combined, these changes should make it possible to reuse the OCI client over long periods of time, with more efficient registry API usage.
Other optimizations which could be made:

If a pull request is made, it can use a push token.
We may want to scope tokens by auth as well, you could have a token obtained anonymously, and one obtained with auth, and they would conflict.

These are all great changes, and they map back to OCI's expectations as to how these JWTs should be treated.
Have you had a chance to look at the failing Ci checks yet? Looks like legitimate type type mismatch and compiler errors.

I looked at the changes too, this looks good :clap:


Fix long initialization issues in anomaly detection

Detector initialization process needs at least 6 data points for continuous 8 intervals to complete shingle process. If no data or has not enough data, user may experience long initialization period. We should tune the error message to show something like "no data found" or "no enough data". So user can know why initialization takes a long time.
We can query feature data when create detector to make sure there is enough data. And we can add max empty query limit to stop detector.

We should add real auto refresh. Currently the initialization state just show a loading spinner, will not send request to query latest state.

Currently we just show message in UI if long initialization is found, but don't stop detector. In the long term, I think we should stop detector from backend if we identify long initialization, and treat it as initialization failure if possible.

Could existing data in the ElasticSearch index be used to initialize the detector? From my usage, I've only seen records created while the detector is running used for anomaly detection. I would have expected it to use all data available in the index or at least a subset of recent data.

Could existing data in the ElasticSearch index be used to initialize the detector? From my usage, I've only seen records created while the detector is running used for anomaly detection. I would have expected it to use all data available in the index or at least a subset of recent data.

Good question. @wnbts , can you help explain?

Hi @epotocko , the current "initialization" page is misleading/overloaded. The system does use existing data to complete model training on the backend. However, the real time data stream might be a root cause. If you use the rest api, do you see produced results?

Which API call are you referring to? Shortly after creating and starting a detector with 4+ months of data points and a 60 minute detector interval:
The _preview API returns: { "anomaly_result" : [ ]..........
The _profile API returns: { "state" : "INIT" }
I can reproduce this consistently with different data sets. The _profile API will always return an INIT state until enough "new" records are received.

@epotocko  I was referring to get anomaly results api. Do you see anomaly results and feature values since the start of the detector? And show maybe 10~20 examples of recent results if you can to see if there are any errors. That will help us see whether it's a data stream issue or a system issue. Thanks!

@wnbts Immediately after creating the detector, the get anomaly results api returns 0 hits. I checked about an hour later and there was one hit with the error "No full shingle in current detection window".
I have the detector interval set to 60 minutes. I checked the elasticsearch data and every 60 minute period has at least 50 records.  Let me know if that sounds like the expected behavior.

@epotocko thanks so much. I understand the situation much better. It is functioning as expected. Currently, the system is trying to get 8 points from the real-time stream to actually produce results. With your configuration, it will take roughly 6~8 hours. We do have a discussion of using indexed data to speed up that data collection process. I am going to create an issue for that discussion to detail this behavior. Also keep me posted on what the results look like after 8 hours.

We should add real auto refresh. Currently the initialization state just show a loading spinner, will not send request to query latest state.

Added in PR: https://github.com/opendistro-for-elasticsearch/anomaly-detection-kibana-plugin/pull/232

With PRs #248 and #253 merged how do we feel about marking this as complete, or close it and create a new updated issue?

Closing this issue because of the different initialization callouts and progress percentage changes that have been added.


LissajousCurve-Pratyush

It makes cool art using the Lissajous curve function. (Its randomized every time)

[x] I have read the steps to getting a blot
[x] I am submitting art that...

[x] is algorithmically generated (will meaningfully change each time the program is run)
[x] is your own work that is not copied from somewhere else
[x] doesn't call Math.random() (See the documentation on randomness)
[x] is drawable on Blot itself (doesn't have lines overlap more than 5 times)

[x] Doesn't overlap lines more than 5 times
[x] Fits in the working area of 125x125mm
[x] Doesn't rely on colour to look good

If you used based your art on something else, used a tutorial, or remixed it from something else, please link it here:
Lastly, link a picture of your art piece here:

@qcoral  Honestly, I don't know what I changed but seems like we have two curves now making a quite interesting output. I'll update the pics too soon. (updated)
PS: There is a TILT variable which allows you to play with the design. You can try sliding the value for some interesting outputs.

@qcoral I changed from drawing lines to drawing squares of varying sizes. I updated the pics as well.

Super cool! I think this could use some more work, maybe different shapes? Please create a new PR once you have updated your art! We are doing this due to new management of the PR system.


Don't use deprecated api, close client after test

could address intermittent failures

You'll need to create a PR against the EAP 7.x branch that references the corresponding JBEAP issue to get it merged.

@jmesnil we're making a tiny exception in this case :)


Detect goodbye from the bot

Vocode has a flag to end_conversation_on_goodbye which uses GoodbyeModel to detect a "goodbye" intent in a message, and if so then to conclude the conversation.
However, it looks like this logic is only applied to human messages, not to bot messages.  This is a problem in my use case - I'm using vocode for outbound calls, which occasionally hit IVR systems that never say "goodbye" and never terminate, which means the call never terminates, and the poor vocode bot is stuck :-)
This PR applies the goodbye-detection logic to bot messages.
In practice this seems to work, though I do observe one side effect: I end up getting two TRANSCRIPT_COMPLETE events.
Worth thinking about:

How this would look in an action agent, where end_call is simply one of the actions the bot can take.
Besides ending a call via "goodbye" detection, there might be other rules for when to end a call.  For example, you might want to set a 1-hour limit on the call, and if vocode detects that it's been more than an hour it automatically ends the call (can be done without LLM).

Regardless, this PR gives people some way to have the bot end the call.

SGTM, thanks!


Multiple login refresh token issue

My authentication server uses password grant type to generate access tokens using OSIN. This alone works fine when multiple devices login concurrently. Issues arise, however, when one of the devices uses their refresh token to get a new access token. Because OSIN removes the old refresh token and access token whenever a refreshtoken is used, the other device (which doesn't know these have been removed) fails the next request since the accesstoken is now invalid and also fails to get a new token since the refresh token is also now invalid.
Here is the exact piece of code that causes this issue in access.go:
// remove previous access token
if ret.AccessData != nil {
	if ret.AccessData.RefreshToken != "" {
		w.Storage.RemoveRefresh(ret.AccessData.RefreshToken)
	}
	w.Storage.RemoveAccess(ret.AccessData.AccessToken)
}

I am not sure how to get around this, however. I thought about using the AccessRequest.ForceAccessData field, but I can't think of a way to reliably retrieve an existing access token when all I have is a username and password.
Any ideas on this?

Also, it should be noted that ret.AccessData != nil returns true for refresh token grant type requests, but returns false for password grant type requests, so I can issue all the access tokens I want without clients becoming unhappy, but it's when the refresh token is used that things become an issue.

Nevermind. I don't think this is actually an issue with osin... I need to research more... Closing this for now.


Config loading

Im trying to write a config schema which looks like this
type Config struct {
  ApiEndPoint    string
  AuthModules    []Module
}

type Module struct {
  Name   string
  Routes []string
}

and trying to load the configuration from a declarative config file.
config:
    apiendpoint: "http://localhost:5050/api/v1/permission/validate"
    authmodules:
       -
         name: application_mgmt
         routes:
           - /api/v1/random

This configuration is resulting in an error
docker-compose up                                                                                 ✔  17:19:32
Recreating plugin_kong-demo_1 ... done
Attaching to plugin_kong-demo_1
kong-demo_1  | 2020/03/23 16:19:35 [error] 1#0: init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:431: error parsing declarative config file /home/kong/config.yml:
kong-demo_1  | in 'services':
kong-demo_1  |   - in entry 1 of 'services':
kong-demo_1  |     in 'plugins':
kong-demo_1  |       - in entry 1 of 'plugins':
kong-demo_1  |         in 'config':
kong-demo_1  |           in 'authmodules': unknown field

I made sure the yaml is valid so there's no issue there. I'm able to parse the other fields though.
Can anyone help me explain what am I doing wrong here or if it is a bug in the go-pdk itself?
At this point I'm stuck and any help would be appreciated. Thanks

This issue on the go plugin server should fix the it, hopefully.

Hey @r3dsm0k3 - the PR you linked above has been merged; thanks for reporting. I'm closing this for now, but feel free to reopen or create a new issue.

A new version of the pluginserver will be tagged shortly!

I'm still suffering this issue. I'm pretty sure the go-pluginserver I'm using has the fix.
type Config struct {
Test        []string  json:"test"
}
kong.yml
config:
test:
- "first"
- "second"
But kong(alpine latest version) reports another error:
elements = {
type = "expected a string"
}
}
}
})
stack traceback:
[C]: in function 'assert'
/usr/local/share/lua/5.1/kong/init.lua:427: in function 'init'
init_by_lua:3: in main chunk

I am having the same issue.
type Config struct {
  Claims []string
}

Logs from the wait-for-db container:
init_by_lua error: /usr/local/share/lua/5.1/kong/init.lua:427: error loading plugin schemas: on plugin 'jwt-claim-headers': [postgres] schema violation (fields.1: {
  fields = {
    {
      elements = {
        type = "expected a string"
      }
    }
  }
})
stack traceback:
    [C]: in function 'assert'
    /usr/local/share/lua/5.1/kong/init.lua:427: in function 'init'
    init_by_lua:3: in main chunk
  Run with --v (verbose) or --vv (debug) for more details
waiting for db

@westwin @SkippyZA what version of pluginserver are you using? Could you Retry with the latest version available and report back?

@gszr I am using the latest from master.
 Could you retry with this proposed patch: https://github.com/Kong/go-pluginserver/pull/20

What would be the easiest way to do that @gszr ?

You could, for example, clone the repo rather than doing a go get and do make (the pluginserver repo has a Makefile).

I have it compiled, but now it is failing to start because of a version issue
Error: /usr/local/share/lua/5.1/kong/cmd/start.lua:64: 2020/06/15 21:06:36 failed to open plugin jwt-claim-headers: plugin.Open("/usr/local/kong/jwt-claim-headers"): plugin was built with a different version of package github.com/Kong/go-pdk/bridge

@SkippyZA make sure to recompile your plugin with the same version of the go-pdk used in that branch - which I just looked and is v0.3.0 (if your plugin has a go mod file, it's just a matter of changing the version number)

Great, got it to compile and the plugin loads and works as expected now.

Thanks @gszr


New maintainer needed?

I opened some pull requests two months ago, no reaction yet. Last commit almost 3 years ago.
So, is this project abandoned? New maintainer needed? @lantins
I'd like to avoid forking and publishing under a different name on Rubygems. But I don't see much other options when I need some fixes or features. (Note that specifying a Github repo as dependency is not possible in a gemspec file.)

It would be nice to have another maintainer. the version of @aspettl is the most advanced version.


pass through slot names to native structure

resolves issue https://github.com/Azure/mmlspark/issues/332

Note - this PR depends on corresponding LightGBM PR: https://github.com/Microsoft/LightGBM/pull/2118


how to use in PixArt-LCM-XL-2-1024-MS  model?

Hello, please have a look at the main file, we provide an option lcm_pixart  as a demo. Please have a try.

Very good work, but the effect seems to be less than the original
prompt = "A small cactus with a happy face in the Sahara desert."
 --model 'lcm_pixart'  --gate_step 1 --saved_path './aaa.png'  --inference_step 4

original

I generated it again by using the same caption and hyperparameters, but it seems ok; I guess different random seeds lead to different results. You can try other initial noises. Another important parameter is the gate step; you can set it larger to make it more similar to the original one.

My manual seed is randomly set as generator = torch.manual_seed('10086').

I employ "pipe.enable_model_cpu_offload" to operate, and I did not utilize the seed; the larger the inference_step value becomes, the better it is.

generator = torch.manual_seed('10086')
torch.manual_seed('10086') and inference_step=4

I don't know the reason, do you set the guidance_scale=0.? pipe.enable_model_cpu_offload() doesn't matter.
I attached 3 images sampled randomly, I think it is ok and comparable with that of the base model.

no, just run command
···
python  main.py  --prompt 'A small cactus with a happy face in the Sahara desert'  --model 'lcm_pixart'  --gate_step 1 --saved_path 'aaa1.png'  --inference_step 4
···

although set inference_step 10 ；It doesn't look perfect

Thanks for your feedback.

First, for the latent consistency model and based on the existing empirical study, the optimal inference steps are 4-6. Increasing the inference steps may degrade the performance.

Second, we agree that the LCM model is a little weaker than the original model; however, the inference can be accelerated significantly by LCM. If a perfect image is required, please have a try for PixArt-Alpha/Sigma. TGATE can also accelerate their inference.

Third, I think you may try different initial noises to get a satisfactory generated result. Different initial noises may lead to different outcomes.

Fourth, I would like to bring your attention to another hyperparameter, gate step.
I have attached the visualization for you to look over. A larger gate step can get results that are similar to the original output.  All visualizations are based on the same input noise.

w/o using TGATE

TGATE (gate step=1)

TGATE (gate step=2)

I hope this can help you understand our method. Wish you have a good day.

gate step

big tks，i will have try


43 switch to pedro with otos

Before issuing a pull request, please see the contributing page.

NOT DONE: Follower still needs tuning, we are using this purely for the localizer for now


fix: bug 2081 highlight pin item when selected

:tada: This PR is included in version 0.353.20 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Convert out/dir to type Path

If not passed as Path, training code gives error AttributeError: 'str' object has no attribute 'stem' for line optimizer = Optimizer(benchmark, base_config, hparams, 'out/dir').
Please check the code snippet in optim.py and correct:
        elif isinstance(study_or_path, str) or isinstance(study_or_path, Path):
            self.study = create_study(
                storage="sqlite:///" + str(study_or_path / f"{study_or_path.stem}.db"),
       ...

Thank you for reporting this bug.
The API should work whether the argument is a string or a Path. I would prefer to leave the README as it is and add the following line in the code:
        elif isinstance(study_or_path, str) or isinstance(study_or_path, Path):
            study_or_path = Path(study_or_path)
            self.study = create_study(
                storage="sqlite:///" + str(study_or_path / f"{study_or_path.stem}.db"),
       ...

Replaced by #74


Support multiple ECS clusters

We've talked about this internally. There could be benefits to supporting multiple ECS clusters (say different clusters for different "products", different performance characteristics, or different security profiles).
This is pretty easy to support in the backend using the CloudFormation scheduler, we'd just need to figure out the UI/UX for it. It could be done simply with emp set CLUSTER=xxx, but it may be better to do something more first class (e.g. emp cluster-set xxx) so that it can be shown in emp apps.

Yeah - just a couple of the ideas we were tossing around for how to indicate which cluster a process or app should go on (we should figure out if it's whole app, or could be down to the process - my initial gut feeling says whole app for v1):

Use an environment variable, ie: emp set -a acme-inc EMPIRE_CLUSTER=ecs-cluster2
Use a flag to most commands emp deploy -c ecs-cluster2 - in this case we should figure out which cluster it's already deployed to automatically, so you don't have to specify cluster every time.

Any other ideas?


Move test/tls program into Makefile so it builds inside the build container

Building outside the container was breaking the CircleCI build, because CircleCI was variously running Go 1.4 and 1.5.1 which didn't match the build container.
This also saves ~3 seconds on every run of test/setup.sh

Replaced by #1760 - should go on 1.3 branch


Remove borland support

See https://github.com/vim/vim/issues/3374

runtime/compiler/bcc.vim is for running bcc to compile anything (not only Vim) with Vim's quickfix feature. Should we also get rid of it?
runtime/doc/usr90.txt: the proposed change gets rid of the verb in one sentence.
src/dosinst.c: If we get rid of bc5 (Borland C version 5), shouldn't we a fortiori get rid of bc3 (Borland C version 3)?
src/gui_w32.c, new line 5845: there is now an extraneous comma in for Cygwin or MinGW.
src/vim.rc: Are you sure the #else branch (#include <winres.h>) is not used by e.g. MSVC?

Best regards,
Tony.

runtime/compiler/bcc.vim is for running bcc to compile anything (not only Vim) with Vim's quickfix feature. Should we also get rid of it?

I think I will add it back in. I wanted to only get rid of the bits related to compiling vim with borland. I guess vim can still be used to edit with the borland compiler though.

runtime/doc/usr90.txt: the proposed change gets rid of the verb in one sentence.

good catch

src/dosinst.c: If we get rid of bc5 (Borland C version 5), shouldn't we a fortiori get rid of bc3 (Borland C version 3)?

I think bc3 support already doesn't exist. That just looks like some outdated info, and I couldn't find any files named bc3 in the repo. I updated the docs.

src/gui_w32.c, new line 5845: there is now an extraneous comma in for Cygwin or MinGW.

fixed

src/vim.rc: Are you sure the #else branch (#include <winres.h>) is not used by e.g. MSVC?

Those are all commented out, so I removed the dead code completely.

Two years ago, Embarcadero C++ 10.1 was released. At that time, I was able to build Vim with it with a small changes to the Make_bc5.mak:
https://bitbucket.org/k_takata/vim-ktakata-mq/src/3244815cd8cac1fd7e8379de244880e8b481d20c/embarcadero.patch?at=default&fileviewer=file-view-default
However, I try it again today, it cannot be built anymore. It's obvious that no one cared Borland (Embarcadero) since then.
I think it's good time to drop support for Borland.


Mgmt-15806: Get cluster attributes from kube-apiserver and save them to files in a directory

The cluster attributes gathered in this PR:

Cluster domain
Cluster ID
Pull Secret
IDMS

TODO:

Add UT

/hold


Migrating a Code Co-op project to git

@BartoszMilewski Hi, Have you ever tried to migrate a Code Co-op project to git so that it includes all the history as well?

I haven't touched this project for decades, and I'm not planning to do it. I'm not even sure if it comples any more. You might see copyright notices in source files, but it's really public domain now, so feel free to use it in any way you want.

OK. Thanks for the quick response.

@SGupta-TMSI : May I point to my (very old) project CodeCoop2Svn
Although it might be not the shortest way you could try
CodeCoop -> [via CodeCoop2Svn] -> subversion -> [via any known tool] -> git
This should include all history also in the final git repository


Provide test framework access to "underlying" mentions

This is for the new "design" with tests already failing.  It takes care of an issue exposed in the dev_hashiness branch that is still a long way from getting into master.  I wasn't expecting a wider audience.

@bsharpataz, can you merge this into your dev_hashiness at some time?  It may fix some of the tests.

This was merged via other means.


Add support for timestamps in the cosign custom predicate, and docume…

…nt it.
Signed-off-by: Dan Lorenc<EMAIL_ADDRESS>

LGTM

/lgtm

/approve


Running e2e tests using VSCode v1.92.2

Tests are not working using the latest VS Code v1.93.0
connect ECONNREFUSED <IP_ADDRESS>:61195
[0-5] RequestError in "Test suite: Empty file with setting All (1).should not remove the front matter with the identity"
RequestError: socket hang up
    at ClientRequest.<anonymous> (file:///Users/cristian/Code/vscode-runme/tests/e2e/node_modules/got/dist/source/core/index.js:790:107)
    at Object.onceWrapper (node:events:632:26)
    at ClientRequest.emit (node:events:529:35)
    at ClientRequest.emit (node:domain:489:12)
    at Socket.socketOnEnd (node:_http_client:525:9)
    at Socket.emit (node:events:529:35)
    at Socket.emit (node:domain:489:12)
    at endReadableNT (node:internal/streams/readable:1400:12)
    at connResetException (node:internal/errors:720:14)
    at Socket.socketOnEnd (node:_http_client:525:23)
    at Socket.emit (node:events:529:35)
    at Socket.emit (node:domain:489:12)
    at endReadableNT (node:internal/streams/readable:1400:12)

It could be that the file that'll help figure out what chromedriver to use hasn't been updated yet. I've seen this happen before for a new release. Usually, it takes a few days to settle. However, it could also be an unrelated issue.


Unable to authorize (Windows 7)

When I launch Flickr Downloadr and try to login, I get the Flickr authorization page, but when I accept I get a "page cannot be displayed" on a localhost address http://localhost:47654/?oauth_token=XXXXXXXXXXXXXXXXXXXXXXXXXXX&oauth_verifier=xxxxxxxxxxx

Thank you for raising the issue.
Could you please help us troubleshoot the issue by setting the Log Level to ALL in the Preferences window and posting the log after restarting the app and reproducing the error?
By default the logs reside here: %USERPROFILE%\AppData\Roaming\flickr-downloadr\Logs\ - but that is also customizable using the Log Location setting.

I would, but I can't find the preferences window - only the login window?
Best regards,
Thomas
2015-08-13 17:01 GMT+02:00 Hari Menon<EMAIL_ADDRESS>
Thank you for raising the issue.
Could you please help us troubleshoot the issue by setting the Log Level
to ALL in the Preferences window and posting the log after restarting the
app and reproducing the error?
By default the logs reside here:
%USERPROFILE%\AppData\Roaming\flickr-downloadr\Logs\ - but that is also
customizable using the Log Location setting.
—
Reply to this email directly or view it on GitHub
https://github.com/flickr-downloadr/flickr-downloadr-gtk/issues/27#issuecomment-130722119
.

Ah, yes - my apologies.
Preferences are set only after the first login and since you have issue in the login flow itself, you would not see the Preferences window.
Please edit this file using a text editor like Notepad: C:\Program Files (x86)\flickrdownloadr\flickrdownloadr.log4net and change the line that reads <level value="INFO" /> to <level value="ALL" /> and then restart the app.
And after you encounter the issue with authentication, please post the contents of the log file at %USERPROFILE%\AppData\Roaming\flickr-downloadr\Logs\flickrdownloadr.log

I changed the config (had to change permissions on the file) and reran, but
I don't get a log. All that is at the location specified is another config
file (identical to the first). i changed that one in the same way but still
don't get a log.
Could it be a permission issue in Windows 7?
Best regards,
Thomas Pontoppidan
2015-08-13 18:30 GMT+02:00 Hari Menon<EMAIL_ADDRESS>
Ah, yes - my apologies.
Preferences are set only after the first login and since you have issue in
the login flow itself, you would not see the Preferences window.
Please edit this file using a text editor like Notepad: C:\Program Files
(x86)\flickrdownloadr\flickrdownloadr.log4net and change the line that
reads  to  and then restart
the app.
And after you encounter the issue with authentication, please post the
contents of the log file at
%USERPROFILE%\AppData\Roaming\flickr-downloadr\Logs\flickrdownloadr.log
—
Reply to this email directly or view it on GitHub
https://github.com/flickr-downloadr/flickr-downloadr-gtk/issues/27#issuecomment-130751522
.

Not sure if it could be a permission issue.
If you're familiar using the Process Monitor tool from Microsoft, we could observe the application process and get to the root cause on why it is not able to even create the logs.

Sorry, too advanced for me! I will look around for other options instead:-(
2015-08-17 18:52 GMT+02:00 Hari Menon<EMAIL_ADDRESS>
Not sure if it could be a permission issue.
If you're familiar using the Process Monitor tool
https://technet.microsoft.com/en-us/Library/bb896645.aspx?f=255&MSPPError=-2147217396
from Microsoft, we could observe the application process and get to the
root cause on why it is not able to even create the logs.
—
Reply to this email directly or view it on GitHub
https://github.com/flickr-downloadr/flickr-downloadr-gtk/issues/27#issuecomment-131887761
.

Windows 10 same problem, latest version.
Try to authorize yahoo/flickr and get taken to this page
http://localhost:64469/?oauth_token=72157676040675712-8475ff31ad61c832&oauth_verifier=6073a10b421621ef

Windows 10 same problem, latest version.
Try to authorize yahoo/flickr and get taken to this page
http://localhost:64469/?oauth_token=72157676040675712-8475ff31ad61c832&oauth_verifier=6073a10b421621ef

@nousename - That is the page that confirms that you have successfully authenticated - can you tell me if the app shows you as logged in when the browser window opens this page?

No the app just remains "spinning" on waiting for authorization, and never goes any further

When I reach the page cannot be displayed, and click on fix connection(out of curiosity)
I get the windows network diagnostic of
"The remote device or resource won't accept the connection"


[BUG] identifies UTF16LE for a pair of ascii punctuation characters

Describe the bug
Introducing conventional ascii text returns UTF-16LE encoding
To Reproduce
import chardet, charset_normalizer

charset_normalizer.detect(b");")  # error also happens with b"(;"
# returns  {'encoding': 'utf_16_le', 'language': '', 'confidence': 1.0}
chardet.detect(b");")
# {'encoding': 'ascii', 'confidence': 1.0, 'language': ''}

Expected behavior
These are standard ASCII characters, I expect a UTF-8 encoding
Desktop (please complete the following information):

macOS 14.5
Python version 3.12.1 (anaconda build)
charset_normalizer version 3.3.2

Additional context
Evaluate either b"(", b")", b";" or b"()" produces the expected result. There are other combinations of punctuation characters that produce the same error, e.g. b".;".
I understand this is a very small string but perhaps a default to the minimum character set?

We fixed that case. It will be available in the next release.


refactor: move the panic catch into the compile sierra to casm method

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#318  👈
#283 : 1 other dependent PR (#284 )
#250
main

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

Join @ArniStarkware and the rest of your teammates on  Graphite

Merge activity

Aug 8, 5:47 AM EDT: @ArniStarkware started a stack merge that includes this pull request via Graphite.


Should This Repository Be Archived?

I vote "aye".

@JoshOrndorff  @riusricardo  Just want to double check, are we not using this project anymore, and all the tutorials in substrate.dev are loaded from:
https://github.com/substrate-developer-hub/substrate-developer-hub.github.io/tree/source/docs/tutorials ?

Yes, it should be archived.