Add feature gate support for SecureAccess to host agent

What this PR does / why we need it:
Adds feature-gate support for SecureAccess to host agent. Additionally,

Offloads the agent registration process to the proposed ByoAdmission Controller when the SecureAccess flag is active.
Updates the hostReconciler to delay network interface setup until host registration.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #428

One of the tasks for this PR is to prevent host registration if SecureAccess is enabled and instead depend on the ByoAdmission Controller to perform the registration. However, part of the current registration process also involves setting up the network interfaces:
https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/blob/cc9ae9bda980f2a119b494e4caf518b475151186/agent/registration/host_registrar.go#L67
This in turn is required in thebootstrapK8sNode process as part of reconcileNormal:
https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/blob/cc9ae9bda980f2a119b494e4caf518b475151186/agent/main.go#L180-L191
https://github.com/vmware-tanzu/cluster-api-provider-bringyourownhost/blob/cc9ae9bda980f2a119b494e4caf518b475151186/agent/reconciler/host_reconciler.go#L247-L254
Should we refactor the registration process and separate the network interface setup? @anusha94 @dharmjit wdyt?
cc @sachinkumarsingh092

Codecov Report

Merging #429 (1b3334c) into main (ecdbe20) will decrease coverage by 0.39%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
- Coverage   67.97%   67.58%   -0.40%
==========================================
  Files          23       23
  Lines        1705     1715      +10
==========================================
  Hits         1159     1159
- Misses        474      484      +10
  Partials       72       72

Impacted Files
Coverage Δ

agent/main.go
17.02% <0.00%> (-1.30%)
:arrow_down:

lgtm

lgtm


Override folds

The syntax/cs.vim file contains an autocmd and function to set 'foldtext'. This PR wraps this functionality with an option so it can be disabled in vimrc, allowing users to use their own custom foldtext.
The new option defaults to g:OmniSharp_set_foldtext = 1 so the current behaviour remains consistent.

This plugin contains many bad code due to historical circumstances...
I am thinking the syntax file should be split out from this repo.  (Use Vim's bundled version or create a new syntax plugin)
It's OK to merge this PR now, but we can decide removing this file now too.
How do you think?  Do you need this syntax file?

Thanks @thinca, I think that's a good idea. Diffing the syntax/cs.vim with the cs.vim that ships with vim shows that the version from this plugin has some additions, but not many. I agree that the syntax file could be split to a different plugin.
I'll close this PR and make a new one removing syntax/cs.vim.


Use DynamicJson for CLU tests

This splits the CLU test updates out of #35202 so we can merge immediately. We may be shipping a new version of CLU and/or QA soon, so updating samples to use DynamicJson right now may not align with its own projected GA.

API change check
API changes are not detected in this pull request.


Reevaluate CircuitWatcher

What do do when the sender is latched and it's not durable? Throw at the point of sending? Move the Envelope.QuickSendAsync() logic into ISendingAgent? The variability is there.

I'm judging it to be fine now, especially after the SendingAgent changes early this year.


run makers constantly after location list opened

Expected behavior
Run makers according to my configuration.
Steps to reproduce
After updating neomake I started noticing this strange behavior where the following events happen (this didn't happen prior to the update):

I have neomake to run only when I write the file: call neomake#configure#automake('w'). I can confirm the configuration with NeomakeInfo:

g:neomake: {
  'automake': {
    'ignore_filetypes': [
      'startify',
    ],
    'events': {
      'BufWritePost': {
        'delay': 0,
      },
    },
  },
}

When I start neovim everything works as expected, I can change the buffer, save, etc. and the makers are triggered on save only. I can verify on the logs that the makers are triggered only on BufWritePost. However, once the makers trigger the location list, neomake starts running the makers on TextChanged as well. I can verify that on the logs with the following line:

10:35:25 [D      ] [-.-:1:1] automake: Buffer was changed (TextChanged), restarting make: 3.

Output from (verbose) NeomakeInfo
Neomake debug information
Async support: 1
Current filetype: python
Windows: 0
[shell, shellcmdflag, shellslash]:['/bin/zsh', '-c', 0]
Enabled makers
For the current filetype ("python", used with :Neomake):

python

args: ['/Users/dbarroso/.cache/nvim/plugged/neomake/autoload/neomake/makers/ft/python/compile.py']
auto_enabled: 0
errorformat: '%E%f:%l:%c: %m'
exe: 'python'
output_stream: 'stdout'
serialize: 1
serialize_abort_on_error: 1
short_name: 'py'
version information (/Users/dbarroso/.virtualenvs/napalm-yang-validator/bin/python --version): Python 3.7.1

pylama

args: ['--format', 'parsable']
auto_enabled: 0
errorformat: '%f:%l:%c: [%t] %m'
exe: 'pylama'
postprocess: function('neomake#makers#ft#python#PylamaEntryProcess')
version information (/Users/dbarroso/.virtualenvs/napalm-yang-validator/bin/pylama --version): pylama 7.6.6

mypy

args: ['--check-untyped-defs', '--ignore-missing-imports']
auto_enabled: 0
errorformat: '%E%f:%l: error: %m,%W%f:%l: warning: %m,%I%f:%l: note: %m'
exe: 'mypy'
version information (/Users/dbarroso/.virtualenvs/napalm-yang-validator/bin/mypy --version): mypy 0.650
NOTE: you can define g:neomake_python_enabled_makers to configure it (or b:neomake_python_enabled_makers).

For the project (used with :Neomake!):

makeprg

append_file: 0

args: []

auto_enabled: 1

errorformat: '%[^"]"%f"%\D%l: %m,"%f"%\D%l: %m,%-G%f:%l: (Each undeclared identifier is reported only once,%-G%f:%l: for each function it appears in.),%-GIn file included from %f:%l:%c:,%-GIn file included from %f:%l:%c,,%-GIn file included from %f:%l:%c,%-GIn file included from %f:%l,%-G%[ ]from %f:%l:%c,%-G%[ ]from %f:%l:,%-G%[ ]from %f:%l,,%-G%[ ]from %f:%l,%f:%l:%c:%m,%f(%l):%m,%f:%l:%m,"%f", line %l%\D%c%[^ ] %m,%D%\a[%\d]: Entering directory %['']%f'',%X%*\a[%*\d]: Leaving directory %*['']%f'',%D%\a: Entering directory %['']%f'',%X%*\a: Leaving directory %*['']%f'',%DMaking %*\a in %f,%f|%l| %m'

exe: 'make'

version information (/usr/bin/make --version): GNU Make 3.81
Copyright (C) 2006  Free Software Foundation, Inc.
This is free software; see the source for copying conditions.
There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A
PARTICULAR PURPOSE.
This program built for i386-apple-darwin11.3.0
NOTE: you can define g:neomake_enabled_makers to configure it.

Default maker settings:

output_stream: 'both'
remove_invalid_entries: 0
buffer_output: 1

Settings
New-style (dict, overrides old-style)
g:neomake: {
  'automake': {
    'ignore_filetypes': [
      'startify',
    ],
    'events': {
      'BufWritePost': {
        'delay': 0,
      },
    },
  },
}
b:neomake: unset

Old-style
g:neomake_ctags_maker = {'postprocess': function('PostprocessLintMaker'), 'args': ['-R', '--extra=+f', '--exclude=napalm_yang/models', '--exclude=vendor', '--exclude=.glide', '--exclude=.tox', '--exclude=*/modernizr.min.js', '.'], 'exe': 'ctags', 'append_file': 0}
g:neomake_open_list = 2
g:neomake_place_signs = 1
g:neomake_python_enabled_makers = ['python', 'pylama', 'mypy']
g:neomake_python_pylama_maker = {'postprocess': function('neomake#makers#ft#python#PylamaEntryProcess'), 'errorformat': '%f:%l:%c: [%t] %m', 'args': ['--format', 'parsable']}

makeprg=make
:version
NVIM v0.3.1
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/local/Homebrew/Library/Homebrew/shims/mac/super/clang -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/tmp/neovim-20180831-45030-1c6df1y/neovim-0.3.1/build/config -I/tmp/neovim-20180831-45030-1c6df1y/neovim-0.3.1/src -I/usr/local/include -I/usr/local/opt/gettext/include -I/Applications/Xcode-beta.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.14.sdk/usr/include -I/tmp/neovim-20180831-45030-1c6df1y/neovim-0.3.1/build/src/nvim/auto -I/tmp/neovim-20180831-45030-1c6df1y/neovim-0.3.1/build/include
Compiled by<EMAIL_ADDRESS>
Features: +acl +iconv +jemalloc +tui
See ":help feature-compile"

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

Run :checkhealth for more info

:messages
Already well formatted, good job. (took 0.0065s)
"napalm_yang_validator/validator.py" 10L, 250C written
Already well formatted, good job. (took 0.0060s)
"napalm_yang_validator/validator.py" 10L, 250C written
Already well formatted, good job. (took 0.0066s)
"napalm_yang_validator/validator.py" 10L, 250C written
Cannot parse: 10:0:             root.parse_config(profile, native)
"napalm_yang_validator/validator.py" 10L, 254C written
Already well formatted, good job. (took 0.0057s)
"napalm_yang_validator/validator.py" 10L, 250C written
Copied Neomake info to clipboard ("+).
Cannot parse: 10:0:             root.parse_config(profile, native)
"napalm_yang_validator/validator.py" 10L, 254C written

To add more strangeness to the issue, if I run NeomakeDisable and then NeomakeEnable everything starts working as expected again and the behavior explained in the issue doesn't occur.

Also, if I try to configure to run neomake on "TextChanged" with a delay, the option seems to be ignored (or at least it's run nonetheless immediately)

Reverting to "Dec 13, 2018" seems to fix the issue.

I've refactored list processing recently, and just fixed two issues with it.
I still plan to improve/fix "re-use list with automake".
So please first try current master, and if it still fails, please consider git-bisecting it to a specific commit.
Also please provide the actual debug log.
(will investigate later, but any additional info would help me already)

automake: Buffer was changed (TextChanged), restarting make: 3.

Looks like this autocommand might not be cleared properly - it is meant to abort the make from BufWritePost in case you edit/change the file while the buffer is running.
What is :au TextChanged and :au TextChangedI when this happens?

Could reproduce it, will hopefully push a fix shortly.

The problem is that the make info gets not cleaned properly (w:neomake_make_ids is not cleaned).
It is not really easy to reproduce, but have an idea about how to fix it.

Figured it out finally.
It was related to make info not properly cleaned on cancellation with serialized jobs (the python maker is one of those).
#2208 should fix it.

Thanks for the report, that was a tricky one.

Thanks to you for the quick response and the fix :)


How can i set time range for query to elasticsearch?

I am trying to add an alert to query an error occurrence during last one hour. However i did not find any option to add this into elastalert rule.
I found in the documentaiton "https://elastalert.readthedocs.io/en/latest/ruletypes.html" this >>
filter: A list of Elasticsearch query DSL filters that is used to query Elasticsearch. ElastAlert will query Elasticsearch using the format {'filter': {'bool': {'must': [config.filter]}}} with an additional timestamp range filter. All of the results of querying with these filters are passed to the RuleType for analysis. For more information writing filters, see Writing Filters. (Required, Elasticsearch query DSL, no default)
Can someone help on how to add this.
A sample portion of the  rule i wrote which dint work is given below.
filter:

query:
query_string:
query: 'environment : "PROD" and beat.hostname :  XXXXXXXX  and message: "java.sql.BatchUpdateException: Data truncation" and @timestamp >= "now-15m"'

This was achieved with --start option. Closing now


Refine service account creation command

Remove deprecated flag from service account creation script

Replaced by https://github.com/GoogleCloudPlatform/opentelemetry-cloud-run/pull/23


Command link steamid is not getting the tittle formated.

Issue fixed https://github.com/OriolFilter/Steam_Lobby_Discord_Bot/commit/529b474b3dabeaddc64f0351cae0b0e32e12f8b2
Waiting for pull request.


[SPARK-9786][Streaming][Kafka] fix backpressure so it works with defa…

…ult maxRatePerPartition setting of 0

Merged build triggered.

Merged build started.

Test build #41519 has started for   PR 8413 at commit 2664626.

Test build #41519 has finished for   PR 8413 at commit 2664626.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

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

@koeninger Thank you very very much for debugging and fixing this.
@dragos This was a bug inadvertently introduced in earlier PR

The rate limiter is using Long.MaxValue, while this one uses 0 as a default. They should both use the same.


Check output of test setup command

Current coverage is 81.12% (diff: 100%)

Merging #179 into master will not change coverage

@@             master       #179   diff @@
==========================================
  Files             5          5
  Lines           816        816
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            662        662
  Misses          154        154
  Partials          0          0

Powered by Codecov. Last update 17b09c3...194fbde


JAV-403 ignored non-rest style interfaces using @RestSchema

This issue turns out to be a bug. When the apiOperation is neither null nor hidden, methods annotates with ApiOperation will publish as a service.

Coverage increased (+0.04%) to 86.529% when pulling 8266311cfd7bf192386b4edd7d3894748d10544d on eric-lee-ltk:JAV-403 into 47ac2fdbd91341f7acf4cbb9ff65cded97835c52 on ServiceComb:master.


[Android] Fix gradle clean task

Running the command ./gradlew clean resulted in the following error:
./node_modules/react-native/ReactAndroid/src/main/jni//xreact/jni/Android.mk:42: *** Android NDK: Aborting.    .  Stop.
Android NDK: ./node_modules/react-native/ReactAndroid/src/main/jni//xreact/jni/Android.mk: Cannot find module with tag 'cxxreact' in import path
Android NDK: Are you sure your NDK_MODULE_PATH variable is properly defined ?
Android NDK: The following directories were searched:

The problem was that REACT_COMMON_DIR wasn't defined.
Test plan (required)
Running ./gradlew clean now works.
Adam Comella
Microsoft Corp.

By analyzing the blame information on this pull request, we identified @cjhopman and @mkonicek to be potential reviewers.

Failing tests are on iOS and look unrelated.
@facebook-github-bot shipit

Thanks for importing.If you are an FB employee go to Phabricator to review internal test results.


remove ReactFragmentActivity

Summary
Remove ReactFragmentActivity class, which was deprecated at least since 0.58.
Changelog
[Android] [Changed] - remove ReactFragmentActivity class.
Test Plan

Actually, I just checked, we're still using this internally for other reasons. We could move it to just internal technically, but let's revisit a bit later?

Actually, I just checked, we're still using this internally for other reasons. We could move it to just internal technically, but let's revisit a bit later?

Nevermind, the removed file was just an empty abstract subclass of ReactActivity, so I updated our internal callsites.


Replace ManagedBean with Named

The @ManagedBean annotation is deprecated in Jakarta EE 9, which we'll be using
when we upgrade to the next major Spring version. Replace it with @Named which
is supported in both the current and upcoming versions.
This is a global search-and-replace, no manual edits.

Current dependencies on/for this PR:

main

PR #837   👈

This comment was auto-generated by Graphite.


Rename config value provider

The config value provider conflicts with a service of the same name in my application.
The config provider should be prefixed. I would suggest fitTextConstants or fitTextDefaultConfig would be better names.

Thanks for heads up. This will be rolled out in v4.x
(sorry for delay!)


Added 2 nodes of 1 candidate

Added:

2 nodes of @plussik:matrix.org

@ironoa Element handle of the user was verified


support query data in viewmodel

This PR introduces a feature found at https://github.com/AvaloniaInside/Shell/issues/11#issue-1902105864.
The main purpose of this feature is to find InstanceViewModel properties and set their values.
Here's an example on how to use it:

We have also added an example in SecondView.

Hello @zyknow
I can see this is a draft PR but I can leave some comments for you.

The place you implemented is not correct, the job of the navigator is different than processing argument.
This feature should be an extension because this feature can cause performance issues.
There are many changes such as formatting files which are useless and make the review of PR very hard.
I can see you are using Navigator from the home page to the second page instead of automatic with the attached property. This feature should be implemented in a way that can work in both ways.
Maui used different ways to implement and they used a dictionary, So they used less generic execution.

Overall, it should be implemented in different ways to stay in the same application architecture.

Thank you for your response and comments, and for correcting my mistakes.
It's possible that I didn't fully understand this Shell code and also i have some question.
Here is my understanding for you comments
1,2,5 : add extensions for NavigationService maybe better,arguments in this extensions use Dictionary type maby better.
3      : i will doing better in next commit, thank you for helping me correct the problem.
4      : i think to implemented attached way is difficult for me. or you can tell me how to do it.
about performance in my implemented is use Reflection, may make AOT more difficult, I'm not sure.

Hello again @zyknow
I meant that an extension to be optional can enable feature flipping or replace a service.
This feature can be implemented in a new navigation update strategy.
After I reviewed the MAUI, I understood the query can be applied on Page not View model and the only way to use parameters at View Model is IQueryAttributable. This is back to the query string from the path and not the argument. the dictionary is the additional way to pass parameters which is this Shell has a different way to pass argument.
We already have query string passing but we never used it, so you can focus on that.
So. You never need to change Home page code from the attached property to the navigator service. the second page. because of 2 reasons, first, you can pass the argument as CommandParameter, and second thing because you can pass path.
I hope it is clear.

Thank you for your patient explanation. Now I understand.
i will commit a new pull request later.


Add functional tests for element handle

Test the following ElementHandle apis:

getMappingData
{s,g}etAsXML with Raw and Hex format

:shipit:

:shipit:


bump version in git

To avoid confusion, can you bump the version in git?
This will help people that are installing from master differentiate between master and the latest stable version.
Since you are planning a 5.0.0 release, how about 5.0.0-dev1?
https://github.com/timothycrosley/isort/blob/c54b3dd4620f9b3e7ac127c583ecb029fb90f1b7/pyproject.toml#L3

For sure! Done :) Sorry for any confusion the lack of a version bump caused.
~Timothy

Thanks!
Another thing to consider is maybe to make a pre-release.
(https://www.python.org/dev/peps/pep-0440/#pre-releases)
Such packages are not installed automatically by users pip installing isort, except if they specify an exact package version or using --pre flag when pip installing.
This will make it much simpler for people to test the version in master.
Example of a package with pre-releases : OmegaConf.


ng add @angular/pwa is failing

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Command (mark with an x)
- [X] add

Versions

Angular CLI: 6.1.2
Node: 10.1.0
OS: darwin x64
Angular: 6.1.0
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.7.2
@angular-devkit/build-angular     0.7.2
@angular-devkit/build-optimizer   0.7.2
@angular-devkit/build-webpack     0.7.2
@angular-devkit/core              0.7.2
@angular-devkit/schematics        0.7.2
@angular/cdk                      6.4.2
@angular/cli                      6.1.2
@angular/material                 6.4.2
@angular/pwa                      0.7.2
@ngtools/webpack                  6.1.2
@schematics/angular               0.7.2
@schematics/update                0.7.2
rxjs                              6.2.2
typescript                        2.7.2
webpack                           4.9.2
Repro steps
ng new my-app
ng add @angular/pwa
The log given by the failure

$ ng add @angular/pwa --project=weather
Installing packages for tooling via yarn.
yarn add v1.7.0
[1/4] 🔍  Resolving packages...
⠂ @angular/pwa(node:51115) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
[2/4] 🚚  Fetching packages...
[3/4] 🔗  Linking dependencies...
[4/4] 📃  Building fresh packages...
success Saved 1 new dependency.
info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
└─<EMAIL_ADDRESS>✨  Done in 4.79s.
Installed packages for tooling via yarn.
Path "/ngsw-config.json" already exist.
Desired functionality
ng add @angular/pwa should detect that the file isn't there and generate it
Mention any other details that might be useful
The file doesn't exist, I double checked.

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


Add link to entries for sharing via E-Mail

Now everything seems to work...

Does sharing stuff by email really happen that often? Copy/Pasting the article link doesn't do the job? The number of links for each item start to get crowded.


[Feature]: Possibility to specify boolEncoding and arrayEncoding strategies when adding parameters in bulk

What problem are you facing?
I have a private method in my custom client to build my HTTPRequests as follow:
func buildRequest(
        path: String,
        method: HTTPMethod,
        parameters: HTTPRequestParametersDict?,
        timeout: TimeInterval?
    ) -> HTTPRequest {
        let request = HTTPRequest {
            $0.method = method
            $0.path = path
            $0.timeout = timeout

            if let parameters = parameters {
                $0.body = .formURLEncodedBody(defaultParameters.merging(parameters, uniquingKeysWith: { $1 }))
            } else {
                $0.body = .formURLEncodedBody(defaultParameters)
            }
        }
        return request
    }

The problem with this is that I couldn't find a way to specify the boolEncoding and arrayEncoding.
So my question is:
Is there a way to specify it without iterating over all the parameters and manually use the request.addParameter(..., boolEncoding:, arrayEncoding:)? Or do you seen it as a new feature, like specifying it in the .formURLEncodedBody() method ?
Other Information
No response

Hi @nashfive, currently, there is no a way to explicitly declare it. However, I exposed both of them inside the formURLEncodedBody() in order to avoid iterating over each parameter which, obviously does not make sense. You will find in 1.8.0.


Use -spinStyle instead of -yield

The -yield flag was replaced some time ago: https://github.com/openjdk/jcstress/commit/d5f0de25943e39766df3567899c6d849ef1eb29f#diff-8bc649ce01dd152f0c64ed6c5fd69107c42a6ea774eeee76e321ce3e741cc170

Hey @fwcd - thank you for the contribution. I'm having a problem with Travis CI - once I resolve it, I'll be able to merge your PR to master. That's within this week, I think.

GitHub Actions is also pretty good, in case you are looking for a CI provider. It's built-in and free for OSS repos.

Good evening @fwcd - I've just migrated to GitHub Actions as you mentioned, made a small fix and released.
Would you be able to rebase onto current master, so that Actions pick up your pull request?
Apologies about the delay and thank you for your patience.

Sure, here you go.

Hello @fwcd - I've merged your changes and released the new version, 0.8.13.
Thank you!


Optimizing Build times

Masked fenix overlay and we will be adding packages manually whenever needed.
Building nix-expressions from nix-shell itself, so integrationtests script can use binaries and avoid compiling twice.
Removed SRC_DIR and BIN_DIR from integrationtest script as we will be using binaries generated by nix expressions.
Fix #67

Needs a rebase after #77


[Bug Report] Reshape results in wrong values

Describe the bug
import torch
import ttnn

torch.set_printoptions(precision=10)

with ttnn.manage_device(device_id=0) as device:
    x_torch = torch.randn((128, 1, 1, 128), dtype=torch.float32)

    x_tt = ttnn.from_torch(x_torch, dtype=ttnn.bfloat16, layout=ttnn.TILE_LAYOUT, device=device)
    x_reshaped_tt = ttnn.reshape(x_tt, ttnn.Shape([128, 128]))

    x_tt_torch = ttnn.to_torch(x_tt)
    x_reshaped_tt_torch = ttnn.to_torch(x_reshaped_tt)

    print(x_tt.shape, x_reshaped_tt.shape)
    print(x_tt_torch[1][0][0][0], x_reshaped_tt_torch[1][0])
    print(torch.sum(x_tt_torch), torch.sum(x_reshaped_tt_torch))

Values should not change after reshape.
To Reproduce

Run script above

Expected behavior
Buffer should contains same values.
Please complete the following environment information:

OS: Ubuntu 20.04
Version of software: 948fafb081ace2fd966295548128616b409903cf

Possibly this shape too: @ayerofieiev-tt mentioned it:
Reshape of [1, 64] to [1, 64, 1] in Tile does not work. I have a feeling it's related to this issue.

This issue will be resolved when the above pr is pushed

Resolved


[Feature request] add a loading screen behavior configuration

Hi, first of all, thanks for this amazing addon, helps me a lot !
As you mentionned in your code examples, it's not that hard to add a loading screen, change_scene to your loading screen scene, then inside the loading screen, call change_scene again.
The issue with this is if the second scene is taking some time to load (like a procedurally generated world for instance), in which case you'll have the following:

Start scene
Transition to loading screen
Loading screen
Transition to black screen
Stays in black screen until scene is ready
Transition to target scene
Target scene is now current scene

I'm pretty new to Godot so I might be missing something obvious here but it looks like having the Loading screen add a given scene to the tree to then check when it's ready before triggering transition isn't possible, and that would be amazing as it'd allow for more detailed loading screen, showing data from the scene that's currently being set up.

Hi,
First, I don't think you are missing something obvious, your case is just a little bit different than normal cases and that's because of procedural generating your world.
What I think you can do, if you really want to use this addon, is to just get a little bit deeper into this and I will help you through it.

You change the screen to loading screen.
You wait in the loading screen for the scene to load into the memory.
When the loading is finished and you get informed by SceneManager.load_finished signal, you don't change the scene to the new loaded scene by SceneManager.change_scene_to_loaded_scene but instead what you do, is that you will get the loaded scene by yourself from SceneManagerlike this:

if SceneManager._load_scene != "":
	var scene = ResourceLoader.load_threaded_get(SceneManager._load_scene) as PackedScene
	if scene:
		SceneManager._load_scene = ""

Now you have your scene in the scene variable.
Note: SceneManager._load_scene = "" line is important for scene manager, just let it be. and if you want to know where did I get these lines, I get them from SceneManager.change_scene_to_loaded_scene function.
4. Then after, let's add the scene to the tree and let it load the chunks without touching the current scene (which is the loading screen):
if SceneManager._load_scene != "":
	var scene = ResourceLoader.load_threaded_get(SceneManager._load_scene) as PackedScene
	if scene:
		SceneManager._load_scene = ""
		var root = get_tree().get_root()
		root.add_child(scene.instantiate())

That's it, we are still seeing the loading screen (If I'm not mistaken) and the game is silently generating the world in the background, now we wait for the world generation process to be done and after it is done, what you do is that you do the transaction by yourself using SceneManager singleton:

# this is literally the `SceneManager.change_scene` function but with little less complication considering you know what you're doing
func change_scene_by_hand_after_generating_the_world():
	SceneManager._set_in_transition()
	SceneManager._set_clickable(your_general_options.clickable)
	SceneManager._set_pattern(your_fade_out_options, your_general_options)
	# fade out transaction happens here
	if SceneManager._fade_out(your_fade_out_options.fade_speed):
		await SceneManager._animation_player.animation_finished
	# real change scene goes here
	var root = get_tree().get_root()
	# get the loaded, completely generated scene
	var scene = root.get_child(root.get_child_count() - 1)
	# delete the loaded scene
	root.get_child(root.get_child_count() - 2).free()
	# inform godot which now this is the current scene
	get_tree().set_current_scene(scene)
	# the change scene is done
	# now let's fade in
	SceneManager._animation_player.play(SceneManager.NO_COLOR, -1, 1, false)
	SceneManager._set_pattern(your_fade_in_options, your_general_options)
	if SceneManager._fade_in(your_fade_in_options.fade_speed):
		await SceneManager._animation_player.animation_finished
	SceneManager._set_clickable(true)
	SceneManager._set_out_transition()

And done.
Hope this helps.

And at last, now that I'm looking at it, its not that complicated, if this works for you, I will add this into the addon later.
Keep me updated about this scenario please.

Thanks ! I'll check that once I have a bit of time for it and report the results here 👍

So I had to edit the code a bit, because I need a reference to the scene instance in order to know when it's ready to load (for this I have a signal provided by the root node of the instance that tells me when loading is done).
Here is my basic loading screen:
extends Control

@onready var label = $CenterContainer/HBoxContainer/Label

var no_fade = SceneManager.create_options(0.1, "", 0.1)
var pixel_fade = SceneManager.create_options(1.0, "pixel", 0.2, true)
var general_options = SceneManager.create_general_options(Color(0, 0, 0), 0, true, true)

func _ready():
	# TODO random message, better loading experience
	label.text = tr("LOADING:LOADING_DATA")
	var target = SceneManager.get_recorded_scene()
	SceneManager.load_scene_interactive(target)
	SceneManager.load_finished.connect(_init_target_scene)

func _init_target_scene():
	if SceneManager._load_scene != "":
		var scene = ResourceLoader.load_threaded_get(SceneManager._load_scene) as PackedScene
		if scene:
			SceneManager._load_scene = ""
			var root = get_tree().get_root()
			var instance = scene.instantiate()
			root.add_child(instance)
			instance.load_done.connect(change_scene_by_hand_after_generating_the_world)

func change_scene_by_hand_after_generating_the_world():
	SceneManager._set_in_transition()
	SceneManager._set_clickable(general_options.clickable)
	SceneManager._set_pattern(pixel_fade, general_options)
	# fade out transaction happens here
	if SceneManager._fade_out(no_fade.fade_speed):
		await SceneManager._animation_player.animation_finished
	# real change scene goes here
	var root = get_tree().get_root()
	# get the loaded, completely generated scene
	var scene = root.get_child(root.get_child_count() - 1)
	# delete the loaded scene
	root.get_child(root.get_child_count() - 2).free()
	# inform godot which now this is the current scene
	get_tree().set_current_scene(scene)
	# the change scene is done
	# now let's fade in
	SceneManager._animation_player.play(SceneManager.NO_COLOR, -1, 1, false)
	SceneManager._set_pattern(pixel_fade, general_options)
	if SceneManager._fade_in(pixel_fade.fade_speed):
		await SceneManager._animation_player.animation_finished
	SceneManager._set_clickable(true)
	SceneManager._set_out_transition()

But I'm getting an error at get_tree().set_current_scene(scene) that says "Cannot call method 'get_tree' on a previously freed instance". I assume this is because the loaded scene is being removed two lines before but idk if it's safe to remove that, I think you'll know better than me for sure.

I'm gonna test this.
I'll be back.

Moving the root.get_child(root.get_child_count() - 2).free() call at the end of the function seems to be doing the job, as the loading screen then removes itself as the final operation 🙂

I implemented this feature into v3.8.0 new release.
It wasn't hard to be honest, I was talking nonsense when I said it gets complicated if I add this feature.
Have a look at the methods section: https://github.com/maktoobgar/scene_manager#methods
The last two new functions are exactly for this scenario, function 23 and 24.
Documentation is updated and a new example is now available in the demo for exactly this scenario which the codes are available too and kinda described in here:
https://github.com/maktoobgar/scene_manager#more-complex-example-with-loading-screen-for-scenarios-that-scenes-need-some-time-in-background
And yeah, that's it.
If you want to see the implemented example which was kinda explained in the previously mentioned section of the readme, just take a look at the first new oddly named button in scene3.tscn and follow where that button leads you after:

And that's it.
If you found any bugs, or had any new ideas, please do tell.

I don't think it fixes the issue entirely but this is me not explaining everything properly I think.
So in my use case, the scene itself has signals to give me progress, so I can have my loading screen display different labels based on the generation step, helping for debugging and for UX purpose too.
So I'd say the only thing that the new methods are missing is to return the actual scene instance in add_loaded_scene_to_scene_tree() so we can easily connect any signal to it, or do other operations that one might want to do.

Yeah, this is nice.
I will add this to the later versions too as soon as I found time to do that and I'll inform you about the new version with that change also.
For now just changing the current add_loaded_scene_to_scene_tree function code to what you just wrote will satisfy your needs as you just said.
Just one thing as a suggestion, change Variant return type to Node.
That's it.
Thank you for this nice issue.


IntelliSense in line comments

After upgrading to VSCode 0.10.6 I've been getting IntelliSense suggestions when writing a line comment (starting the line with //) in C# projects. I use VScode for developing Unity3D projects on OSX 10.11.2.
It's an understatement to call it annoying when I write e.g. "will do" and "do" auto completes to a full do...while loop.

It's not just limited to C# either, I get the same thing when writing PHP, a commented line starting with // will still invoke IntelliSense when writing for example 'while', 'for' etc.

It's related to this issue https://github.com/Microsoft/vscode/issues/719

closing this issue as a duplicate of #719


Add sphinx docs

Will resolve #12

The current draft of this PR can be viewed here
http://django-name.readthedocs.org/en/add-sphinx-docs/

Add an env to the build for docs to be built

ping @somexpert @ldko
This is a sort of initial draft of the docs. Once we get this merged in, we can keep it up to date as more features and configurations options are made available.

@ldko, those changes have been completed.

:+1:


can't try fashion_parsing_segmentation

i followed the instructions in readme here
https://github.com/open-mmlab/mmfashion/tree/master/configs/fashion_parsing_segmentation
And i run pip install -r requirements.txt before setup
when try to run
python /mmfashion/mmdetection/tools/demo.py
i get this error
Traceback (most recent call last):
File "/content/mmfashion/mmdetection/tools/demo.py", line 3, in
from mmdet.apis import inference_detector, init_detector, show_result
File "/usr/local/lib/python3.6/dist-packages/mmdet/apis/init.py", line 1, in
from .inference import (async_inference_detector, inference_detector,
File "/usr/local/lib/python3.6/dist-packages/mmdet/apis/inference.py", line 7, in
from mmcv.ops import RoIPool
File "/usr/local/lib/python3.6/dist-packages/mmcv/ops/init.py", line 1, in
from .bbox import bbox_overlaps
File "/usr/local/lib/python3.6/dist-packages/mmcv/ops/bbox.py", line 3, in
ext_module = ext_loader.load_ext('_ext', ['bbox_overlaps'])
File "/usr/local/lib/python3.6/dist-packages/mmcv/utils/ext_loader.py", line 11, in load_ext
ext = importlib.import_module('mmcv.' + name)
File "/usr/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'mmcv._ext'
anyhelp and thanks in advance

i tried mmcv_full and now i get this error
Traceback (most recent call last):
File "/content/mmfashion/mmdetection/tools/demo.py", line 3, in
from mmdet.apis import inference_detector, init_detector, show_result
File "/usr/local/lib/python3.6/dist-packages/mmdet/apis/init.py", line 1, in
from .inference import (async_inference_detector, inference_detector,
File "/usr/local/lib/python3.6/dist-packages/mmdet/apis/inference.py", line 7, in
from mmcv.ops import RoIPool
File "/usr/local/lib/python3.6/dist-packages/mmcv/ops/init.py", line 1, in
from .bbox import bbox_overlaps
File "/usr/local/lib/python3.6/dist-packages/mmcv/ops/bbox.py", line 3, in
ext_module = ext_loader.load_ext('_ext', ['bbox_overlaps'])
File "/usr/local/lib/python3.6/dist-packages/mmcv/utils/ext_loader.py", line 11, in load_ext
ext = importlib.import_module('mmcv.' + name)
File "/usr/lib/python3.6/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
ImportError: /usr/local/lib/python3.6/dist-packages/mmcv/_ext.cpython-36m-x86_64-linux-gnu.so: undefined symbol: _ZN6caffe28TypeMeta21_typeMetaDataInstanceISt7complexIdEEEPKNS_6detail12TypeMetaDataEv

please clone the repo recursively, like in the dockfile

I think it's because of the wrong mmcv version

I got the error "ModuleNotFoundError: No module named 'mmcv._ext' ", too. Have you solved this problem?

You should try installing mmcv-full.

may i know if anyone get this executable ?
i am not sure where i can get the checkpoint 'checkpoint/resnet50.pth', which seems needed in 'mask_rcnn_r50_fpn_1x.py' as enclosed.
hopes some suggestions.  Many thanks.
model settings
model = dict(
type='MaskRCNN',
pretrained='checkpoint/resnet50.pth',
backbone=dict(
type='ResNet',
depth=50,
num_stages=4,
out_indices=(0, 1, 2, 3),
frozen_stages=1,
norm_cfg=dict(type='BN', requires_grad=True),
style='pytorch'),


Removed stuff at the bottom, complete dark mode, added UI fixes to room

You need to pull in the changes from the develop branch. I don't see the lock screen when logging out. Why remove the theme switch?


Custom Favicon possible?

I was wondering if it would be possible to add a custom favicon / website ico to the program?

Hey, thanks for opening an issue.
At the moment, it's not possible without modifying the code and compiling your own version.
We didn't put any effort on allowing such customizations because the scope of the webhook as a tool is mainly aimed towards the automation pipelines, which means that it's mostly invoked by computers and not humans.
Could you tell us more about your use-case for this feature?
Thanks in advance!

Hi adnanh! Sorry for the late reply! I've been using the webhook tool to launch games and programs on my computer. Then, I like to save the links to them to the home screen on my phone.

@ITCMD Sorry for the late reply as well :-( Anyway, one way of doing it would be using response-message field to return the valid HTML using the base64 encoded data url favicon like this:
"response-message": "<title>Favicon example</title><link href=\"data:image/x-icon;base64,iVBORw0KGgoAAAANSUhEUgAAABAAAAAQEAYAAABPYyMiAAAABmJLR0T///////8JWPfcAAAACXBIWXMAAABIAAAASABGyWs+AAACbUlEQVRIx7WUsU/qUBTGv96WSlWeEBZijJggxrREdwYixMnByYEyOvgfsBAMG0xuDsZ/QGc3NDFhgTioiYsmkhBYGLSBkLYR0va8gSjvQXiIT7/l5ibfOd/v3pN7gSmVSMTj8ThRfzdYk8lkMpl83/+AVFVVVXU0eHiVJEmSpB8DIcpkMplsdhCYz+fzhQJROBwOh8PDQN+oQCAQCASIRFEURZHI45GkP0/e7Xa73e70AMJnjel0Op1OA6oaDB4eAkAw6PcDvZ5t6zrw/Hx2trAw/cHYZ426ruu6DtzcGEYuBzQa19etFvD4WKtls4AoRqMPDwBjjLGPrt84ilgsFovF6EOapmmaRiP6O/jbAIguL4vFYpHGqlKpVCoVomq1Wq1Wibxer9fn+w+Q9+cUiUQikQhNrfdgWZZlWf4yyGhj27Zt254MUK/X6/X6F0aiKIqiKIOCYRmGYRjGZADLsizLIgqFQqHV1SkAnp5OTn79ItK0qyuPZ7SxaZqmaU4GKJfPzxmbfAPc/f3pqaIQLS8vLtZqgOP0bYyJoiAARC5Xrwf4/Vtbb2+Th1YqlUqlErC01GgkEkCz2WxyHLC+LsuiCAiCJLlcgM+3vd3pcBzXaJTLR0dEs7Ptdv+D4TiOG/A6DsBxQKvV621sAGtru7vl8ngAjuvXv7xcXIgiwNjMjCj2h+k4fQfPA4LA8xwHCO323V2hABiG223bwPy8xwMAbvfcHGMAY32j47y+3t4OAsZpZ2dzEwAsy7IcBxAExhwHMIxOx3GAlZVUyjT/1WFIudzenstFlEpFo9M8o+Pj/X2eJzo4SCR4fnzdb2N4Pyv9cduVAAAAAElFTkSuQmCC\" rel=\"icon\" type=\"image/x-icon\" />"

Would that work for you? If you're unsure how to get the base64 encoded blob, you could use tool such as this


getting error from mesos_hash_parser

I tried upgrading my puppet setup from last year to the current module. I tried running it and when that failed I tried again using the slave example in the README. In both occasions the puppet run fails on mesos_hash_parser :
Info: Retrieving pluginfacts
Info: Retrieving plugin
Info: Loading facts
Error: Could not retrieve catalog from remote server: Error 400 on SERVER: mesos_hash_parser(): Wrong number of args given (3 for 1) at /etc/puppet/environments/production/modules/mesos/manifests/slave.pp:105 on node mesos01-dev.localdomain
Warning: Not using cache on failed catalog
Error: Could not retrieve catalog; skipping run
The offending line in the module slave code :
mesos_hash_parser($cgroups, 'slave', 'cgroups'),
The $cgroups should just be an empty hash as defined in the slave module itself
Running Ubuntu 14.04, Ruby 1.9.3p484, Puppet 3.8.6
Any hints greatly appreciated

That's strange. Which version of this module do you use? The error message is slightly misleading, it should accept either 1 or 3 arguments. How do you install Puppet modules? Do you have appropriate version of puppet-stdlib?
Latest modification to this part of code was made in #48. Automated tests are passing on Ruby 1.9.3, Puppet 3.8.5.

I manage my modules using r10k :
forge "https://forge.puppetlabs.com"
mod 'deric/mesos', '0.7.0'
mod 'deric/zookeeper', '0.5.1'
mod 'puppetlabs/apache', '1.6.0'
mod 'puppetlabs/apt', '2.1.0'
mod 'puppetlabs/concat', '1.2.3'
mod 'puppetlabs/firewall', '1.4.0'
mod 'puppetlabs/mysql', '3.4.0'
mod 'puppetlabs/stdlib', '4.11.0'
mod 'richardc/datacat', '0.6.1'
mod 'puppetlabs/powershell', '1.0.6' # chocolatey dependancy
mod 'badgerious/windows_env', '2.2.2' # chocolatey dependancy
mod 'chocolatey/chocolatey', '1.2.1'
mod 'puppetlabs/dism', '1.2.0'

That should be ok. What arguments do you pass to mesos::slave? Do you configure somehow cgroups settings?

My original code was :
class { '::mesos::slave':
zookeeper      => $master_zookeepers,
listen_address => $listen_address,
attributes     => {
'env' => 'production',
},
resources      => {
'ports' => '[2000-65535]'
},
options        => {
  'containerizers' => 'docker,mesos',
  'hostname'       => $::fqdn,
},

}
(this is the part I used in a proof of concept las year)
As that failed, I commented it out and tried the example from the README :
class{'::mesos::slave':
zookeeper      => 'zk://<IP_ADDRESS>:2181,<IP_ADDRESS>:2181,<IP_ADDRESS>:2181/mesos',
listen_address => $::ipaddress,
attributes     => {
'env' => 'production',
},
resources      => {
'ports' => '[2000-65535]'
}
}
Both give the same error here.

No other configuration in Hiera? Could you please test latest version from git master? I'm trying to reproduce this on Ubuntu 14.04.2, but everything works as expected.

The only messes related items below :
messos::master_zookeepers: 'zk://<IP_ADDRESS>:2181/mesos zk://<IP_ADDRESS>:2181/mesos'
messos::master_iprange: '<IP_ADDRESS>/24'
I will try the master branch later on

I'm baffled now.
Tried the master branch version -> worked ok
Removed this version and reinstalled released module using r10 -> worked ok
Same setup, same virtual machines, no reboots in between or such.
Maybe it was a weird apache / passenger caching issue, I'm really not sure. I will see what I can do tonight to reproduce the issue.

Yeah, it looks like there was older version on master and new fact on client side. Reopen the ticket if the problem persists.


Please expand example

Hi,
Please expand this example. Including the reference file itself in CVS and JSON format. I can't find a sample for a JSON reference anywhere, so this might be a good place to create a complete example.
Thanks in advance,
Hugo Voerman

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

ID: 64494cba-b4ee-4baf-6058-a533565c7e6b
Version Independent ID: 70f7ee5c-4880-9f38-e53f-0b731069e7a9
Content: Reference Data JOIN (Azure Stream Analytics) - Stream Analytics Query
Content Source: stream-analytics-docs/reference-data-join-azure-stream-analytics.md
Service: stream-analytics
GitHub Login: @mamccrea
Microsoft Alias: mamccrea

Hi @hugovoerman - Thanks for providing feedback. I just wanted to make sure I understand your request. Are you asking for an example of reference input? Here's an article on how to parse different formats, including JSON.

Hi Mary,
I would like to see a complete example of streaming data including json reference data. So the payload of a streaming input, the contents of the reference json, how to put the reference data in a blob (with and without {date} and/or {hh}) , the sa query, and all else I forgot.
I have not been successful in making it work with reference data in blob. Using sql as reference source however was easy. But I want the refdata in blob, so please help with a complete example
Thanks in advance,
Hugo Voerman
Outlook voor Android downloadenhttps://aka.ms/ghei36

From: Mary McCready<EMAIL_ADDRESS>Sent: Saturday, November 23, 2019 12:40:14 AM
To: MicrosoftDocs/azure-reference-other<EMAIL_ADDRESS>Cc: Hugo Voerman<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-reference-other] Please expand example (#94)
Hi @hugovoermanhttps://github.com/hugovoerman - Thanks for providing feedback. I just wanted to make sure I understand your request. Are you asking for an example of reference input? Here's an article on how to parse different formatshttps://docs.microsoft.com/en-us/azure/stream-analytics/stream-analytics-parsing-json, including JSON.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-reference-other/issues/94?email_source=notifications&email_token=ADECCFO2IZRTTTONXYP7IRLQVBUV5A5CNFSM4JJJ7SJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEE7FTFY#issuecomment-557734295, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADECCFJSA7HQXYS6SLZJK33QVBUV5ANCNFSM4JJJ7SJA.

Hi @hugovoerman. Please see Build an IoT solution by using Stream Analytics for an example using reference data.

I will now proceed to close this thread. If you have any additional questions, please reopen this issue.
#please-close


Allow integers to be typed as enums

Hey! Thanks for the library, it works great.
I think the following would be a cool feature.
/// [PaymentStatus]
status        Int       @default(0) @db.TinyInt

enum PaymentStatus {
  Unpaid = 0,
  Paid = 1,
}

Hey @rivatove this is a awesome feature request! Any reason on why prisma enums cannot be used?

Hey @rivatove this is a awesome feature request! Any reason on why prisma enums cannot be used?

Hey, this is funny. I actually forgot these exist because the last time I checked my DB engine didn't support them.
I successfully migrated to DB native enums and they fulfill this use case.
Enums in code have certain advantages such as portability (only integer support is needed) and flexibility (make changes without migrating the database) but I'm not sure if that makes them worth pursuing for the library.
Thanks again!


The results obtained using the pre-trained model were not good

Thank you for your research and sharing. When I used the pre-trained model to test on Ford Log2, the results I got were quite different from those shown by the pre-trained model. The parameters I used in the test were all default parameters in the code, and I wonder if it is the parameter setting problem. The following are the test results of the pre-trained model given in Ford Log2 and the results obtained by me using the pre-trained model.
GPU used in the experiment: 2080TI
results I got：

results you shown：

Problem solved


DeepExplainer: how to load a Tensorflow CNN model(.pb file)

I'd like to  use SHAP to interpret a Tensorflow CNN model (.pb file). But I don't know how to load the model.
"  if framework == 'tensorflow', (input : [tf.Tensor], output : tf.Tensor) A pair of TensorFlow tensors (or a list and a tensor) that specifies the input and output of the model to be explained. Note that SHAP values are specific to a single output value, so the output tf.Tensor should be a single dimensional output (,1)."

I have worked a lot with TF model serialization, but can you just load the model into TF first then pass the constructed graph to SHAP?

@slundberg Thanks for your reply. I have loaded the model into TF and constructed graph. But the graph couldn't be passed to SHAP. I loaded the model as follows:
def load_model(model_filepath):
print("Loading trained model...")
with tf.Session() as sess:
graph_def = tf.GraphDef()
with tf.gfile.FastGFile(model_filepath, 'rb') as f:
graph_def.ParseFromString(f.read())
tf.import_graph_def(graph_def, name='')
#print(graph_def)
return graph_def
......
background = x_test
model = load_model(model_path)
e = shap.DeepExplainer(model, background)
I got the error:
File "./DeepExplainer.py", line 139, in
main()
File "./DeepExplainer.py", line 132, in main
e = shap.DeepExplainer(model, background)
File "D:\AppStudy\Anaconda3.0\envs\python36\lib\site-packages\shap\explainers\deep_init_.py", line 80, in init
self.explainer = TFDeepExplainer(model, data, session, learning_phase_flags)
File "D:\AppStudy\Anaconda3.0\envs\python36\lib\site-packages\shap\explainers\deep\deep_tf.py", line 83, in init
from tensorflow.python.ops import gradients_util as tf_gradients_impl
ImportError: cannot import name 'gradients_util'

Hello, I got the same problem too, have you solved it?

I am also receiving the same issue. Has there been an update on this?


enums variants: replace dots (.) with underscore (_)

This PR fixes #341 and replaces all dots in enum variants with underscores.

@jens1205 The fix was merged as part of #344 ?

@tiagolobocastro : yes, for some reason I accidently merged this one-line change already into #344

FIxed as part of #344


Add support for decoding clone_targets

julia> clone_targets("/home/vchuravy/.julia/compiled/v1.10/SnoopPrecompile/TcfDT_z6cWT.ji")
┌ Info: Target
│   name = "znver2"
│   flags = 0
│   feature_en =
│    44-element Vector{UInt8}:
│     0x03
│     0x32
│     0xd8
│     0x36
│        ⋮
│     0x00
│     0x00
│     0x00
│   feature_dis =
│    44-element Vector{UInt8}:
│     0x00
│     0x00
│     0x00
│     0x40
│        ⋮
│     0x00
│     0x00
│     0x00

What's feature_en and feature_dis? Could those be spelled out more?

I assume it's enabled and disabled?

Added in https://github.com/timholy/PkgCacheInspector.jl/pull/23


ignore inherit from .rubocop_todo.yml

.rubocop_todo.yml is a standard filename to keep issues that need fixing, but haven't yet
https://github.com/bbatsov/rubocop/blob/master/manual/configuration.md#automatically-generated-configuration
It means CI can enforce rubocop on (most) new code while you're chipping away at historical code.
However, adding a .rubocop_todo.yml to a project causes Codeclimate to instantly jump the score up and consider every ignored issue as fixed. Projects jump from a 2.0 GPA to 3.9 with one commit.
Later when the issue are genuinely fixed, Codeclimate does not notice. the score does not change, and the contributor gets no glory.
I request codeclimate-rubocop learns to ignore (remove before analysis?) this exact line at the top of a .rubocop.yml
inherit_from: .rubocop_todo.yml

Hi @Br3nda,
Thanks for the suggestion, but we don't think this is a direction we want to go
in right now for default behavior. In general, we think Code Climate engines
work best for people when they adhere as closely as possible to what the same
tool would do when you ran it locally without Code Climate. In the case you're
talking about, the .rubocop_todo.yml file is explicit configuration saying
that some RuboCop issues should be ignored, and we think it's best for this
engine to continue to respect that by default.
That said, this is behavior you can configure yourself today! You can move the
bulk of your RuboCop configuration to a separate file (e.g.
.rubocop_codeclimate.yml), and then change your .rubocop.yml to have no
actual configuration except inheriting from .rubocop_codeclimate.yml and
.rubocop_todo.yml, e.g.
inherit_from: .rubocop_codeclimate.yml
inherit_from: .rubocop_todo.yml

Once you've done that, you can update your .codeclimate.yml to tell the
RuboCop engine to use .rubocop_codeclimate.yml for its configuration instead
of the default .rubocop.yml:
engines:
  rubocop:
    enabled: true
    config:
      file: .rubocop_codeclimate.yml

With a setup like that, you can continue to get .rubocop_todo.yml applied
locally, but not on codeclimate.com.
I hope that explanation makes sense & is helpful to you. I'm going to go ahead
and close this, but if you further questions or run into any issues with the
setup I suggested, feel free to comment here and I'll get back to you.

Thanks for the  great reply. I couldn't find that in the  docs, and the
code climate support person sent me here to request the feature.
Could it be added to the docs for future reference?
On 3 Dec 2016 12:14 pm, "Will Fleming"<EMAIL_ADDRESS>wrote:
Hi @Br3nda https://github.com/Br3nda,
Thanks for the suggestion, but we don't think this is a direction we want
to go
in right now for default behavior. In general, we think Code Climate engines
work best for people when they adhere as closely as possible to what the
same
tool would do when you ran it locally without Code Climate. In the case
you're
talking about, the .rubocop_todo.yml file is explicit configuration saying
that some RuboCop issues should be ignored, and we think it's best for this
engine to continue to respect that by default.
That said, this is behavior you can configure yourself today! You can move
the
bulk of your RuboCop configuration to a separate file (e.g.
.rubocop_codeclimate.yml), and then change your .rubocop.yml to have no
actual configuration except inheriting from .rubocop_codeclimate.yml and
.rubocop_todo.yml, e.g.
inherit_from: .rubocop_codeclimate.ymlinherit_from: .rubocop_todo.yml
Once you've done that, you can update your .codeclimate.yml to tell the
RuboCop engine to use .rubocop_codeclimate.yml for its configuration instead
of the default .rubocop.yml:
engines:
rubocop:
enabled: true
config:
file: .rubocop_codeclimate.yml
With a setup like that, you can continue to get .rubocop_todo.yml applied
locally, but not on codeclimate.com.
I hope that explanation makes sense & is helpful to you. I'm going to go
ahead
and close this, but if you further questions or run into any issues with the
setup I suggested, feel free to comment here and I'll get back to you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeclimate/codeclimate-rubocop/issues/86#issuecomment-264588145,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAAv6mWDovcbdrnUkK6kmjkHYXYy8ipZks5rEKZIgaJpZM4K_y3Q
.

Could it be added to the docs for future reference?

Yep, I thought we did mention the custom config file in the docs, but I guess we missed that. I'll update the page on docs.codeclimate.com today to mention this, thanks for the suggestion.

I tried this, and rubocop does not respect the second include. You only get one

Oh, sorry, @Br3nda, that was poor proof-reading on my part. The correct syntax is to give inherit_from an array of filenames (docs at https://github.com/bbatsov/rubocop/blob/master/manual/configuration.md#inheriting-from-another-configuration-file-in-the-project).
Can you try that & see if it works better?


[FR] open pdf in obsidian from zotero

if we directly set obsidian as the default app for PDF, when we open the PDF, we will redirected to the vault select panel in fact,
I found a way to use the URI scheme, but this needs extra setup
I create a Python file and packaged it as a .exe file, which simply transfers the open/start  CLI command trigger by Zotero, to an obsidian open file scheme
file_path = sys.argv[1]

# url encode
url_file_path = urllib.parse.quote(file_path)

# open by URL scheme
os.system(f"start obsidian://open?path={url_file_path}")

I just wonder if there is any better method to accomplish this behavior, since now with this plugin, we could use obsidian as the viewer and annotator, but we still need the capability Zotero provided to manage literature.

Thanks for sharing your interesting approach!
Just to confirm, do you put your Zotero PDFs (or symbolic links to them) inside your Obsidian vault? (Otherwise, I think PDF++ cannot help.)
I'm not sure I can answer this question properly because in my use case, I rarely open PDFs from Zotero.
After importing a paper to Zotero, I immediately import it into Obsidian using the Zotero Integration plugin.
My template contains almost all metadata stored in Zotero, so I can query from the literature using Dataview, and I don't have back to Zotero except when updating the metadata.
I put a symbolic link to my Zotero folder, and my template contains an Obsidian link to the paper PDF in the Zotero folder as the PDF property. So I can open the PDF easily from within Obsidian.

I might want to send a feature request to ZotLit.
ZotLit is composed of two components: an Obsidian plugin and a Zotero addon.
So the developer might be able to add the ability to open a PDF stored in the Obsidian vault from within Zotero to their Zotero addon.

(Just to confirm, do you put your Zotero PDFs (or symbolic links to them) inside your Obsidian vault? Otherwise, I think PDF++ cannot help.)

Yes, I use Zotfile to move the PDFs under the vault, while the whole folder is synced by OneDrive.
At the moment, this approach is a bit tricky, as I haven't yet managed to get Zotfile to move the PDFs automatically after import.🤷

After importing a paper to Zotero, I immediately import it into Obsidian using the Zotero Integration plugin.

this is a considerable workflow, i should try this recently

You might want to send a feature request to ZotLit.

i will🫡

This will open the literature note in your Obsidian vault. If you have a link to the PDF in the literature note, you can click it to open the PDF

this a good idea, but actually, these are two workflow

One is what you mention here, opening the PDF via a link that points to the PDF file directly in the Obsidian note. This implicitly suggests that you use Obsidian to browse(choose, or any other proper word😏) the literature.
Another is what I mentioned earlier, browsing the literature in Zotero, then opening it in Zotero, then automatically jumping to Obsidian.

the main difference is how you browse the literatures
currently i use zotero to browse, but i just realized that if i combine your method with canvas, i can somehow emulate the mind map view, which is much better than zotero's plain list view.

One is what you mention here, opening the PDF via a link that points to the PDF file directly in the Obsidian note. This implicitly suggests that you use Obsidian to browse(choose, or any other proper word😏) the literature.

Not quite. You can primarily use Zotero to browse literature, and then right-click the item that you want to open in Obsidian.
The ZotLit addon for Zotero will then take you to the literature note in Obsidian.
I'm afraid there is not much I can do regarding this issue so I'm closing this.

@SSmJaE
Could you maybe give me a hint how to use your python code on macOS to open pdfs in Obsidian from inside Zotero? That would be fantastic. The pdfs that are associated with my Zotero entries are already stored in my Obsidian vault.

@rauchert
Apologies for the late response.
Create an Executable File
To get started, you'll need to convert your Python script into an executable file. PyInstaller is a great tool for this, and it works across multiple platforms including Windows and macOS.
Steps:

Ensure you have Python installed on your system.
Install PyInstaller by running pip install pyinstaller in your command prompt (Windows) or terminal (macOS).
Compile your Python script into an executable file:

For Windows: Run pyinstaller --onefile your_script.py in the command prompt.
For macOS: Run pyinstaller --onefile your_script.py in the terminal.
This will create a single executable file (.exe on Windows and .app on macOS).

Configure Zotero

Open Zotero and go to Edit > Preferences.
Navigate to the General tab.
Under File Handling, select Use PDFs with and then choose Custom.
Browse for and select the executable file that you created in the first step:

On Windows, choose the .exe file.
On macOS, choose the .app bundle.

Note: I'm not very familiar with macOS, but theoretically, the above steps should work. However, on macOS, adding a custom executable file might require some additional steps beyond what is described here.
After configuring Zotero, when you double-click on an item in Zotero or right-click to open the file, Zotero will launch the executable you've configured. You may notice that a terminal window briefly appears (it flashes on the screen and then disappears) before the file opens in Obsidian.

@SSmJaE thanks a lot for you answer!
Basically, everything seems to work, except this:
After having chosen my created .app bundle as the standard app for opening pdfs with Zotero and trying to open a pdf, nothing happens and the reason is this:
sys.argv[1], which should be the filepath of the file to open, does not exist.
If I log sys.argv I can see that it contains only one element, namely the path of the app.
Do you have any idea how to fix this and get the required filepath?


Office user can edit weight ticket set weights

Description
Office user can now edit the full and empty weight of a weight ticket set that they uploaded
Setup
make server_run
make office_client_run
Code Review Verification Steps

[ ] Request review from a member of a different team.
[ ] Have the Pivotal acceptance criteria been met for this change?

References

Pivotal story for this change

closing b/c superseded by #2485


Update Realio Network to v0.8.1

Update chain version to v0.8.1
Fix chain pretty name to match entry in slip-0173
Added info on SDK, tendermint and IBC versions

https://github.com/cosmos/chain-registry/pull/2326


Cannot use mltable on Apple Silicon devices

Package Name: mltable
Package Version: 1.1.0
Operating System: MacOs (Apple ARM)
Python Version: 3.9.13

Describe the bug
Following these docs: https://learn.microsoft.com/en-us/azure/machine-learning/how-to-access-data-interactive?tabs=adls
Am attempting to read a file asset, defined as a Data Asset. When I try to install mltable, I receive the error below.
To Reproduce
Steps to reproduce the behavior:

pip install mltable (into venv)

Expected behavior
Successful installation.
Screenshots
Error message:
pip install mltable
Collecting mltable
  Using cached mltable-1.1.0-py3-none-any.whl (170 kB)
Requirement already satisfied: jsonschema<5.0.0,>=4.0.0 in ./.venv/lib/python3.9/site-packages (from mltable) (4.17.3)
Requirement already satisfied: cryptography!=1.9,!=2.0.*,!=2.1.*,!=2.2.*,<41 in ./.venv/lib/python3.9/site-packages (from mltable) (39.0.1)
Requirement already satisfied: pyyaml<7.0.0,>=5.1.0 in ./.venv/lib/python3.9/site-packages (from mltable) (6.0)
Requirement already satisfied: azure-mgmt-core<2.0.0,>=1.3.0 in ./.venv/lib/python3.9/site-packages (from mltable) (1.3.2)
Requirement already satisfied: msrest>=0.6.18 in ./.venv/lib/python3.9/site-packages (from mltable) (0.7.1)
Requirement already satisfied: python-dateutil<3.0.0,>=2.7.3 in ./.venv/lib/python3.9/site-packages (from mltable) (2.8.2)
Requirement already satisfied: azure-core!=1.22.0,<2.0.0,>=1.8.0 in ./.venv/lib/python3.9/site-packages (from mltable) (1.26.3)
Requirement already satisfied: PyJWT<3.0.0 in ./.venv/lib/python3.9/site-packages (from mltable) (2.6.0)
Collecting azureml-dataprep[parquet]<4.9.0a,>=4.8.0a
  Using cached azureml_dataprep-4.8.6-py3-none-any.whl (43.5 MB)
Collecting pytz
  Using cached pytz-2022.7.1-py2.py3-none-any.whl (499 kB)
Requirement already satisfied: six>=1.11.0 in ./.venv/lib/python3.9/site-packages (from azure-core!=1.22.0,<2.0.0,>=1.8.0->mltable) (1.16.0)
Requirement already satisfied: requests>=2.18.4 in ./.venv/lib/python3.9/site-packages (from azure-core!=1.22.0,<2.0.0,>=1.8.0->mltable) (2.28.2)
Requirement already satisfied: typing-extensions>=4.0.1 in ./.venv/lib/python3.9/site-packages (from azure-core!=1.22.0,<2.0.0,>=1.8.0->mltable) (4.4.0)
Collecting azure-identity>=1.7.0
  Using cached azure_identity-1.12.0-py3-none-any.whl (135 kB)
Collecting azureml-dataprep[parquet]<4.9.0a,>=4.8.0a
  Using cached azureml_dataprep-4.8.5-py3-none-any.whl (43.5 MB)
  Using cached azureml_dataprep-4.8.4-py3-none-any.whl (43.5 MB)
Collecting azure-identity==1.7.0
  Using cached azure_identity-1.7.0-py2.py3-none-any.whl (129 kB)
Collecting azureml-dataprep[parquet]<4.9.0a,>=4.8.0a
  Using cached azureml_dataprep-4.8.3-py3-none-any.whl (43.5 MB)
  Using cached azureml_dataprep-4.8.2-py3-none-any.whl (43.5 MB)
  Using cached azureml_dataprep-4.8.1-py3-none-any.whl (43.5 MB)
  Using cached azureml_dataprep-4.8.0-py3-none-any.whl (43.5 MB)
INFO: pip is looking at multiple versions of azure-mgmt-core to determine which version is compatible with other requirements. This could take a while.
Collecting azure-mgmt-core<2.0.0,>=1.3.0
  Using cached azure_mgmt_core-1.3.2-py3-none-any.whl (26 kB)
INFO: pip is looking at multiple versions of azure-core to determine which version is compatible with other requirements. This could take a while.
Collecting azure-core!=1.22.0,<2.0.0,>=1.8.0
  Using cached azure_core-1.26.3-py3-none-any.whl (174 kB)
INFO: pip is looking at multiple versions of <Python from Requires-Python> to determine which version is compatible with other requirements. This could take a while.
INFO: pip is looking at multiple versions of mltable to determine which version is compatible with other requirements. This could take a while.
Collecting mltable
  Using cached mltable-1.0.0-py3-none-any.whl (149 kB)
ERROR: Cannot install azureml-dataprep[parquet]==4.8.0, azureml-dataprep[parquet]==4.8.1, azureml-dataprep[parquet]==4.8.2, azureml-dataprep[parquet]==4.8.3, azureml-dataprep[parquet]==4.8.4, azureml-dataprep[parquet]==4.8.5 and azureml-dataprep[parquet]==4.8.6 because these package versions have conflicting dependencies.

The conflict is caused by:
    azureml-dataprep[parquet] 4.8.6 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.5 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.4 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.3 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.2 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.1 depends on azureml-dataprep-native<39.0.0 and >=38.0.0
    azureml-dataprep[parquet] 4.8.0 depends on azureml-dataprep-native<39.0.0 and >=38.0.0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts

Thanks for the report @Da1ne!
I'm directing this to the team that maintains that packages to take a look!
cc @azureml-github

@Da1ne are you still hitting the issue? if so would you please uninstall mltable from your env and install current version of mltable (1.3.0)?

I'm not the OP, but I am still having this issue with mltable (1.3.0).
INFO: pip is looking at multiple versions of mltable to determine which version is compatible with other requirements. This could take a while.
ERROR: Cannot install azureml-dataprep[parquet]==4.10.0, azureml-dataprep[parquet]==4.10.1, azureml-dataprep[parquet]==4.10.4, azureml-dataprep[parquet]==4.10.5, azureml-dataprep[parquet]==4.10.6 and azureml-dataprep[parquet]==4.10.7 because these package versions have conflicting dependencies.

The conflict is caused by:
    azureml-dataprep[parquet] 4.10.7 depends on dotnetcore2<4.0.0 and >=3.0.0
    azureml-dataprep[parquet] 4.10.6 depends on azureml-dataprep-rslex~=2.17.0dev0
    azureml-dataprep[parquet] 4.10.5 depends on azureml-dataprep-rslex~=2.17.0dev0
    azureml-dataprep[parquet] 4.10.4 depends on azureml-dataprep-rslex~=2.17.0dev0
    azureml-dataprep[parquet] 4.10.1 depends on azureml-dataprep-rslex~=2.17.0dev0
    azureml-dataprep[parquet] 4.10.0 depends on azureml-dataprep-rslex~=2.17.0dev0

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

ERROR: ResolutionImpossible: for help visit https://pip.pypa.io/en/latest/topics/dependency-resolution/#dealing-with-dependency-conflicts


Move code specific to "fsl_full_example" to test file

This PR moves code specific to test case fsl_full_example001 to the test file export_test_battery.py (rather than being in the main fsl_exporter.py function).

This PR only affects the tests.


some doubts about the principle of the diffusion model

Why does the noise addition and denoising process not destroy the normal part, but “assimilate” the abnormal part into the normal part.  Because the model remembers the features of the normal part while training? Then the reasoning process destroys and transforms the abnormal part？

You've pretty much answered your own question.
This is a common method for unsupervised anomaly detection. Where we have a generator which is trained to generate healthy images. So when we encode a new unlabelled image; in theory if it was anomalous it would be decoded such that the model "imagines" what it would look like if it was healthy. Then we can deduce areas of high reconstruction error are consequently anomalous.
So yes, in short, it is because the "encoding" by the forward process learns what noise is in healthy images, but when reversing this process any anomaly would be considered as part of this noise and removed in the denoising process.


chore: jsonfmt EnterpriseKnowledgeGraph

Ran jsonfmt -w "specification/EnterpriseKnowledgeGraph/**/*.json"

Automation for azure-sdk-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-js

Automation for azure-sdk-for-go

Can one of the admins verify this patch?


feat: spinnter fix search page

Description
Fixed spinner animation to be "static" (before that it was moving and circling same time)
Issues
Closes
DEV-XXX:
Related
Testing
Automated tests
Manual testing
Screenshots
Additional notes

Codecov Report

Merging #61 (a13552c) into main (21d6305) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   81.30%   81.30%
=======================================
  Files         141      141
  Lines        2728     2728
  Branches      814      814
=======================================
  Hits         2218     2218
  Misses        508      508
  Partials        2        2

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


Suggestion: Add support for customer, yourref, yourmail like in scrlttr2 / LaTeX

Hello,
so far I have been using this Latex-Briefvorlage nach DIN 5008 letter template.
It offers this options:

e.g.:
% Kundennummer
%\setkomavar{customer}[\customername]{DE-112233}

% Ihr Zeichen
%\setkomavar{yourref}[\yourrefname]{IZ-12345}

% Ihr Schreiben vom
%\setkomavar{yourmail}[\yourmailname]{1. April 2018}

in LaTeX.

Would it be possible to also add something like this to this awesome typst letter template?
Kind regards.

Hi,
Thanks for the kind words!
Indeed I have a temporary state of this project with such a table. I just pushed it onto the next branch with:
    horizontal_table: (
        "Ihr Zeichen": "Bananalover149",
        "Ihre Nachricht vom": "12.12.2022",
        "Unser Zeichen": "BananaFactory",
        "Datum": "06.08.2023",
    ),

I'm not quite sure yet about the spacing of it as it seems a bit far down but moving it up would violate the DIN norm.

Merged. Feel free to reopen if you wish to discuss this further.

Thank you! :)

Thank you, it looks fantastic!


[Event Request] - WhsePostShipment.Codeunit - OnPostWhseJnlLineOnAfterSplitWhseJnlLine

We would like to request the following event:
Object: WhsePostShipment.Codeunit
Where is the event placed:
local procedure PostWhseJnlLine(var PostedWhseShptLine: Record "Posted Whse. Shipment Line"; var TempTrackingSpecification: Record "Tracking Specification")
var
    ...
begin
    ...
    GetLocation(PostedWhseShptLine."Location Code");
    if Location."Bin Mandatory" then begin
        CreateWhseJnlLine(TempWhseJnlLine, PostedWhseShptLine);
        CheckWhseJnlLine(TempWhseJnlLine);
        OnBeforeRegisterWhseJnlLines(TempWhseJnlLine, PostedWhseShptLine);
        ItemTrackingMgt.SplitWhseJnlLine(TempWhseJnlLine, TempWhseJnlLine2, TempTrackingSpecification, false);
        OnPostWhseJnlLineOnAfterSplitWhseJnlLine(TempWhseJnlLine, PostedWhseShptLine, TempTrackingSpecification, TempWhseJnlLine2);
        if TempWhseJnlLine2.Find('-') then
    ...
end;

Event Code:
[IntegrationEvent(false, false)]
local procedure OnPostWhseJnlLineOnAfterSplitWhseJnlLine(var TempWhseJnlLine: Record "Warehouse Journal Line"; var PostedWhseShptLine: Record "Posted Whse. Shipment Line"; var TempTrackingSpecification: Record "Tracking Specification"; var TempWhseJnlLine2: Record "Warehouse Journal Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Fixed a bug which would be caused when 'inlinestyles=None' or 'linens=None'.

When I used the old codes, if I did not pass the value of inlinestyles or linenos, either of them were with the value of "None", and then would caused the bug as below.
Traceback (most recent call last):
File "env/lib/python3.5/site-packages/mistune_contrib/highlight.py", line 28, in block_code
noclasses=inlinestyles, linenos=linenos
File "env/lib/python3.5/site-packages/pygments/formatters/html.py", line 388, in init
self.noclasses = get_bool_opt(options, 'noclasses', False)
File "env/lib/python3.5/site-packages/pygments/util.py", line 58, in get_bool_opt
string, optname))
pygments.util.OptionError: Invalid type None for option noclasses; use 1/0, yes/no, true/false, on/off

Better with self.options.get('inlinestyles', False)


chore(package): update rollup to version 0.45.0

Closes #23

Codecov Report

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

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   32.53%   32.53%
=======================================
  Files           2        2
  Lines          83       83
  Branches       15       15
=======================================
  Hits           27       27
  Misses         56       56

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 f910291...6877987. Read the comment docs.


Feature: Additional Simple Shapes Intersection Tests

Implement following:
https://github.com/Godlike/Pegasus/issues/12
https://github.com/Godlike/Pegasus/issues/13
https://github.com/Godlike/Pegasus/issues/14
https://github.com/Godlike/Pegasus/issues/15

https://github.com/Godlike/Arion/projects/1 Migration.


Separately categorise FTL runtime logs from user logs

We need to categorise logs that are part of the runtime differently to those from the service.  You can see an example here, where there's 50 lines of "Downloading ..." drowning out any logs from the service itself.

FTL and PFI logs are colocated in Datadog and it is possible but not the easiest to disambiguate.

Explicitly tag FTL logs from both the controller and runners.
Consider using tags for deployed modules (currently event attributes)
Consider an overall tag for module events/logs.
Consider a mechanism for a team like PFI to take all of their modules as "PFI". Currently to capture PFI logs in Datadog requires declaring all modules in the filter criteria.

We just added project names to ftl-project.toml, so for 4 we should use that.


feat(run): match tasks to run with glob patterns

Summary
Closes #1271
This change make it possible to select multiple tasks to run using glob style matching.
Task topics, separated by :, are treated like directories in shell matching such that patterns like a:**:d will match tasks named a:d, a:b:d or a:b:c:d.
Additional Notes

This brings globset in as a direct dependency, but it was transitive already.
Used an Extension Trait since tasks_with_aliases is used in multiple places making a more local change preferable though it may make sense to move this trait or change the return type of tasks_with_aliases in the future.

clever solution using the glob here. I was just going to have a naive wildcard match

this could happen later, but this code should probably be updated to use glob logic as well:
https://github.com/jdx/mise/blob/59c721690fc7fe5ae5f54e3a530f1523114bdba0/src/task.rs#L137

this could happen later, but this code should probably be updated to use glob logic as well:
https://github.com/jdx/mise/blob/59c721690fc7fe5ae5f54e3a530f1523114bdba0/src/task.rs#L137

I can pick that up in a follow up.


It doesn't work with C* 2.1.5

In this line method get returns a null which leads to NPE.

2.1 and 2.2 have a sstable2json tool built in. In 3.0 with storage engine changes things changed so much they just dropped the tool.  This tool targets 3.x+. Previous versions won't work unfortunately as the api to access the data changed so drastically. I would recommend using your bin/sstable2json for now.

Oh, I see. Thanks.


Add File Upload support to Federation plugin

Today RemoteDataSource(the default HTTP executor for subgraphs) doesn't support File Uploads. This can be replaced by @graphql-tools/executor-http.

Current docs use GraphQL Tools which supports File uploads already.


Minor - Adjust regex in Objects/Process.py to avoid deprecation

Describe the bug
tm1py/TM1py/Objects/Process.py:24: DeprecationWarning: Flags not at the start of the expression '#\\*\\*\\*\\*Begin: Gene' (truncated)

Triggered by https://github.com/cubewise-code/tm1py/blob/master/TM1py/Objects/Process.py#L23
To Reproduce
pytest Tests/Application.py
Expected behavior
We should address the deprecation warning
Version
master branch
Additional context
Came up while looking at the tests for the application service.
Googling Flags not at the start of the expression  suggests that it might be as simple as changing:
pattern = r"#\*\*\*\*Begin: Generated Statements(?s)(.*)#\*\*\*\*End: Generated Statements\*\*\*\*"

To:
pattern = r"(?s)#\*\*\*\*Begin: Generated Statements(.*)#\*\*\*\*End: Generated Statements\*\*\*\*"

I can see that regex is trying match to the auto generated block but not sure what the (?s) is trying to do. Maybe someone with a better understanding of regular expressions can help?

(?s) is the DOTALL flag in REGEX
It is useful when the pattern you are searching for may span across multiple lines.
Basically it can match all characters including line breaks.

Cool, thanks @rkvinoth. Can the (?s) be safely moved to beginning of the line then? If I've understood correctly, it shouldn't make a difference when trying to match the generated statement block. What do you think ?

With the new REGEX we are saying that the below statement could span across multiple lines in the new REGEX.

#****Begin: Generated Statements***
#****End: Generated Statements****

With the old REGEX we were saying that there could be one or more line breaks after #****Begin: Generated Statements*** and before #****End: Generated Statements****

Lets go with the new approach:
pattern = r"(?s)#\*\*\*\*Begin: Generated Statements(.*)#\*\*\*\*End: Generated Statements\*\*\*\*"

Thanks @rkvinoth
Note I just realised I didn't copy and paste the code properly, have adjusted my comment now. That is, the original statement has the ?s but halfway through the string.
For clarity, the line I'm talking about is here:
https://github.com/cubewise-code/tm1py/blob/7478943c8fbdcfbdcc701b40cc7dd3b6f9e1d0b3/TM1py/Objects/Process.py#L23
I'll do ad hoc testing to make sure it does what it's supposed to.
Cheers
Alex

I did notice the code link you had posted initially.
The REGEX flag should be placed at the start of the expression..
Since we haven't, it is throwing the error.

Cool, thanks. I'll raise a PR, just didn't want to break anything just to suppress a warning I'm seeing on 3.8.
Cheers
Alex


Raw wpm not being saved properly?

When working on the stats command, I noticed all the raw wpm columns have the same value as the wpm column. Is raw words per minute being calculated correctly?

I was actually just printing the incorrect field when printing columns. Raw wpm is saved correctly.


fix: handle null target instances passed to getPropertyDescriptor

This PR seeks to fix https://github.com/ember-decorators/argument/issues/98.
-private/utils/object.js#L10 was erroring because target passed to getPropertyDescriptor was in some cases null, not undefined.
export function getPropertyDescriptor(target, property) {
  if (target === undefined) return;
  // target = null...
  return (
    Object.getOwnPropertyDescriptor(target, property) ||
    getPropertyDescriptor(Object.getPrototypeOf(target), property)
  );
}

Thanks for submitting a PR for this! I actually came online just now to take a pass at solving it, but approving a PR is always better 😁
Did you try linking this locally and find that it solved the problem for you?

@alexlafroscia The problem I've described in the https://github.com/ember-decorators/argument/issues/98 is resolved with this PR, everything is working great.
@willviles Thanks!

@alexlafroscia yep, linked and it's removed all errors 👍 Not entirely sure whether this PR is the best possible solution, as it's just firefighting methodology. But in the mean time, it solves the problem!

That's OK as far as I'm concerned -- I'm unsure why I haven't come across this case where the prototype is null in my own codebases that are using this package without error right now.
Are you transpiling away native classes? We ship them in our codebase -- maybe that's the difference?

Yeah I'm shipping ES5, though I've just tried 1.0.0-beta.2 without transpiling native classes and the problem persists 🤷♂

Out of interest, should default values still be respected when using the optional type? Here's what I'm getting:
// component.js
@argument('string')
foo = 'defaultValue'

{{! parent-template.hbs}}
<Component />

{{! component-template.hbs}}
{{log foo}} {{! => undefined}}

@willviles Same for me, I'll try to come up during this week with all the problems I'm having.
I'm using es classes with angle bracket invocation syntax and sometimes I'm receiving unexpected errors(probably not because of angle invocation syntax feature, but I've not tried with the regular way)


Correction/documentation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: UgoWithoutH:x: bgdu13You have signed the CLA already but the status is still pending? Let us recheck it.


Support EtherAddr and impl Debug for InterfaceAddress/SockAddr

Add EtherAddr .
impl Debug for InterfaceAddress and SockAddr.

This is a lot of changes, and the commits don't really tell me what has happened here. Each discrete change should really be a separate commit with a description that provides context as to why it's a desired change. I can't really follow what all is being changed here easily, so I'd ask you split this up into separate commits that can evaluated individually.

@Susurrus could you review again ? thx.

You have 2 commits with identical names and one commit with no name. Neither of which have a description.
Please break off the impl Debug changes from your changes that add EtherAddr at a minimum. The goal is to make code changes as small as possible will still being understandable. I have no context for what these changes are about and there seems to be a bunch of changes that aren't necessary, so this is quite hard for me to wrap my head around.

So it looks like there are a lot less changes now, so not sure what happened there.

@Susurrus this PR https://github.com/rust-lang/libc/pull/866 has merged on libc .
so i will remove struct sockaddr_dl in sys/socket/ffi.rs .
and cloud i rebase all change into one commit ?

The goal with the commits is to have changes that are as small and self-contained as possible while still being readable. So in this case I'd like to see two commits: 1) implement Debug for all the types and 2) add the EtherAddr implementation.
I'd also appreciate any tests or example code that could be easily written. This will give us a sense of how this API ends up being used and make sure we don't break anything during major refactorings.

@Susurrus ping ...

easy merge:
git merge 4e7a6ecd2b6fbbb7e0e6de40b93e9da7ccb8a034  # impl `Debug` for all sockaddr types
git merge 640e2fe5fadd4c28a497248f98fd0c7aece6c700  # Impl EtherAddr(MAC Addr)
git merge c3d3ea6062a415da75901d70709dc6d7d43ee618  # Add example for list interfaces

You should be rebasing on top of master, squashing your commits to just 3 as you do, and then force-push into this branch. So it'd be something like:
$ git fetch upstream
$ git rebase -i upstream/master

And on the commits you want to squash, change the pick to a s (for squash). It'll then ask you to modify the commit message if there wasn't a merge conflict. This link describes some of these actions.
Given where you're at, you'll likely want to git fetch upstream && git reset upstream/master to get back to a sane state. Then just make your two commits using git add --patch to choose just the right parts, and the git push origin -f to overwrite the branch this PR is tracking.

@Susurrus wow, that's magic 👍

Awesome, nicely done! Git can be tricky to use, and even harder to explain, I'm surprised what I said helped!
Anyways, let's remove that last commit as that example isn't very useful. And this will also need a CHANGELOG entry.

@Susurrus hi, need any changes ?

@Susurrus updated.

@LuoZijun This isn't passed CI.


Utilities updates

@moonso I told you this was done already but apparently not merged lol
I tried to rebase as best as I could
This will fix the resource-stuff :)

But we can not merge when tests are failing?

of course, I'll check on it


Odd output for "Caused by"

I'm seeing some confusing output when a spec fails after upgrading from 3.3 5o 3.4. For the most part everything is the same, but the new Caused by section says Errno::ECONNREFUSED. Is there some setting I'm missing?
Failures:

  1) Admin::AdminController#ensure_admin! when the user is an admin does nothing
     Failure/Error: expect(flash[:error]).not_to be_nil

       expected: not nil
            got: nil
     # ./spec/controllers/admin/admin_controller/ensure_admin_bang_spec.rb:22:in `block (3 levels) in <module:Admin>'
     # /home/fletch/.rvm/gems/ruby-2.1.7/gems/activesupport-4.2.5/lib/active_support/dependencies.rb:268:in `load'
     # /home/fletch/.rvm/gems/ruby-2.1.7/gems/activesupport-4.2.5/lib/active_support/dependencies.rb:268:in `block in load'
     # /home/fletch/.rvm/gems/ruby-2.1.7/gems/activesupport-4.2.5/lib/active_support/dependencies.rb:240:in `load_dependency'
     # /home/fletch/.rvm/gems/ruby-2.1.7/gems/activesupport-4.2.5/lib/active_support/dependencies.rb:268:in `load'
     # /home/fletch/.rvm/gems/ruby-2.1.7/gems/spring-commands-rspec-1.0.4/lib/spring/commands/rspec.rb:18:in `call'
     # -e:1:in `<main>'
     # ------------------
     # --- Caused by: ---
     # Errno::ECONNREFUSED:
     #   Connection refused - connect(2) for "<IP_ADDRESS>" port 8989
     #   /home/fletch/.rvm/gems/ruby-2.1.7/gems/activesupport-4.2.5/lib/active_support/dependencies.rb:268:in `load'

That is a bit odd.  The Caused by: section was added in #2074.  It uses Exception#cause (a new feature in Ruby 2.1).  In your case, the Errno::ECONNREFUSED exception was somehow exposed as the cause from the RSpec expectation failure exception.  I have no idea how that could occur -- off the top of my head, it seems likely to be a bug in Ruby.
Could you provide an executable example of the problem?

@mockdeep are you using selenium, poltergeist, or capybara-webkite or the like?

@JonRowe Yeah, we're using capybara with selenium, though these are controller specs, which shouldn't be touched by capybara.
I tried it out in another application with the same rspec version and it works as expected, so I'll see if I can narrow down the issue. It's odd because I wouldn't expect a cause to be displayed at all here, since no error was raised...

Yes thats a sideeffect of how RSpec works, we use exceptions for test failures, and in this case the cause is displayed as it was present for some reason

Okay, figured it out. We have --drb in the .rspec file of the project that is behaving strangely. Removing it made the Caused by go away, and when adding it to the other project I saw the same issue. After commenting out huge chunks of our spec_helper and Gemfile the error eventually reduced to this:
Failures:

  1) Admin::AdminController#ensure_admin! when the user is an admin does nothing
     Failure/Error: expect(flash[:error]).not_to be_nil

       expected: not nil
            got: nil
     # ./spec/controllers/admin/admin_controller/ensure_admin_bang_spec.rb:22:in `block (3 levels) in <module:Admin>'
     # ------------------
     # --- Caused by: ---
     # Errno::ECONNREFUSED:
     #   Connection refused - connect(2) for "<IP_ADDRESS>" port 8989
     #   /home/fletch/.rvm/rubies/ruby-2.1.7/lib/ruby/2.1.0/drb/drb.rb:839:in `initialize'

We're not using a drb server, so for now will just remove the configuration.

Thanks for figuring out that the --drb flag was part of it!
I figured out the problem and fixed it in #2156.

Great, thanks!