Icon in volume mixer is too big

What version of WebTorrent Desktop? (See the 'About WebTorrent' menu)
0.3.3
What operating system and version?
Windows 8, 64bit
The issue can be clearly seen in Volume mixer, when any video is playing. Seems that the icon is way too big/doesn't scale correctly

dups #365 also #387

@gediminaspetrikas so wait for next relase, already fixed bugs

Going to try to cut a release today or tomorrow, since there's lots of bug fixes we need to get out.

Awesome, sorry for not digging deep enough, skimmed through but did not notice those


fc_run ERROR: CRITICAL - Error in ...

Operating system
Red Hat Enterprise Linux Server release 6.5 (Santiago)
Package name
pb_assembly
Conda environment
falcon-kit 1.2.6; pypeflow 2.1.1
Describe the bug
I run fc_run with the example data from [https://github.com/PacificBiosciences/pb-assembly] in a cluster . The cfg file is:
fc_run_200kb.cfg.txt
I got the error message in the stderr file:
Error message
pypeflow 2.1.1
pypeflow 2.1.1
    raise Exception(msg)
Exception: Call 'bash -vex build_db.sh' returned 256.
2019-03-16 15:04:37,109 - root - WARNING - Call '/bin/bash user_script.sh' returned 256.
2019-03-16 15:04:37,109 - root - WARNING - CD: '/BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/result/0-rawreads/build' -> '/BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/result/0-rawreads/build'
2019-03-16 15:04:37,109 - root - WARNING - CD: '/BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/result/0-rawreads/build' -> '/BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/result/0-rawreads/build'
2019-03-16 15:04:37,109 - root - CRITICAL - Error in /BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py with args="{'json_fn': '/BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/result/0-rawreads/build/task.json',\n 'timeout': 30,\n 'tmpdir': None}"
Traceback (most recent call last):
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/runpy.py", line 72, in _run_code
    exec code in run_globals
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py", line 267, in <module>
    main()
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py", line 259, in main
    run(**vars(parsed_args))
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py", line 253, in run
    run_cfg_in_tmpdir(cfg, tmpdir, '.')
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py", line 228, in run_cfg_in_tmpdir
    run_bash(bash_template, myinputs, myoutputs, parameters)
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/do_task.py", line 187, in run_bash
    util.system(cmd)
  File "/BIGDATA1/app/anaconda2/5.0.1-gcc-4.8.5/lib/python2.7/site-packages/pypeflow/io.py", line 29, in syscall
    raise Exception(msg)

How can I fix this?
Thanks

For some reason, your stderr does not have all the stack-traces we would need. Go into 0-rawreads/build/ and run bash -vex user_script.sh.  Hopefully, that will help you debug this.
Since the first task is failing, you're probably missing the conda env's bin from your environment on the remote machine. To fix that, you need to learn the proper flag for submitting the job and passing along the environment. (If that's impossible on your system, we can show you a hidden flag to run custom bash code in each remote job.)

For some reason, your stderr does not have all the stack-traces we would need. Go into 0-rawreads/build/ and run bash -vex user_script.sh. Hopefully, that will help you debug this.
Since the first task is failing, you're probably missing the conda env's bin from your environment on the remote machine. To fix that, you need to learn the proper flag for submitting the job and passing along the environment. (If that's impossible on your system, we can show you a hidden flag to run custom bash code in each remote job.)

Hi, thanks for your reply.
This is the complete stderr file:
Uploading run-P6fb1b878b6327d.bash.stderr.txt…
and this the output of "bash -vex user_script.sh"
bash -vex user_script.sh.txt
I have no idea what's going wrong from this.
How do I know wehther I am missing the conda env's 'bin'?  In our cluster, the pb_assembly package was installed by the administrator. I just load the package by "module load" command.

How do I know whehther I am missing the conda env's 'bin'?

Hard to say. It worked when you ran it manually, so it's possible. But I think the problem is something else.
fasta2DB: [WARNING] Could not delete temporary file subreads1.fasta

That's odd. But stranger is this:
+ read fn
+ fasta2DB -v raw_reads -isubreads2.fasta
+ python -m falcon_kit.mains.fasta_filter pass -
+ cat /BIGDATA1/wbgcas_tfeng_1/project/pb_assembly_test/fasta/subreads2.fasta
fasta2DB: raw_reads.db is corrupted, read failed

The mixed lines of stderr indicate that multiple procs are running. They seem to be writing into the DB simultaneously.
This task is not parallel, and it has no "units of work". So something very strange is going on...
I see repetition even at the top of your stderr, e.g. setting our bash variables. So I think your job has been submitted as multiple jobs piped into the same stderr.
This is a problem either with your job submission system or with your "submit" line.
submit = srun --wait=0 -p ${JOB_QUEUE} -J ${JOB_NAME} -o ${JOB_STDOUT} -e ${JOB_STDERR} --mem-per-cpu=${MB}M --cpus-per-task=${NPROC} ${JOB_SCRIPT}

You will need to talk to your sysadmin. This does not appear to be a Falcon bug at all.


Kage: DirectX function returning an array type

Ebitengine Version
v2.6.3 - until now
Operating System

[X] Windows
[ ] macOS
[ ] Linux
[ ] FreeBSD
[ ] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] PlayStation 5
[ ] Xbox
[ ] Web Browsers

Go Version (go version)
1.21.1
What steps will reproduce the problem?
//kage:units pixels

package main

func f() [25]int {
	return [25]int{}
}

func Fragment(dst vec4, src vec2, color vec4) vec4 {
	v := float(f()[0])
	return vec4(v)
}

Compile the above shader on windows+directx
(works with OpenGL)
What is the expected result?
Compilation successful
What happens instead?
directx: D3DCompile failed: shader(51,4): error X3000: syntax error: unexpected token '['
: HANDLE(2147500037)

struct Varyings {
        float4 Position : SV_POSITION;
        float2 M0 : TEXCOORD0;
        float4 M1 : COLOR;
};

float mod(float x, float y) {
        return x - y * floor(x/y);
}

// [...]

Texture2D T0 : register(t0);
Texture2D T1 : register(t1);
Texture2D T2 : register(t2);
Texture2D T3 : register(t3);
SamplerState samp : register(s0);

int[25] F0(void); // Line 51!

int[25] F0(void) {
        int l0[25];
        l0[0] = 0;
        l0[1] = 0;
        l0[2] = 0;
        l0[3] = 0;
        l0[4] = 0;
        l0[5] = 0;
        l0[6] = 0;
        l0[7] = 0;
        l0[8] = 0;
        l0[9] = 0;
        l0[10] = 0;
        l0[11] = 0;
        l0[12] = 0;
        l0[13] = 0;
        l0[14] = 0;
        l0[15] = 0;
        l0[16] = 0;
        l0[17] = 0;
        l0[18] = 0;
        l0[19] = 0;
        l0[20] = 0;
        l0[21] = 0;
        l0[22] = 0;
        l0[23] = 0;
        l0[24] = 0;
        return l0;
}

float4 PSMain(Varyings varyings) : SV_TARGET {
        float l0 = 0.0;
        l0 = float((F0())[0]);
        return (float4)(l0);
}

Anything else you feel useful to add?
Before v2.6.3 there was a panic nil pointer dereference on a drawTriangle command internally, and I can't tell if it's related.
I don't think it ever worked or that it could be due to a breaking change, because the case seems a bit special.

Interesting. Apparently HLSL cannot return an array from a function.
Instead, we should be able to use out parameters.


Enhance pre commit verification

This pull request add few more pre-commit hooks that verify different standards of coding style and quality. specifically flake8 which is the SOTA widely used tool for checking the style and quality of python code.

Can this be closed?

Can this be closed? Given there recent PR that was merged?


How to calculate other EM components

Hi Dieter,
There are some problems in my code when I calculate all EM components (Ex, Ey, Ez, Hx, Hy, Hz). The following is my code:
import empymod
import numpy as np
import matplotlib.pyplot as plt

# x-directed bipole source: x0, x1, y0, y1, z0, z1
source = [-50, 50, 0, 0, 0, 0]
# Source frequency
frequency = 10000
# Receiver offsets
offsets = np.arange(1, 11)*500
# x-directed dipole receiver-array: x, y, z, azimuth, dip
receivers = [offsets, offsets*0, 0, 0, 0]

# Layer boundaries
depth = [0]
# Layer resistivities
resistivities = [1e20, 500]

efield = empymod.bipole(
        src=source,
        rec=receivers,
        depth=depth,
        res=resistivities,
        freqtime=frequency,
        msrc=False,
        mrec=True,
        ab=41,
        verb=4,
)

fig, ax = plt.subplots()
ax.semilogy(offsets/1e3, abs(efield.real), 'C0o-', label='|Real|')
ax.semilogy(offsets/1e3, abs(efield.imag), 'C1o-', label='|Imag|')
ax.legend()
ax.set_title('Electric field at receivers')
ax.set_xlabel('Offset (km)');
ax.set_ylabel('Electric field (V/m)')

Can you help me to solve this problem? Thanks very much.
Best regards,
Zhengguang.

Hi Zhengguang. It appears to me that you mix things from empymod.dipole and empymod.bipole. The ab-parameter is only for the dipole function. Please have a look at the relevant documentation:

https://empymod.emsig.xyz/en/stable/api/empymod.model.dipole.html
https://empymod.emsig.xyz/en/stable/api/empymod.model.bipole.html

So ab=41 has no effect. It is not a problem, but you see a warning that this parameter is not used.
More importantly, the configuration you are trying to compute yields in a zero field (x-directed electric dipole measured with inline x-directed magnetic receivers), see https://empymod.emsig.xyz/en/stable/gallery/fdomain/src_rec_comparison.html : Your configuration corresponds loosely to the fourth row, first column: The field along the x-axis is zero.

I mark the issue as closed. Feel free to keep asking here though if it is not clear to you.

Hi Dieter,
Thanks very much for your reply. I have another question that is how I can calculate the Hy components from empymod.bipole.
Best regards,
Zhengguang.
发件人: Dieter Werthmüller
发送时间: 2022年9月15日 2:42
收件人: emsig/empymod
抄送: Zhengguang Liu; Author
主题: Re: [emsig/empymod] How to calculate other EM components (Issue #169)
Hi Zhengguang. It appears to me that you mix things from empymod.dipole and empymod.bipole. The ab-parameter is only for the dipole function. Please have a look at the relevant documentation:
• https://empymod.emsig.xyz/en/stable/api/empymod.model.dipole.html
• https://empymod.emsig.xyz/en/stable/api/empymod.model.bipole.html
So ab=41 has no effect. It is not a problem, but you see a warning that this parameter is not used.
More importantly, the configuration you are trying to compute yields in a zero field (x-directed electric dipole measured with inline x-directed magnetic receivers), see https://empymod.emsig.xyz/en/stable/gallery/fdomain/src_rec_comparison.html : Your configuration corresponds loosely to the fourth row, first column: The field along the x-axis is zero.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Change pip module to desired license

https://help.github.com/en/github/creating-cloning-and-archiving-repositories/licensing-a-repository

Biggest question is what happens if a commercial enterprise uses their code.

Satish has chosen BSD 3-clause.


Pyaudio problems, can't launch ledfx

(ledfx) C:\Users\user\anaconda3\envs\ledfx>ledfx --open-ui
Loading configuration file from C:\Users\user\AppData\Roaming.ledfx
Could not import the PyAudio C module 'portaudio'.
Traceback (most recent call last):
File "c:\users\user\anaconda3\envs\ledfx\lib\runpy.py", line 193, in run_modu
le_as_main
"main", mod_spec)
File "c:\users\user\anaconda3\envs\ledfx\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\Users\user\anaconda3\envs\ledfx\Scripts\ledfx.exe_main.py", line
7, in
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx_main.py",
line 88, in main
ledfx = LedFxCore(config_dir = args.config)
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\core.py", lin
e 39, in init
port=self.config.get('port', None))
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\http.py", lin
e 19, in init
self.api = RestApi(ledfx)
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\api_init.
py", line 33, in init
super().init(ledfx, RestEndpoint, self.PACKAGE_NAME)
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\utils.py", li
ne 189, in init
self.import_registry(package)
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\utils.py", li
ne 225, in import_registry
importlib.import_module(name)
File "c:\users\user\anaconda3\envs\ledfx\lib\importlib_init_.py", line 126,
in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 994, in _gcd_import
File "", line 971, in _find_and_load
File "", line 955, in _find_and_load_unlocked
File "", line 665, in _load_unlocked
File "", line 678, in exec_module
File "", line 219, in _call_with_frames_removed
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\ledfx\api\audio_dev
ices.py", line 6, in
import pyaudio
File "c:\users\user\anaconda3\envs\ledfx\lib\site-packages\pyaudio.py", line 1
16, in
import _portaudio as pa
ImportError: DLL load failed: The specified module was not found.
Steps what i did:
conda create -n ledfx python=3.6
conda activate ledfx
conda config --add channels conda-forge
conda install aubio portaudio pywin32
conda install -c anaconda pyaudio
pip install pipwin
pipwin install pyaudio
pip install ledfx
ledfx --open-ui
Can anybody explain what a problem with it?

Same here. I’m following the instructions but not able to execute

Looks like pyaudio isn't installed correctly
Adrian Stucker
513.341.LENS
On Wed, Oct 21, 2020, 4:20 PM texdetex<EMAIL_ADDRESS>wrote:

Same here. I’m following the instructions but not able to execute
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ahodges9/LedFx/issues/139#issuecomment-713962929, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJFUGINW3X3LIMJRI3U5BJLSL5UE5ANCNFSM4SR64LSQ
.

Start over on a different pc all good now.
On Wed, Oct 21, 2020 at 4:24 PM datadrian<EMAIL_ADDRESS>wrote:

Looks like pyaudio isn't installed correctly
Adrian Stucker
513.341.LENS
On Wed, Oct 21, 2020, 4:20 PM texdetex<EMAIL_ADDRESS>wrote:

Same here. I’m following the instructions but not able to execute
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ahodges9/LedFx/issues/139#issuecomment-713962929,
or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AJFUGINW3X3LIMJRI3U5BJLSL5UE5ANCNFSM4SR64LSQ
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ahodges9/LedFx/issues/139#issuecomment-713969056, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ARO5FZXRDZ5IL3T2N3LTL6DSL5USVANCNFSM4SR64LSQ
.

Try running:
conda install portaudio
And if that doesn't resolve your issue, follow this: https://stackoverflow.com/a/53087826/14312698


Points2image process dying, related to /camera

Bug
Expected Behavior
I expect the points2image process to not die when I run a node which subscribes to topic /camera/image_raw or when I use rostopic echo /camera/image_raw.
Actual Behavior
After running roslaunch runtime_manager points2image.launch, topic /points_image is published correctly. But when I run my /camera/image_raw subscriber node or rostopic echo /camera/image_raw, the points2image process dies with the following error (exit code -11 means seg fault or SIGSEGV):

[points2image-1] process has died [pid 89528, exit code -11, cmd /home/shared/Autoware/ros/devel/lib/points2image/points2image __name:=points2image __log:=/home/team0/.ros/log/5b772bf8-f4c1-11e7-b8e5-ac1f6b056614/points2image-1.log].
log file: /home/team0/.ros/log/5b772bf8-f4c1-11e7-b8e5-ac1f6b056614/points2image-1*.log" when running roslaunch runtime_manager points2image.launch

I could not find the log file in that location. I have to restart roscore before trying to run points2image again. Otherwise I get the same error repeatedly.
Steps to Reproduce the Problem

Download THIS rosbag which contains topics: /camera/image_raw, /camera_info, /points_raw, and /projection_matrix.
Play the rosbag looped: rosbag play --loop (path)
Run roslaunch runtime_manager points2image.launch
The points2image process fails ("[points2image-1] process has died")

Specifications

Ubuntu version: 16.04
ROS Version: Kinetic
Autoware branch: master

I solved that problem, removing the carmeraInfo data in Raw_bag file.
camera info topic is published by calibration publisher. So camera_info topic in Raw_bag file is overlapped with calibration publisher's camera_info.


Wrong CONST_Database_DSN in local.php

Hello! I have a question about version 3.3.
There is row with
@define('CONST_Database_DSN', 'pgsql:host=<IP_ADDRESS>;port=6432;user=nominatim;password=password1234;dbname=nominatim'); <driver>:host=<host>;port=<port>;user=<username>;password=<password>;dbname=<database>

But in readme it says to uncomment the line
@define('CONST_Database_DSN', 'pgsql://nominatim:password1234@<IP_ADDRESS>:6432/nominatim'); // <driver>://<username>:<password>@<host>:<port>/<database>

What is correct?
pgsql://nominatim:password1234@<IP_ADDRESS>:6432/nominatim

VS
pgsql:host=<IP_ADDRESS>;port=6432;user=nominatim;password=password1234;dbname=nominatim

Starting Nominatim 3.3. the syntax changed, the format with the key=valus pairs is required.

Starting Nominatim 3.3. the syntax changed, the format with the key=valus pairs is required.

You mean this one: pgsql:host=<IP_ADDRESS>;port=6432;user=nominatim;password=password1234;dbname=nominatim

Yes, Nominatim uses PHP PDO for connection management https://www.php.net/manual/en/ref.pdo-pgsql.connection.php

Fixed with https://github.com/mediagis/nominatim-docker/commit/05532d6493deb3cad66fc41a4715162c3f6858e9.


Grabbing the ID of message from bot.sendMessage

How can I do this?

In the callback, you have access to the new message:
bot.sendMessage(msg, "This is a message", function(error, newmsg) {
  console.log(newmsg.id);
}

For smaller questions you'll get quick help on [Discord API#node_discord.js](For smaller questions you'll get quick help on Discord API#node_discord.js)

Alright cool, thanks :)

Also, I'm trying to get it to add to a json file. I've tried everything like:
db["messages"].push({"id": newmsg.id});
&&
db["messages"]["id"] = newmsg.id;

Can I have some help? or look somewhere else...

Please, just join the Discord API server it will make this much easier (and won't spam our channel with github update notifications, as well as 3 ppl's emails)

Okay.

I literally did that in my first reply though.


valueSelected not selecting the RadioButtonGroup option in UI after re-render

I am having a radioButtongroup and use Mobx as store. My set up is as follows:
this.items= this.props.generalInfoStore.statusTypes.map(type => (
      <RadioButton key={type.id} id={type.id} name={type.name} label={type.name} value={type.id} />
      )
);

<RadioButtonGroup
name="status"
valueSelected={generalInfoStore.status.value}
onChange={this.onChange} >
          {items}
</RadioButtonGroup>

In the above code with valueSelected , the option is not selected on the ui when value is updated, whereas if i use defaultSelected in place of valueSelected, it works as expected.

Material-UI: 0.17.1
React: 15.4.2
Browser: Chrome

based on further testing it looks like RadioButtonGroup  should be running the logic to select radios on ComponentDidUpdate.

Figured out. So my value was integer and my onChange was like:
onChange (event,value) {
    this.props.store.updateValue(event.target.name,  event.target.value);
}

for some reason event.target.value is always string, so i updated the onchange as follows:
onChange (event,value) {
   //notice the use of value arg inplace of event.target.value
    this.props.store.updateValue(event.target.name, value);
}

But it would be nice if someone can answer, if event.target.value is always string or is it bug?

We have been porting the component on the next branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it.


doc: Add initial architecture document

Add initial content for the architecture documentation of peer pods
Please feel free to send back typos, or incorrect information, but this isn't intended to be exhaustive, but an initial placeholder for others to contribute better information that I created!

Fixes: #1319

I've push the post-review updates as a separate commit for ease of review, but will fixup before merging (and our old friend the commit message check will make sure I remember to do that 😄)

Hi @stevenhorsman ! Very good write up! Now I understand Peer pods ;)
I pointed some typos because I've eagles eyes for them!

Hi @stevenhorsman ! Very good write up! Now I understand Peer pods ;)
I pointed some typos because I've eagles eyes for them!

Thanks so much for the review I think I've fixed all the issues. I'd like to blame writing it late on a Friday as the reason for them, but I think I'm just bad at writing English accurately!

Hi @stevenhorsman ! Very good write up! Now I understand Peer pods ;)
I pointed some typos because I've eagles eyes for them!

Thanks so much for the review I think I've fixed all the issues. I'd like to blame writing it late on a Friday as the reason for them, but I think I'm just bad at writing English accurately!

Okay that as an Englishman you are a natural heir of Shakespeare, but the "wrote late on a Friday" is a very good excuse. To be or not be is the question... :)


Make parsing a whole lot better

This PR allows for a bunch of errors to be detected by the built-in Go inspections, and by proxy, it adds support for highlighting as well as some other stuff. It's not perfect by any means (in fact, there are a number of glaring issues), but this is what happened when I went down the rabbit hole that started with #30.

Thanks! I'll take a look. I'm also not very practiced in Kotlin/Java so this repo probably isn't a shining light of code quality anyway lol! If @myrjola has time to review you might get some better feedback but I'll see what I can do!

This looks good, seems that the main feature is implementing the new rawgo feature. I'll do some manual testing then merge.

Don't merge just yet, this isn't quite done (tho I will try and wrap it up later today). I just needed a second pair of eyes to review.

Additionally, it seems I maybe should have a look at https://plugins.jetbrains.com/docs/intellij/implementing-lexer.html#embedded-language. It probably would cover at least some of the crazy tricks I'm pulling.

Okay, this is at a point where I need more eyes on it. I need some help figuring out how to add tests for the GO and HTML PSI files. I need people to find where this breaks, I don't use templ enough to really find that (I used it a drop recently and the experience was poor enough that I'm working on this now).

There are still a bunch of certain errors from Go Inspections, if anyone has any idea how to appease that, please lmk.

@chabad360 can you compile a "latest" version and upload the zip? Here myrjola did the same so we could test his version.

Templ-0.0.14.zip
Here you go

Here's the first badge of feedback, I'll come back later with more as I'm on the train and have to leave now 😅 . All regarding the version you posted 1 hour ago.

The package statement doesn't seem to work correctly

Component Children (for template composition) dont work

I get "unused import" for every package I import

Templ-<IP_ADDRESS>.zip
@bastianwegge fixed that issue.

Here's the next badge of feedback:

I played around with this piece of code:

package main

type Option struct {
	Value string
	Label string
}

type Config struct {
	Options *[]Option
}

templ Select(c *Config) {
	<div>
		<select>
			for _, option := range *c.Options {
				<option value={ option.Value }>{ option.Label }</option>
			}
		</select>
	</div>
}

templ SimpleSelectUsage() {
	@Select(&Config{
		Options: &[]Option{
			{Value: "1", Label: "One"},
			{Value: "2", Label: "Two"},
			{Value: "3", Label: "Three"},
			{Value: "4", Label: "Four"},
		},
	})
}

It shows the following errors for me:
slice:

options:

comma:

brace:

variadics don't seem to be recognized right now

Great progress! Thanks for taking a stab at this ❤
I didn't know it was possible to add support for multiple languages through the TemplateLanguageFileViewProvider. Your approach looks promising. Let's squash the remaining bugs.
How did you come up with this approach? Are there any existing plugins you're taking inspiration from? They might be helpful in figuring out the missing parts.
A testing tip that was helpful for me was to try the plugin against all the templ files in the Templ repository. There's many examples inside the generator directory.
Here's some issues I encountered besides the ones reported above. I'm afraid I won't have time to dive into the rabbit hole to understand the issues better but I hope they can be fixed before merging these changes.
Multiple notNullChild crashes in Go plugin

I'm getting crashes any time there's string expressions in the templ component.
java.lang.IllegalStateException: @NotNull method com/intellij/psi/impl/PsiElementBase.notNullChild must not return null
	at com.intellij.psi.impl.PsiElementBase.$$$reportNull$$$0(PsiElementBase.java)
	at com.intellij.psi.impl.PsiElementBase.notNullChild(PsiElementBase.java:282)
	at com.goide.psi.impl.GoVarDefinitionImpl.getIdentifier(GoVarDefinitionImpl.java:39)

I believe this is caused by the Go parser's PSI tree not populating identifiers for the string expressions. I can see that we're prepending var _ string =  before the expression to make it a proper Go statement.
                TemplTypes.GO_EXPR -> {
                    if (baseLexer.state == _TemplLexer.IN_EXPR) {
                        modifications.addRangeToRemove(baseLexer.tokenStart, "var _ string = ")
                        modifications.addAll(
                            this.appendCurrentTemplateToken(baseLexer.tokenEnd, baseLexer.tokenSequence)
                        )
                    }

Notice that the VAR_DEFINITION is empty in the PSI Viewer

But not when I write a similar statement in the raw Go code outside templ components.

I don't know what's causing this. I tried working around it by creating a Go statement by wrapping the string expression with print( and ). Unfortunately, it gives a very similar notNullChild error but this time for the print identifier being empty.
Could be interesting to see if it behaves the same with MultiHostInjector.
Do you encounter this notNullChild crash in your testing?

Autoimport corrupts the file
When I autocomplete new libraries to a Templ file like this:

The package and import statements sometime become corrupted

I'm suspecting this is caused by adding the import "github.com/a-h/templ" in GoLanguageType. I'm not sure how to approach this. The Templ LSP is already doing autoimports as of https://github.com/a-h/templ/pull/138, maybe there's some way to disable autoimports in the Go plugin and delegate them to the LSP?

How did you come up with this approach?

Brute force.

A testing tip that was helpful for me was to try the plugin against all the templ files in the Templ repository. There's many examples inside the generator directory.

This is a good idea. Honestly tho, it would be a lot more useful if I had some sort of tooling that could show me what's being sent to the Go PSI and HTML PSI and show me errors that it throws (rn I need to set some specific breakpoints and have only one file open when I start the debug session, it's annoying).

Multiple notNullChild crashes in Go plugin

Ah yes, that's been bugging me and I haven't bothered yet, to quite pursue it, so thanks for tracking it down.

Could be interesting to see if it behaves the same with MultiHostInjector.

Hmm, this approach certainly seems like it might be a bit saner than the way I'm doing things currently, I may switch to using that at some point in the near future.

I'm suspecting this is caused by adding the import "github.com/a-h/templ" in GoLanguageType. I'm not sure how to approach this. The Templ LSP is already doing autoimports as of https://github.com/a-h/templ/pull/138, maybe there's some way to disable autoimports in the Go plugin and delegate them to the LSP?

I was worried about that. I can probably make a quick fix that prevents the corruption (change the lexer to include the new line), but this would still be an issue. It's probably possible to disable Go plugin auto-imports, but I'll need to do some digging to figure it out.

Templ-<IP_ADDRESS>.zip
@bastianwegge

I played around with this piece of code:

I think I have fixed the bugs here.

variadics don't seem to be recognized right now

Hrm, I only specifically blocked children... because it's a special case. I'm not sure what to do about attrs because it's a param that can be named whatever one wants and that's a bit of a minefield (there might be some shenanigans that I can do with state tracking). In general tho, varidics are a bit of a complicated case, I may just block them for now and hope that doesn't break anything.
@myrjola

Autoimport corrupts the file

I'm not able to test that I actually fixed this atm, but if you can give it a try on your end and let me know if it's corrupting the file that would be great.

Templ-<IP_ADDRESS>.zip
Fixed a regression.

Is there a roadmap/plan for releasing this as a new version?

@up1io I think that's up to the maintainers, ultimately. But I would like to finish this up, yes. I got a bit busy with work and some other stuff recently. But I will get some more stuff done here between this week and next week.

Okay, thanks @chabad360 for the update and thanks x10 for your efforts. Your latest build is a real improvement for me.

Ha! No.
I'm travelling at the moment, but when I'm back home I'm gonna try again against the latest intellij, and if the bug is still present I'm just gonna write up a bug report (which I should have done months ago).


$request->secure() returns false by handle in middleware

Basically identical issue as this one:
http://laravel.io/forum/06-22-2014-requestsecure-return-false-no-ssl-but-ssl-certificate-is-applied?page=1
I'm running nginx with fastcgi

Never use 'HTTP_X_FORWARDED_PROTO'.
You need to correctly set your trusted proxies, and symfony will automatically setup the request class correctly.

Actually ended up being a problem with nginx/fastcgi. Fastcgi doesn't/wasn't identifying itself so setting trusted proxy wouldn't work. Ended up with this solution: http://laravel.io/forum/05-09-2015-https-not-being-detected


feature: add allinone to deploy pouch as available container to kubernetes

Signed-off-by: Zou Rui<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
This script allows users to init kuberenete master node with pouch containers available.
Both Ubuntu and CentOS are supported.
Ⅱ. Does this pull request fix one issue?

NONE
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
After running this script, the user can then use
kubeadm join ...

to test whether the master node has been successfully initiated.
Ⅴ. Special notes for reviews

@YaoZengzeng PTAL

LGTM


Add new hydrophones to ML pipeline & document process

[ ] Deploy at least general OrcaHello classifier on all new hydrophone locations currently streaming to live.orcasound.net listeners for human detection.
[ ] Document the administrative and/or dev/op procedure for adding a new node to the inference system in the Orcasound administrative wiki

In 2023, the number of active nodes in the network has increased from 3 to these 7 locations ready for production:

The current nodes and some metadata should be accessible by the time of the 2023 Microsoft hackathon programmatically via a new Orcasound API.

Steps involved per location:

Add configuration file: see Port Townsend config for reference. Place new file in same directory.
Modify last line of Dockerfile to point to new config (NOTE: we should move away from having to bake the config file into the docker image so that we can build one image and specify the relevant configs externally).
Build docker container: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#building-the-docker-container-for-production
Push docker image to Azure Container Registry: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#pushing-your-image-to-azure-container-registry
Deploy to Azure Kubernetes Service: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#deploying-an-updated-docker-build-to-azure-kubernetes-service (create namespace, secret, deployment)

Need to check with @micowan on whether anything needs to be done for moderator portal.

Will have to look at the code again. I know we were looking at putting the hydrophone locations into a config, but don't know if that ever happened. Are the feed turned on for the new ones/are they creating records in the CosmosDB?
From: Michelle Yang @.>
Sent: Wednesday, August 2, 2023 6:24 PM
To: orcasound/aifororcas-livesystem @.>
Cc: Mike Cowan @.>; Mention @.>
Subject: Re: [orcasound/aifororcas-livesystem] Add new hydrophones to ML pipeline & document process (Issue #128)
Need to check with @micowanhttps://github.com/micowan on whether anything needs to be done for moderator portal.

Reply to this email directly, view it on GitHubhttps://github.com/orcasound/aifororcas-livesystem/issues/128#issuecomment-1663050815, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AFM7GJAFXX5UZW7TGG2JA6LXTLHQ7ANCNFSM6AAAAAA3ARCUKE.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

From the description, I don't believe the inference system has been brought up yet. So no records in Cosmos DB yet.
No additional handling needs to be done for inference system -> Cosmos DB, since Cosmos DB is really storing a blob of json which accepts any arbitrary string.

Checked the code. We did in fact turn the locations into a config setting last go round. So, adding from the UI perspective should be as simple as updating that config with the new locations. I know Scott showed a spreadsheet or api or something during the discussion last week that listed the locations. If they are updating that one themselves, and we can pull from that, we could make the list "live" vs a config setting. But that is just a thought.

Hey @micowan et al!  I see two possible routes to updating the config file, or more dynamically managing the ML pipeline:

The orcasite wiki lists a recent dump of the feeds table and I could update it this weekend for the hackathon
Recent Orcasound backend improvements make it possible to access the feeds table itself programatically, e.g. here -- https://beta.orcasound.net/graphiql via queries like:

{feeds
	{nodeName}
}

Also @micowan, I mentioned to @skanderm that your existing config file held JSON, so he said he could work on new API endpoint that could provide JSON to you...

You should be able to get an updated list here: https://beta.orcasound.net/api/json/feeds
You may need to set these headers as well:
curl -s -H "Content-Type: application/vnd.api+json" -H "Accept: application/vnd.api+json" https://beta.orcasound.net/api/json/feeds

@scottveirs and @skanderm, the url: https://beta.orcasound.net/api/json/feeds was absolutely perfect!
I have already added this to a new hydrophones endpoint in the API so that we can access it from the UI. I also brought in the url and html in case it makes sense to add them to the UI somewhere.
Thanks!!!

Glad you found it useful! Will the config be modifiable? We’re planning to deploy the changes to https://live.orcasound.net at some point.

If I am understanding the question correctly, yes. We will be able to change the URL we are pointing to on the fly by updating the configuration setting in Azure.

Good question!
Yes they are one and the same. But the official name for that feed is
indeed “ Orcasound Lab.” Since there will eventually be more than one
hydrophone node in Haro Strait, switching to “Orcasound Lab” would be
prudent long-term strategy.
Scott
On Tue, Sep 12, 2023 at 17:47 catskids3 @.***> wrote:

@scottveirs https://github.com/scottveirs and @skanderm
https://github.com/skanderm a quick question, there is a hydrophone
location you call Orcasound Lab, can you confirm that this the Haro Strait
hydrophone that we reference in the Cosmos DB. And if so, which is the
correct name/label? We may need coding/configuration changes on our end if
it is "Orcasound Lab".
—
Reply to this email directly, view it on GitHub
https://github.com/orcasound/aifororcas-livesystem/issues/128#issuecomment-1716773787,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLE3M7ZX5GFLZ57IDUZNDLX2D7DNANCNFSM6AAAAAA3ARCUKE
.
You are receiving this because you were mentioned.Message ID:
@.***>

OK. Great. Since we are changing the partition strategy, which requires as rebuild of the data set, I can take care of that one off during the migration. We will need to speak with @micya or @pastorep about how it is marked coming out of the ML pipeline. Thanks for the feedback and quick turnaround.

I found that location information is hardcoded in the inference system script: https://github.com/orcasound/aifororcas-livesystem/blob/2ed0955d062965b34d0db91b7d93ea1c2fef4e47/InferenceSystem/src/LiveInferenceOrchestrator.py#L36.
We should probably pull that out and configure it via an environment variable.

Michelle. Thanks for finding that. Also, if you are going to be changing the data port, we will want to incorporate the changes I requested earlier. i.e. remove the reviewed and SRKWFound properties (may have these spelled wrong) and replace with a new property called "state" which will be populated with the term "Unreviewed". "state" is also the new partition key. Also need a new property called "locationName" at the top level of the JSON that duplicates the name in the Location portion of the JSON.
Thanks.

@micowan and @micya --
@salsal97 is teaching David and I here in Redmond how to add the new Sunset Bay location to the ML pipeline.
If the new model deployment creates a candidate, will it show up in the Moderator portal auto-magically now? Or is there some hardcoding of the new location within the UI portal code? (i.e. "Sunset Bay metadata that's now available via the API provided by Skander).
It looks like your recent pull request, Mike, might be the answer my question?
Maybe Tara or someone else who knows C# could review the PR?

Once the new API is deployed (which won't be until after the Moderator UI
is updated), as long as the location appears in the site Skander provided
me, it will show up in the Moderator portal. I cannot speak to the ML
pipeline except to say Michelle indicated they had it hardcoded in one of
the python scripts. If they could call the new API as well, then it could
be removed from there and we would all be pulling from the same place. But
I don't know if that is a possibility from the ML side.
On Wed, Sep 13, 2023 at 6:45 PM Scott Veirs @.***>
wrote:

@micowan https://github.com/micowan and @micya
https://github.com/micya --
@salsal97 https://github.com/salsal97 is teaching David and I here in
Redmond how to add the new Sunset Bay location to the ML pipeline.
If the new model deployment creates a candidate, will it show up in the
Moderator portal auto-magically now? Or is there some hardcoding of the new
location within the UI portal code? (i.e. "Sunset Bay metadata that's now
available via the API provided by Skander).
It looks like your recent pull request
https://github.com/orcasound/aifororcas-livesystem/pull/131, Mike,
might be the answer my question?
Maybe Tara or someone else who knows C# could review the PR?
—
Reply to this email directly, view it on GitHub
https://github.com/orcasound/aifororcas-livesystem/issues/128#issuecomment-1718408659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6IJXCXNFHMONOWWTZJATDX2IZQ5ANCNFSM6AAAAAA3ARCUKE
.
You are receiving this because you commented.Message ID:
@.***>

This PR should be a step toward getting this issue squared out https://github.com/orcasound/aifororcas-livesystem/pull/136

Hi everyone! We've updated the live site. As referenced here: https://github.com/orcasound/aifororcas-livesystem/issues/128#issuecomment-1712531406, please update the endpoint to https://live.orcasound.net/api/json/feeds. Thank you!

@skanderm, Thanks for this. I have replaced the beta url with this new one in the codebase I am working.

Based off https://live.orcasound.net/listen, are there any new hydrophones available to add?

Scott gave me a URL last year: https://live.orcasound.net/api/json/feeds which has 7 hydrophones listed (including Haro Strait as Orcasound Lab). I have changed the API to pull this list for all Moderator features (picklists, etc.)

Before taking the steps that Michelle outlined, we need to fix a change that was recently made to the Amazon S3 buckets where the live audio data are stored. In the process of moving the data streams and archive to Amazon-sponsored buckets (and dramatically reducing our storage and egress costs), we had to rename the streaming data bucket.
The new name of the bucket from which OrcaHello should acquire data is audio-orcasound-net
My understanding is that the S3 bucket URI is hard-coded into the Docker images for each location. Ideally, we'd move the audio data source URI/URL outside of the image and into a configuration file.
The other place I see the S3 bucket name is hard-coded is here in the Orchestrator.py code -- https://github.com/orcasound/aifororcas-livesystem/blob/736c864e17dddf2da548ca04fcb16384968c9419/InferenceSystem/src/LiveInferenceOrchestrator.py#L145

Steps involved per location:

Add configuration file: see Port Townsend config for reference. Place new file in same directory.
Modify last line of Dockerfile to point to new config (NOTE: we should move away from having to bake the config file into the docker image so that we can build one image and specify the relevant configs externally).
Build docker container: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#building-the-docker-container-for-production
Push docker image to Azure Container Registry: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#pushing-your-image-to-azure-container-registry
Deploy to Azure Kubernetes Service: https://github.com/orcasound/aifororcas-livesystem/tree/main/InferenceSystem#deploying-an-updated-docker-build-to-azure-kubernetes-service (create namespace, secret, deployment)


BUG: Can't store callables using setitem

[X] closes #13299
[X] tests added / passed
[X] passes git diff upstream/master | flake8 --diff
[X] whatsnew entry

yeah that's a better soln

lgtm.

lgtm. @jorisvandenbossche

[Current coverage][cc-pull] is 84.34%

Merging [#13516][cc-pull] into [master][cc-base-branch] will increase coverage by <.01%

@@             master     #13516   diff @@
==========================================
  Files           138        138
  Lines         51107      51113     +6
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          43103      43109     +6
  Misses         8004       8004
  Partials          0          0

Powered by Codecov. Last updated by [bf4786a...2a6607a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/pydata/pandas/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/pydata/pandas/compare/bf4786a15f51d14ccf1318e44ae080868d8595ee...2a6607aae9119fbd02f25c61adf98bd1cb74b50a
[cc-pull]: https://codecov.io/gh/pydata/pandas/pull/13516?src=pr

Thanks a lot!


Full test coverage in B64LineBreaker

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

Project coverage is 84.99%. Comparing base (f985979) to head (4d6bca0).
Report is 2 commits behind head on main.

:exclamation: Current head 4d6bca0 differs from pull request most recent head 86e648a. Consider uploading reports for the commit 86e648a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #197      +/-   ##
==========================================
+ Coverage   84.77%   84.99%   +0.22%
==========================================
  Files          24       24
  Lines        1806     1806
==========================================
+ Hits         1531     1535       +4
+ Misses        162      160       -2
+ Partials      113      111       -2

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


Supply a example systemd file

Provide a dcrpool.service file

I used the following as a guide to craft up my systemd files.  YMMV.  I have dcrpool running from a non-privileged account and systemd will still start it up on a privileged port (443).   These configs are part of my test pool so I will be making these better as time allows.  But for now this appears to work.  Note: I added the output redirection config as all these services still dump messages to stdout for some reason.  I don't know why as I assumed systemd would have redirected all the output.
I created one file for each service (dcrwallet, dcrd, dcrpool).   Just note that dcrd and dcrwallet don't use homedir switch and instead use appdata switch.  I also moved the dcr binaries to /usr/local/bin directory.
/etc/systemd/system/dcrwallet.service
/etc/systemd/system/dcrd.service
/etc/systemd/system/dcrpool.service
Enabled the services
systemctl enable dcrwallet.service
systemctl enable dcrd.service
systemctl enable dcrpool.service
Started the services
systemctl start dcrwallet.service
systemctl start dcrd.service
systemctl start dcrpool.service
Example systemd file for dcrpool
[Unit]
Description=DCR mining pool web ui and stratum
After=network.target dcrpool.service

[Service]
Type=simple
User=decred
WorkingDirectory=/home/decred
ExecStart=/usr/local/bin/dcrpool --homedir=/home/decred/.dcrpool --configfile=/home/decred/.dcrpool/dcrpool.conf
Restart=on-failure
StandardOutput=file:/var/log/decred/dcrpool.log
StandardError=inherit
# Other restart options: always, on-abort, etc

# The install section is needed to use
# `systemctl enable` to start on boot
# For a user service that you want to enable
# and start automatically, use `default.target`
# For system level services, use `multi-user.target`
[Install]
WantedBy=multi-user.target


Don't return 1 if coverage isn't 100%

If you return 1 when there are any missed statements, we cannot use SharpCover on codebases that aren't 100% covered in CI environments that require all steps to return 0 to continue execution.

I'll make changes to the tests too, so the build passes.
What do you think about this change @gaillard?

I would rather see an argument to the check command that took a number (0-1) that the coverage must be over or at to give back a zero success code, defaulting to 1.0
However I had intended the selective ignoring of code to make the zero exit status the norm so that why and what is being ignored is documented, preferably in code via the line exclusion.

I see. Such argument would suffice in my use case. I still find this behavior very strange. I don't know any other code coverage measuring tool that enforces any percentage of coverage, certainly not by default.
I think it doesn't really make sense to maintain by hand an exclude (or include) list to show what's uncovered, when SharpCover already tells me what's uncovered (MISS!) in the report. The value of a code coverage measuring tool is to measure both what's covered and what's uncovered, so we can work towards increasing the overall coverage. If SharpCover enforces 100% coverage by default, it'll be harder to see the coverage improvements over time.
I understand that for some people the coverage enforcement may be useful, but I would argue that it shouldn't be the default.
What do you think @gaillard? I'll be working on adding the coverage threshold option to SharpCover, we just have to decide on whether it will be the default.

@caiofbpa I agree, it can be tedious for some and not how lots of people want to use, so just having what percentage makes a zero exit code would be nice.

@gaillard I'll be closing this since clearly this repo is abandoned


Add tool to check source allowlists

This PR provides an example tool for checking the impact of aa source allowlist on an executable module with debuginfo.
If no allowlist is provided, the "allow-all" default allowlist is used.
By default, it emits TSV to stdout of the form <function-name>\t<source-file>.
If -v/--verbose is passed, emits TSV of the form  <included>\t<function-name>\t<source-file>, where <included> is the string 1 of the file/function is included, and 0 otherwise.

Codecov Report

Merging #3246 (092597a) into main (9d9df7c) will decrease coverage by 6.44%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #3246      +/-   ##
==========================================
- Coverage   32.33%   25.90%   -6.44%
==========================================
  Files         180      152      -28
  Lines       22907    16383    -6524
==========================================
- Hits         7407     4244    -3163
+ Misses      15500    12139    -3361

see 332 files with indirect coverage changes


add versioning

You may find the full explanation of the workflow inside Versioning.cmake.
Please let me know if you need any other values in your version header.
Currently available are: major.minor.patch, git commit short hash, git branch name, release date.
Not sure, if it's the correct position, but one could also store website, email, and github link, next to the
library name inside the version header file. Then you have basically all meta-data describing the library in one file.
Purpose of this is kind-of two-fold: it's not only to have the values ready for display from the lib/app,
but also for base-versionizing the filenames of the continous releases. The VERSION file makes this easy.

I have the following versioning output:
[build] -- [ -- Versioning Overview -- ]
[build] --
[build] -- [Version] TTAURI_PROJECT_VERSION -> 0.2.3+build.efe96a2
[build] -- [Version] TTAURI_PROJECT_VERSION_SHORT -> 0.2.3
[build] -- [Version] TTAURI_PROJECT_MAJOR -> 0
[build] -- [Version] TTAURI_PROJECT_MINOR -> 2
[build] -- [Version] TTAURI_PROJECT_PATCH -> 3
[build] -- [Version] TTAURI_COMMITS_SINCE_TAG  -> 7
[build] -- [Version] GIT_TAG_RCS -> v0.2.3-7-gefe96a2becf722930178de3c8939d2e00973dd1d-dirty
[build] -- [Version] GIT_COMMIT_HASH -> efe96a2
[build] -- [Version] GIT_BRANCH -> versioning
[build] -- [Version] BUILD_DATE   (when compiled)  -> 2021-02-24
[build] -- [Version] RELEASE_DATE (when committed) -> 2021-02-23

Are these pieces of version information ok for you or are there any changes needed?
I'll remove BUILD_DATE, like requested.
Do you want to have the GIT_TAG_RCS as the TTAURI_PROJECT_VERSION?


Document dependency on<EMAIL_ADDRESS>

when using<EMAIL_ADDRESS>as documented in their readme, go run will fail compiling whereas go build will successfully compile.
https://github.com/second-state/wasm-learning/issues/41
DarumaDocker hinted to me that reactr depends on<EMAIL_ADDRESS>and this is the issue. I was not able to find this out without this hint though and more developers will face the same issue. Therefore it would be beniftial to document this.

You make a good point @chris-aeviator , we do not document this, however you can see the currently supported version in go.mod: https://github.com/suborbital/reactr/blob/main/go.mod#L15
I am going to move this issue to our docs repo, as that's where the change should be made.


bugfix: when use Statement.executeBatch() api can not generate after …

…image and undo log

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
fix when use Statement.executeBatch() api can not generate undo log
Ⅱ. Does this pull request fix one issue?

fixes https://github.com/seata/seata/issues/4833
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

我建议直接删了这个判断影响行数先吧，或者你可以进行标记batch的动作再来决定是否进行相关判断

Codecov Report

Merging #4838 (ea2ee64) into develop (350ce31) will increase coverage by 0.00%.
The diff coverage is 75.00%.

:exclamation: Current head ea2ee64 differs from pull request most recent head 57d960d. Consider uploading reports for the commit 57d960d to get more accurate results

@@            Coverage Diff             @@
##             develop    #4838   +/-   ##
==========================================
  Coverage      49.20%   49.21%
- Complexity      4090     4092    +2
==========================================
  Files            736      736
  Lines          25723    25725    +2
  Branches        3177     3178    +1
==========================================
+ Hits           12658    12660    +2
- Misses         11716    11718    +2
+ Partials        1349     1347    -2

Impacted Files
Coverage Δ

...ta/rm/datasource/exec/AbstractDMLBaseExecutor.java
59.25% <66.66%> (+4.54%)
:arrow_up:

.../rm/datasource/exec/mysql/MySQLInsertExecutor.java
69.47% <100.00%> (+0.32%)
:arrow_up:

...er/src/main/java/io/seata/server/ServerRunner.java
50.00% <0.00%> (-15.39%)
:arrow_down:

.../rm/datasource/exec/BaseTransactionalExecutor.java
55.92% <0.00%> (+0.65%)
:arrow_up:

...very/registry/zk/ZookeeperRegisterServiceImpl.java
61.76% <0.00%> (+0.73%)
:arrow_up:


5 lookup memory efficiency

@OlegIlyenko  I have the impression that the actual transform shouldn't throw but just return the error here. Is that correct?  I introduced the throw because without it it was not possible to find out the Error cause when the tests fail.
would fix #5

you are right, you shouldn't throw error. it may be fine within then, but for instance here:
https://github.com/sphereio/csv-mapper/pull/7/files#diff-c48457cd5d215f1321cd6290973c52f8R247
you should reject a promise instead of throwing an exception. Most of the codebase is using promises, so I would suggest to use promises here as well.

first part done, second later (I'm totally confused by the relationship of the Q() vs. node API promises / futrures etc. used).  It's probably a three-character thing for someone who is into the Q() library.

@OlegIlyenko  my brain is still not Q compatible again. I thought I can just wrap the calls into a Q() but the scope is not "this" any more.
    else
      Q().then ->
        @_setColumnIdx()
        @_buildLookupMap()
        this

feeling very dumb

You probably need to use fat arrow =>

@emmenko if it were ES6 yes. But it's coffee :-(

Coffeescript has -> and => as well (with the same semantics)

=> in coffeescript binds this

right. I should have RTFM.
side note: there seem to be very varying opinions of whether the fat/thin arrows in coffee & ES6 are actually the same:  https://www.airpair.com/coffeescript/posts/migrating-coffeescript-to-es6-javascript   ( this guy says: "ES6 JavaScript has no thin arrow function ->. Essentially, CoffeeScript -> == ES6 =>").
Anyhow, thanks for the help.

ak, ok, good to know! It's a pity, since i strongly believe that coffescript ⇒ should be the default... can't imagine working without it. If ES6 does not have it at all, then it probably should be very frustrating

I'm done now.  promises should be OK now and my lookup-critical job (joining category name and description translations into the categories import) is down from 1.4GB->out of memory to  between 200 and 320MB.

ping @OlegIlyenko  any opinions on whether to merge this?  (I'm just cleaning up old open issues, no specific urgent need).

@nkuehn sure, I'll review an mergit. Sorry, I missed your previous comment. Thanks for reminding me.

Thanks a lot for this improvement. I will release it with the next version shortly.

thank you!

Released: https://github.com/sphereio/csv-mapper/releases/tag/v1.0.9

+1


Strongly Lensed Supernova+AGN, Host Galaxy, and Lens Galaxy Population Models

Subtask of #56.
@dannygoldstein will provide @rbiswas4, @jbkalmbach, and @linan7788626 with OM10-style catalogs of lensed transient images, host galaxies, and lens galaxies.
Tagged folks, could you please let me know exactly what columns you require.

@dannygoldstein Hi Danny, as you know, I will be tied to something else at the beginning of January, but I can work on our project during the holidays. Therefore, I am wondering whether you could please send me a toy catalog (10 systems are enough)? So that I can kick off the work on my side asap. The below information should be included in the toy catalog, thank you so much.

the redshift of the sources (Supernovae, AGNs, Host Galaxies).
the relevant positions of point sources (Supernovae, AGNs) to the center of lenses.
the relevant positions of the host galaxies of point sources to the center of lenses.
the light profile of the host galaxies, e.g., components, effective radii (in arcsec), axis ratio, position angle, apparent magnitudes...

Closing this as the lens galaxy population model is handled by the sprinkler, and the host galaxy population model is provided by the DC2 cosmology sims.


don't require other approval for bors

This change removes the required approvals condition that was required by convention before we switched to our own organization.
With this change, more care has to be taken into account from maintainers (who have bors r+ privileges) when merging their own PRs, but this also allows quicker development when members are not available to review prs and sufficient thought has been taken into account. Ideally, "bad" changes would be caught by CI.
For the above reason, before merging this I think it'd be good if current @cross-rs/maintainers could sign off and approve the PR (leave a tick next to your name and possibly approve via review) before it's ultimately merged (we do not currently have a policy on how breaking changes or expectations are managed, so this is kinda arbitrary but takes from regular practice).

[x] @adamgreig
[x] @Dylan-DPC
[x] @Emilgardis
[x] @reitermarkus
[x] @svenstaro
[x] @therealprof

Fine with me.

we're in agreement then! Can someone bors r+ please, bors doesn't apply changes unless they are on the default branch so I can't do it myself just yet

I don't think bors cares so long as someone has approved it (it's just approving you can't do yourself) but still:
bors r+


fix: allow:phase and allow:request actions, extends related tests

It seems that allow:phase and allow:request actions are not working as expected, skipping only one rule and not the whole phase (or all the expected phases). Proposing a fix and extending related engine tests.

Because of https://github.com/corazawaf/coraza/pull/527, I would really appreciate @fzipi feedback, I hope to have not misinterpreted some of your logic 🙏


Remove indices and tables

I personally find the Indices and tables section clutter, so I suggest removing it. Feel free to ignore.
https://github.com/trchudley/pdemtools/blob/3b38cd395f110a7f8477ae6d7eab2d0ea508be4b/docs/index.rst?plain=1#L79-L85

Done, thanks!


Create text for Code of conduct

Translate and edit a first draft of Code of conduct

Code of conduct has been created - see: https://github.com/fremtind/jokul/blob/master/CODE_OF_CONDUCT.md


updates content migration to migrate v1-list as v2-markdown

Closes #1087

updated content migration to migrate v1-list as v2-markdown with list

p.s: we aren't fixing the existing data (deployed sites) as of now

I noticed that the line spacing is different between list elements and lists in markdown blocks:

I assume this difference is acceptable?


Can you make the Tweaks options searchable like all other options?

Thanks for the great extension.  Can you make the Tweaks options searchable?  Example:  When I go to Tools -> Options and use the "search options" filter for "save", I expect to see "Tweaks 2019" since it has an "Auto save" option, but it does not appear.

(from https://stackoverflow.com/questions/68592238)

@gundermanc shouldn't DialogPage properties be searchable by default? How do it make them searchable?

shouldn't DialogPage properties be searchable by default?

No, unfortunately from Tools > Options infrastructure perspective, pages are just a UI element, and we don't know any of the content within.
To make your page show up in search under a different term, you can add additional keywords to your ProvideOptionPageAttribute on your VS package.

I don't think that https://github.com/madskristensen/Tweakster/commit/23f8f3e9f8154f2fa275ac6245bfc82ea5eebd80 actually fixed this - I tried searching for some of the terms in the commit without any results pointing at Environment > Tweaks:

Tested under VS 2019 and 2022, both at latest versions.  Tested with the search box in the Options dialog, and in the Search everything (Ctrl-Q) box.


fix: default print styles for toggleables and gh-pages

:tada: This PR is included in version 4.9.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


dallastemperature with not 115200 baud

Hi,
If I a use nanpy with standard baud 115200 it works but if i set 9600 or 38400 or other don't work
port='/dev/nano'
baud=57600
connection = SerialManager(device =port , baudrate=baud)
a = ArduinoApi(connection=connection)
print "connesso"
lcd = Lcd([12, 11, 5, 4, 3, 2], [16, 4],connection=connection)
a.pinMode(8, a.OUTPUT)
a.pinMode(9, a.OUTPUT)
sensors = DallasTemperature(10)
print "dallas"
program still in sensors = DallasTemperature(10)
without error or timeout.

any update here?

You need to change your baudrate in SerialManager too
from nanpy import DallasTemperature
from nanpy import SerialManager

connection = SerialManager(device='/dev/cu.usbmodem141131', baudrate=57600)

sensor1 = DallasTemperature(4, connection=connection)
sensor2 = DallasTemperature(5, connection=connection)

print("There are %d devices connected on pin %d" % (sensor1.getDeviceCount(), sensor1.pin))
print("There are %d devices connected on pin %d" % (sensor2.getDeviceCount(), sensor2.pin))
addr1 = sensor1.getAddress(0)
addr2 = sensor2.getAddress(0)
sensor1.setResolution(9)
sensor2.setResolution(12)
while True:
    sensor1.requestTemperatures(0)
    sensor2.requestTemperatures(0)
    temp1 = sensor1.getTempC(addr1)
    temp2 = sensor2.getTempC(addr2)
    print("The temperature 1, in Celsius degrees is %0.2f" % temp1)
    print("The temperature 2, in Celsius degrees is %0.2f" % temp2)
    print("Let's convert it in Fahrenheit degrees: %0.2f" % DallasTemperature.toFahrenheit(temp1))

I'm experiencing the same problem. Changing baud rate doesn't actually change dallasTemperature's baud rate

Vous devez également modifier votre débit en bauds dans SerialManager
from nanpy import DallasTemperature
from nanpy import SerialManager

connection = SerialManager(device='/dev/cu.usbmodem141131', baudrate=57600)

sensor1 = DallasTemperature(4, connection=connection)
sensor2 = DallasTemperature(5, connection=connection)

print("There are %d devices connected on pin %d" % (sensor1.getDeviceCount(), sensor1.pin))
print("There are %d devices connected on pin %d" % (sensor2.getDeviceCount(), sensor2.pin))
addr1 = sensor1.getAddress(0)
addr2 = sensor2.getAddress(0)
sensor1.setResolution(9)
sensor2.setResolution(12)
while True:
    sensor1.requestTemperatures(0)
    sensor2.requestTemperatures(0)
    temp1 = sensor1.getTempC(addr1)
    temp2 = sensor2.getTempC(addr2)
    print("The temperature 1, in Celsius degrees is %0.2f" % temp1)
    print("The temperature 2, in Celsius degrees is %0.2f" % temp2)
    print("Let's convert it in Fahrenheit degrees: %0.2f" % DallasTemperature.toFahrenheit(temp1))


Need to copy new files in build folder to docs folder when using with GH actions

After building changes with
../goku/zig-out/bin/goku site -o build

There are new files which need to be copied to the docs folder for the site to show the new information even though the build succeeded on GH.

So, the command should be
../goku/zig-out/bin/goku site -o build && cp -r build/* docs/

I can submit a PR for a bash script to be executed when the programmer is ready to build and deploy.
Note: This is not necessary for localhost / hot reload; it is only necessary for deploying to GH pages through GH actions.

If you're using docs/ as a subpath for your GH pages content, you can adjust your goku command to output the generated site directly to the docs/ folder, rather than the build/ folder and save the copy step:
../goku/zig-out/bin/goku site -o docs


Change from one input field to three number pickers

Instead of one input field and tabLayout for timeUnit use three number pickers (hours, min, sec).

merged in feature branch.


feat: add DNS and TCP reports to test-connectivity

@amircybersec Check out this implementation.
I am able to get both TCP and DNS reports, while keeping the main test agnostic to the transport. (Thanks to your httptrace trick!)

Thoughts?

I added timing for DNS, but not TCP yet. It requires a map of net-address to start time.

@fortuna I like the proposed approach. It basically makes the context specific to a given dialer which can solve the concurrency issue and provide access to hostname. We can technically refactor this into a reporter dialer that can be used as the baseDialer. We can use the existing approach in complement to this and add do53-udp, do53-tcp, doh, dot, https, quic, etc test reports on the top of this. I am going to take this code for a spin 🚘

@fortuna I made a light refactor. I also added start time and duration to TCP connection.
Also, it seems like your branch is on the top of an older version that does not have the recent udp support for socks. I can do a rebase.
2024/09/05 22:03:29 Failed to create PacketDialer: config scheme 'socks5' is not supported for Packet Dialers

I've moved your changes to https://github.com/Jigsaw-Code/outline-sdk/pull/275.
I'm not touching any dependencies in my PR, so if it doesn't have UDP SOCKS5, it's because main doesn't have it.

Oh, nevermind, I started in an old main

Ok, my branch is not on top of main @ HEAD

Now with colored slog!


Where I can find netopeer2  documentation?

Where I can find netopeer2  documentation?

Hi,
it is mentioned in the sysrepo docs but, unfortunately, does not have its own documentation. However, there is not much to document from the user point-of-view, it is a NETCONF server and it behaves as a NETCONF server should.
Regards,
Michal

Thanks


Specify need  the checkmax function in custom configs

I noticed we say what check_max() does but forget to mention it needs to be copied into any custom config
@netlify /docs/usage/configuration

I'm hoping that we can remove this custom function very soon - there's a PR ready to merge in Nextflow that makes it redundant 🤞🏻

FINALLY

Attempt two @ewels !

@nf-core-bot fix linting

Phil just didn't like the duplicatoin, which is now gone so I guess that's sufiicent with Maxim'es approval :)


To parallelize data table execution

Created from the feedback received on getgauge/gauge#493
Run steps in a single spec, parallelly; providing a table at the setup. In that way I could save execution time for different inputs red from the csv table.

This is duplicate of #306. Closing this for now.


Include babel-polyfill in entry

Includes the babel-polyfill because we do that in build@4
@trunkclub/ui

babel-polyfill is big but we are already including it in our apps so... ehkay.


Filling areas between data sets?

Does anyone have a strategy to work within the existing XYSeriesRenderStyle line and area styles to fill the areas between data sets or between a data set and a specific (non-zero) y-axis value? I am trying to achieve effects similar to these:

Thanks!

Would love to see this implemented...


[BUGFIX] [v1.x] ElementWiseSum fix for oneDNN (#18777)

Cherry-pick from https://github.com/apache/incubator-mxnet/pull/18777

Fix ElementwiseSum for DNNL

Fix sanity and replace push_back with emplace_back

Change order of the data format conditions

Add NOLINT to avoid readability error

Add test for oneDNN ElemwiseSum

Description
(Brief description on what this PR is about)
Checklist
Essentials

[x] PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage
[x] Code is well-documented

@mxnet-bot run ci [website]

@anko-intel @pengzhao-intel


Code cleanup

basic update to readme
passes govet
files moved to root folder for easier import
updated tests

Thank you.


tesseract:arm-ios build failure

Host Environment

OS: MacOS
Compiler: llvm - xcode command line tools
Target: cross compile to ios arm or arm64

To Reproduce
Steps to reproduce the behavior:
./vcpkg install tesseract:arm-ios or ./vcpkg install tesseract:arm64-ios
Failure logs
  Package: tesseract:arm-ios
  Vcpkg version: 2020.06.15-unknownhash

From the error log:
CMake Error at CMakeLists.txt:540 (install):
  install TARGETS given no BUNDLE DESTINATION for MACOSX_BUNDLE executable
  target "tesseract".

Additional context
When i tried compiling without vcpkg, in cmake i needed this parameter:
CMAKE_MACOSX_BUNDLE:UNINITIALIZED=FALSE

This issue has not been updated for over 3 months, so if it is still an issue, please reopen the issue.


Skeema complains/skips tables when "USING BTREE" index clause present

Describe the bug
Skeema unexpectedly skips tables due to "unsupported features" when a SQL file on-disk contains USING BTREE for an index, but the corresponding table in the target database does not. As BTREE is the InnoDB default when no index is specified, Skeema doesn't need to do anything and in my opinion this difference in the diff should not cause the table to be skipped.
This was previously reported in #13 and then closed and reported fixed in c51eee3.
Yet it seems to be present in v1.5.2:

This is frustrating as some database management tools do include USING BTREE in their CREATE TABLE statements. PHPMyAdmin is being used in this case, which does append those clauses. The SQL files in the filesystem have it in their table definitions; the database I'm trying to skeema push to apparently does not, yet there are no other issues.
Environment

Skeema v1.5.2, amd64 build

MySQL 5.7

To Reproduce

SQL file with USING BTREE on an index definition.
Push to database with table version that lacks the clause.
Skeema skips and complains about unsupported features.

Expected behavior
Skeema should not skip tables where the only diff is missing USING BTREE on an index definition on the push target.

Arg the offending tables are actually MyISAM - I suppose that's the culprit here?

Yes, Skeema is primarily aimed at InnoDB tables. All other storage engines are best-effort only.
The functionality to strip USING BTREE clauses works correctly for InnoDB tables, and is covered by integration tests to ensure there's no regression.
Your best bet for a work-around is to strip these clauses using automation prior to running skeema diff or skeema push. For example, you could write a small wrapper script which uses sed. Alternatively, you could just skip these specific tables with an ignore-table regex.
That said, use of MyISAM in 2021 is discouraged fairly universally.


[Java] optimize map performance for reference tracking and serializer dispatch

Is your feature request related to a problem? Please describe.
This issue is to continue the performance optimization discussion of map in https://github.com/apache/incubator-fury/issues/1274#issuecomment-1997259888
Additional context
Tha hash lookup is a time-consuming operations. Especially after F applied codegen speedup, the cost of hash look for class dispatch or reference tracking is considerable, and sometimes it become the bottleneck of the serialization.

Hi @tommyettinger , let's continue the discussion in https://github.com/apache/incubator-fury/issues/1274#issuecomment-1997259888 .
We used 0.25 as the default loader actor for class serializer dispatch in https://github.com/apache/incubator-fury/blob/main/java/fury-core/src/main/java/org/apache/fury/resolver/ClassResolver.java#L201 . But for reference tracking, we used 0.51f, since the object graph may be huge, a smaller loader factor will use more memory, and then map is big, there may have some L1 cache miss. But I thinks we can change it to 0.5f.
For class serializer dispatch, some times some class are more hot than others, I was also thinking whether is it possible to adjust the map hierarchy to make it faster for hot keys. But not sure whether it will bring big difference.

Just found that fury has already removed the hash multiplication:
  protected int place(K item) {
    return System.identityHashCode(item) & mask;
  }

RE: removed multiplication, good, that was one of the optimizations I wasn't sure had made it into Fury.
I've got a JMH benchmark that can test several different Map implementations at once, but I can't run it at the moment because all 12 logical cores on this laptop are currently devoted to an unrelated statistical test on a random number generator, and that test will run for at least several more hours. I've got the JMH benchmark set up to test HashMap, ObjectObjectMap from jdkgdxds, the corresponding Map type in FastUtil, and the hash-based Map in Koloboke 1.0.0 . I'll see if I can add FuryObjectMap, it just needs to implement Map OK, it doesn't implement Map, so it would need to be run separately, which isn't quite a fair comparison. However, ObjectObjectMap from jdkgdxds is very close. It's possible Koloboke will do well here, which it appeared to do 8 years ago when it last updated, but a lot has changed in the current JDK versions. I'll see if I can test on a large number of Class instances, to have similar conditions to the large object graph scenario.

Looking forward to your benchmark results

I was able to run the benchmark, and I may have an avenue that could improve contains() and get() performance.

This was testing just contains() with Object keys, some of which were Class instances from java.** packages, and some of which were simple Dummy instances, where Dummy is a class that stores its name and that's it, using identity comparison like Class. The one case where anything performed noticeably better was when hashing the .toString() result of the Object (but .getName() on Class would also work), and I believe that only did well because the String and its hashCode are cached. That didn't do better every time, and some other operations took longer.
The full benchmark data on various operations is available here as a raw .txt:
https://github.com/tommyettinger/assorted-benchmarks/blob/529bd6a8e48ceb3fbbb7f0021f990394e1fd3cf6/jmh/Identity_Map_21.txt
and here as a nicely-formatted spreadsheet (.fods format from LibreOffice and I think OpenOffice): (direct link)
https://github.com/tommyettinger/assorted-benchmarks/raw/2538fcf164e27b3defc368436f8e2799225f75e9/jmh/Identity_Map_Performance_21.fods
The confusing thing is that there seems to be some deoptimization going on in the JVM, since the benchmarks each start about twice as fast as when they actually start recording data. It could be my laptop heating up quickly, but that seems unlikely.
You can run the benchmarks yourself by loading the jmh project in the repo I linked and running the shadowJar task. Then you can use a command like...
java -jar benchmarks.jar "JDKIdentityMapBench" -p impl=JDK_O2O_HASH,JDK_O2O_IDENTITY,JDKGDXDS_HASH,JDKGDXDS_TOSTR,JDKGDXDS_IDENTITY -p size=100,1000,10000 -wi 4 -i 5 -f 1 -w 5 -r 5 -p payloadType=OBJECT_UNIFORM

... which will run the Class-and-Dummy-Object test with the implementations for HashMap, IdentityHashMap, ObjectObjectMap, the new ObjectMapToString, and IdentityObjectMap, on sizes 100, 1000, and 10000, with 4 warmup iterations, 5 actual iterations, and each iteration taking 5 seconds. My apologies for how messy the benchmarking project is... It's mostly a testbed to see if something is worth investigating further.

OK, I've been thinking about this for a while, and tinkering a lot with different options. I was surprised to find that at least one version of cuckoo hashing really can perform quite well (though not as dramatically better on containsKey() performance), as long as it never encounters two different keys with the same exact hashCode(). I modified an existing Apache v2-licensed cuckoo hash map, reducing allocations as much as possible, and the performance gains are considerable on some operations:
POPULATE
IdentityCuckooMap  : 198676.832 ns/op
JDK IdentityHashMap: 508930.787 ns/op
CONTAINS
IdentityCuckooMap  :   1023.728 ns/op
JDK IdentityHashMap:   1418.562 ns/op
COPY
IdentityCuckooMap  :  28171.380 ns/op
JDK IdentityHashMap: 302847.095 ns/op
ITERATE
IdentityCuckooMap  :  27954.503 ns/op
JDK IdentityHashMap: 149939.553 ns/op

The catch here is that if a cuckoo hash "fails," it can enter an infinite loop or allocate memory forever, etc. Very bad worst-case properties. However, the internals of this particular cuckoo-hashed map and the existing FuryObjectMap are quite similar, and I'm working on an approach that will "flip" from using cuckoo hashing as long as it is viable, to using linear probing as FuryObjectMap currently does, if fully-colliding keys are present (or some other situation would force a rehash). I'm not sure what kind of speed penalty the flipping will impose; it is possible branch prediction will figure out that fully-colliding keys are extremely rare, and that would mean the cost would be very low relative to the above numbers. But, the code is larger for each method, which might be a problem for inlining. I'll have to benchmark and see... I know the JDK HashMap can also change its implementation to use a TreeMap-like Red-Black Tree if it encounters excessive collisions on Comparable keys, so this isn't an unprecedented model.
I'm testing the FlipMap code in my jdkgdxds repo for correctness, and I'll be benchmarking it using what should be similar code in my assorted-benchmarks repo.

Great, looks pretty promising! The caller is org.apache.fury.resolver.MapRefResolver#writeRefOrNull, would be nice if it can be inlined into this method.
I took a look at https://github.com/tommyettinger/assorted-benchmarks/blob/master/jmh/src/main/java/de/heidelberg/pvs/container_bench/FlipMap.java#L207. Would it be possible to seperate the branch less hit into a separate method, so the method body is smaller for inline. JVM has a max 325 bytecode size for inline be default. And in fury, we merged the put and get together into putOrGet to reduce two hash look up into one.

I've done the steps you described, and it has gone from not inlining (code too large) to inlining when hot! I keep finding various small things to change, but they're at least getting smaller. The last major change wouldn't have even affected a map with Class keys (I had accidentally left in some comparisons as == from when this was purely an identity-comparing map; they use equals() now). I'll look into putOrGet() next.
I have a nagging worry that some major part of this might not behave correctly. I have some tests from a very old version of Apache Harmony that I've been using for some data structures... If you happen to know of a better suite of tests for compatibility with Set and Map, that would be great! I know implementing the interfaces isn't critical for Fury since what's there now doesn't have to implement Map, but being able to run a standard set of tests makes me more confident these will work how they should.

I'm making progress on the rest of the map types now; IdentityMap seems done and ObjectIntMap might be done or close to it. Hybridizing the two shouldn't be hard at all. Is there a particular benchmark that you think would best test the performance of these map types? It looks like running all benchmarks would take well over a day.

Hi @tommyettinger , thanks for this hard work.
Fury has a src/main/java/org/apache/fury/benchmark/MapSuite.java which did some benchmarks for IdentityMap.
For end-to-end tests, src/main/java/org/apache/fury/benchmark/UserTypeSerializeSuite.java can be used.  You can use following config:
  public static void main(String[] args) throws IOException {
    if (args.length == 0) {
      String commandLine =
          "org.apache.fury.*UserTypeSerializeSuite.fury* -f 3 -wi 10 -i 10 -t 1 -w 2s -r 2s -rf csv";
      System.out.println(commandLine);
      args = commandLine.split(" ");
    }
    Main.main(args);
  }

org.apache.fury.benchmark.state.BenchmarkState#references should be set to true only, otherwise referecne tracking won't be enabled:
  @Param({"true"})
  public boolean references;

Well, the FlipMap turned out to be significantly slower than all other maps I tried, due to an error I made when writing put() and related code. With put() fixed so it actually puts a key and value in place, FlipMap is quite slow... I've been thinking about alternatives, though.
I've never used ClassValue, myself, but could it maybe be possible to avoid any Class-based hashtable if a ClassValue was registered per-Fury-instance that stores (somehow) a registered ID for a Class, in the Class itself? I also looked into hacky ways that might allow sun.misc.Unsafe to store an id directly in a Class, but even if that could work, it doesn't allow multiple Fury instances to have different registered classes. ClassValue might be able to do it, but I really have no idea what I am doing with that code.

What do you mean use class value? You mean use it to cache a I'd for class? This will have a hash look up cost too


If --hass-discovery and --mqtt-topic are provided, send both types of message

I want to use both mqtt2prometheus and Home Assistant to listen to these messages. The "normal" style messages are much easier to handle with mqtt2prometheus than the Home Assistant style.
It would be nice if ecowitt2mqtt could send both types of message if I request it to. At the moment it seems that the --hass-discovery flag disables the normal-style messages.

How would you propose configuring this scenario? It seems strange that including both options would mean "publish to the same broker in two different formats."

Well, at the moment if the --hass-discovery option is supplied then it takes precedence, which is not very clear from the documentation.
The Home Assistant-style messages are verbose and annoying to consume with most other MQTT consumers. The standard method, with all the data in a single message, is a lot easier. I just remembered I have another consumer which sends off a packet of weather data to a service elsewhere - if I had to adapt that to use the Home Assistant style of messages, it would significantly increase the complexity because I'd have to wait to receive all the messages before sending it off.
I don't really see the problem with sending both types of message if --hass-discovery is enabled - it's hardly any additional data compared to the Home Assistant messages. I can always fork and add this myself but I suspect I'm not going to be the only person with this issue.

I don't really see the problem with sending both types of message if --hass-discovery is enabled - it's hardly any additional data compared to the Home Assistant messages.

The challenge isn't related to resources—it's related to the configuration "interface". I don't think most users would find it clear that providing both flags means that both formats are published (especially if we have other formats over time).
That said, your overall request is certainly reasonable. I think on the right way to do it—I'm thinking that expanding the configuration file to allow for multiple configurations for the same gateway might be optimal.

I don't really see the problem with sending both types of message if --hass-discovery is enabled - it's hardly any additional data compared to the Home Assistant messages.

The challenge isn't related to resources—it's related to the configuration "interface". I don't think most users would find it clear that providing both flags means that both formats are published (especially if we have other formats over time).
That said, your overall request is certainly reasonable. I think on the right way to do it—I'm thinking that expanding the configuration file to allow for multiple configurations for the same gateway might be optimal.

I fully expect the data to published both ways. Using other mqtt integrations as examples, valetudo publishes both to the homeassistant topic as well as whatever I've defined and the same applies for rtlamr2mqtt. I'm only just setting up Home Assistant, but I'd like to be able to fall back to any other setup I currently have for consuming the mqtt data such as Node-RED or a tsdb

FYI, this was released in 2023.06.0.


Incorrect instruction

Is item 3 in this section valid? https://docs.microsoft.com/en-us/azure/migrate/set-discovery-scope#work-around-for-server-folder-restriction
There doesn't appear to be a documented method to complete this action.

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

ID: 928360b6-2db9-0890-2029-067b09835223
Version Independent ID: 52484741-51e7-c552-0de3-33d070a146b1
Content: Set the scope for discovery of servers on VMware with Azure Migrate - Azure Migrate
Content Source: articles/migrate/set-discovery-scope.md
Service: azure-migrate
GitHub Login: @vineetvikram
Microsoft Alias: vivikram

Hello @joecarlyle , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

This step is detailed at below two locations:

 What is Azure Migration Planner? - NET
Azure Migrate Full Guide

Let me know if this answers your question.

We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

More details about this step can be found here:

 What is Azure Migration Planner? - NET
Azure Migrate Full Guide

Let me know if this answers your question.

Sorry neither of these are valid, the first is a service that does not exist. The second includes the same limited detail as the docs.


Lesson 8 and fixed the broken links

@jlooper please confirm I inserted the quiz links properly. thanks!


Release notes for April 29 release

This release includes a patch update to the firestore-bigquery-export extension with some bug fixes.

@taeold apologies for not noticing this sooner, but if the only PRs in this release are for the BQ import and gen-schema-views scripts, then we don't need to bump the versions of any extensions. We should still sync next to master to get the histories realigned and get the latest code for the scripts into master

Closing per @jhuleatt's instructions. Will manually publish the import and gen schema scripts.


[feature request] Add a new api "POST /containers/upgrade" to upgrade a container

Ⅰ. Issue Description
In some cases, if a user has one old container with old image, user needs to upgrade this container with new image but keep some old things of the old container. For example, the network configuration, the ssh_keys and something else could be kept to the new containers.
More details on demand will be added.
Ⅱ. Describe what happened
We can not upgrade a container with new image but keep the old configurations and so on.
Ⅲ. Describe what you expected to happen
We can upgrade a container with new image and keep the old configurations and so on.
Ⅳ. How to reproduce it (as minimally and precisely as possible)

Ⅴ. Anything else we need to know?
Ⅵ. Environment:

pouch version (use pouch version):
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

define an method like: UpgradeContainer(id string, config Config, hostConfig HostConfig), whose last two parameters are the same as CreateContainer.
in the method found the container that will be upgrade by the id, we call it old_container.  Firstly we rename the old_container to a name which adding a suffix _bak to the original name and stop the old_container, then compare and merge the Config and HostConfig with those of the old_container and use the merged Config and HostConfig to create a new container, the new container will VolumeFrom the old_container. if the new container is running properly after start, we remove the old_container. otherwise we rollback the upgrade. which means we remove the new container and start the old container.

Since #852 and #923  have merged, i think we can close this issue.
@yyb196 you  can check upgrade command now, if has any problem,  please let me know at any time.


What's the possible reasons about the wrong image inferenced by my own model?

I used monet2photo downloaded from https://github.com/junyanz/CycleGAN
the trainning step is 9100,
the other parameters are all default,
and the result looks like this

What's the possible reasons?

By default, checkpoints are autosaved every 10000 steps. One possible reason is you exported the model too soon then the exported model actually just trained by only 1 step instead of 9100 steps.
By the way, 9100 steps are not enough in most cases. You can check Tensorboard to see the progress.

wow, thank you so much for the answer, I will try it again.


daemon/containerd: fix compat for "docker run" with image that's not present

It's horrible, but some clients (such as the docker 17.03 CLI used in CI) still perform string-matching to detect that the image wasn't found. This requires the error to be prefixed with "No such image:". Without this prefix, the CLI prints the error message, and exits.
Before this:
docker run -it --rm hello-world
docker: Error response from daemon: id not found.
See 'docker run --help'.

After this:
docker run -it --rm hello-world
Unable to find image 'hello-world:latest' locally
faa03e786c97: Download complete
...

- A picture of a cute animal (not mandatory but encouraged)

github.com/containerd/containerd/vendor/github.com/miekg/pkcs11
#50 293.6 cgo: malformed DWARF TagVariable entry

Looks like we need https://github.com/moby/moby/pull/44571

opened https://github.com/rumpl/moby/pull/123 to fix CI

Rebased after https://github.com/rumpl/moby/pull/123 was merged; CI should (hopefully) go green now 👍

All green 🎉 bringing this one in 👍

This should be upstreamed together with

https://github.com/rumpl/moby/pull/88
https://github.com/rumpl/moby/pull/8


Add ExampleDatasetEqualityOperator

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (86d58a6) 100.00% compared to head (a5a26cf) 100.00%.

:exclamation: Current head a5a26cf differs from pull request most recent head a428e08. Consider uploading reports for the commit a428e08 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #284   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          318       318
  Lines         9888      9908   +20
  Branches       951       956    +5
=========================================
+ Hits          9888      9908   +20

Impacted Files
Coverage Δ

src/gravitorch/data/datasets/__init__.py
100.00% <100.00%> (ø)

src/gravitorch/data/datasets/example.py
100.00% <100.00%> (ø)

src/gravitorch/data/datasets/factory.py
100.00% <100.00%> (ø)

src/gravitorch/datasources/dataset.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Spike: Investigate inline widget support

Super Editor needs to support inline widgets.
Examples:

Typing a pattern might convert into a small icon, e.g., :fire: -> :fire: - when it's an emoji, an inline widget isn't needed, but the desired image might be a small bitmap
A checkmark or progress indicator at the end of a line. Positioning such a visual in a Row won't work because if the text is multiline, we don't want the earlier rows of text to leave any space at the end of the line.
Avatar images that replace a given user name, e.g., converting @SuprDeclarative to a small avatar
Inline code (I'm not sure if this should be solved with an inline widget or styled text with a custom painter behind it)

These use-cases involve multiple, independent concerns:

Rendering arbitrary content within text
Adjusting AttributedText and its associated rendering behavior to allow for non-text content
Selection dynamics:

What happens when the user taps somewhere in the inline widget?
What happens when the user drags part of the way into the inline widget?
What happens when the user drags across the inline widget?
What happens when the user presses backspace at the end of the inline widget?
What happens when the user presses delete at the beginning of the inline widget?
What happens when the user presses arrow keys into/out of the inline widget?

This ticket is a spike to investigate the available level of control that Flutter provides for rendering inline widgets and how much access we have to the inline widgets' bounds an other properties. By the end of this ticket, we should be in a position to propose tacit answers to the questions above.

Inline code (I'm not sure if this should be solved with an inline widget or styled text with a custom painter behind it)

There is one limitation to the custom painter behind inline code. I believe there is no way to add some extra padding around the inline code span to allow room for the custom decoration (background and/or border).
Ideally, it would be nice if TextSpan allowed to specify such customizations, e.g. additional padding around the span and BoxDecoration to paint behind it. I looked at the C implementation which is almost 1-1 based on SkiaText and such customizations are not supported there.
So WidgetSpan is probably the only available option here. Which should work fairly well for the inline embed case where embedded content is indivisible and always 1 character long. Reporting text metrics using TextPainter should work out of the box. But for inline code it'll require a bit more work since user generated events (hitTest, selection, cursor movement) will need to be propagated down to the inline widget and back to the user.

@pulyaevskiy you might be right about that. How would one handle an inline code sample that needs to break across lines? A WidgetSpan is forced to render a single block within the text, right?

Oh, you're right. This was actually the reason I gave up trying to fit inline code in WidgetSpan.
So, yeah, not a lot of options here in the end. It is worth exploring if C implementation of Flutter's Paragraph can be extended to allow extra (horizontal) padding around a text span.

Another possibility is for Flutter to support another type of Widget span, or perhaps a RenderSpan that allows the span to construct multiple rectangles of content. But either way, it probably needs Flutter's intervention.

Here's a write-up for this ticket: https://github.com/superlistapp/super_editor/discussions/392


Content Security Policy prevents page from loading.

Due to timelinejs's heavy dependency on external resources, content security policies are preventing both the font.scss file and the timeline.js file from loading and acquiring the needed resources.

It was later discovered, this was not the primary issue with content not loading.