Help with kubernetes provider

Hello community,
I'm trying to import resources from a GKE cluster with kubernetes provider.
Starting simple I've used the code below
provider "kubernetes" {
  config_path = "~/.kube/config"
}

and I'm able to import resource using terraform import
$ terraform import kubernetes_namespace.default default
kubernetes_namespace.default: Importing from ID "default"...
kubernetes_namespace.default: Import prepared!
  Prepared kubernetes_namespace for import
kubernetes_namespace.default: Refreshing state... [id=default]

Import successful!

The resources that were imported are shown above. These resources are now in
your Terraform state and will henceforth be managed by Terraform.

Using terraformer I have issue with authentication instead
$ ./terraformer-kubernetes-linux-amd64 import kubernetes --resources=deployments -v

2021-03-08T10:54:09.944+0100 [INFO]  plugin: configuring client automatic mTLS
2021-03-08T10:54:10.020+0100 [DEBUG] plugin: starting plugin: path=/home/igpix/.terraform.d/plugins/linux_amd64/terraform-provider-kubernetes_v2.0.2_x5 args=[/home/igpix/.terraform.d/plugins/linux_amd64/terraform-provider-kubernetes_v2.0.2_x5]
2021-03-08T10:54:10.021+0100 [DEBUG] plugin: plugin started: path=/home/igpix/.terraform.d/plugins/linux_amd64/terraform-provider-kubernetes_v2.0.2_x5 pid=2289342
2021-03-08T10:54:10.021+0100 [DEBUG] plugin: waiting for RPC address: path=/home/igpix/.terraform.d/plugins/linux_amd64/terraform-provider-kubernetes_v2.0.2_x5
2021-03-08T10:54:10.085+0100 [INFO]  plugin.terraform-provider-kubernetes_v2.0.2_x5: configuring server automatic mTLS: timestamp=2021-03-08T10:54:10.085+0100
2021-03-08T10:54:10.154+0100 [DEBUG] plugin: using plugin: version=5
2021-03-08T10:54:10.154+0100 [DEBUG] plugin.terraform-provider-kubernetes_v2.0.2_x5: plugin address: network=unix address=/tmp/plugin941342150 timestamp=2021-03-08T10:54:10.154+0100
2021-03-08T10:54:10.247+0100 [TRACE] plugin.stdio: waiting for stdio data
2021-03-08T10:54:10.305+0100 [INFO]  plugin.terraform-provider-kubernetes_v2.0.2_x5: 2021/03/08 10:54:10 [WARN] Invalid provider configuration was supplied. Provider operations likely to fail: invalid configuration: no configuration has been provided, try setting KUBERNETES_MASTER environment variable: timestamp=2021-03-08T10:54:10.305+0100
2021/03/08 10:54:10 kubernetes importing... deployments
2021/03/08 10:54:11 Unauthorized
2021/03/08 10:54:11 kubernetes error importing deployments, err: kubernetes: deployments not supported resource
2021/03/08 10:54:11 kubernetes Connecting....
2021-03-08T10:54:11.307+0100 [DEBUG] plugin.stdio: received EOF, stopping recv loop: err="rpc error: code = Unavailable desc = transport is closing"
2021-03-08T10:54:11.309+0100 [DEBUG] plugin: plugin process exited: path=/home/igpix/.terraform.d/plugins/linux_amd64/terraform-provider-kubernetes_v2.0.2_x5 pid=2289342
2021-03-08T10:54:11.309+0100 [DEBUG] plugin: plugin exited

If I'm using provider configuration as suggested here for GKE, I get the following error instead:
Error: Attempted to load application default credentials since neither credentials nor access_token was set in the provider block.  No credentials loaded. To use your gcloud credentials, run 'gcloud auth application-default login'.  Original error: google: could not find default credentials
What I'm doing wrong?
Thanks in advance

Hi, sorry for late response but you should try to pass credentials for env variables. Terraformer won't read configuration in TF


ActionPlanner: adopt SK's config methods for registering models

SK uses the pattern of kernel.Config.AddAzureTextCompletionService() to configure the models that should be used. The ActionPlanner should adopt a similar pattern to better align with SK style prompts. So in JS you could add a model using planner.addAzureTextCompletionService()

P0

Closing since we have forked away from SK


CRDs must be installed separately

Helm 2.9 install had issues with CRDs because validation failed. The fix for < 2.9 is to create CRDs separately using kubectl apply. Helm 2.10-beta fixes the issue but it is not stable. When 2.10 is released we will recommend users to use this version. Recommend users to wait a few seconds after applying CRDs.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: andraxylia
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sdake
If they are not already assigned, you can assign the PR to them by writing /assign @sdake in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@andraxylia: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

One issue I have with this approach is that I'm installing CRDs I don't need.  Secondly, I'm adding conditions in CRDs as it is a template file, so when mixer is enabled, install these, pilot is enabled installed the other CRDs.   see https://github.com/istio/istio/pull/7408. When the conditions are added, we won't be able to run kubectl apply -f install/kubernetes/helm/istio/templates/crds.yaml as it is a template file.
cc @costinm @rshriram @gyliu513 @ymesika

@andraxylia: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@andraxylia @linsun I was a bit confused, with this fix, how can I install istio helm chart in one step via helm install? Does it mean I cannot install Isito helm chart in one step?

I was also having some discussion at https://github.com/helm/helm/pull/3982#issuecomment-408444874 , as I think helm already support install CRDs before other resource can reference it at https://github.com/helm/helm/pull/3019 in helm 2.7, but do not know why helm community created another PR to fix it again for 2.10.

/cc @morvencao

+1 to not change user experience. Istio 1.0 is a milestone release, using two steps to install does not seems to be a good solution.

@morvencao @gyliu513 Indeed, this is a two step installation instead of one, but it is the only solution that works until helm 2.10 is released. It is not a matter of preference, it is obvious we would not add an extra installation step for no reason. The matter was at large discussed on slack and in the environments meeting.
"Helm 2.9 install had issues with CRDs because validation failed. The fix for < 2.9 is to create CRDs separately using kubectl apply. Helm 2.10-beta fixes the issue but it is not stable. When 2.10 is released we will recommend users to use this version. Recommend users to wait a few seconds after applying CRDs.
"
The corresponding change in the helm charts has already been made, I am merely updating the docs.
When helm 2.10 is released, we will remove this step.

Thanks @andraxylia , I see, +1 to merge this first, but this breaking change is not good for Istio.

This is a temp solution before we can get helm 2.10 which is expected next month cc @gyliu513 @morvencao

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andraxylia, linsun, sdake
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

content/OWNERS [linsun,sdake]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Remove spurious state attribute

As far as I can see, this attribute is no longer used?

I'm not sure why the pathological test is failing or how to fix it?

I'm not sure why the pathological test is failing or how to fix it?

The test checks that pathological tests in commonmark.js repo didn't change since last time a human synced them.
The way to fix it is to change the hash (and also hopefully verify that nothing important changed over there). Or propose to remove after arguing against its usefulness.


Revert "Workday Connector Doc Templated - Add files via upload DO NOT MERGE"

Reverts mulesoft/docs-connectors#341
Version 11.1.0 is not released yet. Released version is 11.0.0

Going to update the version instead of reverting.


Create react-intl-redux_v0.3.x.js

https://github.com/ratson/react-intl-redux

Hey, if you're still interested in adding this def (since it seems there isn't one for react-intl-redux), add a couple tests, and I'm sure we can get this merged!

Not going to lie; I have little interest in pursuing this anymore as I don't work on a project that uses react-intl-redux anymore. Happy to either keep this PR open and someone else pick up the tests, or we can just close it and someone else can look at making a PR for it. 😄

Sounds good! I'll close it for cleanup, but if someone ever requests it we can always point in this direction. Thanks again for your contribution 💪


:bug: wheel does not include templates

Hey There!
The current wheel 0.1.0  on PyPi does not include the templates. You can simply change the file extension to .zip to unpack it and verify for yourself.
Best
-Joe

Thanks should be resolved in 0.1.1.

#41


Fix incorrect docker tag placeholder

Description and Motivation
Change PCS_DOCKER_TAG to dockerTag.
Change type

[ ] Bug fix
[ ] New feature
[ ] Enhancement
[ ] Breaking change (breaks backward compatibility)

Checklist:

[ ] All tests passed
[ ] The code follows the code style and conventions of this project
[ ] The change requires a change to the documentation
[ ] I have updated the documentation accordingly

This change is

close it because of #325 already include fix.


About newlly added eager_operation.h

In the latest Eager core update, I see that eager_operation is moved to tensorflow/core/common_runtime/eager. However this file require c_api as one of the dependency.
The c_api is suppose to be built after cpu_core_lib is generated, I am now a bit confuse as I am trying to rewrite the CMake files in order to fit the new eager structure.
Any hel?

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce

I face this issue as well (Ubuntu 14, latest master head). My current workaround is to disable the build of transform_graph, summarize_graph, compare_graphs and benchmark_model (see here).

@akshaym could you look into this and if our dependency graph is healthy?


Urgent Help needed regarding plugin support.

https://github.com/lelouchB/final-space-api/issues/33
kindly visit the link you will better understand the issue

I'm sorry but if you want me to invest time in helping you, please invest first your time to create a proper issue, with all the relevant infos summarized.
If you have a bug related to ideal image plugin, please explain what you tried, what you expected, and what you got, for each of your attempts.


added untrained model Inception v3

I have followed pytorch implementation of Inception3 (here).
Please let me know if this is not the correct way. Thank you

Thanks for this! At first glance things took good. I will provide a more detailed review later. But can you add this to this PR instead?
How did you find this repo? I want to make sure future additions go to that PR.

I found this from here. I did not  the last comment sorry about that.
I will make a pr to that. Thank You.

I added￼ you as a collaborator to my fork. You can make commits directly to the branch in the PR.

Great to see someone seeing our tracker and contributing!


F841 false negative on assignment to multiple variables

def foo():
    a = bar() # error: F841
    b, c = bar() # no error

def bar():
    ...

playground

Could this be intentional? From the tests:
def f(tup):
    x, y = tup  # this does NOT trigger F841

Unsure if that was left there as a TODO or consciously omitted. @charliermarsh could you clarify?

Yeah this is intentional for parity with Flake8. I think it's reasonable. An improvement could be to flag x and y if both are unused.

I don't get why it should be treated differently. If it's for the case where you only want to use one of the values from the tulle, you can just prefix it with an _ to mark it as intentionally unused:
a, _b = foo()

Best practice is really
a, _ = foo()

I don't see why we wouldn't raise a violation here, other than flake8 compatibility

I'm not opposed to changing it. I didn't feel strongly so erred towards compatibility.


Have a function to convert boolean to Option<()> in std

Taking inspiration from haskell (http://hackage.haskell.org/package/base-<IP_ADDRESS>/docs/Control-Monad.html#v:guard), we could have a guard : fn(bool) -> Option<()> function in std
This lets us write more pleasant code. For example instead of
let x = do catch {
    if is_admin(user) {
        calculate_important_stuff()
    } else {
        None
    }
};

we can do this
let x = do catch {
    guard(is_admin(user))?;
    calculate_important_stuff()
};

basic implementation:
fn guard(b: bool) -> Option<()> {
    if b { Some(()) } else { None }
}

Or maybe even have a Try trait implementation for bool? Not sure if that's beautiful though

I'd prefer having a .to_option method like this:
let x = is_admin(user).to_option(calculate_important_stuff);

@CryZe, for this case, that indeed looks like a pretty solution. though! it's not as universal as guard, and I wouldn't really expect a function called to_X take any arguments

Why not impl From<bool> for Option<()>?

Previously in this genre...

#2180
https://github.com/rust-lang/rfcs/pull/2180#issuecomment-340189046: my own crazy proposal
#2202

I used to think that a Try implementation for bool would be insane, but I'm coming around to it because it would fit really nicely for generalizing the || operator: https://internals.rust-lang.org/t/something-for-coalescing-aka-generalized-improved-or-else/9295/3?u=scottmcm

I've seen a few places where this would help reduce cruft like this, where a function checks a condition, and calls a function with an optional error, using a condition to determine if the error is there::
do_thing(index, if array[index] == 0 { Some(UnexpectedZeroError) } else { None });

With this implemented, I could reduce code like this to:
do_thing(index, Option::from(array[index] == 0).map(|| UnexpectedZeroError))

Even better for me, I'd love to see methods like or_opt, or_else, and_opt, and_else (or maybe or_opt and and_opt if ) added to bool:
do_thing(index, (array[index] == 0).or_opt(UnexpectedZeroError);

Amusingly, this exists as Some(something).filter(|_| is_admin(user)).  You'll want the short circuiting version though:
impl bool  { fn map_option<F: FnOnce() -> T>(self, F: F) -> Option<T> {..}  }

If you only have Option<()>: From<bool> then you've short circuiting with is_admin(user).into().map(|()| ..) but not sure if that .into() ever gets ambiguous.

This is available as condition.then_some(()) now. A method special-cased to () doesn't seem necessary in light of the more general method.


chore: use bridge for s3_files_host for local

Checklist

[ ] Affected Issues have been mentioned in the Closing issues section
[ ] Documentation has been written/updated
[ ] PR title is ready for inclusion in changelog

When using make ui-logs-development locally, the default value for S3_FILES_HOST is localhost which doesn't resolve to the minio instance correctly. This fixes it to use the docker bridge network like we do for registry etc.

Pretty simple one, only impacts make ui-logs-development locally


Compiler error

Hello, I was recently getting compiler errors and can't fix them. Can anyone tell me how to fix this?
source/PlayState.hx:255: characters 12-24 : Type not found : VideoHandler
source/Character.hx:14: lines 14-770 : ... Defined in this class
source/HScriptTool.hx:84: characters 17-29 : hscript.Interp has no field errorHandler
source/HScriptTool.hx:132: characters 34-52 : { r : Dynamic, depth : Int } has extra field depth
source/HScriptTool.hx:132: characters 34-52 : ... For function argument 'value'
source/Note.hx:165: characters 18-28 : Class has no field scrollType
I have no clue why did this happen. I am using a Windows x86_64 host. If anyone came up with a solution, please tell me. I modified some stuff in PlayState of the engine for a mod and I ran into this issue.

I now get this

I fixed the issue of the hscript thingy

dawg, haxelib git hxCodec https://github.com/polybiusproxy/hxCodec

@DavidDXOfficial close this issue please

Im gonna close it. Thanks for the solution again


Module build failed when run build command

Are you reporting a bug?
I think yes. Cause i didn't change anything. I just install the npm packages then run build command.

Provide the steps to reproduce the issue, e.g.:

Clone the repo
Install the npm packages with npm or yarn
Run build command (npm run build or yarn run build)

Observed behaviour: Build failed
Expected behaviour: Build should be succesfull
npm version: 6.13.4 || yarn version: 1.21.1
node version 10.18.0
Hello,
The problem is when i clone the repo than install the npm packages and run build command it's crash when build step. I don't know what is wrong with it. By the way i need clone the repo because of the react-whatever package version. I need to fix it. Cause it will crash our production app (cause of react lifecycle method renaming...)
Thanks for help...

Fixed.


compiled against a different Node.js (v69)

Am trying to use this module from "escpos" with electron v4.1.1 and node v10.16.0 but am getting this error :
<EMAIL_ADDRESS>was compiled against a different Node.js version using NODE_MODULE_VERSION 64. This version of Node.js requires NODE_MODULE_VERSION 69.

And this is the line that causes the issue :
const escpos = window.require("escpos")

I tested the v7.1.5 and the beta 8.0.0 and both don't work.

Fixed , thanks to : #214


Sort the hierarchy of banks naturally

Code-review checklist

Documentation

[x] Interfaces describe how other programmers would use them.
[x] Code that is not obvious has documentation describing why
that code is the way it is.

Looks good.


add a RUSTLINKFLAGS environment variable

The motivating example here is when you want to have development and release profiles with lto = false, because LTO is so expensive.  Why not put lto = true in your release profile?  Because the release profile might not be used for releases, per se; it might be used for people doing local development, and they don't want to bear the burden of compiling with LTO all the time.  They want to compile with optimization on so at least their performance measurements are not completely out of whack.
But you do want to compile with LTO at certain times: when you're doing a formal release of your software, for instance.  It would be convenient to turn LTO on via -C lto in RUSTFLAGS...but -C lto isn't valid for all places where RUSTFLAGS would be passed.
Hence the request for a RUSTLINKFLAGS or similar that would specify flags to be used in the final link of a binary artifact.
This could be made somewhat superfluous by #2007, where we could specify another profile for "release releases", but it might still be handy to have for flags that can't be injected via Cargo.toml.
/cc @luser @glandium

This is a pressing enough problem that I don't think it's worth waiting for custom profiles (as those may take awhile to get underway).
Do you have a gist of the errors you were seeing with RUSTFLAGS=-Clto? I think the "best" solution here may be to just fix the compiler to stop complaining about -Clto on things like dylibs/cdylibs, seeing how -Clto is just an "optimization preference" it may make sense to just bypass it in those situations.
Failing that though I may prefer to take a more targeted route for now where we just allow LTO (e.g. CARGO_BUILD_LTO=1) rather than arbitrary flags at the "link" stage. That'd solve the problem for now and still leave room for the final solution (custom profiles) I believe. Were you thinking of solving other problems with RUSTLINKFLAGS though? If so seems like not the worst feature to add!

A random thought I just had is that an argument to cargo that allows to override what is set in Cargo.toml would go a long way (similarly to e.g. hg or git allowing to set/override their config from the command line). Something like:
cargo build -c profile.release.lto=false

Hm yeah an excellent point! It's worth noting though that cargo's system configuration is pretty different than Cargo.toml configuration, but I think we should make an exception for profiles which really should be build time configuration

Do you have a gist of the errors you were seeing with RUSTFLAGS=-Clto? I think the "best" solution here may be to just fix the compiler to stop complaining about -Clto on things like dylibs/cdylibs, seeing how -Clto is just an "optimization preference" it may make sense to just bypass it in those situations.

The error when lto = false in Cargo.toml and RUSTFLAGS="-C lto" is:
error: lto can only be run for executables, cdylibs and static library outputs

which I think fits in with your suggestion of dropping the warning?

Were you thinking of solving other problems with RUSTLINKFLAGS though?

I can't think of any problems at the moment!  I like the CARGO_BUILD_LTO=1 idea and/or @glandium's command-line configuration.  @glandium's command-line configuration stuff seems less likely to cause problems at a later date and more generally flexible/useful anyway.  (I have some reservations about exposing that much build configuration to the command-line, but I suppose it's not wildly different from what you already have with --features.)  We'd want that flag to be a global flag to cargo, though, and not cargo build-specific, yes?

If we're going to fix this in a simple way, making rustc not warn about -C lto for rlib compiles seems like the simplest fix. For a better long-term solution I think custom profiles is the best way to go, personally. Being able to override individual profile bits on the commandline is a neat idea as well, though, and I could see either of them being a reasonable solution.

The problem with custom profiles is that they're quadratic. You may need variants for debug and non-debug. Opt vs. no-opt. LTO vs no-LTO, abort panics vs. unwind, etc.

@froydnj

which I think fits in with your suggestion of dropping the warning?

Ah yes, indeed! In retrospect I think at most a warning should be emitted in that case and in reality we should just ignore it in the compiler, doing a 'best effort' thing when possible and otherwise ignoring it for things like rlibs.

I can't think of any problems at the moment! I like the CARGO_BUILD_LTO=1 idea and/or @glandium's command-line configuration. @glandium's command-line configuration stuff seems less likely to cause problems at a later date and more generally flexible/useful anyway.

Indeed! I'll comment on the PR wrt this strategy.

Cross-linking https://bugzilla.mozilla.org/show_bug.cgi?id=1386371 here

If I'm understanding correctly, there is a somewhat related RFC at rust-lang/rfcs#3310

See also

#12739
#5931
#8716

Also, if we expose more rustflags in Cargo-native ways (#12739), then Cargo can then decide to only fingerprint and pass the flag to final artifacts.

btw LTO is not a great case for this as we now do change dependencies build based on lto as discussed in #3244.


Assets with the Same Name, but Different Path, are Copied Over Each Other

🐞 Bug report
Command (mark with an x)

[x] serve
[x] build

Is this a regression?
I don't know, since never needed this before.
Description
Assets with the same name, but different path, are copied over each other on ng serve or ng build --prod with "outputHashing": "none".
🔬 Minimal Reproduction
.dark {
  background-image: url('../../../assets/images/dark/logo.png');
}

.light {
  background-image: url('../../../assets/images/light/logo.png');
}

Both images will be copied to the output directory with the same name, so you only get the dark or light logo, that will be used in both cases.
This won't be an issue on ng build --prod with "outputHashing": "all" or "outputHashing": "media".
This also won't be an issue on ng serve --prod, as per this comment, but will definitely slow down local debugging.
This could be fixed by using assets relative path (/assets instead of ../../../assets), but then it would be an issue on ng build --prod, since I don't include assets/images in assets in angular.json. The reason is that I don't want them to be duplicated (in assets and root folder), and this is by design, as per this comment.
🔥 Exception or Error
WARNING in Conflict: Multiple assets emit different content to the same filename logo.png
🌍 Your Environment

    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 11.0.7
Node: 14.15.0
OS: win32 x64

Angular: 11.0.9
... animations, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1100.7
@angular-devkit/build-angular   0.1100.7
@angular-devkit/core            11.0.7
@angular-devkit/schematics      11.0.7
@angular/cli                    11.0.7
@schematics/angular             11.0.7
@schematics/update              0.1100.7
rxjs                            6.6.3
typescript                      4.0.5

Duplicate of https://github.com/angular/angular-cli/issues/12186

Duplicate of https://github.com/angular/angular-cli/issues/12186


Android Support/Docs

I've been trying the example and connecting to Metamask app on my phone, I am unsure if this is web-only, or does it support android too, because it doesn't show a list of apps when I open it on my android phone, do I need to make some additional configurations ? I was unsure as there is not any documentation around it, within the codebase too. It'd be great if I can get some help figuring this out.

Hi, we meet same issue for android phone. It does not show a list of wallet app. Have you resolved this issue? Thanks.

Hello @Sameerkash, how do you solve it? I know that the UX is diff on Android and is not showing the list of wallet apps.
https://docs.walletconnect.com/mobile-linking
But I couldn't find information about how to setup the deep link on Android.
https://stackoverflow.com/questions/66451913/android-deep-link-wc-scheme


CLIP Interrogation - possible solution found

Under Todo > Image Enhancement > image describe feature in the Readme is an entry for "https://github.com/pharmapsychotic/clip-interrogator (blip + clip)"
The ability to pass an image and phrases to CLIP is already present as enhancers.describe_image_clip.find_img_text_similarity. The ability to use BLIP is already marked as done on the Todo list. The only remaining part of the pharmapsychotic/clip-interrogator code that is not implement here is the long lists of artists, mediums, etc..
There is a package on pypi called clip-gaze that includes cleaned and expanded upon versions of those lists (as well as implementing a version of find_img_text_similarity). It has no dependencies beyond what is already needed in this repo.
If the lists are desirable I suggest importing them from clip_gaze, and if not then I think that part of the Todo can be checked off.

I'm hesitant to introduce the dependency just to load what are essentially text files. I haven't implemented yet because I'm unhappy with how long the analysis takes.

I spent a bit of time playing with various clip interrogators and I don't find them accurate enough to include in this library.


File descriptor listener

This code change adds the ability to accept connections directly from a file descriptor passed by a socket manager parent process, like systemd et al.
It's experimental and largely untested, especially for UDP transports. Mostly putting up this PR for discussion.
If there's a path to having this in the main source tree, I'll clean it up a bit and write some tests and docs.
Closes #789.

I personally don't care either way, but I wonder if there's a way to find out if more people would be interested in this? We're starting to get a proliferation of options nobody uses. Any user groups/etc or examples of other projects doing this that can provide feedback?

This change is currently only used in an experimental capacity for an embedded memcached deployment. I agree that adding unused options adds a lot of smell; perhaps let's close this PR for now and I can revisit it in the future if I'm able to find some compelling use cases within the wider community.

I think that's a decent compromise.. thanks!


Move away from highlight.js

Not entirely happy with the library as there seems to be some unexpected results in some test cases (not published as of yet).

prism.js looks promising


Refuse to build if user is importing something that isn't declared in package.json

I think Brunch does this, if I recall correctly.
This is an important feature because people often misunderstand that transitive dependencies aren’t supposed to be importable.
See https://github.com/facebookincubator/create-react-app/issues/1714 and https://github.com/facebookincubator/create-react-app/issues/1622 for examples.

eslint-plugin-node has similar rules for this:

https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-missing-require.md

https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-missing-import.md

@jokeyrhyme I don't think these are similar. The rules you link to protect against importing a non-existing file (which would break the build anyway). But I'm talking about a file that exists but comes from a transitive dependency.

@matoilic This one looks right. Want to send a PR adding it?

One potential issue here is I don't think editing package.json would trigger an eslint-loader recheck, so the user would be stuck with a misleading message in our setup.

@gaearon Sure, I can work out a PR. What do you think about importing dev dependencies? Should there we a warning / error when importing a dev dependency, especially in a production build?
I'm not sure on how to trigger a recheck or rebuild when changing editing package.json. Could a Webpack plugin be a solution? I haven't come across one which does something like that. We might need to create one to solve the issue.

What would you think about implementing this upstream in webpack? This seems like it'd be super helpful for all / most projects, not only React projects / projects using that ESLint plugin.

Could a Webpack plugin be a solution?

That may be the case. I’m not sure.

What would you think about implementing this upstream in webpack?

It may be better longer term but if we can prototype this feature here and figure out the edge cases sooner, that might work out just as well. IMO plugins like this end up good when they start in userland, and if proven useful, move into the core.

Another good point: https://twitter.com/_jayphelps/status/838489432801959936

I think this is important enough to not tag as a future idea and instead for 0.10. 😄

Sounds good. I’m not sure I’m aware of a good way to make it happen tho.

Oops 😊
I was thinking of these rules but navigated to the wrong ones in my haste

https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-require.md

https://github.com/mysticatea/eslint-plugin-node/blob/master/docs/rules/no-unpublished-import.md

Measure twice, cut once :D

I think those two rules are more intended for package creators. When you install react-scripts the NPM registry will only contain "published" files and thus only files which already pass the check. That's how I understand it. But those rules might be useful to solve #1720.

Pushing back to 0.11.

Any news ?
@matoilic

This would be a great addition for junior engineers who don't understand the reason their import works is due to hoisting and/or node_modules flattening
In the meantime, I'm setting up depcheck

@gaearon is this open?

@gaearon is this open?

Hi @Timer,
The behaviour is expected because you are trying to import something which is not there.
Let's say If I am importing with the below code:
import {debounceTime} from 'rxjs';
so, for me to access the debounceTime Property from rxjs, i need to have the rxjs object right? so as you said earlier that the build is getting refused because user is importing that isn't declared in the package.json and whether it is declared it doesn't matter.
I imported rxjs which was not declared and build failed so, I installed the package, deleted it's entry from package.json and ran the app with no errors along with the import statement.
The thing is we are importing from node_modules not from package.json so, in the build.js file we can add a warning message saying , the $package is not installed and suggest the user to install the package instead of just saying the below error:
Failed to compile.
Module not found: Error: Can't resolve 'rxjs' in 'F:\github\test3\my-app\src'
Why? Because with the module not found is kind of too vague as a user warning
else {         console.log(chalk.red('Failed to compile.\n'));         printBuildError(err);         console.warn(" Please install the unresolved dependency and then run npm build")         process.exit(1);

Good night.
Any updates for this issue? If not, this may resolve the issue.
Thanks.


Automatically deflate based on Content-Encoding

Twitter returns deflate data despite the client only requesting gzip
data.  net/http automatically handles the latter but not the former:
https://github.com/golang/go/issues/18779
Fixes #170.

Thanks!
Out of curiousity, do you know under which circumstances Twitter returns deflated responses?

Sorry I'm not exactly sure how I encountered this issue since I worked on it some months ago.

No worries. Thanks again for the patch!

I reproduced the symptoms with this call:
2017/09/05 00:00:30 Could not authenticate: Get https://api.twitter.com/1.1/friends/list.json?cursor=1538070094339682341&tweet_mode=extended returned status 503, x��VJ-*�/*V���V�M-.NLOU�R�/K-RHN,HL�,�T�QJ�O
�������N

I get deflate data responses randomly every so often. Thanks for the fix @gaul @ChimeraCoder !


Increase visibility of CAIRR pipeline

Updated the submission manual to increase the visibility of CAIRR pipeline.

Hi @marcosmro , thanks for the pull request. Sorry I wasn't aware you were doing this work otherwise I would have informed you earlier. The documentation is being restructured (#199) and all the work is on the metadata-docs branch.
Could you please apply your changes and have the pull request go against that branch?
As your changes are fairly constrained, hopefully it will be straight forward.

Sure, no problem.


Upscalers messed up, how can I fix it without reinstalling?

Something messed up with my upscalers, I think after the infinite restart of the download loop I already wrote in another issue, and now a few of my upscalers are duplicated in the list and when I try to use one of these, I get an error-message with stack trace about Swin failing (some includes not found). It's probably some dependency of the upscalers that's messed up.
I am new to this python environment stuff, is there a way to reinstall only the failing package/dependency? (I'm on windows btw)
PS: Everything else, the models, the loras and the image generation seems to work, just these duplicated upscalers fail, all of them with the same message.
 Failed loading SwinIR model W:\pinokio\api\automatic1111.git\app\models\SwinIR\RealESRGAN_x4plus.pth: Error(s) in loading state_dict for SwinIR:
Missing key(s) in state_dict: "patch_embed.norm.weight", "patch_embed.norm.bias", "layers.0.residual_group.blocks.0.norm1.weight", "layers.0.residual_group.b
locks.0.norm1.bias", "layers.0.residual_group.blocks.0.attn.relative_position_bias_table", "layers.0.residual_group.blocks.0.attn.relative_position_index", "layers.0
.residual_group.blocks.0.attn.qkv.weight", "layers.0.residual_group.blocks.0.attn.qkv.bias", ... continues for pages

Failed loading SwinIR model W:\pinokio\api\automatic1111.git\app\models\SwinIR\ESRGAN_4x.pth: 'params_ema'

fixed it now with reinstalling

fixed with reinstalling


Missing text on Vector Tiles docs due to font-awesome bug

Console on page https://mapzen.com/documentation/vector-tiles/use-service/:
[Warning] Unexpected CSS token: : (font-awesome.min.css, line 4)
[Warning] Unexpected CSS token: : (font-awesome.min.css, line 4)
[Warning] Unexpected CSS token: : (font-awesome.min.css, line 4)
[Warning] Unexpected CSS token: : (font-awesome.min.css, line 4)
[Warning] Unexpected CSS token: : (font-awesome.min.css, line 4)
[Warning] Unexpected CSS token: : (documentation.css, line 3128)
[Warning] Unexpected CSS token: : (documentation.css, line 3136)
[Error] Failed to load resource: the server responded with a status of 404 (Not Found) (jquery-1.10.2.min.map, line 0)

In Safari 9.0.2, looks like:

Hm, the font appeared correctly when I returned to the page later. Not liking the unpredictability of this.

Not sure what would be causing this, though...

I’ll keep an eye out for it, but I will close this ticket for now.


Rename multimodalmaestro to maestro.

Description
Rename multimodalmaestro to maestro.

 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.

@capjamesg Welcome WorkFlow / 👋 Welcome pass for me. I'm confused.

@SkalskiP Does pip install multimodal-maestro still work? If not, can you make it backwards compatible?


Doesn't log to TTY after break to debugger

I have CocoaLumberjack setup to log to DDASLLogger, DDTTYLogger, and DDFileLogger.  After the debugger is tripped in Xcode 7 I noticed no logs are sent to the console for the entire method.  Luckily it seems to still log to the FileLogger.

Please create a demo project with this issue, if possible. It doesn't sound like an issue with the library itself.

Version?

CocoaLumberjack version 2.1.0, installed via CocoaPods.
On Wed, Oct 21, 2015 at 6:24 PM, Ernesto Rivera<EMAIL_ADDRESS>wrote:

Version?
—
Reply to this email directly or view it on GitHub
https://github.com/CocoaLumberjack/CocoaLumberjack/issues/601#issuecomment-150041731
.

I'll create one as soon as I get a chance, probably next week.
On Wed, Oct 21, 2015 at 4:28 PM, Bogdan Poplauschi<EMAIL_ADDRESS>
wrote:

Please create a demo project with this issue, if possible. It doesn't
sound like an issue with the library itself.
—
Reply to this email directly or view it on GitHub
https://github.com/CocoaLumberjack/CocoaLumberjack/issues/601#issuecomment-150015153
.

I can't reproduce this in the demo app.  Closing it for now.


Bad IL for move with an immediate value

Version and Platform (required):

Binary Ninja Version: 3.4.4248-dev, c26685e2
OS: macos
OS Version: 13.3
CPU Architecture: arm64

It looks like BN is producing the wrong IL for movw r2, #0x47e, e.g.
00000bdfa  40f27e42   movw    r2, #0x47e
00000bdfe  86e80300   stm     r6, {r0, r1} {var_a68} {var_a68} {var_a64}  {0x41e5999a}  {0x41dc0000}
00000be02  04f10801   add     r1, r4, #8  {SUB_BDD8_DATA}
00000be06  b2a8       add     r0, sp, #0x2c8 {var_920}
00000be08  bbf08cff   bl      #memcpy

According to IDA, this is
 v6 = memcpy((unsigned int)v176, (char *)SUB_BDD8_DATA, 0x47Eu);

but BN produces HLIL
memcpy(&var_920, &SUB_BDD8_DATA, &data_47e)

This starts from LLIL where it does not recognize 0x47E as an immediate value
40 @ 00000bdfa  r2 = &data_47e

It's being inferred to be a pointer to data - it looks like this is because 0x47e is a valid data address. Is the binary base in IDA also 0?

Ok so while it's TECHNICALLY correct, the way we show it is really misleading. We should either simplify instances of &<datavar> to their static value, or never emit that as a reference in the first place

V35 should search for "Encourage Salesman Prompt Delay" to find the database.

Forgot to add that rebasing binary to a non-zero value may not work as it may break string references.

The only time it would break string references I would expect it to break all data variable references if the binary actually should be based at 0. Are you seeing strings break when rebasing and the binary image isn't supposed to be based at zero? If so, that might be a different bug.

I have a binary that must be based at address 0x500000 (example) for strings to be referenced. This is because the compiler inserted a bunch of fixed references to 0x50xxxx into the code.
I don't think it matters what the base address is, though. You will break string references as long as there are fixed addresses in the code and you rebase to the address the compiler does not expect.

That's my point though. Only if those fixed locations that require a base address of 0x0 are ones that would cause issue. All the rest, like your example don't require the work around but should already have less issue if they're based correctly.


Catch/Finally blocks executed more than once if the finally block contains inner finally blocks throwing an error with a return in inner try/catch/finally

Description:
The catch/finally block execution happens more than once/incorrectly when the finally block contains inner try/catch/finally blocks where

the inner finally block throws an error
the inner try/catch or finally block contains a return statement

This may require fixing the generation of the error table.
Sample:
import ballerina/io;

function main(string... args) {
    io:println(test());
}

function test() returns int {
    	try {
        	io:println("executing try block");
        	return 4;
    	} catch (error e) {
        	io:println("executing catch block");
        	return 22;
    	} finally {
	    try {
		io:println("executing inner try block");
		return 4;
	    } catch (error e) {
		io:println("executing inner catch block");
		return 22;
	    } finally {
		io:println("executing inner finally block");
		error abc = {};
		if (true ) {
		    throw abc;
		}
		return 40;
	    }
    	}
}

Related PRs:
https://github.com/ballerina-platform/ballerina-lang/pull/9119

No longer relevant with the changes to error handling.


Increased font size in add lobby page

Before:

After :

Merged :rocket:. Good work @Solai-Adithya.


Sign in: Back-end change: Change the service to allow longer time between app uses before we reset you (and force the sign-in)

Fixed in https://github.com/MusicAsLanguage/webservice/pull/47


Left sidebar panes are not scaling correctly

Issue Type: Bug
The results of a search in file search (linux: ctrl + shift + F)` are not rendered correctly. The results field doesn't use all possible space (See gif). Behaviour also appears with all extensions disabled.

VS Code version: Code - Insiders 1.44.0-insider (eb4ebee6a3310df8ff5f36f78f209c2658cfcdfc, 2020-03-11T05:33:48.554Z)
OS version: Linux x64 5.3.0-40-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 3900)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
62.63GB (45.05GB free)

Process Argv
. --no-sandbox

Screen Reader
no

VM
0%

Extensions (29)

Extension
Author (truncated)
Version

better-comments
aar
2.0.5

numbered-bookmarks
ale
7.0.0

path-intellisense
chr
1.4.2

bracket-pair-colorizer
Coe
1.0.61

vscode-markdownlint
Dav
0.34.0

githistory
don
0.6.2

xml
Dot
2.5.0

gitlens
eam
10.2.1

vsc-material-theme-icons
equ
1.1.4

prettier-vscode
esb
3.20.0

git-project-manager
fel
1.7.1

todo-tree
Gru
0.0.169

restructuredtext
lex
120.0.0

vscode-ros
ms-
0.6.2

anaconda-extension-pack
ms-
1.0.1

python
ms-
2020.2.64397

cpptools
ms-
0.27.0-insiders2

autodocstring
njp
0.4.0

ros-snippets
pij
0.3.1

material-icon-theme
PKi
4.0.1

vscode-xml
red
0.10.1

vscode-yaml
red
0.7.2

rescuetime
Res
1.0.1

code-spell-checker
str
1.8.0

vscodeintellicode
Vis
1.2.6

vscode-icons
vsc
10.0.0

vim
vsc
1.13.0

highlight-trailing-white-spaces
yba
0.0.2

markdown-all-in-one
yzh
2.7.0

(1 theme extensions excluded)

Updates:

As pointed out by @furai it appears to affect all panes.

I'm getting same issue but with all the panes in on the left. They are all cutting off midway. Seems like the height of the pane with search/files is set to whatever is the current width of the pane right now.

@furai Ah thanks a lot for pointing that out. I didn't notice the other panes being off. I updated the title.

Actually it has been fixed for me in the latest insiders that has been just pushed out.

Fixed in v1.44.0-insider commit ef75c4bb445e5021065cf0012cfa2c66e55bfa16.


Possible Enhancement - useLocation without Redraws

Intro
I encountered a scenario where I was using the useLocation hook, and the < Link > component in a navigation. My navigation component is a child component of < Route >.  But I didn't want my Navigation component to redraw when the Link component was clicked on, because it is a child of < Route >.
Possible Solution
import { useRef, useState } from "react";
import { useLocation } from "wouter";

/**
 * A hook that returns wouters location object, which you can use
 * to determine the current location at page load, but will not update, unless
 * trigged through a setState call.
 * @returns {Array<string, import("wouter").LocationHook, import("react").Dispatch<import("react").SetStateAction>>}
 */
export const useInitialLocation = () => {
    const locationRef = useRef(useLocation());
    const [location, setLocation] = useState(() => locationRef.current[0]);
    const [redirect] = useState(() => locationRef.current[1]);
    return [location, setLocation, redirect];
};

What this custom hook does is, get the location string from wouter, and converts it to state. The redirect part is excess/nice to have i.m.o. So that in my use case, setState occurs and remount/redraw of my component doesn't occur.
Usage
const LeftNavBar = () => {
    const [location, setLocation, redirect] = useInitialLocation();
    return (
        {
            links.map(link => (
                <Link key={link.name} to={link.url} onClick={() => setLocation(link.url)}>
                    <a title={`Go to ${link.name}`} aria-current={location.startsWith(link.url)} className="nav_link">

                                {link.icon}

                            <span className={`nav_text ${isExpanded ? "sr-only" : ""}`}>
                                {link.name}

                </Link>
            ))
        }
    );
};

Conclusion
After than long description of the possible feature/enhancement. Could wouter have this hook in it's npm module or as a part of it's documentation? Thanks for taking time out of your day to consider this possibility.
Cons
Requires certain versions of React. Not sure if it would work with Preact.

Closing issue as https://github.com/molefrog/wouter/blob/master/static-location.js solves the same problem and already exists.


Modal component

Are there any plans to make a "modal" component, like bootstrap maybe?
ref: http://getbootstrap.com/javascript/#modals

It's implemented already as dialog.

Thanks

you guys rock


AttributeError: 'AIMessage' object has no attribute 'find'

Describe the bug
metric.measure method with a custom evaluation LLM returns an attribute error "'AIMessage' object has no attribute 'find'"
To Reproduce
Steps to reproduce the behavior:

I defined a custom evaluation model by inheriting DeepEvalBaseLLM class, following this example

from deepeval.models.base import DeepEvalBaseLLM
class DeepEvalAzureOpenAI(DeepEvalBaseLLM):
    def __init__(
        self,
        model
    ):
        self.model = model

    def load_model(self):
        return self.model

    def _call(self, prompt: str) -> str:
        chat_model = self.load_model()
        return chat_model.invoke(prompt)

    def get_model_name(self):
        return "Custom Azure OpenAI Model"

In my main script, I imported the custom eval model, constructed metrics, and used them

from langchain.chat_models import AzureChatOpenAI
from DeepEvalAzure import DeepEvalAzureOpenAI
from deepeval.metrics import AnswerRelevancyMetric, ContextualRelevancyMetric
custom_model = AzureChatOpenAI(
    azure_deployment=os.getenv("OPENAI_API_MODEL_GPT4"),
    openai_api_version=os.getenv("OPENAI_API_VERSION"),
    base_url = os.getenv("OPENAI_API_BASE"),
)
azure_openai = DeepEvalAzureOpenAI(model=custom_model)

answer_relevancy_metric = AnswerRelevancyMetric(
    threshold=0.7,
    model=azure_openai,
    include_reason=True
)

actual_output = "We offer a 30-day full refund at no extra cost."

retrieval_context = ["All customers are eligible for a 30 day full refund at no extra cost."]

test_case = LLMTestCase(
    input="What if these shoes don't fit?",
    actual_output=actual_output,
    retrieval_context=retrieval_context
)
answer_relevancy_metric.measure(test_case)

Observed this error:

---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[43], line 1
----> 1 answer_relevancy_metric.measure(test_case)

File ~/repos//aia-chatgpt/.venv/lib/python3.11/site-packages/deepeval/metrics/answer_relevancy.py:46, in AnswerRelevancyMetric.measure(self, test_case)
     41     raise ValueError(
     42         "Input, actual output, or retrieval context cannot be None"
     43     )
     44 with metrics_progress_context(self.__name__, self.evaluation_model):
     45     # generate statements
---> 46     self.statements: List[str] = self._generate_statements(
     47         test_case.actual_output
     48     )
     50     # generate verdicts based on statements, and retrieval context
     51     self.verdicts: List[AnswerRelvancyVerdict] = (
     52         self._generate_verdicts(
     53             test_case.input, test_case.retrieval_context
     54         )
     55     )

File ~/repos//aia-chatgpt/.venv/lib/python3.11/site-packages/deepeval/metrics/answer_relevancy.py:120, in AnswerRelevancyMetric._generate_statements(self, actual_output)
    115 prompt = AnswerRelevancyTemplate.generate_statements(
    116     actual_output=actual_output,
    117 )
    119 res = self.model(prompt)
--> 120 json_output = trimToJson(res)
    121 data = json.loads(json_output)
    122 return data["statements"]

File ~/repos//aia-chatgpt/.venv/lib/python3.11/site-packages/deepeval/utils.py:92, in trimToJson(input_string)
     91 def trimToJson(input_string: str) -> str:
---> 92     start = input_string.find("{")
     93     end = input_string.rfind("}") + 1
     94     return input_string[start:end] if start != -1 and end != 0 else ""

AttributeError: 'AIMessage' object has no attribute 'find'

Screenshots
Also, evaluate([test_case], [answer_relevancy_metric]) fails without any error messages. On my Jupyter notebook, it flicks between "You're using DeepEval's latest Answer Relevancy Metric..." and "Evaluating testcases..." and after few seconds, it fails without any output.

Desktop (please complete the following information):

OS: MacOS 14.1.2 (23B92)
IDE: VSCode 1.86.2
Python: 3.11.5
langchain: 0.1.7
langchain-openai: 0.0.6
deepeval: 0.20.65

Additional insights from myself. The issue seems to lie in _generate_statements method which is called within the measure method. More specifically, it's due to the way the res (prediction result) generated by the evaluation model is passed to utils.trimToJson, which converts the text into json compatible format. Since the res is of type langchain_core.messages.ai.AIMessage not a string, it throws the error. As an temporary solution, I modified the _generate_statements method like below and it works. Same for _generate_verdicts.
def _generate_statements(
    self,
    actual_output: str,
) -> List[str]:
    prompt = AnswerRelevancyTemplate.generate_statements(
        actual_output=actual_output,
    )

    res = self.model(prompt).content # adding .content to convert the response to string. otherwise res is a type langchain_core.messages.ai.AIMessage and it throws an error.
    json_output = trimToJson(res)
    data = json.loads(json_output)
    return data["statements"]

But I would still be interested to know more stable solution.

btw @makoto-velux fixed the docs, one line of update but now it works: https://docs.confident-ai.com/docs/metrics-introduction#azure-openai-example

@makoto-velux I followed our quick start in colab https://colab.research.google.com/drive/1PPxYEBa6eu__LquGoFFJZkhYgWVYE6kh?usp=sharing up until the "🚀 Run Your First Evaluation" section and couldn't reproduce the error. Which metric are you using?

Also if you're on discord we have an issues channel: https://discord.com/invite/a3K9c8GRGt

I was using answer relevancy, context relevancy and faithfulness metrics. And evaluate didn't work for either of them.

Can you try modifying the colab instead and let me know how it works out? It shouldn't be a problem with your test case since its from the docs

@makoto-velux is it still a problem?


Unable to load float=32 created binary resource  on float=64

Godot version
db5271dd34ea2b1894e1e2cf9ee9519c84142608 (closest)
System information
Fedora 35, Nvidia 3000 series
Issue description
Godot cannot load scn in float=64 when they're from float=32. Errors for Vector<Vector2>, Vector<Vector3>, Vector<Color>.
Vector2 size is NOT 8!"
	case VARIANT_PACKED_VECTOR2_ARRAY: {
			uint32_t len = f->get_32();

			Vector<Vector2> array;
			array.resize(len);
			Vector2 *w = array.ptrw();
			if (sizeof(Vector2) == 8) {
				f->get_buffer((uint8_t *)w, len * sizeof(real_t) * 2);
#ifdef BIG_ENDIAN_ENABLED
				{
					uint32_t *ptr = (uint32_t *)w.ptr();
					for (int i = 0; i < len * 2; i++) {
						ptr[i] = BSWAP32(ptr[i]);
					}
				}

#endif

			} else {
				ERR_FAIL_V_MSG(ERR_UNAVAILABLE, "Vector2 size is NOT 8!");
			}

			r_v = array;

		} break;

Steps to reproduce
Builds of godot with float=64 fail to load binary serialized PackedScene defined as float=32.

create a resource with these types in godot float=32
open in float=64
See variant error

Minimal reproduction project
Pending.

It may have to check for FileAccess::real_is_double instead

@fire can you try with #59324

Testing.


No subtitles in source kickass

Had this issue too. I'm not sure, but I think changing subtitle language from hardsub to English fixed it. Now I'm back on hardsub and it still works. 🤷
I think the app might have been using subtitles that were only signs and text.

update to latest stable


fix: sync clocks across L2 nodes on startup

Closes: https://github.com/ethereum-optimism/supersim/issues/102
Part of https://github.com/ethereum-optimism/supersim/issues/100
This PR synchronizes the anvil block timestamps across all L2 nodes prior to supersim startup. This is done using the following steps:

start the nodes in automine mode so no new blocks are created
check which node has the latest timestamp
use evm_setNextBlockTimestamp to set all the nodes to the same timestamp using the latest timestamp
turn on 2 second interval mining for all nodes using evm_setIntervalMining

This synchronization is not perfect but it will make sure that the clocks between l2 nodes never drift further than 2 seconds apart.

#103  👈
main

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

Join @tremarkley and the rest of your teammates on  Graphite

For fork mode, it seems like the latestL2HeightFromL1Header logic correctly returns the L2 heights that has the same timestamp, so we prob won't need to change anything here. Anvil should correctly fetch the l2 block to start on, and they should all have the same (or very similar) timestamps.

This is roughly true. It looks for an L2 height which should approximately be in the same range as all the other chains.
Agree with @jakim929 thought. Maybe on eth_sendRawTransaction we could also progress the timestamp of a chain that hasn't had any txs yet?


Expose function to determine whether an address is private or public

So it can be used in the courier and gateway apps.
See: https://github.com/relaycorp/relaynet-gateway-android/pull/29#discussion_r448887740

This is no longer applicable, since private and Internet addresses are now defined explicitly without relying on syntax.


python

It has nice, terse syntax while still allowing for a lot of useful tricks.

Don't forget to assign yourself to the issue!

Update: Github Doesn't allow me to give you access to self-assign; I will assign you


[HIGH PRIORITY] Performance Profile Entire App

title

This app will be sunseted very soon and we are NOT maintaining this app
unless of a security issue. We encourage you to try Cider, our new music app built from scratch. Issues filled on that repository will be seen and will be acted upon. Please join our discord and see the #nightly-builds channel for a current build.
Cider can be found here


Remove CI/CD Pipeline section in tutorials

Closes #23167

/assign @zacharysarah

looks good for DE
/approve

/assign @sftim

LGTM for ID 👍

@habibrosyad Please limit changes to one language per PR. In this case, please limit this PR to English.
/hold

@zacharysarah sorry for that, missed that part 😅

Removing labels for other languages (maybe later in separate PRs) and limit this PR to English for now.
/remove-language zh
/remove-language uk
/remove-language ru
/remove-language pt
/remove-language pl
/remove-language ko
/remove-language ja
/remove-language it
/remove-language id
/remove-language fr
/remove-language es
/remove-language de

/hold cancel
/lgtm
/approve


Feature request: change ssh user on agent

I have a git repository whose's SSH credentials do not begin with the usual<EMAIL_ADDRESS>Instead, it is<EMAIL_ADDRESS>It seems like the agent's behavior to clone repositories using SSH credentials is hardcoded to use<EMAIL_ADDRESS>More specifically, doing git clone<EMAIL_ADDRESS>results in permision denied, whereas doing git clone<EMAIL_ADDRESS>works just fine.
Can you please add a way to modify the SSH user for the agent?
Thanks.

Hi @milongo
Good news the new RC includes this feature.
Add to your ~/clearml.conf the following line:
agent.force_git_ssh_user = "git"

And upgrade the clearml-agent to the latest RC:
pip install clearml-agent==0.17.2rc2

@bmartinn should this work agent-side in clearml-agent==1.0.0?
I have:
$ pip freeze | grep clearml
clearml==1.0.4
clearml-agent==1.0.0

and in clearml.conf:
...
agent: {
    force_git_ssh_user: "git"
    force_git_ssh_protocol: true
    default_docker {
        image: "my_image:latest"
    }
...
}

Now after running Task.init locally, I see in the UI this URL pattern:
ssh://my_server_url:<ssh_port>/<user>/<repo>.git

and the execution fails with:
...
cloning: ssh://my_server_url:<ssh_port>/<user>/<repo>.git
fatal: Could not read from remote repository.
Please make sure you have the correct access rights
and the repository exists.
...

Should this work? After adding force_git_ssh_user: "git", should I instead see cloning: ssh://git@my_server_url:<ssh_port>/<user>/<repo>.git, or is that handled somewhere in the backend?

Sry if I misunderstand - I am still new to ClearML :slightly_smiling_face: . Though is the commit you referenced really addressing the issue that @milongo mentioned?
 But the issue raised by you @milongo is about 'userless' ssh clone links, right?
At least we have the issue that running ClearML in our git repo creates links like:
ssh://my_server_url:<ssh_port>/<user>/<repo>.git

and we do not want to fix them all by hand. Alternative solutions could be changing what repo url is written when a Task is initialized, but solving this 'agent-side' would be better.

OK I found two workarounds, both rather hacky. If you need a solution, you can use it, but I personally would prefer a cleaner way.
Both rely on having a ~/.ssh/config file like the one below:
Host git.mycompany.com
  HostName <IP_ADDRESS>
  IdentityFile ~/.ssh/id_rsa
  IdentitiesOnly yes
  User git
  Port 2022

One gotcha is that you need to spell out the domain as in the ssh url. People often use abbreviations like Host gitserver and then state the domain as HostName. However in this case, you need the full domain as in the url as Host and the IP of the server (might work with DNS, have not tested) as HostName.
You can test it outside of the docker container by running e.g. git clone ssh://git.mycompany.com:2022/myuser/repo.git. This should work outside of the container.
For making this available in the docker container, we have two options.
Option 1: Own config/key with root, mount it manually
The docker containers executed by the clearML agent run as root. Without any changes, ssh will complain about a bad owner of the config file, even when file permissions are 600. To circumvent this, you can own the config and key as root. Outside of the docker container:
sudo chown root:root ~/.ssh/config && sudo chown root:root ~/.ssh/id_rsa

Though now you will get an error (when running the clearML agent with --foreground) because your user can no longer copy the whole .ssh folder:
Failed creating temporary copy of ~/.ssh for git credential

So in your clearml.conf, you need to mount the credentials manually:
agent: {
    ...
    extra_docker_arguments: ["-v", "/home/datapipelineuser/.ssh:/root/.ssh"]
    ...
}

Option 2: Own the ssh credentials in the container
Instead of owning the credentials outside and mounting them manually, you can add a command to own the credentials in the container. Adding
agent: {
    ...
    extra_docker_shell_script: ["chown -R root:root /root/.ssh"]
    ...
}

to your clearml.conf should do the trick. I prefer the second solution, but overall, the best would be if we could either have a user set when creating the task or overriding the user even when a ssh:// domain is given.

@milongo can we close this?

Hi @jkhenning, we can! I'm closing it now.


Error whilst trying to run docker compose up

When I try to do the following commands I get an error. I am trying to run a docker-compose to run the flask-todolist. I have Windows 10 version :  10.0.19043 N/A Build 19043 and docker version  20.10.16, build aa7e414. I am unable to run docker-compose up
commands on cmd
docker-compose build
docker-compose up

Error:
C:\Users\Edge\Desktop\todolist\flask-todolist>docker-compose up
[+] Running 1/0
 - Container todolist  Created                                                                                0.0s
Attaching to todolist
todolist  |  * Tip: There are .env or .flaskenv files present. Do "pip install python-dotenv" to use them.
todolist  | Usage: flask db upgrade [OPTIONS] [REVISION]
todolist  | Try 'flask db upgrade --help' for help.
todolist  |
todolist  | Error: While importing 'todolist', an ImportError was raised:
todolist  |
todolist  | Traceback (most recent call last):
todolist  |   File "/usr/local/lib/python3.10/site-packages/flask/cli.py", line 256, in locate_app
todolist  |     __import__(module_name)
todolist  |   File "/code/todolist.py", line 1, in <module>
todolist  |     from app import create_app
todolist  |   File "/code/app/__init__.py", line 2, in <module>
todolist  |     from flask_login import LoginManager
todolist  |   File "/usr/local/lib/python3.10/site-packages/flask_login/__init__.py", line 16, in <module>
todolist  |     from .login_manager import LoginManager
todolist  |   File "/usr/local/lib/python3.10/site-packages/flask_login/login_manager.py", line 24, in <module>
todolist  |     from .utils import (login_url as make_login_url, _create_identifier,
todolist  |   File "/usr/local/lib/python3.10/site-packages/flask_login/utils.py", line 13, in <module>
todolist  |     from werkzeug.security import safe_str_cmp
todolist  | ImportError: cannot import name 'safe_str_cmp' from 'werkzeug.security' (/usr/local/lib/python3.10/site-packages/werkzeug/security.py)
todolist  |
todolist exited with code 2

Found this on stackoverflow. Seems that werkzeug removed the required method.
I've updated all dependencies and it built for me. Will push the fix in a second.


Google Play Music updated web interface

Breaks interface in Radiant Player.
See Google blog post

1.11.2 (dark theme) still works for me with the new Music UI.

even on the dark theme i get an error since this update by google

FWIW, the other sections are still usable and you can still listen to music. The homepage is just broken.

I figured it out! You just need to log out and log in again to get the new homepage to work in Radiant. The visuals need tweaking (especially if you're using one of the dark themes) but it works!

Closing this for now - if you're experiencing this issue, try logging out and logging back in as @shawn-mitch mentioned.
Feel free to post back if that doesn't solve.


Remove hardcoded "myriad" prefix.

Currently the API root is hard-coded to "myriad".  Investigate whether this root can be dynamically assigned from a config file instead.

Override Attribute "Route" class, read from file.
Look at SynapseAuthorization attribute in Synapse server.

Added optional "RoutePrefix" field into server config file that overrides default "myriad" route prefix.   Actually, this will allow the override of ANY RoutePrefix for an assembly defined in the server config.


⚠ Samsung has degraded performance

In d71f328, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2904 ms

Resolved: Samsung performance has improved in ac3e6b4.


AttributeError: 'NoneType' object has no attribute 'split'

Describe the bug
Throwing AttributeError: 'NoneType' object has no attribute 'split' when trying to mk_truss
To Reproduce
Directly taken from "Quickstart: making a truss"
...
rfc.fit(data_x, data_y)

# Create the Truss (serializing & packaging model)
tr = truss.mk_truss(rfc, target_directory="iris_rfc_truss")  # this line fails
...

Expected behavior
Expected to see working sample
Screenshots/Logs
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[149], line 15
     12 rfc.fit(data_x, data_y)
     14 # Create the Truss (serializing & packaging model)
---> 15 tr = truss.mk_truss(rfc, target_directory="iris_rfc_truss")
     17 # Serve a prediction from the model
     18 # tr.server_predict({"inputs": [[0, 0, 0, 0]]})

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/build.py:267, in mk_truss(model, target_directory, data_files, requirements_file, bundled_packages)
    257 def mk_truss(
    258     model: Any,
    259     target_directory: str = None,
   (...)
    264     # Some model objects can are callable (like Keras models)
    265     # so we first attempt to make Truss via a model object
--> 267     model_scaffold = mk_truss_from_model_with_exception_handler(
    268         model, target_directory, data_files, requirements_file, bundled_packages
    269     )
    270     if model_scaffold:
    271         return model_scaffold

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/build.py:203, in mk_truss_from_model_with_exception_handler(*args)
    200 def mk_truss_from_model_with_exception_handler(*args):
    201     # returns None if framework not supported, otherwise the Truss
    202     try:
--> 203         return mk_truss_from_model(*args)
    204     except FrameworkNotSupportedError:
    205         return None

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/build.py:99, in mk_truss_from_model(model, target_directory, data_files, requirements_file, bundled_packages)
     97 else:
     98     target_directory_path = Path(target_directory)
---> 99 model_framework.to_truss(model, target_directory_path)
    100 scaf = TrussHandle(target_directory_path)
    101 _update_truss_props(scaf, data_files, requirements_file, bundled_packages)

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/model_framework.py:69, in ModelFramework.to_truss(self, model, target_directory)
     61 python_version = map_to_supported_python_version(infer_python_version())
     63 # Create config
     64 config = TrussConfig(
     65     model_name=self.model_name(model),
     66     model_type=self.model_type(model),
     67     model_framework=self.typ(),
     68     model_metadata=self.model_metadata(model),
---> 69     requirements=self.requirements_txt(),
     70     python_version=python_version,
     71 )
     72 with (target_directory / CONFIG_FILE).open("w") as config_file:
     73     yaml.dump(config.to_dict(), config_file)

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/model_framework.py:29, in ModelFramework.requirements_txt(self)
     27 def requirements_txt(self) -> List[str]:
---> 29     return list(infer_deps(must_include_deps=self.required_python_depedencies()))

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/environment_inference/requirements_inference.py:44, in infer_deps(must_include_deps)
     41 if not must_include_deps:
     42     must_include_deps = set()
---> 44 pkg_candidates = _extract_packages_from_frame(relevant_stack[0].frame)
     45 imports = must_include_deps.union(pkg_candidates)
     46 requirements = set([])

File ~/.virtualenvs/datasciense/lib/python3.8/site-packages/truss/environment_inference/requirements_inference.py:98, in _extract_packages_from_frame(frame)
     96     pkg_name = val.__name__.split(".")[0]
     97 elif hasattr(val, "__module__"):
---> 98     pkg_name = val.__module__.split(".")[0]
     99 else:
    100     continue

AttributeError: 'NoneType' object has no attribute 'split'

Desktop (please complete the following information):

OS: Linux 5+
Environment versions: python 3.8.12, sklearn 1.1.3 with poetry 1.3.1
Truss version: 0.1.12

Additional context
Nothing

@marquicus Thanks a lot for reporting the issue, I've attempted a fix in the attached PR. We'll land this asap after review and create a release for you to see if that fixes the issue in question.

@marquicus Thanks a lot for reporting the issue, I've attempted a fix in the attached PR. We'll land this asap after review and create a release for you to see if that fixes the issue in question.


Differences in platform output

Test case - DiffSimpleIncludeAndMacros
Problem - On Posix, slpp_unit\work\top_1.v and slpp_all\work\top_1.v are identical but the same are not identical on Windows.
Tracing the code, everything tells me that Windows is right and that the files shouldn't be identical. But that raises the question as to why the files are identical on Posix.
Attached are the outputs from both Windows & Linux. Notice the lines 10 & 12 in files slpp_unit\work\top_1.v and slpp_all\work\top_1.v. Those lines are non-empty with fileunit=true but and empty with fileunit=false. top.v gets parsed ahead of top_1.v and in the case of fileunit=false run, the macro _my_incl_vh_ would be defined and so the rest of the content gets skipped i.e. non active branch. When run with fileunit=true flag, those same macros aren't defined (though top.v is still parsed ahead of top_1.v) and so the content are part of active branch. That argument holds good only for Windows. On Posix, fileunit flag has no affect and those lines are empty in both cases.
Appreciate some insight into this issue. I have already spent far too long trying to narrow down this but haven't made any real progress.
DiffSimpleIncludeAndMacros_windows.zip
DiffSimpleIncludeAndMacros_linux.zip

Never mind. Figured it out.

There is something not right here with this test. I thought I figured it out yesterday night but I was wrong. I still can't seem to reproduce the results locally for this test to match what CI is producing. And, for what is being produced locally, it seems correct and CI wrong but I can't find a reason as to why the CI build is doing what it is doing. Attached are the logs for this test from both the CI and local.
DiffSimpleIncludeAndMacros_local.zip
DiffSimpleIncludeAndMacros_ci.zip
Problem statement - CI reports differences in only 2 preprocessed source files - top_3.v and top_4.v. However, local build reports differences in 4 files - top_1.v, top_2.v, top_3.v and top_4.v.
Here's what I have learnt so far -

Source files are processed strictly in the order - top.v, top_1.v, top_2.v, top_3.v, top_4.v

CompilationUnit holds the macros that are found during preprocessing

PreprocessFile instances use a shared CompilationUnit instance when fileunit == false and unique instances when fileunit==true. That would mean, when fileunit==true, macro declarations found during the processing of one file will not be available to the other unless both files declare the macro (potentially thru' a common header include).

When fileunit == false

When top.v gets preprocessed, it registers a macro "_my_incl_vh_" because it includes "my_incl.vh". This is a file header guard which controls the content of entire file.
top_1.v gets processed next. The macro registered is still visible since it uses the same instance of CompilationUnit. So, the entire body of the file "_my_incl_vh_" should be considered inactive and discarded.
This is all good. I can reproduce this result as expected.

When fileunit == true

In step 1 above, When processing top.v the macro will get registered for the first time. However, the macro is registered with a unique instance of CompilationUnit which belong to this specific PreprocessFile instance and is not visible to any other PreprocessFile instance (or contained CompilationUnit). The content of "_my_incl_vh_" would be active and be included in the preprocessed output for top.v. So far so good.
In step 2 above, when processing top_1.v, the CompilationUnit is still pristine and unmodified because it belongs to a different instance of PreprocessedFile. The PreprocessedFile instance for top.v and the one for top_1.v are distinct and have no relation. It doesn't have the same macro that was defined while processing top.v. So, the content of "_my_incl_vh_" should still be active and included. BUT it is not, at least not on the CI build.

Given the two scenarios, the output of fileunit==true and fileunit==false will always be different when the source files have macros involved. How is this supposed to work?

As you can see your starting statement is incorrect:
Problem - On Posix, slpp_unit\work\top_1.v and slpp_all\work\top_1.v are identical but the same are not identical on Windows.
./slpp_unit/work/_6142509188972423790/top_1.v and ./slpp_all/work/_6142509188972423790/top_1.v  are different on Posix (See below).
Did you mingle with the -nohash by any chance? Your paths do not contain hash.
alain@alain-xps:~/Surelog/tests/DiffSimpleIncludeAndMacros$ /home/alain/Surelog/build/bin/surelog top.v top_1.v top_2.v top_3.v top_4.v -d 0  +incdir+.+..  +libext+.v -writepp -y blah +incdir+foo -v lib.v  +define+N=8  -diffcompunit -mt  max -parse
|-------|------------------|-------------------|
|       |  FILE UNIT COMP  |  ALL COMPILATION  |
|-------|------------------|-------------------|
| FATAL |         0        |         0         |
|SYNTAX |         6        |         6         |
| ERROR |        23        |        25         |
|WARNING|        12        |        12         |
| INFO  |                  |                   |
| NOTE  |         6        |        16         |
|-------|------------------|-------------------|

FILE UNIT LOG: "./slpp_unit/surelog.log"
ALL FILES LOG: "./slpp_all/surelog.log"

DIFFS:
./slpp_unit/file.lst and ./slpp_all/file.lst
./slpp_unit/work/_6142509188972423790/top_1.v and ./slpp_all/work/_6142509188972423790/top_1.v
./slpp_unit/work/_6142509188972423790/top_2.v and ./slpp_all/work/_6142509188972423790/top_2.v
./slpp_unit/work/_6142509188972423790/top_3.v and ./slpp_all/work/_6142509188972423790/top_3.v
./slpp_unit/work/_6142509188972423790/top_4.v and ./slpp_all/work/_6142509188972423790/top_4.v

Good to know I haven't lost my sanity, at least not yet. :)
The log you just posted from your local run is exactly what I am seeing as well on my local run. But the log in the repository is different. It shows differences only in two files top_3.v and top_4.v. In fact, I just checked the logs from the latest CI build as well and it's the same - only two files are different. There's a difference in number of notes from unit compilation as well. My local output matches your local run but not the log in the repository. However, CI builds are generating different outputs.
Why are CI builds different from local builds? I can't seem to get a successful regression run locally. Try running the following command locally.
python scripts/regression.py run --filters DiffSimpleIncludeAndMacros

Attached is the output from Build 4108
DiffSimpleIncludeAndMacros.zip

All good here:
alain@alain-xps:~/Surelog$ python3 scripts/regression.py run --filters DiffSimpleIncludeAndMacros
Starting Surelog Regression Tests @ 2022-07-08 18:23:00.501256
Environment:
      command-line: scripts/regression.py run --filters DiffSimpleIncludeAndMacros
   current-dirpath: /home/alain/Surelog
 workspace-dirpath: /home/alain/Surelog
     build-dirpath: /home/alain/Surelog/build
  surelog-filepath: /home/alain/Surelog/build/bin/surelog
uhdm-dump-filepath: /home/alain/Surelog/build/third_party/UHDM/bin/uhdm-dump
roundtrip-filepath: /home/alain/Surelog/build/bin/roundtrip
     test-dirpaths: /home/alain/Surelog/tests; /home/alain/Surelog/third_party/tests
    output-dirpath: /home/alain/Surelog/build/regression
   multi-threading: None
  multi-processing: None
          max-jobs: 1
         max-tests: 598
 blacklisted-tests: 7
    filtered-tests: 1

Running 1 tests ...
Running DiffSimpleIncludeAndMacros ...
... DiffSimpleIncludeAndMacros Completed.

Results:
  +----------------------------+--------+-------+--------+-------+---------+------+----------+---------+---------+-----------+
  | TESTNAME                   | STATUS | FATAL | SYNTAX | ERROR | WARNING | NOTE | CPU-TIME | VTL-MEM | PHY-MEM | ROUNDTRIP |
  +----------------------------+--------+-------+--------+-------+---------+------+----------+---------+---------+-----------+
  | DiffSimpleIncludeAndMacros | PASS   |     0 |     12 |    48 |      24 |   18 |     0.00 |      29 |      14 |       0/0 |
  +----------------------------+--------+-------+--------+-------+---------+------+----------+---------+---------+-----------+

Summary:
  +----------------+---------------------------------+
  | PASS           | 1                               |
  | DIFF           | 0                               |
  | FAIL           | 0                               |
  | FAILDUMP       | 0                               |
  | SEGFLT         | 0                               |
  | NOGOLD         | 0                               |
  | TOOLFAIL       | 0                               |
  |                |                                 |
  | MAX CPU TIME   | 0 (DiffSimpleIncludeAndMacros)  |
  | TOTAL CPU TIME | 0                               |
  | MAX WALL TIME  | 0 (DiffSimpleIncludeAndMacros)  |
  | MAX MEMORY     | 14 (DiffSimpleIncludeAndMacros) |
  +----------------+---------------------------------+

Surelog Regression Test Completed @ 2022-07-08 18:23:00.807616 in 0 seconds

The output you showed in your previous message does not match the golden log though and yet the regression succeeded? And what about the files being identical in the output from CI build?

Here's the output from the CI run. This is not the same as the one you have above -
Compare the number of note entries in unit compilation and the number of files that are different.
|-------|------------------|-------------------|
|       |  FILE UNIT COMP  |  ALL COMPILATION  |
|-------|------------------|-------------------|
| FATAL |         0        |         0         |
|SYNTAX |         6        |         6         |
| ERROR |        23        |        25         |
|WARNING|        12        |        12         |
| INFO  |                  |                   |
| NOTE  |         2        |        16         |
|-------|------------------|-------------------|

FILE UNIT LOG: "../../build/regression/DiffSimpleIncludeAndMacros/slpp_unit/surelog.log"
ALL FILES LOG: "../../build/regression/DiffSimpleIncludeAndMacros/slpp_all/surelog.log"

DIFFS:
../../build/regression/DiffSimpleIncludeAndMacros/slpp_unit/file.lst and ../../build/regression/DiffSimpleIncludeAndMacros/slpp_all/file.lst
../../build/regression/DiffSimpleIncludeAndMacros/slpp_unit/work/_6142509188972423790/top_3.v and ../../build/regression/DiffSimpleIncludeAndMacros/slpp_all/work/_6142509188972423790/top_3.v
../../build/regression/DiffSimpleIncludeAndMacros/slpp_unit/work/_6142509188972423790/top_4.v and ../../build/regression/DiffSimpleIncludeAndMacros/slpp_all/work/_6142509188972423790/top_4.v

But it is the same when ran thru' regression.

I have no time to investigate. Ignore this test on Windows. Leave it in the regression. It passes on Linux.

Yes, it passes on both Linux & Windows in the CI but the results are wrong for both platforms, in fact wrong on every platform because regression is passing across all platforms. I can spend more time investigating the issue. Can you spend a few minutes to confirm this is in fact a real problem and I am not on a wild goose chase.

Put the test in blacklist on all platforms and ignore it.
Sent from Yahoo Mail on Android
On Sun, Jul 10, 2022 at 11:21, @.***> wrote:
Yes, it passes on both Linux & Windows in the CI but the results are wrong for both platforms, in fact wrong on every platform because regression is passing across all platforms. I can spend more time investigating the issue. Can you spend a few minutes to confirm this is in fact a real problem and I am not on a wild goose chase.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were assigned.Message ID: @.***>


Next-Cloudinary still cannot access NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME

I have properly set-up the  in the root layout, and I have properly set the environment variable in running a container, but I still get this error
Error: A Cloudinary Cloud name is required, please make sure NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME is set and configured in your environment.
But in my .env file where I use with docker compose to run a container, I have already set it there
NEXT_PUBLIC_CLERK_SIGN_IN_URL=/sign-in
NEXT_PUBLIC_CLERK_SIGN_UP_URL=/sign-up
NEXT_PUBLIC_CLERK_AFTER_SIGN_IN_URL=/
NEXT_PUBLIC_CLERK_AFTER_SIGN_UP_URL=/
NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY=pk_test_bmVhdC1wYW5kYS0xMi5jbGVyay5hY2NvdW50cy5kZXYk
NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME="cloudnamehere"
The cloud name works in development
Also, when I build the NextJs application with a NEXT_CLOUDINARY_CLOUD_NAME variable, I do not get an error, but when I put a different value for that variable in running a container, it still uses the old variable, so in essence, it is not able to read the variable I put at run-time

@reannu123 - Can you tell us a bit more about your build pipeline?

Are you using the app or pages directory?
What version of Next.js and next-runtime-env are you using?
Are you setting the env vars during the build, during runtime or both?
In which file are you configuring next-runtime-env and where are you configuring Cloudinary/Clerk?
How are you accessing the env vars? Using process.env. or using the env() util?

Hello @HofmannZ thanks for responding,

Im using app directory
Next.js 14.1.0 and the latest version of next-runtime-env (3.2.1)
Im trying to set the vars at build time and runtime (docker run). When I set NEXT_PUBLIC_CLOUDINARY_CLOUD_NAME at build time, I do not get an error but the runtime vars I set for it is not being read (it is using the var I set at build time, which I do not want). When I don't set any var at build time, but I set the var at runtime, i get a client exception error stating that I do not have the Cloud name set. So in conclusion, it's not reading the var I set at runtime.

The weird thing is, the other NEXT_PUBLIC variables work as intended.
With next-runtime-env, it works when I change the clerk NEXT_PUBLIC vars (doesn't work without next-runtime-env), but the one used by next-cloudinary doesn't work for some reason.

Sorry I do not understand, I just followed the set-up where I place the PublicEnvScript in the head tag in the main layout.tsx,
However, the env vars are being accessed by the libraries that use them, so I'm not sure how Next-Cloudinary accesses the variables, but Clerk does it properly.

This issue was resolved when next-cloudinary added support for custom cloud names as a parameter. From there, I used the env() util and it worked


extension is broken on chrome/firefox

Extension does not seem to be working on chrome and firefox. I tried manual installation for both.

Yes. Because dndbeyond had changes, I need to rework some of the selectors. Once dndbeyond is done with their changes (as far as I know, some are still coming), I'll fix it.
Sent with Shortwave https://www.shortwave.com?utm_medium=email&utm_content=signature&utm_source=aGlicmFnaW1vd0BnbWFpbC5jb20=
On Wed Sep 11, 2024, 04:06 PM GMT, Forrest Miller @.***> wrote:

Extension does not seem to be working on chrome and firefox. I tried manual installation for both.
—
Reply to this email directly, view it on GitHub https://github.com/Orbytale/BeyondMarkdown/issues/12, or unsubscribe https://github.com/notifications/unsubscribe-auth/BGQUBHMORYC2IH7DDYCNCFTZWBTCDAVCNFSM6AAAAABOBJXRO2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZDAMJVGUZDEMI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


TTGO T-Beam V1.1 - analogRead - Uplink not scheduled because TxRx pending

I have add the following (see below). It sends one uplink and then "Uplink not scheduled because TxRx pending", not sending any more uplinks (see serial print)
If I remove the analogRead it sends uplinks as per normal scheduled time interval.
#define loadV 33
And then in --- if (LMIC.devaddr != 0) ---
        //Read Voltages

        int16_t int16_loadV;
        int16_loadV = 0;

        float Vload = analogRead(loadV);
        int16_loadV = Vload;
        Serial.print("int16_loadV: " );
        Serial.println(int16_loadV);

Serial print:
000000772779:  Event: EV_JOINED
               Network Id: 19
               Device Address: 2xxxxxx4
               Application Session Key: A5-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-CC
               Network Session Key:     7A-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-xx-1F

000000773629:  doWork job started
int16_loadV: 22
000000773829:  Input data collected
               downLink: 0
000000774275:  Packet queued
000000774492:  Event: EV_TXSTART

000002648629:  doWork job started
int16_loadV: 4095
000002648639:  Input data collected
               downLink: 0
000002648639:  Uplink not scheduled because TxRx pending

It's not clear to me.
Is this a question, a proposal for a fix or something else?

Hi,
I can not do a analogRead, so it's is a fix that needs to be done.
As soon as you use analogRead, it stops transmitting any date on the LoRaWan side.
Thank you

I can not do a analogRead

Sounds like a problem you have.

so it's is a fix that needs to be done.

Still unclear. Is the fix in your above example? If so that is not clear at all.

As soon as you use analogRead, it stops transmitting any date on the LoRaWan side.

Apparently there is some conflict between the use of analogRead, the LMIC library and your board.
Your post lacks essential details (e.g. your board, more detailed code).
This also does not sound like a LMIC-node specific issue so I cannot help you with that.
I suggest to use the TTN forum for more help instead.
In case you have a specific fix, I might be able to add it to LMIC-node.

As soon as I add a analogRead to LIMC-Node, the issue occurs where it stops sending uplinks, tried it on 3 different boards (TTGO T-Bean V1.1) and the problem resists.
I then took a Adafruit Feather, this preforms the analogRead.
So it seems the problem might be the ESP32.

Your post lacks essential details (e.g. your board

TTGO T-Beam v1.0, v1.1

detailed code

Add a analogRead in the section where

// Collect input data.
// For simplicity LMIC-node uses a counter to simulate a sensor.
// The counter is increased automatically by getCounterValue()
// and can be reset with a 'reset counter' command downlink message.

float Vload = analogRead(loadV);

I overlooked the board in the title, read it at first but then couldn't find it in the text.🤦♂
Good you were able to also test it with a different board.
It may be caused by several reasons but I have no clue. It seems to cause a timing issue that effects operation/timing of LMIC. This may be a general ESP32 related issue but could also be TTGO T-Beam specific, possibly related to its power management chip (configuration).

I were making my references to their, LilyGO, board layout, they don't mention on the board layout the LoRa connections. Although I am full aware of the pin mapping in bsf_ttgo_t_beam_v1.h file.
Using analogRead on pin 2 and it works 100%.
PS. I do own a shotgun, I thing both of us used it ;-)

I'm glad I was able to help fix your issue.
As expected LMIC-node was not the culprit for your issue.
_A lot of preparation went into making LMIC-node work out of the box with its supported boards.
It helps prevent others having to go through a steep learning curve first and step into the many (at first sight invisible) possible pitfalls, of which poor product documentation from the manufacturer is only one.


Manage Local Storage binding file not found

I started to use the Local Storage binding for my service application.
During my tests I notice that if I try to retrive a file that doesn't exist yet, I expect to receive a NotFound exception or a BindingResponse with the Data field null, instead I received an Internal exception just like in the image.
Would it be possible to fix to receive one of the two proposals?
Also is it possible to add in the yaml component a metadata field, time to live, for the files?

Another bug I found during my tests is that if I get the data inside a file and then delete this, I receive an exception in which is described how my file cannot be delete because it is used by another process:

Maybe is it cause because the file isn't close after getting the data information?

Any update to this problems?

@ItalyPaleAle any quick thoughts? Maybe your PR already addressed this by chance.

@federicocodo Thanks for the report.

On the first issue, we can't change the returned status code sadly, nor we can return an empty data (in that case, it wouldn't be possible to distinguish whether the file doesn't exist or it exists but it's empty). However, I've changed the code so the error of the exception contains file not found: <path> and you can look at that.
On the second issue, looks like a file descriptor was not closed. Fixed

Fixes in #2449 and will be shipped with Dapr 1.10

@ItalyPaleAle Thanks a lot for the fixes, however what about the option to add in the yaml component or in the metadata field of the request class a time to live for files?

@ItalyPaleAle Thanks a lot for the fixes, however what about the option to add in the yaml component or in the metadata field of the request class a time to live for files?

I'm not in favor of that feature as a maintainer right now. That doesn't really fit the paradigm of this component. I want to avoid to introduce a separate thread for watching expiration times of files and then deleting them.
File a separate feature request for that please and we can consider it in the future.

I agree with @berndverst . However we should probably consider offering a state store for accessing local files. The TTL could be a feature of that.

Let's not brainstorm that here @ItalyPaleAle.
Right now I'm not convinced that files + TTL should be combined in any form. 😅 but if there is enough demand, sure.


[Bug] When switching between inputs, the focused input bounces out of view

Description
When the email field is selected, and i click on the password field the scrollview will scroll down appropriately, but bounces back up, which covers the entire password input
If I click on the password field first, the issue does not happen.
Example:
https://github.com/APSL/react-native-keyboard-aware-scroll-view/assets/24516988/640834ac-8934-4e82-b653-44fe7d123baa
Code snippet
  <KeyboardAwareScrollView extraScrollHeight={-insets.bottom}>
   {...input data}
  </KeyboardAwareScrollView>

note: Extra scroll height prop is just because I'm using a safe area view. Removing SafeAreaView does not fix the bug

the issue lies within the keyboardOpeningTime. If i do this i am able to bypass the weird bounce animation. However, im not exactly sure what the memory implications of using numbers.max_value are
 <KeyboardAwareScrollView extraScrollHeight={-insets.bottom} keyboardOpeningTime={Number.MAX_VALUE} >

Thank you @subhyde


Iterating an int[] where it has become null

Description:

Define a record where an int[] intArray is optional.
Write a code where you get this int[] field and iterate.
When you do not specify a value to int[], Ballerina code crashes at the point of iterate. It does not give out an error or the codes below that point is not executed.

Affected Versions:
0.991.0

This issue is fixed in ballerina 1.0.0-alpha
Consider following:
import ballerina/io;
type RecordName record {
    int[] ar?;
};

public function main(string... args) {
    RecordName r = {};
    r.ar.forEach(function (any a) {
        io:println(a);
    });
}

This produces error: invalid operation: type 'RecordName' does not support field access for non-required field 'ar'


Mock test server based on hyper

Connects-To: #3
This commit is only for implementing the mock test server functionality, but we do not include yet real tests based on the JSON schemas.
The changes in src/main.rs are just to be able to verify that the test server is working. The communication with the endpoint will go into a different module in a separate commit.
Since this is not a library, but an application, integration testing will not import code from the main crate. It is done by running the application itself. This is why the most straightforward way for testing is to compare its stdout/stderr and exit code to expected values.
For the same reason an OS_CONFIG_BASE_URL environment variable is introduced. The mock JSON server is behind the address provided by it.

The CircleCI test is failing because the image does not contain the development package of openssl used for linking. And the branch is not rebased. I will fix those before merging.

Alright, now the application compiles on CircleCI - I added the libssl-dev dependency to the image and upgraded it to use a more recent Rust nightly. The mock server test is passing as well. Branch is rebased now.

BTW in the tests I used a lot of unwraps, but that is fine there, since if an unwrap fails for some reason, the test will fail as well.


Lights do not appear at the launch

Probably an unity editor issue, problem fixes itself after 2-3 minutes.

Yes, it's editor issue 100%

Too many REALTIME light sources at SAME position cause this.

Turns out there's a "Per Object Limit" which causes this problem. Easy fix.


Bundlewatch action

Bundlewatch should report statuses on PR: should try to authorize app by someone else, I guess, to target the Organization — @Lausselloic would you mind trying https://service.bundlewatch.io/setup-github and request access for organization, if that's ever proposed to you?
I think I authorized it for my personnal acount only and cannot get back.

The organization approved access for Bundlewatch so let's see in the next PR. However I didn't get any new token on their behalf, not sure my own token will suffice.

Works fine on #548 <3