[CI] reuse cargo package cache

Motivation
Reuse cargo package cache between workflows using Cargo.lock as the key.  Net saving is 1 minute in the end-to-end workflow time. It also reduces the exposure to crates.io backend issues we saw yesterday.
Test Plan

First workflow bootstraps the process. Cache misses. It writes the fetched crates into cache.
https://circleci.com/gh/libra/libra/49960
Re-run the workflow. Cache hits. No fetch is necessary.
https://circleci.com/gh/libra/libra/50017
Apply a dummy commit (6cade29 which bumps headers from 0.3.1 to 0.3.2) on top.  Cargo.lock changes and cache misses.
https://circleci.com/gh/libra/libra/50070

Awesome.   At some point the only improvement I could think of would be to always cache on successful master builds and let every build afterwards always take masters cache.   Would prevent downloads of a bunch of crate sources, and cargo would fill in the gaps during it's build.   The reason to only cache on successful master builds is incase of failure to download crates/etc.   It also has the sideeffect of cleaning out the cache.   This was the caching strategy used by travis.ci and has the side effect of ensuring the cache occasionally wipes out old deps.    Also they only maintain one cache file that occasionally gets overwritten.  @bors-libra delegate+

:v: @sausagee can now approve this pull request

Awesome. At some point the only improvement I could think of would be to always cache on successful master builds and let every build afterwards always take masters cache. Would prevent downloads of a bunch of crate sources, and cargo would fill in the gaps during it's build. The reason to only cache on successful master builds is incase of failure to download crates/etc. It also has the sideeffect of cleaning out the cache. This was the caching strategy used by travis.ci and has the side effect of ensuring the cache occasionally wipes out old deps. Also they only maintain one cache file that occasionally gets overwritten. @bors-libra delegate+

Another thing to try - with @metajack's workspace hack, we should be able to build only the 3rd party crates and cache them.

:v: @sausagee can now approve this pull request

Cool stuff

@bors-libra r=rexhoffman

:pushpin: Commit 0b880a3 has been approved by rexhoffman

:hourglass: Testing commit 0b880a32acb120e209cac53e480daea6bf3177e5 with merge 74c16777a0fc6105addcd1cfc529ebf740d60ff0...

:broken_heart: Test failed - checks-actions_land_blocking_test

@bors-libra retry

:hourglass: Testing commit 0b880a32acb120e209cac53e480daea6bf3177e5 with merge ff7b3144ae9c16a772c4a6254857c9b6dd286dc2...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: rexhoffman
Pushing ff7b3144ae9c16a772c4a6254857c9b6dd286dc2 to master...


Returning byte array from plugin function

Hi, I need to return byte buffer from the function, it doesn't matter whether it's allocated by extism or my plugin.

When I tried just to return byte[], it responded with error above.

I'm very sorry, I misunderstood how return values work.

No worries :) feel free to let us know if you have any questions


librsync faiture linking

Describe the bug
I found and installed the librsync library. It was installed as x64-windows-static since it can only compile into a static library. Then I wrote find_library and find_path in my cmakelists. CMake has successfully found both the library and the path to include. But when compiling, the linker cannot find symbols from Librsync
Environment

OS: Windows 10
Compiler: MSVC 19.28.29335.0

To Reproduce
Steps to reproduce the behavior:

./vcpkg install librsync
CmakeList.txt
find_library(RSYNC_LIB "rsync.lib") find_path(RSUNC_INCLUDE "librsync.h") add_library(patcher ${SOURCES} ${HEADERS}) target_link_libraries(patcher PRIVATE ${RSYNC_LIB} ) target_include_directories(patcher PUBLIC ${RSUNC_INCLUDE})
Use any function from librsync
See linking error

Expected behavior
Success linking with librsync

Hi @Amf1k
Thanks for posting this issue.
Have you set CMAKE_TOOLCHAIN_FILE and the triplet x64-windows-static when you compile your project?
In CMakeSettings.json:
"variables": [
        {
          "name": "CMAKE_TOOLCHAIN_FILE",
          "value": "F:\\vcpkg\\scripts\\buildsystems\\vcpkg.cmake"
        },

        {
          "name": "VCPKG_TARGET_TRIPLET",
          "value": "x64-windows-static"
        }
]

Hi! @NancyLi1013
i checked and set VCPKG_TARGET_TRIPLET on x64-windows-static or on x64-windows but it still didn't help. The library is not linked anyway.
My cmake output
"C:\Program Files\JetBrains\CLion 2020.2.4\bin\cmake\win\bin\cmake.exe" -DCMAKE_BUILD_TYPE=Debug -DVCPKG_TARGET_TRIPLET=x64-windows-static -DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake -G "CodeBlocks - NMake Makefiles" C:\Projects\untitled
-- The CXX compiler identification is MSVC 19.28.29335.0
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/BuildTools/VC/Tools/MSVC/14.28.29333/bin/Hostx86/x64/cl.exe
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/BuildTools/VC/Tools/MSVC/14.28.29333/bin/Hostx86/x64/cl.exe - works
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Detecting CXX compile features
-- Detecting CXX compile features - done
C:/vcpkg/installed/x64-windows-static/debug/lib/rsync.lib
C:/vcpkg/installed/x64-windows-static/include
-- Configuring done
-- Generating done
-- Build files have been written to: C:/Projects/untitled/cmake-build-debug

[Finished]

My full CMakeLists.txt
cmake_minimum_required(VERSION 3.5)

project(test LANGUAGES CXX)

set(CMAKE_CXX_STANDARD 11)
set(CMAKE_CXX_STANDARD_REQUIRED ON)
find_library(RSYNC_LIB "rsync.lib")
find_path(RSYNC_INCLUDE "librsync.h")
message(${RSYNC_LIB})
message(${RSYNC_INCLUDE})

add_executable(${PROJECT_NAME} main.cpp)
target_link_libraries(${PROJECT_NAME} PUBLIC ${RSYNC_LIB})
target_include_directories(${PROJECT_NAME} PUBLIC ${RSYNC_INCLUDE})

My main.cpp
#include <librsync.h>
using namespace std;

int main() {
  FILE* file = rs_file_open("test.txt", "rb", false);
  rs_file_close(file);
  return 0;
}

My Complile output:
"C:\Program Files\JetBrains\CLion 2020.2.4\bin\cmake\win\bin\cmake.exe" --build C:\Projects\untitled\cmake-build-debug --target test
Scanning dependencies of target test
[ 50%] Building CXX object CMakeFiles/test.dir/main.cpp.obj
main.cpp
[100%] Linking CXX executable test.exe
LINK Pass 1: command "C:\PROGRA~2\MICROS~1\2019\BUILDT~1\VC\Tools\MSVC\1428~1.293\bin\Hostx86\x64\link.exe /nologo @CMakeFiles\test.dir\objects1.rsp /out:test.exe /implib:test.lib /pdb:C:\Projects\untitled\cmake-build-debug\test.pdb /version:0.0 /machine:x64 /debug /INCREMENTAL /subsystem:console C:\vcpkg\installed\x64-windows-static\debug\lib\rsync.lib kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib /MANIFEST /MANIFESTFILE:CMakeFiles\test.dir/intermediate.manifest CMakeFiles\test.dir/manifest.res" failed (exit code 1120) with the following output:
main.cpp.obj : error LNK2019: unresolved external symbol __imp_rs_file_open referenced in function main
main.cpp.obj : error LNK2019: unresolved external symbol __imp_rs_file_close referenced in function main
test.exe : fatal error LNK1120: 2 unresolved externals

So I noticed that this is a problem I have with all libraries, for example, libtorrent. Exactly the same linking error (only libtorrent specific functions). I have installed libtorrent: x64-windows

I think I figured it out. You need to add define.
add_compile_definitions (LIBRSYNC_STATIC_DEFINE)
Thanks!

Thanks for your above details and further investigation about this issue.
This should be a bug for librsync. I will fix this later. Please keep this issue open until fixed.


gefyra run --no-cache option?

What is the new feature about?
In some cases it might be useful to ensure that the used image during a docker run is pulled freshly from an external container repository. Thus I suggest the addition of a gefyra run "--no-cache" flag
Why would such a feature be important to you?
I run into python dependency issues because the gefyra run command used an 2 month old image on my local machine instead of the newest version in the external repository. I had to delete the local image first in order to get my setup working again. Maybe a --no-cache flag might be useful in some cases to prevent this issue. Furthermore, it might be helpful to shortly document this caching behavior in the CLI docs.
Anything else we need to know?
No response

Good idea! Since we are mirroring docker run with gefyra run I would suggest to go with the --pull flag that is already provided by docker run:
https://docs.docker.com/reference/cli/docker/container/run/#pull
What do you think @buschNT ?

Yes, I agree. The --pull flag seems to be the better fit!


Server Setup not possible without generating private key

It would be nice, if it would be possible to use an existing private key. Thank you!

Thanks for the suggestion! I can't promise that I'll be able to get to these very quickly because I have limited time to work on this project.
Remember that you can always bypass WS4W and work with WireGuard directly whenever there are limitations. The server configuration file is stored at %appdata%\WS4W\server_wg\wg_server.conf. You can edit it manually, then sync it to WireGuard with wg syncconf wg_server wg_server.conf.

Custom private keys can be used for server and clients in the latest release, v1.7.8.


Don't commit (.space) folder to git

to not commit any folder to git, just add .gitignore file contains * to folder
Ex. line105 ioutil.WriteFile(filepath.Join(m.spacePath, ".gitignore"), []byte("*\n"), filePermMode)
https://github.com/deta/space-cli/blob/aa93f1476e56268fab0c4444db1e6326feb8f59f/internal/runtime/manager.go#L105
thnx

hi sorr, i'm not sure i understand this issue. could you please rephrase?


adds feature to retrieve users folders and root playlists

if there is anything more to do to get this into the project please let me know
I'd appreciate a new release with this additional functionality

This seems fines to me. Thanks for the feature.
For now there's not much to do than optimizing (maybe) the codebase that is really trash.
If you have any other features idea in mind, let us know! (my Discord glazk0#1312)

@glazk0 can you bump up the project version so that there will be a new release on npmjs? Thanks


change move buffer size in $swipe.bind  in angular-touch

currently in angular-touch.js the move buffer radius is set to 10 px (set on line 57 in version 1.4.8 and used on line which  173) is too small for some applications particularly ones that draw on a canvas can a third parameter to the bind function be added to increase the buffer size? from what i can tell the move buffer radius determines what is a swipe vs what is a scroll.

Sounds like a reasonable addition. PRs welcome :smiley:

i haven't worked with github that much and i haven't contributed to angular before but i was able to create a personal branch of angular with the changes desired but i don't if or how i could submit those changes in a pull request.

Edit i found out that the move buffer size is dealing with the sensitivity more then determining a swipe vs a scroll so to adjust accordingly i think a new variable should be created to deal with a vertical buffer that separates swipe from scroll

I'm closing this issue since the ngTouch module will be deprecated in AngularJS 1.7.0, and we will not do any more work on it. This is because touch handling is a complex topic, and there are stand-alone libraries that are better equipped to handle all the nuances and edge cases. For example, https://hammerjs.github.io/, which is also used by Angular.


Add prometheus metrics

Replacement PR for https://github.com/Yelp/elastalert/pull/2211

@andres-tw FYI

Hi All, When the metrics part will be available to use. We are all eagerly awiting for this PR to get merge... Please do the needful

Just mentioning this here as well:
https://github.com/abhishekjiitr/elastalert/pull/1#issuecomment-582265932

Can someone assist me with the build issues?

@nicholasgibson2 the code seems to be running fine, flake8 is exiting with NZEC because of some code-style errors:
./elastalert/prometheus_wrapper.py:3:1: E302 expected 2 blank lines, found 1

./elastalert/prometheus_wrapper.py:15:26: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:16:23: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:17:26: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:18:29: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:19:30: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:20:141: E501 line too long (144 > 140 characters)

./elastalert/prometheus_wrapper.py:21:25: E221 multiple spaces before operator

./elastalert/prometheus_wrapper.py:22:141: E501 line too long (144 > 140 characters)

./elastalert/elastalert.py:58:1: E302 expected 2 blank lines, found 1

./elastalert/elastalert.py:108:141: E501 line too long (145 > 140 characters)

./elastalert/elastalert.py:2055:9: F841 local variable 'p' is assigned to but never used

Check detailed log here: https://travis-ci.org/Yelp/elastalert/jobs/648613367?utm_medium=notification&utm_source=github_status

Alright fixed the build issues

@nicholasgibson2 have you thought about submitting this PR to https://github.com/jertel/elastalert ?

@andres-tw I'll check it out!


A design sketch for the dataset creation endpoint

Provides design for FLOC-1152.

Looks fine, thanks.

Please address above as necessary, and then merge (assuming Buildbot is happy, anyway).


add patient_child mapping to elasticsearch

mongodb has DBRef to represent the relationship. how to add this relation to elasticsearch?

This isn't yet supported. See #3.

OK, I see that. thank you for your reply  :)


Error: "Cannot load image from server. Might be CORS related."

While trying to view someone else's track, I got an error message I haven't seen before: "Cannot load image from server. Might be CORS related."
It appears to be related to this specific track, as all others I viewed worked without error.  Track in question: https://openstreetcam.org/details/1122436/

Hey Malcom, we had a temporary problem with one of the storage servers. Back online now.
We're in the process of migrating data to a distributed filesystem to avoid this in the future.

Thanks!

This is probably not a temporary problem... Today, yesterday... - "Cannot load image from server. Might be CORS related." - https://openstreetcam.org/details/1168953/0/track-info

#244

Must be down, because we are still getting this error.

About half of the sequences I've uploaded get that error during processing. Also, none of those sequences have successfully finished processing, and the thumbnail in the tile doesn't show up. Just blank boxes.
They do have the number of images, kilometres, and so forth in the caption; but not the thumbnail images.


Could not build bashtop due to failing clean step

When i tried to build the fsl-imx-wayland distro for tc0710 board yocto resulted in a error during the bashtop do_configure():
| DEBUG: Python function extend_recipe_sysroot finished
| DEBUG: Executing shell function do_configure
| NOTE: make clean
| make: *** No rule to make target 'clean'.  Stop.
| ERROR: oe_runmake failed
| WARNING: exit code 1 from a shell command.

This error is due to the makefile of bashtop not containing a clean target. To fix this i added CLEANBROKEN = "1" to the recipe. This will skip the clean step.
See https://docs.yoctoproject.org/2.4/mega-manual/mega-manual.html#var-CLEANBROKEN

Attached pull request will implement suggested fix


Issues running No-Bling-builder.sh on Linux Mint

I've been trying to get this working on my computer and haven't had much luck. I've hit a few issues, mainly minor ones, but at this point I'm at one that I don't know how to resolve.

# csc compile vpkmod tool via mono if ! type mono > /dev/null 2>&1; then     printf "\033[33m"; read -s -n 1 -p "[ERROR] You need to install mono "; exit 1 fi csc /out:vpkmod /target:exe /platform:anycpu /optimize /nologo vpkmod.cs
I needed to install the chicken-bin package to get the csc command.

csc /out:vpkmod /target:exe /platform:anycpu /optimize /nologo vpkmod.cs
The syntax for this doesn't work with csc from chicken-bin, which as far as I know is the right package. I modified it to csc vpkmod.cs -o vpkmod

After making that change so that csc is pointed to the right file, compilation fails:
Error: (line 156) invalid use of `.'
Call history:
library.scm:3448: print-exit54375438
library.scm:2290: body3981
library.scm:2292: assign
library.scm:3448: current-print-length54395440
library.scm:2290: body3981
library.scm:2292: assign
library.scm:3926: ##sys#print
library.scm:3188: case-sensitive
library.scm:3189: keyword-style
library.scm:3190: ##sys#print-length-limit
library.scm:3297: outchr
library.scm:3188: g5148
library.scm:3927: print-call-chain
library.scm:3882: ##sys#get-call-chain
library.scm:3834: ##sys#make-vector
library.scm:1371: ##sys#allocate-vector	  	<--

Error: shell command terminated with non-zero exit status 17920: '/usr/bin/chicken' 'vpkmod.cs' -output-file 'vpkmod.c'
This is the point where I don't know how to fix the issue.

./No-Bling-builder.sh: 31: read: Illegal option -s
This one's weird because I can run the same command outside of the script with no issues. Anyway, it doesn't look like this line impacts the functionality of the script so no big deal.

that does not look like a proper mono installation
try following this guide:  https://forums.linuxmint.com/viewtopic.php?t=299939
you don't need the monodevelop part

Looks like you're right, it was a bad mono-complete install. Thanks for letting me know. Running the .sh script as you wrote it has compiled and started vpkmod. Right now I'm seeing
 /steamapps/libraryfolders.vdf not found in STEAMPATH
 STEAMPATH = /home/theo/.local/share/Steam

probably because libraryfolder.vdf is in /home/$USER/.steam/steam/steamapps and /home/$USER/.local/share/Steam doesn't exist.

Ok, I hardcoded STEAMPATH in the .cs file and everything is working, including that weird read issue above.
var STEAMPATH = "/home/$USER/.steam/steam";
//var STEAMPATH = SlashPath(Environment.GetEnvironmentVariable("STEAMPATH"));

I'm all set, but I'll leave this issue open in case you want to try to figure out that one line. Let me know if you need any more info. Great work on the code, I'm really excited to play without all the cosmetics.

@ttshaw1 Hardcoding STEAMPATH in the .cs file is not ideal, since it's the file that will see most updates
I have updated the shell script to check a few alternate Steam installation paths, please see if it's enough to fix it
It could also be some issue with the export of the variable, worth changing
mono vpkmod -b
to
STEAMPATH="$HOME/.steam/steam" mono vpkmod -b
in the .sh script

Perfect, latest version works without modification. Thanks!


Compilation error in http_integration.cc

The HttpIntegrationTest::waitForNextUpstreamRequest in test/integration/http_integration.cc has the following code:
  uint64_t upstream_with_request;
  if (!fake_upstream_connection_) {
    ...
        upstream_with_request = upstream_index;
    ...
  }
  ...
  return upstream_with_request;

and it fails to compile with
test/integration/http_integration.cc:364:10: error: 'upstream_with_request' may be used uninitialized in this function [-Werror=maybe-uninitialized]
   return upstream_with_request;
          ^~~~~~~~~~~~~~~~~~~~~

(gcc, opt build)

The fix is trivial; I'll make a PR shortly.


--- Missing Vgui material texturemissing

Hi there!
Okay, so, I recently installed Wire onto my server and everything was working fine until this error came up. It seems whenever I disable Wire for my server, it fixes it but whenever it is enabled, I get this error message in my client console: --- Missing Vgui material texturemissing
What is this and how can I fix it?
Thank you!

This is nothing to worry about, and is expected (if a bit messy).
The surface.GetTextureID API is very lacking - we can ask it for an ID but not check if that ID is valid. So at some point, the code checks:
if CogTexture == surface.GetTextureID("texturemissing") then CogTexture = nil end

The idea being that "texturemissing" is an intentionally missing texture, and so we'd hope it'd have the same ID as any other missing texture, so we can check if CogTexture is a missing texture.
There's probably a better way to do this; I don't know who wrote this code. (Maybe surface.GetTextureID returns some constant that we can just hardcode, or maybe we could just use surface.GetTextureSize to check.)
I'll leave this open but as a cleanup task rather than a bug.

Oh, yeah, I also forgot to mention, and is crucial, Whenever someone tries to connect to the server with that... error, thing... it will just say "Sending Client Info" over and over again. Once, I waited 10 minutes but I still didn't spawn in my server.
Sorry for leaving this crucial information out...

That sounds like a completely unrelated bug. You can confirm by commenting out "texturemissing" line.

Hmm. If it's completely unrelated, then how come whenever I remove Wire, it fixes the server and people can join again but whenever Wire is added to the server, it does that glitch again.
I've also contacted the host of my server and they say that it's a Wire problem as they've looked into the server files also. What could be causing this?

I've never had a problem with wire causing issues on a server and/or other servers I've gone on, you may want to reinstall wire and makes sure all files are validated. As for your host I'm not to sure how they would know, as the most info you can get would come from server console so I'm not to sure how they'd know that's your issue.
Also are you using the workshop version or git version?

I am using the version that the server provides me, right now. But previously, I tried workshop and git versions but they also caused the same problem.

Hmm. If it's completely unrelated, then how come whenever I remove Wire, it fixes the server and people can join again but whenever Wire is added to the server, it does that glitch again.

I'm not saying it's not Wire's fault, merely that the --- Missing vgui material texturemissing issue isn't causing you to fail to connect.

Well... If it's not that... Then I have no clue what Wire is doing to cause the problem. I COULD try and reinstall it (Git and workshop, not together) and see what happens from there?

You should make a new issue for whatever trouble you're having.

Alrighty then. I'll close this issue and create a new one, since I know now that Missing Vgui isn't the problem. Before I make a new issue though, I'm going to reinstall both versions (git and workshop) and if none work, then I'll start a new issue. If it works, you won't be seeing any more issues from me for a while.


fix: Call to a member function set() on null

CalendarComponent::startup callback not fired if load component directly to controller's action.

You made it to the wrong branch, needs to point to my repo.


Multiple getx in code completion

Why getx in my Android Studio IDE show multiple code suggestion with the same code ?, did anyone know how to fix this ?
using
get: ^4.6.6

maybe this solution will solve your problem click


Issue #100: Added feature sort_together with tests

Apologies for introducing the conflicts. If you fetch from this repository and do git cherry-pick 6f9cd5f35129327de5e421da0f305a8beb921789 then you can get my fixes for them.

After fetching and entering git cherry-pick 6f9cd5f35129327de5e421da0f305a8beb921789 I end up with a fatal error: bad object FYI. Thanks for your careful review! I agree with all your points (didn't even know about NORMALIZE_WHITESPACE!). Is there anything you need from me still? Are any of your comments still unfixed? Thanks!

Hm, as far as I can see there are only the first two commits here, so still those changes are needed?

I had a thought to add a usage note to the docstring; I'll push a different PR for that with your commits in it with that addition (plus fixes for the merge conflicts I caused).


In Settings, Product Attributes don't have Link "#"

Hi guys,
I am writing to report a bug, in Settings / Product Attributes, the link is not defined, it does not point to anything "#".

And I would like to be able to translate it to Spanish and not lose the translations when updating (So I need to publish the translation files) /resources/lang/es
I appreciate your help.
Thanks

This is being worked on, I hadn't actually completed the product attributes section as yet.
In regards to spanish translations, would you be interested in creating this and submitting a pull request. I would be happy to include this in the package as right now we don't have anymore work on translations. If you would like some help on how to do this please let me know


An accessor cannot be declared in an ambient context

I'm submitting a ... (check one with "x")
[ ] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
Getting these errors
node_modules/@swimlane/ngx-datatable/lib/directives/long-press.directive.d.ts:16:9 - error TS1086: An accessor cannot be declared in an ambient context.
16     get press(): boolean;
~~~~~
node_modules/@swimlane/ngx-datatable/lib/directives/long-press.directive.d.tsm:17:9 - error TS1086: An accessor cannot be declared in an ambient context.
17     get isLongPress(): boolean;
~~~~~~~~~~~
node_modules/@swimlane/ngx-datatable/lib/services/column-changes.service.d.ts:8:9 - error TS1086: An accessor cannot be declared in an ambient context.
8     get columnInputChanges$(): Observable;

Expected behavior
Application should compile without errors

Reproduction of the problem
Angular version : 8.2.3
ngx-datatable version: latest 17.0

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Mac OS

Table version:

Angular version:

Angular version : 8.2.3
ngx-datatable version: latest 17.0

Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Language: [all | TypeScript X.X | ES6/7 | ES5]
"typescript": "~3.5.3",

Downgraded to 16.0.3 and it seems to be working. Looks like the latest version does not work on Angular 8

Can we fix this for Angular 8? Or how I can get the cssClasses feature in 16.0.3

+1 Angular 8 with TS 3.5 is not working.

Any Update on this? Same error with Angular 10

Any update? Same here:
"@angular/core": "~8.2.11",
"typescript": "~3.5.3"


Chrome's Clicking-Into Gray/Blue Area Doesn't Work

I think the title pretty much sums it up ;)

That is by design. Each location is 1km square. The gray/blue areas are beyond that range and are considered non-teleportable areas.


skani for eukaryotic genomes

Hello Jim，
Another question about skani，which happens to be the weakness of FastANI， very abundant kmers when anchoring，will be filtered for fastANI. This is ok for prokaryotes because nearly all genes are single copy. However， eukaryotic genomes have many copies of genes. Therefore，FastANI can not be used for eukaryotic genomes. I am wondering whether the sparse chaining idea here is subjected to this problem.
Thanks，
Jianshu

Hi Jianshu,
Thanks for the questions. I think this will be helpful for future users.
I don't think skani is suitable for complex eukaryotic genomes, but simple ones it seems to work okay. I do not think skani is suitable for say, human or plant genomes.
The k-mer size is quite small, which means that some seeds are very abundant. It should however work for simpler eukaryotic genomes, e.g. yeast. It even works on Drosophila genomes. However, importantly, I have no guarantees on the accuracy.
FYI: There is actually a repetitive k-mer masking procedure in skani, just like in minimap2, for genomes of size 20 MB that I left in there a long time ago.
Jim

Hello Jim and also Yun,
Thanks for the response. It is clear to me. Another question related to skani is whether it is metric. I know it is symmetric. I think this is quite complicated question because with FracMinHash and also the new seeding and chaining algorithm it is difficult to derive the metric property. FastANI is clearly not metric, which limits its application in large scale searches using some graph/geometric ideas.
Thanks,
Jianshu

skani is not a metric. Consider the following situation: Genome A and genome B are not related at all. But genome C is a 50% mix of genome A and genome B. In this situation, skani would output 100% ANI for comparing A and C, and 100% ANI for B and C, but nothing for A and B (i.e. the triangle inequality with 100 - ANI as the "metric" fails). Of course, skani would also output 50% aligned fraction for each comparison here.
The question of how badly the triangle inequality fails in practice is an interesting question. This is not something I've explored fully.

Ok then this is cleat. Yeah very interesting topic. I think minhash (Jaccard) is metric but hyperminhash? I am considering feeding hyperminhash to some graph based search scheme, which requires metric as input. Still some thoughts but not mature yet.
Jianshu

Hi Jianshu. MinHash and HyperMinHash are both sketches that estimate Jaccard index, which is not itself a metric. The only difference between the two is the space complexity required, but in terms of output, it is basically the same.
If you require a metric, you’ll need something like Jaccard distance = 1 - Jaccard index. But, my initial impression is that it is not a very nice metric, and then you also have to deal with the amount of fuzziness in using a (Hyper)MinHash approximation of it. So it’ll depend a lot on what kind of search scheme you want to feed it into.

Hello Yun,
Thanks for the response. In practice, I am wondering,  as long as we use enough sketches to approximate Jaccard index/distance to a level that the metric property of Jaccard distance is preserved well enough, then we should not worry too much about whether minhash/hyperminhash approximation is 100% metric or not. I am thinking using Hierarchical Navigable Small World Graphs, a nearest neighbor search idea, which requires metric distance as input. Creating/Storing sketches on disk for a very large number of genomes, like a million bacterial genome does take a lot of memory/space (if fully parallelized). Therefore, I am wondering whether replacing minhash with hyperminash could be an advantage. I have been trying ProbMinHash without problems.
Thanks,
Jianshu

HyperMinHash’s only advantage over MinHash is in space complexity. Typically, if you are normally use a 64-bit hash in MinHash, you could pretty reasonably reduce it down to a 16-bit hash using HyperMinHash. Think of it as storing lossily compressed versions of the hash values. Exactly how much compression you can get without losing accuracy depends on the problem domain.
In general, for most applications, I think you could replace MinHash with HyperMinHash, but the question is whether a factor 2-4 savings in the data structure space is worth the additional programming complexity (though the additional complexity is also fairly minimal).


Update sbt and dependencies, drop 2.11

Closed in favor of https://github.com/fs2-blobstore/fs2-blobstore/pull/7


Prints out distorted colors on Mac terminal

Describe the bug
DrawImage.draw_image() seems to fail when using the default Mac terminal. It almost seems that colors are printed at random. However, the output from PyCharm terminal had no issue printing the image.
To Reproduce
Using Mac terminal. I ran the following lines of code:
from image import DrawImage
 pfp = DrawImage.from_url(url-to-image, (32, 32))
 pfp.draw_image()
View from Mac default terminal

View from PyCharm terminal

Device: MacBookPro15
OS: macOS Catalina 10.15.7 19H1417 x86_64

Thanks for opening the issue, I'll look into it

Can you please run the program on another terminal emulator(preferably iterm) and send the results to make sure that the problem isn't with the mac terminal. Thanks.

The reason might be that ANSI colors is disabled in your mac terminal settings. You might have to check that.

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @ShangXoL, @yokgs, @AzureQian, @antx-code, @leesander1, @youngfreshunique, @Mr-Ye-Cao, @schrimel, @x-SERZH-x, @kCorral, @ivanachillee, @shawn-wm, @gavindsouza, @zhy6421867, @Samirdelta, @GNash-2, @jmskov, @fourteentheme, @neelkarma, @mchl18, @wilbertcaba, @rlnash01, @another-green, @spencerpro, @wu8346, @gjb117, @un-simp, @Anthonysidesappss, @ben-gy, @saleh71004, @flxxyz, @eduardomessias, @sleepntsheep, @zh735652914, @disturb16, @strix, @DarrenVictoriano, @indl-coding, @leo-smi, @bonzai-green, @jarrod1020, @r-a-f-a, @REW1L, @ShemJM, @Huangxt57, @p404, @yongtae1975, @misterio-sys, @FireflyTheReal, @nd1511


Does swapi support pagination?

There is no documentation about this.

there's no documentation about it, you are right. but it is paginated. if you consume any resource (https://swapi.co/api/films/) in the result you will see the attribute next and previous that are URLS for the next and previous pages.
hope this helps :)


"The trusted IP address ranges can be private or public."

The article claims that you can use either private or public IPs in the trusted IPs section but in fact you must use the public IP of your network to skip MFA.
How and When can we use private IP range?

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

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @iainfoulds
Microsoft Alias: iainfou

@MarileeTurscak-MSFT Thanks for your feedback! We will investigate and update as appropriate.

Thansk for the feedback, @basseldamra, and sorry for the delay.
Private IP address ranges can only be used when using MFA Server. For cloud-based Azure Multi-Factor Authentication, you can must specify public IP address range. Conditional Access also no longer supports the use of private IP address ranges in named locations. I've created a PR to update the doc to clarify. This doc update should publish later today.


Convert documentation to asciidoc format

Convert documentation to asciidoc format, to be consistent with
AddressBook-Level4.

Staging: https://pyokagan.github.io/addressbook-level1/

@damithc As usual, you'll need to enable travis and setup the GITHUB_TOKEN.
Let me know when you are done so I'll merge this PR to trigger the travis build.

@damithc As usual, you'll need to enable travis and setup the GITHUB_TOKEN.

Done.

Nice, it's working now. 2 more to go.


#20 - February release notes

Closes #20
The list of changes is small this month, that's why I decided to describe docker permissions settings.

@g4s8 @genryxy please take a look, do we have smth to add?

@olenagerasimova probably this one. Unpublish single version in Npm.
This issue was also resolved but I'm not sure if it should be mentioned.
This changes although refer to build lifecycle. This page was added to clarify used tools

@genryxy thanks, corrected

@genryxy also thanks for the additions, I've added about npm unpublish, other changes seem too internal to be mentioned.


SDF Parser only works with double apostrophes in XML

Parser will fail if using single apostrophes in your .sdf, which is the default of gz sdf converter.
Eg. <model name='robot'> fails parsing
but <model name="robot"> succeeds
Quick fix for users is to search and replace all single apostrophes and replace with double.
PS: This project is absolutely fantastic, keep up the great work!

Thanks for pointing it out. I added a command to convert the ' with " to the readme.

Should work without the conversion now


Wiki is missing

README.md says "Details about the features can be found in the Wiki" but that URL simply redirects back to https://github.com/Netflix/ndbench/ and doesn't display a Wiki. Clicking the Wiki link in Github (immediately below the Netflix/ndbench line in the Github UI) has the same effect.

Are you sure about it, I am not seeing that?

I am closing this issue due to inactivity. If there are any more questions, please feel free to respond and we will follow up.


Intermediate node iterables NOT working in fMRIPrep v21: the pipeline did not generate corresponding sets of final results based on multiple sets of intermediate outcomes

What happened?
By making a small modification in t2s.py, which is located in fmriprep/workflows/bold:
original:
t2smap_node = pe.Node(T2SMap(echo_times=list(echo_times)), name='t2smap_node')
modified :
t2smap_node = pe.Node(T2SMap(echo_times=list(echo_times)), name='t2smap_node')   (No change)
(just just add the following two lines to implement a simple node iterables)
fittype_list = ['curvefit', 'loglin']
t2smap_node.iterables = ("fittype", fittype_list)
i.e., simply adding a node iterables procedure into t2smap node to do a loop of two kinds of optimal combinations of the same multiple echo data based on two choices of the model fitting types:  "curvefit"  and "loglin".  By doing so, we can look at the effects of the fitting types on the combined data.  We expect that such a loop can go through the entire processing procedure from this point to all the end so that the pipeline will generate two sets of final results saved into the pipeline output folder "func".   The outcome of running the pipeline is summarized as follows:

The pipeline with the above modification was run successfully without an error

In the working folder (set with -w option when running the pipeline),  we found that two sets of intermediate results (optimal combined data: "desc-optcom_bold.nii.gz")  in fmriprep_wf/....../bold_t2smap_wf, one for "curvefit" with a sub-folder name "_fittype_curvefit/t2smap_node", and the other  "'loglin'" with a sub-folder name "_fittype_loglin'/t2smap_node".

So, the pipeline did work to generate two sets of optimal combined data, "desc-optcom_bold.nii.gz", with the same file name, but saved in two different sub-folders. The two sets of optimal echo-combined data "desc-optcom_bold.nii.gz" should be used in all the remaining procedures.  Unfortunately, in the final outcome folder, we could only find one set of  "_desc-preproc_bold.nii.gz" result.  We assume that the file "_desc-preproc_bold.nii.gz" is based on "desc-optcom_bold.nii.gz", but we have two sets of "desc-optcom_bold.nii.gz", why only one set of "_desc-preproc_bold.nii.gz" ?   And the "_desc-preproc_bold.nii.gz" is generated based on which set of "desc-optcom_bold.nii.gz" then?

Thank you!
David
What command did you use?
docker run -v $PWD:/Work -it --rm nipype/fmriprep:v21.0.0  /Work/bids_input_data /Work/fmriprep_output participant  --participant-label 1  -w /Work/working_dir

What version of fMRIPrep are you running?
v21.0.0
How are you running fMRIPrep?
Docker
Is your data BIDS valid?
Yes
Are you reusing any previously computed results?
No
Please copy and paste any relevant log output.
No response
Additional information / screenshots
No response

It sounds like the iterables worked as expected, but when it came to sink the outputs in the output folder there was no distinction in the file naming, so you probably just have whichever output node ran last.
To do what you want in fMRIPrep, you will need to find a way to pass corresponding metadata to the DerivativesDataSink nodes, which might work by moving your iterables into an IdentityInterface and passing them to T2Smap and DerivativesDataSink. However, that will end up being a somewhat complicated patch that you would need to rebase on any future release you want to use.
A cleaner solution might be to use --me-output-echos and run Tedana tools on the outputs.

effigies:
  Thank you!
  I have found a way to deal with this:   by modifying "desc" (compared to "suffix",  "desc" is flexible to modify),

the two sets of results generated from iterables can be saved with different file names with different "desc" names to avoid being overwritten.

Hi effigies:
More details here:  from t2smap_node, output a fittype variable, then use this  variable to control desc in the output data sink nodes.


Add OSDC Discord Server link

Changes to be made

OSDC has its Discord server at https://discord.gg/2sv322nY9v .
It needs to be added here likewise as for other platforms - https://github.com/osdc/bots/blob/master/telegram-bot/main.go#L187-L188

Hi, If it is the first time that you contribute to the bot, follow these steps:
Write a comment in this issue thread to let other possible contributors know that you are working on this bug. For eg : Hey all, I would like to work on this issue., follow Contributing to the bot 🔥✨ and feel free to ask anything related to this issue ✌.

Hey! I would like to do this :)

Hi @iLorah , thanks for showing your interest to work on this 😁. Are you facing in difficulty in setting up the project or working on this? Feel free to drop your queries here ✌


Remove a spammy log

LGTM.

Test failures are known flakes.
I'm going to merge this one as this is just removing one log line (and that line is messing up our logs).


PHPによるプログラミング教育

What does this PR do?
Add resource(s)
For resources
(https://aue.repo.nii.ac.jp/record/1470/files/kenkyo56205209.pdf)
Description
I added a resource graduation thesis about "Programming education using PHP".
Why is this valuable (or not)?
Because this is valuable for a teacher who wants to teach programing using PHP.
How do we know it's really free?
It's PDF that uploaded on google website.
For book lists, is it a book? For course lists, is it a course? etc.
A book but pdf file.
Checklist:

[X] Read our contributing guidelines.
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[X] Add needed indications (PDF, access notes, under construction).
[X] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Be sure to check the free-programming-books-lint output for linter errors. Alphabetical order and spacing are the most common problems.

6 page journal article is not a book


Debug method not work in go 1.21

In none of example the Debug method works =/

Hi Igor, the default log level is set to Info, try adding the following line at the beginning of the functions: log.SetLevel(log.DebugLevel).

Hi @IgorDePaula, the default log level is set to Info, try adding the following line at the beginning of the functions: log.SetLevel(log.DebugLevel).

Oh, ok, I will try. Thank you very much.

It Works, Thank you very much for your attention.


Schematic blank command doesn't allow to specify the package manager.

Command
new
Description
I cannot see that schematics cli commands added to the command select options.
When you create a new schematic project you only have the ability to specify the name and the author. However, you cannot specify the package manager.
Describe the solution you'd like
The NodePackageInstallTask already allows the package manager option. To support the package manager option we need to do:

We need to modify the factory: https://github.com/angular/angular-cli/blob/main/packages/angular_devkit/schematics_cli/blank/factory.ts in the line 93, we need to add the package manager option.

We need to add the input to the schema.json: https://github.com/angular/angular-cli/blob/main/packages/angular_devkit/schematics_cli/blank/schema.json

Describe alternatives you've considered
No response

I would like to work on this issue. Could you please assign it to me?


Error when trying to search a string column

I have a column called PO-Number which returns string Data with the format [0-9]'-'[0-9].
When I search by the first number it works fine but when I add the minus and a number it crashes and throw me this error
Do I have to add some validation or I am missing something?

I haven't seen this issue before.
The best thing to do is to fork the project, and add a similar case to the example project. If you can repeat the problem there, then I can debug it myself.
You can also post full stack trace, your ViewModel type and maybe a copy of the posted DataTablesParam?

I've Just created a new project and installed the library via nuget package (version 1.3.34)
I created a table and added dummy data.
When you try to search via the PO-Number Column or the date columns I get the same issue.
For example when I try to search 1-1 or 26/4.
Just run the Index in home controller.
I uploaded it in my repository https://github.com/chirislash/Mvc.DataTablesSample

Interestingly this is being caused by searching for a string which can be parsed into a DateTime e.g. 1-1 is parsed as 1/Jan 20116. If you do search for 1-1 in the multi-column search you will only get stuff from that date! Still investigating...

Is there a DataTableParam to indicate that the column wich contains the PO-Number is a string?
Or the DateTime columns are in the format dd/mm/yyyy
I tried with this in the JavaScript:
"aoColumns": [
{ "sType": "numeric" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "date" },
{ "sType": "date" },
{ "sType": "string" },
{ "sType": "string" }
]
But it didn't seem to work

I'll be pushing a fix for this in the next couple of days - in the meantime
you could set [DataTables(Searcheable = false)] on the datetime properties.
I'll update this issue when the fix is ready.
On 28 April 2016 at 18:06, chirislash<EMAIL_ADDRESS>wrote:

Is there a DataTableParam to indicate that the column wich contains the
PO-Number is a string?
Or the DateTime columns are in the format dd/mm/yyyy
I tried with this in the JavaScript:
"aoColumns": [
{ "sType": "numeric" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "string" },
{ "sType": "date" },
{ "sType": "date" },
{ "sType": "string" },
{ "sType": "string" }
]
But it didn't seem to work
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/mcintyre321/mvc.jquery.datatables/issues/145#issuecomment-215496621

It's ok and thanks for the support :v:  :octocat:

No problemo!

(Better keep this open as a reminder for pushing the fix)

This is fixed now, please update your packages.

Thanks man you are awesome!


dclexperience: billboard conversation does not work

Explorer Alpha build version:
Issue Description:
The NPCs having a conversation with billboard bubbles does not work. The conversation does not happen, and the bubble does not face the camera
Unity Renderer

Explorer Alpha

Steps to reproduce:

Expected behaviour:
Current behaviour:
Reproduction rate:

[ ] < 10%
[ ]  < = 25%
[ ]  50%
[ ]  < = 75%
[ ]  100%

Severity:

[ ] Cosmetic
[ ] Minor
[ ] Medium
[ ] High
[ ] Critical
[ ] Blocker

Priority:

[ ] High
[ ] Medium
[ ] Low

Platforms / OS:

[ ] Windows 11
[ ] Windows 10
[ ] Mac Sonoma
[ ] Mac (specify the version on Additional Notes)

Evidence:
Additional Notes:

I found some inconsistency with SDKTransform and TransformComponent (or maybe it's just an effect from PBBillboard)

Fixed in https://github.com/decentraland/unity-explorer/pull/1251


Problem with remote read

Hi, I've configured promemtheus 2.7.1
remote_write:

url: "http://<IP_ADDRESS>:9201/write"
remote_read:
url: "http://<IP_ADDRESS>:9201/read"

I set retention period - 1d in Prometheus
I see that data writing to PG (TimeScale DB), but grafana and prometheus UI don't show data older 1d.

Not sure if this is a bug, will investigate in the meantime your metrics are in TSDB have you tried pointing Grafana at this using the postgresql connector and building your dashboards this way?


loading library gives Uncaught TypeError: Cannot read property 'appendChild' of null

When loading mo.js I get this error:
Uncaught TypeError: Cannot read property 'appendChild' of null

on this line:
  Helpers.prototype.vars = function() {
    var ua;
    this.prefix = this.getPrefix();
    ...
    this.div = document.createElement('div');
    return document.body.appendChild(this.div);   <<<<<<< error here.
  };

What is the correct way to require/initalize this library?

I'm just trying to include it into a plain vanilla HTML page.

Hey @suranyami, sorry to hear that!
Please move the script link into the body (better to the end of the body tag).
Hope it helps.

I'm having the same problem. Trying to get this working with angular.

@firefountain you are linking the library inside body tag?

I'm trying to move the whole angular logic after that. Give me a few minutes

@legomushroom if i move the whole thing to body it's working fine.

@firefountain ok cool, thanks for your feedback.

@firefountain that's because mojs use body tag as a fallback for parent option to place it's modules, thus if you link the library in head tag, the body isn't initialised yet and you get this error.

Shouldnt though be a blocker when you have your javascript outside of body!

Really cool job though. Congrats on this achievement!

Hi @suranyami,
As described by @legomushroom, loading library into the document <head> is inappropriate if the library need to refer to the <body>...
I suggest you to move the <script> tag on the bottom of your document, or add a simple DOMContentLoaded event to check for the <body> availability, and load the library according to that.
Improving helpers in a next version should fix this issue.
Closing for now.


Radar - Change Labels size

Hi, I search a lot on internet and cannot find the solutions.
I want to change the label font size from the radar sample exemple :

Running
Eating
Drinking
Sleeping
etc.

I tried "pointLabelFontSize" in all position without success
image 3

@magicyoda please add a fiddle with what you tried

Here is the fiddle : https://jsfiddle.net/1z28s9aq/1/
on line #43, 44, I added without success
    pointDot: false,     pointLabelFontSize: 20
from http://stackoverflow.com/questions/31522001/chart-js-change-size-of-label-of-radar-chart

Here is the fiddle : https://jsfiddle.net/1z28s9aq/1/
on line #43, 44, I added without success
    pointDot: false,     pointLabelFontSize: 20
from http://stackoverflow.com/questions/31522001/chart-js-change-size-of-label-of-radar-chart

@magicyoda those options are for v1. I updated your fiddle here: https://jsfiddle.net/rpr2n9n7/2/

thanks a lot ! I tried this one also but certainly with error.. ;-)

@etimberg Hi, awesome answer there, I really wonder where did you get that information?
I can't find any in the docs.
I need to change some configuration of the radar chart.
https://s19.postimg.org/5079i7gfn/radar-need-to-config.jpg
Thank you :)

options: {
  scale: {
    pointLabels: {
      fontSize: 20
    }
  }
}

@chenhailun2006 Thanks to you, I did it !

in ChartJs 3.4.1, a lot has changed, now specify
options: {
scales: {
r: {
pointLabels: {
font: {
size: 18,
},
},
},
},
to change the size of the radio chart labels.
https://www.chartjs.org/docs/3.4.1/axes/radial/#point-labels
}


Generate AuditEvents for PIXm, PDQm, MHD, mCSD and accept/forward Record Audit Event [ITI-20]

The receiving actors should generate FHIR AuditEvents according to the specs (currently it generates IHE ATNA logs) and send them to an FHIR Audit Record Repository

The MAG itself implement the Record Audit Event [ITI-20] as a RESTful ATNA Audit Record Repository and forward those AuditEvent to an external FHIR Audit Record Repository

IPF uses for that org.openehealth.ipf.commons.ihe.fhir.support.audit.marshal.AbstractFhirAuditSerializationStrategy which you need to configure


Server crashes on node.js > v14

Using iOS with frida 14.08 , selecting an app in the web application triggers a proxy error:
Running Ubuntu 18.04 with nodejs 14
Proxy error: Could not proxy request /socket.io/?EIO=3&transport=websocket from <IP_ADDRESS>:8081 to http://localhost:31337/.
See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (ECONNREFUSED).
Proxy error: Could not proxy request /socket.io/?EIO=3&transport=websocket from <IP_ADDRESS>:8081 to http://localhost:31337/.
See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (ECONNREFUSED).
Proxy error: Could not proxy request /api/remote/add from <IP_ADDRESS>:8081 to http://localhost:31337/.
See https://nodejs.org/api/errors.html#errors_common_system_errors for more information (ECONNREFUSED).
Please advise.

This means that the server has crashed. I've noticed the problem and it seemed to be related to WebSocket.
Still trying to figure it out.
A temporary workaround is to restart the server. You can either:

Ctrl - C to close all tmux panels and run npm start again
Add a comment or any code with no side effects to the server/app.ts , save it to trigger the autoreload, and nodemon wil reload the server side

The second workaround is faster but you have to make sure that your changes won't break the server

I've pushed a workaround for this to avoid accessing the icon of the devices.


[not finished] Rework Network Stack, Rewrite Init, Implement Tor

Adds support for running as and connecting to Tor hidden services. *.onion addresses are mapped into the IPv6 range in the same way as OnionCat does.
See doc/Tor.txt for more information.
Taken from pull #1174

This needs some edits before it's merged, as it throws some errors upon compilation.

Have you also been able to test this? I can see if I can get a onion node up and running if you need?

@mitchellcash I spoke to @jwrb last night, he did not get to test this in detail.

Just a general note, it's best if each commit is build-able (in order of course), so probably be best to go back and start with what's needed first.

Yeah, I'll close this down and go individual. Although we've got some changed (but not all) this implements. So it'll be a difficult call to get working

@jwrb yeah I know we have some of this stuff at least in part, previous devs liked to take only parts of features instead of all of them.


Update developer-setup.md

Updated path to webapp folder to runnvm install

Summary

Ticket Link

By the way, we can now remove the
 cd mattermoset-webapp in step 7

One more thing to note: if the mattermost-server is not running (step 7 in https://developers.mattermost.com/contribute/server/developer-setup/) then step 8 will fail.
I propose adding a new step after 7 that links back to the developer-setup page as well as has instructions to open a new terminal session and cd'ing into mattermost-server and doing make run-server.

Thanks for the contribution, @dontoisme!
Feel free to make the other changes you suggested in this PR; they sound good to me. :smile:

@dontoisme - I've applied your feedback. Thank you! Really appreciate the thorough, hands-on documentation updates!
The malformed link you identified is present in all of the final step of all OS setup steps to the Web App Developer Setup. I'm able to reproduce this locally, but the link follows our standard convention. I'll see if my commit generates a new preview that shows the links working to rule out whether it's simply a caching issue, and investigate further if needed.


False positive - Cloudwatch log metric pattern is invalid

Currently failing in the notification-terraform repo.
pattern        = "{ $.delivery.providerResponse = \"Blocked as spam by phone carrier\" }"

Log: https://github.com/cds-snc/notification-terraform/runs/5586297935?check_suite_focus=true#step:8:444

Fixed in main branch


allow specifying an ACL token for consul interaction

Consul can optionally restrict changes to clients with ACL tokens. This allows using cortex if consul is configured that way.

Nice @BertHartm!  LGTM
(CI didn't run on this, I guess thats not enabled on your CircleCI @bboreham?)


legendPosition:'right' not working

In NVD3 library legendPosition: 'right' is working but using the directive it is not working
Link of nvd3
https://github.com/novus/nvd3/pull/951
https://gist.github.com/rreinhardt9/6b055863fcf41c76a5ce

I'm also facing the same issue

It works with the latest version [1.0.4].


Signing JWT via ssh-agent

Is it possible to sign a JWT via ssh-agent instead of having access to the private key which may be locked with a passphrase?

I managed to do it my writing and using my own signing method.


Link Ferret with NetCDF4.1.2

Reported by ansley b manke on 5 Jan 2011 18:33 UTC
Set to be released from Unidata shortly.
Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1780

Modified by ansley.b.manke on 26 Aug 2011 18:11 UTC


Corrections to your plugin.

Remove an unnecessary register_asset line from plugin.rb
Whitelist all of the classes you need for displaying the overwatch fonts

Fantastic. Thank you!


Omit returning clause when no default fields (PostgreSQL)

Problem
When a ormx::Table does not have any fields marked as default, it would generate a SQL insert statement with a RETURNING clause that does not have any arguments. This results in a database error.
Example
#[derive(Debug, SimpleObject, ormx::Table)]
#[ormx(table = "addresses", id = id, insertable)]
pub struct Address {
    id: Uuid,
    address_line1: Option<String>,
    address_line2: Option<String>,
    city: Option<String>,
    province_code: Option<String>, // ISO 3166-2
    country_code: Option<String>,
    postal_code: Option<String>,
}

error: error returned from database: syntax error at end of input
   --> src/domain.rs:113:31
    |
113 | #[derive(Debug, SimpleObject, ormx::Table)]
    |                               ^^^^^^^^^^^
    |
    = note: this error originates in a macro (in Nightly builds, run with -Z macro-backtrace for more info)

Database logs:
2021-07-22 20:14:16.009 UTC [57] ERROR:  syntax error at end of input at character 152
2021-07-22 20:14:16.009 UTC [57] STATEMENT:  INSERT INTO addresses (id, address_line1, address_line2, city, province_code, country_code, postal_code) VALUES ($1, $2, $3, $4, $5, $6, $7) RETURNING

Solution
When ormx table has no default fields:

Omit the RETURNING clause
Use sqlx's execute function instead of fetch_one (as the latter expects atleast one returning value)

Ah, yes! This is still here because, in an earlier version, ormx was designed for tables with database-generated IDs, so there was always at least one generated column.
Great work, thanks!


feat: proof of concept to inject react components in md files

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

Closes #34494

@ronaldcs Could you redo this component like it done in this example https://github.com/freeCodeCamp/freeCodeCamp/commit/5a8f9bff4134519b477eb7db85940597c3f8a78b

@ValeraS sure, I can work on that.

https://github.com/freeCodeCamp/freeCodeCamp/pull/35278 replaces this PR relative to moving bootcamp cost calculator to guide.


BUG: Preserve columns in cum_returns.

Fixes a bug where we failed to maintain columns when calling
empyrical.cum_returns on a DataFrame.

It looks to me like https://github.com/quantopian/pyfolio/blob/master/pyfolio/tests/test_perf_attrib.py#L176-L180 should have caught this?


Create page 2-farlynx01

HTML file showing information as it may come in from various sources.

Need a better description, and check links back to main page.


Upload build artifacts.

Supported on Linux x64, Windows x86/x64, and macOS x64. You can see the output of this here (scroll down): https://github.com/e3ndr/webview-devel-fork/actions/runs/7265728392

This should be adjusted to a more reasonable value in the main repo as well. The default of 90 days is kind of a lot...

To clarify: this uploads the generated dll/so/dylibs/objs to the action summary upon completion.

Can you clarify the use case?
I'm asking as access to workflow artifacts is fairly limited. Sharing the use case can help to make sure it is met in the intended way.

I talked about this in the Discord, but the idea is that it makes it easier to adopt WV because you'll have fresh libraries available.

Also, I believe Actions Artifacts are available as long as you're signed into GitHub, though I might be wrong.

@e3ndr I found a related question on our Discord server from May 30, 2023, where you asked whether you could create a CI job to automate builds of shared libraries. I can also see that there were some positive responses back then.
Since then I have rewritten the build scripts:

#957
#969

The current build scripts already build shared libraries for Linux, macOS and Windows. This PR appears to use those and upload them as artifacts.
In my opinion we should distribute official releases (tagged in SCM) at places where binaries exist long-term, and upload less frequently than per Git commit.
If the use case is to make development builds of the library available for others then I imagine that the "artifact log retention" is going to be another issue because artifacts can exist for longer than needed or cease to exist before that.
Distributing one set of of binaries built on one specific Linux distro is problematic as well.

In my opinion we should distribute official releases (tagged in SCM) at places where binaries exist long-term, and upload less frequently than per Git commit.

If the use case is to make development builds of the library available for others then I imagine that the "artifact log retention" is going to be another issue because artifacts can exist for longer than needed or cease to exist before that.

Definitely agree that tagged releases should also have this. Maybe frequent pre-releases would be a better way of allowing developers to get their hands on fresh features/fixes?

Distributing one set of of binaries built on one specific Linux distro is problematic as well.
Ah, didn't know that. I'm pretty out-of-the-loop when it comes to development/consuming libraries on Linux.

I propose that this PR get closed and I'll work on a new auto-builder that only uploads artifacts on a tag/release (including pre-releases) be made for the following targets:

Windows

x86
x86_64

macOS

aarch64 (cross-built on Linux to avoid needing to pay for an M-series mac)
x86_64 (can also be cross-built)

Linux (with separate MUSL and GNU binaries)

x86
x86_64
riscv64
armel (a.k.a armv7) (Useful for RaspberryPis and other SBCs)
armhf (this is also technically armv7, but with an FPU) (Useful for RaspberryPis and other SBCs)
aarch64

For arm/riscv compilation on Linux, we can use either this Alpine-based runner (which can do both MUSL or GNU) or this runner one specifically designed for SBC development (which would probably be GNU-only).
Happy holidays to everyone here btw <3

@e3ndr Targeting more systems would be quite nice indeed. Building with MUSL would be pretty cool as well but I'm not sure about the C++ runtime library.
Can you please explain what you mean by a new auto-builder? Are we talking about major changes to the existing scripts?
If there will be major changes then maybe it's time again to consider a proper build system because things can get complicated rather quickly, and someone has to maintain the scripts. I would personally consider having CMake and Meson support good enough.
I've tried to adapt the project for CMake a few times, and each time I was disappointed by the complexity of my CMake scripts.
Uploading releases is good, and even better if we can both verify that a to-be-release is good before publishing it, and require an authorized human to explicitly allow a release to be published. The SCM tag should be signed, and preferably the files as well.
As for runner images, I prefer to minimize third party software pulled in automatically due to increased risk of supply chain attacks.

Targeting more systems would be quite nice indeed. Building with musl would be pretty cool as well but I'm not sure about the C++ runtime library.

What do you mean?

Can you please explain what you mean by a new auto-builder? Are we talking about major changes to the existing scripts?

I'd use GitHub Actions to invoke the current build scripts and use the automation tools there to attach compiled libs to the GH release files.

As for runner images, I prefer to minimize third party software pulled in automatically due to increased risk of supply chain attacks.

Understandable, I can fork and maintain those other runners to ensure quality if you like.
As for general maintenance, I plan on moving my org @Casterlabs to Webview rather than the custom solution we currently use. So I'll be able to dedicate a lot of time to this project since I will be directly consuming it in a production-grade program.

Targeting more systems would be quite nice indeed. Building with musl would be pretty cool as well but I'm not sure about the C++ runtime library.

What do you mean?

Since this project uses C++ extensively, I wasn't sure what to do about the C++ standard library while we were on the topic of using musl (C standard library).

Targeting more systems would be quite nice indeed. Building with musl would be pretty cool as well but I'm not sure about the C++ runtime library.

What do you mean?

Since this project uses C++ extensively, I wasn't sure what to do about the C++ standard library while we were on the topic of using musl (C standard library).

Ah sorry. Again, my lack of knowledge around stdlibs is showing.

Closing this to avoid polluting the repo. We can continue the talk on Discord :)


Division using dec sometimes make a wrong answer

It looks like this happens with numbers greater than 340.
right answer 1
» 339dec / 340

0.997058823529411764 : Dec

right answer 2
» 340dec / 340

1. : Dec

wrong answer 1
» 341dec / 341

1.152921504606846975 : Dec

wrong answer 2
» 341dec / 342

1.152921504606846975 : Dec

very intriguing. Not surprised something is broken here given it is so untested. Thanks for the bug.
My guess is we are doing something wrong with scale of intermediate computations. That is then leading us to losing information. Which finally leads to the bugs.

@bhansconnect
Thank you for the information !
I'm trying to fix this because this might block what I'm working on (#5218).
I'm not sure if this is even a problem but in the function you mentioned, I see, mul_u128 is used to get U256 version of numerator. And if you give it<PHONE_NUMBER>00000000000 and<PHONE_NUMBER>000000000 as its argument, it returns U256{ .hi = 1, .lo =<PHONE_NUMBER>61536536625392568231788544 } which I think means 340,999,999,999,999,999,999,999,999,999,999,999,999 but is it expected ?

I think that number is correct
low:<PHONE_NUMBER>61536536625392568231788544 -> 0x008A360AE2C66FE7D3BAE05000000000
hi: 1 -> 0x1
full: 0x1008A360AE2C66FE7D3BAE05000000000 -><PHONE_NUMBER>00000000000000000000000000000

Oh I got it. Thank you so much.


-DestinationPath is optional

It is not mentioned what happens if it is not speficied.
Does it automatically use the parent path?

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

ID: 4131d8dd-33e8-0f5b-898f-0a0d65ddeba9
Version Independent ID: 05fc922a-bde2-93ff-7515-c2f14d820fad
Content: Merge-VHD (hyper-v)
Content Source: docset/windows/hyper-v/merge-vhd.md
Product: w10
GitHub Login: @coreyp-at-msft
Microsoft Alias: coreyp

Hi @BlackV
Thanks for reaching out!
Yes, the -DestinationPath parameter is optional. If we don't mention it in the command, it transfer the contents of the differencing disk into its parent. The differencing disk is destroyed at the end of this operation.
Hope this information is helpful. Kindly confirm. Thank you.

Hello @BlackV ,
We are closing this issue since your support question seems to be answered. Feel free to re-open this issue if you still experience problems.
Thanks.

@officedocsbot close


[pgadmin4] Creating users from code

Describe the solution you'd like
Create pgadmin4 users from the Helm chart with secrets with the main pod
Describe alternatives you've considered
Create pgadmin4 users from the Helm chart with secrets with sidecar
Additional context
I am wondering if it is possible to create users, aside from admin, without using UI, so they are pre-made? Currently we are facing issues when using multiple environments or when the chart is deleted then the users get lost. It would be nice if we could define them in Helm chart, similar to admin, where it just takes the password from existing secret and can prepare them. So it would be easy to replicate in multiple environments and so there is no need to make all users again manually in case of disaster recovery scenario.
Maybe others have faced similar issues and have some other workaround that they can share?

Hi there!
We are facing the same issue. I am trying to execute preconfigured users with cli (you can use the config_local.py to execute from python with the "os" library in the chart), example:
/usr/bin/python3 ./setup.py add-user<EMAIL_ADDRESS>password
Traceback (most recent call last):
  File "/pgadmin4/./setup.py", line 24, in <module>
    import config
  File "/pgadmin4/config.py", line 32, in <module>
    from pgadmin.utils import env, IS_WIN, fs_short_path
  File "/pgadmin4/pgadmin/__init__.py", line 24, in <module>
    from flask import Flask, abort, request, current_app, session, url_for
ModuleNotFoundError: No module named 'flask'

I followed this guide and it should work. Should I open a dedicated issue? Is it a pgadmin application bug?
Thanks

This chart is using the official pgadmin4 docker image, these issue are related to that image and not based on the helm chart.

This chart is using the official pgadmin4 docker image, these issue are related to that image and not based on the helm chart.

Thanks for your quick response :+1:


AbstractLichenBlock is too narrow of a name

From @Gaming32 on Discord:

Since it's used for non-lichen as well
In Mojmap, it's called MultifaceBlock

Can I contribute on this?

Can I contribute on this?

of course you can! The current phase of this issue is discussion, so feel free to leave your ideas in this comments section.


ci: merge staging to master

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

Pipeline Attempt on 555639130 for c2b2ac5ff67993456fdd5eca4c5d325d14cb6b39
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/555639130

Pipeline Succeeded on 555639130 for c2b2ac5ff67993456fdd5eca4c5d325d14cb6b39
https://gitlab.com/MatrixAI/open-source/typescript-demo-lib-native/-/pipelines/555639130


Fix deprecation warnings due to invalid escape sequences.

All Submissions Basics:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you checked all Issues to tie the PR to a specific one? #160

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

Pull Request Test Coverage Report for Build 1340

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

Totals

Change from base Build 1324:
0.0%

Covered Lines:
4115

Relevant Lines:
4315

💛  - Coveralls


Leaderboard?

Most of the major benchmarks would have leaderboard to rank LLMs against one another other, but not this one? Where could I find such a result?

Maybe you can find it in some near paper which uses this dataset.I am looking for it too.


Translate

I translated README into Simplified Chinese.
If it's OK, I would like to translate wiki and UI next.

Nice thank you very much for this translation. Feel free to go ahead and also translate the Wiki, it would help a ton!

(Omg, you have already merged. I didn't received the notification. 😳)


Initialise pusher before rendering children

Explanation of Change

This change starts Pusher initialisation before any children of AuthScreens is rendered allowing them to call safely subscribe.  When initialisation was triggered in useEffect it was run after children was rendered leading to error. Now initialisation is triggered during first render and error doesn't happen.
Fixed Issues

$ https://github.com/Expensify/App/issues/54142
PROPOSAL: N/A
Tests

Log in to account which is guide

Optionally

Type Onyx.merge('user', {isGuide: true}); if Onyx is exported to window,
Refresh

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

Offline tests

Same as in Tests
QA Steps

// TODO: These must be filled out, or the issue title must include "[No QA]."
Same as in Tests

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

PR Author Checklist

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

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

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

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

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

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

[x] If any non-english text was added/modified, I used JaimeGPT to get English > Spanish translation. I then posted it in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

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

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

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

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

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

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

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

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] I added unit tests for any new feature or bug fix in this PR to help automatically prevent regressions in this user flow.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
N/A

Android: mWeb Chrome
N/A

iOS: Native
N/A

iOS: mWeb Safari
N/A

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/0f573131-d0c5-40f1-b7b4-692d69ae0de6

MacOS: Desktop
N/A

The test that is failing in CI passes locally

@jnowakow thank you very much for all the details. Just a NAB: Usually code that runs during rendering should be just that: rendering code. What about adding a wrapper component like:
const PusherInitializer = (props) => {
  const [initialized, setInitialized] = useState(false);
  useEffect(() => {
     initializePusher();
     setInitialized(true);
  }, []);

  return initialized ? props.children : <Some spinner>;
}

Not sure if it introduced other problems though.

Yeah but in this case it would case screen to blink as we would wait with rendering whole app before Pusher initialises. So it's not a case here. But I think I have a better solution. I'll update the PR shortly

@aldo-expensify now it works as it should. All subscription logic is handled by Pusher and React components don't care if its initialisation state.

Reviewer Checklist

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

https://github.com/user-attachments/assets/1d569440-bb8f-46fe-b866-04de4749d4df

MacOS: Desktop

@s77rt done 🫡

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

Thank you for handling!


Specify shell or allow setting all jobs to automatically source a file/profile?

Hey there,
I seem to be having a bit of an issue finding a reliable way to access environmental variables and functions from jobs. I see that the jobs return back $SHELL as /bin/sh. Is there some way I can make all jobs run under a different shell (such as zsh) so that it can automatically pick up my .zshenv file or if not that, at least pick up the /etc/bash.bashrc or something along those lines, straight from the cmd line within the job file so that I don't have to make each job individually source it?
Thanks,
-MH

I just thought of this, so I wanted to post it while I was thinking of it. Being able to have settings such as the ones below would be awesome.
prefs:
  shell: /bin/zsh # Shell for all jobs
  source: $HOME/.jobberenv # Sourced for all jobs

jobs:
  myjob1:
     cmd: $HOME/scripts/job1.zsh
     source: $HOME/.job1env # Sourced for specific job
     time: '*/30 * * * * *'
     onError: Continue
  myjob2:
     cmd: $HOME/scripts/job1.xsh
     shell: /usr/bin/env xonsh # Shell for this job
     source: $HOME/.job2env # Sourced for specific job
     time: '*/30 * * * * *'
     onError: Continue


test: remove some useless cases in the test

Problem
some changes for bumping Rust version: https://github.com/anza-xyz/agave/pull/1309
test_new_from_file_crafted_executable is failed on nightly-2024-05-02 (run on linux). I have bisected it. it looks like everything works fine before nightly-2024-03-27.
the reproducible command is:
RUST_BACKTRACE=all cargo +nightly-2024-05-02 test --package solana-accounts-db --lib -- append_vec::tests::test_new_from_file_crafted_executable::storageaccess_mmap_expects --exact --show-output

the error I got:
thread 'append_vec::tests::test_new_from_file_crafted_executable::storageaccess_mmap_expects' panicked at accounts-db/src/append_vec.rs:1416:29:
This didn't occur if this test passed.
stack backtrace:
   0: rust_begin_unwind
             at /rustc/c987ad527540e8f1565f57c31204bde33f63df76/library/std/src/panicking.rs:652:5
   1: core::panicking::panic_fmt
             at /rustc/c987ad527540e8f1565f57c31204bde33f63df76/library/core/src/panicking.rs:72:14
   2: solana_accounts_db::append_vec::tests::test_new_from_file_crafted_executable::{{closure}}
             at ./src/append_vec.rs:1416:29
   3: solana_accounts_db::append_vec::AppendVec::get_stored_account_meta_callback
             at ./src/append_vec.rs:549:22
   4: solana_accounts_db::append_vec::tests::test_new_from_file_crafted_executable
             at ./src/append_vec.rs:1398:13
   5: solana_accounts_db::append_vec::tests::test_new_from_file_crafted_executable::storageaccess_mmap_expects
             at ./src/append_vec.rs:1366:5
   6: solana_accounts_db::append_vec::tests::test_new_from_file_crafted_executable::storageaccess_mmap_expects::{{closure}}
             at ./src/append_vec.rs:1366:38
   7: core::ops::function::FnOnce::call_once
             at /rustc/c987ad527540e8f1565f57c31204bde33f63df76/library/core/src/ops/function.rs:250:5
   8: core::ops::function::FnOnce::call_once
             at /rustc/c987ad527540e8f1565f57c31204bde33f63df76/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Summary of Changes
if I understand correctly, we tried to convert a byte to bool here. I think assert!(!account.sanitize_executable()); has already covered the case and catch it. do we want to keep those undefined behaviors checking? I lean to remove them if we don't have similar use cases in the real world. (or maybe get them like a doc and don't need to run them)
(thanks @brooksprumo for sharing the original issue https://github.com/solana-labs/solana/pull/26009)

It probably would be good to root-cause what happened with nightly-2024-03-26 that caused this to start failing.
ah! sorry, the first error version is nightly-2024-03-27. just updated the PR body!

I'll look around this today. so, this test is failing only in coverage with recent nightly as seen at #1309

I'll look around this today. so, this test is failing only in coverage with recent nightly as seen at #1309

never mind, i remember nightly is used only for coverage, not other ci steps...

https://github.com/rust-lang/rust/compare/5f2c7d2bfd...47ecded35

<EMAIL_ADDRESS>
I'm now pretty sure this is the commit which changed this test behavior: https://github.com/rust-lang/rust/pull/122849

the pr started to omit emitting noundef attribute for -O0 build (ie. our nighty coverage build is affected)
and lack of the attribute inhibits optimizations based on the assumption that there should be no undefined values
https://llvm.org/docs/LangRef.html#parameter-attributes:

noundef
This attribute applies to parameters and return values. If the value representation contains any undefined or poison bits, the behavior is undefined. Note that this does not refer to padding introduced by the type’s storage representation.

https://llvm.org/docs/LangRef.html#undefined-values:

Note
A ‘poison’ value (described in the next section) should be used instead of ‘undef’ whenever possible. Poison values are stronger than undef, and enable more optimizations. Just the existence of ‘undef’ blocks certain optimizations (see the examples below).

i haven't actually dumped llvm ir but I think 0b1111_1110 for bool definitely falls into the category of undefined or poison here.
With this all background info, the conclusion is that: the test has been testing the existence of UB. And that UB was caused by optimizations leveraging noundef. But newer nightly is now preventing these optimizations by omitting noundef. so the problematic optimization has gone, so the ub has. ;)


In View Request

im trying to use this package but itds forcing me to use sheets to get the done callback can we get a customization for blending it in view? maybe onDateChanged callbacks?

It is not clear what you mean?
You can add either screenshots or a code fragment.
Or describe a specific case that you would like to implement but cannot


Firebase credentials are in plain view in the code

You probably don't want your database credentials lying around like this :D

Hi, thanks for the feedback. I did a bit of research on that ... and it seems that those info are not a security issue, even though they are publicly visible. They only identify the firebase application and the use of the api key is restricted to my website only.