Make construction with StaticArrays kinda work

Codecov Report

Merging #215 (ab5c287) into main (877f35a) will decrease coverage by 0.55%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main     #215      +/-   ##
==========================================
- Coverage   73.00%   72.46%   -0.55%
==========================================
  Files          20       20
  Lines         689      690       +1
==========================================
- Hits          503      500       -3
- Misses        186      190       +4

Impacted Files
Coverage Δ

src/componentarray.jl
79.06% <100.00%> (-2.19%)
:arrow_down:

src/utils.jl
96.96% <100.00%> (ø)

... and 1 file with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


How to start and stop chunkwm

It would be nice if the documentation told you how to start and stop kwm for those who are not familair with launchctl. If it is already posted then could you refer me to it? I wanted to try out but it doesn't seem to be tiling well on my 15" macbook pro.

Yeah, really agree on this one.

For me, it was
launchctl load -w ~/Library/LaunchAgents/com.koekeishiya.chunkwm.plist
chunkwm

The above works, but I'm also missing kwmc quit for a quick refresh when things just get out of whack.

If it is already posted then could you refer me to it?

It's listed in the install section under Source.
If you installed using MacPorts, it is printed to the terminal as part of the ports notes section, for things the user needs to do after installing.

The above works, but I'm also missing kwmc quit for a quick refresh when things just get out of whack.

You can just unload and load the tiling plugin again, or whatever other plugin this refers to, just as the .chunwmrc bash script does it.

Hi @koekeishiya thanks for the response. This was not at all obvious to me especially since again I'm not too familiar with launchctl (now I am since I had to look up how to turn off chunkwm). I still think it would be useful to be explicit about this and also how to turn it off.

For posterity (and because this issue has good SEO): I was running into an issue getting chunkwm to start after a brew install. It turned out that the official example .chunkwmrc was assuming that plugins existed in ~/.chunkwm_plugins, but brew does not set this up. To get things running, I symlinked the plugins and restarted:
$ ln -s /usr/local/opt/chunkwm/share/chunkwm/plugins/* ~/.chunkwm_plugins/
$ brew services restart chunkwm

@henry-stripe The caveats for the brew formula include this:
==> Caveats
Copy the example configuration into your home directory:
  cp /usr/local/opt/chunkwm/share/chunkwm/examples/chunkwmrc ~/.chunkwmrc

Installing chunkwm via Homebrew modifies the example chunkwmrc to always use the correct path. This is necessary because this path might be different from the one I quoted above when using non-default Homebrew settings.

Sorry everyone, I still could not exit chunkwm, trying aunchctl unload -w ~/Library/LaunchAgents/com.koekeishiya.chunkwm.plist It says 'no such file or list'.

@ipstone If you installed through Homebrew, the plist is at ~/Library/LaunchAgents/homebrew.mxcl.chunkwm.plist. In general, launchctl dumpstate | grep chunkwm, will tell you a bunch of info, including the absolute path to the plist.

@ipstone If you installed through Homebrew, the plist is at ~/Library/LaunchAgents/homebrew.mxcl.chunkwm.plist. In general, launchctl dumpstate | grep chunkwm, will tell you a bunch of info, including the absolute path to the plist.

this helped tons! thanks!


webOS and Tizen export

Is your feature request related to a problem? Please describe.
Exporting webOS or Tizen package is not possible
Describe the solution you'd like

Project folders (possibly for each webOS and Tizen version)
Commands to create builds.

npx expo export -p webos
npx expo export -p tizen

Describe alternatives you've considered
I tried to create web build with npx expo start --web, move that to the basic LG web app project (created with ares) and then use ares to package/install for LG webOS emulator - it did not run. Basic project did.
Additional context
I am not aware of the nuances there are when packaging for webOS or Tizen and going over each platforms docs takes time. It would be a good addition. If not a priority it could be explained it docs at least.

I tested it with webos with the web export first npx expo start --web and then packaging the js files in an ares package.
What error do you have?
A good thing to check is in the html file created by the basic LG project the js file is correctly pointing to the right path

I tested it with webos with the web export first npx expo start --web and then packaging the js files in an ares package. What error do you have? A good thing to check is in the html file created by the basic LG project the js file is correctly pointing to the right path

The exporting / installing / launching all works without errors but the screen is black. I'm using LG webOS TV Emulator 5.0.0

This sounds like an ES5 issue (or something else which needs poly-filling). webOS 5 is like Chromium 68 and webOS 6 is like Chromium 79, while webOS 22 is like Chromium 87. Though I've only starting looking at this project out of interest, I haven't checked out or compiled anything yet, but have built a lot of apps on webOS 4.5 and 5.0, often finding myself having to check caniuse for thing I thought would've been supported.

This sounds like an ES5 issue (or something else which needs poly-filling). webOS 5 is like Chromium 68 and webOS 6 is like Chromium 79, while webOS 22 is like Chromium 87. Though I've only starting looking at this project out of interest, I haven't checked out or compiled anything yet, but have built a lot of apps on webOS 4.5 and 5.0, often finding myself having to check caniuse for thing I thought would've been supported.

Thank you for the input.
Is there any tool that could tell whether a exported project runs on certain webOS or Tizen version?
Or is it all feature by feature?

Hey @ksvslk I'm running some tests, Is you app not working on webOS 5.0 or 6.0 ? I only have the emulator for 6.0

I was only able to get it work on Simulator 22 and none of the Emulators (if i remember correctly). Neither 2.0, 5.0 or 6.0.

This sounds like an ES5 issue (or something else which needs poly-filling). webOS 5 is like Chromium 68 and webOS 6 is like Chromium 79, while webOS 22 is like Chromium 87. Though I've only starting looking at this project out of interest, I haven't checked out or compiled anything yet, but have built a lot of apps on webOS 4.5 and 5.0, often finding myself having to check caniuse for thing I thought would've been supported.

Thank you for the input. Is there any tool that could tell whether a exported project runs on certain webOS or Tizen version? Or is it all feature by feature?

Nothing beats testing on an actual webOS 5/6 TV. But running your app in Chromium 68 while developing may help catch styling issues early, the simulator will help with testing builds and this tool can validate code for ES5 compliance https://github.com/legend80s/es5-validator annoyingly I don't think you can run it against an entire build folder - but if you're bundling everything into just a handful of js files, you can just point it at those files.
As well as using babel to compile your code down to ES5, I found (for old LGs and various other old set top boxes), needing many polyfills that my babel config seems to have missed:
https://cdnjs.cloudflare.com/polyfill/v3/polyfill.min.js?features=default%2CObject.is%2CArray.prototype.map%2CMap%2CNumber.isInteger%2CReflect%2CReflect.deleteProperty%2CObject.getOwnPropertyDescriptor%2CObject.getOwnPropertyDescriptors%2CObject.getOwnPropertyNames%2CReflect.getOwnPropertyDescriptor%2Csmoothscroll%2CObject.entries%2CSet%2CAbortController%2Cfetch%2Ces5%2CgetComputedStyle%2CIntersectionObserver%2CrequestAnimationFrame
If you hit the URL in a modern browser, you'll get an empty response. But if you spoof your user agent to something old LG flavoured as below, you'll see it gives you the polyfills for those missing features I found I needed.
Mozilla/5.0 (Web0S; Linux/SmartTV) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36 HbbTV/1.6.1 (+DRM; LGE; 32LQ630B6LA; WEBOS22 03.30.73; W22_K8AP; DTV_W22L;) FVC/7.0 (LGE; WEBOS22;) Edg/<IP_ADDRESS>

This sounds like an ES5 issue (or something else which needs poly-filling). webOS 5 is like Chromium 68 and webOS 6 is like Chromium 79, while webOS 22 is like Chromium 87. Though I've only starting looking at this project out of interest, I haven't checked out or compiled anything yet, but have built a lot of apps on webOS 4.5 and 5.0, often finding myself having to check caniuse for thing I thought would've been supported.

Thank you for the input. Is there any tool that could tell whether a exported project runs on certain webOS or Tizen version? Or is it all feature by feature?

Nothing beats testing on an actual webOS 5/6 TV. But running your app in Chromium 68 while developing may help catch styling issues early, the simulator will help with testing builds and this tool can validate code for ES5 compliance https://github.com/legend80s/es5-validator annoyingly I don't think you can run it against an entire build folder - but if you're bundling everything into just a handful of js files, you can just point it at those files.
As well as using babel to compile your code down to ES5, I found (for old LGs and various other old set top boxes), needing many polyfills that my babel config seems to have missed:
https://cdnjs.cloudflare.com/polyfill/v3/polyfill.min.js?features=default%2CObject.is%2CArray.prototype.map%2CMap%2CNumber.isInteger%2CReflect%2CReflect.deleteProperty%2CObject.getOwnPropertyDescriptor%2CObject.getOwnPropertyDescriptors%2CObject.getOwnPropertyNames%2CReflect.getOwnPropertyDescriptor%2Csmoothscroll%2CObject.entries%2CSet%2CAbortController%2Cfetch%2Ces5%2CgetComputedStyle%2CIntersectionObserver%2CrequestAnimationFrame
If you hit the URL in a modern browser, you'll get an empty response. But if you spoof your user agent to something old LG flavoured as below, you'll see it gives you the polyfills for those missing features I found I needed.
Mozilla/5.0 (Web0S; Linux/SmartTV) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/87.0.4280.88 Safari/537.36 HbbTV/1.6.1 (+DRM; LGE; 32LQ630B6LA; WEBOS22 03.30.73; W22_K8AP; DTV_W22L;) FVC/7.0 (LGE; WEBOS22;) Edg/<IP_ADDRESS>

Thank you for the detailed response! I will try these methods in the coming weeks.


cmake error while building armnn using Android-NDK

I got a cmake  error while building armnn . It says that ndk is not found but i didn't get any error while building ndk and i have exported the PATH variable.
i got the below error:
lenovo@lenovo-Lenovo-ideapad-300-15ISK:~/armnn-devenv/armnn/build$ CXX=aarch64-linux-android-clang++  CC=aarch64-linux-android-clang  CXX_FLAGS="-fPIE -fPIC"  cmake ..   -DCMAKE_SYSTEM_NAME=Android   -DCMAKE_EXE_LINKER_FLAGS="-pie -llog"   -DARMCOMPUTE_ROOT=$HOME/armnn-devenv/ComputeLibrary/   -DARMCOMPUTE_BUILD_DIR=$HOME/armnn-devenv/ComputeLibrary/build   -DBOOST_ROOT=$HOME/armnn-devenv/boost/install/   -DARMCOMPUTENEON=1 -DARMCOMPUTECL=1   -DTF_GENERATED_SOURCES=$HOME/armnn-devenv/google/tf_pb/ -DBUILD_TF_PARSER=1   -DPROTOBUF_ROOT=$HOME/armnn-devenv/google/arm64_pb_install/
CMake Error at /usr/share/cmake-3.10/Modules/Platform/Android-Determine.cmake:135 (message):
Android: Neither the NDK or a standalone toolchain was found.
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/CMakeDetermineSystem.cmake:130 (include)
CMakeLists.txt:2 (project)
CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!
-Thank you

Yes , help us in this regard

Hi @VinayKarnam,
The https://github.com/ARM-software/armnn/blob/master/BuildGuideAndroidNDK.md guide incorrectly specifies Ubuntu 18.04, currently, we only test the Android NDK build with Ubuntu 16.04 which I have verified working with the ArmNN 18.08 release. In the current state, the linked guide does not work for Ubuntu 18.04.
Apologies for any delays caused by the typo. We'll update the documentation to reflect the correct version of Ubuntu.

Ok thank you for your reply

It's still incorrect.

They have been tested on a clean install of Ubuntu 18.04.

@georgeokelly are you seeing problems? We do some testing on 18.04 now and all issues are supposed to be resolved.

yes. I met with the same issue. @MatthewARM
OS:  Ubuntu 18.04.2 LTS
cmake version: 3.10.2
ndk@ubuntu:~/armnn-devenv/armnn/build$ CXX=aarch64-linux-android-clang++  CC=aarch64-linux-android-clang  CXX_FLAGS="-fPIE -fPIC"  cmake ..   \
-DCMAKE_SYSTEM_NAME=Android   \
-DCMAKE_ANDROID_ARCH_ABI=arm64-v8a   \
-DCMAKE_ANDROID_STANDALONE_TOOLCHAIN=$HOME/armnn-devenv/toolchains/aarch64-android-r18b/   \
-DCMAKE_EXE_LINKER_FLAGS="-pie -llog"   \
-DARMCOMPUTE_ROOT=$HOME/armnn-devenv/ComputeLibrary/   \
-DARMCOMPUTE_BUILD_DIR=$HOME/armnn-devenv/ComputeLibrary/build   \
-DBOOST_ROOT=$HOME/armnn-devenv/boost/install/   \
-DARMCOMPUTENEON=1 -DARMCOMPUTECL=0   \
-DTF_GENERATED_SOURCES=$HOME/armnn-devenv/google/tf_pb/ -DBUILD_TF_PARSER=1   \
-DPROTOBUF_ROOT=$HOME/armnn-devenv/google/arm64_pb_install/

CMake Warning at /usr/share/cmake-3.10/Modules/Platform/Android-Determine.cmake:125 (message):
  Android: Did not detect API level from

    /home/ndk/armnn-devenv/toolchains/aarch64-android-r18b//bin/clang

  or

    /home/ndk/armnn-devenv/toolchains/aarch64-android-r18b//sysroot/usr/include/android/api-level.h

Call Stack (most recent call first):
  /usr/share/cmake-3.10/Modules/CMakeDetermineSystem.cmake:130 (include)
  CMakeLists.txt:6 (project)

CMake Error at /usr/share/cmake-3.10/Modules/Platform/Android-Determine.cmake:195 (message):
  Android: The API specified by CMAKE_SYSTEM_VERSION='' is not an integer.
Call Stack (most recent call first):
  /usr/share/cmake-3.10/Modules/CMakeDetermineSystem.cmake:130 (include)
  CMakeLists.txt:6 (project)

CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
CMake Error: CMAKE_C_COMPILER not set, after EnableLanguage
CMake Error: CMAKE_CXX_COMPILER not set, after EnableLanguage
-- Configuring incomplete, errors occurred!

OK thanks I'll reopen this issue.

Hi @georgeokelly I can see that you are using NDK r18b rather than r16b as in the guide. Have you tried with r16b and do you require r18b for your project?

Ahah I've found the problem, the armnn_19_02 release has an updated guide, but the master branch has not been updated. The release can be found here: https://github.com/ARM-software/armnn/tree/branches/armnn_19_02
I'll look into why the master branch is out of date.

Hi. I try with NDK r17c and it works. Thanks a lot.

I've made the master branch on github track the master branch from gerrit, so this sort of discrepancy should not happen again.
All the best,
Matthew


Video should stop playing if parent details element is closed

This feature is definitely not required, but is a nice to have.
I simple solution is to grab the embedded iframe source, and reset the value. This causes the entire embedded content to be reloaded, regardless if the video was playing or not. This is a significant performance hit.
A smarter solution is to use the YouTube iframe API: https://developers.google.com/youtube/iframe_api_reference

Description to come later:

$(document).ready(function() {
  // Pause YouTube videos playing within closed details elements
  $("body").on("click", "details[open]", details_element_closed);
});

function details_element_closed() {
  /**
   * Pause any videos playing within closed details element.
   */
  // Find all YouTube iframes
  $('iframe[src*="youtube"]', this).each(function() {
    //  $(this).attr('src', $(this).attr('src'));
    if (!this.id) {
      var src = this.src;
      var video_id = src.substring(src.lastIndexOf('/') + 1, src.indexOf('?'));
      this.id = 'youtube-embed-' + video_id;
    }
    // Create api player
    var player = new YT.Player(this.id, {
      events: {
        'onReady': onPlayerReady,
      }
    });
  });
}

function onPlayerReady(event) {
  // Pause video
  event.target.pauseVideo();
}


SA1102: QueryClauseMustFollowPreviousClause

http://www.stylecop.com/docs/SA1102.html

This one is a little bit tricky. I checked original StyleCop and for code like:
 int [] a =new int[0];
        var query = from m

                        in

            a

                    where m > 5

            & m > 2

            group m by m into c

                    select c.Key;

it only reports  where m > 5 as an incorrect part of query. In my opinion it should report more. I'd understand this SA as: each query clause (from, in, where, group, select, let) should be on the same line as the previous line or at the next line. However, doing it this way we completly change the behaviour of this SA. @sharwell , @pdelvo , what do you think about it ?

The way I interpreted the rule is:

Locate every where, let, group, join, order, and select clause. If the query as a whole spans more than one line, then each of these keywords should appear as the first token on a new line, and there should not be a blank line before it.

I did not include from in this list because it is always the first token of a query, and it may appear at the end of a line containing other tokens which are not part of a query (for example, the var query = part of your sample code). I did not include in because it is part of the from clause, like by is part of the order clause.
For example, the following code, which is your original code without the blank lines, would not produce any SA1102 warnings:
 int [] a =new int[0];
        var query = from m
                        in
            a
                    where m > 5
            & m > 2
            group m by m into c
                    select c.Key;

Ok, so (just curious) you are the author of the original SA ?

No, I'm actually not even a user of it. :grin:
I'm interested in this project primarily for the learning opportunity, although I do want to see it succeed for StyleCop users.

Allright. I'll implement it the way you proposed, because it's a very reasonable solution.


import types from core lib

related https://github.com/react-hook-form/react-hook-form/pull/2733

Thank you @kotarella1110 👍


Add commands for enabling/disabling quips to the help message

The users on my server wished to disable the quips.  I was able to find the command through this Github issue, but less technical users might not find this, so it would be good to add the quip enable and quip disable to the help message.

In a release 18 days ago, along with the whole feature, the quip disable/enable was added  to the @mention help command as lines 4 and 5:
https://github.com/alexberryman/DiscordWordle/blame/main/cmd/bot/help.go#L13-L14

Are you not seeing this help text in your server?

Oh, oops - I forgot that when I set up Heroku to run my own instance of the server, I had deployed from a fork I made of your repo rather than your repo directly. 🤦  Actually merging in the updated code works. :-). Apologies for the false alarm, and thanks for the bot!

Cool. Glad to hear you are enjoying it!


Add calliopemini

This PR aims to add the Calliope mini 3 to support jacdac
I am able to load our fork into makecode.calliope.cc/alpha, but when I press add Blocks the additional extensions are loaded from microsoft/pxt-jacdac which raises errors. So supporting the Calliope mini 3 in this repository would be great.

@pelikhan Could you please review this pull request and provide your feedback? If everything looks good, feel free to merge it directly.

Looks good waiti for build.

We could probably have a pre-built .hex for the jukebox projects on callipemini.

please test.

Looks good so far, button and slider are working well. jacdac-led produces a name conflict when loading the colors package, but that something we have to fix inside the pxt-calliope target.

oh snap. btw, we should have a call about this new board? There's a couple things to iron out for the device registration.

I've removed the "color" package dependency as it is not used anymore.


Update strings_zh-CN.json

Sorry for the inconvenience. For Windows, it's recommended to use just "Exit" without the app name (if you've Windows, you can verify this by checking other apps in the system tray, especially those from Microsoft). I've already sent a corresponding pull request (which hasn't been approved yet), but you added the translation faster. If it's not too much trouble, it would be great if you could make the necessary correction (449 line: "closeWindows"). Thank you in advance.
Best regards,
Sergiy Tkachuk

Sorry for the inconvenience. For Windows, it's recommended to use just "Exit" without the app name (if you've Windows, you can verify this by checking other apps in the system tray, especially those from Microsoft). I've already sent a corresponding pull request (which hasn't been approved yet), but you added the translation faster. If it's not too much trouble, it would be great if you could make the necessary correction (449 line: "closeWindows"). Thank you in advance.
Best regards, Sergiy Tkachuk

Thanks for your remind ;)


[SPARK-46448] InlineTable column should be nullable when converted to Decimal might cause null

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

InlineTable column can also be nullable when there is type conversion to Decimal and it is safe to be converted to null.
How was this patch tested?

UT
Was this patch authored or co-authored using generative AI tooling?

NO

@cloud-fan

Can you fully fill the PR description?

@cloud-fan my bad. The save button didn't work on my previous edit.

Closing this PR for now as this may not be needed given that the real value will not be null.


load plugin

Is your feature request related to a problem? Please describe.
v1/beat2 支持加载插件吗?

例如

apiVersion: apps.emqx.io/v1beta3
kind: EmqxPlugin
metadata:
  name: emqx-lwm2m
  namespace: default
spec:
  selector:
      "foo": "bar"
  pluginName: emqx_lwm2m
  config:
      "lwm2m.lifetime_min": "1s"
      "lwm2m.lifetime_max": "86400s"
      "lwm2m.bind.udp.1": "<IP_ADDRESS>:5683"
      "lwm2m.bind.udp.2": "<IP_ADDRESS>:5684"
      "lwm2m.bind.udp.3": "<IP_ADDRESS>:5685"
      "lwm2m.xml_dir": "/opt/emqx/etc/lwm2m_xml"

Sorry about that, EmqxPlugin only support v1beta3, we plan release this for emqx operator 1.2 version

Sorry about that, EmqxPlugin only support v1beta3, we plan release this for emqx operator 1.2 version

When will emqx operator 1.2 be released? We are going to try it out.

We need the ability to dynamically load plugins. The current helm chart support is not good. We urgently need this function.

We plan to release it by June 15 and demo it on June 15 at 8:00pm Beijing time, so stay tuned!

We need the ability to dynamically load plugins. The current helm chart support is not good. We urgently need this function.

I'm believe EmqxPlugin CRD will meet your needs, if you have any feedback, please let me know

Hi, @siaron EMQX Operator 1.2 will come soon, please check https://github.com/emqx/emqx-operator/issues/223


Cant play the link

The link http://player-castv.c1.biz/get_m3u/piccolo-tv_channels_shqipHD.m3u
Does not work it says chek your link . Its not valid .
Maybe should take another look or i need to find another iptv player

prove working perfect

prove this
;http://player-castv.cf/get_m3u/piccolo-tv_channels_Albania.m3u

This link work thank you .. faleminderit!!


Transparent Search_Bar(Power ToysRun)

Description of the new feature / enhancement

Scenario when this would be used?
When Search An Item
Supporting information
No response

/dup https://github.com/microsoft/PowerToys/issues/4794


Test enhancement

Changed log

The satooshi/php-coveralls package. Using the php-coveralls/php-coveralls instead.
To support future stable PHPUnit version, using the PHPUnit\Framework\TestCase namespace.
Using the final PHPUnit 4.8 version and it can be easy to migrate to latest stable PHPUnit version in the future.

Coverage remained the same at 97.561% when pulling e622425312af183058197c9f94306034f6cc4756 on open-source-contributions:test_enhancement into 602c9ddea2356434e9dfebe43ae57c5f98c1adfb on peter-gribanov:master.

Coverage remained the same at 97.561% when pulling e622425312af183058197c9f94306034f6cc4756 on open-source-contributions:test_enhancement into 602c9ddea2356434e9dfebe43ae57c5f98c1adfb on peter-gribanov:master.

Coverage remained the same at 97.561% when pulling e622425312af183058197c9f94306034f6cc4756 on open-source-contributions:test_enhancement into 602c9ddea2356434e9dfebe43ae57c5f98c1adfb on peter-gribanov:master.

Coverage remained the same at 97.561% when pulling e622425312af183058197c9f94306034f6cc4756 on open-source-contributions:test_enhancement into 602c9ddea2356434e9dfebe43ae57c5f98c1adfb on peter-gribanov:master.

Thank you very much :+1:


05/23/2018 00:16 : Create Pull Request for branch prTesting_2018052300 -> master

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Coloring Cmdlets according to the Verb.

Summary of the new feature / enhancement
Example:
verb get/find - Green
Get-help
verb add - Yellow
Add-localuser
verb write/remove/delete - Red
Remove-adcomputer
Proposed technical implementation details (optional)
No response

I would personally find that distracting, but if this were to be implemented it should probably be in PSReadline and whichever editor you use for PowerShell. I don't think the Parser should differentiate command tokens with and without a verb.

Ideally PowerShell should support generic formatting with flexible coloring by schemas. It's not like that now.


[firebase_messaging] Ways workaround data message on app terminated?

What is the current workaround in order to receive custom data when the user click on a notification when the app is terminated?
Seems like on Android is not possible to receive a data message in the onLaunch (which is the ideal way)
On IOS i hadn't tried yet since i'm facing this issue first.
Any clue?

It works for me, I tested it on both platforms.

Hey @nerder, I'm closing this as stale and seems a question better suited to StackOverflow using tag Flutter. If this remains an issue then please feel free to create a new issue with as much information as possible.


Remove GPL FFT implimentation

The current version of an fft in Speech/tools/FFT and Speech/tools/FourierTransform is licenced under GPL. We should replace it with something with a more permissive licence.
Potential other versions: https://commons.apache.org/proper/commons-math/javadocs/api-3.4/org/apache/commons/math3/transform/FastFourierTransformer.html

Done #245


dependency err on install

upon sudo npm install -g leximaven, ubuntu linux
npm ERR! Error: No compatible version found<EMAIL_ADDRESS>npm ERR! Valid install targets:
npm ERR! ["0.0.0","0.0.1"]
npm ERR!     at installTargetsError (/usr/share/npm/lib/cache.js:719:10)
npm ERR!     at /usr/share/npm/lib/cache.js:638:10
npm ERR!     at saved (/usr/share/npm/node_modules/npm-registry-client/lib/get.js:142:7)
npm ERR!     at /usr/lib/nodejs/graceful-fs/polyfills.js:133:7
npm ERR!     at Object.oncomplete (fs.js:107:15)
npm ERR! If you need help, you may report this log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>
npm ERR! System Linux 3.13.0-74-generic
npm ERR! command "/usr/bin/nodejs" "/usr/bin/npm" "install" "-g" "leximaven"
npm ERR! cwd /home/ubuntu/leximaven
npm ERR! node -v v0.10.25
npm ERR! npm -v 1.3.10
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/ubuntu/leximaven/npm-debug.log
npm ERR! not ok code 0

Looks like it's a dependency issue with good-guy-http's lib. Back in October it was changed from version 0.01 from npm repo to 0.0.2 with an installation from Github. good-guy-http was also recently updated to 1.8.0. Make sure git is installed on your machine, clone the repo, and do an npm install. If it works, then just do npm install -g and it should work. Let me know how it works!


Haskell Start Croatian

⁉

⁉

@diomed apologies for not getting back to you sooner! I'll merge this and see if I can have someone take a look at it later.
Thank you so much for this!

no problem. this was actually done by following Serbian version.
since Serbian and Croatian are alike. so it's as accurate as that one.
still, it's nice to be able to read it in our own language, so I made adjustments for it.


rocm: support rocBLAS 3.0 trmm with 3 matrices A, B, C

rocBLAS 3.0 in ROCm 5.6.0 introduced 3 matrix trmm, with separate B and C matrices that can be aliased. See
https://rocblas.readthedocs.io/en/master/API_Reference_Guide.html#rocblas-xtrmm-batched-strided-batched
This updates BLAS++ to call the new interface when available.

Tested on Frontier. ROCm ≤ 5.5.1 uses the original 2 matrix trmm. ROCm ≥ 5.6.0 uses the new 3 matrix trmm.


Cue fmt drags comments into import block

Originally opened by @ah-edg in https://github.com/cuelang/cue/issues/720

What version of CUE are you using (cue version)?

$ cue version
cue version 0.3.0-beta.3 linux/amd64

Does this issue reproduce with the latest release?
yes
What did you do?
Run cue fmt on this input:
import "encoding/base64"

// comment

#A: {}

What did you expect to see?
no change
What did you see instead?
import ( "encoding/base64"

	// comment
)

#A: {}

This is same as cmd/fmt: single import and comment combined #1447.
The newline after the comment triggers the bug, as the comment gets associated with the previous import line, then the comment is misplaced into the import block.
In the absence of a newline between the comment and the code following it there is no problem, and the fmt output is same as input.


[Feature request] add imagePullSecrets to pull the customImage from Private Registry

Is your feature request related to a problem? Please describe.
add support for imagePullSecrets to pull the customImage from Private Registry
Describe the solution you'd like
add imagePullSecrets with Image and ImagePullPolicy Similar to deployments

Hello @prasanna12510 ,
Thank you for taking the time to report this enhancement. This is a nice addition to make the operator more feature complete. That being said, atm, this won't make it to the top of my priority list. However, I strongly invite you to open a PR with this contribution and i can review it and work together on it till it is added to the project.
I am assuming that you are looking for something similar to this: https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/

@AbdelrhmanHamouda I could help with this.
Just to be sure, you're talking about the LoadTest resource, right? Another possibility would be adding this to the operator itself - @prasanna12510

solved with https://github.com/AbdelrhmanHamouda/locust-k8s-operator/pull/98


chore: resolve merge conflicts for upstream-58bf161

What :computer:

Resolves merge conflicts for 58bf161bc9dd6e74de8cb61e3ae23f701feb5512

Shuldn't the base be something else? I don't think we want to merge this directly, right?

@Karrq Yes! just changed the base. Like always I put the wrong branch name so I had to redo it


Maximum Likelihood Calibration Experiment

First attempt in-simulation of demonstrating the maximum likelihood calibration method. Documenting here so that it's easier to reproduce on Derp.
Calibrating Polarimeters is tough, gang. What if stats made it easier?
The goal of this experiment is to see if we a statistical model of a polarimeter can be used to calibrate a real polarimeter by finding the maximum likelihood. How do we do this?
In simulation this is easy and should go well if the theory is correct. Say we have some forward model $f$ that is a function of model parameters $x$, which returns a time series of power measurements $p$ observed throughout an experiment.
$$ f(x) \rightarrow p $$
The trouble with polarimetry is that we can never know $x$ exactlty, so we get two expressions - one we see in lab, and another in our model
$$ f(x_{lab}) \rightarrow p_{lab}; f(x_{model}) \rightarrow p_{model} $$
$p_{lab}$ and $p_{model}$ will not necessarily agree if we don't first calibrate the measurement. @wcmelby has done some good work showing that fitting to an air measurement will help us calibrate the measurement. But in all things (especially in high-contrast imaging) we want to go deeper

The only reason that the air measurement would break down (if we ignore spatially-varying effects for a second), is the presence of noise. This is where the statistical model kicks in. We similarly cannot know exactly how our noise will behave, but we can apply a statistical model to constrain the uncertainty it introduces. In the lab, we will assume that we are in the case where we are photon noise limited (though this is subject to some debate when going through a polarimetric null). When dominated by photon noise, we can consider our signal to be principally governed by Poisson statistics, i.e.
$$P(x_i | \lambda) = \frac{e^{-\lambda} \lambda^{x_{i}}}{x_{i}!}$$
Where $P(x_i | \lambda)$ is the probability of observing $x_i$ photons when we expect $\lambda$ photons. I've been thinking of $x_i$ as the measurement and $\lambda$ as the model prediction. If you want to compute the probability of all of your observed polarimetric events occurring, we take a product of the probabilities to get the likelihood
$$ L(\lambda) = \prod_{i=0}^{N} p(x_i | \lambda) $$
This is easier to represent in log-space because those factorials will really blow our numbers up, also it let's us turn that product into a sum, which is nice.
$$ \mathcal{L} = ln (L(\lambda)) = -N \lambda + \sum_{i=0}^{N} x_i ln(\lambda) - \sum_{i=0}^{N} ln(x_i !)$$
For my own edification I implemented $\mathcal{L}$ in Python, but the Poisson probability mass function is available in scipy.stats.poisson.logpmf so I've since switched to just using that.
Okay Jaren that's a lot of math but how does that impact my life?
Hopefully it doesn't, but if you too are on an adventure through the forest of polarimetric calibration - I hope it will make your life easier. So, we have a framework for computing the likelihood of an experiment occurring, and some forward model to simulate an experiment. All that's left is to link them together like a pineapple pen

To compute the likelihood of a polarimetric experiment, we really do just plug all the equations above together, like so
$$\mathcal{L}i = ln(P(f(x{truth,i}) | f(x_{model, i})))$$
And because we are in log space, we can just sum over all of the individual intermediate images to get one big likelihood for the whole experiment
$$ \mathcal{L} = \sum_i \mathcal{L}_i $$
Note that if this is done with spatial resolution (i.e. only sum along the temporal direction), you end up with a Log-Likelihood "pupil"?

I wasn't really sure what to do with this because optimizers need a scalar output so I just summed all the values within the illuminated aperture since log-likelihoods sum anyway. But if anyone thinks this is wrong please air your grievances in the comments below.
In any case, you can feed a guess at your model parameters to an optimizer that aims to max the log-likelihood (or in this case, minimize the negative log-likelihood because optimizers like to minimize) and get the model parameters. I did this for a case where I intentionally fed an incorrect guess, and got the following calibrated values.

I'm attempting to visualize the angles like compass needles because that's how I think of them in my head. Note that the "wvp err" parameters don't really mean anything in terms of the orientation. We just want them aligned.
There appears to be a consistent couple-degree offset in the start angles, which don't actually determine anything about the polarimetric measurement other than a global rotation of the Mueller matrix. One thing to note is that the starting angle of the waveplate is off by 90 degrees (plus that offset), which is not something that I realized the calibration was ignorant too. If we plot the predicted power by maximum-likelihood over the observed mean power, it looks like this:

Now I'm going to try the same experiment with the air calibration to see how it compares

Is there a 90 degree degeneracy in a QWP? I know there is in a HWP, but I thought QWPs were 180 degrees degenerate. Perhaps my memory is poor. I also don't work with QWPs as often..

It could be interesting to carry out the calibration on a per-pixel value in the pupil rather than averaging over it. It might help us identify some of the spatially varying issues in the system.

When you rotate a QWP 90 deg it kind of looks like you flipped the sign of the retardance. So I believe it results in the same power going through a linear polarizer since it's still a quarter wave. Here are the Mueller matrices:
QWP 0deg +retardance
--------------------
[[ 1.000000e+00  0.000000e+00  0.000000e+00  0.000000e+00]
 [ 0.000000e+00  1.000000e+00  0.000000e+00 -0.000000e+00]
 [ 0.000000e+00  0.000000e+00  6.123234e-17  1.000000e+00]
 [ 0.000000e+00  0.000000e+00 -1.000000e+00  6.123234e-17]]
QWP 0deg -retardance
--------------------
[[ 1.000000e+00  0.000000e+00  0.000000e+00  0.000000e+00]
 [ 0.000000e+00  1.000000e+00  0.000000e+00  0.000000e+00]
 [ 0.000000e+00  0.000000e+00  6.123234e-17 -1.000000e+00]
 [ 0.000000e+00 -0.000000e+00  1.000000e+00  6.123234e-17]]
QWP 90deg +retardance
--------------------
[[ 1.0000000e+00  0.0000000e+00  0.0000000e+00  0.0000000e+00]
 [ 0.0000000e+00  1.0000000e+00 -1.2246468e-16 -1.2246468e-16]
 [ 0.0000000e+00 -1.2246468e-16  6.1232340e-17 -1.0000000e+00]
 [ 0.0000000e+00  1.2246468e-16  1.0000000e+00  6.1232340e-17]]

and yeah 100% agree on the per-pixel calibration. Ultimately the cost function still needs to be scalar - but if we include spatial modes then I think we can calibrate them using the max-likelihood method.

How close was your initial guess? You could image that we should easily be
able to get within 10 degrees (and likely much better) given a marking from
a manufacturer.
On Wed, Dec 4, 2024 at 10:44 AM Jaren Ashcraft @.***>
wrote:

When you rotate a QWP 90 deg it kind of looks like you flipped the sign of
the retardance. So I believe it results in the same power going through a
linear polarizer since it's still a quarter wave. Here are the Mueller
matrices:
QWP 0deg +retardance
[[ 1.000000e+00  0.000000e+00  0.000000e+00  0.000000e+00]
[ 0.000000e+00  1.000000e+00  0.000000e+00 -0.000000e+00]
[ 0.000000e+00  0.000000e+00  6.123234e-17  1.000000e+00]
[ 0.000000e+00  0.000000e+00 -1.000000e+00  6.123234e-17]]
QWP 0deg -retardance
[[ 1.000000e+00  0.000000e+00  0.000000e+00  0.000000e+00]
[ 0.000000e+00  1.000000e+00  0.000000e+00  0.000000e+00]
[ 0.000000e+00  0.000000e+00  6.123234e-17 -1.000000e+00]
[ 0.000000e+00 -0.000000e+00  1.000000e+00  6.123234e-17]]
QWP 90deg +retardance
[[ 1.0000000e+00  0.0000000e+00  0.0000000e+00  0.0000000e+00]
[ 0.0000000e+00  1.0000000e+00 -1.2246468e-16 -1.2246468e-16]
[ 0.0000000e+00 -1.2246468e-16  6.1232340e-17 -1.0000000e+00]
[ 0.0000000e+00  1.2246468e-16  1.0000000e+00  6.1232340e-17]]
—
Reply to this email directly, view it on GitHub
https://github.com/UCSB-Exoplanet-Polarimetry-Lab/derp_control/issues/30#issuecomment-2518258946,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEK2SJHSAJL4YOYIJXHRU6D2D5EQVAVCNFSM6AAAAABTAXZNGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJYGI2TQOJUGY
.
You are receiving this because you commented.Message ID:
@.***
com>

In this one I just introduced errors on the starting angles of the PSG waveplates. I could try feed it a random vector and see how it does

one was 45 deg off and the other was like 77 deg off

To me they look like they're pretty close to 90 degrees of from each other, right (the polarizer vs. the waveplates)? Is that true? I'm still surprised they drifted so far. I mean you didn't mention that there is a true degeneracy in the rotation of the whole system.

gah the image upload didn't work - the thing you mentioned about system rotations is exactly right. If we nail down one of the angles (like using a Wollaston 👀) I think the rest will line right up

Fixed the PSA Pol start angle, there's still this small offset unfortunately - and the waveplate retardances are much less accurate

You can actually tell the fit wasn't great from looking at the data, which is nice

Just some more brainstorming - I can't seem to get away from the calibration converging on this exact same offset. Maybe I have an angle-off error somewhere in the code?

This is a decent motivation for designing an Experiments class to hold the state of everything

how are you minimizing?

scipy.minimize the negative of the summed maximum likelihood

Could be a local minimum. Scipy.minimize gets trapped in those easily
On Mon, Dec 9, 2024, 7:57 AM Jaren Ashcraft @.***>
wrote:

scipy.minimize the negative of the summed maximum likelihood
—
Reply to this email directly, view it on GitHub
https://github.com/UCSB-Exoplanet-Polarimetry-Lab/derp_control/issues/30#issuecomment-2528488672,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEK2SJFJCCNSUZKLPW64KCT2EW4VTAVCNFSM6AAAAABTAXZNGWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMRYGQ4DQNRXGI
.
You are receiving this because you commented.Message ID:
@.***
com>


Added notes to explain Heartbeat and VPN Only Online status types

depends-on: https://github.com/balena-io/debugging-masterclass/pull/30

@cywang117

@20k-ultra @cywang117 We already have a PR open by @srlowe #1869 working on this page. There's also a discussion going on how we can effectively improve device statuses. Check out https://www.flowdock.com/app/rulemotion/r-beginners/threads/Mv9usxt9Ryr56qAtLk2DYXoli8I

@20k-ultra Want to close this and add any info as a comment / suggestion to Scott's PR? #1869

leaving open as a  possible candidate. Especially because this uses another pr which I think is a nice connection.

closing in favour of https://github.com/balena-io/docs/pull/1869
cc @srlowe. Take as much parts that you like from this PR to use in yours :)


callback for "On Song Change"?

I can't seem to  find the right callback to detect a song change
I want to highlight a different element in my dom based on the current song index.
So I want to detect when the current song object changes.
Is there a callback like that?
thanks

You can do this with simple CSS. The "amplitude-song-container", which wraps all infos regarding one track, gets the class "amplitude-active-song-container" assigned, once it is the current track. There is also a class (on the same container) "amplitude-playing" / "amplitude-paused" to handle the respective playing status.
Can you solve your requirements with these classes? Or did I misunderstand you and you are looking for a different kind of functionality?

i think you misunderstood me.
I don't want to highlight the song that's being changed. I want to change my site theme based on the band being played right now
so, eventually, I want to add a class to the body element to change the site's colors based on the band's logo scheme.
So, when someone clicks on a song, it's easy to detect. but if the playlist moves to another band, im not able to detect it at the moment (started going over source code to check which events are being thrown)

I need some sort of onTrackChange event.
if you don't have this, I might add an event like that myself. do you want me to propose a pull request if I do?

I understand. There are currently 4 callback methods:
https://521dimensions.com/open-source/amplitudejs/docs#callbacks
Would "before_play" fit your needs?

I tried it and it looks like it is only triggered when clicking a play button. it doesn't trigger on song change if you play the main songs list

I understand! Good to know!!! Thanks!
I fooled around a bit, and the method "syncActiveContainer" as well as "setNewSong" fire a (manually inserted) console.log() each time the song track changes... maybe that's a place to hook up a "syncTrackLayoutFunc"?

@Thatkookooguy & @rowild I added the song_change callback to AmplitudeJS 3.2. The documentation for that is here: https://521dimensions.com/open-source/amplitudejs/docs#callbacks I also added a callback example. Thanks for the idea!

Thanks, @danpastori !


BatchID passed to X factor for batch data

If BatchID is numeric, information is passed to Python and it appears in SHAP plots.

Fixed with latest release


Remove old field class

delete old field class
update bindings
fixup
add back the most important methods
add back some static methods used by group
bindings

this was premature and is superseded by https://github.com/o1-labs/snarkyjs/pull/974


Back out changes to add new info metrics for 6.0.1

This branch will be used to make the release for 6.0.1. I made the minimal changes to the dashboard for it to function in 6.0.1, which means the new formatting changes and fixes that were added for 6.0.2 will be included in this 6.0.1 release.

Some Qs about where these changes are coming from? Can you link me to the 6.0.2 reference branch?

The 6.0.2 branch is just main, we haven't made any changes for 6.0.3 yet

LGTM


Platform tied to 5.3.9 prevents installation of recent packages

I'm new to Symfony so I'm not sure if I'm using composer in the expected way, but on php 5.6 I cannot install modules that require php >5.5:
>composer require peehaa/opcachegui:^1.0
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for peehaa/opcachegui 1.0 -> satisfiable by peehaa/opcachegui[v1.0.0].
    - peehaa/opcachegui v1.0.0 requires php >=5.5 -> your PHP version (5.6.7) overridden by "config.platform.php" version (5.3.9) does not satisfy that requirement.

Reason is in composer.json:
"config": {
	"bin-dir": "bin",
	"platform": {
	    "php": "5.3.9"
	}
},

is this needed?

the goal is to be sure that the demo project can be run on any PHP version supported by Symfony, to avoid WTF from users trying the demo with an old PHP version.
If you fork it and modify it for your need, you can change the target PHP version in your composer.json.

the goal is to be sure that the demo project can be run on any PHP version supported by Symfony

Isn't that already ensured by require php>=5.3.9? config.platform.php breaks composer installs so you are forced to change it (which is possible of course). Took me some 30 minutes to figure out what was even going one as I haven't seen this composer feature used before.

@andig the problem is that:

We build the Symfony Demo package in a computer which has PHP 5.5+
To build the package, we execute a composer update
The require php>=5.3.9 means: "if you computer has at least PHP 5.3.9, then install the highest packages that you can ... e.g. for PHP 5.5"
The config.platform.php = 5.3.9 means: "I don't care what PHP version does this computer use. Update dependencies so they can work in a computer with PHP 5.3.9"

@andig more details about the problem you can find here https://github.com/symfony/symfony-standard/issues/858

:+1: for @javiereguiluz and @stof , I think we should keep the config.platform parameter to the lowest supported PHP version on each project that has to be built as a zipball or any "download&play" thing.
Removing it is your role, as well as reading Composer's documentation

Also you can use the --ignore-platform-reqs option to ignore the config.platform value.

Removing it is your role, as well as reading Composer's documentation

I understand the workarounds. You got me on the docs but thats why I've started with the demo. I still couldn't figure out why requiring the php version isn't sufficient. Any hints?

@javiereguiluz so the reason for this setting is ensuring you get a build packagethat will run on 5.3.9? If thats the case then why not strip the config setting after build completes? That way it will contain compatible packages only and leave the door open for newer stuff?

It makes sense to do that in the installer for the demo too. We already do that for regular installations.

@xabbuh you are right! I've opened https://github.com/symfony/symfony-installer/issues/255 to implement this idea. Therefore, I'm closing this issue as "can't fix here ... but it will be fixed in the installer".


Performance Bug When Using CMake to Build TFLite r2.4 for Android (2x slower inference times than Bazel)

Please make sure that this is a bug. As per our
GitHub Policy,
we only address code/doc bugs, performance issues, feature requests and
build/installation issues on GitHub. tag:bug_template
System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: MediaTek X20 Development Board
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): r2.4
Python version: 3.6.9 (also happens on 3.9.4)
Bazel version (if compiling from source): 3.1.0
GCC/Compiler version (if compiling from source): 7.5.0
CUDA/cuDNN version:
GPU model and memory:

You can collect some of this information using our environment capture
script
You can also obtain the TensorFlow version with:

TF 1.0: python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
TF 2.0: python -c "import tensorflow as tf; print(tf.version.GIT_VERSION, tf.version.VERSION)"

Describe the current behavior
After building benchmark_model for Android and running it on my development board, I get an inference time on MobilenetV2_Quantized of about 240ms when using Bazel built benchmark_model but over 550ms when using CMake built benchmark_model.
*Building is done on Ubuntu 18.04
*MobilenetV2_Quant is from tflite hosted models (.tflite) https://www.tensorflow.org/lite/guide/hosted_models
*Performance is measured on MediaTek A53 dev board running Android 7
Describe the expected behavior
benchmark_model binary should have the same performance regardless of build system
Standalone code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate
the problem. If possible, please share a link to Colab/Jupyter/any notebook.
Clone Repo and Mkdir
git clone https://github.com/tensorflow/tensorflow.git tensorflow_src
mkdir tflite_build
CMake Build
cd tflite_build
cmake -DCMAKE_TOOLCHAIN_FILE=/home/ubuntu/android-ndk-r21e/build/cmake/android.toolchain.cmake
-DANDROID_ABI=armeabi-v7a -DANDROID_NATIVE_API_LEVEL=28 -DANDROID_ARM_NEON=ON
-DCMAKE_BUILD_TYPE=Release ../tensorflow_src/tensorflow/lite
cmake --build . -j -t benchmark_model
Bazel Build (in tensorflow_src dir)
bazel build -c opt --config=android_arm tensorflow/lite/tools/benchmark:benchmark_model
Running on Mediatek dev board
I adb push both benchmark_model binaries to my dev board and run them, but get drastically different speeds. The board uses A53 cores, hence the android_arm argument in Bazel and armeabi-v7a argument in CMake.
Other info / logs Include any logs or source code that would be helpful to
diagnose the problem. If including tracebacks, please include the full
traceback. Large logs and files should be attached.

@terryheo @multiverse-tf could you take a look at this report?

Could you try to enable RUY with "-DTFLITE_ENABLE_RUY=ON" ?
Also you can add more C flags likes "-DCMAKE_C_FLAGS=-march=armv7-a -mfpu=neon-vfpv4 -funsafe-math-optimizations" and "-DCMAKE_CXX_FLAGS=-march=armv7-a -mfpu=neon-vfpv4 -funsafe-math-optimizations"


Dev/ahn nath: styling and adding the backend to the project

Styled sections and improved the UI of some pages.
Create the UI and base template for the contributions page.
Added the backend.

Did you pull the changes from the main, switched to a new branch and then worked further?

Did you pull the changes from the main, switched to a new branch and then worked further?

Yes, but after I made the changes to my branch. I initially forgot to merge it before continuing on my branch and had to resolve conflicts. Let me know if I deleted something relevant.

Yeah sorry I forgot that we dont need OAuth. Its good to go!


Use compile-time length check in std.algorithm.copy

bearophile_hugs reported this on 2014-12-20T14:05:25Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=13882
CC List

n8sh.secondary
yebblies

Description
Using std.algorithm.copy on two sequences that have compile-time length compiles with no errors and always excepts at run-time (when the source is longer than the destination):

void main() pure @safe nothrow @nogc {
    import std.algorithm: copy;
    int[6] source;
    int[5] dest;
    //copy(source, dest); // Not supported yet.
    copy(source[], dest[]); // No compilation error
}

This means the D type system here has compile-time knowledge, that we throw away, turning a stronger static typing in an always run-time error.

So I suggest to std.algorithm.copy use the compile-time lengths when they are available:

void main() pure @safe nothrow @nogc {
    import std.algorithm: copy;
    int[6] source;
    int[5] dest;
    copy(source, dest); // Compile-time length mismatch error.
}

Another usage example, using the improved std.range.takeExactly from Issue 13881 :

void main() pure @safe nothrow @nogc {
    import std.algorithm: copy;
    import std.range: takeExactly;
    int[6] source;
    int[5] dest;
    copy(source.takeExactly!5, dest[]); // OK
}

dlang-bot commented on 2019-05-02T21:09:56Z
@n8sh created dlang/phobos pull request #6991 "Fix Issue 13882 - Use compile-time length check in std.algorithm.copy" fixing this issue:

- Fix Issue 13882 - Use compile-time length check in std.algorithm.copy

https://github.com/dlang/phobos/pull/6991

yebblies commented on 2015-01-24T15:50:55Z
Implementing issue 13185 would make this possible, without specializing for static arrays.  Length of slices of static arrays can be found via const-folding, and if matching length was checked in an in-contract it could be reported at compile-time.


added q mainnet and testnet

Description
Added Q Mainnet and Q Testnet from q.org
Additional Information

[X] I read the contributing docs (if this is your first contribution)

Your ENS/address:
aschenkuttel.eth

@jxom
Hey, is it possible to leave the space in the symbol atleast? Metamask doesn't like symbols outside 2-6 characters and thats apparently a workaround. Connecting to mainnet doesnt work due that.


Error of preprocessing with CoreNLP

Hi Sheng,
Please see the attached log for more details.
(stog) lfsong@c47:tool.amr_parsing_stog$ ./scripts/annotate_features.sh data/AMR/amr_2.0
[2019-09-05 06:18:53,869 INFO] Better speed can be achieved with apex installed from https://www.github.com/nvidia/apex .
[2019-09-05 06:18:53,885 INFO] Processing data/AMR/amr_2.0/test.txt
Traceback (most recent call last):
  File "/data/home/lfsong/anaconda3/envs/stog/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/data/home/lfsong/anaconda3/envs/stog/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/data2/lfsong/tool.amr_parsing_stog/stog/data/dataset_readers/amr_parsing/preprocess/feature_annotator.py", line 205, in <module>
    annotation = annotator(amr.sentence)
  File "/data2/lfsong/tool.amr_parsing_stog/stog/data/dataset_readers/amr_parsing/preprocess/feature_annotator.py", line 75, in __call__
    annotation = self.annotate(text)
  File "/data2/lfsong/tool.amr_parsing_stog/stog/data/dataset_readers/amr_parsing/preprocess/feature_annotator.py", line 63, in annotate
    tokens = self.nlp.annotate(text.strip(), self.nlp_properties)['sentences'][0]['tokens']
TypeError: string indices must be integers

Hi! Did you solve this problem? I have the same issue, unfortunately.

Hi! Did you solve this problem? I have the same issue, unfortunately. QAQ

Python version should be 3.6


Adding in unlocks controller and specs. This should resolve #927.

This should resolve the core issue of #927.  However, it has these known shortcomings (and perhaps more that I haven't identified):

The redirect after an unlock is just the root path (i.e. '/'), and it is set in a controller method.  I'm not as familiar with how the library does configuration and default handling, but this would make more sense as a configurable option.
The messages for the Unlocks controller only have English translations in the locales

@zachfeldman - oh, haha, probably just human error.  Like I said, I'm awful with Minitest - I couldn't figure out the command to test just one spec in isolation (as opposed to the whole file), so I took to commenting things out once they worked.  Please let me know the rest of your thoughts once you've had a chance to review.
Also, as should be clear from the code, I added an API endpoint to let a user trigger an unlock email to be sent to his/her account (a create method in the controller).  The specs cover that as well, but I do want to call it out since it's functionality beyond the scope just of this issue.

No problem, mind uncommenting that so we can run Travis against it?
Yeah that sounds like a code thing to have! Thanks for pointing it out.

All checks have passed. Going to approve this - if one more person does, I will merge.

Nice work there, I feel you about Minitest, however, you did great testing!
But I don't like some copy/pasted code from SessionsController. In fact you didn't take into account a recent change in the providers' query: #964
Can you in some way refactor the first part of the create method to a concern or to the DeviseTokenAuth::ApplicationController ? Let us know if you can't.

@MaicolBen - Haha, yeah, I ripped a lot of stuff from the PasswordsController actually.  The two are very similar!  That provider change has actually not made its way into the PasswordsController yet I see - good reason to abstract it!  Is that the only part of the create action that you were talking about refactoring?  It seems like a fair bit of the library could use some refactoring (no offense; that's the nature of open source sometimes), but I don't want to have the scope of this PR bleed too far.

Ok, submitted a refactor with a new module.  Some parts of the code still aren't really clear to me - for example, this block appears in theApplicationController, but if I try to move it to a separate module, specs start failing.  And minitest unfortunately gives me so many warnings that it's hard to track down what's actually going wrong.
    def resource_class(m=nil)
      if m
        mapping = Devise.mappings[m]
      else
        mapping = Devise.mappings[resource_name] || Devise.mappings.values.first
      end

      mapping.to
    end

You didn't give me time to tell you to not do it if it were too much effort! But you did great work! I will review it. I'm thinking in cleaning it more in the future.

It seems like a fair bit of the library could use some refactoring

Agreed. The code climate score for this project is really embarrassing x_x

@lynndylanhurley - But the test coverage is solid!  I mean for real - I would not have had the confidence to change anything if I didn't have a test suite to run against to make sure I didn't break anything.  Y'all have done an awesome job keeping the test suite in order!

@MaicolBen - Both of those points are valid.  I have incorporated those changes into the PR.

Looks good to me! Love the ResourceFinder abstraction. Merging


Add a tutorial on intro page

We could add how to use this crate :
dependency to add in cargo.toml file
how to import the package in our own crate ?
how to generate private and public keys ?
how to access them and convert them into std::string ?
how to encrypt with the public key ?
how to decrypt with the private key ?

yes, agreed this would be great to have.
On 31. Jan 2019, 11:09 +0100, crocwolf<EMAIL_ADDRESS>wrote:

We could add how to use this crate :
dependency to add in cargo.toml file
how to import the package in our own crate ?
how to generate private and public keys ?
how to convert them into std::string ?
how to encrypt with the public key ?
how to decrypt with the private key ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

I added a basic example of encryption as a starting point to the readme

Please could you expose example of from_components usage?

@builat is there a specific issue with that, you can see an example of it here: https://github.com/RustCrypto/RSA/blob/master/src/key.rs#L547

@dignifiedquire Well no specific issue. But if you have to use RSA for the first time in your life you need to perform some kind of resaerch and do few more mental loops before you understand that you have to perform specific calculations for n,e,d and fi(n) and not just insert just prime numbers.
Anyway awsome job! and thnx for what you done.

I think it will be much better to link some good external materials instead of trying to squeeze all good-to-know knowledge to the crate docs.

My ideal state is to have proper doc comments on the individual methods, which link to the core definitions on wikipedia + other reference material, in addition to have usage examples.

@newpavlov totaly agree for example best material for me was : https://www.di-mgt.com.au/rsa_alg.html

I wonder if this issue is still relevant, and what methods needs attention from this angle. Especially regarding PKCS1v15 encryption.
For example is it needed to describe why there two modules for padding, and one of them exposes only signatures structs?

I also opened #226 with the suggestion of refactoring PaddingScheme into a trait, in which case the PKCS1v15Encrypt padding could be a struct that lives in e.g. rsa::pkcs1v15::EncryptionPadding.

Going to call this addressed by the current usage examples.
If there are specific things missing, please open new issues.


[$250] Expense Details - Square bracket displayed in system message for unapproved expense report

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

Version Number: 9.0.34-2
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Log in as an admin in OldDot.
Create a Collect workspace.
Add user B as an employee.
As the employee, log in to NewDot and submit an expense in the workspace chat.
As the employee, go to the expense details page.
As the admin, go to the expense report in OldDot.
Approve the expense and then unapprove it.
As the employee in NewDot, check the system message when the expense is unapproved.

Expected Result:
The system message should correctly specify that the admin has unapproved the report.
Actual Result:
Square brackets ("[]") are displayed in the system message
Workaround:
Unknown
Platforms:

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

Screenshots/Videos

https://github.com/user-attachments/assets/8cf1dc7d-9d3c-40af-8b9a-2dadad95cb09

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021838747538236628276
Upwork Job ID:<PHONE_NUMBER>236628276
Last Price Increase: 2024-09-25

Issue OwnerCurrent Issue Owner: @DylanDylann

We think that this bug might be related to #wave-collect - Release 1

@anmurali  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

Proposal
Please re-state the problem that we are trying to solve in this issue.
The system message for unapproved if done from OD shows [].
What is the root cause of that problem?
This is a BE problem. When we receive the unapproved message, the object contains [] as the message. After we refresh, it now looks like this:

For unapproved action, we just show what is inside the action message array. It's different with the approved action where we format it manually and localize it.
https://github.com/Expensify/App/blob/51a64cfe90a9314b51a5f3de00c945aeeea4ee1b/src/pages/home/report/ReportActionItem.tsx#L642-L643
We already has the translation, but we never use it.
https://github.com/Expensify/App/blob/51a64cfe90a9314b51a5f3de00c945aeeea4ee1b/src/languages/en.ts#L4191
The translation was added in https://github.com/Expensify/App/pull/43079 but never being used.
What changes do you think we should make in order to solve the problem?

We need to fix this on the FE & BE by using the translations for the unapproved message, so the message is shown correctly and localized.

Add a new check for unapproved action in ReportActionItem

} else if (action.actionName === CONST.REPORT.ACTIONS.TYPE.UNAPPROVED) {
    children = <ReportActionItemBasicMessage message={ReportUtils.getIOUUnapprovedMessage(action)} />;

Create the new function for getIOUUnapprovedMessage

function getIOUUnapprovedMessage(reportAction: ReportAction) {
    return Localize.translateLocal('iou.unapprovedAmount', {amount: getFormattedAmount(reportAction)});
}

Update the translation to move the unapproved within iou object.

iou: {
    ...,
    unapprovedAmount: ({amount}: UnapprovedParams) => `unapproved ${amount}`,
    ...,
}

Update getFormattedAmount to accept unapproved action.
https://github.com/Expensify/App/blob/51a64cfe90a9314b51a5f3de00c945aeeea4ee1b/src/libs/ReportUtils.ts#L4447-L4450

if (... && !ReportActionsUtils.isApprovedAction(reportAction)) {

The changes on the FE is done, but the message will show unapproved without any amount and currency and that's because the report action original message of the unapproved from OD has an empty originalMessage.

Compared to the unapproved action done from ND.

So, we need a BE fix too to return the correct originalMessage.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Square brackets ("[]") are displayed in the system message
What is the root cause of that problem?
When we unapproved the request in OD, we received the pusher data for UNAPPROVED action with message and originalMessage as an empty array

Then because we don't have a special case for unapproved action, it displays the data from html of message
What changes do you think we should make in order to solve the problem?

From backend, we should return the correct data via pusher for UNAPPROVED action

We should add a case for UNAPPROVED action to translate it as we do for approved action. We're had the translation key for this action here

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/pages/home/report/ReportActionItem.tsx#L632

Create a util to check the action is unapproved action or not in ReportActionUtils

function isUnapprovedAction(reportAction: OnyxInputOrEntry<ReportAction>): reportAction is ReportAction<typeof CONST.REPORT.ACTIONS.TYPE.UNAPPROVED> {
    return isActionOfType(reportAction, CONST.REPORT.ACTIONS.TYPE.UNAPPROVED);
}

Create a util to get the unapproved message in ReportUtils

function getIOUUnapprovedMessage(reportAction: ReportAction) {
    return Localize.translateLocal('iou.unapprovedAmount', {amount: getFormattedAmount(reportAction)});
}

Add a case for unapproved action in getFormattedAmount function

if (!ReportActionsUtils.isSubmittedAction(reportAction) && !ReportActionsUtils.isForwardedAction(reportAction) && !ReportActionsUtils.isApprovedAction(reportAction) && !ReportActionsUtils.isUnapprovedAction(reportAction)) {
    return '';
}

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/libs/ReportUtils.ts#L4435

Add a case for unapproved action in ReportActionItem

} else if (action.actionName === CONST.REPORT.ACTIONS.TYPE.UNAPPROVED) {
    children = <ReportActionItemBasicMessage message={ReportUtils.getIOUUnapprovedMessage(action)} />;

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/pages/home/report/ReportActionItem.tsx#L632

Do the same way as we do for approved action to add a case for unapproved action in other places

else if (lastReportAction?.actionName === CONST.REPORT.ACTIONS.TYPE.UNAPPROVED) {
     lastMessageTextFromReport = ReportUtils.getIOUUnapprovedMessage(lastReportAction);
 }

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/libs/OptionsListUtils.ts#L664-L667
if (parentReportAction?.actionName === CONST.REPORT.ACTIONS.TYPE.UNAPPROVED) {
   return getIOUUnapprovedMessage(parentReportAction);
}

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/libs/ReportUtils.ts#L3718-L3720
else if (reportAction?.actionName === CONST.REPORT.ACTIONS.TYPE.UNAPPROVED) {
     const displayMessage = ReportUtils.getIOUUnapprovedMessage(reportAction);
     Clipboard.setString(displayMessage);

https://github.com/Expensify/App/blob/48e9c176b3111870ebe5e6193312334e32c249ea/src/pages/home/report/ContextMenu/ContextMenuActions.tsx#L445-L447
What alternative solutions did you explore? (Optional)
NA

We need to modify the BE to return the correct data
For FE, I agree with contributors about handling translation for unapproved messages. Let's go with @nkdengineer's proposal because their proposal covered all instances
🎀 👀 🎀 C+ reviewed

From backend, we should return the correct data via pusher for UNAPPROVED action

Yep, sent a PR for this in our backend: https://github.com/Expensify/Auth/pull/12615
Not very familiar with the App code for this, so have a few questions:

Create a util to check the action is unapproved action or not in ReportActionUtils

Why do we do this (in general)? Why not have just one method isActionType(reportAction, type)?

Create a util to get the unapproved message in ReportUtils

Both approve and unapprove have the same data format, no? Could we reuse getIOUApprovedMessage instead of duplicating it (maybe changing the name a bit)?

Add a case for unapproved action in getFormattedAmount function

Why do we keep adding ifs there, wouldn't it be better to check if the amount and currency are set?

@iwiznia Thanks for your feedback.

Why do we do this (in general)? Why not have just one method isActionType(reportAction, type)?

We create a new until then we can use this function in other places of the app with only one param. That is similar to isSubmittedAction

Both approve and unapprove have the same data format, no? Could we reuse getIOUApprovedMessage instead of duplicating it (maybe changing the name a bit)?

We can reuse this. We can rename this function, and change the format of unapproved translation to be similar to approve translation.

Add a case for unapproved action in getFormattedAmount function

This check is used to predict the type of action, and then we can access amount and currency safer and easier
Why do we keep adding ifs there, wouldn't it be better to check if the amount and currency are set?

We create a new until then we can use this function in other places of the app with only one param. That is similar to isSubmittedAction

ok, kind of think that having one method for each action type is more work and makes writing code harder (you need to remember/find the names of each action). But since we already do this everywhere, let's keep it.

We can reuse this. We can rename this function, and change the format of unapproved translation to be similar to approve translation.

Awesome

This check is used to predict the type of action, and then we can access amount and currency safer and easier

It is certainly not easier as we need to add each action to the if, I am not sure I am convinced it's safer, what is the risk? That someone calls this method for an "invalid" report action that casually has currency and amount but they contain some different data than the actual currency and amount?

@DylanDylann this PR is ready for review

@iwiznia we have an open PR here but currently Back End doesn't return message and originalMessage when admin unapproved the expense in Old Dot. Please check again

Yeah, the backend PR should get deployed tomorrow.

@nkdengineer Any updates?

@DylanDylann, looks like the PR on the backend has been merged. This PR is ready for review

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
[@DylanDylann] The PR that introduced the bug has been identified. Link to the PR: This change seems like a new improvement, not a bug
[@DylanDylann] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: NA
[@DylanDylann] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: NA
[@DylanDylann] Determine if we should create a regression test for this bug. Yes
[@DylanDylann] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
Regression Test Proposal

Log in as an admin in OldDot.
Create a Collect workspace.
Add user B as an employee.
As the employee, log in to NewDot and submit an expense in the workspace chat.
As the employee, go to the expense details page.
As the admin, go to the expense report in OldDot.
Approve the expense and then unapprove it.
As the employee in NewDot, check the system message when the expense is unapproved.
Verify that: The system message should correctly specify that the admin has unapproved the report.

Do we agree 👍 or 👎

Waiting for @anmurali to complete the payments and close

Not overdue, @anmurali needs to pay

Done.


Add analytics to docs

🗄 Docs report
Description
 A clear and concise description of the problem...
🔬 Minimal Reproduction
What's the affected URL?**

Reproduction Steps**

Expected vs Actual Behavior**

📷Screenshot

🔥 Exception or Error

🌍 Your Environment
Browser info

Anything else relevant?

Duplicate of #673


bug + fix: Could not verify NeverBounce's SSL certificate.  Please make sure that your network is not intercepting certificates

Hi,
What's the best way to pass the local CA file like these projects?
https://github.com/googleapis/google-api-php-client/issues/788
https://github.com/googleapis/google-api-php-client/issues/1011
I didn't find one so I've patched your code: ApiClient.php, method: request.
I put the following snippet somewhere before: $this->client->execute();
Maybe you can come up with a global static method like the one that sets the API key.
Maybe setDefaultOptions or something.
        // Custom fix         if (getenv('NEVERBOUNCE_CA_FILE')) { 	        $this->client->setOpt( CURLOPT_CAINFO, getenv('NEVERBOUNCE_CA_FILE') ); 	        $this->client->setOpt( CURLOPT_CAPATH, getenv('NEVERBOUNCE_CA_FILE') );         }
Usage:
// CA cert is from here: https://curl.haxx.se/docs/caextract.html
$ca_file = __DIR__. '/cacert.pem';

OS: Windows 10
php -v
PHP 7.2.22 (cli) (built: Aug 28 2019 09:29:01) ( ZTS MSVC15 (Visual C++ 2017) x64 )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.2.0, Copyright (c) 1998-2018 Zend Technologies
with Xdebug v2.7.2, Copyright (c) 2002-2019, by Derick Rethans

Hello Svetoslav,
Could provide more details about the error you received? What was the actual error message you received?
Just as a note, we are using Amazon as our certificate issuer. Most up to date applications should trust our certificate out of the box; unless they've been explicitly removed.

Hi,
The error is client side.
See the links I provided
On Mon, Apr 6, 2020, 10:36 PM Mike<EMAIL_ADDRESS>wrote:

Hello Svetoslav,
Could provide more details about the error you received? What was the
actual error message you received?
Just as a note, we are using Amazon as our certificate issuer. Most up to
date applications should trust our certificate out of the box; unless
they've been explicitly removed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NeverBounce/NeverBounceAPI-PHP/issues/43#issuecomment-609995331,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBSVTO6LX3EF24BEX32XDRLIVMDANCNFSM4L2NVAQQ
.

From the links provided it seems like this can be solved by updating your php.ini to include curl.cainfo = "PATH_TO/cacert.pem". I'm not sure this needs to be addressed in this library. Is there any reason the configuration update will not work?

Yes, it needs to be addressed in the library. I've provided a solution via
getenv.
Not everyone will have access to php.ini
On Tue, Apr 7, 2020, 7:18 PM Mike<EMAIL_ADDRESS>wrote:

From the links provided it seems like this can be solved by updating your
php.ini to include curl.cainfo = "PATH_TO/cacert.pem". I'm not sure this
needs to be addressed in this library. Is there any reason the
configuration update will not work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NeverBounce/NeverBounceAPI-PHP/issues/43#issuecomment-610482860,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBSVSE2OGNMGUT7VSPSFDRLNG3ZANCNFSM4L2NVAQQ
.

Fair enough, we should be able to have a team member address this soon.


async-await and async.series problems fixed.

I'm really sorry.
There was a mistake in the commit. The async-await and async.series issues have been fixed.
I tested it thoroughly.

Unfortunately, this update still gives me two callback errors.  (log attached)  It does seem to produce the .md files correctly though.  Could you have a look when you have a minute?
parser-out.txt

Unfortunately, this update still gives me two callback errors. (log attached) It does seem to produce the .md files correctly though. Could you have a look when you have a minute? parser-out.txt

please check again

Works great, thanks!


Add mission_logger package

This pull request refactors the log_toggle package into a new mission_logger package, with plain text and bag based logging functionality for the duration of a mission.

Alright, going in !


Fix base64 deprecated methods in kinesis.utils

Using right encode and decode methods according to python version.
There was an attempt in #1854 before, but python 2 were not covered.


AWS SQS payload limit

Hi,
developing locally I don't have problems.
Moving to the cloud I saw this error:
Message must be shorter than 262144 bytes
I think it could be useful to implement this validation.
In the meanwhile I'll follow this way: https://github.com/awslabs/amazon-sqs-java-extended-client-lib

I found a bug on the local system, there is a check for that.


feat(aws-ecs): add ECS/Fargate QueueWorkerService constructs

Implement an ECS and Fargate Queue worker that creates an sqs queue and fixes Step Scaling implementation to add ScalingTargetId and default AdjustmentType
Fixes #2396

Pull Request Checklist

[X] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[X] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[X] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

Addressed all PR comments, and updated revision.

Modulo the type of command: string[] and the build failures, I feel comfortable merging this.

Update command to string[]. Build failures look like they're happening in the cdk master branch.

Resolved build failures. @rix0rrr can you merge this in? I'll refactor this into the new module after the fact.


Compile issue for RP2040 Connect

Line 149 of WiFiWebServer.h doesn't work with RP2040 Connect on PlatformIO, fix is simple, just add the !defined as shown below:
// KH, For PROGMEM commands
// ESP32/ESP8266 includes <pgmspace.h> by default, and memccpy_P was already defined there
#if !(ESP32 || ESP8266) && !defined(ARDUINO_NANO_RP2040_CONNECT)
#include <avr/pgmspace.h>
#define memccpy_P(dest, src, c, n) memccpy((dest), (src), (c), (n))
#endif
Thanks to maxgerhardt for the fix. More details on this over here, https://community.platformio.org/t/rp2040-connect-wifiwebserver-build-issues/23801/2

HI @ecosky
Can you try again using the new WiFiWebServer releases v1.4.1 ?
It's possible there is some bug or settings not correct in PIO because there is no issue with  Arduino IDE in compiling at all.
The default lib_compat_mode = soft is really bad for multi-platform code, especially for new PIO users. Wrong libraries could be included and creating compile errors.

Releases v1.4.1

Change option for PIO lib_compat_mode from default soft to strict to minimize compile error in cross-platform
Update Packages' Patches for many boards

Could you check if you have that directory 2.5.2/cores/arduino/api in PIO ArduinoCore-mbed core?

The cores/arduino/api folder exists in the PlatformIO package. framework-arduino-mbed of version 2.4.0 is used in platform-rasperrypi. PlatformIO has got a proper copy of the 2.4.0 version of the core.

If the probklem is that the library needs ArduinoCore-mbed version 2.5.2, that's a problem that has to be solved in platform-raspberrypi. The latest version in the package repo is 2.4.1. I've opened issue https://github.com/platformio/platform-raspberrypi/issues/19 regarding this.

@maxgerhardt
Thanks, I still wonder why @ecosky doesn't have api directory when he's using mbed_rp2040 core v2.4.1
Is it possible some more installation steps necessary to follow to get the Complete framework-arduino-mbed-2.4.1.tar.gz, with included api

to get the Complete framework-arduino-mbed-2.4.1.tar.gz, with included api

The file you linked to literally has the api folder.

When he said

I don't see that directory - could you please provide a more complete path? Perhaps I'm looking in the wrong place. Thanks!

He was probably looking for the Arduino IDE path, not the PlatformIO path C:\Users\<user>\.platformio\packages\framework-arduino-mbed\cores\arduino\api. The folder should be there.

Yes he can still get that compile error while having that file. I have that file too

but since the PlatformIO Arduino-mbed builder script does not add the folder to the include path in
https://github.com/platformio/builder-framework-arduino-core-mbed/blob/ceb4f608f9b4834ac4fa5472e1db2a2c2f9d216e/arduino-core-mbed.py#L96-L100
the include error is presented.
I opened https://github.com/platformio/builder-framework-arduino-core-mbed/pull/1 to have that fixed.


Add Addie Lopshire-Bratt

I am adding Addie Lopshire-Bratt to Before I Die with images and text

Thank you for contributing to Before I Die Code, Addie! Amsterdam is a beautiful and vibrant city with a great biking lifestyle, especially in the summertime. I haven't spent too much time in other parts of the Netherlands but the area is rich in history. Living in Europe allows for easy access to many different cultures and languages. Keep upskilling every day so that you can live and explore Europe! If you have any feedback or ideas on the code, please let me know. Your information looks great on the website!


Implement EmailEditorGlobalSettingsTab

wont implement dark mode toggle as this is not an email setting, rather a preview option


Adding code tags around any nwo#number text string

Attempting to improve our link sanitation to prevent <org>/<repo>#<pr number> text strings from being hydrated into full github.com links by PR creation machinery.
I found that a number of our tests produce redirect links, however the link text itself is in the format above, and adding those links to a PR comment or description results in a valid github.com link and creates a mention on the targeted PR timeline.
This should also break up any text string in the above format to prevent flood of mentions experienced in the below issue:
Closes https://github.com/dependabot/dependabot-core/issues/5639
I also added common tests to the test debug dropdown in vscode and fixed a path issue with the vscode debugger.
This is still a WIP as there is 1 failing test which I need to fix, and the approach may be overly heavy handed.  I'm not sure my tests which resulted in lots of valid links were valid, so feedback there would be good.

I've now limited this to only find text strings and not link text.  I am concerned as copy/pasting a link such as: https://github.com/dependabot/dependabot-core/blob/main/common/spec/dependabot/pull_request_creator/message_builder/link_and_mention_sanitizer_spec.rb#L278
results in:
rust-num/num-traits#144\n
I feel like we should probably be escaping the link text as well.

What does nwo stand for?
"name with owner" shorthand for org/repo


Provide synchronous deploy

Hi,
when deploying stacks with ros-cdk deploy, the deployment process is asynchronous, so we need to develop some custom ways of checking what's the status of the deployment.
It would be much more useful if ros-cdk deploy would work in a synchronous manner, which is more natural for transactional nature of the deployment process.
If that's not possible, could you suggest the best way to programmatically monitor the deployment status?

We have received suggestions and will support this feature in the future.
Current solution：

CLI：

Check stack event status via ros-cdk event.
Check stack status via aliyun ROS CLI.
For example: aliyun ros ListStacks --RegionId <region_id> -StackName.1 <stack_name>.

Code：Check stack status via aliyun ROS SDK

ros-cdk-cli 1.0.15 is released. The progress of resource creation is added.


RandomAlivePause RandomPause runs even disabled

I notice these run even disabled in config
RandomAlivePause
RandomPause
Your FULL config.json (remove your username, password, gmapkey and any other private info)

 {
    "//NOTE: This task MUST be placed on the top of task list": {},
    "type": "RandomAlivePause",
    "config": {
      "enabled": false,
      "min_duration": "00:00:10",
      "max_duration": "00:10:00",
      "min_interval": "00:05:00",
      "max_interval": "01:30:00"
    }
  },
  {
    "type": "HandleSoftBan"
  },
  {
    "type": "RandomPause",
    "config": {
      "enabled": false,
      "min_duration": "00:00:10",
      "max_duration": "00:10:00",
      "min_interval": "00:10:00",
      "max_interval": "02:00:00"
    }
  },

Output when issue occurred

[2016-09-16 21:09:07] [RandomAlivePause] [INFO] Next random alive pause at 21:24:19, for a duration of 0:06:03
[2016-09-16 21:09:07] [RandomPause] [INFO] Next random pause at 22:01:19, for a duration of 0:08:44
Steps to Reproduce

start bot
Other Information
Branch:
master
Git Commit:
a8ee31256d412413b107cce81b62059634e8c802

Unless I'm mistaken, that's only an output at start of the bot, it won't actually pause at the specified time.

@kureijir You're right, but it should be fixed anyway.

#5591 is a blanket approach which will not be approved as-is. Suggest fixing the individual randompause and randomalivepause tasks instead.

@Gobberwart It is not a issue, just a messages of scheduling functions called from init.

#5591 merged.


build: add build helpers and basic CI

Info
Adding some build helpers like Makefile and some very basic CI tools, like a build workflow, kodiak, wip and semver bot configs.
Proposed changes

Add dependabot config
Add kodiak config
Add GitHub action for testing and linting any PRs
Add a Makefile
Add go-generate directives to notify.go
Update contribution guidelines

Note
Dependabot will mark PRs that are related to dependencies with the label automerge which should trigger Kodiak to automatically merge the updated dependencies.

@j-wil could you take a quick look and give me feedback if you agree with the changes and additions?

@j-wil could you take a quick look and give me feedback if you agree with the changes and additions?

@nikoksr  I'll take a look at this tonight

hey @j-wil, just wanted to check back to see if you agree with the changes.

@nikoksr  Sorry we landed a couple of new projects at work so I've been underwater. I think everything looks pretty good here

@j-wil hey man, no worries! Hope things go well and thanks for the review.


how to import com.android.location.provider.LocationProviderBase ?

I am writing a custom network location provider like UnifiedNLP . In order to create an unbundled location provider we should inherit from LocationProviderBase . But this class is not in Public SDK . How you import this class in  LocationProviderV2 class ? I saw LocationProviderBase is @hide and we cannot use in code in IDE .

Why are you writing something like UnifiedNLP instead of a plugin for UnifiedNLP for a particular service?   Perhaps you could file bug/enhancement requests for the reasons that are leading you to want to do this.


perf: update dependencies

Alaska Airlines Pull Request
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[ ] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team
Summary by Sourcery
Enhancements:

Update the loader version from 3.1.0 to 3.1.1.

:tada: This PR is included in version 8.1.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


hyperlink link in a tweet

Is it possible to have hyperlink using your bot? if no I would love for you guys to add it thanks

This is the project for the fxtwitter/fixupx domain, not for any 3rd party Discord bots. Unfortunately, website embeds do not allow for hyperlinks on Discord like it does on Telegram.


Specify sequences in S57

Interesting question in mo.  So as specified in Counterexamples, i.e. before your change, the example was really a class of examples with at least $2^\mathfrak{c}$ homeomorphism classes between them.
But the same happens with the Single ultrafilter topology (S111).  Choosing a specific ultrafilter or specific converging sequences for this example determines the topology and different choices may give non-homeomorphic results.  We should definitely mention that.  However the topological properties discussed here are agnostic to these choices.  I feel that being too specific (like choosing the truncation of the decimal expansion of the irrational numbers) is a little bit misleading as this particular choice is never used specifically to prove any property.  Maybe we can leave it out until the day that a particular future property needs a further level of detail to differentiate things?
Also a nitpick: "Specifying ... necessary to characterize a unique topology".  Replacing a sequence with one with the same tail filter would give the same thing.  So maybe rephrase in a simpler way without using the word "necessary".

Actually that could be a good topic to add to the agenda for our next meeting: discussion of generic examples of spaces (i.e. not fully determined up to homeomorphism).

I think this would be a good discussion. Psi-spaces also come to mind.

As of 9612daf I've removed the specification of sequences, and merely note that this choice could be important someday - upon reflection I think it makes sense to only add in such specificity once we have a property that actually depends upon it. I've added this to be discussed at a future meeting.


保存一次,多次编译

[问题简单描述]
保存修改后,小程序总是显示pages/xxxxxx/main.js 出现脚本错误或者未正确调用 Page()
但编译是完全通过的.....
有时候重新修改点内容再保存一次,让它重新编译一下,又不会出现这种情况
如果实在不行的话,重新在控制台npm run dev之后,小程序里面这个报错也会消失
[版本]

"_from"<EMAIL_ADDRESS>"_id"<EMAIL_ADDRESS>"_from": "mpvue-loader@^1.0.13","_id"<EMAIL_ADDRESS>"_from": "mpvue-template-compiler@^1.0.10","_id"<EMAIL_ADDRESS>
[复现步骤]
监听 -> 修改内容 -> 保存 -> cli里面输出多个编译在同时进行(下面的代码)
[cli输出]
$ npm run dev

><EMAIL_ADDRESS>dev E:\Project\test1
> node build/dev-server.js

8080端口被占用，开启新端口8098

Starting to optimize CSS...
 DONE  Compiled successfully in 4383ms19:13:54

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01
 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

 WAIT  Compiling...19:14:01

Starting to optimize CSS...

Starting to optimize CSS...

Starting to optimize CSS...

Starting to optimize CSS...

Starting to optimize CSS...
 DONE  Compiled successfully in 557ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 580ms19:14:02

 DONE  Compiled successfully in 661ms19:14:02

Starting to optimize CSS...

Starting to optimize CSS...
 DONE  Compiled successfully in 680ms19:14:02

 DONE  Compiled successfully in 687ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 666ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 740ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 759ms19:14:02

 DONE  Compiled successfully in 758ms19:14:02

Starting to optimize CSS...

Starting to optimize CSS...

Starting to optimize CSS...

Starting to optimize CSS...
 DONE  Compiled successfully in 816ms19:14:02

Starting to optimize CSS...

Starting to optimize CSS...
 DONE  Compiled successfully in 889ms19:14:02

 DONE  Compiled successfully in 897ms19:14:02

 DONE  Compiled successfully in 947ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 923ms19:14:02

 DONE  Compiled successfully in 961ms19:14:02

 DONE  Compiled successfully in 984ms19:14:02

 DONE  Compiled successfully in 998ms19:14:02

 DONE  Compiled successfully in 984ms19:14:02

Starting to optimize CSS...
 DONE  Compiled successfully in 1087ms19:14:02

是用 mpvue-quickstart 创建的项目吗，有没有对 webpack 配置做过修改，另外提供下运行环境信息，谢谢


fix typo in docs heading

Fix typos in tutti i gusti sono gusti heading.

removes linebreak that terminates heading in README.md
re-apply fix from #395 (missing in index.md, regressed in README.md with 4f8351fd4b05caabf8665f66234d35864b312c38)

Thanks! 116e092fb23ec3df0921e4fb42293fb8de065052


failure in make build complaining about  undefined symbol to python

[exec] /usr/bin/ld: .build_release/src/main/cpp/tools/caffe_mini_cluster.o: undefined reference to symbol '_ZN5boost6python17error_already_setD1Ev'
[exec] //usr/lib/x86_64-linux-gnu/libboost_python-py27.so.1.54.0: error adding symbols: DSO missing from command line
[exec] collect2: error: ld returned 1 exit status
[exec] make[1]: *** [.build_release/src/main/cpp/tools/caffe_mini_cluster.bin] Error 1
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] caffe ............................................. SUCCESS [0.003s]
[INFO] caffe-distri ...................................... FAILURE [47.767s]
[INFO] caffe-grid

Can you check your boost version? BVLC caffe requires boost >= 1.55,
http://caffe.berkeleyvision.org/installation.html,
We have been using boost 1.57 for the entire CaffeOnSpark compilation.

Thank you. Upgrading to boost 1.57 now.

[exec] CXX src/main/cpp/tools/caffe_mini_cluster.cpp
[exec] /usr/bin/ld: .build_release/src/main/cpp/tools/caffe_mini_cluster.o: undefined reference to symbol '_ZN5boost6python17error_already_setD1Ev'
[exec] //usr/lib/x86_64-linux-gnu/libboost_python.so.1.57.0: error adding symbols: DSO missing from command line
[exec] collect2: error: ld returned 1 exit status
[exec] make[1]: *** [.build_release/src/main/cpp/tools/caffe_mini_cluster.bin] Error 1
[exec] CXX/LD -o .build_release/src/main/cpp/tools/caffe_mini_cluster.bin
[exec] make[1]: Leaving directory `/home/bda-master/nhe/CaffeOnSpark/caffe-distri'

After upgrade to boost 1.57 similiar error happened.

Looks like this comes in due to the option WITH_PYTHON_LAYER in caffe-public/Makefile.config.
You can either:

Comment out this option in caffe-public/Makefile.config
If you really need it then you can install boost_python and in caffe-distri/Makefile specify LIBRARIES += boost_python. We will either remove this dependency from caffe_mini_cluster or link to boost_python (will file this as a request)

It looks like a library setting issue. Since we don't use src/main/cpp/tools/caffe_mini_cluster.cpp for spark operation (it is for debugging only), you can just delete it for now and see if the compilation passes.

I got the same error with libboost_python-py27.so.1.55.0...

I will send a patch for this soon. Till then you can follow whatever Jun or I comment earlier in this thread.

This should be solved by the latest merge. Closing it for now. Please reopen if you continue to have problems.

rm src/main/cpp/tools/caffe_mini_cluster.cpp worked