htonll() and Windows 7

Is your feature request related to a problem? Please describe.
htonll() (used in orm_lib/inc/drogon/orm/SqlBinder.h) is only available since Windows 8 and when compiling with NTDDI_VERSION>=NTDDI_WIN8. However I'd like to be able to compile and run our applications still on Windows 7.
Describe the solution you'd like
ATVL add a preprocessor condition to the existing ones that determine whether a fallback for htonll() and ntohll() need to be defined ad hoc:
#if defined __linux__ || defined __FreeBSD__ || defined __OpenBSD__ || \
    defined __MINGW32__ || (defined NTDDI_VERSION && NTDDI_VERSION < NTDDI_WIN8)

Ideally I'd even like to see a custom macro denoting the minimum Windows SDK version to use, which can be set at configuration time, e.g. DR_NTDDI_VERSION which defaults to NTDDI_VERSION. This would separate the concerns of the library and the application (which possibly pulls in a higher Windows SDK version, but is using runtime function bindings).
Additional context

OS: Windows 7

@trueqbit Thank you very much for your suggestion. But I've no windows 7 environment in my machines, would you please make a PR for this?

got compilation error on Windows 11
Drogon: 1.7.4
MSVC: 19.31.31104
FAILED: CMakeFiles/xxx.dir/models/xxx.cpp.obj
C:\PROGRA~1\MIB055~1\2022\COMMUN~1\VC\Tools\MSVC\1431~1.311\bin\Hostx64\x64\cl.exe  /nologo /TP -DBENCHMARK -DFMT_LOCALE -DFMT_SHARED -DJSON_DIAGNOSTICS=0 -DJSON_USE_IMPLICIT_CONVERSIONS=1 -DNOMINMAX -IE:\link\code\xxx++ -IE:\link\code\xxx++\models -IE:\link\code\xxx++\cmake-build-debug\vcpkg_installed\x64-windows\include /DWIN32 /D_WINDOWS /W3 /GR /EHsc /utf-8 /arch:AVX /arch:AVX2 /arch:AVX512 /MDd /Zi /Ob0 /Od /RTC1 /wd4251 /wd4275 -std:c++20 /showIncludes /FoCMakeFiles\xxx.dir\models\xxx.cpp.obj /FdCMakeFiles\xxx.dir\ /FS -c E:\link\code\xxx\models\xxx.cpp
Please define _WIN32_WINNT or _WIN32_WINDOWS appropriately. For example:
- add -D_WIN32_WINNT=0x0601 to the compiler command line; or
- add _WIN32_WINNT=0x0601 to your project's Preprocessor Definitions.
Assuming _WIN32_WINNT=0x0601 (i.e. Windows 7 target).
E:\link\code\xxx\cmake-build-debug\vcpkg_installed\x64-windows\include\drogon/orm/SqlBinder.h(388): error C3861: “htonll”: 找不到标识符
E:\link\code\xxx\cmake-build-debug\vcpkg_installed\x64-windows\include\drogon/orm/SqlBinder.h(388): note: “htonll”: 函数未在模板定义上下文中声明，只能通过实例化上下文中参数相关的查找找到
E:\link\code\xxx\cmake-build-debug\vcpkg_installed\x64-windows\include\drogon/orm/SqlBinder.h(536): note: 查看对正在编译的函数 模板 实例化“drogon::orm::internal::SqlBinder &drogon::orm::internal::SqlBinder::operator <<<Json::Value::Int64>(T &&)”的引用
        with
        [
            T=Json::Value::Int64
        ]
ninja: build stopped: subcommand failed.

Try the version of 1.7.5

Try the version of 1.7.5

same error

遇到到了相同问题，改了下SqlBinder.h文件，可以过了
#if defined linux || defined FreeBSD || defined OpenBSD ||
defined MINGW32 || defined HAIKU ||
defined _WIN32_WINNT && _WIN32_WINNT < _WIN32_WINNT_WIN8
#ifdef linux
#include <endian.h>  // __BYTE_ORDER __LITTLE_ENDIAN
#elif defined FreeBSD || defined OpenBSD
#include <sys/endian.h>  // _BYTE_ORDER _LITTLE_ENDIAN
#define __BYTE_ORDER _BYTE_ORDER
#define __LITTLE_ENDIAN _LITTLE_ENDIAN
#elif defined MINGW32
#include <sys/param.h>  // BYTE_ORDER LITTLE_ENDIAN
#define __BYTE_ORDER BYTE_ORDER
#define __LITTLE_ENDIAN LITTLE_ENDIAN
////////////////////////
#elif defined _WIN32_WINNT
#include <basetsd.h>
#include <netiodef.h>  // BYTE_ORDER _LITTLE_ENDIAN
#define __BYTE_ORDER BYTE_ORDER
#define __LITTLE_ENDIAN _LITTLE_ENDIAN
////////////////////////
#endif

Got the same error as @kotori2 also on Win11 and MSVC 19.x (latest)
[build] include\drogon/orm/SqlBinder.h(382,25): error C3861: 'htonll': identifier not found
[build] include\drogon/orm/SqlBinder.h(382,25): message : 'htonll': function was not declared in the template definition context and can be found only via argument-dependent lookup in the instantiation context
[build] include\drogon/orm/SqlBinder.h(523,1): message : see reference to function template instantiation 'drogon::orm::internal::SqlBinder &drogon::orm::internal::SqlBinder::operator <<<Json::Value::Int64>(T &&)' being compiled
[build]           with
[build]           [
[build]               T=Json::Value::Int64
[build]           ]


When setting multiple serveralias in config only the last one is generated

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 2.2.7
VirtualBox 5.2.22r126460
ansible 2.6.3
  config file = None
  configured module search path = [u'/Users/production/.ansible/plugins/modules', u'/usr/share/ansible/plugins/modules']
  ansible python module location = /usr/local/lib/python2.7/site-packages/ansible
  executable location = /usr/local/bin/ansible
  python version = 2.7.15 (default, Aug 17 2018, 22:42:59) [GCC 4.2.1 Compatible Apple LLVM 8.0.0 (clang-8<IP_ADDRESS>)]

Your OS

macOS (version)

Full console output
Summary
Hi while working on multisite drupal 8 instance i wanted to set multiple server aliases for my vhost.
When i set multiple serverailas in default.config.yml only the last one is processed when provisioning.
eg with this config :
- servername: "profile.vbox.local"
    serveralias: "profile1.vbox.local"
    serveralias: "profile2.vbox.local"
    documentroot: "/var/www/drupalvm/d8vm/profile.vbox.local/public_html/web"
    extra_parameters: "{{ apache_vhost_php_fpm_parameters }}"

only profile.vbox.local and profile2.vbox.local are set in vhost and host updater only create rules for those while serveralias: "profile1.vbox.local" is discarded
==> drupalvm: [vagrant-hostsupdater]   <IP_ADDRESS>  profile.vbox.local  # VAGRANT: 823504acb7deec376443d2fed70a1faa (drupalvm) / 1136af15-7c8f-4a7f-b5c0-57fda15a9943
==> drupalvm: [vagrant-hostsupdater]   <IP_ADDRESS>  profile2.vbox.local  # VAGRANT: 823504acb7deec376443d2fed70a1faa (drupalvm) / 1136af15-7c8f-4a7f-b5c0-57fda15a9943

I managed to have it by creating new vhost instead of aliases, but imho it would be convenient and more efficient to be able to set multiple aliases instead of multiple vhosts.
Br
Eric

@agence-tuesday - ServerAlias can take a string with one or more hostnames, separated by a space; see the documentation here: https://httpd.apache.org/docs/2.4/mod/core.html#serveralias
- servername: "profile.vbox.local"
  serveralias: "profile1.vbox.local profile2.vbox.local"
  documentroot: "/var/www/drupalvm/d8vm/profile.vbox.local/public_html/web"
  extra_parameters: "{{ apache_vhost_php_fpm_parameters }}"

So that should work out for you.

Lighting fast answer, yes it works perfectly, thanks a lots.


karma-webpack 5 fails to read files with a query parameter (e.g. font-awesome)

Operating System: MacOS
Node Version: v14.16.1
NPM Version: 6.14.12
webpack Version: 5.61.0
karma-webpack Version: 5.0.0

Expected Behavior
As with earlier karma-webpack versions, a CSS referencing a file with a cache breaker (?v=4.7.0) should work.
Actual Behavior
Error: ENOENT: no such file or directory, open
  '/var/folders/s8/x7zt810n7458m_tpwt8ky5qr0000gp/T/_karma_webpack_358674/c1e38fd9e0e74ba58f7a.svg?v=4.7.0'

How Do We Reproduce?
mkdir bug && cd bug
npm init -y && npm i -D webpack karma karma-webpack karma-mocha mocha style-loader css-loader && npm i font-awesome
echo 'import "font-awesome/css/font-awesome.css"' > test.js

Use this text as karma.conf.js
const path = require("path");
const os = require("os");

const defaultWebpackOptions = {
  mode: 'development',
  output: {
    filename: '[name].js',
    path: path.join(os.tmpdir(), '_karma_webpack_') + Math.floor(Math.random() * 1000000),
  },
  watch: false,
  optimization: {
    runtimeChunk: 'single',
    splitChunks: {
      chunks: 'all',
      minSize: 0,
      cacheGroups: {
        commons: {
          name: 'commons',
          chunks: 'initial',
          minChunks: 1,
        },
      },
    },
  },
  module: {
    rules: [{
      test: /\.css$/,
      use: [
        "style-loader",
        {loader: "css-loader", options: {sourceMap: true}},
      ]
    }, {
      test: /\.(woff2?|eot|ttf|svg)$/,
      type: "asset/resource"
    }]
  }
};

module.exports = (config) => {
  config.set({
    frameworks: ['mocha', 'webpack'],
    plugins: ['karma-webpack', 'karma-mocha'],
    files: [{ pattern: 'test.js', watched: false }],
    preprocessors: {'test.js': ['webpack']},
    webpack: defaultWebpackOptions
  });
}

npx karma start

Output:
karma-webpack does not currently support customized filenames via
webpack output.filename, if this is something you need consider opening an issue.
defaulting [name].js to [name].js.
Webpack bundling...
assets by status 162 KiB [cached] 1 asset
assets by info 929 KiB [immutable]
  asset c1e38fd9e0e74ba58f7a.svg?v=4.7.0 434 KiB [emitted] [immutable] [from: node_modules/font-awesome/fonts/fontawesome-webfont.svg?v=4.7.0] (auxiliary name: commons) (auxiliary id hint: commons)
  asset 8b43027f47b20503057d.eot 162 KiB [emitted] [immutable] [from: node_modules/font-awesome/fonts/fontawesome-webfont.eot] (auxiliary name: commons) (auxiliary id hint: commons)
  asset 1e59d2330b4c6deb84b3.ttf?v=4.7.0 162 KiB [emitted] [immutable] [from: node_modules/font-awesome/fonts/fontawesome-webfont.ttf?v=4.7.0] (auxiliary name: commons) (auxiliary id hint: commons)
  asset f691f37e57f04c152e23.woff?v=4.7.0 95.7 KiB [emitted] [immutable] [from: node_modules/font-awesome/fonts/fontawesome-webfont.woff?v=4.7.0] (auxiliary name: commons) (auxiliary id hint: commons)
  asset 20fd1704ea223900efa9.woff2?v=4.7.0 75.4 KiB [emitted] [immutable] [from: node_modules/font-awesome/fonts/fontawesome-webfont.woff2?v=4.7.0] (auxiliary name: commons) (auxiliary id hint: commons)
assets by path *.js 148 KiB
  asset commons.js 139 KiB [emitted] (name: commons) (id hint: commons)
  asset runtime.js 8.39 KiB [emitted] (name: runtime)
  asset test.372843621.js 992 bytes [emitted] (name: test.372843621)
Entrypoint test.372843621 148 KiB (1.06 MiB) = runtime.js 8.39 KiB commons.js 139 KiB test.372843621.js 992 bytes 6 auxiliary assets
webpack 5.61.0 compiled successfully in 501 ms
29 10 2021 17:37:57.523:ERROR [karma-server]: Error during file loading or preprocessing
TypeError [ERR_INVALID_ARG_TYPE]: The "data" argument must be of type string or an instance of Buffer, TypedArray, or DataView. Received undefined
    at Hash.update (internal/crypto/hash.js:84:11)
    at Object.sha1 (/private/tmp/karma2/node_modules/karma/lib/utils/crypto-utils.js:9:8)
    at runProcessors (/private/tmp/karma2/node_modules/karma/lib/preprocessor.js:70:26)
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async FileList.preprocess [as _preprocess] (/private/tmp/karma2/node_modules/karma/lib/preprocessor.js:106:5)
    at async Promise.all (index 0)
    at async /private/tmp/karma2/node_modules/karma/lib/file-list.js:90:11
    at async Promise.all (index 4)
29 10 2021 17:37:57.552:INFO [karma-server]: Karma v6.3.6 server started at http://localhost:9877/
29 10 2021 17:37:57.552:ERROR [karma-server]: UnhandledRejection: Error: ENOENT: no such file or directory, open '/var/folders/s8/x7zt810n7458m_tpwt8ky5qr0000gp/T/_karma_webpack_848897/c1e38fd9e0e74ba58f7a.svg?v=4.7.0'
    at Object.openSync (fs.js:476:3)
    at Object.readFileSync (fs.js:377:35)
    at /private/tmp/karma2/node_modules/karma-webpack/lib/webpack/plugin.js:20:66
    at Array.forEach (<anonymous>)
    at /private/tmp/karma2/node_modules/karma-webpack/lib/webpack/plugin.js:16:29
    at Hook.eval [as callAsync] (eval at create (/private/tmp/karma2/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:1)
    at Hook.CALL_ASYNC_DELEGATE [as _callAsync] (/private/tmp/karma2/node_modules/tapable/lib/Hook.js:18:14)
    at /private/tmp/karma2/node_modules/webpack/lib/Compiler.js:491:23
    at Compiler.emitRecords (/private/tmp/karma2/node_modules/webpack/lib/Compiler.js:883:39)
    at /private/tmp/karma2/node_modules/webpack/lib/Compiler.js:483:11
29 10 2021 17:37:57.552:ERROR [karma-server]: Error: ENOENT: no such file or directory, open '/var/folders/s8/x7zt810n7458m_tpwt8ky5qr0000gp/T/_karma_webpack_848897/c1e38fd9e0e74ba58f7a.svg?v=4.7.0'
    at Object.openSync (fs.js:476:3)
    at Object.readFileSync (fs.js:377:35)
    at /private/tmp/karma2/node_modules/karma-webpack/lib/webpack/plugin.js:20:66
    at Array.forEach (<anonymous>)
    at /private/tmp/karma2/node_modules/karma-webpack/lib/webpack/plugin.js:16:29
    at Hook.eval [as callAsync] (eval at create (/private/tmp/karma2/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:9:1)
    at Hook.CALL_ASYNC_DELEGATE [as _callAsync] (/private/tmp/karma2/node_modules/tapable/lib/Hook.js:18:14)
    at /private/tmp/karma2/node_modules/webpack/lib/Compiler.js:491:23
    at Compiler.emitRecords (/private/tmp/karma2/node_modules/webpack/lib/Compiler.js:883:39)
    at /private/tmp/karma2/node_modules/webpack/lib/Compiler.js:483:11 {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/var/folders/s8/x7zt810n7458m_tpwt8ky5qr0000gp/T/_karma_webpack_848897/c1e38fd9e0e74ba58f7a.svg?v=4.7.0'
}

Thanks for opening an issue, I'll try and take a look at this sometime this weekend

A temporary workaround for me is to modify plugin.js to remove anything including and after a question mark:
        this.controller.bundlesContent[webpackFileObj.name] = fs.readFileSync(
          filePath.replace(/\?.*/, ""),
          'utf-8'
        );

I have unfortunately proven that files can be named in this way so introducing the proposed change would be breaking for users in the situation 😞
I think this is potentially an outside issue related to file-loader I'm actually encountering the same issue as you in another project.
It seems that this file is picked up from this line in the font-awesome project
@font-face{font-family:'FontAwesome';src:url('../fonts/fontawesome-webfont.eot?v=4.7.0');

It seems to me that as it is pulled in from a url, it should parse the file name as foo/fonts/fontawesome-webfont.eot without the query param. It looks like it is incorrectly loaded into webpack this way under asset.info
  info: {
    sourceFilename: '../node_modules/font-awesome/fonts/fontawesome-webfont.woff2?v=4.7.0',
    immutable: true,
    contenthash: '4a2487b5cc6987c4f1de',
    size: 82
  },

It looks like in webpack 5 file-loader is deprecated, if you change your file-loader rule to something like this for your tests
            {
                test: /\.(jpg|png|gif|svg|woff|woff2|eot|ttf)$/,
                type: 'asset/inline',
            },

Does that solve this issue for you?

Hey @codymikol thanks for looking into this. I've switched my project from Karma over to Jest, so this issue is no longer relevant for me.
Nonetheless, if you look at my minimal example above, there is no file-loader being used. And I had converted everything in my project to use various asset loading strategies before I observed this issue.

uff. found a workaround. remove the default [query] for assets  eg by
output.assetModuleFilename = '[name][ext]',

or setting filename/generator for individual assets in the module.rules


[e2e] Switch droute pod for RP and update

Description

Switch to a better-configured pod to send test results to ReportPortal.
Update droute to 1.2
Use new droute functionality to finalize the status in ReportPortal automatically.

Which issue(s) does this PR fix

Fixes RHIDP-4011

PR acceptance criteria
Please make sure that the following steps are complete:

[ ] GitHub Actions are completed and successful
[ ] Unit Tests are updated and passing
[ ] E2E Tests are updated and passing
[ ] Documentation is updated if necessary (requirement for new features)
[ ] Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

/test all

/test e2e-tests

/test e2e-tests

@zdrapela I was thinking if we should add set +e at start of droute_send() and set -e at the end? so in case of droute failure it should not stop the rbac tests from running and also it will not mark e2e-tests as failure just because droute failed to upload result.

@subhashkhileri Yes, I agree, droute should have a lower priority. The drawback is, that we might miss when droute gets broken.

@subhashkhileri Yes, I agree, droute should have a lower priority when it comes to marking tests as failed. The drawback is, that we might miss when droute gets broken.

Okay, you are adding rsync, does this lowers chances of failure? if yes then we can go with your changes, else we need to think on adding set +e

@subhashkhileri Yes, now the pod uses an image with rsync and rh certs baked in. That should be much more reliable.
Let me also use this PR to update the droute to 1.2

@zdrapela can you please move droute to utils.sh

Sure! Done and rebased

/lgtm

/cherry-pick 1.3.x

/cherry-pick 1.2.x

/hold

/cherry-pick 1.3.x

@zdrapela It will need a manual cherry-pick. in 1.3.x droute is set +e, you can make appropriate changes and also remove the cluster redirect change that made as pr-os is working fine now.

Ok, opened PR here https://github.com/janus-idp/backstage-showcase/pull/1657

/hold cancel


[fixes 19464273] Remove unrequired data release values.

Removes 'not specified' from the data release study type list and forces
'genotyping or cytogenetics' to be the default choice.
Removes 'not applicable' from the data release strategy and forces
'managed' to be the default choice.

I'm removing this pull request as there are several related tickets that should come as one batch.


Create production-ready ELI file

What's your idea for a cool feature that would help you use OHM better.
In order to make it easier to trace old maps, it'd be great if OHM's iD layer selector pointed at a list of historical x/y/z servers.
We have such an ELI set up (https://openhistoricalmap.github.io/ohm-editor-layer-index/imagery.xml), but we do not a) have OHM iD pointing at it, and b) the list isn't ready to be pointed at.
To make (b) happen, we need to merge:
a) the existing OHM ELI
b) appropriate layers from the standard OSM ELI
c) any other cool x/y/z servers
So... here are the instructions for completing this task:
https://gist.github.com/Chaser324/ce0505fbed06b947d962
Current workarounds
Using the "custom" layer option in iD, which can be a pain and only allows use of 1 historical layer at a time

I setup https://github.com/OpenHistoricalMap/ohm-editor-layer-index so we can store imagery layers which aren't useful or usable in OSM so can't go directly in OSM's editor-layer-index.
As for how we should deal with layers in OSM's editor-layer-index but still useful and usable (wrt licensing) by OHM, we could either:

Attempt to migrate layers across from OSM's ELI to OHM's ELI directly in this git repo.
Setup the build process to pull and merge OSM's layers into our output file which iD and JOSM consume, but don't commit those sources here in the source code.

I think (2) is best, that way we can focus on maintaining only OHM specifc layers here and the rest just come in automatically without cluttering our repo. We might need to maintain a whitelist/blacklist of layers we don't bring across (eg Bing?).

If we are just talking about a handful of global layers then I think that's okay with option 1, they don't change much.
If we did the full OSM index, as a one time sync it would get out of date, endpoints change etc so dumping it in without the maintenance is not an option in my opinion. Either we decide to maintain it, or we go with option 2 and have OSM layers sync through automatically.
So I think option 2 is the best way forward, the downside is it needs some code to merge OSMs index into ours and some thought about which layers from OSM we can't use for OHM.


feat: 优化自定义 dict 的逻辑

PR 的功能

自定义 dict 中的单字会被添加至 DICT1 中
修复 dict remove 失败的问题

你的预期是什么
是否进行了详细的自测？
是

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

Project coverage is 100.00%. Comparing base (ae73e88) to head (b6f1010).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##              main      #239   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           23        23
  Lines        13211     13273   +62
  Branches       424       440   +16
=========================================
+ Hits         13211     13273   +62

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


Add a creator to Workflow Documentation

Let the user identify him/herself in the MaRDMO Questionnaire and add the creator to the Documentation Entry. Allow MaRDMO creators to update their own workflows through MaRDMO.

Done, maybe at some later point intoduce other IDs next to ORCID or zbmath ID.


Llamafile comparison?

@mukel thank you for creating this project! I would like to discuss the following topics:

Please enable the Discussions tab for posts like this, which are not real "issues"

Do you plan on releasing Llama3 code?

Do you plan on quantized llama models with Java vector api?

Can you run a benchmark against llamafile, the vector version of which (AVX, neon) claims to be the performance king for inference.
(I am deciding between using your project or wrapping around the llamafile c code with Java 22 foreign function apis)

Do you plan to implant model training as well? If so, take a look at Andrey's LLM.c repo

Please enable the Discussions tab for posts like this, which are not real "issues"

Done! Thanks for the hint!

Do you plan on releasing Llama3 code?

Yes, I have a VERY cool demo running locally, stay tuned. I'll release the demo first, then Llama (3) as a Java library later e.g. including a backend for Langchain4j, supporting function calling and structured responses (JSON) based on NousResearch's Hermes fine-tunes.

Do you plan on quantized llama models with Java vector api?

Yes, I have already fast matmul routines for Q4_0, Q4_1 and Q8_0 using the Vector API.  k-quants are still a WIP.

Can you run a benchmark against llamafile, the vector version of which (AVX, neon) claims to be the performance king for inference.
(I am deciding between using your project or wrapping around the llamafile c code with Java 22 foreign function apis)

I bet llamafile is a bit faster, but hopefully soon, the Graal compiler will add enough support of the Vector API to optimize, at least, the matmul routines. I big advantage is that you have direct access to the model guts e.g. you can serialize/compress/cache/hack the model state, tweak the inference rather easy.

Do you plan to implant model training as well? If so, take a look at Andrey's LLM.c repo

I was tempted to port llm.c to Java as well, but training requires way more resources and accessing GPUs from Java is rather cumbersome ATM. With TornadoVM and Project Babylon this could change...

Please check Llama3.java.
I compared mistral-7b-instruct-v0.2.Q4_0.llamafile and Mistral.java using the following HW configuration:
Intel 13900H 6pC/8eC/20T 64GB (5200) Linux 6.6.30
Mistral.java
jbang Mistral.java \
  --model Mistral-7B-Instruct-0.2-Q4_0.gguf \
  --max-tokens 512 \
  --prompt "Why is the sky blue?"

Output:
Parse Mistral-7B-Instruct-v0.2-Q4_0.gguf: 75 millis
Load LlaMa model: 100 millis
...
7.41 tokens/s (186)

llamafile
./mistral-7b-instruct-v0.2.Q4_0.llamafile \
  -n 512 \
  -p "[INST] Why is the sky blue? [/INST] "

Output:
...
llama_print_timings:        load time =     239.69 ms
llama_print_timings:      sample time =       4.91 ms /   164 runs   (    0.03 ms per token, 33421.64 tokens per second)
llama_print_timings: prompt eval time =     651.57 ms /    15 tokens (   43.44 ms per token,    23.02 tokens per second)
llama_print_timings:        eval time =   25324.82 ms /   163 runs   (  155.37 ms per token,     6.44 tokens per second)
llama_print_timings:       total time =   26000.08 ms /   178 tokens

I discovered that llamafile, by default, uses 1 thread per performance core, no hyper-threading, no economy cores ...
Running llamafile with -t 20 (all threads available):
...
llama_print_timings:        load time =     285.27 ms
llama_print_timings:      sample time =       4.32 ms /   141 runs   (    0.03 ms per token, 32646.45 tokens per second)
llama_print_timings: prompt eval time =     680.35 ms /    15 tokens (   45.36 ms per token,    22.05 tokens per second)
llama_print_timings:        eval time =   19403.89 ms /   140 runs   (  138.60 ms per token,     7.22 tokens per second)
llama_print_timings:       total time =   20105.27 ms /   155 tokens

With -t 12 (all threads of performance cores):
...
llama_print_timings:        load time =     234.36 ms
llama_print_timings:      sample time =       4.14 ms /   100 runs   (    0.04 ms per token, 24154.59 tokens per second)
llama_print_timings: prompt eval time =     958.05 ms /    15 tokens (   63.87 ms per token,    15.66 tokens per second)
llama_print_timings:        eval time =   13545.93 ms /    99 runs   (  136.83 ms per token,     7.31 tokens per second)
llama_print_timings:       total time =   14523.26 ms /   114 tokens

This is a simple comparison, nothing rigorous, llama.cpp beats Llama3.java and llamafile probably outperforms Llama3.java and Mistral.java; I find myself these results rather surprising and unexpected TBH.
One thing to take into account is that llamafile/llama.cpp do ingest prompts much faster.


ENH Add L_11 matrix norm

Quick PR to add the matrix L1 norm, I need it for sparse dictionary learning.
Note that in scikit-learn, the Lasso accept a target matrix and solves a multi-task objective with a L_11 penalty (see https://github.com/scikit-learn/scikit-learn/pull/24843) . As per the principle of least surprise, we should add it in the Lasso fit logic of skglm.

I agree with the principle of least surprise, but our experience is also that trying to follow sklearn increases the maintenance cost on our side, so I'm not sure there's an easy decision here.

Closing since it is not compatible with our working set algorithm.


Add usmba.txt

Add usmba.txt

@hatim-essadeq I am sorry, the domain usmba.ac.ma is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).

Hello @philipto,
I am a legitimate student from USMBA, and I want to express my strong disagreement with the decision to ban our entire domain. This blanket ban is causing serious problems for my studies and work:

I'm being forced to migrate to another IDE that I'm not familiar with
This disrupts my entire workflow and learning process
I have to spend extra time learning new tools instead of focusing on my actual coursework

Instead of banning the entire usmba.ac.ma domain, a more targeted approach would make more sense:

Ban specific accounts that were involved in abuse
Implement better verification systems for new accounts
Work with our university's administration to prevent misuse

It's unfair that all students have to suffer because of the actions of a few. Please reconsider this policy and implement a more nuanced solution that doesn't punish innocent students who are just trying to learn and work.
Could you please review this decision and consider a more targeted approach to handling abuse?


Nullability

Support for array and map types #135 implies the ability to query them. Since querying an array or a map is a partial operation, we need a way to handle errors / missing values.
Since biscuit datalog is untyped (for good reason imo, as much as i love types, i'm not sure how they could fit in the picture), a good solution for that would be to use a special null value for that.
Along with null, we need to define how it would interact with the rest of datalog:

behaviour with == (imo: behave as other values and return true when compared with null, false otherwise)
behaviour with other operators / methods (imo: behave as other values and blow up)
should it be possible to define it as a literal, or should it be only introduced by faillible operations?
defaulting operator (eg ??)

noting that it is related to https://github.com/biscuit-auth/biscuit/issues/130 because if we test equality of null with any other types, then equality between different types should not return an invalid type error


Update the schema to support Overwatch 2

pretty self explanatory, prob smthn missing, but this does a good job for now.

Can you run gofmt against this so we can keep the tags lined up


Allow the visibility behavior to be customized

Adds enum Visibility to LifetimeTrackerDashboardIntegration with options alwaysVisible, alwaysHidden, and visibleWithIssuesDetected.
The LifetimeTrackerDashboardIntegration initializer now has an argument for visibility with the default value .alwayVisible, preserving current behavior as the default.

thanks for contributing! 👍


Qabot fixes

diff against previous qabot model:

auto download data
data cleaning
fix cos sim operator, add output value check numpy implementation for cos sim
cudnn rnn output value check, gradient check
qabot loss function output value check
added a util tensor operation tensor.random, for quick testing
set individual cudnn rnn weights by new api

problem to check: pooling 1D layer has some issue


mlcroissant should not indent the JSON-LD by default

By default, the library returns an indented JSON-LD, but we should probably return a more compact version (with an option indent=True or False).

Solution: mlcroissant outputs a dict and lets the user choose the indentation.


More null checks in NotifyAnalysisBegins

Probably overkill, but #968 doesn't appear to have been enough.

IMO, fixes like that hide errors instead of fixing them. Not sure how hard is that in this case, but if nulls weren't expected originally, I'd prefer to find out why they are null, cause it may be a symptom of an invalid state.

For example:

Children can't be null and can't contain null elements, because AddChildScope is used only in one place and always with instance
Variables can't be null and can't contain null elements, because it is enumeration of _variables it is always assigned with the real Variable instance in either DeclareVariable or LinkVariable.

So the only thing that can be null is actually GlobalScope, which is legally null until it is set in NotifyAnalysisComplete

I agree. Here is the stack trace I've been seeing after the last PR:
   at Microsoft.Python.Analysis.Modules.PythonModule.<>c.<NotifyAnalysisBegins>b__106_0(IScope c)
   at Microsoft.Python.Core.EnumerableExtensions.TraverseDepthFirst[T](T root, Func`2 selectChildren)+MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at Microsoft.Python.Analysis.Modules.PythonModule.NotifyAnalysisBegins()
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerSession.AnalyzeEntry(PythonAnalyzerEntry entry, IPythonModule module, PythonAst ast, Int32 version, CancellationToken cancellationToken)
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerSession.Analyze(PythonAnalyzerEntry entry, Int32 version, CancellationToken cancellationToken)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerEntry.<>c.<GetAnalysisAsync>b__23_0(Task`1 t)

So... Children cannot be null in both Scope and EmptyScope and GlobalScope. Variables is readonly property always initialized to new VariableCollection. VariableCollection always has _variables as ConcurrentDictionary...

Before #968 was:
   at Microsoft.Python.Analysis.Modules.PythonModule.<>c.<NotifyAnalysisBegins>b__106_1(IScope s)
   at System.Linq.Enumerable.SelectManySingleSelectorIterator`2.MoveNext()
   at System.Linq.Enumerable.WhereEnumerableIterator`1.MoveNext()
   at Microsoft.Python.Analysis.Modules.PythonModule.NotifyAnalysisBegins()
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerSession.AnalyzeEntry(PythonAnalyzerEntry entry, IPythonModule module, PythonAst ast, Int32 version, CancellationToken cancellationToken)
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerSession.Analyze(IDependencyChainNode`1 node, AsyncCountdownEvent ace, Stopwatch stopWatch, CancellationToken cancellationToken)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.Python.Analysis.Analyzer.PythonAnalyzerEntry.<>c.<GetAnalysisAsync>b__23_0(Task`1 t)
   at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
   at System.Threading.ExecutionContext.RunInternal(ExecutionCont


sdk/state: handle extra signatures in transactions

WHAT
check that open and close agreements have only 2 sigs max for declarationSigs, closeSigs, and formationSigs (open only)
also adds a unit test file sdk/state/close_test.go for close.go logic
WHY
we're not currently checking an agreement has too many sigs, which will cause the transaction to fail when submitted. Here we're checking that there's no more than 2 signatures.
After the CAP-40 change we're no longer supporting participants having multiple sigs, so each participant is allowed 1 signature (see comment here), for a total of 2 sigs allowed per agreement
closes https://github.com/stellar/experimental-payment-channels/issues/142

@leighmcculloch PTAL, I moved the check to the end of the Confirm functions as you suggested here


Fix: moved banner config from cloudcannon.config.yml to data/banner.yml

Context
Missing ID in Tweet shortcode

Additional information

Caused by banner 'id' field being set in global cloudcannon.config.yml
Removed banner config options from cloudcannon.config.yml to data/banner.yml

Testing (tester)
The reviewer should check on this branch:

The code
The site Link here

Before merge (PR owner)

[ ] Delete the test site in CloudCannon
[ ] Remove example tweet from index page

Added these changes in a seperate PR. This one had a few files that still had testing data.


Simplified fetch urls to relatives

This attempts to fix the issue in: https://github.com/davidusb-geek/emhass/issues/230
This should be checked before Pulled. Should validate with proxies.
From testing with Nabu Casa, everything seems to work. Will wait for response with @davidusb-geek or @purcell-lab.

This may be a PR to revert/remove if there is any issues with Proxies. As {{ basename }} was added in the first place to resolve those issues. From what I have testes it seems to work. However it will be good to validate before merge.


[d3d8] Soldiers: Heroes of World War II - crash on startup

The game crashes on startup with dxvk 2.4, although it used to work with latest d8vk master. I suspect the crashes are caused by: https://bugs.winehq.org/show_bug.cgi?id=32235 , namely the game relying on use-after-free in certain places.
Besides that (when it works) 3D rendering looks good, however game intro videos and campaign intro videos flicker (this doesn't happen with WineD3D). The flickering seems to occur because the game is using multiple swapchains whenever video playback is involved.
Software information
Soldiers: Heroes of World War II (GOG)
System information

GPU: RTX 4070
Driver: 535.183.01
Wine version: 9.12 Staging
DXVK version: ba47af5

Apitrace file(s)
Soldiers.trace.tar.xz
Log files

d3d8.log: Soldiers_d3d8.log

Adding this here for posterity's sake: the game starts up just fine with the d3d8 frontend (as-is) backported to dxvk 1.10.3 levels, so the crash is actually a regression in the d9vk/dxvk backend. Things used to be more accommodating when it came to UAF, at least.
Since the game was also working just before the d8vk merge into dxvk, at least with the workarounds I added for it, I expect it was some change done around that time that caused it.


Unlock All Items Doesn't Include Dragonmaid Deck Box

The unlock all items in settings currently doesn't unlock the new Dragonmaid structure deck box from the 11th July update.

You can try updating the game itself on Steam

Thanks for the report. Fixed in v1.11.


cert server does not create server on 9999

certificate server appears not to create httpd server on 9999.
locally available on that interface, but not from another host.
iptables?

[root@certserver tmp]# curl <IP_ADDRESS>:9999 | head
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  4897  100  4897    0     0  1770k      0 --:--:-- --:--:-- --:--:-- 2391k
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd"><html><head>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
                <title>Apache HTTP Server Test Page powered by CentOS</title>
                <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

    <!-- Bootstrap -->
    <link href="/noindex/css/bootstrap.min.css" rel="stylesheet">
    <link rel="stylesheet" href="noindex/css/open-sans.css" type="text/css" />

<style type="text/css"><!--

Can curl on master1 before iptables runs.

Actually, not iptables:
  * remote_file[Retrieve the aggregator certs] action create_if_missing[2018-10-01T20:20:15+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-01T20:20:20+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-01T20:20:25+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"

[root@certserver generated_certs]# ls
openshift-master1.vagrant.test      openshift-master1.vagrant.test.tgz.enc  openshift-master-master1.vagrant.test.tgz
openshift-master1.vagrant.test.tgz  openshift-master-master1.vagrant.test   openshift-master-master1.vagrant.test.tgz.enc
[root@certserver generated_certs]# pwd
/var/www/html/master/generated_certs

This is 3.9.

I think the wire_aggregator only gets run if it's on a master.

Will wait to see if Will wants to fix - otherwise I think fix will involve moving the wire aggregator code to the certificate server.

@ianmiell Good catch, will push an update :+1:

Yes, I'd forgotten to make it part of the standard tests...

I think it's still failing:
https://jenkins.meirionconsulting.com/job/cookbook-openshift3-quick/job/master/162/
==> /tmp/chef.log.1538743201 <==
  * template[/etc/sysconfig/origin-master-api] action create (up to date)
  * template[/etc/sysconfig/origin-master-controllers] action create (up to date)
Recipe: cookbook-openshift3::wire_aggregator
  * remote_file[Retrieve the aggregator certs] action create_if_missing[2018-10-05T12:43:23+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:28+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:33+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:38+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:43+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:48+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"
[2018-10-05T12:43:53+00:00] WARN: remote_file[Retrieve the aggregator certs] cannot be downloaded from http://<IP_ADDRESS>:9999/master/generated_certs/wire_aggregator-masters.tgz.enc: 404 "Not Found"

not sure whether this was supposed to have fixed it.

@ianmiell Should be ok now :+1: https://github.com/IshentRas/cookbook-openshift3/commit/3ef6528be62ce94f5bc9ec3ae419337cf2bcc217


🚀 Feature: Kubernetes plugin: logs --previous=true

🔖 Feature description
An additional button that shows logs for the previous container instance. Equivalent to kubectl logs -p ....
🎤 Context
Key feature for troubleshooting. https://github.com/backstage/backstage/issues/15753
✌ Possible Implementation
function getPodLogs() already has the parameter previous in KubernetesProxyClient.ts
👀 Have you spent some time to check if this feature request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
None

Hi @ifkit, @camilaibs
I wanted to contribute to this issue, and I think we can build upon the suggestion you gave. Here’s one way we can go about it:

In KubernetesProxyClient.ts, locate the getPodLogs() function, which already fetches logs. We'll ensure it can handle logs from the previous container instance.

Introduce a "View Previous Logs" button in the UI, allowing users to easily check logs from before a pod restarted.

When clicked, the button will trigger the log retrieval with the --previous=true flag, using the existing infrastructure.

Let me think what you think of this approach, and I can possibly collaborate on this issue.

@backstage/kubernetes-maintainers any recommendations?

Hey all, the code for this exists already and is used in the error detection functionality.
Did you want to suggest adding this as a option somewhere else?

Hey all, the code for this exists already and is used in the error detection functionality.
Did you want to suggest adding this as a option somewhere else?

Are these logs only available when the pods are running into error? Because I am assuming the original need was for collecting the logs even for the successful pod instances as well.


ACE-zh

Hello, dear author.
I encountered a problem with Chinese data processing. I tried to use 'bash./scripts/process_ace05ep. sh' to change the English content to Chinese, but an error occurred as shown in the following figure:

, I have successfully preprocessed the English data. Can you provide some help?'? Thank you very much.

Hi, thanks for your interest in our work. For this work, we do not consider Chinese data. If you are interested in this part. You can check another work and script https://github.com/PlusLabNLP/X-Gear/tree/main/preprocessing


isVersionCompatible should use version compability, not equality

In the following test for version compatibility:
https://github.com/cqframework/clinical-reasoning/blob/938ed778c19ccf34c5a7726d9949c2798f46521e/evaluator/src/main/java/org/opencds/cqf/cql/evaluator/engine/execution/TranslatingLibraryLoader.java#L266
This will in general always fail because the compability level is expressed as major.minor, where the implementation version will be specified to the patch level, and potentially include a snapshot label.

That's intentional. We don't currently consider how ELM generation is impacted in the major.minor.bugfix versions in the cql compiler so have to assume that a mismatched version is not compatible. For example, this change here:
https://github.com/cqframework/clinical_quality_language/pull/1163
required runtime changes to support correctly and (intentionally, due to performance considerations) pushed overload resolution from runtime to compile time.


Updated default version of Kubernetes to 1.26.6

...  following deprecation of 1.24 on AKS
PR Summary
Updated all references to "1.25.6" to "1.26.3":

bicep/main.bicep
helper/src/config.json
samples/SampleAppMain.json
samples/SystemPresetExample.json
samples/shared-acr/main.json

Fixes #626
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] This PR is ready to merge and is not Work in Progress
[x] Link to a filed issue
[x] Screenshot of UI changes (if PR includes UI changes)

Hey, thanks for the update!  We do tend to follow the default kubernetes version from the portal, so when aks pg are happy to bump the version, we follow.  Today looks like that is 1.25.11

Hi Keith,
It looks like different regions have different default versions, for example East US and UK South both have 1.26.6 as their default:

Whereas West Europe and Central US still have 1.25.11 as their default:

This seems to correlate with the release they're running: regions on Release v20230730 or newer are defaulting to 1.26.6, whereas regions still on  Release v20230723 are defaulting to 1.25.11.
If you like, we can wait until all regions are on release v20230830 or higher on the Release tracker?

Thanks for that Paul, approving!


Create a nice landing page

We need a good design on the landing page that gives an idea what this is for and also sets the right expectations for filling out the form.

Sketches from Franck: https://xd.adobe.com/view/83d34536-0b92-474a-83aa-1a7efae2905c-d680/screen/186c773a-da64-4f70-820f-8efacfd9e4b5


Update lol gTLD RDAP address in tests

CentralNic submitted the new records earlier this week, and the IANA DB
has been updated.

Thanks @tbregolin!

@tbregolin I think what you'll need to do here is:

Update the rdapURLs value in metadata/lol.json with the new CentralNic URL
Run make normalize (which will run go generate and update the root zones.go file)
Run the tests again, which should pass.
Commit these changes to your branch

@case thank you and my apologies -- I didn't realize the tests had passed locally only because my working tree was dirty form a previous local update. Updated.

No worries, the documentation and tooling for this workflow is probably a bit confusing.
Feel free to update the Readme from your experience, if that would have made contributing more straightforward — we appreciate your contribution here!

Looks like the nightly update job picked up the changes for the other Uniregistry -> CentralNic zones:
https://github.com/zonedb/zonedb/commit/b73642925fcdea4409901f560a0bc776496fcc59


Support admin credentials

It can be useful to connect to an AKS cluster using az aks get-credentials --admin.
Please support it in the extension

💡 Thank you for this @aelij , I am not entirely sure scenario is underlying for adding --admin flag for this but this extension is child extension of : https://github.com/Azure/vscode-kubernetes-tools extension.
Might be worth to detail your scenario which you think will benefit having the --admin flag.

Essentially here is what happens for get-credentials: https://github.com/Azure/vscode-kubernetes-tools/blob/master/src/components/clusterprovider/azure/azure.ts#L359

Also might be worth to move it to the parent extension.
Thank you 🙏

Thanks, @Tatsinnit. Could you maybe transfer the issue? I don't have permissions :)

Update:
Hiya @aelij, just to keep you in loop, I am planning to scope some work for this and see if there is a way to handle it before we transfer.
Will keep you posted. Thank you so much. 🙏

💡 🕵 Here is a possible idea / possible solution so I will share below as a note and will try it out sometime, if it is doable.

Create the shelljs wrapper in this project for the az command here https://github.com/Azure/vscode-kubernetes-tools/blob/master/src/components/clusterprovider/azure/azure.ts#L359
Then, create sub-menu with another menu option called Merge to Kubeconfig (Admin)
Then, I guess some how use the cluster explorer API to make it appear in the cluster explorer with (admin) alias next to the cluster merges with admin permission option. 🔑 Please Note: This part is something I am not sure about and will explore once I will start the implementation.

Just as FYI mentioning here as Plan of action (which could work).
Thanks heaps, 🙏☕

This has become much less needed now that AKS supports Azure RBAC.
But there is another issue, with RBAC kubectl requires some form of interactive login, and it causes the plugin to get stuck loading the cluster tree.

The cluster tree is a known issue in the vscode-kubernetes-tools extension (which this one builds on top of) and frankly I’m stumped on how to address it. The issue, as you’ve identified, is that we use kubectl to interact with k8s, including e.g. to get the list of known clusters; and in RBAC mode, kubectl pauses to prompt the user, and we can’t easily detect that, so we end up hanging forever.
I did some major rearchitecting of the CLI subsystem a year or two ago, with this as one of the things I wanted to lay the groundwork for fixing, but never quite cracked this issue.  Do feel free to hound me about it on the parent extension repo, and perhaps we can get a few more brains on it!

One small improvement that you could add - currently after it gets stuck, you have to restart VS Code. To fix it, I run kubectl manually, login interactively, then restart VS. It would be helpful if the extension would cancel the stuck kubectl command when clicking the refresh button instead of remaining stuck.
I'd also add a timeout for all kubectl commands.

That might be an acceptable compromise, and at this point an acceptable compromise is better than leaving it as is while trying to figure out an ideal solution. Thanks for the suggestion!

I'm closing this issue as it's less important


Fix #179 remove Full JRE dependency.

Codecov Report

Merging #180 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master     #180      +/-   ##
============================================
+ Coverage     91.11%   91.13%   +0.02%
  Complexity      100      100
============================================
  Files             2        2
  Lines          1609     1613       +4
  Branches        404      404
============================================
+ Hits           1466     1470       +4
  Misses           46       46
  Partials         97       97

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/picocli/CommandLine.java
91.53% <100%> (+0.02%)
62 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Why is it worth to add this package?
It is very important
Home page URL
No response
Source code URL
https://gitlab.freedesktop.org/fontconfig/fontconfig
Packaging policy acknowledgement

[X] I have read the Termux-Pacman Package Addition Policy and understand that my request will be denied in case of violation.

Additional information
very important

The fontconfig package is already in the gpkg repo. To install it run the following command:
pacman -S fontconfig-glibc


Mastopod doesn't appear to start

I have signed up but there is a blank screen after I sign in with username and password:

OK I tried firing up the tutorial stuff Jena Fuseki, backend frontend which are all fine then tried logging which failed, then tried password reset which also gave error message then tried creating a new account which gave:

Feel free to close

Thanks for reporting this @MarkCatch ! On what browser was it not working ?

Edge browser won't load Mastopod for me

Chrome works:


False positive on Invalid operands to binary expression when working with std::views::transform.

I was using clangd in Neovim when I noticed some odd error messages coming from the LSP that was not manifesting when I was compiling. I created a minimal example, which is shown here:

I am using std::views::transform to extract a view of the double data entries inside a std::vector<std::pair<double, int>>. For some reason, clangd does not like the use of the pipe operator (l) in this context. However, this code compiles and runs without issue (using g++ 11.4.0, not clang).
Here is a minimal example which recreates the problem:
#include <iostream>
#include <ranges>
#include <utility>
#include <vector>

auto get_doubles_view(const std::vector<std::pair<double, int>>& vec)
    -> decltype(vec | std::views::transform(&std::pair<double, int>::first)) {
    return vec | std::views::transform(&std::pair<double, int>::first);
}

int main() {
    std::vector<std::pair<double, int>> test = {{0.5, 1}, {1.1, 3}, {7.0, 2}};
    auto doubles_view = get_doubles_view(test);

    for (const auto& d : doubles_view) {
        std::cout << d << ' ';
    }
    std::cout << std::endl;
    return 0;
}

This is my CMakeLists.txt file:
cmake_minimum_required(VERSION 3.15)
set(CMAKE_CXX_STANDARD 20)
set(CMAKE_CXX_STANDARD_REQUIRED True)
add_executable(reinterpret_vec_pair reinterpret_vec_pair.cpp)

And I compile/run using these commands
cd build
cmake -DCMAKE_EXPORT_COMPILE_COMMANDS=ON ..
cmake --build .
./reinterpret_vec_pair

Additional note: The example code provided in the std::views::transform documentation gives rise to similar errors.
Logs
I am somewhat new to neovim, and I am not familiar with the process of extracting logs. I spent some time researching trying to figure it out, but I have not yet been able to extract logs related to clangd. I would appreciate some support on this front if you happen to have a straightfoward answer for me to try.
System information
Output of clangd --version:
Ubuntu clangd version 12.0.1-19ubuntu3

Editor/LSP plugin:
I am using neovim (output of nvim --version):
NVIM v0.9.5
Build type: Release
LuaJIT 2.1.1692716794

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/__w/neovim/neovim/build/nvim.AppDir/usr/share/nvim"

Run :checkhealth for more info

as well as lspconfig (running the default clangd settings, as found on the lspconfig github page)
Operating system:
Ubuntu 22.04.4 LTS

System information
Output of clangd --version:
Ubuntu clangd version 12.0.1-19ubuntu3

Could you try with clangd 18 (the latest version) from https://github.com/clangd/clangd/releases?
Clangd 12 was released in February 2021, a lot of bugs in clang's C++20 support have been fixed since then.

System information
Output of clangd --version:
Ubuntu clangd version 12.0.1-19ubuntu3

Could you try with clangd 18 (the latest version) from https://github.com/clangd/clangd/releases?
Clangd 12 was released in 2021, a lot of bugs in clang's C++20 support have been fixed since then.

This worked! Using clangd18:
clangd --version
clangd version 18.1.3 (https://github.com/llvm/llvm-project c13b7485b87909fcf739f62cfa382b55407433c0)
Features: linux+grpc
Platform: x86_64-unknown-linux-gnu

these errors are gone. I used the install instructions on https://clangd.llvm.org/installation.html for Ubuntu/Debian, but I didn't realize just how old that version would be. Thank you!

One more thing that might be handy: up-to-date versions of clangd (and other LLVM components) can be obtained in package form from https://apt.llvm.org/ as well.


Artifacts when moving PatternSprite

Unfortunately when I move a PatternSprite I get a black bar above it as shown in the attached image. The PatternSprite in this image is wrapping horizontally, and is moving downwards

We see the same issue when moving them in any direction, but only on HTML5 and only on mobile browsers. Typically it shows up as white lines for us though.
For example:
http://www.nick.com/games/data/nickjr/blaze_monster_machines/blaze-tool-duel/
Watch at the very bottom of the screen during the driving portions between the road and the sidewalk


Implement runs - staves and simultaneous playing

@all-contributors please add @viktordanov for code and infra


Use ctrl+c for graceful shutdown

Idea
Currently, you need to use the two python files digsinet-start.py and digsinet-stop.py to start and stop the application. It would be more ergonomic if normal ctl+c would gracefully shut down the application.

absolutely! ctrl+c can already be used to shutdown the controllers etc. If you rerun digsinet-start.py afterwards, contrainerlab will skip the creation and start of already running topologies (unless "--reconfigure" is not used as command line parameter). digsinet-stop.py was more or less initially developed to support stopping and cleaning up all running containerlab instances etc., but as digsinet-start.py and digsinet-stop.py share common code, it might be better to implement a command line parameter "--stop" or "--cleanup" etc. and use only one file? Also, regarding your idea maybe we should check that ctrl+c ensures a graceful shutdown, though I'm pretty confident this should be the case already, besides gNMI ops etc. possibly being interrupted?

update: just saw that you already started working on a branch that uses a single digsinet.py. Looks perfect! Go for it! 🚀

Now also at least partially implemented in main branch. Maybe we can close this issue for now?

Yup! Doing it with a custom SIGINT handler as it is done now is completely fine. I tried it with a similar approach before, but winded up with problems regarding containerlab. Calling sys.exit should free all used Resources and terminate all child processes.


Check the vote value is > Abstain in Governance vote function.

Description
Expected Behavior
Either value>Abstain should imply value=Abstain or revert
Current Behavior
Governance vote reverts on uint8 value=None but not uint8 value>Abstain
Other changes
Describe any minor or "drive-by" changes here.
Tested
Added unit test to test this change.
Related issues

Fixes #6948

Backwards compatibility
The Abstain votes are reverted.
Documentation
N/A

@yorhodes @nambrot Now the Abstain votes are not allowed and 4 tests are failing due to this change. Please review the change. Thank you.


[Feature] Multiselect Table widget should have a select all button

Summary
Select all button will help users select all rows in that page

@areyabhishek @Debsourabh A user requested for a single way to select + deselect all the rows of the table when the multi-select option is enabled.

Note:

The toggle to select all should have 3 states: all selected, some selected, none selected

Review Designs
https://www.figma.com/file/nx6spVrpSLSgD2AyLYN3jp/?node-id=2%3A13396

Note:

This column shows up when "multi select" is enabled in the table.
Hover highlights the box as shown in the designs
Clicking selects the particular row and shows a check mark in the box as in the designs
SelectedRow colour will change to the one in the designs
Clicking on the header checkbox will select all items in the "current page"
When some rows are selected, the header will identify this by having a "some selected" state @Debsourabh we need designs for this.
@Debsourabh We need to understand how the users would be able to select all rows including the ones not in the current page.

@Debsourabh selected row color should have opacity so that it works with underlying cell colors

Updated design : https://www.figma.com/file/trkiMc12VVpQBqsm7natiU/Table-Changes-Wed-7-Apr-2021?node-id=2%3A3024
Check user flows > multi-select and select options


Information Request - Using MySqlBuilder with mysqli

Hi there,
When building a select() query with a where() and an equals() call, when I write the query it looks like "select * from user where user.id = :var1". Am I able to change the ":var1" to the actual value I passed in? getValues() returns an empty array().
Thanks for your help!

Similar problem here. It would be nice to add support for mysqli (question marks instead of labeled parameters).
Temporary BUT NOT RECOMMENDED solution is:
$stringQuery = $builder->write($q);
$stringQuery = preg_replace('/\:v\d+/', '?', $stringQuery);

yeah I agree it will be it would be nice to use question marks @evanirla @zuffik


adds workflow yml

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@7f02c53). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #1   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         7
  Lines             ?       164
  Branches          ?         0
==========================================
  Hits              ?       164
  Misses            ?         0
  Partials          ?         0

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 7f02c53...667ed92. Read the comment docs.


bugfix: fix vulkan validation error VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT /VK_IMAGE_CREATE_2D_ARRAY_COMPATIBLE_BIT

https://registry.khronos.org/vulkan/specs/1.3-extensions/man/html/VkImageCreateInfo.html
If flags contains VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT, imageType must be VK_IMAGE_TYPE_2D
If flags contains VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT, extent.width and extent.height must be equal
If flags contains VK_IMAGE_CREATE_2D_ARRAY_COMPATIBLE_BIT, imageType must be VK_IMAGE_TYPE_3D
these rules overlap and are incompatible. if its a 2d with array then its a cubemap but can't also be a 3D image at the same time.

Hi! Thanks for raising this. Sorry, I misunderstood 2D_ARRAY_COMPATIBLE bit (TEXTURE_TYPE_2D already natively supports arrays). After reading the docs I would suggest the following revisited code (for the entire block):
    VkImageCreateFlags flags = VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT; // typeless
    if (textureDesc.arraySize >= 6 && textureDesc.width == textureDesc.height)
        flags |= VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT; // allow cube maps
    if (textureDesc.type == nri::TextureType::TEXTURE_3D)
        flags |= VK_IMAGE_CREATE_2D_ARRAY_COMPATIBLE_BIT; // allow 3D demotion to a set of layers // TODO: hook up "VK_EXT_image_2d_view_of_3d"?
    if (m_Device.GetDesc().isProgrammableSampleLocationsSupported && textureDesc.format >= Format::D16_UNORM)
        flags |= VK_IMAGE_CREATE_SAMPLE_LOCATIONS_COMPATIBLE_DEPTH_BIT_EXT;

What do you think?

v2 (true typeless):
    VkImageCreateFlags flags = VK_IMAGE_CREATE_MUTABLE_FORMAT_BIT; // typeless
    const FormatProps& formatProps = GetFormatProps(textureDesc.format);
    if (formatProps.blockWidth > 1)
        flags |= VK_IMAGE_CREATE_BLOCK_TEXEL_VIEW_COMPATIBLE_BIT; // format can be used to create a view with an uncompressed format (1 texel covers 1 block)
    if (textureDesc.arraySize >= 6 && textureDesc.width == textureDesc.height)
        flags |= VK_IMAGE_CREATE_CUBE_COMPATIBLE_BIT; // allow cube maps
    if (textureDesc.type == nri::TextureType::TEXTURE_3D)
        flags |= VK_IMAGE_CREATE_2D_ARRAY_COMPATIBLE_BIT; // allow 3D demotion to a set of layers // TODO: hook up "VK_EXT_image_2d_view_of_3d"?
    if (m_Device.GetDesc().isProgrammableSampleLocationsSupported && textureDesc.format >= Format::D16_UNORM)
        flags |= VK_IMAGE_CREATE_SAMPLE_LOCATIONS_COMPATIBLE_DEPTH_BIT_EXT;

Thanks Michael! That was fast! Integrated.


Analyzer Feedback from IntelliJ

Version information

IDEA AI-201.87<IP_ADDRESS>58069
2.10.1
AI-201.87<IP_ADDRESS>58069, JRE 1.8.0_242-release-1644-b01x64 JetBrains s.r.o, OS Windows 10(amd64) v10.0 , screens 1536x864

Closing due to lack of info.  Please re-open with error details.


Save and Re-attach disk on restart

Confirm with Chris whether this happens automatically.

Apparently some feature for this is coming.  I'm going to close it.


Transaction result conversion

This PR adds filtering for the transaction results, such that only Arrow encoded parts of the multi-part are deserialized. Using this I cleaned up the fast-path response and added a test exercising it. This uses a recent fix in HTTP.jl that correctly parses the content-type. @quinnj Are we already on #master of HTTP.jl? Otherwise should we do another release in HTTP.jl? :)
The current RAI engines send a relation-count part in the /transaction/results/ path that was breaking the result conversion.
@NRHelmi adding you as I don't know how to do a new release. Should we just create a new tag?

As the tests show we are not on #master of HTTP.jl :)
Should we do another release in HTTP.jl @quinnj ?

We're very close on HTTP.jl 1.0 release; my goal is in the next few (business) days. We've had a number of breaking changes, so it's a bit hard to parse out bugfixes along with the bigger breaking refactorings.

🤔 yeah not sure what to do about the HTTP 1.0 requirement.. 🤔 maybe we need to hardcode another crappy fix for this in the meantime, or just wait until it's released. :) Thanks for the fix, david!

@bachdavi I remember you need to comment or create an issue that contains
@JuliaRegistrator register

hopefully this won't trigger the package registration now :sweat_smile:

Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue.

I think HTTP 1.0 has been released. Can we add the compat bounds and try this again? :)

I think HTTP 1.0 has been released. Can we add the compat bounds and try this again? :)

I added 1.0 to compat here on this branch. Running the tests just now


Calling Mongo when using mongodb_ecto how to get the topology_pid

Hi I am using:
{:mongodb_ecto, "~> 1.0.0"}

The repo config is:
config :my_app, MyApp.Repo,
adapter: Mongo.Ecto,
database: "test_db",
hostname: "mongodb"

I want to count and group a status field. In ecto this can be done with something like that:
query = from c in MySchema,
select: {c.status, count()},
group_by: c.status

Repo.all(query)

This does not work, yet which is in the docs and ok. In mongdb shell it works with:
db.collection.aggregate([{"$group": {_id:"$status", count: {$sum: 1}}}])

So I want to call Mongo.aggregate directly, but I have problems getting topology_pid.
When I try to use MyApp.Repo.pool() as documented here this does not work, because the pool() function does not exist.
How to I get that pid, or how can I perform that aggregate.
Any help is welcome.
I also tried to find the pid with Process.whereis(MyApp.Repo) which did not work.
When I start a new process with:
{:ok, conn} = Mongo.start_link(url: "URL")

The function is working but it is not returning what I was thinking.

The function is working now, I got back a cursor and forgot to pipe that into Enum.to_list
Now the only question is how to get to the pid.

I think the answer is supposed to be MyApp.Repo.Pool, but that doesn't appear to be working.
This is ugly, and I am not going to guarantee that it is going to work all the time, but you can use the following code in order to get the pid of the actual mongo process.
MyApp.Repo |> :sys.get_state() |> get_in([Access.elem(3), Access.elem(1), Mongo, Access.elem(1)])

Thank you, its ugly but it works.
Aside: I hope that I can also use that to access a mongodb on DigitalOcean.


Survey Acknowledge type can cause database inconsistencies

Survey 2.0.4, Services 2.0.4
There is an inconsistency how boolean value from the Acknowledge question is stored in the local database from Survey and how the sync protocol stores the data.

Theses fixes will be released in version 2.0.5


Error for _apple-mobdev2 protocol

I'm getting an error on my MacBook (running Yosemite) when browsing for all services:
events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: protocol must be either '_tcp' or '_udp' but is '_apple-mobdev2'
    at checkProtocolU (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:184:11)
    at ServiceType.fromString (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:62:3)
    at new ServiceType (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:15:14)
    at Object.makeServiceType (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:133:10)
    at Browser.on_service_changed (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/browser.js:43:19)
    at SocketWatcher.MDNSService.self.watcher.callback (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/mdns_service.js:18:40)

It seems like some apple service reports a very strange protocol, but it means that I'm unable to browse all services. Can anyone else reproduce this? This command gives me an error straight away:
echo "require('mdns').browseThemAll().start()" | node

Could be an actual bug. Note that browseThemAll() is rather special. It browses for all available service types, not services. It might be ok, they don't have a protocol and the parser is just a little to paranoid. I'd apply the remove-offending-line-refactoring around here and see what happens.
HTH

Thank you for the quick reply!
I did not realize that distinction for browseThemAll(), and now that I try again I don't get the error, but get this service:
type: { name: 'apple-mobdev2', protocol: 'tcp', subtypes: [] }.
Maybe there was some error in parsing so that the name was interpreted as the protocol? It's strange in any case.
The reason I was looking into browseThemAll() in the first place was that I wanted to browse for this service type:
{ interfaceIndex: 5,
  type: { name: 'pro4_sd', protocol: 'tcp', subtypes: [] },
  replyDomain: '.',
  flags: 3,
  networkInterface: 'en1' }

But I get an error for the underline character in the name.
service_type.js:163
    throw new Error('type ' + str + ' may only contain alphanumeric ' +
          ^
Error: type pro4_sd may only contain alphanumeric characters and hyphens
    at checkLengthAndCharset (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:163:11)
    at ServiceType.fromArray (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:89:3)
    at new ServiceType (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:24:12)
    at Object.makeServiceType (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:133:10)
    at Object.tcp (/Users/johan/Projects/websites/node/stagedisplay/node_modules/mdns/lib/service_type.js:143:36)
    at Object.<anonymous> (/Users/johan/Projects/websites/node/stagedisplay/bonjour.js:2:39)
    at Module._compile (module.js:456:26)
    at Object.Module._extensions..js (module.js:474:10)
    at Module.load (module.js:356:32)
    at Function.Module._load (module.js:312:12)

Is this a bug in mdns or is the service violating the specification by using an underline? I notice that all other services I see use dashes for separation.

Okay, I saw that the service is actually violating the specification (page 19 of http://www.ietf.org/rfc/rfc6763.txt). I will fork and deviate from the standard because I need to use that service.


macOS: command not found: linguist

Describe the bug
When trying to run linguist selfdrive/ui/translations/your_language_file.ts as stated here, I get zsh: command not found: linguist
Is mac_setup supposed to install linguist, or is it somewhere hidden in qt@5?
Installing brew install --cask qt-creator did not seem to help.
I found some prebuilt binaries here that worked (by using the gui, not the linguist command) https://github.com/lelegard/qtlinguist-installers/releases
OS Version
Ventura 13.5.2
openpilot version or commit
master
Additional info
No response

@royjr Does Linguist (capitalized) work? Is your file system case-sensitive (can check that in Disk Utility app)?

Doesn't look like it. APFS (Encrypted)
blah@Someones-MacBook-Pro ~ % which ping
/sbin/ping
blah@Someones-MacBook-Pro ~ % which Ping
/sbin/Ping
blah@Someones-MacBook-Pro ~ % which linguist
linguist not found
blah@Someones-MacBook-Pro ~ % which Linguist
Linguist not found

Seems like homebrew distribution of qt@5 is packaging linguist as .app bundle, which is located at $(brew --prefix qt@5)/libexec/Linguist.app. It can be accessed from terminal using:
open "$(brew --prefix qt@5)/libexec/Linguist.app" --args selfdrive/ui/translations/your_language_file.ts

That works! Any way to make it an alias in mac_setup so that the instructions make sense or should the instructions just include that command for mac users?

We're considering re-focusing our macOS efforts on making the devcontainer experience good. Want to try that and post any feedback or issues? You can get started here: https://github.com/commaai/openpilot/tree/master/tools#dev-container

Recommended setup is now to use the Dev Containers for non-20.04: https://github.com/commaai/openpilot/tree/master/tools#dev-container-on-any-linux-or-macos


./gradlew :system-tests:end2end-test:catalog-runtime:shadowJar returns error  Failed to create Jar file C:\Users\XXX.gradle\caches\jars-9\87491e8cdb0229bafcb7120e23222513\jackson-core-2.15.2.jar.

Bug Report
Observed Behavior
I followed the guide written in https://github.com/eclipse-edc/FederatedCatalog/blob/main/system-tests/end2end-test/README.md
When I run command: ./gradlew :system-tests:end2end-test:catalog-runtime:shadowJar
The system returns following error:

Expected Behavior
BUILD SUCCESSFUL should be returned
Context Information

Used version [e.g. EDC v1.0.0]
OS: Windows 11 + WSL2
java version "20.0.1" 2023-04-18

Investigations
This issue doesn't occurs if I run:  gradle :system-tests:end2end-test:catalog-runtime:shadowJar
To use local configured gradle
PS C:\EDC\FederatedCatalog> gradle --version

------------------------------------------------------------
Gradle 8.2.1
------------------------------------------------------------

Build time:   2023-07-10 12:12:35 UTC
Revision:     a38ec64d3c4612da9083cc506a1ccb212afeecaa

Kotlin:       1.8.20
Groovy:       3.0.17
Ant:          Apache Ant(TM) version 1.10.13 compiled on January 4 2023
JVM:          20.0.1 (Oracle Corporation 20.0.1+9-29)
OS:           Windows 11 10.0 amd64

could you please report the full stacktrace?
it could be related to java version... could you try it with the version 17? (the current supported one)

Java sdk 17 works, thanks


Adding Cribl to Microsoft Azure Solutions Repo

Change(s):
Cribl Steam Solution added
Added a new folder named "Cribl" in the "Solutions" directory
Created a new Cribl sample files in the Samples Folder
Added Cribl-Logo.svg to "Logos" directory
Reasons for Change(s):
Cribl Stream Solution being added into the Microsoft Azure Sentinel repository.
Version 1.0.0 Added.
Tested and validated Solution file with preview function in Sentinel.
@microsoft-github-policy-service agree [company=“Cribl"]
@microsoft-github-policy-service agree

Hello @amiracle,

Please add graph queries to the Data connector.

Add id property to the parsers

Also check on Solution metadata's Domain.

Refer this Solution for more clarification.
https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Akamai Security Events

Hello @amiracle, Please create custom table schema named as CriblInternal_CL and CriblInternal at location
https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables

Hello @amiracle, we are waiting for your reply

Hello @amiracle, Can you please work on the above requested changes.

Hello @amiracle,

Please add graph queries to the Data connector.
Add id property to the parsers
Also check on Solution metadata's Domain.

Refer this Solution for more clarification. https://github.com/Azure/Azure-Sentinel/tree/master/Solutions/Akamai Security Events

This has been completed.

The graph queries have been added to the Data connector.
Id's have been added to the parsers.
The Solution domain of Security - Other has been added.

Hello @amiracle, Please create custom table schema named as CriblInternal_CL and CriblInternal at location https://github.com/Azure/Azure-Sentinel/tree/master/.script/tests/KqlvalidationsTests/CustomTables

All of the custom tables have been added to the repo:

CriblAccess_CL
CriblAudit_CL
CriblInternal_CL
CriblUIAccess_CL

@v-prasadboke - I'm trying to decipher the error messages as to why my code is not passing validation. Can you please explain the error codes so that I can properly address them and get this solution added to the repo?

Hello @amiracle, Can we get on a call for this.
Please share your time zone and availability so that we can plan for the same.
You can ping me on teams too if needed -<EMAIL_ADDRESS>

reference for parser : https://github.com/Azure/Azure-Sentinel/blob/master/Solutions/Snowflake/Parsers/Snowflake.yaml
V3 tool : C:\GitHub\Azure-Sentinel\Tools\Create-Azure-Sentinel-Solution\V3


feat: add direct links to repo, pr/branch, author

only implemented for github for the moment (also doesn't support github enterprise yet)

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign vbehar
You can assign the PR to them by writing /assign @vbehar in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign vbehar
You can assign the PR to them by writing /assign @vbehar in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

@vbehar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
fac9f5a2f62775dcf7a4e77d676aceeb140f5747
link
/test

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

@vbehar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
fac9f5a2f62775dcf7a4e77d676aceeb140f5747
link
/test

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

/retest

/retest

@vbehar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
fac9f5a2f62775dcf7a4e77d676aceeb140f5747
link
/test

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

@vbehar: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

pr
fac9f5a2f62775dcf7a4e77d676aceeb140f5747
link
/test

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

/approve

/approve

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

Needs approval from an approver in each of these files:

OWNERS [vbehar]

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

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

Needs approval from an approver in each of these files:

OWNERS [vbehar]

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


[2.7] Re-enable several RKE2 tests

Issue: https://github.com/rancher/rancher/issues/40055
Problem
A prior PR temporarily disabled several test cases for RKE2 clusters, but we never reenabled them until now.
Solution
Remove conditions which resulted in tests being skipped for RKE2 clusters, the tests being reenabled are
TestMachineTemplateClonedAnnotations TestMachineSetDeletePolicyOldestSet TestFiveNodesUniqueRolesWithDelete and TestFourNodesServerAndWorkerRolesWithDelete
Testing
Ensure the drone CI runs these tests and that they pass
Engineering Testing
Manual Testing
I've done the above, everything seems to be working properly and these tests run without issue
Automated Testing
Reenabled several RKE2 test cases
QA Testing Considerations
n/a, a quick look at the drone build should be all that's needed
Regressions Considerations
n/a

Probably a good idea to get @Oats87 input since it looks like he originally disabled the tests.

I'm going to wait for CI to pass before reviewing.
The original reason we disabled these tests was due to test runtime.

After bumping the Kubernetes version used in our CI to 1.25 these tests are hitting drone's runtime limit of 60 minutes killing builds, as Chris said in his comment. Interestingly, they did not hit this limit when running on an older K8's version. I'm going to run these tests locally on 1.25 to confirm they work, and if so the only issue with reenabling these tests will be the increased runtime exceeding drones limit.

@HarrisonWAffel , looks look bumping from 1.22 to 1.25 did not have much effect on already enabled tests, in fact it got a little faster on one pair of runs I found - 26:16 on 1.25 (https://drone-pr.rancher.io/rancher/rancher/25702/2/2) and 28:27 on 1.22 (https://drone-pr.rancher.io/rancher/rancher/25700/2/2).
The last run linked to this PR (https://drone-pr.rancher.io/rancher/rancher/25769/2/2) only logs ~1000 seconds of tests (so well under 1h), so it may be a single test that got stuck or something like this.

I can confirm that all of these tests pass independently (TestMachineTemplateClonedAnnotations, TestMachineSetDeletePolicyOldestSet, TestFiveNodesUniqueRolesWithDelete, TestFourNodesServerAndWorkerRolesWithDelete), however when run together the build times out at 1hr.
Looking at the tests more closely it seems that two of them are not specific to RKE2 but instead focus on behavior within Rancher itself. These tests are named TestMachineTemplateClonedAnnotations & TestMachineSetDeletePolicyOldestSet, and  focus on the behavior of provv2 controllers within Rancher which execute regardless of the underlying k8s distribution.
The tests TestFiveNodesUniqueRolesWithDelete TestFourNodesServerAndWorkerRolesWithDelete do seem to be specific to our integration with RKE2, so they need definitely need to be reenabled. I'll re-request reviews once I can confirm these tests pass without issue

The two tests which specifically focus on RKE2 are passing without any needed modifications. I think that this PR is ready for final review, as the re-enabled tests are not failing or timing out the CI process and will provide additional test coverage.

I've added a brief comment above the two tests which have not been re-enabled explaining why we do not want to enable them in the future for the RKE2 CI

@HarrisonWAffel , as discussed offline, let's hold off merging this one until after the 2.7.2 release. Changed the milestone on the issue to reflect this.


BP Filters for Mobile

Filtering BPs is not as easy for users when using mobile. Promoting the filters from sidebar to main section is probable solution. A dialog can also be a good way to use slider filters.
Filters to main section can be seen here:
https://app.zeplin.io/project/5a58ea3341f76658994e000c/screen/5bbf793bf616cc09a62926f6
Activated slider dialog can be seen here:
https://app.zeplin.io/project/5a58ea3341f76658994e000c/screen/5bbf793a6bc6d61900e2d331
What do you think?


Configure python tests in PyCharm and improve test coverage

Configure
In Run/Debug Configurations panel, add py.test configuration, see the image below:

Configure the test configuration according to the image below:

Save the changes and run coverage test:

After that, you should see the percentage of coverage for all files in lingobarter module.
Improve test coverage
Haoxiang: Improve lingobarter/core and lingobarter/utils modules test coverage
Coco: Improve lingobarter/ext and lingobarter/modules modules test coverage
Intentions
Hope you can be more familiar with the server-side code of our project. Very important!!!

@Coco-Yang @HaoxiangXu Please take a look at this issue.


Use a separate url for replies for sms transport

From the API docs:
If MO Message ID is validated, MT will not be charged.
Only one free MT is allowed for each MO.

I only noticed now that replies should be sent over a different URL for this.

Ready for review.

Ready for review.

Left one comment, otherwise looks good.

@hodgestar thanks for review. After testing integration, it doesn't seem like the reply api endpoint is working at the moment, going to change the PR to make this optional.

Now relies on #4 because of some overlap, otherwise ready for re-review.

:+1:


The 2 bugs/glitches of traincart.

BkCommonLib version:
1.67
TrainCarts version:
1.75
Spigot version:
1.8.7 SNAPSHOT

Problem or bug:
Train goes in floor
Train get stuck in turns
Expected behaviour:
A train is not doing this! ( Normally )
Steps to reproduce:
Like in the video :)
Video to explain:
https://www.youtube.com/watch?v=qC7XmpMPzEc
Thanks for resolving this in the future.

Update spigot to 1.8.9.

Ok i will try.

I can't find the 1.8.9 version 0_0

I think you need 1.8.8. There is no server version of 1.8.9, since that was only a realms patch.

@harm27 forgot that :smile:
@72juju Sharp 'U'-turns cause individual carts to lose perceptive of their direction. Keep at least one piece of track in between corner sections! Thus it can't be fixed.
The train in ground problem is already know and hasn't been fixed yet.

@mg-1999 That can happen.

@72juju check #43 or #20


report is_replica_updated to root

When revising the root allocator logic, it's valuable if it can know whether the replica log is updated to the leader.
For example:  when we try to rebalance replicas between nodes, the allocator chooses which replica be removed from the source node, it's better not to remove the updated replica when its updated replica count is just equal to the quorum(e.g. 3 replicas(a, b, c), 2 of them are updated(a, b), it's better to choose to remove c instead of removing a, b... to avoid group became unavailable), and root can try move other group's replica if no suitable replica in the current group, so root allocator need know "does replica is updated?"

I think we can report the last index of the raft log to the leader.


Include the pending message in json formatter output

Fixes #1924

Builds failing on Travis due to support? Seems unrelated to this.

If the pending message is not set, what do you think about using nil for it rather than a blank string?  Seems more natural to me...after all, that's what RSpec::Core::Example::ExecutionResult#pending_message returns, and it seems this should mirror that.

Builds failing on Travis due to support? Seems unrelated to this.

I pushed a commit last night to skip that spec until we fix it:
https://github.com/rspec/rspec-support/commit/9198ac30b7cfdf2915a615fdce21ee7a35a083db

I have a minor preference for an empty string (because you'd expect this to be a string value) but it's minor enough that if you'd prefer nil I'll change it.

I do prefer it.

Changed.

LGTM


First input box on modal should have focus

Currently adding or editing a card opens a modal for the card, but you must click in the form to set the focus. Putting the focus on the first element of the modal will improve the user experience.

In this case, I think just setting the autofocus attribute to the input elements will be sufficient.


MInimum api level

The minimum api level to use these lib is API - 16 . Can you downgrade that at least to api 14

Although it's not necessary, I would change the minimum api level to 14:)

Hey thanks


what version of node is the latest node-gyp compatible with?

For posterity, it's nearly always best to use the node-gyp that shipped with node.


reverting docs and updating readme file

Codecov Report
Patch coverage: 79.16% and project coverage change: -1.27% :warning:

Comparison is base (ac700e1) 94.01% compared to head (1698a6a) 92.74%.
Report is 4 commits behind head on master.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #65      +/-   ##
==========================================
- Coverage   94.01%   92.74%   -1.27%
==========================================
  Files          10       10
  Lines        1838     1848      +10
==========================================
- Hits         1728     1714      -14
- Misses        110      134      +24

Files Changed
Coverage Δ

nyoka/statsmodels/exponential_smoothing.py
83.83% <72.72%> (-0.70%)
:arrow_down:

nyoka/skl/pre_process.py
95.87% <80.00%> (-0.45%)
:arrow_down:

nyoka/metadata.py
100.00% <100.00%> (ø)

nyoka/skl/skl_to_pmml.py
97.72% <100.00%> (-0.29%)
:arrow_down:

... and 1 file with indirect coverage changes

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


⚠ ClamAV has degraded performance

In baab572, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 84 ms

Resolved: ClamAV performance has improved in 653ba03.


[version] Update to 3.2.4

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Refer or integrate mobilenet-v3-rs into examples

It happens that I had to implement MobileNetV3 (repo link) for some purpose. I think it's small enough to fit into examples. It provides an example program and library interface. I have plan A and B to publish this work.

A: Integrate into the examples. It would be better to let users import the model in the code.
B: Keep it a standalone repo, and I will publish it to crates.io.

If plan A, we would need someone else to verify the correctness of the impl. So far I'm only certain that the precision reaches paper's claim in minutes.

This seems very nice. We only have mobilenet v2 currently in the examples directory (mobilenet.rs). Having mobilenet v3 would sound like a nice addition if you want to go the A way.
Do you know if there are some pretrained weights for the original model ? This could make it easier to verify correctness + this way it could directly be supported for things like the inference example.

The paper is shinny new and only handful of MNetV3 impls out there, and yes, I agree pretrained weight is good source for verification. That implies some work, so I haven't put it into my schedule. I would compete the performance b/w existing MNetV2 and V3 to see if it really works.
For MNIST, the V3 reaches 98% accuracy on train and test set. For CIFAR-10, the accuracy sways near by 96%, and gets 75% on test set. It's clear clue of overfitting. I looked into existing V2. I gets 96% most of time, and 68% on on test set. It overfits as well, and V3 did some improvement.
We could give plan A a chance. I prefer some customizability for users, so there are things to talk about:

Let users specify model parameters like input channels, dropout, width multi and n. of classes. Unlike existing V2 we can only specify n. of classes.
Make building blocks available to users. We have bottleneck module, squeeze-excitation module, and {,H-}swish activation. If you think it's too verbose,  provide {,H-}swish because it's already in PyTorch.

Thanks, happy to help with importing some pretrained weights if there is already a good PyTorch implementation somewhere.
Re cifar10 I'm probably missing something but 75% test accuracy seems pretty low even with some strong overfitting. Looking at existing implementations for v2, it should be more than 94% accuracy on test/validation, e.g. https://github.com/tinyalpha/mobileNet-v2_cifar10 or https://github.com/kuangliu/pytorch-cifar
Also the models in examples/cifar also reach 94% test accuracy (and 99% on the training set). Is there something that I'm missing here ?

That's weird. Even changing the example to train existing v2 model does not reach that high accuracy. In the recent test I got 73% on val set, with batch size 32 and 100 epochs. Of course, v3 has no luck. The only catch is that there is no shuffling. Not sure if it crucial part and I'll give a try.

Closing as this has stalled a bit, feel free to re-open if you want to push more on this (it would certainly make a nice additional example).


tmux issue with more recent update

With the most recent update, tmux won't honour my .zshrc. Even when I run source .zshrc I get the following:
$ echo $SHELL
/bin/zsh
$ source .oh-my-zsh/oh-my-zsh.sh
-bash: autoload: command not found
-bash: .oh-my-zsh/oh-my-zsh.sh: line 31: syntax error near unexpected token `('
-bash: .oh-my-zsh/oh-my-zsh.sh: line 31: `for config_file ($ZSH/lib/*.zsh); do'

I was in error here. What had actually happened (with me recently moving to mac and iterm) was that I had configured the default command to use bash with reattach-to-user-namespace...
set-option -g default-command "reattach-to-user-namespace -l bash"

when it should have been the default shell, obviously...
set-option -g default-command "reattach-to-user-namespace -l ${SHELL}"


[Bug Report] Wrong button size in round buttons

Element UI version
2.0.0-beta.1
OS/Browsers version
Chrome 61
Vue version
2.5.2
Reproduction Link
https://jsfiddle.net/1pg6x0z6/
Steps to reproduce
view: http://element.eleme.io/2.0/#/zh-CN/component/button#bu-tong-chi-cun
What is Expected?
Different buttons size should have different sizes
What is actually happening?
Different buttons size should have the same sizes

Fixed in #7634


Add prebuilds for node 18

There was released new major LTS version (18.x.x) of node.js on 25th October 2022.
It will be great to add zookeeper prebuilds for this version.

Good idea @miroslavpokorny! I will add Node.js 17 prebuilds as soon as possible.

A new version of this package is released to npm! The latest version contains Node.js v.18 prebuilds for Mac OS X (intel and M1), plus Windows.


release: Zephyr v3.1.0-rc3

Release tag for Zephyr v3.1.0-rc3.
Signed-off-by: Martí Bolívar<EMAIL_ADDRESS>

DNM yet, just lining this up for review. I plan on merging this at the end of the day to give people as much time as possible to close low priority bugs today.


Are cache updates in Apollo client an over optimization?

Should we just refetch instead? Could remove the need for a common package.

Fine for now


Make Jörmungandr NetworkLayer use raw blocks

Issue Number
#711
Overview

[x] I changed the constructors in Cardano.Wallet.Jormungandr.Network to create NetworkLayers with raw jörmungadnr-specific blocks, and later used toWLBlock <$> nl. This way we can later use toSPBlock <$> nl for stake-pools.

Comments

bors r+

bors r-

@KtorZ Why not the retry?
bors r+

Failures:
--
  |
  | test/integration/Test/Integration/Jormungandr/Scenario/CLI/Server.hs:174:37:
  | 1) CLI Specifications, LOGGING - cardano-wallet serve logging, LOGGING - Exits nicely on wrong (yet hex-encoded)genesis hash,<PHONE_NUMBER>111111111111111111111111111111
  | uncaught exception: IOException of type InvalidArgument
  | fd:29: hGetContents: invalid argument (invalid byte sequence)
  |
  | To rerun use: --match "/CLI Specifications/LOGGING - cardano-wallet serve logging/LOGGING - Exits nicely on wrong (yet hex-encoded)genesis hash/1111111111111111111111111111111111111111/"
  |
  | Randomized with seed<PHONE_NUMBER>

bors r+

@KtorZ Why not the retry?

Because I was prioritizing some other PRs for the release.

bors r+


[WIP] Make ListOp.num_qubits return a list of numbers of qubits

Rather than returning the number of qubits in the first op,
return a list of the number of qubits in each op.
Fixes #1086
✅ I have added the tests to cover my changes.
✅ I have updated the documentation accordingly.
x I have read the CONTRIBUTING document.

One thing to consider is that there are several methods, e.g. PrimitiveOp._check_zero_for_composition_and_expand
which rely on self.num_qubits returning the int. So this will force the modification on existing code that call self.num_qubits on OperatorBase.

As #1189 provides an alternative to this, that we will go forwards with, can we close this off.

As #1189 provides an alternative to this, that we will go forwards with, can we close this off.
Yes, I think we agree #1189 is the better option. I am closing the this PR.


fix: Fix order of events in stapp-select module

Pull Request Test Coverage Report for Build 324

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

Totals

Change from base Build 323:
0.0%

Covered Lines:
661

Relevant Lines:
661

💛  - Coveralls


Root Node Array Exception

If the root node is an Array, then an exception is thrown.
new JSONParse(json).get('root[0]')
The above code fails with  No match found for <root[0]>
I'm able to work around this by doing new JSONParse(json).get('root').get('[0]').

Okay, I figured out the issue. The proper syntax is root.[0] . User error :)