Integration test requiring training via Ludwig failing on personal machine

Search before asking

[X] I have searched the EvaDB issues and found no similar bug report.

Bug
$ ~ PYTHONPATH="." python -m pytest test/integration_tests/long/test_model_train.py -k 'test_ludwig_automl'
ERROR    evadb.utils.logging_manager:plan_executor.py:182 Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument weight in method wrapper_CUDA__native_batch_norm)
Traceback (most recent call last):
  File "/home/hershd23/Desktop/evadb/evadb/executor/plan_executor.py", line 178, in execute_plan
    yield from output
  File "/home/hershd23/Desktop/evadb/evadb/executor/project_executor.py", line 34, in exec
    batch = apply_project(batch, self.target_list, self.catalog())
  File "/home/hershd23/Desktop/evadb/evadb/executor/executor_utils.py", line 42, in apply_project
    batches = [expr.evaluate(batch) for expr in project_list]
  File "/home/hershd23/Desktop/evadb/evadb/executor/executor_utils.py", line 42, in <listcomp>
    batches = [expr.evaluate(batch) for expr in project_list]
  File "/home/hershd23/Desktop/evadb/evadb/expression/function_expression.py", line 129, in evaluate
    outcomes = self._apply_function_expression(func, batch, **kwargs)
  File "/home/hershd23/Desktop/evadb/evadb/expression/function_expression.py", line 188, in _apply_function_expression
    return func_args.apply_function_expression(func)
  File "/home/hershd23/Desktop/evadb/evadb/models/storage/batch.py", line 173, in apply_function_expression
    return Batch(expr(self._frames))
  File "/home/hershd23/Desktop/evadb/evadb/udfs/abstract/abstract_udf.py", line 36, in __call__
    return self.forward(args[0])
  File "/home/hershd23/Desktop/evadb/evadb/udfs/ludwig.py", line 33, in forward
    predictions, _ = self.model.predict(frames, return_type=pd.DataFrame)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/api.py", line 895, in predict
    predictions = predictor.batch_predict(
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/models/predictor.py", line 142, in batch_predict
    preds = self._predict(batch)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/models/predictor.py", line 188, in _predict
    outputs = self._predict_on_inputs(inputs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/models/predictor.py", line 324, in _predict_on_inputs
    return self.dist_model(inputs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/models/ecd.py", line 136, in forward
    combiner_outputs = self.combine(encoder_outputs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/models/ecd.py", line 81, in combine
    return self.combiner(encoder_outputs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/combiners/combiners.py", line 451, in forward
    hidden, aggregated_mask, masks = self.tabnet(hidden)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/ludwig/modules/tabnet_modules.py", line 113, in forward
    features = self.batch_norm(features)  # [b_s, i_s]
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/modules/batchnorm.py", line 171, in forward
    return F.batch_norm(
  File "/home/hershd23/Desktop/evadb/env/lib/python3.10/site-packages/torch/nn/functional.py", line 2450, in batch_norm
    return torch.batch_norm(
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu! (when checking argument for argument weight in method wrapper_CUDA__native_batch_norm)

This could be due to my machine configuration however I was asked to report this for further analysis
Environment

Python 3.10
OS Ubuntu 22

Are you willing to submit a PR?

[ ] Yes I'd like to help by submitting a PR!

Found this
https://stackoverflow.com/questions/66091226/runtimeerror-expected-all-tensors-to-be-on-the-same-device-but-found-at-least
Maybe a to_device() param is missing for the model

Problem also exists on ada-01 server. However, the training work on a machine that without GPU. Worth more investigation here. Thanks Hersh for raising the issue.

The problem has been fixed on  ada-01, with a new clean install. pip install ".[dev,ludwig,qdrant]". Hi @hershd23, could you verify whether the problem has also been fixed on your personal machine.

Hmm this still isn't resolved on my machine.
Steps I did

Pulled from latest staging
Installed the packages with the command you specified
Re ran the model training test.

It still fails with the same message

Hmm this still isn't resolved on my machine.
Steps I did

Pulled from latest staging
Installed the packages with the command you specified
Re ran the model training test.

It still fails with the same message

Could you post the output of pip freeze ?

DMed you the output file


Incorrect subtype relation with boolean

Description:
type B boolean;
type TF true|false;

B should be a subtype of TF.
types.isAssignable(B, TF) should be true.
Affected Versions:
swan-lake beta6

type B boolean;
type TF true|false;

B should not be a subtype of TF. types.isAssignable(B, TF) should be false.

Shouldn't this be the opposite?
B should be a subtype of TF. types.isAssignable(B, TF) should be true?

Yes, it was my mistake, edited the original issue.

The behavior with SL 2201.8.2
type B boolean;
type TF true|false;

public function main() {
    B x = true;
    TF y = x; // incompatible types: expected 'TF', found 'B'
}


Add winrt module for opencv

A new module "winrt" is added, it exposes WinRT interfaces which can be consumed by any supported programming languages including c++, c# and javascript in a windows universal application.
In this initial commit it contains the support for some basic functions in imgproc, videoio and objectdetect.

it's too big chunk of code for a single platform support. I'm afraid, we cannot accept it. Please, consider contributing it to http://github.com/itseez/opencv_contrib instead. The basic winrt support is already present in videoio and highgui modules.


Multiple sessions

I think is a good enhancement to make this bundle can handle many refresh tokens for different sessions. Do you have any idea how can implement this?

+1

@wujku im thinking about it, i think is better extract refreshToken from UserEntity and make a new entity called Token with User_ID, RefreshToken.
In this way we could have more than one refresh token per user.
The flow then works like this:
A user logs in (with his credentials) then we generate a new refresh_token in system.
A user use his refresh_token to log in, then we revoke the current refresh_token and generate another one.
One problem im seeing is how to revoke unused refresh_tokens, i mean, if i log in in one app using credentials many times, i generate many refresh_tokens that persist forever.
Do you think is a good practice include an expiration date in Token entity?
In this way we could invalidate refresh_tokens when date expires.

You could add date of expiration to Token entity.
Token expiration time may be 2-3 times longer than json token.
I would add the parameter to set the expiration time of the Token manually.

Yes we can configure token_refresh ttl as a config parameter.
And maybe create a command to use as cron to remove invalid refresh tokens

Or create a simple query to clean the old entries during refresh action.

@markitosgv at first sight everything looks good.
Does Username is necessary when refreshing the token? In my opinion, it would suffice use UserRefreshTokenManager and get associated user by refresh token from request.

im going to open another issue about this question and unique token problem


Fix initial gender selection.

Both selections got man ( neurohacker ), no woman ( drophunter ), it's wrong.

@soulofmischief I tested this has been fixed by @memelotsqui and merged into new-ui-implementation, can be closed.


@azure/video-analyzer-edge - cspell found spelling errors in public API surface

Spell check scanning of package at sdk/videoanalyzer/video-analyzer-edge detected spelling errors in the public API surface. This directory is opted out of PR spell checking in PR #17142 to keep PRs unblocked.
What to do

Ensure Node.js is installed (https://nodejs.org/en/download/).
Delete the entry in .vscode/cspell.json's ignorePaths field. It will look like: sdk/videoanalyzer/video-analyzer-edge/review/**/*.md. You need to do this to enable checking the files.
From the root of the repo run spell check using ./eng/common/spelling/Invoke-Cspell.ps1 -ScanGlobs "sdk/videoanalyzer/video-analyzer-edge/review/**/*.md"
Fix detections according to http://aka.ms/azsdk/engsys/spellcheck use the "False positives" section to fix false positives
Check in changes (including the change to .vscode/cspell.json where the ignorePaths is updated to remove the entry for this service.). You may need to run git add -f .vscode/cspell.json to force adding the changes to the file in git.

Spell checking output
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:57:41 - Unknown word (onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:60:41 - Unknown word (onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:83:28 - Unknown word (Onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:91:28 - Unknown word (Onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:92:23 - Unknown word (Onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:107:92 - Unknown word (Grpc)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:114:68 - Unknown word (Grpc)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:130:13 - Unknown word (Grpc)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:131:40 - Unknown word (Grpc)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:253:5 - Unknown word (Abgr)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:253:13 - Unknown word (abgr)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:254:5 - Unknown word (Argb)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:254:13 - Unknown word (argb)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:256:5 - Unknown word (Bgra)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:256:13 - Unknown word (bgra)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:302:18 - Unknown word (Onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:626:264 - Unknown word (onvif)
./sdk/videoanalyzer/video-analyzer-edge/review/video-analyzer-edge.api.md:626:288 - Unknown word (onvif)

Acknowledging


Introduce ProcessWaves

part of #155
Note that behavior of some function has changed.

See changes in test for reorder algorithm (Now we save original order of the functions that could be execute in parallel)
See estimateWaves function
It used to be

let f = estimateWaves :: [F String Int] -> [String] -> M.Map String Int
     l1 = loop 0 "b2" ["a1"]
     l2 = loop 1 "c" ["b1", "b2"]
     a = add "a1" "b1" ["c"]
     M.fromList [] @=? f [l1, l2, a] [] -- because there's no ready vars

let f = estimateWaves :: [F String Int] -> [String] -> M.Map String Int
     l1 = loop 0 "b2" ["a1"]
     l2 = loop 1 "c" ["b1", "b2"]
     a = add "a1" "b1" ["c"]
     M.fromList [("a1",0),("b1",1),("b2",1),("c",0)] @=? f [l1, l2, a] ["a1", "b1", "b2"]

And current behavior are visible in tests

Sorry for such amount of notes…


IdentityKeys are not always immediately saved when trying to trust identities

This is not reliable reproducible, which makes it very hard to debug. I assume, that OmemoService.processBundle() writes the identityKey lazy to disk, so the when the key is requested for trusting, it is null. I'm not yet sure though.
Restarting the client helps.

It might be worth to take a look at OmemoStore.initializeOmemoSessions(). Maybe parts of this method have to be called for new sessions?

Fixed


Make get_texture_id report autoshader material

Description
It currently vec3(base/overlay/blend) but ignores the autoshader. If the autoshader is enabled, it should report that texture_id instead.

Pending architechture changes, this should just be a case of duplicating the autoshader calculations to get correct blend value and reading the material settings for the auto texture ids / slope / falloff

https://github.com/TokisanGames/Terrain3D/pull/371 has been merged. Now the Material can directly access Storage.


Using ico file in system tray

I modified some code so that Virgo's system tray icon can use ico files, and it allows for transparent backgrounds, high DPI, and a high degree of customisation.
The address of the repository is virgo-icon

已收到，谢谢！


Set an ifname if there are no network interfaces

Fixes https://github.com/google/uuid/issues/31

@darkfeline: your changes were merged, right? https://github.com/google/uuid/commit/0cd6bf5da1e1c83f8b45653022c74f71af0538a4
Please close the PR.


dart/trigger_gc_in_native_test/3 is crashing on vm-kernel-precomp-android-release-arm

Started to fail on https://ci.chromium.org/p/dart/builders/ci.sandbox/vm-kernel-precomp-android-release-arm/1859 on seemingly unrelated change

/cc @rmacnak-google


Performing EndTypeChangeRequest for resource type "System.BizTalk:BizTalkAssembly".EXEC : error : Access to the path is denied.

I've been deploying this project for years but the last few releases I've been getting the above error. Sometimes it's after Gacing the Schema assembly but usually after Gacing the Maps assembly. Yesterday on my dev server it succeeded after the 3rd Redeploy. After 10 redeploy attempts on my QA server I gave up. I created an msi from the Admin console on my dev box with just the Map and Orchestration dll's and imported it successfully on the QA box. I used the bindings file that was generated during the failed BDF installation. I've attached the BDF Results files from the successful installation on the dev box and the last failed redeployment from the QA box. I didn't see how to attach multiple files so I combined them into 1 with the failure on top. Thanks.
Attachments
Both_DeployResults.txt
This work item was migrated from CodePlex
CodePlex work item ID: '11139'
Vote count: '1'

[tfabraham@2/14/2017]
In this case, the error is from BizTalk's BTSTask.exe, not from the Deployment Framework:
BTSTask.exe AddResource -Type:BizTalkAssembly -Source:"..\xxx.Payroll.Tax.Maps.dll" -ApplicationName:"xxx.Payroll.Tax" -Options:GacOnAdd,GacOnImport,GacOnInstall
As a result, I'd expect BizTalk Admin to generate the same sort of error.  You might find more information about this on the BizTalk MSDN/TechNet forums.
Did you ever find a resolution?

[UnknownUser@2/14/2017]

[johnamuller@2/15/2017]
No. My workaround is to create an msi from the Admin console on my Dev server, containing the dll's that weren't installed (Maps and Orchestrations). When we did the release it failed on the DR servers, so I imported the missing dll's and bindings from the Admin console, but the BDF install worked in Prod.

[tfabraham@2/15/2017]
Such an odd issue.  I've seen it before, and you're not alone by any means.  Seems like people rarely, if ever, find a cause because it always seems to affect only specific computers.
One person determined that a virus scanner was the cause, and got their admins to add scan exclusions on the folders that BizTalk touches.  That was mentioned in another thread here.
There are mentions of similar issues on StackOverflow and blogs too.
You could use Process Monitor to trace what's happening when BTSTask.exe runs, and try to catch the Access Denied error to see what file or directory is involved.  That's probably your best bet to find the root cause.
Are you OK if I close this issue?  This is a BizTalk/Microsoft Support issue, not BTDF.
Thanks,
Tom

[johnamuller@2/15/2017]
I'll look at those links. Thanks for your time and a great application. You can close this.

[UnknownUser@2/15/2017]

Issue closed by tfabraham with comment
This is something for Microsoft Support since it's generated by BTSTask.exe.
Reason closed
Not BTDF


Support for AOP

provide support for AOP
look into a way which will work with any IoC

NOTES:
Preferred Castle.Core DynamicProxy, there is no need to abstract this framework. (ATM) unless someone can suggest a reason why.

implemented in 0.4 release


Manually destroy devices when extension is disabled

Devices weren't being destroyed when the extension exists, leaving dangling signals and errors in the system logs.
Closes #24

Thank you!


Installation steps for impact framework is inconsistent across github and docs

Couple of points I noticed:

The installation of impact framework steps is talking about yarn whereas the github page is talking about npm. We  need to keep these 2 consistent.
The link https://github.com/Green-Software-Foundation/if#alternate-version which documents the alternate  version needs to be checked. It did not work for me. Also if you do  intend to keep this , recommend you name the first installation steps as one that uses npm and the second one as the one that uses yarn.

Thanks @srini1978 - good point, @narekhovhannisyan and I discussed it and think we should favour npm over yarn
We will:

remove the "alternate version" instructions
remove the yarn lock files from all our repositories
check all other related docs for yarn-based instructions.

Closing as complete


chore(SHUTTLE-1205): centralizing node unit test workflow

Workflow repo is private, so no access from here


[Bug] OAuth not working - invalid_request: Missing "redirect_uri" in request

Whilst running the react-colyseus example, after I authenticated I got this error:
{
    "code": 5000,
    "message": "OAuth2 Error: invalid_request: Missing \"redirect_uri\" in request."
}

Update: I have fixed it by adding in any URL at the Redirects at the OAuth tab of the developers console.

Update: I have fixed it by adding in any URL at the Redirects at the OAuth tab of the developers console.

OMG, thank you so much for this question. I have been debugging this issue for several hours now without any luck. I think this should either be patched and not happen, or at least properly communicated in some documentation.


Fixed intermittently failing test for Cassandra

The test fails when a process is already listening on port 7010. Fortunately, Cassandra provides a way to start the server on a random available port. Changed the code to use a random port instead of a hardcoded one.

Looks good! Let's merge this change into master, or wait for #55 to land first (I need to fix the Muzzle build in #55).


How can we prevent elm-live from monitoring node_modules/?

> elm-live Main.elm --output=hoverpad.js --open

Success! Compiled 1 module.
Successfully generated app.js
elm-live:
  You’ve added `node_modules/.bin/strip-indent`. Rebuilding!
elm-live:
  You’ve added `node_modules/watchify/node_modules/.bin/browserify`. Rebuilding!
elm-live:
  You’ve added `node_modules/npm/node_modules/request/node_modules/.bin/har-validator`. Rebuilding!
elm-live:
  You’ve added `node_modules/npm/node_modules/request/node_modules/http-signature/node_modules/.bin/sshpk-conv`. Rebuilding!

Refs https://github.com/tomekwi/elm-live/pull/90

Hey, thanks for opening 🙂 Can you please post more info about the version of elm-live and the OS you’re running?
Also, please read through https://github.com/tomekwi/elm-live/issues/97 for more context about picking up changes from node_modules. (TLDR: This is not intended and we’re still trying to put the short, self-contained, compilable example. Your help would be appreciated.)
I’m keeping this issue open though, because #97 seems to involve two cross-cutting concerns – and picking up node_modules is only one of them.

I have no idea how to write such a test. I am bit puzzle because you seem to be using **/*.{html,css,js} and **/*.elm so I don't see why this command are monitored to be honest.
I guess a way to reproduce the test is to run elm-live in a thread and touching some files in a fake node_module/.bin repository in the meantime and see that elm-live doesn't rebuild when we do.

After investigation it seems that the problem is coming from chokidar.
https://github.com/paulmillr/chokidar/issues/544

Any help on fixing this, it is really a pain while using Elm-live on a daily basis


Expect more specific document

Could anyone provide some guides about the whole flow? I cannot pass test when running mvn package.

Uh you can disable tests when compiling with maven. https://stackoverflow.com/questions/27953039/disable-the-integration-test-phase-in-pom-xml
mvn package -DskipTests should work ;D


Q: Dependabot does not respect semver version ranges specified in package-lock.json

Package manager/ecosystem
npm_and_yarn
Manifest contents prior to update

package.json
{
  "name": "sample",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1"
  },
  "author": "",
  "license": "ISC",
  "dependencies": {
    "angular": "1.5.7"
  }
}

package-lock.json
{
  "name": "semver",
  "version": "1.0.0",
  "lockfileVersion": 1,
  "requires": true,
  "dependencies": {
    "angular": {
      "version": "1.5.7",
      "resolved": "https://registry.npmjs.org/angular/-/angular-1.5.7.tgz",
      "integrity": "sha1-Ts79DHE0tydXujtBu6Oh5NXv9aw="
    }
  }
}

Updated dependency

angular
What you expected to see, versus what you actually saw
I expect the version to be the same, but "1.8.0" was suggested.
Images of the diff or a link to the PR, issue or logs

You can configure dependabot to only update the lockfile, and never package.json by using the versioning-strategy: lockfile-only strategy as described here: https://docs.github.com/en/github/administering-a-repository/configuration-options-for-dependency-updates#versioning-strategy and configure angular to only update patch versions in your package.json like "~1.5".

@jurre Thanks for your reply. Would you please confirm that versioning strategy feature does not exist in dependabot-omnibus ? I did a search in this repo, but I didn't found anything relevant.

Thanks !

sorry late to the party, maybe someone can help me out to fully understand this:
When I have this in my package.json
"prop-types": "^15.7.2",
"react": "16.13.1",

I'd expect:
A) dependabot to open a pull request for all minor/patch versions of prop-types < 16
B) dependabot to not touch react dependency
Unfortunately dependabot does update the manifest file for react :(
When I enable lockfile-only, will it also update the manifest file, when updating prop-types to say 15.9.1?
Will it even attempt to do this?
Thanks in advance

@ferdicus I'm facing the same issue. Did changing to lockfile-only worked for you?


phpF abandoned

phpF has been abandoned:
https://github.com/subins2000/phpF/blob/master/README.md
Some new features like nullable types, "yield from", trailing commas in lists (and probably more after  PHP 7.2/3) are not supported and (i think?) will not be implemented in the near future.
The fork of MKCG looks like a good replacement (see subins2000 README):
https://github.com/MKCG/phpfmt_stable

Is there another text editor that offers these functions without having to use this plugin?

You can replace the "fmt.phar" file in your local codeformatter installation with the fmt.phar from here:
https://github.com/phpfmt-next/fmt/tree/master/bin

Please look at https://github.com/akalongman/sublimetext-codeformatter/issues/355#issuecomment-381722577
phpF will be removed soon


TypeScript dependency is bloating the library

Is the TypeScript dependency actually necessary in this library? That lone modules makes up 2/3 of the node_modules footprint (and doubles the size of the library overall).
With TypeScript

Without TypeScript

The unit tests appear to yield the exact same outcome with or without the library, and I don't think any of the other modules are depending on it (I'm not a TS user).
I may be missing something, but this seems like a needless waste of space.

Bingo; I was wondering why too -- unless for TypeScript typings support later on in which case it should be at most a devDependency.
Definitely a waste of space in the current artefact.

Huh, yeah, looks like when we switched to using ts-node, it no longer needs to have typescript as a dep.  (Maybe it used to once upon a time?)
I think you're right, it can be removed.

@isaacs I was assuming the same - this seems like it was probably a remnant from a past requirement. Given this seems to just be a one-line deletion, maybe it's less hassle to change it yourself? I'd be happy to PR it though, if you prefer.

From what I can tell typescript is a peer dependency of ts-node so it is probably needed.

Oh, I spoke too soon.  Typescript is a peer dep of ts-node.
Removing it breaks the test/run/ts.js test.  My guess is that require.resolve('typescript') is finding something elsewhere on your machine if it doesn't break any tests to remove it?
In any event, it is indeed huge, and we could probably make it an optional-but-not-installed-by-default dep by just catching the error.  If we're going to go that route, though, we should probably not install ts-node either, and just raise a useful error if no ts-node is found.

I'm not sure why the tests would pass when removing it, since I didn't think I have TS anywhere else.... but maybe I overlooked something.  Regardless, making it optional in v15 seems like a solid plan to me.

Letting the user install typescript also allows them to choose which version gets installed so this seems like a plus.  I don't use typescript but I've heard they play fast and loose with semver (semver-minor changes are sometimes viewed by many as breaking).

FWIW, I created tappity-tap, a self-contained bundle consisting of tap together with TypeScript .d.ts typings, as, being an avid TypeScript developer, I prefer types to be in lock-step with the correct version of the runtime package.
What astounds me is that the unpacked size is 86.1 MB (see metrics here on npm). The implication of this is that any developer using tap sustains an installation payload of over 85 megabytes. This is totally ludicrous.
Appears that from humble minimalist beginnings, tap has become a totally bloated library.
Justin Johansson, @indiescripter on Twitter

@indiescripter TypeScript causes most of the install size bloat.  Just the typescript package has an unpacked size of 54MB.  I agree it will be nice to reduce the size of tap, that is the reason we're discussing not bundling TypeScript.  Since you are targeting TypeScript support this is not the place to complain about install size.

Hi @coreyfarrell, sorry if sounded overly complaining; really meant it as constructive criticism though realise it didn't come over that way & do acknowledge that tap is OSS that people can use free-of-charge & for which contributors have donated their time for free.
If there is a way of unbundling TypeScript, that would be +ve move; understand that some things may appear easy by not always straightforward.

I loathe TS. And I default all of my tap installs to disabling the TS loader. But I really couldn't care much less about the size of the testing library. It's not harming any production deploy, and all package managers implement some sort of local cache nowadays. If it's really a big issue, managers like pnpm will abstract this size annoyance away.
That being said, I am totally on board for making it an optional dependency with a runtime error. We do the same in pino for prettification support. It works well for the majority of users. We get occasional complaints, but they are few and far between.

Besides bloating, on modern installs with Typescript 4 as the main interpreter tap pulls Typescript 3.9 for no apparent reason.

The reason is very plainly apparent -- https://github.com/tapjs/node-tap/blob/4d926fb5e07eceac0f83830e6e0abe2eecdbe39b/package.json#L53

@jsumners A package.json entry is not a reason, it's a cause.
node-tap itself can work fine with TS4, there were no breaking changes. I just tested it against<EMAIL_ADDRESS>

You asked for the reason. I gave it to you. If you don't want to understand it, that's on you.
https://docs.npmjs.com/misc/semver#caret-ranges-123-025-004

Tap v15 will reference typescript and ts-node as optional peerDependencies, so that they'll be exposed to tap by pnpm, but not installed by default (except when using npm v7, and a different version of typescript is already present in the tree).
Also, typescript support now requires passing the --ts flag on the command line, or setting { "tap": { "ts": true } } in the package.json file.  .tsx support requires enabling both typescript and jsx.


microprofile-health-128 org.eclipse.microprofile.health.HealthCheckResponse leaks

fixes #128 with multiple providers support addition
Tests also added to api to make sure multiple provider resolution works as expected

Hi @rmannibucau, could you check if this PR meet your concern in #128 ?

Hi @antoinesd , from my understanding it just moves the issue somewhere else (basically while there is a setter, with or without a filter the issue remains). A clean solution is likely to make the factory (builder) an injectable bean which would therefore fully inherit CDI context definition in next major. Until then keeping current solution sounds like acceptable.

@rmannibucau the proposed solution allows the usage of multiple implementations even impl provided in app as your mentioned  in ticket #128, so I really don't get your issue. As we won't drop the service loader usage I don't see how your suggestion could be viable.

@antoinesd the PR still leaks the impl (the provider and the predicate) and is still not aligned on the CDI container lifecycle (which is this ticket). Why serviceloader wouldn't be dropped? There is no technical reason to duplicate the impl loading.

Sorry Romain, there is no mention of CDI in the initial ticket. The use case you mentioned in it was to support multiple impl (even one included in the app) which is now the case.
Regarding ServiceLoader usage it's a good practice used in a lot of specs (including CDI), so we won't drop it without a very good reason. As some people are using Health without CDI it would break a lot of usage if we should based the impl loading on CDI injection.

@antoinesd hmm I didn't write "CDI" but that the builder should be "injected", sorry if it was not explicit enough.
Regarding ServiceLoader, the only reason most spec use it is to be independent from others, MP is not and clearly depends on CDI so only CDI must use a ServiceLoader to lookup the provider, then everything should just be beans to avoid context issues.
For cases where you don't have CDI, it is exactly the same - we did it at Geronimo were we extracted part of the impl in a "common" module to let it be used in OSGi or standalone. The main difference is the caller must define the "context"/scope and not rely on an implicit global variable which leaks if you re-execute some processes.
Another broken use case is when you start weld twice in the same JVM and want one impl per container (cause a lib depends on it).
So whatever wayt I'm thinking about it, I only see the builder as being injectable sane. For standalone case, the SPI could still be used if set by vendors or a plain new could be done of the provider but the spec does not handle or need it.

Obviously, I totally missed your point and I'm not quite sure to understand what you suggest. As most MP are using ServiceLoader, I guess that the problem you raise is at the platform level and should be discussed in the MP mailing list.
In the meantime, I close this PR and move your ticket for 4.0.

@antoinesd oki, agree it can be tackled at platform level too. What I hit was the leak between containers instances (recreating classloaders i switched of provider unintentionally in a complex setup) but the underlying issue is really the fact to not align properly on CDI using ServiceLoader (or alike) API in specs.


Fix MarketItemStore RequestMany batching

This should fix the function only returning as many worlds as there are items specified

Closes #1121


fix how task retries value is acessed in rate_limit decorator

See https://docs.celeryq.dev/en/latest/userguide/tasks.html#task-request


Coban not save my data

this is coban device log

[2024-03-30T17:25:04+00:00] ##,imei:864893035975666,A
[2024-03-30T17:25:15+00:00] imei:864893035975666,tracker,240330205512,,F,172512.00,A,2937.16612,N,05231.38756,E,3.664,105.51,,1,0,0.00%,,
[2024-03-30T17:25:19+00:00] imei:864893035975666,tracker,240330205514,,F,172514.00,A,2937.16434,N,05231.38844,E,4.429,158.61,,1,0,0.00%,,
[2024-03-30T17:25:20+00:00] imei:864893035975666,tracker,240330205516,,F,172516.00,A,2937.16167,N,05231.38764,E,5.604,195.15,,1,0,0.00%,,
[2024-03-30T17:26:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:27:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:28:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:29:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:30:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:30:29+00:00] imei:864893035975666,tracker,240330210027,,F,173027.00,A,2936.49700,N,05231.37515,E,2.796,164.96,,1,0,0.00%,,
[2024-03-30T17:31:05+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:32:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:33:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:34:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:34:29+00:00] imei:864893035975666,tracker,240330210427,,F,173427.00,A,2936.29039,N,05230.86648,E,14.946,0.87,,1,0,0.00%,,
[2024-03-30T17:35:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:36:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:36:58+00:00] imei:864893035975666,tracker,240330210655,,F,173655.00,A,2935.76635,N,05230.55519,E,11.485,10.55,,1,0,0.00%,,
[2024-03-30T17:37:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:38:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:39:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:40:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:41:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:41:08+00:00] imei:864893035975666,tracker,240330211106,,F,174106.00,A,2935.39023,N,05231.33674,E,3.262,143.24,,1,0,0.00%,,
[2024-03-30T17:42:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:42:31+00:00] imei:864893035975666,acc off,240330211229,,F,174229.00,A,2935.38717,N,05231.34096,E,0.027,0,,0,0,0.00%,,
[2024-03-30T17:43:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:44:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:45:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:46:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:47:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:48:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:48:14+00:00] imei:864893035975666,acc on,240330211812,,F,174812.00,A,2935.38715,N,05231.34092,E,0.121,0,,1,0,0.00%,,
[2024-03-30T17:48:29+00:00] imei:864893035975666,acc off,240330211827,,F,174827.00,A,2935.38730,N,05231.34103,E,0.059,0,,0,0,0.00%,,
[2024-03-30T17:49:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:50:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:51:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:51:59+00:00] imei:864893035975666,acc on,240330212158,,F,175158.00,A,2935.38760,N,05231.34051,E,0.112,0,,1,0,0.00%,,
[2024-03-30T17:52:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:52:05+00:00] imei:864893035975666,acc off,240330212202,,F,175202.00,A,2935.38750,N,05231.34041,E,0.063,0,,0,0,0.00%,,
[2024-03-30T17:52:21+00:00] imei:864893035975666,acc off,240330212219,,F,175219.00,A,2935.38711,N,05231.33995,E,0.060,0,,0,0,0.00%,,
[2024-03-30T17:53:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:54:10+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:55:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:56:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:57:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T17:58:05+00:00]<PHONE_NUMBER>75666
[2024-03-30T18:21:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T18:22:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:41:05+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:42:04+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:46:45+00:00] ##,imei:864893035975666,A
[2024-03-30T20:47:45+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:48:45+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:49:45+00:00]<PHONE_NUMBER>75666
[2024-03-30T20:50:45+00:00]<PHONE_NUMBER>75666

this device save with serial number =<PHONE_NUMBER>975666 on my database
but dont have save any data!

oh thank you

Thanks for your support

Can we close the issue? 🙂


Ensure generated_html? handles single line files

Description
Whilst testing for the next release of Linguist, I found one of the GitHub.com tests failed as follows:
bundler: failed to load command: bin/github-linguist (bin/github-linguist)
NoMethodError: undefined method `start_with?' for nil:NilClass
  /Users/lildude/github/linguist/lib/linguist/generated.rb:612:in `generated_html?'
  /Users/lildude/github/linguist/lib/linguist/generated.rb:89:in `generated?'
  /Users/lildude/github/linguist/lib/linguist/generated.rb:12:in `generated?'
  /Users/lildude/github/linguist/lib/linguist/blob_helper.rb:357:in `generated?'
  /Users/lildude/github/linguist/lib/linguist/lazy_blob.rb:52:in `generated?'
  /Users/lildude/github/linguist/lib/linguist/blob_helper.rb:380:in `include_in_language_stats?'
  /Users/lildude/github/linguist/lib/linguist/repository.rb:164:in `block in compute_stats'
  /Users/lildude/github/linguist/lib/linguist/repository.rb:149:in `each_delta'
  /Users/lildude/github/linguist/lib/linguist/repository.rb:149:in `compute_stats'
  /Users/lildude/github/linguist/lib/linguist/repository.rb:116:in `cache'
  /Users/lildude/github/linguist/lib/linguist/repository.rb:68:in `languages'
  bin/github-linguist:30:in `<top (required)>'

The test is failing on a very simple .html file that contains:
I'm remote.html, a different file loaded with ajax.

The problem is the generated_html? method makes a few assumptions about the number of lines in the file and doesn't consider there may only be a single line.
This PR corrects that behaviour .
Checklist doesn't apply as I'm fixing a bug and we don't have a section for that 😄.

Yeah, I couldn't find an easy way to test this either.

Oops, sorry about that. Chalk that one up to my lack of Ruby experience. 😅
@lildude Is there anything I can do to help with whatever's going on at lildude/fix-gen_html-error...? Or is it unrelated to this PR's changes?
Also, major WTF @ the recent commit logs on your branch:
Author: The rugged tests are fragile<EMAIL_ADDRESS>
/cc @TheRuggedFragileTestFormerlyKnownAsLilDude

@lildude Is there anything I can do to help with whatever's going on at lildude/fix-gen_html-error...? Or is it unrelated to my earlier PR's changes?

Just opened a PR for that - https://github.com/github/linguist/pull/4628

Also, major WTF @ the recent commit logs on your branch:
Author: The rugged tests are fragile<EMAIL_ADDRESS>

Whoops. Not sure how I did that 😊 .  Will rewrite my author info.


Fuel and Ground Handling Costs not being checked from settings

Describe the bug
If Airport fuel and ground handling costs are not defined pirep finances does not use default prices defined at settings
To Reproduce
Steps to reproduce the behavior:

Remove fuel cost from your departure airport
Remove ground handling cost from your arrival airport
File a manual pirep or just re-calculate finances of a previous pirep using edited airports
Check finances / no fuel and gh cost will be there

Expected behavior
Finances should use the default settings if airport costs are not defined

Closed with PR #1050


AI-LEARN-03c Conversational Language (Learn) - clock-client.py returns "key must be a string."

Hi, I filled the code according to the lab, and have setup model in studio, but the code returns for me "key must be a string." for every input. Can you help me out? Thanks

without any code to look at, it's hard for me to help. Are you using C# or Python? I'm guessing the key would need to be in quotation marks and it isn't?

.env
AI_SERVICE_ENDPOINT=https://...........test.cognitiveservices.azure.com/ AI_SERVICE_KEY=771fe6b5.............................fe83e
clock-client.py
`
from dotenv import load_dotenv
import os
import json
from datetime import datetime, timedelta, date, timezone
from dateutil.parser import parse as is_date
Import namespaces
from azure.core.credentials import AzureKeyCredential
from azure.ai.language.conversations import ConversationAnalysisClient
def main():
try:
    # Get Configuration Settings
    load_dotenv()
    ls_prediction_endpoint = os.getenv('LS_CONVERSATIONS_ENDPOINT')
    ls_prediction_key = os.getenv('LS_CONVERSATIONS_KEY')

    # Get user input (until they enter "quit")
    userText = ''
    while userText.lower() != 'quit':
        userText = input('\nEnter some text ("quit" to stop)\n')
        if userText.lower() != 'quit':

            # Create a client for the Language service model
            client = ConversationAnalysisClient(
                ls_prediction_endpoint, AzureKeyCredential(ls_prediction_key))

            # Call the Language service model to get intent and entities
            cls_project = 'Clock'
            deployment_slot = 'production'

            with client:
                query = userText
                result = client.analyze_conversation(
                    task={
                        "kind": "Conversation",
                        "analysisInput": {
                            "conversationItem": {
                                "participantId": "1",
                                "id": "1",
                                "modality": "text",
                                "language": "en",
                                "text": query
                            },
                            "isLoggingEnabled": False
                        },
                        "parameters": {
                            "projectName": cls_project,
                            "deploymentName": deployment_slot,
                            "verbose": True
                        }
                    }
                )

            top_intent = result["result"]["prediction"]["topIntent"]
            entities = result["result"]["prediction"]["entities"]

            print("view top intent:")
            print("\ttop intent: {}".format(result["result"]["prediction"]["topIntent"]))
            print("\tcategory: {}".format(result["result"]["prediction"]["intents"][0]["category"]))
            print("\tconfidence score: {}\n".format(result["result"]["prediction"]["intents"][0]["confidenceScore"]))

            print("view entities:")
            for entity in entities:
                print("\tcategory: {}".format(entity["category"]))
                print("\ttext: {}".format(entity["text"]))
                print("\tconfidence score: {}".format(entity["confidenceScore"]))

            print("query: {}".format(result["result"]["query"]))

            # Apply the appropriate action
            if top_intent == 'GetTime':
                location = 'local'
                # Check for entities
                if len(entities) > 0:
                    # Check for a location entity
                    for entity in entities:
                        if 'Location' == entity["category"]:
                            # ML entities are strings, get the first one
                            location = entity["text"]
                # Get the time for the specified location
                print(GetTime(location))

            elif top_intent == 'GetDay':
                date_string = date.today().strftime("%m/%d/%Y")
                # Check for entities
                if len(entities) > 0:
                    # Check for a Date entity
                    for entity in entities:
                        if 'Date' == entity["category"]:
                            # Regex entities are strings, get the first one
                            date_string = entity["text"]
                # Get the day for the specified date
                print(GetDay(date_string))

            elif top_intent == 'GetDate':
                day = 'today'
                # Check for entities
                if len(entities) > 0:
                    # Check for a Weekday entity
                    for entity in entities:
                        if 'Weekday' == entity["category"]:
                        # List entities are lists
                            day = entity["text"]
                # Get the date for the specified day
                print(GetDate(day))

            else:
                # Some other intent (for example, "None") was predicted
                print('Try asking me for the time, the day, or the date.')

except Exception as ex:
    print(ex)

def GetTime(location):
time_string = ''
# Note: To keep things simple, we'll ignore daylight savings time and support only a few cities.
# In a real app, you'd likely use a web service API (or write  more complex code!)
# Hopefully this simplified example is enough to get the the idea that you
# use LU to determine the intent and entities, then implement the appropriate logic

if location.lower() == 'local':
    now = datetime.now()
    time_string = '{}:{:02d}'.format(now.hour,now.minute)
elif location.lower() == 'london':
    utc = datetime.now(timezone.utc)
    time_string = '{}:{:02d}'.format(utc.hour,utc.minute)
elif location.lower() == 'sydney':
    time = datetime.now(timezone.utc) + timedelta(hours=11)
    time_string = '{}:{:02d}'.format(time.hour,time.minute)
elif location.lower() == 'new york':
    time = datetime.now(timezone.utc) + timedelta(hours=-5)
    time_string = '{}:{:02d}'.format(time.hour,time.minute)
elif location.lower() == 'nairobi':
    time = datetime.now(timezone.utc) + timedelta(hours=3)
    time_string = '{}:{:02d}'.format(time.hour,time.minute)
elif location.lower() == 'tokyo':
    time = datetime.now(timezone.utc) + timedelta(hours=9)
    time_string = '{}:{:02d}'.format(time.hour,time.minute)
elif location.lower() == 'delhi':
    time = datetime.now(timezone.utc) + timedelta(hours=5.5)
    time_string = '{}:{:02d}'.format(time.hour,time.minute)
else:
    time_string = "I don't know what time it is in {}".format(location)

return time_string

def GetDate(day):
date_string = 'I can only determine dates for today or named days of the week.'
weekdays = {
    "monday":0,
    "tuesday":1,
    "wednesday":2,
    "thursday":3,
    "friday":4,
    "saturday":5,
    "sunday":6
}

today = date.today()

# To keep things simple, assume the named day is in the current week (Sunday to Saturday)
day = day.lower()
if day == 'today':
    date_string = today.strftime("%m/%d/%Y")
elif day in weekdays:
    todayNum = today.weekday()
    weekDayNum = weekdays[day]
    offset = weekDayNum - todayNum
    date_string = (today + timedelta(days=offset)).strftime("%m/%d/%Y")

return date_string

def GetDay(date_string):
# Note: To keep things simple, dates must be entered in US format (MM/DD/YYYY)
try:
date_object = datetime.strptime(date_string, "%m/%d/%Y")
day_string = date_object.strftime("%A")
except:
day_string = 'Enter a date in MM/DD/YYYY format.'
return day_string
if name == "main":
main()
`
In env I dont put any quotes around endpoint and key, this works elsewhere fine. And I also just tried to put key in '' and in "" and still same output message.  Also, I didnt change much in code. Just put code pieces from lab under comments where supposed to. Also I made sure key is correct to the endpoint. Thanks

You're correct the .env file doesn't need quotations. Are you sure that you're using the right .env file? The one you posted uses "AI_SERVICE_KEY" which is used for other labs in this repo but the code for lab 3/the code you posted is looking for LS_CONVERSATIONS_KEY.
Check that the .env you have your values in is next to clock-client.py. You can also test by changing lines 16 and 17 to not use the getenv method and paste your resource values directly there.

Closing due lack of response

You're correct the .env file doesn't need quotations. Are you sure that you're using the right .env file? The one you posted uses "AI_SERVICE_KEY" which is used for other labs in this repo but the code for lab 3/the code you posted is looking for LS_CONVERSATIONS_KEY.
Check that the .env you have your values in is next to clock-client.py. You can also test by changing lines 16 and 17 to not use the getenv method and paste your resource values directly there.

Hi, sorry for late response. You are very correct, I made a mistake copying the .env from other lab :( Thank you for your help!!!!!


Year subscription with monthly billing

Hi all,
I'm trying to setup two types of subscriptions: 3 months and a 12 months subscription.
Both are monthly charged from the bank account. And auto-renewal for the year subscription one month before the end of the contract.
Is this possible with the current version?
Should I e.g. set the end_at column in the Subscription model at 3 months and 12 months from now at the start?

Hi,
I'm trying to understand your use case... Do you mean the subscription should stop at a preset date?
So the 3 months subscription would result in 3 monthly invoices and then be completed?
If that's the case, that's not supported. Not yet anyway. I have been thinking about adding a times attribute to the Plan, but this will not be anytime soon.

Alternatively, you could run a check that cancels "completed" subscriptions prior to processing them with cashier:run.

Yes correct, the subscription should stop at a certain date.
The times option should be a good fit, I already tried to add it to the cashier_plan.php like described in the mollie manual (https://docs.mollie.com/payments/recurring#payments-recurring-charging-periodically). I soon discovered that it is not supported yet.
Will this be in the next couple of months?
Can you maybe describe the alternative option in more detail?

Cashier Mollie is not using Mollie's subscription API, because it's too limited in its functionality for this.
I can't tell if/when times will be available on the Plan at this point. I'm not sure about the impact yet. Also, preparations for Spark (#119) is consuming a lot of my time.
The alternative would involve checking how many processed OrderItems exist for the Subscription before running cashier:run. Having that $count you could then do something like:
if($count >= $times) {
    $subscription->cancel();
}


Error in v1.0.13: No DOM element with id 'plotly-' exists on the page

What happened: Plots do not get rendered and mentioned error is thrown.
What i would expect: Plots rendering
What i did:

Install the plugin as per the instructions from README.md
create a chart as per plotly docs

Reproduction minimal example
Code
<template>
    <div class="">
        <client-only>
            <nuxt-plotly :data="data"
                :layout="{ title: style.title || data[0].meta.title, plot_bgcolor: 'rgba(0,0,0,0)', paper_bgcolor: 'rgba(0,0,0,0)', margin: { b: 150 }, barmode: 'stack', showlegend: false }"
                :config="{ staticPlot: true }"></nuxt-plotly>
        </client-only>
    </div>
</template>

<script setup lang="ts">
const data = [
    { "y": Array.from({ length: 40 }, () => Math.floor(Math.random() * 20)), "x": Array.from({ length: 20 }, (_, i) => "element_" + i), "type": "bar", "marker": { "color": "#fbbf24" }, "meta": { "title": "Example Graph" } }]
const style = {}

const test = ref(0)

</script>

In version v1.0.12

In verrsion v1.0.13
Following error in the Console and a blnak page:
Uncaught (in promise) Error: No DOM element with id 'plotly-0bbf9e71-3c30-4cf3-9581-24b3a875dca2' exists on the page.

(this uuid is different every time the page is reloaded)
Curiously doing any kind of edit to the file while running the hot reloading dev server results n the graph being rendered on reload.
Fix
Downgrading to 1.0.12 fixes the issue, this is not a viable long term solution

Same error in 1.0.14 version :(

Same issue.

Same issue :(


inference.exe啟動中閃退

Traceback (most recent call last):
File "inference.py", line 99, in
AttributeError: 'list' object has no attribute 'keys'
[56296] Failed to execute script 'inference' due to unhandled exception!

好的，我再試試看


"Single User" access mode has been renamed to "Assigned"

Apparently, the drop-down list value changed recently so the table shown on this page should be updated to match the product.

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

ID: 066b3491-41c2-401c-2d91-402a9e1fc6f0
Version Independent ID: 066b3491-41c2-401c-2d91-402a9e1fc6f0
Content: Clusters UI changes and cluster access modes - Azure Databricks
Content Source: databricks/clusters/cluster-ui-preview.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

Sorry, I realize I'm accessing a workspace that has some preview features turned on. Please disregard!


Add standalone postgres compose

This compose will be used for new persistent setup instructions. It is a standalone file and doesn't need any additional files to be mounted into the containers. It is a complete single file (not multiple files merged) and doesn't contain anything unnecessary. Future docs will build upon it for adding additional collectors, etc.
Tested with release on my fork:
https://github.com/jeffmendoza/guac/releases/tag/v0.6.0-betajeff
https://github.com/jeffmendoza/guac/releases/download/v0.6.0-betajeff/guac-postgres-compose.yaml
Went through some demos and working well.
Description of the PR

PR Checklist

[ ] All commits have a Developer Certificate of Origin (DCO) -- they are generated using -s flag to git commit.
[ ] All new changes are covered by tests
[ ] If GraphQL schema is changed, make generate has been run
[ ] If OpenAPI spec is changed, make generate has been run
[ ] If collectsub protobuf has been changed, make proto has been run
[ ] All CI checks are passing (tests and formatting)
[ ] All dependent PRs have already been merged

Note: not messing with the existing container_files/ent.yaml file. The existing files used by the Makefile in integrations tests can stay. Those are multi-part to avoid duplication in test and dev, but we need a simplified one for users to use. We may end up consolidating somewhat in the future.


Import DynArray Structs.

Looks like Struct Definitions are being imported correctly from FREE contracts now. So removed a redundant function that also checked that they were imported.
Unfortunately, it does not fix dynArray structs being imported since the node representing them is actually a function definition stub that just writes the struct def to the file.
This PR fixes part of tests/behaviour/solidity/test/libsolidity/semanticTests/freeFunctions/storage_calldata_refs.sol.
The above test will be fully fixed once this PR: https://github.com/NethermindEth/warp/pull/441 is merged.

Not fixed, example contracts failing


Build Error Keeps Coming

vyraun@vyraun:~/tensorflow$ bazel build //tensorflow/examples/android:tensorflow_demo
WARNING: Bazel Android NDK crosstools are based on Android NDK revision 11. The revision of the Android NDK given in android_ndk_repository rule 'androidndk' is '12.1.2977051'.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/debug:debug_graph_utils.cc' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/debug:debug_graph_utils.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:avgpooling_op.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:bounds_check.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_activations.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_attention.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_backward_cuboid_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_backward_spatial_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_cuboid_convolution.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_patch_3d.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_pooling.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_softmax.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:eigen_spatial_convolutions.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:maxpooling_op.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:ops_util.cc' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:ops_util.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:pooling_ops_common.cc' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/kernels:pooling_ops_common.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_entry.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_scorer.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_beam_search.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_decoder.h' directly. You should either move the file to this package or depend on an appropriate rule there.
WARNING: /home/vyraun/tensorflow/tensorflow/core/BUILD:646:12: in srcs attribute of cc_library rule //tensorflow/core:android_tensorflow_lib_lite: please do not import '//tensorflow/core/util/ctc:ctc_loss_util.h' directly. You should either move the file to this package or depend on an appropriate rule there.
INFO: Found 1 target...
ERROR: missing input file '@androidsdk//:build-tools/23.0.1/aapt'.
ERROR: /home/vyraun/.cache/bazel/_bazel_vyraun/b219210c52a294104956e4ed70d84022/external/androidsdk/BUILD:5:1: @androidsdk//:aapt_binary: missing input file '@androidsdk//:build-tools/23.0.1/aapt'.
Target //tensorflow/examples/android:tensorflow_demo failed to build
Use --verbose_failures to see the command lines of failed build steps.
ERROR: /home/vyraun/.cache/bazel/_bazel_vyraun/b219210c52a294104956e4ed70d84022/external/androidsdk/BUILD:5:1 1 input file(s) do not exist.
INFO: Elapsed time: 2.821s, Critical Path: 0.03s
vyraun@vyraun:~/tensorflow$ bazel clean
INFO: Starting clean (this may take a while). Consider using --expunge_async if the clean takes more than several minutes.
vyraun@vyraun:~/tensorflow$ bazel build //tensorflow/examples/android:tensorflow_demo
ERROR: no such package '@androidndk//': Could not read RELEASE.TXT in Android NDK: /home/vyraun/.cache/bazel/_bazel_vyraun/b219210c52a294104956e4ed70d84022/external/androidndk/ndk/RELEASE.TXT (No such file or directory).
ERROR: no such package '@androidndk//': Could not read RELEASE.TXT in Android NDK: /home/vyraun/.cache/bazel/_bazel_vyraun/b219210c52a294104956e4ed70d84022/external/androidndk/ndk/RELEASE.TXT (No such file or directory).
INFO: Elapsed time: 2.406s
I tried changing the api levels in workspace. Didn't work. I do not have any folder for 23.0.1 in my build-tools directory. I guess there is some configuration issue. How could I resolve them? Thanks for your help.

Pete, help with Android build?

Hi, thanks for the reply. I had to change the workspace SDK to my own
version 24.0.2 and then it worked.
On Sep 1, 2016 10:40 AM, "Martin Wicke"<EMAIL_ADDRESS>wrote:

Pete, help with Android build?
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/tensorflow/issues/4099#issuecomment-243977366,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQa2LKQYY7gloQFjYMOitjrYjsHaNtvKks5qll4_gaJpZM4Jv_mM
.


Fix newsletter subscribe url for shops with urls like /de

1. Why is this change necessary?
Newsletter subscribe don't works for domains like "https://www.my-shop.at/de".
Same bug as in pull request https://github.com/shopware/platform/pull/1044
2. What does this change do, exactly?
Fix missing /de for subscribing newsletter create opt in link.
3. Describe each step to reproduce the issue or behaviour.
Our domain setup of our sales channel:
https://www.my-shop.at/de
https://www.my-shop.at/en

-> No domain with https://www.my-shop.at is set!
Now when we try the newsletter subscribe on one of the domains - it don't works because it validates the storefrontUrl "https://www.my-shop.at" to that non existing domain.
We changed it, that it sets the storefrontUrl as "https://www.my-shop.at/de" and not "https://www.my-shop.at".
4. Please link to the relevant issues (if any).
https://github.com/shopware/platform/pull/1044
5. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hey @acris-cp, thanks for your contribution 💙 You forgot our checklist and the changelog file again. Please take care of it

This PR was closed due to inactivity. If this change is still important to you, feel free to create a new pull request.
For more information about our contribution guidelines, see https://docs.shopware.com/en/shopware-platform-dev-en/contribution/contribution-guideline


Fixing overlap of labels with each other in select Widget

Fixes #1960

@sneridagh can we get this into the next release?

@iFlameing Having a picture of what was the problem in the first place would be great to have to know what are we fixing, and the outcome of the fix as well.

@sneridagh here you go:

I am fine with postponing this to figure this out properly.

@tisto let me take a look tomorrow. So the problem is that if the option is too long, then it jumps, right? We can do the release tomorrow morning.


@typescript-eslint/non-nullable-type-assertion-style conflicts with other rule

The code below in my codebase triggers a warning suggesting I use ! instead of as Entity. The warning comes from this rule: @typescript-eslint/non-nullable-type-assertion-style.
if (childCache.has(path)) {
  return childCache.get(path) as Entity;
}

However, when I make the change, I get an error because of rule @typescript-eslint/no-non-null-assertion. I personally prefer the casting to be explicit -- ! can easily be missed -- so I would recommend disabling @typescript-eslint/non-nullable-type-assertion-style.

Thank you - I agree, have also had a few issues with said rule and will look to disable it soon :)

Awesome, thanks a lot! And for the other changes as well 😉


Simplification step doesn't keep applying function/simplification rules until the term stabilizes

When proving the following spec on KWASM semantics:
requires "kwasm-lemmas.k"

module SIMPLE-ARITHMETIC-SPEC
    imports WASM-TEXT
    imports KWASM-LEMMAS

    rule <k> (ITYPE:IValType . const X:Int) => . ... </k>
         <valstack> S:ValStack => < ITYPE > X : S </valstack>
      requires #inUnsignedRange(ITYPE, X)

endmodule

I found that in depth 2, after deriveSeq step, the top of k cell is chop(...). In the following simplify step, the chop function gets evaluated to modInt and simplification step terminates. However, there is still a simplification rule about modInt can be applied. The simplification rule is applied in the next simplification step. The steps between two simplifications are TrivivallyValid, CheckProven, and CheckRemainder, which doesn't modify the term at all.

This issue should be fixed by https://github.com/kframework/kore/pull/1086


Minimum HTML sample

I've put together a small demo file, using vanilla JS, and the necessary imports
https://gist.github.com/EdwinChua/0a5d66dc561fe7d3866021b18a320585

Awesome. Thanks. I'll add it to the ReadME.md


fixing issue #1724

Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : https://github.com/karatelabs/karate/issues/1724
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

Do NOT SEND ME ANY NOTIFICATIONS, I have been unsubscribing it and still able to receive them. Unsubscribe me please. Thanks

@joelpramos actually can you just remove the <exclude> that exists for the httpclient dependency, see screenshot. that's probably the right way to do this. looks like I was too eager to reduce the fatjar size

@joelpramos actually can you just remove the <exclude> that exists for the httpclient dependency, see screenshot. that's probably the right way to do this. looks like I was too eager to reduce the fatjar size

When I tried that I got some issues with the ClassPath loading in my app in Spring Boot. I assumed it's because of the JUnit that comes with commons-codec thus this approach.

Do NOT SEND ME ANY NOTIFICATIONS, I have been unsubscribing it and still able to receive them. Unsubscribe me please. Thanks

Go google how to remove subscriptions from all actions in a Github repository or go complain to Github on Twitter.

closing because I think this is fixed here: https://github.com/karatelabs/karate/issues/1724#issuecomment-974836738


"Something went wrong. Please try again"

on IPS cloud installation. see my forums @ forums.juicylandgaming.com to see the error.
it sometimes works, it sometimes doesn't. let me know if there are any logs that you need.

should have a log printed for it in System Error or Error Logs... and what action is being taken to get this error to occur? logging in? registering? acp login?

I was able to register with steam just fine without error, logged out and back in without issue as well.

also whats the app you use to prompt steam only login?

I'm having the same issue.
Please note that this is using IPB 4.2.0.

At some point this week I'm going to be performing an upgrade to 4.2.0 on a site I help maintain, I'll look into at that time.

I just updated my IPB to 4.2.1, and it seems to be working again.
Can you confirm, @Fiiily?

@Hyrius still seems to be an issue now and then. Fresh install with 4.2.1 and this plugin errors quite a lot (something went wrong) when attempting to register.
No errors seem to be generated in logs.

@MStruntze Agreed. I get errors sometimes, too.

@Lavoaster Sorry to disturb, but... Any news on this one? :)

@madman32395 Hey, how's it going? I'm sorry for getting back to you late.
The issue is resolved with recent IPS updates and I am using template modifications to lock you into using Steam authentication.

Same issue.

I've got this error every time

I can confirm this, no sign-in possible.
The plugin pulls the profile picture from steam but signing in is not possible. There are no error logs written. I cound this as critical
IPB 4.2.5

Also having this issue.

I don't think IPS Cloud's www server is misconfig'd.

Having the same issue here. Not super familiar with PHP however through a little editing I've identified it's throwing the exception found here: https://github.com/Lavoaster/IP.Board-Steam-Authentication-Method/blob/master/upload/system/Login/Steam.php#L77
I hope that helps. I'm gonna attempt to fumble my way further through this and hopefully come back with a little more info and maybe even a pull request if I get lucky.

Like others said - it's randomly working (or not). Sometimes I need to try logging 5 times before I log in.
@jrod9k1 the problem is with private function validate(), I guess.

I did some test on the 2FA with the Steam App, perhaps other users could try this too.
I'm using the App with the Code generator and have those said problems randomly as the login is on halt until I enter my 2FA code. A buddy of mine does not use the 2FA and could log in instantly without any problems.

I seem to be having the same issue on 4.2.6. Logins work at random, sometimes they work the first time, other times it takes 3-4 tries.

I have the exact problem, sometimes it works, sometimes it doesn't. This is a big problem trying to grow a community. Please fix this

@madman32395 Any news?

I can confirm that this was solved for me by turning off friendly URLS

for giggles, try again after updating against #98  ; another community that I can play with, this seemed to fix "Something went wrong. Please try again".

awesome. let us know what happens :)

So far so good, but will have to wait and see because it only occurred occasionally for myself

Was working fine on IPB 4.2.8. Now getting "Something went wrong. Please try again." Tried upgrading to Sign in through Steam - 2.0.4 still same issue. Even tried uninstalling to no avail. This is a cloud IPS community. Nothing I have tried has fixed this.

@JMan49 for a CiC setup have you had ips upload the files in upload folder?

@madman32395 Yeah as far as I know they have.

As I'll be working on this soon for the 4.3 upgrade, did anyone actually come up with a reproducible way to produce this?

Please download this and follow the instructions in the readme for upgrading.
The update will either resolve your issue, or make it more obvious what the error is now as I've all errors that I throw in the login method much more verbose, and less generic. E.G Unable to connect to Steam, Unable to validate OpenID credentials, etc...
Also I found out during the that it was possible for a steam account to be linked more than one forum account, which would then show this error message. During the update it will remove all duplicates and force them to resync again. Thanks to Aiwa for this and for the code that helps sort it out!
Let me know if you have any issues, this should be up on the marketplace soon.

@Lavoaster
Just tested the latest version per the instructions on readme.md. Working fantastic! I did notice one minor issue that I inadvertently caused:
After I installed the plugin I ended up deleting and re-adding the steam login handler for circumstantial reasons. That seemed to iterate the "login_id" of the handler from 10 to 11 under "core_login_methods". After doing so the "token_login_method" under "core_login_links" was still set to 10 per the old method causing existing steam users to be prompted to set up a new account. Not sure if it's practical but perhaps have some way to detect if the handler was deleted/re-added and auto-update "core_login_links"
Thanks for all of your hard work on this!

If the linked auths are still there but references the uninstalled handler
you may have to run a manual query to change them from 10 to 11 (or what
ever number the steam login is) , so they can login normally.

@madman32395 Yeah I've already updated the links to the new handler, that wasn't an issue. Was more just noting that I caused the problem to begin with as other people using the plugin may not be technically apt enough to fix it on their own.


[YBUG.IO] the qr code of the address copy...

the qr code of the address copy also ethereum:0x...... or just :0xcdndj....
Source url: https://live.sovryn.app/wallet
Reported by:
Reported at: 14 Apr at 22:52 UTC
Location: HU, Budapest, Budapest
Browser: Chrome 89.0.4389.128
OS: macOS 11.2.3
Screen: 1440x900
Viewport: 1440x768
Screenshot:

For more details please visit report page on Ybug:
https://ybug.io/dashboard/reports/detail/rr8rpvh3p0d5tak9ffat

Make fastbtc qr code string with prefix btc:


[Launcher] Check for disabledPackageSources (in NuGet.config)

Release Type: Official Release
Version: Xenko Launcher 3.0.1
Platform(s): Windows
Describe the bug
Because Xenko Launcher doesn't check for disablePackageSources in NuGet.conf, if the xenko nuget packages are disabled, the xenko launcher will think it's offline and won't be able to connect to the internet to download the xenko builds from the internet
To Reproduce
Steps to reproduce the behavior:

open NuGet.config in %APPDATA%\NuGet\NuGet.config
add these lines in configuration

<disabledPackageSources>
    <add key="Xenko" value="true" />
    <add key="Xenko Dev" value="true" />
  </disabledPackageSources>

The Xenko launcher cannot connect and download xenko builds

Expected behavior
The launcher should check if the xenko package are disabled and automatically enable them, or ask the user the authorization to enable them
Screenshots

Isn't this expected behavior though ? If users explicitly disable nugget sources they probably don't want xenko/stride to attempt fetching them ? I'll mark this one as enhancement instead, seeing as we could extend the window to mention why it failed.

@Eideren Yes I would also expect that the Launcher doesn't find them if the sources are disabled.
Not sure if the API let's us find out about disabled packages. If so, displaying a more explicit error message could be an option.


Add a "cron" mode to Tcorr daily export script

When in cron mode, the Tcorr daily export script should look through some fixed date range (last month or 2) and see if the Tcorr image is either missing or if the number or list of available Landsat images is different than what was used to generate the initial image.

A "cron" mode was added, but additional testing will be required to see if it is operating correctly.  I need to make sure that in a few days it is correctly picking up dates that have additional images that could be included in the Tcorr daily image.


Add embedding ops aten

Summary: Adds embedding ops for aten
Reviewed By: digantdesai
Differential Revision: D64477035

Mirror internal stamp


Make minor edits to Device Plugin documentation

These changes are a follow-up to https://github.com/kubernetes/website/pull/42001. They make minor edits to the documentation of the DevicePlugin API.

/cc @sftim


Bootstrap flake8 directly instead of relying on flakehell

Justification
Although we see a lot of benefits from using flakehell, like pyproject.toml support, baseline files, etc... it also has some unfortunate drawbacks. For the time being, I think we should step back from flakehell and investigate a solution where we bootstrap Flake8 directly. This also allows the door to be left open for when we're ready to also bootstrap formatting.
Implementation

Use click and toml as support libraries as we're already pulling them in black and they are easy to use.
Repeat the options we want to from flake8 and then fully manage the flake8 invocation. This will allow us to stop the need for duplication in the config(s) between the tools we build on top of (E.g. no need to make an exclude for black and flake8, now it's under ni-python-styleguide).

Thoughts for the future

Add a formatting command which applies fixes for lint errors in addition to running black
Separate out all this business logic into its own library/framework

Run poetry run ni-python-styleguide lint
  poetry run ni-python-styleguide lint
  shell: /bin/bash -e {0}
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.6.12/x64
./tests/test_cli/test_bare.py:13:1: BLK100 Black would make changes.
##[error]Process completed with exit code 1.

:(

Black would make changes.

Yeah, apparently black just released a new version. We hit this in a different project. If your code includes a list that has a trailing comma, e.g. (1, 2, 3,) then the new black will think you're asking it to split the list across multiple lines. Maybe there's something else you're hitting here, but that's what it was in the other project.


Remove .meta files from included .gitignore

I've been having troubles with this plugin, scripts were lost when sharing project between team members.
The cause : there is a .gitignore in the project that ignores .meta files.
Solution : Remove .meta from the .gitignore

That can't happen, unfortunately. Removing the .meta files from the .gitignore means they will be included in the plugin's GitHub repository. You should download only the .cs files or use the .unitypackage that contains only the required files without any extras.

What's the problem with including the .meta files? They are part of any packages.
See the following guide : https://docs.unity3d.com/Manual/CustomPackages.html
I highly recommend you restructure the project using the following guide, this will allow to include as a package (with Unity Package Manager (git) or OpenUPM)

What's the problem with including the .meta files? They are part of any packages.
See the following guide : https://docs.unity3d.com/Manual/CustomPackages.html
I highly recommend you restructure the project using the following guide, this will allow to include as a package (with Unity Package Manager (git) or OpenUPM)

Thanks for the link 🙂 I already read that guide.
There are no problems including the .meta files (I include them in the pre-packaged release binaries). They are extras in the context of this repository. They don't add or remove any value. Plus, when they change they're extra diff noise 🤷🏻♂
I will look into supporting the Unity Package Manager and OpenUPM soon. Thanks for the suggestion.


flutter apps dont run on windows sandbox

Steps to reproduce

build a Flutter app for Windows
run the built app
a bunch of errors are shown

Expected results
run
Actual results
when opening this what happens

Code sample
any code
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
no need

Please see https://docs.flutter.dev/platform-integration/windows/building#distributing-windows-apps for information about what needs to be part of your application. The C++ redistributable libraries are necessary dependencies, so if you are running your app in a restricted environment it's your responsibility to make sure those libraries are available there.


Added the ability to add work logs

Unable to find 'Newtonsoft.Json.9.0.1.nupkg'. Make sure the project has been built.

Should I have done my PR against another branch?

I'm still having difficulties with creating my nuget packages which ends up breaking the builds.
I still need some time to fix this, will try to do this in the upcoming month.
Van: "Sean Sparkman"<EMAIL_ADDRESS>Aan: "jonas0007/Jira.SDK"<EMAIL_ADDRESS>Cc: "Subscribed"<EMAIL_ADDRESS>Verzonden: Donderdag 15 maart 2018 03:34:38
Onderwerp: Re: [jonas0007/Jira.SDK] Added the ability to add work logs (#43)
Unable to find 'Newtonsoft.Json.9.0.1.nupkg'. Make sure the project has been built.
Should I have done my PR against another branch?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, [ https://github.com/jonas0007/Jira.SDK/pull/43#issuecomment-373240752 | view it on GitHub ] , or [ https://github.com/notifications/unsubscribe-auth/AGC6Mxvj_hJeOpzzT_3rg9OnXfVGZKYtks5tedM9gaJpZM4SrdXc | mute the thread ] .

Hi,
The automated build process should have been fixed.
Could you try and merge my master into your branch, and push the changes agian? It should fix the build.


Add get_model and get_mcs function

This makes it much easier to quickly run the RFF SPs and take a look at the output.
We should add docs at some point, but ideally would merge before that, I need this for something else :)

Codecov Report

Merging #34 (ef0559a) into main (3f1c9cb) will decrease coverage by 20.31%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##             main      #34       +/-   ##
===========================================
- Coverage   96.49%   76.19%   -20.31%
===========================================
  Files           3        3
  Lines          57       63        +6
===========================================
- Hits           55       48        -7
- Misses          2       15       +13

Flag
Coverage Δ

unittests
76.19% <0.00%> (-20.31%)
:arrow_down:

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

Impacted Files
Coverage Δ

src/MimiRFFSPs.jl
11.76% <0.00%> (-88.24%)
:arrow_down:

src/components/RegionAggregatorSum.jl
100.00% <ø> (+33.33%)
:arrow_up:

src/components/SPs.jl
100.00% <ø> (+1.96%)
:arrow_up:

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

@davidanthoff that sounds fine to me looking over it quickly, we should add these functions to the testing at minimum since I sort of already test this in those files, can you add an Issue and tag me and I'll do that next week?


Move IBM-Blockchain samples to fabric repository

We created a couple of samples and tutorials as part of our Bluemix deployment of the fabric.  We've seen / heard interest in having some of them moved under the HL organization.  We're happy to do so, but wanted to get some feedback on how to proceed - e.g. do we just move the chaincode and keep the webapps outside, etc

@christo4ferris should be able to help with details on how to proceed.

@mastersingh24 discussed with @srderson ... he was thinking we might have a wiki page that refers to various samples - not just ours - but with a YMMV caveat. I would like to port the learn_chaincode one to hyperledger and contribute that (unbluemix-ized - just focus on running local)

@christo4ferris - ok.  cool.  we can port the learn_chaincode no problem.  Just let us know where we should put it and consider it done.  we'll need to add some tests to it so it will keep building as interfaces change, but that's not an issue

@mastersingh24 when things calm down, let's do this. Thanks


add github-browser

Our legal team has advised us to disallow apps that are using the names of other companies or icons that we find too similar to the logos of other companies without verifying their permission to do so. We would still be happy to feature your app. If you are an author of this app and would like to see it on the site, please change the icon and update this PR to use the new icon.
Apologies for the inconvenience, and thanks for your understanding.
cc @lee-dohm

For future reference, with regard to GitHub's logos see the GitHub logo guidelines. As for the name of the app, it shouldn't start with the word "GitHub". In general, you may refer to GitHub in a relational phrase to say that the project is "compatible with", "on", or "for" GitHub, or something along those lines.

understood, thanks!

@tborychowski please do submit another PR if you get around to making those changes.


How do you preserve the real Client IP?

Hello,
i have a Problem where I can't see the the real client IPs in my Traefik access log. I only see the cilium interface IP of the Node where Traefik is running.
The Traefik service is from the type LoadBalancer and I use ciliums integrated Layer 4 LoadBalancer to allocate an IP Address.
After much googling I found out that I have to either use externalTrafficPolicy: local or use ciliums kube-proxy replacement to enable DSR.
I would use PROXY Protocol or correct X-forwarded-for headers when I would know how to configure these in cilium.
So my question ist. Is there any way I can use the Cilium LoadBalancer and preserve the real client IP without the hassle of switching to cilium kube-proxy replacement or use externalTrafficPolicy? This is pretty important to me because otherwise it is impossible to block access from specific IPs.
Best regards
Marie

If you are not running kube-proxy replacement, then the component that causes the loss of original client IP is kube-proxy. Therefore you would need to ask the kube-proxy community about mitigations. In general, the Cilium solution to this problem is documented here.


Localize strings

This pull request wraps all UI strings in localization functions, adds the Grunt i18n task (which generates/updates a .pot file for localization). To update the .pot file (found in /languages/hypothesis.pot), run the following from within the plugin directory:
npm install && grunt i18n

Thanks much, @greatislander!


Can't install any plugins through Neovim/WSL.

So, I can't seem to install any plugin, at all. I have the exact same init.vim file (and general setup) as on another machine, which works flawlessly. Re-installed both neovim, the extensions and vim-plug itself, to no avail. It's run through ConEmu.
The plugins are fetched just fine (~/.local/share/nvim/plugged/ctrlp.vim/ etc.), but neovim seems to hate the plugin code with the fury of a thousand suns.
The error messages vary wildly depending on the plugin so I decided to limit it to only ctrlp. The error message also pops up when launching neovim. This is the output:
Error detected while processing /home/[username]/.local/share/nvim/plugged/ctrlp.vim/plugin/ctrlp.vim:
line    7:
E492: Not an editor command: ^M
line    8:
E15: Invalid expression: ( exists('g:loaded_ctrlp') && g:loaded_ctrlp ) || v:version < 700 || &cp^M
line   15:
E475: Invalid argument: ^M g:ctrlp_allmixes, g:ctrlp_buftags, g:ctrlp_ext_vars, g:ctrlp_builtins]^M = [[], [], [], [], {}, {}, [], 2]^M
line   71:
E171: Missing :endif

I have a feeling this isn't a bug, but something unique to my system, but any help is appreciated.
NVIM 0.2.0-dev
Build type: RelWithDebInfo
Compilation: /usr/bin/x86_64-linux-gnu-gcc -g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Werror=format-security -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -O2 -g -DDISABLE_LOG -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wvla -fstack-protector --param ssp-buffer-size=4 -DINCLUDE_GENERATED_DECLARATIONS -DHAVE_CONFIG_H -D_GNU_SOURCE -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/build/config -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/src -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/.deps/usr/include -I/usr/include -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/build/src/nvim/auto -I/build/neovim-SoO7pA/neovim-0.1.7ubuntu1+git201702241003+3156+23~ubuntu14.04.1/build/include
Compiled by root@lcy01-06

Optional features included (+) or not (-): +acl   +iconv    +jemalloc +tui
For differences from Vim, see :help vim-differences

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/share/nvim"

Type:

[x] Bug
[ ] Enhancement
[ ] Feature Request
[ ] Question

OS:

[ ] All/Other
[ ] Linux
[ ] OS X
[x] Windows

Vim:

[ ] Terminal Vim
[ ] GVim
[x] Neovim

I'm not sure, but it looks like git for Windows is being used in WSL. Reinstall git on WSL (apt-get) and make sure that version is used (which git) and see if it helps.

Uninstalled Git for windows (.exe) as well as in WSL (apt-get remove git). Re-installed it in WSL. Still the same error.
which git
/usr/bin/git

Did you delete the plugins that are already cloned before reinstalling git?

Yep. Just did it again to be sure (removed plugged folder, reinstalled).

Well, it doesn't seem like an issue of vim-plug.
You might want to check if the plugin works without vim-plug. Comment out plug#begin() ~ end() block and manually add the directory to your &rtp and see if it works; i.e set rtp+=~/.local/share/nvim/plugged/ctrlp.vim.

Yep, error still occurs.

I see, so, it's not an issue of vim-plug.

Thank you for your time...

No problem. It still seems like a Windows git issue to me.

I'll look into it.

A quick update: Deleted everything related to Git on Windows (to my knowledge), reinstalled WSL. Installed latest version of git (2.11.0). Tried in both Vim and Neovim. No luck.
I guess I'll just have to wait or something...


Added option for generating 'struct' for types defined with 'struct' keyword wherever possible.

Added everything and as faar as I can see it is working fine.
Please review @ddobrev @tritao

Had to change two tests from 'struct' to 'class' because they were generating erroneous code when marshalled with 'struct' keyword.

Instead of changing the tests I thought it's better to change the allocation of 'IsStruct' property.

@ddobrev Yes, the build was failing due to obe of the static members in ValueType..
Will try to fix it...

Although i combined them but now the CLI generator is generating erroneous code in Basic.cpp 'Exception' and STL.cpp
I am looking for the cause.

In case you have problem with this, wouldn't setting of Class.Type to ClassType.ValueType instead of setting IsStruct to true, work better?

@ddobrev Thanks. Using that and a few other bits reduced the errors a lot. I am working on removing the left errors. For that we need some changes which I am wroking on in #506 .

Once #506 is merged, i will open a fresh pull with all the working changes.
But I had to change the TestDestructors test from 'struct' to 'class' because structs can't have destructors in C#.
Should I improve the check to include this or we could provide a new macro CS_REF_TYPE.. as CS_VALUE_TYPE marshals a C++ class to C# struct.
CS_REF_TYPE could be used to force marshaling a C++ struct to C# class...

Overall I don't think this approach makes sense, we can't just assume something is a struct just by looking if it was originally declared as a struct in C++. We should be instead checking its structure (operators, inheritance chain, members) and if it's a POD type.

What doesn't make sense? He does make all of the mentioned checks.

@tritao I have pushed a couple of changes here. But, I will need the changes of #506 for these to work correctly. I will submit another PR with fresh changes.
.
Also, I too don't see why it is not making sense?
We have all the proper checks here before deciding. The 'struct' keyword is just one of those checks!

How a type was written in native code (whether with class or struct keyword) should not make any difference to how its classified (ref vs value type). We should solely look at its structure and usage to make that determination.

@tritao I have added a new pass to work for that and also I have removed the checking of 'struct' keyword in a separate branch. Please merge #506 first if you have no objections on the way that is done.
Then I will open the pull for this because I really need the changes on that pull.
Till then I am closing it since it is obsolete I looked at the changes and they are confusing here.

@tritao I think it should make a difference. The way we do it now does not check if a type's been written as a struct, it's actully the other way around. It's checked if it's a class - in which case it's left alone. If it's written as a struct, only then we apply the checks to see if it can be a struct.
@genuinelucifer when you get back to this issue, please add an additional check - if the type has any virtual functions.