Telerik removal results in erros when loading the Resource manager

Steps to reproduce
List the precise steps to reproduce the bug:

Follow the guide to remove the old DAM and telerik: https://docs.dnncommunity.org/content/getting-started/setup/telerik-removal/index.html
Open any of the pages containing the RM module.

Current behavior
The module does not load and the persona bar shows no icons.
Screenshots

Error information
Following errors on log:
Uncaught SyntaxError: invalid regular expression flag v [js:1586:17](https://a.b/DependencyHandler.axd/2316ea21dbd5369ed8a85a9893712190/43/js)
Uncaught SyntaxError: invalid regular expression flag v[js:1586:17](https://a.b/DependencyHandler.axd/2316ea21dbd5369ed8a85a9893712190/43/js)
TypeError: can't access property "MenuItems", menuStructure is undefined [main.js:52:21](https://a.b/DesktopModules/admin/Dnn.PersonaBar/scripts/main.js?cdv=43)
Uncaught TypeError: can't access property "MenuItems", menuStructure is undefined [util.js:364:21](https://a.b/DesktopModules/admin/Dnn.PersonaBar/scripts/util.js?cdv=43)

Affected version

[ X ] 09.10.02

Affected browser

[ ] Chrome
[ X ] Firefox
[ ] Safari
[ ] Internet Explorer 11
[ ] Microsoft Edge (Classic)
[ ] Microsoft Edge Chromium

Is it possible that page has some custom javascript on it that expects the file manager to be there. I am curious what throws the very first error in the console, if you click on the link, it should point you to that place...

Thanks @eXistenZe - closing per above and referencing #4885


Load shiro.ini error on v5.1.3

5.1.3
Shiro Authentication set cas.authn.shiro.config.location=classpath:shiro.ini ,run after,some error
I saw cas to load shiro.ini from the system temp dir but not exists and then thows exceptions

Please review the issue template to understand the guidelines before submitting issues. The report here simply does not contain enough data for it to merit any further diagnostics. The issue template presents itself when you begin to submit a new issue. It is also available here.
Additional guidelines can be found here.

Thank you, I will.
First, add cas-server-support-generic dependency in the pom.xml,   add a line cas.authn.shiro.config.location=classpath:shiro.ini in the application.paoperties, add file shiro.ini to resources directory ,next I run the cas-server, that'is ok. But I find shiro.ini the same as project resources in my system of temporary directory.
After delete  the shiro.ini in my system of temporary directory and rerun.
I try again and again.
Debug:
https://github.com/apereo/cas/blob/b6cfef38b6615aa1ceee3f0956083c42e01b8a34/support/cas-server-support-generic/src/main/java/org/apereo/cas/adaptors/generic/config/ShiroAuthenticationEventExecutionPlanConfiguration.java#L65
Next to execute loadShiroConfiguration
Expression shiro.getConfig().getLocation().exists() is true.The description is illustrated in classpath.
https://github.com/apereo/cas/blob/b6cfef38b6615aa1ceee3f0956083c42e01b8a34/core/cas-server-core-util/src/main/java/org/apereo/cas/util/ResourceUtils.java#L162
I don't know when cas created  cas directory for me on the temporary directory.
Is there something wrong with my configuration?
Thanks again.
Caused by: org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'scopedTarget.shiroAuthenticationHandler' defined in class path resource [org/apereo/cas/adaptors/generic/config/ShiroAuthenticati
onEventExecutionPlanConfiguration.class]: Bean instantiation via factory method failed; nested exception is org.springframework.beans.BeanInstantiationException: Failed to instantiate [org.apereo.cas.authentication.Authenticatio
nHandler]: Factory method 'shiroAuthenticationHandler' threw exception; nested exception is java.lang.RuntimeException: java.io.FileNotFoundException: C:\Users\ADMINI~1\AppData\Local\Temp\cas\shiro.ini (系统找不到指定的路径。)
        at org.springframework.beans.factory.support.ConstructorResolver.instantiateUsingFactoryMethod(ConstructorResolver.java:599)
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateUsingFactoryMethod(AbstractAutowireCapableBeanFactory.java:1173)
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1067)
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:513)
        at org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:483)
        at org.springframework.beans.factory.support.AbstractBeanFactory$2.getObject(AbstractBeanFactory.java:345)
        at org.springframework.cloud.context.scope.GenericScope$BeanLifecycleWrapper.getBean(GenericScope.java:359)
        at org.springframework.cloud.context.scope.GenericScope.get(GenericScope.java:176)
        at org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:340)
        at org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:197)
        at org.springframework.aop.target.SimpleBeanTargetSource.getTarget(SimpleBeanTargetSource.java:35)
        at org.springframework.aop.framework.JdkDynamicAopProxy.invoke(JdkDynamicAopProxy.java:192)
        at com.sun.proxy.$Proxy118.getName(Unknown Source)
        at org.apereo.cas.authentication.DefaultAuthenticationEventExecutionPlan.registerAuthenticationHandlerWithPrincipalResolver(DefaultAuthenticationEventExecutionPlan.java:40)

@mmoayyed 5.2.0-RC3 MongoDb Events Document is cas.events.mongodb I saw is cas.events.mongo, Right?
https://apereo.github.io/cas/development/installation/Configuration-Properties.html#mongodb-events

I'd recommend that you move away from classpath-based configuration and define the file under /etc/cas or some such path.


Add ballot box emoji as favicon

Based on https://css-tricks.com/emojis-as-favicons/
https://emojipedia.org/ballot-box-with-ballot/

Adorable, and we all love the awesome hack of an emoji in SVG.


Direct link to GitHub for Content feedback is broken

On this documentation page, the direct link to leave feedback on GitHub is broken.

Instead, it points to https://docs.microsoft.com/en-us/dotnet/api/%7B0%7D

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

ID: 71654f5f-8c0e-87c9-8b8c-ad17f578d427
Version Independent ID: ab6d5a69-3cbc-fcf7-10f2-6e2deb1ed30c
Content: DataType Enum (System.ComponentModel.DataAnnotations)
Content Source: xml/System.ComponentModel.DataAnnotations/DataType.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: riande

Thanks @Structed for letting us know. I've moved this feedback to https://github.com/MicrosoftDocs/feedback/issues/807 for our engineering team to take a look.


[Feature request] Use mdns (avahi/zeroconf) for re-discovering a device on the network

Feature suggestion
Detailed Description
As the title suggests, kore could use mdns so as to rediscover a device from the network that is already registered in it.
The following screenshot is from a device that was already added on kore. It is the third one on my network that runs kod and (right now) it is the only one without a static ip.
I don't know how, but when I first set it up on kore, it showed up instantly, without having to enter an ip etc, so I think mdns is the reason. I use mdns on my networked devices (avahi/zeroconf under linux, bonjour on windows) since last year, so that I no longer have to look up for every device's ip (from the ones that don't use static).
Context
The reason is simple. Kodi has built in mdns capabilities (via avahi/zeroconf) and, usually, the devices that it runs on get their ip via dhcp. So, when one is added in kore, it is registered through its ip. And when that ip changes, it's settings inside kore must be changed as well so it can discover the device again.
Possible Implementation
No idea, but I know that android does not implement mdns at system level (unlike other oses), so any solution will be specific for kore.
Additional context, screenshots or links
Screenshot of the "issue"

A few days later, the ip was changed to e.g. <IP_ADDRESS>, so I had to change it inside kore. If kore was using mdns, setting its hostname as mxqpro4k.local (= name of the device + .local suffix, easy discovery via mdns), it would find the device regardless of its ip.

Kore does use mDNS, as implemented by [jmDNS](https://github.com/jmdns/jmdns in https://github.com/xbmc/Kore/blob/master/app/src/main/java/org/xbmc/kore/ui/sections/hosts/AddHostFragmentZeroconf.java
I find that mDNS is somewhat fickle, but the fact that you're not seeing it is most likely an issue on your network.

I can see (= ping to it, connect via web browser etc) from my pc(s) which runs avahi/bonjour, so it is not an issue of the box or the network.

Will kore re-discover a device that is already registered in it? For instance, my pc that uses a static ip and was registered manually in kore with this static ip, can it be rediscovered again via zeroconf as mypc.local?
I tried it now, for both my pc and an old laptop running libreelec (both using static ip, already registered in kore and discoverable via zeroconf) and kore did not find any of the 2.
On the other hand, a skin (bello 7) borked my kodi installation on the tvbox I mentioned above, so I had to set it again from scratch. I removed it from kore and readded it, so now I am waiting for it to get a fresh ip so as to recheck.

I tried substituting the ips with their .local hostnames, e.g. <IP_ADDRESS> became mypc.local, but kore failed to find them when I pressed test, so I reverted back to their ips.
Are you sure it does use mdns when it comes to re-discovering the devices on the network?
As usual, all of them are pingable, ssh-able, browsable via samba and connectable though their web interfaces with their .local names. Proofs
$ ping -c 4 mxqpro4k.local
ping: socket: Address family not supported by protocol
PING mxqpro4k.local (<IP_ADDRESS>) 56(84) bytes of data.
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=1 ttl=64 time=3.43 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=2 ttl=64 time=4.07 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=3 ttl=64 time=2.75 ms
64 bytes from <IP_ADDRESS> (<IP_ADDRESS>): icmp_seq=4 ttl=64 time=3.88 ms

--- mxqpro4k.local ping statistics ---
4 packets transmitted, 4 received, 0% packet loss, time 3005ms
rtt min/avg/max/mdev = 2.746/3.530/4.068/0.508 ms

I tried kore on a friend's network and devices. Different phones, different tvbox, different router.
On Saturday that I first set up kore, the tvbox had <IP_ADDRESS> and kore lists this as its ip. Today, 5 days later, the tvbox is on <IP_ADDRESS> and kore can not find it anymore. It can not re-discover it via its .local domain, exactly as it does for my network.
And it happened on both his phones, not on just one.
Thus, I assume mdns is not actually working for kore and it is not an issue of my network.


dist must be a Distribution instance

(base) klaus@ubuntu:~/Desktop/pytorch-retinanet/lib$ bash build.sh
Compiling nms kernels by nvcc...
Including CUDA code.
/home/klaus/Desktop/pytorch-retinanet/lib/nms
Traceback (most recent call last):
File "build.py", line 31, in
extra_compile_args=['-std=c99']
File "/home/klaus/conda/lib/python3.7/site-packages/torch/utils/cpp_extension.py", line 228, in init
super(BuildExtension, self).init(*args, **kwargs)
File "/home/klaus/conda/lib/python3.7/site-packages/setuptools/init.py", line 163, in init
_Command.init(self, dist)
File "/home/klaus/conda/lib/python3.7/distutils/cmd.py", line 57, in init
raise TypeError("dist must be a Distribution instance")
TypeError: dist must be a Distribution instance
i do not know what the problems is!!! does anybody know,please tell me!

I also meet it, do you solve it?

syr@amax:~/retinanet-masking/lib$ bash "/home/syr/retinanet-masking/lib/build.sh"
Compiling nms kernels by nvcc...
nvcc fatal   : Unsupported gpu architecture 'compute_30'
Including CUDA code.
/home/syr/retinanet-masking/lib/nms
Traceback (most recent call last):
File "build.py", line 24, in
ffi = BuildExtension(
File "/opt/anaconda3/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 378, in init
super(BuildExtension, self).init(*args, **kwargs)
File "/opt/anaconda3/lib/python3.8/site-packages/setuptools/init.py", line 172, in init
_Command.init(self, dist)
File "/opt/anaconda3/lib/python3.8/distutils/cmd.py", line 57, in init
raise TypeError("dist must be a Distribution instance")
I can't compile it too!!


[bug] [langchain] Cannot parse message of type: ToolMessage

Describe the bug
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/openinference/instrumentation/langchain/_tracer.py", line 274, in wrapper
    yield from wrapped(*args, **kwargs)
  File "/usr/local/lib/python3.10/dist-packages/openinference/instrumentation/langchain/_tracer.py", line 419, in _parse_message_data
    raise ValueError(f"Cannot parse message of type: {message_class_name}")
ValueError: Cannot parse message of type: ToolMessage

Expected behavior
No ValueError to be raised.
Additional context
I think _parse_message_data in https://github.com/Arize-ai/openinference/blob/21b83e9207e68d561a26e4a4b5a7bb373cc2892b/python/instrumentation/openinference-instrumentation-langchain/src/openinference/instrumentation/langchain/_tracer.py#L400 just needs to be updated to include ToolMessage (source)

@yeesian Thanks for the issue report! Do you have a snippet to reproduce?

Does the following work for you?

Installation:
!pip3 install --upgrade --quiet \
    langchain \
    langchain-google-vertexai \
    openinference-instrumentation-langchain \
    opentelemetry-sdk \
    requests

Reproducing the issue:
from openinference.instrumentation.langchain import LangChainInstrumentor
from opentelemetry import trace as trace_api
from opentelemetry.sdk import trace as trace_sdk
from opentelemetry.sdk.trace.export import ConsoleSpanExporter, SimpleSpanProcessor

from langchain import agents
from langchain_google_vertexai import ChatVertexAI
from langchain_core.prompts import ChatPromptTemplate
from langchain.tools import tool

tracer_provider = trace_sdk.TracerProvider()
trace_api.set_tracer_provider(tracer_provider)
tracer_provider.add_span_processor(SimpleSpanProcessor(ConsoleSpanExporter()))

LangChainInstrumentor().instrument()

@tool
def get_exchange_rate(
    currency_from: str = "USD",
    currency_to: str = "EUR",
    currency_date: str = "latest",
):
    """Retrieves the exchange rate between two currencies on a specified date."""
    import requests
    response = requests.get(
        f"https://api.frankfurter.app/{currency_date}",
        params={"from": currency_from, "to": currency_to},
    )
    return response.json()

model = "gemini-1.5-pro-001"
llm = ChatVertexAI(
    model_name=model,
    project="your-gcp-project", # https://cloud.google.com/resource-manager/docs/creating-managing-projects
    max_tokens=500,
    temperature=0.5,
)

tools = [get_exchange_rate]

prompt = ChatPromptTemplate.from_messages(
    [
        ("human", "{input}"),
        ("placeholder", "{agent_scratchpad}"),])

agent = agents.create_tool_calling_agent(llm, tools, prompt)
agent_executor = agents.AgentExecutor(agent=agent, tools=tools, verbose=True)
agent_executor.invoke({"input": "What is the exchange rate from US dollars to Swedish currency today?"})

This issue seems to be affecting only gemini. OpenAI seems to be fine.

@yeesian Thank you for bring this to our attention! This should be fixed in the latest release.
openinference-instrumentation-langchain>=0.1.19

Thank you for the quick fix! I can confirm that it is working on my end with >=0.1.19 too

Awesome, thanks @yeesian

I am getting similar errors. Is there a compatibility issue with OpenAI inputs (vision models)

@rhlarora84 Can you file us an issue with a snippet to reproduce?


Add imperial support

Currently it is only possible to request voice instructions with metric units. This is a first approach to enable imperial voice units. Would be happy to get your feedback

Thanks for providing this PR! This already looks really good.
One thing we need to think about is that you changed the spoken distances for imperial vs. metric. So if there are for example 1400m between two turn instructions, the 1km turn instruction would be missing, because 1 mile is at ~1600m. So there is a trade off between using even numbers and amount of instructions. Only receiving the 400m turn instruction could be a bit late. WDYT @karussell?
Somehow the VoiceInstructionConfig gave me a bit of a headache :). I find it hard to understand what happens exactly and I find myself switching between the VoiceInstructionConfig and NavigateResponseConverter. Maybe we could move more logic into the VoiceInstructionConfig and try to abstract as much as possible accross the three different subclasses?
The ideal case would be that we can reduce the for loop like this:
for (VoiceInstructionConfig voiceConfig : distanceConfig.voiceInstructions) {
   if(voiceConfig.shouldBeAdded([parameters... like distance etc.])){
      putSingleVoiceInstruction(voiceConfig);
   }
}

The overall goal would be to remove all the instanceof differentiation. I think this would clean up the code a lot. WDYT?

Nice.
Yeah. I was hoping for your experience with the timing/position of the voice instructions because i was not sure about the values either.
And i am on your side with the instanceOf logic. I think moving more logic into VoiceInstructionConfig would also make it easier to test it. I will have another look it this part.

I refactored the VoiceInstructionConfig logic and moved a bit of logic. Looks better now i think :)

@boldtrn what do you think about the refactored code?

Sorry for my late reply now. Holiday time :D
I am glad to hear that you like it. I am also not sure about the distances. But. yeah. It is easy to change now :)

Thanks @skienzl and @boldtrn !

@skienzl Can you have a look into https://github.com/graphhopper/graphhopper/pull/2071? We would like to merge this repo into the core to make it easier to maintain and also consume.  And also I need a short confirmation that I can squash all changes into one commit (see discussion in the issue). after that he co-authorship is still maintained and visible, I think.


prevent issues from multiple native symbol sections

i suspect it's better to have a bit less cryptic description.


TIC-80 is now MIT!

https://github.com/nesbox/TIC-80/blob/master/LICENSE

Also, it can run javascript as well as Lua

Neat! Updated.


[Summer of code] Add admin tools for controller mode

Make sure set the target branch to develop
What is the purpose of the change
Tracking issue: https://github.com/apache/rocketmq/issues/4330
Brief changelog
Add two admin tools for controller mode:

Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Codecov Report

Merging #4388 (71107fd) into 5.0.0-beta-dledger-controller (97e6cec) will decrease coverage by 0.27%.
The diff coverage is 0.00%.

@@                         Coverage Diff                         @@
##             5.0.0-beta-dledger-controller    #4388      +/-   ##
===================================================================
- Coverage                            43.54%   43.27%   -0.28%
+ Complexity                            6348     6328      -20
===================================================================
  Files                                  835      839       +4
  Lines                                59369    59629     +260
  Branches                              8094     8123      +29
===================================================================
- Hits                                 25854    25804      -50
- Misses                               30172    30478     +306
- Partials                              3343     3347       +4

Impacted Files
Coverage Δ

.../rocketmq/broker/hacontroller/ReplicasManager.java
0.00% <0.00%> (ø)

...ocketmq/broker/processor/AdminBrokerProcessor.java
25.64% <0.00%> (-0.21%)
:arrow_down:

...g/apache/rocketmq/client/impl/MQClientAPIImpl.java
24.01% <0.00%> (-0.48%)
:arrow_down:

...in/java/org/apache/rocketmq/common/EpochEntry.java
0.00% <ø> (ø)

...g/apache/rocketmq/common/protocol/RequestCode.java
0.00% <ø> (ø)

...rocketmq/common/protocol/body/EpochEntryCache.java
0.00% <0.00%> (ø)

...rocketmq/common/protocol/body/InSyncStateData.java
0.00% <0.00%> (ø)

...he/rocketmq/controller/impl/DledgerController.java
72.67% <0.00%> (-0.86%)
:arrow_down:

...cketmq/controller/impl/event/ControllerResult.java
68.00% <0.00%> (-5.92%)
:arrow_down:

...q/controller/impl/manager/ReplicasInfoManager.java
61.80% <0.00%> (-6.15%)
:arrow_down:

... and 26 more

Continue to review full report at Codecov.

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

Coverage decreased (-0.3%) to 47.5% when pulling 71107fda53adcccbb146b062a12c6becb618c51f on hzh0425:feature/controller-tools into 97e6cecbf9d6df1c548b110363b2928a7348da66 on apache:5.0.0-beta-dledger-controller.


Godot 3.2 GLES2 macOS + HTML5 stuttering (some frames idle time > 60 ms)

Godot version:
3.2.3.stable
OS/device including version:
MacOS Catalina 10.15.7
MacBook Pro (13-inch, 2017, Four Thunderbolt 3 Ports)
CPU: 3,1 GHz Dual-Core Intel Core i5
RAM: 16 GB 2133 MHz LPDDR3
GFX: Intel Iris Plus Graphics 650 1536 MB
Backend: GLES2
Issue description:
Game (even empty project) is stuttering. Getting spikes with frame times > 60 ms, whilst most frames have < 1ms when selecting GLES2 renderer. For GLES3 all frames stay < 1ms and everything runs smooth. HTML5 export seems to have the same issue.
Note that there are no scripts running / no expensive rendering going on. It is purely the engine running.
Profiling GLES2: Very slow frames!

For comparison GLES3:

In order to make sure it is not vsync related, provfiling with vsync disabled
GLES2:

Note: you can clearly see the spikes > 60ms once or twice every 12-16 frames
For comparison GLES3:

Most frames < 1ms, only one random frame with still perfect 14ms
Steps to reproduce:
Seeing the same behavior already in a empty Project:
Start from a empty project, GLES2 backend. Create a empty Node2D. Start profiling.
Minimal reproduction project:
Plain default template with empty Node2D already shows this behavior.

See also several issues related to performance problem on some Macs, including:
#39758
#17584

I am sure a similar issue (spikes on empty project) was already reported, but I can't find it atm.

The HTML5 version and the MacOS one share very little platform code. I doubt the two things are related unless it's a driver issue with some GL function which is used by both the desktop version (GLES2) and the browser (WebGL).
Also worth mentioning that you can't disable vsync in HTML5 since the rendering is done in the animation frame callback from the browser.

See #39758 for a test build with a possible fix. (Note the test build is GLES3 but is easy to extend to GLES2).

Fixed by #42734.


Oracle DDL - Unchecked fully qualified name does not remove schema name

Reproduce:

with Oracle connection generate DDL for any object in any schema
with popup window open uncheck the "Use fully qualified names" checkbox

Expected: The DDL should regenerate with the schema name removed
Actual: It does regenerate but schema name remains, no actual change
Occurs: 100% of the time
DBeaver versions : 4.3.3+ (and probably earlier as well)
Additional info:
In OracleUtils class the below call can be added with the value conditional on the "Use fully qualified names" preference setting, this will correct table DDL generation.
DBMS_METADATA.SET_TRANSFORM_PARAM(dbms_metadata.SESSION_TRANSFORM, 'EMIT_SCHEMA', false);

will be fixed in scope of #7042


generate merkle path proof to the global mst

closed at https://github.com/okx/proof-of-reserves-v2/pull/41


ES6-modularization

Issue
Use ES6 modules to modularise JavaScript
What has Changed
-Index.js has been reduced to a single line of code to initialize the application
-Created folders based on function
-Code has been split into multiple files to maintain readability and function
-firebase_quill_init.js renamed quill_init.js to avoid confusion
Folders*

/init

avatars.js  - a single exported variable of the avatar list
init.js - exports default function to initialize application
setUser.js - sets or gets user from Local Storage initialized in init()
showContainers - removes "hidden" class from containers initialized in init()

/events

eventHandlers.js - moves handlers for event listeners to separate file
eventListeners.js - moves all event listeners to a single instance to be initialized upon init()
serviceWorker.js - moves serviceWorker handling to events folder

/lessons

helper.js - moves helper function to own file for creation of lessons
index.js - leaves all lesson functions in single index file, might need to be split more

/firebase
-commented out firebase.js to be removed

So I think I fixed the tag issues, wasn't up to date in my local repository when I finished up so I didn't see the commit four days ago. I think I fixed what was necessary there.
I don't know what I am doing wrong that I can't recreate those caching errors though. All of the files from the sw.js CACHE_FILES are caching for me. I don't know if you can explain any differently how to recreate those errors.

So I think I fixed the tag issues, wasn't up to date in my local repository when I finished up so I didn't see the commit four days ago. I think I fixed what was necessary there.
I don't know what I am doing wrong that I can't recreate those caching errors though. All of the files from the sw.js CACHE_FILES are caching for me. I don't know if you can explain any differently how to recreate those errors.

Here is what my cache looks like after loading the page.

Here is what my cache looks like after loading the page.

So I think I fixed the tag issues, wasn't up to date in my local repository when I finished up so I didn't see the commit four days ago. I think I fixed what was necessary there.
I don't know what I am doing wrong that I can't recreate those caching errors though. All of the files from the sw.js CACHE_FILES are caching for me. I don't know if you can explain any differently how to recreate those errors.

Just checked your latest changes locally and the tagging issues are fixed, thanks!
Regarding the sw caching errors, I've tested this again and it's working fine for me too. Let's get this merged in, if there are any issues we can always raise another ticket to address them.
Thanks again!

So I think I fixed the tag issues, wasn't up to date in my local repository when I finished up so I didn't see the commit four days ago. I think I fixed what was necessary there.
I don't know what I am doing wrong that I can't recreate those caching errors though. All of the files from the sw.js CACHE_FILES are caching for me. I don't know if you can explain any differently how to recreate those errors.

Just checked your latest changes locally and the tagging issues are fixed, thanks!
Regarding the sw caching errors, I've tested this again and it's working fine for me too. Let's get this merged in, if there are any issues we can always raise another ticket to address them.
Thanks again!


Create Seeds for Tables

[ ] Users
[ ] Orders
[ ] Products
[ ] Orders_Products

Be sure to reverse this order for dropping the tables.

Got the first table "users" created, still need to follow it through with insert users. Once that vertical slice is working it should be quick work to generate the other, smaller tables.


Why do docs recommend opening/closing sqlite3 connection for each request?

In this page the Flask docs present opening and closing a connection per request as the "official" way of working with sqlite3.
This does not make sense to me - surely opening and closing connections has some overhead, which can be avoided if a connection is shared across an entire application. Perhaps this overhead is negligible here, but I still couldn't get over it.
The problem is that a global connection created with sqlite3.connect(DATABASE) will raise an Objects created in a thread can only be used in that same thread error, due to the way (I assume) Flask processes web requests in worker threads.
As it turns out and as explained here this can be safely resolved by creating the connection with sqlite3.connect(DATABASE, check_same_thread=False) instead, as long as sqlite3.threadsafety == 3 (the available variant of sqlite3 is in serialized mode).
Given this information, shouldn't the documentation suggest a global shared connection as the standard way of working with sqlite3?

If you use sqlite, performance is unlikely to be your main goal anyway. It's just something simple that works well for an example app.
In a real application you are likely using sqlalchemy or something else to maintain a connection pool and thus do not have the problem. Connecting to the sqlite database on startup / on first request just adds complexity though, for no real gain.
Also, by sharing a connection there's a good chance that you cannot have separate transactions for separate parallel requests, can you?
This is something on that topic I fond when googling "sqlite one connection per request": https://sqlite.org/forum/info/ff2fe46b1ff102bfbc78c112ea4eb25105354571b6c7551956d3cd72fe3e4624

It's just something simple that works well for an example app.

But then maybe that should at least be made clear in the docs? Currently it is more or less implied that the presented way is the best approach.
I think this is particularly bad because one may interpret this to mean that it is always the best approach to open a connection per request, for any database!

Also, by sharing a connection there's a good chance that you cannot have separate transactions for separate parallel requests, can you?

It does make sense that if Flask is multithreaded (perhaps you can clear this up for me), transactions that consist in more than one call to connection.execute() could find themselves mixed with some other thread's calls.
Although that seems like a smaller issue to me, and one that could be pointed out as a use case for having a request-specific connection.
In a way, this is also Python's fault for defaulting check_same_thread to True even if sqlite3 is in serialized mode. If that weren't the case, I probably wouldn't even have checked Flask's sqlite3 guide.

Well, we're definitely not going to do a single global connection, because that's definitely not safe. And we're not going to implement a connection pool for simple examples. The pattern of "get connection, store on g, return connection" is what's being demonstrated by the docs, and remains the same whether you're pooling or not.
Use Flask-SQLAlchemy (or just SQLAlchemy) if you need a pool. That said, even SQLAlchemy thinks that SQLite doesn't need a pool: https://docs.sqlalchemy.org/en/14/dialects/sqlite.html#threading-pooling-behavior

SQLite file-based connections have extremely low overhead, so pooling is not necessary. The scheme also prevents a connection from being used again in a different thread and works best with SQLite’s coarse-grained file locking.

If you're at the point where the SQLite connection is your bottleneck, you're way, way beyond the scope of this documentation.

Well, we're definitely not going to do a single global connection, because that's definitely not safe.

@davidism Why is it so clearly not safe? Wouldn't it be similarly unsafe to have a single connection for an entire web request - which could be arbitrarily complex?
And regardless of whether we agree on the reasoning - shouldn't it be presented in the docs, so there is enough information to avoid readers having to make potentially erroneous assumptions?
The thread-global g pattern is not super elegant, and I find it hard to rely on it for database connection management. It would be great not to have to!
You have good arguments as to why this doesn't matter enough to think about - I just think what does matter is the readers - this isn't just a random documentation page, it builds up background knowledge.


[Dynamic Regions] with v2, the compaction flow has significant impact on the foreground QPS

Development Task
The compaction flow has huge impact on application query latency and QPS.

The below configures does not solve the QPS jitter issue.
    config: |
      log-file = "/var/lib/tikv/log/tikv.log"

      [raft-engine]
        purge-threshold = "50GB"

      [resource-control]
        enabled = false

      [rocksdb]
        max-background-flushes = 4
        rate-bytes-per-sec = "0B"
        write-buffer-flush-oldest-first = true
        [rocksdb.defaultcf]
          compression-per-level = ["zstd", "zstd", "zstd", "zstd", "zstd", "zstd", "zstd"]

      [storage]
        engine = "partitioned-raft-kv"
        reserve-space = "0MB"
        [storage.io-rate-limit]
          level-zero-compaction-priority = "low"
          max-bytes-per-sec = "400MiB"

/assign @busyjay

The issue should be greatly mitigated due to many optimizations done in 7.1.  But need to verify with this test case. Downgrade it to major for now.

with the flow control
      [storage]
        engine = "partitioned-raft-kv"
        [storage.io-rate-limit]
          max-bytes-per-sec = "300MiB"


Fix CI

Updated actions to use node20
Updated vcpkg slice to capture new libgit2 1.8.0 with fetch depth options
Added setup-msbuild action as msbuild doesn't exist on runner out of the box

What's the reasoning behind truncating the version in ./version

There is no reason. I guess, its because of broken autoversion in cloned repo. I'll restore status quo when PR will be ready to merge

I've made a new commit where explicitly specify the version of libgit2 without updating the vcpkg baseline, to prevent side effects.

Yeah, I got it when saw Resolve conflicts 😄 Nice job. I'll revert all about libgit2 and stuff
What about other changes? In my cloned repo I haven't success untill I've added setup-msbuild. My guess is GitHub Action caches in your repo.

Yeah, you're totally right. I must have changed something removing the setup of MS build, but I didn't matter at the time because it was already cached. Good thing you caught it.

Just revert the changes made to vcpkg-configuration.json and version and it should be good to merge. Also, just checking, your proposed changes to vcpkg is just a submodule update correct? GitHub doesn't provide much information.

Something seems to have messed up in one of these commits. It seems libgit2 isn't being statically linked and its requiring zlib1.dll. It builds correctly locally, but not from the CI

Not entirely sure what caused it, but it seems one of our recent commits messed up Millennium. Its been fixed but I've reverted your changes.

Oh, okay. Sadly it's expected.

Oops, missed your message, GitHub decided it wasn't going to notify me. That's a great idea and its been something I've wanted to do however I just didnt have time to focus on it.


fix(dependencies): make react-native-app-intro work with React 16

Instead of finding alternative and changing the respective code, I just went along to help fix react-native-app-intro by including prop-types as needed thereby making it compatible with React 16.^
Kindly review and send feedback where needed.
#395

Thanks a million @Ernoff, looks like you've addressed publishing to a package #400


'vapp_storage_profile'

Hey,
We would like to configure our microbosh to provision vms for our delployments with the Gold storage policy.
media_storage_profile: does not cover this, and we would like to set the storage policy across a vapp.
We have tried setting vapp_storage_profile: Gold in both
properties:
  vcd:
    entities:
      vapp_storage_profile: Gold

and
plugin: vcloud
vcds:
  - entities:
       vapp_storage_profile: Gold

in our microbosh deployment manifest, but neither has worked.
On inspecting the cpi.json file in /var/vcap/jobs/cpi/config we saw that this property does not exist.
Is there support for this? Or are we setting this property in the wrong section of our microbosh deployment manifest.

It does not look like we expose vapp_storage_profile option for configuration even though CPI supports it. If you are using bosh-init (http://bosh.io/docs/init-vcloud.html) you should be able to:

modify bosh-vcloud-cpi-release to add vapp_storage_profile property (similar to https://github.com/cloudfoundry-incubator/bosh-vcloud-cpi-release/blob/master/jobs/cpi/spec#L23)
modify bosh-vcloud-cpi-release to add vapp_storage_profile value to cpi.json (similar to https://github.com/cloudfoundry-incubator/bosh-vcloud-cpi-release/blob/master/jobs/cpi/templates/cpi.json.erb#L17)
then bosh create release --force --with-tarball in bosh-vcloud-cpi-release repo
use the tarball file in a bosh-init manifest to specify cpi release via file:// url.

I've also filed a story to add it: https://www.pivotaltracker.com/story/show/102693284. PR is welcome though.

Btw official home for vcloud_cpi is now at https://github.com/cloudfoundry-incubator/bosh-vcloud-cpi-release.

Have opened a pull request https://github.com/cloudfoundry-incubator/bosh-vcloud-cpi-release/pull/3


Query Builder onConflict with merge don't generate the correct sql

Describe the bug
Using Query Builder method onConflict with merge don't make the ON DUPLICATE in the resulted SQL.
Also the column/s required in onConflict don't make difference in the ON DUPLICATE, so would be nice to make optional at least.
Code
let data = [
	{
		idProvince: 44,
		name: "Province 1"
	}
	{
		idProvince: 44,
		name: "Province 2"
	}
]

return em.createQueryBuilder(Poblation).insert(data).onConflict("name").merge().getFormattedQuery();

// Ouput:
// insert into `poblations` (`id_province`, `name`) values (44, 'Province 1'), (44, 'Province 2')

To Reproduce
Steps to reproduce the behavior:

Get the Query Builder
Create an insert, add onConflict and merge
See the resulted query

Expected behavior
Output:
insert into `poblations` (`id_province`, `name`) values (44, 'Province 1'), (44, 'Province 2') on duplicate key update `id_province` = values(`id_province`), `name` = values(`name`)

Additional context
https://knexjs.org/#Builder-onConflict
https://mariadb.com/kb/en/insert-on-duplicate-key-update/
Versions

Dependency
Version

node
14.16.1

typescript
4.2.4

mikro-orm
4.5.4

mariadb
2.5.3

@mikro-orm/mariadb
4.5.4

Didn't know the parameter can be also array, that's actually different problem. It is not even allowed on the TS level, so the example you gave should result in TS error TS2559: Type 'string[]' has no properties in common with type 'EntityData '..
I guess it is working because you are passing the this inside the createQueryBuilder method, which itself looks also weird, it should be entity class, not instance. It's weird if that is working for you (that it compiles), because the entity name should be taken as this.name then, which is quite weird.

Unfortunately the merge(['name']) with array of keys won't be supported in 4.x, as we can't update knex and this was added in knex 0.95.
https://github.com/knex/knex/blob/master/CHANGELOG.md#0950---03-march-2021
knex 0.95 is breaking as it requires node and TS version bumps, it is already working in v5, so if you feel brave enough, you could give the dev version a try. keep in mind there will be more breaking changes, but so far it was mostly just about the version bumps and internals, not public api.
The fix is available as 4.5.5-dev.12, but as mentioned above, you can either use merge() without params, or with object parameter e.g. merge({ foo: 'bar' }).

In fact I don't wan't to give params to merge, so it's nice! I will try tomorrow.
Thanks for your work!!

@B4nan it works like a charm!
Would be nice to make the field param in onConflict() optional, it don't affect to the output.
em.createQueryBuilder(this).insert(data).onConflict("name").merge().getFormattedQuery();
// Following are empty or non existing fields
em.createQueryBuilder(this).insert(data).onConflict("test").merge().getFormattedQuery();
em.createQueryBuilder(this).insert(data).onConflict("").merge().getFormattedQuery();

The three lines generates, the fields param doesn't seem to affect (at least in MariaDB):
insert into `poblations` (`id_province`, `name`) values (44, 'Province 1'), (44, 'province 2') on duplicate key update `id_province` = values(`id_province`), `name` = values(`name`)

I see, I thought the parameter is mandatory in knex, but apparently it is not. Will handle that as part of #1803 so subscribe there for updates.


Add a more helpful error message when unkown options are encountered

When there is a bad option, docopt-go silently prints the usage and gives no indication of which option(s) it didn't like. It's very confusing when a program with a lot of options has a command line with one of them misspelled.
This patch complains about unknown options instead of silently printing the usage.

Coverage increased (+0.08%) to 88.344% when pulling 871d90846fc38b116fc1dff75bbe5e2b262d1f54 on caldwell:unknown-option-err into ee0de3bc6815ee19d4a46c7eb90f829db0e014b1 on docopt:master.

I just ran into this, so now my go.mod looks like this:
module github.com/username/project-name

require (
    github.com/caldwell/docopt.go v0.0.0-20180924233533-871d90846fc3
)


Show more of filename

Now that we have fewer pages a very quick fix to #200 is to just show more of the filename - @duncanMR will this do? Or do you think a larger change is needed?

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

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

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
686

Relevant Lines:
833

💛  - Coveralls

Yes that works fine for me; I just tried it out below in a half screen-width window. It does push the tabs quite far to the right, but I can't think of a way around that. I don't think trying to put the title above or below the tabs would look good.

I'd like to see the filename on the overview page too, we will have very long filenames and the most significant stuff tends to be at the end. It's just another row in the output pane, right?

Great idea - I've added the file path to the overview.


Add phone input with formatting

What:
Add phone input within one step of the widget
Why:
We need to make things easier in terms of formatting input for our users
How:

Add dependency to use react-phone-input-2

Media:

Usage interaction

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Invalid installation

I put SuperPNG.8bi in the following location
C:\Program Files\Adobe\Adobe Photoshop 2020\Plug-ins\Generator
After starting PS, I still don’t see superPNG, should this option be in the filter?

I mean should be in save as ... option,right?

I mean should be in save as ... option,right?

Copy it inside C:\Program Files\Adobe\Adobe Photoshop 2020\Plug-ins.
In Photoshop you have to use "save as copy" to see the SuperPng option.


Dimension table: microbiology items

Is there an alternative table to d_micro in v0.4? Unable to process microbiologyevents since d_micro has been removed. Thanks.
Alternatively, has d_micro been incorporated into microbiologyevents in v0.4?

What in particular are you looking for?

Fixed the issue myself. Thanks.


Merge CI suites into one

They were split to make rerunning easier, but rerunning does not work.
Merge them back to make cancelling jobs easier.

They were split to make rerunning easier, but rerunning does not work.

What do you mean it doesn't work? I used that few times.

I do like having them in one, but rerun worked for me after updating to checkout@v2.

I attempted rerun just before posting this PR. it failed at checkout.
I believe rerun works "occasionally enough" that I should not even bother trying
@electrum unfortunately we have quite a few flaky tests and environment outside is sometimes flaky too. We should eliminate, but we will not eliminate all of the flakiness... Big kudos to @kokosing for all the work he did in this space. Thank you!


Added logo to documentation website.

Solves #272.
This is how it looks:

I'll merge by the end of the day if there are no more comments.


W8004 'FrameCount'

[bcc32 Warning] Export_EbuCore.cpp(1514): W8004 'FrameCount' is assigned a value that is never used
https://github.com/MediaArea/MediaInfoLib/blob/e6b5ceb31a852e1213829c064af8ec42a9adac94/Source/MediaInfo/Export/Export_EbuCore.cpp#L1465

Looks like old code not used anymore.


Misc (Typography) not accessible from Utilities (CSS)

After the new arrangement of the Utilities (CSS) page, I guess you just missed adding a link for Misc (Typography) section.

Thanks for the heads-up. The util page has been updated. Cheers.
https://codyhouse.co/ds/docs/framework/utilities


Pick and register with an external threat detection system

If we choose to use Google Web Risk, then we will need to contact them before we can do anything. This might take some time, so we should probably get ahead with this work.
The sign up for the API (to contact sales) is: https://go.chronicle.security/webriskapi
We have to be whitelisted to use it, so I guess they may want to check we aren't pedalling malware or something.

Contacted Google to get a quote and sign up for the service. Will update when I hear back.

Working on this issue has officially solidified my disdain for Google's Admin platform. Nothing is straightforward on Google's platform. I've spent hours trying to figure out how to set this up and I'm still not sure if things are squared away.
Here's the steps I took:

Gave @jon-betts access to the Google Cloud Platform as part of the H org
Set up account info for Web Risk/Google Cloud Platform - forwarded the email I received to Jon
After the account setup, I was taken to this page, which I just cannot figure out. So maybe @jon-betts, we can sync up on this tomorrow when we're both online?

Yeah not sure about that. We can try and work through it together if you like

Seeing as this isn't going very well. We're starting a parallel track to use https://urlhaus.abuse.ch/
This would have protected us from at least some of the threats and should be easier to integrate

I'm fairly sure this is done now, but we're going to do the URLHaus stuff as a stepping stone first.


Expose all property

Description
Exposes the all property. (Allthough I would prefer calling it events.)
This makes it possible to clear all events by executing mittInstance.all.clear().
Is a (lightweight) alternative to #104.
Relates to
#70 #72 #102

Error: compressed-size-action was unable to comment on your PR.
This can happen for PR's originating from a fork without write permissions.
You can copy the size table directly into a comment using the markdown below:

Size Change: +11 B (1%)
Total Size: 941 B

Filename
Size
Change

dist/mitt.es.js
216 B
+3 B (1%)

dist/mitt.js
215 B
+2 B (0%)

dist/mitt.modern.js
215 B
+3 B (1%)

dist/mitt.umd.js
295 B
+3 B (1%)


Adding link to a customer profile in comments lists

Questions
Answers

Description?
Adding a link to the customer profile in BO - product comments lists (just like it was done recently in Orders list). This allows moderators to quickly navigate to comment author's profile - e.g. we use this feature to add royalty points to comments authors

Type?
improvement

Absolutely agree that this is an awkward way of creating a link... The problem is, I cannot find a way to display ANY link in a table generated by Helper::generateList() (apart from the standard View/Edit/Delete icons). Maybe you'll give me a hint? I'll gladly re-write my solution if I'm able to do this without re-writing the Helper class.....

Absolutely agree that this is an awkward way of creating a link... The problem is, I cannot find a way to display ANY link in a table generated by Helper::generateList() (apart from the standard View/Edit/Delete icons). Maybe you'll give me a hint? I'll gladly re-write my solution if I'm able to do this without re-writing the Helper class.....

You're able to change the template: https://github.com/PrestaShop/PrestaShop/blob/develop/classes/helper/HelperList.php#L196
Maybe you can find something here: https://github.com/PrestaShop/PrestaShop/blob/develop/classes/helper/Helper.php#L74

Hello @vallka
Please take a look at this PR:
https://github.com/PrestaShop/productcomments/pull/124
Let me know if that is a solution you wanted to have :-)

Hello @vallka
Please take a look at this PR: #124
Let me know if that is a solution you wanted to have :-)
You can see how it is done, basically, every field rendered by the HelperList may be decorated through the callback, and this was the easiest way create a link.

I like what you did and I approve :p

Exactly what I wanted to do :)
On Fri, 24 Dec 2021 at 11:24, Krystian Podemski @.***>
wrote:

Hello @vallka https://github.com/vallka
Please take a look at this PR:
#124 https://github.com/PrestaShop/productcomments/pull/124
Let me know if that is a solution you wanted to have :-)
—
Reply to this email directly, view it on GitHub
https://github.com/PrestaShop/productcomments/pull/123#issuecomment-1000800813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADBMNHO4Y2YGPIWUU4NLZYTUSRJ53ANCNFSM5KTJSGBQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID:
@.***>

Closed in favor of #124
Thank you @vallka and @kpodemski


[BUG] Pyprland shows "Client window not found" error when disabling eDP-1 monitor

Pyprland version
Which version did you use? 2.4.0
Describe the bug
I have a two monitor setup on Hyprland 0.43, my laptop monitor (eDP-1) and the HDMI monitor. If I login in my laptop, plug it to the hdmi and disable eDP-1 (so I can use only the HDMI monitor) pyprland shows error "Client window not found" and after a little while, if I try to acess the scratchpad again it crashed hyprland (only happens sometimes)
To Reproduce
Steps to reproduce the behavior:

Start Hyprland on laptop
Plug laptop to the hdmi monitor
disable laptop eDP-1 (hyprctl keyword monitor "eDP-1, disable"
) monitor so I can use only the HDMI monitor
pypr shows error "Client window (window number) not found)
If I try to use the scratchpads again it crashes Hyprland sometimes

Expected behavior
Pypr should move the other monitor
Configuration (provide following files/samples when relevant):

pyprland.toml
hyprland.conf
debug log

debug-log.txt
hyprland-config.txt
pyprland-config.txt

From the log it looks like the client window has crashed/ been closed:
Traceback (most recent call last):
  File "/var/cache/pypr/package.whl/pyprland/command.py", line 183, in _run_plugin_handler
    await getattr(plugin, full_name)(*params)
  File "/var/cache/pypr/package.whl/pyprland/plugins/scratchpads/__init__.py", line 307, in event_monitorremoved
    await self.run_hide(scratch.uid, autohide=True)
  File "/var/cache/pypr/package.whl/pyprland/plugins/scratchpads/__init__.py", line 726, in run_hide
    await scratch.update_client_info(clients=clients)
  File "/var/cache/pypr/package.whl/pyprland/plugins/scratchpads/objects.py", line 159, in update_client_info
    raise KeyError(msg)
KeyError: 'Client window 0x5f5c7cece830 not found'

Can you see the window in hyprctl clients ?

I made it a bit more robust on git version, it should keep processing other scratchpads.


freerdp can't connect to 2012r2 rdp from xubuntu 22.04 client

I can connect to 2012r2 server from xubuntu 18.04 (without /sec:rdp option) (freerdp 2.0.0)
But I can't connect to this server from xubuntu 22.04 (freerdp 2.6.1 & freerdp3 nightly):
(xfreerdp /v:ipaddress:port /cert:ignore /u:username /p:password):
[com.freerdp.core] - transport_connect_tls:freerdp_set_last_error_ex ERRCONNECT_TLS_CONNECT_FAILED [0x00020008]
(/opt/freerdp-nightly/bin/xfreerdp3 /v:ipaddress:port /cert:ignore /u:username /p:password):
[com.freerdp.crypto] - [freerdp_tls_handshake]: BIO_do_handshake failed
[com.freerdp.core] - [transport_default_connect_tls]: ERRCONNECT_TLS_CONNECT_FAILED [0x00020008]
(xfreerdp /v:ipaddress:port /cert:ignore /u:username /p:password /sec:rdp):
[com.freerdp.core.transport] - BIO_read returned a system error 104: Connection reset by peer
[com.freerdp.core] - transport_read_layer:freerdp_set_last_error_ex ERRCONNECT_CONNECT_TRANSPORT_FAILED [0x0002000D]
(/opt/freerdp-nightly/bin/xfreerdp3 /v:ipaddress:port /cert:ignore /u:username /p:password /sec:rdp):
[com.freerdp.core.transport] - BIO_read returned a system error 104: Connection reset by peer
[com.freerdp.core] - transport_read_layer:freerdp_set_last_error_ex ERRCONNECT_CONNECT_TRANSPORT_FAILED [0x0002000D]
I can successfully connect via freerdp/freerdp3 from xubuntu 22.04 to 2008r2/2016/xubuntu+xrdp, but not 2012r2...
Also I have macos (monterey) on this pc and I can successfully connect via ms rdp client for mac to this server...
Also I tried to connect this 2012r2 server from another pc with archlinux and with xubuntu 22.04 and got the same errors...
Anybody help me!

most likely https://github.com/freerdp/freerdp/wiki/FAQ#windows-7-errconnect_tls_connect_failed
e.g. the SSL ciphers supported by server 2012 are too old and blocked by default.
ping me if it is something different, closing until then.

Thanks for the answer, but it doesn't work...
xfreerdp /v:ipaddress:port /cert:ignore /u:username /p:password /tls-seclevel:1
[com.freerdp.core] - transport_connect_tls:freerdp_set_last_error_ex ERRCONNECT_TLS_CONNECT_FAILED [0x00020008]
xfreerdp /v:ipaddress:port /cert:ignore /u:username /p:password /tls-seclevel:0
[com.freerdp.core] - freerdp_tcp_connect:freerdp_set_last_error_ex ERRCONNECT_CONNECT_FAILED [0x00020006]
[com.freerdp.core] - failed to connect to ipaddress

@vovchok70 hmm, then it might be due to your openssl not having legacy ciphers activated.
that you can connect to 2008 does only mean that that installation has a better configuration.

https://services.dartmouth.edu/TDClient/1806/Portal/KB/ArticleDet?ID=136495
This helped me (with /tls-seclevel:0)


[Bug] Flattening Bulk Workflow

Using Brazil site, I came across this issue:
Executing:  56%|█████▌    | 20/36 [00:02<00:02,  6.89cell/s]
Traceback (most recent call last):
  File "/Users/cmarshak/bekaert-team/calval-RTC/bulk_validation_scripts/bulk_papermill_OPERA_RTC_flattening.py", line 275, in <module>
    main()
  File "/Users/cmarshak/bekaert-team/calval-RTC/bulk_validation_scripts/bulk_papermill_OPERA_RTC_flattening.py", line 271, in main
    flatten(input_data_dir)
  File "/Users/cmarshak/bekaert-team/calval-RTC/bulk_validation_scripts/bulk_papermill_OPERA_RTC_flattening.py", line 233, in flatten
    pm.execute_notebook(
  File "/Users/cmarshak/mambaforge/envs/opera_calval_rtc/lib/python3.11/site-packages/papermill/execute.py", line 128, in execute_notebook
    raise_for_execution_errors(nb, output_path)
  File "/Users/cmarshak/mambaforge/envs/opera_calval_rtc/lib/python3.11/site-packages/papermill/execute.py", line 232, in raise_for_execution_errors
    raise error
papermill.exceptions.PapermillExecutionError:
---------------------------------------------------------------------------
Exception encountered at "In [11]":
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
Cell In[11], line 15
     13     land_cover = list(data_dir.glob('merged_*.tif'))[0]
     14 else:
---> 15     land_cover = land_cover[0]

IndexError: list index out of range

output_OPERA_L2-RTC_S1A_IW_SLC__1SDV_20210528T092214_20210528T092242_038086_047EB2_5BE8_30_v1.0_prep_flattening_part_1.pdf
Above is the output jupyter notebook.

Looks like wget needs to be installed in the conda environment.

Thanks, @cmarshak. I added wget to both yamls in this PR (https://github.com/OPERA-Cal-Val/calval-RTC/pull/15), and requested you as a reviewer.


[Pager] Page before last hint sometimes disappears when swiping to last page

Describe the bug
In HorizontalPager, when using contentPadding, page before last hint sometimes disappears when swiping to last page.
When that happens, slightly swiping the last page will show before to last page.
To Reproduce
Steps to reproduce the behavior:

List with 3 elements (see code below)
Swipe to last page
See that the before to last page hint disappears on the left of the screen
If the issue doesn't occur, you can swipe to previous page and retry from 2.

@OptIn(ExperimentalPagerApi::class)
@Composable
private fun Sample() {
    Column(Modifier.fillMaxSize()) {
        val colorPages = remember {
            listOf(Color.Cyan, Color.Magenta, Color.Yellow)
        }
        val pagerState = rememberPagerState()

        HorizontalPager(
            state = pagerState,
            count = colorPages.size,
            modifier = Modifier.fillMaxSize(),
            contentPadding = PaddingValues(horizontal = 26.dp),
        ) { page ->
            Text(
                text = "hello",
                modifier = Modifier
                    .fillMaxSize()
                    .background(colorPages[page])
            )
        }
    }
}

Expected behavior
Page before last hint should be displayed consistently on last page.
Screenshots?

Environment:

Android OS version: Tested on Android 28
Device: Emulator
Accompanist version: 0.20.2 and 0.21.2-beta

It was a bug in LazyRow and is now fixed in the main Compose repository - https://android-review.googlesource.com/c/platform/frameworks/support/+/1900422.
Thanks for the report


WIP: Deploy to staging

[x] Configure static/media urls & paths - #40
[x] Collect statics during pixel build
[x] Add staging docker compose script
[x] Set nginx host port (reverse proxy) - should ask @biorosetics
[ ] Create bin/deploy script (#14):

[ ] Create .env file

[ ] Copy nginx configuration

[ ] Copy docker compose script

[ ] Perform database migrations

[ ] Restart docker compose

Coverage remained the same at 100.0% when pulling a305e15277f5abdc5e918ecdc5881025c4300797 on add-staging-compose into f7042a9ba0a6e6be4f5ed71584b7b1b4cb8a5679 on master.

Coverage remained the same at 100.0% when pulling 3f703d01ad7298789ad52bad8b8d6ddff95f4189 on add-staging-compose into f395cc63fc3c533b635c7ec59ed06117b929711b on master.

Coverage remained the same at 100.0% when pulling e15823929496e6720aac7ef7e08784af9e7c1e48 on add-staging-compose into 7a5e39731e369bbdeb7869b17306689a9c71dcac on master.

Coverage remained the same at 100.0% when pulling e15823929496e6720aac7ef7e08784af9e7c1e48 on add-staging-compose into 7a5e39731e369bbdeb7869b17306689a9c71dcac on master.


No typechecking during running tests

Issue

No typechecking during running tests

Expected behavior

Expect to get typechecking errors inside tests.

Link to a minimal repo that reproduces this issue
Repo is creating based on create-react-app with typescript

In component there is a not valid assignment,however tests are passing successfully.

@ValeryLosik there is something wrong with your development environment if you are not getting an error with that. Also tests cannot run type checks since there are no types in the compiled code. TypeScript type checks are run on compile time. Also if you have an IDE that supports TypeScript you will get error messages while you are typing.
Correct me if I am wrong, ts-jest as far as I can tell simply is a connector for jest and TypeScript so you can test your compiled code more easily. I don't believe any testing framework can check the types from the compiled code.
Also I don't see  jest json config.

ts-jest is meant to handle only the 'testing with jest' part and nothing else. Type errors are typically handle by the IDE and/or the build/live reload process.
I haven't tried this but it might be possible to do what you want by setting the noEmitOnError flag.
Anyway, closing as this falls outside the scope of this project


Fix Tune PG import

Btw @krfricke do you think we should just remove resources dict support (pre Ray 1.3), and only do PlacementGroupFactory?

Let's remove it after 1.4 release


feat(styles): add new sizing variables

This PR needs to be on hold, until the next major release!

Preview environments are getting posted here, as soon as they are ready!

We close this until we are fully ready to implement the new sizes


print cube roots and square roots within a range (max 100000)

Enter calculator name -
print cube roots and square roots within a range (max 100000)
Describe calculator functionality -
print cube roots and square roots within a range (max 100000)
Under which category you wish to place it -

[ ] Computer Science
[x ] Mathematics
[ ] Physics
[ ] Chemistry
[ ] Biology
[ ] Finance
[ ] Scientific
[ ] Health
[ ] Sports
[ ] Let project owner decide
[ ] Suggest a new category - (Answer here)

kindly assign this issue to me

Hey @munfaridzahera,
Assigned.
You may contribute your code under the folder path -
calc_for_everything\calculators\src\mathematical\cubeRootAndSquareRootCalc
Issue completion deadline - 12th Oct 8:00:00 am IST
NOTE - In case the issue completion deadline is not met, the issue will be assigned to a new participant


Make dependency on serde on generated code optional / document which features are needed

Every dependency counts, and for users that do not intend to serialize the structs to serde, the dependency may be an extra burden.
Some ideas:

consider making the dependency optional in the generation step
consider document that serde is required, and document which features of serde are required

These may justify a small guide.

I think this should definitely be optional, however it's blocked by #31.

I've opened #31 and #43 to cover the two fixes we want to get done regarding this.


Eliminate internal usage that mutates feature/geometry

Basically: run tests with python -Werror:... to find instances and fix them.

Resolved by #1116.


api docs: Add explicit -G for curl requests.

Some versions of curl will do a POST if you supply -d but don't supply -G.
Also, several endpoints in Zulip have the same name for two different endpoints.  For example, "/messages" can POST messages or GET messages.
Newbies testing the API may think they're doing a GET when actually a POST occurs, and the error messages can be confusing.
We just need to sweep our docs to explicitly add -G to the curl examples for GET endpoints.
Discussion here: https://chat.zulip.org/#narrow/stream/127-integrations/topic/Curl.20bot.20help/near/732532

@zulipbot claim


texlab: 5.10.0 -> 5.10.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for texlab is: An implementation of the Language Server Protocol for LaTeX
meta.homepage for texlab is: https://github.com/latex-lsp/texlab
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 5.10.1 with grep in /nix/store/qikm6anz32glqqr6icjsh75axlf8h12s-texlab-5.10.1
found 5.10.1 in filename of file in /nix/store/qikm6anz32glqqr6icjsh75axlf8h12s-texlab-5.10.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
texlab

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/qikm6anz32glqqr6icjsh75axlf8h12s-texlab-5.10.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A texlab https://github.com/r-ryantm/nixpkgs/archive/984733dd948d2085c739133905d18eaf5edf71b8.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/984733dd948d2085c739133905d18eaf5edf71b8#texlab

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

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

texlab

Maintainer pings
cc @doronbehar @kira-bruneau for testing.

Result of nixpkgs-review pr 262417 run on x86_64-linux 1

2 packages built:

texlab
texlab.man


Error: 5000: Application needs OAuth consent from the user

I had Google Fit working previously, but all of a sudden I'm getting this error. I didn't use it for a week or two, but now when I try to get exercises it returns this error instead:
Error: 5000: Application needs OAuth consent from the user
All GoogleFit.isEnabled, GoogleFit.isAuthorized and GoogleFit.isAvailable are returning true, just before this error is thrown. However it seems it's not authorized somehow?
I do get the pop up dialog to select which google account I want to use. However it does not show the dialog which asks for access to Exercise data. It did show it before, but not anymore somehow.
In GCP I have a project with an Android OAuth Client, which has the scope to read exercise data.
Any idea what I could be doing wrong?
How I'm using the library currently:
async function isGoogleFitAuthorized(): Promise<boolean> {
  try {
    await GoogleFit.checkIsAuthorized();
    return GoogleFit.isAuthorized;
  } catch (error) {
    Monitoring.error(
      error,
      "[Google Fit] Error trying to authorize Google Fit"
    );
  }

  return false;
}

const isGoogleFitEnabled = (): Promise<boolean> =>
  new Promise(res => {
    GoogleFit.isEnabled((err, isEnabled) => {
      if (err) {
        res(false);
      } else if (isEnabled) {
        res(true);
      } else {
        res(false);
      }
    });
  });

const isGoogleFitAvailable = (): Promise<boolean> =>
  new Promise(res => {
    GoogleFit.isAvailable((err, isAvailable) => {
      if (err) {
        res(false);
      } else if (isAvailable) {
        res(true);
      } else {
        res(false);
      }
    });
  });

type GoogleFitWorkoutSessionOptions = Parameters<
  typeof GoogleFit.getWorkoutSession
>[0];

type ExerciseAndroid = {
  id: string;
  from: number;
  to: number;
  activityId: number;
  activityName: string;
  sourceName: string;
  sourceId: string;
  calories: number;
  distance?: number;
};

export const fetchExerciseDataAndroid =
  (defaultFetchScope: number) =>
  async (
    from: number = Date.now() - defaultFetchScope,
    to: number = Date.now(),
    limit: number = 30
  ): Promise<ExerciseAndroid[]> => {
    await initGoogleFit();

    await GoogleFit.checkIsAuthorized();

    const authOptions = {
      scopes: [Scopes.FITNESS_ACTIVITY_READ],
    };

    const authResponse = await GoogleFit.authorize(authOptions);

    console.log(authResponse); // {"success": true}

    if (
      !(await isGoogleFitEnabled()) ||
      !(await isGoogleFitAvailable()) ||
      !(await isGoogleFitAuthorized()) ||
      from >= to
    ) {
      return [];
    }

    const options: GoogleFitWorkoutSessionOptions = {
      startDate: new Date(from).toISOString(),
      endDate: new Date(to).toISOString(),
      readSessionFromAllApps: true,
    };

    const workouts = await GoogleFit.getWorkoutSession(options);
    console.log("workouts", workouts); // Error: 5000: Application needs OAuth consent from the user
  };

Seemed this was due to location permission. I added:
   const options = {
        scopes: [Scopes.FITNESS_ACTIVITY_READ, Scopes.FITNESS_LOCATION_READ],
      };

And to AndroidManifest.xml:
    <uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />
And the error went away.
It was confusing because it worked at first without those. Not sure at all what happened there.

Pretty confusing error message for not having the correct scopes or permissions in manifest


Support multipart Upload

Multipart send form with files

@Yamashou can you approve this PR? I think it may be useful feature

Thanks, notify this PR. I had not noticed.

Please tell me about the need for this response. It concerns my motivation.

need support of Uploading files as https://gqlgen.com/reference/file-upload/
curl localhost:4000/graphql
-F operations='{ "query": "mutation ($file: Upload!) { singleUpload(file: $file) }", "variables": { "file": null } }'
-F map='{ "0": ["variables.file"] }'
-F<EMAIL_ADDRESS>

Looks mostly good to me! Thanks!


Notification Sound Effect / Taskbar Icon Issue

Aknowledgements

[X] I have checked that there's no other issue describing the same or
similar problem that I currently have, regardless if it has been
closed or open.

[X] I can confirm that this is not an issue with the Discord website,
but it is a problem specific to the WebCord itself.

[ ] I have tried running the build from the master branch and it does
not have any fixes implemented according to my issue.

[ ] My issue describes one of the unstable and/or not fully implemented
features.

[ ] I have found a workaround to mitigate or temporarily fix this issue
(please write it in Additional context section).

Operating System / Platform
🪟 Windows
Operating system architecture
x64 (64-bit Intel/AMD)
Electron version
v19.0.5
Application version
v3.3.0
Bug description
The first issue is that the notification sound effect will play, but only if WebCord is the active window. So if the program is minimized to the tray or I am using any other program instead of WebCord, I wont get the notification sound. It appears to be related to a change made in v3.3.0, as if I go back to v3.2.0 the issue no longer happens.
The second issue has to do with how WebCord appears on the taskbar. If WebCord is pinned to the taskbar it works fine, but only for a little while. If I happen to close the WebCord window and open it again from the system tray it will launch what seems to be a different instance of WebCord that isn't pinned. If this happens, I have to fully close WebCord and relaunch it again and it will show up on the taskbar properly again. However, occasionally after a restart WebCord will show up as a separate icon again, but this time the only way to fix it is to remove the previously pinned icon and pin the new one. Changing versions doesn't fix this. I speculate it may have to do with the fact that WebCord runs multiple processes when open.
Additional context
Downgrading to v3.2.0 works to fix the Notification Issue
Below is a screen capture of the duplicate icon.

Please do not describe two separate issues in one ticket! This will cause this issue to be closed under the reason of invalid. This is also a duplicate of #161.

Other than this, thank you for actually expanding some details about #161, this actually described the case I didn't tested. Please however comment on existing issue instead than opening a new ones.

Apologies for that, I wasn't sure if the difference of it playing when active qualified it as different enough. I still did have the issue with the taskbar icon though, should I open that on its own?

Apologies for that, I wasn't sure if the difference of it playing when active qualified it as different enough. I still did have the issue with the taskbar icon though, should I open that on its own?

I will definitely work on it, I have some ideas why it might work like this. And yes, you should definitely open it as its own, this make it easier for me to manage issue states – I can't "partially close" issues, that's why there should be one bug description per issue, unless they seem to have a common cause (then you can open it under the same ticket, but it is still preferable to open two issues if you're unsure).

Gotcha, sorry to complicate things. Not super experienced with GitHub and issues and stuff, I'll make sure to do it correctly going forward


Revert "Use ESLint 9 to lint libs/**"

Reverts guardian/csnx#1668

accidentally left automerge on when updating the base


Early ivar assign

This is an alternate strategy discussed in #95 in which we decide what the index of an invention variable is on the spot instead of leaving that as a secondary inner search.
This PR seems to have the same results and about the same runtime (maybe even faster) compared to the original across a variety of domains including tower building ones.
This change is nice because it removed the most complicated part of the codebase (assignments_of_pattern) and also means that all the pruning happens in a single place rather than being replicated in many places

In the long term It's not clear whether this or the inner search strategy is better, however I think that this switch is merited. The only code is still there if we really want it in git history. You could explore the viable_prefixes approach but things would just remain a lot less clean.
Also I suspect since we added the force_multiuse checks that might give most of the gain that not splitting early was giving before.
At higher arities the new method vastly outperforms the old one (clearly because of force_multiuse, which fires a lot more as arities get higher as observed in the stats)
Overall I would say the factored assignment approach is not worthwhile unless you can come up with some clear benefit


Check compatibility comment block with asterisk

When the universal selector is used right after a comment block and the CSS is built, the browser parses this incorrectly. Reproducible by including https://github.com/twbs/bootstrap/blob/master/scss/bootstrap-reboot.scss.

Issue not reproducible, probably related to Hubspot.


Error in VM nameresolution on delete cloned-vm

There is a error on resolving the name of the cloned vm to get deleted.
If you name a vm "Test" and the second "Test2" the nameresolution might fail if you try to backup "Test".
This is because the "" on line 84 in vmtools.py:
vm_search_regexp = ("name=%s%s" % (vm_name, config.get_vm_middle()))
and result in the following log:
2020-01-26 00:06:15,490: Start backup for: Test
2020-01-26 00:06:15,541: Delete cloned VM (Test2) started ...
2020-01-26 00:06:16,956: !!! Can't delete cloned VM (Test2)
I've edited the line to:
vm_search_regexp = ("name=%s%s__*" % (vm_name, config.get_vm_middle()))
and now running a test.

I’m closing this issue because it has been inactive for a few months. This probably means that it is not reproducible or it has been fixed in a newer version.

I think this issue should not be closed blindly.
AFAICS the code part in question has not been changed and the issue raised seems reasonable to me.

Thanks for the quick response, I reopen the issue.
Have you tested the issue with the latest release? And please tell us more about your environment.

I am first going to test the api4 changes against our oVirt 4.3 installation anyway.
So I can include this corner case as well.
I will let you know of the results here.
Thanks for still caring for this project!

i've tested my changes from my initial post now for almost one year and it fixed the behavior and working without any issues on oVirt 4.3.

@MEy3 Thanks for the response
@stolp If you still have problems please open a new issue

@MEy3 Thanks for the response
@stolp If you still have problems please open a new issue

So you added my changes to your code? The Issue is in your lastest public version of this project still present until you changed your code like i did and written in the initial post of this issue.

So you added my changes to your code? The Issue is in your lastest public version of this project still present until you changed your code like i did and written in the initial post of this issue.

@MEy3 I created a pull request with your changes.
Sadly I was not able to test this, as I wasn't able to run current master against our oVirt 4.3 installation.

@MEy3 I created a pull request with your changes.
Sadly I was not able to test this, as I wasn't able to run current master against our oVirt 4.3 installation.

I created a new branch to test against my oVirt 4.3 environment. There I changed the line. I'm not sure if double "_" solves the problem. Please test the branch with your version.

I created a new branch to test against my oVirt 4.3 environment. There I changed the line. I'm not sure if double "_" solves the problem. Please test the branch with your version.

I will test this, thanks!

I will test this, thanks!

The „“ is the separator between VMname and Date. Without “” the script fails to get the proper name of the Test1 vm snapshot and deletes the last one that can be found from Test2 which is wrong. Also it happend that not only the wrong snapshot get’s deleted also so entire VM Test2 incl. Discimage image. That happens twice so I had to restore it from an older backup and modified the script last year. “__*” takes care to get the exact name of the vm and ensure due to the separator to only delete the correct snapshot.

The „“ is the separator between VMname and Date. Without “” the script fails to get the proper name of the Test1 vm snapshot and deletes the last one that can be found from Test2 which is wrong. Also it happend that not only the wrong snapshot get’s deleted also so entire VM Test2 incl. Discimage image. That happens twice so I had to restore it from an older backup and modified the script last year. “__*” takes care to get the exact name of the vm and ensure due to the separator to only delete the correct snapshot.

It is not possible with any release that the whole VM is delete only if you have nothing set to "vm_middle" in your config. But if you wish I add this to the master and oVirt4.3 branch

It is not possible with any release that the whole VM is delete only if you have nothing set to "vm_middle" in your config. But if you wish I add this to the master and oVirt4.3 branch

If you still unsatisfied, leave a comment.

If you still unsatisfied, leave a comment.


Opencast crashes on startup when database is not running

Description:
When starting Opencast, the application crashes if the database service is not running. Ideally, Opencast should handle the absence of the database gracefully, possibly by retrying the connection or providing a meaningful error message without crashing. See similar issue with elastic search here: https://github.com/opencast/opencast/issues/5943 This issue was reported following a power failure at an institution.
Steps to Reproduce:

Ensure the database service required by Opencast is stopped or not running.
Attempt to start the Opencast application.
Observe that Opencast crashes immediately after startup.

Expected Behavior:
Opencast should not crash if the database service is not available. Instead, it should:

Log a meaningful error message indicating that the database connection could not be established.
Retry the connection to the database after a certain interval.
(Optional?) Allow for a configuration option to set the retry interval and the number of retry attempts.

Additional Information:
This issue affects the robustness and reliability of the Opencast application. Implementing a more fault-tolerant startup process would improve the user experience and operational stability 🚀

Maybe we have a different definition of a crash, but for me that implies that the software stops running.
But Opencast activates the db module during start which then tries to establish a connection. If no connection can be established, the module's activation fails and db continues to be inactive which means that all other modules dependent on this will wait for the startup.
The error is also clearly logged:
2024-06-25 13:22:23,283 | INFO  | (Activator:154) - Testing connectivity to database at jdbc:postgresql://<IP_ADDRESS>/opencast
2024-06-25 13:22:53,384 | WARN  | (Slf4jMLog$Slf4jMLogger$WarnLogger:223) - com.mchange.v2.resourcepool.BasicResourcePool$ScatteredAcquireTask@42fdc166 -- Acquisition Attempt Failed!!! Clearing pending acquires. While trying to acquire a
needed new resource, we failed to succeed more than the maximum number of allowed acquisition attempts (30). Last acquisition attempt exception:
org.postgresql.util.PSQLException: Connection to <IP_ADDRESS>:5432 refused. Check that the hostname and port are correct and that the postmaster is accepting TCP/IP connections.

The error message states that what you want is also already happening: Opencast keeps trying to establish a connection. It also only fails after the (in this case) 30 attempts all fail which isn't really “failing immediately”.
The question for me would be if we want to adjust the connection settings to try indefinitely or have Opencast retry this indefinitely during the db activation so we can log the errors. The alternative, just waiting a bit linger and then fail I wouldn't do.

Same remark as in https://github.com/opencast/opencast/pull/5944#issuecomment-2180833468. Personally, I like applications to crash quickly on startup when dependencies are not fulfilled. Again, it's tricky with Opencast, so implementing it within Opencast is fine by me. However, if the DB comes up later, Opencast should be able to start up automatically without user intervention. I would therefore suggest to try indefinitely.


RFM - Spek

https://chocolatey.org/packages/spek needs new maintainers.

Can I take this one?
What should I do to be the maintainer?
My choco id is also chantisnake

package is ready: https://github.com/chantisnake/chocolateyPackage/tree/master/spek-choco

Added as maintainer pending email confirmation.

Once the package is approved, feel free to comment here so this can be closed.

package accepted


No module dotenv

Hi, I encounted new problem. When I used 'tfoda' to detect object of interest, an error was noted as follows. How can I do next to solve, and I cannot determine whether my pretrained model downloaded from 'https://github.com/opencv/opencv/wiki/TensorFlow-Object-Detection-API#use-existing-config-file-for-your-model', is wrong. I don't find weight and config file, could you please help me to continue?
Thanks in advance.

Your screenshot shows that python-dotenv isn't installed. Run pip install -r requirements.txt.

Thanks. I have done!


Update seeds with real trns

Context

Changes proposed in this pull request

Guidance to review

Link to Trello card

Things to check

[ ] If the code removes any existing feature flags, a data migration has also been added to delete the entry from the database
[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] If this code adds a column to the DB, decide whether it needs to be in analytics yml file or analytics blocklist
[ ] API release notes have been updated if necessary
[ ] If it adds a significant user-facing change, is it documented in the CHANGELOG?
[ ] Required environment variables have been updated or added to the Azure KeyVault

Screenshots

 Use TRNs in review seeds that will not throw errors to Sentry


Trackers Session : user_id is null

Hello,
I am under Laravel 6 and since the last update of the Package, in "tracker_sessions", the value of "user_id" is always null.
Would it be possible to correct this bug?
Thank you in advance.

@antonioribeiro  ^^

Hi.
Vendor/Laravel/ServiceProvider.php
` public function boot()
{
parent::boot();
    if (!$this->getConfig('enabled')) {
        return false;
    }

    $this->loadRoutes();

    $this->registerErrorHandler();

    // $this->bootTracker();

    $this->loadTranslations();
}`

You can just comment this line out ($this->bootTracker();) in the package Service Provider boot method.

https://github.com/antonioribeiro/tracker/issues/479

I edited the package directly in the FTP, then commented the line as you suggest. But it still doesn't work. Are there a few more things to do?

@kazemimorteza68 @antonioribeiro
Can you help me ?

I have the same error! Configured everything properly but still doesn't work.
Using Laravel 6.18

I have a problem when the user login at the website, and doesn't save user-id at the table tracker_sessions. it is always null.
please help me and give me a solution.

My solution was in the Login controller:
It's not cute but it works hahaha.
$visitor = \Tracker::currentSession();
if($visitor) {
$visitor->user_id= Auth::user()->id;
$visitor->save();
}

I figured this out after following through the code. There are two crucial steps that must be followed .

ensure that the correct auth guard is set in config/auth.php this usually is set to web and the provider model is App\User::class in fresh installations . If your application is using a different auth gaurd and path for model it may work fine but missing user_id on saved data in this package may be sign of a miss-configured auth guard
With this done now you need to set the guard in (1 ) in config/tracker.php on the authentication_guards key for example

'authentication_guards' => ['web']
If this is done the package should now pick up the user_id

I made a PR to solve this issue
https://github.com/antonioribeiro/tracker/pull/513

calazcano commented on 9 Apr 2020 -> Excellent!! It's working fine.

calazcano commented on 9 Apr 2020 -> Excellent!! It's working fine.

$visitor = \Tracker::currentSession();

it works, Thanks


1.18 API Compatibility

This library has a lot of inconsistencies and mismatches with the Docker Remote API. The most common issue is when fields are added to the Docker API but not added to the objects in this library; consequently, these new API fields are not exposed to users of this library.
This PR is part one of a series. I (and whoever else wants to help) will review in turn each version of the Remote API docs, and verify that anything added to the Remote API is added to this library. I will try to add tests as well, where feasible.
This PR covers Docker version 1.6, API version 1.18.

Thanks @johnflavin. Could we get tests for this too?

I haven't had much time lately, so I haven't yet finished my survey of the API 1.18 docs. But once I can come back to this, I'll add some new tests and update existing ones as appropriate.

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

Merging [#427][cc-pull] into [master][cc-base-branch] will increase coverage by 10.22%

@@             master       #427   diff @@
==========================================
  Files            80         80
  Lines          2972       2893    -79
  Methods           0          0
  Messages          0          0
  Branches        417        392    -25
==========================================
+ Hits           1532       1787   +255
+ Misses         1305       1106   -199
+ Partials        135          0   -135

Powered by Codecov. Last updated by [8793ab5...b2f3f46][cc-compare]
[cc-base-branch]: https://codecov.io/gh/spotify/docker-client/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/spotify/docker-client/compare/8793ab592b8fba025ff3429dad2ff85fb1101109...b2f3f46727efc2508adbd72e1191fd31d19e0505
[cc-pull]: https://codecov.io/gh/spotify/docker-client/pull/427?src=pr

hey spotify folks - are you planning to merge this PR? the pid namespace specifically would be really helpful for my use case. tvm.

I'm not sure this PR is entirely done yet.
My aim was to exhaustively review the 1.18 API docs and update the library until the two were in sync. But, man, that's hard. The PR has taken a long time, which has given the API docs time to change (way more than I would have expected for a version that has been out for a pretty long time), and has given me time to forget what I have and haven't done yet. Also, I've changed a lot of stuff, and have not written a whole lot of test code for all the changes.
On the other hand, it may be totally done and ready. I doubt it, but maybe. It would certainly benefit from review by someone who is not me, whether that's @mattnworb or @davidxia, or even yourself, @sayarg.

When you get some time, @davidxia and @mattnworb, this PR is ready for review.

@johnflavin thanks so much for all of the effort on this. At first glance this looks good to me 👍
@davidxia any thoughts?

Looks like there are some merge conflicts in the src/main/java/com/spotify/docker/client/messages/HostConfig.java class

I rebased on master and fixed the merge conflicts.

👍
@davidxia do you want to review too?

I was going to, but if you're confident enough I can use the time for other
PRs in our FOSS repos.
On Tue, Aug 9, 2016 at 10:42 AM, Matt Brown<EMAIL_ADDRESS>wrote:

👍
@davidxia https://github.com/davidxia do you want to review too?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spotify/docker-client/pull/427#issuecomment-238575355,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdVberFsgTEqn2d38T4RaWAuvqSYsM1ks5qeJHtgaJpZM4IWVLp
.

--
David Xia
Software Engineer
<EMAIL_ADDRESS>

thanks again @johnflavin

Yes, thanks for the contribution, @johnflavin.
On Tue, Aug 9, 2016 at 10:47 AM, Matt Brown<EMAIL_ADDRESS>wrote:

thanks again @johnflavin https://github.com/johnflavin
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spotify/docker-client/pull/427#issuecomment-238576763,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdVbVmSrwqALW66KaMn949FkZMjLeTWks5qeJLqgaJpZM4IWVLp
.

--
David Xia
Software Engineer
<EMAIL_ADDRESS>


jorani / ics feed support

Hi,
I'm using vdirsyncer combined with khal for a while and I love them both.
My company has swith to https://github.com/bbalet/jorani to manage days off. The only feed available in this sofware is a ics url. Do you think there is a way to use that with vdirsyncer ?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/26452443-jorani-ics-feed-support?utm_campaign=plugin&utm_content=tracker%2F574345&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F574345&utm_medium=issues&utm_source=github).

Yes, use the http storage type.
On 4 September 2015 18:26:38 CEST, Gael Pasgrimaud<EMAIL_ADDRESS>wrote:

Hi,
I'm using vdirsyncer combined with khal for a while and I love them
both.
My company has swith to https://github.com/bbalet/jorani to manage days
off. The only feed available in this sofware is a ics url. Do you think
there is a way to use that with vdirsyncer ?

Reply to this email directly or view it on GitHub:
https://github.com/untitaker/vdirsyncer/issues/263

--
Sent from my phone. Please excuse my brevity.

Just works. Thanks a lot.


Translations do not end up in cache and used as keys

It's easier to show on example

repo with reproduction: https://github.com/shanginn/spiral-translator-test

Testing on ru encoding and the file app/locale/ru/messages.ru.php.
There are 3 translations:
'string in template' => 'строка в шаблоне',
'string using l' => 'строка используя l',
'string using say' => 'строка через trait и say',

The 'string in template' is used in the template home.dark.php and during the translation compilation, it gets cached, all good.
The other 2 are used in app/src/Domain/User/Exception/WithLTestException.php and app/src/Domain/User/Exception/WithSayTestException.php respectively.
And during the translation compilation using:
php app.php i18n:reset
php app.php i18n:index ru

they do not get cached but appear "as is", i.e., as keys:
// runtime/cache/i18n.ru.php
'string in template' => 'строка в шаблоне',
'string using l' => 'string using l',
'string using say' => 'string using say',

PS
The application skeleton is created with the file app/locale/ru/messages.en.php which I renamed to app/locale/ru/messages.ru.php, but even with the old filename, the result is the same.
Additional Info

Q
A

Framework Version
3.11.1

PHP version
8.3

Operating system
MacOS

hi
looks like it works fine on dev-master version 👍

@shanginn Thanks!
Closed by: #1062


Hassles experienced installing to existing Vite project

First of all, I just want to say I love the work you are all doing. I just want to chime in with my experience trying it out, I hope it can be a smoother experience in the future.
Followed this guide to try to add Solid-UI to my existing project, and got into many issues:

It doesn't state properly that the Edit tsconfig.json file step is optional. I have a vanilla js project
Editing vite.config.ts (well in my case vite.config.js not ts), I got error that __dirname does not exist (used chat-gpt to get it resolved)
After running the CLI, i got error that tsconfig.json does not exist, even if i chose no to Typescript
It deleted all the existing css in my app.css file and file was empty. Probably should also be more clear that it is the tailwinds global css that is targeted, and not other "global" css files one might have from the project.
It deleted all the existing config in my tailwind.config.js file and file was empty
After i added a button, i got the error missing class-variance-authority. Went to npm and installed it manually

After all this, I recovered some of the deleted stuff and decided to go to the manual section, and after trying for a while I finally got it working with a little luck.

Thanks for your feedback! :)
The JS integration was never completely finished, but I'm currently in the process of reworking the CLI which includes finally fixing all the JS related issues.


Fix internal links not clicking

The internal links in card don't work

can I work on it ?

@binismail Yes sure. Let me clarify the issue more

@subeshb1 do let me know if you understand the issue or not. I will assign you once you confirm

@subeshb1 yeah I do understand.

Hi @binismail any update on this?

Hi, @subeshb1 am working on it, although I ran into some error because I can't find the exact code written I don't know if you can guide me more.

No worries @binismail. The files is src/components/DeveloperCard/index.jsx The developer card needs to be modified. Do let me know if you have any problem

okay, thanks

I will work on it as soon as am back from work today.

Sure @binismail. Take your time

Fixed!


Feature request - rclone serve ftp

I would like to be able to access an rclone remote as though it were a directory on an FTP server.  Basically, rclone would be the ftp server, and remotes are subdirectories under the root.  I could then use the client of my choice (filezilla) to do things like move files via drag-n-drop and rename files/folders.  Of course, I shouldn't be able to delete the remotes, but everything else should work as though it were a normal file or folder on an ftp server.  Any reason this would not be feasible?

Tried using webDAV for this? Need a different client like filezilla pro, but windows and other operating systems can mount webDAV as a drive without extra software.

I can try something if that is ok? I find serve command to be very usefull to use rclone inside container without extra permissions to serve as a proxy to external hosting and ftp protocol should be a good addition to it.
I would suggest using https://github.com/fclairamb/ftpserver or https://github.com/goftp/server.
To compare ftp features that they offer :

https://github.com/fclairamb/ftpserver/blob/d2ce42e2772d016e168a22d4b024ba4fc08a564c/server/server.go#L31
https://github.com/goftp/server/blob/3bb68cbcc4fd8e6fb51088155a304782c62f477c/cmd.go#L28

FTP Commands: https://www.iana.org/assignments/ftp-commands-extensions/ftp-commands-extensions.xhtml
Both use standard library (except fclairamb/ftpserver that use go-kit for logging)


Eslint fixes in app

@chriddyp the whole code in app mainly throws no-unused-vars and [react/prop-types] errors. I think it'd be a good idea to let you review a few commits before continuing. Let me know if you prefer less granular commits.

@rmoestl , this looks good to me, thank you!
we can ignore the travis tests for now.
💃


Edit rezconfig docstrings for package_filter to reflect python form rather than YAML

closes #1372

Cheers! I'm not going to bother with releases for these doc-only changes - not until we actually have versioned docs anyway. I'll merge this right in to master.