[Security][x86] Kong still connected to original docker container ip after restarting scheduler

On TAF test case IntervalactionGET006, the support-scheduleris restarted after updating Consul configuration.
After restarting the scheduler, the test always got failure with error "message":"An invalid response was received from the upstream server".
On kong log, found the following error.
2021/05/06 08:28:38 [error] 113#0: *536 upstream timed out (110: Operation timed out) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "POST /scheduler/api/v2/interval HTTP/1.1", upstream: "http://<IP_ADDRESS>:48085/api/v2/interval", host: "localhost:8443"
But After restarting the scheduler, the docker container IP is changed to <IP_ADDRESS>
cherry@cherry-snapshot:~/edgexfoundry$ docker inspect edgex-support-scheduler | grep IPAddress
            "SecondaryIPAddresses": null,
            "IPAddress": "",
                    "IPAddress": "<IP_ADDRESS>",

After 10 minutes, kong connected to the correct scheduler container IP.
Running full TAF test always got this error. If only run the case IntervalactionGET006, this error might not happen.
This issue only occurred on x86_64 platform.

@beaufrusetta @lenny-intel Do you have any idea for this error?

@hutchic , here is the Kong issue we discussed yesterday that you were wanting to recreate.
@cherrycl , can this test need to be add to the SmokeTests for edgex-compose so we can play with version of Kong to see if it has been resolved in newer version? THX!

@cherrycl , FYI we have updated the 3rd party dependencies to their latest versions, which includes Kong to 2.4.
https://github.com/edgexfoundry/edgex-compose/pull/78
Please let us know if this has any impact on this issue. THX!

@hutchic , here is the Kong issue we discussed yesterday that you were wanting to recreate.
@cherrycl , can this test need to be add to the SmokeTests for edgex-compose so we can play with version of Kong to see if it has been resolved in newer version? THX!

@lenny-intel Only add the IntervalactionGET006 can't recreate the issue, so I added the whole suite to the SmokeTest in PR #404.

@cherrycl , FYI we have updated the 3rd party dependencies to their latest versions, which includes Kong to 2.4.
edgexfoundry/edgex-compose#78
Please let us know if this has any impact on this issue. THX!

@lenny-intel I raised an issue edgexfoundry/edgex-compose #81 for the mosquitto v2 issue. Would you please help to take care of it?

git clone git@github.com:edgexfoundry/edgex-compose.git
cd edgex-compose
make run
docker-compose -p edgex ps #wait for healthy
make get-token
export token=TOKENFROMABOVE
curl -H "Authorization: Bearer $token" --insecure https://localhost:8443/scheduler/api/v1/ping #works
docker-compose -p edgex restart scheduler
curl -H "Authorization: Bearer $token" --insecure https://localhost:8443/scheduler/api/v1/ping #if scheduler changed IP's this fails

Kong logs
kong                     | 2021/05/12 17:26:48 [error] 106#0: *550 connect() failed (113: Host is unreachable) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "GET /scheduler/api/v1/ping HTTP/2.0", upstream: "http://<IP_ADDRESS>:48085/api/v1/ping", host: "localhost:8443"
kong                     | 2021/05/12 17:26:51 [error] 106#0: *550 connect() failed (113: Host is unreachable) while connecting to upstream, client: <IP_ADDRESS>, server: kong, request: "GET /scheduler/api/v1/ping HTTP/2.0", upstream: "http://<IP_ADDRESS>:48085/api/v1/ping", host: "localhost:8443"

Expected result: it proxies to the new IP
Actual result: some of the nginx workers seem to continue to try to proxy to the old ip

Was this issue fixed? The related tests passed on Jenkins functional-test #451.

Was this issue fixed? The related tests passed on Jenkins functional-test #451.

I also verified on my local env. All tests are passed.

The issue happened again when using the latest edgex image.

@cherrycl , nothing has been changed to fix this issue. I suspect that it will randomly pass due to the restart not causing a new IP.

@cherrycl does this issue only happen in x86?

@cherrycl does this issue only happen in x86?

@cloudxxx8 It seems also happened on arm64, but arm64 only failed once so far.

@cherrycl , @cloudxxx8 , @JamesKButcher , @jpwhitemn , One possible work around for TAF testing is to immediately restart the scheduler container once all service have started. Then have the Scheduler test toward the end of testing so there is time for Kong cache of original IP to timeout and use the new IP. This assumes no other containers are restarted in between so that the scheduler container will get same IP when it is started as part of the existing test.
This is based on the the fact that once the container is restarted and receives new IP, it always gets the same IP on subsequent restarts as long as no other containers are restarted.

@lenny-intel I just discussed with Cherry.  The purpose of this particular test case is to verify the behavior after modifying a non-writable configuration.  It requires the service restarting, and the test should be atomic.  It's hard to diagnose the root cause in the future we workaround by the sequence change.  The simple way to avoid this failure is to disable this test case in the security enablement.

@cloudxxx8 , @jpwhitemn , this makes sense for this test since it isn't about testing access via the API Gateway, so disabling it for security enablement makes sense.

For the purposes of testing we can set dns_valid_ttl to a sufficiently low value

@hutchic sounds like a good idea.  we can set it to 1 second, and sleep 1 second after the service restarting.

@lenny-intel we verified this change works in our local environment
should we add an environment variable KONG_DNS_VALID_TTL=1 to kong in add-security.yml or in the taf folder only?

@cloudxxx8 , yes, I have that WIP. But still see issues on my system that I need to verify.

Issue fixed by edgexfoundry/edgex-compose PR #116.


Upgrade go.mod

upgrade to v2

need to update the pipeline at the same time like authorizer
https://github.com/yahoojapan/athenz-authorizer/blob/master/.circleci/config.yml#L52
The current pipeline does NOT tag with the v prefix
https://github.com/yahoojapan/athenz-authorizer/blob/d896d3edec8f0d2abd90d20ff3ffd84abb2e702c/.circleci/config.yml#L52

@WindzCUHK
oh OK.

After this fix, I'll put a tag with a v-prefix on it.

After this fix, I'll put a tag with a v-prefix on it.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   86.73%   86.73%
=======================================
  Files          14       14
  Lines         641      641
=======================================
  Hits          556      556
  Misses         75       75
  Partials       10       10

Impacted Files
Coverage Δ

handler/handler.go
92.85% <ø> (ø)

handler/transport.go
100.00% <ø> (ø)

main.go
46.05% <ø> (ø)

router/debug_router.go
95.65% <ø> (ø)

router/debug_routes.go
88.88% <ø> (ø)

service/option.go
100.00% <ø> (ø)

service/server.go
97.63% <ø> (ø)

service/tls.go
78.94% <ø> (ø)

usecase/authz_proxyd.go
100.00% <ø> (ø)

Continue to review full report at Codecov.

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

I need to include another modification, so I'll close it.


simplify __wrapprecs

previously the preconditioner being set was a very complicated IdentityOperator. Using a regular IdentityOperator means that solvers that don't support one of the preconditioners won't throw warnings since they mostly know that IdentityOperators aren't real.

Note that this will be obsoleted by https://github.com/SciML/NonlinearSolve.jl/pull/462 but that Pr is slightly breaking so it may need some more time.

@avik-pal is all of the test failures known?

Pretty much. We need a tag with updated enzyme releases

I have marked master version as -DEV to avoid an accidental release

once I merged the DI PR, it will fix the dep hell

Remove those imports and that should do it

imports removed.


Saved checkpoint "fcn_xs"

Hello.
I run example/fcn-xs/fcn-xs.py
Where is the trained data stored?
and
Is it possible to train my owndata without pre-trained parameters?

@Heidisnaps The README now includes additional info on where training data should be stored and where it can be downloaded from https://github.com/apache/incubator-mxnet/blob/master/example/fcn-xs/README.md.  To train your own data you just need to store your data in VOC2012 directory. You can modify the hyperparams used in https://github.com/apache/incubator-mxnet/blob/master/example/fcn-xs/fcn_xs.py. Let us know if you have additional questions.

@szha can you please remove "Need Triage" and add "Pending Requester Info" and "Example"

I think this issue has already been addressed above. @yzhliu please close this issue.


Navigation Styles Missing

Hello there,
Can you please add Navigation styles too in Documentation.

The left-side navigation column isn't included because it's not recommended for apps. The EASDK frame is inside the Shopify admin site and would create redundant navigation columns.


Problem compiling ORB-SLAM2 with cuba

Hello,
I am trying to run orb-slam with cuba (and possibly later add the LocalBundleAdjustmentCuba) by following the wiki (https://github.com/fixstars/cuda-bundle-adjustment/wiki/Use-cuda-bundle-adjustment-in-ORB-SLAM2). Aside from the steps mentioned I also had to comment the line if(pbStopFlag)         optimizer.setForceStopFlag(pbStopFlag) as the new optimizer doesn't have that parameter.
I compiled the cuba repo in the thirdparty folder as mentioned on the wiki and when I build orb-slam from their buid script it doesn't flag any errors when compiling Optimizer.cc. But when it is in building past vier.cc.o it accuses an error
[ 60%] Linking CXX shared library ../lib/libORB_SLAM2_CUDA.so
/usr/bin/ld: cannot find -lcuda_bundle_adjustment
collect2: error: ld returned 1 exit status
CMakeFiles/ORB_SLAM2_CUDA.dir/build.make:693: recipe for target '../lib/libORB_SLAM2_CUDA.so' failed

If it helps, in my cmakelist this is the order of added libraries (I am trying to build this solution together with another one that runs orb in cuda):
cuda_add_library(${PROJECT_NAME} SHARED
src/System.cc
src/Tracking.cc
src/LocalMapping.cc
src/LoopClosing.cc
src/ORBextractor.cc
src/ORBmatcher.cc
src/FrameDrawer.cc
src/Converter.cc
src/MapPoint.cc
src/KeyFrame.cc
src/Map.cc
src/MapDrawer.cc
src/Optimizer.cc
src/PnPsolver.cc
src/Frame.cc
src/KeyFrameDatabase.cc
src/Sim3Solver.cc
src/Initializer.cc
src/Viewer.cc
src/cuda/Allocator_gpu.cu
src/cuda/Fast_gpu.cu
src/cuda/Orb_gpu.cu
src/cuda/Cuda.cu
)

Could some one point me in the right direction? Thanks in advance!

An update:
By adding
link_directories(${PROJECT_SOURCE_DIR}/Thirdparty/cuda-bundle-adjustment/lib) above cuda_add_library(...) I receive several compilation problems regarding conversion between libraries. They can be summarized in two big groups of errors:
relocation R_AARCH64_ADR_PREL_PG_HI21 against symbol `_ZN6thrust8cuda_cub3cub11EmptyKernelIvEEvv' which may bind externally can not be used when making a shared object; recompile with -fPIC
I tried to fix that by adding target_compile_definitions(mono_euroc PRIVATE -fPIC) after each cuda_add_executable(..) but with no success.
The second type of error is
/home/xxxx/ORB_SLAM2_CUDA/Thirdparty/cuda-bundle-adjustment/lib/libcuda_bundle_adjustment.a(cuda_bundle_adjustment_generated_cuda_block_solver.cu.o): In function `void thrust::cuda_cub::parallel_for<thrust::cuda_cub::tag, thrust::cuda_cub::__transform::unary_transform_f<thrust::permutation_iterator<thrust::device_ptr<float const>, thrust::device_ptr<int const> >, thrust::device_ptr<float>, thrust::cuda_cub::__transform::no_stencil_tag, thrust::cuda_cub::identity, thrust::cuda_cub::__transform::always_true_predicate>, long>(thrust::cuda_cub::execution_policy<thrust::cuda_cub::tag>&, thrust::cuda_cub::__transform::unary_transform_f<thrust::permutation_iterator<thrust::device_ptr<float const>, thrust::device_ptr<int const> >, thrust::device_ptr<float>, thrust::cuda_cub::__transform::no_stencil_tag, thrust::cuda_cub::identity, thrust::cuda_cub::__transform::always_true_predicate>, long)':
/usr/local/cuda/include/thrust/system/cuda/detail/util.h:213:(.text._ZN6thrust8cuda_cub12parallel_forINS0_3tagENS0_11__transform17unary_transform_fINS_20permutation_iteratorINS_10device_ptrIKfEENS6_IKiEEEENS6_IfEENS3_14no_stencil_tagENS0_8identityENS3_21always_true_predicateEEElEEvRNS0_16execution_policyIT_EET0_T1_[_ZN6thrust8cuda_cub12parallel_forINS0_3tagENS0_11__transform17unary_transform_fINS_20permutation_iteratorINS_10device_ptrIKfEENS6_IKiEEEENS6_IfEENS3_14no_stencil_tagENS0_8identityENS3_21always_true_predicateEEElEEvRNS0_16execution_policyIT_EET0_T1_]+0x1b4): dangerous relocation: unsupported relocation

Don't know if it helps, but I'm trying to build this project in the jetson nano

Hi, @OniasC
I'm sorry to have troubled you.
It seems ORB_SLAM2 needs to link with shared libraries.
Try to fix CMakeLists.txt as follows, and build and install.
-cuda_add_library(cuda_bundle_adjustment ${srcs})
+cuda_add_library(cuda_bundle_adjustment SHARED ${srcs})

Regards,

Hi, @atakagi-fixstars
Thank you for answering, and your suggestion did eliminate that error, thank you very much for that! But that didnt eliminate all the errors, unfortunately. When, building I get this error:
Built target cuda_bundle_adjustment
[ 60%] Built target ORB_SLAM2_CUDA
[ 63%] Linking CXX executable mono_euroc_savemap
../lib/libORB_SLAM2_CUDA.so: undefined reference to `cuba::CudaBundleAdjustment::create()'
collect2: error: ld returned 1 exit status

My cmakelists is as follows:
[...]
cuda_include_directories(
  ${CUDA_TOOLKIT_ROOT_DIR}/samples/common/inc
)

link_directories(${PROJECT_SOURCE_DIR}/Thirdparty/cuda-bundle-adjustment/lib)

cuda_add_library(${PROJECT_NAME} SHARED
src/System.cc
src/Tracking.cc
src/LocalMapping.cc
src/LoopClosing.cc
src/ORBextractor.cc
src/ORBmatcher.cc
src/FrameDrawer.cc
src/Converter.cc
src/MapPoint.cc
src/KeyFrame.cc
src/Map.cc
src/MapDrawer.cc
src/Optimizer.cc
src/PnPsolver.cc
src/Frame.cc
src/KeyFrameDatabase.cc
src/Sim3Solver.cc
src/Initializer.cc
src/Viewer.cc
src/cuda/Allocator_gpu.cu
src/cuda/Fast_gpu.cu
src/cuda/Orb_gpu.cu
src/cuda/Cuda.cu
)

cuda_add_library(cuda_bundle_adjustment SHARED ${srcs})

target_link_libraries(${PROJECT_NAME}
${OpenCV_LIBS}
${EIGEN3_LIBS}
${Pangolin_LIBRARIES}
${PROJECT_SOURCE_DIR}/Thirdparty/DBoW2/lib/libDBoW2.so
${PROJECT_SOURCE_DIR}/Thirdparty/g2o/lib/libg2o.so
cuda_bundle_adjustment
${CUDA_cusparse_LIBRARY}
${CUDA_cusolver_LIBRARY}
nvToolsExt
)
[...]

Hi, @OniasC
You need to edit cuda-bundle-adjustment/src/CMakeLists.txt, not ORB_SLAM2/CMakeLists.txt.
Add SHARED to this line
https://github.com/fixstars/cuda-bundle-adjustment/blob/master/src/CMakeLists.txt#L38
Then, build and install.
$ cd cuda-bundle-adjustment
$ mkdir build
$ cd build
$ cmake .. -DCMAKE_INSTALL_PREFIX={YOUR_ORB_SLAM2_DIR}/Thirdparty/cuda-bundle-adjustment

$ make -j 4
$ make install

Regards,

Hi @atakagi-fixstars
It worked! Thank you for the support! I'll mark this issue as closed.
Regards,


Overworld

[]https://github.com/users/kdeng2001/projects/2/views/1?pane=issue&itemId=67041909

-[ ] https://github.com/users/kdeng2001/projects/2/views/1?pane=issue&itemId=67041909

[ ] https://github.com/kdeng2001/Final-Melody/issues/26


Discount statement is not in the correct position.

The discount statement "Get the discount on your first order" should be positioned at the top to capture the customer's attention.

Please add labels under GSSOC'24.


Send Max + Custom Priority

Task Description
Currently, send max and custom priority are working fine independently but we've to set up a logical flow that combines both of them and enables custom fee for send max.

UI flow for setting transaction priority at send screen before the confirmation screen. The set amount will update/change according to the fee and user's balance
Section:
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=5590-6434&t=udEvSEySdLMeJyAM-1
s.s

#4847 same

Verified this issue on keeper app v 1.2.20(419)


Netatmo, handle missing thermostat devices

Description:
The netatmo platform should not crash or log an error if the user does not have any thermostats.
Only tested with an account without thermostats.
Some code clean up.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable ([example][ex-requir]).
[x] New dependencies are only imported inside functions that use them ([example][ex-import]).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

Untagging for milestone as it fixes things not part of the beta.


Fix exceptions with exam scores

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hi, this will not affect the users. The current action is broken and this fixes it. Thanks
Sent from my iPhone
On Nov 4, 2022, at 5:07 PM, Amjed Ayoub @.***> wrote:

[EXTERNAL] This message originated outside of Billhighway/Greekbill/Impexium.
@Amjed-Ayoub requested changes on this pull request.

In certified-connectors/Impexium/apiDefinition.swagger.jsonhttps://github.com/microsoft/PowerPlatformConnectors/pull/2126#discussion_r1014467401:

         "type": "object",

         "properties": {

           "pageNumber": {

             "type": "integer",

             "format": "int32",

             "description": "Page Number",

             "title": "Page Number"

           },

           "dataList": {

             "type": "array",

             "items": {

               "$ref": "#/definitions/ExamScoreResultData"

             },

             "description": "dataList"

           }

         "type":"array",

@RjImpxhttps://github.com/RjImpx The type has been changed for the output field. Existing flows may be broken if they are using this field and actual payload has been changed. can you please confirm that this will not affect any users?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerPlatformConnectors/pull/2126#pullrequestreview-1169193651, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARLDTJ7HBO2OUFYL2V2PMB3WGV3H5ANCNFSM6AAAAAARXE7CWU.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @RjImpx,
Thank you so much for the confirmation.
Congratulations, your pull request is approved and merged. Please submit the files to ISV Studio or update your submission using your latest commit ID, please allow us up to 1-2 weeks to review your submission, and our Engineers will notify you in the “Activity Control” area in ISV Studio. Please make sure that the icon meets all the requirements, that it’s size is 230x230, icon background is not white or transparent and it matches the iconBrandColor property in apiProperties.json file.
After making the submission in ISV Studio, please attach an intro.md file in the activity control section. This file is required and it's different from readme.md. You can find an example of intro.md file on this page at step 6.
Please Create an environment in the Preview region. You will use this environment later to test your connector after Microsoft is done with the functional verification for your connector.
We expect all tests to be completed within 1-2 weeks.
If your connector passes all tests, the deployment process begins, and it typically takes up to 3 to 4 weeks to deploy your connector to all of our regions.
Please let me know if you have any questions or run into any problems while creating or updating your submission.
Thank you very much for working with us.


The onchange method of the TextField component in iOS browser receives a different string than the input string

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
TextField
(
  textInputAction: TextInputAction.search,
  onChanged: (value) {
    Toast.show(value);
  },
)

Expected results
show correct string
Actual results
show incorrect string
Code sample
Code sample
TextField
      (

      textInputAction: TextInputAction.search,
      onChanged: (value) {
        Toast.show(value);
      },
    )

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/2295872/5f882fd4-d85d-4c9c-bd9b-385547b79f12
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
╰ flutter doctor -v
[✓] Flutter (Channel stable, 3.13.9, on macOS 13.0 22A380 darwin-arm64, locale zh-Hans-CN)
    • Flutter version 3.13.9 on channel stable at /Users/ones/Library/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision d211f42860 (7 weeks ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn

[✗] Android toolchain - develop for Android devices
    • Android SDK at /Users/ones/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.

[✓] Xcode - develop for iOS and macOS (Xcode 14.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 14C18
    • CocoaPods version 1.12.1

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

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/macos#android-setup for detailed instructions).

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • Flutter plugin version 76.3.4
    • Dart plugin version 232.9559.10

Hi @lyf571321556.
Thanks for filing this.
Please can you provide a minimal complete reproducible code sample that doesn’t include 3rd party plugins or complex production code?

TextField
(
textInputAction: TextInputAction.search,
onChanged: (value) {
debugPrint('currentValue ${value}');
},
)

Hi @lyf571321556.
Thanks for your response. I can't reproduce this on my end, I get the correct and expected output values
flutter: currentValue c
flutter: currentValue ce
flutter: currentValue 测

Can you upgrade to the latest Flutter versions to confirm this on your end, if it does persist, can you share the IDE you are running this on?

Hi @lyf571321556. Thanks for your response. I can't reproduce this on my end, I get the correct and expected output values
flutter: currentValue c
flutter: currentValue ce
flutter: currentValue 测

Can you upgrade to the latest Flutter versions to confirm this on your end, if it does persist, can you share the IDE you are running this on?

This issue was reproduced on Flutter version 3.13.9, and upgrading to the latest version carries significant risks.

Thanks for your response.
Per our triage process, if an issue no longer occurs on the latest versions, we will consider this fixed and close.
Running this on version 3.13.9, I can't seem to reproduce as well.
Running on browser, I have this (see video below)
Video

https://github.com/flutter/flutter/assets/53122008/a1b91d68-2440-4aa2-9b5b-f662d977af9c

Code Sample Used

Code Sample

import 'package:flutter/material.dart';

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

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

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      theme: ThemeData.dark().copyWith(
        scaffoldBackgroundColor: darkBlue,
      ),
      debugShowCheckedModeBanner: false,
      home: Scaffold(
        body: Center(
          child: MyWidget(),
        ),
      ),
    );
  }
}

class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return TextField(
        textInputAction: TextInputAction.search,
        onChanged: (value) {
          print('currentValue $value');
          _showToast(context, value);
        });
  }

  void _showToast(BuildContext context, String text) {
    final scaffold = ScaffoldMessenger.of(context);
    scaffold.showSnackBar(
      SnackBar(
        content: Text(text),
        action: SnackBarAction(
            label: 'UNDO', onPressed: scaffold.hideCurrentSnackBar),
      ),
    );
  }
}

Can you check this out as well?


Get unused members

Hello,
I was getting ready to write something similar and came across your repo.  I took a quick look and it doesn't seem to have a few features I need.  Sorry if I missed them.

List of structs that are not referenced
Starting and ending line number of each member (structs and functions)
List of custom functions referenced by a function

My use case:
I build up shaders by including snippets.  This may cause the shader code to be unnecessarily large because it includes code that is not being used.  I would like to remove all the unused elements after the includes have been processed.
Example:
#include common_structs
#include vec2_utils
...

@vertex
fn vs...

After expansion of the includes, I would run reflect and identify any structs and functions that are unused by the vertex shader.  I would then remove them so the final shader code is smaller.
Thanks,
Wil

I think those features are doable. I'll try to get to them sometime soon. Ping me if I forget, work has been busy.

Thanks!  No worries, I know how it is - I'm pretty busy now myself.

Chipping away at it while it's fresh in my mind, and waiting for my work project to compile.
I added startLine and endLine to StructInfo and FunctionInfo, and I added "functions" to WgslReflect, which is the list of all functions (including entry functions).
I'll add struct and function tracing next, probably adding a member to StructInfo and FunctionInfo along the lines of "inUse: boolean", and I can add a list of all the custom functions called from an entry function (directly and indirectly).  But if the function inUse is false, that means the function wasn't called, so it's questionable if the list is necessary.
Here's the code I have for the unit test for line numbers,
const reflect = new WgslReflect(`
      struct A {
        a: u32,
      };
      struct B {
        b: u32,
      };
      struct C {
        c: u32,
      };
      @group(0) @binding(0) var<uniform> foo : A;
      @vertex
      fn vertex_main() -> void {
        let b = B(1);
}`);
    test.equals(reflect.structs.length, 3);

    test.equals(reflect.structs[0].startLine, 2);
    test.equals(reflect.structs[0].endLine, 4);

    test.equals(reflect.structs[1].startLine, 5);
    test.equals(reflect.structs[1].endLine, 7);

    test.equals(reflect.structs[2].startLine, 8);
    test.equals(reflect.structs[2].endLine, 10);

    test.equals(reflect.entry.vertex.length, 1);
    test.equals(reflect.entry.vertex[0].startLine, 13);
    test.equals(reflect.entry.vertex[0].endLine, 15);

    test.equals(reflect.functions.length, 1);
    test.equals(reflect.functions[0].name, "vertex_main");

Squeezed in one more update for the night: added inUse and calls fields to FunctionInfo, as in:
FunctionInfo {
  inUse: boolean;
  calls: Set<FunctionInfo>;
}

inUse will be set to true if it's called either directly or indirectly from an entry function.
calls is the set of all custom functions called either directly or indirectly from the function.
As an example, the unit test has been updated to
const reflect = new WgslReflect(`
      struct A {
        a: u32,
      };
      struct B {
        b: u32,
      };
      struct C {
        c: u32,
      };
      fn notInUse() -> void { }
      fn inUse2() -> void { }
      fn inUse() -> void { inUse2(); }
      @group(0) @binding(0) var<uniform> foo : A;
      @vertex
      fn vertex_main() -> void {
        let b = B(1);
        inUse();
      }`);
    test.equals(reflect.structs.length, 3);

    test.equals(reflect.structs[0].startLine, 2);
    test.equals(reflect.structs[0].endLine, 4);

    test.equals(reflect.structs[1].startLine, 5);
    test.equals(reflect.structs[1].endLine, 7);

    test.equals(reflect.structs[2].startLine, 8);
    test.equals(reflect.structs[2].endLine, 10);

    test.equals(reflect.entry.vertex.length, 1);
    test.equals(reflect.entry.vertex[0].startLine, 16);
    test.equals(reflect.entry.vertex[0].endLine, 19);

    test.equals(reflect.functions.length, 4);
    test.equals(reflect.functions[0].name, "notInUse");
    test.equals(reflect.functions[0].inUse, false, "notInUse");
    test.equals(reflect.functions[0].calls.size, 0, "notInUse calls");
    test.equals(reflect.functions[1].name, "inUse2");
    test.equals(reflect.functions[1].inUse, true, "inUse2");
    test.equals(reflect.functions[1].calls.size, 0, "inUse2 calls");
    test.equals(reflect.functions[2].name, "inUse");
    test.equals(reflect.functions[2].inUse, true, "inUse");
    test.equals(reflect.functions[2].calls.size, 1, "inUse calls"); // inUse2
    test.equals(reflect.functions[3].name, "vertex_main");
    test.equals(reflect.functions[3].inUse, true, "vertex_main");
    test.equals(reflect.functions[3].calls.size, 2, "vertex_main calls"); // inUse, inUse2

So I think the remaining requested feature is to add inUse to structs to see if they have been used, as a binding type or within a used function.
Unless I completely missed the mark on what was asked. That happens sometimes.

Ok, added inUse to StructInfo for struct tracking. Guess I got to this quicker than I was expecting, minus extensive testing. Minus any feedback I think it's done.
Here is the example unit test including struct tracking,
struct A {
        a: u32,
      };
      struct B {
        b: u32,
      };
      struct C {
        c: u32,
      };
      fn notInUse() -> void { }
      fn inUse2() -> void {
        let b = B(1);
      }
      fn inUse() -> void { inUse2(); }
      @group(0) @binding(0) var<uniform> foo : A;
      @vertex
      fn vertex_main() -> void {
        inUse();
      }`);
    test.equals(reflect.structs.length, 3);

    test.equals(reflect.structs[0].startLine, 2);
    test.equals(reflect.structs[0].endLine, 4);

    test.equals(reflect.structs[1].startLine, 5);
    test.equals(reflect.structs[1].endLine, 7);

    test.equals(reflect.structs[2].startLine, 8);
    test.equals(reflect.structs[2].endLine, 10);

    test.equals(reflect.entry.vertex.length, 1);
    test.equals(reflect.entry.vertex[0].startLine, 18);
    test.equals(reflect.entry.vertex[0].endLine, 20);

    test.equals(reflect.structs[0].inUse, true, "A inUse"); // A, used by uniform
    test.equals(reflect.structs[1].inUse, true, "B inUse"); // B, used by inUse2
    test.equals(reflect.structs[2].inUse, false, "C inUse"); // C is not used

    test.equals(reflect.functions.length, 4);
    test.equals(reflect.functions[0].name, "notInUse");
    test.equals(reflect.functions[0].inUse, false, "notInUse");
    test.equals(reflect.functions[0].calls.size, 0, "notInUse calls");
    test.equals(reflect.functions[1].name, "inUse2");
    test.equals(reflect.functions[1].inUse, true, "inUse2");
    test.equals(reflect.functions[1].calls.size, 0, "inUse2 calls");
    test.equals(reflect.functions[2].name, "inUse");
    test.equals(reflect.functions[2].inUse, true, "inUse");
    test.equals(reflect.functions[2].calls.size, 1, "inUse calls"); // inUse2
    test.equals(reflect.functions[3].name, "vertex_main");
    test.equals(reflect.functions[3].inUse, true, "vertex_main");
    test.equals(reflect.functions[3].calls.size, 2, "vertex_main calls"); // inUse, inUse2

Wow!  That is awesome!  Thanks for your quick response.  I do agree that with the inUse flag, the list of functions aren't strictly necessary.  It can be good for debugging though, for example displaying a graph to get an overview of the flow in order to tweak dependencies.
Everything looks good.  The one thing I would change is, if keeping the list of functions, the list should only contain those functions called directly.  My reasoning is you have all the direct references and can "walk" the list to get all indirect ones, if needed - makes creating a graph pretty simple.  If they are all in the list, you can't tell what is direct from indirect.  Memory usage will also be less because the same information won't be repeated at multiple levels.  What do you think?

Yeah, that makes sense from a graph perspective, and easy enough to change. I'll clean that up shortly.
I also finally got around to putting the library up on NPM, make things a bit easier to maintain. Better late than never, only took me two years to get to that 🙂

K, I think I got that fixed up, FunctionInfo.calls should only list the direct custom function calls from that function. I pushed the publish version. Trying to get to all this before I get swamped again with work and family stuff again in the coming couple of weeks.

Oh BTW, I came across your repo via webgpufundamentals.org and their reference to webgpu-utils which uses this library.  I believe that's by user greggman.


Returns a different result on a 32-bit platform vs 64-bit

With a code like this:
echo Tuupola\Base85Proxy::encode(md5("test", true)), "\n";

amd64:
$'/lH7Np6%b2,mG-7?1o

armv7el:
$'/lH7Np6%epLbF-7?1o

PHP 7.4 on both.
--
Just reported a similar bug to another base85 package: https://github.com/scottchiefbaker/php-base85/issues/2
that one has a more severe problem. "unpack" is problematic on 32-bit vs 64-bit, yours almost works, but also couldn't escape it.

I wonder which result is correct. I would guess the 64-bit one.

Might also be related to this: https://github.com/tuupola/base85/issues/7

Yes the 64-bit value is correct.
See some investigation on the other bug report, and some suggestions in PHP doc comments:
https://github.com/scottchiefbaker/php-base85/issues/2#issuecomment-2038821556
https://www.php.net/manual/en/function.unpack.php#106041


PersistentGpuBufferable probably shouldn't be an unsafe trait

A good example of an unsafe trait is TrustedLen or WorldQuery, where the correct implementation of the trait is required to satisfy the safety invariants in its application, and currently PersistentGpuBufferable is not used in such a way. It is not used in any unsafe context CPU-side, and as far as I know, wgpu's validation should prevent any improperly uploading invalid inputs to the GPU.
Of the two implementations of the trait, the Arc<[u8]> implementation does not actually satisfy the safety invariants requiring proper alignment of the contents (it's min alignment is 1 and wgpu::COPY_BUFFER_ALGINMENT is currently 4 and this is not enforced in it's implementation), and both implementations do not document how they're satisfying the safety invariants.
Potential solutions here:

Make the trait not unsafe.
Remove the Arc<[u8]> implementation or enforce its safety invariants.
Document the safety invariants and enforce them properly.

Arc<[u8]> is used for vertex data which is always in multiples of 48 bytes, so it satisfies the alignment.

However, that's not enforced as a part of the implementation, which means the trait implementation itself is unsound according to the safety invariants of the trait.
IMO this should instead just be replaced with an assertion to validate instead of marking this as unsafe, as it's pretty clear that this isn't a memory safety issue, as far as the Rust compiler is concerned.


juliacall segfaults when loading a package that uses PythonCall

juliacall always segfaults on me when I try to load a package that uses PythonCall.
This should reproduce the error:
import juliapkg

juliapkg.add("BATPythonCall", "dc6699a0-bee0-4ed4-84d6-ad34e0e2513f", rev = "loaddebug", url = "https://github.com/bat/BATPythonCall.jl.git")

juliapkg.resolve()
juliapkg.status()

from juliacall import Main as jl

jl.seval("using BATPythonCall")

results in
free(): invalid pointer

signal (6): Aborted
in expression starting at none:1
gsignal at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)
abort at /lib/x86_64-linux-gnu/libc.so.6 (unknown line)

BATPythonCall.jl (on branch "loaddebug") is basically an empty package that only does using PythonCall. Removing using PythonCall makes the segfault disappear.
CC @philippeller

That's very odd. I can't reproduce it on Linux in a fresh Conda environment like this:
conda create -n temp -c conda-forge python pip
conda activate temp
pip install juliapkg juliacall
python
# then enter your python code above

Can you please ensure you are on the latest versions of things? Preferably in a fresh environment (venv/conda/poetry/whatever). And if that doesn't fix it, we'll have to drill down into what's different about your system - please send the output from juliapkg.status(), pip list, conda list, conda info, etc.

Can you please ensure you are on the latest versions of JuliaPkg and JuliaCall?

Will try!

It's weird - I can reproduce the error with the juliapkg-0.1.9, juliacall-0.9.4, Julia 1.8.0-rc4 running in an Ubuntu Apptainer container. But when I run with the same container and same package/Julia versions under Docker, the segfault disappears.
I'll try more computing environment combinations ...


Disposable Turret for syndie identification?

Description

The Disposable Turret ignores syndies, meaning sec could in theory capture a turret and use it for identification.

Additional context

sorry

this is only an issue while it has no ammo, then it can be used to validcheck easily without casualties
turrets shouldnt do anything when empty

a couple of mildly shot innocents are pennies compared to the majesty of a perfect syndie identifier, plus the new changes allow sec to reconstruct the turret to refill its ammo if it gets empty

pingus can already be used for a more controlled validcheck and i actually havent ever seen it used
a turret seems far more dangerous than keeping a pingu locked in a crate


Make notes optional for event service deliveries

Issue number: n/a
Description of change
This makes notes optional for event service deliveries. (Notes are still required for other types of interaction.)
Checklist

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

Codecov Report

Merging #984 into develop will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #984   +/-   ##
========================================
  Coverage    96.72%   96.72%
========================================
  Files          193      193
  Lines         5434     5434
  Branches       513      513
========================================
  Hits          5256     5256
  Misses         116      116
  Partials        62       62

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 83c3796...3d6bf2d. Read the comment docs.


Unsolved import when using the derivative crate

rust-analyzer version: (eg. output of "Rust Analyzer: Show RA Version" command)
v0.2.956
rustc version: (eg. output of rustc -V)
rustc 1.59.0 (9d1b2106e 2022-02-23)
relevant settings: (eg. client settings, or environment variables like CARGO, RUSTUP_HOME or CARGO_HOME)
Using VS Code
The following project causes an "unresolved import" on the first line of main.rs:
Cargo.toml
==========
[package]
name = "scratch"
version = "0.1.0"
edition = "2021"

[dependencies]
derivative = "2.2.0"

main.rs
=======
use derivative::Derivative;

#[derive(Derivative)]
#[derivative(Debug)]
struct Test;

fn main() {
    let value = Test;
    println!("{value:?}");
}

It compiles and runs fine.

Duplicate of #7459. You can try setting rust-analyzer.cargo.unsetTest.

Setting unsetTest, as recommended in #7459, solved this. Thanks.


Fix VS scraper

Replace \  with a real space, and make regexp more forgiving
Closes: https://github.com/openZH/covid_19/issues/312

Is this PR ready? I know the validation fails (which is not your fault).

Yes. It is ready now, I was doing some last second minor optional changes, but it is now finished. Validation is not related to this PR.

Like really now. :)

I was just replacing some of the spaces with \s*. Just in case.

OK. Done. Sorry for hurrying it.

I think I'll wait some more minutes :joy:

I think I'll wait some more minutes joy

Ok then, let me do one more small change.

Perfect. Now there are optional spaces everywhere where they can be.
Sorry again. I was tired.

All good! Thanks for fixing.

@metaodi Thank you! Have a nice evening.


The app references non-public symbols in Frameworks/JPVideoPlayer.framework/JPVideoPlayer: _dispatch_benchmark

OK，I has deleted it.


Verifier errors (complexity) when deploying latest master on AWS EKS

Bug report
General Information

master
Kernel:

❯ ks exec -it cilium-75tj9 -- uname -a
Linux ip-192-168-40-159.us-west-2.compute.internal 4.14.198-152.320.amzn2.x86_64 #1 SMP Wed Sep 23 23:57:28 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

AWS EKS K8s

How to reproduce the issue

Deploy latest master of Cilium to EKS following standard EKS guide: https://docs.cilium.io/en/v1.9/gettingstarted/k8s-install-eks/#k8s-install-eks
Inspect logs for verifier errors like so:

level=error msg="Command execution failed" cmd="[tc filter replace dev lxc_health ingress prio 1 handle 1 bpf da obj 2621_next/bpf_lxc.o sec from-container]" error="exit status 1" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="Prog section 'from-container' rejected: Argument list too long (7)!" subsys=datapath-loader
level=warning msg=" - Type:         3" subsys=datapath-loader
level=warning msg=" - Attach Type:  0" subsys=datapath-loader
level=warning msg=" - Instructions: 3031 (0 over limit)" subsys=datapath-loader
level=warning msg=" - License:      GPL" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="Verifier analysis:" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="Skipped 9219443 bytes, use 'verb' option for the full verbose log." subsys=datapath-loader
level=warning msg="[...]" subsys=datapath-loader
level=warning msg=" *)(r0 +8)" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R1=inv(id=0) R6=ctx(id=0,off=0,imm=0) R7=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R8=inv(id=0) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="1319: (0f) r1 += r7" subsys=datapath-loader
level=warning msg="1320: (7b) *(u64 *)(r0 +8) = r1" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R1=inv(id=0) R6=ctx(id=0,off=0,imm=0) R7=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R8=inv(id=0) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="1321: (05) goto pc+178" subsys=datapath-loader
level=warning msg="1500: (18) r7 = 0xffff8881c6b66800" subsys=datapath-loader
level=warning msg="1502: (bf) r1 = r6" subsys=datapath-loader
level=warning msg="1503: (bf) r2 = r7" subsys=datapath-loader
level=warning msg="1504: (b7) r3 = 1" subsys=datapath-loader
level=warning msg="1505: (85) call bpf_tail_call#12" subsys=datapath-loader
level=warning msg="1506: (05) goto pc-1360" subsys=datapath-loader
level=warning msg="147: (b7) r8 = 2" subsys=datapath-loader
level=warning msg="148: (bf) r0 = r8" subsys=datapath-loader
level=warning msg="149: (95) exit" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="from 1314 to 1489: R0=inv0 R6=ctx(id=0,off=0,imm=0) R7=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R8=inv(id=0) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="1489: (7b) *(u64 *)(r10 -56) = r7" subsys=datapath-loader
level=warning msg="1490: (b7) r1 = 1" subsys=datapath-loader
level=warning msg="1491: (7b) *(u64 *)(r10 -64) = r1" subsys=datapath-loader
level=warning msg="1492: (bf) r2 = r10" subsys=datapath-loader
level=warning msg="1493: (07) r2 += -136" subsys=datapath-loader
level=warning msg="1494: (bf) r3 = r10" subsys=datapath-loader
level=warning msg="1495: (07) r3 += -64" subsys=datapath-loader
level=warning msg="1496: (18) r1 = 0xffff8881fb2a0000" subsys=datapath-loader
level=warning msg="1498: (b7) r4 = 0" subsys=datapath-loader
level=warning msg="1499: (85) call bpf_map_update_elem#2" subsys=datapath-loader
level=warning msg="1500: safe" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="from 1292 to 148: R1=inv0 R2=inv0 R3=inv1076582406 R4=inv(id=0,umin_value=1536,umax_value=65280,var_off=(0x0; 0xff00)) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv4294967162 R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="148: (bf) r0 = r8" subsys=datapath-loader
level=warning msg="149: (95) exit" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="from 1288 to 1290: R1=inv2 R2=inv1 R3=inv1076582406 R4=inv(id=0,umin_value=1536,umax_value=65280,var_off=(0x0; 0xff00)) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv4294967162 R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="1290: (77) r1 >>= 31" subsys=datapath-loader
level=warning msg="1291: (4f) r1 |= r2" subsys=datapath-loader
level=warning msg="1292: safe" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="from 25 to 102: R1=pkt_end(id=0,off=0,imm=0) R2=inv4294967157 R3=inv(id=0) R4=inv(id=0,umin_value=1536,umax_value=65280,var_off=(0x0; 0xff00)) R6=ctx(id=0,off=0,imm=0) R7=inv0 R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="102: (b7) r1 = 2" subsys=datapath-loader
level=warning msg="103: (63) *(u32 *)(r6 +64) = r1" subsys=datapath-loader
level=warning msg="104: (18) r1 = 0x4" subsys=datapath-loader
level=warning msg="106: (63) *(u32 *)(r6 +48) = r1" subsys=datapath-loader
level=warning msg="107: (63) *(u32 *)(r6 +60) = r7" subsys=datapath-loader
level=warning msg="108: (63) *(u32 *)(r6 +56) = r2" subsys=datapath-loader
level=warning msg="109: (63) *(u32 *)(r6 +52) = r7" subsys=datapath-loader
level=warning msg="110: (61) r8 = *(u32 *)(r6 +0)" subsys=datapath-loader
level=warning msg="111: (7b) *(u64 *)(r10 -56) = r7" subsys=datapath-loader
level=warning msg="112: (7b) *(u64 *)(r10 -64) = r7" subsys=datapath-loader
level=warning msg="113: (b7) r1 = 512" subsys=datapath-loader
level=warning msg="114: (7b) *(u64 *)(r10 -136) = r1" subsys=datapath-loader
level=warning msg="115: (87) r2 neg 0" subsys=datapath-loader
level=warning msg="116: (73) *(u8 *)(r10 -136) = r2" subsys=datapath-loader
level=warning msg="117: (bf) r2 = r10" subsys=datapath-loader
level=warning msg="118: (07) r2 += -136" subsys=datapath-loader
level=warning msg="119: (18) r1 = 0xffff8881fb2a0000" subsys=datapath-loader
level=warning msg="121: (85) call bpf_map_lookup_elem#1" subsys=datapath-loader
level=warning msg="122: (15) if r0 == 0x0 goto pc+7" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="123: (79) r1 = *(u64 *)(r0 +0)" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="124: (07) r1 += 1" subsys=datapath-loader
level=warning msg="125: (7b) *(u64 *)(r0 +0) = r1" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R1=inv(id=0) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="126: (79) r1 = *(u64 *)(r0 +8)" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R1=inv(id=0) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="127: (0f) r1 += r8" subsys=datapath-loader
level=warning msg="128: (7b) *(u64 *)(r0 +8) = r1" subsys=datapath-loader
level=warning msg=" R0=map_value(id=0,off=0,ks=8,vs=16,imm=0) R1=inv(id=0) R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="129: (05) goto pc+11" subsys=datapath-loader
level=warning msg="141: (18) r7 = 0xffff8881c6b66800" subsys=datapath-loader
level=warning msg="143: (bf) r1 = r6" subsys=datapath-loader
level=warning msg="144: (bf) r2 = r7" subsys=datapath-loader
level=warning msg="145: (b7) r3 = 1" subsys=datapath-loader
level=warning msg="146: (85) call bpf_tail_call#12" subsys=datapath-loader
level=warning msg="147: (b7) r8 = 2" subsys=datapath-loader
level=warning msg="148: (bf) r0 = r8" subsys=datapath-loader
level=warning msg="149: (95) exit" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="from 122 to 130: R0=inv0 R6=ctx(id=0,off=0,imm=0) R7=inv0 R8=inv(id=0,umax_value=4294967295,var_off=(0x0; 0xffffffff)) R9=pkt(id=0,off=0,r=14,imm=0) R10=fp0" subsys=datapath-loader
level=warning msg="130: (7b) *(u64 *)(r10 -56) = r8" subsys=datapath-loader
level=warning msg="131: (b7) r1 = 1" subsys=datapath-loader
level=warning msg="132: (7b) *(u64 *)(r10 -64) = r1" subsys=datapath-loader
level=warning msg="133: (bf) r2 = r10" subsys=datapath-loader
level=warning msg="134: (07) r2 += -136" subsys=datapath-loader
level=warning msg="135: (bf) r3 = r10" subsys=datapath-loader
level=warning msg="136: (07) r3 += -64" subsys=datapath-loader
level=warning msg="137: (18) r1 = 0xffff8881fb2a0000" subsys=datapath-loader
level=warning msg="139: (b7) r4 = 0" subsys=datapath-loader
level=warning msg="140: (85) call bpf_map_update_elem#2" subsys=datapath-loader
level=warning msg="141: safe" subsys=datapath-loader
level=warning msg="BPF program is too large. Processed 131073 insn" subsys=datapath-loader
level=warning subsys=datapath-loader
level=warning msg="Error fetching program/map!" subsys=datapath-loader
level=warning msg="Unable to load program" subsys=datapath-loader

Because all endpoints are failing to compile, this leads to cilium-health status reporting:
❯ ks exec -it cilium-75tj9 -- cilium-health status
Probe time:   2020-11-12T01:36:28Z
Nodes:
  ip-192-168-40-159.us-west-2.compute.internal (localhost):
    Host connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=285.256µs
      HTTP to agent:   OK, RTT=417.697µs
    Endpoint connectivity to <IP_ADDRESS>:
      ICMP to stack:   Connection timed out
      HTTP to agent:   Get "http://<IP_ADDRESS>:4240/hello": context deadline exceeded (Client.Timeout exceeded while awaiting headers)
  ip-192-168-6-117.us-west-2.compute.internal:
    Host connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=3.132166ms
      HTTP to agent:   OK, RTT=1.409103ms
    Endpoint connectivity to <IP_ADDRESS>:
      ICMP to stack:   OK, RTT=3.106842ms
      HTTP to agent:   Get "http://<IP_ADDRESS>:4240/hello": dial tcp <IP_ADDRESS>:4240: connect: connection refused

The same Cilium version deployed on GKE running 4.19 kernel is fine. This seems to be an issue with < 4.19 kernels, likely 4.9.

If you revert the two recent commits ...

https://github.com/cilium/cilium/commit/1ed1db1b943582afadca64b8477fcfdd94a7252c
https://github.com/cilium/cilium/commit/a7999315286d203fae2614ce7ed10c640dc4d13c
... is the complexity issue gone?

I suspect after we merge #13347 (which includes some changes to reduce the complexity also on older kernels) this issue should go away.
Let me first try to reproduce it locally

Sysdump:
cilium-sysdump-20201112-085216.zip

I can observe the same issue with cilium 1.9.0, ubuntu 20.04.1 and kernels 5.9.8-050908-generic and 5.8.18-050818-generic:

@sakharovaan Did you deploy with the same Helm flags as @christarazi? Could you post a sysdump?

@pchaigno sure, here is sysdump and helm values
cilium-sysdump-20201113-183708.zip
values.yaml.txt

@christarazi's issues seem to be gone on latest master on EKS
@sakharovaan: I'm not sure how to reproduce your EKS setup, so you could either test an image based on master or wait a few days for the changes (which hopefully should solve also your issue) to get backported to 1.9

@jibi that's actually not an EKS, but a on-premises setup with same error message.

@sakharovaan Could you open a new issue with steps to reproduce?
It looks like the issue may be different from the bug reported at the beginning of this thread. Providing reproduction steps (e.g., what Helm/cilium-agent command you used to deploy) would help us validate that.
If the bug is the same, then v1.9.1 should fix it. You can already test the fix today with cilium/cilium:latest.

I'm closing this ticket for now as I can't reproduce the EKS issues anymore


Run OnChanges when flyout visibility changes

When a block is dragged from the flyout and it automatically closes, I guess a new block is generated after the script runs?

No... even after that event, when the flyout is closed and blocks are dragged around, the blocks in the flyout fail to update


fix: chart-push

oci repository name needs to be lowercase

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 68.163%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
987

Relevant Lines:
1448

💛  - Coveralls


Release 1.39.0

What
Nomis dataset landing pages copy changed
How to review

Check nothing has slipped into this PR that shouldn't be here
Check that all work has been signed off

Who can review
Anyone except me

Wrong repo


Cleanup #3677: use snprintf instead of deprecated sprintf

Avoid warnings from clang about sprintf being deprecated

I added changes for LBComm, envelope, pup, and various ck-libs. The build is now completely free of deprecated warnings.

This is good cleanup.
Not to add extra work to a PR that changes many lines, but in places where snprintf is passed a numeric literal reproducing the buffer's size, it is better to use sizeof(bufName) so that the two never go out of sync.

Not to add extra work to a PR that changes many lines, but in places where snprintf is passed a numeric literal reproducing the buffer's size, it is better to use sizeof(bufName) so that the two never go out of sync.

Done

This is ready to merge, but should be squashed first.


As a user, I want the labelling option extracted

Description
Context
Labelling options indicate whether a fertilizer uses "record keeping" or "no record keeping":

Record keeping: The label does not list active ingredients, using a generic statement like "Active ingredients derived from registered and exempt materials."
No record keeping: The label explicitly lists active ingredients, e.g., "Active ingredients derived from: Composted turkey litter, etc."

Problem Statement
We need to add a field in the pipeline that identifies the labelling option, indicating whether record keeping is used or not.
Acceptance Criteria

Implement a field in the pipeline to extract and indicate whether the labelling option is "record keeping" or "no record keeping" (e.g., a boolean).
Ensure that the pipeline accurately extracts this information from the provided data.
Validate the implementation with clear test cases.

Additional Information

T-4- 131: Record keeping requirements under the Fertilizers Act and Regulations
Example PDF: Record Keeping Example PDF

This issue is on hold due to an insufficient number of sample labels needed for solution design. For now, we will implement the field in the backend with a hardcoded default value. The user will be responsible for determining whether record keeping is present. The frontend will provide an input option for users to manually enter this information.


Making application id randomly generated based on the current time.

Short description outlining key changes/additions
JIRA Ticket Number
Type of change

[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Infrastructure change

Pull request checklist

[ ] I have added unit tests for new code that I have added
[ ] I have added/updated functional tests where appropriate
[ ] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: No security warnings found

CI: No security warnings found


Gradle incremental annotation rocessing

Hi @doanduyhai
I'd liek to propose the following enhancement: Gradle supports incremental annotation processing, meaning the annotation processor won't have to run on every new compilation.
For that there is a simple guide to follow there to make sure the annotation processor is eligible to incremental annotation processing: https://docs.gradle.org/current/userguide/java_plugin.html#sec:incremental_annotation_processing
If I'm not mistaken achilles core is ok with these points :

[x] They must generate their files using the Filer API. Writing files any other way will result in silent failures later on, as these files won’t be cleaned up correctly. If your processor does this, it cannot be incremental.

[x] They must not depend on compiler-specific APIs like com.sun.source.util.Trees. Gradle wraps the processing APIs, so attempts to cast to compiler-specific types will fail. If your processor does this, it cannot be incremental, unless you have some fallback mechanism.

[x] If they use Filer#createResource, the location argument must be one of these values from StandardLocation: CLASS_OUTPUT, SOURCE_OUTPUT, or NATIVE_HEADER_OUTPUT. Any other argument will disable incremental processing.

I believe this annotation processor falls aggregatig category as it reaches to other elements like codecs or compiler registry. I followed the specific constraints :
"Aggregating" processors have the following limitations:

[x] They can only read CLASS or RUNTIME retention annotations
[x] They can only read parameter names if the user passes the -parameters compiler argument.

The changes looks to be minimal.
If possible it would be great to backport this change to the 5.3.x line.
Thank you in advance for the consideration.

Thanks for the PR @bric3
Unfortunately, Achilles processor cannot be incremental for several reasons:

It does not use the Filer API. Instead it is using com.squareup.javapoet.JavaFile to generate the source classes: https://github.com/doanduyhai/Achilles/blob/master/achilles-core/src/main/java/info/archinnov/achilles/internals/apt/processors/meta/AchillesProcessor.java#L118-L162

It does relies on internal Sun compiler classes to parse nested structures like Map<String, @Frozen MyUDT>: https://github.com/doanduyhai/Achilles/blob/b85a7f9b5e6415000787b833cdb3aa85c01ff660/achilles-core/src/main/java/info/archinnov/achilles/internals/parser/AnnotationTree.java#L274-L300

At that time, this was the only solution to parse nested annotations inside generic types. I don't know if since then things have improved on the front of Java annotation processor to avoid using internal classes. If so I'll be happy to refactor this part of the code to get rid of the Sun internal classes

Oh I missed those 😅.
I have seen other annotation processors using JavaPoet and activating Gradle's incremental processing. In preliminary tests this worked either in isolating and aggregating modes, but the model don't rely on UDT. Maybe I was mislead.

My bad. The code https://github.com/doanduyhai/Achilles/blob/master/achilles-core/src/main/java/info/archinnov/achilles/internals/apt/processors/meta/AchillesProcessor.java#L118-L162 still uses the Filer API in the end. It is just handled by com.squareup.javapoet.JavaFile class itself: https://github.com/square/javapoet/blob/8ebce31cd655cf181ebdde59dcf04c50cbda7264/src/main/java/com/squareup/javapoet/JavaFile.java#L164
However the 2nd issue still remains, if I stop using Sun javac internal class, it will break all support of nested annotations


Melee Attack Speed doesn't increase the attack rate for Molten Strike's balls

This can be seen with the Born to Fight node and with multistrike

Closing as duplicate


Add missing build job

Try to introduce an actual build job in GitHUb Actions workflow.
Optionally add --noEmit if you just want the build without files.
Fixes: https://github.com/fastify/demo/issues/40
Checklist

[x] run npm run test and npm run benchmark
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[x] commit message and code follows the Developer's Certification of Origin
and the Code of conduct

You could also introduce a production config for tsc and run: tsc --project tsconfig.production.json

Optionally add --noEmit if you just want the build without files.

I would prefer if we don't need to emit files.
Can we do everything in one job instead of installing dependencies twice for the same version? You can still rename it the job.

Sure. 1 sec.

Sure. 1 sec.

Done

I'm always surprised by the rate at which Node maintainers release major new releases...
Maybe we can stay in 23, I had used matrix at first for supporting several versions, but for a demo it might not make much sense.
We could leave it like fir a while.

Ok for v22 and v23?  @mcollina

I would add 20 as well to align with the fastify ecosystem

Agreed. v20 is added as well since this is Node.js v20 LTS. with updates and fixes until April 2026.

Is there a real value to run on Node 20? From a end-user perspective?

Is there a real value to run on Node 20? From a end-user perspective?

According to Rod Vagg himself: https://nodesource.com/blog/essential-steps-long-term-support-for-nodejs
And I will quote:

The point of establishing an LTS plan for Node is to build on top of an existing stable release cycle by delivering new new versions on a predictable schedule that have a clearly defined extended support lifecycle. While this may seem at odds with the open source tradition of “release early, release often” it is an essential requirement for enterprise application development and operations teams. It also affects companies like ours that provide professional support for Node.js (see our recent N|Support announcement).

For a demo this is not important I think, but ok.

Well at least you validate that your code is building correctly on Node LTS.


[Bug] Fix: CD workflow for production build

What happened 👀
There are bugs on the CD workflow for uploading production iOS builds to App Store Connect:

Having bitcode mode
Wrong bundle identifier (bug from iOS templates).

Insight 📝
First, I disable bitcode mode as it's not supported by Xcode 14. Second, I fixed the wrong bundle identifier
Proof Of Work 📹

Got approvals from this pull request: https://github.com/suho/kmm-ic/pull/144


Add CI testing as part of push to master branch and PRs. Avoid accidental bad code merges

Test PR with CI testing

Successful CI tests on branch push - https://github.com/yashs360/dice/actions/runs/9856408115/job/27213337521
Working on build on PR creation.

Tests getting triggered on PR. Showing in-progress/success status.

wip - test induced error to block merge on test failures

Tested failure scenario by purposeful error induced -

Ready for review @arpitbbhayani @JyotinderSingh

Thanks @yashs360!


"after about ten minutes my IP address was banned by Grooveshark"

Originally posted by @showcasefloyd at https://github.com/frosas/spotify-to-mp3/issues/5#issuecomment-9193739

The same issue here.
grooveshark (0.2.12)
spotify-to-mp3 (0.7.1)

Same issue here guys

My songs don't even download. I just get 'No data for this song. Maybe Grooveshark banned your IP.'

Trying again. Will post results.

Yup, happened again.

The following worked for me:

Don't try to download tons of songs at the same time, just download like 20 or so.
Repeat again.

Most of the times Grooveshark does not ban your IP.
Not the best solution, but got the job done

Okay, cool. I'll try that now.

@daniel-lobo Worked. Thank you +1

:+1:

btw for future reference
I split my entire playlist into 33 different text files, 20 songs in 32 files and the remaining 11 in a third.
Downloaded each list with wget -v
for i in `seq 1 33`; do wget -v http://<IP_ADDRESS>/spotify/lists/$i.txt; done
Then, ran a similar command to automate the downloading process.
for i in `seq 1 33`; do spotify-to-mp3 $i.txt; done

@cxdy how did you create each of the $i.txt files?

Manually. Just copied 20 tracks and pasted them into a new file. Probably the longest part of the process but I figured I'd end up spending more time figuring out how to automate it than it would have taken to paste the songs into the files.

Re-opening, probably permanently as it's hard (impossible?) to find a long-term solution to this.
Long ago I already tried the throttling approach but I never got clarifying results. In any case, I'm more than happy to you guys try it and share your experiences.
An easy way to do this is to throttle the tracks being piped to the app:
$ cat songs.txt | while read track; do echo $track; sleep 0.5; done | spotify-to-mp3

perhaps it would work to automate the process that @cxdy described (creating temp files that are split up), then automating the download of these smaller song lists.

When I have time later, I'll try to think of a way to automate the creation
of 30 text files with 20 lines of text in them.
On Sun, Apr 12, 2015, 11:15 PM Nelson Liu<EMAIL_ADDRESS>wrote:

perhaps it would work to automate the process that @cxdy
https://github.com/cxdy described (creating temp files that are split
up), then automating the download of these smaller song lists.
—
Reply to this email directly or view it on GitHub
https://github.com/frosas/spotify-to-mp3/issues/6#issuecomment-92181854.


[bug] Dark theme not working correctly on iOS

Describe the bug
Trying to setup the example app with dark theme on native iOS, but only some elements are updating their colors. Works fine on web.

Light Theme
Dark Theme

Light on Web
Dark on Web

To Reproduce
Repro repo: https://github.com/META-DREAMER/tamagui-app
System Info (please complete the following information):

  System:
    OS: macOS 12.6
    CPU: (10) x64 Apple M1 Max
    Memory: 64.89 MB / 64.00 GB
    Shell: 3.3.1 - /opt/homebrew/bin/fish
  Binaries:
    Node: 14.18.2 - /var/folders/r1/c3ctl7_s2dl2wld20w1mgj7m0000gn/T/fnm_multishells/64611_1666398493594/bin/node
    Yarn: 3.2.3 - /opt/homebrew/bin/yarn
    npm: 8.3.0 - /var/folders/r1/c3ctl7_s2dl2wld20w1mgj7m0000gn/T/fnm_multishells/64611_1666398493594/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Firefox: 103.0.2
    Safari: 16.0
  npmPackages:
    @babel/runtime: ^7.18.9 => 7.18.9
    @manypkg/cli: ^0.19.1 => 0.19.1
    @types/react: ^18.0.15 => 17.0.21
    @types/react-native: ^0.69.5 => 0.69.5
    eslint: ^8.21.0 => 8.21.0
    prettier: ^2.7.1 => 2.7.1
    turbo: ^1.4.2 => 1.4.2
    typescript: ^4.7.4 => 4.7.4

Tamagui info
check repro repo

@META-DREAMER   I have the same question. Can you manually change the simulator to dark mode and the app will use the dark theme.  But it will not run in dark mode automatically.

I was seeing a few problems due to the compiler that look like this sometimes, perhaps a recent beta caused. But the next release fixes all of it. It's very close, out this weekend.

@META-DREAMER try to specify a size - this was fixing it in my environment.

I was seeing a few problems due to the compiler that look like this sometimes, perhaps a recent beta caused. But the next release fixes all of it. It's very close, out this weekend.

any update on this? still not working as of beta.201

Fixed in beta 204 via 3d69c3dde8791d3c1b1346abc4916d69881508ec, sorry about that.


Typo in Elasticsearch config for Chrome extension

Had to remove the trailing slash from http.cors.allow-origin (matching the config submitted by @MrOrz over in https://github.com/appbaseio/dejavu/issues/48) in order for the Chrome extension to work.

Thanks for the contribution @chrisdfeld!


Added a licensing definition to the Maven project

In order to meet the prerequisites of plugin hosting in the Jenkins Organization, I'm adding the licensing definition in the Maven project.
@reviewbybees, What do you think?

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@reviewbybees done

I think it's safe to assume "MIT" in this case, but it needs +1 from the current maintainer. Not sure about the previous ones. @olivergondza ?

MIT, what else?

It has always been MIT since I joined the project.

More context:

https://github.com/jenkinsci/plugin-pom/pull/85
https://github.com/jenkinsci/plugin-pom/pull/85


JS- Function for grabbing typed info (and drop down bar)

Added drop down to html and added to js to grab typed input


mcu_task_avg and mcu_task_stddev are not reported

I see that when mcu metrics were added these two were left commented out, and are to this day. Is there a technical reason for this? I would like them to be available given their use to compute and graph the mcu load heuristic.

I don’t think that applies here; these metrics are effectively calculated by the mcu, and the variance from each stats report (at a 1s interval when printing) is used in a heuristic to estimate load. It might be possible to get this data over a longer interval with prometheus, but I’m not sure it would be possible to replicate an accurate load graph like that?

Fixed in release v0.12.0
Thanks for the PR!


Add state documentation

Add state documentation - please add details..

There is some documentation at https://github.com/lum-ai/odinson/wiki/Coding-the-State.


Rename the -sample module to -integration-tests.

Resolves #157

Codecov Report

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

@@            Coverage Diff            @@
##             master     #176   +/-   ##
=========================================
  Coverage     86.55%   86.55%
  Complexity      666      666
=========================================
  Files            80       80
  Lines          2067     2067
  Branches        107      107
=========================================
  Hits           1789     1789
  Misses          205      205
  Partials         73       73

Impacted Files
Coverage Δ
Complexity Δ

...ud/appbroker/integration/AppBrokerApplication.java
33.33% <ø> (ø)
1 <0> (?)

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 cec984a...775d39e. Read the comment docs.


Allow providing custom templates

This tool is very useful but it can be very useful to allow passing a path for custom scaffolding templates, and fallback to the default one if not set.

It would be a nice improvement! Do you want to work on this?


throttling out of date msgs

ensure we don't get spammed by incoming out of date messages
@keybase/react-hackers

:mag_right: No visual changes found as a result of these commits.

:+1:


Re-rendering failing with conda-build version 24.7.2

We are seeing issues re-rendering with this feedstock ( https://github.com/conda-forge/cuda-nvdisasm-feedstock/pull/11#issuecomment-2265437200 ). Also seeing in several other CTK feedstocks
It looks like logic like this is where re-rendering has issues
https://github.com/conda-forge/cuda-nvdisasm-feedstock/blob/659d7dfb1e9be90dbb56a84a519a2f94c8a6e5f4/recipe/meta.yaml#L4-L7

@conda-forge-admin , please re-render

Filed upstream issue: https://github.com/conda-forge/conda-smithy/issues/2011

cc @msarahan (for looking into this a bit more)

Reverting this commit fixes it: https://github.com/conda/conda-build/commit/87194f681b8c50a2502bdf5a60c5ffe8e08281ab
Haven't isolated exactly which part yet.

https://github.com/conda/conda-build/pull/5458

This was fixed in conda-build 24.9.0 with Michael's PR. Closing as resolved


Rename script folder.

Description
To  see better the separation between the code and the tools.

@ldez why ? build script are part of the code 😰

I agree with vdemeester, I am not sure this is needed or sensible...
I have things setup in my shell and editor that dotfiles are invisible... this adds an un-needed level of friction to a newcomer to the project who might wan't to find out how the build works

As this PR leads to debates, I suggest we close it.
The idea of @ldez was to put these build scripts somewhere else.
Using dotfiles for this may not be the best solution, but I still think it would be great to simplify the actual project tree:
acme
autogen
cluster
cmd
contrib
dist
docs
examples
healthcheck
integration
job
log
middlewares
provider
safe
script
server
static
templates
testhelpers
types
vendor
version
webui

Any idea is welcome!

Instead of trying to hide/move every non-code folder, we could also just put the Go code below a single directory. That way, everything else in the top-level directory is, by definition, not directly related to the source code.
I believe that many projects keep everything below /pkg. I may have sensed a vibe on Twitter for this being not advisable or typical for Go, though I don't have any hard facts supporting this concern.


mdadm: 4.2 -> 4.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/mdadm/versions.
meta.description for mdadm is: Programs for managing RAID arrays under Linux
meta.homepage for mdadm is: http://neil.brown.name/blog/mdadm
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 4.3 with grep in /nix/store/agp802698726i8yd5hamwr7xhi062232-mdadm-4.3
found 4.3 in filename of file in /nix/store/agp802698726i8yd5hamwr7xhi062232-mdadm-4.3

Rebuild report (if merged into master) (click to expand)

140 total rebuild path(s)

140 package rebuild(s)

First fifty rebuilds by attrpath
CuboCore.coreaction
CuboCore.corearchiver
CuboCore.corefm
CuboCore.coregarage
CuboCore.corehunt
CuboCore.coreimage
CuboCore.coreinfo
CuboCore.corekeyboard
CuboCore.corepad
CuboCore.corepaint
CuboCore.corepdf
CuboCore.corepins
CuboCore.corerenamer
CuboCore.coreshot
CuboCore.corestats
CuboCore.corestuff
CuboCore.coreterminal
CuboCore.coretime
CuboCore.coretoppings
CuboCore.coreuniverse
CuboCore.libcsys
adapta-gtk-theme
almanah
ayatana-indicator-datetime
booster
bubblemail
budgie.budgie-control-center
calls
cantata
chatty
cinnamon.cinnamon-common
cinnamon.cinnamon-control-center
cinnamon.cinnamon-gsettings-overrides
cinnamon.cinnamon-screensaver
cinnamon.nemo
cinnamon.nemo-fileroller
cinnamon.nemo-python
cinnamon.nemo-with-extensions
clementine
deepin.dde-device-formatter
deepin.dde-file-manager
deepin.dde-gsettings-schemas
deepin.deepin-album
deepin.deepin-compressor
deepin.deepin-music
deepin.udisks2-qt5
deepin.util-dfm
endeavour
enlightenment.enlightenment
eos-installer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/agp802698726i8yd5hamwr7xhi062232-mdadm-4.3 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A mdadm https://github.com/r-ryantm/nixpkgs/archive/994232c0341269b4eda21707b781ad6a0e28af96.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/994232c0341269b4eda21707b781ad6a0e28af96#mdadm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/agp802698726i8yd5hamwr7xhi062232-mdadm-4.3
ls -la /nix/store/agp802698726i8yd5hamwr7xhi062232-mdadm-4.3/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review took longer than 45m and timed out

Maintainer pings
cc @Ekleog for testing.

No regression according to nixpkgs-review, just merging to staging for safety.


Header text does not update when locale is changed

Hi, I am having some trouble with the locale property of the calendar. I have an app where users can change
the language and the selection is stored in the global state of the app. When I use this state as the value
of locale, the calendar header text doesn't get updated when the language is changed. I have been debugging it
and confirmed that the locale value gets updated, but the text doesn't change.
I have set the locale like this:
child: Consumer<AppStateModel>(
  builder: (context, appState, child) {
    return CalendarCarousel<Event>(
      locale: appState.locale.languageCode,
      //...other configurations
    );
  }
)

And the header text is not updated:

However if I just set the same appState.locale.languageCode straight as the headerText instead of locale it does get updated when changed:
headerText: appState.locale.languageCode,

So how could I get the header to update when just the locale is changed?

Version of flutter_calendar_carousel
1.5.3
Expected behavior
Calendar header should update when locale changes: e.g. locale changes from "en" to "fi" -> header changes from "Mar 2021" to "maalis 2021".
Actual behavior
Calendar header is not updated.
flutter doctor result (run flutter doctor in terminal)
[√] Flutter (Channel stable, 2.0.1, on Microsoft Windows [Version 10.0.18363.1379], locale en-FI)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 4.0)
[√] IntelliJ IDEA Community Edition (version 2020.2)
[√] Connected device (3 available)
• No issues found!
Steps to reproduce the behavior
Create a state for the CalendarCarousel locale and change its value.

Ah, forgot to leave a comment here. Anything about this?


Add Tisura zk-KYC app and codebase

This PR adds:

one reference to blog post section
one reference to projects section

I think the term zk-KYC is misleading. You can't actually have zkKYC, since KYC laws typically require that companies doing KYC checks are able to provide someone's personal identifiable information on request. Doing it with ZK makes this impossible. Perhaps a different name is warranted.

Yes you are correct. I understand the impetus behind the naming but am disappointed by the inaccuracy of the term. It is probably a losing battle to try change the convention at this point.


:seedling: Enforce make add-projects for GitHub and GitLab repos

What kind of change does this PR introduce?
CI change and corresponding documentation

[X] PR title follows the guidelines defined in our pull request documentation

What is the current behavior?
make add-projects is optional and it doesn't apply to GitLab repos
What is the new behavior (if this is a feature change)?**
make add-projects is required and it does apply to GitLab repos

[ ] Tests for the changes have been added (for bug fixes/features)

Which issue(s) this PR fixes
Related to #3764

Special notes for your reviewer
Does this PR introduce a user-facing change?
For user-facing changes, please add a concise, human-readable release note to
the release-note
(In particular, describe what changes users might need to make in their
application as a result of this pull request.)

NONE

Leaving as a draft for right this second, pending discussion from @raghavkaul about the gitlab project list


API key

Unfortunately Schoolmaster BV has decided to authorise all incoming API requests. That does not mean that this API does not work anymore. It does mean however that you need an API key in order to get access to the data.
I will take a look at this. I guess it will be necessary to have an API key now, so probably there will be an option to feed your magister API key and gain access again.

I really hope they make api-keys publicly Accessible. A lot of handy apps are made for magister that dont work anymore now.
What i saw is that Labda, a ios application for viewing magister agenda and grades, still works. Maybe they got a api-key from magister?

see also this old issue:  https://github.com/simplyGits/MagisterJS/issues/73

I mean, if it's possible to use the site from a browser (just using javascript) it shouldn't be too hard to get the API key one way or another, right?

(Probably) Solved! Magister 6 sends a request to https://accounts.magister.net/connect/authorize/callback. The response of this request is, amongst other things, a baerer authentication token. I guess that the token is the missing "api key". Since I had not the time to tinker with this, I have not made a working implementation yet. I will keep you updated.

Our API already has the option to provide an API key. I tested the package and everything seems to work fine for Magister version v6.1.29.
However, Magister, did change it's authentication procedure which may have implications for our package. I guess that could trigger the "API Key Invalid" response.
Does anyone get the "API Key Invalid" response for any version of Magister after version v6.1.11?


Changing existing int id to DatabaseGeneratedOption.None doesn't migrate properly

I know this is a difficult problem, and I'm sure this is way down on the to do list if it's there at all, but wanted to get it logged...
of course a property public int id { get; set; }  with no attributes will generate [id] INT IDENTITY(1,1) NOT NULL.
adding [DatabaseGenerated(DatabaseGeneratedOption.None)] to the property and generating a migration simply drops all the FKs in the DB and does an AlterColumn without the HasAnnotation.
no actual changes are applied to the database and no errors are generated during database update.

Ultimately this requires table rebuilds (tracked by another issue), but in the meantime we will warn during apply.

Closing as a dupe of #5345. We'll throw now telling you to rebuild the table. Automatically doing the rebuild is tracked by #329.


UntrustedTypesFoundException raised for standard library usage in lightgbm's Booster

Hello there,
after updating to 0.10.0 we got the following error:
...
File "/app/sales_prediction/common/result_storage.py", line 74, in get_train_result
return TrainResult(booster=skops.io.loads(model_data, trusted=["lightgbm.basic.Booster"]))
File "/opt/pysetup/.venv/lib/python3.10/site-packages/skops/io/_persist.py", line 191, in loads
audit_tree(tree)
File "/opt/pysetup/.venv/lib/python3.10/site-packages/skops/io/_audit.py", line 59, in audit_tree
raise UntrustedTypesFoundException(unsafe)
skops.io.exceptions.UntrustedTypesFoundException: Untrusted types found in the file: ['collections.defaultdict'].

In the serialised object we only have a single lightgbm.basic.Booster object.

Should the standard library be untrusted?
In this case the defaultdict is an internal details in the lightgbm's Booster. When the Booster is marked as trusted, should the internally used subdependencies still be untrusted?

Thank you for the quick reply, here is a minimal example with LGBM:
>>> import pandas as pd
>>> import lightgbm
>>> import skops.io

>>> booster = lightgbm.train(train_set=lightgbm.Dataset(pd.DataFrame({"results": [1]})), params={})
>>> serialised_data = skops.io.dumps(booster)
>>> skops.io.loads(serialised_data, trusted=["lightgbm.basic.Booster"])

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/unicsovicsm/.pyenv/versions/sales-prediction-services/lib/python3.10/site-packages/skops/io/_persist.py", line 191, in loads
    audit_tree(tree)
  File "/Users/unicsovicsm/.pyenv/versions/sales-prediction-services/lib/python3.10/site-packages/skops/io/_audit.py", line 59, in audit_tree
    raise UntrustedTypesFoundException(unsafe)
skops.io.exceptions.UntrustedTypesFoundException: Untrusted types found in the file: ['collections.defaultdict'].

For my understanding did I understand it correctly, that the trusted objects are recursively checked further for untrusted data?

Thanks for the reproducer.

For my understanding did I understand it correctly, that the trusted objects are recursively checked further for untrusted data?

Yes, trusting a "parent" type doesn't mean children included in that type are all trusted.

I encountered a bug(?) regarding defaultdict: the default value is not being saved (or loaded).
import collections
import skops.io as sio

class MyClass:
    def __init__(self):
        self.default_dict = collections.defaultdict(set)
        self.default_dict['key-0'] = set(['foo'])

my_obj = MyClass()
print('my_obj: ', my_obj.default_dict)
sio.dump(my_obj, 'dump.skops')

my_obj_loaded = sio.load('dump.skops', trusted=['__main__.MyClass', 'collections.defaultdict'])

print('my_obj_loaded: ', my_obj_loaded.default_dict)  # There is no default value
my_obj_loaded.default_dict['key-1']  # So this line raises "KeyError: 'key-1'"

The output of this code is
my_obj:  defaultdict(<class 'set'>, {'key-0': {'foo'}})
my_obj_loaded:  defaultdict(None, {'key-0': {'foo'}})
Traceback (most recent call last):
  File "/home/local/jmminkin/tmp/poc.py", line 18, in <module>
    my_obj_loaded.default_dict['key-1']  # So this line raises "KeyError: 'key-1'"
    ~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^
KeyError: 'key-1'

This occurs also when using also list, tuple, int or a lambda as the default value, not only when using set.
I used skops.io 0.10.0 on Python 3.12.4.

So one thing: I checked and this is not a regression really, with the latest lightgbm, skops=0.9 also fails with the same error. But I'll see what we can do here.

Version 0.9.0 worked if you used trusted as a boolean flag:
skops.io.loads(serialised_data, trusted=True)

But this is not possible since 0.10.0


Redesign CompositeAccessPoint

Toward 3.0.0 release, I'm thinking about redesigning CompositeAccessPoint family of classes. Perhaps  Internet Archive is the only user of this class at this moment, because of its inflexibility. OpenWayback may want to move it to external (IA-owned) project, I wonder.
If we want to keep it in OpenWayback, I'd like to address some pain points with the framework:

oracleUrl, staticExclusions: hard-coding specific combination of exclusion filters. already refactored to use CompositeAccessPoint.getExclusionFactory(). these deprecated members shall be dropped.
AccessPointConfigs is just a wrapper around a HashMap<String, AccessPointConfig>, and getAccessPointConfigs() exposes that HashMap object (not Map). This makes it hard to implement dynamic loading of configuration, say, from external database. Define an better interface with richer functionalities.
isProxyEnabled(): feels misplaced, because proxy/archival-url is not specific to CompositeAccessPoint. CompositeAccessPoint itself does not use this property.
AccessPointAdapter.getSwitchCollPath(): feels misplaced. it simply returns ProxyAccessPoint.SWITCH_COLLECTION_PATH.
AccessPointConfig.beanName: set through Spring's BeanNameAware interface, and used for storing collection identifier. It's unreliable because Spring sometimes modify bean name to ensure uniqueness. While it's nice to be able to write <bean name="collectionId" ..., it's only useful when you configure collections in Spring config. Rename the member, add setter/getter, and let setBeanName() update the member.
findConfigForFile(String) method sounds too specific to particular deployment (=IA). Remove it from OpenWayback.

I'll update this description, as I find more. Inputs are welcome.

One enhancement needed for them is a class level JavaDoc explaining the purpose of the classes.
Given how undocumented this part of OWB is, I'd be surprised if it's used outside IA.


Handle empty keywords

Fix the illegal argument and wrong help command issue for FindCommand and FindCommissionCommand.

Codecov Report

Merging #150 (426d47e) into master (1b37acc) will decrease coverage by 0.04%.
The diff coverage is 48.27%.

@@             Coverage Diff              @@
##             master     #150      +/-   ##
============================================
- Coverage     56.33%   56.28%   -0.05%
- Complexity      775      780       +5
============================================
  Files           143      143
  Lines          3309     3335      +26
  Branches        358      364       +6
============================================
+ Hits           1864     1877      +13
- Misses         1341     1350       +9
- Partials        104      108       +4

Impacted Files
Coverage Δ

...rc/main/java/seedu/address/logic/LogicManager.java
35.71% <0.00%> (-0.88%)
:arrow_down:

...ress/logic/parser/FindCommissionCommandParser.java
0.00% <0.00%> (ø)

.../java/seedu/address/model/commission/Deadline.java
66.66% <0.00%> (-8.34%)
:arrow_down:

...ain/java/seedu/address/ui/CommissionListPanel.java
0.00% <ø> (ø)

src/main/java/seedu/address/ui/MainWindow.java
0.00% <0.00%> (ø)

...ava/seedu/address/model/commission/Commission.java
74.79% <33.33%> (-1.04%)
:arrow_down:

...edu/address/model/commission/CompletionStatus.java
70.00% <42.85%> (-14.62%)
:arrow_down:

.../seedu/address/logic/parser/FindCommandParser.java
44.89% <75.00%> (+1.41%)
:arrow_up:

...rc/main/java/seedu/address/model/ModelManager.java
90.08% <85.71%> (-0.27%)
:arrow_down:

src/main/java/seedu/address/model/Model.java
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


command line version?

Is there some way that the treesub program could be run through command line without the gui?

If you are comfortable with the command line, you can run RAxML and PAML manually with the appropriate options for ancestral reconstruction. Then you can do this:

If you already have run PAML analysis, you can run the following to create substitutions.tree and substitutions.tsv files. You need to have run PAML with the appropriate options. For example, if the rst file is in /path/to/paml/results run:
java -cp dist/treesub.jar treesub.ancestral.ParseRST /path/to/paml/results


DIsplaying comments that don't fit on the screen

Single comments that are too large to fit in the screen will only be partially displayed. There is currently no way to view the hidden part of the comment, as scrolling down jumps all the way to the beginning of the next comment. Suggestions on how to solve this problem are welcome.

There could be a secondary key bind that allows the user to scroll through the post manually. The amount of scroll increases as they go down the post and resets if another post is reached.
Or we could have to completely separate key binds. One for scrolling post by post and the other for scrolling down line by line.

I like the second idea. Would be tricky to implement though, based on how I wrote the page scrolling.

I tinkered around with the code and got very basic scrolling using 'y' and 'u' as scroll up and down. Curses made this pretty basic:
# Setting scrolling support
self._content_window.scrollok(True)

# Scrolling the content
self._content_window.scroll(1)
self._content_window.refresh()

However since the content is only drawn to what is visible on the screen scrolling does not do much as of yet. I'll try tinkering with the drawing method when I get time.

In addition, posts that don't fit on the screen aren't displayed at all. For example, when nagivating to this post, we simply see (Not enough space to display), no comments or anything. I think it should show at least some of the content and the comments.

@Brobin Fixed in ca41b77

Closing this because it would require a major rewrite of how screen paging is currently implemented. If anybody has additional input or ideas about this, feel free to open a new issue.

Perhaps an option to open a post in vim? (read-only mode)

Actually this feature was added here https://github.com/michael-lazar/rtv/issues/93. It uses less instead of vim though.


#28 - Ensure _execute and _tryExecute Functions Correctly Handle Calls to EOAs

Updated _execute and _tryExecute functions to include an extcodesize check, ensuring calls to EOAs are not incorrectly marked as successful. This change aligns the behavior of low-level calls with Solidity's expectations

ensuring calls to EOAs are not incorrectly marked as successful.

I notice in case of eoa you're just doing this
mstore(result, 0) // Set result length to 0
mstore(0x40, add(result, 0x20)) // Update free memory pointer
but the other code following can still be executed.
Can you bring auditers attention to this comment again please?
a. Assume all calls to EOA invalid / unsuccessfull
b. sucessfull if it's just value transfer > 0 and calldata is 0x
Ensure it returns in such cases and does not go to other control paths.
Write appropriate tests for it

@Aboudjem

inclined to close this.


Update npm package

Hi,
I've seen that someone changed React.PropTypes to the prop-types pacakge but you forgot to update the npm package. Could you please update it?
Thank you! And awesome component!

This is now fixed!


Fixed the tuple label error

Resolves the build errors:
swift-corelibs-libdispatch/src/swift/Wrapper.swift:94:17: error: tuple element cannot have two labels
                                  handler: (_ error: Int32) -> Void) {

This change is

You're reverting #126 here which is my understanding was to adapt for SE-0111. Update your compiler.

Oh ok, sorry about that. I was on Swift 7/29, assumed that this change was helping with that.


Fix typos

Fix a couple of typos in Readme.md

Thanks for that. Have merged your PR

Thanks for that. Have merged your PR

Thank you. This was my first PR ever. I am working on another one. I am not sure when I'll be able to finish it. I'm using Single Sign On (SSO) to sign into AEM. I've added that option, maybe others can use it too.


Player: option to change the default point color

We should have a parameter default_point_color that accepts a tuple(R, G, B) to style the point colors, because selecting a white background makes the default light grey too light.

Solved in 22592ffab04f615be79e50bd3d5dcc368ddb1d77


Reindex Site/List - How to check status

It would be great if this document included some information on how to tell when a reindex is completed.

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

ID: d18cdf74-1e86-aa79-3282-69ffc9cdd282
Version Independent ID: 39cea6c4-be38-1908-7815-aa9ee552cedf
Content: Manually request crawling and reindexing of a site, a library or a list - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/crawl-site-content.md
Service: sharepoint-online
GitHub Login: @SerdarSoysal
Microsoft Alias: serdars

Hello @fowl2 James,
I think that is not currently available.
Thanks,

@scanum

I am writing this with a stern yet empathetic tone *
This needs to be reopened in my opinion. There needs to be some form of status checking and the SharePoint community has been asking for years.
Legitimately all that I, and so many others, want is something like an email or a notification that says "Site Crawl in Progress" and then a "Site Crawl Completed."
That is it.
We just want any type of confirmation that our actions are being processed, or, at the very least, queued.

I keep just seeing posts pointing to other posts and issues.
Can we have a real reason why this isn't possible? It has been an issue since before 2018.  Where can I request this where it will be seen?


fully functional offline analysis

pr to merge into master once this is done :D

stuff left to do:

How to move the infline
Component eval
ROI Manager like Mesmerize
More Viz functions


zfsbootmenu-ui.sh: fix typo in fallback message

Please fix the typo in the commit message:
fsbootmenu-ui.sh -> zfsbootmenu-ui.sh

The irony.
Fixed.

Not the issue title, but the commit message.

Done.

Thanks!