A bug report about uploadFile in kintone/rest-api-client

Target Package

https://unpkg.com/<EMAIL_ADDRESS>
Environment

Operating System: MAC OS
Browser: Chrome

Current Behavior

I upload the file to kintone through new KintoneRestApiClient().file.uploadFile(file), and then add the returned fileKey to the kintone attachment field,
Check the "Content-Type" of the attachment field in record event.
The "Content-Type" is normal when I upload png or text file
When I upload a pdf or javascript file, the "Content-Type" becomes "application/octet-stream".
We get the js file object through the "onChange" event of , and then upload it to kintone through the "uploadFile" method of kintone-rest-api-client.
Expected Behavior

When I upload a pdf or javascript file, the "Content-Type" will be same as 'pdf/javascript'

I uploaded file with XMLHttpRequest  and set file type  when convert file to blob object.
The “content-type” in kintone record  was correct.
Maybe we should set file type when convert to blob object in intone-rest-api-client.
Thank you very much!

@h000780
Thanks for your reporting.
We are checking your issue and will reply to you soon.

@h000780
Thank you for your patients.
We decided to fix this issue by supporting auto-detect MIME-type.
You can follow us at https://github.com/kintone/js-sdk/issues/1893

@h000780
This issue is fixed by PR #1893

The fixing PR was released in<EMAIL_ADDRESS>


Feature/update module overview

This PR adds a description of the abiotic_simple_model to the module overview.
I don't remember what we agreed regarding module/ model, I thought is was model but it is a bit mixed in this file. I am happy to make the changes with your ok.
Type of change

[ ] New feature (non-breaking change which adds functionality)
[x] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Codecov Report

Merging #224 (bea675a) into develop (531c93d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #224   +/-   ##
========================================
  Coverage    95.18%   95.18%
========================================
  Files           35       35
  Lines         1494     1494
========================================
  Hits          1422     1422
  Misses          72       72

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


Does DeepFace support to analyse the attributes of the images which contains many faces

Hi, when I use DeepFace to analyse the images, I find that the code returens  only one face attributes, but the image contains several faces, so I wonder if the DeepFace supports to analyse the attributes of the images which contains many faces?

it is designed to analyze just a face in an image

@Hwang64 You can use Dlib face detected or some similar library to detect faces, and crop them out of the image in separate images. After that you can send those cropped face images to deepface for analysis.

as @hardiknahata mentioned, it can be handled as shown below.
from deepface import DeepFace
from retinaface import RetinaFace
faces = RetinaFace.extract_faces("img1.jpg")

for face in faces:
     obj = DeepFace.analyze(face)
     print(obj)


Merge #264

@zachborboa - Can you add the braces check and @author check to your tests so I can handle that prior to PR?

@thebeline Will do

For what it is worth, ^[ \t]*(?![ \ta-z]*[ \t]+)?function(?![ \t]+[ \ta-z]*)?[ \t]+[a-z_]+\([^\)]+\)[ \t]*{ will detect opening brackets on the same line as the method declaration.

Also, it is rudimentary, but /^[ \t]+\/\*\*((?!\*\/).)*\n[ \t]+\*[ \t]+@author((?!\*\/).)*\*\//ism will detect any DocBlock that contains an @author tag.
May or may not help.


[ZEPPELIN-1122] BugFix for repositories snapshot is always set to true.

What is this PR for?
This PR fixes a bug about snapshot setting and select box(true,false) html.
What type of PR is it?
Bug Fix | Improvement
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-1122
How should this be tested?

go Interpreter menu.
click repository information button that is right top.
click plus button (to Add New Repository)
and refer to screenshot.

Screenshots (if appropriate)

before

after

Questions:

Does the licenses files need update? no
Is there breaking changes for older versions? no
Does this needs documentation? no

The name "Snapshot" that presented on popup before is not snapshot information but the "Enable" flag for the associated repository information.
So I removed the snapshot from popup.

Thank you for fixing!  Looks like CI fails dues to https://github.com/npm/npm/issues/13284
Looks good to me, merging if there is no more discussion

Looks great to me, thank you, CI passes now.

@astroshim can you please next time set "Fix Version" in JIRA and make sure issue is assigned to you?
This helps release manager a lot!

@bzz oh I see. Thank you reminding me.


[SPIKE] Understand under the hood functionalities

Research into this repo

What do we need to work on
Update themeing? Existing bugs
Icons? Branding
https://github.com/opensearch-project/oui/issues/22

Requirements:

Need a roadmap
Get user interaction information (how is this information collected)

@kaddy645 @bandinib-amzn @AMoo-Miki @opensearch-project/opensearch-ux

Working version of the roadmap for theming here: https://quip-amazon.com/pacoA2FVtJwg/Theming-Roadmap-Draft

This issue is out of date, and no longer needed. Closing.


Tuple.new discards type information

Tuple.new always creates a new tuple type based on the exact (narrowest) types of its arguments, even if it's called on a specific Tuple subtype.
Tuple(Int32, Int32).new([1,2,3], "test")
puts typeof(foo)

# {Array(Int32), String}

alias MyValue = Int32 | Float64
alias MyTuple = {MyValue, MyValue}

def test(value : MyValue)
  MyTuple.new(5, value)
end

puts typeof(test(6.0))

# {Int32, Float64}

Thanks to Tetsumi from IRC for finding this

There isn't a way to implement this right now. We'll probably need to have a way, at compile time, to know if the tuple is the generic type or an instantiation. If it's an instantiation we type check (we probably can use the cast method of types), otherwise we infer the types (the current approach).

And probably related to #1811. I guess if we do this then for that issue we could use:
thing, n = Tuple(String, Int64).new(JSON.parse(%(["world", 2])) as Array)

I decided to implement this by allowing a splat restriction on a splat argument. So, after the next release (we can't do it right now because there's no syntax for this) we can write:
struct Tuple
  def self.new(*args : *T)
    args
  end
end

tup = Tuple.new(1, 2) # OK

Tuple(Int32, Int32).new(1, 2) # OK
Tuple(Int32, Int32).new(1, 'a') # Error

alias Point = {Int32, Int32}

Point.new(1, 2) # OK
Point.new(1, 'a') # Error

Another example, outside of tuple:
def foo(*args : *T)
  T
end

t = foo 1, 'a'
t # => {Int32, Char}

Basically, a *T restriction (which is only allowed on a splat argument, for now) will put the matching types into a tuple. I think this is kind of intuitive and similar to what * does for an argument. In any case, the main use for this is Tuple.new.
In fact, one can also do:
def foo(*args : *{T, U})
  p T
  p U
end

foo 1, 'a'
# prints Int32 and Char

though of course there's no real point in doing that, better to specify two arguments with a restriction each.
I also did the same for NamedTuple and double splats:
struct NamedTuple
  def self.new(**options : **T)
  end
end

alias Point = {x : Int32, y: Int32}

Point.new x: 1, y: 2 # OK
Point.new y: 1, x: 2 # OK
Point.new x: 1, y: 2, z: 3 # Error
Point.new x: 1, y: 'a' # Error

def foo(**options : **T)
  T
end

t = foo x: 1, y: 'a'
t # => {x: Int32, y: Char}

I believe this is a good solution to this problem because it's a general solution and kind of intuitive :-)
Relevants commits:

https://github.com/crystal-lang/crystal/commit/2ea1c100d8c8a127baea062caa547e3a38f2b0d2
https://github.com/crystal-lang/crystal/commit/d9706c7e738179b10091e87b7f524554038d9dc3

@asterite, you're not a 10Xer, you're more like a 100xer!


show default values in prompts

when a user hits a prompt like
ArDrive Wallet Backup Folder Path:
it would be nice to show instead:
ArDrive Wallet Backup Folder Path (hit enter for current dir):
since thats how it acts by default

This does not apply to current CLI version.
Closing


CodeQL Action Workflow

Describe the contribution
Implemented CodeQL Action as a workflow to automatically detect common vulnerabilities and coding errors.
Expected behavior changes
Results should be displayed in the Security tab under Code Scanning Alerts. The workflow is triggered when code is pushed to any branch in a repository and during pull request events.
Additional context
For more information visit: https://github.com/github/codeql-action
Contributor Info - All information REQUIRED for consideration of pull request
Ariel Adams, ASRC Federal

Do you have a sample of what our report looks like? I can't find it on your fork.

Do you have a sample of what our report looks like? I can't find it on your fork.

Do you have a sample of what our report looks like? I can't find it on your fork. Is there a configuration file for CodeQL?

CodeQL results  will be found in the code scanning alerts section of the security tab. The configuration file for CodeQL is the codeql-build-cfs.yml file.

Do you have a sample of what our report looks like? I can't find it on your fork. Is there a configuration file for CodeQL?

CodeQL results  will be found in the code scanning alerts section of the security tab. The configuration file for CodeQL is the codeql-build-cfs.yml file.

Cool!
I couldn't access your link but I was able to see the list at
https://github.com/nasa/cFS/security/code-scanning?query=ref%3Arefs%2Fpull%2F165%2Fmerge

Cool!
I couldn't access your link but I was able to see the list at
https://github.com/nasa/cFS/security/code-scanning?query=ref%3Arefs%2Fpull%2F165%2Fmerge

For the configuration file, I meant something like this: https://github.com/github/codeql-action#configuration-file so we can specify which vulnerabilities to look for.

For the configuration file, I meant something like this: https://github.com/github/codeql-action#configuration-file so we can specify which vulnerabilities to look for.

Opened #175. Please Update commit format and PR title to reflect that this PR closes that issue.

Opened #175. Please Update commit format and PR title to reflect that this PR closes that issue.

Opened #175. Please Update commit format and PR title to reflect that this PR closes that issue.

@astrogeco Completed.

Opened #175. Please Update commit format and PR title to reflect that this PR closes that issue.

@astrogeco Completed.

Looks fine to me, but I haven't used CodeQL.

CodeQL is the engine behind LGTM so we're technically using it.

Looks fine to me, but I haven't used CodeQL.

CodeQL is the engine behind LGTM so we're technically using it.

@ArielSAdamsNASA can you add a screenshot of what the code report looks like. It might be useful for the CCB

@ArielSAdamsNASA can you add a screenshot of what the code report looks like. It might be useful for the CCB

CCB:2021-01-21 APPROVED

CCB:2021-01-21 APPROVED


Make Fin.t finType instance transparent + small lemmas

This contains a few small additions that my Bachelor's project depends on.
The most relevant one is to make the definition of the finType instance of Fin.t transparent, which is needed to prove that the cardinality of Fin.t n is actually n.

Hi, any updates on this? Could this be merged?

Happy to merge. Thanks Lennard!

Do we need a new opam release for this?

Thanks!

Do we need a new opam release for this?
If this can be done, it would be great. We're looking into merging my repo into Fabian's, so I need to get it to build with the opam version of the base library.

Do we need a new opam release for this?

Would you mind releasing this on opam?

I'm not against releasing on opam.
However, my preference would be that we only release new versions of the base library on opam once they are required by other opam packages. This way coq-undecidability-library has to be tested against one version of base-library only.
Once there is a coq-complexity-library package we can release the new base-library and bump the requirement in coq-undecidability-library as well. As long as coq-complexity-library is still in a pre-alpha phase you probably can work with README files which tell a user which versions of which dependency to call make install on?

If that doesn't work for you, a good compromise might also be to introduce a base-library+dev version which just installs from the current status of the branch.


build: sonarqube

Added sonarqube back in now we have a comparison branch

yarn run v1.22.18
$ eslint .
/home/jenkins/workspace/_org_ui-service-interaction_PR-2/project/src/index.js
1:16  error  'actAs' is missing in props validation  react/prop-types
/home/jenkins/workspace/_org_ui-service-interaction_PR-2/project/src/public/NumberGeneratorButton.js
5:3  error  'id' is missing in props validation  react/prop-types
/home/jenkins/workspace/_org_ui-service-interaction_PR-2/project/src/public/utilities/ModalButton.js
2:8   warning  'PropTypes' is defined but never used. Allowed unused vars must match /React/u                      no-unused-vars
3:1   error    'dom-helpers' should be listed in the project's dependencies. Run 'npm i -S dom-helpers' to add it  import/no-extraneous-dependencies
6:3   error    'id' is missing in props validation                                                                 react/prop-types
7:3   error    'renderModal' is missing in props validation                                                        react/prop-types
8:3   error    'renderTrigger' is missing in props validation                                                      react/prop-types
29:17  error    'onClose' is missing in props validation                                                            react/prop-types
30:15  error    'onClose' is missing in props validation                                                            react/prop-types
✖ 9 problems (8 errors, 1 warning)
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.


add error code and message when customcommand end with exception

Codecov Report

Merging #301 into release-rc4 will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@                Coverage Diff                @@
##             release-rc4     #301      +/-   ##
=================================================
- Coverage          21.76%   21.75%   -0.02%
  Complexity           375      375
=================================================
  Files                129      129
  Lines               5085     5089       +4
  Branches             379      379
=================================================
  Hits                1107     1107
- Misses              3912     3916       +4
  Partials              66       66

Impacted Files
Coverage Δ
Complexity Δ

...ecross/network/rpc/handler/ResourceURIHandler.java
3.64% <0.00%> (-0.11%)
2.00 <0.00> (ø)


Pet not found

Sometimes, when searching for a pet, event though the pet is on the website, Petfinder API says that the pet is not found
Example:
https://www.petfinder.com/dog/ms-wiggles-52649180/la/lafayette/lafayette-animal-shelter-care-center-lascc-la40/
pf.animal.show(52649180).then(function(response) {     console.log(response); }).catch(function(error) {     console.log(error); });

Petfinder allows organizations to de-list their animals from the api.
#16


How to set priority of the application running on GPU?

I want to test the job preemption on Mali GPU, and I find the cl_khr_priority_hints extension on OpenCL, but my GPU (Mali-G71) does not have this feature. Is there any way in armnn to set the priority of a GPU application? Or, is there any where else I can ask for the answer?

Hello again.
The GPU is a black box to Arm NN. We interact with it through Arm Compute library.
Maybe you could ask the question over there: https://github.com/ARM-software/ComputeLibrary/issues
Colm.

Thanks Colm, I will try to ask the question in ACL's repo.


[LW-10080] Single delegation now opens drawer on transition and fetches pools

Checklist

[x] JIRA - LW-10080
[ ] Proper tests implemented
[x] Screenshots added.

Proposed solution
Remove the check that returns if it's a hardware wallet so we can still fetch the pools.
Search the by id when searching for a pool id so it returns the correct pool ID and the flow continues in a full screen.
Testing

Run the app with a hardware wallet
Go to pop up view and try to stake
Switch to full screen and see the drawer opened with the delegation you started in the popup view
Go back to pop up view and check that the search and stake pools in delegation tab have the correct number of pools (274 in preprod) and the search works.

Screenshots

coverage: 50.595% (+0.03%) from 50.567%
when pulling 0e85f273160b41cdbfc2f062b8e524442ecb92e2 on fix/lw-10080-ledger-browser-history-lost
into e8a9096cf2da863f8e75f0933ea9c709589ac0eb on main.


Windows Security Threats Warning

Windows Security Anti-virus gives warning; Threats found: Trojan:Win32/Wacatac.B!ml

Also seeing this from virus total.

I also received this warning from my Firewall yesterday.

Weird, it seems like it targets the TaskbarX.exe on @arslanmuhsin and the Taskschedule on @vMambaaa .
It's clear that it's an automatic detection not specifically TaskbarX targeted. It may not like MSAA. I can try to contact Microsoft about this but, that won't fix it for a next update.

Weird, it seems like it targets the TaskbarX.exe on @arslanmuhsin and the Taskschedule on @vMambaaa .
It's clear that it's an automatic detection not specifically TaskbarX targeted. It may not like MSAA. I can try to contact Microsoft about this but, that won't fix it for a next update.

Had the same issue as well. So, can we just tell the antivirus to ignore it?

@halkliff Add the TaskbarX.exe or just it's whole directory to Windows Defenders exclusions.

This is what it says when I open the configurator

This is what it says when I open taskbarX.exe
i extracted the zip file and opened these applications
if that's the problem i can't find the setup for taskbarx as it was not in the zip file

i run windows 10 home edition

it also says the same thing about the win32 trojan virus on my windows defender

got virus error on defender Trojan:Win32/Emali.A!cl for version <IP_ADDRESS>

so what do we do about this

@PhantomX165 The best thing to do is to submit a false positive sample to Microsoft or for the AV your'e using.
I already did this for couple including Defender. Or just add TaskbarX to your exclusions.


SB3-zoo has models for v1 tasks but only registration specs for v2 tasks

Hi, I'm trying to get a SB3 model to train the harder tasks (so far I've failed with SAC+HER), so I went to SB3-zoo to see some examples of successful models. I can't get them to load, and it looks like its because zoo has trained models for the v1 versions, but only has the v2 environments registered. Do you have successful trained V2 models you can push to zoo / did you successfully train any SB3 models out of the box on the v2 versions of the task?
thanks for making this environment!

FYI you can get around this by replacing the "1" with a "2" in the env_id after the model is loaded in enjoy.py for the "ExperimentManager.is_atari" check and then in the "create_test_env" call.

Indeed.
First of all, this issue would be more appropriate in sb3-zoo than here. I will answer it anyway.
I am updating the sb3-zoo baselines for the v2 version of panda-gym. I should open a PR on sb3-zoo soon. I will put the link to the PR here when it is opened.
For the tasks already present in v1, the changes are quite small : it's mostly about some physical parameters like friction. So it should be possible to just replace v1 by v2 in the model loading without major trouble.
Be careful though, if you do that, you will necessarily get worse results than if you use a model trained with v2
Translated with www.DeepL.com/Translator (free version)

https://github.com/DLR-RM/rl-baselines3-zoo/pull/244


Remove chainer_setup_build.py

This PR removes an artifact file chainer_setup_build.py, which is no longer used in Chainer v2.

thank you


Bug Report: Libitina crashes in several places on Linux

Bug Description
When playing Libitina on Linux, the song will hard crash shortly after beginning, but before the song itself properly starts. Error below:
Error : [openfl.display.Shader] ERROR: Error compiling fragment shader
0:11(12): warning: extension `GL_NV_non_square_matrices' unsupported in fragment shader
0:302(9): error: syntax error, unexpected NEW_IDENTIFIER, expecting ',' or ';'

-- Complete dump of vertex shader file removed for brevity, it seemed to be a shader for a glitch effect.

I see it's a shader issue, so I disable shaders in the settings. This bypasses the initial crash and allows the song itself to be played. Some errors pop up in the console at this point, but it doesn't seem to affect the game just yet.
[00007f6fe0c1eec0] main decoder error: Timestamp conversion failed (delay 1000000, buffering 100000, bound 9000000)
[00007f6fe0c1eec0] main decoder error: Could not convert timestamp<PHONE_NUMBER>8 for FFmpeg
[h264 @ 0x7f6fe0cc23c0] get_buffer() failed
[h264 @ 0x7f6fe0cc23c0] thread_get_buffer() failed
[h264 @ 0x7f6fe0cc23c0] decode_slice_header error
[h264 @ 0x7f6fe0cc23c0] no frame!

However, once the song finishes, the game will crash once more, this time with a proper error popup instead of closing immediately.
hxcodec/vlc/VLCBitmap.hx (line 248)
PlayState.hx (line 2662)
flixel/FlxGame.hx (line 629)
flixel/FlxGame.hx (line 729)
flixel/FlxGame.hx (line 682)
flixel/FlxGame.hx (line 556)
openfl/events/EventDispatcher.hx (line 402)
openfl/display/DisplayObject.hx (line 1399)

Uncaught Error: Null Object Reference
Please report this error to the GitHub page: https://github.com/Jorge-SunSpirit/Doki-Doki-Takeover

> Crash Handler written by: sqirra-rng

System Details

OS: Arch Linux
Kernel: 6.2.2-arch1-1
DE: Plasma Wayland 5.27.2
GPU: Nvidia RTX 2070 Max-Q
Graphics Driver: 525.89.02-6

Interesting, the shader compilation error also happened on the Steam Deck (even when running via Proton). I might try merging the openfl shader PR into DDTO again and see if I have any luck this time around.
Looking into the VLCBitmap error though, I wonder if this is a hxCodec issue, as it's crashing where I put in code to clean up the VideoSprite.
For now you can enable Low End mode for Libitina (which uses FlxSprite instead of VideoSprite) or try compiling with the VideoSprite cleanup in PlayState's destroy function commented out/removed.

Low End mode works as a temporary solution on my machine as well, I'll attempt the other solution sometime tomorrow when I have the time


Use jku of token to retrieve keys

Use jku of token to retrieve token keys instead of building
url.
Keep JWTGenerator compatible by setting JKU in token in transparent way.
Assumption: Mock xsuaa url is 'http://localhost:33195'

@firnkes
Awesome, thanks!

remove xsuaa.uaadomain from Test properties
in class override

@Override
public String getUaaDomain() {
                String localhost = "localhost";
		try {
			localhost = InetAddress.getByName("localhost").getHostAddress();
		} catch (UnknownHostException e) {
			logger.log(....);
		}
}


Running pbfetch fails due to `FileNotFoundError'

The full error is:
`FileNotFoundError: [Errno 2] No such file or directory: '/usr/share/pbfetch/config/config.txt'`

This is likely due to Python not having superuser permissions when installing via pip install pbfetch. A password should be requested during installation to grant this permission, but that is not currently done.

I don't know either, but I think the key is setuptools and a setup.py file.

Thank you for your suggestion, however, setuptools and setup.py are not suitable solutions in this case. The project is already using rye and follows modern packaging standards.
Using a setup script that requires root access would pose significant security risks and is beyond the scope of what a package should handle. Managing system-wide directories, such as the share directory, should be the responsibility of the distribution's package manager.
A better approach would be to either:

Not provide a default configuration: This would force users of the pip package to manually copy or create a configuration file themselves.
Embed the default configuration within the script: This would eliminate the dependency on the share directory and simplify configuration management.

Handling the "file not found" error more gracefully by printing a warning message is probably be a good idea regardless. This way, we can inform the user that a configuration is needed, preventing the program from crashing unexpectedly.

Good idea regarding embedding the default config into the script itself. This simplifies everything for the user and myself, and I just pushed an update with this change. The /usr/share directory is no longer affected by pbfetch.


Build an a JS Test environment as an artifact in CI

There is a test application that exercises bucakroo, found via npm run dev.  This shows how buckaroo renders as a front end only component, not depending on the notebook.  Sometimes it is nice to build functionality into the frontend that the backend isn't yet generating outputs for.  Separating the two dev processes can encourage more contribution from differently skilled people.
Insert a screenshot of the JS Dev environment.


[VS Code]

Error:
[Description of error message]
Steps to Reproduce:
1.
2.

Version Data

extensionName
VSLS

extensionVersion
1.0.488

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.36.1

LiveShareLogs.zip

|platformName|Windows|
|platformVersion|10.0.18362|

Hey @fubaduba , could you provide some more info about what you filed an issue for?

Going to close for now. Please reopen if this issue persists.


ESLint support messes with "wrappingStyle: prettier"

It would seem that when formatting in VS Code with the newly added ESLint support, it messes with indentation that causes ESLint to error for indentation.

I am using VS Code to perform code actions onSave: ["source.formatDocument", "source.fixAll.tslint"]. I think your extension runs last which causes the problem.
For my use case, it would be awesome if format-imports handled grouping/sorting while tools like prettier and eslint handle formatting the imports how they see fit. Could we maybe have a config for disabling eslint support?

Could you share your ESLint config?

{
  "root": true,
  "ignorePatterns": [
    "projects/**/*"
  ],
  "overrides": [
    {
      "files": [
        "*.ts"
      ],
      "parserOptions": {
        "project": [
          "tsconfig.json"
        ],
        "createDefaultProgram": true
      },
      "extends": [
        "plugin:@angular-eslint/recommended",
        "eslint:recommended",
        "plugin:@typescript-eslint/recommended",
        "plugin:@typescript-eslint/recommended-requiring-type-checking",
        "plugin:@angular-eslint/template/process-inline-templates"
      ],
      "rules": {
        "@angular-eslint/contextual-lifecycle": "warn",
        "@angular-eslint/no-forward-ref": "off",
        "@angular-eslint/no-output-on-prefix": "warn",
        "@angular-eslint/no-output-native": "warn",
        "@angular-eslint/no-host-metadata-property": "warn",
        "@angular-eslint/no-input-rename": "warn",
        "@angular-eslint/no-conflicting-lifecycle": "warn",
        "@typescript-eslint/naming-convention": "warn",
        "@typescript-eslint/no-empty-function": "error",
        "@typescript-eslint/no-inferrable-types": "off",
        "@typescript-eslint/unbound-method": "warn",
        "@typescript-eslint/no-floating-promises": "warn",
        "@typescript-eslint/no-explicit-any": "warn",
        "@typescript-eslint/no-unsafe-member-access": "warn",
        "@typescript-eslint/no-unsafe-assignment": "warn",
        "@typescript-eslint/ban-types": "warn",
        "@typescript-eslint/no-unsafe-call": "warn",
        "@typescript-eslint/no-unsafe-return": "warn",
        "@typescript-eslint/restrict-plus-operands": "warn",
        "@typescript-eslint/prefer-regexp-exec": "warn",
        "@typescript-eslint/restrict-template-expressions": "warn",
        "@typescript-eslint/member-ordering": "warn",
        "@typescript-eslint/await-thenable": "warn",
        "@typescript-eslint/require-await": "warn",
        "@typescript-eslint/no-misused-promises": "warn",
        "@typescript-eslint/no-namespace": "warn",
        "@typescript-eslint/no-var-requires": "warn",
        "@typescript-eslint/adjacent-overload-signatures": "warn",
        "@typescript-eslint/ban-ts-comment": "warn",
        "@typescript-eslint/no-this-alias": "warn",
        "@typescript-eslint/no-unnecessary-type-assertion": "warn",
        "@typescript-eslint/indent": [
          "error",
          2
        ],
        "@typescript-eslint/no-empty-interface": "warn",
        "brace-style": [
          "error",
          "1tbs"
        ],
        "id-blacklist": "off",
        "id-match": "off",
        "max-len": [
          "error",
          {
            "ignorePattern": "^import [^,]+ from |^export | implements | ('|\")(http|https):",
            "code": 140
          }
        ],
        "indent": "off",
        "no-duplicate-imports": "error",
        "no-empty": "error",
        "no-redeclare": "warn",
        "no-underscore-dangle": "off",
        "prefer-const": "warn",
        "no-fallthrough": "warn",
        "no-var": "warn",
        "prefer-rest-params": "warn",
        "no-extra-boolean-cast": "warn",
        "no-prototype-builtins": "warn",
        "no-empty-pattern": "warn",
        "object-curly-spacing": [
          "error",
          "always"
        ],
        "no-unused-vars": [
          "error",
          {
            "argsIgnorePattern": "^_"
          }
        ],
        "getter-return": "warn",
        "no-useless-escape": "warn"
      }
    },
    {
      "files": [
        "*.html"
      ],
      "extends": [
        "plugin:@angular-eslint/template/recommended"
      ],
      "parser": "@angular-eslint/template-parser",
      "rules": {
        "@angular-eslint/template/no-negated-async": "warn"
      }
    }
  ]
}

The rules are in a monorepo where I don't have the authority to change the config, sadly. It would really be ideal if format-imports handled sorting/grouping exclusively and let other tools handle the formatting.

@Nxt3 - Thanks for the info!
Apparently the extension is missing @typescript-eslint/indent and indent from ESLint. I'll add support on them in the next release.
Regarding the suppression of formatting, my opinion is it's inevitable. Once the extension needs to modify code, it has to consider formatting, e.g. new lines, spaces, indentation. If you don't want its style, you can always add another formatting step AFTER its running. Maybe https://github.com/daidodo/format-imports-vscode/issues/44 is a good way to make that easy.

Oh sweet glad to see you're familiar with that extension; that's actually what I'm doing now and was still getting this issue.
"editor.codeActionsOnSave": [
      "source.addMissingImports", // vscode
      "source.formatDocument", // prettier
      "source.fixAll.eslint"
]

@Nxt3 - I've released v7.3.1 to support both indent and @typescript-eslint/indent from ESLint, which should fix your problem.
Please try it out. Thanks!

Looks like that fixed it. Thanks so much :)

@Nxt3 - FYI
I've released v7.3.2 with code action support:
"editor.codeActionsOnSave": ["source.organizeImports.sortImports"],
"tsImportSorter.configuration.autoFormat": "off"

Note: You should turn off autoFormat to avoid formatting twice.
Thanks!

Aw heck yeah! This is awesome. Thanks so much for implementing :D

@Nxt3
I've release v7.3.4 which fixes the "formatting twice" issue, so you can enable both autoFormat and editor.codeActionsOnSave now.


Requirement already satisfied: blinker>=1.4 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 5))
Requirement already satisfied: boltons>=16.5.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 6))
Requirement already satisfied: dnspython>=1.14.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 7))
Requirement already satisfied: geoip2>=2.4.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 8))
Requirement already satisfied: geojson>=1.3.3 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 9))
Requirement already satisfied: icalendar>=3.10 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 10))
Requirement already satisfied: Jinja2>=2.8 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 12))
Collecting markupsafe>=0.23 (from -r requirements.txt (line 13))
Using cached MarkupSafe-0.23.tar.gz
Collecting matplotlib>=1.5.3 (from -r requirements.txt (line 14))
Using cached matplotlib-1.5.3.tar.gz
Complete output from command python setup.py egg_info:
============================================================================
Edit setup.cfg to change the build options
BUILDING MATPLOTLIB
matplotlib: yes [1.5.3]
python: yes [3.3.2 (default, Jul 14 2014, 15:06:48)  [GCC
4.8.2 20140120 (Red Hat 4.8.2-16)]]
platform: yes [linux]
REQUIRED DEPENDENCIES AND EXTENSIONS
numpy: yes [not found. pip may install it below.]
dateutil: yes [using dateutil version 2.6.0]
pytz: yes [using pytz version 2016.7]
cycler: yes [using cycler version 0.10.0]
tornado: yes [tornado was not found. It is required for the
WebAgg backend. pip/easy_install may attempt to
install it after matplotlib.]
Traceback (most recent call last):
File "", line 1, in
File "/tmp/pip-build-qi56u3/matplotlib/setup.py", line 167, in
result = package.check()
File "./setupext.py", line 1255, in check
import pyparsing
File "/usr/lib/python2.7/site-packages/pyparsing.py", line 2480
except ParseException, err:
^
SyntaxError: invalid syntax

Command "python setup.py egg_info" failed with error code 1 in
/tmp/pip-build-qi56u3/matplotlib/
Failed to install python requirements with pip
[root@www king-phisher]# sudo python -m pip install -r requirements.txt
Requirement already satisfied: advancedhttpserver>=2.0.6 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 3))
Requirement already satisfied: alembic>=0.8.8 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 4))
Requirement already satisfied: blinker>=1.4 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 5))
Requirement already satisfied: boltons>=16.5.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 6))
Requirement already satisfied: dnspython>=1.14.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 7))
Requirement already satisfied: geoip2>=2.4.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 8))
Requirement already satisfied: geojson>=1.3.3 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 9))
Requirement already satisfied: icalendar>=3.10 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 10))
Requirement already satisfied: ipaddress>=1.0.17 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 11))
Requirement already satisfied: Jinja2>=2.8 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 12))
Requirement already satisfied: markupsafe>=0.23 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 13))
Requirement already satisfied: matplotlib>=1.5.3 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 14))
Requirement already satisfied: msgpack-python>=0.4.8 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 15))
Requirement already satisfied: paramiko>=2.0.2 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 16))
Requirement already satisfied: pluginbase>=0.4 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 17))
Requirement already satisfied: psycopg2>=2.6.2 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 18))
Requirement already satisfied: pyotp>=2.2.1 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 19))
Requirement already satisfied: python-dateutil>=2.5.3 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 20))
Requirement already satisfied: python-pam>=1.8.2 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 21))
Requirement already satisfied: pytz>=2016.6.1 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 22))
Requirement already satisfied: PyYAML>=3.12 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 23))
Requirement already satisfied: requests>=2.11.1 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 24))
Requirement already satisfied: six>=1.10.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 25))
Requirement already satisfied: smoke-zephyr>=1.1.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 26))
Requirement already satisfied: SQLAlchemy>=1.0.15 in
/usr/lib64/python2.7/site-packages (from -r requirements.txt (line 27))
Requirement already satisfied: termcolor>=1.1.0 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 28))
Requirement already satisfied: tzlocal>=1.2.2 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 29))
Requirement already satisfied: XlsxWriter>=0.9.3 in
/usr/lib/python2.7/site-packages (from -r requirements.txt (line 30))
Requirement already satisfied: Mako in /usr/lib/python2.7/site-packages
(from alembic>=0.8.8->-r requirements.txt (line 4))
Requirement already satisfied: python-editor>=0.3 in
/usr/lib/python2.7/site-packages (from alembic>=0.8.8->-r requirements.txt
(line 4))
Requirement already satisfied: maxminddb>=1.2.1 in
/usr/lib/python2.7/site-packages (from geoip2>=2.4.0->-r requirements.txt
(line 8))
Collecting setuptools (from geojson>=1.3.3->-r requirements.txt (line 9))
Using cached setuptools-30.1.0-py2.py3-none-any.whl
Requirement already satisfied: numpy>=1.6 in
/usr/lib64/python2.7/site-packages (from matplotlib>=1.5.3->-r
requirements.txt (line 14))
Requirement already satisfied: cycler in /usr/lib/python2.7/site-packages
(from matplotlib>=1.5.3->-r requirements.txt (line 14))
Requirement already satisfied: pyparsing!=2.0.0,!=2.0.4,!=2.1.2,>=1.5.6 in
/usr/lib/python2.7/site-packages (from matplotlib>=1.5.3->-r
requirements.txt (line 14))
Requirement already satisfied: cryptography>=1.1 in
/usr/lib64/python2.7/site-packages (from paramiko>=2.0.2->-r
requirements.txt (line 16))
Requirement already satisfied: pyasn1>=0.1.7 in
/usr/lib/python2.7/site-packages (from paramiko>=2.0.2->-r requirements.txt
(line 16))
Requirement already satisfied: idna>=2.0 in
/usr/lib/python2.7/site-packages (from
cryptography>=1.1->paramiko>=2.0.2->-r requirements.txt (line 16))
Requirement already satisfied: enum34 in /usr/lib/python2.7/site-packages
(from cryptography>=1.1->paramiko>=2.0.2->-r requirements.txt (line 16))
Requirement already satisfied: cffi>=1.4.1 in
/usr/lib64/python2.7/site-packages (from
cryptography>=1.1->paramiko>=2.0.2->-r requirements.txt (line 16))
Requirement already satisfied: pycparser in
/usr/lib/python2.7/site-packages (from
cffi>=1.4.1->cryptography>=1.1->paramiko>=2.0.2->-r requirements.txt (line
16))
Installing collected packages: setuptools
Successfully installed setuptools-30.1.0
[root@www king-phisher]#
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/securestate/king-phisher/issues/174#issuecomment-264704526,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB9oP2ba911CjM5CI4Ppx7WcSqhXCLMsks5rEsK8gaJpZM4LARXr
.

Hello steiner ,  i've pip the requirements with pip3; see below the results
[root@www king-phisher]# sudo python3 -m pip install -r requirements.txt
Downloading/unpacking advancedhttpserver>=2.0.6 (from -r requirements.txt (line 3))
Running setup.py egg_info for package advancedhttpserver
Downloading/unpacking alembic>=0.8.8 (from -r requirements.txt (line 4))
Running setup.py egg_info for package alembic
warning: no files found matching '*.jpg' under directory 'docs'
warning: no files found matching '*.sty' under directory 'docs'
warning: no files found matching '*.dat' under directory 'tests'
no previously-included directories found matching 'docs/build/output'

Downloading/unpacking blinker>=1.4 (from -r requirements.txt (line 5))
Running setup.py egg_info for package blinker
Downloading/unpacking boltons>=16.5.0 (from -r requirements.txt (line 6))
Running setup.py egg_info for package boltons
Downloading/unpacking dnspython>=1.14.0 (from -r requirements.txt (line 7))
Running setup.py egg_info for package dnspython
warning: no files found matching 'TODO'
warning: no files found matching '*.txt' under directory 'examples'
warning: no files found matching '*.txt' under directory 'tests'

Downloading/unpacking geoip2>=2.4.0 (from -r requirements.txt (line 8))
Running setup.py egg_info for package geoip2
Downloading/unpacking geojson>=1.3.3 (from -r requirements.txt (line 9))
Running setup.py egg_info for package geojson
warning: no files found matching '*.txt' under directory 'tests'

Downloading/unpacking icalendar>=3.10 (from -r requirements.txt (line 10))
Running setup.py egg_info for package icalendar
warning: no previously-included files matching '*.pyc' found under directory 'src/icalendar'
warning: no previously-included files matching '*~' found under directory 'src/icalendar'

Exception:
Traceback (most recent call last):
File "/usr/lib/python3.3/site-packages/pip/basecommand.py", line 139, in main
status = self.run(options, args)
File "/usr/lib/python3.3/site-packages/pip/commands/install.py", line 266, in run
requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
File "/usr/lib/python3.3/site-packages/pip/req.py", line 954, in prepare_files
req_to_install.check_if_exists()
File "/usr/lib/python3.3/site-packages/pip/req.py", line 686, in check_if_exists
self.satisfied_by = pkg_resources.get_distribution(self.req)
File "/usr/lib/python3.3/site-packages/pkg_resources/init.py", line 553, in get_distribution
dist = get_provider(dist)
File "/usr/lib/python3.3/site-packages/pkg_resources/init.py", line 427, in get_provider
return working_set.find(moduleOrReq) or require(str(moduleOrReq))[0]
IndexError: list index out of range
Storing complete log in /root/.pip/pip.log
[root@www king-phisher]#

can you advises me how to install king phisher with python 2.7 to run a phishing compagne ; and after i'll try to make a new installation for python 3.x

No we don't provide manual installation instructions. Once again I advise that you sit tight until the install.sh script can be updated for CentOS 7 at which point you'll probably need to use it on a fresh installation.

Ok thank you i'll wait untill you publish the new install.sh , can you just tell me when it will be published;

This has been fixed in commit 5d9bb259 which is now available. Once again, I highly recommend simply running the new install.sh script from a fresh installation of CentOS 7.
Tested with CentOS 7 x64 from the CentOS-7-x86_64-Everything-1511.iso media.


Added member_users and member_groups to resource_user_group

This PR aims to resolve issue #13 , which I created to discuss the possibility of adding a connection to a pre-existing user. The way that this resolves this PR is through adding properties for member_users and member_groups to resources/user_group.go. Within the official Apache Guacamole web client, an administrator may create a User Group, and may assign multiple users and connections to the connection group. Upon creation, the users of the group may see the connections within the group. The changes made within this PR call the same API endpoints which are called by the official Guacamole web client.

I also updated the GitHub Actions to HashiCorp's currently recommended release action here and deleted the vendor folder. I'm brand new to go and creating terraform providers, but things did not seem to be working as expected with that vendor folder being there, then they started working when I deleted the vendor folder. From a little research, it seems as though that directory is similar to node's node_modules directory, and typically wouldn't be pushed to a repo.
I also went about figuring out how to release a provider on the terraform registry and can confirm that the changes work as expected. My only issue is that guacamole-api-client could benefit from a validateUsers function similar to the validateGroups function that exists, so that we could prevent trying to add a user if the user doesn't exist. If you do that currently, nothing fails. The group is created, it just doesn't have the (non-existant) user added to it, which can be a little confusing until you realize what's going on.


use DDLINK/DDSUBLINK instead of LINK2 and raw HTML

DDLINK/DDSUBLINK use ROOT_DIR which is more robust than a relative path.
Together with https://github.com/D-Programming-Language/dlang.org/pull/1159 this fixes the links in the online DDox docs.

Auto-merge toggled on


Not able to handle events like button click

i am not able achieve event handling in fragments

Can you please tell me how to achieve event handling

Try to handle onOptionsItemSelected(MenuItem item) in your Activity

Handle with ScreenShotable onSwitch(Resourceble slideMenuItem, ... method.You can use slideMenuItem name for the which button click.


De-couple from react-bootstrap

react-rich-table should be usable without react-bootstrap. Instead, react-bootstrap should only be a possible theme.
The consumer should be able to decide if react-boostrap or another theme should be used.
The consumer is free to use a completely custom theme (this is already possible technically, but we still have react-bootstrap as required dependency).

With the recent changes the higher-order components are completely de-coupled from react-bootstrap. The only thing left to do is more or less to create a separate package for the components depending on react-bootstrap.

Instead of creating a new theme package we will just use react-bootstrap in the example. Every user will create his own theme anyway.


Hooks didn't applied to affected rows in batch operation

Your Question

Hook AfterDelete doesn't applied to every deleted rows when delete like this db.Where(...).Delete(&Obj{})
I did some digging, looks like hooks only applied to Delete's argument value, which in this case an empty object.
Is this expected? or a missing feature?
The following code should reproduce this issue:
Code

package main

import (
	"time"

	"gorm.io/driver/mysql"
	"gorm.io/gorm"
	"gorm.io/gorm/logger"
)

type Org struct {
	gorm.Model
	Name string `gorm:"VARCHAR(64)"`
}

func (a *Org) AfterDelete(tx *gorm.DB) error {
	if err := tx.Model(&Repo{}).Where("org_name = ?", a.Name).Delete(&Repo{}).Error; err != nil {
		return err
	}
	if err := tx.Model(&Env{}).Where("org_name = ? and repo_name = ?", a.Name, "*").Delete(&Env{}).Error; err != nil {
		return err
	}
	return nil
}

type Repo struct {
	gorm.Model
	Name    string `gorm:"VARCHAR(64)"`
	OrgName string `gorm:"VARCHAR(64)"`
}

func (u *Repo) AfterDelete(tx *gorm.DB) error {
	return tx.Model(&Env{}).Where("repo_name = ?", u.Name).Delete(&Env{}).Error
}

type Env struct {
	gorm.Model
	OrgName  string `gorm:"VARCHAR(64)"`
	RepoName string `gorm:"VARCHAR(64)"`
	Name     string `gorm:"VARCHAR(64)"`
	Value    string `gorm:"VARCHAR(64)"`
}

func main() {
	dsn := "root:root@(mysql:3306)/testing?charset=utf8mb4&parseTime=True&loc=Local"
	db, err := gorm.Open(mysql.New(mysql.Config{DSN: dsn}), &gorm.Config{
		Logger:      logger.Default.LogMode(logger.Info),
		PrepareStmt: true,
	})
	if err != nil {
		panic(err)
	}
	sqlDB, _ := db.DB()
	sqlDB.SetConnMaxLifetime(time.Minute * 3)
	sqlDB.SetMaxIdleConns(3)

	db.AutoMigrate(&Org{}, &Repo{}, &Env{})
	db.Model(&Org{}).Create(&Org{Name: "org-1"})
	db.Model(&Repo{}).Create(&Repo{Name: "repo-1", OrgName: "org-1"})
	db.Model(&Repo{}).Create(&Repo{Name: "repo-2", OrgName: "org-1"})
	db.Model(&Env{}).Create(&Env{OrgName: "org-1", RepoName: "*", Name: "e-1", Value: "a"})
	db.Model(&Env{}).Create(&Env{OrgName: "org-1", RepoName: "repo-1", Name: "e-2", Value: "b"})
	db.Model(&Env{}).Create(&Env{OrgName: "org-1", RepoName: "repo-2", Name: "e-3", Value: "c"})

	var org Org
	db.Where(&Org{Name: "org-1"}).First(&org)
	db.Delete(&org)
}

what i got after running the code above:
Output

2020/11/25 07:37:56 /data/testing/main.go:33
[1.046ms] [rows:0] UPDATE `envs` SET `deleted_at`='2020-11-25 07:37:56.546' WHERE repo_name = '' AND `envs`.`deleted_at` IS NULL

2020/11/25 07:37:56 /data/testing/main.go:17
[2.900ms] [rows:2] UPDATE `repos` SET `deleted_at`='2020-11-25 07:37:56.545' WHERE (org_name = 'org-1') AND `repos`.`deleted_at` IS NULL

2020/11/25 07:37:56 /data/testing/main.go:20
[1.275ms] [rows:1] UPDATE `envs` SET `deleted_at`='2020-11-25 07:37:56.548' WHERE (org_name = 'org-1' and repo_name = '*') AND `envs`.`deleted_at` IS NULL

2020/11/25 07:37:56 /data/testing/main.go:67
[8.560ms] [rows:1] UPDATE `orgs` SET `deleted_at`='2020-11-25 07:37:56.543' WHERE `orgs`.`id` = 1 AND `orgs`.`deleted_at` IS NULL

DB

mysql> select * from orgs;
+----+-------------------------+-------------------------+-------------------------+-------+
| id | created_at              | updated_at              | deleted_at              | name  |
+----+-------------------------+-------------------------+-------------------------+-------+
|  1 | 2020-11-25 07:37:56.508 | 2020-11-25 07:37:56.508 | 2020-11-25 07:37:56.544 | org-1 |
+----+-------------------------+-------------------------+-------------------------+-------+
1 row in set (0.00 sec)

mysql> select * from repos;
+----+-------------------------+-------------------------+-------------------------+--------+----------+
| id | created_at              | updated_at              | deleted_at              | name   | org_name |
+----+-------------------------+-------------------------+-------------------------+--------+----------+
|  1 | 2020-11-25 07:37:56.518 | 2020-11-25 07:37:56.518 | 2020-11-25 07:37:56.545 | repo-1 | org-1    |
|  2 | 2020-11-25 07:37:56.523 | 2020-11-25 07:37:56.523 | 2020-11-25 07:37:56.545 | repo-2 | org-1    |
+----+-------------------------+-------------------------+-------------------------+--------+----------+
2 rows in set (0.00 sec)

mysql> select * from envs;
+----+-------------------------+-------------------------+-------------------------+----------+-----------+------+-------+
| id | created_at              | updated_at              | deleted_at              | org_name | repo_name | name | value |
+----+-------------------------+-------------------------+-------------------------+----------+-----------+------+-------+
|  1 | 2020-11-25 07:37:56.528 | 2020-11-25 07:37:56.528 | 2020-11-25 07:37:56.548 | org-1    | *         | e-1  | a     |
|  2 | 2020-11-25 07:37:56.533 | 2020-11-25 07:37:56.533 | NULL                    | org-1    | repo-1    | e-2  | b     |
|  3 | 2020-11-25 07:37:56.537 | 2020-11-25 07:37:56.537 | NULL                    | org-1    | repo-2    | e-3  | c     |
+----+-------------------------+-------------------------+-------------------------+----------+-----------+------+-------+
3 rows in set (0.00 sec)

The document you expected this should be explained

https://gorm.io/docs/hooks.html
Expected answer

I expect hook applied to every deleted rows because hook named AfterDelete.
If this behavior is expected, it would be nice mention in doc.

Hook AfterDelete doesn't applied to every deleted rows when delete like this db.Where(...).Delete(&Obj{})
I did some digging, looks like hooks only applied to Delete's argument value, which in this case an empty object.
Is this expected?

This is expected, Hooks can only apply to model values.


InlinePanel Orderable is not working after first click on element to move

Issue Summary
I'm using Orderable model as an InlinePanel in another model in my app. I have an issue with changing the order of items on the wagtail admin panel - it doesn't work when I click the first time on the element. After the first click, it starts working.
Is related to jquery script: /wagtailadmin/js/page-editor.js?v=8b6dafbe (starts from line 39)
class PhoneNumber(Orderable):
    general_setting = ParentalKey(
        "wagtail_settings.GeneralSetting",
        on_delete=models.CASCADE,
        related_name="phone_numbers",
    )
    phone = models.CharField(_("phone"), max_length=30)
    panels = [FieldPanel("phone")]

I have this problem with all InlinePanels using Orderable feature.
Steps to Reproduce

Create a new models.Model with @register_setting decorator
Use PhoneNumber model mentioned above and add proper ParentalKey to your model from step 1
Add InlinePanel MultiFieldPanel([InlinePanel("phone_numbers")], heading=_("phone numbers")) to your model from step 1
Visit Wagtail dashboard and open your registered model - add few records
Save and refresh the page, click on the last item to move it up
You will see animation because it's not related to move functionality but the element will stay in the same place
Click again and element will move up correctly

Effect before and after first click:

Excepted behavior
It should be possible to change item order on the first click
Technical details

Python version: 3.8.2.
Django version: 3.0.7.
Wagtail version: 2.9.
Browser version: Chrome 83.

Hi @bplociennik - thanks for the report. Just to confirm, have you been able to replicate this on a clean newly-created Wagtail project?

hey @gasman I didn't have time to check it on fresh version. I will try check it in the next week.

Thanks! If you do manage to replicate it on a new project, it'd be really helpful if you could share the complete models.py - that makes it much easier for us to verify the issue independently.

Closing as unconfirmed.


feat: Add KServe Integration notebook

Introduce notebook to test KServe Integration:

create an Inference Service
perform inference

This only works with Serverless deployment mode at the moment, see the following issue for more:

https://github.com/canonical/kserve-operators/issues/148

Running with Serverless
In order to switch to Serverless (see here for more):

update the kserve-controller config:
juju config kserve-controller deployment-mode="serverless"

relate kserve-controller to knative-serving:
juju relate kserve-controller:local-gateway knative-serving:local-gateway

Patch ConfigMap
Even with KServe deployed in Serverless mode, we still have the bug described in the following issue:

https://github.com/canonical/kserve-operators/issues/150

Until the above is resolved, we have to manually patch the inferenceservice-config ConfigMap in order to be able to hit any created InferenceServices:
kubectl -n kubeflow edit cm inferenceservice-config

Apply the following change:
-      "localGateway" : "kubeflow/knative-local-gateway",
+      "localGateway" : "knative-serving/knative-local-gateway",

Force-pushed to squash fixup and add signature.


Locale persistence + minor bugfixes

Closes #313

On web it works great. Url to nl is preserved. Some mixing text but that's prob cuz there aren't translations for it yet.
Mobile designs seemed to have been affected though. Sizing isn't correct.

On web it works great. Url to nl is preserved. Some mixing text but that's prob cuz there aren't translations for it yet.
Mobile designs seemed to have been affected though. Sizing isn't correct.

I think this is unrelated to this PR as the same behavior is visible on the live website - https://savingsatoshi.com/nl/chapters/chapter-1/intro-2 (might be due to the missing translation)

Yes there are some missing translations for nl currently.


feat: Add chart and dashboard labels

This PR introduces a label property to the Meta object both in chart configs and layout definitions. For now, it's only used in chart config to label chart tabs, but in the future could be extended to e.g.:

label dashboard charts in the left panel in layouts different than tab layout,
label dashboards to keep a "working name", instead of showing the full title in user profile.

How to test

Go to this link.
✅ See that there's an Add label button in chart tab.
✅ Click on it and see that you can edit the label.
Add a new chart.
Proceed to layout options.
✅ See that it's no longer possible to edit the labels.
Publish the chart.
✅ See that [ Add label ] is not displayed and that tabs can't be edited.

This PR introduces a label property to the Meta object both in chart configs and layout definitions. For now, it's only used in chart config to label chart tabs, but in the future could be extended to e.g.:

label dashboard charts in the left panel in layouts different than tab layout,
label dashboards to keep a "working name", instead of showing the full title in user profile.

How to test

Go to this link.
✅ See that there's an Add label button in chart tab.
✅ Click on it and see that you can edit the label.
Add a new chart.
Proceed to layout options.
✅ See that it's no longer possible to edit the labels.
Publish the chart.
✅ See that [ Add label ] is not displayed and that tabs can't be edited.

Hi you forgot to add the link

Oops, forgot to add it 😅 Should be fine now, thanks @noahonyejese 🙇♂

Oops, forgot to add it 😅 Should be fine now, thanks @noahonyejese 🙇♂

worked all fine visually, wd.

Thanks a lot. Looks great! My only question is if we should restrict the amount of characters that can be entered in the label? Maybe to 50 characters?

@KerstinFaye I thought about this during the implementation, but as we have a live visual feedback, when adding them and their goal is not to replace the title, but rather they would be short "naturally" (at least I hope 😅) I thought maybe we don't need to have this limit.
Maybe we can limit the number of characters displayed in the tab label, so we would cut the text as soon as it's over 50 characters, but not limit the entry?

@bprusinowski Thank you for this suggestion. You're right that users should be reasonable with label names. This aligns with their original request for separate titles for label tabs, given the long chart titles. I appreciate your suggestion to truncate the text after 50 characters—it's a good solution if it's not too much effort.


Asn1 dsl

Coverage decreased (-6.76%) to 83.28% when pulling c812a62ba7afad69f116ea6146be658c883f7efe on asn1_dsl into 92a0847ef44d4ef813457f06255fc6bec4024c40 on master.

Coverage decreased (-6.76%) to 83.28% when pulling c812a62ba7afad69f116ea6146be658c883f7efe on asn1_dsl into 92a0847ef44d4ef813457f06255fc6bec4024c40 on master.


Table Grouping (Basic)

Bug type
Component
Component name
Table
What happened?
according to the docs, the table grouping feature should behave as follows,

however when I run the code in my project or even in your very own playground this is what I get,

the code base is the same yet the outcome is completely different.
Expected behavior
I'm expecting it to behave the same way it is listed in the docs.

Reproduction link
https://try.mudblazor.com/snippet/rVdLj-JGEL7zJ6bVygGk2OwhyQEMIsM8s8tkFEa5RNGosWugM2231W4PsIj_nuq2sdswZlbZcADb9dVXT1cXz88zxhNfsa9SXUzyjCdLMt9mGmL_AbR_p3Xq_5bJpFPKZnl0KQzYv96wOBWQ-VdMM38mIxBZZ8KTfyDUxOhNBYdEk1V12ekEmd4KGHcIfvw4jzzNFgK8EITwwjzTMvaWSuYp2VmI-bzIRHtr4MuVHpCfP30aWsm-8yGH9yKlBuVQpSyKMAZvITWiDFu6GVZSDRvtMcGXyYAoY-5gKeiXXncCjP7JWCN38g3UiGqVAyWXCthrKnmiR7S-9ucxJXfW8RFFx9MNJTd8A9EdsKhSrswTco9Jz0Z0ci0gxmxlruxPrnSOzn0FBDy_VXcu5tZEfbk1AJuAK3jhCddcJieowoW5CWtEFyx8NQpJ5IVSSDV4Y6rreSa1KROgNXgOZKlg2zshvLG5ngqWYQTxwvvJRVxBkqGd58j8uoJZLjSfg8COQS8REdcPaNElwVQKa2Fc6U34C-m60F4l2jnkqIvhkKwIcs0jvRqQX0zJKenXbPsj3qPUYZunLIlM0f8nMxbuSL7nPug30xMUhZ3iO4Md5OiYxl2NH1TQL66OJXNs-zbZA4uhTfYoM5umNvlMCqZIjG3RQAT9dxwNnNZ8ApwtTMMxITm0WPt7TwkmKMOi4VtHx5PuD3QXGjMb7VsDJpz9gFQPP8N2T3tH7rW6Evwh1y3ORcRMwi9sAWJEHxTaPph4yOMFlLmPzmmZMjh6dVXOapmIXGtVvc5qHUrnaDareapdDoyTSUld3rrkDrN9SBu8Qf8klYEzS76jA8jHp0KzSZ6kZoLMkHdAKpftOPbnedztQo-MxgSKKN5tlWOfC4xxoTw25muuwxWZLDjO0OkKwleIqpFIpmbujqj98R8Vj5na0rGdm5ap0B6fY3JHZwufM26_ldU5aVpI65PpWzmPJ-x9glnQzPZiixEH8l-t1HO8zUiNaNjoYEdEQHbE1jtV_A0rTBYSp3BRPjIiL0xkMHwH4FQFYebEfw9VZ7niMqBOA2q76bYZVVDuCmNyHC8SJbDuFmdWfV5VExDl1N7QHyuhk-WDH7Wwzs6prAhQKpRU74olLzaoYTOQ--skj0HZRFf-H5aewm_yhWe6EnZ7FQO-ZaGGiJj1S3EsC8u2SYi5yV7J78m9id3mMfrVPD8J37HC1oy7y6l_C_pGydgsu1Y5aPgw7tI1LFjK-ykoLiMe2gFDe4ctcX9RQv0wu2js0dhm5YaToZEE1cNhp4F4MvPGGPbnKDUB2CJg1AmWCsdUCB8t3kWMab4QPCShGZGHYI9SUEIyrYwDt8W2TZaghyQzX_tjJK6y5HAwnEeWnLa9PqQszsQjXAX8y2TjUUlMtt4awi7NYiYE7f3dYtWcleetRjI33VucT2eR97b0vOzLUCssXZunpUrVkKU7T3JuL7ptW6MCnauE4HZiPN8Tj-zsZkLrfyT7Q2v9Cw
Reproduction steps

click on the Reproduction link
compare with what's listed in the docs under Grouping (Basic).

Relevant log output
No response
Version (bug)
6.1.2
Version (working)
No response
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Enlarge your browser's view.

First please click on the save Icon before sharing a Try so the URL is shortened, thanks. ;)
To your problem, this is expected behaviour and supposed to be better for mobile use.
You may disable this by changing the 'Breakpoint' property of the table to 'None' or any other size you want.


Add convenience skeleton mapping function for JABS export

While we currently support JABS format export, JABS itself expects a specific skeleton format:
https://github.com/talmolab/sleap-io/blob/b15e36ed2a658f72c3f36b337655e7a0fc465403/sleap_io/io/jabs.py#L25-L38
This is supposed to be a 12-node skeleton with specific ordering. Here's the equivalent definition on the JABS side:
class PoseEstimation(ABC):
    """
    abstract base class for PoseEstimation objects. Used as the base class for
    PoseEstimationV2 and PoseEstimationV3
    """
    class KeypointIndex(enum.IntEnum):
        """ enum defining the 12 keypoint indexes """
        NOSE = 0
        LEFT_EAR = 1
        RIGHT_EAR = 2
        BASE_NECK = 3
        LEFT_FRONT_PAW = 4
        RIGHT_FRONT_PAW = 5
        CENTER_SPINE = 6
        LEFT_REAR_PAW = 7
        RIGHT_REAR_PAW = 8
        BASE_TAIL = 9
        MID_TAIL = 10
        TIP_TAIL = 11

(source)
We should make it easier to establish a manual correspondence between custom skeletons and the expected JABS format. This is important since feature computation downstream expects the keypoint ordering above.

@SkepticRaven


Search always returns 404 (Not Found), none of the pages work (Chromium)

Starting the server with npm run dev and opening localhost:3000 in a Chromium browser (QtWebEngine 5.15.9, based on Chromium 87.0.4280.144), search always returns 404, and the "Rejected" popup appears.
I also get the error that the manifest is not found, which might relate to #50:
GET http://localhost:3000/site.webmanifest 404 (Not Found)

hmm just tried the following, and everything's working fine:
git clone git@github.com:intavia/web
cd web
npm i
npm run dev

I'm getting the same error message as Max in Edge and Firefox on the dev branch when searching.

please try running npx msw init public

for context: https://github.com/mswjs/msw/discussions/1015#discussioncomment-1747585
service worker script is now gitignored but should be autogenerated on package install.
going to add npm script for manual install

hmm just tried the following, and everything's working fine:
git clone git@github.com:intavia/web
cd web
npm i
npm run dev

Interesting. This now works for me, flawlessly both in Firefox (#50) and in Chromium.

service worker script is now gitignored but should be autogenerated on package install.

Yesterday, I did a
git pull
npm i
npm run dev

which should have run the prepare script as well. But it seems a fresh repository does something different.

Apparently the service worker script is only auto-generated at package install. That's why it didn't re-generate it when you ran npm i in the existing repo.

just added script to manually regenerate service worker with npm run generate:msw

That's why it didn't re-generate it when you ran npm i in the existing repo.

Maybe the generate:msw and the generate:favicons scripts (I get favicon 404s on the old install as well) should be run as part of the prepare script then. That runs on every npm install.
Otherwise, if it now works for @samuelbeck on Edge and Firefox as well, I guess this and #50 can be resolved.

the generate:favicons script currently runs in prebuild because i didn't think we'd need to have favicons in dev, but i'm fine with changing

I agree that the favicons are not essential. But there should be an easier way to get a cloned repository to a "fresh" state that deleting and re-cloning it. I guess just running npm run generate would do that now, but what if in the future there are other stateful changes introduced into the build? Already now, there are a lot of npm scripts, and especially for me as an infrequent contributor, it is really hard to determine what they do and when they are called. Maybe the scripts could be briefly documented, for example in the README.


Doesn't work on macOS with vim

This is a great idea, thanks.
But it doesn't work on my macOS with Homebrew-installed vim:

oh right now it only looks for vim, I will also have it check vi.

I changed my EDITOR to "vim" but still have the same problem.

I just pushed a fix about 10 seconds ago ("version": "1.1.10")  Let me know if that fixes it!

One thing I'm unsure of: it logs a few messages after opening the editor, which appears in vim on top of vim's text.
Is it better if I silence the logs for vim/vi?

The new version works. Thanks.
Yeah, I noticed that too. I think it's better to open the editor after you've fetched the repo.
Another issue that I'm having is that I only see the README file and no other files.

It would be a lot slower to wait for the entire repository to download, but I see what you're saying – it only shows the README file because it hasn't finished downloading the rest of the repository by the time vim opens.
I guess what I could do instead is make it open the readme on vim by default, so that if you want to go to a different file, when you go up, its probably loaded most of the files by then.
If it delayed opening until the repository is completely finished, it wouldn't be fast enough I think

I'm personally fine with waiting for the whole repo to download before opening the editor.

I tried it with "version": "1.1.15" and the issue still persists when $EDITOR is vim. For now i'm just using git peek -e code

@rushi can you screenshot/paste terminal output? I'm not sure how to reproduce.
One issue I'm seeing on macOS is that $EDITOR isn't propagating to the node process. It sounds like something related to System Integrity Protection, which is unfortunate.

@rushi can you screenshot/paste terminal output? I'm not sure how to reproduce.

This is how it looks like when EDITOR=vim for git peek https://github.com/Jarred-Sumner/git-peek
Technically the editor does open up, but the output is still on the CLI so it interferes with vim, and the vim UI doesn't refresh when new directories and such are fetched

One issue I'm seeing on macOS is that $EDITOR isn't propagating to the node process. It sounds like something related to System Integrity Protection, which is unfortunate.

/tmp
○ export EDITOR=code

/tmp
○ node test.js
process.env.EDITOR is code

/tmp
○ export EDITOR=vim

/tmp
○ node test.js
process.env.EDITOR is vim

/tmp
○ node -v
v12.19.1

I justed on my environment and it's definitely being picked up by node v12

@rushi  @soheilpro I added a --wait flag which waits for the entire repository to finish downloading & extracting before opening the editor. This flag is always enabled if the chosen editor is vi/vim, but otherwise defaults to disabled. So it should just work the way you expect now.

I justed on my environment and it's definitely being picked up by node v12
Hmm, I think you're right. I read this somewhere when investigating an $EDITOR issue earlier. In any case, I manually tested it in vim after setting $EDITOR to vim and it seems to work right


Update volume for the docker-compose file

This PR

Updates the volume for the docker-compose file (such that we do not need to restart the server every time for the changes)
Update the version for docker compose

Thanks!


Support for multiple Sampling rates for Neuralynx Recordings

Currently the NeuralynxRawIO.py doesn't support sessions where there are continuously sampled files with multiple sampling rates. Is there a plan to add this feature in the next release?

@PeterNSteinmetz @JuliaSprenger : any knwoledge on how neuralynx handle multi sample rate ?
In my mind the rate was global and not per channel.

Of course! And thank you along with all the other creators of this wonderful tool!
I also emailed the files to you - let me know if you need something else!

This feature could be implemented at the Neo IO core level, which is likely the most appropriate. It could also be implemented by SpikeInterface making different calls onto the RawIO classes. Really a question for @samuelgarcia or @JuliaSprenger.
I have worked mostly on the NeuralynxRawIO and getting it to work with the different variety of Neuralynx files. If you need something in the shorter term, suggest writing code to manipulate the current versions of the NeuralynxRawIO classes.

For immediate use, I would suggest modifying your code to make calls directly using the RawIO.NeuralynxRawIO class. You can construct as many of these objects as you need to accommodate the differing sampling rate groups of files.

Another use case for multiple sampling rates is that there are experiments (including ours) which use real time spectral analysis and sometimes that software that does this doesn't play well with the wideband sampling that is needed for Automatic spike sorting. In this case, we physically copy the required signals on the acquisition box to an unused input channel on the box and this copied signal can then be sampled at a rate which is compatible with the online spectral analysis tool.
Now this software can stream this differently sampled signal from Neuralynx's acquisition software and do its work. It is worthwhile for us to save this signal too for post hoc analysis.


[BUG] Error setting network-mode on non linux platforms

Recently a story_microshift was added to the e2e scenarios by #3779 , the background steps include setting the network-mode to user.
On non-linux platforms that config option does not exist as network-mode user is the only mode available; this results on an error executing the story on non-linux platforms:
<testsuites name="crc" tests="1" skipped="0" failures="1" errors="0" time="0.6364514">
  <testsuite name="Microshift test stories" tests="1" skipped="0" failures="1" errors="0" time="0.6312547">
    <testcase name="Start and expose a basic HTTP service and check after restart" status="failed" time="0.6312547">
      <failure message="Step setting config property "network-mode" to value "user" succeeds: command 'crc config set network-mode user', expected to succeed, exited with exit code: 1
Command stdout:
Command stderr: level=debug msg="CRC version: 2.25.0+97705a\n"
level=debug msg="OpenShift version: 4.13.6\n"
level=debug msg="Podman version: 4.4.4\n"
level=debug msg="Running 'crc config set'"
Configuration property 'network-mode' does not exist
"></failure>
...

@adrianriobo I tested the scenario in mac and it does have network-mode setting. Is it only for windows?
✗ uname -a
Darwin prkumar-mac 22.6.0 Darwin Kernel Version 22.6.0: Wed Jul  5 22:22:05 PDT 2023; root:xnu-8796.141.3~6/RELEASE_ARM64_T6000 arm64

$ PULL_SECRET_FILE=/Users/prkumar/pull-secret BUNDLE_LOCATION=/Users/prkumar/.crc/cache/crc_microshift_vfkit_4.13.6_arm64.crcbundle make e2e-story-microshift
go install -tags "containers_image_openpgp"  -ldflags="-X github.com/crc-org/crc/pkg/crc/version.crcVersion=2.25.0 -X github.com/crc-org/crc/pkg/crc/version.ocpVersion=4.13.6 -X github.com/crc-org/crc/pkg/crc/version.okdVersion=4.13.0-0.okd-2023-06-04-080300 -X github.com/crc-org/crc/pkg/crc/version.podmanVersion=4.4.4 -X github.com/crc-org/crc/pkg/crc/version.microshiftVersion=4.13.6 -X github.com/crc-org/crc/pkg/crc/version.commitSha=97705a "  ./cmd/crc
Log successfully started, logging into: /Users/prkumar/work/github/crc/test/e2e/out/test-results/e2e_2023-8-22_12-24-07.log
Running e2e test in: /Users/prkumar/work/github/crc/test/e2e/out/test-run
Working directory set to: /Users/prkumar/work/github/crc/test/e2e/out/test-run
Feature: Microshift test stories
Copying /Users/prkumar/work/github/crc/test/testdata/hello.yaml to /Users/prkumar/work/github/crc/test/e2e/out/test-run
Copying /Users/prkumar/work/github/crc/test/testdata/httpd-example-index.html to /Users/prkumar/work/github/crc/test/e2e/out/test-run
Copying /Users/prkumar/work/github/crc/test/testdata/httpd-example.yaml to /Users/prkumar/work/github/crc/test/e2e/out/test-run
Copying /Users/prkumar/work/github/crc/test/testdata/pipeline-sub.yaml to /Users/prkumar/work/github/crc/test/e2e/out/test-run
Copying /Users/prkumar/work/github/crc/test/testdata/tekton-task.yaml to /Users/prkumar/work/github/crc/test/e2e/out/test-run

  Background:
    Given setting config property "preset" to value "microshift" succeeds                                                                                # testsuite.go:863 -> github.com/crc-org/crc/test/e2e/testsuite.SetConfigPropertyToValueSucceedsOrFails
    And setting config property "network-mode" to value "user" succeeds                                                                                # testsuite.go:863 -> github.com/crc-org/crc/test/e2e/testsuite.SetConfigPropertyToValueSucceedsOrFails
    And executing single crc setup command succeeds                                                                                # testsuite.go:885 -> github.com/crc-org/crc/test/e2e/testsuite.ExecuteSingleCommandWithExpectedExitStatus
    And starting CRC with default bundle succeeds                                                                                # testsuite.go:751 -> github.com/crc-org/crc/test/e2e/testsuite.StartC

@adrianriobo I tested the scenario in mac and it does have network-mode setting. Is it only for windows?

network-mode is disabled in releases, but is available in builds from git.
https://github.com/crc-org/crc/blob/97705a008eda801ba291fe3b9982af464d4759e0/pkg/crc/config/settings.go#L106-L111

I build released alike bits for testing

network-mode is disabled in releases, but is available in builds from git.

That's explain, will work on that.

fixed by #3798


Append README.md to include Arch Linux documentation

A quick writeup that also documents setting up the project on Arch Linux.

Pull Request Test Coverage Report for Build 496972095

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

Totals

Change from base Build 494299676:
0.0%

Covered Lines:
24712

Relevant Lines:
39496

💛  - Coveralls

Pull Request Test Coverage Report for Build 496972095

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

Totals

Change from base Build 494299676:
0.0%

Covered Lines:
24712

Relevant Lines:
39496

💛  - Coveralls


如何处理插件、文件同步的问题

我遇到在一个设备上删掉一个文件，在另一个设备没有同步的问题，并且拉去远程数据库，依然无法重建
如何让第三方的插件也能够同步

I fully relied on DeepL, and I am sorry if I have misunderstood something.

Does it happens always? Could you please check the deleted file has been correctly detected on the device? (It can be seen in the log).
We can use Sync hidden files(comprehensive) or Customization sync(Selective)!

2. comprehensive

OMG,   we  can enable only one of  this two option?

We can use Sync hidden files(comprehensive) or Customization sync(Selective)!
OMG, we can enable only one of this two option?

Thanks for your great plugin! I also have the problem for sync other plugins and configure from the win11 to my phone. And I try to use customization sync to select the plugin I need to use in my phone, but I don't know how to use this function. And I can't find the help document about customization sync. It is really important for me to sync plugin and configure. Can you tell me how to custom sync clearly?


Add mapping to run entire buffer

A very small update I added one line to ipy.vim to create a mapping to run the entire buffer. I am a bit of a newbie to vim scripting so I'm unsure if I should have <c-u> at the front of the mapping and also whether join() is a good function to call on a list of all the lines in the buffer.

Merged, thanks. <c-u> shouldn't be needed for a n mapping, though we could now change all mappings to <cmd> mappngs.


Exit early instead of t.Skip

Otherwise test success is not reported

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
5bc11b2c7b7fe9c46bc2024ae4f26fa1df52916e
link
/test integ-distroless-k8s-tests_istio

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-distroless-k8s-tests_istio
5bc11b2c7b7fe9c46bc2024ae4f26fa1df52916e
link
/test integ-distroless-k8s-tests_istio

integ-security-k8s-tests_istio
5bc11b2c7b7fe9c46bc2024ae4f26fa1df52916e
link
/test integ-security-k8s-tests_istio

integ-pilot-multicluster-tests_istio
5bc11b2c7b7fe9c46bc2024ae4f26fa1df52916e
link
/test integ-pilot-multicluster-tests_istio

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

/retest

@howardjohn: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
5bc11b2c7b7fe9c46bc2024ae4f26fa1df52916e
link
/test integ-pilot-multicluster-tests_istio

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

/retest


Task Statuses "Review" and "Visited" are not differentiated in numbered list of tasks

This issue may be fixed after completing #29, but wanted to make sure this doesn't get lost.
Would help the user have a visual queue after pressing the Restart button on the initial task page.

From #46

After clicking the restart link that appears on the bottom right, or after starting a QS then clicking back, the restart button appears to the right of the continue button, but still in the bottom left. To me it should be removed when the continue button is there, but possibly I am missing its use case.

For clarity this is the restart link which will appear on every screen except the initial one:

Closing because this issue is addressed in #56


Continue Your OData Exploration with EPM

Tutorial URL: https://developers.sap.com/tutorials/odata-02-exploration-epm.html
Please specify the step you are referring to
Issue:
I have changed my password as suggested in the tutorial.
While accessing the service https://sapes5.sapdevcenter.com/sap/opu/odata/sap/EPM_REF_APPS_SHOP_SRV
Credentials are required Username/Password.
I am using my Credentials but unable to login.
Solution Steps:
1 ->  I logged off
2 -> I went to https://sapes5.sapdevcenter.com
3 -> I log In again.
4 -> Then again accessing the service and it worked

I have seen this issue before, there seems definitely something amiss with the authentication configuration in ES5. I have also seen folks experience this in the Developer Garage at TechEd. @jonathanbaker7 can you please take a look? Thanks.

This will be investigated post SAP TechEd BLR 2019

@qmacro Any update?


update(kubernetes): Deployment files

Updating deployment files (automatically generated by helm template). Made using the update-falco-k8s-manifests ProwJob. Do not edit this PR.\n\n/kind update\n\n/area manifests

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: poiana
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

LGTM label has been added.  Git tree hash: 42ca4c987066b00a44d688d6ecec42d03672dede

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

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

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


Fix otel config

Signed-off-by: Bryan Aguilar<EMAIL_ADDRESS>Description
The OTEL example contained an incorrect configuration. I have updated the configuration to use the proper syntax that can be found here.
Issues Resolved
n/a
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.
[ ] New functionality has javadoc added
[x ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1790 (1ac9e8f) into main (5cae696) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main    #1790      +/-   ##
============================================
- Coverage     94.01%   93.97%   -0.05%
  Complexity     1454     1454
============================================
  Files           188      188
  Lines          4263     4263
  Branches        344      344
============================================
- Hits           4008     4006       -2
- Misses          174      176       +2
  Partials         81       81

Impacted Files
Coverage Δ

...rwarder/discovery/AwsCloudMapPeerListProvider.java
92.59% <0.00%> (-2.47%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Support for binary sensor history graph

Checklist

[x] I am running the latest version of the card
[ ] I checked if there isn't another feature request opened with the same request
[ ] I checked that the feature is not already available in the dev branch

Hi RomRider,
please add support for binary sensors like history graph. The original HA history-graph does not offer the possibility to set the color, invert the values and hide status values.
Thank you!

It is already possible, please read the documentation, especially this section

Hi RomRider,
thank you for your answer. Can you please send me an example code of the following diagram?

(Geschlossen = fallse, Offen = true)
Thank you!

Horizontal bar charts is not possible with this card, at least not for now. I might consider it for the future.

Thanx. That's a shame to hear. Exactly this bar chart was missing in the mini-graph-card and I was hoping to solve the problem with your custom card.

I would love to see this too, like this demo. It seems like support for rangeBar would solve both this issue and #137. I know there is some binary sensor support using transforms but I'm interested in this kind of visualisation.

Just a comment to keep this open. 😀

My attempt at keeping this request alive. If there are any workaround using card-mod (for example), I'll take that as a solution too, but I haven't find any way of achieving this.

RomRider, any chance this may still happen, or is it too different to all the other charting options?

@RomRider is this still something that you might consider for the future?

Keeping open...

The stock history-graph card properly shows unavailable values. I would be happy to replace it with apexcharts-card if this issue is resolved:
https://github.com/RomRider/apexcharts-card/issues/236

I'm going to work on that soon :)

cool! thank you! -------- Ursprüngliche Nachricht --------Von: "Jérôme W." @.>Datum: Di., 4. Jän. 2022, 18:02An: RomRider/apexcharts-card @.>Cc: fullcavediver @.>, Author @.>Betreff: Re: [RomRider/apexcharts-card] Support for binary sensor history graph (#109)
I'm going to work on that soon :)
—Reply to this email directly, view it on GitHub, or unsubscribe.Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.Message ID: @.***>

Keeping it open.

.

.

Hallo, I also follow this, but seems like noone can come with sufficient solution so far - unfortunatelly

.

The other option is to use history-graph card and pre-define the view for the history needed to be shown...
then spanning and zooming to detail is also possible without any limitation, the only thing is, that this graph looks a bit ugly (and can't be customized much)
(imho)

Hallo Romrider - any chance plz that we can look forward for horizontal bar graphs with history? :-)
(keeping this thread alive, but I guess this would be my last time to prolong the dying of this beast...)

I'm going to work on that soon :)
Hallo Romrider - any chance plz that we can look forward for horizontal bar graphs with history? :-)
(keeping this thread alive, but I guess this would be my last time to prolong the dying of this beast...)

.

.

.

seems like no solution so far...
I would be satisfied also if we could only draw these simplified graphs in apex chart:

then I may be able to combine it with minigraph card and have these cards showing the data correctly with no waste of space on the dashboard...

.

.

.

Is it still not possible to use a chart like this one?

The colours can at least now be manipulated directly in HA: https://www.home-assistant.io/integrations/frontend/.


Error: Too many private routes

lnd seems to check that invoices have less than 20 route hints, but creates invoices with more than 20 route hints itself.
i believe this happened when creating an invoice on a recent lnd build with 250+ private channels.
for more information you can take a look at this issue:
https://github.com/LN-Zap/zap-iOS/issues/182
sorry about the little amount of information. 🙏

lnd shouldn't create invoices with more than 20 route hints.
https://github.com/lightningnetwork/lnd/blob/aa1cd04dbf07a9195d5ada752f383988d8d01fa7/lnrpc/invoicesrpc/addinvoice.go#L271

just woke up so I might be wrong:
shouldn't this break if numHints >= 20 otherwise it will add 21 hints 🤔
right now the check passes when it's 20 and adds another hint.

You're right, nice catch.


[Python] Add workaround for NPY_LONGLONG == NPY_INT64 on Apple i386

Close #129. See discussion there.
On Apple's universal / i386 built-in Python, NPY_LONGLONG and NPY_INT64 are not distinct types, however both enum values can occur in the wild. This prevents a compilation error on these platforms.

[Current coverage][cc-pull] is 79.82%

Merging [#133][cc-pull] into [master][cc-base-branch] will increase coverage by +0.98%

@@             master       #133   diff @@
==========================================
  Files            50         45      -5
  Lines          2901       2597    -304
  Methods          36          0     -36
  Messages       1434          0   -1434
  Branches        402          0    -402
==========================================
- Hits           2287       2073    -214
+ Misses          575        524     -51
+ Partials         39          0     -39

14 files (not in diff) in cpp/src/feather were modified. more

Misses -1
Partials -38
Hits +39

1 files (not in diff) in cpp/src were modified. more

Partials -1
Hits +1

4 files (not in diff) in python/feather were deleted. more
1 files (not in diff) in python were deleted. more

Powered by Codecov. Last updated by [ad390a4...f294c3b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/wesm/feather/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/wesm/feather/compare/ad390a4be8d9759356344e7678373e138b6450af...f294c3b69a732417910000939b8a7c53fc2ad833
[cc-pull]: https://codecov.io/gh/wesm/feather/pull/133?src=pr


fix: unterminated string literal on table update

resolves #1429
(python)
-    return f"'{value}'"
+    return f'"{value}"'

Sure, I will add one tonight.


Table col name auto-completion?

for table which type defined in default_db, it is ok to check if table.col1 is valid.
However, auto -completion is not available.
e.g.
module default_db {
type tableType = [{c1 = text, c2 = text}]
let table1<tableType>
}
from table1
select {
  test = table1.xxx
}

Yes, this is something we really want to add, but we're not there yet in the compiler. We're currently rewriting the resolver, so this sort of thing will come after that. Check out https://prql-lang.org/roadmap/ for more...


'breakindent' is not drawn after diff filler lines

Problem:  'breakindent' is not drawn after diff filler lines.
Solution: Correct check for whether 'breakindent' should be drawn.

wow, it has been like this for a very long time and nobody noticed...

Yes, and the cursor position still takes 'breakindent' into account, so it doesn't match drawn text.


how to push route without tab in tabs

I run the example code, all works great.  I want the feature :
the first screen is with tab, then push the 2nd screen,  without tab showing, is this possible ?  if possible how to do it ?

I have no idea about Android. From iOS, it is supported from native code, using hidesBottomBarWhenPushed to the pushed view controller. To make it work when calling from JS however, you need to modify this method to be able to do what you want.

@esam091  thanks.  that can solve it.  if this library can make it like navigation like instagram click the comment icon, it would be better.
i guess it maybe hard to do it, because native-navigation use one global navigation?

I've been using the library the last days, and i am really enjoying it, but it seems that the tabs API isn't that ready, and pushing or presenting screens doesn't do anything with tabs, and the mode  ('screen' | 'tabs') property is not there yet.
A better solution to work around this, and not change the library itself, would be to write a small native module to extend the native navigation to your needs.
I have a similar working case, a Login screen that doesn't has tabs and pushes to a Dashboard screen that has tabs, and the Dashboard screen has a List that each item pushes a 2nd screen without showing the tabs.
DashboardController.swift

import NativeNavigation

@objc(DashboardController)
class DashboardController : NSObject {
  fileprivate let coordinator: ReactNavigationCoordinator

  override init() {
    coordinator = ReactNavigationCoordinator.sharedInstance
  }

  @objc func presentScreenWithTabs() -> Void {
    DispatchQueue.main.async {
      let nav = self.coordinator.topNavigationController()
      let screen = ReactTabBarController(moduleName: "ScreenName")
      nav?.presentReactViewController(screen, animated: true, completion: nil)
    }
  }

  @objc func pushScreenWithoutTabs(_ props: [String: AnyObject]) -> Void {
    DispatchQueue.main.async {
      let nav = self.coordinator.topNavigationController()
      let screen = ReactViewController(moduleName: "ScreenName", props: props)
      screen.hidesBottomBarWhenPushed = true
      nav?.pushViewController(screen, animated: true)
    }
  }
}

Link the functions with the bridge.
DashboardBridge.m

#import "React/RCTBridgeModule.h"

@interface RCT_EXTERN_MODULE(DashboardController, NSObject)

RCT_EXTERN_METHOD(presentScreenWithTabs)
RCT_EXTERN_METHOD(pushScreenWithoutTabs:(NSDictionary *) props)

@end

Then invoke it as follows

import { NativeModules } from 'react-native'

handlePress() {
  NativeModules.DashboardController.pushScreenWithoutTabs()
}

Note that this isn't actually the best solution, but just a way to extend the native-navigation to our needs, until we don't have a full tabs API, which may take some time.
I will try to update my answer for android when i get there.

fwiw, I've implemented this over on our fork: https://github.com/taxfix/native-navigation/pull/9 (amongst some other things, ymmv)


Allow usage of multiple input files

We use schema-registry-gitops in our build pipeline to manage the schema registry.
The tool currently only supports the use of one input file, which becomes confusing and long with a larger number of subjects/schemas.
Would it be possible to extend the functionality so that multiple input files could be used? For example, to allow separation by domains

First of all: thanks for using schema-registry-gitops! Would totally make sense to load different files and I think I would be possible as well. Give me some time to try out what the best way to implement is and I'll get back to you.

@S1M0NM I merged support for multiple input files and released 1.7.0 🎉

@domnikl Thank you for the quick implementation! Tested it and it works as expected.


V3 content

This branch tracks work on Spin 3 updates.  Initially it's a clone of the Spin 2 docs but this is where we will add / remove / tweak for the v3 stuff.
Content must go through a pre-merge checklist.
Pre-Merge Content Checklist
This documentation has been checked to ensure that:

[ ] The title, template, and date are all set
[ ] Does this PR have a new menu item (anywhere in templates/*.hbs files) that points to a document .md that is set to publish in the future? If so please only publish the .md and .hbs changes in real-time (otherwise there will be a menu item pointing to a .md file that does not exist)
[ ] File does not use CRLF, but uses plain LF (hint: use cat -ve <filename> | grep $'\r' | wc -l and expect 0 as a result)
[ ] Has passed bart check
[ ] Has been manually tested by running in Spin/Bartholomew (hint: use PREVIEW_MODE=1 and run npm run styles to update styling)
[ ] Headings are using Title Case
[ ] Code blocks have the programming language set to properly highlight syntax and the proper copy directive
[ ] Have tested with npm run test and resolved all errors
[ ] Relates to an existing (potentially outdated) blog article? If so please add URL in blog to point to this content.

Superseded by #1392 which will allow us to more readily PR topic changes on top of it.


[BXMSDOC-4670, -4671, -4672] Review of Kogito Operator CLI messaging, CSV files, and README

See JIRAs for details:

BXMSDOC-4670: General review
BXMSDOC-4671: CSV files
BXMSDOC-4672: CLI messaging

@ricardozanini , ready for your review. I apparently don't have perms to assign you as reviewer, so pinging you.
Just FYI, I reviewed for overall consistency, clarity, and correctness. I didn't do a full-blown review like we do with official docs, which follow strict template and stylistic requirements. But this brings things together a bit more nonetheless.
One thing, for future ref: Going forward, we should be using sentence-style headings ("Kogito CLI installation") instead of headline style headings ("Kogito CLI Installation"). I've already made this change, along with my others. This is more consistent with other Kogito docs, Red Hat docs, and the industry best practice that is becoming more prominent.

@sterobin many thanks for this! We're about to merge https://github.com/kiegroup/kogito-cloud-operator/pull/104, that introduce a whole new section to the docs. Do you mind rebasing your PR after we merge it and do a quick review there also?

@ricardozanini Will do.

Hi @sterobin ! We just merged the mentioned PR, could you please rebase your PR?
Then I'll apply your changes and follow up in the future with your recommendations.
Many thanks!

@ricardozanini , done. Rebased, resolved conflicts, and fixed a few other things. My changes to the TOC in the README might be hard to follow in the PR given how I had to bring in latest changes and then tweak those. So you might need to refer to the README rendering from my fork for the TOC at least. And I was careful and thorough with ensuring that all text and links that I changed in the TOC were updated throughout the project where applicable.

@xiezhang7 @radtriste could you guys please take a look?

@ricardozanini , all fixed, and re-squashed.


Work with epartment/nova-dependency-container

I need make nova-checkboxes work with nova-dependency-container on a specific checkbox.
Could you help me?

@lfukumori , Were you able to have the checkboxes working with nova-dependency-container ? If yes what was the solution.
Thanks !


chore: backport tss updater

backports https://github.com/zeta-chain/protocol-contracts/pull/363

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (node/v20.0.2@a144b7a). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             node/v20.0.2     #367   +/-   ##
===============================================
  Coverage                ?   88.82%
===============================================
  Files                   ?        7
  Lines                   ?      331
  Branches                ?      103
===============================================
  Hits                    ?      294
  Misses                  ?       37
  Partials                ?        0

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Make the deployment idempotent.

Still a work in progress. Adding guards to statements where I can. Some will need to be re-written slightly, like the file downloads in bash execute blocks.
My editor also deleted a lot of hanging whitespace.

All tests passing on Vagrant ESX, Vagrant Virtualbox, and Physical builds.  Shipping it.