pilot failed from Fastfile, but success directly from command line

Question Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Is there something wrong with my pilot setup?
lane :build do
     gym(
         workspace: "XXXxxx.xcworkspace",
         configuration: "Release",
         include_bitcode: false,
         scheme: "Production",
         silent: true,
         clean: true,
         output_directory: "build_appstore",
         output_name: "my-app.ipa",
         export_method: "app-store",
         export_team_id: "1234abcd",
         codesigning_identity: "Apple Distribution: XXXX (xxxxxx)"
     )
 end

 lane :distribute_to_testflight do
     build
     pilot(
         username<EMAIL_ADDRESS>         app_identifier: "xxx.xxx.xxx.xx",
         dev_portal_team_id: "1234abcd",
         team_name: "xxxxx",
         team_id: "123456"
     )
 end

Question Description
I got this error message if I run fastlane distribute_to_testflight
ERROR ITMS-90034: "Missing or invalid signature. The bundle 'xxx.xxx.xxx.xxx' at bundle path 'Payload/XXX.app' is not signed using an Apple submission certificate."
Return status of iTunes Transporter was 1: ERROR ITMS-90034: "Missing or invalid signature. The bundle 'xxx.xxx.xxx.xxx' at bundle path 'Payload/XXX.app' is not signed using an Apple submission certificate."
The call to the iTMSTransporter completed with a non-zero exit status: 1. This indicates a failure.

but it will upload successfully to testflight if I run fastlane build cd to the .ipa folder and then fastlane pilot upload
still can't figured it out since yesterday. any ideas, are there something wrong with my pilot in Fastfile?
sorry for my bad English

solved:
Change Apple Distribution: xxxxx (xxxx) to iOS Distribution
thanks


Mute button

In fortnite mute button act as a master mute on the controller. I even went further and explore it on sound setting with and without headphones and it doesn't show incoming input when activated.

Thats because Windows doesnt support this. And i dont that will be added in the future

Thats because Windows doesnt support this. And i dont that will be added in the future

When the mute button is lighted up the mic isn't muted by the controller. The controller stills hear everything.

Also is there a function where I can check if the mute button is being pressed.

just pushed <IP_ADDRESS> which has a option to mute the microphone with the library. it also toggles the led accordingly.
therse is also a new variable micBtn in state which indicates if the button was pressed

As I'm reading your response I also made a new variable micBtn in the state which indicates if the button was pressed and also made the ability to mute and unmute the current using microphone. My only issue is that turning on the led takes a full second.

You take your time learning how to use a dualsense for Python. It helps to know people like you out there, and I appreciate it.


Add support for ENSIP-10

[ ] ENSIP-10 [^1]
[ ] Requires EIP3668 [^2] support

Relevant announcement: https://medium.com/the-ethereum-name-service/upgrade-ethers-js-to-5-6-1-to-activate-ens-l2-offchain-integration-40ee1a0fdf2a
I found the post from etherjs quite useful: https://blog.ricmoo.com/highlights-ethers-js-march-2022-f511fe1e88a1
Sample data
offchainexample.eth -> 0xd8dA6BF26964aF9D7eEd9e03E53415D37aA96045
1.offchainexample.eth -> 0xb8c2C29ee19D8307cb7255e1Cd9CbDE883A267d5 //This relies on wildcard resolution support
2.offchainexample.eth -> 0xb8c2C29ee19D8307cb7255e1Cd9CbDE883A267d5 too //This relies on wildcard resolution support
ricmoose.hatch.eth -> error
hweeboon.eth -> 0xbbce83173d5c1D122AE64856b4Af0D5AE07Fa362

ENSIP-10
Key part of ENSIP-10 spec starts at:

If a resolver implements this function, it MUST return true when supportsInterface() is called on it with the interface's ID, 0x9061b923.

Pre-ENSIP-10:
resolver = ensContract.resolver(originalName)
address = resolver.addr(originalName)

With ENSIP-10 wildcard resolution:
name = originalName
while name.notEmpty
    resolver = ensContract.resolver(name)
    if resolver != 0x
        break //found it
    end
    name = parent(name) //`1.offchainexample.eth` -> `offchainexample.eth`
end
if resolver.supportsInterface("0x9061b923")
    dnsEncodedName = dnsEncode(originalName)
    calldata = addr's //as if calling resolver.addr()
    address = resolver.resolve(dnsEncodedName, calldata)
else
    address = resolver.addr(originalName)
end

The resolver.resolve(bytes,bytes) call is the one that might trigger EIP3668 with our test data.
EIP3668
This seems to add EIP3668 support to web3j:
The diagram at https://eips.ethereum.org/EIPS/eip-3668#overview describes the flow well. But basically, if a transaction call reverts with OffchainLookup(address,string[],bytes,bytes4,bytes) then we need to trigger EIP3668 support by:

parsing the OffchainLookup error
HTTP GET/POST against the gateway URL(s) specified in the OffchainLookup error
call the contract with the function specified with data from the gateway response + data extracted from the OffchainLookup error
result from (3) replaces the original smart contract call that triggered EIP3668

Step 3 might trigger EIP3668 again.
Pre-EIP3668 support:
func web3j.call(contract, func)
    return contract.func
end

With EIP3668 support:
func web3j.call(contract, func)
    result = contract.func
    if result == OffchainLookup
        //Trigger EIP3668
        urls, sender, callBack, callData, extraData = extract(offchainLookup)
        gatewayResult = httpGetOrPost(urls, sender, callData)
        return contract.callback(gatewayResult, extraData) //This could trigger EIP3668 again
    else
        return result
    end
end

I found etherjs's implementation great for understanding how to implement EIP3668. Look for the implementation of ccipReadFetch and where it is called: https://github.com/ethers-io/ethers.js/blob/bc400c7b87b03438abb2f9874ec136dbf7b6437c/packages/providers/src.ts/base-provider.ts#L854
iOS PRs for reference

AlphaWallet iOS for ENSIP-10: https://github.com/AlphaWallet/alpha-wallet-ios/pull/4237
web3Swift support EIP3668: https://github.com/AlphaWallet/web3swift/pull/10

[^1]: ENSIP-10 https://docs.ens.domains/ens-improvement-proposals/ensip-10-wildcard-resolution
[^2]: EIP3668 https://eips.ethereum.org/EIPS/eip-3668#use-of-get-and-post-requests-for-the-gateway-interface

@JamesSmartCell I have updated the first comment because the addresses for these 2 aren't 0xb8c anymore:
1.offchainexample.eth -> 0xb8c2C29ee19D8307cb7255e1Cd9CbDE883A267d5 //This relies on wildcard resolution support
2.offchainexample.eth -> 0xb8c2C29ee19D8307cb7255e1Cd9CbDE883A267d5 too //This relies on wildcard resolution support


[BUG]

I can't run him properly

The arrangements are not over yet.

The arrangements are not over yet.

So when can it be used properly?Do you have an email address?I think it's easier to get in touch

will be up and running soon.
email<EMAIL_ADDRESS>


[help] how to make the slice image in x, y, z view to fit the render window

I want the slice image to be full of the render window.
current:

Hopeful:

i spend two day to look into vtk.js source code, and find the zoom interface. but how to set the zoom factor? some suggestion is welcome, thanks

I think it is the parallelScale on the camera since we use the parallel projection but that's just on top of my head.

@jourdain thanks, i have got it down.
paste the code if someone else need.
feature:

fit all 2D view
auto adjust by slice image ratio and container ratio

function Fill2DView(view) {
  const viewName = view.getName();
  if (viewName === 'default') return;

  const bounds = view.getRenderer().computeVisiblePropBounds();
  const dim = [
    (bounds[1] - bounds[0]) / 2,
    (bounds[3] - bounds[2]) / 2,
    (bounds[5] - bounds[4]) / 2,
  ];
  const w = view.getContainer().clientWidth;
  const h = view.getContainer().clientHeight;
  const r = w / h;

  let x;
  let y;
  if (viewName === 'x') {
    x = dim[1];
    y = dim[2];
  } else if (viewName === 'y') {
    x = dim[0];
    y = dim[2];
  } else if (viewName === 'z') {
    x = dim[0];
    y = dim[1];
  }
  if (r >= x / y) {
    // use width
    view.getCamera().setParallelScale(y + 1);
  } else {
    // use height
    view.getCamera().setParallelScale(x / r + 1);
  }
  view.renderLater();
}

Thanks this should probably go in the View2D proxy without the viewName but based on the camera orientation.
@floryst we should try to integrate it at some point.
Thanks for your feedback @chenkaiC4

Adapted @chenkaiC4 his code for VTK.js ^11.1.0:
function Fill2DView(view, viewName) {
  // Based this code: https://github.com/Kitware/paraview-glance/issues/230#issuecomment-445779222
  const bounds = view.getRenderer().computeVisiblePropBounds();
  const dim = [
    (bounds[1] - bounds[0]) / 2,
    (bounds[3] - bounds[2]) / 2,
    (bounds[5] - bounds[4]) / 2,
  ];
  const w = view.getContainer().clientWidth;
  const h = view.getContainer().clientHeight;
  const r = w / h;

  let x;
  let y;
  if (viewName === 'x') {
    x = dim[1];
    y = dim[2];
  } else if (viewName === 'y') {
    x = dim[0];
    y = dim[2];
  } else if (viewName === 'z') {
    x = dim[0];
    y = dim[1];
  }
  if (r >= x / y) {
    // use width
    view.getRenderer().getActiveCamera().setParallelScale(y + 1);
  } else {
    // use height
    view.getRenderer().getActiveCamera().setParallelScale(x / r + 1);
  }
  view.resize();
}

Thanks @EmileSonneveld! I forgot about getting this integrated; it's back on my list now.


Average Rating Calculation Only Works for Zero to Two Different Ratings

Try: 1 + 2 + 4 + 5 = 12 & 12 / 4 = 3
In class ReviewCreate(generics.CreateAPIView):
...
if watchlist.number_rating == 0:
watchlist.avg_rating = serializer.validated_data['rating']
else:
watchlist.avg_rating = (watchlist.avg_rating + serializer.validated_data['rating'])/2
    watchlist.number_rating = watchlist.number_rating + 1

...
Should be:
    if watchlist.number_rating == 0:
        watchlist.avg_rating = serializer.validated_data['rating']
        watchlist.number_of_rating = watchlist.number_rating + 1
    else:
        watchlist.avg_rating = (watchlist.avg_rating * watchlist.number_rating + serializer.validated_data['rating'])
        watchlist.number_rating = watchlist.number_rating + 1
        watchlist.avg_rating = watchlist.avg_rating / watchlist.number_rating

Should be:
if watchlist.number_rating == 0:
watchlist.avg_rating = serializer.validated_data['rating']
watchlist.number_rating = watchlist.number_rating + 1
else:
watchlist.avg_rating = (watchlist.avg_rating * watchlist.number_rating + serializer.validated_data['rating'])
watchlist.number_rating = watchlist.number_rating + 1
watchlist.avg_rating = watchlist.avg_rating / watchlist.number_rating

I used the name watchlist.number_of_rating but didn't get them all changed back before posting.

Hi RLR-NDH,
Honestly,
The calculation part was included to give an idea of how we can add these features.
Calculation is not right since if someone deletes their review, we are not updating the average currently, I discuss about the same thing later in the course. Each company currently calculates differently, I took simple calculations to explain the topic.
For example,

some company store rating for last 90 days and average them with every single new rating,
some companies that deals with lot of ratings, they do it with ratings of last 7 days only,
most common is to calculate average of recent 20 ratings.

The feature is just for demo purpose and later can be improved according to self requirement.


Add Support for VAT rate as decimal value to support new base VAT rate in Finland

Finland is changing the base VAT from 24% to 25,5% in september
https://www.roschier.com/newsroom/finnish-vat-rate-to-increase-to-25-5-starting-september-2024
the .net sdk only support int as VATpercentage. The new paytrail api support VATpercentage as float 0-100 with . as decimaldivider

Thanks for reporting, this will be updated asap

@obetsak This was updated with latest release 1.0.14


refactor(transformer): reduce cloning and referencing Rcs

#3576  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @overlookmotel and the rest of your teammates on  Graphite


log in error

Hi there,
I get this error after attempting to log in via terminal:
login error: tok = AuthManager Service not responding.
AuthManager Service not responding.

I logged into astro without a problem with the same credentials. Is this an issue a few users are experiencing?

Hi Joanna, thanks for reaching out. We have now switched to the new noirlab.edu domain (the old noao.edu domain expired end of November). This required an update to the datalab client. Please install the latest version (v2.20.0) via:
pip install --ignore-installed --no-cache-dir noaodatalab
and also rename or delete your local $HOME/.datalab/dl.conf file (it will be re-created the next time you run a datalab command).
Let us know if you still experience problems.
Best,
Robert

Closing this due to no activity. Please submit a new issue if the problem reoccurs.


[iOS] Update template icons

Updates the AppIcon and LaunchImage assets in the iOS app template to meet Apple's icon guidelines as of iOS. Assets were moved to the flutter_template_images package, which includes them as of version 4.2.0.
Ref: https://developer.apple.com/design/human-interface-guidelines/foundations/app-icons#specifications
Update the test on creating iOS templated icons
Validate that at least one of the iOS templated icons gets downloaded and put in the right spot.
Issue: #106110
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@jmagman Sorry to bother you again. 😅
This commit should be based on the latest repository, but the automated check still has errors.
Warning: one or more pubspecs have invalid dependencies. Please run "flutter update-packages --force-upgrade" to update them correctly.

I ran this command, and no files were changed.
$ flutter update-packages --force-upgrade

Upgrading packages...
WARNING: the following packages use hard-coded version constraints:
  - archive: 3.1.11
  - flutter_gallery_assets: 1.0.2
  - flutter_template_images: 4.0.0
  - shelf: 1.1.4
  - video_player: 2.2.11
Running "flutter pub upgrade" in flutter_update_packages.Nz9N8F...         4.9s
Ran pub get in dev/integration_tests/web in 0.5s...
Ran pub get in dev/tools/vitool in 0.8s...
Ran pub get in dev/integration_tests/web_compile_tests in 0.4s...
Ran pub get in dev/tools/gen_keycodes in 1.1s...
Ran pub get in dev/tools/gen_defaults in 1.1s...
Ran pub get in dev/tools in 1.2s...
Ran pub get in dev/integration_tests/ios_platform_view_tests in 1.2s...
Ran pub get in dev/integration_tests/deferred_components_test in 1.3s...
Ran pub get in dev/bots in 1.3s...
Ran pub get in dev/integration_tests/ui in 1.3s...
Ran pub get in dev/integration_tests/release_smoke_test in 0.7s...
Ran pub get in dev/integration_tests/android_views in 1.6s...
Ran pub get in dev/integration_tests/ios_app_with_extensions in 0.6s...
Ran pub get in dev/integration_tests/ios_add2app_life_cycle/flutterapp in 0.6s...
Ran pub get in dev/integration_tests/android_embedding_v2_smoke_test in 0.8s...
Ran pub get in dev/integration_tests/gradle_deprecated_settings in 0.6s...
Ran pub get in dev/integration_tests/non_nullable in 0.6s...
Ran pub get in dev/integration_tests/flavors in 1.1s...
Ran pub get in dev/integration_tests/abstract_method_smoke_test in 0.4s...
Ran pub get in dev/integration_tests/external_ui in 1.2s...
Ran pub get in dev/integration_tests/android_semantics_testing in 1.2s...
Ran pub get in dev/integration_tests/web_e2e_tests in 1.3s...
Ran pub get in dev/manual_tests in 0.6s...
Ran pub get in dev/integration_tests/platform_interaction in 1.1s...
Ran pub get in dev/integration_tests/hybrid_android_views in 1.3s...
Ran pub get in dev/forbidden_from_release_tests in 0.4s...
Ran pub get in dev/tracing_tests in 0.6s...
Ran pub get in dev/integration_tests/channels in 1.1s...
Ran pub get in dev/integration_tests/flutter_gallery in 1.7s...
Ran pub get in dev/customer_testing in 1.2s...
Ran pub get in dev/benchmarks/multiple_flutters/module in 1.0s...
Ran pub get in dev/devicelab in 1.5s...
Ran pub get in dev/benchmarks/complex_layout in 1.4s...
Ran pub get in dev/benchmarks/test_apps/stocks in 1.6s...
Ran pub get in examples/splash in 0.7s...
Ran pub get in dev/benchmarks/platform_views_layout_hybrid_composition in 1.6s...
Ran pub get in dev/benchmarks/platform_views_layout in 1.6s...
Ran pub get in dev/benchmarks/macrobenchmarks in 1.8s...
Ran pub get in dev/benchmarks/microbenchmarks in 1.7s...
Ran pub get in dev/benchmarks/platform_channels_benchmarks in 1.7s...
Ran pub get in examples/platform_view in 0.4s...
Ran pub get in dev/conductor/core in 1.4s...
Ran pub get in examples/flutter_view in 0.4s...
Ran pub get in dev/automated_tests in 1.5s...
Ran pub get in examples/layers in 0.7s...
Ran pub get in examples/image_list in 0.7s...
Ran pub get in packages/integration_test/integration_test_macos in 0.4s...
Ran pub get in examples/hello_world in 1.4s...
Ran pub get in packages/flutter_web_plugins in 0.7s...
Ran pub get in packages/integration_test in 0.7s...
Ran pub get in examples/platform_channel_swift in 1.3s...
Ran pub get in packages/flutter_goldens_client in 0.4s...
Ran pub get in examples/platform_channel in 1.2s...
Ran pub get in packages/flutter_goldens in 0.8s...
Ran pub get in packages/flutter_tools/test/data/asset_test/main in 0.2s...
Ran pub get in packages/flutter/test_private in 0.5s...
Ran pub get in packages/flutter_test/test/test_config/project_root in 0.1s...
Ran pub get in packages/flutter/test_private/test in 0.8s...
Ran pub get in examples/api in 1.3s...
Ran pub get in packages/integration_test/example in 1.3s...
Ran pub get in packages/flutter in 0.7s...
Ran pub get in packages/flutter_localizations in 0.5s...
Ran pub get in packages/flutter_test in 0.4s...
Ran pub get in packages/fuchsia_remote_debug_protocol in 0.7s...
Ran pub get in packages/flutter_driver in 0.7s...
Downloaded lcov data for package:flutter in 0.0s...
Ran pub get in packages/flutter_tools in 1.9s...
Running "flutter pub get" in affected packages...                   7.7s

Ran 'pub get' 66 times and fetched coverage data in 7.7s.

A ha! This is caused by https://github.com/flutter/packages/pull/1951.
This logic is checking if package.packageUriRoot directory exists, which in this case is the flutter_template_images-4.2.0/lib directory, which no longer exists.
https://github.com/flutter/flutter/blob/ed3c32c9d2c27b6259b7226063317d07b85f2134/packages/flutter_tools/lib/src/flutter_cache.dart#L101
Because it doesn't exist, on every run it thinks the pub dependencies are out of date, and pub needs to be run.
@stuartmorgan I see from that PR it says

Removes the empty Dart file, which used to be required by pub.dev checks but no longer is.

I'm not sure what feature this is referencing, is the tool logic correct?

Filed https://github.com/flutter/flutter/issues/107957 to track this.  Thanks for your patience, please hold 🙂

Removes the empty Dart file, which used to be required by pub.dev checks but no longer is.

I'm not sure what feature this is referencing

It used to be that if you tried to pub publish a package that didn't contain any Dart code, it would error out, so we had packages/plugins (like this one, which exists only to hold images) that had a dummy Dart file in lib/. That restriction was lifted at some point, so I've been cleaning up those dummy files as I find them.

is the tool logic correct?

I'm not sure what that tool logic does exactly, but it shouldn't be assuming that all packages have lib directories.

I'm not sure what that tool logic does exactly, but it shouldn't be assuming that all packages have lib directories.

This is the logic:
https://github.com/flutter/flutter/blob/ed3c32c9d2c27b6259b7226063317d07b85f2134/packages/flutter_tools/lib/src/flutter_cache.dart#L100-L104
package.packageUriRoot is lib in this case.

Need some better way to check this then, I don't see anything else interesting in that package object.

Sorry, I meant I don't know what that logic is for. I.e., I don't know what "lib/ exists" is intended to be a proxy for, and thus what to replace it with. Is there a reason it's checking for something in the package, not the folder itself? Is there a reason it's not checking for pubspec.yaml instead, or was lib/ arbitrary content?

Sorry, I meant I don't know what that logic is for. I.e., I don't know what "lib/ exists" is intended to be a proxy for, and thus what to replace it with. Is there a reason it's checking for something in the package, not the folder itself? Is there a reason it's not checking for pubspec.yaml instead, or was lib/ arbitrary content? Someone who knows the answers to those questions should decide how to fix it :)

Someone who knows the answers to those questions should decide how to fix it :)

Yup, wasn't saying you should fix it, the tool team is already on it. Was just asking for more context since you knew more than I did about how these packages should be formatted. 🙂

Thank you for your patience! Can you try rebasing/merging this on https://github.com/flutter/flutter/pull/107968?
And did you want to track this work in https://github.com/flutter/flutter/pull/106111 or this PR?

Thank you for your patience! Can you try rebasing/merging this on #107968? And did you want to track this work in #106111 or this PR?

I seem to have made a mistake, so many commits..

I seem to have made a mistake, so many commits..

Looks correct now, you figured it out. 🙂

I seem to have made a mistake, so many commits..

Looks correct now, you figured it out. 🙂

Oops, there are still errors.

So flutter update-packages --force-upgrade will need to be run on your PR.  That command hasn't been run in a bit, and there are some updates it generates unrelated to your change.  I created https://github.com/flutter/flutter/pull/108178 which will run it on master, then you can rebase again when that lands.  Then you can run flutter update-packages --force-upgrade on your change, and the only diff should be the CHECKSUM in packages/flutter_tools/pubspec.yaml:
-# PUBSPEC CHECKSUM: b869
+# PUBSPEC CHECKSUM: 5d6a

Apologies that you've hit so many rough edges helping us with this change.  Thanks again, we're almost there!

So flutter update-packages --force-upgrade will need to be run on your PR. That command hasn't been run in a bit, and there are some updates it generates unrelated to your change. I created #108178 which will run it on master, then you can rebase again when that lands. Then you can run flutter update-packages --force-upgrade on your change, and the only diff should be the CHECKSUM in packages/flutter_tools/pubspec.yaml:
-# PUBSPEC CHECKSUM: b869
+# PUBSPEC CHECKSUM: 5d6a

Apologies that you've hit so many rough edges helping us with this change. Thanks again, we're almost there!

I'm delighted to contribute to Flutter!
Do I need to wait for https://github.com/flutter/flutter/pull/108198 to be merged?

Do I need to wait for https://github.com/flutter/flutter/pull/108198 to be merged?

Yes, once it's in master then merge or rebase to get that change.

Do I need to wait for https://github.com/flutter/flutter/pull/108198 to be merged?

It's merged.

The strange thing is that after I run flutter update-packages --force-upgrade, nothing changes.

Looks like it's passing now, marking ready for review.

Looks like it's passing now, marking ready for review.

CHECKSUM was changed manually by me because I ran the command without effect.

CHECKSUM was changed manually by me because I ran the command without effect.

@jmagman When I updated my own copy of flutter, I became aware of this: by running flutter update-packages —force-upgrade, I was updating the flutter code that I set the path to, so the copy I cloned did not change. 🤦♂

CHECKSUM was changed manually by me because I ran the command without effect.

@jmagman When I updated my own copy of flutter, I became aware of this: by running flutter update-packages —force-upgrade, I was updating the flutter code that I set the path to, so the copy I cloned did not change. 🤦♂

That would do it!


Trash compacter

Removed all ONS dependencies from the pom.xml. Migrated all required code from the ons libraries to their own quarantine folder. Imports changed to match the new java paths.
Given versions to the dependencies - they match current versions, nothing has been upgraded. Some transitive dependencies, used in the apis, have been migrated.
To test it - i have enabled emails in my environment to ensure that messages are being picked up from the queue.

what xsds am i missing?

All xsds in the resources folder


Cannot log into LeetCode extension anymore, help me please!!

when I tried to log in leetcode extension by using cookie or  third-party account, it always occur these below error.
Is there anybody could help me to solve this log problem?
please help me
`TypeError:` Cannot read properties of null (reading 'isCurrentUserPremium')
    at c:\Users\Fan Zhang\.vscode\extensions\leetcode.vscode-leetcode-0.18.1\node_modules\vsc-leetcode-cli\lib\plugins\leetcode.js:530:27
    at Request._callback (c:\Users\Fan Zhang\.vscode\extensions\leetcode.vscode-leetcode-0.18.1\node_modules\vsc-leetcode-cli\lib\plugins\leetcode.js:436:12)
    at self.callback (c:\Users\Fan Zhang\.vscode\extensions\leetcode.vscode-leetcode-0.18.1\node_modules\request\request.js:185:22)
    at Request.emit (node:events:519:28)
    at Request.<anonymous> (c:\Users\Fan Zhang\.vscode\extensions\leetcode.vscode-leetcode-0.18.1\node_modules\request\request.js:1161:10)
    at Request.emit (node:events:519:28)
    at IncomingMessage.<anonymous> (c:\Users\Fan Zhang\.vscode\extensions\leetcode.vscode-leetcode-0.18.1\node_modules\request\request.js:1083:12)
    at Object.onceWrapper (node:events:633:28)
    at IncomingMessage.emit (node:events:531:35)
    at endReadableNT (node:internal/streams/readable:1696:12)

Node.js v21.7.0

哥，切换到 leetcode-cn 👀  #605

It doesn't work. It sends to the Chinese login page of Leetcode which does not even have Google sign in of course. So logging in is not possible for me as I sign in with a Google account.


Add hints to addCluster and updateCluster

BTW, we should set a FAQ list and some issues should be closed.


Clarify document sql-database-elastic-query-getting-started.md

Database-scoped master key and credentials

This is not clear about:
 CREATE MASTER KEY ENCRYPTION BY PASSWORD = '<password>';

 CREATE DATABASE SCOPED CREDENTIAL ElasticDBQueryCred
 WITH IDENTITY = '<username>',
 SECRET = '<password>';

"username" and "password" should be the same as login information used in step 6 of ?? I could not find step 6 in anywhere? And password of master key and database scoped are the same?
Then, how could Execute this query on the ElasticDBQuery database on SSMS? which connection to access and execute this?

@alo360 Thank you for the detailed feedback.
This is being assigned to the content author to evaluate and update as appropriate.

@MladjoA The reference to step 6 in Download and run the sample app seems to be invalid.

@KalyanChanumolu-MSFT , I am not owning nor familiar with this article.

I was referring to the link for Step 6 from @KalyanChanumolu-MSFT's comment

@anumjs Could you please check this issue.
Looks like the document was modified to remove Step 6

@KalyanChanumolu-MSFT when I modified the linked doc, I only see steps till 5. Could you please email me the link to the version that contains step 6 so I can figure out what the issue is. I don't own the original doc so I can't tell right away. It could just be a typo as the username and password step is 2.

@MladjoA - Updated you to be the assignee for this issue because you are listed as the current document owner in the article metadata. Could you please work to address the feedback and close the issue? See these instructions.
@KalyanChanumolu-MSFT @anumjs - Suggested fix is to simply remove the words "step 6 of."
""username" and "password" should be the same as login information used in Download and run the sample app in Getting started with elastic database tools."

@KalyanChanumolu-MSFT , @v-thepet
I just signed off PR that should address all issues reported.

@KalyanChanumolu-MSFT , @v-thepet , can you please take a look? I believe we have full clarity in the article now.

#please-close


Revert "Bump aquasec/kube-bench from 0.3.1 to 0.5.0 in /plugins/kube-bench"

Reverts FairwindsOps/insights-plugins#365

Dependabot tricks me every time!


Auto-sync mechanic breaks when a fleeting note has a '/' in the title

Summary:
The auto-sync mechanic in the app is currently encountering an issue where it breaks when a fleeting note contains a '/' character in the title. This character is being interpreted as part of the filepath, causing the auto-sync to malfunction. To prevent this issue, a warning message should be added in the app when attempting to add this character in the note title.
Background/Reason:
Currently, when a fleeting note includes a '/' character in the title, the auto-sync mechanic in the app fails to function properly. The '/' character is mistakenly considered as part of the filepath, leading to incorrect interpretation and subsequent malfunctioning of the auto-sync feature.
To address this problem, it is recommended to implement a warning message within the app. This warning message should be displayed whenever a user tries to add a '/' character in the note title. By providing a clear warning, users will be alerted about the potential issue and can avoid using this character, preventing any disruption in the auto-sync functionality.

Summarized feedback from<EMAIL_ADDRESS>:

Auto-sync mechanic breaks when a fleeting note has a '/' in the title, interpreting it as part of the filepath. Request to add a warning message in the app when trying to add this character in the note title.

#785


fix: CXL host exerciser  segmentation fault non-root user mode

-CXL host exerciser segmentation fault occurs in non-root user mode,
exit application if clx device open fails.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.003%) to 65.191%

Totals

Change from base Build<PHONE_NUMBER>:
-0.003%

Covered Lines:
15756

Relevant Lines:
24169

💛  - Coveralls


mbtools initial run

Hi Jared and Thanks for this tools.
I am trying to clone and run it but it seems like I am missing something or that something is missing in the repo.
There is no make or bash file and simply running " mbtools reference-frequency input.bam > output.tsv"
as in the README will not do. the rm file is not excusable.
Any suggestions?
Assaf

Apologies, I put some information here: https://github.com/jts/mbtools/tree/main#compiling

Thanks Jared,
Now its working.
However, I understood that it should work with ONT modified bams (MM ML tags). But it seems that these modifications are not recognized by mbtool, running:  mbtools read-region-frequency -r '39Bisulfite.tsv'  -g '/home/access/ref_genomes/hg38.fa'   --cpg 'my_modified_bam.bam'
results in "Processed 1564146 reads in 44.714603338s. Mean modification frequency: NaN"
Am I missing something?
Is there a way to turn ONT modified bam into a standard modified bam?
Assaf

Hi @assafgrw,
ONT's bams are standard mod bams so should work. Can you share a few lines or part of the bam file with me?
Jared

Hi Jared,
I am sharing here the first 100 lines of a sample bam file. I suspect that
there is a difference in how the modification data is stored in bam files
generated by ONT tools.
One more place I and others were facing this problem is with wgbs tool
(generating an atlas as in loyfer et al 2023):
https://github.com/nloyfer/wgbs_tools/issues/48.
Anyway, I needed mbtools to generate input for nanomix, for now I have
overcome it with modkit but still it is an interesting issue and I guess
that solving it is needed for other tools in the future.
Assaf
On Wed, Sep 13, 2023 at 4:29 PM Jared Simpson @.***>
wrote:

Hi @assafgrw https://github.com/assafgrw,
ONT's bams are standard mod bams so should work. Can you share a few lines
or part of the bam file with me?
Jared
—
Reply to this email directly, view it on GitHub
https://github.com/jts/mbtools/issues/5#issuecomment-1717639239, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHP7JFEDH47SMTYNBYI7X7LX2GYLFANCNFSM6AAAAAA4SC3KTE
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry I realise that the other bam file was bad.
On Thu, Sep 14, 2023 at 9:27 AM Assaf Grunwald @.***> wrote:

On Thu, Sep 14, 2023 at 8:53 AM Assaf Grunwald @.***> wrote:

Hi Jared,
I am sharing here the first 100 lines of a sample bam file. I suspect
that there is a difference in how the modification data is stored in bam
files generated by ONT tools.
One more place I and others were facing this problem is with wgbs tool
(generating an atlas as in loyfer et al 2023):
https://github.com/nloyfer/wgbs_tools/issues/48.
Anyway, I needed mbtools to generate input for nanomix, for now I have
overcome it with modkit but still it is an interesting issue and I guess
that solving it is needed for other tools in the future.
Assaf
On Wed, Sep 13, 2023 at 4:29 PM Jared Simpson @.***>
wrote:

Hi @assafgrw https://github.com/assafgrw,
ONT's bams are standard mod bams so should work. Can you share a few
lines or part of the bam file with me?
Jared
—
Reply to this email directly, view it on GitHub
https://github.com/jts/mbtools/issues/5#issuecomment-1717639239, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AHP7JFEDH47SMTYNBYI7X7LX2GYLFANCNFSM6AAAAAA4SC3KTE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Crashes attempting to parse Pods/Pods.xcodeproj/project.pbxproj

while doing 'rnpm link NODE_MODULE_NAME' program crashes attempting to parse ios/Pods/Pods.xcodeproj/project.pbxproj   an xml file written by pod install
src/config/ios/findProject.js  findProject  gets a projects array
[ 'ios/Pods/Pods.xcodeproj',
'ios/Scandy.xcodeproj',
'ios/ScandyNative.xcodeproj' ]
[ 'ios/SCDOne.xcodeproj' ]
and goes with projects[0]
if all my debugging didn't mess things up, then the parser throws an syntax error uncaught when
executing
const didLinkIOS = registerDependencyIOS(dependency.config.ios, project.ios

Hey,
Check out similar issue we had -> https://github.com/rnpm/rnpm/issues/111#issuecomment-202561638
BTW - shouldnt it select ScandyNative for instance?

some more info
$ pod --version
0.39.0
Main project is ios/Scandy.xcodeproj.  Wish I could tell more about the logic of the structure, but is something I am walking into, not something I set up on day 1.

Fixed in #128, thank you!

Verifying that this no longer crashes on me.  Thanks.
Having other trouble, will generate new issue in the unlikely event that I'm not the source of the other trouble.

Haha, ok! In case it's our fault - feel free to post an issue, some of the things are still hypothetical solutions we haven't covered fully yet :)


Update egui to the latest version (0.29.1).

Hi, I stumbled across this project while I was prototyping with an egui project using the latest version of egui (0.29.1) when I noticed the crate needed some version bumping to function with the current version of egui so here we go.
It seems to work and the tests pass.

Thank you

hmm, looks like it is broken on Windows

fixed it, but the emoji selection doesn't look that good.
i guess we can ignore this for now


Language and character sets for names?

Spun off from #188, so it doesn't distract from the search for other documentation.
On Wed, Feb 18, 2015 at 09:16:16AM -0800, Greg Wilson wrote:

Use local names (e.g., Uniwersytet w Białymstoku) for locations
rather than anglicized names (e.g., University of Bialystok), but
stick to something close to the Latin alphabet (no Cyrillic or
kanji).

Local names makes sense to me, but I'm not clear on why we should avoid the local character set (e.g. Cyrillic, kanji, etc.).  It makes sense to me to just go completely native.  I wouldn't restrict this to locations either.  I don't see why people can't use their native character set for their personal names, or for site names, etc.  Python 3 won't have a problem with the Unicode data, and it's easy for users to install a free font (like DejaVu or Unifont) if they are missing the non-Latin glyphs.
Previous discussion on amy@:
On Tue, Feb 17, 2015 at 10:08:27PM -0800, W. Trevor King wrote:

On Tue, Feb 17, 2015 at 13:40:55 +0000, Giacomo Peru wrote:

Should the full name of a site in English or in the local language
(University of Bialystok or Uniwersytet w Białymstoku)? I guess in
English.

Heh, I'd have guessed in the local language :p.  I think we do want
Event.language using ISO 639 alpha-2 or alpha-3 codes 1.  If/when we
have multi-language workshops, we'll want separate language model
(maybe django-languages-plus?) and a many-to-many table
connecting the Event and language tables.

So:

Do we have a reason to prefer almost-Latin characters?
Do we (or will we) have any multi-language workshops?  This touches on swcarpentry/workshop-template#160, where we could use both "en" and "ase" codes to mark a workshop as speaking both English and American Sign Language (e.g. if there was an ASL interpreter or the instructors and helpers spoke ASL and English).
Do we want to list language-skills for the instructors?  I think we should, which would make it easier to do things like find Portuguese-speaking instructors for a Portuguese-language event.  I wouldn't worry about scoring fluency, just let instructors list the languages they'd feel comfortable teaching in.

On 2015-02-19 12:03 AM, W. Trevor King wrote:

Local names makes sense to me, but I'm not clear on why we should
avoid the local character set (e.g. Cyrillic, kanji, etc.).
Because almost every scientist can read things written in the Latin
alphabet (or some close to it), but most of us can't read other alphabets.

On Thu, Feb 19, 2015 at 02:35:12AM -0800, Greg Wilson wrote:

2015-02-19 12:03 AM, W. Trevor King:

Local names makes sense to me, but I'm not clear on why we should
avoid the local character set (e.g. Cyrillic, kanji, etc.).

Because almost every scientist can read things written in the Latin
alphabet (or some close to it), but most of us can't read other
alphabets.

If the workshop isn't in a language you speak, do you need to read the
name?  For the odd times that you do need a pronounceable name, you
can always paste the native form into a translation service.  I'm also
fine restricting the slug to ASCII, and having non-native-readers
refer to the event by its slug.

Slug in ASCII + title on Software Carpentry website in extended Latin ==
workable compromise?

On Thu, Feb 19, 2015 at 09:34:28AM -0800, Greg Wilson wrote:

Slug in ASCII + title on Software Carpentry website in extended
Latin == workable compromise?

“title on Software Carpentry website in extended Latin” sounds like
your starting position ;).  I'd rather “Slug in ASCII + title on
Software Carpentry website in the event-native language and charset”.
I don't think we need to enforce this policy in amy though, I'm happy
leaving the title charset up to the admins and convention.  That would
let us experiment with both approaches and see which was more popular.

At my university we have a special resolution (you can run it through Google Translator if you want) that says "in English you should call this university: AGH University of Science and Technology (abbreviation: AGH UST)".
In Poland we do keep English names when setting up workshop repositories. I think that, at least in Poland, we don't have problems with using only English names of universities for two reasons:

English is our second most-common language -- virtually everyone knows at least a little bit of it.

We know that Software Carpentry is US (or Canada) based and we don't expect anyone from the Foundation (maybe except for @apawlik) to be able to read and write Polish.

I would not force you to use Polish name of my university ("Akademia Górniczo-Hutnicza im. Stanisława Staszica w Krakowie") and I know people from other Polish universities feel similar. I encourage using English names.

I'm making a call on this one for Version 1: Latin-like names (i.e., Latin characters, possibly accented) for site names, in whatever language the host chooses (so either Uniwersytet w Białymstoku or University of Bialystok).  Ditto for personal names and everything else, and we'll revisit after CSV upload is working and this is in production.


Socks4/5 Proxy Support?

It would be nice to have built in Socks4/5 proxy support. I may be able to add it in the near future. Is this within the current scope of this project?

It might be.  I don't really know enough about SOCKS -- thankfully I have only NAT to worry about in my environment :-)
I presume you just want to implement a socks client capability?  Would you imagine that this is a new URL specification, or would have this come from the environment?


[Feature] Expand scope to include Youtube TV applications

This project looks amazing!
I was testing this project and was happy with how it worked for casting however I was quite confused as to why it didn't work for Youtube TV apps like isponsorblocktv. Is it possible to expand this project's scope to cover both casting and YouTube TV Apps?
In the meantime, I am running both projects but am concerned they will clash.
The other problem I ran into was Advert muting doesn't work on my TV. CastSponsorSkip attempts to mute the TV however a grey alert bubble on the TV says something along the lines of using the remote to control the TV volume. I suspect this is due to setting the volume to 0 rather than using "Mute". My TV CEC implementation only provides volume increase, decrease and mute and will complain if I try to use the volume slider within the Chromecast remote app however the dedicated mute button functions as expected. Functionally equivalent but ever so slightly different. This functionality does work with isponsorblocktv.
Thanks for all the hard work and I hope this makes sense!

@kmaid Do you mean YouTube's TV streaming offering, or the YouTube app on a TV? Because it works with the latter, but you'll need to generate a YouTube API key. I use it with an Nvidia Shield and it was broken for the past couple of weeks due to a YouTube bug, but it seems to be fixed now!

I mean the Youtube TV App. I have generated a Youtube APII key for the other project. I will retry everything shortly

#71 has been been resolved by the latest YouTube app, so it should work for you again!

it has not resolved it for me. I checked for updates on the Chromecast YouTube app and none were available. I stopped ISponsorBlock and recreated CastSponsorSkip.
2024/03/01 20:15:43 INFO Video ID not set. Searching YouTube for video ID... device="Living Room TV"
2024/03/01 20:15:44 INFO Detected video stream. device="Living Room TV" video_id=Jt1teDSV0Y4
2024/03/01 20:15:44 INFO No segments found for video. device="Living Room TV" video_id=Jt1teDSV0Y4

It detected a video playing but did not skip any of the advertisements on the youtube app.
Casting video works as before. Ads are skipped however muting adverts fails like the photo below

"No segments found for video" means SponsorBlock doesn't know of any sponsored segments. I checked the video ID in your logs and can confirm there are no segments available.

Hmm the mute issue is an interesting one. I hadn't thought about HDMI-CEC devices that don't support volume. For some reason, I thought the Chromecast could still mute. I'm not sure if there's anything I can do here, since the only available mute function obviously just shows a toast. I'll let you know if I think of anything!

"No segments found for video" means SponsorBlock doesn't know of any sponsored segments. I checked the video ID in your logs and can confirm there are no segments available.

Any idea why I get always no segments fouund for video?
I am running the YouTube app on the Chromecast TV 4K and it never managed to skip a single add. I have added they key, but no change.
This is an example from a few minutes ago:
2024/04/22 20:09:45 INFO Detected video stream. device="Living Room TV" video_id=ZKnf4HfGGZc
2024/04/22 20:09:47 INFO No segments found for video. device="Living Room TV" video_id=ZKnf4HfGGZc


update to ember-cli 2.10.0

@slindberg @workmanw I'm thinking we should release after this. @slindberg when would you be available to go through the release process?

I'm not sure what to do about the beta fails with regards to https://github.com/lytics/ember-data-model-fragments/issues/224

Hey @jakesjews -- I believe I have things squared away for ember-data-beta (2.11.x). Unfortunately, there will be a non-backwards compatible change and we'll have to bump our incompatibility.
For that reason I'd like to do a few things here.

I'd like to go ahead and merge this.
Cut a new tag release after that for a patch update (v2.3.3).
After that, we'll get my PR submitted / reviewed / merged. Then we'll bump this version to 2.11.

Does that all make sense and sound right?

@workmanw yeah that sounds good. Thanks for taking the lead on the fixes!


Use 'uuid' for SpecIF identifiers

So far, we have a home-grown pattern for SpecIF identifiers. Consider to use 'uuid' as defined in JSON Schema ("A Universally Unique Identifier as defined by RFC 4122.")
Currently there is no potential issue when using ReqIF identifiers in SpecIF and vice-versa. Check whether this is still the case when SpecIF uses 'uuid'.

Not considered for schema v1.2.


How i installed it on Ubuntu 20.04 with Emacs 27.091 (default make install from source)

[Installation]
Linux & Windows
I have no idea. Please let me know!

ICON="$HOME/.local/share/icons/doom.png"
DESKTOP_FILE=/usr/local/share/applications/emacs.desktop
wget https://raw.githubusercontent.com/eccentric-j/doom-icon/master/doom.png -O "$ICON" &&
sudo --preserve-env=ICON,DESKTOP_FILE sed -i "s|Icon=.*|Icon=$ICON|" $DESKTOP_FILE

And restart Emacs.
Not sure if there's a better way, though...
Loving it :)
Get the vanilla icon back with:
ICON="$HOME/.local/share/icons/doom.png"
DESKTOP_FILE=/usr/local/share/applications/emacs.desktop
rm $ICON
sudo --preserve-env=ICON,DESKTOP_FILE sed -i "s|Icon=.*|Icon=emacs|" $DESKTOP_FILE

Thanks very much! I added your snippet to the README with proper credit.


Merge crossroads changes

This integrates the changes from the crossroads fork's rebased-with-upstream branch, except for the switch to AMS 0.10.

Rebased on master to get .travis.yml fixes.


Static Site Generators

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #35670

@RandellDawson sir please review this

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


fix default value for --database.old-system-collections

Scope & Purpose
The announced default value for the new --database.old-system-collections options is false, but the original PR set it to true in the code unintentionally. This PR rectifies that.

[x] :hankey: Bugfix

Backports:

[x] No backports required

Testing & Verification

[x] This change is a trivial rework / code cleanup without any test coverage.

http://<IP_ADDRESS>:8080/view/PR/job/arangodb-matrix-pr/12115/

Tests blue.


Implement Merge Sort Algorithm

Algorithm:
Merge Sort
DS:
Sorting
Languages Supported:

C
C++
Java
Javascript
Python

Can I work on this for languages C++, Java, and Python?

@UtkarshChaurasia You already have a lot of Issues under your belt. You can complete the rest of the Issues and then I can assign this to you.

@HarshCasper Could you assign this issue to me , I can work on this for C , C++ language ?

@HarshCasper  can you assign this issue to me I can work on this for java language?

@HarshCasper Could you assign this issue to me? I can work on this issue in C++.

I have assigned you all the Issue. You can start working on it.
Please follow the general guidelines:

Make a Branch off your Master and work on that.
Make a proper README of how your Script works and what all libraries and modules you have used.
Add Proper Comments and focus on Clean Code, in practice.
Also reference this Issue in your PR so that it becomes easy for us to review.

@HarshCasper  i have raised a pull request is anything missing in it .
I am asking this because it hasnot been merged yet.

@HarshCasper can you assign this issue to me ,I will code in python

Someone already submitted it for Python :(

Can I implement this algorithm in Golang.

Can I take it up in Java?


When running on the simulator, 'net/route.h' file  not found

When running on the simulator, 'net/route.h' file  not found...
What should I do?

@wushengdubai #31, it can help you.

Hi guys, I fixed the issue.


Buggy

Plugin is a little bit buggy, freezes the server sometimes, I'll try to fix this soon.

i dont have bugs


Adding Powershell snippets to Five Minutes Tutorial

Description
Adding Powershell snippets. FWIW PowerShell is a kind of bash in MS Windows.
I've checked these commands on main devSlim.
They are little bit raw now. Also, I don't know how to represent tabs in *.adoc.

you know, if we got the tabs working, this actually would be nice to add..  At first I thought having Powershell implied a lot of change and maintenance burden...   But this looks pretty straightforward.   And it's not like these commands and therefore the work to keep updated in ref guide are changing often....

BTW, I think this was a nice idea, and maybe in the future we'll add it if we see more folks using powershell.   I'm also thinking that in the future as our CLI maybe gets richer, we may find that it's possible to use the CLI for all the thigns int he tutorial, and not need language specific examples...


Dispatch

I need to check the changes locally. Let me know when you're done.

I need to check the changes locally. Let me know when you're done.

This will take several cycles of iteration. Let me know what you  think of the proposed organization and route changes.

I need to check the changes locally. Let me know when you're done.

This will take several cycles of iteration. Let me know what you think of the proposed organization and route changes.

This looks good to me. I can continue the rest issues in this PR itself if you want, And I will make the changes in those related routes and services. What do you think?


release version 0.4.2

This switches the default transit model to exoplanet-core, to get around the compiler issues that folks seem to keep having installing batman-package on Windows machines. Hopefully this'll help make the installation smoother for more people!

Eep, maybe this is actually 0.4.3?


Redundancy in route

As I am developing enterprise level application I feel that I need to define routes two times
in header section and module route section
can we improve this????
just throwing an idea. :)

@sinedied any comment on this?

If you want you can iterate from the route definitions to display the links in the header, but you won't gain much and you will introduce tight coupling.
As the route may be defined on lazy-loaded modules and because you may not always want to show a header link when a new module is added, I think that we cannot do fundamentally better than the current solution using loose coupling.

Closing this as @110692 thumbed-up the answer.


IncrementStyleFixer - handle after ")"

Fixes https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/4657

ping @gharlan do you see any risk doing this?


Failed Travis build on long lasting COPR build

Travis treats a build as stalled (and fails it) after 10 minutes of no new output. For long lasting COPR builds it is not enough.
There is a workaround which could be applied. The better approach to that use cases is planned in copr-cli.

The Lazarus package has been rebuilt in Rawhide (and updated to 1.7.0), but as I got informed on the test mailing list there are some ongoing issues with 'compose' to update the packages in the DNF repo.

In theory, it should be possible to add the Koji repo [1] into the buildroot configuration ...

The dependency problem was solved long time ago, but the failing (due to timeout) builds are more  often.


stdlib.gfx.rumble()

In src/libs/stdlib/gfx.py, add a rumble function which rumbles the area at a specified rate with a specified intensity. This can be accomplished easily using viewport scripting.

Added in a7bb9b8fe57817a96db730fe5c4ab56063155cc1.


Scaling for control.impulse_response discrete time is not correct

Tested on control 0.9.2 and again on 0.9.4
The resulting impulse response (unit sample response) for the following transfer function (accumulator scaled by 0.1) is not properly scaled:
$$H(z) = \frac{Tz}{z-1}$$
With the sample duration set to $T=0.1$ is being scaled again by $1/T$ such that the result is a step with every output sample set to 1. This is not correct as the result should be 0.1 for every output sample.
To further clarify below is the implementation diagram:

If we pass a unit sample into this, it would scale by $T$ to be $0.1$ per the transfer function and block diagram shown and then the output would go to $0.1$ and then repeat for every subsequent sample.  I confirmed with Octave that the result is as I expect. Further if I use con.step_response on the same transfer function, the result is also what I expect for that case (the output increments by 0.1).
Code to repeat problem:
import control as con
dt=0.1
accum = con.tf([dt, 0], [1, -1], dt)
t1, y1 = con.impulse_response(accum)     # doesn't scale properly
t2, y2 = con.step_response(accum)           # scales properly

I do see in the docs "For continuous time systems, the initial condition is altered to account for the initial impulse. For discrete-time systems, the impulse is sized so that it has unit area", but would argue that it should not be rescaled especially if Octave and MATLAB are not doing that (I haven't confirmed MATLAB) and should represent the output of the discrete time system described by $H(z)$ alone without modification when a unit sample is applied to the input.

There is some discussion of the current approach in #812.

Ah thank you! I was trying to find if this was already posted. I see now from a friend that MATLAB also scales it this way.
Regarding your comment on scaling, yes it totally makes sense in the mapping of an ODE from continuous time IF we were doing that. If I have a discrete time system then there is no further mapping and any connection to some continuous time system should already be in the scaling for that function.  My example of an accumulator was exactly that: I included the $T=0.1$ in the transfer function as the "dt" you would see in an integrator (the accumulator is a discrete time integrator or mapping of the continuous time integral to discrete time). Similarly to do the dx/dt for differentiation, you scale by $1/T$. So if I had a continuous time function and wanted to differentiate it in discrete time, then yes I would multiply by $1/T$ as is done. And we note that an impulse response is the integration of the step response. However if I have an arbitrary discrete time system and want it to match some continuous time system at a specific update rate, then the scaling mentioned should already be included in the transfer function. Case in point is the step response: I already included the $T=0.1$ and the step increased by $0.1$ in each step. The function didn't multiply by $0.1$ as would be required to properly represent an accumulator increasing at period $T=0.1$.

(Should I be writing this under https://github.com/python-control/python-control/pull/812 and not here?)

#812 is merged, so better to keep the comments here.

Hi @cdboschen
This is typically tied to the common problem of defining the dirac function as a standalone unit sample function however that is not the definition of a unit impulse function. Hence, there is no right or wrong, it's just the definition. The discrete version is the Kronecker delta and both have only mathematical meaning under an integral or summation sign because they are technically generalized functions (or tempered distributions and so on depending on who you are hanging out with). They are not functions of time. And they should have the property
$$
\sum_{j}\delta_{ij} a_{j} = a_i
$$
Because an impulse does not carry the value of 1 throughout the whole sample but when summed up (again only under summation sign) equals to unity mimicking the dirac delta
$$
\int _{-\infty }^{\infty }f(x),\delta (dx)=f(0)
$$
That's why it is not a proper function of $\mathbb{z}$, similar to the dirac delta. What you want is the unit sample function response and can be done by supplying [1, 0, 0, 0...] which is exactly what it is.

Regarding your comment on scaling, yes it totally makes sense in the mapping of an ODE from continuous time IF we were doing that. If I have a discrete time system then there is no further mapping and any connection to some continuous time system should already be in the scaling for that function.

By mentioning that there is a continuous time system, we don't mean that we always start from a continuous one and derive all discrete-time models from. What I meant over #812, is that there is an "equivalence-class" between such systems that you can argue about an impulse response of all such systems which should result in the same behavior.

And we note that an impulse response is the integration of the step response.

I think you meant the other way around.
In control software generally you don't need to include $T$ in discrete transfer functions. The software takes care of it by the dt keyword. So if you want a discrete backwards euler integrator tf(1, [1, 0], dt=dt) is sufficient and it will arrange things for you (I haven't checked it but should). Hence you don't need to add it yourself or if you want to model everything then you should keep track of $T$ everywhere.
Note that if this is an integrator, then either it is your interpretation since you already have the information that it is linked to continuous time integrator or it is an information that you have beforehand. However suppose someone hands you a Tustin approximation of it $\frac{T}{2} \frac{z+1}{z-1}$ and did not tell you that, then you would not know if it is an integrator or something else. Hence we have to be a bit careful about talking about continuous equivalences and behaviors and etc.
I am not suggesting that your argument is wrong or inaccurate but I am trying to portray how the convention is typically constructed.

Thank you @ilayn and I understand your points (and yes, you are correct, I meant the other way around).
I'm thinking of applications where I am simulating a discrete time system, and for a step and "impulse" response, I really want the result to be for the unit sample function (so the unit sample response specifically), and for my model to match the specific results I would get without "hidden" gain modifications (hence my examples). An "impulse response" does not really exist in discrete time so I would have expected the unit sample response from a con.impulse_response function.
To your point about "interpretation"; I am using the discrete time models as they are without concern of a mapping or equivalence to a continuous time model: my interpretation of the Tustin approximation of an integrator if that was handed to me as $H(z)$ is simply an accumulator with a gain factor followed by a two sample moving average, so to me the gain modifications beyond what I write in the transfer function are a nuisance. (Going back to if I want the true unit sample response for the block diagram as it is, what would the result be?). So my concern isn't how it was mapped but more to say that I'm not concerned with any equivalence to continuous time-- I want the results to be exactly what my system produces exactly at each dt. (These are models of discrete time hardware or software implementations where the gain values are obvious and precise - so would be easy for me to write $H(z)$ with those gains rather than have something derive it for me from dt. I just wanted that part of my concern to be clear.
So, that's my two cents, but I also get that there must be value in having the dt automatically scale when using the discrete time transfer functions to model continuous time systems. I can get to what I need without too much grief by simply always using dt=1 for my applications and scaling the time axis myself, and perhaps that is the best solution. (And I see that if it was changed to what I really want, then the inclusion of dt is trivial since it would only serve to scale the t that the function returns and have no other effect).
I really appreciate you taking the time to respond to my comments and helping me understand the broader perspective.

(Should I be writing this under #812 and not here?)

#812 is merged, so better to keep the comments here.

Ok understood I deleted my comment there to clean up.


Improve ci setup

Closes #
📑 Description

I saw that you had to update the PNPM version within the workflow file in a recent commit, which is no longer required if you specify PNPM in the package.json
Doing this change I took the liberty to update the CI from node 18 to 20 and remove the custom cache setup, which should already be handled by the setup-node task.
Status

[ ] Not Completed
[x] Completed

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] I have checked the page with https://validator.unl.edu/
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

Can you fix this error?

 Can you fix this error?

I pushed the new change, specifying the exact version of PNPM to use within the package.json. Everybody who's using corepack will now also use the exact same version to ensure the build is reproducible in the CI and locally.


[BUG] click on parts of the drop-down doesn't work

Description
should be the whole thing rather than just the name and the icon opening the dropdown, the problem should be coming from here:
https://github.com/Dun-sin/Code-Magic/blob/fd16c0625b9a83d59b4a1a2dcad45bfe24fb76a7/src/main.ts#L434-L455
Screenshots
https://github.com/Dun-sin/Code-Magic/assets/78784850/76027e73-3c5f-46eb-ba8a-149260e89da2
Additional information
No response
🥦 Browser
Mozilla Firefox
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue?
None

I would like to work on this!

I would like to work on this!

Definitely, thanks for wanting to contribute, Make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed. Good Luck

Facing this issue when adding subject using the git cz command. The issue is fixed, just facing the problem here.

 Facing this issue when adding subject using the git cz command. The issue is fixed, just facing the problem here. What can I do now?

You should read the instructions cz is giving you😂

-42 characters🫠

-42 characters🫠

😂You were writing your back story and it wasn't even the right place

didn't get it, please help😶🌫

oh, got it got it😬

Please check and let me know if anything needs to be corrected.

Please check and let me know if anything needs to be corrected.

keep in mind in the future you don't need to comment back on the Issue, it's not my first rodeo, so I see your PR and will definitely do the necessary. It will just cause unnecessary notifications for me and others.

:tada: This issue has been resolved in version 2.66.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Looking for automation QA

I am looking for automation QA with Cypress or Selenium to join the project.

Hello! I am interested in participating in the project in questions related to tests and automation with Cypress

Hello, that is good. I think you can make e2e tests for login and register. If you need, I will create you an issue.


Revert "Fix WelsTraceCallback conversion. (#3722)" (#3739)

This reverts commit 28b533af6c79e545b17533a4f04f6a5cda65e81c.

Same as https://github.com/cisco/openh264/pull/3740.
This was already done in https://github.com/cisco/openh264/pull/3738.


Adding Ansible and Migration entitlements

Set to true like all others entitlements

@ryelo Can you review or assign someone please?

@kruai Can you take a look?

Looks good to me. Before merging, though, I wanna quickly check with our resident Entitlements expert @tahmidefaz just to be safe :)


Merge release notes from dev

Closing because needed change for documentation without logos changes are merged into release/24.2 on PR-828.


Add Homebrew Release Recipe for Closing the Archive Ecosystem Loop

Description
Since we use Homebrew heavily for environment setup, it's time to close the loop and allow any product built from this system to be usable on Homebrew itself. Let's support it.
Spec: https://docs.brew.sh/Formula-Cookbook
Blog: https://devopstales.github.io/linux/brew-repo/
Expected Behavior
Release job now release HomeBrew to a designated tap (a GitHub Homebrew-[REPO])
Current Behavior
HomeBrew is not supported.
Associated Data Files
No response

Implemented.


(Question) drone_id_test.grc failed to output file

Very interesting work and write-up. I'd love to test against a Mavic 3, however, after installing w/ GR 3.8 and attempting to run w/ a B205mini I get the following when running the drone_id_test.
FFT SIZE: 1024, sample rate: 1.536e+07
Traceback (most recent call last):
  File "/home/dragon/dji_droneid/gnuradio/drone_id_demod_test.py", line 350, in <module>
    main()
  File "/home/dragon/dji_droneid/gnuradio/drone_id_demod_test.py", line 328, in main
    tb = top_block_cls()
  File "/home/dragon/dji_droneid/gnuradio/drone_id_demod_test.py", line 218, in __init__
    self.droneid_demodulation_0 = droneid.demodulation(samp_rate / decimation)
  File "/usr/local/lib/python3/dist-packages/droneid/droneid_swig.py", line 845, in make
    return _droneid_swig.demodulation_make(sample_rate)
RuntimeError: Failed to open output file

I suspect it's something to do with the note about 1024 FFT and maybe my use of the 205. Although the failed to open output file has me confused. I'm not super familiar with how the droneid_swig.py is generated besides what I quickly researched after reading the top of the file saying that it's automatically generated.
The correlation_test flow graph has no problems running.

Thanks for the ticket!  The issue you're seeing is that I added debugging logic to the various modules that outputs files to /tmp/bursts.  If that directory doesn't exist it'll throw an exception.  An example of that is https://github.com/proto17/dji_droneid/blob/gr-droneid/gnuradio/gr-droneid/lib/demodulation_impl.cc#L88.  You can either create that directory or comment out the following lines:
https://github.com/proto17/dji_droneid/blob/gr-droneid/gnuradio/gr-droneid/lib/demodulation_impl.cc#L88
https://github.com/proto17/dji_droneid/blob/gr-droneid/gnuradio/gr-droneid/lib/demodulation_impl.cc#L162
If you choose not to comment out the lines be aware that you might run out of disk space if you leave the graph running for hours and hours.  The files are very small and what's getting written is just the burst itself for debugging purposes (I throw them into MATLAB and plot things)
Just to reiterate, this is a WIP and I might end up breaking things here and there.  Feel free to post tickets like this to bring it to my attention :)
Oh, and the code has only been tested against the DJI Mini 2.  I know that some drones only send 8 OFDM symbols while my drone sends 9.  Mine sends one additional symbol at the beginning, but that symbol zeros out after the scrambler is removed, so it's not actually used.  The logic was updated to not look at the first symbol for frequency offset correction.

Thank you, works great now. I'll close the ticket and try this out on as soon as I can. There's an aerial photographer person around here, I think they've got a Mini 2 as well as the Mavic 3. Have to see if they'll let me experiment again! This was an easy test on DragonOS as it already had everything needed to run.

Sorry one other question, based on the flow graph defaults for sample rate and my limited knowledge at the moment, is it safe to assume you need an SDR that’s at least capable of doing a sample rate of 30msps or greater?

DroneID is 10 MHz of occupied bandwidth, so in theory you could sample at ~ 10 MSPS (assuming you have a really good filter with sharp roll-off) and resample to 15.36 MSPS to do demod.  I was using 30.72 as it gave me the ability to offset tune to get the DC spike out of the way, and made it easier to hunt for the signal (since it hops around).
You should be able to change samp_rate to 15.36e6 and then set decimation to 1.  I haven't tried it recently, so not 100% sure everything will play nice.

Thank you. I’ll try it as is and also change to what you explained above and see if the hackrf is sufficient. I also have a lime/blade to try out as well.

I should be able to try this again tomorrow but while trying to capture drone ID I would occasionally have the flow graph exit out. I went as far away from anything on 2.4 as possible, checked and made sure nothing was in the spectrum and forced the controller to 5Ghz.
on the default freq of the flow graph I could see a spike and would occasionally get activity talking about burst in the bottom left gnuradio window along with activity in the QT graphs. Picture attached. I’m not certain If that’s what I should experience.

So there are going to be two main problems:

I don't think the GNU Radio OOT module was updated to ignore the first OFDM symbol (note that in your time domain display the first symbol is absent which happens for some drones).  This means that the constellations are going to be hot garbage since the frequency offset correction logic is going to fail in a major way
The MATLAB/Octave and GNU Radio OOT module both have a flaw in that they cannot deal with large (> a few KHz) of frequency offset.  The coarse estimate right now uses the cyclic prefix of one of the OFDM symbols (this is where MATLAB/Octave and the OOT module differ) which cannot detect larger frequency offsets.  The other problem here is that the ZC sequence correlation will move around in time if there is a frequency offset.  I remember stumbling across this when tinkering with detecting bursts at any frequency offset.  The ZC will do a great job of correlating for and detecting a burst, but if the burst is way off in frequency (tens of KHz or higher) then there will not only be a degradation in the correlation score, but the spike will show up before or after the actual start time of the ZC sequence in the recording.  This time offset causes a phase offset in the frequency domain which means that the constellation will not stay still (keeps rotating from sample to sample or FFT bin to bin).

I'm looking at a way to fix the second issue by autocorrelating for the true ZC sequence start.  That logic will be in the MATLAB/Octave before the OOT module though as the OOT module is secondary for me at the moment.
I'll take a look at the first issue tonight and see if it's an easy fix.  For drones that send 9 OFDM symbols, the first one is just a constant value that ends up as all zeros after removing the scrambler.  So it's not all that useful anyway.

Quick correction: The OOT modules does do CFO estimation and correction based on the second (your first) OFDM symbol.  Working on adding in some debugging to show extra information to help with troubleshooting.
I just have the DJI Mini 2 here to test with, so these kinds of issues will keep happening =\

If it helps any, I’ll leave this up for grabbing for awhile. Screen capture of everything I was trying https://drive.google.com/file/d/17m9iNO1ioXHAJ2SLDc3t7V7Gzp4l8rcG/view?usp=drivesdk

Oh that's perfect!  Thank you!
The signal you're seeing in the first part is likely just Bluetooth.  They are usually pretty narrow and look like little cones.
Around 2:00 is where you start to see Drone ID bursts.  I'm not sure why you're not getting a constellation plot that makes any kind of sense.  Usually if there is just a freq offset you'll see a ring, not a blob.
As for the rampant crashing, well, poop.  I haven't given this branch any love in a while due to trying to make the MATLAB/Octave work reliably.  I'll try running for longer here to see if I can recreate that crashing.

Additionally, the signal does jump around between frequencies.  It doesn't stay put for long which is why I made the GNU Radio graph in the first place.  Taking really long collects just to get a frame or two wasn't working out well.  I've been told that it might be moving between 2.4 and 5.8 GHz vs the usual stay put in one band that most systems seem to use.  You only get to see a burst every ~ 640 milliseconds and I don't normally see more than ~ 15 bursts at a single frequency before it moves on.

One possible thing that's causing you to not see constellations: The amplitude of the constellation is too high.  Try setting the X/Y max/min values to something like -10 and 10 instead of -2 and 2 that are set by default.  Alternatively, enable auto scaling.  I think that what you are seeing in the constellation plot is actually the FFT bins that are not carrying data.  Those will all usually be fairly low amplitude which means they cluster together in the center of the constellation plot.
The graph is exploding regularly for me too :(  Will be looking at what's causing the problem

I’ll take all this info and try again tomorrow evening. I’ll look into what else may have been transmitting and go even further away from stuff and/or not just disable but power off anything additional I have on me. Exciting stuff, not to mention the first time I’ve actually messed with a DJI.

The crashing issue you were seeing was due (at least in part) to walking off the edge of an array.  There is a commit in the gr-droneid branch [1] that should fix it.
https://github.com/proto17/dji_droneid/commit/f8bde33203b6876cab4d526eeedf6b4317d15938
No real need to get a completely RF quiet area.  It's rather unlikely to get a false trigger.  The real thing to stay away from is WiFi.  Try not to use frequencies that are being used by your WiFi AP as those transmit wide bandwidths and very frequently.

Getting a fatal error on extractor_impl.cc of droneid/misc_utils.h No such file or directory after doing the pull. I’ll retrace my steps.

Man... I really should have just gone to bed.  I goofed twice in a row with that fix.  The local copy of the repo that I am using has some changes to the utilities that don't exist in the remote just yet.  Gimmie a sec to fix that.

Give the new main a go and see if that helps.  Apologies for the busted code!

Pulled main, switched to gr-droneid and did another pull. Builds and installs fine now. I’m probably not helping with the sleeping part haha! Thanks for the update. I’ll give it another go this evening.

Flow graph was so much more stable, in fact I only got it to close out by cranking the gain next to some WiFi networks. It took me a little to realize I was seeing (I think) the burst. Updated screen recording https://drive.google.com/file/d/1g9vjwSdWYWytFRODoxMLfWtmBERWeoGl/view?usp=drivesdk

You're definitely seeing bursts.  If you zoom out on the constellation plot (bottom plot) you'll likely see some constellations :)
The equalizer is disabled, so the constellation points are going to be rough and rotating.

I didn’t even realize I could zoom out or just didn’t think of that, but good to hear I’m looking at the right thing. The last record to file sink I did in the video came out to about 2.3GB so I’ll see if I can work with that and use your other tools. Hopefully I can borrow the drone again some time. Safe to assume this ticket can probably be closed. Although it’s been helpful to share info and learn.

To do anything with the smaller capture, is that where I'd need to start learning about Matlab/Octave? I compiled the turbo decoder and was reading the notes at the top of it, doesn't seem like I can just feed it a 2.3GB file and expect it to do something with it.

There is a very brief and vague explanation at https://github.com/proto17/dji_droneid/#how-to-process-samples.  There is a script called processing_file.m that you edit the top of to have it read your file.  You will need to change the sample rate, file name, file type (usually just 'single' if you're using floating point values), and any frequency offset in your recording.  My code cannot deal with large offsets (greater than one carrier spacing - or 15 KHz), so you have to make sure your center frequency is good.
You don't need to process the full file if there were bursts.  Check your /tmp/bursts directory after running the flow graph.  That directory will contain one file per burst detected by the flow graph.  This problem of processing giant files is why I created the graph and partially why it doesn't do full equalization and demod.  I just needed it to pull bursts out instead of having to wait for minutes and accumulating GBs of samples.  You can feed the individual bursts to the process_file.m script just as you can with the large file.  If you want you could probably go one step further and merge all of the bursts into one file using cat and process them all at once.

Now I get what the tmp part is for. I wasn't connecting the dots. I went in and disabled that part of the code. I'll put it back, then use my file capture to run back through GNU Radio and get the bursts this time in /tmp, hopefully.

Last question since this ticket has grown to more then initially asked, ha! I ran the captured file back through the flow graph after rebuilding with comments in code removed. I see both channel and zc in the bursts directory, each about 8k. Opening zc in inspectrum looks like the attachment. I think I’m getting somewhere and that this is what I’d have to use within matlab?

Whoops, it looks like I commented out the saving of the bursts.  Uncomment line 85 in time_sync_impl.cc (https://github.com/proto17/dji_droneid/blob/gr-droneid/gnuradio/gr-droneid/lib/time_sync_impl.cc#L85) to have each burst saved off.
The file you're looking at now is the golden reference for the ZC sequence.  Though as this thread has been going, I'm wondering if my ZC sequence is wrong.  I think it's supposed to be one sweep, not two O.o  Might need to remove the call to fftshift.
Sorry about that!

I actually commented out the code and didn’t realize what it was doing up until your explanation last night. I couldn’t get the flow graph to initially run and never reverted those changes... You’re end is fine, I just keep commenting the stuff out.

Oh I gotcha, I’ll go do that. That way I’ll get more bursts I suppose.

Should it be pdf_element_count at the end of line 85 in time_sync_impl?

It would seem I’m close, I ended up having to process the entire IQ file as for some reason it wouldn’t detect individual bursts I generated by processing the file back through the flow graph. Maybe because of what I did above to get it to compile to save the multiple bursts. Here’s the result of processing the entire file. Images 1-9 all basically look exactly the same, image 10 is the one that looks messed up.

Were you by chance tuning around during the recording?  It's very strange to me that you found a good ZC sequence, but didn't have any symbols before, and the symbols after were rather garbage except for number 7.
The version of the MATLAB/Octave code in the GNU Radio branch is a little bit behind.  You can comment out these lines in the process_file.m script https://github.com/proto17/dji_droneid/blob/gr-droneid/matlab/updated_scripts/process_file.m#L151-L153.  Then run it again and you should see frames.

I reran again as I walked away to bed on a much larger capture, woke up thirsty just now and couldn’t resist checking. I’m looking at frames printed to screen with a good looking png. I think we have success!

Messing with the resulting frames and attempting to line up the info against an image on Twitter of how the drone ID breaks down. It seems like some parts of the beginning and end match up, but I’m curious if the matlab/decoder script blanks out any portion of the dji packet.

The structure of the DroneID frame is slightly different on the "RF" (I
don't have a better name) link vs the WiFi.  The beginning and end are
slightly different.  The middle does match up if memory serves.  See
https://github.com/proto17/dji_droneid/blob/main/matlab/updated_scripts/transmit/create_frame_bytes.m
for a full breakdown of the fields.  It's not documented at all but the
order of the fields can be seen at line 185
On Fri, May 6, 2022 at 9:41 AM alphafox02 @.***> wrote:

Messing with the resulting frames and attempting to line up the info
against an image on Twitter of how the drone ID breaks down. It seems like
some parts of the beginning and end match up, but I’m curious if the
matlab/decoder script blanks out any portion of the dji packet.
[image: 0A2FC076-8313-4AC4-9AF8-1D114182B082]
https://user-images.githubusercontent.com/44436101/167144074-0efb42ce-7a87-4f73-8c55-1f873b37ecea.jpeg
—
Reply to this email directly, view it on GitHub
https://github.com/proto17/dji_droneid/issues/1#issuecomment-1119635813,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABALJH3MTC5PIGKJOF3VPGDVIUOOXANCNFSM5UDYNUDQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Thank you for the help along the way! I think this ticket is filled with valuable info. At this point everything seems to be working perfectly. I’ll go ahead and close and continue to test and refine on my end. Super awesome, thanks sharing!

I ran the entire IQ file again (same one) through the octave script and the frames were exactly like I got back 4 days or so ago, except, after the last number in my previous attempt there was lots of zeros and then another string of numbers this time. Basically the frame was longer, but everything up till the ten or so zeros was exactly the same as before.
I'm just wondering could this have been do to code changes in main and the decoding process? I'm going to attempt again and see if the frame is still the longer length. I failed to keep track of my git pulls, so it's possible I was running older code in main last week. Just thought this was odd and wondered if you've ran into this on your end.

There was an update to remove_turbo.cc that started printing out all of the received bytes, and not just the DroneID related bytes.  As I understand it there are a bunch of random bytes from memory stuffed in with the DroneID payload.  These bytes will be constant until the drone is power cycled.  Is that what you're seeing?  It should be 100 or so bytes of DroneID, a lot of zeros, some completely random garbage that always looks the same frame to frame, and then the CRC
The relevant commit is https://github.com/proto17/dji_droneid/commit/000d76e6fb168637eeb71abc4fb6b5ef1487334a

I did multiple IQ files on different days, so power cycles happened. I'll look closer now at all of them and see what remains the same using the latest code in main.
Without sharing the frame from last week online, what I'm seeing is about 68 characters before hitting more or less 50 zeros (there's a few non zeros) and then back to roughly 64 various numbers. Sorry for being vague. I'll study all results closer and attempt to decode them again and see if my lat/long and other information is correct.

I checked the remove_turbo.cc file on the machine I decoded the burst with initially and it did not have the commit you referenced above. This explains why my output is different now that I'm decoded it with the newer code in main.

Check out [1] and [2] to see how to convert the value.  This is the arithmetic to convert from a floating point lat/long to the packed integer representation that is used in the DroneID frame.
[1] https://github.com/proto17/dji_droneid/blob/main/matlab/updated_scripts/transmit/create_frame_bytes.m#L158
[2] https://github.com/proto17/dji_droneid/blob/main/matlab/updated_scripts/transmit/create_frame_bytes.m#L173

Got it and it comes out spot on. Had a little extra help with the conversation but all good now. Thank you.


Do not allow durations with fractions of a sec

Hi everyone,
I just noticed some slightly confusing behaviour for sub-second intervals:
If you start a job with @every 1ms it will run every second, if you start it with @every 1s999ms it will run every second as well. This seems to be intended behaviour but according to https://github.com/robfig/cron/issues/401 it's not worth implementing anything below one second.
Do you think it might be a good idea to return an error instead of rounding to a full second? It's quite tempting to use milliseconds in unit tests.
Cheers :)

Hi @stubents, yes, seems a good idea. Thank you for pointing this out.

Great,
how would you implement it?
Making Every(...) return an error will make constructs like
cron.Schedule(Every(time.Minute), func() {println("hello world")})
impossible.
We could either just accept this, or introduce a new method like MustEvery(...)
What do you think?

I gave it a try :)
see https://github.com/gdgvda/cron/pull/23


Text Magic

Feature Description
combo of color, fill style/highlight, line height, frosted glass effect
Alternatives Considered

Additional Context

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

This is now a duplicate of #7718 which prometheus is already working on.

Hmm I always thought text magic referred to this awesome demo Paul built a while ago: https://codepen.io/pbakaus/pen/gOpZXbW
Which is quite different from #7718 I think

i guess i saw it as iterations, but i don't know. This is just an old ticket and I wanted to get some stuff cleaned up and more current.


[CWS] eventmonitor stops dispatching event while stopping

What does this PR do?
Add a stop a state to Probe which aims to handle the different states of the Probe especially the concurrency issue between the dispatch events to the event monitor and the stop/close functions. Prior this PR consumers could still receive event even after being stoppped.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

@DanielLavie I updated the PR using the manager fix.


Check TPM.Capabilities of SINIT and Capabilities of TPM-Module

Ensure that SINIT ACM correlate with the used tpm-module and tpm-specification.

Is this still open?

Puh, I think so. Need to take a look again D:

#130


A rustc error may be related to the cargo

Problem

Compiler panics on macos after I did some refactors on Cargo.toml:
thread 'rustc' panicked at 'Failed to get crate data for crate22', compiler/rustc_metadata/src/creader.rs:136:32
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

error: internal compiler error: unexpected panic

note: the compiler unexpectedly panicked. this is a bug.

note: we would appreciate a bug report: https://github.com/rust-lang/rust/issues/new?labels=C-bug%2C+I-ICE%2C+T-compiler&template=ice.md

note: rustc 1.52.0-nightly (4a8b6f708 2021-03-11) running on x86_64-apple-darwin

note: compiler flags: -C opt-level=3 -C linker-plugin-lto -C codegen-units=1 --crate-type lib

note: some of the compiler flags provided by cargo are hidden

query stack during panic:
end of query stack
error: could not compile `clitrans`

To learn more, run the command again with --verbose.
Error: Process completed with exit code 101.

Details can be found in this CI report (Note Windows and Ubuntu target were both built successfully):
https://github.com/wfxr/clitrans/runs/2091606690
I reported this error to the rustc (https://github.com/rust-lang/rust/issues/83045) but it seems to be related to cargo so I also report it here.
Steps
https://github.com/wfxr/clitrans/commit/57ca835c2e009168f29e21208c3b01a9cc1809bd
Sorry I don’t have a macos computer at the moment so it's hard to debug which is the exactly change caused this error.
Possible Solution(s)

Notes
Output of cargo version:

toolchain: nightly-x86_64-apple-darwin (default)
cargo 1.52.0-nightly (970bc67c3 2021-03-07)
rustc 1.52.0-nightly (4a8b6f708 2021-03-11)

Thanks for the report!  ICE is a rustc issue, so it is correct to file it in rust-lang/rust.  It's best to only file one issue.  If it is in the wrong repository, someone will move it for you.  Closing for https://github.com/rust-lang/rust/issues/83045.


Suggest @objc for overrides of declarations from/in extensions.

The Swift class model does not support overriding declarations where either
the overridden declaration or the overriding declaration are in an extension.
However, the Objective-C class model does, so marking the declaration as
@objc (when possible) will work around the limitation.
Customize the "cannot override declaration in extension" diagnostic to
suggest adding @objc to the overridden declaration in cases where
@objc is permitted. Fixes SR-6512 / rdar://problem/35787914.

@swift-ci please smoke test and merge


[bug] WagmiAdapter to correctly infer wagmiConfig type

Link to minimal reproducible example
https://stackblitz.com/edit/viem-getting-started-dqqqdr?file=index.ts
Summary
When using the WagmiAdapter, we can access the wagmi config by doing wagmiAdapter.wagmiConfig as stated in the documentation . However, wagmiAdapter.wagmiConfig doesn't leverage generics from wagmi's createConfig.
import { WagmiAdapter } from '@reown/appkit-adapter-wagmi';
import { createConfig, http } from 'wagmi';
import { arbitrum, sepolia } from 'viem/chains';

const wagmiAdapter = new WagmiAdapter({
  projectId: process.env.NEXT_PUBLIC_REOWN_PROJECT_ID!,
  networks: [arbitrum, sepolia],
});

// Poorly typed: Config
wagmiAdapter.wagmiConfig;

const config = createConfig({
  chains: [arbitrum, sepolia],
  transports: {
    [arbitrum.id]: http(),
    [sepolia.id]: http(),
  },
});

// Strongly typed: Config<chains, transport>
config;

declare module 'wagmi' {
  interface Register {
    config: typeof wagmiAdapter.wagmiConfig; // Won't work because the type is not correctly inferred
  }
}

Doing so prevent strong type-safety as stated in wagmi's documentation.
Could it be possible for the WagmiAdapter to take those in consideration ?
List of related npm package versions
@reown/appkit: ^1.3.1
@reown/appkit-adapter-wagmi: ^1.3.1
viem: 2.21.44
wagmi: 2.12.29

I'm actually using the following workaround to mimic wagmi's createConfig behavior in WagmiAdapter
declare class WagmiAdapter<
  const networks extends readonly [AppKitNetwork, ...AppKitNetwork[]]
> implements ChainAdapter {
  public wagmiChains: networks;
  public wagmiConfig: AdapterOptions<Config<networks>>['wagmiConfig'];
  public constructor(configParams: Partial<CreateConfigParameters> & {
    networks: networks;
    projectId: string;
});

It does the tricks for me since I only use Viem's chain, but I guess it wouldn't work with the Solana network.

May be realted to #2823

Wagmi doesn't support Solana - looks like you found a workaround - curious @AzzouQ what app are you building?

You are totally right, Wagmi and Solana are not related as it is handled by 2 differents adaptaters.
Still the issue remain, i'm not sure why you closed it. The WagmiAdapter doesn't leverage generics from wagmi's createConfig, which prevent strong type-safety as stated in wagmi's documentation.


Remove redundant and error-causing code.

 CloudAssets: Error when upload images in admin Files section. "Parent method doesn't exist on Image" (4)


Fix tests

Upon fixing #30, I found out that, some tests were failing because of Windows path standarts. Fixed them, so they are OS independent.

And this is dependent on #31, as I changed my previous unittests as well.

I see you are fixing them in #33

Thank you, I did not see this PR before.
But I think they have some differences. Feel free to update it if you think so.


fix flappy "ping-req not ok" test

In ringpop, a node E is declared suspect trough the indirect ping mechanism: When a node A fails to ping E; A randomly picks three members to send a ping-req to, say B, C and D. Upon receiving the ping-reqs the three nodes will ping E and send back the status (ping ok / ping not ok) to A. If all three results are "ping not ok", A will declare E suspect.
In this test we check ringpop's behaviour from the perspective of B (or C or D). We check if B responds with "ping not ok" since E is not responsive. It is important that B doesn't decide that E is a suspect when the ping from B to E fails, which could be a likely programmer mistake. Therefore we check that the all members are alive and there are no suspects in B's membership at the end of the test.
However, sometimes B found out that E is suspect trough it's own ping mechanism, which caused the test to fail. By disabling all pings and ping-reqs, B will never be able to decide if E is unreachable and therefor no longer declares E suspect.
rewrite comment

LGTM :+1:

LGTM
But I am thinking, wouldn't it be better to turn off the gossip of the real node during the test to prevent it from pinging on its own?

Agreed @thanodnl. This functionality isn't present in the integration tests yet, and this is a quick and somewhat dirty fix.


daemon, fqdn: Add flag to control FQDN regex LRU size

Advanced users can configure the LRU size for the cache holding the
compiled regex expressions of FQDN match{Pattern,Name}. This is useful
if users are experiencing high memory usage spikes with many FQDN
policies that have repeated matchPattern or matchName across many
different policies.
Please see second commit msg for the performance benchmark results.
Signed-off-by: Chris Tarazi<EMAIL_ADDRESS>Related: https://github.com/cilium/cilium/pull/17224

/test

/test-1.23-net-next

/test-1.21-5.4

Will push short to resolve conflict. Noting that CI has passed besides known flakes.

Travis passed as a sanity check for the rebase. Marking ready to merge given https://github.com/cilium/cilium/pull/19383#issuecomment-1112528595.


Typos and updated broken links and followed readme pattern for hashing folder

Proposed changes
Typos and updated broken links and followed readme pattern for files
Types of changes
What types of changes does your code introduce to this project?
Put an x in the boxes that apply

[ ] Proofreading (fix spelling, grammar, punctuation, accuracy) More about Proofreading
[ ] New Files/Folder (adding folder without removing existing one)
[ ] Add/Remove Files (adding/removing existing files/folder)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. Don't hesitate to ask if you're unsure about any of them. We're here to help! This is simply a reminder of what we will look for before merging your code.

[ ] Tested any code against its gcc/clang or the official binaries and it passes locally with my changes
[ ] I have added the necessary explanation
[ ] I have added the necessary links in the working section
[ ] Added external links

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...
Reviewers:
This will be filled by the people who review this particular Pull Request.

@all-contributors please add @Jayesh2812 for doc


Add loading status in application

App has long loading times, and there are no indicators whether it crashed or what is going on. To show users that everything is working well, or something has crashed, we should add status bars in the application that show if the action they just did is successful or not.
I really started to feel the necessity of this task as we started to integrate database into the application, there should be ui changes that signal what is going on in the background, users who have no access to console or server logs have no idea what is going on it is a little stressful to use it

Made it an issue to link a PR


add LifeCycleListener to log construction and destruction process of guice modules

This is a step to separate org.embulk.guice module to a separated OSS module.

this is done by #367.