"right truncation" error from sqlsrv_fetch_array of long VARCHAR(max) column containing 8-bit characters

PHP Driver version or file name
sqlsrv support => enabled
ExtensionVer => 5.9.0+14119
Directive => Local Value => Master Value
sqlsrv.WarningsReturnAsErrors => On => On
sqlsrv.LogSeverity => 0 => 0
sqlsrv.LogSubsystems => 0 => 0
sqlsrv.ClientBufferMaxKBSize => 10240 => 10240
sqlsrv.SetLocaleInfo => 2 => 2
SQL Server version
15.0.2000.5 (earlier versions also repro)
Client operating system
Ubuntu 20.04.2 LTS
PHP version
PHP 7.4.3 (cli) (built: Oct  6 2020 15:47:56) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.3, Copyright (c), by Zend Technologies
with Xdebug v2.9.2, Copyright (c) 2002-2020, by Derick Rethans
Microsoft ODBC Driver version
unixodbc-dev |      2.3.7 | https://packages.microsoft.com/ubuntu/20.04/prod focal/main amd64 Packages
unixodbc-dev | 2.3.6-0.1build1 | http://archive.ubuntu.com/ubuntu focal/main amd64 Packages
Problem description
When using sqlsrv_fetch_array fails with String data, right truncation under the following conditions:

The column is type VARCHAR(max) or the column type is just barely big enough to fit the exact data
The column is over about 2048 bytes long
The column contains ASCII values > 127
The collation of the column is NOT LIKE '%UTF8' (i.e. it is something like SQL_Latin1_General... and not Latin1_General_100_CI_AS_SC_UTF8)
The connection options CharacterSet is not set or set to SQLSRV_ENC_CHAR (and not set to 'UTF-8')

In this case the column's binary data happens to be UTF-8, but I don't think that matters since all byte sequences are legal Latin1 and I think it's converting from suspected-Latin1 to UTF-8.
I suspect that some code that does dynamic allocation to fetch results is starting with a 2048 byte buffer and mis-handling the case where the Latin1->UTF-8 conversion causes the string to grow. In the VARCHAR(n) cases that same code probably does exact allocation and avoids the bug whenever n is big enough to contain the encoded result.
Expected behavior and actual behavior
Expected successful read of string data (possibly, in this case, with the 8-bit characters presumed to be Latin1 and actually turned into UTF-8 codepoints).
Actually got String data, right truncation
Repro code or steps to reproduce
Note that the 8 bit characters Ã± represent n-with-tilde-on-top as raw UTF-8-expressed-as-Latin1
 $connInfo = array(
    'Database' => 'Test',
    'UID' => 'test',
    'PWD' => 'test',
    'LoginTimeout' => 5,
    'CharacterSet' => SQLSRV_ENC_CHAR
);

$conn = sqlsrv_connect($host, $connInfo);

# leading string >= 2045 leading to result length > 2048
# column must be VARCHAR(MAX) or VARCHAR(2048) (starts working with bigger VARCHAR(n), e.g. 2060)
# 'CharacterSet' connInfo must not be set to SQLSRV_ENC_CHAR (works with UTF-8)
# COLLATE must not be %UTF8% (e.g. Latin1_General_100_CI_AS_SC_UTF8 works)
$sql = "DROP TABLE IF EXISTS #tmpTest;
		SET NOCOUNT ON;
        DECLARE @val VARCHAR(8000) =  REPLICATE('a', 2045) + 'Ã±';
        CREATE TABLE #tmpTest (testCol VARCHAR(MAX) COLLATE SQL_Latin1_General_CP1_CI_AS);
        INSERT INTO #tmpTest (testCol) VALUES (@val);
        SELECT * from #tmpTest;";
$stmt = sqlsrv_query($conn, $sql);
$row = sqlsrv_fetch_array($stmt, SQLSRV_FETCH_ASSOC);
$errors = sqlsrv_errors(SQLSRV_ERR_ERRORS);
# String data, right truncation
var_dump($row, $errors);

Thanks @bjj for bringing this to our attention. We can reproduce it in Linux Ubuntu 18.04 as well and will get back to you after some investigation.

Hi @bjj , when connecting with CharacterSet set to SQLSRV_ENC_CHAR, the sqlsrv driver will not do any conversion for the characters. In Ubuntu (or most recent Linux distros), the default is UTF-8. Therefore, your php script is likely in a UTF-8 format. You can easily check by running file <filename>.
In your test scenario, because of the 8 bit character Ã±, your php script should not be saved in UTF-8 format. In other words, make sure the output of the file command for your script looks something like this: sqlsrv_1261a.php: PHP script, ISO-8859 text.
Otherwise, if the output of file gives you PHP script, UTF-8 Unicode text, then you will get this string truncation issue. The reason is that the special A character is interpreted as a UTF-8 character that takes 4 bytes instead of 1 byte.
If you run this php script in command line, make sure your terminal uses the right character set and the correct locale has been generated and set as well (e.g. en_US.ISO-8859-1). If not, you might see a question mark instead of the special 8 bit character.
Once you make sure you have the right settings, you should not run into this truncation issue again.

Thanks for looking into this @yitam

when connecting with CharacterSet set to SQLSRV_ENC_CHAR, the sqlsrv driver will not do any conversion for the characters

I think that might be the root cause of the bug. It is doing conversion of the characters. You can easily observe (with a slightly modified version of that script) that there are 2 bytes of data in the table (which correspond to one desired UTF-8 codepoint) but because they get converted again to UTF-8 (as if they were something else, probably ISO-8859-1) they expand to 4 bytes (2 UTF-8 characters representing what the original binary happened to be). I have not found any settings that avoid this conversion on Linux. This same script works fine on Windows.
I created that minimum repro case just to report this problem. The encoding of that PHP script is not the cause of (or solution to) this problem. The original data that showed this issue was already in the database. It can be inserted into the database in other ways. You can verify that the correct bytes 195, 177 are at the end of the column with SELECT ASCII(RIGHT(testCol, 2)) as A, ASCII(RIGHT(testCol, 1)) as B FROM #tmpTest.

The reason is that the special A character is interpreted as a UTF-8 character that takes 4 bytes instead of 1 byte.

The special two binary bytes 195, 177 which are a valid UTF-8 string are interpreted as a valid ISO-8859-1 string and expanded to 4 bytes. That triggers some bug in sqlsrv. As I said, I think the root cause may well be that the change in encoding should not be happening (I can't find any way to turn it off on Linux, but on Windows it does not happen). I think something is assuming that there can be no change of encoding (due to SQLSRV_ENC_CHAR being selected) and allocating a fixed buffer length, but later something else erroneously changes the encoding, uses more bytes, runs out of space, and returns "right truncation"

Hi @yitam , sorry to let this comment thread get derailed by character encoding issues. While I do believe they are related, the core elements of this bug are:

There is C++ calling into the ODBC API with a buffer with length chosen by the C++ code
The ODBC API is returning an error to the C++ code based on that buffer length
The C++ code is then letting this internal error leak out to the PHP layer

So while I think the C++ code in (1) is quite possibly choosing the wrong buffer length due to encoding assumption mismatches, that is not relevant to (3). I'm calling a PHP function whose contract is "return an arbitrary length string". There's no reason for that to return an error that effectively says "sorry, for reasons outside your control, the internal buffer allocated by the C++ implementation of this PHP function was not large enough to contain the string you wanted".
(also I did try your suggested setlocale idea with no change in observed behavior)

@bjj I see your points, but using setlocale() (must be set before making the first connection) seems to solve the issue on my side. Anyway, let me do some more investigations and get back to you.

using setlocale() (must be set before making the first connection) seems to solve the issue on my side

My mistake -- I did the test in a container that did not have en_US.ISO-8859-1 configured. I added it and I see now and it does make a difference.

Glad to hear that using setlocale() makes a difference for you @bjj :)
After consulting with the ODBC team, you might want to read the documentation about autotranslation when using different codepages

I would love to turn off AutoTranslate in this case, but that feature of ODBC is not exposed to sqlsrv_connect().

FYI, @bjj , [ISO-8859-1 is not exactly the same as Windows 1252](https://www.i18nqa.com/debug/table-iso8859-1-vs-windows-1252.html#:~:text=ISO-8859-1 (also,assigned to these code points.)
If you're using both Windows and Linux or have no control how the data gets inserted, the client and SQL Server might be using different ACPs. With AutoTranslate turned off, extended characters may be misinterpreted, as explained in Using Connection String Keywords.
The recommendation is to deliberately use setlocale in your php scripts (which some of our users do) or shift to UTF-8 to ensure data consistency.

@bjj FYI, in light of ODBC driver's possible autotranslation when using different codepages, we have decided to fix this issue in PHP drivers to make it more flexible in handling the possible UTF-8 characters when fetching ASCII data.

Closing this issue as per 5.10.0-beta1

I'm still getting this error when using 5.10.0-beta1 with PHP 8.1
https://github.com/shlinkio/shlink/runs/4110069594?check_suite_focus=true

Hi @acelaya can you provide repro steps or a script to help us investigate?

Sure thing.
You can reproduce it on this repository: https://github.com/shlinkio/shlink
The workflow above is this one: https://github.com/shlinkio/shlink/blob/develop/.github/workflows/ci.yml#L179
It basically starts a SQL server instance via docker compose, and then runs some database integration tests (composer test:db:ms).
The jobs are run both with PHP 8 and pdo_sqlsrv 5.9, and with PHP 8.1 with pdo_sqlsrv 5.10 beta. In the second case they throw this error: Doctrine\DBAL\Exception\DriverException: An exception occurred while executing a query: SQLSTATE[22001]: [Microsoft][ODBC Driver 17 for SQL Server]String data, right truncation.
The test failing is this one https://github.com/shlinkio/shlink/blob/develop/module/Core/test-db/Domain/Repository/DomainRepositoryTest.php, which covers this class  https://github.com/shlinkio/shlink/blob/develop/module/Core/src/Domain/Repository/DomainRepository.php
I will try to isolate the failing query this afternoon after work, if the information above is not enough.

@acelaya, did you mean that with 5.10beta1 this exception is thrown but not with 5.9?
Please isolate the failing query and explain how to repro this by creating a new issue and filling in the questions.

@acelaya, did you mean that with 5.10beta1 this exception is thrown but not with 5.9?

Yes, exactly.

Please isolate the failing query and explain how to repro this by creating a new issue and filling in the questions.

Perfect, will do.

@yitam sorry for the delay, but I took me a lot of time to reproduce the issue on a small context, and I had to debug a lot on the DBAL library I use to figure out what was going on, but I have finally created the ticket https://github.com/microsoft/msphpsql/issues/1329
I included a supre small repo with reproducible steps. I hope it simplifies debugging it.


Dump Parameters along with decisions

Until now, the user was responsible to provide the right parameters along with a dump in which decisions were serialized. Now we dump both parameters and decisions, which allows the user to care only for using the right kernel with the right dump. A binary allowing to execute a dump by reading command line argument was also added. Not having to provide Parameters comes in handy here, as the parameters have different types depending on the kernel, which made the dynamic dispatch difficult.

bors r=elarnon

bors retry

bors retry


Updated passport package dependency

jaredhanson/passport#400

Thanks for the PR.  Based on reading that thread though, it sounds like the real recommendation is actually to depend on passport-strategy instead of passport directly.  I'll make that change.  Let me know if I'm missing something!

You're right. Changing the dependency to passport-strategy will be the better solution in the long term.


<EMAIL_ADDRESS>won't install

Hello 👋
I'm getting an error when trying to install DeltaChat
Console log:
ERROR Exit code was 2!
Failed to extract files from C:\Users\Magi\scoop\apps\deltachat\1.46.0\$PLUGINSDIR\app-64.7z.
Log file:
  ~\scoop\apps\deltachat\1.46.0\$PLUGINSDIR\7zip.log

7zip.log:
Scanning the drive for archives:

ERROR: The system cannot find the file specified.
C:\Users\Magi\scoop\apps\deltachat\1.46.0\$PLUGINSDIR\app-64.7z

System ERROR:
The system cannot find the file specified.

Ref: deltachat/deltachat-desktop#3973

@aliesbelik Update: DeltaChat merged their fix for packaging.


one-line: Allow to specify "next-line"

PR checklist

[ ] Addresses an existing issue: #0000
[X] New feature, bugfix, or enhancement

[X] Includes tests

[X] Documentation update

Overview of change:
Before, you would specify:
"one-line": [true, "check-catch", "check-else", "check-finally", "check-open-brace", "check-whitespace"]

"one-line": [true, {
	"catch": "next-line",
	"else": "next-line",
	"finally": "next-line",
	"open-brace": "same-line",
	"whitespace": true
}]

And get code like:
if (so) { // '{' on same line

}
// 'else' on next line
else {

}

Is there anything you'd like reviewers to focus on?
This is a breaking change to "one-line" as the options have changed.
We could instead create a new rule and deprecate "one-line". (The old name is less accurate now.) Or, we could deprecate the old options style but still handle it in parseOptions.
CHANGELOG.md entry:
TBD

Added parsing for old-style options. Shouldn't be a breaking change any more.

Closing due to age and inactivity. Feel free to re-open or create a new pull request if you decide to continue working on this.


Bug 1806913: Remove run-level for OCM and operator

The openshift-controller-manager and operator do not need to have run-level set.
Normal admission controller rules should be followed.

/test e2e-aws-builds

/lgtm

/bugzilla refresh

/cherrypick release-4.4


Avoid calling ypcat if not installed

On fedora 22, each time I try to auto-complete a hostname (e.g. ssh + TAB), I get Install package 'yp-tools' to provide command 'ypcat'? [N/y] printed 3 times in the middle of my command prompt. This is a minor annoyance, and I should probably just install yp-tools and move on but ideally zsh should only call ypcat if it is present in the system.

+1

The completion module is your problem. Just install it. It's found by default on other systems.

That's not a reasonable solution, Sorin.  These days, it's perfectly reasonable to leave out any trace of NIS, and not having ypcat installed while tabbing through hostname completion in scp is fairly invasive.

+1 Just come across this whilst trying to debug the ypcat error. I have no use for NIS, especially in a home environment.

This issue is ocurring again. (Though I just installed yp-tools)

this is still an issue

Same issue. Seemed that it would start looping the "do you want to install yp-tools..." and it will cause unstable/slow terminal even after it seems it would've been terminated.


Tags separator not working

Please mark this as a question.

In this image, the conditon is applied to tag 'character'. The conditions had been applied to all tags, which is:
Separator: [,] (a comma)
if more than n tags (n = 4):
[x] Keep n tags, then add: [n=3] [_(+%count%)]
For example, if a file contains multiple names of characters, it will leave kaga_(kancolle),akagi_(kancolle),takao_(kancolle)_(+999). This condition is applied to all tags other than characters.
But sometimes the condition won't work. For example, some image will give the filename "aang katara sokka", without comma included. This happened to all tags, like copyrights, artists, etc. which creates confusion.
Please help on how to make the comma works as a separator. Thank you.

It's not a question but indeed a bug.
When using "Keep n tags, then add", it would seem that the tag separator is being hard-coded to " ":
https://github.com/Bionus/imgbrd-grabber/blob/310089584b5e7c0f62fdc1d4142de21d46635ccb/src/lib/src/models/filename.cpp#L72-L74
I  just pushed a fix to make it use the correct separator.


[admin] Publish

Docs Build status updates of commit ffb9cb1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/excel/excel-data-types-concepts.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

docs/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Problem in yacl/crypto/ossl-provider/helper.h

In yacl/crypto/ossl-provider/helper.h, the path returned by std::string GetProviderPath() should contain “execroot” and “bin”. But after bazel building, I do not find such paths.
Do they really exist? If they do not exist, how do the functions relying on std::string GetProviderPath() work?

Sure, the paths do exist. The output directory layout could be found in this link.
Additionally, bazel-* is a link. (You could view the original files by using ls -lrt)

OK, I see the execroot paths in ~/.cache/bazel/_bazel_root/...


custom_targets.json overwrites are not taken into account

mbed-os 5.7
custom_targets.json
    "OUT_TARGET_XZY": {
        "default_toolchain": "GCC_ARM",
        "inherits": [
            "NRF52_DK"
        ],
        "macros_add": [
            "MBED_TICKLESS",
            "CONFIG_NFCT_PINS_AS_GPIOS",
            "SOFTDEVICE_NONE",
            "NDEBUG=1",
            "OS_TASKCNT=3",
            "OS_IDLESTKSIZE=32",
            "OS_STKSIZE=4096",
            "OS_TIMERS=3",
            "OS_FIFOSZ=4",
            "OS_MUTEXCNT=3",
            "MBED_FAULT_HANDLER_DISABLED=1"
        ],
        "device_has_add": [
            "FLASH"
        ],
        "device_has_remove": [
            "TRNG"
        ],
        "macros_remove": [
            "S132",
            "SOFTDEVICE_PRESENT",
            "CONFIG_GPIO_AS_PINRESET",
            "BLE_STACK_SUPPORT_REQD"
        ],
     ....
}

Still has S132 defined as compilation parameter. SOFTDEVICE_PRESENT on the other hand is removed successfully by this setup.
This is also true if this chunk is appended to mbed-os/targets/targets.json

Workaround - remove the macro it in the mbed_app.json:
{
	"target_overrides": {
		"*": {
             // ....
		},
		"OUT_TARGET_XZY" : {
			"target.macros_remove" : ["S132"]
		}
	},
        //...
}

which defeates the prupose of using a custom_targets.json

https://github.com/ARMmbed/mbed-os/blob/mbed-os-5.7/targets/targets.json#L3454-L3499
Is the offending target definition. It looks like you should be able to remove S132 from the macros, and if SOFTDEVICE_PRESENT is correctly removed, then that should be too.

But that is exactly the point - S132 is not removed when done within a custom_target.json target definition. It works fine within an mbed_app.json

@drahnr Does it only affect S132? is there any other macro that behaves similarly? If there is we might have a pattern here. Further, any fixes for this behavior (supposing that there is a bug) will come to 5.9.0 or later.

ARM Internal Ref: MBOTRIAGE-442

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.


Ah! Debugging a lot of times or not.

tack traceback:
[C]: in function 'readInt'
/root/usr/local/torch/install/share/lua/5.1/torch/File.lua:259: in function 'readObject'
/root/usr/local/torch/install/share/lua/5.1/torch/File.lua:409: in function 'load'
main.lua:31: in main chunk
[C]: in function 'dofile'
...ocal/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:150: in main chunk
[C]: at 0x004064f0
Error using run (line 37)
Failed to run Torch7 script.

Need more information.


Edge support sticky footer.

Hi there,
I was really happy when i found the sticky footer css file. It was the exact solution I needed.
However I recently found out that it doesn't seem to work on Edge.
Any idea if this can be fixed?
EDIT: I came across your blogpost (https://philipwalton.com/articles/normalizing-cross-browser-flexbox-bugs/) and tried playing around but so far no luck.

I'm not sure what isn't working for you. The sticky footer example here works in Edge:
https://philipwalton.github.io/solved-by-flexbox/demos/sticky-footer/

I explain the situation better in this stackoverflow question. I also posted some images there.

From the looks of the Stack Overflow question, it seems your issue is specific to your site. I'm going to close this because the Solved by Flexbox demo works in Edge, and Stack Overflow is a better place to get answers to implementation questions anyway.


Batch Edit Permissions Does not work

From what I can tell the edit form gets and ID associated with the item it is going to batch edit for permission editing.  This id is only valid in one of the objects and not the other object.  This causes the batch edit to fail with:
NoMethodError - undefined method `update' for nil:NilClass:
  hydra-access-controls (10.5.0) app/models/hydra/access_control.rb:32:in `block in permissions_attributes='
  hydra-access-controls (10.5.0) app/models/hydra/access_control.rb:25:in `permissions_attributes='
  hydra-access-controls (10.5.0) app/models/concerns/hydra/access_controls/permissions.rb:21:in `permissions_attributes_without_uniqueness='

@awead is suggesting that the permission get removed and then reapplied.

Adding items to the batch edit works, but removing items does not work.


Point to GitHub Releases in CHANGELOG.md

I really appreciate wanting to help out! 👏
However, as laid out in GH-21, I’m not very interested in keeping this in source. Even if it’s just a one liner, I feel it just adds a another not so useful file... 😬

@wooorm responding in #21 to keep discussion centralized :)


Main Menu Crash in use with ModularUI API

When running the anti-matter chemistry modpack and introducing Cubic Chunks into it-
it causes the main menu to crash due to modularUI.
I have no idea how these two could possibly have issues but removing ModularUI just fixes it.
https://www.curseforge.com/minecraft/modpacks/antimatter-chemistry
https://www.curseforge.com/minecraft/mc-mods/modularui
its worth noting that the ModularUI api is used only for the scrollwheel sorting mod included in the modpack
https://www.curseforge.com/minecraft/mc-mods/inventory-bogo-sorter

Can you provide a crash log?

Here
crash-2023-01-16_20.27.52-client.txt
crash-2023-01-16_20.22.26-client.txt

I would need to see the whole debug.log file. This is likely some weird mod i traction breaking Mixin on LaunchWrapper because coremods do weird things

log.gz? I apologize I tried googling it but I didn't find the location of just debug.log, if you could point me to the right folder that'd be great
2023-01-16-6.log.gz

It should be in the logs folder. If it isn't there, try a different launcher.

I tried downloading CurseForge but that doesn't vibe with my system apparently so I cannot use that. Can you please recommend me some other modpack launchers compatible with CurseForge modpacks?

Okay I recreated the issue with a minimal forge environment.
That means just forge 1.12.2, modularUI, mixinbooter, VoidislandControl, and CubicChunks/CubicWorldGen.
Crashes in menu as expected
[crash-2023-01-16_22.49.44-client.txt](https://github.com/OpenCubicChunks/CubicChunks/files/10429618/crash-2023-01-16_22.49.4
debug.log
4-client.txt)
latest.log
think this should be the correct logs

I don't see anything obviously wrong here. Can you try without CubicWorldGen? I feel like the issue is MixinBooter but i don't have no idea what it actually does. It's a really strange way to use mixin that isn't needed for anything afaik.

Interesting. Still almost definitely an issue with MixinBooter. But that will be interesting to debug.


[WIP] Version 0.2.0

Status: WIP
Reviewers: @fractaltheory @haroldtreen @jnurse @Helen-Mobify
Preparing 0.2.0 release with @jnurse and @Helen-Mobify's changes.
Todos:

[ ] Engineer +1
[x] Merge Jen's PR
[x] Merge Helen's PR
[ ] Update readme and build files
[ ] Release prep

Feedback:
none so far
Related PRs

[x] https://github.com/mobify/hijax/pull/8
[x] https://github.com/mobify/hijax/pull/11

How to test

Checkout this repo
Ensure tests pass
Ensure that new features work as expected
Break features and ensure tests correctly detect failures

@jnurse can you check to see if I've got your changes in correctly, and my tweaks make sense? Thanks!

@Helen-Mobify can you confirm that I've pulled in your changes correctly? Thanks!

:+1:  Looks good, thanks @donnielrt

@donnielrt :+1: The pulled in change for additional xhr param being passed back to callback function looks good!


Button which should hide all primary radar aircrafts crashes ES

When the button is clicked to hide primary radar aircrafts, ES crashes.

Fixed


Cleanup new babel interface before new version cut

We had a few different PRs going around with different fixes and changes to the interface, which blew the complexity out a bit. This PR brings all of that work into a single, unified interface with better option names:

dirWeb -> webModulesUrlPath
dirFile -> webModulesDir
resolveDependency -> importType
resolveRelative -> moduleResolution (matches TypeScript's option)

Also, I forgot to strip console.warn statements from my last PR 🤦♂ Thanks @francislavoie for catching!

Awesome! Tested the changes on my project and everything looks good :+1:

Awesome! thanks for the extra sanity check

It seems like CI doesn't run on master? :thinking:

Yea, I think right now we only run on PRs. Would love to see them run on master as well, if you're familiar with Github CI and have time to make a quick PR!

Yep! https://github.com/pikapkg/snowpack/pull/269 :grin:

lol thank you!


subsys: logging: use of timestamping during early boot may crash MMU-based systems

During the work on #51035, it has turned out that the combination of:

a MMU-based system, in this case an ARMv7 Cortex-A CPU (e.g. arm/qemu_cortex_a9),
a system timer driver mapping its register space via the MMU using the DEVICE_MMIO API - if the system timer is the first device to be initialized,
alternatively, the driver of the first device of any kind to be initialized using the DEVICE_MMIO API,
CONFIG_KERNEL_LOG_LEVEL being set to 'debug' and
memory address 0 having no read permission

causes a data abort exception whenever the first device driver using the DEVICE_MMIO API and therefore the MMU attempts to map the respective device's register space. This is due to a LOG_DBG call in kernel/mmu.c, z_phys_map(), line 748. Similar LOG_DBG calls exist in the kernel's MMU code for anonymous page mapping and memory unmapping.
Within said LOG_DBG call, if the log level is configured accordingly, the logging subsystem will attempt to acquire a timestamp from the system timer driver. At this point, there's a problem if the system timer driver maps its own register space using the DEVICE_MMIO API:

Either the attempt of the system timer driver to map its own register space will lead us to z_phys_map() and therefore the LOG_DBG call contained therein if the system timer driver is the first driver to be inititialized,
or we're not even at the point yet where an attempt is made to get the timer's register space mapped as some other device is first in the list of devices to be initialized, if this is the case, it's likely the UART which is initialized before the timer.

Either way, the logging subsystem will now attempt to acquire a timestamp prior to the system timer's register space being added to the MMU page table. This has worked so far on ARMv7 Cortex-A, although the result was garbage: device driver instances whose register space has not yet been mapped will use the base address 0 for any premature driver API calls - such as the system timer driver's timestamp get-function. Before the development of #51035, the first 4k page of memory had RX permissions as the exception vector table was located at address 0. Therefore, a read call for the 64 bit current timer value prior to the timer driver being fully initialized used to return the first two jump instructions from the exception vector table, interpreted as a timestamp.
If the 4k memory page at address 0 is not accessible, as is the intention of #51035, the system will crash instead.
To Reproduce
Build a simple demo for qemu_cortex_a9 using the forked repo and branch referred to in #51035 with CONFIG_KERNEL_LOG_LEVEL set to 'debug'. However, do not apply the PR's deletion of the LOG_DBG statement from the kernel's MMU code (the only change to kernel/mmu.c), as this is a workaround affecting multiple arches that should not be generalized
Environment
#51035 is based on Zephyr version 3.2.99.
Expected behavior
System booting successfully. The logging subsystem should consider that due to the kernel's MMU code, it is possible to trigger logging calls prior to the underlying hardware required for timestamping being available. Maybe timer access should generally be omitted in PRE_KERNEL_1, as it's not even assured by default that the system timer is the first device to be initialized.
Impact
Currently prevents the move of the exception vector table away from address 0 with a corresponding fixed MMU page table entry to leaving the vector table at its original location within the Zephyr binary image, pointed to by the VBAR register, if the Kernel's log level is set to 'debug'.

Closing with the correct reason as this was unproperly marked as "completed". @ibirnbaum please feel free to re-open.


SyntaxError: invalid syntax

I have this error:
./main.py   File "./main.py", line 71     fixed_name = f'{info["artist"][0]} - {info["name"]}'                                                        ^ SyntaxError: invalid syntax

I have the same issue on a raspberry pi.
the same project, same folder to another pc and smd works fine.
someone know what due that?
thanks

it must be python3 (in my case - 3.6.7), but desktop version isn't available at the moment

Yep, it is due to the Python version.
I have updated to Python3.6.5 and I don’t have this problem now.

i have this problem too.
i use Python 3.7.2 and it doesn't work.
what should i do ?
./main.py File "./main.py", line 71 fixed_name = f'{info["artist"][0]} - {info["name"]}' ^ SyntaxError: invalid syntax
thanks!

=3.6 should work

Not working for me too
iMac:smd Ducho$ ./main.py -sa "https://open.spotify.com/album/2xbzJIow0xg4zryI2hWgnX"
  File "./main.py", line 30
    notify.send(f'Downloading from YouTube', downloaded=False)
                                          ^
SyntaxError: invalid syntax

Donate? Why?
iMac:smd Ducho$ ./main.py -sa "https://open.spotify.com/album/2xbzJIow0xg4zryI2hWgnX"
Traceback (most recent call last):
  File "./main.py", line 3, in <module>
    from youtube import Youtube
  File "/Users/Ducho/Downloads/smd/youtube.py", line 16, in <module>
    from moviepy.editor import *
  File "/usr/local/lib/python3.7/site-packages/moviepy/editor.py", line 26, in <module>
    imageio.plugins.ffmpeg.download()
  File "/usr/local/lib/python3.7/site-packages/imageio/plugins/ffmpeg.py", line 73, in download
    force_download=force_download)
  File "/usr/local/lib/python3.7/site-packages/imageio/core/fetching.py", line 127, in get_remote_file
    _fetch_file(url, filename)
  File "/usr/local/lib/python3.7/site-packages/imageio/core/fetching.py", line 183, in _fetch_file
    os.path.basename(file_name))
OSError: Unable to download 'ffmpeg-osx-v3.2.4'. Perhaps there is a no internet connection? If there is, please report this problem.


HDFS-16737. Fix number of threads in FsDatasetAsyncDiskService#addExecutorForVolume

Description of PR
The number of threads in FsDatasetAsyncDiskService#addExecutorForVolume is elastic right now, make it fixed.
Presently the corePoolSize is set to 1 and maximumPoolSize is set to maxNumThreadsPerVolume, but since the size of Queue is Integer.MAX, the queue doesn't tend to get full and threads are always confined to 1 irrespective of maxNumThreadsPerVolume.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 56s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  2s

detect-secrets was not available.

+0 :ok:
xmllint
0m  2s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
41m 11s

trunk passed

+1 :green_heart:
compile
1m 43s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
1m 32s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 20s

trunk passed

+1 :green_heart:
mvnsite
1m 39s

trunk passed

+1 :green_heart:
javadoc
1m 21s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 40s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 47s

trunk passed

+1 :green_heart:
shadedclient
26m 27s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 24s

the patch passed

+1 :green_heart:
compile
1m 28s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
1m 28s

the patch passed

+1 :green_heart:
compile
1m 17s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  1s

the patch passed

+1 :green_heart:
mvnsite
1m 26s

the patch passed

+1 :green_heart:
javadoc
0m 59s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 26s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 34s

the patch passed

+1 :green_heart:
shadedclient
25m 55s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
333m 53s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 59s

The patch does not generate ASF License warnings.

452m 42s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4784

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 5c60f210dffb 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 4987d2a33c28fffa706d48724d1f264e48aec89a

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/1/testReport/

Max. process+thread count
2326 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/1/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@ayushtkn Sir, can you help me review this patch?

@ayushtkn Thanks for your review. About HDFS-16386, the increased memory maybe caused by the stacked async deletion takes, increasing or decreasing the maximum number of threads may cannot fix it. Can increase the corePoolSize of the threadPoolExecutor maybe can reduce the memory.
cc @jianghuazhu
I have updated the patch, please help me review it again. Thanks

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
1m  4s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  1s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
40m 35s

trunk passed

+1 :green_heart:
compile
1m 38s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
compile
1m 27s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 17s

trunk passed

+1 :green_heart:
mvnsite
1m 37s

trunk passed

+1 :green_heart:
javadoc
1m 17s

trunk passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 36s

trunk passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 41s

trunk passed

+1 :green_heart:
shadedclient
25m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 22s

the patch passed

+1 :green_heart:
compile
1m 26s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javac
1m 26s

the patch passed

+1 :green_heart:
compile
1m 17s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 17s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  1s

the patch passed

+1 :green_heart:
mvnsite
1m 24s

the patch passed

+1 :green_heart:
javadoc
0m 59s

the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS>

+1 :green_heart:
javadoc
1m 25s

the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 31s

the patch passed

+1 :green_heart:
shadedclient
25m 42s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
327m 52s

hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
0m 56s

The patch does not generate ASF License warnings.

444m 28s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4784

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 5122abd2c850 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 845f3331ff2dbe3f367a7a0681c3a37f4ee67779

Default Java
Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu<IP_ADDRESS> /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/2/testReport/

Max. process+thread count
2283 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/2/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

@ayushtkn @jianghuazhu Sir, I added one UT to test the active count  and core pool size of the executer. Can you help me review it again.
I found this issue because we encountered many stacked pending deletion in our prod environment and caused ReplicationNotFoundException.  So this PR is used to increase the active thread count of the executer. And we will raise another PR to resolve ReplicationNotFoundException.
@ferhui @haiyang1987 Masters, can you help me review this PR?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 48s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+0 :ok:
xmllint
0m  0s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
42m  8s

trunk passed

+1 :green_heart:
compile
1m 38s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
compile
1m 28s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
checkstyle
1m 17s

trunk passed

+1 :green_heart:
mvnsite
1m 37s

trunk passed

+1 :green_heart:
javadoc
1m 22s

trunk passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 38s

trunk passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 43s

trunk passed

+1 :green_heart:
shadedclient
26m  7s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
1m 25s

the patch passed

+1 :green_heart:
compile
1m 26s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javac
1m 26s

the patch passed

+1 :green_heart:
compile
1m 18s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
javac
1m 18s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
1m  1s

the patch passed

+1 :green_heart:
mvnsite
1m 24s

the patch passed

+1 :green_heart:
javadoc
0m 56s

the patch passed with JDK Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04

+1 :green_heart:
javadoc
1m 31s

the patch passed with JDK Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

+1 :green_heart:
spotbugs
3m 35s

the patch passed

+1 :green_heart:
shadedclient
25m 43s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
347m 45s
/patch-unit-hadoop-hdfs-project_hadoop-hdfs.txt
hadoop-hdfs in the patch passed.

+1 :green_heart:
asflicense
1m  0s

The patch does not generate ASF License warnings.

466m  2s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.namenode.TestFileTruncate

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/4784

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient spotbugs checkstyle codespell detsecrets xmllint

uname
Linux 0e0ad1a30cba 4.15.0-191-generic #202-Ubuntu SMP Thu Aug 4 01:49:29 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / b13903175a636d20c0a993cdbb61608c9d706bb2

Default Java
Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.16+8-post-Ubuntu-0ubuntu120.04 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_342-8u342-b07-0ubuntu1~20.04-b07

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/3/testReport/

Max. process+thread count
2245 (vs. ulimit of 5500)

modules
C: hadoop-hdfs-project/hadoop-hdfs U: hadoop-hdfs-project/hadoop-hdfs

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-4784/3/console

versions
git=2.25.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

The failed UT hadoop.hdfs.server.namenode.TestFileTruncate works well locally.
@Hexiaoqiao Sir, can you help me review this patch too?


feat(alerting): Add group-specific WebHook URL for Discord

Add group-specific webhook URL for discord alert
Provides support for paging multiple Discords based on the group selector while keeping backward compatibility to the old Discords configuration manifest
integration per team can be specified in the overrides sections in an array form.
alerting:
  discord:
    webhook-url: "https://discord.com/api/webhooks/**********/**********"
    # You can also add group-specific to keys, which will
    # override the to key above for the specified groups
    overrides:
      - group: "core"
        webhook-url: "https://discord.com/api/webhooks/************"

endpoints:
  - name: website
    url: "https://twin.sh/health"
    interval: 30s
    conditions:
      - "[STATUS] == 200"
      - "[BODY].status == UP"
      - "[RESPONSE_TIME] < 300"
    alerts:
      - type: discord
        enabled: true
        description: "healthcheck failed"
        send-on-resolved: true

  - name: back-end
    group: core
    url: "https://example.org/"
    interval: 5m
    conditions:
      - "[STATUS] == 200"
      - "[CERTIFICATE_EXPIRATION] > 48h"
    alerts:
      - type: discord
        enabled: true
        description: "healthcheck failed"
        send-on-resolved: true

ref: #96
Signed-off-by: Bo-Yi Wu<EMAIL_ADDRESS>

@TwiN All Done, Please help to review again.

@TwiN All Done.

Codecov Report

Merging #271 (9b764cf) into master (5281f80) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #271      +/-   ##
==========================================
- Coverage   83.70%   83.68%   -0.03%
==========================================
  Files          53       53
  Lines        3462     3476      +14
==========================================
+ Hits         2898     2909      +11
- Misses        439      441       +2
- Partials      125      126       +1

Impacted Files
Coverage Δ

alerting/provider/discord/discord.go
91.78% <100.00%> (+1.95%)
:arrow_up:

config/maintenance/maintenance.go
92.40% <0.00%> (-3.80%)
:arrow_down:

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 5281f80...9b764cf. Read the comment docs.

@TwiN Done. Please review again.


Not getting any replies both on Teams and Web Bot App

Hi,
Completed all the steps of the deployment guide and tried it 2 times without errors. I'm not getting any replies from the chat bot  - both from Teams and the Test Web Chat on the Web Bot App Service. I've checked application insights and I'm just seeing a few 404 errors due to OPTIONS /api/message. Any tips on troubleshooting this issue. QnA Maker Cognitive Service works fine. I also tried connecting a basic QnA bot from the samples and it worked fine.

Found the issue. It seems it doesn't work with the Test Web Chat on the Web Bot App Service. But Teams working fine.


Cookie banner info

What happened?
Just remove the "any" at the end of the message

What browsers are you seeing the problem on?
No response
Version
No response
Relevant log output
No response

:tada: This issue has been resolved in version 1.1.0-beta.3 :tada:
The release is available on:

GitHub release
v1.1.0-beta.3

Your semantic-release bot :package::rocket:


Option + Right Bracket keyboard shortcut does not show alternate suggestions.

Operating System (OS) Version: macOS 13.5 (arm64)

Neovim Version: NVIM v0.9.1

Build type: Release
LuaJIT 2.1.0-beta3

Programming Language Versions:

Python: 3.11.5

Issue Details:
Utilizing the Option + Right Bracket keyboard shortcut does not yield the expected outcome of viewing alternative suggestions. Could you please assist in resolving this issue?

You might need to check the setting "use option as meta key"

https://www.shell-tips.com/mac/meta-key/#gsc.tab=0

I have the same problem.

item2 still not work

I am interested in rebinding the "Next Suggestion" functionality to a different key, such as Ctrl + Right Bracket.
I have looked through the documentation and source code but could not find the specific function responsible for the "Next Suggestion" feature. Could you please provide the name of this function so that I can rebind it in my init.lua configuration file?

The Keybindings section of the readme very explicitly explains how to rebind the Next Suggestion command: https://github.com/Exafunction/codeium.vim#-keybindings

I too am failing to get any of the keybindings to work correctly.
Neovim 0.9.5 (Mac)
Lazy.vim
A couple issues:

I think this repo should explicitly state Windows only? It fails trying to load an .exe binary at launch. The other Codeium.nvim repo installs and runs fine. However it is missing the documentation on setting up alternative keybindings, but I'm assuming it's the same.
I'm fairly certain <c- is not a valid keymap for CONTROL, should all be <C- ?
Even still, with default config:  accepts suggestion for me, not . (I can live with that)
Neither the default cycle bindings work for me, nor does the fixed up suggested starter custom config. Oddly the directional up/down arrows do but who wants to use them? 🤣 I actually have a 60% keyboard anyway.

This is the separate lazy.vim config file I have:
codeium.lua:
return {
  {
    "Exafunction/codeium.nvim",
    dependencies = {
      "nvim-lua/plenary.nvim",
      "hrsh7th/nvim-cmp",
    },
    lazy = false,
    config = function()
      -- Change '<C-g>' here to any keycode you like.
      -- vim.keymap.set('i', '<C-g>', function () return vim.fn['codeium#Accept']() end, { expr = true, silent = true })
      vim.keymap.set("i", "<C-;>", function()
        return vim.fn["codeium#CycleCompletions"](1)
      end, { expr = true, silent = true })
      vim.keymap.set("i", "<C-,>", function()
        return vim.fn["codeium#CycleCompletions"](-1)
      end, { expr = true, silent = true })
      vim.keymap.set("i", "<C-x>", function()
        return vim.fn["codeium#Clear"]()
      end, { expr = true, silent = true })
    end,
  },
}

Wasn't working for me. I was able to do this instead, though:
" codeium
imap <S-Tab> <Cmd>call codeium#CycleCompletions(1)<CR>
set statusline+=\{…\}%3{codeium#GetStatusString()}

This makes Shift + Tab the next suggestion. Single Tab remains accept as per defaults. No combination of Ctrl + key worked.


Go through and remove redundant if (IsDZelnodeActive()) checks

Now that deterministic zelnodes is active on mainnet for a while. Some places don't need to check if it is active anymore.
This include rpc calls, and some consensus code. We should be careful when removing this check as in some places if could save time while syncing blocks earlier in the chain.

In the current master there are no references to IsDZelnodeActive, this issue can be closed

@w2vy what about IsDFluxnodeActive?

Good point, those I will look at to understand better and then we can talk about which can go.

@blondfrogs
I reviewed all the IsDFluxnodeActive calls and they are all inside src/rpc/zelnode.cpp and defined in src/zelnode/zelnode.cpp
I went back to v5.2.0 (before I touched anything) and the instances of IsDZelnodeActive were the same.
I do see in main.cpp (ConnectBlock), miner.cpp (CreateNewBlock) and zelnode.cpp (rebuildfluxnodedb)
where the block height is compared to StartFluxnodePayments
So it seems IsDFluxnodeActive is only used in the RPC calls.

@w2vy That sounds correct. Lets get together and see if we need them anymore.

closed with PR#192


Run integration tests with vars-gridview.

@kevinsbarnard I'm almost done a revamp of both the annosaurus and vampire-squid services. One important step is to run integration testing with VARS grid view against the changed services. If you have tests for the api calls in place, I can probably just run them and make changes. If you don't then we may have to sit down together and discuss the upcoming changes. Hopefully this won't be painful, they're should be only a few relatively minor changes.

@hohonuuli I don't have unit tests for the M3 API calls in place. My inclination is to spin up the M3 services on rocinante, then test all of the read/write endpoints by running GridView. There aren't too many of them that GridView hits, so this shouldn't be too bad. Let me know whenever you'd like to chat about this!

@kevinsbarnard vars-gridview seems to work as is.


Easier/faster way to get objects from Glade file?

I'm using gotk3 with Glade. I use a function like this to get all my buttons:
func getButton(b *gtk.Builder, name string) (*gtk.Button, error) {

	obj, err := b.GetObject(name)
	if err != nil {
		return nil, err
	}

	button, ok := obj.(*gtk.Button)
	if !ok {
		return nil, err
	}

	return button, nil
}

Then I can just use something like
firstButton, _ := getButton(bldr, "first_button")
secondButton, _ := getButton(bldr, "second_button")
// and so on...

That works, but I also have a headerbar, panes, GtkBoxes, and loads of other types of Gtk objects. And for each object class/type I need a new function like "getButton". (i.E. getWindow, getPopover, getGtkbox, and so on.)
Isn't there an easier way to do this?
I used to use PyGObject and it was fairly simple:
self.builder = Gtk.Builder()
self.builder.add_from_file("gui.glade")

# And now I can get all types with just one line like so:
self.headerbar = self.builder.get_object("headerbar")
self.some_button = self.builder.get_object("some_button")
self.some_paned = self.builder.get_object("some_paned")
self.some_window = self.builder.get_object("some_window")
# and so on...

Is there a way to do this in gotk3 as well? So that I don't need to have an extra init function for each type of Gtk object/class?

I think.the initial Design idea in gotk3 was to not risk letting you run into a nil dereferencing error. Therefore this probably won't be introduced.

Alright. Thanks for your input!


refactor(cli): query run progress message

Summary

I feel the way this function is written allows us to make easy mistakes such as 1) having the arguments in a different order and getting the start/end time wrong, and 2) passing less than two arguments which will throw a panic trying to access elements that don't exist since we are not checking that the array has at least two elements.

How did you test this change?
Unit testing
Issue
https://lacework.atlassian.net/browse/ALLY-1015

Refactors looks O.k. - But now I have a comment from a different function.

No worries.  Fixed.


Can't interact with scrollable tooltips in Single Window mode

Tested versions

Reproducible in v4.3.stable.official [77dcf97d8]

System information
Windows 11, Intel Core i5 10300H, nVidia GTX 1650
Issue description
In Single Window Mode, tooltips with long texts that need scrolling to access the whole content can't be scrolled. The scroll wheel will instead interact with the editor as if the tooltip wasn't there (viewport will zoom in or out, inspector window will scroll, etc).
Steps to reproduce

Create a new project
Enable Single Window Mode in Editor settings
Hover the mouse cursor on any UI element that has a long text for tooltip. Example: Tonemap mode of the Sky background type in Environment resources.

Minimal reproduction project (MRP)
N/A

related https://github.com/godotengine/godot/pull/93967

This is a duplicate of #92545


Indicate files and templates are managed by Chef

It would be nice if Foodcritic enforced phrases similar to "This file is managed by chef; local modifications will be overwritten" in templates and files that Chef renders. That way, an end-user can be certain that the file is, in fact, managed by Chef.

Not all formats support comments and those that do have no unified formatting. Probably not worth chasing this.

Ah, I see what you mean. Maybe just templates then, since those are rendered as text?

But we have no idea what kind of file it is.

I think we're going to have to close this one out. Like @coderanger said not everything can have a comment. In fact a lot of comments can't so this would be incredibly problematic.


Error: order [PAYPAL ORDER] does not exist
Due to PrestaShop Core split sometime a Cart in multiple Orders : https://github.com/PrestaShop/docs/issues/508

Thank you @Matt75


add chat send history sample

closes #1091
Added

samples: Added chat send history sample, by @a-b-r-o-w-n, in PR #1678

I'm unclear why this isn't provided by the component out of the box!

@cwhitten one thing I can think of in opposition to adding this to the bundle is concerns over security / managing the session state. Would it be by user / conversation / bot / something else? IMO this is better suited as a custom add on that suits the customer's need. Also, while this feature is helpful for testing, I am not convinced that it is useful for the majority of bot interactions.


pymongo issue

i use mongodb as db to store data, i found that the project status bar always show pending, then I the data of pojectdb taskdb and resultdb respectively, i found there only is {"taskid": "on_start"}  in each task collection, and no result collection in resultdb. Track back your source code , I found you used pymongo collection_names() to get all collections from database, I manually used this command to get collections from resultdb , the result is an empty list []. I also used two version mongos 3.0.3 and 3.2.10 to verify, I can get collection by using version 3.0.3, but 3.2.10 not. My pymongo version is  'pymongo==2.7.1'. I guess that issue caused by compatibility between mongod and pymongo. If you see this issue, please give me some suggestions to fix this, thanks.

Did you run (click run button on dashboard) the project? The tasks may be discard if it's crawled.
Use http://docs.pyspider.org/en/latest/apis/self.crawl/#itag if you want to crawl again.

yes, I clicked run button, the status info shows all pending. finally, i found this issue is caused by pymongo. The spider works well after upgraded pymong from 2.7.1 to 3.3.0. Thank for your quick response.


Random "No device found" error inside CVM

So recently I've been doing kernel programming and a lot of debugging things and sometimes when I reloaded the KVM and/or VFIO module and re-log into the CVM and tried to nvidia-smi conf-compute -f  after nvidia-persistenced there would be error:
$ sudo ./NvidiaCC/guest_nv_init.sh
[sudo] password for h100:
nvidia-smi conf-compute -f
No devices were found
ERROR: nvidia-smi conf-compute -f

The above script just does nvidia-persistenced and enable confidential computing mode. The dmesg command shows that the RM initialization timed out.
[   36.077455] NVRM _threadNodeCheckTimeout: _threadNodeCheckTimeout: currentTime: 3d08c22901a000 >= 3d08c20857cf00
[   36.077466] NVRM _threadNodeCheckTimeout: _threadNodeCheckTimeout: Timeout was set to: 4000 msecs!
[   36.077473] NVRM kgspBootstrapRiscvOSEarly_GH100: Timeout waiting for GSP target mask release. This error may be caused by several reasons: Bootrom may have failed, GSP init code may have failed or ACR failed to release target mask. RM does not have access to information on which of those conditions happened.
[   36.077507] NVRM kfspDumpDebugState_GH100: FSP microcode v4.76
[   36.077509] NVRM kfspDumpDebugState_GH100: NV_PFSP_FALCON_COMMON_SCRATCH_GROUP_2(0) = 0x0
[   36.077511] NVRM kfspDumpDebugState_GH100: NV_PFSP_FALCON_COMMON_SCRATCH_GROUP_2(1) = 0x0
[   36.077513] NVRM kfspDumpDebugState_GH100: NV_PFSP_FALCON_COMMON_SCRATCH_GROUP_2(2) = 0x0
[   36.077515] NVRM kfspDumpDebugState_GH100: NV_PFSP_FALCON_COMMON_SCRATCH_GROUP_2(3) = 0x0
[   36.077518] NVRM kgspInitRm_IMPL: cannot bootstrap riscv/gsp: 0x65
[   36.077529] NVRM RmInitAdapter: Cannot initialize GSP firmware RM
[   38.591844] NVRM: GPU 0000:01:00.0: RmInitAdapter failed! (0x62:0x65:1660)
[   38.593638] NVRM: GPU 0000:01:00.0: rm_init_adapter failed, device minor number 0
[   38.715488] NVRM gpumgrCheckRmFirmwarePolicy: Disabling GSP offload -- GPU not supported
[   38.715518] NVRM osInitNvMapping: *** Cannot attach gpu
[   38.715520] NVRM RmInitAdapter: osInitNvMapping failed, bailing out of RmInitAdapter
[   38.715546] NVRM: GPU 0000:01:00.0: RmInitAdapter failed! (0x22:0x56:631)
[   38.717604] NVRM: GPU 0000:01:00.0: rm_init_adapter failed, device minor number 0
[   38.913733] NVRM gpumgrCheckRmFirmwarePolicy: Disabling GSP offload -- GPU not supported
[   38.913771] NVRM osInitNvMapping: *** Cannot attach gpu
[   38.913774] NVRM RmInitAdapter: osInitNvMapping failed, bailing out of RmInitAdapter
[   38.913806] NVRM: GPU 0000:01:00.0: RmInitAdapter failed! (0x22:0x56:631)
[   38.917068] NVRM: GPU 0000:01:00.0: rm_init_adapter failed, device minor number 0
[   39.032908] NVRM gpumgrCheckRmFirmwarePolicy: Disabling GSP offload -- GPU not supported
[   39.032932] NVRM osInitNvMapping: *** Cannot attach gpu
[   39.032935] NVRM RmInitAdapter: osInitNvMapping failed, bailing out of RmInitAdapter
[   39.032962] NVRM: GPU 0000:01:00.0: RmInitAdapter failed! (0x22:0x56:631)
[   39.035083] NVRM: GPU 0000:01:00.0: rm_init_adapter failed, device minor number 0
[   39.099929] nvidia-uvm: Loaded the UVM driver, major device number 237.

I attached vfio-pci.disable_idle_d3=1 to the kernel boot command line to avoid VFIO to make the GPU into idle mode.
I also tried unbinding and rebinding the GPU to vfio-pci but that didn't work either. However, this issue would be resolved mysteriously after several times of reboot.
### Tasks

Please ensure you are following the flows in our deployment guides.
it looks like nvidia-persistenced is being run without --uvm-persistence-mode which is required.
vfio-pci.disable_idle_d3=1 should no longer be required with the GA of our CC software stack.


No base class for types.

Currently, AST nodes have a private type field that is a symbol. In order to enforce that this symbol is actually a type, the filterValidTypes function in TypeMechanics.kt is invoked when a symbol is assigned to this field. There should really be a base class for valid types, and the AST should use this base class instead of symbol for the type field.

This can be solved with the sealed interface feature in Kotlin.


MissingUriError: View 258 has no URI

It happens rarely and unfortunately I have no idea how to reproduce it. FWIW, here goes:
[LSP-pyright][DEBUG] Due to "dev_environment", new "analysis.extraPaths" is (operation = 'prepend'): ['/opt/sublime_text/Lib/python38', '/opt/sublime_text/Lib/python3', '/home/giampaolo/.config/sublime-text/Lib/python38', '/opt/sublime_text/Packages', '/home/giampaolo/.config/sublime-text/Packages']
Traceback (most recent call last):
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/core/types.py", line 131, in run
    f()
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/documents.py", line 429, in _on_selection_modified_debounced_async
    self._do_highlights_async()
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/documents.py", line 857, in _do_highlights_async
    params = cast(DocumentHighlightParams, text_document_position_params(self.view, point))
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/core/views.py", line 271, in text_document_position_params
    return {"textDocument": text_document_identifier(view), "position": position(view, location)}
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/core/views.py", line 228, in text_document_identifier
    uri = uri_from_view(view_or_uri)
  File "/home/giampaolo/.config/sublime-text/Installed Packages/LSP.sublime-package/plugin/core/views.py", line 221, in uri_from_view
    raise MissingUriError(view.id())
LSP.plugin.core.views.MissingUriError: View 258 has no URI

I doubt that this is a LSP-pyright issue.


Attach to running container.

This would be a pretty sweet feature if we could build it: attach to a running container from emp ps. Something like:
$ emp attach <task id> -a <app>

Thinking about this a bit. If we pointed Empire at swarm for the Docker daemon, then we could just use the /containers endpoint and filter on the com.amazonaws.ecs.task-arn label.


docs: add forum to topnav

To drive more traffic to the forum, and to align with the rest of the site, this PR adds FORUM to the desktop and mobile topnavs.
Fixes #582

This change is

@jess-edwards: I just noticed that, when the screen is between standard desktop and mobile widths, the topnav now looks bad.

This is also true of the rest of the site:

Should I take a shot at fixing that just for the docs, or should we just have DivisionOf handle that across the board?

@jseldess I'll file an issue with Division Of.


Can't include no_op in input_map when i import graph_def

System information

TensorFlow version (use command below): current master

Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
When i pass no_op  into input_map, it makes an error.
i tracebacked some codes and
the function _PopulateTFImportGraphDefOptions() in tensorflow/tensorflow/python/framework/importer.py can't support no_op as a control dependency.
under the first loop, it registers the input map operation into options when the input_src starts with '^'.
However, no_op doesn't have _as_tf_output() and the line
dst_op = input_dst._as_tf_output().oper occurs an exception.
Temporarily, i add the lines
      src_name, src_idx = _ParseTensorName(input_src[1:])
      src_name = compat.as_str(src_name)
      dst_output = c_api_util.tf_output(input_dst._c_op, -1)
      c_api.TF_ImportGraphDefOptionsAddInputMapping(options, src_name, -1,
                                                    dst_output)
instead of the lines
      src_name = compat.as_bytes(input_src[1:])
      dst_op = input_dst._as_tf_output().oper  # pylint: disable=protected-access
      c_api.TF_ImportGraphDefOptionsRemapControlDependency(
          options, src_name, dst_op)
However, I hope this problem to be fixed in the next version.
Thank you
Source code / 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. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce
Mobile device

It has been 17 days with no activity and the awaiting response label was assigned. Is this still an issue?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Get active carousel page from another component

Hello, recently used your nice package and it seems to fit my needs. However, I needs to get current page index to sync with another components, so when the page is displayed, the other component should react to that updates. Searching through your code and still have no clue about it. Is there any possibilities to get the current page index, or maybe listening to event handling the changes? Thanks in advance.

Hello, thank you!
Is it ok for you if I add this event?
<Carousel
  on:pageChange={event => console.log(`Current page index: ${event.detail}`)}
>
  // ...
</Carousel>

Wow, that's was exactly what I need. Thank you for this nice support and I'm waiting for the release :)

@elzdave, released, version 1.0.2

Allright, updating my deps right now! Have a nice day 😎

Thanks! Have a good day! 🙂


[Frontend] Short info to the user when there's an error

What problem or use case are you trying to solve?
There are system errors that backend encounters, and the user should know the system isn't working right or they can change something. The easiest example is authentication errors, when I pick a non-existent model or have no access to it or my settings are otherwise wrong.
Describe the UX of the solution you'd like
A system status field, maybe to the left of the Settings wheel, where we can display 'Authentication unsuccessful, please check settings' or some other clue about what went wrong. On backend, we can send some specific messages.
Do you have thoughts on the technical implementation?
On backend we can try to use the task stop/reset feature on at least some exceptions.
Describe alternatives you've considered
Maybe the field currently displaying info about the task (task has stopped, etc) could be used for this? I imagine we will also say the task is stopped though, so it seems like we need some other place for the reasons we can inform about. It doesn't have to be large (at all).
Additional context
Auth is the easiest example, but maybe not the best: there are other suggestions on it discussed in other issues, like some sort of ping before starting the actual task, I'm not sure what that would imply, but I think there can be errors that we cannot know before they happen, like failing to condense despite best efforts and stopping on context window exceeded.
Related:

https://github.com/OpenDevin/OpenDevin/pull/1445
https://github.com/OpenDevin/OpenDevin/pull/1296

https://github.com/OpenDevin/OpenDevin/pull/1445

@enyst there is a status now in the bottom middle of OpenDevin that shows high level status and I've seen some errors there. Is this what you were looking for?

Yes, it's fine as it is! Thanks for the heads-up, this can be closed.

I'm reopening this, because lately there have been multiple requests for important messages to be in a more accessible or visible way than in the logs, specially with docker app. Users request that relevant errors are more readily available or visible somehow. And we are listening, but we have started to make some changes to the UI adding some system / error messages in the chat box instead, since that's the ready place. Frankly, I think that doesn't look great, exceptions and raw code in-between natural language chat with the agent. Maybe there is a better way?
Perhaps we do need an actual "system status" after all, where the user can see the most important stuff (like budget exceeded).

Great idea!
Maybe next to the settings cog wheel a "Notifications" indicator that on click opens pop-out sidebar from the right that shows those collected messages, with a clear list button?
Everywhere where "toast.error" occurs, should also use that.

For example https://github.com/OpenDevin/OpenDevin/issues/2494

Going to use this as a catch all for better errors to the users going forward since you have some eyes on it.

https://github.com/OpenDevin/OpenDevin/pull/2657#issuecomment-2226781143

I encountered another instance of this. This time my request was blocked by cloudflare, causing API calls to fail over and over again:
litellm.exceptions.APIError: litellm.APIError: APIError: OpenAIException - <!DOCTYPE html>

<title>Attention Required! | Cloudflare</title>
<meta charset="UTF-8" />
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
<meta http-equiv="X-UA-Compatible" content="IE=Edge" />
<meta name="robots" content="noindex, nofollow" />
<meta name="viewport" content="width=device-width,initial-scale=1" />
<link rel="stylesheet" id="cf_styles-css" href="/cdn-cgi/styles/cf.errors.css" />

Please enable cookies.

Sorry, you have been blocked

You are unable to access onrender.com

Why have I been blocked?

This website is using a security service to protect itself from online attacks. The action you just performed triggered the security solution. There are several actions that could trigger this block including submitting a certain word or phrase, a SQL command or malformed data.

What can I do to resolve this?

You can email the site owner to let them know you were blocked. Please include what you were doing when this page came up and the Cloudflare Ray ID found at the bottom of this page.

    Cloudflare Ray ID: 8d48d4151f7e425b
    •

      Your IP:
      <button type="button" id="cf-footer-ip-reveal" class="cf-footer-ip-reveal-btn">Click to reveal</button>
      <IP_ADDRESS>
      •

    Performance & security by Cloudflare

I think we should try to fix this soon!


extend the response from API to also send default value for config

Fixes #2869
e.g response for a single config cpus:
{
  "Configs": {
    "cpus": {
      "value": 2,
      "isDefault": true,
      "defaultValue": 2
    }
  }
}

Note: This will also need changes on the tray

/retest

/hold
(needs changes on the tray side, before merging need to decide how we'll handle that)


Background image of the website is not related

Sometimes, based on the city, the background image shown is not related with the city. The image is requested from Unsplash API.

sure

start from tomorrow

I think there is no other option than change the api from unsplash to some other... I have tried making the query name to lowercase which is improving the images but still some of them are not related

It's better than noting. You can also try it by changing the API. If it works perfectly, I'll merge it; otherwise just change the query name as you've mentioned. I'll merge that too

Hi @DeepNinja07x I looked after this issue and found 3 things:-

Using Google Places API. Article on it: - https://codeburst.io/adding-city-images-to-your-react-app-14c937df2db2
Using Teleport API. http://developers.teleport.org/api/getting_started/
Using unsplash API https://unsplash.com/developers

Have a look at this and let me know which one would you prefer. I would like to work on this.

I made you a PR with the google place API


Enhanced formatting for alerts

If the event type is the pre-defined "alert" type, the email shows the details of the alert, results that triggered it, and a link to view these interactively.

These alerts look really interesting.  Is there a doc page on www.getseq.net that can tell me more about how to create and use them?

Thanks for the note Norman. Documentation is on this week's list. The UI should be reasonably self-explanatory; from a dashboard chart, expand the chart details and select the Alerts tab. It would be really valuable to hear of any "gotchas" or points of confusion/difficulty that you encounter trying to set it up (we can use that to improve the UI and/or docs :-))
I'll post a link here when 4.0 documentation is online 👍

I think I missed if this feature is only available in the 4.0.55
pre-release?
On May 2, 2017 6:03 PM, "Nicholas Blumhardt"<EMAIL_ADDRESS>wrote:

Thanks for the note Norman. Documentation is on this week's list. The UI
should be reasonably self-explanatory; from a dashboard chart, expand
the chart details and select the Alerts tab. It would be really
valuable to hear of any "gotchas" or points of confusion/difficulty that
you encounter trying to set it up (we can use that to improve the UI and/or
docs :-))
I'll post a link here when 4.0 documentation is online 👍
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/datalust/seq-apps/pull/32#issuecomment-298774803, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABj7xAn42EgBcua-VVDOlIWxO8ALV_jNks5r16hHgaJpZM4NJtqN
.

@normanhh3 thanks for the follow-up. Yes, it's in the latest pre-release only.

Thanks for the tweet earlier with pointers and docs. Looks like great stuff coming!


App loses focus on text-field when specialcharacters are input with blackberrykeyboard

Summary
Mattermost on Android escapes (or loses its focus) text-input-field when entering character using 'sym' key on blackberry key one with real keyboard (no touchpad).
Environment Information

Device Name: blackberry key one
OS Version: android 8.1
Mattermost App Version: 1.43
Mattermost Server Version: 5.35

Steps to reproduce
Should be reproducible using the device, maybe others from bb
Expected behavior
Happily input any text without getting thrown out of text-input and having to click on it again and again.
(),*äü;  (these and many more very often used signs are accessed via sym-key)
Observed behavior (that appears unintentional)
In all other chat-apps this is not a problem and I can just happily type away in my language. Im using symkey alot to quickly type Ü Ä Ö and other "umlaute" and special characters. only mattermost kicks me out of the textfield after input of the chosen letter.
Possible fixes

@josephbaylon Are you able to reproduce?

@iameru Is this still an issue for you? Have you tried with the latest beta version (v2)?

@iameru Is this still an issue for you? Have you tried with the latest beta version (v2)?

I dont own this device anymore. If you would like I could borrow it from a friend and try it again.

Always happy to have someone test it, but if it is much of an issue for you, we can close this issue now, and reopen it if someone have the same issue again in the future.


Suggestion: More verbose error handling for actions

I can implement something for you if you'd like and make a PR, I just thought I would get your opinion first. I think it would be useful to improve the error handling here. In my project, I forgot to register a middleware, so this was being thrown for one of my actions that the middleware normally handles. I was able to figure it out by editting instrument.js and logging the action that caused the error.
if (typeof action.type === 'undefined') {
  console.log(action);
  throw new Error('Actions may not have an undefined "type" property. ' + 'Have you misspelled a constant?');
}

Again, I'm not sure what the best way to handle this would be, just bringing this up.

I'm not getting any error details.. have to rely on redux logger middleware for that


Clarity on roll out and use in Data Migration

Hi
The Constraint entitled "Data migration slow-down with enabled dual-write" requires clarity.
I believe the presumption here is on the FIRST legal entity, i.e. do not have the maps enabled whilst running the data migration element. However where you have mutliple roll outs you cannot disable the dual-write maps, they are in production. Naturally you do not have to add the legal entity into the synch prior to the data migration, but this needs to be made clear.
We should be able to run a data migration plan into legal entity AB2 whilst dual-write is active in legal entity AA1.
It this is not the case then this needs highlighting, and if this is the case clarity on the advice is needed.
Thanks
Steve
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: caa6a38c-03b6-7949-e383-aaaa4ca05ff9
Version Independent ID: 0bf0e3d4-33b7-d54b-e2bc-1531bfbeee6e
Content: Considerations for initial synchronization - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/data-entities/dual-write/initial-sync-guidance.md
Service: dynamics-365-cross-app
GitHub Login: @RamaKrishnamoorthy
Microsoft Alias: ramasri

@josaw1 looking at the constraint section of the page the original issue persists with the statement made. What am I missing?


Updating containers

Needing updates:

staphb/freyja from 1.3.11 to 1.3.12
staphb/ivar from 1.3.1 to 1.3.2
staphb/mafft from 7.475 to 7.505
staphb/vadr from 1.5 to 1.5.1

Closes https://github.com/UPHL-BioNGS/Cecret/issues/140 and https://github.com/UPHL-BioNGS/Cecret/issues/138

Of note, iVar 1.3.2 is too slow and fails these GA. Perhaps a newer release would be better.


Version 2.0.3

Landed in https://github.com/metarhia/impress/commit/0f1c9dbebdd957581e5b6db72ba839ce4eb02f27

Landed in https://github.com/metarhia/impress/commit/0f1c9dbebdd957581e5b6db72ba839ce4eb02f27


[OPPRO-167] Add output metrics

What changes were proposed in this pull request?
Added output rows and batches to each transformer.
How was this patch tested?
Verified on Jenkins.

Hi @zhztheplayer, I found below error on Github action for this PR. Do you know what the issue refers to, or how to debug it? Thanks.

Hi @zhztheplayer, I found below error on Github action for this PR. Do you know what the issue refers to, or how to debug it? Thanks.

There is an error in CI log
java.lang.NullPointerException
	at org.apache.spark.sql.execution.SparkPlan.sparkContext(SparkPlan.scala:67)
	at io.glutenproject.execution.FilterExecBaseTransformer.metrics$lzycompute(BasicPhysicalOperatorTransformer.scala:58)
	at io.glutenproject.execution.FilterExecBaseTransformer.metrics(BasicPhysicalOperatorTransformer.scala:57)
	at org.apache.spark.sql.execution.SparkPlan.longMetric(SparkPlan.scala:135)
	at io.glutenproject.execution.FilterExecBaseTransformer.updateMetrics(BasicPhysicalOperatorTransformer.scala:113)
	at io.glutenproject.execution.WholeStageTransformerExec.updateMetrics(WholeStageTransformerExec.scala:193)
	at io.glutenproject.execution.WholeStageTransformerExec.$anonfun$doExecuteColumnar$3(WholeStageTransformerExec.scala:325)
	at io.glutenproject.backendsapi.velox.VeloxIteratorApi$$anon$2.next(VeloxIteratorApi.scala:252)
	at io.glutenproject.vectorized.CloseableColumnBatchIterator.next(CloseableColumnBatchIterator.scala:53)
	at io.glutenproject.vectorized.CloseableColumnBatchIterator.next(CloseableColumnBatchIterator.scala:30)
	at org.apache.spark.InterruptibleIterator.next(InterruptibleIterator.scala:40)
	at io.glutenproject.backendsapi.velox.VeloxSparkPlanExecApi.$anonfun$createBroadcastRelation$1(VeloxSparkPlanExecApi.scala:154)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitions$2(RDD.scala:863)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitions$2$adapted(RDD.scala:863)
	at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
	at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
	at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
	at org.apache.spark.scheduler.Task.run(Task.scala:131)
	at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:497)
	at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1439)
	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:500)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:750)

I guess it's related to the lazy keyword used in BasicPhysicalOperatorTransformer
https://github.com/oap-project/gluten/blob/7bbf25ef2c1b72b8dc5e2585162bc85cb9550608/jvm/src/main/scala/io/glutenproject/execution/BasicPhysicalOperatorTransformer.scala#L57-L61
sparkContext doesn't seem to be accessible from task thread. It should only be valid for driver thread.

Fixed this issue by call the metric once on driver. Thanks for your great help!


Add my-streaming-services extension

Description

Set your list of streaming services to access or search directly.
For now I have created a somewhat small list of streaming services.

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

After doing the extension I thought the same thing. It doesn't provide anything that Quicklinks don't already do...
Thanks @pernielsentikaer !


Reverse Indexer / DOGE Implementation

The reverse indexer is still a WIP, but I am indexing DOGE at the moment.

Testing the unified bitcoin/doge codebase but now is probably a good time to take a look @aph5nt

Irrelevant and old, could be closed


add scons sputnik option

add scons sputnik option and do sputniktests

test262 has command-line runner.
http://wiki.ecmascript.org/doku.php?id=test262:command
so, use it.


tests: make all subnet uniq per scenario

If two environments are using the same subnet, we will get trouble
because of ips addresses conflicts.
This commit ensures each scenario has a uniq subnet for both public and cluster
network so we can setup several test environment at a time on a same hypervisor.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test all

jenkins test dev-ansible2.3-lvm_osds

jenkins test luminous-ansible2.3-bluestore_dmcrypt_journal_collocation

jenkins test luminous-ansible2.3-bluestore_docker_dedicated_journal

jenkins test luminous-ansible2.3-bluestore_docker_dmcrypt_journal_collocation

jenkins test luminous-ansible2.3-dmcrypt_journal

jenkins test luminous-ansible2.3-docker_cluster_collocation

jenkins test luminous-ansible2.3-docker_dmcrypt_journal_collocation

jenkins test luminous-ansible2.3-shrink_mon

jenkins test luminous-ansible2.3-update_dmcrypt

jenkins test luminous-ansible2.3-docker_cluster_collocation

jenkins test luminous-ansible2.3-docker_cluster_collocation

jenkins test luminous-ansible2.3-docker_cluster_collocation

jenkins test luminous-ansible2.3-bluestore_cluster

jenkins test luminous-ansible2.3-docker_cluster

jenkins test luminous-ansible2.3-centos7_cluster

jenkins test luminous-ansible2.3-update_cluster

jenkins test luminous-ansible2.3-xenial_cluster


ES5 API [Discussion]

Make the library work in ES5 projects.

Do you want to see that in the library?

Is it possible to just target ES5 within the tsconfig?

There're still projects being written in ES5, meaning they can't use decorators and other good things  from ES6+. And I'm trying to get some feedback from people who might actually be interested in that.


Add "How to fix issue: MutatingWebhookConfiguration mutating-webhook-configuration exists?"

…configuration exists?"
Added another question.
To cover issue #794

please rebase to trigger CI to run


DependencyMergingAnalyzer also merges all evidences to main

@jeremylong Thanks for #630!  I actually had some local changes very similar but yours is great!
The only side effect is that by merging all evidences to the main dependency, the boundary of dependencies from distinctive analyzers is blur now.
Example
when both dependency by CocoaPodsAnalyzer and dependency by SwiftPackageManagerAnalyzer co-exist (which is not uncommon), it's great DependencyMergingAnalyzer will recognize it and make the CocoaPodsAnalyzer depd the main one, SwiftPackageManagerAnalyzer depd the related.  But by merging the evidence to the main dependency sort of "polluted" the evidence in SwiftPackageManagerAnalyzer depd, even though all evidence are still available from the related dependencies.
Can the evidence be kept in their original dependency objects so the boundary are kept?  If any use case needs the complete list of evidence, they can still be collected from the bundled dependencies?
I'd be happy to provide a PR if this sounds reasonable.

I can't think of a use case for needing the evidence separated more then is already done by the "source" field? As these two package management systems are pointing at the same dependency/artifact - the evidence should be the same. Do you have a concrete reason why this would be needed?

In addition to vulnerability, we also use the packages identification and their metadata (evidences) for other purposes, where consistent results for packages is key.  So when multiple artifacts co-exist, the distinction of the main vs. related dependencies is important. For example, when both Swift.package and .podspec exist for the same package, evidences from the main dependency source always has priority, unless other sources provide higher confidence evidence.  It worked well before v1.4.5.
Interestingly, it seems the related dependencies do not have evidences anymore, even after the evidence merge to the main dependency is removed.  But I haven't found when the evidence removal was done.
Any reasons for this change?

@jeremylong PR #654 is available.
We heavily rely on package metadata for open source package and license for legal compliance purpose.  Hence we are a bit more sensitive to the consistency of package metadata.  The new test class should be able to cover the two cases here.

@biancajiang one solution to meet your needs would be to disable the core dependency merging analyzer and replace it with your own. would that work?

@biancajiang I saw you closed the PR. Do you have a solution to this? If the current functionality does not suite your needs, my above solution should work and would allow you to incorporate your modifications into your deployed instance of ODC. If that works I'd like to close this issue. If not, we can discuss other solutions.