[adaptive_scaffold] drawer stays open after selection with no way to close it

Steps to reproduce

create a component using AdaptiveScaffold, leave useDrawer property unset (or set to true)
run on desktop or web (as drawer is not used in mobile)
make window smaller than smallBreakpoint so drawer is shown.
use drawer to click any destination

Expected results
Drawer should close after clicking on a destination.
Actual results
Drawer does not close after clicking on a destination.
Although that seems the right approach for AdaptiveScaffold, it could also provide a closeDrawer method like Scaffold. Sadly, even though AdaptiveScaffold uses a Scaffold internally, using Scaffold.of(context).closeDrawer() does not work when used on onSelectedIndexChange as the Scaffold context is a child of AdaptiveScaffold and so not found on the current context inside the handler.
Code sample
Code sample
import 'package:flutter/material.dart';
import 'package:flutter_adaptive_scaffold/flutter_adaptive_scaffold.dart';

const Color darkBlue = Color.fromARGB(255, 18, 32, 47);

void main() {
  runApp(App());
}

class App extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      home: HomePage(),
    );
  }
}

class HomePage extends StatefulWidget {
  @override
  State<StatefulWidget> createState() => HomePageState();
}

class HomePageState extends State<HomePage> {

  int selectedDestination = 0;
  static const List<String> destinationLabels = <String>['Home', 'Settings'];

  @override
  Widget build(BuildContext context) {
    return AdaptiveScaffold(
      selectedIndex: selectedDestination,
      destinations: <NavigationDestination>[
        NavigationDestination(
          icon: const Icon(Icons.home),
          label: destinationLabels[0],
        ),
        NavigationDestination(
          icon: const Icon(Icons.settings),
          label: destinationLabels[1],
        ),
      ],
      body: (_) => Center(
        child: Text(destinationLabels[selectedDestination]),
      ),
      onSelectedIndexChange: (newDestination) {
        setState(() {
          selectedDestination = newDestination;
          // The following line does not work as parent context does not contain a scaffold
          // Scaffold.of(context).closeDrawer();
        });
      }
    );
  }
}

The example can be copied to DartPad to reproduce the bug.
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.16.7, on macOS 14.2.1 23C71 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.2)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.1)
[✓] IntelliJ IDEA Ultimate Edition (version 2023.3.2)
[✓] Connected device (2 available)
[✓] Network resources

• No issues found!

Thanks for filing this @rivasdiaz.
I can reproduce this on the latest master and stable channels but this seems to me to be WAI.
I'll be labelling for more insights.
stable, master flutter doctor -v

[!] Flutter (Channel stable, 3.16.8, on macOS 14.2.1 23C71 darwin-arm64, locale
    en-NG)
    • Flutter version 3.16.8 on channel stable at
      /Users/damilolaalimi/sdks/flutter
    ! Warning: `dart` on your path resolves to
      /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your
      current Flutter SDK checkout at /Users/damilolaalimi/sdks/flutter.
      Consider adding /Users/damilolaalimi/sdks/flutter/bin to the front of your
      path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 67457e669f (5 days ago), 2024-01-16 16:22:29 -0800
    • Engine revision 6e2ea58a5c
    • Dart version 3.2.5
    • DevTools version 2.28.5
    • If those were intentional, you can disregard the above warnings; however
      it is recommended to use "git" directly to perform update checks and
      upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android
      Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build
      17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.3

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

[✓] Android Studio (version 2022.2)
    • 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
      17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio
      Preview.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
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build
      17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        •
      android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios
      • iOS 17.1.1 21B91
    • iPhone 15 Pro (mobile)      • 08CB1E0F-4299-4651-AF30-498EE2040E78 • ios
      • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                •
      darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                • chrome                               •
      web-javascript • Google Chrome 120.0.6099.234

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

[!] Flutter (Channel master, 3.19.0-9.0.pre.50, on macOS 14.2.1 23C71 darwin-arm64, locale en-NG)
    • Flutter version 3.19.0-9.0.pre.50 on channel master at /Users/damilolaalimi/fvm/versions/master
    ! Warning: `dart` on your path resolves to /opt/homebrew/Cellar/dart/3.1.5/libexec/bin/dart, which is not inside your current Flutter SDK checkout at /Users/damilolaalimi/fvm/versions/master. Consider adding /Users/damilolaalimi/fvm/versions/master/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5dea6b994e (16 hours ago), 2024-01-21 12:01:23 -0500
    • Engine revision a7b207d5a1
    • Dart version 3.4.0 (build 3.4.0-52.0.dev)
    • DevTools version 2.31.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/damilolaalimi/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • ANDROID_HOME = /Users/damilolaalimi/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b802.4-9586694)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C65
    • CocoaPods version 1.14.3

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

[✓] Android Studio (version 2022.2)
    • 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 17.0.6+0-17.0.6b802.4-9586694)

[!] Android Studio (version unknown)
    • Android Studio at /Users/damilolaalimi/Downloads/Android Studio Preview.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
    ✗ Unable to determine Android Studio version.
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)

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

[✓] VS Code (version 1.83.1)
    • VS Code at /Users/damilolaalimi/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.78.0

[✓] Connected device (5 available)
    • sdk gphone64 arm64 (mobile) • emulator-5554                        • android-arm64  • Android 14 (API 34) (emulator)
    • Damilola’s iPhone (mobile)  • 00008110-001964480AE1801E            • ios            • iOS 17.1.1 21B91
    • iPhone 15 Pro (mobile)      • 08CB1E0F-4299-4651-AF30-498EE2040E78 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)             • macos                                • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)                • chrome                               • web-javascript • Google Chrome 120.0.6099.234

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 2 categories.

Hi @dam-ease , thanks for your quick response.
Can I ask to please reconsider this issue as a bug instead of intended behavior? According to the material specification (both v2 and v3), modal navigation drawers can be dismissed by selecting an item (https://m3.material.io/components/navigation-drawer/guidelines#:~:text=Selecting a drawer item). This is not possible when using this component.
If not, thanks for leaving it as a proposal.

I agree - We can have a bug fix to dismiss drawer in case of already open for that specific Scaffold. Apparently, I will try to add some fix into this one & lets see how that goes.

Added potential fix for the mentioned issue - https://github.com/flutter/packages/pull/6289

Fix added, we can close the issue.

@rivasdiaz - Can we please mark this issue as fixed/closed as fix has been deployed & available to users.


[PT-BR] Translate Async Await and Immutability

Translation for:

Async Await
Immutability

@khaosdoctor Applied those suggestions

lgtm


Missing headers

Add Accept-Ranges: bytes to response headers
Add Content-Range to response headers

I believe this was fixed in #279


Larval precursors for adult testis cell types were added in #1520. Since there’s nothing to do for the “adult type (A|B) central brain neurons” of FBrf0251482, this can can closed.


fix: add integration test for preview

This PR

Add a new package “@sap-ux-private/playwright”
Add simple integration test for preview. It test preview with different UI5 Versions:

1.84.35
1.96.22
1.108.19
1.111.8
1.114.0
1.115.0

Add Github Actions. Integration tests are triggered if there is any changes in "@sap-ux/preview-middleware" package and playwright-reports are uploaded
Pave the ground for integration test for CPE on adaption project. Integration tests for CPE will follow in another PR.

closing this PR as CLA is never turning green. Changes are move to new PR https://github.com/SAP/open-ux-tools/pull/1507


fix(deps): pin dependencies

This PR contains the following updates:

Package
Type
Update
Change

@opentelemetry/api
dependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/api
devDependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/api-metrics
devDependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/core
devDependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/metrics
devDependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/node
dependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/resources
devDependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/tracing
dependencies
pin
^0.16.0 -> 0.16.0

@opentelemetry/tracing
devDependencies
pin
^0.16.0 -> 0.16.0

📌 Important: Renovate will wait until you have merged this Pin PR before creating any upgrade PRs for the affected packages. Add the preset :preserveSemverRanges to your config if you instead don't wish to pin dependencies.

Renovate configuration
📅 Schedule: At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you check the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: samples/trace/package-lock.json
bash: line 0: hash: npx: not found


 Why is this change necessary?
This change prevents irrecoverable container crashes that stem from not backwards(container)compatible changes in plugins from one version to another by clearing the container cache once the plugin zip is extracted.
2. What does this change do, exactly?
This change clears the container cache once the plugin zip is extracted, thus making sure that new files or XML changes brought with the new plugin are picked up from the container. It also removes surplus calls to plugin:refresh as this routine is called in all relevant workflows again anyways.
3. Describe each step to reproduce the issue or behaviour.

Take a subscriber from a plugin which is subscribed onto some DAL Event, for example entity:write
Add non-optional (and thus backwards incompatible) constructor parameters to both the class and the DI
The container will not be rebuilt naturally, as the XML DI change is not picked up by symfony
The container will not be cleared as a previous call to ::refreshPlugins fails because it triggers an entity:write which in turn requires the subscriber to be loaded, which in turn fails because the subscriber can only be loaded with the new XML.

As symfony did not pick up on any XML changes so far, it tries to rebuild the container from a higher level, ignoring the XML changes. We thus need to tell Symfony explicitly to rebuild the whole container, which is done here by clearing the container cache (which is just a serialized Version of the container).
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/NEXT-23826
Please use this issue to track the state of your pull request.

@maximilianruesch  i'm closing this for inactivity now, feel free to reopen when you get to work on it again

I will work on this when i get to it. When testing this i had issues running the cypress tests locally, but i will look into this again soon.

I have just taken another look at this using the trunk which is currently on 6.5. However, no first install modal popped up and the cypress test fails as i am immediately logged out again after finishing the installer.
Can you take another look at this? Is there a configuration i need to change in order to get a first install modal in SW6.5 (in the trunk version from three days ago, i am on 0aaed327)?

@maximilianruesch  I guess you forgot to delete the install.lock file in the root directory

I removed the install.lock and executed the test. After finishing the/installer (inside the test), the install.lock was created again as should be normal. The test then proceeds to log into the admin and test for the First-Run-Wizard Modal to appear. However, the test runner was immediately logged out again for some reason.

When resetting the state and manually proceeding through the install, i also do not have a first run wizard popup (and i am also logged out immediately and need to reenter my just created credentials). Next login no wizard pops up and thus i cannot test the install of the paypal plugin.

You can find the wizard in Settings > System > First Run Wizard. As e2e/cypress can sometimes be a bit finicky, I recommend you testing manually. If I remember correctly, it's not even necessary to reset the environment every time 😄

Thanks for this hint. I could test the installation of the paypal plugin, just not under the 6.5RC (as it is still marked incompatible here).
When testing this with 6.4, i finally got the error to show up. I will push a fix for this shortly and thus i kindly ask you to reopen this PR @cyl3x

For some reason i am having troubles with the CI. A (seemingly unrelated i would guess) unit test for the BundleConfigGenerator fails under 8.2. When testing locally with 8.2 on this branch, it does not...
I will proceed to test if my changes had any impact on this by removing them from this PR temporarily. Do you have any idea what could be the cause of the discrepancy between the CI and local results?

No worries, at this point I will take over :smile:

As a little status update, the PR is in internal review, but due to the upcoming 6.5 release, this will take a little longer than usual :smile:

It would be great to have this in the initial 6.5 Release as we can then always rely on this being fixed with the new major.

Is there a status update if this PR is contained in the initial 6.5 release? As mentioned, it would be great to be able to rely on this after the major break.

I'm sorry, but initial release of 6.5 only receives updates, so sadly this PR will be in the next 6.5 release

Thanks again for your contribution, your PR was merged! :tada:

@maximilianruesch I have really good news for you! We decided to include this PR in the initial release of 6.5 after all :tada:


Add reference to authenticate Python SDK with Dex

Added reference to authenticate Python SDK with static Dex users

@googlebot I signed it!

/lgtm
/apprvoe

/approve


Fix a few github file references

https://github.com/openssl/openssl/blob/master/.github/SUPPORT.md ->
https://github.com/openssl/openssl/blob/master/SUPPORT.md
Fixes #13396

As a side note, I just re-read the SUPPORT.md file. I find the recommendation to disable asm in that file disturbing. This is not something we should be encouraging users to do without suitable caveats. The asm implementations often have security properties (constant time) that the C implementations may not have (not to mention of course the performance benefits). Building with no-asm might be a useful thing to do to rule in or out asm issues - but using it in production should be treated with caution.

Hmmm, I don't see anything suggesting that really suggests this for production.  What I do see, though, is that SUPPORT.md repeats the same thing twice, where the second one has section titles explaining the intent a bit better.
I'll take a dive at editing it to be, er, better.  Another PR.

Hmmm, I don't see anything suggesting that really suggests this for production. What I do see, though, is that SUPPORT.md repeats the same thing twice, where the second one has section titles explaining the intent a bit better.

"If you have problems with OpenSSL....configure with no-asm".
I think anyone reading that could reasonably interpret it as something you might consider for production. My point above was that there aren't sufficient caveats.

How about looking #13398 and see if that's a better read?

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged
322d56cd645fa2372aaddc78ea5db9c71f58d28f Fix a few github file references


Error importing rediscluster

Used pip3 to install redis-py-cluster
pip3 install redis-py-cluster
from rediscluster import RedisCluster.
Giving the following error
File "/Users/pallab.nag/india/redis.py", line 1, in <module>     from rediscluster import RedisCluster ImportError: cannot import name 'RedisCluster' from partially initialized module 'rediscluster' (most likely due to a circular import) (/Library/Frameworks/Python.framework/Versions/3.9/lib/python3.9/site-packages/rediscluster/__init__.py)

@xtreme03 The error message says itself what the issue is, you have a circular import where rediscluster package is installed twice or you have a separate file/module named rediscluster.py somewhere where you should not have. Print out all paths in sys.path that is used for imports and see where you have the two versions of rediscluster.py or that as a module and remove one of them and your code will start to work.

Got same error in python 3.9.7.


Refactor: memory-efficient geo file decoder

This PR implements a simple decoder based on LEB128 used in protobuf. It reads geo files byte by byte to reduce peak memory usage when V2Ray parses configuration.
If the decoder fails to decode the geo file, it will fallback to the original ReadFile method to keep backward compatibility.
About protobuf encoding and decoding, read https://developers.google.com/protocol-buffers/docs/encoding

As a side effect, this impl causes v2ray-core to have full cpu usage, hang and even become unresponsive for a few seconds at startup (test with default geo files). Think twice?

This change my subject to revert, please keep your branch.


Changed assignment of "throws" property

...from dot notation to square bracket notation.
In few environments, like Closure Compiler, throws is considered a reserved word and will cause an error.
Other implementations of assert already take this into account, e.g. https://github.com/Gozala/narwhal/blob/master/lib/assert.js#L282

Valid in this context. Reserved word only if used as such.


Table API: factor out the table connector from the table format

Problem description
Related to #181 , the 'unified' connection API for table sources/sinks allows the format of the data to be decoupled from the connector.   In other words, we don't need FlinkPravegaJsonTableSource (a subclass of FlinkPravegaTableSource), we need to create a subclass of org.apache.flink.table.descriptors.ConnectorDescriptor, and users will be able to use a standard table format (json, avro, etc.).
Here's an example of a table factory that uses TableFactoryService to find the configured DeserializationSchema:
https://github.com/apache/flink/blob/master/flink-connectors/flink-connector-kafka-base/src/main/java/org/apache/flink/streaming/connectors/kafka/KafkaTableSourceSinkFactoryBase.java
See: https://ci.apache.org/projects/flink/flink-docs-master/dev/table/connect.html
Problem location
Suggestions for an improvement

closed https://github.com/pravega/flink-connectors/pull/205


missing dcterms:replaces property for dwc:collectionID-2009-09-11

In the normative (Type 1) Darwin Core document (dwctermshistory.rdf), the term version http://rs.tdwg.org/dwc/terms/collectionID-2009-04-24 (dwc:collectionID-2009-04-24) is flagged as superseded.  However, there is no other term version that is linked to it through a dcterms:replaces property.  I think this was caused by a missing dcterms:replaces property of http://rs.tdwg.org/dwc/terms/collectionID-2009-09-11 (dwc:collectionID-2009-09-11).  The missing triple is probably:
dwc:collectionID-2009-09-11 dcterms:replaces dwc:collectionID-2009-04-24.

Amended in commit 2b3edebaa486491bb3a5c277e01cf0c6034ef403.


Flamingo FA500 support

Home Assistant release (hass --version):
I'm using HASS version 0.35.3
Python release (python3 --version):
Python 3.5.2
Component/platform:
Docker container on Synology DSM 6
Description of problem:
Since today i own the RFXcom rfxtrx
I managed to add my Klik aan Klik uit switches.
Now i'm trying to add my Flamingo FA500 switches.
When i add the following lines to the config:
switch:
  platform: rfxtrx
  automatic_add: True

and press on on the remote i get 2 codes:

Here the log entries:
16-12-28 17:04:06 INFO (Thread-13) [homeassistant.components.rfxtrx] Receive RFXCOM event from (Device_id: 001f2042 Class: LightingDevice Sub: 0)
16-12-28 17:04:06 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=switch.0b1100000001f20402000070, old_state=<state switch.0b1100000001f20402000070=off; friendly_name=0b1100000001f20402000070, assumed_state=True @ 2016-12-28T18:04:04.932438+01:00>, new_state=<state switch.0b1100000001f20402000070=on; friendly_name=0b1100000001f20402000070, assumed_state=True @ 2016-12-28T18:04:06.329269+01:00>>
16-12-28 17:04:06 INFO (Thread-13) [homeassistant.components.rfxtrx] Receive RFXCOM event from (Device_id: a11 Class: LightingDevice Sub: 1)
16-12-28 17:04:06 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: entity_id=switch.07100101410b0070, old_state=<state switch.07100101410b0070=off; friendly_name=07100101410b0070, assumed_state=True @ 2016-12-28T18:04:05.287064+01:00>, new_state=<state switch.07100101410b0070=on; friendly_name=07100101410b0070, assumed_state=True @ 2016-12-28T18:04:06.684900+01:00>>

16-12-28 17:08:48 INFO (Thread-13) [homeassistant.components.rfxtrx] Receive RFXCOM event from (Device_id: 9ddd6632 Class: RFXtrxDevice Sub: 10)
16-12-28 17:08:51 INFO (Thread-13) [homeassistant.components.rfxtrx] Receive RFXCOM event from (Device_id: 9c6d5032 Class: RFXtrxDevice Sub: 10)
16-12-28 17:08:52 INFO (Thread-13) [homeassistant.components.rfxtrx] Receive RFXCOM event from (Device_id: 001f2045 Class: LightingDevice Sub: 0)
16-12-28 17:08:52 INFO (Thread-13) [homeassistant.components.rfxtrx] Automatic add 001f2045 rfxtrx device (Class: LightingDevice Sub: 0 Packet_id: 0b11002b0001f20405010f70)

When i try to turn them on/off from hass it is not working.
I have looked around and with Domoticz it is working, here some infor from RFXcom
Switch fa500_remote_A { rfxcom="<2622466.1:Command" }
Switch fa500_remote_B { rfxcom="<2622466.2:Command" }
Switch fa500_remote_C { rfxcom="<2622466.3:Command" }
Switch fa500_remote_D { rfxcom="<2622466.4:Command" }

// Note: does not mimic the remote but uses another subtype and code which
// should be learned to the socket separately
Switch virtual_fa500s_switch_1 { rfxcom=">2061.1:LIGHTING5.IT:Command" }
Switch virtual_fa500s_switch_2 { rfxcom=">2061.2:LIGHTING5.IT:Command" }

Is it possible to add the IT functionality to hass? Thanks

You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum.
Someone have to add support for it to the library we are using: https://github.com/Danielhiversen/pyRFXtrx


Cannot access logs from LUCI runs

e.g. https://ci.chromium.org/ui/p/flutter/builders/prod/Mac_ios flutter_view_ios__start_up/5783/overview
I can't access https://storage.googleapis.com/flutter_logs/flutter/a04e6b7618700f7b1cadee23f82ef11e693ef44c/flutter_view_ios__start_up/988a44ed-d3f7-4762-983e-5ad9a03c4639/b4873f14ffb248dfcb00c8cb7fdd4cdcff87f1f0.log
/cc @godofredoc

Are you able to view: https://storage.cloud.google.com/flutter_logs/flutter/a04e6b7618700f7b1cadee23f82ef11e693ef44c/flutter_view_ios__start_up/988a44ed-d3f7-4762-983e-5ad9a03c4639/b4873f14ffb248dfcb00c8cb7fdd4cdcff87f1f0.log

The above link is for authenticated access.
Seems the bucket was switched to Not public, which makes it not accessible via the public endpoint: https://storage.googleapis.com

The authenticated link only works if I'm logged into my corp google account, which doesn't seem like the right ACL for this - we want people to be able to read these logs without working for Google.

(I also got the unauthenticated link from the LUCI build page, so if we really need people to log into view logs then we need to update luci - but again I'm hoping these can just be moved back to public and am assuming this was part of the bucket rename).

Yes, this was because when we moved the bucket we didn't move it to public. Let me fix that.

waiting on approval for an exemption to make the bucket public.

The changes to allow public access to the bucket haven't been approved yet.

latest update: Google folks managing this on the backend have asked for a potential improvement that might help with security.

The bucket is now public again. @dnfield can you please take a look to see if this now working as intended?

I am able to access the bucket, thanks!


Question about the size of Y in DARNN code written in Keras

I have a question regarding the DARNN code written in Keras.
In line 439 of model.py file (da_rnn/keras/model.py), Y is extracted from the inputs.
However, using Y = inputs[:, :, -self.y_dim:], the size of Y becomes (batch_size, T, y_dim) instead of (batch_size, T - 1, y_dim).
Could you please check this code?

Sorry, this repo is only for research purpose. Actually, the performance of DARNN is really poor, so I did not spend too much time on it.
The keras implementation is buggy, but pytorch one works. You might as well check the pytorch implementation.
Don't use this model in production, it will lose money.

Thank you for replying to my question.
I was also studying DARNN for the purpose of the study.
I will check the pytorch version as well.
Thank you again!


do not use $0 for call to mktemp

$0 will contain the entire path being called which can inject
additional path segments into the file name.  this causes
problems.
the desire was to have the base name that mktemp creates be the
script name so its hardcoded now

@mikeln passed CI, finally.


Display categories on the list view, details view and crds with relat…

Description
Changes proposed in this pull request:

...
...
...

Related issue(s)

Resolves #727

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s


doesn't work on VS 2022

I click import and a bunch of warnings pop-up and the sarif file is displayed as a json...

Hi @Leonardo-Ferreira, can you please share the repro steps and the SARIF log file so we can investigate further? Thanks

@yongyan-gh just install the extension in Visual Studio 2022 and you will get the error.

Same sort of problem for me. The vsix installed just fine in VS2022, but after loading my SARIF file I just get JSON and a File Open dialog popping up saying it can't 'Locate missing file ....'
....for every single one of my .csproj and .cs files. That's rather a lot of dialogs I have to respond to. What am I doing wrong?

Visual Studio doesn't work for me either. Only VS Code works. Is Visual Studio no longer supported?


[Snyk] Upgrade @types/node from 20.10.4 to 20.10.6

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @types/node from 20.10.4 to 20.10.6.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 2 versions ahead of your current version.
The recommended version was released 22 days ago, on 2023-12-30.

Release notes

Package name: @types/node

20.10.6 - 2023-12-30

20.10.5 - 2023-12-17

20.10.4 - 2023-12-07

    from @types/node GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

Comparison is base (c9128a5) 74.78% compared to head (011de20) 74.78%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #72   +/-   ##
=======================================
  Coverage   74.78%   74.78%
=======================================
  Files           8        8
  Lines        1967     1967
  Branches      204      204
=======================================
  Hits         1471     1471
  Misses        489      489
  Partials        7        7

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


--offline still requires wireless configuration

Running
lychee --offline .

errors with the message
Error: Failed to create request client

Caused by:
    0: Network error
    1: builder error: error reading DNS system conf: No such file or directory (os error 2)
    2: error reading DNS system conf: No such file or directory (os error 2)

if there is no network configuration. Ideally --offline would modify the codepath so dns isn't required at all.
My specific use case is a bit contrived, but basically I want to use lychee in a sandboxed environment without network. It's still useful to lint that local links are valid as cross-linking can easily get out of date when one is re-organizing their webpage.
If it's too much effort to support such a niche use case I understand, thanks!

It's not necessarily about wireless configuration, but requiring an /etc/resolv.conf to exist is a problem for me too.
It seems that this problem could be solved in the most general way in https://github.com/seanmonstar/reqwest/issues/2046.

Thanks for creating the issue and also thanks for providing a swift pull request to upstream. Here's hoping that this will be merged soon.

Oh that read like I referenced a PR, but I didn't write one (yet?). I'm not productive in Rust yet, so I figured I'd wait for a response first.

Oh, sorry, I checked on my phone and it looked like a PR.
As a workaround, can you try to create a dummy /etc/resolv.conf?
Something like
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>

@roberth, any update on this? The upstream issue on reqwest didn't seem to get much traction. Did you try the workaround above by any chance?

So looking at the upstream issue, it seems like we can pass our own resolver in that case.
We could define our own dummy resolver, similar to
https://github.com/seanmonstar/reqwest/blob/e639bdc1264c266a84dae3bae3193e07cae6861b/src/dns/gai.rs#L9
and use that when --offline is set.
If someone wants to give this a shot, please add a comment or create a PR.

Oh, okay. Thanks for the update.
Invalid configs should definitely not be silently ignored. Perhaps we should switch to https://github.com/bnjjj/twelf
We used it for another project, and it worked quite fine.

I'm adding a helper function in Nixpkgs to make this use case easier, as well as a test in Nixpkgs that can catch a regression of this use case when lychee gets updated there.

https://github.com/NixOS/nixpkgs/pull/298665

Should I add Nix integration to the features when that's done?

Interesting. I don't know much about Nixpkgs.
We already mention NixOS support in the installation section here.
Is Nix integration different from this?

It is a bit different, and a somewhat more advanced use case. It does make use of package, which was always defined in the Nixpkgs repository, but it doesn't install it to a user's profile or anything.
The goal of the testers.lycheeLinkCheck function is to use Nix as a build tool to manage both the building and checking of a static site. That's especially nice if you're already building the site in the Nix sandbox.
You could think of it as a generic CI integration, that's easy to run locally.

I've updated https://github.com/NixOS/nixpkgs/pull/298665 to explain the use case a bit more.

I see. Thanks for the explanation.
Perhaps we can mention it in the installation section or a separate section in the README.md somewhere, as it might not be strictly a feature of the CLI tool then?

However, when putting together a test for it, I wasn't able to reproduce the failure, and indeed the latest lychee in Nixpkgs seems to work fine in the Nix sandbox. I don't really know what changed, but I'm quite happy about it!

I can confirm this as well. Thanks roberth as well for the PR to nixpkgs!


styletags option doesn't always work

I took a random site from hackernews, that doesn't use service workers and does have some <style> tags; http://wiremock.org/
First I wanted to add a fix because --styletags was an available flag but it wasn't sent to the main minimalcss run function.
▶ node bin/minimalcss.js -o /tmp/before.css http://wiremock.org/
▶ node bin/minimalcss.js -o /tmp/after.css --styletags http://wiremock.org/

But!
▶ diff /tmp/before.css /tmp/after.css  # Note empty output

and
▶ ls -l /tmp/before.css /tmp/after.css
-rw-r--r--  1 peterbe  wheel  12612 Sep  7 15:23 /tmp/after.css
-rw-r--r--  1 peterbe  wheel  12612 Sep  7 15:20 /tmp/before.css

Same size :(
I can see that the rendered HTML has this:
<style>.async-hide { opacity: 0 !important} </style>

But that string (.async-hide) doesn't come into the output CSS.
Can you take a look @jc275 ?

Seems like expected behaviour, as the async-hide class is unused in the DOM.
There's a script that applies it at some point after page load:
<script>(function(a,s,y,n,c,h,i,d,e){s.className+=' '+y;h.start=1*new Date;
  h.end=i=function(){s.className=s.className.replace(RegExp(' ?'+y),'')};
  (a[n]=a[n]||[]).hide=h;setTimeout(function(){i();h.end=null},c);h.timeout=c;
})(window,document.documentElement,'async-hide','dataLayer',4000,
  {'GTM-WQTGSSM':true});</script>

I'm not familiar with it, but this might be one of those edge cases where the style is only used transiently between the page-load and network-idle events.
https://github.com/peterbe/minimalcss/blob/a5e8f41f40a33a611e06931cd83e18a01356154c/src/run.js#L298-L299
Likewise, the fluid-width-video-wrapper class in the other style tag appears to be unused.

Found it: the "page-hiding snippet"

And the <style>.fluid-width-video-wrapper{...}</style> element is added after pageload by the fitvids.js jquery plugin. Presumably, it does nothing unless there are embedded videos on the page.

Good sleuthing!


Get the versions of the software used in the MultiQC report

Add new bin/scrape_tool_versions.py inspired by @ewels and @apeltzer
Add new processes and containers to get the versions of the tool used
Replace GenerateMultiQCconfig process by createMultiQCconfig() function
Use new SarekUtils.defineDirectoryMap() function instead of in script defineDirectoryMap() function
Add forgotten QC steps
Code polishing

I'll definitively need to squash some commit together on this PR.
It's not that big actually, but there was a lot of trials and errors...

@ewels @apeltzer I had some fun making a library with some QC stuff, I think it could be interesting to use such a library system in nf-core, once we manage to sort out the reproducibility.

wow, time for some git rebase maybe..? 😆

I think I was a little too extreme with my rebase...


Validate code before signing in

Details
Fixed Issues
$ https://github.com/Expensify/App/issues/53404
PROPOSAL: https://github.com/Expensify/App/issues/53404#issuecomment-2513491605
Tests
Same QA step

[x] Verify that no errors appear in the JS console

Offline tests
QA Steps

Navigate to https://staging.new.expensify.com/.
On the login screen, enter an existing user email.
Navigate to the email inbox and open the validation email.
Click on the magic link provided in the email.
Insert “staging.” in the URL before “new.expensify.com”.
Wait until the magic link expires.
Request a new magic code from the login page.
Modify the last portion of the expired link by changing a single character (e.g., if the original link was https://staging.new.expensify.com/v/7453760/123884, modify it to https://staging.new.expensify.com/v/7453760/1238**).
Navigate to the modified and expired link and verify that it does not lead to the 2FA page but instead redirects to a “Hmm… it’s not here” page.

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/30dbc232-38ed-4ad2-aa70-c480e083b96e

Android: mWeb Chrome
https://github.com/user-attachments/assets/2ff7021a-14f2-48db-bda7-4da26fd2f478

iOS: Native
https://github.com/user-attachments/assets/f5710f45-a25d-4399-9075-f6627eb23dd7

iOS: mWeb Safari
https://github.com/user-attachments/assets/29359d31-8238-48f1-bdf2-dc2dd6679faf

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/f4a00f21-8089-40bf-922b-7af8990ccb60

MacOS: Desktop
https://github.com/user-attachments/assets/fb14a7c8-4e22-457b-9fcf-776396f05250

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.ts or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] For any bug fix or new feature in this PR, I verified that sufficient unit tests are included to prevent regressions in this flow.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop


API is broken (Internal Server Error 500)

The API currently returns (Internal Server Error 500)
Steps to reproduce:

Goto https://instagram-fullsize-picture.yasinatesim.vercel.app/api/{any_username}
Error 500

Closing as it seems to be resolved.

Looks good to me, it works fine

https://instagram-fullsize-picture.yasinatesim.vercel.app/api/instagram

The instagram username should be put after the slash char. For example

https://instagram-fullsize-picture.yasinatesim.vercel.app/api/instagram
https://instagram-fullsize-picture.yasinatesim.vercel.app/api/twitter
https://instagram-fullsize-picture.yasinatesim.vercel.app/api/buhaftanereyegitsem


[DebugForms] Fixed build error with Xcode14.3.0

This PR fixes build error in Xcode 14.3.0.
See Also:
https://github.com/apple/swift/issues/63808

Interesting. Thanks for fix!


E2E Testing

First pass at putting together an overall e2e test for the tool! Had been doing some large refactoring/testing && realized we didn't actually have any tests for the overall tool working. The test cases here still need to get fully fleshed out for the different featues of knamespacer, but this will validate the tool invocation from the cobra command on down.

@ejether this guy should be gtg for your 👁 !
Changes:

Updated package name from knamespacer to github.com/ejether/knamespacer
Exposed the rootCmd variable by updating it to RootCmd. This lets us invoke the command from other packages for testing purposes
Exposed the k8s client by updating it to K8s
Added an end to end test that uses the root cobra command and an external config file


name choice of maxLinkDim

Hi,
first let me say that I am really excited about iTensor coming to Julia. I used the C++ version a couple of years ago and this new port looks really promising. thanks for your work!
I was wondering regarding the naming of maxLinkDim, for two reasons:

According to the Julia style guide functions should be in lowercase, so it seems that maxlinkdim would be more julian (this is also true for maxDim).
Wouldn't calling it maxbonddim be more in line with the tensor networks literature ? I never saw it referred to as "link dimension" (but I am aware the core authors of the library have much greater expertise in tensor networks than me, so please correct me if I'm wrong). I guess this is related to the fact that the bond indices of the MPS are tagged as "Link".

Probably point 2 is less important, although it might make it more intuitive and discoverable for new users who are familiar with the literature.

Hi,
Thanks for your comments, contributions and interest in ITensor.

Miles and I went back and forth about using camel casing like we do in the C++ version versus using the all lower case style of Julia. We both felt like camel casing is more readable, so we were trying to decide if following along with the Julia convention for the sake of being consistent with Julia Base and other Julia packages was worth it. In the end, we decided that we will continue with the C++ convention, since we prefer how it looks and reads and there didn't seem to be much of a downside (in general, it is pretty clear when someone is using an ITensor function, and we can follow the Julia convention for Base functions that we overload on ITensors which makes it clear it is an overloaded function). I think once people realize that camel casing is the convention that ITensors.jl uses, they will get used to it and it won't lead to too much trouble.
Right now the convention in ITensors.jl is mixed, just based on when the functions were written and who wrote them, but we plan to change the names to camel casing and use camel casing going forward.

Part of the goal of ITensor going forward is to be of interest to people not just in the physics/tensor network community (for example, we hope it will be of interest in data science/machine learning). From that perspective, I think the term "bond" is fairly jargony (I'm not sure if it is used outside of the tensor network community). I think "link" is a bit more neutral, and I don't know if there is any standard term that means "indices/dimensions summed over in a series of tensor contractions". I would say that it has worked pretty well so far, and I don't see a very strong reason to switch right now (once you learn what we use "link" for, we use it consistently throughout the library so I think it is easy enough to remember).

Cheers,
Matt

Hi Matt,
That makes sense, thanks for the detailed reply!


Cant compile with default doorbell.yaml

I try compile  default doorbell.yaml with esphome 2022.1.1
Error
  variables:
    tune: int
    volume: int
    mode: int
  then:  [source doorbell.yaml:30]
    - [source doorbell.yaml:30]

      Unable to find action with the name 'remote_transmitter.transmit_szy8039b'.
      remote_transmitter.transmit_szy8039b:  [source doorbell.yaml:31]

In gl03 in yaml was section external_componets, may be this should have it too?

Sorry - yes, that section is needed.  I developed this as a local custom component, and forgot to add this section when preparing it for release as an external component.  I've just pushed a commit which adds the missing external_components section.


Standalone Tunnel establishment should disable border gateway configuration at platform level

Standalone tunnel establishment should disable the platform-level border gateway configuration directly instead of relying on the restricted routing flag sent as part of the StatusReport in response to a TunnelOpen.

@pidarped, can this issue be closed now that 173 and 174 have been closed and merged?


Serialization can be negotiated using id to improve performance.

When user set serialization="hessian2", use corresponding id internally.

@chickenlj Could u pls. describe it with more details?

Firstly, this is a performance tuning task.
Dubbo internally have a name-id mapping for each serialization implementation, for example, hessian - 2
fastjson - 6
What I suggest is, use serialization id instead of name whenever we need to negotiate serialization between Provider and Consumer.

the URL registered to the registry.
payload on the wire during an RPC.

At present, I think it's no big deal whether we are using serialization name or id. Considering the tasks we have on hand, we can tune it later.


update prehook job to use namespace override

Description
updating the prehook Job in the helm chart to allow for overriding the namespace the resources are created in as well as the namespace the secret is created in
Related Issue(s)

fixes https://github.com/konstructio/gitops/issues/81
How to test
helm template kubefirst-api ./charts/kubefirst-api

let's get this merged 🚀


Issue #73: Release of CT Starter Kit narrative. ...

Purpose

SLIM Starter Kit/Continuous Testing with Dependabot sample to illustrate Software Composition Analysis (SCA)

Proposed Changes

Add Dependabot starter kit for:

Dependency updates
Security scanning of OSS dependencies

Issues

#73

Testing

Generated Markdown available at https://github.com/NASA-AMMOS/slim/blob/issue_73/continuous-testing/starter-kits/README.md

Please review. Here's the Starter Kit in context:
https://github.com/NASA-AMMOS/slim/blob/issue_73/continuous-testing/starter-kits/README.md

Changes made. Please re-review to add any additional comments.

The Dependabot file is currently on the develop branch pending review at: https://github.com/NASA-AMMOS/slim-starterkit-python/blob/develop/.github/dependabot.yml


[Workaround available] Can't stop vertex snapping

Reproduction steps:

Select a mesh object(not a csg brush)
Press the V key

RealtimeCSG 1.314
Unity 5.4.2f2

I'll look into it, thanks for reporting

To end building a shape, you can release the V key (which is broken somehow), but you can also move the final point on the first point (closing the loop) and that should still work

Just confirmed that "releasing the V key" doesn't work when drawing a shape on a brush, but still works on when drawing the shape on the grid (oddly enough)

Oh, you found other problem ;)
Actually my report is about Unity's vertex snapping mode. Sorry for confusing you.

Ok, yes, that's something very different ;)
I'll look into this

Related to #107

A temporary workaround until I fix this: disable realtime CSG using control-F3 (available in 1.326 and up) while moving a mesh and turn it back on (using, again, control-F3) afterwards

This has been fixed in version 1.350 (key changed to X). Which can be downloaded from https://prenominal.nl/updates/
This ticket will be closed when this version is available from the asset store

This version is now available from the asset store, so I'm closing this ticket


Restore window state

Currently, if the app starts, it doesn't restore to the previous window state. See https://github.com/LinwoodDev/Butterfly/issues/274
For example here: https://github.com/leanflutter/window_manager/blob/main/lib/src/window_manager.dart#L112 This will override the current size if it's maximized

Please set to Hidden at launch and restore your saved state values during the hiding phase.

You mean I should save the position and size in a shared_preference and load them after that?
Is there no native way to do this?

Yes, I think this is the easiest way to do it. I don't want to use native to implement this function, it will take me a lot of time and is not worth it.


Pin View Redesign

This bounty rewards 6M Blockpoints.
Our current pin view is in need of major love.

Remove the confirmation button and add a custom keyboard like the send screen.
Present and dismiss it beautifully with both light and dark modes.
Add an option to enable and disable it within settings and for 3 scenerios - during launch, for sending payments, when trading. Viewing mnemonic  should always require a passcode.
Use Coinbase as an inspiration.

... I'll take it... again :)

I think that does it for this bounty, when #43 gets accepted!


Read Me Update

Hi @ambrishm, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;


util is undefined build.js

/home/jakob/projects/op/tasks/build.js:104
        util.log(util.colors.blue('[browserify]'), util.colors.red(`Error: ${err.message}`));
        ^

ReferenceError: util is not defined
    at Readable.<anonymous> (/home/jakob/projects/op/tasks/build.js:104:9)
    at emitOne (events.js:101:20)
    at Readable.emit (events.js:188:7)
    at Labeled.<anonymous> (/home/jakob/projects/op/node_modules/read-only-stream/index.js:28:44)
    at emitOne (events.js:96:13)
    at Labeled.emit (events.js:188:7)
    at Labeled.<anonymous> (/home/jakob/projects/op/node_modules/stream-splicer/index.js:130:18)
    at emitOne (events.js:101:20)
    at Labeled.emit (events.js:188:7)
    at Deps.<anonymous> (/home/jakob/projects/op/node_modules/stream-splicer/index.js:130:18)

Should be $util.

Thanks for report. Both this and #6 are fixed in v3.0.4.


Language wishlist

A list of languages currently considered for training and adding to the Repo:

[x] Swedish
[x] Norwegian
[x] French
[x] Turkish
[x] Simplified Chinese
[x] Russian
[x] Ukrainian
[ ] Catalan
[ ] Dutch
[ ] Farsi
[ ] Italian
[ ] Portugese
[ ] Spanish
[ ] Vietnamese

I'll see if I can train models for languages on this list. If you want to speed it up, just train it yourself following https://github.com/bminixhofer/nnsplit/blob/master/train/train.ipynb :)

Hello!
Can you add French to the list?
Thanks :)

Can you please Add Turkish ?

Sure!

Sure!

THanks, will you add it today or later ?

I added it to the list. I can't promise when I'll get around to training it.

drive-download-20200912T101135Z-001.zip
I have trained them on your training code, can you see and update today please ?
or can you tell me how can I use the model on these code
def recognize_speech(wav_path, lang="en", buffer_size=4000):
download_model(lang)

vosk.SetLogLevel(-1)

wav_file = wave.open(wav_path, "rb")

recognizer = vosk.KaldiRecognizer(
    vosk.Model("{}/{}".format(get_model_path(), lang)),
    wav_file.getframerate())

words = []

for index in tqdm(range(0, wav_file.getnframes(), buffer_size)):

    frames = wav_file.readframes(buffer_size)

    if recognizer.AcceptWaveform(frames):

        result = json.loads(recognizer.Result())

        if len(result["text"]) > 0:

            for token in result["result"]:
                words.append({
                    "start": token["start"],
                    "end": token["end"],
                    "text": token["word"],
                })
print(words)

return words

Can you please Add Turkish ?

Hey good match , I was looking that very long time ....

Thanks for training the model! Your torchscript exported model was somehow broken.. Which pytorch version do you use?
I managed to recover it using the weights from the ONNX graph. This should work:
torchscript_cpu_model.zip
(rename it from .zip to .pt, I'm too lazy to upload it externally right now)
Load it like this:
import nnsplit

splitter = nnsplit.NNSplit(torch.jit.load("torchscript_cpu_model.pt"), "cpu")
for sentence in splitter.split(["Bu bir cümle Bu ikinci bir cümle."])[0]:
    print(str(sentence))

which prints:
Bu bir cümle
Bu ikinci bir cümle.

(sorry if this is broken turkish)
I'll properly add it to the repository later!
Also, I'm happy to see some more interest in the library now, I'll move forward with some changes I had thought about (customizable speed / accuracy tradeoff, more robust training).

Thanks for training the model! Your torchscript exported model was somehow broken.. Which pytorch version do you use?
I managed to recover it using the weights from the ONNX graph. This should work:
torchscript_cpu_model.zip
(rename it from .zip to .pt, I'm too lazy to upload it externally right now)
Load it like this:
import nnsplit

splitter = nnsplit.NNSplit(torch.jit.load("torchscript_cpu_model.pt"), "cpu")
for sentence in splitter.split(["Bu bir cümle Bu ikinci bir cümle."])[0]:
    print(str(sentence))

which prints:
Bu bir cümle
Bu ikinci bir cümle.

(sorry if this is broken turkish)
I'll properly add it to the repository later!
Also, I'm happy to see some more interest in the library now, I'll move forward with some changes I had thought about (configurable speed / accuracy tradeoff at inference, more robust training).

Thanks a ton.  Great work.

Could you please also add Simplified Chinese? Thanks a lot.

Hi,
You can use this file for training, just change the link of the file given below.
https://github.com/bminixhofer/nnsplit/blob/master/train/train.ipynb
Chinese library
https://linguatools.org/tools/corpora/wikipedia-monolingual-corpora/
https://www.dropbox.com/s/hp1saohatuxm1mv/zhwiki-20181001-corpus.xml.bz2?dl=0
All the best
Regards,
Suti.

Very interested in french model as well !

I was a bit busy lately. I'm now working on training and evaluating all models currently in the list (Norwegian, French, Swedish, Turkish, Simplified Chinese). Will be done tomorrow.
I also improved the model a bit, it's now faster and more accurate through a downsampling trick (downsample -> LSTM -> upsample).

I trained all the models and released them as Release 0.5.0.
You can now do:
import nnsplit

print(nnsplit.__version__) # should be 0.5.0-post0

# english
nnsplit.NNSplit.load("en")
# german
nnsplit.NNSplit.load("de")
# turkish
nnsplit.NNSplit.load("tr")
# french
nnsplit.NNSplit.load("fr")
# norwegian
nnsplit.NNSplit.load("no")
# swedish
nnsplit.NNSplit.load("sv")
# chinese
nnsplit.NNSplit.load("zh")

Training went well, metrics are in the README.
I'll have to retrain the chinese model though: Chinese punctuation (e. g. 。) is not in string.punctuation so it wasn't getting removed.
Also, as I mentioned, I made some improvements to the model architecture so it's quite a bit more accurate now.

@bminixhofer Awesome! This is super helpful, thank you for putting effort into helping random people on the internet! :D

You're welcome!

@aguang-xyz As of 0.5.2 I retrained the Chinese model with fixed punctuation removal. It should now work properly for text without punctuation. Metrics are still not very good but consistently better than Spacy.

Hi, could you please add Russian?

And for Ukrainian if this is not so hard. Anyway, I will try to build a model by myself using informatino from the notebook.

Hi, sure!
I added them to the list. I'll give training them a go as well starting with Russian.

@egorsmkv I noticed there is some code hardcoded to use a compound splitter at the moment, it needs some small changes in train.py to remove that. I'll fix it so you can train a model.

The train.ipynb notebook is up to date now and the compound splitter issue fixed so training a model should work now.
I trained a model for Russian already and it looks good, I'll train another one for a bit longer and a Ukrainian model over night.

Russian and Ukrainian are now trained & integrated in the Repo. Would be great if you could do a quick sanity check @egorsmkv @marlon-br i.e. check if they split text without errors correctly, don't split on abbreviations and split text with some missing punctuation and case correctly using the demo:
https://bminixhofer.github.io/nnsplit/#demo
since I speak neither of these languages. Metrics also look good:
https://bminixhofer.github.io/nnsplit/#metrics

@bminixhofer I tried Russian sample text. After I removed one coma (between воплощение and построенная) it started to split the sentence on two sentences. In this case second sentence doesn't make sence because it is totally dependent on the first part of the original sentence.

Thanks for checking! There seems to have been a problem with , also being removed as punctuation during training. This might also impact some other languages, I'm retraining the affected models.

@bminixhofer I think that it would be a great synergy if you add all languages that are supported by vosk:
https://alphacephei.com/vosk/models
A lot of people would be very interested to get sentence boundaries etc. for the texts from ASR

Just tested with Ukrainian sentences, looks really good! Thank you, Ben!

Great. I'm retraining the models for 10 epochs to match the other models and fixed the punctuation issue, release will be latest on Monday.

@bminixhofer did you change something in the code or training script to fix the punctuation? I started to train Russian model by my own. Would like to understand if I should fix something too

Yes, I just pushed the commit. The nnsplit training procedure is just:

split sentences using SpaCy from a corpus which is assumed to not contain any punctuation errors
with some probability, remove punctuation at the end of the sentences
train a sequence labelling model, where for each byte the label is 1 if it is the last byte in the current sentence, zero otherwise

The problem was that here:

with some probability, remove punctuation at the end of the sentences

all chars in string.punctuation were considered punctuation. Now, this is configurable with an argument to the SpacySentenceTokenizer. I use .?! for Russian.
The underlying problem is that SpaCy makes some mistakes e. g. splits after a comma in some cases. This is not solved by the update but not removing commas should be an improvement.

@bminixhofer I see, thanks!
How many time does it generally takes to train for 10 epochs?

The bottleneck is often the SpaCy sentencizer. On my machine with an RTX 2080TI it takes ~ 2 hours for Ukrainian and ~ 10 hours for Russian.
Also, I should've said train with 10M samples. One epoch in the train.ipynb is set to use 500k samples and set to 1M by default in the Python scripts.

@bminixhofer I run Russian language model training on Google Colab Pro with V100 and it takes 2 hours for one epoch. So I expect it will take about 20 hours for 10 epochs. And this is 2 times more than you expect to have in your setup. But V100 is faster than 2080, so I wonder why it is longer

It could very well be CPU bound. I use an i5 8600k.

The bottleneck is often the SpaCy sentencizer.

Ukrainian and Russian updated models are now in the Repo. Both are significantly better now but in Russian there is still the same issue with the comma in the example text, I don't think there is anything I can do about that.
@marlon-br If you're training models you might be interested in https://wandb.ai/bminixhofer/nnsplit where I track the experiments e. g. https://wandb.ai/bminixhofer/nnsplit/runs/3poigs9a is the latest Russian run.

@bminixhofer AFAIK commas in Russian (and similar languages) are tricky even for native speakers. They have much more importance and sence than in other languages:)

This would be great if you could add next languages: Catalan, Dutch, Farsi, Italian, Portugese, Spanish and Vietnamese
Thanks in advance!

Sure, I added them to the list. I am currently focusing on nlprule so this may take some time, I appreciate PRs :) Ideally models should be trained on 10M samples but less is ok too.

@bminixhofer nlprule looks very interesting because you know, I use sentences splitting after ASR and since ASR is not perfect plus speaking language differs from language from Wikipedia sentence boundaries detection is also not perfect. For example the text after ASR looks like this:
"hey guys i'm gabby wallace and this is a go natural english lesson i got a great question from a viewer about pronunciation you know one of the most difficult sounds an english but also one of the most common sounds is that are sound and i love teaching the sound because it kind of sounds funny i was think it sounds like a pirate right or can you imagine me with a little pirate high in a hook yeah or maybe well that's exactly what it is it's a pirate sound that's what i call it anyway so we're we're to work on our pirates sounds today one particular word the question that my view or us was how do you say gee i r l girl girl is a really common word right woman girl girls a young woman okay so this is a very common word we need to know how to say it especially if you are a girl you need to be able to say i'm a girl or hey girls only girls club i don't know when i was a teenager or not a teenager maybe more a kid we used to have girls only clubs okay anyway i'm getting off the point year let's talk about pronunciation"
I think nlprule could improve the results a bit :)

Could you please also add Simplified Chinese? Thanks a lot.

will also for Traditional Chinese?

Hi, feel free to keep requests coming here (so I know what to prioritize when I circle back to this library). However, I am currently not training any new models. You can train models yourself here: https://github.com/bminixhofer/nnsplit/blob/main/train/train.ipynb.

Hi, I'd like to request the language support for this model into Indonesian language (as the nearest lang to Indonesian is English, but the result is quite unsatisfactory). While I'll try to train the model by myself (from ur last comment), I'd like to keep a lang request for Indonesian (ISO 639-1 code: "id"), until I have the time to replicate the training process and make a PR if the result is quite satisfactory.
Thanks!

Indonesian is now supported! Along with 84 other languages (all languages mentioned in this thread unless I missed anything).
I am now not looking to expand language support further for now, so closing this issue.


Analysis of the hallucination benchmark result in Appendix of your paper

Hi，nice work.
In table 7, you report the POPE result, which decreased in some sets of experiments（comparing with and without）. As your method assigns low weights to contradictory text tokens, an increase of hallucination benchmark metrics is expected in my opinion.
Do you have any comments on this, thank you.

Hi, I apologize for the delayed reply as I am currently occupied with graduation preparations and related travels.
Thanks for your kind opinion. In my view, the POPE benchmark may not be optimal for evaluating hallucination due to its excessively high scores and minimal variability. Alternative benchmarks may indeed be more suitable for these assessments (for more information, please refer to https://arxiv.org/pdf/2312.00849). After my vacation, I will augment the evaluation results from these related benchmarks if possible.


add tutorial for coredns

There is no coredns tutorial for externalDNS. This pull request makes
coredns based on minikube for working with externalDNS.

@xunpan Thank you!


Rename the build/ directory to dist/

This will allow putting all temporary build artifacts in build/ in the future, keeping all the build output artifacts in dist/
Part of https://github.com/bjorn3/rustc_codegen_cranelift/issues/1272

I will likely merge this in about 24h.


Vectored environment

Let me know your thoughts about this implementation. This will change almost all algorithms.

For now, I have kept this file inside common I guess it will be better to have a new directory for this.

I think it would be good to shift the env initialisation also inside the constructor. And have an option to pass some envs. For e.g. if Env name is cart pole and number of envs is 32. There are 32 processes instantiated in the constructor
We also might have to check how it helps in training, as it just slows it it’s not very helpful as such? Can you benchmark this on any one algo?

I think it would be good to shift the env initialisation also inside the constructor. And have an option to pass some envs.

This is the eventual aim of the whole process. Like maybe have 2 different envs that train in parallel.

We also might have to check how it helps in training, as it just slows it it’s not very helpful as such? Can you benchmark this on any one algo?

I am currently trying to incorporate this in DQN. It'll take some time refactoring the code. Till then let me know if there are any errors that you can find.

Just keep this up to date please? I'm going to add code somewhat dependent on it.

Everything seems fine to me. Just correct this build error.

Also, should we move these files to a different directory?

I think it's ok in common for now

@ajaysub110

Looks good to me!


./train.py error Synology DS1513+ DSM 6.1

Need help, thx.
Traceback (most recent call last):
File "./train.py", line 266, in
initial_weights=initial_weights)
File "./train.py", line 174, in train
digits_loss, presence_loss, loss = get_loss(y, y_)
File "./train.py", line 132, in get_loss
[-1, len(common.CHARS)]))
File<EMAIL_ADDRESS>line 1555, in softmax_cross_entropy_with_logits
labels, logits)
File<EMAIL_ADDRESS>line 1510, in _ensure_xent_args
"named arguments (labels=..., logits=..., ...)" % name)
ValueError: Only call softmax_cross_entropy_with_logits with named arguments (labels=..., logits=..., ...)

This error due to the new version of tensorflow so you should change the line  softmax_cross_entropy_with_logits to "softmax_cross_entropy_with_logits(labels=
tf.reshape(y[:, 1:],
[-1, len(common.CHARS)]),logits=
tf.reshape(y_[:, 1:],
[-1, len(common.CHARS)]))"
and the same for sigmoid.

Thx @lovemoon23
I have done the correction on both...
it'd seemed working... was in progress
Receiving just this alert...

WARNING:tensorflow:From /volume1/@appstore/py3k/usr/local/lib/python3.5/site-packages/tensorflow/python/util/tf_should_use.py:170: initialize_all_variables (from tensorflow.python.ops.variables) is deprecated and will be removed after 2017-03-02.
Instructions for updating:
Use tf.global_variables_initializer instead.

End then it returned

Killed

an idea why ?

It's just warning, i don't think it killed. Could you show me a stack trace??

I'd love. Were can I find those traces ?
Maked the changes for the alert and received "Killed"...

Depreciated in new version of tensorflow (https://github.com/tensorflow/tensorflow/blob/master/RELEASE.md#release-0120)
init = tf.initialize_all_variables()
Replaced with
init = tf.global_variables_initializer()

I also ran
python3 -m trace --count -C . ./train.py
but did nothing else than returning a

Killed

after +/- 1 min.

Thx @lovemoon23 you did the job.
My server is not strong enough to run a training
dmesg | grep -E -i -B100 'killed'

[1737318.688401] [16606]  1024 16606   903682   250893    1407   121306             0 python3
[1737319.001678] Out of memory: Kill process 16606 (python3) score 278 or sacrifice child
[1737325.126753] [16606]  1024 16606   903682   261508    1426   121708             0 python3
[1737325.432111] Out of memory: Kill process 16606 (python3) score 286 or sacrifice child
[1737325.440058] Killed process 16606 (python3) total-vm:3614728kB, anon-rss:1046032kB, file-rss:0kB


some error with aei_inference.py

when run aei_inference.py,the error occur:
init() got multiple values for argument 'hp'

the same error for me too!

the same error for me too!

@fsybigman @accountvb what pytorch_lightning version do you guys use?

@olegkhomenko I got the same error and I use pytorch_lightning 0.9.0 version. Because I followed requirements.txt.


Dynamic workflow mocks

Problem
Our GitHub workflow mocks, such as workflow_tests/mocks/platformDeployMocks.ts are statically written and updated (manually, by developers). Generally most of our team doesn't really know how these tests work, and while they're helpful, having to manually update the mocks to match the workflow files is inefficient and comes with a substantial amount of overhead.
Solution
Update our workflow tests such that the workflow mocks are dynamically generated from the workflow yaml file at the start of the test run, and optionally cleaned up afterwards. There is some existing code to generate boilerplate in workflow_tests/utils/preGenerateTest.ts that can serve as a starting point.
The goal of this issue would be that everything in workflow_tests/mocks would no longer be committed, but would be:

Generated at runtime when you start each test
(ideally) cached such that they're only re-generated if the workflow file being mocked changes

Issue OwnerCurrent Issue Owner: @Pujan92

There's not really a good GH project to associate this one with, and we won't need any design help.

Kind of looks like the Upwork automation didn't work for this one

Looking at the PR, there aren't any contributors that need payment right @roryabraham?

We're still waiting on proposals for this one. I didn't have a proposal for this one

It's actually kind of tricky to do to, since sometimes we need to tweak the mocks. And the TS functions need to be generated from YAML. Not entirely sure how the type-checking will work for that...
I wonder if the original SWM author is available to iterate on and improve these tests. cc @AndrewGable

Not sure I agree with the problem, it did take me a while to get up to speed on the mocks, but I think a dynamic mock generator will be much harder to debug than static mocks.

ok, going to close this one as not planned then. Probably pretty low ROI if you aren't sold on the problem


Create new database table to track job status

Background on the problem the feature will solve/improved user experience
There are three weekly jobs that maintain the database to the latest levels of legislation for United States and the associated States.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
These jobs have not run since June!  Nobody knows why, and when they do run, nobody knows how much they processed.
Describe the solution you'd like
a)
Create a new table, it would contain these five values:
Job Name, Region, Job Start Time (date/time stamp),  Job End Time (date/time stamp), # of objects processed successfully, # of errors encountered.
b) In each of the three jobs, insert a new record with its job name, state, start/stop times, number of objects processed, and number of errors encountered.  For "get_datasets" the number of objects would be the number of data sets downloaded and stored in FOB storage.  For "extract_files" and "analyze_text" the number of objects would represent the number of laws.
The "region" would be AZ, OH, etc. for each state, and "US" for Federal, etc.
If you run each of the three jobs weekly, and have four regions (AZ, OH, GA and US), then you should have 3x4 or 12 new rows every week.  If the database table gets too big, we can cull after so many months of activity.
A clear and concise description of what you want to happen.
New database defined, and the three python jobs modified to insert new records into this table.
The table can be reviewed in the Admin panels, or we can create a new report/display/email as a separate issue.
Tasks
Description of the development tasks needed to complete this issue, including tests,
Acceptance Criteria
Standards we believe this issue must reach to be considered complete and ready for a pull request. E.g precisely all the user should be able to do with this update, performance requirements, security requirements, etc as appropriate.

adding @shubham599

looks like I can't assign this to you yet @shubham599 through Github. but here in the notes:

assigned to @shubham599


X button does not clear input

When I enter some text in the input and then press the X button, the text disappears but once I start typing again, the text I had entered previously reappears and the text I'm currently typing comes after the previous one.
Concretly put:
Start typing "Hello" -> Input = "Hello" -> X button press -> Input = "" -> Start typing " world" -> Input = "Hello world".
Any clue ?

@clementino36 can you share a code example that reproduces this? It may be how you're saving the input.

Hi, sorry for the late answer.
SearchBar component:
<SearchBar
      style={{ width: '100%', height: 50, borderWidth: 5, borderColor: 'grey' }}
      placeholder={'Search for a vehicle'}
      ref={(ref) => this.searchBar = ref}
      data={vehicles}
      showOnLoad={true}
      onFocus={(input) => { setSearchResults(vehicles); }}
      onBack={() => {
           if (shouldGetEcoDriveReports) {
                Keyboard.dismiss();
                navigate('EcoDriveReport');
           }
           if (shouldGetEcoSummaries) {
                Keyboard.dismiss();
                navigate('Score');
           }
       }}
       handleResults={(results) => this.handleResults(results)}
       focusOnLayout={true}
       heightAdjust={-10}
       allDataOnEmptySearch={true}
       backButton={searchIcon}
       clearOnBlur={false}
/>

handleResults:
handleResults(vehicles: IVehicle[]) {
        const { setSearchResults, appReducer: { searchResults } } = this.props;
        const text = this.searchBar.getValue().toLowerCase();
        const results = vehicles.filter((vehicle) => {
            let { name } = vehicle;
            name = name.toLowerCase();
            return (name.includes(text));
        });
        if (!_.isEqual(searchResults, results)) {
            setSearchResults(results);
        }
    }

Apologies also for the late reply!
I haven't managed to reproduce the issue you describe using the above code which leads me to believe the problem lies within your setSearchResults function which you haven't provided.
Please ensure you're using the latest version @1.11.0.

No worries!
My mistake was to call the setSearchResults method on the onFocus event of the SearchBar, I commented this line and it all seems to be fine now. I'm not sure why though, setSearchResults is just a redux action that saves the results...
Thanks for your time anyway! Cheers

can you guys answer 82 ?


Fix OpenShiftStrimziOperatorKafkaWithoutRegistryMessagingIT by bumping Kafka instance to 3.7 supported by the operator

Summary
The test is failing and Kafka operator supports only 3.6.0, 3.6.1 and 3.7.0.
Please check the relevant options

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Dependency update
[ ] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[x] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests


Apollo dataset and other dataset

Hi，your work is very great and I appreciate you very much.I run your code,but i find the Apollo Competition website is no longer available.How can I upload the results and get the real results?And  do you experiment on other datasets？If you do,can you send it to<PHONE_NUMBER>@qq.com?Thank you very much!

You can download the raw Apollo dataset through this link: apollo. I have also updated the information in the readme. I hope this helps you.


Return inner stream from ForEach when done.

This allows resources to be reclaimed from the stream, similar to how StreamFuture returns its inner stream when the end is reached.
After this change, cargo test continues to pass for tokio-core, but there are some new "unused result which must be used" warnings.

Hm. Perhaps it would make sense to define a wrapper type DrainedStream<S> for this case and the StreamFuture case, in order to avoid spurious "unused result" warnings.

Hm yeah I feel like we haven't quite figure out the best story here in terms of ownership of streams and what combinators should do to be as flexible as possible. On one hand consumption in a context like this is costly due to the loss of potentially valuable resources, but we can also take a hit to ergonomics in some cases.
In this case though could the future just resolve to S itself instead of Drained<S>?

In this case though could the future just resolve to S itself instead of Drained?

That's what my first commit does. I added Drained in the second commit because it seemed awkward to return a stream that's going to give a warning like "must use -- streams do nothing unless polled" when in fact it would be an error to poll() the returned stream.

On one hand consumption in a context like this is costly due to the loss of potentially valuable resources, but we can also take a hit to ergonomics in some cases.

As far as I can tell, the ergonomic downside to my proposed changes is that you might occasionally need to add.map(|_| ()).
In the case I have in mind, my proposed changes have significant ergonomic upside. I want to read from a socket until EOF and then write something to the socket.  I want to do the reads through a ForEach and then get the socket back to do the write. If ForEach consumes its stream, then I need to split the socket or figure out some other way to keep it around for the write.
I admit that it's possible that the case I have in mind is unusual. Regardless, I think "don't unnecessarily drop things" is a pretty solid principle.

That's what my first commit does. I added Drained in the second commit because it seemed awkward to return a stream that's going to give a warning like "must use -- streams do nothing unless polled" when in fact it would be an error to poll() the returned stream.

Oh good point. I think though that as arguments to closures and such it's fine to ignore, so I don't think the warning would crop up here, right?

I think you've sold me on these semantics, seems good to me!

If I don't add the Drained<S> wrapper, the warning does crop up in the tokio-core examples:
warning: unused result which must be used: streams do nothing unless polled, #[warn(unused_must_use)] on by default
  --> examples/sink.rs:35:5
   |
35 |     l.run(server).unwrap();
   |     ^^^^^^^^^^^^^^^^^^^^^^^

warning: unused result which must be used: streams do nothing unless polled, #[warn(unused_must_use)] on by default
  --> examples/hello.rs:24:5
   |
24 |     core.run(server).unwrap();
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^

Moreover, I like that type Item = Drained<S>; succinctly conveys the semantics, while I think that type Item = S; would possibly be confusing.

Hm that seems reasonable to me actually, I wouldn't mind adding some calls to drop. I'd prefer to not expand the surface area with Drained as it seems like it may be an otherwise unnecessary wrapper.

OK. Please feel free to only pull in the first commit (https://github.com/alexcrichton/futures-rs/pull/176/commits/82afb2e62761a954041ff5313cc90450b04f193d) if you think that would be better.

I actually think that this can be done without any loss of ergonomics by adding inherent fns to individual future structs.
For example, ForEach could add:
impl<T> ForEach<T> {
    pub fn with_inner(self) -> WithInner<Self> {
        WithInner::new(self)
    }
}

such that WithInner polls the future (ForEach in this case) but then returns the final value plus ForEach::into_inner.

Ok now merged into 0.2!


Broken link.

Summary

Test plan

Codecov Report

Merging #4654 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4654   +/-   ##
=======================================
  Coverage   57.13%   57.13%
=======================================
  Files         194      194
  Lines        6565     6565
  Branches        3        3
=======================================
  Hits         3751     3751
  Misses       2814     2814

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 172c0b2...f46f94e. Read the comment docs.

/cc @ericnakagawa

Thanks for catching this!


google-services.json

Reasons for making this change:
google-services.json file should be commented on default if they want they will change otherwise it'll destroy the project

@aslamanver any links to StackOverflow discussions or other documentation about this change?


Add the ability to nest agendas

Add the ability to include other agendas within an agenda (or, perhaps, more generally to reference yaml snippets in other files).

Implemented with PR https://github.com/ARM-software/workload-automation/pull/880


Add option for disabling the proxy

This allows one to enable the proxy conditionally (e.g. via environment variables)

@clemyan v2.2.0 is out! :smile:


MAINT/DEV: add ignore[import] tags for Pylance

Reference issue
n/a
What does this implement/fix?
Pylance doesn't like when you import from optional dependencies or from extension modules. This PR adds   # type: ignore[import] tags to let it know that the imports are intentional. For a user of Pylance this is a big QOL improvement, IMO.
Additional information
An alternative is to tell contributors to turn off this Pylance check and run the test suite instead. I think my solution here is preferable, so that problems can be caught statically before running tests.
Will obviously add a blame ignore before merge.

Hmm, nevermind, this makes Mypy really unhappy. Perhaps I can configure Pylance differently locally, not sure.