[Bug/Question] UnreadBits not zero after reading all data from Notify message

When receiving a Notify message I noticed that UnreadBits is often not zero after reading all available data. I am unsure of this is intended - maybe messages include some footer? Seems to only happen when sending over the net. Here is a super tiny reproducible example:
// Client
var message = Message.Create(MessageSendMode.Notify);
message.AddInt(42);
SendMessage(message);

// Server
var val = message.GetInt();
var unreadBits = message.UnreadBits; // sometimes 3, sometimes 4, sometimes 11, seems slightly random

It's not so much "intended" as it is expected. Bytes are the smallest unit of data that you can send over the network, so if a message consists of 60 bits (7.5 bytes) it will be sent as 8 bytes, with the last 4 bits being empty "filler" data. The receiving end will see that it received 8 bytes and set the writeBit (which is used to calculate UnreadBits) to 8 bytes * 8 bits per byte = 64 bits, even though the last 4 bits technically shouldn't be interpreted as data.
To prevent this from happening, senders would have to write the total number of usable bits into the message itself (near the beginning) so the receiver can read it and know exactly where the data ends. I figured this extra overhead wasn't worth it, especially since it's easy to implement yourself if you really want/need it. Additionally, you probably shouldn't be relying on there being a specific number of bits left over for any logic decisions, as a malicious client could send more or less bits (or manipulate/lie about the length value) and thereby cause problems.
The number of "extra" UnreadBits  that remain after you read out all your data depends on how many bits it takes to round up to the nearest byte, meaning that you should never be seeing more than 7 "extra" bits on the receiving end. The fact that you say it's "sometimes 11" is very odd and shouldn't be happening unless you haven't read out all your "real" data...
In the example you gave, UnreadBits should always be returning 4. If it's changing/fluctuating then that's also concerning.

Thank you! Lack of knowledge from me, I just assumed network data was sent as bits, now the padding makes sense. And I was not using it for any game logic, it was just something I noticed when I added some sanity-checks around the code, one of them checking that all data was read. I'll just change the check to account for this padding. Cheers!

"sometimes 11" is very odd and shouldn't be happening unless you haven't read out all your "real" data
Given my silly typo in my other issue I think this was also just user error, I went through and checked all my current messages and when they arrive they never have more than 7 bits of padding. My bad again 🙏


Can't download on Windows: ERROR: ENOENT: no such file or directory

C:\Users\Administrator>where aria2c
d:\program files\!cli\aria2c.exe

C:\Users\Administrator>where ffmpeg
C:\ProgramData\chocolatey\bin\ffmpeg.exe

C:\Users\Administrator>bilibili-get https://www.bilibili.com/video/av29332332
extracting video metadata...
[1/1] av29332332: start download "某伊藤姓高颜值特摄女饭庆生活动现场 伊藤美来201
7生日live"...
[1/1] av29332332: video quality:
[1/1] av29332332:     74      flv720_p60     高清 720P60
[1/1] av29332332:     64      flv720         高清 720P
[1/1] av29332332:   * 32      flv480         清晰 480P
[1/1] av29332332:     15      flv360         流畅 360P
[1/1] av29332332: output format: flv
[1/1] av29332332: output path:
[1/1] av29332332:     C:\Users\Administrator\av29332332 某伊藤姓高颜值特摄女饭庆
生活动现场(1) 伊藤美来2017生日live.flv
[1/1] av29332332: downloading video segment 1/8...
[1/1] av29332332: downloading video segment 2/8...
[1/1] av29332332: downloading video segment 3/8...
[1/1] av29332332: downloading video segment 4/8...
[1/1] av29332332: downloading video segment 5/8...
[1/1] av29332332: downloading video segment 6/8...
[1/1] av29332332: downloading video segment 7/8...
[1/1] av29332332: downloading video segment 8/8...
[1/1] av29332332: merging segment files...
ERROR: ENOENT: no such file or directory, unlink 'C:\Users\Administrator\av29332
332-1.flv'

mark

can not reproduce this issue. please check the directory which files are actually downloaded.
if you use anti-virus program or ransom protection, they would prevent from modifying your files under C:\Users.

Windows 7 user here.
This issue should be already fixed by now thanks to #14, which is likely why you couldn't reproduce it. However for some obscure reason, executing npm i -g bilibili-get on my machine fetches the old versions of the files where the downloader subprocess never gets spawned, so the merger miserably fails to find the expected files hence this error.
I've checked my package.json and it does say<EMAIL_ADDRESS>but the files are from an earlier version. Any clue? Or might just as well manually overwrite the files.

#14 is not merged in 1.5.1
It's scheduled for 1.6.0

Oh I see. Just noticed indeed that 1.5.1 was released in October while #14 only got fixed a month later. My mistake 😅. Thanks for the head-up! Looking forward to the next release of this awesome tool.

#14 introduces a filename quote issue needs to be fixed before next release.

Hi. I just meet the same problem. Looking forward to the 1.6.0.


fix(functions): edge-runtime: apply per-function configs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

53 of 103 (51.46%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 60.417%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

internal/functions/serve/serve.go
53
103
51.46%

Files with Coverage Reduction
New Missed Lines
%

internal/functions/serve/serve.go
3
56.83%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
4170

Relevant Lines:
6902

💛  - Coveralls


Wavenumber page not in toctree

#67 introduced the wavenumber page (wave_number.rst) inside of /Users/lindseyjh/git/geosci/em/content/maxwell1_fundamentals/physical_principles/maxwells_equations_in_homogeneous_media that is not currently in a table of contents tree.
The content should be included, as relevant, to the wave propagation in free space (topic of the pull request #58) @sdevriese, you may want to take a look at this file and see if the content @micmitch generated meshes with what you have done

@sdevriese, @micmitch : has this been resolved?

The things Mike wrote were in the pages I created.

Hi Sarah,
            Can you drop up?

Thanks
Doug
From: sdevriese<EMAIL_ADDRESS>Sent: May-11-16 2:30 PM
To: ubcgif/em
Subject: Re: [ubcgif/em] Wavenumber page not in toctree (#71)
The things Mike wrote were in the pages I created.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/ubcgif/em/issues/71#issuecomment-218596120 https://github.com/notifications/beacon/AIdCvByqqxjQvaB8M42gk6cUBdsmY3fSks5qAkpdgaJpZM4HZYmk.gif


Subcribe more than 1 symbols

i think we had some problem like i want to get subscribe 2 symbol,and only getting return 1 item

duplicated #2


reverting 3362

This addresses #3481 and will revert #3362 back to its prior state. We'll likely want to reopen #3359 since this will reintroduce the bytes/text issue.

I'm good with this. How about you @Lukasa? (To be honest, I would have rathered let GitHub generate the revert than have it done artisanally.)

I'm fine with this too.

@sigmavirus24 sorry, I tried the revert button and got an error so did it by hand.


GetGLContextInfoKHR() crashes (Windows)

I've worked on an application that make an OpenGL/OpenCL interop on linux (ubuntu 14.10) using Boost.Compute. The application was running fine on linux until the nvidia drivers update.
The application uses :
GLFW3, MATIO, GLEW, Boost.Compute, clFFT, LUA5.2
Now I'm porting the application on Windows 8.1 but the code working perfectly on linux just crashes on windows.
IDE : Visual Studio 12 (2013) 64bits
Graphic cards tested :

NVIDIA GeForce 750M (laptop)
NVIDIA Titan X (workstation)

I've tried
GLEW, GL3W, glloadgen, glad for OpenGL
and
intel INDE 2015 Update 2, NVIDIA (CUDA 7.0) for OpenCL
VS gives me :
Unhandled exception at 0x00000000CCD82A90 in Hologram.exe: 0xC0000005: Access violation executing location 0x00000000CCD82A90.
The breakpoint is set to this line in context.hpp
cl_int ret = GetGLContextInfoKHR(
            properties,
            CL_CURRENT_DEVICE_FOR_GL_CONTEXT_KHR,
            sizeof(cl_device_id),
            &gpu_id,
            0
        );

from the function compute::opengl_create_shared_context();
Locals :
+		gpu	{m_id=0xcccccccccccccccc {...} }	boost::compute::device
		ret	-858993460	int
		gpu_id	0xcccccccccccccccc {...}	_cl_device_id *
		GetGLContextInfoKHR	0x00000000ccd82a90	int (const __int64 *, unsigned int, unsigned __int64, void *, unsigned __int64 *) *
+		properties	0x00000084e77bf1a8 {4228,<PHONE_NUMBER>60, 8200, 65536, 8203, 285289982, 0}	__int64[7]
+		platform	{m_platform=0x00000084e9b5efa0 {...} }	const boost::compute::platform &
		i	0	unsigned __int64
+		cl_gl_sharing_extension	0x00007ff75455d7f8 "cl_khr_gl_sharing"	const char *
+		platforms	{ size=2 }	std::vector<boost::compute::platform,std::allocator<boost::compute::platform> >

CallStack :
 	00000000ccd82a90()	Unknown
>	Hologram.exe!boost::compute::opengl_create_shared_context() Line 116	C++
 	Hologram.exe!Interop::lua_initKernel(std::basic_string<char,std::char_traits<char>,std::allocator<char> > kernelName) Line 141	C++
 	Hologram.exe!Interop::clua_initKernel(lua_State * L) Line 9	C++
 	lua52.dll!lua_getinfo() + 4562 bytes	Unknown
 	lua52.dll!00007ffcc6419846()	Unknown
 	lua52.dll!lua_callk() + 374 bytes	Unknown
 	lua52.dll!lua_getinfo() + 2962 bytes	Unknown
 	lua52.dll!lua_yieldk() + 212 bytes	Unknown
 	lua52.dll!lua_pcallk() + 388 bytes	Unknown
 	Hologram.exe!LuaWrapper::run() Line 48	C++
 	Hologram.exe!main() Line 181	C++
 	[External Code]

EDIT: Simplified version of main
#include <iostream>
#include <stdlib.h>
#include <stdio.h>
#ifdef _WIN32
  #define GLFW_DLL
#endif
#include <GL/glew.h>
#include <GLFW/glfw3.h>
#include "Interop.h"

int main(void)
{
  GLFWwindow* window;
  if (!glfwInit())
    exit(EXIT_FAILURE);
  glfwSetErrorCallback(error_callback);

  const GLFWvidmode * mode = glfwGetVideoMode(glfwGetPrimaryMonitor());
  width = mode->width;
  height = mode->height;

  window = glfwCreateWindow(width, height, "Hologram", glfwGetPrimaryMonitor(), NULL);

  if (!window)
  {
    glfwTerminate();
    exit(EXIT_FAILURE);
  }
  glfwMakeContextCurrent(window);

  // Initialise GLEW
  if (glewInit() != GLEW_OK) {
    fprintf(stderr, "Failed to initialize GLEW\n");
    abort();
  }

  glfwSwapInterval(1);
  glfwSetKeyCallback(window, key_callback);
  glfwSetCharCallback(window, char_callback);
  glfwSetMouseButtonCallback(window, mouseButton_callback);
  glfwSetScrollCallback(window, scroll_callback);
  glfwSetCursorEnterCallback(window, cursorEnter_callback);

  Interop* interop = NULL;

  while (!glfwWindowShouldClose(window))
  {
    glfwPollEvents();
    int width, height;
    glfwGetFramebufferSize(window, &width, &height);
    if (menus->isSetFlag(menus->flag_hologramChanged)) {
      delete interop;
      interop = new Interop(width, height);
      [...]
    }
[...]
  }
  glfwDestroyWindow(window);
  glfwTerminate();
  delete interop;
  exit(EXIT_SUCCESS);
}

Thanks.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26956763-getglcontextinfokhr-crashes-windows?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F402515&utm_medium=issues&utm_source=github).

Bump, I have exactly the same problem. The function "GetGLContextInfoKHR" (clGetGLContextInfoKHR) is crashing. I thought at first it was due of the shitty opencl implementation of nvidia, but even on a amd, It doesn't work.
I have tested theOpenCL-OpenGL sample from Nvidia and it work perfectly (they don't use clGetGLContextInfoKHR:http://developer.download.nvidia.com/compute/cuda/4_2/rel/sdk/website/OpenCL/html/samples.html) .
IDE : Visual Studio 14 (2015) 64bits
Graphics Cards tested:
Quadro 2000
Geforce gtx880m
AMD HD7970

Ohhh, I have similar problem on Intel HD Graphics 530. How to solve it?

@dPavelDev
I just remove the clGetGLContextInfoKHR:
compute::context createOCLContextFromOGLContext(const compute::device& device, const OGLContext& oglContext) {
		compute::context context;
		cl_device_id rawDevice = device.get();
		GLFWwindow* previousContext = glfwGetCurrentContext();
		oglContext.enable();

		#if defined(__APPLE__)
			// get OpenGL share group
			CGLContextObj cgl_current_context = CGLGetCurrentContext();
			cl_context_properties properties[] = {
				CL_CONTEXT_PROPERTY_USE_CGL_SHAREGROUP_APPLE,
				(cl_context_properties)CGLGetShareGroup(cgl_current_context),
				0
			};

			cl_int error = 0;
			cl_context cl_gl_context = clCreateContext(properties, 0, 0, 0, 0, &error);
			if (!cl_gl_context)
				throw OCLException(compute::opencl_error::to_string(result));

			context = compute::context(cl_gl_context, false);
		#else
			cl_context_properties properties[] = {
				CL_CONTEXT_PLATFORM, (cl_context_properties)device.platform().id(),
				#if defined(__linux__)
                    CL_GL_CONTEXT_KHR, (cl_context_properties)glXGetCurrentContext(),
                    CL_GLX_DISPLAY_KHR, (cl_context_properties)glXGetCurrentDisplay(),
				#elif defined(WIN32)
                    CL_GL_CONTEXT_KHR, (cl_context_properties)wglGetCurrentContext(),
                    CL_WGL_HDC_KHR, (cl_context_properties)wglGetCurrentDC(),
				#endif
				0
			};

			if (!device.supports_extension("cl_khr_gl_sharing"))
				throw OCLException("cl_khr_gl_sharing is not supported by the device");

			context = compute::context(device, properties);

		#endif

		oglContext.disable();
		glfwMakeContextCurrent(previousContext);

		return context;
	}

OGLContext is just a wrapper for GLFWwindow (GLFW).
I didn't test this code on MacOS, and now on Linux my intel HD don't have the cl_khr_gl_sharing extension to test it.

Ops.. What is it? In file context.hpp:
GetGLContextInfoKHRFunction GetGLContextInfoKHR =
            reinterpret_cast<GetGLContextInfoKHRFunction>(
                reinterpret_cast<unsigned long>(
                    platform.get_extension_function_address("clGetGLContextInfoKHR")
                )
            );

When I replace it:
GetGLContextInfoKHRFunction GetGLContextInfoKHR =
            reinterpret_cast<GetGLContextInfoKHRFunction>(
                //reinterpret_cast<unsigned long>(
                    platform.get_extension_function_address("clGetGLContextInfoKHR")
                //)
            );

This works fine on my machine (Windows 10 x64, MSVC 2015 x64). Who wants to answer this question: Why do we need this construction? MSVC swear on it.

sizeof(unsigned long) == 4 in MSVC, which is not enough to hold a pointer. Possible solution could be to switch to size_t in reinterpret_cast.

Yes, it is bug!

@dPavelDev, can you make a pull request for it or should I do it?

Yes, I've already created a pull request.

Great! Thanks! :+1:

Fixed in PR #636. Thanks @dPavelDev!


[AVM Module Issue]: dns-zone nameServers output

Check for previous/existing GitHub issues

[X] I have checked for previous/existing GitHub issues

Issue Type?
Feature Request
Module Name
avm/res/network/dns-zone
(Optional) Module Version
0.3.0
Description
We would like to get the nameServers property in the module output in order to create NS delegation records in another zone.
(Optional) Correlation Id
No response

Apologies for the delay in picking this up, will get this updated and published this week


Add binop to past

Implementation of past.BinOp to e.g. accept operation within domain bounds:
    @field_operator(backend=fieldview_backend)
    def fieldop_domain(a: Field[[IDim, JDim], float64]) -> Field[[IDim, JDim], float64]:
        return a + a

    @program
    def program_domain(
        a: Field[[IDim, JDim], float64],
        lower_i: int64,
        upper_i: int64,
        lower_j: int64,
        upper_j: int64,
    ):
        fieldop_domain(a, out=a, domain={IDim: (lower_i - 1, upper_i), JDim: (lower_j, upper_j)})

    program_domain(a, lower_i, upper_i, lower_j, upper_j, offset_provider={})

Duplicated by #989

@samkellerhals Please push your changes to this PR as I already did a review here.

@samkellerhals Please push your changes to this PR as I already did a review here.

All my changes are here now.


IWord needs to contain the number of times a token occured

IWord.getTokens() -> Set. However we want to get a count of the occurrence of each of this token. This is a requirement to check if an IWord is a proper noun.

I'll do it


bigip-fast-templates/microsoft_iis TLS Certificate & Key drop-down boxes do not show my cert

Environment

FAST RPM Version: 1.6.1
BIG-IP Version: <IP_ADDRESS>

Summary
Cannot select new TLS certificates in the microsoft_iis template (template only offers the default pre-installed F5 certs).
It works as expected (I can select my cert) if I create the configuration using the "bigip-fast-templates/http" template instead of the iis template.
This problem was repeated on a Virtual Edition as well as on an i2800 physical appliance running the same software version.
Steps To Reproduce
1 - Import your cert via the "System >> Certificate Management >> Traffic Certificate Management" page
2 - Go to "iApps >> Application Services >> Applications LX" and click on "F5 Application Services Templates"
3 - Click on "Deploy"
4 - Select "bigip-fast-templates/microsoft_iis"
3- Leave "FAST-Generated TLS Server Profile" enabled and "try to" chose your cert and key under the TLS cert Certificate and TLS Key sections (only the default F5 certs show up).
Expected Behavior
I expect to be able to select my own certs for the IIS web application I deploy via this template.
Actual Behavior
I am forced to select the wrong certs.

Thanks for reporting this issue. We are now tracking this internally with ID MYSTIQUE-345.

I am observing the same situation with a new custom template I just created. I believe the issue to be related with the fact that drop-down is making use of the enumFromBigip feature? It would seem that the enumerated values are somehow cached when the FAST RPM is installed or when the template is initially deployed and new items created afterwards do not trigger a "refresh" of the presented enumFromBigip values?

Update: newly released FAST v1.7 appears to have resolved this issue.

Closing.
This issue was resolved with Release 1.7.


Standalone pod erroring out with lock issue in the very first run with a clean volume

Very first helm install of the neo4j-community chart with a clean volume is resulting in: Unable to obtain lock on file: /data/databases/store_lock issue.
Directories in use:
home: /var/lib/neo4j
config: /var/lib/neo4j/conf
logs: /logs
plugins: /plugins
import: /var/lib/neo4j/import
data: /var/lib/neo4j/data
certificates: /var/lib/neo4j/certificates
run: /var/lib/neo4j/run
Starting Neo4j.
2021-06-18 14:22:18.148+0000 INFO Starting...
2021-06-18 14:22:21.255+0000 INFO ======== Neo4j 4.2.4 ========
2021-06-18 14:22:32.893+0000 ERROR Failed to start Neo4j on dbms.connector.http.listen_address, a socket address. If missing port or hostname it is acquired from dbms.default_listen_address.
java.lang.RuntimeException: Error starting Neo4j database server at /var/lib/neo4j/data/databases
at org.neo4j.graphdb.facade.DatabaseManagementServiceFactory.startDatabaseServer(DatabaseManagementServiceFactory.java:207) ~[neo4j-4.2.4.jar:4.2.4]
...
Caused by: org.neo4j.kernel.lifecycle.LifecycleException: Component 'org.neo4j.kernel.internal.locker.LockerLifecycleAdapter@17ca8b92' was successfully initialized, but failed to start. Please see the attached cause exception "No locks available".
at org.neo4j.kernel.lifecycle.LifeSupport$LifecycleInstance.start(LifeSupport.java:463) ~[neo4j-common-4.2.4.jar:4.2.4]
... 5 more
Caused by: org.neo4j.kernel.internal.locker.FileLockException: Unable to obtain lock on file: /var/lib/neo4j/data/databases/store_lock. Please ensure no other process is using this database, and that the directory is writable (required even for read-only access)
at org.neo4j.kernel.internal.locker.Locker.storeLockException(Locker.java:175) ~[neo4j-kernel-4.2.4.jar:4.2.4]

Here are the versions in my environment:
Kubernetes v1.19
Helm v3.6.0
kubeadm 1.19.7
Nodes running Ubuntu 20.04.1 LTS
I suspect neo4j is restarting for some reason and as reported in https://github.com/neo4j/docker-neo4j/issues/289, neo4j is not clearing the lock from previous start and failing.
Have you experienced a similar issue anytime? Any ideas why an auto-restart may be happening? I'm seeing the same issue with the helm chart from neo4j-contrib.

https://github.com/neo4j/neo4j/issues/10625 provided the answer. I was using a NFS mount. Setting the no lock option on the mount fixed the issue.


Is there a trigger event for clicking the current day?

When clicking a different day, dateChanged is triggered, but what about clicking the existing selected day? Is there a way to have something happen when clicking the current day? Even having it trigger dateChanged is ok with me. Currently I have to click another day and then click back the original date to trigger a dateChanged for the existing day.

Hey @badbob001 I'll look into it this weekend. Probably will add a new event like selectedDateClicked or something.

Hey @badbob001 I'll look into it this weekend. Probably will add a new event like selectedDateClicked or something.

This feature is done. Will release in next build.


User management dashboard

Create a user management dashboard

@padmajabhol this is the issue that we can discuss and start working

Sure, I first need to understand what all we need in the management dashboard and plan the UI accordingly.

@padmajabhol sure thing.

change password, change email - individual users
Admin dashboard: remove / add users, raise prviliges from normal user to admin and back

we also need a sketch of what the UI might look like. Assigning this to either @yochannah or @ikayz

@yochannah I would like to work on this if no one is working currently on it. I can also develop the UI if it's not available.

Hi @ORKO06 , the organization has proposed this as a part of gsoc'22 and we don't yet have the prototype. Meanwhile you can have a look at other issues.


UX: Increase touch target size for easier drag in mobile

Discovered in UX testing that touch target size is not big enough. Participants were able to resize only about 50% of the time.

Request from @jouni: make it 44x44px by default, and add a way to customize the size.

Tomi says: might be an ios thing, cannot reproduce with Android

Increasing the touch target handles the issue quite well. Will be fixed by vaadin/vaadin-split-layout#19


Consider alternatives to docker, e.g. podman

Alternatives:

use podman with docker-cli
use podman with http-api (must be active)
use k3s binary on linux (some packages must be present)

Podman does not work because of privileged containers


Add summary of extra achievements to main view when there's an overflow of extra achievements

Going to update this to just be a summary of all the achievements for a given dungeon. Closed by #25


EXAMPLE - New Analytics Screen View Modifier

What it Does

Closes #197
Create new ViewModifier called AnalyticsView
Use the analyticsView on the DashboardView

How I Tested

Run the app
Go to the DashboardView tab
See the log in the console

Notes

N/A

Screenshot

N/A

LIVESTREAMED on My YouTube channel
https://www.youtube.com/live/-rYYKXrCyro?si=r9D0ii3D1T7OLhlM

Closing, this was an example on my livestream


ci: update workflows

Description

Related Issue

Type of Change

[ ] ✨ feat -- New feature (non-breaking change which adds functionality)
[ ] 🛠 fix -- Bug fix (non-breaking change which fixes an issue)
[ ] ❌ ! -- Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🧹 refactor -- Code refactor
[x] ✅ ci -- Build configuration change
[ ] 📝 docs -- Documentation
[ ] 🧪 test -- Test
[ ] 🗑 chore -- Chore

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

Project coverage is 62.18%. Comparing base (4235d9d) to head (31c78db).
Report is 4 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #926   +/-   ##
=======================================
  Coverage   62.18%   62.18%
=======================================
  Files          34       34
  Lines        1145     1145
=======================================
  Hits          712      712
  Misses        433      433

Flag
Coverage Δ

generator
63.13% <ø> (ø)

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


Integration of AMSPy into AzureRM Library

Hi Guy,
This is the first version of the integration of AMSPy into AzureRM. The code is working and all the examples (in the examples/media_services directory). The AMS has switched from access/secrety keys to AAD authentication recently, and we have until mid 2018 to replace the auth procedure to the new approach.
Thanks!
-- Leal

We will integrate in parts...


Feature Request? - have a way to not show any chats?

In Google Hangouts (im on the mac client) you could click out of any active chat thus keeping anything from showing up.  Say i dont want my previous chat glaring on my screen as someone walks by I could simply click out of the chat and it wouldnt show the contents of that particular chat with that specific person or group.
Is there a way to do this already?  If not i propose this functionality be included?
Thanks!

A bit of a hacky workaround, but you can click on the plus to start a new conversation, don't choose any people for the conversation, and click OK. Shortly thereafter the previously open chat closes. I'd love to be able to just press escape while in a conversation to go to this blank screen.

Yes, would prefer to have some privacy features added so when people walk up on me at work, they aren't reading my last conversations with other employees.

plus button, minimize button or any operating system shortcut would do the trick.
newest skypeforlinux adds a generic overlay when clicking on the username on top of the contacts, but that is also achieved with the plus button.


Add kubernetes-asyncio library

I want to add kubernetes-asyncio library to the list of community-maintained client libraries.
Thank you.

/approve


fix external task definition must exist before first run

what
Change external task definition to a list(string) so that it can be flagged on without needing the task definition to already exist.
why
Fix this issue when using an external task definition
│   49:   count                    = local.enabled && var.task_definition == null ? 1 : 0
│
│ The "count" value depends on resource attributes that cannot be determined
│ until apply, so Terraform cannot predict how many instances will be
│ created. To work around this, use the -target argument to first apply only
│ the resources that the count depends on.

references
Similar to how this was solved for the task role arn https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/blob/main/variables.tf#L216-L226. The workaround for now is to use -target like mentioned in #123.

Hello @mightyguava.
Thanks for your contribution.
Could you pls fix the following issues:

Code style:

  main.tf:364:83: warning: List items should be accessed using square brackets ()

https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/actions/runs/5179362958/jobs/9332797160?pr=204#step:6:120

Update README by running

README.md is outdated. Please run the following commands locally and push the file:
  make init
  make readme

https://github.com/cloudposse/terraform-aws-ecs-alb-service-task/actions/runs/5179362958/jobs/9332793659?pr=204
Thanks.

/terratest

hey @goruha i'm happy to fix the lint issue, but wanted to double-check that it won't break any backwards compatibility contract the module has? The lint is complaining about "${join("", aws_ecs_task_definition.default.*.family)}:${join("", aws_ecs_task_definition.default.*.revision)}" matches the original code on main.

@mightyguava
I checked tflint log again.
It failed because of runs out of warning limits.
All warnings are about list items' access
aws_ecs_task_definition.default.*.family should be aws_ecs_task_definition.default[*].family
Could you fix all of them?
[]  main.tf:364:40: warning: List items should be accessed using square brackets ()
[]  main.tf:364:83: warning: List items should be accessed using square brackets ()
[]  main.tf:364:137: warning: List items should be accessed using square brackets ()
[]  main.tf:463:40: warning: List items should be accessed using square brackets ()
[]  main.tf:463:83: warning: List items should be accessed using square brackets ()
[]  main.tf:463:137: warning: List items should be accessed using square brackets ()
[]  main.tf:562:40: warning: List items should be accessed using square brackets ()
[]  main.tf:562:83: warning: List items should be accessed using square brackets ()
[]  main.tf:562:137: warning: List items should be accessed using square brackets ()
[]  main.tf:661:40: warning: List items should be accessed using square brackets ()
[]  main.tf:661:83: warning: List items should be accessed using square brackets ()
[]  main.tf:661:137: warning: List items should be accessed using square brackets ()

Done!

/terratest

@mightyguava Thanks for your contribution.
LGTM


[ask] Where can I see the version of llama.cpp used for each version of ollama?

I think it would be good to include the version of Ollama used in the release notes to know the new features.

Go to the desired tag and into the llm folder there you can see the llama.cpp submodule and its version.
https://github.com/ollama/ollama/tree/v0.1.22/llm

Thanks a lot  :)


FuncDataTemplate issue with usercontrols

Describe the bug
On the site https://docs.avaloniaui.net/docs/concepts/templates/creating-data-templates-in-code described code snippet doesn't work with usercontrol.
To Reproduce
Replace TextBlock with a usercontrol in the code, and check in usercontrol's codebehind the property's value. It will be always null.
The code throws no error.
Expected behavior
Same behaviour as normal controls.
Avalonia version
11.1.1
OS
Windows
Additional context
No response

Please provide your UserControl implementation with the property.

Here you are. :)
axaml:

<StackPanel Name="Panel" Orientation="Vertical" MinWidth="100" MinHeight="75">

</StackPanel>

axaml.cs:
using Avalonia;
using Avalonia.Controls;
using Avalonia.Data;
using Avalonia.Markup.Xaml;
using Avalonia.Media;
using System;
using System.Collections.Generic;
using System.Collections.ObjectModel;
using System.Linq;
namespace SchFeladatutemezo.UserControls;
public partial class ucFoNaptarStack : UserControl
{
public ucFoNaptarStack()
{
InitializeComponent();
    var t = Tag;

    int i = 0;  //Breakpoint
}

}

I'm sorry but that doesn't demonstrate anything binding-related, so I can't determine what the issue is. Can you create a full repro and either upload the zipped project or link a GitHub repo?

Of course and thanks.
It runs on .NET 8.0, WIN2019 Server, VS2022
SchFeladatutemezo.zip

Ok, I see. You expect Tag to be available in the constructor on this line:
public ucFoNaptarStack()
{
    InitializeComponent();

    var t = Tag;
}

From this binding:
foreach (var ember in EmberekLista)
{
    var template = new FuncDataTemplate<FoNaptarSor>((value, namescope) =>
        new ucFoNaptarStack
        {
            [!ucFoNaptarStack.TagProperty] = new Binding("FoNaptarFeladatblokkok")

        });

    FoNaptar.Columns.Add(new DataGridTemplateColumn()
    {
        Header = ember.Nev + Environment.NewLine + ember.Rendszam,
        Width = new DataGridLength(200, DataGridLengthUnitType.Pixel),
        CellTemplate = template,
        Tag = new Binding("SID")
    });
}

So this issue is your expectations of when data is available. Controls that do not create and assign their own DataContext in the constructor (or in their XAML) will not have any data binded properties available at that time. The DataContext is created and assigned later on. You can check DevTools to see that Tag does have the binding established at runtime:

How much later depends on the external code. It may be available when the Loaded / AttachedToVisualTree events occur, but you may need to wait until DataContextChanged (which would be the safest).

protected override void OnAttachedToVisualTree(VisualTreeAttachmentEventArgs e)
{
    base.OnAttachedToVisualTree(e);
    var t = Tag;
}

This appears to work better than overriding OnDataContextChanged as the property bindings seem to not be established at that point.
You could also subscribe to a property change:
public ucFoNaptarStack()
{
    InitializeComponent();
    TagProperty.Changed.Subscribe(OnTagChanged);
}

private void OnTagChanged(AvaloniaPropertyChangedEventArgs<object?> args)
{
    var t = args.NewValue;
}

Thanks a lot! It works like a charm!


Work in Progress: Basic support for signup forms

This adds support for Signup Forms.

Cloning a signup form
Processing a signup (mapping a hash of data to form fields)

Not done yet:

specs for process_signup/form_fields

Need to add new functionality to README

Code looks reasonable. Should test against our sandbox account.

Looks good


VSCode Extension: allow for dragging the tab around

I'm not sure if this is a VSCode bug, but I realised that when I'm trying to drag the Portal tab around, the whole window is also being dragged with it. A very strange behaviour, as if the drag events are passed to both the tab, and the window itself.
To reproduce:

Make a left/right split in VSCode, open a few files in each side.
Open portal via the extension.openPortal command (it shows up always on the left side)
Try to drag any editor tab from the left side.

Expected behaviour:

The editor tabs can be rearranged

Actual behaviour:

The whole window is moved

The behaviour persists even if the Portal tab is closed, until a window reload.
The right side split is unaffected.

Hi @orestis, thanks for opening up this issue 👍  Quick question, what operating system is this on? I can't seem to reproduce on OSX.

I did notice the issue on windows and https://github.com/djblue/portal/commit/e625065fda64b3e85fe3ba212a5b061bd613f340 seems to fix it 👌

This should be resolved with 0.26.0


Rendered markdown note show additional lines

Version: 1.66.2 (Universal)
Commit: dfd34e8260c270da74b5c2d86d61aee4b6d56977
Date: 2022-04-11T07:49:20.994Z
Electron: 17.2.0
Chromium: 98.0.4758.109
Node.js: 16.13.0
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.4.0
This is the markdown under editing mode:

This is the markdown after exit editing mode:

Thank you

Please share a file (or text) that demonstrates the issue
/needsMoreInfo

If the length of the markdown in a single markdown box exceeds a certain amount of volume, when I exit editing, it will show as the image presents:

However, when there are a limited amount of texts, the situation does not exist.

For the detail part, I am not that sure, but I am sure it has something to do with the number of characters.

Please share the notebook file itself

This is my notebook file on Google Drive:
https://drive.google.com/file/d/1u0nBvCiuavEDd13xe37CyRxgWU76VTF2/view?usp=sharing

Thanks @Andd54!


Feature/jwt for diff zip

I added JWT for Diff.zip in all the TestExample.

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


Can anyone guide me how to program NRF51822 using Arduino IDE?

I am beginner for NRF51822.
I have this board : https://www.tanotis.com/products/tanotis-nrf51822-bluetooth-module-networking-module-wireless-communication?gclid=Cj0KEQjwnsPGBRDo4c6RqK-Oqu8BEiQAwNviCRQ49by-kmeBS09n_kP5e8Ycsf8Dwl1P7d0CamqwNpMaApAM8P8HAQ&variant=20624084165
How can i program this using Arduino IDE?
Thank You!

Looks like thats just a bare module. Youll need to hook it up to a programmer/debugger chip of some kind.
The professional tool is the segger jlink, https://www.adafruit.com/product/1369
The 1bit squared guys are getting popular https://1bitsquared.com/products/black-magic-probe
Or something? running the open source cmsisdap daplink firmware https://github.com/mbedmicro/DAPLink
Or forget that module for now until you're more comfortable and just buy one of the already supported boards that already has a programmer/debugger on board.


Do not run PR validation for "Draft PRs"

We should not run PR validation on Draft PRs, Draft PRs may receive many commits and at every "git push" we are rerunning the tests.
Sending Draft PRs is useful in order to have an early feedback from the community.
If the developer wants to run the full suite that can be done in his private repo or by sending a non Draft PR.

@lhotari PTAL

LGTM

LGTM, running CI on draft PRs will waste unused CI resource.

Is anyone interested in sending a patch?

Is anyone interested in sending a patch?

we need to get https://github.com/apache/pulsar/pull/9963 merged before making this change. A solution was proposed in the mailing list discussion, it's https://github.community/t/dont-run-actions-on-draft-pull-requests/16817/16 .

@lhotari we committed #9963
You are good to go now

@eolivelli I created #9994 , but it comes with a risk for merging PRs before checks have passed. Let's continue the discussion on the PR.

@eolivelli @lhotari thank you for your efforts in driving this.
I was wondering how do we judge whether a PR is draft or not? Do we mark it as draft by adding the [WIP] as prefix in the PR title, and the Github Action will not run test jobs for this draft PR?

Or if I send a draft PR, what should I pay attention to?

@eolivelli @lhotari thank you for your efforts in driving this.
I was wondering how do we judge whether a PR is draft or not? Do we mark it as draft by adding the [WIP] as prefix in the PR title, and the Github Action will not run test jobs for this draft PR?

In GitHub, you can create a PR in draft status or change an open PR into draft status.
Here's an old GitHub announcement about the feature:
https://github.blog/2019-02-14-introducing-draft-pull-requests/#tag-your-work-in-progress
An existing open PR can be switched to draft mode by clicking the link "Still in progress? Convert to draft". This is in the beginning of the PR page, on the right side column. When it's in draft, it can be changed back to normal state by click the "Ready for review" button which is at the bottom of the PR page, just above the PR check status.

Or if I send a draft PR, what should I pay attention to?

Setting a PR in draft status indicates to reviewers that it's work-in-progress and the PR author is still revisiting the changes before making it ready for review. The author of the PR might request reviewers to provide early feedback as guidance.

@lhotari Thank you very much for your detailed info. I got your point now and I totally agree with you on this great idea.

The issue had no activity for 30 days, mark with Stale label.

Now we have the precondition to prevent too much CI workload, I think we can close this issue as not an issue.


Cannot read file sourcemain.d

How to reproduce:
Try to dub run dido:
https://github.com/p0nce/dido/blob/master/dub.json
For some reason the dmd command gets called with source/main.d and sourcemain.d
dmd -c /* lots of include dir and source files */  source\main.d sourcemain.d
Error: cannot read file sourcemain.d

Introduced with 3b2af9931e2440531242076ad6a3e52a3bb540a4.


Add pre-built tasks

I open the PR so you can already have a look at it, but I plan to leave comments here and there to explain the changes.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

10 of 32 (31.25%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 53.545%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/send_command.rs
3
5
60.0%

src/client.rs
7
27
25.93%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
657

Relevant Lines:
1227

💛  - Coveralls

Hey Clement! Thanks for the PR! :-)
I'm still not home but just a few thoughts to (hopefully) unblock you a bit.
I think that we have roughly a few work packages here: Tasks that can already be added, tasks that need changes (IDLE), (START)TLS, and other improvements.
To better focus on any part, we could try to split this PR accordingly and first try to get all things in that are unrelated to SansIO. Most tasks, I guess?
We should also treat STARTTLS as a connection "prefix", and only progress the IMAP protocol to the point where both parties start sending TLS data (doing (almost) no parsing).
We don't even need to check capabilities:

read greeting (no IMAP parsing, greeting has no literal, just read and discard line as we MUST forget everything we learn here anyway)
send A starttls (don't care if supported)
receive line (don't care if ok or not)
start with TLS (exactly as an implicit TLS client)

This way, we have an isolated STARTTLS prefix and can proceed exactly as with implicit TLS. This would be secure by design. More in section 8 https://www.usenix.org/conference/usenixsecurity21/presentation/poddebniak.
Jakob is currently working on TLS. I would suggest to wait a few more days before coming back to it.

Awesome, then I will open small PRs until you come back.

We should also treat STARTTLS as a connection "prefix", and only
progress the IMAP protocol to the point where both parties start
sending TLS data (doing (almost) no parsing).

It is more clear now, thank you for the post as well, really
interesting.

Jakob is currently working on TLS. I would suggest to wait a few more
days before coming back to it.

I will wait for Jakob's work then.
Thanks for your feedback!

fyi: SansIO is ready for review, see #158

fyi: SansIO is ready for review

Awesome, I will try to integrate my core changes from your branch to
see. As soon as it is merged, I will open the first PR (and close this
one).

Thanks! :-) Lot's of progress happening, great :-)


Idea : Integrate RECORDID in FlowField and Link properties

We often create generic functionalities that are using RecordID to link data between records. On the record, fields are added to show the no. of linked records, ... On the pages, factboxes are added to show the linked records (see #1552).  The above is always coded, since there's no native support for the 'RecordID' integration inside the FlowField definition of the SubPageLink property. Adding support for the RECORDID function (or an alternative to it) in various properties would ease this integration.

The way BC deals with e.g. the 'attachment' factboxes use the 'SubpageLink', which is nice since it updates without the need for code and it also update when page filters are set that return no records / results. (However, the PK is badly designed since not portable to any kind of records, if the PK differs from the No / Document Type / Line No pattern)

The way we used to deal with the factboxes was by coding the RECORDID. It would be nice though to integrate the RecordID in the 'SubpageLink' property.

As a reference (Feel free to upvote)
https://experience.dynamics.com/ideas/idea/?ideaid=4bcb9af5-aa6a-e911-b047-0003ff689546

I will close this issue. Any future readers should vote on the idea so that it can be considered for a future release.


Add ctrlf-mode-map

Attempt to fix #62.
I'm not sure I fully understand your
comment(my
elisp is weak), but I figure it's easier to continue the discussion
with some concrete code.

Oh yes, also the CI is currently failing (due to the reference to ctrlf--keymap).

Thanks for the review. I think I addressed all your comments.

Thanks for the review. I think I addressed all your comments.

I can make an analogous change for ctrlf-minibuffer-bindings. Should it be called ctrlf-minibuffer-map?

I can make an analogous change for ctrlf-minibuffer-bindings. Should it be called ctrlf-minibuffer-map?

@raxod502 sorry for the delay. should  be ready now

Would you mind fixing the CI failures? This also may need a bit of additional consideration with the change https://github.com/raxod502/ctrlf/commit/ddfbc358b092351167219ca5884a25b2eb8bac00 from February.

This thread is being closed automatically by Tidier because it is labeled with "waiting on response" and has not seen any activity for 90 days. But don't worry—if you have any information that might advance the discussion, leave a comment and I will be happy to reopen the thread :)


Pusher should return 200 on pusher/auth

https://pusher.com/docs/authenticating_users

closed in #1861


Twitter Report: App shows "Begegnungen an 2 Tagen mit erhöhtem Risiko", however, the contact journal only shows an encounter on one day

Avoid duplicates

[X] Bug is not mentioned in the FAQ
[X] Bug is specific for iOS only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[X] Bug is not already reported in another issue

Technical details

Device name: not relevant
iOS version: 15.3
App version: 2.16.2

Describe the bug
The app shows this on the "Status" tab:

However, in the contact journal only one day is shown:

Steps to reproduce the issue
Not clear
Expected behaviour
If there were encounters on two days, I'd expect two days are marked in the contact journal.
Additional context
Maybe there were two increased risk encounters with different persons on the 21.?

Internal Tracking ID: EXPOSUREAPP-11612

So, today the second day is shown correctly. I guess this is some kind of race condition which can occur.

Thanks for your report @Ein-Tim. We have created an internal ticket for it and will raise this topic internally.
Internal Tracking ID: EXPOSUREAPP-11612

Corona-Warn-App Open Source Team

As the CWA project went into ramp-down mode, I don't expect this issue to be addressed anymore. I'm therefore closing this issue.


Backport "[Ruby]fix a rake compile error since rake-compiler ~> 1.1 (#28666)"

Backports #28666

Can you take a look at the Basic Tests Ruby Linux job?

Failures:

  1) ClientStub #server_streamer via a call operation raises GRPC::Cancelled after the call has been cancelled
     Failure/Error: replys.each { |r| c.remote_send(r) }

     GRPC::Core::CallError:
       call#run_batch failed somehow
     # ./src/ruby/lib/grpc/generic/active_call.rb:200:in `run_batch'
     # ./src/ruby/lib/grpc/generic/active_call.rb:200:in `remote_send'
     # ./src/ruby/spec/generic/client_stub_spec.rb:977:in `block (2 levels) in run_server_streamer'
     # ./src/ruby/spec/generic/client_stub_spec.rb:977:in `each'
     # ./src/ruby/spec/generic/client_stub_spec.rb:977:in `block in run_server_streamer'
     # ./src/ruby/spec/generic/client_stub_spec.rb:22:in `block in wakey_thread'
     # /usr/local/rvm/gems/ruby-2.5.8/gems/logging-2.3.0/lib/logging/diagnostic_context.rb:474:in `block in create_with_logging_context'

Finished in 6.33 seconds (files took 0.31988 seconds to load)
150 examples, 1 failure

Failed examples:

rspec ./src/ruby/spec/generic/client_stub_spec.rb:597 # ClientStub #server_streamer via a call operation raises GRPC::Cancelled after the call has been cancelled

It looks like the ruby test failure has been flakey on master for a while.
I ran repro it at HEAD, on this branch, and on the 1.45 branch.


plausible: fix admin user password seed and SMTP passwords

Description of changes
LoadCredential was misused as it is not building any environment variable, it is the responsibility of our preStart to do it so Plausible's script can pick it up.
Fixes the fallout of #143345.
If I had time and 1.5.0 didn't remove "admin initialization", I'd add a test for it with pleasure, but I am short of time.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

One small thing to be aware, is that the secrets are viewable by anyone doing a systemctl status. Its not ideal, but from what i know, there is no better way to handle it when upstream doesnt read secrets from a file.

One small thing to be aware, is that the secrets are viewable by anyone doing a systemctl status. Its not ideal, but from what i know, there is no better way to handle it when upstream doesnt read secrets from a file.

Is it? I think the export won't make the environment variables available at all.


Create .deb package

This work has been moved to https://github.com/DynomiteDB/docker-build-dynomite


SMTP issues

Hi,
my .env file is as follows:
LC_ALL=it_IT.UTF-8
LANG=it_IT.UTF-8
MailConfig__EmailServer="smtp.sendgrid.net"
<EMAIL_ADDRESS>MailConfig__UseSSL="true"
MailConfig__Port=587
MailConfig__Username="apikey"
MailConfig__Password="mypassword"
#LOGGING__LOGLEVEL__DEFAULT=Error

I am trying to test that the emails are sent out correctly by opening the page https://lubelogger.mydomain.net/api/vehicle/reminders/send?urgencies=NotUrgent while logged in with my root user. I get:
success	true
message	"Emails sent"

but I don't receive any emails... In facts, my firewall doesn't even show that an outbound connection to port 587 is being made.. I've opened a shell into the container and I am able to reach smtp.sendgrid.net at port 587.
Is there a way to debug this? I don't see anything that can help me in the app logs
Thanks

I've re-opened the issue because I initially thought something was wrong from my end but it doesn't seem the case.

If there are no logs or errors, that means that the email was flagged and failed by SendGrid's servers because they think it's spam. I would consider trying a different SMTP provider such as Gmail and see if it works, unfortunately this is beyond our control.

Hi @hargata
But I can see from my firewall logs that a connection is made to the sendgrid server if I telnet it from the container shell, whereas when I use the API route, no external connection is made.
Furthermore, there is nothing logged in Scalegrid

Yeah I'm not sure then. I don't have a sendgrid account to test this with.

I don't think the problem is Scalegrid here, I've changed the SMTP config to use a Postfix relay which I am using for services that don't support authentication. Even with this one no emails are sent and no connections are made to the relay...
But I still get
success: true
message: "Emails sent"

I've tried with both
MailConfig__Username=""
MailConfig__Password=""

and also I've tried to remove them entirely

I'm not sure then, sorry. I'll keep your issue open just in case someone else can chime in

Sure, no worries. I'll try to investigate more this weekend

Hi @hargata,
I've conducted additional tests and discovered that when I generate a new token from the admin panel, the email is sent without any issues.
It appears that the problem lies with the API route. Even though I receive a response:
    "success": true,
    "message": "Emails sent"
}

something breaks and the email is not actually sent. Do you have any idea why this might be happening?

@marcogiorgio
I am running into teh same problem it seems. I got the successful API call ( {"success":true,"message":"Emails sent"} ).
After reading your comment, I was trying to create a user in the Admin Panel page.
However I do not have a button to create a new user !?

@johenkel there is no button to create  a new user in the admin page. You have to generate a token by clicking on the 'Generate user token' button.
Then you have to logout and click on 'Register' and use that token to crete a new user

Hi @hargata, I managed to find the problem root cause... Basically when I enabled the authentication feature I didn't notice that the user that you create during this process is not a "real user" (I guess this is just a wrapper for the HTTP basic auth?)... I had to create a real user from the admin page with an email associated to it...
Anyway if I got it right, I think this is a little bit confusing, maybe it would be better to have a "full" user created when you enable authentication. In facts, I was able to create another user using the token registration functionality with the same username as the one I used when I enabled the authentication...
As a results, when I login I don't really know which user I am actually using

In next release, you will be able to inject a new environment variable named DEFAULT_REMINDER_EMAIL and that will allow the root user to receive reminders for all vehicles instead of having to create a new user.

@marcogiorgio
In 1.3.7, instead of having to create a new user, you can just add a Default Reminder Email in the settings tab and you will receive reminder emails for all vehicles as the root user.


As a manager, an overview page for each round (submission group), displaying key details, current status, and listing associated applications

Total number of applications, amounts requested, number of open applications, amount of funds approved, averages, etc, ability to view and filter associated applications on the round page as you can on the submissions tab, activity feed as you see on the submissions tab but only for those apps, ability to comment on the round generally, field that links to the external doc that is being used by the round lead to capture notes

This is planned to be covered in the work we are doing in the Winter 2019 Sprints. Adding to project board.


Support for deleting tags that are not used

Is your feature request related to a problem?
After deleting the historical memos, it leaves a useless tag
Describe the solution you'd like
Support for deleting tags that are not used
Additional context
No response

I know how to delete

请问你是如何删除不用标签的，谢谢


[proposal] multi hierarchy elastic quota management

What is your proposal:
Why is this needed:
Is there a suggested solution, if so, please add it:
https://github.com/koordinator-sh/koordinator/pull/398

same as #202


Plotting in Jupyter shows nothing, 404 error

Using ipyvolume 0.4.0, ipywidgets 7.0.1 from conda. All extensions installed and enabled.
$ jupyter nbextension list
Known nbextensions:
  config dir: /home/noah/.jupyter/nbconfig
    notebook section
    tree section
      ipyparallel/main  enabled
      - Validating: OK
  config dir: /home/noah/anaconda3/etc/jupyter/nbconfig
    notebook section
      jupyter-js-widgets/extension  enabled
      - Validating: OK
      ipyvolume/extension  enabled
      - Validating: OK
      jupyter-webrtc/extension  enabled
      - Validating: OK
  config dir: /usr/local/etc/jupyter/nbconfig
    notebook section

When creating a plot, the notebook server console outputs
[W 18:19:52.795 NotebookApp] 404 GET /static/components/preact/preact.min.js.map (<IP_ADDRESS>) 4.11ms referer=None
[W 18:19:52.801 NotebookApp] 404 GET /static/components/proptypes/index.js.map (<IP_ADDRESS>) 2.53ms referer=None
[W 18:19:52.804 NotebookApp] 404 GET /static/components/preact-compat/preact-compat.min.js.map (<IP_ADDRESS>) 2.57ms referer=None
[W 18:19:52.810 NotebookApp] 404 GET /nbextensions/ipyvolume/three.js.map (<IP_ADDRESS>) 4.62ms referer=None

And I get the following in my js console:
:8083/nbextensions/jupyter-js-widgets/extension.js?v=20170919181906:21509 Error: Could not create a view for model id eba62fc927eb48fd8650930d73a4f04b
    at promiseRejection (:8083/nbextensions/jupyter-js-widgets/extension.js?v=20170919181906:21509)
    at <anonymous>
promiseRejection @ :8083/nbextensions/jupyter-js-widgets/extension.js?v=20170919181906:21509

I had the same issue with the same version of ipywidgets, ipyvolume installed by pip3. Uninstalled ipyvolume, installed with pip3 again, enabled. Now it works. Why? I do not know.

Gave that a shot - no dice. I've tried this on several machines now so I'm convinced this is a real bug.

Can you test if ipywidgest is working, can you get a slider to appear?

Yes, I can get basic ipywidgets to appear.
I just tried on another machine and was more careful about how I installed and activated all the extensions. I removed absolutely every extension from all my notebook.json files except for my Anaconda root's notebook.json. Now everything is working as expected.


Question: Why there is parameters verification that point dimension must be at least 2?

I would like to create a R*-tree for points that could have a single dimension, but it says "thread 'main' panicked at 'Point dimension too small - must be at least 2'"

Could you explain more about what you're trying to achieve? Preferably with a minimal reproducible example. I mostly work with geometries in R2, so I have to admit that I simply don't understand the phrase "points that could have a single dimension".

I'm not sure it makes sense to use an RTree for less than 2 dimensions.
If I wanted to efficiently insert and query over a 1 dimensional range, I think I'd use a btree:
https://doc.rust-lang.org/stable/std/collections/struct.BTreeMap.html#method.range
Or do you mean that your points will be a mix of 1 and 2(+) dimensions?

I am working with a program that works with problems could be set in different dimensions - I just want to solve all situation with the same piece of code. It's right that btree or other solution is better for one dimensional problem. And I have found the crate that is more suitable for me (kd_tree)

@urschrei I have a use case where I currently use a BTreeMap<u32, u32> to represent intervals (key as the start, value as the end). It's really read heavy because I do a lot of range queries over it to see which intervals overlap. I think a one-dimensional RTree could be a better alternative for this use case and I'm currently attempting to do performance testing, but ran into this point dimension limitation. I'm going to work through it on a fork so I can test it out anyway, but just wanted to mention my use case here.
@michaelkirk BTreeMap is good but there are optimizations you can apply with a 1D bounding box/envelope that wouldn't apply to a generic BTree. My intuition is that a 1D r*-tree could perform better, but I haven't confirmed this yet.


Fix admin search for integer fields

Coverage remained the same at 88.293% when pulling 8e2dfd201d524d5a368f3d1cf51ed4b20c83bd93 on fix-admin-search into e233e8554b28c3a1182734c9621fdf264a115a29 on master.


(PDB-3195) Default transaction isolation in connection pool config

Before this change, our internal transaction wrapper macro was explicitly
passing a :read-committed isolation level to clojure.jdbc when not otherwise
specified. This causes a call to .setTransactionIsolation on the jdbc driver,
which executes SQL on the server. So it ends up making an extra network round
trip for each query.
This commit sets read-committed transaction isolation in the connection pool
config, so each connection already has that by default. It also allows nil
isolation levels to be passed to clojure.jdbc, which will then use whatever the
connection already has.

Test PASSed

CLA signed by all contributors.

Test PASSed


Upgrade role for 0.9.5

Release changes for 0.9.5 – InfluxDB CHANGELOG

[ ] Make sure breaking changes are accounted for

:+1:


Template resource: Does Create() need to GET the template immediately after creation?

Yes. It does need to GET.
We only have rack IDs at this stage. Running GET populates the full details of each rack, logical device, etc...


I cannot modify time of a message!

Hi,
I have a list of times expressed in ticks and a list of messages  (meta), I'd like to change times of the messages to match the times in the list.
tickList = [....]

i= 0
for msg in t:
    while i < l-2:
        if msg.is_meta and hasattr(msg, 'data') and len(msg.data) == 7:
            msg.time = tickList[i] if i == 0 else tickList[i+1]-tickList[i]
        i +=1

However, times remain the same! why, please?
10 <meta message sequencer_specific data=(67, 123, 1, 52, 0, 52, 127) time=1920>
11 <meta message sequencer_specific data=(67, 123, 1, 50, 8, 50, 127) time=1920>
12 <meta message sequencer_specific data=(67, 123, 1, 50, 8, 50, 127) time=3840>
13 <meta message sequencer_specific data=(67, 123, 1, 50, 8, 50, 127) time=1920>
15 <meta message end_of_track time=0>

Any help would be greatly appreciated.
CS

I am not sure what exactly you want to do. Do you have a list of time values that you want to attach only to messages that match the if statement?
What the code does is to loop over all messages, and then for each message it for loops and tries to set the time attribute of that message multiple times. This doesn't seem quite right.
If your tick list has one tick value per message you can simply do this:
for msg, ticks in zip(track, tickList):
    msg.time = ticks

but it looks like you're doing something a bit more complex.


'TwoParamLog' object has no attribute 'fit'

When I run the example codes provided, I get this error.

Hi, are you referring to the example code here? https://github.com/nd-ball/py-irt/tree/master/examples
If so, those are old examples and should be depreciated. The package now uses the command line to fit the IRT models. Sorry about the confusion!


Add done_callbacks param to create_task

Adds a done_callbacks kwarg to the create_task scheduling util, and adds each callback to the task. This is done as part of python-discord/bot#2580 (ability for more refined error-handling on schedules, such as the incident-crawler), however is a general-purpose thing too.
Suggested and approved by @ChrisLovering.

What's the benefit of this over adding the callback to the returned task?

What's the benefit of this over adding the callback to the returned task?

I'm not familiar with tasks, so you'd have to ask @ChrisLovering.


Formatted files

Maintainer merge checklist

[ ] Title is descriptive/clear for inclusion in release notes.
[ ] Applied a Component: xxx label.
[ ] Applied the api-deprecation or api-break label.
[ ] Applied the release-highlight label to be highlighted in release notes.
[ ] Added to the milestone version it was merged into.
[ ] Unittests are included in PR.
[ ] Properly documented, including versionadded, versionchanged as needed.

Formatted animation.py, app.py, atlas.py, base.py, cache.py.

Thanks for your interest in contributing to Kivy. However, we don't accept PRs that change only formatting and don't fix actual bugs or add new features. Because it may cause new bugs that is hard to notice when reviewing a large PR.
But also reviewing takes time, but formatting changes doesn't add much of a benefit to the project. In fact - it's likely to make reading file git history harder.
Here's a link that @tshirtman likes to share that explains some of this: https://www.youtube.com/watch?v=wf-BqAjZb8M.

Thanks for your interest in contributing to Kivy. However, we don't accept PRs that change only formatting and don't fix actual bugs or add new features. Because it may cause new bugs that is hard to notice when reviewing a large PR.
But also reviewing takes time, but formatting changes doesn't add much of a benefit to the project. In fact - it's likely to make reading file git history harder.
Here's a link that @tshirtman likes to share that explains some of this: https://www.youtube.com/watch?v=wf-BqAjZb8M.


Adding the right age to the first block in Explorer

Resolving issue #214

Please describe the underlying issue and the solution used to fix it.  Thanks.

I realized that after the age updates it returns to the previous age of 47y 9m @RogueElement how do you suggest i fix this?

I noticed that the 2nd block on the explorer page was showing the age for the first block, which can be noticed if you go into the block pages. So i tried to rearrange it

@gozart1 should take a look.

Hey guys, sorry about this one.
Explanation and solution in.
https://github.com/dcrdata/dcrdata/pull/224


Add workflow for testing Yosys plugin support

This adds a separate CI workflow that:

checks out current confirmed-to-work tool versions from https://github.com/antmicro/yosys-uhdm-plugin-integration
builds the tools against current Surelog
uses the tools to build Ibex as a reasonably complex test case

This aims to help catch regressions that break the SystemVerilog plugin for Yosys.

Please make sure there is something like this that is checking the latest versions, so we know when an change causes breakage.

FYI - @hzeller

@rkapuscik, This is still a work in progress looks like? It's crashing.

@rkapuscik any update here ?

@hzeller looks like it consistently passes now, so should be ready for review.


chore(upgrade-job): bump the expected umbrella chart version to 4.0

bors merge

bors merge

bors try

bors merge


Add WebSocket Validation Feature

Issue Link :link:
WebSocketRequest Should Support Validation #3853
Goals :soccer:
This pull request introduces a validation feature for WebSocket communication. The feature ensures that all outgoing requests contain valid status codes and that the responses adhere to the expected format. This addition improves the robustness of our WebSocket communication by preventing invalid requests and ensuring consistent error handling.
Implementation Details :construction:

Validation for Status Codes: Added checks to ensure that status codes returned by the WebSocket server are valid and within the expected range.
Request Validation: Introduced validation to check the format and correctness of incoming WebSocket requests before processing them.
Error Handling: Implemented better error handling for invalid status codes or requests, providing clearer error messages to users and developers.
Extensibility: Designed the validation feature to be easily extensible in the future, allowing for custom validations if needed.

Testing Details :mag:
webSocketRequest.validate { request, response, error in
    // Perform custom validation checks
    if let error = error {
        return .failure(error) // Fail if there is an error
    }

    if response?.statusCode != 200 {
        return .failure(AFError.responseValidationFailed(reason: .unacceptableStatusCode(code: response?.statusCode ?? 0)))
    }

    return .success // Return success if no issues found
}

fix for WebSocketRequest Should Support Validation #3853

I'm sorry, but this doesn't seem to do any sort of validation. validate() is automatic, and unfortunately URLSessionWebSocketTask doesn't actually provide the hooks necessary for that automatic validation to actually occur. Adding manual validation like this doesn't really do anything, as users could just put that logic in the response handlers themselves.

ok no problem


Add lifecycle pv relcaim tc, improve lifecycle suite readability

Automated 54201, 54202, 54203, 54204, 54206, 54207
Add lifecycle pv relcaim tc, improve lifecycle suite readability and consistency.

@greyerof as explained in the huddle, in the beforeEach block, I clean all resources that exist in the namespace,
including PVs and PVCs.
but, I was mistaken by relating the PVs deletion with the namespaces.clean, since PVs are cluster wide and not namespaced.
hence I moved the cleanPVs func to a different location.
As for the comment on the duplicated pv name, good catch, thank you, uploaded a new commit.


test_dsl

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

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

The Build&Test check has failed. Please check the logs.


Update Composer dependencies (2023-12-12-00-17)

Visual regression test passed!
View the visual regression test report


Fix coverage percent

Coverage increased (+4.4%) to 100.0% when pulling 546ae9523de946bfea2bd928d8f66a771541434b on v5.0.0rc6 into 491978f295eb065d790e66001e9353588308f29a on main.


Performance Issues in Edge with lots of Columns

Issue Type

[x] Bug

Specs
What version are you using?
2.12.2
What browser?
Edge
Expected Behavior
What did you expect to happen?
Relatively quick load and functionality
Actual Behavior
What actually happened?
Edge (and IE) is extremely slow to render the table as well as perform functions (such as selecting a rows or changing the page) when there are a large number of columns in the table (something like more than 50 columns). This does not occur in Chrome or Firefox.
jsfiddle
https://jsfiddle.net/57a2a4ce/492/

On first glance, the culprit seems to be flushCallbacks on Vue.
On IE 11, Timer (flushCallbacks) was executed for 413ms exclusively (excluding any function calling within flushCallbacks()) while it only took 69ms on Chrome.
Can't say for sure but looks like Edge & IE's JavaScript engine isn't really optimized for Vue.js's reactivity to me.


error on update

i will update, then i get an error.
pip3 install weconnect-cli --upgrade Requirement already up-to-date: weconnect-cli in /home/fhemadmin/.local/lib/python3.8/site-packages (0.24.0) Requirement already satisfied, skipping upgrade: ascii-magic~=1.6 in /home/fhemadmin/.local/lib/python3.8/site-packages (from weconnect-cli) (1.6) Collecting weconnect[images]~=0.36.0   Downloading weconnect-0.36.2-py3-none-any.whl (140 kB)      |████████████████████████████████| 140 kB 1.8 MB/s Requirement already satisfied, skipping upgrade: colorama in /usr/lib/python3/dist-packages (from ascii-magic~=1.6->weconnect-cli) (0.4.3) Requirement already satisfied, skipping upgrade: Pillow in /home/fhemadmin/.local/lib/python3.8/site-packages (from ascii-magic~=1.6->weconnect-cli) (9.0.0) Requirement already satisfied, skipping upgrade: requests~=2.27.1 in /home/fhemadmin/.local/lib/python3.8/site-packages (from weconnect[images]~=0.36.0->weconnect-cli) (2.27.1) Requirement already satisfied, skipping upgrade: oauthlib~=3.1.1 in /home/fhemadmin/.local/lib/python3.8/site-packages (from weconnect[images]~=0.36.0->weconnect-cli) (3.1.1) Requirement already satisfied, skipping upgrade: urllib3<1.27,>=1.21.1 in /usr/lib/python3/dist-packages (from requests~=2.27.1->weconnect[images]~=0.36.0->weconnect-cli) (1.25.8) Requirement already satisfied, skipping upgrade: certifi>=2017.4.17 in /usr/lib/python3/dist-packages (from requests~=2.27.1->weconnect[images]~=0.36.0->weconnect-cli) (2019.11.28) Requirement already satisfied, skipping upgrade: charset-normalizer~=2.0.0; python_version >= "3" in /home/fhemadmin/.local/lib/python3.8/site-packages (from requests~=2.27.1->weconnect[images]~=0.36.0->weconnect-cli) (2.0.9) Requirement already satisfied, skipping upgrade: idna<4,>=2.5; python_version >= "3" in /usr/lib/python3/dist-packages (from requests~=2.27.1->weconnect[images]~=0.36.0->weconnect-cli) (2.8) ERROR: weconnect-mqtt 0.28.1 has requirement weconnect[images]~=0.35.1, but you'll have weconnect 0.36.2 which is incompatible.

This is when several packages need different dependencies. You can solve these problems with virtual environments. See for example here. https://realpython.com/python-virtual-environments-a-primer/
Anyway this problem should be resolved after the latest releases today that share the same dependencies.


[FancyZones] Add an icon to the FZ process

Microsoft PowerToys version
bdf8598
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Icon in the Task Manager is missing
✔ Expected Behavior
No response
❌ Actual Behavior

Other Software
No response

Resolved with the release of v0.41.


Update the docs about newly created registries and their features

### Tasks
- [x] Copy and paste the architecture from RFC, update it.
- [x] Describe setup of the public registry: enabled options, authentication modes.
- [x] Describe setup of the private registry
  - [ ] Where is it and what's the name and URL.
  - [ ] How to generate access keys.
  - [ ] Security: how to distribute private image in case it's rapidly needed

Some notes after I worked on the docs:

Access to Aliyun private images is currently unclear (not documented)
Unclear whether private images should be accessed from Docker Hub or Quay. It's possible, but I guess it's not desired.
For ACR credentials (scope map plus token), a naming convention seems to be a must-have. Otherwise it will become impossible to track ownership and clean up.
For creating ACR image pull secrets, I think that the CLI method is far superior to the GUI method. We should only document that one, especially to enforce a naming convention.

We merged our intranet PRs and have these pages updated now:

https://intranet.giantswarm.io/docs/product/architecture-specs-adrs/specs/registries/
https://intranet.giantswarm.io/docs/dev-and-releng/container-registry/

done


[IntelliJ Plugin] Fix plugin for latest grammar changes

Purpose
$subject.
Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[x] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #18433   +/-   ##
=======================================
  Coverage   15.15%   15.15%
=======================================
  Files          48       48
  Lines        1254     1254
  Branches      197      197
=======================================
  Hits          190      190
  Misses       1050     1050
  Partials       14       14

Continue to review full report at Codecov.

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


t test with violin plot

Hi islab,
I was wondering if there is a way to get statistical significance for a given gene expression between 2 groups on a violin plot or make a calculation separately?
for example:
here is the expression of GZMB  between 2 groups and O would like to see if the difference is statistical significant with a simple t-test.

Thank you fore your help,
Shen

Hi @shendong124,
This is a usage question that should probably be asked on the discourse server here: https://scanpy.discourse.group/
Just a quick answer:
Check out the function sc.tl.rank_genes_groups() here https://scanpy.readthedocs.io/en/stable/api/scanpy.tl.rank_genes_groups.html#scanpy.tl.rank_genes_groups
If you have any further questions, let's continue that on the discourse.


chore(package): update travis-deploy-once to version 4.3.2

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Data Cy: Fix namespace

Data Cy: Fix namespace
This update fixes the autogenerated data-cy namespace for HSDS: React
components. The solution was to adjust how Fancy compiled the namespace to
make it more compatible with HSDS: React integration.
This required an update to Fancy to the latest v2.4.1
React Utils and Wedux were also updated to their latest versions
Fixes: https://github.com/helpscout/hsds-react/issues/595

Pull Request Test Coverage Report for Build 2064

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 2063:
0.0%

Covered Lines:
7699

Relevant Lines:
7699

💛  - Coveralls

Closing in favour of this solution:
https://github.com/helpscout/hsds-react/pull/597


Update techdocs README.md

Light editing
Signed-off-by: Bodil Björklund<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Minor edits to readme.
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[X] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[X] All your commits have a Signed-off-by line in the message. (more info)

Replaced with new PR  with correct sign-off. Sorry!


KAFKA-5439: Verify that no unexpected threads are left behind in tests

@ijuma Can you take a look, please? Thank you!

@ijuma Thank you for the review. I will merge to trunk after the builds complete. I don't think it is worth merging to 0.11.0 at this stage. Do you agree?

Yes, maybe we can leave this in trunk only.

@rajinisivaram there were a bunch of tests which failed due to unexpected threads in a recent run: https://jenkins.confluent.io/job/kafka-trunk/2198
What do you make of it?

It looks like the consumer bounce test failed, causing these assertions to fail in following tests:
17:40:31 kafka.api.ConsumerBounceTest > testClose FAILED
17:40:31     kafka.common.KafkaException: Socket server failed to bind to localhost:34860: Address already in use.

@apurvam If one broker startup fails, we don't shutdown any of the brokers. Hence the failures. As you would have already noticed, it is usually easy to tell from the list of failures the test which actually caused the threads to be left behind (all tests which weren't responsible for the failure have two classMethod failures in setUp and tearDown). Not sure why the address is in use in this failed test though. I suppose we could potentially improve cleanup for this case to avoid the additional failures.

Port conflicts are an inherent risk with those bounce tests since they use a fixed set of ports per run. Not sure what we can do to cleanup better after we hit this exception though.

@apurvam Yes, you are right, I had forgotten that ConsumerBounceTest was using fixed ports. At the moment, set up does:
servers = configs.map(TestUtils.createServer(_)).toBuffer

So if one of the brokers could not be created servers is not assigned. We can turn this into a for loop that adds to servers instead, so that tearDown will shutdown any brokers that started successfully. I will submit a PR.


Break-up the DataLakeSasTest to test individual components more granularly so that it is easier to debug and isolate failures

Follow unit-testing approach so that a test failure more easily indicates what the potential root causes of the failures are. If a larger, integration test is needed, it might be OK to keep this test, but it is better to have specific isolated tests.
https://github.com/Azure/azure-sdk-for-cpp/blob/24c0e726013d175fd6fca96b4f1bf87050aa63c9/sdk/storage/azure-storage-files-datalake/test/datalake_sas_test.cpp#L11-L18
For example, this test is failing in CI in https://github.com/Azure/azure-sdk-for-cpp/pull/897
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=645512&view=logs&j=85fce603-27e7-519e-7104-469f916f4b56&t=edb25b00-9fec-5e59-0001-6ccd95be0f23
108: [ RUN      ] DataLakeFileSystemClientTest.DataLakeSasTest
108: D:\a\1\s\sdk\storage\azure-storage-files-datalake\test\datalake_sas_test.cpp(405): error: Expected: verify_file_read(sasToken) doesn't throw an exception.
108:   Actual: it throws.
108: [  FAILED  ] DataLakeFileSystemClientTest.DataLakeSasTest (10818 ms)

But the issue doesn't reproduce locally. Figuring out what state or service API call is the prime root cause is quite difficult, let alone understanding the goal of this test.
cc @katmsft, @Jinming-Hu

Looks like this test failure is intermittent.

It hasn't failed in a long time. I think it's stable enough and the refactoring is not necessary for now.


Completed Kata - Still Listed as Unfinished

Describe the bug
Some completed Katas are still listed as "Unfinished".
To Reproduce

Start a new Kata.
Start it again.
Complete Kata.

Kata should now be listed as "complete" and gone from "Unfinished" section but it looks like some katas are listed in both "Complete" and "Unfinished" sections.
Please note that completing the Kata again does not increase the score, and neither does it clear the Kata from "Unfinished" section.
Expected Behavior
Once Kata is complete, it should be cleared out from "Unfinished" section.
Screenshots
Unfinished:

Complete:

Contexts
Please complete the following information:

Codewars username (if account related): Newman101
OS: Windows 10
Browser: Firefox
Browser Version: 70.0

Same has happened here

Why my profile has unfinished kata 'Directions Reduction' in PHP ?
I have 3 finished solutions of this kata, which are attempted and submited.
My profile is dselivanov86.

This is caused by old bug that was creating multiple records for user for a kata. The bug itself was fixed, but the affected data is not yet cleaned up. See https://github.com/Codewars/codewars.com/issues/1683 for more details if interested.
https://github.com/codewars/codewars.com/issues/1981#issuecomment-562275233

@DenSel86 In your case, there was a duplicate record created on 2019-07-20 and I just removed it.

Thanks! Sory about the duplicate issue.

except 2 which are unfinished, others are also showing in unfinished
https://www.codewars.com/users/Sudipto09/unfinished


“ENOENT: no such file or directory, open '…/docs/assets/css/main.css'”

Hello there,
I am trying to use typedoc in order to automatically generate the documentation of a angular application.
Whenever I am running npx typedoc --out docs app --exclude "**/*.{spec.ts}" --exclude node_modules --ignoreCompilerErrors it's generating markdown files and the rendering finishes, but then it aborts with the error stated above, saying that .../docs/assets/css/main.css does not exist.
From my understanding, this main.css is part of the typedoc stack and should be moved to the assets folder upon running this command.
Am I misunderstanding this? What is the reason for the failure and how can I fix this?
Since it does not seem to be a bug, just wrong usage, I sticked to "other issue".
// edit:
I have played a little bit with it and the problem only seems to appear with the "default" theme:
$ typedoc --out docs app --theme default
Loaded plugin /home/name/work/stuff/src/main/webapp/node_modules/typedoc-plantuml
Loaded plugin /home/name/work/stuff/src/main/webapp/node_modules/typedoc-plugin-markdown

Using TypeScript 3.8.3 from /usr/local/lib/node_modules/typescript/lib
Rendering [========================================] 100%
fs.js:130
    throw err;
    ^

Error: ENOENT: no such file or directory, open '/home/name/work/stuff/src/main/webapp/src/docs/assets/css/main.css'
    at Object.openSync (fs.js:451:3)
    at Object.readFileSync (fs.js:353:35)
    at Renderer.<anonymous> (/home/name/work/stuff/src/main/webapp/node_modules/typedoc-plantuml/lib/plugin.js:117:23)
    at triggerEvents (/usr/local/lib/node_modules/typedoc/dist/lib/utils/events.js:128:43)
    at triggerApi (/usr/local/lib/node_modules/typedoc/dist/lib/utils/events.js:110:13)
    at eventsApi (/usr/local/lib/node_modules/typedoc/dist/lib/utils/events.js:21:18)
    at Renderer.trigger (/usr/local/lib/node_modules/typedoc/dist/lib/utils/events.js:264:13)
    at Renderer.render (/usr/local/lib/node_modules/typedoc/dist/lib/output/renderer.js:39:18)
    at CliApplication.generateDocs (/usr/local/lib/node_modules/typedoc/dist/lib/application.js:100:23)
    at CliApplication.bootstrap (/usr/local/lib/node_modules/typedoc/dist/lib/cli.js:48:26) {
  errno: -2,
  syscall: 'open',
  code: 'ENOENT',
  path: '/home/name/work/stuff/src/main/webapp/src/docs/assets/css/main.css'
}

typedoc --out docs app --theme minimal runs without problems.
Best regards

This is an issue with typedoc-plantuml. That plugin assumes that the generated docs includes an assets/images directory, but it doesn't since typedoc-plugin-markdown doesn't create one.
It looks like that plugin has been abandoned, I know @krisztianb forked it recently, the bug might be fixed there.

No, its not fixed there either. Don't use my fork as I also abandoned it recently.
@Gerrit0 why do you think it has something to do with the images directory?
I know that the plugin is attaching some CSS to the main.css file. So the problem seems to be that the plugin assumes that there is such a CSS file while there isn't.

@Gerrit0 why do you think it has something to do with the images directory?

I was looking at the source when I replied rather than the original issue, it appeared to also add items to the images directory, assuming that it exists. Looks like you're right and this specific issue is related to the CSS file.


Fixed spacing

The spacing on the first zone reference was confusing and different.

I'll just PR the whole thing.


@keplr-wallet/types version is very old

@keplr-wallet/types version is not compatible with modern versions of the library
If you use @keplr-wallet/types in a project, the latest versions of the library are not compatible with the v0.12.23 that currently graz is using.
Latest version of @keplr-wallet/types is 0.12.156 from 13/10/2024.
The version in use for latest graz (version 0.1.26 from 20/11/2024) is still 0.12.23 from 7/9/2023 (more than one year older and 133 versions of difference). Bearing in bind @keplr-wallet/types updates big changes (sometimes not compatible with previous versions) even for bug versions, that difference of versions is a lot.
Solution liked
Update dependencies to the latests ones.
Alternatives to considered
Move @keplr-wallet/* to peer dependencies if possible
Additional context
Here an example of TS error when providing ChainInfo to GrazProvider component:
<GrazProvider
    grazOptions={{
        chains: [myChainInfo],
    }}
>
// ...
</GrazProvider>

Type 'import("/.../node_modules/@keplr-wallet/types/build/chain-info").ChainInfo' is not assignable to type 'import("/.../node_modules/graz/node_modules/@keplr-wallet/types/build/chain-info").ChainInfo'.
  Types of property 'nodeProvider' are incompatible.
    Type '{ readonly name: string; readonly email?: string | undefined; readonly discord?: string | undefined; readonly website?: string | undefined; } | undefined' is not assignable to type '{ readonly name: string; readonly email: string; readonly website?: string | undefined; } | undefined'.
      Type '{ readonly name: string; readonly email?: string | undefined; readonly discord?: string | undefined; readonly website?: string | undefined; }' is not assignable to type '{ readonly name: string; readonly email: string; readonly website?: string | undefined; }'.
        Types of property 'email' are incompatible.
          Type 'string | undefined' is not assignable to type 'string'.
            Type 'undefined' is not assignable to type 'string'.ts(2322)

yes we will update this

fixed in graz v0.1.28


minor spelling fixes

Coverage remained the same at 96.85% when pulling 750a02bb65a9edb0fd0978bbc963db574c3af4a7 on cgar:spelling into 1ff561357b079b1c83d102722fec2c202918bb42 on ccpgames:master.

Hi cgar. You should open a PR against rgalanakis/brennivin so it can be merged into master.


Manel vignette individual genetic distances SNP

New vignette SNP

@smanel it looks like this is on a very old branch that is way behind the master branch. I went ahead and rebased your changes on the latest version of master. The result is on the Manel_new branch - if this looks OK to you, please go ahead and submit that as a pull request (using the Github web UI).
I also noticed that your branch that use here came down as a forced update. This means you either deleted and then recreated the branch, or ignored some upstream changes on the branch and forced an update from your local clone. Either of these can cause its own set of problems. It's best not to reuse old branches once the pull request has been merged, and not to reuse the branch name either. Just consider the branch "spent" after merge (it's fine to update until it's merged, though). (It's perfectly possible to reuse a branch, but that requires some technical details about working with git that we aren't covering in our tutorial, and that you wouldn't need to care about if you just update master in your local clone after a merge and branch off from scratch for future work.)

I should add that the rebased version on Manel_new shouldn't give you the error above either, so I wouldn't bother understanding what the problem is until a build error shows up in the new branch.

Update: build passes for the rebased version in Manel_new.

I'm going to close this pull request; please open a new one with the Manel_new branch.


Only text in navigation menu items are links

In the example OSCAL Viewer application's navigation menu, only the text of the links actually operates as a link; though the entire menu item has the you-can-click-this style of pointer. So in the image below, you have to click the blue text; if you click any of the white area, the page will not navigate.

Acceptance Critera

Clicking anywhere in a menu item (on the link or around it) works correctly to navigate to the intended page

Implementation Notes
This probably means doing something like turning the entire <MenuItem> into a button with a <Navigate> or "flipping" the <Link> and <MenuItem> but I think it's improper for an anchor to contain complex elements.

@kylelaker, can you retroactively estimate the story points on this one?

@rgauss I added an estimate based on my understanding of the complexity relative to other tickets in this sprint; happy to discuss on/after standup if we want to talk about it as a team though!


GH-34687: [CI][Python] Create job to remove old nightly wheels from gemfury

Rationale for this change
We haven't been cleaning nightly wheels from gemfury ever.
What changes are included in this PR?
Add a new job that should clean them. This is mainly reused from the one on arrow-adbc: https://github.com/apache/arrow-adbc/blob/main/ci/scripts/gemfury_clean.rb
Are these changes tested?
I will trigger via crossbow
Are there any user-facing changes?
No

Closes: #34687

@kou @lidavidm @jorisvandenbossche  before trying (and removing a bunch of wheels) I wanted to validate we are ok with keeping 90 days of nightly pyarrow wheels.

Oh, I meant it as "at least I'm ok with this" :) personally I think fewer days is fine, even.

I'm OK with 90 days.

I think 90 days is more than long enough, and indeed can probably even be a bit less. The nightly wheels are meant to test development versions, so you should never rely on the exact wheel long term.

I am running this locally first. Just for reference the first wheel on gemfury was:
Yanked pyarrow 0.16.1.dev342 (created 2020-03-25T17:56:25.572+00:00)

I assume we will trigger this with a cron job in crossbow?

This will already be covered on the wheel-* group and run as part of the nightly-packaging.

Side note: another bit of release infra that is duplicated from one of the arrow subprojects. IMO we should really find a way to avoid all of this C&P

Maybe we could open an issue, discuss some approaches and propose something on the ML. Without giving too much thought we could move some of this tooling / release to its own repo and either clone or use a submodule.

@github-actions crossbow submit wheel-clean

The failure is expected until the PR is merged because crossbow can't find the ruby script when cloning arrow/main for cleaning the wheels.

Yes, it would be good to discuss sharing some of the infrastructure now that we have three/four subprojects using similar infrastructure (not sure how similar arrow-julia is to the rest).