Request: Tap on buttons in the background in 'active mode'

When I open the action button and then click other buttons (for example tab bar button) in my page, the first click on the tab bar button would close the action button, and then I need to click  tab bar button again to navigate. Is there a good way to modify this? I don't want to click twice.

that's not possible in the current version @yelynn2015 :/

OK, looking forward to your update @mastermoo

what is the expected behaviour? the action button is in active state, you click once, and the tabbar button is clicked PLUS the action button goes from active to closed state?

because i have no idea how to implement this. i could allow the background to be tappable, but the action button would keep its active state

I try the new one, i stll need to click twice. Do I need to add some code to adjust?

I try the new one, i still need to click twice. Do I need to add some code to adjust? @mastermoo

Yes, there is a new prop you need to add to ActionButton: backgroundTappable={true}
On Mon, Oct 17, 2016 at 5:28 AM +0200, "yelynn2015"<EMAIL_ADDRESS>wrote:
I try the new one, i still need to click twice. Do I need to add some code to adjust? @mastermoo
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Thank you @mastermoo Could I add some code to make the button auto close, I try the props.active, it can not work.


LOG macros with CONFIG_NO_OPTIMIZATIONS & CONFIG_LOG disabled causes a blowout of the stack use

Describe the bug
Functions with LOG macros will use abnormal amounts of stack space if CONFIG_LOG is disabled AND CONFIG_NO_OPTIMIZATIONS=y is set.
Create a simple function (I expanded on hello_world):

Then compile the project with:
CONFIG_NO_OPTIMIZATIONS=y
#CONFIG_LOG=y
CONFIG_MYAPP_LOG_LEVEL_ERR=y
Because CONFIG_LOG is disabled, the function myfoo2() only increments the cnt2. But the compiler generates a function preamble that decides to adjust the stack pointer by 0x0154 bytes or 340 bytes.

If you turn on CONFIG_LOG the SP adjustment is 32 bytes:

And if you use any other optimization level, the code doesn't have these pre/postambles.
Expected behavior
Shouldn't do this. We should be able to reliably use any debug level (within limits).
Impact
CONFIG_NO_OPTIMIZATIONS is practically useless without making huge adjustments to the stack sizes.
Environment (please complete the following information):

OS: Windows
Toolchain (e.g Zephyr SDK, ...)
Commit SHA: 7039f8ce9c2787f37f5a97c3aca1e1541a0d7d07

Additional context
Add any other context that could be relevant to your issue, such as pin setting,
target configuration, ...

It feels like the compiler is getting very confused about the state of things when using the CONFIG_NO_OPTIMIZATIONS (which is -O0). The logging subsystem should null out all the LOG macros if CONFIG_LOG is not set:
#ifdef CONFIG_LOG
...
#else

#define LOG_ERR(...)
#define LOG_WRN(...)
#define LOG_INF(...)
#define LOG_DBG(...)
#define LOG_PRINTK(...)

#endif /* CONFIG_LOG */

This addresses the stack problem but opens up other warnings of unused variables that are setup for consumption by the logging system.

This is also observed on LTS.

The issue is CONFIG_LOG_ALWAYS_RUNTIME. If enabled, it significantly reduces the required amount of stack space.
If logging is enabled, CONFIG_LOG_ALWAYS_RUNTIME defaults to y if CONFIG_NO_OPTIMIZATIONS is set. However, if logging is disabled (CONFIG_LOG=n), CONFIG_LOG_ALWAYS_RUNTIME is not configured by Kconfig, which leads to log_msg.c allocating a lot of stack space.
I was able to fix the issue by adding the following to Kconfig:
if !LOG
config LOG_ALWAYS_RUNTIME
	bool
	default y if NO_OPTIMIZATIONS
endif

Proposed fix: Kconfig should always set CONFIG_LOG_ALWAYS_RUNTIME=y if CONFIG_NO_OPTIMIZATIONS is set, even if CONFIG_LOG=n.


raidboss: fix crystallize+aureole in 2nd EW extreme

Fixes #3817.

This will cause the Crystallize before chakrams to carry through to the follow up Aureole, no? I haven't gone back in to Hydealyn for a few days, but seems like it could be a problem since there are no call outs for chakrams.
This bit
2016.7 "Anthelion" sync / 1[56]:[^:]*:Hydaelyn:6594:/
2026.9 "Mousa's Scorn" sync / 1[56]:[^:]*:Hydaelyn:65C0:/
2032.1 "--middle--" sync / 1[56]:[^:]*:Hydaelyn:6E7A:/
2038.7 "Parhelion" sync / 1[56]:[^:]*:Hydaelyn:65B0:/
2039.0 "Beacon 1" #sync / 1[56]:[^:]*:Parhelion:65B2:/
2040.6 "Beacon 2" #sync / 1[56]:[^:]*:Parhelion:65B2:/
2042.2 "Beacon 3" #sync / 1[56]:[^:]*:Parhelion:65B2:/
2047.5 "Crystallize" sync / 1[56]:[^:]*:Hydaelyn:6ED5:/
2055.6 "Subparhelion" sync / 1[56]:[^:]*:Hydaelyn:6C56:/
2057.6 "Crystalline Water III" sync / 1[56]:[^:]*:Hydaelyn:65A2:/
2057.8 "Beacon 1" #sync / 1[56]:[^:]*:Parhelion:65B3:/
2058.7 "--sync--" sync / 1[56]:[^:]*:Hydaelyn:65A1:/
2059.4 "Beacon 2" #sync / 1[56]:[^:]*:Parhelion:65B3:/
2061.0 "Beacon 3" #sync / 1[56]:[^:]*:Parhelion:65B3:/
2065.8 "Aureole/Lateral Aureole" sync / 1[56]:[^:]*:Hydaelyn:(6F11|65C5):/
2074.9 "Equinox" sync / 1[56]:[^:]*:Hydaelyn:65C3:/
2082.1 "--top-middle--" sync / 1[56]:[^:]*:Hydaelyn:65A9:/
2088.6 "Radiant Halo" sync / 1[56]:[^:]*:Hydaelyn:65B4:/

Yes it does, forgot to post the issue (or trying to fix it) but the crystalize > chakrams > aureole is not correct with the corrent version

Whoops, sorry.  Fixed in #3847.  Thanks for the bug reports.


⚠ Bofrost.nl has degraded performance

In da63098, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1297 ms

Resolved: Bofrost.nl performance has improved in 21111af after 2 minutes.


Fix use of bool in extension header

Commit db7d8c2d87a9704f7404398815c9d686b9a099b9 introduced the type bool, which limits C portability.
This commit replaces it with the type specifically defined to represent booleans: GDExtensionBool.
The methods accepting it cast to bool at definition site.

We also have a few cases of int (even prior to this commit), despite the GDNativeInt type existing.
int however is a standard type in C. I guess that's fine?

Good question. I see int, int32_t and int64_t, and GDExtensionInt is a typedef for int64_t. I'm not sure why we have this typedef for int - is it to represent the Variant::INT type?

I'm not sure why we have this typedef for int - is it to represent the Variant::INT type?

GDExtensionInt seems to be used for:

counts/sizes (except when those are int 🙂)
indexes
hashes
value of integer constants (classdb_register_extension_class_integer_constant)

As far as I can tell, the Godot API also consistently uses int64_t for those things.
I guess it exists as a typedef to signal that it represents the GDScript type int.

int on the other hand has only these usages:
typedef void (*GDExtensionPtrBuiltInMethod)(GDExtensionTypePtr p_base, const GDExtensionConstTypePtr *p_args, GDExtensionTypePtr r_return, int p_argument_count);
typedef void (*GDExtensionPtrUtilityFunction)(GDExtensionTypePtr r_return, const GDExtensionConstTypePtr *p_args, int p_argument_count);
int64_t (*worker_thread_pool_add_native_group_task)(GDExtensionObjectPtr p_instance, void (*p_func)(void *, uint32_t), void *p_userdata, int p_elements, int p_tasks, GDExtensionBool p_high_priority, GDExtensionConstStringPtr p_description);

Would probably make sense to replace them with GDExtensionInt, too?

Small request, if this PR passes and there are no objections, could we merge it already?
That would unblock me on the Rust binding 🙂

Yes but there's ~4 hours of macOS CI backlog :|

No worries, it's fine if I have it by tomorrow! Thanks a lot ❤

Thanks!

@akien-mga  @Bromeon this is not the first time we have a rogue int/bool ending up in this header file, do you think we would gain at adding a test to prevent this.
I'm thinking of a small regex-based script, something like:
CMD = "grep -E "\Wbool\W" core/extension/gdextension_interface.h"
if [[ -z "$(CMD)" ]]
then
     echo "Type bool should not be used, use GDExtensionBoolean instead:"
    $(CMD) --color
    exit 1
fi
CMD = "grep -E "\Wint\W" core/extension/gdextension_interface.h"
if [[ -z "$(CMD)" ]]
then
     echo "Type int should not be used, use GDExtensionInt instead:"
    $(CMD) --color
    exit 1
fi

Yes I think that would be useful. We need to sort out the int situation first though.

do you think we would gain at adding a test to prevent this.
I'm thinking of a small regex-based script, something like:

What about running the header through a C compiler?
The root of the problem is that this file is only run through a C++ CI pipeline, while it actually constitutes a C API. This would also catch any other portability hazards, e.g. struct X {}; vs typedef struct {} X;. It wouldn't need to be very involved, simply a Linux gcc with C99 settings or so. Compilation will take less than a second; we could add a dummy .c file on the fly (no need to be versioned):
#include "gdextension_interface.h"
int main(int argc, char** argv) {}


Fixed Price And URL When Filtering With Combinations

Added id_product_attribute To Select-Fields

Edit: I just noticed the product default combination gets ignored if a filter based on attribute groups is active.

Questions
Answers

Description?
When filtering products with combinations, only the default combination was used to determine price and url of the results.

Type?
bug fix / improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
Fixes PrestaShop/Prestashop#15738 and PrestaShop/Prestashop#15164

How to test?
Filter for products with multiple combinations and different prices. Click on a product to navigate to the correct combination.

This change is

I just made changes that restore the default combination behaviour when using attribute filters (mentioned in edit).
Furthermore, it is now possible to display every combination of a product as a separate product. I'm not sure if I should leave this feature in this branch. I will now check why tests are failing.

Some tests need to be updated, because I modified useFiltersAsInitialPopulation . I'll fix and update later this week.

@Hlavtox have you seen this PR? shall we take over it?

@kpodemski I unfortunately wasn't able to find the time. If you have any questions, let me know.

@TheKruspe Implemented in develop branch. :-)


Equation warning

Reference Issues/PRs
#121
What does this implement/fix? Explain your changes.
In some cases some functions would face exceptions when calculating values in specific points in the given range. This PR would solve this issue by raising a warning for this exception and skips those points.

Codecov Report

Merging #125 (58a26dc) into dev (7444b9b) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##               dev      #125   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines          397       409   +12
  Branches        61        63    +2
=========================================
+ Hits           397       409   +12

Impacted Files
Coverage Δ

samila/functions.py
100.00% <100.00%> (ø)

samila/genimage.py
100.00% <100.00%> (ø)

samila/params.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


DO NOT MERGE

Just starting up spinmint of 4.7

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-03d046e1195e1714d.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-03d046e1195e1714d

Spinmint test server destroyed

Setup Test Server label detected. Spinmint test server created if build succeeds (checks pass and no conflicts with base branch).

Spinmint test server created at: http://i-0590f9dffd26a2851.spinmint.com
Test Admin Account: Email<EMAIL_ADDRESS>| Password: user-0
Test User Account: Email<EMAIL_ADDRESS>| Password: user-1
Instance ID: i-0590f9dffd26a2851

Spinmint test server destroyed


newTree.optimize is not a function

I'm submitting a...

ng add @ng-toolkit/universal

newTree.optimize is not a function

In package

[ ] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[ ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

Expected behavior

Minimal reproduction of the problem with instructions

Example repository

What is the motivation / use case for changing the behavior?

Environment

Angular version: 9.0.6

- Node version: XX
- Platform:  Linux

Others:

Please take a look at #788


[feaLib] Generate Adobe feature files from feaLib AST

Emit Adobe feature files from the feaLib AST (Abstract Syntax Tree). @mhosken would like to work on it.

That would be great, thanks!

Does the ft2fea tool in #479 use the feaLib AST?

do we keep this open now that #776 has been merged?
@mhosken in his https://github.com/fonttools/fonttools/pull/776#issue-196626145 mentioned two features which are not implemented yet:

Ranges are expanded in the parser and not contracted in fea generation. This might be worked around by keeping a side copy of the original glyph list specification including ranges.

nameid attributes are often defaulted and expanded in the parser. These are not defaulted out in the fea generation.

Ok, both of the above mentioned issues are fixed now, thanks to Martin. 🎉
Feel free to open new specific issues.


Legendaries only use struggle in the wild

The most current version I have used is 2.10.1 and in that version just like all the other versions I’ve used before I’ve noticed that random Legendaries for some reason can’t use any moves when encountered in the wild. Often when I catch them they actually have 1 move but in the wild encounter they still use struggle. It has also happened that they didn’t have any moves. I sadly don’t know why that happens, but I thought I should let you know. Anyways Thanks a lot for all the work you put into the randomiser! I’ve been having a blast with it.

Can you give me a specific example (Pokemon and Level) I'll see if it's an issue with the movesets or encounter moveset generation.

It happened to every legendary and mythical that I have encountered so far. And the levels didn’t seem to make a difference. For example a Level 17 Moltres that only knew circle throw. A level 2 Raikou with no moves(when I caught a level 42 entei in a test file it also had no moves). The other legendaries all had one move but still used struggle. But I don’t remember which moves. It didn’t even make a difference whether I randomised the moves or not.


account for SignomialEqualities in SPData

@whoburg, ready for review

test gpkit please
Fixing the dependency tests.

test models please

test models please

test models please

tests gpkit please

test gpkit please


Meld interesse fungerer ikke

What happened?
Når man sender inn skjema for å melde interesse fungerer det ikke..

What browsers are you seeing the problem on?
Chrome
What device are you seeing the problem om?
PC
Relevant log output
No response

Fixed:)


[🐞] dropdown doesn't close when tapping outside on mobile

Which package is affected?
Headless Kit
Describe the bug
Opening dropdown on mobile, tapping outside, nothing happens. I expected it to close.
Reproduction
qwikui.com
Steps to reproduce
No response
System Info
Can't get it from my phone 😅

Additional Information
No response

@maiieul, can you confirm if this is still an issue? I can't repeat in my android phone.

Yup it does seem to work now. Thanks @cwoolum !

So just an issue on an iphone 6s I have. Probably a version of IOS where the dropdown doesn't work.

weird indeed
@maiieul do you have any idea what's causing it?
does it work on other newer devices?
if we can't simulate it on newer devices or don't have a potential lead to investigate, I suggest let's close it until more people will complain and then we'll re-open and investigate more
WDYT?


namedtuple Pylance error Variable not allowed in type expression

I have the following code and i'm not sure how best to resolve the error:
WifiConfig = namedtuple('WifiConfig', ('ssid', 'password'))

def read_wifi_config() -> WifiConfig | None:
  try:
    with open(config.WIFI_FILE, 'r') as f:
      data = json.load(f)
      f.close()

    return WifiConfig(
        ssid=data['ssid'],
        password=data.get('password'),
    )
  except:
    return None

Is this a problem with the stubs or a limitation of trying to use Pylance with micropython?

Hi , thanks for reporting.
I am already have a todo on another issue with NamedTuples.
A QA test has been created : https://github.com/Josverl/micropython-stubs/blob/main/tests/quality_tests/feat_stdlib/check_collections/check_namedtuple.py but I've not had time to  solve this yet.
Im not sure yet where the problem lies:

in the micropython-stdlib-stubs
or in the generated stubs based on the documentation.

The snippet checks out on the pyright playground
Does it run OK in Micropython ?
on what port / board ?

It runs just fine on the device, its only showing linting errors when the stubs are installed.

I think I found the cause of the problem
In the generation of the stubs I cast the return to typing_extensions.NamedTuple rather than 'stdlib.collections.namedtuple'
I assumed the were equal, but apparently they are not
# .../collections.pyi
from stdlib.collections import namedtuple as stdlib_namedtuple # <--- Add this
# from typing_extensions import NamedTuple as stdlib_NamedTuple # not this

def namedtuple(name, fields) -> stdlib_namedtuple: # <-- change this
   ...

I need some time to integrate this and generate new stubs , but for now you could manually update the stub.

# .../collections.pyi
from stdlib.collections import namedtuple as stdlib_namedtuple # <--- Add this
# from typing_extensions import NamedTuple as stdlib_NamedTuple # not this

def namedtuple(name, fields) -> stdlib_namedtuple: # <-- change this
   ...

This works for me. Thanks!

as I side note : I think you should add a guard clause even with the above fix,
...
    if not isinstance(data, dict):
        return None
    return WifiConfig(
        ssid=data['ssid'],
        password=data.get('password'),
    )
  except:
    return None

I'm not getting that linting error, but good point. Thanks

I could not get consistent results - I may need to update micropython-stdlib to get consistent results across the board
Happy the workaround works on your system -
Which Python version / OS  are you running ?

Python 3.11
MacOS

After some more testing, it seems to work fine unmodified with venv, but if I install via  --target ./typings --no-user, thats when I get the error.
Clicking on the collections or namedtuple import and selecting Go to Definition or Go to Declaration takes me to the source node of my system python instead of the one definition in .venv/lib/python3.11/site-packages. The same happens with asyncio and json, but not with sys or time.

Ok, after experimenting more I think your solution didn't work, instead I think what was happening was it was defaulting back to my installed python types

@North101 , I came to a similar conclusion.
My current understanding is that the type checkers have some "special case handling" to make type-checking work for namedtuples. : https://github.com/python/typeshed/blob/ba7bd9f98ad9bba957dfb4175eac5448f6273575/stdlib/collections/init.pyi#L38
I think that means that namedtuples typechecking works only for that exact module:  stdlib_stubs.collections.nametuple
and not for the level up where the micropython definition lives.
I'm currently working to test if i can automate updates to the typeshed stdlib stubs, and if that solves more than it may break.

@North101 Santa delivered a prensent-stub for you :-)

Thanks! It works


docs(readme): add info about the memdb storage engine

Codecov Report
Base: 91.39% // Head: 91.39% // No change to project coverage :thumbsup:

Coverage data is based on head (9ef101d) compared to base (5884e8d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #27   +/-   ##
=======================================
  Coverage   91.39%   91.39%
=======================================
  Files          33       33
  Lines        2358     2358
=======================================
  Hits         2155     2155
  Misses        139      139
  Partials       64       64

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


[Semantic API] Resolve cyclic union type in TypeParamResolver

Description:

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Since the TypeParamResolver is used when getting the lang lib methods of a type symbol and, it is highly unlikely that a cyclic union type dependency would be present in langlibs, deprioritizing this issue.


ask users on the profile page to edit their profile (if they didn't so far)

dependency: #402
would be great to connect this with community upgrade status in sense of we ask the users to update their profile when they reached community status

Do you think that this is still relevant? Let's discuss it the next time we meet, whether it is still relevant. I guess we will do this anyway when we see a need that this is necessary. Thus, I'll close the issue.


Garbage collect actors when --ponynoblock is in use

Prior to this commit, actors wouldn't be garbage collected when using
--ponynoblock. With this change, any actors that aren't part of a cycle
will be garbage collected if they can no longer do work.
Before this change, using --ponynoblock with an application that
created transient actors would result in never ending memory growth as
the actors would never be gc'd. You can see an example of this by taking
a Pony program that listens for TCP connections and do a repeating cycle
of open/close. None of the created TCPConnection actors would be
garbage collected.
Closes #2306

One OSX CI build failed on the process/STDIN-WriteBuf test. I've restarted the job.


less than combined with typecast causes bad syntax highlighting

Being fairly new and ignorant in the ways of Atom, I don't know if this belongs here or Atom, but assume somebody would correct me if I'm putting this in the wrong place.
I have a code file where the bottom half of it is all yellow for some reason.  It appears to be triggered by the combination of less-than and a typecast inside a for loop.  By removing either the typecast or the less than, the syntax highlighting looks great.  Attached is an image of the problem as well.
export class Foo {
    public doStuff(item: Bar) {
        // for some reason, we know that item is a Bar2
        for (var i = 0; i < (<Bar2>item).getStuff(); i++) {
            // stuff to do
        }
    }
}

export class Bar {

}

export class Bar2 extends Bar {
    public getStuff() {
        return 2;
    }
}

Belongs here. Bug in the grammar : https://github.com/TypeStrong/atom-typescript/blob/master/grammars/typescript.cson
up for grabs :rose:

@mjohnsonengr was this a tsx file?

was this a tsx file?

Answered my own question. Ts file

any effort in grammar should go to https://github.com/TypeStrong/atom-typescript/issues/675 :rose:

Should work with release https://github.com/TypeStrong/atom-typescript/releases/tag/v8.0.0 :rose: If not raise issue at https://github.com/Microsoft/TypeScript-TmLanguage/issues and link us :rose:


App version

Hey,
After the latest update, the versioning hasn't changed, it still shows 1.4.0 in the Android apps settings.
Thank you.

Hey, After the latest update, the versioning hasn't changed, it still shows 1.4.0 in the Android apps settings. Thank you.

Check the latest release now.


[BUG] Can't download games,always show "Downloading metadata"

Describe the bug
When I try to download any games it show "Downloading metadata" and dont download the game. I try to use Vpn and this is dont working.
Steps to Reproduce
Click on "Download variants" and choose any variant.
Expected behavior
The game will download.
Screenshots

Operating System
Windows 11
Hydra Version
2.0.1
Additional Information
Other games in library i download from torrent trackers

У меня такая же проблема

We are working on a solution to this problem, it is taking us a while as it requires a complete refactoring of the download system, when a new version coming out, please report if the problem persists

We are working on a solution to this problem, it is taking us a while as it requires a complete refactoring of the download system, when a new version coming out, please report if the problem persists

i too have this problem

Install the new v2.0.2 version. This bug has been fixed

fixed on #722


[stable/newrelic-infrastructure] Update image version

Signed-off-by: Alejandro Do Nascimento<EMAIL_ADDRESS>What this PR does / why we need it:
This PR updates the container image tag of the newrelic infrastructure chart.
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test

/lgtm


I'm going to update the Demo presets on the main repo. Let me know when you are finished here and I'll migrate the presets

I did this already


opae-tools: fix hssi Ethernet tools integration bugs

set bit 31 to 1 to read HSSI Stats  LSB
set bit 31 to 0 to read HSSI Stats  MSB
fix HSSI read / write date CSR  offset
fix bit fields

Signed-off-by: anandaravuri<EMAIL_ADDRESS>

Coverage increased (+0.03%) to 65.795% when pulling ebcc33f932d7e287fab8feda2543cbfe05b6b3f2 on aravuri/fix_hssi into da815b44a361ce03912e3ef8978e73be08b86284 on master.


v3.0.7299.2772 Git-4b03f163 - Entry point Pss*Snapshot() not found in to Kernel32.dll for Win7/Win8

v3.0.7299.2772 Git-4b03f163:
OS Win7 SP1 x64.
Entry point PssQuerySnapshot() not exist in to Kernel32.dll for Win7/Win8:

this functions use in to Git c990c55ed5fd3 "Update svcapi.c", but added in to OS only start up in Windows 8.1 (I find in to MSDN records "Minimum supported client - Windows 8.1 [desktop apps only"). Source for this errors is this diiff:
index b5572f26..053aef05 100644
@@ -1456,22 +1454,50 @@ NTSTATUS PhSvcApiWriteMiniDumpProcess(
         &callbackInfo
         ))
     {
-        if (PssFreeSnapshot_Import() && snapshotHandle)
+        if (snapshotHandle)
         {
-            PssFreeSnapshot_Import()(processHandle, snapshotHandle);
+            PSS_VA_CLONE_INFORMATION processInfo;
+
+            if (PssQuerySnapshot(
+                snapshotHandle,
+                PSS_QUERY_VA_CLONE_INFORMATION,
+                &processInfo,
+                sizeof(PSS_VA_CLONE_INFORMATION)
+                ) == ERROR_SUCCESS)
+            {
+                NtClose(processInfo.VaCloneHandle);
+            }
+
+            if (PssFreeSnapshot_Import() && snapshotHandle)
+            {
+                PssFreeSnapshot_Import()(processHandle, snapshotHandle);
+            }

Other error's I don see before PH can't startup.:)

Ok! Thanks!  I go to testing.

Build and try:

Possible needed additional check for OS version in to ProcessHacker/phsvc/svcapi.c :: 1461 like If OsVversion >= 6.3 then if (PssQuerySnapshot_Import() ( ... ) else skip end :?

All function used PSS_WALK_INFORMATION_CLASS, example PssCaptureSnapshot(), PssFreeSnapshot() and PssQuerySnapshot added to kernel starting in Windows 8.1 (source - MSDN).
System behavior:
If entry point not found Windows 7 display standard error dialog this only OK button and wait for users press it, then button is pressed OS just cancel startup for this program.

v2677 - started success. Thanks!


ordered point cloud format

How to export ordered point cloud format?Thanks

@yuliangxiao, can you provide more information about what you need?

@yuliangxiao please look at whether this function provide what you need already:  k4a_transformation_depth_image_to_point_cloud(). Please read the remarks in this function description that it output an xyz_image which is an organized point cloud. Here is an example of how you can use the API

Hello, this is the format I need。

I don't know how to get the last parameter(z),Thanks

Thank you. I know what to do.


net: lwm2m: Make query buffer large enough to encode all query strings

Make sure query string used by the lwm2m_rd_client is large enough to
encode any query string that can be sent during bootstrap/registration.
As the maximum query string length is related to the endpoint name,
which is limited by CONFIG_LWM2M_RD_CLIENT_ENDPOINT_NAME_MAX_LENGTH,
make the query string corellated to the value of this config.
Fixes #32566

CC @mlasch Feel free to review.


Set up the development environment

As a developer
I need to set up the development environment
So that I can start working on the project
Details and Assumptions
Development environment includes tools and dependencies.
Acceptance Criteria
Given a new development machine
When I follow the instructions
Then I should have all necessary tools and dependencies
And the development environment setup is done.


WIP Skip Travis build for doc-only changes

closes #575

Would it make more sense to look for changes in the doc directory to ensure non-md changes are captured (e.g. images)?

@marun agreed... i'll add the filter accordingly. thanks for pointing that out.

@marun updated, PTAL.

/lgtm
/approve
Thank you!


how to use rtmp-wrapper with AVCaptureSession?

is it possible to "feed" this wrapper with live data from AVCaptureSession?

@haemi did you find any way to do this ?


Add multiple Docker sockets

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
How can I add multiple docker sockets? Currenty I am running Uptime Kuma on VM1, which runs Docker. I can monitor these containers just fine.
Is there a way to add the Docker socket for my VM2 and VM3 to also monitor the containers directly?
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
v1.23.11
💻 Operating System and Arch
Ubuntu 22.04
🌐 Browser
Firefox
🖥 Deployment Environment

Runtime: Docker
Database:  sqlite
Filesystem used to store the database on: ZFS on HDD
number of monitors: too many

Exposing the sockets like this is not something that I would recomend for security reasons.
Please refer to method 2 instead: https://github.com/louislam/uptime-kuma/wiki/How-to-Monitor-Docker-Containers#method-2-tcp---bridge-mode


Sort through devtests

The scripts in devtests (including the archive subfolder) aren't critical for CI, but some are still interesting/useful. Steps:

[ ] Check to see which still run (devtests/run_dev_tests)
[ ] Go through and sort into (a) broken and won't fix, (b) broken and will fix, (c) interesting and useful
[ ] For the interesting/useful ones, add documentation and promote to a worked usage example (maybe leave in that folder, but prefix with example_)
[ ] Make filename prefixes consistent

Low priority; will revisit if the need arises.


FormOptions not work

In  .net core 2.0
I want to upload a file that size maybe 1G or more
I  modify the  FormOptions  like
services.Configure(x =>
{
x.ValueLengthLimit = int.MaxValue;
x.MultipartBodyLengthLimit = int.MaxValue;
x.MultipartHeadersLengthLimit = int.MaxValue;
});
But it is invalid
I still can't upload files over 10M in size

@Eilon

This issue was moved to aspnet/HttpAbstractions#930


Enable the Django Rest Framework built-in documentation feature

Description of change
This enables the Django Rest Framework built-in documentation feature at the URL path /docs.
This is currently only enabled if the ENABLE_API_DOCUMENTATION environment variable is set to True as the documentation is not fully functional as yet.
Currently, a user must log into Django admin prior to accessing the documentation.
Some minor changes had to be made to a few views in order to get the view introspection working. (In the case of the D&B views, they were not really generic views and so have been changed to use APIView instead of GenericAPIView.)
The next version of DRF moves more towards OpenAPI-based schema generation and these changes will help us in switching to that when we upgrade.
I'm planning to fix a few things in further PRs, and then will probably enable the docs by default for (at least) local and the dev environments. Will also add some docs to make it clear what needs to be done for new endpoints.
Screenshots
These show the docs functionality in DRF 3.9 (this may change when we upgrade to 3.10).

Checklist

[x] Has a new newsfragment been created? Check changelog/README.rst for instructions
[ ] Have any relevant search models been updated?
[ ] Have any relevant fixtures (fixtures/test_data.yaml) been updated?
[ ] Have any relevant select-/prefetch-related field lists in the views and search apps been updated?
[ ] Has the admin site been updated (for new models, fields etc.)?
[ ] Has the README been updated (if needed)?

Also a question - with the OpenAPI changes proposed, will the swagger definition go as far as explaining field validation criteria..?

I expect not, but it should hopefully still be possible to add a description (with general comments) for each view.
You can have a look at it looks like in DRF 3.10 via the docs on their master branch:

https://github.com/encode/django-rest-framework/blob/master/docs/topics/documenting-your-api.md
https://github.com/encode/django-rest-framework/blob/master/docs/api-guide/schemas.md

They seem to be in the process of releasing DRF 3.10.0 right now so it shouldn't be far off at all (but we will need an update to django-filter as well for https://github.com/carltongibson/django-filter/pull/1086).


ImmutableJS Still required

looks like immutablejs is still required for me, using 3.0.1. Based on commit history and the build/ folder installed via npm, it looks as though maybe a new version was not published to npm after updating source.

Yes, v4.0.0 has just been released :tada:


Issue with thomasloven/lovelace-layout-card in Grid layout

Checklist

[X] I updated the card to the latest version available
[X] I cleared the cache of my browser
[X] I verified that I'm really running the lastest version in my browser console
[X] I checked if there is another issue opened with the same problem

Describe the bug
When using thomasloven/lovelace-layout-card in grid layout the following parameters are added to the lovelace cards to position it in the defined grid:
layout:
  grid-area: [position]

When using this with ApexCharts an error is thrown.
There appears to be some conversation around layout-card here: https://github.com/RomRider/apexcharts-card/issues/121 but don't think this is the same issue
Version of the card
Version: 1.8.2
To Reproduce
This is the configuration I used:
Card Configuration:
      - type: 'custom:apexcharts-card'
        header:
          show: true
          title: Temperature
          show_states: true
          colorize_states: true
        all_series_config:
          show:
            legend_value: false
        series:
          - entity: sensor.lounge_temperature
          - entity: sensor.landing_temperature
          - entity: sensor.roof_temperature
        layout:
          grid-area: p22

View Configuration:
  - title: People Places
    path: people-places
    background: center / cover no-repeat url("/local/blue-back2.jpg") fixed
    icon: 'mdi:motion-sensor'
    panel: false
    type: 'custom:grid-layout'
    layout:
      grid-template-columns: 20px 1fr 20px 1fr 20px 1fr 20px
      grid-template-rows: auto
      grid-template-areas: |
        ". p11 . p12 . p13 ."
        ". p21 . p22 . p23 ."
      mediaquery:
        '(max-width: 800px)':
          grid-template-columns: 95%
          grid-template-areas: |
            "p11"
            "p12"
            "p13"
            "p21"
            "p22"
            "p23"
    badges: []
    cards:

Screenshots

Expected behavior
To be able to place an apexchart-card in my chosen layout-card grid location
Desktop (please complete the following information):

Chrome
Version 89.0.4389.90 (Official Build) (64-bit)

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Appreciate this is probably not an apexchard-card specific issue and a result of using layout-card, specifically the Grid option. Raising it see if there is anything could be done but happily accept an answer that this isn't in scope.

Just seen this on the layout-card page so maybe addressed by Thomas
https://github.com/thomasloven/lovelace-layout-card#due-to-name-collisions-with-other-popuplar-lovelace-cards-the-parameter-layout-will-likely-very-soon-be-called-something-else

This will be fixed in layout-card.

Closed as per Thomas comment


add: link to pipenv action

This pull request is ready for review. Could you confirm this?

@peaceiris your changes have been merged, thanks for your contribution 👍


Plane issues when flying.

Expected Behaviour.
When aboard the plane (server) the chat should work normally and terrain should load in and when it lands the doors should open normally and allow people to exit.
Actual Behaviour.
When aboard the plane (server) the chat stops working for the time any player is on it, terrain doesn't load in at all and when the aircraft lands then nothing is loaded in and doors don't open so people can't get off. Shift to dismount also doesn't work.
Reproduction Steps.
Steps to reproduce the behaviour:

Create a working flight route system with the airports at least 5 thousand blocks apart or more.
Board the plane flying to the other destination.
The chat won't work so you can't message other players and all issues start here after about 15 to 30 seconds aboard the aircraft.

Crash Report.
Nothing crashed so I can't provide a log sorry.
Screenshots.
Sorry but I can't provide any also the screenshots won't help explain.
Environment.

Minecraft Version: FORGE 1.20.1
Installed Mods:
SCREENSHOT OF MODS FOLDER PROVIDED

Additional Information.
I am on an exaroton server. 4gb ram, problem also happens with more ram. Happens on fabric 1.19.2 server with different mods so I don't think its a mod compatibility issue (could be though). Thank you for reading this report - mattyx2009x (Minnow). For any extra information contact me via Discord, username: mattyx2009x

Extra information.
I have added 16gb of RAM to the server and 8 CPU cores and chat does work but when I land the terrain is still not loaded. I press shift to dismount and I end up to the airport I started at. The camera is also very buggy and I can't move it nor can anyone else. Thank you for reading this extra message, remember you can always contact me for more information and screenshots via Discord (username: mattyx2009x).

Another bug I found is that PIDs don't work for planes anymore.

Having the same issue with the PIDS on my World. And my guess on the Doors not opening is Server Lag

this issue seems to be something to do with the device the files for the server are stored on, i had the same thing happen on kingsburgh when i was using an sd card for storage. my guess is the server can't read the chunks fast enough

yeah its probably a server issue ill try to test it on a normal world to see if it persists

飞机太快了，不是所有服务器的区块加载都能跟上。我的也是勉强能在飞行中看到地面一小块的。
如果你在飞行途中不再显示地面，并且诸如切换模式、发送聊天等服务器互动失灵，那你可能会掉线。我遇到过https://www.mtrbbs.top/thread-5314-1-1.html
实际上你掉线后重新回来发现你根本不在终点站，可能会在路途中

https://bugs.mojang.com/secure/attachment/276318/mcbug20200326.mp4
請問你們遇到的是類似於以上影片的狀況嗎
is your issue like shown in video?

https://bugs.mojang.com/secure/attachment/276318/mcbug20200326.mp4 請問你們遇到的是類似於以上影片的狀況嗎 is your issue like shown in video?

Pertty like, but I can reproduced it on the MTR's train with mutiplayer server. During this bug occur, I found a lot of moved too quickly on my server log. And after I add this mod, problem solved. That not looks like chunky load too slow, it seems like some anti-cheat feature. I'm not pretty sure about if this solution works on higher speed (i'm 80km/h now), but it works for me

For now, you can use the Carpet mod and enable the antiCheatDisabled feature. I will work on improvements with high speed travel.


Support for most popular databases/data warehouses

This ticket seems very old and vague in the current context.


Set/Test-DbaDbCompression fix for issue #3698

Type of Change

[X] Bug fix (non-breaking change, fixes #3698)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose

Fix issue #3698 where function is using columns not available in older version.
Also added logic to avoid node/edge tables as they can't be compressed.
Approach

Removed logic from main query, added logic to determine sql version and remove applicable objects depending on that version.
Screenshots

ur the best! thank you. will merge and release shortly


ft: ZENKO-404 lifecycle service account

Add support for service account 'service-lifecycle' in addition to
existing 'service-replication'. ACL checks are shunted in both cases,
we may implement more fine-grained access control later.

PR has been updated. Reviewers, please be cautious.

PR has been updated. Reviewers, please be cautious.

PR has been updated. Reviewers, please be cautious.

PR has been updated. Reviewers, please be cautious.

Merging this for now as we seem to be having timeout/service issues with Azure Storage.


added script to pull transformers and upload to s3

I get when i test this: download_transformers:22: bad substitution


Add confirm Password text box

What is the change?
Adding confirm password text box
Related issue?
close: #270
Checklist:
Before you create this PR, confirm all the requirements listed below by checking the checkboxes [x]:

[x] Have you followed the Contribution Guidelines while contributing.
[x] Have you checked there aren't other open Pull Requests for the same update/change?
[] Have you made corresponding changes to the documentation?
[x] Your submission doesn't break any existing feature.
[x] Have you tested the code before submission?

Screenshots or Video:

Open a new PR from a new branch.
Closing this PR for now, reopen if required.

/reviewed


Add arrayParameters to RequestType protocol

Addresses #124

Thank you for the PR.
I'm considering to change the type of parameters to AnyObject in the future version, but it is breaking change. To keep compatibility in current major version, adding secondary property seems to be a good solution.
IMO, var objectParameters: AnyObject is better than var arrayParameters: [JSON] because NSJSONSerialization receives AnyObject and [JSON] cannot express nested arrays.

Could you merge master to fix CI?

Rebased to master.
Also changed arrayParameters to objectParameters because you're so right ! 😁

Looks like OHHTTPStubs has carthage build issue.
This little tweak solved the Build failure on my machine, but I'm not sure it's the right solution for it. (after carthage build and snapshot reset_simulator)
diff --git a/OHHTTPStubs/OHHTTPStubs.xcodeproj/xcshareddata/xcschemes/OHHTTPStubs tvOS Framework.xcscheme b/OHHTTPStubs/OHHTTPStubs.xcodeproj/xcshareddata/xcschemes/OHHTTPStubs tvOS Framework.xcscheme
index cde7ba1..e75d1e0 100644
--- a/OHHTTPStubs/OHHTTPStubs.xcodeproj/xcshareddata/xcschemes/OHHTTPStubs tvOS Framework.xcscheme
+++ b/OHHTTPStubs/OHHTTPStubs.xcodeproj/xcshareddata/xcschemes/OHHTTPStubs tvOS Framework.xcscheme
@@ -3,8 +3,8 @@
    LastUpgradeVersion = "0710"
    version = "1.3">
    <BuildAction
-      parallelizeBuildables = "YES"
-      buildImplicitDependencies = "YES">
+      parallelizeBuildables = "NO"
+      buildImplicitDependencies = "NO">

Thanks for pointing out.
Fixed missing URL query support in 07d4d76 .

Nice :sparkles:
I'll merge this PR into master after fixing CI issue on a merge branch.


Add ability to select attributes

Fixes issue #23 (which was closed without actually being fixed).
AngleSharp's IAttr does not implement INode (due to current DOM spec not classifying them as such) so I made a wrapper that implements both IAttr and INode. This allows attributes to be returned from SelectNodes and SelectSingleNode. This changes the behavior of HtmlDocumentNavigator.CurrentNode to construct a wrapper if _attrIndex != -1.
Also added a test to confirm that it can return "nodes" that are instances of IAttr.
This is technically a breaking change if people were relying on the old incorrect behavior of returning the element that owns the attribute instead of the selected attribute itself.

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

I screwed up and somehow amended the previous commit instead of adding a new one. Forgive me. I will close this, fix it, and reopen a proper pull request.


[fastlane][rocketchat] Include integration with Rocket.Chat

This adds Rocket.Chat support. (http://rocket.chat)
Rocket.Chat is a open source web chat platform.

The Ultimate Open Source WebChat Platform
Have your own Slack like online chat, built with Meteor

Hey @thiagofelix,
Thanks for your fastlane pull request, this looks great!
We recently introduced the new fastlane plugin system that allows you to distribute and maintain your own plugins, and still be exposed to fastlane users.
We'd love this new action to be a fastlane plugin so people who want to use it, can easily add it to their project.
Please let us know how the integration goes, and if you need any help getting up and running 🚀


adds html code but mostly css style

A new goal for this week is to start the day with a pull and finish with a push. Another goal is to remember to work in branches.


UnboundLocalError 'strategy' referenced before assignment during SSD mobilenet FPN evaulation

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

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): no custom code. Using stock example in tensorflow
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): windows 10
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: -
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below): v2.3.0-rc2-23-gb36436b087
Python version: 3.6
Bazel version (if compiling from source): -
GCC/Compiler version (if compiling from source): -
CUDA/cuDNN version: 10.1
GPU model and memory: RTX 2080, 16GB

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
I used this python model_main_tf2.py --model_dir=models\my_mobilenet640v2 --pipeline_config_path=models\my_mobilenet640v2\pipeline.config --checkpoint_dir=models\my_mobilenet640v2 --wait_interval=60 --eval_timeout=60 in stock model_main_tf2.py to evaluate a new trained object detection model using my own custom set of image data while training on top of a pre-trained model.
It throw a error 'UnboundLocalerror: local variable 'strategy' referenced before assignment.
pastebin of error: https://pastebin.com/rjfY6dhp
Describe the expected behavior
strategy scope should not have have error
Will ignoring this error cause any potential problems when exporting to tflite using export_tflite_ssd_graph.py ?
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

no custom code. Using stock example in tensorflow

@Ethylbenzol,
Could you please provide a minimal code snippet to reproduce the issue reported here or share the link of the example.
Also, please update TensorFlow to the latest stable version v2.4.1 and check if you are facing the same error. Thanks!

Hey man thanks for the follow up.

Could you please provide a minimal code snippet to reproduce the issue reported here or share the link of the example.
I used the exact code here at
https://github.com/tensorflow/models/blob/master/research/object_detection/model_main_tf2.py

I did however modified it a bit using tensorflow GPU ram control default code, and reduce the wait time from 300 to 120. You can read my code at https://pastebin.com/c3X3dvGY

please update TensorFlow to the latest stable version v2.4.1 and check if you are facing the same error.

I dont think using default code would affect anything. I will make an upgrade to to TF 2.4.1 and make a follow up on your recommendation to see if anything fixes it.

I'm actually using tensorflow object detection and that officially says it is still in TF2.2/ 2,3? Will there be issues when upgrading the tensorflow package to 2.4.1?

Updated to 2.4.1. Error is resolved

@Ethylbenzol,
Thank you for the update. Closing the issue as it is resolved. Please feel free to re-open if necessary.


Ionic2 Navigation Tabs are not resizing correctly when deployed to iPhone6

When deploying more than 3 tabs to an actual device (iPhone 6), the tabs are not resizing to fit the screen correctly.
Example App (Ionic2):
https://github.com/bwalker72956/testIonic2TabApp
Steps to reproduce:

I started with a sample app and added 2 additional tabs to navigation.
I tested in the browser and emulator.  Everything worked fine.
When deployed to an actual device directly from xCode, the tabs run off the screen in Portrait mode.

I can confirm this issue. Does not seem to happen in the iphone 6 simulator, but i can reproduce on my iphone 5.

Is this the same problem as #6056 ?

This is the same issue.

On Apr 24, 2016, at 2:39 AM, Ben<EMAIL_ADDRESS>wrote:
Is this the same problem as #6056 ?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

Hey all, this should be fixed in the latest nightly and will hit beta when beta.7 hits. To install Ionic 2 nightly you just run npm install ionic-angular@nightly in your project and you should be ready to go!

Just tested it out and it works.  Thank you.

@bwalker72956 Awesome! Nice to hear. Thanks for using Ionic!

This is still the issue persist on 3.20.0 running the application on ipad devices where ios - max-width of tab = 240 px so that it cuts down from edges so what can be possible solution for it ? Android working fine
Thanks for any suggestions.


Remove version: Codemao.TurtleEditor version 1.7.6

This pull request is the first part of pull request #198314.

close #198314

URL: https://static.codemao.cn/wood/client/production/海龟编辑器x32-1.7.6.exe
OK
(Automated message - build 923)


Fix handling the error when parsing the http request timeout for processing the http route rule

Description:
Handling the error when parsing the http request timeout for processing the http route rule. The timeout could be empty according to the definition of HTTPRouteTimeouts in gateway-api/apis/v1/httproute_types.go.

please fix dco

please fix dco

Hi @zirain done! ^_^

/retest

why are we trying to set the timeout if timeout is invalid, won't the default value kick in anyway ? imo if there is a error parsing the timeout, we should flag it in the Status


Crisp Chat

Has anyone succesfully added a  customer service chatbox? i am trying to get crisp working, but no luck so far.

Not being funny , but I really don't understand the logic of the build, and whether Introduction is a typical hugo build or not. E.G. I adapted the footer text, primarily to create a new target link that opens independently, but my website reverts to the original footer text. However when I search my file structure for "contributeurs open-source" as appears on my live website, this text does not appear, and only appears in the unabridged themes/introduction/i118/fr.toml file.
I am just an old school bodger who used to tinker with WP may years back. Maybe I am out of place here, and need to revert to a more simple site builder, I don't know. Even basic science such as locating the header eludes me.

Ah! Thanks Victoria. I inadvertently erased that line when modifying the language settings. Thank you for your advice, it is very helpful. I simply reformulated the footer that is all, with a link that opens in a new browser. The link comes up blue actually.
Text   = 'Thanks to Victoria @Introduction Theme.'                      #

Great! I’m glad it worked out 😁


HTML being cut off in PDF

I am rendering a very simple invoice and it is cutting off HTML in the middle of the page when I set page top and bottom margins. When the margin is removed it seems to sort itself out. Can send html it's trying to parse if that would help

On latest nuget package of tmlRenderer.PdfSharp
broken htmlrenderer.txt
Attached test method, most noticably the table headers wont appear. Any help is hugely appreciated,
Thanks.

Fixed.
See example.
Notice that it requires prerelease version of PdfSharp

thank you! I will upgrade to pre-release

Are there still plans to update the nuget package to include these changes?
Thanks,
Pam

Where can I get that prelelese version that you talking about?
Thanks

Hey guys. I just uploaded a pre-release version of the packages to nuget (*-beta1). That is the most up to date code and includes the fix gyfke made for page breaks.
Cheers!

PawelMaj thank you so much!!  I'm not seeing it when I search within NuGet though, even though I have "Include Prerelease" selected.

https://www.nuget.org/packages/HtmlRenderer.Core/1.5.1-beta1

Thanks very much, I wasn't familiar with the package manager console.
One other question - is this compatible with the HtmlRenderer.PdfSharp library?  When I try to generate a PDF in my application after updating the core to the beta release, it's blank except for the footer and one horizontal rule.  Interestingly, it has that correct spacing/number of pages for the text that should be there.
Thanks,
Pam

@pdreid I am currently using it in a large project specifically with PdfSharp. Make sure you have the right version of PdfSharp installed. You need the beta version.

Hello, I'm getting the same with the prerelease version, simple 's with LOL text, margins top-bottom 60, left-right 10, pageSize Letter and Orientation Landscape:

Any ideas?

@darkguy2008 Hey! I have recently decided to join this project and started looking into the page-breaking code in specific. That being said, I think the words will break a page correctly if the immediate parent element has that attribute on it and if the parent element is smaller than the size of the page.
Should work
Hello
Should fail
Hello
If you only really care about making sure words are not split in half you can change the following code:
CssRect.cs ln307
From:
if (!box.IsFixed && box.PageBreakInside == CssConstants.Avoid)
{
word.BreakPage();
}
To:
if (!box.IsFixed)
{
word.BreakPage();
}
If I am correct (I may be wrong, I have only started to get into this code) this change will make sure that no words will ever be split on a page break. I am a bit busy right now, but if I find time I will try to push these changes in and get a new version into nuget.

Hello @PawelMaj ! Thanks for your answer!. While I would love to help you in this endeavor, the library's code is huge for my timeframe and I'm also surprised on how wee you able to find that out. I made the change and I can say it's working nicely, at least for one-line paragraphs. Now I'm working with a table using rowspans and the split is happening again :( see!

I tried to look at the code in that function but it's hard to understand for me, what do you think it could be? thanks again for your time! it would be great if you can make a nuget package out of that fix too :)

+1

@darkguy2008 The table code uses different logic unfortunately....

+1

This is still having problems. I'm using this for a production application and this is the last remaining bug. Please fix this.

@KthProg I solved it by using WxHTMLToPDF (another project) by just making an HTML and rendering it through Webkit... it might be unrelated but it solved my problem, this renderer has a lot of issues regarding line breaks and so on, and the code is too convoluted for me to understand :(

@darkguy2008 How did you do "rendering through WebKit"?

@KthProg that project does it, there's a NuGet package called TuesPechkin ( https://www.nuget.org/packages/TuesPechkin/ ) which is a wrapper for that project, you just make an HTML and supply it to the wrapper and it works kinda like PhantomJS, but with PDF output built-in.

@darkguy2008 I'll take a look thank you!

So I can implement the quick fix I mentioned earlier. I will not push it to a Nuget package just yet because I am afraid that the change may have some side affects. This quick fix would be changing the code:
From:
if (!box.IsFixed && box.PageBreakInside == CssConstants.Avoid)
{
word.BreakPage();
}
To:
if (!box.IsFixed)
{
word.BreakPage();
}
From what I have seen, this will make sure that each INDIVIDUAL word/picture will not be cut in half on page breaks.  This will introduce a possible smaller issue. If the words in a single line have different line heights, here is a possibility of some words being placed on the next page while others stay on the previous page.
Now I have thought of doing a more complex fix to this. Unfortunately the entire process seems to be using recursion to determine where to place each small element(aka word/image). I would have to redo a large/important part of the code base. I am not comfortable doing this, since I did not write any of the code, nor do I have time to do that. Thus, I will just make a commit with my simple fix for now and hope it does not cause other problems. I will try to do this tonight...

@PawelMaj That fix would do the trick for me. If you add this fix how would I pull it down/build from it? (I know I should know this, but I use TFS at work...)

@PawelMaj I made the change you suggested and did my own build. When I used the updated DLLs my problem was fixed. Thanks 👍

Yes, I can confirm that the above change fixes this problem for my simple test case as well.

@PawelMaj  can this fix be merged to master and pushed to nuget?

@PawelMaj As mentioned, could you possibly merge your fix to master and update the nuget package? This is the last of the issues with my PDF generation project, would be good to see it fixed.

Hey guys, just to let all solution more clear following the step-by-step below:
1 -  Add to your project  https://www.nuget.org/packages/HtmlRenderer.Core/1.5.1-beta1
2 -  Add to your project  https://www.nuget.org/packages/HtmlRenderer.PdfSharp/1.5.1-beta1
3 - After that, if you wanna to break table, td, h3 .. just put td {page-break-inside: avoid; } will work.
4 - if you wanna to break imgs just put the tag img inside a span and in span add page-break-inside: avoid;
Cheers Guys.

Hi guys,
I'm new to HTML-Renderer but its use as been until now simple et straightforward. I'm still having one issue, the page break one... I've updated to beta1 both core and pdfsharp libs, added the page-break-inside css rule too, but I'm still having lines appearing cut between 2 pages. Has anyone any idea why ? It appears inside a  for example or an h3.
I really need this to work and can't use a solution like TuesPechkin since my app is running as an Azure wep app.
Here's how I create the pdf from some simple html. Again, page-breaking appears even in  tags...

Hi guys,
i had an Text that where cut off. My Problem was that i had an Text inside an
 where an
-HTMLElement was present wich broke the whole Text. I needed to replace all
-HTMLElement with \n & add * { page-brake-inside: avoid; }.
I used these NuGet-Packages:
https://www.nuget.org/packages/HtmlRenderer.Core/1.5.1-beta1
https://www.nuget.org/packages/HtmlRenderer.PdfSharp/1.5.1-beta1
Maybe this will help someone.


Fix bug in air density calculation

Fixed bug in air density calculation

@yixin - can you point us to the line in VIC 4.2 that had the correct formulation for air density.  Also, do you have a before and after plot comparing 4.2 and 5?

@jhamman  the line in VIC4.2 that calculates air density is here: https://github.com/UW-Hydro/VIC/blob/support/VIC.4.2.d/src/initialize_atmos.c#L1156
I have comparison plots for each grid cell in the Stehekin 10-day example for all output variables. For air density, they look identical after the fix.

looks good.  Thanks @yixinmao!


Add a redirection to the pathfinder template file

Instead of executing: rails new new_app -m http://blabla.github.com/pathfinder/template.rb
it would be nice to have an url like http://pathfinder.marsbased.com pointing to the github template file

I'm closing this issue because it doesn't work. Rails templates don't follow redirects 😞


Odd vertical spaces in pre elements (2nd attempt)

Same issue as #329, which was fixed, so I can't reproduce this issue on my local setup with v0.2.4 of the theme.
However, this issue still present on the rtd, see matrix example at the bottom of the introduction page:

@Blendify, told me that installed on the rtd theme now is up to date (i.e. v0.2.4).  But apparently, something doesn't work.

Ok we finally traced the issue and will be fixed tomorrow on RTD.

Now works.  Thank you.


Resolve #5

We shouldn't even think about putting experimental versions in the master branch.

asm2bf in experimental version was highly unstable if used incorrectly, so we revert every evilness it has caused.

Before this can get merged, we need log up and running.


Convert to .NET Core 2.1 typed client

@SeanFarrow FYI, I updated a few things - I assume the main typo you were referring to was Pwned Passports ? 😉

I can't seem to find where this is used in the sample app though

The client is added implicitly when you call the validator, if it hasn't already been added. Good call on showing the usage with Polly though, so I've added that to the sample.
Also, I updated the Validator so it doesn't throw if there's an exception calling the endpoint - I think we should be conservative in that case, and assume that the password is ok, otherwise noone would ever be able to create a password if the API is down!
I'll merge this shortly, and work on updating the readme.
I'm also thinking I should rename the repo to be PwnedPasswords instead of HaveIBeenPwnedValidator. I went back and forth about PwnedPasswords  vs HaveIBeenPwned in the project, but seeing as the client only uses that one part of the API (rather than the full data-breach API) it seemed best.


Added abstracted idb and sp metafiles

Think I finally got a mergeable buildable commit.. Sorry for the newbe behaviour with multiple bogus pull requests.. I just figured out I didn't need to do that.

-1
I see many problems in this PR, from small ones: useless formatting changes, useless headers to most important ones: Spring classes directly copy/pasted...

Let's make sure that first of all you understood what I needed to implement.. 1) An abstraction on the meta files used by SAML so I can store and retreive those files from a back end rather than a file 2) Properly handle ajax because you can't to a normal redirect on an ajax call since browser takes away control before the caller can react to it 3) allow reading key from a different keystore type and alias
If you agree to those things then 2) is easy enough to reimplement.. 1) you need to tell me if the spring method of abstraction works for you and finally 1) and 3) need to make sure wrt how I'm passing the new info to SAML via the changed constructors
At this point in my code all that I have checked in prior to the merge is working and has been tested. I noticed you changed to java 8 so I can't use your latest version in re-encorporate and test anymore hence I can attempt to re-make the changes but I won't be able to test them since our code is still on java 7.

About the AJAX mechanism, there is already a: https://github.com/pac4j/pac4j/blob/master/pac4j-core/src/main/java/org/pac4j/core/http/AjaxRequestResolver.java concept.
Regarding 1) and 3), I'm not sure this will go in pac4j but for sure, pac4j could be changed to make these changes really easy. About the spring dependency, I don't want it by default, but this can an optional one (and thus, you don't need to copy any Spring file).
We must be very careful when changing pac4j and especially pac4j-core, so that's why I expect a PR where only the changes are visible (no formatting mess) and fully tested.
pac4j v1.9 is Java 8 while pac4j v1.8.x is still Java 7. It may make sense to target both branches in your case.


feat: outlook ".msg" file converter

This pull request introduces a new feature to the application: the ability to convert Outlook .msg files into markdown format by extracting email metadata and content. The implementation adds a new class, OutlookMsgConverter, which extends the DocumentConverter base class.

Key Features
Email Metadata Extraction:
Extracts and includes key headers like From, To, Subject  in the markdown output.
Email Body Conversion:
Reads the email body content and formats it into markdown.
Robust Encoding Support:
Attempts to decode content in UTF-16 first, falling back to UTF-8 and handling edge cases to ensure accurate conversion.

Implementation Details
File Validation:
Checks the file extension (.msg) before proceeding with the conversion.
Stream Parsing:
Uses olefile to parse the .msg file structure, extracting streams for headers and body content.
Error Handling:
Includes comprehensive exception handling to manage invalid files or unexpected errors during the conversion process.
Please review the implementation and provide feedback or suggestions for improvement.

@microsoft-github-policy-service agree

could you add tests?

added the tests.


No audio with CameraUVC source

Hello Pedro, I have a small problem:
when choosing UVC camera as source stream has no audio.
Smartphone mic, webcam mic, external usb mic - none of them works.
Any other source, for example Camera 2,  works as expected

Thank you, I indeed had just to change default value for MicrophoneSource class


feat: generate accessed files list for workload

We can use fanotify to generate an in-order accessed files list for workload. This function is useful for workload analysis and image optimization.
We can set --fanotify flag to enable this feature.

Set --wait-time flag to specify the recording time.

nerdctl run --rm --net host --fanotify --wait-time 30 nginx

Set --wait-line to specify a string line that fanotify wait for.

nerdctl run --rm --net host --fanotify --wait-line "ready for start up" nginx

The task will be killed when getting the expected string line from the standard output.

Set --wait-signal flag to wait for SIGINT signal.

nerdctl run --rm --net host --fanotify --wait-signal nginx

The task will be killed when getting SIGINT (ctrl + C) signal.

Set --output flag to specify the output path of fanotify.

If we do not set the --output flag, the default value is accessed_files.out.
This PR references the implementation of stargz-snapshotter and relates to https://github.com/containerd/nydus-snapshotter/pull/228.
Meanwhile, This PR is related to #1687.
Signed-off-by: Bin Tang<EMAIL_ADDRESS>

Can we rather use /etc/containers/oci/hooks.d/*.json ?
https://manpages.ubuntu.com/manpages/impish/man5/oci-hooks.5.html

We are implementing this with OCI Hooks. Thanks for @AkihiroSuda's advice. It's time to close this PR.

By the way, are there any plans to support loading the OCI Hooks configuration from files? I have found an internal subcommand oci-hook in which createRuntime and postStop Hooks are available.

Thanks for all comments.


HTTPS again :)

Have done a quick flick through the code, this seems to be the last reference.

this one won't work on windows, it can't read s3 assets off of ssl

Is there another (trusted) place that we could read VERSION from?

I don't think it's really necessarily personally, this is just an indicator of whether or not an update needs to happen. The download itself is validated against a sha too.


KAFKA-9258 Check Connect Metrics non-null in task stop

Connect sometimes will stop task when start() has failed. In these cases, we must guard against NPE as noted in https://issues.apache.org/jira/browse/KAFKA-9258

@wicknicks,
@C0urante has helped out with the testing, and the broader feature "restart a failed task" is now covered with integration test -- thanks @C0urante!


Default to empty config in AWS EBS provider constructor

Change Overview
This PR fixes the panic problem where the argument config is nil.
Pull request type
Please check the type of change your PR introduces:

[ ] Work in Progress
[ ] Refactoring (no functional changes, no api changes)
[ ] Trivial/Minor
[X] Bugfix
[ ] Feature
[ ] Documentation

Test Plan

[X] Manual
[ ] Unit test
[ ] E2E

@tdmanv @pavannd1 Updated the PR.


Inconsistent results when changing node category with identifier

Described here: biothings/biothings.api/issues/223

This would appear to be occurring because the SRI normalizer lookup doesn't return the category Disease for HP:0000217 (as seen here), causing category inference to determine PhenotypicFeature to be the most-specific category available.
When instead Disease is specified as a category, category inference adds PhenotypicFeature, but as Disease is also kept, it causes additional sub-query operations to be found, specifically one with MyDisease which structures into the following query config:
{
  url: "http://mydisease.info/v1/query",
  params: {
    fields: "disgenet.genes_related_to_disease.gene_id, disgenet.genes_related_to_disease.pubmed, disgenet.genes_related_to_disease.source, disgenet.genes_related_to_disease.DPI, disgenet.genes_related_to_disease.DSI, disgenet.genes_related_to_disease.EI, disgenet.genes_related_to_disease.score",
  },
  data: "q=C0043352&scopes=disgenet.xrefs.umls",
  method: "post",
  timeout: 50000,
  headers: {
    "User-Agent": "BTE/dev Node/v17.6.0 darwin",
  },
}

This and 3 other sub-queries return records which are kept in results assembly, with this specific query resulting in the addition of IRF5.
So, this would appear to be an issue with the SRI node normalizer not returning the complete set of categories? It could probably be resolved by modifying the category inference to take parent categories when not all child categories are present, however this would lead to other cases where inferred categories are made too general. Specific cases could be added, such explicitly applying said behavior to only mixins or a specific set of categories, though the latter would likely be trouble to maintain adequately.
Tagging @andrewsu @colleenXu for their thoughts.

I want to confirm my understanding here.
Case 1: If the query specifies "categories": ["biolink:DiseaseOrPhenotypicFeature"] for n2 / HP:0000217, then BTE will consult the biolink model and find its children biolink:Disease and biolink:PhenotypicFeature.  Then BTE will query the SmartAPI metakg to find APIs that link any of those three categories with biolink:Gene (the category for n01).  And that query will result in both IRF5 and Amylases.
Case 2: If the query does not specify a category for n2 / HP:0000217, then BTE queries the node normalizer (https://nodenormalization-sri.renci.org/1.2/get_normalized_nodes?curie=HP:0000217) and determines that the appropriate category is biolink:PhenotypicFeature. So BTE will query the metaKG only linking biolink:PhenotypicFeature to biolink:Gene, and we miss IRF5 because it presumably came from an API annotated to biolink:Disease
Case 3: If the query specifies "categories": ["biolink:PhenotypicFeature"] for n2 / HP:0000217, we get the same behavior as not specifying a category for n2 / HP:0000217, same as Case 2.
Case 4: If the query specifies "categories": ["biolink:Disease"] for n2 / HP:0000217, somehow "category inference adds PhenotypicFeature" (quoting from @tokebe's last post), so BTE queries the metakg for APIs that link either of those two class types (biolink:PhenotypicFeature or biolink:Disease) , and we get both IRF5 and Amylases.
Corrections/clarifications appreciated!

@tokebe Thanks for the detailed analysis. My hunch was confirmed, and I think that the current behavior is fine. I'll explain...
We rely on x-bte annotation of APIs that exist somewhat-outside of Translator. These data provided by 1 endpoint will often not cleanly fit into 1 biolink semantic type.

For example, resources often use a particular ontology to label their entities (e.g. CHEBI for chemicals). But Translator/biolink-model/SRI-Node-Normalizer may map those CHEBI IDs to different semantic types (MolecularMixtures / SmallMolecules / etc).
The API's responses often don't include extra info from the ontology, that would help us filter responses for CHEBI IDs that only map to a particular biolink semantic type.
It's not clear what SRI Node Normalizer is doing to map IDs to biolink semantic types.

In this situation, we've been doing "fuzzy" or "somewhat-imprecise" mappings between the ID namespaces / ID sets used by an API endpoint + the biolink-model. This is more tractable and makes x-bte annotation easier to write.

In this issue's case, we've labeled all the UMLS IDs from the DisGeNET data in MyDisease as Disease. And we've labeled HP IDs in various operations as PhenotypicFeatures (even though some look like diseases and have been mapped by SRI Node Normalizer as biolink-model Disease).
In the case of CHEBI IDs, we label them as "SmallMolecule" in various operations. This is partially due to historical reasons (there used to be only a ChemicalSubstance semantic type, and that was mapped to SmallMolecule....so we map all chemical-things to SmallMolecule now)

Why don't we just strictly adhere to the SRI Node Normalizer's mapping of IDs? Well...

we don't know how the SRI Node Normalizer does this mapping (for example, I've opened this ticket on how this UMLS mapping doesn't match the biolink-model). And even if we knew that...
imagine asking every API (some of which are completely external to our team/Translator) for ALL IDs in the underlying data for each endpoint. Then we can map ALL of them to the biolink-model with SRI Node Normalizer.....which may hit issues if the vocab/IDs aren't covered by the service. And then we write x-bte operations for each biolink-semantic type returned by each endpoint. But we probably can't query the endpoint with such specificity (for example, only return the IDs corresponding to a specific biolink-semantic type)...

Can we do better?

In this specific situation, we could....maybe.

DisGeNET's data includes a diseaseType column which we haven't been parsing. This column uses DisGeNET's mapping of UMLS IDs into "disease", "phenotype", and "group". I only took a quick look, but I think they somewhat correspond to biolink-model's Disease (DisGeNET "disease" and "group") and PhenotypicFeature (DisGeNET "phenotype"). I think they might not exactly match though...
If we include the diseaseType in MyDisease's DisGeNET data, we may be able to write more specific operations since we'll be able to query for subsets of the DisGeNET data.

This is really on an endpoint-by-endpoint / API-by-API basis. It depends a LOT on whether resources allow specific querying based on the intricacies of the vocabs they use...

Err to address Andrew...
@andrewsu All the cases, but particularly 1 and 4, relate to #418 . We decided to keep the categories provided by the user (and expand with biolink-model's hierarchy), while also adding the category provided by SRI Node Normalizer when it differs. Both because of the "fuzziness" of the x-bte annotation (explained in detail in the above post) AND because the user might not really be "wrong" in their choice of semantic type. This becomes even more complicated for data sources like SEMMEDDB, where the UMLS-semantic-type for an ID may differ between associations....and therefore the ID maps to different semantic types for those associations...

And TDLR for the big comment above @tokebe @andrewsu:

I think the current behavior is fine. Perhaps MyDisease's parsing of DisGeNET could be improved, and then we could edit the x-bte annotations
I think any fixes for this would have unintended consequences. For example, maybe one decides to write a Disease->Gene and a Phenotype->Gene operation for the MyDisease-DisGeNET operation. But this is still the same underlying sub-query.....so if someone uses a more general category somewhere, you'll end up running the same sub-query twice and maybe treating the results as two separate sets of data when they aren't...

Interesting issue - I can speak a little bit about what nodenorm is doing, though I don't know that it will shed any light on this.  It's a difficult problem because you have many vocabularies and we are finding equivalent identifiers across them, and then those different vocabularies assign classes to this new thing that may or may not agree with one another.    Disease/Phenotype is probably the worst offender, because it's really not clear what the difference even is.
There are a number of ways to approach that problem, but the one that we chose is to establish some (fairly ad-hoc) practical rules to assigning a class to a clique.   They're situationally dependent to handle cases like the ones @colleenXu noted above where HP has things in it that sure look like diseases to everybody else.   We're thinking of cleaning up disease/phenotype by treating it as a conflation like we do for genes/proteins so that people can keep things more carefully pulled apart and then merge as needed.  Would that help in this case?
As far as helping this particular issue, I am certainly no bte annotation expert, so take this with a grain of salt, but I think of those annotations as defining what kinds of things (in biolink/translator speak) you are getting from this api.  In this case, it seems like you're getting both diseases and phenotypes, so can the annotation just include both?

@cbizon we discussed this issue at our internal meeting earlier, and we ended up with these two points:

Conflation of Disease with PhenotypicFeature seems like a reasonable way to ensure consistency in the various versions of this query.  It is perhaps timely in that it ties into some upcoming discussions in the Data Modeling group around other conflations (genes and genomes, drugs and smallMolecules, etc.).  The main question here is whether that gets individually implemented by each tool, or whether that gets implemented through node normalizer.  (Since we already implemented Gene / Protein conflation in BTE, that would be simple enough for us to follow that same pattern within BTE again...)

Regarding editing smartAPI annotations to ensure consistency, while it's a possible solution it is one that we are not enthusiastic about.  It would be a lot of work and there are lots of issues with generalizability and readability that make the net benefit a bit less clear...

Should be resolved by https://github.com/biothings/bte_trapi_query_graph_handler/pull/106?

I think so? I didn't test this one though...

Marking as resolved (forgot to add that I had tested the above-mentioned PR prior to merging).


modify Authentication methods

Currently, Security Defaults can use  "Verification code from mobile app", so please change the table in this article.

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

ID: cfac0ecf-e5a9-c5ff-f8c2-ce27789708f5
Version Independent ID: 0f5e0cc9-a54f-a5c8-192d-d60a6a4db3ff
Content: Azure Active Directory security defaults
Content Source: articles/active-directory/fundamentals/concept-fundamentals-security-defaults.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@abcastner Thank you for your feedback . We will investigate and update the thread further.

@abcastner We have verified and made the required changes.
@MicrosoftGuyJFlo Could you approve the changes to the PR#54425 ?

@MicrosoftGuyJFlo   Could you approve the changes to the PR#54425 ?

Correct you can use the verification code from the Authenticator app but you cannot register that method using security defaults.
The registration of the Authenticator app does enables code and security defaults allows that as a method.
I will be working on this doc this week and will add some clarification.


Typo in the First Example

Below sentence is not in accordance with the image
Current:
A synchronous secondary replica on COMPUTER03
Should be:
An Asynchronous secondary replica on COMPUTER03
Also, the Script to create the Availability Group
'COMPUTER03' WITH
(
ENDPOINT_URL = 'TCP://COMPUTER03..:5022',
AVAILABILITY_MODE = SYNCHRONOUS_COMMIT,
Either the Image or aforementioned Text is wrong

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

ID: 948a3def-7af7-e4dc-e14e-c49613ffc2bb
Version Independent ID: 7d0b6654-eb60-1640-10e1-1f00b6245cf4
Content: SQL Server secondary to primary replica read/write connection redirection-Always On Availability Groups - SQL Server Always On
Content Source: docs/database-engine/availability-groups/windows/secondary-replica-connection-redirection-always-on-availability-groups.md
Product: sql
Technology: high-availability
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@edvaldocastro -- Edvaldo, thank you for your observations.
@MikeRayMSFT -- Mike, please look into this issue 3692.

@edvaldocastro - thank you for reviewing this and commenting on it. We'll take a look.

Thanks again for contacting Microsoft!  We've submitted a documentation change based on your suggestion, and it should be live at this point. I will proceed with closing out this Git Issue, but please feel free to commend should you need anything further.
Thanks again and hope you have a wonderful rest of your day!
Masha from the SQL Docs team


Cow: Watch 09831~749~976

Cow requests the watch of the watch_number 09831~749~976. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Self-rejected by Cow in Charcoal HQ. Reason: 'replacing with blacklist instead zomg'


Remove unnecessary StringBuilder

Relates to #1378 .

I think these StringBuffers can be replaced with simple String concatenations: more readable, even more efficient.

Please async the origin/master branch to resolve the problem of CI check failure

@liketic  StringBuffer will more efficient and the compiler also optimizes the string concatenation into a stringbuilder， it is  unnecessary to replace current codes.

Codecov Report

Merging #1388 into master will decrease coverage by 0.04%.
The diff coverage is 10%.

@@            Coverage Diff            @@
##           master   #1388      +/-   ##
=========================================
- Coverage   33.04%     33%   -0.05%
=========================================
  Files         691     691
  Lines       34625   34549      -76
  Branches     6851    6851
=========================================
- Hits        11443   11404      -39
+ Misses      21238   21199      -39
- Partials     1944    1946       +2

Impacted Files
Coverage Δ

...ting/exchange/support/header/HeartbeatHandler.java
6.81% <0%> (+0.43%)
:arrow_up:

...ubbo/examples/heartbeat/impl/HelloServiceImpl.java
0% <0%> (ø)
:arrow_up:

...baba/dubbo/examples/heartbeat/HeartbeatClient.java
0% <0%> (ø)
:arrow_up:

.../alibaba/dubbo/rpc/cluster/merger/ArrayMerger.java
68.42% <0%> (+6.51%)
:arrow_up:

...va/com/alibaba/dubbo/rpc/filter/GenericFilter.java
0% <0%> (ø)
:arrow_up:

...a/com/alibaba/dubbo/common/utils/ExecutorUtil.java
0% <0%> (ø)
:arrow_up:

...o/rpc/cluster/support/MergeableClusterInvoker.java
43.82% <0%> (+5.95%)
:arrow_up:

...om/alibaba/dubbo/rpc/filter/GenericImplFilter.java
0% <0%> (ø)
:arrow_up:

...baba/dubbo/common/beanutil/JavaBeanDescriptor.java
52.38% <0%> (+1.61%)
:arrow_up:

...baba/dubbo/examples/heartbeat/HeartbeatServer.java
0% <0%> (ø)
:arrow_up:

... and 11 more

Continue to review full report at Codecov.

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


ERROR: unable to download video data: HTTP Error 403: Forbidden

I am not able to download

Cannot download large files


Fix bc19e182395f45ce53ea7303098e805de50640e1 commit

Class O1 has method getObj1(). getObj1() declared to return Object
Class O2 extend O3 and O3 has method getObj2(), but O2 does not override getObj2()
If we evaluate getObj1().getObj2() then method getObj2() will not be found

Sorry, but your description of the problem is a bit confusing: what is O3?
Please add a test case to your PR that reproduces the problem you're fixing.

O3 is some abstract class and it implements getObj2() method.
After bc19e182395f45ce53ea7303098e805de50640e1 cls.getClass().getDeclaredMethods() will not return methods for abstract class, only declared in parent one.

Also fix can be a simple replacing getDeclaredMethods to getMethods

The test case I added here https://github.com/mvel/mvel/commit/860984b400e6925c456c7d4aa5d5ff4b7972edaf demonstrates that the use case you described is already working


Reword sentence in 'Initializing projects using the CLI Tool' page

Description:
The text "Select a generated archive suitable for your platform (i.e., Mac, Windows, Linux) and extract it the CLI tool that you downloaded to a desired location and cd into it." at https://docs.wso2.com/display/AM260/Initializing+projects+using+the+CLI+Tool#InitializingprojectsusingtheCLITool-LoggingintoAPIManagerviatheCLItool should be reworded to say something like "..and extract this archive which contains the CLI tool.. "
Suggested Labels:
CLITool
Suggested Assignees:

Affected Product Version:
2.6.0
__

fixed on 2.6.0.


Tian subcortical atlas

Hi @mattcieslak,
I have some analyses for which subcortical parcellation that includes the hippocampus would be invaluable. Schaefer atlases are my general preference, but the CIT168 atlas doesn't include the hippocampus.
I realize that not everyone needs every atlas in the universe so what would I need to do locally to use the Tian atlas (https://github.com/yetianmed/subcortex/tree/master) in recon workflows?
Thanks

Alternatively, is there a way to utilize the updated Schaefer atlases currently in AtlasPack (just saw the commits with the inclusion of the hippocampal nodes)

Does this page help? https://qsiprep.readthedocs.io/en/latest/reconstruction.html?highlight=atlas#using-custom-atlases

Yep... knew that was somewhere...


Fix custom templates issue with defaultValue

Fixes #424

@danielnaab not sure there is. Have you seen #430? It says they'll prioritize UK public sector (which includes me) so hoping it can go out as part of #433 but I haven't heard anything

Hi @ediblecode,
We've been looking at this to try and review it for you, but are struggling to get our head around what the correct behaviour should be without a more realistic example. Can you talk us through your use case and what you expect to happen? When would you set a defaultValue that is not one of the options?
Thanks!

Thanks for getting back to me @36degrees.
We're using it for a site search typeahead (rather than a list of finite options for a form). The suggestions are requested from a server API endpoint using a function passed to the source prop. Which means defaultValue isn't one of the options, as there aren't any when you first focus.
This is how we're using it if it helps: https://github.com/nice-digital/global-nav/blob/master/src/Header/Search/Autocomplete/Autocomplete.jsx#L123.
Hopefully that, in tandem with the codepen in the issue I raised might be a bit clearer but if not let me know and I'll try add some more details. Thanks

We're using it for a site search typeahead (rather than a list of finite options for a form). The suggestions are requested from a server API endpoint using a function passed to the source prop. Which means defaultValue isn't one of the options, as there aren't any when you first focus.

Ah, OK, so then when you display the search results page you're using defaultValue to fill in the terms the user searched for?
I appreciate at the same time that the current behaviour (displaying undefined) is definitely not desirable, I'm also wary that we're stretching the component to do things that it was never really designed to do – we're guilty of this too, as we also use it to power the search on the Design System! – so I want to be really careful that we don't start changing the component in ways that negatively affect its primary use case, which is selecting a pre-defined value from a finite list of options.
The expected behaviour here seems a little undefined – what should happen when you focus the input? Given we're in a search context, I think I'd expect the search to be run again and to see the same results that I saw before I submitted the page, not an empty list.
I'll review the code as it stands at the moment with this in mind, and let's see where we get to.

I think having reviewed your proposed changes again there's something odd about the fact that we're relying (I think?) on the inputValue function being passed a string if the current value of the textbox was either entered by the user or was the defaultValue, versus an object if it's a result that's come from the source function, and using that to decide whether things should be shown in the autocomplete?
The fact that inputValue could be passed either seems problematic.

Thanks for the review, I'll have a look at the specific feedback. And to answer some other points:

so then when you display the search results page you're using defaultValue to fill in the terms the user searched for?

yes, that's right.

I'm also wary that we're stretching the component to do things that it was never really designed to do

yup, fair enough. Do you think this legit within scope? It feels to me, that because source, templates.inputValue and templates.suggestion all take functions, that this use case in tandem with defaultValue seems plausible, BUT making changes to specifically support site search might be a step too far.

The expected behaviour here seems a little undefined – what should happen when you focus the input?

Very, very good point. I think you're right actually that the expected behaviour should maybe be to display suggestions for the term on focus, and that is new feature I think, specific to site search - I can't see a way to get suggestions to appear on focus based on the current options.
Showing suggestions for search on focus is certainly a common paradigm for search engines. I understand that is different to the current behaviour within accessible-autocomplete though, so would be a new feature. Maybe an option of showSuggestionsOnFocus or something - I'm not sure it would be right to switch that behaviour based on templates.inputValue being a function rather than a string. And also that feels specific to site search so probably a no-go!
So in light of that, I think the expected behaviour would be for it to show no options on focus. I can't see any other way round that? Unless we change the behaviour so the focus doesn't just show [defaultValue] as it does now, but loads suggestions - but that feels like a breaking change and the wrong approach.

In #424 you suggested another approach:

OR we change the options.map call within the render, to only render options where the result of this.templateSuggestion(option) is not falsey.

Did you explore this at all? Effectively, I think we'd instead be trying to solve a slightly simpler bug:

if you return empty string then you still get an 'empty' suggestion

This would then allow you to address this in your own suggestiontemplate function, rather than requiring changes to the autocomplete logic? I can't think of a situation where 'gracefully handling'suggestion` returning false would cause a problem.

Interesting, so I did. And I even suggested option 2 was better didn't I?! Thanks, past me. I'll look at that then, might be more elegant. I'm sure there was a reason why I went down the other route, but pushing the logic to the consumer as to what they return from the function makes sense if that works.

Hey @ediblecode, long time no see (took a little squinting at your profile photo!)
We've given this PR a milestone and I've rebased it with main to help us review it


Add Table.to_pandas() method

This is used by plotly express 5.15.
Deprecate the Table.to_dataframe() method (check deprecating code docs and replace it with Table.to_pandas(). Set the deprecation date as 2024.3 (which corresponds to the Q1 release).

@GavinHuttley any specific reason for deprecation, or just a rename?

For a public project, we have to avoid breaking the code of users. By adding a deprecation warning, they know how to change their code and by when that change must be done.

Sorry, to be more specific, any reason for deprecating Table.to_dataframe() in place of Table.to_pandas()? This is for the @deprecated_callable(reason=)

Having Table.to_pandas() means cogent3 tables will be usable by plotly express. So perhaps "to enable usage by plotly"


[Astro build] CustomerError: The size of the build output (424304711) exceeds the max allowed size of 230686720 bytes. Please reduce the size of your build output (/codebuild/output/src1007620978/src/xxx/.amplify-hosting/compute/default) and try again.

Environment information
I am trying to deploy below template using AWS amplify: https://astro.build/themes/details/copper-astro/. My build output is approximately 424 MB, exceeding the limit of 220 MB. I am reporting this as a bug since I cannot control the max allowed size in Amplify.
I followed the steps in the aws doc to deploy the web app:
https://docs.aws.amazon.com/amplify/latest/userguide/get-started-astro.html
The instruction in the above aws doc:

In the amplify.yml file, locate the frontend build commands section. Enter mv node_modules ./.amplify-hosting/compute/default

causes an error due to the max allowed size of 230686720 bytes.
Build image:
Amazon Linux 2023
amplify.yml:
version: 1
frontend:
    phases:
        preBuild:
            commands:
                - 'npm ci --cache .npm --prefer-offline'
        build:
            commands:
                - 'npm run build'
                - 'mv node_modules ./.amplify-hosting/compute/default'
    artifacts:
        baseDirectory: .amplify-hosting
        files:
            - '**/*'
    cache:
        paths:
            - '.npm/**/*'

Describe the bug
Below is the deployment log:
2024-11-03T00:28:54.805Z [INFO]: 24:28:54 ✓ Completed in 3.98s.

234

2024-11-03T00:28:54.829Z [INFO]: 24:28:54 [@astrojs/sitemap] `sitemap-index.xml` created at `dist`

235

2024-11-03T00:28:54.830Z [INFO]: 24:28:54 [build] 29 page(s) built in 15.72s

236

24:28:54 [build] Complete!

237

2024-11-03T00:28:54.874Z [INFO]: # Executing command: mv node_modules ./.amplify-hosting/compute/default

238

2024-11-03T00:28:54.876Z [INFO]: # Completed phase: build

239

## Completed Frontend Build

240

2024-11-03T00:28:54.886Z [INFO]: ## Build completed successfully

241

2024-11-03T00:28:54.887Z [INFO]: # Starting caching...

242

2024-11-03T00:28:54.895Z [INFO]: # Creating cache artifact...

243

2024-11-03T00:28:54.898Z [INFO]: # Created cache artifact

244

2024-11-03T00:28:54.898Z [INFO]: # Uploading cache artifact...

245

2024-11-03T00:28:54.963Z [INFO]: # Uploaded cache artifact

246

2024-11-03T00:28:54.963Z [INFO]: # Caching completed

247

2024-11-03T00:29:13.700Z [ERROR]: !!! CustomerError: The size of the build output (424304711) exceeds the max allowed size of 230686720 bytes. Please reduce the size of your build output (/codebuild/output/src1007620978/src/palxi2/.amplify-hosting/compute/default) and try again.

248

2024-11-03T00:29:13.804Z [INFO]: # Starting environment caching...

249

2024-11-03T00:29:13.804Z [INFO]: # Environment caching completed

Reproduction steps
Hosting the same website on amplify using the aws docs. Its a paid template I already downloaded it happy to email you the template if you need it.

Hi @akila-ocj 👋 , thanks for reaching out. You've correctly stated that this issue occurs since the build output size exceeds 220 MB due to the size of the node_modules being copied over to the .amplify-hosting/compute/default directory. In the long term, to mitigate it the process of copying over the node_modules will need to be optimized and tree shaking will need to be implemented.
In the short term, our recommendation will be to review this troubleshooting guide and manually remove the binaries or node_modules that aren't required at runtime.
After downloading the build artifacts, you can review the dependencies locally:
- cd compute/default/node_modules
- du -csh *

Identify dependencies that you can remove

Next, modify the amplify.yml to remove them at buildtime:
build:
      commands:
        - npm run build
        - rm -rf node_modules/{deps1}
        - rm -rf node_modules/{deps2}
        - mv node_modules ./.amplify-hosting/compute/default

@akila-ocj 👋 , checking in to see if the above steps helped and whether you are unblocked?

Closing this thread since we have shared guidance on how to debug the build outputs folder.