Video not playing from url & minimum api level 26

Hi, after implementing your library android studio tells me to increase the minimum api level of 26 because it is required for invoke only methods.
After increasing the minimum api level i can't play a video from url, because it will just keep loading.
Code used: JzvdStd jzvdStd = (JzvdStd) findViewById(R.id.videoplayer);
jzvdStd.setUp("http://jzvd.nathen.cn/c6e3dc12a1154626b3476d9bf3bd7266/6b56c5f0dc31428083757a45764763b0-5287d2089db37e62345123a1be272f8b.mp4"
, "饺子闭眼睛" , Jzvd.SCREEN_WINDOW_NORMAL);
Tested on google pixel 3 xl, api 28, using both androidx and normal libraries

keep loading  , what log

I am sorry, but I have not understood what you are asking me

what log when it is loading


Launch Discourse Forum

Based on discussion in #152 , we are in the process of launching a new Discourse forum in collaboration with OpenAstronomy.

This is done. Thanks @Cadair !


"xapth" Typo Fix

Coverage remained the same at 100.0% when pulling d2ca97a0eed904aebc6991ad532b8163c9f8e264 on xpath_typo_fix into 5cdc0120bba6abec9a8fd971fba349f908780441 on master.


⬆🪝 update pre-commit hooks

updates:

github.com/astral-sh/ruff-pre-commit: v0.1.1 → v0.1.3
github.com/psf/black-pre-commit-mirror: 23.10.0 → 23.10.1

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.
List any dependencies that are required for this change.
Fixes # (issue)
Checklist:

[ ] The pull request only contains commits that are related to it.
[ ] I have added appropriate tests and documentation.
[ ] I have made sure that all CI jobs on GitHub pass.
[ ] The pull request introduces no new warnings and follows the project's style guidelines.

While you are at it, you could just adapt https://github.com/cda-tum/mqt-qcec/pull/324 to QECC. Should be straight forward and makes this PR obsolete.

made obsolete by #181


Small fixes & Level Descriptions on level start

Every level now has a descriptions.
Fixed some wrong tint colors.
Random Item Drop now drops Water instead of Mushrooms.
Also fixed random Health Potion Weights
JavaDocs

Perfekt. Danke für dein kurzes Review 😀


optimize: compatible with integration.grpc API

[ ] I have registered the PR changes.

Ⅰ. Describe what this PR did
compatible with integration.grpc API
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Comparison is base (486f582) 51.94% compared to head (105b64d) 51.97%.

Additional details and impacted files

@@             Coverage Diff              @@
##                2.x    #6328      +/-   ##
============================================
+ Coverage     51.94%   51.97%   +0.02%
+ Complexity     5186     5185       -1
============================================
  Files           921      921
  Lines         32166    32166
  Branches       3874     3874
============================================
+ Hits          16709    16717       +8
+ Misses        13821    13812       -9
- Partials       1636     1637       +1

see 6 files with indirect coverage changes


Add enabled/disabled to user

Do we need a ticket for this, but for group?

Depends on #358

#581 implements


Add Neoscroll.nvim

Neoscroll is a very customizable smooth scrolling plugin written in lua. Could you please add it to the website?
By the way, thank you for setting up the website. Looks awesome! :smiley:

Thanks man! I hope it will be used :)

as soon as you remove the lua tag I can merge this one

oops! forgot remove it. I'll do it in a bit.


Error while uploading file

Error while uploading file
Good morning!
We are using Dspace 6.3, xmlui. After including 60 productions, it is showing an error, it does not accept bitstream upload. Do not participate in making any further submissions due to the error.
thanks!
Related work
Link to any related tickets or PRs here.

Please use one of our recommend Support options to get support for errors you are seeing in DSpace.  https://wiki.lyrasis.org/display/DSPACE/Support
You also should use our Troubleshooting guide to find the underlying error.  DSpace never shows the full error message in the user interface (as that can provide hints on possible vulnerabilities to hackers).  You must locate the error message in the DSpace logs to determine the cause of the error. https://wiki.lyrasis.org/display/DSPACE/Troubleshoot+an+error
Closing, as this is a support question & should be asked on support mailing lists or similar.


add RELEASE in loglevel for release version log

add RELEASE in LogLevel to log warning and error  in release version.

LogLevel is removed from the project. New approach is to use isLoggable method for each adapter


rdb save may sometimes cause downtime

Hello, during the process of using tairHash, when saving the RDB, I found that an error occurred. Through the debug source code, I found that some functions do not exist in the header file of the redis module.
In the redis official website code, I found that you are in Redis submitted No. 8999 on June 16. If a function lower than this version is used, the redis instance will be down.
ps: I developed a simple client in python language, which is compatible with tairHash commands. If you have time, please help me and take a look.   https://github.com/631086083/tairClient

Hello, during the process of using tairHash, when saving the RDB, I found that an error occurred. Through the debug source code, I found that some functions do not exist in the header file of the redis module.
In the redis official website code, I found that you are in Redis submitted No. 8999 on June 16. If a function lower than this version is used, the redis instance will be down.

yes, we have a notice here.

ps: I developed a simple client in python language, which is compatible with tairHash commands. If you have time, please help me and take a look. https://github.com/631086083/tairClient

Thank you for your contribution!
Before adding it to the TairHash README list, I have some suggestions:

add some tests
the result of api can be map or list etc.. instead of the result of execute_command, you can refer:  alibabacloud-tairjedis-sdk

when saving the RD

Thank you for your interest in tairhash. Currently, tairhash implements a more efficient active expire at the cost of the need for new module api support, which makes tairhash unable to be used on lower versions of redis. We also recognize this and are thinking about whether we can solve this problem by some means, such as adding a macro compilation option to make tairhash run in some lower versions of redis. Of course, this will definitely discount some features. At the same time, if you have any ideas, welcome to communicate with us or submit PR.
Finally, thank you for your python client for tairhash, it is an amazing project:）.

sorry， it‘s my problem, my English is relatively bad, I translated that sentence into a version that needs to use redis4 or higher...
——————
Thank you very much for your suggestions, this python client does have a lot of improvements at present:

some test cases will be added to the tests in the future (including the overall module test, and the unit test of the command);
in the callback value part, by adding the callback function to analyze the return value to deal with different return methods
add more module command support and documentation

Hello, during the process of using tairHash, when saving the RDB, I found that an error occurred. Through the debug source code, I found that some functions do not exist in the header file of the redis module.
In the redis official website code, I found that you are in Redis submitted No. 8999 on June 16. If a function lower than this version is used, the redis instance will be down.
ps: I developed a simple client in python language, which is compatible with tairHash commands. If you have time, please help me and take a look. https://github.com/631086083/tairClient

Hi guys, we have recently merged a new PR, which allows TairHash to run in low version redis ( redis 5.x and redis 6.x), which may be of interest to you.


Matrix: Fix overlapped insert/remove in same undo group (#6288)

In the following scenario, SharedMatrix's UndoProvider had a bug:

Insert a segment
Remove part of a segment  <-- Splits the original segment
Undo the remove   <-- Inserts a new segment as a replacement
Undo the insertion

The bug was that when undoing the insertion at step 4, the SharedMatrix could lose track of the fact that it should remove the segment inserted at step 3 (which replaces the fragment of the original insertion at step 1 that was split/removed.)
(The fix is to replace the split/removed segment with the newly inserted replacement in the undo/redo tracking group created at step 1.)

Thanks, @anthony-murphy !


Parsing a key containing . is failing

Hi,,
As per the example:
   "fav\.movie"         >> "Deer Hunter"

This does not work with below program:
const json = {   "name": {"first": "Tom", "last": "Anderson"},   "age":37,   "children": ["Sara","Alex","Jack"],   "fav.movie": "Deer Hunter",   "friends": [     {"first": "Dale", "last": "Murphy", "age": 44},     {"first": "Roger", "last": "Craig", "age": 68},     {"first": "Jane", "last": "Murphy", "age": 47}   ] }
package main
import (
"fmt"
"github.com/tidwall/gjson"

)
func main() {
value := gjson.Get(json, "fav/.movie")
fmt.Println(value.Raw)
}

I think you are using the wrong slash. It should be fav\.movie


TestConvertService race

https://prow.istio.io/view/gs/istio-prow/logs/unit-tests_istio_postsubmit/1528828487668862976
I don't know a great way to solve this... feature flag is mutated by one test and causes a race with another test who is not 100% shutdown yet.

the cleanup should be done.


feat:use user return

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Had a chat with Darius, the css in PR is not written by him here, it comes from another merge probably, so I guess it is fine to not translate it to tailwind here.


Modernize supported Python and Django Versions

Follow on to #172

Add support for Python 3.11
Drop support for EOL Django 2.2, 3.0, 3.1, 4.0
Add support for Django 4.1, 4.2

@hishnash In order to merge this, you will need to remove UnitTests (3.10.5, Django==4.0.6) from the required checks in the GitHub project.

@johnthagen  thank you for doing this :)


Scroll doesn't work after modal is closed

For some reason after the modal window is closed, the parent is unresponsive.. even scroll doesn't work. Have to refresh the page to go back normally.
Any help would be appreciated.

Please verify and make sure you are not loading Alpine twice. Alpine is already included in Livewire.

Well, I have only this in my component. BTW I am using livewire-modal in the Filament resource.

I would recommend injecting the modal in your layout once instead of including it inside another component. Having multiple instances of the base modal component will result in issues as well. This might also be the issue causing the scroll to no longer work.


Mm23 metrics final state

Proposed changes
Describe the use case and detail of the change. If this PR addresses an issue on GitHub, make sure to include a link to that issue using one of the supported keywords here in this description (not in the title of the PR).
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[ ] I have read the CONTRIBUTING document
[ ] If applicable, I have added tests that prove my fix is effective or that my feature works
[ ] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] I have updated any relevant documentation (README.md and CHANGELOG.md)

Closing in favor of https://github.com/microservices-march/messenger/pull/20


add disable prompt option for OculusQuest

In OculusQuest Using VirtualKeyboard then , babylon.js show prompt because OculusQuest is android mobile device.
When prompt shown , browser stop webvr until resolved prompt. So can not input virtualKeyboard via vr controller until resolved prompt.
So we should to add disable prompt option for OculusQuest(and other android base vr devices).

Excellent!


[BUG] Unable to award using the new 5e "award" option

Module Version: v0.8.0
Describe the bug
When I create a "group" and set it to encounter or group I am able to open it an click the assign button and it opens the award popup. I dragged the group I created to the rewards section and it opens the group window, but the award button doesn't work. If I go back to the group again the award button works.
Expected behavior
I would expect to be able to create groups to use as encounters and/or loot and be able to award them to the players.
Foundry Version:  V12 (329)

Hi @MaineKnight... v0.8.0 hasn't quite shipped yet / posted in the official package manager. It will soon, so verifying that you have indeed installed the latest version of the module is useful in this case. I'll be putting together a brief overview video of using FQL with Item Piles and the text enrichment option of the dnd5e game system.
There is not enough information in your post above to verify if you are using the general feature as designed. Perhaps you can write down the steps and include some images of exactly what you tried.
As mentioned though verifying that you are on the latest install is the first order of business.

Hi,
I had used the json path from installation section on this page: https://github.com/Forien/foundryvtt-forien-quest-log

Steps:

Created a group with some loot assigned. This way I can just hit the award button and assign the loot as needed.

Then I assigned it to the quest and checked the award still worked, but the button was inactive.

It looks like there are general issues with the dnd5e group actor award process. If you open the "Rat Loot" group directly from the Foundry sidebar I have not been able to award any of the PC actors with that workflow here locally with latest dnd5e 4.0.3 . You also generally need to drop actors in the group actor too which your images above show an empty group.
However there is one small improvement in FQL to make. When opening the actor / "Rat Loot", etc. from the quest rewards editable: false is passed to the sheet on render. This can be changed from this line of code / HandlerDetails.js#L734
to:
const appId = await Utils.showSheetFromUUID(data, { permissionCheck: false, editable: questPreview.canEdit });

That will make the sheet editable for those that can edit it and the buttons won't be disabled. However this doesn't fix the apparent bug of no awards being granted from the group sheet itself which might be an active problem in the dnd5e game system.
As mentioned previously [[/award 1000gp]] as a custom reward does work when clicked.
I'm leaving this note here for @Forien to address in a future update.

If you access the "Rat Loot" from the right hand side bar and click award this is what you get. Then you simply mark who to award and click the award button.


Add initial Brave extension

Description
Adds an extension for the Brave browser.
The initial commands are:

Search Brave Tabs
Search Brave History
New Brave Tab

Based on the google-chrome extension
Screencast

https://user-images.githubusercontent.com/10678906/153683651-dedafdb0-f40e-4713-be11-d5e3222777a6.mov
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast

Hey @ron-myers, Looks awesome. Amazing work! Can you please clarify if it also supports profile management?

Thanks @s-r-aman - most of the work is based on the Google Chrome extension.  So shout out to the maintainers of that extension.
No profile support currently.  Could be added - for the Google extension this is a separate extension.  For Brave - I think I would prefer to maintain one extension with support for everything Brave related.

Hey @pernielsentikaer !!
New Brave Tab should always open Brave.   The phrase Chrome doesn't exist in the extensions codebase.
After reviewing the code for ways this could happen, there are two calls to OpenInBrowserAction within the extension.  My thought is Chrome is your default browser and when you use the Recently Close list it opens in your default browser(Chrome).
I will remove the calls to OpenInBrowserAction and back-port the change into the Chrome extension.
I confirmed this with the New Tab, under the recently closed section, from the Chrome extension opens Brave on my machine where Brave is my default browser.
Since this is an existing issue, I will leave it to your discretion if we merge this now or wait for correction on the above.

I would expect both New Tab commands to open a new tab with the right browser, the rest of the command could follow default browser. I'm not sure if it can help in any way, but in the latest API version, we added appPicker - is that any help?

Hi @ron-myers 👋
Could you run Prettier, so the linter is happy? Let me know if you need any help 😄

Done.  😄
That confused me for a minute @pernielsentikaer. Locally there were no errors, my version of raycast/api was out of date and needed an upgrade to get the same lint errors as on CI.  Adding a check for this may be worth considering.

Meta data added @pernielsentikaer


standardizes the version representation in blog

In this article, two different formats were used to refer to the version. This PR standardizes the version representation to the format "v1.xx."

I think this makes sense to do
/lgtm


Fixed errors in AnimationPlayerEditor when switching current_animation in the inspector

This change handle cases where the current animation has been changed in the inspector property itself, and the player can't retrieve the last played animation and its position.
Case 1: start and stop an animation in the inspector
Case 2: start an animation in the inspector, then stop and start again in the track editor
Fixes #34021

Thanks!


Uno Feature Request

Problem Statement
The Mac version does not display the binary representation of a number when the second button from the left is pushed when it is programmer mode. This functionality already exists on the following website Calculator Currently when the button is pushed the screen is blank.
Evidence or User Insights
This feature already exists in the web version. This feature is used by embedded engineers to prototype and verify calculations for masks and offsets
Proposal
Expand the capabilities of Web Version to Mac Version. One could use the web version, but there are times when someone can not access the web, working in a lab.
Goals

Verify raw register reads and writes
Verify Calculations
Exact Functionality of the Microsoft Calculator, which is heavily used by engineers

Non-Goals
See the following screenshot of what is being done

Low-Fidelity Concept
The following screenshot displays what the web version is doing .........

Thanks. We know of the issues for the mac version, and we're getting there updated builds of Uno. We may have updates soon.


problem with text spam

when i try to use the text spam bot it shows me what's on the screen then the window closes.
can you help me please.

Update discum

How to update it?

im having the same issue it says:
Traceback (most recent call last):   File "bots/server/discord_text_spam.py", line 8, in <module>     import discum ModuleNotFoundError: No module named 'discum'
how do i update if thats the problem

im having the same issue it says:
Traceback (most recent call last): File "bots/server/discord_text_spam.py", line 8, in <module> import discum ModuleNotFoundError: No module named 'discum'
how do i update if thats the problem

Type
pip install discum

im having the same issue it says:
Traceback (most recent call last): File "bots/server/discord_text_spam.py", line 8, in <module> import discum ModuleNotFoundError: No module named 'discum'
how do i update if thats the problem

Type pip install discum

i tried this and it didnt crash but it showed me this and didnt do anything


Configuration API Dependency on System.Drawing in Xamarin

I am using NLog in combination with Common.Logging, and have facaded logging throughout my library dependencies to use NLog at the consumer application level.
At this stage, NLog works well in Windows implementations, but falls over when using the Configuration API with the Xamarin.Mac framework (not Mono, rather Xam's official deviation) citing a failure to load System.Drawing which is a problem I have faced before.  System.Drawing is No longer supported as of the new Unified API.
As I understand it, this is an essential reference for the NLog.Windows.Forms extension, however after a quick look don't see any references outside of that. This makes me wonder wether the core csproj files require this reference.
If it is required, does anyone have a sneaky little workaround, short of changing and recompiling?
Cheers

Which NLog version are you using?
I think we removed System.Drawing in NLog.dll in one of the latest versions (there is PR for. have to find it)

<IP_ADDRESS> as a part of Common.Logging.NLog41 dependency.

But do you use the NLog.Windows.Forms extension or other extensions?

Negative, just the core library.

OK clear. Do you know which .Net version xamarin chooses? You used the nuget package?

I see indeed that system.drawing is still referenced. https://github.com/NLog/NLog/pull/754
I will check this weekend why, and fix it if possible.
If I can't be fixed,  we need to have a separate xamarin build. Can you help me with that? (nuspec and testing), as I never used xamarin.

Xam.Mac uses a broad subset of the 4.5 framework (based on mono), rendering most standard packages usable. The only iffy things I have seen are with some reflection components, and the System.Drawing dll.
As far as I can tell, this may be a relic of the System.Windows.Forms assembly, as I believe they are tightly related.
Pulled latest and performed a reference check with the 4.5 project, and found no usages inside the core library, and built fine. Will repackage and test locally in the morning to verify then will PR in.

I also checked, it seems not be used indeed.
If you fix a PR, then I will release 4.1.3 soon :)

OR maybe 4.2. Doubting...only small things added, but according to semver it should be 4.2 and not 4.1.3

Just about to remove it and noticed the change. Thanks.

This is will be released this week.


Strange Pin Response After Update

Prerequisites
Please check the following before posting an issue / bug report.

[X ] I am running the latest version of the firmware found HERE
[ X] I have checked the documentation HERE and found no answer
[X ] I checked to make sure that this issue has not already been filed HERE
[X ] I am reporting the issue to the correct repository

Context
Please provide all relevant information about your setup. This is important in case the issue is not reproducible except for under certain conditions.

Firmware Version: v.0.7.7
Name of device: haute 42
Link to where you purchased the device: https://www.amazon.com/dp/B0CN6M44F4?ref_=cm_sw_r_apan_dp_2G4FK0H77Z7EANP66ZFG&language=en-US&th=1
Is this a custom build?: not sure
Operating System: Windows 11
Browser: Google Chrome

Expected Behavior
I expect the buttons I press to send the signals intended to whatever device it's currently plugged into,
Current Behavior
I wasn't having any issues initially. However, since I updated to v.0.7.7 I seem to be experiencing a strange discrepancy with how the pins are responding. When I use the "pin mapping" feature in the web configurator, the pins respond as expected when pressed. However, after I save & close out of the configurator, the controller only gives me pin 26 if I press pin 21, and it only registers pin 19 if I press pins 26 & 19 simultaneously. For reference, I also have pin 18 mapped to the same controller button as pin 26 (as this is the default settings), and all of what I mentioned previously is still true when pressing pin 18 instead of 26.
Steps to Reproduce
I haven't changed any settings since I updated to v0.7.7. I even used the "reset settings" feature
Screenshots & Files

Hi @ghoticat ,
Please nuke and reload the v0.7.7 firmware then report back if the issue persists.
The nuke file can be found here:
https://github.com/OpenStickCommunity/GP2040-CE/releases/download/v0.7.7/flash_nuke.uf2

Thanks for your quick response! I tried this just now, and it still seems to be having the same issue. I even tried redownloading the flash nuke & haute42 0.7.7 update just incase I somehow managed to download the wrong files.

Actually, I apologize. Seems like after further testing, everything is working as normal. Boy, do I feel dumb! Thanks for your help!


Added runtime option to run full GC by generating GC data for all nodes

Fixes #4958.
Adds a runFullGC option in IContainerRuntimeOptions. When this is true, it generates GC data for all nodes irrespective of whether data in the node changed or not.
Added a parameter called fullGC to getGCData which when true will always generate GC data.
This will be useful in the following scenario and also in tests:
Currently, GC work has not been done by all the apps (like Bohemia). So the GC data generated and stored in summaries for a node may be incorrect. If such nodes do not change when GC is enabled end-to-end, GC will run on this incorrect data.
So, adding this option will enable us to run full GC on such documents.

Ping. @vladsud @arinwt


Show CaseNumber column as Hyperlink record

Hi,
CaseNumber couldn't shown as a hyperlink record by using lightning-datatable. Is this possible to implement it in this component?
Thanks.

You can provide custom options to the columns. So yes, you should be able to set a column as a hyperlink. Consider this project is still under heavy development and is not yet complete.


Add warning when snapshot size exceeds 50MB

Description
Closes: #247
Implementation details
The warning is added to the modal when creating a job (but only visible if the snapshot size would exceed the 50MB threshold):

Note: This is the same style of warning when creating an invalid cron string.
Todo

[ ] Fixing the styling

[ ] The icon is misaligned (due to this code which is done to fix the alignment for the cron string warning.
[ ] The hyperlink to the docs should probably be prettier.

Neat! Can you make the link in the warning white for readability?

I like how it's up front!

@joe-bell Can you make the two style changes, i.e. the hyperlink and alignment of the icon?

@joe-bell Can you make the two style changes, i.e. the hyperlink and alignment of the icon?

No longer needed ;)


change default values in sentinel.conf to mainnet

Will change this before mainnet release.

Better yet, have dash.conf's setting for the override it. I suppose since dash.conf is set to testnet, or not testnet, that can be similarly ambiguous?
sentinel.conf (testnet), dash.conf (unset) -- we are testnet
sentinel.conf (unset), dash.conf (testnet) -- we are testnet
sentinel.conf (unset), dash.conf (unset) -- we are mainnet
Isn't this how it is supposed to work? Just checking.

Fixed as of 4374690b37102369ae192be48f9d1448a4720158


Show event type in message

Hi! Is there a way to return the event type in the messages? With relation to MySQL, I can't seem to find where to configure the inclusion of the event type (INSERT, UPDATE, DELETE).
Thanks for making such a great tool available.

hey @johnwilson
That's a great question.
I'll see what we can do. Basically, we do track these events under the hood. But I need to check the state of it at the moment.


Retry mechanism should take time elapsed into account on connection failure

cc @jeromefroe
Currently the connection retry logic is purely based on the number of times the connection is used to write data buffers, which is somewhat adhoc and hard to tune. In particular, if an agg server is down for a long time before it comes back up, it could take a long time (e.g., minutes+) for collector to even retry establishing the connection, causing the writes before then to be lost.
Alternatively, I think the retry mechanism should also take time elapsed into account. In particular, when determining whether connection should be retried, it should look at two things:

Time since we retried establishing the connection last time (t)
Number of writes ince we retried establishing the connection last time (n)

and compare t and n against configurable thresholds respectively (e.g., 5s and 5000 writes). This limits the impact of write losses both time-wise and write count-wise.

#38 fixes this issue by incorporating the time since we last tried to connect into the decision of whether we should try to reconnect again.


Check for duplicate files among enabled mods

Related to https://github.com/R2NorthstarTools/FlightCore/issues/793

Basically some crashes in Northstar like audio system related ones are caused by the same audio file being overwritten twice.
Similar if two or more enabled mods replace the same script there's most likely gonna a be an issue.
As such, check between all enabled mods if they replace the same files and show a warning message if they do

@Alystrasz suggested to this in launcher directly instead ^^"

(progress tracked here: https://github.com/Alystrasz/NorthstarLauncher/tree/feat/audio-multioverride-detection)
At this point, I can detect two mods trying to override the same audio file, logging a warning message before Northstar crashes; should I also prevent file override by mod B if it was already overriden by mod A?
Also, that's the first time I hear of a script override issue, does anybody already encountered this problem?

Also, that's the first time I hear of a script override issue, does anybody already encountered this problem?

Yup, some common files get overriden by mods a lot, with most of them exposing a new global function or something, causing compile errors on conflict.
I think an example would be halo announcer + client kill callback maybe?

should I also prevent file override by mod B if it was already overriden by mod A?

Works for me. I would just make sure that if there's two mods that override the same file we should still prevent the game from launching somehow.
That way the player has to fix it now instead of just forever having two conflicting mods spewing warning messages but they keep ignoring them ^^

quoting gecko Works for me. I would just make sure that if there's two mods that override the same file we should still prevent the game from launching somehow.
i think this is the wrong way of handling it.
Instead of not launching the game when two mods have the same file there should just be a proper rework on LoadPriority.
remove load priority from the mod.json so its not set by mod authors. instead set it on northstar launch in a loadorder file, that file can be editited by people themelves or mod managers easily and northstar just adds new mods that arent in there to the file on launch.
a single file for the entirety of a users mods.
if a mod overwrites something it should overwrite it. thats how patch mods for certain issues work, two mods overwriting the same script should never result in the game not launching.
for audio files this sort of makes sense todo but also not. the last mod in the loadorder should just be the one loaded.

to append to my reply:
this isnt an issue of northstar, instead its an issue by the user. northstar should not automatically not work just because two mods overwrite the same file

At this point, I can detect two mods trying to override the same audio file, logging a warning message before Northstar crashes; should I also prevent file override by mod B if it was already overriden by mod A?

Honestly it would be better to just fix the crash rather than prevent mod loading or whatever, multiple mods should be able to add audio overrides to the same audio event (one event can have multiple source files, so we should be able to just combine them)

(progress tracked here: https://github.com/Alystrasz/NorthstarLauncher/tree/feat/audio-multioverride-detection)

Also, it looks like you are using the json file name to determine collisions, are we sure that that's accurate? Is it two audio overrides with the same file structure that causes crashes, or two audio overrides that override the same audio event (which is specified within the json)

quoting gecko Works for me. I would just make sure that if there's two mods that override the same file we should still prevent the game from launching somehow.
i think this is the wrong way of handling it.

To maybe elaborate on my initial comment, my main concern is trying to resolve partially broken setups and in the process introducing more complexity.
To maybe better explain what I mean, let me mention my favourite example on this which is VTOL patching broken mods. In the past VTOL would attempt to fix incorrectly formatted mods during installation.
This would even work for the most part but gave us the result, that modders (who primarily used VTOL) would upload incorrectly formatted mods to Thunderstore and on testing with VTOL would not notice any issues.
Meanwhile, Viper (which was more popular with players/non-modders) would not have this functionality that would attempt to correct the incorrectly formatted mod and as such mods would fail to install on there (same on r2mm and FlightCore as well). Resulting in players not being able to install mods while modders were under the impression that their mod was formatted correctly. [^1]
[^1]: And no, requiring all mod-managers to add some correcting functionality was not a solution here as the it is entirely heuristic based and a major factor for some of the bugs in VTOL. The real solution ofc is checks during mod upload but that's a different story I really don't wanna go into rn...
Basically, what I was trying to say with my original comment is that, if anything is considered incorrect, fail fast, safely, and early to indicate an issue. If there's a crash cause we don't handle it properly under the hood, we should fix the underlying issue. ^^

Instead of not launching the game when two mods have the same file there should just be a proper rework on LoadPriority.

Can you add this to https://github.com/R2Northstar/Northstar/discussions/618

quoting gecko Works for me. I would just make sure that if there's two mods that override the same file we should still prevent the game from launching somehow.
i think this is the wrong way of handling it.

To maybe elaborate on my initial comment, my main concern is trying to resolve partially broken setups and in the process introducing more complexity.
To maybe better explain what I mean, let me mention my favourite example on this which is VTOL patching broken mods. In the past VTOL would attempt to fix incorrectly formatted mods during installation. This would even work for the most part but gave us the result, that modders (who primarily used VTOL) would upload incorrectly formatted mods to Thunderstore and on testing with VTOL would not notice any issues.
Meanwhile, Viper (which was more popular with players/non-modders) would not have this functionality that would attempt to correct the incorrectly formatted mod and as such mods would fail to install on there (same on r2mm and FlightCore as well). Resulting in players not being able to install mods while modders were under the impression that their mod was formatted correctly. 1
Basically, what I was trying to say with my original comment is that, if anything is considered incorrect, fail fast, safely, and early to indicate an issue. If there's a crash cause we don't handle it properly under the hood, we should fix the underlying issue. ^^
Footnotes

And no, requiring all mod-managers to add some correcting functionality was not a solution here as the it is entirely heuristic based and a major factor for some of the bugs in VTOL. The real solution ofc is checks during mod upload but that's a different story I really don't wanna go into rn... ↩

that issue u mentioned is based on not properly formatted mods, not on multiple mods containing the same script/files tho?
dont see how that is relevant.
northstar should not just stop working simply because two mods add the same file,
theres mods that are literally designed to overwrite the script of another mod this way which just load AFTER the original mod thus work.
this isnt a norhtstar issue but once again a user issue that can not be automatically solved properly

no, i no longer engage with that topic. my last words on that have been said.

not really a discussion then is it?

that issue u mentioned is based on not properly formatted mods, not on multiple mods containing the same script/files tho?
dont see how that is relevant.

I was simply giving an example why failing early is important in the right situation.

northstar should not just stop working simply because two mods add the same file,
theres mods that are literally designed to overwrite the script of another mod this way which just load AFTER the original mod thus work.

I'm 100% in agreement with you and @ASpoonPlaysGames on this, which is why I wrote:

If there's a crash cause we don't handle it properly under the hood, we should fix the underlying issue. ^^

Also, it looks like you are using the json file name to determine collisions, are we sure that that's accurate? Is it two audio overrides with the same file structure that causes crashes, or two audio overrides that override the same audio event (which is specified within the json)

We can discuss that again on the PR when it's opened (soon)!
I think I'll address audio overrides and script overrides in separate PRs.

To maybe better explain what I mean, let me mention my favourite example on this which is VTOL patching broken mods. In the past VTOL would attempt to fix incorrectly formatted mods during installation. This would even work for the most part but gave us the result, that modders (who primarily used VTOL) would upload incorrectly formatted mods to Thunderstore and on testing with VTOL would not notice any issues.

I added warning to checks to modders back in 1.20.0.
Issues like above should not be prevealant anymore, you recieve a big warning banner on trying to export.


[1.28] Consistency: orientation on layout.Splitter vs layout.FixFlex

To determine the orientation of controls the sap.ui.layout.Splitter uses a "orientation" property with the sap.ui.core.Orientation Enum as valid values. The sap.ui.layout.FixFlex on the other hand uses a "vertical" property with boolean values for the exact same decision.
I think the usability (from a developers point of view) would improve if both of them would use the same properties. Preferably the "orientation" one because that is how these config are set in the whole framework.

Hi themasch,
thanks for reporting. I opened internal incident<PHONE_NUMBER>  for this. We will come back to you when there is news on this one.
Best Regards,
Martin

I agree that it's unfortunate that the API is inconsistent here. However, for backwards-compatibility we can't remove any of these properties. We could just add the "orientation" property to the FixFlex control as you suggested. But the "vertical" property would still remain (marked as deprecated). Do you think this would be an improvement? Or would it be more confusing?

Oh, yeah, you can't just remove properties, that's pretty clear.
Adding the orientation property and deprecating 'vertical' sounds good for me!

Hi @themasch,
quite old issue. I synced again with the responsible product owner.
This enhancement will not come. Feel free to provide a contribution here.
All the best, Florian


ValueError: Layer weight shape (3, 3, 256, 256) not compatible with provided weight shape (3, 3, 128, 256)

I want to set the weights of a particular layer(9) to of another layer(7), getting this error.
Please refer to line 70 onwards from the gist. here.

Solved!


Object.addLeftTile is deprecated.

Use version ^1.0.0 of the status-bar Service API.
Object.addLeftTile (/usr/share/atom/resources/app.asar/node_modules/status-bar/lib/main.js:174:16)
CtagsStatusView.mount (/home/anders/.atom/packages/ctags-status/lib/ctags-status-view.coffee:17:31)
Object.consumeStatusBar (/home/anders/.atom/packages/ctags-status/lib/ctags-status.coffee:107:21)
Provider.provide (/usr/share/atom/resources/app.asar/node_modules/service-hub/lib/provider.js:30:34)
ServiceHub.provide (/usr/share/atom/resources/app.asar/node_modules/service-hub/lib/service-hub.js:29:18)
Package.activateServices (/usr/share/atom/resources/app.asar/src/package.js:372:65)

Hey @baelter, how do you receive that "deprecation" notice? I cannot see the deprecation notice from dev-console in Atom. Do I have to setup something?

Got it from the "Deprecation Cop". It's gone in 1.2.5. There are two other deprecated calls though, #86 and #87


Fix incorrect type used in WindowEvent::SetSize and related events.

Fixes window size not being round-tripped correctly when being saved/loaded via cx.window_size() then Application::inner_size(). I believe it was previously working only on systems without DPI scaling; my windows desktop behaved as expected but macos laptop did not.
It seems to be the cause was that the window_size() method was retrieving the size from cx.cache.get_bounds(parent_window) which uses physical units rather than logical.

with this change, using inner_size((x, y)) in Application now uses PhysicalSize, which is incorrect for systems with DPI scaling.


XHR options should be added and passed to vast-client

There should be an option to define some xhr options for each vast config (or in general) and pass them to the vast-client. The current version of vast-client (not the version used within this repo) seems to offer the following options for vast requests.

timeout
withCredentials

My current problem is, that cookies will not be send with the request to a foreign vast url. I figured out that this is because of the withCredentials option which is set to false by default.
This limitation avoids e.g. any kind of cookie targeting and frequency capping using the majority of adservers.
See:

https://github.com/theonion/videojs-vast-plugin/blob/master/videojs.vast.js#L75
https://github.com/dailymotion/vast-client-js/blob/master/src/client.coffee#L11
https://github.com/dailymotion/vast-client-js/blob/master/src/urlhandlers/xmlhttprequest.coffee#L16

+1

I know this thread is pretty old but i can't find any solution anywhere so, anyone managed to get this cookie thing working?


LRO should support return 202 intermediate response

Currently the LRO methods can only return the final response data, but not directly return the 202 responses at first call when use waitUntil.Started.
The work around is manually parse the raw data, but definitely not the right behavior.

Here is the case reported by storage team when trying our track2 SDK: Storage restoreBlobRange.
The 202 response also contains some valuable information.
Moreover, in this case, the responses for 200 and 202 are the same. What if they are different?

related to https://github.com/Azure/azure-sdk-for-net/issues/29060


Migrate applicant question tests to use TestContext

Description
See #3323 for bigger context.
Checklist

[x] Added the correct label: < feature | bug | dependencies | infrastructure | ignore-for-release >
[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

Codecov Report

Merging #3336 (5cf80d8) into main (4473e16) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #3336   +/-   ##
=========================================
  Coverage     80.00%   80.00%
  Complexity     2343     2343
=========================================
  Files           287      287
  Lines          9242     9242
  Branches        617      617
=========================================
  Hits           7394     7394
  Misses         1608     1608
  Partials        240      240

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Feature: add an option to automatically convert all non-luna staking rewards to Luna or token of choice

When withdrawing staking rewards, the rewards come in several different types of tokens. This is a request to add a feature that automates swapping non-luna rewards into currency of choice or into Luna.
Example:

user withdraws the rewards of 5 Luna, 1 UST, 200 KRT, 1 EUT.
user selects option to auto-swap non-luna tokens into UST.
upon withdrawal, user account is credited with 5 Luna and the sum of 1 UST + UST equivalent of 200 KRT + UST equivalent of 1 EUT.

Users should be able to choose which currency they would like rewards to be auto-swapped into.

It is already possible to do with a function called multiple swap.
We are also preparing the feature to withdraw rewards and automatically multiple swap them.

https://stake.tax/

This feature will be implemented soon in the next version of Terra Station


Handle an hidden field

Hi!
There's a problem in validation with hidden input ( passed by jquery .hide() ). The submit button is activated even if the hidden fields are not populated.
Any solution?
Thanks :)

Yes, you'll need to override the default selector so it doesn't exclude hidden fields:
$.fn.validator.Constructor.INPUT_SELECTOR = ':input:not([type="hidden"], [type="submit"], [type="reset"], button)'

I had the same issue and added data-validate="true", but it didn't work. I believe it to be a bug.
Only way to get it working was using class="hidden" instead of type="hidden".
Similar problem in #531


Add a new directory “modules/security”

Proposal: Add a new directory “modules/security”
Author: Massimo Ferrari <@cross-logic>
Date: 2018-04-05

Status: New
Proposal type: core design
Targeted release: Ansible 2.9
Estimated time to implement: 1 day to 1 week

Motivation
We want to make it easier for users to consume, find and contribute modules interacting with security-specific solutions and services.
Problems
With the increasing adoption of security automation and orchestration practices also grows the number of Ansible modules supporting security solutions. In the absence of a dedicated module category those are spread across multiple categories making their consumption harder and diluting the value of the contributions.
Solution proposal
The creation of new top level module directory lib/ansible/modules/security as a default destination for all modules dedicated to security tools, under development or already submitted. The relocation of all the modules falling in this category to this new directory.
Which modules will belong in modules/security?
The “Security” domain encompasses a broad variety of cybersecurity products and services which protect individuals and organizations from the loss or damage to their data, applications, IT systems, networks and devices from malicious or unintended activities.
Ansible Security category will be congruent with the industry definition of "logical security."
IT Compliance, IT Governance and Physical Security solutions will not be included.
This is a list of the modules that will be relocated:
https://docs.google.com/spreadsheets/d/1YWfQ0SqasbIUVx3ETaPctyWr4bjqip4RRMX-bTB4aEk/edit?usp=sharing
Testing (optional)
Unit tests which invoke module code from the Python package name and not the Ansible flattened namespace need to be tested.
Any existing unit tests under test/units/modules/ will need to be moved to test/units/modules/security
Documentation (optional)
The current categories are in fact hardly documented at all. Thus there is no need to change documentation.
Anything else?
We are aware that there is also ongoing discussion to move from directory in favor of a more flexible way of content management based on Mazer. However, we are not there yet, and it will take more time and development until that is release ready.

Your Google document is not public. Please allow public access.

@MarkusTeufelberger Looks like publicly sharing files is not allowed with our company google apps. This is the list of modules:
https://gist.github.com/liquidat/9f81dc1b469dc8e278a2952e5c990ba4

Currently the directory structure is used for giving people rights on specific collections of code, but also to generate documentation related to collections of modules. If you move some of the e.g. Cisco ASA modules out of network/asa directory structure, people looking for ASA modules will not find some of the modules, or need to be defined in 2 locations in order to maintain these modules.
On the other hand, security goes beyond a firewall or an access list. Managing users on a system relates to security. Managing permissions of files relates to security. Creating a new container relates to security. Authorized keys ? Sudoers ?
So I am not in favor of this. Promoting security and security automation is a noble goal, but touches all technology domains and is a state of mind covering everything you do when automating.

@dagwieers thank you for the feedback.
The plan is to follow a "per platform" criteria. Which means, as per your example, all Cisco ASA modules will be under security/asa, and the documentation will reflect that accordingly.
Indeed security covers many different areas and practices, but this proposal is not an attempt to being omni comprehensive. As mentioned in the PR, the goal of this change is to promote the organic growth of contributions from security practitioners. The "logical security" domain (which includes all the security applications and platforms that organizations use to provide security management, perimeter defense, intrusion detection and prevention (IDP), access control, authentication, malware protection, encryption, data loss prevention (DLP), etc.) is a convenient starting point, synergetic to other contributions in existing areas (like networking). This doesn't exclude further expansions in a second iteration.

I sympathize with that noble goal, but I am not in favor of a generic "security" directory in modules/, as it makes it harder to know where to put a specific module. Cisco ASA is a network security product, there's no doubt it goes into network/ now and that's where people will look for it. If we also have security/ things will be harder to find IMO.
We are better off with an orthogonal classification for the directory structure, and use a more flexible solution for documentation and indexing, which does matter to end-users.
I assume you'd like to see crypto/ and identity/ move to security/ as well ? How about win_audit_*, win_certificate_store, win_firewall and win_security_policy ?

Hey @dagwieers , maybe this is a misunderstanding: with our approach we do want to follow the same way the other directories in the repo are already set up. If you look at the directory dedicated to cloud, the modules in there are focused on platforms’ functionalities. Even if cloud adoption comprises a broader set of topics, including workloads lifecycle and management, user management, networking, and so on. The same is true for storage for example: the modules in that directory focus on platforms even when some functionalities also cover networking and other topics which are strictly speaking not storage.
This is what we want to do for security - and why the “grouping” we want to create is not generic but platform focused. With security, we want to retain the same logic and focus on security technologies/platforms/applications (whose primary goal is providing security capabilities) instead of the general security posture.

Isn't this proposal obsolete since collections have been introduced? I also remember a PR that I can't find now that tried to remove the folder hierarchy of modules.

Isn't this proposal obsolete since collections have been introduced

You are correct. I will close this out.


chore: Update dependencies

Add the prettier extends so it runs as part of the linting
Bump eslint-plugin-next for a Windows only issue https://github.com/vercel/next.js/pull/28745

Thanks. I've already had a local branch for the updates, but I was waiting for Next 12.0.3 to release.
I've updated the PR to upgrade the other dependencies as well.


Feature/add merge method

Codecov Report

Merging #14 (2db95a2) into main (b4219e8) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
+ Coverage   95.47%   95.57%   +0.10%
==========================================
  Files           3        3
  Lines         221      226       +5
==========================================
+ Hits          211      216       +5
  Misses          7        7
  Partials        3        3

Impacted Files
Coverage Δ

collection/collection.go
94.87% <100.00%> (+0.13%)
:arrow_up:

Continue to review full report at Codecov.

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


GET_ACCOUNT_DATA error

I am running main.py from Windows, seem to get passed two-factor authentication.
Getting response below:
{"spHeader":{"SP_HEADER_VERSION":-1,"success":false,"userGuid":"c87d2e4d4b0f4fce908042c406c398a2","authLevel":"USER_IDENTIFIED","errors":[{"code":200,"details":{"actionType":"GET_ACCOUNT_DATA"},"message":"Authorization required."},{"code":201,"message":"Session not authenticated."}],"username":"xxx@yyy.com","status":"ACTIVE"}}
Please advise.

Looks like you didn't authenticate successfully from the error message. There's no API doc available from Personal Capital so it's all guess work really.

@iantrich Once you load a saved session you shouldn't need to authenticate any more.

@haochi Have you tried the main.py from the project recently? I don't think I'm missing anything, but running that a 2nd time I get the following
Traceback (most recent call last):
  File "pc.py", line 87, in <module>
    main()
  File "pc.py", line 80, in main
    accounts = accounts_response.json()['spData']
KeyError: 'spData'

which makes me believe that the session.json is not being loaded properly. I admittedly know very little about sessions/cookies, but was hoping that my PR to bump the requests module might help.

@haochi also, if you're curious, this is the project I'm currently working on with your module: https://github.com/custom-components/sensor.personalcapital
It a custom-component for Home Assistant. It currently works great, but I have to reauthenticate every restart and the session seems to expire within a day when trying to use it every 30 minutes to keep it alive.

I had a friend try main.py and he had no issues, so it must be something wrong on my end.

@haochi figured it out! I had my account set to always require a 4 digit code. All this trouble has been my own doing 🤣 Thanks for the assistance and the module.
Sent with GitHawk

Noted. Thanks

@iantrich I'm encountering the same issue, how did you turn that functionality off in your account?

@4G-Gregg it's in the account settings


Add a single record without removing existing records

I have a domain that I purchased from GoDaddy, let's call it "domain.com".
I want to use this plugin to add a dns record that maps my google_compute_instance which has a static google_compute_address to a subdomain, let's call it "subdomain.domain.com".
To do this I added a godaddy_domain_record resource with a single record.
The problem is that I have additional records which are used for "domain.com" and for other subdomains under it.
According to your documentation my configuration will cause Terraform to replace all my existing records with just that one record that I specified in my configuration (unless I misunderstood the documentation).
I don't want to specify all my dns records in my Terraform configuration, I only want to specify the ones that are used by this specific google_compute_instance. I also still want to be able to add dns records manually through the GoDaddy website.

@guysaha1 - if I understand the behavior described above, using Terraform's import feature might do the trick. Please let me know if this resolves the issue being described.

Hello, I'd like to use this provider from within Terraform Cloud.
So, having an already populated domain (with MX records for my business email and so on), I'd like to know how to proceed.
Should I first perform an import locally, extracting domain records from the state file?
Thanks in advance.

@n3integration I have come across the same issue as @guysaha1.  I would like to simply write a single record into my domain but the plugin will erase all other records on apply.  I think the terraform import command would solve this problem for me but based on the documentation seems tedious.  Do you have any suggestions on how to import all the records of the domain before writing the new one in?

Same here as guysaha1. Do you have any suggestions on how to import all the records of the domain before writing the new one in?

To those interested in this ticket: check out https://github.com/anna-kampani/terraform-provider-godaddy. They appear to have forked and solved this problem.

To those interested in this ticket: check out https://github.com/anna-kampani/terraform-provider-godaddy. They appear to have forked and solved this problem.

Facing similar issues, I've decided to re-implement provider from scratch to operate on record level.


Added:

1. lifted_structure.c:
    reference paper is https://arxiv.org/abs/1511.06452
    port from https://github.com/rksltnl/Caffe-Deep-Metric-Learning-CVPR16
2. lifted_structure_loss:
    example of how to use lifted_structure loss function

Merged and added to branch 'dev'.

Re-open as it is still a work in progress.
What needs to be done:

CPU-only implementation.
Clean the local allocation.
Re-factor the cost layer to be able to choose between euclidean loss and lifted structured loss.
Update bcnn-cl
Ideally, provide a specific data iterator that automatically constructs training batches. Currently, batch has to be pre-specified offline.


Add CentOS installation guide

Currently we're blocked on CentOS as there is no 9p support:
$ sudo docker run -ti busybox sh
docker: Error response from daemon: OCI runtime create failed: qemu-lite-system-x86_64: -device virtio-9p-pci,disable-modern=true,fsdev=extra-9p-kataShared,mount_tag=kataShared: 'virtio-9p-pci' is not a valid device model name: unknown.


add the prefix dm- to the filenames for DM docs

What is changed, added, or deleted? (Required)
As a preparation step of merging DM master docs to TiDB master docs, this PR adds the prefix dm- to the filenames for 5 DM docs to avoid filename conflicts during the merge.
Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version)
[ ] v5.3 (TiDB DM 5.3 versions)
[ ] v2.0 (TiDB DM 2.0 versions)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from:
Other reference link(s):

Do your changes match any of the following descriptions?

[x] Delete files
[x] Change aliases
[ ] Need modification after applied to another branch
[ ] Might cause conflicts after applied to another branch

LGTM


Adding Lockdoor Framework

Adding Lockdoor Framework to the template config file ( as requested on Twitter )

Hello Sofiane,
Thank you for opening this pull request.
I have tried to launch your revision, and have run into some problems.
After having tested your submission, here is a list of small fixes so I can merge the PR correctly. I have developed the two biggest issues to help you navigate through fixing.

Issues

Branch is 8 commits behind master

No python installed in docker container

Logo as static image; not a GIF

Move lockdoor under Kali Web No VNC and above DVWA

Thanks again for contributing, feel free to use this thread to as further questions ! :+1:
Regards,
k

Branch is 8 commits behind
It seems you forked a older version Recloud, where docker installation issues had been fixed. Please update your fork, and simply create a newer one and replace the templates file with the one including lockdoor

Screenshot of lockdoor behind master

No python installed in your docker container
When trying to run your container, it seems you did not include python, or missplaced it during installation? I did not try to fix this further, but some additional testing might be required to ensure your project has been successfully ported to Docker.

Screenshot of lockdoor container without python

Hi khast3x,
I just noticed that, I forked your repo long time ago.
I’ll fix that ASAP

So, I'll fix the 4 issues:

[x] Branch is 8 commits behind master
[x]No python installed in docker container
[x]Logo as static image; not a GIF
[x]Move lockdoor under Kali Web No VNC and above DVWA
About the docker container, I released Lockdoor v2.0 this morning, so I just checked the last build on docker, and it was git cloning an empty repo that I had this morning before releasing the v2.0


Day & Tower Input Parameters.

Ticks per day now dependent on other variables.
Summary
This PR is aiming to solve the issue #13 . Day and tower parameters as set from discussion in meeting.
Additional Information
From meeting: input params: composition of agents (and therefore total number of agents), number of agents per floor (this will need to be a factor of the total number of agents), time spent per floor. (Courtesy of @jzzheng22 )
Test Plan
Tested by observing logs of reshuffling etc while changing ticksPerFloor in main.go - changes day length as expected.

Sorry for the re-reviews, needed to resolve a conflict and I think it automatically makes the reviews stale?


Improve section overflow error message

When a section overflows, it's useful to know by how much; for example, if a bugfix causes a section to become too large, it's better to split the section if the overflow is too much, or if a byte or two might be optimized out somewhere in it.
It's also useful when INCBINing files into sections.

Add information to the commit message...

Bumping because information was added. That said, if you weren't/aren't satisfied with it, doesn't the "Merge and squash" option allow you to write a new commit message?

I don't like to modify other people's commits. I would only do it if the changes are actually important and the person doesn't show any signs of life for ages.

Okay, noted.


Update README.md for Sentinels

[x] Rephrase content as needed
[x] Fix typos and formatting

@ramesht007 see if the most recent edit works.


[soundcloud] Added soundcloud license detection for file name formatt…

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests

What is the purpose of your pull request?

[ ] Bug fix
[ ] New extractor
[x] New feature

Description of your pull request and other information
Added the ability for a file formatted with -o and downloaded from soundcloud to use the %(license)s tag, and when used prefixes it with the footers license.
Available soundcloud licenses are
all-rights-reserved
cc
cc-by
etc...

Cherry-picked in4bfd294e2f83301921494c02e497cccf1a26cfd5, thanks.


Do mpeg4?

Hi, it seems there is some preparation for mpeg4 support but no code yet, is that right? Or maybe I misunderstood something?

Ah, from my tests,. it seems an .avi file with mpeg4 stream is working well, but an .mp4 file with h264 is not.


Fix #810 bool value

Fix #810
Signed-off-by: Xian Chaobo<EMAIL_ADDRESS>

@jimmyxian can you please add a few tests for boolValue and intValueOrZero ?
Thanks.

@vieux aha, my bad.....forget unit test........I will add some tests....

ping @vieux
Done to add unit test. Check again, thanks

LGTM


Fix error handling on load

[x] tests and linter show no problems (npm t)
[x] code is properly formatted (npm run fmt)
[x] description of changes is added in CHANGELOG.md

Landed in https://github.com/metarhia/impress/commit/355cf54fa8d1f1fd06c2ed02bd283b3afce4bdde


Add Triton Transpiler doc

Description of changes:
Add Triton Transpiler doc
Related Issues:
Linked Issues:

Issue #

Issues closed by this PR:

Closes #

@NorthmanPKU Can you pull the changes from nki to your triton branch? In this case, the changes in this PR will only include the doc file.

@NorthmanPKU Can you pull the changes from nki to your triton branch? In this case, the changes in this PR will only include the doc file.

OK


Saving shared photo with message yields an obs with no photo

Describe the bug
When you share a photo into the app along with a message, you land on ObsEdit and all looks fine, but when you save the observation has no photo
To Reproduce
Steps to reproduce the behavior:

Go to the Photos app and share a photo with iNat Next
In the sharing dialog, write some text before pressing "Post"
On ObsEdit, note that your photo is there and your message is in the description
Tap SAVE
On MyObs, the observation has no photo
If you tap it, ObsEdit shows no photo

Expected behavior
Sharing a photo into the app with a description should save an observation with a photo.
Context (please complete the following information):

Device: iPhone SE (3rd gen)
OS: iOS 17.4.1
App version: 0.32.5 (93)

Additional context
Reported by Carrie.

Definitely was happening but apparently got fixed in the latest build.


Cedar standard library

Rendered
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

A meta question: Is it for human-readable format only? If not, we should provide the JSON representation as well.

I support the "extends" feature and have several scenarios where it would help us manage schemas in large-scale systems with many services and application builders, in which central teams define a common structure which others can expand.
The Cedar standard library is a special case in which a common structure is so ubiquitous as to be nearly universal. I think this could be useful, but will naturally take longer to finalize these types and so it makes sense to decouple the standard type definitions (like OIDC User) from the rest of the proposal. We'd probably want to run the standard types through a process that looks like a standards community, including audiences from the OpenID Foundation, for example.

When it comes to bikeshedding the naming, if we sought to avoid "extends" because of it's object-oriented connotations, I would be in favor of the aforementioned suggestion of "expands".

@aaronjeline says

This RFC doesn't actually say what the type deceleration for OIDC::User is, is that intended?

@D-McAdams says

will naturally take longer to finalize these types and so it makes sense to decouple the standard type definitions (like OIDC User) from the rest of the proposal. We'd probably want to run the standard types through a process that looks like a standards community, including audiences from the OpenID Foundation, for example.

These are good thoughts.  I would say that this RFC proposes three things:

the concept/mechanisms for a standard library
the extends keyword (or whatever we end up bikeshedding it as)
the idea of adding OIDC::User as a standard library type

but this RFC does not propose a specific declaration for OIDC::User, only proposes that we somehow arrive at a consensus one, and then add it to the standard library. I propose that we can delay hashing out the details of the OIDC::User declaration to an implementation PR, or to some other kind of process with more stakeholders as @D-McAdams suggests.

I'm a big fan of extends, that will make my life easier in my own schema definitions. One detail on the stdlib - I like the idea of a reference library, but I'm not a fan of prepending it to the schema because from what I understand this would allow any policy author to reference those entities regardless of whether they're being used in a particular application. If there's a way to "not import unless referenced or used somehow," that would solve my concern.

I think it would make more sense to break this into two RFCs: adding libraries to Cedar (combining this RFC with parts of https://github.com/cedar-policy/rfcs/pull/61) and an RFC proposing what types to include in the standard library.

Here's a schema we're using for our Janssen Project Cedar science project:
A few things to think about that this schema has:

I left the user claim section pretty sparse, as domains can always extend.
I added the ubiquitous "Roles"
I added some standard "Context" claims... which will make all the AI/ML people excited
A lot of tokens share certain standard claims, like iss or jti... so I abstracted these
A lot of enterprise policies are about OAuth clients and OAuth access tokens, so these need to be modeled too
It's important to make the relationship between the trusted issuer and the assertion

These last three comments (@khieta, @andrewmwells-amazon, and @nynymike) all make good points; I'll work on splitting this RFC into (at least) two parts, separating the ideas of library mechanisms, extends, and a Cedar-maintained "standard library".

As the first step of that, I just posted RFC 69: Schema libraries. Comments welcome.


Add setting for maximum number of parent directories

At the moment, parent directories will be added to the display names indefinitely until they are unique. A setting should be added to limit the number that can be added.

Also, could consider abbreviating the names of parent directories, either as necessary (to not go over some maximum width) or always.

can we also add file name only mode

Sure, I'll add that as an option

Filename only mode has now been added


Support TypeScript import type

Hey @beenotung
Thanks a lot, creating this tool
When using NodeNext import type imports also need to resolve in .js extensions, otherwise ts compiler will throw:
gen/api.ts:16:36 - error TS2835: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './configuration.js'?

16 import type { Configuration } from './configuration';

I see that these are currently ignored in the script via:
https://github.com/beenotung/fix-esm-import-path/blob/aa831df0a0e7561d863e371a4bcf39e84ecf58dd/fix-esm-import-path.js#L210
When I am removing this line, they are fixed as well.
Am I missing something, why we don't want to replace those as well by default?
In this case, we could maybe expose a CLI option to include/exclude those imports, if this makes sense to you?
🙌

The program is not modifying import type because those import are supposed to be removed when transpiled into js file.
Is it the case that, your transpiler / bundler preserve import type statements in the output?

Running with bare bones, vanilla typescript compiler:
"typescript": "^5.0.0"

tsconfig.json
{
	"compilerOptions": {
		"strict": true,
		"target": "ESNext",
		"module": "NodeNext",
		"moduleResolution": "nodenext"
	}
}

build command
tsc

Which will throw for all non-types imports (fixed by this package), but will still throw for type-imports:
gen/common.ts:17:34 - error TS2835: Relative import paths need explicit file extensions in EcmaScript imports when '--moduleResolution' is 'node16' or 'nodenext'. Did you mean './base.js'?

17 import type { RequestArgs } from "./base";

I'm working on --process-import-type and --preserve-import-type flag to address this issue.
I'm thinking what should the default behavior be. It seems easier to process them by default, and only preserve them as is if the project is not affected by this issue and want to minimize "git changes"

Wonderful, thanks a lot ✨
Yeah, that sounds good

The patch is released in v1.4.0, thanks for your suggestion.
Please feel free to reopen this issue if it is not fixed in your case

Wow, that was fast, thanks a bunch - works great! 🙏

tsconfig.json
{
	"compilerOptions": {
		"strict": true,
		"target": "ESNext",
		"module": "NodeNext",
		"moduleResolution": "nodenext"
	}
}

maybe u can change to "moduleResolution":"Bundler"


error during save with cell double

the following function:
void createExcel() async {
var excel = Excel.createExcel();
Sheet sheetObj = excel['Sheet1'];
var cell = sheetObj.cell(CellIndex.indexByString('A1'));
cell.value = DoubleCellValue(10.50);
var fileToBytes = excel.save(fileName: 'Esempio.xlsx');
}
stops on the last instruction with the message:
Error: Exception: null does not work for DoubleCellValue

My flutter doctor -v
[√] Flutter (Channel stable, 3.13.6, on Microsoft Windows [Versione 10.0.19045.3693], locale it-IT)
• Flutter version 3.13.6 on channel stable at c:\Users\Daniele\Documents\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision ead455963c (9 weeks ago), 2023-09-26 18:28:17 -0700
• Engine revision a794cf2681
• Dart version 3.1.3
• DevTools version 2.25.0
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
• Android SDK at C:\Users\Daniele\AppData\Local\Android\sdk
• Platform android-33, build-tools 33.0.0
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.6.2)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.6.33723.286
• Windows 10 SDK version 10.0.22000.0
[√] Android Studio (version 2021.3)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
[√] VS Code (version 1.84.2)
• VS Code at C:\Users\Daniele\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.78.0
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Versione 10.0.19045.3693]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 119.0.6045.200
• Edge (web)        • edge    • web-javascript • Microsoft Edge 119.0.2151.93
[√] Network resources
• All expected network resources are available.
• No issues found!

@FeliceT57 How does "cellstyle" solve the issue? Please can you post some code with your solution. This is still an issue!

This is still an issue, please open the ticket.

this will be fix to this pr https://github.com/justkawal/excel/pull/311

CellStyle sn = CellStyle(numberFormat: NumFormat.standard_0);
cell = sheetObj.cell(CellIndex.indexByString('C$j'));
cell.value = DoubleCellValue(element.quantita);
cell.cellStyle = sn;

Has this gone to stable release yet? Will it be in 4.0.3?

Please help me understand when this PR is pushed out and in which version this fix will be?


SCP-2438 - Add formatting to constant params

This PR essentially adds the formatting options to value template parameters in the same way as we added it for choices in SCP-2394, SCP-2408, and SCP-2409.
It also adds formatting to the errors when adding choices out of range (which was missing)
Probably easiest to look at the commits independently than the full diff at once
Deployed to: http://pablo.marlowe.iohkdev.io/
Pre-submit checklist:

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages
[x] Formatting, materialized Nix files, PNG optimization, etc. are updated

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

Pre-merge checklist:

[ ] Someone approved it
[x] Commits have useful messages
[ ] Review clarifications made it into the code
[x] History is moderately tidy; or going to squash-merge

LGTM... minor note, with this new formatter, does this description still makes sense? or should it now be "Amount of ADA..."

I think we probably can remove it once Marlowe Run has the same, since metadata is for the simulator in the first place


WARN PuppetWebBridge init() inject FINAL fail

I met a problem when running the demo ding-dong bot on my mac.
Here is the console log
✘ programath@wentuopudeMaxBook-Air ⮀ ~/wechaty ⮀ ⭠ master ● ⮀ node example/ding-dong-bot.js
| __        __        _           _
| \ \      / /__  | |   __ | | _   _
|  \ \ /\ / / _ / _| ' \ / ` | __| | | |
|   \ V  V /  / (| | | | (| | || || |
|    _/_/ _|_|| ||_,|_|_, |
|                                     |__/
=============== Powered by Wechaty ===============
-------- https://github.com/zixia/wechaty --------
I'm a bot, my super power is talk in Wechat.
If you send me a 'ding', I will reply you a 'dong'!

Hope you like it, and you are very welcome to
upgrade me for more super powers!
Please wait... I'm trying to login in...
info Wechaty v0.3.9 initializing...
undefined
WARN PuppetWebBridge init() inject FINAL fail: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! PuppetWeb initBridge() exception: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! PuppetWeb init exception: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! Wechaty init() exception: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! Bot Error: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! Bot     at Bridge. (/Users/programath/wechaty/src/puppet-web/bridge.js:84:15)
ERR! Bot     at next (native)
ERR! Bot     at onFulfilled (/Users/programath/wechaty/node_modules/co/index.js:65:19)
ERR! Bot     at ManagedPromise.invokeCallback_ (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:1315:14)
ERR! Bot     at TaskQueue.execute_ (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2736:14)
ERR! Bot     at TaskQueue.executeNext_ (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2719:21)
ERR! Bot     at asyncRun (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2595:27)
ERR! Bot     at /Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:639:7
ERR! Bot     at process.tickCallback (internal/process/next_tick.js:103:7)
ERR! Bot  init() fail: %s Error: execute proxyWechaty(init) error: 503, init() without a ready angular env
ERR! Bot     at Bridge. (/Users/programath/wechaty/src/puppet-web/bridge.js:84:15)
ERR! Bot     at next (native)
ERR! Bot     at onFulfilled (/Users/programath/wechaty/node_modules/co/index.js:65:19)
ERR! Bot     at ManagedPromise.invokeCallback (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:1315:14)
ERR! Bot     at TaskQueue.execute_ (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2736:14)
ERR! Bot     at TaskQueue.executeNext_ (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2719:21)
ERR! Bot     at asyncRun (/Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:2595:27)
ERR! Bot     at /Users/programath/wechaty/node_modules/selenium-webdriver/lib/promise.js:639:7
ERR! Bot     at process._tickCallback (internal/process/next_tick.js:103:7)
The version of node is v6.4.0 and npm is 3.10.3
Can you tell me what's going wrong?

it looks like you are using the old source. see #21 .

@programath did you make wechaty work with WECHATYA_HEAD=chrome ? if you have any problem, please feel free to let me know if I could help.

hope you had already made it working as well.

thx. I have just tried the latest version and it now works well.

haha, great.
you are lucky, because I can't run it last weekend. shame. :P


[Bug] [dolphinscheduler-dao] An error is reported when executing the MySQL table creation statement

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
An error is reported when executing the MySQL table creation statement :sh script/create-dolphinscheduler.sh

What you expected to happen
.
How to reproduce
When configuring the URL of the database, add usessl = false
Anything else
No response
Version
2.0.2-prepare
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hi @janeHe13 , maybe it's not a bug, please provide more info, such as jar version, driver name, url.


[FEATURE REQ] Investigate Playback and record system

One potential option is to leverage the work Mike Harder is doing.

There should be an agent listening to requests and based on previous cached responses from the live service, return back the expected response.
On option is to run the tests to the actual endpoint, and record the results, ~once a week, to stay up to date.
This reduces cost and improves testing performance.

@RickWinter , we should only create one playback-record test framework for C++ and see if we can reuse it here.

I think we should have live tests + unit tests, and NOT have response recorder.
For the response recorder, we need to filter out secrets from the recorded responses, and understand timestamps. And two - it won't be possible to have request-to-response map for some cases, when on a second try, the service is supposed to give another reply for exactly the same request.
Look at what we need, and if we are achieving the goals that we set, regardless of the technology used, versus what shiny technology can we use, if it can enable no new scenarios.
I also think that checking in responses is going to turn tests into a harder to understand and manage repo, where it is hard to track everything that is happening or is supposed to happen.
I understand that in other languages, you get frameworks, but if we are supposed to write playback/record engine ourselves, I don't think it is worth it. It needs to take care of secret filtering, timestamp handling, and failures need to be easily debuggable.
I  think that writing all that by ourselves is a bad idea. The only possible option I see is to write a wrapper that makes interop calls for the .NET/Java record/playback engine that is already written (we can write a transport adapter doing that).
But I think that we should set the goals by "make sure we verify X", instead of "make sure we use response playback/record".

But I think that we should set the goals by "make sure we verify X", instead of "make sure we use response playback/record".
What we want is the ability to run "live tests" without needing the actual endpoint.  The framework also doubles as a tracing mechanism as customers can turn it on to log what went in and what came out.

Labeling this one "Client" since this is about finding a solution for C.  Agreed it's possible the shared test proxy service may be a solution here.


dependencies: bump several invenio modules

Coverage remained the same at 0.0% when pulling edbd35cb93551ba81fa6fe8099d028e7013497d2 on ppanero:v0.13.0 into 382a5d6968df425b639bb7a2c185ca3d298ae64f on inveniosoftware:master.


cursor moving to top of query when switching queries

Issue Type: Bug
I am working on row 2000+ on Quesry A and then I switch to also open Query B.  When I return to Query A, I am no longer at row 2000+, instead I am at row 1.
This happens with all queries open.  i.e. I miss the row I am working as soon as I temporarily switch to other queries.
Very annoying.
please help.
thanks,
Andreu
SQL Operations Studio version: sqlops 0.30.6 (df7e3ec7d52a82febee5b597a1019cabf021b883, 2018-06-19T21:45:43.589Z)
OS version: Windows_NT x64 10.0.15063

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-6300U CPU @ 2.40GHz (4 x 2496)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
7.93GB (2.56GB free)

Process Argv
C:\Program Files\SQL Operations Studio\sqlops.exe

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

server-report
Mic
0.1.2

@andreusg23 we're tracking this issue in https://github.com/Microsoft/sqlopsstudio/issues/114.  The basic issue is that the shell tears down the DOM elements for a page when it's switched to the background and then rebuilds the page when it's brought back to the foreground.  The scroll state isn't correctly being reapplied when the page is rebuilt.  We plan to fix in a future release.  I'll close this as a duplicate.


Existing accounts cannot be merged when the email address obtained from LDAP has uppercase characters.

I've been asked to integrate Wekan with our company's Active Directory. I'm unable to merge existing users.
The error I am getting is
wekan_1    | [DEBUG] Identifying user with: objectGUID
wekan_1    | [INFO] Querying user
wekan_1    | [DEBUG] userQuery {
wekan_1    |   "services.ldap.id": "da4a9f103466e34baa9e40bd34280c29"
wekan_1    | }
wekan_1    | [DEBUG] userQuery {
wekan_1    |   "_id": "last.f",
wekan_1    |   "emails.0.address"<EMAIL_ADDRESS>wekan_1    | }
wekan_1    | [INFO] No user exists with username "last.f"
wekan_1    | [DEBUG] userQuery {
wekan_1    |   "emails.0.address"<EMAIL_ADDRESS>wekan_1    | }
wekan_1    | [INFO] User does not exist, creating "last.f"
wekan_1    | [DEBUG] Identifying user with: objectGUID
wekan_1    | [DEBUG] Identifying user with: objectGUID
wekan_1    | [DEBUG] New user data {
wekan_1    |   "username": "last.f",
wekan_1    |   "email"<EMAIL_ADDRESS>wekan_1    | }
wekan_1    | [ERROR] Error creating user {
wekan_1    |   "isClientSafe": true,
wekan_1    |   "error": 403,
wekan_1    |   "reason": "Email already exists.",
wekan_1    |   "message": "Email already exists. [403]",
wekan_1    |   "errorType": "Meteor.Error"
wekan_1    | }

The user names are in the format <last>.<first initial>, and the email is <first initial>.<last>. The email addresses stored in Active Directory use uppercase letters for the first initial and the first letter of the last name. It turned out that this is an issue.
When non-LDAP users are created in Wekan, it appears the email address is converted to lowercase. When merging an LDAP account with an existing account, the email address is used to match the accounts. However, the email address retrieved from Active Directory is not converted to lowercase before calling Meteor.users.findOne. Since the match failed, Wekan attempts to create a new user, using the lowercase version of the email address, which fails because the email already exists.
I think all that's needed to fix this issue is to convert the email address to lowercase when creating the query that gets fed into Meteor.users.findOne.
To confirm this was the issue, I manually modified the database entry for an existing user's email address to make it match the case of the Active Directory entry. After doing that I was able to successfully login using the LDAP credentials, and the logs indicated the accounts were merged. This will be my workaround until the actual issue is fixed.
While tracing this issue I came across a very minor secondary issue. The log message log_info('No user exists with username', username, '- attempting to find by e-mail address instead'); is not fully written. The third argument, starting with - attempting..., is not written. That argument to log_info is lost since the underlying log function only accepts two arguments in addition to level.
My day is done, but I'll try to set up a development environment and fix the issues tomorrow, unless someone wants to beat me to it.

@njpaul
You are welcome to send pull request. Some development info is here https://github.com/wekan/wekan/issues/3071#issuecomment-622300835

Some problems: https://github.com/wekan/ldap/pull/82#issuecomment-2192080410


Improve sp_sproc_columns performance

Description
Previously, sp_sproc_columns had major performance issue due to some convoluted joins and it did not display table-valued parameters. This commit focuses on fixing those issues by reworking the query and allowing sp_sproc_columns to detect Table-valued parameters (TVP). This commit also adds some minor improvements such as:

Showing correct COLUMN_TYPE value for output parameters in stored procedures,
Corrected the data type fields for table-valued-functions
Displays some missing return parameters from procedures
Fix nullable field to correspond with official documentaiton
Fix datatypes being returned (some were using int instead of sys.int and varchar instead of sys.varchar in the internal view)

NOTE: Any previous limitations from this proc is still present, the main focus is on performance and adding TVP support.
Task: BABEL-3555, BABEL-3595
Signed-off-by: Favian (Ian) Samatha<EMAIL_ADDRESS>Issues Resolved

BABEL-3555,
BABEL-3595

Check List

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

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Could you please add the below test template in the PR description?
Test Scenarios Covered

Use case based -
Boundary conditions -
Arbitrary inputs -
Negative test cases -
Minor version upgrade tests -
Major version upgrade tests -
Performance tests -
Tooling impact -
Client tests -


Add force flag to add submodule

Motivation
If an unwitting user mistakenly tries to remove a lib using the rm command, a project can end up in a "broken" state where the respective dependency can be neither installed, removed nor upgraded. See: https://github.com/foundry-rs/foundry/issues/2207
Solution
Use the --force flag of the git submodule add command to enable a user to reinstall a package even after comitting  above mentioned mistake.

--force will allow adding an otherwise ignored submodule path. https://git-scm.com/docs/git-submodule/2.6.7


[BUG] Not able to build KVS plugin in release mode

We are using AWS KVS in our production application to ingest Video from Camera to AWS. Our edge pipeline is built with GStreamer + KVS sink on a Windows 10 machine. Given this context, we are able to build the KVS sink plugin in both debug & release mode but the plugin built in release mode is not working. As it is a production system, release mode will be optimal & convenient. Need support in this regard.

Could you provide more information about what errors are occurring when attempting to use the plugin when it is built in release mode?
During build process there are no errors but for a kvs sink plugin built in release mode, when we run gst-inspect-1.0 kvssink is not showing any plugin information. But in debug mode it shows plugin information.

Could you provide the exact commands used to build and to run the plugin in both debug and release mode?
cmake -DCMAKE_BUILD_TYPE=Release -DBUILD_GSTREAMER_PLUGIN=TRUE ..

@RajPrasanna-Quantiphi ,
We do not use the flag while building kvssink. kvssink releases with our CPP SDK and you can use our official release (Release 3.2.0) to use kvssink.

@disa6302 Thanks for the response. Can you please provide the link to download the kvssink 3.2.0 binary?

Sorry for the delay in getting back. We don't provide binaries for our releases. Suggest you build the plugin from source but without using the release flag.


There is no 'setMarker' method

In the line 83 from 'Project_Code_3_WindowShoppingPart1.html' file, the method infowindow.setMarker(null); not exist.
The correct is set the attribute infowindow.marker = null; to unbound marker object of infowindow.

Thank you!

It's working though. I have no idea how or why :D