custom dtypes

Hi,
First of all, thanks a lot for your work, this is a really helpful tool to work with npy files!
Now to my issue ;-)
I have a npy file that holds an array with custom dtype, e.g. something like:
my_array = np.empty(20, dtype=[('name', 'S32'), ('value', '<f8')])
my_array['value'] = np.arange(20)
np.save('tmp.npy', my_array)

Trying to load this results into it being treated as 1d-array, where the column contains both names and values.
Do you plan to support such custom dtypes as individual columns as well? Or could you comment on what changes would be required to do so (I could potentially contribute)?
Thanks!

Custom data types are currently not supported.
The only supported data format is 2D lists of datatypes that can be converted to strings.
Although this is an important constraint for NPYViewer, I intended to keep the program as simple and naive as possible at the current stage .
I will possibly provide support for custom data types in the future, but I do not have a specific timeframe.
Changes should be made in the "OpenNPY()" function to support custom data types. Also, it would be good to introduce extra widgets to allow viewing lists within lists, etc.
Additional modifications should take place for the plotting widgets (e.g., view 3D pointcloud), so as to be able to handle custom datatypes.
Feel free to contribute if you have the time.


Changed occ print format to real(6.2)

Please push to the haddocking pdb-tools repo

On 12 May 2017, at 15:30, CunliangGeng<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/JoaoRodrigues/pdb-tools/pull/7 https://github.com/JoaoRodrigues/pdb-tools/pull/7
Commit Summary
Changed occ print format to real(6.2)
File Changes
M pdb_occ.py https://github.com/JoaoRodrigues/pdb-tools/pull/7/files#diff-0 (2)
Patch Links:
https://github.com/JoaoRodrigues/pdb-tools/pull/7.patch https://github.com/JoaoRodrigues/pdb-tools/pull/7.patch
https://github.com/JoaoRodrigues/pdb-tools/pull/7.diff https://github.com/JoaoRodrigues/pdb-tools/pull/7.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/JoaoRodrigues/pdb-tools/pull/7, or mute the thread https://github.com/notifications/unsubscribe-auth/AFx-oLYPIJuOjf6cmhP3ACRnvyiYLsXzks5r5F7_gaJpZM4NZSfn.

==================================
Alexandre Bonvin
Talingweide 91, 3403 AC IJsseltstein, NL
Tel: +31-30-7525883
Email<EMAIL_ADDRESS>

Yes,  done!


Problems

,

Hello, you opened this issue but didn't add any information. Is there something I can help you with regarding that article?


Roll engine 12315736f9ca..513f8fbae726 (2 commits)

https://github.com/flutter/engine/compare/12315736f9ca...513f8fbae726
git log 12315736f9ca..513f8fbae726 --no-merges --oneline
513f8fbae Roll src/third_party/skia 16322637c477..858f648bdae3 (7 commits) (flutter/engine#6240)
9a7d7215a Roll src/third_party/skia<PHONE_NUMBER>3b..16322637c477 (10 commits) (flutter/engine#6238)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

Trybots failed. These were the failed builds: https://flutter-dashboard.appspot.com/build.html

Commit queue failed; closing this roll.


[A-584] Merged with Apache NiFi

Verified locally with the application + flow file template. Looks good!


Error while loading '*.blend'.

Basically as the title says. Whenever I try to create a new inherited scene it gives me this error. I grabbed logs that I think is relevant.
Thanks!
 Cannot open file 'res://.import/Player.blend-5b7cbcd268029d5a74c4a5701a4730d9.scn'.
 Cannot open file 'res://.import/Player.blend-5b7cbcd268029d5a74c4a5701a4730d9.scn'.
 modules/gdscript/gdscript_functions.cpp:774 - Blender import failed with code=127.
Command: /home/paul/Godot/3D RPG/models/player/Player.blend --background --python-expr import bpy; bpy.ops.export_scene.gltf(filepath='/home/paul/Godot/3D RPG/.import/Player.blend-5b7cbcd268029d5a74c4a5701a4730d9.glb',export_format='GLB',export_colors=True,export_all_influences=False,export_extras=True,export_cameras=True,export_lights=True,export_apply=(len(bpy.data.shape_keys)==0))
Output: sh: line 1: : command not found

Same issue

macOS Monterey
ARM-Based processor (Apple Silicon M1 Pro)
Godot 3.4.3
Blender 3.0.1 and 2.93.8

Cannot open file 'res://.import/qbe.blend-52ac478b4083ff3be3a50c42525cfa95.scn'.
Failed loading resource: res://.import/qbe.blend-52ac478b4083ff3be3a50c42525cfa95.scn. Make sure resources have been imported by opening the project in the editor at least once.
Failed loading resource: res://blend/qbe.blend. Make sure resources have been imported by opening the project in the editor at least once.
 editor/editor_data.cpp:531 - Index p_idx = 1 is out of bounds (edited_scene.size() = 1).
 Cannot open file 'res://.import/qbe.blend-52ac478b4083ff3be3a50c42525cfa95.scn'.
Running [/Users/jferdelyi/Documents/Godot/JLP/blend/qbe.blend, -b, --python-expr, import bpy; bpy.ops.export_scene.gltf(filepath='/Users/jferdelyi/Documents/Godot/JLP/blend/qbe.blend.glb')]
 modules/gdscript/gdscript_functions.cpp:774 - Command `[/Users/jferdelyi/Documents/Godot/JLP/blend/qbe.blend, -b, --python-expr, import bpy; bpy.ops.export_scene.gltf(filepath='/Users/jferdelyi/Documents/Godot/JLP/blend/qbe.blend.glb')]` failed with code: 127
 core/os/file_access.cpp:626 - Condition "!f" is true. Continuing.
Cannot open file 'res://.import/qbe.blend-52ac478b4083ff3be3a50c42525cfa95.scn'.

I appreciate the reports!. This now a built-in feature of Godot Engine 4.0. Archiving old issues. Please post bug reports to https://github.com/godotengine/godot/issues.


remove debug printing in tf_idf.rs

I was trying to redirect output from my program to a textfile, instead of saving it to a file myself and these got in the way.

This was my bad. Looks like I forgot to remove some of the debugging output. I reviewed the changes and they checkout, pretty simple, just removed 2 occurrences of println.

Thank you @Ryan1729
Will publish the new version to crates later today.


Router replaces browser history entry if state changes

require engine pr: https://github.com/flutter/engine/pull/26456
fixes: https://github.com/flutter/flutter/issues/80546
Pre-launch Checklist

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[ ] I signed the CLA.
[ ] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Hi @goderbauer this is ready for another look, I added more comments to the doc for this change. I didn't change any functionality in the new commit. I replied to the comments instead, let me know what you think.

@goderbauer  PTAL :)


feat: param changes for block and cost per byte

Closes: #XXX
What is the purpose of the change
Param changes for block size and cost per byte
TODO

make sure cost per byte is what we want

How to test

test with in-place-testnet and
curl http://localhost:1317/cosmos/consensus/v1/params | jq

{
  "params": {
    "block": {
      "max_bytes": "2000000",
      "max_gas": "300000000"
    },
    "evidence": {
      "max_age_num_blocks": "100000",
      "max_age_duration": "172800s",
      "max_bytes": "1048576"
    },
    "validator": {
      "pub_key_types": [
        "ed25519"
      ]
    },
    "version": {
      "app": "0"
    },
    "abci": {
      "vote_extensions_enable_height": "0"
    }
  }
}

osmosisd q auth params --output json | jq

{
  "params": {
    "max_memo_characters": "256",
    "tx_sig_limit": "7",
    "tx_size_cost_per_byte": "30",
    "sig_verify_cost_ed25519": "590",
    "sig_verify_cost_secp256k1": "1000"
  }
}

Nice, where were these values picked from btw?

Blocksize:
The 2mb was based on a discussion after looking at benchmarks more details here => https://linear.app/osmosis/issue/CHAIN-603/update-max-bytes-param-in-consensus-param-to-be-2mb
CostPerByte:
I need to do more investigation here, this was just a hand wavy estimate, will add more details to the PR description today


Move from rust-crypto to ring

This replaces all use of rust-crypto with ring. I think ring is a better choice, because it is better documented, more actively developed and more minimal. Besides, this is a prerequisite for #2.

I think I addressed all the comments.

Awesome. Thanks @vks!


Pass startTime & endTime in Trace Get request

Test would fail on proto marshal
Running tool: /usr/bin/go test -timeout 30s -run ^TestGetTraceTraceIDError$ github.com/jaegertracing/jaeger/cmd/query/app/apiv3

--- FAIL: TestGetTraceTraceIDError (0.00s)
    /home/xxx/projects/go/jaeger/cmd/query/app/apiv3/grpc_handler_test.go:132:
        	Error Trace:	/home/xxx/projects/go/jaeger/cmd/query/app/apiv3/grpc_handler_test.go:132
        	Error:      	Received unexpected error:
        	            	rpc error: code = Internal desc = grpc: error while marshaling: proto: time.Time does not implement Marshal
        	Test:       	TestGetTraceTraceIDError
FAIL
FAIL	github.com/jaegertracing/jaeger/cmd/query/app/apiv3	0.007s
FAIL

Thanks, I see why this test is failing and know how to fix it. However, it raises an interesting question about which data type we should use for the timestamps in the API. In api_v2 we used google.protobuf.Timestamp, so api_v3 was trying to replicate that. However, in api_v2 the Span object also uses google.protobuf.Timestamp, whereas in v3 we are using OpenTelemetry Span type which represents timestamps as fixed int64
  fixed64 start_time_unix_nano = 7;

So while our api_v2 was consistent between data model and the service interface, api_v3 is not consistent.

I see, but unix nano level of timestamp seems to be too granular for API queries.
Current HTTP API uses micro seconds level of timestamp for "loopback". I think we'd better have a very good reason to break it

I agree, I don't really want to change api_v3 as it has been declared stable a long time ago. So we will continue using google.protobuf.Timestamp, we just need to fix some annotations in the IDL and make a change in the Go repository. Specifically, the IDL file was missing these annotations:
// Enable gogoprotobuf extensions (https://github.com/gogo/protobuf/blob/master/extensions.md).
// Enable custom Marshal method.
option (gogoproto.marshaler_all) = true;
// Enable custom Unmarshal method.
option (gogoproto.unmarshaler_all) = true;
// Enable custom Size method (Required by Marshal and Unmarshal).
option (gogoproto.sizer_all) = true;

And then in the Go code I had to add a MarshalToSizedBuffer function to the Traces struct.
Do you want to try to make these changes?

I have a PR https://github.com/jaegertracing/jaeger/pull/6233

Thanks, I see why this test is failing and know how to fix it. However, it raises an interesting question about which data type we should use for the timestamps in the API. In api_v2 we used google.protobuf.Timestamp, so api_v3 was trying to replicate that. However, in api_v2 the Span object also uses google.protobuf.Timestamp, whereas in v3 we are using OpenTelemetry Span type which represents timestamps as fixed int64
  fixed64 start_time_unix_nano = 7;

So while our api_v2 was consistent between data model and the service interface, api_v3 is not consistent.

Just remembered this discussion, it looks like Jaeger has adopted nano level of timestamp in HTTP requests already.
Considering update the time field as type of fixed64 start_time_unix_nano?


⚠ Cioos National CKAN has degraded performance

In 1a19718, Cioos National CKAN (https://catalogue.cioos.ca/) experienced degraded performance:

HTTP code: 200
Response time: 8559 ms

Resolved: Cioos National CKAN performance has improved in b528995 after 10 minutes.


"static" keyword being highlighted in variable names

Basic info

OS: Windows
VSCode version: Latest
Extension version: Latest

Further Information/To Reproduce
Any variables starting with the word static will have the keyword highlighted.

Looks similar to #139, but still occurs even if "editor.semanticHighlighting.enabled" is set to false.
Expected Behavior
static should not be highlighted if part of a variable name.

Fixed, thank you!


Sessions that are closed and reset and all inputs and outputs are out of scope, do not release GPU memory.

Please make sure that this is a bug. As per our GitHub Policy, we only address code/doc bugs, performance issues, feature requests and build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):Yes
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):MacOS 10.13
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
TensorFlow installed from (source or binary):source 1.12.0
TensorFlow version (use command below):
Python version:NA
Bazel version (if compiling from source):0.19.2
GCC/Compiler version (if compiling from source):Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)
CUDA/cuDNN version:10.0.130
GPU model and memory:GTX 1060

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with: 1. TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)" 2. TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"
Describe the current behavior
Memory from tensorflow sessions is not freed when a session is closed and no variables are in scope, memory is only freed on program exit
Describe the expected behavior
Memory should be freed from the GPU on calling close and reset in C++, or some other way to release GPU resources without forking a process.
Code to reproduce the issue
#include <stdlib.h>     /* getenv */
#include <iostream>
#include <tensorflow/core/platform/init_main.h>
#include <tensorflow/core/public/session.h>
#include <tensorflow/core/framework/tensor_shape.h>
#include <string>
#include "cuda_runtime_api.h"
#include <cuda.h>

int freeCudaMem(){
		int gpuCount, i;
		CUresult res;
		CUdevice dev;
		CUcontext ctx;
		int result;
		cuInit(0);
		cuDeviceGetCount(&gpuCount);
		size_t curClockRate = 0;
		size_t curMaxMem = 0;

		for (int i = 0; i < gpuCount; i++) {
			cudaSetDevice(i);
			cudaDeviceProp curDeviceProp;
			cudaError_t err = cudaGetDeviceProperties(&curDeviceProp, i);
			if (err != cudaSuccess) {
				continue;
			}
			size_t free_mem, total_mem;
			cuDeviceGet(&dev, i);
			cuCtxCreate(&ctx, 0, dev);
			res = cuMemGetInfo(&free_mem, &total_mem);
			result= (int)free_mem;
		}
		return result;
}

int main (int argc, char *argv[]) {
	setenv("TF_CPP_MIN_VLOG_LEVEL", "3", 1);
	setenv("TF_CPP_MIN_LOG_LEVEL", "3", 1);

	{
		std::cout << "Start of program:  " << freeCudaMem() << std::endl;
		tensorflow::SessionOptions options;
		tensorflow::ConfigProto &config = options.config;
		config.mutable_gpu_options()->set_allow_growth(true);
		auto* device_count = options.config.mutable_device_count();
		device_count->insert({ "GPU", 1 });
		device_count->insert({ "CPU", 1 });

		std::unique_ptr<tensorflow::Session> session = std::unique_ptr<tensorflow::Session>(tensorflow::NewSession(options));

		tensorflow::GraphDef graph_def;
		std::string graphFile("/path/to/large/frozen_model.pb");
		tensorflow::Status graphLoadedStatus = ReadBinaryProto(tensorflow::Env::Default(),graphFile.c_str(),&graph_def);
		auto inputTensor2 = tensorflow::Tensor(tensorflow::DT_UINT8, { 1, 2048, 2048, 3 });
		std::vector<tensorflow::Tensor> outputs;
		tensorflow::Status session_create_status = session->Create(graph_def);
		tensorflow::Status run_status = session->Run({ { "InputTensor", inputTensor2 } }, { { "OutputTensor" } },{},&outputs);
		outputs.clear();
		int number3;
		std::cout << "After session:     " << freeCudaMem() << std::endl;
		tensorflow::Status closeStatus3  = session	->Close();
		std::cout << "Close Status: " << closeStatus3 << std::endl;
		std::cout << "After close:       " << freeCudaMem() << std::endl;
		session.reset();
		std::cout << "After reset:       " << freeCudaMem() << std::endl;
	}
	{
		std::cout << "Start of program2: " << freeCudaMem() << std::endl;
		tensorflow::SessionOptions options2;
		tensorflow::ConfigProto &config2 = options2.config;
		config2.mutable_gpu_options()->set_allow_growth(true);
		auto* device_count2 = options2.config.mutable_device_count();
		device_count2->insert({ "GPU", 1 });
		device_count2->insert({ "CPU", 1 });

		std::unique_ptr<tensorflow::Session> session2 = std::unique_ptr<tensorflow::Session>(tensorflow::NewSession(options2));

		tensorflow::GraphDef graph_def2;
		std::string graphFile2("/path/to/large/frozen_model.pb");
		tensorflow::Status graphLoadedStatus2 = ReadBinaryProto(tensorflow::Env::Default(),graphFile2.c_str(),&graph_def2);
		auto inputTensor22 = tensorflow::Tensor(tensorflow::DT_UINT8, { 1, 2048, 2048, 3 });
		std::vector<tensorflow::Tensor> outputs2;
		tensorflow::Status session_create_status2 = session2->Create(graph_def2);
		tensorflow::Status run_status2 = session2->Run({ { "InputTensor", inputTensor22 } }, { { "OutputTensor" } },{},&outputs2);
		outputs2.clear();

		int number2;
		std::cout << "After session2:    " << freeCudaMem() << std::endl;
		tensorflow::Status closeStatus2  = session2->Close();
		std::cout << "Close Status2: " << closeStatus2 << std::endl;
		std::cout << "After close2:      " << freeCudaMem() << std::endl;
		session2.reset();
		std::cout << "After reset2:      " << freeCudaMem() << std::endl;
	}
	{
		std::cout << "Start of program3: " << freeCudaMem() << std::endl;
		tensorflow::SessionOptions options3;
		tensorflow::ConfigProto &config3 = options3.config;
		config3.mutable_gpu_options()->set_allow_growth(true);
		auto* device_count3 = options3.config.mutable_device_count();
		device_count3->insert({ "GPU", 1 });
		device_count3->insert({ "CPU", 1 });
		std::unique_ptr<tensorflow::Session> session3 = std::unique_ptr<tensorflow::Session>(tensorflow::NewSession(options3));
		tensorflow::GraphDef graph_def3;
		std::string graphFile3("/path/to/large/frozen_model.pb");
		tensorflow::Status graphLoadedStatus3 = ReadBinaryProto(tensorflow::Env::Default(),graphFile3.c_str(),&graph_def3);
		auto inputTensor32 = tensorflow::Tensor(tensorflow::DT_UINT8, { 1, 2048, 2048, 3 });
		std::vector<tensorflow::Tensor> outputs3;
		tensorflow::Status session_create_status3 = session3->Create(graph_def3);
		tensorflow::Status run_status3 = session3->Run({ { "InputTensor", inputTensor32 } }, { { "OutputTensor" } },{},&outputs3);
		outputs3.clear();
		int number3;
		std::cout << "After session3:    " << freeCudaMem() << std::endl;
		tensorflow::Status closeStatus3  = session3->Close();
		std::cout << "Close Status3: " << closeStatus3 << std::endl;
		std::cout << "After close3:      " << freeCudaMem() << std::endl;
		session3.reset();
		std::cout << "After reset3:      " << freeCudaMem() << std::endl;
	}

}

Other info / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached.
Here is the output from the program above:
Start of program:<PHONE_NUMBER>
After session:     700043264
Close Status: OK
After close:       635219968
After reset:       570400768
Start of program2: 505581568
After session2:    440766464
Close Status2: OK
After close2:      375943168
After reset2:      311111680
Start of program3: 246292480
After session3:    181473280
Close Status3: OK
After close3:      116654080
After reset3:<PHONE_NUMBER>

Note the GPU memory available keeps going down even though the session has been closed and reset and all variables are out of scope.

By default TensorFlow allocates GPU memory for the lifetime of the process, not the lifetime of the session object. More details at: https://www.tensorflow.org/programmers_guide/using_gpu#allowing_gpu_memory_growth
Thus, if you want memory to be freed, you'll have to exit the program, not just close the session.
Hope that helps.

That does not help, the tensorflow library is embedded as a plugin to a parent application.
Either that process will need to fork a subprocess so that application can terminate the entire subprocess.
But at that point I will basically have a client server architecture to a plugin in a parent application, with is not the intended design.
Surely the memory for the session can be managed without shutting down the application.

The request to clear the memory without shutting down the application has been a request from at least two users in the last six months, I cannot deliver on this user story without changing the architecture of the application dramatically, which will be different on Windows, OSX and Linux, because I would be spawning subprocesses so they could terminate just to release memory.

Any update on this?

Any eyes on this?

A response would be great

@samhodge sorry for late reply. As mentioned by @ymodak above, there is currently no way to reset the memory allocator without exiting the program. There is a test-only method that can help to reset the memory owned by the process but that is not exposed.

Sorry for the lack of results reply. It seems odd to be that an API with so many features lacks a tear down function apart from at the operating system level.

Hi @samhodge,
We don't have cycles to investigate this in the near term.  Are you open to contributing a patch to implement this hook?

I would be more inclined to move away from Tensorflow or to have a subprocess architecture to work around the problem but the possibility to fix it is something I might entertain.
Sam

Hi There,
We are checking to see if you still need help on this, as you are using an older version of tensorflow which is officially considered end of life . We recommend that you upgrade to the latest 2.x version and let us know if the issue still persists in newer versions. Please open a new issue for any help you need against 2.x, and we will get you the right help.
This issue will be closed automatically 7 days from now. If you still need help with this issue, please provide us with more information.


Add wall clock sanity check at startup

If clock is before some sane point, say before 2015, fail to start server.  This will prevent and catch whole classes of errors in third party libraries.

@dain -- if it's still relevant, I can take this one.

Yep this is still relevant, so go for it.


embedding Guide Wiki has no pairing quotation mark

in https://github.com/dart-lang/dart-pad/wiki/Embedding-Guide

That's a nice catch. I just updated the wiki.
Thanks for letting us know!


Orval generates wrong import paths in ESM-only package

What are the steps to reproduce this issue?
Use custom Axios instance in ESM-only package (i.e. that uses type: "module" in package.json and "module": "node16", "moduleResolution": "node16" in tsconfig.json).
What happens?
Generated code that imports the custom Axios is:
import {customInstance} from './customInstance';

What were you expecting to happen?
Generated code that imports the custom Axios should be:
import {customInstance} from './customInstance.js';

Any logs, error output, etc?
TypeScript errors: Relative import paths need explicit file extensions in ECMAScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './customInstance.js'? typescript(2835)
What versions are you using?
Package Version: 6.25.0

Hmmm i wonder if there is a way orval can detect what ts-config you are using?  I somehow doubt it.

Maybe an option like useExtensionsInImportPaths: true would suffice? Anyway, extensions are mandatory when using ES modules, so it's not just TypeScript-related bug.

PR is welcome!

@melloware the same is for allowImportingTsExtensions in tsconfig to directly run your ts with node. Now the generated code needs to be:
import type {
    Foo
} from './index.schemas.ts'
import { client } from '../mutator/client.ts'
import type { ErrorType } from '../mutator/client.ts'

If you can give any pointer how this would work great?

@soartec-lab do you have an idea how to get something like this landed?

Well, I think it's a good approach to specify whether to include the extension when importing.
However, there are both .js and .ts cases, so I think they need to be considered.


how to add another layer of data?

I'd need a second table (with lat, long fields of course) to be visible on the map.
It's a good time to start to learn how to develop a plugin for saltcorn :smile:
How to add markers from another table?
I tried to start tweaking this plugin locally
BTW, why run() AND renderRows()? These two are very similar.
Thanks

@glutamate I now add icons to the plugin, and published this new plugin for saltcorn!
https://github.com/j2l/leaflet-map-icon
Please, please, let me know how to add another table and how to fetch data from REST API to populate this second table.
I'm banging my head on walls :)
Thanks

Thanks, I will look at your new plugin!
It's not so easy to add another table, I think the best bet is to create another view that is configured differently:

tableless, so it isn't tied to a table.
configuration uses field repeat where in each repeated section you choose a table, lat and long variables

You can ignore the complexities about runMany and renderRows, they are there to make things more efficient if you have repeated displays of a view. In this case I had some dream about a kanban board with a map in each card, used to track lost kittens, but if you only want one map you can just implement run.
I might be able to implement a multi table map soon

Thank you!

I might be able to implement a multi table map soon

Hooo, great!
A "one map, many tables" solution could be valuable for other people too :)

And what about an action merging (upserting and rewriting IDs) two tables (having the same fields) into one?
This way, the action takes care of getting a final table to work with for any view.
A button to duplicate a table with fields configured might help :)
What do you think @glutamate ?

multitable map implemented in 0.1.5.
You need to create a new view, of the "leaflet map multitable" template, in that you can select multiple tables which each has its own lat/long fields and views
I also stole your icon code (Single table maps only) @j2l


A surface area calculator is now added

Fixes Issue🛠
It fixes issue #750
Description👨💻
A surface area calculator of different shapes has been added which will help us easily calculate surface area of objects like Cube, Cuboid, Cone, Cylinder, Sphere, Prisms, Torus. Also negative length inputs has been avoided using a "Message".

Type of change📄

[x] New feature (non-breaking change which adds functionality)

How this has been tested✅
I have tested the calculator by checking its result using random values and cross verifying the results using officially available calculators in the internet.
Also negative length inputs has been avoided using a "Message".

Checklist✅

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added demonstration in the form of GIF/video file
[x] I am an Open Source Contributor

Screenshots/GIF📷

Please check and merge the PR

Please check the comment on your opened issue and work accordingly @debangi29


Use _jll binaries instead of building from source

It's the system described in https://github.com/JuliaPackaging/BinaryBuilder.jl. I'm trying to set up the binaries for the library right now so hopefully it will be possible to make the change soon.


Problem with sklearn 1.3.0

Hi,
I'm getting an error when using scikit-learn-intelex 2023.2.0 with the latest version of scikit-learn (1.3.0).
  File "PATH\sklearnex\__init__.py", line 18, in <module>
    from . import utils
  File "PATH\sklearnex\utils\__init__.py", line 17, in <module>
    from .validation import _assert_all_finite
  File "PATH\sklearnex\utils\validation.py", line 18, in <module>
    from daal4py.sklearn.utils.validation import _assert_all_finite
  File "PATH\daal4py\sklearn\__init__.py", line 17, in <module>
    from .monkeypatch.dispatcher import enable as patch_sklearn
  File "PATH\daal4py\sklearn\monkeypatch\dispatcher.py", line 21, in <module>
    from ..model_selection import train_test_split
  File "PATH\daal4py\sklearn\model_selection\__init__.py", line 17, in <module>
    from ._split import train_test_split
  File "PATH\daal4py\sklearn\model_selection\_split.py", line 261, in <module>
    train_test_split = validate_params({
TypeError: validate_params() missing 1 required keyword-only argument: 'prefer_skip_nested_validation'

Environment:

OS: Windows 11 and Ubuntu 18.04
The same problem occurs when installing from pip install and from conda-forge

Thanks!

It's expected: 2023.2.0 doesn't contain application of latest sklearn 1.3 change, incoming sklearnex version 2023.2.1 will fix it.

Pinned until 2023.2.1 release.


TileMap Custom Data Can Not Edit Key of Dictionary Type

Tested versions

Reproducable in Godotv4.2.1.stable.official [b09f793f5]

System information
Godot v4.2.1.stable - Windows 10.0.22631 - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3060 (NVIDIA; <IP_ADDRESS>52) - AMD Ryzen 9 5950X 16-Core Processor (32 Threads)
Issue description
When working with TileMaps, adding a custom data layer to its tile set of the dictionary type prevents editing the key entry after setting the key type and the value type and then clicking "Add Key/Value Pair". This makes it difficult to work with, and differs from typical usage in scripts when the @export decorator is used.
Steps to reproduce

Create a new scene.
Add a new node of the type TileMap.
Add a new tile set by clicking on the tilemap's tile set field and clicking "New TileSet".
Click on the created tileset in the tilemap's tile set field to edit its properties.
Open the "Custom Data Layers" field and click "Add Element".
Give the element a name and give it the type of "Dictionary".
In the tiles view, click add and add a new tile map, I used: Godot TileSet Tutorial.
Set the Tile Size field of the tile set to 64x64 to ensure there are tiles to click on in the Tiles view.
Click one of the tiles in the Tiles view and open the Custom Data dropdown.
Set the type of the key to any type, and the type of the value to any type.
Click "Add Key/Value Pair", you can now only edit the value type and value.

Minimal reproduction project (MRP)
N/A

Please try to provide a minimal reproduction project. You instructions seem to be lengthy to follow and it would help others to reliably reproduce your problem.

Bug.zip
Here is a minimum project sample. The scene to look at is named "BugScene". To view the bug in question, click on the tilemap, open the TileSet view on the bottom pane, and in select mode select the first tile in the godot icon. The key for the dictionary of the custom data value can not be modified. This differs from creating a variable of the Dictionary type, and forces the user to delete the entry in sequence to modify the values.


Dark Theme

Edit style.css to have a dark theme!

I'll take this, can I? Also, do you have some specific color?
Thanks.

All you. Whatever you think looks good!


Trouble compiling or handling test data for 3 transformer based models

gpt2-10, bidaf-9, and inception-v2-6 are all having trouble compiling/handling test data. In particular, gpt2-10 and inception-v2-6 are currently failing for unknown reasons, meanwhile the test data for bidaf-9 is not currently supported for our test harness. I'm willing to help with these in the mid term once I find time.

The link has been updated with a direct link to the onnx models. I'll handle bidaf-9 once I find time unless someone else gets to it first as I would also like to refactor some of the test harness code itself at the same time.

gpt2-10, bidaf-9, and inception-v2-6 are all having trouble compiling/handling test data. In particular, gpt2-10 and inception-v2-6 are currently failing for unknown reasons, meanwhile the test data for bidaf-9 is not currently supported for our test harness. I'm willing to help with these in the mid term once I find time.

Can you provide more details about the failure?

I did hit a similar issue and wasn't necessarily intending to address this in detail right now. Regardless, though, I apologize for not providing much detail as I didn't have much to truly go on. GPT2-10 exits immediately when you try to compile it with nothing to review,  inceptionv2-6 will core dump, and bidaf9 just needs some test data changed around which I'm planning to do. That's really all I've got but the current results can be seen in the onnx-mlir through the onnx-mlir status badge on the main page of the repo.

I looked into the output of onnx-mlir --EmitONNXIR --mlir-print-ir-after-all. I do not quite understand what happened. I just put the all the pass header print here. Obviously, the last pass SymbolDCEPass was not executed.

FYI, these failures also reported by our Jenkins here: https://www.onnxmlir.xyz/jenkinx/job/ONNX-MLIR-Pipeline-Docker-Build/Model_20Zoo_20Report/

bidaf-9 failed because of unsupported Scan version as I heard from @negiyas.

too old (deprecated) or too new?

too old (deprecated) or too new?

It uses an old opset (opset 8) for Scan while onnx-mlir supports opset 16 of Scan that is not backward compatible: https://github.com/onnx/models/issues/540

@messerb5467 is it possible to get a similar model with a newer opset? opset 8 is not really supported.

@messerb5467 please let us know if the current error message is clear enough when provided a model with this older opset.
If it is seen as key to our users to support this older version, let us know. But we would prefer focusing on future ops.

will do. I've not tried to compile this one myself and more saw it through the lense of it's corresponding data failing to convert. Once I get the time, I'll check it out and provide that feedback.

I just tried to compile this for the different accelerator/environment combinations and it all boils down to the same issue:
error: Invalid axis value
error: shape inference failed

Within reason, that won't be enough to help me diagnose the issue that we're seeing. What would help me is the normal tenets of good First Failure Data Capture (FFDC) and help give me direction on whether I would need to do more debugging on my own or if I should open an issue. While I do know that there are lots of good documentation within the project, I usually like to keep just the right amount of information in the error message and build the trail of cookie crumbs out to better places to look. Personally I'm still finding the right balance between project README.md's and longer form comments in documentation, so I'm open to artistic influence of what that should look like.

This just got bumped up on my plate. Just a small amount of time to do one other thing and I'll get back to this for full-time focus.

I just updated bidaf-9 available, but can't attach it since it is too large. Let me know who needs it and I can forward it over. Within the company makes sense as it is very straightforward with our own solutions, but cloud storage even for something more project centric if pretty cheap would be appreciated. I pay $2 a month for 100GB on drive for instance, but don't know how that would scale to enterprise users of an open source project.

Thanks for the eagerness here everyone. I worked through bidaf-9 with @chentong319 and one of the squeeze ops isn't compatible with onnx-mlir.

This patch # https://github.com/onnx/onnx-mlir/pull/1854 is to fix the GPT2-10 compilation issue.

I've confirmed gpt2-10 on my side. Thanks for the great work!

Retested inception-v2-6 today and it now passes. bidaf-9 isn't yet supported by onnx-mlir so I'm moving this to closed.


SPA with angular library and [parentScroll]="scroll.window"

<virtual-scroller [items]="visibleSites" #scroll style="height: 100vh; overflow-x: hidden !important" [parentScroll]="scroll.window">

         <dc-site-card *ngFor="let site of scroll.viewPortItems; trackBy: trackByFn" [site]="site" class="col-sm-12 col-md-6" [ngClass]="getColClass()"></dc-site-card>

   </virtual-scroller>
So I am having an issue with multiple vertical scrolls showing up, the component that has the data is in a library and loaded in a SPA application.  When I add the scroll.window it does not scroll through the data, it only shows the initial amount of cards.  When I remove the parentScroll add the data is loaded properly however there are 2 vertical scrolls.
Love this component, but I am stuck on this for too long, any help would be greatly appreciated.  Just not sure why it does not scroll through the data with the parentscroll set.

<virtual-scroller [items]="visibleSites" #scroll style="height: 100vh; overflow-x: hidden !important" [parentScroll]="scroll.window">

 <dc-site-card *ngFor="let site of scroll.viewPortItems; trackBy: trackByFn" [site]="site" class="col-sm-12 col-md-6" [ngClass]="getColClass()"></dc-site-card>

 </virtual-scroller>
So I am having an issue with multiple vertical scrolls showing up, the component that has the data is in a library and loaded in a SPA application. When I add the scroll.window it does not scroll through the data, it only shows the initial amount of cards. When I remove the parentScroll add the data is loaded properly however there are 2 vertical scrolls.
Love this component, but I am stuck on this for too long, any help would be greatly appreciated. Just not sure why it does not scroll through the data with the parentscroll set.

randym127 – did you solve this issue? I have the same problem

<virtual-scroller [items]="visibleSites" #scroll style="height: 100vh; overflow-x: hidden !important" [parentScroll]="scroll.window">

 <dc-site-card *ngFor="let site of scroll.viewPortItems; trackBy: trackByFn" [site]="site" class="col-sm-12 col-md-6" [ngClass]="getColClass()"></dc-site-card>

 </virtual-scroller>
So I am having an issue with multiple vertical scrolls showing up, the component that has the data is in a library and loaded in a SPA application. When I add the scroll.window it does not scroll through the data, it only shows the initial amount of cards. When I remove the parentScroll add the data is loaded properly however there are 2 vertical scrolls.
Love this component, but I am stuck on this for too long, any help would be greatly appreciated. Just not sure why it does not scroll through the data with the parentscroll set.

randym127 – did you solve this issue? I have the same problem

I don't have a solution sorry, we ended up pulling this component all together.

@randym127 Can I ask which virtual scroll component you ultimately went with?

@randym127 Can I ask which virtual scroll component you ultimately went with?

We didn't use virtual scrolling at all, we went with paging.

I've switched to ngx-virtual-scroller as you can define the window as the scroll target.
https://github.com/rintoj/ngx-virtual-scroller

Make sure it's the window that has the scrollbar and not the body.
I had the body scrolling and it wouldn't render new items.
After removing:
body {
    overflow-y: scroll;
}

... it worked fine. You might need to remove overflow-y: scroll; from other parent elements too.


Feature/scan bluetooh device

rename scanDevice to scanDevices

Fixed in another commit (https://github.com/leesiongchan/react-native-esc-pos/commit/a58c650d3fed86d94fe96c94696239fa3af5c6c8)


Multiqc uses information from metrics_summary

Multiqc doesn't use all information from cellranger, only from cellranger count and vdj.
In order to have more informative multiqc report I am going to create a cellranger parser, that fetches information from the cellranger metrics_summary.csv and creates a table.
Can be closed when:

[ ] Module for parsing cellranger metric_summary
[ ] Added module to all workflows and properly tested
[ ] Pushed changes into dev branch

Done!


Does not match new TLDs

The .directory TLD is not matched. Because of this regex
[a-z]{2,4}

yeah .. this is a super nightmare for this kind of case. i guess its necessary to work with an explicit list of TLDs because just allowing more characters after a dot is going to lead to a lot more false positives.
that being said, this Bundle should adopt a general purpose library (see the open PRs) rather than trying to invent one on its own.

Will #14 be fixed/merged?

+1, all the new TLDs are not matched. Creating a list is almost impossible, there're new TLDs added every months and there's support for non-ascii characters (e.g. https://www.eurodns.com/international-domain-names/xn--3ds443g-domain-registration/, https://www.eurodns.com/international-domain-names/xn--ngbc5azd-domain-registration/ and https://www.eurodns.com/international-domain-names/xn--i1b6b1a6a2e-domain-registration/)

we are not really maintaining this library anymore because nobody at liip is currently using it. it is not up to date with recent symfony versions, not tested on php 7.1 or newer and so on. the travis-ci build fails for the antique php versions and obsolete symfony versions.
in our opinion, the evolution should be to use an existing library for finding urls in text, or splitting out the logic implemented here into a library and have the bundle only integrate the library into symfony.
we archive this bundle now.


Connect airflow to postgres database

Description
To connect airflow ci to the postgres database, UUID's of successful runs are written into one of the database tables.
Fixes
This is required for the data standardization/data science/data visualization work.

Codecov Report

Merging #220 (557610c) into master (84bd992) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #220      +/-   ##
==========================================
+ Coverage   93.32%   93.33%   +0.01%
==========================================
  Files          24       24
  Lines        1004     1006       +2
==========================================
+ Hits          937      939       +2
  Misses         67       67

Flag
Coverage Δ

gha
93.33% <100.00%> (+0.01%)
:arrow_up:

python-3.9
93.33% <100.00%> (+0.01%)
:arrow_up:

unit
93.33% <100.00%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

dags/openshift_nightlies/tasks/benchmarks/e2e.py
92.85% <100.00%> (+0.10%)
:arrow_up:

...t_nightlies/tasks/install/hypershift/hypershift.py
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


Get qna result data from Bot Composer - QnAMakerDialog.cs

I am using QnaMaker dialog in Bot Composer.
In my dialog flow, I need to have an if branch after QnAMaker is returned either an answer or fallback message to the user.
Depending on the response, I will send a message to the user.
I found out that qnaresult is stored into the turn memory variable. But the memory variable has hashcode which I cannot find how to generate that code.
[Ln427] dc.State.SetValue($"turn.qnaresult{this.GetHashCode()}", response);
QnAMakerDialog.cs
I checked all the source code of Bot Builder and couldn't find any usage case of hashcode in memory variables. It is always stored without that.
My question is; Is there a specific reason to have hashcode in the turn.qnaresult variable?
Second question, How can I access  to qna result data from Bot Composer ?
Thanks,
Şebnem

Thank you for the questions, @sebnemars
The answer to your first question can be found in the comment above the line of code you posted:

we use hashcode so we don't conflict with any other qnamakerdialogs out there

To answer your second question, I recommend using the "QnA Intent recognized" trigger instead of a QnAMakerDialog. You can find out how to do that using Composer's QnA Sample. Have you tried the QnA Sample yet?

@sebnemars - Are you still working on this?

@v-kydela , Thanks for the answer. We had used "QnA Intent recognized" trigger before. But in our QNA user journey, the bot continues to the flow by checking the Qna score then calls other KB or asks to user to rephrase the question. Is it possible to add a memory variable to "QnAMakerDialog" as you do in "QnA Intent recognized"?

@sebnemars - Sorry, that's not possible, but feel free to post that as a feature request. If you have any further questions, you can ask them on Stack Overflow and my team will see them. I will close this issue now.


clarify service account token expiry

The wording "or the pod is deleted" is unclear. I can find two meanings in it:

or whenever the pod is deleted
or
otherwise, the pod is deleted

The first one makes more sense, hence correcting to "or whenever the pod is deleted" to clarify.

@axelsimon
Pls sign the CLA in order for the PR to be reviewed.
more info
https://github.com/kubernetes/community/blob/master/CLA.md

/check-cla

👋 @axelsimon .
Have you signed the CLA?

/close


Add template for tsconfig.json

Description
Added a template for tsconfig.json file to the project templates.
Changes

Added tsconfig.json template file, removed tsconfig.json

Testing

[x] Tested template generation with new project creation

To-Do

[ ] Update TEMPLATE examples with the new tsconfig file

Hey port can you share the repo(extension) where you have the .args.mjs?

Could you please take a look at this table? We added it recently. Looks like you can use an raw object instead of a stringified and maybe simplify some logic

Hey thanks for the comments, and I somehow did not receive a notif about Shiv's comment.
So here is the repo:
https://github.com/portdeveloper/test-tsconfig-ext
Maybe we should remove include, exclude and paths and keep the other two?
I tried using stringify but that made the process so complicated :/

Thanks! Could you please check again to see if i got it right? @rin-st
I have also updated the example args file

Thanks @portdeveloper ! Added some comments.
Also, you can test resulting template for example with this defaults
export default withDefaults(contents, {
  extraPlugins: [{name: "next-abcd"}],
  extraCompilerOptions: {abc: "def"}
})

@technophile-04
We need this for serwist to work!

And there doesn't seem to be anything an extension-builder can add besides more plugins and more compiler options?
I have updated https://github.com/portdeveloper/test-tsconfig-ext/blob/main/extension/packages/nextjs/tsconfig.json.args.mjs too

And there doesn't seem to be anything an extension-builder can add besides more plugins and more compiler options?

yeah true lol there not other options to add, also for pwa do you need any other more template file like manfiest.json or it was just this file?

@technophile-04
We also need these:

[ ]  manifest.json template file PR
[ ]  getMetadata template file PR
[ ]  Change the nextjs template so that we can put things outside of the nextConfig object as in the code below:
https://github.com/portdeveloper/scaffold-pwa/blob/main/packages/nextjs/next.config.mjs

Should I get the manifest.json and getMetadata in this PR as well?

Yes could you do that please?

I think we need to update main SE-2 repo with next.config.js => next.config.mjs then we make the PR here to add templatise that file

Thanks @portdeveloper!!

I think that won't be necessary! Because I am using the next.config.js version in this template!
Let me quickly update that as well.

I think that won't be necessary! Because I am using the next.config.js version in this template!

Ohh @portdeveloper, I think we should switch the default SE-2 next config => .mjs because its nice, easy to configure and clean. I have been wanting to do that from past!

The latest nextjs uses a next.config.ts.

I wonder if that's also an option for us as well?

Ohh I see! I think last time I tried using .ts in SE-2 and it gave me weird error but might be fixed and we should use .ts if it works now 🙌 Lets tinker on SE-2 main repo
PS: I am replying to my own comment because Port replied to og comment by editing it 🤣

The latest nextjs uses a next.config.ts.

I wonder if that's also an option for us as well?

Ohh I see! I think last time I tried using .ts in SE-2 and it gave me weird error but might be fixed and we should use .ts if it works now 🙌 Lets tinker on SE-2 main repo
PS: I am replying to my own comment because Port replied to og comment by editing it 🤣

I am never gonna work under the sun again :(((((( i was like why does it say technophile there this should be a bug lmaooo

Hey just added a commit for the manifest.json file, and hopefully I am not editing anyone's message hehe.
args.mjs i used to test things:
https://github.com/portdeveloper/manifest-test/blob/master/extension/packages/nextjs/public/manifest.json.args.mjs
Had to make iconPath optional as that is not needed in the pwa template.

Honestly, I don't understand how to templatize getMetadata.ts.
I did it in a way but most probably not in a good way:
`import { stringify, withDefaults } from '../../../../../utils.js'
const contents = ({ titleTemplate, extraIcons, extraMetadata }) => `
import type { Metadata } from "next";
const baseUrl = process.env.VERCEL_PROJECT_PRODUCTION_URL
? `https://${process.env.VERCEL_PROJECT_PRODUCTION_URL}`
: `http://localhost:${process.env.PORT || 3000}`;
const titleTemplate = "${titleTemplate[0] || '%s | Scaffold-ETH 2'}";
export const getMetadata = ({
title,
description,
imageRelativePath = "/thumbnail.jpg",
}: {
title: string;
description: string;
imageRelativePath?: string;
}): Metadata => {
const imageUrl = `${baseUrl}${imageRelativePath}`;
return {
metadataBase: new URL(baseUrl),
title: {
default: title,
template: titleTemplate,
},
description: description,
openGraph: {
title: {
default: title,
template: titleTemplate,
},
description: description,
images: [
{
url: imageUrl,
},
],
},
twitter: {
title: {
default: title,
template: titleTemplate,
},
description: description,
images: [imageUrl],
},
icons: {
icon: [{ url: "/favicon.png", sizes: "32x32", type: "image/png" }],
${extraIcons[0] ? Object.entries(extraIcons[0]).map(([key, value]) => ${key}: ${JSON.stringify(value)}).join(',\n      ') : ''}
},
${extraMetadata[0] ? Object.entries(extraMetadata[0]).map(([key, value]) => ${key}: ${JSON.stringify(value)}).join(',\n    ') : ''}
};
};
`
export default withDefaults(contents, {
titleTemplate: '',
extraIcons: {},
extraMetadata: {}
})
`


Add reset action in settings

In the Settings page, one can sometimes accidentally use the "select all" or "clear all" action and thus lose the complete list of his favorites.
It would be nice to have a reset action getting displayed in the menu once there is some modication done in the list. This way such mistakes can be easily reverted without having to go back through the whole list and selecting again each favorite comic.

We can make settings page as dialog and doesn't save selected comics until Accept dialog action will chose.
What do you think about this?

Your idea sounds to be a better solution to the problem I described in this issue :)
I don't recall if there was any reason I made the settings page the way it is, without using a dialog.
So I guess, you can go for it


Cannot remove compat layer from session

Using Remove-Module  CompatPowerShellGet does not disable the compatibility mapping features for the PS session. Attempts to run Install-Module after removing the compat module still fails with deprecated warnings.
Recent PS blog post mentions the ability to temporarily bypass the compatibility features of this module.

If you have this compatibility module installed and would not like it to be used, you can remove it from the PowerShell session using the Remove-Module command.

Steps to reproduce:

Latest versions of both PowerShellGet and CompatPowerShellGet are installed & available
>  Install-Module -Name Pansies -AllowClobber (Output: "WARNING: The cmdlet 'Install-Module' is deprecated, please use 'Install-PSResource'.")
> Remove-Module -Name CompatPowerShellGet
>  Install-Module -Name Pansies -AllowClobber (Same output warning, no operation performed)

Good point!
As PowerShellGet V3 has a few issues right now installing quite a lot famous Microsoft 365 PS modules (Exchange Online, MS Teams, PnP PowerShell, …), it is not possible to use CompatPowerShellGet at the moment. This is due to the fact that to install these modules, one still needs to use the Install-Module command from PowerShellGet V2 until a fix for this was introduced (hopefully as part of v3.0.14-beta, but there is v3.0.13-beta first…).
There is a workaround described here:

Please also note that this compatibility module will not be called if you use fully qualified cmdlets. For example, if you use PowerShellGet\Install-Module this will call a legacy version of PowerShellGet.

However, this workaround does not seem to work. Not sure this is a bug or something I'm doing wrong.
Any hints or tipps what needs to be done, besides not installing CompatPowerShellGet?


Does Amazon Timestream support User Defined Function?

I am currently working on a benchmark of time series database systems for analytical (complex) queries. These queries are algorithms that cannot (easily) be implemented in SQL language so they require a third-party programming language such as C, C++, Python, Java, etc.
I am writing to you to ask if Amazon Timestream supports such a feature of a third-party programming language for User Defined Functions?Does GridDB support User Defined Function

Hi @AbdelouahabKhelifati, thank you for your interest in Amazon Timestream. Unfortunately, Timestream does not support user-defined functions.
We do support many query operators and functions including time series specific functions such as interpolation, integration/derivation, correlation, and filter/reduce. With these functions, hopefully it is easier to implement your queries than you anticipated.

@AbdelouahabKhelifati  - Can you please submit your feature requests via AWS Support - https://console.aws.amazon.com/support/home#/case/create?issueType=technical


Another readme update

Codecov Report

Merging #345 (d627c9a) into main (004d92a) will decrease coverage by 0.08%.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #345      +/-   ##
==========================================
- Coverage   41.04%   40.97%   -0.08%
==========================================
  Files         117      117
  Lines        9657     9653       -4
==========================================
- Hits         3964     3955       -9
- Misses       5693     5698       +5

see 10 files with indirect coverage changes


Conditional entry point

Is there a away to create a conditional entry point? In Python this is a function, in TS it seems that it don't exist.

You can add conditional edges from the "__start__" node:
builder.addConditionalEdges("__start__", condition, conditionalEdgeMapping)

@hinthornw wow, great answer! Didn't know this one exists. Documentation is not yet complete. But I love LangGraph. Thanks for the help!


Set --set-infra-only=true flag in hostMonitoring mode

For me it sounds that there is some misunderstanding what is really need needed to fix #1289. Especially based on comment https://github.com/Dynatrace/dynatrace-operator/pull/1304#issuecomment-1309906205 There is no anything which needs decission but this operator need set OneAgent parameters like it is documented. Same way like it is done when deploying OneAgent without this operator to standalone Docker nodes, etc. So I decided to create pull request about.
Description
Set --set-infra-only=true flag in hostMonitoring mode to correctly consume host units like they are explained in "Infrastructure" column in this table https://www.dynatrace.com/support/help/monitoring-consumption/application-and-infrastructure-monitoring#host-units
Fixes: #1289
How can this be tested?
After this change it is visible in Dynatrace portal that host "Monitoring mode" is Infrastructure only instead of Full stack

Checklist

[ ] Unit tests have been updated/added
[ ] PR is labeled accordingly
[x] I have read and understood the contribution guidelines

Hi @olljanat, thanks for the PR. However the operator team understood clearly what you wanted to address in your issue and comments.Nevertheless the licensing topic for host-monitoring is currently being discussed internally. This means for now the operator works as expected and does not set the --set-infra-only=true flag. So this works as expected and as designed. However there may be changes to this in the future, depending on the outcome of different discussion not related to the operator itself. Whenever there are changes to this behaviour our team will take care to make the necessary changes and change it to --set-infra-only=true.
Thanks!
Lukas

@StevenCane unfortunately there looks to be some weird policy inside of Dynatrace, etc which prevents this to be merged. So I recommend you to create ticket for them.
In mean time you also use my custom image ollijanatuinen/dynatrace-operator:fix-infra-mode in https://github.com/Dynatrace/dynatrace-operator/blob/69009ef8db7f4c31fea2febc55ed632647a31df4/config/helm/chart/default/templates/Common/operator/deployment-operator.yaml#L56 (probably it can be also provided with values file but I didn't tested that) which have this modification included.


[KOGITO-4697] - Fix Native compilation on OpenApi Codegen for Quarkus…

… runtime
See: https://issues.redhat.com/browse/KOGITO-4697
The RestClientBuilder originally used by the OpenAPI generator breaks the native compilation on the httpclient package. For Quarkus applications, we borrow the MicroProfile RestClientBuilder to create our clients. It's not ideal since it's meant to be used for interfaces registered with RestEasy.
I opened KOGITO-4701 to consider moving our code generation to MicroProfile Rest Client instead. It can be tricky and bring problems to SpringBoot applications. I considered this workaround for now to unblock the native feature.
The ApiClient.mustache is a copy of the original template provided by the OpenAPI Tool Generator. I'd rather extend the generated class instead of modifying their template, but unfortunately, the buildHttpClient function is private, so we can't overload it. I would talk to the OpenAPI generator community to consider a CDI "friendly" approach for these classes, but since we are considering moving to Microprofile, I'll leave this conversation for now.
The culprit of the native compilation was the Configuration.mustache template, which used to generate a class with a private static field forcing the initialization of the ApiClient. This is another thing to consider open a PR for them if we do not move to Microprofile.
The commons-logging library was clashing with natively compiled code since we use jboss-logging, which also has the same interfaces and implementations. For the SW SDK, I'll remove it from the SDK pom file.
Signed-off-by: Ricardo Zanini<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Your code is properly formatted according to this configuration
[x] Pull Request title is properly formatted: KOGITO-XYZ Subject
[x] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains link to any dependent or related Pull Request
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Pull Request
Please add comment: Jenkins retest this

Quarkus LTS checks
Please add comment: Jenkins run LTS

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

The Linux check is successful.

The Linux check is successful.


Fix Venafi conformance test

Follow-up PR for https://github.com/cert-manager/cert-manager/pull/7110, adding missing required feature.
Fixes periodic CI job: https://prow.infra.cert-manager.io/view/gs/cert-manager-prow-artifacts/logs/ci-cert-manager-master-e2e-v1-30-issuers-venafi/1805033541822582784
Kind
/kind bug
Release Note
NONE

/test ?

/test pull-cert-manager-master-e2e-v1-30-issuers-venafi-tpp

/approve


Add zbirenbaum/copilot-cmp

Checklist:

[x] The plugin is specifically built for Neovim.
[x] The lines end with a .. This is to conform to awesome-list linting and requirements.
[x] It's not already on the list.
[ ] If it's a colorscheme, it supports treesitter syntax.
[x] The title of the pull request is Add `username/repo`  when adding a new plugin.

Thanks for the PR!
Something went wrong with the syntax of your line, when you fix that I can merge your PR. See the failing CI.


Add datadog APM instrumentation

This instruments CA for Datadog APM and adds tracing support to a few of the main functions.
Followed DD's go instrumentation docs initially among other web services.
The purpose of this is to get better visibility into what's slowing down in CA during periods of heavy scaling activity. We do have various duration metrics. being generated which give a broad picture on what might be slowing down but we want more details. To start, we traced CA operations that map to the metrics in the link above, as defined in metrics.go (minus a few legacy and seemingly unused ones).
Traces are coming in, the separate spans are visible in the span list tab for each trace.
We realized while doing this that we may actually just need profiling for now, rather than adding spans for every single call/function. We are working on that separately, and I believe you can have traces link to profile code blocks so it may be helpful regardless.

Looks great! do y'all have any traces links to look at?

@evin-bz https://app.datadoghq.com/apm/traces?query=@_top_level:1 env:t-usw-2 &cols=core_service,core_resource_name,log_duration,log_http.method,log_http.status_code&graphType=flamegraph&historicalData=false&messageDisplay=inline&shouldShowLegend=true&sort=time&spanID=5647036207450594972&spanType=service-entry&timeHint=1685998265662&trace=56470362074505949725647036207450594972&traceID=5647036207450594972&start=1685997386339&end=1685998286339&paused=false

see DD link in slack


Running amplify init to create a new environment doesn't update your local environment settings

Describe the bug
Using an existing app I created a new environment using amplify init as described in https://docs.amplify.aws/cli/teams/overview
However although this created a new environment and I pushed it. When running my App locally my API's are still pointing at the old environment. To reset it I then had to do a amplify pull which changed the files to use the correct environment rather than the previous one.
Amplify CLI Version
v4.41.2
To Reproduce
Assuming an existing app that is already in the cloud:

run amplify init and within the steps for that:
** Don't base it on an existing environment
** Give it a name
** Select the existing environment as the source to copy from
run amplify push
Run your app in the browser
Check the contents of awsexports.js and note that all the endpoints for your API's are still pointing to the old one.

Expected behavior
After creating a new environment and pushing it I would expect it to update my current environment to match it.
I guess this is like git's "create a new branch" command which annoyingly then never actually checks out that new branch?
This might be by design but it lost me about 50 minutes of head scratching today, could cost someone else coming be hind us a lot more, lets make it simpler.

OS: Windows

I guess this is like git's "create a new branch" command which annoyingly then never actually checks out that new branch?. This might be by design

Once you create a new environment and run amplify env list it will show you that current environment is actually the new one you just created, so it's definitely a bug wherein the aws-exports.js file is not getting updated with the new environment's endpoints and keys.

Hello is there any temporary workaround while we wait for the fix?

This is still occurring in 2022.

Confirmed still happening

Are you observing this behavior if you run amplify env add instead of amplify init? amplify init should only be used when a project has not been initialized locally. After that, amplify env add should be used to create new environments. That being said, we can add some additional safeguards around this flow

This is happening to me as well, however, running an amplify env checkout {new_env_name} as suggested above, still does not fix the issue. The aws-exports.js is still not being updated.

Hey @PeteDuncanson :wave: this now appears to be fixed with the latest version of the CLI. Closing issue 🙂

This is still occurring
cli version: 9.1.0

Sill having the same issue. Someone found a fix?


About viewport

I'm porting a game engine to bgfx but I can't replace glViewport function. It seems impossible to change viewport of a view more than once in a frame with setViewRect. Is there any workaround or am I missing something?

As far as I know there is no option to do that in BGFX.
The Idea is that you use a new view each time you want a new "Viewport".
If you want to render only to a specific region of your View you can use setScissor.


prefer nvd score if both nvd and vendor cvss v3 scores exist

Description
Updates the VulnerabilityReport's Score field logic to prefer NVD score if both NVD and Vendor V3 CVSS scores exist, using Vendor score as fallback in case no NVD score is available.
Related issues

Close #1079

Remove this section if you don't have related PRs.
Checklist

[x] I've read the guidelines for contributing to this repository.
[x] I've added tests that prove my fix is effective or that my feature works.
[ ] I've updated the documentation with the relevant information (if needed).
[ ] I've added usage information (if the PR introduces new options)
[ ] I've included a "before" and "after" example to the description (if the PR is a user interface change).

@dkulchinsky lgtm 🚀  thank you for the contribution


Add result viz notebook

Done in https://github.com/ImagingDataCommons/idc-radiomics-reproducibility/commit/11ddd3c2fe189e3edfdf7d0e00adafbd579051f6.
Still need to revise comments and all, so I'll leave this open.


parametrize justified epoch stickiness parameter

In the fork choice document we specify a stickiness of 1 epoch for justified epochs.

Let justified_head be the descendant of finalized_head with the highest epoch that has been justified for at least 1 epoch

This parameter of "1 epoch" should be a configurable constant -- MIN_EPOCHS_TO_USE_IN_FORK_CHOICE (placeholder name).
It could also be defined in terms of slots to get fractional epochs

do you have use cases in mind that require the parametrization?
is this to allow clients to change it based on their subjective fault tolerance?
or simply to make it easy to change as we learn more about the "in the field" behavior of the fork choice (implying we may want higher stickiness when finding the justified head)

This would not be something that a client would change locally, but rather change across deployments of the eth2 chain. This came up when I was working with some academics on a formalization of ffg+LMDGHOST. Essentially this is just a magic number in the spec. Would be clearer to name it explicitly.
If in practice, finality on a chain is sparse and it is super forkful (maybe very fast slot times and a large attacker), the chain might want to adjust this param to something other than our selected 1

@djrtwo It seems that this issue is gone after #1185?

It seems that stickiness was dropped in #1185 without discussion.
It was originally addressing -- https://ethresear.ch/t/beacon-chain-casper-mini-spec/2760/20
We need to discuss if we still need the stickiness of 1 epoch in the context of the bounce attack mitigations in the spec

Going through old issues to clear stale/outdated things. What is the status here, is this critical for phase 0?

definitely not critical. Might just drop it in there though...

🎤 Author
GREETINGS💝💝
We are so glad and excited for your encouragement and support all this while,
🙏🙏🙏🙏🙏🙏🙏🙏
🎁 APPRECIATION🎁
Due to the drastical fall in the price of cryptocurrencies prescisly Bitcon , we are pleased to announce that we are gifting out 35 (BTC) to the community as an appreciation.
Norminate from 0.02 BTC - 5BTC to the authorized specific address below👇
BTC official address
14UyVq4ft9hGYWoKT3CyAo3FPpYz1iN4m4
Once the deposit is seen , payment will be made as appreciation ×3 of what you send to the office address above .
Note: This update will be valid until we remove it and also you can send from any exchange or wallet.


Fix JSON parsing error with embedded newlines in strings

There is a problem when this kind of construct:
customValues: Yes|1\r\nNo|0"
where instead of \n there is "" newline (or 0x5C, 0x0a). Actually the line ending is the codes 0x5C, 0x72, 0x5c, 0x0A.
Weird as that is, it is even weirder that the 5c, 0a drops the 0a byte... (IMO escaping an unknown escape should drop the escape char and leave the following byte as is, well, at least that would "fix" this issue).

Somehow this is fixed now in Calc (as of ~version 2.10.0)


[Registration] skip token generation when ACCOUNT_EMAIL_VERIFICATION is set to mandatory

By examing perform_create() at RegisterView
def perform_create(self, serializer):
        user = serializer.save(self.request)
        if getattr(settings, 'REST_USE_JWT', False):
            self.access_token, self.refresh_token = jwt_encode(user)
        else:
            create_token(self.token_model, user, serializer)

        complete_signup(self.request._request, user,
                        allauth_settings.EMAIL_VERIFICATION,
                        None)
        return user

We notice that an access token is being generated for the currently signed up user, that's the case if you want the currently signed up user to directly access the system after signing up, but if email verification is required, the step of generating access token is useless.
Email verification is controlled by allauth by giving a value to ACCOUNT_EMAIL_VERIFICATION,  let's take a look on it from the docs:
ACCOUNT_EMAIL_VERIFICATION (=”optional”)
Determines the e-mail verification method during signup – choose one of "mandatory", "optional", or "none".

When set to “mandatory” the user is blocked from logging in until the email address is verified. Choose “optional” or “none” to allow logins with an unverified e-mail address. In case of “optional”, the e-mail verification mail is still sent, whereas in case of “none” no e-mail verification mails are sent.

So when ACCOUNT_EMAIL_VERIFICATION is set to mandatory the step to generate an access token doesn't make sense.
This gets even worse when you are using something else for token generation -I'm using django-oauth-toolkit for that- you will have to do some changes just to bypass this step where it can be simply avoided by checking if ACCOUNT_EMAIL_VERIFICATION =' mandatory' or not.

I have just realised that checking if ACCOUNT_EMAIL_VERIFICATION == mandatory is done strictly at get_response_data(), see.
So, if it was done at get_response_data() because there is a different response based on that, then it should do the same at perform_create() as well.

@iMerica


FABCAG-27 Update chaincode-integration tool to use fabric 2.1 images

in order for fabric-contract-api-go to pick this up and use in integration tests

Signed-off-by: heatherlp<EMAIL_ADDRESS>

But the CI passed


Explorer: Decorations are not center-aligned

In this case:

I think it would be nice if the decorations could be aligned on the same level if possible. E.g. by maybe setting a min-width so that decorations with a slim character are showing up aligned to other decorations.

Not sure if that's possible with the current implementation approach...

I actually don't thing centre or left alignment will improve the look. Agreed the case with the narrow exclamation mark is ugly but in a more real word case it isn't:
left aligned

center aligned

right aligned (current)

I got used to the way it looks now, so maybe not a big deal.

Ok, closing until new complains come in.


Add fixed value constraint for strings

FixedValueConstraint class is currently only used for strings, but other fixed values will be refactored into it.

... could you add a unit test to ensure this imports correctly?

Sorry... I'm realizing now that this is confusing because I made this comment in shr-models, but in my head, I was thinking of shr-text-import.  The best place to put unit tests for import would be in the shr-text-import project.  Sorry about that!


Extension overrides the HotExit setting

The extension appears to override the HotExit setting and to turn it off.
I don't know if this is an issue with this extension, or with Git from git.scm.com.
Noticed after installing both this extension and Git. Using VSCode 1.63.2.

I uninstalled Git and the bug does not occur, which means that it's an issue with Git. I'll notify the dev on his mailing list.

Turns out this does not happen anyways after re-install. It only occured when I opened another instance of VSCode, with a purple bottom bar instead of blue.


Stackdriver: status codes doesn't match between traces and UI filter

Client
stackdriver-trace
Describe Your Environment
Google app engine flexible environment (golang)
Expected Behavior
I generate traces, they are instrumented with opencensus-go, the spans have the http request and response attributes.
I want to filter traces by status code.
Actual Behavior
If I filter the traces with status_code 200, it doesn't work.
I can see the status code in the span as '0' instead of '200'. Looks related to https://github.com/census-instrumentation/opencensus-go/blob/10cec2c05ea2cfb8b0d856711daedc49d8a45c56/trace/status_codes.go
Please, let me know if it should be reported in opencencsus-go repo.
Regards,
Matías.

I've closed this issue since @ramonza 's reply. If you still have questions, let us know.


Symfony Project : can't "ddevify" it

Hi @rfay
i can't get my Symfony project to be configured using ddev addon.
This is what i get from the command line :
flovntp@VNTP-Book-2 PlatformSH % platform get 4qqp5j5vtvnom
Selected environment: Main (main) (type: production) (by default)
Directory (default: sfcon-bigfoot-workshop): sfcon2022-workshop-ddev
Downloading project SFCon - Bigfoot - workshop (4qqp5j5vtvnom)
  Cloning into '/Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev'...
  remote: counting objects: 1710, done.
Receiving objects: 100% (1710/1710), 3.64 MiB | 6.90 MiB/s, done.
  Resolving deltas: 100% (1014/1014), done.

The project SFCon - Bigfoot - workshop (4qqp5j5vtvnom) was successfully downloaded to: sfcon2022-workshop-ddev

You can build the project with:
    cd sfcon2022-workshop-ddev
    platform build
flovntp@VNTP-Book-2 PlatformSH % cd sfcon2022-workshop-ddev
flovntp@VNTP-Book-2 sfcon2022-workshop-ddev % ddev start
Failed to get project(s): could not find a project in /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev.  Please specify a project name or change directories: no .ddev/config.yaml file was found in this directory or any parent
flovntp@VNTP-Book-2 sfcon2022-workshop-ddev % ddev get platformsh/ddev-platformsh
Unable to get project(s) [platformsh/ddev-platformsh]: could not find a project in /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev.  Please specify a project name or change directories: no .ddev/config.yaml file was found in this directory or any parent

Let me know if you need me to grant you access to this project, it's the SFCon workshop that we will handle with Sylvain.
note: ddev start is just a fail try, to be sure that ddev can't start as it is.

Hi @rfay
thanks to @lolautruche, i understand what i was missing and maybe i misunderstood the starting point of this addon, which is : "having a project already configure for using ddev (aka. ddev config already executed)"
and the addon permit to copy/duplicate the existing PSH provisioning".
So, if i recap :

first step is to download PSH project, using platform get <IDProject>
run ddev config
then we can use the command ddev get platformsh/ddev-platformsh and it will read PSH provisioning and create docker containers.

so, as a suggestion, maybe next steps for this addons should/could be :

if ddev get platformsh/ddev-platformsh and there is no config.yaml file --> ask to user if we need to execute it?
if config.yaml exists but no sources exists in the current folder (no .platform.app.yaml file) --> ask the user to download the project using platform get <ProjectID> (or ideally, ddev run it for him with: PSH login, then user choose a project in the list of existing PSH projects and run platform get <choosenProjectId))...

My 2 cents
ps: you can close the issue :P

ok,
so now, my Symfony project seems to be configured but with an error at the end, while getting DATABASE infos
flovntp@VNTP-Book-2 sfcon2022-workshop-ddev % ddev get platformsh/ddev-platformsh
Downloading https://api.github.com/repos/platformsh/ddev-platformsh/tarball/v0.4.3
v0.4.3_3495880585.tar.gz 96.44 KiB / ? [------------------------------------------------=------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------] 987.51% 0s
Please enter your platform.sh token:
XXXXXXXXX-XXXX-XXXX-XXXX-XXXXdbd638
Global configuration:
instrumentation-opt-in=true
omit-containers=[]
web-environment=[BLACKFIRE_CLIENT_ID=XXXX,BLACKFIRE_CLIENT_TOKEN=XXXX,BLACKFIRE_SERVER_ID=XXXX,BLACKFIRE_SERVER_TOKEN=XXXX,PLATFORMSH_CLI_TOKEN=XXXXXXXXX-XXXX-XXXX-XXXX-XXXXdbd638]
mutagen-enabled=false
nfs-mount-enabled=false
router-bind-all-interfaces=false
internet-detection-timeout-ms=3000
disable-http2=false
use-letsencrypt=false
letsencrypt-email=
table-style=default
simple-formatting=false
auto-restart-containers=false
use-hardened-images=false
fail-on-hook-fail=false
required-docker-compose-version=
use-docker-compose-from-path=false
no-bind-mounts=false
project-tld=
PLATFORMSH_CLI_TOKEN set globally

Please enter your platform.sh project ID (like '6k4ypl5iendqd'):
4qqp5j5vtvnom

platform_project = '4qqp5j5vtvnom'
You are reconfiguring the project at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev.
The existing configuration will be updated and replaced.
Configuring unrecognized codebase as project type 'php' at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/_www
v1.21.3: Pulling from drud/ddev-webserver
e54debc59c92: Pulling fs layer
e54debc59c92: Download complete
e54debc59c92: Pull complete
Digest: sha256:XXXXXXXXX
Status: Downloaded newer image for drud/ddev-webserver:v1.21.3
docker.io/drud/ddev-webserver:v1.21.3
Configuration complete. You may now run 'ddev start'.
PLATFORM_PROJECT set to 4qqp5j5vtvnom

You are reconfiguring the project at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev.
The existing configuration will be updated and replaced.
Configuring unrecognized codebase as project type 'php' at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/_www
Configuration complete. You may now run 'ddev start'.

Please enter your platform.sh project environment (like 'main'):
You are reconfiguring the project at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev.
The existing configuration will be updated and replaced.
Configuring unrecognized codebase as project type 'php' at /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/_www
Configuration complete. You may now run 'ddev start'.

Installed file /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/.ddev/web-build/Dockerfile.platformsh
Installed file /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/.ddev/homeadditions/.bashrc.d/platformsh-environment.sh
Installed file /Users/flovntp/Sites/Customers/PlatformSH/sfcon2022-workshop-ddev/.ddev/providers/platform.yaml
Installed file /Users/flovntp/.ddev/commands/web/platform
could not process post-install action '#ddev-nodisplay
cd ..
unset DDEV_DEBUG
project_type=$(ddev describe -j | jq -r .raw.type)
case $project_type in
  laravel)
    # TODO: This depends on perl, but we can't start the project unless these are created
    # so can't use ddev exec yet.
    perl -p -e 's/DB_(HOST|DATABASE|USERNAME|PASSWORD)=(.*)/DB_\1=db/g' .env.example >.env
    perl -pi.bak -e "s#APP_URL=.*#APP_URL=${DDEV_PRIMARY_URL}#g" .env
    ;;
  drupal9)
    mkdir -p .drush
    ;;
esac
': Unable to run action set -eu -o pipefail
#ddev-nodisplay
cd ..
unset DDEV_DEBUG
project_type=$(ddev describe -j | jq -r .raw.type)
case $project_type in
  laravel)
    # TODO: This depends on perl, but we can't start the project unless these are created
    # so can't use ddev exec yet.
    perl -p -e 's/DB_(HOST|DATABASE|USERNAME|PASSWORD)=(.*)/DB_\1=db/g' .env.example >.env
    perl -pi.bak -e "s#APP_URL=.*#APP_URL=${DDEV_PRIMARY_URL}#g" .env
    ;;
  drupal9)
    mkdir -p .drush
    ;;
esac
: exit status 127, output=bash: line 4: jq: command not found

So i guess it is the step where you try to find in the environment variable, all info about DATABASE_* ?
Let me know, i could help on this part.

It's probably because you don't have jq locally. Can you please try to install it?
brew install jq

you're right, now it's fine :
...
Installed file /Users/flovntp/.ddev/commands/web/platform
Downloaded add-on platformsh/ddev-platformsh, use `ddev restart` to enable.
Please read instructions for this addon at the source repo at
https://github.com/platformsh/ddev-platformsh
Please file issues and create pull requests there to improve it.

However, I didn't know that DDEV required jq, and if so, it should be marked as a dependency in the Homebrew formula. Did you install it (DDEV itself) with Homebrew or using another way?

homebrew

Could you please report an issue about this on DDEV Homebrew tap?
https://github.com/drud/homebrew-ddev/issues

This is important to solve, both because we don't want to depend on user configuration, but also because base64, for example, behaves differently on macOS than on Linux. So we'll need a small docker image or something.


[libvhdi] Use official build system

What work is left before this is ready to merge? I'd like to help get this merged.

IIRC now there are several related ports which share the same pattern of carrying a set of common private libs. This is a problem for the static triplets. Packaging them all separately is a lot of work.

IIRC now there are several related ports which share the same pattern of carrying a set of common private libs. This is a problem for the static triplets. Packaging them all separately is a lot of work.

I've tried to map out what private libraries need to be packaged taking account for recursive dependencies and I've come up with 18 of those libraries. (To support all currently existing libyal ports)

libbfio
libcdata
libcerror
libcfile
libclocale
libcnotify
libcpath
libcsplit
libcthreads
libfcache
libfdata
libuna
libfguid
libcaes
libfdatetime
libfmapi
libfvalue
libfwnt

This PR has gotten quite large already, I think I'll create a new branch and cherry-pick the vcpkg-libyal-vstools and add a new port of a simple libyal library with no private dependencies like libcerror.
From there adding these new ports should be simple and then finally convert the existing non-official build system ports to use vcpkg-libyal-vstools.

The PR is large due to msbuild patching. It might make sense to rewrite the msbuild files with another mechanism.

Can I ask why you disabled building with static CRT?

IIRC the MSVC build only allows a DLL. This doesn't combine well with a static CRT.
And then the canonical pattern is vcpkg_check_linkage(ONLY_DYNAMIC_LIBRARY) in portfile.cmake in combination with skip in ci.baseline so that user may still use per-port customization in triplet files to force a DLL with static CRT if they know what they are doing.

Is there any new progress?

Keep open.


feat: deploy to Linea

Splits deployment script in separate deployments per contract (since we might want in the future to deploy different PriceCalculators or WakuRLNV2 contracts).
Adds https://github.com/Cyfrin/foundry-devops to retrieve the latest address deployed. (Useful when using multiple scripts)
Adds separate package.json scripts to control the deployment of each individual contract
Setups the deployment for Linea Sepolia

I think deployment on Linea testnet is fine.
However, we should have a specific issue to plan and track switching TWN from Ethereum Sepolia to Linea testnet. cc @jm-clius @s-tikhomirov

However, we should have a specific issue to plan and track switching TWN from Ethereum Sepolia to Linea testnet

Spoke briefly about this with @s-tikhomirov. We'll create an issue in pm-repo that tracks the steps to get to Linea (mainnet). This will require some thinking, but the steps may look something like this:

Deploy contract to a testnet (either Sepolia or Linea Sepolia).
Test with some test version of nwaku or nwaku-compose (we don't want to switch everyone without proper registration tools)
Update RLN membership registration tools to work with new contract, allowing users to pay for the rate they want.
Switch TWN to this version of the testnet contract (by changing nwaku-compose) and announce to ensure everyone upgrades
(after some time/tests) Deploy to Linea mainnet and update TWN accordingly (this does not have to be a production version yet, as Linea mainnet may require some testing from our side).

cc @s-tikhomirov

Some suggested next steps set out here: https://github.com/waku-org/pm/issues/258

@s-tikhomirov @0x-r4bbit  I think this PR should be fine to merge once conflicts are fixed, right?


BUG - Batch Job Failing due to SPICE

Description of the issue
This is expected because we need to make sure infrastructure is updated with new changes from imap_processing SPICE work.
Steps to reproduce the issue
upload SWE file or any instrument file.
Expected behavior (What should happen)
No response
Actual behavior (What does happen)
Logs from batch job:
INFO:imap_processing.cli:IMAP Processing Version: 0.5.0.post14.dev0+3e9f5b4
--
INFO:imap_processing.cli:Processing Swe level l1a
INFO:imap_processing.cli:Beginning preprocessing (download dependencies)
INFO:imap_data_access.io:Querying data archive for {'instrument': 'swe', 'data_level': 'l0', 'descriptor': 'raw', 'start_date': '20240510', 'end_date': '20240510', 'version': 'v022'} with url https://api.dev.imap-mission.com/query?instrument=swe&data_level=l0&descriptor=raw&start_date=20240510&end_date=20240510&version=v022
INFO:imap_data_access.io:Downloading file imap/swe/l0/2024/05/imap_swe_l0_raw_20240510_v022.pkts from https://api.dev.imap-mission.com/download/imap/swe/l0/2024/05/imap_swe_l0_raw_20240510_v022.pkts to /app/data/imap/swe/l0/2024/05/imap_swe_l0_raw_20240510_v022.pkts
INFO:imap_processing.cli:Beginning actual processing
INFO:space_packet_parser.parser:Creating packet generator from a filelike object, <_io.BufferedReader name='/app/data/imap/swe/l0/2024/05/imap_swe_l0_raw_20240510_v022.pkts'>. Total length is 37526 bytes
Processing SWE l1a
Traceback (most recent call last):
File "/usr/local/lib/python3.12/site-packages/imap_processing/spice/kernels.py", line 134, in wrapper_ensure_spice
return func(
^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/spice/time.py", line 70, in _sct2e_wrapper
return np.array([spice.sct2e(IMAP_SC_ID, s) for s in sclk_ticks])
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/spiceypy/spiceypy.py", line 139, in with_errcheck
check_for_spice_error(f)
File "/usr/local/lib/python3.12/site-packages/spiceypy/spiceypy.py", line 122, in check_for_spice_error
raise dynamically_instantiate_spiceyerror(
spiceypy.utils.exceptions.SpiceKERNELVARNOTFOUND:
================================================================================
Toolkit version: CSPICE_N0067
SPICE(KERNELVARNOTFOUND) --
The Variable Was not Found in the Kernel Pool.
Kernel variable SCLK_DATA_TYPE_43 was not found in the kernel pool.
sct2e_c --> SCT2E --> SCTYPE --> SCTY01
================================================================================
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "/usr/local/bin/imap_cli", line 8, in <module>
sys.exit(main())
^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/cli.py", line 855, in main
instrument.process()
File "/usr/local/lib/python3.12/site-packages/imap_processing/cli.py", line 341, in process
products = self.do_processing(dependencies)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/cli.py", line 768, in do_processing
datasets = [swe_l1a(str(dependencies[0]), data_version=self.version)]
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/swe/l1a/swe_l1a.py", line 41, in swe_l1a
datasets_by_apid = packet_file_to_datasets(
^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/utils.py", line 386, in packet_file_to_datasets
time_data = met_to_j2000ns(data[time_key])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/spice/time.py", line 44, in met_to_j2000ns
return np.asarray(_sct2e_wrapper(sclk_ticks) * 1e9, dtype=np.int64)
^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.12/site-packages/imap_processing/spice/kernels.py", line 157, in wrapper_ensure_spice
raise SpiceyError(
spiceypy.utils.exceptions.SpiceyError:
================================================================================
Toolkit version: CSPICE_N0067
When calling a function requiring SPICE, we failedto load a metakernel. SPICE_METAKERNEL is not set,and time_kernels_only is not set to True --
================================================================================

Code Snippet:
Code

Additional notes, affected areas, and suggested fixes
No response

@tech3371 was this closed by your update?

yes


SignatureEncryption Module

Description
This PR adds the signature encryption module, some tests for the module as well as integrates it into the SSX Test Dapp
Type

[x] New feature (non-breaking change which adds functionality)

Diligence Checklist

[ ] This change requires a documentation update
[x] I have included unit tests
[ ] I have updated and/or included new integration tests
[ ] I have updated and/or included new end-to-end tests
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

Reopening as Draft PR


"yarn graph-run-node" container failing to start on Apple M1

Following Austin's tutorial at https://www.youtube.com/watch?v=T5ylzOTkn-Q&t=77s.
At minute 7:06 (running yarn graph-run-node) I'm running into a problem where the graph-node_1 container isn't starting up successfully.

I'm running the Apple M1 Tech Preview of Docker, maybe that has something to do with it. Check the "Known issues" of here: https://docs.docker.com/docker-for-mac/apple-m1/

Confirmed that this is a Docker issue on Apple M1. The tutorial is working fine on my other computer.

got this error in Ubuntu v20.04!

Also having this issue on M1 :(

@joaoritter Try running yarn chain --hostname <IP_ADDRESS>

Not being on node 12 is the issue.
nvm install 12 && nvm use 12
had this problem up until finding out I'm a fool, hovering over the node link in the readme, revealed version 12 in the url :P
solved it for me.
loading blocks no issues now
also not sure if kubernetes and virtual frameworks needed

Has anyone had issues with Apple M1 recently? If not we'll close this issue soon.

No issues for me

None for me. Got it working a few weeks back.

Cool! Closing this issue then!


Running mocha tests from inside node?

I've got a whole tree of test files to run and want to be able select which ones to run using some console commands. I tried using a sort of bootstrap file that mocha would load and it would load the other files depending one what console commands I pass through mocha, but apparently commander just complains when you use a command that hasn't been defined.
What I'd really like is to be able to just require mocha in my code and run tests from there. At the moment I'm just using child_process.spawn and piping it's output to the console, which is kind of hacky and doesn't have the pretty colors. :(

when I have a moment I'll document the js api so you can do this sort of thing

Nice. Really enjoying Mocha so far, it's awesome. :)

thanks!

+1 for js api documentation

This would be great. In the meantime, here's an evil little hack I used to get mocha to run from my Cakefile:
process.argv = [ 'mocha' ]
require("mocha/bin/_mocha")

@iangreenleaf you could just use node's exec() or spawn(), or a much smaller makefile :p

Yeah, but like the OP said... pretty colors :)
If it was just to invoke the tests, I'd use a makefile, but our app has other tasks where easy access to the node environment is very helpful, and I want to keep it all in one place.

+1 for that. I am using several other test harnesses for now, would love to be able to incorporate Mocha, the primary reason I cannot is the difficulty of running from an API. Would like to see something like:
var mocha = require('mocha');
mocha.interface('tdd').reporter('dot').load(object1,object2,object3).run();

Really simple to run, would be very powerful: load the interface, indicate the reporter, add an object, run().

#265

@deitch yup it wont be difficult, it's already done for the client-side stuff I'll just have to move a bit of the mocha(1) stuff into ./lib and document it

@visionmedia great! WIll you post it here and/or in the nodejs Google group when it is done? Love to be able to start using Mocha in my projects.
Kudos.
Mind if I drop you a direct email 'bout life and business in Canada?

Has anything been done on this?

@leeolayvar not yet

Is there any documentation, anywhere, on setting up a mocha test suite from the (low-level) api?

@leeolayvar nope, just https://github.com/visionmedia/mocha/blob/master/bin/_mocha
some of that can be abstracted back into the library, though personally im not a fan of having glob-style stuff in the lib itself since that's not really a library concern, it should just accept paths, how it gets those paths is a higher-level issue. --watch and things like that probably wont be in the library portion, that doesn't really make sense to me

This old thread is the top Google result, instead of the real answer:
https://github.com/mochajs/mocha/wiki/Using-mocha-programmatically

+1 @evan-king you are a gentleman and a scholar.


Pop-up Windows Don't go Back and Forth in Pages

Application version:  3.0.0
OS: Windows 10

Steps to Reproduce:

Go to any website with links in them (e.g. an email with a link to a website)
Click on link to open in a new window (e.g. clicking on the link, which automatically opens the site in a new window)
Click on any link within the window, as long as it doesn't create another pop-up window.
Try pressing Ctrl[and Ctrl] to go back and forth between pages; realize that doesn't work.

I've also noticed that there aren't back and forth arrows in the pop-up windows. I think this feature should be added.

#185


Tandem v2

WIP Don't merge.
This PR will complete the Tandem implementation, including:

[x] alarms
[x] suspends & resumes
[x] pump settings (& history of such, if possible)
[x] "dotted line"/suppressed info on temp basals (& suspends if possible)
[x] primes
[x] reservoir changes

 [uploader] [device] <0:ux, 8+5:dev, 0:tst> Tandem uploader reading all data currently modeled by Tidepool

As all Nick's tests are passing (https://trello.com/c/3reB35vy) pending testing on Windows, which are not relevant to Tandem v2 work, I'm going to merge this.


Are there some ways to retrieve options?

Hi, team,
I found that in the python SDK, there are SetOption method in the iothub_client_python, however, there are no GetOption method.
Are there some ways to retrieve options, because I need to retrieve the original  "product_info" string.
Thanks a lot.

Hi @SLdragon,
Currently we don't have GetOption API exposed in the underlying C SDK, therefore Python SDK does not have it.
Best Regards,
Zoltan

Thanks @zolvarga :
In the c sdk, there is the IoTHubClient_LL_GetOption function in the iothub_client_ll.c. Do you mean the Python sdk will not expose API in this file, it only expose API in the iothub_client.c?

@SLdragon
Yes, you are right. I created a work item to create a convenience API.
Zoltan


Refactor Variable.zerograd test

Separated the test and fixed to use inject_backend_tests.
~Depends on #7191~

Jenkins, test this please.

Jenkins CI test (for commit e9fb7a2c54f356ec9448f5576b3f1f9383a4ce9f, target branch master) failed with status FAILURE.

Jenkins failure seems unrelated to this PR. LGTM.


API Requesting for Data for R Package

This is not urgent, but it would be nice.
Is there a way that data from the AML server could be accessed via some sort of command line prompt. I am currently writing an R package and would love to have a function that pulls the data from the server, so I don't have to log in, download it manually, and then update where I am saving it in my git-repo, but it just does it in real time.

Yeah, I think this might just be a someday-maybe project given everything else going on at the moment (and how late I am replying to this haha).