#492 update atlas docker build with latest patches

Signed-off-by: Nigel Jones<EMAIL_ADDRESS>See text in #492 - now updated with patches, mvn build install is clean (so can be merged) but docker build failing on spring issue currently.

I had no issues with previous patches.
I'll check thee current ones too.

Latest Atlas code build works.
After apply all 4 patches from https://issues.apache.org/jira/browse/ATLAS-1773
i get this error:
[ERROR] Failed to execute goal com.webcohesion.enunciate:enunciate-maven-plugin:2.11.1:docs (default) on project atlas-webapp: Enunciate compile failed. -> [Help 1]

@bogdan-sava I didn't get the enunciate error. However I did get issues with the later spring version.
I created an atlas patch which I've added into my docker build - so i now seem to be able to go from atlas master, via the patches, to a runnable atlas. Will try some more tests.
By building in docker (which I need for demo environments) we should get consistent behaviour.
However the build will be fragile (depends on patches), plus as you mentioned we may still have too many interdependencies

Yes! Indeed.
What I found until now is that this new version brings to atlas spring-web and also spring-boot of Egeria and shouldn't. And that's because the ibm-igc-repository-connector is brought to atlas and shouldn't.
And, why ibm-igc-repository-connector it depends on spring-boot-starter-web, which get's dependency of embbeded-tomcat?
Solution:
First exclude  ibm-igc-repository-connector from atlas. In open-metadata module pom.xml :
There is in two dependencies!

org.odpi.egeria
repository-services-implementation
${egeria.version}

ch.qos.logback
*

org.slf4j
log4j-over-slf4j

org.odpi.egeria
ibm-igc-repository-connector

    <dependency>
        <groupId>org.odpi.egeria</groupId>
        <artifactId>admin-services-server</artifactId>
        <version>${egeria.version}</version>
        <exclusions>
            <exclusion>
                <groupId>ch.qos.logback</groupId>
                <artifactId>*</artifactId>
            </exclusion>
            <exclusion>
                <groupId>org.slf4j</groupId>
                <artifactId>log4j-over-slf4j</artifactId>
            </exclusion>
            <exclusion>
                <groupId>org.odpi.egeria</groupId>
                <artifactId>ibm-igc-repository-connector</artifactId>
            </exclusion>
        </exclusions>
    </dependency>

Next step is to exclude admin-services-server and epository-services-implementation from atlas-webapp as there is already dependency of open-metadata module.

It might be runnable but contains ibm-igc-repository-connector and even an embedded tomcat.

Thanks for the investigation.
I can use this PR to build/package an atlas that at least is buildable, hopefully functional, and best-so-far. This could be at least used for a demo. On the basis it's an improvement of what goes before & a tactical aid in enabling demos,  I'd like to integrate it into master so it's consistent with docker hub.
It also means in parallel i can continue to work on any automation of the docker build with the LF team.
I propose we open a new issue to track addressing the igc repo connector & tomcat dependencies, and it's super this issue was identified!.  I completely agree we should try and remove these dependencies from the atlas patch build, but this will take slightly longer

Verified this PR results in a patched Atlas server that starts up when the 'vdc-open' helm chart is deployed
Merging this now so that I can now focus on inbound network routing to atlas (for external access to UI), linking to cohort, and configuring


Jupyter view always back to top when I trying to select the last line

Issue Type: Bug
Jupyter view always back to top when I trying to select the last line
Extension version: 2019.11.50794
VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:54:40.719Z)
OS version: Linux x64 4.19.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-5500U CPU @ 2.40GHz (4 x 2894)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: disabled_softwareskia_renderer: disabled_offsurface_control: disabled_offsurface_synchronization: enabled_onvideo_decode: unavailable_offviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
7.70GB (0.45GB free)

Process Argv
--no-sandbox --unity-launch

Screen Reader
no

VM
0%

Thanks for the issue.
I believe this is being caused by our auto scroll code for focus in the editor. It's auto scrolling to the line with the cursor before the cursor can be updated.

Validated. I'm not able to repro this anymore.

Validated. I'm not able to repro this anymore.

Thanks for resolving this issue. Is it possible to install this change? or do we have to wait till the next release?

Our insider's build should have the fix, but it's rather flakey right now. We should be releasing in the next week or so.

Hello @rchiodo
I just updated the vscode with the latest release. According to the latest release this bug fix is included. However, I am still facing the same problem. Should I have to change any setting? Thanks

Are you talking about the insider's build or the last general release. The last general release does not have this fix in it.

@rchiodo  Latest insiders hasn't been updated with the latest changes. https://github.com/microsoft/vscode-python/issues/8882
I'll get this resolved ASAP

Are you talking about the insider's build or the last general release. The last general release does not have this fix in it.

Sorry for the inconvenience. I mix-up the general release with the insider build. I was talking about the general release.


[FEAT] Improve the website rendering in mobile format

Closes #111

Lighthouse diagnostics on Desktop

Lighthouse diagnostics on Desktop

I think we should fix it in another PR. We have other problems with all the image.
This PR was to fix the margin between the sub section & the image position on the mobile version 🙂


Update download-oledb-driver-for-sql-server.md

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

File
Status
Preview URL
Details

docs/connect/oledb/download-oledb-driver-for-sql-server.md
:white_check_mark:Succeeded

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

#sign-off

#sign-off

#hold-off


Assembly.Location not returning correct value

A recent update changed the way that private assemblies were being loaded from the bin directory. A side effect of this is that now calls to Assembly.Location return an empty string. Some libraries depend on this, and as such, aren't functioning properly.
While we work on a fix, there are two workarounds:
Workaround 1 -- Move the assembly to a different folder:

Move your assembly to a folder named something other than bin. For example, sharedfolder.
Update your references to match the relocated assemblies. For example #r ..\sharedfolder\assembly.dll
Restart the site by clicking Function app settings, then Go to App Service Settings, and clicking Restart at the top of the opened portal blade.

Workaround 2 -- Revert to the previous Functions version:

In the portal, click Function app settings
Click Configure app settings
In the App settings section of the new portal blade, set FUNCTIONS_EXTENSION_VERSION to 1.0.10661 and click Save. This will restart your site and configure it to use the previous version. Note that you will need to reset this value to ~1 in order to receive future updates.

It also happen on me and Workaround 2 completely avoid issue, thank you very much for share!
My situation was here.

Using Roslyn Compiler service inside Azure Functions to evaluate C# Code requested from Slack.
Issue happen when loading Assembly info at https://github.com/guitarrapc/AzureFunctionsIntroduction/blob/master/CSharpScripting.csx#L39

Issue fix at my environment with following.
https://github.com/guitarrapc/AzureFunctionsIntroduction/pull/3
Appreciate for advice! Great help.

@guitarrapc thank you for sharing the details! I'm glad the workaround addressed the issue without requiring you to target the previous version.
As mentioned on the Twitter conversation, we'll be addressing this issue. Apologies for the inconvenience.

For private assemblies, this behavior will be kept and won't change on the current major version. Closing.


does we support to setup value for each element in enum schema definition?

define in scheme like:
{
enum Duration {
DAY
NIGHT
WEEK
}
}
generated java like:
public enum Duration {
DAY, NIGHT, WEEK;
private Duration() { /* compiled code */ }
}
Do we support to generated java enum with values like:
public enum Duration {
DAY("day"), NIGHT("night"), WEEK("week");
private String value;
private Duration(String value) {
this.value = value;
}
public String getValue() {
return this.value;
}
}

@WaterZhang - no we don't support this currently.

got it. Thanks.


Can send message but can't receive

Hi.
Thanks for the sample, it's been very helpful to get me started!
I tested the sample on two devices:

Nexus 5 (Android 6.0.1)
ASUS K019 (Android 4.4.2)

I paired both devices, and ASUS K019 can send message perfectly to my Nexus 5 (and received perfectly too).
But, my Nexus 5 can't seem to send message to ASUS device. There's no error message on Nexus, but ASUS isn't showing up any messages sent from Nexus device.
Any idea?
Thanks!

I just tested using two ASUS K019 (Android 4.4.2) devices.
(These two devices are identical)
Device A can send message to Device B (and received perfectly), but Device B can't send message to Device A.

Hi fmulyono,
I am a newcomer to android so sorry if my my post is not good enough.
I have tried the sample on these devices:
1. Xiaomi HM 1W (Android ver 4.2.2)
2. Samsung S3 ( Android ver 4.3)
The Devices can connect to each other, however it is always a one way chat.
I have researched on my own and found this link:
https://android-er.blogspot.sg/2016/04/problem-of-bluetoothchat-example.html
It just shows that changing the build SDK could solve some of the problem, but the root cause is unknown.
If anyone has any information please feel free to share. Thanks!

Give it a try https://github.com/sirmordred/android-BluetoothChat/commit/c9f8491e9fc9628071bfb8aa707a29786ecf9a24

@sirmordred this works!

@sirmordred Good！

Thank you @sirmordred !!!!

Can someone please explain why this worked?

Current state must be defined BEFORE starting ConnectedThread. Failure to do so opens a race condition - the "while (mState == STATE_CONNECTED)" loop in run() may exit because mState is not yet current. Particularly nasty, since it doesn't ALWAYS happen.
also more complete solution is https://github.com/googlesamples/android-BluetoothChat/pull/9

@sirmordred Please see my comment for issue #8 (duplicate). Is the subtitle of the action bar correctly set upon connection or do you get a "connected to null" message?

@luigidifraia  yeah, experienced the same issue after resolving race condition..and fixed it in another commit
see https://github.com/googlesamples/android-BluetoothChat/pull/9
all the patches in PR should be applied in order to get correct action bar changes and logs in logcat

@sirmordred Cool. I see the PR is now closed, which explains it. It's a shame it's not yet merged though. I will keep working on my own fork for now.

@sirmordred I had a throughout look at PR #9 and here's some feedback I'd like to share:

Thank you for the excellent work on fixing those 2 race conditions and the rest of the rationalisations :)
BluetoothChatService.java's line 88 has no real effect. Was it a debug instruction to set a breakpoint or something along those lines?
I seem to understand the setState() method itself might have been initially introduced in order to log state changes at the same time as they were requested: if it were me, I would not have included the functionality to also message back the UI Activity about the change, but that was probably done for convenience and in order to avoid code repetition - i.e. one cannot forget to update the UI Activity after a state change and the code to do so is in a single location. You opted for local changes to mState instead of using a setState() method but are now logging state changes in updateUserInterfaceTitle() and messaging back the UI Activity: again this is something that mixes a logging task about state changes and an UI update task, for convenience, although it feels like the method name is misleading.
I would go for something along these lines, with a method name that explicitly states both purposes and also addresses the fact the device name needs to be shared prior to advertising the state changing to STATE_CONNECTED (as we commented in Issue #8):

    /**
     * Set the current state of the chat connection, log it, and share it
     * with the UI Activity
     *
     * @param state An integer defining the current connection state
     */
    private synchronized void setStateAndUpdateUI(int state) {
        Log.d(TAG, "setState() " + mState + " -> " + state);
        mState = state;

        if (state == STATE_CONNECTED) {
            // Send the name of the connected device back to the UI Activity
            Message msg = mHandler.obtainMessage(Constants.MESSAGE_DEVICE_NAME);
            Bundle bundle = new Bundle();
            bundle.putString(Constants.DEVICE_NAME, device.getName());
            msg.setData(bundle);
            mHandler.sendMessage(msg);
        }

        // Give the new state to the Handler so the UI Activity can update
        mHandler.obtainMessage(Constants.MESSAGE_STATE_CHANGE, state, -1).sendToTarget();
    }

Invocation of the above method would occur everywhere you set mState directly. All calls to updateUserInterfaceTitle() would be removed.
Any thoughts?


Runtime Error when using the autopilot mode of manual.py in a custom map

CARLA version: 0.9.14
Platform/OS: ubuntu 20.04
The problem you have experienced:  I made a customed opendrive map and load into carla. When I run the manual.py and use autopilot mode to drive the vehicle, after some seconds, the UI was off and the terminal showed these runtime errors.
Traceback (most recent call last):
  File "PythonAPI/examples/manual_control.py", line 1284, in game_loop
    world.tick(clock)
  File "PythonAPI/examples/manual_control.py", line 339, in tick
    self.hud.tick(self, clock)
  File "PythonAPI/examples/manual_control.py", line 695, in tick
    t = world.player.get_transform()
RuntimeError: trying to operate on a destroyed actor; an actor's function was called, but the actor is already destroyed.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "PythonAPI/examples/manual_control.py", line 1380, in <module>
    main()
  File "PythonAPI/examples/manual_control.py", line 1372, in main
    game_loop(args)
  File "PythonAPI/examples/manual_control.py", line 1297, in game_loop
    world.destroy()
  File "PythonAPI/examples/manual_control.py", line 364, in destroy
    self.player.destroy()
RuntimeError: trying to operate on a destroyed actor; an actor's function was called, but the actor is already destroyed.

What you expected to happen: It should run without any runtime error.
Steps to reproduce:

put the map into Import Folder and "make import".
Use the map in UE4 and play.
Run manual.py and press "p" to switch to autopilot mode

Hello @john123741 were you able to fix this issue?


Fixed exit code retrieval of spawned processes on Windows

The current implementation assumed that WaitForSingleObject returns the exit code of the run command, but according to the docs  it returns 1 of 4 fixed values indicating the event that caused the function to return (which is not the exit code of the program).
I never used the Windows API, but the GetExitCodeProcess  functions is probably what we need here. I tested this with various different commands and it works as expected. As i said im not familiar at all with the Windows API, so feel free to correct me if there is a bettter way to do this.
Fixes #43580

@Calinou Would be good to make sure that #42069 includes a test for this case too.

Thanks!

Cherry-picked for 3.2.4.


Compilation errors

Hi,
After resolving encoding errors I still have some errors during maven compilation. It's mostly errors about types :
[ERROR] .../deck-editor/MTG Deck Editor/src/editor/database/characteristics/ManaType.java:[111,64] array required, but java.util.List<editor.database.characteristics.ManaType> found
Do you have them ?
Do you think I miss some step during the initialization of the project ?
I found a way to correct them http://stackoverflow.com/questions/20067956/array-required-but-arrayliststring-found but I would like to make sure I don't miss something before begin : there is almost 100 errors of this type.
Thanks!

That's probably because of the java-oo requirement, which adds some rudimentary operator overloading.  I pretty much only use it for list and map accesses, but I think it improves code readability by a lot.  I use Eclipse, so I have it installed as a plugin which is why it isn't part of the pom.xml.

You're right, I now use eclipse with the java-oo plugin and it's work !
Thanks


GH-42087: [Swift] refactored to remove build warnings

Rationale for this change
Building warnings were being generated from the swift code.
What changes are included in this PR?
Refactored the code in ArrowCExporter and CodableTests to remove the build warnings.
Are these changes tested?
Yes, also ran CDataWGo tests locally to test CData interface changes.

GitHub Issue: #42087

Is there a compile option that treats warnings as errors to detect this in CI easily?

@kou Thanks for the review!  Please merge when you get a chance.

Done!


Get the oldest available data?

Is there a way to get the date of the oldest available data in google fit?
With some Apple health kit packages, you can set the start date to 1970 and end date to today, and limit the results to just 1 - Meaning it will return after the first result is found. Is there some similar workaround?

interesting question,
Based on this one
https://developers.google.com/android/reference/com/google/android/gms/fitness/request/DataReadRequest.Builder#setLimit(int)
it look like a definitely yes with a lot of flexibilities, but need to test out if it really works.

Furthermore, I'd love to contribute to this project, however I'm not sure how to get a working local copy of this repo. Aside from the contribution doc, is there anywhere else I can find info?

@shmkane  1970 is the bottom-line timestamp for Google Fit Api. If you called less than that, you will get an app crush without error handler.
So I think the enableServerQueries() is to deal with unsync history across multiple devices instead of you can't read more than a certain timeframe back as long as it exists in server related to that account. But this indeed points out we need to do some work around from it.
However, 2500-3000 based on my past tests is the rough threshold for a single DataReadRequest, which mean if your app risks to do a relative wide range request (5 years with 1 day bucket) or (1 hour with 1s bucket), you might stuck even crush your app
because the amount of data excesses the threshold, assume if you get perfect 3600 data entries from that particular hour.
So for your naïve solution, I felt like it might just be slow, it could be faster if we can use bucket.
Unfortunately setLimit only takes the latest, and hopefully I missed some doc that can do its reverse operation,  otherwise it's quite annoying to make some workaround.

@aboveyunhai not worried about anything before 1970 😅
I do want to explicitly ask: heart rate doesn't have an option to bucket right?
A bit about my setup- I'm using an Apple Watch connected to an iPhone with Google Fit installed. I have a separate Samsung S10 that also has Google Fit installed. The device I'm testing on is the S10.
You mentioned unsync across multiple devices - I wonder if I'm experiencing this issue because of the devices?
For example, I know for a fact that I have HR data from my watch going back to 2019, but if I try to query for data from the beginning of September, I get an empty array returned.

To be clear, I can get the past week or so of data. But nothing from the beginning of last month or even before that.

@aboveyunhai This is borderline a different issue, but after some frustration on getting this sort of functionality to work, I found this SO thread that claims that you can't read more than a certain timeframe back. I wasn't sure if this project had support for the mentioned enableServerQueries()
My current naive solution for no-bucket data is start from the year 2000 (could be older... but lets be real who has older data in fit?) and iteratively, month by month, query and check to see if any data exists. Once I find a month that contains data, the first datapoint should be the first data object available...
However, because I can't look back more than around a couple weeks, I'm stuck.
Am I approaching this the wrong way?

Did you found a way to enable ServerQueries in this project ?


Minor issue reported by a user

I've used the plugin for one of my projects and it looks great but I've had a trivial issue reported to me by a user and said I'd pass it on to the plugin developers. It seems that the checks are, rarely but annoyingly for this user, sometimes a slightly different color than they should be.
Attached is a screenshot showing two checks that should be the same. My guess is that the animation fading the checks in sometimes ends one frame too early resulting in a check that is slightly darker.
The html for these checkboxes is as follows:

Thanks for the awesome plugin.

		<input type="checkbox" checked="checked" />


Avoid duplicate git clone

Only git clone when the value of ADOPTOPENJDK_REPO or ADOPTOPENJDK_BRANCH changed.

Close: #1113
Signed-off-by: Longyu<EMAIL_ADDRESS>

reviewers: @llxia

Tested internally on ppcle.
Saved average around 18 seconds per build.


[no-restricted-paths] rule. Needs help with implementation.

Hey guys. Thanks for a nice work with current library.
I would like to use no-restricted-paths inside my project, but faced some problems.
Idea: I need to disallow importing old components to the new rebirth component directory. My main directory is ./web.
This is current eslintrc.json:

After running eslint I get this error

Eslint should show an error if someone will try to use old components from "components-legacy" in the new folder "components/PageBuilder/".
Will be really appreciated for help

The rule is import/no-restricted-paths, not no-restricted-paths.


Open links in non-default browser

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

Hi! I was wondering if there's a way to make Hyper open links in a specific non-default browser. My default browser is Safari but I'd like links to open in Chrome.

This is something I would be interested in two. I want links to open in Firefox Developer Edition rather than default Firefox.

Currently out of scope


test: Disable deadline on some Hypothesis tests

They might be slow in some cases, but it should not make the CI fail

Codecov Report

Merging #726 into master will decrease coverage by 0.06%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #726      +/-   ##
==========================================
- Coverage   99.42%   99.36%   -0.07%
==========================================
  Files          58       58
  Lines        3668     3632      -36
  Branches      633      633
==========================================
- Hits         3647     3609      -38
- Misses          9       10       +1
- Partials       12       13       +1

Flag
Coverage Δ

#unittests
99.36% <ø> (-0.07%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/schemathesis/extra/pytest_plugin.py
97.53% <0.00%> (-2.47%)
:arrow_down:

src/schemathesis/specs/graphql/schemas.py
96.55% <0.00%> (-0.23%)
:arrow_down:

src/schemathesis/models.py
97.59% <0.00%> (-0.07%)
:arrow_down:

src/schemathesis/runner/impl/core.py
99.09% <0.00%> (-0.02%)
:arrow_down:

src/schemathesis/lazy.py
100.00% <0.00%> (ø)

src/schemathesis/hooks.py
100.00% <0.00%> (ø)

src/schemathesis/targets.py
100.00% <0.00%> (ø)

src/schemathesis/stateful.py
100.00% <0.00%> (ø)

src/schemathesis/cli/context.py
100.00% <0.00%> (ø)

src/schemathesis/cli/cassettes.py
100.00% <0.00%> (ø)

... and 6 more

Continue to review full report at Codecov.

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


"Where" query does not work as expected in specific case

I'm trying to retrieve specific data from my table using a predicate. The researched sql query is as follow :
SELECT [plan].[Id] FROM [Plan] AS [plan] WHERE [plan].[IdCampagne] = 39
Plan is a table with columns

Id [int]
IdCampagne[int]
OtherStuff [int]

Steps to reproduce

WORKING CODE

 var a = _context.Plan.AsNoTracking()
                    .Where(plan => plan.IdCampagne == 39)
                    .Select(plan => new { Id = plan.Id })
                    .ToList();

gives :
SELECT [plan].[Id] FROM [Plan] AS [plan] WHERE [plan].[IdCampagne] = 39

NOT WORKING CODE

Func<Plan, bool> predicate = p => p.IdCampagne == 39;
var a = _context.Plan.AsNoTracking()
        .Where(predicate)
        .Select(plan => new { Id = plan.Id })
        .ToList();

gives :
SELECT [p].[Id], [p].[IdCampagne], [p].[OtherStuff] FROM [Plan] AS [p]
Further technical details
EF Core version: 2.1.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows Server 2012 R2 Standard
IDE:  Visual Studio 2017 15.7.5

You need to create lambda of Expression<Func<Plan, bool>>. Raw Func<> are opaque in expression tree and we cannot look what is inside. Using Expression<Func<>> integrate the lambda inside the expression with inner body.


Some Fixes v2

Removed the previous Pull, since I added an attempted fix for the auto attack script. It does not fully function like the original one, but I hope its a good base.

@loopuleasa I'm gonna leave messages here so I don't spam everyone's emails (RIP Inbox yesterday).
The autoattack command got changed. I used modes 0 and 2 for the fix, You may want to change those depending on what you want the config to achieve (0, 1, 2 follows the settings order I think). Secondly, the auto attack has some strange interactions and needs a stop command in there, but dota_stop isn't working how I want it to. Maybe you can fix that.

I can look into the dota_stop command
I have a question: What are with the .png file changes? They don't load correctly on my side.

I don't know, they kept on preventing me from submitting a pull. They shouldn't be changed except they are for some reason. I suspect its my virus scanner being a whinny bit** as usual (need a better one anyways). If you can "uncommit" those or something that be fantastic.

Oh and the PNG's don't load on my side either.

What did you change on the pics?
I can try to do the changes manually on my side then merge them.

No they shouldn't have. For some reason my PC hates PNG files and corrupts them...
Just tried downloading the original config and they are also bugging out on me, which is really weird.
In any case, all my changes are:
Add the following to dota2_keybinds_default.cfg at the end of the file:
//Mouse3 Fix
bind mouse3 +cameragrip
Autoattack script (dota2_functions_active.cfg):
//  Auto attack toggle (must be first configured here to work properly)
alias "auto_attack_toggle" "auto_attack_toggle_on"
alias "auto_attack_toggle_on" "playsound sounds/ui/tutorial_ui_ding_01.vsnd_c;dota_player_units_auto_attack_mode 2;  alias auto_attack_toggle auto_attack_toggle_off; say_student Auto attack enabled"
alias "auto_attack_toggle_off" "playsound sounds/ui/ui_upgrade_ability_01.vsnd_c;dota_player_units_auto_attack_mode 0; alias auto_attack_toggle auto_attack_toggle_on; say_student Auto attack disabled"

k, I will handle these


docs(security): change bugbounty provider from h1 to bugcrowd

Our bugbounty program is now with Bugcrowd. The HackerOne program is deprecated.

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

Comparison is base (65abc21) 72.64% compared to head (ea4d1d6) 72.62%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #168      +/-   ##
==========================================
- Coverage   72.64%   72.62%   -0.03%
==========================================
  Files          36       36
  Lines        4058     4058
==========================================
- Hits         2948     2947       -1
- Misses       1110     1111       +1

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Escape a pipe ("|") in Snippet

Is it possible to escape a pipe character in a snippet?

Hey there.
I'm using Emmet 1.0.1 on TextMate and I'm still getting this bug.
Was it eventually fixed?

doesn't work with curly brackets '{}'. tried escaping it using double backslashes but it expands to '{'.


fix comment in setup rake task

:) thanks

TY for curating the resource.  I haven't mastered it yet by any means, but
this is really helpful as an example to try and adapt from.
I'm actually pretty green at this, would you mind if I asked a question
about configuring the api keys for production?
Thanks!
-dwin
On Fri, Feb 26, 2016 at 12:24 PM, Ryan Funduk<EMAIL_ADDRESS>wrote:

:) thanks
—
Reply to this email directly or view it on GitHub
https://github.com/rfunduk/rails-stripe-connect-example/pull/9#issuecomment-189412019
.

Sure shoot me an email anytime (contact info here).

Thanks!!!
On Fri, Feb 26, 2016 at 5:06 PM, Ryan Funduk<EMAIL_ADDRESS>wrote:

Sure shoot me an email anytime (contact info here
https://ryanfunduk.com/contact/).
—
Reply to this email directly or view it on GitHub
https://github.com/rfunduk/rails-stripe-connect-example/pull/9#issuecomment-189520411
.


Big size

It's big size bundle because react included by react-color

Waiting to netlify/netlify-cms#1292 to be merged


Set up base image pushes to eks-distro-build-tooling ECR public registry

This setup script makes sure we use the proper Role to push to the eks-distro-build-tooling repo where we have the base image repos.
/hold
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/unhold


BBS Crashing and Failing over

Thank you for submitting an issue to the diego-release repository. We appreciate the feedback. To help us address your issue, please fill out the sections in the following template to the best of your ability:
Summary
We have seen in our very busy production env we see the BBS will crash and failover.  Once the failover occurs the new BBS sometimes takes a while to recover and be in a performant state.
Expected Result
When the BBS is running there should be no unexpected crashes and failovers.
Actual Result
The BBS crashes :)
Context
This is IBM Public Cloud us-south deployment.  We have about 780 Cells.  This is diego 2.25.0 version with CF-Deployment 6.8.0.
This is on SoftLayer.   This is a very large 16 core VM, we are using postgres as the backend and have 400 db connections (min and max).  We cannot change this due to limitations in the postgres deployment.
The file description limit is 100000 as set in bpm.yml.
Steps to Reproduce
Unable to reproduce in smaller envs as this must be due to load or particular errors in some queries.
Possible Causes or Fixes (optional)
Additional Text Output or Screenshots (optional)
I will be attaching logs (or sending them via slack as they are very large in the next day or two.
This was already discussed here ...
https://cloudfoundry.slack.com/archives/C02FM2BPE/p1556639064033100
There are some inital goroutine dumps there (small parts) but will try and get the rest shortly.

This is happening several times a day in our env.  can you please prioritize this so we can get a fix for why the bbs is crashing?   Is there anything else I can provide?

@andrew-edgar just pulled this up in priority, should be the next thing in the backlog

@andrew-edgar are all failures due to the same error: concurrent map read and map write? Could you turn on debug logs in bbs and share them?

@mariash we cannot turn on debug logs.  this is our largest production env and that would cause a HUGE amount of logging which would effect performance

@andrew-edgar We have a fix for the issues we saw in your logs around panics with the error concurrent map read and map write. Do you have any evidence of other failure scenarios causing BBS crashes you see in your environment?

@andrew-edgar closing this out since we have a fix and the story was accepted, it should come out in the next diego-release. Please feel free to open this up again or open a new issue if you see a different class of crashes


[ENTMQBR-1044] Large messages are not deleted if spring MessageTemplate.sendAndReceive method timeout

Issue: https://issues.jboss.org/browse/ENTMQBR-1153
Upstream issue: https://issues.apache.org/jira/browse/ARTEMIS-1791
Upstream PR: https://github.com/apache/activemq-artemis/pull/2000

Can one of the admins verify this patch?

this is ok to test

@mtaylor can you kick this please?

@mtaylor can you kick this please?

@mtaylor is this what you were thinking about?


Discrepency between Reqwest and cURL/python using self-signed certificate and danger_accept_invalid_certs

I'm getting a discrepency between the response returned using reqwest and curl/python when querying a server with a self-signed certificate on a private network.
I'd love to dig deeper into why this happening as it seems quite weird to me. Looking for advice on how to proceed.
cURL
curl --verbose -i -k -X POST https://<IP_ADDRESS>/cgi-bin/luci -d "luci_username=admin&luci_password=admin"

Returns a 302 status (expected)
Reqwest
use reqwest::{Client, ClientBuilder};
let client = Client::builder()
    .danger_accept_invalid_certs(true)
    .build()
    .unwrap();

let status = client
    .post("https://<IP_ADDRESS>/cgi-bin/luci")
    // Tried with and without the following headers which cURL automatically adds
    .header("User-Agent", "curl/7.58.0")
    .header("Accept", "*/*")
    .header("Content-Type", "application/x-www-form-urlencoded")
    .header("Host", "<IP_ADDRESS>")
    .header("Content-Length", "39")
    .body("luci_username=admin&luci_password=admin")
    .send()
    .await
    .unwrap()
    .status();

assert!(status == 302);

Returns a 403 status (unexpected)
cURL Log
*   Trying <IP_ADDRESS>...
 It doesn't seem like the problem is with accepting invalid SSL certificates.

Thank you so much. I didn't see the "redirect" in the logs, only looked for "302".


Issues on "Functions" and "Global Temporaries" context popup menu

ISSUE 1
1- Right mouse click on a Functions node: Verify that the context popup menu show Declare new instead of Create new like it happen with others Database objects (e.g.: Procedures):

ISSUE 2
2 - Right mouse click on a Global Temporaries node: Verify that the context popup menu is NOT shown with Create new and Refresh options like it happen with others Database objects (e.g.: Procedures):

Retest result on FlameRobin v<IP_ADDRESS>: PASSED!


python3Packages.devolo-home-control-api: 0.17.3 -> 0.17.4

Motivation for this change
Update to latest upstream release 0.17.4
Change log: https://github.com/2Fake/devolo_home_control_api/blob/master/docs/CHANGELOG.md#0174---20210701
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Relase notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

3 packages built:

home-assistant
python38Packages.devolo-home-control-api
python39Packages.devolo-home-control-api

@ofborg eval
(Sorry, ofborg's certs recently had an issue, meaning that it was unable to
react to PRs. This has been fixed, but all PRs filed in this period will need
ofborg to be manually kicked off.)


Notifications.getInitialNotification() is undefined when click on notification in quite state [ANDROID]

same hear

same here

In which version of Android are you experiencing this problem?
It may be related to https://github.com/wix/react-native-notifications/pull/920

In which version of Android are you experiencing this problem? It may be related to #920

Android 13

Same for me in fact...

This issue is still valid.

May be solved by https://github.com/wix/react-native-notifications/pull/966

Still valid

Indeed it is solved by #966 , well done @kristofb !!
(using sdk 33)

thanks @kristofb - works.  would appreciate a merge!

The problem still valid

You can apply a patch for now @volle241. Although we still see the issue on some devices so I'm not 100% convinced this solves it.
react-native-notifications+4.3.5.patch

If your provider is FCM, what works for me consistenly was to add yarn add @react-native-firebase/messaging and use:
messaging()
.getInitialNotification()
.then(remoteMessage => {
if (remoteMessage) {
console.log(
'Notification caused app to open from quit state:',
remoteMessage.notification,
);
}
})

Hi, is anyone still stuck on this issue? Because I am. Tried all the options provided here and there:

applying the patch
testing the recent react-native-notifications 5.0.0
use firebase's getInitialNotification instead
but none of these helped

Recap of the bug:
Tapping a notification:

opens correctly when app is on background
opens correctly when app is quitted
opens by restarting the activity when app is on foreground

The activity doesn't seem to really restart entirely: SplashScreen appears but the navigation state is preserved.
It would still be okay if getInitialNotification() would return the notification tapped, but it's not.
Am I the only one?

@TwistedMinda in my case, it works. Can you share the code where you call Notifications.getInitialNotification() ?

@zabojad Hey, good to know it can work!
Well I don't really do anything fancy, just calling it:
Notifications.getInitialNotification().then((notification) => {
  if (notification) {
    handleNotification(notification)
  }
})


Multiple Inputs

Hello,
I was wondering if I could use multiple inputs to generate the structural tree? If I had structures for some proteins but they do not have Uniprot IDs and combined this with a list of Uniprot IDs that I do have, is it currently possible to generate a structural tree this way?
Thank you and I appreciate the work you're doing.
Aamir

Hi. Currently the workaround for this would be to download the alphafold db structures ( or whatever structuress you need ) and run this as a job with a custom set of pdbs. Keep in mind that some crystal structures may have discontinuities or may be truncated proteins due to the constraints of crystalography and this may affect comparison with full length AFDB models for example.


prompt=select_account automatically go to "Other User" when there is no account to choose

Preflight Checklist

[X] I could not find a solution in the existing issues, docs, nor discussions
[X] I have joined the ZITADEL chat

Describe your problem
If you log into zitadel by appending the query parameter prompt=select_account it always shows the account selection page even if there is no account to select. In this case, the user has to choose "Other Account" anyway.

Describe your ideal solution
Can we just automatically go the "Other Account" path?
Version
No response
Environment
Self-hosted
Additional Context
No response

@muhlemmer @livio-a whats your opinion on that?

I've checked the specs. from what i see the user has to select any account. it does not specify exactly how. as we already allow the other user option, the user has the possibility "select" a none existing session and still be compliant with the specs.
BTW. others (Google, Azure, ...) do the same
i'll move i into the backlog / small issues list


I Need Help

Even though I've downloaded  haxelib git linc_luajit https://github.com/nebulazorua/linc_luajit.git and haxelib git hxvm-luajit https://github.com/nebulazorua/hxvm-luajit, I'm still getting this error...
source/LuaClass.hx:35: characters 46-58 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:1669: characters 42-54 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:1469: characters 42-54 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:1144: characters 42-54 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:855: characters 41-53 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:635: characters 40-52 : Type not found : StatePointer
source/Replay.hx:55: lines 55-145 : Defined in this class
source/LuaClass.hx:370: characters 42-54 : Type not found : StatePointer
What do I do to fix this?

https://github.com/KadeDev/Kade-Engine/issues/2074#issuecomment-908939026

From the troubleshooting part of https://github.com/KadeDev/Kade-Engine/blob/stable/docs/building.md

If you are experiencing one of the common problems below, you can use the provided steps to fix the issue.
Type not found : StatePointer
If you experience this problem, you may see lines like
source/LuaClass.hx:33: characters 46-58 : Type not found : StatePointer
This problem occurs when running an old version of the libraries that power the Lua-based modcharting system. These libraries were replaced in Kade Engine v1.7.
To fix this issue, run the following commands to reinstall the proper versions:
haxelib remove linc_luajit
haxelib remove hxvm-luajit
haxelib git linc_luajit https://github.com/nebulazorua/linc_luajit.git
haxelib git hxvm-luajit https://github.com/nebulazorua/hxvm-luajit


Pinning chef-provisioning to 1.1.1

Later versions of chef-provisioning will depend on winrm 1.3, which conflicts with knife-windows (0.8.4 depends on winrm 1.2) when bundled in the ChefDK.  We can include chef-provisioning 1.2 when knife-windows 1.0 is released because both of those will depend on winrm 1.3
\cc @chef/client-maintainers

:+1: I think the ChefDK build is somehow getting chef-provisioning from master... my guess is that even though I removed chef-provisioning as a project dep from chef-dk, it's probably still a software dep of the chef-provisioning drivers and now we're building it with the default omnibus version which is master.

@danielsdeleo I agree - that was also my guess


2490: add feature flag to enable/disable CC notifications about user …

…changes
Context
When CC api stops accepting requests for user changes we get a lot of failed jobs in the Retries queue of sidekiq. Computacenter::ServiceNowUserImportAPI::Error
Changes proposed in this pull request
A new feature flag to enable/disable the generation of background jobs to hit CC api for user changes.
Guidance to review

 Computacenter::ServiceNowUserImportAPI::Error


fix: Admin customize code highlight cannot be changed

https://redmine.weseek.co.jp/issues/106061

Close this and new pr will take care of this fix


"Cannot enlarge memory arrays" error processing large graph

I've got following error message trying process large graph. I suppose it is referring to Emscripten compile options.
Uncaught abort("Cannot enlarge memory arrays. Either (1) compile with -s TOTAL_MEMORY=X with X higher than the current value 16777216, (2) compile with -s ALLOW_MEMORY_GROWTH=1 which adjusts the size at runtime but prevents some optimizations, (3) set Module.TOTAL_MEMORY to a higher value before the program runs, or if you want malloc to return NULL (0) instead of this abort, compile with -s ABORTING_MALLOC=0 ") at Error at Ga (http://viz-js.com/bower_components/viz.js/viz.js:49:124) at Fa (http://viz-js.com/bower_components/viz.js/viz.js:49:22) at v (http://viz-js.com/bower_components/viz.js/viz.js:1183:101) at Function.ba [as Wa] (http://viz-js.com/bower_components/viz.js/viz.js:34:488) at uc (http://viz-js.com/bower_components/viz.js/viz.js:1103:161) at eQ (http://viz-js.com/bower_components/viz.js/viz.js:1168:7157) at zz (http://viz-js.com/bower_components/viz.js/viz.js:1160:92650) at yz (http://viz-js.com/bower_components/viz.js/viz.js:1160:92582) at Wi (http://viz-js.com/bower_components/viz.js/viz.js:1163:4858) at Vi (http://viz-js.com/bower_components/viz.js/viz.js:1163:4250) If this abort() is unexpected, build with -s ASSERTIONS=1 which can give more information

When processing a large graph, you can try using the totalMemory option. For example:
Viz(src, { totalMemory: 32 * 1024 * 1024 })

It doesn't seem that this option is working anymore.
Also by inspecting the code I was not see evidence that this parameter is being passed down.
PS. I'm using url web workers

@mdaines any new set suggestions for working around.?

+1 would also find this helpful


Add an option to run Swift build functions silently (without any output to stdout)

For usage in dedicated build / bundling tools, it would be handy to configure or use the swift_bridge_build functions silently, without producing output to stdout (being aware of the drawbacks, this includes cargo-commands (cargo-rerun-if, etc.)).
A few possible solutions:

crate feature flag (e.g. silent) that is disabled by default (+ does not bother users who do not need this, - makes the setting global, so no fine-grained control possible)
introduce additional parameter to functions (+ fine-grained control, - bothers everybody, even if this is not needed / not advisable for people who use swift_bridge_build in a build.rs file)
introduce a silent function for every function (+ fine-grained control, - refactoring / maintainance and probably migration overhead, - invoking the wrong version of a function accidentally is possible)
provide no solution for this. If using swift_bridge_build in separate tools instead of invoking it in build.rs is strongly discouraged, this is also an option to consider.

Context: I am building cargo-swift that is intended to be an opinionated, user-friendly convenience layer above swift_bridge, mainly aiding in setting up boilerplate and building a Swift package from your codebase without requiring any extra build configuration (steps that would be required in build.rs (e.g. generating bridging header) as well as scripts such as lipo. It therefore pursues other goals than the swift-bridge-cli which enables more fine-grained control but requires setting up a custom project build config and external tools (e.g. lipo) to be invoked manually)
I am willing to work on this feature after discussing how it should be integrated. I am in favor of providing this as a crate feature that is disabled by default.

I'm only aware of the cargo:rerun-if-changed lines being written to stdout. Is there anything else that you have in mind?
I think we want swift-bridge-build to be silent when there aren't any errors.
We can actually delete the cargo:rerun-if-changed that you're running into since it isn't needed.
(The files that we're printing out don't even exist anymore.)
So, we can delete this:
https://github.com/chinedufn/swift-bridge/blob/54b9c09337701dd7580010d1d5ef6527b855ee98/crates/swift-bridge-build/src/generate_core.rs#L46-L54
Does that solve your problem?

If using swift_bridge_build in separate tools instead of invoking it in build.rs is strongly discouraged, this is also an option to consider.

Thanks for listing out all of the potential options that you had in mind. I'd just like to say that using swift-bridge-build from other tools is very much encouraged!
Please feel free to open issues whenever you run into something that cargo swift needs.

Perfect. I'll add a PR for this.
The culprit for the remaining stdout is actually xcodebuild (which prints out "xcframework successfully written out to: ..."). Since xcodebuild has a --quiet option, this can be silenced. But as this output can be actually useful, always silencing is probably not appropriate. I'll include a suggested implementation in my PR.

But as this output can be actually useful

I'm sure that the stderr output is useful if you're trying to figure out what went wrong during a build, but when is the stdout output useful?

We should be able to capture the stdout from the xcodebuild command by using Command.output instead of Command.spawn.
https://github.com/chinedufn/swift-bridge/blob/03de7ea4fb646de1b71358a717488ddac5d67e97/crates/swift-bridge-build/src/package.rs#L230-L233

Re-opening since I think we should capture the xcodebuild output and, at least for now, do nothing with the stdout.
This should make the command completely silent on success, and panic with with errors.
Then at some point in the future we'd want to refactor the create package function to return a Result.

Closed via https://github.com/chinedufn/swift-bridge/pull/144


Add overriding of setup.php config file, via constants

I'm working on a custom install of Grav, where being able to dynamically swap setup files was required. I've modified the $file assignment to permit overriding it's value via a defined constant, "GRAV_SETUP_PATH".
And I thought that might be useful to others, so I'm submitting a pull request.

Looks good... would be good to get this documented in learn.getgrav.org too.


Debug prompts use_default_colors() returned ERR

I was running a example code from tensorpack on Pycharm, which runs properly, then I changed the session to
sess = tf_debug.LocalCLIDebugWrapperSession(sess)  in order to debug
as suggested by official example https://www.tensorflow.org/programmers_guide/debugger, but then I got:
Traceback (most recent call last):
  File "/home/user/.pycharm_helpers/pydev/pydev_run_in_console.py", line 53, in run_file
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "/home/user/prj/shufflenet_v1/shufflenet.py", line 231, in <module>
    launch_train_with_config(config, SyncMultiGPUTrainerParameterServer(nr_tower))
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/train/interface.py", line 96, in launch_train_with_config
    config.steps_per_epoch, config.starting_epoch, config.max_epoch)
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/train/base.py", line 288, in train
    self.main_loop(steps_per_epoch, starting_epoch, max_epoch)
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/utils/argtools.py", line 171, in wrapper
    return func(*args, **kwargs)
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/train/base.py", line 239, in main_loop
    self.loop.update_global_step()
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/train/base.py", line 59, in update_global_step
    self._global_step = get_global_step_value()
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorpack/tfutils/common.py", line 77, in get_global_step_value
    get_global_step_var())
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/training/training_util.py", line 67, in global_step
    return int(sess.run(global_step_tensor))
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/wrappers/framework.py", line 455, in run
    is_callable_runner=bool(callable_runner)))
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/wrappers/local_cli_wrapper.py", line 253, in on_run_start
    self._prep_cli_for_run_start()
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/wrappers/local_cli_wrapper.py", line 275, in _prep_cli_for_run_start
    self._run_cli = ui_factory.get_ui(self._ui_type)
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/cli/ui_factory.py", line 56, in get_ui
    return curses_ui.CursesUI(on_ui_exit=on_ui_exit)
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/cli/curses_ui.py", line 285, in __init__
    self._screen_init()
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/cli/curses_ui.py", line 400, in _screen_init
    self._screen_color_init()
  File "/home/user/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/debug/cli/curses_ui.py", line 405, in _screen_color_init
    curses.use_default_colors()
error: use_default_colors() returned ERR

which I have no clue for the reason, can someone help?

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

I run tfdbg in pycharm, switch to terminal solve the problem

Thanks for sharing your solution! cc/ @caisq


Add Null Mask to Prefix and Suffix Iters

Which issue does this PR close?

This is a follow-up (possible?) of #6231 and incur discussions https://github.com/apache/arrow-rs/pull/6306/files#r1731703196.
# xinli @ arch-dev in ~/source/repos/arrow-rs on git:master o [20:44:53]
$ critcmp master_08_27 nullable_08_27
group                                        master_08_27                           nullable_08_27
-----                                        ------------                           --------------
like_utf8view scalar complex                 1.00    171.9±3.44ms        ? ?/sec    1.00    171.2±2.01ms        ? ?/sec
like_utf8view scalar contains                1.00    127.1±1.40ms        ? ?/sec    1.02    129.7±2.94ms        ? ?/sec
like_utf8view scalar ends with 13 bytes      1.00     30.9±0.95ms        ? ?/sec    1.09     33.8±0.72ms        ? ?/sec
like_utf8view scalar ends with 4 bytes       1.00     31.9±0.55ms        ? ?/sec    1.10     35.0±0.51ms        ? ?/sec
like_utf8view scalar ends with 6 bytes       1.00     31.5±0.45ms        ? ?/sec    1.10     34.7±0.42ms        ? ?/sec
like_utf8view scalar equals                  1.00     26.9±1.08ms        ? ?/sec    1.00     26.9±0.44ms        ? ?/sec
like_utf8view scalar starts with 13 bytes    1.00     30.5±0.49ms        ? ?/sec    1.14     34.6±0.42ms        ? ?/sec
like_utf8view scalar starts with 4 bytes     1.25     21.9±0.31ms        ? ?/sec    1.00     17.5±0.51ms        ? ?/sec
like_utf8view scalar starts with 6 bytes     1.00     30.9±0.46ms        ? ?/sec    1.15     35.7±0.61ms        ? ?/sec

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

New benchmark
like_utf8view scalar complex
----------------------------
nullable_08_27       1.00     171.2±2.01ms       ? ?/sec
master_08_27         1.00     171.9±3.44ms       ? ?/sec
nullable_08_27_2     1.02     174.8±6.85ms       ? ?/sec

like_utf8view scalar contains
-----------------------------
nullable_08_27_2     1.00     124.7±1.50ms       ? ?/sec
master_08_27         1.02     127.1±1.40ms       ? ?/sec
nullable_08_27       1.04     129.7±2.94ms       ? ?/sec

like_utf8view scalar ends with 13 bytes
---------------------------------------
master_08_27         1.00      30.9±0.95ms       ? ?/sec
nullable_08_27       1.09      33.8±0.72ms       ? ?/sec
nullable_08_27_2     1.19      36.8±0.79ms       ? ?/sec

like_utf8view scalar ends with 4 bytes
--------------------------------------
master_08_27         1.00      31.9±0.55ms       ? ?/sec
nullable_08_27       1.10      35.0±0.51ms       ? ?/sec
nullable_08_27_2     1.19      38.0±3.34ms       ? ?/sec

like_utf8view scalar ends with 6 bytes
--------------------------------------
master_08_27         1.00      31.5±0.45ms       ? ?/sec
nullable_08_27       1.10      34.7±0.42ms       ? ?/sec
nullable_08_27_2     1.18      37.1±0.90ms       ? ?/sec

like_utf8view scalar equals
---------------------------
nullable_08_27       1.00      26.9±0.44ms       ? ?/sec
master_08_27         1.00      26.9±1.08ms       ? ?/sec
nullable_08_27_2     1.02      27.4±1.58ms       ? ?/sec

like_utf8view scalar starts with 13 bytes
-----------------------------------------
master_08_27         1.00      30.5±0.49ms       ? ?/sec
nullable_08_27       1.14      34.6±0.42ms       ? ?/sec
nullable_08_27_2     1.19      36.2±0.61ms       ? ?/sec

like_utf8view scalar starts with 4 bytes
----------------------------------------
nullable_08_27       1.00      17.5±0.51ms       ? ?/sec
master_08_27         1.25      21.9±0.31ms       ? ?/sec
nullable_08_27_2     1.29      22.6±0.30ms       ? ?/sec

like_utf8view scalar starts with 6 bytes
----------------------------------------
master_08_27         1.00      30.9±0.46ms       ? ?/sec
nullable_08_27       1.15      35.7±0.61ms       ? ?/sec
nullable_08_27_2     1.23      38.2±2.02ms       ? ?/sec


--master=fqdn:port fails to work

If you use:
--master=ip:port
It works.  If you use:
--master=http://fqdn:port
It works.  But if you use
--master=fqdn:port it fails with:
unsupported protocol scheme "fqdn"
Seems like this should work too....

@eparis Is this for the master/kubelet/kubectl command line?

I'd have to go back and look, but as I recall:
kubectl doesn't take --master= it takes --server
kubelet doesn't take --master= it takes --api_servers=
so this would be limited to the proxy, scheduler, and controller-manager.  Which all take --master, although I don't remember if that is shared code...

kubelet also has a similar same problem:
works:
--api_servers=<IP_ADDRESS>:8080
--api_servers=http://<IP_ADDRESS>:8080
--api_servers=http://localhost:8080
fails:
--api_servers=localhost:8080


WIP: Attempt to build arm64 on GitHub Actions

Might improve #34
GitHub Actions' yaml is super hard to debug locally, so expect this PR to contain lots of frustrated commits :)

Oops the latest workflow is running for over an hour. @zamazan4ik could you cancel it, or does it matter? I'll address this in the next commit.

Don`t worry - it will ba cancelled automatically

Looking at this line https://github.com/rust-serverless/lambda-rust/runs/5419998450?check_suite_focus=true#step:4:539,
  1.58.1-aarch64-unknown-linux-gnu installed - rustc 1.58.1 (db9d1b20b 2022-01-20)

I'm reasonably confident that the image is building on QEMU. Currently we only run the tests against the x64 image, since running the test on QEMU is too slow.
I'll run the test with the arm64 image on my local machine once it's published. @zamazan4ik sorry for the GitHub workflow spam, please feel free to make any edit, too!

Thanks!


Benchmark information?

@ipazc , could provide more details about the benchmark setup?  In particular:

What minFaceSize, scaleFactor and stepsThreshold where used?
Could you please share the benchmark test images, so i can compare it to my benchmarks?

Thank you!

@ipazc I'm also interested in this


Design and implement readonly version of missing form components

Add prop readonly to the new form components:

[ ] Select
[ ] Text input
[ ] Textarea
[ ] Date input
[ ] Password input
[ ] Number input
[ ] File input
[x] Radio Group
[ ] Others

The read-only state has some key features:

When it's true, it disables onChange event. No interaction with the component changes its value.
The regular states of the component work as usual (focus, hover, active, etc).
Changes the look of the component.

Requested in https://github.com/dxc-technology/halstack-react/issues/1290.


Issue with map

Describe the bug
Map has been taking extremely long to preprocess my data.
It seems to process 1000 examples (which it does really fast in about 10 seconds), then it hangs for a good 1-2 minutes, before it moves on to the next batch of 1000 examples.
It also keeps eating up my hard drive space for some reason by creating a file named tmp1335llua that is over 300GB.
Trying to set num_proc to be >1 also gives me the following error: NameError: name 'processor' is not defined
Please advise on how I could optimise this?
Steps to reproduce the bug
In general, I have been using map as per normal. Here is a snippet of my code:
###########################        DATASET LOADING AND PREP        #########################

def load_custom_dataset(split):
    ds = []
    if split == 'train':
        for dset in args.train_datasets:
            ds.append(load_from_disk(dset))
    if split == 'test':
        for dset in args.test_datasets:
            ds.append(load_from_disk(dset))

    ds_to_return = concatenate_datasets(ds)
    ds_to_return = ds_to_return.shuffle(seed=22)
    return ds_to_return

def prepare_dataset(batch):
    # load and (possibly) resample audio data to 16kHz
    audio = batch["audio"]

    # compute log-Mel input features from input audio array
    batch["input_features"] = processor.feature_extractor(audio["array"], sampling_rate=audio["sampling_rate"]).input_features[0]
    # compute input length of audio sample in seconds
    batch["input_length"] = len(audio["array"]) / audio["sampling_rate"]

    # optional pre-processing steps
    transcription = batch["sentence"]
    if do_lower_case:
        transcription = transcription.lower()
    if do_remove_punctuation:
        transcription = normalizer(transcription).strip()

    # encode target text to label ids
    batch["labels"] = processor.tokenizer(transcription).input_ids
    return batch

print('DATASET PREPARATION IN PROGRESS...')

# case 3: combine_and_shuffle is true, only train provided
# load train datasets
train_set = load_custom_dataset('train')

 # split dataset
raw_dataset = DatasetDict()
raw_dataset = train_set.train_test_split(test_size = args.test_size, shuffle=True, seed=42)

raw_dataset = raw_dataset.cast_column("audio", Audio(sampling_rate=args.sampling_rate))

print("Before Map:")
print(raw_dataset)

raw_dataset = raw_dataset.map(prepare_dataset, num_proc=1)

print("After Map:")
print(raw_dataset)

Expected behavior
Based on the speed at which map is processing examples, I would expect a 5-6 hours completion for all mapping
However, because it hangs every 1000 examples, I instead roughly estimate it would take about  40 hours!
Moreover, i cant even finish the map because it keeps exponentially eating up my hard drive space
Environment info

datasets version: 2.18.0
Platform: Windows-10-10.0.22631-SP0
Python version: 3.10.14
huggingface_hub version: 0.22.2
PyArrow version: 15.0.2
Pandas version: 2.2.1
fsspec version: 2024.2.0

Default writer_batch_size is set to 1000 (see map).
The "tmp1335llua" is probably the temp file it creates while writing to disk.
Maybe try lowering the writer_batch_size.
For multi-processing you should probably pass the processor as an argument (with e.g. partial) to the function or create it inside so that the sub-processes have access to it and maybe add if __name__ == "__main__"  (not sure that's necessary?).

I'm having a similar issue with the spectrographs taking up an incredibly large amount of space. (i.e. 100GB for 3GB of audio). Is this really normal behavior?

Same problem, dataset.map takes long time to process 12GB raw audio data and create 200GB cache file. Is there any method can run process(map) during train, instead current run
once and save cache file ?

Same issue here. Just trying to normalise image data for a 300MB dataset, ends up with an 11GB cache. The initial .map() call takes 80s over the 15000 images, but then simply iterating over the dataset takes after that almost 5 minutes. It should be doing no processing here! Something seems wrong.

@eufrizz How you handle this using set_transform?
I have a really big dataset of size 1.2TB and i am going to use it for fine-tunning whisper model. if i use map for dataset_preparing function it will take over 20 days!!!

@eufrizz How you handle this using set_transform?
I have a really big dataset of size 1.2TB and i am going to use it for fine-tunning whisper model. if i use map for dataset_preparing function it will take over 20 days!!!

Just give the preprocessing function you were using for map to set_transform. Just look at the set_transform documentation. If you're going to do lots of epochs you might be better off just saving the preprocessed data into a new dataset.


Change beetle mini-map color scheme to be more colorblind friendly

A color scheme like this would be a big improvement:

After looking at the map above on a different (worse) monitor, the green and yellow are too similar to each other IMO. I've tried making the green a little bit darker in Photoshop and this is now super clear on all monitors. Hopefully it looks decent aesthetically too!

And it passes the grayscale test with flying colors 👍

After some iteration, we have decided on the following colors:
#00B800 (Low Risk)
#FFEF0A (Moderate Risk)
#FB2F18 (High Risk)


Application of Test Categorization

EDC upstream issue #819
Feature #150
Description
PR 868 introduced use of Junit Tags to run Integration tests.
To further classify the different kinds of tests use proposed tags in upstream issue and add gradle tasks to run tests related to tags mentioned in upstream issue only.
(if needed Paul can help in order to understand which test belongs to which tag category)
Acceptance Criteria

[ ] Tests categorized using proposed tags.
[ ] Added gradle task related to proposed tags.
[ ] Update integration-test doc to describe use of tags to categorize different type of tests.

Tasks
Same as acceptance criteria.

closing per upstream status


Set default decimal number in UI to 2 from 4

Set default decimal number in UI to 2 from 4

Seems there's no way to set the default behavior and a util method was written to do this.


Magento2 data import doesn't take configuration from local.json

Current behavior
Quote from the docs:

Magento2 data import is now integrated into vue-storefront-api for simplicity. It's still managed by the mage2vuestorefront - added as a dependency to vue-storefront-api.

The problem is that you still need to customize the configuration file in mage2vuestorefront. Now located in: ~/vue-storefront-api/node_modules/mage2vuestorefront/src/config.js. The docs don't mention this.

Expected behavior
IMHO, the configuration for mage2vuestorefront should be located in the config folder of the vue-storefront-api
Steps to reproduce the issue

Install Vue storefront
Connect to a Magento 2 instance
Change elastic/host settings in vue-storefront-api/config/local.json. In my case i've changed the Elastic host.
Run yarn mage2vs import
Notice that the custom settings from step 3 aren't used. Instead the config in this location is used ~/vue-storefront-api/node_modules/mage2vuestorefront/src/config.js

Repository
https://github.com/DivanteLtd/mage2vuestorefront
Can you handle fixing this bug by yourself?

No, we're trying to finish a Vue Storefront project before the end of the year :)
Additional information
Link to relevant the Vue storefront documentation

@Tjitse-E it’s taking the config from Vue-storefront-api/config/local.json but not the full config - just the Magento2 api, the rest is passed to mage2vs via ENV variables set in here:
https://github.com/DivanteLtd/vue-storefront-api/blob/258cc2a2f2b0f531f402737a6032e1118658de51/scripts/mage2vs.js#L107
Please let me know about Your use case, which properties are missed in there?
I think we should do two things:

let the user to set the properties via command line parameters of “npm run mage2vs import” - either using Vue-storefront-api ENV variables (commandline context) - which is also fine.
load the configuration from config/local.json
check mage2vuestorefront if some of the options are not possible to override via ENV (afaik all of them are open for ENV variables:
https://github.com/DivanteLtd/vue-storefront-api/blob/258cc2a2f2b0f531f402737a6032e1118658de51/scripts/mage2vs.js#L107)

The conclusion: the communication between Vue-storefront-api + mage2vuestorefront is in this mode done via ENV variables, let’s make sure user can modify them via Vue-storefront-api config file + commandline arguments (for on-demand calls)

@patzick can You please take care of this one as well?

@pkarw  in my use case I had to modify the elastic host directly here https://github.com/DivanteLtd/mage2vuestorefront/blob/master/src/config.js#L30, but the same setting is also configured in the API local.json: https://github.com/DivanteLtd/vue-storefront-api/blob/master/config/default.json#L11

You're right, DATABSE_URL is currently missed in the mage2vs. Thanks! @patzick could You please fix this ?


test: explicitly encode arguments for Python 3

The argument handling for subprocess.check_output differs between
Python 2.7, [3.0, 3.6], [3.7).
With Python 2.7, the list is implicitly decoded and we cannot use the
encode method as it will attempt to decode it as ASCII which will fail
with unicode data.
Python [3.0, 3.6] will attempt to encode the arguments before executing,
but will do so based on the system locale.  In the case that the locale
is POSIX, the encoding will be attempted in ASCII, which again fails
when unicode data is used.
Python [3.7) will accept the encoded strings and pass them through.  It
will implicitly encode the arguments using the file system encoding
(normally UTF-8), which will allow it to actually work with the content
encoded or decoded.
In order to provide maximal compatibility, pre-encode the arguments as
UTF-8 when using python 3.  Rely on the interpreter encoding the
argument implicitly on Python 2.  This allows running the
incrParse.simple test with python 2.7, python 3.6, python 3.7, python
3.8 with LC_ALL=POSIX as is the default on Ubuntu Server.
Thanks to @tbkka for the details on how to reproduce this issue and the
help with the python conversion!

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

CC: @drexin @tbkka @tachoknight

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows platform

Build failed
Swift Test OS X Platform
Git Sha - 162c5e1bc72f2a313cf7d5e5b16e6a79205c2093

@swift-ci please test

@swift-ci please test

@swift-ci please test Windows platform


Export to Custom Vision doesn't work if you already have a maximum number of projects

Hi!
At Paris ML OpenHack I tried to demo how you can build custom object detection service using Custom Vision and VOTT. I found that if you have maximum number of projects in your customvision.ai account, VOTT just doesn't export images with no user-friendly errors as it can't create a new project to export images.
It would be great to add at least basic user-friendly error handling for export for Custom Vision Service.
Thanks,
Stas

Resolved in v2.0.0


Add fstrim conf

Add fstrim conf
This pull request aims to include an fstrim configuration to util-linux and install it in pkg_postinst. By doing so, the default configuration shipped with upstream will be overwritten. This change is necessary to prevent the failure of fstrim.service on Flatcar Linux due to the absence of /etc/fstab.

By doing so, the default configuration shipped with upstream will be overwritten.

I don't see an upstream configuration that is being overwritten.

This change is necessary to prevent the failure of fstrim.service on Flatcar Linux due to the absence of /etc/fstab.

Jun 02 06:58:08 localhost systemd[1]: Finished fstrim.service.
core@localhost ~ $ ls -la /etc/fstab
ls: cannot access '/etc/fstab': No such file or directory

Can you follow the PR template that appears when you open a PR:

what is the error currently (show output)
how do you validate that it works after this PR
the commit message should have more detail (at least a couple of sentences), and start with the package name being touched.

My bad @jepio . It appears that this was fixed in this commit https://github.com/flatcar/scripts/commit/faf115acdec955cc31d528ada266d9053c13bc71 and in flatcar release stable-3510.2.2.


Updated Readme.md

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Docs Update
Why was this change needed? (You can also link to an open issue here)
Hello Sir/ma'am,
I have updated readme.md file
Other information:
I have made some changes in the readme file
I have updated some alignments
Made some Bold font to some alphabets etc.

Hey @dartpain, could you take a look at the changes and merge the PR if everything looks good? Thanks!

Hey @dartpain, could you take a look at the changes and merge the PR if everything looks good? Thanks!


Typo in explorer url

https://github.com/shardeum/network-status/blob/a62aeb23014bdbf5bb327134d1d2fe3a4ef77db9/backend/endpoints.json#L46

Thank you @chrypnotoad this is resolved now


Feature Request: Support auto discovery of IKEA models with merged naming scheme

Hi!
I am using Zigbee2MQTT, and there, some IKEA Tradfri models have naming scheme where two model names are merged into one string. Currently, Powercalc is unable to recognize any of those model names, as it detect the model name as being the string inside the parenthesis.
Example:
                "manufacturer": "IKEA",
                "model": "TRADFRI LED bulb GU10 400 lumen, dimmable, white spectrum (LED1537R6/LED1739R5)",
                "name": "guest_room_desktop_lamp",
                "sw_version": "2.0.029",
                "entry_type": null,
                "id": "4fb205e9ec7bbf33ec9701b55965640b",
                "via_device_id": null,
                "area_id": null,
                "name_by_user": null,
                "disabled_by": null,
                "configuration_url": null

This bulb is being recognized as having model name LED1537R6/LED1739R5 in Powercalc. As a result of this, they are currently not recognized in the model library, and won't work with autodiscovery.

This is a special (and rare) case - LED1537R6 and LED1739R5 are in reality different models, and they have different characteristics.
It can be seen even in in model numbers itself - it is in format:
LEDyynntpp
yy - design year
mm - model number (in that year, maybe week based)
t - bulb type (globe, PAR. etc.)
pp - max power draw (rounded to 0 decimals)
Koenkk's database
https://www.zigbee2mqtt.io/supported-devices/
has some errors inside (it is just build by users), so best way is correcting this database.

I agree with @nepozs.
These lights are quite different in power consumption. Should be fixed in the Zigbee2Mqtt database.
Not something which can be fixed in powercalc.
No need to modify const.py, and it's really bad practice as you will have problems upgrading powercalc.
You can just enter the correct manufacturer/model manually when it cannot be autodiscovered:
sensor:
  - platform: powercalc
    entity_id: light.livingroom_floorlamp
    manufacturer: ikea
    model: LED1739R5


Cannot undo object move and sort

Cannot undo object move and sort
Description

Cannot undo object move and sort
Add object move and object attach action

Platform

Web Version [X]
Windows [X]
Linux [X]

Implemented!


VennDiagram version conflict in conda environment

Currently, conda-forge::r-venndiagram=1.6.17 is not compatible with conda-forge::r-gridextra=2.3.
I am updating the feedstock to conda-forge::r-venndiagram=1.6.18  in conda-forge/r-venndiagram-feedstock#2. This issue can be closed once that is tested and merged.

the feedstock was merged successfully, but it appears the error goes deeper.
The environment builds successfully, but running Rscript scripts/viz/variant_overlaps.R results in the following error:
Error in dyn.load(file, DLLpath = DLLpath, ...) :
  unable to load shared object '/home/gway/anaconda3/envs/pdx-exomeseq/lib/R/library/stringi/libs/stringi.so':
  libicui18n.so.54: cannot open shared object file: No such file or directory

Based on this error, I added - anaconda::icu=54 to the environment and rebuilt. I then received the following error:
UnsatisfiableError: The following specifications were found to be in conflict:
  - icu 54*
  - plotnine 0.3.0* -> matplotlib >=2.1.0 -> icu >=58.2,<59.0a0

It also appears there is the same hidden conflict between VennDiagram and ggplot2

#41 replaces the r-venndiagram dependency with matplotlib-venn which removes this error


Prisma has stopped inferring/creating types after added this library

Prerequisites

[X] I have written a descriptive issue title
[X] I have searched existing issues to ensure the bug has not already been reported

Versions
"prisma": "^5.20.0",
"prisma-json-types-generator": "^3.1.1",
"@trpc/client": "^11.0.0-rc.485",
"@trpc/next": "^11.0.0-rc.485",
"@trpc/react-query": "^11.0.0-rc.485",
"@trpc/server": "^11.0.0-rc.485",

A minimal reproducible example
Don't have it
Description
Before using this libray:

After using this library:

I will try to create a reproducible demo as soon as possible. I decided to post it without one in case you have any idea about what could be happening. I've seen this issue https://github.com/arthurfiorette/prisma-json-types-generator/issues/302 which also has a conflict. Maybe is related?
Steps to Reproduce
schema.prisma
model Company {
  id       Int     @id @default(autoincrement())
  user     User    @relation(fields: [userId], references: [clerkId])
  userId   String  @unique
  name     String?
  imageUrl String?
  salary   Price[]

  /// [WorkDayHoursType]
  workDayHours Json?
}

prisma.d.ts
declare global {
  namespace PrismaJson {
    type WorkDayHoursType = {
      mon: number;
      tue: number;
      wed: number;
      thu: number;
      fri: number;
      sat: number;
      sun: number;
    };
  }
}

npx prisma generate. Just it.
Expected Behavior
Keep Prisma types.

Hi! thanks for creating this issue. Would you like to open a PR? Remember to add unit tests :)
Without a reproduction example I can't help you...


[BUG]Lamadava Service: Too Many Redirects.

When I use the Lamadava Service (v1/user/by/id) today, it does not work with the error: Too many redirects.
And when I try the api online at https://api.lamadava.com/docs#/, it shows the same info:

I've tried to change the api.lamadava.com to api.instagrapi.com but it still does not work.
Could U help me with it？ Thank U!

The service works well in last 7 days. And I believe I have used for around 60,000 requests which is less than 100, 000 requests/month.

Besides, when I try the api v1/user/by/username, there's another problem:

Maybe this can help you to fix the bug~

It works now. It seems like that waiting for some hours can handle this problem.
Anyway, thanks for your service~

Sorry for bothering you, but My service dose not work now. I check the server for the script and I found nearly every request shows that: Too Many Redirects.
And several of them shows that Instagram Server Error.
I try to use different procy and online 'try it out', it still does not work. Could u help me? Thanks!

Sorry for bothering you, but My service dose not work now. I check the server for the script and I found nearly every request shows that: Too Many Redirects.
And several of them shows that Instagram Server Error.
I try to use different procy and online 'try it out', it still does not work. Could u help me? Thanks!

Now I receive a new error info: No free accounts. I want to know that this is some problem of my own Lamadava account or it is caused by your service TAT. If it is caused by your service, I do deeply hope that it can be fixed soon.

@adw0rd Druing last 3 days, I find that the problem usually happened between  00:00 (UTC) to 06:00 (UTC).
Although there's enough time for me to get data in a day. But it still cause some problem and make meaningless bill cost for Lamadava.
I'll appreciate it that you'll reply soon. Thanks.

@SoonWhy this occurs when instagram massively bans accounts, and at this time I'm sleeping and I can't react. I will try to fix this problem this month


Add ability to store VPC flow logs on S3

I think it would be grate to have ability to store VPC flow logs on S3 out of the box.

This feature is already supported. See examples in examples/vpc-flow-logs/s3.tf:
https://github.com/terraform-aws-modules/terraform-aws-vpc/blob/d9e7a6f871a02b13b5b89a8e126ea4092cd5b006/examples/vpc-flow-logs/s3.tf#L4-L21

@antonbabenko , I planed to implement this the same way as VPC Flow logs to CloudWatch and create RP.
Do you think this would be redundant functionality?

I think VPC flow logs support in this module is already complete - it supports both saving to Cloudwatch Logs and S3 bucket (see examples).
What else do you think is missing?

Ok, I got your point. Sorry for disturbance!
Have a nice day!


Allow node name to be auto-generated when added

[x] New feature (non-breaking change which adds functionality)

Closes #3418
Proposed changes
The original proposal has been replaced by an action based approach
Adds a new action:

core:generate-node-names

When invoked, will apply a default name to the selected nodes.

It only applies to nodes that do not currently have a name
It uses a name of the form <paletteLabel> <N> where N is the next available integer that avoids a name clash. This is scoped to the whole editor - not just individual flows.

It can also be invoked on an individual node to allow it to be used in a node's onadd function to generate a name when being added to the workspace.
This PR adds onadd functions to the debug and link in/out/call nodes - as they all most benefit from having more human readable, distinct, default names.

Coverage increased (+0.03%) to 67.355% when pulling fd30b7101c62c33c447d5f19417ce236259acc73 on auto-gen-name into 6a5c50ff774f174f1788e12ad9fd17557f1eaa81 on dev.

Hi nick. This might be feature creep but I think 2 more features would make this even sweeter...

An action to run the auto making against selected nodes (would make quick work of updating an existing flow)

Perhaps a linting corrective action could also call this feature?

Make the auto naming work with split wires to links.

@Steve-Mcl you're quite right an action is the way to go.
Having thought about it some more, I'm inclined to completely change the approach and remove the autoGenerate property from the defaults object - and go back to this being something a node can do via an onadd function by invoking the new action.

The consequence of moving to an Action approach is we no longer allows individual nodes to customise the behaviour:

we use <paletteLabel> <N> as the name format. It cannot be translated unless a node provides its own paletteLabel function to return a translation
the 'scope' of the number is fixed a single behaviour. It can either be scoped to the flow, or globally - we can't have some nodes doing one thing and others another.

In the live-stream where I put this together, a lot of people wanted 'flow' scoped number for the Debug nodes. That's a reasonable choice for that node type, but it wouldn't work (I don't believe) for Link nodes - which are more likely to want to reference each other across flows.
On balance, I think going with globally scoped numbering is the best option.

I have replaced the implementation with an action based approach and updated the PR description to match

Nick, I've been running DEV with this change merged. I'm liking it :+1
That said, there is one frustruation.

I drop a new debug it gets default "debug 1" name.
I setup that node to my liking
I then copy/paste it (as it has all the settings I want) but the name remains "debug 1"!

Would you consider applying the auto name generation if a nodes name meets the pattern '^'+paletteLabel+' (\\d+)$' ?
Alternatively, we could set an autogenName property to same value as the name property & if name == null || autogenName === name then run the auto naming?

That said, the import function does take some flags to customise its behaviour - such as whether to generate new Ids or not. So it would be feasible to add a flag to cause naming updates for paste actions only

I think the experience would be greatly improved if we did Nick.
Are you ok with me raising an issue around this so its tracked?

Does this also force new names into "old" nodes that have no name ? While I may want to name debug nodes I also like being able to see that it is set to msg.payload or msg.foobar  or just msg etc

Does this also force new names into "old" nodes that have no name

No Dave, only newly added nodes & imported/pasted nodes without a name.
It does mean however to see msg or msg.xxxx as the name, you would need to open the editor to clear the auto name - BUT - you would need to open the editor anyway (to set the debug node to something other than msg.payload).

are you ok with me raising an issue around this so its tracked?

Yes please.

Does this also force new names into "old" nodes that have no name
No Dave, only newly added nodes & imported/pasted nodes without a name.

The 'rename on paste' should only happen if the existing name matches the auto-generated name pattern. It should not modify nodes with no name.

With the current version in dev - If I have an existing node with no name - (showing msg.payload as label) - and copy/paste it to create another - it becomes "debug 1" - If I then copy paste that is creates another "debug 1". I would rather those actions were swapped... if blank - leave blank... if debug 1 then become debug 2... etc

and copy/paste it to create another - it becomes "debug 1"

I may have lost track, but I don't think that is intended behaviour.
I agree the expected behaviour should be to not change names unless they match the autogenerated name pattern.

This is existing dev branch... not sure if @Steve-Mcl  is still working on this branch.

Nothing has been changed in dev since this PR - so that is a bug with this code that should be resolved as part of the improving copy and paste as per the discussion above.


fix(presenter): fix memory leak

🍰 Pullrequest

fix memory leak
This PR fixes a memory leak caused by sentry in frontend and presenter.
We observed a memory leak on stage1 where the presenter grew in size till the server exploded.
According to this issue a likely reason is the sentry implementation done lately. The issue states that sentry should never run serverside when utilizing SSR.
The branch is currently checkout out on stage1 to observe
Issues

relates https://github.com/getsentry/sentry-javascript/issues/9982
relates https://github.com/getsentry/sentry-docs/issues/11705

Todo

[ ] Check if the solution fixes our memory problems on stage1

It works

(image taking after several hours, constant uptime check on the presenter, no more ram accumulation - would be several hundred megabyte at least)


powerstat: 0.02.15 -> 0.02.16

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/powerstat/versions.
These checks were done:

built on NixOS
/nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16/bin/powerstat passed the binary check.
1 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 0.02.16 with grep in /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16
directory tree listing: https://gist.github.com/07ed4981848777e18cca2c072d0fb30c
du listing: https://gist.github.com/5766491662635d4a960a7cd3a4911db0

meta.description for powerstat is: '"Laptop power measuring tool"'.
cc @womfoo for testing.

Success on aarch64-linux (full log)
Attempted: powerstat
Partial log (click to expand)
cp powerstat.8.gz /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16/usr/share/man/man8
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16
shrinking /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16/bin/powerstat
gzipping man pages under /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16/share/man/
strip is /nix/store/jk6j4lh9v5mvjdbdc35sj0zffhhf6s56-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16/bin
patching script interpreter paths in /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16
checking for references to /build in /nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16...
/nix/store/l0w677jji9745f7hl0xrlqiaw2d0z6d1-powerstat-0.02.16

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: powerstat
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: powerstat
Partial log (click to expand)
cp powerstat.8.gz /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16/usr/share/man/man8
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16
shrinking /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16/bin/powerstat
gzipping man pages under /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16/share/man/
strip is /nix/store/j75dgadrff2d1fyc4fczmcgqkid2imdx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16/bin
patching script interpreter paths in /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16
checking for references to /build in /nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16...
/nix/store/r8r2fb581gxng6pkwyvbssyl3w21f2pg-powerstat-0.02.16


Add support for non-CLI usage

Add nonzero grouping
Move enum to lib to enable non-CLI usage

Merged in #6


graph: use src edges in dst graph

Please take a look.
Fixes #824.
Depends #867.

This will need to be re-approved after I rebase onto master to get it passing.

Thanks


Meaning of coordinates

Hi,
Thanks for sharing the code.
For the ranking dataset, what does the four numbers mean?
[X1, Y1, X2, Y2] or [X, Y, W, H] or anything else?

It means [x y w h] of a crop.

I found some images of size 2048*1536 has crop coordinates
130 | 92 | 640 | 480, which is too smal for the image size.Is there a specified scaling relationship to the image size?

@zyq950309
No, there is no scaling needed. Did you try the cropping_browser.py script to visualize it and see if that makes sense to you?


In Search of a Clean Project - Multiple errors in \android\app\src\main\AndroidManifest.xml

Android Studio shows errors in AndroidManifest.xml. These errors are not material and do not cause a problem building the Android app version of the Flutter project. But 'Problems' are shown on the Project.
I think this has been a problem with Flutter projects in Android Studio since BEFORE Flutter v1. Untidy errors like this create confusion and undermine confidence in Flutter. Given we are at Flutter v2.5.x and we want to improve Flutter maturity, we should seek to remove such 'bogus' errors reported in the Google supported IDE.
This may be an Android tool chain issue when part of a Flutter application, but I would have thought we could have resolved these by now. It snot a good look for Devs new to Flutter.
Steps to reproduce

Android Studio / File / New / New Flutter Project
Open AndroidManifest.xml located in /android/app/src/main

These errors shown:

Previous Issues Logged but Closed
This issue or very very similar ones keep getting raised and closed. Note that every single Flutter ap I have ever created still has these errors. There are more but here is a sample of previous issues:
June 19th 2021: https://github.com/flutter/flutter/issues/84866
May 10th 2017: https://github.com/flutter/flutter/issues/9944
March 2018: https://github.com/flutter/flutter/issues/15722
Flutter Doctor:
[✓] Flutter (Channel stable, 2.5.2, on macOS 11.6 20G165 darwin-x64, locale en-AU)
• Flutter version 2.5.2 at /Users/gamma/Documents/flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 3595343e20 (11 days ago), 2021-09-30 12:58:18 -0700
• Engine revision 6ac856380f
• Dart version 2.14.3
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
• Android SDK at /Users/gamma/Library/Android/sdk
• Platform android-S, build-tools 30.0.2
• ANDROID_HOME = /Users/gamma/Library/Android/sdk
• Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
• Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
• 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.11.2
[✓] Chrome - develop for the web
• Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome
[✓] Android Studio (version 2020.3)
• Android Studio at /Applications/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.10+0-b96-7281165)
[✓] VS Code (version 1.61.0)
• VS Code at /Applications/Visual Studio Code.app/Contents
• Flutter extension version 3.27.0
[✓] Connected device (2 available)
• macOS (desktop) • macos  • darwin-x64     • macOS 11.6 20G165 darwin-x64
• Chrome (web)    • chrome • web-javascript • Google Chrome 94.0.4606.81
! Error: Mike's iPhone is not connected. Xcode will continue when Mike's iPhone is connected. (code -13)
• No issues found!

@workerbee22
I believe this issue is more related to the flutter plugin for Android studio, rather than a framework bug. I suggest you to open this issue in plugin's dedicated repo, although I already see a similar issue open there: https://github.com/flutter/flutter-intellij/issues/2243
Closing from here.

@darshankawar I'm curious how invalid AndroidManifest.xml content is an Android Studio issue?

@darshankawar That issue that you linked  and titled Improving Flutter Support in Android Studio https://github.com/flutter/flutter/issues/87057 does have a couple of related issues that are very much the same as this one.

Hey @workerbee22,
The issues in AndroidManifest.xml don't get flagged in every IDE, it isn't visible as an issue in VSCode (which I use). I think many Flutter developers ignore the Runners, as they are just an implementation detail.
The way that issues like this get fixed is we get votes on the issues from the community. The Flutter engineering team uses vote counts as a way of understanding the focus of the developer community.

Good point ... I'm on it 👍

I doubt that this is actually flutter fault, most likely its just some stupid import issue in android studio (again). If you open android folder directly in Android studio, there will be no error present.
But I upvoted it;) However we should link this into android studio team somehow

Yep looking like it and thus it should be straight forward to fix. Thx for upvoting.

@darshankawar and @domesticmouse Have logged new issue in the Flutter-IntelliJ repo issues here: https://github.com/flutter/flutter-intellij/issues/5817

@darshankawar I'm curious how invalid AndroidManifest.xml content is an Android Studio issue?

@domesticmouse Looking at the issue and similar issues logged were directed to flutter intellij plugin citing the same. Also, it's mentioned in the flutter intellij issue that, to avoid seeing such errors, the best way is to open Android project in AS. Hence I informed OP about same and directed to dedicated repo and also since it's related to IDE's being used.  Apologies if it isn't the case.

yep I think so


documentHighlights-full could not find file

documentHighlights-full
Error processing request. Could not find file: '%PATH%'.
Error: Could not find file: '%PATH%'.
    at getValidSourceFile (C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:117781:23)
    at Array.map (<anonymous>)
    at Object.getDocumentHighlights (C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:118100:53)
    at IOSession.Session.getDocumentHighlights (C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:125811:71)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:125254:61)
    at C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:126635:88
    at IOSession.Session.executeWithRequestId (C:\Program Files\Microsoft SDKs\TypeScript\3.3\tsserver.js:126626:28)
    at IOSession.Session.executeCommand (C:\Progra…

~2600 hits in 24 hours
3.3.0-dev
Individual files have hundreds of hits

Fixed by #32791


Update Delta.io Banner

The delta.io website banner is linking to the Delta 2.1.0 release. This should be updated with the most recent release, Delta 2.1.1 - https://github.com/delta-io/delta/releases/tag/v2.1.1

@dennyglee @MrPowers

Thanks @cakerly-lf - will work on this!

This has been resolved via #174 - thanks!