Cannot allocate memory when "retraining from scratch"

Hi, folks,
I would highly appreciate your help. I am trying to train Tesseract 4 from scratch on 400 000 single-line images containing characters from MNIST dataset of handwritten digits.
The training could not finish (with 400 000 or 350 000 single-line images) and ends up with "cannot allocate memory". I have 16 GB RAM and I am using Windows Subsystem for Linux.
Your help would be invaluable. Thanks in advance!

@mittpy I am not sure whether your approach can actually work. tesstrain is supposed to support training lines of text. The MNIST data set contains, as you know, images of single characters. I am not even sure if the tesseract model training facilities support this particular use case. To find out whether your idea might work, please try with a small sample of the whole data set. This may help to isolate the error.

Hi, @wrznr ,
Thanks for your answer. I am using 400 000 single-line images, each line (image) consists of n-number randomly selected digits from MNIST. These are not single-character images

Please post an example and please try training with 1000 lines as a primer. This could turn out to be an interesting experiment.

@wrznr ,
"It is ok with smaller number of images. That has already been tested."
Each image consists of a single-line of n-number randomly selected MNIST digits.

@mittpy Great! What is the number of images which still works? 10,000? 100,000?

Thanks, @wrznr ,
I have tried "fine tuning" with up to 100 000 single-line images and "retrain from scratch" with 1000 images. "Retrain from scratch" should be done with larger dataset ... That is why I went for 400 000 images.

It would be great if you could run training from scratch with 100,000 images. Maybe there is some kind of (OS-dependent) hard upper boundary.
@stweil Do you have experience with such huge data sets?

@mittpy, there is no upper limit for the number of images which are supported for training, but the default setting caches all line images in RAM. So on computers with up to 8 GiB RAM you might run out of memory when the image cache fills all available RAM. See issue #109 for details.

I have 16 GB RAM and I am using Windows Subsystem for Linux.

As Windows also requires some memory, this might be still not enough for caching of all line images in the Linux subsystem. Try the --sequential_training option as explained in the documentation.

@stweil , @wrznr ,
You've been really helpful, guys :)
The --sequential_training option will be given a try after the attempt to train from scratch 300 000 images finishes. I will keep you informed. Thanks a lot :)

@stweil ,
Is --sequential_training option appropriate in case one is having 250 styles of handwritten digits, let's say 250 shrifts? The documentation says: "If ... data ..., ... is all from the SAME STYLE (a handwritten manuscript book for instance) then you can use the --sequential_training flag for lstmtraining." docs

I suggest to simply try it, as I don't have personal experience with it.


UI examples are deeply nested and not fun to read

What problem does this solve or what need does it fill?
UI examples could be more readable.
What solution would you like?
Reduce the nesting by extracting instances creation to functions.
This will not only make the whole layout cleaner,
but naming the functions appropriately will help defining what certain components do.
Additional context
I can do the PR just let me know whether this is desired.

The other pattern that I like is storing the entity id, and then using .push_children.
I'm in favor of this change though!

Yeah, there are many ways to go about it,
I'll experiment with few layouts in the PR,
I'm looking forward to feedback 😃


Epg portuguese channels missing

Description
Is possible grab epg for portuguese channels from this site? Thks
https://tvepg.eu/pt/portugal/epg

Unfortunately, we cannot use this site because it has a CAPTCHA


Float turned into int along the way

Using master
Good:

Not so good:

I think this is due to the deserialization on the back-end in Python. When Python receives a number that "appears" to be an integer, it will cast it to integer. The fact that JavaScript does not have integers is annoying...
I was thinking that, maybe, instead of having numeric cell type, we should have int and float cell types, and validate the value both on the front-end and on the back-end.

Interesting, that is another issue. And it might be easier to fix. I'll see if I can fix this one.

I cannot reproduce your issue, what version are you using?

Very sorry Martin, was using the wrong env

No problem! :)


New preliminary values for MQDNN.

Description of changes:
Updated Evaluations according to the changes in #668
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@lovvge These are the results of the tests I just ran.
There are some significant improvements for the MQRNN for some reason. Otherwise it's a mixed bag.
I think I will work tomorrow on creating a proper evaluations script with regression tests to get this right once and for all.

@lovvge These are the results of the tests I just ran.
There are some significant improvements for the MQRNN for some reason. Otherwise it's a mixed bag.
I think I will work tomorrow on creating a proper evaluations script with regression tests to get this right once and for all.

I suspect that the relatively few epochs and the single run just have a too high variance in the results.

@AaronSpieler if these numbers were obtained with the final code in #668 (at least for what concerns the models network and transformation) I think this can be merged too

Im pretty sure its out of date.


Alt+Tab does not disable if a third key is also pressed.

Microsoft PowerToys version
0.68.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce
Press Alt + Tab +
✔ Expected Behavior
Disabling the Alt +Tab shortcut disables only Alt + Tab. If you press a third key also then Alt + Tab works as normal. This is painful for gaming as you frequently use other key inputs at the same time, which causes the window to become minimised. You also cannot disable a specific three-key combination as PowerToys (as far as I know) will only allow you to input shortcuts with two key combinations.
❌ Actual Behavior
Alt + Tab functions normally if a third key is also pressed.
Other Software
No response

i play world of warcraft and have macro that use alt key modifiers so when happen to press alt then w then tab it does the alt+tab thing and seams powertools does not like blocking this combination


TestEventFieldWithoutDocumentation unit test failing

TestEventFieldWithoutDocumentation is currently failing because the number of diagnostics reported dies not match the number of diagnostic expected.

I'll get this. Thanks!


Minor changes to the scores documentation.

Changes were mainly to improve clarity and presentation.  I wasn't sure whether I could add comments so created a separate email with a MS Word copy of the documentation with comments inserted: https://bom365.sharepoint.com/:w:/s/PST_P20/Ee5j3LNcAR5FiuhOT8jtNYsBTKVaAYqLpYZaJibGyOikeg?e=UPOdDa  (this file is in the Scores directory in the P20 sharepoint)
Main things to resolve in the documentation (in my opinion):

Need to add "regular" reliability diagrams (not isotonic) since they are still in wide usage and stipulated by WMO for its standard ensemble verification.
A link to the "emerging" part of the site would be good so people can see what's on the list and being developed. For example, Spearman rank correlation is widely used outside of meteorology, would be a good one to include, and I suspect there are many others.
Need to be clear about who will do the scientific and code reviews, including whether the community may be asked to participate in scientific reviews (do we have a view on this? We'll still want ultimate oversight)

I've added an issue for traditional reliability diagrams https://github.com/nci/scores/issues/312

Also added an issue for Spearman rank correlation https://github.com/nci/scores/issues/313


Upgrade Android SDK lib we compile against to 30

Upgraded Android SDK we build against to 30. This will allow support for new features such as Haptics.
Having the android.jar on the repo uploaded by a contributor is not ideal, hopefully https://github.com/libgdx/libgdx/pull/5976 will fix this.

Comparing with 7693da46147cde4d67, at least dependencies.gradle should be changed. DependencyBank is useful to change, too.

Comparing with 7693da4, at least dependencies.gradle should be changed. DependencyBank is useful to change, too.

Wouldn't that set target SDK to 30 on new gdx setup projects? That may require some more testing. The motivation for this is to be able to compile API 29 code on libGDX project. IMO, upgrading the default target SDK to 30 on libGDX projects may be done separately.

Thanks for the tests. Ok, I agree we can update dependencies.gradle compile sdk to 30 for consistency sake between Maven/Ant and Gradle builds. Considering they're independent, Id rather set it on this other PR https://github.com/libgdx/libgdx/pull/6289 that already makes some changes on the names of the variables and other Gradle build related stuff, is that ok?

Maven and me, we have some kind of hate relationship, that's why I fell in love with Gradle so fast.


fix deprecated implicit conversion

Description
Added explicit conversion to avoid the build warning mentioned in #122
Fixes #122  (<-- this will close the issue automatically when this PR is merged)

Thanks! Will review it soon.


added css classes for core fields

Any updates?

I updated the PR here #318. will try to fix travis errors now


Deprecate label_time_courses

Addresses #1169.

This must cause warnings during tests, right? All warnings in tests should be caught.

cool tip on git grep. very useful!

@Eric89GXL the only test that called label_time_courses was test_label_time_course

this is good to go

please rebase then +1 for merge

thx @teonlamont


[$500] Web - Settings - The same text is also displayed as tooltips

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: <IP_ADDRESS>
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Issue found when executing PR https://github.com/Expensify/App/pull/34702
Action Performed:

Navigate to preferences - priority mode
Hover over the text of the options

Expected Result:
It shouldn't show any tooltips.
Actual Result:
The same text is also displayed as tooltips.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/eef4215d-74db-43cb-9e9c-1cdba676cb58
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01938e9b3ecfb536b8
Upwork Job ID:<PHONE_NUMBER>977869824
Last Price Increase: 2024-01-23

Proposal
Please re-state the problem that we are trying to solve in this issue.
Priority mode selector shows tooltips
What is the root cause of that problem?
We do not disable tooltips on the PriorityModePage.
What changes do you think we should make in order to solve the problem?
We should set shouldShowTooltips={false} here:
            <SelectionList
                sections={[{data: priorityModes}]}
                onSelectRow={updateMode}
                shouldShowTooltips={false}
                initiallyFocusedOptionKey={_.find(priorityModes, (mode) => mode.isSelected).keyForList}
            />

What alternative solutions did you explore? (Optional)

Proposal
It shouldn't show any tooltips.
Please re-state the problem that we are trying to solve in this issue.
What is the root cause of that problem?
We do not disable tooltips here

PriorityModePage
LanguagePage
ThemePage

What changes do you think we should make in order to solve the problem?
We should use shouldShowTooltips = {false} in
https://github.com/Expensify/App/blob/addfa8860ac031bd4c3a98c563e124cc3a95154c/src/pages/settings/Preferences/PriorityModePage.js#L59-L62
https://github.com/Expensify/App/blob/addfa8860ac031bd4c3a98c563e124cc3a95154c/src/pages/settings/Preferences/ThemePage.js#L52-L56
https://github.com/Expensify/App/blob/addfa8860ac031bd4c3a98c563e124cc3a95154c/src/pages/settings/Preferences/LanguagePage.js#L37-L41
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
On hover, tooltip popups as shown in the video.
What is the root cause of that problem?
Not writing code for disabling the tooltips.
What changes do you think we should make in order to solve the problem?
Write code shouldShowTooltips = {false} here:
 <SelectionList
     sections={[{data: localesToLanguages}]}
     onSelectRow={(language) => App.setLocaleAndNavigate(language.value)}
     initiallyFocusedOptionKey={_.find(localesToLanguages, (locale) => locale.isSelected).keyForList}
 />

Writing this piece of code shouldShowTooltips = {false} should probably fix it.

@paultsimura Please add the message if you edit proposal

Proposal
Updated proposal to add a more complete list of places where the same change should be made.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/iadilsher

Update proposal to explain the list we should does not show the toollip

Proposal!
The problem is in this file.
src/components/SelectionList/BaseSelectionList.js
const renderItem = ({item, index, section}) => {
const normalizedIndex = index + lodashGet(section, 'indexOffset', 0);
const isDisabled = section.isDisabled || item.isDisabled;
const isItemFocused = !isDisabled && focusedIndex === normalizedIndex;
// We only create tooltips for the first 10 users or so since some reports have hundreds of users, causing performance to degrade.
const showTooltip = normalizedIndex < 10;
    return (
        <BaseListItem
            item={item}
            isFocused={isItemFocused}
            isDisabled={isDisabled}
            isHide={!maxToRenderPerBatch}
            showTooltip={showTooltip}
            canSelectMultiple={canSelectMultiple}
            onSelectRow={() => selectRow(item, true)}
            onDismissError={onDismissError}
            shouldPreventDefaultFocusOnSelectRow={shouldPreventDefaultFocusOnSelectRow}
            keyForList={item.keyForList}
        />
    );
};

So showToolTip is based on indexOffset of the section.
The section data is in PriorityModePage
<SelectionList
sections={[{data: priorityModes}]}
onSelectRow={updateMode}
initiallyFocusedOptionKey={_.find(priorityModes, (mode) => mode.isSelected).keyForList}
/>
So you should configure to
<SelectionList
sections={[{data: priorityModes, indexOffset: 20}]}
onSelectRow={updateMode}
initiallyFocusedOptionKey={_.find(priorityModes, (mode) => mode.isSelected).keyForList}
/>
Thanks for consideration.

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0165317e4bab2a9148?viewMode=1

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0165317e4bab2a9148

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~0165317e4bab2a9148

Proposal
Please re-state the problem that we are trying to solve in this issue.
The same text is also displayed as tooltips
What is the root cause of that problem?
In UserListItem and RadioListItem we show exactly the same text for tooltip and its content
                <Tooltip
                    shouldRender={showTooltip}
                    text={item.text}
                >

What changes do you think we should make in order to solve the problem?
I think we shouldn't show the tooltip by default unless it's defined, also it's not quite useful to have the same text with content.
I'd recommend adding textTooltip and alternateTextTooltip to determine if it's defined and allow us to input custom text.
In UserListItem and RadioListItem we can modify logic to:
                <Tooltip
                    shouldRender={showTooltip && Boolean(item.textTooltip)}
                    text={item.textTooltip}
                >
                    <Text
                        style={StyleUtils.combineStyles(textStyles, style)}
                        numberOfLines={1}
                    >
                        {item.text}
                    </Text>
                </Tooltip>
                {Boolean(item.alternateText) && (
                    <Tooltip
                        shouldRender={showTooltip && Boolean(item.alternateTextTooltip)}
                        text={item.alternateTextTooltip}
                    >
                        <Text
                            style={StyleUtils.combineStyles(alternateTextStyles, style)}
                            numberOfLines={1}
                        >
                            {item.alternateText}
                        </Text>
                    </Tooltip>
                )}```

### What alternative solutions did you explore? (Optional)

The reason we need a tooltip in the first place is so that if the title or subtitle of the SelectionList item are longer than the container width and showing ellipsis, we want the user to be able to view the full content of the title/subtitle. We did that by making the SelectionList items show tooltips by default. This is not ideal because we don't always need the tooltips, in many SelectionList the titles are short and there's no need for tooltips, the user can still see the full page.
And also, Tooltips are expensive, rendering a lot of them can degrade performance.
So we should only show Tooltips when necessary, that means when the "ellipsis" effect occurs. We can create a custom TextWithTooltip component that contains this logic. In onLayout, we'll check if the scrollWidth of the Text is larger than offsetWidth, if yes that means ellipsis is showing and we should show the Tooltip (we can put the Tooltip shouldRender to a state and only set it to true if ellipsis is happening)

This is a great explanation of why tooltips were enabled by default in the first place and I think it's the best solution we could implement. Disabling tooltips on a case-by-case basis is not reliable and it would still yield bad results in some cases (imagine a device with an extremely small screen and an item with a long title, which doesn't fit on screen)
I think we should proceed with this solution from @tienifr
🎀👀🎀 C+ reviewed!

@tienifr do you happen to have the issue/conversation where we decided that's why we needed tooltips here? If that is in fact why they added them then your solution makes the most sense to me as well

Great, thanks for the context! Assigning you now

Continuing the discussion in https://github.com/Expensify/App/pull/35460#issuecomment-1930174774
@kavimuru, @tienifr, could you double-check if this was resolved please?

@eVoloshchak is https://github.com/Expensify/App/pull/35460#issuecomment-1930174774 a regression?

melvin, I just commented on this issue, not overdue

@eVoloshchak bumping my question above https://github.com/Expensify/App/issues/34961#issuecomment-1948786094
is https://github.com/Expensify/App/pull/35460#issuecomment-1930174774 a regression?

@sonialiap, I cannot reproduce this, works fine for me on both Chrome and Safari
https://github.com/Expensify/App/assets/9059945/ed725db2-d87f-4f55-a13b-e6cb3fb56624

Sounds good, making payments
@eVoloshchak $500 please request in ND and complete the checklist
@tienifr $500 - paid ✔

The PR that introduced the bug has been identified. Link to the PR: there isn't a PR that caused this, this is a limitation of the original implementation
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: N/A
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: I don't think an additional discussion is needed, the bug is very minor and we've applied a different approach that resolves it throughout the app

Regression Test Proposal

Navigate to Profile -> Preferences -> Priority mode
Hover over the text of the options
Verify that the tooltip does not appear

Do we agree 👍 or 👎

$500 approved for @eVoloshchak based on summary above.


Add style for auth layout

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


Abstract Crypto Implementation

As preparation for the code move this PR does the following:

It abstracts Crypto class into a generic base class and uses it via singleton like Timer and Network already
Becaue TS do not support "abstract static" class members I decided to use a normal class which becomes via the singleton again kind of "static"
Additionally the places where "Buffer" was returned by Crypto methods and so also ended in storage the Buffer is always converted into a ByteArray type. Becauseof this also Buffer support is removed from Storage. IMPORTANT: This breaks existing storages potentially!!

The PR should be relatively straight forward to review

Yes exactly. Currently Crypto is a "core" function but implemented node.js specific and the goal is to allow platform specific implementations (platform aka Web, Nodejs, maybelow.js or whatever other JavaScript based implementations). Thats why all these topics are currently built in an abstractable way (e.g. Network, Time/Date), so Crypto is just "the same"


ASUSWRT component creating devices in known_devices.yaml that are not connected to ASUS router

Home Assistant release with the issue:
88.0
Last working Home Assistant release (if known):
Unknown
Operating environment (Hass.io/Docker/Windows/etc.):
Docker
ASUS RT-AC58U running firmware <IP_ADDRESS>.380_8457
Component/platform:
https://www.home-assistant.io/components/asuswrt/
Description of problem:
I put my iPhone and my Apple Watch in airplane mode, stopped Home Assistant and deleted known_devices.yaml
When I restarted Home Assistant known_devices.yaml was recreated
There was no entry in known_devices.yaml for my iPhone (expected result)
There was an entry in known_devices.yaml for my Apple Watch (unexpected result)
I logged into my ASUS router and neither my iPhone nor my Apple Watch were showing as connected
In the Home Assistant states screen there is an AsusWrtDeviceScanner device tracker for my Apple Watch with a state of home even though it is not connected to my router as it is in airplane mode
There is not a device tracker for my iPhone (which is also in airplane mode)
The known_devices.yaml entry and the device tracker should not be created for my Apple Watch
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
asuswrt:
  host: <IP_ADDRESS>
  username: !secret asuswrt_user
  password: !secret asuswrt_password

device_tracker:
  - platform: asuswrt
    track_new_devices: true

Traceback (if applicable):
Not Applicable
Additional information:

@kennedyshead Something you can help with?

I have not seen this, but I will investigate.

Try connecting over SSH/Telnet to the router, and running the command Home Assistant runs:
PATH=$PATH:/bin:/usr/sbin:/sbin && ip neigh
This returns the IP/MAC address pairs that the router is aware of. Notice that some of the results might return a state other than REACHABLE - maybe the ASUSWrt component doesn't check the state, and assumes that all of the devices returned should be added to known_devices.yaml?

@aronsky - you could be on to something there - the IP address for my Apple Watch had a state of STALE when in airport mode and was not showing as a connected device in the router UI but was still showing as home in the device tracker
I have now given my apple watch a fixed IP address on the router, it is currently disconnected from the wifi and the device tracker is showing not_home

Makes sense. As far as Home Assistant and the device tracker component are concerned, any MAC address that appears in the aforementioned command should be treated as a device (and thus, added to known_devices.yaml). The actual state of the device is derived from the device being REACHABLE.
Overall, it seems that the component is working as expected.

I can understand that the component uses that list to build the known devices but there looks like there is an issue with setting the home / not_home status in the device tracker
When my Apple Watch is connected to the router its status is REACHABLE and the device tracker shows the status of home
When my Apple Watch is disconnected from the router its status is STALE and the device tracker status still shows home
Do I need to raise a new issue for this?

https://github.com/kennedyshead/aioasuswrt/blob/master/aioasuswrt/asuswrt.py#L138
Its already filtered out, has to be something else

would it be worthwhile if I added a _LOGGER.debug to my local asuswrt to see what devices it is picking up?
are the results of _LOGGER added to the standard home assistant log?

I'm having the same issue with an iPhone. ip neigh shows the device as STALE, but device tracker is showing it as "home", and re-adding it to known_devices if I remove it.
My Android device does not show up in ip neigh at all when away,


Stack Club Holiday Party 2018, Friday December 21, 0900 PST

You are warmly invited to join the @LSSTScienceCollaborations/stack-club for its Holiday Party, on Friday December 21 at 0900 PST in the Clubhouse Zoom room at http://stanford.zoom.us/j/946390572 .
We'll be spending a jolly hour celebrating all the jupyter tutorial notebooks that we have made this year, with short presentations by club members gift-wrapped for the LSST Science Community.  Come and see what we have made with our LSSTC Enabling Science grant, and a lot of help from the Project DM team!
Best wishes for a festive yuletide,
Phil, Alex and Greg

Festive slides and video are now online, on the Meetings page, and I'll send them around to those who couldn't be with us this morning as well. Happy holidays, everyone! See you again in the New Year :-)


ot-daemon drops responses with InvalidSourceAddress when using service ALOCs

Communication from host to SEDs fails if using service ALOCs, from commit d0d2b71af38733fd539ce939f1c10f05316a678f onwards. I don't know if this is a bug or a misconfiguration on our side.

Configuration: ot-daemon+RCP -> 10 SEDs.
Host adds service ALOCs, eg. a CoAP server and a NTP server.
Devices reach the servers using their service ALOCs.

Before that commit, everything worked as expected. From this commit onwards, we see that every server response is dropped with [[WARN]]-PLAT----: processTransmit: InvalidSourceAddress.
Thanks!

@bukepo , what was the motivation for #5632?

@dismirlian , we submitted a fix in #5747. Please review.

Resolved by #5747


Service filtering when clicking on service relation doesn't work

If I click on the relation to the back_bone service in the attributes of the mobile core service here:

I end up here, with no back_bone being displayed: (The filter seems correct, but I would expect the service being filtered to be shown)

https://user-images.githubusercontent.com/20844316/192288756-8f02a21c-1f97-4ce9-8391-0f6cd5d98917.mp4
Web console commit hash: edd03185e0ba74af29f03b80050702b17e5e3d9d
Service orchestrator version:     5.2.1.dev20220926031235
db_dump.zip

This db dump contains credentials to access the 5g lab (if you are in our internal network), you can mess up with it as much as you like, as long as @bartv is not doing a demo at the same time as you.  (If you leave the environment stopped this is fine)

The reason this does not work is that the value of the label "id" depends on whether there is a service identity or not. In case of the 5g demo an attribute is used as the identity in the first column.
If the attribute has a value instead of a UUID, then that value should be set in the filter and not the uuid.

Hey team! Please add your planning poker estimate with Zenhub @LauraStordeur @matborowczyk


Fix bit shift math in CFCharacterSet equality code

The code was originally shifting a 32-bit value by 32 bits in
some cases, which is undefined behavior. The intent was that
all bits would be shifted out, leaving a value of 0 to perform
computation with. We now treat the value as 64-bits for the shift
and re-cast to 32-bits for the computation.

The code below exercises the code paths in fast path comparison __CFCSetIsBitmapEqualToRange between character sets with an internal representation of __kCFCharSetClassRange and __kCFCharSetClassBitmap. This code fails prior to the fix.
Interestingly, this bug also symptomizes in shipping OS X and iOS.
    CFCharacterSetRef rangeSet, bitmapSet;

    for (CFIndex startOffset = 0; startOffset < 32; startOffset++) {
        for (CFIndex endOffset = 0; endOffset < 32; endOffset++) {
            rangeSet = CFCharacterSetCreateWithCharactersInRange(NULL, CFRangeMake(1 + startOffset, 100 + endOffset));

            CFDataRef data = CFCharacterSetCreateBitmapRepresentation(NULL, rangeSet);
            bitmapSet = CFCharacterSetCreateWithBitmapRepresentation(NULL, data);
            CFRelease(data);

            XCTAssertTrue(CFEqual(rangeSet, bitmapSet));

            CFRelease(rangeSet);
            CFRelease(bitmapSet);
        }
    }

    for (CFIndex startOffset = 0; startOffset < 32; startOffset++) {
        for (CFIndex endOffset = 0; endOffset < 32; endOffset++) {
            rangeSet = CFCharacterSetCreateWithCharactersInRange(NULL, CFRangeMake(startOffset, 0 + endOffset));

            CFDataRef data = CFCharacterSetCreateBitmapRepresentation(NULL, rangeSet);
            bitmapSet = CFCharacterSetCreateWithBitmapRepresentation(NULL, data);
            CFRelease(data);

            XCTAssertTrue(CFEqual(rangeSet, bitmapSet));

            CFRelease(rangeSet);
            CFRelease(bitmapSet);
        }
    }

cc: @modocache

@mark1smi Good catch. Will it be possible to include a unit test for this?

@pushkarnk Thanks. I provided a unit test in the comment to exercise the C API, but didn't find anywhere in the project where C unit tests run. I spent a number of hours attempting to provide a test in NSCharacterSet.swift, but encountered two problems:

It was unclear to me how much the swift tests are used because they would not link (I submitted pull request https://github.com/apple/swift-corelibs-foundation/pull/937 to address that)
After fixing the link problem, it appears that something is fundamentally broken with NSCharacterSet in swift, such that you can't even compare two NSCharacterSets (infinite recursion with NSCharacterSet swift's isEqual sending its _cfObject down to CFEqual, and CFEqual sending back up via CFTYPE_SWIFT_FUNCDISPATCH1(Boolean, cf1, NSObject.isEqual, (CFSwiftRef)cf2); ). You'll notice that there are no enabled tests in TestNSCharacterSet.swift that compare two NSCharacterSets!

For reference, the swift code for the unit test would look like:
`
func test_RangeCompareWithBitmap() {
for startOffset: Int in 0..<32 {
for endOffset: Int in 0..<32 {
//let rangeSet = CharacterSet(Range:(1 + startOffset, 100 + endOffset))
let rangeSet = CharacterSet(charactersIn: Range (uncheckedBounds:(UnicodeScalar(1 + startOffset)!, UnicodeScalar(100 + endOffset)!)));
let bitmap = rangeSet.bitmapRepresentation
let bitmapSet = CharacterSet(bitmapRepresentation: bitmap)
XCTAssertEqual(rangeSet, bitmapSet)
}
}
    for startOffset: Int in 0..<32 {
        for endOffset: Int in 0..<32 {
            let rangeSet = CharacterSet(charactersIn: Range<UnicodeScalar> (uncheckedBounds:(UnicodeScalar(startOffset)!, UnicodeScalar(endOffset)!)));
            let bitmap = rangeSet.bitmapRepresentation
            let bitmapSet = CharacterSet(bitmapRepresentation: bitmap)
            XCTAssertEqual(rangeSet, bitmapSet)
        }
    }
}

`
I could add that as a disabled test to the diff if you would like, but it felt to me a strange way of testing the C API.

We know that NSCharacterSet is basically un-subclassable, which is why we switched the implementation over to has-a CFCharacterSet instead. That is likely the issue you ran into with it in Swift.

I'll run our internal CF tests against your change to verify this.

@swift-ci test and merge


Wrong screen for Networking settings

the Azure screen doesn't match the one in text. No Connectivity button
[Enter feedback here]

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

ID: 3da2d20f-0457-b8af-2f72-84741e8cb81b
Version Independent ID: 3bcd4ccd-9b70-65d0-a720-57fd9a566767
Content: Create a single database - Azure SQL Database
Content Source: articles/azure-sql/database/single-database-create-quickstart.md
Service: sql-database
Sub-service: single-database
GitHub Login: @stevestein
Microsoft Alias: sstein

@leshaved Thank you for the feedback. We are actively investigating and will get back to you soon.

@leshaved Could you please confirm if you are creating a new server or using an existing server while creating the Azure SQL Database. In case you are using the existing server, we would not be able to see the options as mentioned by you are the Azure SQL Server setting have already been defined while creating the server.
However, if we create a new server, we can see the network connectivity option as mentioned in below screenshot. Even the article mentions about the new server.

Please let me know if this helps or else we can discuss further on the same.

@leshaved We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


PDR508HP not shown in app

The problem
Hi,
I have a PDR508HP (semi-professional dryer).
I can see the device in the official Miele app, but the HACS Miele Integration does not show the appliance.
If I grab the configuration file from the HomeAssistant I see that the integration has found the dryer, but it's not shown in the GUI.
Diagnostic file is attached + screenshot from the integration.
Best regards,
Tobias
config_entry-miele-01916ef10e3bbe1af126d85fe442e062-06042024.txt

Version of miele integration having the issue?
v2024.3.0
Version of Home Assistant Core having the issue?
2024.4.1
Anything in the logs that might be useful for us?
No response
Additional information
No response

That was odd... The PDR508HP is working for other users. Please turn on Debug-Protokollierung from the screen-shot above and restart HA. Check in the log if there are any unexpected messages from the miele integration.

error_log-2.txt
I only see the message "Manually updated miele data".
Looks normal. I attached the debug logging.

error_log-3.txt
I restarted the HomeAssistant again with enabled debug logging and know I see the device and entities. Mysterious.

After a few minutes all entities and the device are gone and not longer available.
Restarted HA a few times, without success.

Same problem here; already reported

Also reported to Miele.
My investigation looks like, that the 3rd party API, not show the PDR508HP after one data fetch.
You can get the data once from the API and then for a unknown time not anymore.
Miele has to investigate.
The swagger UI from the Miele Homepage also delivers empty outputs in case of error.
So I look forward for a answer from Miele.

@TOKL1992 Good catch! Let's hope that Miele fixes this promptly.

Anyone seen a change for this device? Miele reacted?
How can I submit also a request to Miele for change on this item; maybe a link?

Via Mail: developer(at)miele.com

Am 19.06.2024 um 10:27, Bart Gysens  @.***>  schrieb:
Anyone seen a change for this device? Miele reacted? @TOKL1992 How can I submit also a request to Miele for change on this item; maybe a link? — Reply to this email directly, view it on GitHub, or uns
Achtung! Diese Nachricht stammt von außerhalb der Organisation!
sophospsmartbannerend
Anyone seen a change for this device? Miele reacted?
@TOKL1992  How can I submit also a request to Miele for change on this item; maybe a link?
—
Reply to this email directly,   view it on GitHub, or   unsubscribe.
You are receiving this because you were mentioned.Message ID:  @.***>

Answer of Miele support:

We are sorry to inform you that the current API only supports Miele domestic appliances.
Your Miele tumble dryer is a (semi)professional appliance.

Same answer to my mail request.


It is better to remove the 'Rename...' action from the context menu of one broken QA item

Storage Explorer Version: 1.28.0-dev
Build Number: 20230110.7
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Ventura 13.1 (Apple M1 Pro)
Architecture: ia32/x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Expand one storage account -> Queues.
Attach one queue -> Pin the attached queue to the Quick Access node.
Detach the attached queue -> The QA queue shows as broken.
Right click the broken item -> Observe the context menu.

Expected Experience
It is better to remove the 'Rename...' action.
Actual Experience
The 'Rename...' action shows on the context menu.

Additional Context
Refer to the bug 6408.

Verified this issue on the build main 20230111.6. Fixed.
The 'Rename...' action has been removed.


Handling ingress traffic into the AKS cluster

Great work, thanks for persisting with terraform especially with the AKS/Ingress controller work, looked very difficult to implement :)


Orthomosiac Generation

Hi there,
I'm considering adding functionality to OpenSfm to create an orthophoto as it doesn't look like this exists. I was wondering if that would be useful? If there is a partial work I'm happy to build of that or take direction on a suggested approach?
Regards,
Nick

Hi Nick,
This is something I would be very interested in and pleased to see added to OpenSfM.
Cheers,
Will

Hi Will,
Is there anything I can build off to get started or a preferred approach?
Nick

Hi Nick and Will,
OpenSfM does not compute orthophotos on its own, but can be used as a part of the process.
The typical approach to create orthophotos is

Compute the camera positons (OpenSfM does this part)
Compute depth maps or dense point clouds
Compute a mesh and texture it using the images
Take a picture (the orthophoto) of that mesh

https://github.com/OpenDroneMap/OpenDroneMap is a project that installs and runs the required software for all of the process. The python-port branch has the option to use OpenSfM for computing the camera positions.

Does this branch generate the orthophoto using Python? If it does it might
be a good starting point.
On Thu, Oct 22, 2015 at 1:26 PM, Pau Gargallo<EMAIL_ADDRESS>wrote:

Hi Nick and Will,
OpenSfM does not compute orthophotos on its own, but can be used as a part
of the process.
The typical approach to create orthophotos is

Compute the camera positons (OpenSfM does this part)
Compute depth maps or dense point clouds
Compute a mesh and texture it using the images
Take a picture (the orthophoto) of that mesh

https://github.com/OpenDroneMap/OpenDroneMap is a project that installs
and runs the required software for all of the process. The python-port
branch has the option to use OpenSfM for computing the camera positions.
—
Reply to this email directly or view it on GitHub
https://github.com/mapillary/OpenSfM/issues/43#issuecomment-150346930.

It generates the orthophoto using a C++ code (https://github.com/OpenDroneMap/OpenDroneMap/tree/python-port/odm_orthophoto).  Python there refers only to the run.py script with is the one in charge of running the required tools for each of the processes

Then Would it make sense to just port the cpp implementation to Python and
integrate into osfm?
On Thursday, October 22, 2015, Pau Gargallo<EMAIL_ADDRESS>wrote:

It generates the orthophoto using a C++ code (
https://github.com/OpenDroneMap/OpenDroneMap/tree/python-port/odm_orthophoto).
Python there refers only to the run.py script with is the one in charge
of running the required tools for each of the processes
—
Reply to this email directly or view it on GitHub
https://github.com/mapillary/OpenSfM/issues/43#issuecomment-150350932.

super late answer here: that would be more complex than just using ODM directly. I don't think is a good plan. I would rather help improving ODM on that side.


Project no longer supported?

Is this project no longer supported? @nicholasrobinson

I’m still here @dVIAN13 ! If you have a specific issue please raise a ticket or propose a PR.

I’m still here @dVIAN13 ! If you have a specific issue please raise a ticket or propose a PR.

Glad to see you're still around! Is there anything we can add in the pending issues to help diagnose them and get them addressed/fixed? in both node-smartcielo and homebridge-smartcielo. thanks!


Ability to sort SFTP directory content

Feature description
Would be great with the usual sort by name or modified date. Personally I like to be able to set a default sort order.
By Oliver F.

Nice app, but it should be improved adding the option of sorting files by date and multiple selection to download several files in one shot.

By ** m00n_xyz** on the App Store


Class annotations

Support annotations on the command class directly. Make method inherit these, and override where necessary.

Done in 1.1.0-SNAPSHOT https://github.com/Incendo/cloud/pull/84


Fix conflicting versions

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 89.895%

Files with Coverage Reduction
New Missed Lines
%

django_q/core_signing.py
1
87.5%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
3336

Relevant Lines:
3711

💛  - Coveralls


Integrate VPR with SDC alias fix.

SDC files that were previously using generated aliases now uses one of
the net names.

@acomodi Thanks, new VPR is working great!

CI went green on all targets. Merging.


Update SUBSCRIBE.cs

必须发送固定长度的byte[]，否则客户端会断开连接

Could you use buffer pool like the current version in the master branch?
https://github.com/SuperSocket/SuperSocket.MQTT/blob/master/src/SuperSocket.MQTT.Server/Command/SUBSCRIBE.cs
I think you just need write the Qos into buffer in this method:
private void WriteBuffer(byte[] buffer, SubscribePacket packet)
        {
            buffer[0] = 144;
            buffer[1] = 3;

            BinaryPrimitives.WriteInt16BigEndian(buffer.AsSpan().Slice(2), packet.PacketIdentifier);
        }

Could you use buffer pool like the current version in the master branch?
https://github.com/SuperSocket/SuperSocket.MQTT/blob/master/src/SuperSocket.MQTT.Server/Command/SUBSCRIBE.cs
I think you just need write the Qos into buffer in this method:
private void WriteBuffer(byte[] buffer, SubscribePacket packet)
        {
            buffer[0] = 144;
            buffer[1] = 3;

            BinaryPrimitives.WriteInt16BigEndian(buffer.AsSpan().Slice(2), packet.PacketIdentifier);
        }


New package: NewsLookout v0.0.1

Registering package: NewsLookout
Repository: https://github.com/sandeep-sandhu/NewsLookout.jl
Created by: @sandeep-sandhu
Version: v0.0.1
Commit: e5a5ebacb8f224794644c2771b42c00a51768dd2
Git reference: HEAD
Release notes:

The NewsLookout.jl information retrieval platform powered by the Julia language. It provides an extensible and flexible application that caters to sourcing, tagging and deep-learning NLP based information retrieval from public News sources.

@sandeep-sandhu please see the message above.  Once you fix the automerge issues you can register again the new revision without changing the version number and this pull request will be automatically updated.
[noblock]


[desktop] FocusTraversalPolicy._sortAllDescendants crashes/throws error when tab to focus on unfocusable widgets

Description
When running the app on desktop or web on desktop, if you press tab to focus on widget without focusability, app fatally crashes on windows with
                    ══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
                    The following _CastError was thrown during a platform message callback:
                    Null check operator used on a null value

                    When the exception was thrown, this was the stack:
                    #0      FocusTraversalPolicy._sortAllDescendants (package:flutter/src/widgets/focus_traversal.dart:340:52)

Meanwhile on macOS and linux, the error is thrown but no fatal crash.
After looking into the issue, I see the under _sortAllDescendants in focus_traversal.dart, groups = <FocusNode?, _FocusTraversalGroupInfo>{}; has no data but  visitGroups is trying to access it anyway and throws Null check operator used on a null value. You can add a simple print statment to see if groups is empty or not.
    print('groups - $groups');
    // Visit the children of the scope.
    visitGroups(groups[scopeGroupMarker?.focusNode]!);

To make it easier to reprdouce here I've created a minimal sample
Steps to reproduce

Run the code sample on any desktop environment on desktop app or web

minimal code sample
import 'package:flutter/material.dart';

void main() => runApp(const MyApp());

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Material App',
      theme: ThemeData.dark(),
      home: const Home(),
    );
  }
}

class Home extends StatelessWidget {
  const Home({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: const Text('Material App Bar'),
      ),
      body:  ListTile(title: Text('title'),
        // onTap: () {}, // Uncomment to get rid of the error
      ), // Error if the ListTile has no onTap or no focusability
      // body: const ElevatedButton(
      //   child: Text('Button'),
      //   onPressed:
      //       null, // Error if the button has no onPressed or no focusability
      // ),
      // floatingActionButton: FloatingActionButton(
      //   child: Icon(Icons.add),
      //   // onPressed: () {}, // Focus is moved here hence no error
      //   onPressed: null, // Provide null here to reproduce the issue
      // ),
    );
  }
}

Tab to focus

Actual results:
When you tab to focus on a widget that can't focus,
ex: A ListTile without onTap
body: ListTile(title: Text('title'),),
When tab to focus
final Map<FocusNode?, _FocusTraversalGroupInfo> groups = <FocusNode?, _FocusTraversalGroupInfo>{};  returns {};
Error is thrown at visitGroups(groups[scopeGroupMarker?.focusNode]!); in focus_traversal.dart
Print statement with groups returns
flutter: groups - {}

Expected results:
Doesn't crash or throw any error

if you provide onTap to ListTitle or onPressed to ElevatedButton in the code sample

Run the app again on desktop and tab to focus

Actual results:
Print statement with groups returns
flutter: groups - {FocusNode#e3df3(FocusTraversalGroup [IN FOCUS PATH])(context: Focus, NOT FOCUSABLE, IN FOCUS PATH): Instance of '_FocusTraversalGroupInfo'}

App doesn't crash or throw any error
Logs (windows- fatal crash with error)

logs
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/fonts/MaterialIcons-Regular.otf
[   +5 ms]   -- Installing:
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/kernel_blob.bin
[ +120 ms]   -- Installing:
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/NOTICES.Z
[   +1 ms]   -- Installing: C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/packages
[        ]   -- Installing:
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/packages/cupertino_icons
[        ]   -- Installing:
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/packages/cupertino_icons/assets
[   +1 ms]   -- Installing:
C:/Users/Taha/StudioProjects/master_flutter/build/windows/runner/Debug/data/flutter_assets/packages/cupertino_icons/assets/Cupe
rtinoIcons.ttf
[  +15 ms] FinalizeBuildStatus:
[        ]   Deleting file "x64\Debug\INSTALL\INSTALL.tlog\unsuccessfulbuild".
[        ]   Touching "x64\Debug\INSTALL\INSTALL.tlog\INSTALL.lastbuildstate".
[        ] Done Building Project "C:\Users\Taha\StudioProjects\master_flutter\build\windows\INSTALL.vcxproj" (default targets).
[        ] Build succeeded.
[        ]     0 Warning(s)
[        ]     0 Error(s)
[        ] Time Elapsed 00:00:21.83
[  +15 ms] Building Windows application... (completed in 22.6s)
[ +425 ms] Observatory URL on device: http://<IP_ADDRESS>:51904/051-RwIn5ck=/
[   +4 ms] Caching compiled dill
[  +31 ms] Connecting to service protocol: http://<IP_ADDRESS>:51904/051-RwIn5ck=/
[ +113 ms] Launching a Dart Developer Service (DDS) instance at http://<IP_ADDRESS>:0, connecting to VM service at
http://<IP_ADDRESS>:51904/051-RwIn5ck=/.
[  +72 ms] DDS is listening at http://<IP_ADDRESS>:51908/m59vPYTY6xI=/.
[  +56 ms] Successfully connected to service protocol: http://<IP_ADDRESS>:51904/051-RwIn5ck=/
[  +16 ms] DevFS: Creating new filesystem on the device (null)
[  +18 ms] DevFS: Created new filesystem on the device
(file:///C:/Users/Taha/AppData/Local/Temp/master_flutter695fc225/master_flutter/)
[   +4 ms] Updating assets
[  +69 ms] Syncing files to device Windows...
[   +2 ms] <- reset
[        ] Compiling dart to kernel with 0 updated files
[   +3 ms] <- recompile package:master_flutter/main.dart c268f178-5fa6-42e9-bc6f-cd75af9b45c4
[        ] <- c268f178-5fa6-42e9-bc6f-cd75af9b45c4
[  +78 ms] Updating files.
[   +1 ms] DevFS: Sync finished
[   +1 ms] Syncing files to device Windows... (completed in 87ms)
[   +1 ms] Synced 0.0MB.
[   +2 ms] <- accept
[   +3 ms] Connected to _flutterView/0x1e2e896a550.
[   +4 ms] Flutter run key commands.
[   +3 ms] r Hot reload.
[   +1 ms] R Hot restart.
[   +1 ms] h List all available interactive commands.
[   +1 ms] d Detach (terminate "flutter run" but leave application running).
[        ] c Clear the screen
[        ] q Quit (terminate the application on the device).
[   +1 ms]  Running with sound null safety
[   +1 ms] An Observatory debugger and profiler on Windows is available at: http://<IP_ADDRESS>:51908/m59vPYTY6xI=/
[ +277 ms] DevTools activation throttled until 2021-07-21 01:02:31.495488.
[ +438 ms] The Flutter DevTools debugger and profiler on Windows is available at:
                    http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:51908/m59vPYTY6xI=/
[ +739 ms] Unable to parse JSON message:
[   +1 ms] The document is empty.
[  +11 ms]
           ══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY ╞══════════════════════════════════════════════════════════
           The following _CastError was thrown during a platform message callback:
           Null check operator used on a null value

           When the exception was thrown, this was the stack:
           #0      FocusTraversalPolicy._sortAllDescendants (package:flutter/src/widgets/focus_traversal.dart:340:52)
           #1      FocusTraversalPolicy._findInitialFocus (package:flutter/src/widgets/focus_traversal.dart:167:42)
           #2      FocusTraversalPolicy.findFirstFocus (package:flutter/src/widgets/focus_traversal.dart:139:55)
           #3      FocusTraversalPolicy._moveFocus (package:flutter/src/widgets/focus_traversal.dart:381:47)
           #4      FocusTraversalPolicy.next (package:flutter/src/widgets/focus_traversal.dart:223:39)
           #5      FocusNode.nextFocus (package:flutter/src/widgets/focus_manager.dart:1153:56)
           #6      NextFocusAction.invoke (package:flutter/src/widgets/focus_traversal.dart:1885:19)
           #7      ActionDispatcher.invokeAction (package:flutter/src/widgets/actions.dart:520:21)
           #8      ShortcutManager.handleKeypress (package:flutter/src/widgets/shortcuts.dart:834:38)
           #9      _ShortcutsState._handleOnKey (package:flutter/src/widgets/shortcuts.dart:1175:20)
           #10     FocusManager._handleRawKeyEvent (package:flutter/src/widgets/focus_manager.dart:1681:50)
           #11     RawKeyboard._handleKeyEvent (package:flutter/src/services/raw_keyboard.dart:682:69)
           #12     BasicMessageChannel.setMessageHandler.<anonymous closure>
(package:flutter/src/services/platform_channel.dart:73:49)
           #13     BasicMessageChannel.setMessageHandler.<anonymous closure>
(package:flutter/src/services/platform_channel.dart:72:47)
           #14     _DefaultBinaryMessenger.setMessageHandler.<anonymous closure>
(package:flutter/src/services/binding.dart:359:35)
           #15     _DefaultBinaryMessenger.setMessageHandler.<anonymous closure>
(package:flutter/src/services/binding.dart:356:46)
           #16     _invoke2.<anonymous closure> (dart:ui/hooks.dart:203:15)
           #20     _invoke2 (dart:ui/hooks.dart:202:10)
           #21     _ChannelCallbackRecord.invoke (dart:ui/channel_buffers.dart:42:5)
           #22     _Channel.push (dart:ui/channel_buffers.dart:132:31)
           #23     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
           #24     PlatformDispatcher._dispatchPlatformMessage (dart:ui/platform_dispatcher.dart:536:22)
           #25     _dispatchPlatformMessage (dart:ui/hooks.dart:90:31)
           (elided 3 frames from dart:async)
           ════════════════════════════════════════════════════════════════════════════════════════════════════
[ +514 ms] Service protocol connection closed.
[   +1 ms] Lost connection to device.
[   +2 ms] DevFS: Deleting filesystem on the device
(file:///C:/Users/Taha/AppData/Local/Temp/master_flutter695fc225/master_flutter/)
[ +262 ms] Ignored error while cleaning up DevFS: TimeoutException after 0:00:00.250000: Future not completed
[   +6 ms] "flutter run" took 28,314ms.
[  +76 ms] ensureAnalyticsSent: 73ms
[   +1 ms] Running shutdown hooks
[   +1 ms] Shutdown hooks complete
[   +1 ms] exiting with code 0

Logs (macOS -  no fatal crash with error)

logs
[  +81 ms] Updating files.
[        ] DevFS: Sync finished
[   +1 ms] Syncing files to device macOS... (completed in 88ms)
[        ] Synced 0.0MB.
[   +1 ms] <- accept
[   +2 ms] Connected to _flutterView/0x7fb00d863220.
[   +3 ms] Flutter run key commands.
[   +1 ms] r Hot reload. 🔥🔥🔥
[   +1 ms] R Hot restart.
[        ] h List all available interactive commands.
[        ] d Detach (terminate "flutter run" but leave application running).
[        ] c Clear the screen
[        ] q Quit (terminate the application on the device).
[        ] 💪 Running with sound null safety 💪
[        ] An Observatory debugger and profiler on macOS is available at:
           http://<IP_ADDRESS>:50301/c2bLnT1Z0zk=/
[ +296 ms] DevTools activation throttled until 2021-07-20 22:07:27.896527.
[ +585 ms] The Flutter DevTools debugger and profiler on macOS is available at:

http://<IP_ADDRESS>:9100?uri=http://<IP_ADDRESS>:50301/c2bLnT1Z0zk=/
[+5945 ms] flutter: groups - {}
[  +84 ms]
           ══╡ EXCEPTION CAUGHT BY SERVICES LIBRARY
           ╞══════════════════════════════════════════════════════════
           The following _CastError was thrown during a platform message
callback:
           Null check operator used on a null value

           When the exception was thrown, this was the stack:
           #0      FocusTraversalPolicy._sortAllDescendants
           (package:flutter/src/widgets/focus_traversal.dart:341:52)
           #1      FocusTraversalPolicy._findInitialFocus
           (package:flutter/src/widgets/focus_traversal.dart:167:42)
           #2      FocusTraversalPolicy.findFirstFocus
           (package:flutter/src/widgets/focus_traversal.dart:139:55)
           #3      FocusTraversalPolicy._moveFocus
           (package:flutter/src/widgets/focus_traversal.dart:382:47)
           #4      FocusTraversalPolicy.next
           (package:flutter/src/widgets/focus_traversal.dart:223:39)
           #5      FocusNode.nextFocus
           (package:flutter/src/widgets/focus_manager.dart:1153:56)
           #6      NextFocusAction.invoke
           (package:flutter/src/widgets/focus_traversal.dart:1886:19)
           #7      ActionDispatcher.invokeAction
           (package:flutter/src/widgets/actions.dart:520:21)
           #8      ShortcutManager.handleKeypress
           (package:flutter/src/widgets/shortcuts.dart:834:38)
           #9      _ShortcutsState._handleOnKey
           (package:flutter/src/widgets/shortcuts.dart:1175:20)
           #10     FocusManager._handleRawKeyEvent
           (package:flutter/src/widgets/focus_manager.dart:1681:50)
           #11     RawKeyboard._handleKeyEvent
           (package:flutter/src/services/raw_keyboard.dart:682:69)
           #12     BasicMessageChannel.setMessageHandler.<anonymous closure>
           (package:flutter/src/services/platform_channel.dart:73:49)
           #13     BasicMessageChannel.setMessageHandler.<anonymous closure>
           (package:flutter/src/services/platform_channel.dart:72:47)
           #14     _DefaultBinaryMessenger.setMessageHandler.<anonymous closure>
           (package:flutter/src/services/binding.dart:359:35)
           #15     _DefaultBinaryMessenger.setMessageHandler.<anonymous closure>
           (package:flutter/src/services/binding.dart:356:46)
           #16     _invoke2.<anonymous closure> (dart:ui/hooks.dart:203:15)
           #20     _invoke2 (dart:ui/hooks.dart:202:10)
           #21     _ChannelCallbackRecord.invoke
(dart:ui/channel_buffers.dart:42:5)
           #22     _Channel.push (dart:ui/channel_buffers.dart:132:31)
           #23     ChannelBuffers.push (dart:ui/channel_buffers.dart:329:17)
           #24     PlatformDispatcher._dispatchPlatformMessage
           (dart:ui/platform_dispatcher.dart:536:22)
           #25     _dispatchPlatformMessage (dart:ui/hooks.dart:90:31)
           (elided 3 frames from dart:async)

════════════════════════════════════════════════════════════════════════════════
           ════════════════════

stable
beta
dev
master

❌
❌
❌
❌

Check flutter doctor -v outputs for each channel below
Note: Same Flutter version applies to windows and app crashes

flutter doctor -v
[✓] Flutter (Channel stable, 2.2.3, on macOS 12.0 21A5284e darwin-x64, locale en-GB)
    • Flutter version 2.2.3 at /Users/tahatesser/Code/flutter_stable
    • Framework revision f4abaa0735 (3 weeks ago), 2021-07-01 12:46:11 -0700
    • Engine revision 241c87ad80
    • Dart version 2.13.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5284e darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.164

• No issues found!

[✓] Flutter (Channel beta, 2.3.0-24.1.pre, on macOS 12.0 21A5284e darwin-x64, locale en-GB)
    • Flutter version 2.3.0-24.1.pre at /Users/tahatesser/Code/flutter_beta
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 615957513e (4 weeks ago), 2021-06-25 09:24:44 -0700
    • Engine revision 9d517f475b
    • Dart version 2.14.0 (build 2.14.0-188.5.beta)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5284e darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.164

• No issues found!

[✓] Flutter (Channel dev, 2.4.0-4.0.pre, on macOS 12.0 21A5284e darwin-x64, locale en-GB)
    • Flutter version 2.4.0-4.0.pre at /Users/tahatesser/Code/flutter_dev
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision cc00e7e6bc (9 days ago), 2021-07-11 18:21:02 -0400
    • Engine revision ed25e8f01e
    • Dart version 2.14.0 (build 2.14.0-301.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5284e darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.164

• No issues found!

[✓] Flutter (Channel master, 2.4.0-5.0.pre.105, on macOS 12.0 21A5284e darwin-x64, locale en-GB)
    • Flutter version 2.4.0-5.0.pre.105 at /Users/tahatesser/Code/flutter_master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision a5b9b5774a (10 hours ago), 2021-07-19 20:23:45 -0700
    • Engine revision 9edde7423f
    • Dart version 2.14.0 (build 2.14.0-332.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/tahatesser/Code/SDK
    • Platform android-30, build-tools 30.0.3
    • ANDROID_HOME = /Users/tahatesser/Code/SDK
    • Java binary at: /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.5.1, Build version 12E507
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] Android Studio (version 4.2)
    • Android Studio at /Users/tahatesser/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/202.7486908/Android
      Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] VS Code (version 1.58.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0 21A5284e darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 91.0.4472.164

• No issues found!

✅ : No Issue ❌: Issue reproduced

cc: @gspencergoog
I see you're active in this file :)

Thanks, @TahaTesser, that was indeed a bug that needed fixing!


Fix crash when SimdComparison calls SIMD instructions

Fixes https://github.com/facebookincubator/velox/issues/8668

@kgpai Do you have any other questions? Please help to re-trigger the tests. Thanks!

@xiaodai1002 The change looks good, but it will still be good to write a test to validate. I have described one way but feel free to reach out to me if you need help.

@kgpai   Please help to re-trigger the tests and review the PR. Thanks!

@kgpai Test failed again,help to re-trigger the tests,Thanks!

@xiaodai1002 Rebase from main to fix the failing macos test.

@kgpai Please help to trigger the test workflows, Thanks.

Looks great, thanks for the fix; would be great if you can fix the minor nit though.

I've made the revisions as suggested. Please help to trigger the test workflows, Thanks.


JBPM-5239

Non blocking AbstractAvailableJobsExecutor with command callback using CommandAsync interface.
Interfaces is in droolsjbpm-knowledge project (JBPM-5239)
PR set:
https://github.com/droolsjbpm/droolsjbpm-knowledge/pull/184
https://github.com/droolsjbpm/jbpm/pull/657

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

Can one of the admins verify this PR? Comment with 'ok to test' to start the build.

PR set:
https://github.com/droolsjbpm/droolsjbpm-knowledge/pull/184
https://github.com/droolsjbpm/jbpm/pull/657

@nikolobok thanks for sending the PR it makes it easier to discuss what is the actual need. First thing that comes to my mind is that I don't see clear benefits of moving the execution of the command to another thread.. as it's already running in separate on - managed by jbpm executor.
Next, looks like we don't need another type of command but more another type of results and then logic around handling the results. The implementation of a command can still be the same and even spin another thread to do the processing and then directly return AyncExecutionResults that would work similar to Future where it can either block when asked for results or use callback to notify when it's done. This will then better fit the design behind executor as the only part that has to be changed is how executor deals with response/results.

I`ve assumed using the callback style to delegate execution to different system, in my case it should be soap over jms (request/response) service, this unit of work not managed by jbpm executor moreover it could be a different machine.
That is why it should not block any thread in jbpm jvm and that is why we can not use Future.
May be it could be done by using CompletableFuture.
Anyway it needs different Command interface and other logic would be the same as in PR except not calling scheduleResponse() directly from user code.
Is Java 8 requirement would be affordable for JBPM project? If so I will try to code it using CompletableFuture.

I have looked more about CompletableFuture, so it could not be used in this case. Because here we should completely break execution flow, and answer could be received on different server.

java 8 is the minimum for version 7 (master).
Sorry, I still don't see this use case... so what you try to do is not block any thread on jbpm then you should not use executor at all. You should have your own handler that simply sends the message to jms/soap and then waits for signal when the job is completed. That signal should carry on all the data it has. What's the point of using executor here?

@mswiderski The main point do not overload bpmn diagram with technical data about implementing service call.
When we call request-response SOAP web service with regular HTTP transport, we only have one activity in bpmn diagram, why it should be different when we change transport to jms, not interface nor business operation.
Secondly I believe signals are good for one-way conversation or
separated  interfaces one for request and other for response so they show a real business mean about waiting and receiving some data. But I want encapsulate two-way service call regardless of it`s implementation.

@nikolobok that's all good and I agree with what you described - don't change the process when technical/implementation details changed. And that's why there is work item handlers that can be replaced without changing the process at all.
What I don't get why you would like to change the way jbpm executor works? Wouldn't using work item handler impl be a better fit here?

@mswiderski I believe it can`t be done without patching executor, of course we could implement something that will replace executor but why if we already have working code, that we can extend with backward compatibility. So executor is default way to implement async execution for jbpm, only I have done is append some new states to executor jobs, I have not changed default (blocking) way executor works, there is no any need to change existing code.

if you see executor is valid for you use case you could use it without any modifications. What you actually do in the pull request is to schedule "response" once the job is completed with execution.
But why do you need to change it to "response"? why simply not to schedule another command that will get the the data (input) and copy to results?
Because what you have done in schedule Response is pretty much scheduling another command execution - but instead of giving a new request you update the state which means (from executor point of view) second execution of the same job - that is against the idea behind the executor.
So instead of extending the executor itself I'd recommend to give it a try with using its capabilities to achieve what you need - I am quite confident you can already do that with command chaining.

@mswiderski Well I have thought about it, but in this case we will need somehow handle all CommandContext  over invocation of remote service, and it is not possible in general, and some other troubles arise.
But when we switching status to WAITING_ASYNC in case of new CommandAsync job implementation, it does not pass status DONE and we could transfer over invocation only correlation id of request, then receive it, find job in WAITING_ASYNC status, and pass it to executor to complete it to status DONE.
So we does not pass same job twice, we just making delay in it`s execution, and we have all needed data to correctly manage this job when it waiting an answer, it could be killed for example.
When there is error in request it will be handled as usual, by current code, tried X times, and then break.
If we mark it as recur it will be correctly repeated as new job, with new id, by existing code.
All will be as usual, except of non-blocking pause of execution in case of CommandAsync.

I understand your attention to retain executor logic stable and not change it.
And in case of this PR the default life cycle did not changed, it just extends in case of using this new async command feature, in all other cases it keeps previous behavior.

actually it does change the behavior slightly (and that's why I mentioned command chaining) as the scheduleResponse calls the event listener job scheduled event so it's actually marks is as new command - in the way executor treats the commands.
What I am trying to say is that this behavior can be achieved without changing the life cycle of the executor. You can do all that stuff with custom commands and callbacks. When you async command completes you can call another command explicitly or via callback. Because the changes in this PR directly do that via additional methods - waiting async is considered as queued or retrying so why not simply use these states to perform the completion?

@mswiderski This behavior is only when we use new CommandAsync interface, and yes event listener will be fired in this case, but why not? anyway I can change it. BUT in case of regular Command executor job life cycle did not changed at all, event fired only once.
As about do this stuff by custom work item handler it can`t be done, because executor will complete work item after command returned result and will move process forward to next item. But it is not what should be in case of two-way service call from business (BPMN) point of view.

This behavior is only when we use new CommandAsync interface, and yes event listener will be fired in this case, but why not?

because that would mean there is a new command scheduled while it is not, so it's abusing the executor mechanism to let the command be re-excuted to return results.

As about do this stuff by custom work item handler it can`t be done, because executor will complete work item after command returned result and will move process forward to next item. But it is not what should be in case of two-way service call from business (BPMN) point of view.

it can be done, as your command is in full control over the execution and if you don't want to complete the work item simply remove callbacks from the command context and then none will be executed.


README.md updated

description of 'updateSinceUtilBuild' option corrected to reflect the actual behavior

Should be fixed in https://github.com/JetBrains/gradle-intellij-plugin/blob/master/src/docs/configuration.md, sorry

applied manually


Contenteditable div cursor outside of div

When making a contenteditable div, the cursor is placed on the inside of the div.

When you type something, then delete what was typed, the cursor displays on the outside of the div.
After deleting then clicking out of the div then clicking back on the div again, the cursor is outside of the div.

This image shows the cursor outside of the div:

JSFiddle demonstration: http://jsfiddle.net/y3ms3f7a/4/
Here is an example:

one

two

Here is the extra css attached to it:
.tag{
    background-color: #01b6ef;
    color: #012935;
    display: inline-block;
    padding: 1px 5px;
    margin-right: 3px;
}

.tag a{
    color: #ffffff;
    margin-left: 5px;
    font-size: 12px;
}
.tag a:hover{
    color: #ffffff;
}

What OS and browser are you using?

Trying to force revalidation:
http://jsfiddle.net/y3ms3f7a/1/

Hi @cvrebert!
You appear to have posted a live example (http://fiddle.jshell.net/y3ms3f7a/1/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 53, column 3: E014: Columns (.col-*-*) can only be children of .rows or .form-groups
W005: Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

I am using Chrome 41.x on Ubuntu, someone also tried on Windows 8.1 with Chrome 41.x also and same thing.

http://jsfiddle.net/y3ms3f7a/3/

Hi @TheColorRed!
You appear to have posted a live example (http://fiddle.jshell.net/y3ms3f7a/3/show/light/), which is always a good first step. However, according to Bootlint, your example has some Bootstrap usage errors, which might potentially be causing your issue:

line 53, column 3: E003: Found one or more .rows that were not children of a grid column or descendants of a .container or .container-fluid

You'll need to fix these errors and post a revised example before we can proceed further.
Thanks!
(Please note that this is a fully automated comment.)

http://jsfiddle.net/y3ms3f7a/4/

Putting .form-control on a div isn't valid.

X-Ref: https://github.com/twbs/bootlint/pull/263

This should be valid, or there should be something that formats a div in a similar way as an input field since divs can be editable and act as a formattable form element.

@cvrebert
I found that I can solve the issue using an input-group within the form-control and it seems to work:

    Tags

                {foreach item=row from=$tags}
                    {$row.tag}
                    {/foreach}

X-Ref: #15486. contenteditable styles are off the table until Bootstrap v4.
For now, the closest thing we have is .form-control-static.

Okay, sounds good. Thanks for the input!

Has anyone found a valid fix for this issue? I am having the same occur when I have button groups inline in a contenteditable div and using the (invalid) .form-control .input-group fix suggested by @TheColorRed is not working for me


Supports health checking

As https://github.com/GreptimeTeam/greptimedb/issues/4674 suggests, a method like is_connected is good to have in our Go SDK.

We already have the heath checking protocol
https://github.com/GreptimeTeam/greptime-proto/blob/973f49cde88a582fb65755cc572ebcf6fb93ccf7/proto/greptime/v1/health.proto#L26

@killme2008 @MichaelScofield
In the greptime healthcheck proto, the message returned by HealthCheckResponse is empty. It can only determine whether the network is connected. It should return a judgment similar to ready or ok field?
I added a PR and here are the results:

The GreptimeDB running:

go run main.go
2024/09/11 01:54:47 health check response:

The GreptimeDB stop:

go run main.go
2024/09/11 01:54:38 rpc error: code = Unavailable desc = connection error: desc = "transport: Error while dialing: dial tcp <IP_ADDRESS>:4001: connect: connection refused"
2024/09/11 01:54:38 health check response: <nil>

@daviderli614 It's expected that the response body is empty. When the response is successfully returned, it means the server's connectivity is good, and that's all we need (fow now).

@daviderli614 It's expected that the response body is empty. When the response is successfully returned, it means the server's connectivity is good, and that's all we need (fow now).

ok, PTAL: https://github.com/GreptimeTeam/greptimedb-ingester-go/pull/45

@daviderli614 I've merged #45 , please release it and don't forget to update our docs.

https://github.com/GreptimeTeam/greptimedb-ingester-go/pull/48
After this is merged, can write health check and hints docs together.


Doc about accessing not allocated array's element

I think it's better if we say this in the docs for #[] and #[]= (which are still missing).

Yes, I thought about it, but since I couldn't find any doc for #[] and #[] I put this text where I think it would be nice.
I will think about a text for those two methods and will send another PR. Thanks!

Since you opened a pull request from your master branch, you can update it just by pushing to it.


fix: add terraform failure if reset api key fails to create api key

Description
Issue: https://github.ibm.com/GoldenEye/issues/issues/11046
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content

Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

Updating the cluster autoscaler in this PR itself, so have to skip upgrade test.

/run pipeline

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


Error: Cannot find module node-gyp

Node v10.14.2, NPM 6.5.0
Windows 10
Hi,
Whenever I called node-gyp, I got same error message.

node-gyp
Error: Cannot find module C:\Users\Asus\AppData\Roaming\npm\node_modules\npm\node_modules\node_modules\node-gyp\bin\node-gyp.js
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:581:15)
at Function.Module._load (internal/modules/cjs/loader.js:507:25)
at Function.Module.runMain (internal/modules/cjs/loader.js:742:12)
at startup (internal/bootstrap/node.js:282:19)
at bootstrapNodeJSCore (internal/bootstrap/node.js:743:3)

İssue is module path is wrong. I check paths
Correct Path:
"C:\Users\Asus\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js"
Wrong Path:
"C:\Users\Asus\AppData\Roaming\npm\node_modules\npm\node_modules\node_modules\node-gyp\bin\node-gyp.js"
under node_modules directory there is no another node_modules directory and I check .cmd file it has same path structure with other modules, but system generate wrong path, so it cannot find the module. I see I am not alone, many people has same problem on Windows. Is the module tested on Windows? Because I am sure about I read the installation instructions carefully. I set all configuration variables and installed windows-build-tools successfully

I solved problem. I want to share maybe it can help others too.
Firstly, there are a few node-gyp.cmd files, so I have to find correct cmd file
CMD Filepath:
"%userprofile%\AppData\Roaming\npm\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin"
and open notepad as administrator and change content as
@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\..\..\..\..\..\node-gyp\bin\node-gyp.js" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\..\..\..\..\..\node_modules\node-gyp\bin\node-gyp.js" %*
)

It is work for me, I hope it helps you too.

I solved problem. I want to share maybe it can help others too.
Firstly, there are a few node-gyp.cmd files, so I have to find correct cmd file
CMD Filepath:
"%userprofile%\AppData\Roaming\npm\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin"
and open notepad as administrator and change content as
@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\..\..\..\..\..\node-gyp\bin\node-gyp.js" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\..\..\..\..\..\node_modules\node-gyp\bin\node-gyp.js" %*
)

It is work for me, I hope it helps you too.

It is work for me, thanks man! @Catutigernos

I solved problem. I want to share maybe it can help others too.
Firstly, there are a few node-gyp.cmd files, so I have to find correct cmd file
CMD Filepath:
"%userprofile%\AppData\Roaming\npm\node_modules\npm\node_modules\npm-lifecycle\node-gyp-bin"
and open notepad as administrator and change content as
@IF EXIST "%~dp0\node.exe" (
  "%~dp0\node.exe"  "%~dp0\..\..\..\..\..\node-gyp\bin\node-gyp.js" %*
) ELSE (
  @SETLOCAL
  @SET PATHEXT=%PATHEXT:;.JS;=;%
  node  "%~dp0\..\..\..\..\..\node_modules\node-gyp\bin\node-gyp.js" %*
)

It is work for me, I hope it helps you too.

Didn't helped, and I simply removed this dir node-gyp-bin under:
C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle - worked just fine.

Also got it right by removing the node-gyp-bin folder under C:\Program Files\nodejs\node_modules\npm\node_modules\npm-lifecycle. Thanks @TemaSM

For some reason, deleting that folder fixed this error for me as well.  But I can't understand why?  Any ideas?

@Catutigernos @TemaSM @shirilele @psyklopz  for whatever reason when calling node-gyp you end up running npm internal node-gyp script. I know it has been some time, but maybe the issue still reproduces on your machines? If so:

can you run where node-gyp and paste the results here?
can you share your PATH environment variable? Pasting the result of running echo %PATH% would be great.
do you remember maybe your "Node history"? Which version you installed, if you upgraded it, maybe the packages that you installed? There is definitely something off in your setups, knowing your upgrade path would be helpful to find the offending package.

Thanks!

C:\Users\Eric>where node-gyp
C:\Users\Eric\AppData\Roaming\npm\node-gyp
C:\Users\Eric\AppData\Roaming\npm\node-gyp.cmd

I don't recall my node history, this is a fairly new laptop, and the issue occurred almost immediately.  There may have been npm install -g node-gyp and npm remove -g node-gyp.  I was using v10 when the problem first started.

@psyklopz Thanks!

seems like this might be resolved, also see https://github.com/felixrieseberg/windows-build-tools/pull/188


Specify Feedback links to go to the Qualtrics survey

Changes in this PR

Changed link on banner and specification sidebar to the Qualtrics survey link

Screen-shots or screen-capture of UI changes

https://user-images.githubusercontent.com/77166906/163205078-cd6797b9-4766-451b-ae5c-463b7581c76c.mov

 Specify Feedback links to go to the Qualtrics survey


Add support for PTR records in forward zones

It's a common case when you manage only part of C-class to have this kind of records in forward zone.
And as I've just verified there's no support for CNAME RR's in reverse zones - same use case scenario.

I've never heard of the use of PTR records in forward zones before and could not find anything relevant via Google. Can you show me some examples of this usage?
Regardless, it should be trivial for us to add support for both PTR records in forward zones and CNAME records in reverse zones.

It's described in RFC2317, more specifically here: https://tools.ietf.org/html/rfc2317#section-5.2

Perfect, thank you for the explanation. Support added.

That's what I call lightning fast issue resolution :smile: Thnx


integration with Jenkins

How can I integrate the dpxdt with Jenkins as I intend to use the visualization diff as a part of CI?
Regards.

There's currently no "output" from approving a build. Nothing like a webhook or even an API to poll for when the state of a build changes. I wonder if you'd need that too?

Not to drag old conversations up but this would be a great addition any chance it could get included at some point?

@hutchic Thanks for the feedback. Could you explain how you'd like the API to work? I could use some guidance on the details. Are you okay with polling for statuses of builds like @slmoloch described?

polling would work web hook would be better (IMHO) :)

+1 for webhook - I want one in general for posting back to our deploy slack channel


OWD project: use webref as the source for CSS formal syntax, instead of mdn/data

This proposes that we should start using the w3c's webref project as the source of CSS formal syntax, instead of mdn/data.
Problem statement
In our CSS documentation we have a section "Formal syntax". This is built using a KS macro CSSSyntax, which gets the data from the mdn/data repo.
The specifications are the source of truth for this data, so it's not optimal having this separate copy in mdn/data. Sometimes the version in mdn/data falls behind and we have to remember to update it, or people get outdated information.
It turns out that the w3c already makes this information available in a machine-readable way in their webref project. So this issue proposes that we:

start using webref as the source for this, rather than mdn/data
stop maintaining the formal syntax in mdn/data.

Priority assessment
This table checks this project against the OWD prioritization criteria.

Criteria
Assessment

Effort
Small. We'd need to update the CSSSyntax to fetch the data from webref rather than mdn/data. It's a complex macro but I hope we can leave most of the complexity untouched.

Dependencies
Depends on dev team for review of changes to the KS macro and to bless a new dependency in Yari. Also depends on any key users of mdn/data (most obviously CSS Tree) being informed and helped to move away from mdn/data.

Community enablement
Low (though helps us build more things with the w3c).

Momentum
None.

Enabling learners
No. CSS formal syntax is pretty much a power-user thing.

Enabling professionals
Would help ensure professionals get up-to-date formal syntax in MDN pages.

Underrepresented topics / ethical web
N/A

Operational necessities
N/A

Addressing needs of the Web industry
N/A

Task list

Understand the interface of webref and how it maps to mdn/data, and whether there is anything missing that we would need.
Get agreement from the dev team to add webref as a Yari dependency.
Get agreement from key users of mdn/data to migrate away from it, towards webref probably.
Update Yari dependencies to include webref.
Update CSSsyntax to use webref instead of mdn/data.

Additional comments
There's a bigger version of this proposal in which we replace not just the formal syntax, but all the CSS data in mdn/data, with webref, and may then be in a position to retire mdn/data entirely. But this is a major step towards that goal, has value in itself and keeps the project small. If, when we get into it, replacing the other CSS data seems straightforward we could consider doing that in this project as well.

Would this project cover the generation/replacement of https://github.com/mdn/yari/blob/main/kumascript/macros/InterfaceData.json ?
I think all the info in it could be generated by one of the package offered by w3c/webref.

I'm not sure this project would - it's intentionally quite a narrow scope - but definitely making more use of webref generally would be worth looking into, to replace various data sources that we sometimes struggle to maintain.

@dontcallmedom , I filed https://github.com/mdn/yari/pull/4656 the other day, which replaces mdn/data with webref as a source for formal syntax.

I'd like to add that we have a draft PR that implements this proposal at https://github.com/mdn/yari/pull/4656.
It not only uses webref, but also reimplements the macro to use https://www.npmjs.com/package/css-tree to parse the syntax, as a result of which it's able to add pretty-printing and syntax highlighting to the rendered syntax, making it much more readable.

Oh, this is done :)


feature: support cgroupstat metric in runc shimv2

The cgroupstat is to indicate cpu load per cgroup. It is useful for user
to monitor container metrics. Unlike common stats(cpu, memeory, pids
etc.), the cgroupstat comes from kernel, which needs generic netlink
socket as client to fetch data.
The subproject containerd/cgroups provides interface to retrieve stats.
	Stat(handlers ...ErrorHandler) (*v1.Metrics, error)

But it is too heavy to maintain generic netlink socket in each cgroup
instance or socket connection pool in subproject.
As mentioned above, the cgroupstat is for container metric and
containerd-shim is container lifecycle manager which knows container's
lifecycle and configuration better than any kind of sidecar agent.
containerd-shim is best place to handle metric things, like cgroupstat.
With shimv2 API, the container can export metrics by ttrpc interface.
The containerd-shim-runc-v1,2 service will maintain one generic netlink
socket for the container(s). When there is call for Metric for
container, shim will combine common stats with cgroupstat.

Why not support it in runc-v1 shim?

The runc-v1 metrics comes from containerd side, not the containerd-shim.
No plan to bring the generic socket connection pool in containerd to
make it complicated. And it is time to moving on shimv2.
Signed-off-by: Wei Fu<EMAIL_ADDRESS>

ping @crosbymichael and @Random-Liu


release: 3.12.0

Checklist

[x] Assign to next release manager
[ ] Close previous release milestone
[x] Create next release milestone
[ ] Associate issue with next release milestone

Closing via https://github.com/cdr/code-server/pull/4172


DBZ-4726: Remove SQL Server single-partition mode

Breaking changes:

The database.dbname configuration parameter for the SQL Server connector is no longer supported.
The message keys in the Kafka Connect offsets topic are now expected to contain the source database name.
The database history is now expected to contain the database name in the source metadata of its messages.
The topic and the schema names now include the database name: server1.dbo.MyTable → server1.testDB.dbo.MyTable.
Snapshot SELECT override statements should now include the database name. Not sure how this is going to work with multiple databases. See: https://github.com/debezium/debezium/blob/f465f5ad8672ac4689cfc033e1c105d2e0901fe7/debezium-connector-sqlserver/src/test/java/io/debezium/connector/sqlserver/SnapshotWithSelectOverridesIT.java#L112-L125 The closest open issue in Jira is DBZ-4799 but there's no issue namely about overriding SELECT statements.
The connector metrics will be exposed in a different format. See https://github.com/debezium/debezium/pull/2946 for details and examples.

TODO:

[x] Update documentation.

Documenting the upgrade path is tracked in DBZ-4827.

@morozov I'll take a look at the PR later but yes breaking changes are acceptable for 2.0.

IIRC, if the signaling table is in the same database as the one being incrementally snapshotted, it should be all fine. The existing integration tests should cover this.
The issue is in fact about not being able to incrementally snapshot multiple databases (for which the multi-partition mode is required), not about the impact of switching to the multi-partition mode as such.

We haven't tried that. It's possible to incrementally snapshot a single database in the multi-partition mode according to the tests. How does one incrementally snapshot multiple databases or not seems irrelevant to this patch and is the job for DBZ-4799.

@morozov Applied, thanks!


Memory leak from props that capture the surrounding component

Reproduction
https://github.com/softwareCobbler/formkit-leak-repro
Describe the bug
I noticed my app had a small memory leak, and I think I have narrowed it down to cases where a FormKit element has event handlers that capture refs in a surrounding Vue component.
I've linked a repro as a git repo, I think it works better as a local build so that there is no vite devtools stuff polluting the retainers view in Chrome's memory inspector.
The reproduction works in the following way:

hit http://localhost:8000 (or whatever url it spins up at) and toggle the element, see that after forcing GC the large allocated objects are collected. This is the non-FormKit plain ol' input element behavior.
hit http://localhost:8000/?fk and toggle the element, see that after forcing GC the large allocated objects are not collected. This is the FormKit variant.

It's recommended to open each new url in a fresh tab because chrome seems to reuse heaps when it sees fit to do so (and it seems Chrome will share heaps across "duplicated" tabs, too).
I think that when unmounting a component C, the refs C creates in its setup() should become collectable; the bug being that if there is a FormKit element E as a child of C, and E captures a ref in C, then the ref leaks when C is unmounted.
Also included is an image of the heap snapshot focused on a leaked value (class name is "Leak" to make it easy to identify). In this image, the component owning "Leak" has been unmounted.

Environment
• OS: [e.g. iOS]
• Browser Chrome
• Version 129

Well, this might be a bug in chrome. https://github.com/vuejs/core/issues/12221#event-14743813645