python3Packages.pymisp: init at 2.4.174

This PR adds the pymisp package - https://pymisp.readthedocs.io/en/latest/ to the list of python packages.

This is a great first attempt without having built it!
Let's add these changes (I built something quite like this just now, but it will need a separate check). I just want to update your PR title and these changes should do the trick.

Thanks for proposing the changes. I committed all the proposed changes. Let me know if this looks good and also what would be the best PR title/description.

This is a great first attempt without having built it!
Let's add these changes (I built something quite like this just now, but it will need a separate check). I just want to update your PR title and these changes should do the trick.

Thanks for proposing the changes. I committed all the proposed changes. Let me know if this looks good and also what would be the best PR title/description.

Let's give them a quick test and this should be good!

can you apply this please, then squash the commits into one? i just built with these changes:
commit 046686cd8aafbfe02188e03ecf182243eb0fc36d
Author: John Soo<EMAIL_ADDRESS>Date:   Wed Aug 2 09:02:41 2023 -0700

    fixes

diff --git a/pkgs/development/python-modules/pymisp/default.nix b/pkgs/development/python-modules/pymisp/default.nix
index b7f9347fe0d2..74ebd04d0ca9 100644
--- a/pkgs/development/python-modules/pymisp/default.nix
+++ b/pkgs/development/python-modules/pymisp/default.nix
@@ -7,38 +7,44 @@ let version = "2.4.174"; in
 buildPythonPackage {
   pname = "pymisp";
   inherit version;
-  version = "2.4.174";
-  pname = "pymisp";
-  format = "pyproject.toml";
+  format = "pyproject";
   disabled = pythonOlder "3.8";
+
   src = fetchFromGitHub {
     owner = "MISP";
     repo = "PyMISP";
     rev = "v${version}";
-    hash = "sha256-q67LDJiwibG1X9QtzYQSP/qSTSXDSHp60gJIa7coq+A=";
+    hash = "sha256-OBchUEz+ZK6XNbnFFL6nsCuXniZ0vAfdJLj6LaFC2i0=";
   };
+
+  patches = [ ./pymisp-remove-poetry-group.patch ];
+
   postPatch = ''
     substituteInPlace pyproject.toml \
-      --replace 'jsonschema = "^4.5.1"' 'jsonschema = "*"'
+      --replace 'jsonschema = "^4.18.4"' 'jsonschema = "*"' \
+      --replace 'requests = "^2.31.0"' 'requests = "*"' \
+      --replace 'deprecated = "^1.2.14"' 'deprecated = "*"' \
+      --replace 'publicsuffixlist = {version = "^<IP_ADDRESS>30730", optional = true}' ""
   '';

   nativeBuildInputs = [ poetry-core ];
-
+
   checkInputs = [ pytestCheckHook pytest-mock ];

   propagatedBuildInputs = [
     requests python-dateutil jsonschema deprecated
   ];

-  # "testing optional email feature"
+  # testing optional email feature
   pytestFlagsArray = [ "--ignore tests/test_emailobject.py" ];

   disabledTests = [
-    "test_mimeType"
+    "test_mime"
     "test_default_attributes"
     "test_feed"
     "test_git_vuln_finder"
     "test_obj_default_values"
+    "test_object_galaxy"
     "test_object_tag"
     "test_to_dict_json_format"
   ];
@@ -48,6 +54,6 @@ buildPythonPackage {
     homepage =  "https://github.com/MISP/PyMISP";
     maintainers = [ maintainers.jsoo1 ];
     platforms = platforms.unix;
-    license = [ licenses.bsd2 ];
+    license = licenses.bsd2;
   };
 }
diff --git a/pkgs/development/python-modules/pymisp/pymisp-remove-poetry-group.patch b/pkgs/development/python-modules/pymisp/pymisp-remove-poetry-group.patch
new file mode 100644
index<PHONE_NUMBER>00..0c17d0c358ec
--- /dev/null
+++ b/pkgs/development/python-modules/pymisp/pymisp-remove-poetry-group.patch
@@ -0,0 +1,25 @@
+diff --git a/pyproject.toml b/pyproject.toml
+index 7fc1a65..15548d8 100644
+--- a/pyproject.toml
++++ b/pyproject.toml
+@@ -71,20 +71,6 @@ url = ['pyfaup']
+ email = ['extract_msg', "RTFDE", "oletools"]
+ brotli = ['urllib3']
+
+-[tool.poetry.group.dev.dependencies]
+-requests-mock = "^1.11.0"
+-mypy = "^1.4.1"
+-ipython = [
+-    {version = "<8.13.0", python = "<3.9"},
+-    {version = "^8.13.0", python = ">=3.9"}
+-]
+-jupyterlab = "^4.0.3"
+-types-requests = "^<IP_ADDRESS>"
+-types-python-dateutil = "^<IP_ADDRESS>"
+-types-redis = "^<IP_ADDRESS>"
+-types-Flask = "^1.1.6"
+-pytest-cov = "^4.1.0"
+-
+ [build-system]
+ requires = ["poetry_core>=1.1", "setuptools"]
+ build-backend = "poetry.core.masonry.api"

Applied the changes and squashed the commits into one. Thanks @jsoo1.
Let me know if this looks good.


Show full type in typescript type hover hint

When hovering over a Union type with a lot of options, some of the types are not shown.  As far as I can tell there is no way to "view all" or "view more".  Having this functionality would be very helpful.

Duplicate of #64566


Developer Guide: Standardised User Case Format

Changed up UC1 to follow the rest since UC1 is the only oddity.
Also made some other minor changes.

Codecov Report

Merging #25 (dec65d3) into master (f6c9706) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

Continue to review full report at Codecov.

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


monitoring: Add GitOps Service's Argo CD Dashboard

The contents of the dashboard that monitors the Argo CD instance in gitops-service-argocd is already part of the ConfigMap, we just need to create the corresponding GrafanaDashboard CR.

Merging as tests are passing, and it's a dashboard change only.


Fixed tutorial - code block ended early

This is the same as https://github.com/angular-architects/module-federation-plugin/pull/356


Outline doesn't respect win_position + previews don't popup on subsequent toggles

(n.b.: let me know if these should be separate issues)
Expected Behavior

On default config, or when specifically setting win_position = 'right', the symbols outlines should appear on the right side.
When toggling the outline on for the second time or more, the symbols preview should appear, like it does on the first toggle.

Actual Behavior
win_position is always on the left side, regardless of config; but, the preview window pops up on the expected side.
Also, the outline's preview popup window only appears on the initial outline toggle, but not on subsequent outline calls.

Details
config
-- packer config
    use {
      'wbthomason/packer.nvim',
        }
    use {
      'williamboman/nvim-lsp-installer',
      'neovim/nvim-lspconfig',
      'glepnir/lspsaga.nvim',
    }
-- packer config ^^

local saga = require 'lspsaga'
saga.init_lsp_saga({
  show_outline = {
    win_position = 'right',
    auto_enter = true,
    auto_preview = true,
    virt_text = '┃',
    jump_key = 'o',
    -- auto refresh when change buffer
    auto_refresh = true,
  },

})

-- Outline toggle
vim.keymap.set("n", "<Leader>a", ':LSoutlineToggle<CR>', { silent = true, noremap = true })

Reproduce

:LSoutlineToggle
Navigate outline to see previews
Repeat

Environment

nvim --version output: 0.7.2 (also on nightly, 8.0)
Operating system: MacOS 12.4
lspsaga commit: #362

@fennelcakes  did you config the set nosplitright ?

I add a patch that can fix this win position issue.
about preview . when you close outline. then open it again there is no preview window?

Correct, like so:

fixed.

fixed.

Yes, works as intended! You're incredible, thanks so much for your work on this plugin.  I need to learn lua properly so I can contribute.

I'm having an issue where the outline window open on the right side from the currently focused window but the float window is always relative as if it were in the right of the vim window. I'm using default config. I've just configured keymaps.

@augustocdias  create a new issue for this . fix it later. go to dinner now


c++: error: unrecognized command line option ‘-mavx2’

[ 15%] Building CXX object src/ATen/CMakeFiles/ATen.dir/native/cpu/ReduceOpsKernel.cpp.AVX.cpp.o
c++: error: unrecognized command line option ‘-mavx2’
src/ATen/CMakeFiles/ATen.dir/build.make:81862: recipe for target 'src/ATen/CMakeFiles/ATen.dir/native/cpu/ReduceOpsKernel.cpp.AVX2.cpp.o' failed
make[2]: *** [src/ATen/CMakeFiles/ATen.dir/native/cpu/ReduceOpsKernel.cpp.AVX2.cpp.o] Error 1
make[2]: *** Waiting for unfinished jobs....
c++: error: unrecognized command line option ‘-mavx’
src/ATen/CMakeFiles/ATen.dir/build.make:81886: recipe for target 'src/ATen/CMakeFiles/ATen.dir/native/cpu/ReduceOpsKernel.cpp.AVX.cpp.o' failed
make[2]: *** [src/ATen/CMakeFiles/ATen.dir/native/cpu/ReduceOpsKernel.cpp.AVX.cpp.o] Error 1
CMakeFiles/Makefile2:233: recipe for target 'src/ATen/CMakeFiles/ATen.dir/all' failed
make[1]: *** [src/ATen/CMakeFiles/ATen.dir/all] Error 2
Makefile:129: recipe for target 'all' failed
make: *** [all] Error
Has anyone ever met this question? How to solve it?

I met the same question on my mac M1 when installing STAR, waiting for solution


Pre-Launch Tasks

Tasks

update AppleID in About and EasterEgg screens
update About screen links
remove usesCleartextTraffic from AndroidManifest.xml

should we close this now @pwltr?

No, these are tasks for when we do a full launch, meaning moving from Testflight to App Store. We also use some non TLS endpoints currently that we should swap at some point.


fix:[CUS-286] Fix dips is collected snapshot assets for vertex

Description

issue: the issue is in the way we collect and describe snapshots in the collector. The collector seems to skip collecting if its not able to find and describe a snapshot id that it had listed
in this case, the snapshot could not be found using snapshotID
fix: handle exception during describing individual snapshots, so that it doesn't stop describing other listed snapshots
handled the case when snapshot could not be found using snapshotID

Problem
collector skips collecting snapshot if there is an issue describing a single snapshot
Solution
handle exception for individual snapshots
Fixes # (issue if any)
Type of change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Chore (no code changes)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

[ ] Replicate the issue by hardcoding an invalid snapshotID
[ ] Verify the other snapshots are collected

Checklist:

[ ] My code follows the style guidelines of this project
[ ] My commit message/PR follows the contribution guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Other Information:
List any documentation updates that are needed for the Wiki

Actual issue in collector loaderror

Collector exception:

@ershad-paladin please raise this to v1.8.0

@ershad-paladin please raise this to v1.8.0

@santhosh-challa raised PR to v1.8.0 https://github.com/PaladinCloud/CE/pull/2163


Switch off inline blame

The settings do not seem to offer any simple toggle to enable/disable inline blame. I am NOT talking about currentLine but ANY in-editor blame display, e.g. "You, 5 months ago"
I find these extra lines inside my code quite confusing.

Check out the Code Lens section of the GitLens interactive settings editor (from the command palette run GitLens: Open Settings). From there you can turn on/off those items (vscode calls them Code Lens) as well as most of the other GitLens UI features.
There is also a command GitLens: Toggle Git Code Lens to quickly toggle them back on and off -- though it will only affect the current window and will revert to the configured settings on a reload.
Hope that helps.

i only see toggles for the whole GitLens functionality, not for inline only?

Can you send a screenshot of what you are asking about?
Here are the code lens settings -- the image in the settings will dynamically update as you toggle the settings

Here are the current line settings:

Thanks! Wasn't aware of the gitLens own settings dialog, very convenient! And that the name I was looking for was codeLens!


[PL] add translation of enum lesson

in scope of issue #202

@kdebowski could you take a look at this when you have a moment?

@Koziolek I've found one unwanted word. Despite that word it's OK!

Fixed

@doomspork it's ready to be merged

Thank you @Koziolek & @kdebowski


Add PSGI support

This resolves [https://github.com/openshift/sti-perl/issues/58]. It requires the augmented test/run from [https://github.com/openshift/sti-perl/pull/60] pull request.

small nit, LGTM

@ppisar which PR you want to merge first :-) I think you will have to rebase the other one.

On Thu, Nov 12, 2015 at 01:32:14AM -0800, Michal Fojtik wrote:

@ppisar which PR you want to merge first :-) I think you will have to rebase
the other one.

I rebased and pushed both of them. The PSGI thing (PR #60) is built on top of
the extlib thing (PR #61) as it was previously.
So first is apply PR #60:
654099c2536dcc8703ee0d701933b7b0932a09fd
a0d33ab9e1b2332b07f0f9558d15bdf1012ce0eb
and then PR #61:
fc0eb0e0b0e98651ca23b8f998d1531012c803e1
de17b48b6ebf4080bad29a77f849e5a9a9d8ef69
Maybe GitHub cannot comprehend it. I can file a fresh new pull request, if it
help.
-- Petr

And maybe you got bitten by the silly GitHub's merge commits that prevents from fast-forwarding consequential patchsets (pull request in GH langugage). After applying previous one an artificial merge commit is inserted and that effectively diverts from next one commit line.
Should I rebase this pull request again against now-master? I can see previous robot comment "This branch is up-to-date with the base branch".

@ppisar I think rebase and force push should get rid of the first commit and prevent merge commit :-) I would prefer that if it is possible.

Done. Try it again.

Merged. Thanks!
On Nov 12, 2015 1:11 PM, "Petr Pisar"<EMAIL_ADDRESS>wrote:

Done. Try it again.
—
Reply to this email directly or view it on GitHub
https://github.com/openshift/sti-perl/pull/61#issuecomment-156087547.


Feature/passer and receiver roles

This branch includes a Pass Play and Passer and Attacker Roles

Closes #899 #902 #906


eml export form Exchange emails backup

Closing this as we are currently only planning for pst exports for exchange.

Re-opened - this is in-progress

eml export support is now available in main.


feat: As a user, I want to be able to see the list of attestations on Explorer

What does this PR do?
Related ticket
https://github.com/Consensys/linea-attestation-registry/issues/410
Type of change

[ ] Chore
[ ] Bug fix
[x] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

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

Comparison is base (a464ff0) 94.24% compared to head (bff64da) 94.24%.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #443   +/-   ##
=======================================
  Coverage   94.24%   94.24%
=======================================
  Files          19       19
  Lines         313      313
  Branches       68       68
=======================================
  Hits          295      295
  Misses          9        9
  Partials        9        9

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

@orbmis I've changed the naming according to your review, seems that I've fixed the all errors with linting and finally we have same entry point with explorer. We have two more PR we should rework during some error while moving explorer directory to website. This change makes all changes related to website folder. I've fixed that in this MR.


Adding a simple working program under the Usage section of the README

Hello,
I am writing a simple Go program which uses this package to make some API calls to GitHub as part of learning Go. It took me some redirections to get started with the package to make native Unauthenticated and Basic authenticated calls; probably due to my naivety at reading/writing Go code.
Hence, I thought of mentioning this so as to help beginners (like me) pick the package up quicker in the future. I would like to suggest having at least one complete working code snippet which can be used exclusively as a copy-paste program. This can be achieved in two ways:

Adding a link to the examples directory in the "Usage" section.
Modify the already present examples to be inclusive enough for direct use as recipes.

Since I am totally new to Go development environment, I beg pardon if my suggestions seem rather ingenuous to the more experienced community members.

Hi there, thank you for feedback and suggestions, they are appreciated.
I agree this is important, especially because go-github is a good first package to play with for people who just installed Go.
We do need to be careful to strike the right balance of providing the right amount of information in the README and package documentation (i.e., https://godoc.org/github.com/google/go-github/github), so that it's helpful and easy to scan for most people trying to look something up, as well as contain enough information for beginners to be able to get started.
I've looked at PR #810 and I think it's a good start. I have some feedback on how we can make it better, I'll post it there.

@kshitij10496 thanks for the feedback and #810. Highly appreciated.

Thank you guys for the appreciation.
It would be my pleasure to learn Go while contributing to this package.
Since all the issues tagged "good first issue" are currently under development and the endpoints mentioned in the roadmap have all been implemented(needs update), I would appreciate if you can point me to other beginner-friendly ways I can contribute to this package.

Since all the issues tagged "good first issue" are currently under development

Is that really the case? I'd suggest double checking if you haven't already. If an issue was started but there's been no progress for over a week, it's completely okay to ping and inquire on the status. Sometimes issues are started but then get stuck, so you might be able to resume one of the existing ones and help push it along.
Otherwise, it means there's not much to do, which isn't a bad thing. :) Just watch this repository and when a new issue comes up, see if it's something you'd want to try to tackle. Thanks again!

Hi,
I too am very new to Go but in investigating this repo I wrote up what I think you're talking about.
I never intended if for anyone else so it's rough and undocumented but if it's helpful I'd be happy to polish it up for you or anyone else:
https://github.com/FenwickElliott/GoGit/blob/master/main.go
wrt: the token, I simply saved it as a string in txt file called 'token' in the same directory. I have another script for gettin OAuth2 access through a browser if you'd like that too.


Updated the README for @backstage/plugin-newrelic

Updated the README for @backstage/plugin-newrelic to fix grammatical errors and update the current limitations of the plugin.
Hey, I just made a Pull Request!
Updated the README for the @backstage/plugin-newrelic plugin to fix grammatical errors and update the current limitations.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[x] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

Oh sorry, I missed the DCO check - you need to sign every single commit. Check out the instructions in the comment above.


Evaluation on KITTI-360 Test set

I have a problem viewing the performance evaluation numbers when i run
python src/eval.py experiment=kitti360  ckpt_path='downloaded checkpoint from your website'

below is the result i got
┏━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━┓
┃        Test metric        ┃       DataLoader 0        ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━┩
│         test/loss         │            0.0            │
│         test/macc         │            0.0            │
│         test/miou         │            0.0            │
│          test/oa          │            0.0            │
└───────────────────────────┴───────────────────────────┘

but when i trained a new model from scratch with the kitti360
python src/train.py experiment=kitti360

i can get view the numbers during training
val/miou_best: 63.757 val/oa_best: 92.886 val/macc_best: 79.989

I get this warning during evaluation
"You are using cuda device ('nvidia geforce rtx 4090') that has Tensor Cores. To properly utilize them you should set 'torch.set_float32_matmul_precision('medium' | 'high) which will trade-off precision for performance. For more details  read https://pytorch.org/docs/stable/generated.torch.set_float32_matmul_precision.html#torch.set_float32_matmul_precision
i tried using high and medium precision but theres no change in the evaluation result.
i used this project in a docker container with gpu passthrough according to your setup. cuda11.8
please help, thank you for the project

Hi @hansoogithub, thanks for your interest in the project.

I have a problem viewing the performance evaluation numbers when i run

This is normal behavior. KITTI-360's test set has held-out labels. Meaning you do not have access to the labels for performance evaluation, those are stored on a benchmarking server (see the official website. So the local performance evaluation of SPT can only be run on the validation set, as communicated in our paper. This is why you see empty test performance when running python src/eval.py experiment=kitti360.

I get this warning during evaluation
"You are using cuda device ('nvidia geforce rtx 4090') that has Tensor Cores. To properly utilize them you should set 'torch.set_float32_matmul_precision('medium' | 'high) which will trade-off precision for performance. For more details read https://pytorch.org/docs/stable/generated.torch.set_float32_matmul_precision.html#torch.set_float32_matmul_precision

This is unrelated to the above comment. You can safely ignore this warning.
Best,
Damien

how to get evaluation metrics on validation data[kitti-360]? where should I mention that to get evaluation metrics?


feat: support darwin

I tried to test this on my MBP with m1 chip. nix build . works.
I tried to build .#cache but failed.
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking sources
unpacking source archive /nix/store/y941nzsc6gnvxm5j9b1f82914lvjfzab-crate-cpufeatures-0.2.4.tar.gz
source root is cpufeatures-0.2.4
setting SOURCE_DATE_EPOCH to timestamp<PHONE_NUMBER> of file cpufeatures-0.2.4/tests/x86.rs
@nix { "action": "setPhase", "phase": "patchPhase" }
patching sources
@nix { "action": "setPhase", "phase": "updateAutotoolsGnuConfigScriptsPhase" }
updateAutotoolsGnuConfigScriptsPhase
@nix { "action": "setPhase", "phase": "configurePhase" }
configuring
@nix { "action": "setPhase", "phase": "buildPhase" }
building
Building lib: RUSTC 'src/lib.rs' '--out-dir=/nix/store/0gv5l8sd6zxq5j2gajaggzlvl6gwsfml-rust_cpufeatures-0.2.4/lib' '--crate-name=cpufeatures' '--crate-type=lib' '--emit=metadata,link' '-Cembed-bitcode=no' '-Cextra-filename=-d0377ca44dd1550e' '--color=always' '-Copt-level=3' '-Cdebuginfo=0' '-Cdebug-assertions=no' '-Coverflow-checks=no' '-Clto=no' '-Cpanic=unwind' '-Ccodegen-units=16' '--cap-lints=allow' '-Cmetadata=d0377ca44dd1550e' '--edition=2018' '-Ldependency=/nix/store/fawhjckz2y94w3brj9qxpsc43ivmbxqc-rust_cpufeatures-0.2.4-dev/rust-support/deps-closure'
error[E0433]: failed to resolve: use of undeclared crate or module `libc`
   --> src/aarch64.rs:133:14
    |
133 |     let rc = libc::sysctlbyname(
    |              ^^^^ use of undeclared crate or module `libc`

error[E0433]: failed to resolve: use of undeclared crate or module `libc`
   --> src/aarch64.rs:135:38
    |
135 |         &mut value as *mut _ as *mut libc::c_void,
    |                                      ^^^^ use of undeclared crate or module `libc`

error: aborting due to 2 previous errors

For more information about this error, try `rustc --explain E0433`.

I don't think it's related to my modification but still needs some investigation.

It failed on CI. Seems to be missing dependencies.

:thinking: Just a remind. Is this still ongoing?

I'm a bit busy these days so I might finish this later.


When a new course is created, the start and end date should automatically display the selected period's dates.

@thdebay  is this issue closed?

The issue was closed when a temporary fix was made. When creating a course, the default start/end dates are initialized to the current period's dates. But of course it would be better to dynamically update the course start/end date fields, when a different period is selected in the "period" form control, and I'll accept a PR that adds this feature


Issue with using a component in a file and style attributes with a semicolon

I'm noticing this behavior:

With this minimum reproducible example:
package components

templ Example() {

		@templ.Raw("

")
		Hello

}

I have a file where all syntax is highlighted gray between the component usage and an inline style (like 60 lines for me). If you delete the semicolon, highlighting appears fine.
I'll change my file, but wanted to raise this issue. I'd try and figure it out and submit a PR but I have no idea how treesitter parser development works
EDIT: I am noticing this with any semicolon. If I have an javascript inside an HTML tag, I cannot use a semicolon without strange behavior

Thanks for reporting the issue.
Looks like parsing is broken, this is why you have weird highlighting:
(source_file [0, 0] - [8, 0]
  (package_clause [0, 0] - [0, 12]
    (package_identifier [0, 8] - [0, 12]))
  (component_declaration [2, 0] - [7, 1]
    name: (component_identifier [2, 6] - [2, 13])
    (parameter_list [2, 13] - [2, 15])
    (component_block [2, 16] - [7, 1]
      (element [3, 1] - [5, 37]
        (tag_start [3, 1] - [3, 10]
          name: (element_identifier [3, 2] - [3, 9]))
        (ERROR [4, 2] - [5, 33]
          (component_identifier [4, 3] - [4, 8])
          (css_property_value [4, 9] - [5, 25])
          (identifier [5, 28] - [5, 33]))
        (tag_end [5, 33] - [5, 37]
          name: (element_identifier [5, 35] - [5, 36])))
      (ERROR [6, 1] - [6, 11]
        (identifier [6, 3] - [6, 10])))))
foo.templ	0 ms	(ERROR [4, 2] - [5, 33])

by the way without a ; it also doesn't parse correctly but the error is different:
(source_file [0, 0] - [8, 0]
  (package_clause [0, 0] - [0, 12]
    (package_identifier [0, 8] - [0, 12]))
  (component_declaration [2, 0] - [7, 1]
    name: (component_identifier [2, 6] - [2, 13])
    (parameter_list [2, 13] - [2, 15])
    (component_block [2, 16] - [7, 1]
      (element [3, 1] - [6, 11]
        (tag_start [3, 1] - [3, 10]
          name: (element_identifier [3, 2] - [3, 9]))
        (component_import [4, 2] - [4, 27]
          name: (component_identifier [4, 3] - [4, 8])
          (ERROR [4, 8] - [4, 12]
            (identifier [4, 9] - [4, 12]))
          arguments: (argument_list [4, 12] - [4, 27]
            (interpreted_string_literal [4, 13] - [4, 26])))
        (element [5, 2] - [5, 36]
          (tag_start [5, 2] - [5, 27]
            name: (element_identifier [5, 3] - [5, 4])
            (attribute [5, 5] - [5, 26]
              name: (attribute_name [5, 5] - [5, 10])
              value: (quoted_attribute_value [5, 11] - [5, 26]
                (attribute_value [5, 12] - [5, 25]))))
          (element_text [5, 27] - [5, 32])
          (tag_end [5, 32] - [5, 36]
            name: (element_identifier [5, 34] - [5, 35])))
        (tag_end [6, 1] - [6, 11]
          name: (element_identifier [6, 3] - [6, 10]))))))
foo.templ	0 ms	(ERROR [4, 8] - [4, 12])

I'll try to take a look.


Adapt aca-py start readme example

Adapt README example to change "old-school" -oq parameter into -ot parameter.

Pending modules for the plugin have now been implemented. This should be fixed along with other issues. Appreciate your contribution.


LL-6664  Button spinner

Add a spinner to existing Button component that appear when button is pressed and onPress function is a Promise.
https://user-images.githubusercontent.com/89014981/130989447-bb19f7a6-d46c-4bfa-83df-42fa455cb173.mp4
Type
Rebranding component
Context
[LL-6664]
Parts of the app affected / Test plan
Uilib / storybook

could you separate this in a new component like PromiseButton or something i don't want to have the logic of the spinner all over the app on, places where it should not occur

Or maybe an option to disable this behavior and logic, true by default ?
After all, it's still a button and need all the same props and params of Button.

could you separate this in a new component like PromiseButton or something i don't want to have the logic of the spinner all over the app on, places where it should not occur

Or maybe an option to disable this behavior and logic, true by default ?
After all, it's still a button and need all the same props, styles and params of Button.

Just be careful not to have Animated nodes that are passively computing stuff even if unused, its a terrible performance issue to have on a widely used component cause it scales with the usages in the pages.
Either you split it or you do a split in an entry component for the logic to not even be on the base button :shrug:

could you separate this in a new component like PromiseButton or something i don't want to have the logic of the spinner all over the app on, places where it should not occur

Or maybe an option to disable this behavior and logic, true by default ?
After all, it's still a button and need all the same props, styles and params of Button.

Just be careful not to have Animated nodes that are passively computing stuff even if unused, its a terrible performance issue to have on a widely used component cause it scales with the usages in the pages.
Either you split it or you do a split in an entry component for the logic to not even be on the base button

I have now splitted the spinner button to a special "PromisableButton"

closing this as it has been included in ledgerHQ/ui


Optional links?

Links should not be optional since every entity must contain at least one link to itself.

According to the spec, self links are a SHOULD, not a MUST.


Unable to install High Sierra (no FS1)

Hi @geerlingguy,
thanks for your time and effort you put together to make this work. It saved me a lot of time, but unfortunatly I'm stuck, right before it should all work. I tried all ways and also downgraded VirtualBox to 5.2.16 r123759. I also experienced https://github.com/geerlingguy/macos-virtualbox-vm/issues/52 and used the solution from https://github.com/geerlingguy/macos-virtualbox-vm/issues/52#issuecomment-421266398.
After following your steps closely, I'm always stuck with a system, which is booting into "Recovery Mode", if I leave the DVD in place or I only see the following screen:

Since there is no FS1, I think this issue is not related to https://github.com/geerlingguy/macos-virtualbox-vm/issues/53 and as you can see in the screenshot, there is data folder in FS0.
I'm wondering if someone could help me out here. I'm a little bit stuck here and would really like to bring High Sierra into VirtualBox to separate several tasks and processes for testing etc.
Thanks again for all the work and effort!

I had the same problem. Don't use APFS, use Mac OS Extended Journaled when you format your disk. It appears Virtualbox doesn't support APFS.

Thanks for helping out. I read about that at https://github.com/AlexanderWillner/runMacOSinVirtualBox and decided to give that a try. So I'm fine, running Mojave now on VirtualBox, they use Clover for booting, even with APFS.


[Bug]: Very Tiny Radio Textures on screen

I can barely make it out but it appears to be textures from the radio (Perhaps others too). On all screens except actual race:

That might be the FXT overlay, the controls it shows there seem to be arcade ones from FXT anyway - maybe try running the game through OR2006C2C.exe instead of OR2FXT.exe, might also help with the Splash Wave issue you had too.
Did hear some had FXT working fine with tweaks, but really FXT never launched properly for me even with vanilla so I haven't been able to test with it... (if someone knows how to get them working together properly I'd be happy to update readme about it though)

When I used the  OR2006C2C.exe instead of OR2FXT.exe, your correct it solve the issues. But I also noticed that OR2FXT.exe worked properly if I opened FXTConfig.exe and and enabled "Force Menus to 4:3". Unfortunately, the entire game (menus and all) now only opens up in a window 1/9 or so the size of my monitor.

Doesn't FXT use some weird bootleg overlay of images rather than actually replacing the textures? Maybe that caused it. They're not properly scaling.

Doesn't FXT use some weird bootleg overlay of images rather than actually replacing the textures? Maybe that caused it. They're not properly scaling.

from the image they provided, yes.
It's better to add proper texture mode for 2SP.

It would be better to play the game on a fresh install without FXT installed.


[Bug] semantic models always included as part of state:modified

Is this a new bug in dbt-core?

[X] I believe this is a new bug in dbt-core
[X] I have searched the existing issues, and I could not find an existing issue for this bug

Current Behavior
Currently, dbt always detects that semantic models have changed during state:modified, even if none of their model, description, defaults, entities, dimensions, measures, configs, primary entities, or groups have changed.
Expected Behavior
dbt only includes semantic models if one of the above listed properties has changed.
Steps To Reproduce
dbt ls --select "state:modified" on a project with unchanged semantic models
Relevant log output
No response
Environment
- OS:
- Python:
- dbt:

Which database adapter are you using with dbt?
No response
Additional Context
No response

I wasn't able to recreate this. Could you give me a brief reproduction test case?

I think it was already resolved by the combination of https://github.com/dbt-labs/dbt-core/pull/9549 and https://github.com/dbt-labs/dbt-core/pull/9561


Allow to choose redshift schema

I am able to run bqshift against public schema (by default) but I don't see any way to choose any other Postgres schema. I only can choose the database and the table, and by default, it asumes that table is located on the public schema, it is possible to change it?

@gepser   I have a branch on my fork with that option:
https://github.com/DanielSAndrade/bqshift/tree/search_path

Thank you for this. I tried to use it but didn't build, maybe I did something wrong.
But in the end, I was using segment and I think segment handle the migration better than myself so, for now, I don't need to do anything else.
Thank you @DanielSAndrade


Building example and linking to serac

Sorry I should have done this before the merging of #159 and reviewed it there. I went toexamples/ then mkdir using-with-cmake/build && cd using-with-cmake/build, I configure with ccmake, choose my compiler (same than I used for building serac, specified serac_DIRto~/workspace/repos/smith_install/lib/cmake` (installation directory) where the cmake files are. When I build, I get this linking error:
[ 50%] Linking CXX executable serac_example
/usr/tce/packages/binutils/binutils-2.30/bin/ld: cannot find -lmfem
collect2: error: ld returned 1 exit status
make[2]: *** [serac_example] Error 1
make[1]: *** [CMakeFiles/serac_example.dir/all] Error 2
make: *** [all] Error 2

Looking at the link.txt file:
CMakeFiles/serac_example.dir/link.txt
1:/usr/tce/packages/mvapich2/mvapich2-2.2-gcc-8.1.0/bin/mpicxx  -std=c++17
 CMakeFiles/serac_example.dir/serac_example.cpp.o
  -o serac_example -Wl,-rpath,/usr/workspace/miguel/repos/smith_install/lib:/usr/workspace/miguel/local/axom-develop/lib:/usr/workspace/miguel/local/conduit-master/lib:/usr/workspace/miguel/local/hdf5-1.8.21/lib
 /usr/workspace/miguel/repos/smith_install/lib/libcoefficients.so
/usr/workspace/miguel/repos/smith_install/lib/libsolvers.so
/usr/workspace/miguel/repos/smith_install/lib/libcontact.so
/usr/workspace/miguel/repos/smith_install/lib/libcommon.so
/usr/workspace/miguel/repos/smith_install/lib/libintegrators.so
/usr/workspace/miguel/local/axom-develop/lib/libaxom.so
/usr/workspace/miguel/local/conduit-master/lib/libconduit_relay.so -ldl -lrt -lpthread
/usr/workspace/miguel/local/conduit-master/lib/libconduit_blueprint.so
/usr/workspace/miguel/local/conduit-master/lib/libconduit.so
/usr/workspace/miguel/local/hdf5-1.8.21/lib/libhdf5.so
//usr/lib64/libz.so //usr/lib64/libdl.so //usr/lib64/libm.so -lmfem

It seems that the mfem library is not added to the rpath when building the examples. Do I know to call cmake with MFEM_DIR as well? It does not seem to be an option.

I also tried by using cmake -C host-config.cmake that I found in the examples folder (after installation). I ran into the same issue.


Prevent deployment creation while previous deployment is being canceled

Having a CANCELING and a (newer) DEPLOYING deployment in parallel results in unexpected and potentially non-deterministic results.
Mitigates issue #3019.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Superseded by #3072


[Feature] TeX Support

Problem
A lot of my personal notetaking includes taking notes with equations and LaTeX. Would it be possible to include this feature inside TakeNote? Otherwise, looks great.
Solution
We could either use KaTeX or MathJax to render math blocks. These blocks could be denoted by $$ and $$ as in $$\LaTeX$$ or a \( and \), such as \(\LaTeX\).

Thanks for the suggestion! I don't really intend to support LaTeX syntax in the application. The app is meant to be very simple and not add a lot of features to address every use case. This would be a bit of work to add complexity into the application for something I personally will never use.


Move ARMv8/gcc CI job from Travis to Azure

Looks great -- another 50 minutes out of the Travis burden!
Hopefully we get Shippable CI at some point for native ARMv8, which I suspect would be faster / more suitable in the long term.

@tylerjereddy, have you looked at drone.io for arm? It has parallel builds for open source projects. (conda-forge will be using drone.io instead of shippable in the near future)

@isuruf Oh cool! Never heard of it--no problems with GitHub integration / hours per month?

@tylerjereddy, works fine with github integration. No hours per month. 96 core native hardware.

I find the documentation at drone.io a bit confusing - while they do mention providing a free cloud service for opensource projects, the documentation appears to be all about setting up their CI framework on local hardware.

BTW the ARM/clang jobs timed out on Azure as well, so not much point in moving them.


Oracle update update error

Question
When I used sharding-jdbc update oracle 11g databases,then something wrong.

Error updating database.  Cause: java.sql.SQLException: ORA-01410 无效的 ROWID
Here is my pom.xml:
   <!-- sharding-jdbc -->   	<dependency>   	    <groupId>io.shardingsphere</groupId>   	    <artifactId>sharding-jdbc-spring-boot-starter</artifactId>   	    <version>3.0.0</version>   	</dependency>   	<dependency>   	    <groupId>io.shardingsphere</groupId>   	    <artifactId>sharding-jdbc-spring-namespace</artifactId>   	    <version>3.0.0</version>   	</dependency>

How can I fix it?

Can you provide necessary information? please reference by template

Project use mybatis to connect the databases。
Here is my mapper xml file.

update

CORP_BR_NO = #{corpBrNo,jdbcType=VARCHAR},
OPN_BR_NO = #{opnBrNo,jdbcType=VARCHAR},
AC_NO = #{acNo,jdbcType=VARCHAR},
SPLIT_TABLE_KEY = #{splitTableKey,jdbcType=VARCHAR},

rowid = #{rowid,jdbcType=VARCHAR}

I wanna update data by using rowid,but something wrong.

Can you provide necessary information? please reference by template
template FYI:
Bug Report
For English only, other languages will not accept.
Before report a bug, make sure you have:

Searched open and closed GitHub issues.
Read documentation: ShardingSphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.
Please answer these questions before submitting your issue. Thanks!
Which version of ShardingSphere did you use?
Which project did you use? Sharding-JDBC or Sharding-Proxy?
Expected behavior
Actual behavior
Reason analyze (If you can)
Steps to reproduce the behavior, such as: SQL to execute, sharding rule configuration, when exception occur etc.
Example codes for reproduce this issue (such as a github link).

If no response more than 7 days and we cannot reproduce it on current information, we will close it.


chore: move plugin_cmd to a package

Should lib/modules/plugin_cmd also be removed from packages/embark/src?

Suggestion: .npmrc, eslint config, and monorepo-related scripts can be copied/adapted from other packages.
rimraf can become a devDep and would be used in the "reset": script.

Good catch, @michaelsbradleyjr and @PascalPrecht.
This package has no transpilation nor tests, so I'm not going to add tasks that aren't necessary.

Could add a test stub   — requires the package's index.js without assignment to a variable and has a dummy test like expecting 1+1===2. That would help improve our understanding of what has coverage and what doesn't.
lint, package, clean and reset scripts still seem applicable — they're just part of the flow of being in the monorepo.

@michaelsbradleyjr you're probably right, but I wish we documented this process and what's required to have in a module instead of making it up as we go along.
I'll ask what I usually do: which things on that list fall in the context of this PR and which don't? Take under account that I based the new package from embark-reset, which follows the same structure.

@andremedeiros good points.
embark-reset happens to be a special case because it's designed in such a way that it doesn't require a build step, i.e. it will still run correctly for consecutive yarn reboot[:full] regardless of whether yarn build was run or not.
For most of the others, e.g. embark-graph, embark-process-logs-api, etc. there's a forward-looking setup that uses babel, has typescript related scripts in place (but disabled), and so on. I don't think having all that stuff in place is necessary for a refactor PR like this, and you're right that we can and should do a better job of documenting how we want these refactors accomplished.

I'll copy the stuff over from embark-graph


Feature/integrate app secret proof and bump version

Fixes https://github.com/jgorset/facebook-messenger/issues/239
So this was more work than I thought when I first looked in, due to not wanting to recalculate a hash on every request, here is an outline on the thoughts behind this:

appsecret_proof is a hash, so we don't want to calculate it on every request
deliver needs appsecret_proof and is also used internally by other classes
Common will need to cache the appsecret_proof to avoid redo-ing the work. so we need a default way of calculating this on the credential provider
deliver itself is a class method, and so probably shouldn't be doing the caching by itself, so it takes in a raw appsecret_proof by itself.

One possible extension is to have an environment_variable or something on the credential provider like ENV['USE_APP_SECRET_PROOF'] so that people can opt out of it in the common methods like reply and typing_on, (currently this will always happen).
Thoughts?

Hi Devs, do you have a rough timeframe for when this will be merged?
Let me know if there is anything I can help out with to get this ready and merged.
Thanks :)

I'll throw in the suggested improvements now. (sorry, meant to get to it earlier).

@egor-moysevich I made the suggested changes, and added a few more tests.
@jgorset I think this Pr is ready to be merged, what do you think? Let me know if you've got any questions.

@egor-moysevich comments addressed.

@mijoharas , thanks!
Just added one more comment to the one of resolved:
https://github.com/jgorset/facebook-messenger/pull/241#discussion_r272235438

@egor-moysevich sure, seemed reasonable.

Hey guys. Any chance we can get this merged? I don't want to be annoying, but we're rapidly approaching Facebook's April deadline (7 days away from today) for this and if it isn't available soon everyone using this gem will have their access cut off.
Happy to help in any capacity to get this over the line, but it looks like it's been approved and just needs to be merged in and published now. 👍

Hey @abury, i just don't know what is the release policy for this gem. For me it seems like we're waiting for approve from @jgorset and then we're ready to go with this.

Also, @abury, i've looked into Facebook docs, https://developers.facebook.com/docs/graph-api/changelog/, and 2.9 is available until July 18, 2019.
But anyway, my opinion, i don't see any reason why we should pause it until mid of summer.

Not needed for a while, but 2.9 is EOL on July 22nd (https://developers.facebook.com/tools/api_versioning).
From the issue I raised (@abury it looks like the 22nd of July, was 18th above a typo?). Either way, not urgent, but would be good to get this stuff live anyways.

Apologies for the confusion, I misread the date on Facebooks page, so certainly not as urgent as I thought. 22nd of July is indeed the cutoff date 👍

@abury Hahaha, I actually did the same thing at first. :)

Hi everyone! Sorry I couldn't get to this earlier -- I've been on holiday in Japan. :slightly_smiling_face:
This is one hell of a pull request. Thanks to everyone involved. I'll get it merged and bump the version.

This made it into version 1.2.0. Thanks @mijoharas! :sparkles: :cake: :sparkles:

Thanks @jgorset, and thanks for the review comments @egor-myosevich


Error while getting retrieving data for a newly created variable

Steps to reproduce,

Create a new variable with no qualifiers

test_variable = {
    "name": "test variable for test dataset",
    "variable_id": "TEST000-1"
}

Fetch data for newly created variable

[2020-07-08 16:47:28,052] ERROR in app: Exception on /datasets/TEST000/variables/TEST000-1 [GET]
Traceback (most recent call last):
  File "/Users/amandeep/Github/datamart-api/datamart_env/lib/python3.6/site-packages/flask/app.py", line 1950, in full_dispatch_request
    rv = self.dispatch_request()
  File "/Users/amandeep/Github/datamart-api/datamart_env/lib/python3.6/site-packages/flask/app.py", line 1936, in dispatch_request
    return self.view_functions[rule.endpoint](**req.view_args)
  File "/Users/amandeep/Github/datamart-api/datamart_env/lib/python3.6/site-packages/flask_restful/__init__.py", line 468, in wrapper
    resp = resource(*args, **kwargs)
  File "/Users/amandeep/Github/datamart-api/datamart_env/lib/python3.6/site-packages/flask/views.py", line 89, in view
    return self.dispatch_request(*args, **kwargs)
  File "/Users/amandeep/Github/datamart-api/datamart_env/lib/python3.6/site-packages/flask_restful/__init__.py", line 583, in dispatch_request
    resp = meth(*args, **kwargs)
  File "/Users/amandeep/Github/datamart-api/api/variable/main.py", line 11, in get
    return imp.get(dataset, variable)
  File "/Users/amandeep/Github/datamart-api/api/variable/get.py", line 87, in get
    return self.get_direct(dataset, variable, include_cols, exclude_cols, limit, regions)
  File "/Users/amandeep/Github/datamart-api/api/variable/get.py", line 188, in get_direct
    temp_cols)
  File "/Users/amandeep/Github/datamart-api/db/sql/dal/variables.py", line 289, in query_variable_data
    return query_to_dicts(query)
  File "/Users/amandeep/Github/datamart-api/db/sql/utils.py", line 55, in query_to_dicts
    cursor.execute(sql)
psycopg2.errors.SyntaxError: syntax error at or near "FROM"
LINE 8:     FROM edges AS e_main

This is because there are no qualifiers and the sql query formed is malformed as a result

Ah, a new variable with no uploaded data. I should definitely return an indication for this. It can either be 204 NO CONTENT, or perhaps 422 Unprocessable Entry with a message saying no data has been uploaded to the variable. Yes, I think an error with an explanation is better.

Fix pushed to development. A 422 error with the appropriate message is returned whenever there is no P585 qualifier associated with the variable.

Keeping this open until we have a proper fix

The API returns an empty string in this scenario, closing


YAML + Commit Names + Files

closes #245
Use YAML syntax to set the commit names and to add files

Will you update the level templates to showcase the YAML syntax in this PR? Or will that implemented elsewhere?

@sahansk2 I think this is done. Can you take a look to see if anything's missing or if any of the code doesn't make sense?

@sahansk2 can you check this out again. I updated this so files created have names such as 2.txt instead of 2 when using git gud commit or setting up a level. They also come with a message. Hopefully this resolves the issues you brought up

@benthayer Yep, this all looks good!


[EventRequest] AdjustGenJournalBalance.Codeunit.al (codeunit 407 "Adjust Gen. Journal Balance")

Hi MS,
Please help us in migrating our verticalized solution by adding below events in AdjustGenJournalBalance.Codeunit.al Codeunit.
Standard Code

Modified Code

Event Signature

Thanks
SUD

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


BACKLOG-19141 - Fixed serialization of UserDefinedJavaClass and RegEx…

@rmansoor @tkafalas

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  .gitignore
  engine/src/main/java/org/pentaho/di/trans/steps/named/cluster/NamedClusterEmbedManager.java
  engine/src/main/java/org/pentaho/di/trans/steps/regexeval/RegexEvalMeta.java
  engine/src/main/java/org/pentaho/di/trans/steps/userdefinedjavaclass/UserDefinedJavaClassMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.getpop.MailConnectionMeta

Branch Change:  + 1.00%
Complexity Change:  + 1.41%
Instruction Change:  + .30%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -.66%:small_red_triangle_down:
Complexity Change: -.93%:small_red_triangle_down:
Instruction Change: -.13%:small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change:  + .11%

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -1.92%:small_red_triangle_down:
Complexity Change: -1.69%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:
Instruction Change: -.14%:small_red_triangle_down:
Line Change: -.37%:small_red_triangle_down:

org.pentaho.di.trans.steps.ivwloader.IngresVectorwiseLoader

Branch Change:  + .71%
Complexity Change:  + 1.14%
Instruction Change:  + .55%
Line Change:  + .88%

org.pentaho.di.trans.steps.ivwloader.IngresVectorwiseLoader.FifoOpener

Branch Change:  + 16.67%
Complexity Change:  + 12.50%
Instruction Change:  + 7.14%
Line Change:  + 10.53%
Method Change:  + 20.00%

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.12%
Complexity Change:  + 1.30%

org.pentaho.di.trans.steps.named.cluster.NamedClusterEmbedManager

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -1.80%:small_red_triangle_down:
Instruction Change:  + .22%

org.pentaho.di.trans.steps.regexeval.RegexEvalMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.userdefinedjavaclass.UserDefinedJavaClassMeta

Instruction Change:  + .03%
Line Change:  + .04%

org.pentaho.di.www.TransformationMap

Branch Change:  + 1.14%
Instruction Change:  + 4.70%
Line Change:  + 3.85%

org.pentaho.di.trans.steps.googleanalytics.GaInputStepMeta

Instruction Change:  + .07%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change:  + .93%
Complexity Change:  + .80%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.salesforceinsert.SalesforceInsertMeta

Instruction Change:  + .15%

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change:  + .14%

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/trans/steps/regexeval/RegexEvalMeta.java. Found 2016. Was expecting 2017

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/trans/steps/userdefinedjavaclass/UserDefinedJavaClassMeta.java

:heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 427 Column: 45

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  .gitignore
  engine/src/main/java/org/pentaho/di/trans/steps/named/cluster/NamedClusterEmbedManager.java
  engine/src/main/java/org/pentaho/di/trans/steps/regexeval/RegexEvalMeta.java
  engine/src/main/java/org/pentaho/di/trans/steps/userdefinedjavaclass/UserDefinedJavaClassMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change:  + 3.28%
Line Change:  + 3.87%

org.pentaho.di.trans.step.StepMeta

Branch Change: -.68%:small_red_triangle_down:
Complexity Change: -.58%:small_red_triangle_down:
Instruction Change: -.19%:small_red_triangle_down:
Line Change: -.26%:small_red_triangle_down:

org.pentaho.di.trans.steps.combinationlookup.CombinationLookupMeta

Instruction Change: -.04%:small_red_triangle_down:

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change:  + .05%

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change:  + 1.92%
Complexity Change:  + 1.69%
Instruction Change:  + .17%

org.pentaho.di.trans.steps.dbproc.DBProcMeta

Instruction Change: -.11%:small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.22%:small_red_triangle_down:
Complexity Change: -.76%:small_red_triangle_down:
Instruction Change: -.29%:small_red_triangle_down:
Method Change:  + 1.01%

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change: -1.04%:small_red_triangle_down:
Complexity Change: -1.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change:  + 1.12%
Complexity Change:  + 1.30%

org.pentaho.di.trans.steps.named.cluster.NamedClusterEmbedManager

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -1.80%:small_red_triangle_down:
Instruction Change:  + .22%

org.pentaho.di.trans.steps.processfiles.ProcessFilesMeta

Branch Change: -3.57%:small_red_triangle_down:
Complexity Change: -2.13%:small_red_triangle_down:
Instruction Change: -.33%:small_red_triangle_down:

org.pentaho.di.trans.steps.regexeval.RegexEvalMeta

Instruction Change: -.03%:small_red_triangle_down:
Line Change:  + .05%

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change:  + .06%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.33%:small_red_triangle_down:
Complexity Change: -1.85%:small_red_triangle_down:
Instruction Change: -.28%:small_red_triangle_down:

org.pentaho.di.trans.steps.tableoutput.TableOutputMeta

Branch Change:  + .91%
Complexity Change:  + .82%

org.pentaho.di.trans.steps.userdefinedjavaclass.UserDefinedJavaClassMeta

Instruction Change:  + .03%
Line Change:  + .04%

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.79%:small_red_triangle_down:
Complexity Change: -1.19%:small_red_triangle_down:
Instruction Change: -.07%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -.17%:small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/trans/steps/regexeval/RegexEvalMeta.java. Found 2016. Was expecting 2017

Checkstyle violations
The code submitted is not complying with one or more of our guidelines. To perform these checks locally, please use this tool.
/home/buildguy/project/engine/src/main/java/org/pentaho/di/trans/steps/userdefinedjavaclass/UserDefinedJavaClassMeta.java

:heavy_exclamation_mark: '(' is not followed by whitespace. at Line: 427 Column: 45

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -B -fn -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl '.,engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  .gitignore
  engine/src/main/java/org/pentaho/di/trans/steps/named/cluster/NamedClusterEmbedManager.java
  engine/src/main/java/org/pentaho/di/trans/steps/regexeval/RegexEvalMeta.java
  engine/src/main/java/org/pentaho/di/trans/steps/userdefinedjavaclass/UserDefinedJavaClassMeta.java

Unit test coverage change
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.
org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change:  + .52%
Complexity Change:  + .56%
Instruction Change:  + .07%

org.pentaho.di.job.entries.sql.JobEntrySQL

Instruction Change: -3.28%:small_red_triangle_down:
Line Change: -3.87%:small_red_triangle_down:

org.pentaho.di.trans.step.BaseStep

Branch Change: -.15%:small_red_triangle_down:
Complexity Change: -.18%:small_red_triangle_down:
Instruction Change: -.05%:small_red_triangle_down:
Line Change: -.08%:small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change:  + .10%

org.pentaho.di.trans.steps.csvinput.CsvInputMeta

Instruction Change: -.05%:small_red_triangle_down:

org.pentaho.di.trans.steps.groupby.GroupByMeta

Branch Change:  + 3.12%
Complexity Change:  + 3.16%
Instruction Change:  + .14%
Line Change:  + .37%

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -.06%:small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -4.49%:small_red_triangle_down:
Complexity Change: -5.19%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:
Line Change: -.46%:small_red_triangle_down:

org.pentaho.di.trans.steps.named.cluster.NamedClusterEmbedManager

Branch Change: -.91%:small_red_triangle_down:
Complexity Change: -1.80%:small_red_triangle_down:
Instruction Change:  + .22%

org.pentaho.di.trans.steps.regexeval.RegexEvalMeta

Instruction Change: -.03%:small_red_triangle_down:
Line Change:  + .05%

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change:  + .08%

org.pentaho.di.trans.steps.setvalueconstant.SetValueConstantMeta

Instruction Change: -.14%:small_red_triangle_down:

org.pentaho.di.trans.steps.singlethreader.SingleThreaderMeta

Instruction Change: -.10%:small_red_triangle_down:

org.pentaho.di.trans.steps.sortedmerge.SortedMergeMeta

Instruction Change:  + .17%

org.pentaho.di.trans.steps.synchronizeaftermerge.SynchronizeAfterMergeMeta

Instruction Change:  + .04%

org.pentaho.di.trans.steps.textfileinput.TextFileInputMeta

Branch Change: -.53%:small_red_triangle_down:

org.pentaho.di.trans.steps.userdefinedjavaclass.UserDefinedJavaClassMeta

Instruction Change:  + .03%
Line Change:  + .04%

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -.93%:small_red_triangle_down:
Complexity Change: -.80%:small_red_triangle_down:
Instruction Change: -.17%:small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change:  + .17%

org.pentaho.di.trans.steps.salesforceupsert.SalesforceUpsertMeta

Instruction Change:  + .14%


PICO PIO testing with WS2812B

Congratulations Patricio! I've already managed to do some tests on the Raspberry Pico with the WS281B example. Thank you.
_https://github.com/tinygo-org/pio/tree/main/rp2-pio/examples/ws2812_
But I found some things that need correction or change.
ws, err := piolib.NewWS2812(sm, ws2812Pin, 16000000)
This function lacked the definition of the Baud rate.
Through testing, I discovered the frequency of 16MHz (maybe it is based on Arduino?).
func NewWS2812(sm pio.StateMachine, pin machine.Pin, baud uint32) (*WS2812, error)
The color sequence appears to be incorrect.
Based on the WS2812B datasheet, we can verify that the color sequence in the 24 bits is green-red-blue and not RGB.
_https://cdn-shop.adafruit.com/datasheets/WS2812.pdf_

_https://github.com/tinygo-org/pio/blob/main/rp2-pio/piolib/ws2812.go_
func (ws *WS2812) SetRGB(r, g, b uint8) {
	color := uint32(r)<<16 | uint32(g)<<8 | uint32(b)
	println("r", r, "g", g, "b", b)
	ws.sm.TxPut(color)
}

If you find it convenient to make changes to faithfully adapt the bit time specifications (+- 150ns):
WS2812B:
T0H = 400ns   /    T0L = 850ns
T1H = 800ns   /    T1L = 450ns

My measurements with logic analyzer:
T0H = 500ns   /    T0L = 458 ns  (can improve)

T1H = 875ns   /    T1L = 417ns (very good)

I've reverted the changes and moved all changes to a PR for better following of changes.

Thank you Patrício!

Latest main now contains changes that match the tinygo drivers implementation behaviour, please give it a try!

First of all I want to say a huge thank you for developing tinygo infrastructure for Pico.
I tested the library too, connected a strip of 92 WS2812 LEDs to the Pico W, added the blocking write function and I managed to set the colour for all diodes.
I noticed a problem: the brightness does not match what I set. If I try to set RGB{1, 0, 0}, the red one shines as if I set RGB{200, 0, 0} or so. There is a small difference between RGB{1, 0, 0} and RGB{255, 0, 0}, but it's very slight.
I'm not sure (and I can't check it since I don't have an oscilloscope), but I think there is a timing problem.

@mishamyrt Hey! thanks for reporting this :)
So the issue you are noticing is only with RGB{1,0,0}? Or does the same happen with RGB{2,0,0} or RGB{0,1,0}?

Yes, the problem is reproduced with any values. They all look too high. As if the brightness is encoded in 2-3 bits.

What version of this repo are you using? You can share your go.mod project file here

@mishamyrt I've implemented a new version of the algorithm to fix timings a bit. Should be consistent with https://cdn-shop.adafruit.com/datasheets/WS2812B.pdf datasheet. Please try it out and let me know if it fixes the issue.
Waveshare note
Note: Waveshare Zero board has a different clock!
Below is a Low bit with the current WS2812B implementation, in its own branch. I'm setting a CLKDIV of whole=17, frac=0 which should yield 1e9/(125000000/17) = 136ns per cycle, but somehow the high time is 640ns, which is not a multiple of 136ns!

Solved by merging #10 as far as I can tell! Let me know if not the case (may be for WS2812, but that's a separate issue in any case)


What is the advantage of using multiple routers?

I have a question regarding routers. If I want to have for example 5 chimneys, is there some advantage of connecting each chimney to a separate router over connecting all the chimneys to a single router? Because when using multiple routers like in a mesh, it takes more cycles for a transaction to complete between two chimneys connected on first and last router. Why can't we just always connect all the required chimneys to a single router?

You can definitely do that to decrease latency, but whether that makes sense depends on the number of links you attach to a single router. A router scales quadratically with the number of links, so at some point, a single all-to-all router is not feasible anymore (this is why NoCs were introduced in the first place).
Also, If you have a smaller system an all-to-all interconnect is still feasible, a NoC doesn't really make sense in the first place. You could also just use normal AXI Xbars.
There is also a an example configuration, which uses a single router with a couple of endpoints

Thank you so much for answering my question.

I have a question about the router links. A chimney is connected to a router through 6 channels (req, rsp, & wide input and output). Are these 6 channels considered 6 links, or are they considered as a single link?


Bigtable: Auto-gen update.

We really can not merge it into master as it would be merged in a not-working state...can you build the library with this branch as a base?

@lukesneeringer, was that request directed to me?

@zakons, you and @aneepct should work on this.

@lukesneeringer and I discussed the possibility of a "clean" import of BigQuery gapic - one that would not break existing code.

@jonparrott Do you mean Bigtable gapic?

Yep

Ah, that would be awesome :)

@jonparrott We have successfully integrated the existing gapic calls into the BigTable client code and tested with an existing performance testing client with good results.  Now we are trying to update the unit test cases to use the BigtableClient that is part of the gapic build but facing an issue that the client requires credentials.  Please let us know how to use the credentials in the unit test cases  or if there is a 'back-door' approach that does not require credentials.

@zakons you should just be able to use mock.create_autospec(google.auth.credentials.Credentials, instance=True).

@jonparrott
Following code to mock the Credentials on unit test case
from google.auth.credentials import Credentials
from google.cloud.bigtable_v2.gapic.bigtable_client import BigtableClient

credentials = mock.create_autospec(Credentials, instance=True)
client = BigtableClient(credentials=credentials)

It throws following error
google.api_core.exceptions.Unauthenticated: 401 Request is missing required authentication credential. Expected OAuth 2 access token, login cookie or other valid authentication credential. See https://developers.google.com/identity/sign-in/web/devconsole-project.

@aneepct well, yeah, it's still talking to the full backend. Consider mocking the gRPC channel using https://github.com/GoogleCloudPlatform/google-cloud-python/blob/master/api_core/google/api_core/grpc_helpers.py#L256

@lukesneeringer was the admin package generated in this PR?

@jonparrott The current PR requires us to have a hybrid solution where there are actually two clients:  the new BigtableClient and the old bigtable.Client().  This permeates the unit tests and classes like table.py, column_family.py, instance.py, cluster.py and client.py, which need the old client for admin API's.  Below is a list of these admin API's.  When will these admin API's have the functionality they need added to the gapic API?

table_pb2.py
1.a) CreateTableRequest
1.b) DeleteTableRequest
1.c) GcRule

bigtable_instance_admin_pb2.py
2.a) CreateInstanceRequest
2.b) GetInstanceRequest
2.c) CreateInstanceMetadata
2.e) DeleteInstanceRequest
2.f) ListClustersRequest
2.g) ListTablesRequest

bigtable_table_admin_pb2.py
3.a) ListTablesRequest
3.b) ModifyColumnFamiliesRequest

instance_pb2.py
4.a) Cluster

bigtable_instance_admin_pb2.py
5.a) CreateClusterRequest
5.b) GetClusterRequest
5.c) UpdateClusterMetadata
5.d) DeleteClusterRequest
5.e) ListInstancesRequest

I look to @lukesneeringer to answer that.

@lukesneeringer, it looks like we need the admin GAPIC client.
I'm ok with merging the code with a hybrid data on GAPIC, admin on the older version.

@aneepct @zakons, the build seems to fail.  That needs to be fixed.

Sorry, I completely forgot this thread existed. @danoscarmike just poked about it.
Catching myself up and hopefully I will have something useful to say. :-)

@sduskis I will add the admin client to this PR momentarily (ETA ~10 minutes).

Thanks!  This will allow us to have a single GAPIC BigtableClent for all interfaces.
@aneepct Please commence converting the admin API's as soon as @lukesneeringer has added this.

Okay, it should be in place. My local tests did not pass (failing at the import of the manual client), which i am fairly certain is the same as the status when I staged this PR to begin with. (When the manual client is rebased to use this, then tests should pass.)
The import path is google.cloud.bigtable_admin_v2, to more or less match what Spanner did.

In this branch, the following imports appear to be missing when loading bigtable_instance_admin_pb2.py
from google.iam.v1 import iam_policy_pb2 as google_dot_iam_dot_v1_dot_iam__policy__pb2
from google.iam.v1 import policy_pb2 as google_dot_iam_dot_v1_dot_policy__pb2
Since this branch is being merged into our branch that has the first round of GAPIC changes we made for bigtable_v2, we are assuming that we would find the iam_policy_pb2 and policy_pb2 modules in this branch now that the admin GAPIC code has been added.  Can this please be added?

@sduskis Adding the iam dependency to setup.py resolved that issue. Thanks!
@lukesneeringer There is another issue whereby code is included in each admin method in bigtable_instance_admin_client.py and bigtable_table_admin_client.py to add a routing_header to the metadata. There are around 26 such methods.  The particular code in each of these methods that is failing looks like this:
        routing_header = google.api_core.gapic_v1.routing_header(
            [('parent', parent)], )
        metadata.append(routing_header)

The failures in the CircleCI build all contain the following error message for this code:
    [('parent', parent)], )
TypeError: 'module' object is not callable

Any information on how to resolve this would be very helpful as this seems like the last issue to resolve for us to submit our PR, which integrates all the GAPIC changes to cloud bigtable.

@zakons you're looking for routing_reader.to_grpc_metadata , defined here.

@jonparrott Thanks as usual for your help.  What I find confusing here is that the GAPIC code is assumed to be auto-generated based on the protobuf specification.  So if the to_grpc_metadata method name was not included in the routing_header code, then this should be fixed and the code regenerated, no?
We added the to_grpc_metadata method name manually in 32 locations in the above two modules and now CircleCI test cases are (mostly) passing for these GAPIC admin API calls.
@lukesneeringer Please comment if this really is auto-generated code or if we should commit the manually fixed code for our pull request.

Thanks as usual for your help. What I find confusing here is that the GAPIC code is assumed to be auto-generated based on the protobuf specification. So if the to_grpc_metadata method name was not included in the routing_header code, then this should be fixed and the code regenerated, no?

Are you saying that some of the autogenerated code made calls to router_header that were incorrect? If that's the case, then yeah, it's a bug in the generator and @lukesneeringer should fix it. In the meantime it's okay to manually fix those things to move this along.

I think that this PR can be closed, since it was superceded by #5178.


mismatch client default ping delay < server ping limit

Default ping delay is 15 sec. Client will send one ping every 15 sec. Server will ignore 50% of the pings.
Example:
Client send ping at sec 00
if now()- last_seen >= 15 sec then last_seen=now() -> second now() can be sec 01 and not sec 00
Next ping at sec 15
15-01 will be ignored.
I see 2 possible solution.

new_variable=now() if new_variable-last_seen >= 15 then last_seen=new_variable -> no time difference
Higher default delay on the client side.

Implemented solution 1.

Online list looks better now. I see only some clients with 16 sec as expected. No more 30 sec clients. Retest ok. Bug closed.


The compressed png file's background filling with black, not transparent.

fix the problem, try again


Issue 3269 Fix incorrect OR evaluation when multiples ORs are optimized to IN

https://github.com/yandex/ClickHouse/issues/3269 In this case when multiple equality evaluation are bundled into one IN set evaluation, since tuple_function is generated from optimizer, it doesn't have a proper range which is later used as indexation key for prepared set.
Add fake range for tuple ASTFunction generated by Optimizer so that prepared set indexation works correctly.
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Thank you!
PS. Maybe we should get rid of AST range and use something other for prepared sets indexation.

Please add a test in separate PR.

Why don't add a test?


Remarks section references a non-existent topic in Application Verifier help

The remarks section says, "see "Setting Up a Debugger for Application Verifier" in the Application Verifier documentation."
However, the online Application Verifier documentation does not appear to have this material.  An online search for the phrase "Setting Up a Debugger for Application Verifier" will only return the reference in this document.  The App Verifier docs do not appear to have this content (checked here - https://docs.microsoft.com/en-us/windows-hardware/drivers/devtest/application-verifier ) even though it seems to be relevant as I was looking to resolve exactly the problem identified in the remarks.

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

ID: c88257d2-444f-9d2c-73e0-a120bcdbfbe1
Version Independent ID: 3ebf16c7-7a23-0d52-c746-ed267162e3b6
Content: avrf (WinDbg) - Windows drivers
Content Source: windows-driver-docs-pr/debugger/-avrf.md
Product: windows-hardware
Technology: debugger
GitHub Login: @DOMARS
Microsoft Alias: domars

@cpowers534at - Thanks for pointing this out, I updated the topic as some of the names of some of the docs sections have changed. As we have moved the CHM to online topics, I also added links to the primary new online topic related to debugger use.  Good luck with your use of AppVerif, and thanks again for filing this issue.


Implement record spill command

For https://github.com/bitfocus/companion-module-bmd-hyperdeck/issues/85

This is already implemented https://github.com/nrkno/sofie-hyperdeck-connection/blob/fb7a34bda966022b96b6f2c382aae850a4a0ec9a/src/commands/record.ts#L23-L38


Restore toolbar logo and replace PNG logo with SVG version

This pull request fixes a typo in https://github.com/concrete5/concrete5/blob/4f8c62cc63b8d1631293fabbee1d65a99237c881/concrete/src/Application/Service/UserInterface.php#L170-L189.
While I was there I replaced logo.png with an SVG version in the toolbar, dashboard update page, and login authentication. I did not remove logo.png because it is relied on for some tests that appear to require a bitmap image.
Current:

Changes:

Based on the input of @Mnkras, I have updated the pull request. The main change is that this allows for square and non-square white label logo images.
Here is an example of a non-square white label logo:

I tested this pull request in IE10, IE11, Edge, Firefox, and Chrome (all on Windows).

Thanks - I merged this manually.


Fixed issue #7181

Issue Reference
#7181

@Sanjaybhattwebkul Check now

@akathuria289
This is working fine. please find the screenshot Here


MINOR: add memory management section to streams docs

@miguno @guozhangwang @bbejeck @enothereska

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6466/
Test FAILed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6451/
Test FAILed (JDK 8 and Scala 2.12).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.11/6467/
Test PASSed (JDK 7 and Scala 2.11).

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/6452/
Test PASSed (JDK 8 and Scala 2.12).

LGTM, please feel free to merge as is.


Tron-Gothic-City Passing through gate

The Avatar can pass through gate but could not return back into it.
https://user-images.githubusercontent.com/104460803/168872903-1bd72d63-3c7f-4771-b1f9-5308a1a2036e.mp4

This is basically the case with all our models. The physics mesh doesn't works for backside/Downside of most of the models


⚠ GitLab Server has degraded performance

In cd3140a, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11524 ms

Resolved: GitLab Server performance has improved in 23d483f.


How to convert predict to real item ?

Following the document, Currently I run TiSAS model the output include [[score grouth truth, k-score-item],[....],[....]] and I don't know how to convert score to item? Please help me! Thank for your support.

The scores correspond to the [item_id, neg_items] in test.csv / dev.csv. Maybe you can refer to the last reply of this issue #5.

thank you

Just also need to export the recommended item list for each user in the test set. Leave the codes here for reference.
predictions = runner.predict(data_dict['test'])  # n_users, 1 + n_candidates
users, rec_items = list(), list()
for i in range(len(data_dict['test'])):
    info = data_dict['test'][i]
    users.append(info['user_id'])
    item_dict = dict(zip(info['item_id'], predictions[i]))
    sorted_lst = sorted(item_dict.items(), key=lambda x: x[1], reverse=True)[:100]  # top-100
    rec_items.append([x[0] for x in sorted_lst])
rec_df = pd.DataFrame(columns=['user_id', 'rec_items'])
rec_df['user_id'] = users
rec_df['rec_items'] = rec_items


feature request: Add a Oil Toggle command

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
It would be nice to have oil toggle so that we could return to last or previous buffer.
Provide background
Previously when I open a file tree or file explorer like nvim-tree it used have a toggle command so that we could return back to the previous buffer. I made a command to close the netrw plugin when I switched to it. So it would be nice If we have OilToggle command to close the oil buffer.
What is the significance of this feature?
strongly desired
Additional details
No response

Yeah I think it might work. but rather than returning to previous buffer, we can delete the buffer so that it wont appear on buffers list. Thank you.

vim.keymap.set('n', '<Tab>', function()
   vim.cmd((vim.bo.filetype == 'oil') and 'bd' or 'Oil')
end)

this works like charm, thank you :)


Go Live

Push to live

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

File
Status
Preview URL
Details

msteams-platform/tabs/how-to/create-personal-tab.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/deploy.md
:white_check_mark:Succeeded
View

msteams-platform/toolkit/use-CICD-template.md
:white_check_mark:Succeeded
View

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


fix(JSON-RPC): fix declare v0&v1 serde bug by failing deserialization for declare v0 is version!=0x0

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

Current dependencies on/for this PR:

main

PR #952   👈

This comment was auto-generated by Graphite.


SGDK 1.5

Any plans to update to the new version? or how could I update myself? thanks!

Updated to 1.51


Can I call functions from one Powershell task in a different task?

I would like to get some modularity to these deployment pipelines. Is there a way I can import functions in one step and then use them in other steps? I'd rather not have different copies of the same functions.

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

ID: 77731c06-e4fd-8125-0daf-870c9ccedd3f
Version Independent ID: 0d5c338d-80f9-c64e-b675-bf7b4eb8e32b
Content: Use a PowerShell script to customize your build pipeline - Azure Pipelines
Content Source: docs/pipelines/scripts/powershell.md
Product: devops
Technology: devops-cicd
GitHub Login: @chasewilson
Microsoft Alias: phwilson

Thanks for your question. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your question:

Azure DevOps on Stack Overflow
Azure DevOps Support Bot


API Export

A universal program interface would allow other library implementations to interact with the runtime.
Would also progress support towards plugin capability.

Example concept for author generation hook
// very cut and dry at the moment
const { register, events } = require('channel-backup/hooks');

// message:post_render
hook.register(events.postRender, (ctx, payload) => {
  // scoped config for the specific plugin on a getter attribute
  if(!ctx.config.authors)
    return;

  if(!payload.avatarURL && ctx.config.authors[payload.username]) {
    payload.avatarURL = ctx.config.authors[payload.username];
  }

  payload.embeds = payload.embeds.map((embed, index) => {
    // embed.author?.iconURL
    if((embed.author && !embed.author.iconURL) && ctx.config.authors[embed.author.username]) {
      embed.author.iconURL = ctx.config.authors[embed.author.username];
    }

    // return embed;
  });

  // return payload;
});

I had originally looked at the possibility of using a class to generalise the optional abstract extension, which is something I will also look into, but then I would either need to export an interface function to:

call for that context and run as is
call the context but have register / return a predefined predicate which is used later as part of plugin cache
create a common eventemitter used across all loaded plugins and only declare the events to be used on the plugin

It may also be possible to move the context ctx to a physical export, but then that would mean the loss of detection to which config is currently running. If plugins are added, configuration will also need to be modified so that the core is not polluted with unnecessary data that the main program will not use. It may also be necessary to declare run order and dependances, but that is yet to be written in stone.

Example concept for author generation hook
// very cut and dry at the moment
const { register, events } = require('channel-backup/hooks');

// message:post_render
hook.register(events.postRender, (ctx, payload) => {
  // scoped config for the specific plugin on a getter attribute
  if(!ctx.config.authors)
    return;

  if(!payload.avatarURL && ctx.config.authors[payload.username]) {
    payload.avatarURL = ctx.config.authors[payload.username];
  }

  payload.embeds = payload.embeds.map((embed, index) => {
    // embed.author?.iconURL
    if((embed.author && !embed.author.iconURL) && ctx.config.authors[embed.author.username]) {
      embed.author.iconURL = ctx.config.authors[embed.author.username];
    }

    // return embed;
  });

  // return payload;
});

I had originally looked at the possibility of using a class to generalise the optional abstract extension, which is something I will also look into, but then I would either need to export an interface function to:

call for that context and run as is
call the context but have register / return a predefined predicate which is used later as part of plugin cache
create a common eventemitter used across all loaded plugins and only declare the events to be used on the plugin

It may also be possible to move the context ctx to a physical export, but then that would mean the loss of detection to which config is currently running. If plugins are added, configuration will also need to be modified so that the core is not polluted with unnecessary data that the main program will not use. It may also be necessary to declare run order and dependances, but that is yet to be written in stone.


 There is likely a somewhat better way to do this, but at least the tests pass.

seems like the tests pass, but the CI script is broken:
https://travis-ci.org/travisbrown/iteratee/jobs/548514706#L3731
I don't see any error lines.

@johnynek Thanks much for doing this! I think it's a step in the right direction, but providing stack safety only for tailRecM arguments returning pure / enumOne (or even for any enumeration of Enumerator constructors or transformations, since I don't think we can seal Enumerator) feels a little like cheating.
For example this still overflows:
Enumerator.enumeratorMonad[IO].tailRecM(0)(i =>
  Enumerator.enumVector(Vector(if (i < 50000) Left(i + 1) else Right(i)))
)

(The CI failure is related to the switch to Coursier in sbt 1.3—it should be fine after #298.)

Yeah I agree it is not a complete solution but hopefully it is a useful direction to explore.
Why is it not okay to seal Enumerator? Seems like we should be able to build them up from a smallish set of fundamental operations.


Test failure scenarios with streaming

Moved to JIRA: https://datastax-oss.atlassian.net/browse/SPARKC-49


Lock not being detected. "Total August Locks Found: undefined" "TypeError: deviceLists.map is not a function"

Current Situation

Installed latest beta to test api based events; homebridge-august v0.1.0-beta.18
Plugin appeared to authenticate as 2FA code was received and added to plugin config.
August WiFi Smart Lock (Gen 4) does not appear in accessories within Homebridge.
Attempted both directly connected to WiFi, and connected via August Bridge (preferred to save battery life.)
Lock does not appear in either WiFi setup method. (Works via August app and other Homebridge August plugins.)

Logs
[10/6/2022, 9:31:08 PM] [August] Total August Locks Found: undefined
[10/6/2022, 9:31:08 PM] [August] TypeError: deviceLists.map is not a function

Configuration
{
            "name": "August",
            "credentials": {
                "augustId"<EMAIL_ADDRESS>                "password": "xxxxxxxxx",
                "validateCode": "xxxxxxx",
                "notice": "Keep your tokens a secret!"
            },
            "options": {
                "logging": "standard"
            },
            "platform": "August"
        },

Environment

OS: Ubuntu Focal Fossa (20.04.4 LTS)
Software: https://github.com/oznu/docker-homebridge
Node: v16.17.1
npm: 8.5.0

Additional Context
No response

can you try latest beta? also please clear out validateCode after installing and get new code.

Upgraded to beta 22.
Cleared out code and entered new one.
Still get "[10/6/2022, 10:17:45 PM] [August] TypeError: deviceLists.map is not a function" and no locks listed.

can you get me debug logs?

[10/6/2022, 10:18:41 PM] ---
[10/6/2022, 10:18:42 PM] Loaded plugin<EMAIL_ADDRESS>[10/6/2022, 10:18:42 PM] Registering platform 'homebridge-august.August'
[10/6/2022, 10:18:42 PM] ---
[10/6/2022, 10:18:44 PM] [August] Initializing August platform...
[10/6/2022, 10:18:44 PM] [August] Using Config Logging: debug
[10/6/2022, 10:18:44 PM] [August] [DEBUG] Finished initializing platform: August
[10/6/2022, 10:18:44 PM] [August] Platform Config: {"logging":"debug","refreshRate":[Undefined],"pushRate":[Undefined]}
[10/6/2022, 10:18:44 PM] [August] [DEBUG] Using Default Refresh Rate (5 minutes).
[10/6/2022, 10:18:44 PM] [August] [DEBUG] Config OK
[10/6/2022, 10:18:44 PM] [August] [DEBUG] Executed didFinishLaunching callback
[10/6/2022, 10:18:44 PM] [August] [DEBUG] {"config":{"apiKey":"xxxxxxxxx","pnSubKey":"xxxxxxxxx","installId":"xxxxxxxxxxxx","idType":"email","augustId":"xxxxx@xxxxxx","password":"xxxxxxxxx"}}
[10/6/2022, 10:18:45 PM] [August] Total August Locks Found: undefined
[10/6/2022, 10:18:45 PM] [August] [DEBUG] August Platform Config Not Set: [Undefined]
[10/6/2022, 10:18:45 PM] [August] TypeError: deviceLists.map is not a function

Can you manually change your config?and just keep the following:
"name": "August",
            "credentials": {
                "augustId"<EMAIL_ADDRESS>                "password": "xxxxxxxxx",
                "validateCode": "xxxxxxx",
                "notice": "Keep your tokens a secret!"
            },
            "platform": "August"

Still hitting "TypeError: deviceLists.map is not a function"
I meant to copy what was there before, it had the debug option but also two other lines, one I believe was a MAC address.

@hrhnick, I am adding this code this.errorLog(`COPY THIS LOG: ${deviceLists}`); into the next beta: 0.1.0-beta.26, copy this code and paste it once you see it. Should be red.

@donavanbecker [10/6/2022, 11:11:03 PM] [August] COPY THIS LOG: [object Object]

sorry about that try beta.28

Shot them to your email listed at beckersmarthome at icloud dot com
I was not sure what was sensitive/user specific regarding the keys listed and such.

Thanks, can you give next beta a try? and let me know what happens?

New error: 10/6/2022, 11:41:33 PM] [August] TypeError: devices is not iterable

I am guessing you only have one lock right?

Correct. (I had the WiFi bridge accessory previously configured. But it has been unpaired and removed from my August account.)

can you try adding in your lock id into the config? I will email you what your config should be.

Added to the config, now [10/7/2022, 12:00:40 AM] [August] TypeError: a1.map is not a function

okay I think this next fix should fix it. try next beta = v0.1.0-beta.31

Same ai.map error on 31.
Happy to try a factory reset on the lock if you think that it would help, but it does work as expected under the homebridge-august-smart-locks plugin.

clear your config and start over.

v1.0.0 should fix your issue.

Thanks @donavanbecker! Woke up and indeed 1.0.4 resolved the issue. You can close this out.
I will test the plug-in as well as attempt with the WiFi bridge and report back if anything comes up.


Connect with Zenodo for citeable DOIs for versions of this repo

Consider using Zenodo to make releases of this get permanent DOIs, as per https://guides.github.com/activities/citable-code/

Resolved via a recent PR.