maxSurge: 1 prevents fixing deployment where there's extra pod in CrashLoopBackOff state from previous failed deploy

I think I'm finally able to reproduce this issue. I also think it's not kubernetes issue but helm's one as I'm able to manually edit deployment (change image) with kubectl and everything works.
How to reproduce:

Create helm deployment with following strategy on deployment:

  strategy:
    type: RollingUpdate
    rollingUpdate:
      maxSurge: 1
      maxUnavailable: 0

Set livecheck on deployed service:

        readinessProbe:
          initialDelaySeconds: 1
          failureThreshold: 3
          periodSeconds: 10
          initialDelaySeconds: 5
          tcpSocket:
            port: {{ $d.service.internalPort }}
        livenessProbe:
          initialDelaySeconds: 10
          failureThreshold: 3
          periodSeconds: 15
          initialDelaySeconds: 15
          tcpSocket:
            port: {{ $d.service.internalPort }}

Deploy correct service with helm
Change image to broken one (eg. docker tag broken that crashes during startup)
Newly deployed image enters crashed loop while healthy one still works and is serving:

kubectl get pods --namespace blog
NAME                    READY     STATUS             RESTARTS   AGE
blog-5978979fd-j7zgl    0/1       CrashLoopBackOff   1          20s
blog-7f6f977d7d-5nm6h   1/1       Running            0          4

Change image back to working one (e.g. latest)
Notice how healthy release is not deploying (probably because it's waiting for crashing one to be ready which will never happen

Now if you change maxSurge to 2 everything is deployed correcly because new service can start while bad one is crashing.
Solution?

Make maxSurge ignore pods in  CrashLoopBackOff status.
???

Info

Output of helm version:
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"8", GitVersion:"v1.8.2", GitCommit:"bdaeafa71f6c7c04636251031f93464384d54963", GitTreeState:"clean", BuildDate:"2017-10-24T21:07:53Z", GoVersion:"go1.9.1", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.7", GitCommit:"0c38c362511b20a098d7cd855f1314dad92c2780", GitTreeState:"clean", BuildDate:"2018-08-20T09:56:31Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AKS

Side note: I suspect maxSurge: 2 will prevent deplyments where 2 pods are in CrashLoopBackOff status (e.g. when replica count is set to 2), so there's really no maxSurge number that will work at all times..

This doesn't look like a Helm issue, but how kubernetes handles failing pods. What do you suggest we need to fix inside Helm to accomodate this?

I'm going to close this ticket due to inactivity, but please re-open if this still needs to be addressed. Thanks!


blocks[i].copy is not a function

var modbus = require("modbus-stream");

module.exports = {
    initModbus: function() {
         modbus.tcp.server({ debug: "server" }, (connection) => {
              connection.readHoldingRegisters( {address: 0, quantity: 1}, (err, info) => {
                   console.log("response", info.response.data);
              });
         }).listen(12345, () => {
              modbus.tcp.connect(12345, { debug: "client" }, (err, connection) => {
                   connection.on("read-holding-registers", (request, reply) => {
                        reply(null, [234]);
                   });
              });
         });
    }
}

Hi, thanks for your effort. Could not achieve just a simple thing. What can be wrong here?
Error output:
events.js:141
      throw er; // Unhandled 'error' event
      ^

TypeError: blocks[i].copy is not a function
    at Object.blocksToBuffer [as buildResponse] (/home/ubuntu/workspace/EasygenSlave/node_modules/modbus-stream/node_modules/modbus-pdu/lib/Helpers.js:111:13)
    at Object.build (/home/ubuntu/workspace/EasygenSlave/node_modules/modbus-stream/node_modules/modbus-pdu/lib/Modbus.js:113:30)
    at /home/ubuntu/workspace/EasygenSlave/node_modules/modbus-stream/lib/transport/transport.js:165:75
    at Stream.<anonymous> (/home/ubuntu/workspace/EasygenSlave/modbus-tcp-server.js:12:13)
    at emitTwo (events.js:87:13)
    at Stream.emit (events.js:172:7)
    at Transport.<anonymous> (/home/ubuntu/workspace/EasygenSlave/node_modules/modbus-stream/lib/stream.js:44:10)
    at emitThree (events.js:97:13)
    at Transport.emit (events.js:175:7)
    at handle (/home/ubuntu/workspace/EasygenSlave/node_modules/modbus-stream/lib/transport/transport.js:156:16)

Can you give me guidance when you are available? Cheers.

Change the line:
reply(null, [234]);

To something like:
reply(null, [ Buffer.from([ 0x02, 0x34]) ]);

It worked but one thing is not clear to me. What are the addresses for these values? How can we specify them?

I'm not sure you know the basics around modbus.
In your example you requested address 0 and quantity 1, so you're asking only for data in address 0. The reading is sequential so if you would request quantity 3 you would read addresses 0-2.
In this case, holding registers, each address containers 2 bytes, so I represent those 2 bytes as a nodejs Buffer with 2 bytes. If something would request you to read from address 0 to 2, you could reply something like:
// address 0 = 0x0102
// address 1 = 0x0304
// address 2 = 0x0506
reply(null, [ Buffer.from([ 0x01, 0x02 ]), Buffer.from([ 0x03, 0x04 ]), Buffer.from([ 0x05, 0x06 ]) ]);

Thanks for the answer. What if master asks for register address 450010? Is there a solution to this?

If you mean 450 010 that doesn't exist as the addresses are 2 bytes (65 536 addresses).
If you mean 45 010, you're probably talking about the addresses of the holding registers. The standard does not mention any offset but many hardware suppliers have the datasheets for the holding registers starting at 40 001 which means address 0. Using this logic, the address 45 010 would be 5 009 (45 010 - 40 001).

http://www.modbus.org/docs/Modbus_Application_Protocol_V1_1b3.pdf

Also, this could help understand the protocol bytes behind the request.
http://www.simplymodbus.ca/FC03.htm

Sorry it was (4)50010 which means holding register #50010. Thanks for your help. How can I check if the request is made to this address?

You can enable debug and look at the bytes in hexadecimal. You have to know the protocol (the site above can help). Or... you can make a real request to a modbus device and see if the responde makes sense.

Really sorry, asked too many questions. I meant when we are the slave/server.

const modbus = require("modbus-stream");

modbus.tcp.server({ debug: "server" }, (connection) => {
	connection.on("error", (err) => {
		console.log("server error", err);
	});
	connection.on("read-coils", (req, reply) => {
		console.log(req); // <--- look at this object
		return reply(null, [ 1, 0, 1, 0, 0, .... ]);
	});
}).listen(502, () => {
	console.log("server ready");
});

As with read-coils, you can listen for the other events (read-holding-registers, ..).

Can we be slave on serial port with this?

With this module you can create any role and make requests from either role (slave or master)

Hi man. Just realized again this is a wonderful work. Pure and easy to use. Is there a ready way to give illegal data exception response?

Yes, just use the reply method and pass a string:
reply("IllegalDataAddress");

You can use any of these exception strings here.

Useful as always. Thanks.


google-cloud-storage: Read from well known files

// Google provide DefaultCredentials support via env.
// It will read credentials via:
// - file path in GOOGLE_APPLICATION_CREDENTIALS
// - Well known files on different platforms
//   - On unix platform: `~/.config/gcloud/application_default_credentials.json`
//   - On windows platform: `$APPDATA/gcloud/application_default_credentials.json`
// - Metadata server in Google App Engine or Google Compute Engine

Implemented in Implemented in https://github.com/Xuanwo/reqsign/pull/199


fix: partially fixes the issue with the windows build images script

Signed-off-by: Harikrishnan Balagopal<EMAIL_ADDRESS>

Partially fixes https://github.com/konveyor/move2kube/issues/921


Allow for the use of a proxy server during Python Language Server download

Fixes #2385

[x] Title summarizes what is changing
[x] Includes a news entry file (remember to thank yourself!)
[x] Unit tests & code coverage are not adversely affected (within reason)
[x] Works on all actively maintained versions of Python (e.g. Python 2.7 & the latest Python 3 release)
[x] Works on Windows 10, macOS, and Linux (e.g. considered file system case-sensitivity)

@DonJayamanne please re-review.

@ericsnowcurrently, @DonJayamanne: All PR comments are addressed, if someone could find the time to re-review? Thanks!

Retry the CI on VSTS - uncertain why this would fail! (Cannot reproduce it on Ubuntu linux...)


WIP - Update topics for content audit

Proposed changes

https://docs.docker.com/ee/ucp/admin/monitor-and-troubleshoot/troubleshoot-configurations/
https://docs.docker.com/ee/ucp/admin/configure/add-labels-to-cluster-nodes/

Related issues (optional)

Content audit
https://docker.atlassian.net/browse/ENGDOCS-291

This PR will be recreated in the Mirantis docs repo.


Esp idf issue Importing an existing projects other than examples in eclipse (IDFGH-5248)

I am facing an issue while Importing an existing project other than examples in eclipse Esp idf 2.6 and also in eclipse Esp idf 2.8. both versions run examples and build with no errors,
but whenever I add new > header and C files in the existing example project it is unable to add and find header and C files in the project, I also put these files in the main folder too.
after searching on forums :
I also tried to add these files CMakeLists.txt but the problem not solved.
2nd: whenever I existing project other than examples , all files [.C files and .H files ] are added but this time
eclipse will unable to add these fils in the project ,
#include <stdio.h>
#include "sdkconfig.h"
#include "freertos/FreeRTOS.h"
#include "freertos/task.h"
#include "esp_system.h"
#include "esp_spi_flash.h"
and also
c:/users/fuuast/.espressif/tools/xtensa-esp32-elf/esp-2020r3-8.4.0/xtensa-esp32-elf/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: esp-idf/esp32/libesp32.a(cpu_start.c.obj):(.literal.main_task+0x18): undefined reference to app_main' c:/users/fuuast/.espressif/tools/xtensa-esp32-elf/esp-2020r3-8.4.0/xtensa-esp32-elf/bin/../lib/gcc/xtensa-esp32-elf/8.4.0/../../../../xtensa-esp32-elf/bin/ld.exe: esp-idf/esp32/libesp32.a(cpu_start.c.obj): in function main_task':
C:/Users/FUUAST/Desktop/esp-idf/components/esp32/cpu_start.c:592: undefined reference to `app_main'
is any way I can import my existing projects without facing the problems. I only want to use eclipse right now.

Hi @naveedabbasi,
I noticed you've also opened a Pull Request with this content (#7016) and also via the Eclipse plugin project
https://github.com/espressif/idf-eclipse-plugin/issues/267
As this issue seems to be related to the Eclipse plugin then I suggest we close this duplicates. My colleague should reply to the issue in the Eclipse project, soon.


Confirm Dialog doesn't let you modify original form

In the confirm dialog, we stringify the form so that we can send it off to the server, but if I want to add hidden elements using JS, those don't get sent. Instead I have to just submit the original form.
i.e.
    let input;
    if (element.dataset.sendConfirmToServer === 'true') {
      input = document.createElement("input");
      input.setAttribute("type", "hidden");
      input.setAttribute("name", "server_confirm");
      element.appendChild(input);
    }

   //   and later
   if (element.dataset.sendConfirmToServer === 'true') {
        element.submit();
        resolve(false)
   }

Ran into this problem too


Resize while dragging to make fit

When dragging a sparse multi-qubit gate (e.g. a swap) as a single unit, the individual gates making it up could get closer together if doing so will let the thing fit on the circuit.

Better to just always make room.


paraview: 5.4.1 -> 5.5.2

This update should also fix the build the hydra: one part of the
change is to build paraview with Qt-5.9, as this is the configuration
that paraview upstream supports.
Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Fits CONTRIBUTING.md.

@GrahamcOfBorg build paraview

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: paraview
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Failure on x86_64-linux (full log)
Attempted: paraview
Partial log (click to expand)
[  2%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageThreshold.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageTranslateExtent.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageWrapPad.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkRTAnalyticSource.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageResize.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageBSplineCoefficients.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageStencilData.cxx.o
[  3%] Building CXX object VTK/Imaging/Core/CMakeFiles/vtkImagingCore.dir/vtkImageStencilAlgorithm.cxx.o
building of '/nix/store/njai8nwy1wvvvzjdksaa1dxmg8kr8x84-paraview-5.5.2.drv' timed out after 3600 seconds
error: build of '/nix/store/njai8nwy1wvvvzjdksaa1dxmg8kr8x84-paraview-5.5.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: paraview
Partial log (click to expand)
[ 28%] Linking CXX static library ../../lib/libvtkPVVTKExtensionsPointsCS-pv5.5.a
[ 28%] Built target vtkPVVTKExtensionsPointsCS
[ 28%] Building CXX object VTK/Filters/General/CMakeFiles/vtkFiltersGeneral.dir/vtkQuadraturePointsGenerator.cxx.o
[ 28%] Building CXX object VTK/Filters/General/CMakeFiles/vtkFiltersGeneral.dir/vtkQuadratureSchemeDictionaryGenerator.cxx.o
[ 28%] Building CXX object Wrapping/ClientServer/CMakeFiles/vtkFiltersCoreCS.dir/vtkArrayCalculatorClientServer.cxx.o
[ 28%] Building CXX object VTK/Filters/General/CMakeFiles/vtkFiltersGeneral.dir/vtkRandomAttributeGenerator.cxx.o
[ 28%] Building CXX object VTK/Filters/General/CMakeFiles/vtkFiltersGeneral.dir/vtkRectilinearGridClip.cxx.o
[ 28%] Building CXX object VTK/Filters/General/CMakeFiles/vtkFiltersGeneral.dir/vtkQuantizePolyDataPoints.cxx.o
building of '/nix/store/h56niqm6dfas0pa3xxcdadk2n1dz3cia-paraview-5.5.2.drv' timed out after 3600 seconds
error: build of '/nix/store/h56niqm6dfas0pa3xxcdadk2n1dz3cia-paraview-5.5.2.drv' failed

I didn’t see anything go obviously wrong in that build log. This is a terrifically long build, so it may just be that.


crashes on configuration change

Describe the bug
the sample app crashes on configuration change while showing some bottom-sheets
Library Version:
1.1.1
To Reproduce
open a multipleChoices bottom-sheet and rotate phone
Expected behavior
the app should not crash and also app should maintain state of bottom-sheet and selected options
Additional context
maybe some navigationComponent-ish changes could help

Found the issue. However, isn't the normal behavior of bottom sheets to dismiss at configuration change?

Found the issue. However, isn't the normal behavior of bottom sheets to dismiss at configuration change?

thanks for quick response, also check out fragment new dependency (1.3.0-rc01) it's pretty much stable and provide new ways to do things

thanks for quick response, also check out fragment new dependency (1.3.0-rc01) it's pretty much stable and provide new ways to do things


lsmash: fix build

lsmashsource does not build with any version of ffmpeg after 4, so we need to pin it to that version.

Seems like @tadeokondrak is no longer the maintainer as of #28, and removing tadeokondrak from with maintainers is necessary to get this PR to work again.


Moves tests into "tests" directory. Fixed imports where necessary. Updated MANIFEST

See #126

Looks good! Thank you!


streaming: eliminate usages of RwError

When we were refactoring the streaming executors with async-stream, we decided to also refactor the error type with the per-crate manner. For compatibility, we still capsulate some RwErrors in the StreamExecutorError.
https://github.com/singularity-data/risingwave/blob/f2c5758789ddd22ba6edbfeb6d7f5279ac919f9c/src/stream/src/executor/error.rs#L35-L57
After #2366 was introduced, we can start to eliminate these usages now.

[x] #3133
[x] #3134
[x] #3135
[x] #3136

Is this issue cleared? cc @BugenZhao

Is this issue cleared? cc @BugenZhao

I'm not sure, as we still rely on it for executor builders & stream manager. 🤔


Compile issue in case of compile_time_purge_matching logger option

Precheck
Adding logger compile_time_purge_matching option is causing ecto compilation fail.
Environment

Elixir version (elixir -v): 1.7.4
Database and version (PostgreSQL 9.4, MongoDB 3.2, etc.): -
Ecto version (mix deps): 3.0.1
Database adapter and version (mix deps): -
Operating system: Mac OS X

Current behavior
Here is an example of application.
config :logger,
  compile_time_purge_matching: [
    [application: :ecto_demo, level_lower_than: :info]
  ]

$ mix deps.compile ecto
==> ecto
Compiling 1 file (.ex)

== Compilation error in file lib/ecto/log_entry.ex ==
** (FunctionClauseError) no function clause matching in Keyword.fetch/2

    The following arguments were given to Keyword.fetch/2:

        # 1
        {:metadata, [line: 50], nil}

        # 2
        :application

    Attempted function clauses (showing 1 out of 1):

        def fetch(keywords, key) when is_list(keywords) and is_atom(key)

    (elixir) lib/keyword.ex:362: Keyword.fetch/2
    (logger) lib/logger.ex:829: anonymous fn/4 in Logger.compile_time_purge_matching?/2
    (elixir) lib/enum.ex:2850: Enum.all_list/2
    (elixir) lib/enum.ex:2864: Enum.any_list/2
    (logger) lib/logger.ex:800: Logger.macro_log/4
    (logger) expanding macro: Logger.log/3
    lib/ecto/log_entry.ex:50: Ecto.LogEntry.log/3
could not compile dependency :ecto, "mix compile" failed. You can recompile this dependency with "mix deps.compile ecto", update it with "mix deps.update ecto" or clean it with "mix deps.clean ecto"

Expected behavior
It is compiled.

So, I haven't looked very deeply into this yet, but looking at the documentation for compile_time_purge_matching, shouldn't your config be this instead?
config :logger,
  compile_time_purge_matching: [
    [application: :ecto_demo],
    [level_lower_than: :info]
  ]

The config is correct. This is a bug in Elixir. :) @van-mronov can you please report it on the elixir issues tracker? Thanks!

Just came back to correct my comment after reading the docs a bit more but you beat me to it :)

@josevalim ok)

Issue is reported in elixir repo.


Update es6-module-transpiler dependency

I am trying to solve error which (I believe) is caused by es6-loader using old version of es6-module-transpiler which (in version 0.4.0) uses buggy version of es6ify.
(https://github.com/esnext/es6-module-transpiler/issues/128)
I got the error while trying to set up karma to work with the same webpack settings as my build system.

+1

+1

+1

Same here. Also, the module transpiler is not picking up generator syntax for me.

switched to https://github.com/6to5/6to5-loader, more recent and active

You rock @tomatau!

Any objections to me deprecating this lib in favor of 6to5-loader?

Great suggestion @tomatau! @shama no objections from me.

There's also https://www.npmjs.com/package/traceur-loader that works fine too. Happy for this to be deprecated

No objections from me either.
Alternatives suggested by @tomatau looks good, thanks.


Clean up function help pages

This was an old note I had. Figure out which help pages I was talking about and turn them into a checklist.

Done


Prisma Migrate: Allow data transformations through code (in addition to SQL)

Problem
Currently, it seems possible to use SQL files in migrations. For greater flexibility, it would be nice to be able to provide JS files as well.
Suggested solution
Allow code to be executed in migrations

This would be super useful. There are scenarios where writing raw SQL to manage data migrations is a huge headache without the benefits of JS/TS.
This is especially critical if you are working on a multi tenancy application and need to move client specific data only. Something that you could only access via something like env variables (maybe you can configure this in SQL, I'm not sure, but it definitely would be time consuming and should be delegated to the node tool-chain).
Another, probably more common use case would be complex data that requires iteration to process properly: Using a for loop in SQL is not nice and it's error prone.
I think the implementation that TypeOrm or MikroOrm currently use is very nice, where each generated migration is a TS file that will then get converted to SQL via their query runner. This or a similar API would be a huge benefit for Prisma.

I agree. Migrations should always be ran via the ORM rather than plain SQL. Anything else limits you SO much, for 0 benefit.
Take this example from a TypeORM migration:
# database/migrations/AdsAbstraction1634246375736.ts

import { MigrationInterface, QueryRunner } from 'typeorm';

import { WorkflowEntity } from '../../../email-platform/entities/workflow.entity';
import { fixWorkflowUi } from '../../shared/utils/temp';

export class AdsAbstraction1634246375736 implements MigrationInterface {
  public async up(queryRunner: QueryRunner): Promise<void> {
    queryRunner.connection.logger.logMigration(AdsAbstraction1634246375736.name);

    await queryRunner.query(`
      ALTER TABLE "workflow-broadcast"
      RENAME COLUMN "fetchCarInsuranceAds" to "fetchAds"
    `);

    const workflowRepository = queryRunner.connection.getRepository(WorkflowEntity);

    const workflows = await workflowRepository.find();

    for (const workflow of workflows) {
      const workflowDocument = workflow.workflowDocument;
      fixWorkflowUi(workflowDocument);
      await workflowRepository.update({ id: workflow.id }, { workflowDocument });
    }
  }

  public async down(_: QueryRunner): Promise<void> {
    //
  }
}

Note that in addition to the plain SQL queries, I can use the ORM itself to loop through individual records and update them with extremely complex TS algorithms. This is absolutely impossible in plain SQL, and it's trival / easy using the ORM.
We should convert Prisma to always use ORM migrations instead of SQL migrations, similar to nearly all other major ORMs in all major languages.

@JVMartin Agree, having plain sql will make migrations unnecessarily harder, if there's a change in the data during the upgrade it needs to happen at the time of schema changes. not having this functionality within Prisma requires the developer to have a separate method for handling data changes after a schema change.

One problem that needs to be solved if planning to make Prisma Client available for data migrations is related to schema versioning. So the schema file should become part of the migration assets together with ts/js and sql files. So that each migration is pinned to a specific compatible version of the Prisma generated client.

This would help a lot in backfilling data with cuid values when adding non nullable columns to a table filled with data

Another orms migrations are implemented using orm, so I see no problems to make same in prisma. Look at laravel, typeorm or django/flask(SQL alchemy). Sometimes there is a need to run action before or after migrations, and I don't think that it is good idea to run them in clean SQL.


Windows 11 24H2 Support

When will this get updated to reflect 24H2? I am using 7.2.4 and do not see it.  I would rather use the exe vs the PowerShell directly.

I created a branch for this and uploaded my ps1 file. It has support for WIN11 24H2 and the script was updated to reflect v 7.2.5 as the prod one is currently 7.2.4.  Only issue is that it's not highlighting the Known Model column, but it does seem to select them.
https://github.com/rtruss/DriverAutomationTool_rtruss/blob/master/Content/DriverAutomationTool.ps1


Test deleting config.inis and seeing what breaks in the UI (user need to be able to fix the config from the UI still)

Following fails:
new Client(`${this.props.config['network.topology']['leader_address']}`, 9001)

I deleted the contents of config.ini and was still able to access the UI


run examples issue

ERROR in ./examples/auth-flow/app.js
Module build failed: SyntaxError: /tmp/react-router/examples/auth-flow/app.js: Line 7: 'this' is not allowed before super()
   5 | class App extends React.Component {
   6 |   constructor () {
>  7 |     this.state = {
     |     ^
   8 |       loggedIn: auth.loggedIn()
   9 |     };
  10 |   }
    at File.errorWithNode (/tmp/react-router/node_modules/babel-core/lib/transformation/file/index.js:488:13)
    at NodePath.errorWithNode (/tmp/react-router/node_modules/babel-core/lib/traversal/path/index.js:155:26)
    at NodePath.ThisExpression (/tmp/react-router/node_modules/babel-core/lib/transformation/transformers/es6/classes/vanilla.js:132:20)
    at NodePath.call (/tmp/react-router/node_modules/babel-core/lib/traversal/path/context.js:56:28)
    at NodePath.visit (/tmp/react-router/node_modules/babel-core/lib/traversal/path/context.js:89:8)
    at TraversalContext.visitSingle (/tmp/react-router/node_modules/babel-core/lib/traversal/context.js:132:12)
    at TraversalContext.visit (/tmp/react-router/node_modules/babel-core/lib/traversal/context.js:148:19)
    at Function.traverse.node (/tmp/react-router/node_modules/babel-core/lib/traversal/index.js:76:17)
    at NodePath.visit (/tmp/react-router/node_modules/babel-core/lib/traversal/path/context.js:106:26)
    at TraversalContext.visitSingle (/tmp/react-router/node_modules/babel-core/lib/traversal/context.js:132:12)
    at TraversalContext.visit (/tmp/react-router/node_modules/babel-core/lib/traversal/context.js:148:19)
    at Function.traverse.node (/tmp/react-router/node_modules/babel-core/lib/traversal/index.js:76:17)
webpack: bundle is now VALID.

This happens when i try to run npm run examples on Fedora, does anyone know what I can do about this ?

I'm assuming constructor should look like this ...
constructor(props) {
  super(props);

  this.state = {
    loggedIn: auth.loggedIn()
  }
}

I don't have a huge understanding of how the above works but I just know I need to do this to make this accessible in my class components :)
See https://facebook.github.io/react/docs/reusable-components.html#es6-classes
Maybe example is just broken?

So, do i have to prepare the examples ?  The readme just says, npm install, npm run examples, which fails also on OS X Yosemite.

:+1: Just checked out the 0.13.x branch and had this break.

all examples on master should be running, sorry about that.


Manage alert templates

As a rOps user, I want the ability to see the current alert banner running, and have the ability to remove it so that I may swap the alert banners which are displayed to respondents.
What has changed

How to test?

Links

/deploy terryj

/deploy terryj


Kucoin false order created event

Describe the bug
Upon initiating a buy order, I received a BuyOrderCreated event, indicating that the order had been successfully created. However, upon further inspection, it became evident that the order had not actually been created on the exchange. the order did not have an exchange order id. indicating that the order was never created.
Steps to reproduce
1.Execute a script to create an order on Kucoin.
2. Monitor for the BuyOrderCreated event to be triggered, indicating order creation.
3. Attempt to cancel the created order. At this point, an error is encountered, suggesting the order does not exist on the exchange
Release version
1.25
Type of installation
Docker
Attach required files

Issue reproduced when creating market order as well for v2 scripts.
# This is where SLMUT613256e6981c11091011461f21c03b0c06b market order created
# Observed that it does not have exchange_order_id
2024-03-08 20:35:47,388 - 5370 - hummingbot.strategy.script_strategy_base - INFO - Creating LOOM-USDT sell order: price: NaN amount: 108.3916.
2024-03-08 20:35:47,390 - 5370 - hummingbot.smart_components.executors.position_executor.position_executor - INFO - Placing close order --> Filled amount: 108.3916 | TP Partial execution: 0
2024-03-08 20:35:47,712 - 5370 - hummingbot.connector.client_order_tracker - INFO - Created MARKET SELL order SLMUT613256e6981c11091011461f21c03b0c06b for 108.3916 LOOM-USDT.
2024-03-08 20:35:47,712 - 5370 - hummingbot.core.event.event_reporter - EVENT_LOG - {"timestamp":<PHONE_NUMBER>.0, "type": "OrderType.MARKET", "trading_pair": "LOOM-USDT", "amount": "108.3916", "price": "NaN", "order_id": "SLMUT613256e6981c11091011461f21c03b0c06b", "creation_timestamp":<PHONE_NUMBER>.0, "exchange_order_id": "None", "leverage": 1, "position": "NIL", "event_name": "SellOrderCreatedEvent", "event_source": "kucoin"}
2024-03-08 20:35:47,716 - 5370 - hummingbot.smart_components.executors.position_executor.position_executor - INFO - Close Order Created

....

# Now getting error fetch data SLMUT613256e6981c11091011461f21c03b0c06b
2024-03-08 20:36:07,521 - 5370 - hummingbot.connector.exchange.kucoin.kucoin_exchange.KucoinExchange - WARNING - Error fetching status update for the active order SLMUT613256e6981c11091011461f21c03b0c06b: 'data'.

....

# Stop command initiated
2024-03-08 20:38:35,149 - 5370 - hummingbot.client.hummingbot_application - INFO - stop command initiated.
2024-03-08 20:38:35,388 - 5370 - hummingbot.connector.exchange.kucoin.kucoin_exchange.KucoinExchange - ERROR - Failed to cancel order SLMUT613256e6981c11091011461f21c03b0c06b
Traceback (most recent call last):
  File "/Users/rapcomia/github/hummingbot/development/hummingbot/connector/exchange_py_base.py", line 520, in _execute_order_cancel
    cancelled = await self._execute_order_cancel_and_process_update(order=order)
  File "/Users/rapcomia/github/hummingbot/development/hummingbot/connector/exchange_py_base.py", line 540, in _execute_order_cancel_and_process_update
    cancelled = await self._place_cancel(order.client_order_id, order)
  File "/Users/rapcomia/github/hummingbot/development/hummingbot/connector/exchange/kucoin/kucoin_exchange.py", line 231, in _place_cancel
    if tracked_order.exchange_order_id in cancel_result["data"].get("cancelledOrderIds", []):
KeyError: 'data'

This cause the script to fail and letting us exit the client. Therefore we have to use exit -f

Steps to reproduce:

Setup any v2 scripts with kucoin spot connector
Setup asset with 0 base asset and start the script
Observe the logs files until you get Error fetching status update for the active order then execute stop command

logs_conf_v2_dman_v4_config_1.log

Fix has been deployed to development and will be part of version 1.28.0


Track receipt_chain_hash and voting_for for accounts

Block explorers display this information with the account. Add receipt_chain_hash and voting_for to the Account Ledger.

Possibly related to #748

First, we should log these two values (receipt_chain_hash and voting_for) in our modified Mina node.

We decided that we won't do this a while back but this issue remains. Closing.


First Pass Cleanup

Cleans up many of the warnings printing in compilation.
Moves max objects to constants.h from kernels.cuh
adds 1 more sigfig to timing printout.

Validation: https://www.classe.cornell.edu/~mgr85/www/SegmentLinking_keep/PR132/eff_plots__GPU_explicit_cache_fd5eef7_PU200/


Drop to bottom of container

This isn't a bug, more so a support request and the reason this is being posted is I came here after reading about the gravity and thinking it is what I'm about to ask about.
What I'd like to do is drag an element, and when I finish dragging it, the element drops to the bottom, that being either the container if there are no other elements, or on top of the highest element below it, so that I can achieve something like this:

I appreciate the top down nature of web pages, and that this is turning the idea on it's head, so it may be considered a little unconventional, but this is a feature of a grid system many have sought for many years now and your library seems as if it could achieve this.
So I was wondering with a little of your help and knowledge, can we see if we can figure this one out? TIA.

Hi @BonnieSoftware, thank you for the comment.
Sorry, my English is poor.
Could you explain that by using other words? Or, Could you show me an example?
I can't understand what the building blocks picture mean.

No reply came, then this abandoned issue is closed.


Update README to document usage on<EMAIL_ADDRESS>

Closes #148

In version 1.0.0 I've removed the dependency to docstrap (for other reasons), then it works now on both versions of npm using the same config.
Thank you for your PR anyway !!!

That's nice, thanks!

On Oct 2, 2015, at 5:32 AM, Bertrand CHEVRIER<EMAIL_ADDRESS>wrote:
In version 1.0.0 I've removed the dependency to docstrap (for other reasons), then it works now on both versions of npm using the same config.
Thank you for your PR anyway !!!
—
Reply to this email directly or view it on GitHub.


fix: changed the wrong cheap-eval-source-map to eval-cheap-source-map

fixes the wrong cheap-eval-source-map to eval-cheap-source-map
Reporting source : https://twitter.com/rossipedia/status/1230913226000330753

Thanks!


SMS Chatting Enhancement

Tried sending out texts(SMS) from YakYak via my Hangouts Google Voice. The hangouts in browser shows the text sent but the recipient reported they did not receive it. Would like to see an ability to do SMS via Hangouts/GV messages in YakYak. Thank you.

Oh. Did not see this thanks.


Terrain analysis rasters are shifted

Description
The data coming out of the terrain analyses is inaccurate by several KMs and it's certainly not an ideal situation:

I'm certain this is due to the CRS being wrong after many of the changes that occurred when switching the sources of data over to WFS/WCS services. Will investigate this ASAP.

Fixed along with several bugs and enhancements with 254b1e3bdc7372fc28a9c681f26df7ba5438662d (accidentally pushed to master). Changes were as follows:

Fixed a bug where rasters were being improperly warped

Fixed a bug where shape properties were being converted to json twice (This was creating unreadable properties values for all reprojected shapes)

Removed GeoPandas from generic_terrain_analysis (Geometries are now read into Shapely objects within a list comprehension, removing the need for GeoPandas)

Removed obsolete HydroBASINS tables

Changed default Quebec Lambert EPSG from 32198 (retired) to 6622 * Adjusted clipping options to ensure no artefacts in slope raster


Allow passing arguments by reference

When creating a function, option to give parameters as reference or by value 👍

Implemented partially on the master branch


Web gui ?

Hello
Is there a WEB gui to make subscriptions ?
I have a
 { "code": 404, "debug": null, "error": "Not Found" }
when I brows in the /admin/ exposed

Ok I found that I need nmos-js. Issue solved
Just not understanding why some tabs are greyed out: example, in a receivers all the tabs except the summar are greyed, so I'm not able to make a connection... any help?


When loading files in MR2V, the screen is blank but thumbnails show image

When I load frames into MR2V, my screen is black but when I go to the timeline, it shows the thumbnails fine

This seems to be permission issues. So closing this one for now.


Skip Kubewarden namespace in the webhook configuration.

Updates the reconciler to add a namespace selector in the webhooks configuration to skip request from the Kubewarden's namespace. Therefore, avoiding issues during the policy server rollout due request being rejected by the old policy server instance.

The previous change adding an environment variable with the namespace to be skipped does not work. During the rollout request will be send to the old policy server instance asking to validate a policy, the policy which the rollout suppose to deploy, which is not running yet.

Important note: this fix does not suppose to be a final solution. It's just a short-term fix to allow users to use Kubewarden without big problems. We still need to come up with a better solution


trufflehog: 2.0.91 -> 2.0.97

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/trufflehog/versions.
meta.description for trufflehog is: '"Searches through git repositories for high entropy strings and secrets, digging deep into commit history"'.

Checks done (click to expand)

built on NixOS
/nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97/bin/.trufflehog-wrapped passed the binary check.
/nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97/bin/trufflehog passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 2.0.97 with grep in /nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97
directory tree listing: https://gist.github.com/63843e50bcd7cc7082afa0cd6f00bcf6
du listing: https://gist.github.com/8aa4356f41c669fdda7527d42db3f439

Outpath report (click to expand)

Outpath difference report
4 total rebuild paths
1 package rebuilds
1 x86_64-linux rebuilds
1 i686-linux rebuilds
1 x86_64-darwin rebuilds
1 aarch64-linux rebuilds
First ten rebuilds by attrpath
trufflehog

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(r-ryantm's Cachix cache is only trusted for this store-path realization.)
Or, build yourself:
nix-build -A trufflehog https://github.com/r-ryantm/nixpkgs/archive/ff4fdc8cb369f21776bbaf8686d3e132d8c6ae11.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97
ls -la /nix/store/h3gpn0l86s4mraqbl2l26izkxqs2wis4-truffleHog-2.0.97/bin

cc @bhipple for testing.

Success on aarch64-linux (full log)
Attempted: trufflehog
Partial log (click to expand)
Successfully installed truffleHog-2.0.97
/build/truffleHog-2.0.97
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97
strip is /nix/store/p9akxn2sfy4wkhqdqa3li97pc6jaz3r1-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97/lib  /nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97/bin
patching script interpreter paths in /nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97
checking for references to /build in /nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97...
wrapping `/nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97/bin/trufflehog'...
/nix/store/jhvrskyy9gwgzw02cqa8mrb23h246mvf-truffleHog-2.0.97

Success on x86_64-darwin (full log)
Attempted: trufflehog
Partial log (click to expand)
  The script trufflehog is installed in '/nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97/bin' which is not on PATH.
  Consider adding this directory to PATH or, if you prefer to suppress this warning, use --no-warn-script-location.
Successfully installed truffleHog-2.0.97
/private/tmp/nix-build-truffleHog-2.0.97.drv-0/truffleHog-2.0.97
post-installation fixup
strip is /nix/store/g5r4apl0za012ffs6ladinwa5w0m1l3k-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97/lib  /nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97/bin
patching script interpreter paths in /nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97
wrapping `/nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97/bin/trufflehog'...
/nix/store/9i6iwkjrvn31bxia3x2hwxbhki3pn86f-truffleHog-2.0.97

Success on x86_64-linux (full log)
Attempted: trufflehog
Partial log (click to expand)
Successfully installed truffleHog-2.0.97
/build/truffleHog-2.0.97
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97
strip is /nix/store/vcc4svb8gy29g4pam2zja6llkbcwsyiq-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97/lib  /nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97/bin
patching script interpreter paths in /nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97
checking for references to /build in /nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97...
wrapping `/nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97/bin/trufflehog'...
/nix/store/aaykgb1fziqi77i9z68af3ca5dhhqbl0-truffleHog-2.0.97


Follow up question on TreeSHAP runtime (KDD presentation 2020)

Thanks for the awesome presentation this morning. Our team is interested in applying TreeSHAP to explain LightGBM in production. However we have very strict requirement on runtime latency - down to ms scale per transaction. Of course the runtime depends on the LGBM complexity and computing resource. But is there any benchmark study done on TreeSHAP runtime to give a ballpark idea? Our engine is written in C. Is there any direct plug in from the SHAP package in C if C might be faster than python? Thanks!

Great questions. A couple thoughts:

You can directly integrate with the C++ version of TreeSHAP by importing https://github.com/slundberg/shap/blob/master/shap/tree_shap.h into your code base. It should be self explanatory based on how https://github.com/slundberg/shap/blob/master/shap/_cext.cc uses it. You will need to build a sklearn style representation of the LightGBM model though, and there is not support yet for the "categorical splits" the LightGBM does in tree_shap.h.
There is a path dependent version of TreeSHAP inside LightGBM, I don't recall if it is exposed through the C++ bindings, but it is worth a look.
You can always try the approximate=True flag to make TreeSHAP even faster, and it is a reasonable approximation for large ensembles.
I don't have careful C++ level runtimes. But you run those tests please share them back here :)

Is there an available recording of your hands-on tutorial "Intelligible and Explainable Machine Learning: Best Practices and Practical Challenges" please?
@slundberg

Not sure. We are checking with the organizers.

Hi @slundberg, I'd like to follow up on our earlier conversations:
I did a quick benchmark in python env and the option approximate = True gave significant faster runtime. Using LGBM with 800+ trees, max_depth of 7 and 700+ features, approximate = True & feature_perturbation = interventional took 0.2ms compared to 39ms when approximate = False. We are very interested in moving forward with the approximate option.
(1) I wasn't able to locate the documentation, but how does approximate = True save time? Does it look at a limited number of trees?
(2) I wonder if you have any suggestions on how to save the SHAP explainer and load it later in C++ env for real time processing?
say after we finish this step,
explainer = shap.TreeExplainer(model,df.sample(n=1000),feature_perturbation="interventional")

Hey @TGalaxy!
I am running into similar challenges in my current project. I want to gain insight into a random forest with 500 dependent variables (I know it looks like a bad fit, but it does a great job ;)).
I could also drive the running time down with approximate=True. Though I was concerned by the quality of the approximation. So I benchmarked the differences between the returned values of the TreeExplainer with and without the parameter approximate=True. The short answer is that they are very different. Mainly the approximated version gives middler results, which was already a consequence of the relatively high dimension, but get emphasized with the approximation of the shap values.
Looking at how the shap values are approximated, it makes sense to have something very different. Regardless of this difference, the approximate values gave me good insights into the business value of my model. I could drive conclusions and validate the quality with domain experts. So in this case and as far as I could verify, different didn't mean incorrect.
You can find an answer to your first question in this issue.
I am curious to hear about the real-time processing in a C++ env!


Change Content Type of response

I am trying to figure out how to change Content-Type in headers of routers response. Is there a way maybe to add/change some of them from lets say text/html to text/xml?

$response->header($name, $value);


perf: create manual partition metadata provider

Issue #, if available:
Description of changes:
This change should save ~10MB of heap space at runtime, particularly during a deployment (when memory is the most tight).
Why is this change necessary:
How was this change tested:

[ ] Updated or added new unit tests.
[ ] Updated or added new integration tests.
[ ] Updated or added new end-to-end tests.
[ ] If my code makes a remote network call, it was tested with a proxy.

Any additional information or context required to review the change:
Documentation Checklist:

[ ] Updated the README if applicable.

Compatibility Checklist:

[ ] I confirm that the change is backwards compatible.
[ ] Any modification or deletion of public interfaces does not impact other plugin components.
[ ] For external library version updates, I have reviewed its change logs and Nucleus does not consume
any deprecated method or type.

Refer to Compatibility Guidelines for more information.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Looks like we won't be able to avoid this. Creating any AWS SDK client will trigger the metadata loader code anyway. So we will avoid the call that we're making, but then we're pay the price later on anyway.

AWS SDK issue: https://github.com/aws/aws-sdk-java-v2/issues/3664


Support for declaring an entire document as immutable

No longer is it necessary to add the immutable parameter to each of a document's property validators in order to make the entire document immutable. Now one can simply add the immutable parameter directly to the document definition.
However, it should be noted there are two functional differences between these two approaches:

If immutable: true for all user-defined document properties:

it is possible to delete documents of this type
it is possible to create, modify or delete attachments when a document is replaced (assuming the document type allows attachments), EXCEPT in the case that the document definition also explicitly specifies the _attachments property as immutable: true

If immutable: true at the document root level:

it is NOT possible to delete documents of this type
it is NOT possible to create, modify or delete attachments except when a document of this type is created (assuming the document type allows attachments)

Also updated the prepare-tests.sh script to automatically scan for and make sync functions from document definitions that are defined in the test resources directory rather than having to manually add a new entry for each new file.

💥

Beaut


Command to reload configuration / bot

https://github.com/CraftIRC/4/issues/1

https://github.com/CraftIRC/4Sponge/commit/387453776bce5029e2b855e65322ff47be39cb53


podman compose - broken log/trace to disk when using the supplied build/config_example.toml

We've recently broken logging and tracing to disk when run via podman compose up.
We are NOT writing to the shared directory which is accessible by both the container AND the host.
Below is a snippet of configuration
kai_1               | Config loaded: KaiConfig(log_level='info', file_log_level='debug', log_dir='$pwd/logs', demo_mode=False, trace_enabled=True, gunicorn_workers=8, gunicorn_timeout=3600, gunicorn_bind='<IP_ADDRESS>:8080', incident_store=KaiConfigIncidentStore(solution_detectors=<SolutionDetectorKind.NAIVE: 'naive'>, solution_producers=<SolutionProducerKind.TEXT_ONLY: 'text_only'>, args=KaiConfigIncidentStorePostgreSQLArgs(provider=<KaiConfigIncidentStoreProvider.POSTGRESQL: 'postgresql'>, host='kai_db', database='kai', user='kai', password='dog8code', connection_string=None, solution_detection=<SolutionDetectorKind.NAIVE: 'naive'>)), models=KaiConfigModels(provider='ChatIBMGenAI', args={'model_id': 'mistralai/mixtral-8x7b-instruct-v01', 'parameters': {'max_new_tokens': 2048}}, template=None, llama_header=None, llm_retries=5, llm_retry_delay=10.0), solution_consumers=[<SolutionConsumerKind.DIFF_ONLY: 'diff_only'>, <SolutionConsumerKind.LLM_SUMMARY: 'llm_summary'>])
kai_1               | Console logging for 'kai' is set to level 'INFO'
kai_1               | File logging for 'kai' is set to level 'DEBUG' writing to file: '/kai/logs/kai_server.log'

Note: File logging for 'kai' is set to level 'DEBUG' writing to file: '/kai/logs/kai_server.log'
When running from within the container we need the logging directory to be:  /podman_compose/logs
Which is set as  KAI__LOG_DIR: "/podman_compose/logs" from https://github.com/konveyor/kai/blob/main/compose.yaml
I am using the build/config_example.toml we have for an example
My steps were:

cp build/config_example.toml build/config.toml
Build the latest code into a local image with tag 'local'
TAG="local" podman compose up

Example of the config.toml I used:
$ cat build/config.toml
log_level = "info"
file_log_level = "debug"
log_dir = "$pwd/logs"
demo_mode = false
trace_enabled = true

# **Solution consumers** This controls the strategies the LLM uses to consume
# solutions.
# - "diff_only": consumes only the diff between the the initial and solved
#   states
# - "llm_summary": If the incident store's `solution_producers` field is
#   configured to use "llm_lazy", the solution used will include an
#   llm-generated summary of the changes required.

solution_consumers = ["diff_only", "llm_summary"]

[incident_store]

# **incident_store.solution_detectors** This controls the strategies the
# incident store uses to detect when an incident was solved.
# - "naive": Incidents are considered the same if every field in the incident is
#   the exact same.
# - "line_match": Same as "naive", except we take into account if the line has
#   moved about in the file.

solution_detectors = "naive"

# **incident_store.solution_producers** This controls the strategies the
# incident store uses to produce incidents.
# - "text_only": Only the textual information (diff, code before, code after) of
#   the incident is stored.
# - "llm_lazy": Same as "text_only", but will earmark the solution for LLM
#   summary generation upon retrieval.

solution_producers = "text_only"

# Only set this if you want to use a different incident store than the default.
# If you are running it using podman compose, you should probably leave this
# alone.

# **Postgresql incident store**
# ```
# [incident_store.args]
# provider = "postgresql"
# host = "<IP_ADDRESS>"
# database = "kai"
# user = "kai"
# password = "dog8code"
# ```

# **In-memory sqlite incident store**
# ```
# [incident_store.args]
# provider = "sqlite"
# connection_string = "sqlite:///:memory:"
# ```

[models]
provider = "ChatIBMGenAI"

[models.args]
model_id = "mistralai/mixtral-8x7b-instruct-v01"

# **IBM served granite**
# ```
# [models]
#   provider = "ChatIBMGenAI"

#   [models.args]
#   model_id = "ibm/granite-13b-chat-v2"
# ```

# **IBM served mistral**
# ```
# [models]
#   provider = "ChatIBMGenAI"

#   [models.args]
#   model_id = "mistralai/mixtral-8x7b-instruct-v01"
# ```

# **IBM served codellama**
# ```
# [models]
#   provider = "ChatIBMGenAI"

#   [models.args]
#   model_id = "meta-llama/llama-2-13b-chat"
# ```

# **IBM served llama3**
# ```
#   # Note:  llama3 complains if we use more than 2048 tokens
#   # See:  https://github.com/konveyor-ecosystem/kai/issues/172
# [models]
#   provider = "ChatIBMGenAI"

#   [models.args]
#   model_id = "meta-llama/llama-3-70b-instruct"
#   parameters.max_new_tokens = 2048
# ```

# **Ollama**
# ```
# [models]
#   provider = "ChatOllama"

#   [models.args]
#   model = "mistral"
# ```

# **OpenAI GPT 4**
# ```
# [models]
#   provider = "ChatOpenAI"

#   [models.args]
#   model = "gpt-4"
# ```

# **OpenAI GPT 3.5**
# ```
# [models]
#   provider = "ChatOpenAI"

#   [models.args]
#   model = "gpt-3.5-turbo"
# ```

# **Amazon Bedrock served Anthropic Claude 3.5 Sonnet **
# ```
# [models]
# provider = "ChatBedrock"

# [models.args]
# model_id = "anthropic.claude-3-5-sonnet-20240620-v1:0"
# ```

# **Google Gemini Pro**
# [models]
# provider = "ChatGoogleGenerativeAI"

# [models.args]
# model = "gemini-pro"

Issue was:

I used the example build/config_example.toml we supplied and copied to build/config.toml
The example we supply has the line: log_dir = "$pwd/logs"
We have changed precedence rules so that the config file passed in as command line arguments takes precedence over environment variables.

Example of what I changed to allow writing to disk from the container to the shared location in the host filesystem
$ cat build/config.toml
log_level = "info"
file_log_level = "debug"
#log_dir = "$pwd/logs"
demo_mode = false
trace_enabled = true

I commented out the log_dir entry so that it uses what we supply via compose.yaml (which sets an environment variable)


Fix screengrab to doesn’t exit if the app could not be uninstalled

Checklist

[x] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[x] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary.

Description
Using the new Screengrab flag reinstall_app, Fastlane will uninstall the APKs for each locale configured in the Screengrabfile. Fastlane should not exit if APKs could not be uninstalled, so I remove the code that the runner checks the output of adb uninstall command.
Motivation and Context
At the first run of Screengrab, the APKs may not be installed yet, so the uninstall command will fail.
After the first run, the APK will be installed and the uninstall command will success normally.

Thanks!


Replace LocTags with LocSubjects missed in #351

I added three other small, cleanup commits to this PR.

Thanks! Since you touched the bulk downloads script, just note that the bulk downloads are a huge mess and I suggest we leave that till the very end. We can do the basic DB work so that builds/searches/ebook pages are all working, and release that live to shake it out, then later we can work on bulk downloads last.

Understood, thanks for the guidance.


[bug] Autoloop feature incorrectly flags 'no open channels'

On a mainnet node with 20 active channels the autoloop Tab flags the following, preventing me from setting up autoloops.

Are you trying to set up an Autoloop or do you have channels actively Autolooping? The function of this tab is to filter from the list of all your channels, the subset that is currently Autolooping. To setup Autoloops, you can just drag and drop the slider on a particular channel and then click Autoloop. From there, just follow the UX.

Yes, I am trying to set up an Autoloop, and apologies, I was mistaking this tab for the place to set things up. The slider setup is actually very clear and intuitive in hindsight.


docs: Add a missing link to qrl docs

Overview

What is it?

[ ] Feature / enhancement
[ ] Bug
[x] Docs / tests / types / typos

Description
I used a descriptive link as that's good for screen readers.
Closes #5305.
Use cases and why

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] Added new tests to cover the fix / functionality

Thanks for your commitment 💪


SE-0111 Work in progress: stripping argument labels from function references

What's in this pull request?
This pull request implements the first part of SE-0111, by finding places where we have a reference to a function (or initializer) that is not immediately called, and then stripping the argument labels from the function reference within the type checker. It intentionally does not change the representation of the types of functions/initializers in the type system nor rely on declarations for argument labels.
This implementation is feature-gated behind the compiler flag -suppress-argument-labels-in-types for now.
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    All supported platforms     | @swift-ci Please smoke test and merge
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

A smoke test on macOS does the following:

Builds the compiler incrementally.
Builds the standard library only for macOS. Simulator standard libraries and
device standard libraries are not built.
lldb is not built.
The test and validation-test targets are run only for macOS. The optimized
version of these tests are not run.

A smoke test on Linux does the following:

Builds the compiler incrementally.
Builds the standard library incrementally.
lldb is built incrementally.
The swift test and validation-test targets are run. The optimized version of these
tests are not run.
lldb is tested.

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    All supported platforms     | @swift-ci Please test and merge
    OS X platform               | @swift-ci Please test OS X platform
    OS X platform               | @swift-ci Please benchmark
    Linux platform              | @swift-ci Please test Linux platform

Lint Testing
    Language     | Comment
    ------------ | -------------
    Python       | @swift-ci Please Python lint

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test Linux

The Linux failure is not related to this change.


prompt user to provide a sync schedule

simple change to inform a user that they must provide a sync schedule as well.
based on a suggestion from @holta

Codecov Report

Merging #553 (017c803) into master (3bf8167) will increase coverage by 1.73%.
The diff coverage is 55.97%.

@@            Coverage Diff             @@
##           master     #553      +/-   ##
==========================================
+ Coverage   76.71%   78.44%   +1.73%
==========================================
  Files          45       51       +6
  Lines        2766     3327     +561
==========================================
+ Hits         2122     2610     +488
- Misses        644      717      +73

Impacted Files
Coverage Δ

opwen_email_server/config.py
100.00% <ø> (ø)

opwen_email_client/webapp/views.py
40.75% <21.05%> (-0.84%)
:arrow_down:

opwen_email_client/webapp/tasks.py
43.58% <23.07%> (-3.08%)
:arrow_down:

opwen_email_client/util/network.py
30.00% <30.00%> (ø)

opwen_email_client/webapp/actions.py
34.34% <30.95%> (-0.92%)
:arrow_down:

opwen_email_client/webapp/forms/email.py
34.94% <60.00%> (-0.38%)
:arrow_down:

opwen_email_client/webapp/forms/register.py
60.71% <60.71%> (ø)

opwen_email_server/services/sendgrid.py
91.79% <63.63%> (-2.61%)
:arrow_down:

opwen_email_server/mailers/wikipedia.py
97.77% <97.77%> (ø)

opwen_email_client/webapp/config.py
100.00% <100.00%> (ø)

... and 7 more

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 55c47f8...017c803. Read the comment docs.


feat: move to circleci 2

: 0
https://aws.amazon.com/about-aws/whats-new/2018/08/use-amazon-dynamodb-local-more-easily-with-the-new-docker-image/

@russellballestrini yep - just added a link #18

@dwmkerr your image has been really useful to me and I've got some familiarity with Circle v2. If you'd still like to ship this PR I'd be happy to help contribute to it.

@ryanwilsonperkin I'd love the help! The circleci team have enabled the machine executor for me, so it should just be a case of installing the right software and taking it from there! Any help would be much appreciated 😄

@dwmkerr sorry for my delayed response. I went to look to add this feature tonight but found it might have already been done in #17
Was there some other CircleCI 2 improvement you were looking to ship with this PR?

Completed in #17, so closing now 🚁


chore(comments): remove edit comment from kebab menu

[ ] What does this PR do? [Mandatory]
$title
[ ] What issue/task does this PR references? [Mandatory]
Closes openshiftio/openshift.io#2836
[ ] Are the tests Included? [Mandatory]
N/A
[ ] Is the documentation Included?
N/A
[ ] Are the Release Notes included?
N/A
[ ] Is/Are the @mention(s) to reviewer(s) included
Yes

@debloper Deployment is failing. How do I validate the fix?

Codecov Report

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

@@          Coverage Diff           @@
##           master   #2520   +/-   ##
======================================
  Coverage    9.89%   9.89%
======================================
  Files         193     193
  Lines       12603   12603
  Branches     1781    1781
======================================
  Hits         1247    1247
  Misses      11287   11287
  Partials       69      69

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 cda1f50...dd57497. Read the comment docs.

@debloper Your image is available in the registry: Run docker run -e PROXY_PASS_URL="https://api.free-stg.openshift.com" -p 8080:8080 MISSING_IMAGE.

@debloper LGTM.


Ran of out disk space on C7

I installed Ubuntu (Gnome) on a brand new Acer C7 and it was pretty awesome - got Chromium and Sublime Text 2 installed as well.
I tried installing RVM and halfway through installing a Ruby I got an error saying I've run out of disk space.
Is there a way to bushwhack some more disk space out of this thing, or a way to check if I'm even on the correct disk? I know it only has 16gb of space total but I thought I'd at least be able to install Ubuntu/Gnome/Chromium/Sublime/Ruby.

How do I do a recovery with the usb? I am having the same problem.

@matheusparisotto https://support.google.com/chromebook/answer/1080595?hl=en&source=genius-rts There is also a linux way, just scroll down the page under "Other ways to recover a Chromebook"


Filtering Dates with no Results

Hello!
I'm Using Metabase v0.35.3 and Driver 0.7.1
I have a DATE column and have data on the table like this picture below:

When I filter like this, it doesn't  return values:

The generated SQL query is like this:
SELECT *
FROM `client`.`aggregated_stock`
WHERE (`date` >= parseDateTimeBestEffort('2020-04-26 00:00:00.000Z')
   AND `date` < parseDateTimeBestEffort('2020-04-27 00:00:00.000Z'))
ORDER BY `date` DESC
LIMIT 1048576

0 Results!
This is really a bug, but I think its because of the parseDateTimeBestEffort() function and it turns into a TIMESTAMP and the column is DATE type. BUT the interval, like it's on the query should not to this.
Other filters like BETWEEN doesn't work either.
PREVIOUS,BEFORE,CURRENT, it works!
@enqueue , do you know whats happening?

My solution for now:
I altered all the date columns on click house from Date to DateTime. And I changed on MetaBase metadata the formatting of the columns to show only Dates. That's my way around it today.

Yes, it is a bug. There were some optimizations in Metabase 0.34 and above to reduce the number of explicit casts, because some DBMS could not optimize such queries. I will have to find the right place in the driver to differentiate between Date and DateTime.

I think we might get away with it, because more recent ClickHouse server versions will happily process such a statement. Could you perhaps try and check it out? I just updated my server to <IP_ADDRESS> and did the following:
CREATE TABLE datesversusdatetimes
(
  `foo` Date,
  `bar` DateTime
)
ENGINE = MergeTree
PARTITION BY foo
ORDER BY foo;

INSERT INTO datesversusdatetimes (foo, bar) VALUES
('2020-06-16', '2020-06-16 16:01:01');

SELECT
  foo,
  bar
FROM
  datesversusdatetimes
WHERE
  (foo >= parseDateTimeBestEffort('2020-06-16 00:00:00.000Z'))
AND
  (foo < parseDateTimeBestEffort('2020-06-17 00:00:00.000Z'))
ORDER BY foo DESC;

The result is:
┌────────foo─┬─────────────────bar─┐
│ 2020-06-16 │ 2020-06-16 16:01:01 │
└────────────┴─────────────────────┘

1 rows in set. Elapsed: 0.008 sec.

Or did you have a different use case in mind? If so, please explain.

With the latest changes and newer ClickHouse versions this should no longer be an issue


Added theme_color

Now theme color matches navigation bar color

Oh, this is very helpful! Thank you.


How can download pdf as base64, after user have filled form fields ?

How can i down form as base64 when user have filled it.

Similar question here. I can save the program filled in field but not user filled field.

I think I understand now. There's no way pdf-lib can handle input by an user since the rendering is done by the browser. pdf-lib can create/load, manipulate and deliver to browser, it cannot get any feed back from a browser

Hello @mdhanju and @jumpypixels! What @jumpypixels observed is correct. pdf-lib handles creating and loading PDF files. It's up to you to figure out how to get a PDF file into pdf-lib, as well as what you wish to do after saving it in pdf-lib. pdf-lib is not designed to render PDF files. If you are rendering a PDF in the browser via an iframe (or some other method), pdf-lib has no built-in way to interact with edits the user makes in the iframe. However, if you have some other way of knowing what the user has done to edit the PDF (e.g. by rendering an HTML form instead of a PDF), you could use pdf-lib to map these changes into PDF modifications (see: https://github.com/Hopding/pdf-lib/issues/48).


Does FFI support iOS static library？

as the official doc :
Symbols from a statically linked library can be loaded using DynamicLibrary.executable or DynamicLibrary.process.
but i got the error: Invalid argument(s): Failed to lookup symbol (dlsym(RTLD_DEFAULT, pause_all_task): symbol not found)，
-------------here is my code--------------------------------
import 'dart:ffi' as ffi;
import 'dart:io';
import 'package:ffi/ffi.dart';

// C function signatures
typedef pause_all_task_t = ffi.Void Function();
typedef get_cache_instance = ffi.Pointer<ffi.Void> Function();

// Dart function signatures
typedef PauseALLTask = void Function();
typedef GetCacheInstance = ffi.Pointer<ffi.Void> Function();

// Getting a library that holds needed symbols
ffi.DynamicLibrary _lib = Platform.isAndroid
		? ffi.DynamicLibrary.open('libflash-downloader-lib.so')
		: ffi.DynamicLibrary.process();

// Looking for the functions
final PauseALLTask pauseAllTaskF = _lib
		.lookup<ffi.NativeFunction<pause_all_task_t>>('pause_all_task')
		.asFunction();
final GetCacheInstance getCacheInstanceF = _lib
		.lookup<ffi.NativeFunction<get_cache_instance>>('get_cache_instance')
		.asFunction();

#include "dart_ffi_adapter.h"
//#include "YKCacheCenter.h"
extern "C" {
attribute((visibility("default"))) attribute((used))
void* get_cache_instance(void) {
printf("my_download:调用GetCacheInstance构造单例");
void* p;
return p;
}
attribute((visibility("default"))) attribute((used))
void pause_all_task(void) {
// yk_cache::IYKCache *cache = yk_cache::GetCacheInstance();
// cache->pauseAllTask();
printf("my_download:调用pause_all_task暂停所有下载中的任务");
}
attribute((visibility("default"))) attribute((used))
void ff_pause_all_task(){
printf("my_download:call ff_pause_all_task");
}
}

@JackPanda8
Check out this document for reference and see if it answers your question.
You add the sources to the ios folder, because CocoaPods doesn’t allow including sources above the podspec file, but Gradle allows you to point to the ios folder. It’s not required to use the same sources for both iOS and Android; you may, of course, add Android-specific sources to the android folder and modify CMakeLists.txt appropriately.

See discussion on symbols not being found on https://github.com/flutter/flutter/issues/62666#issuecomment-668377567.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

i meet the same error.
add  use_frameworks!  in pod file  can solve this problem.
but i do not know the reason.


Fix Le parsing for simple APDUs with Lc > 0

In simple APDUs, Le is encoded by replacing 256 with 0.  In the Case 4S
from the spec, a simple APDU with Lc > 0, the command parser added one
to the encoded length before decoding it.  This patch fixes this mistake and
adds a unit test for this case.  (Note that the other cases already
parse the Le correctly so no change is required there.)

Looks good to me. This bug happens in the tests of https://github.com/solokeys/apdu-dispatch/pull/5


Support interactive scripts

WTR's interactive mode invoked via --watch is useful for runtime debugging. When using this in a wireit command, I get an error:
Error: Cannot run watch mode in a non-interactive (TTY) terminal.

The feature of needing interactivity (at least on first run) and needing caching seems orthogonal to me, so that they shouldn't be directly coupled. It may be that for practical reasons (not wanting to script stdin) I have an inline script that needs a TTY to complete, but that doesn't mean I don't want the result cached, if anything it suggests that I might NEED it cached so that it's only the exceptional run (in which an upstream file has changed) that breaks automation and requires interactivity.

Would love to have this capability. There's no way to interrupt vitest or jest in watch mode to add filters.
fwiw we're using a monkey-patched version based on https://github.com/google/wireit/pull/186 which seems to be working fine for our use case.

@aomarks any plans for this? Would be really useful.

Bumped into this today...would be cool to see it added!


Shorter load time & updated git to 2.13.5

Implemented

Resolved #12 (reduced load time) using bin-minify & lambda-bin.
Added stageGit.js to untar git and satge it with bin-minify.
Added config.js to ensure path consistency between stageGit.js and index.js while preventing API breaks and keeping run-time footprint as small as possible.
Updated git from 2.4.3 to 2.13.5 - now source controlling binaries minPack directly as well

Testing

Added eslint config file with rules from existing code, and fixed current code to adhere to this configuration.
Added mocah/chai testing.
Added coveralls.
Updated bats testing file - test case 3 not passing in Nodejs 10 & 11

CI

Updated Travis CI config file to add new Nodejs versions, drop old Nodejs versions, and include coveralls.
Added files section to package.json to avoid publishing the git tar file (and reduce module size).
Added prepublishOnly script to package.json to automate the staging of git binaries prior to npm publish.

Documentation

Created HOWTO.md with details for maintainers on how to update to new git versions and changing the code files.
Added CHANGELOG.md.
Edited README.md to format sample code and add NPM, Travis CI, and Coveralls badges.

@pimterry could you please let me know if you are ok with using gulp due to the issue with filecompare I shared a few weeks back?
I would like to close this PR and cross this off my to do list ;-)

Sorry for the delay @marcelobern, I've been away. Totally agree that layers are a nicer more general solution to this, great to see that! I'll update the readme to link there, and deprecate this package.

Any plans to continue this PR?

@rarkins seems like not, but it needs a reboot anyway. I'm well overdue to take a look at this, especially with Node 10 hitting Lambda this week (which breaks this package).
Gonna close.


Modify estimation ingest to remove assumption that we are reading in from a file

Current RAIL/estimation is coded such that it assumes we are reading from an hdf5 file.  We should separate the read-in and instead have the data ingest as a separate stage and feed Estimator dictionaries for the test and (optional) training data.

I think the scope of this could be generalized to moving i/o to a standalone module shared across creation, estimation, and evaluation. Then we'd guarantee their mutual consistency no matter how many times we refactor the formats (probably unavoidable given XWG dependencies, etc.), and those refactorings would only have to touch one file. Thoughts?

closed by #60


Token streaming

What this PR does
This PR implements token streaming, which aims to reduce perceived latency of the chatbot's responses by allowing responses to be printed out in multiple chunks. Because the generate API is much faster in returning answers with small size,
making multiple calls to it and printing its responses in several chunks feels faster than waiting for the large response to be returned. Also, instead of disabling the whole screen when awaiting response, we only disable the text input box.
How it was tested
One functionality was added to prompt_chatbot.py, named partial_reply() and it is added to the test_prompt_chatbot.py in the same way the reply() functionality was tested. The remaining tests, such as testing if UI behaves appropriately were done manually.
Suggestions are welcome!
PR checklist

[✅ ] No API keys or other secrets committed to source?
[✅] New functionality is added alongside appropriate tests?
[✅ ] All source files have the following header?

# Copyright (c) {YEAR} Cohere Inc. and its affiliates.
#
# Licensed under the MIT License (the "License");
# you may not use this file except in compliance with the License.
#
# You may obtain a copy of the License in the LICENSE file at the top
# level of this repository.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Irem Ergun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add design doc for ChainConfigRegistry

Description
Proposes a ChainConfigRegistry contract to move the chain configuration out of op-program.

Yep going to close this as I think being able to just call a setter for the absolute prestate in the new creator pattern will solve this well enough and we can then avoid the complexity and risks associated with having to hash the rollup json and defining a cross-fault-proof-program compatible way of specifying the full chain config.


Update dependencies for NPM

I'm a bit confused... it seems like some of these are updated to newer versions, but some are lower version numbers?

Unless we manually updated  it doesn't make a lot of sense.
node_modules/dom-serializer is for example down graded

We already have GitHub actions configured for NPM packages?
https://github.com/FOSSBilling/FOSSBilling/blob/main/.github/dependabot.yml

We already have dependabot configured for NPM packages?
https://github.com/FOSSBilling/FOSSBilling/blob/main/.github/dependabot.yml

Why are there so many out of date...
But it is strange...


visual studio 2019 include is broken.

Read these guidelines. They are relatively simple and will allow me to help you better!
For Error Reports:

Produce a simple, short, compilable test case that reproduces your problem.
Make a descriptive title that summarises the bug as a whole.
Explain the bug in as much detail as you can in the body of the issue.
Include Compiler/IDE (Visual Studio, XCode...), Build and Deployment System, Language (C++, Objective-C++), and any special defines you have set.

If you want to request a feature:

Produce any relevant imaginary code that illustrates what you would like or desired behavior.
Include a description and title of what you would like.
Annotate and describe the behavior through comments, asserts or just a small write up.

Thanks for helping sol2 grow!

If you have an issue with the library you should explain the steps you took and the issue you're having, and someone will help you. Coming in here without putting any effort in reading even the things you post and complaining about the hard work someone's put in and is providing to you for free doesn't help anyone.

#define SOL_ALL_SAFETIES_ON 1
#include <sol/sol.hpp>
int main()
{
sol::state _lua;
}
C++ incomplete type is not allowed

Opened this issue (their first ever)

Poor dear.
Good luck elsewhere. (Sorry for the trouble, @OrfeasZ - I do need to update the tutorial section and getting started, but I'm still busy trying to hammer down the Standard C and C/C++ text stuff at the moment! I'm hoping to come back to sol2 when I start my official I Am Not Doing Anymore Work On Standard Things come 2023, which I blocked the whole year off for relaxation / vanity projects, which sol2 is on the list!)


Update specs2-core to 4.12.11

Updates org.specs2:specs2-core from 4.12.9 to 4.12.11.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.specs2", artifactId = "specs2-core" } ]

labels: test-library-update, semver-patch

Codecov Report

Merging #38 (4fb8976) into master (252fb5f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #38   +/-   ##
=======================================
  Coverage   31.78%   31.78%
=======================================
  Files          17       17
  Lines         258      258
  Branches        6        6
=======================================
  Hits           82       82
  Misses        176      176

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 252fb5f...4fb8976. Read the comment docs.


#2116 Front End Authentication Implementation

Issue #: /bcgov/entity#2116
** DRAFT **

Looking for feedback with regards to the extrapolation of the JWT and Auth Methods out of the APP component and this new approach to retrieving data pre-mounting of the app.

Still need to apply the getters.

Description of changes:

Implements an Authorization class which handles the fetching, parsing and storing the keycloak roles as well as the authorization roles.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Cool!
I think my concerns about error handling might go away if you moved the Authentication Service to App.vue. Thoughts?

I definitely agree! I will explore the options a little further, but that is 100% a good option.


TaskQueue: Error with task : TypeError: TypeError: TypeError: Cannot read property 'number' of undefined

Do you get this for for all or some contracts. Sounds like the contact is undefined.


Updates Dev guide

Description
Updates Developer guide to add more explanation to the steps

Documentation
Updating steps

Check List

[x] Commits are signed per the DCO using --signoff

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

Codecov Report

Merging #1590 (41493dc) into main (7592846) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #1590   +/-   ##
=========================================
  Coverage     64.51%   64.52%
  Complexity     3214     3214
=========================================
  Files           247      247
  Lines         17350    17350
  Branches       3073     3073
=========================================
+ Hits          11194    11195    +1
  Misses         4599     4599
+ Partials       1557     1556    -1

Impacted Files
Coverage Δ

...a/org/opensearch/security/tools/SecurityAdmin.java
47.52% <0.00%> (+0.13%)
:arrow_up:

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 7592846...41493dc. Read the comment docs.

@DarshitChanpura  Anything I can do to help get this PR merged?

@DarshitChanpura Could I help getting this PR merged?

@peternied yes..that would be awesome!

@DarshitChanpura I've created a PR with some of this feedback inside it, thanks for waiting for me to get around to this :)


Changed default processing to streaming to CSV

The getReviews.py will now process and stream results to CSV page
by page.  streamReviewsToCSV will also take a
max results to either cap results or go until complete if set to None.
Also, implemented logging, made previewing output as optional
(defaulting to False), and added some extra logging outputs that
will be nice to track process on now longer running jobs.
Closes #11

Hey, I just noticed that some of the CSV names were still around. (I shouldn't review pull requests on a phone. :) )
Not sure if you were going from CSV to Csv, or vice-versa. Since some funcs were defined as Csv, I went that route, and it conforms to Camel casing style (Capitalize Acronyms, unless more than Two characters), not sure if that's true in Python, but that's my story and I'm stickin' to it.


Remove the viewerCssTheme-option, since it's partially broken

The viewerCssTheme-implementation has always been somewhat hacky, and now it's also partially broken ever since we've started using CSS nesting.
Trying to support nested media queries would thus require a lot more parsing of the CSS rules, which seems inefficient and thus generally undesirable.[1]
As discussed on Matrix, let's try to remove the viewerCssTheme-option and see if there's any (significant) fallout from this.

[1] If this option is brought back, it seems to me that it (in Firefox) should probably be set through the platform-code that handles theming.

/botio-linux preview

I'm one day late to the party... but can this functionality be restored? The Chrome extension has historically been using the dark theme, and the viewerCssTheme preference is relied upon by the extension in https://github.com/mozilla/pdf.js/pull/13841

If runtime parsing is a concern, an alternative is to do that at compile time, by extracting the parts with "prefers-color-scheme", and generate a separate stylesheet with only these rules. If the pref is set to anything but the default (forced light/dark), then the viewer code can load the extra stylesheet that unconditionally applies the light/dark theme.

Is there any other way to manually change pdfjs theme? It was very useful option for us.

FYI: The viewerCssTheme option has been restored in #17293, implemented in a more reliable way than before to avoid #17057.

so how can i force dark theme in pdf viewer in version 121.0?
my windows and firefox use light theme.


Automatic retries

Should implement automatic retries just like the official js client.

It already retries in case of ConnectionReset as it is the the error most likely to be temporary.
A general retry on error seems like a bad idea. In my own code if there is any other error it only retries after 1 hour.
Pull request with a plug-able retry strategy is welcome.


[feature request] deno 2.1 supports native .wasm imports

Currently the compiled wasm code is loaded dynamically with an encoded string in lib.js. I believe you can replace that file and the init step in canvaskit.ts with a plain import if you are comfortable releasing a new version that is only compatible with deno 2.1+. I am mainly going off the information I see in this deno blogpost https://deno.com/blog/v2.1. It is possible that there is something more complicated about this module that will prevent this. At the very least though, it should remove an async init stage to canvas usage, which should help simplify usage.

That would be interesting, I believe it would be more complicated than that due to how canvaskit loads the wasm. If there was a way to import as a WebAssembly Module, not an instance, that would make it easier to integrate but this requires quite a bit of rewiring


Git<EMAIL_ADDRESS>Permission Denied (publickey) when I press "synchronize changes" button

VSCode Version: 1.43.2
OS Version: Windows 10 Pro 1909

Steps to Reproduce:

Use windows and cmder and have the default terminal set to cmder's bash term. Then have a password protected ~/.ssh/id_ed25519 ssh key file.

Here is my ~/.ssh/config file although I've tried almost every possible setting for this file already.
Host gitlab.com
HostName gitlab.com
User git
IdentityFile ~/.ssh/id_ed25519

I have been working on fixing this for 2 entire days now. No matter how I set this up it always says "Git<EMAIL_ADDRESS>Permission Denied (publickey)" when I press the synchronize changes button in the lower left corner. I have tried everything including making sure "remote.SSH.showLoginTerminal": is set to "true." and doing almost every possible configuration of ~/.ssh/config. When I run "git pull" or "git push" from the terminal it prompts me for the password to my keyfile and I type the password and it works fine. For some reason whenever I hit that synchronize changes button I get the same error "Git<EMAIL_ADDRESS>Permission Denied (publickey)."

Does this issue occur when all extensions are disabled?: Yes/No

Does this issue occur when all extensions are disabled?: Yes/No

which is it?

Does this issue occur when all extensions are disabled?: Yes/No

which is it?

I just tried disable all extensions and I still get the same error. So "Yes" disabling extensions still gives me the same error.

It looks like a duplicate of #13680 : you can find some workarounds there.
I think the easiest way is to start a ssh-agent in your terminal, then to start code from the same terminal.

#13680


Milestone 1:  Create Enumerable

In this PR, We have completed the current milestone of the project.

no linter errors
the project is professionally documented.
best practices have been utilized using ruby.
We have created a class called MyList with an instance variable @list .
We have created a module called MyEnumerable with functionalities #all, #any, #filter.

Thanks alot @sumairq  for your time spent reviewing our project, we greatly appreciate it.😊😊


add distribute evaluate

add distribute evaluate for all easyrec model.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Provide serviceAccountName to configuration

Describe the problem
We should provide an easy way to manage the RBAC configuration for fluent-bit and fluentd deployment.
Proposed solution
Provide a service account override configuration option to skip the RBAC creation part. Although we should provide usable defaults if not set.

It will help a lot in the context of this operator being deployed on EKS and be able to use the new IAM-Role/ServiceAccount authorisation. No need to create Users and have AWS Credentials in Output.

I just made logging-operator and fluentd work together and will push the changes I made a bit later after tidying them. :smile:
The problems I encountered were :

The service account used by fluentd can't be changed for the moment through the operator

resolved by granting access to the default service account as it runs in a protected namespace

The fluentd process doesn't have read access to the injected token by the aws eks mutatingwebhook (the path to this token is /var/run/secrets/eks.amazonaws.com/serviceaccount/token from within the pod). This is caused by the docker image using fluentd user and the token is read-only by root per default. (see https://github.com/aws/amazon-eks-pod-identity-webhook/issues/8 for more context)

resolved by setting securityGroup: { "fsGroup" : 65533} in the fluentd stateful set, 65533 being fluent user group id.

The fluentd process was then not able to access to region (I don't really know why).

This was fixed by adding AWS_REGION as an environment variable for the fluentd container.

I just made logging-operator and fluentd work together and will push the changes I made a bit later after tidying them. 😄
The problems I encountered were :

The service account used by fluentd can't be changed for the moment through the operator

resolved by granting access to the default service account as it runs in a protected namespace

The fluentd process doesn't have read access to the injected token by the aws eks mutatingwebhook (the path to this token is /var/run/secrets/eks.amazonaws.com/serviceaccount/token from within the pod). This is caused by the docker image using fluentd user and the token is read-only by root per default. (see aws/amazon-eks-pod-identity-webhook#8 for more context)

resolved by setting securityGroup: { "fsGroup" : 65533} in the fluentd stateful set, 65533 being fluent user group id.

The fluentd process was then not able to access to region (I don't really know why).

This was fixed by adding AWS_REGION as an environment variable for the fluentd container.

Thank you for sharing your experiences. I think they are really helpful!

We added some ServiceAccount and PSP enhancement. If you need anything else to support your use-case  please open a new issue.

Hello, is it possible to use AWS EKS IRSA (service account mapping to IAM roles) with logging-operator fluentd ?


Pie charts show for data that classifier was trained on and not new data

We fit a model using a DecisionTreeClassifier on upsampled data. We then wanted to visualise this model using the original data. In the plot all the numbered labels are correct but the ratios in the final pie charts are for the upsampled data.

It would be better to use the same dataset for both training and dtreeviz visualisations.
Internally, dtreeviz is using both the tree metadata and the dataset sent as paramenter.
btw, what ml library are you using ?

Sklearn. Okay that makes sense. I have a highly skewed dataset so I'm training my model on upsampled data, but I then want to visualise the decision tree using the fitted classifier on the original data. Is there a way to do this?On 23 Feb 2023 18:27, Tudor Lapusan @.***> wrote:
It would be better to use the same dataset for both training and dtreeviz visualisations.
Internally, dtreeviz is using both the tree metadata and the dataset sent as paramenter.
btw, what ml library are you using ?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Based on the current dtreeviz implementation, I think no :(
But it's good that you raised this issue, we could take it into consideration as a next possible feature.

That's great thank you!On 25 Feb 2023 10:30, Tudor Lapusan @.***> wrote:
Based on the current dtreeviz implementation, I think no :(
But it's good that you raised this issue, we could take it into consideration as a next possible feature.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I'm running into a similar problem. I trained a decision tree on my train set, but would like to visualize its performance on the test set. When I use the test set as input for dtreeviz.model, the plots are incorrect, the plots show a weird mix of the data I gave it and the data the model was trained on.
As a train test split is a very common procedure, is there no workaround for this?

hi @thomsentner, I put here some limitations : https://github.com/parrt/dtreeviz/issues/269#issuecomment-1442238799
btw, that library are you using ?
I will try to take a look the next days, for sklearn we could have a chance to make it work for new data, I guess.

Thanks. I'm using sklearn as well.

Looking into the code, the change supposed to be small.. but things get complicated a little in case the class_weight parameter is used at model training.
I have to spend more time to better understand the overall picture.

@thomsentner thanks for the PR, I just observed it while creating my PR also for this issue :d
https://github.com/parrt/dtreeviz/pull/282
I will take a look also on yours ;)

@parrt for me it would be to visualize the validation dataset, and as such, visualize the true world performance I can expect from the tree, not so much to test just any specific test case. Looking at training samples will give me a very biased view of what will be happening.

@HannahAlexander your feedback would also help :)

@parrt the plan is to use the tree structure/metadata learned from training set and make the plots based on another dataset, like validation.
As we know, an important step in any ML project is to do a good train/validation (and even test) split, which should reflect the production data. Only interpreting the tree based on train dataset, doesn't mean that the model will perform the same in production also.
For example let's say we have 92% accuracy on train and 80% on validation (or even 99%). The question is why ? Interpreting the tree structure(learned from train) and making visualisations based on validation data should help to get the answers.

Another useful think would be to compare in parallel the visualisations for train and validation and to check for some differences. Ideally the node value distribution/ranges should be the same, but sometime they are not.

@parrt any thoughts on this ? :)

Sorry for the delay. I'm 100% focused on some machine learning stuff at work haha.
Ok, I think I understand the purpose now. You want a mechanism to visualize how the tree structure interprets the validation set in a large sense instead of where we run a single test instance down the tree now.  In other words the tree structure does not change but the distributions in the decision notes and the leaf know it does, according to the information in the validation set. Do I have that correct?

Indeed, it's for the entire validation set and the tree structure (decision split nodes learned during training) doesn't change. In other words, it's how the tree sees/interprete/predict on a new dataset (which is different from training).
It would be a pretty powerful feature I think for the library. No other ones allow this, from what I know :)

I'm a bit nervous about the feature, even though I see the utility and understanding a large validation set. Would this require a lot of changes or add complexity to the code base?

It should be a minimal change in the code. For sklearn is this PR https://github.com/parrt/dtreeviz/pull/282/commits/648c24be56413aff3242534583e4057ea260e652 which as we can see there are just few lines of code.
For the other libraries it should be the same, but I have to double check.
Still, I propose to do this change for sklearn first and to see the community feedback.

Still, I propose to do this change for sklearn first and to see the community feedback

Agreed, let's give it a try and let people report back.

@tlapusan tried working with your PR already and it seems to work great! Just running into one problem still, if both leafs in the test set contain one label, the plotting of the leaf above will fail, as the code tries to plot the opposite class there, but doesn't find any data:

@thomsentner nice that you had time to check it ! thanks
The issue which you mentioned happened to me also and we just merged a PR for it into master few days ago..https://github.com/parrt/dtreeviz/pull/284
@parrt we have https://github.com/parrt/dtreeviz/pull/282 for this issue. If we can merge it, @thomsentner should have the mentioned issue solved.

@tlapusan merged! thanks :)

Resolved by #282

Sorry I've been busy getting up to speed in my new position, but I'm very glad this was implemented - a great addition to dtreeviz!

no prob!  I'm super busy too!