Overriding blueprints method is Broken

Sails version: 0.12.3
Node version: 5.8.0
NPM version: 3.7.3
Operating system: ubuntu

I am trying to override blueprints 's add method like mentioned in the doc
I just copied the add.js default file into a newly created api/blueprints folder. I change the require('actionUtil') with the good path and everything works until i am trying to add some collection to another. It go through the new add.js file but this file seems broken by dependencies on loadash
here is the error
TypeError: _.findWhere is not a function
    at addToCollection (/var/www/momi2/api/blueprints/add.js:45:27)

I also add that i tryed whole things severals times. And was a completely new install
sudo sails<EMAIL_ADDRESS>-g
sails new projectName
// and generate on Article api and one Tag api to test

ok fixed

Hi @Oldwo1f.  Would you mind setting up a repo that reproduces this in a new sails app real quick? I think that'd be the easiest way to see what's going on.
e.g.:
sails new sails-issue-3716
cd sails-issue-3716
# Now make whichever are the simplest/quickest changes which reproduce the issue,
# and when you for sure have the issue isolated, run:
git init
hub create  # or use the GitHub UI to create the repo and `git remote add` it manually
git add .
git commit -am 'Reproduces https://github.com/balderdashy/sails/issues/3716'
git push -u origin master
hub browse # or open the repo in your browser manually
# Then copy the URL of the new repo and paste it as a comment on this issue.
# We'll check it out, and since this way we'll be able to experience the problem
# ourselves, we should have no problem getting to the bottom of it.

Thanks!

Ok, i have it  -->  https://github.com/Oldwo1f/sails-issue-3716
What i did :

create a new sails app (no other node module installed)
generate 2 api, one for article and one for tag.(with one simple text field and the oneToMany association)
creating the api/blueprints  folder with 2 file in it : actionsUtil.js and add.js
i had to change path  in require at the top of these 2 files to load dependencies correctly <<-- first problem
i use blueprint to create new article and new tags independently.
i Try to use blueprint to add on tag to one article, but here, i got the error mentioned above.

When I try to lift your repo, I get:
error: A hook (`blueprints`) failed to load!
error: Error: Cannot find module 'lodash'

which tells me two things:  you didn't do use --save when you installed the Lodash dependency in your repo, and most likely you didn't install a version of Lodash that has the findWhere function.  Try npm install lodash@3 or updating your code to not use _.findWhere.
The docs encourage you to look at the internal blueprint code as a jumping-off point when making your own custom blueprints, but it's not intended to mean that they'll necessarily work if you just copy/paste them; Sails code has different dependencies than your local app.

i didnt installed lodash at all in my project.
If these methods are not itended to work, maybe you can tell me where i can find the real blueprints add method in order to override it just a little.

i didnt installed lodash at all in my project.

Are you sure?  I don't know how this line in your repo works otherwise.  That's where I'm getting the cannot find module 'lodash' error.

If these methods are not intended to work, maybe you can tell me where i can find the real blueprints add method in order to override it just a little.

add.js is the default "add" blueprint; all I meant to say is that the fact that you have to adjust it a little (for example by copying over actionUtil.js and changing the path) is not a bug, it's just a fact of copying code from one place to another.

Ok thank you Sgress, i understand it better.
It seams works with npm install lodash@3
And yeah, i am sure i didnt install lodash before, i just run a new project for this issue, i thought it was included in sails.
btw, overriding is working, so we can close this issue.


Make sure that the original object fed to the sample method is cleaned up before multiprocessing

To avoid forking this big memory object.

attn @wcatino, this should help . Let me know if it does?


fix: attestation pool

Please read contributing guidelines and remove this line.

#8854  👈
master

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

Join @Maddiaa0 and the rest of your teammates on  Graphite


Yard does not document prepended Modules in Classes

Yard does not generate any documentation for prepended Modules in a Class.
Steps to reproduce
Write the following code and generate the documentation from it.
module Bar
  def meth
    super + 'bar'
  end
end

class Foo
  prepend Bar

  def meth
    'foo'
  end
end

class Moo
  include Bar

  def meth
    'foo'
  end
end

Actual Output

Expected Output
I expected it to have at least a somewhat similar output to include.

Environment details:

OS: > uname -srmov
Linux 4.4.0-62-generic #83-Ubuntu SMP Wed Jan 18 14:10:15 UTC 2017 x86_64 GNU/Linux
Ruby version: > ruby -v
ruby 2.3.0p0 (2015-12-25 revision 53290) [x86_64-linux]
YARD version: > yard -v
yard 0.9.8

I have read the Contributing Guide.

Closing out some old issues. Given the significant age of this, it would be best to re-open with new context if it's still relevant.


Relax pinned dependencies

Is your enhancement request related to a problem? Please describe
It might be wise to take a look at our existing dependencies in setup.py and see if we can loosen them. We've already had one issue (#65) that spawned from too strict dependencies.
Describe the solution you'd like
Take a look and adjust dependency pins (or unpin) as needed

closing in favor of edgetest integration #195

https://github.com/capitalone/edgetest


scripts: west: be more explicit in copy warning message

Print the exact path to the copy of west that was run, to further avoid confusion. Explain exactly what is going away and why.

Codecov Report

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

@@           Coverage Diff           @@
##           master   #11537   +/-   ##
=======================================
  Coverage   48.42%   48.42%
=======================================
  Files         270      270
  Lines       42126    42126
  Branches    10139    10139
=======================================
  Hits        20400    20400
  Misses      17645    17645
  Partials     4081     4081

Continue to review full report at Codecov.

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


Backup to my own e-mail

Hi
can I use SMS Backup + to backup sms and etc to my own email address?

If you mean IMAP-server: Yes, as described in https://f-droid.org/packages/com.zegoggles.smssync/ and https://github.com/jberkel/sms-backup-plus/blob/master/README.md
Can you please close this issue, if this answers your question?
Thanks

no activity, closing


Delain Armen Nomination Does Not Exist

Delain Armen is not an option to select in East Luterra. This is very important as it is one of the main things people who haven't completed LWC are looking for now (me personally also).
Confused why there is an option to be notified, but no option to nominate it in the dashboard.

I'm not sure what you mean by dashboard, it's available on the Morris merchant as well as in Push notification settings.


Improving of Tweet block

We can simplify including the Tweet block by deleting requirement of getting tweet info from the Twitter API.
We can include tweet just with URL e.g:

Demo: https://jsfiddle.net/n7cc20dk/
What do you think?

Hi @chekalskiy,
Thanks for the suggestion.
When the tweet block is created we don't know all the ways that it might be rendered in the future.
As you show you want to use the default widget.js without placeholder blockquote text. However someone else might want to use the json data as a placeholder, or others might want to custom render tweets in accordance with the Twitter display requirements.
I think there are more people like you that want to use the tweet block but don't want to implement the twitter api backend.
There are 2 ways you could contribute:

Create a pull request for the twitter block with a config option for using the twitter api. The default should probably be api backed to maintain backwards compatibility.
Create a custom block which someone can load to override the inbuilt block.

Raffi

Hi @chekalskiy!
This approach would work, but it's good to note that you'd want to ensure that the Twitter JS isn't included multiple times.
Another consideration is that it can be good to avoid having to wait for Twitter to respond before rendering tweets, this can severely affect the user experience (especially on mobile devices).
Those two things being said, there's no reason you couldn't have an alternative block for tweets — that's what custom blocks is all about!

@chekalskiy did you implement your sugestion?

@titu00 normally when you embed a tweet you'll get a blockquote which is enhanced using javascript. If you do it without a server call then you'll only have the status_url given in the editor.
An example of this block is https://gist.github.com/raffij/d99f4d1d19fc7e49efa533ecf763befa
Take it and customise as required.


runtime: remove some unnecessary use of llvm/Support/Compiler.h

Replace many of the LLVM attributes in favour of their Swift namespaced
counterparts.  This has no impact on the code itself, just uses a
separate namespace for the macros that we already have in the Swift
shims rather than dragging in Support/Compiler.h from LLVMSupport.

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @jckarter

@swift-ci please test

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 4a9852651487cb699c7ed9338bb2986cfabf8160

@swift-ci please test Windows platform

Build failed
Swift Test OS X Platform
Git Sha - 35e8f5a0e8ca8efd97637b6fef57395e5a0d5adc

Splitting this up since it didn't just immediately pass.


Need custom fetch options for native fetch

Frameworks like Next.js modify the native fetch implementation to add caching strategies to individual requests (server side). This, in the case of the Stytch Node SDK, creates an issue when making native fetch requests.
For example:
A call using the SDK such as:
const loginOrCreateResponse = await stytch.otps.email.loginOrCreate({ email });

... will work fine on the first attempt, and the user will receive their OTP code. However, if a second request is made with the exact same parameters (email in this case), it will result in a cache hit in Next.js/Vercel and the request won't actually go out to Stytch to complete the request, but instead returns a cached response. No second OTP code is sent.
https://github.com/stytchauth/stytch-node/blob/2479bd3877e66ee745d6ff6607751626112384a7/lib/shared/client.ts#L55-L60
☝ Here is where the fetch configurations are created. For Next.js, special options need to be passed in order to prevent the above behaviors from occurring.
Please let me know if a reproduction is required, or if it's preferable I can make a contribution to the SDK to add the ability to pass custom fetch arguments.

Note: I might be missing some configuration I can pass to a custom undici agent, but I can't seem to find the right configuration if that's possible.

Heya @devjmetivier - thanks for opening the issue. @chris-stytch actually ran into this internally the other day 😞. As an immediate workaround, you can modify the fetchConfig directly by casting the client as type any annotations - e.g.
function monkeyPatchStytchClientSettings(client: ...) {
  /* eslint-disable */
  const cl = <any>client;
  /* eslint-enable */
  cl.fetchConfig.cache = 'no-store';
}

It's a little icky, but it should do the trick.
You can also call a dynamic function like headers() to disable the fetch cache for an entire route, if that is preferred.

Regarding changing the Stytch SDK itself - I have a few questions around implementation that I'll try to get answered:

Are there other SDKs or libraries that have experienced this same issue?
How do they expose custom fetch options?
Does it make more sense to expose global fetch options on the new Stytch constructor, per-method options, or both?

@max-stytch

Are there other SDKs or libraries that have experienced this same issue?

How do they expose custom fetch options

Auth0's Node SDK experiences the same issue, but they allow a custom fetcher (as opposed to a custom config for fetch).

Details on how they do that

👇 Here they start by adding fetch to their default configuration options in types
https://github.com/auth0/node-auth0/blob/master/src/lib/models.ts#L8-L42
And this moves up the chain eventually to their Auth Client options...
https://github.com/auth0/node-auth0/blob/master/src/auth/base-auth-api.ts#L17-L25
https://github.com/auth0/node-auth0/blob/master/src/auth/index.ts#L12-L22
And finally gets passed down the chain where they'll either use your custom fetcher, or native fetch:
https://github.com/auth0/node-auth0/blob/master/src/lib/runtime.ts#L24-L52

Does it make more sense to expose global fetch options on the new Stytch constructor, per-method options, or both?

This is the part where I would inject my opinion. Rather than sending along fetch config options, I'd prefer to pass my own custom fetcher entirely. But I also recognize the following:

A quicker fix is likely more necessary as I assume more will experience this issue, so the ability to pass the fetch config would be preferable (all that's technically needed to fix this is the ability to modify the cache property in the fetch config)

Note: I'm unsure if anyone would need what Next.js adds to fetch. The next property is the only thing they add. Not sure how that would work.

I'm not sure where this leaves the SDK's use of undici, or what the implications of adding custom fetchers would have for it.

I'm also not even sure what the purpose of undici is in the Stytch SDK anyways

The fetch implementation from undici is never actually used, so I'm not sure what passing a custom dispatcher even does here. I might be missing something 🤷♂
https://github.com/stytchauth/stytch-node/blob/a0eae9b99caca56c5cec36870034cb3aa67bb45e/lib/shared/index.ts#L36
Undici docs example using their custom fetch implementation

I'm not sure where this leaves the SDK's use of undici

We don't use undici directly. On Node environments, undici is the backing library for globalThis.fetch, and we allow passing custom Dispatchers to it for configuration of telemetry, SSL certificates, HTTP Keepalive, etc.

I'd prefer to pass my own custom fetcher entirely.

That's probably the most robust solution for power users. We were hoping we could avoid the complexity of custom fetchers now, since Node 18 made the fetch API globally available and we were able to drop support for Node 16 earlier than expected. I don't like how custom fetchers put the burden on the caller. Ideally we'd just work in NextJS environments, and we wouldn't require every caller on NextJS 13 to implement the same monkeypatching approach.

unless there's a specific use case for doing both

There are valid API calls that could be cached - for example, stytch.users.get({ user_id }) is a great candidate for caching. But any call that is a POST, PUT, or DELETE is a bad candidate.
In https://github.com/stytchauth/stytch-node/pull/266 we're experimenting with adding custom options to requests. If we stick with that approach, passing in cache options or additional fetch options would be quite nice.

Related: https://github.com/supabase/supabase-js/issues/725

@max-stytch

We were hoping we could avoid the complexity of custom fetchers now

Totally. I don't personally/professionally use custom fetchers often anyways. Sticking with adding custom fetch config options for now is a great start.

There are valid API calls that could be cached - for example, stytch.users.get({ user_id }) ...

This is a great point. The only thing I'll add is that cache ought be opt-in for auth related requests. Modifying the default cache behavior of fetch to be no-cache regardless of the environment/framework the SDK is being used in is probably a safe bet. Food for thought 🍜

Modifying the default cache behavior of fetch to be no-cache regardless of the environment/framework the SDK is being used in is probably a safe bet.

I agree completely. https://github.com/stytchauth/stytch-node/pull/271 should do just that and resolve the immediate issue with NextJS v13+. I'll open up a separate issue to track if anyone does want to cache responses, and would like the ability to pass arbitrary options into native fetch.


Stop requiring SP credentials when user-based auth is used

Description
The EnvironmentVariableLoader currently assumes there are always environment variables set for service principal authentication (a tenant ID, client ID, and client secret). User-based authentication doesn't use these variables, so there's no reason to continue requiring them.
This PR updates the preparer to handle unset variables when user auth is requested instead of failing with a KeyError, as well as unset corresponding AZURE_* variables accordingly. I debated whether we should unset missing variables or leave missing variables untouched, but settled on the former to make our behavior as consistent as possible.
When the EnvironmentVariableLoader is used, the intent is to only use environment variables that begin with the service directory's prefix. When running KV tests, AZURE_CLIENT_ID, etc. are always set to KEYVAULT_CLIENT_ID, etc. If we stopped changing the value of AZURE_CLIENT_ID when KEYVAULT_CLIENT_ID is unset, that would be inconsistent and cause confusion if tests were to inspect AZURE_CLIENT_ID and expect that it's set to a KV-specific value. Setting AZURE_CLIENT_ID to a placeholder value would also be inconsistent since we usually don't modify KEYVAULT_CLIENT_ID -- leaving the latter variable unset could be intentional, and we should reflect this in the former variable.
If any of these variables were unset in the past, the preparer would fail -- so any behavior updates are non-breaking in that sense.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

/check-enforcer override


Fix instrument and posinf warnings

Fix warnings for gcc and clang. Closes #1245 and closes #1230.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

@AlexandreEichenberger I removed the comma after PERF_MAIN() as the smaller delta, if you would prefer I refactor the macro to resolve the warning please let me know.

@AlexandreEichenberger I removed the comma after PERF_MAIN() as the smaller delta, if you would prefer I refactor the macro to resolve the warning please let me know.

In general, fixing the macro would be the better approach

@jenkins-droid test this please

Can one of the admins verify this patch?

@jenkins-droid test this please

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Jenkins Linux amd64 Build #4569 [push] Fix instrument and posin... failed after 1.3 sec

Jenkins Linux s390x Build #4586 [push] Fix instrument and posin... failed after 2 sec

Jenkins Linux ppc64le Build #3706 [push] Fix instrument and posin... failed after 1.5 sec


Resolve conflict, accept branch being merged in

Fixes #1019
Traced to this commit
Looks like a local preview branch was edited without pulling remote preview branch beforehand (or something similar).
Whatever happened, it looks like the branch being merged in is the desired version. I accepted all changes for that branch (With one exception: the get_modified_files function, which I commented out as it is unused)

@justinmilner1 this is awesome, thank you!

@sestinj Whoops, does it matter that the yarn.lock file was in there? Quick google says no... but want to make sure

It's all good! Thanks for asking. There's an OS-specific dependency in there so sometimes commits will end up toggling that back and forth, but it doesn't break anything.
Though now that you bring it up I ought to look for something similar to --no-save that might prevent this


ERROR: Assembly with same name is already loaded when running in Azure Function

I'm trying to get this to work in a Azure Powershell function (httptrigger)
Test code I have is the following
Import-Module MSAL.PS

$TenantId = "xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
$ClientId = "xxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
$CertThumbPrint = "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"

$connectionDetails = @{
    'TenantId'          = $TenantId
    'ClientId'          = $ClientId
    'ClientCertificate' = Get-Item -Path "Cert:\CurrentUser\My\$CertThumbPrint"
}

$Token = Get-MsalToken @connectionDetails

Local in vs code debug it works fine.
However, when testing this code in Azure I get the following error

2021-03-24T15:52:24.871 [Error] ERROR: Assembly with same name is already loadedException             :Type       : System.IO.FileLoadExceptionMessage    : Assembly with same name is already loadedTargetSite :Name          : LoadFromPathDeclaringType : System.Runtime.Loader.AssemblyLoadContextMemberType    : MethodModule        : System.Private.CoreLib.dllStackTrace :at System.Runtime.Loader.AssemblyLoadContext.LoadFromPath(IntPtr ptrNativeAssemblyLoadContext, String ilPath, String niPath, ObjectHandleOnStack retAssembly)at System.Runtime.Loader.AssemblyLoadContext.LoadFromAssemblyPath(String assemblyPath)

I have MSAL.PS <IP_ADDRESS> included in the Modules folder of the project
Any clue what might be causing this issue and how to resolve it?

I think this is fixed in the following PR that has been merged - #26
A new release hasn't been created but I just updated MSAL.PS.ps1 with the code from the PR and that solved my issue with the dll conflict

thx, I used the updated MSAL.PS.ps1and it is working as a charm in a Azure Function after removing Import-Module MSAL.PS
from my test code

@jasoth any idea when this PR will get released?

Hi everyone,
Sorry about the delay. I just released an update (v<IP_ADDRESS>) to address this. Hopefully there is a more elegant solution with AssemblyLoadContext that can be implemented at some point in the future but let me know if the updated module works for you.


Dev/workflow id skip

Hi @bmoore-msft ;  please accept this PR in replacement of the previously closed one.


nn_models refactoring

base_nn_model refactoring
improved validation

Thanks for update, @technocreep!

In the file benchmark/benchmark_TSF.py:

Line 92:35: E126 continuation line over-indented for hanging indent
Line 95:58: E126 continuation line over-indented for hanging indent
Line 125:33: E126 continuation line over-indented for hanging indent
Line 132:27: E128 continuation line under-indented for visual indent

In the file fedot_ind/api/main.py:

Line 356:54: F541 f-string is missing placeholders

In the file fedot_ind/api/utils/recurrent_image.py:

Line 38:19: W292 no newline at end of file

In the file fedot_ind/core/architecture/preprocessing/data_convertor.py:

Line 92:13: E722 do not use bare 'except'
Line 266:13: E722 do not use bare 'except'
Line 408:121: E501 line too long (122 > 120 characters)
Line 486:13: E722 do not use bare 'except'

In the file fedot_ind/core/metrics/metrics_implementation.py:

Line 43:26: E126 continuation line over-indented for hanging indent
Line 74:16: E126 continuation line over-indented for hanging indent
Line 75:74: W503 line break before binary operator

In the file fedot_ind/core/models/nn/network_impl/base_nn_model.py:

Line 191:47: F632 use ==/!= to compare constant literals (str, bytes, int, float, tuple)

In the file fedot_ind/core/models/recurrence/reccurence_extractor.py:

Line 8:1: F403 'from fedot_ind.core.metrics.metrics_implementation import *' used; unable to detect undefined names
Line 54:46: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 69:24: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 70:17: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 76:33: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 84:48: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 94:50: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation
Line 102:35: F405 'np' may be undefined, or defined from star imports: fedot_ind.core.metrics.metrics_implementation

In the file fedot_ind/core/models/ts_forecasting/ssa_forecaster.py:

Line 14:5: F401 'seaborn' imported but unused
Line 176:48: E127 continuation line over-indented for visual indent

In the file fedot_ind/core/operation/IndustrialCachableOperation.py:

Line 20:51: F541 f-string is missing placeholders

In the file fedot_ind/core/operation/interfaces/industrial_model_strategy.py:

Line 2:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 7:121: E501 line too long (122 > 120 characters)
Line 23:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 32:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 35:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 35:100: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 38:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 38:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 45:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 54:1: E302 expected 2 blank lines, found 1
Line 66:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 73:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 81:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 85:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 85:100: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 89:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 89:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 98:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 106:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 109:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 109:99: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 113:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 113:107: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 122:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 125:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 125:99: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 130:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 130:107: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 139:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 142:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations

In the file fedot_ind/core/operation/interfaces/industrial_preprocessing_strategy.py:

Line 7:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 23:26: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 43:121: E501 line too long (121 > 120 characters)
Line 54:42: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 56:61: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 84:14: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 99:34: F405 'np' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 107:30: F405 'np' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 109:42: F405 'np' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 118:68: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 143:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 182:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 182:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 216:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 216:100: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 241:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 267:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 271:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 276:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 276:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 286:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 299:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 306:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 306:100: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 312:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 312:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 322:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 327:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 332:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 339:39: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 340:58: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 350:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 353:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 359:53: F405 'Optional' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 362:31: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 365:56: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 365:99: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 371:64: F405 'InputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 371:108: F405 'OutputData' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations

In the file fedot_ind/core/operation/transformation/basis/eigen_basis.py:

Line 119:9: E704 multiple statements on one line (def)
Line 122:9: E704 multiple statements on one line (def)
Line 126:9: E704 multiple statements on one line (def)
Line 131:9: E704 multiple statements on one line (def)
Line 171:9: E704 multiple statements on one line (def)

In the file fedot_ind/core/operation/transformation/data/kernel_matrix.py:

Line 1:1: F401 'PIL.Image' imported but unused

In the file fedot_ind/core/operation/transformation/regularization/spectrum.py:

Line 20:49: E225 missing whitespace around operator
Line 65:9: E704 multiple statements on one line (def)

In the file fedot_ind/core/repository/constanst_repository.py:

Line 21:1: F403 'from fedot_ind.core.models.topological.topofeatures import ' used; unable to detect undefined names
Line 21:1: F401 'fedot_ind.core.models.topological.topofeatures.' imported but unused

In the file fedot_ind/core/repository/industrial_implementations/abstract.py:

Line 2:1: F401 'typing.Iterable' imported but unused
Line 38:1: E302 expected 2 blank lines, found 0

In the file fedot_ind/core/repository/industrial_implementations/optimisation.py:

Line 64:42: F821 undefined name 'GPAlgorithmParameters'
Line 344:36: E127 continuation line over-indented for visual indent
Line 346:37: E127 continuation line over-indented for visual indent
Line 510:5: F841 local variable 'other' is assigned to but never used
Line 523:17: F841 local variable 'parent_operation' is assigned to but never used
Line 525:21: E125 continuation line with same indent as next logical line

In the file fedot_ind/core/repository/model_repository.py:

Line 14:1: F403 'from fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations import *' used; unable to detect undefined names
Line 67:36: F405 'PCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 68:41: F405 'FastICAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 69:46: F405 'PolyFeaturesImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 125:20: F405 'ScalingImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 126:26: F405 'NormalizationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 128:30: F405 'ImputationImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 130:23: F405 'KernelPCAImplementation' may be undefined, or defined from star imports: fedot.core.operations.evaluation.operation_implementations.data_operations.sklearn_transformations
Line 239:32: E131 continuation line unaligned for hanging indent
Line 254:62: W292 no newline at end of file

In the file fedot_ind/core/tuning/search_space.py:

Line 60:111: E231 missing whitespace after ','
Line 821:36: W292 no newline at end of file

In the file fedot_ind/tools/loader.py:

Line 11:1: F401 'datasetsforecast.m4.M4' imported but unused
Line 51:80: E271 multiple spaces after keyword
Line 76:30: F541 f-string is missing placeholders
Line 102:26: F541 f-string is missing placeholders
Line 228:25: F841 local variable 'problem_name' is assigned to but never used
Line 268:29: F841 local variable 'univariate' is assigned to but never used
Line 303:25: F841 local variable 'class_label_list' is assigned to but never used
Line 347:59: E126 continuation line over-indented for hanging indent
Line 476:53: W503 line break before binary operator
Line 487:53: W503 line break before binary operator
Line 526:53: W503 line break before binary operator
Line 621:68: E126 continuation line over-indented for hanging indent
Line 621:121: E501 line too long (143 > 120 characters)
Line 649:60: E126 continuation line over-indented for hanging indent
Line 695:33: F841 local variable 'ex' is assigned to but never used
Line 702:33: F841 local variable 'ex' is assigned to but never used
Line 711:45: E126 continuation line over-indented for hanging indent


primitives: Inline public functions

Audit the primitives module and all submodules; add inline to all public methods that are small enough (excl. error impls).

concept ACK. I'm not sure what your exact criteria is and I'm not sure how to vet your choices, but nothing here seems particularly egregious.
If we wanted a specific policy I think "one-liners and functions we've actually benchmarked to be faster with inlining" would be a good one.

I'm not sure what your exact criteria is

My critera was, after re-reading #1595 discussion, "kind of guess at what the assembler would be, given I know very little about compiler optimisations, and then aggressively inline".


There is no ExtSdkApiRN native module on ios!?

Description
when I started my server and ran on ios simulator, I found there was an error:
Invariant Violation: Failed to call into JavaScript module method AppRegistry.runApplication(). Module has not been registered as callable. Registered callable JavaScript modules (n = 11): Systrace, JSTimers, HeapCapture, SamplingProfiler, RCTLog, RCTDeviceEventEmitter, RCTNativeAppEventEmitter, GlobalPerformanceLogger, JSDevSupportModule, HMRClient, RCTEventEmitter.
Finally, I checked the source code of this project and found there was no definition of NativeModules.ExtSdkApiRN.
there is no native module file in the ios folder?!!!
Version
1.0.11
Output of npx react-native info
Invariant Violation: Failed to call into JavaScript module method AppRegistry.runApplication(). Module has not been registered as callable. Registered callable JavaScript modules (n = 11): Systrace, JSTimers, HeapCapture, SamplingProfiler, RCTLog, RCTDeviceEventEmitter, RCTNativeAppEventEmitter, GlobalPerformanceLogger, JSDevSupportModule, HMRClient, RCTEventEmitter.
Steps to reproduce
started my server and ran on ios simulator
Snack, code example, screenshot, or link to a repository
there is no any native module file in ios part...
Platform

[ ] All
[ ] Android
[X] iOS

SimulatorOrDevice

[X] All
[X] Simulator
[X] Device

iOSVersion

[X] All
[ ] iOS Earlier than 8
[ ] iOS 8
[ ] iOS 9
[ ] iOS 10
[ ] iOS 11
[ ] iOS 12
[ ] iOS 13
[ ] iOS 14
[ ] iOS 15
[ ] iOS 16 or above
[ ] Android Earlier than API Level 23
[ ] Android API Level 23
[ ] Android API Level 24
[ ] Android API Level 25
[ ] Android API Level 26
[ ] Android API Level 27
[ ] Android API Level 28
[ ] Android API Level 29
[ ] Android API Level 30
[ ] Android API Level 31
[ ] Android API Level 32
[ ] Android API Level 33 or above

'npx react-native info' run result cannot be seen
The emulator version cannot be seen
Please provide more detailed development environment and details.


Residue

Description
Moved Residue class in separate residue.py file
Status

[x] Ready to go

Codecov Report

Merging #24 (2f67194) into main (0922621) will increase coverage by 0.05%.
The diff coverage is 93.50%.


Fix powershell variable assignment code snippet in Restore Process

@dlepow
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


fix #1324 Define selectedComponentProperty in the way it could be redefined

fixes #1324

Does an issue already exist for this pr? If so could you please reference the issue, if not could you please create an issue for this pr giving a brief description. Also, specific if it is a breaking change, a fix or an enhancement.
Thank you for your submission and choosing to contributing to the Augury project.

Sorry, it's my first time doing PR. I referenced the issue in the title of PR, should I mention it elsewhere?

Can you send this message in the PR you created. So we can centralize
communication to one place and keep track. Also, your doing great :) I know
doing PRs can be scary and confusing when your first starting out. I'm
learning too :)
On Thu, Oct 31, 2019, 2:22 AM Denis<EMAIL_ADDRESS>wrote:

Sorry, it's my first time doing PR. I referenced the issue in the title of
PR, should I mention it elsewhere?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rangle/augury/pull/1411?email_source=notifications&email_token=ADJ3WYET6CFR52ZUQBMUHFDQRJ2TDA5CNFSM4JHD7C5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECWV36I#issuecomment-548232697,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADJ3WYCI6SXN6PXRMTLAQRTQRJ2TDANCNFSM4JHD7C5A
.

This PR fixes #1324
Please review

@TaPO4eg3D There seems to be some conflicts blocking the merge, could you resolve these conflicts?
Thanks!

Done

@feeloor Done

Great, thanks for your contributions @TaPO4eg3D!


fix: add test coverage for rate limits with 0 permitted events

Verify that rate limits such as "0/1h", "0/24h" work as intended. This is a tests only change to ensure this behavior is preserved in the future.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 57.035%

Totals

Change from base Build<PHONE_NUMBER>7:
0.0%

Covered Lines:
9546

Relevant Lines:
16737

💛  - Coveralls


Add cpu_features option

Travis is failing on the linter (just needs some renames fwics):
virtcontainers/persist/api/config.go:58:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures string
	^
virtcontainers/pkg/annotations/annotations.go:85:2: const `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures = kataAnnotHypervisorPrefix + "cpu_features"
	^
pkg/katautils/config-settings.go:19:5: var `defaultCpuFeatures` should be `defaultCPUFeatures` (golint)
var defaultCpuFeatures = ""
    ^
pkg/katautils/config.go:95:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures             string   `toml:"cpu_features"`
	^
virtcontainers/hypervisor.go:279:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures string
	^

Travis is failing on the linter (just needs some renames fwics):
virtcontainers/persist/api/config.go:58:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures string
	^
virtcontainers/pkg/annotations/annotations.go:85:2: const `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures = kataAnnotHypervisorPrefix + "cpu_features"
	^
pkg/katautils/config-settings.go:19:5: var `defaultCpuFeatures` should be `defaultCPUFeatures` (golint)
var defaultCpuFeatures = ""
    ^
pkg/katautils/config.go:95:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures             string   `toml:"cpu_features"`
	^
virtcontainers/hypervisor.go:279:2: struct field `CpuFeatures` should be `CPUFeatures` (golint)
	CpuFeatures string
	^

OK, thanks

/test

/test

Hi guys @chavafg @jodh-intel
I've seen this same error here
go: error loading module requirements
Makefile:107: recipe for target 'go.mod' failed
make: *** [go.mod] Error 1
The command ".ci/static-checks.sh" failed and exited with 2 during .

maybe we need to update tarvis.yaml in all repos like what i did here https://github.com/kata-containers/tests/pull/2435

Hi @Pennyzct, I have made the change with #2593. @justin-he can you please rebase with latest changes?
Thanks

Hi @Pennyzct, I have made the change with #2593. @justin-he can you please rebase with latest changes?
Thanks
Ok, done, Thanks

Hi @Pennyzct, I have made the change with #2593. @justin-he can you please rebase with latest changes?
Done, thanks

Hi @Pennyzct, I have made the change with #2593. @justin-he can you please rebase with latest changes?
Thanks

Sure, done for the rebasing

/test

/re-test

travis got stuck, @justin-he please rebase this

/test-clh

travis got stuck, @justin-he please rebase this

Okay. rebased without functional changes

/test

/test


Proxy SignalR and appending access tokens

Hi! I'm using ProxyKit for building a BFF for a React SPA application. We are using Oath2/OIDC for protecting our API calls. My first attempt to use ProxyKit was just proxying the requests like in your samples and we added the Bearer token on the front end. This worked without problem. On the client we are using @aspnet/signalr as their signalR client and we supplied the token like this
 this._connection = new signalR.HubConnectionBuilder()
            .withUrl(CASE_SIGNALR_URL, {
                accessTokenFactory: () => localStorage.getItem("My Access Token")
            })
            //.configureLogging(signalR.LogLevel.Error)
            .configureLogging(signalR.LogLevel.Information) // Change to LogLevel.Error before final deployment
            .build();

One purpose of using the BFF is to handle the token there instead of in the client so we removed the
accessTokenFactory
part:
 this._connection = new signalR.HubConnectionBuilder()
            .withUrl(
                CASE_SIGNALR_URL
            )
            //.configureLogging(signalR.LogLevel.Error)
            .configureLogging(signalR.LogLevel.Information) // Change to LogLevel.Error before final deployment
            .build();

In my startup class I have this code:
            app.Map("/signalR", signalR =>
            {
                signalR.UseWebSocketProxy(
                    context => new Uri(_appSettings.ApiGatewayRootUrlWs));
                signalR.RunProxy(async context =>
                {
                    var forwardContext = context.ForwardTo(new Uri(_appSettings.ApiGatewayRootUrlHttp));
                    var token = await context.GetTokenAsync("access_token");
                    if (token == null)
                    {
                        _logger.LogError("No token available for the SignalR related request with Path: " + context.Request.Path);
                    }
                    forwardContext.UpstreamRequest.Headers.Add("Authorization", "Bearer " + token);
                    forwardContext.AddXForwardedHeaders();
                    return await forwardContext.Send();
                });
            });

It seems that we get an error when trying to negotiate the connection, the SignalR service returns a 500 insted of 101
System.Net.WebSockets.WebSocketException (0x80004005): The server returned status code '500' when status code '101' was expected.
   at System.Net.WebSockets.WebSocketHandle.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken, ClientWebSocketOptions options)
   at System.Net.WebSockets.ClientWebSocket.ConnectAsyncCore(Uri uri, CancellationToken cancellationToken)
   at ProxyKit.WebSocketProxyMiddleware.AcceptProxyWebSocketRequest(HttpContext context, Uri upstreamUri)

I understand that this may not be  enough information to help with my trouble shooting but maybe you can give my feedback if I'm doing something obviusly wrong
Best regards Hans

Hi @hoaol , reading through the snippets there isn't anything jumping out to me as to what the issue could be. To be honest, there could be any number of things going on - I'm going to need a I need a runnable project that reproduces the issue. There is a signalr chat app in the repo, perhaps adjust that to match your setup.

Hi! I just wanted to mention that I solved this.
I think it was more related to how we have implemented the security in our SignalR Services (or maybe not, I have not implemented that part my self).
In addition to attaching the Bearear Token for the http related request to the SignalR Service I also needed to add it to the wss query string, My code ended up like this:
app.Map("/signalR", signalR =>
{
	signalR.UseWebSocketProxy(
		context =>
		{
			var token = Task.Run(async () => await context.GetTokenAsync("access_token")).Result;
			context.Request.QueryString = new QueryString(context.Request.QueryString + "&access_token=" + token);
			return new Uri(_appSettings.ApiGatewayRootUrlWs);
		}
	);
	signalR.RunProxy(async context =>
	{
		var forwardContext = context.ForwardTo(new Uri(_appSettings.ApiGatewayRootUrlHttp));
		var token = await context.GetTokenAsync("access_token");
		forwardContext.UpstreamRequest.Headers.Add("Authorization", "Bearer " + token);
		forwardContext.AddXForwardedHeaders();
		return await forwardContext.Send();
	});
});

Thanks for you quick feedback and a great proxy toolkit :-)
Best regard Hans

Good to hear you solved it!
Cheers.


fix: db_url add tls=skip-verify and add tls connection

Summary
db_url add tls=skip-verify
add tls connection
Does this close any open issues?
Closes #5659
Related to https://github.com/apache/incubator-devlake-website/pull/648
Screenshots
previous:

now:

Other Information
Any other information that is important to this PR.

doc link: https://github.com/apache/incubator-devlake-website/pull/648


Bump: Longhorn to v1.7.2-rc2

Bump LH to v1.7.2-rc2.

LH team will release rc2 tomorrow, I think we can wait for one more day.

Verified with VM image creation

Uploading
Downloading
Exporting

Verified with snapshot restore operations

Creating VM snapshot from a running VM
Creating VM snapshot from a stopped VM
Restoring VM snapshot to a new VM

cc @bk201 @WebberHuang1118


Handle duplicate service path with better error

Describe the bug
If a user tries to register a service that has already been added, it will have a 500 error that looks like the following:
Duplicate key value violates unique constraint "partial_service_name"→  Detail: Key (sourcecontrol, organization, repository)=(github.com, foo, bar) already exists.

We found this throwing 500 errors on production.
To Reproduce
Since this occurs via github app calls, it is easier to recreate using the API. The only way services are added is through the release command, so this must be occurring from calling this command multiple times. However, there should already be checks for duplicate service names, so based on the error these are not working.
Expected behavior
Should be thrown as a CustomWebApplicationException, not a db error. Also should be a 400 level error since it is a duplicate registration.
┆Issue is synchronized with this Jira Bug
┆Issue Type: Bug
┆Fix Versions: Dockstore 1.9.X
┆Sprint: Sprint 41 Pufferfish
┆Issue Number: DOCK-1428

Couldn't reproduce this bug.
This error stopped appearing on production after June 24, 2020 which seems to align with 1.9.0 release. Seems to no longer be an issue.


Using IISAdministration and WebAdministration Module together can lead to Script breaking errors

I was working on a script to automate the setup of a test-environment and encoutered following problem.
Example for failing code
Get-IISAppPool
New-WebAppPool -Name TestAppPool
New-IISSite -Name TestSite -PhysicalPath D:\ -BindingInformation *:80:test

Explaination
If using Get-IISAppPool from the IISAdministration PowerShell Module
followed by New-WebAppPool from the WebAdministration PowerShell Module,
the command Get-IISAppPool will somehow prevent New-WebAppPool from closing the applicationHost.config file and probably leaves it in an open state.
If we now run New-IISSite, we will get the Error:
New-IISSite : Filename: \\?\C:\Windows\system32\inetsrv\config\applicationHost.config Error: Cannot commit configuration changes because the file has changed on disk
I was able to reproduce this with other Commands too if followed the order described above:

Get-*Cmdlet from IISAdministration Module
New-*Cmdlet from WebAdministration Module
New-IISSiteBinding from IISAdministration Module

Prevention of the issue
To workaround this issue I found two solutions.

Remove the IISAdministration Module before executing New-IISSite
Wrap Get-IISAppPool and New-WebAppPool in Start-IISCommitDelay / Stop-IISCommitDelay
Only use one of the Modules. I decided to only use the IISAdministration Module. It's not in the Microsoft Docs but you can create/delete AppPools with it too using the Get-IISServerManager cmdlet as described here.

Workaround 1
Get-IISAppPool
New-WebAppPool -Name TestAppPool
Remove-Module IISAdministration
New-IISSite -Name TestSite -PhysicalPath D:\ -BindingInformation *:80:test

Workaround 2
Start-IISCommitDelay
Get-IISAppPool
New-WebAppPool -Name TestAppPool
Stop-IISCommitDelay
New-IISSite -Name TestSite -PhysicalPath D:\ -BindingInformation *:80:test

This repository is for PowerShell core documentation. You seem to be looking for support, which we can't provide here.
I suggest you try posting your issue with context in one of the available community support forums.


bump to v0.2.0

Codecov Report

Merging #403 (1275499) into main (958cc77) will decrease coverage by 5.97%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #403      +/-   ##
==========================================
- Coverage   90.80%   84.83%   -5.98%
==========================================
  Files          62       62
  Lines        3751     3751
  Branches      718      718
==========================================
- Hits         3406     3182     -224
- Misses        182      409     +227
+ Partials      163      160       -3

Flag
Coverage Δ

unittests
84.83% <ø> (-5.90%)
:arrow_down:

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

Impacted Files
Coverage Δ

...mmingbird/ml/operator_converters/sparkml/linear.py
42.85% <0.00%> (-57.15%)
:arrow_down:

...bird/ml/operator_converters/sparkml/discretizer.py
47.36% <0.00%> (-42.11%)
:arrow_down:

hummingbird/ml/_container.py
70.25% <0.00%> (-18.54%)
:arrow_down:

hummingbird/ml/convert.py
66.82% <0.00%> (-16.83%)
:arrow_down:

hummingbird/ml/_topology.py
70.07% <0.00%> (-14.79%)
:arrow_down:

hummingbird/ml/_parse.py
75.78% <0.00%> (-10.07%)
:arrow_down:

...ml/operator_converters/sparkml/vector_assembler.py
90.00% <0.00%> (-10.00%)
:arrow_down:

...l/operator_converters/_pipeline_implementations.py
60.00% <0.00%> (-10.00%)
:arrow_down:

hummingbird/ml/_utils.py
83.33% <0.00%> (-4.55%)
:arrow_down:

hummingbird/ml/supported.py
87.25% <0.00%> (-3.93%)
:arrow_down:

Continue to review full report at Codecov.

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


u-boot: fix possible freeze in uart driver

This change is already present for the majority
of the Pi boards, let's add it for the Pi4 rebased
patch too.

@balena-ci rebase


Error in xll function when using Math.NET

I teach a graduate level math-finance course where we cover both VBA and VB.NET.
In an attempt to demonstrate how to build an xll that contains a user-defined function utilizing the bisection method in Math.NET, when called from Excel, the result shown is #VALUE.
In previous sections of the course, we successfully covered Math.NET in the context of VB.NET alone, and we also discussed building xll's in VB.NET alone -- no external libraries -- that can be called from Excel.  It is when putting these together that I seem to run into trouble.
As for the technical details:

The version of VB.NET is 3.7.0 (Visual Studio 2019)
The Math.NET library is installed using NuGet
We use Excel-DNA to generate the xll (also installed from NuGet)

The repro code is as follows:
First, write a function for which we wish to find a root using the bisection method:
Public Module RealModule
    <ExcelFunction(Description:="Cubic Polynomial")>
    Public Function Cubic(x As Double, a As Double, b As Double,
               c As Double, d As Double) As Double

        Cubic = ((a * x + b) * x + c) * x + d

    End Function

Next, write a function that will calculate the nearest root using the Math.NET bisection method:
   <ExcelFunction(Description:="Root of Cubic Polynomial")>
    Public Function CubicPolyRoot(a As Double, b As Double,
               c As Double, d As Double, xLow As Double, xHigh As Double)

        Dim CubicFcn As Func(Of Double, Double) =
            Function(x) (Cubic(x, a, b, c, d))

        If xLow < xHigh And CubicFcn(xLow) * CubicFcn(xHigh) < 0 Then
            Dim root As Double
            root = Bisection.FindRoot(CubicFcn, xLow, xHigh)
            CubicPolyRoot = root
        Else
            CubicPolyRoot = Double.NaN
        End If

    End Function

End Module

When the function CubicPolyRoot is called from Excel, the #VALUE error is returned in the calling cell.  This code does run successfully in a separate VB.NET executable, independent from Excel, however.
By removing the Math.NET bisection method function, however, and replacing the return value with a constant, the function can be called successfully from Excel.
My questions are:

Should this work?
If not, could you recommend a reasonable alternative?
In particular, would I need to download the Math.NET source code and compile my code with it into an xll?

Any help would be greatly appreciated.  This would be a very nifty tool for my students to know before they go out into the computational finance workforce.
Thank you very much, in advance.

Ended up solving this...a bit embarrassing, but I forgot to include the MathNet.Numerics.dll in the same trusted location as the compiled xll file.
In summary, for anyone else wanting to write xll's with Math.NET functions, and the Excel-DNA package (highly recommended) to compile VB.NET code to an xll, the procedure is:

Create a VB.NET Class Library (.NET Framework) project in Visual Studio.
Use the NuGet package manager to install the Excel-DNA and Math.NET libraries in your project.
Write your code and compile it.
Copy the ProjectName-AddIn-packed.xll file (or ProjectName-AddIn64-packed.xll file for 64-bit versions of Excel) that is generated by Excel-DNA in the bin/release folder for your Visual Studio project into the trusted location for your Excel add-ins.
Copy the MathNet.Numerics.dll from the packages/MathNet.Numerics.4.12.0/lib/net461 under your Visual Studio project director (this is installed by NuGet in step 2) into the trusted location for your Excel add-ins as well.
Open Excel and install your xll Add-in in the usual way.
Call the function(s) as you would any other cell formula in Excel.

The same should work for a C# xll.


Consider using an application registration routine

In order to handle configuration that may need to be passed to the application server prior to invoking the application, I am contemplating the addition of a special, optional subroutine that may be used to configure the application:
sub app(%env) { ... }
sub register { &app, %options }

Whether a P6SGI script is returning an application or a register function is determined based on the arity of the routine. If the arity is 0, it's the registration method. If the arity is 1, it's an app. (There might be a better detection mechanism.)
The register routine returns a 2-element capture/list. The first element is the application subroutine and the second is the configuration for that subroutine.
Options I've thought of so far may include the following (but could really be extended like the environment for per-server extensions and such). Honestly, this is the sort of thing that might explode into lots of options.
'p6sgi.protocols' => ('HTTP/1.0', 'HTTP/1.1', 'HTTP/2', 'WebSocket/13'),
'p6sgix.ws.subprotocols => ('xmpp'),

As I see it, the protocols option lists the protocols the application expects to support. The server has to support at least one of the options or you have an error. There would need to be a matching p6sgi.protocol variable in the environment to inform the application which protocol has been negotiated by the server.
The subprotocols extension is a WebSocket specific setting, which tells the server that it should allow the named subprotocols in the Sec-WebSocket-Protocol header during ordered by preference. Again, this would need something liek p6sgix.ws.subprotocol to inform the application which actual subprotocol was selected.
The downside to this, which I still want some thought put into is the fact that we might want to support certain protocols sometimes and not others. This just turns it on for all, so that's not right yet, but to make multi-protocol handling not totally suck, something like this may be needed.

Let's not do this. I don't think this sort of communication is needed and if it is, it's going to be an administrative detail rather than an application detail.


Run Windows tests on Travis CI

Travis CI now has support for Windows. Given that Appveyor is painfully slow at the moment, we should probably try to replace Appveyor with Travis CI at some point. This allows us to use a unified test setup for all environments (Windows, Linux, OSX).
Related: https://github.com/TuringLang/Turing.jl/issues/530

@TuringBenchBot bm("issue-725")
// just for test.

Hi Sir, a new benchmark job will be dispatched soon at your command,
you can track it here: https://github.com/TuringLang/TuringBenchmarks/issues/25.

Hi Sir, a new benchmark job will be dispatched soon at your command,
you can track it here: https://github.com/TuringLang/TuringBenchmarks/issues/26.

Closed in favour of #823


Add type annotations to GroupBy objects

#type: ignore's in b20a3b1 are mostly because mypy cannot identify _attributes of self.

Codecov Report

Merging #1889 into master will decrease coverage by 2.84%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1889      +/-   ##
==========================================
- Coverage   94.19%   91.34%   -2.85%
==========================================
  Files          40       40
  Lines        9939     9858      -81
==========================================
- Hits         9362     9005     -357
- Misses        577      853     +276

Impacted Files
Coverage Δ

databricks/koalas/frame.py
93.40% <100.00%> (-3.33%)
:arrow_down:

databricks/koalas/groupby.py
90.96% <100.00%> (-0.23%)
:arrow_down:

databricks/koalas/series.py
95.68% <100.00%> (-1.29%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
25.66% <0.00%> (-66.65%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
47.82% <0.00%> (-52.18%)
:arrow_down:

databricks/koalas/__init__.py
76.66% <0.00%> (-13.34%)
:arrow_down:

databricks/conftest.py
88.13% <0.00%> (-11.87%)
:arrow_down:

databricks/koalas/spark/functions.py
88.88% <0.00%> (-7.41%)
:arrow_down:

databricks/koalas/accessors.py
86.06% <0.00%> (-6.97%)
:arrow_down:

databricks/koalas/namespace.py
78.45% <0.00%> (-4.85%)
:arrow_down:

... and 16 more

Continue to review full report at Codecov.

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


Added the prettify to the api endpoints ?pretty=true

Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] Optimization
[x] Documentation Update

Description

prettify the api response by passing the query ?pretty=true
/api/jobs?pretty=true
/api/jenkins?pretty=true
/api/airflow?pretty=true

Related Tickets & Documents

Related Issue #
Closes #

Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.

Testing

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

@chentex ptal 👋

@athiruma can you please rebase?


Having trouble with the Mad Lib

Challenge Name

Issue Description

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

@sharonwischkaemper if you get stuck with a challenge, please ask questions to the Help Room. There are many people there willing to give hints and help on these challenges. Good luck and happy coding!


[WIP] Mc 1.17 mixin -> asm

Moved to new repository https://github.com/OpenCubicChunks/CubicChunks2/pull/24


NSFW Project

Hi @sswangg
I am a CTO at WALTB and we are building video chatting application.
In our application, we need to implement NSFW processing and I need your help.
I tried yahoo NSFW engine and it's a little slow than we need.
We need to process image within 30 ~ 50 ms but yahoo engine takes 70 ~ 80ms.
So I tried to find alternative solution and I am reaching out to you.
Can we schedule a call and discuss the project?
My email is<EMAIL_ADDRESS>My github handle is dreamsoftech
Thanks,
Andrew

Go away ew person illega;l distribution minor stuff idk


[defense in depth] Ensure file permissions on auth cache

It's possible in some cases that an already existing file would not have permissions intended in os.WriteFile
https://github.com/Azure/azure-dev/blob/main/cli/azd/pkg/auth/file_cache.go#L57
A defense-in-depth fix here would be to ensure that the file permissions are as intended after the file closes. Perhaps a defer os.Chmod

Discussed in standup. Not a must-fix for GA. Moving to Backlog.


Intel Fix: Soviet ar Hero [garrison][MOnDo53]

Name
Soviet ar Hero
Intel Id
MOnDo53
Fix Needed
the name of the intel should be Soviet War Hero, you have accidentally left off the W in war, lol
Any Other Info
this is such an awesome and wonderful tool to have, thank you so much for making it.

Thanks for reporting and the nice comments @TheMobiusAce.
Should be fixed for you now 👍


Animation not working with 1.4.x and ui-router: ng-hide-add and ng-hide-remove classes are not generated

My issue seems similar with https://github.com/angular/angular.js/issues/12267, I left a comment there but since that issue is closed, people might not read it.
@matsko worked on that issue.
ng-hide-add and ng-hide-remove classes are not generated. I am not using Material.
Debugging ngAnimate I noticed that all animations are skipped because:

function 'areAnimationsAllowed' always returns false because:

rootElementDetected is always false because:

$rootElement var points to the element before the actual root element:

The element $rootElement points to is a comment  and it is inserted by the ui-view directive which is present on the angular root element.
"ui-view" is a directive belonging to https://github.com/angular-ui/ui-router.
When ngAnimate is initialized $rootElement receives the correct value but probably when ui-router inserts its comment right before the angular root element, $rootElement is not updated. So when the animation happens, $rootElement points to  instead of
Because 'areAnimationsAllowed' function does not find the angular root element, it retursn false and all animations are skipped.

Is $rootElement always bound to a comment node when ui-router is used? Does this mean that all animations are broken when it is used? Will the comment node be placed as a sibling always to the element that is originally the rootElement?

Look at this example we have an app where the rootElement is on <html>:
https://angular-ui.github.io/ui-router/sample/#/

Right click somewhere on the page and select inspect element.
Goto console and run this angular.element($0).injector().get('$rootElement')

See how it's the HTML element?
How did your code end up being situated to where the view was created? Could you share your some code to show how your app is configured?

I'll try to create a plunker and share it with you

starting to work on plunker, but until I have it I can answer you question: in our code the ng-app directive is not on the html element but on the div element that has the ui-view directive.

Here you have the plunker: http://plnkr.co/edit/uB94enS5J2AgI0fPiePM?p=preview
Put the following two breakpoints in angular-animate.js:

breakpoint B1: on line 1939: var activeAnimationsLookup = new $$HashMap();
breakpoint B2: on line 2392: var bodyElementDetected = false; (first line in function areAnimationsAllowed(element, parentElement, event))

Refresh the page

breakpoint B1 will be hit and in console you can see that $rootElement is "[<div ng-app="myapp" ng-controller="MainCtrl" ui-view>…]" = correct

Click on the "Test Animation" button

breakpoint B2 will be hit and in console you can see that $rootElement is "[]" = incorrect
F8 in Developer Tools to continue and you will see that the animation "sort of" happens

"sort of happens" - because the height doesn't start at zero

Disable breakpoints in Developer Tools and keep clicking on the "Test Animation" button. you will see that the animation will not happen anymore

This behavior is a bit different than what I see in our app. I our app the animation doesn't happen even when debugging. I think that here it works when debugging because of the plunker environment.

Moving the ui-view directive from the ng-app element to its own element workarounds the ngAnimate issue. The  comment is inserted now in front of the  element and  is not affected. The animation works now.
Using this

Instead of this

@ndaian  wait how does the workaround work, i have exactly the same problem, but didnt get it to work.

Same problem here on angular 1.4.4, any UI View animations just stopped working, pretty sure this is not an ngAnimate issue but a uiRouter issue. We can probably move this to their issues with the same plunk if @ndian agrees.

@ndaian would you mind testing against these files to see if things work:
http://www.angular-dev.com/raf-fix/angular.js
http://www.angular-dev.com/raf-fix/angular-animate.js

I've just had a similar issue however the problem was with the following row:
bodyElementDetected = isMatchingElement(parentElement, $$body);
It is always false because the ui-view directive was assigned to the  element and in the angular-animate code the $$body variable is initialized before ui-router populate the view. so it is never equal.
Just want to put it out here for anyone else who might have this problem.

Is this still an issue? In 1.4.7 there was a fix to the $$body service.

I am using the latest AngularJS 1.5.0beta + UIRouter 1.4.7. The issue doesn't repro for me.

@ndaian what about you?

i started having this issue as well. transitions between states are not adding classes from nganimate. any updates on progress?

This bug still repros in this plnkr with the current snapshot: https://github.com/angular/angular.js/issues/12583#issuecomment-131942396

having the same problem, but on ng-switch-when. using 1.4.8

I had a look at this and it seems this happens because ngView / uiView are transclude: element directives, which means they replace the element on which they are with a comment node. But I'm not yet sure how the $rootElement value is actually replaced by the comment node.


evaluate shelve module for Stack object.

shelve is dict-like db that writes directly to disk. This could potentially outperform our current defaultdict approach for the Stack container class (always in-memory, huge cPickle pickle and unpickle times, compression costs, etc.).

Too slow as well.


[#269] Removed unecessary absolute positions,  aligned and resized el…

…ements

@Omoorion Looks like you have merge conflicts 😆

@Omoorion Did you bother testing your changes?

@Omoorion Looks like you have merge conflicts 😆

RIP 🫡

I fixed them 😈

@Omoorion Did you bother testing your changes?

What does brother mean

@Omoorion Did you bother testing your changes?

What does brother mean

It looks completely different from the current UI and for the worse

Personally I think it looks better, what's bad about it?

How does this mess look better to you

How does this mess look better to you

The only problem I see is the missing margin on the top-left side. Please use master branch as a reference.

@Omoorion same margin problem with the top side of the logs container

Closing due to inactivity @Omoorion.
Should you wish to revisit this in the future, please feel free to reopen or submit a new PR.


[WIP] Finish the count/sizes/num operation and the flatten operation

I'm going to leave this open to do flatten as well. It's coupled: some of the tests had to be disabled because flatten depended on count64, and now those functions are gone. I don't want to put skipped tests into master.

See https://github.com/scikit-hep/awkward-1.0/issues/117#issuecomment-596047227 for an explanation of why "num" is a good name (e.g. ak.num(muons) >= 2).

nicely done!


ARTEMIS-2702 QuorumVoteServerConnect with requestToStayLive is voting order sensitive

I still need to validate the behavior at an higher level of abstraction ie

ServerConnectVoteHandler unit test
with an end 2 end test

And run the whole test suite to see if this isn't causing regressions (especially on QuorumVoteServerConnect::await that doesn't have any test coverage and I'm not sure of the original semantic. Probably it can create issues on the live when vote-on-replication-failure is set and we're awaiting the votes to complete.

I've sent a second commit that (assuming that I've correctly understood the original logic) simplify the current logic. @clebertsuconic @jbertram @andytaylor please take a look at the 2 commits separately, if I've misunderstood how QuorumVoteServerConnect is supposed to work...
In addition: I see that we should check for targetNodeId vs ServerConnectVote::getNodeId on QuorumVoteServerConnect::vote too (probably).

I've decided to merge the 2 commits into one, hardening the expected behaviour after solving some test failures.

I need to take care of a couple of log messages on the test and It seems ok (looking at the CI test too)


changed query output variable to projection canonical

Pull Request Test Coverage Report for Build 560

36 of 36 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 82.747%

Files with Coverage Reduction
New Missed Lines
%

lib/ast/function_def.js
1
90.82%

Totals

Change from base Build 559:
0.01%

Covered Lines:
6615

Relevant Lines:
7777

💛  - Coveralls


What is blocking a new release?

In this commit you mention something about waiting on an upstream issue with async-graphql. Does that remain an issue?

I can't fully remember but looking at the dates it would be this which looks to have been fixed. I should have linked to the issue I referred to in the commit :facepalm:.
I will look into doing a new release of this library as soon as I can find some time.

Imma just keep this issue open until I finish it as a reminder.

Any updates on this? I was thinking of using async-graphql-relay since my frontend is in Relay. The current version works I'm guessing?

I have just done a quick dependency update and publish.
I am not actively using this project any more so I can't commit to continue maintaining it but would be happy to merge and publish PR's if changes are needed.
I would be happy to hand over to another maintainer but they would need to prove themselves before I am willing to do that.

Thanks, does it still work fine or are there any breaking or showstopping issues?


git-log page formatting problem

In the git-log page, the text within the section "History Simplification" becomes an orange code listing which continues until halfway into the section "Diff Formatting."

This has been fixed for some time. Asciidoctor is stricter about block delimiters: they have to be ±1, not ±2.  I submitted a patch to Git which fixes those issues, and the website has been automatically updated.

Looks like this has been resolved so I will close this issue. Let us know if that is not the case :)


Reduce the risk of id clashes

When rendering over 10 charts on a page, I got a periodic clash in the IDs.  This results in the wrong data being loaded into one chart and the other chart not to render.  I have patched this locally to use the logic:
1 . Get the md5 hash of the seed string (in this case, the inner content of the expandable)
2. Split that into an array
3. Shuffle the array
4. Convert the array back into a string
This gives you a unique id like 584abfca542dc4dbc81eccfcd6e4fd8c.  This is much less likely to clash (but not impossible).
I hope this is useful to someone else.

@andyflatt awesome !❤


Feature Request: Include sum of deployment gas in output

I'm routinely "gas golfing" large projects, and one thing I always need to do is dump out the gas report and manually sum it to get a total gas cost for every deployed contract to check if the experiment is reducing overall cost or not.
I can emit to a file and use regex to do this, but it would be nice to include a configuration switch that adds the total deployment gas usage to the output itself.

Indeeed, it would be a great addition!!
Have you found a workaround? besides scripting?

@andreujuanc No workaround besides custom scripting, unfortunately.
I wrote a hardhat task for my project that:

uses a canned hardhat-gas-reporter console output saved to /scripts/lastOutput.txt
builds a new table showing cost per category and the estimated gas cost at n gwei similar to this: https://github.com/wattsyart/cryptoadz-chained/wiki/Gas-Cost-Report

Code is here: https://github.com/wattsyart/cryptoadz-chained/blob/main/tasks/tasks.js#L171


Add a hover icon for external links.

This shows when links go to a different server.

Here is a screenshot when the icon shows. (By default it's not visible, and only appears when hovering over links).


Filter by Tag and Input combined?

HI, awesome work. Thank you!
Just a quick question: can I combine filter by tag and by input on content?
I tried to do this using your codepen examples.  See here-> https://codepen.io/andreasdirscherl/pen/NWwqPVy
But it only works partially: if I input "cat"  or "dog" it works, all other text inputs do not.
So I'm wondering if it could work somehow?
Thank you!

Hi Andreas,
the examples were out of date. I have already fixed the bug with version 0.3 (https://cdn.jsdelivr.net/gh/GianlucaChiarani/AutoFilter@0.3/autofilter.js).
Thank you,
Gianluca

Thank you for your quick response and fixing the code! Great. It now does find 'monkeys', too :-)
Am I right in assuming that when I combine the data-tag and data-to-filter then the text input is limited to the keywords specified in data-tags? Because that's what's happening now for me. When data-to-filter is used alone, it finds all the text between the html tags.
So keeping with the example: When I input "some" or "pretty" nothing is found.
Just wanted to make sure I follow your logic... Thank you!

Exactly: data-tags has priority over data-to-filter. Html content and data-tags' value are not merged together.

OK, good to know. Thanks!

Sorry to bother you again, but I would one more question.
Based on your response, I played around a little more and found an interesting "feature" in your bug. Basically, with V0.2 I can achieve 100% what I was looking for, namely using the buttons and the input fields simultaneously. With the fixed V0.3 it doesn't work anymore.
But since I am taking advantage of that "bug" I wanted to ask you if you see any unwanted side effects or problems with me using it?
This is the html, and you can see it "in action" on my codepen

  All
  Cats
  Pretty
  Dogs
  Monkeys

  <input type="text" placeholder="Search..." data-filter />

Some pretty cats and dogs!

Some pretty Dogs

Cats and Monkeys!

Only Monkeys!

To get the same result in v. 0.3, it is necessary to set subString: true

Of course, duh!
Fantastic - works like a charm now and is EXACTLY what I was looking for.
THANK YOU! 🙏


Remove incorrect note about empty "password key"

Removing this phrase from example 1:

Note that after the second command is complete, the password key in the $UserCredential variable becomes empty.

It's just... wrong.
Nothing about the $UserCredential changes and I wouldn't expect it to (so one can connect to multiple services with one credential object, say SharePoint Online PnP as well as EXO)

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

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

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Connect-ExchangeOnline.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

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

Docs Build status updates of commit 8a03e03:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Connect-ExchangeOnline.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


first step, yarn, fails with no scenarios

Hello,
Running the first command, yarn fails:
~/source/dmsrs$ yarn                                                                                1 ↵ master
00h00m00s 0/0: : ERROR: There are no scenarios; must have at least one.

Thanks!

Running Ubuntu 22.04
$ uname -a
Linux modnar 5.15.0-84-generic #93-Ubuntu SMP Tue Sep 5 17:16:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

$ yarn --version                        master
0.32+git

It seems like you're not using the right yarn. If you don't like YARN, you can still use NPM and it should work in the same way (:


New package: ModularIndices v0.1.0

Registering package: ModularIndices
Repository: https://github.com/ericphanson/ModularIndices.jl
Created by: @ericphanson
Version: v0.1.0
Commit: 00d44f43b86cb6d66402dd683858e9aec966a688
Reviewed by: @ericphanson
Reference: https://github.com/ericphanson/ModularIndices.jl/issues/1#issuecomment-514137062

This is the fixed-spelling version of https://github.com/JuliaRegistries/General/pull/2143; I updated it to fix a docstring but it's now good to go from my side.


Log ContinuationToken for pkRanges request

Log continuationToken for PkRanges request.

Our team will look into this in the future.


Direct link to query

First off this is very cool! It'd be great to be able to share a link to a particular query, is this in the works?

Yes, this is definitely something we want to have! Thanks for the reminder - I'll keep the task open to track progress.

This is now enabled thanks to:
38a5fbc3d8cae4c8ee672238a3402e2327edac91
6cb4ab1dfe3d7a6863ea60d3f6a6c3568e2f8cd5
8905e649baa97de3a99ecaf20d4c07ab7800b687
7edd606658981e55ee5722e65c0005e233c00e06

awesome that's some quick work! :100:


SOLR-16442: upgrade to Lucene 9.4.2

https://issues.apache.org/jira/browse/SOLR-16442

@cpoerschke anything more to do here?


Installation docs incomplete

Unless I'm missing something, there seems to be a bunch of stuff missing from the installation documentation.
The process I followed up to the installation docs was:

create a virtualenv, db
pip install django, psycopg2, django-blog-zinna
create project, add db details to settings...start install process:

as I noted yesterday in #408, there are some minor issues with the tagging software that will be easy to fix.
Sync doesn't work out of the box, needed to explicitly sync zinna first, then in general. Potentially there should be a db sync before zinna is added to the INSTALLED_APPS setting?
settings.py no longer comes with SITE_ID - needed to add it, and because it's not really used any more, I needed to shell in and fix the site's details (from example.com to domain.com)

Thanks for your report but like explained in the documentation I assume that you already have an installed Django project.
For the other points:

it will be fixed
weird it need some test
it will be precised in the documentation

Regards


Update various published content pages.

These now use the "top picture block" item format like much of the other content pages.

 MEDIUM URGENCY: Switch Webinars, White Papers and Podcast landing pages back to original design.


there is no getrusage64

I'm not a linux expert but according to http://nadeausoftware.com/articles/2012/03/c_c_tip_how_measure_cpu_time_benchmarking#getrusage, there used to be a getrusage64 on some OSes, but linux seems not on that list.

Auto-merge toggled on


Add support for the new IOT2050-SM variant and some improvements

Followed up on https://github.com/eclipse/mraa/pull/1102.

iot2050: Add support for the new IOT2050-SM variant
IOT2050-SM board was added into IOT2050 series, in contrast to existing
variants, the IOT2050-SM board comes without a Arduino interfaces.

iot2050: add debugfs pinmux support
This patch adds support for multiplexing pins via debugfs rather
than access memory mapped pad-configuration registers.
The debugfs pinmux offers the possbility to run mraa on iot2050
platforms as regular user instead of root by adjusting privileges
on debugfs files.
Bias settings are currently also configured by accessing pinmux.
Unfortunatelly a proper upstream-like pinconf usage is currently
not possible.
Note: In case debugfs mux fails MRAA falls back to mmap mux.

common: increase pin name size
Some pin names are longer than 12 characters.
32 characters should be enough while consuming not too
much space.

@huaqianli: Some fixes where pushed meanwhile here regarding debugfs due to device tree changes of the iot2050.
There are still issues related to non-root access though, since these changes are still relying on some sysfs functionalities.


Can getUserInfo provide Cover picture?

I'm thinking of a ui where I'd also like to show user's cover picture? But I don't want to resort to some other api that requires access tokens.
Is is possible to get cover picture url?

I'd imagine this will be possible, but will require some investigation.  If you submit a PR with this change, I'd definitely consider it.  @bsansouci

Thanks for the reply. I'll fork the project and check if I can figure out.


ReturnAssignmentFixer with PHPDoc @var for the variable

We have to do something with such annotation as phpdoc_to_comment would want to change it if it wasn't run before - which means of course that in practice it will happen after flushing cache.

Not sure if the fixer should remove the doc.. have to think about it. For now it is an overseen priority issue of sure.

IMO that code should be kept as-is, psalm will complain with the totallyTyped option otherwise

when $bar is removed I expected leaving  /** @var ValueFromSomeService $bar */ will also trigger a warning as it documents the type of a variable that does not exists?

Commenting for backreference: this is case 14 of #3844 (and I would like to see this fixed 😉)


Refresh auto-generated docs for 2.1

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/bec5703c36420f0e7fd03f5c7684776b33554c46/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/50d765b5ab3767a80c107087f801056f5153c7e1/


Can't install using NPM

Install using npm
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>This version of tar is no longer supported, and will not receive security updates. Please upgrade asap.
npm ERR! code 1
npm ERR! path /Users/iamyuu/.volta/tmp/image/packages/.tmpM4vDM7/lib/node_modules/@ayoisaiah/focus
npm ERR! command failed
npm ERR! command sh -c go-npm install
npm ERR! Downloading from URL: https://github.com/ayoisaiah/focus/releases/download/v1.2.0/focus_1.2.0_darwin_amd64.tar.gz
npm ERR! Downloaded binary does not contain the binary specified in configuration - focus

npm ERR!      /Users/iamyuu/.npm/_logs/2021-12-09T03_49_54_338Z-debug.log

Install using yarn
yarn global v1.22.17
[1/4] 🔍  Resolving packages...
warning @ayoisaiah/focus > go-npm ><EMAIL_ADDRESS>This version of tar is no longer supported, and will not receive security updates. Please upgrade asap.
warning @ayoisaiah/focus > go-npm ><EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
warning @ayoisaiah/focus > go-npm > request ><EMAIL_ADDRESS>this library is no longer supported
warning @ayoisaiah/focus > go-npm > request ><EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 🔨  Building fresh packages...
error /Users/iamyuu/.volta/tmp/image/packages/.tmpLPlMaF/lib/node_modules/@ayoisaiah/focus: Command failed.
Exit code: 1
Command: go-npm install
Arguments:
Directory: /Users/iamyuu/.volta/tmp/image/packages/.tmpLPlMaF/lib/node_modules/@ayoisaiah/focus
Output:
Downloading from URL: https://github.com/ayoisaiah/focus/releases/download/v1.2.0/focus_1.2.0_darwin_amd64.tar.gz
Downloaded binary does not contain the binary specified in configuration - focus
info Visit https://yarnpkg.com/en/docs/cli/global for documentation about this command.

npx envinfo --system --binaries:
System:
  OS: macOS 11.6
  CPU: (4) x64 Intel(R) Core(TM) i5-7360U CPU @ 2.30GHz
  Memory: 32.13 MB / 8.00 GB
  Shell: 5.8 - /bin/zsh
Binaries:
  Node: 16.13.0 - ~/.volta/tools/image/node/16.13.0/bin/node
  Yarn: 1.22.17 - ~/.volta/tools/image/yarn/1.22.17/bin/yarn
  npm: 8.1.1 - ~/.volta/tools/image/npm/8.1.1/bin/npm
  Watchman: 2<IP_ADDRESS> - /usr/local/bin/watchman

Sorry about this. I will make a new release tomorrow that should solve this

I have the same issue. Any updates?


apple add _NSGetExecutablePath

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 2317a64b1b8bf773859f3c694db8b6213bd8208c has been approved by JohnTitor

:hourglass: Testing commit 2317a64b1b8bf773859f3c694db8b6213bd8208c with merge e3148d9187921f36d95bba73c845b75860bbcdf5...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: JohnTitor
Pushing e3148d9187921f36d95bba73c845b75860bbcdf5 to master...


Error on rename when 'nw install' on Windows 10 64bit

C:\Users\tim\Documents>nw install 0.16.1
Downloading 0.16.1.zip
[============================================>] 100.0% of 51.75 MB (2.01 MB/s)
shell.js: internal error
Error: EPERM: operation not permitted, rename 'C:\Users\tim\.nwjs\nwjs-v0.16.1-win-x64' -> 'C:\Users\tim\.nwjs\0.16.1'
    at Error (native)
    at Object.fs.renameSync (fs.js:809:18)
    at C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\node_modules\shelljs\src\mv.js:79:8
    at Array.forEach (native)
    at _mv (C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\node_modules\shelljs\src\mv.js:55:11)
    at C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\node_modules\shelljs\src\common.js:240:23
    at Command.<anonymous> (C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\lib\install.js:40:5)
    at next (native)
    at onFulfilled (C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\node_modules\co\index.js:65:19)

I've tried messing around deleting the directory or opening up permissions with no luck. In the end I renamed the directory myself to get it working...

For some reason nw install 0.16.1-sdk didn't have this issue...

It happened to me.
I think this occur randomly..

Error: EPERM: operation not permitted, rename 'C:\Users\tim\.nwjs\nwjs-v0.16.1-win-x64' -> 'C:\Users\tim\.nwjs\0.16.1'
    at Error (native)
    at Object.fs.renameSync (fs.js:809:18)
    at C:\Users\tim\AppData\Roaming\npm\node_modules\nwjs\node_modules\shelljs\src\mv.js:79:8

Seems like it's an error caused by shelljs.mv.
And I found the newest version of mv includes a try/catch on fs.renameSync:
https://github.com/shelljs/shelljs/blob/master/src/mv.js#L82-L91

Looks like it might fix it. I still get this every time I do an install. Luckily I dont do that very often!


Memory leak issue on UploadHandlerRaw

There's a memory leak issue because the UnityWebRequest's UploadHandler is not disposed after use. I reproduce it by running a few completions on the example scene.
UnityWebRequest objects might also need to be disposed (or wrapped in a using statement) but I haven't run into an issue with those

I approved and merged your fix. Thanks!


How do I send commands to the Dedicated server?

How do I send commands to the Dedicated server? If you can't yet, that should be added at some point.

The github repo is for submitting bugs, for help I suggest using the Northstar discord.