fix(volumes): add unicity check on volumes

Close #3272

Done @deviantony

@itsconquest you're hitting Docker API behavior here. Since we did not wanted to create a Portainer specific API for volume-creation, we decided to keep the Docker API behavior available (via the request you sent, proxied through Portainer).
But we also added an overlay on top of that API (that is currently used by the frontend), to use it you'll also need to pass the X-Portainer-VolumeName header to the request. This will trigger an extra validation.

@deviantony since you have already reviewed this is RTM


Fields disappearing in newest version

In the newest version, updated yesterday (1/7/20), when I open any Sessions, the ID field is visible, but upon clicking another session, the ID field of the previous session disappears (the whole field and any label has disappeared). This is happening with the names in the People tab too. But when Saymore X is restarted, the information reappears, but the same thing happens if I click away to another Session/Person.

Note -- now the files edited with SayMore X do not open in normal SayMore, but cause an error which prevents the .sprj file from being opened at all.

note -- now the files edited with SayMore X do not open in normal SayMore, but cause an error which prevents the .sprj file from being opened at all.

Yes, we need to do better... SayMore X is not developed by SIL, so there isn't yet coordinated releases with the SIL SayMore maintainers, and backwards compatibility is not going to be possible. I expect we'll figure out something before SayMore X goes out of beta (it will also be renamed so as not to be confused with SayMore).

@hakimjacob I haven't been able to reproduce this problem. Would you be able to help me figure out the steps?  Three options:
A) Give me steps that start with making a new project
B) Take a screen recording and push it to imgur, using something like https://www.screentogif.com/
C) Share some subset of your project that I can play with.
thanks for your help!


Keypad press

Not an issue, but I am not sure where to put a request?  I would like to detect when the keypad is pressed so I can make an audible sound?  I am using the keypad_textsensor.h and keypadtext.yaml.   Is there a way to do that with existing code in yaml?

With the existing code that is not possible. I will think about it, maybe I can come up with a solution.

With the existing code that is not possible. I will think about it, maybe I can come up with a solution.

Thank you- I think I figured it out.  I added the following after  if (myKey != NO_KEY){
//added for passive buzzer
for (int i = 0; i <15; i++)
{
digitalWrite (10, HIGH) ;
delay (1) ;// delay 1ms
digitalWrite (10, LOW) ;
delay (1) ;// delay 1ms
}

Thank you- I think I figured it out.  I added the following after  if (myKey != NO_KEY){
//added for passive buzzer
for (int i = 0; i <15; i++)
{
digitalWrite (10, HIGH) ;
delay (1) ;// delay 1ms
digitalWrite (10, LOW) ;
delay (1) ;// delay 1ms
}


[release-v3.23] cherry-pick #6287: Support ppc64le image

Signed-off-by: Yussuf Shaikh<EMAIL_ADDRESS>Description
Add support for ppc64le apiserver build image.

Related issues/PRs

Todos

[ ] Tests
[ ] Documentation
[ ] Release note

Release Note

Build ppc64le image for calico/apiserver.

Reminder for the reviewer
Make sure that this PR has the correct labels and milestone set.
Every PR needs one docs-* label.

docs-pr-required: This change requires a change to the documentation that has not been completed yet.
docs-completed: This change has all necessary documentation completed.
docs-not-required: This change has no user-facing impact and requires no docs.

Every PR needs one release-note-* label.

release-note-required: This PR has user-facing changes. Most PRs should have this label.
release-note-not-required: This PR has no user-facing changes.

Other optional labels:

cherry-pick-candidate: This PR should be cherry-picked to an earlier release. For bug fixes only.
needs-operator-pr: This PR is related to install and requires a corresponding change to the operator.

cc @caseydavenport

/sem-approve

lemme check my luck!
/sem-approve

/sem-approve

Removing "merge-when-ready" label due to new commits

/sem-approve

closing as ppc64le manifest is available in quay.io/calico/apiserver:v3.24.0


imprecise message when pulling non-existent asset/genome pair

the response is correct (see code example in: https://github.com/databio/refgenie/pull/28#discussion_r293025543), as there's no bowtie2_index asset for hg38 genome (and it's tru for all genomes), try: http://refgenomes.databio.org/asset/hg38/bowtie2_index/archive
however now the error/message that we get is not precise, and I think it caused by the introduction of the connection pre-check that we do for refgenie pull
Originally posted by @michalstolarczyk in https://github.com/databio/refgenie/pull/28/review_comment/create

What do you want the message to be?

sth like: " '{bundle_name}' not found on the server"?

Well _download_json only gets the URL, so given the current signature it doesn't know what's trying to be accessed. I could make DownloadJsonError wrap a reference to the actual Response object from requests.get, though, so that the refgenconf dependent like refgenie or refgenieserver could make use of the JSON data that it contains (r.json(), where r is the Response instance).

In [8]: r.json()
Out[8]: {'detail': 'No such asset on server'}

So then the client code can decide how it wants to recover from or render the exception.

this sounds like a robust solution
but do you mean it will introduce a package dependency on refgenieserver?

No, sorry for ambiguity. I mean that I'm deliberately not going to use the knowledge of the JSON that refgenieserver places in the response via the MSG_404 formatting, so that refgenconf remains "unaware" both literally and conceptually of refgenieserver. Instead, I'll give access to the fuller response data so that refgenie and/or refgenieserver (or any other package that comes to use refgenconf) can handle it as desired, rather than compressing some of that information as is done now, since refgenconf just injects a simplified error message.

That’s great


Update mongodb dependency to ^3.2.7

workflow-es-monogodb has a dependency on monogodb ^2.2.11.
However, "Versions of mongodb prior to 3.1.13 are vulnerable to Denial of Service."
This pull updates the dependency to ^3.2.7.

For sure. Just updated it.


Storybook bug

This seems to be a duplicate of https://github.com/Frojd/Wagtail-Pipit/issues/1413 so I'm closing this.


feat: [WIP] support x-filter on echarts timeseries zoom

SUMMARY

It would be nice to set a time range filter after using the zoom tool
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[X] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[X] Introduces new feature or API
[ ] Removes existing feature or API

A couple of issues arise:

How to get setDataMask to also update this chart, when doing so is not the default option (done in this example after edit dashboard, change properties so filter is applied to this chart too). Problem otherwise is that the zoom is lost when the filters apply.
A way to create and/or update a corresponding native filter would be nice.

Thanks for the contribution @fullergalway! This is an intriguing feature addition! The designers involved with Superset are rethinking a bit of iconography and so forth around the crossfilter experience, so @kasiazjc might have some input on the design implications. Doubly so, since this creates a new pattern of manually emitting a filter.
We've usually steered users toward dashboard filters for this sort of time range filter, but I think what you've done here makes sense for an efficient means of exploration. My immediate thought on the design is that rather than clicking a button to manually emit the crossfilter each time you want to apply an update, it might make more sense to have a setting where the time range is either emitted continuously... or not. Then any change to the time range/zoom would update the filter live, like the immediacy of clicking a pie slice. We may want to either just do that (without any button click) or add a toggle somewhere to enable/disable that crossfilter broadcast/updating/emission. Curious what Kasia thinks about all this :)
I’ve also pinged @Kamil Gabryjelski (Preset) and @Ville Brofeldt for a review on the PR itself, since they have a lot of experience in implementing crossfilters, and can take a closer look at the code.
Thank you for your contribution!

Not sure the status of this PR after so long (is there still interest?), so I'll just close/open to kick-start the CI process and see how it does.

Yes it's definitely something which would is useful for us, and a wider audience for sure.
@rusackas Evan you had previously mentioned having a couple Preset developers have a look over it?

I'll try to ping some folks (e.g. @kgabryje ) about it who know the datamask implications better than I do, but I see a couple things right away that I'm curious about:

There's a typescript error that will need to be addressed which will prevent merging.
I see there's a vector path hard-coded in here, which raises a few questions:

Can we import/use the path from one of our SVG files (filter icon) so that it's always in sync if we change that?
Is this the right icon to use to emit a cross-filter, or is there something better (cc. @kasiazjc )
This might be getting too fancy, but... can/should the icon/button be togglable to emit/remove the crossfilter?

Yes it's definitely something which would is useful for us, and a wider audience for sure.

You're quite right... I was just looking through older things on the repo, and didn't look closely enough at the PR. This is a cool feature, I hope we can get it across the finish line.

Converting this to draft since it's not passing CI (yet!) but we'd love to see this marked as "ready for review" again if/when it does.


Forced contract version check in controller#push_transaction interface

EOSIO was designed to be able to easily update smart contract deployed on an account. While enjoying the convenience, it also introduced some troubles, some might be even dangerous.
Say Alice have deployed a token contract on the account alice, and created a token named alicetoken. Inside the contract implemented a function transfer to transfer token from an account to another:
void token::transfer( account_name from, account_name to, asset quantity,string memo )
{
    sub_balance( from, quantity );
    add_balance( to, quantity, payer );
}

it all looks good and everyone is happy.
However, suddenly Alice decided she need to take some “tax” from her beloved users, so she silently change the transfer function:
void token::transfer( account_name from, account_name to, asset quantity,string memo )
{
    sub_balance( from, quantity );
    add_balance( to, quantity, payer );

    // pay tax to Alice
    int64_t tax = (int64_t)(quantity.amount * 0.01);
    require_auth( from );
    add_balance( alice, tax, _self );
    sub_balance( from, tax );
}

now Alice is happy but others get hurt.
This is only an example based on token transfer in a simple token contract, but this issue exists in any kind of contract deployed on EOSIO system. Recently one incident already happened in real world: Everipedia made changes to their token contract, that charges transaction fee for whoever sent IQ.
Malicious group can intentionally change the contract to attack their users, cause chaos for the community. Even if it’s a planed contract upgrade, can still cause lost if users didn’t update their program in time to adapt to the contract.
A proposed solution is to enforce contract hash check:

while sending a action to a contract, must along sent that contract’s hash indicating which version of the contract you’re intend to talk to

inside  controller#push_transaction, before executing an action, verifies if the hash matches the contract’s hash on chain. If not, abort action execution immediately.

Looking deep into code base a little bit more, I figure it's more preferable to implement this mechanism in producer_plugin#on_incoming_transaction_async, with something like following:
if( fc::time_point(trx->expiration()) < block_time ) {
  send_response(std::static_pointer_cast<fc::exception>(std::make_shared<expired_tx_exception>(FC_LOG_MESSAGE(error, "expired transaction ${id}", ("id", id)) )));
  return;
 }

if( chain.is_known_unexpired_transaction(id) ) {
  send_response(std::static_pointer_cast<fc::exception>(std::make_shared<tx_duplicate>(FC_LOG_MESSAGE(error, "duplicate transaction ${id}", ("id", id)) )));
  return;
 }

This is a sorely missed feature and It would solve a lot of problems. Not only for trust but also for clients trying to manage the encoding. Currently many APIs just assume that any action will be decodable by whatever the latest ABI is, which fails in many cases.
I think a better approach would be to use the code_sequence and abi_sequence as versions instead of a hash of the code. It would occupy much less space on the wire and chain while serving the same function.
Something like:
{
  "account": "mytrustedbank",
  "name":  "deposit",
  "version": { "abi": 22, "code": 22 },
  "authorization": [ { "actor": "bobby", "permission": "active" } ],
  "data": { "from": "bobby", "quantity": "100000.0000 EOS" }
}

Closing in favor of #6168.

@jgiszczak These are separate issues, the one you linked is talking about accessing code versions from contracts while this one talks about specifying an expected version when submitting a transaction

6168 enables this one.  It inverts the logic, but the affect is the same.  Rather than submit a transaction with an expected version, the version is checked and the transaction is not submitted if the required version is not available.  Same result.
From outside a contract, the required version can already be checked for, and cheaply, since the get_code_hash API was implemented.

I don't see how would it solve the case where you want a guarantee that the transaction you submit executes on code you have verified or not at all. What if a code update happens between my get_code_hash call and the transaction execution? Or if the transaction is using delay_sec?


Remove disabled flag when generating function.json

Plugin name and version
azure-functions-maven
disabled property is not valid for v2. Please see https://github.com/Azure/Azure-Functions/issues/736 details on how to disable a function on V2.

@jdneo - Can you please take a look?Thanks!

cc @Flanker32


Is this project deprecated?

There is no more activity in this project since April 4th.
Was the project deprecated?

+1

Hi, I suppose our maintainers have been very busy with customer projects. I will attempt to take the ball on this in about 2 weeks from now and give the project some work it desperately needs.


aiohttp_skeletona webhook example not working ?

I can't seem to get the webhook example to work.
I start the example with
 $ python3.5 aiohttp_skeletona.py <mytoken> 8443 https://my.url/webhook/
And then from another console I do
 curl -v -k -H "Content-Type: application/json" -H "Cache-Control: no-cache"  -d '{
"update_id":10000,
"message":{
  "date":1441623432,
  "chat":{
     "last_name":"Test Lastname",
     "id":88162342,
     "type": "private",
     "first_name":"Test Firstname",
     "username":"usern"
  },
  "message_id":1365,
  "from":{
     "last_name":"Test Lastname",
     "id":1111111,
     "first_name":"Test Firstname",
     "username":"Testusername"
  },
  "text":"wei"
}
}' "https://my.url:8443/webhook/"

But I get:
*   Trying the.ip.ip.ip ....
* Connected to my.url (the.ip.ip.ip) port 8443 (#0)
* found 173 certificates in /etc/ssl/certs/ca-certificates.crt
* found 694 certificates in /etc/ssl/certs
* ALPN, offering http/1.1
* gnutls_handshake() failed: An unexpected TLS packet was received.
* Closing connection 0
curl: (35) gnutls_handshake() failed: An unexpected TLS packet was received.

And on the server side I get
======== Running on http://<IP_ADDRESS>:8443 ========
(Press CTRL+C to quit)
Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.5/dist-packages/aiohttp/web_protocol.py", line 278, in data_received
    messages, upgraded, tail = self._request_parser.feed_data(data)
  File "aiohttp/_http_parser.pyx", line 273, in aiohttp._http_parser.HttpParser.feed_data (aiohttp/_http_parser.c:4351)
aiohttp.http_exceptions.BadStatusLine: invalid HTTP method

Any ideas?

Did you manage to solve it? I get the same error and I do not understand why

may be you need turn on the https?


Fix support for old tags/refs where %(ref) comes out empty for some reason

Both in unexpected and assetgraph I noticed that releases from before some time in 2013 weren't listed in the changelog. For some weird reason that I didn't explore further, the git for-each-ref format we use here receives empty strings for the %(tag) and %(taggerdate:short) fields, eg.:
||||081240541e4edbca44377538225b1fa16a396b50

Instead of the expected format, eg.:
v1.6.0|2013-11-28|1729ef1f851a7ecf7106d18094675a92e5235316

Seems like we can get a tag anyway by asking for %(refname) instead and stripping off refs/tags/.
Here's how it affects various changelogs:

https://github.com/unexpectedjs/unexpected/commit/d090fd26c5f6cbbda643c0b0488e647c82632133
https://github.com/assetgraph/assetgraph/commit/49c32ddc131c7f7f57a5efe88b340e40d101c773
https://github.com/papandreou/node-cldr/commit/ca00d3eb71fe1641ad20e3f47d2ddf22860259d2

Released in 1.6.1


Using Inner Attributes makes it impossible to include code generation from build script.

Howdy.
I was working on testing running code-generation in a build script, which I feel is preferable for multiple reasons. You can have the code automatically re-run  at build time every time a file in the /prisma folder changes. With the benefit of it not being included in your actual /src folder so there's no way you can git commit or modify the file that gets code-generated.
I've got two crates in my workspace:

prisma-cli:  The standard CLI setup from the docs.
prisma-codegen: A lib crate containing the /prisma/schema.prisma file and a build.rs script.

In the build script, we're executing the prisma-cli command to codegen, and setting an env variable to change the output of the generated file to be in the /target directory.
build.rs
use std::{env, path::Path, process::Command};

fn main() {
    let out_dir = env::var("OUT_DIR").unwrap();
    let codegen_dest_path = Path::new(&out_dir).join("generated.rs");
    env::set_var("PRISMA_OUT_FILE", codegen_dest_path);
    env::set_var("CARGO_TARGET_DIR", &out_dir);
    let output = Command::new("cargo")
        .args(&["run", "-p", "prisma-cli", "generate"])
        .output()
        .expect("Failed run to codegen command");

    println!("cargo:rerun-if-changed=/prisma");
}

schema.prisma
generator client {
    // Corresponds to the cargo alias created earlier
    provider      = "cargo prisma"
    // The location to generate the client. Is relative to the position of the schema
    output        =  env("PRISMA_OUT_FILE")
}

lib.rs
include!(concat!(env!("OUT_DIR"), "/generated.rs"));

This all works. The problem is that the generated code uses  #![allow(warnings, unused)] which isn't allowed when using an include! to bring the generated code into our libraries context. Compiler error for reference:
error: an inner attribute is not permitted in this context
 --> /Users/dnorred/app/back-end/target/debug/build/prisma-codegen-dd61d46b0c3f1ab5/out/generated.rs:3:1
  |
3 |   #![allow(warnings, unused)]
  |   ^^^^^^^^^^^^^^^^^^^^^^^^^^^
4 | / pub static DATAMODEL_STR: &'static str =
5 | |     include_str!("/Users/dnorred/app/back-end/prisma-codegen/prisma/schema.prisma");
  | |____________________________________________________________________________________- the inner attribute doesn't annotate this static item
  |
  = note: inner attributes, like `#![no_std]`, annotate the item enclosing them, and are usually found at the beginning of source files
help: to annotate the static item, change the attribute from inner to outer style
  |
3 - #![allow(warnings, unused)]
3 + #[allow(warnings, unused)]
  |

Seems like this issue has happened with other projects that use code generation as well
https://github.com/google/flatbuffers/issues/6261
I think the fix would be to just replace it.

This might sound dumb, but I just replaced that line in the build script with the recommendation from the compiler error message and that fixed it.
I'm able to use the "empty" prisma-codegen crate without having to write any files to /src, and the contents automatically get re-generated whenever I modify the schema file.
main.rs from the application workspace crate:
use prisma_codegen::{new_client_with_url, PrismaClient};
use color_eyre::Result;

/// Create a new prisma client, and apply any pending migrations
async fn prisma_client(url: &str) -> Result<PrismaClient> {
    let client = new_client_with_url(url).await?;

    #[cfg(debug_assertions)]
    client._db_push().await?;
    #[cfg(not(debug_assertions))]
    client._migrate_deploy().await?;
    Ok(client)
}

build.rs after modifications:
use std::io::{BufRead, BufReader};
use std::{env, path::Path, process::Command};
use std::{fs, fs::File};

fn main() {
    let out_dir = env::var("OUT_DIR").unwrap();
    let codegen_dest_path = Path::new(&out_dir).join("generated.rs");
    env::set_var("PRISMA_OUT_FILE", &codegen_dest_path);
    env::set_var("CARGO_TARGET_DIR", &out_dir);
    let output = Command::new("cargo")
        .args(&["run", "-p", "prisma-cli", "generate"])
        .output()
        .expect("Failed run to codegen command");

    let mut lines = lines_from_file(&codegen_dest_path);

    for line in &mut lines {
        if line.contains("#!") {
            *line = line.replace("#!", "#");
        }
    }

    fs::write(&codegen_dest_path, lines.join("\n")).expect("Failed to update file");
    println!("cargo:rerun-if-changed=/prisma");
}

fn lines_from_file(filename: impl AsRef<Path>) -> Vec<String> {
    let file = File::open(filename).expect("no such file");
    let buf = BufReader::new(file);
    buf.lines()
        .map(|l| l.expect("Could not parse line"))
        .collect()
}

Huh, never knew that include! worked like that. I also never knew that if you put #[allow(...)] on a mod ...; declaration it'll do the same as the inner attributes I'm currently using.
I think I'm going to get rid of them all together and recommend that users add the attributes themselves, this way you can either add them on the mod ...; or inject them at the top of the file in your build script kind of like the text replacement you're already doing.

FWIW I've had absolutely zero issues with the build script since then. I think it's the preferable way to go since every cargo-check will make sure you've got the latest codegen from the prisma schema. One less thing to worry about being fragile.
Appreciate the change. Hopefully I can remove that extra bit from the build script 😅.


I cannot create shared object.

When I am working on #2287 , I got a CPU check failure.
I want to replicate the problem locally so that I can fix it.
python3 tvm/tests/python/unittest/test_module_load.py

What I got is:
Traceback (most recent call last):
  File "test_module_load.py", line 198, in <module>
    test_combine_module_llvm()
  File "test_module_load.py", line 192, in test_combine_module_llvm
    check_system_lib()
  File "test_module_load.py", line 179, in check_system_lib
    cc.create_shared(path_dso, [path1, path2])
  File "~/tvm/python/tvm/contrib/cc.py", line 33, in create_shared
    _linux_shared(output, objects, options, cc)
  File "~/tvm/python/tvm/contrib/cc.py", line 59, in _linux_shared
    raise RuntimeError(msg)
RuntimeError: Compilation error:
/usr/bin/ld: /tmp/tmpcOcGRh/myadd1.o: relocation R_X86_64_32S against `.rodata.cst4' can not be used when making a shared object; recompile with -fPIC
/tmp/tmpcOcGRh/myadd1.o: error adding symbols: Bad value
collect2: error: ld returned 1 exit status

and I followed this stackoverflow to see if .os are compiled with -fPIC.
https://stackoverflow.com/questions/19768267/relocation-r-x86-64-32s-against-linking-error
which shows that .os are compiled properly with -fPIC.

It is probably due to the llvm/gcc compatibility and not related to your change.
Solution and discussion:
https://discuss.tvm.ai/t/runtime-llvm-cc-create-shared-error-while-run-tune-simple-template/1037/5


#12979: Merge erisc data & bss sections

Ticket
https://github.com/tenstorrent/tt-metal/issues/12979
Problem description
This is a redo of https://github.com/tenstorrent/tt-metal/pull/15011, which started encountering errors after review. I'm not sure what caused that, but this one is passing tests.
The erisc app data-like sections are confused
What's changed

Have single .data and .bss output sections
Correctly clear all of bss before writing to it (in erisc-crt0.cc) (this was a doozy to find, because crt0 only worked because the small-data bss wasn't being cleared).
Fix rtos_context_switch_ptr and flag_disable definitions -- these need to be inline, as they are in a header file.

Checklist

[YES] Post commit CI passes
[ ] Blackhole Post commit (if applicable)
[ ] Model regression CI testing passes (if applicable)
[ ] Device performance regression CI testing passes (if applicable)
[ ] New/Existing tests provide coverage for changes

Hi @nathan-TT
https://github.com/tenstorrent/tt-metal/pull/15193 was merged recently
Not sure if it has any conflicts with this. I don't see any hardcoded addresses or sizes so it should be ok. just a heads up.

Hi @nathan-TT #15193 was merged recently Not sure if it has any conflicts with this. I don't see any hardcoded addresses or sizes so it should be ok. just a heads up.

thanks, they don't collide


Upgrade to latest BeyondCompare 4.2.9

This PR updates the version of Beyond Compare to 4.2.9.

Build passed...giving myself the old :shipit: .


Add new semantic token type for an identifier of keyword arguments

Feature request:
add new semantic token type for an identifier of keyword arguments when calling function.
I'd like to assign the new semantic token type to variable.parameter.function-call.python.
Situation
I use semantic highlighting. Assigning a specific color to parameters:python allows us to distinguish between newly defined variables in a function and parameters given as function arguments. (before: first picture, after(using semantic highlighting) second picture)
But, with this change, the identifier of the keyword argument when calling a function is also displayed in the same color.
Because the variable.parameter.function-call.python, which represents the identifier in the TextMate rule, was assigned a different color, it was previously displayed in a different color from the other parameters.(In the first picture, alpha, beta, gamma are displayed in indigo on main method, but second picture, after using semantic highlighting, they are displayed in brown as parameters) This is because semantic highlighting takes precedence over syntax highlighting.
Suggestion
I propose that assigning the new semantic token type to variable.parameter.function-call.python, such as selfParameter, or clsParameter.
①

②

Would really love if this was added, it's the one thing I miss from before I started using semantic highlighting

Would love this as well. Semantic highlighting is one of the few things that keeps me from dropping PyCharm (granted, PyLance has made it MUCH better in VSCode than it was previously in my opinion).
If not this specific change, then maybe a convenient way of excluding specific scopes from PyLance's semantic highlighting through settings.json so any changes made via "editor.tokenColorCustomizations" will no longer be suppressed?

Would love this as well. Semantic highlighting is one of the few things that keeps me from dropping PyCharm (granted, PyLance has made it MUCH better in VSCode than it was previously in my opinion).
If not this specific change, then maybe a convenient way of excluding specific scopes from PyLance's semantic highlighting through settings.json so any changes made via "editor.tokenColorCustomizations" will no longer be suppressed?

I'd also really appreciate this feature. To show an additional use case, having colored named parameters would make a huge difference for functions that pass values down a chain of responsibility and use the same parameter name for clarity.
Additionally, named parameters in decorators currently don't have the semantic type of parameter. While this is currently nice because it means my preferred color for named parameters can be applied inside decorators, it would be great if they also got this new semantic type to avoid needing separate TextMate coloring rule.

Moving this issue to discussion as an enhancement request for comments and upvotes.


Error on install and compile.

I did a go get of this package, and this error popped up.
# github.com/mholt/certmagic ../../mholt/certmagic/client.go:143:8: ht.ForceAttemptHTTP2 undefined (type *http.Transport has no field or method ForceAttemptHTTP2).
go version:
go version go1.10.4 linux/amd64
Edit:
So after some digging around, in go/src/golang.org/x/net/http2/transport.go, struct Transport does not declare ForceAttemptHTTP2, but looking at https://github.com/golang/go/blob/master/src/net/http/transport.go, struct Transport does declare ForceAttemptHTTP2.

Go1.10 was released in Feb 2018; it's probably time to upgrade.


Add rules_exclude setting

It would be nice if one could exclude certain rules in the settings.py file.
I didn't find a way of doing so, other than listing all the default rules (except for my excluded rules) in the "rules" array (settings.py) or THEFUCK_RULES variable.
Suggestion: add something like rules_exclude or THEFUCK_RULES_EXCLUDE.

exclude_rules and THEFUCK_EXCLUDE_RULES were added in 3.0.


Issue Deduplification

Detect similarities between Issues - http://stackoverflow.com/a/8897648/2578205

The article http://stackoverflow.com/a/8897648/2578205 calculates similarity with (tfidf * tfidf.T).A, and the author mentions it is cosine similiarity: http://scikit-learn.org/stable/modules/metrics.html#cosine-similarity .
There are many other similiarity pairwise metrics: http://scikit-learn.org/stable/modules/metrics.html
For example, Euclidian distances: http://scikit-learn.org/stable/modules/generated/sklearn.metrics.pairwise.euclidean_distances.html

Inter-Document Similarity with Scikit-Learn and NLTK: http://sujitpal.blogspot.ca/2013/05/inter-document-similarity-with-scikit.html

Thoughts on Software Development
Python/scikit-learn: Calculating TF/IDF on How I met your mother transcripts: http://www.markhneedham.com/blog/2015/02/15/pythonscikit-learn-calculating-tfidf-on-how-i-met-your-mother-transcripts/

How to see top n entries of term-document matrix after tfidf in scikit-learn: http://stackoverflow.com/questions/25217510/how-to-see-top-n-entries-of-term-document-matrix-after-tfidf-in-scikit-learn


[Android Editor] Editor displays blank screen on opening project with Mobile Renderer

Tested versions
Reproducible in Godot v4.4.dev (4254946de)
Not reproducible in Godot v4.4.dev2
System information
Godot v4.4.dev (4254946de) - Android 14 - Single-window, 1 monitor - OpenGL ES 3 (Compatibility) - Adreno (TM) 618 -  (8 threads) - Galaxy Tab S6
Issue description
On the current master branch, on opening/creating a project with the Mobile Renderer selected, the Godot Editor loads with the logo and loading wheel as usual, but then turns competely blank with a grey screen, and becomes unresponsive.
This does not happen with the Compatibility Renderer, or on 4.4.dev2 with the Mobile Renderer.
On master:

On 4.4.dev2:

Steps to reproduce
Create a new project or change renderer of existing project to Mobile Renderer.
Minimal reproduction project (MRP)
Just create a new project with Mobile renderer.

This might be the same as:

https://github.com/godotengine/godot/issues/97341

But without any further info it's hard to tell

I can confirm this is the same case as #97341, i get the same error and my bisect also points to the same culprit: #92797

Using adb logcat i get a spam of these errors:
ERROR: Number of uniform sets is larger than what is supported by the hardware (4).
at: shader_compile_binary_from_spirv (drivers\vulkan\rendering_device_driver_vulkan.cpp:3125)

ERROR: Attempting to bind a descriptor set (4) greater than what the hardware supports (4).
 at: draw_list_bind_uniform_set (servers\rendering\rendering_device.cpp:3937)

ERROR: This render pipeline requires (0) bytes of push constant data, supplied: (16)
at: draw_list_set_push_constant (servers\rendering\rendering_device.cpp:4051)

ERROR: Condition "shader_data.is_empty()" is true.
at: _compile_variant (servers\rendering\renderer_rd\shader_rd.cpp:309)

ERROR: Condition "rid.is_null()" is true. Returning: rid
at: _allocate_from_uniforms (.\servers/rendering/renderer_rd/uniform_set_cache_rd.h:130)

ERROR: No render pipeline was set before attempting to draw.
at: draw_list_draw (servers\rendering\rendering_device.cpp:4070)

ERROR: Attempted to use an unused shader variant (shader is null),
at: get_render_pipeline (.\servers/rendering/renderer_rd/pipeline_cache_rd.h:75)

ERROR: Parameter "shader" is null.
at: uniform_set_create (servers\rendering\rendering_device.cpp:2816)

ERROR: Parameter "uniform_set" is null.
at: draw_list_bind_uniform_set (servers\rendering\rendering_device.cpp:3947)

ERROR: Parameter "pipeline" is null.
at: draw_list_bind_render_pipeline (servers\rendering\rendering_device.cpp:3859)

Please add this comment in #97341
Closing this in favour of it


Added option to swap "Cast Bloom" and "Wield" on Silver sickle (b)

Should make collecting mort myre fungi easier.

I mostly play with my hands off-keyboard, so shift-click is not an option.

Use either sticky keys or the on screen keyboard and just toggle shift there


feat(dynamic forms): add description to elements

Do you want to request a feature or report a bug?
Feature Request
Add optional descriptions for elements in the form of either a hint or a hover.
Screenshots or link to CodePen/Plunker/JSfiddle
What is the expected behavior?
What is the motivation / use case for changing the behavior?
Which version of Angular and Material, and which browser and OS does this issue affect?
Other information

Isn't this a duplicate of:
https://github.com/Teradata/covalent/issues/826

Yup, duplicate


Added support for staticcheck linter

Fix #771
This is the second pr I open for this; the first one was a mistake, this one is the real one

The failed tests seem not to be linked to the new linter/new version of go (which I had to update to install staticcheck correctly).


Part solution to issue #4831

#Part solution to issue #4831
Hello, I hope this help you with the icon toggle examples that where mention in the issue.

Hey @MicaelaBenitez,
Thanks so much for taking the time to look into this and contribute.
Unfortunately, we were actually referring to the v2 icon toggle example. We wanted to demonstrate how our v2 icon toggles would look in the scenarios described in the issue.
If you're interested in helping us out with this, I'd be more than happy to help get you set up developing our v2 code. Otherwise, I'll have to close this PR. But thank you again for looking into this.

Hey @traviskaufman,
Thanks so much for taking your time and answer my pull request.
I'm interested in helping you with the project so I'll wait for an answer to get started with the new version.

I agree

De: googlebot<EMAIL_ADDRESS>Enviado: martes, 11 de octubre de 2016 01:06 p.m.
Para: google/material-design-lite
Cc: Micaela Benitez; Author
Asunto: Re: [google/material-design-lite] Part solution to issue #4831 (#4842)
Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
?? Please visit https://cla.developers.google.com/ to sign.
About - Google CLAhttps://cla.developers.google.com/
cla.developers.google.com
When you sign a Contributor License Agreement (CLA), you give Google the legal permission to use and distribute your contribution. You do not surrender ownership of ...
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA datahttps://cla.developers.google.com/clas and verify that your email is set on your git commitshttps://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the company's name.

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/google/material-design-lite/pull/4842#issuecomment-252963363, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AOg441h2ZSShO3mSZ167HH5HFC21zDOeks5qy7QfgaJpZM4KTz55.

@googlebot , I've change my email address and signed it.

Great! Closing this and let's follow up on the issue.


Can't clone scrinvex

Hi,
I want to use this great tool in my snRNA-seq analyses, However, I can't clone this tool and gives me these error messages:
Cloning into 'scrinvex'...
Permission denied (publickey).
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
any advice? Thank you so much!
Tao

solved, due to SHH keys.


test: fix assert.Check's argumets to show its parameters correctly

The change I made at db6075fc2 didn't show its parameters correctly.
Signed-off-by: Kazuyoshi Kato<EMAIL_ADDRESS>

We sadly cannot use govet to check this, since govet doesn't like the fact Check takes printf-like parameters. I've tried and got
snapshots/devmapper/snapshotter_test.go:139:2: printf: Check call has possible f
ormatting directive %d (govet)
        assert.Check(t, layer2Usage.Size > sizeBytes,
        ^
snapshots/devmapper/snapshotter_test.go:141:2: printf: Check call has possible formatting directive %d (govet)
        assert.Check(t, layer2Usage.Size < sizeBytes+256*dmsetup.SectorSize,
        ^


⚠ Bofrost.be has degraded performance

In c6fb7c7, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2641 ms

Resolved: Bofrost.be performance has improved in 906d8e2.


gnome-terminal gets pushed to background

Hi! I am running atom 1.22.0 on Ubuntu 17.10. If I launch atom-python-run with the terminal specified as gnome-terminal, gnome-terminal launches on the background. I'd like the terminal to pop up in the foreground, like xterm (or terminator).

Ubuntu 17.10 moved away from Unity and reverted back to Gnome. Gnome now uses version 3 by default and so does 17.10. I happen to use Archlinux with gnome3 and have the same issue, although, it's not neccessarily related to this plug-in.
It has to do with the shell and how certain windows in the evironment are focused. MS Windows is the only OS that seems to allow the terminal to take focus by default while Mac OS X and Gnome3 seem to push it to background. Not sure if any other systems have this issue as well.
Let me know if you figure it out. You can always create a fork, pull request, or post a fix here. Although, like I already stated, this issue seems to be related to how gnome3 handles focusing a window.
In short, I don't know how to change/fix it and I'm open to proactive solutions.

@FredhRodrigues -- README -- #69
Terminal opening in background (Ubuntu 16) #73

Hello guys. I was using everything without a problem, then I don't know if I hit the wrong key or changed the wrong lines in config.cson but now this started to happen. At first I thought f5 was not working anymore then I saw a dozen terminal windows in the back.
This is really annoying. Is there anyway to make it open in the foreground again? Thanks in advance.

Even though issue #73 takes place in Ubuntu 16.xx, which I find interesting because it uses Unity as its WM and DE, it can be cataloged here since it resembles this issue, #70. Issue #73 will be merged with issue #70 to help isolate this general issue.

I was not sure where to post it since it was in a different version of Ubuntu so I figured it would be best to start a new thread. I did check issue 70 before posting and I can use xterm but that doesn't explain why this started to happen to terminal so suddenly.

@FredhRodrigues Both Unity and Gnome use gnome-terminal as their default terminal. So it's okay. Hopefully, someone can help figure it out. I still don't have a solution. If one is resolved, the other won't be, and that will allow some focus to remain on that issue. It might even give a hint or a clue that could help resolve it.

Reinstalling Atom (purge and install) seemed to have solved the problem, everything is working fine now. Thanks for the help.


MINOR: Fix the doc of scheduled.rebalance.max.delay.ms config property

The doc of scheduled.rebalance.max.delay.ms config property was incorrectly the same as the one for the connect.protocol config
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

cc @rhauch
This is a small fix that we should backport to 2.3 branch to be included to future bugfixes and the next version of our docs.

The test failures were unrelated.


Build Logo

Find some simplistic logo for this project
Trying out stable difusion to generate one for us using Cyrano de Bergerac fencing in the style of mondrian

I think I like this one


Rewrite JDA readme

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[ ] Library interface (affecting end-user code)
[x] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This rewrites the current README of JDA. Since this README has been around for such a long time, it needed some rework for the current state of the library. I've also heavily reduced the size of the README.

Given the date, I'm unsure if the most recent commit is a serious one or not...
If it is, then I would like to share my opinion that emojis in headers aren't really nice. Especially in GitHub since 1) They are rendered based on the OS, making them inconsistent and 2) Make header-links less intuitive (i.e. #-overview instead of just #overview).
Not sure what the opinions of everyone else here is on that tho.

I suggest 🔧 (or 🛠) instead of🍴


Crash (No crash screen) on chapter open

Steps to reproduce
This only happens on one particular chapter of a particular entry, as far as I can tell. Debug logs attached.

Download Chapter
Open Chapter

Expected behavior
Chapter Loads
Actual behavior

Chapter load spinner starts
Entire app crashes (No crash "catch" screen), exits to home screen
The residual 'reading mode' toast (eg, left to right) appears
NOTE: Sometimes, it fails back to the entry instead of out of the app. I will investigate further. Both crash logs are now included.

Crash logs
Full crash:
mihon_crash_logs.txt
Entry failback:
mihon_crash_logs_menu.txt
Mihon version
0.17.1
Android version
Android 15
Device
Google Pixel 8 Pro
Other details
Verbose logging was ON for this crash log. I restarted the phone, re-downloaded the chapter. Opening in webview renders fine.
If you would like screencap of this, please @ me on discord
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.17.1.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

Considering you say this only happens for one particular entry, please ask your extension maintainers for support first. It may be the case that this extension in particular may need to be updated to account for website changes.
From my glance at the strange crash data, it appears Mihon is crashing because it breaks during image decoding. That usually points towards malformed/misinterpreted data from the source (wrong file type or something for example).


Support for multiple error messages

Here is the idea :
<input data-abide-validator="projectExists" pattern=".{3,10}" type="text" maxlength="10" required="true"/>
3 chars minimum, 10 maximum
A project holding the same acronym already exists!

@ stands for default abide validators

thanks,
but how we can dive a separate message like below,
required="true" -    this is required.
pattern=".{3,10}" -  3 chars minimum, 10 maximum
projectExists   -   A project holding the same acronym already exists!

We'll look at this for Foundation 6. There's probably some interesting things we can do with defining global error messages for validators, instead of having to define them on individual fields.

Any updates on this? I am thinking about whether to use Abide or not. I need this feature.

+1


feat: support 'pollux.domain' config param

coverage: 48.794% (+0.05%) from 48.743%
when pulling cab3d0b5e751f9f53f27d72e74d8c8b7477152a7 on feat/1422-make-domain-field-configurable-for-vc-issuance
into 7524ec7e12bcbcd1cf9bf0c44fe6c8d11abd8204 on main.


New app: chargent

see
https://github.com/notEvil/BangleApps/tree/chargent/apps/chargent
https://notevil.github.io/BangleApps/index.html?q=gently
Fyi I'm no expert and don't know the hardware ins and outs, so it could create false impressions (best practice, correctness, safety, ...). If I should state this in the readme, let me know.

Hi - thanks! I wonder how much this will really improve battery life, but I guess it's worth having as an option.
... also, this minifying is fine but I really wouldn't worry about minifying stuff too much in the future - the app loader does pretokenising automatically now, which will be almost as good, and it's easier to maintain :)

Hi - thanks! I wonder how much this will really improve battery life, but I guess it's worth having as an option.

I've been using http://www.accubatteryapp.com for a while now and they list their sources here:
https://accubattery.zendesk.com/hc/en-us/articles/210224725-Charging-research-and-methodology
Especially the section "Effect of topping off your charge" is the main motivation. (Probably should have looked it up earlier)
So in theory it should make a difference compared to charging for an extended period of time.

... also, this minifying is fine but I really wouldn't worry about minifying stuff too much in the future - the app loader does pretokenising automatically now, which will be almost as good, and it's easier to maintain :)

True, in this case there is not too much to gain from minification. In general however I really like not having to worry about the length of variable/function names or expression efficiency :)

Hi @notEvil - I believe this will also work on the Bangle2, except we want pin3 instead of pin30, according to these specs.
If that lines up with what you'd expect, I'll push up a PR

Sure, why not. I have no v2 to test, but if the PIN works the same and its a Li-ion, I don't see any reason why it shouldn't?


fix: desciptorId, typo and error export

Codecov Report

Merging #6 (6a58be9) into develop (91f5add) will increase coverage by 0.39%.
The diff coverage is 81.81%.

@@             Coverage Diff             @@
##           develop       #6      +/-   ##
===========================================
+ Coverage    87.22%   87.61%   +0.39%
===========================================
  Files            9       10       +1
  Lines          313      323      +10
  Branches        93       94       +1
===========================================
+ Hits           273      283      +10
  Misses          33       33
  Partials         7        7

Flag
Coverage Δ

unittest
87.61% <81.81%> (+0.39%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

lib/types/index.ts
100.00% <ø> (ø)

lib/issuer/WellKnownDidIssuer.ts
87.50% <50.00%> (ø)

lib/constants/Errors.ts
100.00% <100.00%> (ø)

lib/constants/index.ts
100.00% <100.00%> (ø)

lib/index.ts
100.00% <100.00%> (ø)

lib/utils/index.ts
91.66% <100.00%> (-0.34%)
:arrow_down:

lib/verifier/WellKnownDidVerifier.ts
80.50% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Fix test_cortex_m_corstone_300 ci job

Update path for new corstone300 downloads
Add missing benchmark arena size to generic benchmark test

BUG=Corstone300 Ci Job fails to run
Author: Ryan O'Shea<EMAIL_ADDRESS>

Hmm, I just noticed there was a patch put up to remove ethos u as coprocessor, but this patch fixes that issue if it was intended to use ethos u?

I would prefer option 1 as its very easy to skip the disclaimer and look straight at the numbers


chore(main): release 1.2.4

:robot: I have created a release beep boop
1.2.4 (2024-09-01)
Bug Fixes

saga in title for a couple shows/movies (97c57e7)

This PR was generated with Release Please. See documentation.

Summary by CodeRabbit

Bug Fixes

Resolved an issue with the saga title for several shows and movies, improving user experience.

Chores

Updated the changelog to include the latest version entry (1.2.4) for better clarity on changes and improvements.

:robot: Created releases:

v1.2.4
:sunflower:


Xterm webgl renderer?

Would it be possible to include the webgl addon for xterm? - https://www.jsdelivr.com/package/npm/xterm-addon-webgl
I'm getting 2fps when scrolling in a relatively large terminal window when xterm is set to use the basic DOM renderer
https://github.com/webmin/authentic-theme/assets/98313962/90b89ae0-4250-4ab0-a463-524b8c38c333
Also, the ability to enable and adjust the smoothScrollDuration option of Terminal(xterm.js) would be helpful - but that is just a personal preference.

Hello,
Sounds like a good improvement. I will take a closer look and update this ticket with progress ..

It's a huge improvement! It feels much better working in the terminal now - everything is super responsive. Thank you.

It's a huge improvement! It feels much better working in the terminal now - everything is super responsive.

I'm glad to hear that!

Thank you.

Also, the ability to enable and adjust the smoothScrollDuration option of Terminal(xterm.js) would be helpful - but that is just a personal preference.

Alright, this is also a good suggestion. I have added an ability to pass custom options to Xterm config. E.g., running in the browser console the following will get what you need:
config_portable_module_xterm_options = JSON.stringify({'smoothScrollDuration': '200'});
theme.config.save();

I have added an ability to pass custom options to Xterm config.
Icing on the cake. Thank you.


Fix cube.obj in advanced onboarding tutorial

any objs or test assets should be in litterbox where they can be tested in ci, not in this repo, cc @jgomez720

like all those code snippets and this need to be loaded from tested files, let me know if you need help, we should never have untested code in a tutorial

Superseded by https://github.com/KittyCAD/documentation/pull/107 (I copied the cube.obj)


perf: ⚡ access

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


(profile::core::i2c) fwv

@fritzm I'm going to make some changes to this pr per https://jira.lsstcorp.org/browse/IT-4905?focusedCommentId=4120809&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-4120809


Create a wiki page that shows the links to README.mds

Create a wiki page that shows the links to README.mds

Rough draft is created. There is room for improvement.
https://github.com/CPFL/Autoware/wiki/Readme-List


Bug in heading anchors

Sorry if this is the wrong repo to add this (very minor!) issue, but I was just trying to grab a deep link to a particular heading on https://digital-land.github.io/specification/specification/development-plan/ and the code is showing
<h3 id="{d[" dataset"]{}}-dataset"="" class="govuk-heading-m">
    Development plan
</h3>

so I can't grab the ID in order to link to it.

Hi @jennet - thanks for the heads up! You are definitely in the right place!
We've fixed the issue with commit 62c0397

perfect yep that's sorted it!


Update URL; point to Teams admin center, not main

I'm not sure how to reverse this URL into a fancy go.microsoft.com one, but the current URL and text points to the main admin center, not the Teams-specific one mentioned in the comments that follow.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@tonysmit PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Analyzer that checks for correct usage of model binding related attributes

Recommend BindAttribute over ModelBinderAttribute. See https://github.com/aspnet/Mvc/issues/6944.

We have multiple other issues covering this ask:

https://github.com/aspnet/Mvc/issues/7711
https://github.com/aspnet/Mvc/issues/7415


cannot open .ipynb at the first click

Environment data
Version: 1.51.1 (user setup)
Commit: e5a624b788d92b8d34d1392e4c4d9789406efe8f
Date: 2020-11-10T23:34:32.027Z
Electron: 9.3.3
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042
ipython 7.19.0 py38hc5df569_0 conda-forge
ipython_genutils 0.2.0 py_1 conda-forge
Type of virtual environment used: conda
Jupyter server running: Local
Expected behaviour
open a .ipynb file
Actual behaviour

upond opening a file it prompts "Cannot open resource with notebook editor type 'jupyter-notebook', please check if you have the right extension installed or enabled."
after clicking "Reopen file with VS Code standard text editor" it works

Steps to reproduce:

open a file

Thanks for reporting this issue. I believe we have resolved this in our current dev environment.
The fix will be in the next release.


How to Revoking and Deleting User Profiles

I need to know how to revoke or delete a user profile in KvikeT.

Hey Max,
Deleting users is not implemented right now. I've been running under the assumption that if you use Kviklet seriously you set up SSO and deleting the user in the SSO provider will prevent them from logging in. Which should usually be enough.
The problem with deleting the user is that comments, reviews and requests are attached to the user profile so deleting the author would mess with the auditlog and leave orphaned requests.
If you have password logins, it probably makes sense to simply remove all permissions from this user for now.
I will think about a solution where you can mark a user as disabled, preventing them from taking any action in the future but preserving requests, comments and reviews for auditing purposes.


Sometimes Account Balance, Equivalent Values show permanent loading

Sometimes Account Balance, Equivalent Values show permanent loading:

Not reproduced on v 3.0. Reproduces on v 3.2.

Reproduced on v3.2 and got
XMLHttpRequest cannot load https://rpc.myetherwallet.com/api.mew. No 'Access-Control-Allow-Origin' header is present on the requested resource. Origin 'https://www.myetherwallet.com' is therefore not allowed access. The response had HTTP status code 524.

in chrome's console.

fixed in new nodes that will be primary nodes in ~2wks.


tests: ec_host_cmd: Make sure expected_dut_to_host fits in buffer

If struct ec_response_too_big is too large, then the size of 'struct
tx_structure' is larger than the buffer that expected_dut_to_host
references, leading the compiler to generate a warning about the
possibility of buffer overflow.
Shrink the size of 'struct ec_response_too_big' so that the outer struct
fits within the 256 byte buffer.
This was caught when using GCC 11.3.0 building the test for native_posix.
Signed-off-by: Keith Packard<EMAIL_ADDRESS>

Excuse me, please correct me if I am wrong. Could we ignore this warning? It seems like the test intends to do this.

Excuse me, please correct me if I am wrong. Could we ignore this warning? It seems like the test intends to do this.

The tests are all built with -Werror, but we could add a pragma to ignore that in this case, but I'm not sure that would be better. Given that the over-sized member is never actually used, and the storage used by the operation is allocated separately, it doesn't really matter. I picked a solution that didn't depend on compiler-specific code.


Remove unnecessary canUseDOM check from Popover.

This was causing issues in on RR pages described in:
https://thumbtack.slack.com/archives/C0GKYQVTR/p1616022352008000
(Link is internal only.)
After creating a small reproduction, I realized that removing canUseDOM fixes the problem. I'm not 100% sure why, but I imagine it is related to React's client-side hydration. canUseDOM is always false on the server but defaults to true on the client. I imagine this isn't an issue with components like Tooltip because they default to being closed and require user interaction to appear.

FWIW I wasn't able to reproduce this on our documentation site. Not sure why that is. But I believe it's a safe change since I've tested it in website and also the SSR thumbprint.design site doesn't throw any SSR related errors after the change.


Minichart tooltip and popup

Hi there,
I'm wondering if there are plans to incorporate support for tooltips/popups and legends into leaflet minicharts? Some examples are shown in the R version (https://github.com/rte-antares-rpackage/leaflet.minichart).
Thanks for all of your hard work!

Unfortunately, the leaflet.minichart component seems to have been abandoned,
https://github.com/rte-antares-rpackage/leaflet.minichart
The current version is not compatible with the latest version of Leaflet, and I am thus currently considering if it should be included in dash-leaflet in the future or not.

In the new major version (1.0.0 or newer), Minichart is no longer included. If it is added again, please reopen the issue.


Update ProcessDefinitionServiceImpl.java

fix typos for logger.error

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #7470 (0813b70) into dev (b54482c) will decrease coverage by 0.05%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##                dev    #7470      +/-   ##
============================================
- Coverage     41.30%   41.25%   -0.06%
+ Complexity     3664     3657       -7
============================================
  Files           635      635
  Lines         26426    26426
  Branches       2959     2959
============================================
- Hits          10916    10901      -15
- Misses        14484    14500      +16
+ Partials       1026     1025       -1

Impacted Files
Coverage Δ

...api/service/impl/ProcessDefinitionServiceImpl.java
26.03% <0.00%> (ø)

...er/master/processor/queue/TaskResponseService.java
50.00% <0.00%> (-18.19%)
:arrow_down:

...rver/master/processor/queue/TaskResponseEvent.java
93.54% <0.00%> (-6.46%)
:arrow_down:

...org/apache/dolphinscheduler/remote/utils/Host.java
37.77% <0.00%> (-2.23%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.11% <0.00%> (-1.41%)
:arrow_down:

...r/plugin/registry/zookeeper/ZookeeperRegistry.java
55.85% <0.00%> (+7.20%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b54482c...0813b70. Read the comment docs.

considering that it's your first contribution, I think we can get deep communiction, you can contact me through mail or add wechat(KerwinZhuang), when mail or added, please tell me who you are, I think I can help to familiar with the DolphinScheduler if you meet with problems.


feat(normalize_features): Added function and tests

This function is used for normalizing each column of the dataset with the low and high quantile for each role.

Hi George, this is updated quantile normalization with different low and high quantile as discussed. Thanks.


Fix segfault when request is triggered without setting an url

Signed-off-by: Otavio Pontes<EMAIL_ADDRESS>

merged.


Add #[derive(Deserilaize, Serialize)] to struct Media.

Signed-off-by: Masaki Muranaka<EMAIL_ADDRESS>

@monaka merging is blocked because commit is not signed, could you please sign it and force push it?

done in e29eae7


HDP: Remove <IP_ADDRESS> and <IP_ADDRESS> and add <IP_ADDRESS>

Update to latest HDP versions. Removed <IP_ADDRESS> and <IP_ADDRESS> since they're no longer in the repository.

http://s3.amazonaws.com/public-repo-1.hortonworks.com/index.html#/HDP/centos6/2.x/updates

LGTM 👍


Clients will miss pending notifications upon startup if there is a delay between Monitor creation and client calling NotificationService.subscribe

Describe the bug
Clients will miss pending notifications upon startup if there is a delay between Monitor creation and client calling NotificationService.subscribe
Steps to reproduce
Introduce a delay (for example by waiting for sync to complete) before calling subscribe on NotificationService.
Expected behavior
Clients should not miss notifications
Screenshots
No response
Smartphones
No response
Were you using an atApplication when the bug was found?
No response
Additional context
Reported by a customer, encountered by them when using some sample code I had supplied which waited for an initial sync to complete before calling NotificationService.subscribe()

I have implemented an initial patch which minimizes the risk of this happening and customer has verified. Will raise PR for that
Have filed an enhancement request to fix this properly.

More urgent stuff came up. The patch fix is with the customer via a git branch, but needs a PR raised. Will do in PR50


chore(test/apply_fun): quiet test file

This is part of the process of quieting test files.  Since I was not really able to automate this change, I am PRing it separately.

This doesn't seem like a great approach to me; if any of the tactics being tested try to pull hypotheses from their context, these tests will break.
Can we just silence sorry in tests somehow? Or introduce a test_sorry axiom that is silent?

I am happy to introduce a test_sorry axiom.  I had used systematically #guard_msgs in #6868, but someone, I think Mario, did not like it.

Can you link to where Mario objected?

Let's wait for @digama0 to agree on that strategy before throwing away more work!

apply_fun was quieted in #6868.


Add support for sending messages to channels

Yesterday Telegram added support for channels in the API, so this should be implemented in botogram.
But I don't want to push changes without the ability to test them, and currently no Telegram client supports marking a bot as administrator of a channel.
Now that all the major clients supports adding bots as channel administrators, this can be implemented. The Bot.send method should be fairly easy to fix, but I also want to add an API for the ones who just want to manage the channel.
This is because creating a bot instance just for sending a message to a channel, maybe in an big script, is just a waste of resources. So I designed this API:
import botogram

chan = botogram.channel("@channel_name", "12345678:api_key")
chan.send("Hi there!")

botogram.channel will return an instance of botogram.Chat, so it won't trigger the initialization of the bot instance.

[x] Design the API
[x] Wait for Telegram clients to support this
[ ] Do the actual implementation
[ ] Write a chapter in the documentation

Updated the issue with an API design.

Merged two days ago.


fix: gradient clipping of FP16 optimizer

All-reduce over tensor parallel nodes should be ignored when there are no tensor parallel parameters.

Re-PR with a different source branch


fixed excessive loading of alignment data

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/mango-prb/106/
Test PASSed.


[Insets] Update README to deprecate Insets

Moved to the bottom and marked as deprecated

We may want to also mark https://github.com/google/accompanist/tree/main/insets as deprecated as well (and link to the migration docs)

Great point! Updated


[NRCE] Output Node API

input
sending data telegrams
the channel is the offset of your base id. you can send data on up to 127 channels. each channel represents a virtual device.
The "data" part is according to the eep specification
{
 "meta": {
   "eep": "a5-12-04",
   "channel": 3
   "type": "data",
 },
 "data": {
   "TMP": 0
 }
}

sending teach-in telegrams
in order to send commands to other devices, you have to pair your virtual device with the device you want to communicate with. Pairing is done by sending a teach-in telegram while the receiving device is in learn mode.
{
 "meta": {
   "eep": "a5-12-04",
   "channel": 3
   "type": "teach-in",
 },
 "teach-in":{
   "type": "4BS" // "4BS", "UTE", "RPS", "1BS"
 },
 "data": {
   "TMP": 0
 }
}

there is an option to forward the encoded telegrams to the output (for debugging).
if there is an error during the send operation, an error code will be logged to the console

I'm thinking about splitting the output node as I did for the input node. Introducing the counterpart of the actor node: an encoder node... but I have to think more about the idea

you can now send premade telegrams as strings...

API fixed


'torch module not found' when deploying in gcp app engine

I am trying to deploy a mmdetection app in gcp app engine and it takes the dependencies from requirements.txt file. The installation is failing at the mmdetection installation step with the error that 'torch module not found'. Moving the package name up/down in the requirements.txt file is having no effect, as I believe a dependency tree is created when building the environment via requirements.txt.
Is there any way to enforce an order of installation or any other work around or am I doing something wrong?
Error snippet as below:
Collecting git+https://github.com/open-mmlab/mmdetection.git (from -r requirements.txt (line 43))
Cloning https://github.com/open-mmlab/mmdetection.git to /tmp/pip-req-build-2pqo7f0m
INFO     pip_download_wheels had stderr output:
/env/lib/python3.7/site-packages/pip/internal/req/req_file.py:200: UserWarning: Disabling all use of wheels due to the use of --build-option / --global-option / --install-option.
cmdoptions.check_install_build_global(options, line.opts)
Running command git clone -q https://github.com/open-mmlab/mmdetection.git /tmp/pip-req-build-2pqo7f0m
ERROR: Command errored out with exit status 1:
command: /env/bin/python3.7 -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-req-build-2pqo7f0m/setup.py'"'"'; file='"'"'/tmp/pip-req-build-2pqo7f0m/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(file);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-l9p4d96
cwd: /tmp/pip-req-build-2pqo7f0m/
Complete output (5 lines):
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-req-build-2pqo7f0m/setup.py", line 5, in
import torch
ModuleNotFoundError: No module named 'torch'
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.
Requirements.txt as below:
pycocotools
torch==1.6.0
absl-py==0.10.0           # via tensorboard
addict==2.3.0             # via mmcv-full
albumentations==0.5.0     # via -r requirements.in
backcall==0.2.0           # via ipython
cachetools==4.1.1         # via google-auth
catalyst==20.10.1         # via -r requirements.in
certifi==2020.6.20        # via matplotlib, requests
chardet==3.0.4            # via requests
click==7.1.2              # via uvicorn
cycler==0.10.0            # via matplotlib
cython==0.29.21           # via -r requirements.in, mmpycocotools
decorator==4.4.2          # via ipython, networkx
deprecation==2.1.0        # via catalyst
fastapi==0.61.1           # via -r requirements.in
future==0.18.2            # via torch
gitdb==4.0.5              # via gitpython
gitpython==3.1.9          # via catalyst
google-auth-oauthlib==0.4.1  # via tensorboard
google-auth==1.22.1       # via google-auth-oauthlib, tensorboard
grpcio==1.32.0            # via tensorboard
h11==0.11.0               # via uvicorn
idna==2.10                # via requests
imageio==2.9.0            # via imgaug, scikit-image
imgaug==0.4.0             # via albumentations
importlib-metadata==2.0.0  # via markdown
ipython-genutils==0.2.0   # via traitlets
ipython==7.18.1           # via catalyst
jedi==0.17.2              # via ipython
joblib==0.17.0            # via scikit-learn
kiwisolver==1.2.0         # via matplotlib
markdown==3.3.2           # via tensorboard
matplotlib==3.3.2         # via catalyst, imgaug, mmdet, mmpycocotools, scikit-image
mmcv-full==1.1.5          # via -r requirements.in
git+https://github.com/open-mmlab/mmdetection.git --install-option="develop"
#mmdet
mmpycocotools==12.0.3     #
networkx==2.5             # via scikit-image
numpy==1.19.2             # via -r requirements.in, albumentations, catalyst, imageio, imgaug, matplotlib, mmcv-full, mmdet, opencv-python, opencv-python-headless, pandas, pywavelets, scikit-image, scikit-learn, scipy, tensorboard, tensorboard$
oauthlib==3.1.0           # via requests-oauthlib
opencv-python-headless==<IP_ADDRESS>  # via albumentations, mmcv-full
opencv-python==<IP_ADDRESS>   # via -r requirements.in, imgaug
packaging==20.4           # via catalyst, deprecation
pandas==1.1.3             # via catalyst
parso==0.7.1              # via jedi
pexpect==4.8.0            # via ipython
pickleshare==0.7.5        # via ipython
pycocotools
torchvision==0.7.0        # via -r requirements.in
tqdm==4.50.2              # via catalyst
traitlets==5.0.5          # via ipython
typing-extensions==<IP_ADDRESS>  # via uvicorn
urllib3==1.25.11          # via -r requirements.in, requests
uvicorn==0.12.2           # via -r requirements.in
wcwidth==0.2.5            # via prompt-toolkit
werkzeug==1.0.1           # via tensorboard
wheel==0.35.1             # via tensorboard
yapf==0.30.0              # via mmcv-full
zipp==3.3.1               # via importlib-metadata

If I use mmdet in my requirements.txt file then I get 'numpy not found' error

Hi @Rana-Banerjee
You may need to install PyTorch manually first.


fix invalid empty lists from cog init

The cog init command generates a boilerplate cog.yaml file with commented-out stanzas. This default output includes empty lists, which are not actually valid YAML.
# this is not valid because it's an empty list
foo:
  # - "bar"
  # - "baz"

This PR resolves that issue by commenting out the entire stanza, leaving it up to the user to selectively uncomment or remove the default config as needed.
Resolves https://github.com/replicate/cog/issues/468

Should we add a test that runs cog init followed by cog build to make sure that cog init doesn't break in the future?

Good idea! Added a test in https://github.com/replicate/cog/pull/493/commits/8ee94cddd1755fff4dbfde8f5f3e94343b785c0f. This is ready for another look.

The problem I mentioned above is here: #368


Docker v1.12 can not change hostname by env variable $HOSTNAME

Description
With previous docker versions, the command of "hostname" and "echo $HOSTNAME" returned same thing, after I changed it by sending an env variable into the image.
Why I use this is that I want to insert a new host into /etc/hosts by invoking hostname or $HOSTNAME. So my service/app inside this docker can get the 'real' IP address or the one I assigned, while not the docker internal one.
In the previous docker versions, I can change the hostname by assign env variable HOSTNAME, this will change the hostname correspondingly. But now, with docker version 1.12. This kind of linux action doesn't work in that linux behavior.
Steps to reproduce the issue:

prepare Dockerfile and docker-compose.yml
compose like this:

version: '2'

services:
  test:
    build: .
    extra_hosts:
        thisservice: "<IP_ADDRESS>"
    environment:
      - HOSTNAME=thisservice
    image: dockerv1.12-test

inside the docker container, cat /etc/hosts looks like this:
<IP_ADDRESS>	localhost
::1	localhost ip6-localhost ip6-loopback
fe00::0	ip6-localnet
ff00::0	ip6-mcastprefix
ff02::1	ip6-allnodes
ff02::2	ip6-allrouters
<IP_ADDRESS>	thisservice
<IP_ADDRESS>	80260fa6667f

exec into the container by: docker-compose exec test sh, run commands and check output:

$ hostname
80260fa6667f

$ echo $HOSTNAME
thisservice

Describe the results you received:
From docker v1.12, the hostname command and $HOSTNAME are different.
Describe the results you expected:
the hostname command and $HOSTNAME should be same. So that I can make those services in this container can easily pick the ip corresponding to the hostname.
Additional information you deem important (e.g. issue happens only occasionally):
In the above case, the service in the container will have no idea about the IP I assigned but go for the docker internal IP. This is not what I want. And not correct behavior in my opinion.
Output of docker version:
version 1.12.1
(paste your output here)

Output of docker info:
(paste your output here)

Additional environment details (AWS, VirtualBox, physical, etc.):

@crossz
IIRC the --hostname option sets the container's hostname.
$ docker run -it --hostname myhostname fedora bash
[root@myhostname /]# cat /etc/hosts
<IP_ADDRESS>       localhost
::1     localhost ip6-localhost ip6-loopback
fe00::0 ip6-localnet
ff00::0 ip6-mcastprefix
ff02::1 ip6-allnodes
ff02::2 ip6-allrouters
<IP_ADDRESS>      myhostname
[root@myhostname /]# cat /etc/hostname
myhostname

I don't think Docker has ever done this. Perhaps compose did? cc @shin- any ideas?

@justincormack I tried on both Ubuntu 16.04 and Centos 7.2; Before I upgrade docker v1.10, both distro worked as expected. Both went 'wrong' after upgrade docker to v1.12.1 and v1.12.3.
My services, similar to those registered on zookeeper regisitry, have to find the hostname and get its ip address. Previously all good by the way I showed above. But after upgrade to v1.12, hostname (coomand) aslways points to docker internal IP. This is not what I want.

@ripcurld00d I did not mean how to assign hostname (when container created). They way I described is to pass a customized hostname and corresponding IP, which are different from that automatically created, i.e. container id as hostname and container internal ip.

In another words, How to make the external host IP address assigned to container hostname?

i hava the same problem. how to set hostname when build docker image?

@justincormack @crossz looks like something did change between docker 1.10, and 1.11, although I'm not sure if setting $HOSTNAME should set hostname, so possibly it was a bugfix;
yum remove -y docker-engine
yum install -y docker-engine-1.10.3-1.el7.centos.x86_64
systemctl start docker

docker run -it --rm --add-host "thisservice:<IP_ADDRESS>" -e HOSTNAME=thisservice alpine sh -c "echo \$HOSTNAME && hostname"
thisservice
thisservice

Switching to 1.11.2;
yum remove -y docker-engine
yum install -y docker-engine-1.11.2-1.el7.centos.x86_64
systemctl start docker

docker run -it --rm --add-host "thisservice:<IP_ADDRESS>" -e HOSTNAME=thisservice alpine sh -c "echo \$HOSTNAME && hostname"
thisservice
c855297c6608

However, setting --hostname works;
docker run -it --rm --add-host "thisservice:<IP_ADDRESS>" -e HOSTNAME=thisservice --hostname thisservice alpine sh -c "echo \$HOSTNAME && hostname"
thisservice
thisservice

My services, similar to those registered on zookeeper regisitry, have to find
the hostname and get its ip address. Previously all good by the way I showed
above. But after upgrade to v1.12, hostname (coomand) aslways points to docker
internal IP. This is not what I want.

So hostname is the hostname of the container. The IP-address of the
container is the IP address on the internal Docker network; as far as I can
see, docker is working correct here.
However, for your use case, you want to publish the IP address that the container
is externally reachable on in the zookeeper registry. This IP address is different from the
container's IP-address, and can be even different from the host's IP adress
(for example, if there is a proxy in front of the host, or  in Swarm mode, where
a service is reachable on any node in the swarm).
For swarm-mode itself, we added an --advertise-addr option, for a similar reason;
the IP address that a node is reachable on may not be known to the host (if
it's accessible through a proxy only).
I think there's a number of solutions here;

Don't have your service use the hostname to register the service. Have an extra parameter, to pass (e.g.) the FQDN of the host that it's running on.
If the service only has to be reachable by other services (and not publicly on the internet), don't -p / --publish it's ports. Other containers on the private docker network can reach the service by it's name.
Run the container in --network=host so that the container runs in the hosts's
network stack, and hostname of the container matches the hostname of the host.

Option 1. or 2. is probably the best solution (depending on your use case), as it matches what you're trying to achieve; publishing a service's externally reachable IP-address.


Request failed status 429

Hi,
I I am trying to use this feature after installed and it is not working, I get "Request failed status 429", I followed the steps described,

What is your Plugin Version?
If it's matched with latest version, can you try to click refresh button, then click new chat button?

I tried that but did not work

@gastontancredi which obsidian version you are in? I am on Obsidian 1.4.1 preview and   plugin ver 3.6 (it was added to Obsidian community store) and it works for me

@danieltomasz my version is 1.3.7 in both linux and windows, downloaded from here https://obsidian.md/download
just updated today to the versions available there and didn´t work

Hey I reinstalled from community plugins, now I see it available there, and used default browser and worked on linux https://i.imgur.com/9ocnrQY.png

I'm glad to see it works! Thanks for using the plugin!
I'm really sorry that I'm unable to reply to you, I have been busy lately.
On Thu, Aug 3, 2023 at 3:34 AM gastontancredi @.***>
wrote:

[image: image]
https://user-images.githubusercontent.com/67171481/257935357-4d1532f8-f538-4db9-b98a-693882d0fef0.png
—
Reply to this email directly, view it on GitHub
https://github.com/Aldhanekaa/GoogleBardObsidian/issues/3#issuecomment-1662931595,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQFGVGWWPHNOKUROFN53O7DXTK2WFANCNFSM6AAAAAA2X2DJIQ
.
You are receiving this because you commented.Message ID:
@.***>


RDKCOM-4775: RDKBDEV-2644: fix return value from void

Reason for change:
fix return value from void function WanMgr_UpdateIpFromCellularMgr() Test Procedure: Sanity.
Risks: None.
Signed-off-by: Andre McCurdy<EMAIL_ADDRESS>Priority: P1
Change-Id: I3021ff5d4baa3163d5c412290105e429b4baf95d

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

build-rpi4-broadband-kirkstone Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/3c0832a2df773b64b46c36d72bc5cc06

Job started https://jenkins.code.rdkcentral.com/job/manual-verification/4046/

Job started https://jenkins.code.rdkcentral.com/job/manual-verification/4047/

Sanity tests for ['RPI4 Mediaclient'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/49472/

Sanity tests for ['RPI4 GW BB Kirkstone'] added.
https://jenkinsrdkm.cmf.code.rdkcentral.com/job/test-components-flow/49473/


chore(i18n): update onboarding language strings

Description:
Update translation strings for onboarding screens.
Motivation and Context:
Fix #1039
How Has This Been Tested?
Manually
Types of changes:
Language changes
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[ ] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Tested ACK cec3ce7
however, there is a typo in startup screen where create is written crearte
see screenshot

@lieteau2  I've fixed the typo and updated these strings to make them read a little more consistently across the different options.

Tested ACK c05eb1c


Production mode is off in the documentation

On the guide (see here for instance), Vue production mode has not been activated, you can see that warning:
You are running Vue in development mode.
Make sure to turn on production mode when deploying for production.
See more tips at https://vuejs.org/guide/deployment.html

This is intended, because we want to allow users to play with the devtool.

Ok but then why not just hide that noisy tip using Vue.config.productionTip = false?
This won't prevent the devtool right?
cf. relevant documentation

That'd be fine with me. 🙂 PR welcome!

I would create a PR if only I knew where the Vue initialization scripts were! :)


Issue compiling typescript

I created a fork of this repo and am trying to customize the scripts for my use case and I'm having trouble compiling the typescript files.
I have never used typescript before so I'm probably missing some setup step. I have looked online but can't find any resources on how to setup an existing project, just how to create a new one.
What I've done so far:

run npm install --save-dev
run ./node_modules/.bin/tsc

What I get is this:
node_modules/@actions/github/lib/github.d.ts:5:37 - error TS2507: Type 'typeof import("/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index")' is not a constructor function type.

5 export declare class GitHub extends Octokit {
                                      ~~~~~~~

node_modules/@actions/github/lib/github.d.ts:7:52 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'Options'.

7     constructor(token: string, opts?: Omit<Octokit.Options, 'auth'>);
                                                     ~~~~~~~

src/canMerge.ts:3:49 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'PullsGetResponse'.

3 export function canMergeByMergeable(pr: Octokit.PullsGetResponse) {
                                                  ~~~~~~~~~~~~~~~~

src/canMerge.ts:6:54 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'PullsGetResponse'.

6 export function canMergeByMergeableState(pr: Octokit.PullsGetResponse) {
                                                       ~~~~~~~~~~~~~~~~

src/canMerge.ts:10:17 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'PullsGetResponse'.

10     pr: Octokit.PullsGetResponse,
                   ~~~~~~~~~~~~~~~~

src/isEnabledForPR.ts:3:17 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'PullsGetResponse'.

3     pr: Octokit.PullsGetResponse | Octokit.PullsListResponseItem,
                  ~~~~~~~~~~~~~~~~

src/isEnabledForPR.ts:3:44 - error TS2694: Namespace '"/Users/gustavoambrozio/Development/merge-pal-action/node_modules/@octokit/rest/index"' has no exported member 'PullsListResponseItem'.

3     pr: Octokit.PullsGetResponse | Octokit.PullsListResponseItem,
                                             ~~~~~~~~~~~~~~~~~~~~~

src/mergeIfReady.ts:13:29 - error TS2339: Property 'pulls' does not exist on type 'GitHub'.

13     const pr = await client.pulls.get({
                               ~~~~~

src/mergeIfReady.ts:29:22 - error TS2339: Property 'pulls' does not exist on type 'GitHub'.

29         await client.pulls.merge({
                        ~~~~~

src/pushHandler.ts:13:36 - error TS2339: Property 'pulls' does not exist on type 'GitHub'.

13     const openedPrs = await client.pulls.list({
                                      ~~~~~

src/pushHandler.ts:26:24 - error TS2339: Property 'pulls' does not exist on type 'GitHub'.

26             if (client.pulls.updateBranch({
                          ~~~~~

src/statusHandler.ts:14:20 - error TS2339: Property 'pulls' does not exist on type 'GitHub'.

14             client.pulls.list({
                      ~~~~~

Found 12 errors.

I'd appreciate any help in getting me started on this. And thanks for creating this action!
Cheers,
Gustavo

As reference if someone else is interested, to get this to compile for me I had to change a lot of places that had import Octokit = require('@octokit/rest') and change it to import { Octokit } from '@octokit/rest', including in code that is in node_modules. I also uncommented "allowSyntheticDefaultImports": true in tsconfig.json

hey @gpambrozio, package.json includes script "build" which you can run with npm run build or yarn build in order to produce runnable version in a single file


Required dependencies missing from flydrive plugin

packages/plugins/flydrive/src/internal/plugin.ts
contains:
import { FSDriver } from "flydrive/drivers/fs"
import { GCSDriver } from "flydrive/drivers/gcs"
import { S3Driver } from "flydrive/drivers/s3"

but these are not dependencies, so the plugin fails to build.

@ahmedrowaihi  let me know if you need more context

Fresh create-oberon-app:
1)
pnpx create-oberon-app --email<EMAIL_ADDRESS>--recipe nextjs --database turso --send custom oat
cd oat
pnpm add @oberoncms/plugin-flydrive flydrive

// oberon/adapter.ts
import "server-cli-only";

import { initOberon } from "@oberoncms/core/adapter";
import { authPlugin } from "@oberoncms/core/auth";
import { plugin as developmentPlugin } from "@oberoncms/plugin-development";

import { plugin as databasePlugin } from "@oberoncms/plugin-turso";
import { plugin as sendPlugin } from "./send";

import { FSDriver } from "flydrive/drivers/fs"; // <- import the driver you want to use

import { getFlyDrivePlugin } from "@oberoncms/plugin-flydrive/plugin";

import { config } from "./config";

const fsDriver = new FSDriver({
  visibility: "private",
  location: "./fly",
});

// generate the plugin & api, and export the router to use it in the api
export const { flyDrivePlugin, initFlyDriveRouter } =
  getFlyDrivePlugin(fsDriver);

export const { adapter, handlers } = initOberon({
  config,
  plugins: [
    developmentPlugin,
    databasePlugin,
    sendPlugin,
    flyDrivePlugin,
    authPlugin,
  ],
});

pnpx prebuild

><EMAIL_ADDRESS>prebuild C:\Gits\oat
> tsx ./oberon/prebuild.ts

node:internal/process/esm_loader:34
      internalBinding('errors').triggerUncaughtException(
                                ^
Error [ERR_MODULE_NOT_FOUND]: Cannot find package '@google-cloud/storage' imported from<EMAIL_ADDRESS>    at packageResolve (node:internal/modules/esm/resolve:854:9)
    at moduleResolve (node:internal/modules/esm/resolve:927:18)
    at defaultResolve (node:internal/modules/esm/resolve:1157:11)
    at nextResolve (node:internal/modules/esm/hooks:866:28)
    at resolve (file:///C:/Gits/oat/node_modules/.pnpm/tsx@4.13.3/node_modules/tsx/dist/esm/index.mjs?1719220855449:2:3806)
    at nextResolve (node:internal/modules/esm/hooks:866:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:304:30)
    at handleMessage (node:internal/modules/esm/worker:196:24)
    at Immediate.checkForMessages (node:internal/modules/esm/worker:138:28)
    at process.processImmediate (node:internal/timers:478:21) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v20.12.2
 ELIFECYCLE  Command failed with exit code 1.


iOS 16 - “Exception Alert ERROR: unable to download video data: HTTP Error 403: Forbidden“

To reproduce:

Copy any link
Launch SW-DLT
ERROR Result
“Exception Alert ERROR: unable to download video data: HTTP Error 403: Forbidden“

There’s a comment on Routinehub mentioning the same issue. Been about 3ish days.
Just a heads up it seems down for YT. Using a-shell mini rather than the full, though I don’t think that’s any issue.

Doing the same thing on full as well.

Can you confirm if the error continues after following these steps? https://github.com/net00-1/SW-DLT/blob/master/Docs.md#updating-utilities
I really need to find a way to upgrade yt-dlp/gallery-dl in the background. I think with the new a-shell background commands it is now possible without disrupting the usability.

Can you confirm if the error continues after following these steps? https://github.com/net00-1/SW-DLT/blob/master/Docs.md#updating-utilities
I really need to find a way to upgrade yt-dlp/gallery-dl in the background. I think with the new a-shell background commands it is now possible without disrupting the usability.

This solution fixes the shortcut. Thank you for pointing this out.
Yeah that’s a tough one to crack. I’ve seen some shortcuts mention updating before use every once in a while.

Can you confirm if the error continues after following these steps? https://github.com/net00-1/SW-DLT/blob/master/Docs.md#updating-utilities
I really need to find a way to upgrade yt-dlp/gallery-dl in the background. I think with the new a-shell background commands it is now possible without disrupting the usability.

I did that but still kept saying the errors

Can you post the site link that gives you this error? perhaps authentication is required


Only include user-facing fields in zipped version of anaconda-project

From talking with Rachel, it seems that the extra fields in the anaconda-project are confusing to people and hinder the legibility of the files. This PR deals with the anaconda-project.yml that ends up in the zip file. It strips out two different sets of fields:

fields that we use the build the site, but anaconda-project doesn't know about.
commands and env specs used in testing that aren't user facing.

Original:
# To reproduce: install 'anaconda-project', then 'anaconda-project run'
name: boids
description: Boids models of swarm intelligence using HoloViews
created: 2015-01-01
maintainers:
  - philippjfr
labels:
  - holoviews

channels: []

dependencies: &deps
  - python=3.6
  - notebook=5.7.8
  - ipykernel=5.1.0
  - nomkl
  - bokeh=1.2.0
  - holoviews=1.12.3
  - numpy=1.16.4

packages: *deps

commands:
  notebook:
    notebook: boids.ipynb
  test:
    unix:    pytest --nbsmoke-run -k *.ipynb --ignore envs
    windows: pytest --nbsmoke-run -k *.ipynb --ignore envs
    env_spec: test
  lint:
    unix:    pytest --nbsmoke-lint -k *.ipynb --ignore envs
    windows: pytest --nbsmoke-lint -k *.ipynb --ignore envs
    env_spec: test

variables: {}
downloads: {}

env_specs:
  default: {}
  test:
    packages:
      - nbsmoke=0.2.8
      - pytest=4.4.1

Version in zip file:
name: boids
description: Boids models of swarm intelligence using HoloViews
channels: []
dependencies: &id001
- python=3.6
- notebook=5.7.8
- ipykernel=5.1.0
- nomkl
- bokeh=1.2.0
- holoviews=1.12.3
- numpy=1.16.4
packages: *id001
commands:
  notebook:
    notebook: boids.ipynb
variables: {}
downloads: {}
env_specs:
  default: {}

Note that the only "unusual" thing that I am leaving in is the handling of dependcies/packages. That part will also go away once a newer anaconda-project makes its way into AE.

Ok.  Is it necessary to have all the empty fields (channels, variables, downloads)?  We might maintain a skeleton somewhere that includes those, but I wouldn't think they should be in each YML if they are empty like that.  Is there a good reason that &deps changed to the less readable &id001?

I can get rid of empty fields, but there is no way to maintain that &deps using pyyaml. It would be possible using ruamel.yaml, but I am hesitant to add that dependency since anaconda-project requires pyyaml.
ref: https://stackoverflow.com/questions/37413345/yaml-anchors-definitions-loading-in-pyyaml

name: boids
description: Boids models of swarm intelligence using HoloViews
packages: &id001
- python=3.6
- notebook=5.7.8
- ipykernel=5.1.0
- nomkl
- bokeh=1.2.0
- holoviews=1.12.3
- numpy=1.16.4
dependencies: *id001
commands:
  notebook:
    notebook: boids.ipynb
env_specs:
  default: {}

Ok, thanks.


Projects - Social DB Integration

Projects are stored like this
Tasks are stored like this

Note: I have to update the structure as discusses in planning session but Godwill also needs to work on it, so merging it as it is, will create another PR.


Cleaning up frame ownership when passed around

We used to allocate frames and send them down as reference arguments to methods. Most of the methods were stealing (std::move) the guts from the instances. For this case we should send the frames down as r-value references or values to cleanly define the ownership.
I decided for the r-value references to avoid accidental copies.

LGTM


Installation and configuration steps on Linux

As of version 3.4.0 it is possible to install the service on a Linux machine. Unfortunately I couldn't find any installation (with configuration) steps or any software dependencies for using passcore on Linux with apache2.
To make this more clear. Does using the Linux version means that I can configure the passcore to work on my Linux-Server with apache2 or does passcore only work using Windows 10 Pro or WIndows Server with IIS?

@geoperez Thank you for the link. So basically I must install the dotnet core 2.1.X runtime (maybe build) on my Linux to be able to run the Releases. I saw some other ASP.net app tutorial that uses the dotnet command to start a .dll file that starts a localhost Server on some port, I guess that I have to do the same on my Linux.
Could you please provide me with the command sequence that I need to use to start the passcore app locally?

Yes, you need dotnet exe to run the PassCore Dll. If you check this part (https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/linux-apache?view=aspnetcore-2.1&tabs=aspnetcore2x#monitoring-the-app)
You need to call like PassCore something like this:
[Unit]
Description=PassCore

[Service]
WorkingDirectory=/var/aspnetcore/passcore
ExecStart=/usr/local/bin/dotnet /var/aspnetcore/passcore/passcore.dll
Restart=always
# Restart service after 10 seconds if the dotnet service crashes:
RestartSec=10
KillSignal=SIGINT
SyslogIdentifier=dotnet-example
User=apache
Environment=ASPNETCORE_ENVIRONMENT=Production

[Install]
WantedBy=multi-user.target

@geoperez Thanks for the support, it worked. Do I have to setup the appsettings.json to work with my Microsoft AD or is it similar to a Windows passcore installation? I mean if my Linux server is domain-joined then passcore works out of the box.

Yes, you need to fill your AD/LDAP server settings in the appseting.json before to use PassCore.

@geoperez Thanks for the help

@CleanHit Which .dll are you pointing to to run PassCore on Linux? I am attempting on Ubuntu 18.04 but running dotnet Unosquare.PassCore.Web.dll gets me
Error:   An assembly specified in the application dependencies manifest (Unosquare.PassCore.Web.deps.json) was not found:     package: 'Microsoft.AspNetCore.Antiforgery', version: '2.2.0'     path: 'lib/netstandard2.0/Microsoft.AspNetCore.Antiforgery.dll'
Dotnet is on version 2.2.4.


Lowercase diagnostic and Fix-It messages and remove "after " clause from missing node diagnostics

Remove "after " clause from missing node diagnostics

The “after” clause doesn’t seem to be generally useful. We can always add it back in special cases where it’s useful

Lowercase diagnostic and Fix-It messages

This matches the capitalization of messages in the compiler

@swift-ci Please test


DPXDT Behind Proxy

Hi,
We've just had our proxy updated, and DPXDT no longer works for me, because I am unable to login to the proxy via command line, and idea of a way round this?

Sadly this is not the solution. I am behind a ZScaler proxy, I have set the HTTP_PROXY and HTTPS_PROXY manually, and still phantom refuses to see anything other than local.
Please help
Darren